#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


The basic traversal code looks fine: how does this integrate with the code calculating file sizes?


[The best kind](https://github.com/WikiEducationFoundation/WikiEduDashboard/commit/ed7561750709ebe1530fc1b1282d75726c78e986).


Now TypeCollection.getArgs() is called twice, when it could be called once (just in the main method).


en php no es necesario cerrar el php... :)


Why do we even need this. Players don't care what map type it is (HashMap, ConcurrentHashMap, TreeMap, etc).


private


Do we need this guc anymore? If no, then we should remove it.


si lo armamos primero el uint32 y luego los uint8 por un tema de alignment.


remove prints


bien :)


:(
I really liked how these tests created their own sims. Having a "default sim for the system" is an annoying requirement. Can a test be added to check for the existence of that sim?


Latest git tag is v0.4.0, latest registered version, as stated in [pkg.julialang.org](http://pkg.julialang.org/), is 0.3.6, but none of them include `CLArray` type which is required for CLBLAS's high-level interface.


I mean, I guess one purpose for them is for use with `/dt` on Megas. `evoLevel` and `prevo` seem even more useless then.


esto supongo queda claro que no me gusta no? :)


Good point, I'll remove it!


Nice job Maris!


Ah, I misunderstood this option, thanks.


That would be very nice :)
many thanks.

regards


Fixes iOS Safari Issue https://github.com/systemctl-uk/website/issues/1


Do we need gp_mdversioning_loglevel anymore?


se puede hacer

MODS += modules$(DS)hisio \
                  modules$(DS)drivers \
                     ....


Yep, fixed.


supongo es solo para testear al comienzo? mejor usa ciaaPlatforms.h... para definir los macros.


Fixes #45 (forgot commit separately)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11345](https://teamcity.mesosphere.io/viewLog.html?buildId=103676&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:39


YESSS


Nice!


This breaks the code


Good point.


@xcomSteveJohnson Is this a valid oath key or just looks like it? Either way, this should be changed to a placeholder (like the other values). If it *is* a valid oath key, then whomever's account it belongs to needs to invalidate it! :)


document why this has been removed


Can't we just use `after` here?
the `after` variable is a modified version of a shallow copy. Here you are basically making a shallow copy of a shallow copy then. Perhaps we could also improve that in TypeCollection, idk.


Oh I like this. Also, be aware you can `.catch({code: 'ENOLICENSE'}, ....)`


do I really need to do this?


document why this changed


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - i386-Windows-7-VisualStudio-12: OK (2050 of 2051 tests passed, 0 test warnings)

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


yo preferiria usar bit fields :)


Do we need MDVerWriteLock anymore?


I checked it after several days, and _these_ data was identical. Download stopped working, though, so I may have missed something else.


I changed about 8 characters in the middle.


The tests are failing on Python3 here. This should be:

```python
from io import StringIO
```


We need to remove gp_mdver_max_entries guc as well.


@anthrotype Can you PR these changes to the mainline? Thanks.


ACK


```js
fs.writeFile('./npm-shrinkwrap.json', shrinkwrapString, done);
```


Thank you, Sergio! It was a really annoying bug.


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.450.0](http://build01:8888/viewLog.html?buildId=13810&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.450.0](http://build01:8888/viewLog.html?buildId=13810&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:31


I'm not sure what happens if `priv->old` is negative. I'd say it's UB.


Thanks for the cleanup!  :+1:


LGTM


I would prefer something like:

```js
if (shrinkwrap.dependencies.fsevents) {
  throw new Error("Uh oh! Somehow 'fsevents' snuck back into our shrinkwrap. This means something went wrong with our npm modules somehow. We can't publish fsevents in our shrinkwrap. To fix, run `gulp fixShrinkwrap` and try again");
}
```

so that if someone doesn't know what's up, they'll get an informative error instead of "expected xxx not to exist"


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11346](https://teamcity.mesosphere.io/viewLog.html?buildId=103677&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:45


What is this file for?


closing RMQ connection is not scalable here, having a long running connection will help a lot in the long run.


This is a bug in LEDBackpack, its fixed there now


Technically writing to a well-known file is a security hole: if another process races your process writing to and reading from the file, then whoever owns that process get you to read/display whatever they want...
A good overview of the problem (and some solutions) can be found [here](http://www.linuxsecurity.com/content/view/115462/151)


Sine you did the buffering via protos, you could just write this inline. Best of both worlds?


I still need to think about the use cases. This is just a PoC. I'll work on other things tomorrow, to convert the encoder pulses to distance, given the radius of a wheel and the width of the robot, which, while imprecise and susceptible to drifting, it's a start.


If you're ``/dt``'ing a Mega, you're definitely not doing it to check egg groups.


what if you dont get a response in a timely manner, lets say in 10 secs?


I squashed the bugs causing the crashes after many keypresses and also the proper handling of volume keys. I also introduced handling of the keyboard id by storing it in an id_3 variable similar to the already existing id_1 and id_2 principles. This represents a sort of binding of the keyboard and prevents handle_hotplug() from being called at each keystroke, which I believe was causing the crashes.


This was an attempt to overcome some intrinsic issues with CUDA / OpenCL drivers - this helped once on my local machine, so could possibly resolve the same issue in Travis. Anyway, this led to more troubles, so probably isn't good solution.

Note, that this and many other lines in `.travis` are temporary. As I wrote in a comment to the commit, I couldn't make Travis working, so just left it as is to continue later. If you want to take over it and try Intel driver, I can revert code to the original state.


This is how I was running the benchmark locally. All of the other benchmarks assumed there was a server running on localhost, but the reconnect benchmark needed to do the same type of server control as the reconnect test. I was using `docker build -t node-nats -f Dockerfile.reconnect . && docker run -it node-nats`


how would it be negative? You only set old based on new, and new is only based on 2 bits... to be stored in >=8bits... so never negative.


needs to be removed...


any particular reason you pulled this out of a form? I agree it wasn't necessary, but you don't mention it in your commit msg.


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 64](https://teamcity.labkey.org/viewLog.html?buildId=296411&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 34 Build time: 01:05:06
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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


We would like you to write the actual traversal code yourself... I know "du -abc" is not the example we gave: but we did ask not to use anything similar.


I know, I'm not even experienced with these guys, so I'm just asking questions here :-)

great work, let's see how they will do with concrete samples.


Wait, Megas have ``evoLevel`` and ``prevo``!? That should probably go too.


This was very useful!


A mi tampoco...en el apuro simplemente copié y pequé lo que estaba hecho en el ejemplo baremetal...


https://travis-ci.org/WikiEducationFoundation/WikiEduDashboard/jobs/101941508

:tada:


Yeah fair enough, I'll just ax all of them.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11347](https://teamcity.mesosphere.io/viewLog.html?buildId=103687&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:34


Instead of adding a button/key listener, did you consider saving it on blur?


Ok, si en algunas partes de los makefiles está hecho con \ y en otras con += ...será cuestión de unificar, pero lo hice indistintamente según como lo veía en el makefile, sin pensarlo demasiado


This needs the traversal code adding. The du -abc is fine as a placeholder, but the core marks are for the traversal code


> i followed password change modal (line 142)

Hmm ... I see where you're coming from. Both buttons are basically confirming an operation. Which means my suggestion of `submit_button` is wrong, since there's no form being submitted.

But the "send" in the modal you refer to is an abbreviation of the text displayed in the button. I suppose the correct name in this case would be `mark_all_button`.

> BTW this issue has bin fixed so it doesn't matter anymore

Good point! Thanks for contributing.


Idem anterior, copiado de examples/baremetal para hacerlo andar !


was there a problem this was causing?


Ah, putting it in a form caused it to capture the Enter key and reload the panel, which then caused the form to clear.

Basically, if you pressed Enter in the form previously, it would clear (after setting the value).


Yasssssssss


Would you submit this as a pull request back to Esri/calcite-bootstrap? Or at least get their take on whether or not this line is needed.


I didn't get an email about 0.0.8 being signed. Was it?


Trip will be kept @ `trip.js` until https://github.com/EragonJ/Trip.js/issues/137 is resolved


reverted.


Os numeros podem fazer-se de tipo u32 ou mesmo u8 para representar todos os valores possiveis de melhor maneira.


restore.


Cierto, igual esta struct se va a modificar, en lo inmediato creo se suma también algo como
 uint8_t Nro_Gpio;
 uint8_t Index_GPIO;


Can you elaborate on this comment?


Yes


Also, just noticed you didn't hide the standard CAP format; that is what is usually done.


2.2 was the version where charade was merged back to chardet


Yes, but in my testing the blur event would not fire on panel close, which means you might lose the change in some situations. I think it's a bit more explicit this way, but perhaps there's a better solution?


Should be able to just use server control directly, unless you are on windows. Don't think its needed in this PR.


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2351 of 2384 tests passed, 0 test warnings)

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


Not particularly, but it's a bit cleaner IMO to put the body of this in the showing function. I could change it back if necessary.


Awesome, let me know how it works out!


This comment sort of just restates what the code is doing, but in the opposite way.  How about describing a use case where a passed-in course would be None?


AddNetAreaColumnToEnvelopeSummary (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (2052 of 2052 tests passed, 0 test warnings)

![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)


Hey @marcellustavares!

[These tests](https://test-1-20.liferay.com/job/test-portal-acceptance-pullrequest-batch%28master%29/AXIS_VARIABLE=1,label_exp=!master/12093/testReport/com.liferay.calendar.util.test/CalendarBookingIteratorTest/classMethod/) are breaking due to this use of `ContextUserReplace`: the class package is not being found, preventing the bundle resolution:

    java.io.IOException: Start of bundle with id 424 failed with message: Could not resolve module: com.liferay.calendar.test [424]
      Unresolved requirement: Import-Package: com.liferay.portal.test

I don't know what is going on here, but I'm in talks with other engineers. I'll submit the pull request without this use. Later we re-add it once we understand what is going on. Is that ok?

Thanks!


Es correcta esta inclusión para los IO_ErrorType's ?


rm -rf $tempLayoutDir
should be
rm -rf $tempLayout


The program isn't ready to draw GUI elements on surfaces other than the default one at the moment but it is absolutely capable of it. I don't want to try that yet until I can get frames to work properly. I'll fix the multiple display compatibility in the next few commits.


keys is not a performant way to solve this. can you think of another solution?


Hm, that poses a problem indeed. I wonder if that's a bug.

If that's the sort of hostile environment we're in, I don't mind a button. Timers and others would just complicate things more than it's worth.


Yeah, it's the "temporarily unused" bit that bothers me. It also means that PR is basically untestable


Yeah I originally had it challenge-only, but jas told me they wanted to keep the normal ladder up but just ban Crucibelle, so I threw that in before merging.
Shadow Tag should also technically be allowed in the playtest since it wasn't banned when the process started, but they decided to go with OU as it is now. Go figure!


I'd drop the "Otherwise..." line from this comment and instead describe what will happen if they are in fact on the embargo list.


why do you need locking?


That would definitely make sense to me.


Can a non-nil res be returned from `getObject` with a non-nil err?


The lock could be moved to after the first call to `reset`


Please revert this ASAP. It's busted. I'll follow up with details.


Oh oops, I didn't know that existed. Thanks!


Needs significant JavaDoc comment explaining the usage and behavior, with example code


You're replacing Infof with Info. This results in broken logging like this:

 Assigning node %s CIDR %sip-10-64-1-111.ec2.internal10.172.244.0/24


This seems to be causing a test failure, but I don't understand why.


Fixed.


This could use a regular `std::thread`, and you won't need the gate:
```
std::thread([&]()
{
    this->expect(! lv.get());
    lv.get() = -2;
    this->expect(lv.get() == -2);
}).join();
```


We should use HTML5 tags, iframe is not supported anymore. You must use another kind of tag for the video. Also you should use different types of sources (are explained into the html documentation attached) which browser supports each source. Do not parse the files, just add the sources to let us know that you understand the stuff.

:)


Honestly, I'm not sure why this is needed -- the comment was there before I got to this code. My guess is that it's needed for access control in grading, but I'd need to check with the original author of that line.


This is OK, but add a description in case the sources dont load. And also add some more sources to get OK with the browser compatibility. The same as before check the html docs provided


OK, done, should be live in a few minutes.


Nah, it's fine. I'm just a proponent of "one 'thing' per commit". If you're tidying up code over here, make it a commit. If you're fixing a typo in a comment, that's in another one.

It's kinda the opposite of how BMO trains you to stick everything in one big patch. Hopefully that'll be killed dead now that mozreview's a thing.


Hi Sudheer, I added the empty spaces;
```
Paragraph preface = new Paragraph(); //line 49 in PDFExportVisitCallback
addEmptyLine(preface, 3);
document.add(preface);
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11348](https://teamcity.mesosphere.io/viewLog.html?buildId=103688&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:34


There is no way you can guarantee you'll be making a patch release on a stable. :)


Nice one... This pr will also remove other plugins, like NoNumber Articles Anywhere, Snippets, Modules Anywhere and such


I prepared a comparison of these algorithms, as well as the incumbent one, for moderately large *real* inputs.  (I have not played with the recursion relations yet.)  I've attached plots showing how many significant digits are successfully computed for different `z` values: [ref.zip](https://github.com/tpudlik/scipy/files/88016/ref.zip).

My findings:

1. The Taylor series is very good for a surprisingly broad range of real z.  We could improve on the incumbent algorithm just by using them for all `z` in (-10, 10).
2. The Taylor series starts to break down when `a` and `z` have opposite sign, in the region `|a| > |b|`.  Recursion relations might help here; I'm looking into it right now.  Another possibility is to use `bessel_series` in this region, since it performs well there even for `|z|` as large as 30.
3. The single-fraction method is similar in performance to the Taylor series, but less reliable for extreme values of `z`.
4. The asymptotic series does not work for any `z < 0`, and even for `z > 0` is still not very good at `z = 300`, except close to the lines `a = 0` and `a = b`.  I will look into changing the number of terms summed; perhaps this can improve accuracy.
5. Overall, as it stands `new_hyp1f1` outperforms `hyp1f1` for `z` in (-10, 1), but performs very poorly in the large-`|z|` regime.  By mixing `bessel_series` with `taylor_series` (cf. 2 above), we can certainly get something reliable in the (-30, 30) range or so, at least for `a` and `b` in the range (-100, 100).  I think clever use of recursion, which I've not yet exploited, could help extend this `z` range to (-100, 100) or so.  But large `|z|` values remain a challenge.


We might want to call the format `CAP (Crucibelle Playtest)` so we automatically use the right team format.


I would prefer and "available" field that is a boolean


What does this do? It seems to assume that it can max out the rest of the class A. So if I pass --cluster-cidr=10.65.0.0/16 it assumes there are 191*256 subnets (/24) available? That's wrong, I passed /16 for a reason. It should return 256 in my case. Could you please add some tests and/or logging?


Better to use `.replace('patches.', '', 1)` to avoid replacing another 'patches.' if it happens we have that later


Example : alert gets sent to a mailing list, but its assigned to one person for remedy. So sender/receiver is separate from owner.


???
Why the partRevision has to be checkoutByAnotherUser?


property ? :D


Maybe we can say that in the comment, then :)


AddNetAreaColumnToEnvelopeSummary (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1503 of 1503 tests passed, 0 test warnings)

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


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (951 of 951 tests passed, 0 test warnings)

![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-26.84%25-green.svg)


same here, replace with max 1 occurence


This is great. I have a bunch of Edison boards laying around which are similar and maybe even easier to get running. Any ideas on what it would take? I can send you one if you'd like to add support as well, or offer remote shell on one for testing.


@ekryski @daffl What do you guys think about creating a separate service template for each DB type instead of having a bunch of if statement logic in a single template.  We already have the if statements in the main app file where we could simply load the corresponding template.  I think that it would be more maintainable that way.  The only downside I can think of would be that it would require a little bit more work to change all of the templates if we have a big API change in the future.


There's no problem, if you have a wifi network you can test your code directly with your mobile, to avoid installing the android emulator which is nasty haha. Also write this at english the next time.


@sunaku: HDMI events are working fine on C720 when the lid is open. It was a Chrome OS software update, that made it possible.
Thanks for your consideration.


FIxed.


I'd use labels to relate the input. This is a better markup. You might need to apply that changes here and in other places.


Seems like a silly coding standard to me.  I'm not worried about the performance -- it just seems odd to have unnecessary hidden casts.  Is there some justification for such a practice?  I especially don't like using integers to initialize doubles -- in my mind, that makes the code more likely to be misinterpreted upon a casual read.  Furthermore, this "sloppiness" leads to mistakes such as the one that @mariospr caught on https://codereview.chromium.org/1324513002/.  Testing 1.3f < 1.3 evaluates as true, which is not the intended behavior in this code.  (And, if performance is not a concern, and not dealing with an array, why not simply use doubles rather than floats?  Similar to how you would not use a short rather than an int unless there is a specific reason.)


i've done this problem before, it's just not working now.
def create_collection_of_numbers(nums)
  h={}
  nums.each do |n|
    if h.has_key?(n) == false
      h[n]=1
    elsif h.has_key?(n) == true
      h[n]= h[n] +1
    end
  end
  return h
end

def find_most_repeated_number(nums)
  h = create_collection_of_numbers(nums)
  arr = h.max_by{|k,v| v}
  return arr.first
end


du könntest das doch jetzt dank return $this als chain verwenden $backend->setVisible()->setRelation() ... glaub macht aber kaum unterschied da der interpreter sowieso atomic arbeitet und ned optimieren kann :)


dude .... var ? srsly :D vermisst du php4 ? :D


Hello @ansis
Would that fix these issues related to pixelRatio different to 1?
https://github.com/mapbox/mapbox-gl-js/issues/1476
https://github.com/mapbox/mapbox-gl-js/issues/1067

From what I can read in the changes, that would solve blurry/cropped POI icons

Thanks,
Fabien


It's OK, you might need to self close this tags. All tags must be closed somewhere.


I am still trying to figure how --cluster-cidr=10.65.0.0/16 resulted in ```Assigning node %s CIDR %sip-10-64-1-111.ec2.internal10.172.244.0/24```. Any ideas?


ist da nicht "setShowFields" angebrachter ? also vom verständniss her halt ? :)


but does it, unlike the 1.7 version, render on my graphics card...

YES! \o/


This is a huge step! Our Drupal AngularJS integration efforts will be greatly benefited by this change. We can now proceed with confidence that functionality we develop may at a later date be potentially committed to core. This magnifies the potential benefit and I think will give us a proxy standard for JS development in Drupal.


property :D !


issue #36


a la hora querer imprimir el contenido de algun campo sale un error

Traceback (most recent call last):
  File "C:/Users/MCC1/conexionbdb/conexiondb1.py", line 22, in <module>
    obj_head = result['head']
TypeError: 'long' object has no attribute '__getitem__'


Ha ha! @marshallswain you read my mind man. Already in the works. Just wrapping it up. :smile:


@jimporter, overall I really like a lot of these changes, though I'm unsure about this: it seems we're actually scattering parts of the backends' implementations throughout the source tree (builtins in particular), rather than keeping them isolated to backends/$backend/. I'm concerned we'll lose apparent modularity.

The AST was a kind of common language between the frontend and backends, though I admit it was probably too low-level/verbose. It may be worth it to have the backends implement a common set of APIs (abstract base class?) that the AST->buildfile generator step employs.


I just learned that the official bitflags has a `no_std` feature now: rust-lang-nursery/bitflags#29


Thanks @corani - implemented in 0fb32fb and 4270f62


hahahha


This looks good.  Making it dynamic would be also detecting if the object responded to #call, and calling it with the request in that case.


All the techne converted models are kinda inefficient atm though (the texture is huge and causes big gaps in the atlas when stitched), but hey, it works


Thanks for your great answer :100: Will play around with Compose and Transformer a little bit. For now I went with the simplest solution (https://github.com/stoeffel/classroom-coding/blob/redux/src/frontend/js/model.js#L35)

> Just realized it's probably status 200, even though it's a failure (thanks flickr). Which is why you made the Either the way it is. So...Compose or Transformer...

Jop, Flickr responds with a status 200 and
```
code: 3
message: "Parameterless searches have been disabled. Please use flickr.photos.getRecent instead."
stat: "fail"
```


Also refactored Rules.rb for legibility


ah shiet can't hide easter eggs


still 4 spaces instead of 2 and still a semicolon :stuck_out_tongue_closed_eyes:


Obrigado pelo feedback :+1:


That will still cause the user to see a range of warnings on stdout when he calls lxc-ls especially when he tries to see nested containers.


all file names must use lower case and must split phrases and words with _ or - using your own pattern


Please use `let` instead of `var`


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11349](https://teamcity.mesosphere.io/viewLog.html?buildId=103692&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:25


Commit comment test


This was related to the way the shortcode evaluates wether to display the content or not.
I've fixed the logic behind that validation and it's working properly when using Shortcake UI.


I'd group header and footer selectors to apply the text-align property and then two separate selectors (header and footer are ok) to set the background-color


Why you make this? What's the purpose of this?


The definition should be separate from the declaration


Ok, the commit has been reverted.


c'est vide


-> Orthogonality oder anders gesagt -> show fields wird dir einen error werfen wenn du davor nicht setRelation aufgerufen hast

du kannst natürlich auch einen check einbaun aber sinnvoller ist es bei show fields den propertyName mitgibst

btw mir gefällt der ansatz vom virtuellen register allerdings solltest evtl die "structure" definieren am besten über protected $properties. Ich würd halt eine simple PropertyClass definieren einfach nur damit es ein klares interface mit gettern und settern gibt. obwohl die modernen IDEs zum glück schon array autocompletes unterstützen und so typos vermeiden :)


We dont need to specify if its a user-defined or system-defined alert ?


If you are using an external css file why are you defining this into the html file?


yes


Both those issues were closed already. We're you still experiencing them?

I checked for both issues in master (which includes this commit) and wasn't able to reproduce, so if they weren't fixed earlier they should be fixed now.


naming conventions :D ..... is_admin hab eigentlich kein problem mit camelcase in DBs aber ich bin ein entweder oder mensch :)


Infof


> I assume that this would help reduce the maintenance overhead for mxe once upcoming versions of openscenegraph have these patches included ?

You are right. Ideal package has no patches.


Thanks MrVan, I was also wondering what is running in the normal-world after optee os?


Ok, the URL and Images fields has been added and available to be selected using Shortcake UI.


Hey @marcellustavares!

[These tests](https://test-1-20.liferay.com/job/test-portal-acceptance-pullrequest-batch%28master%29/AXIS_VARIABLE=1,label_exp=!master/12093/testReport/com.liferay.calendar.util.test/CalendarBookingIteratorTest/classMethod/) are breaking due to this use of `ContextUserReplace`: the class package is not being found, preventing the bundle resolution:

    java.io.IOException: Start of bundle with id 424 failed with message: Could not resolve module: com.liferay.calendar.test [424]
      Unresolved requirement: Import-Package: com.liferay.portal.test

I don't know what is going on here, but I'm in talks with other engineers. I'll submit the pull request without this use. Later we re-add it once we understand what is going on. Is that ok?

Thanks!


Ya du progrès sur Bleach, on vois les forme des personnage qui apparaisse dans le menu, ça fait comme des ombre tout noir maintenant, dur d'y screener lol


It's copying to a different ComputationStrategyInfo class used in the secondary lib subproject.


You might want to consider pull #193 instead of JDK Function for TestingHttpClient.


Hi,

Do you have a Plnkr? I can't make it work...


Should these be reversed check inspect_container first if not there then check container?


this is actually wm-agnostic so I'm not sure whether it should go in another, `listers/wm.py` file.


Should this be .onLoad()? Otherwise, is_quiet() is broken by default if the package is loaded but not attached.


Should not it be:

private boolean hasNoAccessibleIteration(PartRevision partRevision) {
         return (partRevision != null
                 && (
                 ((isCheckoutByAnotherUser(user,partRevision) &&
                 partRevision.getPartIterations().size() <= 1)

                || (((!isCheckoutByAnotherUser(user,partRevision) &&
                 partRevision.getPartIterations().isEmpty()) )

));
}

?


I like the idea :D I pointed out the only dependency issue the rest ist fine by me :D the property should be renamed :)


They were not really properly closed, icons were either blurry cropped because the fixes were a bit too "simple" and the real fix was to resize the sprite atlas properly, which your fix does, so I guess it is all good now :)

Thanks again for your great work,
Fabien


Usually this is named `dry-run`


haklısın, sanırım tekrardan okumam lazım. teşekkürler.


Shouldn't idea-community now have unfree license?


As far as I know, ShortCake UI does not support multiple images field type out of the box. This can be included as soon as we find out more about setting up custom field types.


As a first cut, I want to process only system defined alerts, i.e. those that are owned by us.

 For user-defined alerts, I'm assuming that the task and flow will be different, with configuration options etc.


yes, those values are flags, which allow to enable multiple resources and can be used like this:
`HardwareResourceNetworkManager | HardwareResourceUpnpDisovery` -> `0b011000`

see line 60


From what I have gathered was that it was decided at some point to not have that functionality and that code was never removed? I'll go confirm this to make sure.


With WARN?  Can you have lxc_ls.c make sure that by default the log-level is error, then
this won't show up?  Bonus, then the user can do -l trace and get more info from lxc-ls.


.txt-files, which were deleted, were the copies of original .py-files to send them via social networks easier.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1220.4cf0473a857fe13bbc297f3a1cf56963a7c1c759](http://build.palaso.org/viewLog.html?buildId=67891&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 836, ignored: 7 Build time: 00:07:18


BTW I know this was a superficial thing, I'm just getting started (in a separate local copy of the patch) going through in detail.

Looks very nice so far.


Definition should be separate from the declaration


Why do you need this?  Just execute the container with the command.


No need for temporary container when using alt command.


Yes, I was thinking about the same class but in the secondary package.  I probably just missed it.


Issue references of doc. (Core jsdoc documentation started)
refs #10


oh, sorry, I responded to fast! my error! you are right! thx!


I don't understand this change.  self.gridtype_str should be unicode.  All the OP2 strings should be returned to the user as unicode to prevent encoding difficulties.  A numpy dtype=str corresponds to a byte, which is not what this should be.

http://docs.scipy.org/doc/numpy-1.10.1/reference/generated/numpy.chararray.html


Awesome! Thanks a lot!


I guess I decided to add the new field name after the other ids because I thought it made sense to have the user column id and user field name next to each other, but maybe that isn't a very good reason? Both field name and computation strategy are being added to the end of the protobuf messages.


ok


Hi Mr rolandotr

I try to comipl db_comparaison, i not able de do. please i's possible to send me this project compiled (.class files). Or how can me compil it. Thank you for your help.

Student Miri form TUNISIA


maybe remove the error from the example?


Looks good. Looks like you just changed the api keys and callback url


ok, they need to be in specific form.


We can do this easier. I totally missed: We can simply check if we can access the path and if not skip the call to list the containers thereby avoiding the need to touch the function at all.


haha estoy comentando tus git


Weird.  This is is the first battle that the scores showed up on...

8d59f3844ff3486bf1ec0eacc40bad28c9cc5796...60da070e452b59f7494d467a1becf831e5db54a1


You're mixing two concepts that have nothing to do with each other - properties and AST_VarDef. An AST_ VarDef is a variable definition - a `var` or a `const` statement which may define many variables. Properties are children of AST_Objects.

Does this new AST_VarDef fix any concrete test case?


I actually think we should have a new image for the 404 page - it's very blurry at that resolution.

https://atf-eregs.18f.gov/448

Is that what you were asking?

@jehlers What do you think.


You're the second person to ask me about this! I have an Edison port of Buildroot that I did 6 months ago that I need to bring up-to-date and integrate into Nerves. There are actually quite a few differences between the platforms, so it's not a quick task. Intel was a little frustrating in their support last time too. My plan was to make a pass on some other areas of Nerves first and then circle back to this.


Sorry for the messy pull request.. codewise it seems fine, at least.


what is the gain here if you are Locking and using separate goroutines? Iteration wont go further since lock is already aquired.


I think you made a mistake here. It is spelled `isEmpty` twice.

Your tests should not have been able to run..


Good to know! It looks like the official bitflag crate is up-to-date with the current nightly though, and thus doesn't enable the `no_std` feature gate. Right now it won't work for me yet 'cause my build process is still locked in to an older version of Rust.

I'm actually having some problems switching over to your `nightly-libcore` crate, but I'll open an issue for them on your repo for the crate.


Added shit


m


For some reason this is not working for me.


That's ok.  I am just curious.  But I also give point to pushing optional stuff to the end.


Regarde avec ce que je viens de pusher, j'ai fixé un bug


As long as a fix passes https://gist.github.com/avdg/d1372397b255e57408e8 and doesn't form a problem with real code, you get the freedom to propose an other fix.


Here is the issue.  The convert to tabs is missing.


`gzip -9 -c > $FILE | aws s3 cp $FILE`

Porquê o ficheiro à parte, ou porquê o pipe.


Should there be a period at the end of this?


Can this ever happen?


Do you have conventional test/compress/*.js test case? I don't understand what that gist is trying to prove.


Test


Ah ha! https://github.com/DarbyComputerClub/darbyrobocode/commit/f22e1b7cb8f3e134fbce8140e202284815ce097c#commitcomment-15409380


Add #39 to track Edison support


These three are internal functions and should still be od_

It's not super pretty as then they don't have the type name in the function call, but I prefer that to breaking the convention.


Is this still wanted? (not sure what it does)


This seems to assume that this === global?


(non-blocking) at some point we should consider making this an external file


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11350](https://teamcity.mesosphere.io/viewLog.html?buildId=103713&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:55


Ooh, good point. I did that just now.

-- Michael

On Tue, Jan 12, 2016 at 10:42 AM, Jeff Pyke <notifications@github.com>
wrote:

> Do we need to reset our token?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uwfsae/team-website/commit/2bea866b743fd6dfb24a6929a86c64bc73566c20#commitcomment-15405008>
> .
>


@dnephin Can we use ENV variables to create the tag name from eg. `git describe`?


<nobr> and
 have been depreciated for a while now and according to this article should be avoided. You can use
. It is preferred to use css nowrap instead.  https://davidwalsh.name/nobr-css.


hmm, sorry for missing this early, but I figure the word "option" is missing here...

(Also, please suffix the `--machine-id` option here with `=`, I think that makes clearer that this expects a parameter.)


As you can probably tell, the general idea is to put all the code for each *builtin* together. This helps in the short term because there's a lot of shared code for each builtin (mainly between Make and Ninja since they're very similar). However, this is really part of a longer-term goal to enhance the modularity of the various builtins/rules (at the expense of modularity in build backends). Eventually, I'd like to make it really easy for people to provide their own extensions to bfg (e.g. in `my_project/bfg9000`) so that they can add new builtin functions, new languages/compilers, etc. I should probably create an issue to track this.

I think it's a lot more likely that people will want to add those kinds of things instead of adding new build backends; there are only a couple more build backends that I think people might want (XCode, Code::Blocks, Tup, Bazel?), but there are a *lot* more languages/build steps that people would need. Once I make it possible for people to extend bfg's builtins, this structure would mean we can just drop people's extensions into `bfg9000/builtins` and provide them as standard features of bfg. This will (hopefully) be helpful for people who need weird build rules, or common ones I just haven't had time to implement.

There's still a little more work I need to do to make this truly modular, namely making it possible to add custom fields to `BuildInputs`. `TestInputs`, for example, should really go in `bfg9000/builtins/tests.py`, not `bfg9000/build_inputs.py`.


I don't see any reason why not, I'll take a look at creating an issue for it.


Should be noted this is in interpreters only, so people don't expect magic to happen. Good job nailing it BTW :)


you mean the ignore? That is needed for the linter to be happy, otherwise you get `[#12] No docs found for member 'this.ICAL' (Error 220)`


"const" makes no sense in C really, except on pointers. Please don't use it.


That's not really html, but some weird subset parsed by Qt.


@marchof wrong name of option?


Looks great otherwise


oh right ok.


Updated test case. And no I can't convert a test to a test/compress/*.js one as it analyses the ast by checking the arguments scope.


Would prefer to have read_error_log and truncate_error_log functions defined in same file. If it means not to have CREATE FUNCTION for truncate_error_log and directly define it manually as well should be okay.


Left menu stay constant when navigating into different pages, so url must be explicitely set relative to DOL_URL_ROOT or link will be wrong.


wait! I have a newer version!


It shouldn't, but that really depends on the output of `boost::core::demangle`.


I have to clarify the compares though


Haha ok :P


Thank *you*, sir! :smiley:


This is how we are "forcefully" overriding an existing HashJoin plan with a ResilientJoin plan. This enables testing the functional equivalence of ResilientJoin with a HashJoin operator, even without planner/optimizer support.


I decided to go with purecss and html5 boiler plate, the project setup is ready, I'm going to git push --force, ok?


I believe so. You'd have to put the value of `git describe` into a variable before calling compose though, you can't run the command from within the config.


Oh ya! Go for it!


I'm probably making things up - but could you try doing a nested ls where a nested container is i386 and the parent is x86-64?


This is the HashDummy that is just a no-op for ResilientJoin. A pure ResilientJoin plan, generated from optimizer does not have this node. However, if we decide to override existing HashJoin plan with ResilientJoin, we can place a dummy hash node that just passes all the input tuples as output and let the ResilientJoin do all the work.


yes, which is mostly the case because it is top-level


(non-blocking) - we don't really have a standard for generating html from javascript and it depends a bit how much html you're generating, but this could be more maintainable if it used a template or jquery.

can the bulk of it not be on the page and then the dynamic bits filled in as needed (maybe more trouble than it's worth).


I figured this was the case since I noticed it was allowing things like 'Pepsi' as a block name which would begin a bad loop.

Nice fix.


Nitpick: No need the extra scope if there's a single thing in `main`.


This is where in the translator we generate a ResilientJoin operator, instead of a HashJoin operator. Note, a ResilientJoin operator can mimic both HJ and NLJ (and potentially others in the future).


It makes sense in the TB context, I don't what other use cases there currently are.


I might be more tempted to do the following. Since we will need a different tag for each kind of `Metafunction` (because we'll want to print differently for each of them), why not instead use the `template_t` itself as the tag?

In other words, I would simply add the `print_impl` specializations for each of `metafunction_t`, `template_t`, etc., without resorting to explicit tags (such as `template_tag`, `metafunction_tag`, etc.).

What do you think?


For ResilientJoin we bypass the Hash node generation.


So with the release version of uglify - without your patch - you can't create a javascript test case that once compressed and/or mangled produces the wrong result?

Then I don't understand the point of this code.


Other use cases are node and the browser. node gets handled via the first case, browser has this === window, which is what is expected.


It's been 64 commits since my last addition, though!


ls_new() really wants a comment :)  If you prefer I'll be happy to add some comments in a new commit after yours is merged.


nice cleanup!


A ResilientJoin does not need any additional info than the current HashJoin.


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


UpdateLightFractionTableInDocs (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2044 of 2044 tests passed, 0 test warnings)

![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)


Yeah, basically it queues the callback to run the next time there's no other code to execute


Right... So it could happen that `core::demangle` returns mangled BS on some ABI it does not support, and then we would en up in this branch.


This should be done in the logout success callback


I don't know if we have that data yet?


LGTM, sweet recursion


What is the use of those 2 lines?


No, but it's not hurting anything...just showing the program name right now. You'll need to fix the navigation thing.


Using `.find('<')` is IMO much better than hardcoding the length of the type, so I'd leave it that way.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11351](https://teamcity.mesosphere.io/viewLog.html?buildId=103715&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:01


Workaround_Thread_Local_MinGW (lefticus) - i386-Windows-7-VisualStudio-14: Build Failed

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


Workaround_Thread_Local_MinGW (lefticus) - Win64-Windows-7-VisualStudio-14: Build Failed

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


Workaround_Thread_Local_MinGW (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (383 of 383 tests passed, 0 test warnings)

![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)


Utils#valid_path? could potentially be used by frameworks like Rails to make sure a path is valid before calling Utils#clean_path_info.


Workaround_Thread_Local_MinGW (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: Build Failed

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


Yeah, debug mode usually prints more stuff but still does everything the non-debug mode would.


@stevvooe No, multi writer fails if the first writer does not accept the entire buffer. I could use multi writer, and in case of a short write append a bit more to the digest, but I would think that makes things any easier.


j'ai tester après avoir rebase mais sans enlever t'es dernier commit, rr7 crashait encore et bleach par contre c’était encore mieux, on voyais clairement les fil de fers.
Apres avoir supprimé des 2 commit donc, RR7 crash toujours :/, et Bleach n'a plus les model 3d qui arrivent


Aha, I was unaware of that possibility. I guess that is why `hana::tag_of` is idempotent. I'll look into doing that. Thanks.


Shouldn't the default value for ``journal.reversed_order`` be ``true`` instead? The previous behavior reversed the order, so upgrading configurations would be in the "true" state, right? Not entirely sure what this does though, just checking.


The compression tests only allow to compare trees, it does not allow static analytics of the trees which is needed. Consider the example below:

```js
var arguments = [];

var f = function() {
    return arguments.length;
};

console.log(f("a", "b", "c", "d"));
```

A correct js implementation would return 4, because there are 4 arguments passed to function f. Without defining the arguments object, uglifyJS might (if it would evaluate the expression) think wrongly that arguments.length equals 0 because arguments is defined on the first line.

UglifyJS won't be able to know arguments is defined in the function scope.


This test is almost the same as `should write file mtime using the vinyl mtime`, but the additional check on `atime` is no longer valid with contents accessors from the PR at `vinyl`.


 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 13 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/5852412d2c9e8077200dcda1f10e238dc2e40d24/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


It would be good to move this fake initialization of gputop_devinfo into `init_dev_info()` because as we extend `struct gputop_devinfo` we need to also remember to update this fake initialization which we may easily overlook if it's not all in the same place. The code can be something like...

```c
// XXX: note that checking gputop_devinfo.n_eus will let us skip re-initializing in the fake case too
// (which won't init intel_dev.device
if (gputop_devinfo.n_eus)
    return true;

if (getenv("GPUTOP_FAKE_MODE") && strcmp(getenv("GPUTOP_FAKE_MODE"), "1") == 0)
    gputop_fake_mode = true;

if (!gputop_fake_mode) {
    drm_fd = open_render_node(&intel_dev);
    if (drm_fd < 0) {
        ...
    }
}

// XXX: doesn't matter that these are uninitialized args in fake mode case
// since the implementation can check gputop_fake_mode and not reference
// the drm_fd or the given device id
init_dev_info(drm_fd, intel_dev.device);
page_size = sysconf(_SC_PAGE_SIZE);

if (gputop_fake_mode) {
    gputop_oa_add_queries_bdw(&gputop_devinfo);
} else if (IS_HASWELL(intel_dev.device)) {
    ...
} else if (IS_BROADWELL(intel_dev.device)) {
    ...
} else ...

return true;
```


Yaay :tada: :balloon:


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


Width 3.9, size of first td is now set by the titlefield class. All hard coded with must be removed.


So what was the problem here? Did the old story.ni get used, so the new file for Mul (and its contained variable) was not included?


You'll need to send "frontpage", "all", "multi", "friends", "mod", etc for all the listings that aren't specifically subreddits


Done.

On Tue, Jan 12, 2016 at 10:41 AM, Jeff Pyke <notifications@github.com>
wrote:

> Maybe we should also pull the app id into config?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uwfsae/team-website/commit/2bea866b743fd6dfb24a6929a86c64bc73566c20#commitcomment-15404988>
> .
>


RE:  03_command_line

hi @nisharma85
cc: @julialintern @aramnath @Vinoad @galvin-mj @debbiebere
cc: @reshamametis
--

Great job overall! :+1:

Q3 xargs is a command used to execute commands based on arguments from standard input

Please refer to [Prework Solutions](https://github.com/thisismetis/nyc16_ds6/tree/master/00-prework/solutions) for additional commands, examples and formatting.


We can't compare the whole object since `stat.atime`, `stat.mtime` and `stat.ctime` change.


I think this can be something like, "No config file was loaded."


- [ ] this is an incomplete item


This is redundant for the 1 line it contains.


you crazy hehehe


There's 2 `strtolower` call in a row...


Wouldn't it be easier to
```
#if defined(__APPLE__) && defined(__MACH__)
	#define CTRL 0x800
#else
	#define CTRL 0x200
#endif
```
I'm not a Mac user, but macs don't have Control keys at all, do they?


No idea particularly, possibly that.

On Tue, Jan 12, 2016 at 2:13 PM, Wahn128 <notifications@github.com> wrote:

> So what was the problem here? Did the old story.ni get used, so the new
> file for Mul (and its contained variable) was not included?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Nuku/Flexible-Survival/commit/71b88aa91d1fc566e1af4e0b3ca0757ef4789ed3#commitcomment-15409780>
> .
>


If you want there's some old base stuff here that you could integrate into the new boilerplate...

https://github.com/hooklift/www.hooklift.io/blob/whoopsOnMyCompForAges/themes/hooklift/layouts/index.html


The main reason why `tag_of` defaults to the identity is because it allows stuff like `std::string` to be usable with Hana out of the box (since e.g. `tag_of<std::string>::type` is `std::string`, and `std::string` (as a tag) is EqualityComparable, so `std::string` (as a tag) is `hana::Comparable`). But it does allow doing what I'm suggesting, too.


LevelTest.unity is not longer able to start the server up when running it in the Unity editor (a good thing)
Use Setup.unity to start the server up.


Hey Иван,

Thanks a lot for your contributions. :)
I wish to implement this feature but these days I lack of time to do it myself, so I'm glad you want to implement it too.
If I can help, feel free to ask.
I think a good start to implement UI lock is to add a new signal from Document object (cf: https://github.com/wisk/medusa/blob/dev/inc/medusa/document.hpp#L46), thus if a complex task is performed, one can lock the document and then the document sends this signal to all widgets automatically.


And here...

https://github.com/hooklift/www.hooklift.io/blob/whoopsOnMyCompForAges/public/stylesheets/main.css


Doesn't explain the missing Hayato.jpg

On Tue, Jan 12, 2016 at 2:14 PM, David Silver <dms.silver@gmail.com> wrote:

> No idea particularly, possibly that.
>
> On Tue, Jan 12, 2016 at 2:13 PM, Wahn128 <notifications@github.com> wrote:
>
>> So what was the problem here? Did the old story.ni get used, so the new
>> file for Mul (and its contained variable) was not included?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Nuku/Flexible-Survival/commit/71b88aa91d1fc566e1af4e0b3ca0757ef4789ed3#commitcomment-15409780>
>> .
>>
>
>


And here...

https://github.com/hooklift/www.hooklift.io/blob/whoopsOnMyCompForAges/public/javascript/main.js


loop enquanto n>k. such skills, much impressive, wow.


Hi, @amhuber,

This exact same stub/template been in cf-release for a while now: https://github.com/cloudfoundry/cf-release/blob/develop/bosh-lite/stubs/enable-diego-ssh.yml

We do intend to update and to improve the documentation around the SSH configuration required in both the CF and Diego manifests. I have https://www.pivotaltracker.com/story/show/107970964 in the Diego tracker that I'd like to get polished up and prioritized for discussion at the team's next IPM, and completed shortly after that.

Thanks,
Eric


on le vois super bien la
<img width="1296" alt="capture d ecran 2016-01-12 23 16 05" src="https://cloud.githubusercontent.com/assets/5261759/12278642/8b197400-b982-11e5-8da1-8bf69fc82d9b.png">


missing check against NULL on `value.second`


ok, I will look into it. Thanks @jonjohnjohnson!


Really?
Are we going for the commit message of the year award already? :loud_sound:


Make sure the version of stdlib matches the version in the CLI


Workaround_Thread_Local_MinGW (lefticus) - i386-Windows-7-VisualStudio-12: OK (382 of 383 tests passed, 0 test warnings)

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


*All* samples need this.


Can we not use the `logical.Operation` names here?


Ah, I see. Let's throw a comment there, since that slicing is fairly important.


Still nothing in the code. Trying to send an e-mail still results in "Fatal error: Call to a member function getBaseUrl() on a non-object in /var/www/html/tickets/include/class.client.php on line 64"


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] (0.5 pt) Assignment 1 contains directory of files for <user>
- [ ] (0.5 pt) Repository contains screenshot of [github][] account
- [ ] (0.5 pt) Repository contains screenshot of [Slack][] session
- [ ] (0.5 pt) Repository contains screenshot of [Atom][] installed on users computer
- [ ] (0.5 pt) Repository contains screenshot of screenshot showing that user installed [linter-flake8][]
- [ ] (0.5 pt) Repository contains screenshot of the Anaconda [REPL][] on user's computer
- [ ] (0.5 pt) Repository contains screenshot, in the repository, displayed by Github Desktop

## jupyter notebook

- [ ] (0.5 pt) Repository contains ipython/jupyter mini-tutorial as ipynb file
- [ ] (0.5 pt) ipynb Task 1 is presented with a description explaining what is occurring
- [ ] (0.5 pt) ipynb Task 2 is presented with a description explaining what is occurring
- [ ] (0.5 pt) ipynb Task 3 is presented with a description explaining what is occurring
- [ ] (0.5 pt) ipynb Task 4 is presented with a description explaining what is occurring
- [ ] (0.5 pt) ipynb Task 5 is presented with a description explaining what is occurring
- [ ] (0.5 pt) ipynb Task 6 is presented with a description explaining what is occurring
- [ ] (0.5 pt) ipynb Task 7 is presented with a description explaining what is occurring
- [ ] (0.5 pt) ipynb Task 8 is presented with a description explaining what is occurring
- [ ] (0.5 pt) ipynb Task 1 appears to be answered correctly
- [ ] (0.5 pt) ipynb Task 2 appears to be answered correctly
- [ ] (0.5 pt) ipynb Task 3 appears to be answered correctly
- [ ] (0.5 pt) ipynb Task 4 appears to be answered correctly
- [ ] (0.5 pt) ipynb Task 5 appears to be answered correctly
- [ ] (0.5 pt) ipynb Task 6 appears to be answered correctly
- [ ] (0.5 pt) ipynb Task 7 appears to be answered correctly
- [ ] (0.5 pt) ipynb Task 8 appears to be answered correctly
- [ ] (3.0 pt) You can successfully re-run every step of the ipynb notebook


建议在README.MD文件里提供一个链接用于打开这个网页。


Makes sense! Done!


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


You are correct, I'll fix in the next commit. Thanks


I've been meaning to ask what is the purpose of selfcall here. IIUC self compiles to an immediately invoked function expression right?


Turning the warnings into errors make sense.  The compatibility checks are
opt-in, so you can simply remove the flag if not desired.

On Tue, Jan 12, 2016 at 11:27 AM, Wei Song <notifications@github.com> wrote:

> Agreed if the Chisel3 syntax is stable enough.
> Otherwise, IMHO errors could enforce duplicated efforts such as move the
> arguments of Vec(n, type) back and forth.
>
> -Wei
>
> On 12/01/2016, 19:23, Howard Mao wrote:
> > I think we need an option to turn the Chisel3 compat warnings in to
> > errors.
> > It's kind of hard to tease out which warnings you actually need to pay
> > attention to.
> >
> > On Tue, Jan 12, 2016 at 11:20 AM, Yunsup Lee <notifications@github.com>
> > wrote:
> >
> > > Just for the record, this branch was a one-off branch, which I was not
> > > intending to merge in as is.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > >
> > <
> https://github.com/ucb-bar/rocket/commit/55b4f9c174b4dcc0879c322fb2e258dc302e30d8#commitcomment-15405807
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ucb-bar/rocket/commit/55b4f9c174b4dcc0879c322fb2e258dc302e30d8#commitcomment-15405884
> >.
> >
>
> --
> Dr Wei Song ( 宋威 )
> Research Associate
> The Computer Laboratory, University of Cambridge
> http://www.cl.cam.ac.uk/~ws327/
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ucb-bar/rocket/commit/55b4f9c174b4dcc0879c322fb2e258dc302e30d8#commitcomment-15405966>
> .
>


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11352](https://teamcity.mesosphere.io/viewLog.html?buildId=103716&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:07:16


Yes, it is. Perhaps the compiler would be able to optimize it away, but I can definitely imagine code being generated if the copy constructor has side effects. Better safe than sorry :)


RE:  05a_python

cc: @julialintern @aramnath @gfsoileau @Vinoad @galvin-mj @debbiebere
cc: @reshamametis

--
Hi @nisharma85
Great job on these Python Part I exercises!  :+1:

Q1-Q4 : answers are correct and well summarized.
Q5:  computed days are correct.  suggestion: write function instead of repeating code.
Q6:  strings - look good.
Q7: lists - (double check last function "linear_merge"). rest look good.
Q8: [reference for parsing solution](https://github.com/agatorano/dsp/blob/master/python/q8_parsing.py)

Please refer to [Prework Solutions](https://github.com/thisismetis/nyc16_ds6/tree/master/00-prework/solutions) for solutions to lists and strings functions.


This is true, fixed.


This hasn't yet been merged with the development branch. It'll only appear once Jared approves the PR (he's already given it the thumbs-up: https://github.com/osTicket/osTicket-1.8/pull/2837)


Le débogueur indique encore cette ligne


Slightly pedantic, but can you please make this `get_time(void)`. This syntax is a something of a pre-ANSI K&R left over. I think in this case for c99 it does mean what you want but there's an inconsistency in C in that `uint64_t get_time();` as a prototype in a header would instead mean a function that can take an unspecified number of arguments - although the syntax is considered obsolete. Given that C legacy `uint64_t get_time(void)` is clearer I find. (For reference C is inconsistent with C++ here)


for #57


On Tue, Jan 12, 2016 at 02:04:17PM -0800, Serge Hallyn wrote:
> I'm probably making things up - but could you try doing a nested ls where a nested container is i386 and the parent is x86-64?
Will do.


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] Assignment 1 contains directory of files for user (0.5 pt)
- [ ] Repository contains screenshot of [github][] account (0.5 pt)
- [ ] Repository contains screenshot of [Slack][] session (0.5 pt)
- [ ] Repository contains screenshot of [Atom][] installed on users computer (0.5 pt)
- [ ] Repository contains screenshot of screenshot showing that user installed [linter-flake8][] (0.5 pt)
- [ ] Repository contains screenshot of the Anaconda [REPL][] on user's computer (0.5 pt)
- [ ] Repository contains screenshot, in the repository, displayed by Github Desktop (0.5 pt)

## jupyter notebook

- [ ] Repository contains ipython/jupyter mini-tutorial as ipynb file (0.5 pt)
- [ ] ipynb Task 1 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 2 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 3 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 4 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 5 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 6 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 7 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 8 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 1 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 2 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 3 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 4 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 5 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 6 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 7 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 8 appears to be answered correctly (0.5 pt)
- [ ] You can successfully re-run every step of the ipynb notebook (3.0 pt)

[screenshot]: https://en.wikipedia.org/wiki/Screenshot
[git]: https://git-scm.com/
[github]: https://github.com
[Slack]: https://biolprogramming.slack.com
[Atom]: https://atom.io/
[linter-flake8]: https://atom.io/packages/linter-flake8
[syllabus]: https://github.com/biolprogramming/syllabus
[ipython]: http://ipython.org/notebook.html
[jupyter]: http://jupyter.org/
[REPL]: https://en.wikipedia.org/wiki/Read%E2%80%93eval%E2%80%93print_loop


RE:  05b_python

cc: @julialintern @aramnath @gfsoileau @Vinoad @galvin-mj @debbiebere
cc: @reshamametis

--

Hi @nisharma85

Really nice job in completing these **Python (Adv) Part II** exercises! :+1:

Thanks for formatting explanations and results in an easily readable manner.  It is greatly  appreciated and makes my life easier.

----

Some optional extras if you have the time:
a.  [Explore 'defaultdict' option in python](http://stackoverflow.com/questions/5900578/how-does-collections-defaultdict-work)

defaultdict means that if an key is not found in the dictionary that instead of a KeyError being thrown a new value is created. The type of this new pair is given by the argument of defaultdict.

b.  Explore pprint in python
Prints output in a more readable and structured format.


Isn't that an oversight? :p


One Frost is the operations center for Frost Bank. This work is part of a large collection on 4 floors of their new building.

Time is Money is an analog timekeeping device displaying hours, minutes and seconds. The view infers the time from the relationship of the "hands" to unmoving index marks, just like on a standard round clock. Hours on the left. Minutes on the right. Seconds and index marks occupy the middle 3 stacks.


extra whitespace.


Hello. The most recent commit of the license file replaces all of the MIT license by the two two lines containing the year and copyright holder. I'm sure this is a typo, but can you please confirm that reshape is still available under the MIT license. Thanks! Devan


Yup, but only the functional tests will fail on the CI.


variables shouldn't start with caps.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11353](https://teamcity.mesosphere.io/viewLog.html?buildId=103724&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:50


I'll just rename Debug to Dry-run. Done.


Maybe avoid the unnecessary call by moving the GetPadding call into the conditional block.


Nit, but would recommend:

for (i = 0, n = m; i < len; i++, n++) {
	if (ls_serialize(...))
		goto out;
}


This is unfortunately not a typo but is instead how CRAN currently want MIT licenses. The license has not changed.


The space here? @mixcas


Workaround_Thread_Local_MinGW (lefticus) - Win64-Windows-7-VisualStudio-12: OK (382 of 383 tests passed, 0 test warnings)

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


These extension methods are very specific to the Views/Text/ReactTextInputManager.cs, can you put this class there, and update the namespace?


But uglify does not evaluate that expression. From the point of a javascript minifier, mangler or code beautifier there is no issue.


I would probably add the retry count to the constructor and then also make sure you have a sleep between retries.


Yep


should not be called like Layout.prototype.StartLiveUpdate(); ?


RGBA values should be just 1 integer, not 4. The functions ModAPI_ColorToInt and ModAPI_IntToColor are here to convert them from both format.


This is how the tag comment looks like.


There is still the scope analys


There are other places in this file where this change also needs to be made.


Missing comment.


What do the {0} and {1} indicate?


On Tue, Jan 12, 2016 at 11:22:29PM +0100, Christian Brauner wrote:
> On Tue, Jan 12, 2016 at 02:04:17PM -0800, Serge Hallyn wrote:
> > I'm probably making things up - but could you try doing a nested ls where a nested container is i386 and the parent is x86-64?
> Will do.

Testcase:
        Parent: x86_64
        \child: x86_64
         \grand-child: i386

Works without a problem for me. (I assume you're interested in the case where
they are all running.)


Don't worry, get the code working first, these details can be taken care of later. Just try to stick to clean interface and try to hide implementation details.


ah yep, a little less pedantic here; this doesn't have the type safety you think it has. Empty parenthesis here don't mean this function takes zero arguments, rather that the function takes an unspecified number of arguments. `uint64_t get_time(void);` explicitly tells the compiler the function takes zero arguments so you'd get a warning if you mistakenly tried to pass an argument.


Its so that plugins could check as well without using our double underscore (intended to be hidden) variables.


I have mixed feelings about this. I know it's bad to have multiple expectations in the same unit test, but do you think there's enough benefit from breaking these out as separate tests to justify running the same (db-reliant) test 4 times? The more I think about it, the more I regret creating this model test, actually. I already had full coverage with the request spec. I created a model spec b/c I thought I should have a test for every public method, but now I can't imagine a scenario in which these extra tests provide real benefit.

Also, upcase has made me want to avoid `let`, `subject`, `before` and instance variables in tests wherever possible. Part of the refactoring I wanted to do was to go back and find my old `let` and `before` blocks and remove as many as I could. I guess what's more important is that we follow the same style -- do you prefer using subject, etc?


hmmm, so this would only work on the shield atv or similar devices? the id_s were only being used for these weird usecases, the standard codepath was a lot more straightforward


I meant *almost* because you backed it out before I woke up. I'm lumbered with a Windows box at work, so anything make related is non-joyful to use! I just did a git clone from the last commit before you went back to make (8e00435505a66f6e0a14e0a1eae296d3be1a5b3f).

Only issues I could *immediately* see was sinon was missing from packages.json so had to npm install it. Other than that, I thought it was pretty solid.


Nitpick: Should be "action-oriented screens or dialogs".


The api described on the link below would not return the correct and expected result:
http://lisperator.net/uglifyjs/scope


maybe the input header format could be a template which defines where the year(s) is/are, meaning we wouldn't need a generic regex.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11354](https://teamcity.mesosphere.io/viewLog.html?buildId=103730&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:09:13


pop-out ~out~


Would be nice to get the standard indentation on the wrapped line.


What do you mean by "std::string (as a tag)"? like `std::string{}`??


Hm, no I don't think so. idea-community itself is free but has a non-free dependency


lalala


To hide global variables, add static before the definition: eg: static int g_iX = 5; //Only the file that has this in it can see g_iX.


lmao


small typo, that should be "Female Hair" right?


Just a note that after discussions with Courtney we decided to keep the checkbox so we need to keep `clearParameter` and need to use `showWithCheckbox()` below


premefeed week


Thanks :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11356](https://teamcity.mesosphere.io/viewLog.html?buildId=103742&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:50


No it should work on all Android units, it's just using the same principle for keeping track of the id of the keyboard.


Ok, I see what you're getting at - at the API level only, but as a JS to JS transpiler the output result is not discernable.


No, I mean the `std::string` type when seen as a Hana tag, rather than the `std::string` __C++ type__ when seen as a __C++ type__. The difference is only in viewpoint.


@a-detiste: is this just because the Doom 2 Master Levels `.desktop` file assumes this path for all distros, or is it a genuinely Arch-specific oddity, like Arch policy being to use `/usr/share/games/doom` despite putting other games' assets directly in `/usr/share/$game`?

(I ask because I'm going to add variable substitution for the desktop files, which would make the former reason go away.)


Workaround_Thread_Local_MinGW (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3041 of 3041 tests passed, 0 test warnings)

![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)


Why have you deleted this method ?


Please don't create these entry points


Is this just a corner case bug or is it really a show-stopper? Does this affect all jetbrains products?


Could you declare all `vars` outside of the condition. I normally declare all vars either at the beginning (line 150) or sometimes, if I define a "fast return" like in 151, below this line (152).


The overly complicated paragraph above is not saying anything insightful. You can just ignore it, lol.


that's actually not true. If sig is None, it means that it didn't exit because of a signal. @nicolaskruchten


Toujours le meme crash


Sounds good.  I noticed this a few times too.  Will this actually run the function as well, as is performed when using `lamda`? I am used to defining `lambda`'s in `add-hook`, but not `defun`'s


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11355](https://teamcity.mesosphere.io/viewLog.html?buildId=103741&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:08:55


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1502 of 1502 tests passed, 0 test warnings)

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


That was fixed in previous commit


It will eventually creep into all of them since they use the same base.


com**m**anding surface


This will need to be a generator because multiple sites will be used without resetting the references. I'd suggest making one standard parser, and only returning custom parsers if they have site specific settings.


Fabric**'**s


There is a BC break only when it existed in a previous tagged version. This is `master`.


these tests did not create their own sims before. they just assumed their existence. i merely made that assumption explicit in the docs


A **component** for


Bazowa wersja strony


It might still influence some transformations while compressing, but anyway, feel free survive breaking tests might one be in these circumstances :-)


place**d**


Why do you use `scrollWidth` and not `offsetWidth` or `clientWidth`? This might be interesting for some bgset features, but to me it seems more "correct" to use `clientWidth`. Do you have a special use case, that forces you to do this?


Respect for updating this - I suppose you got it to run? Anyway, this spec is really overcomplicating things, a lot. To expect the exact SQL that is created by a scope is not a useful spec. I suggest to just remove this altogether. If you are worried about the functionality of a scope and really want to test it, I suggest writing a different spec that creates a few records in the database, and then call the scope to see if it returns the right records. But I think we are already better off when this spec here is deleted.


The license is still asl and, as already seen, not changing the dependency just leads to confusion/noise.


indent these lines in both try / catch block


minor nit, this could fail with a perfectly valid implementation of hashcode if for some reason we encounter a collision, so I'm not sure exactly what we're testing for here, other than that the hashcode is not constant.


Descriptions all make sense to me, just noticed a few minor typos.


can you open a PR again to take a look?


I didn't use the term "unorthodox" without reason. =)

There's the theory, there's the practice and then there is human reality. The theory is that you shouldn't do it. The practice is that it is untestable. In reality we are humans and if you tell me that to facilitate the review process you'd like to introduce some classes as the first step of a bigger set of changes (which can be seen elsewhere if needed), I wouldn't mind.

If that makes you uncomfortable however, think about using the commits to provide structure instead. The first commit would introduce the top-level classes/interfaces and then the second commit would introduce users of those classes. Something along these lines can be seen [here](https://github.com/MovingBlocks/Terasology/pull/1459). The PR would remain atomic, and if commit-level atomicity is required, when everybody is happy we could squash the commit into one and merge that instead.


I'm thinking: there will be further Windows Kits versions, and it seems a bit annoying to add a line for each new version.

Normally, one would run CMake in a Visual Studio development console - that is, a console started via script provided in the Visual Studio Tools folder in the start menu. That script adds stuff to the PATH environment variable, which should allow discovery of DirectSound without this commit.

This is already explained in the Marsyas Manual:
http://marsyas.info/doc/manual/marsyas-user/Building-latest-Marsyas-on-Windows.html#Building-latest-Marsyas-on-Windows

Can you please try if this works for you?


Looks good.

Minor note: If you're not passing a query, you can do Model.find() instead of Model.find({})


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11360](https://teamcity.mesosphere.io/viewLog.html?buildId=103755&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:20


This is OK as long as no key inside composer.json ever contains a backslash.


Already did :)


See http://trac.cppcheck.net/ticket/7283


Hi, I wouldn't mind you removed that Arch tweak altogether; I only have some artificial experience of this distro and this is maybe a mistake of packager not following their own policy.


just a note here, `wsu_actions` is a var already and such


In these places should you be going to next_no_put, rather exiting the loop entirely?


Let's keep the discussion in #2826. I'm going to do some more testing around this and post back there.


Now `API_BASE` is unused, right? So we can remove it entirely?


Could we rearrange the `static const` keywords to `const static` on this `ACTION_CONNECTION_FAILED_MSG` for consistency as well?


hmm, c'est ce changement qui fait apparaitre le model 3d sur bleach, mais pas les fille de fer


Please fix the typo here


Let's not add line breaks if we don't need them :)


Yes, that is a weakness of the compiler since it started.

As you know, in Lisp everything is an expression. But it is not the case in JS.  The compiler just convers Lisp expressions into JS recursively,  so converting every Lisp expression in a JS expression ensured it composes easily.  But it introduces too many nested functions.

A practical example:

you want to compile `(car x)`. That means you have to emit no only the "v1.car" expression, but also the error checking. So we generate:

```javascript
(function(){
    if (x is not a cons) throw "error";
   return x.car;
})();
```

Now I am experimenting what it is like if I generate statements always, a completely flat JS, with variables for every intermediate value.  But in order to make it incrementally, I have to introduce some scaffolding so I can use both styles at the same time.

There would be another benefit.  The code with the flat-style is much more similar to the kind of code you use to generate IR, that would be the next step.


Mind if I ask why you did this? Just looking through the code, thought I'd ask.


`arguments` being in the wrong scope won't break any compress transformations in the current code base. The default compress transforms only work with AST_Constant folding. The "unsafe" option does some additional stuff on arrays and join and strings that I wouldn't recommend using. But no compress or mangle options optimize arguments in any way.


基本的にはうちの開発者（新規参入してGitの使い方がわからない人）向けに、と思っていますが、もっと広い範囲で役立てていただけたらな、と思っています！
どういった言葉を足せば解りやすいでしょうか…？ :sob:


For Mageia, according to Akien told me the paths are in theory the same is Debian; but there are always a few glitches (ioquake3 is in /usr/share/ioquake3 there)
https://people.mageia.org/u/akien.html

Similarly we have /usr/share/vcmi on Debian


Technically this cast isn't safe, since the exception could be a raw `Throwable` or subclass other than `Error` or `Exception`. I'll change this to `Throwables.propagate()`.


I'm usually don't be that strict: `elemW === 0` becomes `!elemW`. You can keep this, if you want, but in this line `elemW !== 0 && elemH !== 0`, I think `elemW && elemH` is always to prefer. (Refactoring can make those `undefined` for example.)


As it turns out, it wasn't chosen by VS -- it was just copied and NuGet didn't remove it. Turns out these aren't needed for Net45. I've deleted them here and updated the app.configs elsewhere.


I misunderstood. Sure. +1


So was it not working when loadObjectData is executed immediately? It's not obvious to me why we want to defer execution until all the initialization is completed, since much of the initialization depends on the objectData and so lots of stuff must be run twice if we load the objectData later.

That said, if we do want to defer execution, it might be cleaner/more canonical to use a Diva event. Maybe it would make sense if we move `ViewerDidLoad` to fire when all initialization is complete rather than just upon loading the metadata, and then bind to that. Or fire an `InitializationComplete` event at the end of `init()` and change the name of `ViewerDidLoad`.

This didn't occur to me before merging but I suppose we can just make any modifications in `develop`.


You probably wanted public inheritance, so you're missing "public" keyword. And name of the first parameter in constructor below should not have trailing underscore.


At least, it's now future proof :-)


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - Win64-Windows-7-VisualStudio-12: OK (2050 of 2051 tests passed, 0 test warnings)

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


Ah, [here we go](https://github.com/CenterForOpenScience/osf.io/commit/1cbbdd357e2a709aaa7ceab06bad56c43c74508c#diff-4b7e48e859dd52cd5cd245a4e3bec9e8L15). :-)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11362](https://teamcity.mesosphere.io/viewLog.html?buildId=103759&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:11


It doesn't have an affect on predicted boxes. It accurately picks up people on the top half of the image, but still doesn't find those at the bottom half.


Why arren't you using python [argparse module](https://docs.python.org/2.7/library/argparse.html#module-argparse) for that kind of thing?


Instead of passing a new kind of ready callback here I think we should instead change the type of the callback so the same type can be used in either case.

This can be changed to `void (*ready_cb)(struct gputop_perf_stream *),`

Then we can add a corresponding member to struct gputop_perf_stream:
```c
struct gputop_stream {
    ...
    void (*ready_cb)(struct gputop_perf_stream *);
    ...
};
```

Below when initializing the newly allocated stream, save the callback pointer:
```
stream->ready_cb = ready_cb;
```
When calling uv_poll(), unconditionally pass `perf_ready_cb` or when calling uv_timer_init, unconditionally pass `perf_fake_ready_cb`

Modify perf_fake_ready_cb() to be like:
```c
static void
perf_fake_ready_cb(uv_timer_t *poll)
{
    struct gputop_perf_stream *stream = poll->data;

    if (stream->ready_cb)
        stream->ready_cb(stream);
}
```
(and the same kind of change for `perf_ready_cb()`)

Now the two places in gputop-perf.c that currently call `gputop_open_i915_perf_oa_query()` can be made to pass gputop_perf_read_samples as the ready_cb argument. The call in gputop-server.c can currently pass `NULL` since the callback isn't used for anything there.


Toujours pareil pour RR7


I think you got this backwards. `monClim': 'mon'` should be `'mon': 'monClim'`, now?


>  is this just because the Doom 2 Master Levels .desktop
>> TryExec=/usr/share/games/doom/doom2-masterlevels-tryexec

Whoops, I didn't even think of that


Hi Libby,

Thanks a lot for your replies.  I’m still determined to get it working so I’ll keep trying whenever I get the time.  Ok so not worried about the wpa/samba issues.

I think there may be some kind of issue with Supervisor or one of the apps it’s managing as I mentioned before.  I tried several different URLs and get the same message.  In fact putting any URL in there for me generates the same error:

Moved Temporarily. Redirecting to /

If I understand correctly supervisor is managing those 4 apps listed in the config to make the piab work.  Is the one that processes the URL the “offline-piab app”

As this is being run by node I’m guessing it could be something to do with node install or the version of node but I’m new to this so learning as I go.

If you come up against the same issue please let me know.  In the meantime I’ll first try re-installing node in the directories you mentioned for a pi2.

That being said, I might go and attach the NFC reader I have (this is the one you originally used so let me know if you think that will be an issue as the last set of instructions I saw uses a slightly different reader).

When I go to radiodan.local and put in a URL without the nfc reader attached it does seem to correctly find the streams.

Thanks again for all your help,

Tristan

> On 5 Jan 2016, at 12:38, libbymiller <notifications@github.com> wrote:
>
> hi Tristan,
>
> I wouldn’t worry about the samba. As you’ve seen, it’s fixable and won’t effect anything else.
>
> The wpa stuff; no, that’s it, so you could just go in and edit wpa_supplicant config manually and do:
>
> /etc/wpa_supplicant/wpa_supplicant.conf
>
> network={
> ssid="XXX"
> psk="XXX"
> }
>
> The server at localhost 5000 is run by supervisor:
>
> https://github.com/libbymiller/radiodan-client-podcast/blob/master/radiodan-type-piab.conf
>
> I really need to update all the instructions, sorry...
>
> Libby
>
>
> > On 2 Jan 2016, at 01:27, straysontour <notifications@github.com> wrote:
> >
> > Hi Libby,
> >
> > Thanks for your reply. I'm using a Pi B and B+ so hopefully don't have all of those node issues although I'll look into that.
> >
> > I'm trying the kermode feed at the moment though earlier when I changed the feed url it didn't seem to alter the message I was getting but I will try some more to see.
> >
> > A couple of questions if they are easy for you to answer. Does the WPA element of the provision do more than just setup the standalone access point functionality?
> >
> > Is the samba install fail I was talking about likely to affect anything to do with the podcast player? The web server element of samba seems to work just fine in that the pi shows up on the network as an smb share.
> >
> > Finally, the localhost at port 5000 which takes the rssFromNFC and the stopFromNFC commands. Which component is running that or is it a combination? I haven't looked deeper but was guessing it was nginx or mopidy.
> >
> > Thanks again for any time you have.
> >
> > All the best,
> >
> > Tristan.
> >
> >
> >
> >
> > Sent from my iPhone
> >
> > > On 2 Jan 2016, at 01:08, libbymiller <notifications@github.com> wrote:
> > >
> > > Sorry, I’ve really not looked at the code for a good while. I want to get it working again but I’m not sure when I’ll have time. The underlying radiodan code currently has problems on Jessie and Wheezy, and in particular the provision fails on a Pi2 - you have to go in and manually recompile all the node applications.
> > >
> > > Which RSS feed are you using?
> > >
> > > Libby
> > >
> > > > On 2 Jan 2016, at 00:59, straysontour <notifications@github.com> wrote:
> > > >
> > > > Hi, just wondering if you have got your piab box working with jessie?
> > > >
> > > > I have been trying to get the piab software running on both wheezy (as you saw elsewhere) and jessie to see which would work first. Again this is all on a Pi B and a B+
> > > >
> > > > With jessie I seem to be able to move through the radioman provision install pretty well apart from the fail with samba - as I mentioned I bypassed/resolved by commenting out the "restart service samba".
> > > >
> > > > The next slight issue I encountered was:
> > > >
> > > > sudo cp radiodan-type-offline-piab.conf /etc/supervisor/available/
> > > > sudo radiodan-device-type radiodan-type-offline-piab.conf
> > > >
> > > > I assumed it was ok to rename the file: radiodan-type-piab.conf contained in /opt/radiodan/apps/radiodan-podcast-client to the filename included in the instructions and proceed from there.
> > > >
> > > > When I try and use curl to test the rss feed.
> > > >
> > > > I'm getting the message below:
> > > >
> > > > "Moved Temporarily. Redirecting to /"
> > > >
> > > > Any help very welcome, would really like to get this working on either jessie or wheezy as it's a great project.
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub.
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/libbymiller/radiodan-client-podcast/commit/8cfeee1b58e0f110a5b34ceb6b15c1517cf0c521#commitcomment-15264041>.
>


if ls_get_cgroup_item() return NULL, strstr will segv here.


Workaround_Thread_Local_MinGW (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (383 of 383 tests passed, 0 test warnings)

![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)


Ugh. Yup.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11361](https://teamcity.mesosphere.io/viewLog.html?buildId=103756&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:08:42


The correct annotation is `javax.annotation.Nonnull`. `@NotNull` is a Bean Validation constraint. But I'll remove this since we always assume not null and instead use `@Nullable` for the rare places that need it (even more rare in Java 8 due to `Optional` being in the JDK).


Please use /*  */ for multi-line comments.


This change makes it very difficult to integrate this change into spring-cloud-netflix. Since it's not a `Feign.Builder` I would have to duplicate all the logic to setup each builder. Maybe there could be an interface that get's implemented?


Is this one not a show stopper type failure?


OK, thanks.  I spent at least an hour trying to find what the ssh-proxy client needed to look like so I'm glad it's somewhere but it's certainly hard to find.  :-(


Could you talk a little bit about the webpack overrides you are doing? Why are they necessary, why doesn't the current webpack code work? Thanks!


ahaha, http://lea.verou.me/2012/01/why-tabs-are-clearly-superior/


I just thought that the order of entries was actually reversed (chronologically), so it should be  called "true".

There would be no problem if there was an option to force the book to open at last page. In such case players could look at most recent entries quickly and they would go "back" in the right direction (switching pages from left to right). But that's not possible without mods.


pointer-events is only supported from IE11. Aren't we going to have a solution for at least IE10?


`[applied]]` instead of `[applied]`


I think it is better to name these new properties identical to the ${version ...} variants.

``` xml
 <version.org.hawkular.bus>${version.org.hawkular.bus}</version.org.hawkular.bus>
```


OK, it seems like free programs with nonfree dependencies suit the Apache2.0 requirements and are treated as free software. @edwtjo: what do you think about adding idea14 as standalone set element of this expression?


This was stupid copy-paste on my part. I've changed the targetFramework to Net45 and removed the Microsoft.* references that are no longer needed.


Hi Martin,

Should we also change the description here?


Fair enough.


Is bundling glib really an issue?


might also help to give an example here in the prompt:

`Enter a real name for this key (for example: "Ned Snowben")`


primer commit


No, you should not commit stuff with merge conflicts (>>>). The order is the following
1) git merge master
If there merge conflicts:
1.1) resolve conflicts in your editor by keeping only the needed parts and deleting the obsolete ones
1.2) git add FILES WHICH HAVE BEEN FIXED
1.3) git merge (it commits and weite the appropriate message)


Workaround_Thread_Local_MinGW (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (383 of 383 tests passed, 0 test warnings)

![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)


I'm assuming these came from WaybackMachine?


Workaround_Thread_Local_MinGW (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: Build Failed

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


the `libkb.CheckUsername` checker won't work since it doesn't allow spaces.


Good call on adding these instructions.


Yeah, I'd since changed it to `@Nonnull` in Platform.


ok, fixed.


It sure is. I would prefer to use an ASSERT here, but that has been frowned on. I can ROS_ERROR and return if you'd like.


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


But why have u deleted the new data?


Why do you have to specify OSGroup for netcore50 targetgroup? and why not for net46


@smcv

The Arch port is just at a proof-of-concept stage, don't mind breaking it;
I was currious to see how I could make this `makepkg` thing work
(it's just a huge shekk scrip); but in the end I just ripped it out and kept the bare minimum.
Getting the tarball with the right paths was a huge pain; had to recompile
the package manager to add debug messages.

It's like replacing dpkg-deb by "ar + tar + (compressor)", something we shouldn't do
I was tempted to do it to replace xz by pixz; but that's not worth it &
parrallel xz is a popular request that would be handled some day properly by dpkg.


also, one last issue, can you refuse to accept duplicated emails?  If the email is repeated, just log an error and don't append it to the list (but don't error out)


5156-HeatExchanger (Nigusse) - x86_64-MacOS-10.9-clang: OK (2048 of 2048 tests passed, 0 test warnings)

![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)


UpdateLightFractionTableInDocs (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2082 of 2082 tests passed, 0 test warnings)

![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)