Fraser Greenroyd

Results 15 issues of Fraser Greenroyd

Hi! I've been trying to use Buildalyzer to analyse CSProject files for compliance to guidelines with the framework I'm working in. This works well for `netstandard2.0` and `net5.0` CSProject files,...

⚠ Bug

This class does not make full sense to me in the basefolder next to the BHoMObejct. Would suggest moving it to a subfolder and potentially sub-namespace. `System` as a suggestion,...

type:compliance

Introduced typo `the from namespace` - sorry I didn't review in time. _Originally posted by @alelom in https://github.com/BHoM/documentation/pull/46#discussion_r786176783_

severity:medium
type:documentation
priority:low

Convert the Python script [here](https://github.com/BuroHappoldEngineering/LifeCycleAnalysis_Toolkit/blob/master/Python/LCA_tool_part_tostart.py) into a C# Compute Engine method within LCA_Engine.

type:feature

Both methods in `Environment_Engine/Query/IsShade.cs` require null handling protections being added.

type:compliance

Note, where the `method` begins with `BH.oM` - this means it is a `Create` method. The following methods need appropriate `null` handling of their inputs. You can use `@BHoMBot check...

type:compliance

Note, where the `method` begins with `BH.oM` - this means it is a `Create` method. The following methods need appropriate `null` handling of their inputs. You can use `@BHoMBot check...

type:compliance

The following 4 methods are saying (and their CS files confirm) that they are part of the Serialiser_Engine namespace. However, the files that contain the code, sit within the Reflection_Engine...

type:compliance
type:question

Following discussion offline with @adecler we've come to the conclusion we should remove the `BH.oM.Data.Collections.Table` class. It's interfering with our beautiful serialisation for one, and apparently, @IsakNaslundBh you have also...

type:compliance

`.ci\Datasets\Matter_Engine\Query\Mass.json` This dataset needs updating following the changes to LCA objects and workflows. The dataset has been removed currently owing to the issues with unit tests for the engine. Please...

type:test-script