Skip to content
This repository has been archived by the owner on Dec 31, 2019. It is now read-only.

New core integration #40

Closed
carmenfan opened this issue Jul 17, 2015 · 2 comments
Closed

New core integration #40

carmenfan opened this issue Jul 17, 2015 · 2 comments
Assignees

Comments

@carmenfan
Copy link
Member

Integrate the GUI with the new core. This means ripping out manipulation logic from the GUI and adding those features within the new core library

@carmenfan carmenfan changed the title New core Integration New core integration Jul 17, 2015
@carmenfan carmenfan self-assigned this Jul 17, 2015
carmenfan added a commit that referenced this issue Jul 24, 2015
Ripped out the core so this is currently just a gui stub.
carmenfan added a commit that referenced this issue Aug 4, 2015
carmenfan added a commit that referenced this issue Aug 4, 2015
Note: Transformation seems borked at the moment
carmenfan added a commit that referenced this issue Aug 5, 2015
carmenfan added a commit that referenced this issue Aug 5, 2015
carmenfan added a commit that referenced this issue Aug 5, 2015
slightly buggy still.
carmenfan added a commit that referenced this issue Aug 5, 2015
carmenfan added a commit that referenced this issue Aug 6, 2015
carmenfan added a commit that referenced this issue Aug 10, 2015
carmenfan added a commit that referenced this issue Aug 10, 2015
carmenfan added a commit that referenced this issue Aug 10, 2015
possible seg faults trying to commit a scene that is already destroyed!
carmenfan added a commit that referenced this issue Aug 10, 2015
carmenfan added a commit that referenced this issue Aug 12, 2015
showing federated scenes on the other hand..... needs some work.
carmenfan added a commit that referenced this issue Aug 13, 2015
carmenfan added a commit that referenced this issue Aug 13, 2015
It's always been there, just not wired in
carmenfan added a commit that referenced this issue Aug 19, 2015
carmenfan added a commit that referenced this issue Aug 21, 2015
images aren't loading properly.
carmenfan added a commit that referenced this issue Aug 24, 2015
carmenfan added a commit that referenced this issue Aug 25, 2015
carmenfan added a commit that referenced this issue Aug 26, 2015
Where reposcenes are moved into core::model
carmenfan added a commit that referenced this issue Aug 26, 2015
carmenfan added a commit that referenced this issue Aug 27, 2015
carmenfan added a commit that referenced this issue Aug 28, 2015
repogui - bouncer already gives this error message so just give a
generic one in gui level
repo_logger - add fixme for future work
@carmenfan
Copy link
Member Author

At this point the new gui can be considered working as well as the current master with the new bouncer library (this version of the bouncer 3drepo/3drepobouncer@8ffc266) to replace the legacy core.

There are a few outstanding issues/differences to note:

  • Occulus is removed (as it's available on the web and gui right now is not considered a flashy tool. and also gui provides a web browser which technically means it already has occulus support)
  • Selection tree isn't supported (deferred until multipart integration is complete)
  • because there's no access to selection tree, PDF attachment doesn't work (not coded in yet! but should be simple)
  • Graph optimization is ignored right now (awaiting multipart integration)
  • export doesn't work too well, potentially an assimp problem (ISSUE Assimp Export doesn't really work. 3drepobouncer#10 on bouncer)
  • 3D Diff won't work (not ported). I'm not sure what we want to do with it

Things to do in the future:

  • This gui deseperately needs some refactoring and testing put in place
  • projectSettings needs to support multi group.
  • Whilst heavy processes are threaded out, there's still a lot of freezing going on. Probably need to multi thread some windows out? I don't know.

But the next step is revive the QT project files, multipart integration. Then this should be the new master. Everything mentioned above should be new ISSUES.

carmenfan added a commit that referenced this issue Sep 1, 2015
carmenfan added a commit that referenced this issue Sep 1, 2015
carmenfan added a commit that referenced this issue Sep 1, 2015
carmenfan added a commit that referenced this issue Sep 1, 2015
carmenfan added a commit that referenced this issue Sep 1, 2015
…that is setting a database icon instead of 3dRepo
carmenfan added a commit that referenced this issue Sep 2, 2015
carmenfan added a commit that referenced this issue Sep 4, 2015
This has gone wrong weird since we no longer track progress there.
carmenfan added a commit that referenced this issue Sep 9, 2015
It's not working. Even on the old gui. the simple camera test works, but
anything with a complicated camera it's coming out blank (even lost the
mesh visualisation.) disabled for now.
carmenfan added a commit that referenced this issue Sep 9, 2015
carmenfan added a commit that referenced this issue Sep 9, 2015
carmenfan added a commit that referenced this issue Sep 9, 2015
carmenfan added a commit that referenced this issue Sep 9, 2015
@carmenfan
Copy link
Member Author

done and merged. Any issues arised with code base on this should be a new issue

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

No branches or pull requests

1 participant