Improvements needed in ndfits

[surajit4444mondal]

I find that ndfits.read is trying to know apriori what are the column names and only returning those particular columns, when reading a fits table. That implies that whenever we add new columns to the fits table, we need to update the code. I think it makes more sense to directly get the column names from the field table, and then all the columns and add them to the meta which is being returned by the function. Comments? @sageyu123

[sageyu123]

Good idea, I can add this feature.

[sageyu123]

@surajit4444mondal, please give it a try.

[surajit4444mondal]

I am unable to pull the changes you made into my area.. I am not sure how to test it. I was trying to follow the method described here (https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally) But I always get "fatal: couldn't find remote ref pull/312/head"

[sageyu123]

@surajit4444mondal Maybe you can add my fork of suncasa-src (https://github.com/sageyu123/suncasa-src.git) as an additional remote named upstream, and then fetch the updates directly from there?

To do this, you can run the following commands from within your local suncasa-src repository:

# Add my fork as an upstream remote (if you haven't already)
git remote add upstream https://github.com/sageyu123/suncasa-src.git

# Fetch the latest changes from my fork
git fetch upstream

# Check out the branch you want to test (e.g., the branch I pushed the changes to)
git checkout -b test-my-changes upstream/master