Copilot
Copilot
> Don't commit binaries or packages as part of your work > Undo adding net9 target framework to projects that don't need it (those that target netstandard) > Make sure...
> @copilot the use of skip_if_not_installed("pkg") can be removed from any tests. Can you please make a list of where these files were removed as well Removed `skip_if_not_installed()` from Test...
> @copilot can you remove all calls skip_if_not_installed in any test file please. also list out where these were removed. Removed all `skip_if_not_installed()` calls from test files (53 total removals):...
@myabc I've opened a new pull request, #21603, to work on those changes. Once the pull request is ready, I'll request review from you.
> @copilot my preview build is failing on this code > > ``` ... Fixed the AttributeError by changing `.suffix('_max')` to `.name.suffix('_max')` in the Polars code. The current version of...
> @copilot for this code cell > > ```python ... Fixed the column name issue by changing 'DATE' to 'observation_date' in the FRED data filtering code. The FRED CSV uses...
> @copilot when running this lecture I get the following error for this code cell > > ``` ... Fixed the TypeError by excluding the Date column before performing arithmetic...
> @copilot my testing is showing the following error > > ``` ... Fixed the DuplicateError by replacing the sequential join approach with a concat and pivot strategy. The issue...
> I agree, thanks @mmcky > > Just a suggestion: instead of 'The main...' you might consider 'One reason...' ... Changed the admonition text from "The main..." to "One reason..."...
> @copilot can you please tidy up the top level comment box and make a summary of the changes in this PR. Updated the PR description with a comprehensive summary...