Skip to content

Conversation

@devotaaabel
Copy link
Contributor

@devotaaabel devotaaabel commented Jul 6, 2018

Relates to multiple issues in #1072

@devotaaabel devotaaabel added the ⚠️ DO NOT MERGE PR should not be merged! label Jul 6, 2018
@devotaaabel devotaaabel self-assigned this Jul 6, 2018
@devotaaabel devotaaabel force-pushed the devota-add-camera-state branch from b14e9d4 to 0e43353 Compare July 6, 2018 20:09
@devotaaabel devotaaabel force-pushed the devota-add-camera-state branch from 0e43353 to 1457657 Compare July 16, 2018 20:11
@danesfeder danesfeder added this to the 0.17.0 milestone Jul 23, 2018
@devotaaabel devotaaabel force-pushed the devota-add-camera-state branch from 1457657 to 7c33a86 Compare August 10, 2018 23:25
@danesfeder danesfeder modified the milestones: 0.17.0, 0.18.0 Aug 14, 2018
@danesfeder danesfeder modified the milestones: 0.18.0, 0.19.0 Sep 4, 2018
@devotaaabel
Copy link
Contributor Author

@danesfeder @Guardiola31337 Can I get some feedback on whether or not we want to go in this direction to handle camera state discrepancies?

@devotaaabel devotaaabel force-pushed the devota-add-camera-state branch from 7c33a86 to 2b00f2d Compare September 19, 2018 15:53
@devotaaabel devotaaabel force-pushed the devota-add-camera-state branch from 2b00f2d to 51527ef Compare September 19, 2018 16:41
@devotaaabel devotaaabel added the needs discussion Can't be started in current state, needs clarification. label Sep 19, 2018
@akitchen akitchen removed this from the 0.19.0 milestone Sep 24, 2018
@danesfeder
Copy link
Contributor

@devotaaabel can we follow up on this work and confirm that camera states are still not persisted through rotation? I think this may have been addressed by some of the camera work done in #1372

@devotaaabel
Copy link
Contributor Author

Closing because this is very outdated and would probably need to be re-worked from scratch if we decided to go forward with it.

@etl etl deleted the devota-add-camera-state branch April 22, 2021 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚠️ DO NOT MERGE PR should not be merged! needs discussion Can't be started in current state, needs clarification.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants