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

TNS: 'host_redshift' not found in columns #14

Open
0xCoto opened this issue Jan 4, 2023 · 4 comments
Open

TNS: 'host_redshift' not found in columns #14

0xCoto opened this issue Jan 4, 2023 · 4 comments

Comments

@0xCoto
Copy link

0xCoto commented Jan 4, 2023

@loostrum - Any idea why that is?

Traceback (most recent call last):
  File "/home/runner/work/FRBSTATS/FRBSTATS/FRBSTATS/tns_compare.py", line 56, in <module>
    tns = TNS(path='/home/runner/', tns_name='my_user_name', tns_id='my_user_id')
  File "/usr/local/lib/python3.10/dist-packages/frbcat/tns.py", line 54, in __init__
    self.get_data()
  File "/usr/local/lib/python3.10/dist-packages/frbcat/tns.py", line 127, in get_data
    self.clean_df(df)
  File "/usr/local/lib/python3.10/dist-packages/frbcat/tns.py", line 308, in clean_df
    df = df.astype({'tns_id': int,
  File "/usr/local/lib/python3.10/dist-packages/pandas/core/generic.py", line 6212, in astype
    raise KeyError(
KeyError: "Only a column name can be used for the key in a dtype mappings argument. 'host_redshift' not found in columns."

Not sure if something suddenly changed in the TNS (has been working fine so far).

Thanks!

@0xCoto
Copy link
Author

0xCoto commented Jan 8, 2023

@davidgardenier @fjankowsk @andrewzwaniga looping you guys in, in case you happen to know if something might have changed with the TNS recently.

@fjankowsk
Copy link
Contributor

Hi,

I ran into this issue too recently. Have you or anyone else found a fix for it in the meantime? It would be nice to get this fixed.

Cheers.
Fabian

@0xCoto
Copy link
Author

0xCoto commented Feb 2, 2023

@fjankowsk Try this fork: https://github.com/HeRTA/frbcat

Haven't submitted a PR because it may break other things that require these parameters, but it works for me considering I don't need access to such data.

@davidgardenier
Copy link
Collaborator

@loostrum You're probs best placed to look into this, but see how it fits in your schedule.

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

No branches or pull requests

3 participants