Skip to content
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

Dash.js version #109

Closed
YaxiongXiePrinceton opened this issue Jul 22, 2020 · 8 comments
Closed

Dash.js version #109

YaxiongXiePrinceton opened this issue Jul 22, 2020 · 8 comments

Comments

@YaxiongXiePrinceton
Copy link

Can I ask which version of dash.js code are you using when building pensive project?

I ask this question because the code of dash.js has changed so much and I want to compare what kind of changes you have made to the dash.js project.

Thanks and regards.

@hongzimao
Copy link
Owner

I believe it's dash.js v2.0. We have a mirror copy of the dash.js code for the project: https://github.com/hongzimao/pensieve/tree/master/dash.js

@YaxiongXiePrinceton
Copy link
Author

YaxiongXiePrinceton commented Jul 22, 2020 via email

@hongzimao
Copy link
Owner

I see what you need to do. I just checked our development branch and it seems we downloaded a version around Oct 2016. Unfortunately I can't find the exact dash version. But can you diff on the dash versions before this date and after v2.0 (i.e., v2.0.0 - v2.3.0 in https://github.com/Dash-Industry-Forum/dash.js/tags?after=v2.6.1). It's likely that you can find the right version by finding the least amount of diff.

I don't think we add more than a thousand lines of code in dash. Remember to only diff on the source code but not the compiled files like dash.all.min.js.

Hope this helps!

@hongzimao
Copy link
Owner

Feel free to confirm here the files and the code we modified when you find the version with least amount of diff. We can then double check. Thanks!

@YaxiongXiePrinceton
Copy link
Author

YaxiongXiePrinceton commented Jul 22, 2020 via email

@YaxiongXiePrinceton
Copy link
Author

image
I guess it is dash.js-2.3.0.

@hongzimao
Copy link
Owner

This looks right. We mainly changed AbrController.js.

@YaxiongXiePrinceton
Copy link
Author

Yep. I believe we found the correct version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants