-
Notifications
You must be signed in to change notification settings - Fork 11
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
Future of this project? #8
Comments
From looking at the readme.md, there probably aren't any plans to merge this back into the minecrift mod. There hasn't been much activity on it and that mabrowning's fork was built for the oculus. He mentions that this was haphazardly thrown together to make it work so I doubt true integration is a top priority. |
It changes too many things and to merge cleanly back into the main branch. It also prioritizes room scale over seated, so would break things for Oculus users. Maybe a merge will get more interesting if the Rift gets high quality room scale support. 1.9 support is possible, though I noticed in the main branch that 1.8 support took quite a long time and Minecraft 1.9 was released before 1.8 VR support was finished. So maybe we pick a version that adds some good features and doesn't break mods too badly like 1.8 did. |
I'm tempted to port in the Oculus stuff into this repo, and continue from here to be honest. Once the Oculus Touch becomes available, it would definitely be worth merging the code bases again.
|
Is there undue harm in targeting OpenVR even for Oculus support? Who -Mark On Wed, May 25, 2016 at 1:18 PM, StellaArtois notifications@github.com
|
Agreed. And this repo already supports it.
|
@Automat-GH are there any concerns about jrbudda's fork? Will those changes get folded in or is that just going to be a stand alone fork? I don't know why he hasn't or if he intends to pull request. |
What is the future of this codebase? Will it be updated for 1.9? Will it be merged into mabrowning's codebase?
The text was updated successfully, but these errors were encountered: