Does this mean rotated files will be closed immediately?  So possible to lose message on a busy system?


This function has changed a bit, so when you merge to the latest changes in the master branch you'll need to update this function. It should be pretty quick to update.


vel gert!


This a create approach and cool example of the power of `Enumerable` when used within an `Array`.  Don't forget though that `Posts` are stored in the database.  You have a SQL background.. is there a way you can use the database to check if a `Post` matching the title and body exist?

Make sure to check out the [Active Record Querying Guide](http://guides.rubyonrails.org/active_record_querying.html) for inspiration.


initializing start and end to -1 would reduce this to `if (start !== -1 && end !== -1)`

as is, this looks like maybe it was intended to be `((start || end) && start !== -1 && end !== -1)`


97204612_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1805 of 1805 tests passed)

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


dispatched ;)


w00t! Green on Linux finally.


No.


LGTM.


It did get a bit slower, but not significant. I think this is acceptable.


Thanks!!


It's not required in this specific instance. I like that `{;` makes it explicit the curly braces are not introducing a hash reference (which `+{` would make explicit), since the inside looks like it could be one.

I can take it out if it's more distracting than helpful.


do we ever actually have "Other"? Let's call this bucket "Invalid values"


`if val.startsWith(outputBegin)` would be more readable


That's because the `geventwebsocket.Resource` expects an application class (rather than instance) in its `OrderedDict` (see line 609 for reference).


era una prueba


`if val.endsWith(outputEnd)`


Sorry if it seems like nitpicking, but although this is an improvement it still has risks.  A developer who learns the API but not its underlying source code would be reasonable to infer that to_hash returns something _other than_ the actual storage object, and that that returned thing would be safe to modify without affecting the Hashish instance; but this is not the case.

I'd really recommend renaming the instance variable as well...you could call it something like hash_store (or data, or data_store) and just call attr_reader :hash_store, etc. to generate the getter method.

A couple of other suggestions...

* if you truly want to hide the Hash methods that are not explicitly exposed in the Hashish class then you might want to make private the method that returns this data store.

* implement a hash method that Set, Hash, etc., can call that delegates the calculation of the hash value to the hash method of the underlying Hash (phew!), like this:

```ruby
def hash
  @hash_store.hash
end
```


Why are you not on IRC? :P


Dispatch


Tbh I don't like this approach.
It's like a contract, if "y" is "e", then there shouldn't have a "r". Or even there is a "r", we shouldn't try to get any benefits from that.

Basically, we should implement it just follow the spec, otherwise it's not compatible to other implementation.


hm, your commit message (4.9), echo (4.8) and update-alternatives (4.6) say 3 different things... now I'm confused. ;-)


I think I had `include_empty` default to `False` because that was how the existing locust code had it.


@NicoleGrondin - these should include a link to the repo and the source code


Use enum for image format.


ah, I see how update-alternatives is handled, now. still, the commit message and echo disagree


I doubt this line is needed.


@dansilivestru oops, thanks!


Misspelling: "sddio" should be "stdio"


You've presumably discovered by now they are called `GrayA` and `AGray` (depending on storage order). That seemed more consistent with the append/prepend just "A" for all the other colors (e.g., `RGBA` vs `ARGB`, etc.). But if you like the longer names we could switch back to `GrayAlpha` and `AlphaGray`.


:) yeah thanks, i just changed it


does this behave nicely if they delete/backspace while a portion of the input is selected?


Show or Hidden attribute


Ready to merge.


I will add other properties


No, it is indeed more consistent :) I just didn't see it in the exports and it was missing in Images, so I just assumend I need to integrate it. That's the disadvantage with these macros.


weeks may be left unitialized.


Confirmed


How much of decision to add to your dotfiles was due to the name vs the functionality?

On Tue, Aug 18, 2015 at 1:59 PM, Gabe Berke-Williams
<notifications@github.com> wrote:

> git-shalector (from @christoomey)
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gabebw/dotfiles/commit/72179487355454016b9643e32b99fbe67f726541


94006714-MultipleTowers (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1795 of 1795 tests passed)

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


:highfive: dotfiles are meant for sharing

On Tue, Aug 18, 2015 at 1:46 PM, Adarsh Pandit <notifications@github.com>
wrote:

> This is very good. Thank you.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/christoomey/dotfiles/commit/aad108b56384dfb6196046dd1c9ae29fd28987ea#commitcomment-12766814


gitattribute and gitignore remove


What's the importance of this?


Maybe s/NewPrefixConfig/MakePrefixConfig/, since it returns PC not *PC?


this code is in a bunch of places. would it be worth doing a global handler for `inserted.atwho` that does this unwrapping? or is there ever a time when we need the atwho-inserted container to exist?


The changes I am going to push in that I spoke about on the call will conflict with this.  I will work on the merge.  I also change the .cts to .dts after group agreement on the choice.


Sorry, I forgot the name so I used an alternative. Must be the commit changed or is it ok?


rax retry tests


+1, done


Prevent slide from advancing immediately after resuming.


Fixed misspelling of "stdio"


can be simplified to `if (target.closest('.jstree-drop').length)`


It's ok. I just told you for learning purposes.


please encapsulate if ... else ... code in separate functions. Something like
`get_cycles_images`, `get_blender_render_images`


My apologies, overlooked the commit note and went straight to the code to see what was changed.


regeisters


Those are actually good suggestions @keithrbennett. No such thing as nitpicking when it comes to clean code. :-) At this point, I'm not in a rush to support those features as I don't have any need for them. If you'd like to take on the challenge, I'll happilly merge your PR.


The first sentence is literally 'Change to Window stance.'


This corresponds to JIRA ticket PR-2175


that's fixed in: https://github.com/feedbackex/CryBlend/commit/cf8bccb3affb3a166ad4ca7589c316d838924efd
;]


similar to above, drop the if statement and replace `target` with `target.closest('.jstree-drop')`


This is a neat program, and well done. Thank you for sharing it.

Perhaps this hard-coded '10' should be the width of the board.


CC @edolstra, @cillianderoiste


pop_topoplot is hanging  - it looks like there is problm in some cases and not others, looking into it ..


We'd have some scoping issues: onChange is defined within this function, and we need to run this code also when there's an error - adding it to the async series wouldn't work as the series breaks when there's an error.


maybe worth commenting this function to be clear it might return HTML


Need to delete m_eventEngine in ~Game() !


or renaming it. getMugDisplayMarkup?


Fixed:

https://github.com/Mei152/blocitoff/commit/5961be66e6ddd28e8af0b72b96150ac398a661bb


This is cleaner than the previous incarnation of this code, but I still think dropping the function altogether would be an improvement.


I noticed the docs website is out of date too. It would be helpful to update that as well. Is there a repo for that?


input with rows=5? shouldn't that be a textarea?


Calling a non-splat an error condition is a bit confusing. It's just a pre-condition for finding errors further up. The array literal splats, on the other hand, are error conditions. But I would prefer if you had a variable called, for example, `found_non_splat` in `splat_offenses()`. The term `error_condition` is used for two different things now.


Ah gotcha. Good idea :+1: I will update the README in the other branch.


I'm sorry about all the weird words in autocomplete.out...


Wow, this solution does look ugly!


97204612_MultiSpeedFan (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1805 of 1805 tests passed)

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


So here are are just calculating total visits and not distinguishing from local vs remote right?


Ugh! I pushed this on "autopilot" without paying attention!


타지스매쉬 is the original name. 타지 being Targe and 스매쉬 being the romanization of smash.


Follow up note (it took a while to remember this since it's been a while since i touched this code!): the only reason why Hashish exists is because I wanted a HashWithIndifferentAccess but didn't want to have to require ActiveSupport. This was a requirement in one of the projects I was working on in the past. Also, I didn't want to inherit from or monkeypatch Hash since that's just opening up a bag of worms. It was never my intention to hide Hash methods or anything like that. There's nothing to stop a dev from accessing it anyway no matter what I do.


Dispatched


You forgot to delete the old code


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - x86_64-MacOS-10.9-clang: OK (1786 of 1790 tests passed)

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


seriously, fuck windows


This variable is now unused.


97204612_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1280 of 1280 tests passed)

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


I've actually been wondering what would happen if I just deleted the "master" branch, because I've done it too. Anyway we need to talk about git flow soon after install.


@mscoutermarsh @hugolmendez Because of how the game is written and it is not possible to click outside of the game board, hence it would not be possible to move the piece off of the board, should we include these extraneous conditionals?


On second thought, there's an edge case here where every component in the graph is terminated with `Component.terminate()` when there's an uncaught exception.

Since we can't guarantee the state of each component will be in a {ACTIVE, SUSP_RECV, SUSP_SEND} state and unconditionally want them to terminate in this case, I think it would suffice for now to be more lax on state checking here.


> How much of decision to add to your dotfiles was due to the name vs the functionality?

Exactly 50/50, though I probably wouldn't have given it a second look without the dope name. ~marketing!~


97204612_MultiSpeedFan (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1805 of 1805 tests passed)

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


:+1: for at least pulling it out into a `let`

Bonus points if you start using `instance_double`


These were pretty random images...


but the spec doesn't say anything about fields _not_ existing, just the ones that can be expected to exist.


line too long, PEP-8


saw approval from zeeshanlakhani
at https://github.com/basho/yokozuna/commit/bc0efa7cbeececd92368a49d4f362b997d1699d3


merging basho/yokozuna/jdm/only-update-plan-per-nval-with-type = bc0efa7c into borshop-integration-539-jdm/only-update-plan-per-nval-with-type


basho/yokozuna/jdm/only-update-plan-per-nval-with-type = bc0efa7c merged ok, testing candidate = 09b250d7


Need to review use of `max-width` / `max-device-width` and decide on appropriate sizing for phones/tablets (see Meistertask)


Or ConcurrentQueue which *may* reduce head contention; then just lock on Alloc->Create rather than each Alloc


This markup is in three places. Thoughts on pulling it out into a utility function?

More generally, how painful is it going to be to change the bubble markup - if, say, we decide in a month that bubbles shouldn't have close buttons anymore?


Alternative solution: https://github.com/fredemmott/definitions-finder/blob/master/.gitattributes


Maybe move the OutputRedirect struct to the shared script utils? It will be needed in VB as well


Last comment, then I lunch. But it's the most important.

You're having to do a lot of testing at this level because all the logic is tangled up in the controllers and the models. Pull some of the logic into plain-old-ruby-objects (PORO) so that you can test there, and then you can actually start doing integration tests that do things like, login, generate a recipe at random and see that things appear, create your own recipes, check the about page and see that it exists and maybe includes the name David somewhere. That's the level you want your integration tests to be at.


I got your phone number


@neerajbaid Love this delimiters array.  Any idea as to when it might get pushed to an official release?


Yeah, it works without the c()

Thanks for your help!

Brendon

> On Aug 18, 2015, at 00:24, Eric Johnson <notifications@github.com> wrote:
>
> It's been a while. Does it work with your suggested modification?
>
> If not, one idea could be that c() can concatenate many types of things. But passing in the ids Vector ensures I get a Vector type back as a return value.
>
> —
> Reply to this email directly or view it on GitHub.
>


@gubatron I think you will like this

![screen shot 2015-08-18 at 5 38 47 pm](https://cloud.githubusercontent.com/assets/143813/9343459/2140dc38-45d0-11e5-9e1c-b840192f308c.png)


Agreed, though I'm deprecating this project once https://github.com/hackpack/hack-scanner is ready.


as in, you want to see these delimiters as default?


thank you

When you have time please collect and share with me all the OrgDev role related tasks you have outside _OrgDev role_ in _Asana_ so I can take them over.


You should check that `settings['IMAGE_PROCESS'][derivative]['type'] == 'image'`


Amazon makes me sad sometimes. But that's not the main issue. What if I don't run on AWS? Nothing in this plugin made it obvious that this was only for collectd running on AWS.


Using ConfigureAwait(false) stops the next bit of execution requiring the context thread that it was first initially run on. but once it falls out of that execution block it will resume with the context thread (by not supplying ConfigureAwait(false)).

I'm guessing if they all are just worker threads then it wouldn't my any difference as their wont be anything that is required by the context thread such as HttpContext or UIContext but i normally try to stick to these rules:
https://msdn.microsoft.com/en-us/magazine/jj991977.aspx

Hope this makes a little sense.


lets have a meeting about this


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - i386-Windows-7-VisualStudio-12: OK (1792 of 1796 tests passed)

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


I know that feel


This metric name should have a `sf.` namespace prefix then. Also, why not have the datapoint value be a timestamp?


:+1: When you get a chance, could you push this up to master?


doesn't this require jQuery?


This change breaks map downloading in 0.6 branch because NETMSG_MAP_DATA is never sent with NET_CHUNKFLAG_VITAL.


Every time you call `send()` or `send_notifications()`, you'll re-query the host info stuff, including the slow 6 requests to the AWS metadata API, etc.


`if old_host_info` is enough, it checks for emptiness too


I think it makes much more sense to pass the name to the submodule, because the object and the memory slot for the object are distinct, which makes it much easier to go by name, so the module can manipulate the object AND the memory just as well.


old_host_info, METADATA = METADATA, host_info


Ah, I see now.
But it would be great if we could find a way to break this 118 line function up, into more maintainable code.


What was the change of heart to support both Python releases as oppose to one (Python 2.7). Wouldn't this increase the file size immensely?


:+1: this is an assignment that should be merged to master.


You must pick one during build, don't work on both at the same time.


Gracias flaco

      Ing. Laura Nelly González Serna

     Tel. (81)80402350, (81)84780995

               Móvil : 8112554589

                  <http://www.etiprint.com/> www.etiprint.com

------------ La presente información se envía únicamente para el destinatario, y contiene Información de carácter CONFIDENCIAL o PRIVILEGIADA. La modificación, retransmisión, difusión, copia u otro uso de esta información por cualquier medio, por personas distintas al destinatario esta estrictamente prohibido. Si usted no es el destinatario, por favor notifique al remitente respondiendo a este mensaje, y borre el mismo y sus anexos sin retener copia alguna. Gracias. The information contained in this message is being sent to the intended recipient, and contains PRIVILEGED / CONFIDENTIAL Information. The modification, retransmission, disclosure, copy or other use of such information by persons other than the intended recipient is strictly prohibited. If you are not the intended recipient, please advise the sender immediately by replying to this e-mail and delete this message and any attachments without retaining a copy. Thank you ---------------------------------------------------------------------

De: Jorge [mailto:notifications@github.com]
Enviado el: martes, 18 de agosto de 2015 01:10 p. m.
Para: jogonzal-msft/Etisys <Etisys@noreply.github.com>
CC: LauraGzz <laurag@etiprint.com>
Asunto: Re: [Etisys] cambios 18 de agosto laura (eca3e54)

Muy bien mami! A la otra intenta hacer varias changelists (varios "check in to master") para que los cambios sean mas granulares, pero tu descripcion de los cambios esta perfecta. Yo le doy una checada a lo de la guia mas tarde.

—
Reply to this email directly or view it on GitHub <https://github.com/jogonzal-msft/Etisys/commit/eca3e54842ba28db8cde9a1c988f592b66e3addc#commitcomment-12767260> .  <https://github.com/notifications/beacon/ALoMF7FQIu3zev5O5ip8EDjbF9idcKjQks5oo2xmgaJpZM4FtyFV.gif>


Thank you for adding the test


You mean when I launch the app the on Mac.


Hab mal testweise Google+ in die configs und unter interactions erscheinen die von facebook


Due to the nature of this change, it might be worth thinking about bumping up a version number? 2.0.0 or at least a 1.2 increase. I believe should be a 2 though


it's a jadecore copy ...


Shouldn't have changed things in the masterProcess script so reverted back to help with any conflicts that could arise.


No, during the build of the application. When you launch the application you are forced on the python version chosen during build time.

This commit makes it possible for people to (re)build Natron using either Python2 or Python3 (not both).


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1228


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1801 of 1801 tests passed)

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


Just stumbled onto this. I haven't seen a related PR in glob2base.

@contra would you be open to just swapping out glob2base for glob-parent directly in glob-stream? Now that @jonschlinkert has provided some performance evidence, we could follow through on what you said in https://github.com/es128/glob-parent/issues/1#issuecomment-70400061. If that happens, this commit in gulp-watch ought to get reverted.

Either that, or https://github.com/jonschlinkert/glob2base/blob/glob-parent/index.js could just get merged into glob2base - which may have the advantage of keeping the ecosystem aligned with the other consumers of gulp's dependency tree. However, assuming it would come with a major version bump, there's probably little to gain by doing it this way.


Got it. Thanks!!!


Right, because we only need to figure out if there are any, so we save ourselves the complexity.


Thanks TheShotz for the bug submission :)


If you do this then I think the pair will have the wrong type, you'll get `std::reference_wrapper<Entry>` for `first` instead of `Entry&`. You might just want:
`std::pair<Entry&, bool>{ SuppressionMap.emplace(index, Entry ()).first->second), true }`


Strange, TravisCI is green, since TravisCI repeats whole build process it looks like an issue related with differences between environments. It looks like you are using different virtualenv version. Build process was tested on Ubuntu 14.04 with default virtualenv 1.11.6.

To avoid such situations usually I [download specific virtualenv version and use it](https://github.com/python-dirbtuves/akl.lt/blob/master/Makefile#L41-L46). But here I decided that for only two developers we can rely on virtualenv provided by distribution.

`pip install -e .[postgresql]` installs python package [in current directory](https://pip.pypa.io/en/stable/reference/pip_install.html#editable-installs) with `postgresql` [extra feature](https://pythonhosted.org/setuptools/setuptools.html#declaring-extras-optional-features-with-their-own-dependencies) in [development mode](https://pythonhosted.org/setuptools/setuptools.html#development-mode).


Changed from getCanonicalName to getName, since the canonical name replaces '$' with '.' for inner classes, and the classloader subsequently cannot find the class. We need to keep the '$' in the class name.


Hmm, originally I didn't think it did, but now that you mentioned it I've looked into it and yes it does use jQuery. I suppose this needs to be changes to simply calculate the size of the vertical margins and then add it to the `offsetHeight`?


Forgot __stdcall topkek


Nice catch! Should be fixed with 56d6a3d846ebc8c87cfabafa1e549298f3103df0


let's then work on the same set of data and follow the same pipeline. Can you share the STUDY you are using with me?


@Acrobot Unfortunately no. It even happens with another user.


@jonschlinkert `file-conflicts` isn't published to npm


I'm using the 10.6 sdk on OS X and I was using the core-audio-utility-classes from 2010. As files that are now included are coming from a newer version I cannot compile against the 10.6 sdk anymore. Is there a way to include an older version. Maybe a define that tells Juce to not use the embedded files would already help.


I don't care about deprecating glob2base in favor of glob-parent, it just seems like really poor open source to create a new module instead of sending a PR to an existing one when you have improvements


rax retry tests


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 355](https://teamcity.labkey.org/viewLog.html?buildId=267176&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:09:53
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


Still generates the source map though?


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yes, these should all point to ForeignKeys.


Eben nochmal auf beiden Systemen getestet. Liegt wirklich an der Blender-Version, warum auch immer ?_?
Also einfach Blender Version 2.69 (schätze höhere Versionen sollten auch gehen) installieren, dann sollte in Unity alles korrekt angezeigt werden. Wenn es doch noch Probleme geben sollte, bitte melden.


Yes! #GroovyMaster!

This is beautiful. Now we just specify product variant on the manifest? Sweeeeeeet.

Wonder if instead of reading that `IS_GOOGLE_PLAY` constant, we can read the flavor, and control the whole thing directly from the manifest.


interesting, why does it come as `plus1`?


plus seem like a reserved word, no idea.


lambda, how pretty


And since the ForeignKeys are all ```id```s of some kind, you should name them accordingly.


Now it is:
```
public static final boolean IS_GOOGLE_PLAY_DISTRIBUTION = BuildConfig.FLAVOR.equals("basic") && !BuildConfig.DEBUG;
```


Can you provide an example of "name accordingly"


Don't add a helper for this, just use `RandomCollectible(class=CardClass.PALADIN)`.


Ok, but as discussed in the past, I wasn't aware of glob2base when I authored glob-parent (maybe I never would have if I had found it, which as it turns out would have been a shame).

But so the question is how should we reconcile this now? Totally happy to approach this according to whatever your preference is.


The you need to add a ```relationship(...)``` for each to map the actual object. See here: http://docs.sqlalchemy.org/en/latest/orm/tutorial.html#orm-tutorial-relationship


lol


we need to figure out how this is going to be exposed. either as separate flag, or just use the --module flag value.


I was just wandering around Literally Canvas's network graph and saw this. Why didn't you submit a pull request? :-(

I fixed the `fromJSON` bug in master. How good was IE 8 compatibility after this patch?


do you need a `continue` here?


Testing testing. HIV testing.


Does this affect the Travis environment? See 781d1a3


You are quite welcome to it!


No. As far as I'm aware there isn't an option to generate the source map but remove the mapping path. (It's a bit of a packaged deal).


Thank you so much! It looks great!

On Tue, Aug 18, 2015 at 5:07 PM Tom Lynch <notifications@github.com> wrote:

> You are quite welcome to it!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/replicatorg/ReplicatorG/commit/dea45d796dd1aba703f14ab898587164f356c7bd#commitcomment-12771917>
> .
>


101280704_LowTempRad (Nigusse) - x86_64-MacOS-10.9-clang: OK (1776 of 1791 tests passed)

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


Naming example: ```owner_id = Column(Integer, ForeignKey('users.id'))```


Let me know if you need other versions, I can dig it out… Not like MakerBot cares to use it...


That's what I thought I remembered. I think we need to keep building the map file, and just strip out the comment afterwards. Right?


@vanevery ^^


Don't we have the same problem then with the other fields like anpa-category or subject etc?


Did you get any peer review prior to changing this?


I'm really not convinced that it should be one function.

First of all code would be more straightforward if split up into cycles/blender_render

Second, all colors are arrays https://www.blender.org/api/blender_python_api_2_75_3/bpy.types.Material.html (except emit and ambient) so you can refactor `color_to_string` and pass arrays to it.


Why is this ++m_eeCacheSize and not m_eeCacheSize+1?
Confusing to assign to the same value twice in one statement.


Do you need a `continue` here?


I've punted on source maps since they're trivial enough to generate and may be handled by a users grunt/gulp/bundler setup.

Modifying the generated file is kind of gross. There's a UglifyJS option to customize the `--source-map-url`. May work setting it to empty.


same here and all others


wrea


I don't think you need the cast to `Slice` here.  Same for the other calls (except for the block one above)


Mislim da je ovdje ideja bila da sastavis ipak upit tipa: SELECT * FROM tableName where attribute = 'value', tako da nisi trebao staviti attribute u select attribute from tableName.
Takodjer ipak mislim da si ovdje trebao da se zastitis u metodi tj, da ispitas da li ti je neko predao nesta konkretno npr, pitanje za tebe sta se desi ako ja predam ovdje null, null i sta na kraju dobijem kao spojeni upit ;).
Znaci uvijek se u metodi stiti kada ocekujes nepredvidjene parametre kao sto je to slucaj sa String tipom.


Second part of the naming example  would be:
   `owner = relationship("User", backref="user")`

And then leave the _init__ still:
 `self.user=user`


IMHO you need 4 spaces for indentation not just 2.


Delete underscore.min.map?


You sure that line works? “behaviou**sr**” can't be right ;)


Is there a vector/hires version, and is there a version without the gradient?


just use `new ArrayList<>(array.getPositionCount())`


Just tested and passing " " (note the space) for `--source-map-url " "` will set the `//# sourceMappingURL= `. I donno how that effects tooling or warnings.


@blasten I'm not super familiar with versioning in GitHub.  However, I just did a bower update with #master indicated as the requested version to be pulled for this element.  Bower did pull the master branch for iron-list and I then tested my app to see if it resolved the issue I'm having with rendering 19 unnecessary instances of list items (it should only have one).  Each of the unnecessary 19 hidden items contain a fairly complex custom element.  That causes all kinds of delays on initial load/readying of the list.  Should this fix have resolved that issue?


whoa this is long. so much logic in this ling. refactor a little bit.


new up `ArrayList` directly


TeamCity ATK :: QA :: Integration Per Branch [Build 1886](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546370&buildTypeId=Atk_IntegrationPerBranch) is now running


talk to lily about how awesome haml is - lets talk about u starting to use haml on net project.


Please follow the commit message format: https://wiki.gentoo.org/wiki/Gentoo_git_workflow#Commit_message_format


I made this mistake while I switch from Visual Studio Code to Visual Studio 2015 RTM.
Thanks @matsprea,


There is both not liking the gradient? :( Let me know… I was about to put a new Mac icon in for you and stuff...


Seems ok, no warnings. I'll ammend the commit to just use an empty source map url. So the map is still generated until a call is made on its continued support.


This seems to have broken the System.ServiceModel.Discovery tests:

https://wrench.mono-project.com/Wrench/ViewWorkTable.aspx?lane_id=33&host_id=8&command_id=917


is the `object_.parent is None` needed?
I guess you are creating flat structure because of removing `if object_.children:` in line 975


static import `constantInt`


Is there a way to check out submodule to a specific commit?
In this case:
1868d4c7f646e0201d79cd76fb54f740820983fd


you don't need comments like this anymore... The above byte code expression will render like this anyway (and the comment doesn't match your code).


> [idx] == callout IDs are correct but callout list renders underneath example

That's because, as I pointed out earlier, that taking away the `+ 1` ensures the condition never runs, so you are just back to the default behavior.

The point of `+ 1` is to advance to the sibling block to see what it is.

> if the next block is a `<calloutlist>`, then don't render this tag.

That's probably a safer approach. The problem may be that the state "skip" is getting overridden...so perhaps best to avoid the use of state and just control the output directly.


oh, I see now.


same here


Yes it does, but in case of other fields we don't validate schema like we do in case of dateline. Here is the reference of definition of dateline validation: https://github.com/superdesk/superdesk/blob/master/server/apps/prepopulate/data_initialization/validators.json#L55-L63


Looks good to me.


think we could make the switch to Jackson?


Could you please rely on the common sense next time, not just *TravisCI is green*?


You may also want to set the `Internal Field Seperator` to NOT use  a blank space
See the argument here `IFS=$'\n\t` This will cause bash to not cut arguments too liberally, but you'll need to test it, and see the caveats for space separated command line options.

```sh
#!/bin/bash
IFS=$' '
items="a b c"
for x in $items; do
    echo "$x"
done

IFS=$'\n'
for y in $items; do
    echo "$y"
done

=>
    a
    b
    c
    a b c
```


btw @aldenml I think this year you've become a Super Sayayin Coder, Kudos, it's a pleasure, honor working and learning from you.
![super-sayayin](https://cloud.githubusercontent.com/assets/163977/9344477/e068bac6-45d6-11e5-8be8-baea242aa27b.png)


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1796 of 1796 tests passed)

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


please never ever join path manually: https://docs.python.org/2/library/os.path.html#os.path.join
CryBlend may be used on Linux (used to work) or even Mac OS (never tested) but with windows style path it would not work.


Damn, images didn't attach...

![flat-icon](https://cloud.githubusercontent.com/assets/147143/9344446/95d97854-4600-11e5-9495-435a63f49bbf.png)
![256](https://cloud.githubusercontent.com/assets/147143/9344494/f7050c92-4600-11e5-958b-076669a69481.png)


I went this was based on http://en.cppreference.com/w/cpp/utility/pair/make_pair, which says
> The deduced types V1 and V2 are std::decay<T1>::type and std::decay<T2>::type (the usual type transformations applied to arguments of functions passed by value) unless application of std::decay results in std::reference_wrapper<X> for some type X, in which case the deduced type is X&.

and http://en.cppreference.com/w/cpp/utility/functional/reference_wrapper, which says
> Instances of std::reference_wrapper are objects (they can be copied or stored in containers) but they are implicitly convertible to T&[....]

If you still think it's wrong, I'll happily change it. Let me know.


Why .smex-items? For me it is not a hidden file or am I misunderstanding?


do you mean [os.path.join](https://docs.python.org/2/library/os.path.html#os.path.dirname)?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 356](https://teamcity.labkey.org/viewLog.html?buildId=267195&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:09:52
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


I didn't know that!


this is wrong


I tried to combine the original two lines into one. Basically we want to increment the counter but not above the limit.

So the same logic can be like also:
if (m_eeCacheSize < EE_PLAN_CACHE_SIZE) {
    ++m_eeCacheSize;
}

Do you like this one better?


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (749 of 749 tests passed)

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


easy as pie.
```shell
cd http/www
git checkout 9dc9302a80a2f3a3c51d9b9c2b799cff5c13f927
```

Note that https://github.com/volumio/Volumio2-UI/commit/9dc9302a80a2f3a3c51d9b9c2b799cff5c13f927 (dist) is the same commit as https://github.com/volumio/Volumio2-UI/commit/1868d4c7f646e0201d79cd76fb54f740820983fd (master).
They have different hashes because the `dist` branch is rebased by `git subtree split`


rax retry tests


If it's a control key you don't call base.ProcessCmdKey, so whatever that would normally do is presumably suppressed...I wondered what consequences that might have. For example, it's imaginable that not calling that could result in Gecko not seeing ctrl-B/I/U. Most likely not, but if you're not TRYING to suppress something, why not just call it after doing our special thing?


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1792 of 1796 tests passed)

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


Yes. Makes sense.


Should probably change this to an Assertion :)


Documentation is awesome!


Done.


 git checkout 9dc9302a80a2f3a3c51d9b9c2b799cff5c13f927
fatal: reference is not a tree: 9dc9302a80a2f3a3c51d9b9c2b799cff5c13f927


Also fixed with the above change.


It is definitely good work in right direction, it could be improved.
First it does not need to be map in map but just array in map (iteration of array is much faster over 'for in').
Adding lineHeight independently will be much shorter than whole code just to solve this exception.
It is nice puzzle in compression. You also didn't use fact that you are running inside browser (it is not clear if it would help or would not be too slow). Extract common subarrays you have ["Top", "Right", "Bottom", "Left"] there 3 times - try to think about size even without gzip.


Hi Lucas:   Hoping this would pass LTC.

Did you test this change as a stand-alone (without the other changes above)?   It looks promising locally.
 // Futility pruning: parent node
          if (predictedDepth <= 7 * ONE_PLY)


@coding46 thanks for refactoring this . . . suggest to remove commented code.


Let me see if I can find the file.


My strange `height` formula was an attempt to preserve the shape of the input matrix (square, tall or wide). You may know of a better way to achieve the same effect. It's difficult for the user to guess what will look nice.


Would it ever be needed to recompute these paths?


ready to merge


@coding46 suggest to remove commented code


J'aime :p


sorry, the name of the repo is `detect-conflicts`


		+const bufsize = 128 &redef;

&redef;


"Ore" should have been renamed "filler"


The difference is that GitHub forking is not about "independent work", certainly not "because of different perspectives on how the program should be developed."  GitHub's "fork" means, "make a clone of the repo so you can work on it and eventually submit pull requests back upstream."  It's unfortunate they used "fork" for this, especially since "clone" was available, but there it is.


Thanks guys for completion follow me on g plus HykeemWells


&redef;

reef;


@chrisbutler Why prototype?


@coding46 thanks for adding the test! I'll try to follow your example.


@mgorny I think we need to fix/enhance the devmanual in that aspect.


```
# git remote -v
origin  https://github.com/volumio/Volumio2.git (fetch)
origin  https://github.com/volumio/Volumio2.git (push)

# git status
On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working directory clean

# cd http/www

# git status
HEAD detached at 2b1078f
nothing to commit, working directory clean

# git checkout 9dc9302a80a2f3a3c51d9b9c2b799cff5c13f927
Previous HEAD position was 2b1078f... Tempting System. To fix: gulp build and serve tasks Axiom theme WIP
HEAD is now at 9dc9302... Fixes: mock data updated with correct value types, visibleIf directive updated for correct type, Templating system WIP - Build

# cd ../..

# git diff
diff --git a/http/www b/http/www
index 2b1078f..9dc9302 160000
--- a/http/www
+++ b/http/www
@@ -1 +1 @@
-Subproject commit 2b1078fbc494e764f86dfb6eaab61296daf5699c
+Subproject commit 9dc9302a80a2f3a3c51d9b9c2b799cff5c13f927
```


Thanks we need to still deploy and keep it moving


TeamCity ATK :: QA :: Integration Per Branch [Build 1886](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546370&buildTypeId=Atk_IntegrationPerBranch) outcome was **FAILURE**
Summary: Tests failed: 399 (399 new), passed: 0, ignored: 15, muted: 25; number of failed tests is too large: 399 (new) Build time: 00:29:05
### Failed tests
```
testcases.assign_sample_test.dataSplitter.test_sample_bin (Test splitting on the sample_bin column): <no details avaliable>

testcases.bin_col_test.BinColTest.test_bin_column_cutoff_monad: <no details avaliable>

testcases.bin_col_test.BinColTest.test_bin_column_cutoff_string: <no details avaliable>

testcases.bin_col_test.BinColTest.test_bin_column_cutoff_empty: <no details avaliable>

testcases.bin_col_test.BinColTest.test_bin_column_cutoff_none: <no details avaliable>

testcases.Covar_vect_400ColX2KRows_test.CovarVect400ColX2KRowTest.test_covar_on_vectors (Compute a covariance matrix based on the 400 elements in): <no details avaliable>

testcases.bin_col_test.BinColTest.test_bin_column_cutoff_string_list: <no details avaliable>

testcases.SVM_2D_Slope1_test.Svm2DSlope1.test_svm_model_test (Perform the test against a dataset with the same): <no details avaliable>

testcases.bin_col_test.BinColTest.test_bin_column_name_collision (Validate that default column naming avoids name collision_): <no details avaliable>

testcases.bin_col_test.BinColTest.test_bin_column_bad_col: <no details avaliable>

testcases.bin_col_test.BinColTest.test_bin_column_cutoff_mixed (Error case: cutoffs are not monotonic): <no details avaliable>

##### there are 389 more failed tests, see build details
```


I know what you mean, but it's difficult to enforce size when the user is in a terminal. That can easily backfire. I think by annotating the dimensions the user should have a good idea if the matrix is tall or wide and can adjust the height width accordingly.

Then again I am not a typical end-user of the spy plot. I might take your word for it to enforce some constraint (or at least make it default


This is from https://github.com/ShareX/ShareX/blob/master/ShareX.HelpersLib/Helpers/ShortcutHelpers.cs


Hi,

I was able to resolve all issues and example client seems to be registering with leshan server fine...next step would be resolving issues when pointing to internal lwm2m server...Thanks...


I think that should do it.


Ok I got it. It just seem to complain that it cannot work in current working directory.
git clone ....
npm install

npm WARN cannot run in wd Volumio2@2.0.0-alpha4 git submodule init && git submodule update (wd=/volumio)


What are the consequences of having a hardcoded argument in here? Kinda seems like it ought to be "compile" if anything.


German disks has 13 cabinets only


s/Shuffeled/Shuffled


The check for `m_dht` here is not necessary anymore, is it?


@jteplitz602 you should setup up `git` to globally ignore .swp files, see https://help.github.com/articles/ignoring-files/#create-a-global-gitignore - related to `.index.dart.swp` above


THis function gives us the FIRST sub directory, this is needed for RA2 and YURY
German: AR2
English: RA2


Although the concept of merging the enosys stubs into kfi proper is laudable, the reason it has not been done previously is due to kernel name space pollution. Each enosys_ stub would need to be exported such that providers could link with the stubs and therefore kernel globally visible (hence the pollution).
The patch is rejected at this time.


Thanks again!


Would you like the flat one then, I have been going through finding places where the old icon is used.


callだとコマンドが失敗したときにも先に進んでしまいますが、それだと不正な状態のまま処理が進んで環境を壊したりするかもしれませんので、エラーが起こったら諦めて終わるか、頑張って回復するかのどちらかにしてください。エラー無視は良くないです。


This function returns the index from a file.


Looks good :+1:   Thanks Robin!


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1792 of 1796 tests passed)

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


hi


Yes, I tested it stand-alone:
http://tests.stockfishchess.org/tests/view/5599bd5a0ebc590abbe1b8be


Why are you ignoring the converted logo?


sysは使っていないように見えます


ready for merge if tested


German disks has 13 cabinets. so i decided to check for data1.hdr only.


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This should say "to migrate permissions" or "to migrate permissions and roles".


Also, fixed typo in `TODO.md`


This is my first trial


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-MacOS-10.9-clang: Tests Failed

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


## first edition


関数の分け方が綺麗でいいですね


What do you think about passing in the urls as environment variables?


このコメントいいですね。
後で読んだ人がsleepの意図をよく理解できそうです。


Anytime, glad people are using it..


You can simply use `publisherOutputDir` since the method itself handles parent dir.


This is a crazy line of code. In any sensible programming language, this expression should always evaluate to false and the if-block never entered. But JavaScript is totally nuts and states that `NaN` is not equal to `NaN`. So when `s` is `NaN`, this evaluates to true. Never would have believed it until I read [this StackOverflow question](http://stackoverflow.com/questions/10034149/why-is-nan-not-equal-to-nan).


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 357](https://teamcity.labkey.org/viewLog.html?buildId=267205&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:08
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


I really think it should report the values of `node` and `Num_sexp_nodes`, as well.


That's fine. I guess I was just focused on splitting out the `format"ABC"` specification, which to me was cleaner than relying on the extension for format information. But if you like `file"somefile.ext"` as a shortcut for `query("somefile.ext")`, I'm OK with that. But needs documentation and a test.


Indeed.


is there a spurious space here?


@pranavkm Why was this change made? aspnet release branch should point to azure ad release feed, not nightly.


Can't reproduce.


This line doesn't do anything to my knowledge. What did you intent of doing with this line?


LGTM.


right....


A couple comments:
1. This won't run without SUPRIME_DATA_DIR in my environment.  Is there a way to make this run with test data or some other thing everyone will have?
2. What does the call to displayRGB do?
3. Why do you need to reload cameraGeomUtils?  Should work without, right?
4. There is some code that clutters the notebook.


commandsは推奨されていないパッケージのようなので、subprocessの使用をお勧めします。
http://docs.python.jp/2.7/library/commands.html


ah, yes. I missed this one.


I'm frankly not sure what's best. Let's first see if anybody uses it ;-).


@mickhansen this check breaks code because isNumber doesn't allow for string values which is what you usually want with Decimal since they may be outside the range of js numbers.

What was the desire behind this PR? Why not let the db handle the validation?


これではコマンドが失敗したときにも先に進んでしまいますが、それだと不正な状態のまま処理が進んで環境を壊したりするかもしれませんので、エラーが起こったら諦めて終わるか、頑張って回復するかのどちらかにしてください。
subprocess.check_callを使うのがいいと思います。


@tenderlove : Sorry but typically, what string could have tags not escaped if passed to`unwrapped_html_escape`directly *(“# This method is not for public consumption”)* ?


This effectively changes the meaning of `freemem` but I admit it probably the cleanest way to solve this issue.  We're going to need to remember to revisit this once the ARC is backed by the page cache and therefore is accounted for in NR_INACTIVE_FILE.  But until that that time this seems little a pretty clean fix.  Let me verify it doesn't cause build failures for older kernels and then I'll get it in.  I'm glad this was caught.


Old commit :)


Doing this here actually breaks showCcd.  This needs to be done in showCamera so that showCcd still works.


Maybe the checks help prevent other types of errors which is certainly good. Guess just this use case needs to be handled better.


rax retry tests


There is a similar thing that needs to be done in showCcd.  Do you mind updating the display there as well?


`moment` doesn't appear to be used


Currently this line isn't doing anything!


```java
"Temporal column must be of type timestamp or date" + column.getColumnName()


```java
throw new PrestoException(NOT_SUPPORTED, "Temporal column must be of type timestamp or date" + column.getColumnName());


What does `COVERAGE_FILE` do in this line? It doesn't look used.


Should this be `PLATFORM_MACOSX`?


Maybe update comment to
`how long message is unavailable to others`


Make this a `checkArgument` in `TemporalCompactionSetCreator()`


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1796 of 1796 tests passed)

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


Seems like an issue with our branching.


Could we have a debug flag that would enable logging all shader sources?


what is VPATH mean?


The comment here still refers old name "Scm_VMWriteParameter".


How about:

`However, if you have a post that falls the above categories yet you feel still belongs in HH, you can message a [moderator](#moderators) to discuss it. We've had posts like these in the past that turned into constructive discussions with a bit of reframing. Talking about hard problems is, itself, hard.`


It is setting an environment variable `COVERAGE_FILE` that tells `coverage` where to write its output. Unfortunately there isn't a command line option for this.


Thanks Robin! Very intiguing, and 2 questions:
* Why use ` ` to return code instead of " "?
* What does this ${ } do and must there a method (like htmlSafe) be called after it?

Thanks!


git no esta ignorando los archivos copia que hace linux  por ejemplo este NuevoDocumento, me creo un NuevoDocumento~ oculto que lo borre para que no me lo agregue al repositorio. agregar la regla para que no estos archivos, pienso que con un *.*~ seria suficiente.


You serious? You can have my tabs, when you pry them from my cold, dead hands.

IOW: Resubmit for whole code base. After I've left.


This is very close but there are some go idiomatic ways to write this I thought I'd suggest
here is what I usually get as a recommendation

if err, path := os.Getwd(); err != nil {
fmt.Fprintf(os.Stderr, "%v\n", err)
} else {
fmt.Println(path)
}

Also, thinking a bit more, should we use the log package for errors? I kind of think so because then we'll get a good exit value. So the error could be this:
log.Fatalf("%v", err)

Fatalf won't return and will exit with a non-zero value.

I've done a bad job of following this convention but maybe now is a good time to start.


It's more like that the binder in `MethodCallee` never binds anything than anything specific to overloaded deref.


you can extract this in a separate method. Something like "CountSetBits(int n)".


ewww


Changed in 6591cdfae15c3e9ec9b75ebb6747adedabd9fe72.

I still don't like the now-required tight coupling, e.g. the `buddylist` and `chat` directives and the `uicontroller` now have to know that the `identity` events are triggered by the `endToEndEncryption` service. I would rather have a global `events` service that acts as hub for events triggered by any component and that can be subscribed from anywhere.


I'm trying to figure out how this doesn't turn into hell


This one looks like it hasn't changed.


I do like the flat icon, so yes, that would be the one I'd like to bundle
(can't be mistaken for the old one, looks good on a menubar with a bunch of
flat apps)

On Tue, Aug 18, 2015 at 5:55 PM Tom Lynch <notifications@github.com> wrote:

> Would you like the flat one then, I have been going through finding places
> where the old icon is used.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/replicatorg/ReplicatorG/commit/dea45d796dd1aba703f14ab898587164f356c7bd#commitcomment-12772668>
> .
>


if not use point of this, is it ok?


Just as I had done the icons with the gradient... hold on... I'll submit another pull request in a mo... I'll put the flat icon in there also...


According to tests this file is not available there. But I'll request members to check it. I don't have Wifi here.


unit tests should be sufficient to test the code. This is a 'driver' method and you have 3 of them. Please consider consolidating them.


should add const after two_loop function?


From writing convenience it does not offer much, but I find it more understandable. Though, I dislike the idea of having two functions doing exactly the same. That is just asking for user confusion.
I'll over think it. I might be misusing `query` and should just use `load` instead.


You could always get creative and use `arr.reduce` ;)


remove commented code


Bitte Beispielkonfiguration mitliefern.

On Tue, Aug 18, 2015 at 11:47 PM, Felix <notifications@github.com> wrote:

> Hab mal testweise Google+ in die configs und unter interactions erscheinen
> die von facebook
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/secure-software-engineering/rose/commit/5958628f99d8ee5d5162581a6f4d77df305283e3#commitcomment-12771507>
> .
>


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (748 of 748 tests passed)

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


remove


should we add const after line_search function? if not , why?


![giphy 11](https://cloud.githubusercontent.com/assets/736728/9345484/dc1fc05c-45e7-11e5-82a4-75dc2beae08e.gif)


Don't bother, the current one's great.


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1276 of 1276 tests passed)

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


space after bracket


space


I don't mind won't take a moment to put the flat icon in...


Sure! I'll bump it up to 1.2 :D before I merge into develop


Yeah however it's code that was moved around.


94006714-MultipleTowers (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1801 of 1801 tests passed)

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


Is there a reason this is called twice? (see line 516)


It is possible with an if else statement in the #twitter method

check the nodejs implementation for reference

On Tue, Aug 18, 2015 at 11:44 PM, RKushnir <notifications@github.com> wrote:

> Thanks. Now I see. I was thinking that you somehow cusomized satellizer
> for this 2-step flow.
> Maybe it's possible to handle both steps in a single Rails action, to
> avoid having a specialized JS controller.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sahat/satellizer/commit/0b781cfa8a83ae9fd4a67230607b8d5dbe8dc73f#commitcomment-12770346>
> .
>


Should be handled by this going forward - https://github.com/aspnet/Universe/pull/262


Typo: `remote` -> `remove`


it seems ok


Those tests I done it when they changed this.


Almost suggested a switch statement but then I double checked and for whatever reason python doesn't have it. Why?!


I just checked deployment of ARTIQ on a fresh install of 14.04 and confirm
no errors. Using m-labs.hk conda.

On Mon, Aug 17, 2015 at 9:53 AM, Yann Sionneau <notifications@github.com>
wrote:

> I rebuilt and uploaded llvmlite-artiq for linux-64 : built on Ubuntu 14.04
> LTS.
> That should fix the issue of needing a recent libstdc++
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/m-labs/artiq/commit/075bf331ac440dcab49ee73a522acfe2230a0104#commitcomment-12741819>
> .
>


LGTM


Not sure where this is used, wanted to point it out to make sure this is right


```JSON
{
    "roseCommentsIsEnabled": true,
    "roseCommentsRatingIsEnabled": true,
    "salt": "ROSE",
    "hashLength": 5,
    "repositoryURL": "https://secure-software-engineering.github.io/rose/example/base.json",
    "autoUpdateIsEnabled": true,
    "updateInterval": 28800,
    "fileName": "rose-default-configuration.json",
    "fingerprint": "25E769C697EC2C20DA3BDDE9F188CF170FA234E8",
    "timestamp": 1437753156000,
    "networks": [{
        "name": "google",
        "descriptiveName": "Google+",
        "identifier": "plus.google.com"
        },{
        "name": "facebook",
        "descriptiveName": "Facebook",
        "identifier": "facebook.com",
        "observers": [{
            "name": "chat",
            "network": "facebook",
            "type": "input",
            "priority": 1,
            "version": "0.1",
            "patterns": [{
                "node": ".fbNubFlyoutInner textarea.uiTextareaAutogrow",
                "container": ".fbNubFlyoutInner",
                "extractor": "chat-user-link"
            }, {
                "node": "#pagelet_web_messenger textarea.uiTextareaAutogrow",
                "container": "#pagelet_web_messenger div[role=\"main\"]",
                "extractor": "chat-user-link"
            }]
        }, {
            "name": "chat-reply",
            "network": "facebook",
            "type": "click",
            "priority": 1,
            "version": "0.1",
            "patterns": [{
                "node": "#pagelet_web_messenger .uiButtonConfirm input",
                "container": "#pagelet_web_messenger div[role=\"main\"]",
                "extractor": "chat-user-link"
            }]
        }, {
            "name": "unlike-comment",
            "network": "facebook",
            "type": "click",
            "priority": 1,
            "version": "0.2",
            "patterns": [{
                "node": ".UFICommentActions .UFILikeLink:contains(\"Unlike\"), .UFICommentActions .UFILikeLink:contains(\"Gefällt mir nicht mehr\")",
                "container": ".userContentWrapper",
                "extractor": "status-update"
            }]
        }, {
            "name": "unlike-content",
            "network": "facebook",
            "type": "click",
            "priority": 2,
            "version": "0.3",
            "patterns": [{
                "node": ".UFILinkBright span",
                "container": ".userContentWrapper",
                "extractor": "status-update"
            }]
        }, {
            "name": "unlike-photo",
            "network": "facebook",
            "type": "click",
            "priority": 2,
            "version": "0.2",
            "patterns": [{
                "node": ".fbPhotosPhotoLike span.unlike",
                "container": ".fbPhotoSnowliftPopup"
            }, {
                "node": ".UFILikeLink span:contains(\"Unlike\"), .UFILikeLink span:contains(\"Gefällt mir nicht mehr\")",
                "container": "#fbPhotoPageContainer"
            }, {
                "node": ".UFILikeLink span:contains(\"Unlike\"), .UFILikeLink span:contains(\"Gefällt mir nicht mehr\")",
                "container": ".fbPhotoSnowliftPopup"
            }]
        }, {
            "name": "unlike-page",
            "network": "facebook",
            "type": "click",
            "priority": 3,
            "version": "0.1",
            "patterns": [{
                "node": ".PageLikeButton:contains(\"Unlike\"), .PageLikeButton:contains(\"Gefällt mir nicht mehr\")",
                "container": ".PageLikeButton"
            }]
        }, {
            "name": "like-comment",
            "network": "facebook",
            "type": "click",
            "priority": 4,
            "version": "0.2",
            "patterns": [{
                "node": ".UFICommentActions .UFILikeLink:contains(\"Like\"), .UFICommentActions .UFILikeLink:contains(\"Gefällt mir\")",
                "container": ".userContentWrapper",
                "extractor": "status-update"
            }]
        }, {
            "name": "like-content",
            "network": "facebook",
            "type": "click",
            "priority": 5,
            "version": "0.3",
            "patterns": [{
                "node": ".UFILikeLink span",
                "container": ".userContentWrapper",
                "extractor": "status-update"
            }]
        }, {
            "name": "like-photo",
            "network": "facebook",
            "type": "click",
            "priority": 2,
            "version": "0.2",
            "patterns": [{
                "node": ".fbPhotosPhotoLike span.like",
                "container": ".fbPhotoSnowliftPopup"
            }, {
                "node": ".UFILikeLink span:contains(\"Like\"), .UFILikeLink span:contains(\"Gefällt mir\")",
                "container": "#fbPhotoPageContainer"
            }, {
                "node": ".UFILikeLink span:contains(\"Like\"), .UFILikeLink span:contains(\"Gefällt mir\")",
                "container": ".fbPhotoSnowliftPopup"
            }]
        }, {
            "name": "like-page",
            "network": "facebook",
            "type": "click",
            "priority": 1,
            "version": "0.1",
            "patterns": [{
                "node": ".PageLikeButton:contains(\"Like\"), .PageLikeButton:contains(\"Gefällt mir\"),.PageLikeButton:contains(\"Like\") *, .PageLikeButton:contains(\"Gefällt mir\") *",
                "container": ".PageLikeButton"
            }]
        }, {
            "name": "view-profile",
            "network": "facebook",
            "type": "click",
            "priority": 5,
            "version": "0.2",
            "patterns": [{
                "node": "a.friendHovercard * , a[data-hovercard*=\"user\"] *, a[href*=\"=ufi\"] *, a.titlebarText, a[href*=\"=ufi\"][data-hovercard], a[data-hovercard*=\"user\"], .webMessengerMessageGroup strong > a, #webMessengerHeaderName a",
                "container": "a",
                "extractor": "user-link"
            }]
        }, {
            "name": "comment-reply",
            "network": "facebook",
            "type": "input",
            "priority": 6,
            "version": "0.1",
            "patterns": [{
                "node": ".UFIReplyList *",
                "container": ".userContentWrapper",
                "extractor": "status-update"
            }]
        }, {
            "name": "comment-content",
            "network": "facebook",
            "type": "input",
            "priority": 7,
            "version": "0.1",
            "patterns": [{
                "node": ".UFIAddCommentInput *",
                "container": ".userContentWrapper",
                "extractor": "status-update"
            }]
        }, {
            "name": "comment-content",
            "network": "facebook",
            "type": "input",
            "priority": 7,
            "version": "0.1",
            "patterns": [{
                "node": ".UFIAddCommentInput *",
                "container": ".userContentWrapper",
                "extractor": "status-update"
            }, {
                "node": ".UFIAddCommentInput *",
                "container": ".fbPhotoSnowliftContainer"
            }, {
                "node": ".UFIAddCommentInput *",
                "container": ".fbxPhotoContentContainer"
            }]
        }, {
            "name": "upload-photo",
            "network": "facebook",
            "type": "click",
            "priority": 8,
            "version": "0.1",
            "patterns": [{
                "node": "form[action*=\"upload/photos\"] button[type=\"submit\"]",
                "container": "form"
            }]
        }, {
            "name": "upload-video",
            "network": "facebook",
            "type": "click",
            "priority": 8,
            "version": "0.1",
            "patterns": [{
                "node": "form[action*=\"video/upload\"] + div form[action=\"#\"] button[type=\"submit\"]",
                "container": "form[action=\"#\"]"
            }, {
                "node": "div:contains(\"Upload Video\") ~ div button[type=\"submit\"]",
                "container": "div[role=\"dialog\"]"
            }]
        }, {
            "name": "update-status",
            "network": "facebook",
            "type": "click",
            "priority": 9,
            "version": "0.1",
            "patterns": [{
                "node": "form[action*=updatestatus] button[type=\"submit\"]",
                "container": "form[action*=updatestatus]"
            }]
        }, {
            "name": "share-status",
            "network": "facebook",
            "type": "click",
            "priority": 9,
            "version": "0.1",
            "patterns": [{
                "node": "a[ajaxify*=\"share/dialog/submit\"] *, a[ajaxify*=\"share/dialog/submit\"]",
                "container": "a"
            }, {
                "node": "form[action*=\"sharer/submit\"] button[type=\"submit\"]",
                "container": "form[action*=\"sharer/submit\"]"
            }]
        }, {
            "name": "create-event",
            "network": "facebook",
            "type": "click",
            "priority": 10,
            "version": "0.1",
            "patterns": [{
                "node": "form[action*=\"plans/create\"] button[type=\"submit\"]",
                "container": "form[action*=\"plans/create\"]"
            }]
        }, {
            "name": "join-event",
            "network": "facebook",
            "type": "click",
            "priority": 9,
            "version": "0.1",
            "patterns": [{
                "node": "a[ajaxify*=\"/join.php\"] *, a[ajaxify*=\"/join.php\"], a[ajaxify*=\"response=join\"]",
                "container": "a"
            }]
        }, {
            "name": "associate-event",
            "network": "facebook",
            "type": "click",
            "priority": 9,
            "version": "0.1",
            "patterns": [{
                "node": "a[ajaxify*=\"/associate.php\"] *, a[ajaxify*=\"/associate.php\"]",
                "container": "a"
            }]
        }, {
            "name": "decline-event",
            "network": "facebook",
            "type": "click",
            "priority": 9,
            "version": "0.1",
            "patterns": [{
                "node": "a[ajaxify*=\"/decline.php\"] *, a[ajaxify*=\"/decline.php\"]",
                "container": "a"
            }]
        }, {
            "name": "friend-request-add",
            "network": "facebook",
            "type": "click",
            "priority": 9,
            "version": "0.1",
            "patterns": [{
                "node": ".stickyHeaderWrap .button.FriendRequestAdd, .stickyHeaderWrap button.FriendRequestAdd *, .uiBoxWhite .button.FriendRequestAdd, .uiBoxWhite button.FriendRequestAdd *",
                "container": "#globalContainer"
            }, {
                "node": ".friendBrowserListUnit button.FriendRequestAdd, .friendBrowserListUnit button.FriendRequestAdd *",
                "container": ".uiProfileBlockContent"
            }, {
                "node": ".hovercardButtonGroup button.FriendRequestAdd, .hovercardButtonGroup button.FriendRequestAdd *",
                "container": ".uiContextualLayer > div > div > i + div"
            }, {
                "node": "button.FriendRequestAdd, button.FriendRequestAdd *",
                "container": ".clearfix"
            }]
        }, {
            "name": "unfriend",
            "network": "facebook",
            "type": "click",
            "priority": 10,
            "version": "0.1",
            "patterns": [{
                "node": ".FriendListUnfriend *",
                "container": "#globalContainer"
            }, {
                "node": ".FriendListUnfriend *",
                "container": "will never be reached"
            }]
        }, {
            "name": "friend-request-confirm",
            "network": "facebook",
            "type": "click",
            "priority": 11,
            "version": "0.1",
            "patterns": [{
                "node": ".requestStatus button[name=\"actions[accept]\"]",
                "container": ".requestStatus"
            }, {
                "node": "#pagelet_above_header_timeline a[role=\"button\"][ajaxify*=\"add_friend\"], .stickyHeaderWrap a[role=\"button\"][ajaxify*=\"add_friend\"], .requestResponseMenu li.accept *",
                "container": "#globalContainer"
            }]
        }, {
            "name": "friend-request-reject",
            "network": "facebook",
            "type": "click",
            "priority": 12,
            "version": "0.1",
            "patterns": [{
                "node": ".requestStatus button[name=\"actions[reject]\"]",
                "container": ".requestStatus"
            }, {
                "node": "#pagelet_above_header_timeline a[role=\"button\"][ajaxify*=\"reject\"], .stickyHeaderWrap a[role=\"button\"][ajaxify*=\"reject\"], .requestResponseMenu li.reject *",
                "container": "#globalContainer"
            }]
        }],
        "extractors": [{
            "name": "status-update",
            "network": "facebook",
            "version": "0.2",
            "fields": [{
                "name": "sharerId",
                "selector": "> div > div > a",
                "attr": "href",
                "match": ".+profile\\\\.php\\\\?id=\\\\d+(?=\\\\&)|.+(?=\\\\?)|.+",
                "hash": true
            }, {
                "name": "contentId",
                "selector": "> div > div.clearfix > div > div > div > div > div > span > span > a:first-child",
                "match": ".+(?=\\\\?)|.+",
                "attr": "href",
                "hash": true
            }]
        }, {
            "name": "user-link",
            "network": "facebook",
            "version": "0.1",
            "fields": [{
                "name": "userId",
                "attr": "href",
                "match": ".+profile\\\\.php\\\\?id=\\\\d+(?=\\\\&)|.+(?=\\\\?)|.+",
                "hash": true
            }]
        }, {
            "name": "chat-user-link",
            "network": "facebook",
            "version": "0.1",
            "fields": [{
                "name": "userId",
                "selector": "h4.titlebarTextWrapper > a.titlebarText, #webMessengerHeaderName a",
                "attr": "href",
                "match": ".+profile\\\\.php\\\\?id=\\\\d+(?=\\\\&)|.+(?=\\\\?)|.+",
                "hash": true
            }]
        }, {
            "name": "privacy-settings",
            "network": "facebook",
            "version": "0.1",
            "type": "url",
            "informationUrl": "https://www.facebook.com/settings/?tab=privacy",
            "interval": 5000,
            "container": ".fbSettingsSections",
            "fields": [{
                "name": "viewFuturePosts",
                "attr": "content",
                "selector": ".fbSettingsSectionsItem:nth-child(1) .fbSettingsListItem:nth-child(1) .fbSettingsListItemContent > div:last-child"
            }, {
                "name": "sendFriendRequests",
                "attr": "content",
                "selector": ".fbSettingsSectionsItem:nth-child(2) .fbSettingsListItem:nth-child(1) .fbSettingsListItemContent > div:last-child"
            }, {
                "name": "MessageFilter",
                "attr": "content",
                "selector": ".fbSettingsSectionsItem:nth-child(2) .fbSettingsListItem:nth-child(2) .fbSettingsListItemContent > div:last-child"
            }, {
                "name": "EMailLookup",
                "attr": "content",
                "selector": ".fbSettingsSectionsItem:nth-child(3) .fbSettingsListItem:nth-child(1) .fbSettingsListItemContent > div:last-child"
            }, {
                "name": "PhonenumberLookup",
                "attr": "content",
                "selector": ".fbSettingsSectionsItem:nth-child(3) .fbSettingsListItem:nth-child(2) .fbSettingsListItemContent > div:last-child"
            }, {
                "name": "SearchEngines",
                "attr": "content",
                "selector": ".fbSettingsSectionsItem:nth-child(3) .fbSettingsListItem:nth-child(3) .fbSettingsListItemContent > div:last-child"
            }]
        }]
    }]
}
```


Might be an idea to backport this to 1.2.


For the same reason that we have to manually number the elements in Python enums.


All operators need to have their rvalue arguments be actual rvalues to avoid stuff like #27054.


What's the reasoning for copter not allowing editing of EKF USE?


Just a note here. Take care using ammend right to the master branch.
Use it only in separated branches. Probably any user with an old copy of master (like me) will have problems from now on.


Asked help herehttp://forum.xda-developers.com/oneplus-one/development/rom-cm-12-builds-t3045053/page373#post62395295

Will report back


Syno was a guess, sorry. However with Clinton's qpkg it is removed... sickbeard-tvrage.sh


@dr-dimitru that's how the controller stores the params. it's the only way to access them


Odd message and the line number is wrong.


around here, should we write the actual shortcuts in here? i.e.
> to push the current line (`CTRL-ENTER`)


May as well not declare the types, and just have it as `String`


		-	// We use the smallest time-out possible to return almost immediately if
+	// ### We use the smallest time-out possible to return almost immediately if
 	// no packets are available. (We can't use set_nonblocking() as it's
 	// broken on FreeBSD: even


@rmcd1024  did you use a text editor that added hard returns here? just curious :wink:


Why not `ServiceInstance<ZookeeperInstance>` as it was?


Could these be refactored to something like self.rankedToggles = { Factions.AEON : SomeToggleClass() ... }


Is there a similar problem with using newRV_noinc() in __push_header()?


@alexblair1 is the branch/commit I should be working with? it's 16 hours old...


Could this be done using similar magic like #attributes= ? It looks vaguely similar to how that would function.


forgot to remove this file XD


Hmm that odd let me push everything up and I'll get back to you with the
latest version.
On Tue, Aug 18, 2015 at 5:45 PM dothework <notifications@github.com> wrote:

> @alexblair1 <https://github.com/alexblair1> is the branch/commit I should
> be working with? it's 16 hours old...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alexblair1/Bait/commit/abfad1b81e7af5bca2a3d1eba56c650f4e0b8fae#commitcomment-12773332>
> .
>


Are you suggesting to use log.Fatalf, right?

Em terça-feira, 18 de agosto de 2015, ron minnich <notifications@github.com>
escreveu:
> This is very close but there are some go idiomatic ways to write this I
thought I'd suggest
> here is what I usually get as a recommendation
>
> if err, path := os.Getwd(); err != nil {
> fmt.Fprintf(os.Stderr, "%v\n", err)
> } else {
> fmt.Println(path)
> }
>
> Also, thinking a bit more, should we use the log package for errors? I
kind of think so because then we'll get a good exit value. So the error
could be this:
> log.Fatalf("%v", err)
>
> Fatalf won't return and will exit with a non-zero value.
>
> I've done a bad job of following this convention but maybe now is a good
time to start.
>
> —
> Reply to this email directly or view it on GitHub.<
https://ci4.googleusercontent.com/proxy/1io6Bxma7Byw5t7r2TMdHoiuRYGST6Qj-lx_gSwsgGWd0vfEc1xmd3GVeeQbgXjp8Y15IZUvErB7KDpr9RPAJ75flSGwFzgyuTQrDgTEM3QfKRsriWNmg0X-LXsU0SerI-5cKC_pYewxyOjpWIWoJd97fhFpEw=s0-d-e1-ft#https://github.com/notifications/beacon/AFaXJbcj9j8dBzZ_1aBR7J99kt8Lz0x2ks5oo7aygaJpZM4Ft-2X.gif
>


mobs_sheep_white.png
Need to be tested, i think this texture needs to be our modified sheep texture (or maybe a more white version of our texture?)


Oh yeah, that fixes #108 as noted in my commit remark.


Great work !


spelling ("reselect"). Also I feel like this is repeating much of the information already in the Tip callout.


Here's the fresh repo

https://github.com/alexblair1/Bait/tree/new_feature

On Tue, Aug 18, 2015 at 5:46 PM Alex Blair <alexblair190@gmail.com> wrote:

> Hmm that odd let me push everything up and I'll get back to you with the
> latest version.
> On Tue, Aug 18, 2015 at 5:45 PM dothework <notifications@github.com>
> wrote:
>
>> @alexblair1 <https://github.com/alexblair1> is the branch/commit I
>> should be working with? it's 16 hours old...
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/alexblair1/Bait/commit/abfad1b81e7af5bca2a3d1eba56c650f4e0b8fae#commitcomment-12773332>
>> .
>>
>


Pssst ... doet Node-Red niet wat je wilt? http://github.com/node-red/node-red


@alexblair1 excellent.


> アイコンがセクシー... エロいっ！


Is there a way to link to that lesson here? @naupaka I feel like there must be..


This solves the UI problem but it's not addressing performance. Hopefully this list will not get too long right?


Do you need to initialize the stream->endpoint too? There would be 2 bytes uninitialized here too?
i.e at label update_addr:
	stream->endpoint.ip46 = fsin->sin6_addr;
	stream->endpoint.port = ntohs(fsin->sin6_port);

Maybe it would be more efficient to compare the members of struct rather that doing memset?
```
int tmp = memcmp(&endpoint.ip46, &stream->endpoint.ip46, sizeof(endpoint.ip46));
if(!tmp) tmp=endpoint.ip46.port-stream->endpoint.port;
```


missed that one, thanks


optional?


perhaps "run" instead of "call"? Probably confusing for beginners (even though technically correct)


wrong doc


hayy


Hi @istiaqueahamed - thanks for forking the Mozilla Study Group repo! How are things going? Let me know if you need any help getting set up!


nice one, I will update the others that don't use this.


No because the streams (and all other instances I could find) are nulled out when allocated.


Hey @jhollist - how are things going? Thanks for checking out the Mozilla Study Groups framework; let me know if there's anything I or Mozilla can do to help & support!


By doing it this way we will prevent new misues from creeping in. This will make the developers accountable for new code.


rax retry tests


Just found out that the storage definition changed for "what was the reason again?". Query methods need to be changed now.


can i use remote url if i installed php 5.6?
and how can i add new server for files


Well... so much for the github.io idea, afaik github.io doesn't support php.


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/16476/testReport/) for this commit**


Hej @minimax59,
I totally missed your tweet back then! You should have create an issue here. However, it's fixed now. I'll add you to changelog as well for coming up with the same fix.


Try to keep your code clean. This might be the result of copy-pasting segments of code, but keep spacing and indentations consistent. Now it's a little bit painful to look at it


This concept of IdOfFirstAncestor is important for page migration, too. I implemented basically the same thing, though I didn't pull it out into a method or achieve it exactly the same way. It would be nice to rework my change to use yours.


Very good, thanks! I will comment more later.


Nice, I didn't know that, thanks!

On Mon, Aug 17, 2015 at 1:04 PM, Thomas B. Mooney <notifications@github.com>
wrote:

> Using prove you can pass arguments to the test using :: after all the
> prove options and before the options to the test, e.g.
>
> prove Genome/Ptero/IntegrationTest.pl :: '/path/to/test/'
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/davidlmorton/genome/commit/2e01a156dd073bad3abd4f06bf863ccb2fc2081f#commitcomment-12744484>
> .
>


This does not fix anything, you just removed part of the queueing prototype


any news?


I think that, in the future, we may want to restrict this kind of changes via the use of Python's `@property` annotations.


For compact things like `valid`, this compact form is better. But it's less used because you can't express very much with it.


I am having trouble using your source code. I pull and run it in the simulator, but it doesn't appear to do anything. Can you help me?


`go get` ?


Yes, this is a "linked list" and, in fact, all the course can be regarded as a tree structure. More on that later.


Not even if we ran out of memory, or disk space, or encountered a null reference? I don't suppose there's any way to pick out the exceptions that it would help the user to know about, or that should be reported as crashes...but we'll be sorry if we start getting bug reports that just say "ctrl-N stopped working". I guess to me this feels like a case for a yellow box..."Bloom could not add a page. We're not sure why. If this keeps happening please click the button and tell us..."


Shit, I knew I left one of those somewhere but forgot which branch.  Thanks for catching that.


can we add a rule called `deps-dev` and move all development requirement to there?


....humbug. Wanted this on a separate branch/pull.


these are the last typo fixes for german translation. hopefully i got them all :)


I forgot here, but shouldn't a subsection have a list of units? (The same for a course having a list of sections?)


test


s/Resourses/Resources/


Hmm, hadn't seen that when I wrote the previous comment. Makes it better, at least. Would it be worth logging the stack dump, so  if a user reports that Bloom keeps bringing up a dialog when the doc says it should just add a page, we have something to ask him for?


It works like u-boot or other bootloader. there is no bootloader before it because of this environment. so it wait for calling from kernel.


Needs to be rebased on plugins-migrations to be compatible with the new migrations


http://stackoverflow.com/questions/400113/best-way-to-implement-keyboard-shortcuts-in-a-windows-forms-application says you can intercept a control key anywhere in a window by overriding Form.ProcessCmdKey()


Magic Numbers. If these are constants though, you could move them to the top of the file. `BALLS_GRAV_CONSTANT = 1.05` (or whatever the hell 1.05 is)


I am still working on it. That's going to be the last thing that I do .


this is a kind of silly solution for the connecting problem


Gotcha, thanks for the explanation!


This file is not supposed to be part of our repository. Please add to the .gitignore file


Thanks. As you probably know this is the first time messing with a kernel. Thank you very much for pointing it out


asd


Put these into your .gitignore file


Just a reminder


I'd be happier about this if we could make ctrl-Z undo it. Is there any likelihood that a user who didn't get the kind of page he wanted will have trouble figuring out how to delete it? Of course we're going to do "change the type of this page" sometime soon, aren't we?


Yeah, it's an empty class file. Fixing it was part of my next commit which is half done


Look here for the fix
https://github.com/DespairFactor/N6/commit/c2dee13d8ea67b4a90cf51ec43f561ad5f3e7e78


We could automate the build process with [GitHub Releases](https://help.github.com/articles/creating-releases/) + MinGW on [Travis CI](http://docs.travis-ci.com/user/deployment/releases/), perhaps. I will investigate it.


Thank you very much for pointing it out. It has been reverted. I have a bunch of local stuff merged but since I am going to do more tomorrow, Ill push it all at the same time. i will look at the fix you have provided


good


Also, I know what `og` and `fg` are, but you could use `final_gravity` and not hurt your fingers too badly :P


@aleos89

    if (!strcmp(command,"unitwalk")) {


None of these suggestions are things I think must be done (except for making ctrl-N always work somehow); just options to consider.


Okay, i will do that.


Pesky integers breaking everything. Good fix :)


typo?


Yes


101242918_EvapCndPump (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1798 of 1798 tests passed)

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


Yep, we can definitely use AngularJS. But, even with AngularJS, how do you plan to separate the header and footer?


why can't you use 15+0.05 for STC ?


101242918_EvapCndPump (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1798 of 1798 tests passed)

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


nit: there's an extra space after "anchor"


The way these are named these feel like Commands instead of Queries. Views shouldn't cause any change in state (even memory state), they should only ask about what's in a model. `calc_abv` makes me think that it's not already calculated before this point. But you could have one calculate method in the controller and then all of these will magically act like properties.


This should be indented by 2, not 4.


can we move </section> to the next line and deindent by 2? Seems more symmetric.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.438.57ad4e8627db317d0e2f2e80ce0a732378cfe2a2](http://build.palaso.org/viewLog.html?buildId=61383&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:25:36


ok


I didn't notice this before, but I think it would be good to indent <head> and all the stuff below it (until the closing </html> tag) by 2 (similar to the rest of the file). This seems to apply to the other files as well.


Do not grade--correct version already uploaded.


It is remaining stuff of adding vcpu. And vmpidr works like vcpu id in this situation when linux guest because there is no vcpu. pull requesting code is assumed in bmguest situation.
Okay, i will remove it and explain more thing from it after.


Perhaps the titles of these pages should be different?


I'm currently very short on time.


These could be nice methods like `@recipe.bittering_hops` or if they're an object themselves, `@recipe.bittering_hops.to_s`


@liangchenye No,it is OK now, we have found out the problem and fix it in testcase itself. In framework ocitd, both of the systemd and golang exec can work well.


Please grade this one.


`let` is considered better style.


Also, one other thought -- what's your general intention for the organizational structure of the github pages site -- are you thinking of preserving the old wiki structure, or doing something new? I was thinking that it might be sensible to combine some of the shorter pages into longer ones, since scrolling is a pretty natural thing to do.

Basically I'm wondering how navigation would work.


Don't grade this one, either.


`kind_of` suffers the same problems as `respond_to` tests. It's just not that helpful of a test, but it does make certain that typos will haunt you :)


Hassan made these changes and I just committed them.


101242918_EvapCndPump (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This can be done in the factory as well. I think with `association :yeast` or something like that.


yeah, and it works like that. The VMPIDR is be the MPIDR in non-secure PL1.


okey :+1:


Do not grade this one, either.  Sorry, this program is slow.


Although gnatsd is blessed version, could be talking to the ruby-server or another type in the future not written in Go. Don't think its needed here.


I stumbled on this commit because I've just been dealing with `ResourceVanished` myself while communicating with AWS. So, I don't have much context, but could `\(SomeException _) ->` be too broad? I used a pattern similar to this recently:

```haskell
import qualified GHC.IO.Exception as GHC (IOErrorType (ResourceVanished))
```
```haskell
case e of
  HTTP.FailedConnectionException2 _host _port _secure e' ->
    case ioeGetErrorType <$> (fromException e' :: Maybe IOError) of
      Just GHC.ResourceVanished -> throwIO ConnectionClosedByPeer
```


Thanks! i will apply your advice.


Ok fair enough. I will close.


Hopefully this isn't necessary. If it is causing problems, there are bugs in your database cleaner which would make me sad. The `database_cleaner` gem might be helpful here, but I'm not sure if it's necessary.


why


OK, very well. I think this is OK to merge then.


I talked some time ago with @iemejia about not treating the youtube urls as separate and just putting them on the list of resources, but he didn't agree.

I can see his reasoning, as, usually, some courses have the youtube video as their "primary thing". On the other hand, it may be worth experimenting with creating these classes in something like a tree, as in:

* each Course contains a list of Sections
* each Section contains a list of SubSections
* each Subsection contains a list of Unit
* each Unit contains a list of "Things"

Here, "Things" would be any kind of resource, like Youtube URLs, links to PDFs, links to source code, spreadsheets etc. that the instructors may happen to leave for the students.

This way, we capture the essence of what we scrapped from various pages in our own tree structure and we can traverse it as much as we please.

Perhaps we can accomodate @iemejia's desire of distinguishing some URLs by having multiple kinds of "Things", perhaps a "YoutubeVideo" or something like that?

I would sincerely love to have the tree structure as I mentioned above, especially if we provide `__repr__` and `__str__` methods to the classes, which will be super helpful for debugging purposes.

For instance, I would love to be able to print one course (say, `print(cur_course)`, where `cur_course` is a Course) and have all the tree structure dumped, so that I can diff two runs of the script and see if we are being deterministic, if edX may be randomizing what they are supplying us, if we are indeed grabbing things more than once (see my previous comments of the script downloading some videos multiple times) etc.

In fact, one of the worst "offenders" to this downloading the same video multiple times was Stanford's Convex Optimization course (recommended, BTW!). Each week, there was only 1 video with more than 1 hour and there were some weeks where we were capturing the same video more up to 9 times.

Printing a course and doing a post-morten analysis would be super helpful to debug this kind of behavior. Also, now that we started using the `logging` module, we should be littering the code with those, so that we ask users to enable the debug command line option and see the difficulties that they are experiencing.

Comments? If any of you think that what I wrote above is helpful/worthwhile, feel free to copy the parts and file an issue with my observations.

Thanks,

Rogério.


fdf


101242918_EvapCndPump (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1798 of 1798 tests passed)

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


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.19.86f073a9e3167cd06ef1db60736f42448e4b60db](http://build.palaso.org/viewLog.html?buildId=61381&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:35:49


OK!


This is awesome!  I choose: Resident Evil, Half-life, Final Fantasy

One thing I noticed is the names for your custom subclasses of UITableViewController, MainList and SubList.  It's a common practice to drop off the prefix, 'UI', and put your own custom prefix. It doesn't change anything functionally but it's good practice.  It makes it a lot easier to identify things, especially as your program gets larger.

So instead maybe calling them something like CQMainTableViewController & CQSubTableViewController.

[Here](http://cocoadevcentral.com/articles/000082.php) is a good article that explains this in more detail. This [official documentation](https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/CodingGuidelines/CodingGuidelines.html) goes over naming conventions but actually doesn't mention too much about this in particular.


:clap:


mergered for synax error


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.20.57ad4e8627db317d0e2f2e80ce0a732378cfe2a2](http://build.palaso.org/viewLog.html?buildId=61382&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:37:33


Would you like to tweak this commit up a bit, including adding the suggested comment and editing the commit message, so it can be merged? Or would you like me to come up with something?


98812296-SkyTemperatureShiftsSchedules-4866 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

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


LGTM


Phone should be a CharField() type. Fixing in next commit.


erm... ¿

Windows detected.


增加增加appVerify校验头


I have to say...

Thanks a lot !!


Для единообразия стоит сделать здесь от 0 до 6, а внутри `i+1`.


Потерялась отписка.


I think the API controller should inherit from UmbracoAuthorizedApiController instead.
http://issues.umbraco.org/issue/U4-4990
http://umbraco.com/follow-us/blog-archive/2014/1/17/heads-up,-breaking-change-coming-in-702-and-62.aspx


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.439.57ad4e8627db317d0e2f2e80ce0a732378cfe2a2](http://build.palaso.org/viewLog.html?buildId=61384&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:19:43


I didn't really change anything.


I love you guys. SWS4EVAR!


Doesn't this commit mid-kid/CakeHax@284052f183cef8a548ffeae4baf5348129a1c7c8 changes the offset?


utf-8 [>>](http://mathworld.wolfram.com/MuchGreater.html) 1252


sry, are they supposed to be instances of Error? pls fix if so


Hi Josh,
I found a problem , it only work if post title value is numeric or English.It didn't work for Chinese.


silly silly silly


@kot2002 yeah, so he should have not synced submodule codes


This releases the audio resources such as samples that have been loaded up to now. This means sound effects are not played during the second game.


LGTM, I'm sorry I didn't include this in my PR!


This should return Vector2&


You are missing the operators for multiply and divide as well as all operators with scalar parameters.


Yup, should change that. And no waits... too late for that. =P


Yup, should change that. It was a quick fix after all. =P

On Sun, Aug 16, 2015 at 12:35 PM, Robert Krawczyk <notifications@github.com>
wrote:

> f64 static inline abs(f64 a) ? or maybe use standard fabs (
> http://en.cppreference.com/w/c/numeric/math/fabs) ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KrossX/Durazno/commit/faf6d6255a3355e68339694af4d546cd15ad8b25#commitcomment-12728126>
> .
>


You can strip out this semicolon :)


Much better!


I didn't profile FLAnimatedImage specifically when I made this change, but I did just do a pretty big investigation into performance issues in Flipboard and this didn't show up near the top.

Profiling a bit now, here's the sample project running on my iPad Air with these changes applied (at e638ebf5e507cc64a985e2274035751be8e92a30).

And here it is prior to my changes (at b6657dfa83a6644e7e692887a4ec1e1bda559489).


so at index 15 (for RIGTH D-PAD) :
https://w3c.github.io/gamepad/#gamepadevent-interface

the value returned is 17 :
![image](https://cloud.githubusercontent.com/assets/4033090/9322074/13e004a8-45a1-11e5-8a13-4c019999b4ce.png)

so this is correct, and actually working, as the player is going right ;P


I think perf wise we're ok, though I'm not a pro with instruments.


Wait, now we don't copy any property not in the list, right?
Which mean that if I add new configuration to KafkaConfig and LogConfig, I must add it here or it will be lost.

Are we sure this is what we want? If yes, then we need to document this very clearly in LogConfig.


lets increase the killsNeeded (so like, you have to defeat more enemies to unlock the door) >:D


how to ignore or hidden those binary files & django migration files ???


Два листенера выше можно объединить в один с помощью функциональных интерфейсов и анонимных функций.
Вместо того, чтобы передавать в конструктор комбо, у которого мы возьмем текст или константную строку, можно передать сюда произвольную функцию, возвращающую строку.

Джава 8 предоставляет для этого параметризованный интерфейс `Supplier<T>`, позволяющий описать любую функцию, не принимающую никаких параметров и возвращающую результат типа `T` (в данном случае в качестве типа надо задать `String`). Получить результат у этого интерфейса можно по методу `get()`.

```java
object.put(key, supplier.get());
```

А дальше начинается все самое красивое, потому что анонимные функции позволяют передать в качестве такого супплаера произвольное выражение, возвращающее строку, которое будет вычисляться каждый раз при вызове `supplier.get()`.

Например, передать константную строку можно так:
```java
solvableOnly.addSelectionListener(new ConfigurationSelectionListener(
		"solvable", () -> "true"));
```
Лямбда `() -> "true"` читается так: функция не принимает никак параметров ( `()` ) и возвращает строку `"true"` при ее вызове.

Аналогично для комбо листенеров или листенеров эвристики:
```java
leftCombo.addSelectionListener(new ConfigurationSelectionListener(
		"computeLeft", () -> leftCombo.getText()));
```


This change overrides the other test of the same name in line 39, basically removing the last assert without touching it.