Skip to content

chore: TestPython Achille#149

Draft
abossu-aneo wants to merge 6 commits intomainfrom
testpython
Draft

chore: TestPython Achille#149
abossu-aneo wants to merge 6 commits intomainfrom
testpython

Conversation

@abossu-aneo
Copy link
Copy Markdown
Contributor

Motivation

Adding my version of client/worker in order to be reviewed and upgraded.

Description

This test is my first attempt of using parallelization in ArmoniK. Tasks are creating parts of a larger python list that is constructed by aggregating those small parts.

Testing

Tested some cases where it should brake, modified the code to resolve it. But I plan to make my computation more reliable by using dictionaries instead of string or list.

Impact

None.

Additional Information

Beside some the mistakes I did, please also specify areas in the code that needs clarifications or modifications concerning code formatting.

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • I have thoroughly tested my modifications and added tests when necessary.
  • Tests pass locally and in the CI.
  • I have assessed the performance impact of my modifications.

@abossu-aneo abossu-aneo requested a review from aneojgurhem April 2, 2025 09:37
@abossu-aneo abossu-aneo changed the title TestPython Achille chore : TestPython Achille Apr 2, 2025
@abossu-aneo abossu-aneo changed the title chore : TestPython Achille chore: TestPython Achille Apr 2, 2025
Copy link
Copy Markdown
Contributor

@aneojgurhem aneojgurhem left a comment

Choose a reason for hiding this comment

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

I think you should write some documentation to explain what does your application

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.

2 participants