Skip to content

Conversation

@aa5sh
Copy link

@aa5sh aa5sh commented Mar 18, 2025

I noticed the when looking at code for the email earlier that the DXCC counts were still including Deleted Entities. Mine was showing 323 instead of 320 confirmed. I made an adjustment to correct but not sure if it is the ideal way.

I noticed the when looking at code for the email earlier that the DXCC counts were still including Deleted Entities.  Mine was showing 323 instead of 320 confirmed.  I made an adjustment to correct but not sure if it is the ideal way.
@foldynl
Copy link
Owner

foldynl commented Mar 19, 2025

I'm a little confused now. So should deleted entities be included or not? I fixed it to include deleted entities. This doesn't work?

@aa5sh
Copy link
Author

aa5sh commented Mar 19, 2025 via email

@foldynl
Copy link
Owner

foldynl commented Mar 19, 2025

Please, could you send me an example ? I changed the Awards in 0.42.2 where delete entities ARE included in the details AND are counted at the top - commit da70f4a

@aa5sh
Copy link
Author

aa5sh commented Mar 19, 2025 via email

@foldynl
Copy link
Owner

foldynl commented Mar 19, 2025

Would you consider having an option at the top to include/exclude deleted countries? I just saw the count and knew it was high.

That's a good question. I'll have to look into the DXCC rules. But if DXCC accepts removed entities, then we should NOT implement such an option.

@iu0dhv
Copy link

iu0dhv commented Dec 16, 2025

Hello it was me to suggest to include deleted entities in the list (also because if you want to add past QSOs it is good to have the possibility to include them with the proper dxcc attribution). And actually the are indeed reported by the ARRL in the list of worked countries and properly labelled as deleted. So thank you very much for adding them in one of the latest release of the log.
At the same I would also like to have a flag that allows to include or exclude them from the count of Worked/Confirmed countries on the top lines of the Award window. So it would be great if this could be added!
73

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants