Skip to content
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

MSBuild target copies both 32 & 64bit native GE to bin #216

Closed
andyward opened this issue Sep 12, 2019 · 0 comments
Closed

MSBuild target copies both 32 & 64bit native GE to bin #216

andyward opened this issue Sep 12, 2019 · 0 comments
Milestone

Comments

@andyward
Copy link
Member

The build targets packaged in Geomerty Nuget automatically copies the native geometry DLLs to te target bin folder during build as per https://github.com/xBimTeam/XbimGeometry/blob/master/Xbim.Geometry.Engine.Interop.build.targets

That's fine for Windows/Console apps but in web Apps like IIS this creates a BadImageFormatException as the host attempts to load all DLLs at startup - resulting in the web apps crashing at start

We should only copy the DLL over for the appropriate platform

Previously we deployed the native folders to a sub folder (per platform) but this was removed in 5.0.

@andyward andyward added this to the 5.1.x milestone Sep 12, 2019
andyward added a commit that referenced this issue Sep 12, 2019
Means that consumers only get the native DLL appropriate to their current platform build (x86/x64)
Fixes #216
andyward added a commit to andyward/XbimWebDemo that referenced this issue Sep 12, 2019
andyward added a commit that referenced this issue Sep 22, 2019
guenter1holzeder added a commit to OrcaIfc/XbimGeometry that referenced this issue Oct 29, 2019
* Test added for Radian Parameters over PI in value

* Updated MemoryModel package to handle radian conversion factors correctly Github xBimTeam#92 issue

* Fixing issue xBimTeam#145

using filtering of representation items instead of finding first only

* Some corrections for fix xBimTeam#145

Evaluation wasn't triggered because there was no consumer of stream

* Fix for imprecisely defined planar wires github issue xBimTeam#73
Fix for SurfaceBaseModels being defined as multiple solids

* Extruded area solids with compound profiles treated to return a solid set

* Fixing issue xBimTeam#158

Second operand has never been added to difference build chain.
Refactored IfcBooleanClippingResult to XbimSolidSet conversion.

* Experimental: Changed ProjectTarget to 8.1 which should support Win 7SP1 clients

* Added targetver to set WINVER as per
https://docs.microsoft.com/en-us/cpp/porting/modifying-winver-and-win32-winnt?view=vs-2017

This is to support Win 7SP1 runtimes.

* Update azure-pipelines.yml for Azure Pipelines

Removed tests

* removed target.h

* Updated Nuget dependencies

* Geometry engine unreferenced after tests completed

* Workaround for boolean precision removed xBimTeam#166

* Fix for excessively large solid extrusions xBimTeam#160

* Fix for Ifc4 Add2 IfcTriangulatedFaceSet not rendering  xBimTeam#167

* Fix for inaccurately defined IfcPolyloops that are not planar

* Faceted Face set re-implemented to support tolerances better

* Threading Model changed to avoid race conditions, boolean cut ops optimised for non-intersecting arguments

* Composite curve reimplemented to correctly handle polylines as edges

* IndexedPolyCurve implemented as curve

* Fix for orientation of trimmed composite curve segments that are reversed

* IfcSweptDiskSolidPolygonal fixed for closed directrix.
XbimWire trim fixed for incorrect parameterisation

* Composite Curve as wire added

* IfcSectionedSpine fixed incorrect orientation error

* Non-Intersecting cuts not returning the body shape resolved

* Erro in booleans caused by Solids created from two coincidental faces fixed

* Trimming of compound curve wire fix

* Support for IIfcPolygonalFaceSet commenced

* Support for 2D Polylines as curves added

* Closed Shell chnaged to allow faulty solids with zero volume

* Undone zero volume closed shells

* Support for IfcPolygonalFaceSet completed.
Very large facesetsare meshed rather than OCCed to prevent extremely slow performance

* Reinstate tests now stabilty issue fixed

* Closed Seep for swept disk solid fixed

* Removed last traces of the large file

* Adding axis tags to logging output for convenience

* Adding fix for zero grid bounding box in CreateGrid

assuming that the bounding box has a zero Z-extent. Does this hold?

* Fix for unsupported curve type issue xBimTeam#180

* SweptDiskSolid fixed to align sweep with directrix direction

* Interval Points for Wire fixed to include last point
Composite curve wires now allow partial curves where segments are badly specified, a warning is issued rather than a failure error

* Support for composite curves containing multi-edge segments added

* Continue IfcGrid construction on exception

* Fixing pipe maker tolerance issue

* Half Space solid reimplemented with OCC MakeHalfSpaceSolid
Polygonally bound half space amended to construct semi-infinite prism
Fuzzy value for booleans increased to 6*tolerance
Max faces to sew increased to 3000

* Grid creation fixed for errors on building lines with near precision thickness

* Polygonal bounded half space corrected for potential boolean hangs with large extrema.
Edge start and end points fixed to handle null vertices

* Empty profile definition handled, polygonally bounded half space extrusion limited to prevent booleans hanging, compositve curve creation handles incorrect reverse segment definition

* References update to Master Nuget dependencies
Warnings fixed

* No change

* Minor readme update for VS versions

* Added extra tests to help with failing CI test

* Updated to 5.1 RTM essentials

* Added changelog for 5.1

* Update build for 5.1

* Fixed changelog typo

* Added changelog for 5.1

* Update build for 5.1

* Fixed changelog typo

* Updated to 5.1 version data.
Removed FileVersion so stamped by build

* Updated to latest pre-release 5.1 dependencies

* Updated latest RTM dependencies
(with fileversions)

* Updated 5.1 dependencies (xBimTeam#187)

* Updated latest RTM dependencies
(with fileversions)

* Release 5.1 with 5.1 AssemblyVersion (xBimTeam#188)

* Added changelog for 5.1

* Update build for 5.1

* Fixed changelog typo

* Updated to 5.1 version data.
Removed FileVersion so stamped by build

* Updated to latest pre-release 5.1 dependencies

* Updated latest RTM dependencies
(with fileversions)

* Confirmed release number

* Regression causing incorrect clipping of HalfSpaceSolids fixed

* Partial fix for incorrectly parameterised Polyline trims, further work is required to correctly implement Composite and Polyline curve trims when the BIM authros start to correctly write out the parameter values

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Extended CI build timeout from default (60 minutes) to 120 minutes

* Trying to fix pipeline with explicit definition of the job name

* Trying to fix pipeline with extended timeout

* Trying to fix the pipeline

* Creation of solid sets from IfcPolygonalFaceSet added

* Create Method for IfcPolygonalFaceSet amended to respect closed shells

* Updated Essentials to Release

* Attempt to fix large coordinates by reducing them and only keeping the transformation.

* Updated Tessellator

* Updated tessellator fixes shape bounding box position. Fixed transformation of the product bbox for region computation.

* Updated Essentials

* Added explicit test runsettings to help with failing tests

* Update azure-pipelines.yml for Azure Pipelines

Add runsettings

* Refactored tests to use paths relative to deployed items.

* Only deploy the native DLL for the current platform

Means that consumers only get the native DLL appropriate to their current platform build (x86/x64)
Fixes xBimTeam#216

* Better management of trims for IIfcSweptDiskSolid

Typically used for rebars.

* Tests was expecting wrong number of faces.

* Resumed caching to help see files in Xplorer.

When /caching is issued as parameter, the files are persisted as in xbim
format so they can be seen in xplorer to check the results of meshing.

* Tolerant of missing Z coordinate for some 3D pts.

* More cases of Z missing tolerance.

by code analysis, given a bug with unusual IFC files.

* Cutting wires of IIfcCompositeCurve only if needed

* Closed profile definitions with no outer profile caught and ignored

* Fix for OCC infinite loop in ShapeHealing
Time out added to ShapeHealing and Boolean Performs

* All shape healing functions protected with a timeout

* Fixed local tests runs

Wasn't running tests in VS test runner due to "An exception occurred while invoking executor 'executor://mstestadapter/v2': Could not load file or assembly 'System.IO,"

Since we introduced DeploymentItem, we were impacted by microsoft/testfx#295

* Copy both native files when building for AnyCPU

Broken fixing xBimTeam#216

* Added Test for Advanced Brep failure

* Added additional guard to be more tolerant when processing models without valid Contexts supplied.

Should address xBimTeam/XbimWindowsUI#94

* More graceful handling of representations without a ContextOfItems. These are in violation of the specs but we should support them better.

E.g. 2x3_Arboleda_Bldg-Arch.ifc

* Going for green.
Ignore an unimplemented test in this branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant