Thank you so much Wayne,
Just a comment, not sure if it matters.
I assume you took the LnL values directly from the RAxML output produced
by the different runs.
However, in our experiments we took the final trees and then executed
the tree evaluation option on these with more conservative epsilon
parameter settings.
Maybe if you do this, some final LnL values obtained under the more
liberal epsilon settings for the tree searches might further improve in
this case?
Alexis
On 25.07.22 19:30, Pfeiffer, Wayne wrote:
> Hi Julia,
>
> Thanks for the revised instructions. I downloaded the code directly to
> Expanse, built the code after changing the thresholds, and repeated all
> of the benchmarks. An updated spreadsheet with new results is attached.
>
> The speedups in Column P are much better now! Across all eight data
> sets, the median and average speedups are 1.51 and 1.72, respectively.
> That is really impressive and will be welcomed by our users!
>
> I also highlighted two final scores from the fast code in Column O.
>
> - The pink score in Cell O40 is the worst compared to the original code
> across all the data sets. Not surprisingly, the speedup in the next
> column is also the largest.
>
> - The blue score in Cell O30 is actually better than that from the
> original code and is the best such case.
>
> Best regards, Wayne
>
>
>> On Jul 25, 2022, at 8:46 AM, Haag, Julia <
julia...@h-its.org
>> <mailto:
julia...@h-its.org>> wrote:
>>
>> Hi Wayne,
>>
>>> I discovered that pll-modules and terraphast were missing in lib, so
>>> I copied them from the production version of 1.1.0
>> For terraphast that should be fine, but in order for the brlen epsilon
>> setting to work, you need a special version of the pll-modules which
>> you can find here:
https://github.com/tschuelia/pll-modules
>> <
https://urldefense.com/v3/__https://github.com/tschuelia/pll-modules__;!!Mih3wA!DslakEwgDjrXMwk9XgWRjPVJFUsfOeSHHkkxKsED729Lj8Tew7a6APkoAPKcUQ5-ZPjEiP9qt9Le9BfDjVjPtA$> (pll-modules
>> in the unmodified version has this threshold hardcoded to 0.1).
>> Alternatively you can get all required modules at once by cloning the
>> repo recursive (git clone --recursive
>>
https://github.com/tschuelia/raxml-ng.git
>> <
https://urldefense.com/v3/__https://github.com/tschuelia/raxml-ng.git__;!!Mih3wA!DslakEwgDjrXMwk9XgWRjPVJFUsfOeSHHkkxKsED729Lj8Tew7a6APkoAPKcUQ5-ZPjEiP9qt9Le9BfI9-17TA$>)
>>
>>> I tarred them before uploading to Expanse
>> The OS X error is very odd, I have never seen this one before, but I
>> have build my RAxML-NG version on different machines (Mac and Linux).
>> I’m guessing that your Mac adds something to the files during the tar
>> process. Can you try to download the files directly to Expanse (e.g.
>> using git clone) without using the Mac and try building it again?
>>
>> Greetings,
>> Julia
>>
>>> Am 25.07.2022 um 17:17 schrieb Pfeiffer, Wayne <
pfei...@sdsc.edu
>>> <mailto:
pfei...@sdsc.edu>>:
>>> <
https://urldefense.com/v3/__https://github.com/tschuelia/raxml-ng__;!!Mih3wA!EBO5DPxGttMbeYRi65pgPGk-xamd33R6Vkwff_XXknUx3AgOhjzJKyrNBkc5Y9x9E85ynPKI4KBMHbiDs2KPKw$> to
>>>> <
https://urldefense.com/v3/__https://github.com/tschuelia/raxml-ng__;!!Mih3wA!EBO5DPxGttMbeYRi65pgPGk-xamd33R6Vkwff_XXknUx3AgOhjzJKyrNBkc5Y9x9E85ynPKI4KBMHbiDs2KPKw$>
>>>> Note that there are more than two thresholds settable, so in order
>>>> to get the correct threshold settings you need to change all of the
>>>> following in the constants.hpp from 0.1 to 10 or 1000:
>>>>
>>>> Line 7: DEF_LH_EPSILON 10
>>>> Line 8: DEF_LH_EPSILON_AUTO 10
>>>> Line 9: DEF_LH_EPSILON_FAST 10
>>>> Line 10: DEF_LH_EPSILON_SLOW 10
>>>>
>>>> Line 15: DEF_LH_EPSILON_BRLEN_FULL 10
>>>> Line 16: DEF_LH_EPSILON_BRLEN_TRIPLET 1000
>>>>
>>>> Greetings,
>>>> Julia
>>>>
>>>>
>>>>
>>>>> Am 25.07.2022 um 15:18 schrieb Pfeiffer, Wayne <
pfei...@sdsc.edu
>>>>> <mailto:
pfei...@sdsc.edu>>:
>>>>>> <mailto:
julia...@h-its.org>> wrote:
>>>>>>
>>>>>> Hello Wayne,
>>>>>>
>>>>>> Thanks for sharing your results, they are very interesting! :-)
>>>>>>
>>>>>> First of all what Alexey wrote about the second threshold is
>>>>>> unfortunately correct, but the threshold he linked is yet a
>>>>>> different threshold, you should best use this RAxML-NG
>>>>>> versionhttps://
github.com/lukashuebner/param-raxml-ng
>>>>>> <
https://urldefense.com/v3/__https://github.com/lukashuebner/param-raxml-ng__;!!Mih3wA!C4ZheRdD0R_Wo5DL1TrFInj9FJilUZN4CsUPgB7ABg2Yi5yPt6aqYva_Zj_vIcFj0Ftg0seDe-HBUGD75fFPkQ$> and
>>>>>>> <
alexei...@gmail.com <mailto:
alexei...@gmail.com>>:
>>>>>>>
>>>>>>> Hello Wayne,
>>>>>>>
>>>>>>> many thanks for your tests! Really appreciate that you always
>>>>>>> keep a sharp eye on our work :)
>>>>>>>
>>>>>>>> As suggested in your paper, I changed two lines in constants.hpp
>>>>>>>> of RAxML-NG 1.1.0 to relax two numerical thresholds as follows:
>>>>>>>> #define DEF_LH_EPSILON 10.
>>>>>>>
>>>>>>> that's correct
>>>>>>>
>>>>>>>> #define RAXML_BRLEN_DEFAULT 1000.
>>>>>>>
>>>>>>> This is unfortunately the wrong constant, since it defines the
>>>>>>> default branch length, i.e. starting point and not the likelihood
>>>>>>> epsilon for branch length optimization. I'm a bit surprised that
>>>>>>> you nevertheless got reasonable results :)
>>>>>>>
>>>>>>> This epsilon was actually hardcoded, and we made it configurable
>>>>>>> in Julia's branch of raxml-ng only. I guess it's this one:
>>>>>>>
>>>>>>>
https://github.com/amkozlov/raxml-ng/commit/f8bc1985c4d5a0fe972633d437bca66b0d55c5d9
>>>>>>> <
https://urldefense.com/v3/__https://github.com/amkozlov/raxml-ng/commit/f8bc1985c4d5a0fe972633d437bca66b0d55c5d9__;!!Mih3wA!EBO5DPxGttMbeYRi65pgPGk-xamd33R6Vkwff_XXknUx3AgOhjzJKyrNBkc5Y9x9E85ynPKI4KBMHbho8k5A8Q$>
>>>>>>>>> <mailto:
alexandros...@gmail.com><mailto:
alexandros...@gmail.com
>>>>>>>>> <
https://urldefense.com/v3/__https://www.biorxiv.org/content/10.1101/2022.07.13.499893v1__;!!Mih3wA!HsKI560bu08iL24_UZnDEag71ybNtTfeVKPMh8xojsKWmbe779ckAr88YPa_ce3EUcmjZaxaDnLDW-8z-lnroAh_d20$><
https://urldefense.com/v3/__https://www.biorxiv.org/content/10.1101/2022.07.13.499893v1__;!!Mih3wA!HsKI560bu08iL24_UZnDEag71ybNtTfeVKPMh8xojsKWmbe779ckAr88YPa_ce3EUcmjZaxaDnLDW-8z-lnroAh_d20$
>>>>>>>>> <
https://urldefense.com/v3/__http://www.exelixis-lab.org__;!!Mih3wA!HsKI560bu08iL24_UZnDEag71ybNtTfeVKPMh8xojsKWmbe779ckAr88YPa_ce3EUcmjZaxaDnLDW-8z-lnrZhPqvvc$><
https://urldefense.com/v3/__http://www.exelixis-lab.org__;!!Mih3wA!HsKI560bu08iL24_UZnDEag71ybNtTfeVKPMh8xojsKWmbe779ckAr88YPa_ce3EUcmjZaxaDnLDW-8z-lnrZhPqvvc$
>>>>>>>>> it, send an email
toraxml+u...@googlegroups.com
>>>>>>>>> <mailto:
raxml+un...@googlegroups.com><mailto:
raxml+un...@googlegroups.com
>>>>>>>>> visithttps://
urldefense.com/v3/__https://groups.google.com/d/msgid/raxml/757401e5-d91e-5c52-ae02-330d13308f69*40gmail.com__;JQ!!Mih3wA!HsKI560bu08iL24_UZnDEag71ybNtTfeVKPMh8xojsKWmbe779ckAr88YPa_ce3EUcmjZaxaDnLDW-8z-lnrUNLWdxA$
>>>>>>>>> <
https://urldefense.com/v3/__https://groups.google.com/d/msgid/raxml/757401e5-d91e-5c52-ae02-330d13308f69*40gmail.com__;JQ!!Mih3wA!HsKI560bu08iL24_UZnDEag71ybNtTfeVKPMh8xojsKWmbe779ckAr88YPa_ce3EUcmjZaxaDnLDW-8z-lnrUNLWdxA$><
https://urldefense.com/v3/__https://groups.google.com/d/msgid/raxml/757401e5-d91e-5c52-ae02-330d13308f69*40gmail.com__;JQ!!Mih3wA!HsKI560bu08iL24_UZnDEag71ybNtTfeVKPMh8xojsKWmbe779ckAr88YPa_ce3EUcmjZaxaDnLDW-8z-lnrUNLWdxA$
>>>>>>>> it, send an email
toraxml+u...@googlegroups.com
>>>>>>>> <mailto:
raxml+un...@googlegroups.com><mailto:
raxml+un...@googlegroups.com
>>>>>>>> visithttps://
groups.google.com/d/msgid/raxml/7446509B-9DD0-4608-B0A1-BE3AD58607BC%40ucsd.edu
>>>>>>>> <
https://urldefense.com/v3/__https://groups.google.com/d/msgid/raxml/7446509B-9DD0-4608-B0A1-BE3AD58607BC*40ucsd.edu__;JQ!!Mih3wA!EBO5DPxGttMbeYRi65pgPGk-xamd33R6Vkwff_XXknUx3AgOhjzJKyrNBkc5Y9x9E85ynPKI4KBMHbhP-ABc8w$><
https://groups.google.com/d/msgid/raxml/7446509B-9DD0-4608-B0A1-BE3AD58607BC%40ucsd.edu?utm_medium=email&utm_source=footer
>>>>>>>> <
https://urldefense.com/v3/__https://groups.google.com/d/msgid/raxml/7446509B-9DD0-4608-B0A1-BE3AD58607BC*40ucsd.edu?utm_medium=email&utm_source=footer__;JQ!!Mih3wA!EBO5DPxGttMbeYRi65pgPGk-xamd33R6Vkwff_XXknUx3AgOhjzJKyrNBkc5Y9x9E85ynPKI4KBMHbht5pl7Qw$>>.
>>>>>
>>>>> <slurm-affall>
>>>>
>>>
>>
>
--
Alexandros (Alexis) Stamatakis
Research Group Leader, Heidelberg Institute for Theoretical Studies
Full Professor, Dept. of Informatics, Karlsruhe Institute of Technology
Affiliated Scientist, Evolutionary Genetics and Paleogenomics (EGP) lab,
Institute of Molecular Biology and Biotechnology, Foundation for
Research and Technology Hellas
www.exelixis-lab.org