Repository navigation
V03/smoothemulatorscott - #72
Conversation
Merge remote-tracking branch 'refs/remotes/origin/v03/smoothemulatorscott' into v03/smoothemulatorscott
improvements
|
Marking as ready for review and seeking reviewers. |
|
@DanielRPhillips for socialization? |
|
@scottedwardpratt - is there any way that we can make SmoothUtilities part of SmoothEmulator? |
|
Hi, The problem is that the utilities are not particularly related to the emulator project. I use them across many projects.
I planned on updating the utilities in the band repo by simply doing occasional copies of source and include files from my “real” utilities repo.
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
% Scott Pratt ***@***.*** %
% FRIB +1-517-908-7460 %
% Michigan State University Office 2133 %
% East Lansing, MI 48824 http://nscl.msu.edu/directory/pratt.html %
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
On Oct 3, 2023, at 11:01 AM, Stefan M Wild ***@***.***> wrote:
@scottedwardpratt<https://urldefense.com/v3/__https://github.com/scottedwardpratt__;!!HXCxUKc!1dUQTKgBjZW12fRvWeRngFaTRR42tWgYG50-1tuR_Vi7YSbfcjsV6je5oj5Y8foaJD8Ccbx9zB93oHcZxHzNHHg$> - is there any way that we can make SmoothUtilities part of SmoothEmulator?
If you want to keep them fully separate, then I suggest having part of the install procedure for SmoothEmulator to be to clone the external dependency (probably within SmoothEmulator).
—
Reply to this email directly, view it on GitHub<https://urldefense.com/v3/__https://github.com/bandframework/bandframework/pull/72*issuecomment-1745161928__;Iw!!HXCxUKc!1dUQTKgBjZW12fRvWeRngFaTRR42tWgYG50-1tuR_Vi7YSbfcjsV6je5oj5Y8foaJD8Ccbx9zB93oHcZClIwCgc$>, or unsubscribe<https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AB6P62SBPBVQRKQ3AAIRLPDX5QSDLAVCNFSM6AAAAAA5ILDRD2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBVGE3DCOJSHA__;!!HXCxUKc!1dUQTKgBjZW12fRvWeRngFaTRR42tWgYG50-1tuR_Vi7YSbfcjsV6je5oj5Y8foaJD8Ccbx9zB93oHcZ1jU7BHg$>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Bringing up to date with develop
|
@scottedwardpratt - what is the location of the repo containing smoothutilities? I am proposing that the user be instructed to git clone this repo from GITHOME_BAND/SmoothEmulator (so that it would show up as a subdirectory of GITHOME_BAND/SmoothEmulator). In general, it would be good for everything needed to be brought in for smoothemulator to be in GITHOME_BAND/SmoothEmulator and not in a parallel file location. |
|
I am proceeding assuming that the instruction will reflect cloning and building commonutils (see below for my suggested way to proceed). Here are the steps I took:
I believe that this is because the correct Eigen3 include is not being used. |
…ndframework into v03/smoothemulatorscott
|
Scott says: it’s ready to be looked at. To clone the repo: If you go to the doc/ directory you can look at the User Manual, which has installation instructions, plus a tutorial chapter. Thanks to Stefan and Kyle for previous checks. Please let me know what problems you might be having. All the best — Scott To clone the repo: If you go to the doc/ directory you can look at the User Manual, which has installation instructions, plus a tutorial chapter. Thanks to Stefan and Kyle for previous checks. Please let me know what problems you might be having. All the best — Scott |
Bringing the latest develop into smoothemulator
…cott' into v03/smoothemulatorscott
|
@kylegodbey - are you reviewing this one? |
I will, once the yada yada's subside |
|
Hi, It’s all ready. The only things that we changed had to do with pybind11, which is not mentioned in this release, and does not interfere with anything else.
Thanks!
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
% Scott Pratt ***@***.*** %
% FRIB +1-517-908-7460 %
% Michigan State University Office 2133 %
% East Lansing, MI 48824 http://nscl.msu.edu/directory/pratt.html %
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
On Sep 13, 2024, at 10:53 AM, Kyle Godbey ***@***.***> wrote:
@kylegodbey<https://urldefense.com/v3/__https://github.com/kylegodbey__;!!HXCxUKc!wa_RWc9Vw1OLijvgFs6Z3h08tijAjMz3D6IM9SjFm_eB3ciZ53z4dFMrKaega9g2LRaZju1UeUW46dgtNbPDpBA$> - are you reviewing this one?
I will, once the yada yada's subside
—
Reply to this email directly, view it on GitHub<https://urldefense.com/v3/__https://github.com/bandframework/bandframework/pull/72*issuecomment-2349148531__;Iw!!HXCxUKc!wa_RWc9Vw1OLijvgFs6Z3h08tijAjMz3D6IM9SjFm_eB3ciZ53z4dFMrKaega9g2LRaZju1UeUW46dgt7VZUlFg$>, or unsubscribe<https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AB6P62XGNTUIDUSYIUUUW3LZWL36VAVCNFSM6AAAAAA5ILDRD2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBZGE2DQNJTGE__;!!HXCxUKc!wa_RWc9Vw1OLijvgFs6Z3h08tijAjMz3D6IM9SjFm_eB3ciZ53z4dFMrKaega9g2LRaZju1UeUW46dgtxBdq5WE$>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
|
A couple notes on the manual and process:
Other than that, I was able to run everything fine with no bugs. I think it's good to merge once some of the tutorial instructions are polished up. |
|
THANKS Kyle!
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
% Scott Pratt ***@***.*** %
% FRIB +1-517-908-7460 %
% Michigan State University Office 2133 %
% East Lansing, MI 48824 http://nscl.msu.edu/directory/pratt.html %
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
On Sep 18, 2024, at 5:17 PM, Kyle Godbey ***@***.***> wrote:
A couple notes on the manual and process:
* Maybe listing the C++ standard required somewhere would be a good idea -- I was building on an older Ubuntu version and had to fast forward to g++-11
* The cmake -D YOUR_PREFERRED_COMPILER instruction led to an error for me, so I instead did -D CMAKE_CXX_COMPILER=g++-11
* It is claimed that fakerhic makes a directory ${MY PROJECT}/fullmodel_testdata/, but for me it was in smooth_data. That may be user error.
* For the smooth emulator parameter file, perhaps put the name of the file in the manual's tutorial. Right now it says: For the template used in this tutorial, that file is
* Starting in section 10.2, there are some LaTeX errors -- perhaps due to \end{verbatime}
* Manual says figs/YvY/ but it should be figs/YvsY/
* My fullmodel_testdata directory discrepancy shows up again for the YvsY business. Perhaps I went awry
* The tutorial sections are a bit messed up. Looking in the ToC, we see section 8, then 9 and 10 that should be part of the tutorial, before jumping back to 9.
Other than that, I was able to run everything fine with no bugs. I think it's good to merge once some of the tutorial instructions are polished up.
—
Reply to this email directly, view it on GitHub<https://urldefense.com/v3/__https://github.com/bandframework/bandframework/pull/72*issuecomment-2359421488__;Iw!!HXCxUKc!3bsBl2sAJbbcbYLTyd3QxctgYPishV4lpqwOyYOe4QBhYRjKXAY7dWwtAiZVpnAPMBEpCnQifoT1XjX_OzsN-Eg$>, or unsubscribe<https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AB6P62VJ72TJFD4TGIIPJMTZXHUX3AVCNFSM6AAAAAA5ILDRD2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJZGQZDCNBYHA__;!!HXCxUKc!3bsBl2sAJbbcbYLTyd3QxctgYPishV4lpqwOyYOe4QBhYRjKXAY7dWwtAiZVpnAPMBEpCnQifoT1XjX_9slwHKM$>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Minor edits to sdk file.
Bringing up to date with develop
Added to SDK Index
Nice, spellcheck worked
|
Thanks @kylegodbey & @scottedwardpratt |
This is the general pull request to bring in @scottedwardpratt 's SmoothEmulator package.
It will start as a draft PR and then reviews will be done here. Please assign yourself as a reviewer if you would like to sign up to test it.