Compare commits
3 Commits
feature/IS
...
feature/IS
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
dbf7d5ddde | ||
| babd667c5f | |||
|
|
c3a3338c8f |
@@ -1011,14 +1011,14 @@ fun EngineerScreen(
|
|||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > lampCount: $value")
|
Timber.d("onClick > lampCount: $value")
|
||||||
|
|
||||||
engineerViewModel.setLifeTime(
|
mainViewModel.setLifeTime(
|
||||||
lifeTime.copy(
|
lifeTime.copy(
|
||||||
lamp = value
|
lamp = value
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveLifeTimeToPreference()
|
mainViewModel.saveLifeTimeToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showLifeTimeLampCountPopup = false
|
showLifeTimeLampCountPopup = false
|
||||||
@@ -1034,13 +1034,13 @@ fun EngineerScreen(
|
|||||||
onClick = { ok, value ->
|
onClick = { ok, value ->
|
||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > hp5x5: $value")
|
Timber.d("onClick > hp5x5: $value")
|
||||||
engineerViewModel.setLifeTime(
|
mainViewModel.setLifeTime(
|
||||||
lifeTime.copy(
|
lifeTime.copy(
|
||||||
hp5x5 = value
|
hp5x5 = value
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveLifeTimeToPreference()
|
mainViewModel.saveLifeTimeToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showLifeTypeHP5x5Popup = false
|
showLifeTypeHP5x5Popup = false
|
||||||
@@ -1056,13 +1056,13 @@ fun EngineerScreen(
|
|||||||
onClick = { ok, value ->
|
onClick = { ok, value ->
|
||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > hp7x7: $value")
|
Timber.d("onClick > hp7x7: $value")
|
||||||
engineerViewModel.setLifeTime(
|
mainViewModel.setLifeTime(
|
||||||
lifeTime.copy(
|
lifeTime.copy(
|
||||||
hp7x7 = value
|
hp7x7 = value
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveLifeTimeToPreference()
|
mainViewModel.saveLifeTimeToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showLifeTimeHP7x7Popup = false
|
showLifeTimeHP7x7Popup = false
|
||||||
@@ -1078,13 +1078,13 @@ fun EngineerScreen(
|
|||||||
onClick = { ok, value ->
|
onClick = { ok, value ->
|
||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > hp10x10: $value")
|
Timber.d("onClick > hp10x10: $value")
|
||||||
engineerViewModel.setLifeTime(
|
mainViewModel.setLifeTime(
|
||||||
lifeTime.copy(
|
lifeTime.copy(
|
||||||
hp10x10 = value
|
hp10x10 = value
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveLifeTimeToPreference()
|
mainViewModel.saveLifeTimeToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showLifeTimeHP10x10Popup = false
|
showLifeTimeHP10x10Popup = false
|
||||||
@@ -1100,13 +1100,13 @@ fun EngineerScreen(
|
|||||||
onClick = { ok, value ->
|
onClick = { ok, value ->
|
||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > hp12x12: $value")
|
Timber.d("onClick > hp12x12: $value")
|
||||||
engineerViewModel.setLifeTime(
|
mainViewModel.setLifeTime(
|
||||||
lifeTime.copy(
|
lifeTime.copy(
|
||||||
hp12x12 = value
|
hp12x12 = value
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveLifeTimeToPreference()
|
mainViewModel.saveLifeTimeToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showLifeTimeHP12x12Popup = false
|
showLifeTimeHP12x12Popup = false
|
||||||
@@ -1122,13 +1122,13 @@ fun EngineerScreen(
|
|||||||
onClick = { ok, value ->
|
onClick = { ok, value ->
|
||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > hp3x15: $value")
|
Timber.d("onClick > hp3x15: $value")
|
||||||
engineerViewModel.setLifeTime(
|
mainViewModel.setLifeTime(
|
||||||
lifeTime.copy(
|
lifeTime.copy(
|
||||||
hp3x15 = value
|
hp3x15 = value
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveLifeTimeToPreference()
|
mainViewModel.saveLifeTimeToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showLifeTimeHP3x15Popup = false
|
showLifeTimeHP3x15Popup = false
|
||||||
@@ -1143,13 +1143,13 @@ fun EngineerScreen(
|
|||||||
onClick = { ok, value ->
|
onClick = { ok, value ->
|
||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > detector: $value")
|
Timber.d("onClick > detector: $value")
|
||||||
engineerViewModel.setLifeTime(
|
mainViewModel.setLifeTime(
|
||||||
lifeTime.copy(
|
lifeTime.copy(
|
||||||
detector = value
|
detector = value
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveLifeTimeToPreference()
|
mainViewModel.saveLifeTimeToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showLifeTimeDetectorPopup = false
|
showLifeTimeDetectorPopup = false
|
||||||
@@ -1164,13 +1164,13 @@ fun EngineerScreen(
|
|||||||
onClick = { ok, value ->
|
onClick = { ok, value ->
|
||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > water: $value")
|
Timber.d("onClick > water: $value")
|
||||||
engineerViewModel.setLifeTime(
|
mainViewModel.setLifeTime(
|
||||||
lifeTime.copy(
|
lifeTime.copy(
|
||||||
water = value
|
water = value
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveLifeTimeToPreference()
|
mainViewModel.saveLifeTimeToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showLifeTimeWaterPopup = false
|
showLifeTimeWaterPopup = false
|
||||||
@@ -1187,7 +1187,7 @@ fun EngineerScreen(
|
|||||||
onClick = { ok, value ->
|
onClick = { ok, value ->
|
||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > ktp: $value")
|
Timber.d("onClick > ktp: $value")
|
||||||
engineerViewModel.setTemperatureWrite(
|
mainViewModel.setTemperatureWrite(
|
||||||
temperatureWrite.copy(
|
temperatureWrite.copy(
|
||||||
ktp = value.toFloat()
|
ktp = value.toFloat()
|
||||||
)
|
)
|
||||||
@@ -1199,12 +1199,12 @@ fun EngineerScreen(
|
|||||||
CMD.OVEN,
|
CMD.OVEN,
|
||||||
Oven(
|
Oven(
|
||||||
type = 0x41,
|
type = 0x41,
|
||||||
ktp = value.toFloat()
|
ktp = temperatureWrite.ktp
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveTemeratureWriteToPreference()
|
mainViewModel.saveTemeratureWriteToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showTempKTPPopup = false
|
showTempKTPPopup = false
|
||||||
@@ -1220,13 +1220,13 @@ fun EngineerScreen(
|
|||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > chamber1: $value")
|
Timber.d("onClick > chamber1: $value")
|
||||||
|
|
||||||
engineerViewModel.setTemperatureWrite(
|
mainViewModel.setTemperatureWrite(
|
||||||
temperatureWrite.copy(
|
temperatureWrite.copy(
|
||||||
chamber1 = value.toFloat()
|
chamber1 = value.toFloat()
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveTemeratureWriteToPreference()
|
mainViewModel.saveTemeratureWriteToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showTempChamber1Popup = false
|
showTempChamber1Popup = false
|
||||||
@@ -1241,13 +1241,13 @@ fun EngineerScreen(
|
|||||||
onClick = { ok, value ->
|
onClick = { ok, value ->
|
||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > chamber2: $value")
|
Timber.d("onClick > chamber2: $value")
|
||||||
engineerViewModel.setTemperatureWrite(
|
mainViewModel.setTemperatureWrite(
|
||||||
temperatureWrite.copy(
|
temperatureWrite.copy(
|
||||||
chamber2 = value.toFloat()
|
chamber2 = value.toFloat()
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveTemeratureWriteToPreference()
|
mainViewModel.saveTemeratureWriteToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showTempChamber2Popup = false
|
showTempChamber2Popup = false
|
||||||
@@ -1261,13 +1261,13 @@ fun EngineerScreen(
|
|||||||
count = temperatureWrite.water,
|
count = temperatureWrite.water,
|
||||||
onClick = { ok, value ->
|
onClick = { ok, value ->
|
||||||
if (ok) {
|
if (ok) {
|
||||||
engineerViewModel.setTemperatureWrite(
|
mainViewModel.setTemperatureWrite(
|
||||||
temperatureWrite.copy(
|
temperatureWrite.copy(
|
||||||
water = value.toFloat()
|
water = value.toFloat()
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveTemeratureWriteToPreference()
|
mainViewModel.saveTemeratureWriteToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showTempWaterPopup = false
|
showTempWaterPopup = false
|
||||||
@@ -1282,13 +1282,13 @@ fun EngineerScreen(
|
|||||||
onClick = { ok, value ->
|
onClick = { ok, value ->
|
||||||
if (ok) {
|
if (ok) {
|
||||||
Timber.d("onClick > basePlate: $value")
|
Timber.d("onClick > basePlate: $value")
|
||||||
engineerViewModel.setTemperatureWrite(
|
mainViewModel.setTemperatureWrite(
|
||||||
temperatureWrite.copy(
|
temperatureWrite.copy(
|
||||||
basePlate = value.toFloat()
|
basePlate = value.toFloat()
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveTemeratureWriteToPreference()
|
mainViewModel.saveTemeratureWriteToPreference()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
showTempBaseplatePopup = false
|
showTempBaseplatePopup = false
|
||||||
@@ -1393,9 +1393,9 @@ fun EngineerScreen(
|
|||||||
mainViewModel.saveLampCountToPreference()
|
mainViewModel.saveLampCountToPreference()
|
||||||
mainViewModel.saveSprayDcdListToPreference()
|
mainViewModel.saveSprayDcdListToPreference()
|
||||||
mainViewModel.savePresetListToPreference()
|
mainViewModel.savePresetListToPreference()
|
||||||
engineerViewModel.saveLifeTimeToPreference()
|
mainViewModel.saveLifeTimeToPreference()
|
||||||
engineerViewModel.saveTemeratureWriteToPreference()
|
mainViewModel.saveTemeratureWriteToPreference()
|
||||||
engineerViewModel.saveEnergyMeasuredWriteToPreference()
|
mainViewModel.saveEnergyMeasuredWriteToPreference()
|
||||||
mainViewModel.saveEnergyRefer2ToPreference()
|
mainViewModel.saveEnergyRefer2ToPreference()
|
||||||
mainViewModel.saveProductSerialListToPreference()
|
mainViewModel.saveProductSerialListToPreference()
|
||||||
mainViewModel.savePowerSupplySerialListToPreference()
|
mainViewModel.savePowerSupplySerialListToPreference()
|
||||||
@@ -1521,14 +1521,14 @@ fun EngineerScreen(
|
|||||||
},
|
},
|
||||||
onClickSet = { v ->
|
onClickSet = { v ->
|
||||||
// write measured value
|
// write measured value
|
||||||
engineerViewModel.setEnergyMeasuredWrite(
|
mainViewModel.setEnergyMeasuredWrite(
|
||||||
EnergyMeasured(
|
EnergyMeasured(
|
||||||
measured = v
|
measured = v
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
|
||||||
scope.launch {
|
scope.launch {
|
||||||
engineerViewModel.saveEnergyMeasuredWriteToPreference()
|
mainViewModel.saveEnergyMeasuredWriteToPreference()
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
|||||||
@@ -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)
|
||||||
|
|||||||
@@ -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)
|
||||||
|
|||||||
@@ -2130,11 +2130,10 @@ class MainViewModel @Inject constructor(
|
|||||||
} else {
|
} else {
|
||||||
0f
|
0f
|
||||||
}
|
}
|
||||||
val repetitionAngle = if (newRepetitionList.isNotEmpty()) {
|
val repetitionAngle = repetitionIndexToAngle(
|
||||||
repetitionStep.stepToDegree(totalSteps = newRepetitionList.size)
|
index = repetitionStep,
|
||||||
} else {
|
totalSteps = newRepetitionList.size,
|
||||||
0f
|
)
|
||||||
}
|
|
||||||
|
|
||||||
Timber.d("pulseStep: ${pulseStep} fluenceStep: ${fluenceStep} repetitionStep: ${repetitionStep}")
|
Timber.d("pulseStep: ${pulseStep} fluenceStep: ${fluenceStep} repetitionStep: ${repetitionStep}")
|
||||||
|
|
||||||
@@ -2178,23 +2177,21 @@ class MainViewModel @Inject constructor(
|
|||||||
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
|
||||||
@@ -2301,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)
|
||||||
}
|
}
|
||||||
@@ -2313,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
|
||||||
|
|||||||
237
docs/change_summary_2026-03-10.md
Normal file
237
docs/change_summary_2026-03-10.md
Normal 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` 변경 시 잘못된 필드가 갱신되던 버그도 함께 바로잡은 변경입니다.
|
||||||
Reference in New Issue
Block a user