Skip to content

Clean-up assign-example ids.#3625

Merged
alex-jansen merged 2 commits intomainfrom
less_execs
Sep 23, 2024
Merged

Clean-up assign-example ids.#3625
alex-jansen merged 2 commits intomainfrom
less_execs

Conversation

@MatthiasWiesmann
Copy link
Contributor

Clean up the assign-example-ids script so it can be called directly in python.

  • move all the logic into a function.
  • use logging systematically
  • use a dictionary structure to avoid iterating twice over all examples
  • only import the library, not the classes.
  • raise an error in the case where two examples have the same file, instead of overwriting.

@alex-jansen alex-jansen merged commit c084da0 into main Sep 23, 2024
@alex-jansen alex-jansen deleted the less_execs branch September 23, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants