#481 - Pass metric name via constructor.

[vzurauskas]

PR for #481

Passing metric name via constructor to fit the Calculus interface to Java implementations.

[vzurauskas]

@paulodamaso Can you check why no reviewer has been assigned?

[paulodamaso]

@0crat in

[0crat]

@0crat in (here)

@paulodamaso Job #485 is already in scope

[paulodamaso]

@0crat status

[0crat]

@0crat status (here)

@paulodamaso This is what I know about this job in C7JGJ00DP, as in §32:

• The job #485 is in scope for 2mon
• The role is REV
• The job is not assigned to anyone
• The budget is not set yet
• These users are banned and won't be assigned:
  • @vzurauskas/z: This user reported the ticket
• Job footprint (restricted area)

[paulodamaso]

@rultor merge

[rultor]

@rultor merge

@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here

[0crat]

There is an unrecoverable failure on my side. Please, submit it here:

PID: 4@9ca55d38-31cb-44eb-9fbd-eb71fd9e124a, thread: pool-134-thread-1
com.zerocracy.radars.github.GhProject[69] com.zerocracy.radars.github.GhProjectNotFoundException: Github project for 'cqfn/jpeek' repo was not found

0.54.5: Issue: #485, Comment: 695219680

[0crat]

There is an unrecoverable failure on my side. Please, submit it here:

PID: 4@9ca55d38-31cb-44eb-9fbd-eb71fd9e124a, thread: pool-135-thread-1
com.zerocracy.radars.github.GhProject[69] com.zerocracy.radars.github.GhProjectNotFoundException: Github project for 'cqfn/jpeek' repo was not found

0.54.5: Issue: #485, Comment: 695220632

[rultor]

@rultor merge

@paulodamaso @vzurauskas Oops, I failed. You can see the full log here (spent 16s)

+ set -e
+ set -o pipefail
++ dirname ./run.sh
+ cd .
+ echo 31725
+ echo '1.69.1 839260fab'
1.69.1 839260fab
+ date
Sat Oct 31 16:31:22 CET 2020
+ uptime
 16:31:22 up 34 days,  1:37,  0 users,  load average: 1.09, 0.67, 0.34
+ cat
.rultor.yml in yegor256/home doesn't allow cqfn/jpeek to use its assets (there are 108 friends), see http://doc.rultor.com/reference.html#assets
+ exit -1
'cid' file is absent, container wasn't started correctly

[paulodamaso]

@rultor merge

[rultor]

@rultor merge

@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here

[rultor]

@rultor merge

@paulodamaso @vzurauskas Oops, I failed. You can see the full log here (spent 16s)

+ set -e
+ set -o pipefail
++ dirname ./run.sh
+ cd .
+ echo 406
+ echo '1.69.1 839260fab'
1.69.1 839260fab
+ date
Mon Jan 11 21:43:12 CET 2021
+ uptime
 21:43:12 up 106 days,  6:49,  0 users,  load average: 0.16, 0.33, 0.23
+ cat
.rultor.yml in yegor256/home doesn't allow cqfn/jpeek to use its assets (there are 108 friends), see http://doc.rultor.com/reference.html#assets
+ exit -1
'cid' file is absent, container wasn't started correctly

[paulodamaso]

@yegor256 could you please fix this? Seems that renaming the repo messed with some asset permissions in yegor256/home

[baudoliver7]

@vzurauskas Why not use an iteration on snippet :

if (this.params.containsKey(code)) {
    reports.add(
        new XslReport(
            chain.transform(skeleton), new XslCalculus(code),
             new ReportData(code, this.params, mean, sigma)
        )
    );
}

with code, mean and sigma some parameters. You could use mean and sigma default values where it appears only two parameters in ReportData constructor. You can also create a todo for that if you want.

[vzurauskas]

@baudoliver7 That is a good suggestion, however this PR has been blocked for a very long time and I no longer have the time to work on it (at least in the near future). You can take it over if you want.

[baudoliver7]

@vzurauskas Ok. But how can I do that ?

[vzurauskas]

@baudoliver7 You can ask @paulodamaso to assign this task to you and then copy this branch.

[yegor256]

@paulodamaso done in 18efdff

[baudoliver7]

@paulodamaso could you assign to me this PR ? I want to do some few changes on it.

[baudoliver7]

@paulodamaso ping

[0crat]

Job gh:cqfn/jpeek#485 is not assigned, can't get performer