Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implicitly split parameters through add_param, corrected encode events to support empty iterator input #51

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

MattiasFredriksson
Copy link
Contributor

@MattiasFredriksson MattiasFredriksson commented Dec 28, 2022

Changes:

Notes:

  • Separated changes to follow PEP standard and .format() rules in a separate branch as it became a distraction from the other changes.
  • Writing parameters is slightly confusing, input expects Fortran format while reading returns data in C format. I have no intention to fix this until the remaining changes and documentation is merged.
  • Implicitly splitting parameters is slightly problematic since there is no function for reading them as a single parameter (e.g. see More POINT:LABELS #47 ).

@Peaj
Copy link

Peaj commented Jun 4, 2024

Is there any reason this has not been merged yet? Works perfectly for me fixing the issue with c3d files using more than 255 labels.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants