Skip to content

Glide snow tool - #790

Merged
fso42 merged 4 commits into
masterfrom
glideSnowTool
Dec 22, 2022
Merged

Glide snow tool#790
fso42 merged 4 commits into
masterfrom
glideSnowTool

Conversation

@matthiasto

@matthiasto matthiasto commented Dec 1, 2022

Copy link
Copy Markdown
Contributor

Add cohesion option in the DFA module
Add triangular initialization method

documentation available here : https://docs.avaframe.org/en/glidesnowtool/

@matthiasto
matthiasto requested review from awirb and fso42 December 5, 2022 14:18
@fso42 fso42 added the enhancement New feature or request label Dec 6, 2022
@fso42 fso42 added this to the Version 1.4 milestone Dec 6, 2022
Comment thread avaframe/out3Plot/outDebugPlots.py
Comment thread avaframe/out3Plot/outDebugPlots.py
Comment thread avaframe/out3Plot/outDebugPlots.py
Comment thread avaframe/out3Plot/outDebugPlots.py
Comment thread avaframe/out3Plot/outDebugPlots.py
Comment thread avaframe/com1DFA/particleTools.py
Comment thread avaframe/com1DFA/particleTools.py
@codecov

codecov Bot commented Dec 6, 2022

Copy link
Copy Markdown

Codecov Report

Base: 75.34% // Head: 75.17% // Decreases project coverage by -0.16% ⚠️

Coverage data is based on head (8687bac) compared to base (7cfdf8c).
Patch coverage: 79.82% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #790      +/-   ##
==========================================
- Coverage   75.34%   75.17%   -0.17%     
==========================================
  Files          59       59              
  Lines       12570    12929     +359     
==========================================
+ Hits         9471     9720     +249     
- Misses       3099     3209     +110     
Impacted Files Coverage Δ
avaframe/com3Hybrid/com3Hybrid.py 0.00% <0.00%> (ø)
avaframe/out3Plot/outDebugPlots.py 9.73% <12.50%> (+0.37%) ⬆️
avaframe/com1DFA/particleTools.py 49.32% <16.66%> (-4.41%) ⬇️
avaframe/out3Plot/outCom1DFA.py 38.41% <16.66%> (-0.78%) ⬇️
avaframe/com1DFA/com1DFA.py 91.16% <84.21%> (-1.98%) ⬇️
avaframe/ana1Tests/simiSolTest.py 83.33% <85.56%> (ø)
avaframe/com1DFA/DFAfunctionsCython.pyx 87.75% <88.75%> (+0.38%) ⬆️
avaframe/com1DFA/deriveParameterSet.py 82.86% <90.47%> (+0.62%) ⬆️
avaframe/in1Data/getInput.py 97.66% <97.05%> (-1.71%) ⬇️
avaframe/in3Utils/cfgUtils.py 88.41% <97.87%> (+0.29%) ⬆️
... and 10 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Comment thread avaframe/com1DFA/com1DFACfg.ini Outdated
Comment thread docs/glideSnowToolCom1DFA.rst Outdated
Comment thread docs/glideSnowToolCom1DFA.rst Outdated
Comment thread avaframe/com1DFA/DFAfunctionsCython.pyx Outdated
Comment thread avaframe/com1DFA/DFAfunctionsCython.pyx Outdated
Comment thread docs/com1DFAAlgorithm.rst Outdated
Comment thread docs/com1DFAAlgorithm.rst Outdated
Comment thread docs/com1DFAAlgorithm.rst Outdated
Comment thread docs/com1DFAAlgorithm.rst Outdated
Comment thread docs/glideSnowToolCom1DFA.rst Outdated
Comment thread docs/com1DFAAlgorithm.rst Outdated
@fso42 fso42 self-assigned this Dec 21, 2022
Comment thread avaframe/com1DFA/DFAfunctionsCython.pyx Outdated
Comment thread avaframe/com1DFA/DFAfunctionsCython.pyx Outdated
Comment thread avaframe/com1DFA/DFAfunctionsCython.pyx Outdated
Comment thread avaframe/com1DFA/DFAfunctionsCython.pyx Outdated
Comment thread avaframe/com1DFA/com1DFA.py Outdated
Comment thread avaframe/com1DFA/com1DFA.py Outdated
Comment thread docs/com1DFAAlgorithm.rst
Comment thread docs/com1DFAAlgorithm.rst
Comment thread docs/com1DFAAlgorithm.rst
Comment thread docs/com1DFAAlgorithm.rst Outdated
@qlty-cloud-legacy

Copy link
Copy Markdown

Code Climate has analyzed commit 05afaa2 and detected 8 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 2
Duplication 6

The test coverage on the diff in this pull request is 73.7% (50% is the threshold).

This pull request will bring the total coverage in the repository to 73.2% (-0.2% change).

View more on Code Climate.

@fso42

fso42 commented Dec 22, 2022

Copy link
Copy Markdown
Contributor

Standard tests ok

@fso42
fso42 merged commit eb19f7e into master Dec 22, 2022
@fso42
fso42 deleted the glideSnowTool branch December 22, 2022 12:31
@fso42 fso42 mentioned this pull request Dec 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants