Deltares / Deltares/ProbabilisticLibrary
list of sections is too long when du = 3 and max length u = 20
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 8
- Forks
- 1
- Avg merge
- 4d 18h
- Merged PRs (30d)
- 4
Description
Directional sampling with settings as normally used in Hydra-Ring, calculates too many sections for directions that are slowly decreasing and keep > 0.
The list of sections looks like:
u low = 0, 3, 6, 9, 12, 15, 18, 21, 20
u high = 3, 6, 9, 12, 15, 18, 20, 20, 40
Reporter: @epsig
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
The issue names no source file, test, or entry point. Start by reproducing the directional sampling case with du = 3 and maximum length u = 20, then trace where the section lists are built. Done means the lists no longer include the extra sections shown for slowly decreasing positive directions, with regression coverage for this case.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100