'dat commit message :smile: It's true, though, and it caused problems before. Cleaning this up is something we should really look into for future changes to i3bar. Having rendering and click calculation duplicated attracts errors.


You can use the generic rules (.o.c:) for that =)


:bug: gone


It's an enum rename, on an enum that's not used anywhere in TrinityCore scripts. You would need to check your custom code to fix your issues.


Please keep these 2 strings together. We already have the string in newsletter.lang
```Firefox is coming soon to iOS! <a href="{url}">Sign up to learn more</a>.```


i tried to change sw to ogl no joy,what am i doing wrong?


Woot!


yep thanks!


Can it be optional?


Yes. Feel to put changes you are making locally in a pull request.


Conferido e OK


Changed CLI argument parser to recognize flag '-s' option in different positions


1: M100 needs more reviewers
1: M101 signature is missing


1: D400 First line should end with '.', not 's'
1: D205 Blank line missing between one-line summary and description
1: D208 Docstring is over-indented
3: L102 copyright year is outdated, expected 2015 but got 2013
33: D400 First line should end with '.', not 's'
33: D205 Blank line missing between one-line summary and description
70:5: B901 blind except: statement
81:5: B901 blind except: statement
93:5: E266 too many leading '#' for block comment
104: D400 First line should end with '.', not '2'
104: D205 Blank line missing between one-line summary and description
126: D203 Expected 1 blank line *before* class docstring, found 0
126: D204 Expected 1 blank line *after* class docstring, found 0
126: D400 First line should end with '.', not 's'
135: D102 Docstring missing


use git checkout ogl-renderer-dirty to load this branch


Grade: 1/2

You shouldn't be manually defining @user and @user.name, etc. Objects can be generated in the Rails console (as you answered for one of the questions) and the entire table can be assigned in the controller using @variable = Table.all. The correct way to display all objects of a model would have been to iterate through all objects in a table using <% @variable.each do |var| %>.


@BebZ Will add 'if >= 3.8' to Mobilid to make use of the new functions


Grade: 2/2


The anonymizer creates a local sqlite database and puts an audit trail in there of some of the elements you are anonymizing in case you need to do some post processing. For example, maybe you want to anonymize the files, but still be able to hook them up to an existing de-identified patient record. The database contains tables for each element that it is setup to audit with the element value before it was erased.


@tcopeland,
Per our last meeting, I'm trying to test that `show_90_day_data` only returns tweets of last 90 days.
I can't seem to access `created_at` because it's created on active record instantiation.
Curious, is this the job of a mock? If so, do I put it in `#new_tweet` or in the test block?


C-style cast.


Hi Thomas,

I just added it, it´s this one: https://bugs.eclipse.org/bugs/show_bug.cgi?id=461874

Regards

Dennis


Ruby didn't seem to like a method beginning with a number, so I added the `show_`


Because they both use `templates` brunch option. `jaded-brunch` will try to create dynamic templates and `jade-ngtemplates-brunch` will try to build angular-compatible templates module. `jade-pages-brunch` doesn't try to create templates at all, it only builds static pages specified by the `pattern` option.


the original comment was better because it says what the if statement was supposed to do


@SamuelDavis I hope you'll explain to me how and why to do objects instead of arrays tomorrow!


Might want to update wording on the homepage as well. Right now it says:

> If you're wondering what this collected data actually looks like, have a look at the last 5 tracks that have been submitted:

This might be better:

> If you're wondering what this collected data actually looks like, have a look some tracks that have been submitted recently:


This is the first time I've touched a Python codebase, so I didn't try anything special, literally just added the text. It would be good to automatically add the constituency name. I think it's in flask.session['constituency'] but wasn't sure.

Let me know if this is ok or needs modification.


Cool, it worked.


Actually, these library files are not suitable for ARM, their arch is INTEL80386.


tested and working


`if self.watcher is not None` is much better IMHO


@mattcollier See 79d5649 - maybe that will help. Hopefully this commit comes up in search results for that error, too.


No.
But in my environment I have created a user with these caps and to try to run this image I had to do so.


Made the changes.

On Wed, Mar 11, 2015 at 1:39 AM, Luca Barbato <notifications@github.com>
wrote:

> You can use the generic rules (.o.c:) for that =)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shreshthamundra/avahi_chat/commit/502ac8a5cde64339385f428b121181317ea76976#commitcomment-10128681>
> .
>


@ZimbiX :+1: :beers:


ok


No more progress as for now


@ZimbiX moved from Ruhoh to Jekyll, though. what can I do, I'm a chronic blogging engine hopper.


this will make who adds `actionpack`, `4-2-stable` to fail, as `rails-html-sanitizer` is not yet released.
cc @tenderlove @rafaelfranca @kaspth


As far as I am aware there isn't any benefit other than grouping in the final HTML. The benefit of calling ```@Scripts.Render("path/to/scripts/bundle")``` as opposed to hard coding a script tag like so ```<script src="~/Scripts/bootstrap.min.js"></script>``` is that way during debug it will return each script in the bundle separately and unminified while in release mode it will combine all the scripts into one file and then minify it in order to have faster page load times (as it will only have to load one file per bundle as opposed to many script files). Though I don't find our pages loading slowly so really I don't think we need to really care about bundling if we don't want to.

Referring to "nice catch" it wasn't really as I was attempting to do Login code and the view wouldn't render. So it was more of the framework "catching it"


Should also the `LANGUAGE=C` part be copied to `abc.ly` and `toly_dialog.py`?


Responded in issue #234. For proto3, this is correct: only message fields have `has_...()` methods. This was an intentional design decision by the proto3 design team. Note a few lines up that for proto1/proto2, all fields have `has_...()` methods.


jshint можно настроить, чтобы он не ругался на такие вещи, ибо если ты работаешь на клиенте, и на нем не будет доступен window - значит что-то идет не так


That's fine by me.


You should merge this into main OpenXCom. Training facility code for new facility type :)


I think 512 would be safe here.  I know I wrote this...


> Will do later.

Done.


Excellent!! Thank you very much. In which version is available this functionality?


Will this provide a way to stream "hidden" channels, ie ones in the MUX with only a Video and Audio ES but no PMT?  Perhaps even a way to stream a single IP data PID to a pipe where it can be processed by an app that dumps the video files out of the data stream?  Currently using dvbstream to stream these types of PIDs, but tvheadend has to be shutdown to allow dvbstream to access the dvb adapter.


OK, mas precisa implementar p LocalizableString para usar no código, da forma como está só funciona na interface usando Storyboard ou Xib


Grade: 2/2


This is for the vast which uses the old filterSources signature.

androidhls should be normalized to sources or playlist items. We have a ticket to do this for all global config options that apply to playlist items.


Beatriz Carretero Parra


If you're changing this:
"Convert the possible returned packet into an SCP packet.  If the sequence number matches the expected sequence number then the acknowledgement has been returned."


Hrm... imperically this seems like a divergence with geolocate in the Google API.

When I switched the top level radioType to use the TelephonyService phoneType - I get worse results. This looks like geoip.  So yeah, let's just keep using the same radioType as the first reported record.   cc/ @hannosch

```java
I/Stumbler_StumblerBundle(25742): Geolocate JSON: {
I/Stumbler_StumblerBundle(25742):   "timestamp": 1426018981291,
I/Stumbler_StumblerBundle(25742):   "longitude": -79.490288,
I/Stumbler_StumblerBundle(25742):   "radioType": "gsm",
I/Stumbler_StumblerBundle(25742):   "latitude": 44.033571,
I/Stumbler_StumblerBundle(25742):   "cellTowers": [
I/Stumbler_StumblerBundle(25742):     {
I/Stumbler_StumblerBundle(25742):       "asu": 10,
I/Stumbler_StumblerBundle(25742):       "psc": 122,
I/Stumbler_StumblerBundle(25742):       "cellId": 854569,
I/Stumbler_StumblerBundle(25742):       "radioType": "wcdma",
I/Stumbler_StumblerBundle(25742):       "mobileNetworkCode": 490,
I/Stumbler_StumblerBundle(25742):       "mobileCountryCode": 302,
I/Stumbler_StumblerBundle(25742):       "locationAreaCode": 16010
I/Stumbler_StumblerBundle(25742):     }
I/Stumbler_StumblerBundle(25742):   ]
I/Stumbler_StumblerBundle(25742): }
```

![2015-03-10 16 25 50](https://cloud.githubusercontent.com/assets/4829/6584500/400fbdee-c742-11e4-8c97-a5e0a00ee3bb.png)


90022444-FewReleasePrepThings (Myoldmopar) - 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)


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


Can we add an optional argument to addjob that allows keeping the past behavior? I really need addjob to be asynchronous. Or we could add an asyncAddJob?


90022444-FewReleasePrepThings (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1653 of 1653 tests passed)

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


Just to clarify - using the "lte" radioType gives better results.


Can we clean this up a little bit? Repeat over button, make the text come from the type?


Actually won't this commit make it impossible for more than one thread to be active at a time? I think it will as adding a job forces any current job to complete. So say you add N jobs, then each call to addjob will force a wait for the previous job to complete. So N threads would really mean 1 thread?


@sethlyons yeah, unfortunately documentation updates are probably the best bet here. Do you have the bandwidth to submit a PR for that, since you're probably better versed in the issue and how to resolve it. Thanks!


:hatched_chick: hatched_chick


If I understood how this works correctly the only place where we do a query to db to get translations is already before fill_in_delta is called? Then shouldn't we ensure we always include the fallback_locale to locales before the query or otherwise the possible fallback data is not there when processing the result set to fill in fallbacks?


> Button doesn't turn green when I edit some settings

@Dashed, that's a bug. Which settings?


TeamCity Helios :: Helios PR Build [Build 8](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=189&buildTypeId=Helios_HeliosPrBuild&guest=1) is now running


Para que serve esta String?


Why?


Grade: 2/2


Conferido e OK


GridViewBugFixes (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2165 of 2184 tests passed)

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


added a new pull request with the --no-gpgcheck.


Firefox pukes if it receives a relay candidate that is missing raddr and rport, this patch fixes that issue.


Skhorn, colleague, glad you are alive. Things don't want to work as quick as we want. I wonder how much we can raise the efficiency. Probably every few moments children starve ... and I'm not getting my boots up.  :crying_cat_face: Must accelerate ...


NO


TeamCity Helios :: Helios PR Build [Build 8](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=189&buildTypeId=Helios_HeliosPrBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 107, ignored: 19 Build time: 00:01:46


This is going to double up padding with a nested `code` tag


NO!


@cespare Should be good to go.


@siewertj  As you can see, i've tried out sandman working off of a simple database. Give it a shot (i won't rebase this branch until you've tried it). Hopefully the instructions are good enough to get you started. If not, let me know.


@bclymer will not approve!


Grade: 1.5/2

Can't have two migrations with the same name (CreateCats)


Все гуд :+1:

Но есть одно замечание.
Я могу сейчас записать для ленивых вычислений абсолютно любые ключи, даже те, которые граф не вычисляет :(
````
console.log(lazyStats({
        xs: [1, 2, 3, 6]
    }, ['n', 'n', 'd', undefined, console.log]));
````

![2015-03-10 23-35-32 graph](https://cloud.githubusercontent.com/assets/1178825/6584715/2d52ce62-c77e-11e4-8d3a-fe7eb4e91c90.png)


But the maintainers of the sqlsrv and pdo sqlsrv extension did not pull it. I then looked in my dev environment and even VC++ 6.0 (VC98) has this in C:\Program Files (x86)\Microsoft Visual Studio\VC98\Include\ODBCINST.H:

	/*--------------------------------------------------------------------------
	  ODBCInst.h -- Prototypes for ODBCCP32.DLL

	  (c) Microsoft Corp., 1990-1998
	--------------------------------------------------------------------------*/

	#ifndef __ODBCINST_H
	#define __ODBCINST_H

	#ifndef __SQL
	#include "sql.h"
	#endif

	#ifdef __cplusplus
	extern "C" {                               // Assume C declarations for C++
	#endif	// __cplusplus

	#ifndef ODBCVER
	#define ODBCVER 0x0351					   // Assume ODBC 3.51
	#endif

	#ifndef WINVER
	#define  WINVER  0x0400                     // Assume Windows 4.0
	#endif

I think it is safe to follow that example and make the default value in this case 0x0351 as well. I could only find higher values, for instance 0x0380 in the Windows SDK's v7.1A, 8.0 & 8.1.


> jshint можно настроить, чтобы он не ругался на такие вещи
ага так и сделал


On Vagrant in the past, we did not declare the cloud in the grains.conf on the minions.

This broke that assumption.

Fixing that up now.


Conferido e OK


Not pretty indent LOL


When I did the find closest in JavaScript
I found centroid I believe of region.

This would turn the regional polygon into a point... You could create a
collection of points and then easily use the built in turf js function for
closest target.

On Tuesday, March 10, 2015, Mike Loffland <notifications@github.com> wrote:

> @joekarl <https://github.com/joekarl> Is this code functioning? It seems
> like it would make more sense to use points instead of polygons.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/codeforokc/school-finder/commit/ae240dd08d76fb828b300e0e57d1ba80f4f80615#commitcomment-10128016>
> .
>


This makes a lot of sense. Thank you. How to I do merges?


If you’re tracking the current master branch (you are if this commit is
part of the repository history) then `git apply file.patch` should work
fine (i.e. the patch applies cleanly on top of that commit). But I guess
you’re not using the commandline git binary, are you?


A forma de fazer o bronze 3 está incorreta. Deve ser corrigida


Hi,

Thanks for the this useful extension. I just thought I'd mention that its not necessary to add minor versions. `3.15` is sufficient for all `3.15.x` in the series.  :-) Cheers!


I am not sure if all the other providers provide a cloud value on their minions grains.conf


Se ve padre la notación de lambda "->"  :+1:

Que quede este comment para la posteridad :stuck_out_tongue:

Rails 4.0 requires that scopes use a callable object such as a Proc or lambda:
```
scope :active, where(active: true)

# becomes
scope :active, -> { where active: true }
```


I would suggest being much more friendly and explicit.  I should probably rewrite this if I'm going to voice my opinion here. maybe tomorrow.

just some comments here:

"predefined" doesn't make so much sense as a leading statement, its basically an implementation detail.

for the user these do not "effectively represent global state" they are global variables. plain and simple. everything that is not in a class is in the interpreter's scope, so these are global pretty much everywhere.

say that all variables need to be defined eg. `var xyz;` except these ones that are predefined.  at that point it makes sense to mention that they are predefined.


Now getting an error compiling the vsix project: C:\Program Files (x86)\MSBuild\Microsoft\VisualStudio\v14.0\VSSDK\Microsoft.VsSDK.targets(502,5): error VSSDK1031: Extension 'CodeCracker.Vsix..5b99e64c-1418-4a06-990c-fd4cf01f1234' could not be found. Please make sure the extension has been installed.


aws does not provide a grains.conf entry for cloud on the salt-minion
azure does.

i will update vagrant and aws to always include the cloud.


Ce ne serai pas plutot : forceAddUniform ?


@mabrand I'm not sure if that's possible. I do know we can use GMSL's `substr` function:

```make
$(call substr,deedbeefff1,1,7)
```


Grade: 1.5/2

To display all objects of a table after assigning @variable = Table.all in the controller, you can't just call <%= @variable %>. Your view ends up displaying "#<User::ActiveRecord_Relation:0x007fec4ea18820>", which is what happens when you try reference a model in a view. Instead, use a loop to iterate through all elements of a table, such as <% for @variable.each do |var| %>.


take that back, it was just vagrant that did not.


Maybe, `STIC_CAT(too_many_lines_in_file_, __FILE__)`?


Oof. Sorry, it seemed like a safe check.


@Schmoozerd I tried to remove the CAST_INTERRUPT_PREVIOUS flag, but there is still a problem.
It seems that the boss will still interrupt the channeling despite the fact that it should have AURA_MOD_STUN.

Also, the patch proposed by @kvipka won't apply to this spell (but might be good for others)

```diff
diff --git a/src/game/SharedDefines.h b/src/game/SharedDefines.h
index 26dd5f4..e7f4f51 100644
--- a/src/game/SharedDefines.h
+++ b/src/game/SharedDefines.h
@@ -429,7 +429,7 @@ enum SpellAttributesEx4

 enum SpellAttributesEx5
 {
-    SPELL_ATTR_EX5_UNK0                        = 0x00000001,// 0
+    SPELL_ATTR_EX5_CAN_CHANNEL_WHEN_MOVING     = 0x00000001,// 0 don't interrupt channeling spells when moving
     SPELL_ATTR_EX5_NO_REAGENT_WHILE_PREP       = 0x00000002,// 1 not need reagents if UNIT_FLAG_PREPARATION
     SPELL_ATTR_EX5_UNK2                        = 0x00000004,// 2 removed at enter arena (e.g. 31850 since 3.3.3)
     SPELL_ATTR_EX5_USABLE_WHILE_STUNNED        = 0x00000008,// 3 usable while stunned
diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index 18d6ec4..1bb6720 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -3564,7 +3564,7 @@ void Spell::update(uint32 difftime)
         (m_spellInfo->Effect[EFFECT_INDEX_0] != SPELL_EFFECT_STUCK || !m_caster->m_movementInfo.HasMovementFlag(MOVEFLAG_FALLINGFAR)))
     {
         // always cancel for channeled spells
-        if (m_spellState == SPELL_STATE_CASTING)
+        if (m_spellState == SPELL_STATE_CASTING && !m_spellInfo->HasAttribute(SPELL_ATTR_EX5_CAN_CHANNEL_WHEN_MOVING))
             cancel();
         // don't cancel for melee, autorepeat, triggered and instant spells
         else if (!IsNextMeleeSwingSpell() && !IsAutoRepeat() && !m_IsTriggeredSpell && (m_spellInfo->InterruptFlags & SPELL_INTERRUPT_FLAG_MOVEMENT))
```


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2225](http://teamcity.monogame.net/viewLog.html?buildId=5656&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 441, ignored: 6 Build time: 00:13:57


@BonsaiOfHappiness Das ist übrigens ein relativ großes Bild (sowohl subjektiv nach Höhe als auch objektiv nach Dateigröße) – ich hatte z. B. in 70c9b408 das vorige Bild durch eine halb so große Version ersetzt, die immer noch größer war, als im Feed angezeigt wurde (und trotzdem nur 1/7 der Dateigröße hat von dem Bild, das du gerade eingefügt hast)

Vielleicht stört das schon den einen oder anderen Landnordfriesen, wenn nur dieses eine Bild schon 10 Sekunden zum Laden braucht ;)


Ah I see, okay!  Well if it offers us that optimization and it is easy to do, we might as well.  I will utilize this now.  Thanks again. :)


Fix the spec, please.


I thought I had to provide custom bgfx.lua due to config names mismatch and other stuff; it is easier than expected.


Why not?


Grade: 2/2


gostei


Making a utility function would be incorrect, the code is different in those two locations.

line 395:

   field.value.type.base_type == BASE_TYPE_BOOL
                ? (field.value.constant == "0" ? "false" : "true")
                : field.value.constant;

new code proposed for line 511:

if (lang.language == GeneratorOptions::kCSharp)
   {
    code += field.value.type.base_type == BASE_TYPE_BOOL
     ? (field.value.constant == "0" ? " = false" : " = true")
     : " = " + field.value.constant;
   }
   else
    code += " = " + field.value.constant;

Note the differences highlighted (in blue).

I'd rather not change the logic in the other location.. didn't test/examine.

I also would rather not remove the kCSharp check.... because then I'd have to test to make sure it works for all other languages... And it works as above for C++

I can fix the indents to use spaces instead of tabs.

-Mark

________________________________
From: gwvo [notifications@github.com]
Sent: Monday, March 09, 2015 4:19 PM
To: reynolma2/flatbuffers
Cc: Mark Reynolds
Subject: Re: [flatbuffers] Update idl_gen_general.cpp (48e55af)

Thanks for catching this!
Can you do this before I merge:
1) pull the code at line 395 into a utility function and use that instead to output true or false.
2) probably remove the check against kCSharp, since a future language is more likely to support these keywords than not.
3) ensure that your code uses 2 spaces for an indent.

—
Reply to this email directly or view it on GitHub<https://github.com/reynolma2/flatbuffers/commit/48e55af4bc02e7299029458a6ac9ca14c2b9b854#commitcomment-10110879>.


Conferido e OK


closes #6


But you need to adjust orphaned.next.pred = null;


I was using the bbox coordinates to determine the centroid (on the client). But, it would be more efficient to have this data already determined by a process on the server. Which is what Joe seems to be doing in this branch.


