Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Let user set point data type #205

Closed
NubeDev opened this issue Feb 1, 2021 · 3 comments
Closed

Let user set point data type #205

NubeDev opened this issue Feb 1, 2021 · 3 comments
Assignees

Comments

@NubeDev
Copy link
Contributor

NubeDev commented Feb 1, 2021

Let user set point type

  • string
  • float
  • int
@NubeDev NubeDev changed the title Let user set point type Let user set point data type Feb 1, 2021
@RaiBnod RaiBnod self-assigned this Feb 2, 2021
@RaiBnod
Copy link
Member

RaiBnod commented Feb 2, 2021

It should be only for generic point.

@RaiBnod RaiBnod assigned enjuthulung and unassigned RaiBnod Feb 2, 2021
@RaiBnod
Copy link
Member

RaiBnod commented Feb 2, 2021

@NubeDev while we use the point data type as string, we need to change on followings:

On point_store and point_store_history tables:

  • value from float to string (we store 1 as "1", "value" as "value", "20.12" as "20.12")
  • value_original float to string

& also on InfluxDB we will store value as string previously it was storing as float.

Moreover, our priority_array array fields values are also float, they needs to convert it on string.

Please confirm on it before proceeding the changes.

@Shiny380
Copy link
Collaborator

Shiny380 commented Feb 2, 2021

Related #80
This is a major change and should be left until later after better discussion
More important issues to be worked on atm

@Shiny380 Shiny380 closed this as completed Feb 2, 2021
@NubeIO NubeIO locked and limited conversation to collaborators Feb 2, 2021

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants