yeah, but I think there is a difference from the module author's perspective. Someone can aggregate the functionalities of multiple modules in a single s-c-s-module which is powerful on the standalone context.


It is fine if ai contains csvs instead of .zips (and ebs and goa). Please
make the corresponding note in the update instructions, though, since the
website provides .zips.

On Thu, Aug 27, 2015 at 11:28 AM, Ryan Batt <notifications@github.com>
wrote:

> @mpinsky <https://github.com/mpinsky> the folder ai is unzipped, but it
> only contains zip files. Are you suggestion this for space-saving reasons?
> There isn't a significant (if any) space-savings from zipping up zip files
> (I guess you're compressing the small metadata files, but those are so
> tiny, and all the other files are .zip, so they're already compressed).
>
> I'm pushing back on this b/c I don't know if my read.zip.csv function
> will work well on nested zip files, and I don't want to futz with it if I
> don't need to. And, to be clear, my new approach is relying very heavily on
> these folders existing ... i.e., I need the contents of data_raw to stay
> the same (a folder for each region, and inside the regional folder there
> are .zip files named only by date, and that name has to be in the current
> format (but the dates can change)).
>
> Repeat, I am relying heavily on this current setup being a stable
> organization. If anything changes, everything will likely break. It does,
> however, expect new files to be added etc. Just naming patterns and
> directory organization need to be the same.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mpinsky/OceanAdapt/commit/d6ae079c3c23f219c5b15b360892bf5134c5c994#commitcomment-12929191>
> .
>


It would also be good to document whatever files your script expects to
exist there as well.

On Thu, Aug 27, 2015 at 1:59 PM, Malin Pinsky <malin.pinsky@gmail.com>
wrote:

> It is fine if ai contains csvs instead of .zips (and ebs and goa). Please
> make the corresponding note in the update instructions, though, since the
> website provides .zips.
>
> On Thu, Aug 27, 2015 at 11:28 AM, Ryan Batt <notifications@github.com>
> wrote:
>
>> @mpinsky <https://github.com/mpinsky> the folder ai is unzipped, but it
>> only contains zip files. Are you suggestion this for space-saving reasons?
>> There isn't a significant (if any) space-savings from zipping up zip files
>> (I guess you're compressing the small metadata files, but those are so
>> tiny, and all the other files are .zip, so they're already compressed).
>>
>> I'm pushing back on this b/c I don't know if my read.zip.csv function
>> will work well on nested zip files, and I don't want to futz with it if I
>> don't need to. And, to be clear, my new approach is relying very heavily on
>> these folders existing ... i.e., I need the contents of data_raw to stay
>> the same (a folder for each region, and inside the regional folder there
>> are .zip files named only by date, and that name has to be in the current
>> format (but the dates can change)).
>>
>> Repeat, I am relying heavily on this current setup being a stable
>> organization. If anything changes, everything will likely break. It does,
>> however, expect new files to be added etc. Just naming patterns and
>> directory organization need to be the same.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/mpinsky/OceanAdapt/commit/d6ae079c3c23f219c5b15b360892bf5134c5c994#commitcomment-12929191>
>> .
>>
>
>


:+1: since this is so gently maintained as it is


```
{phrase: response for phrase, response in csv_reader}
```


I think it won't work in an ordinary web view, only in Safari, SFSafariViewController or WKWebView (not sure…).


pathVsLoaders is private, so you could not correctly override `createDefaultLoaders` now
IMHO pathVsLoaders should be init outside `createDefaultLoaders` and map path to mime, not to loader


Обычно events пишут на первом месте


I see your point. That's more or less what happens here: https://github.com/mbogoevici/spring-cloud-stream/commit/f6bc3f1ae156f60f08cfb4cc80e76e8dac76d925#diff-754bab36aba16bb528d216952fe84d5bR29

(The reason why we try to aggregate entire applications in that example is that, at least for a lot of our modules, they need other beans to operate - which are typically added in the bootstrapped context - e.g. properties and so on).


@ovr нихуй, все ок


Evidence that this refactor is complete:

```
➜  sandstorm git:(non-logged-in-fixups) ✗ git grep  currentRoute | wc -l
0
➜  sandstorm git:(non-logged-in-fixups) ✗ git grep  isCurrentRoute | wc -l
5
```


Меня беспокоит /system/vendor/lib/hw/gralloc.sun9i.so
Указан TARGET_BOOTLOADER_BOARD_NAME := exdroid
Хотя похоже это sun9i.


PROGMEM caused problems in the serial output.


https://github.com/SthephanShinkufag/Dollchan-Extension-Tools/commits/master
Тут хеши обрезаются до семи символов. Можеть тоже сделать семь, как и в списке коммитов.


Ok, then fix the pb in this file...
At present, I can not compile the doc with it on OS X, or Linux
due to an obscure ASCII code which I can not see, even in vim...
Until it is fixed, the doc will not be updated automatically on the web either.


хотя @ovr прав, http://backbonejs.org/#View


#4952-VRF-hard-crash-if-terminal-unit-list-error (rraustad) - x86_64-MacOS-10.9-clang: OK (1799 of 1799 tests passed)

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


Note this won't catch `String('foo')` (which nobody uses). So you could change to:

```javascript
if ( {}.toString.call(data) !== '[object String]' ) data = JSON.stringify(data);
```


what


Besides, why should we include the doc of the code we reuse ??
We do not include the doc of google tests, e.g.


@mferrero : opinion ?


Yesterday I was wondering about this too.


Do we need this and `commandDictList`? They don't appear to be in use.


so then @hypery2k  u removed the method, so .. i dont see what happened in older versiones of oc, i suppost to does not hapend nothing due this its only for inmediate see (after the install) the app in action.. so i think removal must be good.


We should have a better way to express module embedding though, it's just that I'd like to avoid scope creep.


:clap:


@nmittler  LGTM


Hm, I made Sphinx ignore `doc/sphinxext` and everything inside this dir (see change to `doc/conf.py.in`).
Are you still having problems?


On 27/08/2015 19:50, chengg11 wrote:
> hi, this commit seems to create problem for 15.05 build. failing saying
> no rule to make libbz2/host. (tp-link archer c7 v2).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openwrt/packages/commit/da6d7b57f501eaece621f8f834646aa22b618fa5#commitcomment-12932128>.
>

ok, will fix it in a sec


Yup. Derp. Looking at the vanilla furnace, it turns out that I actually REMOVED proper cabinet item inventory rotation. That's why I reverted it


I had some trouble starting m4.xlarge spot instances, but that may have been a temporary problem. I'll try again later.


watchout, this is generated code, and only commited for the convenience with the go tool


@ovr Не хуй, а не нихуй


Ah I see, I am retrying ...


`if @link.valid? && @link.save` is equivalent to simply `if @link.save`.


You used `home_and_end_to_start_end_of_doc` in the code, but `home_and_end_beginning_end_of_doc` in the preferences.

Also in setting `home_and_end_to_start_end_of_doc` to true you would think **home and end** would make you go **to** the **start** and **end of** the **doc**ument, but it's actually backwards.


I'm quite sure `@notice` will not be available after you do a redirect


Undo


Probably should have been two commits: Separate User class tests and user helper functions tests. Easy to untangle, but I'm not gonna do it.


I think the best way to do this is to have variables in calendar for `start_date` and `end date`, and each implementation of the calendar can set those variables and `date_range` would no longer need to be overridden. Thoughts?


No, we haven't resolve the current >1h problem. Misko explained that his
change which appeared to break it was really just that one job started
waiting for other matrix job to finish.
We just had the "dev productivity" meeting. Here is my shortlist for the
dashboard:
- ability to scroll left in time. if I switch to year, the most recent
builds disappear, and in month, the interesting event has scrolled off the
left. Ideally I'd like to be able to set the start and end time of the
graphs, with the current day/month/year buttons just a shortcut for setting
those.
- Currently shows 18m at the top which is wrong, our build is a little over
an hour right now. I think it's averaging in the quick matrix builds. It
needs to be the "long pole" time - wall time between build starting and
getting a result
- Finer breakdown of time. Everything in our build is in "script.1". You
had earlier proposed a message we could print into the log that you would
parse. Can we do that?

This doesn't appear to overlap neatly with the list of things you planned
to do. How do you feel about doing these sooner?

Thanks again!
Oh, and I'm doing a talk at AngularConnect about tooling, and I plan to
plug the project during the "how to keep your tooling effective" part. :)
-Alex

On Tue, Aug 25, 2015 at 12:36 PM Dieter Adriaenssens <
notifications@github.com> wrote:

> Hi Alex,
>
> I'm glad you were able to find a build/commit that caused a jump in the
> buildtime of your project. :)
> Were you able to reduce the buildtime again after you found it?
>
> I agree with you on the notification when a buildtime jumps. It's a feature
> I'd definitely would like to implement some day.
>
> About the issue/feature list for buildtimetrend. The full list of issue is
> on github : https://github.com/buildtimetrend/python-lib/issues
> Basically it boils down to :
> - improvements to the dashboard :
> - more and more comprehensive charts
> - better filtering (filter on build properties, change time interval)
> - make it easier to see properties (build id, branch, build matrix, ...)
> of a build job in a chart;
> - drilling down to the details of a build by automatically setting a
> filter on another chart
> - jump to the culprit build on Travis or the culprit commit on Github
> - get more detailed timing data by supporting custom timing tags.
> - analyse data to find trends and issue notifications when a build doesn't
> fit with a trend, or if a trend is changed, like a jump in buildtime.
> Preferably looking for a culprit build stage that took longer than before.
> - support private repos/builds
> - support other build environments like Jenkins, Circle CI, ...
>
> Suggestions for improvements for Buildtime Trend are always welcome. It's
> always interesting to see what the needs of different projects are, and
> how Buildtime Trend can help to gain some insights.
>
> What are your plans to continue using Buildtime Trend to monitor your build
> times in the future? What are your expectations regarding desired features
> and timeline of having those implemented?
>
> Kind regards,
>
> Dieter
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/angular/commit/30bd2a3fc72f8f1cdb0ffc4b9d11b4ca42e9ece7#commitcomment-12888089>
> .
>


On 27/08/2015 19:50, chengg11 wrote:
> hi, this commit seems to create problem for 15.05 build. failing saying
> no rule to make libbz2/host. (tp-link archer c7 v2).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openwrt/packages/commit/da6d7b57f501eaece621f8f834646aa22b618fa5#commitcomment-12932128>.
>

fixed, please retest


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (758 of 758 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-9.23%25-green.svg)


OK, that makes sense.  But the smell is the `if...else` cascade.  Maybe something that generically maps the `pivotal.*` config to a JS object?


correct local should be Serbian, Macedonian and Bosnian


[100%] Generating html/contents.html
Running Sphinx v1.3.1
loading pickled environment... not yet created
Checking tag file cache for doxy.tag: No cache at all, rebuilding...
Skipping function triqs::gfs::gf_mesh< cartesian_product< Ms...> >::is_within_boundary(Args const &...args) const . Error reported from parser was: Expected ")" (at char 13), (line:1, col:14)
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 324 source files that are out of date
updating environment: 324 added, 0 changed, 0 removed
Starting on 1 Nodes at : 2015-08-27 20:13:15.902521ravais
reading sources... [ 93%] sphinxext/autorun/README
Encoding error:
'ascii' codec can't decode byte 0xc3 in position 9: ordinal not in range(128)
The full traceback has been saved in /var/folders/wx/d74s2nbd1sg97zwsf51s44z00000gn/T/sphinx-err-1Gir2U.log, if you want to report the issue to the developers.
make[2]: *** [doc/html/contents.html] Error 1


@junkdog this branch still relevant? Cleaning.


can we get a version bump?


The actual line [UI_20150317_000755	{@st43}{b}             Select Language {/}{/}] has too many spaces:
![screenshot_20150827_00001](https://cloud.githubusercontent.com/assets/12136149/9529005/318eadfe-4cce-11e5-99ee-33a2ab6479dc.jpg)

This pull request is edited to be like this:
[UI_20150317_000755	{@st43}{b}Select Language{/}]

![screenshot_20150827_00003](https://cloud.githubusercontent.com/assets/12136149/9529022/5822bc8a-4cce-11e5-8486-fe3c65314ae5.jpg)


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)


Hi!
It's so great that you reply - and so fast!
First things first: I work with Illustrator CS6, Win 7, Office 2003
Whenever I try importing csvs i get an error - the names of the variables, the first row, is not found - he only finds one variable
so i use txt and it works for texts and image paths

but the %graph does not work unfortunately, no matter what kind of csv (there are three kinds: dos, mac and another I can't remember) or txt I link to
It isnt even the final dataset, it is just a otherwise empty file with only numbers in two colums and six rows

By saying "individual graphs as separate txt", do you mean "only ONE collumn of data"? If so: that would most probably be the solution ;)

As for what I am going to use it:
It is for a weekly weather-report with temperature and rain data, as well as a past analysis in graph-form

Unfortunately I have no idea where you attachment might be and where to find it, also I am currently at home and not at work so I can send you any files you want soonest tomorrow.

a hundred Thank-Yous anyway for your great script - it is sooo helpful even without the graph-functionality

cheers from Austria!

p.s.: please excuse any weirdnesses in my writing, as I am no native speaker. Also: I tried to contact you on linkedin too, about the same issue


Nice :)

I'm just not sure if cache_max_age as a hardlimit is really necessary. Even with md5 the cache file name includes the date of the last published issue. So I think it would got this way:
1. Check if a cache file for the current date (=md5) exists. If so, load it (as it is now)
2. If not, create a new and delete any file with glob('../cache/toc-$ISSN*.cache*');
> This would require to stick with the old naming way, at least partially, BUT you are very right, it should not just be an implode of $_GET. But it would make sense to explicitly take the issn and md5 everything that remains. If one takes the date explicitly too, the md5 is just a failsafe for future use. We always can only have one toc for a single issn, so keeping it in the name, would be enough of an identifier

The deletion could be done in getLatestJournals.php or "on-the-fly" in ajax_toc.php. I think at fetching the toc would be easier, since there is already a check for the specific toc. So there always can only be one old cache file. This way a toc really is only regenerated if a new issue is available and not after cache_max_age days. And some journals only appear annually... Ok, once in 31 days having to wait is not really a big deal, but if it is not necessary... :)

For the index file the md5-everything-way is fine :)

Hmm, maybe it would have been better to try it, instead of writing such a long text. But maybe I miss a point and a max age is necessary?


```
if __name__ == '__main__':
    do_yer_stuff()...
```


@briceburg to save you some possible grief down this road, I used to have most things relaxed, until I found out that some of them would stop working, either at all or with each other. https://github.com/BlueAcornInc/Node-Magento-DB-Tools/commit/4a7be26378b630c5bba88e2c7709c813def4542c

did anything change with the update to libxmljs, though?


// ConsoleApplication1.cpp : Defines the entry point for the console application.
//
#include "stdafx.h"
#include <iostream>
using std::cout;
using std::cin;
using std::endl;

#include <string>
using std::string;

int main()
{
	string output;
	int input;

	cout << "Please enter an integer: " << endl;
	cin >> input;

	while (input >= 1000)
	{
		input = input - 1000;
		cout << "M";
	}

	if (input > 499 && input < 1000)
	{
		input = input - 500;
		cout << "D";
	}

	while (input >= 100)
	{
		input = input - 100;
		cout << "C";
	}

	if (input > 50 && input < 100)
	{
		input = input - 50;
		cout << "L";
	}

	while (input > 9)
	{
		input = input - 10;
		cout << "X";
	}

	if (input == 9)
	{
		input = input - 9;
		cout << "IX";
	}

	if (input > 4 && input < 9);
	{
		input = input - 5;
		cout << "V";
	}

	if (int i = 0; i < input; i++)
	{
		input = input - 1;
		cout << "I";
	}

	return 0;
}


"know" :)


Can you email me at vasily.hall@gmail.com and I will send attachments!


Yes :+1:


`_phi_cached` to clarify the difference with `self.phi_`


:p


No, the type-level `index_analyzer` and `search_analyzer` settings only got set for the `default_index` and `default_search` analyzers respectively.


`self._phi_synchronization != self._synchronizations_processed`? is it not working this way?


This API change caused a
-2:15-:: ::[TypeError: Cannot read property 'token' of undefined]
in setVideoEncoderConfiguration  and I have to start over.... Orz


As I remember this might be, that in some case the database user has no permission to truncate tables, although I don't know, why this particular permission should not have been granted to that user…

Beginning with MySQL 5.1.16, the DROP privilege is required for TRUNCATE TABLE (before that, TRUNCATE TABLE requires the DELETE privilege).


I still didn't tested it but I hope that it fixed weird issue with weapons as well :O ?


Looks like the version 3 from kalibro_install uses a cache that is not compatible with Travis' legacy infrastructure.

We should or create a 2.7 version for kalibro_install or make kalibro_client dependent on MetricFu rather than Analizo so we can run our build on the new infrastructure.


Sorry. I did a mistake.


@yavor87 can we not remain consistent then throughout?  What I am struggling to understand is why you would use one pattern for `Time` and another for `Publish` when both are part of the same Realtime API?


Maybe that is why Saagar was reporting that it does not work. The tester may have showed that it was not working.


LGTM


Too many abbreviations.
I recommend to expand them all.
Abbreviations is good in short programs, not library code


The filtered arguments need to be grouped under the module name in the array so that each module would get their corresponding arguments in `ModuleAggregator`


ok, trying to fix this...


You could import objects from modules
```python
from .regularizers import Regularizers
```
if you want


We could possibly assert the number of `mainClassNames` is equal to the number of modules.


Haha, never realized I spelled this wrong.


We could possibly assert the number of mainClassNames is equal to the number of modules.


Pushed a fix for this one!


Could do:

```rb
[tasks, attachments, comments].flatten.each(&:update_pg_search_document)
```


The match is positional right now (in the `arguments` variable). This will change once https://github.com/spring-cloud/spring-cloud-stream/pull/110 is merged - since there is clearer correspondence between modules and their sets of arguments.


