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

78 Extract performance profiling cli. #88

Merged
merged 21 commits into from
Mar 5, 2025
Merged

Conversation

chenkins
Copy link
Contributor

@chenkins chenkins commented Nov 20, 2024

Changes

Related issues

Closes #78
Closes #59

Checklist

  • Tests are included for relevant behavior changes.
  • Documentation is added in the docs folder for relevant behavior changes. If you made important user-facing
    changes, describe them under the [Unreleased] tag in CHANGELOG.md.
  • New package dependencies are declared in the pyproject.toml file.
    Requirement files have been updated by running tox -e requirements.
  • Code works with all supported Python versions (3.10, 3.11 and 3.12). Checks run with all three version and are
    required to run successfully.
  • Code is formatted according to PEP 8 (an IDE like PyCharm can do this for you).
  • Technical guidelines listed in CONTRIBUTING.md are followed.

@chenkins chenkins requested a review from aiAdrian November 20, 2024 14:17
@chenkins chenkins force-pushed the 78-performance-evaluation-cli branch from 8421342 to d9d4da7 Compare November 20, 2024 15:58
@chenkins chenkins changed the title Extract performance profiling cli. 78 Extract performance profiling cli. Nov 20, 2024
@chenkins chenkins modified the milestones: 4.0.4, 4.0.5 Jan 17, 2025
@chenkins chenkins modified the milestones: 4.0.5, 4.0.6 Jan 30, 2025
aiAdrian
aiAdrian previously approved these changes Feb 3, 2025
@chenkins chenkins force-pushed the 78-performance-evaluation-cli branch from 7c58fa0 to 2d7f1e7 Compare February 19, 2025 14:25
@chenkins chenkins marked this pull request as ready for review February 19, 2025 16:10
@chenkins chenkins requested a review from a team as a code owner February 19, 2025 16:10
@chenkins chenkins requested a review from aiAdrian February 19, 2025 16:10
@chenkins chenkins mentioned this pull request Feb 19, 2025
7 tasks
@chenkins chenkins requested review from a team and Holzchopf February 20, 2025 12:33
@chenkins chenkins modified the milestones: 4.0.6, 4.0.5 Feb 20, 2025
Holzchopf
Holzchopf previously approved these changes Feb 21, 2025
Copy link
Contributor

@aiAdrian aiAdrian left a comment

Choose a reason for hiding this comment

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

I tested this changes also with flatland_policy_solver and flatland_railway_extension

works well

@chenkins chenkins force-pushed the 78-performance-evaluation-cli branch from 8e81b99 to 3498e88 Compare March 5, 2025 13:57
@chenkins chenkins enabled auto-merge (squash) March 5, 2025 13:57
@chenkins chenkins merged commit e61d56a into main Mar 5, 2025
7 checks passed
@chenkins chenkins deleted the 78-performance-evaluation-cli branch March 5, 2025 14:11
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.

Cli/notebooks for (performance) evaluation Include a performance test and report in the CI
4 participants