-
Notifications
You must be signed in to change notification settings - Fork 9
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
117 adding kepler handler #188
Conversation
Signed-off-by: Mark W. <32108590+zenon18@users.noreply.github.com>
Nice! Pulled these changes in and tested the results on kepler and everything seems to be working on my end. Curious to hear what @robfitzgerald thinks! |
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.
looks great, runs locally just fine for me. thanks for handling SimTime serialization along the way, the types and tests.
i did try dumping it straight into kepler and it does work but it's not obvious how i should play with the settings to get things to be pretty, i totally forget how i did that 🤦 but psyched now that we have this, i can get back to exploring that and we can get some nice ooos and aahhs when folks land on the readme.
thanks for all of your work, i hope you enjoyed figuring this one out!
@robfitzgerald we noticed with the 1970's data you want to have the trailing length needs to be 1, we noticed the later data (say putting 2023 instead of 1970) you can use higher trailing lengths (like 180+) |
Completing issue #117 by adding a Kepler Handler that generates a kepler.json file that one can import into Kepler.gl