Closed BoatrightTBC closed 3 years ago
ok, I'll look at it.
Here is a general question. I am going through the CHV processing and found that some spectra are coming back as Q, ? and d. By default I am mapping these to M class. If you want them to be mapped otherwise, let me know.
Here is a general question. I am going through the CHV processing and found that some spectra are coming back as Q, ? and d. By default I am mapping these to M class. If you want them to be mapped otherwise, let me know.
Works for me, we have to guess anyway, and the only point of this is to load the OLD data so we can see where polities are & etc.
From: LJ Ramones @.> Sent: Friday, March 12, 2021 12:46 PM To: ljramones/trips @.> Cc: Rick Boatright @.>; Author @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Here is a general question. I am going through the CHV processing and found that some spectra are coming back as Q, ? and d. By default I am mapping these to M class. If you want them to be mapped otherwise, let me know.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797677695, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEYUK2E5MBCECWTIP3AKZNTTDJHOLANCNFSM4ZCFLYWA.
ok, I figured out the issue. I also discovered that I need to add the label handling logic to the CHV file imports. I have it for CSV and Excel but not the CHV ones
here are the bad spectral values for CHV : Could not figure out stellar classification: spectra =PseudoString(value=Q9VIId, length=8, name=true) 2021-03-12 14:27:59.699 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q7VIId, length=8, name=true) 2021-03-12 14:28:01.424 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q9VIId, length=8, name=true) 2021-03-12 14:28:05.398 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=QP8VIId, length=9, name=true) 2021-03-12 14:28:14.185 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:27.977 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q9VIId, length=8, name=true) 2021-03-12 14:28:29.315 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=QP9VIId, length=9, name=true) 2021-03-12 14:28:36.912 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q8VIId, length=8, name=true) 2021-03-12 14:28:39.869 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:44.349 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q8VIId, length=8, name=true) 2021-03-12 14:28:45.026 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:45.575 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:46.256 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q5VIId, length=8, name=true) 2021-03-12 14:28:46.863 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q5VIId, length=8, name=true) 2021-03-12 14:28:47.533 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q8VIId, length=8, name=true)
I'm not sure why he would use this mapping
Is "Q" his signal for a fake star I wonder?
From: LJ Ramones @.> Sent: Friday, March 12, 2021 1:36 PM To: ljramones/trips @.> Cc: Rick Boatright @.>; Author @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
here are the bad spectral values for CHV : Could not figure out stellar classification: spectra =PseudoString(value=Q9VIId, length=8, name=true) 2021-03-12 14:27:59.699 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q7VIId, length=8, name=true) 2021-03-12 14:28:01.424 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q9VIId, length=8, name=true) 2021-03-12 14:28:05.398 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=QP8VIId, length=9, name=true) 2021-03-12 14:28:14.185 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:27.977 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q9VIId, length=8, name=true) 2021-03-12 14:28:29.315 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=QP9VIId, length=9, name=true) 2021-03-12 14:28:36.912 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q8VIId, length=8, name=true) 2021-03-12 14:28:39.869 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:44.349 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q8VIId, length=8, name=true) 2021-03-12 14:28:45.026 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:45.575 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:46.256 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q5VIId, length=8, name=true) 2021-03-12 14:28:46.863 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q5VIId, length=8, name=true) 2021-03-12 14:28:47.533 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q8VIId, length=8, name=true)
I'm not sure why he would use this mapping
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797708738, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEYUK2FIMOBWXZR4KRJHIWDTDJNLJANCNFSM4ZCFLYWA.
its the "QP8VIId" that confuses me. That is so exacting for a mystery star
want me to allocate a default specific spectral class for it?
no, it was just a thought. Use M.
He's just weird, as we had already established.
From: LJ Ramones @.> Sent: Friday, March 12, 2021 1:40 PM To: ljramones/trips @.> Cc: Rick Boatright @.>; Author @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
want me to allocate a default specific spectral class for it?
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797710444, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEYUK2C7LM2O6EIJGSYYYWDTDJNYPANCNFSM4ZCFLYWA.
sure
The answer should be on his Sol Station website. I'll go look. Feed me a list of the mystery spectra.
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: Rick Boatright @.> Sent: Friday, March 12, 2021 2:38:18 PM To: ljramones/trips @.> Cc: Subscribed @.***> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Is "Q" his signal for a fake star I wonder?
From: LJ Ramones @.> Sent: Friday, March 12, 2021 1:36 PM To: ljramones/trips @.> Cc: Rick Boatright @.>; Author @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
here are the bad spectral values for CHV : Could not figure out stellar classification: spectra =PseudoString(value=Q9VIId, length=8, name=true) 2021-03-12 14:27:59.699 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q7VIId, length=8, name=true) 2021-03-12 14:28:01.424 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q9VIId, length=8, name=true) 2021-03-12 14:28:05.398 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=QP8VIId, length=9, name=true) 2021-03-12 14:28:14.185 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:27.977 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q9VIId, length=8, name=true) 2021-03-12 14:28:29.315 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=QP9VIId, length=9, name=true) 2021-03-12 14:28:36.912 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q8VIId, length=8, name=true) 2021-03-12 14:28:39.869 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:44.349 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q8VIId, length=8, name=true) 2021-03-12 14:28:45.026 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:45.575 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q6VIId, length=8, name=true) 2021-03-12 14:28:46.256 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q5VIId, length=8, name=true) 2021-03-12 14:28:46.863 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q5VIId, length=8, name=true) 2021-03-12 14:28:47.533 ERROR 8328 --- [ Thread-7] c.t.trips.file.chview.ChviewReader : Could not figure out stellar classification: spectra =PseudoString(value=Q8VIId, length=8, name=true)
I'm not sure why he would use this mapping
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797708738, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEYUK2FIMOBWXZR4KRJHIWDTDJNLJANCNFSM4ZCFLYWA.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797709566, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNQZ75UMPDTPBZAT623TDJUSVANCNFSM4ZCFLYWA.
Not yet. Lemme.research his site.
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.> Sent: Friday, March 12, 2021 2:40:07 PM To: ljramones/trips @.> Cc: Subscribed @.***> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
want me to allocate a default specific spectral class for it?
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797710444, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNVYBC4KCZYHCGJ5V2DTDJNYPANCNFSM4ZCFLYWA.
@cegannon1 here are the values TRIPS barfed on
Q7VIId Q9VIId QP8VIId Q9VIId Q6VIId Q5VIId Q8VIId
No wait. I'm researching. And I have a guess...
that Q = suspected quasar
P= suspected.pulsar
I'll let you know asap
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: Rick Boatright @.> Sent: Friday, March 12, 2021 2:51:04 PM To: ljramones/trips @.> Cc: Subscribed @.***> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
no, it was just a thought. Use M.
He's just weird, as we had already established.
From: LJ Ramones @.> Sent: Friday, March 12, 2021 1:40 PM To: ljramones/trips @.> Cc: Rick Boatright @.>; Author @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
want me to allocate a default specific spectral class for it?
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797710444, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEYUK2C7LM2O6EIJGSYYYWDTDJNYPANCNFSM4ZCFLYWA.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797716306, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNWSP7BR3HHTB3FMPF3TDJWCRANCNFSM4ZCFLYWA.
Hmm...can you give me complete examples and the Chv dataset in which they are found?
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.> Sent: Friday, March 12, 2021 3:24:26 PM To: ljramones/trips @.> Cc: cegannon1 @.>; Mention @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
@cegannon1https://github.com/cegannon1 here are the values TRIPS barfed on
Q7VIId Q9VIId QP8VIId Q9VIId Q6VIId Q5VIId Q8VIId
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797734155, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNQM5O34EYLIU4CNZFDTDJS6VANCNFSM4ZCFLYWA.
do you want the full record that includes that spectra type? I was doing testing on the 100 ly dataset
please
From: LJ Ramones @.*** Sent: Friday, March 12, 2021 3:38 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
do you want the full record that includes that spectra type? I was doing testing on the 100 ly dataset
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797741586, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNQZ75UMPDTPBZAT623TDJUSVANCNFSM4ZCFLYWA.
That is probably going to work. Here’s where the weird spectrum factors come from. Read the first para here. It suggests these may be very old catalog objects. Are they the only rating for the star? Does the star have multiple names/catalog codes?
https://astronomy.swin.edu.au/cosmos/h/harvard+spectral+classification
From: Rick Boatright @.*** Sent: Friday, March 12, 2021 2:51 PM To: ljramones/trips Cc: Subscribed Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
no, it was just a thought. Use M.
He's just weird, as we had already established.
From: LJ Ramones @.> Sent: Friday, March 12, 2021 1:40 PM To: ljramones/trips @.> Cc: Rick Boatright @.>; Author @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
want me to allocate a default specific spectral class for it?
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797710444, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEYUK2C7LM2O6EIJGSYYYWDTDJNYPANCNFSM4ZCFLYWA.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797716306, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNWSP7BR3HHTB3FMPF3TDJWCRANCNFSM4ZCFLYWA.
This gives a little more insight into old records (The Ballmer line classification system that included Q) as well as modern variants, most of which are modifications to D classification (read down for that). It’s a Wiki and I wouldn’t trust it entirely. But it’s a good springboard.
https://space.fandom.com/wiki/Stellar_classification
Rick, this is the moment when my expertise (all things CHV) must give way to one of your areas of specialty (spectra and their classifications). Hopefully, this is the lead that will allow us to tease out what is going on.
Best, chuck
From: Rick Boatright @.*** Sent: Friday, March 12, 2021 2:51 PM To: ljramones/trips Cc: Subscribed Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
no, it was just a thought. Use M.
He's just weird, as we had already established.
From: LJ Ramones @.> Sent: Friday, March 12, 2021 1:40 PM To: ljramones/trips @.> Cc: Rick Boatright @.>; Author @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
want me to allocate a default specific spectral class for it?
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797710444, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEYUK2C7LM2O6EIJGSYYYWDTDJNYPANCNFSM4ZCFLYWA.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797716306, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNWSP7BR3HHTB3FMPF3TDJWCRANCNFSM4ZCFLYWA.
a month ago, I write a full spectral class parser that takes into account the extended classes, the luminosity, the multiplier i.e. G3.5IV, the peculiarities, hydrogen lines, etc. So if the spectral class is properly defined, my parser will handle it correctly.
Here is the regex parser that deals with it.
private final String prefixPatternStr = "^(sd|sg|d|c|g)";
Pattern prefixPattern = Pattern.compile(prefixPatternStr);
private final String curlyBracketsStr= "\\{(.*?)\\}";
Pattern curlyBracketsPattern = Pattern.compile(curlyBracketsStr);
private final String regBracketsStr= "\\((.*?)\\)";
Pattern regBracketsPattern = Pattern.compile(regBracketsStr);
//private String classPatternStr = "^(O|B|A|F|G|K|M|L|T|Y|P|Q|WN|WC|WR|DA|DQ|DB|DZ|DO|DC|DX|C\\-R|C\\-N|C\\-J|C\\-H|C\\-Hd)d(\\.d)";
private final String classPatternStr = "^(O|B|A|F|G|K|M|L|T|Y|P|Q|WN|WC|WR|DA|DQ|DB|DZ|DO|DC|DX|C\\-R|C\\-N|C\\-J|C\\-H||C\\-Hd)(\\d(\\.\\d)?)?";
Pattern classPattern = Pattern.compile(classPatternStr);
private final String yerkesPatternStr = "^(0|Ia\\+|Ia|Iab|Ib|II|III|IV|V|VII|VIII|IX|X)";
Pattern yerkesPattern = Pattern.compile(yerkesPatternStr);
private final String pecularitiesPatternStr = "^(:|...|!|comp|e|\\[e\\]|er|eq|f|f\\*|f\\+|\\(f\\(|\\(f\\+\\)|\\(\\(f\\)\\)|\\(\\(f\\*\\)\\)|h|ha|He wk|k|m|n|nn|neb|p|pq|q|s|ss|sh|var|wl)";
Pattern pecularitiesPattern = Pattern.compile(pecularitiesPatternStr);
This resource makes me wonder if Q is actually now ~ to one of the sub M types.
Search for “Balmer” to find the relevant sections for starting:
https://www.e-education.psu.edu/astro801/book/export/html/1755
From: Rick Boatright @.*** Sent: Friday, March 12, 2021 2:51 PM To: ljramones/trips Cc: Subscribed Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
no, it was just a thought. Use M.
He's just weird, as we had already established.
From: LJ Ramones @.> Sent: Friday, March 12, 2021 1:40 PM To: ljramones/trips @.> Cc: Rick Boatright @.>; Author @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
want me to allocate a default specific spectral class for it?
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797710444, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEYUK2C7LM2O6EIJGSYYYWDTDJNYPANCNFSM4ZCFLYWA.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797716306, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNWSP7BR3HHTB3FMPF3TDJWCRANCNFSM4ZCFLYWA.
Hi Larry:
What I mean is: specific stars with the Q spectral qualifier. I’ll then go into the CHV I can get running on my machine. The notes for the star are often tied to Sol Station entries. I should be able to data mine.
Just give me specific stars, by name, and tell me if they are in the 100 LY set or 150 or etc.
Thanks,
Chuck
From: LJ Ramones @.*** Sent: Friday, March 12, 2021 3:51 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
a month ago, I write a full spectral class parser that takes into account the extended classes, the luminosity, the multiplier i.e. G3.5IV, the peculiarities, hydrogen lines, etc. So if the spectral class is properly defined, my parser will handle it correctly.
Here is the regex parser that deals with it.
private final String prefixPatternStr = "^(sd|sg|d|c|g)";
Pattern prefixPattern = Pattern.compile(prefixPatternStr);
private final String curlyBracketsStr= "\{(.*?)\}";
Pattern curlyBracketsPattern = Pattern.compile(curlyBracketsStr);
private final String regBracketsStr= "\((.*?)\)";
Pattern regBracketsPattern = Pattern.compile(regBracketsStr);
//private String classPatternStr = "^(O|B|A|F|G|K|M|L|T|Y|P|Q|WN|WC|WR|DA|DQ|DB|DZ|DO|DC|DX|C\-R|C\-N|C\-J|C\-H|C\-Hd)d(\.d)";
private final String classPatternStr = "^(O|B|A|F|G|K|M|L|T|Y|P|Q|WN|WC|WR|DA|DQ|DB|DZ|DO|DC|DX|C\-R|C\-N|C\-J|C\-H||C\-Hd)(\d(\.\d)?)?";
Pattern classPattern = Pattern.compile(classPatternStr);
private final String yerkesPatternStr = "^(0|Ia\+|Ia|Iab|Ib|II|III|IV|V|VII|VIII|IX|X)";
Pattern yerkesPattern = Pattern.compile(yerkesPatternStr);
private final String pecularitiesPatternStr = "^(:|...|!|comp|e|\[e\]|er|eq|f|f\|f\+|\(f\(|\(f\+\)|\(\(f\)\)|\(\(f\\)\)|h|ha|He wk|k|m|n|nn|neb|p|pq|q|s|ss|sh|var|wl)";
Pattern pecularitiesPattern = Pattern.compile(pecularitiesPatternStr);
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797747435, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNSSPZYMARTNU4Z6NGDTDJWDHANCNFSM4ZCFLYWA.
I sent a separate email with the full list of strange spectra and the records associated with the spectral class
all fixed. got all the CHV files to load. For those with weird spectral classes. I defaulted to M class until you guys make a decision.
Hi Larry:
Okay, once I saw the CHV data, I realized something.
We don’t need to keep this data. Reason follows.
Practical instruction:
Simply create “?” as a recognized spectral class.
And then all the faulty spectral classes you list below should simply be converted to ? .
Reasons:
Firstly, it turns out that the “Q” value is almost certainly a holdover from catalog listings that are almost 100 years out of date. That means that the observational methodologies are equally ancient. Q was part of a precursor spectral classification system that preceded the one we use today. It was based on hydrogen fusing lines in spectroscopic observations of the stars, and ran from A-R, I think. Q would usually indicate a faint object, which is probably why a great number of these faulty Spectrascopic classes are observed with closer rather than farther objects.
But since then, so much has changed (accuracy of parallax, accuracy of spectroscopy, accuracy of damn near EVERY observational metric) that we can’t be sure about ANY of these values which were generated using methods that are museum pieces.
Furthermore, this data is in CHV. Meaning that, if the star name syncs up with the/a star name or catalog code in the Gaia/Simbad/Hipparcos 2 data, none of it is going to be imported anyhow. All these classifications will get scraped off anyhow.
Best of all, using “ ? “ as a spectral class actually solves ANOTHER issue; how to code stellar objects in the Gaia data for which there are not spectral classes generated. Rick and I have worked out a pretty fair “approximation via statistical Kentucky windage” method of assigning everything a spectral class in the FICTIONAL / TERRAN REPUBLIC data. But in the “virgin Gaia” dataset (or any derived therefrom) lack of spectral class should be notated as such. And this answers that challenge.
Any questions? Just ask.
Best and thanks,
Chuck
From: LJ Ramones @.*** Sent: Friday, March 12, 2021 3:24 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
@cegannon1https://github.com/cegannon1 here are the values TRIPS barfed on
Q7VIId Q9VIId QP8VIId Q9VIId Q6VIId Q5VIId Q8VIId
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797734155, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNQM5O34EYLIU4CNZFDTDJS6VANCNFSM4ZCFLYWA.
Hi
Ok, however a few caveats.
Point 1) ? Is a reserved character for pattern processing. I would rather not use it. I would have to escape it everywhere and that would mean lots of changes in the lexical analyzers.
How about I change the “?” To an X and handle it that way. X for the great unknown.
Point 2) I need a default set of spectral properties for the X class. Should I treat it as a M class or something else. M class is our sort of default condition.
On Mar 13, 2021, at 4:45 PM, cegannon1 @.***> wrote:
Hi Larry:
Okay, once I saw the CHV data, I realized something.
We don’t need to keep this data. Reason follows.
Practical instruction:
Simply create “?” as a recognized spectral class.
And then all the faulty spectral classes you list below should simply be converted to ? .
Reasons:
Firstly, it turns out that the “Q” value is almost certainly a holdover from catalog listings that are almost 100 years out of date. That means that the observational methodologies are equally ancient. Q was part of a precursor spectral classification system that preceded the one we use today. It was based on hydrogen fusing lines in spectroscopic observations of the stars, and ran from A-R, I think. Q would usually indicate a faint object, which is probably why a great number of these faulty Spectrascopic classes are observed with closer rather than farther objects.
But since then, so much has changed (accuracy of parallax, accuracy of spectroscopy, accuracy of damn near EVERY observational metric) that we can’t be sure about ANY of these values which were generated using methods that are museum pieces.
Furthermore, this data is in CHV. Meaning that, if the star name syncs up with the/a star name or catalog code in the Gaia/Simbad/Hipparcos 2 data, none of it is going to be imported anyhow. All these classifications will get scraped off anyhow.
Best of all, using “ ? “ as a spectral class actually solves ANOTHER issue; how to code stellar objects in the Gaia data for which there are not spectral classes generated. Rick and I have worked out a pretty fair “approximation via statistical Kentucky windage” method of assigning everything a spectral class in the FICTIONAL / TERRAN REPUBLIC data. But in the “virgin Gaia” dataset (or any derived therefrom) lack of spectral class should be notated as such. And this answers that challenge.
Any questions? Just ask.
Best and thanks,
Chuck
From: LJ Ramones @.*** Sent: Friday, March 12, 2021 3:24 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
@cegannon1https://github.com/cegannon1 here are the values TRIPS barfed on
Q7VIId Q9VIId QP8VIId Q9VIId Q6VIId Q5VIId Q8VIId
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797734155, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNQM5O34EYLIU4CNZFDTDJS6VANCNFSM4ZCFLYWA.
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
Rick, this has gone my competence.
Again.
re.1) X is not good.
re 2) why must it have any properties, other than just being a distinctive handle?
thanks,
Chuck
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.***> Sent: Saturday, March 13, 2021, 7:56 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Hi
Ok, however a few caveats.
Point 1) ? Is a reserved character for pattern processing. I would rather not use it. I would have to escape it everywhere and that would mean lots of changes in the lexical analyzers.
How about I change the “?” To an X and handle it that way. X for the great unknown.
Point 2) I need a default set of spectral properties for the X class. Should I treat it as a M class or something else. M class is our sort of default condition.
On Mar 13, 2021, at 4:45 PM, cegannon1 @.***> wrote:
Hi Larry:
Okay, once I saw the CHV data, I realized something.
We don’t need to keep this data. Reason follows.
Practical instruction:
Simply create “?” as a recognized spectral class.
And then all the faulty spectral classes you list below should simply be converted to ? .
Reasons:
Firstly, it turns out that the “Q” value is almost certainly a holdover from catalog listings that are almost 100 years out of date. That means that the observational methodologies are equally ancient. Q was part of a precursor spectral classification system that preceded the one we use today. It was based on hydrogen fusing lines in spectroscopic observations of the stars, and ran from A-R, I think. Q would usually indicate a faint object, which is probably why a great number of these faulty Spectrascopic classes are observed with closer rather than farther objects.
But since then, so much has changed (accuracy of parallax, accuracy of spectroscopy, accuracy of damn near EVERY observational metric) that we can’t be sure about ANY of these values which were generated using methods that are museum pieces.
Furthermore, this data is in CHV. Meaning that, if the star name syncs up with the/a star name or catalog code in the Gaia/Simbad/Hipparcos 2 data, none of it is going to be imported anyhow. All these classifications will get scraped off anyhow.
Best of all, using “ ? “ as a spectral class actually solves ANOTHER issue; how to code stellar objects in the Gaia data for which there are not spectral classes generated. Rick and I have worked out a pretty fair “approximation via statistical Kentucky windage” method of assigning everything a spectral class in the FICTIONAL / TERRAN REPUBLIC data. But in the “virgin Gaia” dataset (or any derived therefrom) lack of spectral class should be notated as such. And this answers that challenge.
Any questions? Just ask.
Best and thanks,
Chuck
From: LJ Ramones @.*** Sent: Friday, March 12, 2021 3:24 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
@cegannon1https://github.com/cegannon1 here are the values TRIPS barfed on
Q7VIId Q9VIId QP8VIId Q9VIId Q6VIId Q5VIId Q8VIId
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797734155, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNQM5O34EYLIU4CNZFDTDJS6VANCNFSM4ZCFLYWA.
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-798810091, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNWWMU2CT3NQ2JYJ7ZTTDQCUDANCNFSM4ZCFLYWA.
Ok, I can make the changes to use ? but the risk associated with it pushes the release for v0.6.7 to at least Wednesday. It's a lot of changes and I have to do a lot more testing.
The reason is that everything is governed by how the lexical analyzer parse the spectral class. "?" is a reserved character and I have to write a lot of extra stuff to deal with it.
As to default spectral properties. I need a color to plot ? as. Computer programs are exact things. No hand waving allowed. So if I have to plot ?, I need to know what to color it as.
I could just not plot it and that would work.
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ On Saturday, March 13, 2021 8:09 PM, cegannon1 @.***> wrote:
Rick, this has gone my competence.
Again.
re.1) X is not good.
re 2) why must it have any properties, other than just being a distinctive handle?
thanks,
Chuck
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.***> Sent: Saturday, March 13, 2021, 7:56 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Hi
Ok, however a few caveats.
Point 1) ? Is a reserved character for pattern processing. I would rather not use it. I would have to escape it everywhere and that would mean lots of changes in the lexical analyzers.
How about I change the “?” To an X and handle it that way. X for the great unknown.
Point 2) I need a default set of spectral properties for the X class. Should I treat it as a M class or something else. M class is our sort of default condition.
On Mar 13, 2021, at 4:45 PM, cegannon1 @.***> wrote:
Hi Larry:
Okay, once I saw the CHV data, I realized something.
We don’t need to keep this data. Reason follows.
Practical instruction:
Simply create “?” as a recognized spectral class.
And then all the faulty spectral classes you list below should simply be converted to ? .
Reasons:
Firstly, it turns out that the “Q” value is almost certainly a holdover from catalog listings that are almost 100 years out of date. That means that the observational methodologies are equally ancient. Q was part of a precursor spectral classification system that preceded the one we use today. It was based on hydrogen fusing lines in spectroscopic observations of the stars, and ran from A-R, I think. Q would usually indicate a faint object, which is probably why a great number of these faulty Spectrascopic classes are observed with closer rather than farther objects.
But since then, so much has changed (accuracy of parallax, accuracy of spectroscopy, accuracy of damn near EVERY observational metric) that we can’t be sure about ANY of these values which were generated using methods that are museum pieces.
Furthermore, this data is in CHV. Meaning that, if the star name syncs up with the/a star name or catalog code in the Gaia/Simbad/Hipparcos 2 data, none of it is going to be imported anyhow. All these classifications will get scraped off anyhow.
Best of all, using “ ? “ as a spectral class actually solves ANOTHER issue; how to code stellar objects in the Gaia data for which there are not spectral classes generated. Rick and I have worked out a pretty fair “approximation via statistical Kentucky windage” method of assigning everything a spectral class in the FICTIONAL / TERRAN REPUBLIC data. But in the “virgin Gaia” dataset (or any derived therefrom) lack of spectral class should be notated as such. And this answers that challenge.
Any questions? Just ask.
Best and thanks,
Chuck
From: LJ Ramones @.*** Sent: Friday, March 12, 2021 3:24 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
@cegannon1https://github.com/cegannon1 here are the values TRIPS barfed on
Q7VIId Q9VIId QP8VIId Q9VIId Q6VIId Q5VIId Q8VIId
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797734155, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNQM5O34EYLIU4CNZFDTDJS6VANCNFSM4ZCFLYWA.
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-798810091, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNWWMU2CT3NQ2JYJ7ZTTDQCUDANCNFSM4ZCFLYWA.
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
just so you know what the list of reserved characters, here they are
. ? ( ) {} [] ^ * \
Whenever you use one of these as a search character, you have to escape it. Its a pain in the ass, but doable. I already escape () and {} because some of the files use these in the the spectral class definitions
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ On Saturday, March 13, 2021 8:18 PM, ljramones @.***> wrote:
Ok, I can make the changes to use ? but the risk associated with it pushes the release for v0.6.7 to at least Wednesday. It's a lot of changes and I have to do a lot more testing.
The reason is that everything is governed by how the lexical analyzer parse the spectral class. "?" is a reserved character and I have to write a lot of extra stuff to deal with it.
As to default spectral properties. I need a color to plot ? as. Computer programs are exact things. No hand waving allowed. So if I have to plot ?, I need to know what to color it as.
I could just not plot it and that would work.
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ On Saturday, March 13, 2021 8:09 PM, cegannon1 @.***> wrote:
Rick, this has gone my competence.
Again.
re.1) X is not good.
re 2) why must it have any properties, other than just being a distinctive handle?
thanks,
Chuck
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.***> Sent: Saturday, March 13, 2021, 7:56 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Hi
Ok, however a few caveats.
Point 1) ? Is a reserved character for pattern processing. I would rather not use it. I would have to escape it everywhere and that would mean lots of changes in the lexical analyzers.
How about I change the “?” To an X and handle it that way. X for the great unknown.
Point 2) I need a default set of spectral properties for the X class. Should I treat it as a M class or something else. M class is our sort of default condition.
On Mar 13, 2021, at 4:45 PM, cegannon1 @.***> wrote:
Hi Larry:
Okay, once I saw the CHV data, I realized something.
We don’t need to keep this data. Reason follows.
Practical instruction:
Simply create “?” as a recognized spectral class.
And then all the faulty spectral classes you list below should simply be converted to ? .
Reasons:
Firstly, it turns out that the “Q” value is almost certainly a holdover from catalog listings that are almost 100 years out of date. That means that the observational methodologies are equally ancient. Q was part of a precursor spectral classification system that preceded the one we use today. It was based on hydrogen fusing lines in spectroscopic observations of the stars, and ran from A-R, I think. Q would usually indicate a faint object, which is probably why a great number of these faulty Spectrascopic classes are observed with closer rather than farther objects.
But since then, so much has changed (accuracy of parallax, accuracy of spectroscopy, accuracy of damn near EVERY observational metric) that we can’t be sure about ANY of these values which were generated using methods that are museum pieces.
Furthermore, this data is in CHV. Meaning that, if the star name syncs up with the/a star name or catalog code in the Gaia/Simbad/Hipparcos 2 data, none of it is going to be imported anyhow. All these classifications will get scraped off anyhow.
Best of all, using “ ? “ as a spectral class actually solves ANOTHER issue; how to code stellar objects in the Gaia data for which there are not spectral classes generated. Rick and I have worked out a pretty fair “approximation via statistical Kentucky windage” method of assigning everything a spectral class in the FICTIONAL / TERRAN REPUBLIC data. But in the “virgin Gaia” dataset (or any derived therefrom) lack of spectral class should be notated as such. And this answers that challenge.
Any questions? Just ask.
Best and thanks,
Chuck
From: LJ Ramones @.*** Sent: Friday, March 12, 2021 3:24 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
@cegannon1https://github.com/cegannon1 here are the values TRIPS barfed on
Q7VIId Q9VIId QP8VIId Q9VIId Q6VIId Q5VIId Q8VIId
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797734155, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNQM5O34EYLIU4CNZFDTDJS6VANCNFSM4ZCFLYWA.
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-798810091, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNWWMU2CT3NQ2JYJ7ZTTDQCUDANCNFSM4ZCFLYWA.
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.> Sent: Saturday, March 13, 2021 8:18:40 PM To: ljramones/trips @.> Cc: cegannon1 @.>; Mention @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Ok, I can make the changes to use ? but the risk associated with it pushes the release for v0.6.7 to at least Wednesday. It's a lot of changes and I have to do a lot more testing.
The reason is that everything is governed by how the lexical analyzer parse the spectral class. "?" is a reserved character and I have to write a lot of extra stuff to deal with it.
As to default spectral properties. I need a color to plot ? as. Computer programs are exact things. No hand waving allowed. So if I have to plot ?, I need to know what to color it as.
I could just not plot it and that would work.
Sent with ProtonMail Secure Email.
\\\\\\\ Original Message \\\\\\\ On Saturday, March 13, 2021 8:09 PM, cegannon1 @.***> wrote:
Rick, this has gone my competence.
Again.
re.1) X is not good.
re 2) why must it have any properties, other than just being a distinctive handle?
thanks,
Chuck
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.***> Sent: Saturday, March 13, 2021, 7:56 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Hi
Ok, however a few caveats.
Point 1) ? Is a reserved character for pattern processing. I would rather not use it. I would have to escape it everywhere and that would mean lots of changes in the lexical analyzers.
How about I change the “?” To an X and handle it that way. X for the great unknown.
Point 2) I need a default set of spectral properties for the X class. Should I treat it as a M class or something else. M class is our sort of default condition.
On Mar 13, 2021, at 4:45 PM, cegannon1 @.***> wrote:
Hi Larry:
Okay, once I saw the CHV data, I realized something.
We don’t need to keep this data. Reason follows.
Practical instruction:
Simply create “?” as a recognized spectral class.
And then all the faulty spectral classes you list below should simply be converted to ? .
Reasons:
Firstly, it turns out that the “Q” value is almost certainly a holdover from catalog listings that are almost 100 years out of date. That means that the observational methodologies are equally ancient. Q was part of a precursor spectral classification system that preceded the one we use today. It was based on hydrogen fusing lines in spectroscopic observations of the stars, and ran from A-R, I think. Q would usually indicate a faint object, which is probably why a great number of these faulty Spectrascopic classes are observed with closer rather than farther objects.
But since then, so much has changed (accuracy of parallax, accuracy of spectroscopy, accuracy of damn near EVERY observational metric) that we can’t be sure about ANY of these values which were generated using methods that are museum pieces.
Furthermore, this data is in CHV. Meaning that, if the star name syncs up with the/a star name or catalog code in the Gaia/Simbad/Hipparcos 2 data, none of it is going to be imported anyhow. All these classifications will get scraped off anyhow.
Best of all, using “ ? “ as a spectral class actually solves ANOTHER issue; how to code stellar objects in the Gaia data for which there are not spectral classes generated. Rick and I have worked out a pretty fair “approximation via statistical Kentucky windage” method of assigning everything a spectral class in the FICTIONAL / TERRAN REPUBLIC data. But in the “virgin Gaia” dataset (or any derived therefrom) lack of spectral class should be notated as such. And this answers that challenge.
Any questions? Just ask.
Best and thanks,
Chuck
From: LJ Ramones @.*** Sent: Friday, March 12, 2021 3:24 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
@cegannon1https://github.com/cegannon1 here are the values TRIPS barfed on
Q7VIId Q9VIId QP8VIId Q9VIId Q6VIId Q5VIId Q8VIId
― You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797734155, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNQM5O34EYLIU4CNZFDTDJS6VANCNFSM4ZCFLYWA.
― You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
― You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-798810091, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNWWMU2CT3NQ2JYJ7ZTTDQCUDANCNFSM4ZCFLYWA.
― You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
― You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-798812696, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNXFQIRTVDOKJPOGEULTDQFHBANCNFSM4ZCFLYWA.
I will make the changes.
Hopefully I can finish tomorrow.
On Mar 13, 2021, at 9:06 PM, cegannon1 @.***> wrote:
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.> Sent: Saturday, March 13, 2021 8:18:40 PM To: ljramones/trips @.> Cc: cegannon1 @.>; Mention @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Ok, I can make the changes to use ? but the risk associated with it pushes the release for v0.6.7 to at least Wednesday. It's a lot of changes and I have to do a lot more testing.
The reason is that everything is governed by how the lexical analyzer parse the spectral class. "?" is a reserved character and I have to write a lot of extra stuff to deal with it.
As to default spectral properties. I need a color to plot ? as. Computer programs are exact things. No hand waving allowed. So if I have to plot ?, I need to know what to color it as.
I could just not plot it and that would work.
Sent with ProtonMail Secure Email.
\\\\ Original Message \\\\ On Saturday, March 13, 2021 8:09 PM, cegannon1 @.***> wrote:
Rick, this has gone my competence.
Again.
re.1) X is not good.
re 2) why must it have any properties, other than just being a distinctive handle?
thanks,
Chuck
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.***> Sent: Saturday, March 13, 2021, 7:56 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Hi
Ok, however a few caveats.
Point 1) ? Is a reserved character for pattern processing. I would rather not use it. I would have to escape it everywhere and that would mean lots of changes in the lexical analyzers.
How about I change the “?” To an X and handle it that way. X for the great unknown.
Point 2) I need a default set of spectral properties for the X class. Should I treat it as a M class or something else. M class is our sort of default condition.
On Mar 13, 2021, at 4:45 PM, cegannon1 @.***> wrote:
Hi Larry:
Okay, once I saw the CHV data, I realized something.
We don’t need to keep this data. Reason follows.
Practical instruction:
Simply create “?” as a recognized spectral class.
And then all the faulty spectral classes you list below should simply be converted to ? .
Reasons:
Firstly, it turns out that the “Q” value is almost certainly a holdover from catalog listings that are almost 100 years out of date. That means that the observational methodologies are equally ancient. Q was part of a precursor spectral classification system that preceded the one we use today. It was based on hydrogen fusing lines in spectroscopic observations of the stars, and ran from A-R, I think. Q would usually indicate a faint object, which is probably why a great number of these faulty Spectrascopic classes are observed with closer rather than farther objects.
But since then, so much has changed (accuracy of parallax, accuracy of spectroscopy, accuracy of damn near EVERY observational metric) that we can’t be sure about ANY of these values which were generated using methods that are museum pieces.
Furthermore, this data is in CHV. Meaning that, if the star name syncs up with the/a star name or catalog code in the Gaia/Simbad/Hipparcos 2 data, none of it is going to be imported anyhow. All these classifications will get scraped off anyhow.
Best of all, using “ ? “ as a spectral class actually solves ANOTHER issue; how to code stellar objects in the Gaia data for which there are not spectral classes generated. Rick and I have worked out a pretty fair “approximation via statistical Kentucky windage” method of assigning everything a spectral class in the FICTIONAL / TERRAN REPUBLIC data. But in the “virgin Gaia” dataset (or any derived therefrom) lack of spectral class should be notated as such. And this answers that challenge.
Any questions? Just ask.
Best and thanks,
Chuck
From: LJ Ramones @.*** Sent: Friday, March 12, 2021 3:24 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
@cegannon1https://github.com/cegannon1 here are the values TRIPS barfed on
Q7VIId Q9VIId QP8VIId Q9VIId Q6VIId Q5VIId Q8VIId
― You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797734155, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNQM5O34EYLIU4CNZFDTDJS6VANCNFSM4ZCFLYWA.
― You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
― You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-798810091, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNWWMU2CT3NQ2JYJ7ZTTDQCUDANCNFSM4ZCFLYWA.
― You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
― You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-798812696, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNXFQIRTVDOKJPOGEULTDQFHBANCNFSM4ZCFLYWA.
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
But haven't given you character or color yet.
I am in a bad home situation and I just can't do more tonight.
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.> Sent: Saturday, March 13, 2021 9:13:04 PM To: ljramones/trips @.> Cc: cegannon1 @.>; Mention @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
I will make the changes.
Hopefully I can finish tomorrow.
On Mar 13, 2021, at 9:06 PM, cegannon1 @.***> wrote:
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.> Sent: Saturday, March 13, 2021 8:18:40 PM To: ljramones/trips @.> Cc: cegannon1 @.>; Mention @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Ok, I can make the changes to use ? but the risk associated with it pushes the release for v0.6.7 to at least Wednesday. It's a lot of changes and I have to do a lot more testing.
The reason is that everything is governed by how the lexical analyzer parse the spectral class. "?" is a reserved character and I have to write a lot of extra stuff to deal with it.
As to default spectral properties. I need a color to plot ? as. Computer programs are exact things. No hand waving allowed. So if I have to plot ?, I need to know what to color it as.
I could just not plot it and that would work.
Sent with ProtonMail Secure Email.
\\\\ Original Message \\\\ On Saturday, March 13, 2021 8:09 PM, cegannon1 @.***> wrote:
Rick, this has gone my competence.
Again.
re.1) X is not good.
re 2) why must it have any properties, other than just being a distinctive handle?
thanks,
Chuck
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.***> Sent: Saturday, March 13, 2021, 7:56 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Hi
Ok, however a few caveats.
Point 1) ? Is a reserved character for pattern processing. I would rather not use it. I would have to escape it everywhere and that would mean lots of changes in the lexical analyzers.
How about I change the “?” To an X and handle it that way. X for the great unknown.
Point 2) I need a default set of spectral properties for the X class. Should I treat it as a M class or something else. M class is our sort of default condition.
On Mar 13, 2021, at 4:45 PM, cegannon1 @.***> wrote:
Hi Larry:
Okay, once I saw the CHV data, I realized something.
We don’t need to keep this data. Reason follows.
Practical instruction:
Simply create “?” as a recognized spectral class.
And then all the faulty spectral classes you list below should simply be converted to ? .
Reasons:
Firstly, it turns out that the “Q” value is almost certainly a holdover from catalog listings that are almost 100 years out of date. That means that the observational methodologies are equally ancient. Q was part of a precursor spectral classification system that preceded the one we use today. It was based on hydrogen fusing lines in spectroscopic observations of the stars, and ran from A-R, I think. Q would usually indicate a faint object, which is probably why a great number of these faulty Spectrascopic classes are observed with closer rather than farther objects.
But since then, so much has changed (accuracy of parallax, accuracy of spectroscopy, accuracy of damn near EVERY observational metric) that we can’t be sure about ANY of these values which were generated using methods that are museum pieces.
Furthermore, this data is in CHV. Meaning that, if the star name syncs up with the/a star name or catalog code in the Gaia/Simbad/Hipparcos 2 data, none of it is going to be imported anyhow. All these classifications will get scraped off anyhow.
Best of all, using “ ? “ as a spectral class actually solves ANOTHER issue; how to code stellar objects in the Gaia data for which there are not spectral classes generated. Rick and I have worked out a pretty fair “approximation via statistical Kentucky windage” method of assigning everything a spectral class in the FICTIONAL / TERRAN REPUBLIC data. But in the “virgin Gaia” dataset (or any derived therefrom) lack of spectral class should be notated as such. And this answers that challenge.
Any questions? Just ask.
Best and thanks,
Chuck
From: LJ Ramones @.*** Sent: Friday, March 12, 2021 3:24 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
@cegannon1https://github.com/cegannon1 here are the values TRIPS barfed on
Q7VIId Q9VIId QP8VIId Q9VIId Q6VIId Q5VIId Q8VIId
― You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-797734155, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNQM5O34EYLIU4CNZFDTDJS6VANCNFSM4ZCFLYWA.
― You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
― You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-798810091, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNWWMU2CT3NQ2JYJ7ZTTDQCUDANCNFSM4ZCFLYWA.
― You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
― You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-798812696, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNXFQIRTVDOKJPOGEULTDQFHBANCNFSM4ZCFLYWA.
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/247#issuecomment-798820508, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNSY5SCSRX53ANAPW2LTDQLTBANCNFSM4ZCFLYWA.
Plot "unk" same color as M since M is the most common and we're just guessing.
Plot Q, P, and all the D's as the same color as O since they're all white dwarfs and thus HOT tho tiny.
If you could plot Q, P and D as the same color as O but the same size as K, that would be perfect unless you wanted to add them to the list of things in setup to do colors and radius for. In any event, they're not as big as O's.
If you do want to add them that way, you can put all 3 on the same line since they're all 3 white dwarfs.
On Sat, Mar 13, 2021 at 8:05 PM Charles Gannon @.***> wrote:
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Android https://aka.ms/AAb9ysg
From: LJ Ramones @.> Sent: Saturday, March 13, 2021 8:18:40 PM To: ljramones/trips @.> Cc: cegannon1 @.>; Mention < @.> Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Ok, I can make the changes to use ? but the risk associated with it pushes the release for v0.6.7 to at least Wednesday. It's a lot of changes and I have to do a lot more testing.
The reason is that everything is governed by how the lexical analyzer parse the spectral class. "?" is a reserved character and I have to write a lot of extra stuff to deal with it.
As to default spectral properties. I need a color to plot ? as. Computer programs are exact things. No hand waving allowed. So if I have to plot ?, I need to know what to color it as.
I could just not plot it and that would work.
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ On Saturday, March 13, 2021 8:09 PM, cegannon1 @.***> wrote:
Rick, this has gone my competence.
Again.
re.1) X is not good.
re 2) why must it have any properties, other than just being a distinctive handle?
thanks,
Chuck
Sent from my Verizon, Samsung Galaxy smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
From: LJ Ramones @.***> Sent: Saturday, March 13, 2021, 7:56 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
Hi
Ok, however a few caveats.
Point 1) ? Is a reserved character for pattern processing. I would rather not use it. I would have to escape it everywhere and that would mean lots of changes in the lexical analyzers.
How about I change the “?” To an X and handle it that way. X for the great unknown.
Point 2) I need a default set of spectral properties for the X class. Should I treat it as a M class or something else. M class is our sort of default condition.
On Mar 13, 2021, at 4:45 PM, cegannon1 @.***> wrote:
Hi Larry:
Okay, once I saw the CHV data, I realized something.
We don’t need to keep this data. Reason follows.
Practical instruction:
Simply create “?” as a recognized spectral class.
And then all the faulty spectral classes you list below should simply be converted to ? .
Reasons:
Firstly, it turns out that the “Q” value is almost certainly a holdover from catalog listings that are almost 100 years out of date. That means that the observational methodologies are equally ancient. Q was part of a precursor spectral classification system that preceded the one we use today. It was based on hydrogen fusing lines in spectroscopic observations of the stars, and ran from A-R, I think. Q would usually indicate a faint object, which is probably why a great number of these faulty Spectrascopic classes are observed with closer rather than farther objects.
But since then, so much has changed (accuracy of parallax, accuracy of spectroscopy, accuracy of damn near EVERY observational metric) that we can’t be sure about ANY of these values which were generated using methods that are museum pieces.
Furthermore, this data is in CHV. Meaning that, if the star name syncs up with the/a star name or catalog code in the Gaia/Simbad/Hipparcos 2 data, none of it is going to be imported anyhow. All these classifications will get scraped off anyhow.
Best of all, using “ ? “ as a spectral class actually solves ANOTHER issue; how to code stellar objects in the Gaia data for which there are not spectral classes generated. Rick and I have worked out a pretty fair “approximation via statistical Kentucky windage” method of assigning everything a spectral class in the FICTIONAL / TERRAN REPUBLIC data. But in the “virgin Gaia” dataset (or any derived therefrom) lack of spectral class should be notated as such. And this answers that challenge.
Any questions? Just ask.
Best and thanks,
Chuck
From: LJ Ramones @.*** Sent: Friday, March 12, 2021 3:24 PM To: ljramones/trips Cc: cegannon1; Mention Subject: Re: [ljramones/trips] ChView Dataset import errors 0,66 on SOME datasets - see details. (#247)
@cegannon1https://github.com/cegannon1 here are the values TRIPS barfed on
Q7VIId Q9VIId QP8VIId Q9VIId Q6VIId Q5VIId Q8VIId
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub< https://github.com/ljramones/trips/issues/247#issuecomment-797734155>, or unsubscribe< https://github.com/notifications/unsubscribe-auth/AQPXLNQM5O34EYLIU4CNZFDTDJS6VANCNFSM4ZCFLYWA .
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub< https://github.com/ljramones/trips/issues/247#issuecomment-798810091>, or unsubscribe< https://github.com/notifications/unsubscribe-auth/AQPXLNWWMU2CT3NQ2JYJ7ZTTDQCUDANCNFSM4ZCFLYWA .
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ljramones/trips/issues/247#issuecomment-798812696, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQPXLNXFQIRTVDOKJPOGEULTDQFHBANCNFSM4ZCFLYWA .
Import failure on three distributed CHV files
NOTE TERRAGRP.CHV loaded successfully, these others did not.
100LY-H.CHV failed to import with the following information displayed on the log screen:
SOPHONTS.CHV failed to import with the following error in the log screen:
The following datasets also failed at the same point with the same error: 25LY-H.CHV, farexile.chv, polities.chv
All of them display this when they freeze.
The cancel import button is disabled. The close button works, and the program proceeds without having imported anything.