Subset.as_dataframe() hides targets instead of raising an error

[JacksonBurns]

Changelogs

• Subset.as_dataframe() would raise an error when called on a test set (which hides its targets), this PR allows method to work, just doesn't return targest

---

Checklist:

• Was this PR discussed in an issue? It is recommended to first discuss a new feature into a GitHub issue before opening a PR.
• Add tests to cover the fixed bug(s) or the newly introduced feature(s) (if appropriate).
• Update the API documentation if a new function is added, or an existing one is deleted.
• Write concise and explanatory changelogs above.
• If possible, assign one of the following labels to the PR: feature, fix, chore, documentation or test (or ask a maintainer to do it for you).

---

Resolves #257

[cwognum]

That was fast! 😅

Thanks @JacksonBurns! Would you mind adding a test case to make sure the as_dataframe() method returns the expected results for the train and test set?

[JacksonBurns]

@cwognum sure thing, done

[cwognum]

@JacksonBurns Just wanted to let you know that we're looking into why the CICD is failing here... I believe you're our first external contributor, so things may be breaking in new ways! 😅

[JacksonBurns]

Happy to be 'breaking' new ground, as it were

[jstlaurent]

Hi @JacksonBurns . Thanks for contributing. It seems our CI pipeline is having some issues unrelated to your PR. I'm looking into it.

[jstlaurent]

Hi @JacksonBurns. Could you rebase your fork, or merge the new changes in our main? That will fix the issue you're encountering with the CI. It was indeed an issue related to pull requests coming in from forks of the repo. The failing tests should be skipped for you now.

[JacksonBurns]

Just merged in the changes!

[cwognum]

Thanks again, @JacksonBurns ! 🙏