#2504 closed feature request (done)

Transport plugin for SharpDX external library

Reported by: gkronber Owned by: ascheibe
Priority: medium Milestone: HeuristicLab 3.3.14
Component: ExtLibs Version: 3.3.12
Keywords: Cc:

Description (last modified by gkronber)

  • SharpDX is available also via nuget
  • Nuget tools have problems with our long folder names (.../HeuristicLab.SharpDx-2.6.3/2.6.3/HeuristicLab.SharpDx-2.6.3/...).
  • Include the library using official SharpDx binaries instead.

Change History (19)

comment:1 Changed 20 months ago by gkronber

  • Status changed from new to accepted

comment:2 Changed 20 months ago by gkronber

  • Description modified (diff)
  • Summary changed from Transport plugin for SharpDX nuget package to Transport plugin for SharpDX external library

comment:3 Changed 20 months ago by gkronber

Notes

  • SharpDX.Toolkit.Yaml is not included because it has another external dependency to SharpYaml.
  • Initially, I tried to store all SharpDX files in a specific folder in the bin folder (bin/SharpDX). However, I couldn't get this to work correctly as the .NET binding mechanism doesn't search this folder when resolving references. E.g., when trying to load SharpDX/SharpDX.dll it needs to resolve a reference (e.g. SharpDX.Toolkit.Game) which is not found, even though it is located in the same folder. Thus, I decided to copy SharpDX files also to bin/ for now.

r13159: created transport plugin for SharpDX and added SharpDX binaries.

comment:4 Changed 20 months ago by gkronber

r13160: svn:ignore, and by mistake many other changes all over the solution! (affects #1967).

This changeset has already been partially merged to stable with r13287 and r13298.

Last edited 20 months ago by gkronber (previous) (diff)

comment:5 Changed 20 months ago by gkronber

  • Owner changed from gkronber to ascheibe
  • Status changed from accepted to reviewing

comment:6 Changed 20 months ago by gkronber

r13162: removed SharpDX tools (tkfont.exe, tkfxc.exe, tkmodel.exe) because they were problematic to the plugin-infrastructure and are not needed anyway

comment:7 Changed 20 months ago by ascheibe

  • Owner changed from ascheibe to gkronber
  • Status changed from reviewing to readytorelease

Reviewed r13159,r13160,r13162, looks good. The plugin gets loaded in HL and I also tested it with bin packing, works without any problems.

comment:8 Changed 20 months ago by gkronber

r13230: removed unused references in SharpDX project and import of nuget.targets from project file

comment:9 Changed 20 months ago by gkronber

  • Status changed from readytorelease to assigned

comment:10 Changed 20 months ago by gkronber

I found a problem in combination with the BinPacking implementation.

When trying to open a BinPacking solution view (which uses SharpDX) one of the assemblies cannot be loaded leading to an exception and unstable behaviour.

comment:11 Changed 20 months ago by gkronber

  • Milestone changed from HeuristicLab 3.3.13 to HeuristicLab 4.0

BinPacking has been postponed to HL 4.0 therefore we decided to postpone the integration of SharpDX as well.

comment:12 Changed 20 months ago by ascheibe

r13160 has been partially merged from trunk to stable in r13298. The merge info was not recorded in stable, which needs to be done when the rest of r13160 is merged to stable. Na, all good, overlooked it.

Last edited 20 months ago by ascheibe (previous) (diff)

comment:13 Changed 19 months ago by gkronber

  • Milestone changed from HeuristicLab 4.0 to HeuristicLab 3.3.14

comment:14 Changed 18 months ago by gkronber

This external library can be removed again since we changed the BinPacking visualizations to WPF.

comment:15 Changed 17 months ago by gkronber

r13571: deleted SharpDX external library again (too heavy weight for our purposes)

comment:16 Changed 17 months ago by gkronber

  • Owner changed from gkronber to ascheibe
  • Status changed from assigned to reviewing

I decided against reverting the changes for this ticket from the trunk because r13160 was already partially merged to stable.

All changesets of this ticket should be merged to stable although the combined change has no effect.

comment:17 Changed 17 months ago by gkronber

r13572: removed SharpDX from HL.ExtLibs solution

comment:18 Changed 14 months ago by gkronber

  • Status changed from reviewing to readytorelease

Please merge to stable and check that the sum of all changesets has no effect.

comment:19 Changed 13 months ago by ascheibe

  • Resolution set to done
  • Status changed from readytorelease to closed
Note: See TracTickets for help on using tickets.