Colin Kerr

@jackson-at-bentley your suggestion to update insertAspect to return the aspects instanceId is a good suggestion. Since we have other API which requires the aspect instance Id we should return the...

> @ColinKerr , @calebmshafer suggested that `ecschama-locaters` may be a better package for `IModelSchemaLoader`. That's fine with me, but it can't be in core-backend. Please review/modify/approve. Yes we agreed on...

Thanks for reporting this and providing a nice test case, @ImVeryLost. I have labeled this issue as a bug, I will provide a schedule update once we have done triage...

There should be 100% overlap between the units defined in the `BasicUnitsProvider` and the units defined in the standard Units schema. Other units could be defined in other schemas stored...

> > @AkshayN5 @ColinKerr How can this PR possibly take a **month** to complete? It will now conflict with my other PR. Can **someone** please do something about this? >...

> > > @AkshayN5 @ColinKerr How can this PR possibly take a **month** to complete? It will now conflict with my other PR. Can **someone** please do something about this?...

> Corresponding native PR [New PR](https://bentleycs.visualstudio.com/iModelTechnologies/_git/imodel02/pullrequest/271724)

@kabentley There was no good place for the IModelSchemaLoader so I split it into a generic SchemaLoader class that takes a function that returns the schema and could go in...

the US Survey volumes were added in Units 1.0.7: https://github.com/iTwin/bis-schemas/blob/fac8b94583872514c97cc2e4aabbe425e4af39e4/SchemaInventory.json#L298 This iModel has units 1.0.4. Units 1.0.7 was added about 3 years ago which is around the same time the...

Here is an approximate path ... the problem is the way`\\?\` is handled `\\?\D:\src\app\common\temp\node_modules\.pnpm\@[email protected]\node_modules\@bentley\imodeljs-native\imodeljs-win32-x64\Assets\ECSchemas\Standard`