-
Notifications
You must be signed in to change notification settings - Fork 7
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
mmyster/feature/lk quad digital to analog converter #940
mmyster/feature/lk quad digital to analog converter #940
Conversation
File comparision analysis report🔖 Info
Click to show memory sections
📝 SummaryClick to show summary
🗺️ Map files diff outputClick to show diff list
|
File comparision analysis report🔖 Info
Click to show memory sections
📝 SummaryClick to show summary
🗺️ Map files diff outputClick to show diff list
|
00972cd
to
e6ca578
Compare
8094d0c
to
2f45bae
Compare
Codecov Report
@@ Coverage Diff @@
## mmyster/feature/quad-digital-to-analog-converter #940 +/- ##
===================================================================================
Coverage ? 95.91%
===================================================================================
Files ? 125
Lines ? 2936
Branches ? 0
===================================================================================
Hits ? 2816
Misses ? 120
Partials ? 0 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
2f45bae
to
a5c68f8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quelques requêtes pour la clarté et un résultat un peu étonnant à expliquer.
Sinon tout est très bien en guise de spike pour montrer comment ça s'utilise, c'est bien 👍
spikes/lk_qdac/main.cpp
Outdated
|
||
log_info("Init !\n\n"); | ||
dac.init(); | ||
dac.writeAllChannels(0x0000, true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Autre détails marquant avec cette ligne, c'est qu'on inscrit 0x0000
pour chaque channel et que après on obtient
000:00:01.002 [INFO] [main.cpp:32] printInputRegistersAndMemory > ================= Channel 0 =================
000:00:01.005 [INFO] [main.cpp:34] printInputRegistersAndMemory > Input Registers === Data: 0
000:00:01.007 [INFO] [main.cpp:36] printInputRegistersAndMemory > EEPROM Memory ===== Data: fff
000:00:01.007 [INFO] [main.cpp:32] printInputRegistersAndMemory > ================= Channel 1 =================
000:00:01.010 [INFO] [main.cpp:34] printInputRegistersAndMemory > Input Registers === Data: 0
000:00:01.013 [INFO] [main.cpp:36] printInputRegistersAndMemory > EEPROM Memory ===== Data: fff
000:00:01.013 [INFO] [main.cpp:32] printInputRegistersAndMemory > ================= Channel 2 =================
000:00:01.016 [INFO] [main.cpp:34] printInputRegistersAndMemory > Input Registers === Data: 0
000:00:01.019 [INFO] [main.cpp:36] printInputRegistersAndMemory > EEPROM Memory ===== Data: fff
000:00:01.019 [INFO] [main.cpp:32] printInputRegistersAndMemory > ================= Channel 3 =================
000:00:01.022 [INFO] [main.cpp:34] printInputRegistersAndMemory > Input Registers === Data: 0
000:00:01.025 [INFO] [main.cpp:36] printInputRegistersAndMemory > EEPROM Memory ===== Data: fff
Pourquoi fff
pour la mémoire et pas 0 comme pour l'input registers?
1b571cf
to
19f5ca8
Compare
spikes/lk_qdac/main.cpp
Outdated
// | ||
|
||
auto corei2c = CoreI2C {PinName::SENSOR_PROXIMITY_MUX_I2C_SDA, PinName::SENSOR_PROXIMITY_MUX_I2C_SCL}; | ||
auto dac = CoreQDACMCP4728 {corei2c, 0xC0}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Même résultat pour l'autre DAC avec l'adresse 0xC2
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reste à corriger les 2 code smells et c'est bon pour moi 👌
19f5ca8
to
0b2a233
Compare
e6ca578
to
ed3cfbb
Compare
ed3cfbb
to
c8e19a2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
aff29a3
to
eb90226
Compare
0b2a233
to
0b9fdc6
Compare
adfa308
to
cfeffdf
Compare
cfeffdf
to
0109d91
Compare
0b9fdc6
to
d95b276
Compare
0109d91
to
0933a53
Compare
d95b276
to
430ae56
Compare
430ae56
to
a2cbd0b
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Need :