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

obsolete files left after SAM install #12

Open
willhl opened this issue May 11, 2021 · 0 comments
Open

obsolete files left after SAM install #12

willhl opened this issue May 11, 2021 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@willhl
Copy link
Member

willhl commented May 11, 2021

The current install script copies all files and overwriting existing ones, but no deletions, so obsolete files from previous install are left.

Easiest fix is just to delete "%appdata%\SAM" before install since there is no user data which needs to be persisted.

@willhl willhl added the bug Something isn't working label May 11, 2021
@willhl willhl self-assigned this May 11, 2021
michaldengusiak added a commit that referenced this issue Oct 15, 2021
	modified:   SAM
	modified:   SAM_BHoM
	modified:   SAM_IFC
	modified:   SAM_LadybugTools
	modified:   SAM_Revit
	modified:   SAM_Tas
	modified:   SAM_Topologic
	modified:   SAM_gbXML

 Submodule changes to be committed:

 * SAM 2594fecf...e2c1d47f (25):
   > Update AnalyticalModel.cs
   > Bake Fixes
   > Update GooPanel.cs
   > Merge pull request #273 from HoareLea/NewClasses
   > Opening Class Changed to IOpening Interface
   > Create Methods Added for PlanarIntersectionResult and IFace3DObject
   > IFace3DObject Implementation, ElevationDictionary, SectionDictionary, Shells Queries Added
   > IPartition and ITerrain Updated
   > Create Method for ArchitecturalModel Updated
   > Update Normals Method  Implemented for ArchitecturalModel
   > Partition Class Changed to IPartition Interface
   > Update on Architectural Data Structure
   > Create ArchitecturalModel Updated
   > Fixes for MapInternalConditions Grasshopper Component
   > Invoke Queries Updated
   > Invoke Methods Moved from Modify Namespace to Query
   > new modify function added (TryInvokeMethod)
   > TryInvokeMethod Implemented
   > Create ArchitecturalModel Method Initial Implementation (WIP)
   > Create Method for ArchitecturalModel Updated
   > Terrain Initial Implementation
   > Update SAMAnalyticalGetZonesByName.cs
   > Update on Face3Ds Query from Mesh3D
   > Update on FixNames Component to Handle Panel, Space, Aperture, ApertureConstruction and Construction Input
   > AddAirPanels Grasshopper Component Bug Fix for Spaces Input

 * SAM_BHoM 265b031...78d0f61 (1):
   > Merge pull request #12 from HoareLea/NewClasses

 * SAM_IFC 526cc0c...4277465 (1):
   > New Architectural Classes Implemented

 * SAM_LadybugTools ffbe271...55b5b21 (26):
   > HoneybeeSchema to SAM Conversion Updated (WIP)
   > Backward Compatibility Fix
   > Construction and ApertreConstruction Conversion From HoneybeeSchema Added
   > MaterialLibrary from HoneybeeSchema Conversion Added
   > WIP
   > update to 1.46.0
   > WIP
   > WIP
   > Write Method for Utf8JsonWriter updated
   > WIP
   > Write Method Added for Utf8JsonWriter
   > Fixes for ClassName
   > Code Reorganization (WIP)
   > Update String.cs
   > Convert To String Updated (WIP)
   > Update ToJson.cs
   > ToJson Updates (WIP)
   > Update ToJson.cs
   > ToJson Conversion Updated (WIP)
   > Update ToJson.cs
   > ToJson Update (WIP)
   > Update HoneybeeSAMAnalytical.cs
   > ToJson Conversion (WIP)
   > Additional Fixes (WIP)
   > Honeybee to SAM Conversion Initial Implementation (WIP)
   > Geometry Conversion from LadybugTools to SAM Added

 * SAM_Revit 3484adb...f9bbe1a (3):
   > Merge pull request #143 from HoareLea/NewClasses
   > Merge pull request #142 from HoareLea/ApertureGeometryUpdate
   > fixes

 * SAM_Tas 0450b15...e4e340d (1):
   > Update SpaceSimulationResults.cs

 * SAM_Topologic eb7a39d...8bc5530 (3):
   > Merge pull request #41 from HoareLea/NewClasses
   > Revert "GooSAMObjectParam updated to GooJSAMObjectParam"
   > GooSAMObjectParam updated to GooJSAMObjectParam

 * SAM_gbXML 662ead4...e3bd331 (2):
   > Minor Code Reorganization
   > FIX import
michaldengusiak added a commit that referenced this issue Aug 12, 2022
	modified:   SAM
	modified:   SAM_Mollier
	modified:   SAM_Tas
	modified:   SAM_UI

 Submodule changes to be committed:

 * SAM a5a31d7c...4335d5ae (14):
   > fix update Node params - Paths
   > TryGetValue for xAttribute added
   > Paths Grasshopper Component Refresh Bug Fixed
   > Merge branch 'master' of https://github.com/HoareLea/SAM
   > NCMSystemType Added
   > NCMData added to DefaultInternalConitions
   > NCM Country Implemented
   > NCMData Added
   > Merge pull request #300 from MaciejDengusiak/master
   > Updates for Inputs of ModifyAirHandlingUnits Grasshopper Component
   > ModifyAirHandlingUnits Grasshopper Component Implemented
   > Merge pull request #299 from MaciejDengusiak/master
   > CreateProfileByValues Grasshopper Component Added
   > UserSAMTemporaryDirectory Implemented

 * SAM_Mollier b78efe8...517fb6f (17):
   > Merge pull request #12 from MaciejDengusiak/master
   > Merge pull request #11 from MaciejDengusiak/master
   > ADP MollierPoint Template implemented
   > update icon
   > Grasshopper Asset Excluded
   > Merge pull request #10 from MaciejDengusiak/master
   > Merge pull request #9 from MaciejDengusiak/master
   > Missing File Added to the project
   > Merge pull request #8 from MaciejDengusiak/master
   > Merge pull request #7 from MaciejDengusiak/master
   > Merge pull request #6 from MaciejDengusiak/master
   > Guids of Grasshopper Components Updated
   > Merge pull request #5 from MaciejDengusiak/master
   > Create Components for MollierProcesses Added
   > Merge pull request #4 from MaciejDengusiak/master
   > SAM.Weather.Grasshopper.Mollier Added
   > Create Heating Process By DryBulbTemperature Component Implemented

 * SAM_Tas 952361f...1b3590d (25):
   > Update Enumerator.cs
   > UKBR ProjectFile Added
   > Update for LightingDetail
   > Invalid Query Added
   > Code Cleaning
   > UKBR Data Structure Updated
   > Update TasUpdateUKBRFile.cs
   > Code Restructuration
   > UKBRFile Handling Implemented
   > RemoveZoneGroups Implemented
   > Update UpdateZoneGroupsByPartL.cs
   > PartL Workflow Updated
   > UpdateInternalConditionByPartL Moved to WIP
   > added Calendar  2021
   > CopyCalendar Grasshopper Component Added
   > Update UpdateInternalConditionByPartL.cs
   > DayTypes removed from TBD.zone and DesignDays Cleared in UpdateInternalCondintionsByPartL
   > Update UpdateZones.cs
   > WIP PartL
   > UpdateInternalConditionByPartL Grasshopper Component Added (WIP)
   > added part L 2021
   > SpaceParameter for Tas Added
   > UKBR Initial Implementation
   > InternalCondition Conversion from TIC Updated
   > TIC Initial Implementation

 * SAM_UI 4ed02d4...a9b1210 (25):
   > Merge pull request #63 from MaciejDengusiak/master
   > Merge pull request #60 from MaciejDengusiak/master
   > Merge pull request #59 from MaciejDengusiak/master
   > Bug Fixes for WPF Application
   > Merge pull request #58 from MaciejDengusiak/master
   > Merge pull request #57 from MaciejDengusiak/master
   > Merge pull request #56 from MaciejDengusiak/master
   > Merge pull request #55 from MaciejDengusiak/master
   > Merge pull request #54 from MaciejDengusiak/master
   > Merge pull request #53 from MaciejDengusiak/master
   > Merge pull request #52 from MaciejDengusiak/master
   > Merge pull request #51 from MaciejDengusiak/master
   > Merge pull request #50 from MaciejDengusiak/master
   > Merge pull request #49 from MaciejDengusiak/master
   > Code Reorganization
   > Merge pull request #48 from MaciejDengusiak/master
   > Merge pull request #47 from MaciejDengusiak/master
   > Code Reorganization
   > Merge pull request #46 from MaciejDengusiak/master
   > Merge pull request #45 from MaciejDengusiak/master
   > Merge pull request #44 from MaciejDengusiak/master
   > WeatherMollierPoints Updated
   > Merge pull request #43 from MaciejDengusiak/master
   > Merge pull request #42 from MaciejDengusiak/master
   > Merge pull request #41 from MaciejDengusiak/master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant