<div dir="ltr">I've created the issues:<div><br></div><div><a href="https://github.com/cctbx/cctbx_project/issues/125">https://github.com/cctbx/cctbx_project/issues/125</a><br></div><div><a href="https://github.com/cctbx/cctbx_project/issues/126">https://github.com/cctbx/cctbx_project/issues/126</a><br></div><div><br></div><div>-Aaron</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 23, 2018 at 10:50 AM, Paul Adams <span dir="ltr"><<a href="mailto:pdadams@lbl.gov" target="_blank">pdadams@lbl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Aaron,<br>
<br>
thanks for the email. I think your proposal to remove the compile time dependencies by moving code to extension modules is great. I think this would be greatly appreciated by Rob and others.<br>
Fable, contrary to recent rumors, is actually often used by others outside of the cctbx community. While it is true that we wrote it for mainly one task, it turns out there are lots of people with old f77 code they’d like to translate. Given that we don’t seem to have any dependency or support issues with the code I’d recommend it stays - making and supporting a separate package would be too much.<br>
<br>
Cheers,<br>
Paul<br>
<div><div class="h5"><br>
> On Jan 23, 2018, at 9:53 AM, Aaron Brewster <<a href="mailto:asbrewster@lbl.gov">asbrewster@lbl.gov</a>> wrote:<br>
><br>
> Hi folks,<br>
><br>
> There are lots of threads in this email. Pulling a couple of them...<br>
> • The LBL team runs the libtbx/phil/tst_interface.py test nightly in buildbot as part of testing phenix, so the test is exercised (and it passed last night). Whether it should be moved to phenix_regression is another issue (I don't think it's necessary, the has_module call seems clear to me).<br>
> • I think there are many projects that shouldn't be moved out of cctbx_project. IOTA, xfel, prime, simtbx, and dxtbx are all under active development and are useful to many crystallographers. cctbx is a toolbox after all. It just has a lot of tools.<br>
> • That said, It may be worth having a discussion about fable, and maybe one or two of the others. That can be done in separate threads for each project?<br>
> • I would like to break compile-time dependencies on external packages in dxtbx as identified by Rob. If the C++ code in dxtbx/iotbx that imports cbf.h and hdf5.h is moved to separate boost-python extension modules (some of this is done already), then the SConscript can be made smarter and compile only the libraries it can find headers for. I'd be in favor of making two issues to track this work, one for cbflib and one for hdf5. That would be a step in the direction restoring the 'standaloneness' of the cctbx package.<br>
> -Aaron<br>
><br>
> On Tue, Jan 23, 2018 at 3:16 AM, <<a href="mailto:markus.gerstel@diamond.ac.uk">markus.gerstel@diamond.ac.uk</a>> wrote:<br>
> Hi Marcin<br>
><br>
> Hmm, so in other words "his free / open source software cannot be fully tested without agreeing to this license"?<br>
><br>
> No, still doesn't make sense to me. It only raises more questions. Such as: does Travis have a phenix license?<br>
><br>
> Currently we don't run these tests at all - although we *should* run all tests on libtbx - because we don't have phenix installed during cctbx tests.<br>
> Either the required test data should fully go into libtbx or the test should fully go into phenix. Distributing a test in two parts is no good to anyone.<br>
><br>
> -Markus<br>
><br>
> ______________________________<wbr>__________<br>
> From: <a href="mailto:cctbxbb-bounces@phenix-online.org">cctbxbb-bounces@phenix-online.<wbr>org</a> [<a href="mailto:cctbxbb-bounces@phenix-online.org">cctbxbb-bounces@phenix-<wbr>online.org</a>] on behalf of Marcin Wojdyr [<a href="mailto:wojdyr@gmail.com">wojdyr@gmail.com</a>]<br>
> Sent: Tuesday, January 23, 2018 10:55<br>
> To: cctbx mailing list<br>
> Cc: Randy Read<br>
> Subject: Re: [cctbxbb] Tidy up CCTBX<br>
><br>
> Hi Graeme,<br>
><br>
> > Turns out that the assert that libtbx has no dependencies is not strictly true:<br>
> ><br>
><br>
> > if (not libtbx.env.has_module(name="<wbr>phenix")):<br>
> > print "phenix module not available: skipping advanced tests"<br>
> > return<br>
><br>
> This is optional dependency (cctbx optionally uses Phenix for testing<br>
> in various places) and doesn't cause problems.<br>
> If you'd like to do disentangling, dxtbx-iotbx would be a good<br>
> starting point as it really causes problems.<br>
> ______________________________<wbr>_________________<br>
> cctbxbb mailing list<br>
> <a href="mailto:cctbxbb@phenix-online.org">cctbxbb@phenix-online.org</a><br>
> <a href="http://phenix-online.org/mailman/listinfo/cctbxbb" rel="noreferrer" target="_blank">http://phenix-online.org/<wbr>mailman/listinfo/cctbxbb</a><br>
><br>
> --<br>
> This e-mail and any attachments may contain confidential, copyright and or privileged material, and are for the use of the intended addressee only. If you are not the intended addressee or an authorised recipient of the addressee please notify us of receipt by returning the e-mail and do not use, copy, retain, distribute or disclose the information in or attached to the e-mail.<br>
> Any opinions expressed within this e-mail are those of the individual and not necessarily of Diamond Light Source Ltd.<br>
> Diamond Light Source Ltd. cannot guarantee that this e-mail or any attachments are free from viruses and we cannot accept liability for any damage which you may sustain as a result of software viruses which may be transmitted in or with the message.<br>
> Diamond Light Source Limited (company no. 4375679). Registered in England and Wales with its registered office at Diamond House, Harwell Science and Innovation Campus, Didcot, Oxfordshire, OX11 0DE, United Kingdom<br>
><br>
><br>
> ______________________________<wbr>_________________<br>
> cctbxbb mailing list<br>
> <a href="mailto:cctbxbb@phenix-online.org">cctbxbb@phenix-online.org</a><br>
> <a href="http://phenix-online.org/mailman/listinfo/cctbxbb" rel="noreferrer" target="_blank">http://phenix-online.org/<wbr>mailman/listinfo/cctbxbb</a><br>
><br>
> ______________________________<wbr>_________________<br>
> cctbxbb mailing list<br>
> <a href="mailto:cctbxbb@phenix-online.org">cctbxbb@phenix-online.org</a><br>
> <a href="http://phenix-online.org/mailman/listinfo/cctbxbb" rel="noreferrer" target="_blank">http://phenix-online.org/<wbr>mailman/listinfo/cctbxbb</a><br>
<br>
--<br>
</div></div>Paul Adams<br>
Division Director, Molecular Biophysics & Integrated Bioimaging, Lawrence Berkeley Lab<br>
Division Deputy for Biosciences, Advanced Light Source, Lawrence Berkeley Lab<br>
Adjunct Professor, Department of Bioengineering, U.C. Berkeley<br>
Vice President for Technology, the Joint BioEnergy Institute<br>
Laboratory Research Manager, ENIGMA Science Focus Area<br>
<br>
Building 33, Room 347<br>
Building 978, Room 4126<br>
Tel: <a href="tel:1-510-486-4225" value="+15104864225">1-510-486-4225</a>, Fax: <a href="tel:1-510-486-5909" value="+15104865909">1-510-486-5909</a><br>
<a href="http://cci.lbl.gov/paul" rel="noreferrer" target="_blank">http://cci.lbl.gov/paul</a><br>
<br>
Lawrence Berkeley Laboratory<br>
1 Cyclotron Road<br>
BLDG 33R0345<br>
Berkeley, CA 94720, USA.<br>
<br>
Executive Assistant: Louise Benvenue [ <a href="mailto:LBenvenue@lbl.gov">LBenvenue@lbl.gov</a> ]<a href="tel:%5B%201-510-495-2506" value="+15104952506">[ 1-510-495-2506</a> ]<br>
--<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
cctbxbb mailing list<br>
<a href="mailto:cctbxbb@phenix-online.org">cctbxbb@phenix-online.org</a><br>
<a href="http://phenix-online.org/mailman/listinfo/cctbxbb" rel="noreferrer" target="_blank">http://phenix-online.org/<wbr>mailman/listinfo/cctbxbb</a><br>
</div></div></blockquote></div><br></div>