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

Webgl Distort module #1021

Closed
tech4GT opened this issue Apr 17, 2019 · 12 comments
Closed

Webgl Distort module #1021

tech4GT opened this issue Apr 17, 2019 · 12 comments

Comments

@tech4GT
Copy link
Member

tech4GT commented Apr 17, 2019

@jywarren I don't have a lot of experience with out gl based work, could you guide me a little here, I think I just need to pretty much model the fisheye-gl module right, but using webgl-distort npm module?

@vibhorgupta-gh
Copy link

I'd really like to collaborate here, the fisheye-gl with puppeteer opened a lot of avenues. I think we can start with @jywarren 's webgl-distort repository

@jywarren
Copy link
Member

Some notes here! #299

The key is:

We'll need to refactor webgl-distort to not interact with the HTML DOM, so restructuring it to work with an Image object instead of an image element, in this line:

Thanks both!!!

@vibhorgupta-gh
Copy link

Nice. I'll try mocking it tonight. Hopefully, I can come up with some ideas/code. Will post here ASAP

@tech4GT
Copy link
Member Author

tech4GT commented Apr 17, 2019

Oh, so we need to edit webgl-distort library then?

@tech4GT
Copy link
Member Author

tech4GT commented Apr 17, 2019

Also, Vibhor thanks a lot!! Actually we need this fairly quickly since this is a crucial part of the new exporter system, and with my end terms coming in, I really appreciate your help!! :D

@vibhorgupta-gh
Copy link

Let's get this done by the weekend, then? Mine start in about two weeks as well. Let's make it happen @tech4GT

@tech4GT
Copy link
Member Author

tech4GT commented Apr 17, 2019

Okay fist thing, we need to make the library node compatible, right now requiring it in node is not doing much for me. Then using fisheye-gl as a template let's implement the example code in the readme over at jywarren/webgl-distort
You can open the pr, since I would be able to push code on top of that.

@tech4GT
Copy link
Member Author

tech4GT commented Apr 17, 2019

Hmm, @jywarren since the script is not too long do you think maybe we can copy and paste the dist file in is source files?
Or do you think we should edit the jywarren/fisheye-gl??

@jywarren
Copy link
Member

jywarren commented Apr 17, 2019 via email

@Divy123
Copy link
Member

Divy123 commented Apr 18, 2019

Looks interesting!
Please count me in!!

@vibhorgupta-gh
Copy link

@Divy123 head over to the PR for the issue if you will, I could use some ideas there. Mentioned some scopes of improvement on the issue comments there

@jywarren
Copy link
Member

#1022 !!

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

No branches or pull requests

4 participants