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

Enrichment analysis tool header/footer update #38

Open
raymond91125 opened this issue Apr 15, 2023 · 5 comments
Open

Enrichment analysis tool header/footer update #38

raymond91125 opened this issue Apr 15, 2023 · 5 comments
Assignees

Comments

@raymond91125
Copy link
Member

Currently the enrichment analysis tool CGI
wobr1.caltech.edu:/home/azurebrd/git/caltech-tools/tea.cgi
fetches the wormbase header/footer from tazendra and it needs to be changed to
https://caltech-curation.textpressolab.com/files/pub/wormbaseheader/WB_header_footer.html
when the production server goes live.

@raymond91125 raymond91125 self-assigned this Apr 15, 2023
@valearna
Copy link
Collaborator

@azurebrd it looks like the headers from tazendra and the dockerized system are old (the release versions are not automatically updated). Can you take a look?

@azurebrd
Copy link
Collaborator

@valearna Yeah, this seems to break really often. And by really often I mean, every couple or few releases. Eventually someone notices, and I update the script. It might be easier to remove the version from the local header, like we did for the Alliance version that can't be automated. Or maybe at some point someone said they could change it manually, I don't recall.

@azurebrd
Copy link
Collaborator

@valearna well. We could ask people what they want. The current script is trying to figure out what the WB header and footer and javascript are and scrape it and generate a file that can be added to every form. If all they want is a script that updates the Version string, we can do that, and that's less likely to break. I feel like often I just manually change the file when someone complains, but it looks like I edited the cronjob in april, so I'm not sure.

@azurebrd
Copy link
Collaborator

@valearna fixed. somehow a variable was the wrong variable, not sure how that typo happened, that script shouldn't have been able to generate anything while being tested, with that typo.

@azurebrd
Copy link
Collaborator

@raymond91125 @valearna For the original ticket, I've updated the file mentioned on the ticket (tea.cgi on wobr) to use the correct URL, but I don't have permissions for git on there. I don't remember if I should or I shouldn't. Also I'm on wobr2 and this says to do it on wobr1, and I don't recall how the proxies and live/dev server stuff works with wobr. Either way though, if someone else has access to the git repo and can change things, it's a literal change on line 389 to the URL on this ticket.

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

3 participants