Removes second line of Cycle code in importMIDN. #114
Conversation
Closes issue #113.
|
I think the issue is that Cycles2 is used for the NCBN parks as they are on a slightly different cycle. |
|
Cycles2 is there because parks in NCBN are off from MIDN by a year, although the visualizer doesn't treat NCBN parks differently yet (I think), so it should be okay to delete that line of code for now. |
|
Actually, I just found places in importMIDN that call in Cycles2: Lines 186, 294, 321. To make this work for the PennState shiny server, you'll need to change those to InCycles. I'm not sure if this is a change we want to make with the NPSForVeg package just yet though, since it's technically not the correct cycle info for NCBN's parks. I'll leave that up to JP whether or not to make this change to the master NPSForVeg package. I'm also curious what this has been causing problems with the PSU shiny server when it runs fine on my computer. Will this be a problem with the NPS server at FOCO too? |
|
OK. I will try putting a Cycle2.csv in the MIDN Data folder on the PSU
server that is identical to MIDN's Cycle.csv and see if that fixes the
issue.
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Eastern Rivers and Mountains Inventory & Monitoring Network
National Park Service
426 Forest Resources Building
University Park, Pennsylvania 16802
814-441-9643
[email protected]
…On Wed, Jun 13, 2018 at 11:14 AM, Kate Miller ***@***.***> wrote:
Actually, I just found places in importMIDN that call in Cycles2: Lines
186, 294, 321. To make this work for the PennState shiny server, you'll
need to change those to InCycles. I'm not sure if this is a change we want
to make with the NPSForVeg package just yet though, since it's technically
not the correct cycle info for NCBN's parks. I'll leave that up to JP
whether or not to make this change to the master NPSForVeg package. I'm
also curious what this has been causing problems with the PSU shiny server
when it runs fine on my computer. Will this be a problem with the NPS
server at FOCO too?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#114 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ATRa2pfJOLOFx3Vr_ZF5vkR1hZSZRzDzks5t8SxKgaJpZM4UmN-B>
.
|
|
Putting in an identical Cycle2.csv ile should work fine. Should not be a problem for FOCO server. |
Closes #113.