-
Notifications
You must be signed in to change notification settings - Fork 30
Feature/new decider #34
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.
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
Open
sjnarmstrong
wants to merge
75
commits into
main
Choose a base branch
from
feature/new-decider
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
75 commits
Select commit
Hold shift + click to select a range
e03e666
out with the old
sjnarmstrong a4fcbb9
Added initial commit
baddedd
start file
f59c8cd
add a few more planning thigns
b6242c1
commit
027a374
Hello world
df37283
extra stuff
6fa67fc
extra stuff
897dc6c
add todo
4a7a927
initial experimentation
sjnarmstrong fefd245
starting to take shape want to adjust inject a bit pre inject adjust
sjnarmstrong a5adab6
initial stab at things
sjnarmstrong e03791d
enable lazy frame execution
sjnarmstrong 62e9139
scorecard looks semi functioning
sjnarmstrong 0cbdf68
started with dtable
sjnarmstrong fd7e219
config examples
sjnarmstrong b49b1ae
config stuff
sjnarmstrong 8867891
quite close wiht implementation
sjnarmstrong 75bcc18
quite happy with the parameter sources for now.
sjnarmstrong 55bf59a
started with flow. This is before refactoring out the parameter confi…
sjnarmstrong 509a259
move over modules and start with graph builder design
sjnarmstrong af9f4eb
update decion table to be a new module type
sjnarmstrong 1def018
move some files and get further with config
sjnarmstrong 67e48f6
rename adaptor
sjnarmstrong 7cc57b7
added ped node
sjnarmstrong ae76617
move over builder and finish adaptor
sjnarmstrong e15de41
Looking good on most of the graph building. commit before binging ove…
sjnarmstrong 6ad8053
able to build graph
sjnarmstrong 6afaba1
remove some junk testing code
sjnarmstrong 1a7079b
make some moves to simplify structure
sjnarmstrong f6db6b7
the move of primitives
sjnarmstrong 780e919
few changes for moving to ped way
sjnarmstrong f307677
few refactors
sjnarmstrong f1b2e81
lots of changes
sjnarmstrong 3af34bc
fix the pipe functionality
sjnarmstrong 7292847
fox the model dump
sjnarmstrong 2fd33be
Get end to end getting started
sjnarmstrong ac594f3
Allow dt to have null values improve things a bit
sjnarmstrong 5edd0f4
just a rerun on getting started to make it look nice
sjnarmstrong 1451aad
hamilton working
sjnarmstrong 2ac349f
fix external module
sjnarmstrong 021890c
allow module names to be changed once imported
sjnarmstrong e2dbf81
test custom modules start with tree implementation
sjnarmstrong bfbe72d
lots of changes for tree started with type adaptor
sjnarmstrong 72fdd03
renames
sjnarmstrong 9b27e0f
adaptor -> adapter
sjnarmstrong 070ae1d
rerun getting started
sjnarmstrong ecfec36
removed decider and moved over last componant the streaming component
sjnarmstrong 37fec25
continued working on tree and started with some test stuff
sjnarmstrong afe9a57
working first tree execution
sjnarmstrong 3ec73b3
started with serving layer
sjnarmstrong a8eaaad
Subtree meta
sjnarmstrong d089364
Multi-tree
sjnarmstrong 1b908e1
tree changes
sjnarmstrong 76e21e0
moved a whole bunch of stuff
sjnarmstrong f62bb63
First simplified version
sjnarmstrong 49e5f3b
added mapper module and started with configmodel
sjnarmstrong 7c2d22e
slots
sjnarmstrong 3140f63
stab at config model
sjnarmstrong 51f6653
small fixes
sjnarmstrong f719c68
current executor before simplification
sjnarmstrong d56ed16
started config stuff
sjnarmstrong 599717a
add servers
sjnarmstrong 704d86e
full end to end
sjnarmstrong 521d7d2
minor improvbements
sjnarmstrong 5313160
some nice to haves
sjnarmstrong 8f0efa7
bit of a cleanup
sjnarmstrong 32e654d
getting ready for pypi release
sjnarmstrong 9ee1da9
move over decider stuff
sjnarmstrong a784da3
move over
sjnarmstrong 6bdd434
update tree structures to make use of modules and migrate tests over
sjnarmstrong 4b0b52a
cleanup and add more tests
sjnarmstrong 75fa900
more fixes and testing
sjnarmstrong aaf0463
fixes to roles
sjnarmstrong d8b2520
Some small cleanup
sjnarmstrong File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,36 +1,25 @@ | ||
| # Ignore everything (use whitelist approach rather to ensure only important files kept) | ||
| # Ignore everything then whitelist what the image actually needs | ||
| * | ||
|
|
||
| # Allow files and directories | ||
| # Package source and build artefacts | ||
| !/decider | ||
| !/dist | ||
| !/spockflow | ||
| !/requirements | ||
| !/pyproject.toml | ||
| !/uv.lock | ||
|
|
||
| # Ignore unnecessary files inside allowed directories | ||
| # This should go after the allowed directories | ||
| **/*~ | ||
| **/*.log | ||
| **/.DS_Store | ||
| **/Thumbs.db | ||
| # Strip noise from allowed directories | ||
| **/__pycache__ | ||
| **/*.pyc | ||
| **/*.pyo | ||
| **/*.pyd | ||
| **/.Python | ||
| **/env | ||
| **/pip-log.txt | ||
| **/pip-delete-this-directory.txt | ||
| **/.tox | ||
| **/.coverage | ||
| **/.coverage.* | ||
| **/.cache | ||
| **/nosetests.xml | ||
| **/coverage.xml | ||
| **/*.cover | ||
| **/*.egg-info | ||
| **/.DS_Store | ||
| **/Thumbs.db | ||
| **/*.log | ||
| **/.git | ||
| **/.mypy_cache | ||
| **/.pytest_cache | ||
| **/.hypothesis | ||
| **/.DS_Store | ||
| **/.coverage | ||
| **/.coverage.* | ||
| **/.tox |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1,5 @@ | ||
| spockflow/_version.py export-subst | ||
| # Ensure consistent line endings | ||
| * text=auto | ||
|
|
||
| # Mark generated version file so it survives `git archive --export-subst` | ||
| decider/_version.py export-subst |
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,31 @@ | ||
| name: PR Checks | ||
|
|
||
| on: | ||
| pull_request: | ||
| branches: | ||
| - main | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| jobs: | ||
| test: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - name: Install uv | ||
| uses: astral-sh/setup-uv@v4 | ||
|
|
||
| - name: Set up Python | ||
| uses: actions/setup-python@v5 | ||
| with: | ||
| python-version: "3.12" | ||
|
|
||
| - name: Install dependencies | ||
| run: uv sync --group dev | ||
|
|
||
| - name: Run tests | ||
| run: uv run pytest | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.