ncsco / pinemap-dss-help

Issue tracker for PINEMAP DSS
0 stars 0 forks source link

Suggestions for Seedling Deployment Tools #84

Closed hadinon closed 8 years ago

hadinon commented 8 years ago

Comments from Tomasz Koralewski about Seedling Deployment tools:

image

hadinon commented 8 years ago
daviswx commented 8 years ago

Ah, I think I understand what he's saying, and he might be correct. Since we're highlighting the range around the future (not the current) deployment zone/isotherm, you could theoretically move seedlings even farther north than the isotherm indicates. Otherwise, if you're moving it to more southerly locales, you're not getting the full benefit of the northward movement.

For the Seedling Sources tool, our approach of showing the range highlighting warmer temperatures should be correct, since you'd want to source your seeds from as warm/southerly of a location as possible to get the full benefit.

I also need to think over this a bit more, but maybe we should also ask Gary about it to make sure we're all on the same page.

daviswx commented 8 years ago

Possibly add some text to the menu noting that Projected Temperatures can only be displayed when a location is selected.

Proceed with updating the 5° ranges for the Seedling Markets tool. This will require adding a new color palette to THREDDS, but the other changes required for this should be straightforward.

hadinon commented 8 years ago

Bug report: After location is selected and you switch back to historical map display, it currently keeps displaying the "Future Emissions" options. We discussed not showing those options unless "Projected Temperatures" is being displayed.

daviswx commented 8 years ago

A note has been added in the main menu stating "(select a location to view projected temperatures)". This note disappears once a valid location is selected.

The bug with the Future Emissions option has been fixed. It now goes away when the Historical Temperatures are being viewed.

I will work on updating the color palettes for the 5° ranges so those will show the northward ranges.

daviswx commented 8 years ago

Quick update on this: The new, reversed color ramps for the Seedling Markets tool aren't in place just yet. I had them ready last week but John said they broke THREDDS, so we will do some troubleshooting to get them working.

daviswx commented 8 years ago

The reversed color ramps are now working for the Seedling Markets tool. FYI, the reason the palettes initially broke THREDDS is apparently because there is a character limit for palette file names. I had previously named them something like this:

pinemap_seed_deploy_range_high_base_reverse.pal (47 characters)

I changed them to be something like this:

pinemap_seed_deploy_reverse_high_base.pal (41 characters)

So apparently something between 41 and 47 characters is the limit.

I think that's all for this issue, unless we still need to contact Gary about anything. But after talking to Ryan last week, I think we were all in agreement that showing the ranges this way was correct.

hadinon commented 8 years ago

Sounds good, thanks for the update. The only other thing we might want to tweak is the FAQ for each tool. Should the "warmer" mentioned in the 2nd paragraph be changed to "cooler"?

daviswx commented 8 years ago

I tweaked the wording for the FAQ boxes to match what each tool shows.

Seedling Markets: "...seedlings can be moved to locations with annual minimum temperatures up to 5°F cooler without appreciably increasing the risk of cold damage."

Seedling Sources: "...seedlings can be moved from locations with annual minimum temperatures up to 5°F warmer without appreciably increasing the risk of cold damage."

daviswx commented 8 years ago

Suggestions from Ryan for the Seedling Sources tool:

daviswx commented 8 years ago

I have updated the title and source range names on the Seedling Sources tool as suggested by Ryan.

hadinon commented 8 years ago

As we discussed earlier, we may want to take a look more closely at the descriptions on these tools, e.g. 5 degree range explanations below the maps.

daviswx commented 8 years ago

Task: Change the wording in the 5° range legends on the sources tool. Ryan suggested this: "Current range for projected cold tolerance in the XXXX-YYYY period" The markets tool legend is fine.

daviswx commented 8 years ago

A slightly tweaked version of this new wording is now in place: "Current range of cold tolerance for the projected XXXX-YYYY period"

I am closing this issue, but the other suggested changes for the seedling deployment tools are still documented in #80.