stdenv.lib.optional is a bit nicer to use instead of if stdenv.isCygwin then ... else null


I'm not sure - I think that would involve some encoding insanity.


This is currently not formatting correctly as rst, you probably want to make this a section instead:

```
print title
print '-' * len(title)
```


In short you should check it into the version control if you have a application, like a Rails website. If you have a library, like a Ruby gem, you should not check it into the version control.

Digging on google you can find more about this :)


I suspect this is not necessary since you have your view decorated with `POST`.  DRF will return a 405 (method not allowed) before your view code is even run.  Consider removing this and the spot where it is set on the server.


Thanks for reporting this I have credited you in the latest commit and changed the text to Duel from instead of duel request from. thanks.


Changed to fetch_Patient_Tracker_Events


I understand. Well, it looks good to me. Just please amend your commits so that they form a single one. @tritao, your turn.


Is it necessary to both return the errors and set them on the model attributes?


that's not true ;)


done


Don't forget to do your better idea, so that we still have a way of getting dates! :P


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


You can omit the `0 => ``in arrays (php does that automatically) and use foreach in stead of for.


Not sure if this is the best workflow.  Interested in people's perspective on whether the "Forgot" link should always be visible, not just after a failed login attempt.


It should not be needed to re-deploy the module


I think that this element should have a different ID since it accompanies a different button, the one from the `{form_html}` partial template. Therefore, I would suggest that this paragraph has `payment` for the ID, while the one from the [Line 153](https://github.com/edx/edx-platform/commit/f5b166f120c950f0290a6c166dceb756b7dc964e#diff-dfe065bc6d506bc0c00cd6635357985bR162) has something like `billing` for the ID.


cygwinConfigureEnableShared  = !stdenv.isCygwin;


Yep! Sorry about that 😊


Could you remove this line also?


preConfigure = stdenv.lib.optionalString ...


Will ammend the comments after Joao gives a clear sign. :)
On 27-May-2015 5:39 PM, "Dimitar Dobrev" <notifications@github.com> wrote:

> I understand. Well, it looks good to me. Just please amend your commits so
> that they form a single one. @tritao <https://github.com/tritao>, your
> turn.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/genuinelucifer/CppSharp/commit/19fbecbb7032b3f48a89fefeec749dd4c58148f1#commitcomment-11384148>
> .
>


请求权限


preConfigure = stdenv.lib.optionalString (stdenv.isCygwin) '' ... ''


Ну там есть и in-place операции: scale, fill, toBinary, еще какие-то. Если под каждую такую операцию мы будем заводить метод в самом общем интерфейсе, то так и распухнуть недолго.


python2 eller python3?


I've seen that comment as well and also ignored it. But when I made "make distcheck" work last week, this was one reason for failure. So while I don't know why, it looks like the 99 char limit is vital for "make distcheck".


done


preConfigure = stdenv.lib.optionalString (stdenv.isCygwin) ''...''


For some reason the end_vertex() information of some of the particles is unavailable in the failing RelVal. Added additional checks prior to the access of these functions to avoid the crash


do we need to default jenkins_cluster here?


better:

    define first_in_path
      $(or \
        $(firstword $(wildcard \
          $(foreach p,$(1),$(addsuffix /$(p),$(subst :, ,$(PATH)))) \
        )) \
      , $(error Need one of: $(1)) \
      )
    endef

    PYTHON = $(call first_in_path python2* python-2* python)

plus, you should substitute `$(PYTHON)` into the tar_scm.py shebang.


Looks good to me.  Except I'm going to miss the "Top Repositories" at home.


2, desværre!
Den 27/05/2015 kl. 14.14 skrev Oleksandr Shturmov <notifications@github.com<mailto:notifications@github.com>>:

python2 eller python3?

—
Reply to this email directly or view it on GitHub<https://github.com/christiankjaer/pop-webhelp/commit/ec2dea2d1e3613a6a6f924ac58f133e41abafa41#commitcomment-11384232>.


This is needs to be optimised. Its doing 2 database transactions per iteration. Should instead create a list of everything that needs to be inserted and do it as one big atomic transaction so nothing should be interrupted.

Will eventually change to something that uses this pattern
with db.atomic():
    Model.insert_many(data_source).execute()
https://peewee.readthedocs.org/en/latest/peewee/querying.html#bulk-inserts


Why was this removed?


@gibiansky The button shows, and clicks. Will work afterwards on making it respond to events.

![screenshot from 2015-05-27 16-54-27](https://cloud.githubusercontent.com/assets/5645741/7835705/894b940e-0498-11e5-8ff6-788eb080722c.png)


@emmenko I am not able to access the link u gave in the comment. I need to see how you get all the localised names


I can't see the comment TextView while typing.

![screenshot_2015-05-27-20-17-05](https://cloud.githubusercontent.com/assets/573807/7835709/8bb6c6a4-04ad-11e5-9471-a996db864feb.png)


Good point, let me check it here. I thought this was introduced by a change from @objectiser , but his change was isolated somewhere else on the same file.


Consider dropping the `ajax_` prefix for all these views and instead using the `request.is_ajax` inside of them.  I don't think we necessarily need to expose that implementation detail in the name.


changes again


just be glad you haven't seen the ENDIF . Changed


Of course right after I wrote this, the latest build passed just fine. But these two tests seem to fail randomly quite often anyway.


Good catch! I think that probably it should be changed such that `$data` is populated based upon the primary key configuration of the component model.


done


regarding the route / constant thing, didn't we do that in https://github.com/ezsystems/ezpublish-kernel/commit/1cec53413b6be61287d09d19fe222060e9aaee95?


In the future, general cleanup (which is good) is easier to evaluate when it's in a separate commit from the main feature being reviewed.


This check is redundant if you've already checked for null and you are just iterating over the list.


We probably could/should add sorting options to the repository list fragment (name/pushed/created); that would help with that. I also wonder whether we should store the last selected values (drawer position, repo ordering, etc.) in shared prefs, or whether we should add settings options for those?


edit to udacity version


You have a typo in the commit message.


Actually, we could probably simplify this with a `$('#user-modal').find('input').val('');` if they're all text boxes.  Mind the mixed quotes as well.


And good point, I'll check that.


Ok, thanks!


Wow c'est un très gros commit (trop gros en fait :)


Again, nice catch! No, the primary key value should not be hard-coded in this way but instead should use component configuration.


Consider changing the indentation/spacing here. I usually try and line up the verbs:

```
 this.model
    .fetch({
        method: 'POST',
        data: formData
     })
    .done(_.bind(this.handleSuccess, this))
    .fail(_.bind(this.handleFail, this));
```

Not sure what people's preference is and we don't have anything in the style guide, so I'm open to input.


Getting build errors after this http://pastebin.com/dcigLXkN


this assignment is not necessary


I see you've now added it in rev 2780a0102e675c36e12d6a08980d7d4b0726d5b4 thanks


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


After thinking about it, I think what you should do is have another file as the main, import this module and set it on the window, then use the re-export syntax: `export * from './simple-dom'`


This was my bad. I didn't know about `is_ajax` at the time and followed an old Django snippet. Agree with your suggestion.


done


WTH, I specifically remember typing the import line.

Aparently we're not allowed to use the `nova` cli.


Pourquoi faire simple en 100 lignes, quand on peut faire compliqué avec 25 fichiers et 2000 lignes...


Great!


`csvlint` throws a `1. inconsistent_values. Column: 12`, which I believe is due to <90% of `Å` values being the `-` null value, which doesn’t correspond to the current floats.


-Volar el filtro de enfasis de alta frecuencia. Me suma ruido en las altas. La imagen recuperada es de muy mala calidad.
-Me falta el threadhold después del sobel. Igual, voy a probar tmb el LoG para luego procesar por hough si fuese necesario. Después de esto tengo que aplicar morfología para borrarme los puntos indeciados que no aportan a las lineas.


Causes #64 @arekkas


Still not valid, should be $4.


What `Observable.observe()` returns should be kept somewhere so that it can be cleaned up as necessary.


Ok Issue solved.


It tells LaTeX to stop looking for more pieces to the number and to process the previous command.  Without it, TeX would keep looking for more thing that evaluate to a number and tack the on. Now, since the beginning of the next line is `\global`, which cannot be evaluated to a number, TeX would realize that the number is complete when it sees `\global` and process this line to the same result without the `\relax`.  However, it's not good practice to rely on that and to always put in the `\relax` to explicitly mark the end of the number.


Shit.
I'll look into it asap after I go home(~1hr)
On May 27, 2015 9:27 PM, "apascual89" <notifications@github.com> wrote:

> Getting build errors after this http://pastebin.com/dcigLXkN
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_libcore/commit/4d46de25bdb6d3b8150612a49849e251dcfcb035#commitcomment-11384440>
> .
>


The link doesn't matter, see the code I gave you. That's all you need


Were these changes supposed to be checked-in?


Not really.  We've already used it to loop until the control plane was both up and reporting the old host as down.


@dixonlo DWMR. Press on.


Extra space


lyy eidt orderActivity


does this need to be separated?


done


Extra?


Pinging @lizlux and @garthwebb

http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/15504/testReport/(root)/EmailIntegrationTest/testEmailImages/


Indentation comment applies here too.


And here.


do you mean ifdef?


Fixed the aforementioned issues in https://github.com/JetBrains/kotlin/commit/a8b1660e9e811206fce4e2793067ba2363cf3937


I think this should be a new variable, instead of using DEBUG


this thing breaks the whole thing.
./kiex: line 70: curl_with_args: command not found
chmod: cannot access ‘/home/user/.kiex/bin/kiex’: No such file or directory
Running initial environment setup
./kiex: line 75: /home/user/.kiex/bin/kiex: No such file or directory


Good point, done :)


Is there a spec for this method? Can't seem to find it anywhere


@opdenkamp Isn't the right logic just the other way round here? Clicking "Yes" ("Disable polling...?) should stop polling, not "No", right?


@fniephaus @matthias-springer


You know that 'ip' doesn't support IPv6 right?  I use not_analyzed strings for that reason.


No, it's been refactored to a different name now :)


Nope. It means it wants video buffers backed by *contig*uous memory.


will do


I was explicitly and repeatedly told the opposite, which was the major driver for converting the fence_agent to use nova's python API.


noice.


Wouldn't you be better off moving the plugins under a generic object and setting the properties on that, otherwise you're going to need a declaration for every plugin.


The previous had a space after the `1`, so is that good enough, or do you still need the `\relax`?  I suppose it's definitely more visible than the space, so it's definitely better this way, but just so I know.


You can change the logging level:

Mongo::Logger.logger.level = Logger::WARN


would like to optimize this line


Ah I see!


you should put enable it within the screencast controller.


I just feel less anxious with `1\relax ` than with `1 ` when it comes to unwanted spaces in the output, but functinnaly I think it's the same


Current behaviour
![screen shot 2015-05-27 at 3 27 51 pm](https://cloud.githubusercontent.com/assets/3919211/7836144/bdf88e98-049b-11e5-861e-f2e924cfd6ba.png)

In spark jira
![screen shot 2015-05-27 at 3 28 25 pm](https://cloud.githubusercontent.com/assets/3919211/7836153/ca8a5074-049b-11e5-88b5-740fdff171f4.png)

In YARN UI
![screen shot 2015-05-27 at 3 29 40 pm](https://cloud.githubusercontent.com/assets/3919211/7836161/dd184c8c-049b-11e5-838b-c1bcf0d9ee9b.png)


replacing dependency on Guava with dependency on Commons IO? isn't it one dependency for another? same problem in the end?


one space indentation missing


I agree with the sorting options.  Default is sorted by pushed and remember the selection if user change the option (or we can just fix the sort option to pushed?).

To store the last selected value like drawer position is just a nice to have feature I think.


RealUrl Autoconfig is a feature you will need to enable in EM.


You can't, really. That's one of the ones that actually needs the headers in the project. I removed json.h, json-forwards.h, and jsoncpp.cpp and put them in the zip. Don't think they need to be in there for linux, but if they do I can fix that.


sorry, that should've been `$(call first_in_path,python2* python-2* python)` (comma between the function name and the arguments)


maybe use `with preserve_sys_path(), suppress_stdout():` so we can reduce indentation in line 188-190 a bit making the whole thing a bit simpler?


```attachments.ToList()``` crashes


Can we remove ```::params``` since it's inherited anyway?


why the static bloc? One of HashSet's constructors takes a collection


Сега, проблема е дали `Array` отговаря на `#find_each` :D