So if the object is of the wrong type, then the old data remains?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.930.460fbb15f07471388e46182682d328af1e7f8ab1](http://build.palaso.org/viewLog.html?buildId=62034&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 464, ignored: 6 Build time: 00:03:25


Would be great to display the avatar instead of showing the avatar_url value.


`current_user.project_ids` is more concise


 psiphon-tunnel-core/ 6bd3e26
Revert asynchronous embedded server list import
* From time to time, the Update() call in StoreServerEntry would hang.
* Wasn't properly asynchronous since the Wait() happened at the end
  of Start(). Would need to do something else, such as have the
  Controller manage asynchronous importing, allowing Start() to
  return before the import completes.
* May not even be a performance issue with boltdb.
@rod-hynesrod-hynes authored about 1 month ago
1 changed file
7 additions and 34 deletions
 AndroidLibrary/psi/psi.go
@@ -74,40 +74,13 @@ func Start(
 		return fmt.Errorf("error initializing datastore: %s", err)
 	}

-	// If specified, the embedded server list is loaded and stored. When there
-	// are no server candidates at all, we wait for this import to complete
-	// before starting the Psiphon controller. Otherwise, we import while
-	// concurrently starting the controller to minimize delay before attempting
-	// to connect to existing candidate servers.
-	// If the import fails, an error notice is emitted, but the controller is
-	// still started: either existing candidate servers may suffice, or the
-	// remote server list fetch may obtain candidate servers.
-	// TODO: duplicates logic in psiphonClient.go -- refactor?
-	if embeddedServerEntryList != "" {
-		embeddedServerListWaitGroup := new(sync.WaitGroup)
-		embeddedServerListWaitGroup.Add(1)
-		go func() {
-			defer embeddedServerListWaitGroup.Done()
-			// TODO: stream embedded server list data?
-			serverEntries, err := psiphon.DecodeAndValidateServerEntryList(embeddedServerEntryList)
-			if err != nil {

Cancel  Leave line note
-				psiphon.NoticeError("error decoding embedded server entry list file: %s", err)
-				return
-			}
-			// Since embedded server list entries may become stale, they will not
-			// overwrite existing stored entries for the same server.
-			err = psiphon.StoreServerEntries(serverEntries, false)
-			if err != nil {
-				psiphon.NoticeError("error storing embedded server entry list data: %s", err)
-				return
-			}
-		}()
-
-		if psiphon.CountServerEntries(config.EgressRegion, config.TunnelProtocol) == 0 {

Cancel  Leave line note
-			embeddedServerListWaitGroup.Wait()
-		} else {
-			defer embeddedServerListWaitGroup.Wait()
-		}
+	serverEntries, err := psiphon.DecodeAndValidateServerEntryList(embeddedServerEntryList)
+	if err != nil {
+		return fmt.Errorf("error decoding embedded server entry list: %s", err)
+	}
+	err = psiphon.StoreServerEntries(serverEntries, false)
+	if err != nil {
+		return fmt.Errorf("error storing embedded server entry list: %s", err)
 	}

 	controller, err = psiphon.NewController(config)
0 notes on commit 6bd3e26

Comment on commit_id

Comment
 Desktop version


I'll take a look at these since I'm an Arch (Parabola) user primarily.

I'm also seeing an occasional error with execvp and permissions when banning folders.  I'll open an issue on the project if I cannot resolve it soon.


Sounds like a ValueError?


committed separately because they wouldn't commit from within QTCreator


Why?


Doesn't make sense. I thought this was the angular controller. So, the order of objects is html => angular controller => jsController.


Not needed. Remove.


`item.project_id` will avoid a second query. Not sure how to avoid the queries on the others, but at least using the foreign key `project_id` column will keep it one level shorter each time


Nej. Det felet löste sig själv. Det var nån azure-uppdatering. Detta var en ändring för att trigga en deploy ;)

Sent from my iPhone

> On 27 aug 2015, at 17:43, John Korsnes <notifications@github.com> wrote:
>
> Var dette feilen som var snakket om før sommeren? :) GET på http redirecter jo til https på tv.nrk.no?
>
> —
> Reply to this email directly or view it on GitHub.
>


It doesn't build and I don't want to spend time on it now


If you want to keep the Attribute as a Rock Video or Audio Field type, you could using the handy 'RawValue' attribute filter.  In this case, you Lava would be

```
<audio src="{{ item | Attribute:'AudioURL','RawValue' }}" preload="auto" />
```


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2281 of 2343 tests passed)

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


Is there a specific reason for using a XIB for the tableview cell instead of defining a prototype cell in the main storyboard?


This commit is not related to issue #70, but resolves #81 instead.


No.  The one DLL will work for all platforms. 'dotnet' applies to everything that supports contracts (net45 and later)  The only reason we need the additional dependency groups is to suppress the transitive dependencies on platforms that have inbox support.


@siltastic LOL. Love it.


Good question. It totally makes sense. I will change it :+1:


@IgorAntun what does this mean? `data` variable is not used after the assignment;

should it be something like

```js
if (data.type === events.team_join) {
  self.slackData.users.push(data.user);
}
```
ps. taken from https://api.slack.com/events/team_join


`exclude_trees` is superseded by `exclude_patterns` in Sphinx 1.3 ...
Please try it now.


#1

The files were added intentionally.


We can denormalize comments so that they contain the story ID as well as the polymorphic association. I think it also might be wise for these things to have a project ID but we can talk about that. Then we can load up the current users project IDs and story IDs in two queries and memoize and then we don’t need to load anything else, stops N+1 queries. I think these questions will be asked several places throughout the system so something like this might work well:

```rb
class User < ActiveRecord::Base
  def can_access?(item)
    if item.respond_to?(:project_id)
      project_ids.include?(item.project_id)
    elsif item.respond_to?(:story_id)
      story_ids.include?(item.story_id)
    end
  end

  private

  def project_ids
    @project_ids ||= super
  end

  def story_ids
    @story_ids ||= Story.where("storys.project_id IN (?)", project_ids)
  end
end
```


This causes a regression. In Quake II ProcessDList is called multiple times per frame, gDPFullSync() is called once per frame. After this commit Quake II will be slow again


@SthephanShinkufag уменьшил до 7 символов.


```rb
# Simplifies the search code to
PgSearch.multisearch(params[:q]).select do |result|
  current_user.can_access?(result.searchable)
end
```


Restore


hu?


`result.class.name` is the string version of it


What do you think? :)


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Would be nice to name this local variable to something like `ModuleAggregator` :-)


Nice to update things!


OK, I think I have this all sorted out now and I am getting a positive (i.e., '*') response back from the camera. I did some debugging via directly telnet-ing to the camera's PCIC TCP port and I now understand what the camera expects and how to formulate the commands properly.

I'm going to get back to testing the frame grabbing now.


This seems very odd! Why do you need this? if you don't have the assert, it fails with:
TypeError: invalid file: None, which seems clear enough to me


Related to: https://github.com/ModelSEED/ModelSEEDDatabase/issues/9


Good idea to set a date if there is none. But wouldn't it be better to take a date like 1970-01-01. So - if ever needed - one could tell which issues had no date? Also if sorting issues by date, they would probably be listed at the top. Not sure...


why did you need to move it (in addition to rename it)?


@namratanehete Okay Thanks :) i will make sure these small issues are not present from next time


@jsudufour DWMR. :+1:


Can you please post a tutorial on how to load up B3D files? Cuz nobody knows it atm :(


Sorry for all the inconvenience. I'll send you some ruby code tomorrow. This should clarify some stuff I hope. I use this for my testing.


откуда взялась эта строчка?


можешь выложить отдельно boot.img?


I thought you'd like that better :)

Interesting you can have nested quotes like that, I wonder why that works?

Would this technique be worthwhile to use for banner and thumbnail images?


This is not an unittest. It is an integration test; you rely here on the fact that network_connect is ok, which you don't know.


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

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


No inconvenience Christian, as I mentioned above, I have this worked out now. I was just interpreting the documentation incorrectly. I hope to commit this fix today after updating the unit tests to ensure everything is working as expected.


Is there a particular reason why you initialize the buffer?


Every time so much code is written without a single line of test a programmer dies.


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2303 of 2343 tests passed)

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


Ahh yes, that code did have an aroma. See the commit I just pushed.


I've been trouble-shooting the biochemistry inconjunction with Mike's proposed move forward, to break free from downloading KBase biochemistry objects again, but there'll be further changes to the biochemistry itself, and its tough to "see" changes within a single line of a flat-file, even when using git diff.

What I'm doing here is using a pretty-printed JSON representation of the reactions.master.tsv file in order make it easier to see where the changes are within a reaction's attributes.


I know the extension is called 'TLS' but it's porrly named and is wrong. Can you mention STARTTLS in this sentence instead?


