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

KTX-산천에 수송량 1명이 표시됨 #164

Open
telk5093 opened this issue Apr 1, 2021 · 10 comments
Open

KTX-산천에 수송량 1명이 표시됨 #164

telk5093 opened this issue Apr 1, 2021 · 10 comments
Labels
bug Something isn't working spec Issue about vehicle's spec technical limit There is a technical limit so it is not able to fix

Comments

@telk5093
Copy link
Member

telk5093 commented Apr 1, 2021

No description provided.

@telk5093 telk5093 added bug Something isn't working spec Issue about vehicle's spec technical limit There is a technical limit so it is not able to fix labels Apr 1, 2021
@telk5093

This comment was marked as outdated.

@cyluss
Copy link

cyluss commented Apr 30, 2021

기대하는 동작: 수송량이 표시되지 않아야 함

@cyluss
Copy link

cyluss commented Apr 30, 2021

@telk5093
Copy link
Member Author

telk5093 commented Apr 30, 2021

https://github.com/KoreanGRF/KoreanTrainSet/blob/67aea49/src/locomotive/KTX/KTX2N.pnml#L48
이 부분은 관계없는건가요?

KTX-산천이 개조가 가능하게 되려면 최소 1명의 수송량이 있어야 하고 그걸 지정하는 부분입니다.
https://github.com/KoreanGRF/KoreanTrainSet/blob/67aea49/src/locomotive/KTX/KTX2N.pnml#L70
에서 cargo_capacitypurchase_cargo_capacity를 0명으로 덮어씌우고는 있는데, dual_head: true인 차량은 뒤쪽 차량에 해당 부분을 적용하지 않는 것 같습니다.
근거는 없으나 개인적으로는 NML의 버그가 아닌가 생각하고 있습니다.

@cyluss
Copy link

cyluss commented May 1, 2021

KoreanTrainSet_issue_164_reprod
재현방법

@cyluss
Copy link

cyluss commented May 1, 2021

image

뒤쪽 차량에 해당 부분을 적용하지 않는 것 같습니다

에 대한 좀더 자세한 설명 부탁드립니다

@telk5093
Copy link
Member Author

telk5093 commented May 1, 2021

image

뒤쪽 차량에 해당 부분을 적용하지 않는 것 같습니다

에 대한 좀더 자세한 설명 부탁드립니다

실제 운행되는 차량 정보 부분 말고,
구매 목록 상에 표출되는 차량 역시 앞/뒤 두 부분으로 구성되어 있는데
앞 부분은 graphic 블록에서 덮어씌운 cargo_capacity: 0;이 적용되지만 뒷 부분이 적용받지 못하는 것으로 추정하고 있습니다.

@cyluss
Copy link

cyluss commented May 1, 2021

네 말씀하신대로 dual_head: 1 동력차는 다르게 작동하네요
https://github.com/OpenTTD/OpenTTD/blob/3dbd647/src/engine.cpp#L220
openttd 동작인듯 합니다

@telk5093
Copy link
Member Author

Note: OpenTTD/OpenTTD#10032 가 이 이슈와 관련된 이슈일 수 있음

@telk5093
Copy link
Member Author

JGRPP 0.48.3에서 해당 이슈를 수정하였으나 여전히 동일 현상 발생. 다른 문제인 것으로...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working spec Issue about vehicle's spec technical limit There is a technical limit so it is not able to fix
Projects
None yet
Development

No branches or pull requests

2 participants