Korf, Sascha Alexander

Results 39 comments of Korf, Sascha Alexander

Quick remark: In the benchmark get_subpopulation_combined is very slow, this maybe needs to be investigated.

Thank you! I mean i don't think its high priority, but as it is in the benchmark debuggging, it becomes unusable. Maybe we just think about a better way to...

Please remember to account for the benchmark, as testing is really computationally extensive :-D

I'll probably also rename epi to mio while i'm at it

There are several phrases to change and we need to decide if we want to rename it into mobility or movement, e.g.: - Person::migrate_to - migration - MigrationEdge - make_migration_sim...

We decided on: - mobility if not a verb - go (to) as a verb (comment functions with “mobility” keyword)

@reneSchm fixed it with the following code: ``` World(const World& other) : parameters(other.parameters) , m_persons() , m_locations() , m_trip_list(other.m_trip_list) , m_cemetery_id(add_location(LocationType::Cemetery)) { for (auto& origin_loc : other.get_locations()) { if (origin_loc.get_type()...

Works for me on windows: I did: Pull main clean memilio-simulation from any artifacts i had python -m venv epi_venv .\epi_venv\Scripts\activate pip install --upgrade pip pip install scikit-build pip install...

Alright, could you delete your virtual envorinment and all artifacts and follow the steps above? Apart from that i had the same error once and it worked after doing that....

Skimmed over that too, the workflow though looks easy enough so that it would be usable. But yes, as it isn't urgent I'd wait too.