Skip to content
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

[Bug] 화차 유지비용이 매우 낮음? #414

Closed
SerpensNebula opened this issue May 31, 2023 · 3 comments
Closed

[Bug] 화차 유지비용이 매우 낮음? #414

SerpensNebula opened this issue May 31, 2023 · 3 comments
Labels
bug Something isn't working prio: normal spec Issue about vehicle's spec

Comments

@SerpensNebula
Copy link
Member

한국 열차 세트 버전(Version of Korean Train Set):
2.4.0

의도된 사항인지 궁금합니다.

1
2

@SerpensNebula SerpensNebula added the investigate Need an investigate or it is not sure label May 31, 2023
@telk5093
Copy link
Member

telk5093 commented May 31, 2023

객차의 경우 running_cost_base 값이 RUNNING_COST_DIESEL 또는 RUNNING_COST_ELECTRIC 으로 되어있고
화차의 경우 150의 magic number로 하드 코딩되어 있는데, 버그로 보입니다.

RUNNING_COST_DIESEL이나 RUNNING_COST_ELECTRIC은 기관차에 사용하는 유지비로 보이므로 화차와 마찬가지로 150의 magic number로 통일하는 방향으로 수정하겠습니다.

@telk5093 telk5093 added bug Something isn't working prio: normal spec Issue about vehicle's spec and removed investigate Need an investigate or it is not sure labels May 31, 2023
@SerpensNebula
Copy link
Member Author

SerpensNebula commented Jun 1, 2023

제 예상으로는 화차도 객차만큼 비싸게 조정하실 것 같았는데, 반대군요.

※발전차도 유지비용이 객차처럼 설정되어있으니 참고해주시기 바랍니다.

@telk5093
Copy link
Member

telk5093 commented Jun 3, 2023

바닐라 객화차의 경우 유지비가 모두 없는 점,
NML spec 상 running_cost_base는 기관차에 사용하는 값(eg. RUNNING_COST_DIESEL 등)만 있는 점
등을 이유로 화차에서 유지비를 제거하는 것이 맞다고 판단하여 그렇게 변경합니다.

414aa19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working prio: normal spec Issue about vehicle's spec
Projects
None yet
Development

No branches or pull requests

2 participants