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

(WIP) Support 3D rendering in headless mode #833

Closed
wants to merge 1 commit into from

Conversation

nicolaspayette
Copy link
Member

I started tinkering on this a couple of weeks ago after a chat with Seth made me realize that the 3D view currently cannot be exported from headless mode.

I would very much like to have that working in order to automatically generate the preview images for 3D models. It would also improve regression testing for NetLogo 3D.

My efforts so far have failed, and then I had to move on to other stuff. I'll give it a another try when/if time permits, but I wanted to at least have the work in progress out there in the meanwhile.

@nicolaspayette nicolaspayette self-assigned this Jul 16, 2015
nicolaspayette added a commit to NetLogo/models that referenced this pull request Jul 16, 2015
This reverts commit e2e8f97.

I mistakenly thought these previews could be generated automatically generated.
If NetLogo/NetLogo#833 ever gets completed and merged,
I'll revert the revert...
nicolaspayette added a commit to NetLogo/models that referenced this pull request Jul 16, 2015
...regardless of what the preview commands say.

If NetLogo/NetLogo#833
gets completed and merged, this could be reverted.
nicolaspayette added a commit that referenced this pull request Jul 22, 2015
Running `all-previews` in headless 3d is not very useful (at least until, if ever, #833 is completed), but this will at least avoid messing up the models themselves.
nicolaspayette added a commit that referenced this pull request Sep 22, 2015
Running `all-previews` in headless 3d is not very useful (at least until, if ever, #833 is completed), but this will at least avoid messing up the models themselves.
@nicolaspayette
Copy link
Member Author

Old comment made by @frankduncan on gitter (pasting here so I don't forget):

My guess is nothing came out
Because 3d export needs a framebuffer to read from
And that requires the gpu to be initialized via glut
Which means you have to actually open a 1x1 pixel window
(I use xfvb on travis to accomplish this on travis)

@nicolaspayette nicolaspayette added this to the NetLogo "Hexy Honeycomb" milestone Sep 29, 2015
nicolaspayette added a commit to NetLogo/models that referenced this pull request May 18, 2016
- Some of them had custom or default commands that didn't compile
- We want models to tell the truth about their preview commands

that being said, if/when NetLogo/NetLogo#833
is completed, some the custom commands eliminated here could very
well be reused.
nicolaspayette added a commit to NetLogo/models that referenced this pull request May 18, 2016
- Some of them had custom or default commands that didn't compile
- We want models to tell the truth about their preview commands

that being said, if/when NetLogo/NetLogo#833
is completed, some the custom commands eliminated here could very
well be reused.
@mrerrormessage
Copy link
Contributor

@nicolaspayette , I should have asked you this question a while back, but do you consider this PR to still be useful and important? It will need some modifications to make it into hexy, but seems doable at this point.

@nicolaspayette
Copy link
Member Author

Useful, yes. Important? Not especially. I'd like to resume working on it
one day, but that won't happen for a while. Feel free to close for now.

Nicolas
(mobile)
On Jul 6, 2016 6:39 PM, "Robert Grider" notifications@github.com wrote:

@nicolaspayette https://github.com/nicolaspayette , I should have asked
you this question a while back, but do you consider this PR to still be
useful and important? It will need some modifications to make it into hexy,
but seems doable at this point.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#833 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/AA3d0r58X4K8D2hoqB7Z9sgrf5N5zc0vks5qS-hXgaJpZM4FaMP_
.

@mrerrormessage
Copy link
Contributor

Closing as this is pointing to 5.x

@mrerrormessage mrerrormessage removed this from the NetLogo DCI milestone Feb 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants