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

APR PR master v0p3p8 #40

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

alexpreynolds
Copy link
Contributor

This PR for the HiGlass higlass-transcripts plugin includes Pixi resource memory leak fixes, PNG/SVG export fixes, more graceful error handling for null Pixi objects, and feature enhancements for mouseover tooltips and tabix input handling. This also integrates fixes that have been merged into the upstream (base) HiGlass repository in the master branch (currently post v0.3.8).

README.md Outdated Show resolved Hide resolved
@pkerpedjiev
Copy link
Member

Looks like the build is failing because there's no package-lock.json file?

@alexpreynolds
Copy link
Contributor Author

alexpreynolds commented Feb 1, 2025

I apologize but I'm a little confused about this requirement. I'm happy to add what I have locally, but is this file typically committed to source control? I thought this was specific to the end user's environment (and what packages are available on the host platform at the time npm install is run).

@pkerpedjiev
Copy link
Member

We do typically check this file in to make sure that when we run tests and deploy the exact version dependencies are used that were used when developing the component.

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

Successfully merging this pull request may close these issues.

2 participants