```
if(num % (3 * 5) == 0) return "fizzbuzz";
```
This is also a possibility so that way you simplify machine instructions.

At a minimum this check case happens at most once in every 15 operations so this block gets hit 14 times that it doesn't need to.

If we mod by three, it happens every third time, which is more common.
If we mod by five, it happens every fifth time which is also more common.

Our problem with the above is our approach would have to be modified.

There are many ways to skin this cat so I won't drone on about them as this solves the problem but just things that crossed my mind as I thumbed through this.

You could also be a goofball and make it functional!
```
public static void Main()
	{
		Enumerable.Range(1, 100)
                .Select(DetermineFizzBuzz)
                .Select(FizzBuzz)
                .ToList()
                .ForEach(Console.WriteLine);
	}

	public static string DetermineFizzBuzz(int input)
	{
		return String.Format("{0}{1}",
							 input % 3 == 0
							 	? "Fizz"
							 	: string.Empty,
							 input % 5 == 0
							 	? "Buzz"
							 	: string.Empty);
	}

	public static string FizzBuzz(string fizzOrBuzz, int index)
	{
		return String.IsNullOrEmpty(fizzOrBuzz)
			? (index + 1).ToString()
			: fizzOrBuzz;
	}
```


da-DK


./configure --with-icc-asm


@jonmeredith, @JeetKunDoug The legacy gossip removal patch, being reverted by this commit, still remains in develop. I suppose it should be reverted there, too?

On the other hand, I'm trying to understand what API breakage it was. There are three cases where exported functions get different signatures or removed:
* `riak_core:join/5` -> `/4`: the only call site is in `riak_core_gossip`, and there is a corresponding change in it;
* `riak_core_console:print_member_status/2` -> `/1`: single call site in same module, changed accordingly;
* removal of `riak_core_gossip:legacy_gossip/0,1` and `any_legacy_gossip/2`: all calls to these are eliminated in the commit.

I'd rather err on the conservative side than be seen as trying to lobby my commit through: but I am still curious about the particulars of the breakage @JeetKunDoug has observed. And in any case, alert to the fact that this legacy removal commit exists unreverted in develop.


Soon we will launch our new website, with also new favicons, I would appreciate if you could optimize the PNGs then again :-)


I guess you could remove the || true now.


@h-kataria you may want to rename `datatype` to `t` for compactness.  Love all the code deletion!!


Oops! Fixed in 1c46d17. Thanks for catching.


Називай всі змінні з однаковим початком, наприклад STATUS_


J'ai poussé sur cette branche deux autres commits qui trainaient sur mon PC, dont le renommage `mods/perl5` -> `mods/perl`.


@chessbyte renamed datatype to t


I found that by reading the gitHub commit changelog :dancer:


ooooh. did you come across this because of a bug, or did you discover it randomly in code?


wait, ignore me, didn't read the description


we don't we check for `r` as well?


Fixed.


Don't want to just use `trimLeft` & `trimRight`?


will this add orderedTable and core again? That may lead to trouble... please make sure it works


the second one should probably be `rhs_tail` - did you ran analysis again with your changes?


    A-z == A-Z + [ + ] + _ + ^ + `

`[\w]*` can be more clear.

Also, one of these has a trailing `$`, while the other does not. That's important because line 67 has `prog.match()`. I *always* use `re.search()` instead of `re.match()` because the latter is oddly, asymmetrically anchored.


It needs manual intervention after installation, though.


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

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

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

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

```


In the few previous lines, I check to ensure it doesn't already exist before adding the dependencies.
Am testing anyways.


Yeah fair enough, visual studio has been getting better with respect to
C++. I actually use SlickEdit myself. If I could work out a way to
integrate the gui build into visual studios would you be interested?

On 25 July 2015 2:02:25 am Duncan Ogilvie <notifications@github.com> wrote:

> Visual studio is a far superior tool for development. I wanted to compile
> everything in visual studio, but that doesn't work very nicely so I
> decided  to use qt creator. You can use qmake to generate visual studio
> project files if you want.
>
> On Fri, 24 Jul 2015 01:14 fileoffset <notifications@github.com> wrote:
>
> > Yeah that makes sense. I actually didn't have any spaces in my path, but
> > the files were still failing to copy unless I wrapped them in spaces. I
> > didn't think to put the spaces in the qt gui.
> >
> > Btw Duncan, I've been meaning to ask you: why do you compile the back end
> > dll's in visual studio, but the gui in qt creator? I was going to try and
> > push the whole build into qt creator land to make the build setup and
> > maintenance easier. If you're interested in that let me know and I'll have
> > a crack at it.
> >
> >
> > On 24 July 2015 8:37:18 am Duncan Ogilvie <notifications@github.com>
> > wrote:
> >
> > > You must use quotes in the qt creator gui. Otherwise it will split the
> > > arguments anyway.
> > >
> > > On Thu, 23 Jul 2015 23:43 Nukem <notifications@github.com> wrote:
> > >
> > > > Add on ( @fileoffset <https://github.com/fileoffset> ): This will
> > never
> > > > work easily without quotes. A space in the directory path causes it to
> > be
> > > > interpreted as another parameter. Adding quotes in the .bat won't help.
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > >
> > > <
> >
> https://github.com/x64dbg/x64dbg/commit/f4970331e9dd30479fb5b28419d69eae23851a7f#commitcomment-12331251
> > >
> > > > .
> > > >
> > >
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > >
> >
> https://github.com/x64dbg/x64dbg/commit/f4970331e9dd30479fb5b28419d69eae23851a7f#commitcomment-12332162
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/x64dbg/x64dbg/commit/f4970331e9dd30479fb5b28419d69eae23851a7f#commitcomment-12332708>
> > .
> >
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/x64dbg/x64dbg/commit/f4970331e9dd30479fb5b28419d69eae23851a7f#commitcomment-12346408


Looks like :-( I can fix it tomorrow sorry about that


I am wondering what the significance for these ranges are... How come some multibyte characters are not serialized to hex?


one of the next onces should dump core in that case, might be better - not hiding the impossible error. Or use `assert`


Q.v. https://github.com/uwescience/myria-docs/blob/master/raco.markdown


again - if this is impossible, it's better to `assert` on this..., not hide the error


just whitespace change?


Поки в нас є скінченна мнжина можливих екшинів:
EMAIL_SENT
EMAIL_OPENED
EMAIL_CLICKED
EMAIL_UNSUBSCRIBED
SEND_TO_REWRITE
REWRITEN
REGISTERED

Поки це все


:smile: Didn't know that was part of ES7.


if null, it is very bad - assert?


@heliocastro Is this Ok for you ?


Are you sure this works? I think argparse passes the argument to the 'type' function as 'str'. I think you should simply cast to 'float' in `_validate_option()`, rather than testing for the type. No?


hmm can this be a non public fn?


This revision is clean.


How do you automatically trigger bower install with npm install?


Refer to story https://www.pivotaltracker.com/story/show/99723656


мови прийнято зберігати у форматі малих букв (ua, ru, en, pl, de)


else what shall we do? Is this short read, or read error? If the latter, just fail properly.


The removal of the old apk is missing. Please amend the commit with the deletion and push --force it to the repo


Great job!


Awesome!!!


це поле не має бути унікальним


@cogmission Hi David! Yes. In this experiment, the SP is trained before the TM which is itself trained before the CLAClassifier. The idea is that by phasing out the learning like this, you get "cleaner" input from the previous level (SP => TM => CLAClassifier). For example, once the TM is trained, you get less bursting columns, and that gives you better inputs to feed to the CLAClassifier -- in the sense that you have more predictive active cells and that the representation is sparser. Whereas, if you were training them all at the same time (i.e. the CLAClassifier is learning while the TM is still learning too) then you are training the CLAClassifier with less specific input -- because more column are bursting. Same idea with the SP to TM, you want to separate the learning so that each learning phase is cleaner. Does that long blurb answer your question? :)


`1000` ain't much these days


Hi Marcin,

I’ve just compiled and analyzed libsass with Visual Studio 2015 trail. My fork commit was with VS 2013.
Still getting warnings and code analyze issues so maybe you are right and I forgot to rerun it.

I will try fixing the warnings and issues in VS 2015 tomorrow and will re commit my changes.

Thanks for the feedback.

Kind regards,

Henry

> On 24 jul. 2015, at 23:54, Marcin Cieślak <notifications@github.com> wrote:
>
> second


Test!

# Cool


else? is it critical bug ("should never happen") or some situation we should handle, like bad input?


назви це поле краще updated, бо date_joined не несе ніякого смислового зачення для моделі Екшин


this is so much fun.

autogenerating tests of all of the permutations of input arguments!

```
♡ nosetests plotly/tests/test_core/test_image/ --pdb -v
_test_image_get_returns_valid_image_test(jpeg, 300, 300, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(jpeg, 300, None, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(jpeg, None, 300, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(jpeg, None, None, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(pdf, 300, 300, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(pdf, 300, None, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(pdf, None, 300, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(pdf, None, None, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(png, 300, 300, 5) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(png, 300, 300, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(png, 300, None, 5) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(png, 300, None, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(png, None, 300, 5) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(png, None, 300, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(png, None, None, 5) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(png, None, None, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(svg, 300, 300, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(svg, 300, None, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(svg, None, 300, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
_test_image_get_returns_valid_image_test(svg, None, None, None) (plotly.tests.test_core.test_image.test_image.TestImage) ... ok
```


WTF?


:abc:


No worries at all, it can wait until monday :+1:


пробіл після лапок


Last time I looked into the 4k,  was last autumm I guess.  Then we fixed some bugs and it worked well.

@mirjam234  If you don't write a better description on what you did, traces,  there is no help to give you.

@buggii  Easy now, everyone gets frustrated, especially when they haven't had time to learn to use the PM3.

The old quote from the Wiki https://github.com/Proxmark/proxmark3/wiki
"It should be pointed out quite early that the proxmark3 is not really for beginners. If you are not already fairly familiar with electronics, embedded programming, some RF design and ISO standards, this device will probably bring you more frustration than anything else ! Users that do not understand the basic principles behind RFID may have difficulty using the device."


Use a npm module ```brfs``` to read shader files to build p5.js
By doing so, shader files could be saved as a more readable format instead of string.

Please ```npm install``` the module


передавай два параметри:
1. Ссилку куди саме треба буде далі редіректити
2. Замість емейла краще передавай id відповідного екшина який відповідає за цей емейл


Please note that this project is protected under Creative Commons BY-NC


change `should` to `is_expected.to`


Тут так само, треба лише id екшина


So much more clear.


Here too.


я сьогодні добавив собі таку штуку в партнерську програму
<img src="{{ open_api_url }}" height="1px" width="1px">


Wohoo! Thanks.


@mfonville Look's like it was cause of the upload script, see logs posted on issue opengapps/opengapps#73


This fix does not work for me. I place the sign expecting to not be prompted for text, but the game prompts me for text anyway.


oops, it's the credit algo, not claim


oh yeah. But there is an issue. If the existing modules include orderedTable, then line 204 will not add it. Now line 207 adds core, and then the remaining modules, including orderedTable, are added. So core would come before orderedTable? And that would make core fails to load?


What happens when there are more than 30 serializations happening?


Its still in proposal phases: https://esdiscuss.org/topic/string-prototype-trimright-trimleft


AFAIK we don't have any build triggering on this repo.

On Fri, Jul 24, 2015 at 5:42 PM, Ondrej Zizka <notifications@github.com>
wrote:

> Thanks jess. I wanted to check it here, hence no "ready for review" label.
> You were faster :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/windup/windup-rulesets/commit/aef5ef7d848f7157fbf46b139e3ff325819807f2#commitcomment-12352584>
> .
>


runWithInstanceFromPool would be a better name?


Would it be better to create a new "private val"?


This can be simplified to use Rails [.blank?](http://api.rubyonrails.org/classes/Object.html#method-i-blank-3F) method.

```ruby
  if !params[:upload_type].blank?
```


Hmm, right. I think that since `core` depends only on `orderedTable-hbase` and `orderedTable-hbase` depends on nothing, we can probably make `orderedTable-hbase` the first element of this list, `core` the second element of the list, and then simply filter it out if it already exists in the existing declared modules.
Thoughts?


Only row 129 added, everything else is the formatting from WebStorm


As far as I am understanding the whole thing, the quad_segmentation/sortCorners should produce surfaces that have equal mass center in the same orientation. This allows one to take advantage of the actual surface functionality. This is related to #164.


@avital Confirmed. I am no longer getting DOM Exception 8. But, I have another one for ya :)

Exception from Tracker recompute function:
Error: Invariant Violation: unmountComponentAtNode(...): Target container is not a DOM element.

This happens when I try to navigate to a route with a Blaze layout containing a React component for it to yield, but I deny the route due to the user not being logged in. I redirect the user to a login route, which is still just a regular Blaze layout + Blaze yield.


I think a better test strategy is to create test studies, verify that they show up in the list API, delete the studies, then finally verify that they don't show up in the list API. As is, this test has side effects and relies on state external to the unit test. This means that the unit test isn't self-contained and impacts test repeatability.


agreed. That should work. Since this dataset include pfs or tpfs, it needs both hbase-orderedTable and core


hi


pushed commit to reflect that.


Similarly, deleting all studies (either before or after) means this test has side effects. It's also dangerous if this test is accidentally run in UAT or Prod. It would be better to only delete studies created by the tests.


I'm a little intrigued that such a small change was the cause of a large performance change!

have you tried a numerical for loop and moving function calls out of the loop? something like this:

```lua
function DDoS:bpf_match(d)
  local rules = self.rules
  local len = #rules
  local payload = d:payload()
  for i = 1, len do
    local rule = rules[1]
    if rule.cfilter(payload) then
       return rule
    end
  end
  return nil
end
```

that should skip the repeated calls to `d:payload()`, which probably doesn't change, and it also skips calling the iter function that pairs returns.


cacheSolve function starts here


За оцей кусок коду тебе треба бити по попі!!!!!!!!!!!!!!
Ось як це треба було написати:

```
try:
    tutor = Tutor.objects.get(email=receiver_mail_from_request)
    Action(tutor=tutor, type='UNSUBSCRIBED').save()
    tutor.status = UNSUBSCRIBED
    tutor.save()
except:
    pass
```

А ти чогось витягнув з бази всіх!!!! репетиторів і йшов циклом по них


Why? That's not a reqirement. It will use the package.json description: https://github.com/yeoman/yeoman-generator-list/blob/bc3a300070e74f5ac94dde26d4d708a845790bc1/index.js#L14


Though this hack is actually messing up with the marker detection, the hack can be adapted, of course, to live in parallel with the marker detection. #164


Why can this not just be `default_value = AdventureBlock.xml_content.default` or even `return (value == DEFAULT_XML_CONTENT)`?


Some square invertible matrices to practice


not tested yet


Initialize the makeCacheMatrix with a sample


Читай коментар вище!!!!!


Знову ж таки читай коментар який вище


So the implementation of ResourcePool is based on ArrayBlockingQueue and uses its poll and offer methods. When the pool is empty a new instance is created without any blocking. Similarly, when there is an attempt to release the instance and the pool is full the instance is not added to the pool and the method returns right away (which means the instance is lost in that case).


@Sheeo Include this in the hotfix as well


I'm aware. There's several issues regarding that I'm just not in a position
to fix it right now.

On Sat, 25 Jul 2015 01:25 Robert Kummer <notifications@github.com> wrote:

> You have missed the update for the Console Command:
> PHP Fatal error: Call to undefined method
> Dingo\Api\Routing\Route::isProtected() in
> /home/vagrant/workspace/donepm/vendor/dingo/api/src/Console/Command/Routes.php
> on line 73
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dingo/api/commit/29084209b7e5f0fdd6af50f684c5159adebfac69#commitcomment-12345594>
> .
>


Мені задється що в тебе проблеми з відступами, перевір, всюди має бути по одному Табу (або 4 пробіли)


powrot


Should we be pro-active here and put this in a <code>#ifdef LINUX</code> or something? if nothing else, it's another reminder not to mess with that code in a year or so when we've forgotten all about this :)


Well, for some reason, if there's no description on Github, they don't get listed...

I agree this is weird, and we probably should fix yeoman-generator-list to not require this.


Yes, probably! To be honest some parts of it were copied from an existing solution in Spark core and I didn't change it.


What do these numbers actually mean? Could we put a one or two line code comment about this "just to be sure"?


Punch Through loves you


Yes, It's true. My bad


I was hesitating a bit and in the end decided that that I want to change as little existing code as possible, at least at the beginning.


OOverride


wrong number!


It would be nice if we have a "fault-case" in the commit log. What's the "default priority" priority here? Technically, it should be 0 (at least that's how/what process are started with). But is this different for kernel processes/threads?


Why did you change the css @yadomi  ?


Ok, so an instance will be provided without blocking all the times and it's just that it will keep at max 30 serializers in the pool. Cool, then.


:+1: :hankey:


This is exactly what fails indeterminately, as if it takes time to create studies and to delete them after the call returns. Very frustrating.


Text correction

Signed-off-by: Bhargav Bhikkaji <bharbhi@gmail.com>


Seems like a preferable solution, given that the strangeness of having `_setup_fetch_workers` not actually return a value is now contained entirely within that function.


I did not, `Jekyll serve` did it


You unnecessary made "defaultSerializer" var when it can be a val. I don't know Scala very well, but do you know if it's desirable to define a constructor param as "private var"?


Update Message

Signed-off-by: Bhargav Bhikkaji <bharbhi@gmail.com>


Again, a <code>#ifdef LINUX</code> or something, just to be absolutly sure we (or our successors) remember this?


Then, i'll merge it tomorow, just in case.
:shipit:


Nit: looks like `-no-reduce-relocations` snuck in here twice.


Do we need to duplicate these constants?


@scottschurr I think the magic you're looking for is here https://github.com/ximinez/rippled/commit/9a85fb8041fd60a652a81c2049c6f3c7e7247204#diff-535096304a45bfdaac2ed98a663f5d6fR133 and here https://github.com/ximinez/rippled/commit/9a85fb8041fd60a652a81c2049c6f3c7e7247204#diff-63e38aef10c237031210901286f42aa2R56

The intention of this change is that every call to `STTx::checkSign` is wrapped (directly or indirectly) in a check of the `HashRouter` for the `SF_SIGGOOD` or `SF_SIGBAD` flags. Notice that every call except the one in `Transactor` is _already_ wrapped in the `HashRouter` check, so this change is really just bringing the `Transactor` back in line, and hopefully reducing the chance that a signature gets checked more than once.

(If I may have missed any, please let me know, though!)


We like `JLOG`, its a lot more efficient especially when we call `toBase58`


string interpolation instead?


This creates an ambiguity. What about types T and U for which both `__basic_common_reference<T, U>::type` and `is_reference<__builtin_common_t<T, U>>::value` are satisfied. I suggest you revert this change.


@sellout: here you go. Now generating a simpler, non-fixpoint, type for AccumOp (see below).

I don't know, trying to actually use this, there are some rough edges. For one thing, you'd like to say `import ExprOp._` and be off to the races, but I'm having to define functions in a separate `object` which you have to explicitly import. There might be a way to turn that on its head, but it's not obvious.

Be that as it may, I think this is sufficient to generate usable a ExprOp and AccumOp. Do you agree?

Where does that leave us?

```scala
sealed trait AccumOpGen[A]

object AccumOpGen {
  object Types {
    final case class $addToSet[A](value: A) extends AccumOpGen[A]
    final case class $push[A](value: A) extends AccumOpGen[A]
    final case class $first[A](value: A) extends AccumOpGen[A]
    final case class $last[A](value: A) extends AccumOpGen[A]
    final case class $max[A](value: A) extends AccumOpGen[A]
    final case class $min[A](value: A) extends AccumOpGen[A]
    final case class $avg[A](value: A) extends AccumOpGen[A]
    final case class $sum[A](value: A) extends AccumOpGen[A]
  }

  object $addToSet {
    def apply[A](value: A): AccumOpGen[A] = Types.$addToSet(value)
    def unapply[A](obj: AccumOpGen[A]): Option[A] = obj match {
      case Types.$addToSet(value) => Some(value)
      case _ => None
    }
  }
```


I don't it well either, I learned it this week and this is my first change :D I guess we'll have to ask someone more experienced. I understand your concerns, the semantics have changed a bit.


@bradenmacdonald It probably can be. I'll see about simplifying it, then, but I may not hit it til Monday.


If you're not using these arguments, just take them as `*args` and pass them on as `*args`.


That's the right idea, yes. Your "python3" part actually requires Python 3.3, but there's not really a reason to support anything older.

Cython uses this code to adapt to PEP393:
```
/* new Py3.3 unicode type (PEP 393) */
#if PY_VERSION_HEX > 0x03030000 && defined(PyUnicode_KIND)
  #define CYTHON_PEP393_ENABLED 1
  #define __Pyx_PyUnicode_READY(op)       (likely(PyUnicode_IS_READY(op)) ? \
                                              0 : _PyUnicode_Ready((PyObject *)(op)))
  #define __Pyx_PyUnicode_GET_LENGTH(u)   PyUnicode_GET_LENGTH(u)
  #define __Pyx_PyUnicode_READ_CHAR(u, i) PyUnicode_READ_CHAR(u, i)
  #define __Pyx_PyUnicode_KIND(u)         PyUnicode_KIND(u)
  #define __Pyx_PyUnicode_DATA(u)         PyUnicode_DATA(u)
  #define __Pyx_PyUnicode_READ(k, d, i)   PyUnicode_READ(k, d, i)
#else
  #define CYTHON_PEP393_ENABLED 0
  #define __Pyx_PyUnicode_READY(op)       (0)
  #define __Pyx_PyUnicode_GET_LENGTH(u)   PyUnicode_GET_SIZE(u)
  #define __Pyx_PyUnicode_READ_CHAR(u, i) ((Py_UCS4)(PyUnicode_AS_UNICODE(u)[i]))
  #define __Pyx_PyUnicode_KIND(u)         (sizeof(Py_UNICODE))
  #define __Pyx_PyUnicode_DATA(u)         ((void*)PyUnicode_AS_UNICODE(u))
  /* (void)(k) => avoid unused variable warning due to macro: */
  #define __Pyx_PyUnicode_READ(k, d, i)   ((void)(k), (Py_UCS4)(((Py_UNICODE*)d)[i]))
#endif
```
You could do the same in a C header file to #define `PyUnicode_GET_LENGTH()` and `PyUnicode_DATA()` if they are not available (i.e. in Py2). Or you could just declare the mangled function names as provided by Cython and use them, thus taking the somewhat low risk that they might change at some point.

If by "IF" etc. you meant the Cython upper case "IF" statement, then no, you should avoid that. Platform adaptations are best done at C compile time, not Cython compile time.


This looks pretty good.

Need just a couple changes to accept:

- Match original formatting. Use real tabs instead of spaces please.
- Lets use full name for latitude and longitude. There are so many ways people like to abbreviate those words I don't want to pigeon hole into a particular camp.
- Set a maximum precision on the values to 6 decimal places. With geocoding large, non-circular geographical boundaries to a single GPS point, anything beyond 6 decimal places gives a false sense of accuracy I think.

Thank you!


@tarsius I don't think you need to read this as a personal insult. This is a personal emacs config, and we all know from experience that commit messages and comments for personal code are not to be taken too seriously.

From my perspective, you're both very people with strong principles & opinions, and with very direct styles of communication, so I would honestly expect you to accidentally offend or annoy each other at some point. But as 2 of the people whose judgement and taste in all things Emacs I most respect, I hope you'll see that you're both working towards the same outcome of helping the community.


I'd make that one final if possible


Hi Juharris , I get the error

Traceback (most recent call last):
  File "/Users/ismail/PycharmProjects/techx15/cfm/main.py", line 7, in <module>
    from libs import myo as libmyo; libmyo.init()
  File "/Users/ismail/PycharmProjects/techx15/cfm/libs/myo/lowlevel/ctyping.py", line 131, in init
    self._lib = ctypes.cdll.LoadLibrary("myo")
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/ctypes/__init__.py", line 443, in LoadLibrary
    return self._dlltype(name)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/ctypes/__init__.py", line 365, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: dlopen(myo, 6): image not found

Heres is my project setup  , Any ideas on whats going wrong?

Thanks!

<img width="329" alt="screen shot 2015-07-24 at 3 31 45 pm" src="https://cloud.githubusercontent.com/assets/6295150/8885934/1e607a16-3219-11e5-9b36-65988649c696.png">


Great, i like this convention ;)


fixed in 5f248a5f556a53401888c238ac67ed7a8a84d2e8


Gotcha sorry I'm still getting used to using GitHub. I'll work on it thanks for the help!


You can cherry-pick f3b882b


blasphemy!


Closes #3.


This docstring is getting pretty long. I think it's good to show some examples with a `with` statement, but I don't think every example needs a `with` statement version and a subclass version. In fact, I would maybe do something like: have one or two examples just with the `with` statement, and have the other examples with subclassing. That way, users get exposed to examples of both methods, and to different things they can do with SPA, without any redundancy. If you really want to have an example both ways, I would just do the first short example, and then alternate.


You can see what I'm trying to do here — fire off the delegate work into GCD, then call `req->notify()` on the proper thread/run loop later (as setup with the passed `loop` in https://github.com/mapbox/mapbox-gl-native/commit/cba59fe29ebf6d8952c07e50ddec819a69660095#diff-78420c8d466421b5eb35d8ca3e2df9f8R57). Can't quite figure out the syntax. Any ideas @kkaefer @brunoabinader @tmpsantos?


This could better reuse the PacketError.Condition class.


`[0]` is the move, `[1]` is the pokemon that has the move.


Blah, pullreq it yourself.


...also, feel free to add yourself as a contributor in the package.json file.


Nit: I found this a bit confusing. The field value is never strictly boolean `False`, right? If you say "if it's currently a falsy value" or "if it's currently None" it would make more sense IMHO.


I thought about it, but `Team` and `Teams` are way too similar. It's easer to tell apart `Team` and `List` at a glance.


What is this public setter good for? As far as I can see, the connection is only set in the constructor and can therefore be final.


They're actually entirely unused in practice.  The user space wrappers for these illumos functions just ignore setting these values on all platforms.  They're just exist to avoid a build failure.  Although in practice there's no reason why the user space wrappers couldn't set the priority/niceness.  Perhaps they should.


Fixed.


Or maybe a wrapper function?


Good point.  It's not really useful.  Will remove.


Now that you've got me reexamining this, I'm looking at the `Transaction` ctor. I'll probably have a follow up.


`defclsyspri` is defined to be whatever your kernel would have set the priority of the process to be if you hadn't done anything at all.


Then shouldn't it be `'[of] ' + warnMove[1]`?


Yeah, I do like `JLOG`.  It makes it easier to do the right thing.


Closes #4.


>pullreq


Was removal of the "Shop" menu intentional?


@macumber I have the first half of the thing working inside the measure, and with the new BSDF/model setup. I cannot read the weather file, so I'm kinda stuck there for finishing the annual simulation part. The good news is that that half is relatively unchanged so it should slot in with minimal effort.


Unfortunately I'm not going to be able to test against IE9 for awhile. Other work has taken priority. Will let you know if that changes.


r+


Je vais me rapprocher de mon service cartographie car je ne connais pas les abrégés utilisés par le SDIS du Var. En effet, sur notre cartographie web (Rémocra) je n'ai pas trouvé de noms de filaires de voies écrits en abrégé.

Pour Calade, je ne connais que l'écriture avec un "L" qui est sûrement liée à l'origine Occitane du terme (Calada)


hey,
readBytesReg is a method in my mraa lib, what method do you suggest I use. I have the most current mraa lib.
Thanks


Add a test case where a link is broken across multiple lines. That used to work, but I think it won't any more.


Nice feature!

I think you are missing the use case for the "h-value-overflow". ("reset from 23^2-1 back to zero")

And do we really need the old v2 namespace? Are there any clients which *only* support v2?


Actually this string is needed so executeScript can be executed on the opened tab.


Sounds like overkill… A simple one-line comment might suffice to.

My point was just that some day in the (very?) distant future, we're going to have ONE code base, and this part is never going away - it's simply a fundamental difference between Linux and "everyone else" (?). So being pro-active and making sure we don't forget this when (if?) that happens would be good.

I know we never done so in the past, but maybe we should start now?


Final step: Setup mailer to send emails with the token.


Ah I see, OK let's leave it that way for now.


saw approval from graydon
at https://github.com/MonsieurNicolas/stellar-core/commit/4977e70aef1e48cbdaa6a447310a05b7e43a9c3c


MonsieurNicolas/stellar-core/scpFixes1507 = 4977e70a merged ok, testing candidate = a91c2d3d


merging MonsieurNicolas/stellar-core/scpFixes1507 = 4977e70a into auto


Fixes #5.


Is the MaxOrMin to support calculating both max and min in one loop?


Cool! All ready


Il existe aussi l'écriture "draye" (l'abrégé de "draille" reste utilisable)


Isn't that a "hardcoded default" in some kernel header or other? Not something you set in something like menuconfig or whatever one uses to setup the kernel build with… ?


неверно

Здесь нужно находить ряды tr, а не ячейки.
Назвать их rows

Перебирать ряды в цикле
а в каждом ряде уже искать ячейки row.findElement(...)

из ячеек достать больше инфы, не только фамилии


We can't say this. From the reviews:

> 2) experimental evaluation of the Ricci-Ollivier curvature for small n. The
curvature is negative. The Markov chain on unweighted trees is very likely
mixing in polynomial time (using [9] and comparison techniques). This means
that one should look for other coupling metric or that coupling is not the
right technique (this is a useful information).


It feels like you're almost at 100% test coverage.

Why not just visualize the test coverage in an HTML file and cover the remaining things? For example, using coverage.py: http://nedbatchelder.com/code/coverage/


Oh, and something about it depending on https://github.com/zfsonlinux/spl/pull/466 would be nice to :)


I'm not certain what you mean.   `[README]\n(README.md)` ? Does that work? It doesn't match the link regex:
```
linkRE = regexp.MustCompile(`\[([^]]*)\]\(([^)]*)\)`)
```


Commit on github is great feature.


I asked Alain about this on hangouts.

He is not opposed to having *_test.py files next to the actual code.

Let's agree on this: When there will be a second unit test file, we'll go with this approach.

That also means we'll have to create a new Makefile target then e.g. python_unit_test

You're currently using test/utils.py. When we move to extra unit tests, I would like us to reuse as little as possible from it and try to avoid it in general such that "test/" and "py/" become more independent.


Not this way though.
If you really want this feature, it's better to watch $scope.map.markers and call zoomToIncludeMarkers().


At the moment when you enter the command it returns saying False : So it needs fixing a bit still...


AMAZING)))


Peut-être pourrais t-on ajouter le "Layon" (sentier rectiligne ouvert en forêt pour séparer les parcelles ou les coupes et pour permettre la circulation). Mais est-ce véritablement une voie de communication pérenne ?


@seyyaw this commit again contains many changes that are not functional but are only whitespace. You asked me to make you aware of it.


What "user space wrappers"?


Il existe aussi le terme ancien "Trimard". Est-il encore usuel de nos jours ?!


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/72541802


this is a good comments


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

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

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

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

```


Go's regexp isn't line bound and it absolutely matched links like these, the link checker changed them all of course:

```
[Link with spaces and
really](long/long/long/long/long/path.md)
```


no worries, happens to us all :-)


Helloworld


https://www.youtube.com/watch?v=j6JALaG4_OA

Just this video from Ziu.
Here's some more info:
 As the refinement increases, the enhance weapons is a piece of cake?? Not only that, but there's also attribute in it.??????????
The crimson weapons will drop when it's given a random attribute. (Nothing / water / ground / fire / wind / saint / darkness)

Crimson Weapon Series Drop Monster Description

            Tamruan    Katana[4]    Crimson two hand sword
            Tamruan of Anger    Katana[[4]    Crimson two hand sword
            Pasana    Falchion [4]    Crimson saver
            Pasana    Stiletto[3]    Crimson dagger
            Fast Pasana    Falchion [4]    Crimson saver
            Fast Pasana    Stiletto[3]    Crimson dagger
            Evil druid    Bible[2]    Crimson bible
            Floating evil druid    Bible[2]    Crimson bible
            Li Me Mang Ryang    Club[3]    Crimson mace
            Floating Li Me Mang Ryang    Club[3]    Crimson mace
            Headless horse    Jamadhar[1]    Crimson katar
            Metalling    Crimson Bolt[2]    Crimson revolver
            Strong Metalling    Crimson Bolt[2]    Crimson revolver
            Boss Metalling    Crimson Bolt[2]    Crimson revolver
            Slipper    Damascus[2]    Crimson dagger
            Floating slipper    Damascus[2]    Crimson dagger
            Floating slipper    Damascus[2]    Crimson dagger
            Chung E    Hora[3]    Crimson knucle
            Stem Worm    Glaive[2]    Crimson sphere
            Fast Stem Worm    Glaive[2]    Crimson sphere
            Miyavi doll    Mandolin[3]    Crimson violin
            Boss Miyavi doll    Mandolin[3]    Crimson violin
            Holden    Six shooter[2]    Crimson revolver
            Strong Holden    Six Shooter[2]    Crimson revolver
            Porcellio    Main Gauche[4]    Crimson dagger
            Strong Porcellio    Main Gauche[4]    Crimson dagger
            Bathory    Arc Wand[2]    Crimson road
            Bathory of Anger    Arc Wand[2]    Crimson road
            Petit    Flail[2]    Crimson mace
            Fast Petit    Flail[2]    Crimson mace
            Strong Petit    Flail[2]    Crimson mace
            Scaraba    Knife[4]    Crimson dagger
            Marionette    Chain[3]    Crimson mace
            Boss Marionette    Chain[3]    Crimson mace
            Wild Ginseng    ROpe[4]    Crimson wire
            Owl Duke    Two-Handed Spear[2]    Crimson lance
            Owl Duke    Morning star[1]    CrimsonStaff
            Boss Owl Duke    Two-Handed Spear[2]    Crimson lance
            Boss Owl Duke    Morning star[1]    CrimsonStaff
            Solider    Chain[2]    Crimson mace
            Strong Solider    Chain[2]    Crimson mace
            Bloody Butterfly    Waghnakh[4]    Crimson knuckle
            Fast Bloody Butterfly    Waghnakh[4]    Crimson knuckle
            Injustice    Jamadhar[1]    Crimson katar
            The anger of Injustice    Jamadhar[1]    Crimson katar
            Dark Illusion    Broad sword[2]    Crimson two hand sword
            Fairy    Mandolin[3]    Crimson violin
            Fairy    Lute[3]    Crimson wire
            Floating fairy    Mandolin[3]    Crimson violin
            Floating fairy    Lute[3]    Crimson wire
            Anacondark    Glaive3]    Crimson lance
            Anacondark of anger    Glaive[3]    Crimson lance
            Executioner    One-Handed Sword[3]    Crimson saver
            Kobolds    Hammer[3]    Crimson wo hand axe
            Desert wolf    Stiletto[3]    Crimson dagger
            Griffon    Two-Handed Spear[3]    Crimson lance
            Majoruros    Two hand axe[2]    Crimson two hand axe
            Majoruros of Anger    Two hand axe[2]    Crimson two hand axe
            Elliott    Crow[2]    Crimson knuckle
            Kobold Leader    Flail[3]    Crimson mace
            Ogre Tooth    Dagger[3]    Crimson dagger
            Ogre Tooth    Stiletto[3]    Crimson saver
            Naga    Pike[4]    Crimson sphere
            Violin    Violin[4]    Crimson violin
            Floating Violi    Violin[4]    Crimson violin
            Fast Violi    Violin[4]    Crimson violin
            Strong Violi    Violin[4]    Crimson violin
            Owl Baron    Horn bow[2]    Crimson bow
            Giant hornet    Staff[3]    CrimsonStaff
            Boss Giant hornet    Staff[3]    CrimsonStaff
            Agent of Divine Punishment    Two hand sword[2]    Crimson two hand sword
            Floating Agent of Divine Punishment    Two hand sword[2]    Crimson two hand sword
            Floating Agent of Divine Punishment    Two hand sword[2]    Crimson two hand sword
            Fast Agent of Divine Punishment    Two hand sword[2]    Crimson two hand sword
            Minor Ouros(Nightmare)    Axe[3]    Crimson two hand axe
            Knight of the Abyss    Broad sword[2]    Crimson two hand sword
            Comforter    Harp[1]    Crimson violin
            Floating comforter    Harp[1]    Crimson violin
            Misteltein    Slayer[3]    Crimson dagger
            Misteltein    Bastard Sword[3]    Crimson saver
            Misteltein    Claymore    Crimson two hand sword
            Bow Guardian    Bow[3]    Crimson bow
            Wickebine Tres    Gladius[3]    Crimson dagger
            Eland Ebesi    Morning star[2]    Crimson knuckle
            Eland Ebesi    Sword Mace[1]    Crimson mace
            Boss Laurell Weinder    Staff[3]    Crimson road

            Boss Laurell Weinder /td>Staff[3]    Crimson road
            Sword Guardian    Claymore    Crimson two hand sword
            Armeyer Dinze    Busta[2]    Crimson saver
            Armeyer Dinze    Battle axe[4]    Crimson two hand axe
            Kavac Icarus    Horn bow[2]    Crimson violin
            Kavac Icaru    Great Bow[3]    Crimson bow
            Ygnizem Senia    Saver[3]    Crimson saver
            Ygnizem Senia    Slayer[3]    Crimson two hand sword
            Skeleton general    Gladius[3]    Crimson Dagger
            Boss Skeleton General    Gladius[3]    Crimson Dagger
            Skeleton General of Anger    Gladius[3]    Crimson Dagger
            Ygnizem Senia    Broad sword[2]    Crimson two hand sword
            Ammut    Sword Mace[1]    Crimson mace
            Amon-Ra(Nightmare)    Tablet[1]    Crimson bible

          The crimson weapons will drop when it's given a random attribute. (The same monster can drop an item of the other property.)
          In the accordance to the crimson weapon being applied, the equipment items that the monsters drop will no longer drop.

Source: http://ro.gnjoy.com/news/update/View.asp?seq=162


97903100_SolarCollectors (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


I don't think the formatting worked here...


No such thing as auto fec in S2.


orlyphemy?


DONE


yarly


While I'm OK with using PNG, I think I'd also want to include SVG examples, which I think is a more flexible format (and doesn't requite cairo as a dependency).


Thanks! I was going to add it in later.


hmmmmmmmmmmmmmm

i dun believe u


Many thanks Sir Kelly. This will help protect my fife and lower taxation of my serfs.


Whats up with this repo man? Are you trying to solve HackerRank or something?


Why the 8974? shouldn't we be using the 8226 repo?


@obastemur Regarding `typeof type == 'undefined'`, couldn't this be replaced by `type === undefined` ?

For that matter, couldn't `type === undefined || type === null` be replaced by `type == null`?  Note the intentional double equals.

See https://dorey.github.io/JavaScript-Equality-Table/


Repariert mit commit 0ebf6ccfa.


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


Fixes #6.


@wsnipex, @montellese: my add-on needs these two extra .so's copied to `${CMAKE_PREFIX_PATH}lib/kodi/addons/game.netplay/resources`. is there a way to copy these as part of the `make install` step?


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 32](http://teamcity.otenv.com/viewLog.html?buildId=1422464&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 32](http://teamcity.otenv.com/viewLog.html?buildId=1422464&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 34 Build time: 0:0:0


branch #3


I added a new rule that disallows that the curly braces is on a newline


Another option is something like cmake, which can output solutions for
visual studios as well as eclipse and other popular editors. I've used
cmake a few times and it's actually pretty good, and should be able to
handle this build quite easily.

I'd been happy to try convert it to cmake.

On 25 July 2015 2:02:25 am Duncan Ogilvie <notifications@github.com> wrote:

> Visual studio is a far superior tool for development. I wanted to compile
> everything in visual studio, but that doesn't work very nicely so I
> decided  to use qt creator. You can use qmake to generate visual studio
> project files if you want.
>
> On Fri, 24 Jul 2015 01:14 fileoffset <notifications@github.com> wrote:
>
> > Yeah that makes sense. I actually didn't have any spaces in my path, but
> > the files were still failing to copy unless I wrapped them in spaces. I
> > didn't think to put the spaces in the qt gui.
> >
> > Btw Duncan, I've been meaning to ask you: why do you compile the back end
> > dll's in visual studio, but the gui in qt creator? I was going to try and
> > push the whole build into qt creator land to make the build setup and
> > maintenance easier. If you're interested in that let me know and I'll have
> > a crack at it.
> >
> >
> > On 24 July 2015 8:37:18 am Duncan Ogilvie <notifications@github.com>
> > wrote:
> >
> > > You must use quotes in the qt creator gui. Otherwise it will split the
> > > arguments anyway.
> > >
> > > On Thu, 23 Jul 2015 23:43 Nukem <notifications@github.com> wrote:
> > >
> > > > Add on ( @fileoffset <https://github.com/fileoffset> ): This will
> > never
> > > > work easily without quotes. A space in the directory path causes it to
> > be
> > > > interpreted as another parameter. Adding quotes in the .bat won't help.
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > >
> > > <
> >
> https://github.com/x64dbg/x64dbg/commit/f4970331e9dd30479fb5b28419d69eae23851a7f#commitcomment-12331251
> > >
> > > > .
> > > >
> > >
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > >
> >
> https://github.com/x64dbg/x64dbg/commit/f4970331e9dd30479fb5b28419d69eae23851a7f#commitcomment-12332162
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/x64dbg/x64dbg/commit/f4970331e9dd30479fb5b28419d69eae23851a7f#commitcomment-12332708>
> > .
> >
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/x64dbg/x64dbg/commit/f4970331e9dd30479fb5b28419d69eae23851a7f#commitcomment-12346408


Why .clone()?


The second version (which is definitely cleaner) has a couple of problems:
- Won't work if `rolldev / step` is not an integer.  In these cases you won't get `nom_roll` in the list.
- The final ordering is somewhat ambiguous because you are sorting equal values.

I still prefer this:
```
plus_minus_rolls = [[-r, r] for r in range(d_roll, max_off_nom_roll, d_roll)]
off_nom_rolls = np.concatenate([[0]] + plus_minus_rolls)
rolls = nom_roll + off_nom_rolls
```


Hey @IntruderHT ,

If this is ready to get merge, go ahead and do a pull request, and I will get it in there in the next release. Thanks for keeping up with it man.

Loushou


Might be good to add `override` to `addSuppression()`, `addSuppressionPeer()`s, `addSuppressionFlags()`, `setFlags()`, `getFlags()`, and `swapSet()` as well (starting on line 99).


This should be:

```javascript
var windForce = new THREE.Vector3( 0, 0, 0 );
```


Yes I agree - I wasn't sure what to do with this so kept the raw field names in the GSP and a few references in Controllers/Services. The GSP code in _advanced.gsp needs to be made more flexible to handle any facet drop-down fields. I think this can be done with config variables. Will create an issue for it.

--
Nick dos Remedios
Atlas of Living Australia
Tel. (+61 2) 6246 4434
CSIRO National Facilities & Collections
GPO Box 1700
Canberra ACT 2601, Australia
http://www.ala.org.au/
________________________________
From: Dave Martin [notifications@github.com]
Sent: Friday, 24 July 2015 5:52 PM
To: AtlasOfLivingAustralia/biocache-hubs
Cc: dos Remedios, Nick (NRCA, Black Mountain)
Subject: Re: [biocache-hubs] Removed FacetsName enum for config var: facets.cached (d830bc8)

Theres a lot of Australia specific logic here (references to specific layer IDs etc). Should this be config or push up to the ala-hub ?

—
Reply to this email directly or view it on GitHub<https://github.com/AtlasOfLivingAustralia/biocache-hubs/commit/d830bc89f514f4c865ef740dc67f32962d1d8440#commitcomment-12337702>.


This should be:

```javascript
for ( i = 0, il = faces.length; i < il; i ++ ) {
```


Is there a parameter so this ends up being `} else {`?


you mean (!synced) ?


Running `rake db:reset` always seemed to work for this. I totally get why in a production environment that is a terrible way to apply database changes.

In the bEtsy project, my team used a lot of migrations as we updated the schema, so I feel like I'm comfortable enough to use them now instead of just cheating and changing the migration file like I did in MediaRanker. :)


LGTM


oops


I'll have a look


NVM... brain lapse. I was thinking query data was just a string.


Cool. Yes, agree. Your unsubscribe text is better. Let's do that!

On Fri, Jul 24, 2015 at 1:16 PM -0700, "Max Kueng" <notifications@github.com> wrote:

Great edits! Though "It's not recommended" in the unsubscribe text sounds a bit weird. Like its not the proper way for unsubscribing.

How about "We'd be sad to see you go but you can certainly unsubscribe any time. "

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


@Eskymak Add this as a new issue?


I would recommend connect your UGA email to Github as Github doesn't know who you are. Also, don't add temporary files, such as .proj2.py.swo. add `*.swo` to you personal .gitignore file to ignore these.

A couple notes on your programming.
* I would recommend never committing code that does not run; instead you should comment out the unfinished sections. These comments that can make a good start to your documentation when you actually write the code.
* If your override the built-in function `__str__` in your Molecule class, you can call `print(my_mol)`, which is more pythonic than `my_mol.print()`.
* I think you are looking for [`os.mkdir()`](https://docs.python.org/3.5/library/os.html#os.mkdir).
* Make a template string that you can use `.format()` to substitute the desired values for your input files

Good work so far! I'm just bored and wanted to look at code.


I'm pretty sure I was just debugging something & forgot to remove the raise statement before committing after I confirmed that my fix worked. Is it better to leave comments for things like that, even when they'll go away in the next commit?


FYI: On Github you can comment on files in a commit or the pull request in general.

This is a comment on a file in a commit.

For LGTM we usually comment it on the pull request itself which would be here: https://github.com/youtube/vitess/pull/920

(Due to the force push this commit is dangling and the reference to the pull request got lost.)


This isn't going to work, since in the `int` return code case the new sort function expects a return code  `< 0` when `item1` is less than `item2`. @lipari, you can back this one out, I have an update in my travis fixes branch.


blech. ok. will think.


Oh, interesting! I've only recently starting paying attention to the schema file. I'll have to watch out for it more often to see what changes happen to it when!


@brentvatne yeah, @sahrens said you'd be pleased ;-)


Also, I just want to say now in case I forget: you are the first code reviewer who has dug around a bit in the history, and I really appreciate this feedback! It is super helpful to get advice like this about the process & about choices that I've made while still sort of in development. (I'm counting the PR as deploying to production.)


hoping it works


Actually, it probably isn't working at the moment. I had made some changes to loading procedures, but it looks like the loader is completely gone now. That wasn't intentional, so thanks for pointing it out.


Just an FYI, you can override any properties by reopening the specific component in your app's tree if you'd prefer. I'd be easier to keep things up to date.


Looks good to me, thanks!


:smiley:


It looks like you're on 1.5.3.

https://github.com/palantir/plottable/blob/develop/package.json


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 33](http://teamcity.otenv.com/viewLog.html?buildId=1422512&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


It ended up this way because of it's own evolution. Originally, I wasn't sure how I wanted to implement StockPlus. At one point, it was going to be it's own add-on, but I wouldn't have been able to support that. Over time, I ended up with a StockController, but each module is also capable of turning itself off/on in addition to globally disabling StockPlus.

With the impending U5 and the lack of time I get to spend on new fixes to begin with, I've been hesitant to go back and rework the StockPlus interface again.


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 33](http://teamcity.otenv.com/viewLog.html?buildId=1422512&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 34 Build time: 0:0:0


@Zuluru How can someone reproduce the issue you're having? If you have a way to reproduce the issue a new issue would be great.


Are you an op?


In this "repo" I will store/share my responses to the prompts provided by Natasha_socal on a Meerkat stream which airs weekly on Tuesday evenings. It is meant  to encourage fledgling writers.


Missing comment?


Looks good!


We want to keep the commit for the 2.next release, we just need the upgrade/downgrade calls.

Example (sorry OSS souls): https://github.com/basho/riak_repl/blob/master/src/riak_repl_util.erl#L84


Yes! That's why I wanted to make the change. I didn't want to write everything three times, and I couldn't figure out how to set up inheritance in a rails environment. But I also didn't think it made sense to have all these weird media things in the application controller.

I'm totally taking this to mean that my instincts were right here. :)

This is also a good reminder that I still haven't figured out rails inheritance & need to experiment someday when I have free time. >_>


Not sure what you're implying here. That's what it says.


What reason for this change ? This is actually breaking several functionalities that assume the `line` variable to be composed of several line.
I'm thinking about filters and formatters.


Can you pull the `NormalizeSequence()` and `NormalizeHarvesterSequence()` before the `if`s to reduce clutter? Should be something like:
```
var baseSequence = NormalizeSequence(self, wsb.Info.Sequence);
var sequence = NormalizeHarvesterSequence(self, baseSequence );
if (!IsModifying && wsb.DefaultAnimation.HasSequence(sequence))
    if (wsb.DefaultAnimation.CurrentSequence.Name != sequence)
        wsb.DefaultAnimation.ReplaceAnim(sequence);
```
And then you can combine the two `if`s and remove the redundant `{ }`.
Similar for `void Harvested()`.


yeah!


That is the 1.4.0, not 1.3.0. We released this friday.


Problem with that is I don't get the lazy features that Stream has


Would of been better to check if the first was empty, then try the edge case.  Would of prevented a regression and solve the team one.  Maybe we can get that in another gem bump.


fixed by adding rule "requireSpaceAfterBinaryOperators": [","]


manipulating the strings is unfortunate.  it is fine within the Xcode task's own impl, but I think where possible we should use the Java library to concat paths, which handle the slashes just fine.
i.e. either new File(destSrcMainDir, 'objc').absolutePath or Paths.join(SOMETHING)


ditto everywhere


file.read() function reads arbitrary number of characters which may or may not be aligned to line. If it is not, it would cause issues in filters (may not match) and formatters (may format broken lines).


Fixed by adding rule  "disallowKeywordsOnNewLine": ["else"]


this makes me uneasy.
it means we prolly have a bug and we are papering over it.  better to let it fail and find the bug IMO.
or even better, use new File(parent,child) to concat paths


this doesn't seem like an easy to understand convention.
XDirFile --> a Input/OutputDirectory File object
XDir --> an Input specification String object
but MainTestResources is an InputFiles?
I think it would be MainTestResourcesFileTree if we really want the reader to glean the type from the name.


@fewizz Why are you so inactive with this project?


Thanks @maximilianmetti


Incidentally, since we weren't enforcing the presence of a UUID, if you ran ``pbpst -U`` without passing a UUID, it would segfault. Now, we handle that case correctly.


In the cast the last line of a block should be remove and keep for the next read.
If line are process line by line it become impossible to handle correctly multi-line log in filters and formatters.
* The filter will only filter the first line of the multi-line log
* The formatter will append to every single line the information


тут можно более нейтрально: "для каждого" -> "для всех"


Whoops, forgot semi-colon.


will this work with django 1.8, python 2.7? I'm having trouble getting the magnifying glass to be a link that opens up a new window (basically the widget is not really working)


delete this, it is not needed anymore.


@BRONSOLO @theengineear see above. not sure if the commit level notifications goes to you guys


м.б. "найдете" -> "получите"? В оргинале "have" не "found out"


This is what I get for a java error:
![java-error](https://cloud.githubusercontent.com/assets/321204/8886757/ba160d30-3224-11e5-96fc-49661d875fec.png)


LGTM


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 34](http://teamcity.otenv.com/viewLog.html?buildId=1422541&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


External Cocos Helper Android Frameworks/src/sonar/systems/framework/SonarFrameworkFunctions.java

I don't really need this cast.

@SonarSystems do you remember  lets get rid of `SonarFrameworkActivity` at all.


remove the comment and remove the affectation. Just keep that in this line:

requests.post(url, data=payload, headers=make_auth_header())


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 34](http://teamcity.otenv.com/viewLog.html?buildId=1422541&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 34 Build time: 0:0:0


UPDATE

The addition of "weight" in the tweaks is OK with a quick mandelbulb, BUT
with a mandelbox the additional overhead is much greater (more
iterations?). Therefore need an "enable weight" checkBox,  if weight is
required for a tweak.

On Thu, Jul 23, 2015 at 12:10 PM, carbuncle grim <mclarekin@gmail.com>
wrote:

> Sebastian, yes a "dynamic array", I like those words, sounds more
> technical than "build-a-loop"..
>
> What is fine grained setup?
>
> Yes it would require lots of coding by someone capable.
>
> Mandelbulb5 is a powerful but temporary thing, (but could be included in
> the next release as a "unsupported temporary thing"  to play with until the
> dynamic array replaces it) .
>
>  Currently Krzysztof wishes some of the tweaks to be added to the hybrid
> tab, and I tried and failed.
>
> Folding , Rotation , Fabs , const are just some of the potential
> modifiers. In V1.21 hybrid tab,  below the fractal formulas you can see
> other tweaks i.e.  x^2/(x +p) onwards. And there is a lot more as well. A
> dynamic array would be a practical way of doing it and allows for the
> addition of extra tweaks in the future, but it would be a big project.
>
> Cheers
>
> Graeme
>
>
> On Wed, Jul 22, 2015 at 8:28 PM, Sebastian Jennen <
> notifications@github.com> wrote:
>
>> This is a good idea. Looping through a dynamic array should give only
>> little overhead.
>> But this would be quite a lot coding to do.
>> Since this would erase all fine grained setup of mandelbulb5 i would
>> suggest to realize this in a New mandelbulb6.
>> So i See modifiers:
>> Folding , Rotation , Fabs , const
>> With min Iteration , max Iteration , pre / post Main formula flag.
>> And we need to make this graphically self explainatory, for this will be
>> more abstract than other formulas.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/buddhi1980/mandelbulber2/commit/9620b313cfc8011a7f9d5620dcc5e9f395c9bbca#commitcomment-12293142>
>> .
>>
>
>


saw approval from shino
at https://github.com/basho/riak_cs/commit/2d72a4af27d005b7619859682d50129d5a666753


merging basho/riak_cs/bugfix/proper-timeout = 2d72a4af into borshop-integration-1184-bugfix/proper-timeout


basho/riak_cs/bugfix/proper-timeout = 2d72a4af merged ok, testing candidate = 423ac0d7


Is there relationship between the equation and physical formulas? How can I derive the formula?


Service clients just return the dto, all service clients should have the same dto structure. The service then enriches the dtos with other objects to create kinda domain classes


Precisely my point. @acatton is avoiding re-inventing the wheel AND even on top of that, he's trying to re-use as much of as possible of an existing implementation. It's as if he's just painting the rubber of an existing wheel.


单机版的测试文件，测试环境 单机版windows 7。无论从各个方面来看，都是优秀于 FileInputStrem，不过还需要在 Linux 上进行测试。


测试文件 ：309MB的文本文件
测试比较对象 ：AltFileInputStream.java   &  FileInputStream.java 的读时间(millisecond)性能比较
测试环境 ：win7 x64 单机版
read (byte b[], int off, int len) 测试结果如下 ( 仅供参考 )

7月23号晚上测试

AltFileInputStream.java   	FileInputStream.java
                   2140	                     4677
                   2565	                     2097
                   2172	                     2547
                   1429	                     1881
                   1150	                     1947
                   1007	                     1897
                   1039	                     1878
                   1007	                     1928
                   1120	                     1888
                   1495	                     2009
                   1297	                     2299
                   1299	                     2881
     平均值 ：1476.66667                                2327.41667

7月24号早上
                  11197	                     12288
                   2212	                      5030
                   1188	                      2616
                   1070	                      2636
                   1062	                      2695

循环一次测试中观测到的时间差异是因为classloading的时间 EOM
需要再  Linux 下再进行测试


For explained_variance_ratio_ you need to divide by the total variance of the data, which is typically greater than the total explained variance (which depends on the number of components) you currently have in the code. Cheers.


This was needed due to http://github.com/aslatter/parsec/commit/ec7d75098740d9315736f2464f09b699bf37f436

While working on this others recommended other parsers like attoparsec which apparently perform better in binary data.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1247 of 1247 tests passed)

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


dfjdkfjd


dfdfd


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1391


@aledbf should we drop the -P here.  It seems unnecessary (I tried it w/out it) and you'd also noted previously that it doesn't belong.


This seems good for now; in a future release of queuedjobs it would be nice to have it expose a LoggerInterface directly, but that doesn't block this PR.


worked on four different models of IR remotes I had around, these mappings seem to be pretty universal.


The repository has moved to prebid.github.io branch master.


This commit make `test-variable-value.R` work, but it causes a failure in `test-save-separate-chunks.R`. I still push it as it can reveal a bug in saving common chunk when there are multiple selectors.

Besides, `tests_run(filter = "chunks")` will test both `test-save-separate-chunks.R` and `test-chunks.R`. So I will give the later a more specific file name.


I am gathering more detailed information here https://github.com/PeyTy/Vulkan/issues


Medal part is broke again.


什么时候出正式版，期待


sss


sdfs


fast-forwarding develop to borshop-integration-1184-bugfix/proper-timeout = 423ac0d7


I think it should be 1. This json file is reflecting the records in DB layer. As the tests shows, it will show release_id and be queried by release_id, and these are implemented in filter and serializer. In DB, they just connected by PK.


Same as above.


Great! Thanks @tyu011


console.log(json_data_object);が残った状態でコミットしているので、消しときましょ。
研修課題なので、残ってても別に問題無いけど、本番を想定して作業をしておいた方がいいので、念の為console.log()や、PHPだったら、var_dump() Sdx_Debug::dump()等、デバッグ用の記述はコードに残してコミットしないように、気をつけましょう。

ちなみにだけど、私は、デバッグ用とか、テスト用のコードを書く時、インデントを一切せずに書きます。
```PHP
  public function test()
  {
    $test_var = 'test';
var_dump($test_var);
    $this->view->assign('test_var', $test);
  }
```

こうやっておくと、後でコードを見なおしている時に、すぐに気付くので。

これが最良かは分からないけど、参考までに。


Okay, back on the subject of debug messages.

I think the right way to do this would be to have a constant integer for DEBUG_LEVEL. That is, how much stuff do you want to see printed? Only the final stats? Only the screw ups? Everything the players do? Everything the players think?

In addition to the DEBUG_LEVEL constant, we make this function:

    def debug_note(importance, message):
        # If importance is greater than or equal to DEBUG_LEVEL, print message

Every time we'd normally write a "print" statement, we instead call this function. We pass it the thing we want to print and a number indicating how important the statement is:

    debug_note(1, "I am thinking about paper.")
    debug_note(7, "My legs are on fire.")

The first note gets a low importance because who cares, the second one gets a high importance because it's really alarming.

We'd define meanings for the possible values of DEBUG_LEVEL, e.g.:

    0: Print everything you could possibly want to print
    1: Print everything the players even think about
    2: Print everything the players actually do
    ...
    7: Print very serious issues
    8: Only print the final stats

You get the idea. Whatever you think some useful "levels" would be.

Anyway, that saves us from having to have "if" statements all over the code, and allows us to define custom levels of how much crap we want to see, rather than just have it be all-or-nothing.

What do you think? Worth doing?


I also added a new field in the JSON output for Stacks, "elementID".  This is the id for the element that represents a parameter in a method.  It can be used to link back to the parameter on the page.


Oh, you should be able to just update your branch and then push to your Github and then it should automatically update the PR.


haha nice, I implemented the same as well *g*
4d520255268730f6903bdbda04e8d9234c85e6ef


是不是执行命令有错。。。。。

Firmwar 没有改过来，还加了一个错误进去。。。。

[^Open_FirmWar] 要删掉。。。。


两边都用一个 \` 就可以。

类似：\`#ifdef\`


So, are we no longer set the contact number to null to resolve NMS-236?


同样，所有的正文内命令行，前后各用一个 \` 就可以的。。。可以批量替换掉。

s/``/`/


Not currently.  It's marked as required, so I need to look over the SRS and
the code to see if there is every a case (MCTS upload maybe) where we will
not have an msisdn.  If not and it really is required then we shouldn't
null it out.

On Thu, Jul 23, 2015 at 8:36 PM, kosh <notifications@github.com> wrote:

> So, are we no longer set the contact number to null to resolve NMS-236?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/motech-implementations/mim/commit/c597b6ff27b0a6172aab5594d0262da747325227#commitcomment-12335456>
> .
>


Glad that we switch to diamonds. I'm working on the small multiples with it.


@sdfd 问题不大了，
1. 命令行是需要这样括起来的：\`This is a command\`，而不是每边两个 \`。
2. 另外，FirmWar 少了一个字母
3. 请 Rebase 把该文件相关的所有 Commit 合并成同一个变更，创建新分支，发送 Pull Request 到 develop


aaa


jajal ae


s/subsequent/first/


@dpoirier note this was necessary as a cherry-pick to production.

unfortunately changing the entry point in https://github.com/python/psf-chef/pull/152/files let `DEBUG=true` slip by into production.

we've rotated the Secret Keys for the applications as a precaution and will have the database password recreated asap (it's not a massive rush due to restrictive `pg_hba.conf` settings on the hosting provider), the issue was identified by @GrahamDumpleton https://twitter.com/GrahamDumpleton/status/624418234884698114.


ref. https://github.com/laravel/framework/pull/6979


At the next lines of 296 and 738, don't we need hash_freefn (resrc->properties, tier.key, free) to avoid the leak of memory allocated by strdup()?


Null-objects FTW :100:


I had wondered if I really needed a 2gb shared heap when I was attempting a Windows port of Hack. I guess the answer is no, I really don't.


This seems dangerous.


Good thought nesting this a `context` block


Same as previous checkpoints, this is in every example so move it to a `let` or `before` block


These could probably be made final


If port_vnic_type is set, ports in the passing create_kwargs will be override, which will cause the inconsistence.


ping @dysinger


Same comment as before. At 294 and 700, there are small leaks?


This is not semantically correct, you're not debugging because the HGNC identifier is used.

The checkstyle plugin is used to help write better code, as long as you solve its complaints in a good way. Here you can use this instead:

    params.put("debugQuery", Boolean.TRUE);


This is a creative use of expect, but I recommend using the [built in matchers](https://www.relishapp.com/rspec/rspec-expectations/v/3-3/docs/built-in-matchers) whenever possible.  If this fails, you're spec will say something like "expected true, received false" which is not at all helpful in fixing the error.

If you use the `be_a` matcher like below, you would get something like "Expected entry to be ab be a Entry, but received a Hash"

    expect(entry).to be_a('Entry')

Try out both in failing cases for yourself


Why did you remove these comments? They were useful.


Since you had to remove the first `$_panel_body` assignment; e.g., `$_panel_body = ' .... is a law that sets the rules...`, the next line should exclude the concatenation char `.=`. That way the value of `$_panel_body` starts fresh here, instead of appending to a previous value. See: http://php.net/manual/en/language.operators.string.php

In other words, line 263 should be `$_panel_body = '<table>'.` i.e., remove the `.=` in favor of just `=`


> They have not fixed these functions in the Windows 10 SDK. They are still not annotated with Success.

GRRR. *GRRR.* ***GRRR.***


@kristineds You can also remove the `sprintf()` call here, which includes the CAN-SPAM compliance link. The value of `label` can simple be `__('Mailing Address', $this->plugin->text_domain)`


What about `Double.NaN`, `Double.PositiveInfinity`, and `Double.NegativeInfinity`?  These are valid `Double`, but not valid `JNumber`.

Prior art:
- json4s-3 doesn't validate.
- spray-json turns these into `JNull`.  (This affects the return type of this constructor.)
- play-json doesn't offer that constructor, but does not validate its `Writes[Double]`.
- blueeyes, which appears to also derive from lift-json, returned `JUndefined`.
- argonaut has two methods: one returns an `Option[JValue]` and another returns a `JValue` that may be `JNull`.

I think I like the Argonaut approach here.

An even leaner approach is to only accept a `BigDecimal`, and say it's the responsibility of the library's codec to create `JNumber` from other types.  This is the play-json model.


Carefully consider doing assignments inside other expressions like `if` statements.  This is slick, but can easily be missed when other developers are reading your code


[detect](http://devdocs.io/ruby/enumerable#method-i-detect) might be a good alternative fit here


This is super cool :+1:


These changes sound be reverted since it doesn't compile. I had fixed these issues with 586b337da4d905174d2439a47ef885b9ed588b08. I feel like this must have been merged into the incorrect branch


(Y)


Its not required, but where's your rspec?  I'm a big fan of TDD and automated testing in general.


I notice blueeyes renders this as "null".  This restores totality of serialization, but means we can't round trip the value.  If we can't get rid of this as a `JValue`, I think that's my favorite compromise.


@kristineds You can also remove

```
, $this->plugin->utils_markup->x_anchor('http://comment-mail.com/kb-article/can-spam-compliance/', __('CAN-SPAM Compliance', $this->plugin->text_domain))
```

from this line, since the second argument to `sprintf()` is no longer used now. We still use `%1$s` (i.e., the first argument to sprintf), but the second argument (the link) is no longer necessary; i.e., `This is required for %2$s.` was removed.


This will require that the Codec used has a default constructor, we should document that somewhere if haven't already (I might have not seen it yet).


Do you think this logic is more appropriate to be here in the controller or in the `AddressBook` class itself?  Why?


I think the fate of this is tied to the fate of the alternate `JNumber` constructors.  It offers convenience for primitive types, but is this the codec's job?


Cool that neither of these are required anymore.


@dysinger: Loading the BuildConfig before switching to the Docker container isn't going to work, because that means the correct GHC has to be available on the host:

```
$ stack image container
Downloaded lts-2.16 build plan.
Caching build plan
Fetched package index.                                                                              Populated index cache.
GHC version mismatched, found 7.4.1 (x86_64), but expected version 7.8.4 (x86_64) (based on resolver setting in /home/manny/smart-device-sw/stack.yaml). Try running stack setup
```
(downloading the build plan and index also shouldn't be required when using Docker, since the image's version of those are to be used).  The outside-Docker portion of the code must work without a BuildConfig, which is why `withBuildConfigAfter` runs `after` in a context with only a Config.


This, along with the addition of `JUndefined`, is the only incompatibility I see with spray-json.  If it's already that close, perhaps it would increase adoption to switch to `Vector`.  I do not feel strongly, but my lack of strong feelings nudges me toward copying what has been successful elsewhere.


If these survive, I think they might benefit from the `isValid*` methods on `BigDecimal` for bounds and scale checking.  If not, keep those in mind for the reader type class. :)


:alarm_clock: Time's up!

By coincidence, I just happened to see this on... July 23, 2015.


Upload final data set created in step 5


@Reedyseth Awesome work! Thank you.

Would you mind changing:

```
</strong><em><?php echo __(' Your reply
```

To this? What I'd like to do is pull the space out of the translation entry, and put that into the markup instead; i.e., between `</strong> <em>` instead.

```
</strong> <em><?php echo __('Your reply
```


Coke - demo


Unimpressive, but yeah, when running all tests, which still included some not-rebased ones, there was a 5 second improvement compared to before. But, the total run times were 8mins 22 seconds vs 8mins 17 seconds. And those are the average values out of the 5 tries i did in each case to account for fluctuations.


Yay!


Please mention the significance of 'Response'.
Probably in comments !!
We can even have the string as a configured parameter with default set to 'Response'


I think this is a solid effort overall.

I would question the `@inline` annotations before a benchmark proves that they are useful.  I suspect the JVM does a good job on this already, and if not, they [handcuff us in the future](http://stackoverflow.com/questions/4593710/when-should-i-and-should-i-not-use-scalas-inline-annotation).  (Less of an issue on a project that is designed long term stability, but still.)


如果是升级后，这里会报错，导致扩张根本不能加载成功，因为 `rpc_list` 为 `undefined` 了。
我认为还是加个判断的好， `rpc_list` 为 `undefined` 时自动将 localStorage 里的值改写过来才行。


Mention in TODO that we need to support the verification shortly.


Thanks for submitting this.  I've always been wary of ConfigureAwait(false) because it's one of those bits of magic that I don't fully understand.  After reading Steven Cleary's piece from MSDN on the subject I feel a bit better, but to globally say "use it when you don't need the Context" without knowing exactly what the "Context" contains is a bit concerning.  As far as I know, none of the async callback code cares what thread it is run on, so I'm going to assume it is all "context-free".


This works?


Maybe later we can see what you can implement to stop that. Surely there has to be a way, like the majority of the network can disagree with a node or something.

Sent from my iPod

> On Jul 24, 2015, at 1:20 AM, Felix Ableitner <notifications@github.com> wrote:
>
> I guess so. There's really no way to stop them. But hopefully, no one will want to use that.
>
> —
> Reply to this email directly or view it on GitHub.
>


Can we have a headerless-table display as well.
The CLI can mention the order of column-names & will expect the output to have the column values without any headers.
This will be helpful for the scripts that will make use of this CLI.


can you try to add a counter in the condition to see how many open actions it saves?

it's possible that Casperjs save the reopen same url for you.

Sent from Outlook<http://taps.io/outlookmobile>

On Thu, Jul 23, 2015 at 9:15 PM -0700, "Carlos N. Abreu Takemura" <notifications@github.com<mailto:notifications@github.com>> wrote:

Unimpressive, but yeah, when running all tests, which still included some not-rebased ones, there was a 5 second improvement compared to before. But, the total run times were 8mins 22 seconds vs 8mins 17 seconds. And those are the average values out of the 5 tries i did in each case to account for fluctuations.

-
Reply to this email directly or view it on GitHub<https://github.com/currysensei/PhantomFlow/commit/b30c1f50249c8f3b9ce92e44149dbef204ba40c5#commitcomment-12335798>.


Lets put this as a configuration parameter with default set to 'Response'.
Mention appropriate details about the parameter.


Should it be https ?


We need to come out with our own symbol !!
Need to talk to the designer team.


There is no 5.3.0 here.


Mention the significance of profile name in comments.

In general, write details about each config property.


Yes, I have done the same but have not had the chance to submit the code.  Thanks Vaclav.


@christopheredge is this alright with you?


So you're suggesting that casperjs already optimizes behind the scenes if the same url is being hit for open?

I can try and verify that tonight or early tomorrow morning.


Removed 1st line for other


Need to mention the location of this config file.
Need to mention the steps to follow if this config file is modified.
Do we have any CLI command to dump this config file?


@christopheredge i mean the css thing -- i saw that you had changed the css in a certain way.


This looks wrong to me, @LowWeiLin


"Higuma" means Brown Bear in Japanese.


...
196 名前：login:Penguin [sage]： 2015/07/20(月) 14:42:29.05 ID:xAUGDbjF
>>132
のRESPONSE_REGEXですが
avi：映像制作［レス削除］
ttp://qb5.2ch.net/test/read.cgi/saku/1310326982/
の4-6、9、42のようにリモホのところに<8080><3128><8000><1080>とかついてるとレスが取得できないようです
...
に対応。
.+ にしただけなので副作用の可能性があります。


His name is "Higuma Duke".
のほうがシンプルですかね


I don't know. if you save a lot of reopen and saves no time, it means there is caching or some thing. if your code doesn't actually save a lot of reopen, then debug the code:)

Sent from Outlook<http://taps.io/outlookmobile>

On Thu, Jul 23, 2015 at 9:23 PM -0700, "Carlos N. Abreu Takemura" <notifications@github.com<mailto:notifications@github.com>> wrote:

So you're suggesting that casperjs already optimizes behind the scenes if the same url is being hit for open?

I can try and verify that tonight or early tomorrow morning.

-
Reply to this email directly or view it on GitHub<https://github.com/currysensei/PhantomFlow/commit/b30c1f50249c8f3b9ce92e44149dbef204ba40c5#commitcomment-12335847>.


s/commond/common/


We have an original duke for Hokkaido Java Engineer Group.


This is unused. It can be deleted.


Broke howso?  If they're just narrow, then that's intentional.


Good, glad you were able to eliminate the `var`.


Recommend always putting a space before `{` to be consistent.


第一次提交，显示超时；
第二次，改动了一点代码：将
while(j < s.length())
改为了
int length = s.length();
        while(j < length){ }
就提交通过了，说明之前每次都要计算s.length()，比较耗时


Once I get around to it, I'll get the sauce labs cross-browser testing integrated into the CI pipeline (#10)


Haha did you steal this from my code? ;)


ahh gotcha


Hi Roberto.

I just committed all your fixes to the codebase. Thanks and looking forward to more :)

-brady


what happens if there are multiple attributes in the order_by?  Will they all be ascending?


Mutable map is ok for now, but try to start thinking of how an immutable map could be used. The `Map` class has methods on it to produce new ones, so you can think of any operation that mutates the board as producing a new board.


Now that I look at my code, I'm pretty sure this could just be `ranks foreach draw()`...nice and concise.


Fixed it ind2f08e. Thanks.


In "pcsx2\gui\Dialogs\CreateMemoryCardDialog.cpp", the string "Automatically manages memory card contents so that the console only sees files related to the currently running software." cought my attention. Sure it shouldn't be "so that the emulator only sees" instead of "so that the console only sees"?


No, for me they showed as constantly expanding/shrinking. I set it back to 15 locally and everything's fine again.


Fixed it in d2f08e0. Thanks.


Much better to keep the string together for translation purposes. So would recommend instead creating a flag above instead of adding it to alertmsg. Then can create the full string here.


확실해요?


How about formatting this statement in a way that maybe looks like the layout of the board? I'd at least try to get some better alignment of the code.


Oh that was 1.0.1


sure


ok 반영합니다.


Nice. I see you're getting into stuff on your own. Regex's are a good target for decomposition and unit testing.


In Japan, Higuma taking salmon in his mouse is famous as symbol character of Hokkaido.
日本では、鮭を加えたヒグマは北海道を象徴するキャラクターとして有名です


Está neste pull request de gaiato, mas porque é útil.


@TurboGit
So i have only now noticed this.
Why do you get full input buffer if it is not used later on? (comment is correct)
```
// set mem pointer to 0, won't be used.
```
`dt_focus_draw_clusters()` does pretty much the same thing, and it does not call `dt_mipmap_cache_get()`...


After re-ordering my package loading, it seems to work fine. I'm not sure what exactly fixed it, but this change is probably not the cause of the crash.  Sorry for blaming.


nice!


Handle events async? Then how do you interact with the API in the event!


Oh yes, within the $classname arg list.  Definitely.


Simple vanish is vulnerable to mods, and is actually laggy for players.


This doesn't look like right. Some user may try to do this:

```
brew install <bottle file> --flags --with-options
```

They may be surprised that these flags have no affect after this change.


This would fail if it is RecordReadable and RecordWritable. I think you better check for (!recordReadable), right?


Add Framework support


add a test for a dataset that is both RecordReadable and RecordWritable?


There was typo for the output of deleteConnection() I corrected on this commit as well.


We hope Higuma Duke will get many friends around the world.

want everyone to like だと、好きになってよ！って印象が強かったので、
「世界中で友達がたくさんできるといいな！」ってのはどうですか。


cool. what is this from?


It is from the incredibles.


Only classes in `inc/core/..` have a namespace, all other, i.e. old classes need `use \Dataset;` or just `$Dataset = new \Dataset();`


@aa65535 已经更新修复啦～～～


API key changed


Wow!


This method is currently only called in `getYearComparisonTable()`, the 'default' view fails as `$this->Dataset` is unknown.


Actually, this should be pretty obvious given that you provide a class when providing the Codec on the annotation, so maybe this isn't necessary.


Sorry. All the GRASS_LIB_EXPORT in methods are no more required because it was added to QgsGrass class declaration?


![Uploading freebase-clap-herman-10.gif…]()


:clap:


I really like this idea. as much for a debugging tool as a learning tool for playing a smart game of hanabi and the decisions made when dealt an unlucky deck. it will also give the opportunity to prioritize steps in the take turn process.


This doesn't work the way you want to - it will always be true. See http://lua-users.org/wiki/TernaryOperator
Fix: since you just want a boolean, don't use ternaries. Invert the condition when you want false. Here, the line becomes: `lcdDisplay = bValue[8] ~= 0`
Same for the other "and false or true" expressions you use below.


lol


Yeah, maybe have the apply return an `Option[JNumber]` for `Double`, and have an unsafe extractor. There is of course a broader question of whether or not these should be here


Could be useful to standardise a format for something like `Array[Byte]`, i.e. any library which will use `json4s-ast` can be guaranteed to output the `Byte`'s in a certain format, which we can use

Which brings me to my second point, should be add a converter for `Byte[Array]`?


I think we should have both the conditions to render 404

render_404("User #{params[:twiter_handle] || params[:user_id]}") if @user.nil?


why not just use @project?


All 3 settings `LOGIN_REDIRCT_URL`, `LOGIN_URL`, `LOGOUT_URL` should use `'module_name:view_name'` as a value (not an absolute or relative path).


In terms of source compatibility, its the same. If you use the default constructors, you will always get a `Vector`. The only way you can get a `List` is if you do `JArray(List(JString("A")))`.

It might have an issue with extractors, not in terms of actual data extraction but in terms of performance. I think we should get a comment from the `spray` guys


nice helper


Excellent!

No more updating all the time.

Thanks


This will return all posts from the timeline with offset 0?

According to docs:
127.0.0.1:6379> zadd myzset 1 "one"
(integer) 1
127.0.0.1:6379> zadd myzset 2 "two"
(integer) 1
127.0.0.1:6379> zadd myzset 3 "three"
(integer) 1
127.0.0.1:6379> zrevrange myzset 0 -2  # -2 = offset+limit-1
1) "three"
2) "two"


similar to profile, we might want to list projects, and follow them directly from the list, in which case we can only pass in a local `project` variable


This might conflict with what I just changed that also helped fix these bugs -- we should double check.


<!DOCTYPE html>

<title>person.html</title>
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <meta name="content-type" content="text/html; charset=UTF-8">
    <link rel="stylesheet" href="css/normalize.css">
    <link rel="stylesheet" href="css/person.css">
    <link rel="stylesheet" href="css/button.css">
    <link rel="stylesheet" href="css/font-awesome.css">
    <link rel="stylesheet" href="css/bootstrap.min.css">

				冶成

<audio src="Still Loving You.mp3" controls  >不支持</audio>

首页

菜单

反馈

1

2

3

4

					冶成

爱我你就点点我

				        yc的主页

  玩转HTML5移动APP页面（优化篇）

承接上文《

                                            玩转HTML5移动APP页面（动效篇）

                                       》，上次说的是让页面动起来的一些小技巧。
                                      而页面动起来的根基是功能可用的页面，因此有必要分享一些优化细节的技巧和方向，熟悉掌握一些方法论还是会
                                      对页面开发大大提高效率的，并且也能防止疏忽缺漏...

                                      阅读全文

  玩转HTML5移动APP页面（动效篇）

作为一名前端，在拿到设计稿时你有两种选择：

                        1.快速输出静态页面
                        2.加上高级大气上档次狂拽炫酷屌炸天的动画让页面动起来
                        作为一个有志向的前端，当然是选2啦！可是需求时间又很短很短，怎么办呢？
                        这次就来谈谈一些动画设计的小技巧，能在你时间不多又没有动画想法的时候瞬间让页面增色不少...

                         阅读全文

  CSS3动画属性实用技巧教程

目前CSS3动画已经越来越流行，在一些网页专题的产品介绍页面，手机APP首屏展示页面，又或者只是一个按钮简单变形，都可
                        以使用CSS3来实现，虽然动画简单，但却让静态枯燥无味的页面增加一点生气，更吸引用户，更加高端...

                         阅读全文

  网页汉堡包菜单之关闭按钮的六种玩法

汉堡包菜单图标在APP或响应式设计上比较常见，一般用来展开/关闭网站导航菜单，然而这个菜单图标有很多新鲜的玩法，
                        下面一起来看看网页菜单之关闭按钮的六种玩法，小创意，好体验。
                        第一种：伪元素 + 网页图标字体
                        TIPS: 点击图标下面的按钮...

                         阅读全文

标题

a

a

a

a

a

a

&copy 冶成

top


I will have a look to see how `JNothing` is treated by other libraries that use `Json4s` and I will see how we go from there


I would rather pass in 'project: @project' here to avoid variables in views


You'll get empty timeline *ONLY* if you had only one post in it.


굿굿굿


shouldn't this be replaced by an unfollow button?


shouldn't the id be an unfollow button?


still getting error:
![email error](https://cloud.githubusercontent.com/assets/13478913/8868232/f9320276-31f2-11e5-9c2a-5a10931b44c2.png)


Github has hidden the conversation here, but you can see the discussion in https://github.com/Homebrew/homebrew/pull/41913 under `mikemcquaid commented on an outdated diff 3 days ago` as to how we ended up at this point. If you can fix the problems described in a better way, literally no objection to you doing so :)


nope, there are 2 buttons inside the partial, at most one of them can be active at a given time. We just need to rerender the whole partial.


You are right that these modules (brms-layer and bpms-layer) will not be build in parallel -- but all the previous ones will. So this saves a lot of time as we can for example build 16 modules in parallel. And in the end just package those in to the layer zips.

In ideal case the layer modules should be build in parallel too, to speed up the build even more. This should only be temporary solution until the modules plugin supports parallel execution.


For exiling the top of graveyard, I looped as seen in other cards that exile the top creature card. For getting the bottom, I didn't know how to grab just the first card other than doing the loop and breaking it after the first iteration. It didn't like getCards because it was getting a set instead of a single. Let me know the best way to do it and I'll update. It seems to be working fine, though sometimes the graveyard seems to display the cards out of order for whatever reason.

Additionally Flickering Ward (and possibly the other pro-color spells) is leaving the protection on the enchanted creature when the enchantment leaves the battlefield. Didn't know how to fix that.


This has been implemented for value map and range widgets only, correct?


This outer if-else can be written more functionally as

```re.findFirstMatchIn(command) map { matched =>
  ...
} orElse {
  throw new RuntimeException("command not recognized")
}```

However that's not as functional as it could be - the map has a result type of `Unit`, which usually indicates imperative-style code. If you can think of a way to make `makeMove` have a result type that's not `Unit`, that would be better.

Generally, you don't want to `map` if you're doing side effects (i.e. `Unit` result type). You want to use foreach. `foreach` won't work here because you need to throw the exception if the match fails; also, I dislike `foreach` on `Option` (it exists to parallel the other collection classes) because "each" on at most one element doesn't make sense to me.

Try to think of the state of the board as an object composed of smaller objects, and each operation maps the pieces or squares or entire board from one position to another. You map and produce a new state. Try to avoid `Unit` as much as possible.


Are there scenarios in which this can happen (given our usage of the function)?


for と in で迷ったんですけど、forでも大丈夫ですかね？


if you return here, that means it cannot list w1 devices AND other OW devices; right?


This won't end if order left battlefield.
There should be examples how todo this.


Hey @nathanrice, how will you be using this?


If minBound == 0 && maxBound == 0 you have division by zero in line 305.


Looks like we've got a bit of duplication of the `moveTo` code amongst the various subclasses of `ChessPiece`. Doing some design diagrams should help with the refactoring.


Check permanent != null


Wait, I got confused by the diffs. Looking at the actual code I see this is only in one place.


Nice, I just ran it and saw the unicode pieces. Cool! They're a bit small though, maybe there's a way to make them bigger. Would be hard to play with them that small. :)


for の方が関連度強く見えますよ！


That's strange -- how `lintr` missed the double quotes and missing spaces around the operator?


Good one, and this is closer to the original. It's a shame the following doesn't work in `/bin/sh` (while it does work in bash):

    git ls-files --modified -z | while read -d $'\0' _file; do
      ...
    done


This is coming along well. :+1: It's hard to comment with all these changes, but unfortunately github won't let you comment on the source itself, only on commits or PRs. These comments along with my notes in Hipchat should keep you busy for a day or two. :)


Based on language change enable/disable require features that language can support.


These are actually syntax errors, `linux` is not a variable and is causing the build to fail


Not sure what emerald price it should be.


Any reason we *prepend* the lines? This doesn't matter for our use case, but it's less `xargs`-y :).


Again, not sure what emerald price it should be.


Nice catch, feel free to send a pull request if nobody else gets to it


@kiennguyenquoc em move cái _show này vào show.html.erb nhé.


勉強になります :open_mouth:


Doesn't this step act as a no-op in this case?


Fantastic. Thanks a lot for these comments.


Maybe this is better:
```ruby
bottle = formula.bottle
return true  if force_bottle? && bottle
return false if build_from_source? || build_bottle? || interactive?
return false unless options.empty?
return true if formula.local_bottle_path
return false if formula.file_modified?
return false unless bottle && formula.pour_bottle?
```


lol DarkRealms BigRiddenMonsta...


@kiennguyenquoc em sử dụng class ".text-left" ở đây nhé. Ko dùng style inline trong html, dùng riêng ở ngoài css


Thanks. Wanted to start over from scratch :)


@kiennguyenquoc Chỗ này em có thể dùng Category.all


Although I understand that this approach is appealing so that we can keep `futile.logger` as a recommended and not imported package, but the `installed.packages` lookup is rather expensive compared to `require` (around +0.005 second, which can add up very easily when running `Pandoc.brew`). Either let's cache this on pkg load or let's use `require` with the `pos` argument if you do not want to attach to the global namespace.


Thanks for the comments. I'd keep the error messages for now. It's easy to make them shorter later; much easier than adding info that is not present.


@kiennguyenquoc chỗ này lỗi căn lề


@kiennguyenquoc chỗ này thừa dòng trắng


@kiennguyenquoc chỗ private này có dùng gì ko?


Why the extra comma?


Lol my tab was super weird wtf....


Hello @carlosgarciac, if you make such a change you should explain why. Otherwise other developers will not be able to understand the problem until they run into it themselves. Something like
```# Don't check for business package if the database was not yet configured (in the installer).```
maybe?


why make changes to this commit after it was reviewed?


Hey @srowen @mallman - can we actually revert the fact that this patch changes the name of the script? The improvement seems just like a minor cosmetic, and this is a major incompatibility pain for everyone who builds for 2.11. For instance, it broke our own test harness. Also, it will make it confusing for users who stumble on old docs that have the existing script name. I just don't see compelling reason to change it given this high cost.


GaGa......


Oh I also updated the Forge version to recommended. Everything in my last couple of commits is untested because I can't get my dev environment to launch, because for some reason, Forge does not count 10.13.4 as a higher version than 10.13.2. @xbony2