they do need to be included for linux. I thougt it was silly that there is no package to install for jsoncpp - and that the recommended way is to include it the project. Like.. dependencies should be packaged upon build, not with the source :-(


Any particular reason you use an exec here instead of ```file```?


good point


you could also keep the immutability with `Collections.unmodifiableSet`


maybe its better to replace `coverage_error = $1` with `return $1` making this whole routine a lot simpler and if it returns false print the hint at the invocation point?


typo: est => test :D


Е, ще ги науча едно по едно :) Сега пускам update-че


Just that it's a member


오 벌써 해봤군요 굿


How/why are we commenting on this commit?   This file has changed a lot in the past year:
https://github.com/DeltaSystems/dewdrop/commits/master/Dewdrop/Fields/Listing.php


why not making the list unmodifiable anymore ?


Same here about removing ```::params```.


corollas: why not making this list a constant?


a


lots of modification in this commit seem to not have anything to do with removing the Guava dependency. I am wrong?


I didn't want to refactor as it's deprecated code, but that's not a good reason.


Judging from the signature of `execute` method, the `HttpClient` is meant to be an async client. If it's true, I personally would suggest for a `close` to return a future instead of `void`, otherwise I can't really call it an async client. This would also mean, that it can't really implement an `Cleseable` interface.

Alternatively, you just can add one extra method `close` which returns the future. I find it's important to have this non-blocking alternative, because in some scenarios (especially if you are writing event-loop-based application that uses non-blocking IO) it is important to at least have a possibility to close the client in a non-blocking fashion and not block an event-loop (for example, afair http://vertx.io/ and http://ratpack.io/ both are event-loop based and don't allow/discourage blocking calls in the event-loop thread).


Exploring different kinds of map charts in R


Existing code, but presumably the `mkdir -p`.  I don't think stdlib has anything to help here (e.g. splitting a file path) without us adding another dep.


The idea is to provide also tags in Spanish. For concatenation, you can use `RepoCategories.append("", "soundimage", "music", "música", "background", "música de fondo", "space", "espacio", "looping", "bucle", "fantasy", "fantasía");`


uh, yes :blush:


Fair enough


Please add docstring for this method


Perfect. Just asked about this on the [Q&A](https://community.appcelerator.com/topic/541/is-it-possible-to-add-multiple-versions-of-a-language-to-i18n/2)


Hello Tao, you know that you can customize the layout of the AppCoreKit controllers in the apps using the stylesheets or in the viewWillAppear method of your controller if you subclass it.

AppCoreKit provides a default layout but if you need to tweak it you don't need to modify it in the AppCoreKit.


no, there's a relation. CSS was loaded with Guava. And there's no need for CSS here, so I dropped the code instead of refactoring it.


Yes, it's calling one of vector_t<amp_t> (== std::vector<double>)'s constructors -- but only line 104 is. Lines 101-104 collectively are defining a constructor of block_t, the "{}" is just the body of the constructor

The stuff start by the ":" and ended by the "{}" is the "member initialization list" (C++ jargon), it's a list of member variable names and corresponding calls to their constructors, which can use either the paren/function-call notation or "uniform initialization syntax" (C++ jargon, C++11 feature) which uses curly brackets -- notice how block_size_ and num_channels_ use curly brackets and samples uses parens


Yes


extra line


try :

Xy = np.genfromtxt(GzipFile(fileobj=f), delimiter=',', dtype=object)


Removal of dependency on commons-* is not planned yet.


true


Hello! This is more of a style quirk, but there is one more small change that you could do. The new option is added to the process dictionary but it is not stored afterwards in the #options record, which leads to it being retrieved from the process dictionary at this line, unlike all other Dialyzer options. One could argue about this 'fear of the process dictionary', but it would be better if style is consistent, IMO.


Tao, I don't think we need to expose CKPropertyObjectViewController cause it is only used in the AppCoreKit debugger and I don't think people will ever use it :)


Hi Julio,

Based on my tests the fix doesn't change the existing records in the database, but upon updating the Permissions for a Site Team it adds the record to the 'resourcePermission' table with the correct data.
Please let me know if this answers your question.

Thank you,
László


I like this approach and topic. I would throw in a sentence or two regarding your view of Twitter before TIY and the front end design conference. Why didn't you use Twitter before TIY? How do you think Twitter will help you going forward? :+1:


Mmmm I wonder why would you like to place names and descriptions in separate files within the zip files. It makes it more complicated to edit/improve the descriptions. I'd suggest placing just the sounds into the zip files


didnt know about `unsetenv` . always used `(setenv "BUNDLE_GEMFILE")`


Fixed.


Faltou o "@GeneratedValue(strategy = GenerationType.IDENTITY)" na classe Curso
Faltou o "@OneToMany(fetch = FetchType.LAZY, mappedBy = "defesaAluno", targetEntity = DefesaComentario.class)" no atributo comentarios da classe DefesaAluno

No demais tá ok! :D


@richardkchapman Please merge and close


please do as we discuss this morning (remove the .toJSON)


No, not quite

http://en.cppreference.com/w/cpp/utility/functional/function

std::function is a class template that wraps a callable object, in this case the template argument is "block_v(boost::optional<args_v>)" which you interpreted correctly

kernel_t is not an anonymous function, though, it's a type -- I use "_t" to indicate a type throughout. The statement is a type alias ( http://en.cppreference.com/w/cpp/language/type_alias ) which is a statement that lets you alias a type :cat2:


Find ich schöner so die Sterne, wenns di garnich gefällt machs halt wieder raus ^^


I did wonder if we should have both, but wasn't sure.


Nice. Got it!


Weil "May." mit dem punkt blöd aussieht


please do as we discussed this morning (put back this.screencast.attributes.metadata.attributes.title)


Oops, it's not Italian


The thing is that jsoncpp is being used as an amalgamated source, not a lib or dll. If I can get one of those binaries for use with Electromedia, I'll switch from the amalgamated source files.


this.screencast.attributes.screens.models


OK, but just to clarify - this is arbitrary and it's not really something I think that should be kept once the sound is imported to a Mokap. It was done this way just because I needed a way to store the sound component within the entity


Yes Please! How did you find that?


A space after the `1` is not sufficient because there are TeX constructs which have spaces in them which evaluate to numbers.  As a result, TeX doesn't stop scanning for numbers when it encounters a space.


Yea, need to add for other pacmans. Alternatively we can just have "Acquire these dependencies with your package manager"...


This needs to have additional check to ensure that it's run from source - other operations require that trezorlib is present


wouldn't you just do ` $(package_manager_install_command ) libavcodec-dev libavformat-dev libswscale-dev libjsoncpp-dev` or something like that?


优化界面，但是还需要继续美工


Good start and I can tell this is going to be a helpful reference for those of us that struggled with Full Width Tabs. Early suggestion is maybe consider using 3 tabs instead of 5 to cut down on all the repetitive code examples? :+1:


Frankly, I'm not going to have an argument about why we should avoid introducing nils, but if someone wants to fix this one spot, then I will fix it properly later.


@deads2k @liggitt @ncdc all the cases below works and the Push/PullSecret is populated with Secret reference (\o/).
I'm getting no match for this one (dunno if it is valid image pull spec).


this is a declaration of a function called begin which is const and noexcept that returns a const reference to a block_v!

- a block_v is a boost::variant that i'm using to represent "blocks" i.e. of output
- references are pointers disguised so that they have value semantics
- since a reference to something is returned, you can specify the underlying type as const which means that the returned reference is read-only
- the function itself is declared const which means that it won't modify the state of the object that it's a member of
- it is also declared noexcept meaning that it is guaranteed not to throw an exception when it's called

this is just a function that you use to peek at this thing's block when you want to get values from it


I agree.  It should go before line 737.


> TODO: What about serialization? The file descriptor is hardly serializable, right? (See machine_save and machine_load in src/machine/machine.c)

Do you know what is the use case for serialization? Is it to be able to restart the process machined while containers are running? Or reboot the computer?

In the first case, maybe it's possible to store the fds in systemd's FDStore (see [FileDescriptorStoreMax](http://www.freedesktop.org/software/systemd/man/systemd.service.html#FileDescriptorStoreMax=))... but I would not waste time on this before getting feedback from systemd people.


What's going on here?


disallows that function from modifying the state of the object of which it is a member


I don't think it should be emitRestoreStack(stackDepth - 1). You have already subtracted 1.


Typo: PostreSQL


Actually there is no need to do that. MarionetteJS take care of destroying the previous view. Check documentation https://github.com/marionettejs/backbone.marionette/blob/master/docs/marionette.layoutview.md#destroying-a-layoutview


Do you have a `vagrant/settings_local.py` file in your vm? We have `settings_local.py` in our `.gitignore` so that may have prevented it getting into the repo. Or does the `copy: src: ` need to be `/home/vagrant/src/settings_local.py` ?


changed.


If there is no verificationcheckpoint against this attempt then skip all this process because this attempt verification was not about incourse reverification instead it was initial reverification


no -- there is no return statement so the return type of the lambda is automatically deduced to be void

the square brackets of a lambda allow for the author of the lambda to control what is closed over and how -- in this case, including this in the list of things in the square brackets explicitly indicates that the lambda is reaching into the object from which it is called -- in this case it's calling get_children_as_args, which is a member function of the enclosing object


Actually I just got a issue with this:
```
bundler.el:220:1:Warning: the function `unsetenv' is not known to be defined.
```
I think it should be `(setenv "BUNDLE_GEMFILE")`


I found it , it is set to auto...but the url of the blog post still have the same format..


Well, actually... it's kind of nice because then when you open the previewer you can actually listen to the sounds... why don't just we use initBehavior() instead of touchBehavior() for all the cases?


yes


The CreditRequirement table will be updated to use the reverification xblock usage_id instead of name. Then this method should have to be updated to use the new attributes


It's been moved


Yes, the "$binary"ies are there because that's how mongodict structures the database (since these are pickled python objects). When NAMD/pypln.backend#173 is implemented this JSON should be much cleaner. The idea behind having this pre-generated JSON is that it makes it much easier to test. We don't need the backend running (which should, in my opinion, be the case to run pypln.web tests) and the tests can rely on the test data that's here (even though it was much clearer what they were when we didn't store pickled objects).


Keep!


Did you mean 'canola'?


move
 _.extend(this, _.pick(options, ['screencast', 'screenId']));
this.model = this.screencast.model.getScreenById(this.screenId).get('note');

in the initialize function


... fair enough. I'll fix it.


@paulolieuthier
>SSInterpreter.exe "(comment \"gabriel
a testando\")"
not a function.
[]

> SSInterpreter.exe "(if (lt ((Number 5
):(Number 8):[])) then \"Verdade\" else \"Mentira\" )"
"Scheme" (line 1, column 20):
unexpected ":"
expecting space or ")"
[]

>SSInterpreter.exe "(if (lt ((Number 5
):(Number 8):[])) then \"Verdade\" else \"Mentira\" )"


Is this comment from the JACK documentation or is this your own observation?


I am not sure ... @jbrundage How come this line got changed?


Please open an issue for further discussion.


My mistake, of course there is no connection between prepare_size and total_size(


remove `updateView` function


Yes!


Good question. For my part, I was responding to Ceili's comment. Perhaps she has this version of Dewdrop installed somewhere?


Isn't `std::move` superfluous here?


Ok I got a bit deeper into the issue and found the following CMakeFile.error:

![image](https://cloud.githubusercontent.com/assets/329973/7836680/e3c4799e-0436-11e5-976d-d54bc801981b.png)


I'm considering doing it for the AppBarWidget too :confused:


@eventualbuddha Fixed in 0.6.5. Thanks


Tas no CIn?? To no G2.


Thank you @fccoelho . The fix is really pretty simple. I'm more worried about the impact this bug may have had in production. Any ideas about that?


http://www.boost.org/doc/libs/1_55_0/doc/html/boost/variant.html
http://www.boost.org/doc/libs/1_56_0/libs/optional/doc/html/optional/reference.html#boost_optional.reference.synopsis

what do you think, specifically, is happening?


you forgot a small semicolon at the end here, so
```int findLargestNum(int * array, int size);```


Add also SAP and BRMS


This breaks JavaProcessor in tdi-studio-se.


because you have 5 elements here


Both ... conclude


this now has to be of size 5, so
```void (*inPlaceSortingFunctions[4])(int *x, int n);```


Acabei de entrar para a aula de comunicação, mas tu vai ficar aqui até que horas hoje?


Maybe use a dot instead of comma


LGTM.

You may want to use cloud.get<whatever>() as I read the preferred pattern for @DataBoundSetters is to use accessors (for ease of compatibility).


I was trying to find a way to update the start at click without reload the treeview without success.
I found two things than could be useful:
* We already have a method that updates the dates column without reloading the treeview:
https://github.com/sugarlabs/sugar/blob/master/src/jarabe/journal/listview.py#L589

* We can disconnect the signal used to update the treeview, and connect after was modified doing something like this

```
     def _favorite_clicked_cb(self, cell, path):
         row = self._model[path]
         metadata = model.get(row[ListModel.COLUMN_UID])
         if not model.is_editable(metadata):
             return
         if metadata.get('keep', 0) == '1':
             metadata['keep'] = '0'
         else:
             metadata['keep'] = '1'
+        model.updated.disconnect(self.__model_updated_cb)
+        model.write(metadata, update_mtime=False,
+                    ready_callback=self.__reconect_updates_cb)
+
+    def __reconect_updates_cb(self, metadata, filepath, uid):
+        logging.error('__reconect_updates_cb')
+        model.updated.connect(self.__model_updated_cb)
```
But the problem is I can't make the two parts work together. If I add the code to disconnect the treeview, the star do not change color.

Maybe this is not possible at all. But is bad that we need reload all the treeview just for modify the metadata in a single row....


Only add "~" to CPACK version variable for rc builds on Linux. Tests are OK.
Another minor change: correct source file version. "${version}" is full version which is the same as "${CPACK_RPM_PACKAGE_VERSION}-${stagever}".


Also here, I think you wanted to write
```G - RadixSort```, as `F` is already 'occupied'


With these tiny changes, your algorithm does indeed work just fine


Add a dot to the end of the sentence. Also consider removing "the" in "the Business Rules design".


Now I'm doing testing, got it working thanks to your pointers here; sorry for taking so long, I've been visiting people this whole week.

* I'll try to add the nodes in there, thanks for showing me exactly where to add the code.
* `2^2 + 2 - 1` shows up as an `OperatorNode`, but is a constant (has a value of 5). So what `constTag` does is a DFS on the tree to see if there are any `SymbolNodes` that match the variable we are taking the derivative of. Because I want to avoid traversing subtrees a bunch of times I simply store its constant-ness in the map. Removing the map won't reduce the amount of code, so I don't think it makes much sense to remove it, but if the map access looks too ugly I can wrap that in a function. Also, there is a bug in this version of `constTag`, so don't look too hard at it just yet lol


Approach


Ok, I'll shorten it. :-)

--
Regards,
Janmejay

PS: Please blame the typos in this mail on my phone's uncivilized soft
keyboard sporting it's not-so-smart-assist technology.

On May 27, 2015 5:45 PM, "Rainer Gerhards" <notifications@github.com> wrote:

> I've seen that comment as well and also ignored it. But when I made "make
> distcheck" work last week, this was one reason for failure. So while I
> don't know why, it looks like the 99 char limit is vital for "make
> distcheck".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/janmejay/rsyslog/commit/9441bbd67f000625b8b4b9e7c697f1d7a70cf3d7#commitcomment-11384242>
> .
>


I'd love a space after the return keyword


if all were prefixed with catalystTypes, it would be nice to leafe them prefixed, in this case it adds clarity


the comment could be better or removed IMO


you could rename this method to "getName". This could be a patch on top of the whole stack.


2015-05-27 15:16 GMT+02:00 Janmejay Singh <notifications@github.com>:

> Ok, I'll shorten it. :-)
>
>
It's already merged. And as I wrote, this one seems to be short enough...
So no need to do anything.

Rainer

> --
> Regards,
> Janmejay
>
> PS: Please blame the typos in this mail on my phone's uncivilized soft
> keyboard sporting it's not-so-smart-assist technology.
>
> On May 27, 2015 5:45 PM, "Rainer Gerhards" <notifications@github.com>
> wrote:
>
> > I've seen that comment as well and also ignored it. But when I made "make
> > distcheck" work last week, this was one reason for failure. So while I
> > don't know why, it looks like the 99 char limit is vital for "make
> > distcheck".
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/janmejay/rsyslog/commit/9441bbd67f000625b8b4b9e7c697f1d7a70cf3d7#commitcomment-11384242
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/janmejay/rsyslog/commit/9441bbd67f000625b8b4b9e7c697f1d7a70cf3d7#commitcomment-11385301>
> .
>


what stands in the print statement is way better anyway :)


Wouldn't this be better in the docker-java project? Possibly as a replacement / extension for
com.github.dockerjava.api.model.Identifier ?


Aren't the ```delta_diagonal_rod_trim_*``` parameters really for tower_1, 2 and 3?
Since we use the X_AXIS to really mean TOWER_1  let's change things to reflect the that nomenclature and then equate them with a #define


My username is incorrect in release notes - correct username is @mrkd

Not a big deal - thanks for getting this in the release! :+1:


Absolutely!
On 27 May 2015 7:24 pm, "granitechief" <notifications@github.com> wrote:

> I have the SolarInfoBank API from Sungrow if you want it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kronicd/sungrow2pvoutput/commit/c89fa648cf7c3ef50e526e574d6a5594f94af214#commitcomment-11383377>
> .
>


this method can now be static


He modificado el script de instalacion:

#!/bin/bash
cd /home
sudo apt-get -y install transmission-daemon
sudo service transmission-daemon stop
sudo cp /etc/transmission-daemon/settings.json /etc/transmission-daemon/settings.json.copia
sed -i "s/$(head -n 24 /var/lib/transmission-daemon/info/settings.json | tail -1)/    \"incomplete-dir-enabled\": true,/g" /var/lib/transmission-daemon/info/settings.json
sed -i "s/$(head -n 24 /etc/transmission-daemon/settings.json | tail -1)/    \"incomplete-dir-enabled\": true,/g" /etc/transmission-daemon/settings.json

sed -i "s/$(head -n 52 /var/lib/transmission-daemon/info/settings.json | tail -1)/    \"rcp-whitelist\": \"0.0.0.0\",/g" /var/lib/transmission-daemon/info/settings.json
sed -i "s/$(head -n 52 /etc/transmission-daemon/settings.json | tail -1)/    \"rcp-whitelist\": \"0.0.0.0\",/g" /etc/transmission-daemon/settings.json

sed -i "s/$(head -n 53 /var/lib/transmission-daemon/info/settings.json | tail -1)/    \"rcp-whitelist-enabled\": false,/g" /var/lib/transmission-daemon/info/settings.json
sed -i "s/$(head -n 53 /etc/transmission-daemon/settings.json | tail -1)/    \"rcp-whitelist-enabled\": false,/g" /etc/transmission-daemon/settings.json

sed -i "s/$(head -n 55 /var/lib/transmission-daemon/info/settings.json | tail -1)/    \"script-torrent-done-enabled\": true,/g" /var/lib/transmission-daemon/info/settings.json
sed -i "s/$(head -n 55 /etc/transmission-daemon/settings.json | tail -1)/    \"script-torrent-done-enabled\": true,/g" /etc/transmission-daemon/settings.json

sed -i "s/$(head -n 56 /var/lib/transmission-daemon/info/settings.json | tail -1)/    \"script-torrent-done-filename\": \"/home/mensajetodo.sh\",/g" /var/lib/transmission-daemon/info/settings.json
sed -i "s/$(head -n 56 /etc/transmission-daemon/settings.json | tail -1)/    \"script-torrent-done-filename\": \"/home/mensajetodo.sh\",/g" /etc/transmission-daemon/settings.json

echo "USER=root" >> /etc/default/transmission-daemon

sudo chown -R pi:debian-transmission /var/lib/transmission-daemon/downloads
sudo chown -R root:debian-transmission /var/lib/transmission-daemon/downloads
sudo chmod -R 777 /var/lib/transmission-daemon/downloads

sudo service transmission-daemon restart
echo "INSTALACION COMPLETADA"
echo "Las descargas se guardaran en /var/lib/transmission-daemon/downloads"
cd /home/


you should round to the closest integer value to get the proper index for the image !
please  use iround from boost to get these values
http://www.boost.org/doc/libs/1_58_0/libs/math/doc/html/math_toolkit/rounding/round.html


your problem is on this line


@slentzen
Could you file a pull request? Thx


@mafagafogigante Do what you feel is best, and we can adjust afterwards based on feedback. :grinning:


indentation의 상태가?


hi fellow,
using @enc in you commit msg is including me in your project ;)

Have a nice day!


Unfortunately, * seems not to work, here (at least the way I pictured it working).  Various permutations such as

   1. `#*`
   2. `#"*"`
   3. `*`
   4. `"*"`
   5. `#modeling_distric*`
   6. `#modeling_distric"*"`

all fail to work.


I tend to like (guava) Optional<TagName>, as it means you can do nice compositions and it forces clients to acknowledge the not-existent possibility


se crea nueva tabla llamada "adultos" que contendra los datos de los padres y tutores:
 db.define_table('adulto',
                Field('apellido',label=T('Apellido')),
                Field('nombre',label=T('Nombre')),
                Field('nacionalidad',label=T('Nacionalidad')),
                Field('profesion',label=T('Profecion')), #profesion u ocupacion
                Field('cond_act',label=T('Condicion de Acticidad')), #condicion de actividad
                Field('nivel_inst',label=T('Nivel de instruccion')), #nivel de instruccion
                Field('completo'), #completo o incompleto
                Field('grado'), #hasta/grado
                Field('vive'), #¿vive?
                Field('tipo_doc'), #tipo de documento
                Field('nro_doc'), #numero de documento
                Field('estado_doc'), #estado del documento
                Field('calle'), #calle
                Field('nro_calle'), #numero de la calle
                Field('piso'), #piso
                Field('torre'), #torre
                Field('dpto'), #dpto
                Field('localidad'), #localidad
                Field('codigo_postal') #codigo postal
                )


@tmihalik I know. We just made that commit to update the server. It will be fixed shortly.

We had an issue with our CDN last night hence this commit


What is this for?


Eine Frage zu den Mephisto Projekten. V3 sind mehr als nur gelungen aus,
gibt es dazu, ausser auf Github, weitere Informationen, Anleitungen etc, zb Foren?
Würde das ganze gerne auf einem PI2 nachbauen.
Hier gabs leider bisher keine Antworten:
http://www.forum-raspberrypi.de/Thread-andere-mephisto-iii-internet-radio

Eine Mailadresse ist leider hier auch nicht zu finden...

Gruß aus dem Nord-Westen


`other_user` 안 쓰는 거 같으당


으아니 rubocop 찡이 왜 안알려줬지 ㅠ 이따 고침


we're in the process of deprecating it, we shouldn't depend on it being `.novalocal` in other cloud environments like GCE and AWS. For now the workaround is to add it in your group vars, we'll be moving it to Consul DNS shortly.


This is happening because the keyring is comparing the image `docker.io/...` against the index `index.docker.io` and failing to find a match. Here is some debugging output I added to basicKeyring's Lookup():

```
LOOKUP if image "docker.io/secret2/image" has prefix "index.docker.io/v1/"
LOOKUP if image "docker.io/secret2/image" has prefix "index.docker.io"
```


se realizan primeras pruebas de alta/insercion a la tabla "alumno" mediante la funcion SQLFORM.factory


Used to be item.remove() and that doesn't work in IE11. Use jQuery instead to accomplish this.


I've added an issue for this, see #363


문법 틀림 ㅎㅎ (다음 컨텍스트도)


Nice idea, but that matches `python2-config`.


루보찡이 안 알랴준 이유는 migration 같이 자동으로 만들어지는 파일들은 걸러내고 보라고 설정해놨기 때문이지


What happens to the rest of the MIDI events here?
Are they unreachable anyway and lost forever?

Or is it possible that a later MIDI event is valid?
IOW, would it make sense to call `continue` here?


@dan-f I admit it's a bit confusing to me. Will a truly-authenticated user be assigned both a user_id and an anonymous_user_id? Why?


They are two different msons, and each mson is put on it's own line. Right?


ack


you're right, i'll rework it.


isn't it cleaner to do `return (wholestuff) != 0`? More explicit.


Test


I am going to document in the help message that context directories are applied serially to the specified repositories (so that you have to take care not to specify as last repo the one you want to build from a context directory). Feels better than specifying one directory that can be applied to all repos...


also I think we can rely on python3 to the try except part is unneeded right?


unneeded


the previous typo fix was accidentally overwritten somehow.  Need to restore the typo fix in a2b988fcdd1c609e36eb5fec4a9dd1ddb1d0f6a2


My bad; I didn't catch last night that the file I was looking at wasn't at master.


The coverage module itself may throw an error is it doesn't have a dependency or it's not properly installed, hence I left it


If like you mentioned before we never actually relied of _id and not file name to interact with the files, I think the impact must have been negligible, if any. However the code is much more future proof in its current form.


This is just a snippet of example code. It is expected that 'App' in this case is a React Component. Please reference to example/ for full usage.


Why not? It is perfectly fine to use the cli tools. What I may agree is that we shouldn't base on the output from it (since the output may change), otherwise you'll have to use pure api call using curl or something.


La formattazione si intende con un editor di testo impostato a 80 colonne di larghezza e tabulazione a 4.


The 'the' seems necessary in my point of view


How about bumping the verbose level?


@gaudenz you commit broke the 4.3/5.0 builds. Please revert or fix the failing test.

http://jenkins.plone.org/job/plone-5.0-python-2.7/1251/


so how about:

```
coverage_available = False
with suppress_stdout():
    coverage_available = subprocess.call(...) == 0

if coverage_available is True:
    self.args.cover = False
    print(stuff)
```

Reason why I want that print stuff back in is because without the try except its not so complex anymore and its only one branch. (It was several before.) Then we can move out complexity from `run_tests`


This can be an `assert_raises`, so you can lose the `&& echo found` part


Great stuff.

Wouldn't it be a good time to add the `gnome.org` https enforce? Maintainers had tweaked the mirroring logic to have actual https mirrors and those even have priority over plaintext ones:
https://bugzilla.gnome.org/show_bug.cgi?id=749481#c3


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1149 of 1149 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)


> The coverage module itself may throw an error if it doesn't have a dependency or it's not properly installed, hence I left it

if it throws an error then we'll get non null return value, there's no way to pass an exception through a return value of a process is there?


Done


here you go :)  https://github.com/FluidTYPO3/fluidpages/pull/262


Impact only Agapé France


Hmm. True, I'll remove it


Ok I get it. we can change this later, it's not a blocker.


Had no idea it was that straightforward. Learned something new. Really nice, @croaky.


This helper is superfluous - clients can call `weave_on ... dns-add' instead


Agape France skin


Что будет, если написать `BEM.DOM`?


@SubJunk Bump :+1:  Should I post pull request for adding fps from MediaInfo for FFmpeg instead of using no params which forces internal FFmpeg detection which seems to be broken for some types of files?
Or it will be waste of time? Thx


Thanks @JackAdams.


You have to install the `libpam-dev` package (or equivalent for your distribution)


@TaoNhan I understand now why you tried to use this controller. It was not done for doing menu rows in tables but to display an image that is retained by an object's property.

To do menu rows, you should use the CKStandardContentViewController instead. They are plenty of helpers like
+ (instancetype)controllerWithTitle:(NSString*)title imageName:(NSString*)imageName action:(void(^)(CKStandardContentViewController* controller))action;


@TaoNhan  I will revert the two commits and we'll update the apps.


This can go too, as only `tell_dns` depends on it


Do we really need these forward declarations?


Test comment. :)


We special-case the fast paths (7-bit or UTF-8 encodings).  And where we fall short, we should probably optimize the methods anyway (e.g., `String#include?`).  We can wait until it's a bottleneck, of course.  But this special-handling of `toString` has never really sit well with me -- I think because there's not a guaranteed mapping from Java charset to Ruby encoding so there are currently some holes in the implementation.


This comment is premature optimization but as I wrote it I just do not feel like trashing it.
~~~ c++
auto res = mCriteria.emplace(name, CriterionWrapper(new CSelectionCriterion(name, logger)));
return res->second ? &*res.first : nullptr;
~~~
+ this code is changed in a following patch.


ack


Ok this was that dumb issue I brought up a while ago, I guess you're getting it too now.


Fair enough if we are talking about output of such commands (as I mentioned above). But how about changes in… nova API? If the API changes, nova client will change as well… and I really doubt that nova commandline utility will disappear or dramatically change.

Aside of that, why not utilize the work which is already there -  osp-ha-deploy/pcmk/nova_client.py written by Sylvain?


a `)` is missing


The comment for `assert_dns_record` should say `ips` as you can pass a list


`$@` needs quoting


this is a app


hmm...

I wouldn't call `log_dest_driver_init_method()` within `init_connection`.
This method is not for connection initialization, I think.


Superfluous - we don't do this anywhere else


`if (log_dest_driver_init_method(s) && afsocket_dd_setup_transport(self))`


This parser does not allow users to run dnf-automatic without config path argument. Hint: check out  `nargs` parameter of `add_argument` function.


@NikolaMandic What version of `bash` are you using? It seems that `bash` is not finding the function that's defined later in the file.


@ktakagaki
I can't figure out how to make this pull request use the latest commits. Instead I am checking out a "newest" branch and using that to create a new pull request.


You should also update `synopsis()`.


Same errors as above on my end too. Good work, though.


You set a default value for `opts.conf_path`, therefore it can't be `None`.


int in base 10 ? I am not sure to understand


Possible to package each type of data submission into objects? I.e. `Department` `Student` `Pay-Period` `Hours`


Describing in the class javadoc what a class is meant to be used for is helpful.


I'm just not sure we want people to think their settings got applied when they didn't.  This is a pretty rare option, as is specifying multiple git repos, so I'm less concerned about supporting the multiple case as I am about not having complex options to explain.

> On May 27, 2015, at 9:29 AM, Michail Kargakis <notifications@github.com> wrote:
>
> How about bumping the verbose level?
>
> —
> Reply to this email directly or view it on GitHub.
>


s/wether/whether/


An empty array is falsy in Python? How weird..

On Tue, May 26, 2015 at 2:08 AM, karl <notifications@github.com> wrote:

> not necessary to put len here. See
>
> >>> recent_datasets = []
> >>> if recent_datasets:
> ...     print 'foo'
> ... else:
> ...     print 'bar'
> ...
> bar
> >>> if len(recent_datasets):
> ...     print 'foo'
> ... else:
> ...     print 'bar'
> ...
> bar
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hallvors/compat-entomology/commit/85b70c91a542bbadf88d403c82a47f5cc2778077#commitcomment-11355435>
> .
>


I've tested this with the following 500 MB memory faker test tool and searched for 0:
````
#include <stdlib.h>
#include <stdio.h>
#include <unistd.h>

int main (void)
{
    long *data = (long *) calloc(1, 500 * 1024 * 1024);

    if (!data)
        return -1;
    while (1)
        sleep(1);
    return 0;
}
````
The result is exactly what I've expected from reading this code: The first **two dots** are printed immediately. But it is the same with before this patch. Might be the chance now to fix it.


Could you explain why you use `new` here instead of simply `Settings s;`?


Debug message should also include container name


Possible to update messages to be modular. Maybe taking the specific part of the message as parameter?


GNU bash, version 4.3.11(1)-release (x86_64-pc-linux-gnu)


@cburgdorf So you think it's worth testing this out? (due to it's nature we don't really know if it works until it lands in master :P)


should we remove this comment as well?


Having said that, the charset thing might be a bigger issue to be fixed in jcodings.  I'll follow up with lopex.


I think we should place sleep before metadata downloading(`fill_sack`).


Change format to `MM-DD-YYYY H:M:S`.


Please update also the following outdated paragraph:

>The operation of the tool is completely controlled by the configuration file and the command >only accepts single optional argument pointing to it. If no configuration file is passed from the >command line, ``/etc/dnf/automatic.conf`` is used.


maybe an enum could be useful here?


Done


Ok. Fixed.


about how a const or a define for both `900`?


Fixed.


Yes, sipa has an alternative version.

Benchmarking both has been on my TODO list for weeks now....


Prioritize `ID` before `name`?


Do we need department as an attribute of hours?


Is it important for this to be last?
And is it intentional that while swtiching order you switched from the pressgang xsl to the hibernate one?


We decided that Empty is fairly special concept that is only useful for ``Layout``. It contains no useful data (the label and group are useless as all Empty elements are identical).

``Empty`` is simply an identifier that means 'skip this space'. As a result it was decided that ``Empty`` shouldn't be an ``Element`` to make sure it doesn't nest into HoloMaps (for instance), to make sure it only affects layouts and to make sure it doesn't interact with the rest of HoloViews in surprising ways.


@Wackerbarth addressed in

https://github.com/BuzzBumbleBee/Marlin/commit/79795ae71337efeefd0a983692a9eb02d2f531d5

&

https://github.com/BuzzBumbleBee/Marlin/commit/6b574930a16da972db74987f997d48396fc46938


Prioritize `department` before dates.


I noticed that, was broken in a rebase with the current master

Just pushed the fixed version :)

On Wed, May 27, 2015 at 2:37 PM, Baptiste Lafontaine <
notifications@github.com> wrote:

> a ) is missing
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brettminnie/serializer/commit/96b7960142acb5a9d98a22eba48cd63d7bd4c837#commitcomment-11385716>
> .
>


@cableman EntityOperations already use RedirectDestinationTrait. This line leads to a strict warning (Strict warning: Drupal\views\Plugin\views\field\EntityOperations and Drupal\Core\Routing\RedirectDestinationTrait define the same property ($redirectDestination) in the composition of Drupal\koba_booking\Plugin\views\field\BookingOperations. This might be incompatible, to improve maintainability consider using accessor methods in traits instead.)


here's a `not` missing


Re-pushed.

On Wed, May 27, 2015 at 10:41 PM, Matt Booth <notifications@github.com>
wrote:

> Same errors as above on my end too. Good work, though.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_libcore/commit/4d46de25bdb6d3b8150612a49849e251dcfcb035#commitcomment-11385773>
> .
>


Start with ID number?


unack


I think there must be a better way to address the root of the issue, rather than just avoiding the panic.

We (I) should figure out why the process inside iop.proc is nil, and ensure we don't get to this point in that case.


While implemented and tested, this framework is not used yet, as it requires most of CBRAIN's documentation to be converted to Markdown


Possible to extract these into reusable methods if not updating to classes?


server_params = dict(
    label=ex_label,
    ...


I'm confused. you switched from pressgang to hibernate styles and now back again?


Good idea. I'll do that


Yeah in reference to https://github.com/dimagi/commcare-hq/commit/52160132828c6bf6f470a7aed3d759393fddb561#commitcomment-11305238


I know, but I splitted it in two lines because I think it is more readable this way...


does this need double quoting?


`selection = 0` to remain consistent with datatype.


minor stroke: wierd formatting


Possibly extract all messages?


Fixed.


+1 `TODO` text.


E assim?
````C
int obterVetor(ListaEnc* lista, int *v){
    if (lista == NULL)
        return ESTRUTURA_NAO_INICIALIZADA;
    if (estahVazia(lista))
        return ESTRUTURA_VAZIA;
        v = (int *) malloc(sizeof(int) * lista->tam);
        No *aux;
        aux = lista->inicio;
    for(int i=0; i < lista->tam; i++){
        v[i] = aux->item;
        aux = aux->prox;
    }
    return OK;
    }
````


seems like this leaks memory, no? Any reason to use `new` at all?


Global variable material?


rena ami me hacia lo mismo , no lo use, lo hice de nuevo digamos asi:

double A=1.3;//medio img_nueva=(A * img_denoised - img_denoised.get_convolve
(mask(6)));//alta potencia

El 27 de mayo de 2015, 9:28 a. m., Renato José Manzo<
notifications@github.com> escribió:

> -Volar el filtro de enfasis de alta frecuencia. Me suma ruido en las
> altas. La imagen recuperada es de muy mala calidad.
> -Me falta el threadhold después del sobel. Igual, voy a probar tmb el LoG
> para luego procesar por hough si fuese necesario. Después de esto tengo que
> aplicar morfología para borrarme los puntos indeciados que no aportan a las
> lineas.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lucianodato/pdi-fich2015/commit/5a692f9e18a336583a5d9fc4e3079b4426bdb600#commitcomment-11384470>
> .
>


ack


or:
```C
gboolean
afsocket_dd_init(LogPipe *s)
{
  AFSocketDestDriver *self = (AFSocketDestDriver *) s;

  if (!log_dest_driver_init_method(s) ||
      !afsocket_dd_setup_transport(self))
     return FALSE;

  afsocket_dd_try_connect(self);
  return TRUE;
}
```


Commit #1000 get.


Right, it was used for debugging. Removed.


Prioritize department ID over sm_period.


Please unlock in a `finally` block


k


Ups! Fixed...


`department_id` > `sm_period`.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1152 of 1152 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)


:heart:


Would be better to find the current version instead of hardcoding it. Should be easy to get from the MavenProject instance.


(any reason not to use `auto`, btw?)


ID > name.


is there no race condition that can happen due to this?


cleanup of useless shizzle.


length.


Actually looks like the version is not even needed since there is a plugin(String groupId, String artifactId) method.


DbSetup вроді як виглядає простіше, але він не розділяє код і данні, шо не єсть хорошо. З DBUnit'ом дійсно треба було деякий час погратись, але в принципі на данний момент для роботи з ним все що треба - редагувати xml файл. Як шановна громада вирішить.

Поділ проекту як він є зараз не дуже життєздатний, оскільки використовуючи пейдж-паттерн так функціонал не розділиш. Тим не менше поділ самих сторінок міг би бути і таким. Це приблизно і буде як в Олі.

Я спробую зараз остаточно змерджити все разом, але ви пропонуйте правки.


Another way to do this that's a bit cleaner?


Der commit scheint unvollständig / defekt. Hab diese Version in eine neue VM ausgecheckt und bestimmte variablen waren nicht definiert: z.B. windows aus dem main dialog.


:ABC:


Do you not want the `linux_headless_env` for the css tests?


:abc:


You can just close the client in another thread. AutoCloseable is the Java interface for a resource that needs to be closed.


use @extend %t-copy-sub1;


k


`finally` is only since php 5.5

Currently if an exception happens the unlock is done in the shutdown handler


develop (mjwitte) - 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)


Change format to `MM-DD-YYYY H:M:S`.


Updated as suggested:
- removed string concatenation
- use data instead of c_str in string_is functions
- updated tests and docs accordingly

Thanks!


Change format.


Ah, damn. Fine then.


Better way to handle departments? Might remedy itself once integrated into main DB.


Updated commit: https://github.com/joeshaw/gohub/commit/94dd29092d25195096a51c2e3ad581d40f5a962f


use @extend %t-strong;


The trailing comma on this seems to breaking nimble:
Error: unhandled exception: .../.nimble/packages.json(1, 0) Error: { expected [JsonParsingError]


remove comment


Neither am I ! I'll correct this at once :)


var or const ?


ack


Yes of course it would be better, didn't find the time to do it. The signature without version is probably taking the latest version which is not what we want.


Did you disagree with collapsing these two statements into one? In that event, the second one should still use `assert_raises` so you can lose the `&& echo found` and comparison to `"found"` at the end


Fix to "Set environment DB_USER=root"
[docker-gitlab] https://github.com/sameersbn/docker-gitlab
> DB_USER: The database database user. Defaults to root


so instead of `clone` now i am just assigning fallback table

it's not right way because if background and foreground are set they will overwrite fallback values

but i think such problem can be ignored


You probably meant isNotBlank() instead of isNoneBlank().


这是我做的第二次修改！


i think it's pretty weird case when `color15` is set but for example `color7` is not

but if u want i can check them all


You certainly can do this, but in my opinion the point of non-blocking IO is to avoid doing this. But anyways, it was just a small suggestion.


use baseline: ($baseline*.75)


>I noticed that you reverted the coding style fixes in basic_file_server_backend.hpp that I introduced yesterday, was that intentional?

No. - just pushed - the file with the lines outside the edit fixed


Thanks!


Ok, I'll do.


Well it was easy to test if the version is the current one or the last one, at least a TODO would have been nice.


Why the switch to hidden? Is there a chance that a long string will get cut off?


@NikolaMandic Interesting. It's newer than mine on OS X, where this works just fine. Could you try moving the `curl_with_args` function to the top before the first invocation of it and see if it fixes the issue for you?


What will be the difference between free and plus versions?


You're right.


@levicole DWMR. Looks good :+1: Press on!


use @extend %t-strong;


To allow a user to interact with your objects, you have to supply him(her) a simple interface. For that purpose, forms remain the best way of proceeding. This chapter really attacks(affects) the part(party) CRUD in itself.


Oh right, I hadn't even thought of that. #1430 right?

I'm thinking this might be a Haxe bug, but I couldn't really reproduce it in a smaller test case, so I just went with this workaround for now..


Just pushed it.


`$sectionEl` and `$inputEl` should be declared at the beginning of the function.  (They are global variables the way it is.)


learn from http://www.learnopengles.com/android-lesson-one-getting-started/


use @extend %t-regular;


Also with PHP 5.6.9 as far as I can tell from errors in my symfony project.


RTL?


@pejakm https://play.google.com/store/apps/details?id=acr.browser.barebones

>Free version is limited to ten tabs open at a time.


Thanks @sheknows ... this appears to fix https://github.com/sripathikrishnan/redis-rdb-tools/issues/57


is 겠지? 호호호 부끄럽네


> Furthermore the API is ill defined for what the crc of a directory would be.
> The spec says that for directories, CRC and size fields are undefined.

Ahh.. I think the Interface classes should have the documentation of the API and what is expected of the interface.

```
  /**
     * Backend for uavcan.protocol.file.GetInfo.
     * Implementation of this method is required.
     * On success the method must return zero.
     */
    virtual int16_t getInfo(const Path& path, uint64_t& out_crc64, uint32_t& out_size, EntryType& out_type) = 0;
```

The above should have better documentation so that an implementer is not forced to have to read the calling code all the way back to the DSDL and infer what needs to be implemented,


Just curious, what happens if this boolean is `true`?


this is not how you search a list


shadows builtin function


Meu exercício não tinha a questões pedindo para imprimir as cores.


@pejakm You can always get full version for free from https://github.com/anthonycr/Lightning-Browser/releases or just compile it on your own, of course.


Dann sieb dir mal ein Alumite T2 Werkzeug zusammen :) bei durchschnittlich 1/60 brauchts 2 Stacks per Barren macht bei nem VollAlumite 48 Stacks sieben ohne Reperaturmaterial


