Seems to work alright. Note that the Sonic Advance games do something weird (e.g. changing the language does not immediately save, but completing a level works just fine). Loading saves from other emulators should work just fine though.

This is for 64KB games only (no Pokemon saves, yet).


Literal


Great. That was the understanding I came to as well.


Got distracted mid-thought?


Yep. From my point of view, when you configure LDAP, first use it for user authentication and then, if any specific LDAP fields are available, configure how to extract GN privileges and in that case users may turn this to true. BTW, if you want it to be true by default, why not.


Yep, you're right. This fixes it: 1b0b235c5c35c7b8975ba50464163be0be639425


What tool do you use for the decompile? BON? I have not tried to decompile 1.4.7, only 1.6.x so far. You are right it was monster in 1.4.7.

I think we need to tweak the overall spawn rates a bit. I think they have changed the spawn a bit since 1.4.7. I have a MultiMC of the old DRPG to compare against.

I might do a pull request if I come up with some better numbers for 1.7.10. I have not seen a single Cori since your fix. I think they need to be made back to monsters and the spawn rate adjusted.

I also noticed that moonwolfs spawn out of control. I will look at all of the spawns and compare them against 1.4.7.


asdasdsa


Any reason we just don't default this to the current d/l URL?


@robmachado não é necessário usar o try/catch para fazer não explodir o teste, ou seja, se o teste é para ver se uma determinada exception é lançada, basta usar o docblock com a tag `@expectedException InvalidArgumentException`

segue outro exemplo que fiz: [tests/Common/Files/FilesFoldersTest.php line 49](https://github.com/robmachado/nfephp/blob/df186aef6dc7e80fbbf74520b8b3d5b8594499a2/tests/Common/Files/FilesFoldersTest.php#L49)


Hi guys @udeved and @manjaro this is my stance so for:

1. I will generate one last iso for 0.8.12 with manjaroiso (which I built today). After that I wont be involved with the isos.

2. I have deprecated the openrc-eudev repo based on @udeved's packages, and have removed the instructions from my blog. So I wont be involved with the packages as well.

3. In other words, this means that from now onwards I will be a mere user rather than a team member.

Thx for your support.
Yours sincerely,
Aaditya


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 80](http://192.168.44.2:8111/viewLog.html?buildId=239&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) is now running


It would be nice to add info about trial key to readme file, m?


JD-GUI and BON. You don't need to decompile 1.6, it can be found at github.com/theslayermc/divine-rpg


var Widget = {
    foo: ""
}


Shouldn't this line be changed to:
```
setLocale(cookieStore ? cookieStore.get(...) : ($window.... || ... || localeConf.defaultLocale));
```

With the current implementation, on new clients that do not have the cookieStore, the implementation will always use the defaultLocale, ignoring the browser detection.


I forgot to push views.py to the repo. Sorry!

On Mon, Jan 26, 2015 at 11:15 AM, Vladimir Sudilovsky <
notifications@github.com> wrote:

> I guess you're right about the JSONP -- comes down to a style/design
> choice.
>
> re the test:
>
> vagrant@vagrant-ubuntu-trusty-64:~/export_service$ git log -n1
> commit 1b4b9df5793b449fe7e517bf2630e1991a7fff31
> Author: Vladimir Sudilovsky <vsudilovsky@gmail.com>
> Date:   Mon Jan 26 16:00:28 2015 +0000
>     tests: add travis
> vagrant@vagrant-ubuntu-trusty-64:~/export_service$ python tests/run_tests.py
> test_AastextRoute (unittests.testExportService.TestExports) ... ok
> test_BibtexRoute (unittests.testExportService.TestExports) ... ok
> test_EndnoteRoute (unittests.testExportService.TestExports) ... ok
> test_emptypayload (unittests.testExportService.TestExports) ... ok
> test_jsonp (unittests.testExportService.TestExports) ... FAIL
> test_ResourcesRoute (unittests.testWebservices.TestWebservices)
> Tests for the existence of a /resources route, and that it returns properly formatted JSON data ... ok
>
> ======================================================================
> FAIL: test_jsonp (unittests.testExportService.TestExports)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
>   File "/home/vagrant/export_service/tests/unittests/testExportService.py", line 115, in test_jsonp
>     self.assertEqual(r.json, u"JSONP_CALLBACK({'msg': u'Retrieved 1 abstracts, starting with number 1.  Total number selected: 1.', 'export': u'aastex response'});")
> AssertionError: {u'status': 500, u'message': u'Internal Server Error'} != u"JSONP_CALLBACK({'msg': u'Retrieved 1 ab[89 chars]'});"
>
> ----------------------------------------------------------------------
> Ran 6 tests in 0.131s
>
> FAILED (failures=1)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/adsabs/export_service/commit/737f4504cbac45f190aad1cd719a4ae33adf8194#commitcomment-9442905>
> .
>


Converting bytes to String can fail if bytes is not valid UTF-8 data. Which might happen if there are bugs elsewhere.

More robust (if more complicated) to use CharsetDecoder.


Sorry.

Correct commit note:
- Character title (player, mechanic, manager, constable) translated outside /libs.via https://github.com/pioneerspacesim/pioneer/pull/2521/commits
- Translated string do not interfere when you change the game language. via #2749


Hey, Lukas - thanks for your report!  This definitely worked as a way to get our attention.

I'm personally on 14.04, and while there are a variety of OS versions, I think that's the most common OS for rippled developers.  I just did a clean build and :-( I didn't see this issue, and I suspect that our other developers aren't seeing this either or they'd be telling us.

So I don't have a great solution for you, unfortunately.  No one wants to hear "works for us" but, well, that's what's happening.  Have you done an `apt-get update && apt-get upgrade` recently?  Rebooted?  I know that's not much of a solution...


First delete all *.pyc file by #git rm command.
Then create a '.gitignore' hidden file, there you can specify the extension of files which you don't want to commit.
Then commit the '.gitignore' hidden file

Example https://github.com/openstack/horizon/blob/master/.gitignore

*.pyc
*~
*.sw?


a  i2c_acquire too much? (L107)


@lukapusic There's a simple fix - you need more RAM in order to build.


dwLength: either use hungarian or don't


Right, I did not realize!

What if there's an `EXDATE` with more than one property?


So if we turned dots to underscore, added a leading underscore for leading digits, and let services collide env names in a predictable order, I think we could relax restrictions a bit for services at minor cost for edge cases.

For services an interesting question is whether allowing dots is worth it (or for any resource) - I may be using the wrong validation, I thought I was using the "no dots" version, because that's what I remembered us agreeing on

That being said, I'm now definitely leaning to letting names vary more across types which are less likely to manifest as real resources (services, pods are "real", but higher level concepts are not).  Code that needs to generate names should use server or LCD (we can define the LCD as a-z0-9 no edge dashes) to do so - code that needs to prepend names can use LCD rules.  Are all resources names created alike, or does the problem domain (builds, users, tokens, mappings) start to change the name rules?

> On Jan 26, 2015, at 12:13 AM, Tim Hockin <notifications@github.com> wrote:
>
> Also should probably bake up some test for DNS that ensures services with dots works properly.
>
> —
> Reply to this email directly or view it on GitHub.
>


Spoke too soon. It obviously doesn't work if `PAGER` is not set.

To give you an insight on how it *currently* works, this is what `ps` says after running `man test`, etc.:

    man test
    sh -c /usr/bin/more -s /tmp/man.XXXXXXXXXX
    /usr/bin/more -s /tmp/man.XXXXXXXX

`man` seems to call `more -s` when `PAGER` is not set - this is of course for system manuals. For individual files `mandoc` is the recommended way. To sum it up, could you make sure to check whether `PAGER` is set or do it some other way, please? I'm more than happy to rest it :^)


sounds good


Maybe include this helper class to improve SecureRandom ?

http://android-developers.blogspot.com/2013/08/some-securerandom-thoughts.html


Yup, this should work. Thank you.

Do you mind giving [ba9f95](https://github.com/MetaScale/nt2/commit/ba9f95c85d9b2ee022be966b4063110ea2035030#commitcomment-9340203) another look? This quirk complicates using `nt2` via `ExternalProject`.


It is better to do something like server_config.getboolean("debug") when testing for a boolean. See the ConfigParser doc ;)


stop kyle


"bundle" is unclear.  jena-osgi-bundle?  Elsewhere we use "apache-jena*" for the distinguished delivery modules (the distribution and the stable POM root for the libraries currently).  "apache-jena-osgi" or "apache-jena-osgi-bundle".


Overall seems good. Please continue!


I have two things to point out on this task/commit:

1. If you remember we talked that the configuration must be duplicated for all 4 configuration files and that you can create separate common files and include them.
2. Why did the jquery script file name changed from jquery-1.9.1.min.js to jquery-1.9111.1.min.js?


I don't think this should include SDB. (1) we don't encourage it's geenral use (2) aren't there JDBC driver loading issues?


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 80](http://192.168.44.2:8111/viewLog.html?buildId=239&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:08:03


TeamCity Ybackend :: zmactep/ybackend [Build 54](http://192.168.30.225:8111/viewLog.html?buildId=670&buildTypeId=Ybackend_ZmactepYbackend&guest=1) is now running


Yep, you are right it could be better. Maybe /virtualboxes?


This fix is unrelated to the rest of the changes, but I realized we were re-opening this file for no reason.


TeamCity Ybackend :: zmactep/ybackend [Build 54](http://192.168.30.225:8111/viewLog.html?buildId=670&buildTypeId=Ybackend_ZmactepYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:15


@VicDeo Would be great if you could help……


TeamCity Ybackend :: Feodorov/ybackend [Build 84](http://192.168.30.225:8111/viewLog.html?buildId=671&buildTypeId=Ybackend_FeodorovYbackend&guest=1) is now running


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 155](http://192.168.30.225:8111/viewLog.html?buildId=672&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 155](http://192.168.30.225:8111/viewLog.html?buildId=672&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 127 Build time: 00:00:01


I think this is correct - grab the latest 29 .grb files and leave the .tbl file which will sort to last - but it's not robust if they start adding an extra random file or one of the .grb's gets lost or something. Might want to do a regex but maybe I'm being overly paranoid.


I'm just committing something


TeamCity Ybackend :: Feodorov/ybackend [Build 84](http://192.168.30.225:8111/viewLog.html?buildId=671&buildTypeId=Ybackend_FeodorovYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:15


@zag2me afaik tvh has no automatic db upgrade script to the new "db" layout. If you wont stuck forever (yea forever) at 3.4 you have to complete reconfigure tvh or use the [migrate script](https://github.com/tvheadend/tvheadend/blob/master/support/conf_migrate.py).
3.9.x works great with OE5.0 if you upgraded correctly.


I'm trying to have this send email when recipient is offline too.
Can someone please help me to check here for offline status as well?


Yeah, it safe to remove @zevarito


aka "It would be great if you could apply those patches and merge to all branches" ;-)


We don't quite want to return here, we still need to add the Suite to the database. Each build job has many different test jobs associated with it. So the rough pseudo code we need is:

    if build not in database:
        # this is the first test job associated with that build
        put build in database
    do processing
    put suite in database

Also the `buildid` alone should be a unique key you can use when checking if the build already exists.


That's weird... I don't get that.
Can you try the same with the example project of android-store ?


@CvH that's not true, the migration script has been redundant for a long time, we did actually start to prepare for a 4.0 release and then I simply got overloaded at work and haven't touched TVH in months.

The code will self upgrade 95%+ of your config, there is some minor fettling required to re-configure the tuners, but that's mostly limited to "this tuner is connected to that network", all the network, mux, service, channel config "should" be auto-migrated.


I tested this on iOS 8.1 (whatever version of Safari that uses) and Chromium Version 39.0.2171.65.

Both worked by tapping/clicking. No extra handler should be necessary, but testing this on other browsers can't hurt.

You're welcome, Vinny


Add a "..versionadded:: foo" here.


Why are you interpolating `$base-font-weight` there?


I thought this line needed to change. Your addition below means at this point we'll only have the 29 .grb files from a certain hour, which won't necessarily be 09. So you could do `if re.match(r'^sref_xwwd_us_[0-9]{10}f[0-9]{2}.grb$', filename):` and that should be safe. But if you hard-code 09 in it will break 3/4 of the time, right?


@SubJunk Is there no other way how to not defer to MEncoder? Using 8.3 equivalent (Windows), escaping etc.?
I ask because UMS started to use h264 instead of mpeg2 by default so my old quad core is ok with mpeg2 and slower MEncoder but not with h264 (ffmpeg is better in realtime transcoding with h264 on my PC)
I have a lot of 1080p files so transcoding to h264 with MEncoder is almost impossible in realtime.
Thx!


fgdfgsfgsgfsfg


# com


Our concatenation order should ensure that the component handler is first in order prior to any of the other components being used. If not I would expect this to fail.


Great! Looks like some debug stuff in there still?


Yep good idea

On Mon, Jan 26, 2015 at 5:28 PM, Jeremy Grossmann <notifications@github.com>
wrote:

> Yep, you are right it could be better. Maybe /virtualboxes?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GNS3/gns3-server/commit/ff63530f521ad77d9d7b9bc8a56e9735e90df9dd#commitcomment-9443189>
> .
>

--

*JULIEN DUPONCHELLE*
Software Engineer

E: julien@gns3.com
LinkedIn <http://linkedin.com/in/julienduponchelle/>

<http://www.linkedin.com/in/stephenguppy>288, 200 – Rivercrest Drive SE
Calgary, AB, T2T 2X5
Twitter <http://www.twitter.com/gns3_official> | Facebook
<https://www.facebook.com/gns3official> | LinkedIn
<http://www.linkedin.com/company/gns3-net> | Google+
<https://plus.google.com/+Gns3Net/posts> | CrunchBase
<http://www.crunchbase.com/organization/gns3>


this my friend is why i asked you.. excellent job


find . -name "*.pyc"


FYI: I've edited the modman file to be compatible to the Hackathon installer in case you want to stick to that


uhmmm

Todo dia aprendemos mais um pouco  [?]

Em 26 de janeiro de 2015 14:19, Antonio Oertel Spinelli <
notifications@github.com> escreveu:

> @robmachado <https://github.com/robmachado> não é necessário usar o
> try/catch para fazer não explodir o teste, ou seja, se o teste é para ver
> se uma determinada exception é lançada, basta usar o docblock com a tag @expectedException
> InvalidArgumentException
>
> segue outro exemplo que fiz: tests/Common/Files/FilesFoldersTest.php line
> 49
> <https://github.com/robmachado/nfephp/blob/df186aef6dc7e80fbbf74520b8b3d5b8594499a2/tests/Common/Files/FilesFoldersTest.php#L49>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robmachado/nfephp/commit/dc6691534408fd425d2bd02de50b6bfb228e1c61#commitcomment-9442992>
> .
>

--
Roberto

*Nisi utile est quod facimus stulta est gloriae* (Julius Phaedous)


Why are we naming the download %NAME%.dmg ? It's not a valid dmg (yet). Why not just leave it "install_all_mac_pl_sgn.z"


Sorry, I've added message.count for this, but forgot to change here.


The word "source" itself might be at least as clear as "preferred" or "easily" and gets more to the point.
"The license may require copies or derivatives of a licensed work to be made available in source form."


@rec @vinniefalco Thank you both for your quick response and suggestions. More RAM solved the problem!


Looking at the implementation, let's do this:
`public void addSlice(FileSlice slice) throws InvalidStateException, IllegalArgumentException`

[Principle of Least Astonishment](http://c2.com/cgi/wiki?PrincipleOfLeastAstonishment)

Don't want to surprise API users with unexpected crashes.


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 81](http://192.168.44.2:8111/viewLog.html?buildId=241&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) is now running


Please update the documentation to show that this is now supported, then I'll merge in.


I'm on a CentOS 6.5 machine with GCC 4.4.7.


@adamsutton
at least for me (looks like I´m not alone) this wont work, but I did no real investigation about it because it was quicker to reconfigure.


@r4vi LGTY?


for Rhinoceros,Grasshopper


Yes it's fixed

On Mon, Jan 26, 2015 at 5:27 PM, Jeremy Grossmann <notifications@github.com>
wrote:

> It is better to do something like server_config.getboolean("debug") when
> testing for a boolean. See the ConfigParser doc ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GNS3/gns3-server/commit/1bfb201368f9bcc34e30ea906c8dda59582fc27e#commitcomment-9443159>
> .
>

--

*JULIEN DUPONCHELLE*
Software Engineer

E: julien@gns3.com
LinkedIn <http://linkedin.com/in/julienduponchelle/>

<http://www.linkedin.com/in/stephenguppy>288, 200 – Rivercrest Drive SE
Calgary, AB, T2T 2X5
Twitter <http://www.twitter.com/gns3_official> | Facebook
<https://www.facebook.com/gns3official> | LinkedIn
<http://www.linkedin.com/company/gns3-net> | Google+
<https://plus.google.com/+Gns3Net/posts> | CrunchBase
<http://www.crunchbase.com/organization/gns3>


Our experience with names internally was that people used more that 64 characters pretty regularly.  That said, labels will perhaps alleviate a lot of the need for that.  Are you concerned about dots for a reason or just "too loose"?

There's some value it names being the same all over.

I don't know what you mean by LCD rules here.


`... throws InvalidStateException`


Env isn't needed, but your tests should be running on PhantomCI :+1:


An `id` should only appear once per page. (`truncate` shows up four times!) Any JS that depends on this will be thrown for a loop.


Yes, It is weird. it's a hack around lack of polymorphism.

SeqAnnRegion is suppose to he the core object, used for any kind
of generic case.  Merges the BED 3 concept with more attributes.

SeqAnnGene, SeqAnnTranscript are conceptually polymorphic with
SeqRegion.  (SeqAnnGene is a SeqRegions).

SeqAnnData is never intended to used directly, it's the tool to
keep from having to copy and paste what it contains.  In
retrospect, it might actually be far better to just copy and
paste.

Daniel Zerbino <notifications@github.com> writes:
> Weird naming: SeqAnnRegion - points to -> SeqAnnData - points to -> Region
>
> Would it not be more intuitive to rename the record classes to fit this model:
> SeqAnnObject -> SeqAnnRegion -> Region?
>
> —
> Reply to this email directly or view it on GitHub.*
>


I will add it because we are in java land, but declaring exceptions in the signature and mandatory try-catch is a thing of the past


No problem, I already committed the fix, moreover added some corner cases to tests (like MaxValue and MaxValue + 1). Everything is ok now.


looks good for now but let's discuss more comprehensive solution for call_centre/provider etc when you're here


Then you might just change this to `WP101_Plugin::get_instance()`


This would correct any errors in the original string that are allowed by sqlToDate (e.g. out of range dates). No strong objections against removing it though.


`qboolean` was a typedef to `bool` anyway


I was thinking about it as well but then discarded the idea because I thought that settings these variables in CTKConfig.cmake could potentially overwrite the settings from projects including it. Thinking about it again, it is probably not an issue because this would only cause conflicts if e.g. different Python installations would be used for building CTK and later on using it which is not a good idea anyway.

So I could remove the `FIND_PACKAGE_PythonQt_VARS` label and have all variables set in `CTKConfig.cmake` without `CTK_` prefix. But I would still need the `FIND_PACKAGE` and `FIND_PACKAGE_VARS` labels for generating correct `find_dependency` calls.


yeah, that has occurred to me


Et ca


Personally, I agree with this change, but please note that all contributors must agree if the licensing terms are changed.


i guess it also has to do with using or not exceptions for logic flow.


@advancedxy And yes, we are using TeamCity CI server, but we can't run tests for every PR as it's potential security issue.


hey hey. please stop the unrelated discussion. we already told our users that we dont support upgrading from OE 4.2 to OE 5. such upgrade is lot more than tvheadend bumped... we consider "migration" as not-an-issue at all.


I am afraid your code did not get attached :(


Et ca ?
Scénarios : Ajout d'une fenêtre d'aide à la création de condition.


At least for this test jQuery is safe if `$('<input>', {value: items[i].name})` or `$('<input>').value(items[i].name)` is used.


Daniel Zerbino <notifications@github.com> writes:
> You can do this?  map<union {...}> Wow....

it's a pretty horrible way around not being able to say:

     record Fred implements Person {
     }


nörgler ;)


Please consider bumping the npm version. `npm install node-storm` gets 0.0.0, a previous commit.

I worked around it with `npm install --save RallySoftware/node-storm#b649d4765403ba42f7138a846ece3bf3189d878d`


We need a better name for this. And it needs to follow the "private" convention. How about `static int uv__pipe_getsockpeername` ?


cf. http://www.legifrance.gouv.fr/affichCodeArticle.do?idArticle=LEGIARTI000006750602&cidTexte=LEGITEXT000006073189
Le nombre d'heures travaillées n'a pas été mis à jour suite aux 35h pour cette formule


Don't see a visio file here ?


Personally, I agree with this change, but please note that all contributors must agree to a relicensing.


Also, maybe you can typedef this `int (*func)(int, struct sockaddr*, socklen_t*)`.


This change left me with this error:
`The ..controllers.myController name is not a valid component or interface name.`
discoverBeansInFolder function came with `mapping="./controllers"`, then the returned value from deduceDottedPath is

    CFCPATH  ./controllers
    DOTTED   ..controllers
    EXPPATH  C:/inetpub/wwwroot/app/controllers
    MAPPED   ./controllers

Reverting this change fixed it for me, but of course does not fix #283. I'm not sure what your strategy here is, but as an outsider, I can see a brute-force fix to add ` dotted = reReplace(dotted, "^\.+", "");` after line 338 to correct it.


Of course. I forgot we track field changes in this much detail.


As I said, a variable is already set for other font properties so I felt it made sense to set one here too.

Given that 300 isn't the default value for either body or heading font weights it isn't unreasonable to expect that a user might want to change it; now they can simply change the var rather than make the change in two places in base.scss


I'm in the process of removing all this anyway, freeze and checkFrozen is obiuosly a bad design.


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 81](http://192.168.44.2:8111/viewLog.html?buildId=241&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:08:17


Ah yes, forgot that step.


y


The ...


You can reflow "for a" to the previous line


very cool


ok. But I do think there are much less prs for inteliij-scala than spark or scala project. You don't have to run tests for every PR, just the PRs you have briefly looked and think It's ok to test. That's what spark do. For intellij-scala, the overhead for reviewing PRs isn't quite large.


It wasn't before?

Well, in any case, it'll be a great counter for Matter Disruptors or Lasers now. Which is probably bad news for the laser-heavy Guardian. :D


On the other hand `qwchar` is a place where things can really go wrong. Cause replacing it with `wchar_t` can brake things. And `wchar_t`size can differ from platform to platform. So it's probably better to keep it for now.


This goes in "deprecated features" not "backwards incompatible."


> On Jan 26, 2015, at 11:41 AM, Tim Hockin <notifications@github.com> wrote:
>
> Our experience with names internally was that people used more that 64 characters pretty regularly. That said, labels will perhaps alleviate a lot of the need for that. Are you concerned about dots for a reason or just "too loose"?
>
Agree, some of the name limits were originally chosen to match DNS (when we thought DNS would be on pods).  Now they're only on service (for now...).  Dots let it grow larger, at the cost of potentially complicating name assignment.  I guess I don't care either way.  Labels do absorb some of it.
> There's some value it names being the same all over.
>
And some in not - we only get to pick once unforutunately.  The concrete case we have right now is in OpenShift for builds (someone wanted to add a timestamp to the build object name, which includes a capital T and Z) and for users (where we want to ensure user identity is namespaced with a "source", so we added a separator character).  If a resource we represent via a "Kube like" resource ever has uppercase in its name, the name will have to have it, so it seems like eventually there will be "Kube convention" APIs that don't follow name (and we can't force them to).

> I don't know what you mean by LCD rules here.
>
Lowest common denominator - all names must support a-z0-9, so client code can prefix names with a common prefix if it doesn't know the type more specifically.
> —
> Reply to this email directly or view it on GitHub.
>


10K is not a lot when you start to consider things like massively sharded batch work.  Hell, I have jobs internally that use that many replicas and each one runs for just a minute or two.  It may be far enough away to ignore for now, but I *am* concerned that people come to depend on it being int-like, so maybe we should just break that aspect of it, and then we can later expand from 4 to 5 to 6 characters as needed.


Looks good to me. Unless @jdegoes has any comments, I'll merge.


retém


nós


To not break all legacy migrations, that somebody will want to run locally or on test/staging, I would consider having here additionally:

```ruby
def without_lock
  original_value = ActiveRecord::Base.mysql_online_migrations
  ActiveRecord::Base.mysql_online_migrations = true
  yield
  ActiveRecord::Base.mysql_online_migrations = original_value
end
```

(common code could be extracted)


I won't be able to accept this PR, since you've inserted a new config param, rather than appending to the end.  You have to keep in mind that this same order of params is sent to each platform (i.e.: iOS, WP, Android) -- if you shunt the params array in one platform, you must edit the params in each (even if those other platforms don't use that param).  Also, the order-of-args is transparent to the user, since the Javascript api uses Hashed-args -- it's the internal api which re-organizes the Hash to an Array of args.

eg:  here's ios https://github.com/christocracy/cordova-plugin-background-geolocation/blob/master/src/ios/CDVBackgroundGeoLocation.m#L106

If I merged your PR, 2 ios params would be broken (#activityType & #stopOnTerminate).

The reason for this mess was my poor choice to use Array-args rather than Hashed-args.  I'd like to refactor that.


I'm agnostic as to the exact terms used so long as it is clear that https://www.gnu.org/licenses/gpl-3.0.html is compliant, unless of course we want to take radical course of stating that the GPL is non-open. Note that the GPL defines source in terms of preferred form for making modifications. Whether that level of definition is needed within this term (of the Open Definition), don't know.


I would change "Match every" to "Match frequency" as with other fields we don't have "Starting at" and so on, let's make it more descriptive that way.


Where do we allow Status to be settable by users?

Are we converging on a position here?


ich hab doch nicht umsonst mein diplom in klugscheißen und besserwissen gemacht ;)


lol


'pi**nn**orman'?  Not 'pi**n**orman'?


Not exactly what you are looking for, but nnx has an extension to list non-container modules : https://github.com/clementfarabet/lua---nnx/blob/master/Module.lua
Maybe we should merge this into nn?
We will need it if we want to merge the nn.Recurrent Module from nnx to nn, which is also pretty stable.


If we're not using Bootstrap's panels, we don't need these divs.


If someone POSTs, then GETs, then POSTs again, do we want to have a lot of sympathy?  Why would they do that?


Is it duplicated on line #243 as same method?


I don't get this. For example we added your request for arch based folders in cache: 0e3e8b2906562ed759c206a2ff09f64841167d9a We only ment not break working code. Everything else was fine.


@liewegas you mentioned in IRC that this works but it doesn't look like it's been updated


If if so, why not just
```
 {
       "name": "",
       "randomNameBase": "my-prefix-"
 }
```

I'm not SURE it's better, but we also want to be moving away from structs for RestartPolicy


good to know


Yes, this looks like what I did, although I only made the change on line 192. It worked for me.


Nice catch! :+1:


Why ever we need a merge ...


:shipit:


jena-sdb was removed some time ago from my pull request - you are right the
JDBC driver loading is tricky.

jena-tdb is included:

https://github.com/stain/jena/blob/jena-osgi-bundle/jena-osgi/pom.xml#L89

I am getting worried you want to integrate the old proposal!

See https://github.com/apache/jena/pull/10/files

On 26 January 2015 at 16:27, Andy Seaborne <notifications@github.com> wrote:

> I don't think this should include SDB. (1) we don't encourage it's geenral
> use (2) aren't there JDBC driver loading issues?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stain/jena/commit/d1361295bf4106d3b63563f15482e8c557f203a9#commitcomment-9443182>
> .
>

--
Stian Soiland-Reyes, eScience Lab
School of Computer Science
The University of Manchester
http://soiland-reyes.com/stian/work/    http://orcid.org/0000-0001-9842-9718


Technical note:
At the moment SCRIPT_COMMAND_RESPWAN_GO will use
> uint32 time_to_despawn = m_script->respawnGo.despawnDelay < 5 ? 5 : m_script->respawnGo.despawnDelay;
as respawn delay.
This might result in despawn-spawned mobs to be spawned only for a short time. Maybe we should change this to preserve 0 if given in the database.

The limitations on excluding Buttons, .... is in my opinion stupid, and i think it is save to only limit doors with this command.


You should consider a pull request, this was exactly what I was looking for =)


While simple, this is business logic which should neither be in the view nor in the model.


NPE check here maybe?


This should consult the model instead of updating the view directly. The model then alerts its listeners of the appropriate changes.


All numerical values are sorted like strings, try sorting them by their numerical values ( http://stackoverflow.com/questions/19234744/angularjs-ngoptions-sort-numerically-by-label )


@fraank is doing PHP??


I don't see a model in your submission. The model is an intermediary between layers. Hopefully this will become more apparent in the next lab.


I do not understand all the code. I though we have to xor the two images but there is just one image opened in the script. Furthermore, why you get the lsb of red value at line 11 ? Same thing at line 17.


We may have some client apps where the list of facet on the search UI depends on the views (like Sextant where many apps are using the same catalog). But in that case, we'll probably create as many facetsSummaryType as required. Therefore, that was configured at the Liferay portlet level and was easily configured as far as the facets were exisiting. Now it will require to rebuild/redeploy the app to have a new set of facets. That may be fine for Sextant cases. To be asked. Another option could be to provide the capability to update Lucene config from the admin and reload facet configuration.


FFR, Cython does not like empty lists as args in the cythonize call


Job pr-scala failed for a72853ee Took 77 min. (ping @RomanHargrave) [(results)](https://scala-webapps.epfl.ch/jenkins/job/pr-scala/9571/):

To retry exactly this commit, comment `PLS REBUILD/pr-scala@a72853eea534fe210c6a252ec8617eeb46a66aad` on PR 4266.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.


this is pretty cool @aldenml. How come you decided on a TreeMap and not a HashMap?
`isComplete` would be faster, now it's `O(log(n))`


Why not use just a simple list instead of objects having same value in both fields? Doesn't really make sense.


Thanks, I'm very pleased to see this project get some attention again!

- Please update .gitignore (s/liblokatt.a/liblokatt.so)

- Please update run-tests.py to automatically set LD_LIBRARY_PATH, eg

```C
diff --git a/run-tests.py b/run-tests.py
index 418837c..79ca4db 100755
--- a/run-tests.py
+++ b/run-tests.py
@@ -68,13 +68,15 @@ def _parse_valgrind_output(raw):
         retval += 1

 def _execute_test(executable, testcase='', valgrind=False):
+    env = {'LD_LIBRARY_PATH': os.path.dirname(executable)}
+
     if not valgrind:
         cmdline = '{} {}'.format(executable, testcase)
-        return subprocess.call(shlex.split(cmdline))
+        return subprocess.call(shlex.split(cmdline), env=env)

     pipe_r, pipe_w = os.pipe()
     cmdline = 'valgrind --trace-children=yes --leak-check=full --track-origins=yes --suppressions=adb.supp --xml=yes --xml-fd={} {} {}'.format(pipe_w, executable, testcase)
-    retval = subprocess.call(shlex.split(cmdline))
+    retval = subprocess.call(shlex.split(cmdline), env=env)
     os.close(pipe_w)
     # pipe_r will be closed automatically at the end of the 'with' block
     with os.fdopen(pipe_r) as fin:
```

- Minor nitpick: please change first line of commit message to 'Build a shared library instead of a static one'. Also, please add to the commit message that this is the first step towards exposing the lokatt functionality to Python.


Are you saying that the v1beta1/ directory is actually sharded out per-resource type?

I want etcd to be wrapped in a "storage" abstraction and never mentioned by name anywhere outside of its own pkg.  Am I crazy?

Agree that getting rid of "registry" would be great.

Each REST type should be wholly contained, with the probably exception of the core types that are really the kernel of the whole system.


What changed here? :<


@valeriomazzeo could you please contribute such tests?


ok


I've made some comments inline. Looks good. I wonder if "League configuration" in navbar is a right place (or maybe we should have subpage regarding both viewing and configuring leagues) - but let's not care about that for now. :+1:


Closes #156


I don't recall, can we assume 3 is opened by the parent?


Don't need the buffered reader, can give the file straight to the scanner.

Also, use a try-with-resource for the scanner.  This will automatically close it for you in the finally statement (which you should also be doing here).
```java
try (Scanner s = new Scanner(tFile)) {
   ...
} catch (IOException e) {
   ...
}
```
Instead of:
```java
try {
   Scanner s = new Scanner(tFile);
   ...
} catch (IOException e) {
   ...
} finally {
   try {
      s.close();
   } catch (IOException e) {
      ...
   }
}
```


I'm okay with removing this file, but I think the README in the project root could use some love.


Yes, my bad. I'll fix that right now, thanks.


Just saw this commit title <3


FYI, this means you will have to specify the maven raspberry profile from the command line if you want to execute client that way.


No need to use `== true` or `== false`.
Instead use `if Actor6.IsInWorld then` or `if not Actor6.IsInWorld then`.


I'm not sure why this project was packaged as a PEAR library, but I'm pretty sure it isn't used as such anywhere.  And if it is, it probably shouldn't be.  So removing these files is okay with me.


/ping @evert and @armin-hackmann for a review of this commit please :-).


Why remove these lines?


Maybe it should be called ucs2char, not qwchar for less ambiguity, but idk.


You do know that you can run the tests locally. yes? :-)


It was a proof of concept command line utility distributed out to CIT long long ago.

I don't believe anyone is using it as-such today, but this feature was an awesome concept.


You're right of course.  I'll count you as a yes and contact the others.

+++ Nick Wellnhofer [Jan 26 15 08:47 ]:
>Personally, I agree with this change, but please note that all contributors must agree to a relicensing.
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/jgm/cmark/commit/58ce8ce3600a125aee8579a37b92474ebb4e8308#commitcomment-9443600


Logback...


rerere


This is iffy because it breaks vCard 3.


What was static cache, and why don't we need it anymore?


Don't log the toString of an exception, you should log the exception itself.  This preserves all the information about the exception.

```java
log.warn("", ex);
```


@evert Oh… Ok, so I have to check the version of vCard?


The vCard 4 specification says:

     Value type:  By default, it is a single free-form text value (for
      backward compatibility with vCard 3), but it SHOULD be reset to a
      URI value.  It is expected that the URI scheme will be "tel", as
      specified in [RFC3966], but other schemes MAY be used.


Please open a new ticket, including the output of `select postgis_full_version();` and **exactly** which Hesse file produced the error?


I agree.  I don't like the orange color, but the black was driving me
crazy.

It may be a good idea to open up a separate feature branch to experiment
with formatting, and colors, etc.

We can talk about it at our conference call.

Brian

On 26.01.2015 05:18, FWAJL wrote:
> @bsaiken [1] :
>
> To think about: the orange color you set on the category header looks
> too similar to the selection color, don't you think?
>
> --
> Reply to this email directly or view it on GitHub [2].
>
> 	*
>
> Links:
> ------
> [1] https://github.com/bsaiken
> [2]
> https://github.com/FWAJL/FieldWorkAssistantMVC/commit/5d7800f304ed2f19de976fbb716bc43c049604af#commitcomment-9435425


Just a small thing. I just started using your library in a tiny application here:
https://bitbucket.org/makononov/ed-x52-mfd-controller

It works great, thank you!


No, this is the correct fix :)

```diff
@@ -332,8 +332,8 @@ class Horde_Compress_Tnef extends Horde_Compress_Base
                     $data_len = $id_len + ((4 - ($id_len % 4)) % 4);
                     $this->_logger->debug(sprintf(
                         'TNEF: Named String Id: %s',
-                        substr($this->_getx($data, $data_len)), 0, $id_len)
-                    );
+                        substr($this->_getx($data, $data_len), 0, $id_len)
+                    ));
                     break;

                 default:
```

Without this fix the code will throw this error:
```
PHP ERROR: substr() expects at least 2 parameters, 1 given
```


No, it's also TEXT in vcard 4 by default, but it _may_ be a URI as well and a URI should be preferred.


Hi Nick, I was hoping you could help with my question about the inverted wheel view.

When I have >8 cards in the carousell, it functions with the correct spacing.
Users are able to "Save" cards and make them disappear from the carousel.
The moment it hits <8 cards, the spacing overlaps. (screenshot attached)

How can I ensure that no matter what number of cards are in the carousel, it adheres to the sapce arc/radius/spacing values?

![screen shot 2015-01-27 at 1 15 13 am](https://cloud.githubusercontent.com/assets/1883876/5904228/fa672ab2-a5c1-11e4-804c-b2c1b4c09dd7.png)


But they don't have mybb in name, which implies it was written by us. Agree with @Sama34 here and I'd change it to `2fa`


remain stay --> remain


As written it only applies to the faculty page. By removing the StaticCache dependency the request will always go to the PHP page.


I'm guessing this is the line the compiler is complaining about?  That's because this is really what you have told the compiler:
```java
WxCmdDataSocket<Object> wxSocket = new WxCmdDataSocket<Object>(TEMP_PORT);
```
Object does not implement Serializable.  If you leave off the generics in the variable definition, it assumes Object.  If you want it to compile and work the way you have it currently coded just change the variable definition to...
```java
WxCmdDataSocket<Serializable> wxSocket = new WxCmdDataSocket<>(TEMP_PORT);
```


I think github supports [tasks lists in markdown](https://guides.github.com/features/mastering-markdown/).

:+1:


Thanks Nathan. Which O/S are you on? I'm beginning to suspect this is a Windows vs Mac/Linux difference.


ok


Qu’est-ce que c’est que cette chose ! :D


they are not failing because internally RestKit is not using `RKMappingTest`.

for example:
`- (void)testInverseMappingWithNilDestinationKeyPathForAttributeMapping`

would fail if using `RKMappingTest`that at line 287"
`success = [relationshipMapping isEqualToMapping:propertyExpectation.mapping];`

use the `isEqualToMapping:...` method to find check if two mappings are equals.

isEqualToMapping won't work anymore for mappings which have `nil` as sourceKeyPath because they are not stored in the cached dictionary (nil would be the key).

https://github.com/RestKit/RestKit/pull/2147 fixes this.


Ah, okay, that does sound like a cool concept.  Thanks for the info @saltybeagle


Just curious if this library was used on the yellow-pages print-view? I don't think so, but should it be or could it be?


Mit Chrome (bzw. allen Nicht-Firefox-Browsern) ist das Interface bei der Dokumentenanzeige jetzt wieder relativ kaputt. Das war vorher auch schon mal so, da hatte ich da etwas herum gepatcht, damit es ging. Kannst du die Patches vielleicht auch wieder mit einspielen?


We only have one image. Each pixel of the image consists of red, green, blue and alpha values (see the r,g,b,a in line 11 and 17). Now, because we only have black and white pixels, and because the red/green and blue values are all either 255 (white) or 0 (black), we just get the `r`ed value for each 9x9 black or white pixel *block* in the big picture. Thats the first for-loop. The second for-loop just gets the `a`lpha values of the 9x9 block in the center (this time pixel by pixel and not block by block), because only this center block has different alpha values than the remaining 9x9 blocks.


I didn't get much of it though. Because I still call myself a beginner.
But understood that it ain't easy and you need to build everything from scratch.

Sure I'll contact you as I get new ideas, and anything related to script.
But this isn't a convenient way of contacting. Can i join you somewhere else? Like Hangouts, or Telegram or WhatsApp?

And, I have my final exams starting within a month. So I wont be available till atleast 15th of  March as soon as February starts.


I propose "The license may require for any distribution of the work that corresponding source files be made available in the preferred form for modification."


It was only used on this one page. The lib basically writes the rendered PHP file to the filesystem at the request URI to speed up mostly static content.

This view needs some love as identified by #16.


I support the "preferred form for making modifications" wording. It's an excellent term. It's legally clear too. If someone prefers a different form for their own modifications than the form they release under the license, then they violate that term. I would like to see the phrase more widely recognized.


It was pretty nifty when the package was mostly targeted as a web service integrations, but the repo has grown so much. Could be potential to remake a separate package that is specifically for service integrators.


oooppps


profe, ya no puedo hacer modificaciones al documento


inside-joke or whoops o_O?


Hey, don't forget to make these changes in the .tex.raw file.


O exemplo acima (cria | chama) um alerta


I get that, but I am not sure why you are using the interpolation syntax of Sass.

Having `#{$base-font-size}/#{$base-line-height}` is fine, because of this:

> It’s also possible to use #{} to put SassScript into property values. In most cases this isn’t any better than using a variable, but using #{} does mean that any operations near it will be treated as plain CSS.

However, it's not used here: `font: #{$base-font-weight} ...`. It should be `font: $base-font-weight ...`

Relevant Sass docs: http://sass-lang.com/documentation/file.SASS_REFERENCE.html#interpolation_


dá


We can look into it. But I'd prefer to only use released versions of python
modules. The reason being is what I grab today could be very different from
what you grabbed last week. Plus it is one more dependency that needs to be
micro managed.

On Sun, Jan 25, 2015 at 12:31 PM, hakrishn <notifications@github.com> wrote:

> Do you think the new dev version of Pexpect with 'echo' option would not
> address the issue?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opennetworkinglab/ONLabTest/commit/aea67aa184bad560e8d4869e5115a5d81cde95ec#commitcomment-9433305>
> .
>

--
Jon Hall
ON.Lab
jhall@onlab.us


Would have expected to see at __*least*__ one of each possible byte value?


In this commit message I forgot to note that this was necesarry to make TX to work due to the fact that character shifting doesn't work the same way for TTY/TDD as it does in RTTY


The commit that won't go away ;)

I'm currently refactoring a bunch of stuff in there right now, and already caught this, but thanks!


No objections.


Would it make more sense to order the fields differently, e.g. have endDate and startDate be together?


It's not currently, but may be a good candidate if we can determine what might be a good caching policy.


Does this have to be generic? That it, if we know it happens when the username is already in use and someone is trying to register with the same username, we can tell them it is already registered and reset password (if active). Or to offer to resend activation code or look at spam folder to activate (if the account is not yet active).


Good point, but we may encounter the same issue with mininet too since our
mininet is not same as latest version that has improved performance. We
need to come up with plan to move all our dependencies to be either
standard released versions or move along with dev versions that give us
some edge in solving old issues.

On Monday, January 26, 2015, Jon Hall <notifications@github.com> wrote:

> We can look into it. But I'd prefer to only use released versions of
> python
> modules. The reason being is what I grab today could be very different
> from
> what you grabbed last week. Plus it is one more dependency that needs to
> be
> micro managed.
>
> On Sun, Jan 25, 2015 at 12:31 PM, hakrishn <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
> > Do you think the new dev version of Pexpect with 'echo' option would not
> > address the issue?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/opennetworkinglab/ONLabTest/commit/aea67aa184bad560e8d4869e5115a5d81cde95ec#commitcomment-9433305>
>
> > .
> >
>
>
>
> --
> Jon Hall
> ON.Lab
> jhall@onlab.us <javascript:_e(%7B%7D,'cvml','jhall@onlab.us');>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opennetworkinglab/ONLabTest/commit/aea67aa184bad560e8d4869e5115a5d81cde95ec#commitcomment-9444333>
> .
>


As I said I don't use this anymore but feel free to push a fix :)


Without this, you don't learn specialization spells at spec selection.


Some (if not all) are from my latest merge, sorry!


How about `create_dir` and `create_dir_all`?


This check is probably not needed as you would just write when clicking on "Save as". Which is possible only with GUI.


No shit, this was fully intended.


You can use `collectError` here as it is evidently an error.


No that line is ok.  The lines it doesn't like is:

The compiler says "unchecked call to ...(Q) as a member of Raw type...

wxSocket.writeData(tSensor);

On Mon, Jan 26, 2015 at 11:20 AM, Brian Norman <notifications@github.com>
wrote:

> I'm guessing this is the line the compiler is complaining about? That's
> because this is really what you have told the compiler:
>
> WxCmdDataSocket<Object> wxSocket = new WxCmdDataSocket<Object>(TEMP_PORT);
>
> Object does not implement Serializable. If you leave off the generics in
> the variable definition, it assumes Object. If you want it to compile and
> work the way you have it currently coded just change the variable
> definition to...
>
> WxCmdDataSocket<Serializable> wxSocket = new WxCmdDataSocket<>(TEMP_PORT);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pinorman/wx-mon/commit/2a93438bdc6676646fa17130fbac792b8326edcb#commitcomment-9444182>
> .
>


See lines 39-51 just above, which create the `/etc/mysql/conf.d/docker.conf`.  This makes it easier to add custom launch flags without having to specify the current options that should've been in the config anyway.

```bash
# new way:
$ docker run -it --rm mysql mysqld --verbose

# vs the old way:
$ docker run -it --rm mysql mysqld --verbose --datadir=/var/lib/mysql --user=mysql
```


What's the reason for this? Does, e.g., plain fps (which starts with a `true` value for isOn) not start without it?


@baev What`s happined with previous release attepmt. As I understand my release teamcity task somehow not delivered new version to central? Was I did something wrong?


Sorry, completely misinterpreted your comment! Must read more carefully :blush:

Yes, you're correct of course. Thanks @kleinfreund


This commit breaks my build on Windows using Visual Studio 2013.

I looks like the return value for `tellp` and `tellg` can't be compared to int.

The error is the following:

```
MemoryStreamTest.cpp(375): error C2666: 'std::fpos<_Mbstatet>::operator ==' : 3 overloads have similar conversions
1>          C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\include\iosfwd(105): could be 'bool std::fpos<_Mbstatet>::operator ==(std::streamoff) const'
1>          C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\include\iosfwd(100): or       'bool std::fpos<_Mbstatet>::operator ==(const std::fpos<_Mbstatet> &) const'
1>          C:\Program Files (x86)\Windows Kits\8.1\Include\shared\guiddef.h(192): or       'bool operator ==(const GUID &,const GUID &)'
1>          C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\include\exception(497): or       'bool std::operator ==(const std::exception_ptr &,const std::exception_ptr &)' [found using argument-dependent lookup]
1>          C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\include\exception(502): or       'bool std::operator ==(std::nullptr_t,const std::exception_ptr &)' [found using argument-dependent lookup]
1>          C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\include\exception(507): or       'bool std::operator ==(const std::exception_ptr &,std::nullptr_t)' [found using argument-dependent lookup]
1>          C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\include\system_error(402): or       'bool std::operator ==(const std::error_code &,const std::error_condition &) throw()' [found using argument-dependent lookup]
1>          C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\include\system_error(410): or       'bool std::operator ==(const std::error_condition &,const std::error_code &) throw()' [found using argument-dependent lookup]
1>          while trying to match the argument list '(std::fpos<_Mbstatet>, int)'
```


Although TLS specific, I'd be happier with predicates over certificate living in x509, yeah.


Confusingly enough, your `require` statements don't need to be case-sensitive and are idiomatically written in lowercase. Were you unable to use the `FileUtils` module when requiring `fileutils`?


Why choose LinkedList ?


I'm surprised that blaze isn't deleting the whole form when we clear selectedRegistrationReact. I fear that that fact this is needed is a sign of a lurking bug somewhere. I'll play around with this as soon as I wake up (I'm not really awake ATM)


Snap, didn't realize that `EXDATE` can have more than one property. Will check!


Can you  also add a `.gitignore` line for this?


Nice use of recursive copy!


they do - don't they? I mean ciphersuite -> (keytype, keyusage) needs to live in tls ; but cert_type lives in x509...


This is super nitpicky, but you usually want to avoid comments that describe _exactly_ what happens in the method.

In this case, you don't need to say `unless exists?` which is the Ruby implementation.

Your comment can just say something like "Create 'public' folder unless it exists". Very subtle difference, but this way you're not using your comments to repeat the exact code used in your method.


the `~` files should be ignored. You can create a `.gitignore` file and blacklist `~` files.


All mine from @2792b6b57ea85e362cc3e2f593ca2a9d9e706c57 no worries


C'était un test, j'ai oublié d'effacer.


@clicman nop all was fine. Just notify me next time when release - I need to confirm it in sonatype. Now we rollback wrong [ashot](https://github.com/yandex-qatools/ashot) release and remove your too


If the purpose of this code is simply to create a copy of the `ARGV` array called `img_urls`, you can use the `Array#dup` method:

```ruby
img_urls = ARGV.dup
```


Nice brah. Implemented.


Why not simply reuse the old code and change the init function in a way, so that it will react dynamically on a given parameter. This way you can build arbitrary indexes, dynamically by a given list ...


Haha thanks for cleaning that up for me


A joke. ;)


Yay for templates! :smile:

This is a place where I'd use a constant to store the value of the template file, since it is something that likely won't change over the course of your program. This way you can hard-code the file path in a constant and then reference that constant wherever you need access to that file path:

```ruby
TEMPLATE_PATH = "views/gallery-template.html.erb"
```


I mean I have used JD-GUI and BON on some 1.6 mods. I have not put together a 1.4.7 BON yet. I just started too over the weekend. It would be good to look at the 1.4.7 code that has not been obfuscated.


@mcculls I don't think that a null format is valid, we should always know the format for an artifact, and if we don't RHC/CLM code that expects it will fail down the road.


@kichristensen: I think it would be better to remove the redirect, because not everyone is upgraded to the lasted version. What do you think?

Example: https://304notmodified.github.io/documentation/


Use `${conffile}` before `.example`.  (Or should we include "." in the list of characters that are sufficiently obviously not part of the variable name that we don't need braces?)


Whoops, not yet...


About this update, in Ertheia you should be able to add multiple ServerTypes not only 1.


<version> for vertx-codegen needs to be removed


Looks good.


what about using the static constants above instead of the String litterals? Would be easier to search for occurrences in the code later on.


Ces commits n'ont pas forcément grand chose à voir avec home/index mais il ya quelque chose qui me perturbe sur tout le site : la page est toujours un peu plus grande que l'écran, même vide.


If you intend to sign all addons you've got to do this in celery. There's a lot addons on AMO.


Editorial: I suggest you cut "This document describes...for SSL 3.0." The first sentence is redundant with what you wrote above in the "version" section. The second sentence isn't important.


Looks good, have any issues along the way?


Great work.  Thanks Ben!  Looks so much better.

Our only sources right now are ACS and the Datacommon, so please adjust the list and I'll merge it.

It should have been a little clearer in the issue.  Sorry about that.


I worry about that query on every addon.


@LukasReschke I guess next sync will have these patches applied so I think  there is no need for a special patching script here.


Feels like it should be higher than info.


Why not put all of this text in the "version" section, instead of having the "length" and "fragment" sections in between? It makes sense to me to keep all the text about the record version together.


Fixes #81


Indeed :) https://github.com/clojure/clojure/blob/clojure-1.6.0/src/clj/clojure/core.clj#L471


Yeah we should definitely make use of those constants. I'll add that before submitting a PR for the branch.


This all looks good to me. :+1:

Go ahead and pull it into master. :shipit:


Agreed - i'm struggling trying to get an automated build working with Gulp at the moment b/c it's not picking up the .css file using the main-bower-files plugin.


Hey Tejas, instead of creating Javascript Date instances from the moment, how about using isAfter or isBefore - see http://momentjs.com/docs/#/query/is-after/ for more details


@kellyrob99 no worries, was just applying the same check as in `RepositoryBrowserContainer.js` (which I assume also needs changing if a `null` format is invalid) - anyway will leave it to you and @dariush-griffin to sort out the appropriate fix.


To summarize, @cquammen  will be looking at this issue in the coming weeks.


alarm_msec(0) is not a noop, and should be removed for clarity.


nit, order the kwargs


' not "


As above, alarm_msec(0) is no noop and should be removed.


is it worth asserting this is a particular value?


Tienes toda la razón, este archivo ni lo toca. Gracias!!!


I'm afraid you changed the byte value but not the comment. Am I missing something?


Bring the sample up-to-date.


P


>= for both conditions


This driver is for RGB control, gamma control is a different feature.


r+


Why does the API need to precompile things?


If it's a random name, I would recommend choosing a name that suggests about that fact, such as "dummyfile". Also including it in a variable name in order not to repeat it.


`2fa` is okay for me too. Will change it tomorrow as I won't be on my pc today.


> This check is probably not needed as you would just write when clicking on "Save as". Which is possible only with GUI.

True currently, but unit tests could crash, so best to leave the check in there.


good change


Sure – but the problem is that we can't wait ages to have those patches applied. They fix some sensitive issues that we need to get backported.


```>=``` is what I meant


I just figure the problem,just an error with setup the open cv with eclipse but i need some help from you about this code? Can i have ur email ?


It should not encode any thing now #277


If you use the carousel:valueForOption:withDefault: delegate method, tou can return 8 for item count, then they'll always be spaced as if there were 8 items.


Is there any way of recalculating the signed_status from a file? I guess I'm not sure what the advantage of this is and I worry about keeping a transactional system (the db) and a non-transactional system (the filesystem) in sync. If something can wrong, it likely will :)


looks like 01/26 nigthly has fixed this.


instead of this, put on the prototype

get stableId() {
  return ...;
}


remove


dont let it be settable


throw 'NotImplemented'


I'll just move along then...

![move-along](https://cloud.githubusercontent.com/assets/61721/5905149/9463036c-a553-11e4-87a9-34c598343e21.gif)


ok, but i was formated code in my IDE before pushed, and IDE  was formated  this so


Ah.. Just managed to reproduce the bug. I will only add some checks to make it compatible with url-addon-bar.

Best regards,

Patrick

From: noreply@github.com [mailto:noreply@github.com] On Behalf Of Yuan Xulei
Sent: Monday, January 26, 2015 6:28 PM
To: yxl/Fire-IE
Cc: Yifan Wu
Subject: Re: [Fire-IE] Fix bug #10: IEMode下 无法提交地址栏输入的网址。 (c46dfa5)

This is a workaround with a weird bug that you cannot focus the URL bar sometime. That possibly isn't reproducible any more with the new Firefox. If everything works fine, you can remove it.

—
Reply to this email directly or view it on GitHub <https://github.com/yxl/Fire-IE/commit/c46dfa5496cb1d19687ff9bdb70cc5ea3180ae93#commitcomment-9437916> .  <https://github.com/notifications/beacon/ABb5sG05IN4oT_C9v1-LB7f7SXF3Ipl6ks5nlg4dgaJpZM4DXBLB.gif>


Heres a link to the src: https://www.dropbox.com/s/wqkrz49ii0w16ip/DRPG1.4.7.src.zip?dl=0


impl drifted from docs


doesnt this need a stableid?


Keep your Skype open faggot


r+wc


Yes, my mistake, I meant to write my OS info on that. I'm on Windows 7 using Adobe CF 9.


Fixes dates re: #33


There is another step to be taken here. This file path assumes that wherever this code is executed, there will be a `views` folder in the same directory. In other words, you're using a [relative path](http://glossary.codeunion.io/relative-path).

Because relative paths are dependent upon the _working directory_ of wherever this file is run from, you will run into problems if you ever decide to execute `gallery.rb` from a directory that is not `photo-gallery-cli`, for example if you moved one directory up and ran the file like this (try it and see what happens):

```shell-session
$ pwd
/path/to/photo-gallery-cli
$ cd ..
$ ruby photo-gallery-cli/gallery.rb photo-gallery-cli/photos/bunny-1.jpg
```

To avoid complications with relative file paths, it is best to convert them to absolute paths. In Ruby, this can be accomplished with the use of the [`File.expand_path`](http://ruby-doc.org/core-2.2.0/File.html#method-c-expand_path) method and the `__FILE__` special variable.

In this case, you would rewrite your code like this:

```ruby
TEMPLATE_PATH = File.expand_path("../views/gallery-template.html.erb", __FILE__)
```

What you are telling Ruby here is to create an absolute path from a relative path (`"../views..."`) by **using the current file as a starting point**. The `__FILE__` variable represents the current Ruby file, and so if you write a path that is relative to this file (in this case, you have to include the `../` because Ruby treats the second argument to `expand_path` as a directory, even though it is a file).


If a unit test ever reach this part of code, it should already have mocked the GUI or created a headless server. If a check like this is needed here, then it would be necessary for every interaction with the GUI, not just the Message boxes.


Yep - laravel feels really charming!


We could check the xpi and check the `META-INF` files I guess. Storing the `signed_status` is only for convenience, for example to not re-sign addons that are already signed (with a management command), or not have to unpack the xpi to find out if it's signed or not (and if it's signed for prelim or full).

I guess it's not mandatory, I just thought it would be useful. I can get rid of it if you think it'll cause more issues than it'll solve.


You're right! Fixed in 4af1a4d7e0dce782a46bfcc84dd79839c87e5b35. Thanks!


In this file I suggest having only the function that implements the local strategy and the options. Here's an example

```javascript

var LocalStrategy = exports;

LocalStrategy.options = { usernameField: 'user[email]'
                                       , passwordField: 'user[meta][password]' };

LocalStrategy.verify = function(email, password, done) {
  User.matchUser(email, password, done);
}
// matchUser is a function attached to the statics of userSchema that checks the credentials.
```

And then in auth.js you have something like this:

```javascript
var local = require('./strategies/local')
   , Strategy = require('passport-local').Strategy;

passport.use(new Strategy(local.options, local.verify));
```


I hadn't considered this usecase.  How about I modify it so that if it is a
string it will be config-summary summary type reference otherwise it is a
configuration json?

Jesse

On Mon, Jan 26, 2015 at 6:05 PM, François Prunayre <notifications@github.com
> wrote:

> We may have some client apps where the list of facet on the search UI
> depends on the views (like Sextant where many apps are using the same
> catalog). But in that case, we'll probably create as many facetsSummaryType
> as required. Therefore, that was configured at the Liferay portlet level
> and was easily configured as far as the facets were exisiting. Now it will
> require to rebuild/redeploy the app to have a new set of facets. That may
> be fine for Sextant cases. To be asked. Another option could be to provide
> the capability to update Lucene config from the admin and reload facet
> configuration.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geonetwork/core-geonetwork/commit/5f375592a4f4c00760b8c0b150f01e3d6b28c1de#commitcomment-9443933>
> .
>


У тебя слишком много селекторов по всему DOMу. не надо так. Они медленные.


@Tarnum7 the blizerd broked skype!1


sourceRoot is used to organize how the source files appear in web browser dev tools. the name is arbitrary, but it needs to be separate from the 3rd party stuff to keep things organized.


Hmm, I hope I'm not missing something, but it might not need to be so complicated:

```diff
diff --git a/avocado/loader.py b/avocado/loader.py
index e665369..03deb9c 100644
--- a/avocado/loader.py
+++ b/avocado/loader.py
@@ -178,8 +178,7 @@ class TestLoader(object):
                     test_name, params)
         return test_class, test_parameters

-    def discover_directory(self, dir_path='.', ignore_suffix=None,
-                           omit_non_tests=False):
+    def discover_directory(self, dir_path='.', ignore_suffix=None):
         """
         Discover (possible) tests from a directory.

@@ -207,14 +206,13 @@ class TestLoader(object):
                 continue
             elif os.path.isdir(new_path):
                 params_list.extend(
-                    self.discover_directory(new_path,
-                                            omit_non_tests=omit_non_tests))
+                    self.discover_directory(new_path))
             else:
                 params_list.append({'id': new_path,
-                                    'omit_non_tests': omit_non_tests})
+                                    'omit_non_tests': True})
         return params_list

-    def discover_url(self, url, omit_non_tests=False):
+    def discover_url(self, url):
         """
         Discover (possible) test from test url.

@@ -223,9 +221,9 @@ class TestLoader(object):
         :return: a list of test params (each one a dictionary).
         """
         if os.path.isdir(os.path.abspath(url)):
-            return self.discover_directory(url, omit_non_tests=omit_non_tests)
+            return self.discover_directory(url)
         else:
-            return [{'id': url, 'omit_non_tests': omit_non_tests}]
+            return [{'id': url}]

     def discover_urls(self, urls):
         """
@@ -239,10 +237,7 @@ class TestLoader(object):
         for url in urls:
             if url == '':
                 continue
-            if os.path.isdir(url):
-                params_list.extend(self.discover_url(url, omit_non_tests=True))
-            else:
-                params_list.extend(self.discover_url(url, omit_non_tests=False))
+            params_list.extend(self.discover_url(url))
         return params_list

     def discover(self, params_list):

```

This should cover the same. Because when we discover directory, it implies `omit_non_tests`. Thus when the `discover_url` is file, we can just skip `omit_non_tests` (resultin in None) and hardcode `omit_non_tests = True` in `discover_directory`. What do you think?


sourceRoot is used to organize how the source files appear in web browser dev tools. the name is arbitrary, but it needs to be separate from the our src stuff to keep things organized.


This is a lot of logic to be living out in the open and not wrapped up in a nice method(s).


@Tarnum7 o and im in robotix


As mentioned in [above comment](https://github.com/ric-reyes/photo-gallery-cli/commit/4af9c0acb70defb6a4ebd612945c4520c4b0fcbd#commitcomment-9444982), it is unclear what `compiled_string` is here, since it appears out of the blue. It must be either a method or a local variable, but if it is a method then what does it do, and if it is a local variable then why is it assigned so far from this line?


MUCH CODE


nice


Thank you!


Are you sure we need to do this to IACUC, I didn't see the problem happening in here?


Nice work, @ric-reyes. No worries about the funky diff, but glad you realized your error. :smiley:


Cool solution


Stop merging master. Seriously.


Conditional operator?


I know I will need to iterate the values by offset value (the key). TreeMap provides me such capability.


Let's move the session configuration in a function.

```javascript
function configureSession(session) {
  ...
  return session;
}
```

And then

```javascript
app.use(configureSession(session));
```


> If a check like this is needed here, then it would be necessary for every interaction with the GUI, not just the Message boxes.

If we can render a track to completion without the GUI, I assume we can do most anything without one.  This isn't true for logic that exists solely in UI code, but perhaps that is a problem and it should be.  I guess it depends which elements we're unit testing.  If we're planning to simulate interaction with the UI, then we expect a UI.  I just wanted to mention it. :)


the import didnt work on my computer, because it couldnt find the name of
the module, so i corrected the name

2015-01-22 13:59 GMT-05:00 kovek <notifications@github.com>:

> What is that for?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kovek/MCSC_game/commit/2b3a30f2ac5178fce234d5c6a26bf5a572e5983d#commitcomment-9402966>
> .
>


```
with tempfile.NamedTemoraryFile() as temp_file:
   temp_filename = temp_file.name
```
Is one line shorter :)


Bonsoir, ces modifs n’ont pas été proposées dans une PR?


Yes


@tobibeer I think this is a reasonable proposal. Maybe the function would be clearer called processNewResult?

How many of the existing filters could/should be refactored to use this helper function?


Hey there @zfields , thanks for the pull request!  Unfortunately, I'd rather not merge this in though because I don't want to force the user to `create_spark_library $1` in step 1, that's described in [step 2](https://github.com/spark/uber-library-example#2-call-the-function).

Cheers!

-j


should you use `Charset.defaultCharset()` instead of `"UTF-8"` so that it will use the system specified char encoding?


ok


Looks like you committed an development file: ``.project``. You'll likely want to add ``.project`` to you global  git ignore file (``~/.gitignore``).


Fixes https://github.com/mozilla/marketplace-elements/pull/9.


I got what you mean, I am just saying that it does not make sense, for example, to check if you have a GUI in a method that gets called from an action from a context menu. Which is a different category of check of the one you mentioned, when we render the track (with what we actually have issues, try rendering a track with a VST via CLI, it'll try to open a window).

I think this check is from the first category.


Would have added `gnFacetConfigService` in `gn_facets_config_service` module to avoid specific file and closure dependency.


Further reference: http://www.cmake.org/cmake/help/v3.1/policy/CMP0025.html#cmp0025


We should discuss with @fgravin . Another option is to do something similar to solr where you have a facet.field parameter with a list of fields to return as facet. So you can create your own set of facets (instead of our fix set in summary).


aah.. Good idea. Will update the code.


Further reference: http://www.cmake.org/cmake/help/v3.1/policy/CMP0054.html#cmp0054


We can get rid of this now, right?


will sorting by date work without this?


Hi all @kennycyb , @davidlks , @nurulazradm , @vahakmatavosian , @cjcaj ,

In general this is a good solution because it solves the problem about filtering and customise some columns  but we need to consider some things we'd like to add to the gem before it starts to get big(we're starting to add commits).

1. We're going to need to configure more actions than just the index one. I think we need to be ready for that since this moment in order to avoid overwork in the future. For instance, you'll want to customize the fields in a form when you create a user. So, we'll need to configure the new action.
2. Listing and filters for index actions are different things. Listing and filters play together but they're not the same. We need to provide to users a friendly way to add/remove/customize columns in the table but also a form at the left/top of the table to filter data. I dont think adding filters in the headers of the table is a good approach.
3. In the future(not necessarily right now) We'd like to specify the type of the filter for a given column. For instance, the User table has email(text), and maybe it has an active(boolean) column. It'd be great if we can specify that for "email" the filter type is going to be a text field and for "active" it's going to be a radio button :)

Now we know a little bit more about how Sinatra works I'd like to refactor the gem in order to use it in this way.

```ruby
class MyApp < Sinatra::Base
  use SinatraAdmin::App do |config|
    config.root "Home"
    config.register "Post" #=> Single resource registration. No config.
    config.register "User" do |users|  #=> Single resource registration. Config added.
      users.index do |listing|
        listing.sort_by :created_at, :desc #=> Default sort is by id??
        listing.columns do |config|
          config.add :email #=> Title by default is underscored. Content by default is the database value
          config.add :full_name, content: ->(user){user.firstname + user.lastname} } #=> Can pass a lambda with dynamic logic
          config.add :username, title: "nickname" #=> Can specify a custom title
        end
        listing.filters do |config| #=> Filters will be a form at the left of the listing.
          config.add :username #=> Will display a text input by default
          config.add :active, type: boolean, values: [true, false] #=> It'd be great if we can do this :) Not necessary right now.
        end
      end
    end
    config.register 'Home' do #=> Registration of custom resource. It wont change so far :)
      get '/?:home?' do
        @msg = "Welcome to home page huerco!"
        haml :home
      end
    end
  end
end
```


Gotcha, just make a new branch.


I would wait on anything too fancy because we are going to move to SOLR in
probably the next version (3.1 or so).  I would rather make a simple
solution now and if we want something more complex do that when we move to
SOLR.  I hoping that SOLR experiments will be my Bolsena project.

Jesse

On Mon, Jan 26, 2015 at 7:17 PM, François Prunayre <notifications@github.com
> wrote:

> We should discuss with @fgravin <https://github.com/fgravin> . Another
> option is to do something similar to solr where you have a facet.field
> parameter with a list of fields to return as facet. So you can create your
> own set of facets (instead of our fix set in summary).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geonetwork/core-geonetwork/commit/5f375592a4f4c00760b8c0b150f01e3d6b28c1de#commitcomment-9445098>
> .
>


excellent :)


TeamCity Azmq :: Build (Linux) [Build 69](https://78.46.71.115/viewLog.html?buildId=395&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 53](https://78.46.71.115/viewLog.html?buildId=396&buildTypeId=Azmq_BuildOsx&guest=1) is now running


TeamCity Azmq :: Build (Windows) [Build 92](https://78.46.71.115/viewLog.html?buildId=397&buildTypeId=Azmq_Build&guest=1) is now running


Thank you!


I'm avoiding any change to this logic; it's a refactor to copy code from down below. Note that in the context of the gem this code will never be called anyway.


In this error case, did you check that liba52 is compiled with -fPIC?


justin, look at line 30 - 37 to see if i did erb templating correctly


Check this out: :beers:

https://github.com/LMMS/lmms/blob/master/src/core/main.cpp#L210

-Tres


this seems like it is introducing a bug. if the secret is `['a', 'b', '\0', 'c']` then this will only have `"ab"`. is the fix in the gem going to make sure that we never create a secret with a null byte terminator?


I just tested pexpect master and it looks like we are having the exact same
issue as we were having before.

On Mon, Jan 26, 2015 at 9:33 AM, hakrishn <notifications@github.com> wrote:

> Good point, but we may encounter the same issue with mininet too since our
> mininet is not same as latest version that has improved performance. We
> need to come up with plan to move all our dependencies to be either
> standard released versions or move along with dev versions that give us
> some edge in solving old issues.
>
> On Monday, January 26, 2015, Jon Hall <notifications@github.com> wrote:
>
> > We can look into it. But I'd prefer to only use released versions of
> > python
> > modules. The reason being is what I grab today could be very different
> > from
> > what you grabbed last week. Plus it is one more dependency that needs to
> > be
> > micro managed.
> >
> > On Sun, Jan 25, 2015 at 12:31 PM, hakrishn <notifications@github.com
> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
> >
> > > Do you think the new dev version of Pexpect with 'echo' option would
> not
> > > address the issue?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/opennetworkinglab/ONLabTest/commit/aea67aa184bad560e8d4869e5115a5d81cde95ec#commitcomment-9433305>
>
> >
> > > .
> > >
> >
> >
> >
> > --
> > Jon Hall
> > ON.Lab
> > jhall@onlab.us <javascript:_e(%7B%7D,'cvml','jhall@onlab.us');>
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/opennetworkinglab/ONLabTest/commit/aea67aa184bad560e8d4869e5115a5d81cde95ec#commitcomment-9444333>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opennetworkinglab/ONLabTest/commit/aea67aa184bad560e8d4869e5115a5d81cde95ec#commitcomment-9444419>
> .
>

--
Jon Hall
ON.Lab
jhall@onlab.us


Is it a hardship to say that kubernetes names DO allow DNS-subdomains (and all that entails) but do not allow arbitrary stings, and if you want to jam an arbitrary string with "TZ" in it you have to tr that "TZ" into "tz"?


Probably just remove this commit in a rebase. That is, ``git rebase --interactive mater`` and then remove this commit from the list.


Really? I guess that's when 16:9 screens become handy...


One reason the font is bigger in the sidebar is to make the click targets bigger - e.g. make them easier for you to click on, since you tend to click in the sidebar a lot.


thanks.. then it is misspelled.


TeamCity Azmq :: Build (OSX) [Build 53](https://78.46.71.115/viewLog.html?buildId=396&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 17 Build time: 00:02:48


TeamCity Azmq :: Build (OSX) [Build 54](https://78.46.71.115/viewLog.html?buildId=398&buildTypeId=Azmq_BuildOsx&guest=1) is now running


BTW, I'd recommend [ipdb](https://pypi.python.org/pypi/ipdb/0.8) if you haven't given it a shot.


Sorry, I meant "API actors that are more privileged than normal users who are using the API server as a dumb store, so that the API server can embed less magic logic".  Right now, there is no actor that mutates status that doesn't go through a separate API resource (binding) or is implemented inside the apiserver (pod status, node status, load balancer).  For allowing API consumers to set status we had posited separate resources for status, in which case the logic is mutable.

However, on update, the registry/store may set new status info (like binding) that has to be correctly merged and checked.

----- Original Message -----
> Where do we allow Status to be settable by users?
>
> Are we converging on a position here?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/smarterclayton/kubernetes/commit/0ab7606d9f9be06240bfda58c683daa7ef1107bf#commitcomment-9443734


I actually like the larger font size here as it helps make the text easier to read and type. You'll notice the height of the textarea adapts to the size of the font used.

maybe we can settle on 14px?


Because if you can make that work, you have automatic import and export of a working system?

----- Original Message -----
> If someone POSTs, then GETs, then POSTs again, do we want to have a lot of
> sympathy?  Why would they do that?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/smarterclayton/kubernetes/commit/0ab7606d9f9be06240bfda58c683daa7ef1107bf#commitcomment-9443778


Fixed the formatting on my second link


That's correct; the native implementation interprets secret as a null-terminated string and this ensures consistent buggy behavior.


Hrm... it looks yuckier to me, but let me think on it a bit and see if I can convince myself that it's actually better.

----- Original Message -----
> If if so, why not just
> ```
>  {
>        "name": "",
>        "randomNameBase": "my-prefix-"
>  }
> ```
>
> I'm not SURE it's better, but we also want to be moving away from structs for
> RestartPolicy
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/smarterclayton/kubernetes/commit/0ab7606d9f9be06240bfda58c683daa7ef1107bf#commitcomment-9443828


Extra comma before "change".


Exactly. Note that with `foldp` below, the `update`-functions isn't called on signal initialization at all. The initial value of a `foldp` is the explicit initial value argument, not anything obtained from a call of the updating function. So without this here, no timeout will be created at signal initialization (since the only other call to `setTimeout` is in the `update`-function, which will not get invoked).


Should this be formatting the actual bias value rather than just saying on/off?


You should document `abstractOverClass`. But I wonder if it doesn't make more sense to have `fullyParameterizedType` automatically do the right thing?


@jhulslander, testing IACUC I did see the same issue as document was still in ENROUTE.
After this fix, it is set to FINAL.


OK, I am fine with an explicit field distinct from Name.


ye blizzard gave u aids. open it when you r home, fucker


Hmmm. Why is this `Tls.Config.config` instead of `Tls.Config.server`?

A `config` can only be produced from a `server` (or a `client`), as it's private. And the only way it can be consumed is by turning it back into a `server` (or a `client`). Is there anything achieved this way that could not be done by having a `Conduit_mirage.server` contain a `Tls.Config.server`, the type of fully formed and validated TLS server-side configurations?


----- Original Message -----
> Are you saying that the v1beta1/ directory is actually sharded out
> per-resource type?

Saying that it should be by "group" of resources (pods/nodes, services, and replication controllers).  Not required, but means that we clearly indicate levels of "coreness".

>
> I want etcd to be wrapped in a "storage" abstraction and never mentioned by
> name anywhere outside of its own pkg.  Am I crazy?

Someone has to choose to import it - that's what I'm referring to.  The code that uses the storage has no idea its etcd (RESTStorage interface), the code that injects that dependency (the volume plugin equivalent) into the code that's using it has to know how it's implemented.

>
> Agree that getting rid of "registry" would be great.
>
> Each REST type should be wholly contained, with the probably exception of the
> core types that are really the kernel of the whole system.

Right.

>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/smarterclayton/kubernetes/commit/0ab7606d9f9be06240bfda58c683daa7ef1107bf#commitcomment-9443956


Yes you are right, good catch! I will open an issue and fix this asap.


If you hover over a link in the current state, the arrow appears behind the time. See http://monosnap.com/image/56EFhbU9zyH9pi8neS7PaDLhXLx33E for a screenshot.

(I acknowledge this is super broken in the current master)


This is a header and it makes sense to have a large size here, to emphasize that it's the title element for the block


Typo: goo1.loop -> go1.loop


OK. got it to say openssl would be built in the configure. It built a dll in the C:\php-sdk\phpdev\vc11\x86\php-src\Debug_TS directory.

I set up a php.ini with only:
extension=C:\php-sdk\phpdev\vc11\x86\php-src\Debug_TS\php_openssl.dll

I can't get it to acknowledge OpenSSL. I see the dll built, and the intermediate files in ext/openssl

I tried do get it to list it with a -m and -i.

When I try to run the tests with

C:\php-sdk\phpdev\vc11\x86\php-src\Debug_TS\php.exe -c C:\php-sdk\phpdev\vc11\x86\php-src run-tests.php ext/openssl/tests

it skips them


```
 {
       "name": "",
       "randomNameBase": "my-prefix-"
 }
```

vs

```
 {
      "name": "",
      "generateName": {
            "base": "my-prefix-"
      }
 }
```

If we REALLY think that it will grow more fields, then a struct is OK.  But if we think that we will go to 1.0 with just a single field, then I prefer the simpler form.


This isn't a standard library import. You didn't get an ``ImportError`` here?


Published to http://vui-docs-internal.s3-website-us-east-1.amazonaws.com/28_7e03ca341526603263edec95ae5e0f4112cc8575


typo: Physcial


So, the goal here is that there are a lot of areas in that buildHistory div that aren't actually clickable. For example, if I put my mouse here:

<img src="https://api.monosnap.com/image/download?id=YvvjBvzUpE0byaLxfEftWY84mhGl8p" />

I can't click on anything, but I should be able to. Yes, the padding in master is probably broken but there might be a better fix here.


yes, makes sense --- didn't get that yesterday.


Like the idea but I'm not a huge fan of the color. Any other ideas to try?


.vagrant
.idea


Shouldn't have `const` here since this is not returning a reference now.


- we should include some object deletion to make sure those get rolled back properly
- if you name the objects simple numbers, you can do something like

 rados -p $pool ls | sort -n | uniq

and diff with `seq 1 3 ; seq 5 6` or something to verify that we have exactly the right set of objects in the result?


A comment's good if it explains what's going on, but in this case it just repeats the name of the element below - maybe we can provide some more useful information here?


no const here


it's very subjective. if a function name is often used by users for their own functions then it's better that it's not configured.

I'd *guess* that the strcmp and strlen are very unlikely to be used by users for their own functions. So I think it's good to have configuration.

I *guess* that rename and remove can be used for various purposes by users. So I guess that these functions should not be configured.

I can't say which specific functions we should configure and not.. but please consider if it's likely that users will create custom types/functions with the same names.


Let's create a folder `templates` and store the password reset email in a separate file `password-reset.hbs`.

At some point you'll want to add styling and it will come in handy to edit a file instead of a string.

You're going to load the template using `fs.readFile`.


These keys belong to config.json.


Can we do an if-statement? I'm not sure how comfortable I am with gen checks being this inline.


Sentence initial in uppercase.

Free and Java is always implied, so redundant.


We should create a separate module that is responsible for sending emails. It could export a function

`function sendPasswordResetEmail(user, newPassword, done)`


`validate_usage`, `validate_ext_usage` and `validate_keytype` are dying to move into x509, under names such as `supports_usage` or so. They are purely structural anyway (being parameterized on the actual usage / key type) and misnamed (being predicates).


It might originate for this line though.  Try the recommended change and see if that works.


For streaming responses, better use `StreamingHttpResponse`, as that is its purpose (`HttpResponse` for streaming is deprecated).


i changed, look please or so?


I think it would be better to encapsulate the `set_last_activity()` call with the condition in a method. Instead of `elif not self.is_passive_request(request): set_last_activity(request.session, now)`, have `self.set_last_activity(request, now)` which would encapsulate the `utils.set_last_activity()` call and conditions for call.

Does this make sense ? What do you think ?


Also tried an INI with:

extension_dir = "C:\php-sdk\phpdev\vc11\x86\php-src\Debug_TS"
extension=php_openssl.dll


Either way, you should never define a variable without generics if the class has generics.  The ability to do so is probably only there for backwards compatibility.


Ugh good catch meant to be under backend...


Do they have a documentation in English?


I really like the idea of examples or case studies. Can anyone recommend a DY project or two or example where this is exemplified?


Nit: reference jQuery objects with a `$` prefix, e.g. `var $documentRoot = $(document);`


If I followed this correctly, it still does a string lookup (actually two of them) every time we need a ground transform. Might not be a problem at all, but is worth profiling at some point since it might be the case that this gets called many times per time step.


:+1: Looks good.

I'll test again the next chance I get.


no const here


So to be clear, WRT api VERSION:

POST to /api/v9beta3/ns/foo/pods really says 'create a Pod in Namespace "foo" with APIVersion "v9beta3"'.  If those fields are set to different values in the POST body, it is an error.

GET from /api/v9beta3/ns/foo/pods/bar really says 'get the Pod with Name "bar" in Namespace "foo" in the form of "v9beta3"'

The API version really is per-object (or per small set of objects).  It's POSSIBLE (even if we choose not to) to release Pod v10 and leave ReplicationController at v9, right?  I'm thinking about how external REST plugins get versioned, and what happens if they don't release in lock-step with us.


Thanks @tobibeer ideally we'd have one pull request with the helper function and adjustments for all the relevant filters.


`$(documentRoot)` can be simplified to `$documentRoot`. No need to use the $ selector each time.


This `server` type needs `with sexp`, which `Tls.Config.config` provides but `Tls.Config.server` doesn't, so I assumed `config` was the correct type. Perhaps `ocaml-tls` could add `with sexp` to its `client` and `server` types to allow this?


We might be able to show some of the search updates on Learnivore /cc @danmcclain

Also, I've thought showing the page load difference on a SRA vs a SPA. A video show side-by-side how clicking around a SRA is slow, shows a blackscreen until the page loads. Whereas a SPA will be almost instant.


this doesn't look right...  what if we're using selfmanaged snaps on this ioctx?  the pool snap_seq will be 0.


If you don't have a logo configured, the header looks like this:

<img src="https://api.monosnap.com/image/download?id=FQdy0obGgmSKxDuk05UB6O8X1dsmTC" />


Replacing this text everywhere is a non-starter, unfortunately. I'm open to ideas about how to make this text more configurable.


extra space


r+ I think. There was a bit of code, but it's mostly deletions, yay!


You can use the `save` function only when you initially create an object. Afterwards all the operations must be `atomic` - in this case use `findOneAndUpdate`

Can you imagine a scenario (not necessary related to telegram app) when using "save" instead of atomic updated operations will create problems?


This will go in the new email sending module.


i don't think we want to be looking at osd data structures at this later.  we can just filter things up in the OSD? we need to decode the object_info_t::clones there anyway in order to verify the object exists in a particular snap


You could use `Rx.Observable.interval(probeInterval).subscribe(checkCollisions)` here instead


we should probably add much better matching of functions. Now we only look at the name. We could look at what arguments there are, what the return value is, what includes there are etc. to be sure that we match the correct function.

when we know the correct functions are matched we can definitely add *all* functions.


With the -c it's not loading my php.ini. I copied it to C:\windows (since the output of -i said it was looking there) and it still didn't pick up or parse my ini file.


@atreni My post that goes into the Learnivore example a bit: http://reefpoints.dockyard.com/2015/01/07/complex-search-pages-feel-better-in-ember.html

Feel free to ping me on it


this is a lot better

somehow it feels a bit messy that you're working internally with two versions of the addresses now (the singular and the arrays) but refactoring that can be done later

the preference for the array can be fixed by always adding the single version to the start of the array and using the array internally as much as possible - so when only the singular config is used the multiple array will have count() = 1

if you create a pull request i'll try to get around to testing it as soon as possible


We should make this fields also editable ttw, does not make sense to show them and don't enable edit.

Use case:

You compile out of plone your bundle and you want to move from .min.js to .js and you want to change the real js from that bundle.

R


Shouldn't this be : ```var port = options.port || 0;``` ?


should it be calling `setNull()` again?


admin folder was moved from .admin to admin so I removed the .
We had tested this one without the .


These sizes are set with my own Javascript I think, which is why you need the `!important` here to override it. You may be able to find the right place in src/theme.js to change the ball size setting.


> By definition, the database consists of the targets and the sequence being searched is the query.
Yep! They should have been switched.

> I'm reverting this commit due to compilation problems. It's more suitable to a pull request anyways instead of a direct push, since it changes a lot of things.
I agree now. I'll target this for another release (e.g. 4.1).


Now when I add a contributor and click save it takes me to an error page:
"error at /collections/new
[Errno 61] Connection refused"
Seems to be an error an sending an email to the contributor. The collection is still saved with the contributor though.


Added the [PHP] at the top of the ini file (duh) but it didn't help...


typo: trasform


----- Original Message -----
> So to be clear, WRT api VERSION:
>
> POST to /api/v9beta3/ns/foo/pods really says 'create a Pod in Namespace "foo"
> with APIVersion "v9beta3"'.  If those fields are set to different values in
> the POST body, it is an error.

Yes.

>
> GET from /api/v9beta3/ns/foo/pods/bar really says 'get the Pod with Name
> "bar" in Namespace "foo" in the form of "v9beta3"'

Yes

>
> The API version really is per-object (or per small set of objects).  It's
> POSSIBLE (even if we choose not to) to release Pod v10 and leave
> ReplicationController at v9, right?  I'm thinking about how external REST
> plugins get versioned, and what happens if they don't release in lock-step
> with us.

Yes - the pain point is on the client side in determining which API versions are available and whether they can be out of interlock.  OpenShift for instance only has v1beta1, but that API version follows the semantics (mostly) of v1beta3.  So rules based on "oh, v1beta3 means you follow these constraints" won't work, and saying "i want to use v1beta3 to talk to the server" doesn't make sense if you're talking to two servers at the same time (or two apis via the same server).

>
>
>
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/smarterclayton/kubernetes/commit/0ab7606d9f9be06240bfda58c683daa7ef1107bf#commitcomment-9445447


I believe this commit only changes call updates and not group message updates. Group message updates still show up in a JSQInfoMessage. I'm working on updating that.


extra semicolon


Is this really the way to do this?


I renamed it


agreed. then also the `cert_type`, `cert_usage`, `cert_extended_usage` can be removed from x509/certificate.mli


we need to make sure this works for pools that are selfmanaged, too.  there is a pg_pool_t method to tell which mode we're in.

note that in the selfmanaged case, we will always list head objects.  you can list snaps on each one, but that's horribly inefficient.  and if an object is deleted there is basically no way to see it via list-objects... i suspect that a more complete solution is really needed.

but, it depends on what problem we are trying to solve.... is there a use-case that is motivating this series?


good point, I'll add a constant; it's the length of the salt prefix + string; the usage comes from the reference bcrypt code


I created a new issue for this: #243


good catch!


ship it!


Hey, thanks. That saves me a lot of trouble.


The indentation looks a bit messed up here, I think?


I find this difficult to understand, for a few reasons.
1. reducedType should be called narrowedType, that is the term we use elsewhere
2. I am not sure what a maskState is. Can you use a different name? Maybe subtractFlags and subtractFlagValues?
3. It is not clear what `!(t.flags & maskFlags)` represents. Please add comments explaining it, or rewrite it so that you can assign it to a named variable.


Please convert tabs to spaces.


for consistency with other existing tests, and the ability to tell which example is failing (sigh, JUnit)