Skip to content

Conversation

@TraciPopejoy
Copy link
Contributor

@TraciPopejoy TraciPopejoy commented Jan 20, 2023

Hi NicheMapR coders!

I am running MNMs for many anuran species across the Southeast United States. I've parallelized my code by creating one function that runs one microclimate model and then has a for loop for an ectotherm model for each species.

  1. Thanks for adding the warning about depths in the micro_* functions. I feel a little proud that my ignorance might help future users.
  2. Because of the structure of the function described above, I need ectotherm to only rely on arguments passed to it. The new error catching message about time-starting causes a stack error now. I appreciate the addition of the warning message; this package has great, clear messages to aid users. I edited the error-catch added yesterday to rely on the metout argument instead of micro$metout. The changes reflected in the TraciPopejoy/NicheMapR branch fixed the stack error I received.

I hope this is an appropriate edit! I appreciate the addition of the warning message; this package has great, clear messages to aid users.

Thanks for your time and for building a great R package!
Traci

This is to aid in function stack calls, especially those that might be used to parallelize MNM code
remove reference of micro within function
@mrke mrke merged commit 374c60e into mrke:master Jan 20, 2023
@mrke
Copy link
Owner

mrke commented Jan 20, 2023

Great thanks Traci - that's what I should have done! I put that catch in there because I spent a few hours trying to resolve why the model was behaving badly for a simulation I was doing (with a half-hour timezone that was resulting in the first hour in metout not being zero).

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

Successfully merging this pull request may close these issues.

2 participants