Skip to content

Commit

Permalink
Fix rcs (#82)
Browse files Browse the repository at this point in the history
Co-authored-by: imodeljs-admin <[email protected]>
  • Loading branch information
dbiguenet and imodeljs-admin authored Feb 27, 2024
1 parent 83779ea commit eade093
Show file tree
Hide file tree
Showing 7 changed files with 34 additions and 11 deletions.
10 changes: 5 additions & 5 deletions pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion python/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"

[project]
name = "reality_apis"
version = "1.0.6"
version = "1.1.0"
authors = [
{ name="Bentley Systems" },
]
Expand Down
4 changes: 2 additions & 2 deletions python/reality_apis/RC/rcs_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ def __init__(self) -> None:
self.inputs: RCJobSettings.Inputs = self.Inputs()
self.outputs: RCJobSettings.Outputs = self.Outputs()
self.engines: int = 0
self.merge: bool = True
self.merge: bool = False

def to_json(self) -> tuple[dict, dict, dict]:
"""
Expand Down Expand Up @@ -99,7 +99,7 @@ def from_json(cls, settings_json: dict) -> ReturnValue[RCJobSettings]:
if output_dict["type"] == "OPC":
new_job_settings.outputs.OPC = []
new_job_settings.outputs.OPC.append(output_dict["id"])
if output_dict["type"] == "PNTS":
elif output_dict["type"] == "PNTS":
new_job_settings.outputs.PNTS = []
new_job_settings.outputs.PNTS.append(output_dict["id"])
else:
Expand Down
15 changes: 15 additions & 0 deletions typescript/packages/reality-capture-analysis/CHANGELOG.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,21 @@
{
"name": "@itwin/reality-capture-analysis",
"entries": [
{
"date": "Tue, 27 Feb 2024 10:21:57 GMT",
"version": "1.0.1",
"tag": "@itwin/reality-capture-analysis_v1.0.1",
"comments": {
"patch": [
{
"author": "[email protected]",
"package": "@itwin/reality-capture-analysis",
"commit": "2024c640a60612d1317974327d2c6d777a459e42",
"comment": "Fix RCS Bug (Merge option default value was true instead of false)"
}
]
}
},
{
"date": "Mon, 12 Feb 2024 17:17:32 GMT",
"version": "1.0.0",
Expand Down
10 changes: 9 additions & 1 deletion typescript/packages/reality-capture-analysis/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,17 @@
# Change Log - @itwin/reality-capture-analysis

This log was last generated on Mon, 12 Feb 2024 17:17:32 GMT and should not be manually modified.
This log was last generated on Tue, 27 Feb 2024 10:21:57 GMT and should not be manually modified.

<!-- Start content -->

## 1.0.1

Tue, 27 Feb 2024 10:21:57 GMT

### Patches

- Fix RCS Bug (Merge option default value was true instead of false) ([email protected])

## 1.0.0

Mon, 12 Feb 2024 17:17:32 GMT
Expand Down
2 changes: 1 addition & 1 deletion typescript/packages/reality-capture-analysis/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@itwin/reality-capture-analysis",
"version": "1.0.0",
"version": "1.0.1",
"description": "Reality Analysis sdk to run analysis jobs",
"main": "lib/cjs/index.js",
"module": "lib/esm/index.js",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ class RCOptions {
/** If true, all the input files from multiple containers will be merged into one output file. Else output file will be created per input file.
* @type { boolean }
*/
this.merge = true;
this.merge = false;
}
}

Expand Down

0 comments on commit eade093

Please sign in to comment.