Misunderstood model. Soon we'll submit something right


I backed out the changes and am re-testing.


Ich weis nicht was falsch gelaufen ist. Ich weis ja noch nicht einmal wie ich das gemacht hab.


what happens if `a < 0`?


This is already fixed in newer versions of the ORM.


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


@cbeust
could you keep the pom.xml under the project root, as we can easily import testng into eclipse workspace by "File -> Import -> Maven -> Existing maven project into workspace"
M2E is installed on most of the EPP packages, while gradle project isn't, also i failed to import the testng gradle project into my workspace.


This one is also missing because it's in `.gitignore`. :neutral_face: I'm working past it by copying my local file into `provisioning/roles/kuma/files/vagrant` ... I'll update the main PR with anything else I run into after that.


Hi!

Thank you both for bringing some love to this issue. I've commit it initially in order to make the Sifo compatible with PHP 5.6 as fast as I could, and this issue was in "The ToDo list".

Good job @obokaman-com!


Thanks I'll try if I can use a later version.


Ok *I think* I understands what's going on here but it's pretty weird... add a small comment why this section is doing and otherwise LGTM :)


Opa! Uma duvida, como existe convenções para um monte de coisa tipo o  return    usar aspas simples existe convenções pra o uso de  console.log   e  return   ?
Porque fiquei  meio atordoado aonde usa console.log .  Dentro de funções é uma convenção usar   return    ?
Aonde devo usar console.log   ? Existe um padrão mais usado ?


Just to explain things a bit further here; the reason many people suggest having a "new line" at the end of the file is because of a difference between the Unix and Windows view on what the "new line character" is.

Windows uses `CRLF` as its new line character (actually two characters, Carriage Return and Line Feed) and interprets that as "start a new line here", whereas Unix uses LF as its new line character, and interprets it as "the end of the current line is here". So from Unix's perspective, non-empty files should always end with a LF, because the end of the file is also the end of a line. That's also why GitHub isn't showing a blank line on line 112 in the diff here. Going by the Unix philosophy, that line doesn't exist.


btw, I like this style way better than `whenValidatingPr`, very nice!


ok, at the end we will remove /wordpress/ refs to deploy to my server.


'raise' for 'raises'. Same for following tests.


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1152 of 1152 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)


main page 작성


this code looks right to do a post, but let's try this:

```
<%= link_to item_claims_path(item), method: :post, class: 'btn btn-success' do %>
```

I don't think you really need the Claim.new in there, because you want to post to the create method, and we can just do that in the controller. The only thing we need for a claim is the item id, which we'll have, and the current_user id, which we'll also have.


Many of these new geo terms are extraneous. For example,

`sichuan-wenjiang	China	Sichuan`

should already be caught by:

`Sichuan	China	Sichuan`. It greps to any occurrence of `sichuan` in the geo block of the strain name.

Same thing for `kyoto-c`, `kumamoto-c`, `stockholm20`, etc... When I run the pipeline I don't see these strains as being missed. Here are the currently missed strains for H3N2:

```
A/Guang-Xi-Xixiang-Tang/1698/2013
A/HaNam/Q187/2007
A/MIDCENTRAL/4/2013
A/Haryana/2396/2013
A/Haryana/1256/2013
A/Haryana/1396/2013
A/Haryana/707/2013
A/Haryana/706/2013
A/SouthCanterbury/37/2002
A/SouthCanterbury/433/2003
A/N.I/120940437/2012
A/ViaReal/SU5/2012
A/SantaCruz/694835/2014
A/Olt/161727/2014
A/Nonthanuri/456/2012
A/Van/354/2014
A/Olt/162753/2014
A/Bartin/15/201534
A/Ordu/1603/201428
A/Amasya/1539/201425
A/N.I/120280200/2012
A/Amasya/1478/201422
A/EntreRios755282/2011
A/Stockholm20/2012
A/Giresun/1346/201418
A/N.I/120940430/2012
A/HongKong2180/2010
A/Plateau/GR582/2012
A/Bouafle/GR621/2012
A/BLN/34/2012
A/BLN/52/2012
A/BLN/22/2012
A/RPF/32/2009
A/BLN/23/2012
A/BLN/28/2012
A/BLN/24/2012
A/Poitiers1071/2010
A/RPF/34/2009
A/Georgia_NHRC/33/2007
A/Lota/48166/2014
A/Georgia_NHRC/36/2007
A/RPF/27/2009
A/Nouackchott/1/2002
A/LyonChu/01375/2005
A/NakhonRatachasima/635/2004
A/WashingtonDC/AHERN-L/2003
A/Venezuala/VE-246971-ORIGINAL/2005
A/CHU/2-524/2005
A/Venezuala/VE-254155/2005
```


... That's a very good point.


@t0t ok


Wrong commit message.  This removed the 3 png files I had in the Wear apps assets folder, and added TODO's about that since the app will not function as is without them.


Sounds good.


@ustulation because routing v0.1.50 is not published yet, crates pulls routing v0.1.60.  Please keep this PR for later when v0.1.60 is ready


SNo.0 should use 3 "No." monsters of the same rank to xyz summon "normally".


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1660 of 1660 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)


Out of curiosity, why was this reverted? Doesn't this reopen https://github.com/chaijs/chai/issues/41?


这东西在format里能设置不？


That's one of the advantages of using WebStorm, as it detects stuff like
this. I suppose a decent linter should as well.
On Wed, May 27, 2015 at 6:10 AM Rich Harris <notifications@github.com>
wrote:

> @eventualbuddha <https://github.com/eventualbuddha> Fixed in 0.6.5. Thanks
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rollup/rollup/commit/9ff9d7a186b5ad4e0134735da73aee8cf0560739#commitcomment-11385208>
> .
>


Создание ManagedBean'ов -- прерогатива сервера web-приложений.


why?


Thanks - fixed.


    log.error("Can't prepare statement", e);
отлично справится с выводом стектрейса в лог


Tooltips updated in commit 3c08eb5.


aw, rats - was hoping that I could just have equivalent interfaces for the platform-specific shared memories. I suppose we can disable `xsi_shared_memory` for now, since it's not being used anyway.


где вызов
````
    resultSet.close()
````
?


__API Blueprint syntax error on line undefined__: ETIMEDOUT (Autogenerated message from apiary.io)


The alternative is to use https://github.com/hamcrest/hamcrest-junit

Longer term, JUnitLambda will provide a Lambda-based API for checking exceptions. If you can use Java 8 I highly recommend Lambda-based exception APIs over this class.


:smile:


Add a test case for @OperatorBucketTielTransformer@ and I'll merge.


Also, from man 7 socket,

        SO_REUSEPORT (since Linux 3.9)
                    Permits  multiple AF_INET or AF_INET6 sockets to be bound to an identical socket address.  This option must be set on each socket (including the first socket) prior to calling bind(2) on the socket.  To prevent port hijacking, all of the processes bind‐
                    ing to the same address must have the same effective UID.  This option can be employed with both TCP and UDP sockets.

So, as it says, it has to be before bind(2).


and this code isn't being used, so buh-bye it goes.


@athanatos shall we also sync object_map() in `FileStore::_set_global_replay_guard()`?

see https://github.com/ceph/ceph/blob/master/src/os/FileStore.cc#L2024


I think creating a replacement API is outside of the scope of this pull


Pedro


Cool thing! Thank you again for your help and see you at ECOOP!


ta vu ©a =)

Date: Wed, 27 May 2015 05:25:55 -0700
From: notifications@github.com
To: e-commerce@noreply.github.com
Subject: Re: [e-commerce] amélioration interface page accueil (1351ade)

Wow c'est un très gros commit (trop gros en fait :)

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


ugh - msvc is too permissive on some of these things. It used to be too strict on similar ones...


Why not use GetString from the cs50 library. Because I wrote this code without the appliance. If you are in CS50, use instead of scanf(); The usual GetString.


thanks, took care of this one too.


FlushProcessWriteBuffers can be called from multiple threads at the same time. We need a lock here to make the implementation thread safe, to prevent InterlockedIncrement from hitting PROT_NONE memory.


Sure. Looks good. I'm on a bicycle trip from Venice to Graz via Slovenia
and Croatia though...can't test it out ;)
Am 27.05.2015 15:42 schrieb "Ryman" <notifications@github.com>:

> @cburgdorf <https://github.com/cburgdorf> So you think it's worth testing
> this out? (due to it's nature we don't really know if it works until it
> lands in master :P)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ryman/nickel.rs/commit/6b7ef02551c445d78e0950a5ad00853340e347df#commitcomment-11385807>
> .
>


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1152 of 1152 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)


lol kinda, I guess
I'mma re check this for syntax errors


that's correct, it isn't. And this function apparently isn't being used, so I'm just going to remove it, under the "YAGNI" principle.


don't forgot to `.gitignore` this Makefile also =)


THIS is NOT a free solution but meant to guide you trough the problem set. IF you SUBMIT this code as your own work, all consequences considering your affiliation with CS50 as well as all consequences with the Ad Board and the staff of CS50 are the consequences of YOUR act. Please always keep this in mind and consider this to be like the postmortem. we advise: DO NOT COPY THIS CODE. CS50 IS A COURSE FOR YOU TO LEARN ABOUT COMPUTER SCIENCE AND NOT ABOUT HOW TO BE SMART AND COPY THIS OUT OF THE  INTERNET. Anyway this code will probably be added to the Academic honesty checker.


Shouldn't this be 12 ?


Yes, that's part of why it's unsafe, because you can easily escape references.
The conversion itself isn't the problem though, any method returning this has the same problem, it's the escaping from the local RefCounted that is problematic.


Ah, two-phase name lookup bites again - it's in the parent class.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1149 of 1149 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)


Like above: Shouldn't this be 12 ?


Oh yeah, true that :) I'll change it to be a child ;)


Again: I guess this should be 12


got it.


got it


An was man alles denken muß. Die Änderung war wohl zu radikal.


testing did not expose any problems.


One can easily get whick effect is activated by observing the type of the target.
Similar for 聖占術姫タロットレイ(94997874).


Needs updating http://serverspec.org/resource_types.html#file ?


