mapknitter icon indicating copy to clipboard operation
mapknitter copied to clipboard

Cloud exporter publication roadmap

Open jywarren opened this issue 6 years ago • 15 comments

with @icarito

  • [x] 1) Merge new custom LDI export API (https://github.com/publiclab/Leaflet.DistortableImage/pull/485/)
  • [x] 2) publish version 0.10.0 and pull into MapKnitter (https://github.com/publiclab/mapknitter/pull/1192)
  • [x] 3) get status URLs saving again in export model: https://github.com/publiclab/mapknitter/issues/1190 (https://github.com/publiclab/mapknitter/pull/1192)
  • [x] 4) save map id, user id in export model!!!! LOL (https://github.com/publiclab/mapknitter/pull/1192)
  • [x] 5) display list of exports run with their status.json url (https://github.com/publiclab/mapknitter/pull/1192)
  • [x] 6) parse the status.json to display export products (JS so it's reusable) (#326 and #814 related) (https://github.com/publiclab/mapknitter/pull/1192)
  • [x] 7) test on unstable - http://mapknitter-unstable.laboratoriopublico.org/maps/test4/edit (sebastian may need to fix unstable)
  • [x] 8) test on stable - https://stable.mapknitter.org/maps/testmap/edit
  • [x] 9) PUBLISH MAPKNITTER
  • [x] 10) intro the new system
  • [x] 11) hide the old export button, AND the "exports" dropdown
  • [x] 12) display previous export products from old exporter (example https://mapknitter.org/maps/yaffa/ -- just fallback if no status.json to https://mapknitter.org/export/status/10129?) (DONE: https://github.com/publiclab/mapknitter/commit/e6b280d5f41e0768d3838a53dc1887a9e5f8f449)
  • [x] 13) move it to Cloud container
  • [x] 14) PUBLISH MAPKNITTER

BASIC FUNCTIONALITY DONE HERE!!! 🎉

  • [ ] free up Rackspace

Follow-ups

  • [x] add a progress bar
  • [x] get exports listing to auto-refresh
  • [x] style exports on map view page
  • [x] load test it
  • [x] better UI to initiate exports
  • [x] ensure starting a 2nd export works
  • [ ] 14) add image-sequencer-app export option: https://github.com/publiclab/image-sequencer-app/issues/19#issuecomment-606833260
  • [ ] publish image-sequencer-app JPG export: https://github.com/publiclab/image-sequencer-app/pull/42
  • [ ] figure out runtime calculation and stats like # of images
  • [ ] ensure canceling works

jywarren avatar Jan 10 '20 21:01 jywarren

And ref #1128

jywarren avatar Jan 10 '20 21:01 jywarren

We prob need to get unstable server connected to Amazon S3!

jywarren avatar Jan 10 '20 22:01 jywarren

Noting critical issue for starting a second export via @sashadev-sky - https://github.com/publiclab/Leaflet.DistortableImage/issues/522

Thanks!

jywarren avatar Jan 13 '20 02:01 jywarren

I was able to change the src attributes of images in the dev console and get an export to run from local!

jywarren avatar Jan 13 '20 02:01 jywarren

Some follow-ups:

  • [ ] get exports listing to update when a new one is generated (using a partial and an update)
  • [ ] clean up display of exports: show spinner for in-progress
  • [ ] get exports listing to refetch status.json periodically
  • [ ] allow deleting old exports (that did not succeed?)

jywarren avatar Jan 14 '20 16:01 jywarren

still need to make "free rotate" be the "r" key i think...

jywarren avatar Jan 14 '20 19:01 jywarren

Hi, just checking if there's been the required testing or what is blocking this. If there's any way I can be useful please direct me. I can see that it doesn't seem to work well but I'm unsure of how it is supposed to work.

icarito avatar Feb 28 '20 02:02 icarito

Ok, so we need to do some multi image testing on

https://stable.mapknitter.org/maps/testmap/edit

Then we can publish to production and ask folks to do broader testing! Then... Some UI refinements.

jywarren avatar Mar 27 '20 14:03 jywarren

Yes! It seems to work, and fast, for many images:

This one is from https://stable.mapknitter.org/maps/vibora2/edit

The UI does need some work! Also several warnings on the Javascript console to fix too.

icarito avatar Mar 28 '20 21:03 icarito

Testing out at https://github.com/publiclab/mapknitter/issues/1244

jywarren avatar Mar 31 '20 19:03 jywarren

We are now ready to publish to production with the legacy exporter offline, and test moving this to a container!

jywarren avatar Apr 12 '20 20:04 jywarren

cc @icarito -- we can test the current main branch in a Google Cloud container now as it no longer has legacy in-app exporting offered. 🎉

jywarren avatar Apr 12 '20 20:04 jywarren

That is awesome, I'll look into it today!

El dom, 12 de abr de 2020 a las 13:39, Jeffrey Warren [email protected] escribió:

cc @icarito https://github.com/icarito -- we can test the current main branch in a Google Cloud container now as it no longer has legacy in-app exporting offered. 🎉

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/publiclab/mapknitter/issues/1191#issuecomment-612672833, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABQYS2R43VWVEZEIUUC6W3RMIRG5ANCNFSM4KFNRZ5Q.

icarito avatar Apr 13 '20 17:04 icarito

Linking https://github.com/publiclab/mapknitter/pull/1273

alaxalves avatar Apr 28 '20 16:04 alaxalves

Re-opening to track follow-ups!

jywarren avatar May 05 '20 23:05 jywarren