Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
Trac
Trac
  • Project overview
    • Project overview
    • Details
    • Activity
  • Issues 246
    • Issues 246
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Operations
    • Operations
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Activity
  • Create a new issue
  • Issue Boards
Collapse sidebar

GitLab is used only for code review, issue tracking and project management. Canonical locations for source code are still https://gitweb.torproject.org/ https://git.torproject.org/ and git-rw.torproject.org.

  • Legacy
  • TracTrac
  • Issues
  • #28114

Closed (moved)
Open
Opened Oct 18, 2018 by Trac@tracbot

Improve getter names for boolean fields in metrics-lib

This ticket is created from #24419 (moved). #24419 (moved) notes that not all Onionoo methods could be changed to the isXY() pattern for boolean getter methods due to interfaces such as org.torproject.descriptor.NetworkStatusEntry. This ticket is to update metrics-libs accordingly. From #24419 (moved):

  1. I did not change src/test/org/torproject/metrics/onionoo/updater/DummyStatusEntry's "getUnmeasured()" to "isUnmeasured()" because the interface org.torproject.descriptor.NetworkStatusEntry requires it, and I figured org.torproject.descriptor is outside the scope of this ticket.

Agreed. We'd first have to change metrics-lib interfaces, put out a new major metrics-lib release, and then update the tests in Onionoo. Worth doing, though we would ideally combine this with other changes that require a major metrics-lib version bump.

Trac:
Username: efgyirfe784

To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information
Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
None
Due date
None
Reference: legacy/trac#28114