@hajoeichler Your approach requires to fetch all existing products in order to know if generated slug is unique or check in real time if generated slug already exists.
@emmenko This approach will always generate ugly (with timestamp) slug for new product.

We have to support customers who will provide unique product names so we can have nice slugs. By default i would use slugified name only but with optional parameter ```--generateUniqueSlugs``` the importer would add timestamps for product creates.

In case customer does not provide unique slugs or names we need to ensure that product-syncer does not produce unnecessary update actions which will override existing slug+timestamp with slug+new timestamp.
In case ```--generateUniqueSlugs``` is used and slug information on new product is not available i would set old slug to new product object before sending it to the product-syncer.


any comment


Move these CouchDB client functions out to a separate script and make them easier to re-use.
API call for replicating the database:
```curl -X POST http://<host>:5984/_replicate -H "Content-Type: application/json" -d '{"source":"http://<remote_host>:5984/<db>", "target":"<db>"}'```


Reverted for now until this is resolved.


dont forget to remove whitespace


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (636 of 636 tests passed)

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


Ещё не работает


These 2 lines should be moved down after Stream_SetPosition


Couldn't you put this in `app/helpers/application_helper.rb`? That way it refactors some of the code right off the bat.


d


Yes, I wanted only one url pattern to map to the `home_page` name since I used it in one of the registration templates.


@marcotuts I'm not sure if this is the right way to fix the issue I've mentioned on Hipchat, but it fixes it on my devstack machine ... I will update sandbox instance to make sure it works there as well.

I would like to get your opinion about using `direction` mixin like this.


indentation is wrong here, I suggest you just wrap the `[` to the next line 4 spaces deeper than `coverage_available`


ok, added.


Hi, thank you for the code above, but my question is that can i be able to get airports by city?
for example if i type Johannesburg it must give me all the airports in JHB


See geoserver/geofence#49


Please squash this commit into the previous one.


fds


This looks fine to me


Got it!

We haven't relied on the correct time with pool interaction.
Let's push to `master` as well!

And start building M1 release.

Thank you!


И про parseOnwayResponse ты забыл, без нее кнопка "выехал" не выключиться.


Comment added.


Yup. Me too. :)


The API is similar to that used for personal names.  I decided that using `place.get_name` and `place.set_name` was preferable to `place.set_placename` and `place.set_placename`.

We don't allow `person.set_primary_name(first_name='John', surname='Smith')` either.  Not a bad idea though.


Typo in commit log: s/inifite/infinite/


Maybe wanted to be more specific about this being at the beginning of the line / before text ... but logic solid


Depends on how formal you get.  Undefined if you limit
output to the reals, complex otherwise.  The functions
assume that everything is in the appropriate domain
as they are typically already checked in the parts of the
library that call these functions.

On May 27, 2015, at 3:09 PM, Daniel Lee <notifications@github.com> wrote:

> what happens if a < 0?
>
> —
> Reply to this email directly or view it on GitHub.
>


ok, so what is the basename of the python-2.x frontend on Debian? Arch has `python$MAJ`, `python-$MAJ.$MIN`.


I like the cast differentiation here - nice touch.


I guess this method will be deprecated in 4.13. When it will be removed?


It’s a registered metatype. Can’t you use it properly then?


kewl


`Backbone.history.start` should load the correct controller action based on the initial URL. According to the docs:
> When called, if a route succeeds with a match for the current URL, Backbone.history.start() returns true. If no defined route matches the current URL, it returns false.


Thanks a lot!


Thanks! That's a good idea; it looks like most of the containers in `std::collections` use `usize`, I should do the same.


I'd not remove this overload; this would break binary compatibility


TeamCity xBIM :: Xbim Geometry :: Release [Build 75](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1756&buildTypeId=XBIM_XbimGeometry_Release) is now running


TeamCity xBIM :: Xbim Geometry :: Release [Build 74](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1755&buildTypeId=XBIM_XbimGeometry_Release) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 03:57:08


thanks


Looked into it too, not too much progress so far, would love to talk to you about it in private over Steam or something like that.


Une méthode pour poster rapidement des commentaires sur les pushs des gens ! OpenSource quand tu nous tiens !


Consider merging these 2 error types.


This line is causing issues for me, when db_field is specified as "table.attribute", because it falls back to "id" in this case and causing "ActiveRecord::StatementInvalid: Mysql2::Error: Column 'id' in field list is ambiguous" errors.


c'est vrai ca


Can you tag a release for 1.4.2 so I can install via bower


Moved it to config


Simply registering something doesn't make it work from python, how should python even recognize which type conversion to use...? It only means that if the type is decorated then it would be used.


OnAppNodeDriver.create_node mentions some other required parameters (`ex_required_ip_address_assignment` etc)


unit GB?


Yeah it was being spawned in the middle of nowhere for some reason.


I was asking what happens in code. It doesn't look like it throws an error.
If that's a precondition (one that the user should verify, but the code
will not), then we should label it as such.

On Wed, May 27, 2015 at 10:33 AM, Michael Betancourt <
notifications@github.com> wrote:

> Depends on how formal you get. Undefined if you limit
> output to the reals, complex otherwise. The functions
> assume that everything is in the appropriate domain
> as they are typically already checked in the parts of the
> library that call these functions.
>
>
> On May 27, 2015, at 3:09 PM, Daniel Lee <notifications@github.com> wrote:
>
> > what happens if a < 0?
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/stan/commit/cb6de77af4a37312a7565c1363e613d7aaeae4c9#commitcomment-11386935>
> .
>


Później funkcja getNextObstacleRow() zostanie ulepszona, ale już można z nią pracować.


ok I now understand what you mean! You were actually commenting about the use case when we're in a daemon thread and there's no request. Indeed for this use case, instead of trying to extract the context path from the URL it would be better to get it at webapp init time.

I have added some todo for now, thanks!


`node, = [n for n in driver.list_nodes() if n.id == id]`


value should be a json object


What it KeyError instead of IndexError? Then there is likely the same issue in other modules.


SMG was fine, there was no need to change this.

Also, this looks really weird on play.redeclipse.net and out of place, explode isn't particularly suitable for smg.


Inconsistent indenting with next line (Are you using Tabs? ).


@valorkin add some details please


@Tibor17 See https://github.com/junit-team/junit/wiki/Deprecation-Policy

My guess is we would remove it in JUnit 6


It's because in VBox we use a dict and not an array

On Wed, May 27, 2015 at 4:43 PM, Jeremy Grossmann <notifications@github.com>
wrote:

> What it KeyError instead of IndexError? Then there is likely the same
> issue in other modules.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GNS3/gns3-server/commit/7cad25eb1ad3d2f09a95c015f200c9becce0cdf2#commitcomment-11387144>
> .
>

--

*JULIEN DUPONCHELLE*
Software Engineer

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

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


This looks fine. We have been using StartDetached() elsewhere in src now also.


Argh got it, good catch ;)


*self.pedidos.pedir_sentido()* te devuelve el sentido en el que desea moverse, y no lo estas guardando!
debería ser algo
*sentido = self.pedidos.pedir_sentido()*


Any targeted release date for this commit? 3.0 is unusable to me without it.


Note that money should be 75000000 * 100000000 , i.e. in satoshis rather
than coins.


I cant edit files .class so how can i fix his error?


@bhoomit All the Redis commands should be supported according to the documentation. So there's no need to mention each of them. To quote:

"All of the Redis commands have their own methods with the same name except all in lower case.

You can find the complete list of Redis commands here:

http://redis.io/commands
"

See https://github.com/openresty/lua-resty-redis#methods


Cool! 我找了半天也没找到这个。


Perdón, esto se me re pasó.
No podes actualizar la posición del jugador "a mano" tenes que hacerlo mediante el método *siguiente* del tablero.
Lo que estás haciendo acá es decir que el jugador se va a parar a la posición N, donde N es la suma de las caras de los dados. Por ende el jugador nunca va a poder moverse por el tablero fuera del rango de la suma de dados. Para peor, le estarías pidiendo para nada el sentido en el cual quiere moverse D:

No importa, creo que mezclé demasiado ahí, deberías poner algo así
*self.pos = tablero.siguiente(sum(lista_de_dados), sentido)*

Notese que estoy usando la variable *sentido* que te comente más arriba


Great work, this is a highly requested feature.


I'm a confused by the indentation, which seems to be inconsistent, but may express nested elements.


el objetivo no es dinamico?


Time to change history


Esas validaciones de esNumero, esCadena, etc las sacaste del que subi anteriormente?


k


Oh, this is a general problem in MantidPlot. Most files have tabs in many places but mixed with spaces. In general I took the less intrusive approach, even though in some places I've fixed the indentation of small blocks of code. Fixing it would imply changing most files as a whole which is something that I'm trying to avoid, as it would become a huge change like the clang-format commits.


I don't think the swing back and forth is /necessarily/ harmful - the tempo component of eval does the same, after all. I guess the test will tell us..


Suggestions implemented.


s/test/tests/


indentation?


my first repogitory is on "GitHub".


Again, I would _name_ the systems this stands for, for example: Instant-runoff voting (IRV), alternative vote (AV), transferable vote, ranked choice voting, or preferential voting (see here, for example: http://en.wikipedia.org/wiki/Instant-runoff_voting )

This is currently under-specified and includes Borda count.


@taylor I'm pretty sure the function `curl_with_args()` needs to be near the top in this case. I tend to think that all the functions should be at the top, but that's a style issue, so I'll leave that up to you.


RCV has a specific meaning in US elections, which is IRV / STV (see previous comment).  I have never seen Borda count referred to as a special case of RCV.


all of these are extension parameters as well, i.e. `ex_`


I don't follow this test, I think you just want `-r ".eslintrc"`, right ?


Thanks for the help and support, we'll be seeing more commits with your recomendations soon


Indentation not consistent with next line.


maybe just put `kwargs.get('ex_' + p)` here


The new "finished" success messages seems to be:
```
PLAY RECAP ********************************************************************
developer-local            : ok=106  changed=16   unreachable=0    failed=0
```
?


Improving health does help, right?


Awesome, this will be a big win! @theVKC you rock!


See my previous comment: for the equal and even method of cumulative voting, this description is misleading because voters cast at most one vote for each of the candidates.  It's just that it is tabulated differently.


In the spirit of YAGNI I will comply, although I did see value in differentiating these cases. It should be pretty easy to re-split them in the future when we need.


Good point fixing this now. (and the Makefile.in)


Dear Social Team, please tell me if I'm right about this `endian` key and its translations. I assumed that it takes three possible string values: `"little"`, `"big"` and `"middle"`. @bkoval @kvas-damian @rafalkalinski


I forgot that commit can @ someone,  sorry is I disturbed your day.


This reads like n-of-m.


Hallo  noxx2015,

ich habe mich mal im Raspi Forum registriert und geantwortet. Falls Du Hilfe brauchst, melde dich einfach.


I get a load of deprecation warnings after this. It would be great to break this commit in to two:

1. Update Rspec
2. Add Guard

I think we should try to reduce the deprecation warnings before merging.


all extension arguments should be documented (if we're lucky we can get it merged without though)


done


Should add a little more context in the information text?

I'm also not so sure this is the right page to show, because eslint will be used later for l10n and for generic jshint-like checks too. So maybe we should just create a new file `build/eslint/README.md` that would explain that we want to move to eslint for some reason and then point to your MDN page.


I tried that and it did not help.  Other problems.

Working in a branch to add it back in... tested ;)


Sorry for breaking the build. AFAICS Jens already reverted my change.
Will commit a fixed test later.

Timo Stollenwerk <notifications@github.com> writes:

> @gaudenz you commit broke the 4.3/5.0 builds. Please revert or fix the failing test.
>
> http://jenkins.plone.org/job/plone-5.0-python-2.7/1251/
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/plone/borg.localrole/commit/342347d231f4ac34fa1a5d6910eda420711f6f65#commitcomment-11385536


please remove that, I have already tried to pull request it :)


