Skip to content

[in1Utils/out1Plot/runScripts] Function for generating generic debris-flow topography#54

Open
JuLa96 wants to merge 5 commits intomasterfrom
JL_GenericDebrisFlowTopo
Open

[in1Utils/out1Plot/runScripts] Function for generating generic debris-flow topography#54
JuLa96 wants to merge 5 commits intomasterfrom
JL_GenericDebrisFlowTopo

Conversation

@JuLa96
Copy link
Contributor

@JuLa96 JuLa96 commented Feb 12, 2026

A function is added to in1Utils/generateTopo.py for generating a generic debris-flow topography.
Local subdirectories with dependencies and debrisframe-specific content are also created: out1Plot/outTopo.py and runScripts/runGenerateTopo.py.
in1Utils/generateTopoCfg.ini is set to default parameters that correspond to a data-calibrated terrain model.

@JuLa96 JuLa96 requested review from PaulaSp3 and fso42 February 12, 2026 08:59
@JuLa96 JuLa96 self-assigned this Feb 12, 2026
@qltysh
Copy link
Contributor

qltysh bot commented Feb 12, 2026

All good ✅

@qltysh
Copy link
Contributor

qltysh bot commented Feb 12, 2026

Qlty

Coverage Impact

This PR will not change total coverage.

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

Copy link
Contributor

@PaulaSp3 PaulaSp3 left a comment

Choose a reason for hiding this comment

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

In general, delete all functions that you just copied from avaframe (we co not need the dublicates and can just execute avaframe's functions) as I suggested it for getParabolaParams().

@JuLa96 JuLa96 requested a review from PaulaSp3 February 12, 2026 11:17
Copy link
Contributor

Choose a reason for hiding this comment

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

@fso42: since this file is a dublicate from the corresponding file in avaframe/in3Utils, should we try to avoid dublication? But the only way I know is just using the CfgIni file from avaframe which is not user friendly??

Copy link
Contributor

Choose a reason for hiding this comment

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

Is the function plotGeneratedDEM()? If yes, you can delete this file and execute avaframe.out3Plot.outTopo.plotGeneratedDEM()

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it is plotGeneratedDEM() but I added 'DFTA': 'generic debris-flow topography' in topoNames. Maybe we add this single entry in avaframe.out3Plot.outTopo.plotGeneratedDEM()?

Copy link
Contributor

Choose a reason for hiding this comment

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

I would suggest this!
@fso42 what do you think?


# Local imports
from debrisframe.in1Utils import generateTopo as gT
from debrisframe.out1Plot import outTopo as oT
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
from debrisframe.out1Plot import outTopo as oT
from avaframe.out3Plot import outTopo as oT

Copy link
Contributor Author

Choose a reason for hiding this comment

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

See comment to debrisframe/out1Plot/outTopo.py above.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants