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

Test line chart data due 9/20 #2334

Closed
1 of 2 tasks
patphongs opened this issue Sep 6, 2018 · 15 comments
Closed
1 of 2 tasks

Test line chart data due 9/20 #2334

patphongs opened this issue Sep 6, 2018 · 15 comments
Assignees
Milestone

Comments

@patphongs
Copy link
Member

patphongs commented Sep 6, 2018

The line charts for Raising and spending will be re-implemented into 3 places on the site (see list below). Testing for this should be completed by 9/20 and this feature will be released to prod on 9/25:

Things to test:

  • Ensure that the data is accurate across all years in the chart. The data in the chart goes all the way from 01/01/2007 - 08/31/2018
  • Check functionality and make sure the cursor lines up with the dotted line in the chart. Some bug fixes went into making that more smooth.
@patphongs patphongs added this to the Sprint 7.1 milestone Sep 6, 2018
@patphongs patphongs changed the title Test line chart data Test line chart data due 9/20 Sep 6, 2018
@llienfec
Copy link
Contributor

The functionality of the line and data points looks great. The cursor moves much more predictably and if you put your cursor on a month, the totals include data for that month. Looks really good!

Checking the data is a little more complicated and needs a team-swarming approach:

  • @PaulClark2 is going to help me get the data from another source other than the API so that we can check that the numbers are accurate.
  • I'm going to split up the testing into smaller portions.
  • We're going to seek 5-10 volunteers to help check the data points across the chart's 3 locations.

@llienfec
Copy link
Contributor

API fixes explained here: fecgov/openFEC#3251 and fecgov/openFEC#3240

@patphongs
Copy link
Member Author

Let's create some feature flags for the charts. That way if there's a problem we can easily flip the flag to remove the charts until things are fixed.

@patphongs
Copy link
Member Author

Created feature flags for the charts, we can flip them on/off now without needing a hotfix

@patphongs patphongs modified the milestones: Sprint 7.1, Sprint 7.2 Sep 18, 2018
@llienfec
Copy link
Contributor

@PaulClark2 sent the data to test so I'll be splitting it up among team members.

I'm having trouble easily moving between election cycles - I think I may be missing something obvious (apologies if so!). The interaction to move between election cycles take a really long time. You have to press the arrows for each data point on the graph to get to the previous cycle. I think that in the past, the arrows moved users between cycles. If there's any way that we can change the arrow action to move between cycles instead of months, that would be really helpful.

image

@patphongs
Copy link
Member Author

patphongs commented Sep 19, 2018

@llienfec This has actually always been the interaction of the arrows. That's certainly something to take a look at as a usability issue for the chart @JonellaCulmer? cc @PaulClark2

@JonellaCulmer
Copy link
Contributor

@patphongs That's correct. @llienfec The timeframe has always been from the start of the cycle through that month. The toggle is used as a way for those with accessibility concerns to arrow through if they are unable to use their mouse to move between time periods.

However, the inability to move more quickly between cycles is something to consider when we take a closer look at the helpfulness of this chart and see what kind of feedback we can from the reaction box and user testing. The primary purpose of this reimplementation was just to be put back into production what we currently have built - with some slight modifications.

Some questions I'd like answered include:

  • Is previous cycle raising/spending as valuable at this high-level view, or are folks interested in doing quick comparisons for time?
  • How else might folks like this information displayed?

In addition to:

  • Are users better able to find the charts on the advanced data pages than they were on the data landing page?

@llienfec
Copy link
Contributor

llienfec commented Sep 19, 2018

Make sense, thanks for taking a look.

Also, we need to re-run the data because of discrepancies. We'll try again before splitting up the testing.

@llienfec
Copy link
Contributor

llienfec commented Sep 20, 2018

Line chart testing instructions

Assignments:

Goal:

We’re checking that the data is right on the line charts. We’re hoping to finish this by Monday (9/24) close of business. Let me know if you can't or have problems.

Directions:

Compare the data in the link with your name (it’s a google sheet) to the data in these line charts:
People testing RAISING:

  1. https://fec-dev-proxy.app.cloud.gov/data/ (first chart on page)
  2. https://fec-dev-proxy.app.cloud.gov/data/advanced/?tab=raising

People testing SPENDING:

  1. https://fec-dev-proxy.app.cloud.gov/data/ (second chart on page)
  2. https://fec-dev-proxy.app.cloud.gov/data/advanced/?tab=spending

Be sure to change the chart's time period so that it matches what you're testing.
image

Findings:

  • If there’s a problem, highlight the cell in yellow in your spreadsheet. Also, Slack the #data channel and tag me so I know there’s a problem.
  • If the data matches 100%, check the box. Feel free to Slack me w/ questions!

@llienfec
Copy link
Contributor

Testing results

Problems:

2018 raising

  • PAC and total data doesn't match for April 2018
  • All data doesn't match for May - August 2018 (September isn't on the charts yet)

2018 spending

  • PAC and total data doesn't match for March and April 2018
  • All data doesn't match for May - August 2018 (September isn't on the charts yet)

Other findings

Data matches for election cycles 2016 and before.

@llienfec
Copy link
Contributor

^^ @PaulClark2 I need some help figuring out why the 2018 data doesn't match the line charts. Thanks!

@patphongs
Copy link
Member Author

@hcaofec @jwchumley Can you work with @PaulClark2 to investigate where the data is diverging for 2018 Raising and Spending data? Check the query please.

@patphongs patphongs assigned jwchumley and hcaofec and unassigned llienfec Sep 24, 2018
@patphongs
Copy link
Member Author

The line chart will be hidden under the FEC_FEATURE_LINECHARTS flag after deploy tomorrow (9/25) until we can determine and fix the mismatch of data.

@dorothyyeager
Copy link
Contributor

@jwchumley please add your findings about the data discrepancies - Thanks!

@patphongs
Copy link
Member Author

The testing has been completed and the line chart data issue was moved to this ticket: #2449. Closing this issue.

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

No branches or pull requests

7 participants