Skip to content

Commit

Permalink
Dataset/Config classes fix - alphabet for different splits (#44)
Browse files Browse the repository at this point in the history
  • Loading branch information
omsh authored Oct 1, 2024
2 parents 4426fda + 5bb4f14 commit 97649bf
Show file tree
Hide file tree
Showing 12 changed files with 428 additions and 204 deletions.
7 changes: 6 additions & 1 deletion .github/workflows/pypi.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,16 @@ on:
workflows: ["Build"]
types:
- completed
branches:
- main

jobs:
release:
runs-on: ubuntu-latest
if: ${{ github.event.workflow_run.conclusion == 'success' && github.ref == 'refs/heads/main'}}
if: |
github.event.workflow_run.conclusion == 'success' &&
github.event.workflow_run.head_branch == 'main' &&
github.ref == 'refs/heads/main'
steps:
- uses: actions/checkout@v4
- name: Set up Python
Expand Down
2 changes: 1 addition & 1 deletion src/dlomix/__init__.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
__version__ = "0.1.2"
__version__ = "0.1.3"

META_DATA = {
"author": "Omar Shouman",
Expand Down
29 changes: 3 additions & 26 deletions src/dlomix/data/charge_state.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

from ..constants import ALPHABET_UNMOD
from .dataset import PeptideDataset
from .dataset_config import DatasetConfig
from .dataset_utils import EncodingScheme


Expand Down Expand Up @@ -60,29 +61,5 @@ def __init__(
num_proc: Optional[int] = None,
batch_processing_size: int = 1000,
):
super().__init__(
data_source,
val_data_source,
test_data_source,
data_format,
sequence_column,
label_column,
val_ratio,
max_seq_len,
dataset_type,
batch_size,
model_features,
dataset_columns_to_keep,
features_to_extract,
pad,
padding_value,
alphabet,
with_termini,
encoding_scheme,
processed,
enable_tf_dataset_cache,
disable_cache,
auto_cleanup_cache,
num_proc,
batch_processing_size,
)
kwargs = {k: v for k, v in locals().items() if k not in ["self", "__class__"]}
super().__init__(DatasetConfig(**kwargs))
Loading

0 comments on commit 97649bf

Please sign in to comment.