You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Evgeniya Maenkova (JIRA)" <ji...@apache.org> on 2008/02/09 00:48:07 UTC
[jira] Created: (HARMONY-5478) [classlib] [luni]
Math.floor/ceil/rint can be implemented w/o JNI calls
[classlib] [luni] Math.floor/ceil/rint can be implemented w/o JNI calls
-----------------------------------------------------------------------
Key: HARMONY-5478
URL: https://issues.apache.org/jira/browse/HARMONY-5478
Project: Harmony
Issue Type: Improvement
Components: Classlib
Reporter: Evgeniya Maenkova
Attachments: luni.math.2
With this improvement the test mentioned below shows 17% boost on my laptop.
package math;
public class Floor {
public static void main(String[] args) {
double d1 = -123123.234;
double d2 = 1231233.123123;
double d3 = 23423.23;
for (int i = 0; i < 2500 * 2500; i ++) {
Math.floor(d3);
Math.ceil(d1);
Math.rint(d1);
Math.floor(d2);
Math.ceil(d2);
Math.rint(d2);
Math.floor(d3);
Math.ceil(d3);
Math.rint(d3);
}
long start = System.currentTimeMillis();
for (int i = 0; i < 2500 * 2500; i ++) {
Math.floor(d3);
Math.ceil(d1);
Math.rint(d1);
Math.floor(d2);
Math.ceil(d2);
Math.rint(d2);
Math.floor(d3);
Math.ceil(d3);
Math.rint(d3);
}
System.out.println("floor result: " + (System.currentTimeMillis() - start));
}
}
Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
$ /cygdrive/c/home_test/temp_builds/egm_r618602/bin/java -Xem:server math.Floor
floor result: 17359
Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
$ /cygdrive/c/home_test/temp_builds/egm_r618602_Mj_Mc/bin/java -Xem:server math
.Floor
floor result: 14876
Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
$
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
Re: [jira] Assigned: (HARMONY-5478) [classlib] [luni] Math.floor/ceil/rint
can be implemented w/o JNI calls
Posted by Tim Ellison <t....@gmail.com>.
I'm sure Alexey will catch it anyway, but if the code is re-written to
avoid JNI (which is fine) then the patch should also remove any obsolete
native code too.
Thanks for the enhancement!
Regards,
Tim
Maenkova, Evgeniya G wrote:
> Thanks and for 5473 too.
>
> -----Original Message-----
> From: Alexey Petrenko (JIRA) [mailto:jira@apache.org]
> Sent: Saturday, February 09, 2008 11:02 AM
> To: commits@harmony.apache.org
> Subject: [jira] Assigned: (HARMONY-5478) [classlib] [luni]
> Math.floor/ceil/rint can be implemented w/o JNI calls
>
>
> [
> https://issues.apache.org/jira/browse/HARMONY-5478?page=com.atlassian.ji
> ra.plugin.system.issuetabpanels:all-tabpanel ]
>
> Alexey Petrenko reassigned HARMONY-5478:
> ----------------------------------------
>
> Assignee: Alexey Petrenko
>
>> [classlib] [luni] Math.floor/ceil/rint can be implemented w/o JNI
> calls
> -----------------------------------------------------------------------
>> Key: HARMONY-5478
>> URL:
> https://issues.apache.org/jira/browse/HARMONY-5478
>> Project: Harmony
>> Issue Type: Improvement
>> Components: Classlib
>> Reporter: Evgeniya Maenkova
>> Assignee: Alexey Petrenko
>> Attachments: luni.math.2
>>
>>
>> With this improvement the test mentioned below shows 17% boost on my
> laptop.
>> package math;
>> public class Floor {
>> public static void main(String[] args) {
>> double d1 = -123123.234;
>> double d2 = 1231233.123123;
>> double d3 = 23423.23;
>> for (int i = 0; i < 2500 * 2500; i ++) {
>> Math.floor(d3);
>> Math.ceil(d1);
>> Math.rint(d1);
>> Math.floor(d2);
>> Math.ceil(d2);
>> Math.rint(d2);
>> Math.floor(d3);
>> Math.ceil(d3);
>> Math.rint(d3);
>> }
>>
>> long start = System.currentTimeMillis();
>> for (int i = 0; i < 2500 * 2500; i ++) {
>> Math.floor(d3);
>> Math.ceil(d1);
>> Math.rint(d1);
>> Math.floor(d2);
>> Math.ceil(d2);
>> Math.rint(d2);
>> Math.floor(d3);
>> Math.ceil(d3);
>> Math.rint(d3);
>> }
>> System.out.println("floor result: " +
> (System.currentTimeMillis() - start));
>> }
>> }
>> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
>> $ /cygdrive/c/home_test/temp_builds/egm_r618602/bin/java -Xem:server
> math.Floor
>> floor result: 17359
>> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
>> $ /cygdrive/c/home_test/temp_builds/egm_r618602_Mj_Mc/bin/java
> -Xem:server math
>> .Floor
>> floor result: 14876
>> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
>> $
>
RE: [jira] Assigned: (HARMONY-5478) [classlib] [luni] Math.floor/ceil/rint can be implemented w/o JNI calls
Posted by "Maenkova, Evgeniya G" <ev...@intel.com>.
Thanks and for 5473 too.
-----Original Message-----
From: Alexey Petrenko (JIRA) [mailto:jira@apache.org]
Sent: Saturday, February 09, 2008 11:02 AM
To: commits@harmony.apache.org
Subject: [jira] Assigned: (HARMONY-5478) [classlib] [luni]
Math.floor/ceil/rint can be implemented w/o JNI calls
[
https://issues.apache.org/jira/browse/HARMONY-5478?page=com.atlassian.ji
ra.plugin.system.issuetabpanels:all-tabpanel ]
Alexey Petrenko reassigned HARMONY-5478:
----------------------------------------
Assignee: Alexey Petrenko
> [classlib] [luni] Math.floor/ceil/rint can be implemented w/o JNI
calls
>
-----------------------------------------------------------------------
>
> Key: HARMONY-5478
> URL:
https://issues.apache.org/jira/browse/HARMONY-5478
> Project: Harmony
> Issue Type: Improvement
> Components: Classlib
> Reporter: Evgeniya Maenkova
> Assignee: Alexey Petrenko
> Attachments: luni.math.2
>
>
> With this improvement the test mentioned below shows 17% boost on my
laptop.
> package math;
> public class Floor {
> public static void main(String[] args) {
> double d1 = -123123.234;
> double d2 = 1231233.123123;
> double d3 = 23423.23;
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
>
> long start = System.currentTimeMillis();
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
> System.out.println("floor result: " +
(System.currentTimeMillis() - start));
> }
> }
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602/bin/java -Xem:server
math.Floor
> floor result: 17359
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602_Mj_Mc/bin/java
-Xem:server math
> .Floor
> floor result: 14876
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
--------------------------------------------------------------------
Closed Joint Stock Company Intel A/O
Registered legal address: 125252, Moscow, Russian Federation,
Chapayevsky Per, 14.
This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
[jira] Assigned: (HARMONY-5478) [classlib] [luni]
Math.floor/ceil/rint can be implemented w/o JNI calls
Posted by "Alexey Petrenko (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-5478?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Alexey Petrenko reassigned HARMONY-5478:
----------------------------------------
Assignee: Alexey Petrenko
> [classlib] [luni] Math.floor/ceil/rint can be implemented w/o JNI calls
> -----------------------------------------------------------------------
>
> Key: HARMONY-5478
> URL: https://issues.apache.org/jira/browse/HARMONY-5478
> Project: Harmony
> Issue Type: Improvement
> Components: Classlib
> Reporter: Evgeniya Maenkova
> Assignee: Alexey Petrenko
> Attachments: luni.math.2
>
>
> With this improvement the test mentioned below shows 17% boost on my laptop.
> package math;
> public class Floor {
> public static void main(String[] args) {
> double d1 = -123123.234;
> double d2 = 1231233.123123;
> double d3 = 23423.23;
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
>
> long start = System.currentTimeMillis();
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
> System.out.println("floor result: " + (System.currentTimeMillis() - start));
> }
> }
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602/bin/java -Xem:server math.Floor
> floor result: 17359
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602_Mj_Mc/bin/java -Xem:server math
> .Floor
> floor result: 14876
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (HARMONY-5478) [classlib] [luni]
Math.floor/ceil/rint can be implemented w/o JNI calls
Posted by "Evgeniya Maenkova (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-5478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12567570#action_12567570 ]
Evgeniya Maenkova commented on HARMONY-5478:
--------------------------------------------
I left them to be used by StrictMath....
> [classlib] [luni] Math.floor/ceil/rint can be implemented w/o JNI calls
> -----------------------------------------------------------------------
>
> Key: HARMONY-5478
> URL: https://issues.apache.org/jira/browse/HARMONY-5478
> Project: Harmony
> Issue Type: Improvement
> Components: Classlib
> Reporter: Evgeniya Maenkova
> Assignee: Alexey Petrenko
> Attachments: luni.math.2
>
>
> With this improvement the test mentioned below shows 17% boost on my laptop.
> package math;
> public class Floor {
> public static void main(String[] args) {
> double d1 = -123123.234;
> double d2 = 1231233.123123;
> double d3 = 23423.23;
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
>
> long start = System.currentTimeMillis();
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
> System.out.println("floor result: " + (System.currentTimeMillis() - start));
> }
> }
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602/bin/java -Xem:server math.Floor
> floor result: 17359
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602_Mj_Mc/bin/java -Xem:server math
> .Floor
> floor result: 14876
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Resolved: (HARMONY-5478) [classlib] [luni]
Math.floor/ceil/rint can be implemented w/o JNI calls
Posted by "Alexey Petrenko (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-5478?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Alexey Petrenko resolved HARMONY-5478.
--------------------------------------
Resolution: Fixed
The patch has been applied.
Please verify.
> [classlib] [luni] Math.floor/ceil/rint can be implemented w/o JNI calls
> -----------------------------------------------------------------------
>
> Key: HARMONY-5478
> URL: https://issues.apache.org/jira/browse/HARMONY-5478
> Project: Harmony
> Issue Type: Improvement
> Components: Classlib
> Reporter: Evgeniya Maenkova
> Assignee: Alexey Petrenko
> Attachments: luni.math.2
>
>
> With this improvement the test mentioned below shows 17% boost on my laptop.
> package math;
> public class Floor {
> public static void main(String[] args) {
> double d1 = -123123.234;
> double d2 = 1231233.123123;
> double d3 = 23423.23;
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
>
> long start = System.currentTimeMillis();
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
> System.out.println("floor result: " + (System.currentTimeMillis() - start));
> }
> }
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602/bin/java -Xem:server math.Floor
> floor result: 17359
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602_Mj_Mc/bin/java -Xem:server math
> .Floor
> floor result: 14876
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (HARMONY-5478) [classlib] [luni]
Math.floor/ceil/rint can be implemented w/o JNI calls
Posted by "Alexey Petrenko (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-5478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12567565#action_12567565 ]
Alexey Petrenko commented on HARMONY-5478:
------------------------------------------
Why don't you remove native methods If we do not need then any more?
> [classlib] [luni] Math.floor/ceil/rint can be implemented w/o JNI calls
> -----------------------------------------------------------------------
>
> Key: HARMONY-5478
> URL: https://issues.apache.org/jira/browse/HARMONY-5478
> Project: Harmony
> Issue Type: Improvement
> Components: Classlib
> Reporter: Evgeniya Maenkova
> Assignee: Alexey Petrenko
> Attachments: luni.math.2
>
>
> With this improvement the test mentioned below shows 17% boost on my laptop.
> package math;
> public class Floor {
> public static void main(String[] args) {
> double d1 = -123123.234;
> double d2 = 1231233.123123;
> double d3 = 23423.23;
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
>
> long start = System.currentTimeMillis();
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
> System.out.println("floor result: " + (System.currentTimeMillis() - start));
> }
> }
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602/bin/java -Xem:server math.Floor
> floor result: 17359
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602_Mj_Mc/bin/java -Xem:server math
> .Floor
> floor result: 14876
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-5478) [classlib] [luni]
Math.floor/ceil/rint can be implemented w/o JNI calls
Posted by "Evgeniya Maenkova (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-5478?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Evgeniya Maenkova updated HARMONY-5478:
---------------------------------------
Attachment: luni.math.2
patch for luni module
> [classlib] [luni] Math.floor/ceil/rint can be implemented w/o JNI calls
> -----------------------------------------------------------------------
>
> Key: HARMONY-5478
> URL: https://issues.apache.org/jira/browse/HARMONY-5478
> Project: Harmony
> Issue Type: Improvement
> Components: Classlib
> Reporter: Evgeniya Maenkova
> Attachments: luni.math.2
>
>
> With this improvement the test mentioned below shows 17% boost on my laptop.
> package math;
> public class Floor {
> public static void main(String[] args) {
> double d1 = -123123.234;
> double d2 = 1231233.123123;
> double d3 = 23423.23;
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
>
> long start = System.currentTimeMillis();
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
> System.out.println("floor result: " + (System.currentTimeMillis() - start));
> }
> }
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602/bin/java -Xem:server math.Floor
> floor result: 17359
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602_Mj_Mc/bin/java -Xem:server math
> .Floor
> floor result: 14876
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Closed: (HARMONY-5478) [classlib] [luni]
Math.floor/ceil/rint can be implemented w/o JNI calls
Posted by "Evgeniya Maenkova (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-5478?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Evgeniya Maenkova closed HARMONY-5478.
--------------------------------------
Works, as expected. Thanks, Aleksey
> [classlib] [luni] Math.floor/ceil/rint can be implemented w/o JNI calls
> -----------------------------------------------------------------------
>
> Key: HARMONY-5478
> URL: https://issues.apache.org/jira/browse/HARMONY-5478
> Project: Harmony
> Issue Type: Improvement
> Components: Classlib
> Reporter: Evgeniya Maenkova
> Assignee: Alexey Petrenko
> Attachments: luni.math.2
>
>
> With this improvement the test mentioned below shows 17% boost on my laptop.
> package math;
> public class Floor {
> public static void main(String[] args) {
> double d1 = -123123.234;
> double d2 = 1231233.123123;
> double d3 = 23423.23;
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
>
> long start = System.currentTimeMillis();
> for (int i = 0; i < 2500 * 2500; i ++) {
> Math.floor(d3);
> Math.ceil(d1);
> Math.rint(d1);
> Math.floor(d2);
> Math.ceil(d2);
> Math.rint(d2);
> Math.floor(d3);
> Math.ceil(d3);
> Math.rint(d3);
> }
> System.out.println("floor result: " + (System.currentTimeMillis() - start));
> }
> }
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602/bin/java -Xem:server math.Floor
> floor result: 17359
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602_Mj_Mc/bin/java -Xem:server math
> .Floor
> floor result: 14876
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.