Skip to content

Support extended pdbid #950

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.

Sign up for GitHub

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

Merged
josemduarte merged 41 commits into from
Oct 29, 2021
Merged

Support extended pdbid #950

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
41 commits
Select commit Hold shift + click to select a range
6c22796
Support for extended PDBID PDB_nnnnNXXX
aalhossary Jul 27, 2021
dc12461
Some deprecated functions removal
aalhossary Jul 31, 2021
7b08387
Fix typo
aalhossary Aug 3, 2021
c39632c
Targeted most of Spencer's comments
aalhossary Aug 4, 2021
7dd8528
Fixed a bug in PDBId.equals() method
aalhossary Aug 4, 2021
e8466b9
Changing PDBID in SCOP record to lower case
aalhossary Aug 4, 2021
12efad9
public static PDBId field XXXX
aalhossary Sep 22, 2021
d4dfc95
Behavior is now an enum
aalhossary Oct 1, 2021
22e9158
Expose Pattern objects instead of Strings
aalhossary Oct 1, 2021
5c6bcdc
ID stored internally in a reduced format (without initial "PDB_" prefix)
aalhossary Oct 1, 2021
76112f0
Enhancements and bug fix
aalhossary Oct 2, 2021
d95e561
Added BDBId Unit test
aalhossary Oct 2, 2021
f54dd7d
Merge branch 'master' into Support_Extended_PDBID
aalhossary Oct 2, 2021
727aff6
Added @Deprecated codeTag
aalhossary Oct 2, 2021
960a1e6
Adding more TODOs
aalhossary Oct 2, 2021
605c6f8
more replacement of deprecated methods
aalhossary Oct 2, 2021
ec53cee
Removed PDBIdException
aalhossary Oct 9, 2021
fd48eba
review and untouch regular expressions
aalhossary Oct 9, 2021
84d9654
Un-deprecate, copy javadoc, and remove @author on methods
aalhossary Oct 10, 2021
e900fe5
Remove NullPointerException
aalhossary Oct 10, 2021
711fb8a
Bug fix
aalhossary Oct 10, 2021
f92db27
clean commented out code
aalhossary Oct 10, 2021
d9e15e0
Update CHANGELOG
aalhossary Oct 10, 2021
b410f16
Fix unit test
aalhossary Oct 13, 2021
6c8d267
PdbPair does not accept null
aalhossary Oct 12, 2021
a03a657
Addressing reviewer's comments
aalhossary Oct 13, 2021
a55aa95
Change capitalization state
aalhossary Oct 14, 2021
732a2c2
Fix probable NPE + keep consistent PdbId method naming convention
aalhossary Oct 14, 2021
34cb49e
Addressed some of the reviewer's comments.
aalhossary Oct 14, 2021
40a55d2
Use JUnit 5
aalhossary Oct 14, 2021
141d667
PdbId class Documentation
aalhossary Oct 15, 2021
c91a4dc
JavaDoc style update
aalhossary Oct 15, 2021
9ce50fe
Minor updates
aalhossary Oct 15, 2021
0fb6742
Reverting a wrong optimization
aalhossary Oct 15, 2021
02bfee4
Merge branch 'biojavaorigin/master' into Support_Extended_PDBID
aalhossary Oct 16, 2021
0d96963
Adding BioJava development code
aalhossary Oct 16, 2021
87fbc10
Javadoc
aalhossary Oct 21, 2021
17f366e
XXXX PdbId objects are not equal unless they are the same object
aalhossary Oct 22, 2021
ef9cf3f
No more XXXX. ANY malformed PdbId gracefully set to null
aalhossary Oct 23, 2021
02974cd
Addressing Reviewer's comments
aalhossary Oct 26, 2021
108fe97
Removing TODOs
aalhossary Oct 27, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Some deprecated functions removal
  • Loading branch information
@aalhossary
aalhossary committed Jul 31, 2021
commit dc12461843eb44ff6219e87e2d0640a87bd0bf3a
4 changes: 2 additions & 2 deletions biojava-structure-gui/src/main/java/org/biojava/nbio/structure/gui/util/PDBDirPanel.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,9 +133,9 @@ private Structure fromPDB(JTextField f, JTextField c) throws StructureException{
if ( debug)
System.out.println("using chain " + chain + " for structure " + structure1.getPDBCode());
Chain c1 = structure1.getPolyChainByPDB(chain);
tmp1.setPDBCode(structure1.getPDBCode());
// tmp1.setPDBId(structure1.getPDBId()); // Isn't it a duplicate?
Comment thread
aalhossary marked this conversation as resolved.
Outdated
tmp1.setPDBHeader(structure1.getPDBHeader());
tmp1.setPDBCode(structure1.getPDBCode());
tmp1.setPDBId(structure1.getPDBId());
tmp1.addChain(c1);
System.out.println("ok");

Expand Down
4 changes: 2 additions & 2 deletions biojava-structure-gui/src/main/java/org/biojava/nbio/structure/gui/util/PDBServerPanel.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -118,9 +118,9 @@ private Structure fromPDB(JTextField f, JTextField c) throws StructureException{
if ( debug)
System.out.println("using chain " + chain + " for structure " + structure1.getPDBCode());
Chain c1 = structure1.getPolyChainByPDB(chain);
tmp1.setPDBCode(structure1.getPDBCode());
// tmp1.setPDBId(structure1.getPDBId()); //Isn't it duplicate?
Comment thread
aalhossary marked this conversation as resolved.
Outdated
tmp1.setPDBHeader(structure1.getPDBHeader());
tmp1.setPDBCode(structure1.getPDBCode());
tmp1.setPDBId(structure1.getPDBId());
tmp1.addChain(c1);
System.out.println("ok");

Expand Down
2 changes: 1 addition & 1 deletion biojava-structure-gui/src/main/java/org/biojava/nbio/structure/gui/util/PDBUploadPanel.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,7 @@ private Structure getStructure(JTextField filePath,JTextField chainId) throws St
e.printStackTrace();
}

reduced.setPDBCode(fileURL);
reduced.setPDBCode(fileURL); //FIXME This usage seems wrong and should be changed.
reduced.setName(fileURL);
return reduced;

Expand Down