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.

Sign up for GitHub

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

Jump to bottom

Support for SMT TimeSeries #256

Draft
wants to merge 68 commits into
base: main
Choose a base branch
from
Draft

Support for SMT TimeSeries #256

wants to merge 68 commits into from

Conversation

mjacoby
Copy link
Member

@mjacoby mjacoby commented Nov 10, 2022 •
edited

Current Status

  • New interface for SMT Processors in Service
  • Implementation of SMT Processor for TimeSeries SMT with interface for providers for internal and linked segments
  • DefaultInternalSegmentProvider that servers internal segments as-is (i.e. as defined in the model)
  • LinkedSegmentProvider for Influx v1 and v2

@mjacoby mjacoby marked this pull request as ready for review November 10, 2022 10:52
@mjacoby mjacoby marked this pull request as draft November 10, 2022 10:59
mjacoby and others added 20 commits November 10, 2022 12:00
@mjacoby
add empty file to trigger CI pipeline
b665353
@mjacoby
remove empty file
3d09ec9
@mjacoby
adding initial support for LinkedSegments
74ae667
@JensMueller2709
Initial support of InfluxDb linked segment
a677334
@JensMueller2709
Start using model classes of SMT
b6361e4
@mjacoby
Current state of TimeSeries implementation
8bc6429 
* refactored into multiple modules
* added unit test
* splitting InfluxDB support into v1.x and v2.x
* basic InlfuxDB v1.x support working
@mjacoby
* improve unit tests
dec109f 
* refactoring
@mjacoby
Merge branch 'main' into feature/SMT-TimeSeriesData
7fe109b 
# Conflicts:
#	pom.xml
@mjacoby
implement influx provider & tests
5981239
@mjacoby
Merge branch 'main' into feature/SMT-TimeSeriesData
19428e1
@mjacoby
* add OWASP supression for false positive warning
a223a89 
* improve code quality
@mjacoby
fix & clean code
c26fd23
@mjacoby
improve code quality
452ad88
@mjacoby
fix OWASP
cc8b284
@mjacoby
improve code quality
733df27
@mjacoby
improve code quality
879ac49
@mjacoby
fix typo in constant value
c729b28
@mjacoby
Merge branch 'main' into feature/SMT-TimeSeriesData
9516611 
# Conflicts:
#	core/pom.xml
#	owaspSuppressions.xml
@mjacoby
Merge branch 'main' into feature/SMT-TimeSeriesData
754500e
@mjacoby
Merge branch 'main' into feature/SMT-TimeSeriesData
f7f29f6
@sonarcloud

This comment was marked as outdated.

Sign in to view
gemittmann and others added 23 commits July 31, 2023 08:00
@gemittmann
Change Time property in Record.java to be able to use multiple timest…
a737f06 
…amps per record (WiP).

CHange Metadata to use Record instead of internal MetaRecord class.
@gemittmann
Fix failing InfluxDB test.
cbc6ed7
@gemittmann
fix spotless
a4a436a
@gemittmann
small clean up.
8bd3807
@gemittmann
Move calculatePropertiesIfNotPresent to InternalSegment nly
99340d0 
Add semanticID differentiation for Segment property state and valueType differentiation for duration.
@gemittmann
Bump influxdb-client-java version to 6.10. to avoid CVE-2023-3635.
3deddaa 
Add okio-jvm with fixed version to avoid CVE-2023-3635 in influxdb-java.

Remove abstract classes of CSV ExternalSegment implementation.

In SubmodelTemplateProcessor and OperationProviders: map ExternalSegmentProviders with ExternalSegments via shortID
@gemittmann
explicit removal of transient dependency okio-jvm in pom.
03d5c83 
Define and use TimeType.java for different time options defined in the SubmodelTemplate.

Rename variable time to times in Record.
@gemittmann
Small fix and pom change.
f693931
@gemittmann
TimeFactory with ClassGraph (WiP: set search path).
1f1ac58 
InfluxDB support multiple timestamps (WiP: needs tests).

Set Records AAS to ordered=True.

pom version bump to match main branch.
@gemittmann
Merge branch 'main' into feature/SMT-TimeSeriesData
713dded
@gemittmann
clean up.
ab099ac
@gemittmann
Merge branch 'feature/SMT-TimeSeriesData' of https://github.com/Fraun…
7934f91 
…hoferIOSB/FAAAST-Service into feature/SMT-TimeSeriesData
@gemittmann
Change Time model to time representation in long values.
2e4cffa 
Add mapping from column names to AAS property shortIDs.
@gemittmann
Move SemanticID of Time to Annotation.
5ceda03 
Add AbstractTime.java

Fix equals method.
@gemittmann
Change back to zoneddatetime.
1b152c9 
Get semanticID by annotation.

abstract class for Time interface.

Timespan derivation from record via Factory instead of TimeObject.
@mjacoby
Merge branch 'main' into feature/SMT-TimeSeriesData
76e6982
@mjacoby
fix JSON result serialization
3d6a7c8
@mjacoby
cleanup code
528b41a
@mjacoby
fix external segments
2e54285
@mjacoby
Merge branch 'main' into feature/SMT-TimeSeriesData
3a7c5dc
@mjacoby
Merge branch 'main' into feature/SMT-TimeSeriesData
da901f4
@mjacoby
bump versions
59bea4f
@mjacoby
cleanup
666e642
@FraunhoferIOSB FraunhoferIOSB deleted a comment from sonarcloud bot Apr 17, 2024
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Apr 29, 2024

Quality Gate Passed Quality Gate passed

Issues
17 New issues
73 Accepted issues

Measures
0 Security Hotspots
5.3% Coverage on New Code
2.3% Duplication on New Code

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@mjacoby @JensMueller2709 @gemittmann