search

PredictiveEcology / CBM_core

Other
0 stars 2 forks source link

plot scheduling #5

Closed achubaty closed 4 years ago

achubaty commented 4 years ago

plot scheduling was shuffled around in 27cf8881182810bcd9594b1aa8a5f44684ed5c09, so that annual and postSpinup events schedule plotting instead of allowing plots to be produced at regular intervals within the plot event.

Why was this done?

cboisvenue commented 4 years ago

The plotting was not being scheduled anymore. I could not see why. I changed it so it plotted and I could work at making sure plots were correct. I don't mind the plotting being scheduled by the plot event instead.

achubaty commented 4 years ago

Default .plotInterval is NA (which means no plotting), and no value was passed during simInit to override this default.

achubaty commented 4 years ago

Keeping all the poltting stuff together is easier to deal with and debug. Two events each scheduling a plot event can lead to things like duplicate plotting.

achubaty commented 4 years ago

I have updated the global Rmd file and the module.