You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@jimianelli raised a good point, accompanied with a pull request #2, asking whether the data.R script should start with a call to taf.bootstrap().
One thing that's important - but not quite documented yet - is that the bootstrap procedure should not be run every time the analysis is run.
This is because in many cases, the bootstrap procedure fetches data from databases. We want to be able to always run with the same data, even when the underlying databases keep changing. So things inside the bootstrap folder are in a "glass cage", a snapshot of the data and software.
For this reason, we could for example have a minimal bootstrap.R file that contains a single line taf.bootstrap(). As another alternative, I've created a simple README that guides users to run taf.bootstrap() before sourceAll(). The README still doesn't convey that the bootstrap procedure should not be run every time.
When and how the bootstrap is run, this is something we'll take a closer look at in the next weeks.
The text was updated successfully, but these errors were encountered:
@jimianelli raised a good point, accompanied with a pull request #2, asking whether the
data.R
script should start with a call totaf.bootstrap()
.One thing that's important - but not quite documented yet - is that the bootstrap procedure should not be run every time the analysis is run.
This is because in many cases, the bootstrap procedure fetches data from databases. We want to be able to always run with the same data, even when the underlying databases keep changing. So things inside the bootstrap folder are in a "glass cage", a snapshot of the data and software.
For this reason, we could for example have a minimal
bootstrap.R
file that contains a single linetaf.bootstrap()
. As another alternative, I've created a simple README that guides users to runtaf.bootstrap()
beforesourceAll()
. The README still doesn't convey that the bootstrap procedure should not be run every time.When and how the bootstrap is run, this is something we'll take a closer look at in the next weeks.
The text was updated successfully, but these errors were encountered: