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
add new examples testing webgl texturing #53
Conversation
add utils to save gl buffer into multiple image file format update package.json
…ly as a extension though
You should add in the get-pixels and save-pixels dependencies to package.json. To do this, you can do: |
Strange, i had done that few days ago, redoing again |
Ok, the current status on windows is just 2 fails, they happen on rendering.js: ⨯ rendering_point-no-attributes |
…ated pregenerated angle dlls
All tests passing on windows right now, ready to go! |
I tried this but it is not working.. or wasn't actually.. gl.texImage2D expects an ImageData or in fact, any object with: width, height and data. The problem is that get-pixels returns an object where width and height are in fact shape[0] and shape[1] so to fix the problem I just did:
before calling gl.texImage2D. Is it working for you? Am I missing something? |
@andresiraola Hi! Thanks for the comment, yeah, you're right... that did the trick, i've added a new commit fixing example\webgl_fundamentals\test3.js , now you can display lena. Awesome. |
@mikolalysenko Hi, why is the PR taking so long to be merged? I hope the project won't die and become unmaintained cos is promising ;-) |
@brupelo hm, is it possible to make appveyor succeed at tests? |
@dfcreative , Mmm, no clue, good question actually. I know @mikolalysenko has already looked at that few weeks ago but not sure what's the status of that right now. About the PR, before pushing I'm making sure all tests are correct doing # npm test and testing manually each example. PS: Right now conformance tests showed are 53730/53730 , all ok so far. |
No, it isn't broken or unmaintained. I just had to switch gears to catch up on some other work. I'll pick this up again once I have a block of time to concentrate on it. |
@mikolalysenko I've updated my fork and checked all is working correctly on the windows version. I've tested the whole thing from scratch and it is working flawlessly (building, examples & tests). So, could you accept this PR? I don't understand why it's taking ages to be accepted. Is there anything wrong with the PR? If so, please let me know. Regards |
Previously it wasn't passing the CI tests, which made it difficult to integrate. I'll take another look and if it is all good, I'll merge it in. |
I give up :) . It discourages quite a lot to continue contributing to this great project when the PRs take that much (opened PR 30 march and today is 08 July) to be merged to master. One little advice, even if the library is great per-se but the community (contributors or apps on top of it) is not growing the project will take ages to become mature. IMHO the library should be attractive to the general public and not being only used for a very small set of use-cases. My 2 cents, good luck! |
No description provided.