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

Contact and animations modifications #4

Merged
merged 14 commits into from
Sep 24, 2024
Merged

Conversation

odufour7
Copy link
Contributor

@odufour7 odufour7 commented Aug 24, 2024

  • I addressed your requests by adding docstrings, adjusting whitespaces, and updating the requirements.
  • I modified the contact tab to directly interact with the CSV contact file sourced from the MADRAS My Core Cloud.
  • I made some minor improvements to the animation style. Additionally, with the trajectories from the cloud, you can now view pedestrians from top-view cameras.
  • I completed the CCTV analysis in the tab analysis / density profile

Copy link
Contributor

@chraibi chraibi left a comment

Choose a reason for hiding this comment

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

Thanks Oscar, this looks nice!
Could you please rename other_datasets to something meaninful like for example "gps_trajectories".

Could you also fix the CI?

@odufour7
Copy link
Contributor Author

"Could you also fix the CI?" What does it mean "CI" ?

@odufour7
Copy link
Contributor Author

I added a white background to enhance readability when the web browser has a black background. And, I incorporated survey data and renamed the folders within the data directory

@chraibi chraibi merged commit 3966984 into MADRAS-crowds:main Sep 24, 2024
1 check failed
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