Skip to content

Commit 2e84b48

Browse files
author
Nijat Khanbabayev
committed
Merge branch 'main' into nk/auto_deps_auto_callable_model
Signed-off-by: Nijat Khanbabayev <nijat.khanbabayev@cubistsystematic.com>
2 parents 324fc4e + f14cf9a commit 2e84b48

File tree

4 files changed

+10
-28
lines changed

4 files changed

+10
-28
lines changed

.copier-answers.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# Changes here will be overwritten by Copier
2-
_commit: 3160d4c
2+
_commit: 9b579a3
33
_src_path: https://github.com/python-project-templates/base.git
44
add_docs: false
55
add_extension: python

.github/workflows/build.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ jobs:
6666
if: ${{ matrix.os == 'ubuntu-latest' }}
6767

6868
- name: Upload coverage
69-
uses: codecov/codecov-action@v5
69+
uses: codecov/codecov-action@v6
7070
with:
7171
token: ${{ secrets.CODECOV_TOKEN }}
7272
if: matrix.os == 'ubuntu-latest'

.gitignore

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -112,11 +112,12 @@ dmypy.json
112112
/site
113113
index.md
114114
docs/_build/
115-
docs/src/_build/
116115
docs/api
117-
docs/index.md
118116
docs/html
117+
docs/index.md
119118
docs/jupyter_execute
119+
docs/src/_build/
120+
docs/superpowers
120121
index.md
121122

122123
# JS
@@ -146,6 +147,11 @@ target
146147
outputs/
147148
multirun/
148149

150+
# AI
151+
ROADMAP.md
152+
AGENTS.md
153+
.github/hooks/sdlc.json
154+
149155
# Examples
150156
raw.html
151157
extracted.csv

ccflow/tests/test_local_persistence.py

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,7 @@ def __call__(self, context: NullContext) -> GenericResult:
3737
return GenericResult(value="ok")
3838

3939

40-
# =============================================================================
4140
# Tests for register_ccflow_import_path function
42-
# =============================================================================
4341

4442

4543
def test_base_module_available_after_import():
@@ -117,9 +115,7 @@ def test_sync_to_module_registers_class_not_yet_on_module():
117115
assert getattr(module, unique_name, None) is cls, "Class should be on module after sync"
118116

119117

120-
# =============================================================================
121118
# Tests for _register_on_module (internal function)
122-
# =============================================================================
123119

124120

125121
def test_register_on_module_uses_specified_module():
@@ -168,9 +164,7 @@ def test_register_on_module_import_path_format():
168164
delattr(target_module, parts[1])
169165

170166

171-
# =============================================================================
172167
# Tests for local class registration via BaseModel
173-
# =============================================================================
174168

175169

176170
class TestLocalPersistencePreservesCloudpickle:
@@ -365,9 +359,7 @@ class LocalModel(BaseModel):
365359
assert restored_type_path == original_type_path
366360

367361

368-
# =============================================================================
369362
# Cross-process cloudpickle tests
370-
# =============================================================================
371363

372364

373365
@pytest.fixture
@@ -555,9 +547,7 @@ def test_cross_process_cloudpickle(self, pickle_file, create_code, load_code):
555547
assert "SUCCESS" in load_result.stdout
556548

557549

558-
# =============================================================================
559550
# Module-level classes should not be affected
560-
# =============================================================================
561551

562552

563553
class TestModuleLevelClassesUnaffected:
@@ -588,9 +578,7 @@ def test_module_level_standard_pickle_works(self):
588578
assert type(restored) is ModuleLevelModel
589579

590580

591-
# =============================================================================
592581
# Ray task tests
593-
# =============================================================================
594582

595583

596584
class TestRayTaskWithLocalClasses:
@@ -690,9 +678,7 @@ def process_model(m):
690678
assert result.startswith("hello|ccflow.local_persistence._Local_")
691679

692680

693-
# =============================================================================
694681
# UUID uniqueness tests
695-
# =============================================================================
696682

697683

698684
class TestUUIDUniqueness:
@@ -747,9 +733,7 @@ class TestModel(BaseModel):
747733
assert all(c in "0123456789abcdef" for c in uuid_part)
748734

749735

750-
# =============================================================================
751736
# Nested class and inheritance tests
752-
# =============================================================================
753737

754738

755739
class OuterClass:
@@ -858,9 +842,7 @@ class LocalSubclass(ModuleLevelModel):
858842
assert "__ccflow_import_path__" not in ModuleLevelModel.__dict__
859843

860844

861-
# =============================================================================
862845
# Generic types tests
863-
# =============================================================================
864846

865847

866848
class TestGenericTypes:
@@ -915,9 +897,7 @@ class GenericModel(BaseModel, Generic[T]):
915897
assert GenericModel.__ccflow_import_path__.startswith(local_persistence.LOCAL_ARTIFACTS_MODULE_NAME)
916898

917899

918-
# =============================================================================
919900
# Import string tests
920-
# =============================================================================
921901

922902

923903
class TestImportString:
@@ -955,9 +935,7 @@ def test_import_string_still_works_for_simple_paths(self):
955935
assert result is os.path.join
956936

957937

958-
# =============================================================================
959938
# Registration strategy tests
960-
# =============================================================================
961939

962940

963941
class TestRegistrationStrategy:
@@ -993,9 +971,7 @@ class LocalModel(BaseModel):
993971
assert "<locals>" in LocalModel.__qualname__
994972

995973

996-
# =============================================================================
997974
# Tests for create_ccflow_model wrapper
998-
# =============================================================================
999975

1000976

1001977
class TestCreateCcflowModelWrapper:

0 commit comments

Comments
 (0)