Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-3738

Update find junctions repo even more - main branch, bitbucket pipeline, more readme text

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      Three additional tasks to update / improve the find junction code in bitbucket:

      1) Now that find junctions is recommended to be built using JDK 21, we need to update the bitbucket pipelines file. The pipeline might not even run, so this task may take some effort to debug it.

      2) Also, we may also want to indicate in the README which genome a user needs to open in order to view the "test" bed file, and explain to the user what they ought to see in IGB when they do that. That way, if there is a problem, they will know.

      3) Lastly, let's change the main development branch from "master" to "main".

        Attachments

          Issue Links

            Activity

            Hide
            ann.loraine Ann Loraine added a comment -

            If you are able to create a new branch "main" from "master", that would be awesome. The instructions look good - no changes requested.

            Show
            ann.loraine Ann Loraine added a comment - If you are able to create a new branch "main" from "master", that would be awesome. The instructions look good - no changes requested.
            Hide
            jsirigin Jaya Sravani Sirigineedi added a comment - - edited

            Raised the pull request: https://bitbucket.org/lorainelab/findjunctions/pull-requests/3, please review and let me know if there are any issues.

            Show
            jsirigin Jaya Sravani Sirigineedi added a comment - - edited Raised the pull request: https://bitbucket.org/lorainelab/findjunctions/pull-requests/3 , please review and let me know if there are any issues.
            Hide
            ann.loraine Ann Loraine added a comment -

            PR is merged. Also please note that I have changed to default branch to "main" instead of "master".

            Show
            ann.loraine Ann Loraine added a comment - PR is merged. Also please note that I have changed to default branch to "main" instead of "master".
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            Note that the built jar file in the Downloads requires JDK 21 now. I noticed this when I
            Has anyone tried running the newly build jar file? I tried it on the UNC Charlotte cluster, but the JVM I usually use would not run it because the java versions did not match up.
            To test that the code is still runnable, someone needs to actually try running the find-junctions.jar artifact.
            To do this:

            1) Download the jar file from https://bitbucket.org/lorainelab/findjunctions/downloads/find-junctions-1.0.0-jar-with-dependencies.jar
            2) Follow the instructions in the 'readme" file to test-run the code

            Request for Nowlan Freese or Dylan Marrotte: Would one of you be able to do the above testing?

            Show
            ann.loraine Ann Loraine added a comment - - edited Note that the built jar file in the Downloads requires JDK 21 now. I noticed this when I Has anyone tried running the newly build jar file? I tried it on the UNC Charlotte cluster, but the JVM I usually use would not run it because the java versions did not match up. To test that the code is still runnable, someone needs to actually try running the find-junctions.jar artifact. To do this: 1) Download the jar file from https://bitbucket.org/lorainelab/findjunctions/downloads/find-junctions-1.0.0-jar-with-dependencies.jar 2) Follow the instructions in the 'readme" file to test-run the code Request for Nowlan Freese or Dylan Marrotte : Would one of you be able to do the above testing?
            Hide
            pkulzer Paige Kulzer added a comment - - edited

            With JDK 21 installed on my computer, I downloaded the jar file from the link above and tested the code using the instructions in the "readme" from the FindJunctions repo. The modified readme was easy to follow and straightforward. The test files (1.bam and test.bed) loaded into IGB properly and at least one junction is present in the test.bed track.

            Moving this to Done.

            Show
            pkulzer Paige Kulzer added a comment - - edited With JDK 21 installed on my computer, I downloaded the jar file from the link above and tested the code using the instructions in the "readme" from the FindJunctions repo. The modified readme was easy to follow and straightforward. The test files (1.bam and test.bed) loaded into IGB properly and at least one junction is present in the test.bed track. Moving this to Done.

              People

              • Assignee:
                jsirigin Jaya Sravani Sirigineedi
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: