Time producer gives days from 1970 instead of 1984

Daxbot / node-canopen

CANopen implementation for NodeJS

MIT License

32 stars 11 forks source link

Time producer gives days from 1970 instead of 1984 #29

Closed arkaheb258 closed 3 years ago

arkaheb258 commented 3 years ago

TIME write function (time.js:128) uses conversion typeToRaw (eds.js:242) which operate on epoch timestamp. For CANopen time producer offset should be 441763200 seconds. CAN-CiA.org Wikipedia

wilkinsw commented 3 years ago

You're exactly right, I will fix that this week

wilkinsw commented 3 years ago

Should be fixed in 2.7.2 https://www.npmjs.com/package/canopen/v/2.7.2