In [hook_drush_help_alter()](http://api.drush.org/api/drush/drush.api.php/function/hook_drush_help_alter/master).


@prudhomm La branche gh-pages est à jour. Et le [site](http://csmi.math.unistra.fr/feed.xml) est aussi à jour.


typo in `Contrzact`


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 430](https://teamcity.labkey.org/viewLog.html?buildId=268811&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:32
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


@terkill I'm curious: is there a benefit to using `path.relative` versus `path.basename`?


But of course, we'd want the new file to be normalized. So currently, the JSON file was generated from reactions.master.tsv version that came from this commit: 7aebad4dd32267dbf050a00b519fda1eb13b1372


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.931.afbceccf14fd2c3c2ae9fea2f41ad29e32285050](http://build.palaso.org/viewLog.html?buildId=62035&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 467, ignored: 6 Build time: 00:02:35


Is it a contract or a contract type ?


Yes, finally (suggested [here](https://github.com/wxWidgets/wxWidgets/pull/33))!


sdf


@munkie I have left @badoo some time ago but if you will send pull request I will ask someone to take a look on it. I think @uyga could take a look on it and merge.


I think the safest way would be to use `Object.prototype.toString`. This is how lodash does this:

```js
typeof value == 'string' || (!!value && typeof value === 'object' && Object.prototype.toString.call(value) === '[object String]')
```


Is this still needed? Each import adds to the page load time, so let's remove ones we don't need. (map-search, map-directions as well)


// ne muty figni


@jenningshanna @boonrs
updated light box text... simplified


fyi, in case you ever use this again, the property name really is "gameclient", not "Addon.ID". see https://github.com/garbear/xbmc/commit/70bf422. oops :)


looking forward to giving this a spin...


@clintonfernandes Which tool are we talking about?


Before we merge this, please squash this commit to the previous one, makes a bit more clean history. You don't have to do it now, but the sooner the better I guess.

When you forcepush to github it will update the PR automatically.


thanks!


This one lacks a description and thus fails to evaluate.


@clintonfernandes why so?


This appears to be failing currently, and not due to my code. This fixes it, but isn't very nice. I guess interpolate_funcs.go should be using NewStringList, rather than just generating a StringList directly?


much better, @jxandery !


the table code/structure would  make easier addition of columns


Looks great, @jxandery ! Much more clean :)


I don't see this is covered in the topic...


Can we include all the arguments into parentArguments that are not consumed as `fiteredArgs` (for module) from the original args?


When did this change? libgit2 bump?


Why is this static?


Note that `mkEnableOption` [already prepends](https://github.com/NixOS/nixpkgs/blob/150211741856f6292ebd9ea747f7fef725c84ebc/lib/options.nix#L30) a "Whether to enable " in front of the description


What does passing true do?


There is less need for history at high values of remaining depth because ttable hits and countermoves (which has a lot more entries than history) will provide a decent sorting. Near the leaves the history table as a less precise heuristic will be most effective.

I liked the original version the most because it clears the history when we move to new subtrees by visiting nodes with a moderate search depth.


@clintonfernandes Is it necessary to have a CSS file?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1286 of 1286 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-70.01%25-green.svg)


I think you don't actually need to consider dyncam sens in ui.
Ui usually has sens of 100 and it would be preferable to keep it independent from current game sens in my opinion.


@ntd Done :-)


That would be a good idea.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 431](https://teamcity.labkey.org/viewLog.html?buildId=268836&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:17
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


Yes, it changed due to the latest bump


Why not
https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/RegExp#Description

but i think, we can split this option into ``acceptTypesPattern`` and ``acceptTypesFlags``


@clintonfernandes what name?


The value isn't that important here I think, the logging is basically if the user modified settings XML with random values


#4952-VRF-hard-crash-if-terminal-unit-list-error (rraustad) - i386-Windows-7-VisualStudio-12: OK (1805 of 1805 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)


is software version number dynamic? when 3.4.9. comes out shall it have to be changed or not? if yes, can it be somehow coded so it changes automatically???


Cool, wasn't sure if there was a service method that would get what we need, but this seems to be it.


Can't all of these have an undefinedMetricValue of 0?


The English text of the following two message ids contains a HTML entity which is now incorrectly displayed.
* `user.confirm_resend.success`
* `user.new.terms declined`

(This has been obtained w/ the following script: https://gist.github.com/c5ef9316ba854b1ff571)


What happens if I try to write from a null pointer? or a size of 0 bytes? or a negative byte size?


1579840-GroundTemps (mmatts) - x86_64-MacOS-10.9-clang: Build Failed

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


The patch is doing two things here:
1.  Updating stats with a new formula.
2.  Resetting the stats at a certain depth/value.

The question is...which one (or both) is contributing to ELO gain?
I proposed to do this:

if(abs(int(v)))>512
    return;
table[pc][to] += int(v) * 64;
table[pc][to] -= table[pc][to] * abs(int(v)) / 512;

Doing this we can distinguished between points 1 and 2.

If this fails...then we have an idea what is going on...

And we can try an apply this reset method at our simple formula:
i.e.
if(abs(int(v)))>512
 table[pc][to]=VALUE_ZERO;
else
table[pc][to] += v;


as above, avoid "getAttribute", use `data.` instead


Can you share an apk with all your changes or explain what's needed to build this? I'm having a hard time trying to build.


this one too :)


I was going with the 'don't make up a BS value' rule, but since there is no service they should all indeed probably be 0.


and this one :dancer:


nice job sir! :hammer:


Out of curiosity, what is this file?


Absolutely, that will be my next step


When I try to create a database from scratch, I now get an error on CustomAuthorize.cs / AuthorizeCore() in trying to get the rolesProvider  "The Role Manager Feature has not been enabled".   System.Web.Providers.DefaultRoleProvider doesn't seem to be known, and I'm not sure what the right solution is for this version of Identity Management.... Is there a NuGet package to install?


if we can avoid deleting, it is better.


hm this should never be undefined. there'd be a deeper issue than this


@clintonfernandes Is it possible to have different shades of colour without six digit code?


also you can do ```listify(addons={})```


16-*byte* doubles. 16bit would be a different type of fun


The least of which isn't that I used a tab instead of 2 spaces.


я не могу понять, как оно раньше работало, если в writeRow() на строке 163 Сессия закрывается. И когда выполняется writeRow() второй раз сессия понятное дело уже закрыта.


Ah, now I see what the service method is missing--we don't have access to the reports on this study object, do we? So with the information we have at this point, we can't determine if a study has just been performed, or also reported.  We might want to start doing something a little more complex, we can discuss next week.


asdf asfd


you can do ```this.props.isLoggedIn && this.renderMenu()```


trailing semicolon


As above about `tabindex`. Now you see the joys of real and fake attributes :)


duplicate mockito?

we should probably just biff it up in the parent build script?


Looks good. Should we worried about the following race condition: A thread wants to access outboundnodes, so we start to make a view. The intarrayview gets copied into the stack for the procedure call before the size, and then before the size gets copied, an add-edge routine gets called, increasing the size. Is there some way to make sure the size gets copied into the procedure call stack before the intarrayview?

Other than this issue, ship it!


#4952-VRF-hard-crash-if-terminal-unit-list-error (rraustad) - 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)


We don't need this everywhere


Can you please refer to #15 and ask the author on this choice? I'm sure he has some reason behind it :)
Thanks!


don't use an empty game unless you want an empty deck. if you just want empty hands, discard em.


I've just copied the approach of MeetupCell. I think putting it in the main storyboard would be better.


Should the uplink logic here be integrated with the BindCarrier logic?


This is a bit cocky. "works on _all_ computers"


For the creation of child contexts, I wonder if we can try adding `.child(sources)` with sources being `module` and `BindableContextWrapper`. Did you try that before and saw any issues? I am trying it quickly if that is do-able in this context...


Might even make sense to make a "RadiologyOrderStatus" object that contains the information we want, like date performed, last reported date, etc, and then create a service method that looks through the orders, studies and reports for a patient to build up this object.


repo


Yeah, serialization is fine, but how would the deserialize? We don't currently expose that at all.


I think we should add a reference to the new Java SDK which now has ARM support (which is related to this particular case study, since we were attempting to get GrapheneDB into the early preview)


The change from `ByteArrayInputStream in = new ByteArrayInputStream(this.buf, 0, this.count);` to
`ByteArrayInputStream in = new ByteArrayInputStream(this.buf);` is not correct. The count in the new input stream includes unused bytes.


Well it seems to be close to impossible to trigger the second one... It's there in theory in the code but I always seem to wind up getting sent to the wiki.


Is this going to be like a plugin or something?


why


Is the idea behind enabling auto-configuration to trigger auto configuration at the parent context level and all the child contexts would subsequently use them?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 432](https://teamcity.labkey.org/viewLog.html?buildId=268842&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:33
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


Not sure what best practice is here. I was surprised the kafka etc modules
didn't automatically inherit it as the eventing module has it as a dep.


Все тайна века разгадана, раньше getSession() каждый раз открывал новую сессию, и поэтому все прокатывало, а сейчас он возвращает текущую сессию. В общем думаю все такие try-catch-finall повыпиливать, так как такими вещами должен заниматся transactionManager


Sure, I agree that it was wrong before and now it is fixed, just wondered if we should fix on a style for this. But "it does not matter" is fine by me.


May be related to the cocoapods version you're using right now... We had some bugs of project not being able to compile. We got error about `Hex` but it turned out that Cocoapods version was broken.


@kootenpv I installed yagmail today (love the concept, am enjoying playing with it so far) but in order to get it to pip install onto a machine running Windows Server 2008 w/ python 2.7 I had to undo the changes made in this commit. Specifically I needed to replace all the
`from yagmail.error import...`
with
`from .error import...`
The same for `yagmail.validate` and `yagmail.log`
This may prove to be a problem for other users as well.


fast-forwarding 2.0 to borshop-integration-541-testfix/wait-for-query-yz_pb-test-2.0 = db42531a


Forgot to comment that instead of seperate statements for HTTPS and HTTP, they are all in one, including regex matching for some URLs.


I think this, and the following should be `which pkg-static` in case people use a different PREFIX/LOCALBASE.


I think this should be /nonexistent, it's what we usually use :-)


What exactly do you stumble upon? Besides the stuff in the readme it's important to have the environment variable JAVA8_HOME point to a Java 8 JRE.


@Brit200313 I'm still not ready for a PR; models are at 96% but I haven't started controllers. Please let me know if my specs for Person look ok.

![](http://www.animatedgif.net/people/soldier_e0.gif)


这个class名重新起。。。


好聪明~~


@kushaldas @praveenkumar Can you review this commit?


Add your name to the authors.
Add no also in the example


Hey this turned out to be unrelated to the bug fixed by your PR, you can remove these tests


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.474.882a590c9020f9a5b71eda45743a8b5d3e2d6737](http://build.palaso.org/viewLog.html?buildId=62036&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 138 Build time: 00:05:47


also fixed the tabs :)


butonu kaldır


here comes the catch: you never know how fast is the machine that runs your code ;)


@scene.width && @scene.height will given you the max X and Y without hard coding


BindableContextWrapper cannot be a source. We could add it through a listener ... but not sure what would we gain.


no db calls in views.


Oh I see, ok


ok


map statuses from the set of all enrollees


Also, for this particular case, the registration of shared channels needs to happen before the child contexts get created.


Exactly. There's only one ChannelBinderAdapter created, too.


This is wrong and not where it belongs


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

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


OK this one deserved a lot of attention.  Mike wrote this script to move away from downloading KBase objects, and to create the master list of reactions directly from the files.  Part of the reason that my original file was convoluted to an extent is because I wanted to stay away from the addReactionFromHash() function, but that was unavoidable.

To summarize, the problems were two-fold:
1) Unintentional merging of reactions
2) Hierarchy of sources

1) Most of the initial problems come from how the reactions are merged unintentionally, both in using the same parent object for the two source biochemistries, and within addReactionFromHash() itself, so the first major step was to ensure that a new biochemistry object was generated for each source file, and that a new parameter was added to addReactionFromHash() (findmatch) that would side-step the issue of returning reaction objects prematurely:  https://github.com/ModelSEED/ProbModelSEED/commit/6487bcb3051a9a02ce6062ef472a0684156ff946

2) One of the other aspects of the approach I took in the older files, was to default to the data in the plantdefault biochemistry, being more new.  But what became apparent to me is that the default approach was split between certain fields, so for any field that contains equations, I defaulted to the default biochemistry, but for any other field, I defaulted to the plantdefault biochemistry. You can see this in line 223. The reason this approach was convoluted is because I wanted to use the equations as they should be used in the microbial biochemistry, for the initial stage of ProbModelSEED, we want to make sure that the microbial aspect works, but, because many of the fields had been updated from newer versions of the biochemistry, I then defaulted any other fields (with particular attention to the reversibility and deltaG, which came from a new set of molecular structures) to the plantdefault biochemistry.

You'd think that this would basically reproduce the reactions.master.tsv file as it were in the last commit, but the process, useful as it were, highlighted two problems, which will be apparent in the commit of the updated reactions: A) The first problem is that, somehow, defaulting to the thermodynamic reversibility as calculated in the plantdefault biochemistry didn't actually result in irreversible equations. So by modifying and running Mike's script, I was able to find that there are reactions which should be printed as irreversible. B) The second problem is that, for any transport reaction involving protons, my underlying code attempts a mini-heuristic on selecting the compartment within which the number of protons is modified (they cannot actually be deleted from transport reactions, that's the whole point of transporting protons), but this heuristic is dependent on how compartments were previously organized, and with the new generic numerals, a few reactions are now incorrectly balanced.

I'll commit the biochemistry files themselves, but before I go back to the proton balancing, I'll work on making sure Mike's script can work with KBaseFBAModeling too, so to normalize the ObjectAPI directories.


ok, I also see BindableContextWrapper isn't a configuration class. got it.


sil


Returning a String is not sufficient. There should be a clear API.
I would expect something like:

TicketsResponse checkTickets(TicketsRequest request) { ... }

where TicketsRequest contains some filters of interest (trains, destination, acceptable dates, acceptable places)

TicketRequest {
  String destination;
  String startingPoint;

  Date dateFrom;
  Date dateTo;

  // This is optional, usually the user is interested in a destination, not in a specific train
  List<String> trainCodes;
}
and TicketsResult contains structured results.

TicketResults can have toString method for your debugging, but in itself this class should be structured and documented


This seems a little overboard... I don't mind tooooooo much... but it doesn't seem like that big of a deal to return `this.failure` in two places (unless this is intended to be public?)


Can we prefix these with an underscore if they are internal only?


Your concept looks sound. When looking at http://www.freeasteroids.org/welcome/, the hyperjump stops any velocity. So setting `self.body.v = CP::Vec2::ZERO` might be useful.  If it's still acting buggy, might have to separate the position change into the update loop, like `reset` or `destroy`.


yeah, good point.


Makes the list view printable.


This is how I did on CM previously - I have copied original variables.less from Bootstrap and modified it for my needs. At the end, I was struggling with Bootstrap updates, when additional variables were added to it.

It's better to include original variables.less from Bower components, then after it include custom local variables.less, where modified variables would override the default ones. In this way it's easier to maintain custom variables.less (because you modify and add only what you need), and also when updating Bootstrap, it usually does not conflict (unless variable names are changed, but this does not happen often).


Good catch! I think we should copy the size to the IntArrayView before we
copy the array reference to it. I'll send a commit when my flight lands.
On Aug 27, 2015 2:31 PM, "ashishgpersonal" <notifications@github.com> wrote:

> Looks good. Should we worried about the following race condition: A thread
> wants to access outboundnodes, so we start to make a view. The intarrayview
> gets copied into the stack for the procedure call before the size, and then
> before the size gets copied, an add-edge routine gets called, increasing
> the size. Is there some way to make sure the size gets copied into the
> procedure call stack before the intarrayview?
>
> Other than this issue, ship it!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/plofgren/cassovary/commit/b78408e5af923959c4e8fa4106fbe1043a0a35f5#commitcomment-12934125>
> .
>


```$tmp_op_dir``` doesn't appear to be used in here


I felt like `getDataNotFound` was a bit more descriptive, because in most cases the returned value is `{failure: null}` and indicates that there was no failure (just no data yet).


Sounds fair. What about just returning `this.failure` instead of destructuring it inside `getDataNotFound`?


The changes seen here can be explained in the prior commit: https://github.com/ModelSEED/ModelSEEDDatabase/commit/801a704c79f2f3037784459cba6546f6017bde1c


@prashanthr05 this is the robot with IMU. You can add more sensors if needed. Also add the new objects you will create for the new surfaces to this branch.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 433](https://teamcity.labkey.org/viewLog.html?buildId=268831&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:42
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


Added maxlength for field metadata so string types can be converted to VARCHAR(n).

Added JDBCDialects for OracleDialect, NetezzaDialect and modifed DB2Dialect to use VARCHAR when maxlength exists.

Modified  existing getJDBCType function to take 2 parameters DataType and MetaData, so maxlength metadata can be used in JDBCDialects.

Can you review the change.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.453.4f18395488d892066e720b2f231301d19211342e](http://build.palaso.org/viewLog.html?buildId=62037&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3354, ignored: 134 Build time: 00:18:35


@linuxgurugamer no problem. Sent in a PR. :)


Do you think that makes sense for the equivalent of "package protected" properties? These are internal, but they get called from other files in this package rather than just internally.


Ok done.


Curious, does fruity mesh run on the smart beacon?  Nrf51 PCA20006?


You mean just

```js
return {failure: this.failure};
```

Probably reads a bit better. I've been using this style a bit in my code because it makes it impossible to typo it as something like `return {failre: this.failure}` or something, but it looks a bit awkward in this case.


I forgot to add, I integrated properly reaction balancing, status updating, and even reagent modification into the same script, and was able to do so without working with the original database objects, and thus made the Update_Reaction_Status.pl scripts redundant.


what about these? and any instance of created_at and updated_at


This isn't a duplicate, it allows for support of older MS Word files before they added the 'x' at the end!


and what with compiler variable in line 17?


Good point, disregard.


generic_builder.py:79 does something similar, would be nice to refactor that out since we make this query in alot of places


Actually, I've left out the minified files explicitly. I'd rather the developer use their own process to do so and not provide my own version.


I suppose it should have been

```javascript
compiler = webpack(config)
```


Found and fixed.
-JC

On Thu, Aug 27, 2015 at 12:29 PM, James A. Bednar <notifications@github.com>
wrote:

> Oops; I missed one: "Excution" is still in there somewhere, as it shows up
> on the server console output.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JosephCottam/AbstractRendering/commit/c7e88e8e2fc70682ecf98398bc2907feff5415e1#commitcomment-12930520>
> .
>


@sraue - i'm building an openelec retroplayer and I'm adding the game add-ons as packages. Each game add-on is a simple CMake wrapper around a libretro core, so I need to add packages for both the game add-on and the libretro core.

Here's [package.mk](https://github.com/garbear/OpenELEC.tv/blob/retroplayer-5.95.4/packages/mediacenter/kodi-binary-addons/game.libretro.2048/package.mk) for the game add-on wrapper.

Can you help with the package.mk for the libretro core? I just need to specify that the lib should be built with `make -f Makefile.libretro`. Thanks!


mmk


問題なさそうなので統合した。


needs $(...) -> backticks as well


What do we do about the index? Also, what do we do about negative array indices? Is that handled correctly down the line?


I see mixing tabs and spaces and having 2 space indents here! Naughty!


In a perfect world, that commit message would have said `Port connectionError() from Intuit's code example`...

...but this is not a perfect world.


Will do it when I have time.  Spend alot time, debugging new genfstab :/


赶紧的,想办法出国,然后......我们就更有所期待了.

目测国内要涌现一大批网络信息安全大师,想想就有点小激动呢


话不多说，谢！


It looks like ever since this commit, output capturing is disabled on AppVeyor, which is why the build logs are now huge and hard to read. I don't understand why - the option for capturing output doesn't appear anywhere.


After a bit of testing, this tends to put a 50x50 image. I've addressed it in 4983a6dfb3c0dcd4a030c6c83d092124331c4643 I think, but I suspect I'll mess up the merge.


SQL injection! Any reason you're not using the ORM here?


Was this supposed to say PHPLIB-118?


I think this is more a matter of style -- I tend to like the script to tell me if it's not doing something I told it to do.

That said, if you're going to add the `-p` argument, that will create parent directories automatically without asking or complaining, so there's no need for so many commands in the `wh` build target.  I think you could do the same thing with just two lines.

If you make that change, I'm happy to merge this PR.


This should be changed to 0.9965 to support USD because of the wider fees required for USD - .325%


101917310_waterCoilHumRatFalseWarning (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1801 of 1801 tests passed)

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


Thanks tom this looks good!, will accept this.


don't requireAdministrator, just asInvoker is fine.. if you want to elevate admin on startup then write to HKLM and it will automatically have Admin when startup


Yes of course !! Take a break ! - I'll also make some break ... ;-)


Might be... I currently don't have crashes when uninstalling a theme.


Changing Font Size


Font size again


One of the damn input sequences was 3-**1**-8-8-2 so I needed to allow 1 as a variable option.

This increased the minimum depth search by at least n for n variables, unfortunately.


是为了避免当nameserver刚启动时，由于chunkserver连接nameserver的先后顺序不同而造成的没用的块复制。但是这个new_chunkserver设置时，还没有添加分批汇报的功能，添加分批汇报之后，这里好像用处不大了。就是 #70 所说的问题。
core的信息可以发上来吗？


Thanks!


Nice working with ya!


new readme


The goto development_sku is missing a section so it might break there.  We can probably drop it for now until the SKUs all finalize.


Whoops! Updated that. :grimacing:


whats a noop?


Name of the class should not contain class or interface.Here ControlClass should be control or NetworkController


ここは
`hexString!` を使って実装したいが `phobos2-4.6`には入っていなかった。テストコードを見る限り、改行を含んだ文字列もコンパイル時に解釈できるようなので、`0x`を除いたcsvにしてやればいいと思う。
```
hexString!"30\r\n\t\f\v31 32 33 32 31 30"
```

* 参考
https://github.com/D-Programming-Language/phobos/blob/master/std/conv.d#L5539


No Operation

Often used to complete a flow of logic that leads to no action being
required, but left in just in case future code is needed later.

  _____

From: Js41637 ftw [mailto:notifications@github.com]
Sent: Thursday, 27 August 2015 14:26
To: SchizoDuckie/DuckieTV
Subject: Re: [DuckieTV] fixed #538 by including missing methods for patch in
#530 (fba533d)

whats a noop?

-
Reply to this email directly or view
<https://github.com/SchizoDuckie/DuckieTV/commit/fba533d44b388bb1675d43c49cd
03588f0e08890#commitcomment-12917115>  it on GitHub.
<https://github.com/notifications/beacon/AGnK-OlFvLbdYj_Z3_YAwpDeTIxLW71gks5
ormzLgaJpZM4FzBIU.gif>


101789044_gridviewFilters (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2312 of 2343 tests passed)

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


Looks good!


@achambers this is an attempt at drying up some of the repetition in the fake-redis-client

not loving how I reach into the internals from the `redis` object but I also don't think we should expose a public api just for this, let me know what you think


The TLS BCP is not relevant for what's required in TLS and there are valid reasons not to send SNI.


The TLS BCP is not relevant for what's required in TLS and there are valid reasons not to *send* SNI, which is what this PR requires.


My email id : Aapav01@gmail.com


Again, the BCP is not relevant here.


No need to instantiate map here.Apply null check for nodesInGraph where is used.
Use reference of Map,List,Set every time instead of HashMap,HashSet,ArrayList.


FixupPlantTempSource (kbenne) - i386-Windows-7-VisualStudio-12: OK (2304 of 2343 tests passed)

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


The right way to write this is something like:
"Note: TLS 1.3 servers MAY receive TLS 1.2 ClientHellos which do not contain this extension. If those servers are willing to negotiate TLS 1.2, they MUST behave in accordance with the requirements of RFC 5246"


closes #128 and #78


@munkie you did a great job since initial implementation :+1:


awesome! :tada:


ThermalStorageChilledWaterStratified-fixup (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2289 of 2343 tests passed)

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


RoomAirflowNetwork-Docs (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1800 of 1800 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)


Was v2 intended here?


oops, guess it doesn't actually do this on release-0.3, oh well


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1807 of 1808 tests passed)

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


This commit doesn't require sending. It says:

> All implementations MUST support usage of the "server_name" extension with
> applications capable of using it.

Servers have to be able to accept it and clients have to be capable of sending it, but there's no requirement to send here. I was under the impression that the WG was in agreement that it should be mandatory to send when applicable, but that's not in this commit. It is in the larger changeset which is now split off of this.


Not Change


Wow, great idea!

The updated version of draftsim sets:

pack_image.onload =  preload_bot_images()

so that future images are always loaded after the current ones.


It might be best to fill out the ```placeholder=""``` attribute or remove it until the appropriate good defaults are chosen.


Ok, that's quite clear. I'll change it to that.


FourPipeBeamAirTerminal (EnergyArchmage) - 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)


test


Thanks I added it for testing and forgot to remove.


Yes, I think so.


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


```.classname``` isn't the most semantic class name ever. I'd suggest something more like ```.pin-title``` or something along those lines.


I hate the name of these buttons


Whoops, let's remove this ;)


The behavior of `require` was designed to mimic Ruby's native `require` inasmuch as it simply searches the `$LOAD_PATH` for a file with that name and then `load`s it. The fact that it has a return value at all is because I wanted to make it slightly more useful than native `require`. In this case, we return the path to the file that was found.


Reverted
Causes weird loading errors


isEqual is only used for internal use of method. So no need to make is class member.


ThermalStorageChilledWaterStratified-fixup (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Should this still be skipped?


>Should wait for API update which will be done tomorrow.

Excellent, please let me know when it's ready and I'll update the IA on comserv.duckduckgo.com


RoomAirflowNetwork-Docs (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1800 of 1800 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)


Inicio


Very nice. Works like a charm, too.


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - i386-Windows-7-VisualStudio-12: Tests Failed

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


this isn't working for the SNES roms I've tried. Instead of a single file, they contain a single folder, which contains a single file. I added some logic to RetroPlayer that, if there is one folder and no files (with rom extensions), then enter the folder and look for roms there


I forgot to change it and leave it at 0, sorry.


@j717273419 不用小看了任何由国家意志推动的项目，涌现一大批「网络信息安全大师」，你觉得这些人完全听从国家意志的会占比多少？


As in before this it crashed all the time doing such operation, looking forward to test it out, thanks, it will also save me opening a bug report :D


Tests?


Probably not... THis stuff is really just black magic for trivial cases with one clear uplink. But for BindCarrier things are different, as there isn't a clearly preferred uplink, it just binds stuff together...


Is the code starting from <code> // hit pattern of the track </code> for debugging?

In the moment I make code cleanup and optimization, if the code do nothing I will set the remaining code
inside a comment or delete this part.

Thanks in advance for the feedback


test


I chose to leave those alone for the moment, we can also convert them if we want, but since we don't really do anything with them at this time, I figured I wouldn't touch it for now.


Unnecessary, maybe, but they are in line with our style of always using them in `new Thing()`.


looks good to me


I believe we have phased it out so if it doesn't show up feel free to toss
it.

On Thu, Aug 27, 2015, 09:58 Domen Kožar <notifications@github.com> wrote:

> nettl2.7 fails to build, do we even need it?
> http://hydra.nixos.org/build/25151613/log/raw
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/df000d93e9e67e3f8dc6f4375411a94d2aa9c757#commitcomment-12931155>
> .
>


This triggers a warning:

    errno.c:38:20: warning: extra tokens at end of #ifdef directive [-Wextra-tokens]
    #ifdef _GNU_SOURCE && !PLATFORM_ANDROID
                       ^

It also doesn't make sense in the context of this commit, as later you use `#if PLATFORM_ANDROID` _within the conditional block introduced here_.

I would thus infer that line 83 (`#if PLATFORM_ANDROID`) *shouldn't ever be compiled*, because you wanted it to be *excluded* here.

Please fix the warning and the logic of the `#if`-fu.


A lo mejor podemos evitar la dependencia de JQuery a la hora de hacer cualquier petición a un recurso usando `corbel.request.send()`, implementado en `src/request.js


How is the remake going guys? @miled @AdwinTrave I haven't followed HybridAuth's dev in a while but I remember I was so excited about the 3.0 update.


I probably should be reported, I will examine what is happening in a vanilla build.


Evitaría esta dependencia...En otro caso, habría que declararlo en el `karma.conf.js` para que estuviera disponible en los test


#4952-VRF-hard-crash-if-terminal-unit-list-error (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (755 of 755 tests passed)

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


This looks like it would only ever evaluate the string `'validator'`, which doesn't solve the problem unless the scope object has a method called `validator`. Unless I'm misunderstanding what this is doing?


Why would they want to deserialize things back to a lease object? I mean, I'd expect them to take the lease object, pull all the data they care about out, store it in their own structure, free it. And then they serialize/deseralize only their own structure...

I mean, the lease doesn't really give them that much that they couldn't do on their own...


It's okay, dad.


great !! great !!


Trailing spaces.


@rljacob @jnjohnsonlbl and @jgfouca: This commit adds support for titan. I've tested that it builds and works, but it requires this bug to be fixed before it works without manual intervention https://github.com/CESM-Development/cime/issues/129

Currently the problem is that the BatchMaker doesn't create `#PBS -A` lines in the *.run and *.st_archive scripts, so you can't submit the jobs as is. After adding these though, the cases build / run fine.


Can you just unindent this?


I think they should be changed -- they're date stamps, after all. We should really be making sure they're being used, too...


Fixed in e479a8297c66a2eea75e2d9dcf8f4abad3c77579


The comment is quite unclear; why not just say "for let-rec"?


I think ```views/tree_display``` may not been used any more. You can check [tree_display.jsx](https://github.com/expertiza/expertiza/blob/rails4/app/assets/javascripts/tree_display.jsx).

--Zhewei


Extraneous space.


Indentation.


Nit: Remove unnecessary whitespace on lines 64-65, to be closer to overall code style.


So this commit is a problem, I think, depending on its origin


@codeclarkcode

Let's take off the widths and add left/right margin of like 30px.


What about include paths? or other compiler flags?


Nit: Remove whitespace to be closer to overall coding style.


This shouldn't be commented out; it's still used for the database files if enabled.


#4952-VRF-hard-crash-if-terminal-unit-list-error (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1805 of 1805 tests passed)

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


Leftover debug printf?


That's what I mean, yes. I just agree that it looks a bit awkward.


Trailing space


Ditto


These HDF5 libraries are not necessary; only if using a static ASDF library should you need them. We should probably add some pkg-config type thing to ASDF so that these sorts of things can be checked.


Wrong level of indentation (see above)


Indentation


Comment?


booo.. you went back to the old title


A few double underscores should also be deleted if this software should not only be reused for implementations of C/C++ compilers.
An identifier like "v" would be sufficient here.


fair enough :)


trailing line, i can't believe you done this


"genome-fill-db" should instead be ```$0```


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.454.882a590c9020f9a5b71eda45743a8b5d3e2d6737](http://build.palaso.org/viewLog.html?buildId=62038&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3354, ignored: 134 Build time: 00:20:07


Der Inhalt des $options-Array in search.php wird bei gespeicherten Suchen gespeichert; dort sollte das visibility-Flag nicht rein, da redundant. Es müsste auch schon in dem Template verfügbar sein als $cachelist.visibility.

Die Änderung in search.result.caches.tpl sieht ansonsten ok aus, allerdings kollidiert sie mit dem neuen Commit 37485f7 im master. Muss also nochmal neu aufgesetzt werden auf dem aktuellen Master.


I commented out this block, can you restore the original behavior or explain me the feature so I can restore it?


You could use:
```java
@JsonProperty("start_time") private long startTime;
```
And stick to camelCase :smile:


As requested ;)


