fix: adapt new func names

[tom-grivaud]

Just adapt the name of the function according to the recent edits. The new name was just not applied to sdf_to_nsjon...

Interrogation about a possible improvement:
Makes me think about something, what about including checks in CI to test the full pipeline ? I know it will take time but maybe it would only be necessary to merge from dev to a new "master/main" branch. I haven't thought about all the details about this kind CI checks but it could avoid us using version of NCK in which don't work... I had to made test manually and correct these to make sure the new NCK we're going to use would work, I think this kind of feature could be an interesting improvement.

If you guys agree and if you think this is doable I will write a new issue.

[gabrielleberanger]

Thank you for spotting this Tom 👍

Indeed, we are missing integration tests. It happened several times recently that we merged changes that were actually not functional, and we noticed it afterwards... As for the issue, don't you think that your suggestion overlaps the Issue #102 created by Benoit?

[tom-grivaud]

Thank you for spotting this Tom 👍

Indeed, we are missing integration tests. It happened several times recently that we merged changes that were actually not functional, and we noticed it afterwards... As for the issue, don't you think that your suggestion overlaps the Issue #102 created by Benoit?

Yes it is what Benoît was talking about. It wont be quick to do I guess but it is worth it. As you've said, it happened a lot recently and this kind of issue is critical for both current and new users.