Merged
Conversation
|
This pull request introduces 1 alert and fixes 1 when merging 4102aec into 67a8cd8 - view on LGTM.com new alerts:
fixed alerts:
|
Alfred-Mountfield
suggested changes
Dec 20, 2021
Contributor
Alfred-Mountfield
left a comment
Alfred-Mountfield
left a comment
There was a problem hiding this comment.
Blocking until #116 is merged and this branch is rebased for easier review
|
|
litvand
force-pushed
the
al/fix-python-init
branch
from
4102aec to
f657e82
Compare
github-actions
bot
added
the
area/content > hash.ai
|
This pull request introduces 1 alert and fixes 1 when merging f657e82 into f7bf804 - view on LGTM.com new alerts:
fixed alerts:
|
|
This pull request introduces 1 alert when merging f657e82 into 11d2945 - view on LGTM.com new alerts:
|
github-actions
bot
removed
the
area/content > hash.ai
|
This pull request introduces 1 alert when merging 03b1c4d into 11d2945 - view on LGTM.com new alerts:
|
Alfred-Mountfield
approved these changes
Jan 10, 2022
TimDiekmann
reviewed
Jan 10, 2022
Merged
Alfred-Mountfield
pushed a commit
that referenced
this pull request
Mar 23, 2022
* Fix flatbuffers conversion during Python init (#112) * Fix package loading in Python runner (#113) * Fix line numbers in Python tracebacks (#114) * Fix behavior execution package Python component init (#115) * Fix messages package Python experiment init (#116) * Fix Python runner init (#117) * Fix Python execution (#214) * Log received inbound messages in Python process * Fix Python state snapshot sync * Display flatc errors from genfbs.py * Fix conflicting TaskMsg flatbuffers definitions * Propagate JS inbound message receive errors * Fix Python state interim sync * Fix Python task execution * Make Python error handling more consistent; update docstrings * Enabling Python tests * Adding Python setup to CI * Building before setting up Python script in CI * Only running Python setup from the root and building all * Running python setup from the package root * Changing python setup run cmd * Specifying server in cargo build in workflow * Running tests from the right cwd * Debugging the contents of the github workspace * Fix Python `context.step()` * Changing output format on integration tests * Fix a variety of Errors around the Python runner (#254) * Fixed init tests * Made behavior index private, and fixed incrementing with multi-language, and fixed access in Python * Fixing behavior index field access in python to use full loading * Fix receiving Python errors in Rust * Removing outdated `server` reference from Rust workflow * Fixing optional GroupIndex on TaskMsg * Programmatically search for libhash_engine_lib.so * Run state interim sync before `run_task` in Python * Disable native python logging * Fix outbound group index deserialization * Fix Python integration tests implementation * Properly check for metaversion in python * Fix neighbor access in Python * Disable behavior reusing * Support index notation for Python * Implement state sync for Python Co-authored-by: litvand <8569704+litvand@users.noreply.github.com> Co-authored-by: Alfred Mountfield <am@hash.ai> Co-authored-by: Tim Diekmann <tim.diekmann@3dvision.de>
vilkinsons
added
category/enhancement
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
🌟 What is the purpose of this PR?
Python init PR #6
🔗 Related links
🔍 What does this change?
🛡 Tests
❓ How to test this?