I think this makes it more complicated, or maybe I just didn't get your point….what is the problem you see in the current approach?  Why is not as generic as the approach you proposed?

From: Ritesh Maheshwari <notifications@github.com<mailto:notifications@github.com>>
Reply-To: linkedin/naarad <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Wednesday, March 12, 2014 5:19 PM
To: linkedin/naarad <naarad@noreply.github.com<mailto:naarad@noreply.github.com>>
Cc: Shaoshan Liu <shliu@linkedin.com<mailto:shliu@linkedin.com>>
Subject: Re: [naarad] Extend Diff to support URL based CDF csv file paths, also Fix issue 149: cdf not plotted for sar important metrics (#155)


We should cover both cases. If a metric is in device_type_metrics set above (Jmeter should be added to that set), then check if both items[-1] or '.'join(items[-2:]) are in self.important_sub_metrics.