[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #6232 [Analysis]: Make entropy-over-time graph
#6232: Make entropy-over-time graph
-------------------------+--------------------------------------------------
Reporter: arma | Owner:
Type: enhancement | Status: needs_revision
Priority: normal | Milestone:
Component: Analysis | Version:
Keywords: | Parent:
Points: | Actualpoints:
-------------------------+--------------------------------------------------
Comment(by gsathya):
Replying to [comment:37 karsten]:
> - The stem sources shouldn't be checked into the metrics-tasks
repository. Can you remove the commit adding stem and replace it with
instructions for including stem's sources?
Right now, it's a ugly hack to include stem, since we need stem/stem and
not stem/, we can't use git submodules -
{{{
#!/bin/bash
git clone git://git.torproject.org/stem.git
cp -r stem/stem stem1
rm -rf stem/
mv stem1 stem
}}}
Do we want to do this or just include stem in metrics-lib?
> - The parsing code for Wed, Wee, etc. is a rather dirty hack, because it
depends on the position of weights in the line. I'm not even sure if
weights have been added in the past years, so this is very likely to
break.
I've made it better now. Let me know what you think of this -
https://github.com/gsathya/metrics-
tasks/commit/c87685dd83da7e8abc469d9306b5dfe4d2dfd05d
> As I said, I'm going to do a more detailed review tomorrow, and then
I'll run your code on a few gigs of data. Looking forward to that! :)
Thanks!
Great =)
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/6232#comment:38>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs