Change drop_nulls default in convert() to false

[gwaybio]

Description

Changes the convert() default to not drop nulls by default. This is more in-line with expected behavior.

Closes #56

What is the nature of your change?

• Bug fix (fixes an issue).
• Enhancement (adds functionality).
• Breaking change (fix or feature that would cause existing functionality to not work as expected).
• This change requires a documentation update.

Checklist

Please ensure that all boxes are checked before indicating that a pull request is ready for review.

• I have read the CONTRIBUTING.md guidelines.
• My code follows the style guidelines of this project.
• I have performed a self-review of my own code.
• I have commented my code, particularly in hard-to-understand areas.
• I have made corresponding changes to the documentation.
• My changes generate no new warnings.
• New and existing unit tests pass locally with my changes.
• I have added tests that prove my fix is effective or that my feature works.
• I have deleted all non-relevant text in this pull request template.

[d33bs]

Cheers, LGTM! Marking this as approved with the understanding that the tests will very likely pass after updating from main. The test failure is I believe unrelated to the changes here and instead pertains to issues which are fixed from the merge of #79.

[gwaybio]

Thanks @d33bs - tests pass now, I'm going to merge!

[gwaybio]

Tests passed here, but not in the merge: https://github.com/cytomining/CytoTable/actions/runs/5696050273/job/15440405650 🤔

[d33bs]

Going to follow up on the errors to investigate further.