New schema for the GeoModel SQLite database and updated I/O

==> NOTE: This is a follow-up MR of !317 (closed).

I had to rebase the branch used in !317 (closed) onto main to fix errors raised in the rebase step of the Athena build. But that caused a deep rewrite of the commit history of the original development branch. To avoid affecting the original branch and to remove unnecessary clutter, I renamed the rebased branch and opened a new MR. So, we can test the new rebased branch in the Athena build without modifying the original development branch.

For all details and changes, see the description and the comments of !317 (closed).

Cc: @tsulaia and @boudreau

added GeoModelIO GeoModelIOHelpers labels

changed the description

mentioned in merge request !317 (closed)

changed the description

Hi @rbianchi,

Thanks for following up on this!

Well, our old friend build_geomodel_athena is still acting up I think I should repeat the exercise with the full build of Athena on top of your branch to ensure there are no unexpected issues with it. If everything goes as expected, we can merge it and eventually cut 6.0.0.

I will keep you posted.

I guess, !317 (closed) can be closed at this point, no?

Regarding our “old pipeline friend”, in case I didn’t mention it, Attila plans to fix this on the Athena side after our merge request goes in.

My tests with building Athena and AthSimulation projects on top of this branch were successful. From my perspective, it's OK to merge it. Thanks

added 63 commits

• 06180658...620cee98 - 2 commits from branch main
• 620cee98...e7e22d85 - 51 earlier commits
• 6f5acd99 - Write Parent-Child connections into the new DB schema
• 5fa559ea - Fix safety tests
• 0a73db25 - Move the I/O tests to a package
• 3f4e2b7b - Adapt I/O tests to new DB schema: add all the new tables. And add two new helper functions.
• f0c4a340 - Add getters to retrieve number of nodes of the new tables introduced with the new DB schema
• e0245cbe - Clean I/O tests and add better log messages
• 025f675a - Add getters to retrieve number of shapes with the new DB schema
• 68b45ed1 - Increase debug level to reduce clutter; add new log messages
• 83065ea8 - Fix read of Materials
• ef28ad66 - Fix errors from git rebase

Compare with previous version

What now? Another round of tests against a full Athena Build?

Nope just merge ;)

I was about to ask what are these 63 commits, then I see that in fact only two files have changed and it looks innocent. So I am going ahead. But I do hope to learn how it takes 63 commits in order to change 2 files.

Here we go.

merged

mentioned in merge request !333 (closed)