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

oncoprint gradient shows up as solid block in pdf download #6765

Closed
adamabeshouse opened this issue Nov 1, 2019 · 10 comments · Fixed by cBioPortal/cbioportal-frontend#2911
Closed

Comments

@adamabeshouse
Copy link
Contributor

adamabeshouse commented Nov 1, 2019

we've encountered this before...

#3119

@fedde-s @rnugraha can you help, what could possibly be the issue now?

@adamabeshouse adamabeshouse self-assigned this Nov 1, 2019
@adamabeshouse adamabeshouse changed the title oncoprint gradient legend doesnt show up correctly in pdf download oncoprint gradient shows up as solid block in pdf download Nov 1, 2019
@Sjoerd-van-Hagen
Copy link

Hi @adamabeshouse,

I remember we encountered this before, I think it was Riza who did, as Fedde does not remember. Fedde is also not working on cBioPortal at the moment. Hopefully @rnugraha remembers, but he just started his new job so I am not sure if he can look into this.

My vague recollection is, that when Riza encountered this, he reported it to the maintainers of the library that does the conversion. Apparently, they fixed it.

My suggestion to move forward in case Riza does not respond, is to try and go back to a version of the library that did work, and based on that report this to the maintainers.

@jjgao
Copy link
Member

jjgao commented Nov 4, 2019

@adamabeshouse are we using the frontend sgv2pdf.js for oncoprint?

cBioPortal/cbioportal-frontend#2094

@adamabeshouse
Copy link
Contributor Author

@jjgao yes, it is svg2pdf

@adamabeshouse
Copy link
Contributor Author

@dippindots you're familiar with svg2pdf right? maybe you can take a look?

@adamabeshouse adamabeshouse removed their assignment Nov 25, 2019
@dippindots
Copy link
Member

@adamabeshouse Sure, I can take a look at this.

@dippindots
Copy link
Member

Hey @adamabeshouse. I noticed that I have encountered this problem before, and fixed that #5886 (comment). I am wondering that if we changed the way to render the gradient in <rect> recently?

@adamabeshouse
Copy link
Contributor Author

@dippindots I think the bug has maybe existed ever since then but we forgot about it. so it is just an issue with the svg2pdf library?

@adamabeshouse
Copy link
Contributor Author

@dippindots looks like they merged your PR and released it :) but we never updated the version of svg2pdf in our codebase? i will make a PR now to update

@dippindots
Copy link
Member

Thanks @adamabeshouse! I see, I will try to update the library, seems they changed something after the version we are using, may take some time to figure out.

@adamabeshouse
Copy link
Contributor Author

yes, I encountered the same issue. looks like importing doesnt work the way it used to with jsdpf-yworks somehow

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