This is a little bit weird. It seems you've added this extra method just in case user is nil, right? Since the original way of accessing a user's full name is more conventional, and wouldn't require an extra method, perhaps you could consider something like this:

- if `review.user.nil?` is true, render markup that perhaps shows that the user has deleted their account but the review remains
- otherwise, just use `review.user.full_name`

So for a complete example (untested, mind you):
```
<% if review.user.nil? %>
  Account deleted
<% else %>
  <%= review.user.full_name %>
<% end %>
```


Is the label what is shown in the UI? Should we change this to 'OmniSharp Atom'? (I've been meaning to update the name of the menu shown in the 'Packages' nav UI as well).


So this caused a funny bug. I forgot to rename `img` -> `drawn_img` and thus the *first* request showed the skin instead of the rendered image. :disappointed:


o -d é pra ele rodar como um daemon, seria isso?
Ou criar o script de start e stop?


Good to know. Thanks for the pro tip!


NVM, it seems to work now. Maybe I had mis-typed something when I first tried it. :confused:


No


Make my progress visible to other team members


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (617 of 617 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-5.02%25-green.svg)


This method is a bit confusing. I haven't started up the app to see how the search works, but right now (in your latest commit) it seems like you're only searching by title, correct?

It might be worth cleaning up this method a little. Some points to consider:
- do you need old commented out code, or can you delete it, and let git's history keep it for you?
- what are you searching by exactly?


Create a method that takes the options given to `nameCallback` and set the `nameCallback` to this function name in your config.


No need for this one, remove it, it will also avoid duplication of code.


Done


Try `Bar.prototype.baseClass`.


Further consideration of this "unexpected" event, logging, etc.


great. makes sense.

On Mon, Apr 20, 2015 at 6:36 PM, Vivek Goyal <notifications@github.com>
wrote:

> @vbatts <https://github.com/vbatts>
>
> That option is better than loop devices but even providing actual devices
> using dm.datadev and dm.metadeve has issues. And biggest issue is pool
> management. docker can create pool using those devices but after that
> docker does not know how to do pool maintenance.
>
> For example, if pool gets full, pool will go in read only mode and docker
> commands to create or delete containers will start failing. docker does not
> know how to grow pool. lvm commands do know.
>
> it is advisable to run thin_check on pool before usage to make sure pool
> is in good state and docker does none of that and lvm tools do.
>
> If pool gets into bad shape and needs-check flag is set, it might require
> repair and docker has no idea how to do that repair. lvm tools provide some
> commands to repair pool metadata.
>
> So passing an lvm thin pool is recommended because then get lvm's
> capability to manage pool as a benefit. Passing in data and metadta devices
> will work fine as long as nothing goes wrong with pool.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cgwalters/docker/commit/d1e587bc8562e37c8e946191c1020a5f5504d766#commitcomment-10813624>
> .
>


approved.


    github.on('comment', function(comment) {
      if (comment.author == 'wltsmrz' && comment.text.toLowerCase() == 'no') {
        github.comment(comment.line, 'Why?');
      }
    }


I considered that, but found a rather long discussion about utime issues, so I avoided that.


Remove comment


I think the Container *class* should emit a non-bubbling event for added children.  That would make `this.viewStack` emit the event, so then you could do:

```js
this.viewStack.on("add-child", function(evt){
   ...
});
```

(not sure what the event name should be)


Fantastic.


If we run into efficiency problems with this brute-force approach, we may need to offload the positional logic up a level; instead of taking a delegate to determine if a point is valid, take a delegate that finds the next point.


Change an by a, because the next word starts with consonant


Please add a note about what still needs to be considered.


```github.on('comment', function(comment) {
  if (comment.author == 'clark800' && comment.text.toLowerCase() == 'why?') {
    github.comment(comment.line, 'Because Stone Cold Steve Austin said so!');
  }
}```

A: Server/remote logging and error events ought to be given some more consideration. I'm going to leave it here because I'm not sure if we want to emit `error` on the Remote here and don't want to break anything unnecessarily


nailed it.


It's not really a convenience function, it just returns a different type. Leave this comment here as it's different (int at the end).


the whole unsigned api was removed in 5.3, as we don't use `auxL_Unsigned` at the moment it seems a perfect time to throw it out before it get used again ;)


When is it generally good practice to store a value in a variable? Only if the code itself isn't clear? For example, it's not really necessary to declare `amount`, but I felt like the next line would become too long if it said`.format(int(papp.givingprojectgrant.amount))`


If it's worth leaving a comment, it's worth explaining why it's there. Currently it's just confusing. Maybe just copy your GitHub comment into the code comment.


Couldn't this be done as:
```Python
supported_strip_sizes = list(
    min_strip_size * (2 ** i)
    for i in range(0,int(math.log(int(max_strip_size / min_strip_size), 2) + 1)))
```


Longer variable name for this one, "res" is too short, use "result64".


@TalPasi Good call, added this change to master.  :+1:


Longer variable name for this one, "num64" is too short, use "numbers64".


anonymous Comparator class


ah ha. we'll have to wrap it in another function and lua_pcall it then.


Quite agree about deleting them. Will do so...


@MalcolmCurrie Looks good but people are going to immediately ask why the panel on the right is much dimmer than it should be.


Looks like `hasChildren()` and `getIndexOfChild()` are exactly the same as the ones in delite/Container, so you can remove them from here.


https://github.com/mangoszero/database/blob/develop21/World/Updates/Rel21/21000_08_warden_checks.sql#L345

Is a World Hack. (Seems like its putting you in a different phase)

It's probably doing something much more sinister then a phase change though.. not exactly sure.. but it's defiantly detectable.

Hack: WoW V1.12.1 Trainer (v1.3)


[![Madonna of the Martyrs](http://api.artsmia.org/images/1782/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1782)


[![Ten Years of Uzbekistan](http://api.artsmia.org/images/32715/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=32715)


[![Textile panel](http://api.artsmia.org/images/119343/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119343)


[![Powder Horn](http://api.artsmia.org/images/1805/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1805)


[![Man and woman with a calabash](http://api.artsmia.org/images/2602/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2602)


[![Diptych with scenes from the life of Christ](http://api.artsmia.org/images/3210/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3210)


`beast::weak_fn` please


should be `strictEqual()` not `deepEqual()`, and the arguments should be reversed.


Tobiz,

I finished an initial version of the OGN_db.py code. It's in my github.
Problem with the code is, that if you execute it several times, you get
several identical entries in the db. Same is true for the flarm_db.py
code. At the end of each script some smart sql statment should delete
all duplicate entries and only keep the latest (max(id)) one.

Cheers

Dominic

Am 20.04.15 um 09:20 schrieb tobiz:
> Hi Dominic,
>
> Access to Flarm_db has been ok I think all through the recent flarm
> update.
>
> I didn't know how to (or that you could)  pull the OGN_db so worked on
> flarm_db really as an exercise since code in perl already exists to do
> this, I just don't like perl.
>
> What I suggest is a value in Settings to say which of OGN_db or
> Flarm_db is to be used. Then add in code to pull OGN_db or Flarm_db as
> required, modify the function that does the mapping to Registration
> depend on the new setting and everything after that remains the same.
>
> Rgds
>
> tobiz
>
>
> Sent from Samsung tablet
>
>
> -------- Original message --------
> From: Dominic Spreitz <notifications@github.com>
> Date:20/04/2015 06:26 (GMT+00:00)
> To: tobiz/OGN-Flight-Logger <OGN-Flight-Logger@noreply.github.com>
> Cc: tobiz <pjrobinson@metronet.co.uk>
> Subject: Re: [OGN-Flight-Logger] Created INSTALL.txt file (f8ab361)
>
> Hi Robiz,
>
> Thanks for checking my pull request. Unfortunately I mixed several
> commits in one pull request. The work I did on flarm_db.py should be
> disregared, since I created a new file OGN_db.py which contains all the
> mods for the OGN DDB. To my big suprise, flarm_db.py was still able to
> pull data from Flarmnet. They re-enabled the download of Flarmnet-data.
>
> Rgds,
>
> Dominic
>
>
>
> Am 20.04.15 um 00:28 schrieb tobiz:
> > Hi Dominic,
> >
> > Thanks for your useful modifications, I'll look into including them,
> > they seem useful additions.
> >
> > I had noted some one running the code from the Glidern2 server log.
> > I've made a number of enhancements in my development version that I've
> > not uploaded to the master version yet. When I do next sync these back
> > to the master I'll include your enhancements.
> >
> > Rgds
> >
> >
> > -------- Original message --------
> > From: Dominic Spreitz <notifications@github.com>
> > Date:19/04/2015 20:28 (GMT+00:00)
> > To: tobiz/OGN-Flight-Logger <OGN-Flight-Logger@noreply.github.com>
> > Subject: Re: [OGN-Flight-Logger] Created INSTALL.txt file (f8ab361)
> >
> > Created INSTALL.txt file to make it easier for new users to install
> > the software
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/tobiz/OGN-Flight-Logger/commit/f8ab3614d3162a2212b7711223f4495732e6cd05#commitcomment-10794346>.
> >
>
> --
> Dominic Spreitz
> Kothauer Str. 123a
> 85053 Ingolstadt
> Germany
>
> Mobil: +49.179.9016643
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tobiz/OGN-Flight-Logger/commit/f8ab3614d3162a2212b7711223f4495732e6cd05#commitcomment-10797223>.
>

--
Dominic Spreitz
Kothauer Str. 123a
85053 Ingolstadt
Germany

Mobil: +49.179.9016643


k


unwanted space after quote mark


I'm thinking this doesn't have to be virtual. I worried about the ImageSurface on a non-RenderTargetBitmap impl getting disposed at the time this method was called. I currently stub out BitmapIml for Cairo.


Doh! Iterator, not iterable. Silly me!


The parse() call is no longer needed w/delite 0.7.


If they are integrated intensity plots then the'll be in K km/s units rather than temperature units.


The prefix is supplied in a callback which means the application may need to keep ownership after the library stops referencing the object. A use case is Peter's ndnrtc application. The bind for the OnInterestCallback only mentions the first parameters (prefix), not the last parameter (the InterestFilter). His application needs to keep the prefix (which comes from the InterestFilter) but not the interest filter. Using shared_ptr in callbacks is an easy way to avoid headaches later.

From: Alex Afanasyev <notifications@github.com<mailto:notifications@github.com>>
Reply-To: named-data/NDN-CCL-API <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Monday, April 20, 2015 at 13:54
To: named-data/NDN-CCL-API <NDN-CCL-API@noreply.github.com<mailto:NDN-CCL-API@noreply.github.com>>
Cc: Jeff Thompson <jefft0@remap.ucla.edu<mailto:jefft0@remap.ucla.edu>>
Subject: Re: [NDN-CCL-API] InterestFilter: C++ getPrefix returns a shared_ptr. (bbb7ebf)

why?

—
Reply to this email directly or view it on GitHub<https://github.com/named-data/NDN-CCL-API/commit/bbb7ebf41a1ee13af365dbfc06b010f948e2c9b1#commitcomment-10812053>.


It is mentioned in the previous sentence that they are integrated intensity maps.


lack to word between want and upload words


MarkdownDocs (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1639 of 1639 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)


if you have the downloaded version  cant you still install your registration code ?


Nice！


because im just one of the old players from the game and would just like to play the campaign and play hard computers on large lands lol


*Created table for Georgia county populations in 2000:
CREATE TABLE population_2000 (GISJOIN varchar, YEAR float, REGIONA float, DIVISIONA float, STATE varchar, STATEA varchar, COUNTY varchar, COUNTYA varchar, FL5001 float);

*Used psql console to upload 2000 CSV tabular information:
\COPY population_2000(GISJOIN, YEAR, REGIONA, DIVISIONA, STATE, STATEA, COUNTY, COUNTYA, FL5001) FROM 'P/population_2000.csv' WITH CSV HEADER;

*Created Table for Georgia county populations in 2010:
CREATE TABLE population_2010 (GISJOIN varchar, YEAR float, REGIONA float, DIVISIONA float, STATE varchar, STATEA varchar, COUNTY varchar, COUNTYA varchar, H7V001 float);

*Used psql console to upload 2010 tabular data:
\COPY population_2010(GISJOIN, YEAR, REGIONA, DIVISIONA, STATE, STATEA, COUNTY, COUNTYA, H7V001) FROM 'P/population_2010.csv' WITH CSV HEADER;

*Created Table for Georgia County populations 2013:
*For 2013, Region A and Division A are empty columns in CSV, and therefore could not add data to table.
*Census data for 2013 is estimated population, rather than actual population. Census off-year estimates such as 2013 take samples from mostly populated areas to gauge growth. Therefore, there are only 36 counties accounted for in 2013 data rather than 159.
CREATE TABLE population_2013 (GISJOIN varchar, YEAR float, STATE varchar, STATEA varchar, COUNTY varchar, COUNTYA varchar, SBLE001 float);

*Used psql console to upload 2013 tabular data:
\COPY population_2013(GISJOIN, YEAR,  STATE, STATEA, COUNTY, COUNTYA, sble001) FROM 'P/population_2013.csv' WITH CSV HEADER;

*Used Alter Table sequel statement to rename column containing population information from NHGIS code to word "population" for the sake of clarity.
ALTER TABLE population_2000 RENAME COLUMN fl5001 TO population;
ALTER TABLE population_2010 RENAME COLUMN h7v001 TO population;
ALTER TABLE population_2013 RENAME COLUMN sble001 TO population;


Since it's being assigned to a blank identifier, is there even an allocation taking place? I thought it was nothing more than a compile time [assignability check](https://godoc.org/golang.org/x/tools/go/types#TypeAndValue.Assignable).

P.S. This is extremely unimportant, I was just curious.


But this still adds an event to the entire document (`node` passed in is `document`) and filters to summary elements in the event handling function.  Isn't that overkill?  Surely it should only add event handlers to each summary element in the first place?  To be honest, I think this polyfill isn't too well written.  Should probably put something new together, perhaps using JQuery (if available) for its succinct syntax.


Why not just use (>=)?

Also, the fact that the input type is a String seems kind of fiddly. I understand that Versions aren't super friendly to create, but I don't think Strings are the most natural type for version numbers..


Hmm, that's disappointing, because it's a lot of code.   At a minimum, it needs a bunch of tests to make sure that all the methods you are overriding (like `addChild()` and `appendChild()`) still work as expected, when called both before and after `this.created` is toggled.

I'm also confused how you can get a table layout using a `<div>` rather than a `<table>`, but are you talking about this CSS?

```css
.d-bar.d-bar-h.d-bar-justified > div {
	width: 1%;
}

.d-bar.d-bar-h.d-bar-justified > div > button {
	width: 100%;
}
```

It looks interesting.  I hadn't seen that technique before.


@lkolbly I was looking for something along the lines of this.

```c
int time_interrupt_callback_handler(){
   // called every so often
}

int some_kernel_function(){
     //
     int id = register_interrupt(&time_interrupt_callback_handler, TIMER_INTERRUPT); // possibly take a period argument

    /** logic **/

   pause_interrupt(id); // quick pause
   resume_interrupt(id);  // quick resume
   deregister_interrupt(id);  // the complex deregistration process.
}
```


Initialize


So if an instance doesn't have a version in its metadata, we conclude that it's greater than the requested version?? That seems backwards to me..


I think so, but I'm not sure. Just from first principles,

```
var _ = foo(&Bar{})
```

does call `foo` at init time and allocates `&Bar{}`. So unless they have special cases as optimizations (which would seem reasonable), I think it would allocate.


Yep. My Jenkins outlived Bukkit by 7 months. The latest versions are available on DBO anyways, and I have to move on with my life. I will not discard the possibility that I might come back to coding but not in the current situation. It is way to expensive to keep my server that hosts jenkins, my bouncer and the other Minecraft related stuff that I do not plan to need in the near future.

I will miss you :(


Not sure if you saw my message on skype, but it is important not to send `\` (backslash) in JSON. Characters are escaped (therefore backslash) but for some reason 3rd party receives backslash and does not decode it well.
Therefore data variable must be a "formatted" JSON string.


This fix seems to look good, thanks!
However could you post sample code that repros the issue? You say a KeyNotFoundException is thrown; but it's not obvious how the fix would prevent that exception from being thrown.


try `div(ui-view)`


Good news @shigeki @bnoordhuis ! This patch is going to be backported to 1.0.2 branch!


indentation is wrong here, you have to set it to `2 spaces`


Use let for arr instead


wrong indentation


get rid of this blank line


Remove white space inside destructured array


Commit message has to be in ENGLISH


@ProgramTheHacks I like this! (Y) :P


:+1:   thanks!


needs #if LINUX_VERSION_CODE >= KERNEL_VERSION(4, 0, 0) around it?


Nice! They are partially back :)


Сделал движок. Изменил интерфейс.


60. Yes. Reduced the baud rate (based on what we read about the board).
143. I didn't think it compiled after setting Temp_Sensor_0 to 0? I thought I made this change without success although I may have been killing off the extruder_0 or the thermistor_0.
349 & 350. You disabled the Max & Min end stops because we have not wired them up. So ignore my comment in the latest email about adding and removing jumpers to the end stop pins for simulating movement until we wire the end stops tomorrow.
368. This makes it sound like we still do not have a Y-axis in the design (?)
373 & 374. Disables all extruders and keeps active extruders from being enabled as an exception to the global disable command.
You'll need to disable the hot bed, if I recall correctly.


good call.


sort -u was added as it's more efficient to merge uniq items internally.

sort -o was added as it allows for sorting the file _in place_ which is much more awkward in shell, requiring managing possible very large temp files etc. which sort(1) has to do anyway


Partially! :D Man this has taken forever. Script-bot's been running for a day and a half- now at about 122/190.

One nice side-benefit, I've decided to catalog all the icons made for Magic, you can see a list here:

http://mine.elmakers.com/#textures

Useful when creating custom spells!


s/mater/matter/


Testei pra essas aqui:

![image](https://cloud.githubusercontent.com/assets/7806092/7242690/3f3d9f5c-e79b-11e4-8614-14e7c507bc18.png)


Specifying master as the initial-cluster name fails for us with
```
Apr 20 23:13:35 localhost etcd2[3841]: 2015/04/20 23:13:35 etcd: couldn't find local name "default" in the initial cluster configuration
```
Changing the initial cluster from `master` to `default` did work, but it would be reassuring to know if there is an implication to that.


sort -R is like shuf(1) except that _keys with the same value sort together_


PHP's MySQL and PDO functionality always returns string values from query results unfortunately. It's actually quite annoying. If you know of a way around this, that'd be awesome! There is the idea that we could pipe the output through an iterator that checks the column types and casts the values accordingly, but the overhead seems to be extreme.


send -> sent


trying the above implementation did not achieve what this function does. list(set(input)) returned an identical list to the input. perhaps there is another easier python builtin solution that avoids dedupe_list from being necessary(). also the placement as a utility function in prosperity_set is pretty bad.  where should i put just generic utility functions?


In general there is a _very_ high bar for new features added to GNU coreutils.
See http://www.gnu.org/software/coreutils/rejected_requests.html


yeah I don't question calling it, I question not setting `SYMBOLS_COLUMN_TAG=found` yet using `found` in that call.  Reading a bit deeper seem to suggest it *is* fine though, but I maybe setting `found=tag` in this case would make things clearer.

This said, while I understand that removing the whole `gtk_tree_store_set()` call is probably good for performances, we need to understand that:

* the tag should really be *completely* equivalent (e.g. good comparison function) as we use it for everything starting at the tree (goto, etc.) -- well, in practice it probably doesn't need to be *that* good currently, but better not get weird surprises in the future;
* this might mean higher memory usage, because the tree store would keep the old tag around, while the tagmanager keeps the new one.  Basically, instead of 2 refs to the same tag, we get 2 tags;
* shouldn't be a problem, but a collateral is that the tag *pointer* is *not* in any of tagmanager's lists.


Tengo líos para llevar a testing este método. Hasta el momento, solo he podido recrear el flujo de error hardcodeando la excepción.


Whoa.


Whoa again.


apparently LoadInventory calls LoadStatusEffects (what)


args should probably be const


sorry, just saw this thread -- per my email, a GIF of those tiles building out in the order that Kevin described in his email would be sweet.


wtf... hold this is wrong


I've stopped development on this fork of PEDA.

New development is at https://github.com/zachriggle/pwndbg


true is only 65 lines. Still trivial and also a consistent interface with all the other tools


Published version 1.0 of the App to the Play Store


Good catch :)


Hi Shell, thanks for fixing my bad spelling :)


Awesome! My first commit and push in the hub :)


Thanks for bringing them back. I missed them :) Hard to re-learn all the spells. Also been messing around with your MythicMobs integration. Pretty awesome .. we are working on building a bunch of quests now.


Your reasoning reminds me of the [TOML reasoning](https://github.com/toml-lang/toml/blob/master/versions/en/toml-v0.4.0.md#but-why).


this count is very random, because it counts old elements, minus children of a removed node.


Probably you wanted to type `self.editingAccessoryView = accessoryView;`


Excellent, this will be fast


@SeanFeldman from a unit testing side, I would actually leave the tests the way they were, u are testing the unit not the integration.


Thanks for that update @peterlynch


Btw. this will only go live when the branding can go live since that has all been merged to master.


`glfwKeyCode2Unicode` is not needed, openFrameworks has old `glfw3.h` file that should be updated instead.


While I can understand the rationale, the hard-coded limit seem pretty arbitrary to me, did you have any metric for choosing it?
Also, note that the symbol reparse timeout setting will have an impact on the usability vs. speed.


don't we have a named constant instead of `0`?


I don't have any experience with that software, but I can't replicate the error. After your code, I ran:

```python
In [27]: for item in session.query(Job,Clocktime).first():
   ....:     print("ID: {i.id} || item: {i!r}".format(i=item))
   ....:
ID: 1 || item: <models.Job object at 0x031F7AD0>
ID: 1 || item: <models.Clocktime object at 0x031F7B10>
```


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2106 of 2196 tests passed)

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


Awesome! I have Quests integration, too - though I'm not sure it ever got fully functional.

By the way, I'm testing your "Magnetic Leggings" now :)


why do you remove this? If you don't you cannot use titles in headlines, for example, because they often have carriage returns in them.


We had this discussion with @andreasohlund and @danielmarbach.
Bus configuration is the one that affects the outcome, so it makes sense to go through BusConfiguration to validate assembly scanning behaviour.


@MartinNowack How exactly does this work? [__fprintf_chk](http://refspecs.linuxbase.org/LSB_4.0.0/LSB-Core-generic/LSB-Core-generic/libc---fprintf-chk-1.html) has a different prototype from fprintf


Can we roll back KIF?  Does this interact with test timing?


I would use an AT for that, anyway not a big deal


I think it's okay to leave failing tests.  Better to assert they are *not* equal and document the strange behaviour than to comment out a valid test.


Okay, no prolem. I'm writing this for my personal plain-text notes (mostly bullet-lists) and I currently have no plan to make it extensible enough for public use, so I'll be haphazardly trying out stuff.


CurrentMarketPriceDuration


Please review this, @casey-collab


That was previous attempt @johnsimons, but it didn't work since acceptance framework is overriding changes done through bus configuration for it's own stuff :)


Rectify code


This is disturbing.  We have a "WithTestProfile" class or similar; does it not hook the app early enough?  We must be able to do better than this.


Still confused by this, the date on the file is April 23rd but the training would indicate that it is April 13th to 14th. If it is 23rd, and you can turn it around quickly, I can still get it in.


It leveraged `AssemblyScanner` directly :(


it's a bit sad to have to pass `update=TRUE` here… but well, without an additional cache for this it's no possible to avoid, and it doesn't make sense to cache this, which shouldn't happen too often anyway.


What are you doing?


Why not just make the `app` field public so you can set it with: `api.App = router`?


I think this should be `toRelativeTimeString` or `asRelativeTimeString`.  `to` is best.


nit: space before {


nit: `self / 1000`


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@bluejamesbond Alright. @jorgmuno and I are meeting tomorrow before class, I shall be sure to suggest that API.


Good


I was thinking about using ACC data for better PH, but this is not going to be the next step. After gps-filtering branch is tested, I'll be working on generic navigation framework that would be capable of 3D navigation (althold, PH, WP, RTH etc).


Unfortunately these don't localize well.  Use a localized format string and then format in the years.  There are examples in the tree.  (For example, the version setting in the settings view.)


Could you mark that in README.md? It's a bit confusing now ;)


Nice tests!  Worth including a comment about where they came from, if you adapted them from that link I dropped.


nit: spaces: ```
/*
 *
```


question: if the module its already name .Edit, why use Edit. again? Does this actually work : ProductsApp.Edit.Controller as ProductsApp.Edit.Controller.editProduct(id) ? Just asking


Have you rebased this recently?  We don't want to refresh the whole tabs display 'cuz that'll sync.  We just want to redraw, which you can do with `tableView.reloadData`, I think.


Some nits and a few real issues.  Looking forward to an updated version.  Thanks, @sachin004!


hpwh_residential (nmerket) - x86_64-MacOS-10.9-clang: Tests Failed

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


isnt selectedCategories declared as an global object, if declared without var?


I am not sure what you mean by “line change”.

Also, should everything between “Rails.application.routes.draw do”, and “end”, be indented in routes.rb?

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Monday, April 20, 2015 5:14 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] Complete crud-assignment (bdc6e47)

Unnecessary line change.

—
Reply to this email directly or view it on GitHub <https://github.com/Mei152/Bloccit/commit/bdc6e47197f3b97aa9534f9ce46f897c114d65fa#commitcomment-10798935> .  <https://github.com/notifications/beacon/AKgq0JMqKlSTU_9fiNrxyAw4Mcy_TDGcks5oBLrDgaJpZM4EEIEf.gif>


You might also simplify this by simply cutting in `get_tag_list()`?

Which BTW could probably be improved (removing the useless `tag->type & tag_types` check as it's always passed `tm_tag_max_t`, or maybe even doing a mean `memdup()` on the array and sort and use this instead of a list) -- yet improving might require profiling to really gain anything, and it might be worthless.


Missing this was causing a semistandard lint error :)


A bit cleaner eh? Do you prefer a separate var?


I don't think we need the domain module for this, it only adds complexity and load.

Some thing like this can work:

```js
work(function _work(err, data) {
      if (err) return fn(err);

      cache.set(key, data, ttl, fn);
    });
```


@justinxreese, how do I designate an admin user?

On Mon, Apr 20, 2015 at 4:50 PM, Justin Reese <notifications@github.com>
wrote:

> @cameronscott137 <https://github.com/cameronscott137> this line will
> allow you to access the most recently uploaded sponsor image. To upload a
> new sponsor image, visit /advertisements/new as an admin user.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/builderworksco/acpc_poker_gui_client/commit/b4b2b95db7a5d2c556c40703d07c17a3902bfcf0#commitcomment-10811992>
> .
>

--
Cameron
(610) 883-7342


I can’t take much credit for that. I was just copying the way Bloc demonstrated it.

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Monday, April 20, 2015 5:16 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] Complete crud-assignment (bdc6e47)

The rest of the code is good. The index view using media classes is nice.

—
Reply to this email directly or view it on GitHub <https://github.com/Mei152/Bloccit/commit/bdc6e47197f3b97aa9534f9ce46f897c114d65fa#commitcomment-10798972> .  <https://github.com/notifications/beacon/AKgq0Gq59iYdzRAeWE-5cBVWqQh4EUz_ks5oBLtVgaJpZM4EEIEf.gif>


There are many deletions that surprise me here.  I expect your rebase has failed.


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.219](http://teamcity.monogame.net/viewLog.html?buildId=6187&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


By those files don't inherit from the regular controllers. They would need to be defined here as well.

C

Sent from my iPhone

> On Apr 19, 2015, at 8:46 PM, neema122 <notifications@github.com> wrote:
>
> Hi Chris,
>
> Aren't the items_params and list_params suppose to be defined in the
> regular items_controllers and list_controllers files?
>
> -Neema
>
> On Sat, Apr 18, 2015 at 4:48 PM, Christopher Beck <notifications@github.com>
> wrote:
>
> > And similar here - list_params is not defined in this controller. Is your
> > latest code pushed?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/neema122/open-todo/commit/7dcc27daeeb1d2090cc57dc84e3ad8f81a224961#commitcomment-10790001>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


@Luminarys shouldn't need this line at all after PR


I am taking a look at those test issues. I'm pretty sure it's because the tests themselves are buggy. The first one I looked at it creating two Application instances and only properly cleaning up one of them.


rax retry tests


I spy with my little eye a spelling mistake :D Should be NETHERBRICK instead of NETHERBICK.


The usage of "biggest" is a bit misleading. We usually say "higher" rather than "bigger".


Can you just add *.iml instead?


This is still going to unlock the bounty, it just won't warn the user about it -- is that all right?


nit: formatting


Formatting.


Another misspelling. Should be DisguisedBlockType instead of DisgusedBlockType. There's multiple instances of this misspelling throughout the commit.


Investigar la correcta forma de insertar una imagen.


Works for me too!

---------------
not sent from my phone.


What exactly does this accomplish?


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.219](http://teamcity.monogame.net/viewLog.html?buildId=6187&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 483, ignored: 6 Build time: 00:15:32


Why do you need to suppress exceptions here? (and what exception is it?)


Ah that makes sense. I noticed that views with `tagName === ''` aren't `_registered()` which is probably why these test issues didn't show up before

https://github.com/emberjs/ember.js/blob/idempotent-rerender/packages/ember-views/lib/views/states/in_dom.js#L18-L20

Sounds like you've got this well in-hand, I'll leave this one alone.


remove [[pullingPromise]]


@chteuchteu do you have another French speaker that could sanity check these? Just a second set of eyes to check for grammar, spelling mistakes, etc.


Update the rpctool documentation to explain how to use numpy, and add a comment here so that this line isn't incorrectly removed later as unneeded import.


The other strings for this are capitalized, would you mind adding another commit doing that as well `Login or Email` and then squashing it with this one?


El estándar que estamos utilizando es crear la consulta query, un string, de forma separada y no al interior del get_records_sql. También recordar que la consulta debe ser escriba con doble comillas para que github lo interprete como código sql.


I think this should be `.caprc` instead of `catrc`.


OK


We should probably clean up /js/app.js to remove search filter work...


Can you update the `.idea` line in `.gitignore` to say `.idea/`? I think that way it properly recognizes it as a directory and ignores future files here, or at least that seems to be the case with my IDE's ".ignore" plugin


and here is no exceptions? can we add option for this to not show promoted posts in timeline with default value as _disable_ or something?

_// you know I supposed it's kind of bug in the core actually when I set this check, but nevertheless I allowed spam from users that we follow._


I couldn't decide if this should be a Map or a List<String> that can be parsed by Bind.parse(). If you think the latter, let me know and I can update the pull request.


Why does the controller use bytes and the unit test does not?


Smaller values here would probably be better for overall speed.


No need for empty brackets, and unless you expect this to be called from outside the formatter in which case it should probably be `reset` or something.


You can just use the constant directly rather than caching it like this (consider you're already inside `RSpec::Core` so `Time.now` is in fact `RSpec::Core::Time.now`)


Picky but excess ws where (`=  group` instead of `= group`)


For completely: I've also discovered that it's not good enough to make the top level outlet itself have a tag. That would introduce an *extra* layer of tags in the situation where the ApplicationView already got defined to have a tag.For compatibility, we have to make sure it's not the top outlet but the top outlet's child that is always a view with a tag.


Of course no need for committed comments but I'm sure you just haven't cleaned this up yet :)


`execution_times` prehaps?


Probably best to squash this commit into the other one ;)


And really don't need that comment commited either :)


Borrar var_dump


Is there a reason to have everything unsafe instead of weak? As I am aware 10.8. supports weak fully. We no longer support 10.7. Thanks


With incorrect you mean it uses more bytes, right?


El estándar que estamos utilizando es crear la consulta query, un string, de forma separada y no al interior del get_records_sql. También recordar que la consulta debe ser escriba con doble comillas para que github lo interprete como código sql.


Let's rename this to something more descriptive, maybe `lastQuery`?


Same here with the name


@poinck what OS are you on?


i thin that's good. i think we could release now, rather than wait for a major release... your call, though!


pyc 代码可以不用提交，使用 `.gitignore`


Mareks-MacBook-Pro:MacOS xhruso00$ codesign -v Cakebrew
Cakebrew: code object is not signed at all

I can sign the binary.


is this "affect" or effect?  github.com/nmoinvaz


Glad this made it in :clap:


Was this not a bug? Can we write a test for this?


Actually, I had already changed `[CsoundObj stop]` to wait for the thread to end, so I'm thinking this spin loop is pretty much not needed anymore.


Who is using the result here? Are you just returning to avoid a `break` statement?


Make sure password is not a live one. Don't worry, I changed the online password, you can't get in with this one.


$ git rm readme.md


Even on some circumstances, when one node doesn't have updated backup file, that has no problem because before vgchange action (LVM_start), lvm wil automatically backup metadata to /etc/lvm/back/.
see:
    -A, --autobackup {y|n}
              Controls automatic backup of metadata after the change.  See vgcfgbackup(8).  Default is yes.

After LVM_start the backup file will be updated, then LVM_status/monitor will get right status including for empty vg.


ok


in/into


accessible


You shouldn't need code_orrex, b/c it is already set in the `genc2` call.


lol indeed


LGTM


Why is the `modregrmx` part needed?


这里还是念不通啊 [苦笑]，我觉得是“让苹果暴露更多的网络接口出来”，@croath


:clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap:

Beautiful.


I think, If the bug is really like this,  can use the EnvironmentUtils.isWindowsSystem instead.


@laforge38 Thanks for the pull request. Some changes are needed to merge this:

1. The tests failed because you modified the original `Connect` keyword. You should leave that one as is and add other keywords for secure connect.

2. To provide secure connections, the best way IMO would be to provide keywords like:
  - `Set TLS` that closely mimics the [tls_set](https://www.eclipse.org/paho/clients/python/docs/#option-functions) function
  - `Set Username and Password` keyword for `username_pw_set()` function.
This way, user can use these keywords before using the `Connect` keyword to get secure connections.
  - convenience keywords likes `Connect with TLS` and `Connect with user`, if needed.

3. Make sure to provide reasonable default values to function arguments so users can ignore the ones that are not needed. For example, take a look at this function from paho client:
```python
tls_set(ca_certs, certfile=None, keyfile=None, cert_reqs=ssl.CERT_REQUIRED, tls_version=ssl.PROTOCOL_TLSv1, ciphers=None)
```

Lastly, make sure to add tests for keyword you add. I believe mosquitto.org provides a secure broker that can be used.


I'm thinking octal might be nice here (we have three base evaluation types), but yes, 0b%b if binary.


returned


currentClipTrack is no longer updated when the clip selection changes. This causes TimelineToolbar.qml deleteAction and liftAction to trigger asserts in TimelineDock::remove() and TimelineDock::lift(). Neither is it correct to currentTrack or currentTrack() because the current track can be different than the track of currently selected clip.


lol


currentTrack should not change here. Instead currentClipTrack should be.


Remember to 'rebuild', next to 'build and run' button in CodeBlocks


Seems like nothing that could be controversial :)


You're not returning SymbolFlags anymore


Remind me, do ASI only is a problem when the initial expression is on the next line, right?


appropriate symbol table if appropriate? When woulsn't it be appropriate?


Cherry-picked, pushed to 5.5


@edolstra thinking more about this, I'm wondering if it could make it harder to trust/audit nix behavior: take https://nixos.org/wiki/Download_all_sources for example. Before this, I could be reasonably sure that I had full visibility of every interaction my expressions had with the outside world, since they all show up as fixed-output derivations. After this, someone could sneak an evil `fetchTarball` into nixpkgs and do unpleasant/nondeterministic things where it might take a while to notice them. Do you have ideas for managing the additional scariness?


You did not remove the currentClip property, which is no longer updated and used by 3 actions in TimelineToolbar.qml, which triggers asserts in TimelineDock.


Fixed this in 5c5585dc7673adc8bab3181e5967c53b170c9c52


lk


Replace with push_string instead.


Use pop instead


Nice work :)


如果调用中没有提供脚本名，


那么解释器的名字就放在索引 0 处，后面接着其他参数。


If we're looking at a potential GUI of sorts:
http://kiwwito.com/installing-opengl-glut-libraries-in-ubuntu/


Instead of making dinosaurs duplicates, let va_arg do it's thing by letting it return the pointer to the next argument.


Tested with this changes and worked perfect! (xcode 6.3)
Thanks!


Changing current track should not reset the currently selected clip. They are independent.


Jake,

I am thinking of arranging our milestone and reduce it to 4 as requested.
Then, we need to close all issues under the first two milestone. What do
you think?

On Sun, Apr 19, 2015 at 10:07 PM, Jake Smith <notifications@github.com>
wrote:

> Nice work :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jakehockey10/bird-compare/commit/c2fd740e6976a4bef143deced7366a614f171463#commitcomment-10795745>
> .
>


ジュース一本なｗｗｗｗ


thanks for helping me out with triage @espen


Hi, Mr. ... Shiffman Sir! May I ask a question. Can I record some audio or sound and save to a file with 'Processing.Sound Library'? Is it possible? Well, I'v read the reference and emxampls about processing.sound library from Processing.org. But I don't know.


We should be using `_id` instead of guid, guid is not always usable inside url's.
`guid` and `_id` are not the same anymore for ingested items, so we should use `_id` and avoid using them interchangeably.


Uhhh..
setInterface? really poor naming. It's setting of palette (state palette), not interface. Interface is something about API.
Ok, what's inside? Dig into categories, and then IGNORE that and still set battlescape palette (hope these PAL_BATTLESCAPE/PAL_GEOSCAPE is not hardcoded).
And numerous misspelings and typos.


第一句感觉是：在类似的场景中，苹果提供了越来越多的原生 API，这个意思


Pls buy me GTA V


@pexcn
留意下这里的 .patch，如果你要入坑 cm-12.0 的话


Minor suggestion, if you want to avoid writing `"message \"%s\""`, you can do `"message %q"` instead.

From https://godoc.org/fmt#hdr-Printing:

    %s	the uninterpreted bytes of the string or slice
    %q	a double-quoted string safely escaped with Go syntax

Making this suggestion since I don't know if you're aware of %q or not.


老师，我这个人信息和题目都写了啊


It's fine


Why not move sponsors to the separated block like "Who to Follow"


Looks like this `sed` command is missing its target?


FPS :chart_with_upwards_trend:


Seja Bem vindo a Camarão com pão


welll,It's fine


What you're saying then is you stop dropping once you hit an element that doesn't match the predicate. If other elements that DO match the predicate come later in the list, then they're "safe" from getting dropped.


MASTER_SITES=CHEESESHOP will handle all of the PyPi things for you (See; /usr/ports/Mk/bsd.sites.mk)


Add LICENSE_FILE if one exists in ${WRKSRC}


I think this broke melee weapons on the map zs_gauntlet_reborn_v12.
The player cannot use the primary attack on weapons bought from the vendor.


Sort these values alphabetically


You should/could probably LIB_DEPENDS on the OpenSSL library version that comes with OpenSSL > 1.0.0 instead of this


Strip trailing slash


New lines at the end of the file


@vire the current behaviour of ember-cli-qunit is correct don't you think?  It seems that the right thing to happen is for blanket to do something similar to this commit.  I don't think there's a way we can fix it here without disregarding the user's `autostart` setting.  What do you think?


@i-make-robots  in case you are interested, check out https://github.com/MakeHartford/Makelangelo/commit/eb072cebae23e6f0c8f7461e1637406d76e27604 for my hack.


Good Work!
2/2


"ARDUINOQUADCOPTER_OLD" shouldn't be part of the #define.  That was a mistake.


2 Space indentation


we need this in each binary, no?


New line at the end of a file


Grade 2/2


maybe make sense to put it on every migrate call.

also, this should be unlocked-- right now it isn't ...


@whyrusleeping PR?


`code` is an inline element not a block.

this should be `<pre><code></code></pre>`

Pre's are block elements.


Changing API version does not change any behaviour.


Added comment template


Yes, `dropWhile` drops a prefix of the list.


Include the p_interface check when determining if this device is AHCI/HBA compatible.


@danielgolub  ALWAYS when you call any of the VC lifecycle methods - you must FIRST call super, and only then do whatever you want.
i.e., here: call:
    [super viewDidAppear:animated]


Removed button link from wishlist weapon names.
Corrected id reference in wishlist item deletion.
Fixed error in weapon path; incorrect sql results reference.


also if you want to just add a `pre.styleguide > code` class I'll update the CSS.


@kikito
in this function, the param: method not used.


Two space indentaion


I know. I've forgotten. Thanks.

Sent from my iPhone

> On Apr 20, 2015, at 7:43 AM, Aviel Gross <notifications@github.com> wrote:
>
> @danielgolub ALWAYS when you call any of the VC lifecycle methods - you must FIRST call super, and only then do whatever you want.
> i.e., here: call:
> [super viewDidAppear:animated]
>
> —
> Reply to this email directly or view it on GitHub.
>


name= input("What is your name?")
print("Hello" +'\t'  + name + '\t' + "Lets play a Game")
print("Guess a random number between 1 - 100, and I'll try to guess it.")
numberOfGuess =0
max = 100
min = 0
number =0
guess = (max + min)/2

while True:
	numberOfGuess +=1
	print("Is it " + str(guess))
	answer = input("Yes/No")
	if answer == No:
		print("Is it greater" + str(guess))
		numberOfGuess +=1
	if answer == No:
		guess1 = (guess - min)/2
		print("Is it " + str(guess1))
		numberOfGuess +=1
		if answer == No:
			print("Is it greater" + str(guess1))
			numberOfGuess +=1
		if answer == Yes:
			print("Is it 27")
			numberOfGuess +=1
		if answer == Yes:
			print("I got it right " + numberOfGuess)
			numberOfGuess =0
		print("Do you want to play again")
		if answer == Yes:
			print("Is it " + str(guess))
			numberOfGuess += 1
		if answer == No:
			print("Is it greater" + str(guess))
			numberOfGuess += 1
		if answer == Yes:
			guess2 = (guess + max)/2
			print("Is it " + str(guess2))
			numberOfGuess += 1
		if answer == No:
			print("Is it greater than" + str(guess2))
			numberOfGuess += 1
		if answer == Yes:
			print("Is it 88")
			numberOfGuess += 1
		if answer == Yes:
			print("I got it right" + numberOfGuess)
			numberOfGuess = 0
			print("Do you want to play again ?")
		else answer == No:
			print("Bye Bye!! See you again!!")


@sfarmar  - My comment shouldn't have been put in verbatim as a note,  It was meant as feedback.


2/2  Good work!

Watch out for indentation.


Reviewer 1


2/2 Good Work!


2 Space indentations


That's correct, I'll change it...


New line at the end of the file


Great work 2/2


cc @rafaelfranca @matthewd


I think we have to set check_hostname property before setting verify_mode.
That's why the following error raised.

```
ValueError: Cannot set verify_mode to CERT_NONE when check_hostname is
enabled.
```

Do I misunderstand?

2015-04-20 11:17 GMT+09:00 Karol Woźniak <notifications@github.com>:

> ping @liris <https://github.com/liris>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/liris/websocket-client/commit/b96a2e8fa765753e82eea531adb19716b52ca3ca#commitcomment-10795213>
> .
>

--
Hiroki Ohtani
http://blog.liris.org/


I think you should remove the `ignore: false` line, as this is causing Babel to attempt to compile everything in `node_modules`.


`NSNotificationCenter` has s **strong*** reference on any observer it holds. Yes it sucks, and it means you **must** call `removeObserver...` on the mirror method of the lifecycle for that VC.
e.g., here you add an observer inside `viewDidAppear`, then inside `viewDidDisappear` you should remove that observer:

````
- (void)viewDidDisappear:(BOOL)animated {
    [super viewDidDisappear:animated];
    [[NSNotificationCenter defaultCenter] removeObserver:self name:@"UpgradePressed" object:nil];
}
````

*please read about strong/weak references in ObjectiveC, it's crucial. Swift btw has the same mechanism of auto reference counting with strong and weak, its just covered with a lot of syntactic sugar...


also - I added this code already so you don't need to


I don't think this is needed with the new generic 'handle' implementation.


@bevacqua I don't think this is needed with the new generic 'handle' implementation.


Alright. So I understand strong reference means that this object has to be deallocated from memory manually.
I'll read more about it.

Sent from my iPhone

> On Apr 20, 2015, at 7:51 AM, Aviel Gross <notifications@github.com> wrote:
>
> NSNotificationCenter has s strong* reference on any observer it holds. Yes it sucks, and it means you must call removeObserver... on the mirror method of the lifecycle for that VC.
> e.g., here you add an observer inside viewDidAppear, then inside viewDidDisappear you should remove that observer:
>
> - (void)viewDidDisappear:(BOOL)animated {
>     [super viewDidDisappear:animated];
>     [[NSNotificationCenter defaultCenter] removeObserver:self name:@"UpgradePressed" object:nil];
> }
> *please read about strong/weak references in ObjectiveC, it's crucial. Swift btw has the same mechanism of auto reference counting with strong and weak, its just covered with a lot of syntactic sugar...
>
> —
> Reply to this email directly or view it on GitHub.
>


#4 issue needs to be resolved before moving on to rescuing orphan tags.


@bevacqua "the originally clicked **container**" => "the originally clicked **element**" ?


Whoops, refactoring was previous commit. This is just formatting.


Your king fu is better than mine. I didn't follow any of that.


Fix #1


It does not mean that at all. You still have something called ARC which does that for you.
Please read about it and also read specifically about retain-cycle since it's the most common bug/error that happens with this...


Thanks Shane. I agree. Ideally the document should be a product unto itself and not require an investigation of the PR discussion to understand the content.

I don't mean to be stickler, and usually I'm not. I certainly leave plenty of holes in my own docs. But this document seems really important to me, because other design documents for next gen point to this one to motivate various design decisions.


If you end up not using an import - please remember to remove it. Too many imports makes the class heavy and in efficient... Swift made you spoiled a bit with it's auto import... (:


Testaan tässä kommenttia. Täytyy katsoa lauseiden määrää ja miettiä, kuinka kauan manuaaliseen työhöän tai sen optimoimiseen menisi aikaa.


I'


The reason I removed customized `log4js` is that I don't think hacking a node module is a good practice.

`morgan` does not override `console`'s behavior, it just log access and error information. For network inspection, that's enough I believe.

I understand your concern about user behavior log. I'm looking at a better solution `winston`, and will switch to it soon.

I also noticed that there's a `logModel` in the database. What's the point of it?


I'm fine with building the NEW_DYNAREC x86 code with NASM.  I think it will be needed to build the new dynarec under recent OSX anyway.  Today I was working through some build issues on my macbook pro and I found that OSX's 'ld' linker crashes when linking libmupen64plus with the new dynarec enabled.  Maybe it will work without crashing if this linkage file is assembled with NASM instead of clang.


@till: Quite a good question. I first of all try to get the changes in for the autoloading, so not really related :)

@sad-spirit I wonder if we could lift access rights a little here on github, I perhaps can help a little with HTTP_Request2, too, I have some knowledge about HTTP and some reports were URL related in the past.


Should be ```(reg&8)<<13``` instead, will change.
```modregrmx``` was a lazy way of setting the ```REX_B``` bit when ```reg``` was ```R8```...```R15```, but I should have changed it before creating the PR.


Please  remove that nonsense, quotes around regex are wrong in javascript


Sounds good.

Is it possible that you would use this any other way? Will we break it for other people if we change it over to JSON parsing?


sorry，我看错了，马上给你改回来！


cool


2/2 great job!!


Enjoy the gentle introduction while it lasts
为它还是最近发现而感到高兴吧


Buenoas noches profesor
Subo algunas clases,aun faltan pero subo mi avance


2/2 Nice job!


Really nice annotation! 2/2


Unless the ```(ins & R)``` condition prevents it from happening, I believe the following could occur:
If it sees: ```0x48 0xC7 0xC0 0xFF 0xFF 0xFF 0xFF``` a.k.a. ```MOV RAX, -1```
It will change it to ```0x48 0xB8 0xFF 0xFF 0xFF 0xFF 0x00 0x00 0x00 0x00``` a.k.a. ```MOV RAX, 0xFFFFFFFF```


writing this way adds a space between 'of-' and 'failure'. I guess it all should be in one line


please mention correct package name, it doesn't look correct, if your work is for view module then mention the package name same as view module package name.


Very good!


no need to mention your email id


FYI, Distort does not work in GPU mode.


0/2 - it seems there is no commit from you. If you happened to forget to push, if you push a commit with the answers in the next 2 hours, I will grade that. Otherwise let me know if something else went wrong!


mention file description


nice find.


@i-make-robots, it is some dark arts so no worries. In case you wanted to follow, here is a quick summary:

kabeja is not available through Maven central, so I built it from the sources, checked in the compiled jars, then installed them locally via a [shell script](https://github.com/MakeHartford/Makelangelo/blob/eb072cebae23e6f0c8f7461e1637406d76e27604/java/bootstrap.sh).

I pointed Makelangelo to the [local repository](https://github.com/MakeHartford/Makelangelo/blob/eb072cebae23e6f0c8f7461e1637406d76e27604/java/pom.xml#L134-L139) where I just installed kabeja via the shell script. The repository effectively acts as the ./lib directory you had before except it has some extra Maven metadata.

This is where the opinionated nature of Maven gets to be a little bit too much. After researching, the only viable clean option I found was to roll your own corporate nexus server which is a bit overkill for one dependency.

With Gradle, something like this would have been a cinch.


No need of initialize empty array, but if you want to mention it please leave a blank line after that.


1/2 I don't see any of the views (dogs/index.html.erb, dogs/show, dogs/new). Let me know if something went wrong.


Leave a blank line.


The `logModel` in the database is authored by Pollow and is used to
implement the "history" feature. That is deprecated.

在 4/20/15 1:00 PM, Tong Xingang 写道:
>
> The reason I removed customized |log4js| is that I don't think hacking
> a node module is a good practice.
>
> |morgan| does not override |console|'s behavior, it just log access
> and error information. For network inspection, that's enough I believe.
>
> I understand your concern about user behavior log. I'm looking at a
> better solution |winston|, and will switch to it soon.
>
> I also noticed that there's a |logModel| in the database. What's the
> point of it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pollow/Contacts/commit/eb642a6d0dcf60116998cc75ad5842589f674420#commitcomment-10796070>.
>

--
Gong Deli
Junior
College of Computer Science and Technology
Zhejiang University


Wrong commenting, see hook implementation commenting style https://www.drupal.org/coding-standards/docs


oops :)


I have to start a new branch. The reason is my data get automatically updated. So each time after something changes, I could not access again.


Indentation are wrong in whole function.


Make variable name meaningful.


2/2 Very nice!!


Wrong coding standard, see how to write code in foreach loop.


Hi Robiz,

Thanks for checking my pull request. Unfortunately I mixed several
commits in one pull request. The work I did on flarm_db.py should be
disregared, since I created a new file OGN_db.py which contains all the
mods for the OGN DDB. To my big suprise, flarm_db.py was still able to
pull data from Flarmnet. They re-enabled the download of Flarmnet-data.

Rgds,

Dominic

Am 20.04.15 um 00:28 schrieb tobiz:
> Hi Dominic,
>
> Thanks for your useful modifications, I'll look into including them,
> they seem useful additions.
>
> I had noted some one running the code from the Glidern2 server log.
> I've made a number of enhancements in my development version that I've
> not uploaded to the master version yet. When I do next sync these back
> to the master I'll include your enhancements.
>
> Rgds
>
>
> -------- Original message --------
> From: Dominic Spreitz <notifications@github.com>
> Date:19/04/2015 20:28 (GMT+00:00)
> To: tobiz/OGN-Flight-Logger <OGN-Flight-Logger@noreply.github.com>
> Subject: Re: [OGN-Flight-Logger] Created INSTALL.txt file (f8ab361)
>
> Created INSTALL.txt file to make it easier for new users to install
> the software
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tobiz/OGN-Flight-Logger/commit/f8ab3614d3162a2212b7711223f4495732e6cd05#commitcomment-10794346>.
>

--
Dominic Spreitz
Kothauer Str. 123a
85053 Ingolstadt
Germany

Mobil: +49.179.9016643


Mention function description.


Gotta do quite lot of cleaning work then...


Every operator must have one space character in its both side.


# 主要修改一个文件即可运行
config.proprties

#main方法在com.jacky.thread.Main.java类中


2/2 Nice job! Might wanna check why you have duplicates of all your files with "~" appended on the end of the file name, but that's not a problem haha. Also, you don't need an html.erb file for the ```create``` method for DogsController. Take a look at the controller method if you wanna understand why you don't need such a file :)


Every comma must have postfix with one space character.


2/2 great job!


Nice job! 2/2


2/2 Great job! Although, you don't need an html.erb file for the create method for DogsController. Take a look at the controller method if you wanna understand why you don't need such a file :)


Note: now i know this isn't client interpolation, but rather peer-to-peer lockstepping for the jump action


Always use a space between the dot and make the concatenated parts to improve readability.


wrong indentation


You should take sizeof out of loop, can cause performance issue.


Breaks array in new lines.


Variable name must be in small letters.


Can't use t() function with variable.


@dinesh-sardiwal You should fix your git configuration. it seems that it is using my git settings, this module shouldn't be authored by me, you are the author of this. Please fix  that ASAP.


Variable name must be meaningful.


Mention function description.


Sorry, missed that line in my cleanup.


Can you please group the logical comparisons with braces? I am not sure what this is going to evaluate to unless you have test cases for all these.


Wrong coding standard.


If you figure something out, I'm sure @andytuba would like to hear about it.


Unfortunately, no. We worked around the issue by modifying the server to accept POST requests with certain arguments to do delete.


What's the need of $result_clone variable.


Can't we use empty() funtion ?


again


What's the use of this variable, can't we use this $result_clone ?


Missing file description.


Leave a black line.


thanks, fixed.


No need of ternary operator, read variable_get() function.


This should be an absolute link to "http://www.jackiandnicolo.com". (Without the "http://", the browser thinks it's a relative link to somewhere else on the same domain.)


Taking into account that ```Storage::InstallationOptions``` is singleton and that we are passing three attributes of it to ```registration_ui.register_system_and_base_product``` makes me wonder if we should just use  ```Storage::InstallationOptions``` from inside ```register_system_and_base_product```


cayenneModeler は実質的にはxmlファイルの編集を行っています。


yeah, Please do it.


- I am testing with nodestime = 600, so I am really testing with 30 + 0.1
- fishtest does not rescale increments, so indeed I am testing with
different tc's which is ok for me

2015-04-20 0:45 GMT+02:00 lucasart <notifications@github.com>:

> why run tests at 60+0.2 ?
>
>    - fishtest does not rescale increments, so you can't test reliable
>    with an increment (hence the default 0.05).
>    - it's very costly time control. what makes you think that you cannot
>    test this at, say 15+0.05 ?
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/Stefano80/Stockfish/commit/968cf4a0504ee2542d76c0f7c56fbdbcd6c7bc9d#commitcomment-10794403>
> .
>


自動生成されるファイルになります。直接編集できるのはEipTScheduleFile.javaの方のみになります。


掲示板のクラスを読み込んできるので、最終的にはこれが不要になるはず


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.220](http://teamcity.monogame.net/viewLog.html?buildId=6190&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


https://github.com/aipocom/aipo/blob/master/war/src/main/webapp/WEB-INF/conf/AipoResources.properties#L187
このへんの設定を読み込んでいる感じになります。


The issue I can see also with that kind of translation is that you cannot handler error generate by outside deps. I dependant on Moo, and for instance it can generate errors.
So you may have a partial translation.
Have an idea on our to handle this ?


このへんがscheduleに代わる感じ


Since you're touching the file anyway, would you mind doing a pass to use the C# aliases instead of the CLR names (e.g Double -> double)?


Gracias!  :-)


FWIW, I would just get rid of this thing.  I think it does more harm than good (for example, wehn I see Support.RandomSampleCount, I don't think that's an alias for "3").


Is there a tab here or is GitHub just displaying the spacing weird?


Slightly related to this commit: I did change the behavior of scim due to a bug I could not find an easy solution to: When you start scimming with shift, then click somewhere else before releasing the keyRelease never seems to make it back, so it's stuck in scim mode. I eventually decided on just requiring the mouse to be inside the timeline to scim (hence the onExited signal handler), which makes sense to me anyway.


mysqlの方のorg001.sqlと既存のマイグレーション用SQL 8000to8010.sql にも反映しておいてください。


[![LGTM](http://www.lgtm.in/p/VQ62)](http://www.lgtm.in/i/VQ62)
こっちのファイルもコミット漏れしないように注意が必要


testing comment


引数 topicid はスケジュールなので topicid になっていたかんじかとおもいます。


基本これは直接呼ばないです。EipTScheduleの方を呼ぶようにしておいてください。


削除しておいてください。


Shouldn't there also be a "delete pDialog;" before the return?


This was just meant as a proof-of-concept. When I try to use getAdapter I get an ComponentLookupError:

```
for schema in iterSchemata(context):
(Pdb) schema.providedBy(context)
True
(Pdb) getAdapter(context, schema)
*** ComponentLookupError: (<Document at /plone/doc1>, <InterfaceClass plone.dexterity.schema.generated.plone_0_Document>, u'')
```


A ti!!!


言語ファイルに移しておいたほうがベター


I sometimes find the existing code (5-7 times out of 10), other times someone will point out in PR. So agree, it's already pretty good.


デバッグコードは埋め込まないように注意が必要ですね。


Database.commit();
までコメントアウトしてしまうとUpdate文が発行されなくなってしまうのでは・・・？
消すときは基本的にコメントアウトではなく削除してください。


SQLは他のに書き方あわせておくといいですね。


デバッグコードは埋め込まないように


This is to support a later change.  The declareSymbolXXX functions return the symbol being declared.  The caller can then decide what to do with it.


@timjacobi Please consider adding some description in the commit message or choose a different title because every time the README is updated I have to look in the diff to see what actually changed. Thanks.


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.220](http://teamcity.monogame.net/viewLog.html?buildId=6190&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 484, ignored: 6 Build time: 00:14:15


cool ;-)


The way I think about it is there can be multiple, independent rocker devices instantiated, each with its own list of front-panel ports, where each port belongs to a world.  The world contexts between device aren't shared.  So device A might have this set of P4 tables populated with entries unique for device A, and device B would have different entries in its tables.

So no traffic between device instances.  They're just independent PCI devices on the PCI bus, and there is no back channel to share information between instances.  Someday in the future, perhaps we define a back channel to connect devices on the back end and let them exchange info or pkts.  I think we'll keep pretty busy just with the standalone device for now, since our main focus is on getting the OS interfaces defined and working.  For real-world devices, these problems need to be solved, but for rocker I think for now we can keep it simple.


Guess this should be addressed eventually in mranderson.


heh


legacy/gui.c should be left alone


Fixed later.


Ok it looks ok for this case but there are case where it is not ndarray.


Like here


2/2 You forgot to include validation for age however that's small enough that I won't detract for it. Nice job!


and here.


ok. not sure NIT is suitable for testing tc related patches though.


This doesn't work for OSX, in fact


shouldn't this be removed?


fighting


更新 update function


aaa


Javaの命名規則
http://d.hatena.ne.jp/chiheisen/20110529/1306676847
大文字・小文字の違いに注意が必要ですね。


The above fix is wrong. It should rather be:
```diff
diff --git a/glances/plugins/glances_help.py b/glances/plugins/glances_help.py
index ea0b6e1..50137e1 100644
--- a/glances/plugins/glances_help.py
+++ b/glances/plugins/glances_help.py
@@ -55,7 +55,7 @@ class Plugin(GlancesPlugin):
         self.view_data['psutil_version'] = _(" with PSutil {0}").format(psutil_version)

         try:
-            self.view_data['configuration_file'] = '{0}: {1}'.format(_("Configuration file"), self.config.get_loaded_config_file())
+            self.view_data['configuration_file'] = '{0}: {1}'.format(_("Configuration file"), self.config.loaded_config_file)
         except AttributeError:
             pass
```
The new 'sort processes by USER' is also missing with all the collateral changes.


Hey Kyle, could you take a look at moving from taking the picture to the edit picture view? It's not saving a URI into the JSON file and that ends up breaking the list view activity. I'll take a look myself as well and see if I figure it out.


Definition of DealFactory class for first iteration


#cache the frames


I don't really know how to do that tho. As @expez rightly says the cause of the problem is that two deps coming from diff jars but share the prefix: you can't generate a common mranderson prefix for them. Unless mranderson flattens out the prefixing here.  I am a bit reluctant to start actively change such things in mranderson... I could perhaps warn...


it is ok...


@albrow ok for me :smile:


Isn't it bad practise to modify vim's termcap options?


@keradus

True!

Will fix up as soon as I got out of the :bath:!

Separate question: since you usually squash and cherry-pick, I've given up on rebasing/amending/push-forcing. Cool?


Who throws an exception here?


do you know the size of list upfront? if yes - set capacity. If not - maybe use linkedlist?


I hope none... but I just wanted to be safe. I guess we could just remove the try / finally though. Would prefer to do so ?


There is no point for this method
@mkhanbas


Yeah, I see no reason to protect this.


change to else if @mkhanbas


1. @requestbody 일일이 붙이기 귀찮다.
2. RESTful , 컨트롤러가 섞여있는 게 이상하다.


Change to else if @mkhanbas


the codes are dirty.


maybe add a small comment - what is it about.. Btw, it looks like it can have a race - as value and gagueUsed are not updated atomically... I know it is the old code - maybe create jira, if there's potential race and leave todo with pointer to it


Bug, if pType does not match either if statement a Poison item will be instantiated as a default Item, which will be null


yup, it is simpler now!


Sorry :/ This removes a feature, and didn't even fix the crash.

Being able to move players that are in other dimensions to the one you are in has always been a feature, and has never had any issues. It shouldn't be removed.

How to reproduce the crash:
1. Have a world with the single player "Player" in one dimension, and a random player (ex:"TrazLander") in a different dimension.
2. Open MCEdit
3. Open that world, you should be at the location of "Player".
4. Try Moving "TrazLander", MCEdit crashes. You were never in the same dimension as "TrazLander", so he was never loaded into memory and crashes.

4b. BUT, if you first toggled your dimension to the same one that "TrazLander" is in, you can freely move "TrazLander" and "Player", even across to different dimensions, because now "TrazLander" has been loaded at least once.

Sorry again if I wasn't clear on it before :/


@mkhanbas

pType sounds like it should be a property of the Poison class


:+1: I'd love to see more warnings during development. You could use something similar to react's `invariant`. @zertosh published it as separate package: https://github.com/zertosh/invariant


test


sdfsadfsafsadf


-should be UsePoison

-Needs XML comments


Needs XML Comments
@mkhanbas


Dudeee!

Yepp, that was it! Thanks! You are right, since it's an APP and not a project. Damn. :) Cheers man.


-Should be ConsumeItem

-Needs XML Comments


What happens if theItem is neither?
it should throw a NotSupportedException


else if is safer and faster
@mkhanbas


@benedekfazekas there's basically a full ns parser in `refactor-nrepl` which I wrote for `clean-ns`.  I'm also using it in the `rename-{file,dir]` branch.


Sorry, I didn't know that. I was mislead by wrong syntaxic coloration in giggle…


デバッグ用コードかな？


Better use a general name like: icon-logo-application


http://heppoen.seesaa.net/article/159896457.html
注意が必要ですね。


This is good, you should use this func and apply it to Andy's code in Gameplay.cs when I merge yours in

Andy's changes are on my develop branch already (they haven't been debugged or refactored yet so CAUTION lol)

@mkhanbas


exactly. my secret (?) plan is to rewrite mranderson using your stuff. the most brittle part of mranderson is the regexp based search/replaces. I want something AST based ;)


おっけー！


@mkhanbas

extra whitespace!
![images duckduckgo com](https://cloud.githubusercontent.com/assets/10816697/7224897/ba89a1a4-e6fd-11e4-89c7-943fd6e33884.jpg)


Oh right, getAdapter always looks for an adapter and does not check if the context already provides the interface. Do schema(context) instead.


@hjdivad I think this commit brings an ideal solution allowing the user to configure autostart on QUnit.config level. I like this idea :+1:

Important is, that ember-cli-addon developers respect this option as well and they don't override it directly like ember-cli-blanket was (debugging the reason why it suddenly stopped running tests take some time)


Valid point. I'll make the necessary clarifications. Thanks for taking the time to report this!


マージのやつが残ってる。


コミットに対してのコメントです。


@mkhanbas
whitespaaaaace


a〜gまでの並びだと他の人がソースコードを読んだ時に理解がしづらいのではないかなと。。。
week_monとか曜日ベースだといいかもしれません。


Thanks for visiting pid.
You are very kind.

But your solution is wrong.
I want to add /ㄱ-힇/. It is mean ㄱ to 힇 words like a to Z (a-Z). (for example ㄱ,ㄴ,ㄷ...가나다..각..힆힇). it is all korean.

Your solution is only working for '힇foo', but it is not wokring for '헬로월드foo'

Thanks for your kind. But I'm still not finding for adding a range to speakingurl.


needsXML comments

needs safe checking of pType, as it stands, still unsafe


http://d.hatena.ne.jp/chiheisen/20110529/1306676847
大文字・小文字意識した方がいいです。
getWeekA()とかのほうがわかりやすいんじゃないかなと。。。


Hmm, I'm not sure what you mean. There should be a check if pedigreeTrigger is defined, regardless of contextaction, and if trigger is defined different code paths have to be taken for view and edit modes?


This will always be true - need to use a different variable name for the enumerated indexPath.


@IanVS Shouldn't the `only`, prevent that. The problem with `ignore` as default was that it ignore the locally installed `sane-cli` because it was in node_modules. So my idea was if it only parses things in the `sane-cli\lib` etc folders I don't have to tell babel to ignore any other folders. See any errors in that logic/behaviour?


I don't see when this would be activated, where does poisonStart get incremented?

@mkhanbas


逃单率分析


あと化け化け。


I don't see where poisonStart gets incremented up or down anywhere it just stays at -4

@mkhanbas


boo


R code

#read in train/test
train <- read.csv("train.csv")
test <- read.csv("test.csv")

str(train)
'data.frame':	10886 obs. of  12 variables:
 $ datetime  : Factor w/ 10886 levels "2011-01-01 00:00:00",..: 1 2 3 4 5 6 7 8 9 10 ...
 $ season    : int  1 1 1 1 1 1 1 1 1 1 ...
 $ holiday   : int  0 0 0 0 0 0 0 0 0 0 ...
 $ workingday: int  0 0 0 0 0 0 0 0 0 0 ...
 $ weather   : int  1 1 1 1 1 2 1 1 1 1 ...
 $ temp      : num  9.84 9.02 9.02 9.84 9.84 ...
 $ atemp     : num  14.4 13.6 13.6 14.4 14.4 ...
 $ humidity  : int  81 80 80 75 75 75 80 86 75 76 ...
 $ windspeed : num  0 0 0 0 0 ...
 $ casual    : int  3 8 5 3 0 0 2 1 1 8 ...
 $ registered: int  13 32 27 10 1 1 0 2 7 6 ...
 $ count     : int  16 40 32 13 1 1 2 3 8 14 ...

 #factorize training set
train_factor <- train
train_factor$weather <- factor(train$weather)
train_factor$holiday <- factor(train$holiday)
train_factor$workingday <- factor(train$workingday)
train_factor$season <- factor(train$season)

#factorize test set
test_factor <- test
test_factor$weather <- factor(test$weather)
test_factor$holiday <- factor(test$holiday)
test_factor$workingday <- factor(test$workingday)
test_factor$season <- factor(test$season)

#create time column by stripping out timestamp
train_factor$time <- substring(train$datetime,12,20)
test_factor$time <- substring(test$datetime,12,20)

'data.frame':	10886 obs. of  13 variables:
 $ datetime  : Factor w/ 10886 levels "2011-01-01 00:00:00",..: 1 2 3 4 5 6 7 8 9 10 ...
 $ season    : Factor w/ 4 levels "1","2","3","4": 1 1 1 1 1 1 1 1 1 1 ...
 $ holiday   : Factor w/ 2 levels "0","1": 1 1 1 1 1 1 1 1 1 1 ...
 $ workingday: Factor w/ 2 levels "0","1": 1 1 1 1 1 1 1 1 1 1 ...
 $ weather   : Factor w/ 4 levels "1","2","3","4": 1 1 1 1 1 2 1 1 1 1 ...
 $ temp      : num  9.84 9.02 9.02 9.84 9.84 ...
 $ atemp     : num  14.4 13.6 13.6 14.4 14.4 ...
 $ humidity  : int  81 80 80 75 75 75 80 86 75 76 ...
 $ windspeed : num  0 0 0 0 0 ...
 $ casual    : int  3 8 5 3 0 0 2 1 1 8 ...
 $ registered: int  13 32 27 10 1 1 0 2 7 6 ...
 $ count     : int  16 40 32 13 1 1 2 3 8 14 ...
 $ time      : chr  "00:00:00" "01:00:00" "02:00:00" "03:00:00" ...

 #factorize new timestamp column
train_factor$time <- factor(train_factor$time)
test_factor$time <- factor(test_factor$time)

#create day of week column
train_factor$day <- weekdays(as.Date(train_factor$datetime))
train_factor$day <- as.factor(train_factor$day)
test_factor$day <- weekdays(as.Date(test_factor$datetime))
test_factor$day <- as.factor(test_factor$day)

aggregate(train_factor[,"count"],list(train_factor$day),mean)

    Group.1        x
1    Friday 197.8443
2    Monday 190.3907
3  Saturday 196.6654
4    Sunday 180.8398
5  Thursday 197.2962
6   Tuesday 189.7238
7 Wednesday 188.4113

#create Sunday variable
train_factor$sunday<-ifelse (train_factor$day == "Sunday","1","0")
test_factor$sunday<-ifelse (test_factor$day == "Sunday","1","0")

#convert to factor
train_factor$sunday <- as.factor(train_factor$sunday)
test_factor$sunday <- as.factor(test_factor$sunday)

#convert time and create $hour as integer to evaluate for daypart
train_factor$hour<- as.numeric(substr(train_factor$time,1,2))
test_factor$hour<- as.numeric(substr(test_factor$time,1,2))

#create daypart column, default to 4 to make things easier for ourselves
train_factor$daypart <- "4"
test_factor$daypart <- "4"

#4AM - 10AM = 1
train_factor$daypart[(train_factor$hour < 10) & (train_factor$hour > 3)] <- 1
test_factor$daypart[(test_factor$hour < 10) & (test_factor$hour > 3)] <- 1

#11AM - 3PM = 2
train_factor$daypart[(train_factor$hour < 16) & (train_factor$hour > 9)] <- 2
test_factor$daypart[(test_factor$hour < 16) & (test_factor$hour > 9)] <- 2

#4PM - 9PM = 3
train_factor$daypart[(train_factor$hour < 22) & (train_factor$hour > 15)] <- 3
test_factor$daypart[(test_factor$hour < 22) & (test_factor$hour > 15)] <- 3

#convert daypart to factor
train_factor$daypart <- as.factor(train_factor$daypart)
test_factor$daypart <- as.factor(test_factor$daypart)

#convert hour back to factor
train_factor$hour <- as.factor(train_factor$hour)
test_factor$hour <- as.factor(test_factor$hour)

# installing/loading the latest installr package:
install.packages("installr"); require(installr) #load / install+load installr

install.packages('party')
library('party')

?ctree

#Description

#Recursive partitioning for continuous, censored, ordered, nominal and multivariate response variables in a conditional inference framework.

#Usage

#ctree(formula, data, subset = NULL, weights = NULL,
      controls = ctree_control(), xtrafo = ptrafo, ytrafo = ptrafo,
      scores = NULL)

#create our formula
formula <- count ~ season + holiday + workingday + weather + temp + atemp + humidity + hour + daypart + sunday

#build our model
fit.ctree <- ctree(formula, data=train_factor)

#examine model for variable importance
fit.ctree

#run model against test data set
predict.ctree <- predict(fit.ctree, test_factor)

#build a dataframe with our results
submit.ctree <- data.frame(datetime = test$datetime, count=predict.ctree)

#write results to .csv for submission
write.csv(submit.ctree, file="submit_ctree_v1.csv",row.names=FALSE)


And you are calling this method from the main thread? I think this method shouldn't know if it's on the main thread or not. If there's some special cases where you want this method not to decompress the image, we could add a parameter `shouldDecompress` and inject the latter from outside, where we can do the main thread check. Makes sense?


xDDDDD


:dancer:


You are right. It doesn't quite work that way. Let's see if we remove `ignore:false` and leave the `only:...` path.


is this project dead?


Ok. I see. It makes sense. Thanks!


@keradus

Do we need dedicated accessors for every single error type, then?


There's an r on the end of line 12.


what Julian means is that protect will always call the handler with or
without an error.

its like try ...  finally in other languages.

On Sun, Apr 19, 2015 at 8:44 PM Julian Rohrhuber <notifications@github.com>
wrote:

> just a note: if you want to catch errors, try is what you'd use. If you
> have an action to be completed before the error is raised, you use protect
> { code to be tried } { code to be completed }.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/supercollider/supercollider/commit/01c0b2d0da2f5f2fb3163286dba156ce5089d975#commitcomment-10793517>
> .
>


@mkhanbas

not sure how this would work


These if statements need to be if-else if-else if  or they are each going to be evaluated every time
@mkhanbas


This is the only `unload()` left in place in this file, the rest are removed. Is it needed?


@mkhanbas

Is this not function not redundant with line 18 and 19?


Excellent tool :D I didn't know about it but I will definitely use it.


Could you pull request this commit?Thanks.


glömde commit kommentaren men har ändrat scene 2 till en meny lagt in bilder och gjort en exit funktion i scene 2.


Why is the Quota disabled before running the rebalance ? If there is an exception, the quota will be left disabled and can the next run fix things correctly ?


Hi, I'm not sure why my commit is showing up, because I checked my desktop Github and it says that I did commit with the phrase "I'm Done". Here's an image:

![screen shot 2015-04-19 at 11 52 53 pm](https://cloud.githubusercontent.com/assets/10472363/7225066/407b0956-e6ef-11e4-90c2-8329653d176d.png)


@mkhanbas

Didn't catch this earlier. Should be      while(playerTurn);


I can see why the new function is added. The issue of that is it could potentially evolve into more duplicate code and become harder to maintain later. More importantly, it's asking users to know what type of sql statements they're trying to parse in order to use the right function, which is the job of the parser. The more desirable way is to keep the current `parse` api, and have it handle the new feature (parsing create-table statement) instead of introducing new separate api specifically for only one type of statement.

One way to do so: a separate private function that handles create-table statement can be added. The main `parse` function can check the type of each statement (`split` function below can be used to split the input sql into individual statement, each of which is tokenized), and get the corresponding handler. If no corresponding handler is found, then it will fall back to original handler, i.e. `parsestream` function. This way it can guarantee that the existing functionalities still work.

As for the new handler, try to re-use existing `FilterStack` if possible. It can be customized by passing in different processors and filters as well as setting grouping flags. The new introduced `CreateTableStatement` class then can be the returned object from the new handler.

One issue that comes with this approach is the tokenization will run twice. It'll be great if we can refactor it so that the tokenization only runs once. It's OK to do it later because I think it shouldn't slow down the performance too much to use it.

It seems to me that new defined Token is not necessary. They can be the properties of the new defined TokenList classes. The new defined TokenList classes look good. One suggestion to those TokenList is have the helper functions or properties return a list of Token instead of a list of token values, similar to the existing TokenList classes.


Wow, there was a lot of excess service stuff in that file.


@mkhanbas

Whats the reason its a do-while loop?  just curious, without comments its not apparent


And here's a link to my repository, in case for some reason a duplication occurred: https://github.com/vinaysatish/sp15-hw4


@akielkopf

did you forget about this section? whats going on here


May be use file_system::FileExist method?


May be add debug message?


Btw, Elixir ships with a Base module that does such encoding/decoding.


Any idea about fixing that. Completely removing `ignore` works better but still breaks some tests. I guess worst-case we have to switch back to traceur.


the variable shouldn't be name something like `maxScenariosDisplayed` instead of `scenarisMax` ?


@akielkopf

this will print even every time the player tries to chose a door *even after* he's beat the first door and is trying to go into the 2nd one


@akielkopf  this will print even every time the player tries to chose a door *even after* he's beat this door and is trying to go into a different one


Changing the definition of the function breaks the code in other tests where this method is being used. Please revert your changes related to restart_mgmt_server method definition change.


@akielkopf

right now setting responseIsGood to true makes it so the loop stops and the player can't chose another door because the game moves forward past all the doors


Ok.


Не забывай ставить аксессуары public/protected/private на методы. Сейчас все по умолчанию publice


Please use c4m_og_feature_enabled_content_types() to check what content types are available.


2/2 Sorry about the confusion!! Not sure why I didn't see this commit the first time (either I was seriously blind for a second or something weird on Github's end) but I'll log it as 2/2  :) sorry again!


Wow, thanks a lot! 2f9d54e26a1233e2361d3f7bf50924454b89c377


The server doesn't compile anymore now.
`
/home/travis/build/mc-server/MCServer/src/Items/ItemHandler.cpp:290:20: error:

      comparison of integers of different signs: 'int' and 'unsigned long'

      [-Werror,-Wsign-compare]

        for (int i = 0; i < ARRAYCOUNT(m_ItemHandler); i++)

                        ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~
`


Allows running the command on Windows


Hello, problems with Google Play!


This should be the same name used when this module is required elsewhere.


Aaahhh ok, it seems to work again. Just performance must have gone down :/ That the regex check costs that much. I will push the current version and we can think about switching back to Traceur if it is that much faster. (`sane install` test no takes ~25s)


http://mongodb.github.io/node-mongodb-native/api-generated/collection.html#find

fileds and hint should be object as written in offical document.


This test module does not test any particular facility, just the general module exposed API. There is no need to wrap these tests with a `describe()` block. This is per the hapi.js testing guidelines https://github.com/hapijs/contrib/blob/master/Tests.md.


Isn't `DEFAULT_PERMS` already set to this in `lib.ITest` ?


@akielkopf

try "if (a.Key is Weapon)"


`orig` is the convention when monkey patching. If multiple patches, use more descriptive (e.g. `origLog`).


@akielkopf

try "if (a.Key is Potion)"


toumy


Safer to restore order as soon as possible.


Yes, you are right. But at least we can skip some of the checks (name, scope, arglist, ...) because these are already guaranteed to be equal because of the hash table.


@akielkopf

I said try in the bools, because a.Key is probably tagged as an "item" by the code and thus masking that is is a weapon - so my suggestion might or might not work, in any case yours works but only if the Weapon does NOT have a custom type i.e. "sword" ,"axe"


That exists :O and ok that makes sense


Good


after this change was pushed out I ran into a issue where chocobos would not load when zoning. Reverting this solved the issue.


k


loadFormDataは編集画面の初期値をセットしている箇所になります。


Won't this always terminate the for loop? If that's the case then only return FALSE here if gdi_InvalidateRegion() returns FALSE.


新規登録処理になります。


更新処理になります。


I tried with a file generated with

```
for i in xrange(1, 40000):
        print ("int a" + str(i) + "(void) { return 0;}")
```

and the freeze when you enter a character is like 8s before the patch and 0.8s after the patch so it really seems to work.

But I think we should also limit the number of tags in the tree by not adding more than say 5000 - it's better to be missing some tags in the tree rather than making Geany unusable.

Finally, also the tree creation is slow - it's the second pass. I think we could save some time by not searching for the parent every time but remembering it from the last iteration and if the parent name is the same, using it immediately.

I'll have a look at this.


I'm sorry


Everything is fine as it is now. You can verify this by looking at the files in the `.deps` directories. They refer to libdime’s own OpenSSL header files instead of the system ones.


Please :+1:  update the pypi package :D


言語ファイルからフィールド名をセットしています。


You can use `UA_STRING(charString)` to save one alloc.


getResultDataは一覧画面で呼ばれる処理になります。


Quit 1


Quit 2


On Sun, Apr 19, 2015 at 4:28 PM, Chris Peel <notifications@github.com> wrote:
>
> Thank you for including the package.
> I am not certain that "Abstract Algebra" is the right place. The wikipedia page for lattice reduction is in "Linear algebra", "Computational number theory", and "Cryptography" categories.
>

As the package author, which of the three categories would you like
the package to be listed under?

Thanks, SVAKSHA ॥  http://about.me/svaksha  ॥


toumy


1) Should the loop continue to run if ret is FALSE ? If not I'd suggest to add (ret == TRUE) to the for loop condition.
2) Don't know if checking XCreateImage result for NULL (line 412 above) should be part of this PR.


wwww


are you check on OKHttp and Apache HTTP Client for android?
let me know please


WTF YOU NOOB


quit 3


Should the loop continue if ret == FALSE ? (And I think there's 1 tab to much indention.)


Hi Dominic,

Access to Flarm_db has been ok I think all through the recent flarm update.

I didn't know how to (or that you could)  pull the OGN_db so worked on flarm_db really as an exercise since code in perl already exists to do this, I just don't like perl.

What I suggest is a value in Settings to say which of OGN_db or Flarm_db is to be used. Then add in code to pull OGN_db or Flarm_db as required, modify the function that does the mapping to Registration depend on the new setting and everything after that remains the same.

Rgds

tobiz

Sent from Samsung tablet

-------- Original message --------
From: Dominic Spreitz <notifications@github.com>
Date:20/04/2015  06:26  (GMT+00:00)
To: tobiz/OGN-Flight-Logger <OGN-Flight-Logger@noreply.github.com>
Cc: tobiz <pjrobinson@metronet.co.uk>
Subject: Re: [OGN-Flight-Logger] Created INSTALL.txt file (f8ab361)

Hi Robiz,

Thanks for checking my pull request. Unfortunately I mixed several
commits in one pull request. The work I did on flarm_db.py should be
disregared, since I created a new file OGN_db.py which contains all the
mods for the OGN DDB. To my big suprise, flarm_db.py was still able to
pull data from Flarmnet. They re-enabled the download of Flarmnet-data.

Rgds,

Dominic

Am 20.04.15 um 00:28 schrieb tobiz:
> Hi Dominic,
>
> Thanks for your useful modifications, I'll look into including them,
> they seem useful additions.
>
> I had noted some one running the code from the Glidern2 server log.
> I've made a number of enhancements in my development version that I've
> not uploaded to the master version yet. When I do next sync these back
> to the master I'll include your enhancements.
>
> Rgds
>
>
> -------- Original message --------
> From: Dominic Spreitz <notifications@github.com>
> Date:19/04/2015 20:28 (GMT+00:00)
> To: tobiz/OGN-Flight-Logger <OGN-Flight-Logger@noreply.github.com>
> Subject: Re: [OGN-Flight-Logger] Created INSTALL.txt file (f8ab361)
>
> Created INSTALL.txt file to make it easier for new users to install
> the software
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tobiz/OGN-Flight-Logger/commit/f8ab3614d3162a2212b7711223f4495732e6cd05#commitcomment-10794346>.
>

--
Dominic Spreitz
Kothauer Str. 123a
85053 Ingolstadt
Germany

Mobil: +49.179.9016643

—
Reply to this email directly or view it on GitHub.


Hi,

sorry for my late answer ... I was away on holidays for a couple of weeks and overlooked this email when I got back.

Feel free to play with the code under ``centro``, but be warned that I put this repository only as a backup for old stuff.
This means that the code was written for comparing different technologies for a web project, Django among them.
As such, it is NOT working. Its only purpose was to stress test certain features we were interested on back then.

Cheers

On Tue, 31 Mar 2015 16:44:27 -0700
systemdog <notifications@github.com> wrote:

> Greetings,
>
> I am inquiring about the possibility of you releasing centro under a
> BSD like license (same as django itself). I was hoping to explore
> building upon it.  Feel free to contact me gmorhart@gmail.com anytime.
>
> Cheers
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lichinka/pos/commit/8f6e34e251a28bb4fe5abdd36e784f9dc02da1b8#commitcomment-10507848


This is a recent addition since overwriting a file did not work so easily on Windows.
Hope it still works with those lines removed.


OkHttp does not support DELETE with body either, Apache httpclient does (with a little bit of custom code, see http://stackoverflow.com/questions/3773338/httpdelete-with-body).


Should the loop abort if gdi_InvalidateRegion returns FALSE ?


Let's remove lambda and we could use this one as HDP multi: https://gist.github.com/keyki/54fdc92c5244fd31ab95


https://github.com/aipocom/test-portlet/blob/master/war/src/main/resources/com/aimluck/eip/modules/localization/AipoLocalization_ja.properties#L1182
あたりの言語ファイルから文字列を取ってきています。


2番めの引数は初期値になります。編集時には元々入っていたデータが予め入った状態で表示されるようになります。


I merged your repository (with some additional packaging fixes) back into https://github.com/gliaskos/freebsd-chromium/

I will commit it to the official FreeBSD repository after some more testing.

Thanks for your contribution!


Sorry the HDP multi is only valid for ambari 2


Yes.


I'm using GitHub! YAY!


YOU :guitar:


For the acceptance tests, let's stick with the "new" vocabulary:
type =calculated
nameInExpression -> code


thank you


This has been reported to break 14.04 LTS.
Pb: It was this type that gcc -Wconversion reported.


this has been tested to death. the answer to your question is: pruning at pv nodes has no impact at LTC and a fortiori above, but is a regression at extremely fast tc (which we don't tune SF for).

I put your test on hold at prio -1, because the framework is low on resources and very busy with many tests that are more useful.


Please remove this, we are trying to phase Indicators/Attributes out.
If you want type safety, add a reference to the `:core:model` and use the `FormField` object.


新規登録処理になります。


更新処理になります。


ALdialogfooterの外に記述するとうまく表示されないかもしれません。


We don't need "Error." because, that would be clear from the context. Maybe you can include the character in the message?

```elixir
raise ArgumentError, message: "invalid character when decoding data: #{inspect original_val}"
```

What do you think?

Good error messages are as important as documentation!  :)


Hi Daniel,

Thanks for your help! We appreciate contributions very much.
The idea of your merge patch is absolutely correct:
to have "default_table_charset" and "used_fields" in the same structure.

However, I think I'll resolve this merge conflict in a slightly different way.
Instead of moving "used_fields" into Schema_specification_st,
I'll just put move the new methods into HA_CREATE_INFO.

Let me explain why:

In 10.1 we split HA_CREATE_INFO into two parts:
- Schema_specification_st
- Table_scope_and_contents_source_st

The reasons to do this were:
-  To avoid passing unnecessary information into functions and methods
- To save on the involved structures initialization
- To remove some duplicate code

mysql_create_db() and mysql_alter_db() need only the Schema_specification_st part,
while the entire HA_CREATE_INFO was passed in 10.0 and earlier into these functions.

Before being passed to the above functions, HA_CREATE_INFO was initialized
with bzero(). sizeof(HA_CREATE_INFO) is 304 bytes, while sizeof(Schema_specification_st)
is only 8 bytes at the moment. So most of initialization was not really necessary.

I don't want to move "used_fields" from Table_scope_and_contents_source_st
to Schema_specification_st for exactly the same reason:
because mysql_create_db() and mysql_alter_db() don't need it.

So let's keep "used_fields" in Table_scope_and_contents_source_st, and
just move the new methods on the HA_CREATE_INFO level, which has
access to both "used_fields" and "default_table_charset".


ALtditemfooterはALtditemheaderとセットになっています。


Although this PR is about `delete()` do all tests need a `close()` too?


Also backported to release26


@timharbour could you make these sample files smaller in file size, i.e. increase the JPEG compression? They take an age to load on low bandwidth office wifi connections...


I believe that this is the commit causing the build failure for themusicgod1. Issue https://github.com/MoonchildProductions/Pale-Moon/issues/52


newでオブジェクトを作っています。


Lol yup noticed that when I went to compile.


Дампы в коммиты не должны попадать.


命名規則的にはgetUrlOnLinkのほうがベターです
http://d.hatena.ne.jp/chiheisen/20110529/1306676847


Typo: 'ncdc' not 'ncdd'.


Старье нужно выкидывать из коммитов


一覧画面で呼ばれる処理になります。


А если оставил старье, подпиши, зачем оставил, чтобы другие понимали, зачем оно тут.


詳細画面で呼ばれる処理になります。
ALCommonUtils.compressString は長い文字列を指定文字数で区切るメソッドになります。


loadFormDataは編集画面の初期値をセットしている箇所になります。