* fix(metrics): log ex as debug for jupyter notebook
so apparently in jupyter notebooks this will actually throw
unlike in a regular python environment.
changing severity to debug since no one should ever care except me
when i'm debugging lol!
* fix(metrics): another log level change
* Initial commit
* Create CODE_OF_CONDUCT.md
* Create CONTRIBUTING.MD
* Update CODE_OF_CONDUCT.md
adds authoritative source notice to this repo
* Create ISSUE_TEMPLATE.md
* Update CODE_OF_CONDUCT.md
* Update and rename CONTRIBUTING.MD to CONTRIBUTING.md
* Update README.md
adds basic default social badges - discourse and twitter
* Update README.md
* Update README.md
* Update README.md
* Update README.md
* Create LICENSE
* Update README.md
* Update README.md
* Update CONTRIBUTING.md
fixes link
* Update CODE_OF_CONDUCT.md
* docs: add slack link and badge
* Update README.md
* fix: link and typos
* fix: more links
* refactor: moved files to .github/ folder
* feat: added PR template
Updated docs to reflect it.
* fix: updated old link
* fix: added yaml frontmatter block to issue template
* docs: removes links to slack
* docs: adds link to docs
* Update README.md
* Create open-issue.yml
* Create close-issue.yml
* Fixes: PR template updated to provide detailed instructions
* Add link to speckle-server contribution guide
* Fix link to relative to the repo pull requests
* Feature: separates issue template into bugs and feature requests
* Provides checklist for both issue templates
* Hides instructions in comments
* Add link to contribution guidelines
* Retain some sections from previous issue template
* checklist is clearer
* style: tidy newlines and other small formatting
* Add a SECURITY.md file
* Refer to the code of conduct in the contributing section of the README
* chore(pr_template): adds a reference section to the PR template
The SpecklePY PR template had a reference section, and it made sense to include it for all
repositories.
* Remove redundant issue template
* fix(pull request template): pR template should be the default and not an option
PR template was in a directory which allows selection using queries. The PR template should be
provided by default so should be renamed and placed in the .github directory.
Co-authored-by: Dimitrie Stefanescu <didimitrie@gmail.com>
Co-authored-by: izzy lyseggen <izzy.lyseggen@gmail.com>
Co-authored-by: Matteo Cominetti <matteo@cominetti.org>
Co-authored-by: Alan Rynne <alan@speckle.systems>
Co-authored-by: Alan Rynne <alan@rynne.es>
Co-authored-by: Matteo Cominetti <matteo@speckle.systems>
so apparently in jupyter notebooks this will actually throw
unlike in a regular python environment.
changing severity to debug since no one should ever care except me
when i'm debugging lol!
* chore: update dev container
* feat(client): add server version request
* docs(client): enhanced server version docstring
* test(client): sever version test
* test(base): type check for union type
* refactor(base): move _ attrs to `RegisteringBase`
* refactor: some light cleanup
* chore(container): add pylint line length
* feat(metrics): add "Invite Action"
* feat(client): pass server version to resources
* feat(models): pending stream collaborator model
* fix(client): do not parse if request res is null
* feat(exceptions): add unsupported exception
to distinguish for new invites as of server 2.6.4
* feat(client): add stream invite queries/mutations
* test(client): test stream invites
* feat(client): incorporate last invites changes
* feat(serialization): cache bases on deserialize
allows the deserializer to return the same instance when encountering
objects with the same id
closes#191
* feat(serialization): cache bases on deserialize
allows the deserializer to return the same instance when encountering
objects with the same id
closes#191
* fix(serialization): check before accessing id
obj may not be detached and therefore might not have an id
* feat(serializer): cache w id from obj not base
* chore: update deps
* feat(operations): lil send helper
on send, if `transports` is just a transport, add it to a list.
i see this error a lot so just a friendly lil fix!
* feat(objects): breps omg!
* test(geometry): updates for breps
* test(geometry): more brep serialization tests
* refactor(test): formatting
* style: formatting
* test(geometry): clean up test file
* fix(test): brep trims test fix
* refactor(geometry): clean up encoding outputs
* feat(objects): allow kwargs in encoding
* feat(objects): align curve encodings w sharp
* test(geometry): new curve encodings
* feat(client): update stream permission mutation
guess this changed some time recently and i wasn't made aware of it
* fix(objects): brep face and edge encoding
* fix(geometry): breps units 'none' fix
* test(objects): fix 'none' units issue
* revert(486ea99): use `streamGrantPermission`
to be updated for next server release
Sorts the serializable attributes before they are returned from
.get_serializable_attributes(). Prevents the object id to change if the
order of the member names from .get_member_names() changes.
* feat(metrics): add `merge_ids` helper method
* fix(metrics): use alias ids instead
* fix(metrics): final cleanup for aliasing
* fix(metrics): lol jk scratch the aliasing
* chore: depreciate python 3.6 support & upate ujson
after collection python version info metrics, we fount that only 2 users
are still using python 3.6. since it has been eol for 5 months now,
we believe it's safe to let it go.
rest easy 3.6 ⚰✨- you served us well 🫡
closes Please upgrade the ujson dependency, which has a CVE #160
* chore: upgrade and clean some deps
### SUMMARY
**sqlite transport**
This transport now batches and bulk inserts objects when writing resulting in huge performance improvements (100x).
**base object serializer**
Batching in the sqlite transport necessitated some refactoring here in order to safely call end_write when not using operations.send/receive. This has been resolved by turning traverse_base into a wrapper for _traverse_base which can take care of calling begin/end_write and resetting the writer at the top level. This is not breaking since the top level methods to call have not changed names and the original method has just been prepended with a _
Additionally, missing referenced child objects in the read transport used to raise a SpeckleException. However, using the gql client to call objects.get() will return an object with missing references by design thus throwing an error in serialization. This has been resolved by instead raising a SpeckleWarning when child objects can't be found and just returning the reference + id. ((this method of interacting with objects is discouraged so it is not surprising to me that this bug was lurking for so long - but an oopsie nonetheless!))
**ci / dev**
Updates for the ci config and the dev container to work with the recent changes in server.
NOTE: dev container seems to be pulling an older version of server -- not resolved yet
---
* quick and hacky sqlite batching
* feat(transports): batching sqlite inserts
* chore: upgrade gql3
also removed py-spy as it's not used and i was getting install errors :/
* ci: bump node version
* ci: formatting
* update CI versions
* update to new circleci redis baseimage
* update test fixture auth to non deprecated token based method
* add start and finish write method calls to base object serialize
* chore: dev container update
* fix(serialization): move end and begin write
* style: formatting
* fix(serializer): warn but don't throw if ref not found
this is _not_ an issue with the transports, but an issue with using the
graphql api to fetch objects. since you are only receiving one obj and none of
the children, the transport has no way to find them and should simply
return the reference as is. idk why anyone would really use `object.get`
so tbh i'm not surprised no one has found this bug yet lol
* fix(client): don't parse obj create response
* fix(serializer): wrap `traverse_base`
moving `begin` and `end_write` to the seriazlier due to the new
sqlite transport with batched writes necessitates a wrapper around
`traverse_base` so end/begin write can be called once at the top level.
just adding begin/end write to the original traversal method would make
tons of calls to `end_write` since the traversal is recursive
Co-authored-by: izzy lyseggen <izzy.lyseggen@gmail.com>
* feat(models): a quickie lil update
- adds fave and comment count
(mutations to come)
- adds source app to commit within stream query
* feat(client): add favorite mutation
- add metrics for client init / auth
- add server metrics
- remove incompatible server check in client
(at this point, it's been long enough that I think it's fine and will
save time on request / esp in places like blender)
* fix(geometry): int(index vals) for curve encoding
* fix(client): update poss invalid token check
server now returns `None` instead of a `GraphqlExcetion` when asking for
the user with an invalid token (or no scopes token)
* feat(models): `ActivityCollection` and `Activity`
* feat(client): stream activity method
* test(client): test for stream activity
* refactor(client): use datetime args for activity
* docs(client): clean up stream activity docstring
* feat/test(client): user activity
Gergő Jedlicska