crypto_hist.set_index("time") raises KeyError(f"None of {missing} are in the columns")
This statement is not explained on the Merge Data from BTC, LTC and ETC section from Portfolio #3, of the Financial Analysis workshop documentation.
### Where is the relevant portion of the documentation located? Line 340 from the Financial Analysis workshop file, Portfolio #3. The code, crypto_hist.set_index("time") in the Merge Data from BTC, LTC and ETC section.
crypto_hist.set_index("time")
### What would you suggest it be replaced with? Suggest that this line be removed. If you run the crypto_hist statement in the interactive console, the output already shows the 'time' column indexed.
I ran the code without the crypto_hist.set_index("time") statement on line 340 and the file generated the necessary plots and values.
### Other information
* Is the problem specific to documentation for a particular version of Spyder?
* [ ] N/A
* [X] Spyder 5 (``master``)
* [ ] Spyder 4 (``4.x``)
* [ ] Spyder 3 (``3.x``, frozen)
* Are you planning on submitting a Github pull request with a suitable change?
* [ ] Yes
* [ ] No
* [ ] Maybe so
I was going to, but after reading the contributing guidelines, my interpretation was to submit as an issue since the documentation for the workshop stated that it doesn't really explain the procedures on the code. Perhaps removing this line of code, may be enough to avoid further inquiry or confusion for workshop participants, without generating a pull request? I would be happy to submit one if someone from the team let's me know that it is the approach we should take.
Issue Report
Issue Description
crypto_hist.set_index("time") raises KeyError(f"None of {missing} are in the columns") This statement is not explained on the Merge Data from BTC, LTC and ETC section from Portfolio #3, of the Financial Analysis workshop documentation. ### Where is the relevant portion of the documentation located? Line 340 from the Financial Analysis workshop file, Portfolio #3. The code, crypto_hist.set_index("time") in the Merge Data from BTC, LTC and ETC section. crypto_hist.set_index("time") ### What would you suggest it be replaced with? Suggest that this line be removed. If you run the crypto_hist statement in the interactive console, the output already shows the 'time' column indexed. I ran the code without the crypto_hist.set_index("time") statement on line 340 and the file generated the necessary plots and values. ### Other information * Is the problem specific to documentation for a particular version of Spyder? * [ ] N/A * [X] Spyder 5 (``master``) * [ ] Spyder 4 (``4.x``) * [ ] Spyder 3 (``3.x``, frozen) * Are you planning on submitting a Github pull request with a suitable change? * [ ] Yes * [ ] No * [ ] Maybe so I was going to, but after reading the contributing guidelines, my interpretation was to submit as an issue since the documentation for the workshop stated that it doesn't really explain the procedures on the code. Perhaps removing this line of code, may be enough to avoid further inquiry or confusion for workshop participants, without generating a pull request? I would be happy to submit one if someone from the team let's me know that it is the approach we should take.