Skip to content

map write/read bug fix #2

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

map write/read bug fix #2

wants to merge 1 commit into from

Conversation

aoki-rt
Copy link
Collaborator

@aoki-rt aoki-rt commented May 21, 2025

What does this implement/fix?

バグの現象
MapManager.hにある迷路マップのサイズのMAPSIZE_X、MAPSIZE_Yを16以外にした時、flashROMの書き込みを失敗する
修正内容.
SPIFFSのマップの読み書きを16で固定してたものをMAPSIZE_X、MAPSIZE_Yに変更した.

Does this close any currently open issues?

ありません

How has this been tested?

MAPSIZE_X、MAPSIZE_Yを4にし、4x4の迷路で実機で試しました

Any other comments?

対象はSTEP8

Checklists

Copy link

Memory usage change @ 696b676

Board flash % RAM for global variables %
esp32:esp32:esp32s3 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
Click for full report table
Board pico_classic_v4_STEP1_LED
flash
% pico_classic_v4_STEP1_LED
RAM for global variables
% pico_classic_v4_STEP2_SWITCH
flash
% pico_classic_v4_STEP2_SWITCH
RAM for global variables
% pico_classic_v4_STEP3_Buzzer
flash
% pico_classic_v4_STEP3_Buzzer
RAM for global variables
% pico_classic_v4_STEP4_Sensor
flash
% pico_classic_v4_STEP4_Sensor
RAM for global variables
% pico_classic_v4_STEP5_Straight
flash
% pico_classic_v4_STEP5_Straight
RAM for global variables
% pico_classic_v4_STEP6_rotate
flash
% pico_classic_v4_STEP6_rotate
RAM for global variables
% pico_classic_v4_STEP7_P_control
flash
% pico_classic_v4_STEP7_P_control
RAM for global variables
% pico_classic_v4_STEP8_micromouse
flash
% pico_classic_v4_STEP8_micromouse
RAM for global variables
%
esp32:esp32:esp32s3 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0
Click for full report CSV
Board,pico_classic_v4_STEP1_LED<br>flash,%,pico_classic_v4_STEP1_LED<br>RAM for global variables,%,pico_classic_v4_STEP2_SWITCH<br>flash,%,pico_classic_v4_STEP2_SWITCH<br>RAM for global variables,%,pico_classic_v4_STEP3_Buzzer<br>flash,%,pico_classic_v4_STEP3_Buzzer<br>RAM for global variables,%,pico_classic_v4_STEP4_Sensor<br>flash,%,pico_classic_v4_STEP4_Sensor<br>RAM for global variables,%,pico_classic_v4_STEP5_Straight<br>flash,%,pico_classic_v4_STEP5_Straight<br>RAM for global variables,%,pico_classic_v4_STEP6_rotate<br>flash,%,pico_classic_v4_STEP6_rotate<br>RAM for global variables,%,pico_classic_v4_STEP7_P_control<br>flash,%,pico_classic_v4_STEP7_P_control<br>RAM for global variables,%,pico_classic_v4_STEP8_micromouse<br>flash,%,pico_classic_v4_STEP8_micromouse<br>RAM for global variables,%
esp32:esp32:esp32s3,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant