-
Notifications
You must be signed in to change notification settings - Fork 0
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
Running FloodAdapt on Linux #527
base: main
Are you sure you want to change the base?
Conversation
@LuukBlom can you take a look? We should chat on how we will deal with case-sensitive filepaths (with capitalization or without) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love the new config module!
I only have 1 real comment (about actually validating the paths point to exe/binaries that exist.
Before merging this PR, we need to remove all calls to the old config module in FloodAdapt-GUI as well. (global search for from flood_adapt.config
)
The old way using env vars is kind of clunky anyways and this does the same, but much better.
…es for Linux & Darwin to the system folder in the test database
…s. fix Settings usage in integrator scripts
This requires Pixi so moving it to blocked (#485) |
In order to run on linux, a few changes were made:
exposure
vs.Exposure
fiat
andsfincs
configurable ( no.exe
outside of windows)To do configuration I added
pydantic-settings
to our dependencies (maintained bypydantic
maintainers, which is already in our dependencies.)