method_missing (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (341 of 341 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)


Unnecessary indentation. There is no open element here, so the link should be at the same level as `h1` and `p`.


Very good. I've added one more comment to the questions assignment.


https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/blob/develop/WordPress/Sniffs/WhiteSpace/ControlStructureSpacingSniff.php#L221 looks relevant.


The "Ptr" extension that I was putting for variables in livre code was not very nice. You can remove that.


 Должна быть нормальная, в проекте FOC она настраивается в SystemInit только, но эта функция не вызывается в программе, которая использует lib-у. (В ней я использовал файл инициализации, который ты скидывал)

Среда, 22 апреля 2015, 3:07 -07:00 от Aliaksei Stratsilatau <notifications@github.com>:
>у тебя там точно частота нормальная? Xtal=168MHz, т.е. частота кварца, а PLL он может сконфигурить незаметно так, что проц будет работать на частоте 168/26MHz незаметно для тебя.
>—
>Reply to this email directly or  view it on GitHub .

С уважением,
Александр Федорин
alex_f_87@mail.ru


ちなみに配列の中身の個数を他でも使いまわす可能性がある場合は、一旦`animals.length`を変数に格納してもOKです。
```js
var length = animals.length;
```

↑こんな感じでコメントの中でもmark downが使えます。


OK - i think i got it...


@nfeske: please add to staging


deregister word I used is really nonsense :) If not affecting so many files you can change it to unregister...


avoid unnecessary reallocation since the number of temp points is known


The instruction said:

> Create another new model called Answer. It should reference Question and have a body attribute.

You're missing the "reference Question" part. Even with `belongs_to :question` in the model this is not going to work. Do you know what needs to be added here?


Yes, I'll do it


大声告诉我你这是不是参加什么拿 MBP 的比赛的作品


Hi @alimuldal and @avalentino , @cgohlke is reporting problems with this test on Windows and Python 2.x, 32 and 64 bit, but not on Python 3.x.  I double check that this patch is in PyTables 2.3.0rc1.  Any hints?


To mamy znów przewijanie w poziomie :)


@Paxxi, nice!
There's already a PR that touches this code. See https://github.com/xbmc/xbmc/pull/6974

Don't know if it's going in for beta but if it does you'll have to rebase. If it doesn't, can you cherry pick relevant commits?


Hi @Vinzza,
Does your test was successful?


Where is the Network parameters cpp and header files ?


They indeed are :)


+1! It should only be thrown when the range is not given. We cannot check that the way it's defined now, so it should only be thrown when `range == 0.01`.


I have no problem with adding this for the v2 branch and potentially deprecating later. Released themes with v2 may never encounter v3, and may want to pass Theme Check or just be in a different menu location.


It clicks the first button. It's possible to click an `<a>` with a certain text, but I haven't found for a `<button>`.


Why do we need hostName and port out of the schema ? You can get them from the URL ?


If we cannot find an easy solution for this, I would propose to skip the test on Windows and Python 2 (if possible) and open a ticket on this.


this fails with ```dynare example1.dyn``` and ```dynare example1.mod```


Doesn't this write len*(len+1) bytes of data?


Thanks!

Development is now taking place in a separate branch, which will be merged asap. It's in this same repo.

Would you mind adding your changes to that branch instead? Otherwise, I'll have to merge everything twice, by hand.


Oups sorry for that.
But I think FAT binary should be enable when building JamomaMax and
JamomaPuredata.
Btw as we discuss on BoD, the repo will be reorganize to make Core a
submodule as JamomaMax and JamomaPureData.
Thus the default Fat binary shouldn't be a problem anymore.
What do you think?

Le mer. 22 avr. 2015 00:04, Jean-Michaël Celerier <notifications@github.com>
a écrit :

> ehhh, I'm pretty sure this would break some stuff down the road
> (especially for iOS builds).
>
> The recommended way is to set the variable CMAKE_OSC_ARCHITECTURES, either
> on the command line : "-DCMAKE_OSX_ARCHITECTURES=x86_64;i386" or with a
> call to set(....).
>
> In order to make it easier to specify maybe we should add an option() ? (
> http://stackoverflow.com/questions/5998186/cmake-adding-command-line-options
> )
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jamoma/Jamoma/commit/92c82dd9814dbb6ceaf3cd9723f7f2dcbeee6455#commitcomment-10835406>
> .
>


As requested - I hope that's all okay? Thanks! (I changed 'and' to 'but', I thought it read (even... ;) ) better.)


Done Remarkable job


This function is identical across the Unix implementations.  You could put it in a PathInternal.Unix.cs, which would be in a TargetsUnix section in the csproj and thus would be included by both Linux and OSX, rather than duplicating it.


What happened here?


Nit: needs newline


No. The hostname and port are for REST commands (http). The schema is only for internal zeq events.


Also here?


Simplicity is simple.

If this wasn't for a demo, I'd be all in for `RecordWildCards`. But I do think this change makes it easier to follow for the uninitiated reader.


delete


Hi Drew,

Just had a look at caption line number 1359 on my copy of tom stuart in Drew, captions, #1 - it reads The identity of the array is not a useful thing\Nto protect. It's the contents of it that we care about.

However, the word actually has been added on the copy on github - why hasn't this change appeared on my copy ?

Should I write actually in and then split it into two ? - bit confused. And why do you mention caption 1361?.

--------------------------------------------
On Tue, 21/4/15, Drew Neil <notifications@github.com> wrote:

 Subject: [captions] Caption 1361: reformat (5036ec3)
 To: "nelstrom/captions" <captions@noreply.github.com>
 Cc: "Bonita Jane" <broworth@yahoo.co.uk>
 Date: Tuesday, 21 April, 2015, 23:29

 Caption 1361: reformat

 FIXME: caption 1359 is now 4 lines long.

 @BonitaJane could you split it into two separate captions,
 each 2 lines
 long?

 ---
 Reply to this email directly or view it on GitHub:
 https://github.com/nelstrom/captions/commit/5036ec3bd2c6debd2bcf45f94741dabaddb7fbcf


Maybe JudgmentIndexItem ?


Are you sure ?? For example _publisher( zeq for RTNeuron camera sync) is not a pointer but _rcPublisherPtr ( for RESTConnector) is a shared_ptr. If I remove Ptr how can we differentiate _publisher( non ptr ) with _rcPublisher( ptr ) ?


If someone wants to submit a PR to fix this feel free - personally I don't think it's worth spending time on as all these warnings will be gone in a few weeks anyway.


Yes, but I don't know how to automatically shut it down after the tests :disappointed:


Effectivement on peut sans problème fixer cela à l'aide d'un "screwdriver".


#float?


No creo que funcione el delete debido a #51 A mi no me funciona. Crear usuario lo veo bien


"At least one FROB is required on the command-line if GRONK has not been configured"

That error message is fairly user-hostile.  Our use of CIDR is not standard, and googling the term does little to help.  I'm not sure we expect our users to know or care what IPAM is.  And it would be good to not just tell the user that they have done something wrong, but give more of a pointer about how to rectify it.

Command line should not be hyphenated unless it is being used as an adjective (i.e. "the command line", but "the command-line interface").

And the error message should be wrapped to fit in 80 columns.


No creo que funcione el delete debido a #51 A mi no me funciona.


Sorry for such a late email. After talking with Mitchell, we don't think
that acts as taggable on is what we are looking for. We tried to look into
the associated model Tag and we were confused on what tagger was for. We
drew this diagram out to figure out polymorphic associations and how that
ties into tags.
[image: Inline image 1]

Kevin Li
University of Washington
Biochemistry
Data Processor, UW Dream Project
VP of LEAD

On Wed, Mar 4, 2015 at 12:10 PM, Matt Harris <notifications@github.com>
wrote:

> @efawe <https://github.com/efawe> yes, those ideas are the types of
> things I'm thinking about. There are others, too, but abstracting tags out
> so they can be used on a variety of models would be good. We don't have to
> *implement* those other use-cases now, but we should always write our
> code thinking about those future ideas.
>
> What does this mean for your code? First, it means that you should NOT put
> tags in a text field on the Mentor model. Because then when you want to
> implement this for Participant, you would have to duplicate your code.
> You never want to duplicate your code—Rails always uses the mantra "Keep
> your code D.R.Y. (Don't Repeat Yourself)".
>
> You might want to read this section of the Rails Guides about Polymorphic
> Associations:
>
> http://guides.rubyonrails.org/association_basics.html#polymorphic-associations
> This is a way to share association functionality across many different
> models.
>
> Also, what do you think about that acts-as-taggable-on gem I referenced
> above?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uwdreamproject/DreamSIS/commit/6d33b61df6761ca1b497fcaacaa7ae972600ef09#commitcomment-10037744>
> .
>


Lo veo bien


Hi,

This is not correct. You must _realloc_ `series->column_types` always if series->columns_length != 0 to avoid an invalid write i line 70.

This is the fix:
```c
    if (series->columns_length == 0) {
        series->columns = malloc(sizeof (char *) * INFLUXDB_SERIES_STEP);
        series->column_types = malloc(sizeof (unsigned int));
    } else{
        if (series->columns_length % INFLUXDB_SERIES_STEP == 0) {
        	series->columns = realloc(series->columns, sizeof (char *) * (series->columns_length + INFLUXDB_SERIES_STEP));
        }
        series->column_types = realloc(series->column_types, sizeof (unsigned int) * (series->columns_length + 1));
    }
```


Il y a la variable tour et peut-être la variable coups mais je ne suis pas sûre. Il faut demander à floriant car c'est lui qui a fait le diagramme des classes.
C'est bizarre car dans la classe Tour, il y a une variable coups, donc je ne sais pas à quoi elle sert celle dans la classe solo.


I'm not sure how to do that exactly, but it doesn't really worry me. I think this way is more performant anyway. In 132081608a423804fd0bedb56ac54bce20d9f801 I addressed this issue by making it a *little* bit more flexible, where it doesn't matter if it's a `<dl>` or a `<div>` or whatever.


Copy & Paste ;)


а чего не просто list.Contains(value)? :)


а зачем она mutable? :)


fixed


+1 Good to improve the safety of uninterrupted recording.


Whoah! I didn't had time to highlight that!


A banner chunk is a `rct_scenery_entry` offset 8 is the banner price.


Nit: {}.


So, the intent of that code was to ignore only that exception because it happens for no reason on the MacOS. Now it will ignore all exceptions and we probably don't want that.

Reference:
https://leap.se/code/issues/2187
https://code.google.com/p/protobuf-socket-rpc/issues/detail?id=10


awdawdawd


awdawdwad


LGTM.


FLink Roger


When running the added test in visual studio runner, it can not find the code of AssertionFailedException within
> #if NET40 || NET45
> ...
> #endif

blocks. Thus the unittest is currently crashing as the SerializableAttribute can not be found. Can I set these compileflags for manual test execution?


For now yes, will add nginx support soon.


Need a little fix for the last loop:
The range `[0,hislen)` is partitioned as follows
```c
[0,r1_start)           // newest entries to delete
[r1_start,r1_start+r1) // newest entries to copy
[r1_start+r1,r2_start) // oldest entries to delete
[r2_start,r2_start+r2) // oldest entries to copy
```
So the following loop should start at `r1_start+r1` and not `r1`, otherwise we try to free some entries twice for `newlen==0`.
```c
for (int i = r1; i < r2_start; ++i) {
          xfree(history[type][i].hisstr);
          history[type][i].hisstr = NULL;
        }
```


deregister are from co:: and from equalizer...


Merge


Call it `masterName` as this is the name of a master, nothing else.

Read it like so:

function `to export an Instance of a master` you need (`io`, `a project`, `the name of a **Master**`, `a target directory`, `float precision`)


These changes have not appeared on my copy of tom stuart captions.

Re caption line 1386 - it is 42 characters long which is the max according to your instructions.
Line 1392 is 34 characters long - so why does this need to be split into two?


The problem was that `while` was catching all breaks without checking the ID.

```ruby
def foo
  while true
    yield
  end
  puts 'should not be here'
end

foo do
  puts 'breaking'
  break
end
```

Here the `while` will catch the break if there is no ID checking.

Not sure what you mean about null IDs. I use the ID of the method itself, not the block within the while (as there isn't one as you say).


Valter, vai ter que alterar de "private long idPessoa;" para "private Pessoa pessoa;"


Only version `0.9.1` works properly with Wiremock


Need to delete this pull request


In browser testing the time spent on query selectors in comparison to e.g. AJAX requests is effectively zero. And it's much more important that your tests don't fail whenever you change the DOM, I hope you will experience this annoyance to see what I mean :). Of course, often tests will have to rely on the DOM (best through id and class selectors), but the lesser the better :)

Ask yourself: "What am I testing? Am I testing that the AJAX request will put the response info inside a `<dt>`, or am I testing that it will put the reponse info *somewhere* on the page?"


Why?


ok, assuming this works (didn't test it) feel free to merge it yourself.
I added one last comment to rename Dockers to Docker


@lifecom это не то. Хотелось бы чтобы кто-то полноценно объяснил т.к. описание на сайте апача какое-то слишком обобщенное.


    # ~dwragg/work/weaveworks/go/src/github.com/weaveworks/weave/weave launch
    fa159a06ae231c75d5da55a5f629b7aaeb9b2a9f5ba2ab3ebd8ce77ae146a2ca
    # ~dwragg/work/weaveworks/go/src/github.com/weaveworks/weave/weave reset
    404 page not found


Ambiguous regexp literal. Parenthesize the method arguments if it's surely a regexp literal, or add a whitespace to the right of the `/` if it should be a division.


Assignment Branch Condition size for create_new_abilities is too high. [17.12/15]


Spec path should end with `update_user*_spec.rb`


Yes, havent you seen it, switch you branch and run the solution, then check the database, check the methods both in code and run them and see for any exceptions, i did, happy path was okay


> Information about how many available addresses are owned by the current node is in weave status, albeit cryptically.

Oh yes, I see it now.  Ignore that comment.


I think the ```!(needle->handler->flags & PROTOPT_CREDSPERREQUEST)``` condition is not anyhow specific to NTLM.  It should be checked even when compiled without USE_NTLM, should not it?


After doing these changes, still I am not able to get the swsusp command listed in help. Code compiled without any error. Is there anything need to be done apart from the things mentioned here


Ah yes you're right. Thanks. I'll fix.


`-alloc` seems like an obscure option name.  `-iprange`?


I think this should be:

```
cmd = """
cd {2}/synnefo
git pull --no-edit --no-rebase {0} {1}
""" .format(pull_repo[0], pull_repo[1], work_dir)
```


Nothing happened so, you know, just a reminder


@zorceta 我就试下vue。。


To use this mixin, a function getInitialHistoryState must be set right ?


strange.... http://bazaar.launchpad.net/~acsone-openerp/openobject-server/7.0-xml-test-report/revision/5247


This should also apply in all lines that use `fabric.cd` I believe, e.g., 1071


this.state.historyIndex ?


Why is is possible to add an 'initial' state ?


It was for users who inadvertently hit CTRL-C when in the console and lose control of their workflow (happens quite a lot around here). You can still use CTRL-D when the password has been entered.

Would it be ok if we find a way to enable it after the password prompt?


merge ?


Ír valami hibát?


It prevent also the GUI to be easily stopped and prevent the console to be halted when you use waitUntilEnded.


Yes. I'll send you the results by mail (posten ;-)).


Why not adding logic at the end of
setInitialState ?
setState ?


Please tag stuff like this since it breaks stuff. :smiley:


Don't do this here.  If you don't move these files they will be in the wrong place.  The idea when "NO_WEBDIR_SET" is true is that everything, including the vendor directory, gets moved into the WEBDIR.  The WEBDIR then effectively becomes the root of the application, containing everything, and is where we'll run Composer (i.e. it like the root of your project locally).


Very good


I tried to implement it cleanly with signal catch, threads and semaphore, but the y/N confirmation messes up with the console input stream.


I'm testing 2 things:

1. Did the AJAX call succeed?
2. Were the results injected in the `<dl>` like they were supposed to?

**You** can ask **yourself**: what is the most minimal indication of an AJAX call I can make? :smiley:


Ok then it makes sense to revert the commit and keep the original behaviour.


thx, the change is in master now


Fixed.


Fixed. I'll see what I can do about the `"/"` part.


the variable name should be judgmentIndexingItems.


What happens when the entity doesn't exist in context args?


I really got annoyed by this already and was hoping for this to be fixed :(


var name


Now I understand what you need to do but still these three methods that you added are not necessary and should not be there because it makes API confusing (e.g. there is contextMenu.selectVisibleItem(int) and contextMenu.advanced().selectItem(int)).

Instead, you should use contextMenu.advanced().show() in your test, check the content of context menu and then call contextMenu.advanced().selectItem()


var name here and in other places...


A-ha! What would happen if you decided to change the text in the `<button>`? Better do add a class and use **that** as a reference, maybe?


那网页里的文案什么情况


What if it's only in REPL mode ? Because only then I want to hit Ctrl+C after starting to type a multiline statement but notice a mistake in earlier lines that I cannot go back to. So I always use Ctrl+C wanting to start over and end up quitting the console by mistake.

However, I must admit the scala REPL behaves the same way... annoying, did I mention ;P


I wasn't planning on getting this in until v16. It shouldn't break anything but it's not a fix so it's low prio


Looks nice!


Moving the code into a condition based on if amqp transport is enabled.


@zorceta https://www.zybuluo.com/mdeditor 这里的，试下markdown效果的


I'd rather not mess with React internals. Think of the history state as a layer on top of `this.state`.


Yeah, I reverted most of this commit. This was 2to3 code, but it doesn't make much sense. See the next one for changes.


That is correct, although it should probably be optional.


Regarding, ```ex.waitUntilEnded```, what I generally would expect is that Ctrl+C behaves as in MATLAB. It quits the current execution of whatever command is currently executed by the REPL, not the console itself.


Proposition d'ajout / modification pour ExtazCMS de MrSaooty


We will need a tax model to record each state's tax rate, right?


might want to gitignore npm-debug.log ;)


You don't need to set this here.  Add it to 'defaults/option.json' with a default value of False.  That file is loaded as the basis for the ctx.


nihao


'name': tr('Earthquake ITB fatality function'),


How do we use session in cart model?


@mvdstam there never was a tag for v4.


The `isInitial` flags lets you know if the current state is considered `initial` (can't reset) or not (can reset).


Don't use `ctx.update`, just do ctx['NO_WEBDIR_SET'] = True.


Shallow merge that returns a new object composed of the keys of `currentState` and `newState`.


For now it's not  so it should be documented


is xmlns:g="urn:import:com.google.gwt.user.client.ui" needed?


did you come up with a definition for affected and impacted. I always thought that people were affected and structures impacted??


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


厉害


Good catch :smile:


'on the earthquake model for Indonesia developed by ITB.'),


Yeah, I'd be really grateful :wink:


Cool context menu positioning


> If we cannot find an easy solution for this, I would propose to skip the test on Windows and Python 2 (if possible) and open a ticket on this.

+1

Anyway, do you think that the patch makes sense as is?


That's true, then your test would fail :). But I don't think it's ideal to add CSS classes just for tests. Unfortunately, text is something you will have to rely on, that is you will have to fix tests when text changes. Because text will change much less frequently than DOM. If you imagine your test as a user browsing the site, it's more natural to test "what a user sees" (text), than internal HTML.


you do not really reset with `this._addHistoryEntry(..., true)`
so then the canUndo will be false and so on.


'name': tr('Earthquake PAGER fatality function'),


If I start a weave network of two peers, with an `-alloc` option provided to one `weave launch` but not to the other, then (on the second peer):

    # ~dwragg/work/weaveworks/go/src/github.com/weaveworks/weave/weave status
    ...
    Reconnects:
    ->[192.168.122.1:6783] (write tcp4 192.168.122.1:6783: connection reset by peer) next try at 2015-04-22 12:07:55.243684849 +0000 UTC
    # ~dwragg/work/weaveworks/go/src/github.com/weaveworks/weave/weave status
    ...
    Reconnects:
    ->[192.168.122.1:6783] ([gossip] received unknown channel with hash 891211498) next try at 2015-04-22 12:08:03.853776889 +0000 UTC

I can appreciate that we might not want to support networks that where some peers are using IPAM and others aren't, but the result of attempting to do so should more graceful than this.


'on the Population Vulnerability Pager Model.'),


The canUndo will return `true` after a `reset`, which means you can undo a reset. This is by design.


I think setContextCloseHandler should be named addContextCloseHandler


Please elaborate?


Nice use of bootstrap's icons!


So, if you change `<dl>` to an `<ul>`, you want your tests to fail?


It would be helpful if this showed an actual function showing expected params and what the return data is expected to look like.


เพิม่


hum ok


'To assess the impact of each hazard zone on buildings.'),


I think this should not be in general function , but instead in coa_functions or a new function file
general functions are not project specific.


Is there a way to supply a generic marker that isn't bound to a user but is bound to a project e.g. ``Firefox 28``


'the hazard zone in which it falls.'),


l 117: shouldn't we use :
    else:
        template = facebook_settings.FACEBOOK_REGISTRATION_TEMPLATE
Otherwise the template returned by the function is undefined.


@jan-cerny
Just OOC (out-of-curiosity) why not to introduce the new ```fedora_oval_version = 5.11``` variable here, and keep RHEL's ```oval_version = 5.10``` one intact? Yes, this would also need to modify ```combinechecks.py``` script to check not just for ```oval_version```, but also for ```fedora_oval_version``` on one hand. But on the other would allow the Fedora content to have 5.11 version, while the RHEL one to stay on 5.10 one.


Done. Skal appen egentlig ikke bare blive til en 2.0? @emiltin


G4VModularPhysicsList inherits from G4VUserPhysicsList, so everything shall be the same. The enhance is the new functionalities that are present in G4VModularPhysicsList, specifically the RegisterPhysics() function.


Good catch! I don't like including `mastercore_log.h` in `init.cpp`, so let me test of another way, where the flag is defined in `util.cpp`.


I think more often you want to change the text than the DOM. E.g. change "Add review comment" to "Add comment". I guess it's a matter of taste?


'Provide details about how many buildings fall within '
+                'each hazard zone.'),


I'll note just one more time that this issue is *completely orthogonal* to Atomic, so this conditional is inaccurate.  The mainline (non-Atomic, yum/dnf managed) Fedora 22 cloud Vagrant image *also* will not have a `docker` group after you `yum install docker`.  The same if someone made a non-Atomic CentOS Vagrant box etc.


This assumes a specific order of the ```MouseButton``` bitfield - please either use a map of ```glfw.MouseButton``` -> ```gxui.MouseButton``` instead, or extract the places where we use:

```go
		switch button {
		case glfw.MouseButtonLeft:
			ev.Button = gxui.MouseButtonLeft
		case glfw.MouseButtonMiddle:
			ev.Button = gxui.MouseButtonMiddle
		case glfw.MouseButtonRight:
			ev.Button = gxui.MouseButtonRight
		}
```

 into a helper function similar to [translateKeyboardKey](https://github.com/google/gxui/blob/master/drivers/gl/keyboard_translate.go#L13)


This is duplicate code, and I imagine should be called in more places than just these two. Please move this out to a function:
```go
func getMouseState() gxui.MouseState { ... }
```

Also, this logic is only executed the first time that ```pendingMouseScrollEvent``` is ```nil```, it should probably be moved to just above ```v.Unlock()```


Give that the ```SetMouseButtonCallback``` handler only sets one bit (button), this will overwrite any existing mouse-down state. Please see my general comments below.


All other enums in gxui are ```int```, please keep it as such.


Please keep ```MouseButton``` as-is (an enumerator). We can add ```MouseState``` as a bitfield of pressed state. See my general comments below.


Why did you remove the map here? This will break double click behavior, as pressing down on one button then pressing down on another will trigger an incorrect double-click.


I believe these helpers should be on the ```MouseState``` type. See below.


So, looking over the places you've had to change, it strikes me that we may want the existing ```MouseButton``` type  and ```MouseEvent.Button``` field ***as well*** as a new ```MouseState``` bitfield and new ```MouseEvent.State``` field. Some events are primarily concerned with the button that caused the event to fire - like ```OnClick```, but for other event handlers like ```OnMouseMove```, the current mouse-down state may be useful.

To be clear, I suggest adding a new ```MouseState``` type:

```go
type MouseState int

func (s MouseState) IsDown(b MouseButton) bool { ... }
```

And adding this as a new field to ```MouseEvent```:

```go
type MouseEvent struct {
 	Button           MouseButton
	State            MouseState
	...
}
```

This would also allow you to do interesting things on ```OnClick``` like check the button that was pressed and also inspect the state of the other buttons.

Thank you for taking the time to draft this PR.
Many thanks,
Ben


Ah yes, you're right.

I see a bunch of other `to_sym`s in this same file, might be worth checking to make sure none of them are user-generated either.


Company should be Piwik


 'Provide details about the number of people that are '
               'within each hazard zone.'),


Will do. Thanks!


This should be listed in ``static/js/README``


> I can appreciate that we might not want to support networks that where some peers are using IPAM and others aren't

... although given that we seem to allow clusters where peers were launched with different `-alloc` ranges, I'm not sure why we shouldn't cope with one that has no `-alloc` option.


PHP -> C#


@Notsosof1 done with my review. You are 'getting it'. No surprise there! Checkpoint complete. Press on.


'To assess the the number of people that may be impacted by each hazard zone.'),


OK, hoje arrumarei hoje, obrigada.

2015-04-20 21:23 GMT-03:00 Andre Nunes <notifications@github.com>:

> OK, mas lembre-se de utilizar alguma conversão de campos do tipo data.
> CONVERT ou CAST, para o exercício 3 (não numerado no arquivo).
> Sempre que utilizar TOP deve ser utilizado um ORDER BY.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gabsouza/crescer-2015-1/commit/8a0bfb72224d31e3b590bd973e8388a57f89580a#commitcomment-10814891>
> .
>

--

Att, Gabriela de Souza.


method http_client::connect may throw exception in case if there is no internet connection. and reconnect does not handle this exception.
IMHO method connect has not correct interface.
   it may throw exception and return false in case if cannot connect.


Hi,
You are right, in order to test this issue ACT-2164  we should have the namespaces here.
So I reverted the process definition file process.xml and adjusted the test accordingly.

Currently the test looks like that:
```
Message message = model.getMessage("writeReport");
    assertNotNull(message);
    assertEquals("Examples:writeReportItem", message.getItemRef());
    assertEquals("newWriteReport", message.getName());
    assertEquals("writeReport", message.getId());
```

Thanks,
Vasile


Again, a class.

I guess you're right and I'll see it eventually.


Can probably simplify this by just using `Task.FromResult<TResult>()`


'Provide details about the number of buildings that are within '
                 ' each hazard class.'),


Menuda racha llevas. Sigue así ^^


```diff
diff --git a/src/init.cpp b/src/init.cpp
index 57a4f77..a3a722c 100644
--- a/src/init.cpp
+++ b/src/init.cpp
@@ -29,8 +29,6 @@
 #include "walletdb.h"
 #endif

-#include "mastercore_log.h"
-
 #include <stdint.h>
 #include <stdio.h>

@@ -219,7 +217,7 @@ void HandleSIGTERM(int)
 void HandleSIGHUP(int)
 {
     fReopenDebugLog = true;
-    fReopenOmniLog = true;
+    fReopenOmniCoreLog = true;
 }

 bool static InitError(const std::string &str)
diff --git a/src/mastercore_log.cpp b/src/mastercore_log.cpp
index 483b84a..37968aa 100644
--- a/src/mastercore_log.cpp
+++ b/src/mastercore_log.cpp
@@ -64,8 +64,8 @@ static boost::once_flag debugLogInitFlag = BOOST_ONCE_INIT;
  */
 static FILE* fileout = NULL;
 static boost::mutex* mutexDebugLog = NULL;
-
-volatile bool fReopenOmniLog = false;
+/** Flag to indicate, whether the Omni Core log file should be reopened. */
+extern volatile bool fReopenOmniCoreLog;

 /**
  * Opens debug log file.
@@ -119,8 +119,8 @@ int LogFilePrint(const std::string& str)
         boost::mutex::scoped_lock scoped_lock(*mutexDebugLog);

         // Reopen the log file, if requested
-        if (fReopenOmniLog) {
-            fReopenOmniLog = false;
+        if (fReopenOmniCoreLog) {
+            fReopenOmniCoreLog = false;
             boost::filesystem::path pathDebug = GetDataDir() / LOG_FILENAME;
             if (freopen(pathDebug.string().c_str(), "a", fileout) != NULL) {
                 setbuf(fileout, NULL); // Unbuffered
diff --git a/src/mastercore_log.h b/src/mastercore_log.h
index ae845a4..b0fcf75 100644
--- a/src/mastercore_log.h
+++ b/src/mastercore_log.h
@@ -44,8 +44,6 @@ extern bool msc_debug_metadex1;
 extern bool msc_debug_metadex2;
 extern bool msc_debug_metadex3;

-extern volatile bool fReopenOmniLog;
-
 /* When we switch to C++11, this can be switched to variadic templates instead
  * of this macro-based construction (see tinyformat.h).
  */
diff --git a/src/util.cpp b/src/util.cpp
index a03f55c..e92a6e8 100644
--- a/src/util.cpp
+++ b/src/util.cpp
@@ -104,6 +104,8 @@ string strMiscWarning;
 bool fLogTimestamps = false;
 bool fLogIPs = false;
 volatile bool fReopenDebugLog = false;
+/** Flag to indicate, whether the Omni Core log file should be opened. */
+volatile bool fReopenOmniCoreLog = false;

 /** Init OpenSSL library multithreading support */
 static CCriticalSection** ppmutexOpenSSL;
diff --git a/src/util.h b/src/util.h
index a4aaf29..079a09d 100644
--- a/src/util.h
+++ b/src/util.h
@@ -37,6 +37,8 @@ extern std::string strMiscWarning;
 extern bool fLogTimestamps;
 extern bool fLogIPs;
 extern volatile bool fReopenDebugLog;
+/** Flag to indicate, whether the Omni Core log file should be reopened. */
+extern volatile bool fReopenOmniCoreLog;

 void SetupEnvironment();

```


Also put the refresh function in itemsController


Is it possible to give a real example of when/how you would use this function.  I'm not 100% sure I understand its use case.


Do you think we are going to need more than one logger in the future? If so, it would probably make sense to refactor the logger into an object oriented approach, with variable file name. I was actually thinking about this, but I wanted to spare time, because we only had one. :) Now it could be different though..

Let me know what you think.


Yeah, I guess so.


new db


I think this is against the current consensus on npm package names: the repo is `node-foo` but the npm package is `foo`. See, for example, https://github.com/vpulim/node-soap/blob/master/package.json#L2

Maybe we should call everything `node-artemissbs` or `node-asbs`?


Este sería el gestor para imprimir la figura en el canvas pero todavía no lo han explicado y online encuentro swing pero no awt


'with the number of people in each class'),


Yeah, that's true, although for DBus method handlers, the style has to change because the `GError` is returned by `g_dbus_method_invocation_return_error()`, not passed back up to the caller.

But in general yes, please do try to match that `goto out` style - I know the coding style is a bit weird at first, but I find that the "rigor" it brings makes it harder to introduce errors.


Right, but "m" meters is assigned a nominal value of 0. When defining sl_signal_info, what do I use for defining sl_unit?


No, if it's a class, then it's good, because you can keep the class when you change DOM elements. I was just referring here to the case where you're tying to a `<dl>` specifically.


'whether they are in a flood affected area or not. You can '


Not explicitly passing the class loader used to load orientdb as the parent results in class loading errors if orientdb is not on the application class path, e.g. when orientdb itself is loaded from an embedded class loader (system < application < embedded), because the parent class loader defaults to the application class loader if not passed. Maybe pass Thread.currentThread().getContextClassLoader() or OServerPluginManager.class.getClassLoader() instead, which should be the proper parent loader.


'percentage of the affected (or should it be impacted ??) population affected should be '


@yjun2 done with my review. checkpoint complete. press on!


'based on the user defined minimum needs file.'),


> If the operation is taking a long time, the user might go away (on the other end of the http call). In this case we want to remove the pending op and tear this all down, so we don't allocate an IP which will never be freed.

Don't we face that situation anyway if we do allocate and return an IP address, but the weave script is killed before it manages to do anything with it?

(Though I wonder: We de-allocate addresses when a container dies, right?  Can't interrupted allocations be handled through that mechanism?)


这里可以改成 new FolderPermView(options); 然后在 FolderPermView 初始化函数中直接弹出对话框。


'name': tr('Raster flood on population'),


Yeah, it's bad, I agree.


This code has been [recently](https://github.com/rhatdan/docker/pull/36) removed. See [bz#1212501](https://bugzilla.redhat.com/show_bug.cgi?id=1212501).


> I could make Claim return the IP

Yes, I think that's what I'd do.


Sadly that's .net 4.5 only.


'people and the  required needs based on the user '
+                'defined minimum needs are reported. The threshold can be changed and '
+                'even contain multiple numbers in which case evacuation and '


Fixed these.


I regret wasting your valuable time; first thank you for your input and guidance. However, before I make this edit I would just like to get your suggestion on howto. I know I can find this file and view it in XCODE is that the proper way to edit same.  Thank you very much.


updateValue


Actually, it wasn't a wrong thing to do (I was wondering why the example had a bad practice). `waitFor` methods have a default timeout of 500 ms, so the test will fail after that time, if there's no content injected.


@livingincode do not revise this but do consider what would happen if the method as invoked with a dollars value of 3 or 10 or 99 or any number not equal to the five tests.


Throwing exceptions in tests is not idiomatic (modulo assert). In ScalaCheck return `proved` or `falsified`. You may want to compare SBT diagnostic of failed test and test that threw exception.


@livingincode done with my review. checkpoint complete. press on! ship checkpoints.


That works, or maybe a clearer `requestChange` (borrowed from ReactLink)


Well then use `String.replace("'\"","'")` on the messages before printing them to the user. That way both CLI and IDE can present things in the way that makes sense.


See WordPress-Coding-Standards/WordPress-Coding-Standards#17.


It didn’t strike me that these were constants as their values were intended to be modified by the user. Your point is well taken.

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Wednesday, April 22, 2015 7:02 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] Enhance seeds.rb (6a96e63)

It is a Ruby convention to name such constants all caps. So rather NUM_OF_BULK_POSTS than num_of_bulk_posts. As a bonus you will get a warning if you try to reassign such a constant.

See https://github.com/rails/rails/blob/master/railties/lib/rails/backtrace_cleaner.rb#L5 and https://github.com/rails/rails/blob/master/railties/lib/rails/generators.rb#L26 for examples of defining constants in the Rails code base.

—
Reply to this email directly or view it on GitHub <https://github.com/Mei152/Bloccit/commit/6a96e63030b29efeaef1338fd59e63f5de819fa5#commitcomment-10844371> .  <https://github.com/notifications/beacon/AKgq0L-0bJo-CGumSughxHXNNFc6Fk06ks5oB3c3gaJpZM4EF9cV.gif>


is this multi threshold option recorded anywhere else? And how does the report deal with advising users what their numbers relate to. What format should the multiple thresholds be input as. IIRC - this did not work when i tried it??


thanks
I'm not fully give up with gulp-runtime (as I like the idea) but will wait for some stabilization


We were already catching the exception and if the error doesn't match, it would not be reraised again. Or am I missing something?


看到这个提交的标题，莫名很萌……


Yes, I was wondering about this. I should have tested it in irb to determine the answer for myself.

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Wednesday, April 22, 2015 7:04 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] Enhance seeds.rb (6a96e63)

singe_post_title and single_post_body are already strings, so there's no need for string interpolation here.

—
Reply to this email directly or view it on GitHub <https://github.com/Mei152/Bloccit/commit/6a96e63030b29efeaef1338fd59e63f5de819fa5#commitcomment-10844410> .  <https://github.com/notifications/beacon/AKgq0MqIbUv4T1qV-ISFt7as5Lj5gTXmks5oB3eOgaJpZM4EF9cV.gif>


More idiomatic coffee:

```coffee
window.Todo ?= {}
```


gotcha.


Si tu pouvais le faire pour toutes les nodes qu'impacte builtin_falling ça serait cool :)


Use Conditional Properties
https://github.com/rickynils/scalacheck/wiki/User-Guide


In ammonite REPL they did manage to get the ctrl+C working cleanly
http://lihaoyi.github.io/Ammonite/#Ammonite-REPL

We could check how they did it or try to migrate to Amonite REPL (not sure how it will be maintained in the long run though).


update this if required with change above


as above


Are you saying you also have some of those in the typechecker?


Thanks for the revert.

Christophe


dammit :(


I wasn’t sure if the “if” would just test the truth of the first term in the expression. I figured parentheses would be the belt and suspenders approach. Again, irb would have answered this for me.

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Wednesday, April 22, 2015 7:06 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] Enhance seeds.rb (6a96e63)

Here, as well as in conditions above, the parens are not necessary.

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


This was fixed already by me yesterday, you should rebase your PR on the current `release/4.1`


I think so. At least in principle I _could_ have.


`property` should be descriptive.
"melding heaps produce..." would be a better alternative, or just "meld() works".


 'Provide details about the number of building that are within '
                ' each hazard zone.'),


Autant pour moi, j'ai lu en diagonal !
Quand j'avais essayé en jeu, je n'avais pas réussit...


you're right, the check is not good.
While your patch is ok, i'm checking if we can improve this function, as it seems it trigger useless queries in most cases. i'm on it.


why 'need evacuation' and not be impacted?


Revisions were pushed to GitHub for your inspection.

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Wednesday, April 22, 2015 7:07 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] Enhance seeds.rb (6a96e63)

This is a good use of string interpolation.

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


'Provide details about the number of people that are within '
                'each hazard zone.'),


You may want to decide for yourself when to use braces or curly braces. Otherwise you use both styles arbitrary in single project.


@habuma This project is consisting fundamental problems. There are some major problems such as persisting whole sign in attempt in sessions. I tried to contact with you but I had no chance. If you do not consider fixing those errors, please let us know. There are one year old PR's which are fixing these errors. I personally really love this project but maintaining a fork started to be really cumbersome.


I think we can avoid adding a `null` to this property? at JDO level, if an entity has a _missing_ property is equivalent to `null`


'To assess the impact of a volcano eruption on people.'),


Because Scala does not apply implicit conversion to already implicitly converted types, we do not have implicit conversions form `T` to `Option[T]`  that would lose some type safety and cause confusion.


Chú làm củ chuối phết nhể. Anh làm bên anh thế này nè:

- Làm 1 vector chứa img_names và so sánh dựa trên đó (dùng compare()).
- Cuối cùng rồi mới cộng thêm dir vào.


I don't think you need this change to work around the builder bug.  Only the top code is executed during the build, not the code at line 46.


Nice catch!

Unfortunately, we need to include a little logic here, because we do need to account for the bytes taken up by the websocket header so we can't cut it back quite so drastically.  I recommend:
             test_connection.onmessage = function (response) {
                 var hdr_size;
                 if (response.data.byteLength < 126) {
                     hdr_size = 2;
                 } else if (response.data.byteLength < 65536) {
                     hdr_size = 4;
                 } else {
                     hdr_size = 10;
                 }
                 received_bytes += (hdr_size + response.data.byteLength);

This proposed new code is actually more correct than the previous code, because the previous code has a off-by-3 error because it accounts for the websocket header size and then omits the ndt header size.


I get from the code that it takes the vendor directory (if exists) from `WEBDIR` and move it to the `LIBDIR` so if the vendor dir is already in the `WEBDIR` I shouldn't move it to the `WEBDIR` (which will be equivalent to not moving it).
Maybe I lost myself in some point?


I wonder whether we also need a patch to https://github.com/git/git-scm.com/blob/0e8dfa92ca1d0b47e6b66adecc0c149f191c97d5/app/assets/javascripts/site.js#L41-L42 that enquires via `window.navigator.cpuClass` whether the current browser is 32-bit or 64-bit and selects the appropriate download via conditional blocks?


Start


[SPARK-7056]Update algebird-core to version 0.9.0 from 0.8.1. It will solve object CMSHasherImplicits compatibility issue as well (https://github.com/apache/spark/pull/5629).


the extra 'e' in 'mee' is also redundant. Dickwad...


This line seems to cause a parse error on some versions of PHP but not others.


Yep, good point.  That was an easy fix.  Thanks for the feedback!  :grinning:


Dammit, sorry. I had set this, but it must've gotten lost before I merge. Thanks.


This line seems to cause a parse error on some versions of PHP but not others. Maybe the quote is considered as escaped?!


this is why the tests are failing.

>  var model = project.open(masterName)

You can use jshint, it would have warned you. I use it in my editor (geany) works well.


do we need to validate Secret.Annotations["serviceAccountUID"] and  Secret.Data["token"] as well since they are required?


Granskad.


Fixed the indent. I have a lot yet to learn about HTML.

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Wednesday, April 22, 2015 7:13 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] Add funtionality for questions (f96e66d)

Unnecessary indentation. There is no open element here, so the link should be at the same level as h1 and p.

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


What he said


Hi @timlinux - a few comments in line with suggested edits.
I am not sure if we have a definition for affected, impacted and need evacuation yet.
the EQ on people need the word function in the title
we need to be clear in the wording that in most IF we are simply counting the number of people or buildings or roads that are within a specific area. We are not determining if anyone is 'likely to be affected' really because we do not have vulnerability attributes (yet)
so we can say x people are in the high hazard zone. If we get some solid reference that says eg old people (age .. ) will be more affected by high eq hazard zone that young people; then we can start talking about likely impact or need evacuation etc.

Anyway - great edits, i think we have a long way to go still to explain exactly what the IF do but i like this  start on the consistency.


Nice! I've always wanted to try qbittorrent. Thanks for implementing!


Tại sao phải có đoạn này?

<code>
if (src_idx > dst_idx)
   std::swap(src_idx, dst_idx);
</code>


I'm sorry i'm not an expert at all in python and I get on internet that I should use `update` method on dictionary to make sure that I change the dictionary by reference and not by value.
Just for my personal interest when you pass a dictionary in a method it's always by reference? In php world it's not that why I tried with this.


Hmm, might be a Virtualbox version thing: http://askubuntu.com/questions/238040/how-do-i-fix-name-service-for-vagrant-client

Just testing to check that nothing weird happens if I add it to my copy


Whoops, next commit fixes :)


This seems a little hackish, why do you disable horizontal scrolling, does something flow out of container's bounds?


update database name


I believe it may ignore  all exceptions because of the 2nd pass , the outside the if statement. But since we are not validating for MacOS, aren't we ignoring valid Socket exceptions?


disregard, I see the uid is set in create...was reviewing commit by commit


Lesson 3


This has already been fixed on the current `release/4.1` - please rebase your work.


