Wip research time tracking #16
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* this move should give us better ability to control the flow of the program wrt parts that should be run outside the `go-core` procedure (e.g. the `exit-condition` reporter) * what was previously the `go` procedure is now named `go-core` and called by the new `go` procedure; the interface buttons and the default BehaviorSpace experiment have been adjusted accordingly * the `exit-condition` reporter is now directly evaluated in the `go` procedure and saved in a global variable (`g-exit-condition?`) in case the go procedure is called with the argument `true` (which is what the go-stop button does) * the go-stop button can now be interrupted the same way as the go button
* `fast-sharing` has been renamed to `fast-sharing?` to reflect its boolean nature. * `fast-sharing` is now determined with-local-randomness in the `go` procedure and handed over to `go-core` as an argument. This avoids distorting the rng when evaluating this reporter.
* Currently we don't know how much time researchers spend on each theory apart from when they are all converged on one theory * In ArgABM we use research time as basis for one of our diversity measures. To allow the same measurement in SocNet we introduce two new globals: `g-research-time` and `g-myscientists` and a new procedure `compute-popularity` which records how much scientists are on each theory during the run * The popularity plot in the interface utilizes the new global `g-myscientists` instead of computing the distribution itself every time. This reduces computational load (marginally)
* Our runs are supposed to end at the time when scientists finally converge, even though they might practically last longer. A snapshot of the research-time the scientists spent investigating each theory must therefore be stored at the point when researchers converge. * This snapshot of research-time is stored via the convergence-reporters together with- and in the same way as- other variables which are recorded at the point of convergence * The default BehaviorSpace experiment then uses this reporter to record the research time for each theory * Documentation for the two new globals has been added
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR implements the tracking of research time similar to what we call in ArgABM
research-time-monist. This allows us to see how much time scientists spent researching each theory over the course of a run. To accomplish this it adds a new reporter which records at the time scientists converge for the final time how much time they have spent on each theory during the run.This PR supersedes #15