search

geoneutrinos / reactors

Port/cleanup of the reactor page as an independent repo
2 stars 1 forks source link

Negative Geonu rates displayed if Mantle Flux set to negative value- restrict Mantle Flux to > = 0 #125

Closed stdye closed 3 years ago

stdye commented 3 years ago

Surprise!

Detector set to IMB All cores off except Perry 2018 LFs Geonu crust and mantle off (I thought)

Geonu rate = -10.1 NIU (-8.0 U and -2.1 Th)

mantle flux inadvertently set to -110000

stdye commented 3 years ago

Mantle flux inadvertently set to negative value.

Restrict Mantle Flux values to > = 0

stdye commented 3 years ago

Put sliders on the mantle flux values, too.

Flux- default 1e6, max 5e6, min 1e5, increment 1e5 Th/U ratio- default 3.9, max 10, min 1, increment 0.1 K/U ratio- default 1e4, max 3e4, min 1e3, increment 1e3

This prevents negative values, leading to negative rates

stdye commented 3 years ago

While Update geonu.jsx #145 prevents the up/down increment controls on the right of the value boxes from putting in unwanted non-physical values, it does not prevent a user from typing in negative values. This issue needs to reopen.

stdye commented 3 years ago

It would be okay to disable typed in values altogether for the mantle 238U flux, mantle Th/U, and mantle K/U inputs.

Using a slider, like on elastic scattering T_min, for these values would work well.

stdye commented 3 years ago

I found this on the T_min thread...

Can we get sliders on the mantle flux values, too?

Flux- default 1e6, max 5e6, min 1e5, increment 1e5 Th/U ratio- default 3.9, max 10, min 0.1, increment 0.1 K/U ratio- default 1e4, max 3e4, min 1e3, increment 1e3

This prevents negative values, leading to negative rates. See Issue #125