Make Claim.target_equals take precision into account
Open, Needs TriagePublic
Actions

Assigned To

None

Authored By

	Lokal_Profil
	Jan 17 2016, 9:10 PM

Description

The fact that Claim.target_equals() does WbTime comparison based only on year is counter-intuitive to me.

I would suggest making this take precision target.precision into account and demanding that target.precision < pywikibot.WbTime.PRECISION['year'] but would like to check what the original choice for comparing on year was.

Alternatively (or additionally) having a built in comparison where both target and value are WbTime would be useful. The method I use myself compares year, month, day etc. based on the relevant precision (so that 2010-01-01 and 2010-00-00 are equal but only if precision is year on both).

Related Objects

Mentioned Here: T69568: claimit.py conditionals:only add p:foo if p:bar exists

Event Timeline

Lokal_Profil created this task.Jan 17 2016, 9:10 PM

Lokal_Profil raised the priority of this task from to Needs Triage.

Lokal_Profil updated the task description. (Show Details)

Lokal_Profil added projects: Pywikibot, Pywikibot-Wikidata.

Lokal_Profil subscribed.

Restricted Application added subscribers: pywikibot-bugs-list, StudiesWorld, Aklapper. · View Herald TranscriptJan 17 2016, 9:10 PM

Alternatively I've misunderstood the purpose of Claim.target_equals()and the question it answers is instead "is the provided value within the precision of the target".

In which case if we should do something like:

if target.precision >= pywikibot.WbTime.PRECISION['year']:
    precision = int(1000000000) / pow(10, target.precision)
    return abs(self.target.year - int(value)) <= precision
else:
    return self.target.year == int(value))

target_equals is mostly a utility method to assist -onlyif and -onlyifnot command line options for T69568.

Supporting year only was "minimum viable product", and was discussed in patchset 11&12 of https://gerrit.wikimedia.org/r/#/c/179158/ .

Make Claim.target_equals take precision into accountOpen, Needs TriagePublicActions

Description

Related Objects

Event Timeline

Make Claim.target_equals take precision into account
Open, Needs TriagePublic
Actions