Ideally, you have a consistent naming convention here - i.e. both camel-case like "s3.addQuestion" for the widget and then "addQuestionID", or neither, i.e. "s3.addquestion" and "addquestionID".


4973_heatpump_defrost_bug (nmerket) - x86_64-MacOS-10.9-clang: OK (1674 of 1678 tests passed)

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


Can we handle this gracefully (i.e. not erroring out) or is the model index that crucial?


unused - can be removed


(Particularly because it's the _first_ line of the commit, so it appears like that in the shortlog.)


Blank between if and (


FYI, in case anyone else sees this issue... I ran into a situation where this was always returning a 0.  This was when loading the sample data block which then caused the smallGroupTopicType DefinedType to be null. Unable to figure out why it couldn't find the guid, I killed IIS and restarted debugging... then it magically worked fine.


Err - shouldn't refresh call _handleForm too? (e.g. if there is already a value for "type" if this is an update?)


@TheThirdOne Won't that cause a trailing space?


whoops


This is part of the domain layer actually.
All threading stuff here are abstractions. The main reason is to get out of the Android Main Thread as soon as possible (when the use case is executed) in order to not block it and get back to it when the job is done.


Why read the element and not this.data?


In this case I would agree.


switch/case?


Not quite. Just installing `pui-prism` won't work - it needs to be included in the project. In the project's main javascript file: `require('pui-prism')`.

For install instruction stories like this we should verify our setup instructions in a dummy app or something...


@dblandin Sounds great. In case you want a blueprint for easily matching the necessary API, you can look at this repo: https://github.com/jpodwys/cache-service-cache-module. This is the simple cache module I wrap into cache-service and superagent-cache so that there's a default configuration if the user doens't provide a custom cache instance.


or checking against your variable:

'.'.join((str(Gtk.MAJOR_VERSION), str(Gtk.MINOR_VERSION), str(Gtk.MICRO_VERSION))) < MIN_GTK_VERSION


I can't tell what's happening here...Did this get incorporated in the repo, but not via pull request?
How did that happen? @cfarm @virginiacc


Tested. Please merge.


There's a fun bug where `header.js.erb` only gets localized in english... so this won't have any effect (yet) :smile:.  But still good to have!


Remove this comment.


I'd rather we didn't do `puts` in the initialize method.


This method was only done this way for logging. I'm not presenting any information.


this->


Zfill? Lol 'S%02dE%02d' % (season, episode) is how I do it, less instructions.


typo


Shouldn't worry too much about keeping .apply() compatibility.  This is used for ALWAYS_EAGER and is pretty much useless for anything but as a safeguard from executing tasks


Caution: asynchronous event handling! Must prevent the default behaviour until the change-event on the type-field has been handled - otherwise collectData could be called after the form submission.


Nuked in DOCKER-477


I think expose(:projects) is enough, you don't need Project.all


unused variable


My tabs! :(


Handle Ace selection in a separate method, ideally.


test


EvansHVACObjects (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


That's definitely a better solution! :)


4973_heatpump_defrost_bug (nmerket) - i386-Windows-7-VisualStudio-12: OK (1680 of 1684 tests passed)

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


"is not contained"?


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


@OpenSAMOfficial I am curious and pretty interested. Which LaTeX template do you use ? I can't see any class or style file...


Paste error I think - fixed. Thanks!


:sunglasses:


typo


EvansHVACObjects (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


uh oh, I don't think you wanted to commit this.


Sorry...I did git push upstream master. I guess I should've just done git push and then put in a pull request? Or was it something else?


@nshi Check at this line, does the m_spBigBatchBeginToken get reset when running a small batch later? I think the test case covered that.


Is this something that clients can do, or will need to do, or is this something spark or the sdk does, and how is this accomplished?


mistake


what happens if it's passed by frontend as a boolean?


NUUUUUUUUUU


This catch block needs to be improved. We need to be especially careful with inputstreams and outputstreams, if an exception occurs we need to make sure that outputstreams are closed in the catch block, as well those close function can cause an exception itself which will also need to be handled.


there is no need to put a PANEL column in the tsv files when there is only 1 PANEL/facet to render.

about copying the data from common to chunk, you should be able to use the `group` and it should not matter if they are not the same size. See the original data viz on master to see what the data structure should look like after the copy. Maybe you should copy it before or after applying the d3.nest operator, I am not sure. Try both and pick whichever is easier or faster.

To wait for the common data to finish downloading, use something like

```
while(common data has not yet finished downloading){
 do nothing
}
then copy the data
```


Make sure you fix this during rebase, this was changed in `master` to say something else, and because `git` decided there were too many lines changed, it won't automatically get merged in during rebase


minor edit


@alPacino Great. Would you mind to send a PR? Or is this not finished yet?


it is in WaitForMultipleObjectsEx if one of the handle is a shared handle or a process handle


showDisplay used to show the player "display" (status icon):

https://github.com/jwplayer/jwplayer/blob/v6.12.5/src/js/html5/jwplayer.html5.view.js#L967

It used to pass false intentionally, so yeah don't pass it model.controls.

I don't think this helps anything on mobile since we achieve hiding of mobile controls through other means. Remove the whole thing.


Hello World, it looks good to me.


@nursix : Is it fine if I call collectData and handleForm only here instead of binding the change event? I really think updating the input on change is a lot of overhead(Since the real input field is anyway hidden)?


Cool code has trailing spaces.


> en fait contrairement à ce qu'explique @ralessi, le problème posée par shiftedpstart + plusieur \Pages n'est pas lié à l'emploi de \section, cf le nouveau MWE de #323

En fait, par "section", je voulais dire: "un environnement \begin{pages} \end{pages}" distinct d'un autre, au sens de ce que l'on trouve dans la documentation par exemple quand on parle de "numbered section" (voir le manuel, point 5.1.3, \doendnotesbysection).

Je me suis mal exprimé.


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

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


Yes
 On Jul 5, 2015 2:37 AM, "manaadacinta" <notifications@github.com> wrote:

> J
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/github/platform-samples/commit/c8b24ed66ec13b299a6066ee85455e458f743205#commitcomment-12004512>
> .
>


Tentativa de adaptar o sistema de utilizadores (fornecido como exemplo) ao nosso projecto.


EvansHVACObjects (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


@arichard4 , @peterhurford is setting the policy for now so can give the final position, but I think that pushing to master is indeed fine, for commits like this that you're relatively sure we want to submit to Trike. We try to keep it suitable to submit PRs to Trike directly from master, but obviously we can always edit master to make that possible, or create a temporary new branch to submit PRs from, like we did for https://github.com/tricycle/eaforum/pull/11


oui, c'est un bug terminologique originel de ledmac, d'utiliser "section" pour d�signer une section num�rot��. classique...


In the LUMI POG meeting this morning we discussed the dead module mask. After talking with them, for now I think it would be best to remove the mask. For now that is what is done for the offline validation plots. I discussed a little with @capalmer85, who's working on the offline lumi validation, the mask shouldn't make a big difference.


Is it just me but 'About' and 'Release Notes' tabs are using a serif font, looks like Times.
Only 'General' and 'Advanced' tabs are using sans-serif font, looks like Arial.

BTW "Using DNSCrypt" string, on the right of the green shield, is a bit cut at the bottom, "yp" needs few more pixels below.

I am on iMac 27" Retina 5K.


To match the geode conventions, this test should

1. End with JUnitTest.
2. Use Junit 4 syntax. That means don't extend TestCase. Instead, use @Test annotations. See any other JUnitTest
3. Have an @Category annotation of either UnitTest or IntegrationTest. This looks like an integration test. Again, see some of the other tests for examples.


I don't know about base prefix. Does that refer to the main site?


We should probably ship a socket file and mention it in the docs, making it less of an obstacle for the user.


:shipit:


Thanks


A container without a meta file is still a valid container.


Can you specify the difference in behavior to the old version of `\abs{}`?


4973_heatpump_defrost_bug (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1680 of 1684 tests passed)

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


Good idea mate


Yes. The commit just adds a debug statement instead of returning 404.


The PR looks good. Only recommendation is to make CLI a link

```
Be sure you've installed the [CLI](https://github.com/catalyzeio/catalyze-paas-cli#catalyze-paas-cli)
```


Que j'ai vraiment mis longtemps à comprendre!


I do ;-)


Sorry, I should have said that it doesn't really warrant the debug statement at all. :)


The notifyNewRevisionForManagedObject method should be called on the same thread as the context object already. I think we do not need to wrap the call in the performBlock.


Spellcheck, Reomve or Remove.


moi aussi !


`@:access(flixel.input.gamepad.id)`?


4973_heatpump_defrost_bug (nmerket) - 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)


Submitted upstream
https://review.openstack.org/#/c/201775/


Ok. I just removed the line.


@anarchivist++   thanks for this -- I was able to steal it for some SearchWorks linked data devel.  :-)


I am using this as my control flag to start the thread when send() is called for the first time.


Generally, the real input should be kept up-to-date with any changes. I don't think it's a lot of overhead - and since user input may alter the form, you must update after every change anyway.

Where the user input does not alter the form, you may add a delay (like 400ms) before executing collectData, so you do not unnecessarily collect data before the user has finished his input - but any input that either alters or submits the form must break through that delay and execute immediately.

Remember that if you introduce a delay, you must always collect all inputs - not just the one that triggered the change event ;) (because the user could switch to another input within the interval).

I would say that this is for the next iteration, however.


Hm, interessante.


Only square brackets are marked as `string` and thus need the escaping, the other two have different scopes.


Hj


The -performSelector method is using run loop and I'm not quite sure if this will work inside a dispatch queue. I think the original code doesn't work too when the managed context is configured with a dispatch queue.


var arrayPreferite = [];
arrayPreferite = JSON.parse(window.localStorage.getItem("preferite"));


The above three if...elif are related to blur_level. The url is not related. So add an empty line before line 34.


I think this should be `with the Subversion version control system`


Suggestion: rename blur_helper.r2py to location_blur_helper.r2py.


Just to be clear:  when we make a full offline analysis of pixel clusters we will veto bad (or inconsistently good) modules.  We anticipate needing that list in August when we have the Van der Meer scans and at the end of the year when we should re-do the analysis for the entire 2015 dataset.


reack


I tidied this up slightly differently. Instead of using log masking I copied the debug prints from one of the other files. Thanks for pointing this out though


:kiss:


triple-equals please


Cool :-)
Looking at that commit is much more fruitful than browsing and stumbling through code and documentation.

Collaboration rules! ;-)


*gold star*


I changed the comment to /* C style comments */


That's very clever.


C:\Users\Anton\PhpstormProjects\Symfony Training>composer selfupdate
Updating to version 92faf1c7a83a73794fb914a990be435e1df373ca.
    Downloading: 100%
Use composer self-update --rollback to return to version c36d2a2e501315934b6968e
fa3cea41dd27f2511
---------------------------------------------------------------------------------------------------------------------
Microsoft Windows [Versión 6.3.9600]
(c) 2013 Microsoft Corporation. Todos los derechos reservados.