Breaks the build, fix in https://github.com/aki017/slack-ruby-gem/pull/25


Looks like this behavior differs between SQLite and mongo.  Using mongo as the standard, I need to programmatically floor the `$mod` array arguments during normalization to ensure that the SQL engine and the matcher behave the same as mongo.


You don't need to do this if you are updating a product


Fixed.


Oh, yeah. good point. -.-


I would not mix it with the other promises. Actually it doesn't need to return a promise at all.


This is a little OT but I feel that you should not use @ to mention people unless their attention is required for the issue in question.


*nods*


Change the folder icon


I think, you should just findAllByVerifiedEmail in line 98, it would be useless to list unverified email.
And you can remove the uneccesary looping on line 99-103 then.


`$eslint_result` will be empty if `.eslintrc` is absent; I think you should initialize it to 0 before the `if` block that executes eslint.


Latest fix found here:

https://github.com/samseaver/KBaseFBAModeling/commit/d9bc396ecbadf6d95354850a8b1efe5b189a419c

I'm working on a script that uses a temporary biochemistry object to load compounds from the master file and balance the reactions in master, using the stoichiometry field, so its a good way to find bugs, and effectively means I'm balancing the reactions independently of either default of plantdefault workspace biochemistries, so it'll be easier to see how changes in the compounds affect reaction balancing.


good


Wow thanks for that @kdeloach! This will simplify the code a lot!


Could you please provide code comments for regex statements? It's too hard to understand this line at first look.


single quotes


I would be bold and join with a `/` here for easier copy-paste to the address bar :)


Are we trying to capture the experience as seen by the voter, the method used to tabulate the votes, or a combination of the two?


space


FYI removing this line broke shipit


...instead I would just ensure the slug here and modify the given product. The function also doesn't need to return anything.


No toquéis nada a partir de ahora, por favor. Lo he dejado en blanco.


Ah, okay, the first dot comes from the following line and is not part of the progress:
````
show_user("%02u/%02u searching %#10lx - %#10lx.", ++regnum,
````


Nice
![](http://i.giphy.com/IUZtGhVO8hZ6w.gif)


fw


Just kidding, fixed in https://github.com/DDMAL/RodanClient/commit/a921fe9b977bca07ea5c97a86453a72d87cc996e. After some consideration I'm sticking with this pattern for now.


I have tried to run this, and it can run okay.
I am a bit unsattisfied with the interface, can you use tabLayout on play-commons rather than providing link on the bottom of the page to manage email?
I think, the button Make Primary should not be displayed on primary email in Registered email list.
Please add language to display rather than user.primary_email on messages.en-US file.
Thanks, awaiting for the next commit soon.


Will this not stop executing the test on the first failed assertion?


All these enums are to mirror the ones in 1622.2 in order to encourage compatibility. We can hit up @johnpwack if we'd like to change the underlying values. For now I'm simply trying to provide the best possible guidance to VIP users given that constraint.


On peut imaginer élargir encore. Le flux atom de Diaspora envoie de sliens de la forme de la forme
`https://framasphere.org/p/728558`

http://zinc.mondediplo.net/messages/1214#message1235


Да, но только с этим кодом возникают ошибки

2015-05-27 2:59 GMT+05:00 Nick Reuter <notifications@github.com>:

> You can remove all that code and replace it with BOOT_ANIMATION_SIZE :=
> wvga
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Mirsaid02/android_device_htc_glacier/commit/43a9e1223c76cef5ceffbe539ef0f99b2395534c#commitcomment-11374204>
> .
>


If you want a unique timestamp just do it like this

```coffeescript
uniqueId = (prefix) ->
  _.uniqueId "#{prefix}#{new Date().getTime()}_"
```

See http://underscorejs.org/#uniqueId


Поэтому я внёс необходимые изменения

2015-05-27 20:03 GMT+05:00 Boom 338 <mirsaid2002@gmail.com>:

> Да, но только с этим кодом возникают ошибки
>
> 2015-05-27 2:59 GMT+05:00 Nick Reuter <notifications@github.com>:
>
>> You can remove all that code and replace it with BOOT_ANIMATION_SIZE :=
>> wvga
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Mirsaid02/android_device_htc_glacier/commit/43a9e1223c76cef5ceffbe539ef0f99b2395534c#commitcomment-11374204>
>> .
>>
>
>


This file seems to be expired. It can not convert the matlab model into the required xml file. For example, \<PCAcoeff\><\PCAcoeff> tag is missing.


options unused


i wish we could have figured out that `history` command thing @pbsugg :cry:


Had some time before a meeting / lunch -- question on how you're breaking up the Jade nonsense. Is this how you're breaking up the files? Just wanna clarify for moving forward.

Index -> page elements, text, etcetera
Layout -> JS, references, head, etcetera


You're missing a GLOCKFILE update for this commit.


Started reviewing. Thanks.


it looks silly to have 5, 10, 15 page size options when browsing through "hundreds or thousands of private vlans"


SMG and explosion?
I dont like it, and why the smg needs an explosion? I dont think so that it have explosive ammo :O :D


Ranked Choice Voting is the method voters use to fill out the ballot. IRV is collection of methods for tabulating a set of ranked choice ballots.

Again, @johnpwack probably has more insight as to why RCV and Borda ended up as separate values.


space


no newline!?! ahhh!


Good use of expectation methods. I would change the describe blocks to create a sentence.
describe "Vote" describe "validates" describe "value" so if there is an error the error message is more readable. "Vote validates value is included in -1 or 1"
I would also separate the invalid expectation into a separate example. So there would be two it statments.
"Vote validates value is not outside of -1 or 1"
But these are style issues, you have the basics down.


@jakobmulvad You're probably right about that, but lines would allow tunneling more than polygons do, if your objects move fast. I guess you could use the polygon approach, and split the polygon subshapes into own bodies? Might work :)


We talked about this before, and we decided to go with 1622's values instead of trying to come up with our own enums (e.g. separate enums for voter experience versus tabulation).  It seems they went with terms that describe the whole package.  Or at least that makes the most sense to me with what I see.  This is also consistent with allowing only one per contest, as opposed to mixing and matching.


oops!  thanks for the heads up!


add newline


Does the unit test still pass?


The code behavior is undefined (we definitely can’t
trust the results).  How do you want to doc that the
user should ensure proper values?

On May 27, 2015, at 3:42 PM, Daniel Lee <notifications@github.com> wrote:

> I was asking what happens in code. It doesn't look like it throws an error.
> If that's a precondition (one that the user should verify, but the code
> will not), then we should label it as such.
>
> On Wed, May 27, 2015 at 10:33 AM, Michael Betancourt <
> notifications@github.com> wrote:
>
> > Depends on how formal you get. Undefined if you limit
> > output to the reals, complex otherwise. The functions
> > assume that everything is in the appropriate domain
> > as they are typically already checked in the parts of the
> > library that call these functions.
> >
> >
> > On May 27, 2015, at 3:09 PM, Daniel Lee <notifications@github.com> wrote:
> >
> > > what happens if a < 0?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/stan-dev/stan/commit/cb6de77af4a37312a7565c1363e613d7aaeae4c9#commitcomment-11386935>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


Merging error, thanks!


Download and compile, i  will give a compiled version tomorrow i think , when i will back my pc x)


Blegh, I hate go package management. I'm getting the same behavior on my end.


We [avoid the use of `!!`](https://github.com/bbatsov/ruby-style-guide#no-bang-bang).


Please create a different pull request for this change


I merged the split makefile commit in the gliden64-gles31
https://github.com/mupen64plus-ae/mupen64plus-ae/commit/7b635666b26f6acc541b889cf686530c949da520
No segfault for me but I also had to move cpu_features inclusion macro to the end of the GLideN64 makefile.


remove space after {


Okey, thanks!


yup, it's incoming to make travis pass


Don't merge this commit into the master TurnKey repository. It is DHNow Specific.


@devoncarew PTAL


You can make it more secure by checking the file permissions of the file (much like the ssh client does with `.ssh/`)


do you need this console.log?


omg http://roboticpotato.com/spacecats-in-space/


-			if($player->getName() != "" and $player->isConnected()){
+			if($player->isOnline()){
 				$this->players[] = $player;
really nice =)


:+1:

Try to break your pull requests to self contained ones though ;)


@nutman in required reading there are a link to Martin Fowler "Feature Branching" Article, and as I said, it is pretty well illustrated there


I know that the editor should be in a separate package. However, you have not published any source code, and the git version is only tagged up to version 1.4.1.  Do we use version 1.4.1 as the editor version.


FYI - https://github.com/docker/machine/pull/1248


Should text =auto be there?


The doc is OK, no problem.

I was expecting 0 because it is the operator+ identity the more used.

```
0 + a = a
a + 0 = a
```


Sorry, I suggested the wrong change here - looking at what we've done elsewhere it should read
````
# assert_dns_record <host> <container> <name> [<ip> ...]
````


remove space before }


This is great! Wish I had known about this before — dealing with jcentral was not a whole lot of fun.


Can this `| tr` go, given the regexp?


> Again, @johnpwack probably has more insight as to why RCV and Borda ended up as separate values.

Okay, if that's the case, it seems very strange that we would have an enum value for Borda, which isn't used in any US government elections, and no enum value for IRV or STV, which are used in several.  That's another reason why it makes more sense that RCV would stand for IRV / STV.


remove space after { and before }


@clark-stevenson I think it makes sense to make them interfaces, since they are only exposed via mixing into another class.

If you make them interfaces is the compiler happy (without inline defs)?


use single quotes


wrong indentation


I don't consider either of the following to be an improvement:

`value ? true : false` (silly)

`value != false && !value.nil?` (crazy)


remove space line


@michael-n-cooper You might be aware of this, but in case not:  At the 11-Dec-2014 telecon, we  decided to remove the glossary entry for "value".  Rich added a note to that effect to ACTION-1520 [1], and James and I were tasked with removing links to the definition from the ARIA spec and Core-AAM respectively.  For example, see ACTION-1543 [2].

So why are we putting it back?

[1] https://www.w3.org/WAI/PF/Group/track/actions/1520
[2] https://www.w3.org/WAI/PF/Group/track/actions/1543