-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Fixes a number of issues preventing Kepler from building on fresh checkout #2596
Conversation
✅ Deploy Preview for keplergl2 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Ran the following in git-bash on Windows 10. Run yarn Error results:
|
Ran in /examples/demo-app yarn_output.txt is attached. |
The Example:
The errors you're seeing are the ones that should be resolved by this PR, I'm not able to reproduce those locally from this branch. Can you tell me more about your environment? |
@alex-bain Hi Alex, thank you for the response. My first comment was based on running the demo from kepler.gl root. My bad! My second comment is from the /examples/demo-app. My environment is on Windows 10 with running the commands in git-bash.. |
@echo292 I've pushed up a fix that will silently consume the "directory already exists" error, which should get multiple invocations of |
@alex-bain I tried it with node versions (v18.18.2 and v22.3.0). kepler.gl/examples/demo-app. Let me know if there's anything you want me to try different on my end. |
@alex-bain You're fantastic. With the latest commit, I am able to successfully compile Kepler.gl. My Dockerfile is as follows: FROM node:18
COPY kepler.gl /kepler.gl
WORKDIR /kepler.gl
RUN apt update && apt full-upgrade -y
RUN yarn
CMD ["yarn", "start"] Only thing I had to change was to add For whatever reason, webpack-dev-server listens on IPv6 only. Maybe it's a Docker quirk? Anyway, thanks again! |
I'm beginning to believe that the developers of this started on mac/unix. Gonna try clean without branches on Debian and try again. In all branches and tags cross env is not applied, some env vars are just setted hard way like Unix, so I think it's not compatible to dev on Windows. Maybe with some changes, it's possible, but I'm not gonna fix it due to my personal situation. |
When trying to checkout and run Kepler a number of build issues prevent the application from starting successfully. This collection of changes ensures that the application can be cloned / installed / built without error. Signed-off-by: Alex Bain <alex_s_bain@apple.com>
Signed-off-by: Alex Bain <alex_s_bain@apple.com>
…ady-exists error Signed-off-by: Alex Bain <alex_s_bain@apple.com>
…R feedback Signed-off-by: Alex Bain <alex_s_bain@apple.com>
@echo292 I tried adding all of the Babel transform plugins to each |
Signed-off-by: Alex Bain <alex_s_bain@apple.com>
I can try. Clone fresh, yarn and yarn start, any extra steps maybe on your branch? Thanks |
@alex-bain Thank you! I will give this a try now. @psirdev - As far as I know, this branch commit is specific to the kepler.gl/examples/demo-app. |
Yes, I'm on the same point. Clean fresh, yarn, yarn start and everything crashes trying the demo app. Tried Windows 10, Debian, branches that I tried: master, alex-bain:build-fixes, igr/fix-yarn-start... Your commit also doesn't work on Windows. Gonna try again on Debian... |
Yikes, failing... What can we do? Maybe switch to another commit that works? |
* Adds missing babel transform deps to package.jsons * Resolves dep version mismatches contributing to build failures * Adds ts-ignore statements to workaround TS failures after making these changes Signed-off-by: Alex Bain <alex_s_bain@apple.com>
Hello @alex-bain , I'm on the Slack, if you wanna join and try to resolve with me (pair programming), I'm here all day D: I tried some old branches and it happens, maybe is some NPM dependencies (react, redux or babel) that broke all the code P.D: It happened with other repositories too that are not affiliated with kepler |
Signed-off-by: Alex Bain <alex_s_bain@apple.com>
@psirdev - Thank you. I believe the latest commit should resolve the issues that @echo292 reported, however the scope of this PR has increased as I was tracking down and trying to resolve those errors. |
Thanks for everything. It looks like the pipeline is working. I tried to install on Windows and it shows: But I think it's something on my own. Tried to delete those folders, change the stab scripts... Gonna try on Debian and I will tell you in a couple minutes. Thanks. |
Looks like mkdir -p (stab script on every folder in src) on unix works like a charm, but in Git Bash of Windows it fails due to yarn executing like CMD style. Maybe It's because I have WSL subsystem. Gonna try Cygwin or another bash and try again. EDIT: it fails too with Cygwin. Gonna change workaround and see if I can use another command. |
@psirdev |
root, if you look the yarn start it redirects to the example, and it loads from the node_modules from the root folder I'm slow because I had problems with Virtualbox and switched everything to Hyper-V, so I'm gonna confirm in 10 mins at least |
@echo292 @alex-bain I have a fix for the stab script (for cross compatibility on Windows & Unix) on every folder on src:
But.... It fails later resolving the ts (it's empty) dist folder:
So I give up, throw the keyboard and tried on Debian. And it works!!! Just thinking Windows is not supported or maybe nobody programs on windows anymore lol |
@alex-bain @psirdev I attached the yarn.lock.txt hoping this can provide some clues. |
@alex-bain @psirdev Got the demo to work in WSL/VS Code in Windows 10.
|
Signed-off-by: Alex Bain <alex_s_bain@apple.com>
I've reverted the I'll also call out I had to add a number of |
@alex-bain Thank you! Your latest commit worked on the first try. |
examples/webpack.config.local.js
Outdated
/node_modules\/@loaders\.gl/, | ||
/node_modules\/@probe\.gl/, | ||
/node_modules\/@math\.gl/, | ||
/node_modules\/@kepler\.gl/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think you need to include @kepler.gl
here, the babel loaders of kepler.gl src is handled by makeBabelRule
When I perform a fresh clone, a number of build issues prevent Kepler from starting successfully.
This collection of changes ensures that the application can be cloned / installed / built without error.
Testing
yarn bootstrap
yarn start
Everything should build successfully and the demo app should load without error.
May be related to #2594, #2593, and earlier reports.