Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Calculations Testing #9

Open
ravenwilde opened this issue Jul 22, 2015 · 25 comments
Open

Calculations Testing #9

ravenwilde opened this issue Jul 22, 2015 · 25 comments

Comments

@ravenwilde
Copy link
Contributor

Please be as specific as possible when reporting Calculations errors. It is most helpful for us if you list the following:

  • The compound or premixed product that you tested
  • All other form fields that you had filled in

Thank you :)

@kisanjong
Copy link
Owner

I've posted to Barr report, ukaps and planted tank requesting volunteers for beta testing.

@fablau
Copy link
Collaborator

fablau commented Jul 22, 2015

Hey guys, I found something missing in the latest version:

http://rotalabutterfly.com/test/

If you want to find a solution of CSM+B which exceed the solubility of the compound, you don't get ay warning. Whereas my PHP backend is returning the "sol_error" flag, I can't see that shown in the new JS implementation. Jennifer, thoughts?

Here is what I tried:

75gl tank, DIY, CSM+B, a solution, 500ml container, 5ml dose (too little for solubility)

@kisanjong
Copy link
Owner

I will post comments as i receive them from testers.

"I tested for KNO3 for EI dosing a DIY solution.

However, the result only told me how many grams of dry ferts to add to my aquarium.

What I expected from the old version is that it should tell me how many grams/teaspoons to add to the water in the dosing container"

@kisanjong
Copy link
Owner

"Can we change the contact link to contact & feedback"

@kisanjong
Copy link
Owner

"I just tested the Test page and no output is provided for the concentrations."

@kisanjong
Copy link
Owner

Two comments now mentioned grams/teaspoons not just grams

@kisanjong
Copy link
Owner

Fab I've just tested your csm+b and get this

screenshot_2015-07-22-22-28-55

@fablau
Copy link
Collaborator

fablau commented Jul 22, 2015

Yes Jason, I get the same results, and they should be correct. What I meant is that solubility for that dosing (5ml) would have some issues, and the calculator should warn that. The PHP backend passes the variable "sol_error" set to 1 in case solubility issues are reached. I think the UI is just not handling that passed parameter. Does this make sense?

@fablau
Copy link
Collaborator

fablau commented Jul 22, 2015

Two comments now mentioned grams/teaspoons not just grams

I can add that calculation to the PHP backend, and return the conversion in tsp or tbsp as an additional variable.

@ravenwilde
Copy link
Contributor Author

"Can we change the contact link to contact & feedback"

I'd like to steer away from this - its overlong for a navigation link. What we could do instead is display a note below the calculator that says something along the lines of "This is an evolving project that is currently in beta, your feedback is appreciated, please use our [link]contact form[/link].

We could display the same/a similar message on the contact page itself.

Acceptable solution?

@kisanjong
Copy link
Owner

Sounds good to me Jennifer :)

@kisanjong
Copy link
Owner

"Also labeling urea as toxic is misleading. It's only toxic at a certain concentration like all of the elements."

@kisanjong
Copy link
Owner

Fab it makes sense now and sounds like a good idea to warn about this

@Audionut
Copy link

Would be nice to be able to choose the target element when calculating for "dose to reach target".

For instance, if I wanted to raise dGH by ???

edit: After the calculator is up and running, if you wanted to get really tricky, you could do something like.
"I want to target ??? dGH, using XXX and XXX at a Y:Y ratio"

@kisanjong
Copy link
Owner

Some more feedback from testers...

So, some general feedback/wishlist here after tinkering with it for a bit, some of which is inherited from Wet's original calculator... Thus far looks like a good start. Issues specific to the test calculator, most of which is probably due to the beta nature and may already be well known to you:

  1. Computing "Result of my dose" doesn't seem to work yet... (this is a beta, so not too surprising)

  2. Computing "Dose to reach target" doesn't allow non-integer ppm's... I often target in 0.1 ppm increments for things like Iron, and 0.5 increments for PO4.

  3. Computing EI for Millers microplex results in a rather cryptic toxicity warning: "toxic cu,163,Miller's MicroPlex,1.06,grams" I'm guessing that means Cu is toxic at 0.163ppm, which would be provided by 1.06 grams of Microplex in my 36 gallon tank, but the dose to reach EI Iron levels is 1.70 grams. (which results in 0.187ppm Cu.

Wishlist things that were issues in Wet's calculator:

  1. It has always bothered me that micronutrient mixes like millers and CSM+B are only regarded as iron supplements when being used to calculate dosing. I'd love to be able to do things like calculate "dose to reach a target" on nutrients other than just iron. So a picker that lets you change the target nutrient would be helpful.

  2. I'd love to have something like barr's gh booster added to DIY, particularly if you could target it based on dGH... (3 parts K2SO4, 3 parts CaSO4, 1 part MgSO4). 3) Calculations targeting MgSO4/CaSO4 to ppm of Mg/Ca or dGH would also be nice.

@ravenwilde
Copy link
Contributor Author

If you want to find a solution of CSM+B which exceed the solubility of the compound, you don't get ay warning. Whereas my PHP backend is returning the "sol_error" flag, I can't see that shown in the new JS implementation. Jennifer, thoughts?

I'll make sure I take care of this when I begin working on error messages/form validation, etc. I created an Issue so I won't forget.

@kisanjong
Copy link
Owner

Thanks :), one comment that keeps popping up is the toxicity warnings

@ravenwilde
Copy link
Contributor Author

I'm working on the language selection right now, I'll do the error flags and toxicity warnings next.

@kisanjong
Copy link
Owner

Nice :D

Do we have any Italians that can test this? ;)

@ravenwilde
Copy link
Contributor Author

We're going to need all the languages tested - Because the form labels on our calculator are slightly different than Wet's original I had to adjust the translations - I used Google translate to do so and they will definitely need double checking. I will put a note on the interface asking for help with this.

@fablau
Copy link
Collaborator

fablau commented Jul 26, 2015

Do we have any Italians that can test this? ;)

I am Italian guys! What do you want me to test?

@fablau
Copy link
Collaborator

fablau commented Jul 26, 2015

Jason, I think we can do pretty much everything you have described above... step-by-step we can do it! Jennifer: just let me know when you need me to step-in. Thanks :)

@fablau
Copy link
Collaborator

fablau commented Jul 26, 2015

Fab it makes sense now and sounds like a good idea to warn about this

That was taken from original Wet's implementation. We can change whatever we like. As you already said, this is a good start, but we can shape it as we like.

@kisanjong
Copy link
Owner

Hey Fab I know you are Italian hence my wink smile :)

It's all sounding great and the future addons are endless.

Thankyou both :D

@fablau
Copy link
Collaborator

fablau commented Jul 26, 2015

Hey Fab I know you are Italian hence my wink smile :)
It's all sounding great and the future addons are endless.
Thankyou both :D

Sorry, I missed that!! ;) Just woke up, still sleepy! :)

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

No branches or pull requests

4 participants