Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Duplicate "Energies" Futures Ticker Shortcut to Energy #8009

Closed
3 tasks done
jaredbroad opened this issue May 6, 2024 · 1 comment · Fixed by #8067 or #8082
Closed
3 tasks done

Duplicate "Energies" Futures Ticker Shortcut to Energy #8009

jaredbroad opened this issue May 6, 2024 · 1 comment · Fixed by #8067 or #8082
Assignees
Labels

Comments

@jaredbroad
Copy link
Member

Expected Behavior

Energy isn't countable should not use plural in shortcut.
Futures.Energy.BRENT_CRUDE,

Actual Behavior

Futures.Energies.BRENT_CRUDE,

Duplicate the energy shortcuts to make it backward compatible.

Checklist

  • I have completely filled out this template
  • I have confirmed that this issue exists on the current master branch
  • I have confirmed that this is not a duplicate issue by searching issues
@jaredbroad jaredbroad added the bug label May 6, 2024
@AlexCatarino AlexCatarino transferred this issue from QuantConnect/Documentation May 9, 2024
@Martin-Molinero
Copy link
Member

Reopening after reverting associated PR which was breaking pep8 support, see issue #8080 .
self.add_future(Futures.Energy.BRENT_CRUDE) use case should continue to work, should add unit test asserting this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants