Merge master back into imodel02 #3403
Merged
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
- see this issue report of the leak nodejs/node-addon-api#1140 - the leak appears to be a bug in node[-addon-api]. Tracing the gc shows JS heap size remains stable and old-generation garbage collection sweeps occur in parallel with the JS work, but objects that use `Napi::ObjectWrap` hold on to some memory until the thread yields. See the above issue for more details. - the workaround is to force yield the javascript job in long loops - the transformation service already employs this workaround to handle huge models Co-authored-by: Michael Belousov <[email protected]>
Co-authored-by: MarcBedard8 <[email protected]> Co-authored-by: Arun George <[email protected]>
* Bug fix: Right click on the new layer dialog would close it. * changelog
Co-authored-by: Arun George <[email protected]>
* Respect min/max size when initializing a panel. * Rush change.
* add YieldManager * replace setImmediate with yieldmanager Co-authored-by: Michael Belousov <[email protected]>
* Added extensions project * Added extension changes to core-frontend * Added extension test placeholder * Added extension ui-test-app * Set new extension exports to alpha * Change logs * Added extension package documentation * Updated code owners and removed unused test * Extract-api * Reverted changes to appui-react api * Removed some common types from extension access * Removed some frontend types from extensions * Added real frontend functions * Added more frontend exports * Removed real ToolSettings * Updated extensions to beta * Removed broken link to extension docs * Removed commented-out code * Simplified extension test tool * Merge fixes * clean up API * add kevin as codeowner for extensions * cleanup core-extension * core-frontend * react to PR comments * update lock file * update lock * update api docs * pr comment + dont lint * extract-api * move enum to Extension.ts * use proper types, cleanup tags * fix gnarly comment Co-authored-by: Caleb Shafer <[email protected]> Co-authored-by: Arun George <[email protected]> Co-authored-by: kckst8 <[email protected]>
* mark extension APIs as @Alpha * changes * delete unnecessary change file * re-add empty changes
* Fix requests memoization * Fix a false log message
* update to latest itwinui package * rush change
* Optimize ModelsTree search ruleset * rush change
Co-authored-by: Arun George <[email protected]>
Co-authored-by: imodeljs-admin <[email protected]>
Co-authored-by: Arun George <[email protected]> Co-authored-by: Bill Steinbock <[email protected]>
…ferences (#3256) * Allow blank connections applications use map-layers widget 's UserPreferences * Make MapLayerManager fetch preferences even though thre is no iTwinId / iModelId * Made iTwinid mandatory. Also renamed any instance of projectId to iTwinid. * Removed no longer needed blank connections Map to UserPreferencesMock * Removed UserPreferencesMock leftover * More renaming projectId-ITwinId * iTwinId now mandatory for replaceSource * changelog
… made (#3289) * Ensure that widgetDef set/get widgetState accounts for collapsed panel. * Update test coverage * rush change * extract-api and add change info to NextVersion.md
* Make it easier to override display-test-app configuration. * clarify. Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* Move `printRuleset` function into a shared utils file * Split learning snippets into separate files * Prefix all learning snippets with `Presentation.` to avoid collisions with non-presentation snippets * Fix mixed up content specification learning snippets
nick4598
requested review from
a team,
calebmshafer,
wgoehrig,
bbastings,
kabentley,
bsteinbk,
mdastous-bentley,
pmconne,
williamkbentley,
Ellord207,
MichaelBelousov,
ColinKerr,
mgooding,
dassaf4,
aruniverse,
jchick-bentley,
MarcBedard8,
diegoalexdiaz,
paulius-valiunas and
ramanujam-raman-zz
as code owners
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.