For what it's worth, I'm not strongly in favour of one position or the other on this (I slightly favour using aliases, but not enough to worry me greatly). However, my motivation here was to try and keep the test cases looking as consistent as possible from one file to another within meas_base -- the subsequent commits add a bunch of new tests which do similar things for different algorithms (testing that Psf, Gaussian, etc etc fluxes all get correctly turned into magnitudes), and I wanted all those tests to be consistent with each other. That's hard, because existing meas_base tests aren't very uniform.


@psvmcc спасибо, сделал


Yeah, last time I wanted to refactor it out I realised it'd be fiddly
because of the tests included in cleanup.  Perhaps revisit that later?

Hakim
--
*Developer*, mySociety Services
http://www.mysociety.co.uk

On 12 March 2015 at 15:33, Matthew Somerville <notifications@github.com>
wrote:

> Ah, I was just matching some other files that had it like that. It looks
> like a couple of cleanup functions include their own tests, so I'm not sure
> what would be the best way to do here; get rid of the ENDs entirely, as
> the database is now dropped between runs? Though you'd still have the issue
> when doing a full test run of something going wrong potentially poisoning
> all future tests in that run and confusing the issue.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mysociety/fixmystreet/commit/e0780120c86e070c03c4e1521f48afd1d448f7b6#commitcomment-10166854>
> .
>


Don't mix independent changes in single GIT commit.


TeamCity xBIM :: XbimExchange :: Build [Build 118](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1152&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:07:06


Should you change `module_NameRef` to `object_NameRef` too?


Ah, ok, you mean it's better to allow rather than disallow when we don't know. Well, that sounds fine to me.


nice


I guess what I was trying to describe here was the mechanism whereby surveys are tracked by survey  ID -- or the purpose of the survey ID.  Frankly, using a callback mechanism, one could invent a much nicer high level API for surveys that allowed multiple outstanding surveys to be in flight at a time.   That would require a different API, but could use the same wire protocol.  This document describes the wire protocol, not the API.


There should probably be a period at the end of this sentence.


What was the problem with `lib`?


It'll probably be too late to help this issue, but they'll be getting a lot more uniform on DM-1161.


 @dkogias mind pushing a pull req. upstream? Some notes:

* #check_install method is not required anymore - we flattened the environment variables that control processing lifetime down to just RUBY_GC_TUNE=200 (for 200 units of work). It's all handled in the base spy now

* ditto for #initialize


No, autoproj asks that after downloading it itself. In order to download the autoproj bootstrap (the change I made) you need to do it through http in git-denied environments.


so `save()` was not triggering `createTuple`?


I'm thinking we need to re-do some of the config examples, as these will cause configs to reset each load/reload I believe.


@abeloin limit is 1024


Why do we need a variable here?


So, what happens after the while loop? If you put in a print, does it run?
What are you doing differently after from within that changes what happens?

On Thu, Mar 12, 2015 at 12:11 PM, elijah9 <notifications@github.com> wrote:

> I just tried pressing a bunch of keys when the game is over, nothing
> happens. The code only calls the draw function inside the while loop, which
> should stop at this point.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/elijah9/CMPU-203/commit/fa9068f826ca25e74246f3346fbececfd0f7955e#commitcomment-10167848>
> .
>

--
Jay McCarthy
http://jeapostrophe.github.io

           "Wherefore, be not weary in well-doing,
      for ye are laying the foundation of a great work.
And out of small things proceedeth that which is great."
                          - D&C 64:33


Oops, good point.

Also, should this be -01 or -02?  I'm not sure what the convention is for bumping the numbers.


SGTM


Actually, it can be an instance as ``MPLPlotRenderer`` is a parameterized function. I am happy renaming it if you think ``Store.renderer`` is less confusing...


I'd probably stick to using "pluginname" as the filename here. Thoughts?


If we have a piTypePicker, it should always have a destroy() method right?


@dmitryevseev that are the right tabs?


to be clear, this would mean that individual modules cannot be released independently.


You should give the computer some tolerance when using testing for equality with floats. Something to note when your program doesn't work properly.

Generally, to test for equality between x and y it's in the form of `abs(x - y) < tolerance` where the tolerance should be at least greater than [epsilon](http://en.wikipedia.org/wiki/Machine_epsilon).

In unity floating point equality tests can be made with:
```c#
bool Mathf.Approximately(float, float)
```


Uhh, no. We use 2 spaces as a tab symbol)


"multi-flight" is sometimes used on the mailing list to describe multiple concurrent REQ or REP transactions as yet unfulfilled by a socket. Currently, if for instance two surveys are issued before receiving responses from the first, on sending the second the first is immediately cancelled.

By the way, it feels like you later describe this at: https://github.com/gdamore/nanomsg/commit/177d1214cd4dc183fa1009b6155af7c7fbc31d71#diff-e6e05e155afabffdf8436262f60bd05dR139


if you have the energy add the reference to the improvement JIRA you created.


The only code after the while loop is clearing the screen and displaying "GAME OVER" and "SCORE: X" on consecutive lines. It used to work, so I don't think the problem is with the code after the while loop. I'm not sure what you mean by the second two questions.


yeah that wasn't looking good, ok correcting..


How about?

```
diff --git a/pyanaconda/ui/gui/spokes/custom.py b/pyanaconda/ui/gui/spokes/custom.py
index 625cf0a..c76f40f 100644
--- a/pyanaconda/ui/gui/spokes/custom.py
+++ b/pyanaconda/ui/gui/spokes/custom.py
@@ -2405,6 +2405,8 @@ class CustomPartitioningSpoke(NormalSpoke, StorageChecker):
             return
         log.debug("fs type changed: %s", new_type)
         fmt = getFormat(new_type)
+        if fmt.label:
+            self._labelEntry.set_text(fmt.label)
         fancy_set_sensitive(self._mountPointEntry, fmt.mountable)

     def _populate_container(self, device=None):
```


Thank you for this update, I'm glad to help!


Hello, what's your use case for this? Would you like this into https://github.com/lazywithclass/winston-cloudwatch ?


size_t i = 0;


marina's test


I'd use predicate conditionals only when they fit on one line (for readability)


Probably a good idea.


Hi,
What is the problem?
I just tested on v3.19, 3.19.1 and 4.0-rc3 and it works.
Is in compilation?


Yup.


Yep, not sure why `@state` is necessary, rather than querying the TocStore directly.


Check for null before calling a delegate to avoid `NullReferenceException`s.

```c#
if(OnPlayerStatChange != null) {
    // Call delegate
}
````


Could this be written inline above as:
renderTo: !this.config.piTypePickerConfig.renderInGridHeader && Ext.query('#content .titlebar .dashboard-timebox-container')[0]

...unless you think that's too fugly


what is the problem with spock-report?


This is unused.


Ah, I think that pushes it towards being more attribute-like, so yes, I now vote more strongly for Store.renderer.


cache_line_size dos not refer to the amount of L1 and L2 cache but the cache line size.
Take a look to lines 80 to 82: http://pastebin.com/stM6Wfgz


This is unused as well.  Before landing, can you kill all warnings in this package?


The lua example was already there so i just used it and changed the syntax for js.
Would probably be better to use the self.Config... = self.Config... or "string" style


Yeah, that's what I normally use, I just tried to keep it simple for the example. It'd need to be changed for all.


@devs: when will you bake me a cake!?


Isnt that suppose to be user_id?


OK, I'm on it!


Updated README, to note that the scripts now use the "Hmisc" package instead of the tempermental "big vis" package.


so I'm fine with adding different front-ends to this functionality, but I think that both the code in `impl [T]` and this test-only code should call to a common helper. Otherwise it seems *very* easy for these to get out of sync, no?


user_id is already defined for this table. I don't know why it's showing up


Just a style thing.  Typically we put the constructor as the top-most method in the class declaration (after cls in this case).


check out [Block Regular Expressions](http://coffeescript.org/#regexes), it's got some extra sugar for multi-line regexes.


Right.  I'd like to understand whether in light of this, the paragraph above should be removed.  My intent was to convey the need for the survey ID; that's true no matter what.  If the paragraph is misleading or causes confusion with multiple surveys, then I'd like to change it to be less confusing.  I welcome suggestions!

Meanwhile, I believe libnanomsg only drops surveys if not using AF_SP_RAW,.  With AF_SP_RAW, you get to have more than one survey at a time outstanding, but matching responses to surveys (and applying any timeout if you want one) becomes the application's problem.  AF_SP is the upper "convenience layer" but it comes at a cost (in this case only one survey at a time outstanding.)


my ASCII art skills are improving


I thought I would try github's UI for editing .md files with this patch, but it turns out just using the normal process outlined in the contributors doc is way faster. Sorry for the noise.


If upstream ends up being responsive about those changes feel free to propose them.


VSHPWH (Myoldmopar) - i386-Windows-7-VisualStudio-12: Build Failed

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


You may want to mention somwhere that http://sourceforge.net/projects/pywin32/ is now required in the guest otherwise you get tracebacks in process.py trying to import win32file


ohhh weird


r+


How about the integration with Travis-CI?


Hello, I was attempting to deploy to AWS OpsWorks and in that particular setup, the AccesKey and SecretKey were being passed down by the AMI but not the awsRegion. When attempting to manually set the region, I was receiving an error about the region not being set. It seems that due to that line, it was ignoring the region I was setting, thus the change.

My change was just quick in order to get the app launched, not sure how typical the case is or if there's a cleaner way. Thanks.


VSHPWH (Myoldmopar) - Win64-Windows-7-VisualStudio-12: Build Failed

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


just add "set -e" to the top of the script?


@mikepetersonccv +1!  We were running into errors getting the RestSharp serializer to accept IEntity/IModel.  I think this should help fix that.


http://192.168.30.225:8111/viewLog.html?buildId=922&buildTypeId=Ybackend_SyntenyYbackend&tab=buildLog#_focus=6328

```
[14:52:09][ru.biocad.ybackend:api] ru.biocad.ybackend.api.businesslogic.toolrun.AnnotatorRunnerTest
[14:52:09][ru.biocad.ybackend.api.businesslogic.toolrun.AnnotatorRunnerTest] AnnotatorRunner should work with aminoacid files
[14:52:09][AnnotatorRunner should work with aminoacid files] class java.io.IOException: mark/reset not supported
[14:52:09]
[AnnotatorRunner should work with aminoacid files] java.io.IOException: mark/reset not supported
      at java.io.InputStream.reset(InputStream.java:347)
      at ru.biocad.run.yannotator.AnnotatorJob$class.run(AnnotatorJob.scala:67)
      at ru.biocad.ybackend.api.businesslogic.toolrun.AnnotatorRunner$$anonfun$run$1$$anon$1.run(AnnotatorRunner.scala:28)
      at ru.biocad.ybackend.api.businesslogic.toolrun.AnnotatorRunner$$anonfun$run$1.apply(AnnotatorRunner.scala:28)
      at ru.biocad.ybackend.api.businesslogic.toolrun.AnnotatorRunner$$anonfun$run$1.apply(AnnotatorRunner.scala:19)
      at scala.concurrent.impl.Future$PromiseCompletingRunnable.liftedTree1$1(Future.scala:24)
      at scala.concurrent.impl.Future$PromiseCompletingRunnable.run(Future.scala:24)
      at scala.concurrent.impl.ExecutionContextImpl$AdaptedForkJoinTask.exec(ExecutionContextImpl.scala:121)
      at scala.concurrent.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260)
      at scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1339)
      at scala.concurrent.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979)
      at scala.concurrent.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107)
```


Replace by size_t i = 0;
NB this comment is also available for the next for.


Indeed.  I'm having a hard time keeping all the TODO and future tracking straight :/

In this situation we should mark the incoming GUID as "ensure downloaded in future" because on the next fetch it should be reconciled with the existing uploaded GUID by the normal pathway.  Right?


@synteny кажется, @biocad-teamcity не может комментировать этот проект. Ты добавил его?


awesome!


bonjour bonjour


Okay, but don't change the code @Seitus let me know of the issue, so that I can fix it. I would appreciate if you can share the screenshots when you face anything unwanted/wrong.


Good point. It should have been changed.
I will check that right now.


We track `failures` but don't fail the batch using them?  Is something else inspecting `failures` (it is public).  /me reads


why does size need to be 0?
Can't a dataset that has children specs also have (correspond to) a table?


VSHPWH (Myoldmopar) - x86_64-MacOS-10.9-clang: Build Failed

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


I haven't tried it yet, but I think you can make RestSharp use Newtonsoft.Json as it's serializer


Oh I see. Just looked at the diff and saw the introduction, but I guess it was moved earlier in between release.


No, I see no external consumers of `failures` or `failed`.  I guess we're assuming that status change uploads can fail gracefully?  I agree with that design decision (going back to your material/immaterial document, I suppose) but I don't think we should hide unexpected failures.


you can run
```sh
git rm --cached obj/
git commit
```
to remove that folder and its content from the repo without deleting the file on your computer. (I assume that's what you're trying to do :P)


I'm not sure I understand this. The menu-open class gets added to the body so this doesn't actually do anything. Also, there is already a display: block set for ul.menu when it's open.


I think this logic on line 202 is wrong.

It should be:
```
if (isParent(tableName, specEntry.getValue())) {
  return true;
} else {
  continue;
}
```
Right?


VSHPWH (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

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


Should be `virtual` because it's `virtual` in the base class.  I think compilers tend to warn about this.


Sorry, I've reverted @SoulCover's commit.

I can't possibly keep up with the needs of every PHP framework and rely on trusting the intent of users that send PRs. @SoulCover please don't send PRs if you consider "something quick quick and dirty."

I recommend pegging composer/installers to a specific released version rather than `"*"` to prevent this.

Also a PR that actually fixes this with a test that lets me know when someone sends me a bad PR would be rad. Thanks!


If you change the name of the endpoint in #7292, make sure it gets updated here! I'd also suggest changing the key used to look up this URL from `enrollment` to something that more accurately reflects the behavior of the new endpoint. For example, `orders`.


Sound 1 is basically done. Sound 2 is copy+paste, just without Sweep and Envelope.


If these only work in Ruby, how are EB users supposed to start and stop the timers?


Does it need to be `double` instead of `float`?


r+


It's good that the interface is using uint. :)

N.B. : IIRC, dividing a negative int by a uint should give weird results, but I do think the case cannot happen so it should be fine.


It makes sense to continue IF the table doesn't exist, right?
So:
```
if (!admin.tableExists(tableName) || !isCDAPTable(admin.getTableDescriptor(tableName))) {
```


Did you mean to put a "var" in front of this? Seems like the var should be on line 165.


#　㈱エコサイエンスの仕事ー改修作業

##　メンバー
*　境
*　伊藤

##　㈱シューサンHPから写真などは移植する


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

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


Did you mean to put a "var" here? it will be locally scoped this way


you don't need the entrySet, just the values(), right?


VSHPWH (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: 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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


Which one is it, i've looked for it before. Also why do we need it? We never write anything to the data folder programmatically and all assets are used.


Yep, we had built a NewtonSoft serializer class but I think we can delete that now: https://github.com/NewSpring/Rock-Plugins/blob/develop/cc.newspring.Apollos/Utilities/NewtonsoftJsonSerializer.cs


The fact that there are multiple *distinct* change accumulators with sequential non-overlapping life cycles, each of which gets finished, surprised me.


@mikaelpatel
This update forced some reimplementation of my DHT usage, which really needed doing anyway, and in doing this I think I may have found a limitation about the way it is implemented currently implemented in Cosa. I needed to call DHT::adjust_data() from the implementation of on_sample completed() in my DHT derived class and can't because both DHT11 and DHT22 have this as private even though it is just protected in DHT

Can I propose that we either make DHT11::adjust_data() and DHT22::adjust_data() protected and/or include a call to adjust_data(), when valid, within the DHT::on_sample_completed() implementation.


VSHPWH (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: 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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


Is the following block redundant now?
It seem the Reset will be called twice in the !m_new_win_pos case.


had error 500, realized and fixed it after the deadline


Yeah. https://github.com/tmpvar/jsdom/pull/689 Bummer. I will put in a ticket to switch to phantom.


Hey @shama thanks! No worries, I guessed so much, I didn't know how/who merges those requests here. We currently look into getting rid of the installer. In the meantime will have a look at tests for this :+1:


that should do

you confirm @dmitryevseev ?


Thank you!


Thx, it helps a lot.


r+.  This makes me feel like console.error should be smarter and be able to accept input like:

  console.error('Failed to load module ' + name, ex);

And then do something smart like show a preview of the exception at first and then show the stack / more details if the user expands it or something.


You might add some extra sections on versioning. Specifically mention that versions for upstream would get automatically generated and you'll need to use the 'upstream_version' variable in some places.

---

Also, might be worth mentioning that maintaining the changelog isn't too useful for the upstream package. We can just use the Git log for that.

----

And also might be worth mentioning explicitly our goal is to avoid patches for upstream. Patches that fix critical functionality are acceptable (but we should be trying to land those upstream). Patches to remove things like PBR aren't desirable in the rpm-master branches however.


r+


Hopefully for the last time.


You could also log the value that failed.


not finished ... again ... but it appears to me that it's a better way to do the work. I will add perform the modifications on other screens too ... and polish the all work :)


```
Generated at: 2015-03-12 19:37:22
START
DONE [7 tests]
SUCCESS: true
```

HELL YEAH!
cc @jdonaldson


This describes what the test is doing, not what it's checking.

```rb
it 'passes everything when no methods are given' do
```


Hello jserv , i have removed the file **lab-1/fibseq** , and I checked it again. Is there something trouble with my repository?


I ran into an issue with Firefox where `rel="self"` had lower priority than `rel="alternate"`. :confounded:

I routed around it by outputting the value of `rel="self"` (sans the rel...) comes out first, then the list of links...including the `rel="self"` value again.

Just FYI. :smile:


Should there be spaces before and after the curlies?


Or you can just add the new order to the key issuer thingy.


@kitsunet Nice, thanks! Let me know when you get rid of the installer and I'll add TYPO3 to the natively supported frameworks.


Hi, Just checking your implementation of the Deque ADT. Its quite nice. You might want to integrate the concept of circular array to make it more efficient. for example calling pop(0) in the removeFront() is definitely not efficient as this will cause a backward shift of all the underlying array elements, making it to run in O(n) time complexity; Just imagine if you have 100,000 elements in the deque at the time you call removeFront. You might want to check my implementation of Deque here  (
 https://github.com/pophils/Algorithms-and-Data-Structures/blob/master/Chapter6_Stacks_Queues/ArrayDeque.py).


What platform was this causing problems under?


I'm concerned about this wait being removed. Are we sure that evaluate_script is going to wait until it's 0, or does this only check once?


Specify `.gitignore`: http://git-scm.com/docs/gitignore


It's still true, the C here refers to cgo.


same as above, this is not the same behavior as far as I understand.


> It depends on how detailed we want to get in describing the system

For the case of this section, I was leaning towards the least amount of detail, but still enough to create a ballot UI.  That was the crucial difference.  The reason I mentioned the two cumulative voting variations is that the mechanics of voting would be different (e.g. the ballot would look physically different).  For the different IRV variations, even though the tabulation process is different, the mechanics of voting would still be the same.

In the long-term, I do think there should be a way to handle the variations.  But I don't think that should be hashed out as part of this PR.

> For example, does the standard need to specify if a voter MUST cast a vote in each contest?

Are there such examples?  (I'm assuming this is just for the United States?)


I am probably missing context here, but I am wondering if we should support file extension "kjar"? Even though the kjars have some additional metadata, they are still ordinary jar files. Using "kjar" extension will confuse other tools for sure.


That doesn't help here in any way. That just means that right-clicking will issue `DeployTransform`.


In regards to testing, I would probably put all of this code into a function `specs = get_test_specs(m, py_files, pl_files)` that contains all the logic performed here (a few lines above a few lines below). Then I can unit test this once piece. This pull request represents a minimal change to the code, but I'd be happy to update it as a function with corresponding tests.


Good question. I think it's worked so far to commit tags straight to master. Do you agree?


well, that was the reason why I only commented on the line in the first place ;)
If I had just looked closer at the implementation for PSR-4 I would have seen the much more obvious answer... (https://github.com/composer/installers/pull/223)


@Shevchik may I ask you to add this project to the jenkins server? Since the compiled version is getting old and compiling every change is not a solution.


What version of teaspoon are you using? This has changed a bit on master at least, and so may cause issues.


@tiagocoutinho: was a pleasure to use PyTango. It was my very first day to write a device server and it couldn't have been simpler (okay, maybe that exception thing, but ... doesn't really matter). Thanks again.


Sounds good to me.  Just confirming.  Thanks!


fugly, and renderTo should be undefined in our case. It feels better to drop it completely from the config


print можно убрать, пускать будем фоном через whenever (cron)


Hej Mats.
Den länken du skickade funkar inte. Jag har inte tillgång till repositoriet. Jag har fått  pusha det på samma som innan.
https://github.com/topcom94/Idebeskrinvning2--1dv406-eh222uq
Jag vill ha det knutet till topcom94 istället.
Date: Mon, 9 Mar 2015 06:19:07 -0700
From: notifications@github.com
To: Idebeskrinvning2--1dv406-eh222uq@noreply.github.com
CC: topcom94@hotmail.com
Subject: Re: [Idebeskrinvning2--1dv406-eh222uq] Färdiga planering. ASP.NET (60ed2b0)

*** KOMPLETTERA ***

Du har inte följt anvisningar beträffande idébeskrivningen till det individuella arbetet i kursen ASP.NET Web Forms (1DV406).

Ta del av kraven på https://coursepress.lnu.se/kurs/aspnet-web-forms/individuellt-arbete/ punkt 2.

Jag vill att du klonar repositoriet https://github.com/1dv406/eh222uq-3-1-individuellt-arbete.git och ”pushar” idébeskrivningen dit. Hör av dig om du vill ha det knutet till användaren topcom94 istället.

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


ack. pretty sure this is removed in latest local changes


`assert(inputCatalog.size() == outputCatalog.size());` ?


Item.all слегка загружает систему на самом деле получая все данные в память


ack


clarity?


It should disappear. This would be the final goal.


в вариантах если нет scope методов то лучше делать Item.find_each do |item| очень экономит память


Can you explain this for my edification?

On Thu, Mar 12, 2015 at 11:45 AM, Michael Wehrley <notifications@github.com>
wrote:

> Sounds good to me. Just confirming. Thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cbitstech/guides/commit/7fc213dea70a94396b4d168c730fa4cdc97ec03b#commitcomment-10168686>
> .
>


Oh dear, that looks complicated. If you promise me you tested that you get the thumb :+1:


I believe this may have broken the Combine being able to open Nexus doors.


Fixed in 89ae6f15


I believe latest versions of laravel introduced the ability to inject
straight into the route handler, so we can skip the boilerplate of
constructor injection.

It's technically possible to still stub the dependencies since under the
his it is doing service location, but it's a bit harder and requires
knowledge of the framework, instead of just knowledge of the class and it's
dependencies.

In any case, is not a big deal right now, but I've found its much easier
down the line if you're just strict about this up front. Injection makes it
painfully obvious when your dependencies are getting out of control.

On Thu, Mar 12, 2015, 1:57 AM Brad Smith <notifications@github.com> wrote:

> This might be OK actually - it is still testable. Reading about Laravel
> Facades vs. static access. "In the context of Laravel, Facades are what you
> are using when you make what appears to be a static call to a class." -
> http://taylorotwell.com/response-dont-use-facades/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CompassHB/compasshb.com/commit/cff07f54b8af58a848bc192ef027c8a268c9ea0d#commitcomment-10159269>
> .
>


I'm pretty pleased that the conflict resolution code is pushed down to the accumulator.  That feels like a win compared to our Sync implementation.

The // First. // Second. comments help a lot when parsing the code.


This documentation needs a `@history[#:added "6.1.1.8"]`.


No version annotations on any of these new functions.


This is needlessly ambiguous: can we call this *server* last modified throughout?


The draft already has "BallotMeasureSelection," which is 22 characters long.  I think it definitely helps make things easier to understand when there are consistencies across the spec.


Why are you merge || xip.io?  Just set it to xip.io.  If you override it in your manifest it will replace the value.


Thanks for the heads up.


Looks good


Alors comme ça, on commit ?


I did as much as I could. But we should really think about starting testing `tofu`.


You say it clearly, I just didn't know. That's it.


AFAIK the packaging of the KIE Maven Plugin is KJAR.. so we have files with the KJAR extension (that is most certainly what the User - reporting the BZ - expected too). That said, armed with what I believed to be true, I looked at the KIE Maven Plugin and cannot see that it creates files with the  KJAR extension. Do you know what's meant to be happening (I see you've commited to the plugin in the past).


@caraway Currently we create branches for each change and then submit a pull request that merges with master. In emergencies, @ericcf says we can just commit directly to master so that no pull request is necessary.


This design has to be considered an interim design, because the new constructor story imposes different rules.  Documentation should not describe the behavior here without including the caveat that it is expected to change in the near future.
Specifically:
 1. The special nature of the initialize() function will be removed; if the user wishes to define an initialize() function and call it from his constructor, that is his business.
 2. Whether accessed directly in the body of a constructor or in a method called therefrom, *fields* which are declared as const will be considered const in that context.  An attempt to overwrite a field in these contexts can be diagnosed (with an error message) at compile time.
 3. Whether the object as a whole is const has to be determined in the calling context.  Unless the method carries the "const" this-intent, it may modify any non-const field in that object (i.e. 'this').  Invalid uses of const object can also be diagnosed at compile time.


Would be better to have a generic differential input in Mibuild/Migen, that gets lowered to vendor-specific code using platform hooks (similar to e.g. synthesis constraints that disable retiming, reset synchronizer, etc.)


Live tests should have `Live` in their names and be marked as @IntegrationTests.


I think this commit and your other commit 8b11ea4 will conflict with my commit 0acdfd1d0d21d56d69acb3da46a4f88eaaa72bbf

I worked around this a different way, more or less inheriting the css styles from the active theme by appending the `textarea` class to the filetree divs.  The FileTree's font color and border styling look perfect this way, regardless of the theme, and keeps the styling control in the master css files.


I modified your solution and it is working for me now: 7f7dea574c30eb35f1276c38b3204a2d86f0e783


I'll take a look and submit a hotfix.


Correction


The main problem is that this is injecting in a JS string context as well as an HTML context, so it needs to be escaped for JS and in the correct order. There's also the one that is in a URL so it needs to be URL escaped, then JS escaped, then HTML escaped. :/ laravel is only helping us with the html, which is great, but not sufficient, according to my understanding.


And generic differential output...


Yeah, you're right. We should definitely provide an explanation to why the validation fails. getDescription(Locale locale) sounds good. I'll implement it in the DefaultScreenNameValidator. =)


Eh?  We should make sure this or its equivalent works before landing.


I am using bcmstat on a RPi2 Model B running Arch Linux. It is running as a web server using lighttpd and php. One of the php scripts seems to run very slowly on a Pi compared with a laptop computer so I am using bcmstat to try to sort out where the bottleneck is. bcmstat itself is working perfectly and thank you for such a great tool.

I did overclock the ARM, Core and SDRAM and there was some improvement. When looking the bcmstat output I noticed that not all of the four cpu cores are being used while the script is running so I need to figure out how to get that to happen. The other part of the bcmstat output that shows red is the column labeled "%iowt". I cannot figure out what "%iowt" means and therefore I do not know how to go about reducing the values, which are consistently in the 25 to 40 range while the php script is running and essentially 0 when it is not running. Any advice you can give me on that would be much appreciated.

Thanks,

Paul Crawford


For consistency though, we should probably have the same behaviour for IPv4.


... which means that this recursive search for an enclosing constructor context would be unnecessary.  The compiler can determine everything it needs to by comparing argument intents (including this-intent) with the declared const-ness of the variable or field in question.


This is bad. If you want by default use Windows home directory for users you need just provide proper /etc/nsswitch.conf file.


Nice work, and i thought you guys were all done with the IE8 nonsense awhile back :)


Well, I am totally fine with introducing changes such as these if they make sense, architecturally. I am just not sure that now is the right time to introduce them, since it is a breaking change.

I only found about this because I deployed the latest version of laravel and poof, I had a lovely error page staring back at me.

Of course, I updated my deploy script to modify permissions of these files post-deploy, so for me personally this is not an issue anymore, but choosing when to make these kinds of changes and communicating them clearly is, in my opinion, vital.


Note that people might just be looking at this commit so such messages can be less than helpful. :)


The problem is that we can do this only so far on Travis. But I will set up a cronjob on our infrastructure once I come back. In that job I will also trigger the tests in ufo-core which test things on a lower level. So, go ahead :+1:


@konstan You are not commenting to the right commit.
We are on cf985c2f146160c025519249e8de8f41294e9e1b but the next (and last) commit is 3b5c60473e94e6470e2fcdc9807adf445344fc26.
To have a good view of the pull request you should look here (which is the complete diff): https://github.com/slipstream/SlipStreamServer/pull/257/files


I like fuck


I can not run in my terminal and does not have this code.


k


This is so funky, but this comment is great.  It is technically possible for a malicious user to name their client `$local`.  Is there a character that is *not* valid in a client name?  ASCII character 0?  Or the tab character?  Is it reasonable to use NULL for this placeholder?


Hopefully this clarifies releases a little: https://github.com/cbitstech/guides/pull/7


I am growing more convinced that I'm wrong.. I added the KIE Maven Plugin to the build/plugins for uf-playground/mortgages and a JAR was produced.. that said, I'm not entirely confident the KIE Plugin actually ran.. as the resulting JAR just contains the DRL sources - whereas I expected it to have pre-compiled the DRL and stored in somewhere (where, IDK).


Hi Anthony,

This won't stop us from reading in any propagator, as if we get
propagators from someone else they won't be in this format unless we
specify it to them. Our propagator files are not a standard. So I don't
agree it reduces the flexibility. I think it makes the code cleaner,
allows us to extend various information into the propagator without
adding arguments in the calling function. It also will stop us doing
anything too stupid.

If you wanted to compute something weird you would write a stand-alone
code to do that specific job.

I am also worried about having too many comma separated variables in the
input file with very specific order of options is also very inflexible.

I really think we need to have more information in the header as setting
the propagator data from the header is safer than from the input file.
The IO needs to change anyway because at the moment the props we are
writing are little endian but there are big endian machines out there
and fwrite uses the native machine endianness.

We should talk more about this, I just don't want us to start a big run
and then realise we missed something.

Jamie

On 15-03-12 11:10 AM, afrancis-heplat wrote:
>
> Hm, sure I can. But won't this stop us from reading in any propagator
> given to us? I mean, part of the idea was that the code should be
> flexible enough to read in any propagator with only moderate additions
> to the contraction code. What if we get Nilmani's overlap propagators?
> We won't be able to change the header easily. I might be missing
> something, but if this reduces the flexibility of the code, I am
> against it.
>
> Disregarding cleanliness in favor of flexibility and freedom once
> more, I think these things should be specified in the input file.
> Reading from the header can serve as cross-check at most. In the end
> the user has to be able to control the whole code via the input file.
> Let's for argument say, if I wanted to compute in the NREL basis using
> CHIRAL basis propagators, I should be allowed to do so. In the
> corresponding routine I would call CHIRAL_TO_NREL and everything
> should work regardless. This might not seem like an advantage now, but
> if e.g. I wanted to compute parity projected, non-relativistic mesons
> or baryons, I would need this.
>
> Hm…I'll continue to think about this, but right now I think the IO
> should stay as it is and not be further updated.
>
> Cheers
> Anthony
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RJHudspith/Contractual_Obligations/commit/d1b0a16e1af9936877c4321380a7f3e62f960f08#commitcomment-10166127>.
>


Log for my compilation:

[INFO] Scanning for projects...
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building mortgages 0.0.1
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.4.1:clean (default-clean) @ mortgages ---
[INFO] Deleting /home/manstis/Desktop/temp/uf-playground/mortgages/target
[INFO]
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ mortgages ---
[debug] execute contextualize
[WARNING] Using platform encoding (UTF-8 actually) to copy filtered resources, i.e. build is platform dependent!
[INFO] Copying 13 resources
[INFO]
[INFO] --- maven-compiler-plugin:2.3.2:compile (default-compile) @ mortgages ---
[WARNING] File encoding has not been set, using platform encoding UTF-8, i.e. build is platform dependent!
[INFO] Compiling 4 source files to /home/manstis/Desktop/temp/uf-playground/mortgages/target/classes
[INFO]
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ mortgages ---
[debug] execute contextualize
[WARNING] Using platform encoding (UTF-8 actually) to copy filtered resources, i.e. build is platform dependent!
[INFO] Copying 7 resources
[INFO]
[INFO] --- maven-compiler-plugin:2.3.2:testCompile (default-testCompile) @ mortgages ---
[INFO] Nothing to compile - all classes are up to date
[INFO]
[INFO] --- maven-surefire-plugin:2.10:test (default-test) @ mortgages ---
[INFO] Surefire report directory: /home/manstis/Desktop/temp/uf-playground/mortgages/target/surefire-reports

-------------------------------------------------------
 T E S T S
-------------------------------------------------------

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO]
[INFO] --- maven-jar-plugin:2.3.2:jar (default-jar) @ mortgages ---
[INFO] Building jar: /home/manstis/Desktop/temp/uf-playground/mortgages/target/mortgages-0.0.1.jar
[INFO]
[INFO] --- maven-install-plugin:2.3.1:install (default-install) @ mortgages ---
[INFO] Installing /home/manstis/Desktop/temp/uf-playground/mortgages/target/mortgages-0.0.1.jar to /home/manstis/.m2/repository/mortgages/mortgages/0.0.1/mortgages-0.0.1.jar
[INFO] Installing /home/manstis/Desktop/temp/uf-playground/mortgages/pom.xml to /home/manstis/.m2/repository/mortgages/mortgages/0.0.1/mortgages-0.0.1.pom
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1.983s
[INFO] Finished at: Thu Mar 12 16:58:22 GMT 2015
[INFO] Final Memory: 18M/439M
[INFO] ------------------------------------------------------------------------

No sign of the KIE Plugin actually running.


Won't this result in `gosu elasticsearch elasticsearch elasticsearch` in the default case?

I still think splitting up the detection of `'-'` and the `gosu` part will make things clearer.


And the POM I used:

<?xml version="1.0" encoding="UTF-8"?>
<project xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns="http://maven.apache.org/POM/4.0.0"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <modelVersion>4.0.0</modelVersion>
  <groupId>mortgages</groupId>
  <artifactId>mortgages</artifactId>
  <version>0.0.1</version>
  <repositories>
    <repository>
      <id>guvnor-m2-repo</id>
      <name>Guvnor M2 Repo</name>
      <url>http://localhost/maven2/</url>
    </repository>
  </repositories>

<dependencies>
<dependency>
<groupId>org.kie</groupId>
<artifactId>kie-api</artifactId>
<version>6.3.0-SNAPSHOT</version>
</dependency>
</dependencies>

  <build>
    <plugins>
      <plugin>
        <groupId>org.kie</groupId>
        <artifactId>kie-maven-plugin</artifactId>
        <version>6.3.0-SNAPSHOT</version>
        <extensions>true</extensions>
      </plugin>
    </plugins>
  </build>

</project>


oh i see what your saying now -- simply change:

* python/example-executor -> python/test-executor
* java/example-executor -> java/test-executor

and all should be well to merge, right?


```
<?xml version="1.0" encoding="UTF-8"?>
<project xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns="http://maven.apache.org/POM/4.0.0"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <modelVersion>4.0.0</modelVersion>
  <groupId>mortgages</groupId>
  <artifactId>mortgages</artifactId>
  <version>0.0.1</version>
  <repositories>
    <repository>
      <id>guvnor-m2-repo</id>
      <name>Guvnor M2 Repo</name>
      <url>http://localhost/maven2/</url>
    </repository>
  </repositories>

<dependencies>
<dependency>
<groupId>org.kie</groupId>
<artifactId>kie-api</artifactId>
<version>6.3.0-SNAPSHOT</version>
</dependency>
</dependencies>

  <build>
    <plugins>
      <plugin>
        <groupId>org.kie</groupId>
        <artifactId>kie-maven-plugin</artifactId>
        <version>6.3.0-SNAPSHOT</version>
        <extensions>true</extensions>
      </plugin>
    </plugins>
  </build>

</project>


No, not at this time, but I believe it's possible to eventually support it


Perfect.  Thanks.


@methodmissing will do.  Assume you meant #check_uninstall.  Also, I think I'll just override Que::Job#run "permanently" instead of this singleton method business.


Yeah, this may be definitely confusing. The kjars have packaging "kjar", but the actual file extension "jar". The packaging does not always correspond to the actual file extension.

I have looked at the related BZ and they are using `-Dpackaging=kjar` so the maven deploy plugin uses that extension. I am not really sure why though. I would expect the file extension stays the same, but this seems to a special case with the deploy plugin.

Looking here https://github.com/guvnorngtestuser1/guvnorng-playground/blob/master/mortgages/pom.xml the project does not have the packaging set to "kjar" (+ the kie-maven-plugin needs to be defined under `<build>`), so Maven treats it as ordinary jar.


Huh, so we're not caching the oauth credentials?  I will file follow-up: Bug 1142596.


test


this is a test comment


This broke my program :(


The existing error notifications are tied to the account rather than to the service, so I'm not concerned about overlap.  With Bug 1142596, I intend to move the FxA gubbins out of the RL sync adapter, centralizing the account health checks and with it any account state notifications.


Ditto.


This is an example of POM that uses the correct setup: https://github.com/droolsjbpm/droolsjbpm-integration/blob/master/kie-examples-api/kie-maven-example/pom.xml


useless empty line


what are you talking about, I meant adding
```
PerformKeyboardOrderOnSelection(a => new Order("DeployUpgrade", a, false));
```
to
```

		bool PerformDeploy()
		{
			// HACK: multiple orders here
			PerformKeyboardOrderOnSelection(a => new Order("ReturnToBase", a, false));
			PerformKeyboardOrderOnSelection(a => new Order("DeployTransform", a, false));
			PerformKeyboardOrderOnSelection(a => new Order("Unload", a, false));
			PerformKeyboardOrderOnSelection(a => new Order("Detonate", a, false));
			return true;
		}
```
in WorldCommandWidget


Check out WITH-OUTPUT-TO-STRING - it would be a more efficient model than the SETSAP setup.


Yes thats why i changed it


To my eyes, it looked OK with just the "Sign in" text. The way I see it, there are three options:

1. More generic "Finish" text
2. Generic "Sign In" text (current)
3. Switching the button text to say either "Sign In" or "Register" depending on the radio button

What do you think?


r+


Right, it'd be nice if we can get javascript validation from the validator so it corresponds to the configured validator. However, I'm not really sure how to implement this. Maybe we can return the regex string for js validation in getJavaScriptValidation()? I'll think more about it....


it might confuse users to have different names for the same function (commitCriteria/applyConfigurations). If/when we want to rename it, we can do so on both the c++ and c apis


:+1: Nice, thanks for these updates to Shadowmoon :new_moon: Valley. :smiling_imp:


saw approval from jdm
at https://github.com/yodalee/servo/commit/ff76b2f29030778d135395d35c9cdeac77df3279


merging yodalee/servo/issue4768-textencoder = ff76b2f2 into auto


yodalee/servo/issue4768-textencoder = ff76b2f2 merged ok, testing candidate = 4d25af57


Oups, my fault. You are totally right^^


s/it's/its/


> atomic.StoreUint64(&tailNode[childKey], childVal)

The problem with that line is in order to call the pointer method the ```tailNode[childKey]``` needs to be addressable, a map indexing operation is not addressable in Go there thats how the local var offset comes in.


This should have been linked to this PT story instead: https://www.pivotaltracker.com/story/show/84203024

and should have been "Add remark...."


This should be `if (!g_tracepoint_registered)` I guess?


Is this not PII?


สร้าง gamepanel ขึ้นมาชื่อ gp โดยสามารถคัดลอกต้นแบบและใช้งานคลาส GamePanel ได้


Good job :+1: nice to see things still happening in Shadowmoon :new_moon_with_face: Valley (SMV) :smiling_imp:


Thanks Jem--is there any guide as to what constitutes an emergency?

On Thu, Mar 12, 2015 at 11:55 AM, Jem Hilton <notifications@github.com>
wrote:

> @caraway <https://github.com/caraway> Currently we create branches for
> each change and then submit a pull request that merges with master. In
> emergencies, @ericcf <https://github.com/ericcf> says we can just commit
> directly to master so that no pull request is necessary.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cbitstech/guides/commit/7fc213dea70a94396b4d168c730fa4cdc97ec03b#commitcomment-10168881>
> .
>


Pas forcément gênant, il faut juste clarifier si c'est structurel ou temporaire, et lequel des liens est l'officiel.
Intuitivement ça me semble assez logique… l'environnement contient les personnages, et les personnages perçoivent leur environnement, non ?


Thank you


Not final so we can tweak from an add-on?  And maybe put it in `ReadingListConstants`?  It still may be PGed out :(


This method does not seem to follow the same idiom as other methods above (i.e. return a boolean and have an output argument). See pfwGetCriterion for instance


This looks fine to me.


What is the TODO -- finer grained tracking?


The docs don't make it clear that if the target is nil the defaults will be applied to an empty map.  The 'otherwise' could read that a nil target will result in an empty map with no defaults applied?


%iowt stands for "IO Wait" - this is CPU time spent waiting on IO to complete, most likely writing to the SD card, or some other IO. If you Google "iowait" the first few links should help explain what it is and how to troubleshoot it.

An IO Wait value of 25 to 40 is really very high, and you should see if there's a way to reduce the number of writes your script is committing to disk, for instance by not logging information excessively, or avoid hammering a local database with updates (can you relocate the database to a remote server, or simply update the database less often?)


I think the "kjar" extension should not be allowed, because it will break also other things. For example one can no longer easily depend on the kjars, since Maven by default does not know about this packaging. The jars are very universal on the other hand.

@etirelli, @mariofusco what do you think about this - using ".kjar" file extension (instead of plain ".jar") for kjars


This is a good improvement. Next step is really to to removing the defaults altogether and forcing the user to supply values. I think this is direction I wanted to take here: https://review.openstack.org/#/c/138913/5/thirdpartyci/install_log_server.sh


Right. We should probably add others generic modules: Clock buffer, DDR input, DDR output.
(GMII phy of LiteEth can be generic with a generic DDR output, it would be the same for a future RGMII phy if we have generic DDR input and DDR output).
Other  I/O or clock blocks,are probably too specific to be instanciated correctly.
I'll look at that, thanks.


Looks fine to me.


Then sure I could do that, but that is adding *another* order to the already bogus list IMO.


Hey Al,
wp_head here


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


One more example of different packaging X extension: `drools-core` is using packaging "bundle", but the final artifact is plain jar with OSGi metadata, see https://github.com/droolsjbpm/drools/blob/master/drools-core/pom.xml#L13


Thanks for the contribution!


Eh..


I tend to think it's better to return the full Js script as an String. Any
other solution will be too limiting.

On Thu, Mar 12, 2015 at 6:09 PM, Pei-Jung Lan <notifications@github.com>
wrote:

> Right, it'd be nice if we can get javascript validation from the validator
> so it corresponds to the configured validator. However, I'm not really sure
> how to implement this. Maybe we can return the regex string for js
> validation in getJavaScriptValidation()? I'll think more about it....
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pei-jung/liferay-portal/commit/3d28067afed2d5669935489560acdc72b009d9e4#commitcomment-10169179>
> .
>

--
Jorge Ferrer
Vice President, Engineering
@jorgeferrer <https://twitter.com/jorgeferrer>

*Liferay Inc*
Web <http://www.liferay.com> | Facebook <http://facebook.com/liferay> |
Twitter <http://twitter.com/liferay>

*Aviso de Confidencialidad: *Este mensaje de correo electrónico, incluidos
los archivos adjuntos, enviados en nombre de Liferay SL es para el uso
exclusivo del destinatario (s) previsto y puede contener información
confidencial y privilegiada. Se prohíbe cualquier revisión no autorizada,
uso, copia, divulgación o distribución. Si usted no es el destinatario, por
favor póngase en contacto con el remitente por correo electrónico de
respuesta y destruir todas las copias del mensaje original. Gracias.

*Confidentiality Notice: *This e-mail message, including any attachments,
sent on behalf of Liferay SL is for the sole use of the intended
recipient(s) and may contain confidential and privileged information. Any
unauthorized review, use, copying, disclosure or distribution is
prohibited. If you are not the intended recipient, please contact the
sender by reply e-mail and destroy all copies of the original message. Thank
you.


Thanks @mattwynne, this is better! :)


Rename the Figaro methods to reflect the keyed option (`bucket_name: Figaro.env.dragonfly_s3_bucket_name`)


closes #20

`nims_timestamp` has been using `bson.tz_util` to set a fixed offset for US/Pacific time.  Removing bson  seems like a good reason to add better time zone support.


TeamCity Akka.NET :: Akka.NET PR Build [Build 138](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=235&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


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


Do you need to do s.refresh()

Jay

On Thu, Mar 12, 2015 at 12:20 PM, elijah9 <notifications@github.com> wrote:

> The only code after the while loop is clearing the screen and displaying
> "GAME OVER" and "SCORE: X" on consecutive lines. It used to work, so I
> don't think the problem is with the code after the while loop. I'm not sure
> what you mean by the second two questions.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/elijah9/CMPU-203/commit/fa9068f826ca25e74246f3346fbececfd0f7955e#commitcomment-10168169>
> .
>

--
Jay McCarthy
http://jeapostrophe.github.io

           "Wherefore, be not weary in well-doing,
      for ye are laying the foundation of a great work.
And out of small things proceedeth that which is great."
                          - D&C 64:33


Sick wit it


You can sum up the arrows to get from `List[A => M[A]]` to `A => M[A]` and then apply that to `mp.right[Unmergeable]` but it's not a built-in thing in scalaz.

```scala
def hmm[F[_]: Foldable, M[_]: Monad, A](fs: F[A => M[A]]): A => M[A] =
  a => fs.foldLeft(a.point[M])(_ flatMap _)
```

Not worth it to define this unless you need it more than once. I would leave it as-is.


In chat you had said:

> So the code is name and value is the link right?

This appears to be different. So I'm wondering what your thinking is regarding that.


Seems OK to me. Feel free to merge when appropriated.


Tweeted the man himself to see if he has any ideas: https://twitter.com/Aaronontheweb/status/576069559532150784


If you think you have an emergency, are you going to stop to read a guide first?


Me too!


what you mean with `individual modules` ?
you're talking about using handlebars in multiple versions?


This method, and the above method look similar. Modify `get_controlled_vocabulary_values_for_predicate_name` to call the new method then pluck the value.


A dlaczego Opalskiego będzie dwa razy?


@kchanba were the select changes an accidental check in?


Please keep the test and doc files. Those can be nice for offline usage.


Launch of Arias.


I did not realize that code name is actually is the attribute name.. So I have to use predicate_value_code to store the value


To clarify, I want to make sure I know if these cases are emergencies:

- there's a deadline and I'm working by choice on a weekend and want to
merge in order to proceed
- there's a Rails exception that is a big deal and no one is around to
verify it and a PM addressed it to me
- I'm stopped for N hours during a workday because people aren't around to
merge

On Thu, Mar 12, 2015 at 12:21 PM, Eric Carty-Fickes <
notifications@github.com> wrote:

> If you think you have an emergency, are you going to stop to read a guide
> first?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cbitstech/guides/commit/7fc213dea70a94396b4d168c730fa4cdc97ec03b#commitcomment-10169457>
> .
>


ideally we would have something in the DOM that would be changing that we could use as a queue to then fire this execute. And, to your point, we now have a spec that is 'working on my computer' but not on CI. So, it is my understanding that sleep would sleep the whole process (including the browser) but maybe it is still the best way to go. Absolutely willing to reconsider this. The `wait_until` is no longer a part of capybara, as are any other public methods that invoke some kind of timeout. thanks @jejacks0n for the feedback.


I interpret a leading underscore on the method name as meaning something like "the user shouldn't need to know about this implementation detail" (kinda like `private` in C++).  However, the user is expected to call this from his subclasses.


here


Hi,

Yes, yes, it should.  That is a bug.  Thanks for noticing it!

Chris

On Thu, Mar 12, 2015 at 8:34 AM, Khaled Hosny <notifications@github.com>
wrote:

> Shouldn’t this en be fr?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/simoncozens/sile/commit/a37de379f8817ace4b192a84b5d72652a0db6703#commitcomment-10158961>
> .
>


I don't care much. But `t.ifError()` is properly the correct choice. Also this was caused by the upgrade of `async`.


This link is currently broken because of #18. @b-studios, can you fix or disable the autolinking plugin?


First commit for the Coursera project


Sure, was thinking abt it and missed it


resolves issue #21388


VSHPWH (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1657 of 1659 tests passed)

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


I'm feeling kind of dumb now. Thanks for catching that Jon.


You should document this in the `README.md`. Also why did you decide to do this? It's fine, I'm just curious.


Cool. Thank you.


"starts


:smile:


I would maybe say just use your judgement? However, if it becomes the case that you have multiple emergencies every day, then it may be time to rethink some things.


Well you can't ever mount overlayfs off of a block device. The underlying `lowerdir` must be on a mounted filesystem itself. Hence, the cautious consideration of "let's make sure the user doesn't mount a shared block device" really does not apply here.


> I wonder if this blank line causes any issues with gitignore?

Per [the docs](https://www.kernel.org/pub/software/scm/git/docs/gitignore.html):

> A blank line matches no files, so it can serve as a separator for readability.

So we're good. :+1:


nice!  Is that from your version?  Let me  see!  I'm working my way through the big [haxe unit test suite](https://github.com/jdonaldson/haxe/blob/haxe_lua_rb1/tests/unit/src/unit/Test.hx) right now.  I can get it to compile successfully (which uncovered a large number of problems by itself), but it's failing in the lua runtime.  I'm currently working on implementing a try-catch block, which I'm guessing I'll solve with yet another iife inside of a pcall.


TeamCity Akka.NET :: Akka.NET PR Build [Build 138](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=235&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.MultiNodeTestRunner\bin\Release\Akka.Cluster.Tests.dll, D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Tests\bin\Release\Akka.Tests.dll   ... Build time: 00:10:49
### Failed tests
```
Akka.Tests.dll: Akka.Tests.Actor.InboxSpec.Inbox_have_a_default_and_custom_timeouts: <no details avaliable>

Akka.Cluster.Tests.dll: Akka.Cluster.Tests.EWMASpec.Calculate_the_EWMA_for_multiple_variable_datastreams: <no details avaliable>

```


Ahh yes I need to do that


Oh yeesh sorry meant to remove that but got ahead of myself.  Will update the pull.  Thanks for catching!


Can you link to a jsFiddle with the before and after so I can test?


Why even include `handlebars` at all? It's not even being using to precompile templates, the compilation happens on the client side. There is no need for a dependency on handlebars.


yes, if possible follow a return early pattern


TeamCity Akka.NET :: Akka.NET PR Build [Build 139](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=237&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


Did you check in even the C++ files of the logfor cpp???.

I see lot of checkins..


VSHPWH (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Thanks for the timings!

Something to also consider is floating point stability.  The reason why I reverted to logs in the first place is because I was worried that flat out exponentiation would lose floating point precision.

But perhaps this isn't as big of a deal in this scenario ...


@jdonaldson Hello! Yeah, it is my implementation. I`ll commit soon. I am running the same Haxe`s testing suite - https://github.com/jdonaldson/haxe/blob/haxe_lua_rb1/tests/unit/src/unit/Test.hx


Some times you dosn't choose worst possible. for example in this pic you give me 2 at bottom right. but if you give me 4 the game will be over at one less move.

![screenshot from 2015-03-12 20 47 54](https://cloud.githubusercontent.com/assets/1582954/6623831/b62b21b2-c8fa-11e4-99e2-4969f296492e.png)

please open issues :D


OK. I'll add separate comments for the two functions.


Please revise your English expression.


There's one more thing this should do before falling back to returning the Thread.Domain which can be incorrect.

```csharp
this.Method.DeclaringType.GetTypeObject ().Domain
```

If the frame is managed, this will return the correct domain.


Don't add the generated files such as `lab-1/fibseq` and `example5/example5` into GIT repository.
Please consider to set `.gitignore`.


yummy android development


* monday deadline = not an emergency, should be rare, and probably just poor planning. rush jobs are a leading cause of mistakes that a reviewer would catch :-)
* exception with no one around = possibly an emergency
* not sure why work would come to a standstill due to a single merge AND none of the developers are around for hours on a workday, but if this happens go right ahead and have someone review at the earliest opportunity


1. Enabling and disabling signal deliver through ioctl() should work only if at least one device is "open", which is also the span during which g_tracepoint_registered is true.
The idea is to allow these two ioctls only between ppm_open() and ppm_release() calls.

2. I was checking whether two consecutive compat_register_trace() [or compat_unregister_trace()] are allowed or not but couldn't find anything that says it's not allowed. I could use an additional global in addition to g_tracepoint_registered to block it but didn't find anything reference that says calling compat_register_trace [or compat_unregister_trace()] twice consecutively is fatal or forbidden.


Best function I've seen all year.


Bye bye "data.data" !!! o/


I would simply move the `m_codechars` member from `CInputCodingTable` directly into `CInputCodingTableBaiduPY` where it is actually used.


why is this removed and not replaced? I was expecting something like `private ArrayList<String> slots;` are there no concept of slots now? was this passing?


I'm planning to rebase into commits that make more sense once we're both happier with it.


I think I finally understand how to use delegates and just how awesome they are.  The structure that I am using now is the same that you set up on our last game


No, apparently not. I played a bit also with a generated id, but the `createTuple()` is only invoked during commit. The id is fetched right upon save(), but I don't collect sequence fetches so no context is needed at this time.


Move all "import" stuff to the beginning of a file. (right after #!/usr/bin/python..)


Thanks for the responses sirs.
On Mar 12, 2015 12:35 PM, "Eric Carty-Fickes" <notifications@github.com>
wrote:

>
>    - monday deadline = not an emergency, should be rare, and probably
>    just poor planning. rush jobs are a leading cause of mistakes that a
>    reviewer would catch :-)
>    - exception with no one around = possibly an emergency
>    - not sure why work would come to a standstill due to a single merge
>    AND none of the developers are around for hours on a workday, but if this
>    happens go right ahead and have someone review at the earliest opportunity
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cbitstech/guides/commit/7fc213dea70a94396b4d168c730fa4cdc97ec03b#commitcomment-10169753>
> .
>


wish there were a favorite button for commits


@almul0 @benblazak : This is a nice useful one :), a single key to "select the full line"

for example, Im editing in a website and I need to copypaste line-entries from one to another, so I hit the key and all the line becomes "selected", so i can cut/copy it, then move to the other form and with the same key "select" the other line to be replaced with my ctrl + v   =)

I wanted to include this one long time ago, its pretty useful!


Yes I've also thought about codingtables as addons but I'm not sure if that would be too slow. I'm already wondering if doing a HTTP call and JSON parsing after every button press is a bit much. How big is the whole codingtable? Would it make sense to somehow cache the retrieved values?


Need to check pmap_is_current here.


Can choices have a bit better names, like ["empty", "zombie", "key", etc...] ?


That indeed did the trick. Thank you so much.


The package needs a `setup.py` file for this to work


I probably makes sense to install this into the global bin folder rather then the package-specific libexec folder to be able to run it without `rosrun`. Instead of using `catkin_install_python` you can list it as a script in the setup.py file.


```
$http.get('todo.json').success();
```
초기 코드를 구성할 때에는 success함수를 호출했었지만
에러 로그를 확인할 수가 없어 then을 사용하여 이를 해결하였다.


```Rust
use std::cmp;
use std::num::{UnsignedInt, NumCast};

fn percents<T: UnsignedInt>(input: &Vec<T>) -> Vec<f64> {
    let mul = 100f64 / f64::from(input.iter().fold(T::zero(), |m, &e| cmp::max(m, e))).unwrap();
    input.iter().map(|elem| mul * f64::from(*elem).unwrap()).collect()
}

fn main() {
    let input = vec![0u32, 9, 8, 7, 6, 5, 1, 2, 3, 4];
    let out = percents(&input);
    println!("{:?}", out);
}
```


btw, renamed to gui_selectline in next commit


I'd add something like
    name: 'Avocado HTML report links'
    description: Verify that the links of the HTML report point to the proper locations, by clicking on all of them. No links might point to non existing locations.


###에러 원인

JSON의 양식이 `double quote`로 구성되지 않아(`single quote`도 상관은 없음) 오류를 발생하였다.
`javascript`의 `use strict`선언은 괜찮은 것 같다.


OK! Good suggestion!


This is good, but I'd add the 'functional test' of the avocado HTML report links' as well. Of course, we can think more moving forward on an automated test to ensure the links look correct.


We cannot do this, because we need tinymce as our text editor which you can see in production. I know: In Development environment the description cannot be edited except from directly editing the HTML Code. Would be cool if we got that working again, but priority 1 is that Tinymce Editor runs in production.


Please also push the required rct commits.


Is that still todo with Hardy's work?


Yeah, we have never really settled on how to indicate {{protected}} in Python, which is what I think this would be in C++.


Dude, I checked-in all the files which were there in include folder..
I think all are not necessary, only the ones that we use should be enough Is it ?
I will check!


TeamCity Akka.NET :: Akka.NET PR Build [Build 139](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=237&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Persistence.Tests\bin\Release\Akka.Persistence.Tests.dll   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)   at Fake.XUnitHel... Build time: 00:11:18
### Failed tests
```
Akka.Persistence.Tests.dll: Akka.Persistence.Tests.GuaranteedDeliveryCrashSpec.GuaranteedDelivery_should_not_send_when_actor_crashes: <no details avaliable>

```


Yes, done. I also added in the missing exception handling for the !defined(HAVE_GETADDRINFO) case.


warum das?


Sure...still waiting for someone to review and merge my pull request for this repo; it's not an emergency.


I think that makes a lot of sense. I've pushed a commit to make that change.


Go team olive!


Ok.  Maybe add precedence=-1 to z, so that it won't appear in the GUI?  I don't think it's meaningful in the GUI.


Bro, do you even code?


copy paste error


I don't have a WIN32 build environment, no. I've added in your changes, but won't be able to test on that platform.


do we need this?


It certainly makes sense for public-facing code (when I move the code into separate libraries later). It's probably a good habit for the rest of the code too.


remove?


Merde trompé


I don't understand the significance of this change.  You say that this improves reuse, but there's only one use in this file, and it's in an anonymous namespace so it can't be exported to other files.


go suppr :hap:


Yeah, it looks like this is off by one (after the fix to the `\r` check).  I'm a bit rusty on this code, but here's what's going on.  I'm just going to run through the function with an arbitrary string to illustrate, but the goal is for `loc` and `pos` always to reflect the same string position.

Let the string `ctx.src` be: `abc\r\ndefg\r\nhijk`.
Let `pos = 12` and `loc = { row: 3, col: 1 }`.  They both point at the character `i`.  (Note that in sourcemaps, rows are 1-indexed and columns are 0-indexed.  How annoying is that?)

Now `prefLF = 10`, pointing at the `\n`.  Then it's updated to correct for the `\r`, so `prevLF = 9`.

I'm just going to ignore the next block because it's irrelevant to checking for an off-by-one.

So the question is, what value of `loc` is equivalent to `pos = prevLF`?  In this example, `prevLF` points at the `\r` at position 9.  This is equivalent to row 2, column 4.

Now, the last index of `\n` before `prevLF` is at position 4.  So, `prevLF - ctx.src.lastIndexOf("\n", prevLF) = 5`.  We should be subtracting one to get the actual column number, 4.  (This is equivalent to the difference between the position of `\r` and the position of the first character in the column.)


Ok, so if the player selects some chest multiple times, then he/she will be given different items? Some items could be the same, like if player selects the chest twice, and gets empty(or zombie, etc.) each time?
Basically you're not using user input (u_input) in the selection process.
The way as I see it, when player walks into the room (or maybe when the game starts), items will randomly distributed across treasure chests and will stay the same until the game is finished.
What do you think?


is that expected?


These two `ofPoint`s must be `ofVec2f`s because `ofPoint::angle()` will not return a negative number (which seems to me to be a bug).


I hope this is OK? I couldn't work out why we were converting to a name and back to an address again when matching.


I would have put the colon on the line before (general principle is split after an operator).


wat


Am Donnerstag, 12. März 2015, 10:43:19 schrieb M.Jerger:
> warum das?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/DomainDrivenArchitecture/architecture/commit/4492c00b1285
> 24371fab834b7db08336e0dc5c07#commitcomment-10169932
weil ichs schön finde :-)

--
meissa GmbH
Telefon:
+49-7121-578913
Sitz d. Gesellschaft:
72770 Reutlingen, Zeppelinstr. 13
Geschäftsführer:
Michael Jerger
Registergericht:
AG Stuttgart, HRB746775
UstID:
DE292015368


Yup -- my thinking is along the lines of PEP-8's "weak 'internal-use' indicator". This is not part of the external interface, but is used within subclasses. I am happy to follow convention if there is one.


Add this to the index?


`git mv -f`


This is another thing we should RFC if we want to start calling it a standard (though in this case, I don't have a strong opinion).


This looks familiar.  Any chance you can consolidate the code some more?


hello testing changes from a previous escapade i forgot to revert


Is this the issue with the web client asking for client certificates that @gsfr mentioned?


Woops, it looks like it breaks the build.... Not sure if this is legit failure or just a flake, though...


Don't require these here, as it will mean Staging and Development need this configuration information.


The key `bucket_name` should be part of the `Figaro.env` method. Also make sure that those method names are lazy required (by appending a `!` at the end):

```ruby
access_key_id: Figaro.env.dragonfly_s3_access_key_id!,
secret_access_key: Figaro.env.dragonfly_s3_secret_access_key!,
root_path: Figaro.env.dragonfly_s3_root_path!,
path: Figaro.env.dragonfly_s3_path!,
url_host: Figaro.env.dragonfly_s3_url_host!
```


Sure, I'm adding that one and resending. Thanks!


Ditto on consolidating.


`virtual`


thanks! fixed.


Move the "require 'dragonfly/s3_data_store` into the else block; Otherwise it may bomb on the staging environment as the s3_data_store gem is not available.


Reduced the z-index of table list


i would put this in your css file as a class and then use the addClass function in jquery to add the color.


a per module version of handlebars is a likely scenario, yes.


cursor adjustment when table list is in right side


Need to add "Lesser" to before General in 3 places for LGPL licensing.


I take it these are now flags that get passed to uwsgi? Why is that the case?

I would much rather NOT store exec-wrapping flag names in a config file; this makes the CLI target inflexible.


Also, can you show me where they're consumed?


Mmm, I've just remembered that we added `Element.ALLOW_KEYBOARD_INPUT`to fix
https://github.com/CartoDB/cartodb.js/issues/235. Does your change breaks the fix?

Also: there's an extra "E" before the URL of the stackoverflow page.


and by module, I mean each local declared in the root component (the app)


Setting/removing a [mutation observer](https://developer.mozilla.org/en-US/docs/Web/API/MutationObserver) might be better here.


@stephywells - This is working, but I haven't updated the minimized js file yet. I wanted to ask you about some things first. Is it necessary to have the if ( thisCalc.calc_dec ) and if ( thisCalc.calc_prefix) lines? I was wondering if I need them since fields that have not been updated will not have the calc_dec and calc_prefix saved in their field object.

Also, since this option is now added, can we remove the .toFixed compatibility?


Reason behind this change?

I don't much mind what format we use, as long as the config format is diff-friendly and supports comments.
Depending on your parser, JSON is neither, making it a poor choice for human-relevant config files.


Got it.  Was trying to be consistent.  Would prefer to use ofVec2f throughout, but the Quad plugin uses ofPoint ... so ... anyway.


This time, the question is just a question out of curiosity, not a task-assignment in disguise. :-)

I just reviewed the changes before updating the plugin (which I already did). But this is mostly curiosity, I'm not a Java-Programmer at all.


Some very small changes. It is almost the same, albeit the method in which we now access components.


but why you should use multiple handlebar versions in your app?


I see that this was replaced for slots, but why is this in another commit? it makes more sense for a commit to have both remove and replace changes, otherwise this was confusing for me to read.


Forgot about the values on this one - but should be automatically updated before release


Stroustrup (4th ed, \S20.3.2) explicitly recommends otherwise:

> We can, but do not have to, repeat `virtual` in a derived class. I don't recommend repeating `virtual`. If you want to be explicit, use `override`.

I don't think it bothers the compiler. Again, though, I'm happy to go with LSST convention if there is one.


Should (can?) this be written as

    for (auto inSrc = inputCatalog.cbegin(), outSrc = outputCatalog.begin();
        inSrc != inputCatalog.cend() && outSrc != outputCatalog.end();
       ++inSrc, ++outSrc
    ) {

?

Might use an `assert` on the lengths too.


I don't know about an issue with that or what this does. I just wondered about the inconsistent change.

Then again, I can only read Java, not write it. The tooling required or available is completely unknown to me.


This commit appears to be a style overhaul and several feature changes combined :/

I notice at minimum various prompt changes and a data folder rename; what did I miss?


VSHPWH (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1657 of 1659 tests passed)

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


Does this prevent new users from having to click that eye UI thing?


commented code, not good. always `git diff` or `git diff --cached` before you commit. i'm guessing the unnecessary diffs on gradle files created too much noise.


Skal vi også have noget om hvad en compiler er? De 5/6/7?!?! faser i en compiler? Måske noget PBL ?


again, commented out code.


remove extra attributes


Oh wow, good catch :+1:


what is b? and is this used?


tag field is not critical and used only for maven-release-plugin. Will debug next time.


Dette kan scrum jo ikke alene, eller kan det? Måske skal vi have noget fra bogen/bøgerne om compilere med?


I'm not saying I would - but let's say core was released and tested but a minor change to signup was released a few months later. the reason for my using bundles was to have standalone builds that would not need to be qa tested if the dependencies were self contained. so home would stay at its version and signup would go through the QA/testing processes. in that scenario, core and signup might have different versions of handlebars (assuming the plugin supporting handlebars was updated).


`s/plz/please` ;)


yes.  AFAIK, this problem is specific to Safari.  Safari prompts for permission to send a client certificate, or prompts to select which certificate to send.  If the user 'denies' the certificate, then the SSL negotiation fails, and "Safari Can't Open the Page".

Whether we serve the API over 443 or 8443, doesn't matter.  Safari still does the wrong thing.  So we might as well keep things simple/standard.


Gentager den forrige setning uden at tilføje noget.


Do we want to make this configurable?


It certainly doesn't bother the compiler.  We should revisit this question in the context of C++11 where you have more control.


So I don't think we discussed the case when you need the applied operations when one want to react on a failed operation. Care to elaborate?


thank you my friend


Ne marche pas sur écran 720p...


Good choice!


VSHPWH (mbadams5) - i386-Windows-7-VisualStudio-12: OK (1657 of 1659 tests passed)

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


overvej evt. at omskrive dette til noget mere formelt: The group members are limited by the courses and classes in the semester [...]


I had pointed out to Emmanuel that it was kind of strange to be storing the empty positions as opposed to only the ones that had been taken. Converting to store only the taken positions seemed to incorporate both your and my feedback.

As a side note, I am not a fan of the `Board` tracking `"X"` and `"O"`. To me, `"X"` and `"O"` are view concerns, and I typically make my boards keep track of players/participants.


Ok. Should ``group`` also be parameterized? It is currently fixed to the string ``"Pattern"``.


this is still returning slots.size. what's the point of this method?


What plugin?


this is not an ideal scenario, but something we have encountered in the past and this is how we've been handling it. in normal practice, we release all bundles together or bring all modules up to date as quickly as possible and put them through the QA grinder


Hvorfor er agile/scrum godt, (fordi det er iterativt og så kan man gå tilbage og tilføje features) men er det begrænset til agile, er det ikke alle iterative udviklingsprocesser.


Can you support this change with some benchmarks that show it is beneficial?  I seem to remember that a higher value for inlining-limit decreased performance by overly inlining functions.


It crashed because you tried to shadow an abstract method.
Correct would have been
````
@Shadow
public String[] dropFirstString(String[] par0ArrayOfStr) { }
````


For now, it's pretty standard in our code to use `virtual` in the definition of overrides.  In the C++11 future we may want to switch to thew new `override` keyword or both `virtual` and `override`.


Your not dumb.  I've made plenty of stupid mistakes.


Can you describe the situation you encountered where primitives were not inlined?


\o/


I used `double` because that's what `Calib::getMagnitude()` returns. Whether that's actually necessary, I will check with those who are more expert than I (that means you, @RobertLuptonTheGood or @TallJimbo!)


looks good to me ack


I would prefer `ofVec2f` as well. Perhaps we could change to using `ofVec2f` and just cast ofxQuadWarp's `ofPoint` to `ofVec2f`s. Either way that should be a post-workshop task.


Oops. My bad. Done now.

On Thu, Mar 12, 2015 at 10:42 AM, Benjamin Bannier <notifications@github.com
> wrote:

> Please also push the required rct commits.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Andersbakken/rtags/commit/851df6edc071f928f22da5ca31a7677328943ccf#commitcomment-10169900>
> .
>


Sounds good, we can adjust this in the future, opened #8.


Dog evt. en notits om at vi ikke har uendlig tid som det sker i nogle scrum udviklinger, altså iterere over et produkt for evigt, men at vi kun realistisk har til start/midt maj.


Samuel, the LM2576 already exist in KiCad libraries. You can find it (and many more buck/boost converters) in `dc-dc.lib`.

A few years ago I created a 'dc-dc.lib' library especially designed for all kinds of DC converters. Library 'regul.lib' was to remain for linear regulators. Also AC-DC converter controllers have their own library: `ac-dc.lib` with one exception of Power Integrations controllers which have separate lib.


oops


VSHPWH (mbadams5) - x86_64-MacOS-10.9-clang: OK (1651 of 1653 tests passed)

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


Getting notified upon errors during flushes =>

- Getting notified upon errors during operations in the datastore
- Getting notified upon errors

My point is it's per tx, not per flush.


Yeah, those are all good ideas.  One other thought I had, was disabling just a certain axis completely could give you the option of using free-look to move it manually when it needs to be moved.  For instance, I find the fact that the camera moves up and down while jumping in Super Mario Sunshine to be nauseating.  The game would be a lot better if that was stabilized, but sometimes climbing very tall things still requires the camera to raise a bit.  At those times, the user could use freelook to move it manually.  Not perfect, but better than feeling sick.


~~k~~new.
"implement this new knowledge in our project", by going back and changing mistakes or adding features.


Can you use `label_method: :predicate_value` instead of the lambda? Its less visually chatty.


i had php error with wrong function parameters structure (strict one) ;/ In parrent function it's FlattenException not  \Exception not surewhyi removed it not changed to FlattenException. Try to revert it and test.


@joschiwald it's done for characters created before this fix. log char  play 1 or more arena. Turn server off, start it and login.


Can you use `value_method: :predicate_value_code` instead of the lambda? Its less visually chatty.


89911514-Allow-OtherSideCoefficients-in-AFN (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1656 of 1656 tests passed)

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


We have some tests that are specifically designed to check if None handling works. Though they also increase the robustness of our tests. Should we remove them?

Other than that ack.


Overall er det kort og godt, men der er ingen ulemper ved scrum nævnt, er det så fantastisk? Måske er der problemer ala. man ikke har en konkret overordnet tidsplan og samtidig en meget hård deadline.


you need to say that flush usually happens either before a query, when the tx commits or when the user explicitly calls flush


ack.


No opinion.  Doesn't hurt to parameterize it, I guess, and you could also hide that too with precedence=-1.

The label should probably be the name of the object, not just the name of the class, except that you probably chose the class name because you didn't want to see the auto-generated unique names.  We already have a test in Param's script_repr to see whether the name was auto-generated or not; we should probably make that test public or provide some method or function for Parameterized to return a suitable name for this purpose, i.e. the name if set by the user and otherwise the class name.  We do set the names of Pattern objects in some places, and I think it would be surprising to people that the HoloViews object doesn't use that name.


What is the expected array? Assert that.


@ikeough Is this merged?


That's a very good point. Fortunately numpy's power implementation is more precise than using exp and log to accomplish the same thing (and faster :)):
```python
In [34]: np.max(np.abs(r1-r3))
Out[34]: mpf('1.1368683772161603e-13')

In [35]: np.max(np.abs(r2-r3))
Out[35]: mpf('1.6391277313232422e-7')
```
where `r1 = a ** b` and `r2 = np.exp(b * np.log(a))`, `a` (>0) and `b` big random arrays, and `r3` is an exact power using mpmath's big floats.


need this...


Think of it as a poor man's transaction giving you some flexibility on what to do upon failures.


`access_key_id: Figaro.env.dragonfly_s3_access_key_id!`


They would fail if I didn't initialized the section dicts with `{"default": Section("default")}` before so why not keep them.


@xach Thanks! will do


`secret_access_key: Figaro.env.dragonfly_s3_secret_access_key!`


But then we have no idea if we're modifying the value of the map, do we?  The map is free to return a copy of the int, as far as I can tell.  (Specs are [here](http://golang.org/ref/spec#Index_expressions).)

Ah, indeed, [a short test](http://play.golang.org/p/5DHnWp_NwZ) says it does return a copy (`m["bar"]` isn't modified there), which explains the sore back you wrote... which then breaks the whole thing because writing into the map wasn't atomic.

We might need to make this a `map[string]*uint64` instead :disappointed:


maybe a switch case ?


Thanks, that was what I thought. Just to clarify, the problem is really unrelated to the "\r" problem, right? Take `abc\ndefg\nhijk`: `prevLF - lastIndexOf = 8 - 3 = 5`, should also have been `prevLF - lastIndexOf - 1 = 8 - 3 - 1 = 4`

Another way to verify this is to see what happens when `prevLF` points at the first newline in `ctx.src`. In that case `lastIndexOf` returns `-1` so it's also clear we need to add `1`.

There are only two hard things right? Naming things, cache invalidation and off-by-ones.

Commited fix in ba5c3a0c377ec73ca4ff7498e858fadb3d5c2814


If you don't mind, I'm pushing finer grained commits to fix the warnings first.

I'm changing this to use pread()...


Yes. In fast-gcd I define a lot of accessors for 2x2 matrices, and specialized multiply upper and multiply lower routines, and none of these were inlined because of the (##declare (not inline)) in define-prim.

It took me hours to figure that one out.

Brad

> On Mar 12, 2015, at 2:02 PM, Marc Feeley <notifications@github.com> wrote:
>
> Can you describe the situation you encountered where primitives were not inlined?
>
> —
> Reply to this email directly or view it on GitHub.
>


VSHPWH (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (639 of 639 tests passed)

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


onrollback which is called when the transaciton is rollbacked (either by the user or by the container).


Nit: it's not good style to put essential calls inside an assert(), in case we're compiling with NDEBUG.


let's => lets


89911514-Allow-OtherSideCoefficients-in-AFN (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


up to you. if you do so , you ll have to change how you list them, keys are
unique in a hash.
just use an array of string ["some_suffix_name", "another_name"], and
you can use something like ' self.send @compound_suffix, param ' to call a
function with the same name (no space)
if you need to have something specific done when there's a match
def some_suffix_name(opt)
puts "hello "+opt.to_s
end


nice, thanks


Ahh yes...cool


"with the application the changes of the current flush cycle or abort it."

all the changes? Isn't it call for a specific failed change? That's not clear form this sentence


Let's link to issue #16 from here. GitHub markdown doesn't appear to automatically do this for README files.


Same comment as below.


Yep, you're quite right.


Хорошая мысль.


That one is quite tricky - I was surprised as well.

The problem is that `vc_bvLtExpr(vc, shift, bvConst32(vc_getBVLength(vc,shift), width))` returns a pointer to an STP expression which should be deleted after used in subsequent expressions.
Providing it explicitly as an argument to `vc_iteExpr` doesn't delete it later. There it gets leaked.

The ExprHandle as a typedef of the ref<> template takes exactly care of that as soon as the instance of it goes out of scope.


:ghost:


Introduced regression #137


I want all of the keys that you are using to configure the AWS instance to match the parameter option. In the above cases you are saying that the `secret_access_key` value should be in the `Figaro.env.dragonfly_s3_secret` variable. Why not in the `Figaro.env.dragonfly_s3_secret_access_key` variable? It keeps the variables closer to their meaning. The prefix (`dragonfly_s3`) is what the key is for. The suffix (`secret_access_key`) should map to the configuration option's key (i.e. `secret_access_key:`)


kson is part of the python standard library.  switching to json lets us drop an external dependency.

Instead of one gigantic string, the key/values are separated onto multiple lines.  Is that not diff friendly?

I don't think the config really needs to support comments, but that's just my opinion.  We can definitely discuss this more.


The LTM4609 is a DC/DC converter so it must be placed in `dc-dc.lib`, not 'linear.lib`.


@brosef I'm pretty sure I'm not supposed to be able to see this anymore. Or the tons of other things in the https://github.com/lumoslabs. It looks like I was successfully removed from lumos_rails though.


It does not really makes sense to copy the next received value since that is coming directly from the input (but you are right copying at the collect method). You should actually copy the accumulator value instead in order to guarantee that you dont mutate it afterwards.


I would give a list of concrete examples of things that can be done:

- retry operation upon time out
- log successful operations in a log for manual rollback
- ...


90019124_RadianceData (rpg777) - x86_64-MacOS-10.9-clang: Tests Failed

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


Clock buffer? Isn't that highly platform-dependent too?


@chrishanw
Here is what I have played around with while waiting for trial simulations to finish.

The content was written to be able to see different layout aspects (pseudocode, formulas …) in action and can be changed or entirely removed to fit the final version.

The above needs to be compiled with `xelatex documentation.tex` which shoud be part of every TeXlive installation (just as pdflatex).


I don't want to keep those flags in the config file either. this was just a temporary solution.  They are consumed when generating the fig.yml from template.


89911514-Allow-OtherSideCoefficients-in-AFN (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Coverage Too Low

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


We should also add a fold method to the scala api which has the same signature as the scala foldLeft method which can be applied on collections (which then can take lambda functions too)


smoke_detector is not private. I think you're removed from the lumos org.


Maybe give example of exceptions to react to even if that includes MongoDB or Infinispan specific exceptions


def prime(2, nmax=10):


My general point is that this doc so far is either generic or shows a specific example, we need to encourage people to dream about what they can do.


wrong indentation


some code; fixes #1


@void64 *(not me!)* did a nice job on resolving #1 :zap:.


Oooh okay. Coolios. Nevermind then.


it looks pretty here, but do we *really* want to start using single-line if constructs with no `{` and `}`? ;x


Вот тут не находит


A lot of code is duplicated here. Similar code is used in the home view as far I know. We need to factorize a bit.


Also, careful != `using namespace std;` :-)


Hi! Why my solution has been deleted?


yes, forget Clock Buffer (and it's most of the time inserted automatically by toolchains...).
But DDR input / DDR output can be useful.


Yes, it's not needed any longer, as it is set in the actual bootstrap already.


Ok.


In case you decide to CONTINUE, you wouldn't have a handle to the applied ops otherwise (onRollback() wouldn't be invoked in this case). I think it makes sense e.g. for logging purposes.


So we atomicly store the new offset(childval) against the local ```var offset int64 = 0```, then the atomic operation gives the offset a new memory address, and then we assign the new address against the map ```tailNode[childKey] = offset```
Also looked into the updating the offset to unsigned int64(since we always are going to do seek_set positively and makes sense) but after digging a bit deeper in tail.go I came a cross this:
```
// SeekInfo represents arguments to `os.Seek`
type SeekInfo struct {
	Offset int64
	Whence int // os.SEEK_*
}
```
:scream:


Somewhere I would explain how "transaction" works. Explain that user must demarcate transactions via the explicit API, or JTA or let their container demarcate transactions.

Explain that in case of single failure, you can fail or succeed. If it fails, then a regular exception will be propagated.
And that the transaciton protected by some kind of try catch will call rollback() at some point.


Shouldn't this example be something like:

    let g:syntastic_shell = "/bin/sh"


`microchip.lib` does not contain any PIC Microcontrollers anymore. Use separate libraries for PIC10, PIC12, PIC16, PIC18 and PIC32 families.


We should not allow inbound SSH from just anywhere - should be pinned to our jumpbox ip


right, will fix


Hey,

Good job, I like it very much. :+1:  I added some minor comments.

There are couple of things that would be nice if you could add as a next commit :)

-It would be cool to have a grouped version of this operator that would work on GroupedDataStreams, like the GroupedReduceInvokable, this should be a trivial extension

-Also please do me the favor and integrate the fold methods in the WindowedDataStream, this also should take about 10 minutes.

I think that similarly to reduce, the most used versions in stream processing would be the grouped fold and windowed fold operations.

Thanks again for the contribution, once you added these I would like to merge them right away :)


The cmpiled version is not updated because the update is not finished :)


We have an entire section on flushes right prior to that one. Do you really think we need to repeat that here?


I would agree with @Diaoul, let's try to keep the same approach over the various packages that have backward compatibility issues, makes things easier on maintenance.


Did. Good catch!


hum, I don't know, if that does not add object creation cost, why not.


david this code is a shit


damn spaces...


It's easy to change the text based on the radio button state, so let's do that.


The only real changes were in _content.scss right? Looks good to me!


Sure.


I would avoid the term "transaction" to describe any of the supported behavior. Some specific examples are given, right? I will add the logging one, although it also comes in the code example below.


Why is `flynn-release` the input?


I have finally pushed up the changes you suggested.


Because the link leads to a 404. I cleaned out dead links. Here's the tic tac toe link you gave: https://github.com/dstodolny/tictactoe/blob/master/tictactoe.rb. If you want to update the link you can re-add your solution.


This is ported over from the old rule. I believe it's because the installer imports it, but I'm not certain why. We should look into that separately.


Hum I guess not but then I think I would have expected that feature to be top level instead of buried 2 levels down. WDYT?


`browser.currentURI.spec.asciiSpec` instead of `browser.currentURI.spec`?


Ok