Zeroclickinfo-goodies: Conversions: Mistaking MB for Mb

[pjhampton]

Description

Based on some user feedback, the Conversions IA is not disambiguating between MegaBytes (MB) and MegaBits (Mb). It is, therefore, nessesary to extend this IA to solve this issue

Correct 👍

Not Correct 👎

Get Started

• [x] 1) Claim this issue by commenting below
• [x] 2) Review our Contributing Guide
• [x] 3) Set up your development environment, and fork this repository
• [ ] 4) Create a Pull Request

Resources

• Join DuckDuckHack Slack to ask questions
• Join the DuckDuckHack Forum to discuss project planning and Instant Answer metrics
• Read the DuckDuckHack Documentation for technical help

Instant Answer Page: https://duck.co/ia/view/conversions

[nalinbhardwaj]

I think it's a better idea idea to make a separate issue to add case sensitivity to the IA. This and some other units can be sorted out with that. (cc @mintsoft, I think he's already working on this)

[mintsoft]

Basically a duplicate: https://github.com/duckduckgo/zeroclickinfo-goodies/issues/3544

[pjhampton]

Thanks! Yeah, it's quite chaotic at the moment due to the backlog of solicited + unsolicited issues / prs. One thing that we're taking away from the Programming Mission is the success of consistent and descriptive task templates / Meta Issues. The highest priority tasks are coming from the SERP feedback box. The reason for this is that the feedback is internally vetted and search data is analyzed against the feedback. We'll continue #3544 here.

Edit:
It also seems there is a web of issues growing around certain IAs. If possible we should aim to consolidate them into small-project like tasks.

[nalinbhardwaj]

@mintsoft if you're not working on this issue I can take it up. Also I think this issue should be expanded to enable case sensitivity for all units.

[mintsoft]

@nalinbhardwaj I am already working on it, I'm waiting for https://github.com/duckduckgo/zeroclickinfo-goodies/pull/3988, then case sensitivity is the next step

[mintsoft]

Actually; I missed one, this is next after https://github.com/duckduckgo/zeroclickinfo-goodies/pull/3993 / https://github.com/duckduckgo/zeroclickinfo-goodies/issues/3918

[mintsoft]

@pjhampton Having just looked at it, this is definitely not low hanging fruit; it's going to be ... interesting getting this to behave as users expect

[pjhampton]

Sure, @mintsoft. Someone labeled it low hanging fruit in the internal pipeline - I didn't look into it enough to dispute this reasoning. Feel free to adjust the labeling as you feel fit.

[mintsoft]

@pjhampton I just noticed that both of the screenshot examples are wrong! The first query is actually kilobits to megabytes and the second is kilobits to megabits both mistreat Kb as kilobytes

[pjhampton]

@mintsoft Ah, nice catch! 😄
That's interesting though because that was the user's exact query from the feedback. I believe they were expecting a conversion from Kilobytes. Thanks for pointing this out! I will update the description later on for the sake of clarity 👍