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

Modeler 2.5a: Read models and system yaml files [ANT-2306] #2540

Open
wants to merge 42 commits into
base: develop
Choose a base branch
from

Conversation

payetvin
Copy link
Contributor

@payetvin payetvin commented Dec 19, 2024

Add the main modeler as a new binary
Directory loadFiles added, used to call previous work to read yaml files by handling I/O and errors related to yaml parsing

@payetvin payetvin self-assigned this Dec 19, 2024
@payetvin payetvin changed the title Modeler 2.5: Read models and system yaml files Modeler 2.5a: Read models and system yaml files [ANT-2306] Dec 19, 2024
Copy link
Member

@flomnes flomnes left a comment

Choose a reason for hiding this comment

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

Good start ! Now we need a main

src/solver/modeler/loadFiles/CMakeLists.txt Outdated Show resolved Hide resolved
src/solver/modeler/loadFiles/readLibraries.cpp Outdated Show resolved Hide resolved
@payetvin payetvin marked this pull request as ready for review December 20, 2024 10:03
@payetvin payetvin requested a review from flomnes December 30, 2024 11:40
}
catch (const YAML::Exception& e)
{
logs.error() << "Error while parsing the yaml parameters file";
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
logs.error() << "Error while parsing the yaml parameters file";
const auto fullPath = studyPath / "parameters.yml";
const std::string paramStr = IO::readFile(fullPath);
// ...
logs.error() << "Error while parsing parameter file " << fullPath;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants