-
Notifications
You must be signed in to change notification settings - Fork 4
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
update build stamp and tooling #199
Open
adamhathcock
wants to merge
166
commits into
main
Choose a base branch
from
update-build-stamp
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
+18,752
−32,179
Conversation
This file contains 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
* Moved performance test project * register types * perf tests
* Use Concurrent Dictionaries and ignore case on them to avoid ToLower * remove single array allocation * allocate GetClosures differently * Use JsonReader for closures * add comment * use isdefined * more readonly with less allocations * sorts * fmt * use element type when making an array * first pass * Fix reading for object and array * serialization works? * Fix closure parsing * things are way faster * Deserialize is async * fmt * renames * remove deserialize threads * fmt * faster to use ordinal compare * serialization looks okay * fix closure writing * fixes * use possibly different values for compute id * use closure parser on download * memory test for blobs * decompose serialization * fmt * fmt * for decomposing, values should be used instead of original * set id after computing it * redo more closure parsing * fix memory test * don't throw on try get deserialized * fmt * fix integration tests * fix tests * disable memory blob storage by default * put back ? * merge fixes and delete worker threads * fmt * serialization of old floats pass * serialization of old floats pass * rename class * uncomment and fix tests * don't allow things to exist in the closure table that doesn't exist * detach blob tests * rename serializer * revert * fix blob tests again
Main to dev
* Sdk * Objects * Supressed IDE warnings via editor config instead of nowarn * Nullability and other warnings * using * Fixed warnings * Important fix * More fixes
* Use Concurrent Dictionaries and ignore case on them to avoid ToLower * remove single array allocation * allocate GetClosures differently * Use JsonReader for closures * add comment * use isdefined * more readonly with less allocations * sorts * fmt * use element type when making an array * first pass * Fix reading for object and array * serialization works? * Fix closure parsing * things are way faster * Deserialize is async * fmt * renames * remove deserialize threads * fmt * faster to use ordinal compare * serialization looks okay * fix closure writing * fixes * use possibly different values for compute id * use closure parser on download * memory test for blobs * decompose serialization * fmt * fmt * for decomposing, values should be used instead of original * set id after computing it * redo more closure parsing * fix memory test * don't throw on try get deserialized * fmt * fix integration tests * fix tests * disable memory blob storage by default * put back ? * merge fixes and delete worker threads * fmt * serialization of old floats pass * serialization of old floats pass * rename class * Use async/await on GetObject * await deserialization * fmt * uncomment and fix tests * don't allow things to exist in the closure table that doesn't exist * detach blob tests * rename serializer * async more correct * revert * fix merge * fix blob tests again * more fixes * Fix building * async fixes * more async fixes * fix tests? * rename GetId to GetIdAsync * clean up * more cleanup * fmt * fix test * fix analyzer errors * use ConcurrentDictionary to be thread safe
* Add targets to csproj and fixed up logging * Logging compiles * SDK compiles * Objects compiles * fmt * fix build naming * remove net6.0 * more null fixes * merge fixes
* Add CancellationToken usage to Closure parsing * fmt
Back Merge main->Dev
* Sha256 hash with spans * HashData * NumericFormat * Tests * md5 test * xml * use utf16 encoding rather than utf8 --------- Co-authored-by: Alan Rynne <[email protected]>
* Fixed issue with serialization of detached lists * Added tests for jsonIgnore affecting id calc * use extra writer and pass tests * avoid returning extra object * fix tests * Add span overload back * fmt --------- Co-authored-by: Jedd Morgan <[email protected]>
* Remove async from serializer * Make GetId be sync * fix chunking? * Chunking Tests (#114) * fmt * Fixed speckle type --------- Co-authored-by: Jedd Morgan <[email protected]>
* Simplified props files * Private package
* Sdk * Objects * Supressed IDE warnings via editor config instead of nowarn * Nullability and other warnings * using * Required keyword on lines and meshes * Fixed test project * fixed tests * Proxies * Fixed equality of Point * IEquatable * Fixed issue with serialization of detached lists * Added tests for jsonIgnore affecting id calc * removed comments * Fixed issue with fallback to double on large values * Fixed undocumented large number support
Update Http Send trace (#118)
* initial burn * fix: Minor fixes, some pending * fix: Minor fix + DeepClean target * setup things to be injected * things compile and activites are instanced * compiles except tests * tests compile * fmt * fix merge * Activities are now an interface * fmt * fully remove logging DLL * clean up * fix tests * more cleanup * Fix integration tests * clean up hash tests * clean up path provider * more cleanup for path * merge fixes * Merge fixes * remove some extra classes --------- Co-authored-by: Alan Rynne <[email protected]>
* Clean app plugin registration and all speckle plugin verisons * fix naming
* Smells like v2 * Fixed issue with cancellation of ParallelServerApi * activity ok * Serialize json * settags extension method * Isbusy fix * Final polish * added CallerMemberName * removed absractions
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #199 +/- ##
===========================================
+ Coverage 59.49% 70.68% +11.18%
===========================================
Files 262 278 +16
Lines 10437 11153 +716
Branches 1104 1105 +1
===========================================
+ Hits 6210 7884 +1674
+ Misses 3964 2913 -1051
- Partials 263 356 +93 ☔ View full report in Codecov by Sentry. |
07b75e4
to
ef19bfa
Compare
* add ability for objectloader to skip server too. add test so both cache and server can't be skipped * move testing project to correct dir * remove extra file
…ort-for-vertex-normals
…-for-vertex-normals Vertex Normals
#225) * Serialize now waits for the scheduler to be completed before returning * formatting * wait in deserialize too * Pass cancellation token to download, refactor how things are passed/created * responses are now streamed and cancel them earlier * format * add manual empty message for sending that definitely knows when the channels are empty * Fix configure awaits * more configure await false
…#230) * Batches size now affects sending size accurately. Use extensions to help test this * format * add more and use extensions * Rework to make things clear
…eptions and not ending of the send process. (#237) * Cancel all channels when first exception happens then throw that exception * Use single exception to end things. * fix verifications * fmt * Fix tests as stacktrace was removed * Handle one exception and cancel on receive * moved ThrowIfFailed and made throw speckle exceptions * Fixed tests * fmt
* Use IAsyncDisposable on scheduler which also waits for completion * formatting
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.