openframeworks / openframeworks/openFrameworks
Feature: ofCamera should be used for internal of view
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 10.4k
- Forks
- 2.6k
- Avg merge
- 1d 21h
- Merged PRs (30d)
- 9
Description
I thought I had posted this already.
It would be great if we could have ofCamera do the camera setup in a default OF app.
Right now the code is duplicated.
Also it could allow the user could do something like:
ofCameraRef cam = ofGetCameraRef();
ref.dolly(-200, 0);
It would also allow for swapping out the current view with a different one.
ofSetCamera( newCam );
or
ofSetCamera( newEasyCam );
This is actually not really the api I'm proposing.
Just the overall change I think should be made.
Relates issues:
#410
#765
One other thing that is more of a bigger question is can we make OF more object orientated?
ie instead of:
ofSetCamera( ofCamera cam );
ofGetWindow(0).getContext().setCamera( cam );
Anyway I think that is for a separate GH issue - but I mention it as OF's global methods list is growing quite quickly and it might be time again to look at namespaces or moving some of the things in a more object orientated direction.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing how the default OF app currently sets up its camera and the related issues #410 and #765. Clarify the intended camera ownership, replacement API, and object-oriented scope before implementation; done should include an agreed design for internal camera setup and user camera swapping.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- computer-graphics
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100