rpuig2001 / CDM

CDM
GNU General Public License v3.0
22 stars 9 forks source link

CID in EvCTOT #147

Closed maxlk96 closed 6 months ago

maxlk96 commented 9 months ago

Hello, would it be possible to base the EVCTOT on CID instead of C/S, or even better: Make use of the VATCAN plugin (or just the website API and the generated event specific code to load CTOTs automatically).

We had an event today on ESSA with slots and had to put in CTOTs manually for booked flights. To be able to manually enter CTOTs we had to press "ready TOBT" before entering the CTOT, then when the pilot reported ready entered "ready TOBT" again.

We have Realmode off in our setup, if that makes any difference.

rpuig2001 commented 8 months ago

Hi @maxlk96, This functionallity was added some time ago, but undortunately I had to get it out as it slowed down a lot the Euroscope performance.

I will keep it as a future issue if we can get the CID internally in Euroscope without using external resources.

maxlk96 commented 8 months ago

@rpuig2001

Would having a txt file with CIDs instead of C/S be possble, grabbing the CIDs from a txt file, as it is now with C/S, maybe having the posibility to use C/S or CID in that txt file would maybe be the best.

If I understand correctly, you mean the use of VATCAN API caused ES to slow down?

rpuig2001 commented 8 months ago

I implemented the purposed option of C/S in the txt file in the past. Nowadays it is not possible to retreive pilot's CID directly in Euroscope. But instead we need to get the CID using VATSIM's API from the plugin. This ends up with too many calls.

maxlk96 commented 8 months ago

Ah I see, too bad.. Maybe an idea to suggest here: https://forum.vatsim.net/t/euroscope-plug-in-api-upgrade/4936

rpuig2001 commented 8 months ago

Ah I see, too bad.. Maybe an idea to suggest here: https://forum.vatsim.net/t/euroscope-plug-in-api-upgrade/4936

Already suggested in this comment -> https://forum.vatsim.net/t/euroscope-plug-in-api-upgrade/4936/6

I hope that ES 64bits and ES API improvment, would make easier the development of some functionallities together with a better Euroscope performance.

maxlk96 commented 8 months ago

Ah I see, too bad.. Maybe an idea to suggest here: https://forum.vatsim.net/t/euroscope-plug-in-api-upgrade/4936

Already suggested in this comment -> https://forum.vatsim.net/t/euroscope-plug-in-api-upgrade/4936/6

I hope that ES 64bits and ES API improvment, would make easier the development of some functionallities together with a better Euroscope performance.

I guess I did not read it through myself...

Interesting times aheas!

Adrian2k commented 7 months ago

Hi @maxlk96,

This functionallity was added some time ago, but undortunately I had to get it out as it slowed down a lot the Euroscope performance.

I will keep it as a future issue if we can get the CID internally in Euroscope without using external resources.

@rpuig2001 Is this how also the VATCAN plugins does it? We was planning to use the CDM plugin for CTP, but the CTP planning team was quite concerned about identifying event slots based on callsign instead of CID, because of the possibility of people using other callsigns and/or stealing callsigns.

So we are suddenly in abit of a dilemma on how to solve this for cross the pond.

rpuig2001 commented 7 months ago

Hi @maxlk96, This functionallity was added some time ago, but undortunately I had to get it out as it slowed down a lot the Euroscope performance. I will keep it as a future issue if we can get the CID internally in Euroscope without using external resources.

@rpuig2001 Is this how also the VATCAN plugins does it? We was planning to use the CDM plugin for CTP, but the CTP planning team was quite concerned about identifying event slots based on callsign instead of CID, because of the possibility of people using other callsigns and/or stealing callsigns.

So we are suddenly in abit of a dilemma on how to solve this for cross the pond.

I can’t say if my previous CID implementation is the same as the used in the VATCAN plugin. As of the timing, I can give a possible workaround:

Changing the column EvCTOT for the VATCAN Plugin CTOTs. Instead of click to “add EvCTOT as CTOT”, add a manual CTOT based on the value of the Vatcan plugin column.

It would take a bit more workload adding the manual CTOT, but internally, the CDM will act the same way.

marud94 commented 7 months ago

Hi @rpuig2001 - just testing this for ENOR during CTP. Found out adding the CTOT manual is the way to go and works quite well. The only issue is that it MUST have a EOBT/TOBT first. This means we needs to have realmode on for the event, despite our package doesn't use that or we need to manually set TOBTs.

Is there any chance to add the possibility to add a manual CTOT for aircrafts without EOBT/TOBT? I know it won't make this CTP, but would be good for the future.

rpuig2001 commented 6 months ago

To be released with next version.