PB-366 flatten weights

[janpetschexain]

References

• PB-366

Summary

• use flat weights instead of list of weights
• update tests accordingly
• update documentation accordingly
• removed hellonumproto examples and tests

tested by running a dockerized coordinator and local participants on both sdk examples.

Are there any open tasks/blockers for the ticket?

• none

---

Reviewer checklist

Reviewer agreement:

• Reviewers assign themselves at the start of the review.
• Reviewers do not commit or merge the merge request.
• Reviewers have to check and mark items in the checklist.

Merge request checklist

• Conforms to the merge request title naming XP-XXX <a description in imperative form>.
• Each commit conforms to the naming convention XP-XXX <a description in imperative form>.
• Linked the ticket in the merge request title or the references section.
• Added an informative merge request summary.

Code checklist

• Conforms to the branch naming XP-XXX-<a_small_stub>.
• Passed scope checks.
• Added or updated tests if needed.
• Added or updated code documentation if needed.
• Conforms to Google docstring style.
• Conforms to XAIN structlog style.

[wilk10]

i understand this has been implemented elsewhere in this way (xain_proto repo), but i would advise against using the np abbreviation because that is a convention reserved for the actual numpy package, which may cause some confusion. but it doesn't quite fall under the scope of this PR, i guess

[janpetschexain]

np was chosen explicitly as it stands for numpy (not numproto) to indicate directly that this is a protobuf implementation of numpy, as well as the associated utilites.

[wilk10]

if the only changes to this file are about the docstrings and don't change any functionality, then i wouldn't include this in the PR, otherwise it may cause confusion in the commit history

[janpetschexain]

since it doesn't change the content/meaning of the docstring but only the formatting, i consider this simple cleanup which shouldn't require a dedicated ticket.

[wilk10]

thanks! looks pretty good. only some comment for readability (i didn't test it locally though)

[finiteprods]

just minor comments, otherwise looks fine.

[PanicButtonPressed]

Looks good, but it seems the branch has to be rebased first.