14 Commits

Author SHA1 Message Date
StevenBuzzi
dbf7d5ddde 1. Load에서 편집 기능 사용 시 Repetition 변경할 경우 앱이 튕기는 증상 수정
2. Pulse Duration 및 Fluence 변경 할 경우 Repetition 제한되는 값이 최대 범위를 벗어날 경우, 변경된 Pulse Duration 및 Fluence에 맞게 최대값으로 설정되도록 수정
2026-03-11 09:02:06 +09:00
babd667c5f Merge pull request 'Fix repetition angle handling for single-value list' (#61) from feature/ISSUE-27 into develop
Reviewed-on: #61
2026-03-08 07:07:00 +00:00
areumwoo
c3a3338c8f Fix repetition angle handling for single-value list 2026-03-08 15:24:12 +09:00
8a49c3c832 Merge pull request 'feature/ISSUE-57' (#59) from feature/ISSUE-57 into develop
Reviewed-on: #59
2026-03-08 06:08:11 +00:00
areumwoo
ca8d89e9bb Load preset values even without priority 2026-03-08 14:45:27 +09:00
areumwoo
7ec7d8674f Preserve fluence by value when pulse duration changes 2026-03-08 14:26:35 +09:00
e7a5ce0de2 Merge pull request '제공된 에너지 테이블과 매칭' (#58) from feature/ISSUE-13 into develop
Reviewed-on: #58
2026-03-08 05:14:13 +00:00
steven
c8a9323178 제공된 에너지 테이블과 매칭 2026-03-08 14:13:38 +09:00
aa618627df Merge pull request 'DCD 설정 관련 수정' (#55) from feature/ISSUE-51 into develop
Reviewed-on: #55
2026-03-06 04:43:56 +00:00
StevenBuzzi
55a1398136 Merge remote-tracking branch 'origin/feature/ISSUE-51' into feature/ISSUE-51
# Conflicts:
#	app/src/main/java/com/laseroptek/raman/ui/screens/main/MainViewModel.kt
2026-03-06 13:41:20 +09:00
StevenBuzzi
174cb4fb45 1. DCD 설정 시 Packet 0x41 고정으로 전송하도록 수정
2. DCD 설정값 변경 후 ok 누르면 Packet 전송하여 설정되도록 수정
3. 부팅 시 DCD Packet 0x41 고정으로 전송하도록 수정
2026-03-06 13:38:55 +09:00
StevenBuzzi
71ab4cc8d8 1. DCD Delay 설정 가능 범위 수정
2. DCD Pressure 설정 기본값 수정
2026-03-06 13:38:55 +09:00
StevenBuzzi
d2e375b305 Home화면 내 DCD 숫자 표시 위치 및 크기 조정 2026-03-06 13:38:55 +09:00
StevenBuzzi
92b679b996 잘못 적용된 초기화 값 수정 2026-03-06 13:38:07 +09:00
9 changed files with 432 additions and 86 deletions

View File

@@ -9,7 +9,7 @@ package com.laseroptek.raman.const
val HzTable_10_10 = mapOf( val HzTable_10_10 = mapOf(
// Pulse Width = 0.5f // Pulse Width = 0.5f
Pair(0.5f, 0.5f) to KEY_BLUE, Pair(0.5f, 0.5f) to KEY_GREEN,
Pair(0.5f, 0.6f) to KEY_BLUE, Pair(0.5f, 0.6f) to KEY_BLUE,
Pair(0.5f, 0.7f) to KEY_BLUE, Pair(0.5f, 0.7f) to KEY_BLUE,
Pair(0.5f, 0.8f) to KEY_BLUE, Pair(0.5f, 0.8f) to KEY_BLUE,

View File

@@ -24,7 +24,7 @@ val HzTable_5_5 = mapOf(
Pair(0.5f,3.4f) to KEY_BLUE, Pair(0.5f,3.4f) to KEY_BLUE,
Pair(0.5f,3.6f) to KEY_BLUE, Pair(0.5f,3.6f) to KEY_BLUE,
Pair(0.5f,3.8f) to KEY_BLUE, Pair(0.5f,3.8f) to KEY_BLUE,
Pair(0.5f,4.0f) to KEY_YELLOW, Pair(0.5f,4.0f) to KEY_BLUE,
Pair(0.5f,4.2f) to KEY_YELLOW, Pair(0.5f,4.2f) to KEY_YELLOW,
Pair(0.5f,4.4f) to KEY_YELLOW, Pair(0.5f,4.4f) to KEY_YELLOW,
Pair(0.5f,4.6f) to KEY_YELLOW, Pair(0.5f,4.6f) to KEY_YELLOW,

View File

@@ -108,7 +108,7 @@ val HzTable_7_7 = mapOf(
Pair(1.5f, 8.0f) to KEY_GRAY, Pair(1.5f, 8.0f) to KEY_GRAY,
// Pulse Width = 3f // Pulse Width = 3f
Pair(3f, 1.1f) to KEY_RED, Pair(3f, 1.1f) to KEY_GRAY,
Pair(3f, 1.2f) to KEY_GRAY, Pair(3f, 1.2f) to KEY_GRAY,
Pair(3f, 1.3f) to KEY_GRAY, Pair(3f, 1.3f) to KEY_GRAY,
Pair(3f, 1.4f) to KEY_GRAY, Pair(3f, 1.4f) to KEY_GRAY,

View File

@@ -1,5 +1,5 @@
package com.laseroptek.raman.data.model.serial package com.laseroptek.raman.data.model.serial
data class HandPiece( data class HandPiece(
val type:Int = 1 // ascii (00 ~ 99) val type:Int = 0 // ascii (00 ~ 99)
) )

View File

@@ -304,7 +304,7 @@ fun HomeScreen(
pulseType = pulseType, pulseType = pulseType,
handPieceType = handPiece.type, handPieceType = handPiece.type,
type = LaserStatusType.REPETITION, type = LaserStatusType.REPETITION,
angle = if (pulseType == 0 || repetitionList.size < 2) 0f else repetitionAngle, angle = if (pulseType == 0 || repetitionList.isEmpty()) 0f else repetitionAngle,
onChange = { angle -> onChange = { angle ->
if (handPiece.type == 0) { if (handPiece.type == 0) {
Toast.makeText( Toast.makeText(
@@ -315,7 +315,7 @@ fun HomeScreen(
return@LaserControlView return@LaserControlView
} }
if (pulseType == 0 || repetitionList.size < 2) { if (pulseType == 0 || repetitionList.isEmpty()) {
//Toast.makeText(context, "Single pulse type (0 Hz)", Toast.LENGTH_SHORT).show() //Toast.makeText(context, "Single pulse type (0 Hz)", Toast.LENGTH_SHORT).show()
} else { } else {
mainViewModel.onChangeRepetition(angle) mainViewModel.onChangeRepetition(angle)
@@ -332,7 +332,7 @@ fun HomeScreen(
return@LaserControlView return@LaserControlView
} }
if (pulseType == 0 || repetitionList.size < 2) { if (pulseType == 0 || repetitionList.isEmpty()) {
//Toast.makeText(context, "Single pulse type (0 Hz)", Toast.LENGTH_SHORT).show() //Toast.makeText(context, "Single pulse type (0 Hz)", Toast.LENGTH_SHORT).show()
} else { } else {
//mainViewModel.onClickRepetition(state) //mainViewModel.onClickRepetition(state)

View File

@@ -780,21 +780,25 @@ fun PresetLoadPopup(
.noRippleClickable(onClick = { .noRippleClickable(onClick = {
Timber.d("onClick - Preset Load") Timber.d("onClick - Preset Load")
val selectedPreset = val selectedPreset = presetViewModel.getPreset(selectedPresetIndex)
presetViewModel.getPreset(selectedPresetIndex) if (selectedPreset == null) {
val priority = Timber.w("onClick - Preset Load: selectedPreset is null. index=$selectedPresetIndex")
selectedPreset?.priority ?: 0 return@noRippleClickable
}
val priority = selectedPreset.priority
Timber.d("onClick - Preset Load ($priority)") Timber.d("onClick - Preset Load ($priority)")
// if (priority > 0) { // TODO : 검증 필요 if (priority > 0) {
mainViewModel.setSelectedPresetIndex( priority ) mainViewModel.setSelectedPresetIndex( priority )
mainViewModel.applyPreset(priority) mainViewModel.applyPreset(priority)
presetViewModel.clearPreset() } else {
onClick.invoke(false) // 우선순위가 NONE인 경우, 인덱스 설정 X
// } else { mainViewModel.setSelectedPresetIndex(0)
// Timber.d("SKIP - Preset Load ($priority)") mainViewModel.applyPreset(selectedPreset)
// } }
presetViewModel.clearPreset()
onClick.invoke(false)
}) })
.size(40.px.dp) .size(40.px.dp)
.background(Color.Transparent) .background(Color.Transparent)

View File

@@ -28,6 +28,7 @@ import kotlinx.coroutines.flow.MutableStateFlow
import kotlinx.coroutines.flow.asStateFlow import kotlinx.coroutines.flow.asStateFlow
import kotlinx.coroutines.launch import kotlinx.coroutines.launch
import timber.log.Timber import timber.log.Timber
import kotlin.math.abs
import javax.inject.Inject import javax.inject.Inject
@HiltViewModel @HiltViewModel
@@ -80,6 +81,49 @@ class PresetViewModel @Inject constructor(
_repetitionList.value = r _repetitionList.value = r
} }
private data class ResolvedRepetitionContext(
val fluence: Float,
val repetitionOptions: List<Float>,
)
private fun resolveRepetitionContext(
pulseWidth: Float,
fluence: Float,
): ResolvedRepetitionContext {
val availableFluenceList = energyTable.value.getKey2ListForKey1(pulseWidth)
val resolvedFluence = availableFluenceList.minByOrNull { abs(it - fluence) }
?: availableFluenceList.firstOrNull()
?: fluence
val hzType = hzTable.value.getValue(pulseWidth, resolvedFluence)
val repetitionOptions = RepetitionsByColorKey[hzType] ?: RepetitionsByColorKey[KEY_YELLOW].orEmpty()
if (hzType == null) {
Timber.w(
"No hzType found for pulseWidth=%s, fluence=%s. Falling back to default repetition list.",
pulseWidth,
resolvedFluence
)
}
setRepetitionList(repetitionOptions)
return ResolvedRepetitionContext(
fluence = resolvedFluence,
repetitionOptions = repetitionOptions,
)
}
private fun resolveNearestLowerOrEqualRepetition(
currentRepetition: Float,
repetitionOptions: List<Float>,
): Float {
return repetitionOptions
.filter { it <= currentRepetition }
.maxOrNull()
?: repetitionOptions.minOrNull()
?: currentRepetition
}
fun loadHzTable(handPieceType: Int) { fun loadHzTable(handPieceType: Int) {
Timber.d("loadHzTable($handPieceType)") Timber.d("loadHzTable($handPieceType)")
@@ -251,21 +295,17 @@ class PresetViewModel @Inject constructor(
// 4. fluence update (energyTable) // 4. fluence update (energyTable)
val newFluenceList = energyTable.value.getKey2ListForKey1(newPulseWidth) val newFluenceList = energyTable.value.getKey2ListForKey1(newPulseWidth)
val newFluence = newFluenceList.get(0) val newFluence = newFluenceList.firstOrNull() ?: currentList[index].fluence
// 5. repetition update (hzTable) // 5. repetition update (hzTable)
val hzType = hzTable.value.getValue(newPulseWidth, newFluence) val repetitionContext = resolveRepetitionContext(newPulseWidth, newFluence)
Timber.d("hzType: ${hzType}") Timber.d("Resolved repetition options for handPieceType update: ${repetitionContext.repetitionOptions}")
val list = RepetitionsByColorKey.get(hzType)!! val newRepetition = repetitionContext.repetitionOptions.firstOrNull() ?: currentList[index].repetition
setRepetitionList( list )
val newRepetitionIndex = 0
val newRepetition = list[newRepetitionIndex]
val updatedPreset = currentList[index].copy( val updatedPreset = currentList[index].copy(
handPieceType = newHandPieceType, handPieceType = newHandPieceType,
pulseWidth = newPulseWidth, pulseWidth = newPulseWidth,
fluence = newFluence, fluence = repetitionContext.fluence,
repetition = newRepetition repetition = newRepetition
) )
currentList[index] = updatedPreset currentList[index] = updatedPreset
@@ -311,20 +351,18 @@ class PresetViewModel @Inject constructor(
// 3. update fluence // 3. update fluence
val newFluenceList = energyTable.value.getKey2ListForKey1(newPulseWidth) val newFluenceList = energyTable.value.getKey2ListForKey1(newPulseWidth)
val newFluenceIndex = 0 val newFluence = newFluenceList.firstOrNull() ?: currentList[index].fluence
val newFluence = newFluenceList.get(newFluenceIndex)
// 4. update repetition // 4. update repetition
val hzType = hzTable.value.getValue(newPulseWidth, newFluence) val repetitionContext = resolveRepetitionContext(newPulseWidth, newFluence)
val list = RepetitionsByColorKey[hzType]!! val newRepetition = resolveNearestLowerOrEqualRepetition(
setRepetitionList( list ) currentRepetition = currentList[index].repetition,
repetitionOptions = repetitionContext.repetitionOptions,
val newRepetitionIndex = 0 )
val newRepetition = repetitionList.value[newRepetitionIndex]
val updatedPreset = currentList[index].copy( val updatedPreset = currentList[index].copy(
pulseWidth = newPulseWidth, pulseWidth = newPulseWidth,
fluence = newFluence, fluence = repetitionContext.fluence,
repetition = newRepetition repetition = newRepetition
) )
currentList[index] = updatedPreset currentList[index] = updatedPreset
@@ -354,17 +392,16 @@ class PresetViewModel @Inject constructor(
} }
// 3. update repetition (0) // 3. update repetition (0)
val hzType = hzTable.value.getValue(pulseWidth, newFluence) val repetitionContext = resolveRepetitionContext(pulseWidth, newFluence)
val list = RepetitionsByColorKey[hzType]!! val newRepetition = resolveNearestLowerOrEqualRepetition(
setRepetitionList( list ) currentRepetition = currentList[index].repetition,
repetitionOptions = repetitionContext.repetitionOptions,
val newRepetitionIndex = 0 )
val newRepetition = repetitionList.value[newRepetitionIndex]
// The original code here had a bug: it was updating `pulseWidth = newFluence` // The original code here had a bug: it was updating `pulseWidth = newFluence`
// It should be `fluence = newFluence` // It should be `fluence = newFluence`
val updatedPreset = currentList[index].copy( val updatedPreset = currentList[index].copy(
fluence = newFluence, fluence = repetitionContext.fluence,
repetition = newRepetition repetition = newRepetition
) )
currentList[index] = updatedPreset currentList[index] = updatedPreset
@@ -385,20 +422,28 @@ class PresetViewModel @Inject constructor(
loadHzTable(handPieceType) loadHzTable(handPieceType)
// update repetition // update repetition
val hzType = hzTable.value.getValue(pulseWidth, fluence) val repetitionContext = resolveRepetitionContext(pulseWidth, fluence)
val list = RepetitionsByColorKey[hzType]!! val currentRepetitionList = repetitionContext.repetitionOptions
setRepetitionList( list )
val newRepetitionIndex = repetitionList.value.indexOf(repetition).takeIf { it != -1 } ?: 0 val newRepetitionIndex = currentRepetitionList.indexOf(repetition).takeIf { it != -1 } ?: 0
val newRepetition = if (isUp == UpDownState.Up) { val newRepetition = if (isUp == UpDownState.Up) {
// Original logic: repetitionIndex < PulseDurations.size -1 (Likely a typo, should be Repetitions.size) if (newRepetitionIndex < currentRepetitionList.size - 1) {
if (newRepetitionIndex < repetitionList.value.size - 1) repetitionList.value[newRepetitionIndex + 1] else repetition currentRepetitionList[newRepetitionIndex + 1]
} else {
repetition
}
} else { } else {
// Original logic: repetitionIndex > 1 if (newRepetitionIndex > 0) {
if (newRepetitionIndex > 0) repetitionList.value[newRepetitionIndex - 1] else repetition currentRepetitionList[newRepetitionIndex - 1]
} else {
repetition
}
} }
val updatedPreset = currentList[index].copy(repetition = newRepetition) val updatedPreset = currentList[index].copy(
fluence = repetitionContext.fluence,
repetition = newRepetition
)
currentList[index] = updatedPreset currentList[index] = updatedPreset
//_presetList.value = currentList //_presetList.value = currentList
setPresetList(currentList) setPresetList(currentList)

View File

@@ -134,6 +134,7 @@ import kotlinx.coroutines.joinAll
import kotlinx.coroutines.launch import kotlinx.coroutines.launch
import kotlinx.coroutines.withContext import kotlinx.coroutines.withContext
import timber.log.Timber import timber.log.Timber
import kotlin.math.abs
import javax.inject.Inject import javax.inject.Inject
import kotlin.experimental.or import kotlin.experimental.or
import kotlin.time.Duration.Companion.milliseconds import kotlin.time.Duration.Companion.milliseconds
@@ -842,8 +843,9 @@ class MainViewModel @Inject constructor(
// tx DCD_GAS Write (DEFAULT VALUE) // tx DCD_GAS Write (DEFAULT VALUE)
txPacket(READ_WRITE.WRITE, CMD.DCD_GAS, dcdGas.value.copy(status = 0x50)) txPacket(READ_WRITE.WRITE, CMD.DCD_GAS, dcdGas.value.copy(status = 0x50))
// tx SPRAY_DCD Write (DEFAULT VALUE) // tx SPRAY_DCD Write (DEFAULT VALUE)
txPacket(READ_WRITE.WRITE, CMD.SPRAY_DCD, sprayDcd.value.copy(status = 0x41)) txPacket(READ_WRITE.WRITE, CMD.SPRAY_DCD, sprayDcd.value.copy(status = 0x41))
}
} }
private suspend fun waitUntilSerialReady( private suspend fun waitUntilSerialReady(
@@ -2080,28 +2082,58 @@ class MainViewModel @Inject constructor(
Timber.d("preset: ${preset}") Timber.d("preset: ${preset}")
val newPreset = if (preset == null) { val newPreset = if (preset == null) {
val defaultPulseWidth = PulseDurations.first()
Preset( Preset(
handPieceType = handPiece.value.type, handPieceType = handPiece.value.type,
priority = priority, priority = priority,
fluence = energyTable.value.getKey2ListForKey1(0.5f).first(), fluence = energyTable.value.getKey2ListForKey1(defaultPulseWidth).firstOrNull() ?: 0f,
repetition = repetitionList.value.first(), repetition = repetitionList.value.first(),
pulseWidth = PulseDurations.first(), pulseWidth = defaultPulseWidth,
) )
} else { } else {
preset preset
} }
applyPreset(newPreset)
}
val fluenceList = energyTable.value.getKey2ListForKey1(newPreset.pulseWidth) fun applyPreset(preset: Preset) {
val newPreset = preset.copy()
val pulseStep = PulseDurations.indexOf(newPreset.pulseWidth).takeIf {it != -1} ?: 0 val pulseStep = PulseDurations.indexOf(newPreset.pulseWidth).takeIf {it != -1} ?: 0
val fluenceStep = fluenceList.indexOf(newPreset.fluence).takeIf {it != -1} ?: 0 val resolvedPulseWidth = PulseDurations[pulseStep]
val repetitionStep = repetitionList.value.indexOf(newPreset.repetition).takeIf {it != -1} ?: 0
// 프리셋의 pulseWidth 기준으로 실제 사용 가능한 fluence 목록을 먼저 동기화한다.
val newFluenceList = energyTable.value.getKey2ListForKey1(resolvedPulseWidth)
if (newFluenceList != fluenceList.value) {
setFluenceList(newFluenceList)
}
// 프리셋 fluence가 정확히 없을 수 있으므로, 가장 가까운 값으로 보정한다.
val resolvedFluence = newFluenceList.minByOrNull { abs(it - newPreset.fluence) }
?: newFluenceList.firstOrNull()
?: 0f
val fluenceStep = newFluenceList.indexOf(resolvedFluence).takeIf { it != -1 } ?: 0
// 보정된 (pulseWidth, fluence) 조합으로 repetition 테이블을 다시 계산한다.
val hzType = hzTable.value.getValue(resolvedPulseWidth, resolvedFluence)
val newRepetitionList = RepetitionsByColorKey[hzType] ?: RepetitionsByColorKey[KEY_YELLOW]!!
if (newRepetitionList != repetitionList.value) {
setRepetitionList(newRepetitionList)
}
val repetitionStep = newRepetitionList.indexOf(newPreset.repetition).takeIf { it != -1 } ?: 0
Timber.d("pulseStep: ${pulseStep} fluenceStep: ${fluenceStep} repetitionStep: ${repetitionStep}") Timber.d("pulseStep: ${pulseStep} fluenceStep: ${fluenceStep} repetitionStep: ${repetitionStep}")
val pulseAngle = pulseStep.stepToDegree(totalSteps = PulseDurations.size) val pulseAngle = pulseStep.stepToDegree(totalSteps = PulseDurations.size)
val fluenceAngle = fluenceStep.stepToDegree(totalSteps = fluenceList.size) val fluenceAngle = if (newFluenceList.isNotEmpty()) {
val repetitionAngle = repetitionStep.stepToDegree(totalSteps = repetitionList.value.size) fluenceStep.stepToDegree(totalSteps = newFluenceList.size)
} else {
0f
}
val repetitionAngle = repetitionIndexToAngle(
index = repetitionStep,
totalSteps = newRepetitionList.size,
)
Timber.d("pulseStep: ${pulseStep} fluenceStep: ${fluenceStep} repetitionStep: ${repetitionStep}") Timber.d("pulseStep: ${pulseStep} fluenceStep: ${fluenceStep} repetitionStep: ${repetitionStep}")
@@ -2131,30 +2163,35 @@ class MainViewModel @Inject constructor(
setFluenceList(newFluenceList) setFluenceList(newFluenceList)
} }
// 2-1. Resolve and apply fluence by value (not by old angle/index).
val resolvedFluence = newFluenceList.minByOrNull { abs(it - newFluence) } ?: 0f
val resolvedFluenceIndex = newFluenceList.indexOf(resolvedFluence).takeIf { it >= 0 } ?: 0
if (newFluenceList.isNotEmpty()) {
setFluenceAngle(resolvedFluenceIndex.stepToDegree(totalSteps = newFluenceList.size))
}
// 3. Safely Update Repetition List (Prevents NullPointerException) // 3. Safely Update Repetition List (Prevents NullPointerException)
val newHzType = hzTable.value.getValue(newPulseDuration, newFluence) val newHzType = hzTable.value.getValue(newPulseDuration, resolvedFluence)
val newRepetitionList = RepetitionsByColorKey[newHzType] ?: RepetitionsByColorKey[KEY_YELLOW]!! val newRepetitionList = RepetitionsByColorKey[newHzType] ?: RepetitionsByColorKey[KEY_YELLOW]!!
if (newRepetitionList != repetitionList.value) { if (newRepetitionList != repetitionList.value) {
setRepetitionList(newRepetitionList) setRepetitionList(newRepetitionList)
} }
// Smartly preserve Repetition Angle // 4. Resolve and apply repetition by value (not by old angle/index).
// 4. Check if the old repetition value exists in the new list. val targetRepetition = currentRepetitionValue ?: 0f
val oldRepetitionIndex = if (currentRepetitionValue != null) { val resolvedRepetition = newRepetitionList.minByOrNull { abs(it - targetRepetition) } ?: 0f
newRepetitionList.indexOf(currentRepetitionValue) val resolvedRepetitionIndex = newRepetitionList.indexOf(resolvedRepetition).takeIf { it >= 0 } ?: 0
} else {
-1
}
if (oldRepetitionIndex != -1) { if (newRepetitionList.isNotEmpty()) {
// If the old value exists, set the slider to that position. val resolvedRepetitionAngle = repetitionIndexToAngle(
val preservedAngle = oldRepetitionIndex.stepToDegree(totalSteps = newRepetitionList.size) index = resolvedRepetitionIndex,
setRepetitionAngle(preservedAngle) totalSteps = newRepetitionList.size,
Timber.d("Repetition value $currentRepetitionValue preserved at new angle $preservedAngle.") )
setRepetitionAngle(resolvedRepetitionAngle)
Timber.d("Resolved repetition by value: old=$currentRepetitionValue, new=$resolvedRepetition, angle=$resolvedRepetitionAngle")
} else { } else {
// If it doesn't exist, THEN reset the slider to the beginning.
setRepetitionAngle(0f) setRepetitionAngle(0f)
Timber.d("Repetition value $currentRepetitionValue not supported in new list. Resetting angle.") Timber.d("Repetition list is empty. Resetting repetition angle to 0f.")
} }
// 5. Conditionally reset the fluence angle slider if needed // 5. Conditionally reset the fluence angle slider if needed
@@ -2167,7 +2204,7 @@ class MainViewModel @Inject constructor(
// 6. Any change invalidates the current preset selection. // 6. Any change invalidates the current preset selection.
setSelectedPresetIndex(0) setSelectedPresetIndex(0)
Timber.d("Updated Laser Parameters: pulse=$newPulseDuration, fluence=$newFluence -> newRepListSize=${newRepetitionList.size}") Timber.d("Updated Laser Parameters: pulse=$newPulseDuration, fluence=$resolvedFluence -> newRepListSize=${newRepetitionList.size}")
} }
/** /**
@@ -2187,13 +2224,16 @@ class MainViewModel @Inject constructor(
setPulseAngle(newPulseStep.stepToDegree(totalSteps = PulseDurations.size)) setPulseAngle(newPulseStep.stepToDegree(totalSteps = PulseDurations.size))
val newPulseDuration = PulseDurations[newPulseStep] val newPulseDuration = PulseDurations[newPulseStep]
// When pulse duration changes via slider, we use the first available fluence for the new list. // Keep the current fluence value when pulse duration changes via slider.
val firstFluence = energyTable.value.getKey2ListForKey1(newPulseDuration).firstOrNull() ?: 0f val currentFluenceStep = fluenceAngle.value.degreeToStep(totalSteps = fluenceList.value.size)
val currentFluence = fluenceList.value.getOrNull(currentFluenceStep)
?: energyTable.value.getKey2ListForKey1(newPulseDuration).firstOrNull()
?: 0f
// Call the centralized helper, resetting the fluence slider. // Call the centralized helper using the preserved fluence value.
updateLaserParameters( updateLaserParameters(
newPulseDuration = newPulseDuration, newPulseDuration = newPulseDuration,
newFluence = firstFluence, newFluence = currentFluence,
) )
} }
@@ -2205,12 +2245,17 @@ class MainViewModel @Inject constructor(
setPulseAngle(newStep.stepToDegree(totalSteps = PulseDurations.size)) setPulseAngle(newStep.stepToDegree(totalSteps = PulseDurations.size))
val newPulseDuration = PulseDurations[newStep] val newPulseDuration = PulseDurations[newStep]
val firstFluence = energyTable.value.getKey2ListForKey1(newPulseDuration).firstOrNull() ?: 0f // Keep the current fluence value when pulse duration changes via slider.
val currentFluenceStep = fluenceAngle.value.degreeToStep(totalSteps = fluenceList.value.size)
val currentFluence = fluenceList.value.getOrNull(currentFluenceStep)
?: energyTable.value.getKey2ListForKey1(newPulseDuration).firstOrNull()
?: 0f
// Call the centralized helper, resetting the fluence slider. // Call the centralized helper, resetting the fluence slider.
updateLaserParameters( updateLaserParameters(
newPulseDuration = newPulseDuration, newPulseDuration = newPulseDuration,
newFluence = firstFluence, newFluence = currentFluence,
) )
} }
} }
@@ -2253,8 +2298,12 @@ class MainViewModel @Inject constructor(
fun onChangeRepetition(angle: Float) { fun onChangeRepetition(angle: Float) {
// This logic is simple and has no complex dependencies, so it can remain as is. // This logic is simple and has no complex dependencies, so it can remain as is.
if (angle != repetitionAngle.value) { val normalizedAngle = repetitionIndexToAngle(
setRepetitionAngle(angle) index = angle.degreeToStep(totalSteps = repetitionList.value.size),
totalSteps = repetitionList.value.size,
)
if (normalizedAngle != repetitionAngle.value) {
setRepetitionAngle(normalizedAngle)
} }
setSelectedPresetIndex(0) setSelectedPresetIndex(0)
} }
@@ -2265,12 +2314,23 @@ class MainViewModel @Inject constructor(
val newStep = if (state == UpDownState.Up) currentStep + 1 else currentStep - 1 val newStep = if (state == UpDownState.Up) currentStep + 1 else currentStep - 1
if (newStep in repetitionList.value.indices) { if (newStep in repetitionList.value.indices) {
val newRepetitionAngle = newStep.stepToDegree(totalSteps = repetitionList.value.size) val newRepetitionAngle = repetitionIndexToAngle(
index = newStep,
totalSteps = repetitionList.value.size,
)
setRepetitionAngle(newRepetitionAngle) setRepetitionAngle(newRepetitionAngle)
setSelectedPresetIndex(0) setSelectedPresetIndex(0)
} }
} }
private fun repetitionIndexToAngle(index: Int, totalSteps: Int): Float {
return when {
totalSteps <= 0 -> 0f
totalSteps == 1 -> 270f
else -> index.stepToDegree(totalSteps = totalSteps)
}
}
/** /**
* Perform all heavy I/O in a single background block. * Perform all heavy I/O in a single background block.
* This prevents the "Skipped frames" caused by 30+ sequential bridge calls * This prevents the "Skipped frames" caused by 30+ sequential bridge calls

View File

@@ -0,0 +1,237 @@
# 변경 소스 설명 (2026-03-10)
## 대상
현재 워크스페이스에서 수정된 항목은 아래와 같습니다.
- `app/src/main/java/com/laseroptek/raman/ui/screens/home/preset/PresetViewModel.kt`
- `app/build.gradle.kts`
실제 로직 변경은 `PresetViewModel.kt`에 있습니다.
`app/build.gradle.kts``git status`상 수정으로 보이지만, 확인한 텍스트 diff 기준으로는 의미 있는 코드 변경이 보이지 않았습니다. 줄바꿈(CRLF/LF) 같은 형식 변화일 가능성이 큽니다.
## 이 파일이 하는 일
`PresetViewModel.kt`는 프리셋 화면의 상태를 관리하는 코드입니다.
쉽게 말하면 사용자가 화면에서 아래 값을 올리거나 내릴 때:
- 핸드피스 종류
- 펄스폭 (`pulseWidth`)
- 에너지 단계 (`fluence`)
- 반복수 (`repetition`)
그 조합에 맞는 다음 값들을 다시 계산해서 화면에 반영하는 역할입니다.
## 이번 변경의 핵심
핵심은 반복수(`repetition`) 계산을 더 안전하고 더 자연스럽게 만든 것입니다.
이전 방식은 다음 문제가 생길 수 있었습니다.
- `pulseWidth``fluence` 조합이 테이블에 정확히 없으면 적절한 반복수 목록을 못 찾을 수 있음
- 못 찾았을 때 강제 `!!` 사용으로 앱이 비정상 종료될 가능성이 있음
- `fluence` 변경 시 다른 필드를 잘못 갱신하는 버그가 있었음
이번 수정에서는 공통 계산 함수가 추가되어, 값이 정확히 일치하지 않아도 가장 가까운 `fluence`를 찾아서 처리하도록 바뀌었습니다.
또한 `pulse duration`이나 `fluence`를 바꿀 때 repetition이 무조건 목록의 첫 값(`1Hz` 등)으로 초기화되지 않도록 규칙이 바뀌었습니다.
새 규칙은 아래와 같습니다.
- 현재 repetition 값 이하인 후보들 중
- 가장 큰 값을 선택
- 만약 그런 값이 하나도 없으면 새 목록의 최소값 사용
예를 들어 현재 repetition이 `10Hz`이고, 새 조건에서 가능한 repetition 목록이 `1, 3, 5, 8`이라면 결과는 `8Hz`가 됩니다.
즉, "현재 값에 최대한 가깝게 유지하되, 새 조건에서 허용되는 값으로 내린다"는 방식입니다.
## 새로 추가된 구조
### 1. `ResolvedRepetitionContext`
```kotlin
private data class ResolvedRepetitionContext(
val fluence: Float,
val repetitionOptions: List<Float>,
)
```
이 구조는 반복수 계산 결과를 한 번에 묶어서 전달하기 위한 작은 데이터 상자입니다.
- `fluence`: 실제로 확정된 에너지 값
- `repetitionOptions`: 그 조건에서 선택 가능한 반복수 목록
### 2. `resolveRepetitionContext()`
이 함수가 이번 수정의 핵심입니다.
동작 순서는 아래와 같습니다.
1. 현재 `pulseWidth`에 대해 사용 가능한 `fluence` 목록을 가져옵니다.
2. 사용자가 가지고 있던 `fluence`와 가장 가까운 실제 값을 찾습니다.
3. 그 조합으로 `hzType`을 찾습니다.
4. `hzType`에 맞는 반복수 목록을 가져옵니다.
5. 만약 `hzType`을 찾지 못하면 기본값(`KEY_YELLOW`) 목록으로 대체합니다.
6. 최종 반복수 목록을 `setRepetitionList()`로 화면 상태에 반영합니다.
즉, 예전보다 훨씬 덜 깨지도록 방어 로직이 들어갔습니다.
### 3. `resolveNearestLowerOrEqualRepetition()`
이 함수는 repetition 선택 규칙을 담당합니다.
동작은 단순합니다.
1. 새 repetition 목록에서 현재 repetition 이하인 값만 고릅니다.
2. 그중 가장 큰 값을 선택합니다.
3. 없다면 새 목록의 최소값을 사용합니다.
이 함수가 추가되면서 `pulse duration` 또는 `fluence` 변경 시 repetition이 불필요하게 `1Hz`로 떨어지지 않게 되었습니다.
## 함수별로 무엇이 바뀌었는가
### 1. `updateHandPieceType()`
핸드피스 종류를 바꾸면:
- 해당 핸드피스에 맞는 테이블을 다시 불러오고
- 기본 `pulseWidth`를 고른 뒤
- 가능한 `fluence`를 정하고
- 그에 맞는 `repetition` 목록을 계산합니다
변경 전에는 반복수 목록을 직접 찾다가 실패 가능성이 있었고,
변경 후에는 `resolveRepetitionContext()`를 통해 안전하게 계산합니다.
또한 새 반복수 값도 목록의 첫 번째 값을 안전하게 가져오도록 바뀌었습니다.
### 2. `updatePulseWidth()`
펄스폭을 올리거나 내리면:
-`pulseWidth`에 맞는 `fluence`를 다시 정하고
- 그 조합에 맞는 `repetition` 목록을 다시 계산합니다
여기도 동일하게 공통 함수 사용으로 바뀌어,
정확히 일치하는 값이 없을 때 현재 값 또는 가장 가까운 값으로 안전하게 처리합니다.
추가로 repetition 선택 방식도 바뀌었습니다.
- 예전: 새 repetition 목록의 첫 번째 값으로 사실상 초기화
- 현재: 기존 repetition 이하에서 가장 가까운 최대값 선택
예:
- 기존 repetition = `10Hz`
- 새 목록 = `1, 3, 5, 8`
- 결과 = `8Hz`
### 3. `updateFluence()`
이 부분은 버그 수정이 포함되어 있어 중요합니다.
기존 코드에는 주석으로도 남아 있듯이,
원래 `fluence`를 업데이트해야 하는 상황에서 잘못된 필드를 갱신하던 문제가 있었습니다.
수정 후에는:
- `pulseWidth`에 맞는 `fluence` 목록에서 다음/이전 값을 고르고
- 그 값으로 반복수 목록을 다시 계산한 뒤
- repetition도 기존 값 이하에서 가장 가까운 최대값으로 다시 선택하고
- 실제 프리셋에는 `fluence = repetitionContext.fluence`를 저장합니다
즉, 사용자가 에너지 단계를 바꿨을 때 실제 프리셋 데이터도 올바르게 바뀌도록 고친 것입니다.
이제 `fluence` 변경 시에도 repetition이 불필요하게 최소값으로 초기화되지 않습니다.
### 4. `updateRepetition()`
반복수만 변경할 때도 사실은 현재 `pulseWidth``fluence`에 맞는 반복수 목록이 먼저 계산되어야 합니다.
이번 수정 후에는:
- 현재 조건에 맞는 반복수 목록을 공통 함수로 얻고
- 그 목록 안에서 위/아래 이동을 수행하고
- 필요하면 `fluence`도 보정된 값으로 같이 저장합니다
이전 코드보다 현재 조건과 실제 목록이 더 일관되게 맞춰집니다.
## 왜 이 수정이 필요한가
이 코드는 여러 테이블을 조합해서 동작합니다.
- `energyTable`: `(pulseWidth, fluence)` 조합에 대한 에너지 정보
- `hzTable`: `(pulseWidth, fluence)` 조합에 대한 `hzType`
- `RepetitionsByColorKey`: `hzType`에 따라 선택 가능한 반복수 목록
문제는 사용자가 가진 값이 테이블의 키와 딱 맞지 않을 수 있다는 점입니다.
예를 들어:
- 현재 `pulseWidth`는 존재하지만
- `fluence`가 소수점 차이로 정확히 일치하지 않거나
- 핸드피스 변경 후 이전 값이 새 테이블에 없는 경우
예전 코드는 이런 경우에 취약했습니다.
이번 수정은 이런 경계 상황에서도 앱이 가능한 범위 안에서 계속 동작하도록 만든 것입니다.
## 코틀린을 모르는 사람 기준으로 문법 설명
### `fun`
함수를 만든다는 뜻입니다.
```kotlin
fun updateFluence(...)
```
### `val`
한 번 정하면 다시 바꾸지 않는 값입니다.
```kotlin
val newFluence = ...
```
### `data class`
관련 있는 값 몇 개를 묶는 작은 자료형입니다.
```kotlin
data class ResolvedRepetitionContext(...)
```
### `copy(...)`
기존 객체를 복사하면서 일부 값만 바꾸는 방식입니다.
```kotlin
currentList[index].copy(
fluence = repetitionContext.fluence,
repetition = newRepetition
)
```
이 코드는 "기존 프리셋을 복사하되 `fluence``repetition`만 새 값으로 바꾼다"는 뜻입니다.
### `?:`
왼쪽 값이 없으면 오른쪽 값을 대신 사용하라는 뜻입니다.
```kotlin
val newFluence = newFluenceList.firstOrNull() ?: currentList[index].fluence
```
즉:
- 목록 첫 값이 있으면 그 값을 쓰고
- 없으면 현재 저장된 `fluence`를 그대로 씁니다
### `firstOrNull()`
목록의 첫 번째 값을 가져오되, 비어 있으면 오류 대신 `null`을 돌려줍니다.
안전한 코드에서 자주 쓰입니다.
## 한 줄 요약
이번 수정은 프리셋 편집 시 `pulseWidth`, `fluence`, `repetition`의 연결 계산을 더 안전하게 만들고, `pulse duration``fluence` 변경 시 repetition을 현재 값에 가장 가깝게 유지하도록 개선했으며, `fluence` 변경 시 잘못된 필드가 갱신되던 버그도 함께 바로잡은 변경입니다.