Skip to content

Problem of the order of TIA1_Hela clip_data PrismNet pu#3

Open
ZizhengYang wants to merge 1 commit intokuixu:masterfrom
ZizhengYang:patch-1
Open

Problem of the order of TIA1_Hela clip_data PrismNet pu#3
ZizhengYang wants to merge 1 commit intokuixu:masterfrom
ZizhengYang:patch-1

Conversation

@ZizhengYang
Copy link

If using

exp/EXP_NAME/train.sh pu PrismNet TIA1_Hela clip_data 

then there will have an error

FileNotFoundError: [Errno 2] Unable to open file (unable to open file: name = 'data/PrismNet/pu.h5', errno = 2, error message = 'No such file or directory', flags = 0, o_flags = 0)

I couldn't understand why there is this weird url 'data/PrismNet/pu.h5' so I decide to change position of the command.

It seems the right order should be: exp/prismnet/train.sh TIA1_Hela clip_data PrismNet pu

If using
```
exp/EXP_NAME/train.sh pu PrismNet TIA1_Hela clip_data 
```
then there will have an error
```
FileNotFoundError: [Errno 2] Unable to open file (unable to open file: name = 'data/PrismNet/pu.h5', errno = 2, error message = 'No such file or directory', flags = 0, o_flags = 0)
```
I couldn't understand why there is this weird url 'data/PrismNet/pu.h5' so I decide to change position of the command.

It seems the right order should be: exp/prismnet/train.sh TIA1_Hela clip_data PrismNet pu
Shigo-45 referenced this pull request in Shigo-45/PrismNet Feb 6, 2026
Previously only PlainCNN2D1D accepted mode parameter. PlainCNN2D,
BiLSTMBaseline, and BiGRUBaseline would fail when instantiated via
--arch flag because main.py passes mode=args.mode to all models.

Changes:
- Add mode='pu' parameter to PlainCNN2D.__init__()
- Add mode='pu' parameter to BiLSTMBaseline.__init__()
- Add mode='pu' parameter to BiGRUBaseline.__init__()
- Add docstrings explaining mode is accepted for compatibility

Tested:
- All models instantiate successfully with mode parameter
- Backward compatibility maintained (existing code works)
- All forward passes succeed with correct output shapes

Resolves Medium severity issue #3 from code review.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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.

1 participant