Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Operational updates #612

Merged
merged 15 commits into from
Dec 30, 2022
Merged

Operational updates #612

merged 15 commits into from
Dec 30, 2022

Conversation

jpn--
Copy link
Member

@jpn-- jpn-- commented Oct 3, 2022

A wide variety of updates have been made in the Sharrow PR, including core enhancements that change how ActivitySim work, and ancillary enhancements that add non-core features that are convenient for debugging and performance monitoring.

This PR isolates a limited number of non-core features so they can be add to version 1.1, allowing easier analysis and comparison against the sharrow changes coming for version 1.2:

  • memory_profile option, which launches time-sampling memory monitoring in a separate subprocess.
  • A more resilient resume_after that will fall back to restarting the model at the beginning if the selected checkpoint is not found (especially useful for setting the resume_after checkpoint as "_")
  • An --ext option for the CLI interface for activitysim run which allows the user to give the location of an extension package of add-ons for a model; this can obviate the need for the simulate.py script in most of the examples with add-on packages.

@jpn-- jpn-- requested review from dhensle and i-am-sijia October 3, 2022 13:37
@jpn-- jpn-- added this to the Phase 7 milestone Dec 13, 2022
Copy link
Contributor

@i-am-sijia i-am-sijia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@jpn-- jpn-- merged commit 58cf20e into ActivitySim:develop Dec 30, 2022
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