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.

Sign up for GitHub

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

Jump to bottom

Feature/unst 8528 sources sinks converter #721

Open
wants to merge 78 commits into
base: feature/unst-8490-621_extforce_converter-boundary-condition
Choose a base branch
from
Open

Feature/unst 8528 sources sinks converter #721

wants to merge 78 commits into from

Conversation

MAfarrag
Copy link

No description provided.

MAfarrag added 19 commits December 5, 2024 11:56
@MAfarrag
add a pull request template
5211d96
@MAfarrag
correct test folder name from extold to ext
1726a38
@MAfarrag
optimize the ext.models.Boundary.forcing property to have one retur…
d0b183e 
…n statement at the end of the property.
@MAfarrag
test the ExtOldParametersQuantity class
d68e75d
@MAfarrag
correct assert statement is test_ext_old_initial_condition_quantity…
5db0ceb 
… test
@MAfarrag
move the TestModels.TestLateral from thr test_ext.py to a separate te…
99d47d4 
…st file test_laterals.py
@MAfarrag
correct the test file description
8e5314f
@MAfarrag
reformat the test_boundary.py file
f4246d2
@MAfarrag
clean the test file from unused imported functions
b3c92b1
@MAfarrag
remove unused class ext.models.InitialConditions
eef3351
@MAfarrag
add reference to the initial condition documentation in the `InitialF…
de14bf0 
…ield` class
@MAfarrag
add ParametersConverter class and test
d0ee415
@MAfarrag
merge the BaseConverter class to the converters module
656b112
@MAfarrag
test MeteoConverter
1e3552a
@MAfarrag
merge converter_factory module to the converters module
31a0a99
@MAfarrag
merge the enum_converters module to utils
7387036
@MAfarrag
merge the __contains__ to the ConverterFactory class
c4e5892
@MAfarrag
abstract duplicate lines in the parameter and initial condition conve…
540f4e4 
…rters
@MAfarrag
merge changes from base branch
25bd8c5
@MAfarrag MAfarrag changed the base branch from main to feature/unst-8490-621_extforce_converter-boundary-condition December 12, 2024 10:14
@MAfarrag
add ExtOldSourcesSinks class for the source and sinks quantities
56930f9
@MAfarrag
test ExtOldSourcesSinks class for the source and sinks quantities
5092c75
@MAfarrag
Merge branch 'feature/unst-8490-621_extforce_converter-boundary-condi…
7556303 
…tion' into feature/unst-8528-sources-sinks-converter
@MAfarrag
fix using mutables as default value, and use absolute imports
75988fb
@MAfarrag
abstract the conversion of the interpolation related info in a separa…
726f01b 
…te function
@MAfarrag
add docstring and test for the convert_interpolation_data method
8779cb7
@MAfarrag
add reference to documentation for the tim model
6d6dad0
@MAfarrag
remove unused fields in the TimParser
f17b36c
@MAfarrag
move the convert_interpolation_data from the converters.py to the u…
55ad838 
…tils.py
@MAfarrag
move the convert_initial_cond_param_dict from the converters.py to …
ffa8a95 
…the utils.py
@MAfarrag
test SourceSinkConverter.parse_tim_model with default case
5934485
@MAfarrag
move the SourceSinkConverter tests to a separate test file
8847cc0
@MAfarrag
add missing test files
0396cb4
@MAfarrag
add all cases of the temperature/salinity in the external forcing fil…
3575aba 
…e wrt the tim file
@MAfarrag
add no temperature no salinity to the SourceSink converter
966f8d0
@MAfarrag
make the discharge attribute not optional
6b530a5
@MAfarrag
replace equality be is to check None
0c2b34e
@MAfarrag
move test_extold.py to a sub dir
545a532
@MAfarrag
move TestExtForcing to test_ext_forcing.py
d2bceaa
@MAfarrag
reformat tests
7c92028
@MAfarrag
add quantities property to the ExtOldModel
ebbf589
@MAfarrag
add x and y properties to the PolyFile class
f822a9b
@MAfarrag
move the get_z_sources_sinks method from the SourceSinkConverter …
f06fc1f 
…to the `PolyFile` class
@MAfarrag
remove the discharge_salinity_temperature_sorsin from the old meteo…
12663a7 
… quantities
@MAfarrag
add root_dir as a property to the SourceSinkConverter
314c8a1
@MAfarrag
add source_sink to the attributes of the ExtModel as a list of `S…
6253b9b 
…ourceSink` models
@MAfarrag
add SourceSinkConverter and default test case
2128eee
@MAfarrag
add SourceSinkConverter and default test case
2e4ca9e
@MAfarrag
reformat the test_main_converter.py::TestUpdate and create separate…
8ee1bf6 
… `TestUpdateSourcesSinks`
@MAfarrag
rename test
e29c1d9
@MAfarrag
add an example
845950f
@MAfarrag
add documentations
17819da
@MAfarrag
use absolute imports
edaad5f
@MAfarrag
change the backup function to keep the whole name with the extension …
5656560 
…and add the .bak
@MAfarrag
clean
348c8d9
@MAfarrag
convert the ext_old_to_new_from_mdu function to a method from_mdu…
7267c60 
… in the `ExternalForcingConverter` class
@MAfarrag
add a verbose property the ExternalForcingConverter class
c9465d9
@MAfarrag
move logging to separate method
c296fa3
@MAfarrag
refactor the update method
eea08a1
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@MAfarrag