Согласен, баг! Исправлено https://github.com/fib0n/graph/commit/72643f2486f22e1340030b7986aa8e9cab4260e7


This is great that you were able to customize things, but let's move the grant acknowledgement text into the config file as a field so that we can keep the Cusomizer code generic.


what a convoluted boolean operation. Would you mind splitting it up a bit so its clearer what it is trying to check?


I've been using SVN to keep a local copy of the master branch, thinking that would always be current when I run 'svn update {path}'.
But I will try the same with git and see if it works.


`<model>s_browse_default_sort` filters only the *default* sort, so it only applies when there's no sort param in the URL

`<model>s_browse_per_page`, I'd just mention that its the number returned *per page*

Note: despite the release notes, there are more than just these 2 added filters. I'll actually be taking the mentions of them out from the release notes and just linking to a What's New in Omeka 2.3 on the readthedocs


The normal python way to get parent directory of current file is using `__file__`. This would be cleaner as e.g.
```
filedir = os.path.dirname(os.path.abspath(__file__))
sys.path.insert(0, os.path.normpath(os.path.join(filedir, "../../third_party/PyYAML/lib"))
```


Why so complicated? Why not open the Pixel Perfect window via `window.openDialog()` and the "resizable" option [like the FBTrace console](https://github.com/firebug/tracing-console/blob/06519661e5340a675ace395d27f9749cebc6e61a/chrome/content/traceModule.js#L137)?

Sebastian


Could also do:

```yml
buttons:
  button: Click Me
  submit: Submit
  reset: Reset
```

```html
{% for button in buttons -%}
<button class="base--button" type="{{ loop.key }}"> {{ button }}</button>
{% endfor %}
```


Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=27825


Which would you prefer to do?


detrlet


This came back in a few files. Reason is, we pulled #870 before this one, so obvisously missed some now. Would this be fixed by later locale updates? Otherwise those few should be changed manually.


Not a big deal, but this could conceivably be
```
 let profiles =
     users.map(user => userUtil.getGithubUser(user.login).then(profile => user.name = profile.name));
```


    but if we can avoid running the webhook as root, that would be great.
Which webhook?


Will this affect our capacity to do rolling upgrades? or rather, does this imply a very specific order in which nodes must be updated to prevent errors?


The pattern I represented is more consistent with the one's we've previously used (i.e. inputs). Its more scalable if we want to add any additional attributes as well


Bah


would be good to have a comment right here to say that the user will most likely need to overwrite this ```LOCAL_CONFIG_DIR```


Preferably both of them. Presumably git is secure enough to run as root [1], but if we can run it as www-data instead, I think that's probably preferable.

[1] with a case-sensitive filesystem, anyway... heh


Like, if you want to take out type=button you can do that easily


and a comment for this line explaining that the paths will need to be modified.


0 было выбрано, т.к. тип size_t, сходу не понимаю, как туда запихать что-то другое.


Ah, okay.  Thanks


I know, it might look as if it was part of the message, but it's not (better wording might be needed). `__LINE__` is appended to make that name unique, otherwise two tests will define same array and compilation will fail. `__FILE__` won't make it unique and in fact won't work as it contains symbols like `.` or `/`.


But if run the www-webhook as www-data, then the files for the main site will also have to be owned by www-data, unless you have a different self-consistent suggestion...


A posição '0' do vetor midias não será exibida.


that's insane. it must happen.


Não seria requirement failed: There is no user named andryw ?


Nope, You own the code and you are licensing it to us, so if you use it somewhere else then you can license/attribute/etc anyway that you want, but by licensing it to us you are aware that when someone references or uses it via safecast they are saying it’s from safecast, not from any individual. It’s the same policy with the data, people contribute readings to the safecast dataset knowing that when someone references the safecast dataset they aren’t going to name every single person who contributed a reading, they are just going to say it’s safecast data. But anyone who has taken readings could just as easily do anything else with that data in other venues.

What this all prevents is someone contributing code or data or anything to safecast and then coming back years later and saying safecast can’t use it.

> On Mar 10, 2015, at 11:16 AM, Nava Whiteford <notifications@github.com> wrote:
>
> Hi Sean, this is fine but just FYI
>
> "all the world under these same licenses. Safecast will be attributed as the source."
>
> reads like I wouldn't be able to use code I've written in my other personal projects without attributing Safecast. Nor would I be able to say, use code I've written for Safecast in another project under the GPL. Or use code I've previously released under the GPL in a Safecast project.
>
> In this case I don't mind, but it seems slightly more restrictive than necessary.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Safecast/bGeigieNano_ESP8266/commit/41a89a13efc5f2ebe649ad59b59828fc5eb3723e#commitcomment-10126457>.
>


Makes sense. Changed on my local branch.


this patch should be added for compile on arch arm
http://download.digital-devices.de/download/linux/octonet.source/dvb-apps-0002-Fix-broken-bitops-on-arm.patch


No, I agree with your assessment. Like above, I don't think it's particularly important who owns them; I'd prefer root, but not at the expense of the webhook running as root. I don't think there's a good way we can have them owned by root (without the webhook also running as root), so www-data is fine by me as a compromise. Unless you have another idea?


Saint-Gilles-Croix-de-Vie


@keang
I tried to include this to not take the snapshot when the url is the message snapshot and there is not the message div, which means that is the error page. Doesn't seem to be working though, can you help me testing (and fixing)? I spent my whole afternoon on those few lines of code =/
Thanks man!


Some comments


This is IntelliJ-generated code. It can sometimes generate funny looking code but I'd rather just let it do its thing.


sdhfksdf


Ambitny plan, ale będziemy próbować :D


There are two config files, one for qt and one for glfw executables of citra - make sure you change the correct one. Also in this branch sw and hw renderers should look roughly the same, only the hw has ~2-3x the framerate - nowhere close to the level of speedup/HD you'd get with the proto branch.


subnetId = null should work fine, at least based on my reading of RunInstancesRequest, so the upgrade plan would be to not set any non-null subnetIds until things are on the new version.


Falta testar se encontrou um usuário que deveria estar, se não encontrou um usuário que não deveria estar na lista =P


this is not compiled for arm, at all. would you like to take over the maintainership?
if so, please send the patch as PR, and make sure it builds and works on "all" supported platforms you enable in PKG_ARCH.


This change is correct.


As i wanted to merge your code i realized that it is provided under the 3-clause BSD license. The rest of the plugin is however licensed under GPLv3. As far as i researched the project can contain source files which are covered by another license. I just wanted to pose this question (Can the project contain source files under the BSD 3-clause lincense and GPLv3?) to be asolutely on the save side before i merge.


Strangely enough, it is still killed.
I will look for more clues, since Travis doesn't think it's good enough.


A comment


Done with `find . -name '*.md' -type f -exec sed -i '' -e 's/investigativenewsnetwork.org/inn.org/g' {} \;` and some manual cleanup.


"toontown.cog" yeah


90022444-FewReleasePrepThings (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (637 of 637 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-4.19%25-green.svg)


How was this an improvement? :)


On that note, I bet you have your hands full with other aspects of this project, and I seem to be turning into the "Path of War Guy" anyway.  If you'd like to start forwarding PoW emails to me and/or listing me as the support contact for PoW stuff, I'd be happy to take that off your plate.


?>>> What is this?


@Wobel If you're gonna add stuff here, the room below Ohmna is called "Bio-Phage Containment Pools" -  the room upstairs is called "Techno-Phaged Laboratories". I'm not sure if you need both of these or just one. *Ideally* you just want the Bio-Phage one since that would truly restrict it to only the ohmna room. However, since that SubZoneChanged event would happen during the jumpdown into the pit, I'm not sure if the module would activate before combat with Ohmna is engaged. If the zone is changed after the combat starts, it will load the module but it won't start the encounter :(

Convergence hasn't been added yet to the restrict/enablezone list because I don't know the name of the subzone that boss is in :P


I think you meant to delete this version of PageDispatch, being that it's in a file call DiskDispatcher.


This is why we need mock tests for every method. Isn't there a StorageAccountApiMockTest class? If not, we should add it.


oops didnt see that typo


Another comment


Ok, I have to admit I have no clue what is the proper way to convince git to just bring me the latest copy of your code. I tried following the how-to, created my own fork (seemed unnecessary) and it still seems convinced that I want to give you code - nothing explains how to just get all the latest.
(I'm probably just too old-school for this)
Gonna just download a ZIP copy and work from that.


In case you noticed, these changes didn't go to the github page because you didn't build and commit all.css, I'm commiting a build with this now though


Copypasted from PsGithub :)


yes.... let me git rm that


Input textboxes and pretty much all toggle buttons.  Interestingly, double clicking on labels associated with toggle buttons forces the button to be green; even if nothing changes.

Using 42.0.2311.22 (Official Build) beta (64-bit).


The user will supply a pickler for their record at the same time the supply the record.  The interface for record, from Disk.scala, is:

      def record [R] (desc: RecordDescriptor [R], entry: R): Async [Unit]

The descriptor has a pickler inside it. You need to bind the pickler and entry, and then send that through the dispatcher.


En realidad estaba probando algo como esto https://gist.github.com/jaryl/1750858 pero me da un error Unable to find css "#user_role_chosen" y si existe


maybe make a temp var `hasGlob` before L381


Should it be record or write though ?


Let's have a blurb about the difference between Rainbow and Old Rainbow.


great, I figured that was likely the intension but wasn't 100% on the wording. Thanks!


Brief comment about the range of RGB colors that Blackbody uses?


Good


I see, I just compared it with `STIC_STATIC_ASSERT(too_many_lines, __LINE__ < STIC_MAX_LINES)` below - looks different.


to replace `wh.hasGlob`? why?

If you mean for the result of this entire operation, I don't want it to execute unless it really has to


I've find the following doc: http://www.editpadpro.com/manual/filetypeeditor.html
For Text document file types, you need to uncheck the box "Pressing tab insert spaces instead of tab character"

Also there seem to be a functionality to convert spaces to tabs http://www.editpadpro.com/manual/convertspacestotabs.html, can probably be useful to repair.


True, my bad :) will fix now


Ok, forget it

When you click a second time on the "edit" button or the "delete" button, this cancels the action. that's good !

One improvement : can you send all the data to the server (not just id) when we click the "restore" button, so that the server can reinsert the previously cleared row.

thanks

Sorry for my english, i'm french


Sure thing, it’s the same policy we’ve always had, I just had some lawyers look over our previous wording and they felt it needed a little editing to ensure it covers the bases it’s supposed to.

> On Mar 10, 2015, at 2:06 PM, Nava Whiteford <notifications@github.com> wrote:
>
> great, I figured that was likely the intension but wasn't 100% on the wording. Thanks!
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Safecast/bGeigieNano_ESP8266/commit/41a89a13efc5f2ebe649ad59b59828fc5eb3723e#commitcomment-10129845>.
>


TeamCity Helios :: Helios Unstable Latest [Build 3](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=192&buildTypeId=Helios_HeliosUnstableLatest) is now running


redundant boxing.


@snopoke I think that if we didn't catch this exception then the pillow error/retry mechanism would catch this, but since we catch this it never gets to our pillow error/retry framework. Am I right in thinking that I should just remove the try except here (or add a bare `raise` to the except)? @biyeun FYI


static imports for check isEmpty & isNotEmpty Strings.


Use constants not local String.


TeamCity Helios :: Helios Unstable Latest [Build 3](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=192&buildTypeId=Helios_HeliosUnstableLatest) outcome was **SUCCESS**
Summary: Tests passed: 107, ignored: 19 Build time: 00:01:21


TeamCity Helios :: Helios PR Build [Build 9](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=193&buildTypeId=Helios_HeliosPrBuild&guest=1) is now running


User static imported methods.


@esbrandt: Please give this a try. Thank you.


For my app, `AuthFeature` is not listed inside `HostContext.AppHost.Plugins` (which is what `HostContext.GetPlugin<T>()' checks), but it is listed inside `HostContext.AppHost.PluginsLoaded`.


Which is not accessible from within the iterator. I think your code wins :)


In this case I was trying not to change signatures, but it really isn't a big deal. Eventually digest should replace everywhere we use sumtype/sum.


looks good


Should we change `xsd:sequence` to `xsd:all`?


TeamCity Helios :: Helios PR Build [Build 9](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=193&buildTypeId=Helios_HeliosPrBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 107, ignored: 19 Build time: 00:01:16


According that Controller is singleton- we should use static variables.


another possible improvement ; if "settings.onAjax(action, serialize)" returns false, don't send the data to the server.


GridViewBugFixes (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)


Typo: "respinse" -> "response"


it will return the original string and a validation error.


@eureka2 the delete button isn't actually a delete, it just changes an value in your SQL.
The restore does not remember the data posted by you earlier.

From what i see, the delete and restore is used something similar to show/hide , as the data is not actually removed from the SQL .


These are two different, unrelated changes -- please make separate commits for cases like this.


How did you register AuthFeature?


woops


@AnisGitHub fyi... i think you accidentally reverted back a change.


Hi Steve, I'm having a problem with this section (see my comments).  Its probably a syntax thing.  What I am doing here is trying to iterate through each dictionary stored in self.searchResults.  I want to create another dictionary that adds distance as a key.  I am trying to use fast enumeration.  When I run the code below (that I found online), it tells me that the items at the index row are not NSDictionary objects, however, if i change the code to be NSDictionary *location in self.searchResults[0], as a test, it works.    Obviously this doesn't work though, because I am not enumerating through all the items.


Even better would be to put the empty check in the sql so that we always have 5


Nice, better indeed.


@AnisGitHub on this commit you revert back... i'll change it back.


This field is just the Hex part of the digest string.


@gorhill can you please take a look at [CONTRIBUTING.md](https://github.com/gorhill/uBlock/blob/master/CONTRIBUTING.md) and let me know what you think?

What's also neat is that the [New Issue](https://github.com/gorhill/uBlock/issues/new) page links to CONTRIBUTING.md.


Note, function should be called `parse_semver`, right now it's broken.


:+1:  Все ок


Ouch. I wouldn't even notice it.


I think it's not ok here.


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)


@ljorr1 You can run jekyll locally.


Rada have you consider coming back? It would be cool if you carry out with
the stuff you left pending in CoM and besides it would be cool to have you
back. Consider it if something comes up, we will talk

2015-03-10 15:33 GMT-05:00 J. R.I.B.-Wein <notifications@github.com>:

> Skhorn, colleague, glad you are alive. Things don't want to work as quick
> as we want. I wonder how much we can raise the efficiency. Probably every
> few moments children starve ... and I'm not getting my boots up. [image:
> :crying_cat_face:] Must accelerate ...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/0ADMods/eyecandy/commit/8577325bc1ebd7018a8f0cf43edae1255375775f#commitcomment-10129167>
> .
>


Thank you. I have fixed this issue.

Jason D. Yeatman, PhD
Research Scientist
Institute for Learning and Brain Sciences (ILABS)
University of Washington
http://jasonyeatman.com

On Wed, Feb 25, 2015 at 1:19 PM, Shumpei Ogawa <notifications@github.com>
wrote:

> When 'tube' is not 1, this part cause error. Because there is no 'h'. For
> example running AFQ_MakeFiberGroupMontage. So i think it would be better to
> simply skip this part if tube is not 1.
>
> if tube==1,
> for ii = 1:length(h)
> fvc(ii) = surf2patch(h(ii));
> end
> end
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jyeatman/AFQ/commit/22b70038f3f5dd7eced88638e432cc705af2206f#commitcomment-9923689>
> .
>


So @yhahn these are (expectedly failing), and not (necessarily) due to the fact that I am comparing with the wrong expecteds. They seem to only fail at the image comparison point.
I am testing locally w/ mapnik@3.2.x.

 - I did some poking around, and these are being loaded as pbf (which is correct, right?) - but, it does not seem like `vtile.setData` is the right method: https://github.com/mapbox/tilelive-vector/blob/master/backend.js#L152-L153
 - Referencing the above, I made these tiles via this method, where I used `.addImage`: https://github.com/mapbox/vtfx/blob/raster-query/raster-tests/index.js#L8-L19

Going to keep poking a bit, and possibly testing some tiles on staging.


If the data is stored in Google Analytics in GMT (which I highly recommend that you do), then when you say "today" it may be tomorrow in GMT.  By changing the query to start at yesterday and go to today, you make sure that you're not caught by the GMT time zone difference.


Oups, thanks for catching it, updated in db334f0.


You could have pulled it off with one method by providing defaults.

public static IList<SiteEntity> GetSiteCollections(this Tenant tenant, int startIndex = 0, bool includeDetail = false)


For some reason, it was working in a locally hosted server, but on the BCCP site, I had the same experience as you with both FF and chrome! No idea why the SVG wasn't working. I changed to a 2x resolution PNG in my latest commit.


If a persistent store is needed, I'd recommend something like Redis.  The data stored is so small that an in memory key/value store like Redis is much better and quicker for this than something big and bloated like Mongo.


Sorry, it was an accident.


At the moment – and very much so when this commit was made – Julia's C code is not remotely threadsafe, so making this static was not giving up anything. This will have to be modified when we make it threadsafe, but this isn't the hard.


See also: https://github.com/wycats/handlebars.js/issues/978


What is the purpose of this file?


Was using it, but incorrectly apparently, so my copy wasn't tracking your latest.
RTFM'ing now...


no worries... I changed it back... easy.


I reverted this key change, any reason why you changed these locale keys?


Lol, autocorrected to cosmic for me the other day as well :frowning:


Horrible commit message due to failed fixup on cherry-pick with `git rebase -i HEAD~2` :/


Hey, looks good. Thanks for the creative writing assistance. :)


Looks nice. I had no idea this could be done this way with Github, I was lamenting the fact we could not pin a guideline post at the top of issue, but this is what you did.

I was thinking.. we could give users a tool to gather the information, i.e. programmatically generated into the content of a read-only text field in the _About_ pane of uBlock, and which could be easily copy-pasted into a Github issue. At least for all information which is not privacy-sensitive: uBlock version, OS, browser, selected stock filter lists, settings. For privacy sensitive information though, a user will have to provide it explicitly.


"define the source/style together before initializing the map" -- not sure exactly what your build process does but I think this is equivalent to "writing a custom style" which will certainly be supported long term.

I just added `map.getSource()`.


Shouldn't this be passed  &plain->value[plain->length] ?


Why cipher->length += TAGSIZE here ?


Jeżeli gracz posiada uprawnienie X, ale nie ma opa (przykładowo moderator)? :worried:


Make this regular godoc "StsrtReaper has no effect on Windows".  Actually, eventually we will reap on other unices.


Fine to make it Linux only for now


It's nice to be able to implement `fmt.Stringer` for other reasons. Users of the package can get the string representation of the digest without having to know this implementation detail and coerce it themselves.


+ if(requestContext.getUriInfo().getPath().equals("login")) {
....
}


Criação do novo arquivo.


Change color of Enemy


@gorhill I thought about that, as well — another thing that's relevant I was thinking about it in is Safari usage statistics. The Chrome Web Store and AMO both provide built-in usage statistics, but there's no such thing for Safari.

I was thinking that to see what versions Safari users have, etc., I'd have uBlock inform the server of just its version once it's installed (just once) with a randomly generated identifier (it doesn't in any way identify the user; just provides a way to track how the updates are rolling out).

In any case, I think that's a good idea.

Random thought: would it be useful to have uBlock inject a content script into its GitHub "New Issue" pages, where it would automatically initialize the textbook to have that information?


mach mal nen pull request, das gefaellt mir stellenweise


:+1: thanks @dethi , looks nicer!


@deserat Now it is.  It's currently untested on those platforms.  I can do the Ubuntu one with Lionel... any tips on a disposable Red Hat-based server?


Ok, so this sounds like this would be easier solve by adding a special attribute similar to `imports`, `config`, and `options`.  Thus we will avoid the `scopedImport`, and we could use it for `submodule` options definition of nixops.


@DanieleTorino This is now fixed in 4.0.0.CI-SNAPSHOT and 3.2.7.CI-SNAPSHOT. Can you verify the issue is resolved?


Should this be route('songs')?


90022444-FewReleasePrepThings (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1146 of 1146 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.63%25-green.svg)


Warum ist die Toolbox Klasse verschwunden? Die jetzt muss man die gblAddComponent in jeden View implementieren...


this seems unnecessary


88734010-AFN-horizontal-openings (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1147 of 1147 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.64%25-green.svg)


Is the `f` here a typo?


Why do they need to be prefixed separate from Screendoor's variables? Shouldn't $fr_error_color be the same as Screendoor's $errorColor? I would prefer that we work this out together.


Shouldn't this be changed not to open a new page, @nick?


Yeah, actually, can you just please hold on this?


Agreed .. cruft


@WrinklyNinja This exists on Line 708 in dev : https://github.com/wrye-bash/wrye-bash/blob/dev/Mopy/bash/bish.py#L798

Should that be changed?  Is it used anywhere?  Is that only used when someone is making a taglist by right clicking the File column on the Mods tab?  If I wanted to test this with the changes to the file names from .txt to .yaml, what would I change the regex statements to?


I'm not an unknown author.


try something like
'a[data-params="{\"appver\": \"any\", \"page\": null}"]'


> randomly generated identifier

No.

The tool I am talking about is merely to make easier for the user to fill an issue report, it still requires the user to wilfully cut & paste the information.

Anything beyond that which bypass a user **explicit** choice, I do not want in uBlock. The stats on Chrome store are merely for the number of users/downloads, I did not, and will not, enable analytics for uBlock, this is out of question.


develop (macumber) - i386-Windows-7-VisualStudio-12: OK (2161 of 2184 tests passed)

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


Yup, I just did that part.


Them line endings ;)


It's hanging around for now.


The previous access token was restricted to reading public data only...
However @github revoked it and I generated the new one.


ResultsSchema (sanyalj) - Win64-Windows-7-VisualStudio-12: OK (1639 of 1647 tests passed)

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


This is just personal preference :P


ResultsSchema (sanyalj) - 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)


Thanks for spotting the oversight in the README, @stof. One of the problems with working with many different languages :)


@gorhill

> Anything beyond that which bypass a user explicit choice, I do not want in uBlock.

Of course.

What I was really asking about is auto-filing that information for users when they're creating a New Issue. The information wouldn't be automatically submitted, just filled out in the issue description textbox for the user's convenience.

Either way, I think that having the basic info available in the "About" tab is a great idea — it's not crucial or necessary to have it intelligently fill the issue textbox for the user.


Actually this diverges from most of the type theory texts, so I'd prefer to leave it as the space.


Good catch!


Mmh .. correct me, but Step 1 should be:

Step 0: `if ((!utcb && !regs.fpu_on) || (!ec->utcb && !ec->regs.fpu_on))`
Step 1: `if (!(utcb || regs.fpu_on) || !(ec->utcb || ec->regs.fpu_on))`

or ?


Sad day :(


Is it the case that this always has to be true?


Thanks Scott, didn't think of that. I'll change that then.


Doesn't this alter the size?  Don't we want to not do that when we are doing the exact size thing?


This breaks linux builds..

```
checking for gettext... no
checking for gettext in -lintl...gettext function not found. Please install libintl at Makefile.PL line 18.
 no
builder for ‘/nix/store/100qdbh0ayxnhsk7cdxbnwgs6j5gdfza-perl-LocaleGettext-1.05.drv’ failed with exit code 1
cannot build derivation ‘/nix/store/2n206hn063vrh55fxg7cl1v2mabm6lfl-apparmor-2.8.4.drv’: 1 dependencies couldn't be built
```


I think yo should delete this blank space


I thought we got rid of all of the vestiges of cbufp?


Yes this is a typo. What is strange is that it does not cause an error. Both of these links still return the iframe content:
* https://vimeo.com/api/oembed.json?url=fhttps://vimeo.com/108141337
* https://vimeo.com/api/oembed.json?url=https://vimeo.com/108141337

I'll fix this.


GridViewBugFixes (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2171 of 2184 tests passed)

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


Remove note about upb -- we no longer use a submodule in the latest version (we just include `upb.c` / `upb.h`).


I can't immediately see why this is treated different from the other case that uses __cbufp_alloc_slow.


I don't really know what causes the bug. The display is all messed up when the game detects a screen size thats the iPhone 6 resolution. I just did a temp fix by adding 1 to the screen width until I can figure out what the actual bug is.


moment of silence for this project.               . she was a beauty - and we were totally on the right track.


ICSSolarCollector_#4723 (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1653 of 1653 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)


credentials shouldn't be commited to a git repo for security reasons (better would be to use environment variables, works also with heroku)


We don't want a commented line either.


awesome thank you...


There is also `devtools.lazyRequire` if you like that form, but whatever you prefer.


We know. It was fixed by c904f35728b (without a hash change on Linux from before this commit).


I believe jpm has [switched][1] to a much slimmer `bootstrap.js`, can we use it here?

[1]: https://github.com/mozilla/jpm-core/blob/master/data/bootstrap.js


Just noticed it, sorry


I agree it seems best to remove the GCLI commands and only re-add them later if they work everywhere.


I think you could organize this line like
driver.findElement(By.xpath("//div[@class='col-xs-8']/span[contains(text(),'"+ resourceName + "
                             ')and@class='ng-binding']/ancestor::div[@class='col-xs-8']/following-sibling::div//i[@class='fa fa-plus']")).click();
and why you have a big locacor? :o please verify if you could refactor this locator


:scream:


we like javadoc and comments :)


As suggested, I have changed the following status phrases -
"Pay Later (pending)" to "Pending (pay later)"
"Pending from incomplete transaction" to "Pending (incomplete transaction)".


What does this target do? Does it change the source scss files?


Should .settings have a / prefix?  (That would only ignore it at the top level.)
Should .project and .cproject work the same way?


ResultsSchema (sanyalj) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Coverage Too Low

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


Weird!


Great to have a glossary at hand, thanks @jmarlena. We'll translate it into Spanish as soon as it's merged :smile:


Hi Dimitri,  Sorry about that.  I just copied over the value Alexis put in DATA/Par_file.  For the broken examples, could it be better not to update the Par_files, seeing that there is no way to test them?


"performance/**/*.js" should already cover "performance/*.js".


As above


you can delete those comments because they are not given any useful information


https://msdn.microsoft.com/en-us/library/fs2xkftw%28v=vs.110%29.aspx

```
bool disposed = false;
protected override void Dispose(bool disposing) {
   if (disposed)
       return;
   if (disposing) {
       ReliableConnection.Dispose();
   }
   disposed = true;
   base.Dispose(disposing);
}
```


It's wrapped in a plugin for my own custom config options and loading additional related plugins to be shared with my multiple sites.
And it looks like this is precisely why it's not in the `Plugins` list - I call `LoadPlugin()` myself since doing `appHost.Plugins.Add(authFeature)` at this point doesn't get it registered.

I'll probably remove the `MyFeature` plugin and configure each site independently.

```
Plugins.Add(new MyFeature());
```
```
public class MyFeature : IPlugin
{
    public void Register(IAppHost appHost)
    {
        var authFeature = new AuthFeature(() => new MySession(),
            new IAuthProvider[] {
                new CredentialsAuthProvider(appSettings),
                new FacebookAuthProvider(appSettings)
            });
        appHost.LoadPlugin(authFeature);
        appHost.LoadPlugin(...otherPlugins...);
    }
}
```


:+1: that the bomb, dawg


hostProject?.EditAndContinueImplOpt?.onEdit(documentId);


@madhaviev btw, generated files (like compiled java classes) should be ignored and not be part of the version control


@mowchan DRY'ed the download and language reference pages.


Oh damn, that was a last second change I made, I just knew it was a bad idea to push that without running my tests again :/


I will try to sum up what I know in the issue tomorrow, we can try to find a solution from there. Consolidates are triggered on actions that affect the presence of secret keys, because those are trusted (thus relevant for imported keys)


Fixed.


Is there any case where we would want these committed?  If not, then no, we don't want the prefix.


fixed.


You should re-factor this method by changing the return type, exist a method in UIMethods class to recover if an item exist.


ICSSolarCollector_#4723 (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1653 of 1653 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)


improve this line like second comments


extreme ruby!


This is a good change but it didn't make it into the controller method. How is it supposed to be set?


same thing for the next 2 methods


We have very many of them committed in the java/jms and tools/modeling subdirs


cool, thanks for doing this!


Sure.


Do you mean the assert?
Now I think the allocation should always be successful.
If allocation can return NULL, I will remove those check.


:+1:  for writing tests


If there's a reason frameId isn't stored here, I would think you'd want to comment about it. I know the non-subprocess stuff doesn't currently expose frameIds, but it should in the future since that's how the V8 context is tracked. (i.e. via BrowserId and frameId)


That is a good idea. It would be indeed good to have more fields either in the UI or in the json file to customize the package.


+1 for removing compiled classes


Now it makrs the file offline...thats not true.

1010	10.03.2015 23:09:57	WARNING	Download is offline: 9564532565.part7.rar
1009	10.03.2015 23:09:56	INFO	FilerNet: Direct download link detected

Gosh...thats frustrating :-(


I was thinking we might want to add the general shipping methods as well in case there are any defined for the main shop? Or what do you think?


Keep this note about the upb amalgamation, sorry (this note replaces the earlier "check out the git submodule" note that was removed).


please reorganize this locator, you should not left the point at the end of the line


Because I want to avoid the collections.
We can only collect power-of-2 cbuf and put them back to the freelist.


@jamesdabbs Thanks James, I normally try to avoid config check-ins, guess I missed this one. Thanks for the Figaro point-out.


btw, you could add the link to the heroku app here and give a small hint how to run your app locally :)


this is awesome. Thank you :)


If there's a reason other than CefSharp doesn't currently expose FrameIds, then you should stick a comment here as well. If there isn't any particular reason, then shouldn't the parent/child messaging include the FrameId for completeness?


Yes. csscomb is like esformater it actually fixes any issues it finds.


It could be cool to have every change save automatically, so we could remove the Save button. Lots of programs do that these days, like the browser I'm using right now (Chrome)
What do you think @UniversalMediaServer/developers ?


Same comment here about FrameIds.


you can delete these comments, they are not given value information


added back :)


You cannot state with good confidence that cm is better than dh with only 207 games. (Komodo 8: 34.3% +-7%)
But the patch of cm against master is simpler than dh.


ResultsSchema (sanyalj) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


`is_file` should return trur for symlinks as well, so I believe the additional `is_symlink` check would never run


Copyright should be at beginning (it it isn't already).


> this is awesome. Thank you :)

TL;DR the fix we went with was selectively removing `use strict` from the affected modules (and the modules affected by jit inlining)

We currently regressed on canary due to a build tooling glitch that will be sorted out soon: https://github.com/emberjs/ember.js/issues/5606#issuecomment-77981953


#3. Commit was not referenced earlier.


please improve this method, there is something wrong with getting the resource name that you are given in prameter


>I did not, and will not, enable analytics for uBlock, this is out of question.

+1, that would certainly be ironic, to say the least...

>auto-filing that information for users when they're creating a New Issue...... basic info available in the "About" tab

That would be nice, specially the former. But please, this tool is all about privacy and user's choice, so developers should be really careful about anything that could be considered privacy invasion. I would personally feel comfortable sharing that info for debugging purposes, but that might not be the case for the really paranoid type of user (no offense intended).
So if you go this route, consider at the very least a big warning about sending the info, and ideally some community feedback. I've seen some unfair backslash and hostility towards uBlock in some places for no good reason, and would hate to see those arguments fueled.


A short comment on the idea behind this `IEndpointBehavior` would be awesome.


The next C# client release will use Path.DirectorySeparatorChar instead of hard-coding them.


I realize this comment is really late, but since the deprecation notice remains in the latest version, I think it's worth noting that wrap-file-info works very differently than wrap-content-type w.r.t. guessing the content type.  w-f-i guesses the content type based on the extension of the outbound file while w-c-t guesses the content-type based on the extension of the inbound URI.  Furthremore, w-f-i also adds the Content-Length header, which is not addressed by the two superseding middleware handlers.


you should left blank spaces above a commentary


same comment, you could use a method to return true or false when element exist or not


It's a little wacky for the **deferred** (instead of the promise) to be publicly available, but I suppose this is the easiest way to manage a global promise when the thing that will resolve isn't known to even be running yet.

There is a backwards compat issue here though: only recent Firefox 39 has `gDevToolsBrowser.isWebIDEInitialized`. I guess fall back to always registering if that property does not exist?


Capturing and sending `mdebug` only matters if you capture that as a flag.  Like this: https://github.com/dbashford/mimosa-bower/blob/master/src/index.coffee#L53


The two examples that have popped up for me so far are:
Xml for post data was invalid, Reference to undeclared entity 'oslash'.
Xml for post data was invalid, Reference to undeclared entity 'aacute'.
Am I right in understanding that line 43 (https://github.com/calcinai/xero-php/commit/ffa8afd151368995a933c0ae4a35af9866880728#diff-876615a06b982ae6334801bacf3c826aR43) will address these because the `&` part of `&oslash` will be converted to `&` and thus produce `&oslash`? (see https://github.com/calcinai/xero-php/issues/5)


FrameId again?


> So if you go this route

No, this will be cut & paste by the user. We do not need anything more than this.


Ubistvu sem hotel vprašat, če si pull-al...


@gorhill :+1:


I could name it `username`, but to me it doesn't make as much sense if it's most likely going to be overridden on a per-command basis.  Although ansible calls it `ansible_ssh_user`... *shrug*


It looks as if you can just remove registerCommand since you aren't using it?  Only need to implement it if its a command.


The issue was that things like ```&oslash``` shouldn't have been encoded in the first place.  As I mentioned in the comments - [http://www.w3.org/TR/REC-xml/#sec-predefined-ent] are the only in-element entities that need encoding.


I think:

> `<Visibility>` MUST be `*` for keys visible to non-operators or an implementation-defined description of the key's visibility status; for instance, it may be a permission level or flag.

would be better.


Why should we suggest doing a synchronous JavaScript operation is wise? i.e. Shouldn't we rather only expose an async API at the browser process level? I might be misunderstanding the context of this code snippet.


wait what?  this is just a string I'm passing to test that `run-commands-on-servers` works on multiple machines.  I won't reuse this string/command.


Hmm maybe we can add a customization for that, for instance `helm-github-stars-c-number-limit`?

I think it's not a good idea to hardcode this value, I don't know why.. :), and this customization will surely be useful for others ;)


Running tests overnight to see if this change triggers errors.


Thanks.
BTW in Zikula 1.3 with activated short URL's, on the address bar of the browser is something like:
```http://xyz.com/SeaBreeze ```
and works.


For the values in the DB I though we would have something name, highscore and so on - if we only ever record player 1 and 2 then new players won't have their scores recorded right? Also I think we should be tracking wins, and not scores.


@jzaefferer: https://github.com/jzaefferer/grunt-html/compare/master...XhmikosR:master

Note that upstream tests fail on Windows; still couldn't figure this out yet.


Isn't this going to treat the ItemSpec as if it's relative to the working directory, i.e. the main project, not the declaring project?


I was thinking that each player should handle their own scores but the system handle the logic when it comes to calculating who has won - if you pull the latest master branch you can see how i have reworked the player class so this might make more sense


Do we need two arrays?


So FYI the Helper.FirstCopy tag does not exist on feats. It's only for Class Abilities. So while it does not hurt anything by it being here it can cause confusion for the next person.


Thanks +1


Is this uncaught or will this crash the JS app if hit?


I think I missed the point - each array meant to check for a different type of disk?


I don't like it


I put it back as-is (except updated to new core) in af54b4c. Let's just hope BF_* does not change much now.


Good catch :+1:


you must serialize vector operations with kernel_fpu, see https://github.com/natsys/tempesta/issues/69


Yes, I like this better, because HTML isn't mixed with JS, although they do the same thing. ;)
About CLA, can you update this bit yourself and I'll fill the form after I'll find another issue and decide to help. ;)


@omaromar94
-we don't need 'welcome_controller'
-in 'tasks_controller':
in 'index' action: we have no categories, just list the tasks normally
in 'create' action: you're assigning a variable @user. We have no users right now.
- you made a tasks_controller, but you didn't create 'task' model so there's no 'task' entity!
-you need to add the routes. you can add to the file in config/routes.rb
-make sure  that your code at least works before pushing


It's uncaught & should log to the console. Nothing happens after this in the same event, so I don't believe it will affect the execution other execution.


remember, moving .coveragerc out needs additional care to the omit pattern


ResultsSchema (sanyalj) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Cool, this commit LGTM!


The function is used in one module only, so why to export the symbol?

Please note, that exported symbols in Linux kernel aren't just exporting functions in common user-space C program. In kernel we export a symbol from a module only if the module should provide the functionality to other modules. So kernel modules must be much more separated in comparison with traditional object files.


And revert also https://github.com/ChronoMonochrome/Chrono_Kernel/commit/1008a884190a43e35c3c66820dd48f6ded200ee6 ? Because you partially reverted changes in mcde_hw.c, it can work unexpectedly esp. because of that.


Thanks! I will of course will double check with you about PoW stuff I don't want to list two contacts. I saw this on my real job that multiple contacts just confuses people. =(  But I expect to have questions or will double check stuff with you. And of course if you see a question on the boards you can answer please do. =)

Thanks!


Well done!


Require list of needed modules


You'll want to configure your editor to make trailing newlines, or git will be very angry with you. There's a plugin for Sublime that does this, right, @timkurvers?


![Alt text](https://monosnap.com/file/5u1GzNEN2rzxRRpgIf96Iq5lb5OPE6.png)
vs.
![Alt text](https://monosnap.com/file/yQZrbeqXAjNDsDTXbovkASfajd86iG.png)


Hi Ryan,

No problem!

Regarding the Par_files yes, let us update them (BuildBot will start
testing them in the near future); you can use my script

EXAMPLES/vi_all_2D_Par_files.csh

to make sure you edit all of them when you have a change to make.

(equivalently for the 3D code:

EXAMPLES/view_all_parameter_files_to_modify_them.csh

Thanks,
Dimitri.

On 03/10/2015 11:03 PM, rmodrak wrote:
> Hi Dimitri, Sorry about that. I just copied over the value Alexis put in
> DATA/Par_file. For the broken examples, could it be better not to update
> the Par_files, seeing that there is no way to test them?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geodynamics/specfem2d/commit/9a29a702f54bdbc70c3ac13f59cc7ec01daa771c#commitcomment-10130861>.
>

--
Dimitri Komatitsch
CNRS Research Director (DR CNRS), Laboratory of Mechanics and Acoustics,
UPR 7051, Marseille, France    http://komatitsch.free.fr


I am not sure if that could be made "faster" as I suppose it will allocate a string here. Which it will later anyway so that could also written by introducing a variable and an else block.


This breaks compilation. The type for url.Values is map[string][]string. It should look something like this:
```{go}
lang := url.Values{"language": []string{language}}
```


Awesome!
I'll sort Prototypes tomorrow :)


This script comes from pyinotify tool. it simply monitor metadata changes


Java Conventions,
It should be have a space before and after of the =


Please update module versions on commits.


Will be added after they are created.


Did you consider adding Long Description as well?


Java Conventions,
It should be have a space before and after of the =
the same for all locators, please review that


@jzaefferer: for the Windows test failures see  https://ci.appveyor.com/project/XhmikosR/grunt-html/build/2

Feel free to register an AppVeyor project yourself and ping me to update the patch in order to add proper Windows testing.


How it works: pet always has a passive aura 34457 with 0 (by default) chance to proc. When hunter learns  34455, 34459 or 34460 this spell applies a modifer to 34457 to increase chance to proc from 0 to 100%. When 34457 procs pet casts 34456 on self and all party members within 100 yards.

I just don't see what this code does, talent works perfect for me w/o this.


Why should the files for the main site be owned by root?

2015-03-10 13:55 GMT-07:00 Chris Kuehl <notifications@github.com>:

> No, I agree with your assessment. Like above, I don't think it's
> particularly important who owns them; I'd prefer root, but not at the
> expense of the webhook running as root. I don't think there's a good way we
> can have them owned by root (without the webhook also running as root), so
> www-data is fine by me as a compromise. Unless you have another idea?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ocf/puppet/commit/4f47c700dc78e51f7cd5371a6953815e476f89fc#commitcomment-10129615>
> .
>


Doesn't this break php 5.3 support?


Java Conventions,
It should be have a space before the brace {


So plugins don't always have to start with `com_osvr_` - if, e.g. Leap Motion makes a plugin, it would be `com_leapmotion_xyz`


Shouldn't need to use your device descriptor file at all, actually. All you need to do is add routes (which will soon be automatic and embedded in the descriptor)


Should we change this signature to WriteMethod(odcmClass, actionImport) and let the WriteMethod implementation pull the Action off of the actionImport? What is the benefit of separating the arguments?


Same here for functionImport


Very good. The code also needs to look further up the path for the source files on mono. My code works.

Bert

Sent from my iPad

> On Mar 10, 2015, at 4:17 PM, Brian Nichols <notifications@github.com> wrote:
>
> The next C# client release will use Path.DirectorySeparatorChar instead of hard-coding them.
>
> —
> Reply to this email directly or view it on GitHub.
>


renamed EXAMPLES/vi_all_2D_Par_files.csh below to
EXAMPLES/view_all_parameter_files_to_modify_them.csh (same name as in
the 3D package)

On 03/10/2015 11:40 PM, Dimitri Komatitsch wrote:
>
> Hi Ryan,
>
> No problem!
>
> Regarding the Par_files yes, let us update them (BuildBot will start
> testing them in the near future); you can use my script
>
> EXAMPLES/vi_all_2D_Par_files.csh
>
> to make sure you edit all of them when you have a change to make.
>
> (equivalently for the 3D code:
>
> EXAMPLES/view_all_parameter_files_to_modify_them.csh
>
> Thanks,
> Dimitri.
>
> On 03/10/2015 11:03 PM, rmodrak wrote:
>> Hi Dimitri, Sorry about that. I just copied over the value Alexis put in
>> DATA/Par_file. For the broken examples, could it be better not to update
>> the Par_files, seeing that there is no way to test them?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/geodynamics/specfem2d/commit/9a29a702f54bdbc70c3ac13f59cc7ec01daa771c#commitcomment-10130861>.
>>
>>
>

--
Dimitri Komatitsch
CNRS Research Director (DR CNRS), Laboratory of Mechanics and Acoustics,
UPR 7051, Marseille, France    http://komatitsch.free.fr


XOR is not the perfect hash function. hash(msg_hash, best->hash) should be calculated only when best is changed, not on each iteration.


when would this be false?


I think you can't use basicserver for this anymore, I think you need to use connectionhost, but the osvr_server method is probably the best anyway.


hmm i miss that point.. but it was really suspected for me all the time... xD Thanks for hint.. i'll check again the patch..


You can use three back-ticks as "code fences" - see https://guides.github.com/features/mastering-markdown


@erocs Please double check and make sure this logic makes sense.  This is pretty important and would rather not fuck up.  Testing supports this is fixed.


Why to call external generic function? This is just an address - 16 bytes at most or just 2 vectorized operations.


It's the status quo. I guess the one advantage is that it's harder to deface the website, but I have qualms about running stuff as root too, so...


Use here better ability.getAbilityType().equal(AbilityType.ACTIVATED).
I added this attribute some time ago. It's faster and fits exactly the check condition.
Maybe some older implementations use the instanceof checks, they should also be changed to check for AbilityType.


83782874_LBNL_ThermostatOffsetFault (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1649 of 1649 tests passed)

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


We are using this one WriteMethod for creating OdcmMethod for both Actions/Functions(used by EntityTypes) and ActionImport/FunctionImport (specifically used by EntityContainers).
Action/Function may or may not have a parent Import operation. So thats why we have separate args.


Java Conventions,
Should exist a space after the coma ", " into the parameters


There are (or should) be commands to unlock the drive at the beginning of teleop, so that shouldn't be a problem. The problem is not that the turn command never ends, it's that the drive stays locked after the command executes.


As noted above actions/functions bound to entity types do not have a parent import operation.


THIS IS TOTAL GARBAGE


weirdly by only implementing the fix above it resolved my issue on mobile. I'm using Ember Model https://github.com/ebryn/ember-model and it seems like "TypeError: Attempted to assign to readonly property" happens when doing Ember.RESTAdapter.create(). Even tho I have nested #each the error doesn't come up again.


r+


Sounds good


Please don't forget to update copyrights, now is 2015 ;)


Causes build fail (I forgot put a description )


@OndrejPetrzilka Hello again.
I decided to build your changes at last and now I have an issue - fps is dropping from 1500 to 300 while I am not rendering and in debug mode of DirectX I see the following messages:
```
Warning: Live ComObject [0x62A522D830], potential memory leak: Active COM Object: [0x62A522D830] Class: [SharpDX.Direct3D11.EffectVectorVariable] Time [03/11/2015 00:53:07] Stack:
	d:\Docs\SharpDX-master_integration\Source\SharpDX\CppObject.cs(46,13) : Void .ctor(IntPtr)
	d:\Docs\SharpDX-master_integration\Source\SharpDX\ComObject.cs(35,9) : Void .ctor(IntPtr)
	d:\Docs\SharpDX-master_integration\Source\SharpDX.Direct3D11.Effects\Generated\Interfaces.cs(3974,9) : Void .ctor(IntPtr)
	d:\Docs\SharpDX-master_integration\Source\SharpDX.Direct3D11.Effects\Generated\Interfaces.cs(4648,9) : Void .ctor(IntPtr)
	d:\Docs\SharpDX-master_integration\Source\SharpDX.Direct3D11.Effects\Generated\Interfaces.cs(4300,17) : SharpDX.Direct3D11.EffectVectorVariable AsVector_()
	d:\Docs\SharpDX-master_integration\Source\SharpDX.Direct3D11.Effects\EffectVariable.cs(86,13) : SharpDX.Direct3D11.EffectVectorVariable AsVector()
```

They appeared each time when I am accessing shader variable.
This warning slow down app a lot.
Did you faced with that issues?


This class should be have a description about what it do.


Why are so many things just named a letter?


A `path.join` would probably be nice here just to make this really-obviously correct. There's obviously no directory traversal issue, but it stuck out a bit. No big deal in any case.


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


Will have to futze with this a bit more, locators seem to work here https://gist.github.com/m8ttyB/fbeb3da51ba6aef05906


I did not notice the LongDescription annotation. But it is fairly easy to add support for it. I will create a new commit for that.


@ntdb this shouldn't work, as that's not course number 351. Have you deleted the relevant VCR cassettes?


How would that help including the same functions, definitions or pod types in different library files?


A description about the what the method do, should be here


saw approval from jdm
at https://github.com/glennw/servo/commit/33829c0254d0a239a8f353bafae55fe4ebf599c4


merging glennw/servo/reflow-events = 33829c02 into auto


glennw/servo/reflow-events = 33829c02 merged ok, testing candidate = 19827658


I assume #finalize is a destructor invoked from GC?


ICSSolarCollector_#4723 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1653 of 1653 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)


![screen shot 2015-03-10 at 5 59 14 pm](https://cloud.githubusercontent.com/assets/7408017/6587185/2d599d52-c74f-11e4-8d18-e353b7849bf7.png)

It just displays an empty column,


a enlever


Question on downgrading the log level of this: how should we handle this case? AIUI, it's an error condition, and thus should be flagged up. Or can this occur in a benign execution of the solver?


@ntdb for me, everything has been passing, including (as recently as yesterday) when regenerating all fixtures. (It can take a little while, though, to pull in all that data.)


1 The controls on the home page build a "search query"
2 the app how to find and display only the vendors and merchandise the
3 user is interested in.

Bad Grammar, maybe add "that tells" to the end of line 1 or beginning of line 2.


print incorrect if valid login == true?


testing


There is typo here. It should be `kpsewhich` instead of `kpesewhichc`.


please remove "this."


@udalov
Sorry for delay. It seems "compile" phase is correct and was omitted in these pom files by mistake.


not really necessary to check in both places, eh?


ICSSolarCollector_#4723 (jasondegraw) - 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)


pelase remove "final", it is useless there.


develop (macumber) - Win64-Windows-7-VisualStudio-12: OK (2166 of 2184 tests passed)

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


Thanks again! This is great.


je viens de voir que tu as as marqué tes tâches comme "fini" sur asana. Il me semble qu'il manque des fichiers non ?


> - if (ignoreMethodNames != null) {

as method is public, it might be be possible to set NULL, please explain that udpate.


1 Once Oregon's Catch has finished searching for vendors or merchandise,
2 the results are displayed the list page.
                                         ^
Add "on" between "displayed" and "the" on line 2


Moved it in b4ed1f2


A blank line is missing here


Typically, I include the assembly version in the assembly element. This allows me to provide different annotations for different versions of a given assembly.  Unless there's a compelling reason, I'd prefer that these files also include the assembly version in the following way:

    <assembly name="Microsoft.Practices.Prism.Composition" version="5.0.1">

...assuming, of course, that the version is 5.0.1.

If there's a compelling reason to omit the Assembly Version, please let me know.


the only header needed for cors is 'Access-Control-Allow-Origin'


Do you know what am I doing??


alright, I'm fine with it


Please separate that change in a different commit...


LGTM, pull away!


@ragesoss No, I haven't deleted any cassettes. RSPEC gave me this:

    expected: "HSCI 3013: History of Science to the Age of Newton"
    got: "NUSC 1P10 Professional and Therapeutic Communications"

...so I updated the spec. I'll roll it back and try to figure out where that data is coming from. Any ideas?


1 Users may also
2 select a map node to see more information about that specific vendor.

Maybe say something like "to see more information about a vendor that carries that specific product.".


8 directory levels?   Yikes.  I will take your word for it.


These should all be `topcoat`, not `topcat`


Would you be opposed to me changing your folder structure?


auf knackt=to crack


@ntdb What I mean is, if you haven't deleted the cassette, you should. It may be providing you with data that doesn't actually correspond to course 351, if it's from an old version of this spec.


> jumbo.[ch] is the warehouse for jumbo stuff, I thought

I think our current use is slightly different: misc.c is for JtR-specific functions (core or not) like `strnzcpy()` while jumbo.c is for things like `memmem()` which is available on most systems' libc but implemented here if not.


The AssignNode is rather inefficient and we'll have to update it. I had to downgrade it because the statement is reached whenever we try to place a task that is already assigned. With cs2 this happens in every iteration because the solver returns the entire flow graph.


Why do we need different scripts for watch vs. watchify?  This is a little confusing.


What's with all the commented code?


Unfinished comment. I'd say that RCU is too strong synchronization primitive for the tasks and simple rw spin-lock would do it's job perfectly. But there is no reason to rework it to spin-lock, RCUs are good.


Test Repository.


@ragesoss roger that. I'll regenerate all of my cassettes just in case and roll this back. Thanks!


Sounds good. Maybe, another path - auto-check url protocol prefix (http vs https)?


please just
if browser_has_cache:
    responde_code = 304
etc..


PLus


Oh, I see. The error message does not make it very clear that this case will be encountered when a task is already assigned.

Presumably we'll want to replace this code with the more efficient delta extraction algorithm from Flowlessly?


Looks like you misspelled Canola.


It does remember the selected system in the dropdown now, but... when reopening and XBL was the last used it never loads. If I switch to PSN and back to XBL it fixes it.


That is correct. Will remove.


Test dieser Kommentarfunktion von github.com. Entweder zerschiess ich damit meinen Code, oder es ist tatsächlich nur ein Kommentar


I was thinking about adding it.

> another possible improvement ; if "settings.onAjax(action, serialize)" returns false, don't send the data to the server.

__delete/restore__
If the restore button is activated, the idea is to use a field in the database to "remove" a row, such as 'deleted_at' field of the Laravel framework (http://laravel.com/docs/4.2/eloquent#soft-deleting). But the restore button is only shown while the page is not refreshed, because then will no longer be shown. It's like a button to undo.


I believe the assembly name should be "Microsoft.Practices.Prism.UnityExtensions"

    <assembly name="Microsoft.Practices.Prism.UnityExtensions" Version="5.0.0.0">


Yeah, certainly couldn't hurt.  I just added it in there because I was sick of it erroring out - wanted it to die a bit nicer.  Add a simple if is also about the limit of my bash skills :)


Image is not being loaded correctly, we need to reference the image as a .PNG, because that is how the image file is terminated when it is added to the built docs.


sure! makes sense.


ok, es war tatsächlich nur ein Kommentar


When adding the test cases

For custom values, can you ensure that the `onTokenAdd` gets called for the custom value on the tokenizer?

And that on the Typeahead, `onOptionSelected` gets called?


I believe the assembly name should be "Microsoft.Practices.Prism.Mvvm"

    <assembly name="Microsoft.Practices.Prism.Mvvm" Version="1.0.0.0">

Per: https://msdn.microsoft.com/en-us/library/microsoft.practices.prism.mvvm.bindablebase%28v=pandp.50%29.aspx


You shouldn't need to set `Footman.new` to a variable here (`footman`), since you're not using it inside the function but returning it. You can return it without setting a reference to it by omitting the `footman =`


You might want to check the last improvements I made to the build script last night and rebase your branch :)


Oops, these were left over from some code I tested. Will remove.


What if we tried #F5F5F5 here? I'm thinking the box needs to be a bit less transparent, and I'm afraid the current shade of gray will make the black text tough to read.


Maybe 0.75 or 0.8? I like it being transparent, but I think it needs to be a little less transparent so all the cracks in the wall of the photo don't come through as well and obscure the text. Thanks!


(yo aun pariendo compilando el de linux, y haciendo un fix que no quiero decir hasta que este listo, pero que va a ser importante)


Nice to see you using a ternary to shorten the code! But you can shorten it even further. Since your expression can be either true or false, you don't have to check that value but simply return it. This is equivalent:

`gold >= 135 && food >= 2`

If both conditions are true, then the whole expression evaluates to true and returns that value. If any or both of the conditions are false, the whole expression evaluates to false and returns that value.


Hey, thanks for your comment. This is actually an out-of-date commit. I think your concerns have been resolved in [this commit](https://github.com/lukasolson/kibana/commit/ed43f05cd68a8858c06e2175de6f3e029be06e00). If not, could you comment on [this pull request](https://github.com/elastic/kibana/pull/3280)? Thanks!


Did this version get the rollout on google play halted?  Been anxiously awaiting 2.6.x


I developed the fork originally on an Azteeg X5 Mini. It blew the bed heater FET after a couple months, which is not surprising since it's a tiny SMT component. I got another X5 Mini, which lasted for a couple months, the heater FET didn't blow but the processor started acting weird, stuttering and freezing at random times, so I got a Smoothieboard 5X. It has nice big beefy through-hole FETs.

Long story short, if your bed heat stops working again, that might be why. :(


This broke the Travis build..


> As adding a job forces any current job to complete.
That was the bug with the old code. With the new code, we only wait until there is a free slot in the queue, once a free slot is available, we just add the job (we dont call dojob())


Thank you @michelgomessilva. This helps since I wasn't aware of such attribute. But I'm still struggling to map the JSON string read it with ORMLite to the Dictionary (for example).


What is it for? Compiles on my machine without it.


+ go back to results view and then back to search view to make sure parameters are restored correctly


Travis build is failing because of this line


r+


FYI @xtordoir → now master points to GA4GH current master status. Which mean v0.6.0-SNAPHOST.
We'll have to keep this up-to-date and release when 0.6.0 will be finalized


Cara fiz um lote de testes aqui usando o clitest e realmente quando é redirecionado para /dev/stderr dá erro sempre.
Seja redirecionando para /dev/stderr no teste ou usando "zztool erro".
E sempre ocorre o corte dos dois primeiros caracteres.
Os mesmos testes usando "printf" ou "echo", mas indo para a saída padrão não tem problema.
Creio que neste caso o problema esteja no clitest e na saída direcionada para /dev/stderr.


Change all these assertions `.toBeFalsy()` to `.toBe(false)` (or true) so that the return value is strictly enforced.


Hi @laurencer, I just looked into this and I don't think it is caused by the bug you suggest. If I compare testStruct.binaryField, a `ByteBuffer`, with a new one, `ByteBuffer.wrap(new byte[] {-123, 20, 33})`, the `equals` method says they are identical, although I do see the extra bytes that you do in the debugger.

I think the problem is caused by using `ByteBuffer#array()` without restricting to the buffer's position and limit. When you take those into account, the data matches exactly. When you don't, you end up getting the entire backing buffer, which in this case is identical because both fields are `binary` with the same bytes.


Läuft alles bei mir!
:smile:
Ist allerdings im "falschen" Repo :unamused: , bitte weitermachen bei https://github.com/ohli/videosaeule
Dort stehen auch einige Neuerungen in der Readme, biite beachten!
Ich versuche deine Änderungen mit rüber zu nehmen...Oder wir machen's doch hier weiter, ist momenta noch egal.


that's because of the old shorturl scheme which is removed in 1.4 (for Themes)


Great tool! Works well on most services however I come  across an issue when attempting to download polygons to a featureclass. This error happens if the service publishes the  "Shape.STlength()" and Shape.STArea() fields from the SDE, the Addfield process stops.

![image](https://cloud.githubusercontent.com/assets/8563526/6587476/92f1f836-c7e9-11e4-8af2-cddad4a319cf.png)

If the output FC is in a file GDB, it breaks because it attempts to look for the field name "Shape.STArea__"
If the output FC is in a SDE, it breaks because the field name already exist when the FC is created.

Any suggested workarounds to this? Thanks.


sigh. such a hack. =(


I see.


woops! didn't think the md file would be generated.


If we canceled any, I think we should sleep a few seconds to messages can propagate.


This conflates numbers and strings and others:
```js
_.uniq([1, '1', true, 'true']);
```


Looks good.  But we also need to add Getter and setter for Hit and Miss.


this is merged. Thanks!


Grazie mille! :D


I edited the .gitignore to ignore your userprefs files.

Shouldn't be a problem in the future.  For your new repos'/forks, I reccomend using
https://www.gitignore.io/

to generate gitignore files
with the parameters "Xamarin Studio", "Visual Studio', "Windows", "MonoDevelop", "OSX"


I think that is two sentences.


I see no issue, merging


Should these functions actually be static?


@nicohsieh which version of ember are you using?


Ooookay, es ist möglich, direkt über die Website am Code zu pfuschen. Ich schlage vor, diese Funktion erstmal NICHT zu verwenden :D


yeah that variable is supposed to be invalid_login


fairly certain that `IOLoop.instance()` returns either the active loop (if one exists) or a new loop (if one does not exist). is there a reason for this method?


Juan José Rabadán Barastegui


I don't know, but even if it were optional this parameter could never be used *as* an optional parameter, as the default constructor is more specific (uses no optional parameters).


Do we still want failures in 0.10?


Not only Git, mind you. :wink:


88734010-AFN-horizontal-openings (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1647 of 1648 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)


Ikr
On Mar 10, 2015 5:50 PM, "MonsieurApple" <notifications@github.com> wrote:

> Them line endings ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Plastix/Plast-Pack/commit/c9cb8afabb595aa4f2e5d0fd42937e747554d889#commitcomment-10130651>
> .
>


Looks good to me, I'll merge after a couple of changes:
- Put the new tests into the (not yet existing) identifiable_spec.js, which should go in https://github.com/rackerlabs/barricade/tree/master/test/mixin
- Add a test that asserts the objects are instances of Identifiable (see https://github.com/rackerlabs/barricade/blob/88412af6c12dfc8deedcf575a2e097cf2b0c7231/test/immutable_object_spec.js#L55 for an example)
- See line comment.


I will give it a better name.
Maybe __cbuf_alloc_collect()?


line 1524 and few other still references SOCK_INJECT_OK macro which is removed.


c'est moi, ou il manque un `main()` et une `}` ?


@stefanpenner it's #1.7.1


This whole block is unneeded as `_submit` is never set and doesn't exist


Hello- here we go!


Yes. Also @sindrenm.


This solves draging from elsewhere problem. Good Job!


add License file


subcommunities? or sub-communities? Both seem right.
Also s/ROpenSci/rOpenSci


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: js/tbl_change.js, libraries/insert_edit.lib.php


83782874_LBNL_ThermostatOffsetFault (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1655 of 1655 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)


all tests pass:
success: http://build.servo.org/builders/linux1/builds/985
success: http://build.servo.org/builders/linux2/builds/984
success: http://build.servo.org/builders/android/builds/1019
success: http://build.servo.org/builders/mac1/builds/1200
success: http://build.servo.org/builders/mac2/builds/1201
success: http://build.servo.org/builders/gonk/builds/171


My brain is totally derping on where _dperroom.js_ is even having a 'syntax error' at the very last line of the file right now...  All I could think of is to match up the braces, which seemed fine.  I do believe I'm going to take a break from this for a bit.  I may be too happy 8-) for this right now.


Potentially they could be, but then you'd have to pass both the source and the target into the function. Plus, where would this function live? `:xhp`? I guess I just like the signature of it non-static better. Compare:

```
:xhp:copyAllAttributes($fromXHP, $toXHP);
$fromXHP->copyAllAttributes($toXHP);
```


ICSSolarCollector_#4723 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (637 of 637 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-4.19%25-green.svg)


Чтобы зря не жрать процессорные ресурсы счётчик будет обновляться только в тредах, которые смотрит юзер.


yea this is better since I cant use the sha1 key to open dev...


Perhaps we check the "Clean checkout" flag?


fast-forwarding master to auto = 19827658


This is helping patch for debugging, ok. Neither of the changes are suitable for merge w/ master.


This is based on the [new-shell-banner](https://github.com/rvirding/lfe/pull/102) branch.


83782874_LBNL_ThermostatOffsetFault (rongpengzhang) - 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)