C:\Users\Anton>C:\Users\Anton\PhpstormProjects\Symfony Training
"C:\Users\Anton\PhpstormProjects\Symfony" no se reconoce como un comando interno
 o externo,
programa o archivo por lotes ejecutable.

C:\Users\Anton>cd C:\Users\Anton\PhpstormProjects\Symfony Training

C:\Users\Anton\PhpstormProjects\Symfony Training>composer update
Loading composer repositories with package information
Updating dependencies (including require-dev)
  - Installing psr/log (1.0.0)
    Downloading: 100%

  - Installing twig/twig (v1.18.2)
    Downloading: 100%

  - Installing doctrine/lexer (v1.0.1)
    Downloading: 100%

  - Installing doctrine/annotations (v1.2.6)
    Downloading: 100%

  - Installing doctrine/collections (v1.3.0)
    Downloading: 100%

  - Installing doctrine/cache (v1.4.1)
    Downloading: 100%

  - Installing doctrine/inflector (v1.0.1)
    Downloading: 100%

  - Installing doctrine/common (v2.5.0)
    Downloading: 100%

  - Installing symfony/symfony (v2.3.31)
    Downloading: 100%

Writing lock file
Generating autoload files


NB when I say "delay", I do of course mean that the next change event would either re-start the timer (if it does not alter or submit the form), or remove it and execute immediately. But I think you get the idea.


@julianalucena @fltiago vocês querem dizer que ao invés de except: eu deveria usar only: ? Eu coloquei isso porque minha idéia era mais "O usuário não precisa de autorização para ver listas públicas ou criar listas. Qualquer outra coisa ele precisa"


This probably shouldn't be hardcoded.


window.localStorage.setItem( "preferite", JSON.stringify(arrayPreferite.push(stopData) ) )


Explain in comment what 1 means here. Or alternatively, implement different versions of get_file_from_server (get_city_file_from_server, get_zipcode_file_from_server, etc) in the helper code.


If this form is hidden from the taskbar, then the taskbar progressbar is useless...


merge dependencies into Seq()


good!


Initial


very long string composition. I'd extract some descriptive variables here (clean code)


@cagryInside Thanks!  I'll take a look.


Should this style work?
```js
Package.registerBuildPlugin({
  // ...
  use: ['isobuild:compiler-plugin@1.0.0']
  // ...
});
```
I guess I'm confused about the distinction between the two kinds of `use` directives…


yesss, way to go!


For this and other tests that include upstream/downstream setup of arrival times, it might be useful to include some simple ASCII art in the Javadocs to visualize exactly what is happening.  I think this makes it much simpler to understand, especially for newcomers to the code.


Thanks, correct!


no need for mutable lists. you can use

var myList = List.empty[Int]
myList = 1 :: myList
myList ::= 2

to get
myList == [2,1]


Yep - despite the weird Java soup of date stuff, this looks right. Thanks
:+1:


cool


Does logging support
```python
logging.debug("Thread %s started", self._THREAD_NAME)
```


Ha, I saw a bunch of I18n.t in there so I just assumed that it worked :p

On Tue, Jul 14, 2015 at 1:39 PM, Josh Lory <notifications@github.com> wrote:

> There's a fun bug where header.js.erb only gets localized in english...
> so this won't have any effect (yet) [image: :smile:]. But still good to
> have!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/code-dot-org/code-dot-org/commit/c49ffb7a924314d60d72e61c6dec1d707f570f0c#commitcomment-12165176>
> .
>


Improving WebGL: Maybe! Entities still have to be sorted for draw-order priorities;

WebGL itself supports a depth buffer that is supposed to solve these issues; e.g. a triangle with a depth closer to the camera can be drawn over other existing triangles, but a triangle farther away than existing triangles won't be drawn in the overlapping area.

Sadly, depth buffering does not help when the nearer triangles have semitransparent pixels. Triangles that need to be drawn behind them will not be blend with the existing frame buffer when drawn, because the depth buffer will clip the pixels that are farther away. Draw order is still important. ;) For this reason, we don't currently use depth buffering.

Since the entities are required to be presorted anyway, there isn't much gain to be had by sending depth information to the shaders (at least in the general case). Custom shaders can make use of the extra axis for things like perspective rendering.


Ps.: No only: eu teria que colocar o #destroy também, não apenas o #show.


Note that [the previous generation](http://aws.amazon.com/ec2/previous-generation/) of instance types are still supported so there should be values for these instance types as well


Can you explicitly assign the ephemeral disk mapping to a variable so that you don't implicitly return out of a control block? Also could you consolidate the knowledge around block device mapping, either by moving your additions to where `default_ephemeral_disk_mapping` is defined in `lib/cloud/aws/helpers.rb` or by removing that method and explicitly adding

```ruby
{ '/dev/sdb' => 'ephemeral0' }
```

to the `block_device_mapping` method. It also seems as if you can add the instance type to the `ephemeral_disk_options` hash if you'd like, rather than making the `block_device_mapping` method take two arguments. If you do end up removing the `default_ephemeral_disk_mapping` method you should also note that on the current generation of linux kernels the device will actually be `/dev/xvdb` so you may want to update the code to reflect this.


I'm not sold on the value of this test. It seems like it would be very brittle since we have no programmatic way to discover the instance store sizes of the various instance types. Especially since `InstanceStorageMap` is only used internally, I don't really see why it should be public at all. You may want to add tests for the various instance types by describing `create` and expecting that `create_aws_instance` is called with the block device mapping that you expect. I think that if you add an integration test which exercises the code path you should be ok, rather than testing that each hard coded value has been set.


Same here about the "shared example" pattern (i.e. def method), as in this PR: https://github.com/zhang-hua/bosh/commit/7663e1c1ae7c0d810739d27696e37d69625db239#diff-95a8ff626d595b3d3263b8dbf181690aR442

No need to merge all scenarios into one test method exactly. You can just duplicate the test code if that makes more sense for this case.


It would also be nice to add an integration test which verifies that the block device mapping is as expected when the requested ephemeral disk does and does not fit into the instance store for the virtual machine. Here you would only need to test one instance type with two ephemeral disk sizes.


Wrong branch again.
Why doesn't the pull request default to the branch selected previously?

Anyway it works.


Can we further refactor openfile, readat, and close into the helper code?


doesn't work. "you can't import a private type"


Whenever we're doing time unit conversions, we should use the Java [`TimeUnit` class](http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/TimeUnit.html) instead of doing the math ourselves.  This prevents mistakes in the conversion and also makes the code more readable.  See http://www.javaworld.com/article/2074114/core-java/the-highly-useful-java-timeunit-enum.html.  This applies to the tests and any code we're adding to OBA impl as well.


some explanation would be needed here (why 65536? what's that number?)


good! I see a lot of progress, congratulations :)


Oh! And what I meant to say in regard to changing the Renderable constructor signature:

We *need* to do this, but *only* for v3.0


Yeap, the value can change. The WIP demo at PR #20 might better answer your question, specially https://github.com/kborchers/react-globalize/pull/20/files#diff-7e9d5b1b710adf27b7bc0c42fb6539a6R70. See its [README.md](https://github.com/rxaviers/react-globalize/blob/pr-20/README.md) on how to run it. The development mode is working fine, the production mode is being implemented.


@davidlonjon @kasparsd here's another approach that @gedex did for another plugin which has a name that rhymes with “kitchen sink”:

```diff
diff --git a/php/class-widget-posts.php b/php/class-widget-posts.php
index 97b8a49..f4c613b 100644
--- a/php/class-widget-posts.php
+++ b/php/class-widget-posts.php
@@ -136,6 +136,7 @@ class Widget_Posts {
 		add_action( sprintf( 'manage_%s_posts_custom_column', static::INSTANCE_POST_TYPE ), array( $this, 'custom_column_row' ), 10, 2 );
 		add_filter( sprintf( 'manage_edit-%s_sortable_columns', static::INSTANCE_POST_TYPE ), array( $this, 'custom_sortable_column' ), 10, 2 );
 		add_action( 'admin_menu', array( $this, 'remove_add_new_submenu' ) );
+		add_filter( 'posts_where', array( $this, 'allow_search_by_post_name' ), 10, 2 );
 	}

 	/**
@@ -1104,4 +1105,57 @@ class Widget_Posts {
 		wp_cache_delete( $parsed_widget_id['id_base'], 'widget_instance_numbers' );
 	}

+	/**
+	 * Alter WHERE clause SQL when searching widget instances in admin screen.
+	 * This allows widget instances to be searched by widget ID.
+	 *
+	 * @filter posts_search
+	 *
+	 * @param string   $search Search SQL for WHERE clause
+	 * @param \WP_Query $query  The current WP_Query object
+	 *
+	 * @return string
+	 */
+	public function allow_search_by_post_name( $search, $query ) {
+		global $wpdb;
+
+		$is_unit_testing = function_exists( 'tests_add_filter' );
+		if ( ! is_admin() && ! $is_unit_testing ) {
+			return $search;
+		}
+
+		$s = $query->get( 's' );
+		if ( empty( $s ) || static::INSTANCE_POST_TYPE !== $query->get( 'post_type' ) ) {
+			return $search;
+		}
+
+		$condition = '';
+
+		$pattern = '/\b((?:';
+		$pattern .= join( '|', array_map(
+			function( $id_base ) {
+				return preg_quote( $id_base );
+			},
+			array_keys( $this->widget_objs )
+		) );
+		$pattern .= ')-\d+)\b/';
+
+		if ( preg_match_all( $pattern, $s, $matches ) ) {
+			$widget_ids = $matches[0];
+			$sql_in     = array_fill( 0, count( $widget_ids ), '%s' );
+			$sql_in     = implode( ', ', $sql_in );
+			$condition .= $wpdb->prepare( " OR ( $wpdb->posts.post_name IN ( $sql_in ) )", $widget_ids );
+		}
+
+		if ( ! empty( $condition ) ) {
+			// Remove last parenthesis.
+			$search = rtrim( $search, ' )' );
+			// Concat with widget ID condition.
+			$search .= $condition;
+			// Close last parenthesis.
+			$search .= ' )';
+		}
+
+		return $search;
+	}
 }
```


Thanks, just cleaned up this spaghetti code for changing the current directory.^^


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 140](https://teamcity.labkey.org/viewLog.html?buildId=260891&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 1 Build time: 00:03:58
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest.initializationError: <no details avaliable>

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

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

```


Submitted upstream
https://review.openstack.org/#/c/201783/


We should roll this back. And define this property in `phpunit.xml`.


HP no


I know that I downgraded, I tested locally and on stage before pushing this and looked ok. See also my comment in the commit https://github.com/glogiotatidis/snippets-service/commit/e94ee1f4cea4396c492d61cd1c127db197e0bb55

Do you have reasons to believe that this is not OK?

As far as upgrading to 1.8 I couldn't agree more but not possible in this timeframe.


By the way, I suggest to place your comments on the PR (not in the commits themselves) or they might get lost after a rebase.


nit: "not the empty string" was replaced with "non-empty." We use "the empty string" elsewhere, so it seems appropriate to use it here as well.


nit: container, media data, robustness
so that the content type values are together.


One means "a dependency of the package". One means "statically linked into the plugin inside the package".

The prior implementation required it to be linked into the package because of how it actually mutated `Plugin` to accomplish its goals. This one can just the simpler normal package `use`.  I guess we could technically make both of these work but it would be extra implementation complexity in order to give two ways to accomplish the same goal.


Isn't that for VS 2015?


It fails to read this because there is no UID field in the feed, not obvious how to fix that.


please remove the code completely if you think it's useless


(comments are bad for readability)


Pushing this out to fix crashes ASAP but need to revisit to determine why the main window returns 4 toolbar items when opening an individual file separate from a project (expected: 0).


Added the lat and long fields to the housing form edit.


"This configuration" should probably be "This combination" or just "<var>requested media capability</var>".


created a modal area, loaded on housing_forms.
It holds the div#housing-location-map which will display the map.


Awh, man. That's eventbrite being rubbish. And we have to have a uid,
otherwise we can't know the next time we pull the feed whether it's the
same event. That's dreadful. @mavit do they have alternate feeds, perhaps?
On 14 Jul 2015 10:24 pm, "Richard Cunningham" <notifications@github.com>
wrote:

> It fails to read this because there is no UID field in the feed, not
> obvious how to fix that.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BirminghamIO/calendar.birmingham.io/commit/672603dd914f923c50b823cad619915b0289004a#commitcomment-12166136>
> .
>


It is for VS 2015, 12.0 VS 2013 and 14.0 VS 2015


Adding @andergmartins . A new release might be needed. I'm not sure.


whoa, that's... something. ;)


I would call it "delta".


TODO: Add the libraries to the rails javascript.


FWIW you could use a C#6 property initializer here :)

http://odetocode.com/blogs/scott/archive/2014/08/04/c-6-0-features-part-i-property-initializers.aspx


I am not sure which is the best way to add the functionality to the buttons.
Should I add a listener here, or add in the button onclick="showMap(name, lat, long)"?


Cool.  I won't bother testing it as I assume you have.  I will include this in the next release then.


4973_heatpump_defrost_bug (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (652 of 652 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.45%25-green.svg)


Looks like this breaks it. I believe changing the capitalization back to all lower fixes it


Updated the wiki for this change


nitpicking: This will not block properly if millis() will overflow (happens every 3 weeks or so ;-)


This might be the source of my confusion -- in what scenario does this evaluate to true? In other words, when are value and v not equal?

This actually what confuses me about the use of `reduce()` above, since it seems like a complex way of assigning value->v and cell->fn. Am I missing something? :)

I'm obviously biased :) but what I proposed seems much simpler to read -- unless of course I'm not accounting for some use cases?


I kinda like this one better!


Don't you have to check call and construct signatures too? I would expect a weak type to have no call or construct signatures.


4973_heatpump_defrost_bug (nmerket) - Win64-Windows-7-VisualStudio-12: OK (1680 of 1684 tests passed)

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


This line looks so confusing until I remember that forEach means exists


LGTM


This has broken the build: https://travis-ci.org/google/shipshape/builds/70980676


Thanks! I think this functionally works for us -- but I am still confused about the cases being accounted for. Appreciate the quick response too.


I can see this causing confusion once 3.2 becomes stable. Maybe we go through the docs with a big find/replace as part of a minor release process? Alternatively, I think my wording is less ambiguous ("3.1 if the latest stable release is 3.1.x")


@matejonnet  Do you think it's okay to add a new build status to the SPI, I just wanted to confirm in case you didn't see that I did that.


Never mind the new code in src/Tokens.cc. Its for refactoring testing.


Here I created a button, and I add the information (needed by the method that show the map) in a data attribute as JSON.
This seemed to me the cleanest way to do this. I could also add qualifiers to other tags and find this data on JS.


so the option of not having to initialize this here and let DataDevice handle the things being done in DataOffer didn't work ?


I think we can avoid this patch.


можна менше 120 символів зіпкод


can you squash this with https://github.com/mcatanzaro/ozone-wayland/commit/4a06250c73a9a81b600ae0071b506b89be2adc20 as the approach of using DRWHClient here is not really correct.


source


ти ж здається хотів це в окрему модель?


Just curious, is there a reason for removing this? I tried to run init only once because it isn't foolproof with multiple threads - SectionLockerGlobal::Initialize() might cause issues if 1 thread calls it and another uses AcquireLock().


`plurals` has problems with Russian on some Android versions, but I am open to switching to them


If username/password use special chars this will be problematic. It's better to parse the URL without credentials and then set them by using `u.User = url.UserPass(...)`. Also see https://github.com/vmware/govmomi/pull/278.


навіщо це?


This needs to be sanitized. Not sure how to do that.


3 - винести в константу


що це за модель?


Nice one! Please use CamelCase for your variables.


Yup. Mailed you a fix.

On Tue, Jul 14, 2015 at 2:34 PM, Collin Winter <notifications@github.com>
wrote:

> This has broken the build:
> https://travis-ci.org/google/shipshape/builds/70980676
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/shipshape/commit/307c26c27aa649a4ce3947063c801b4ef76fa021#commitcomment-12166354>
> .
>


Nope, I simply forgot about this. :(  GitHub deleted all your comments when I force pushed over them, so I had to use the emails I got as a to-do list, and must have missed this.


IMO a more common approach is
```javascript
var SUPPORTED_EXTENSIONS = ['.mp3', '.wma', ... ];
if (SUPPORTED_EXTENSIONS.indexOf(extension) >= 0) { ... }
```
or even
```javascript
if (SUPPORTED_EXTENSIONS.indexOf(extension) = -1)
    fail('Extension not supported')
// ... do the job
```

This way you'll save one level of indent.


gotcha, thanks @pietern, totally forgot to check this :smile:


What is in the tarball when i submit a rootfs tarball?


https://github.com/artemsmirnov/sesh/pull/26 из за конфликтов смердженых вручную не могу понормальному скинуть ссылку на коммит


This is really nice! It is very similar to what I had in mind. There are some improvements we could add:

1. the adapter should tell this information instead of rescuing errors

2. we will support constraints in the changeset. so we would set:

      unique_constraint(changeset, :email)

3. we should change migrations to generate something like `table__field__index` for unique indexes (two `__`) since it is less likely to have false positives. We will compare the extracted field with one in the changeset constraint, if none is found, we raise a meaningful error message

4. we will also allow the index to be specified when defining the constraint. in such cases a direct match is required:

      unique_constraint(changeset, :email, index: :my_custom_index)


You want different syntax or what?


static


Thanks!

This reads kind of weird: https://github.com/batisteo/httptest/blob/b6ede8be52cbb6cd4aefcc6c62440369639c8777/README.md#4-mutation. I see two weirdnesses: 1) The indentation level is different than the subsequent code; 2) it's not clear what the relationship of these two code blocks is. Maybe there's some other way to express that the `Mutex` and `Arc` imports are needed, like just in prose before or after the snippet?


@jiahao did you test this locally? it's broken on travis


get_database_filename and load_city_geodata do not belong to find_closest_city (their functions are different). Suggestion: (1) move filename = blur_helper.get_database_filename(lat, lon) into load_city_geodata; (2) move city_dict = load_city_geodata(filename) in the main function get_city_location. So it looks like this:

def get_city_location():
  location_data = get_location()
  city_dict = load_city_geodata (location_data["latitude"], location_data["longitude"])
  closest_city = find_closest_city(location_data["latitude"], location_data["longitude"], city_dict)

  location_data["latitude"] = closest_city["latitude"]
  location_data["longitude"] = closest_city["longitude"]

  return location_data


EvansHVACObjects (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Some `scope` issues. Here is a start of it somewhat working.
```
var Camera = famous.components.Camera;

var math = famous.math;
var physics = famous.physics;
var PhysicsEngine = famous.physics.PhysicsEngine;
var Box = physics.Box;
var Vec3 = math.Vec3;
var Size = famous.components.Size;
var Position = famous.components.Position;
var MountPoint = famous.components.MountPoint;
var Gravity1D = physics.Gravity1D;
var Collision = physics.Collision;
var Wall = physics.Wall;

var DOMElement = famous.domRenderables.DOMElement;
var FamousEngine = famous.core.FamousEngine;

//Physics engine test
function Test() {

  //create scene graph
  this.scene = FamousEngine.createScene('body');

  this.camera = new Camera(this.scene);
  this.camera.setDepth(1000);

  this.simulation = new PhysicsEngine();

  this.node = this.scene.addChild();
  this.node
      .setSizeMode('absolute', 'absolute', 'absolute')
      .setAbsoluteSize(100, 100)
      .setPosition(0, 70, 0)
      .setMountPoint(0.5, 0.5);

  var position = new Position(this.node);
  this.myBox = createBox.call(this, this.node, position);
  console.log('this.myBox', this.myBox);

  FamousEngine.requestUpdateOnNextTick(this);
  console.log('test');
}

Test.prototype.onUpdate = function(time){
  this.simulation.update(time);

  var itemPosition = this.myBox.getPosition();
  //console.log(itemPosition.x, itemPosition.y, itemPosition.z);
  this.node.setPosition(itemPosition.x,itemPosition.y,itemPosition.z);
  FamousEngine.requestUpdateOnNextTick(this);
};

//add node - this node will be static
//create box
function createBox(node, position) {
  //attach a DOM element component to the staticNode
  this.blueDIV = new DOMElement(node, {
    properties:{
      'background-color':'#49afeb'
    }
  });
  var mb = new Box({
    mass: 10,
    size: [100, 100, 100],
    position: new Vec3(window.innerWidth / 2, window.innerHeight / 2, 0)
  });

  this.gravity = new Gravity1D(mb, {direction: Gravity1D.DOWN, strength: 300});

  this.floor = new Wall({direction: Wall.UP, friction: 0});
  this.floor.setPosition(0,window.innerHeight, 0);

  this.collision = new Collision([mb, this.floor]);

  this.simulation.add([mb, this.gravity, this.collision]);
  console.log("hey");

  return mb;
}

FamousEngine.init();

var test = new Test();
```


no, very nice man :+1:


Not thrilled about userAgent inspection for features.


Is the `nullEverywherePartitioning` part of the variable name relevant? How about we just name it `nullChannel`?


`and != '/stories/index.html'`

That's why you should create a pull request and let others review your code …


Sure; that's a lot easier than completely removing those changes from the history. :)


@jleclanche I don't understand the regression you said. Hitting Ctrl+Z anytime will move vim to background in Konsole as well.


It worked for me on 10.11


@sylvanc Out of curiosity, what guarantees that `sizeof(int) == 32` here?


The problem code is actually `GAMSVR`. Could you please rename the `.in`/`.out` files, the `.java` file, and the folder so that your solution could be tested automatically?


this must have been tedious


shouldn't this be in an `else` block?


rename to `NullPartitioning`


This needs to be sanitized. Not sure how to do that. Is it ok if I import JIRATask from jira_shared.py to alm_plugin_base.py to do it?


Typo: "install"


name this `replicateNulls`


The problem code is actually `LNGPOK`. Could you please rename the `.in`/`.out` files, the `.java` file, and the folder so that your solution could be tested automatically?


## This is a markdown file


I added active seednodes and removed all the old inactive ones.


"replicate nulls" instead of " null replicate"


Do we need to alias `lower` and `mode`? Why not just leave `opts.lowercase` for backward compatibility. If `node-slug` handles them we should let it do so, and deprecate the `lower` option in `metalsmith-slug` altogether. And the user can still override them by passing in the keys normally.


replicateNulls


hashPartitionedWithReplicatedNulls


@robpaveza, would not this be an issue in terms of power consumption?


nit: Can I update probabilities? How?


Thanks.  Looks like everything is green now.


I guess this is how?
Example like the one above would be super helpful.


Thanks for the prompt response!


Jap, das stimmt. Ich werde morgen mal noch Kommentare einfügen. Manche Methoden muss ich auch noch umbenennen, die haben durch mehrmaliges Verändern eine Namensänderung verdient.


Yeah, I can add another example


I have been dabbling in the world of android rooting and custom ROMs.  So I figured it was time to graduate and start my own project.  I am very interested in learning to work with android/computers.


I'd call this `enforceCollocatedTableScan`, 'requireCollocatedTableScan' or something like that.


Thanks Vladimir!
I will rework this via `indexOf`.


Some examples were 'person' when it looks like they should have been 'user'.


Left in the "!" for testing.


I'm not sure that g-d-p can approach being the user-friendly tool you want it to be while still operating as a noninteractive/batch tool... this sort of "do what I mean" seems like it would be a much better fit in a GUI or TUI tool. Unfortunately, turning a batch tool into a decent GUI or TUI means making it asynchronous, which is a big design change.


Ich denke die Methode `getNeighbours` lässt sich stark vereinfachen, wenn man so etwas macht wie:

    List<HeightedPoint> l = new ArrayList(9);
    if (y != 0) {l.add(d.getHeightedPoint(x - 1, y);) }
    if (y != 0 && x != 0) {l.add(d.getHeightedPoint(x - 1, y - 1);) }
    // ...
    for (HeightedPoint hp : l) {
    /// ...


@cressie176 why you can't just accept the pull request?
it' simplest to you and i'll be happy seen my name as a Yadda's contributor ?


I just love 13 too much. No love for 20.


@aashish24 I believe this fixes it for tempus' use case.


Guess I forgot that finally didn't catch errors. This seems fine.


I'd personally rather just use branches + PRs for everything, but I can work with this. :)


It would make more sense to me if we only checked for engines if we are actually going to `dpkg -i` the .deb immediately. If we're just building a .deb for later consumption, and it turns out to be uninstallable, maybe that's OK: it might be intended for installation on a machine with a different architecture or set of apt sources.


EvansHVACObjects (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Thanks for the feedback!

> the adapter should tell this information instead of rescuing errors

as I mentioned in the git message, I was pretty embarrassed about some of that code, so this goes without saying :-)

Could you expand how you envision `unique_constraint` being used?

Would it be used similarly to how validations are currently used? e.g.:

```elixir
cast(user, params, ~w(name, email), ~w())
|> validate_length(name, min: 4)
|> unique_constraint(:email)
```

or something entirely different? If that would be the case I suppose it would be a little bit un-DRY (since schema already specifies that constraint) but perhaps more importantly the handling of constraints only happens when we actually insert/update/delete and so I think it's pretty different than validations.


I am getting the following error, with the code I downloaded today (20150715)
This happens when I try and get an item, eg: /api/model/id/

`  File "xxx/lib/python3.4/site-packages/rest_framework/viewsets.py", line 79, in view
    handler = getattr(self, action)
AttributeError: 'xxxViewSet' object has no attribute 'metadata'`


@eduardozulian estou farto de choro de nego dizendo que comunidade é um ou é outro.
Então se já entendesse como comunidade, então que pensem em todos ao mesmo tempo xD


I'm tempted to revert it to make Travis happy, if the only thing it was fixing was a deprecation warning


EvansHVACObjects (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


I think for consistency if nothing less this should be  `method = this.get('blogCreated') ? 'PUT' : 'POST'`


Could you not call `systemctl enable` to do this?


Thanks for responding @meandavejustice (and for your killer talk on Web Audio)!
I'm slowly gathering notes and still haven't written up some of the latter ones, but I appreciate you putting your links up.


> Would it be used similarly to how validations are currently used? e.g.:

Yes, the rationale is that you still want to customize indexes and error messages, so we do need to provide a place where those would be handled and converted to errors.

>  but perhaps more importantly the handling of constraints only happens when we actually insert/update/delete and so I think it's pretty different than validations.

Right, *exactly*. That's exactly why they have a different name (constraint) and why we have deprecated the old Repo.insert. So we can add this new one in its place. :)


This needs to be sanitized. Not sure how to do that. Is it ok if I import JIRATask from jira_shared.py to alm_plugin_base.py to do it?


Die Playlist ist ne sehr geile idee :D


Agreed. So, in an inline somewhere?

(Thank you for your work!)


Gabriel, não entendi esta alteração. Pode me explicar?


Update master with 2015 branch


:+1:

Could also do `!matcher.call(el, "[readonly]")`, obviously - not sure whether you prefer to invert in JS or CSS :P


Can these two lines be combined into two by removing the "zone" variable?


@vram4 I see, these lines of code are used to block other clients.


personally I like the could extra characters for the readability of it, easy to miss the bang, but :not([readonly]) is pretty clear about what it does.  That said, I'm ok either way.


Filename for Firmware Files changed to new Version


@kalashnikovisme чего это у тебя такое страшное? оо


Closes #76


I'm equally not fussed. Up to you.


Ugh. It's not super exciting, but it works.


Kann sein, dass das geht, werds mal testen. Mit der Methode bin ich auch noch nicht zufrieden, ist mir noch zu viel Code.
Hab doch jetzt schon mal Kommentare hinzugefuegt.


FYI, Relay uses `RequestInterface` now.


should be a fat arrow here if you want to use `@scope` in the method below.


**judging**
![](http://quickme.me/disapprove.jpg)


@chelm fyi `tape test/*.js` will run all `*.js` files inside of `test/`, `node test/*.js` only runs the first one found to match that pattern afaik.


Can we wait with adding error handling until we have the approach defined? Just returning error instances is not the right choice in my mind.


`DomainException` would probably be a more correct base.


@pippinsplugins No, it doesn't refer to any site. Basically [`$wpdb->base_prefix`](https://codex.wordpress.org/Class_Reference/wpdb#Class_Variables) just omits the blog ID when on multisite. So it's perfect when wanting to use a single custom table across the entire network when a plugin is network-activated.

```php
$wpdb->base_prefix = 'wp_'
$wpdb->prefix      = 'wp_23_'
```


Ugh. This is not the most exciting thing to add, but it allows the expectations to work properly.


Ok I’ve done it!

Sent from Windows Mail

From: Helen
Sent: Monday, 13 July 2015 23:33
To: polorum/polorum.github.io
Cc: Ojedalicia

Ah. I'm glad you figured out. :)

Yeah, that Sladdin one was taken by Launch.ed for their media. They've said
that we can use it online, but need to check with the photographer if we
want to use it in print.

Helen Williams
hlwilliams92@gmail.com

On Mon, Jul 13, 2015 at 11:02 PM, Ojedalicia <notifications@github.com>
wrote:

> Hi Helen,
>
>
> Perfect thanks! I’ve just spent a while trying to understand this - and
> now I think I do, I’d better go to bed. 😊
>
>
> But I will get back onto this later in the week. Thanks for uploading the
> real files for me. I see the problem was I missed out the “ marks.
>
>
> I’ll check with you on images. I wasn’t certain about the Sladdin one but
> I’m glad that’s fine. It’s a very good one of you both.
>
>
> See you tomorrow
>
> Alice
>
>
>
>
>
>
>
>
> From: Helen
> Sent: Sunday, 12 July 2015 22:54
> To: polorum/polorum.github.io
> Cc: Ojedalicia
>
>
>
>
>
> https://github.com/polorum/polorum.github.io/commit/19b51a2c2436cc64638024f5bfd1b994f48a1499
>
> Check out the new commit, you should be able to pull it down to the Alice
> Experiments branch, but will then probably have to stash these changes
> (save them in a file somewhere that is outside of the polorum.github.io
> folder) and then merge the master into Alice Experiments (overwriting your
> changes) before putting the stashed files back and making a new pull
> request
>
> Helen Williams
> hlwilliams92@gmail.com
>
> On Sun, Jul 12, 2015 at 10:37 PM, Helen Williams <hlwilliams92@gmail.com>
> wrote:
>
> > Ok, I'm just going to pull your branch and check it out. I'll take a
> look
> > at it, and probably upload all the images to the site (I don't like the
> > idea of using images that aren't hosted on our server)
> >
> > Also, we do have permission to use all the pictures that you've added to
> > the website, but it is something that we need to be careful about, so
> just
> > be aware of that. And if you're not sure if we have the rights to use a
> > certain photo, I can probably answer that. :)
> >
> > Helen Williams
> > hlwilliams92@gmail.com
> >
> > On Sun, Jul 12, 2015 at 10:28 PM, Ojedalicia <notifications@github.com>
> > wrote:
> >
> >> Helen please review my changes. Ignore first three images added. I'm
> >> trying out images on our website - but I don't know exactly how to
> store
> >> images on our file and make them link (said 'broken link' when I
> tried).
> >> Let me know what you think
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/polorum/polorum.github.io/commit/dd3312f4f3fb8fb8faec7504f6da73b9feef421b#commitcomment-12124298>
>
> >> .
> >>
> >
> >
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/polorum/polorum.github.io/commit/dd3312f4f3fb8fb8faec7504f6da73b9feef421b#commitcomment-12143853>
> .
>

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


4973_heatpump_defrost_bug (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1169 of 1169 tests passed)

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


Do I have to copy this code on the file httpd.conf located in C:\xampp\apache\conf ? Or do I have to substitute?

<VirtualHost *:80>
    ServerName      aserratorta.com
    ServerAlias     www.aserratorta.com

    SetEnv SYMFONY__DATABASE__USER     "..."
    SetEnv SYMFONY__DATABASE__PASSWORD "..."

    DocumentRoot    "C:\Users\Anton\PhpstormProjects/aserratorta.com/web"
    DirectoryIndex  app.php

    <Directory "C:\Users\Anton\PhpstormProjects/aserratorta.com/web">
        AllowOverride None
        Allow from All

        <IfModule mod_rewrite.c>
            Options -MultiViews
            RewriteEngine On
            RewriteCond %{REQUEST_FILENAME} !-f
            RewriteRule ^(.*)$ app.php [QSA,L]
        </IfModule>
    </Directory>

    CustomLog  /var/log/httpd/aserratorta.com-access.log combined

    KeepAlive            On
    MaxKeepAliveRequests 200
    KeepAliveTimeout     5

    <IfModule mod_filter.c>
        AddOutputFilterByType DEFLATE "application/atom+xml" \
                                      "application/javascript" \
                                      "application/json" \
                                      "application/rss+xml" \
                                      "application/x-javascript" \
                                      "application/xhtml+xml" \
                                      "application/xml" \
                                      "image/svg+xml" \
                                      "text/css" \
                                      "text/html" \
                                      "text/javascript" \
                                      "text/plain" \
                                      "text/xml"
    </IfModule>

    <IfModule mod_headers.c>
        Header append Vary User-Agent env=!dont-vary

        ExpiresActive On
        ExpiresDefault "now plus 1 week"
        ExpiresByType image/x-icon "now plus 1 month"
        ExpiresByType image/gif    "now plus 1 month"
        ExpiresByType image/png    "now plus 1 month"
        ExpiresByType image/jpeg   "now plus 1 month"
    </IfModule>
</VirtualHost>


got it, that makes a lot of sense!


Hi, for to set i the hour and the phrase below that I have to put data? thk


This fixes an issue where the `char *` is not null-terminated.


Thanks! But `grammarScopes` still isn't updated when the option is changed.


It's better to move `import` inside `if` condition so Uglify can eliminate the dead import. The same concerns other places where DevTools-related stuff is imported. Because Babel doesn't allow `import` inside `if` per spec, I use `"loose": ["es6.modules"]` in `.babelrc` which I think allows it.


For some reason I recall there being a debate here on github about other elements being fixed vs absolute, but can't find it...  Happy to see this changed back.


Typo remove -> removed


Wow, thank you so much for the help! I deleted my message and didn't really
expect a reply. THANK YOU!!!!

On Tue, Jul 14, 2015 at 4:46 PM, Tony Alves <notifications@github.com>
wrote:

> Some scope issues. Here is a start of it somewhat working.
>
> var Camera = famous.components.Camera;
>
> var math = famous.math;
> var physics = famous.physics;
> var PhysicsEngine = famous.physics.PhysicsEngine;
> var Box = physics.Box;
> var Vec3 = math.Vec3;
> var Size = famous.components.Size;
> var Position = famous.components.Position;
> var MountPoint = famous.components.MountPoint;
> var Gravity1D = physics.Gravity1D;
> var Collision = physics.Collision;
> var Wall = physics.Wall;
>
>
> var DOMElement = famous.domRenderables.DOMElement;
> var FamousEngine = famous.core.FamousEngine;
>
> //Physics engine test
> function Test() {
>
>   //create scene graph
>   this.scene = FamousEngine.createScene('body');
>
>   this.camera = new Camera(this.scene);
>   this.camera.setDepth(1000);
>
>   this.simulation = new PhysicsEngine();
>
>   this.node = this.scene.addChild();
>   this.node
>       .setSizeMode('absolute', 'absolute', 'absolute')
>       .setAbsoluteSize(100, 100)
>       .setPosition(0, 70, 0)
>       .setMountPoint(0.5, 0.5);
>
>
>   var position = new Position(this.node);
>   this.myBox = createBox.call(this, this.node, position);
>   console.log('this.myBox', this.myBox);
>
>   FamousEngine.requestUpdateOnNextTick(this);
>   console.log('test');
> }
>
> Test.prototype.onUpdate = function(time){
>   this.simulation.update(time);
>
>   var itemPosition = this.myBox.getPosition();
>   //console.log(itemPosition.x, itemPosition.y, itemPosition.z);
>   this.node.setPosition(itemPosition.x,itemPosition.y,itemPosition.z);
>   FamousEngine.requestUpdateOnNextTick(this);
> };
>
> //add node - this node will be static
> //create box
> function createBox(node, position) {
>   //attach a DOM element component to the staticNode
>   this.blueDIV = new DOMElement(node, {
>     properties:{
>       'background-color':'#49afeb'
>     }
>   });
>   var mb = new Box({
>     mass: 10,
>     size: [100, 100, 100],
>     position: new Vec3(window.innerWidth / 2, window.innerHeight / 2, 0)
>   });
>
>   this.gravity = new Gravity1D(mb, {direction: Gravity1D.DOWN, strength: 300});
>
>   this.floor = new Wall({direction: Wall.UP, friction: 0});
>   this.floor.setPosition(0,window.innerHeight, 0);
>
>   this.collision = new Collision([mb, this.floor]);
>
>   this.simulation.add([mb, this.gravity, this.collision]);
>   console.log("hey");
>
>   return mb;
> }
>
> FamousEngine.init();
>
> var test = new Test();
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/a-huang/first-project/commit/7f3ff3e9c694ee4dd82de12d60286827da2ff335#commitcomment-12166591>
> .
>


I don't like this part yet. Will try to rephrase it a bit.


Why use the CDN? Fetching jQuery from our own server will probably be faster because if the server and client use `Connection: keep-alive` in the HTTP headers, then we won't need to have the client create another connection.


Thank you Erich,
I must merge manually this change in the next version.


I think you're missing a return statement here.


I went back and forth on this. Currently, the mount unit doesn't have an Install section, so it can only be statically enabled. I did that to avoid having to add another layer of management with the preset files.

I could add that setup, though. That would require the single disk image to disable it and then to add another preset file for split images that enabled it.


It will work when you push and splice instead of replacing in https://github.com/AtomLinter/linter-jshint/commit/e2c3f349dffdb5e4c6f9fbdd73b8ff8f2f0b6d62#diff-2e1bb36917ff76ab7308da52bc968501R19


Nice&needed


What is the function of `PdoEventSessionHandler::gc()` ?  It looks like that inherits documentation from `SessionHandlerInterface`?


Shouldn't this exception now also contain a message that the user doesn't have the proper permissions? Since after this patch users CAN recover or expunge as long as they are allowed to.


[ ]: Always leave the last comma in a multi-line array (Zend Coding Style)


Same applies here. The Exception doesn't really cover the case.


Found it     ...../media/graeme/TI80162900E/Users/Graeme/mandelbulber  So
file path leads through that  partioning stuff. Still can not find any
auto.fract.

what is weird is why I cannot remember it happening weeks ago, why it just
seems to have started. Hmmm?

 i have just noticed another another problem when I hover over a tooltip
like the select color button, the  text "Parameter name:
.......rgb(255,255,255)"  is repeated several times (it is ok when first
run, then starts repeating.)

 Although  this could have been inadvertently caused by me correcting
spelling in the render_window.ui yesterday.

# Mandelbulber settings file
# version 2.04
# application settings
[main_parameters]
anim_flight_dir ;
camera_absolute_distance_mode 1;
camera_movement_mode 1;
camera_movement_step 36;
camera_rotation_step 30;
flight_sec_per_frame 0.01;
netrender_client_remote_address ;
netrender_client_remote_port 1;
netrender_server_local_port 1;
show_cursor false;

On Wed, Jul 15, 2015 at 6:30 AM, Sebastian Jennen <notifications@github.com>
wrote:

> I don't think you have a problem with your system
> it is most certainly a bug in the program.
>
> Look into /home/graeme/.mandelbulber
> There you should be able to find the mentioned files.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/buddhi1980/mandelbulber2/commit/9620b313cfc8011a7f9d5620dcc5e9f395c9bbca#commitcomment-12162478>
> .
>


I think the assertion should be: "test.ok(q.key[0] === '123');"

The === will check the type for you, and it tests that the string conversion returned the value you expect.


Seems like your editor added some whitespace here? Did it on multiple locations in this file.

Might want to check your editor settings.


@electrofelix Thank you very much for tackling this issue. Please file a new issue if your investigations via https://github.com/pradels/vagrant-libvirt/pull/420 and https://github.com/pradels/vagrant-libvirt/pull/421 lead to identify something to be fixed or improved in Vagrant core.

BTW, we already had another integration problem with `libvirt` provider (#5743), which has been fixed by https://github.com/pradels/vagrant-libvirt/pull/379. It doesn't seem to be direclty related to this current issue (no time on my side to dig into it), but it also had to do with the `read_ssh_info` action, at the moment when no IP address was yet attributed to the machine. Maybe it is worth a quick look for you (just sharing the info to you).


I agree, was even thinking `test.same(q.key, ['123'])`.


Thanks. [Done](https://github.com/erikras/react-redux-universal-hot-example/commit/5cf0d9b99a72ca4ab368da31c4c28fd9c675013d).


Done!


async in a try/catch?

If line 79 fails, the catch statement won't catch that thrown err.


This commit could have been broken into several:
* update java command in readme
* modify whitespace and semicolons
* change int to Integer
* change method name
* add new method


Can you add some tests for delete with semijoins to the distributed query suite? Make sure you add some nulls to the data (e.g., by using nullif), since tpch data doesn't contain nulls.


The white spaces here are for readability. View the diff.

The if-statement starts here:

if (aCmd.getPodId() != null) {

And ends after:

sc.setParameters("poolId", storageId);
}


What about:
```
if len(lhs) == len(rhs) and lhstat.st_ctime == rhstat.st_ctime and lhstat.st_mtime == rhstat.st_mtime:
    print('cannot make a recommendation, files have same filename length and timestamp: {} {}'.format(rhs, lhs))
else if len(lhs) <= len(rhs) and lhstat.st_ctime >= rhstat.st_ctime and lhstat.st_mtime >= rhstat.st_mtime:
    print('recommend deleting: {}'.format(lhs))
else if len(lhs) >= len(rhs) and lhstat.st_ctime <= rhstat.st_ctime and lhstat.st_mtime <= rhstat.st_mtime:
    print('recommend deleting: {}'.format(rhs))
else:
    print('cannot make a recommendation, conflicting filename length and timestamp: {} {}'.format(rhs, lhs))
```


Fixes #16


:shipit:


This [breaks eslint](https://github.com/eslint/eslint/issues/2259), unfortunately. Oh well. Better to be conditionally included and break lint than the alternative.


Helaas!

Op dit moment ben ik niet aanwezig i.v.m. mijn vakantie. Op maandag 27 juli ben ik er weer!

Als u dringende vragen heeft, kunt u het beste contact opnemen met Chris via chris@72dpi.nl of even bellen: 020 77 293 77.

Met vriendelijke groet,
Jeffrey Lauwers
www.72dpi.nl
020 77 293 77


This looks great so far!


This commit could have been broken up into two:
* add get all courses resource
* remove debugging statements


@Eureka22 Well, we dont want people to bypass this easily.


I guess you could just use vanilla `require` in this case..


@vram4 That makes sense. I see


Those plugins don't really matter, but when upgrading a library it should be tested if it compiles in a PR. Or did you test it in a PR. I remember a `Library update` PR not too long ago.


typo?


Injected CSS isn't needed, JS automatically adds it now.


Autoplay is redundant when you have `video.play`


Added the higher res icon supported by Chrome.


Good point. Fixed.


@glasser I believe this commit captures the intent of [your comment](https://github.com/meteor/meteor/pull/4718#discussion_r34531022) :)


Just OCD. There are a few of changes like these. I just have a weird spacing style.


Adding `shia-do-it--` to `.container` increases support with websites that use `.container`


From what I can tell test.same uses assert.deepEqual which does not do strict equality checking. Going with original suggestion.  http://nodejs.org/docs/v0.4.2/api/assert.html#assert.deepEqual


VERY GOOD.


These were the only indents not tabs in this file, OCD again.


(As said before) Adding `shia-do-it--` to `.container` increases support with websites that use `.container`


Sorry if github is the wrong place to comment. Not sure where best to make a general comment.

I think LLVM takes the approach where initial ssa emission ignores phi nodes all together and allocates a stack slot for everything that can't be expressed in phi less ssa. It then uses a pass to eliminate the dumb loads and stores and add phi nodes.

In general I think it is done like that because SSA can be tricky to reason about, and it is easier to do some transforms while ignoring ssa. It is fun to look at clang -emit-llvm -O0 to see how dumb the initial SSA actually is in situations like this.


rileggendo il codice sai che forse c'è un errore? :-/
Forse dovrebbe essere così:
var arrayPreferite = [];
arrayPreferite.push(JSON.parse(window.localStorage.getItem('fermata')));

Comunque anche nel mio c'è qualcosa che non va, perchè se faccio:
console.log(arrayPreferite);
dice che non è stata dichiarata. BOH!


I see, makes sense; it was merged like this, so I think we're all set.


I remember you doing that optimisation ages ago. Your and Peter's changes haven't been incorporated yet.


Local videos prevent issues with slow loading, increased network, and issues with high security websites like GitHub.


Semicolon


[![Madonna and Child Enthroned with Saints](http://api.artsmia.org/images/10448/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=10448)


OCD


`<all_urls>` grabs URLs better. There's better support for subdomains, visual FTP, etc. Remove if an issue is found with HTML on a frequented protocol.


(As said before) Local videos prevent issues with slow loading, increased network, and issues with high security websites like GitHub.


not `async function(` ?


Prep for the high res icon (next commit)


Duh. Sometimes I forget that Ye Olde Way of doing things still works.

 :+1: Worked perfectly.


:smile:


Python 3 doesn't know xrange and modern Python 2.7's xrange is the same as range.


On 7/14/15 5:28 PM, Ola Bini wrote:
> Can we wait with adding error handling until we have the approach
> defined?

Sure.

>Just returning error instances is not the right choice in my mind.

The alternative is panicking for those errors we cant recover.

According to
(http://dave.cheney.net/2012/01/18/why-go-gets-exceptions-right):

"When you panic in Go, you’re freaking out, it’s not someone elses
problem, it’s game over man. panics are always fatal to your program."

IMO, failures to generate things that are necessary to the protocol
should be panics (failures when reading from random number generator,
unexpected nil values).

However, returning errors makes it easier to report back to the
xmpp-client that something bad happened when we received a message.

I'm OK with postponing this - I guess I was afraid of forgetting them
(as well as having lots of work piling up).


this returns a promise due to delAsync above


Injected CSS isn't needed, JS automatically adds it now.


this should also be pushed to primises array


Adds CSS smother and reliably.


Not sure what you mean, but no, this is to fix issue #3.


right. will fix in a moment. was supposed to be sleeping already, but this bug jumed out of my mind


I told you all its a good one. I love it


I blogged about this commit @ http://rubylogs.com/migrate-rspec-to-minitest


Yes, we can avoid most of that because we are only writing the back half of the compiler.  In particular, we already know which variables might be aliased to pointer writes and which aren't.  If you don't know that yet, you can't promote anything to an SSA variable initially.


example for #29


yeah, nice catch. Committed 1da8580c152b7b6443c174382cfe501e940f0744 with  just now.


```ruby
DIRECTIONS = %w[N NNE NE ENE E ESE SE SSE S SSW SW WSW W WNW NW NNW]
```


Because of float imprecision, you MAY want to make these strings.


Yes, in the process if refactoring it. To be honest, I don't really know how it could possible with the original implementation - did this ever work? (I might be overlooking something obvious, but how did the dual-binding work?)


`assert_equal(expected, actual)`


ditto on all of these


```coffee
activate: ->
    @scopes = ['source.js', 'source.js.jsx']
    embeddedJavaScriptScope = 'source.js.embedded.html'

    @subscriptions = new CompositeDisposable
    @subscriptions.add atom.config.observe 'linter-jshint.lintInlineJavaScript',
      (lintInlineJavaScript) =>
        if lintInlineJavaScript is true and embeddedJavaScriptScope not in @scopes
          @scopes.push(embeddedJavaScriptScope)
        else if lintInlineJavaScript is false and embeddedJavaScriptScope in @scopes
          index = @scopes.indexOf embeddedJavaScriptScope
          if index isnt -1
            @scopes.splice(index, 1)
```

This still doesn't work...


Does this have to happen every frame? Can't we get an event when we rollover something and do it then?


Don't ever use `assert_equal` with floats. Use `assert_in_delta` or `assert_in_epsilon` (auto-scaling delta).


Use string interpolation or, better, `File.join`.


If you run tests via rake, there's no reason to `expand_path`. You'll always be in the root directory.


Couple simple mistakes, but looks good!


The way I would do that is
```coffee
activate: ->
  @scopes = ['source.js', 'source.js.jsx']
  embeddedJavaScriptScope = 'source.js.embedded.html'

  @subscriptions = new CompositeDisposable
  @subscriptions.add atom.config.observe 'linter-jshint.lintInlineJavaScript',
    (lintInlineJavaScript) =>
      if lintInlineJavaScript
        @scopes.push(embeddedJavaScriptScope) if not embeddedJavaScriptScope in @scopes
      else
        @scopes.splice(@scopes.indexOf(embeddedJavaScriptScope), 1)
```


Incorporated the change.. looks like it shaves a few more ms off the tests :)


Fantastic work!


Let's leave this until Friday, I think there is something on this that we should discuss.
Although for me it's `r+` (it passes my review!)!


JQ UI


:}

Sent from my iPhone

> On Jul 14, 2015, at 2:15 PM, Mo King <notifications@github.com> wrote:
>
> Noice job m8
>
> —
> Reply to this email directly or view it on GitHub.
>


@nickfletcher so notice the updates to the commonWallet interface, so you'll have to update the wallet in example/example.js


* I'm on Linux, what for do I need this hardcoded path?
----------------- setRootPath("My Computer");
  and here:
-----------------ui->lineEdit->setText("My Computer");
* here is setting an index
-----------------ui->listView->setRootIndex(up_index);
  and later is checking if it is valid
* remove newlines where not needed
* check everything once more
* add yourself to contributors in the header
-------Copyright (C) 2014  by ...


Too late: https://github.com/awesomeWM/awesome/commit/78abb4a54cf58db21396cedc34f0aa46827212ff


Shouldn't this also be in the secrets table?


I love this commit message *so much*.


...and then expose it here.


Try and make many small atomic commits with descriptive commit messages. If you need to roll this back due to a bug or something, it is much harder to do it when you have massive commits like this.


Looks like garbage collection, `gc`* functions generally are.


The migration says `nullable=False`. However, the definition in the schema doesn't specify `nullable`, and the SQLAlchemy docs it says that `nullable` is `True` by default, so the schema doesn't appear to match the migration for this and for other fields.


Looks like this comment should now say 10.


Fix your grammar errors @looperXD


Instead adding _NETBSD, it would be better just to rename _FREEBSD to _BSD and then unify those two, since it looks like there is no much difference.


Only (potentially implicit) object properties can be deleted, but not names bound to global functions. Something like

```javascript
Cu.evalInSandbox('this.MenuCommandSandbox = ' + MenuCommandSandbox.toSource(), sandbox);
...
Cu.evalInSandbox('delete this.MenuCommandSandbox;', sandbox)
```
should work though.


For the record, I fully agree with @Tobion . The problem here isn't that there's a vector. The problem is with people outputting partial URLs. IMHO, the Uri is absolutely the wrong place to solve this.


Your changes to GENie script seems valid if binaries are not compatible (I assume they are not?).


This does still not work and `@scopes.splice(@scopes.indexOf(embeddedJavaScriptScope), 1)` removes 'source.js.jsx' when the option is disabled and wasn't enabled before.


Thanks for the reply, that makes sense. In general I think the new backend is quite approachable.


Just changed the team parameter to test whether filters are getting set correctly


`aFrameScope` is unused.


Several hours later.      Attached is the file, was last modified 29 June
last accessed 30 June. Located in the folder where my old Windows
Mandelbulber V2.04 setting and images are stored.

Should I really have a version of 2,04  on my Ubuntu partition.
computer/home /Graeme/

And to make things worse, QT Creator now will not respond,  loads and turns
grey

On Wed, Jul 15, 2015 at 10:41 AM, carbuncle grim <mclarekin@gmail.com>
wrote:

> Found it     ...../media/graeme/TI80162900E/Users/Graeme/mandelbulber  So
> file path leads through that  partioning stuff. Still can not find any
> auto.fract.
>
>
> what is weird is why I cannot remember it happening weeks ago, why it just
> seems to have started. Hmmm?
>
>
>  i have just noticed another another problem when I hover over a tooltip
> like the select color button, the  text "Parameter name:
> .......rgb(255,255,255)"  is repeated several times (it is ok when first
> run, then starts repeating.)
>
>  Although  this could have been inadvertently caused by me correcting
> spelling in the render_window.ui yesterday.
>
>
> # Mandelbulber settings file
> # version 2.04
> # application settings
> [main_parameters]
> anim_flight_dir ;
> camera_absolute_distance_mode 1;
> camera_movement_mode 1;
> camera_movement_step 36;
> camera_rotation_step 30;
> flight_sec_per_frame 0.01;
> netrender_client_remote_address ;
> netrender_client_remote_port 1;
> netrender_server_local_port 1;
> show_cursor false;
>
> On Wed, Jul 15, 2015 at 6:30 AM, Sebastian Jennen <
> notifications@github.com> wrote:
>
>> I don't think you have a problem with your system
>> it is most certainly a bug in the program.
>>
>> Look into /home/graeme/.mandelbulber
>> There you should be able to find the mentioned files.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/buddhi1980/mandelbulber2/commit/9620b313cfc8011a7f9d5620dcc5e9f395c9bbca#commitcomment-12162478>
>> .
>>
>
>


Fair question. Three reasons

1. The PR contains multiple changes (e.g. annotations on examples, trimming whitespace). It's possible some of these changes may not be appropriate, or (as in the case of blank line separators vs dash example separators) there might be a preferable alternative.

2. I sometimes use Yadda to demonstrate that it's possible to write software without else statements, switch statements or mutators functions, with minimal accessor functions and an average function size of < 4 loc. (See https://github.com/acuminous/yadda/wiki/Notes-for-Contributors for more details). The PR included some medium length functions and relies on switch statements so I want to refactor.

3. Quite some time ago @ami44 created an [issue](https://github.com/acuminous/yadda/issues/53) for docstrings. I think I can implement this using the same approach for multiline examples,

None of this is to say that I don't really appreciate your contribution. I think you've done an excellent job implementing a tricky feature and have considered several edge cases.


Good catch, done.


Ah right. then it would be
```coffee
activate: ->
  @scopes = ['source.js', 'source.js.jsx']
  embeddedJavaScriptScope = 'source.js.embedded.html'

  @subscriptions = new CompositeDisposable
  @subscriptions.add atom.config.observe 'linter-jshint.lintInlineJavaScript',
    (lintInlineJavaScript) =>
      if lintInlineJavaScript
        @scopes.push(embeddedJavaScriptScope) if not embeddedJavaScriptScope in @scopes
      else
        @scopes.splice(@scopes.indexOf(embeddedJavaScriptScope), 1) if embeddedJavaScriptScope in @scopes
```


@nimeshkverma We usually create model name in plural, so it would be Polls here. Just to keep everything similar!


Needs to pass on *which* menu command was clicked, so the `MenuCommandRun` only runs one command, instead of one in each sandbox.


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.16](http://pixelated.design/build/viewLog.html?buildId=40&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 122 Build time: 00:00:10


Fixed in 5bb0df9.


Add a primary id field to model, keep it auto incremental.


Should run it as `command.commandFunc.call()`, so the user script callback can't mess with the command object.


So send a PR?


Why is `import journal` at the bottom here?


@nimeshkverma Same as above, primary id. Are we keeping vote_count here? Or directly fetching on fly from Votes?


I believe the reason that eslint has a warning is because the import is no longer statically analyzable.


4973_heatpump_defrost_bug (nmerket) - x86_64-MacOS-10.9-clang: OK (1674 of 1678 tests passed)

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


Binaries are different, yes (different libc, syscalls, ...).


Could this be a static object instead? I.e. `var ContentObserver = { QueryInterface: ..., observe: ... };`


About *BSD: I'm not sure the other BSDs (OpenBSD, DragonFly, Bitrig, ...) are all the same in the respects that interest bx, so I'm not comfortable joining these clauses. But if you insist...


Yes. But the value does still not change.

I believe that the `grammerScope` does only take the value that is defined on activate and doesn't updates the value when the `@scopes` variable is changed.


Nit: extra whitespace in front of `var`.


thx, better than shits


Thank you.  I have reverted it.


Awww Come on :smiley:
I think I'll clone your repo and do some fiddling and post the diff when I get it working.


linting errors:
```
$ flake8 journal.py
journal.py:15:1: F401 'HTTPNotFound' imported but unused
journal.py:16:1: F401 'IntegrityError' imported but unused
journal.py:57:20: E711 comparison to None should be 'if cond is None:'
journal.py:96:1: E302 expected 2 blank lines, found 1
journal.py:101:1: E302 expected 2 blank lines, found 1
journal.py:107:5: E265 block comment should start with '# '
journal.py:109:1: E302 expected 2 blank lines, found 1
journal.py:116:1: E302 expected 2 blank lines, found 1
journal.py:120:1: E302 expected 2 blank lines, found 1
journal.py:122:22: E231 missing whitespace after ':'
journal.py:147:1: E302 expected 2 blank lines, found 1
journal.py:162:1: E302 expected 2 blank lines, found 1
journal.py:167:57: E251 unexpected spaces around keyword / parameter equals
journal.py:167:59: E251 unexpected spaces around keyword / parameter equals
journal.py:169:1: E302 expected 2 blank lines, found 1
```


What's wrong with bugsnag traces? We're evaluating it and already use Sentry. Thanks.


ATM I'm busy setting up my environment - again - can it wait two days? xD


Ok :D
It's time to get some sleep anyway no...


@psychon
Thanks! :cherries: :)


4973_heatpump_defrost_bug (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1680 of 1684 tests passed)

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


Yeah that makes sense—it's against the spec (and the whole point of ES6 modules). There is probably a more elegant solution with just using ES6 modules + `ProvidePlugin` to switch between `middleware.prod.js` and `middleware.dev.js` but I don't care too much.


Replace "spec/internal" with the configuration parameter for the destination path?


This makes me nervous. On a 32-bit architecture a int is only 32 bits. A PostgreSQL int8 could get silently truncated. I suppose int4 and int2 are always safe, but it feels a little sloppy using int unless you really want architecture pointer size.


Got it working.
```diff
diff --git a/lib/main.coffee b/lib/main.coffee
index ec081d1..80e95ad 100644
--- a/lib/main.coffee
+++ b/lib/main.coffee
@@ -12,13 +12,15 @@ module.exports =
       description: 'Lint JavaScript inside `<script>` blocks in HTML or PHP files'

   activate: ->
-    @scopes = []
+    scopeEmbedded = 'source.js.embedded.html'
+    @scopes = ['source.js', 'source.js.jsx']
     @subscriptions = new CompositeDisposable
     @subscriptions.add atom.config.observe 'linter-jshint.lintInlineJavaScript',
       (lintInlineJavaScript) =>
-        @scopes = ['source.js', 'source.js.jsx']
-        if lintInlineJavaScript is true
-          @scopes.push('source.js.embedded.html')
+        if lintInlineJavaScript
+          @scopes.push(scopeEmbedded) unless scopeEmbedded in @scopes
+        else
+          @scopes.splice(@scopes.indexOf(scopeEmbedded), 1) if scopeEmbedded in @scopes

   deactivate: ->
     @subscriptions.dispose()
@@ -34,11 +36,12 @@ module.exports =
       grammarScopes: @scopes
       scope: 'file'
       lintOnFly: true
-      lint: (textEditor) ->
+      lint: (textEditor) =>
         exePath = atom.config.get('linter-jshint.jshintExecutablePath') || jshintPath
         filePath = textEditor.getPath()
         text = textEditor.getText()
         parameters = ['--reporter', reporter, '--filename', filePath]
+        console.log(@scopes) # Remove after debugging
         if @scopes? and 'source.js.embedded.html' in @scopes
           parameters.push('--extract', 'always')
         parameters.push('-')

```


4973_heatpump_defrost_bug (nmerket) - 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)


Of course. RA2 is not rushed.


Awesome job!
You tested this? boot up?


rax retry tests


This seems reasonable. I'd previously considered it, but erred on the side of strictness. Since someone else independently now wants it, I could go for this.


* use const QString &smth instead of QString smth if you are not going to modify parameters
* refer to coding standard and change parameter names (not lName)
* no need to #include <QWidget> in customtabbar.cpp and class QWidget; in customtabbar.h
* add some comments to methods


I tend to keep global parameters in global scope now to have them in one place. Also, if audio and video can be disabled on OS X, no need to link their libraries...


@zenspider thanks so much! :-)


:+1: to remove `nudgeTransitions()`. I believe with #1548 we have now guarantees that the view will re-render up to the point where all transitions and animations (eg. fading, transform-related, etc) will be completed.


I don't quite understand what this is trying to accomplish -- and there are no tests to demonstrate usage and guarantee correctness. Pointers to interfaces are usually unneeded in Go -- what is the goal of this code?

Also, there is a lot of duplication in this nested switch. Assuming there is not another way to do what you want, maybe we can restructure this method to eliminate the duplication.


获取的分集剧情不对，感觉像是抓的豆瓣评论……


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 141](https://teamcity.labkey.org/viewLog.html?buildId=260926&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 1 Build time: 00:04:09
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest.initializationError: <no details avaliable>

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

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

```


Para hacer pruebas, hacer falso la columna asistio en Alumni y borrar las matriculas en AlumnosAsistentes


`Object.clone` used to do a cheap, shallow copy for objects but now it does the expensive `extend`.
[original code]
```javascript
clone: function(object) {
        var newObj;
        if (Array.isArray(object))
            newObj = object.clone();
        else {
            newObj = {};
            for (var attr in object)
                newObj[attr] = object[attr];
        }
        return newObj;
    }
```
Is there something similar in lively.lang now? (I remember switching from `extend` to `clone` for exactly this reason with the debugger.)


98812224-RoomAirTemperaturePattern-5014 (rglazer2) - x86_64-MacOS-10.9-clang: OK (1678 of 1679 tests passed)

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


Why the absolute path and not the relative path?


This file should not reexport things I think. This is the purpose of the `angular2/change_detection.ts` file...


A good sample that shows how to create a point geometry, and insert it into a feature class using C++.


I'm not sure about this. This method is designed to downgrade types that could be handled in binary by the native interface. I don't think we want to force upgrade to binary format. It should already default to binary and only be set to text if it was explicitly overridden. I wouldn't want to override it back.


I'm pretty certain whatever JSON unmarshalled into would not be a database/sql compatible type.


Yes, but the cursor disappears.


It's crashing for me:

![image](https://cloud.githubusercontent.com/assets/244100/8687805/cc7bccaa-2a48-11e5-9bed-f167204166d7.png)
![image](https://cloud.githubusercontent.com/assets/244100/8687810/e45461de-2a48-11e5-857a-e11ea9c2aaa9.png)


thanks again! we all suffer from the too quick pull request syndrome :) I've done it myself.


confirmed fix on try, looks good :+1:


This actually handles the SIGTERM signal correctly. Obviously, since a SIGTERM and then a SIGKILL are sent, the task to be run on exit has to be very short (ie. the *shutdown* event is not suited for remote synchronization, disk cleanup and the like: for that an *idle time* based condition would be better). This has to be explained in the documentation.


Why did you add a sleep after enable?


I don't understand what this is trying to do.


I'll have to give that more thought. This is only for testing engine_cart itself, so I'm not sure the configuration necessarily applies.


thanks @mgsloan


develop (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


This isn't creating the same file as it's removing...


Only single spaces between methods within a class.


`cerr` would probably be better for this, right?  Same with above.


4973_heatpump_defrost_bug (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1679 of 1683 tests passed)

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


Test


test


Good thought on preventing empty input being converted to `0`.  There's a simpler way to do that if you change your input reading, though.


done


Reading through everyone's submissions, you all have different input formats for trinary numbers (or trinary natural numbers, in this case).  It would be good to document what the input format is.


Thanks you Nuno !

On Mon, Jul 13, 2015 at 8:51 PM, Nuno Goncalves <notifications@github.com>
wrote:

> Now it looks nice! Thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dgiardini/rtl-ais/commit/50608033aa13c2ac374ae605c9837e2648276414#commitcomment-12145146>
> .
>

--
----------------------------------------------------------------------------------------------
This message represents the official view of the voices in my head


4973_heatpump_defrost_bug (nmerket) - 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)


All the other patches have been one per program.  This patch should be split up before submission.


rr


ok


This test should fail, because it fails the `PROGRAM` production in the write-up.


thanks


https://github.com/paragonie/random_compat/blob/5619a9ffcaba8ee284c7060e55d89dbdad7c719f/lib/random.php#L189

We already checked that `$max > $min` so therefore minimum value for `$range` at this point is 2.


It doesn't look like we're testing `print` in a subexpression.


Aca les dejo la version 1 del ante proyecto. Pueden modificarla a gusto, es solamente para tener una base.
Ya estaria lista para solo agregarle ciertas cosas como introduccion, objetivo y conclusion. Hace falta agregarle partes mas tecnicas como las de la base de datos, o los objetos que estariamos usuando, esta parte tectina puede ir abajo despues de donde sale la imagen del instalador.
Cualquier pregunta solo me avisan
[Ante Proyecto - Proyecto Final V1.docx](https://github.com/P3UAM-IICO-GRP3/proyectoFinal/files/695/Ante.Proyecto.-.Proyecto.Final.V1.docx)


Always use `auto`.


safe 말고 force_escape인데 잘못 적었네요.

```python
>>> from django.template import Context, Template
>>> t = Template("{% autoescape off %}Hello {{ unsafe_something | force_escape | linebreaksbr }}{% endautoescape %}")
>>> d = {"unsafe_something": '''<b> I'm unsafe
... a
... b
... c
... </b><script>
... asdf
... </script>
... '''}
>>>
>>> t.render(Context(d))
u'Hello <b> I'm unsafe<br />a<br />b<br />c<br /></b><script><br />asdf<br /></script><br />'
```

만약 더 정교한 handling이 필요하다면 https://github.com/jsocol/bleach 를 이용해보는것도 좋습니다.


i was trying very hard to not add this function at all, since I'm not sure I want to commit to this API.

i was largely in favor of https://github.com/JuliaLang/julia/pull/8745#issuecomment-117299862 with a check enforced by the Pkg manager that the module is actually suitable for incremental compilation.


![2558-07-06_19 13 34](https://cloud.githubusercontent.com/assets/6128413/8665615/19a738f4-2a10-11e5-9b65-8fad9aaffef3.png)
![2558-07-06_19 14 02](https://cloud.githubusercontent.com/assets/6128413/8665616/19a9fd00-2a10-11e5-977b-b5e42093921c.png)


Gotcha, thank you


done. it's only used in testing and debugging code so passing a magic constant is ok i think


Yes, you're right. Changed.


This is not a bug because we have accounts payable/receivable. See https://github.com/gratipay/gratipay.com/pull/3616#issuecomment-121066354.


пробный коммент к  обновлению, от Глеба.


Always use `auto`!


i added that check and updated javadoc


This is linked to the following error:
https://github.com/intridea/grape/issues/1059


good catch


If we're going to print error messages like this, it's almost always better to give the salient information, instead of just `Internal error`.


it's just the backbone I was in the middle of the process when I realised that I need some rest here =)
I will need those


@dsblank  Thanks for the make.py clarification.


We also can't use `switch`.  [`isdigit`](http://en.cppreference.com/w/cpp/string/byte/isdigit) is probably more suitable, too.


@Redjik ah right fair enough :smile:


@tbalden
https://github.com/xiaolu/android_device_htc_htc_hiau_ml_tuhl


This is why we tend to use the `while (cin >> input)` idiom: otherwise you resort to this, where the input statements are duplicated and separated far apart, which is very bad for maintaining.


Well as we discussed earlier neither is the `keysIn` stuff


helllo
thanks a lot for your quick reply.
I got docker*.deb from your git tree and downloaded it. then I manually install it via dpkg tools. so I can continue to do trial.

I'm using FSL LS1021QDS board which based on cortexA7. Linux kernel version is 3.12.
here is my source.list:
root@localhost:~# cat /etc/apt/sources.list
# See http://help.ubuntu.com/community/UpgradeNotes for how to upgrade to
# newer versions of the distribution.

deb http://ports.ubuntu.com/ubuntu-ports/ trusty main restricted
deb-src http://ports.ubuntu.com/ubuntu-ports/ trusty main restricted

## Major bug fix updates produced after the final release of the
## distribution.
deb http://ports.ubuntu.com/ubuntu-ports/ trusty-updates main restricted
deb-src http://ports.ubuntu.com/ubuntu-ports/ trusty-updates main restricted

## Uncomment the following two lines to add software from the 'universe'
## repository.
## N.B. software from this repository is ENTIRELY UNSUPPORTED by the Ubuntu
## team. Also, please note that software in universe WILL NOT receive any
## review or updates from the Ubuntu security team.
# deb http://ports.ubuntu.com/ubuntu-ports/ trusty universe
# deb-src http://ports.ubuntu.com/ubuntu-ports/ trusty universe
# deb http://ports.ubuntu.com/ubuntu-ports/ trusty-updates universe
# deb-src http://ports.ubuntu.com/ubuntu-ports/ trusty-updates universe

## N.B. software from this repository may not have been tested as
## extensively as that contained in the main release, although it includes
## newer versions of some applications which may provide useful features.
## Also, please note that software in backports WILL NOT receive any review
## or updates from the Ubuntu security team.
# deb http://ports.ubuntu.com/ubuntu-ports/ trusty-backports main restricted
# deb-src http://ports.ubuntu.com/ubuntu-ports/ trusty-backports main restricted

deb http://ports.ubuntu.com/ubuntu-ports/ trusty-security main restricted
deb-src http://ports.ubuntu.com/ubuntu-ports/ trusty-security main restricted
# deb http://ports.ubuntu.com/ubuntu-ports/ trusty-security universe
# deb-src http://ports.ubuntu.com/ubuntu-ports/ trusty-security universe
# deb http://ports.ubuntu.com/ubuntu-ports/ trusty-security multiverse
# deb-src http://ports.ubuntu.com/ubuntu-ports/ trusty-security multiverse


yes, done


same


This patch doesn't seem useful.  I think it should have been merged with the one that adds the flags before submission.


Hello - my world


I should have rebased this commit instead of merging it. Derp.


v1.0 you have to input YCICellDynamicHeightManager.h/.m to your Project


коомент 2 от глеба.


Also does es5 shim actually cover the chain cases?
On Jul 13, 2015 10:58 PM, "John-David Dalton" <notifications@github.com>
wrote:

> Nope. That's not es5-shimmable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lodash/lodash/commit/5e8f2ebfc9779a4490a3cfaccb83f4d8c3e49db1#commitcomment-12146865>
> .
>


Don't use `long`.  It doesn't do what you want, and you've actually introduced a bug in the program here without realizing it, because you forgot to change the type everywhere.


It was "correct". Thanks.


This is an optimization, and it clearly be marked as such.  It took me a while to realize it doesn't change your algorithm at all.


done


Spacing on `i+=2` is inconsistent with the rest of the file.


Daylighting_Speed (DeadParrot) - x86_64-MacOS-10.9-clang: Tests Failed

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


LGTM, thank you for taking care of this!


No, I'm really bad about doing 'git commit -av' and not noticing the submodule updates.


line note on line 2


line note on line 9


another line note on line 9


哇喔，Touch icon 根本年代物！XD


ls de ls is right


these tests pass because they're not testing anything.  you need to write tests (`expect(something).to eq something_else`) matching the test description.

and the goal is to use the factory to create the test data.


Just like to mention that 1.7.3 has introduce new errors. If it happens try 1.7.2 instead.


Discussion at https://groups.google.com/d/msg/cryptopp-users/h-YTSAtShow/krTEyXT0cJsJ.


4973_heatpump_defrost_bug (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (651 of 651 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.42%25-green.svg)


@rschiang 就順便 merge 進來了嘛
![default](https://cloud.githubusercontent.com/assets/5615415/8665882/0072e8b8-2a1c-11e5-8839-c9a93c9c07b6.JPG)


I think this optimization is slightly too optimistic, as in we are changing the UI before the data gets persisted. I am ok with preparing the data (and not displaying it) while the round trip is being made. However as for actually displaying the data, there is server side engineer of me who wants to protest.

But 'if' we decide to use this optimization:
+ Currently, we do enforce max in-flight request = 1. But the store layer should not assume that. Checking for -1 would break if there are multiple objects in-flight.
+ Removing and adding elements within a loop is generally not a safe practice and the behavior varies from js engines.


Basically, secret keys are advised to be stored in environment variables or external configuration file.
(Try `os.environ.get`)


Context.propagate throws an exception if this is called more than once per thread.  Since the executor reuses threads, this will die here after the first save, and savePending will be true, preventing further saves.


You need to revert your changes to createTextNote. its functionality is to only create pure text notes not enml notes.
Also you might as well cache the collection of tags at once and update it in the create tag function as well.


@rschiang 我是打算之後寫好再去修才這樣（？


當然不可能把 production 的 key 放這裡啊


@RReverser what's your npm account?


Create the client in the open function.
In the load function, make a comment for the -1 you do for the note with the tag name.
We should just have create tag take care of the fully named note


Good! The problem was finally settled.


The TimeField changed in 1.5.9 to use the latest version of the underlying plugin:
https://github.com/Eonasdan/bootstrap-datetimepicker

The implementation was lighted up a bit and it now rides on top of the DateField.

If you can revisit and implement on top of the new TimeField (if needed), I'm happy to merge.  I was able to pull in two other commits from this PR but this one is not obvious.


Added to head, thanks!


add standardopenoption.truncate


    this.childBounds = this.getBounds().clone();

It is not need here. I found in melonjs container.js init method, there is it. So it's not need .


a


What about `err.toString()` instead of `err.code` (slightly more useful info)?


Tuve el siguiente error al ejecutar foreman start en Ubuntu 12.04.

```bash
/usr/local/foreman/lib/foreman/process.rb:54:in `run':
undefined method `spawn' for Process:Module (NoMethodError)
```
Encontré esta guía:

https://github.com/ddollar/foreman/issues/473

Ahí hay varias soluciones pero la que me funcionó fue:

```bash
sudo apt-get install ruby1.9.1
sudo apt-get remove ruby1.8
hash -r
sudo gem install foreman
```

Tal parece que yo tenía instalado la versión de **ruby 1.8**, así que al desinstalar esta versión e instalar la *1.9.1* y volver a instalar **foreman** para ruby al correr el comando **foreman start** arrancó la aplicación en el puerto **5000** como indican.


エンジン指定しないとHerokuでどーなるんだろう？最新のv0.12になるんかな？


Looks fine, but can you explain what issue this fixes?