Skip to content

Create 19-count-resources.py #28

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

Closed
wants to merge 2 commits into from
Closed

Conversation

ELN-uzun
Copy link
Contributor

I actually made it run wit elabapi_python, which was not easy for me but in the end it worked out. I tested it in my testing team and with a productive used team on our instance

I actually made it run wit elabapi_python, which was not easy for me but in the end it worked out. I tested it in my testing team and with a productive used team on our instance
Copy link
Contributor

@NicolasCARPi NicolasCARPi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove all trailing whitespaces and whitespaces on lines (configure your editor to show them to you!)

2025-04-16-152448_955x631_scrot

Also, use the api_client like here:

# use the locally defined client.py module to get the api_client object, fully configured and ready to be used to instantiate api objects
from client import api_client

this way we avoid repeating all the configuration code (and the warning suppressing code is there already). It makes the example simpler to read, skipping the boilerplate.

I hope I have correctly understood your comments =)
@NicolasCARPi
Copy link
Contributor

Merged in #29 (there was still whitespaces to remove)

Thanks!

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

Successfully merging this pull request may close these issues.

2 participants