Backtrace in gdb:

```
#3  0x00007ffff4632c32 in __GI___assert_fail (assertion=0x1cc9df2 "contains( i )",
    file=0x1cc9d98 "/home/elee/EnergyPlus/GitHub/EnergyPlusTeam/third_party/ObjexxFCL/src/ObjexxFCL/Array1.hh", line=1167,
    function=0x1ccb040 <ObjexxFCL::Array1<EnergyPlus::DXCoils::DXCoilData>::operator()(int)::__PRETTY_FUNCTION__> "T& ObjexxFCL::Array1< <template-parameter-1-1> >::operator()(int) [with T = EnergyPlus::DXCoils::DXCoilData]") at assert.c:101
#4  0x0000000000e2f141 in ObjexxFCL::Array1<EnergyPlus::DXCoils::DXCoilData>::operator() (this=
    0x245b060 <EnergyPlus::DXCoils::DXCoil>, i=1)
    at /home/elee/EnergyPlus/GitHub/EnergyPlusTeam/third_party/ObjexxFCL/src/ObjexxFCL/Array1.hh:1167
#5  0x00007ffff70b2dcb in EnergyPlus::WaterThermalTanks::CalcWaterThermalTankStratified (
    WaterThermalTankNum=1, HeatPumpPartLoadRatio=1)
    at /home/elee/EnergyPlus/GitHub/EnergyPlusTeam/src/EnergyPlus/WaterThermalTanks.cc:6273
#6  0x00007ffff70d6947 in EnergyPlus::WaterThermalTanks::CalcStandardRatings (WaterThermalTankNum=1)
    at /home/elee/EnergyPlus/GitHub/EnergyPlusTeam/src/EnergyPlus/WaterThermalTanks.cc:10195
#7  0x00007ffff7052df7 in EnergyPlus::WaterThermalTanks::SimWaterThermalTank (CompType=16,
    CompName="PLANTHEATPUMPWATERHEATER", CompIndex=@0x7fffffff9af0: 1, InitLoopEquip=true,
    MyLoad=@0x7fffffff9b30: 0, MaxCap=@0x7fffffff9b38: 6.9533558065954924e-310,
    MinCap=@0x7fffffff9b40: 6.9533558065962829e-310, OptCap=@0x7fffffff9b48: 6.9533558065970734e-310,
    FirstHVACIteration=true, LoopNum=..., LoopSideNum=...)
    at /home/elee/EnergyPlus/GitHub/EnergyPlusTeam/src/EnergyPlus/WaterThermalTanks.cc:443

```


the `break` is still misindented!


Does not need to be a `var`


> How a field behaves / moves (or not) is generally determined by the field's own movement-related effects (or lack thereof). Being created in a system or at some other location in space is irrelevant to those effects.

Just out of curiosity: If I insert a moving field into a system, what will happen? Will all attempts to move the field be ignored by the engine, will the field move away but still be technically contained by the system, or will the field move away and removed from the list of objects contained by the system?


What happens if the `Filter` is set and it specifies the `"query"` parameter? I'm not familiar honestly, but it seems like either an exceptional condition or one will trump the other.


I agree... But it's just an example of user space compilation of all dependency.  I really think the cmake strategy will be much better


Can we change this to exit 1 since Windows has crashed the terminal?


This shouldn't be necessary if you are building out of the source tree, which is generally a good idea anyway.


does trailing need an offset here?


LIMIT 1 made it not work and yeah it's slower. If you have a solution, share it @Whitetigerswt


Oops, I misreae


@acdha given this new logic, should we still keep ```should_update```? Seems a bit repetitive to have both, best to have one way that will work when updating individual object or bulk


Merge pull #1


it's just a default. when the app is ready we can point it to production. but it could always be overriden in an initializer.
but if you want we can force the client to explicitly set the vigilion url like we do with the credentials.


Argh! Slipped past, somehow. Fixed.


I force pushed ontop of it - there possibly attached to the other commit?


If you look at http://opencreds.org/specs/source/identity-credentials/, you can see that the issues are linked to GitHub, but do not include content. They did when using respec-gh-issues. When I stepped through the code, as I mentioned, I did not see the `body_html` entry in the returned JSON; just `body`. However, this may have to do with the opencreds.org site, as if I access using [htmlpreview](http://htmlpreview.github.io/?https://github.com/opencreds/website/blob/master/specs/source/identity-credentials/index.html), the content shows up.

Perhaps this is something @msporny or @halindrome knows something about.


Well that escalated quickly :)


I like the suggestion with the verbosity setting.  I'll implement that.  Thanks a lot for the review and the clarifications.  It might take a couple of days until I get to everything.


The one in the [https://github.com/jline/jline2/blob/f1127ba421e9b5ee24c450b336086881632a630d/src/main/java/jline/console/ConsoleReader.java#L2381 ConsoleReader at line 2381]. That won't compile on JDK 1.5. You need 1.6 or later for that.


Why not `assert(info.exists, "undefined: ...`


Why? this only failed in a transient way if you look at http://hydra.nixos.org/job/nixos/trunk-combined/nixos.tests.installer.grub1.x86_64-linux/all


Great!

As you note, I think there are probably enough low-hanging fruit performance-wise in this module that we may be able to incrementally improve it.


Hmmm - I had wondered about how to deal with that possibility. Will do.


Are we using jquery on these pages? I noticed the use of `$(document).ready()` without any jquery library being loaded. I don't think we need it.


Quick test:

```cs
JObject x = new JObject();

x.Add("query", JValue.CreateString("x"));

x.Add("query", JValue.CreateString("y"));
```

This does result in an exception:

> System.ArgumentException : Can not add property query to Newtonsoft.Json.Linq.JObject. Property with the same
name already exists on object.


fyi, if this part was performance critical, we would actually want if conditions here to avoid false sharing of the cache line across multiple cpus.


Prefer a `const`


It's convenient for the authoring form. All pages on the site share common imports, for simplicity. By the time someone gets to one of these, any libraries should be cached.


Maybe force push nukes comments.


Hmm, good to know. Let's revert this part then and add a comment to serve as illustration for others to follow?


can we have description to define the position of the sub stage in the pipeline? For example, for this PORT_ACL, it is at the beginning of the ingress pipeline, before any vlan, l2 or l3 processing.


Actually this is version.1


this is a hot path.
```
assert(info.exists, "undefined: $tp with symbol $sym")
```
will need to form a closure that represents a second argument, even if `!info.exists` and will lead to slowdown.


Oh! My eyes completely missed this line:

```cs
queryRoot = new JObject(new JProperty("filter", Build(searchRequest.Filter)));
```

Looks good! Sorry for the noise.


where is this L2 ACL substage?


is there any danger that this could override manually set values? (make sure we test this)


Should Facter be capitalized in these links? It is pretty much everywhere else.


I'll revert it, but I don't see the logic behind using an end iterator wrapped in an adaptor_sentinel instead of an end iterator wrapped in an adaptor_cursor when the adapted range is bounded but not single pass.


Sorry for delay in works, because this week begin was a hard for me and I was too tired from a job. Anyway I already have a plan for a next update ;-)
I will try to continue works if my tooth will not have a pain or I will don't asleep (because a long path between home and job I should wake up in 5:30 AM)


Because figure elements have margins by default.

See diff without (non-indended) whitespace changes here: https://github.com/htor/3dtransforms/commit/330f56df13b2b066bf3ef0b087df9f832da47f16?w=1


very astute observation, @jfalquez!


Yeah! The only thing that feels slightly hacky is that I've confined the notion of a metavar into this module to avoid overhauling the entire compiler so in a `match` tactic you just have leave vars unbound to signify unification vars. I think there are some tricks that can be played in the parser to fix this but I'm not sure


Does this need to force layout?


If you wanted, you could move these all into lazy initializers like we've been doing.


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

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


