Skip to content

Commit

Permalink
Merge branch 'main' into nk/websocket_dynamic
Browse files Browse the repository at this point in the history
  • Loading branch information
NeejWeej committed Jan 10, 2025
2 parents bd882f7 + 4bcf5cd commit 956e439
Show file tree
Hide file tree
Showing 7 changed files with 751 additions and 9 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/conda.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@ on:
- README.md
- "docs/**"
workflow_dispatch:
schedule:
# Run conda CI on Monday and Thursday at 1:25am EST (06:25 UTC)
- cron: '25 6 * * 1,4'

concurrency:
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
Expand Down
4 changes: 1 addition & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -203,10 +203,8 @@ dependencies-fedora: ## install dependencies for linux
dependencies-vcpkg: ## install dependencies via vcpkg
cd vcpkg && ./bootstrap-vcpkg.sh && ./vcpkg install

## TODO remove pin on cmake below once we identify why 3.31.2 is failing OR cmake releases a new version which installs properly
dependencies-win: ## install dependencies via windows
choco install cmake --version=3.31.1
choco install curl winflexbison ninja unzip zip --no-progress -y
choco install cmake curl winflexbison ninja unzip zip --no-progress -y

############################################################################################
# Thanks to Francoise at marmelab.com for this
Expand Down
1 change: 1 addition & 0 deletions conda/dev-environment-unix.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,3 +57,4 @@ dependencies:
- unzip
- wheel
- zip
- zlib
1 change: 1 addition & 0 deletions conda/dev-environment-win.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,3 +53,4 @@ dependencies:
- twine
- typing-extensions
- wheel
- zlib
10 changes: 7 additions & 3 deletions cpp/csp/python/PyPushInputAdapter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -145,8 +145,9 @@ class TypedPyPushInputAdapter : public PyPushInputAdapter
{
public:
TypedPyPushInputAdapter( Engine * engine, AdapterManager * manager, PyObjectPtr pyadapter, PyObject * pyType,
PushMode pushMode, PushGroup * pushGroup ):
PyPushInputAdapter( engine, manager, pyadapter, pyType, pushMode, pushGroup )
PushMode pushMode, PyObjectPtr pyPushGroup, PushGroup * pushGroup ):
PyPushInputAdapter( engine, manager, pyadapter, pyType, pushMode, pushGroup ),
m_pyPushGroup( pyPushGroup )
{
}

Expand All @@ -164,6 +165,9 @@ class TypedPyPushInputAdapter : public PyPushInputAdapter
<< pyTypeToString( m_pyType.ptr() ) << "\" got type \"" << Py_TYPE( value ) -> tp_name << "\"" );
}
}

private:
PyObjectPtr m_pyPushGroup;
};

struct PyPushInputAdapter_PyObject
Expand Down Expand Up @@ -289,7 +293,7 @@ static InputAdapter * pypushinputadapter_creator( csp::AdapterManager * manager,
[&]( auto tag )
{
pyAdapter -> adapter = pyengine -> engine() -> createOwnedObject<TypedPyPushInputAdapter<typename decltype(tag)::type>>(
manager, PyObjectPtr::own( ( PyObject * ) pyAdapter ), pyType, pushMode, pushGroup );
manager, PyObjectPtr::own( ( PyObject * ) pyAdapter ), pyType, pushMode, PyObjectPtr::incref( pyPushGroup ), pushGroup );
} );

return pyAdapter -> adapter;
Expand Down
10 changes: 7 additions & 3 deletions cpp/csp/python/PyPushPullInputAdapter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -61,8 +61,9 @@ class TypedPyPushPullInputAdapter : public PyPushPullInputAdapter
{
public:
TypedPyPushPullInputAdapter( Engine * engine, AdapterManager * manager, PyObjectPtr pyadapter, PyObject * pyType,
PushMode pushMode, PushGroup * pushGroup ):
PyPushPullInputAdapter( engine, manager, pyadapter, pyType, pushMode, pushGroup )
PushMode pushMode, PyObjectPtr pyPushGroup, PushGroup * pushGroup ):
PyPushPullInputAdapter( engine, manager, pyadapter, pyType, pushMode, pushGroup ),
m_pyPushGroup( pyPushGroup )
{
}

Expand All @@ -80,6 +81,9 @@ class TypedPyPushPullInputAdapter : public PyPushPullInputAdapter
<< pyTypeToString( m_pyType.ptr() ) << "\" got type \"" << Py_TYPE( value ) -> tp_name << "\"" );
}
}

private:
PyObjectPtr m_pyPushGroup;
};

struct PyPushPullInputAdapter_PyObject
Expand Down Expand Up @@ -208,7 +212,7 @@ static InputAdapter * pypushpullinputadapter_creator( csp::AdapterManager * mana
[&]( auto tag )
{
pyAdapter -> adapter = pyengine -> engine() -> createOwnedObject<TypedPyPushPullInputAdapter<typename decltype(tag)::type>>(
manager, PyObjectPtr::own( ( PyObject * ) pyAdapter ), pyType, pushMode, pushGroup );
manager, PyObjectPtr::own( ( PyObject * ) pyAdapter ), pyType, pushMode, PyObjectPtr::incref( pyPushGroup ), pushGroup );
} );

return pyAdapter -> adapter;
Expand Down
Loading

0 comments on commit 956e439

Please sign in to comment.