Well, that particular feature (CHECKOUT_SHALLOW) *is* supposed to work.  We have a plan to fix it tomorrow (er, today) but until then, just turn it for at least the django repo.


This is very strange... are you doing your work in a virtualenv? Have you run `python setup.py install` after `python setup.py build`?


These were values that I derived for my own system. I think that you should use literature values if you are doing these elements.

I think that DLPOLY is in eV - it should be in the user manual.

Cheers

Keith

On 12 Jan 2017, at 11:28, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com>> wrote:

that's great, but i see typo wrong according to literature in some example,

you set lambda for Ag and Pb 0.764955 but this value in
literature 0.850703? why?

unit for dl_poly field is ev?

thank you so much for quick reply.

On Thu, Jan 12, 2017 at 2:50 PM, keeeto <notifications@github.com<mailto:notifications@github.com>> wrote:

> Hi,
>
> lower_limit, upper_limit: For these parameters, usually the defaults
> provided in the files are sufficient. This could change if your potential
> is particularly slowly decaying for some reason.
> I would advise to use these numbers, but to run some small tests (single
> point total energy) to check that the energy is not sensitive to the chosen
> cutoffs. Also, for the sake of convenience it is probably easier to set
> upper and lower cutoffs to be the same for all ELEMENTS in your
> calculation.
>
> Grid density should be good at the default. Again you can try to test
> convergence if you like, as above.
>
> Cheers
>
> Keith
>
> On 12 Jan 2017, at 11:10, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com>> wrote:
>
> Hi
>
> thank you, i am so sorry, some parameters as:
>
> lower limit, upper limit and grid density for example for Fe metal
>
> how set them?
>
> thanks
>
> On Thu, Jan 12, 2017 at 10:43 AM, keeeto <notifications@github.com<mailto:notifications@github.com><mailto:
> notifications@github.com<mailto:notifications@github.com>>> wrote:
>
> > Hello,
> >
> > Thanks for your interest in the code. The parameters can be found in the
> > Appendix of the paper.
> >
> > Best,
> >
> > Keith
> >
> > On 11 Jan 2017, at 22:05, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com>>
> > wrote:
> >
> > > Acta Mater. 49 (2001) 4005 - 4015
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/keeeto/DLPOLY_EAM/commit/
> 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20447717>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/
> AX1FjxSoKTSMeucVidWdct1tEolCXHvzks5rRdKjgaJpZM4LhIH->
> > .
> >
>
>
>
> --
> *Best Regards*
> *Hamid **Mosaddeghi*
>
> *PhD of Physical Chemistry
> *
> *Department of Chemistry*
>
> *Isfahan University of Technology, Iran*
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub<https://github.com/
> keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> 55fc875ccd#commitcomment-20450602>, or mute the thread<https://github.com/
> notifications/unsubscribe-auth/ADhr5OHuLrwfTUElP9LBMjJx8LjpBJ
> tXks5rRgpBgaJpZM4LhIH->.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450735>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AX1Fj8D25_ZxL5yaQ0weLwGcaGFjuwrAks5rRgyZgaJpZM4LhIH->
> .
>

--
*Best Regards*
*Hamid **Mosaddeghi*

*PhD of Physical Chemistry
*
*Department of Chemistry*

*Isfahan University of Technology, Iran*

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<https://github.com/keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450797>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ADhr5FJ7WRG45sq6auXCOqRPIbT3cc5Dks5rRg5AgaJpZM4LhIH->.


