Skip to content

Comments

⬆️ Update workflows and development dependencies#382

Merged
burgholzer merged 15 commits intomainfrom
config-updates
May 22, 2025
Merged

⬆️ Update workflows and development dependencies#382
burgholzer merged 15 commits intomainfrom
config-updates

Conversation

@burgholzer
Copy link
Member

This PR fixes the workflow references after the move to the new GitHub organisation.
It also updates all development dependencies as well as the uv lock file.

Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
@burgholzer burgholzer added the dependencies Pull requests that update a dependency file label Apr 23, 2025
@burgholzer burgholzer self-assigned this Apr 23, 2025
@github-project-automation github-project-automation bot moved this to In Progress in MQT Apr 23, 2025
@github-project-automation github-project-automation bot moved this to In Progress in MQT Compilation Apr 23, 2025
@codecov
Copy link

codecov bot commented May 21, 2025

Codecov Report

Attention: Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 87.0%. Comparing base (dc9ddf4) to head (2642735).
Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
src/mqt/predictor/rl/predictor.py 66.6% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##            main    #382     +/-   ##
=======================================
- Coverage   87.3%   87.0%   -0.3%     
=======================================
  Files         12      12             
  Lines        930     936      +6     
=======================================
+ Hits         812     815      +3     
- Misses       118     121      +3     
Flag Coverage Δ
python 87.0% <66.6%> (-0.3%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@flowerthrower
Copy link
Collaborator

@burgholzer @nquetschlich, it seems like reducing the training load (even further) by fixing more training parameters and de-randomizing the stable baselines policy (through a dedicated seed function) did the trick! All tests now pass and take (roughly) the same time on all supported platforms.

@nquetschlich
Copy link
Collaborator

@burgholzer @nquetschlich, it seems like reducing the training load (even further) by fixing more training parameters and de-randomizing the stable baselines policy (through a dedicated seed function) did the trick! All tests now pass and take (roughly) the same time on all supported platforms.

I am happy to hear that :) (and hope, that this is a lasting solution)

Copy link
Member Author

@burgholzer burgholzer left a comment

Choose a reason for hiding this comment

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

This is great! Thanks for digging into this further. Let's get this merged to finally unblock the progress on this repository.

@burgholzer burgholzer merged commit d1d72b1 into main May 22, 2025
23 of 24 checks passed
@burgholzer burgholzer deleted the config-updates branch May 22, 2025 17:43
@github-project-automation github-project-automation bot moved this from In Progress to Done in MQT Compilation May 22, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in MQT May 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants