1836664-ReturnWaterTempReset (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1664 of 1664 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)


examples/ex_1.rs:17:3: 38:3 error: unterminated block comment


Kind wish there was an `if not let` block that allowed for early return.  `unless let`.  (Personal favourite Lisp-ism -- `when` and `unless` for literate single-branched blocks.)


Dear Krzysztof Rzymkowski,

Thank you for sharing your good work on FullCalendar for Meteor - well done :+1:

I implemented your meteor package, though modified slightly your example for taking full advantage of Meteor's autotracking in "Template HELPERS" for you to review, if you like (reactive already in realtime without callback and refetch, no need for refresh button as changes show immediately due to magic of Meteor) . Works like a charme for me.

example.html :
-------------
    <template name="calendar">
        <input type="hidden" id="lastMod" name="lastMod" value="{{lastMod}}" />
        <div id="fc"> </div>
    </template>
-------------

example.js :
-------------
    // Session variables
    Session.setDefault('editingCalEvent', null);
    Session.setDefault('lastMod', null);

    Template.calendar.rendered = function(){
        if (!this.rendered){
            var calendar = $('#fc').fullCalendar({
                defaultView: 'agendaWeek',
                header: {
                    left: 'prev,next today',
                    center: 'title',
                    right: 'month,agendaWeek,agendaDay'
                },
                editable: true,
                firstDay: 0,
                selectable: true,
                allDayDefault: false,
                defaultView: 'agendaWeek'
            }).data().fullCalendar;
        };
    };

    Template.calendar.helpers({
        // helpers already handle automatic tracking for reactive computing in real time

        events: function() {
            calEvents = Events.find().fetch();
            return calEvents;
        },
    });

    Template.calendar.events({
    dayClick: function(date, jsEvent, view) {

        var startDateTime = date.local().toISOString();
        var endDateTime = date.local().add(1, 'h').toISOString();

        startDateTime = new Date(startDateTime);
        endDateTime = new Date(endDateTime);

        Events.insert({title: Test', start: startDateTime, end: endDateTime, owner: Meteor.user()._id});

        Session.set('lastMod',new Date());
    },

    eventClick: function(calEvent,jsEvent,view) {
        Session.set('editingCalEvent', calEvent.id);
        //code here for example to open dialog for editing a specific calEvent
    },

    eventDrop: function(calEvent) {

        var startDateTime = calEvent.start.toISOString();
        var endDateTime = calEvent.end.toISOString();

        startDateTime = new Date(moment(startDateTime));
        endDateTime = new Date(moment(endDateTime));

        Events.update(calEvent.id, {$set: {start:startDateTime, end:endDateTime}});
        Session.set('lastMod',new Date());
    },

    eventResize: function(calEvent, dayDelta, minDelta, revertFunc) {

        var startDateTime = calEvent.start.toISOString();
        var endDateTime = calEvent.end.toISOString();

        startDateTime = new Date(moment(startDateTime));
        endDateTime = new Date(moment(endDateTime));

        Events.update(calEvent.id, {$set: {start:startDateTime, end:endDateTime}});
        Session.set('lastMod',new Date());
    },

    eventAfterRender: function(event, element, view) {
        $(element).css('background-color', '#216d9a');
        $(element).css('padding-left', '5px');
    },

    loading: function(isLoading, view) {
            //console.log('isLoading');
        },
    });
-----------
Again thank you for your valuable contribution and keep up the good work.
Saludos, Jürgen


Just FYI you are on the public internet.  Let's discuss at the next meeting


Rather than returning nil, should this return VisitType.Unknown?


I wounder if we want to use `IceNine.deep_freeze` here, to avoid the `[...[].freeze].freeze` construct.


I'm fine with keeping this for the time being. Just recall not to grow but shrink these stylesheets in the future — they are insane to deal with.


Consider validating here?  I see you're filtering `fromJSON` below, but it's weird that you could have a history entry with no visits.


Nice catch didn't even think about that one


@dufferzafar Doesn't even matter, all this code has been ripped out


Result:

Full network:

![sabnzbd dashboard - network values](https://cloud.githubusercontent.com/assets/1273502/7711648/2241f76c-fe6c-11e4-9273-66519f8efadf.png)

No Network at all:

![sabnzbd dashboard --- no network](https://cloud.githubusercontent.com/assets/1273502/7711651/288821b4-fe6c-11e4-8cf7-44549ec69d03.png)


Why do you create create TableViewCells as xib files instead of within the MainStoryboard? For example this one.


Is the manual `NSNumber` cast required?  Can't we make `JSON()` do the right thing for Timestamps?


We don't support writing those types yet. When we do, this code will probably be replaced. My thought was this is only for estimating the size, so it doesn't really matter, but I can change this.


Thanks @pauloborges !


This method always returns documents in Tracking mode. I mean EF would be tranking changes for returned entities allways. But most of the time queries for entities are read only so EF is doing too much extra work for nothing. I would offer an extra method GetReadOnly<T>(int id) that would do the same operation but witth AsNoTracking():
return await Task.FromResult(_dataContext.Set<T>().AsNoTracking().SingleOrDefault(x => x.Id == id));


This constraint can be relaxed to "4.00".


This was intended to reference #283.


Yep :)


It's documenting the output from fastboot process we will parse.


Ok, will do.


What does history mean here?  Is it visits?  Is it site and visits?  Something more?


Dependent on https://github.com/plexinc-plugins/Services.bundle/pull/809


Need to update [the README](https://github.com/HashNuke/heroku-buildpack-elixir/blob/master/README.md#features) to reflect this?


pls test before .add translations .i had ton of issues


The same that I mentioned for previous method


Is this all based on flags?  (I.e., no timestamps?)  That would be rad.


I don't plan on exposing Fastboot into WebIDE for now, so I'm not sure the use of Scanner is really a priority. I'll use your idea of Devices.on/Devices.emit to properly trigger fastboot polling rather than this current hack, though.


I'm not sure we want to push that dependency into spree. I would probably start by upstreaming this, and then present `IceNine` as a solution to use across the code-base.


Done, thanks


We will be creating a compatibility layer with RainTPL3 interface. Also the parser don't have any problems with parsing RainTPL3 templates.


Would it perhaps make sense to use the github wiki functionality for this? Maybe/hopefully that has a lower barrier for users to contribute to. What do you think?


nit: ... `InSeconds`.


This is a copy of the one from Hive. For this code, it probably should be, but when used in `OrcHiveRecordCursor`, it needs to additional columns that aren't in old files. This is now gone as I moved it to the `hive-apache` module.


This commits gumbo-google, a project that will allow me to parse the HTML code from hnhh
I am recompiling this for my mac


This seems a bit strange to me. I've left most of the "ignore this" decisions up to History before.


I've updated the README.


@happppp, I see you caught it in r35857, thanks.


Still need a return here


currently, Michelle is working on this container and every time when she is debugging something or trying to run a command against this container interactively, the command defined in the "ENTRYPOINT" will also get executed by default which makes it kinda difficult for her to run anything interactively after the container started.

Once she is done with making her changes/debugging the container, we can switch back to use "ENTRYPOINT" to start apache.


We could try using the WeakList here. Otherwise we'll probably need to remove this in the didNavigationFail type callbacks.


This file addition looks like a mistake :)


Future Firefox version number isn’t nonsense enough for you? It’s included because a bunch of sites I’ve linked to returned nonsense erorrs or dropped the connection entirely when encountering requests’ user-agent. Opted for something honest (Nikola component) but browser-looking rather than robot-like.


The logic is sound. I understand that you can use if and else statements without brackets, but I do not recommend doing it this way. Its harder to read and you can run into a lot of problems with it.


This is good, but in the future you may want to think about special cases where the user makes an error. What happens if they capitalize 'ROCK' or if they enter a number? You may want to define what you are looking for and keep asking for input if the user doesn't give you a valid format.


that was _not_ stray in the sense it was needed to use any RTDM driver, like RT-CAN. I guess rtdm is now folded in?


We are thinking about removing appInjector and having only viewInjector and hostInjector.


Any reason you switch between >= and > in each if?


I know how to edit and compile with CodeBlocks, but I don't how to do it with GetHub or how to submit.


Yeah. In the context of a MultiColumn in LaTeX, the content is usually a string, or sometimes another LaTeX command, so I was trying to accommodate those use cases.

I wasn't entirely sure how the data argument is handled--it appears that in some classes the information is just passed straight to the super initializer--so I followed that pattern.


Looks like this commit is causing encoded html tags in templates to be rendered in some of the code samples: https://angular.io/docs/js/latest/guide/displaying-data.html#section-showing-properties-with-interpolation


What if we use IceNine for our own changes, than on upstreaming fix this in case of resistance? So doing it "our way" by default and reduce correctness on demand for upstreaming?


Hi Ranon,

Sorry for the trouble. One shouldn't work to long after a hard working day.
:-/

Greetings

Frank

On 19 May 2015 at 21:00, Ronan Nello <notifications@github.com> wrote:

> there is the same string name two times. This currently breaks the build
> of the app.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/holylobster/nuntius-android/commit/d9aba298ea53968e3a07c5b12fb1b40e50475417#commitcomment-11268783>
> .
>

--

[image: --]

Frank Zimmer
[image: http://]about.me/frank.e.zimmer
<http://about.me/frank.e.zimmer?promo=email_sig>


Unexpectedly, `Timestamp(…)` failed to compile.


Yes, but I will add more and more functions as soon as possible.


Ah, good catch.  In this condition, I should have been checking that the _capacity_ was less than `MAX_CAPACITY`.


Yeah, this function is exceeded in its horribleness only by other approaches to the same problem.


Dears,
I'm the author of the article & willing to reply to any questions & open for discussions.


Simple plot...


Just want to make sure you know that you don't have to separate large numbers with `_`s . So `annual_revenue: 300000,` would work here.


here it is


ok


@bennyguitar thanks again you rock :)


I lifted this out, annotated as decimal seconds, and as a handy side-effect Timestamp works now!


Great :) After that can I invite players to party? :)


Long term it would be neat to store whether this was a search engine, a search suggestion, etc. But.... I don't think that's going to help us much. I wonder if we should update the rawTypes here so that supporting sub-types is easier someday. i.e. VisitType.Typed could equal anything from 100 - 199?


Why align all of the columns?


Why did you delete these comments?


I tend to agree with aligned columns here. It makes it easier to visually parse what the string literals are.


This variable is never used.


New Project


I will fix it latter this night sorry for memory waste


Looks like it was caused by `escape="html"` being left on the code block. I'm submitting a fix soon.


Newline and return aren't valid inside JSON strings or other tokens, because they're control characters. That means they can only occur in decorative whitespace, and we pass `pretty: false` when we stringify.


tohle je sice krásný ale zapomněl jsem přidat migraci, fixnuto v 9ced6f21d6d88a5f9904356dc61b4c38cd94a039


I believe they're identical. Certainly it doesn't do them in anything but sequential order here, because the expressions are sequentially dependent.

I used `let` on each line because it makes Xcode's identer work much more sanely.


We'll want to bump the SDK constraint up to 1.9.0 for this.


RadianceRefactor (rpg777) - x86_64-MacOS-10.9-clang: Tests Failed

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


This works well. Just a security tip: if the values true or 10000000 were coming from an untrusted source (ex: user input) you would use this syntax:

```
Store.where("womens_apparel = ? AND annual_revenue < ?", true, 1000000)
```

This tells activerecord to replace the question marks with the **escaped** values, which ensures the user input does not mess with your database.


I know Puppet does. The Resource Abstraction Layer is accessible through `puppet resource <type> <name>` so you can do things like `puppet resource package apt` which will return this:

```puppet
package { 'apt':
  ensure => '1.0.1ubuntu2',
}
```


This is a bug. 44 points was for search bar at main screen. It should be zero by default and needs to be set inside UITableViewDelate method `tableView:estimatedHeightForHeaderInSection:`.


It does, but at the first occurrence it logs

2015-05-19 21:32:46 [WARN ] [.c.c.r.AbstractManagedProvider:138  ] - Could not update element with key keba:kecontactp20:dilbeek in ManagedThingProvider, because it does not exists.


Great job!


@Ladel: Thanks you

@Atis6 : Yes and do everything related to parties ;)


As attribute name, `re2` might be enough (see e.g., `re`).


@joshuapinter Sounds like you'd benefit from splitting up your very long job into multiple atomic jobs, so you have less surface area for interruption, and quicker recovery.


Watch for typos...you put 'retInput()' instead of 'getInput()'

You don't need to create a new variable or use an if/else statement here. You can refactor by using the || operator.

```js
return move || getInput();
``

This one line is all you need to put in your function and it's basically saying the same thing as what you have now. If 'move' exists, return 'move' - otherwise, return the result of getInput().


I don't recommend using the window to make global variables. Global variables in general is something that javascript developers seek not to use for a variety of reasons that will become apparent at the very least next week. I love that your writing modular code, splitting up the program into functions that do very specific tasks. However, its not necessary here, especially when you can just create the variables in the playToFive() function and use this block of code to manipulate the variables there.


Same thing here. Try to refactor.


Oddly enough, I was following the example from your https://travis-ci.org/eudoxia0/cl-charms ;-) so thanks


Heh. If it helps, it was here when I arrived :)


Same things as the above comment. This is not really needed and led you to use global variables on the window object. It's not recommended to do this.


The logic looks great. I would try to refactor the two function above into this one and eliminate all global variables created.


<enableDisqus>1</enableDisqus> sets up to use Disqus the next time these documents are published. For now, yes/no helpfulness ratings are the only feedback method available.


I'm okay with `ImmutableList`, but changing it would lead to a big refactoring since most of our operators are using arrays in their constructors, and it is not the purpose of this PR. Also, there might still be some overhead spent on unboxing them, even on critical paths such as  `TupleUtils.tupleEquals()`.


Not using numbers on ProtoMember is dubious - it makes it easy to accidentally break compatibility by removing a field or reordering fields in the file.


Overall, the code looks great and first and foremost, it works. There are a few comments I made above that has to do more with best practice. You're in great shape going forward. Let me or Remina know if you want to us to go over your code more in depth together.


@uberllama You're probably right. It's just an export task that loops through lots of records, does a bunch of math and then generates a CSV with a row for each record. Not sure how I could split that up... any ideas?


Oops - I left a file out when I was doing the commit. I'll remove them in a follow-up [FOLD]


Nice! This is one of the most concise getWinner functions! Great job! :+1:


You can take this console.log out since you are repeating it below within your 'while' loop.


Aside from this being done _while in class during review_...


Hey Eric. Hope all's well. I just happened by and noticed this commit. The reason for the version is optimistic concurrency locking, not ordered retrieval. Consider: an event comes in and is being handled by client a, then client a hydrates a model from the stream, then an other event comes in and is being handled by client b, then client b hydrates a model from the same steam (same aggregate I'd). Without the version check, now we're in a race. If b processes faster than a and appends its event first, the events will get inserted in reverse order. The version prevents this by rejecting conflicting updates to stale models. There should be a PDF of this book:
http://www.informit.com/store/implementing-domain-driven-design-9780321834577

Floating around the office. There is an appendix the outlines the function of the version in an eventstore there. In fact, the whole discussion there is pretty good, though snapshoting is obviously different in this implementation.

Bon chance!


@natecavanaugh Are we not putting parenthesis in those cases anymore?


Is this a testing hook?  If so, we might need to use a more mangled name.


@ChronoMonochrome what is this flag?


1836664-ReturnWaterTempReset (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1664 of 1664 tests passed)

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


Just for readability for me anyways. I don't mind if it's not.


Why the special sentiel, rather than `None`?


It should be noted that since the JobStorage doesn't currently manage the queues in any way, and will just blindly adds whatever is passed to it, you are very likely to get WRONGTYPE exceptions if you drop this commit into production directly.
Delete the `[Prefix]queues` (eg. `hangfire:queues`) key manually if you wish to use this commit before a release is made officially.
I have not yet decided on the best way to avoid this error, but if I had my way in design, queue creation/error checking would be guaranteed at Hangfire initialization, rather than by hoping each JobStorage does it manually. Possibly by adding a `virtual SetQueues(string[] queues)` to the JobStorage definition.

The overall idea behind the commit is that a set never guarantees any ordering, so a sorted set is required for queue priority.


the `_DERIVED` check is redundant, given the early exit above.


r=jedmccaleb


Since move might not have been defined yet, call getPlayerMove() and getComputerMove() without a parameter. The parameter will automatically be set to undefined until the player or computer sets their input.


Initial Commit


OIC -- this definitely needs to be mangled to avoid the possibility of a real `_DERIVED` member in a wrapped object.


Hmmm you could potentially use the initial job as just a kick off which queues multiple jobs using find_each and small batches. Maybe build your transformed rows that represent the CSV in a table, and have a final task that generates the actual CSV from that table. Plausible?


this will fail _very often_ because connections will be cut all the time mid-sends.


this failure should be considered (and handled as) normal operation. it's barely debug.


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/296b55273112d7d276652e6f2ac1a295821de784


merging MonsieurNicolas/stellar-core/loadFrames = 296b5527 into auto


MonsieurNicolas/stellar-core/loadFrames = 296b5527 merged ok, testing candidate = fa4c63f2


This won't work


RadianceRefactor (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


hi @bep, for rss links? How so, for example? Or were you responding to the guy asking about link blogs?


OK, I see that `None` is used to signal "no subclass needed" below (I think).


Needs a lock to avoid double instantiation


@Solistra  Yes please delete your fork, I can't transfer ownership to you until you do.


Thanks @akesterson . :+1:
I'm actually delighted to see true collaboration. ^^


You don't need to pass in these parameters here. Leave the parens empty.


You also need to add these handlers to the Help shim (found in HelpTab.java) just like the onHelpHome command that is there now. This is because if the Help pane has never displayed the commands won't find a handler unless there is a HelpShim abstract handler there is a hint to page in the Help pane.


Can this just be NSOperation*? That way we don't expose more than what's needed. In the future, if required, we can always return the specific subclass without breaking anyone.


@JustBlackBird the problem is when you want \\{foo}} => \bar


I suggest to replace 2) with the following:
```c#
[TestClass] // otherwise all derived will ignore AssemblyInitialize-marked method
    public class TestBase
    {
        [AssemblyInitialize] // runs only once for whole assembly
        public static void AssemblySetup(TestContext ctx)
        {
            TEST_BASE_PATH = Path.GetDirectoryName(Path.GetDirectoryName(ctx.TestDir));
            //init others
         }
```
The benefits here are a)speed improvements b)encapsulation of init logic c)if test are run in-parallel we ppbly don't need to invent the wheel with monitor/lock solution


Also I think its ok here. the problem is the PHP and how it handle \ in strings.


@akesterson Done, and thank you!


Here's where you need to pass in your playerMove and computerMove parameters. Take another look at how you defined your getWinner function on line 44. See how it calls for these two parameters, which are being set by getPlayerMove and getComputerMove?

Pass them in like this:
```js
getWinner(playerMove, computerMove)
```


obrisan md file


Added gemfiles


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (348 of 348 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)


Great! thank you so much Kai-Uwe


AIYAH!  I see what this is trying to accomplish, but am convinced that it will bite me in the butt six months from now, when I have forgotten that the wrapper has a slot for `__dict__`.


This looks like a good fix. Any interest in submitting a pull request? Thanks!


To upgrade existing DBs rename `group_id` to `group` like this:
```
db.projects.update_many({},{'$rename':{'group_id':'group'}})
```
or on pymongo < 3.0
```
db.projects.update({},{'$rename':{'group_id':'group'}}, multi=True)
```

Also at this point one should set the group on each session to that of the project:

```
for s in db.sessions.find({}, ['project']):
   ...:      p = db.projects.find_one({'_id': s['project']}, ['group'])
   ...:      db.sessions.update({'_id': s['_id']}, {'$set': {'group': p['group']}})
```


Just right now, I have pushed a fix… that should show up on our Satis server in the next hour.


Vote for this change. We also needed to apply a patch to this erb to use our own LDAP URL. We are using Oracle Unified Directory, and the user CN is different than this bind_dn here.


Functions dont return values unless explicitly told so. Try using the return statement to return move.


Yeah, that's what I was thinking as well, but the overhead seems a bit too heavy. Might be best just to reduce the time required for the task itself via caching or benchmarking.

Thanks for the tip.


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (348 of 348 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)


Same as above.


Everything appears to be working correctly. I wouldn't worry about it.

On Tuesday, May 19, 2015, Paul Robinson <notifications@github.com> wrote:

> Agreed. Although I was concerned that it was a sign of an
> incorrect/incomplete merge conflict resolution.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/awestruct/aweplug/commit/c0f6b17d5dfd4304a7e2aaf0914bf89e9fc1b759#commitcomment-11268406>
> .
>

--
Sent from Gmail Mobile


:+1: fucking awesome


No!


I dont know, what is the problem at me ... For me didn't works :/
For example : (my char name is Atis) : /inv Atis --> You cannot invite yourself into a group.
But for other player while i trying to invite, nothing happens :/


Still not good


@Solistra transfer request sent. Once it's transferred, I'll send you a PR to remove my "Legal Notice" wording from the README. Good luck with SES VX Ace, it looks like some cool stuff.

@ZeeCoder No problem. I would've liked to have kept working on it, but, C'est la vie. The lesson here is "pay closer attention to the Business Protection Agreements you sign".


Where's the test :stuck_out_tongue:
(Seriously, I think there should be a test guarding against regressions here.)

BTW, I believe there are cases where things can break (not XSS vulnerability, but unexpected/incorrect handling), when `$element` contains other element nodes.
E.g.:

```html
<md-item-template>
  <span md-highlight-text="...">
    <span>{{ item.id }}</span>
    <span>{{ item.label }}</span>
  </span>
</md-item-template>

// e.g. consider entering "spa" as search text
```

(A proper solution might be non-trivial though.)


For upgrading existing DBs, take the SDM UI down and rename the tokens collection to authtokens:
```
db.tokens.rename('authtokens')
```


It turns out that all the wrappers had `__dict__` anyway. The wrappers that are actually used, `ImplicitAcquisitionWrapper` and `ExplicitAcquisitionWrapper` didn't specify `__slots__` so they got a dict by default. I added it to the slot list for documentation.


Indentation


We can mass fixup all of these later and make it an error in the modernizer plugin.


There's an extra opening carrot here.


@alexblair1 DWMR. Press on. Speed is your ally.


Just learned that this is not being actively maintained anymore; would anyone object if I forked the project and started working on it myself (including rewriting -- potentially large -- portions of it)?


"Hide" isn't really right here — at the moment that it appears, you're not really telling it to hide anything. I think "Don't Show Again" would be better. (Firefox appears to use "Don't Tell Me Again" for the slow startup notification.)


tratando de relacionar las tablas db.define_table('cursada',
	Field('cursada_id','type:id'),
	Field('curso_id',db.cursos.curso_id),
	Field('alumno_id',db.alumno.alumno_id)
dije tratando!!


holi


ok!


Good rough draft sir. Spot on. Found a little typo on the word 'each'. Line 11. :+1:


use setTexture


* The Egg Man


THis is a test push.


完成传图片


This should probably declare `@Nullable` in the return type.


This is arbitrary, connector-defined data that then gets passed into the `commitDelete()` method. It is the same as the create/insert APIs.


Delete this line too


I need to fix this ... sorry


Because the length of DNA sequences is 10.


1. Installed devise `rails g devise:install`
2. Set up ActionMailer in development.rb
3. Created views `rails g devise:views`
4. Created a user model, but with devise, not rails `rails g devise User`
5. Added custom attribute to devise migration
6. Ran migration `rake db:migrate`


94639630-HVACTemplateScheduleType-4860 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (627 of 627 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.29%25-green.svg)


Why should composer.lock be added to the repo? What is the use case for enforcing all users of the app to use the same version of included packages? Seems that it now adds maintenance to the package to update regularly.


I saw you copied the original definition and I've been working on a PR to
convert the existing COM interfaces to use it. Likely this weekend I'll
update for the latest definition and finish testing to submit the PR.

On Mon, May 18, 2015 at 7:42 PM, Peter Atashian <notifications@github.com>
wrote:

> I borrowed this macro and turned it into retep998@c51c581
> <https://github.com/retep998/winapi-rs/commit/c51c581e3e18b1e1179524a5530c9faced41b21f>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iliekturtles/winapi-rs/commit/71b2a14ce54ff22886fb4ae9a74e828d3e985512#commitcomment-11251677>
> .
>


I think it would be simpler to return an Optional LocaleId (or even `@Nullable` LocaleId), and perhaps create a helper method which can build the BAD_REQUEST Response.  In this instance (no monads, no lambdas, no flatMap), all the Either stuff just seems to be leading to more code.


I love you this rocks, nice add with all you've done.


I don't have access to Brian's changes, but after our discussion and looking through the code my changes should be very close to his.

The codeconnect plugin still utilizes a transaction due to performing a refresh on the project node in the first request (to remove all dependencies) followed by bulk update of all dependencies in a separate request. Ensure Brian's changes still performs the refresh since it is needed for when dependencies are removed from a project.

Also, the transaction API is the way to go even if we are able to do without since they have already deprecated the cypher rest endpoint for one off queries: http://neo4j.com/docs/stable/rest-api-cypher.html#rest-api-send-a-query

I'll go ahead and merge this patch, have Brian try it out on some of your projects and compare the performance to what he's done. If its still super slow, send a copy of Brian's modified downstream.gradle and I'll see about integrating them into a future patch.


Ah, my bad.


這個變數的意義？


closes #46 (not 47)


建議把 ex.ToString() 放在Show裡面，這樣有發生例外的時候可以直接顯示出來。


Can commitCount ever not be an integer?


did you try it ?


Dawn, I forgot to change the
for (int opposed = 0; opposed <= 1; ++opposed)
for
for (int opposed = 0; opposed <= 2; ++opposed)
!!!


Straight from the Composer project itself:

https://getcomposer.org/doc/01-basic-usage.md

Sent from my GrumpyPhone

> On May 18, 2015, at 10:18 PM, Adam Culp <notifications@github.com> wrote:
>
> Why should composer.lock be added to the repo? What is the use case for enforcing all users of the app to use the same version of included packages? Seems that it now adds maintenance to the package to update regularly.
>
> —
> Reply to this email directly or view it on GitHub.
>


변범도 rubocop 설치해야함~~ css 대괄호 앞에 공백이나 indentation이 잘못된 곳이 조금 있는듯


One thing of note that has changed: I've switched the hard coded neo4j url in the webapp to localhost since it is more appropriate/generalized for open source code. Can look into the possibility of putting this info in a more centralized location (dedicated config/properties file)


Hola


It's used in older version still in use so I re-added the support, no harm in it saying around if not used in newer builds.


It seems ok to me.

Example:
1. `prev = 1000`, `next = 1007`, `diff = 7`
2. `prev = 1000`, `next = 2`, `diff = -998` -> overflow -> no changes on `energy_sum`
3. `prev = 2`, `next = 13`, `diff = 11`
...

What do you think?


Don't update the version number. I do that when I push a new version to crates.io


Would probably make sense to put this logic in the constructor for FlexVersion, taking in a tag as an optional additional parameter.


Thanks @Zangetsu38 .Save me time to compile .


Yeah, that makes more sense. I had "Don't Show Redirect Notifications" but it was absurdly long.


Can you leave a blank line and then an inline comment above this if statement explaining that this strips the directory information out of the full size image name?


D'oh


This is the power of Shinkansen. Adding 11 dynamic animations to a landing page with very few lines of codes. This is the JavaScript, Markup, and Dependencies. Bower helps keep other peoples code out of our repos. The config file makes it easy to see what's being used on the page.


@djdeeles Sure I'll push it in a bit
@Tsunamical You need to have taken the KK ota or manually update your firmware. What variant are you using?
@GalaticStryder What branches are you using?


Thanks. Thought the practice was to keep dev teams in sync rather to enforce users of a package. Now we have these notes here for future generations to see why it was included.


The idea of the PEM password is to protect your certificate in case of system intrusion, if you' re going to include it inside your code (which would be accessible to him in that case) so i suggest you just remove the lassword at the first place. It's an extra layer of protection anyway. For the client SwitchHTTPS() function edit, i'll put it on the next version.


Nit (here and above): consistent spacing, and semi-colon for good measure

```javascript
{label: "settings", callback: function() { _prefsOpenCallback(bw[1]); }}
```

or (particularly if it goes past 100 chars)

```javascript
{
    label: "settings",
    callback: function() { _prefsOpenCallback(bw[1]); }
}
```


@acmeid

这是一个warning，意思是“fis应该全局安装”，一般就是在你当前项目目录下的node_modules里有一个fis安装


rax retry tests


Might be able to get away with calling `scrollToOffset:` and letting the animation block override the duration. @nicklockwood is probably the best person to review this code.


it was me the whole time


If `has_stable_footing` is true, the target doesn't have stable footing?

Also, I'm not sold on the name. Names starting with `has_` can generally be expected to always return the same value unless player's state changes.


area monitores, perfiles y reservas gerente


Interesting. I honestly didn't know that would work. That would prevent editing RCTScrollView and RCTScrollableProtocol. I will try this and see if it works.


I think TEST phase should be examined before since now the moving average includes  the test batch_statistic


Thanks!


Well, RTFS :) All you really need is ``ddb`` and the ``_create_device`` function from ``worker_db``. I guess exporting ``create_device`` and using it directly in ``artiq_coreconfig`` would be a nice solution, as otherwise you'd need to pass dummy pdb/rdbs to DBHub.


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


Take this out


Yup.
      From: gabbyzhou <notifications@github.com>
 To: gabbyzhou/RepData_PeerAssessment1 <RepData_PeerAssessment1@noreply.github.com>
Cc: Chung Cher Tee <chungchertee@yahoo.com>
 Sent: Tuesday, May 19, 2015 8:18 AM
 Subject: Re: [RepData_PeerAssessment1] peerassessment1 (827553e)

The plots can be found in the md file.

On Sun, May 17, 2015 at 11:50 PM, Chung Cher Tee <notifications@github.com>
wrote:

> where are the plots?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gabbyzhou/RepData_PeerAssessment1/commit/827553edef8996732fb9087fa722960a6adf5833#commitcomment-11233348>
> .
>
—
Reply to this email directly or view it on GitHub.


Didn't we miss something in the second step? The way I see it, when a overflow happens, we miss counting some energy packets.


@Rashed97 I've split the device trees for the cm-12.1-caf branches to compile as d802caf and it did compile properly. I'm using all cm-12.1 branches for g2-common, d802 and vendor with some little path changes. The kernel I'm using msm8974-caf with Tsunamical's changes.

I have a build here http://www.slimdevs.com/VexxoAOSP/buildsdev/d802caf/
I couldn't get a logcat cause it didn't pass the boot logo. I'm doing these builds in backgroung for testing, my main branches are all using the old way.


(This one is not in geany.)


The satellite sighted contrail co-ordinates (32.44S,87.00E to 34.11S, 86.37E) do not seem to match a great circle route flying south from Aceh, but rather parallel a curving Rhumb line from IGARI passing through Kuala Lumpur into the SIO.

Since a Rhumb line matches what an uncommanded autopilot would do following the last magnetic heading it was following after a decompression, it seems to suggest a magnetic heading south from IGARI, not a detour around Sumatra.

The Southern Indian Ocean has a significant Agonic variation eastwards towards Australia for any aircraft flying south therefore a Rhumb line south would curve east like a banana.


It would be good to consolidate some of the scrollTo functions. Here are a couple ideas:
```js
// null/undefined duration = default animation, 0/negative duration = no animation
scrollTo(y, x, duration?: ?number)

scrollTo(y, x, options: {animated?: bool, duration?: number})

// Is it confusing that DOM's scrollTo takes (x, y) instead of (y, x)?
scrollTo({x?: number, y?: number}, duration?: ?number)
```

cc @vjeux


場所の記述は正確に！


これも正確に！


Change this to line 3.


まちセンとの日程調整をやってました
追加してください。


I like this idea. If the powers that be feel like we should do that, I can change that and then also add docs that explain it. Right now, the scroll view docs lack any information about "scrollTo"


レビューの結果しか書いていないのはなぜ?
少なくとも、3限に参加できなかった人が
これを読むことで会議の内容が把握できる程度まで書くべき。


Oooh!Oh my God,yeah!Well,it's kind of late now...Oups!


Nothing happened


どのような書籍をつかって
どこまで完了したのか書く


勉強会ではない。


Just replace __cmpxchg32 and __bswap32 and don't abuse endian swapness :-)
```
auto p1 = (uint8_t*)state->memory()->TranslateVirtual(plist_ptr);
auto h1 = *p1; // no need for endian swapness
while (h1) {
    auto p2 = (uint8_t*)state->memory()->TranslateVirtual(h1);
    auto h2 = __cmpxchg32(p1, h1, *p2); // no need for endian swapness
    if (h1 == h2) {
        SHIM_SET_RETURN_32(__bswap32(h1)); // this is the only place where you need endian swapness
        return;
    }
    h1 = h2;
}
SHIM_SET_RETURN_32(0);
return;
```


Apparently it's an "issue" with lodash. I was relying on the fact that `_.intersection` returns the array itself if only called with a single array (https://github.com/lodash/lodash/commit/2884f8acb8e7380c5c77d3522aaa4373c1f674f2).

Sorry for suspecting ast-types @benjamn!


@Bitti09 Now is building here ... If you can test, give it a try


タイトルと内容が合っていない。
ここでは何を伝えたいのか。


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


「ほぼ」ではないです。全員です。


rax retry tests


再度日程調整することが伝わらない


ユースケース図


Derped up git and didn't change to my suggestion branch to commit this. I'm going to go commit sudoku now.


いつ？どこで？


@asvetlov Looks like this commit removed `**kwargs` but the docstring still says they're accepted. Will fix in my patch.


If there is td-jdbc-0.3.2-jar-with-dependencies.jar in lib directory, an error occurs.


Indentation completely wrong


This is duplicate information to the main tag which is always present. If I'm not wrong just delete all run_duration stuff and we can merge.


@vouillon @dinosaure This one seems logical, but the header was not propagated in the code before switching to cohttp. However, it is [here](https://github.com/ocsigen/ocsigenserver/blob/5c9723e63a5231ef7318d5d0c71713ddbc73c19c/src/http/ocsigen_senders.ml#L485). It's weird and I'm not really sure.


RadianceRefactor (rpg777) - 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)


In my realisation controllers only connecting data sources with table view. So the similarity between search and related view controllers is in their data sources. And as you can see [SSARelatedProductsDataSource](https://github.com/heach/iOSAssessment/blob/master/iOSAssessment/TableView%20Accessories/DataSource/RelatedProductsDataSource/SSARelatedProductsDataSource.h) is inherit from search datasource.
But I understand why you asking, I will think about better way of connecting datasource with tableView without creating bunch of controllers.


This is not necessary?


If two servers are passed through in yii configuration then this change will count the number of servers and attempt to ping them.  If a server/node does not respond then it would typically result in an exception rather far up the tree which was impossible to catch.  By attempting a connection prior to using the client for a legitimate purpose we can remove a failed node and still have a functioning site.

This is the result of requiring at least two nodes for any of our services being offered and attempts to mimic the behavior of memcached, although in a very rudimentary way.


Same for me on Xcode 6.4 (6E14), no menu item. Got excited with the new release a few hours ago and tried again. Also uninstalled and re-installed.


No `git commit -a` for you, sir: `git add` and `git commit` like a responsible dev.


Instead of the creatorUserId shouldn't it be the currentUserId? Because if we provide the creatorUserId, when updating  it will still appear in the notification that the user who did the update is the user who created the asset?


I have been digging into that and have found that on [this line](https://github.com/NREL/EnergyPlus/blob/60e89ee745fa06d7e003bc599821ec99620d12b6/src/EnergyPlus/InternalHeatGains.cc#L4509), the `ConvectGainRate` for the heat pump water heater tank is returned as `nan`. It's not clear to me where that is supposed to be set, however.


Maybe refactor that for clarity?  E.g.:

```python
    def __getattribute__(self, name):
        obj = _OGA(obj)
        if name   == '_obj':
           return obj
        container = _OGA(container)
        if name == '_container':
            return container
        if obj is not None or container is not None:
            return _Wrapper_findattr(self, name, None, None, None,
                                     True, type(self)._IS_IMPLICIT,
                                     False, False)
        return _OGA(self, name)
```


If a function is returning something, you must catch that value in a variable.


Uh, this isn't a CSS file... And where did it come from even?


```
[RSS 2.0 template](#the-embedded-rss-xml:eceb479b7b3b2077408a2878a29e1320)
```
==>

```
[RSS 2.0 template]({{< relref "#the-embedded-rss-xml" >}})
```

Will survive renaming and moving (yours would also survive renaming, so this example would be better with a reference to another page).


But `scripts.js` was correct?


And what is this?


It seems like you're having a little trouble with when and how to pass in parameters. Don't worry though, since we haven't officially learned how to do it - we'll be learning about parameters on Thursday during our lesson on functions. Let me know how you're feeling after Thursday and we can go over this assignment if you want.


Should be a tuple?


Cool :-) Now we just need a safety confirmation dialog for the reboot-button.


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (348 of 348 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)


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/63228252


@gkalpak Right, the intent of `md-highlight-text` is to ONLY contain text.  I think proper handling of this would be to warn the user if they mis-use it.  The idea is that your example should look like this:

```
<md-item-template>
  <span>
    <span md-highlight-text="...">{{ item.id }}</span>
    <span md-highlight-text="...">{{ item.label }}</span>
  </span>
</md-item-template>
```


Hey great job on the mininet-wifi so far, I'm working a lot with floodlight and it work out well
sudo mn --wifi --ssid=odin --controller=remote,ip=10.0.2.2,listenport=6633

By the way - Do you know the Odin-Framework it is SDN-like solution for SDWN. I worked with this framework and now it is running on WiFi testbed. Would be great to talk to you about the mininet-wifi.
I would like to virtualize the wireless access points.


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/5fa44d5eef388206d08c18c4ce1732eb5dcc56dd/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/5fa44d5eef388206d08c18c4ce1732eb5dcc56dd/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/5fa44d5eef388206d08c18c4ce1732eb5dcc56dd/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/5fa44d5eef388206d08c18c4ce1732eb5dcc56dd/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/5fa44d5eef388206d08c18c4ce1732eb5dcc56dd/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Fixes #10 .


Could unlzexe split out in it's own standalone program ?

Could refkeen lookup for data in a/several standard locations under `/usr/share/games` ?


You need to get rid of my silly +1's too :-)


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (348 of 348 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)


I can't comment on some of the lines because of the way commits work but:
lines 40-53: It is not recomended to use conditional statements without brackets to indicate their body. Its not intuitive to read and can cause a lot of mistakes. Also, parenthesis are used to house the conditionals, not brackets.

I can definitely see the logic that went into this program, and that looks great. There are some syntax issues, as I've tried to explain in the comments above. As we go through the class (especially after the first week), you should be able to fix most of the problems with the code above. I can't list out all of the things that need to be fixed above here, but if you want more feedback, let me or Remina know and we can set up a time to go through the code more thoroughly. I do recommend going back and fixing the code after the first week (maybe this weekend), as it would be great practice.


Is this still a tab character at the beginning of this line?


Nit: `550px` seems unnecessarily short to me, especially since the current value is `800px`. Could we increase the value a bit?


Hi Sylvain, thanks again for your answer, I have to confess that I’m thrilled that your’re helping me with this. I switched computers because I was not sure about the Cordova installation on the one I was using and I started the project over. The plugin seems to have installed correctly from the command line, and the object OSCSender is being created but the message never arrives at its destination. The listener is working fine, actually it is a little Processing program using oscP5 library in the same network. It listens to a processing sender program in my other computer as well as messages from touchOSC app in my iPad. What I find odd is that sender.send doesn’t throw any response from sucessCb or errCb. If it is not too much to ask I’m sending my code for you to have a look. I’m sending the message from within onDeviceReady. I also tried to create the object within it and send the message when clicking an element on screen but it didn’t work either, thanks a lot:

Axel

   <script type="text/javascript" charset="utf-8">
    	// Wait for PhoneGap to load
    	document.addEventListener("deviceready", onDeviceReady, true);
    	// PhoneGap is ready
    	function onDeviceReady() {
        	//alert("onDeviceReadyy!");
        	var sender = new OSCSender("192.168.1.31", 13000);
        	if(sender){
        		alert("sendersUp!”); //THIS IS WHERE I TEST IF THE OBJECT WAS CREATED AND I ALWAYS GET THE “SENDERSUP” ALERT
        	}else{
        		alert("noSender");
        	}

        	var data = [456, 789];
		//alert("data[0]: "+data[0]);
		//alert("data[1]: "+data[1]); //HERE I WAS JUST CHECKING IF THE ARRAY WAS FINE, IT WAS
		sender.send('/message', data, successCb, errCb);
		sender.close(successCb);

		//CALLBACKS
    		function errorCB() {
    			alert("Error sendind osc msg");
    		}
    		// Transaction success callback
    		function successCB() {
    			alert("success!");
    		}

    	}
    	</script>

> Em 18,mai 2015, à(s) 17:38, sy1vain <notifications@github.com> escreveu:
>
> Hmmm.. I haven't looked at cordova for a while, but it should work if you installed it via the command line correctly. It sets all the permissions on install.
>
> Do you wait on device ready before creating it? And are you sure you are listening correctly on the receiving device?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/sy1vain/cordova-osc-plugin/commit/7e48be6b37a4abf5706e9ff712abd1336bc29858#commitcomment-11249009>.
>


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (348 of 348 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)


Whoops...


I think this should be the primary filter parameter. On the user end they can find users using a partial display name.

Searching by user_login should be protected by api_key validation. I don't think users should be able to see everyone's login names. The user_login field shouldn't be required in the schema.

We can support searching users by id with Path Templating.
For an example look at this method. https://github.com/osu-mist/catalog-api-test/blob/b0818fc6b909dbf83ca2882d39c61e5347f55118/catalogapi.yaml#L109
The path would end up as /users/{user_id}


+    GROUP_UPDATE_FLAG_VEHICLE_SEAT      = 0x00104000,       // int32 (vehicle seat id)

that's strange


Closes #50


Because search view represents the similar list of products as related view does, but using different request to load data. So I create inheritor and override request inside it.

Yes, I was thinking about moving requests out from datasources. So we can load objects separately, at some kind of data provider for example, and then initialise datasource with them. In this case we will have one UITableViewDataSource for every product list in our application.


`css.css` is my favorite so far...


I think you don't need this, you could try:

```ruby
sockets ||= addresses.map do |host, port|
  socket = UDPSocket.new
  socket.connect(host, port)
  socket
end
```


This should be moved underneath /users/{user_id}
Individual users within the users collection are updated with put.


It would quickly overflow your local storage and you wouldn't be able to
save / create any additional presentations.  I think Chrome has finally
fixed their bug with the filesystem api so I may be able to cut over to
https://github.com/tantaman/LargeLocalStorage and use actual files at this
point.

On Tue, May 19, 2015 at 2:40 PM, mwcz <notifications@github.com> wrote:

> I'm pretty tied to base64 images.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tantaman/Strut/commit/7a2a07f3e890e6fe5106b744db2053553fd3583b#commitcomment-11268349>
> .
>


@skeptical nice one!


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (348 of 348 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)


There's a comment right before the import that tells you why we do this :-)


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


Writing on the train... Yeah.


there should be something. I am not quite happy with the pkg-config part yet…


Yeah, I ran into the problem of some values being ints when declaring a stronger type for the data in https://github.com/bamos/reading-list/blob/gh-pages/get-random-quote.hs.
My "solution" is to quote numbers in the YAML document that should be strings,
but I don't like doing this.


Almost... CSS files go where? And with what tags?


Goodjob <3


I don't think this pattern is right, since it establishes separate negative lookbehind assertions for `-arm` and `-armv7l`. Can you try this instead:

    {"(?<!-arm|-armv7l)-[^-]+-linux.*-32$", "CFLAGS", "-m32"},
    {"(?<!-arm|-armv7l)-[^-]+-linux.*-32$", "LDFLAGS", "-m32"},

If that works, I believe we need it on the lines for `-m64` as well.


[object Object],[object Object],[object Object],[object Object]


I really like this line. It's very nice.


Hey,

Please revert this change, it has killed my ember-cli project (and many others as well).

Thanks,
Dan


undefined


>p.s. вихрь на 1.5к строк знатный вышел, конечно
на самом деле я не посмотрел, там как минимум `main()`  на триста строк можно выпилить


where is `filePosition` initialized... should be set to the start position of the first stripe


I object to this change. It depends very much on the PT available to use. PhraseDictionaryOnDisk doesn't get faster when used with more than 4 threads and PhraseDictionaryCompact hits its limit with 16 threads I think. Furthermore most CPUs nowadays have HyperThreading, meaning you'd have twice the number of actual cores exposed as virtual CPUs. It is not always beneficial to run moses on 8 cores if you only have actually 4 cores available on your machine.


1640-AddHVACPart1 (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


/test2.css:53:5: CSS outline not supported by: Opera Mini (5.0-8.0)


Closes #9


yo


We can stick the representation of the resource in the body of the request. Then we can refer to the User schema.


/test2.css:44:5: CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)
/test2.css:45:5: CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)
/test2.css:46:5: CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)
/test2.css:53:5: CSS outline not supported by: Opera Mini (5.0-8.0)


@nmerket That comes from  WaterThermalTank( ).AmbientZoneGain.  The setup is here:
https://github.com/NREL/EnergyPlus/blob/60e89ee745fa06d7e003bc599821ec99620d12b6/src/EnergyPlus/WaterThermalTanks.cc#L4217
```
						} else if ( SELECT_CASE_var == StratifiedWaterHeater ) {
							SetupZoneInternalGain( WaterThermalTank( WaterThermalTankNum ).AmbientTempZone, "WaterHeater:Stratified", WaterThermalTank( WaterThermalTankNum ).Name, IntGainTypeOf_WaterHeaterStratified, WaterThermalTank( WaterThermalTankNum ).AmbientZoneGain );
```


Ah. Now I see what you're referring to. I saw, but did not understand that link syntax fully when I made this edit. I thought I'd better not touch it since I didn't write the original and didn't understand. Are you saying I should edit this before you merge? If so, sorry, but please advise how I do that in git. :-}


Il faut que décider d'enlever cette ligne ou pas.


Why did you change from @ to self.?


In the common (?) case that `name` is *not* `_obj` or `_container`, and that we're actually wrapped around an `_obj`, that results in one more call to `_OGA` than is necessary---which in turn affects how PyPy can optimize and inline this method. To me personally, it's not really more clear, either. But I'll change it if you prefer.


We can stick the delete under /users/{user_id} unless we'd like to delete the whole resource collection with this method.


Since issues aren't really designed for general "feedback" and clearly you guys are getting notified of this comment, I'll use this platform: Huge thanks @samccone @jakecleary @corysimmons and whoever else helped build this. Major development game-changer for me.


I'll have to check.  There was no such lib in the Xenomai-3 package, although the packaging apparently hasn't been updated well.


вот тут ховеры хотя бы точно можно было оставить ))
даже в демке по ТЗ они есть )


@weierophinney Please excuse me if this is the wrong place, but this line is causing an error when the header value is an integer, eg: 'Content-Length' => 10


With path templating this parameter would be part of the resource path.


Yeah, unfortunately I don't know a better way. Doing it in on_vte_realize() is too late because it happens only after the Terminal tab is clicked and if someone has "follow path of the current file" on, the "cd"'s wouldn't be applied until the tab is selected.


Ah. Thanks for the clarification! (and maintenance on the repositories, of course).


If this controller is not being used, you should remove it. It's never a good idea to leave "orphaned" controllers/directives/etc laying around.


If we'll use the different types locally, I could see us doing the work to translate local types into the desktop-understood visit types, but that'll complicate both inbound and outbound syncing, so not in the short term!


You're right, hyperthreads don't help Moses.

Thought you killed PhraseDictionaryOnDisk?

If you want to change it back, be my guest.


Nit: add a `### VISUAL STUDIO ###` header above this section.


Can be moved under /seasons/{season_id} path. Parameters can be replaced with a season schema in the body of the request method.


Since 'playerWins' and 'computerWins' are just returning numbers, you should specify in your string which score is which. Labeled scores will be easier for the user to understand. For instance...

```js
console.log('The score is currently\nPlayer Wins:' + playerWins + '\nComputer Wins: ' + computerWins + '\n');
```


ok for me


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (348 of 348 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)


Turns out that version doesn't exist on https://github.com/jshint/jshint/tags so I guess 2.5.6 was meant to be installed from looking at the time span of this commit.


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/da39b8403b371aa048f39e291dcf5372a15fafee/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/da39b8403b371aa048f39e291dcf5372a15fafee/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/da39b8403b371aa048f39e291dcf5372a15fafee/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/da39b8403b371aa048f39e291dcf5372a15fafee/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/da39b8403b371aa048f39e291dcf5372a15fafee/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Not sinking `url`, best to take by const ref.


Thanks for the context @classicist! I'll look into it.


A season schema should be used in the body of the request instead of separate query parameters.


missed that part, thanks!


For each 'playerMove,' consider using ternary statements to refactor your code. With all the spacing, it's getting a little unreadable.

For example, you could refactor your first 'else if' statement to be

```js
else if (playerMove === "rock") {
     computerMove === "paper" ? winner = 'computer' : winner = 'player';
};
```
You want to be careful when using ternary operators, because they can complicate your code, but for simple 'either/or' one-liners they are very apt! Here's a link to a guide on using them :)

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Conditional_Operator


Should delete whole collection and or moved under /season/{season_id} path.


We're using `nil` here as a signal that this visit shouldn't be recorded — e.g., this allows us to skip the visit that's generated when you turn reader mode on and off.

In general `Unknown` visits should never occur — they'll be filtered out on syncing, for one thing.


Oops.  Forgot to amend the commit to add this. :frowning:


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (348 of 348 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)


@Strate because it makes much more sense. What if i need to open popup again while it is still not closes? With `close` event it is still possible


Hmm, I haven't gotten all the state swapped back in my head.  I can't even come up with a real usecase for ever having a wrapper where either the object or the container is None.


Nice job! :+1:  Just added a few notes for refactoring and usability.


The tidy formula has been removed from homebrew (5 days ago). A new formula has been added instead: tidy-html5.


If you make the triggerDataPoint transient, you can just use the stream.defaultWriteObject() to serialize all the non-transient fields automatically.

Same for reading them back.


Why not use `\n \t \v`, etc?


I'd rather keep the diff as small as possible, except where absolutely critical. We get mostly the same behaviour by freezing in this manner (aside from the symbols not being frozen, but that rarely is an issue).

I kind of would like to focus on getting upstream used to freezing constants and then introduce `IceNine` later on. I think jumping straight to the gem is likely to get rejected outright, as opposed to slowly introducing the idea.


to use need #include "common.h"


@antonl  if it is not tested at all we rather be careful...  once you have it tested please send the pull request. Once the pull request is created it becomes easy to discuss specific changes that you made.


Ah, now I see that that format was there before you started your edits. I thought you added that link. Just let it be.


LGTM as well


Okay, lets go with the nested freeze for now.


`SSALoadingCellStateNoData` message displaying in case of empty response.
![screen shot 2015-05-19 at 23 21 43](https://cloud.githubusercontent.com/assets/4539292/7713193/df9b5c60-fe7e-11e4-8753-daee006d6912.png)

`SSALoadingCellStateError` displaying in case of error while loading request. (bad internet connection for example)
![screen shot 2015-05-19 at 23 23 42](https://cloud.githubusercontent.com/assets/4539292/7713208/e75ef204-fe7e-11e4-8f9e-cb5f9f336cb7.png)


Hàm chạy chương trình


These scripts *were*...


*They* have...


Because the delete method is idempotent we don't care if a user isn't found at that user_id. We could change this to 500 internal server error.


...*an* install...


...to think 'portably'.

or, ...to keep portability in mind.


...(*probably*)...


...*are* called differently


...way *too* easy


Why is `zfs_daemon_start()`, `zfs_daemon_stop()`, `zfs_daemon_status()`, and `zfs_daemon_reload()` in ` etc/init.d/common.init.in`?  Seems like they should be in the zed init script since they're not "common" to anything else.


I think you also need `--pidfile "$PIDFILE"` in the second call to `start-stop-daemon`.

That's the idiom I see used in `/etc/init.d/skeleton:do_start()` on Debian 7.

The `start-stop-daemon(8)` manpage there states:
>Note: unless --pidfile is specified, start-stop-daemon behaves  similar
>to  killall(1).   start-stop-daemon will scan the process table looking
>for any processes which match the process name,  uid,  and/or  gid  (if
>specified). Any matching process will prevent --start from starting the
>daemon.

But I haven't tested this.


`"$PIDFILE"`, not `"$ZED_PIDFILE"`.

`"$DAEMON_BIN"`, not `"$ZED"`.


Extra spacing and unnecessary linefeed.


You should add `rm -f "$PIDFILE"` here before returning. The `$PIDFILE` won't be cleaned up upon receipt of a `SIGKILL`.


I cant write comments on some of the code because of how github handles commits, but here:

line 28-32: Your way works way, and is a very readable way to do it. However, there is an alternative thats a bit shorter (not always better)
``` javascript
  return move() || getInput()
```
Another quick note is that you can refactor some of the nested if statements to use the ternary operator instead of a standard if/else statement. This isn't necessary, and a lot of people overuse it, which can make it confusing and unreadable. Heres an example:
``` javascript
  winner = computerMove === 'paper' ? 'computer' : 'player';
```
It can be confusing, and don't worry if you don't initially understand it. We will go over it in class.

The rest of the code looks awesome. I can see what you changed, and it seems like you really improved since the initial commit. Let me or Remina know if you want to go over the code more thoroughly.


`killproc -p "$PIDFILE" "$DAEMON_NAME"`


As above, `rm -f "$PIDFILE"` before returning.


`status -p "$PIDFILE" "$DAEMON_BIN"`

`"$PIDFILE"` should probably be passed in as an arg.


We're only saving the manifests on a clean shutdown?


`killproc -p "$PIDFILE" "$DAEMON_NAME" -HUP`

Why are you redirecting this cmd's stdout to `/dev/null` here?


Is the assumption that `/sbin/udevtrigger` and `/sbin/udevsettle` will *always* be installed together?

Or should this be something like:

```
else
        [ -x /sbin/udevtrigger ] && /sbin/udevtrigger
        [ -x /sbin/udevsettle ] && /sbin/udevsettle
fi
```


In other parts of this file, you're double-quoting `$ZPOOL`, and here you're not. These should be `"$ZPOOL"` to be consistent.


Why not use:
```
if [ -z "$available_pools" -a "$USE_DISK_BY_ID" = 'yes' ]
```

If `USE_DISK_BY_ID` is set to `yes` then it must also be a non-zero length string.

Also, using `==` within a `[` (a.k.a. `test`) command is a bashism. The `=` operator should be used to test for string equality.


`"$ZPOOL"`


Can pool names contain whitespace? If so, the above filtering won't work as expected.


Assumes pool names can't contain whitespace.


401 Error should be served on all unauthorized delete requests.


I like that you are using the break statement, but it can be a little dangerous sometimes. So just be careful when your using it.


Bashism. Should be a single `=`. This is done throughout. I'm not going to comment on the others since you can find them with a global search for `==`.


...*preferred* way.


...*let's* be paranoid.


The leading `(` should be dropped.

Should this also accept `0`?


History only knows the visit type we pass in, and the page URL. Without expanding the set of visit types, we need two places that encode this logic:

1. History does stuff like "this is an about: page, I don't want to record it": content-based.
2. BrowserViewController does stuff like "I'm restoring a set of zombied tabs/turning off reading list/etc., so this isn't a history visit at all": context-based.

We can't expand the set of types without doing much more work (and running the risk of making a mistake), so this simply adds a little bit more of an air gap between browser events and the history log.


...*need* to be started *extremely* early,


...*too* late.


OK, this handles the case of removing the pidfile if needed (which I mentioned in earlier comments above). But separating part of this function into a "common" `zfs_daemon_stop` (which is only called from here, yet exists in a separate file) makes this needlessly complex and error-prone.


@Discover-: Nice, not bad. I'm glad to see you figured it out. :+1:


r=jedmccaleb


...somewhat *misleading*.


mounted *verbosely*?


...would make *sense*.


on *available* options.


Why did the indentation get messed up? Suggested:
```
template <class UnaryFunc>
std::enable_if_t<!std::is_void<
    typename Function::return_type>::value,
        typename Function::return_type>
foreach (UnaryFunc f)
```
...


One usecase for an object of None but a non-non container is in something like zope.container:

```python
class AcquiringContainer(BTreeContainer):
    def get(self, key, def=None):
       value = BTreeContainer.get(self, key, def)
       return ImplicitAcquisitionWrapper(value, self)
```

I'm not sure about when a None container would arise naturally. I think there are comments in the C implementation about init time.


Updating my fork with darkstar main


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/831bb426548fcaf7c5032b797ed296cb51620c4b


merging MonsieurNicolas/stellar-core/loadFrames = 831bb426 into auto


MonsieurNicolas/stellar-core/loadFrames = 831bb426 merged ok, testing candidate = 2954283d


good lord.  are you being paid by the message???


In that template, you may check whether the `@post` already has a summary. If it has one, then you may display it, otherwise, you'd have a link to create one:

``` erb
<div class="col-md-4">
<% if @post.summary %>
  <p><%= @post.summary %></p>
<% else %>
  <%= link_to "New Summary", new_topic_post_summary_path(@topic, @post), class: 'btn btn-success' %>
<% end %>
</div>
```


Ah, I really think that we continue to not need author comments, personally... everything should be regarded to be mixed-author with who wrote what being recoverable from git.


@andyblackwell nothing happens when you use CMD+Shift+9 either?


@AriMartti ./manage.py generate_test_pdf


Modified code for developer demo by Surya as per comments from Rodger.


Ctrl-5 is taken in Chrome.  It switches tabs.


Keep linking these properties... like you did with `display`! Also try them backticks, they're great. Is CSS case-sensitive?


ok


Examples would be helpful here.


It'll be slightly better memory-wise, because it doesn't create the intermediate closure in advance. It's slightly simpler from a signature perspective, which makes it much easier to read when using it.


thanks @janmyler ! just published on npm


You should use triple-quoted strings for this kind of stuff. It's much harder to read in this format.


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (348 of 348 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)


Sorry for all the notification spam.


The player_1, player_2, and winners fields will be the player_id of the respective players. We should change the schema type to Integer to prevent users from assuming it will be the player_name in string form.


You don't need to use an 'if' statement here. You can refactor by using the || operator.

```js
return move || getInput();
```

This one line is all you need to put in your function and it's basically saying the same thing as what you have now. If 'move' exists, return 'move' - otherwise, return the result of getInput().


Same thing here. Try to refactor.


@borshop: retry


Add a variable 'port' to manage a different port than the default one.


>  ~~~on local system~~~

hum, this was only needed if it was needed to spy on `/etc/ssh/sshd_config`


Nice :) To sum it up, in that assignment, you used the [singular resource notation](http://guides.rubyonrails.org/routing.html#singular-resources) to generate some routes matching the one-to-one association between a `Post` and its `Summary`, an association you set up using [`has_one`](http://guides.rubyonrails.org/association_basics.html#the-has-one-association) / [`belongs_to`](http://guides.rubyonrails.org/association_basics.html#the-belongs-to-association) (see also the [full documentation about `has_one`](http://guides.rubyonrails.org/association_basics.html#has-one-association-reference)).


Consider moving this declaration to line 760 and making it const.


Figured the repo needed an ES6 uplift! :)


Schema field requirements should be set with the post methods in mind. Resource id's are handled by the backend so api users won't be able to supply that at creation.


When you resolve conflicts, be sure to edit the commit message in Atom (or whatever editor), i.e. `git commit` vs `git commit -m`. Good old `git` has queued up a bunch of information for you to start from, but `-m` clobbers that.


Consider moving this declaration to line 760.


Use mixedCase (`authenticateAuthorization`) to indicate that it's a function. Also I liked the old name (`performAuthorization`) better.


This sort of downcast is pretty sketchy. If you only want to accept instances of `Package`, type your method parameter as `Package`.


I dunno, this will be the common pattern for batching in synchronizers — the simple version is used in the storage client for batch uploading, and this two-argument form is the "thread-the-timestamp" approach used in the upload-record-rinse-repeat-return steps for the item synchronizer.

(I expect to see this same function used for passwords, forms, etc., and probably also bookmarks.)

Phrasing this common operation in more obvious vocabulary ("walk these chunks chaining results") and sticking it in Utils seems worthwhile versus stacking `map` and `reduce` inside each synchronizer, no?


still comes up on 8080 for me. the tail of the log shows:
```
Running "connect:dist:keepalive" (connect) task
Waiting forever...
Started connect web server on http://localhost:8080
```
Do I need to do more than
```
  grunt build
  npm start
```
to pick up the change?

here is the top of my grunt/connect.js

```
'use strict';

var developmentPort = process.env.PORT || 9000;
var productionPort = 8080;

module.exports = {
  options: {
    port: developmentPort,
    hostname: 'swagger.na.sas.com',
    livereload: 35729
  },
  livereload: {
    options: {
      open: 'http://swagger.na.sas.com:' + developmentPort,
      port: developmentPort,
      base: [
        '.tmp',
        'app'
      ]
    }
```

note also that it is using localhost not my swagger.na.sas.com setting.


This should be an integer referencing a season_id. Not a season schema.


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (348 of 348 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)


We should add an instruction here to advise to backup the database before processing to the migration , something like : "Before processing to the migration, it is recommended to backup the data. This can be done with the following command : ... "


even if this is impossible i'd opt for an `invalid_arg` here.


Just checked and this UUID for build Xcode 6.4 6E14 (`8DC44374-2B35-4C57-A6FE-2AD66A36AAD9`) is not in the array.


The skip first stripe flag should do that, but you should verify by hand that is happening


You make GITHUB ANGRY!

It's just a syntax error, actually. You commented those tags out and then didn't uncomment them correctly.


Hi @david63,

"Your server IP address appears to be" => appears to be what/how ?


No @supermarin


Don't worry, I already nuked that.


visible on*


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (348 of 348 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)


Should probably be `git pull upstream master`, unless you just want to to do a `git fetch upstream`


This should totally break in the browser, but Chrome is very forgiving, so it might not, actually. GITHUB KNOWS.


I think that the `collect` method is a one-step version of this `filter-match-map`.


Looks like this should be tree.add_hash(datum). Its going to hash this data again.


Would `flatten` work here?


I'd rename the scenario to "Consistency levels are specified via execution options"


May be we can add AssertArgCount function to throw this exception every time the number of arguments is wrong.


Looks like this should be tree.add_content(datum).


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (348 of 348 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)


@Dramacydal : Yes it's a mistake, right value is  0x00004000, but group update flags, like I said in the commit msg, are not used anymore. I've updated them by precaution, but for the moment, there is no packet that send specific info for a member, just the packet SMSG_PARTY_MEMBER_STATS which send all the member informations without filter.


`JAVA_HOME=$JAVA7_HOME ./gradlew clean jar` seems to work fine (doesn't show the bootstrap classpath warning), so I'm OK with this solution rather than clutter the build config with this hack. I could always alias `gradlew` to include the above definition of `JAVA_HOME` if I wanted. Feel free to close the PR.


Since this function returns errors on invalid param data, we might want to check if a hash has a valid hex value and indexes are integral.


lambda_type_resolution (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


Ok, cool for my if statement I had errors with I was able to get it to work
with multiple parentheses.

On Tue, May 19, 2015 at 2:15 PM, Remina Greenfield <notifications@github.com
> wrote:

> Good job. Let me know what error you were getting for your 'if' statement
> and we can try to figure out what went wrong.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/turnerniles/wdi-fundamentals-rps/commit/5ff723a3542ed8900be9c47e6842ee41b58af8ab#commitcomment-11267845>
> .
>


Thank you


you can use `partitionToPageBuffer.computeIfAbsent(key, ...)`


Ok cool, good idea. I will implement this.

On Tue, May 19, 2015 at 2:10 PM, Remina Greenfield <notifications@github.com
> wrote:

> Since 'playerWins' and 'computerWins' are just returning numbers, you
> should specify in your string which score is which. Labeled scores will be
> easier for the user to understand. For instance...
>
> console.log('The score is currently\nPlayer Wins:' + playerWins + '\nComputer Wins: ' + computerWins + '\n');
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/turnerniles/wdi-fundamentals-rps/commit/5ff723a3542ed8900be9c47e6842ee41b58af8ab#commitcomment-11267729>
> .
>


Does this leave `root` out of the listing? Are you sure that you want that?


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Tests Failed

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


You could simplify your code a bit by getting the stream directly from the resources, check out how I did it here:
da96df88115f33aa3ceb09a7e06b2fc220f876b5


It will add one more thing we can forget to edit when changing the interface of a function though.


No error on the code in production just an error on some code I commented
out. But I figured it out now!

On Tue, May 19, 2015 at 2:04 PM, Remina Greenfield <notifications@github.com
> wrote:

> Hmm, I tested this in the console and it seems to work. Were you getting
> an error?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/turnerniles/wdi-fundamentals-rps/commit/5ff723a3542ed8900be9c47e6842ee41b58af8ab#commitcomment-11267626>
> .
>


Just downloaded the PAPI source code and indeed they deal with the overflow: [linux-rapl.c:768](http://gist.github.com/luisgabriel/f91656a6662e21a2ff9c#file-linux-rapl-c-L768). I'm also not sure it works, but I think we can try.


Before launching the migration process I think we should make some checks to be sure the database is ready to be migrated : previous chat collections must exist, new chat collections (room_u, room_s, room_t, room_e) must not exist.


This looks futuristic!


The best Zebra Mussel model I've ever seen!


This is not guaranteed to work by the c++ standard.


really? should be GetPlayer()->GetGUID(); I think


If we have this in `app.js`, do we still need it here? We don't seem to have it in beforeTests in OAE either


saw approval from jonmeredith, jonmeredith
at https://github.com/basho/riak_kv/commit/e973af0f3ecd12c94574d79bb967c557e02f40e5


merging basho/riak_kv/rdb/bug/gh-kv-1103 = e973af0f into borshop-integration-1115-rdb/bug/gh-kv-1103


basho/riak_kv/rdb/bug/gh-kv-1103 = e973af0f merged ok, testing candidate = 2ca606e1


Also, how about `Seq` instead of `List`? A bunch of the compiler is built on `List`, which is a linked list, but that is probably not the best choice, so we should probably not perpetuate it where we can avoid it.


In the block builders we found that this kind of pull design was pretty expensive, so we switched to a mode where the block builders updated the shared state directly.  The code is a bit more complex but not too bad.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/63235099


In your Instagram app settings - is `Disable implicit OAuth` checked or unchecked? Must be unchecked.
![screen shot 2015-05-19 at 4 47 38 pm](https://cloud.githubusercontent.com/assets/9028552/7713608/e697392c-fe46-11e4-8544-01d958df71fd.png)


1836664-ReturnWaterTempReset (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1664 of 1664 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)


Consider moving this to line 49 before the ledger lookup.


good load of work done! Good job all.


Not? :/
Exists a workaround?


shouldn't this verify that it got back a not-done future?


Strange but i'm sure about the value (source: sniffs).


How about `pack.elements.groupBy { case e: ClassDoc => "classes"; case ... }`?


you might want to add fallback options. It may hang up cause of missing requirements.


You might want to put this statement at the beginning or end of your other ones since a 'tie' is something of a special circumstance. You want to organize your code by separating your different cases in a logical way. For instance, first listing all of the cases in which the player wins, secondly all the cases in which the computer wins, and lastly the case in which there is a tie.


Thanks! Was already implemented in #1173 ;)


```json
{
  "version": "0.6.20",
  "approved": true,
  "approver": "mschaaf",
  "approvalDate": 1432068783382
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


PhraseDictionaryBinary is the one that got killed, right?
PhraseDictionaryOnDisk is still the only option for moses_chart.


Once again, I'd think that `collect` is a better fit than this sort of `filter-map` combo


Great!


let me check


@akesterson Sorry that took me so long -- accepted the transfer, just waiting on that PR. (And thank you for the kind words, but I consider the whole SES project more or less inactive for now -- we'll see if it ever makes a comeback.)


Add logs to follow the migration process. Should be added on the connection event (success or failure), on the collections checks, on the beginning and end of each migrated collection, and at the end to notify if the whole process migration was succesful or not.


I'm not a huge fan of this solution, because now we have to check shouldStop everywhere that we might want to cancel the worker... which is exactly what the context is supposed to buy us.

How about if we create a cancelable child context at the beginning of copy (since that's the main place that we actually care about canceling quickly), use that everywhere in the copy method, and cancel that in processError?


This build is failing currently in AppVeyor:
https://ci.appveyor.com/project/ionelmc/cookiecutter-pylibrary
Any idea why?


verify that we block at 21 pages


fast-forwarding master to auto = 2954283d


To je Kvesa mjenjao :smile:


& 0xFFFFFF80


We want to use `ValueError as e` here


well, it looks like its send when there is some error, in many cases its right, but not at all I think, or did you check if its empty even when there is error?


Search works with datasource, but created inside delegate. So I thought, that in this case would be clearer to do this in controller. But if search delegate would be more massive, I'd split it from controller.


@michaelaye  Appveyor is quite flaky. It failed here: https://ci.appveyor.com/project/ionelmc/cookiecutter-pylibrary/build/job/qxk256pdcfhw3pe3 - seems like some internal Appveyor issue.

I'll restart the build.


Valmi miatt még egy rakás angol van, a ts és a desktop magyarítása még nem elég.


```json
{
  "version": "0.6.20",
  "approved": true,
  "approver": "mschaaf",
  "approvalDate": 1432069113071
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


I placed them in common.init.in because we've tried to keep all the compatibility code abstracted away from the init script itself.  And while there is only one daemon today it's not inconceivable that there could be others in the future.


winner, shakeitoff, and example all contain the example comments. I will make sure to remove from shakeitoff, as example seems to make the most sense to contain this information.


Get out the sparkling wine :cocktail:


Cool.  You use a library for comparing versions.


1640-AddHVACPart1 (kbenne) - 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)


Result gets thrown away. Consider this instead:

    if (! jvAccepted.empty ())
    {
        for (Json::Value::const_iterator it (jvAccepted.begin ()); it != jvAccepted.end (); ++it)
            result[it.memberName ()] = it.key ();

        return result;
    }


Let's go for it, then. I hope we have enough time.


Whoops, should be 88cfddfdf109b7504648a80067ba51fc333fd1df (not aac7ead43319674a6a06e0ab79e7c3ed6b9c3a7d).


Haskell and C do make a nice pair. :smile:


This is great. A first-class npm module + CSP fixes. Thanks.


`store_name` is already a string, you don't need to interpolate it into a new one when passing it to `#create`, you can pass `store_name` directly.


I am keen to document how a new translation is added. So maybe @justinclift this would be a good opportunity. Happy to help with this, of course.


Good catch, I'm a bit surprised that was missed.


Yes, for every case it was empty.


1836664-ReturnWaterTempReset (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1664 of 1664 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)


Removing example comments, updating file description.


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_kv/builds/2036


Sweet :)


To make it easier to understand, it's beneficial to name the `key` and `element` (or `value`) variables more descriptively. In this case, the key could be `attribute` or `field` and the value could be `error` or `message`.


Hah! You caught this right when I did! :+1:


then its a bit illogical why there is this field, dont you think? :D its again something special from blizzard :-/


error on compilation after those http://hercules.ws/board/topic/9645-after-updating-hercules-to-latest-rev-got-this-error/


unused


aqui voy a dejar este comentario


No problem. :hamburger:


@jtrmal: `std::isnan` is probably not there in VS2010, and certainly not in VS2008. `stdint.h` is available starting with VS2012. Do you think we need to support anything older than VS2012? AFAIK, the free compiler comes with Visual Studio Express and/or the SDK, so I would not worry about older compilers.

The original `more/type` trick does not even work. I think, however, we can easily support training under Windows, and should (for the record, @danpovey.thinks otherwise, AFAIU). In any case, piping is either not required on Windows (library decoding only), or should support all pipe tricks (if we support training). as for test only, let me figure something out. Good point not to require cygwin even for testing.

A bigger problem is the matrix backend library on Windows. ATLAS is not available for Windows, AFAIK, and MKL is not free.


```json
{
  "version": "0.6.20",
  "approved": true,
  "approver": "mschaaf",
  "approvalDate": 1432069387640
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


Check indentation.


I don't think you need to special case empty input: (code not tested)
```
bool TrimString(const std::string& input, const std::string& trim_chars, std::string& output) {
    const size_t first_good_char = std::min(input.find_first_not_of(trim_chars), input.length());
    const size_t last_good_char = input.find_last_not_of(trim_chars);
    bool input_was_empty = input.empty();  // in case output == &input

    output = input.substr(first_good_char, last_good_char - first_good_char + 1);
    return !input_was_empty && (first_good_char != std::string::npos);
}
```


Ping? Still relevant for 3.1


Since you are including all files, I think this could also just be `<fileset dir="repo.dir" />`


ClangModernize (axelstudios) - x86_64-MacOS-10.9-clang: Tests Failed

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


Because in this case TableViewCell can be used in different table views. It is not possible to reuse cell, that was created within storyboard.


@glawrence It would help me a lot. I don't have a lot of free time to do it, so it could take a while (I'm not sure about how much text I'll have to translate to calculate how many days this would take). Also, maybe I won't be able to translate 100% of the strings (some terms should even stay in English, IMHO), but I don't think this is required.


Is there any known way to make `yytext` contain the format string?

I see it's possible to inject format string via file name and (after dc6f1e0) via unclosed qstring or hstring, but I've not got to get it to `yytext`.

Quick grep over sources found few other `agerr()` calls with second argument not being a fixed string.

https://github.com/ellson/graphviz/blob/31158b1/lib/cgraph/scan.l#L168
Same file, `chkNum()` function.  Format string can be injected via graph file name:

~~~
$ cat fs4-%n%s%s%s%s%s%s.dot
graph G { a [ weight = 0g ] }

$ dot fs4-%n%s%s%s%s%s%s.dot
Warning: *** %n in writable segment detected ***
Aborted
~~~

https://github.com/ellson/graphviz/blob/31158b1/lib/graph/lexer.c#L463
https://github.com/ellson/graphviz/blob/31158b1/lib/graph/lexer.c#L469
https://github.com/ellson/graphviz/blob/31158b1/lib/graph/lexer.c#L472
libgraph is deprecated since 2.30.0.  In older graphviz versions, format string can be injected via graph file content.

https://github.com/ellson/graphviz/blob/31158b1/cmd/tools/gmlscan.l#L130
This case would require format string in `yytext`, so it's unclear to me if it's possible or not.


This probably wasn't mentioned in the assignment (and I'll address that), but it's dangerous to commit credentials to git, especially in a public repo. I'll look into clearing that up in the assignment, but do keep in mind for the future that a file storing credentials should usually be added to your `.gitignore` file and kept only on your local machine.


It was placed here to avoid replicating this code several times in the common script.  But I'm not at all opposed to moving it if the general feeling is that is clearer, I went back and forth on this several times.


Properties we define at runtime could be prefixed in some way. Just to prevent a conflict with anything that we define in `build.properties`. So for instance, `repo.dir` could be `_repo.dir` (a runtime property that is used in scripting, not defined in `build.properties`).


I wondered about that.  In practice it doesn't seem to be required, but it definitely seems like a good idea.


Could avoid the `name` via:

```python
_SPECIAL_NAMES.update([intern('_Persistent' + x) for x in Persistent.__slots__])
```


Yes, maybe another guid in some cases... If I remember, it was not the only guid which was empty when player guid would be expected. Like for phases, tell me why PersonalGUID Is empty xD ?


Property name `repo.remotePath` uses camelCase. That's fine with me if you prefer this, but the other property names that we have use `underscore_names`. Just an observation :-)


```json
{
  "version": "0.6.20",
  "approved": true,
  "approver": "mschaaf",
  "approvalDate": 1432069643087
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


Good refactoring


```json
{
  "version": "0.6.20",
  "approved": true,
  "approver": "mschaaf",
  "approvalDate": 1432069750527
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


`npm start` is now serving the production version. Please use `npm run develop` if you want to run in development mode.


```json
{
  "version": "0.6.20",
  "approved": true,
  "approver": "mschaaf",
  "approvalDate": 1432069778400
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


This appears to be optional, but I agree it's a good idea.


a link to google maps would make it so much easier


@sadeqzadeh Thanks, would that make the new command `brew install homebrew/dupes/tidy-html5`?


@Tele42 added the fix as advised https://github.com/dgibbs64/linuxgsm/commit/bb3efb947ad9ce1f75ebf5fdc01e3ac97d7cd1d6


idk, maybe blizzard is going to use it in the future :D


updated


I love the idea here, where there is a separate target. Cool. I wonder if this should depend on the `build` target also?


Leave no trace!


fyi @emord removed this after merging master into copy-paste-dev


It's the way it is so the error could include a comprehensive list of all the partitions involved in the collision


The example I happened to be looking at did this, although it appears most init scripts done so yes we should drop the `/dev/null` bit.  And updating it as you suggested does seem to be more standard.  I see a few exceptions in /etc/init.d/ but they're clearly oddballs.


I do not know how to distribute a patched openfst on windows. There is a port available from their site and, for later versions, on Github. There is a lot of changes compared to the original version. Perhaps we can add it to the latest version, or maintain a fork under kaldi-asr. Patching before build is not the best option to me.

I tested your workaround, it works.  Very smart. I do not understand why the error happens, and why is the problem patched by this. `fst::int64` somehow preferred to an alias `int64 = kaldi::int64`, but in the absence of `fst::int64`, the latter is ambiguous. Can't explain.

Also, do you think it will do if we simply `#include` it into kaldi tools, possibly gated by `#ifdef _MSC_VER` or like?

> mingw32 (which you definitely have when using git),

I am using cygwin's git (it is version 2.x) with Tortoise, but I am adventurous. :)


I think having no visits isn't necessarily wrong; less *useful* (you're only communicating the title change), but valid.


I was not aware of the `trim=""` attribute. Nice! Do you remember where you found this documented?


fyi @emord, this is a cherry-pick of 3c3befc44a69c83e2567bc4ca6039b6962b51b47, now on copy-paste-dev


/stylesheet-with-issues.css:44:5: CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)
/stylesheet-with-issues.css:45:5: CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)
/stylesheet-with-issues.css:46:5: CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)
/stylesheet-with-issues.css:53:5: CSS outline not supported by: Opera Mini (5.0-8.0)