kodadot / nft-gallery

Generative Art Marketplace
https://kodadot.xyz
MIT License
595 stars 346 forks source link

Aurora drop sn 13 not loading #10144

Closed prury closed 2 weeks ago

prury commented 3 weeks ago

https://kodadot.xyz/ahp/gallery/120-2404230758

image

console:

image

preschian commented 3 weeks ago

Fixed on canary, just need to deploy to production

preschian commented 3 weeks ago

It seems like a valid issue. In this item, html was fine, but not the image:

preschian commented 2 weeks ago

It seems like a valid issue. In this item, html was fine, but not the image:

After further checking, capture workers could not get the image from Elekctonik drops. Puppeteer on Workers seems to have some limitations compared to Puppeteer on Node.js.

The other drops so far look fine. To prevent this issue in the future, can we put an image preview with capture workers under /code-checker for now?

endpoint: https://capture.kodadot.art/screenshot example request: https://github.com/kodadot/private-workers/blob/ea7780e127ba7b8eec5af7115cb579c9a7889c75/fxart/src/routes/drops-metadata-v2.ts#L101-L109

prury commented 2 weeks ago

https://canary.kodadot.xyz/ahp/gallery/121-2404119362

preschian commented 2 weeks ago

To prevent this issue in the future, can we put an image preview with capture workers under /code-checker for now?

Can someone add a capture-worker checker below the /code-checker page? It would be good if we could generate ten previews for both animation and static images below the page cc @kodadot/internal-dev

any hash on Electronik drops doesn't work on capture-worker https://image.w.kodadot.xyz/ipfs/bafybeifyzbjdvfblvs7eowr52w5i77yvro3c7cg7inyksem6kcxqjj6zdu/?hash={hash}

Or maybe anyone has some workaround, and the code is here: https://github.com/kodadot/workers/blob/f9f64dbb1390485f1de8557d6a3c13ae0a81feb6/capture/src/utils/surf.ts#L25-L39.

I already tried some approaches, such as listening to console.log on the page, which doesn't work with puppeteer on workers compared to puppeteer on node.js

vikiival commented 2 weeks ago

Yes because it is webgl :) it wont work.

I can top-up my vercel plan as this is fixed by vikiival/vercelgl

Moreover, code-chcecker already checks for WEBGL related stuff.

Please chceck ops-internal for the drop info

preschian commented 2 weeks ago

Moreover, code-chcecker already checks for WEBGL related stuff.

marked as "Not Using WebGL" here https://github.com/kodadot/ops-internal/issues/1579#issuecomment-2058519609

preschian commented 2 weeks ago

Oh nvm, the newer version was webgl. The question is, why did it pass to production if we already have webgl checker

Screenshot 2024-05-01 at 14 13 11
preschian commented 2 weeks ago

same issue: https://github.com/kodadot/nft-gallery/issues/10097

preschian commented 2 weeks ago

canary.kodadot.xyz/ahp/gallery/121-2404119362

updated, manually upload the image

preschian commented 1 week ago

The other drops so far look fine. To prevent this issue in the future, can we put an image preview with capture workers under /code-checker for now?

I think we still need this