Don't assume order is the same in observed and predicted features

[danielcohenlive]

Summary: This is causing only 3 of 14 arms to show on an experiment. We're still assuming that within an observation the order of the metric names matches the order of the corresponding data.

Differential Revision: D68274239

[facebook-github-bot]

This pull request was exported from Phabricator. Differential Revision: D68274239

[facebook-github-bot]

This pull request was exported from Phabricator. Differential Revision: D68274239

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.88%. Comparing base (ca93faa) to head (0821eaa).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3243   +/-   ##
=======================================
  Coverage   95.88%   95.88%           
=======================================
  Files         513      513           
  Lines       51671    51669    -2     
=======================================
- Hits        49543    49542    -1     
+ Misses       2128     2127    -1     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.