Commit 007bbda
committed
Separate IRF truncation from smoothing and tapering
Previously, truncating the radiation impulse response function (RIRF) was bundled into the TaperedDirect mode -- you couldn't shorten the kernel without also enabling smoothing and tapering. Excitation IRF truncation was buried inside the irregular wave settings.
Introduce three independent, composable controls:
1. Excitation truncation time -- shorten the excitation IRF to ±T s
2. Radiation truncation time -- shorten the RIRF to [0, T] s
3. Kernel processing -- optionally smooth and/or taper the RIRF
Each can be used on its own or combined. All three are exposed via both the C++ API (SetExcitationTruncationTime, SetRadiationTruncationTime, SetRadiationKernelProcessing) and YAML configuration. Truncation works regardless of wave type.
YAML schema changes:
Old: convolution.mode, convolution.taper.end_time
New: excitation.truncation_time, radiation.truncation_time, radiation.smoothing, radiation.taper
Additional quality improvements:
- ExcitationRampType moved to hydrochrono::waves namespace (kLinear, kCosine)
- Frequency defaults extracted to named constants in IrregularWaveParams
- SG filter coefficients documented with citation, constexpr std::array
- Embed RadiationKernelProcessing directly in YAMLHydroData (was 7 flat fields)1 parent ba6219e commit 007bbda
16 files changed
Lines changed: 492 additions & 448 deletions
File tree
- data
- demos/run_hydrochrono/f3of/decay_dt3/inputs
- yaml/rm3
- include/hydroc
- config
- radiation
- waves
- src/hydro
- config
- force_components
- radiation
- waves
Lines changed: 6 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
14 | | - | |
15 | | - | |
| 13 | + | |
| 14 | + | |
16 | 15 | | |
17 | | - | |
18 | | - | |
| 16 | + | |
| 17 | + | |
19 | 18 | | |
20 | | - | |
21 | | - | |
| 19 | + | |
| 20 | + | |
22 | 21 | | |
23 | | - | |
24 | 22 | | |
25 | 23 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
11 | 20 | | |
12 | 21 | | |
13 | 22 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| 14 | + | |
14 | 15 | | |
15 | 16 | | |
16 | 17 | | |
| |||
23 | 24 | | |
24 | 25 | | |
25 | 26 | | |
26 | | - | |
27 | | - | |
28 | | - | |
29 | | - | |
30 | | - | |
31 | | - | |
32 | | - | |
33 | | - | |
34 | | - | |
35 | 27 | | |
36 | 28 | | |
37 | 29 | | |
| |||
49 | 41 | | |
50 | 42 | | |
51 | 43 | | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
| |||
60 | 60 | | |
61 | 61 | | |
62 | 62 | | |
63 | | - | |
| 63 | + | |
64 | 64 | | |
65 | | - | |
66 | | - | |
| 65 | + | |
| 66 | + | |
67 | 67 | | |
68 | 68 | | |
69 | | - | |
| 69 | + | |
70 | 70 | | |
71 | | - | |
72 | | - | |
73 | | - | |
74 | | - | |
| 71 | + | |
75 | 72 | | |
76 | 73 | | |
77 | | - | |
| 74 | + | |
78 | 75 | | |
79 | | - | |
80 | | - | |
81 | | - | |
82 | | - | |
83 | | - | |
84 | | - | |
| 76 | + | |
85 | 77 | | |
86 | | - | |
87 | | - | |
88 | | - | |
89 | | - | |
90 | | - | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
91 | 85 | | |
92 | 86 | | |
93 | 87 | | |
94 | 88 | | |
95 | | - | |
| 89 | + | |
96 | 90 | | |
97 | 91 | | |
98 | 92 | | |
| |||
103 | 97 | | |
104 | 98 | | |
105 | 99 | | |
106 | | - | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
360 | 360 | | |
361 | 361 | | |
362 | 362 | | |
363 | | - | |
| 363 | + | |
364 | 364 | | |
365 | | - | |
| 365 | + | |
366 | 366 | | |
367 | | - | |
368 | | - | |
| 367 | + | |
369 | 368 | | |
370 | | - | |
| 369 | + | |
| 370 | + | |
| 371 | + | |
| 372 | + | |
| 373 | + | |
| 374 | + | |
| 375 | + | |
| 376 | + | |
| 377 | + | |
| 378 | + | |
| 379 | + | |
| 380 | + | |
| 381 | + | |
| 382 | + | |
| 383 | + | |
| 384 | + | |
| 385 | + | |
| 386 | + | |
| 387 | + | |
| 388 | + | |
| 389 | + | |
| 390 | + | |
371 | 391 | | |
372 | 392 | | |
373 | 393 | | |
| |||
376 | 396 | | |
377 | 397 | | |
378 | 398 | | |
379 | | - | |
380 | | - | |
381 | | - | |
382 | 399 | | |
383 | 400 | | |
384 | 401 | | |
| |||
388 | 405 | | |
389 | 406 | | |
390 | 407 | | |
391 | | - | |
392 | | - | |
393 | 408 | | |
394 | 409 | | |
395 | 410 | | |
396 | 411 | | |
397 | 412 | | |
398 | | - | |
| 413 | + | |
399 | 414 | | |
| 415 | + | |
| 416 | + | |
400 | 417 | | |
401 | 418 | | |
402 | | - | |
| 419 | + | |
403 | 420 | | |
404 | | - | |
405 | | - | |
406 | | - | |
407 | | - | |
408 | | - | |
409 | | - | |
410 | | - | |
411 | | - | |
412 | | - | |
413 | | - | |
414 | | - | |
| 421 | + | |
415 | 422 | | |
416 | 423 | | |
417 | 424 | | |
| |||
576 | 583 | | |
577 | 584 | | |
578 | 585 | | |
579 | | - | |
| 586 | + | |
580 | 587 | | |
581 | 588 | | |
| 589 | + | |
| 590 | + | |
| 591 | + | |
582 | 592 | | |
583 | 593 | | |
584 | 594 | | |
585 | 595 | | |
586 | 596 | | |
587 | 597 | | |
588 | | - | |
589 | | - | |
| 598 | + | |
| 599 | + | |
590 | 600 | | |
591 | 601 | | |
592 | | - | |
593 | 602 | | |
594 | 603 | | |
595 | 604 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
8 | | - | |
| 7 | + | |
9 | 8 | | |
10 | 9 | | |
11 | 10 | | |
| |||
23 | 22 | | |
24 | 23 | | |
25 | 24 | | |
26 | | - | |
27 | | - | |
| 25 | + | |
| 26 | + | |
28 | 27 | | |
29 | 28 | | |
30 | 29 | | |
31 | 30 | | |
32 | 31 | | |
33 | 32 | | |
34 | 33 | | |
35 | | - | |
| 34 | + | |
36 | 35 | | |
37 | | - | |
38 | | - | |
39 | | - | |
40 | | - | |
41 | | - | |
42 | | - | |
43 | | - | |
44 | | - | |
45 | | - | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
46 | 39 | | |
47 | | - | |
48 | | - | |
| 40 | + | |
| 41 | + | |
49 | 42 | | |
50 | | - | |
51 | | - | |
52 | | - | |
53 | | - | |
54 | | - | |
55 | | - | |
56 | | - | |
57 | | - | |
58 | | - | |
59 | | - | |
60 | | - | |
61 | | - | |
62 | | - | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
63 | 60 | | |
64 | 61 | | |
65 | 62 | | |
| |||
0 commit comments