see: [`os.path.join`](https://docs.python.org/2/library/os.path.html#os.path.join)


The Idea behind is to add a single parameter called `sandbox`, in the form of `/origin/root-sandobx/path#/container/destination/path`.

Sandbox directories are created during container creation phase, since they share names with containers.


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 962](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43803&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:15


See: (PEP8: Function names)[https://www.python.org/dev/peps/pep-0008/#function-names]


I tried this fix but it displayed nothing, variable was empty.


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 963](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43799&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:21


I was getting error and that's why I did by this way.


Those are bugs, did not notice yet :( Thanks for spotting!


TeamCity Platform Services :: Environment Manager :: Test Client [Build 934](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43804&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:01:30


AddChecksumReport (JasonGlazer) - Win64-Windows-7-VisualStudio-14: OK (2296 of 2356 tests passed, 202 test warnings)

 * 202 tests had: EIO diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.png) ![Test Badge](http://img.shields.io/badge/tests%20passed-97.45%25-yellow.png)


Скобки лишние


По идее можно упростить, но не критично


I've started from the [actual implementation in kotlinx.coroutines](https://github.com/Kotlin/kotlinx.coroutines/blob/master/kotlinx-coroutines-generate/src/main/kotlin/generate.kt), but didn't like it due to an object IteratorContinuation, allocated on each yiealdAll, though otherwise it was neat.


I did not originally use the key file list option, instead built a comma separated string, no real reason.
Regarding the group name, was not sure how different languages would work so played safe with using a known format.
I will delete all my saved settings.


masz 2 wywołania findLightBox co moze byc kosztowne, lepiej wywolaj raz i podstaw pod zmienna


Sorry, i didnt write the filename correctly, its a spelling error...
Then, I run 'python matconv2caffe.py refine_res101_voc2012.mat' in the terminal, and it results as below:

Traceback (most recent call last):
  File "matconv2caffe.py", line 57, in <module>
    matconv_params = loadmat(INPUT)
  File "/usr/local/lib/python2.7/dist-packages/scipy/io/matlab/mio.py", line 135, in loadmat
    MR = mat_reader_factory(file_name, appendmat, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/scipy/io/matlab/mio.py", line 65, in mat_reader_factory
    raise NotImplementedError('Please use HDF reader for matlab v7.3 files')
NotImplementedError: Please use HDF reader for matlab v7.3 files


We generally use the new-style Python formatting strings (See [Pyformat](https://pyformat.info))


tu to drugie wywyłanie


можно объединить в один if


what is your reason for change it? please explain for me.

On Thu, Jan 12, 2017 at 3:04 PM, keeeto <notifications@github.com> wrote:

> These were values that I derived for my own system. I think that you
> should use literature values if you are doing these elements.
>
> I think that DLPOLY is in eV - it should be in the user manual.
>
> Cheers
>
> Keith
>
> On 12 Jan 2017, at 11:28, hamidmosaddeghi <notifications@github.com<
> mailto:notifications@github.com>> wrote:
>
> that's great, but i see typo wrong according to literature in some
> example,
>
> you set lambda for Ag and Pb 0.764955 but this value in
> literature 0.850703? why?
>
> unit for dl_poly field is ev?
>
> thank you so much for quick reply.
>
> On Thu, Jan 12, 2017 at 2:50 PM, keeeto <notifications@github.com<mailto:
> notifications@github.com>> wrote:
>
> > Hi,
> >
> > lower_limit, upper_limit: For these parameters, usually the defaults
> > provided in the files are sufficient. This could change if your
> potential
> > is particularly slowly decaying for some reason.
> > I would advise to use these numbers, but to run some small tests (single
> > point total energy) to check that the energy is not sensitive to the
> chosen
> > cutoffs. Also, for the sake of convenience it is probably easier to set
> > upper and lower cutoffs to be the same for all ELEMENTS in your
> > calculation.
> >
> > Grid density should be good at the default. Again you can try to test
> > convergence if you like, as above.
> >
> > Cheers
> >
> > Keith
> >
> > On 12 Jan 2017, at 11:10, hamidmosaddeghi <notifications@github.com<
> mailto:notifications@github.com><
> > mailto:notifications@github.com>> wrote:
> >
> > Hi
> >
> > thank you, i am so sorry, some parameters as:
> >
> > lower limit, upper limit and grid density for example for Fe metal
> >
> > how set them?
> >
> > thanks
> >
> > On Thu, Jan 12, 2017 at 10:43 AM, keeeto <notifications@github.com<
> mailto:notifications@github.com><mailto:
> > notifications@github.com<mailto:notifications@github.com>>> wrote:
> >
> > > Hello,
> > >
> > > Thanks for your interest in the code. The parameters can be found in
> the
> > > Appendix of the paper.
> > >
> > > Best,
> > >
> > > Keith
> > >
> > > On 11 Jan 2017, at 22:05, hamidmosaddeghi <notifications@github.com<
> mailto:notifications@github.com><
> > mailto:notifications@github.com>>
> > > wrote:
> > >
> > > > Acta Mater. 49 (2001) 4005 - 4015
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub
> > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20447717>,
> > > or mute the thread
> > > <https://github.com/notifications/unsubscribe-auth/
> > AX1FjxSoKTSMeucVidWdct1tEolCXHvzks5rRdKjgaJpZM4LhIH->
> > > .
> > >
> >
> >
> >
> > --
> > *Best Regards*
> > *Hamid **Mosaddeghi*
> >
> > *PhD of Physical Chemistry
> > *
> > *Department of Chemistry*
> >
> > *Isfahan University of Technology, Iran*
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub<https://github.com/
> > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > 55fc875ccd#commitcomment-20450602>, or mute the thread<
> https://github.com/
> > notifications/unsubscribe-auth/ADhr5OHuLrwfTUElP9LBMjJx8LjpBJ
> > tXks5rRgpBgaJpZM4LhIH->.
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/keeeto/DLPOLY_EAM/commit/
> 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450735>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/AX1Fj8D25_
> ZxL5yaQ0weLwGcaGFjuwrAks5rRgyZgaJpZM4LhIH->
> > .
> >
>
>
>
> --
> *Best Regards*
> *Hamid **Mosaddeghi*
>
> *PhD of Physical Chemistry
> *
> *Department of Chemistry*
>
> *Isfahan University of Technology, Iran*
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub<https://github.com/
> keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> 55fc875ccd#commitcomment-20450797>, or mute the thread<https://github.com/
> notifications/unsubscribe-auth/ADhr5FJ7WRG45sq6auXCOqRPIbT3cc
> 5Dks5rRg5AgaJpZM4LhIH->.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450871>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AX1FjwXllelbgszBRFyHqY8UYTLLgCqrks5rRg--gaJpZM4LhIH->
> .
>

--
*Best Regards*
*Hamid **Mosaddeghi*

*PhD of Physical Chemistry
                               *
*Department of Chemistry*

*Isfahan University of Technology, Iran*


Your spacing is very erratic. One space between method definitions, please.


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/39845/testReport/) for this commit**


Avoid that the module commons_trusted_contacts, create a groups in CiviCRM


Because I added the branch as a parameter `BRANCH='master'`
If this is not needed then we can move it to be done in `docker build`

Although `git clone` is not the bottleneck because it's quite fast, the bottleneck is the `yarn build` that takes a few minutes to complete. This only happens once though.


Лишнее


Why go from the article schema to a web page?


rozumiem że to tymczasowa instrukcja, w warstwie aplikacji nie powinno być prezentacji


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 964](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43820&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:16


You should comment out sections using Python comment styles, not casting them to anonymous strings. (The latter gets stored as a non-accessible string by the interpreter, which means it allocates memory to storing it. The former is ignored)


            console.log(typeof result);   // 'string'
            var response = $.parseJSON(result);
            console.log(typeof response);    // 'object'


TeamCity Platform Services :: Environment Manager :: Test Client [Build 935](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43800&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:05:51


AIも多分いいんじゃないかな?時間があってもうちょっと凝るならmin-max法と反復深化を使うかな〜


Why all this space??


For my system - which included other elements and potentials I had a strained interface. You must choose your potential carefully depending on the system that you are investigating.

On 12 Jan 2017, at 11:41, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com>> wrote:

what is your reason for change it? please explain for me.

On Thu, Jan 12, 2017 at 3:04 PM, keeeto <notifications@github.com<mailto:notifications@github.com>> wrote:

> These were values that I derived for my own system. I think that you
> should use literature values if you are doing these elements.
>
> I think that DLPOLY is in eV - it should be in the user manual.
>
> Cheers
>
> Keith
>
> On 12 Jan 2017, at 11:28, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com>> wrote:
>
> that's great, but i see typo wrong according to literature in some
> example,
>
> you set lambda for Ag and Pb 0.764955 but this value in
> literature 0.850703? why?
>
> unit for dl_poly field is ev?
>
> thank you so much for quick reply.
>
> On Thu, Jan 12, 2017 at 2:50 PM, keeeto <notifications@github.com<mailto:notifications@github.com><mailto:
> notifications@github.com<mailto:notifications@github.com>>> wrote:
>
> > Hi,
> >
> > lower_limit, upper_limit: For these parameters, usually the defaults
> > provided in the files are sufficient. This could change if your
> potential
> > is particularly slowly decaying for some reason.
> > I would advise to use these numbers, but to run some small tests (single
> > point total energy) to check that the energy is not sensitive to the
> chosen
> > cutoffs. Also, for the sake of convenience it is probably easier to set
> > upper and lower cutoffs to be the same for all ELEMENTS in your
> > calculation.
> >
> > Grid density should be good at the default. Again you can try to test
> > convergence if you like, as above.
> >
> > Cheers
> >
> > Keith
> >
> > On 12 Jan 2017, at 11:10, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><
> > mailto:notifications@github.com>> wrote:
> >
> > Hi
> >
> > thank you, i am so sorry, some parameters as:
> >
> > lower limit, upper limit and grid density for example for Fe metal
> >
> > how set them?
> >
> > thanks
> >
> > On Thu, Jan 12, 2017 at 10:43 AM, keeeto <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><mailto:
> > notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com>>> wrote:
> >
> > > Hello,
> > >
> > > Thanks for your interest in the code. The parameters can be found in
> the
> > > Appendix of the paper.
> > >
> > > Best,
> > >
> > > Keith
> > >
> > > On 11 Jan 2017, at 22:05, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><
> > mailto:notifications@github.com>>
> > > wrote:
> > >
> > > > Acta Mater. 49 (2001) 4005 - 4015
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub
> > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20447717>,
> > > or mute the thread
> > > <https://github.com/notifications/unsubscribe-auth/
> > AX1FjxSoKTSMeucVidWdct1tEolCXHvzks5rRdKjgaJpZM4LhIH->
> > > .
> > >
> >
> >
> >
> > --
> > *Best Regards*
> > *Hamid **Mosaddeghi*
> >
> > *PhD of Physical Chemistry
> > *
> > *Department of Chemistry*
> >
> > *Isfahan University of Technology, Iran*
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub<https://github.com/
> > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > 55fc875ccd#commitcomment-20450602>, or mute the thread<
> https://github.com/
> > notifications/unsubscribe-auth/ADhr5OHuLrwfTUElP9LBMjJx8LjpBJ
> > tXks5rRgpBgaJpZM4LhIH->.
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/keeeto/DLPOLY_EAM/commit/
> 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450735>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/AX1Fj8D25_
> ZxL5yaQ0weLwGcaGFjuwrAks5rRgyZgaJpZM4LhIH->
> > .
> >
>
>
>
> --
> *Best Regards*
> *Hamid **Mosaddeghi*
>
> *PhD of Physical Chemistry
> *
> *Department of Chemistry*
>
> *Isfahan University of Technology, Iran*
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub<https://github.com/
> keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> 55fc875ccd#commitcomment-20450797>, or mute the thread<https://github.com/
> notifications/unsubscribe-auth/ADhr5FJ7WRG45sq6auXCOqRPIbT3cc
> 5Dks5rRg5AgaJpZM4LhIH->.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450871>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AX1FjwXllelbgszBRFyHqY8UYTLLgCqrks5rRg--gaJpZM4LhIH->
> .
>

--
*Best Regards*
*Hamid **Mosaddeghi*

*PhD of Physical Chemistry
*
*Department of Chemistry*

*Isfahan University of Technology, Iran*

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<https://github.com/keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450947>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ADhr5LMyVzf9mQXDBJKrpMD-Jq8s7WkZks5rRhFVgaJpZM4LhIH->.


@beyondsanity la pagina sulle "custom  background notification" non può essere messa dopo i sample, dato che ci sarà un sample riguardo a questo topic, ovviamente non sarà chiaro se non si è letto la pagina con la spiegazione.
Potrebbe esserci la categoria "Advanced topics" subito dopo "User guide" con dentro "custom bkg notification" e testing come pagina singola.
Così:
User guide:
     -
     -
     -
Adv topic:
     custom bkg
     testing
Sample
Other res


Please add support for Doogee x5 Max Pro. The following data which DualBootPatcher successfully works Doogee x5 Max Pro

`Doogee x5 Max Pro
General information:

Device ID: X5max_PRO

Codenames: X5max_PRO,DOOGEE,X5MP,n370b

Architecture: armeabi-v7a
Block device paths

Base directories:

    /dev/block/platform/mtk-msdc.0/by-name
    /dev/block/platform/mtk-msdc.0/11230000.msdc0/by-name

/system block devices:

    /dev/block/platform/mtk-msdc.0/by-name/system
    /dev/block/platform/mtk-msdc.0/11230000.msdc0/by-name/system
    /dev/block/mmcblk0p20

/cache block devices:

    /dev/block/platform/mtk-msdc.0/by-name/cache
    /dev/block/platform/mtk-msdc.0/11230000.msdc0/by-name/cache
    /dev/block/mmcblk0p21

/data block devices:

    /dev/block/platform/mtk-msdc.0/by-name/userdata
    /dev/block/platform/mtk-msdc.0/11230000.msdc0/by-name/userdata
    /dev/block/mmcblk0p22

Boot partition block devices:

    /dev/block/platform/mtk-msdc.0/by-name/boot
    /dev/block/platform/mtk-msdc.0/11230000.msdc0/by-name/boot
    /dev/block/mmcblk0p7

Recovery partition block devices:

    /dev/block/platform/mtk-msdc.0/by-name/recovery
    /dev/block/platform/mtk-msdc.0/11230000.msdc0/by-name/recovery
    /dev/block/mmcblk0p8

Boot UI

Supported: no
Encryption

Supported: no`


You need more informative commit message subjects. `keepClean` does not describe, even in general terms, what you are doing.


tu za bardzo polegasz na wywyłaniach toString, chodziło raczej o to żeby w osobnych liniach wyprintować poszczególne produkty w lightboxie


then i use the h5py module of python, but i cannot access date well...
In a python script i wrote myself,
'matconv_params = h5py.File('refinenet_res101_voc2012.mat', 'r')
print matconv_params['layers'][:]'

But it ends with:
KeyError: "Unable to open object (Object 'layers' doesn't exist)"


Everything is fine except that even is четный, odd - нечетный)


哈哈哈


tutaj możnaby te wyjątki obśłużyć osobno dla każdej komendy, bo w każdej znaczą co innego i jakieś bardziej szczegółowe błędy wyświetlić


Ok now the problem is the matlab version of your file is higher than the supported on [scipy](https://docs.scipy.org/doc/scipy-0.17.0/reference/generated/scipy.io.loadmat.html). You can use directly h5py [http://stackoverflow.com/questions/17316880/reading-v-7-3-mat-file-in-python](http://stackoverflow.com/questions/17316880/reading-v-7-3-mat-file-in-python). Right now I am currently bussy for updating the code properly and testing but I encourage you to do a pull request. Or you can wait and I will update it as soon as possible.

I am sorry I cannot do it faster.


exactly I have some other molecules and ion, but I don't how set it.

thank you so much for your attention.

On Thu, Jan 12, 2017 at 3:14 PM, keeeto <notifications@github.com> wrote:

> For my system - which included other elements and potentials I had a
> strained interface. You must choose your potential carefully depending on
> the system that you are investigating.
>
> On 12 Jan 2017, at 11:41, hamidmosaddeghi <notifications@github.com<
> mailto:notifications@github.com>> wrote:
>
> what is your reason for change it? please explain for me.
>
> On Thu, Jan 12, 2017 at 3:04 PM, keeeto <notifications@github.com<mailto:
> notifications@github.com>> wrote:
>
> > These were values that I derived for my own system. I think that you
> > should use literature values if you are doing these elements.
> >
> > I think that DLPOLY is in eV - it should be in the user manual.
> >
> > Cheers
> >
> > Keith
> >
> > On 12 Jan 2017, at 11:28, hamidmosaddeghi <notifications@github.com<
> mailto:notifications@github.com><
> > mailto:notifications@github.com>> wrote:
> >
> > that's great, but i see typo wrong according to literature in some
> > example,
> >
> > you set lambda for Ag and Pb 0.764955 but this value in
> > literature 0.850703? why?
> >
> > unit for dl_poly field is ev?
> >
> > thank you so much for quick reply.
> >
> > On Thu, Jan 12, 2017 at 2:50 PM, keeeto <notifications@github.com<
> mailto:notifications@github.com><mailto:
> > notifications@github.com<mailto:notifications@github.com>>> wrote:
> >
> > > Hi,
> > >
> > > lower_limit, upper_limit: For these parameters, usually the defaults
> > > provided in the files are sufficient. This could change if your
> > potential
> > > is particularly slowly decaying for some reason.
> > > I would advise to use these numbers, but to run some small tests
> (single
> > > point total energy) to check that the energy is not sensitive to the
> > chosen
> > > cutoffs. Also, for the sake of convenience it is probably easier to
> set
> > > upper and lower cutoffs to be the same for all ELEMENTS in your
> > > calculation.
> > >
> > > Grid density should be good at the default. Again you can try to test
> > > convergence if you like, as above.
> > >
> > > Cheers
> > >
> > > Keith
> > >
> > > On 12 Jan 2017, at 11:10, hamidmosaddeghi <notifications@github.com<
> mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com>> wrote:
> > >
> > > Hi
> > >
> > > thank you, i am so sorry, some parameters as:
> > >
> > > lower limit, upper limit and grid density for example for Fe metal
> > >
> > > how set them?
> > >
> > > thanks
> > >
> > > On Thu, Jan 12, 2017 at 10:43 AM, keeeto <notifications@github.com<
> mailto:notifications@github.com><
> > mailto:notifications@github.com><mailto:
> > > notifications@github.com<mailto:notifications@github.com><mailto:
> notifications@github.com>>> wrote:
> > >
> > > > Hello,
> > > >
> > > > Thanks for your interest in the code. The parameters can be found in
> > the
> > > > Appendix of the paper.
> > > >
> > > > Best,
> > > >
> > > > Keith
> > > >
> > > > On 11 Jan 2017, at 22:05, hamidmosaddeghi <notifications@github.com<
> mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com>>
> > > > wrote:
> > > >
> > > > > Acta Mater. 49 (2001) 4005 - 4015
> > > >
> > > > —
> > > > You are receiving this because you commented.
> > > > Reply to this email directly, view it on GitHub
> > > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20447717>,
> > > > or mute the thread
> > > > <https://github.com/notifications/unsubscribe-auth/
> > > AX1FjxSoKTSMeucVidWdct1tEolCXHvzks5rRdKjgaJpZM4LhIH->
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > *Best Regards*
> > > *Hamid **Mosaddeghi*
> > >
> > > *PhD of Physical Chemistry
> > > *
> > > *Department of Chemistry*
> > >
> > > *Isfahan University of Technology, Iran*
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub<https://github.com/
> > > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > > 55fc875ccd#commitcomment-20450602>, or mute the thread<
> > https://github.com/
> > > notifications/unsubscribe-auth/ADhr5OHuLrwfTUElP9LBMjJx8LjpBJ
> > > tXks5rRgpBgaJpZM4LhIH->.
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub
> > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450735>,
> > > or mute the thread
> > > <https://github.com/notifications/unsubscribe-auth/AX1Fj8D25_
> > ZxL5yaQ0weLwGcaGFjuwrAks5rRgyZgaJpZM4LhIH->
> > > .
> > >
> >
> >
> >
> > --
> > *Best Regards*
> > *Hamid **Mosaddeghi*
> >
> > *PhD of Physical Chemistry
> > *
> > *Department of Chemistry*
> >
> > *Isfahan University of Technology, Iran*
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub<https://github.com/
> > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > 55fc875ccd#commitcomment-20450797>, or mute the thread<
> https://github.com/
> > notifications/unsubscribe-auth/ADhr5FJ7WRG45sq6auXCOqRPIbT3cc
> > 5Dks5rRg5AgaJpZM4LhIH->.
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/keeeto/DLPOLY_EAM/commit/
> 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450871>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/
> AX1FjwXllelbgszBRFyHqY8UYTLLgCqrks5rRg--gaJpZM4LhIH->
> > .
> >
>
>
>
> --
> *Best Regards*
> *Hamid **Mosaddeghi*
>
> *PhD of Physical Chemistry
> *
> *Department of Chemistry*
>
> *Isfahan University of Technology, Iran*
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub<https://github.com/
> keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> 55fc875ccd#commitcomment-20450947>, or mute the thread<https://github.com/
> notifications/unsubscribe-auth/ADhr5LMyVzf9mQXDBJKrpMD-
> Jq8s7WkZks5rRhFVgaJpZM4LhIH->.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450987>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AX1Fj_6Av82Z1oPirCzjW6PT1dekJckHks5rRhIMgaJpZM4LhIH->
> .
>

--
*Best Regards*
*Hamid **Mosaddeghi*

*PhD of Physical Chemistry
                               *
*Department of Chemistry*

*Isfahan University of Technology, Iran*


it will not compile unless vendor blobs found. I forgot to add vendor files once but still it compiled everything.. later it did not boot and  I spent lot of time finding what went wrong lol ))


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 965](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43811&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:16


warto obsługę każdej komendy przenieś do osobnych metod


TeamCity Platform Services :: Environment Manager :: Test Client [Build 936](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43821&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:05:35


> But it ends with:
> KeyError: "Unable to open object (Object 'layers' doesn't exist)

This is because of the matconvnet version


HTMLElement: virtual destructor is missing


delete is missing.


Please consider security issues. "sql_external_settings.json" is public. This site itself needs authorization in case it is deployed standalone on public access area.


Please test it properly before commit.


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 966](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43825&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:32


Ok sir as you wish


What do people think about this approach?


Well,

 You must have data to fit it to. Either experiment or quantum mechanics calculations. You can always try with the value from that paper and see if it works.

On 12 Jan 2017, at 11:47, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com>> wrote:

exactly I have some other molecules and ion, but I don't how set it.

thank you so much for your attention.

On Thu, Jan 12, 2017 at 3:14 PM, keeeto <notifications@github.com<mailto:notifications@github.com>> wrote:

> For my system - which included other elements and potentials I had a
> strained interface. You must choose your potential carefully depending on
> the system that you are investigating.
>
> On 12 Jan 2017, at 11:41, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com>> wrote:
>
> what is your reason for change it? please explain for me.
>
> On Thu, Jan 12, 2017 at 3:04 PM, keeeto <notifications@github.com<mailto:notifications@github.com><mailto:
> notifications@github.com<mailto:notifications@github.com>>> wrote:
>
> > These were values that I derived for my own system. I think that you
> > should use literature values if you are doing these elements.
> >
> > I think that DLPOLY is in eV - it should be in the user manual.
> >
> > Cheers
> >
> > Keith
> >
> > On 12 Jan 2017, at 11:28, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><
> > mailto:notifications@github.com>> wrote:
> >
> > that's great, but i see typo wrong according to literature in some
> > example,
> >
> > you set lambda for Ag and Pb 0.764955 but this value in
> > literature 0.850703? why?
> >
> > unit for dl_poly field is ev?
> >
> > thank you so much for quick reply.
> >
> > On Thu, Jan 12, 2017 at 2:50 PM, keeeto <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><mailto:
> > notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com>>> wrote:
> >
> > > Hi,
> > >
> > > lower_limit, upper_limit: For these parameters, usually the defaults
> > > provided in the files are sufficient. This could change if your
> > potential
> > > is particularly slowly decaying for some reason.
> > > I would advise to use these numbers, but to run some small tests
> (single
> > > point total energy) to check that the energy is not sensitive to the
> > chosen
> > > cutoffs. Also, for the sake of convenience it is probably easier to
> set
> > > upper and lower cutoffs to be the same for all ELEMENTS in your
> > > calculation.
> > >
> > > Grid density should be good at the default. Again you can try to test
> > > convergence if you like, as above.
> > >
> > > Cheers
> > >
> > > Keith
> > >
> > > On 12 Jan 2017, at 11:10, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com>> wrote:
> > >
> > > Hi
> > >
> > > thank you, i am so sorry, some parameters as:
> > >
> > > lower limit, upper limit and grid density for example for Fe metal
> > >
> > > how set them?
> > >
> > > thanks
> > >
> > > On Thu, Jan 12, 2017 at 10:43 AM, keeeto <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><
> > mailto:notifications@github.com><mailto:
> > > notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com><mailto:
> notifications@github.com<mailto:notifications@github.com>>>> wrote:
> > >
> > > > Hello,
> > > >
> > > > Thanks for your interest in the code. The parameters can be found in
> > the
> > > > Appendix of the paper.
> > > >
> > > > Best,
> > > >
> > > > Keith
> > > >
> > > > On 11 Jan 2017, at 22:05, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com>>
> > > > wrote:
> > > >
> > > > > Acta Mater. 49 (2001) 4005 - 4015
> > > >
> > > > —
> > > > You are receiving this because you commented.
> > > > Reply to this email directly, view it on GitHub
> > > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20447717>,
> > > > or mute the thread
> > > > <https://github.com/notifications/unsubscribe-auth/
> > > AX1FjxSoKTSMeucVidWdct1tEolCXHvzks5rRdKjgaJpZM4LhIH->
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > *Best Regards*
> > > *Hamid **Mosaddeghi*
> > >
> > > *PhD of Physical Chemistry
> > > *
> > > *Department of Chemistry*
> > >
> > > *Isfahan University of Technology, Iran*
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub<https://github.com/
> > > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > > 55fc875ccd#commitcomment-20450602>, or mute the thread<
> > https://github.com/
> > > notifications/unsubscribe-auth/ADhr5OHuLrwfTUElP9LBMjJx8LjpBJ
> > > tXks5rRgpBgaJpZM4LhIH->.
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub
> > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450735>,
> > > or mute the thread
> > > <https://github.com/notifications/unsubscribe-auth/AX1Fj8D25_
> > ZxL5yaQ0weLwGcaGFjuwrAks5rRgyZgaJpZM4LhIH->
> > > .
> > >
> >
> >
> >
> > --
> > *Best Regards*
> > *Hamid **Mosaddeghi*
> >
> > *PhD of Physical Chemistry
> > *
> > *Department of Chemistry*
> >
> > *Isfahan University of Technology, Iran*
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub<https://github.com/
> > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > 55fc875ccd#commitcomment-20450797>, or mute the thread<
> https://github.com/
> > notifications/unsubscribe-auth/ADhr5FJ7WRG45sq6auXCOqRPIbT3cc
> > 5Dks5rRg5AgaJpZM4LhIH->.
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/keeeto/DLPOLY_EAM/commit/
> 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450871>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/
> AX1FjwXllelbgszBRFyHqY8UYTLLgCqrks5rRg--gaJpZM4LhIH->
> > .
> >
>
>
>
> --
> *Best Regards*
> *Hamid **Mosaddeghi*
>
> *PhD of Physical Chemistry
> *
> *Department of Chemistry*
>
> *Isfahan University of Technology, Iran*
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub<https://github.com/
> keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> 55fc875ccd#commitcomment-20450947>, or mute the thread<https://github.com/
> notifications/unsubscribe-auth/ADhr5LMyVzf9mQXDBJKrpMD-
> Jq8s7WkZks5rRhFVgaJpZM4LhIH->.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450987>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AX1Fj_6Av82Z1oPirCzjW6PT1dekJckHks5rRhIMgaJpZM4LhIH->
> .
>

--
*Best Regards*
*Hamid **Mosaddeghi*

*PhD of Physical Chemistry
*
*Department of Chemistry*

*Isfahan University of Technology, Iran*

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<https://github.com/keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20451019>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ADhr5Hxn-RdNgWhQ_nt2f8sLMJRy9885ks5rRhK9gaJpZM4LhIH->.


The first comment now makes sense to me but a bit confused about the last, the text implies -1 is returned on failure, @return The parsed value of date_str on success or -1 on failure


+1 for semantic versioning, even if means multiple major releases. Let's get those improvements out there!

Also, I do not see a small changelog with a new (major) version as a problem. On the contrary, the better and clearer we describe changes and impact between versions, the easier and safer it will be for users of the library to upgrade.

The alternative would be to introduce a versioned API (e.g. getNameId_v2()). This sounds nice, but for a user of the library that's just postponing the inevitable while increasing complexity for everyone in the mean time. At some point a user will have to do the work of moving to the new API.


it's good practice to add `override` keyword after overridden methods in derived classes:
```
string Render() override
{ ... }
```
This protects from annoying mistakes like relying on polymorphic behavior of methods which are actually not virtual.


Ich habe ein wenig mit dem online regex Tester probiert, ich denke so müsste es funktionieren:
`$frequency=~ s/[Ff]([0-9a-fA-F]+$)/$1/ if (defined($frequency));`


The rest is fine! Good job!


TeamCity Platform Services :: Environment Manager :: Test Client [Build 937](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43812&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:05:36


I've started from the [actual implementation in kotlinx.coroutines](https://github.com/Kotlin/kotlinx.coroutines/blob/master/kotlinx-coroutines-generate/src/main/kotlin/generate.kt), but didn't like it due to an object IteratorContinuation, allocated on each yiealdAll, though otherwise it was neat.


please


@aluanhaddad Thanks for explaining. Needs some time to absorb this. Quite a bit there to understand javascript history and its modern offerings. :)


i can't understand the meaning of the percent in the formula percent = val / (conf.nbWeeks).


Right. I remember now. If the saved settings are important to you, you can always manually correct them. These are the changes I have made:
- use ; instead of , for the list
- use the name as part of the group name (it's formatted as CSV - +name)
- start and end line meanings have been aligned. Both now indicate the number of lines to skip at either end. I believe the key for both options has been changed to reflect this as well.

Or you can also make a mix of changes directly in the key file and then do final corrections in the importer itself and resave the settings. Whatever suits you best.

Lastly, I have pushed one more commit minutes ago to deal with invalid characters in the group name. The key file spec indicates [ and ] are the only two characters not allowed in a key file group name. Any attempt to enter either will automatically replace them  with ( and ) respectively.


> It seems that it would be better to use a functor for processing the data.

It was faster for me to write it like this.  I will study how you convert it to functors :-)

> Using the MIDI stuff would make it easier to include page numbers.

And it probably can be incorporated into the existing MIDI extractor.

> I also want to look at first creating a MEI tree structure representing the timemap as suggested by @pe-ro and then outputting it as JSON (instead of the reverse)

The qstamps are an important thing to make available, so good to see how to do it in @pe-ro's note.  But also the locations of the tempo changes are needed to calculate the time-in-seconds from the qstamps.  It would be useful if they were stored in an acessible location along with qstamps.


fix "mbr structures error on image generation #2" issue.


Test


Why have you backed this out again?


I don't see any reason for this changes.
If we wont resolve this matter in #92  the're going to be reverted ASAP.


TeamCity Platform Services :: Environment Manager :: Test Client [Build 938](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43826&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:07:04


ok, mi sembra una buona soluzione


That's just my default packages. I copy and paste my latex template.


No existe ese parámetro "$treat_spaces_as_hyphens" en dev-master de URLify::filter


I have searched about matlab version supported by scipy, i have updated scipy on my computer, but it does not work.

' KeyError: "Unable to open object (Object 'layers' doesn't exist)'

Does the error above means that the current matconv2.py does not fit the version of matconvnet(or the version of refine_res101_voc2012.mat )?


Yeah need to spent a time reviewing this menu... 😭😭😭


I want in first step prepare one file for Ni and I have same file for NI
from https://sites.google.com/site/eampotentials/Ni#TOC-Other-formats

and then compare output. what't your idea?

On Thu, Jan 12, 2017 at 3:20 PM, keeeto <notifications@github.com> wrote:

> Well,
>
> You must have data to fit it to. Either experiment or quantum mechanics
> calculations. You can always try with the value from that paper and see if
> it works.
>
>
> On 12 Jan 2017, at 11:47, hamidmosaddeghi <notifications@github.com<
> mailto:notifications@github.com>> wrote:
>
> exactly I have some other molecules and ion, but I don't how set it.
>
> thank you so much for your attention.
>
> On Thu, Jan 12, 2017 at 3:14 PM, keeeto <notifications@github.com<mailto:
> notifications@github.com>> wrote:
>
> > For my system - which included other elements and potentials I had a
> > strained interface. You must choose your potential carefully depending
> on
> > the system that you are investigating.
> >
> > On 12 Jan 2017, at 11:41, hamidmosaddeghi <notifications@github.com<
> mailto:notifications@github.com><
> > mailto:notifications@github.com>> wrote:
> >
> > what is your reason for change it? please explain for me.
> >
> > On Thu, Jan 12, 2017 at 3:04 PM, keeeto <notifications@github.com<
> mailto:notifications@github.com><mailto:
> > notifications@github.com<mailto:notifications@github.com>>> wrote:
> >
> > > These were values that I derived for my own system. I think that you
> > > should use literature values if you are doing these elements.
> > >
> > > I think that DLPOLY is in eV - it should be in the user manual.
> > >
> > > Cheers
> > >
> > > Keith
> > >
> > > On 12 Jan 2017, at 11:28, hamidmosaddeghi <notifications@github.com<
> mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com>> wrote:
> > >
> > > that's great, but i see typo wrong according to literature in some
> > > example,
> > >
> > > you set lambda for Ag and Pb 0.764955 but this value in
> > > literature 0.850703? why?
> > >
> > > unit for dl_poly field is ev?
> > >
> > > thank you so much for quick reply.
> > >
> > > On Thu, Jan 12, 2017 at 2:50 PM, keeeto <notifications@github.com<
> mailto:notifications@github.com><
> > mailto:notifications@github.com><mailto:
> > > notifications@github.com<mailto:notifications@github.com><mailto:
> notifications@github.com>>> wrote:
> > >
> > > > Hi,
> > > >
> > > > lower_limit, upper_limit: For these parameters, usually the defaults
> > > > provided in the files are sufficient. This could change if your
> > > potential
> > > > is particularly slowly decaying for some reason.
> > > > I would advise to use these numbers, but to run some small tests
> > (single
> > > > point total energy) to check that the energy is not sensitive to the
> > > chosen
> > > > cutoffs. Also, for the sake of convenience it is probably easier to
> > set
> > > > upper and lower cutoffs to be the same for all ELEMENTS in your
> > > > calculation.
> > > >
> > > > Grid density should be good at the default. Again you can try to
> test
> > > > convergence if you like, as above.
> > > >
> > > > Cheers
> > > >
> > > > Keith
> > > >
> > > > On 12 Jan 2017, at 11:10, hamidmosaddeghi <notifications@github.com<
> mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com><
> > > > mailto:notifications@github.com>> wrote:
> > > >
> > > > Hi
> > > >
> > > > thank you, i am so sorry, some parameters as:
> > > >
> > > > lower limit, upper limit and grid density for example for Fe metal
> > > >
> > > > how set them?
> > > >
> > > > thanks
> > > >
> > > > On Thu, Jan 12, 2017 at 10:43 AM, keeeto <notifications@github.com<
> mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com><mailto:
> > > > notifications@github.com<mailto:notifications@github.com><mailto:
> notifications@github.com><mailto:
> > notifications@github.com<mailto:notifications@github.com>>>> wrote:
> > > >
> > > > > Hello,
> > > > >
> > > > > Thanks for your interest in the code. The parameters can be found
> in
> > > the
> > > > > Appendix of the paper.
> > > > >
> > > > > Best,
> > > > >
> > > > > Keith
> > > > >
> > > > > On 11 Jan 2017, at 22:05, hamidmosaddeghi <
> notifications@github.com<mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com><
> > > > mailto:notifications@github.com>>
> > > > > wrote:
> > > > >
> > > > > > Acta Mater. 49 (2001) 4005 - 4015
> > > > >
> > > > > —
> > > > > You are receiving this because you commented.
> > > > > Reply to this email directly, view it on GitHub
> > > > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > > > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20447717>,
> > > > > or mute the thread
> > > > > <https://github.com/notifications/unsubscribe-auth/
> > > > AX1FjxSoKTSMeucVidWdct1tEolCXHvzks5rRdKjgaJpZM4LhIH->
> > > > > .
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > *Best Regards*
> > > > *Hamid **Mosaddeghi*
> > > >
> > > > *PhD of Physical Chemistry
> > > > *
> > > > *Department of Chemistry*
> > > >
> > > > *Isfahan University of Technology, Iran*
> > > >
> > > > —
> > > > You are receiving this because you commented.
> > > > Reply to this email directly, view it on GitHub<https://github.com/
> > > > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > > > 55fc875ccd#commitcomment-20450602>, or mute the thread<
> > > https://github.com/
> > > > notifications/unsubscribe-auth/ADhr5OHuLrwfTUElP9LBMjJx8LjpBJ
> > > > tXks5rRgpBgaJpZM4LhIH->.
> > > >
> > > > —
> > > > You are receiving this because you commented.
> > > > Reply to this email directly, view it on GitHub
> > > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450735>,
> > > > or mute the thread
> > > > <https://github.com/notifications/unsubscribe-auth/AX1Fj8D25_
> > > ZxL5yaQ0weLwGcaGFjuwrAks5rRgyZgaJpZM4LhIH->
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > *Best Regards*
> > > *Hamid **Mosaddeghi*
> > >
> > > *PhD of Physical Chemistry
> > > *
> > > *Department of Chemistry*
> > >
> > > *Isfahan University of Technology, Iran*
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub<https://github.com/
> > > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > > 55fc875ccd#commitcomment-20450797>, or mute the thread<
> > https://github.com/
> > > notifications/unsubscribe-auth/ADhr5FJ7WRG45sq6auXCOqRPIbT3cc
> > > 5Dks5rRg5AgaJpZM4LhIH->.
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub
> > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450871>,
> > > or mute the thread
> > > <https://github.com/notifications/unsubscribe-auth/
> > AX1FjwXllelbgszBRFyHqY8UYTLLgCqrks5rRg--gaJpZM4LhIH->
> > > .
> > >
> >
> >
> >
> > --
> > *Best Regards*
> > *Hamid **Mosaddeghi*
> >
> > *PhD of Physical Chemistry
> > *
> > *Department of Chemistry*
> >
> > *Isfahan University of Technology, Iran*
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub<https://github.com/
> > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > 55fc875ccd#commitcomment-20450947>, or mute the thread<
> https://github.com/
> > notifications/unsubscribe-auth/ADhr5LMyVzf9mQXDBJKrpMD-
> > Jq8s7WkZks5rRhFVgaJpZM4LhIH->.
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/keeeto/DLPOLY_EAM/commit/
> 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450987>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/AX1Fj_
> 6Av82Z1oPirCzjW6PT1dekJckHks5rRhIMgaJpZM4LhIH->
> > .
> >
>
>
>
> --
> *Best Regards*
> *Hamid **Mosaddeghi*
>
> *PhD of Physical Chemistry
> *
> *Department of Chemistry*
>
> *Isfahan University of Technology, Iran*
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub<https://github.com/
> keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> 55fc875ccd#commitcomment-20451019>, or mute the thread<https://github.com/
> notifications/unsubscribe-auth/ADhr5Hxn-RdNgWhQ_
> nt2f8sLMJRy9885ks5rRhK9gaJpZM4LhIH->.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20451059>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AX1Fj93a3egkAhF90lcF0hIz_x74rYTxks5rRhNrgaJpZM4LhIH->
> .
>

--
*Best Regards*
*Hamid **Mosaddeghi*

*PhD of Physical Chemistry
                               *
*Department of Chemistry*

*Isfahan University of Technology, Iran*


**Important**: You have to have installed JUnit 5 and added it to the environment classpath if you're using IntelliJ.


Fixes #22


# test


いや，nullが入らないことは保証されてるのか?


das hier
`$frequency = substr($clock,1);`
müsste ich durch das folgende ersetzen können:
```
$clock=~ s/[Ff]([0-9a-fA-F]+$)/$1/;
$frequency = $clock;
```


Thanks Chris. :smile:


Bahaha. I suck.


This is how I coded the author + timestamp info in my mockup from #23:

`<span class="right" style="margin-top: 10px; font-weight: bold;">Timestamp • Author</span>`


Exact, it is matconvnet now, it has layers defined differently as when I wrote the code.


I agree on the frequency (was just testing it yesterday).

There is a "push" webhook in Github that triggers the sync to poeditor.


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 967](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43835&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:16


hi link is not valid


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 969](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43843&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **UNKNOWN**
Summary: Canceled (Snapshot dependency failed: ... Lint Server (new)) Build time: 00:00:00


TeamCity Platform Services :: Environment Manager :: Test Client [Build 940](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43844&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **UNKNOWN**
Summary: Canceled (Snapshot dependency canceled: ... Test Lambda (new)) Build time: 00:00:01


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 968](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43839&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:15


calling it now; the Alphas
there isn't any other safe space implemented for them yet, and it explains the Met Par Tek's & Kar Ik Vot on Avalon, Host (and some defending the Unfettered-Korath border when you follow the Jump Drive source)


happy you could reuse the TC work :)


As in task there were not point related to delete functionality that's why I did not take into account.


heatingcoil_propane_bugfix (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1783 of 1783 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.png) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.png) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-71.17%25-green.png)


So we want people to switch tabs every time they want to change autowall settings for each weapon?


heatingcoil_propane_bugfix (nmerket) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.png)


It seems that you have referred to the wrong issue! :stuck_out_tongue_closed_eyes:


This is actually a bug; the ranges are exclusive on the upper bound; so this needs to be set to range(4, 6) now.


![](https://github.trello.services/images/mini-trello-icon.png) [Code outside work at least 4 days per week.](https://trello.com/c/9vYX40vW/6-code-outside-work-at-least-4-days-per-week)


There is more that needs to be done here, but it's a good start.


OK, wait for your updating and I will also try other ways to get it done, you really do me a great favor. Thank you!


TeamCity Platform Services :: Environment Manager :: Test Client [Build 939](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43836&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:05:39


TeamCity Platform Services :: Environment Manager :: Test Client [Build 941](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43840&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:06:10


having a hard time broadcasting..u


Strange... Commit-name will not be changed anyways...


To avoid doing Count() twice, can we put to variable?


Fehlt da nicht das = in der Regex?


Важный момент


I've just done only a function finding user's password through one's E-mail.


sgtm


I KNEW IT!
they knew they'd lose in Korath space so they want to take human space


I think it would be a good idea to add a comment or two about the reason for the fix. Looking at this changes it is not immediately clear to me why they were done (and even after thinking about it a bit I only have a vague idea).

Also: According to just the [GHF markdown doc](https://guides.github.com/features/mastering-markdown/) mentioning the commit's SHA-1 sum generates a link to the according commit, so it might be helpful for people reading this in offline editors to not have these links here.


done!


![](https://github.trello.services/images/mini-trello-icon.png) [PCT_EP_VAL_volume_total_2015](https://trello.com/c/7qxFD7mY/10-pct-ep-val-volume-total-2015)


inb4 Elias is assassinated as well :(


ok


strip?


It leads to 'Argument 1 passed to Twig_Filter::__construct() must be an instance of string, string given...' fatal error for PHP 5.6.1.

The reason is https://packagist.org/packages/symfony/twig-bundle#v2.7.22 is depended on
twig/twig: ~1.28|~2.0

https://packagist.org/packages/twig/twig#v1.28.1 requires php: >=5.2.7
But https://packagist.org/packages/twig/twig#v2.0.0 requires php: ^7.0

Please remove type hinting or adjust dependencies


Unnecessary condition, replace by "else".


Always use `find_by(id: params[:id])`. Also in case the user is not found you need to throw a page_not_found.


Please use the [ticket tracker](https://code.djangoproject.com/) to report issues. Commenting on commits is not the place to do that.


why include a human god (Thanatos, Greek/Roman mythology, maybe the rest are too but I don't know them) in a Quarg ship? seems weird to me


да, для отложенной отправки нужен
здесь делал для другого проекта, где не используется отложенное


I like this. :) What are your column names called? `follower_id` and  `followed_id`? Did you index both of them? This should be a high performing table.


Do you know the meaning of `dependent: :destroy`? If not, I can explain it to you!


please move these kinds of comments to steam forum or even better, the discord. the line comments here are way better for highlighting typos and other constructive criticism/questions.


what if you became hostile to the Quarg? maybe make a side branch (although earlier in the storyline you get Quarg help too)


This query is space consuming. If you had a 1000 followings, you need to load all of them to memory. Instead do `followings.find_by(him_id: him.id)`.


I like this name. :)


This breaks compiling on Raspberry Pi Wheezy which contains 3.2. Is the new Python version really necessary?


I was also wondering if this would be needed. The choice does not seem to be following a consistent policy, for instance there is user-add-manager and stageuser-add-manager, but only user-add-principal and user-add-cert. Maybe we should ask the team for their opinion on this.


Why not build the TABEAM for Ni using the parameters from the Acta Mater paper. You can then test the calculations.

On 12 Jan 2017, at 11:59, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com>> wrote:

I want in first step prepare one file for Ni and I have same file for NI
from https://sites.google.com/site/eampotentials/Ni#TOC-Other-formats

and then compare output. what't your idea?

On Thu, Jan 12, 2017 at 3:20 PM, keeeto <notifications@github.com<mailto:notifications@github.com>> wrote:

> Well,
>
> You must have data to fit it to. Either experiment or quantum mechanics
> calculations. You can always try with the value from that paper and see if
> it works.
>
>
> On 12 Jan 2017, at 11:47, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com>> wrote:
>
> exactly I have some other molecules and ion, but I don't how set it.
>
> thank you so much for your attention.
>
> On Thu, Jan 12, 2017 at 3:14 PM, keeeto <notifications@github.com<mailto:notifications@github.com><mailto:
> notifications@github.com<mailto:notifications@github.com>>> wrote:
>
> > For my system - which included other elements and potentials I had a
> > strained interface. You must choose your potential carefully depending
> on
> > the system that you are investigating.
> >
> > On 12 Jan 2017, at 11:41, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><
> > mailto:notifications@github.com>> wrote:
> >
> > what is your reason for change it? please explain for me.
> >
> > On Thu, Jan 12, 2017 at 3:04 PM, keeeto <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><mailto:
> > notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com>>> wrote:
> >
> > > These were values that I derived for my own system. I think that you
> > > should use literature values if you are doing these elements.
> > >
> > > I think that DLPOLY is in eV - it should be in the user manual.
> > >
> > > Cheers
> > >
> > > Keith
> > >
> > > On 12 Jan 2017, at 11:28, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com>> wrote:
> > >
> > > that's great, but i see typo wrong according to literature in some
> > > example,
> > >
> > > you set lambda for Ag and Pb 0.764955 but this value in
> > > literature 0.850703? why?
> > >
> > > unit for dl_poly field is ev?
> > >
> > > thank you so much for quick reply.
> > >
> > > On Thu, Jan 12, 2017 at 2:50 PM, keeeto <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><
> > mailto:notifications@github.com><mailto:
> > > notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com><mailto:
> notifications@github.com<mailto:notifications@github.com>>>> wrote:
> > >
> > > > Hi,
> > > >
> > > > lower_limit, upper_limit: For these parameters, usually the defaults
> > > > provided in the files are sufficient. This could change if your
> > > potential
> > > > is particularly slowly decaying for some reason.
> > > > I would advise to use these numbers, but to run some small tests
> > (single
> > > > point total energy) to check that the energy is not sensitive to the
> > > chosen
> > > > cutoffs. Also, for the sake of convenience it is probably easier to
> > set
> > > > upper and lower cutoffs to be the same for all ELEMENTS in your
> > > > calculation.
> > > >
> > > > Grid density should be good at the default. Again you can try to
> test
> > > > convergence if you like, as above.
> > > >
> > > > Cheers
> > > >
> > > > Keith
> > > >
> > > > On 12 Jan 2017, at 11:10, hamidmosaddeghi <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com><
> > > > mailto:notifications@github.com>> wrote:
> > > >
> > > > Hi
> > > >
> > > > thank you, i am so sorry, some parameters as:
> > > >
> > > > lower limit, upper limit and grid density for example for Fe metal
> > > >
> > > > how set them?
> > > >
> > > > thanks
> > > >
> > > > On Thu, Jan 12, 2017 at 10:43 AM, keeeto <notifications@github.com<mailto:notifications@github.com><
> mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com><mailto:
> > > > notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com><mailto:
> notifications@github.com<mailto:notifications@github.com>><mailto:
> > notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com>>>> wrote:
> > > >
> > > > > Hello,
> > > > >
> > > > > Thanks for your interest in the code. The parameters can be found
> in
> > > the
> > > > > Appendix of the paper.
> > > > >
> > > > > Best,
> > > > >
> > > > > Keith
> > > > >
> > > > > On 11 Jan 2017, at 22:05, hamidmosaddeghi <
> notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com><
> > mailto:notifications@github.com><
> > > mailto:notifications@github.com><
> > > > mailto:notifications@github.com>>
> > > > > wrote:
> > > > >
> > > > > > Acta Mater. 49 (2001) 4005 - 4015
> > > > >
> > > > > —
> > > > > You are receiving this because you commented.
> > > > > Reply to this email directly, view it on GitHub
> > > > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > > > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20447717>,
> > > > > or mute the thread
> > > > > <https://github.com/notifications/unsubscribe-auth/
> > > > AX1FjxSoKTSMeucVidWdct1tEolCXHvzks5rRdKjgaJpZM4LhIH->
> > > > > .
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > *Best Regards*
> > > > *Hamid **Mosaddeghi*
> > > >
> > > > *PhD of Physical Chemistry
> > > > *
> > > > *Department of Chemistry*
> > > >
> > > > *Isfahan University of Technology, Iran*
> > > >
> > > > —
> > > > You are receiving this because you commented.
> > > > Reply to this email directly, view it on GitHub<https://github.com/
> > > > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > > > 55fc875ccd#commitcomment-20450602>, or mute the thread<
> > > https://github.com/
> > > > notifications/unsubscribe-auth/ADhr5OHuLrwfTUElP9LBMjJx8LjpBJ
> > > > tXks5rRgpBgaJpZM4LhIH->.
> > > >
> > > > —
> > > > You are receiving this because you commented.
> > > > Reply to this email directly, view it on GitHub
> > > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450735>,
> > > > or mute the thread
> > > > <https://github.com/notifications/unsubscribe-auth/AX1Fj8D25_
> > > ZxL5yaQ0weLwGcaGFjuwrAks5rRgyZgaJpZM4LhIH->
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > *Best Regards*
> > > *Hamid **Mosaddeghi*
> > >
> > > *PhD of Physical Chemistry
> > > *
> > > *Department of Chemistry*
> > >
> > > *Isfahan University of Technology, Iran*
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub<https://github.com/
> > > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > > 55fc875ccd#commitcomment-20450797>, or mute the thread<
> > https://github.com/
> > > notifications/unsubscribe-auth/ADhr5FJ7WRG45sq6auXCOqRPIbT3cc
> > > 5Dks5rRg5AgaJpZM4LhIH->.
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub
> > > <https://github.com/keeeto/DLPOLY_EAM/commit/
> > 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450871>,
> > > or mute the thread
> > > <https://github.com/notifications/unsubscribe-auth/
> > AX1FjwXllelbgszBRFyHqY8UYTLLgCqrks5rRg--gaJpZM4LhIH->
> > > .
> > >
> >
> >
> >
> > --
> > *Best Regards*
> > *Hamid **Mosaddeghi*
> >
> > *PhD of Physical Chemistry
> > *
> > *Department of Chemistry*
> >
> > *Isfahan University of Technology, Iran*
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub<https://github.com/
> > keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> > 55fc875ccd#commitcomment-20450947>, or mute the thread<
> https://github.com/
> > notifications/unsubscribe-auth/ADhr5LMyVzf9mQXDBJKrpMD-
> > Jq8s7WkZks5rRhFVgaJpZM4LhIH->.
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/keeeto/DLPOLY_EAM/commit/
> 5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20450987>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/AX1Fj_
> 6Av82Z1oPirCzjW6PT1dekJckHks5rRhIMgaJpZM4LhIH->
> > .
> >
>
>
>
> --
> *Best Regards*
> *Hamid **Mosaddeghi*
>
> *PhD of Physical Chemistry
> *
> *Department of Chemistry*
>
> *Isfahan University of Technology, Iran*
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub<https://github.com/
> keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb
> 55fc875ccd#commitcomment-20451019>, or mute the thread<https://github.com/
> notifications/unsubscribe-auth/ADhr5Hxn-RdNgWhQ_
> nt2f8sLMJRy9885ks5rRhK9gaJpZM4LhIH->.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20451059>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AX1Fj93a3egkAhF90lcF0hIz_x74rYTxks5rRhNrgaJpZM4LhIH->
> .
>

--
*Best Regards*
*Hamid **Mosaddeghi*

*PhD of Physical Chemistry
*
*Department of Chemistry*

*Isfahan University of Technology, Iran*

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<https://github.com/keeeto/DLPOLY_EAM/commit/5f63fdc4e557c5945cc6b765b34fbb55fc875ccd#commitcomment-20451160>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ADhr5Ktd8bSwpfevtFeEsBiQc-Zlr9Poks5rRhWcgaJpZM4LhIH->.


Beyond1Airloop (mjwitte) - i386-Windows-7-VisualStudio-14: OK (2298 of 2356 tests passed, 434 test warnings)

 * 434 tests had: BND diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.png) ![Test Badge](http://img.shields.io/badge/tests%20passed-97.54%25-yellow.png)


Inicio


Thanks for the explanation. I will use only install/updates for new entries and ACIs.


ini harusnya ada didalem if


Right, I will remove this as it is redundant.


disini sessionnya belum distart


Agree.


hier nicht die SSID und Passwort vom shack-internen WLAN angeben. Der NodeMCU macht einen Access-Point auf, wenn WLAN nicht eingerichtet. SSID und Passwort wird als Parameter hier angegeben. Name sollte "WifiSettings" oder so sein.


不要 `to reduce app bundle size`，这个和 antd 不一样，antd 默认会引入全部的，antd-mobile 没有全部的可以引用，不存在降低大小。


ok


MDNS macht einen Node mit der Endung ".local" auf. Das wird vermutlich so nicht funktionieren.


*When you destroyed


You'll need the INLA repository too: https://www.math.ntnu.no/inla/R/stable


Hi There is a bu in this file. Line 2089 and 2090 state:
```
 mount_path = module.get_bin_path('mount')
        rc, mount_out, err = module.run_command(mount_path)
```
it must be:
```
 mount_path = self.module.get_bin_path('mount')
        rc, mount_out, err = self.module.run_command(mount_path)
```


wird so JSON geparst?


yeah. You are right, will be fixed with #253.


Bitte nur eine Build-Umgebung!


Nit: I think the contents of the `<style>` block should be indented.


function tick(){
const element=(

hello,world!

it is {new date().to local time string()}.

);
reactDom.render(
element
document.get element ById('root')
);
}
set interval(ticl,1000);


test


@retrue https://en.wikipedia.org/wiki/Particulates


fixed


Well spotted --- I've just fixed it.
G

*Gianluca Baio*

*Reader in Statistics & Health Economics*
Department of Statistical Science
University College London
1-19 Torrington Place, London WC1E 6BT, UK
Telephone +44(0)20 7679 1248 (internal: 41248)
Fax +44 (0)20 7383 4703
Website: http://www.homepages.ucl.ac.uk/~ucakgba
<http://www.homepages.ucl.ac.uk/~ucakgba/>
Blog: http://gianlubaio.blogspot.co.uk

On 12 January 2017 at 12:36, Peter Konings <notifications@github.com> wrote:

> You'll need the INLA repository too: https://www.math.ntnu.no/inla/
> R/stable
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/giabaio/survHE/commit/df504a7981c397aba129790b890d5b5093688868#commitcomment-20451513>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AIUayWSfLxPnYyeG9nTbzt8uETp5Suzpks5rRh43gaJpZM4Lhs_f>
> .
>


Thanks. I will add the user_not_found in the next commit


It means if the user is deleted, the relationship that holds the user also should delete. Am I right.? If not I need an explanation. :)


31 test fails... Will continue today in the evening.


You will laugh :D the spelldata isn't broken.. they implemented a proc spell for demon blades and have one for all other spells hahahahahha

"ID - 236167 Felblade" is the one with 4.5 PPM
 "ID - 203557 Felblade" is the one with 3 PPM


@swaathi
 I though include? will return only boolean value. I didnt know that it will load all of them. Thanks. I'll do it accordingly next time


Nein. Das ist erstmal nur n Test. Alles noch nicht fertig ;)


```bash
if (( $hour >= $i )); then
```
else you will only have the wallpaper 11-Mid-Night


issue добавить? или вы добавите в patch_1.0.3_to_2.0.0.sql?


![](https://github.trello.services/images/mini-trello-icon.png) [Дописати методи для проходження тесту роботи калькулятора](https://trello.com/c/Ih8eWG2E/7--)


Jup da spackte platformio rum


Yes perfect. :)

_
Swaathi K
CTO, Skcript Technologies Pvt. Ltd. <http://www.skcript.com/>
@imswaathik <http://swaathi.com/>

> On Jan 12, 2017, at 6:14 PM, Nav_devl <notifications@github.com> wrote:
>
> It means if the user is deleted, the relationship that holds the user also should delete. Am I right.? If not I need an explanation. :)
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub <https://github.com/Naveendurai/ROR-Day-3/commit/fe5a71e0d1577df0ca2cef93dd15108479e52244#commitcomment-20451599>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFS14-OeMkAcF6N7GKH0cJWivbN7Mag5ks5rRiAVgaJpZM4Lhsa5>.
>


![](https://github.trello.services/images/mini-trello-icon.png) [Написати свій тест для перевірки правильності роботи завдання прикріпленого](https://trello.com/c/cQ47LFtS/8--)


Yes it does return boolean. But it will first load all the values and then check. This will be done in memory. But the SQL query will be done within SQL itself. So it's faster. :)


Could just be a coincidence.


eides ti mpinia paizei to auto complete tou visual studio..to prwto p bgazei einai private kai patas to enter kai mperdeuesai nomizwntas pws s bgale public :P


@ymusiychuk-lohika fix merged to master


You don't need to name these using hyphens -- `:onSaveGrammar="onSaveGrammar"` works. Though I'd consider naming them slightly differently here, e.g. `:onSaveGrammar="saveGrammar"`.


@yairchu ddc442c should be reverted and merge newer fix to avoid future conflicts:
https://github.com/julianstorer/JUCE/commit/f2b2fb281992584ca358e845d33c3c47d177cb27


See issue #51


Okay @swaathi. I'll do it in correct way from now.


Thanks


Not sure how to implement the custom rules, but thanks for the code guys!


Yes, I will add it to the devel meeting agenda as I think this is something that should be solved not only for certmapping feature.

JFYI the original purpose of staging area for users was to be place where external tool (over LDAP) can precreate and amend the user entry before it's is actually activated. It was meant for use in bigger corporations where the user account is created by some HR tool after the interview process and before the user actually starts working. To suffice this requirement 'stageuser-activate' would be the only needed command.
Then broader API was added to offer this 'playground' also to FreeIPA administrators. Now the stageuser is somewhere in between and we should decide whether we want to add all (but password related) commands or just the very few basic ones.


The distro is provided by meta-ossystems-base. It is referenced in the platform:

https://github.com/MarvellEmbeddedProcessors/marvell-embedded-linux-platform/blob/krogoth/default.xml#L18

Are you trying to build it without using the platform?


I guess this was accidental, reinstated in c324386a2603824a4c0a7efac40108231d0a142f


modificacion de algooooo


well, it seems I managed to screw something in NB so it shows wrong author of commit...


I don't understand. I did indexed both of them. Is that wrong. ?


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/39850/testReport/) for this commit**


That was the reason why png rendering didn't work. The title needs to stay a string:

    can't convert Symbol into string


Sorry to keep on bothering you, but there is still something off. This will install the source package (requiring Rtools), not the binary. Is the directory structure on your server correct?


FuelCell (brianlball) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2305 of 2511 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.png) ![Test Badge](http://img.shields.io/badge/tests%20passed-91.8%25-yellow.png)


https://docs.travis-ci.com/user/languages/java/

`The Travis CI environment provides Oracle JDK 7 (default), Oracle JDK 8, OpenJDK 6, OpenJDK 7, Gradle 2.0, Maven 3.2 and Ant 1.8, and has sensible defaults for projects that use Gradle, Maven or Ant.`

So openjdk8 is not available


Hi @olivergierke, sorry for bothering you, but you use @Data annotation for your Customer entity and this is an issue, cause in your case it's ok there are no entity-relationships (eg. @OneToMany, @OneToOne and so on). As you know @Data = @Getter + @Setter + @RequiredArgsConstructor + @ToString + **@EqualsAndHashCode**. It hides @EqualsAndHashCode, I think this is a tricky stuff. In this case equals and hashCode will be based on all fields, it's a disaster for LAZY fields and bidirectional associations, many sql queries will be triggered as soon as you invoke equals or hashCode, it's all for performance sake. In case of bidirectional association you will catch StackOverflowError. As you are an experienced engineer and you are an example for many developers I don't recommend you to use it. I saw many times how developers used this annotation and what happened after. I am sure that many developers look at your example right now and they found your mapping like an example. Thank you.


I don't think you really want these under `data`, they should just be methods. Data is for reactive properties of the component. E.g., imagine you had a property `errorMessage` which would be shown if the login fails for some reason. Then in `onLogin`, you might do `this.errorMessage = "Login failed"`. Then the view will be marked as dirty and automatically re-rendered in the next rendering cycle.


Yes, and wheezy is old, you should upgrade to jessie


mmm... I tested it on a Windows installation and it has worked OK. Let me
remove all the dependencies and try again on the vanilla install...

*Gianluca Baio*

*Reader in Statistics & Health Economics*
Department of Statistical Science
University College London
1-19 Torrington Place, London WC1E 6BT, UK
Telephone +44(0)20 7679 1248 (internal: 41248)
Fax +44 (0)20 7383 4703
Website: http://www.homepages.ucl.ac.uk/~ucakgba
<http://www.homepages.ucl.ac.uk/~ucakgba/>
Blog: http://gianlubaio.blogspot.co.uk

On 12 January 2017 at 12:56, Peter Konings <notifications@github.com> wrote:

> Sorry to keep on bothering you, but there is still something off. This
> will install the source package (requiring Rtools), not the binary. Is the
> directory structure on your server correct?
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/giabaio/survHE/commit/5969685091f9fc887732b83b9d5cd647304adb4c#commitcomment-20451733>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AIUaySklVdCBHmNYl2Re1caTQTUxF5YAks5rRiLsgaJpZM4LhuEH>
> .
>


For reference: https://xamarinhq.slack.com/archives/runtime-bcl-tools/p1481747244000135. Do you think the extension method on IAssemblyResolver would make sense for us?


it is not mandatory at all , you can keep using our binary versions like others do


Also you can upgrade your python installation to any new build


left.jap..........


LINT, referenced color was private not-exported from library (appcompat probably), and for that reason, I cannot rely on it being available further on or that it won't change.


OK I'll add the needed information and fix the links. Thanks!


Could you give me some example of such difficulties? If you need different redirect urls you just define them in the branches and redirect at the end from the same place. This is not mistake actually but something that helps to work few developers together.


hola igor te jodo un poquito el codigo ok??


It actually works, but requires a compiler. Try this to make sure it works when the user does not have Rtools:
```R
> install.packages("survHE",repos=c("http://www.statistica.it/gianluca/R","https://cran.rstudio.org"),dependencies=TRUE, type = "binary")
Installing package into ‘C:/Users/UserName/Documents/R/win-library/3.3’
(as ‘lib’ is unspecified)

   package ‘survHE’ is available as a source package but not as a binary

Warning in install.packages :
  package ‘survHE’ is not available (as a binary package for R version 3.3.2)
```


We could discuss this on Gitter. Firstly, I'd like for you to explain to me how it works, so I have a better understanding of it. And also, if you can reproduce #170's crash, that would be great.


FuelCell (brianlball) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.png)


Efectivamente, fue una customización pero la habías mandado al chico y la había aceptado aquí: https://github.com/jbroadway/urlify/pull/42


I don't know why you do it, you could tell me as you send user id with requests to the server.


in this case the second function don't be called until first function call. So for my case does not need to set global.


Ended up not needing it. I thought I'd removed all references to it, but I guess not. I'll remove it properly next time I check in.


+1 looks good IMO


I will point out a small problem in the current MIDI export.  The exporter uses the default ticks per quarter note value of 120 of the midifile library for all exported files.  For casual listening, this is not a problem as it would be difficult to hear cases where the round-off errors are audible.  But it would probably be good to avoid these round-off errors in the timemap data.

Consider the test piece I was using above:

<img width="1010" alt="screen shot 2017-01-12 at 00 27 28" src="https://cloud.githubusercontent.com/assets/3487289/21889814/a40feb0c-d87f-11e6-90d3-305620f2f1c7.png">

This has a smallest time unit of 1/21st of a quarter note.  From this time unit all durations/rhythms can be expressed as integers.  So technically the ticks-per-quarter-note field in the MIDI file should be based on a multiple of 21 in order to avoid round-off errors.  My qstamp timemap implementation does not have any round-off error problems because I am using rational numbers for the score-times.  This is equivalent to calculating the correct ticks-per-quarter value for the MIDI file, but there is no need to do a complicated rhythmic analysis to figure out proper value.  And rounding error is limited to +/- 0.5 of the tpq value when writing to a MIDI file if using a non-optimal tpq.

Here is the timemap generated from the MIDI file data (qstamp = ticks/120):

```
tick    qstamp  milliseconds
0	0.000	0
68	0.567	340
120	1.000	600
137	1.142	685
205	1.708	1025
240	2.000	1200
274	2.283	1370
308	2.567	1540
342	2.850	1710
360	3.000	1800
411	3.425	2055
479	3.992	2395
480	4.000	2400
520	4.333	2600
560	4.667	2800
600	5.000	3000
840	7.000	4200
960	8.000	4800
```

Compare to the same data without round-off error due to the 120 tpq:

```
qstamp  milliseconds
0.000	0
0.571	343
1.000	600
1.143	686
1.714	1029
2.000	1200
2.286	1371
2.571	1543
2.857	1714
3.000	1800
3.429	2057
4.000	2400
4.000	2400
4.333	2600
4.667	2800
5.000	3000
7.000	4200
8.000	4800
```

And here are the time differences in milliseconds (the round-off error):

```
A       B      difference (milliseconds)
0	0	0
343	340	3
600	600	0
686	685	1
1029	1025	4
1200	1200	0
1371	1370	1
1543	1540	3
1714	1710	4
1800	1800	0
2057	2055	2
2400	2395	5
2400	2400	0
2600	2600	0
2800	2800	0
3000	3000	0
4200	4200	0
4800	4800	0
```
A = exact time according to rhythms and tempo, B = approximation caused by tqp=120 quantization.

There was one case where a time entry bifurcated from 2400 into 2400 and 2395.


@pvomacka has noticed that 'ipaenabledflag' is Bool instead of Flag in other modules. Is there any reason to break this convention?


OK tnx, good to know. I should upgrade indeed, probably best to start with a clean install.. (which is why I'm still running Wheezy)


nice, now Travis fails this because of outdated X11


added


For developing, it's easier to use tabs because pressing tab 1x is faster than pressing space 2x.  Also using tabs is what most programmers do, and have their editors configurated to.
However it's your software, so you should decide.


Wow such a bad name


been running this since weeks without a single deadlock.

kernel log, please.


ok tumbler


Awesome, thanks!


purge*


My comment is not in right place, sorry it needed to be under another your comment: "Here and on the line 21 you set variables which used only on the next line. So they do not have sense and you can set values directly to the $userId and $data['comments'] accordingly." :)


A ver .. varias cosas:

- El nombre está mal $treat_spaces_as_hyphens (camelCase)
- URLify::filter no tiene 8 parámetros, tan solo 7. En este caso existen 8.
- Si hubo un error al poner el parámetro 7 por 8, entonces se debe usar la misma nomenclatura $treatUnderscoreAsSpace en lugar de "treat_spaces_as_hyphens" para evitar confusiones.


@situatedbit With PR #106 merged, this commit seems fine (if you have other related commits, include them), but to be more complete we need to add query filtering depending on services that are visible for client doing the request, or *all* for service professional, or new API endpoints (we chat about *by clientID, any job title*). Would be good to start a spin-off issue from #4 to chat about specifics of this (REST API).


Insert on Line 13

var/delay = *insert time in deciseconds here*

Insert on line 85

if(last_act + delay > world.time)
  return

That should do for spam protection


Fix for windows (#36)


You shouldn't mutate `props`, they are passed in from the parent (or instantiating code) and shouldn't change. Use `data` for properties that can change.

Vue gives a warning about this:

![screen shot 2017-01-12 at 2 09 16 pm](https://cloud.githubusercontent.com/assets/17293/21891023/19da41c4-d8d1-11e6-9567-3e0f2deb880d.png)


This is not printing a new line at the end.


Fixes issue #10


лучше тикет


The note in the commit message should read:

After 8cd6acf there are no operators which use ProductSet-Reader but the code is left in case it is required in the future. Also operators which use ProductSet-Reader will not currently work in GPF models.


claro que siiiiii


Cool, I learned something new! I haven't used `v-model` before.


@xus: We should only fix this in the trunk for now, since this is risky business.

Please try to write a migration that:
loops over all assignments with peer reviews i
loops over all users that received feedback j
loops over all peers that gave feedback k
now for all files that start with .../peer_up_i/j/kfilename:
copy(!) these files to subdirectory .../peer_up_i/j/k/filename
if the assignments uses criteria
loop over all criteria m
now for all files in .../peer_up_i/j/km/:
copy(!) them to subdirectory .../peer_up_i/j/k/m/


This makes it explicitly true that daqprocessor_single_calibrate.py is the currently used script to call daqana, thereby resolving issue #8.
We may like to rename it to something shorted (daqprocessor.py?) in the future but I have not done so for now in the interested of not being confusing.


If `isMultiple = true` and `selectValue = null` an an error occurs


Ah, that part is confusing indeed. On failure the function doesn't return normally any more. It with throw an exception. I have now adjusted the comment to remove the obsolete part and pushed the commit.


Flag inherits from Bool but sets autofill=True with a default value, I thought it was better to have a default setting to enable the rule.


Delete


Uh, this is specific to your home layout.


I just wonder why you sent username and user ID in the parameters as they could be taken from the cookies / session.


What about this your mention, I understand it can be written by this way:
`public function saveComment(){
        $itemId = '';
        if($this->input->post('submit') && $this->input->post('description') && $this->input->post('item-id')){
            $description = $this->input->post('description');
            $itemId      = $this->input->post('item-id');

            $this->load->model('Comments');
            $this->Comments->insertComment($description,$this->input->cookie('user', TRUE), $itemId);
        }elseif($this->input->post('item-id')){
            $itemId      = $this->input->post('item-id');

            $this->load->library('session');
            $this->session->set_flashdata('empty', 'Comment can\'t be empty');
        }

        redirect(base_url("comments/" . $itemId));
    }`


Moving NaiveBayes plugin to hydrator-org.


Yep, I know


`public function saveComment(){
        $itemId = '';
        if($this->input->post('submit') && $this->input->post('description') && $this->input->post('item-id')){
            $description = $this->input->post('description');
            $itemId      = $this->input->post('item-id');

            $this->load->model('Comments');
            $this->Comments->insertComment($description,$this->input->cookie('user', TRUE), $itemId);
        }elseif($this->input->post('item-id')){
            $itemId      = $this->input->post('item-id');

            $this->load->library('session');
            $this->session->set_flashdata('empty', 'Comment can\'t be empty');
        }

        redirect(base_url("comments/" . $itemId));
    }`


@pgathogo @wondie , this commit contains an option for enabling or disabling logger debug information. By default, the option will now be disabled but the user can choose to enable it through the Options dialog. We will encourage users to have it remain disabled unless, under special circumstances, they need to enable it.


`
public function saveComment(){
        $itemId = '';
        if($this->input->post('submit') && $this->input->post('description') && $this->input->post('item-id')){
            $description = $this->input->post('description');
            $itemId      = $this->input->post('item-id');

            $this->load->model('Comments');
            $this->Comments->insertComment($description,$this->input->cookie('user', TRUE), $itemId);
        }elseif($this->input->post('item-id')){
            $itemId      = $this->input->post('item-id');

            $this->load->library('session');
            $this->session->set_flashdata('empty', 'Comment can\'t be empty');
        }

        redirect(base_url("comments/" . $itemId));
    }
`


Looks, like here is an issue: #639


public function saveComment(){
        $itemId = '';
        if($this->input->post('submit') && $this->input->post('description') && $this->input->post('item-id')){
            $description = $this->input->post('description');
            $itemId      = $this->input->post('item-id');

            $this->load->model('Comments');
            $this->Comments->insertComment($description,$this->input->cookie('user', TRUE), $itemId);
        }elseif($this->input->post('item-id')){
            $itemId      = $this->input->post('item-id');

            $this->load->library('session');
            $this->session->set_flashdata('empty', 'Comment can\'t be empty');
        }

        redirect(base_url("comments/" . $itemId));
    }


This version gives me temperature updates (though not properly calibrated to the room) every 2 hours.
I'm getting duplicate sun data again in this version; it's only one duplicate per test (they look to be exact duplicates), so it's still not as bad as it was.


Moving Decision Tree Regression plugins to hydrator-org.
The repo contains the base classes for spark ml plugins i.e, SparkMLTrainer and SparkMLPredictor since moving them to hydrator-common, the spark ml plugins are not able to maintain the state and throws ClassClastException.
Raised JIRA https://issues.cask.co/browse/HYDRATOR-1258 to move the base classes to hydrator-common.


Moving Logistic Regression plugins to hydrator-org.
The repo contains the base classes for spark ml plugins i.e, SparkMLTrainer and SparkMLPredictor since moving them to hydrator-common, the spark ml plugins are not able to maintain the state and throws ClassClastException.
Raised JIRA https://issues.cask.co/browse/HYDRATOR-1258 to move the base classes to hydrator-common.


only comment some Lines


kalau mau redirect gak usah print text


disini tambahin exit(); soalnya kalau pake header location tapi ada textnya, sama browser gak diredirect


test


should be covered by a test?


Twig 2 does not support PHP 5.6


;) also use the 'sudo ...' command to install domoticz (see our main website/download page)


thank you :-)


커밋 실수
기여자 명시 메세지 추가


This looks a lot better, thank you @Nedelosk !


Is this really helpful?


Well, formally yes. But good developer characterised from the rest in that he thinks about his code usage in the real world. This is quite often functionality in the web applications, so developers who worked with this just can't ignore possible CRUD actions with data. It was one from the hidden specifics of test task.


You don't need global variables anymore.


The `120` is hardcoded in Verovio. We definitely should raise the default value to at least `256` and change it to a variable.

To implement reading the MEI `scoreDef@ppq` is an easy task. But a nice addition (as not every MEI file will have the ppq set) would be to hand it over via command line.


This is to have some Java 8 features in the source code so that we can verify that we can build it on our BF.
It is not helpful, it is not to be merged, it simply doesn't make sense... But it serves the purpose: you need to have Java 8 to compile and run it.

@matzew sorry, perhaps you and I have different things in our minds and I cannot understand what you mean.


We could fall back to prctl(PR_SET_NAME, ...) assuming it is available.


Yes, your point is acceptable for me, but I as I've already told you did this as the task is test one.


I am not expecting failure for this fix. There are 57 test cases that show failure and are related to processing tabular diffs [KeyError](https://nrel.github.io/EnergyPlusBuildResults/EnergyPlus-0ac693fbd73a638bec30c136f4d1e97508a54aed-Win64-Windows-7-VisualStudio-14.html). Otherwise, this is a simple fix, ready for review and merging. @mjwitte @Myoldmopar ?


По поводу оператора < так как там не уточнялись детали реализации я сделал
так что вектор меньше другого когда все его компоненты меньше компонент
другого вектора. Так же само с сортировкой в другом задании. Я считал
вектор отсортированным,  когда каждая последующая компонента больше
предыдущей

среда, 11 января 2017 г. пользователь Dmitriy написал:
> Not correct. This leads to double free issue:
>
> Vector<int> v1(5);
> Vector<int> v2(v1);
> v2 = v1;
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub, or mute the thread.<
https://ci4.googleusercontent.com/proxy/_hvj219gVk1pFnl6gsUACZkr0m6hlVBowYyUO_qqsiypxTro1_PWcQxM5oQgZVDEKccp8S_D6zz6e5dpA64CK8Lfd8tLQNduER-9qqYsWf-x4hJ8Cu7IhxRsCrsl5QdXC6Xf8tbPO469vMh5oPyeBxUAlPPtlg=s0-d-e1-ft#https://github.com/notifications/beacon/AWQZ7lObbiAvLhZoDHr4Cepn07J9OWksks5rRGzUgaJpZM4LgNxq.gif>


Well I don't get working this thing about the SHA-1. Do you have a working example?


There's a simpler way to do this, but it's totally not obvious. And things are more complicated than they should be due to the fact that we haven't yet converted the main layout to Vue. The simpler way is this:

```js
var Prompt = require('./components/prompt.vue');
...
var prompt = new Prompt({
  el: '#promptScreen',
  propsData: {
    onSaveGrammar: function(description) {
      ...
    },
    onLogin: function(username, password) {
      ...
    }
  }
});
```

This just instantiates an instance of the prompt component, while the code you have here creates a new component instance with the prompt as its only child. So it's just a bit of unnecessary indirection.

Then, if you move the `show` method into prompt.vue, you should be able to use `prompt.show` exactly like you are currently.

Hope that makes sense and that I didn't get anything wrong, but let me know if you run into trouble.

(Here's a JSFiddle I used to test the general approach: http://jsfiddle.net/bgosm7cr/1/)


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/39856/testReport/) for this commit**


And are we just missing #define _GNU_SOURCE? http://git.musl-libc.org/cgit/musl/tree/include/pthread.h#n217


i'll disagree on this one, especially if we need to rely on debian packaging to install it.
normaly they'd go to a data/ directory, the deamons would go to src ... but as this is python everything is weird.

anyway, we're going to need to change all those service files in sync() (as they are 90% identical) so i'd much rather have them all side by side. then avahi udev and systemd service files aren't dramatically different, so really it seems to make more sense to have them in debian/.

one argument to have them anyother place would be if setuptools was made to handle them and debian packaging would just find them in the package, but until then...


@vmuriart it was probably just because I didn't need it at the time, and it was broken. If it's just the reference then I agree we should add it back.


> I was suggesting supplementing the existing relations, not replacing any of them. Sorry, I should have been clearer.

why add ? NFD already depends on that.


Ok, it is only a few months old: http://git.musl-libc.org/cgit/musl/commit/include/pthread.h?id=8fb28b0b3e7a5e958fb844722a4b2ef9bc244af1


ótimo, criei uma pasta chamada caete_ger onde tem estes códigos que vc me mandou
sempre que rolar uma atualização da parte de vcs façam a mesma coisa... eu copio pra pasta certa e apago os arquivos daqui. um abraço


Can't agree, there is no cases when you can define global variable implicitly in some function legally. Such code quite hard to debug and maintain. One more thing - approach with designing methods in the way they can not be used separately and in any order is nightmare for the big projects, they are completely untestable.
Problem is you've considered test task as something paltry, but the is not the case. This is only opportunity to show yourself the best in the action.


Aren't the @RequiredArgsConstructor annotations superfluous?