but compile with SD2 in "j -4" not working yet=(


I suspect there are other bottlenecks, so it's minor. whichever way.


Shouldn't we use OBJECT_PATH_FOREACH_PREFIX for this?


@atomb I'm inclined to keep the server in the main Cryptol package if we plan to eventually make the console REPL a client of the same API. However I wonder if we could abstract away the transport layer so that if zmq isn't present we could, for example, use regular TCP sockets.


I'll just merge it for now.


Sounds like something to use strdupa() for, no?


My friend and I are trying to play on my server and every time he tries to load it. It says "This Modpack requires 2.5 GB RAM. You are using 1 GB RAM. You can use 2.5 GB" and "This modpack requires Java 1.8. You are using 1.7.0_79 64-Bit.". When he changes the Java it doesn't let him have the 2.5 GB of RAM. When he has the 2.5 GB of RAM he can't use the current Java. Please help!
![cd883d60ebf1b889d7cea616739e0dff03bd457d461b3bf25a pimgpsh_thumbnail_win_distr](https://cloud.githubusercontent.com/assets/13456096/8833463/9e88faae-3063-11e5-80c2-61863132a3c2.jpg)


Am doing some testing on a 20-core Linux workstation. Here are the data so far:

MAX_SLAVES_PER_SPLITPOINT=4
./stockfish bench 8192 20 30 /home/louis/Documents/Chess/Testing/SF_benchx5.epd depth
===========================
Total time (ms) : 3039928
Nodes searched  : 71046031931
Nodes/second    : 23370958

MAX_SLAVES_PER_SPLITPOINT=6
./stockfish bench 8192 20 30 /home/louis/Documents/Chess/Testing/SF_benchx5.epd depth
===========================
Total time (ms) : 2854586
Nodes searched  : 66135230975
Nodes/second    : 23168063

MAX_SLAVES_PER_SPLITPOINT=8
./stockfish bench 8192 20 30 /home/louis/Documents/Chess/Testing/SF_benchx5.epd depth
===========================
Total time (ms) : 2866966
Nodes searched  : 66094998690
Nodes/second    : 23053987

Am about to try MAX_SLAVES_PER_SPLITPOINT=7.


It seems that scipy is not supported by the test system, so I remove the lines calling that library. By now, I just call it for evaluate MatrixExp, but it would be useful for sparse matrix support...


I noticed this only after merging, but you set variant=thing, while the commit doesn't include variants/thing directory. Does SparkFun ESP8266 Thing need a separate board variant?


So @nmerket it's throwing on this line: https://github.com/NREL/EnergyPlus/blob/develop/src/EnergyPlus/WaterThermalTanks.cc#L6273.

Here is a summary of the ```HPWH``` variable at that point:
```
$1 = (const EnergyPlus::WaterThermalTanks::HeatPumpWaterHeaterData &) @0x2a20ae8: {
  Name = "PLANTHEATPUMPWATERHEATER", Type = "WaterHeater:HeatPump:PumpedCondenser", TypeNum = 16,
  TankName = "WATER HEATER TANK", TankType = "WaterHeater:Stratified", TankTypeNum = 32,
  StandAlone = false, AvailSchedPtr = 5, SetPointTempSchedule = 6, DeadBandTempDiff = 2,
  Capacity = 25000, BackupElementCapacity = 4500, BackupElementEfficiency = 0.97999999999999998,
  WHOnCycParaLoad = 10, WHOffCycParaLoad = 10, WHOnCycParaFracToTank = 0, WHOffCycParaFracToTank = 0,
  WHPLFCurve = 0, OperatingAirFlowRate = 1.00695, OperatingWaterFlowRate = 0.0011552499999999998,
  COP = 0, SHR = 0, RatedInletDBTemp = 0, RatedInletWBTemp = 0, RatedInletWaterTemp = 0,
  FoundTank = true, HeatPumpAirInletNode = 0, HeatPumpAirOutletNode = 0, OutsideAirNode = 13,
  ExhaustAirNode = 17, CondWaterInletNode = 14, CondWaterOutletNode = 15, WHUseInletNode = 3,
  WHUseOutletNode = 4, WHUseSidePlantLoopNum = 0,
  DXCoilType = "Coil:WaterHeating:AirToWaterHeatPump:VariableSpeed",
  DXCoilName = "HPWHPLANTDXCOILVS", DXCoilNum = 1, DXCoilTypeNum = 0, DXCoilAirInletNode = 16,
  DXCoilPLFFPLR = 4, FanType = "FAN:ONOFF", FanType_Num = 3, FanName = "HPWHPLANTFAN", FanNum = 1,
  FanPlacement = 1, FanOutletNode = 16, WaterHeaterTankNum = 1, OutletAirSplitterSchPtr = 0,
  InletAirMixerSchPtr = 0, Mode = 0, SaveMode = 0, SaveWHMode = 0, Power = 0, Energy = 0,
  HeatingPLR = 0, SetPointTemp = 60, MinAirTempForHPOperation = 11,
  MaxAirTempForHPOperation = 48.888888888890001, InletAirMixerNode = 0, OutletAirSplitterNode = 0,
  SourceMassFlowRate = 0, InletAirConfiguration = 3, AmbientTempSchedule = 0, AmbientRHSchedule = 0,
  AmbientTempZone = 0, CrankcaseTempIndicator = 3, CrankcaseTempSchedule = 0, CrankcaseTempZone = 0,
  OffCycParaLoad = 0, OnCycParaLoad = 0, ParasiticTempIndicator = 3, OffCycParaFuelRate = 0,
  OnCycParaFuelRate = 0, OffCycParaFuelEnergy = 0, OnCycParaFuelEnergy = 0,

```


Ugh, just discovered it's actually supposed to be in 1.5, but gcj (which Eclipse is using for 1.5 under RHEL 6), [doesn't have it](https://gcc.gnu.org/ml/java/2014-09/msg00003.html). Looks like this is a [gcj-specific](http://comments.gmane.org/gmane.comp.gcc.java.prs/7823) bug. What a pain!

So, even if jline2 doesn't bump to Java 6 requirement (which, I really think it should do at this point, but not my call), it's probably still good to refactor this bit of code to avoid this constructor, since many people are likely still using gcj for their 1.5 development.


Please use the issue tracker, not a comment on a commit :P


Thanks, I hope you didn't mind that I took a look although you assigned for the issue.

If you move the line down to just before the session manuscript are loaded I think it will be fine.

In the unlikely(?) case where the session has stored manuscript that are already loaded in the viewer it may be faster to check for this, but I think we can get away with not considering that...


Скорее всего, останется только places, а "конвертация" будет происходить при сохранении файла конфигов.
Хочется хранить order для наглядности, чтобы при открытии файла конфигов текстовым редактором не спутать, что places это не order. Пока не знаю, веская ли это причина для хранения order.


Okay, looks like the answer is "yes", because Thing has SDA and SCL pins assigned differently from generic variant. Will add variants/thing then.


No, see http://hydra.nixos.org/build/23928152/nixlog/6/raw:
```
machine# [  717.136002] Kernel panic - not syncing: assertion "i && sym_get_cam_status(cp->cmd) == DID_SOFT_ERROR" failed: file "/tmp/nix-build-linux-3.18.18.drv-0/linux-3.18.18/drivers/scsi/sym53c8xx_2/sym_hipd.c", line 3399
```
which is exactly the original panic which was supposed to be fixed.


Oh My B


Where do i do that?


Just asking, I didn't check this. But does this test makes sense at all without PWM?


This looks promising Chris!  I don't have a screen wide enough to actually understand the output.  How can we make that more readable? Maybe have the various log attributes be specified with some additional flags?

force trace list -logs=co,sc,ac,ap,db,sy,va,wf,vf
co = Callout
sc = Scope
ac = ApexCode
ap = ApexProfiling
db = Database
sy = System
va = Validation
wf = Workflow
vf = Visualforce


configure**Robot**Logger()


Probably something @kaysievers should review, but from a superficial look: indenting of the if block is borked, needs to be multiple of 8 spaces.


Where is this used?


Это же значения из выпадающего списка, нет разницы, хранить их с большой или маленькой буквы.


Thanks!


Ideally, we should only turn off the calamari nodes related to SPI (PWM etc). One should be able to use calamari nodes not spi related if only spi is turned off, isn't?


@Armada651: That doesn't justify dummy window in any way. Really I have it working to stay fullscreen in no time. Just a question, what is happening with focus? I mean if you click on other window should I steal focus to our FS window? Should I let it be defocused? Either way seems weird to me.


I'm also new to this so very sorry.


An option is to change it to `macro += new String(Character.toChars(c));`


@sellout: Not saying this is a good idea, just saying. It's simple, easy to debug and extend, and flexible enough to generate whatever we need. And it doesn't require debugging any Scala macros, so only a normal-sized brain is required.

The main problem is it's not really clear where the type would be declared. It has to be somewhere it can be loaded *before* compiling `core`, which means either over in the other project, or in `build.scala` directly, or maybe parsed from JSON (also see the longer note in the diff). All of those options suck, which I guess is what makes macros attractive.

Anyway, this was a fun exercise. Any reaction?


This change breaks relative imports in Dafny: when filename is a full path, this looses the directory part. I'll push a fix to the codeplex repo.


@glasser What's this syntax here in the constructor parameter list? I've never seen an object in the parameter list like that before. Is it parameter list destructuring, as in the function actually takes only one argument (an object containing keys)?


No, this means it **requires** VS2015 to generate the SDK. You can still use the templates in previous versions of VS. This is only used for generating the templates when creating the SDK.
(VS product names and version numbers aren't the same thing - VS2015 is Version 14. The only time the year and version lined up was in VS2010 where the version also happened to be 10. )


Since people use it, we'd need to wait for a major release to remove it. I'd probably save something like this for a more ambitious refactoring of the index/backend relationship in any case since there doesn't seem to be much benefit to removing a style which some people prefer and doesn't use much code. `should_update` does have the advantage of making the filtering style very clear as opposed to having a `raise` in a larger block of code, so perhaps it would be good to start by refactoring so both go through a common path using `SkipDocument`


I'll update the pull request with that, more reserved, change, and the project can decide on Java 6, later.


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


When the range is single pass, the state of the end cursor is never referenced. Better to use an empty sentinel.


oops, this might be jumping the gun a bit... 4.4 is not tested yet.


OMG... **rolleyes**


When you moved the dockerfile into test-images/Dockerfile.1, the image name changed. It is now `atomic-test-1` rather than atomichttpd. You have this image name in the `INAME` variable.

`${ATOMIC} run --display -n TEST1 ${INAME}`

Also, rather than piping the results of `atomic run` to a file and reading it, you can just do something like

```
OUTPUT=`${ATOMIC} run ...`
if [[ ${OUTPUT} != ... ]]; then
    exit 1
fi
```


I'm leaning towards not accepting this patch.  The point is to provide all variations for UUID generation, which includes the hashing ones.  I would prefer to link against another library that provides MD5 and SHA1 if it's available than to remove the functions.


Correct, it's part of ES2015 (formerly known as ES6).  You can see that this plugin uses the `ecmascript` package, which implements ES2015 transpilation using Babel.


"zomg if defs are coming"

+1


We are already clearing everything except linkClickID and LinkClickIdentifier on key change . Only missing was linkCache.


If an object, such as a field or fleet, in a system is moved by an effect, it is removed from the system.


:wink:


Thank you. I am aware of Lacertium's effects - I've looked around the code a few times. As for the breast size, the "stock" naga enemy is size 10. Protagonists should be bustier? ;)

Any suggestions which stats should this modify and by how much? I've tried to look for some pattern in the other transformations, but without much success.


for bulk saving it seemed a bit excessive. think it would best be applied as a post-population scrub.


Great call. I never knew when to use one vs the other - no I know. Thanks man.


@64FF00 The no picture thing?


Unnecessary output. File is overwritten later on anyway.


How about [displaying pretty-printed json](de5dd3c), @dcarroll?  The output could be piped to `jq` if you want to do further filtering or formatting.


Ah, sorry, I always forget dirs need the x flag to list.


excellent point. thanks :)


Seems like the RUN label from the image would require the command to output:

`/usr/bin/docker run -t -i --rm  --cap_add=SYS_ADMIN --net=host -v LOGDIR:/var/log -v DATADIR:/var/lib --name TEST1 atomic-test-1  run.sh`

not `docker create...`


I have a feeling that some other nodes may use gpio c-api directly and will be missing "USE_GPIO". Have you done a specific test about this?


Its great to find other brazilian users using Pligg. tnks!


@64FF00 If it is, then its because I'm using a github client on android, **AND** because I already created 3 account on the same email, this was the last one. I deleted the rest because it was committing as TheDiamondYT. I really should change my email but who has the f**king time


technically we should probably use this here... might save lines. shouldn't break on format change or acf directory move or something. allows for filters and stuff.

http://www.advancedcustomfields.com/resources/get_fields/


Are you planning to host pre-compiled binaries somewhere else? Or do you suggest just using http://frightanic.com/nodemcu-custom-build/ ?


allow passing a router object directly


If you look at the page though, there is no jquery view-source:https://timeline3.knightlab.com/examples/houston/index.html


Hi borisguery. Nice solution. But to have the same behavior of contains,  I added a line code on line 158:
$parameter->setValue('%'.$parameter->getValue().'%');


This was done by the `futurize` tool, I don't think it's necessary (in this exact context).


Done. It is 0.2.1 now.
@garthk do you have interest in maintaining this plugin? It has a lot of flaws and I have no time to work on it.


cli-bug-fix (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1754 of 1761 tests passed)

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


Fixed in b9c200b062562a7e811ee22adb48d05d443f49ae


Hmm, should probably leave this default here, even though we're defaulting path in the CLI. Not a huge issue though.


And none of this is to say that I don't respect your solution. It's nifty.


Hi borisguery. Nice solution. But to have the same behavior of contains, I added a line code on line 158:
$parameter->setValue('%'.$parameter->getValue().'%');


6/6

Nailed it!


Why would we have an unknown timer in general? Shouldn't we only have a widget one and a global inactive timer?


cli-bug-fix (nealkruis) - 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)


@Alucard24
get this change, important!


@davidfowl  It should be theoretically possible to await these calls instead (though Shaq would be disappointed :smile:). The problem is that `ProduceStart` is called from several non-async functions which would have made this a bigger change.

Do you want me to open an issue to change these to awaits?


I think that way the parameters will be Strings and in the previous version they were Booleans. Could you please change that line to "audioMuted === "true", videoMuted === "true"" or something like that?


6/6

Excellent work!


> What is your rationale for changing this?

Simply to have the intersperse_view model BoundedRange when the adapted range is bounded. I suppose with some vague notion that Bounded is a useful property to preserve? I certainly have no data to back up that notion. I'll revisit this if/when I have measurements.


Try the --rebase (download-sources.sh should do it automatically), gives a cleaner history without merges :)


This will produce the wrong result if there is a different number of partitions on different localities.


@Alucard24
get this one too.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 189](https://teamcity.labkey.org/viewLog.html?buildId=262107&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 15 (1 new), passed: 2 Build time: 00:06:31
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

##### there are 5 more failed tests, see build details
```


Noted.


remember about camelCase.


So we can choose to disable tests that rely on R.  Not currently something we're doing, but it's good to have them all annotated for when surprising R problems come up.


Same goes for me. Having a full time summer job makes a difficult for me to contribute.


Terrific! Thanks for correcting this.


nice


Lets not add the short option here - I'm a bit hesitant on global options.


This bit is rough.


mais heu j'av pas fini ;-)


It looks like this import already existed further down. Can you move it back so that the diff disappears?


Только с точки зрения того, как визуально лучше смотрится для пользователя.


Yeah, I like that.  Is that now part of the pull request?


reminder to myself: Still unsure that we want the refresh timer to have a pointer to a widget. Can we guarantee that the widget / refresh driver will clean this up properly and in a deterministic order?


Hey. This doesn't work with my version of `paste`. I see:

    usage: paste [-s] [-d delimiters] file ...


Since RefreshTimers are now ref counted, we can pull the InnerVsyncObserver out to the WidgetVsyncRefreshDriverTimer.


This import should be moved down to just above `from opaque_keys.edx,.keys ...`.


Ok, it's probably fine as-is.


Yes, I added it to the branch with the open pull request.


why not using constructor?


Is there a reason that this is a tuple and not a list?

```python
if category in ['chapter', 'sequential', 'vertical']:
```


Looks good. Pretty simple solution.


ok


Yes, I think so.


How about "even if you've never DJed before"?


No :( VsyncObserver is itself refcounted, as is RefreshTimer.  So multiple inheritance gives me two different refcounted base classes, and everything blows up.  I tried to make it work; it wasn't to be.


Hmm then it should affect all devices in cm 12.1 but i didnt see any reports like our force close problems
İ think it should be dpi changes arco changed asset few months ago from hdpi too xxhdpi so we are now getting xxhdpi assets
i think we should change xxhdpi to hdpi


There shouldn't be empty lines between `if/elif/else` blocks.


Vinyl control should be mentioned too.


All official releases are available at https://github.com/nodemcu/nodemcu-firmware/releases. That is fine because it's not part of the Git repository. You might also be interested in a related discussion at #516.


@acfoltzer: Note that Trevor suggested a fix that eliminates the spurious dependencies when `cryptol-server` is not being built. So, I don't think there's any reason to split the package anymore.

See my earlier comment; maybe you could merge my pull request?


Why is this `xblock_studio_url(child,)` instead of `xblock_studio_url(child)` (note the extra comma)?


Awesome! I'll update to that version !


I can not find the setting useProject in the  in the xml file. I am probably looking at the wrong place...


@imuchnik Thank you!


Thanks for updating to dynamic links, I was gonna get to that.


I think you need the virtual keyword - C++ diamond problem! - http://www.programmerinterview.com/index.php/c-cplusplus/diamond-problem/


This looks like new behavior. Can you explain what's going on here?


Hi,

This patch introduces requirement for mbed-ls which is not listed in setup.py.

Also mbed-ls cannot be installed via pip:

```
W:\TheITF\sdk\sdk-github>pip install mbed-ls
Downloading/unpacking mbed-ls
  Could not find any downloads that satisfy the requirement mbed-ls
Cleaning up...
No distributions at all found for mbed-ls
Storing debug log for failure in C:\Users\Screamer\pip\pip.log

W:\TheITF\sdk\sdk-github>pip install mbed-lstools
Downloading/unpacking mbed-lstools
  Could not find any downloads that satisfy the requirement mbed-lstools
Cleaning up...
No distributions at all found for mbed-lstools
Storing debug log for failure in C:\Users\Screamer\pip\pip.log

W:\TheITF\sdk\sdk-github>pip install mbed_lstools
Downloading/unpacking mbed-lstools
  Could not find any downloads that satisfy the requirement mbed-lstools
Cleaning up...
No distributions at all found for mbed-lstools
Storing debug log for failure in C:\Users\Screamer\pip\pip.log
```

More importantly/urgently I'm not able to run the test suite at all:
```
>python workspace_tools\singletest.py -i ..\config\spec_efm32gg_stk3700.json -M ..\config\muts_default.json

Traceback (most recent call last):
  File "workspace_tools\singletest.py", line 66, in <module>
    from workspace_tools.test_api import SingleTestRunner
  File "W:\TheITF\sdk\sdk-github\workspace_tools\test_api.py", line 56, in <module>
    from workspace_tools.compliance.ioper_runner import get_available_oper_test_scopes
  File "W:\TheITF\sdk\sdk-github\workspace_tools\compliance\ioper_runner.py", line 23, in <module>
    import mbed_lstools
ImportError: No module named mbed_lstools
```


@imuchnik It looks like https://github.com/girldevelopit/gdi-core-git-github has been updated in the last 5 months, should I look through that one instead?


no comment


Thanks to you for the fix!


Why manually add ref versus make the mTimer a nsRefPtr?


super nice :) awesome


Did you mean to add this? Why?


There are several issues with that, but maybe they can be worked out.
- A dictionary is mutable. So if the dictionary gets changed afterwards, equal dicts could have different hashes. It might not be a problem here.
- Python 2 will sort anything and everything, by comparing their `id()`. So this order is not stable from one execution to the next. The problem is when you pickle Ops in a process, and unpickle them in another one, which happens all the time with the compilation cache. This could be solved if we restrict ourselves explicitly to some classes for keys.


since you're testing viewModel I think it would be good to place it in separate file


This looks to be a slight change in behavior; the code previously did not throw any errors.

Why was this change made? When does this case ever happen?

Either way, `NameError` isn't the correct exception to use here [1]:
`Raised when a local or global name is not found. This applies only to unqualified names. The associated value is an error message that includes the name that could not be found.`

[1] https://docs.python.org/2/library/exceptions.html#exceptions.NameError


this is by far the biggest thing that we'll need to verify with these
changes -- making sure that the lifecycle of everything is right and proper.

On Wed, Jul 22, 2015 at 2:47 PM, Mason Chang <notifications@github.com>
wrote:

> reminder to myself: Still unsure that we want the refresh timer to have a
> pointer to a widget. Can we guarantee that the widget / refresh driver will
> clean this up properly and in a deterministic order?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vvuk/gecko-dev/commit/dfec1b1dd44466dc002b72e3a307d4f188e7a21d#commitcomment-12305039>
> .
>


ahhhh

here you go :)


What these files do? o.O


very good


https://gist.github.com/Rushor/e6069c800b5b070b965c - See you on 6.x :)
![Bild](http://i.imgur.com/h3lVVX6.jpg) - since i cannot show here anything, just a picture of sen'jin xD


It's ImmaFreedom not Imma Freedom.


Oh, and I also moved omega base theme back to 4.2 until 4.4 gets tested.


hmmm this looks very very strange


This does not seem sufficient for UTF-8 encoded strings. I tried:

    git for-each-ref --format '%(align:20)|%(*authorname)|'

in git.git, and got a mis-aligned `Dévai Tamás` for `refs/tags/v1.6.2-rc1`


I think you do not need those two lines anymore.


Sweet! It works!


Ein paar Anmerkungen:
- Das Jumbotron heißt Jumbotron nicht Jumbotrone.
- Das header fragment und das jumbotron müssen eine input variable definieren.
- Das modal fragment muss aboutdialog oder ähnlich heißen
- Der VariablesController ist ein VariableSearchController und die Methode sollte IMHO showVariableSearch heißen und das template variableSearch.html

Können wir alles diskutieren :-)


approve


Thanks for picking up support again, made my day!


> some vague notion that Bounded is a useful property to preserve?

It is for Bidirectional ranges, and for the sake of easy compatibility with "legacy" code. Those should be weighed against the size and time win of not having needless state in sentinels. (I see we've located my earlier comments!)


looks good


Now I understand - and I'm thinking that there shouldn't be state in the cursors when the range is single-pass. The state should be in the range. Anyway, I'll go work this out for myself instead of making you painstakingly explain every detail of the design.


Isn't this already taken care of by nsCOMPtr<>?


Haha yeah. Dunno how it wasn't picked up sooner ;p


Not everyone uses nvm, might be worth checking if node is on the standard non-nvm path?


No way to remove the quotes?


Nit: Method names with "and" in them always smell to me. You could make this:

setOverlayState(state: OverlayState (?), animated: Bool)

but this is always animated so (unless you want to always use this to set the state), maybe you just drop the animated bit from the method name....


I updated the ability.
It should work with this.addAbility(new ChampionAbility(this, "")); now.


*sigh* no.


Do we ever access off main thread here? Can we put thread assertions wherever we access widget?


What is the point? Also since you remove randomness, one has to fix [TODO item](https://github.com/PurpleI2P/i2pd/blob/master/TunnelConfig.h#L104). There has to be 29 bytes of random stuff there and not zeros. Also doesn't it affect performance? There are numerous message allocations.


@alexmamonchik Because Stripe's gem is now up to `1.23`?


I was think that it's an extra instantiation of `fast_or` with a slew of different types, rather than a single instantiation of `or_c` with a bunch of `bool`s. But now I'm not sure it makes any difference, and I certainly haven't measured. Either way, it's a small thing.


nit: const int DEFAULT_DISPLAY_ID = -1; I dislike bare constants.


You could even move it into a setter for inOverlayMode, but that seems like it could also get a bit hairy... Even just naming it animateToOverlayState would make me cringe a bit less, so maybe its nothing.


ok


instead of `<button onclick = func1_onclick() class="btn_withJs"> <img src="images/run.png"></button>` you can simply do `<a href="app1_detail_page.html"> <img src="images/run.png"></a>` :)


Right but what is actually wrong here. It seems strange that we would get a bootable machine only to have storage choke less than 20% of the time and only on grub1.


Should be done now: https://github.com/rspec/rspec.tmbundle/commit/d3644929e12edd01d2410ed6aa209735b8227398


try not to use capital variable names. it won't break your code, but it'll confuse the hell out of other developers and possibly yourself


I would not say anything about this because it's like religion:

one people want to keep this in code to show explicitly in code,
others want to remove all comments when its are not super useful,
just choose your religion...


looks good, didn't know about that scope trick

Thanks


@albertopq you want this to be:

```js
if (!this.transitioning) {
```


Awesome, thanks.


the both ways are ok for me


Generate faster popcount/hamming weight? Several choices: http://bisqwit.iki.fi/source/misc/bitcounting/#64BitIntegersOnAmd64%2064BitPlatform

Nifty Parallel Counting would be something like:
````
        const long m1 = (~0L) / 3;   // Binary 01010101...
        const long m2 = (~0L) / 5;   // Binary 00110011...
        const long m4 = (~0L) / 17;  // Binary 00001111...
        const long h01 = (~0L) / 255; // Hex 0101...
        const int testBits = sizeof(long) * 8 - 8;
        protected override int GetCountFast()
        {
            var count = MaybeUnknown?.Count ?? 0;

            var bits = _bits;
            bits = (bits & m1) + ((bits >> 1) & m1);
            bits = (bits & m2) + ((bits >> 2) & m2);
            bits = (bits & m4) + ((bits >> 4) & m4);

            return count + (int)((bits * h01) >> testBits);
        }
````


I think I had `animateToOverlayState` at first, though it wasn't as clear that the `inOverlayMode` value would actually change. I guess I'll go back to that, including some brief docs.


:+1: w000000000000t


Nice. What are the current plans to support ES6/CommonJS/AMD Modules in app and package code, if any? I was almost done with [`rocket:module`](https://forums.meteor.com/t/4620) using the old `Plugin.registerSourceHandler` API (plus a bit of hacks). I'm refactoring `rocket:module` right now to use this new `Plugin.registerCompiler` API. It'd be helpful to know what's planned so I don't duplicate any efforts that might already be in place. I'll be at DevShop in 8 days to hopefully get an idea. :smiley:


Which alert?)
By the way, \<input/>, not \<input>\</input>.


a document in rethinkdb always has some kind of an ID - if nothing is set during the insert, it uses r.uuid()


primer commit de prueba


Thank you very much, my friend I got it :)


```javascript
  if (!this.shown) {
    window.dispatchEvent(new CustomEvent('utility-tray-abortopen'));
    if (!this.transitioning)
      return;
  }
```


`throw errors.X()` Make sure to create an error instance and not throw a function object


So widget itself, and its refcounting is not thread safe.  But the vsync observer stuff is designed to be thread safe; you can add/remove observers on any thread, and the callbacks to them can happen on any thread.


`fieldName instanceof String` is never a thing


Above your knowledge, no need to worry @aggelosQQ


remove log


Yeah me too.  This was going to not stick around for long, I don't know if I even want to have a const for default display ID.  That int is more "I want to pass a parameter to this, I don't know what kind yet"


schon mal die crashdetect source angeschaut? :smiley: (original ist die ausm PAWN Source-Paket)
Zeex hat da echt saubere Arbeit geleistet, ohne crashdetect müsst ich viel Zeug ausprobieren und mit rumexperimentieren :+1:


This is the tiniest gem version ever!  :gem: :microscope:


There's a bare AddRef in NotifyVsync that this matches.  It's to make sure it's kept alive if the RefreshDriverTimer happens to lose all other refs before the main-thread event runnable is processed.  This isn't ideal.


Downside is that this introduces the risk of programmer error by letting the views fall out of sync if ```refresh_view_funcs``` is not provided


This a card based project ....


Probably better [now](https://github.com/ehauckdo/fonttools/commit/418a82088de8f9908a607e53a7abae5bc46a058d).


ok thanks for the info. I'm still trying to understand the in's and out's of Docker.  -v wasn't working initially but now I got it working, originally I thought it was cause of not having the VOLUME in the dockfile.

Thanks!


Kirk, I saw what you did here.


cli-bug-fix (nealkruis) - i386-Windows-7-VisualStudio-12: OK (1753 of 1761 tests passed)

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


looks good


Np


Option to skip NAP and/or STM programming? NAP because mostly we'll be testing same nap version, stm because we'll be running same fw on different rf scenarios


There ya' go, Chris. LGTM


Strings aren't the only things in JavaScript that aren't objects and aren't numbers. There's also symbols, undefined, SIMD vectors, and some others. What is the motivation for separating out and extracting `value` into `string` and `object` rather than just having a single opaque type for any JS value (which JITs know how to deal with already)?


r+


cli-bug-fix (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (729 of 729 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-7.14%25-green.svg)


So this is fine, modulo polish.

I was actually suggesting something more implicit.

like
```
.testr.conf -> as it is today
testr.conf -> an overlay like

[DEFAULT]
instance_execute=coverage.py $CMD
```


Thanks, fixed in master.


Off-by-one with indenting.


@jtrmal: Do `git show e1f762 -- windows/generate_solution.pl` -- there is a stray CR at line 55. Perl sometimes gets very picky at these!


@srowen @mallman This seems to be causing the tests to fail for scala 2.11. Can you take a look?

https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test%20(Dashboard)/job/Spark-Master-Maven/AMPLAB_JENKINS_BUILD_PROFILE=scala2.11,label=centos/14/

The new Scala-2.11 Jenkins master builder started failing with this commit. If this cannot be solved by tomorrow, I am inclined to revert this patch.


yes - totally agree.  I didn't test before I merged.  Probably a terrible faux pas.  won't happen again.

and I will happily buy you a beer(s) the next time I see you.

Sara

> Date: Wed, 22 Jul 2015 11:55:46 -0700
> From: notifications@github.com
> To: case-studies@noreply.github.com
> CC: sarasp@outlook.com
> Subject: [case-studies] Fix all case studies (f4ef22e)
>
> Fix all case studies
>
> For the first time, we finally have a non-broken build.
>
> @SaraSp @joshholmes: I’d recommend that going forward, we only accept
> case studies that do not break the build and pass html-proofer.
> Html-proofer checks if all images and links are reachable.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/CatalystCode/case-studies/commit/f4ef22ee8f6dd97617e510834a7d5872a28a7848


Kk sounds right to me.


This is a duplicate declaration of `org.apache.maven:maven-plugin-api`, see line 1318 ([here](https://github.com/jboss-integration/jboss-integration-platform-bom/blob/99234b9ce004e9e0cf79d57116b82b4aa9891c33/ip-bom/pom.xml#L1318)).


I believe this should be 'farthest'


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/7746ec954de60e1c60de4ff2b7f61b7fff84c895


@sdodson potentially, I'm going to dig in further on this to verify this today.


merging graydon/stellar-core/no-listen-for-local-commands = 7746ec95 into auto


graydon/stellar-core/no-listen-for-local-commands = 7746ec95 merged ok, testing candidate = 12390a3c


Needs to be implemented still


> "Right mouse to nudge a greater distance."
Shouldn't that be removed?


Yeah, it felt good to come up with it, but you're totally right. I just made a commit with the more succinct solution.


<head>
</head>
<body>
</body>


I have a feeling it may be possible to add a parser for these meta-patterns that doesn't affect the rest of the system... But I haven't tried it yet, so I may be wrong.


"Essayé la mise à niveau des paquets:"


@glasser So `compileOneFile` only runs on a file that `isRoot` when using the multi-file compiler? But all the other files in a package, for example, still have to be added with `api.addFiles`? Or does this API automatically keep all files that match the given extensions? I'd like for people who use `rocket:module` to specify only entry points with `api.addFiles` (because there may be way too many files to list). I'm thinking that I can get all the files from the Package Server API. Is this true? Do all files get uploaded to the Package Server API?


Does that one work? I read this

"ACF Pro has a built in options page, and changes the get_fields('options') to get_fields('option')."


I'm merging it since Sourceforge is down so it's less broken than having it just fail, but I think the recipes that use IRSTLM need to be modified to check if it's installed and instruct users to call the script if not.  Don't know if you have time?


Oh, also iron+steel nuggets and localization for homing + wolfpack


I think we should have the soundtouch fork under the mpc-hc organization.

Although, since it's not official I don't really see the gain of the submodule. We could just use vanilla code.


Probably need to change
```
margin-right: 32px;
```
to
```
 margin: 0 32px 0 0;
```
This help me to avoid bad placement of the border-top on the table-pagination.


`House.first` returns a single house, so the correct code here is just `House.first.update!(motto: 'this is the first house')`


@olivergrant: Look, beer!


Cool


I got the idea from looking at the xblock_studio_url() method.  In the method, the "if" statement uses a tuple, rather than a list.  I'll change this line to a list, though!


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/72177226


A few things are wrong here:

 - The function definition is inside another function definition. Incidentally, this will create a top level, global function called OnStopBeingBuilt.
- The function definition isn't terminated by `end`
- The inner if isn't terminated by `end`
- Presumably you want to _call_ self.GetIsSubmerged, which you will want to use the function call syntax for.

I.e., this:

    self:GetIsSubmerged()

Instead of

    self:GetIsSubmerged == -1

Which compares the function itself to the value `-1`

The same with `IssueDive`, which you presumably also want to call.


You understand correctly. Nothing about the work we're doing for Meteor 1.2 changes the basic concept of the package.js format, even though we're aware that a better project/package control file would be better (see eg https://meteor.hackpad.com/New-control-file-brainstorming-ZDBMiEQ6DZl)


Hah, I was fried by 5:09am.


@finscn, I got your question already on issue #601. Don't worry, I'll get to it before the release. But please be a little more patient. Apart from the Mondays where I can focus on Tiled for a long time, my week is packed with work and family. Thanks!


I have to maintain a soundtouch repo for standalone sanear anyway.
mpc-hc can fork it, or don't fork it, or just copy soundtouch code.
This commit  just seems the easiest way of all.


This commit works.


OK I guess if you want to maintain it under your account.


This is really bizarre, netCDF4 should use numpy dtypes (or the like) not this, in my opinion.


Wouldn't it be better to also used the document selector, so newly added boxes will be collapsible without having to register them again?


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


Reading List App/Reading List App/AppDelegate.swift

This `if let ` and the one above can be cleaned up into 1 single statement:
```swift
if let path = NSBundle.mainBundle().pathForResource("Keys", ofType: "plist"),
    let keys = NSDictionary(contentsOfFile: path) {

//use keys dictionary here

}
```


Oh, and re: ES2015 modules: I believe that isn't the plan for 1.2 but we're hoping to move towards that as soon as possible (@benjamn is leading up the ES2015 push).  I'll be on vacation this DevShop but I'm sure other folks will be excited to see you!


How about `createSystemField`?

> Well, yeah, of course, but this docstring for interface functions thing is part of the revision of the whole server side Python interface, which is still on the todo list.

Adding dogstrings together with function will save a lot of time.


@eht16 not sure you know it: you can use any separator you want for the `s` command, not only `/`.  I.e. you could have `s,http://plugins.geany.org/,//plugins.geany.org/,g`.  Comes in handy when there are `/`es in the pattern :)


Uhhh what is that? You are comparing two same things? :sweat_smile:


When I looked at the definition of the method, it read "xblock_studio_url(xblock, parent_xblock=None)".  I thought it would be good form to include the comma for the empty second parameter.  If it's not necessary thought, I'll remove the comma for both xblock_studio_url(child,) and xblock_studio_url(parent,).


fast-forwarding master to auto = 12390a3c


Pull this out completely. This will be part of another ticket.


Are you sure this property defines source (twitter/instagram)?


Ok, I will change it for the other MATRIX_VALUES_ADD_*2 too and uncomment the commented section.


Are you using a version of Angular Material greater than v0.10.0? I am aware of the inaccurate styles for the `md-select` element in master branch of Angular Material. I am waiting for the next release to fix this.


Does not work


This is set up for session 5 where we have a completed, failing test scenario, a ThroughputCalculator class stubbed out with a method to calculate throughput, which returns a hard-coded value of 0.5. Ready to begin TDD to develop the ThroughputCalculator and supporting classes.


I think we should change "even just with" to "even just".


@hstove Do you need to use stripe `1.23` ? Stripe `1.21` breaks stripe-ruby-mock. We have different branch for newest version http://github.com/rebelidealist/stripe-ruby-mock/tree/stripe-1.21.0. This branch still contains some error for some features. I going to upgrade it to latest stripe version.


ActiveRecord's `.where` method returns an array of results even if there is only one match. You can use '.find_by' to return the first matching result, which would have made this code work


@johnnyiller actually you can now use v1.0.7


yeah, i'm with you.


I used it in FINITE_ELASTICITY_SURFACE_PRESSURE_RESIDUAL_EVALUATE and FINITE_ELASTICITY_SURFACE_PRESSURE_JACOBIAN_EVALUATE.


Aaa tak, tutaj moje przeoczenie. Integer jest OK.


This is set up for session one. We have one scenarios, with no matching step definitions.


This does not go well with your typechecker in extract-exp. It needs Tot:

$ fstar --codegen OCaml-experimental extTest1.fst extTest2.fst  extTest.fst
prims.fst(30,43-30,46) : Error
Identifier not found: [Tot]

Perhaps, you can add another special case here:
https://github.com/FStarLang/FStar/blob/master/src/backend/extracttyp.fs#L382


5/6

Great job. Just remember to use `.find_by` when you only want to return a singular match and you're all set


@glasser Does a multi-file compiler run for all files in a single package, or all matching files of all packages in an app?


Yes, for some reason I thought that the variables already had those checks.  It is fixed now.


Hi Jc,
This change caused a regression in SlicerRT. The problem was that NamesInitialised was not set on the color table and vtkMRMLColorTableNode::GetColorIndexByName returned -1, although the colors had names as I explicitly set them using the SetColor method that had name as argument. After finding this out it was easy to fix by enabling the NamesInitialised flag.
Not sure if this is something that needs attention, but maybe we should switch the NamesInitialised flag to true if a color name is explicitly set.
Thanks!


All files. That's how you can have cross-package CSS preprocessor imports.


You don't remove randomness, because the array is initialized with memory from your computer. Sending this data over the network is potentially disastrous (see also heart bleed).


There's some minor changes to FieldExport.c.


Comment


Test Comment.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 190](https://teamcity.labkey.org/viewLog.html?buildId=262122&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 6 (2 new), passed: 19 Build time: 00:36:54
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

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

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

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

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

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

```


Thanks for the translation!


The english text was:
`// Since Sass indented-syntax forces those coding standards
// There is no wrong way of proceeding`

Using "nope" in here isn't correct, but "não" is.


All parsing functions use this way of passing current error in an returning an error. This makes it possible to skip error checking here and make the error fall through. No reason in particular for doing like this, it just looks pretty clean in the main parsing function (see the story review issue parsing code for example).


 From my side I can say that the first pull request (#175) was without
any mistake. But in the second one I forgot to change that detail. I'm
also sorry and the rebase lesson is learnt. Enjoy the summer school!

On 22-07-2015 17:31, Lorenzo Natale wrote:
>
> ...sorry my built-in python compiler did not give any error :)
> seriously I checked the patch and it seemed ok but I cannot make sure
> that it works and we do not have automatic checks for this code...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robotology/icub-main/commit/a7a2e85f29e7f5cadb6795144e759e5b320f2535#commitcomment-12302267>.
>


wow!


@glasser Oh nice. Have a good vacay!

Hello @benjamn I'm curious to know what efforts are planned for ES2015/CommonJS/AMD module support. I was almost done making [`rocket:module`](https://forums.meteor.com/t/4620) with the current `Plugin.registerSourceHandler` API, but am now refactoring to use the `regisgerCompiler` API. The idea is to handle files that are `isRoot` as entrypoints in Webpack, and use Webpack's code splitting across all files (I already have that working in my local version here, but it'll be much cleaner after moving to the new registerCompiler API`).


shouldn't we be making a reaction here, not a species?
eg. ```
reaction, created = Reaction.objects.get_or_create(rPrimeID=rPrimeID)
```


@alexkwolfe this change adds date string parsing support to the range class, reducing the complexity of changes needed in `leadconduit-rules`.

Do you prefer this approach over those in the existing PRs?


@glasser

> All files.

Excellent! :clap:


Line 158 of signaler.cpp needs a `}`. This commit removed the one that was there for the WINDOWS section.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 191](https://teamcity.labkey.org/viewLog.html?buildId=262127&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 16, passed: 3 Build time: 00:06:54
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

##### there are 6 more failed tests, see build details
```


@mbernat The whole parsing works in a way that every parsing function is passed the current error and the scanner that holds the scanning context. In case the error is set, the parsing function returns. This makes it possible to skip error checking in the pain parsing function, the error falls through the parsing functions. The scanner keeps giving the lines to the parsing functions, and in case there is an error, it knows what line number it is and stuff.

There are certainly some questionable things, but I've spent quite some time getting this right, so I am posting this now, I think that it is ok.


Ha, now I remember. Yeah, basically I know but keep forgetting about because I'm so used to use just a slash :). Fixed.


These changes are for #10


Hmm, weird. worked for me. Not sure whats going on -- these eol issues
drive me crazy.
I fixed the perl script so that on linux it has native eols. Could you
please check out
https://github.com/jtrmal/kaldi/commit/9d8a10abbf1e06903cf98604b572893c9f6b7a15
to see if it will work for you?

y.

On Wed, Jul 22, 2015 at 3:18 PM, Kirill Katsnelson <notifications@github.com
> wrote:

> @jtrmal <https://github.com/jtrmal>: Do git show e1f762 -- windows/
> generate_solution.pl -- there is a stray CR at line 55. Perl sometimes
> gets very picky at these!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kaldi-asr/kaldi/commit/e1f76237e019859f5560ec93b8ef2b1f158f4ff1#commitcomment-12305645>
> .
>


This is actually not that useful. Removing.


The function call to ```is_valid_output_file``` results in a compilation error, as it cannot be found (at least on my system; it's eminently possible that I missed something in this regard).

I'm not certain what was intended here, since I can't find any definitions for this function in any MethPipe-linked namespace. Perhaps this function is still being implemented or perhaps some form of file suffix check was intended, via [```is_valid_filename```](https://github.com/smithlabcode/smithlab_cpp/blob/master/smithlab_os.cpp#L95)?


The variable _box in this case is the document unless it was called explicitly with a box component. So we are using the document as the main selector :+1:


I will look in how much time it would need.
y.

On Wed, Jul 22, 2015 at 3:27 PM, Daniel Povey <notifications@github.com>
wrote:

> I'm merging it since Sourceforge is down so it's less broken than having
> it just fail, but I think the recipes that use IRSTLM need to be modified
> to check if it's installed and instruct users to call the script if not.
> Don't know if you have time?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jtrmal/kaldi/commit/eb11b1778fc2e4354ccdbfc8c036a39b5584be35#commitcomment-12305824>
> .
>


Pagination enhancement #4


@aledbf is there any particular reason for putting the entire text of the flanneld unit in the user-data?

This looks pretty close to what's already included with CoreOS out-of-the-box and the documentation (link below) suggest the little configuration that's needed should be accomplished using a drop-in.

https://coreos.com/flannel/docs/latest/flannel-config.html


See line 329 runs the ThermoImporter after the SpeciesImporter?
We need here to run the KineticsImporter after the ReactionsImporter (or before it, if you want to test it without waiting for all the Reactions to load)


I don't know. I'm seeing content right now. Are you seeing content in the sensor spec I listed above?


By default, ember creates 4 routes:
- ApplicationRoute
- IndexRoute
- LoadingRoute
- ErrorRoute


You are the best! thanks so much!


When ember starts, it will render the application template as the main template. Inside `{{outlet}}`, it will render the template associated with the Route we are visiting. Then, inside those templates, we can have more `{{outlet}}` to keep rendering content.


Yes, I think anything that does not need to be trusted may live in `CttUtil`.


can you describe the real issue? Is the android sdk also have an utils/log.h? Or where does the wrong include come from?


@aledbf is there a compelling reason to include the content of the flanneld unit in the user-data?

This looks pretty close to what comes with CoreOS out-of-the-box and the documentation (link below) suggests that the little config/customization needed can be accomplished using a drop-in.

https://coreos.com/flannel/docs/latest/flannel-config.html


Thanks for the pointer @marcelstoer !


`ember g route friends/index` generated these files.
The part in charge of loading the data related to a route is called the model hook
The **route** is responsible for everything related to setting up the application state.


before was:
class CGHostDBMySQL : public CGHostDB
{
private:
    string m_Server;
    string m_Database;
    string m_User;
    string m_Password;
    uint16_t m_Port;
    uint32_t m_BotID;
    queue<void *> m_IdleConnections;
    uint32_t m_NumConnections;
    uint32_t m_OutstandingCallables;
    vector<string> m_Name;

public:
    CGHostDBMySQL( CConfig *CFG );
    virtual ~CGHostDBMySQL( );

    virtual string GetStatus( );

    virtual void RecoverCallable( CBaseCallable *callable );

    // threaded database functions


It may be some Safari issue: I get content from your spec, but not from [the one I referenced](http://opencreds.org/specs/source/identity-credentials/). However, if I open it in Firefox, I do see the content. Some library's not handling this properly, it seems.


@mk01
I know you are a fan of BFQ and BFS - and of course BTRFS.
Do you know this page: https://github.com/hhoffstaette/kernel-patches.
Using this patches it should be easy to re-add BFQ and BFS to the 4.1 kernel sources


Are you sure this really overwrite declaration above?


Ok it will not allocate memory each time this function is called, but this table will stay on stack memory forever. As this function is called once per UART I think is better keep this way, moving it out of the function would case the same effect.


Not necessary new lines.


Raul,

Algumas observações:

1) Você não está usando o template do projeto. O seu commit deve ser feito utilizando o template do git. Já passei isso em um email para o Gabriel.

2) Você retirou o log do Widget da Matriz de Indicadores. Por que?

3) Na UserServiceImpl.java, a implementação está correta. Pode dar revert nesta classe. O [CRIPT] que está na chave de criptografia no web.xml deve ser substituído pela chave de criptografia real. Vou passar para vocês. Vamos pensar em uma maneira de fazer isso em modo de desenvolvimento para futuras contribuições.

Abraços,


Was there a 0.1.0?


I'm not sure to be honest. Cory had those in there and I didn't try to see the repercussions of removing it. Maybe treating fractions as a single string for the `split` call is unreliable. I dunno... wanna test?


Tuve que limpiar la caché de git en local porque me aparecían los archivos y carpetas que había ignorado en .gitignore.

La solución la explican aquí: http://www.iteramos.com/pregunta/7813/gitignore-no-funciona


Why not something like this?
```diff
-  private static final GppLinker machJNILinker = new GppLinker(GPP_COMMAND, objFiles, discardFiles, "lib", ".jnilib",
+  private static final GppLinker machJNILinker = new GppLinker(GPP_COMMAND, objFiles, discardFiles, "lib", libExt(),
       false, null);
+
+  private static String libExt() {
+    return isAppleJava() ? ".jnilib" : ".dylib";
+  }
+  private static boolean isAppleJava() {
+    return System.getProperty("java.vendor").equals("Apple Inc.");
+  }
```


Yes, I've double-checked that and it's actually how it works already for other settings.


@krancour this can be replaced once deis uses a CoreOS version that provides flannel (653.0.0)


I can sync up this repo. Or at the very least diff the changes to see if it is something significant.


You cover when less than 60% or greater than 60%, but not equal to 60%. One of these should be less/greater than or equal to shouldn't it?


Yeah, I mean it fails without it currently.. I might take a stab at re-writing, since the team here is used to Jeet and no quotes..


Fixed.


`store.findRecord` and `store.queryRecord` are also valid to load in data by a given id or query parameters.
i.e.
`this.store.findRecord('friend', 1)` => /api/friends/1
`this.store.queryRecord('friend', {active: true}) => /api/friends?active=true


Are you sure about setting this to `Debug` for release builds?


Desde Terminal hice: $ git rm creditos '*~'


s/Audit/审计功能（Audit）/


s/降低/减少/


Have you update the code? I'm using this code and it appear to have the billing information filled, and shipping address shown
![billing address, shipping details, item details on sandbox](https://cloud.githubusercontent.com/assets/9349240/8816353/fc22ed50-304f-11e5-8281-bc5e583938bb.png)


The initialization doesn't execute because you forget to execute the function. You need to add a `()` at the end of this line.


s/paper/论文/


遗留的问题


I realize that much of this is Walter's code, not originally yours, but I'd be very skeptical of a constructor that exposes this many public properties. Can this be redesigned for better encapsulation?


misspelled canola!


问题不大，稍微修复后即可发新的 Pull Request。切记保持标点符号一致，另外，改完以后务必要通读，检查包括格式，读起来是否顺畅等。


Do you need to be doing this sort of substring comparison, or is there a more structured way that this data could be stored?


抬头没有遵守最新 develop 分支中的 zh/doc/README.md 约定，所有抬头咱们统一下。


Causes BC problems


aaa


А зачем файлы проекта Visual Studio в ветке cmake? Я подразумеваю, что они должны генерироваться автоматически.


hpwh_residential (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1756 of 1763 tests passed)

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


Thanks for this change.

A minor thing: The comment should be deleted.


Thanks, got you!


I'm using my own wrapper for memory alloc/free functions for ease of maintenance. Now whatever changes I make in my own repo is easier to integrate into john. However, the functions used inside the wrapper are john memdbg aware. So, I believe memdbg shouldn't be affected or would it? Other issues are fixed 9e18e45cd257f5bc3.


Related to https://github.com/OctopusDeploy/Issues/issues/1225


I think this may be another place where Walter stuck you with a bad design. There's no way that this should be a `switch` statement; instead, each block of the `switch` should either be a method on its own (and called by name) or resolved with object polymorphism. And even if that weren't so, the `switch` should probably be its own method.

I'm trying to focus my review on your original code, but it looks like the Sugar Labs code you started with was so bad that I had to comment.


Android x86 是移植到标准 **pc** 的一套软件（译者注：项目首期目标是想在 Eee PC 上跑完整的 Android，然后是让 Andriod 在所有 X86 平台上可以跑起来）。


hpwh_residential (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I don't think this is necessary. A function returning a generator will return a new instance on each invocation: http://3v4l.org/gjnOr


所有正文中的命令咱们可以用 \`\` 括起来，效果如下：

`sudo apt-get install gcc-4.4 g++-4.4 g++-4.4-multilib gcc-4.4-multilib`


This is another place where you could probably do `window.savedMatricesNotes || window.savedMatricesNotes.push(4)` or something.


I swear I pushed this before this... Anyway, apparently no fixes, just changing backlinks as well as category members.


This hasn't produced a race so far, we have used this pattern in other tests as well. The only place where it can throw InvalidOperationException, is when we have the process handle open and we wait for 0ms to see if the process exits at the time of Kill.


These variable names are quite unclear, especially the difference between `flag` and `flag1`. Can you come up with clearer names?


please remove this redundant dependency..


s/s. 也可看/也可看/ ??


Notice that you're repeating the `args[0][i] != '/'` both here and in the next conditional. Can you consolidate those to avoid the repetition.

Also, `!=` should almost always be replaced by `!==` (just like it's better to use `===`, not `==`).


While using GCC-specific cache control function, `__clear_cache`, it would be nice to have explicit declarations.


s/媒体/媒介/?


Yeah!:D


Can you do this without magic numbers? The 4500 represents something, and it would be clearer if you put it into a variable with a meaningful name.


在elinux/zh/doc/README.md中，文件头加入的内容，lzufalcon是否写错？
原文：eLinux.org
翻译：@lzulfacon
校订：@lzulfacon

是lzufalcon，而不是lzulfacon？


1. s/android/Android/，所有术语咱们都统一大小首字母，包括 linux -> Linux??
2. 关于 提示符，建议保留英文原文，因为有些术语软件安装工具可能并没有对应的中文版本。


if number is null, a NPE (NullPointerException) might occur.


s/^i/通过/g

通过触摸...


This is probably another good spot for `forEach` or something similar.


有些机器型号则是？

注：这里可能是指该机器的不同机型？


This is getting way too long. This whole block should probably extracted into *at least* one new method, probably several.


This conditional statement could probably be a method on its own.


You're shadowing the outer `temp` here. That seems confusing; is the name collision deliberate?

Also, `temp` isn't usually a great name for a variable. What purpose does it serve? Can you give it a more meaningful name?


I cannot run your application, it seems there's a problem with your manifest file. I did not give points for black  box criteria but will be checking your codes instead..


I guess you guys can handle that internally? It's hosted in the S3 bucket so i cannot make any changes.


^i 通过 cd 命令，加入一些必要的衔接词汇让整个内容读起来通畅，在翻译完以后建议通读全文，确保各个段落读起来没有歧义，而且很顺畅。


@willneo Review 完了一篇，其他的稍后继续 Review。


You may want to use `classpath "net.foragerr.jmeter:jmeter-gradle-plugin:latest.release"` if you do not want to keep updating it for every minor release.


为了更快地有节奏地合并翻译内容，建议在 gitbook 创建单独分支（记得可以在编辑器创建分支的，见右上角）来逐个翻译文件，然后再提交 Pull Request。


there's a better way of providing validation confirmation for your presenter.. this kind of validation is not acceptable..


是的，是 lzufalcon


My bad.


It does return the translated sim name, not sure what's you mean here?


the whole if block can be delegated to your presenter..


Yes, I know we may leave a backdoor there, that's why we constrain the secure register could be set in set_secure_reg(). In fact, I name the SiP call as HDMI_COLOR_EN before, but change it to the current one in the last. The reason is if somehow we need to allow the normal world to set another secure register via SiP call, we only need to update the code here. User in kernel could just use the same SiP function call for it. Looks like this is a bad idea because the function name easily confuse people. I will rename the function, and just do one thing within it in next patch.


please provide appropriate method execution steps on how to navigateToResult..


Para saber si el número es menor a 0.5


when clear button is clicked, the error msgs are not gone.


Actually, in this case you probably don't want to use SpinUntil but rather just use your own loop.  Normally SpinWait is helpful because it allows you to spin while inserting appropriate yields and sleeps and the like, but you don't actually want that because you actually want to burn CPU as part of your test.


Para saber si el número es mayor que 0.5


How didn't you notice that?


Well, you're just setting it above anyways. I'm not sure it matters if it is enumerable or not, it's still part f the implicit api.


Is this going to be sufficient to never have random blips that cause this to fail?


That's great for helping the cocoapods build. This will do a lot help for iOS developers.


Only if it works.  If it doesn't work, it was written by someone else. :)


This one was confusing... Lesson learned do not name files with colons in the name when working on a mac expecting it to work on windows too.


Oops: I had libwebp installed. I downloaded it a time ago from http://windows.php.net//downloads/snaps/ostc/core_deps_7/vc14_rc/x86/libwebp-0.4.3-vc14-x86.zip
So, I know now why php_gd2.dll reports WebP support, I know the version, but I still wonder why new libvpx builds were put at http://windows.php.net/downloads/php-sdk/deps/vc14/x64/


error msgs are not cleared..


For some reason, in [commit 10377](https://projects.coin-or.org/Coopr/changeset?old=10376&old_path=pyomo%2Ftrunk%2Fpyomo%2Fcore%2Fbase%2Fexpression.py&new=10377&new_path=pyomo%2Ftrunk%2Fpyomo%2Fcore%2Fbase%2Fexpression.py), the Pyomo team began enforcing a requirement for Expressions that the "initialize" argument can only be used to supply numeric values, and "rule" must be used for functors.


>insane/psychopathic/murderous

![insane/psychopathic/murderous](https://avatars1.githubusercontent.com/u/90293)


please complete method implementation..


checking whether data inputs are empty or not should be done on your presenter and not on the view..


there's also a redundancy of codes when performing arithmetic operations.. can you find a way to provide only one method to handle add, minus, times, divide operations?..


the only purpose of your showErrorXxx methods must be to display the error msgs.. checking if data inputs are empty or not should be delegated to your presenter..


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


Please update `podspec`file


this constructor is useless.. it does nothing.. :(


@lzz5235 刚修复了，误导了，你之前提交的那个也写错了，回头修复下吧。


cheers..


Yeah, that smells bad, a lot. My bad :hankey: :hankey: :hankey: :hankey:
It was introduced by me long time ago.

Sounds good @viniciusdacal, let's see what @UltCombo says.


Spellcheck, "changed" or "changed". nona


:goberserk:


Subject 不能直接叫 “This file has been reviewed by @lzufalcon”，这个都是要存档的，可以叫：

zh: Translate xxx.md Vn

n=1,2,3,4....


Sounds like an excellent plan. :+1:


It wasn't intention, I was experimenting with name and forgot to add the rest. I re-submitted my code. Just curious, is it easy for you to find my new commit or is it helpful when I include the link?


return 1, 2, or 3 could be mapped to a constant variable in such a way it would be very easy to identify the various error handling mechanisms..


setting of click listeners here serve no purpose as you are already binding them to a specific method via ButterKnife..


I'm curious, why the move to `junit.framework.TestCase`?


the whole if block can be delegated to your presenter..


If you put the build dependencies under the check for the flag that enables the server, you should see that they're only required if the flag is present.  I've encountered this problem a lot working with HaLVM :)


these links don't work at http://davedash.com/writing/ - should those be variables that jekyll should be filling in?


Might be worth quoting the assignment of this and `PROJECT_ROOT` so it doesn't break if there are spaces in the path?


Theoritically it shouldn't fail, the only reason why it was failing before was because of the same DateTime.Now value within that 10 ms. On windows too the resolution is 10 ms. Have to see in future CI builds, if it still fails, then we should be able to find a new time interval.


Profile and post are different entities. So shouldn't be nested


good point @fractalcat - done.


should not posts comes inside profile?


@flavour Its from the spec that Nuwan send


@flavour project.xsl is at the top?


Would be nice to put a comment in here about why the workaround is needed.


Mr. Q 起好早！


OK, I can see it now. Yes, seems fine but it is only changed for one of the add methods? It should go into the other methods. Also, we should make the transition parameter from bisection to linear search a parameter.


This should show what the default is. In this case: `=true`.


Web Application and Exception Handling/Assignment15/Login.aspx

Rename the button ID as per the need.


把 bbcode 视为 HTML 的 tag 标签，markdown 里是支持 HTML 的吧


Web Application and Exception Handling/Assignment15/Optimus.master

The page names should be in Pascal Case.


你告诉我 `[img src](xx)` 应该怎么解析吧


s/Slides/幻灯


Web Application and Exception Handling/Assignment15/techNews.aspx

Rename the label tag according to the requirement.


s/Video/录像/


s/Category/分类


Hmm, SpinWait is a bit confusing, i thought this will put the CPU in intensive cycles, to perform the inner loop tasks, and to keep spinning instead of preempting to another thread while polling for the condition. Checked the times in a console app with normal loop and spinwait, not much difference in times. Will make this a normal loop.


s/Development Tools/开发工具/


Sorry, I wasn't very clear - I meant the assignments `PROJECT_ROOT=$(git rev-parse --show-toplevel)` and `MODULE_LIST=$(cd ...` will fail if there are spaces in the path (e.g., `PROJECT_ROOT=/home/foonly/foo bar` tries to run `bar` with `PROJECT_ROOT=/home/foonly/foo` in its environment). The `cd` would fail too, I forgot about that one.


Sorry，我在 README.md 里头写错了，应该是 lzufalcon

s/lzulfacon/lzufalcon/


굿


s/嵌入式开发者一定会是很感兴趣的/嵌入式开发者一定会很感兴趣的/


@Lotte-Bai 基本Ok了，看下是否需要再发一个 PR，马上合入。Thanks。


这是 markdown 的语法吧，() 里的才是链接地址，用正则？


@fractalcat not sure I follow - if PROJECT_ROOT has a space in it - only needs to be quoted when I cd right? (MODULE_LIST itself is whitespace separated as it is a list of modules, if the modules have paths in it... nothing will work).


Yes, we always want the PDB. It used to be `true`/`false` value but VS15 is using `Debug`/`DebugFastLink`/`false`.


Stale cache issue?


What will be the default behavior, if you don't set file type to neither binary, nor ASCII?


Oh, OK, I got confused by the explanation on msdn. We already set opt ref etc.


Desde Terminal hice: $ git rm master '*~'


Why this particular point type?


Few options:

* we could have to check that all color names have been initialized each time `SetColor` is called. I guess we could keep track of the number of colors that have been initialized and lazily switch `NamesInitialised` to `true` if the number of initialized color names matches the number of entries in the table ... that seems complicated and error prone.

* if you explicitly initialize a color table setting all its values and names,  calling  `myColorTable->NamesInitialisedOn()` afterward is the way to go.

* An other solution would be to remove (or change to a no-op) the API  `NamesInitialised/SetNamesFromColors()` and instead return a generic _color name_ of the form `"R=...G=...B=..."` if none has been set and a valid color value has been set.  We would also have to defined what we mean by _a valid color value_.


- `ember g route friends/index` generated these files.
- The part in charge of loading the data related to a route is called the **model hook**.
- The **route** is responsible for everything related to setting up the application state.


`store.findRecord` and `store.queryRecord` are also valid to load in data by a given id or query parameters.
i.e.
`this.store.findRecord('friend', 1)` => /api/friends/1
`this.store.queryRecord('friend', {active: true})` => /api/friends?active=true


Because our model is a collection, we can just reference it in the template as `model.length`.


did you clone with --recursive? `smithlab_cpp` is included as a submodule; therefore you need git clone --recursive


No problem at all guys. It was a minor glitch that was easy to spot out :)


does this work? it looks commented out.


I wonder if it isn't a bit risky to put non-absolute file names in tokens, given that some code (at least in Dafny) seems to assume that `token.filename` is absolute


remove random line break


remove commented out code if you don't want to use


you can combine `margin-left` and `margin-right` by doing:

`margin: 0em 0.2em`


@glasser I need to run a single compile process for all files that are `isRoot`, but that doesn't seem possible since the API is expecting to run `compileOneFile` individually on each root file. I want to give Webpack a list of "entry point" files that will be handled all at once (root files), and those entry points can "import" other files (the non-root files).

Webpack relies on the filesystem, so what I'm currently doing is telling Webpack the location of each entry point file, and an output location. Webpack then compiles all entrypoint files to new files wrapped in a Webpack closure, and places them in the specified output directory, and additionally Webpack writes one extra file containing the shared modules of all the entrypoint files.

So in essence what I need to do is read sources from Webpack's output files, and pass each source into `inputFile.addJavascript` of the corresponding `inputFile`. It's not possible to compile the entry points individually.

Can I use `allFiles` for this? Perhaps `allFiles.get(...).addJavaScript()`?


end coding!


I'll need to look into this @cviner because I'm now having problems. I thought I had checked that the function existed before committing methcounts.cpp, but I'm having problems locating it at the moment.


i think this should be fine since the import is no longer in setup.py.


start css


I might be losing my mind but these extra parentheses have some deep meaning I can't figure out. :fearful:


Would it be better to also just facade the `type` property of input so that the type can be specified at declaration?


Met le comparator dans une autre classe !


As in the moonstone comment, would rather see this at declaration than at create time.


use this form?


This broke #15209. Was there a reason s3 support was killed for hashes? Should I submit another PR to fix this?


make a `@mixin` out of the font-smoothing; you'll probably use it again later


@SammysHP please file a bug against the kernel on https://bugzilla.kernel.org/ explaining the issue, and referencing this github issue.

Also, if you see issues with our code, please file an issue, don't just comment on some random commit. Commenting on commit sshould be left when there's already a PR. Thanks.


Hi! Does this argument provoke `SpaceNotSupported` error? Or there are some other reasons to remove it?


There should be a space between the if and the brace.
Also, it'd be nice to check whether *p is a hyphen or a digit and throw an error (and keep parsing ?)


Awesome, thanks.


blabla


I just made a simple fiddle using latest version with 2 canvases — http://jsfiddle.net/jthe82xs/1/

I can see the difference very well. Can you?


@mengzhou: Yes, I did. All other links to that sub-module were successful.


this should be `margin: 20px auto 0px auto`


I just tested removing the unquote bit and it doesn't work... don't have time to investigate at the moment, but if you find a solution I'd be all for implementing it.


awesome use of mixins :+1:


My bad. I'll take care of this within the next 2 hours. For now you can just delete those lines.


This just reminded me of [openssl & valgrind](https://blogs.fsfe.org/tonnerre/archives/24)


this can just be `padding-bottom: 5px` for simplicity


Let me see first... Tomorrow i will make a clean compilation including your changes for flashlight if the apps are still crashing then we've got a stupid problem here cause RR is working fine for both devices


what you need to create a MAC OSX release?


Hmmmm.... It might be possible to compile all entry points at once for ever `isRoot` file, resulting in an entry point with shared modules removed, and I can place that into `inputFile.addJavascript` of a `compileOneFile` call. That'd work. The downside would be that I'd be compiling all the files once per `compileOneFile` call.


@aledbf I see in `flanneld.service` in 647.2.0

```
core@deis-01 ~ $ cat /etc/os-release | grep VERSION=
VERSION=647.2.0
core@deis-01 ~ $ systemctl status flanneld
● flanneld.service - Network fabric for containers
   Loaded: loaded (/usr/lib64/systemd/system/flanneld.service; static; vendor preset: disabled)
   Active: inactive (dead)
     Docs: https://github.com/coreos/flannel
```


@andrewdavidsmith: Thanks! I just used revision (3655360) instead. Glad to hear that it wasn't an idiosyncrasy of my system.


cli-bug-fix (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1246 of 1246 tests passed)

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


Right.
"Right mouse to nudge a greater distance." has been replaced with "{0} to nudge a greater distance."

Please, just translate as is, but without the "Right mouse to nudge a greater distance." part. It will be corrected later.


:+1: looking good


does this need to be a separate command?


might want to store colors like this in a scss variable if you're going to keep using it <= like we do with `$primary`


`margin: 0;`


you should put all the pages css in `pages.scss`


feat (roster): bruce banner detected


Andy fixed a memoize bug in config caching for you, I just pushed it. Sync with head, put it back in and check it works.


If I do it that way that I just mentioned, I might be able to take advantage of Webpack's cache... Yep, I think that'll work: http://webpack.github.io/docs/configuration.html#cache

The first call to `compileOneFile` will handle the actual Webpack compilation, and all subsequent calls will simply get results from Webpack's cache.


Adding spaces between parts of the above line would make it look nicer. :)


`#this-week`

for readability


@glasser Any thoughts/suggestions?


the `margin-top` is overriding the `margin` right above. you should just incorporate `margin-top` into `margin`


The issue you linked to above can't be compared to this one, for multiple reasons.
- The random data is not essential in this case. It doesn't break things even more than it does.
- The data is being send over the network.
I'd like to remind you of that other OpenSSL bug, heartbleed.


1) Eu sei que foi errado, eu rodei o comando GIT na pasta do projeto e
mesmo assim não funcionou, aconteceu o mesmo que aconteceu com o Gabriel no
início, por isso pedi ao Gabriel no whatsapp algumas dicas de como ele fez
pra resolver. Parece que a maneira mais simples é instalando um plugin do
jira no Eclipse. Estou fazendo um passo a passo pra poder passar pro resto
do pessoal.
2) Esse commit eu peguei do Gabriel, MOSC-11 ele veio quando eu dei
sincronize no respositorio e na hora do commit ele foi, mas não houve
mudança de código da minha parte.
3) OK, mas se eu reverter antes de vc passar pra gente a aplicação vai
lançar erro, pois ele tenta dar um ParseInt em uma string que não tem o
formato numérico.

OBS: Por isso eu sugeri que de inicio seria interessante que você desse o
merge final, pois vc leria os pull requests dados pela equipe de
desenvolvimento e veria se a issue está coerente, e poderia fazer perguntas
em cima dos pulls dados e se estivesse ok, ai poderia ser autorizado o
merge. Tem informações que só vc pode passar pra equipe, como esta do
[CRIPT] por exemplo.

Abraço.

Em 22 de julho de 2015 16:56, Victor Azevedo <notifications@github.com>
escreveu:

> Raul,
>
> Algumas observações:
>
> 1) Você não está usando o template do projeto. O seu commit deve ser feito
> utilizando o template do git. Já passei isso em um email para o Gabriel.
>
> 2) Você retirou o log do Widget da Matriz de Indicadores. Por que?
>
> 3) Na UserServiceImpl.java, a implementação está correta. Pode dar revert
> nesta classe. O [CRIPT] que está na chave de criptografia no web.xml deve
> ser substituído pela chave de criptografia real. Vou passar para vocês.
> Vamos pensar em uma maneira de fazer isso em modo de desenvolvimento para
> futuras contribuições.
>
> Abraços,
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Plataformas-Cidadania/mapaosc/commit/c27bf0031e6e51d8d68fe2a5acf12eaee99465bd#commitcomment-12306306>
> .
>

--
Mestrando em Engenharia de Sistemas e Computação UFRJ
Bacharel em Ciência da Computação UFRRJ
Linkedin <http://www.linkedin.com/in/raulsenaferreira>
Portifolio <http://www.raulferreira.com.br>


I wasn't aware of `-L`; just read up on it. Makes sense.

Though, for scripts I tend to prefer the longer parameter name, if one is available; `--location` in this case.


i think you can take out importing `bootstrap-sprockets` unless you're planning on using it later.


@jhoffmann same thing here
cc: @enginespot


instead of putting variables at the top of every page like this, just import them from `universal` like so:

`@import "universal.sccss";`


same thing for mixins. just import `universal.scss`


i'll show you how to use helpers later. for now, you should remove all empty helpers like this.


Even if non-cheri crti.o and friends work it will be by accident (and probably will only work if argv isn't accessed), everything else won't and will be impossible to debug with this change.


When is this persisted to the database?


The overhead in terms of space that would be added by db + table would not be much. I would consider 100 partitions in a query to be typical case. With probablilty of false positive at 0.001, that comes to 180 Bytes for the bits needed. Assuming around 20 bytes for db + table, it comes to around 10% overhead. http://hur.st/bloomfilter?n=100&p=0.001


I'm not really sure this is a good solution. Please explain it to me in detail.


Hi @david-mitchell I have also started playing with this, see https://github.com/sperling/coreclr/tree/profiling-unix
and https://github.com/sperling/profiler-test Let me know if I can help you with my limitied skill in this area. I still have a few days left on my vacation :smile:


Also, do you have thoughts on using curl instead of wget? For the splunk installation, further down, I had used wget. We should probably be consistent between the two. (And ideally with how other dockerfiles we write do downloads)


This and below don't seem to work anymore


I actually don't know which egs do use the irstlm -- I did found some which use the kaldi_lm, but didn't find any irstlm references.


I've forgotten to delete an alert on 45 line, which led to several unexpected editings and commits) That was so annoying.
Didn't know about input, but since it's not xhtml no one will complain :D
Anyway, what did you ask in the first comment about?)


The attributes will always be different. At least when I tried writing something like this, it was what I noticed. Try adding some debug `print("emitting widget::updated in set_markup")`.

The fix in my branch was to also save the test with markup in the textbox widget and compare it before parsing the markup.


I'm using curl in my Dockerfiles. Not a strong preference though. I think curl is a bit more powerful and has one or two handy options that wget is missing, but nothing major.


add upgrade script ossn 2.x to 3.0 #421


It's sort of like ceil, except it rounds to even numbers. So if the input was 2, it would stay 2, but if the input was 3, it would round to 4


Several clients #13


How about `if dpi_per_screen[s] then return dpi_per_screen[s] end` (with linebreaks)?


I couldn't think of a better name for the function...


Shall the first argument by sockaddr_in**6**? Also *result* is undeclared in a couple of places. I can prepare a patch if you have no access to Windows.


oki doki no pb

> Le 22 juil. 2015 à 22:30, KaiLangen <notifications@github.com> a écrit :
>
> It's sort of like ceil, except it rounds to even numbers. So if the input was 2, it would stay 2, but if the input was 3, it would round to 4
>
> —
> Reply to this email directly or view it on GitHub.
>


I removed it and opened a Pull Request @fyelles #30


Please go to https://github.com/MultiMC/MultiMC5/issues and open an issue for this.


Great question; "Whos the fire?"


We've seen an issue that looks like this one pop up in Safari due to the CORS headers for the fetched resource not being set appropriately. Safari was being far more pedantic about the headers it expected than the rest of the browsers were, which caused some external resources to not be loaded. @bsletten - were we trying to debug this stuff with you? @davidlehn do you remember what the issue was?


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


Arg! This is completely wrong! It won't work under separate compilation (if the `sym.owner` class was compiled in a separate run), because `jsInterop.isExposed` only works in the same run. How stupid could I be!?

I'll need to use an annotation instead of a `mutable.Set[Symbol]`.


@Tyrael This commit broke compilation under Windows:

    C:\php-sdk\php56dev>nmake php_ldap.dll

    Microsoft (R) Program Maintenance Utility Version 11.00.61030.0
    Copyright (C) Microsoft Corporation.  All rights reserved.

    ldap.c
    ext\ldap\ldap.c(1527) : warning C4018: '<' : signed/unsigned mismatch
    ext\ldap\ldap.c(1543) : warning C4018: '<' : signed/unsigned mismatch
    ext\ldap\ldap.c(1932) : error C2143: syntax error : missing ';' before 'type'
    ext\ldap\ldap.c(1933) : error C2065: 'lvalue' : undeclared identifier
    ext\ldap\ldap.c(1933) : error C2224: left of '.bv_val' must have struct/union type
    ext\ldap\ldap.c(1934) : error C2065: 'lvalue' : undeclared identifier
    ext\ldap\ldap.c(1934) : error C2224: left of '.bv_len' must have struct/union type
    ext\ldap\ldap.c(1938) : error C2065: 'lvalue' : undeclared identifier
    ext\ldap\ldap.c(2582) : warning C4018: '<' : signed/unsigned mismatch
    ext\ldap\ldap.c(2589) : warning C4018: '<' : signed/unsigned mismatch
    NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio 11.0\VC\BIN\cl.exe"' : return code '0x2'
    Stop.


even better dont install 1.8 mods for 1.7.10
OpenComputers-MC1.8-1.5.14.32-universal


fixes_for_VS2015 (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1763 of 1763 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)


@robfrawley You can use [Bintray](https://bintray.com/)?


Yeah, this is an odd one.  I needed to split epel-release out to a separate command otherwise packages that were dependent on the epel repo would fail.  I just retested and confirmed that's the behavior.


cxvxcvcx


@adamcrown! I can blame you since you are gone :) You removed this `embedded_in` then added it back later as a `belongs_to`. https://github.com/biola/buweb-content-models/commit/928cd2a9628e8cab18bbf2238f1eeac4237e704b#diff-0db945fd16111bc4fcaebf0bec031295R6

I'm going to change it back to `embedded_in`


OK, probably easiest to just skip this commit and then add the 2 lines from FieldExport.c and also the changes to __init__.py by the look of it.


@krancour ok, I will change the service to use the provided updating the flannel version


I think I picked wget because it does retry-on-failure by default. Looks like it also does the 3xx redirect handling by default, too. Curl can do these, but needs flags to do so.

The annoying thing was that wget wasn't installed by default. Looks like curl is.


Just because I was using `forEach`, which I wanted to drop.  I've since added docs on `of` and `from`.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 192](https://teamcity.labkey.org/viewLog.html?buildId=262134&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 16, passed: 3 Build time: 00:07:14
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

##### there are 6 more failed tests, see build details
```


fixes_for_VS2015 (mbadams5) - 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)


The description here states: "....on all fields of the native packet before any editing or transformations...".


Would you like stage1 (port acl), stage2 (l2 acl), stage3 (l3 acl)?


+1 :rocket:


Instead of getting a list of all the files in the directory and deleting it one by one, why cannot we use a function like os.removedirs, which will remove all the content of the directory and the directory itself. would be easier and safer


This is a much more solid version, merging this would be great, if you're open to that.


Yes, that'd be nice. I have access to windows, but I don't have anything set up readily. Argument should be `sockaddr_in6` as you say.
Someone needs to test this code.


Looks pretty good to me.


Thanks, Jc!
I like the third option, but I wouldn't have time to implement it now. I guess having to switch the flag is fine. Now at least if someone asks this question we'll know what to answer.


Return the value


Per Kelsey's tweet, https://twitter.com/kelseyhightower/status/619504135596077056, I'm going to provide some comments on this proto API.


Regarding #19 - we link to the data folder and the cucumber tests for real examples.


Another nice feature might be to have an object cache (in addition to the LRU compile cache that the compiler classes currently have) that would serve to cache objects across rebuilds. This would be helpful in keeping a Webpack cache alive during the duration of a `meteor` command execution, not just during a build/rebuild.