Fix dashboard CI for react-three-fiber migration #795
Merged
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
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
Signed-off-by: angatupyry <fierrofenix@gmail.com>
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
aaronchongth
force-pushed
the
ci/fix-max-heap-size
branch
3 times, most recently
from
4c2e00e
to
37e69b9
Compare
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
aaronchongth
changed the title
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
Codecov Report
@@ Coverage Diff @@
## feature/port-three-fiber-branch #795 +/- ##
===================================================================
- Coverage 53.25% 51.98% -1.28%
===================================================================
Files 272 276 +4
Lines 6767 6921 +154
Branches 885 920 +35
===================================================================
- Hits 3604 3598 -6
- Misses 3020 3183 +163
+ Partials 143 140 -3
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 12 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Angatupyry
approved these changes
Oct 11, 2023
aaronchongth
merged commit b584399
into
feature/port-three-fiber-branch
4 checks passed
aaronchongth
added a commit
that referenced
this pull request
Oct 11, 2023
* Add react three fiber and drei Signed-off-by: angatupyry <fierrofenix@gmail.com> * Use component property in Box material ui component to avoid three Box component noise Signed-off-by: angatupyry <fierrofenix@gmail.com> * Use component property in Box material ui component to avoid three Box component noise Signed-off-by: angatupyry <fierrofenix@gmail.com> * Use component property in Box material ui component to avoid three Box component noise Signed-off-by: angatupyry <fierrofenix@gmail.com> * Export function Signed-off-by: angatupyry <fierrofenix@gmail.com> * Add wall draw calculation Signed-off-by: angatupyry <fierrofenix@gmail.com> * Add reac three fiber logical Signed-off-by: angatupyry <fierrofenix@gmail.com> * Migrate to react three fiber Signed-off-by: angatupyry <fierrofenix@gmail.com> * Remove center state Signed-off-by: angatupyry <fierrofenix@gmail.com> * Remove map setting Signed-off-by: angatupyry <fierrofenix@gmail.com> * Make circle shape smaller Signed-off-by: angatupyry <fierrofenix@gmail.com> * Add zindex to for the text layers to appear on top Signed-off-by: angatupyry <fierrofenix@gmail.com> * Ensure that React doesn't attempt to create or update component Signed-off-by: angatupyry <fierrofenix@gmail.com> * Add text rendering Signed-off-by: angatupyry <fierrofenix@gmail.com> * Rendering labels Signed-off-by: angatupyry <fierrofenix@gmail.com> * Hover and zoom text Signed-off-by: angatupyry <fierrofenix@gmail.com> * Show labesl correctly Signed-off-by: angatupyry <fierrofenix@gmail.com> * Remove unused imports Signed-off-by: angatupyry <fierrofenix@gmail.com> * Implementing debounce to show labels with delay Signed-off-by: angatupyry <fierrofenix@gmail.com> * Small fixes to react-three-fiber migration (#788) * Disable walls Signed-off-by: Aaron Chong <aaronchongth@gmail.com> * Use useLoader with url only, catch CORS errors potentially due to timeout issues when images are large Signed-off-by: Aaron Chong <aaronchongth@gmail.com> --------- Signed-off-by: Aaron Chong <aaronchongth@gmail.com> * Add labels for elevator and door names Signed-off-by: angatupyry <fierrofenix@gmail.com> * Wrap checkboxes into a div and use select to choose levels Signed-off-by: angatupyry <fierrofenix@gmail.com> * Add zoom icon Signed-off-by: angatupyry <fierrofenix@gmail.com> * Add levelselect event to persits between tabs Signed-off-by: angatupyry <fierrofenix@gmail.com> * Using AppEvents to persits levels Signed-off-by: angatupyry <fierrofenix@gmail.com> * Camera control component has been added Signed-off-by: angatupyry <fierrofenix@gmail.com> * Zoom in and zoom out events Signed-off-by: angatupyry <fierrofenix@gmail.com> * Add camera control component Signed-off-by: angatupyry <fierrofenix@gmail.com> * Add a wheel function to test event Signed-off-by: angatupyry <fierrofenix@gmail.com> * Expose debounce function Signed-off-by: angatupyry <fierrofenix@gmail.com> * Add wheel function to get the zoom value Signed-off-by: angatupyry <fierrofenix@gmail.com> * Updating zoom properly Signed-off-by: angatupyry <fierrofenix@gmail.com> * Remove conditional useLoader Signed-off-by: angatupyry <fierrofenix@gmail.com> * Remove zoom icon Signed-off-by: angatupyry <fierrofenix@gmail.com> * Remove zoom implementation Signed-off-by: angatupyry <fierrofenix@gmail.com> * Fix lint Signed-off-by: angatupyry <fierrofenix@gmail.com> * Rewrite debounce functino Signed-off-by: angatupyry <fierrofenix@gmail.com> * Add layers for pickup and dropoff waypoints Signed-off-by: angatupyry <fierrofenix@gmail.com> * Fix dashboard CI for react-three-fiber migration (#795) * Set max heap size Signed-off-by: Aaron Chong <aaronchongth@gmail.com> * Stop producing sourcemap for CI Signed-off-by: Aaron Chong <aaronchongth@gmail.com> * max space to ci Signed-off-by: angatupyry <fierrofenix@gmail.com> * Try build without sourcemap for CI, alongside heap size increment Signed-off-by: Aaron Chong <aaronchongth@gmail.com> * Increasing heapsize on bootstrap step Signed-off-by: Aaron Chong <aaronchongth@gmail.com> * Use experimental support for ECMAScript modules Signed-off-by: Aaron Chong <aaronchongth@gmail.com> * Use react-components as a module instead Signed-off-by: Aaron Chong <aaronchongth@gmail.com> * Remove use of meshes for robot icons Signed-off-by: Aaron Chong <aaronchongth@gmail.com> * Revert module call Signed-off-by: Aaron Chong <aaronchongth@gmail.com> --------- Signed-off-by: Aaron Chong <aaronchongth@gmail.com> Signed-off-by: angatupyry <fierrofenix@gmail.com> Co-authored-by: angatupyry <fierrofenix@gmail.com> * Add spaces in layer names Signed-off-by: angatupyry <fierrofenix@gmail.com> --------- Signed-off-by: angatupyry <fierrofenix@gmail.com> Signed-off-by: Aaron Chong <aaronchongth@gmail.com> Co-authored-by: Aaron Chong <aaronchongth@gmail.com>
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.
What's new
HatchbackmeshTODO:
Self-checks