Having `@EnableModule` here might be confusing as all we need is to have `ChannelBindingAdapterConfiguration` and `SharedChannelRegistry` for the `AggregatorParentConfiguration`. Can we simplify this further like this:

```
@EnableAutoConfiguration
	@Import({ChannelBindingAdapterConfiguration.class, SharedChannelRegistry.class})
	public static class AggregatorParentConfiguration {

	}
```
which could also mean, we can remove `SharedChannelRegistry` from `@EnableModule`.


@ammarblue I just downloaded the code and seems to be working fine. Have you followed all the steps on the "How to use this code" section of the README file?


Since this creates a new listener, maybe name it `newListener()` or `addListener()`


I don't think define() is appropriate. WindowDef defines the definition of a window. This method creates an object that embodies the definition. How about instance()?


Should revert this as it could be Date instead of DateTime.  I'm not sure how I feel about changing the timestamp anyways.


Seems to work, notice a clear difference, only issue I can make out is [FACTION] entirely being broken. Otherwise it seems to be fine.


I think `toString` is causing the JSON to be re-stringified.

You should use `getString` here instead, so that you can delete the entire block below that handles escaped characters

http://stackoverflow.com/questions/24417947/difference-between-json-getstring-and-tostring-method-can-be-fatal


I think you are right.  It was used only on instructors' and admins'
homepages, and should have been superseded by Pei's new code.

Thanks,
Ed

On Thu, Aug 27, 2015 at 4:08 PM, Zhewei Hu <notifications@github.com> wrote:

> I think views/tree_display may not been used any more. You can check
> tree_display.jsx
> <https://github.com/expertiza/expertiza/blob/rails4/app/assets/javascripts/tree_display.jsx>
> .
>
> --Zhewei
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/expertiza/expertiza/commit/6772cb9e98aedca580026bdcf600b232a1aaaf1b#commitcomment-12934793>
> .
>


Same as an earlier comment. How about instance()?


Remove this, you no longer need it.


Did you mean to do this?

//URL status = new URL(url.toString() + "/status");
      //new UrlChecker().waitUntilAvailable(20, SECONDS, status);
    //} catch (UrlChecker.TimeoutException e) {
    //  process.checkForError();
    //  throw new WebDriverException("Timed out waiting for driver server to start.", e);


Ram used, swap used, ram free, swap free
![meterpartitionbar](https://cloud.githubusercontent.com/assets/13982108/9532148/c6a9a606-4cd7-11e5-9eab-3c6c0e8b8258.png)


The [`uv_pipe_getsockname`](http://docs.libuv.org/en/latest/pipe.html#c.uv_pipe_getsockname) doc says

> the returned length no longer includes the terminating null byte, and **the buffer is not null terminated**.

This should be consistent, right?


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.30.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=62041&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:39:31


Doesn't seem to be needeed?


Same as above.


In case you're not aware, this line seems to be ignored unless it comes before `[menu.main]`


If too much is hidden, I'll remove the options. Let's see what the output looks like.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 434](https://teamcity.labkey.org/viewLog.html?buildId=268858&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:43
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


[Upload Images to GitHub's Cloud Service](https://vinkla.com/posts/upload-images-github/)


Comment is a full sentence starting with a capital letter and finishing with a full stop.


#4952-VRF-hard-crash-if-terminal-unit-list-error (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1805 of 1805 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)


You can still have log that configuration is being loaded.


Thank you!! Shouldn't this issue be fixed in the mqlight package? Why would a client that subscribe to one topic receive messages with a different topic?


Indentation seems broken?


Who on earth came up with this project name?


getWebhooks


Now they have both :)


meh! Nice catch.


@marr Just as this example sets the value, you should set .checked first. TestUtils.Simulate only uses React's internal event system; it does not touch the actual DOM at all so it can't change any of the DOM properties.


@marr I tried a few times to simulate a click on a checkbox, but without much luck :unamused:


No-op?


@nawilloughby - '&' needs to be changed to & on line 82


`util_config.h` should be on the very top of the file. Order is:

```
#include "self_header_file.h"
#include <system_includes>
#include "includes_of_project.h"
```


configPath?


New


@spicyj so the correct way should be to change `.checked` on the node and then call `Simulate.change` passing in the "mutated" node?


You can have `using std::getline` and then simply do `getline(BLAH)`


You can use slf4j placeholders here


`if (condition)`


Not sure I fully agree or not.

Kitware already bundles stuff like [this](https://github.com/Kitware/CMake/blob/master/Modules/CMakeDetermineJavaCompiler.cmake), and I don't see how llvm/libclang is that different.

At least not if the detection is done for `llvm` (as opposed to `libclang`) and then users of the module can interpolate where they'll find `liblang` based on the `llvm` location-detected.

Not really a big deal to me and I guess I'll let @jaeandersson decide what he thinks is the right move :)


Oh it's because the indentation is always ignored, so top level keys have to appear before all sections (called tables):
https://github.com/toml-lang/toml#user-content-table


Theres a lot of templately stuff in this commit that is removed in other commits. The main thing to look at in this commit is the actual task tasks/fail_fast_task_runner.js


nevermind, i got it (i think): https://github.com/garbear/OpenELEC.tv/blob/retroplayer-5.95.4/packages/mediacenter/kodi-binary-addons/2048/package.mk


Thanks, fixed.


Much better now, just some simple feedback to do while Ton doesn't watch.


This is what you want to look at


A client that is subscribed to one topic will only receive messages on that one topic.

That is not what we have here though. If you have multiple mqlight nodes using the same broker configuration, they all run over the same connection. When a message arrives over that connection it needs to be routed to the node(s) that were subscribed to the topic it arrived on. This is how we do that.


@ryansb and/or @liam-middlebrook and/or @msoucy will likely have some git-foo or input here that may come in handy, but I think I should chime in on the utility of /people.

There was a feature in the latest ofcourse to create routes to view .yaml files from previous years within people/$YEAR/$SEMESTER, so that historical blogposts and participants could be viewed by the current cohort of students.

Because there is no utility within ofcourse to migrate/import that people data when you create a new instance, this would likely be a manual process, and optional, but could come in *quite* handy. For instance, if/when a student asks you somehting like "What bug fixes have been submitted in the past?" you can just say "Check out http://hfoss-ritigm.rhcloud.com/2015/fall and scroll down to see for yourself!"


Going to do another sweep through to see if there are any other remaining Oakland references remaining.


Should actually be `#endif  /* BLAH */` (aka two spaces)


Did you not read the repository description? It's a proxy server that
translates pocket edition packets to pc packets.

On Thursday, August 27, 2015, RandomAltThing <notifications@github.com>
wrote:

> Is this going to be like a plugin or something?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jython234/RedstoneTorch/commit/19acbb47bf899f2f634ddff2299b7182b261e218#commitcomment-12934192>
> .
>

--


я бы все-таки назвал файл так что бы это было связано с прогнозом или погодой. Заодно можно посмотреть как работает git mv


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 406](http://192.168.30.225:8111/viewLog.html?buildId=2271&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Hi
Yeah. I could running it , after reading the your README file.
Thank you for your interest.
I am sorry for urgency.


Название города имеет смыслом сделать названием станции.


Awesome!


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2302 of 2343 tests passed)

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


Currently PHP stores all class names in lowercase


Yes.


91958608DOASEffectOnZoneSizing (wfbuhl) - 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-81.81%25-red.svg)


register this guy as a MutliTask to allow for multiple configurations, defined using arbitrarily named "targets."


I'd prefer to keep it as it is - and make module aggregation as similar as possible to embedding a module in another module (which is something that we'd want to do in the future). Would documenting that make things easier to understand?


May I suggest 501377A there? :P


Agree, will change it.


@jalbano We are also seeing this issue after running a fresh `pod install`. Any insight on solving this?


function that will write data to grunt log


You spelled "improved" wrong


yoyoyo @tunamonster check for review. picture in template is missing, I started it, you can check it on branch adding-picture-upload, file upload works but it gives error because of your functions when trying to view it. don't fully understand your structure of the template so we can have a look at it next week

ignore the pictures needed them for testing lulz


WorkerThreadSpinTime should be 0
If it's not 0 in Area17, that could well explain the stutters people have
Also, WorkerThreadCount >2 is useless.


concurrently run tasks as they are pushed to the queue. default is set to 1


also very important that we have to set up fog to store the pictures on a cloud service before going into production


get any extra arguments passed in to pass on to children tasks


Mmm, I am very used to toml's, but I tried the alias on the about.adoc and there it works (no toml table). But why does it not work here, even if I moved the aliases to the line between `[menu.main]` and title?


I haven't tested this recently. It's possible that a new version of Dropbox has broken this. What version of the Dropbox pod are you guys using?


start spawning children processes that run grunt tasks


This is too much complexity, imho. If we really want to keep the main methods for running tests we should at least come up with a solution that can be documented and modified in a single place, rather than being copied and pasted everywhere. Also, maybe the condition should be [s for s in dir() is isinstance(s, TestCase)]


this is called when the queue is empty. We tell grunt that we are done with a true(success) or false(failure) status


configurable concurrency


Between `[menu.main]` and title, it works for me (C).


Noted, thank you. I seem to get hung up on the `Simulate` stuff way too frequently.


@natep thanks for responding so quickly. We're still getting the version of DropboxSDK inherently imported from DropBlocks; it appears to be `dropbox-ios-sdk-1.3.13`


Using isinstance doesn't work here.

On Thu, Aug 27, 2015, 16:40 Omar Laurino <notifications@github.com> wrote:

> This is too much complexity, imho. If we really want to keep the main
> methods for running tests we should at least come up with a solution that
> can be documented and modified in a single place, rather than being copied
> and pasted everywhere. Also, maybe the condition should be [s for s in
> dir() is isinstance(s, TestCase)]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sherpa/sherpa/commit/1ce5d1ce7adf6f56a1b7f826eaca67c88a808c73#commitcomment-12935455>
> .
>


