Free cookie consent management tool by TermsFeed Policy Generator

Opened 8 years ago

Closed 8 years ago

#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 8 years ago by gkronber

  • Status changed from new to accepted

comment:2 Changed 8 years 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 8 years 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 8 years 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 8 years ago by gkronber (previous) (diff)

comment:5 Changed 8 years ago by gkronber

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

comment:6 Changed 8 years 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 8 years 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 8 years ago by gkronber

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

comment:9 Changed 8 years ago by gkronber

  • Status changed from readytorelease to assigned

comment:10 Changed 8 years 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 8 years 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 8 years 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 8 years ago by ascheibe (previous) (diff)

comment:13 Changed 8 years ago by gkronber

  • Milestone changed from HeuristicLab 4.0 to HeuristicLab 3.3.14

comment:14 Changed 8 years ago by gkronber

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

comment:15 Changed 8 years ago by gkronber

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

comment:16 Changed 8 years 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 8 years ago by gkronber

r13572: removed SharpDX from HL.ExtLibs solution

comment:18 Changed 8 years 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 8 years ago by ascheibe

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