Here you completely ignored the fact that QuickCheckHeap is actually independent of type parameter (also true for BinomialHeap). Properties could be proved without hardcoding exact type. For example:

    lazy val genHeap: Gen[H] = for {
      a <- arbitrary[A]
      h <- frequency( (5, genHeap), (1, const(empty)) )
    } yield insert(a, h)

There I have an option to invent StringHeap and compile the test `with StringHeap`.


I have now merged your changes. Lines numbers changed a bit so I'm showing the text.
"what kind of thing are you iterating over"?  see above comment.
And, you don't even know\nthat it's an array. - also only 32CPL
This works the same as when\nI've been doing: 10.times.each:  - only 35CPL
Although it appears to me that you have split the above three captions into two lines on github, they have not been split on my copy, despite merging. I will wait to hear from you before I alter them on my copy.


Right, my bad. How stable is v4 at the time of writing? Can it be released as a stable release?


Ack. I did like the elegance, though.


Varför använder du get "cases/index" osv ?
när du kan använda dig av
```
  resources :cases
```


`window.navigator.cpuClass` is not *that* reliable. http://stackoverflow.com/a/1741978 convinced me to go for the *just offer 32bit and 64bit links* method.


`moment(new Date())` -dDoesn't `moment()` return the current date? Why do you pass `new Date()` as an argument?


Ok, fair enough.


if maxHandlingTime is less than 0,  the following code in the run() method of DeadlockGuard will throw exception:
Thread.sleep(maxHandlingTime);//this will throw IllegalArgumentException


I guess this line is related to
``` C
inline int sched_yield(void)
```
right?


Ah and ```config``` must be freed in the exit paths with ```libusb_free_config_descriptor(config)```, the ```libusb_get_active_config_descriptor``` doc says this, I noticed it while working on my proposal for inclusion in libusb: http://sourceforge.net/p/libusb/mailman/message/33960201/


In the controller you sign in `Admin` user, but in the layout you check `current_user.admin?` - why not `current_admin`?


Lägg alltid till .DS_Store i .gitignore, dessa bör aldrig aldrig kommittas.


HelloWorld.com ## This is a markdown file


I don't believe it ever was. DAP GA "include" filters are used to determine which agency profile the sites/domains should be assigned to.


View this with          ?w=1              added to the GitHub URL to suppress (some) whitespace differences.


Thanks for the tip, I'm french so maybe sometimes I don't get everything :) I get that you asked me to not make overridable by users this FLAG


Though if we want to allow this pass Theme Check, the `add_admin_menu()` method will need to be changed as Theme Check will freak on the `add_submenu_page()` call.


Borde vara function() inte function () på ganska många ställen.


@arnekolja thank you for reporting, wie plan to add a dedicated clear dyncss cache button, as the  normal clear fe cache should not clear the dyncss cache


It is highly unlikely that humanity could crack the codes in the lifetime of our galaxy, let alone that of a human being (so it is open to debate whether our attempts to do so should be considered inspirational or grossly stupid)

一点小意见：of a human being 指的是人类的寿命。括号里的话是针对后面全球都在合力破解密码说的。

靠人力蛮干的话， 恐怕在银河系消逝之前都干不完， 更别说人类的寿命远没那么长.。（所以这种埋头苦干的行为到底应看成是励志还是一种愚蠢呢？）


Granskad.


Okay... but that comment says that we should not auto-populate that download box with a platform-specific download. However, there is already a way to download Git for another platform than you're currently on, and I would rather have the same extra step only for people who want to download for a different architecture...


I don't think you need to do this, because, you are already visiting this path the in before block.


method_missing (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (340 of 341 tests passed)

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


Yea, sorry about that.  The change is hard to see here because it's just pointing at line 169 & 170.

Looking back at the [original code](https://github.com/cloudfoundry/php-buildpack/blob/master/extensions/composer/extension.py#L172-L179) you're correct.  It's checking for an existing vendor directory in WEBDIR.  If that exists, it moves it to the LIBDIR.  So my initial thought is that we'd want to alter that logic to something like if `os.path.exists(vendor_path) and not self._ctx['NO_WEBDIR_SET']` then move to LIBDIR, else do nothing.  In other words, we only move to LIBDIR if it exists and we have a defined WEBDIR (i.e. user set WEBDIR or included an `htdocs` directory).

Having said that, I think I need to investigate this code path more because it seems like we might be able to remove it.  In the cases where NO_WEBDIR_SET is true, we want the vendor directory to stay in the WEBDIR (i.e. this method does nothing).  In other cases where there is a defined WEBDIR, the vendor directory is almost certainly not in WEBDIR.  It's probably somewhere else already (i.e. does not exist, we do nothing).  What I'd like to see is if we could just remove this whole method without breaking any of our tests.

This would be tricky for you as I'm not sure if it's easy to run the test suite.  Need to check on this with the build pack team.


Follow the instructions in the above comment.


It works fine but it doesn't have as many tests as i would like to.


Isn't just upping the timeout to 500 millis still a bit risky? What if you have a particularly loaded browser?


# ARRAY TRIGONOMETRY
# GWU
# Exercise 3 module 1.
# data:
# Given the following code:
# p = 7
# r = numpy.array([11.2, 4.7, 6.6])
# Compute sin3(p/r) in Python

In [1]: import numpy as np
In [2]: p=7
In [3]: r=np.array([11.2,4.7,6.6])
In [4]: np.power([np.sin(p/r)],3)
Out[4]: array([[ 0.20030151,  0.99009101,  0.66454238]])
In [5]:


Ok ora comprendo meglio, farei un semplice cambiamento tipografico togliendo i sottotitoli "aggiungere dati", "leggere dati" e trasformando "leggere dati" in una voce dell'elenco. Per evitare l'impressione che transazioni, consistency e atomicity riguardino solo le letture.


I don't think that's necessary, though. Once we break the for loop thebitem will receive a tick and remove itself anyway


I had a brief look some time ago.  We need to disentangle and revive this discussion.  At the moment it's a bit lost and confused amongst a few different discussions about APIs.


> > I can appreciate that we might not want to support networks that where some peers are using IPAM and others aren't
>
> ... although given that we seem to allow clusters where peers were launched with different -alloc ranges, I'm not sure why we shouldn't cope with one that has no -alloc option.

Ok, although this isn't supported, there are no complaints when you set things up like that.  It would be nice if it told you.


If I wanted to I would :p


This method can be made idempotent.  See https://github.com/GoogleCloudPlatform/kubernetes/pull/7022.

Likewise for adding a Stop shutdown method.

This will allow an easy refactor into a controller plugin when that framework lands.


Yep, please rename file


What about using a context manager like what was done with `NodeModify`. See https://github.com/Slicer/Slicer/blob/9c507f4b9ed57f243e9c259d1966a9d0e9965766/Base/Python/slicer/util.py#L477-L486

It means you could do the following:

```
with slicer.util.TempDirectory() as tempDir:
  # code that use tempDir
```

At the end of the block or in case of exception, the directory will be deleted.


Even if you don't provide a real name (which would be appreciated), please provide a valid email address so we can contact you should we ever need to relicense this or anything.


On my copy this is one caption
But the problem you've given me is\nfrom the perspective of a client of an API.
If you have split it into two captions this does not appear on my copy.
I have simply moved the \n line break since you added extra text.


what can possibly be a reason to include that log file in your repo?


I would use allow instead since this is not really an expectation.


How yow solve the undefined 'get_lists' ?


Added “t.references :question, index: true” to answer migrate file, ran rake db:migrate, and pushed to GitHub.

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Wednesday, April 22, 2015 7:18 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] Add question and answer models (07f9aa1)

The instruction said:

Create another new model called Answer. It should reference Question and have a body attribute.

You're missing the "reference Question" part. Even with belongs_to :question in the model this is not going to work. Do you know what needs to be added here?

—
Reply to this email directly or view it on GitHub <https://github.com/Mei152/Bloccit/commit/07f9aa19e0f8e8447e4d7e7c08d82774e71b06ba#commitcomment-10844641> .  <https://github.com/notifications/beacon/AKgq0EqG2qqxAYPIydAG6YKReOrWzAYbks5oB3r8gaJpZM4EF-BW.gif>


Didn't notice the later commit


![qq 20150422210147](https://cloud.githubusercontent.com/assets/2126930/7274865/094c1ad8-e933-11e4-9acb-e407fa2269ff.png)


No worries, this is a fair question.  You can modify the dictionary that's passed into the function, by that I mean you can modify the contents of it.  You can't say point it to a totally different dictionary.

Ex:

```
>>> def test(x):
...   x = {'a': 1}
...
>>> a = {'b': 2}
>>> test(a)
>>> a
{'b': 2}
>>> def test(x):
...   x['a'] = 1
...
>>> test(a)
>>>
>>> a
{'a': 1, 'b': 2}
>>>
```


Borde vara "function() {" inte "function(){" eller "function () {"


。。这样就没有办法做json编码了。。。


True, a lot of different things were discussed. That's why I propose to check what I did (which was based on the last conclusions we made) and see if it's worth working on top of it to make it reusable in other ports.


Swagger


Archiver updated.


Granskad.


I assume it's here because the parseTrackXML just transforms the XML 1:1 to a JS-object and this function maps the api response to the format Tomahawk expects


What is odis+http?


Tagged 4.0.0 right now.


:+1: Also was annoying to me but I haven't found time to fix, thanks for this PR


Looks good. I might try to focus a bit more on report writing this week (unless some part could use more attention in which case I'm happy to join in). If it is ok with everyone I might go over the stuff that is already written and propose rephrasings and such, if you want to keep anything I change feel free to change back or comment on the commit so the group can discuss. After that I will try and add or fill out parts that has need of more text.


:ok:


Uhm, do we really want a transparent proxy? [Line 263](https://github.com/jalavik/jsonalchemy/commit/6ec26adf4457c8e1c6ade889c6c7537bdf6eb735#diff-d9c4c201a2dd339280c7995d37479ce6R263) now doesn't raise an exception as it should, since `append` is not enforcing that the thing that is being appended is of the right type.


Unless you are also using this search thing from within the collection methods, they are not supported right now. i.e. you cant return album or artist results for searches


This class needs to perform actions on every session's cached EntryCache. Unfortunately, the ICacheManager lacks a way to retrieve a collection of entries across session by key, such as a cacheManager.getFromAllSessions( KEY ) -> List.

To work around this limitation I have to grab the whole session region and iterate through all keys. In the CacheManager implementation, keys in the session region are a composite of session_id + "\t" + key. This method splits those composites out and performs the given action on entries matching to base key ENTRY_COLLECTOR.

I'm assuming that the configured ICacheManager is in-fact the CacheManager impl or someone constructing session region keys in the same way. Not ideal I know. I'm open to suggestions. I can make this configurable so that someone with a custom impl of the ICacheManager can provide us with a ICacheManagerTransSessionKeyedEntryCollector or something.


"and and"


sfdfds


La variable coups dans Tour compte le nombre de coups dans un tour, la variable coups dans les parties sert à additionner les coups joués par un joueur à chaque tour. Mais dans le solo il n'y as qu'un tour donc tu peut enlever la variable.


it's how the URL is returned by 3voor12 when doing this ajax request..


Fixed, thanks!
The problem was caused by the fact that Infinispan and WildFly apparently used different names for that module.


Where did you get this from? I'm really not sure what this is used for but it seems to be some basic throttling implementation... do you really need it?


The whole code looks really complicated, I'm feeling bad for asking, but would you mind porting to our new Promise based API? I'll walk you through it


File does not end with a new line.


I'm sorry, what do you mean exactly? I'm not familiar with Hibernate Entities. Are you asking how to pass an object to a Sundial Job from the main application class? Because, yes, that is possible.


This is needed for all the controller templates (*.cpy, *.vpy) here: https://github.com/plone/Products.ATContentTypes/tree/master/Products/ATContentTypes/skins/ATContentTypes


Noted. I think I'd favor the combined admission controller so there isn't ordering issues.  This would be the relevant security context application: https://github.com/pweil-/kubernetes/blob/d431e9feb42ad6b1b0c4d518311d82f49d7ad1b3/plugin/pkg/admission/securitycontext/admission.go#L66-L78


Looks good. Just "[[]]" is less portable than "[]". But for majority Linux distributions bash is default shell. It should be OK.


I wonder if we want to rename _start to pageNumber or something similiar?


Hi,
Yup, just pointing out that the example was missing.

On Tue, Apr 21, 2015 at 2:33 PM, tpltnt <notifications@github.com> wrote:

> Hi Keylo,
> your pull requests only adds one empty line in the text of the chapter.
> There is no hint at the "missing Player.cpp code" as mentioned in the
> title. Is that intentional?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Keylo/ofBook/commit/982bca32dca9855a581fed03bc8049dd7278439a#commitcomment-10831461>
> .
>

--

Keylo.
www.WholeNewSpin.com
New York City


Probably don't need that guy :)


Should it return List<List<ReportRawData>>?


Python 2.7 docstring says -  "Return a Unicode string of one character with ordinal i; 0 <= i <= 0x10ffff."

Would generally agree that using the exception to manage the control flow is not ideal.  But I don't know if there are other special cases within the standard unichr that could throw ValueError even within the acceptable range of 0 to 0x10ffff - unicode is complicated that way and I'm not an expert on those possible special cases.


Borde vara "function(a, b) {" inte "function(a,b){"

Borde vara:
if (A < B) {
  return -1;
} else if (A > B) {
  return  1;
} else {
  return 0;
}


Granskad.


* I suppose it fails if there are no statuses with name 'negotiations'
* I see no default sorting in this model, so #take in this case will return almost random record. Does it suits you?
* Maybe this method should be named like default_status_id if it returns an id.


update the all filed on sample app


update


I don't agree.
You should install `grunt` globally. It's really unproductive to run `./node_modules/.bin/grunt` instead of just `grunt`.


Thank you for the clarification.


Borde vara "function() {" inte "function () {"


You can use .freeze method on this string to make it constant. Just for foolproof and safety.


:+1:  Well spotted.


Awesome!


I see why you have added this enum (because of the check in `IntervalStatistics`) but I think it could have be done in a more simple way. At the moment, we use  the first letter of `ITestElement.Result` and then based on that letter we determine `ExecutionResult`. We need a single letter for serialization, but I guess it's possible to use `ITestElement.Result` for statistics. Do you want me to explore this a bit more and try to fix it?


This model does not work, is it wrong as approach? I am summing the reified checks i == item in the series to count the occurrences and then creating an equality constraint with my ith value...I cannot see why it is not working. I know that ```$cardinality``` does exactly that, but I am here to learn...


Can we avoid calling `b.f.Read()` when `b.pb` is empty? I am concerned that `b.f.Read()` is a blocking call (not an asynchronous fd) and `Ajwaj.Read()` could block even though the buffer contains something which matches the expect.


Done!


The scrollbar does not work here and looks bad. There are arrow buttons for scrolling.


![qq 20150422210147](https://cloud.githubusercontent.com/assets/2126930/7275281/9722dd72-e935-11e4-84a9-2aee31e8f4bf.png)


If you need strict time comparing, you should use DateTime.now.utc


Have you noticed any performance issues with the list being so much larger?


Should be `Tuple{Vararg{Vector{T}}}`


This breaks all programs compiled against gnutls. Did you even test this? The gnutls.pc file has `-lgmp` and `-ltspi` but those are not propagated.


forgot but i fix saving the mapfavs too.


@xiangzhai no plans to write any books, but one day I might start a blog and write some stuff about it. It's not like we're doing anything fundamentally different from any other emulator that does JIT/dynarec though..


@Montellese
Thanks for even taking time to reply, even that goes along way in my books ;)


En solo il y a 10 combinaisons à touver, donc il y a 10 tours?


@wkeese Thank you for your review, I have an impression that you meant this particular line, is it correct?


Excellent, my patches are on an older version. But as they are a
strawman only, I think that's fine.

On 04/22/2015 08:31 AM, Michal Minar wrote:
>
> This code has been recently
> <https://github.com/rhatdan/docker/pull/36> removed. See bz#1212501
> <https://bugzilla.redhat.com/show_bug.cgi?id=1212501>.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/llunved/docker/commit/7563f4e55c38c55dd1d281d8c92cee329d9780d3#commitcomment-10845817>.
>


Mostly correct, but actually switching to different orientation is not the same as killing app. App isn't killed. Only activity is destroyed (and then recreated)


If I'm interpreting this correctly, this is supposed to provide highlighting? Why doesn't adding the rednet.* things add to the highlighting?


I do not think I get this one clearly... I am checking boolean value here on the model. What is the point of generating additional helper method for this?


Correct, again need to emphasize it's not only activity got removed from the stack,
When hitting the back button, top most activity is destroyed


vb


Thanks! I guess we can even remove the comment with the sleep version :)

Also, it's probably not necessary but I would like a third test with two expects of the same size:
```
sh -c "echo Hello1 ; echo Hello2"
expect Hello1
expect Hello2
```


My point is that you are checking two different user types here (I assume that this is devise). `before_filter :authenticate_admin!` this line works for `Admin` model and helpers like `current_admin` are generated. So I am a little confused why you are checking `current_user` in the view, afaik current_user will be `nil` here. Or am I missing something fundemantal here?


nice


I don't think we have a pattern for this in the API docs, which I agree is pretty bad. We'll end up with the most detailed explanations in the upgrade guide, so we could link to that for anything complex. For simple things, just a sentence like "The `hoverClass` option has been deprecated in favor of the `classes` option." with a link to the `classes` option should be fine.


Nice :)


CondenserToZone_IDD (Nigusse) - x86_64-MacOS-10.9-clang: OK (1641 of 1641 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 changing the repo name if you're sticking with this name change


You're right. Fixed in f680a07813d937db4f6bd1d3d3fc5f63a9ef6489 Thanks! :)


Whats goin' on with indentation here?


Oh my goodness. Good catch. I guess in an ideal world we should add a test that would catch this. I'll update my PR for the change but and probably not quite clever enough to add the relevant test.


Good call, thanks.


Looks unnecessary, but doesn't matter in a branch either.


Semicolon missing
Fixed in #686


Or maybe not... something else is breaking gnutls maybe, or something else changed such that -L of gmp is not picked anymore, in staging.


I do not agree ;)
I'd never install a npm module globally to avoid conflicts between versions.


Solved in priv, `authenticate_admin!` is a custom method that checks `current_user.admin?`


No, `List` would be a bad idea because then it loses the sense of which one is queries/commands. The only good alternative solution I think it would be a new class, which can also support more reports in the future. But it's up to you, I don't find it so necessary right now. I only wanted to mention the dyslexic problem because I had to struggle a bit more to understand it because someone had the great idea to use left/right for that :P


Why here miliseconds where you just changed the standard to seconds?


@Szeliga there's no Admin model, just a user with boolean `admin` field. Yes, we use Devise, but `authenticate_admin!` method is not from there.


This is the error btw on a build:
```
/nix/store/sfb03nqk9gh67sika5l1f1ivvch0vklq-binutils-2.23.1/bin/ld: cannot find -ltspi
/nix/store/sfb03nqk9gh67sika5l1f1ivvch0vklq-binutils-2.23.1/bin/ld: cannot find -lgmp
collect2: error: ld returned 1 exit status
Makefile:856: recipe for target 'libgiognutls.la' failed
make[3]: *** [libgiognutls.la] Error 1
```


You were quicker ;)


Could you please add a test to assert this ends up in the phraselist when needed?


I still prefer miliseconds but that is because I am used to that from other languages like javascript


Hello to everybody who will assess this code


Does this command also issue the poison pill or does it just communicate intention for Hari Kiri?


I was actually doubting which way to go while implementing this. Will do it as you suggested.


@tcrsavage am I using filter there correctly? Got a bit confused with order of filters and translations.


please do  `$ git rebase upstream/master`


He revisado 1.9.x y ese era el comportamiento con FCKEditor


already using `isdigit()`


echo $obj{"roomnum"}


Bien! Quedará asi entonces.


Could you please add an integration test for yemen to make sure we know if we break anything in future by accident? :)


:+1: ENFIN


Oui j'avais mal lu l'énoncé, donc tu garde la variable tours dans solo et elle contiendras le nombre total de coups que tu as joué dans la partie.


This project is `ng-json-explorer`


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


much correct version is:
`return empty( $toplytics_result['result'] ) ? array() : $toplytics_result['result'];`
because the $toplytics_result can have some other info inside and $toplytics_result['result'] can be empty.


Beleza!


Vamos conversar hoje.
Em 22/04/2015 10:40, "Alexsandro Santos" <notifications@github.com>
escreveu:

> Beleza!
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/lasilva/cursoAndroid/commit/17c8b810df83e91406e1205b1fea0dec26b83f56#commitcomment-10847026>
> .
>


à cái matcher của OpenCV chì đọc phân nửa tam giác trên của matching mask
thôi (vì các cặp ảnh có thể biểu diễn bằng ma trận đối xứng) nên phải luôn
đảm bảo rằng cái pair đó thằng trước luôn nhỏ hơn thằng sau :v

Nguyễn Văn Khôi - Lớp CS0801 - SE61174

2015-04-22 19:49 GMT+07:00 Manh Thiep Nguyen <notifications@github.com>:

> Tại sao phải có đoạn này?
>
>
> if (src_idx > dst_idx)
> std::swap(src_idx, dst_idx);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iDoraemon/BiBo-Kit---Panorama/commit/672eb450ff9769ea3c95f12c237f5514fa74b06d#commitcomment-10846122>
> .
>


I think @csrwng created template section with screen shots of the web console.  Should there be a link to that material here someplace?


There could be one or two options or config choices.  For instance - global enforcing, namespace enforcing, namespace forced context, etc

> On Apr 22, 2015, at 9:13 AM, Paul <notifications@github.com> wrote:
>
> Noted. I think I'd favor the combined admission controller so there isn't ordering issues. This would be the relevant security context application: https://github.com/pweil-/kubernetes/blob/d431e9feb42ad6b1b0c4d518311d82f49d7ad1b3/plugin/pkg/admission/securitycontext/admission.go#L66-L78
>
> —
> Reply to this email directly or view it on GitHub.
>


ok


oh, I see, that makes sense. But I still like having 1 record arg where things are directly connected better than a separate filter arg. Note that we then have the option of replacing any existing callbacks rather than only being able to add to them (not sure which is best)


hàm feed của em chia làm hai trường hợp:
1. có pairwise
2. ko có pairwise
với trường hợp 2 thì em quét cả thư mục rồi và nó cho luôn cái path đúng
nên bước phía dưới ko cần phải cộng dir vào nữa, em làm thế để trường hợp 1
cái img_name cũng sẽ trả về giống với trường hợp 2 thôi.

Nguyễn Văn Khôi - Lớp CS0801 - SE61174

Vào 19:44 Ngày 22 tháng 04 năm 2015, Manh Thiep Nguyen <
notifications@github.com> đã viết:

> Chú làm củ chuối phết nhể. Anh làm bên anh thế này nè:
>
>    - Làm 1 vector chứa img_names và so sánh dựa trên đó (dùng compare()).
>    - Cuối cùng rồi mới cộng thêm dir vào.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iDoraemon/BiBo-Kit---Panorama/commit/9a70f977178e558ba491671d9dd646ceed8df074#commitcomment-10846030>
> .
>


I'm really impressed about your code, it is awesome the way you solved some Bukkit related challenges. I think you have created a new plugin development paradigm.
Congratulations!


I didn't see that `+ (void)initialize`  was a class method.


ta bort


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (339 of 340 tests passed)

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


Looks good


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


jo hvis vi får opdateret designelementerne :-)


Can you add a line about what the search path actually is? "A list of directories on your computer that MATLAB knows about"?


1. We're using two terms here, "path" and "search path". Can we stick to just one?
2. Can we change "folders" to "directories"?


using? mb uses?


fetches, compiles, replaces


Problème avec les signaux et slots, le makefile ne veut plus compiler, je bosse dessus, si quelqu'un a une solution


Sekke ta... BIODATA anne...^_^, kalau bisa setiap fungsi yang kita tulis ada penjelasannya via komentar in java..^_^...


each session has name and/or id and that can be assigned to cart.


Are spaces trimed in params?


wtf is whyle?


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


If some more roles should be added, you will need to create more role? methods. So maybe it makes sense to define these methods dynamically by looping on roles list. Moreover these ones have identically logic.


Move it to corresponding code lines, don't include this in function description - it's for developers wanting get deeper knowledge


Yes, I'm interested in helping out. Do you have any living documentation about your reversing efforts?


I like this default, we should use it on MinimalExample as well.


CondenserToZone_IDD (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1647 of 1647 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 have to connect to database. So how do I configure and use Sundial for this.


I have to connect to database. So how do I configure and use Sundial for
this.

Regards,
Lakshmisri Gopalan
IDEAS2IT Technologies

On Wed, Apr 22, 2015 at 6:43 PM, Tim Molter <notifications@github.com>
wrote:

> I'm sorry, what do you mean exactly? I'm not familiar with Hibernate
> Entities. Are you asking how to pass an object to a Sundial Job from the
> main application class? Because, yes, that is possible.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/timmolter/XDropWizard/commit/c1290b69eb6801fb247fe2d080d788c5bca053d8#commitcomment-10846522>
> .
>


Ai chả biết thế nhưng cái chỗ so sánh như thế kia của chú là không đáng.


@pwidry15 The content is good, but I don't believe you are using a markdown file format, which is why the formatting has not been enriched. I think all you need to do is add .md to the end of the file name and that should do it.

Also, can you please add an image or gif to your takeaway, per the assignment. You can check out my source code for the syntax for turning an image link into a picture in the file. Here is also a helpful link that I used: https://guides.github.com/features/mastering-markdown/

Thanks! Vince


@koke you're indeed right! I'll get that changed in a small PR. Thank you!


Are we sure we only want .txt files to be processed?

Wouldn't an array comparison be a better option here, similar to the isSupprotedImage() method


Can you please do a pull request?


Looks good to me.

I tried to write to a test for this, but the name gets validated differently according to context, which means valid does not mean valid. To fix this `validate-npm-package-name` should always be used.

Apart from the missing test, this is good to merge.

You might save one Array by `(its.errors || []).concat(its.warnings || [])`.


sudah sip, tapi tolong berikan penjelasan pada tiap baris dengan memberikan komentar pada java,

contoh :

+package baru; // nama paket adalah baru

dst.. =)


No, sorry , I didn't do any speed test yet.
I also have a 25MB list from hphosts that I would add to the sources. The fact that the code writes 6 times to files worries me a little bit.

If matter, andLight, supernova and eventHorizon were variables instead of files, would it be faster?

What do you think about moving files with .domains extension to /etc/pihole in order to make them persistent?


Commenting this line re-enable custom theme for GTK3 (disabled by a previous commit); is this intentional ?


Em làm thế để tránh trường hợp tên file này chứa tên file kia ấy ạ (kiểu
25.jpg chứa 5.jpg ấy), thêm pairwise thừa dễ toi lắm anh ạ. Nhất là bộ ảnh
đầu vào có nhiều chỗ giống nhau dù chụp ở 2 vị trí khác nhau. Mấy tâm ở Hòa
Lạc em ko thêm pairwise toàn ghép trật lất ngay cái nền gạch làm đường

Nguyễn Văn Khôi - Lớp CS0801 - SE61174

Vào 20:49 Ngày 22 tháng 04 năm 2015, Manh Thiep Nguyen <
notifications@github.com> đã viết:

> Ai chả biết thế nhưng cái chỗ so sánh như thế kia của chú là không đáng.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iDoraemon/BiBo-Kit---Panorama/commit/9a70f977178e558ba491671d9dd646ceed8df074#commitcomment-10847185>
> .
>


For a number of very short executions (few ms), averageTestDuration is equal to 0, which makes WatchDogView showing nothing on Test Run Duration bar. This is probably not introduced now with this refactoring, but I think this can be investigated a bit (find minimum execution time for which our View displays correct times).


hmmm... I have just checked out the current HEAD of our master branch (commit 9a9900fe5ab8248592f7b3bfb033a00de18600a5) and generated the metapolatorStandAlone.js file based on that. It does not differ from the file available in here.


Sundial is for Job Scheduling, not connecting to databases. You need to look at JDBC or Java ORMs like Hibernate.


done on the binding. consolidated the code so there are less statements


echo 'hello';


New level pack (Ladder Forever) and new level in nero (level_3.xml)


That spec is well-crafted!


rossifederico, regarding rotativa (cannot write there, since the topic was closed either send you a message (cannot find such option in github): via IIS: right-click on exe file -> Edit permission -> Security tab...


I need that job to connect to database. Thats where I am looking for both Sundial. Dropwizard is pretty clean with providing framework for database connectivity. So I would like to leverage Sundial for that. Hence the ask.


Chú vẫn thực sự không hiểu hàm string::compare() nó như thế nào đúng không?


CondenserToZone_IDD (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1647 of 1647 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)


This commit exposes the illustrative test in https://github.com/ximenesuk/openmicroscopy/commit/bbcf1f573de2ccf559efd69aabc1cf5f57f2c348

It leave it as an illustration but should also catch any regression on this issue.


![capture](https://cloud.githubusercontent.com/assets/5550526/7276108/ed2d3e52-e907-11e4-9dbb-c13a67a52b2b.PNG)
Just my opinion: this colour should be blue (rather than grey, because it's to similar to the background).


We are putting them in RAM now via `/run/shm`, but I don't know if a file or a variable is faster--I'll research a bit and see if I can find out.  But I guess for other systems like on a Mac, it might not be as easy to store everything in RAM.

I mostly like having the .domains so that it is easier to troubleshoot and see how each file gets parsed down.  It doesn't really matter where they are, but `/etc/pihole` makes sense.


No. It hasn't gone through full QA yet. I changed this back after Sheel's UAT feedback to not have the edit button in the fixture widget. I was under the impression that model iteration often uses cases, not fixtures, so I figured this might be a better change, otherwise you'd have to go back to custom XML. Thoughts?


Note that ActiveModel::Serializer is smart enough to detect that you are trying to render a collection of `User` models here, so it will use a `UserSerializer` if it is available. You can simplify your code to just state:

``` ruby
render json: users
```


Em chỉ lo chuyện 2 string có nội dung giống nhau hay ko thì (str1 == str2)
cũng như (str1.compare(str2) == 0) có khác nhau đâu hở anh?

Nguyễn Văn Khôi - Lớp CS0801 - SE61174

Vào 20:53 Ngày 22 tháng 04 năm 2015, Manh Thiep Nguyen <
notifications@github.com> đã viết:

> Chú vẫn thực sự không hiểu hàm string::compare() nó như thế nào đúng không?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iDoraemon/BiBo-Kit---Panorama/commit/9a70f977178e558ba491671d9dd646ceed8df074#commitcomment-10847283>
> .
>


Shouldn't this Commit go to https://github.com/rubyonrailspl/rubyonrailspl.github.io? Website on Github Pages isn't updated.


:+1: going to review this asap!


I use Yank, and demos of that are found in XDropWizard. Once you have it integrated you can call

    Yank.execute(...);

directly in any Sundial Job. That's how I do it, and it's really easy. If you want to use a different ORM or JDBC directly, different setup will be needed.


There'is some bug when you active/deactive the switch "previousScene" . When I tested it, the default link was the initial scene always after use the switch, also when I was in the initial scene.


Everything (other than small remarks in line comments) looks good, adding this to the WatchDogView makes it even more insightful. Great! :+1:


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

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


Fixed: https://github.com/combak/ShatteredWorld/issues/5


change description as well


stupid me! I did not merge my own #424 because travis did not execute the tests last night.


Does this allow for "unrestricted" during FEST tests?


sorry!


why the increased timeout duration?


changed to this, deadlock less frequent. still there though.


getting deadlocked here.


Don't be afraid to delete your code instead of comment it. Git is powerful.


i'd favor the implementation in your comment. i see no need to create a class + insatiate it when all you want to do is make sure that you pass an argument to an existing function in all cases. that smells like a wrapper to me. for a rationale for why classes should be avoided whenever possible see: http://pyvideo.org/video/880/stop-writing-classes


Why not something like that?

http://mail-archives.apache.org/mod_mbox/lucy-dev/201412.mbox/%3C5481DAA3.9020209%40aevum.de%3E


Right now your tab size is too big. Decrease to "using spaces" and the number of spaces should be "2". You can find these in the bottom left corner of your text editor.


Do not create sentry instance here. You can use the instance in `current_app.extensions['sentry']`, if present.


Nice commenting! :+1:


file://telnetd \ is missing


iflipb permet de rajouter la fonctionnalité de alt+tab comme dans gnome/win/mac
Les buffers sont vus comme une stack


Tri automatique


Try to keep it to one line in between each block for consistency.


@andylolz feel free to revert this if it's unhelpful!


It seems unused nowhere


Just curious, is this extra indentation intentional?


Очень общее название, стоит обдумать более говорящее


See here: https://github.com/tombatossals/angular-openlayers-directive/pull/93


Также стоит изменить имя аргумента


ace-jump-char-mode est plus efficace que just word qui restreint sur la première lettre des mots.
on peut jumper vers le caractère espace !


Trường hợp đó thì đúng nhưng chú không hề dùng == hay compare() một cách trực tiếp.

Anh cũng không hiểu sao chú không nhận ra: tại sao lại đi cộng (+) 1 chuỗi ngắn thành 1 chuỗi dài rồi lại lấy chuỗi ngắn trong 1 chuỗi dài (substr) để đi so sánh (==) :100: ?

Rốt cuộc thì anh hiểu cách các chú viết code rồi. Anh sẽ không comment những thứ như thế này nữa.


Remove this block since it's not doing anything yet, just to keep things clean :smiley:


The package is finally pusblished to the Bower registry.


you just copy code to that method or made any modification?


Не нужно хранить `seq` и `structure` в списке. Для этих целей лучше подойдёт `tuple`, а ещё лучше `namedtuple`


Let's use this colors "HEX" value. Can you find it and swap it out in your next commit/round of changes? Feel free to ask me any questions about this here.


Delete this extra space between elements!


Same thing here!


I put `deleteAllRecords()` inside DB transaction because if anything went wrong, the original data will not be deleted.

By putting it outside the DB transaction, if anything went wrong during importing, the user would have an empty DB.


Yes, it's either because JUnit does not report a run time, or because that run time is almost infinitely small (something like 0.0001). I suggest we do not look into this further in this PR?


`line` означает одну строку, а на самом деле является набором строк. Запутывание читающего.


Good remark.


You're brilliant! You did such a killer job on this first exercise! I'm really looking forward to how you'll grow as a coder and the badass-ness yet to come.

:100:


I think `repos` will need to be:
```R
repos=c("http:/owi.usgs.gov/R","http://cran.us.r-project.org")
```


Also, good commit message! Keep em coming!


Можно упростить до `if not line[i].strip():`


AFAIK Travis CI don't host artifacts themselves, the only allow you to upload them somewhere after the build (unlike AppVeyor).


Если использовать метод `readlines()`, то уже после этой строчки файл можно закрыть, так как он уже прочитан.


Thanks for pointing out .... fixed.


Remove extra indentation here. Let me know if you have any questions! We'll probably want to clean a lot of this up indentation-wise since we went through everything so quickly!


Same indentation thing here!


Make sure you use a space after the colon here!


Missing space again!


Ni borde använda er av _ separatorer som kodstandard. I rails / rubyvärlden använder sig alltid av detta.

vilket skulle innebära följande
```ruby
#Creates a new mail
def initialize(subject, body,to_address,from_address)
  @mail_subject=subject
  @mail_body=body
  @mail_to_address = to_address
  @mailFromAddress = from_address
end

def get_body()
  return @mail_body.to_s
end

def get_subject()
  return @mail_subject.to_s
end

def set_category(category)
  @mail_category = category
end

def get_category()
  return @mail_category
end

def get_mail_to_address
  return @mail_to_address
end
```


Love this hover transition! Looks gorgeous. :dancer:


CondenserToZone_IDD (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1647 of 1647 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)


Thanks for the heads up. I appreciate it.


Let's find a HEX value for this color property. Generally we want to stick with HEX values if the color is staying in our design :star2:


Hate to be a pain but shouldn't this be `(if (= forms 0))`? :P

Other than this really minor thing LGTM


Yay clearfixing!


Sen har vi även den lilla detaljen att i ruby använder man sig inte av get och set..
Vilket i sin tur skulle resultera i följande kod
```ruby
#Creates a new mail
def initialize(subject, body,to_address,from_address)
  @mail_subject=subject
  @mail_body=body
  @mail_to_address = to_address
  @mailFromAddress = from_address
end

def body()
  return @mail_body.to_s
end

def subject()
  return @mail_subject.to_s
end

def category=(category)
  @mail_category = category
end

def category()
  return @mail_category
end

def mail_to_address
  return @mail_to_address
end
```


Fix indentation here!


@ehdem Can you elaborate on the bug description please? I have a feeling that this parameter will simply avoid the buggy code and not fix it.


routes ser inte ut så längre, om inte jag missminner mig så är detta en "mitt i commit"


This anchor tag `<a>` should be indented because it's nested inside of the `<li>` element.


thanks :)


お、ラムダ式
さんきゅい
あとで勉強してみる


You remembered the `href="#"` trick! Yaaaassss! :fire:


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (340 of 340 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 indentation thing as above!


I agree, it's not anything major, just a small observation (if we'd like to add "in milliseconds" view in the future).


Sen behöver man inte () .. om man inte vill för "getters" samt att man inte behöver skriva return om man inte vill, eftersom att i ruby returneras alltid det sista uttrycket i scopet.
Som i  sin tur resulterar i
```ruby
#Creates a new mail
def initialize(subject, body, to_address,from_address)
  @mail_subject=subject
  @mail_body=body
  @mail_to_address = to_address
  @mail_from_address = from_address
end

def body
  @mail_body.to_s
end

def subject
  @mail_subject.to_s
end

def category=(category)
  @mail_category = category
end

def category
  @mail_category
end

def mail_to_address
  @mail_to_address
end
```


@thet you're right ... I don't understand either, what's going on here ... respectively this commit 3178e194b9082fd0a76a479245b547079176cac9 obsoletes my `_expand_events_start_end` method ... but I have to take a closer look at this...


Лучше использовать символы, или, как минимум, расшифровать условие.


yes de är gjort nu


Damn that sucks. We've been trying to compile the latest crashdetect on our linux systems but we just can't get it to work. Issue #41 is actually with me. We'll keep trying. Thanks.


You're *so* killin' it! Congrats on your first code review! Everything looks great and it's all downhill from here!

Woo! :rainbow:


Correct. This is a mistake.


Also great commit message! :100:


Checking my db, isn't it syncProfile that has the hyphen? Bit risky of course assuming those are "owned" by caldav sync, but suppose that holds for now.


Sorry, I didn't think about final methods when making that change.


Вот это случайно убрал и теперь при переходе из комнаты в комнату свои сообщения перестают показываться. Надо вернуть.


Bryan + I had a chat and we'd like to keep them separate - they do different things.

That being said I've reduced the code duplication (moving some to doOperation, and the rest by changing the cancelling logic) which should make this a non-problem.

What do you think re: the new code?


Agree with @yanns, don't rely on global installs if you don't have to.


Good point - done!


s/locateed/located


@pfalcon Some relatively minor additions and we now support simple async/await as per PEP 492!


El makefile es el comando de compilación? Si es así habría que agregar las opciones:
-Wall (muestra todas las Warnings)
-ansi (toma el ANSI C estándar y no deja pasar los comentarios con // o las "variables on the fly")
-pedantic y -pedantic-errors (toma todas las Warnings cómo erorres)


I could not get this plugin working until fixing line 124 which is fixed in this pull request.


Does this need to be synchronous? Is any other action waiting on this file to be written?


Sen finns det även attr_accessor, attr_reader samt attr_writer
Vilket i sin tur gör att man kan förkorta din kod med
```ruby
attr_accessor :category
attr_reader   :body, :subject, :mail_body, :mail_to_address

#Creates a new mail
def initialize(subject, body, to_address,from_address)
  @mail_subject      = subject
  @mail_body         = body
  @mail_to_address   = to_address
  @mail_from_address = from_address
end
```


So much for my competence at copy-pasting small bits of text.  And I didn't know that one could use the less than operator on lists that way. Neat, thanks.

Well, this is fixed now, but that's just introduced me to another entertaining complication of the astropy test framework, about which you will momentarily see another message...


No intentional changes. I just copied the code to that method so I could write tests for it.


Yup - should do - as long as you've got valid data in the panel
Line 186 opens the door to the FEST tests when network is RED or AMBER. I havne't changed that


Ah - remember this one. This, and the other comments for this commit, are in place now and about to be pushed.


\cc @megawac. To avoid this in lodash I made its `_.now` test look [like this](https://github.com/lodash/lodash/blob/ce180cf2bff7fa44a83622c11c762d9a7b810107/test/test.js#L11205-L11221).


Trước/sau là trước/sau như thế nào? Ý chú là vị trí khi ghép ảnh hay là trong cái mảng ảnh mà chú lưu trong bộ nhớ?


done


Actually, I think my implementation is the best possible approach here. I pushed it further down to JUnitExecution.


ah, that’s much better!


@sergesemashko this looks great.


Вместо списка строк можно использовать ленивое считывание, используя то, что файл итерируемый. Можно воспользоваться функциями `iter` и `next`, чтобы получать следующую строку. Также можно разбить функцию на несколько, например, так (псевдокод):

```python
read_records(filename)
    for line in file:
        seq_name, seq, structure = read_record
                                       get_sequence_name,
                                       get_sequence,
                                       get_structure
```

или как-то иначе (может, и не получится нормально разбить, но попробовать стоит.


I am absolutely open to it!


You did not include the copyright notice. Try `mvn license:format`.


few days ago, the final


I can add a local `grunt` bash file that calls `./node_modules/.bin/grunt`


Not sure that we need this to be installed with `bower`, as it's only one file. Mind downloading this version and placing into `js/vendor` instead?


EclipseCon about, which provide


No it IS necessary because the code that picks the item up may run multiple times per tick!

I could confirm this with the watch by removing the pedestal - blacklist. So time torches or other mods, that don't blacklist the pedestal WILL trigger the duplication.


`stdin` нужно установить в `sp.PIPE`, в `communicate` нужно передать бинарную строку, как первый аргумент


Sen till sist har du faktumet att detta är en modell i databasen & i och med att den ärver av ActiveRecord::Base, resulterar i att du endast behöver definiera dina kolumner i databasen, vilket resulterar i  att attributen sätts på klassen när rails bootar.

Så detta är det enda du behöver:

```ruby
class Email < ActiveRecord::Base
end

email = Email.new(subject: "Test", body: ”<body>...</body>", mail_to_address: "test@test.se", mail_from_address: "from@test.se")
email.save #Sparar persisterar dina ändringar i databasen.
```


Putting reminder here that this would also need to be changed if we place jQuery in the vendor folder.