yep. I changed it about a minute later. :-)


loop though all the gruntfiles we need to run and push them on to the queue to run


Good! =)


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 406](http://192.168.30.225:8111/viewLog.html?buildId=2271&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:10:45


@luca020400 I changed the tab spaces & in live did effect, but not really /facepalm lol


I'd go with that option to err on no initial value, if people request something like that later.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 435](https://teamcity.labkey.org/viewLog.html?buildId=268850&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:09:59
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


Hi Zhewei, could you do me a favor change the default value of Has quiz field from "" to "No"? Thanks!
![image](https://cloud.githubusercontent.com/assets/4420550/9532614/b3c066e4-4cda-11e5-8532-03bd38255eea.png)


That's not helping to deal with nil, but requires to manually `val = x.y; val = val == nil ? 0 : val` which reads so badly against `x.y.to_i`. I'm using this all the time in Ruby to sanitize input :'(


here is the magic callback that will make it fail fast. If the child process encounters an error, it is passed to this callback and we kill the queue and tell grunt we are done.


Yeah, you are right, that was stupid.


:tada:


Fixed


You can checkout the README here https://github.com/gla5001/grunt-fail-fast-task-runner


These tests smell like :shit:


For the record, we're successfully importing Dropbox in our `AppDelegate.m` via:

`#import <DropboxSDK/DropboxSDK.h>`

Although we do not explicitly add the DropboxSDK to our `Podfile`. Interesting.


Isn't this an error scenario too?


Ahh yeah, my grepping was wrong.... except that hugo removes the + sign.


@roneigebert
O algoritmo estava efetuando a conversão manual de 1024 em 1024, por usar de uma técnica chamada Buffered Copy. Ela garante boa performance na leitura de arquivos grandes. Considere um micro-benchmark para isso antes de fechar esta issue.


Hmmm... And for this option, the default would be? Not sending error packet?


There's not much to test here and what I could possibly test is very difficult to figure out. and since this is just a hackathon, im ripping them out


@ysbaddaden But why zero is a sane default? Here the compiler will give you an error and you can sanitize to whatever value you want: `x.try(&.to_i) || 0`


Ok, let me investigate.


Indentation issues?


Ok, now I think I got a better grasp of it, thanks.


Extraneous line?


How about do this on other dependency? or specify it to bash or sh?


Isn't it worth it to add: else (efivars case) name will mean the efivar name or something like that?


Grumble grumble


ok, My reasoning is to scope the `SharedChannelRegistry` into `ModuleAggregator` only as it isn't needed by standalone module. IMO `@EnableModule` is in its terms focus on enabling **per** module channel binding. Even if we use the similar approach like embedding module into one another, we would probably need better classification of these configuration classes. I also noticed that `SharedChannelsConfiguration` can be moved into `AggregatorParentConfiguration` so that aggregator parent would look like this:

```
     @EnableAutoConfiguration
	@Import({ChannelBindingAdapterConfiguration.class, SharedChannelRegistry.class})
	public static class AggregatorParentConfiguration {

		@Bean
		@ConditionalOnMissingBean(SharedChannelRegistry.class)
		public SharedChannelRegistry sharedChannelRegistry() {
			return new SharedChannelRegistry();
		}

	}
```


All sorts. Whitespace is a mess and I don't feel like cleaning it up now until I have a good coding style to follow.


Given the experience with a previous deprecation, where everybody switched away from it immediately, breaking compatibility, should we not wait 2 cycles to deprecate this?


@decause I also mentioned this here https://github.com/ritjoe/hfoss/commit/53bf8c6812b8c99308a1a2fede3165efef6523df#commitcomment-12893248


Except that you can disable specific blocks in a mod, so even if the mod is present, the block might not be, so we still need to ignore that block and just report a warning/info of sort.
Even without that use case, the modid is exactly what starts the block name before the ':', so it would still be 'redundant' ;).


That would do.


This should be applied to pet names as well.


:+1: thanks for this!


Ditto.


This use case is one the first issue I had to fix in the mod on 1.6.4.


Fair.


JavaScript is weird.


```bash
ZeroDivisionError: float division by zero
```
が出る場合があるので例外処理をする


Imports for java.lang are implicit. This can be removed.


Not sure really - only 'fs' is an exception, I'll believe all others will be just a name.


Why is FpMath an enum? It should be a final class with private constructor.

And what is the point in using own abs methods that do exactly the same as Math.abs?


Лучше сделать всё поля private


kp


И тут


3 spaces indentation instead of 4?


:grinning:


@asterite `Nil#to_s` should go away also using the same criteria, right?

It seems the semantic around case will need to be improved to get rid of those superfluous nils.


Second design accomplished. (pushed to branch sibling2child ) Now switching to adoption of 3rd design for Weave Analyst.


I didn't think to do the others, but simply checking for "<executable name> not found" does the trick.

Sent from Outlook

On Thu, Aug 27, 2015 at 1:49 PM -0700, "Han" <notifications@github.com> wrote:

How about do this on other dependency? or specify it to bash or sh?

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


Isn't the point of the test case the instanceof? Should callMe use objFloat? Is callMe needed at all?


This breaks tests on OS X:

```
ERROR: LoadError: On worker 7:
LoadError("/Users/stefan/projects/julia.alt/test/abstractarray.jl",469,ErrorException("test failed: 8 == 32\n in expression: Base.elsize(B) == sizeof(T.parameters)"))
 in default_handler at test.jl:28
 in do_test at test.jl:53
 in test_primitives at /Users/stefan/projects/julia.alt/test/abstractarray.jl:207
 in anonymous at no file:472
 in include_string at loading.jl:233
 in include_from_node1 at ./loading.jl:274
 in runtests at /Users/stefan/projects/julia.alt/test/testdefs.jl:179
 in anonymous at multi.jl:889
 in run_work_thunk at multi.jl:642
 in anonymous at task.jl:889
```


Thanks Martin (@mmaechler).

I just checked, the modifications to the regexes that I did back in May were only whitespace changes. So the new code is not responsible for `(beginning-of-function)` bugs.

I'll address this issue and others related to function motions in a month or so.


Looks good though you may want to consider changing your version to 1.0 when you're finished.


Would probably be a good policy to always go through CI for tests that change things that might be different on different platforms, e.g. code gen.


C'est pour que le singleton s'init.


With the shell name that will be much clearer.

I still think there should be only one XML format and we read what's in it, whether it is structure, filler or whatever.


this is a lot of 3rd party dependencies for a grunt plugin. you sure you can't make do with less?


Great, thanks @natep -- let me know if there's anything I can do or try to help.


Maybe we should lisr the Tango Project as the source under About c:geo


Поля класса инициализируются в конструкторе Transaction(String, int, String). В данном случае, я бы сеттеры не писал.


nothing really useful to add. looks like it was a lot of work.


I just realized that I've not got a good idea whether any other piece of KBase code uses addReactionFromHash(), as changing it to fit Mike's ProbModelSEED approach would break the code. I'll come back to that later.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 436](https://teamcity.labkey.org/viewLog.html?buildId=268856&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:07:43
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


1579840-GroundTemps (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

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


mostly updated search.py

rest can be ignored as changes will be overwritten when validation-rework branch is merged


That test looks wrong for me, It should probably be `T.parameters[1]`


making the directive contain the (1) crumb container, (2) its crumbs and (3) its display options list


Here: https://github.com/bwrsandman/vcmi_editor/commit/eea222e0ff7c3a659e0f44a8fde7f96eb2394a13


Also, whilst I am one for abstracting out code, this is the only place I
think we need it (and will hopefully go away if we re-architect
sherpa.astro.io) so I'm not sure is worth spending a lot of time on it at
this moment.

On Thu, Aug 27, 2015, 16:45 Omar Laurino <notifications@github.com> wrote:

> Yeah, you are right, that was stupid.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sherpa/sherpa/commit/1ce5d1ce7adf6f56a1b7f826eaca67c88a808c73#commitcomment-12935574>
> .
>


Example of how to exclude unreachable branch
Issue #11


Лучше называть такие файлы .sql, тогда редакторы текста будут понимать что это sql и подсвечивать синтаксис


root@OpenWrt:/# swconfig dev switch0 port 0 get status
Unknown attribute "status"

root@OpenWrt:/# cat /etc/openwrt_release
DISTRIB_ID='OpenWrt'
DISTRIB_RELEASE='Chaos Calmer'
DISTRIB_REVISION='r46698'
DISTRIB_CODENAME='chaos_calmer'
DISTRIB_TARGET='ar71xx/generic'
DISTRIB_DESCRIPTION='OpenWrt Chaos Calmer 15.05'
DISTRIB_TAINTS='no-all'

For sure you are using the swconfig with CC?


93651760-RowPerObjectReport (JasonGlazer) - x86_64-MacOS-10.9-clang: Build Failed

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


I don't think it matters for Makefile.am, but the indentation is off by one on this new line.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.932.83b66f807a2c31af6ebc9d4b26788e5e66f49767](http://build.palaso.org/viewLog.html?buildId=62059&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:08:09


Perhaps this shell command can be assigned to a variable so the name indicates what it is doing.


Thanks for pointing this out. Should we never use `toLowerCase` and always use `toLocaleLowerCase`?


1579840-GroundTemps (mmatts) - 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 Chorus :: chorus-precise64-master Continuous [Build 1.5.451.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=62051&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:32:41


#7 Implementation of next C# 6 specifications


#4952-VRF-hard-crash-if-terminal-unit-list-error (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1281 of 1281 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.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.99%25-green.svg)


Woo! +1


Thank you for your contribution, could you update the pull request with some information about which type of export caused the issue? The best thing here would be to have an minimum IDB which triggers the behavior.