Move image queue to use promises, move getArrayBuffer to use promises #3280
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## async-actor #3280 +/- ##
==============================================
Coverage ? 75.23%
==============================================
Files ? 240
Lines ? 19273
Branches ? 4291
==============================================
Hits ? 14500
Misses ? 4773
Partials ? 0 ☔ View full report in Codecov by Sentry. |
HarelM
changed the title
neodescis
reviewed
Oct 30, 2023
neodescis
reviewed
Oct 30, 2023
neodescis
reviewed
Oct 30, 2023
neodescis
reviewed
Oct 30, 2023
neodescis
reviewed
Oct 30, 2023
|
Looks good... just noticed some doc comments that should be updated |
|
Thanks! I'll look into those as well. |
Co-authored-by: neodescis <[email protected]>
Co-authored-by: neodescis <[email protected]>
|
Done, let me know if I can merge to the async actor branch. |
neodescis
reviewed
Oct 30, 2023
neodescis
reviewed
Oct 30, 2023
neodescis
approved these changes
Oct 30, 2023
|
Go ahead and merge! |
HarelM
added a commit
that referenced
this pull request
Nov 23, 2023
* Initial concept * More improvements * Fix lint * Main commit to move most of the actor code to use promises and register. There are still things to do. * Fix actor tests * Remove more any when it comes to worker self typings. * Improve more types, add more tests * Improve more types, add more tests * Fix lint * Fix more tests * Fix tests * Fix some sytle tests * Add comment for todo. * Fix test related to message parameters change * Fix remaining tests * Added todo to accomodate for code review. * Fix build test * Fix geojson types * Rename some events, remove clutter a bit * Fix issue with multiple maps found in integration tests * Fix tests and improve types and docs a bit more * Final fixes to make the tests pass * Fix build test * a couple of tweaks (#3267) * Add changelog item * More improvement to async `Actor` and replace more callbacks with promises (#3269) * Improve vector tile source * Fix unit tests * Fix render tests for the case of 404 * Fix lint * Remove unused method * More improvements to remove callbacks and move to promises * Fixed some tests, more fixes are needed. * Fix remaining tests. * Fix lint * Fix tile parser code to pass render tests * Fix lint * Fix lint * More fixes and tests in the infrasturcure when it comes to cancelling * forgot to inclue this... * Fix lint and build * Convert parse to be a proper async method. * Add more test coverate to actor, allow more testing in mock web worker. * Replace getJson call with async code and remove more callbacks (#3273) * Add abort between actors of main thread and worker thread with tests * Fix lint * Fix lint * Fix node from lts to nvmrc * Improve the code of vector tile and geojson workers * Fix lint * Add debug when tests are failing * Do not throw in case of empty geojson tile * Fix lint * Move geojson worker source to be more async like. * Change getJSON to be async extenally * Update src/ui/map.test.ts Co-authored-by: Bart Louwers <[email protected]> --------- Co-authored-by: Bart Louwers <[email protected]> * Move image queue to use promises, move getArrayBuffer to use promises (#3280) * Migrate getArrayBuffer to be promise based * Remove dead code in ajax.ts file * Code review changes (#3278) Co-authored-by: neodescis <[email protected]> * Make image request return promises. * Move process queue to the right location * Increase build size * Last fixes * fix lint warning * Fix lint * update docs Co-authored-by: neodescis <[email protected]> * Update docs Co-authored-by: neodescis <[email protected]> * Fix docs * Code review comments. * Update src/util/image_request.ts * Update src/util/util.ts --------- Co-authored-by: neodescis <[email protected]> Co-authored-by: neodescis <[email protected]> * Remove TODOs, tidy up last things (#3301) * Remove some TODOs * Remove some todos, make ajax API fully promise based * Move tileJson to be async * fix lint * Remove another TODO * Move makexmlhttprequest to return a promise * Make load_sprite async * Added relevant tests * Final fixes * Update src/source/vector_tile_source.test.ts Co-authored-by: neodescis <[email protected]> --------- Co-authored-by: neodescis <[email protected]> * Async actor docs additions (#3305) * Fix missing classes in the docs * Fixed the relevant docs in the places where API calls were changed. * Fix lint * Remove some "then" from tests * remove todos, remove "then"s, simplify the ajax code a bit * Remove as any from where it's not needed * Fix code review expect stuff... * explicit promises instead of array. * Improve ajax further * Improve docs for RTL plugin status and reduce code in geojson worker test * Last simplification to actor, simplify a style test. * Add missing docs comments * Fix lint... * Fix Actor XSS, introduce subscribe (#3329) * Fix XSS, intorduce subscribe * Update changelog * Fix developer diagram with updated flow * Async actor no log warnings and errors in unit tests (#3368) * Upgrade jest, fix tests, remove console log messages. * Fix incorrect additions, lint * Async actor remove cancelables (#3371) * Remove more places that use cancelable * Remove more places with canceable * Async actor callback and promise (#3374) * Remove more places that use cancelable * Remove more places with canceable * callback and promise * Improve sources code when it is related to callback * Use callback failback when needed * Update build size test * Removed callbacks from various places, adding docs, simplifying tests * Improve type * Update src/util/actor.test.ts Co-authored-by: neodescis <[email protected]> * Make load async for all sources --------- Co-authored-by: neodescis <[email protected]> * Update changelog with most of the changes --------- Co-authored-by: Michael Barry <[email protected]> Co-authored-by: Bart Louwers <[email protected]> Co-authored-by: neodescis <[email protected]> Co-authored-by: neodescis <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The PR #3276 and its branch have too many changes and I can't seem to be able to fix the tests, I need to take slower steps still...
This is now ready for review