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

Wrong z-coordinate for fire in cfast #16

Closed
akrasuski opened this issue Jul 24, 2019 · 7 comments
Closed

Wrong z-coordinate for fire in cfast #16

akrasuski opened this issue Jul 24, 2019 · 7 comments
Labels

Comments

@akrasuski
Copy link
Contributor

Z-coordinate for firein cfast.in (line FIRE) is badly calculated. Z should be calculated with relation to local room coordinates, instead of global building coordinates.
Please change also the ids for staircases.

@akrasuski akrasuski added the bug label Jul 24, 2019
@mimooh
Copy link
Member

mimooh commented Jul 24, 2019 via email

@akrasuski
Copy link
Contributor Author

akrasuski commented Jul 24, 2019

Bad lines:
ONEZ,1000001
HVENT,7,1000001,3,0.9,2.0,0.0,1.74,2,0
FIRE,7,11.82,1.73,4.52,1,TIME,0,0,0,0,medium

Good:
ONEZ,5
HVENT,5,7,3,0.9,2.0,0.0,1.74,2,0
FIRE,7,11.82,1.73,1.02,1,TIME,0,0,0,0,medium
cad_json.txt

@mimooh
Copy link
Member

mimooh commented Jul 26, 2019

Fixed in master. Hopefully works as requested. Demo/three/cad.json was updated to provide the test case for this issue.

@akrasuski
Copy link
Contributor Author

There is still a problem with fire z-dimension. It should also be relative to the room where the fire takes place.

@mimooh
Copy link
Member

mimooh commented Jul 27, 2019

Fixed in master

@akrasuski
Copy link
Contributor Author

I can't check it because of such an error:

Aamks is halting on error:
Traceback (most recent call last): File "aamks.py", line 22, in EvacMcarlo() File "/usr/local/aamks.master/montecarlo/evac_mcarlo.py", line 51, in init self._make_evac_conf() File "/usr/local/aamks.master/montecarlo/evac_mcarlo.py", line 187, in _make_evac_conf self._evac_conf['FLOORS_DATA'][floor]['EVACUEES'][e_id]['COMPA'] = (pos[2]) IndexError: tuple index out of range

@akrasuski
Copy link
Contributor Author

The issue solved this is other problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants