Skip to content

Bug: Interaction between simulation file path and results directory #217

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
carlaKC opened this issue Feb 7, 2025 · 3 comments
Closed
Assignees
Labels
bug Something isn't working Medium Priority
Milestone

Comments

@carlaKC
Copy link
Contributor

carlaKC commented Feb 7, 2025

Describe the bug

If you run the simulator with two relative paths, it'll look for the simulation file in the data dir rather than the path provided and fail:

sim-cli --clock-speedup 1000 -s "lvl_acinq/simln_peacetime.json" -d "lvl_acinq/peacetime/"

[sim_cli::parsing] Simulation file 'lvl_acinq/peacetime/lvl_acinq/simln_peacetime.json' does not exist.
Error: no simulation files found in /Users/carla/Work/jam-ln/data/lvl_acinq/peacetime.

We should check whether we've got a simfile first and only fall back to the dir if it's not specified.

@carlaKC carlaKC added bug Something isn't working Medium Priority labels Feb 7, 2025
@carlaKC carlaKC added this to the V2.4 milestone Feb 12, 2025
@alexzaitsev
Copy link
Contributor

As I'm almost done with #225, I'd like to work on this next.

@alexzaitsev
Copy link
Contributor

@carlaKC should it be closed?

@carlaKC
Copy link
Contributor Author

carlaKC commented Mar 17, 2025

@carlaKC should it be closed?

Yes! In future you can say "Fixes {issue number}" in your PR and then it'll auto-close the issue on merge!

@carlaKC carlaKC closed this as completed Mar 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Medium Priority
Projects
None yet
Development

No branches or pull requests

2 participants