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

improvement: remove unused 'headers' for custom data sources and update docs #2164

Merged
merged 7 commits into from
May 27, 2022

Conversation

larsmoa
Copy link
Contributor

@larsmoa larsmoa commented May 27, 2022

No description provided.

@larsmoa larsmoa added auto-update Makes bulldozer automatically update this PR when there are changes to the target branch preview-docs Deploy preview documentation for a PR slack PRs and issues with this label will be pushed to Slack labels May 27, 2022
@codecov
Copy link

codecov bot commented May 27, 2022

Codecov Report

Merging #2164 (9fc8482) into master (36c37dd) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 9fc8482 differs from pull request most recent head ea0ee84. Consider uploading reports for the commit ea0ee84 to get more accurate results

@@            Coverage Diff             @@
##           master    #2164      +/-   ##
==========================================
+ Coverage   71.22%   71.24%   +0.02%     
==========================================
  Files         259      259              
  Lines       10794    10789       -5     
  Branches     1364     1364              
==========================================
- Hits         7688     7687       -1     
+ Misses       2953     2949       -4     
  Partials      153      153              
Impacted Files Coverage Δ
...packages/modeldata-api/src/CdfModelDataProvider.ts 80.76% <ø> (+5.76%) ⬆️
...ckages/modeldata-api/src/LocalModelDataProvider.ts 25.00% <ø> (+5.00%) ⬆️
viewer/packages/modeldata-api/src/types.ts 100.00% <ø> (ø)
viewer/packages/modeldata-api/index.ts 100.00% <100.00%> (ø)

@github-actions
Copy link

📙 Documentation preview is available from
https://cognitedata.github.io/reveal-docs-preview/2164/docs/next/.

@larsmoa larsmoa marked this pull request as ready for review May 27, 2022 08:54
@larsmoa larsmoa requested a review from a team as a code owner May 27, 2022 08:54
Copy link
Contributor

@haakonflatval-cognite haakonflatval-cognite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, one suggestion

documentation/docs/extending/datasource.mdx Outdated Show resolved Hide resolved
Co-authored-by: Håkon Flatval <70905152+haakonflatval-cognite@users.noreply.github.com>
@larsmoa larsmoa removed the preview-docs Deploy preview documentation for a PR label May 27, 2022
@larsmoa larsmoa enabled auto-merge (squash) May 27, 2022 09:01
@larsmoa larsmoa merged commit 595a8f1 into master May 27, 2022
@larsmoa larsmoa deleted the larsmoa/doc-remove-experimental-datasource branch May 27, 2022 09:08
haakonflatval-cognite added a commit that referenced this pull request May 30, 2022
* chore(deps): update dependency @cognite/sdk to v7.4.0 (#2160)

Co-authored-by: Renovate Bot <bot@renovateapp.com>

* improvement: remove unused 'headers' for custom data sources and update docs (#2164)

* fix: clearing wrong render target (#2166)

* example: add UI for setting point size type (#2157)

* improvement: Make typescript strict (#2163)

* fix: fix all strict typechecking errors

* improvement: don't return primitive collections if empty

* chore: lint fix

* improvement: make byteOffset 0 if undefined

* improvement: unhackify helper function

* fix: fix that byteOffset fix

* fix: fix type errors for tests

* fix: correct BinaryHeap bevaviour

Co-authored-by: Christopher J. Tannum <christopher.tannum@cognite.com>
Co-authored-by: cognite-bulldozer[bot] <51074376+cognite-bulldozer[bot]@users.noreply.github.com>

* chore(deps): update dependency @cognite/sdk to v7.5.0 (#2165)

Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: cognite-bulldozer[bot] <51074376+cognite-bulldozer[bot]@users.noreply.github.com>

* feat: simple styling in the works

* feat: working visualization of pipe system

Pipe data not included

* feat: add simple styling interface for point clouds

Also temporarily add data.ts to allow for testing for people who are not me

* refac: make object style LUT 2D

* refactor: let EPT loader take non-hardcoded object list

* feat: automatically fetch annotations from server on point cloud load

* chore: update Migration example for annotation fetching

* fix: remove unused import

* fix: remove not-yet-available feature

* feat: add arbitrarily oriented boxes as stylable objects

* refactor: some cleanup among stylable shapes

* fix: don't import anything externally to worker

* chore: lint fix

* feat: blend style color with original RGB

* chore: lint fix

* chore: fix LGTM warnings

* chore: update yarn lock

* test: fix constructor in test

* chore: remove unused member variable

* chore: remove unused import

Fixes LGTM error

* fix: make @cognite/sdk-playground a peer dependency

* fix: fix that pesky visual test error

* refactor: factor out default EPT metadata file name

* improvement: make "then" into an "await"

* refactor: refactor annotation translation

* chore: make public variable private-gettable

* fix: correct import path

* fix: refer to private field, not public getter

* refactor: rewrite point cloud buffer parsing

* improvement: be stricter in specifying optional types

* improvement: make use of color constants

* improvement: rewrite 0-vector check

* fix: correct paratheses in shader

* improvement: mappify array transformation

* improvement: make ShapeType an enum

* improvement: remove out-commented import

* chore: lint fix

* fix: make CogniteClientPlayground optionality annotation stricter

* test: add trivial test for CognitePointCloudModel

* fix: add missing creation function file

* refactor: move some responsibility from PointCloudManager to -Factory

* improvement: then -> await

* fix: make indices be of type Uint32

* feat: add fromThreeVector3 utility function

* refactor: parser-worker logic

* improvement: make parse(data) actually return parsed data

* improvement: improve readability

* refactor: simplify primitive translation

* improvement: renamed/simplify objectId assignment

* refactor: change terminology from  "StyledObject" to "StylableObject"

* refactor: simplify Box constructor

* chore: minor cleanup

* improvement: simplify composite bounding box computation

* fix: correct Box constructor call

* fix: repair, please type checker, and lint after merge

* improvement: remove computeBoundingBox for shapes for now

They'll be back for the acceleration structure

* chore: another lint fix

* chore: add warning regarding point-object matching

* refactor: remove "StylableObjectInfo" abstraction

* chore: lint fix

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: Lars Moastuen <lars.moastuen@cognite.com>
Co-authored-by: Christopher J. Tannum <christopher.tannum@cognite.com>
Co-authored-by: cognite-bulldozer[bot] <51074376+cognite-bulldozer[bot]@users.noreply.github.com>
haakonflatval-cognite added a commit that referenced this pull request Jun 14, 2022
* chore(deps): update dependency @cognite/sdk to v7.4.0 (#2160)

Co-authored-by: Renovate Bot <bot@renovateapp.com>

* improvement: remove unused 'headers' for custom data sources and update docs (#2164)

* fix: clearing wrong render target (#2166)

* example: add UI for setting point size type (#2157)

* improvement: Make typescript strict (#2163)

* fix: fix all strict typechecking errors

* improvement: don't return primitive collections if empty

* chore: lint fix

* improvement: make byteOffset 0 if undefined

* improvement: unhackify helper function

* fix: fix that byteOffset fix

* fix: fix type errors for tests

* fix: correct BinaryHeap bevaviour

Co-authored-by: Christopher J. Tannum <christopher.tannum@cognite.com>
Co-authored-by: cognite-bulldozer[bot] <51074376+cognite-bulldozer[bot]@users.noreply.github.com>

* chore(deps): update dependency @cognite/sdk to v7.5.0 (#2165)

Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: cognite-bulldozer[bot] <51074376+cognite-bulldozer[bot]@users.noreply.github.com>

* feat: simple styling in the works

* feat: working visualization of pipe system

Pipe data not included

* feat: add simple styling interface for point clouds

Also temporarily add data.ts to allow for testing for people who are not me

* refac: make object style LUT 2D

* refactor: let EPT loader take non-hardcoded object list

* feat: automatically fetch annotations from server on point cloud load

* chore: update Migration example for annotation fetching

* fix: remove unused import

* fix: remove not-yet-available feature

* feat: add arbitrarily oriented boxes as stylable objects

* refactor: some cleanup among stylable shapes

* fix: don't import anything externally to worker

* chore: lint fix

* feat: blend style color with original RGB

* chore: lint fix

* chore: fix LGTM warnings

* chore: update yarn lock

* test: fix constructor in test

* chore: remove unused member variable

* chore: remove unused import

Fixes LGTM error

* fix: make @cognite/sdk-playground a peer dependency

* fix: fix that pesky visual test error

* refactor: factor out default EPT metadata file name

* improvement: make "then" into an "await"

* refactor: refactor annotation translation

* chore: make public variable private-gettable

* fix: correct import path

* fix: refer to private field, not public getter

* refactor: rewrite point cloud buffer parsing

* improvement: be stricter in specifying optional types

* improvement: make use of color constants

* improvement: rewrite 0-vector check

* fix: correct paratheses in shader

* improvement: mappify array transformation

* improvement: make ShapeType an enum

* improvement: remove out-commented import

* chore: lint fix

* fix: make CogniteClientPlayground optionality annotation stricter

* test: add trivial test for CognitePointCloudModel

* fix: add missing creation function file

* refactor: move some responsibility from PointCloudManager to -Factory

* improvement: then -> await

* fix: make indices be of type Uint32

* feat: add fromThreeVector3 utility function

* refactor: parser-worker logic

* improvement: make parse(data) actually return parsed data

* improvement: improve readability

* refactor: simplify primitive translation

* improvement: renamed/simplify objectId assignment

* refactor: change terminology from  "StyledObject" to "StylableObject"

* refactor: simplify Box constructor

* chore: minor cleanup

* improvement: simplify composite bounding box computation

* fix: correct Box constructor call

* fix: repair, please type checker, and lint after merge

* improvement: remove computeBoundingBox for shapes for now

They'll be back for the acceleration structure

* chore: another lint fix

* chore: add warning regarding point-object matching

* refactor: remove "StylableObjectInfo" abstraction

* chore: lint fix

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: Lars Moastuen <lars.moastuen@cognite.com>
Co-authored-by: Christopher J. Tannum <christopher.tannum@cognite.com>
Co-authored-by: cognite-bulldozer[bot] <51074376+cognite-bulldozer[bot]@users.noreply.github.com>
Savokr pushed a commit that referenced this pull request Jun 21, 2022
* chore(deps): update dependency @cognite/sdk to v7.4.0 (#2160)

Co-authored-by: Renovate Bot <bot@renovateapp.com>

* improvement: remove unused 'headers' for custom data sources and update docs (#2164)

* fix: clearing wrong render target (#2166)

* example: add UI for setting point size type (#2157)

* improvement: Make typescript strict (#2163)

* fix: fix all strict typechecking errors

* improvement: don't return primitive collections if empty

* chore: lint fix

* improvement: make byteOffset 0 if undefined

* improvement: unhackify helper function

* fix: fix that byteOffset fix

* fix: fix type errors for tests

* fix: correct BinaryHeap bevaviour

Co-authored-by: Christopher J. Tannum <christopher.tannum@cognite.com>
Co-authored-by: cognite-bulldozer[bot] <51074376+cognite-bulldozer[bot]@users.noreply.github.com>

* chore(deps): update dependency @cognite/sdk to v7.5.0 (#2165)

Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: cognite-bulldozer[bot] <51074376+cognite-bulldozer[bot]@users.noreply.github.com>

* feat: simple styling in the works

* feat: working visualization of pipe system

Pipe data not included

* feat: add simple styling interface for point clouds

Also temporarily add data.ts to allow for testing for people who are not me

* refac: make object style LUT 2D

* refactor: let EPT loader take non-hardcoded object list

* feat: automatically fetch annotations from server on point cloud load

* chore: update Migration example for annotation fetching

* fix: remove unused import

* fix: remove not-yet-available feature

* feat: add arbitrarily oriented boxes as stylable objects

* refactor: some cleanup among stylable shapes

* fix: don't import anything externally to worker

* chore: lint fix

* feat: blend style color with original RGB

* chore: lint fix

* chore: fix LGTM warnings

* chore: update yarn lock

* test: fix constructor in test

* chore: remove unused member variable

* chore: remove unused import

Fixes LGTM error

* fix: make @cognite/sdk-playground a peer dependency

* fix: fix that pesky visual test error

* refactor: factor out default EPT metadata file name

* improvement: make "then" into an "await"

* refactor: refactor annotation translation

* chore: make public variable private-gettable

* fix: correct import path

* fix: refer to private field, not public getter

* refactor: rewrite point cloud buffer parsing

* improvement: be stricter in specifying optional types

* improvement: make use of color constants

* improvement: rewrite 0-vector check

* fix: correct paratheses in shader

* improvement: mappify array transformation

* improvement: make ShapeType an enum

* improvement: remove out-commented import

* chore: lint fix

* fix: make CogniteClientPlayground optionality annotation stricter

* test: add trivial test for CognitePointCloudModel

* fix: add missing creation function file

* refactor: move some responsibility from PointCloudManager to -Factory

* improvement: then -> await

* fix: make indices be of type Uint32

* feat: add fromThreeVector3 utility function

* refactor: parser-worker logic

* improvement: make parse(data) actually return parsed data

* improvement: improve readability

* refactor: simplify primitive translation

* improvement: renamed/simplify objectId assignment

* refactor: change terminology from  "StyledObject" to "StylableObject"

* refactor: simplify Box constructor

* chore: minor cleanup

* improvement: simplify composite bounding box computation

* fix: correct Box constructor call

* fix: repair, please type checker, and lint after merge

* improvement: remove computeBoundingBox for shapes for now

They'll be back for the acceleration structure

* chore: another lint fix

* chore: add warning regarding point-object matching

* refactor: remove "StylableObjectInfo" abstraction

* chore: lint fix

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: Lars Moastuen <lars.moastuen@cognite.com>
Co-authored-by: Christopher J. Tannum <christopher.tannum@cognite.com>
Co-authored-by: cognite-bulldozer[bot] <51074376+cognite-bulldozer[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-update Makes bulldozer automatically update this PR when there are changes to the target branch slack PRs and issues with this label will be pushed to Slack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants