> should force user to think, does he want to return `a` or `1`

I don't think that's confusing, because it's the same thing: `a` obviously will get a value, before it is being returned, which is obviously `1`. It would be confusing it the old value were returned, which is not the case.

I have explicitly tested this:
https://github.com/paplorinc/intellij-community/commit/217433043780c657df24b258304d91bb7d4854e4#diff-4f64082219e6790c47c075322b21003fR83


This needs values for `ci.tfvars` as well.


published


还可以留comment


need to rewrite these two (`pos` ans `symbol`) because are lost when copied


I assumed that we did, as Andrew wrote this static bundle notice. I guess not then! This PR's not urgent and shouldn't stop going live with the static bundles, so tweaks welcome (or we could look into writing that update guide).


Well using the beacon texture doesn't allow it to be customized unless you change them vanilla one


When you have the new map marker feature set to "on", I notice that now when we stop routing via the route screen's (X) button, we now also remove all set destinations and intermediate destinations (even if they were set up NOT using map markers). Is this a bug or a feature?

It leads to users temporarily having cancelled a route, when they want to put the route back on they have to first re-add all the destinations manually via the Search History or in a similarly awkward fashion (see also #2293)

Thx-Hardy


Should CI use dev or prod settings?


I think dev (at least for the moment)


@alexey-lysiuk:

It should be ok with C++11 which has far more explicit type conversion rules. Aside from that the potentially ambiguous overloads are no longer in the global namespace but in std::, so they no longer apply. Visual Studio 2012 is not a C++11 compliant compiler so whatever errors it reports is not relevant.


@wshallum will try to get it merged later today. BTW how did you check the headers ? I did not get any warnings about implicit declarations - crazy eh.


Style: no comma spacing.


@snicolet
To point a): I think that for bishops the distance to the enemy king only plays a very marginal role,
as unlike knights they have long range effect.
To point b): I think your thoughts are right, but on the other hand IMO outposts become almost meaningless toward endgames.
The LTC test showed only an Elo +0,05 so to be honest I lost all confidence to make it through LTC with this idea. Finally I also think that this eval aspect should be or is already reflected by evaluate_king and it it would go against eval-orthogonality


Style: no spacing in `2*a`.


Style: no spacing.


Closes #140


Is this change in the right commit?


The name of the struct must be modified to BuildModelAndGeomVisitor


Could we replace this with:

```python
if is_reverting:
    messages.success(request, _("Page '{0}' has been replaced with revision from {1}.").format(page.title, previous_revision.created_at.strftime("%d %b %Y %H:%M")))
else:
    messages.success(request, _("Page '{0}' has been updated.").format(page.title))
```

This would make the translators job much easier.


no good


Type steams -> streams


@Kozoriz As I can see we don't use `it2` inside loop body. Why do we need it?


Je nutné ke všem atributům psát prefix "car_"? Máme přece zapouzdření, ty atributy vědí (a my taky podle toho o jakou instanci se jedná) - redundantní, navíc to bude i o cosik kratší.


Podobně jako níže Management - každý view je nějakou obrazovkou, vůbec netřeba to psát znova.


Názvy "_mng" nebo "Management" jsou podle mne redundantní a klidně bych je úplně vypustil. Jízdy samotné se taky stejnou filozofií dají pokládat za management svezení. Ta třída je entitou, stejně jako User, nebo Journey.


I think this is not necessary anymore.


Write an actual docstring.


Add a docstring


Ten komentář patří i ke všem ostatním podobným třídám.


There may be some savings from reduced garbage, but it's hard to say without benchmarking it.


Thanks for reporting the issue. I have push  Ice Builder 4.2.1 to the extension gallery with a fix for this issue.


Yes, the code in both pull requests is the same. I have now cleaned up the code (unused imports, etc.), made it pep8 compliant (I had missed a couple of things) as well as renamed some variables and integrated most comments from those PRs.

BTW, at the moment, the XMPPBot is duplicated in the output and the collector bot, I was not sure where to put it.


You should use the RuntimeService to create an EventSubscriptionQuery.


Closes #9


pictureName


Should probably say "does not produce an error or warning"


Všimni si, že všechny texty jsou psané jedním jazykem (nemíchá se CZ a EN) a jsou nachystané na překlad pomocí gettextu, obal je prosím jako `_("Transportation of animals allowed")` (v šablonách i views).


Move it down inside the `atomic` block.


Should not trigger the execution.


Should trigger the execution.


Always test before push ......


What do you need this header for? You've replaced `list` using with `vector`.


Ok, I have played with it for some days now and think it is doable, and the map marker UI is quite neat now.  Will need to think about how the dualism between map markers and destinations can be simplified, and maybe the ambiguous "Waypoints" expression can be phased out entirely. Maybe a starting point to unify could be that a destination and intermediate destination is always automatically also a map marker (but not the other way round).


`obj = self.model.objects.get(**({self.default_order_field: startorder} + extra_model_filters))`


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


I suggest to use `<cstdint>` instead of `<stdint.h>`


can I give it `//` as start and nothing as end?


An RT output event can be wrongly timed (will not meet RT) either because a) it collides with a previous output event (the phy is busy; that should be reported elsewhere: https://github.com/m-labs/artiq/blob/master/artiq/gateware/rtio/core.py#L216) or because the WB target takes longer than expected (for "real-time") to ack the wishbone transaction because of e.g. its downstream logic being busy (shared buses, busy machinery). For the SPI core this happens only on writes: If you write while one transfer is being shifted and another is already pending, the write will be accepted when the in-flight transfer completes. This is not a timeout or error in the WB sense (because it will be accepted eventually) but it violates RT-ness and should be detected.

The RT2WB interface sounds like the natural point where we might want to enforce a RT timeout/deadline for every transaction (would be two cycles for SPI) and then detect and report both this `RTIOTimeout` case. And also report the RT output event collision case.

Otherwise yes. The WB core itself should report WB timeouts and ack or err.


Should we maybe change to ParameterTool? Like in the other example, for consistency.


@sprint84
sorry , But can I confirm : reason that it can not support 8.0 ?


You can also use the other process definition: SignalEventTests.catchAlertSignal.bpmn20.xml.


ah yeah neat


@Makman2 probably should review this one


inb4 i do housekeeping


Hi @Thomqa, have you tried kodi-rbp-git ? It's currently 16.0rc3.


cc


Your editor should catch it, if it uses `php -l` to check for syntax. But I would also like to use scrutinizer for all developers, it has a lot of good tips. Maybe a topic for the meeting. :)


Should trigger the execution.


You have a typo 'aras' and 'there'.
I think it should be '...to different areas, so the read...'


@yanns that was the solution so far: overriding the stage step to remove all subprojects' target folders afterwards. In this case I don't distinguish between Heroku or any other environment since it is a harmless action.


@pynicolas fixed in 273b3f4cbc6b7354907c0d369b812cc13671e012
Now I will extract them to new module


why is the double quote not aligned with the other? That hurts readability especially for this usecase..


Now it work such way. All intermediate points, start and destination clears right after route has been canceled.


`has_many :attachables, through: ...`?


As I mentioned on IRC earlier, I think with `label-ok =` syntax, the value should be an empty string.  To define a `null` value, you'd do the following:

```
label-ok
  [nominative] First case
```


null: false?


I think we should forbid this syntax, i.e. a placeable with no expression and only with variants.


code actually runs ok.. tho still need to test more


I checked, it's set in my ST3 options.


why return nil in this case? file should always be valid, right?


@Don't we need to use `i`inside loop body? May be there should be `&some_data[ i ]`?


Variants are meant as a special formatter for the expression in the placeable.  If you don't need the expression, you are probably trying to build a map of value and you should use traits instead.


https://habrahabr.ru/post/178405/
тут подробно описано почему нужно правильно закрывать ресурсы:)
мы в целом еще на практике предлагали способ с
```java
Reader in;
        try {
            in = new FileReader("in.txt");

        } catch (FileNotFoundException e) {
            log.error("file not found");
        } finally {
            if (in != null) {
                try {
                    in.close();
                } catch (IOException e) {
                    log.error("error while closing file! msg:" + e.getMessage());
                }
            }
        }
```


Ini mas tugas saya, yang tadi salah..


who can  heip me ？


cool thanks


Bad variable name.


This is a project in which the table will be implemented


you should use a typed SequencerService not raw types


I know they match the API, but Emv and MTB are gross


Stale tu vidim ten nadpis detailu ;)


Some more background on this decision:

```
uint32_t tag = MKTAG('H', '2', '6', '4');
const struct AVCodecTag *table[] = { avformat_get_riff_**video**_tags(), 0 };
enum AVCodecID vcodec = av_codec_get_id(table, tag );
```
This call can extract the AVCodec from fourcc convention, but its a different call for video and audio.
DemuxStream has already descendants for video and audio, and the setCodecName can be overwritten in the respective class definitions. This saves another "if type == video" .... block ( I mislike them really)


coala is awesome at that ;)


Looks good.


Fixed #7


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


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


Please use the same thing I did in #62 with `.replace`


you mean the conversion algorithms?


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


@nirvdrum Would not changing the umask also fix it?


why is the closing bracket escaped? Doesn't need to I think


cutechess-cli.exe ... -pgnout results2000.pgn movecount=3 score=400 -draw movenumber=34 movecount=8 score=20 -concurrency 1 -openings file=2moves_v1.pgn format=pgn order=random plies=16 -engine name=tt6 cmd=stockfish_tt6 -engine name=master cmd=stockfish -each proto=uci tc=18.0+0.18 option.Threads=7 option.Hash=1024
Score of tt6 vs master: 712 - 666 - 3422 [0.505] 4800
ELO difference: +3,33


yes :P


@flavour these are only excuted if there is subscription_id; so I think no need to bring into scope


This will be HTML, so try <br/>


The reason why these tests had to be changed is because XMLUnit uses Saxon with a simple XSLT stylesheet and seems that previous version 9.6.0-1 of Saxon even these comparison was returning true:
```
XMLUnit.compareXML("<?xml version=\"1.0\" encoding=\"UTF-8\"?> <cities>     <city name=\"milan\"  country=\"italy\"   pop=\"5\"/>     <city name=\"venice\" country=\"italy\"   pop=\"1\"/> </cities> ", "<?xml version=\"1.0\" encoding=\"UTF-8\"?> <table> \t<tr> \t\t<td>italy</td> \t\t<td>milan, venice</td> \t\t<td>6</td> \t</tr> </table>").similar()
```

Now with Saxon 9.6.0-7 XMLUnit is correctly comparing the resulted xmls and returning false if they are no similar, which actually they were not similar because in one case the table headers were not present in stylesheet and on the other case the default namespace was propagated to the output therefore not present in expected xml and resulting in a non similar case too.


Please use a function here like we did in other places for the speech options.


Fixade index.html


typo: acquire


این رو من نوشتم
یکی از بچه ها کار دستم بوده نوشتتش


I receive the scrutinizer mail after your fix. And here : i don't use my editor  like i must use it ....


named pattern? like 'app:url'? just use reverse('app:url') in your settings.py. I'm using it that way also.


Wunderschön


Thx.


ziemlich gediegen


Passt


please remove the `a` from the phrase in brackets (similar 2 tests below)


safe


@173210 It is better to keep library untouched and redefine it inside fs.h
```
#ifdef _MIN_SS
#undef_MIN_SS
#define_MIN_SS TMIO_BSS
#endif
#ifdef _MAX_SS
#undef_MAX_SS
#define_MAX_SS TMIO_BSS
#endif
```
Also useful for another ffconf.h defines changed from defaults. This will allow to easily update fatfs sources if needed.


Done


heh I wrote those, thanks ;)


Holy sh*t!! I don't know what I was thinking...!! probably the HTC vive having come out ;)


So, the level is a multiplier for the rgb vector.
If level < 1.0, this is perfect to lower the colour value (hsv), i.e. to make the colour darker.
For level > 1.0, it gets a bit odd, as we are mostly clamping the components.
If we want to allow a level to brighten a colour, we may want to mix with white instead. If we use levels between 1 and 2 for this, that could be something like
```
if( level > 1.0)
{
    w =  255*(level -1.0)
    c = 2.0-level
    red = red*c + w
    green = green* c + w
    blue = blue*c + w
}
```
This would keep the hue constant. What do you think?


Was macht das?


@jagtalon Sweet, the `no-touch` solution is much better! Two things:

- can you remove the `Modernizr.touch = false` since it doesn't do much?
- I'd like this to be 4 spaces instead of 2. :space_invader:


@rpdunn77 @nhogman
I think this is wrong.
Attacked takes in an attack from the player, parameter being the bounding box of the attack-area. You are now passing the bound box of the sprite, which means the player can only hit the zombie if the player is inside of the zombie.


why the temp var, never used?


implementiert und definiert die Variable "guthaben"


Keine Ahnung, du hast das doch gecoded.


Ich bin ein veganer Adler.


@Patater `<=` ?


If you make a comment, make it count – what decision are you making?


Best profil picture ever!


Wer?


ohh i think it was just an issue with my editor it kept thinking it was a format string


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


I know ....

And that is even for the destinations which were not added as/via map markers.

I have been discussing with Victor elsewhere if maybe we need to make sure all destinations are also automatically map markers. Maybe that would be a good concept, so destinations do never simply "completely disappear" after a route cancellation, but at least stay in the map marker history or so.

Let's see what other ideas we have. I am afraid as is it may cause some support issues in the release, although basically it is a good idea to "clean things up" after route cancellation!


Current smarty_dir path works for admin section to manage ACL for installation process your solution is working

I am committing with your solution in a moment


Aside from the one line above that I commented on, it looks like your work is being commented well, Owen. You are using whitespace to separate logical chunks of code, and there are comments for each block. I know that took effort; keep it up.


will unescape )


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


@mbatchelor Marc there were a number of check style violations introduced by this commit. Please review


It doesn't look like you've done much in the past six days, Max, based on your commit history. Why? Remember to commit often as you develop your work. This is an important part of using source control.


Mission accomplished


I think we should use ui-sref here instead of ng-href as the routing system is based upon ui-router :)


Should the inline styling be moved to a class and defined in storage.css perhaps?


Done.


Same for this


This happens when one adds the 'exclude' parameter to Session for some sites

`Traceback (most recent call last):
  File "C:\venv\visions\lib\site-packages\ghost\ghost.py", line 238, in createRequest
    if self._regex and self._regex.findall(str(request.url().toString())):
UnicodeEncodeError: 'ascii' codec can't encode character u'\u2018' in position 267: ordinal not in range(128)

Process finished with exit code -1073741819 (0xC0000005)
`


Spencer, it looks like you've completed a draft of all three scenes of your robot panoply – a few comments on your commenting & organization of the code and your use of GitHub. You committed a few times since our last class, which is good. Frequent commits are better than infrequent commits. Your comments were meaningful on each commit, describing what you did.  There are some sections of your code that have code commented out – if you don't need that code any more, it's better to remove it (you can always view it later in your GitHub commit history). In a few places (where I've noted) add an extra line of whitespace; this is a minor change. On the whole, this work looks good from a code readability and use of source code control perspective.


Add a line of whitespace above this line.


not sure either, something with variables?
changing ...*ipc = NULL; no errors left (althought this will be nonsense...)


No, it fixes:

```
03-01 11:39:01.182  9074  9101 E NetGuard.Service: java.lang.NullPointerException: Attempt to invoke virtual method 'int eu.faircode.netguard.SinkholeService$Command.ordinal()' on a null object reference
03-01 11:39:01.182  9074  9101 E NetGuard.Service: java.lang.NullPointerException: Attempt to invoke virtual method 'int eu.faircode.netguard.SinkholeService$Command.ordinal()' on a null object reference
03-01 11:39:01.182  9074  9101 E NetGuard.Service: 	at eu.faircode.netguard.SinkholeService$CommandHandler.handleIntent(SourceFile:304)
03-01 11:39:01.182  9074  9101 E NetGuard.Service: 	at eu.faircode.netguard.SinkholeService$CommandHandler.handleMessage(SourceFile:226)
03-01 11:39:01.182  9074  9101 E NetGuard.Service: 	at android.os.Handler.dispatchMessage(Handler.java:102)
03-01 11:39:01.182  9074  9101 E NetGuard.Service: 	at android.os.Looper.loop(Looper.java:150)
03-01 11:39:01.182  9074  9101 E NetGuard.Service: 	at android.os.HandlerThread.run(HandlerThread.java:61)
```

The message above means that there was no internet connection while checking for an update.


@phhusson

When compiling lkl with ARM toolchain, dummy console will be automatically set to y (http://lxr.free-electrons.com/source/drivers/video/console/Kconfig). In order to securily disable VGA_CONSOLE, we may introduce a hard-coded ban when ARCH=lkl.

If DUMMY_CONSOLE is enabled, adding a screen_info definition may help solve the compilation issue.


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


What are you drawing with these robots? A short comment to describe intent would be helpful .


Big it did not crash NetGuard?


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


translate="" nem simán translate, automatikusan kiegészítette az IDE?


Ez nem volt jó ugyanúgy ng-options-al, mint a sima select? elképzelhetőnek tartom, mert huncút ez a select2.


@do3cc we have to be very careful with version pinnings in setup.py. This commit breaks bobtemplates.plone buildouts with a flake8 version conflict.


test


Enabling DUMMY_CONSOLE actually causes no harm.


Please describe briefly in which cases substring is falsey.


better to name the test so that it describes the case,
``test_qualified_dark_v1``
is less informational than
``test_qualified_dark_when_multiple_darks_in-time_exist``


We should ensure that we reliably deliver the transaction to WB territory in time. The deadline would be known at bitstream compile time and it would just be a timer on `active` in rt2wb. The reporting code path needs to be done anyways for `rtlink.o.busy & rtlink.o.stb`.


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


Kern, some comments on your use of source control and your code's readability. You have used source control effectively, with a commit after the completion of each scene, and meaningful commit messages. This is all very helpful for others learning from your code (classmates), for example. I expect that others will interested in reading your code too, and this is where you could make some improvements. Specifically, while you have a few comments throughout your code, there are blocks of code where there is no comment to describe intent. I'm suggested a few places where you might make comments (in various commits). Please take a few minutes to make those changes so that you code is readable (and so that you earn as high as possible a grade on the evaluation). Take care, Mr. Gordon


Suggest a line of whitespace above this line of code.


This curly bracket is not needed.


How will this save in the specified filename?


What does this code (lines 127 to 136) do? Comment suggested.


Включаемые области используются несколько иначе. Для подключения нужно использовать компонент bitrix:main.include


В шаблоне сайта не должно быть подключений каких-то сторонних скриптов. В данном случае код из scriptsphp.php нужно перенести в /local/php_interface/init.php


because `databytes` is memory mapped from the backing file


Matt, what was the purpose of the changes you made in this commit? "Added a bunch of code" is not descriptive.


Можно избавиться от этого запроса в цикле. CIBlockElement::GetList тоже может получать цены


I disagree. For example, a core that contains a FIFO will make the timeout error meaningless. What matters is the time when the core effects the transaction (which RT2WB has no way to tell - this question should be answered by the gateware designer), not when it acks it.


nit: s/HBase/Kudu/


I think you want && instead of || here


typo: s/foe/for/


txn.begin() should go before the try block


@Patater a box_id if zero is valid (main box)


Hey @johnnyhowey, @drewbrokke

This upgrade seems to be used to upgrade from an older SO version to 2.0. I don't know if that's something that is going to work in this context any way. I mean, is this going to be fired if someone is upgrading from a previous version? Won't they have to upgrade to 2.0 in a different way?

If so, we can just remove this upgrade classes. What do you think @johnnyhowey.

@drewbrokke Good job in the conversion. Even if we end up removing this class it's good that you've learned how to do this :)


Could we remove more details will be released soon?


А зачем эту в позиционном указывать URLРепозитория


ok, +1


Fix the comment line
Rename the method to "tempSensorGetValue"


Apologies for the trouble caused by this. I have removed your email addresses from the travis.yml file. https://github.com/DevFactory/nanohttpd/blob/master/.travis.yml

I hope this will stop any further email from reaching out to you.

Thank you
Farid


What is 1000?


@spocke Thank You :+1:


Yes, it compiles cleanly except for one section mismatch warning: octonet_driver references octonet_probe in __init section. I fixed this by removing the __init annotation for octonet_probe, because octonet_driver is also used in exit_octonet and can't be annotated both __init AND __exit (as far as I understand the section annotation).

Apart from that, I hoped your repo would fix my long standing wish to have ddbridge and dvbloopback working stable on kernel >= 4.2 but a test yesterday revealed your tree locks-up as hard as any tree I've tried so far :( If you're willing to look into this with me I'd be extremely grateful. I'm bound to 4.1.x until I resolve the problem. Will try this tree once more without section mismatch to see if that was the culprit?


Spaces...


Sorry, my mistake I saw a loop in pe_id in pe_ids.values() which is actually an if.
However I see that user_id = [key for key, value in pe_ids.iteritems() if value == pe_id][0] if value == pe_id][0] only needs executing if the lower condition is true, so that should be moved underneath it?


Thanks for the pull request. Merging it now.


Ihhh Lua


Added spellbar key binding menu. More bug fixes.


initial commit


There is no issue setting invalid HTML you just don't get the nice rendered output.  I don't see any need to validate it at this point unless it becomes an issue

I will be looking at ways to improve the expression builder widget to better display the results.  I'm not really happy with the current results bit at the bottom.


Whoops, looks like this caused a problem:

<img width="874" alt="screen shot 2016-03-01 at 11 32 26 1" src="https://cloud.githubusercontent.com/assets/52038/13426202/ee2787a2-dfa3-11e5-983b-f79d0d17a08d.png">

cc @joelochlann


Hi Ben,

My guess is that you've completed more work on your RoboCop "robot panoply" but haven't yet committed your work.  Please be sure to do this on a more regular basis as you complete your work (e.g.: with each scene that is finished, or even more often).

In this commit, you have made good use of whitespace to separate logical blocks of code, but need some comments, even if brief, to describe the intent with each block of code.

Keep putting in a strong effort,

Mr. Gordon


增加background和item的点击事件构成水波纹效果


I had to revert this change + the one you added changing filter to filter_by() With this change applied newly submitted tasks sit in 'pending' forever. Will test it some later today.

According to SQLalchemy code either way should work:

- filter(Machine.name==machine) -> SQL-like syntax
- filter_by(machine=machine) -> keyword syntax

Not sure whats wrong with the second though.


I thought generateRandomKey was now synchronous?


@gianarb Do you have plans, to release a new version (e.g. 0.9.3) to make this available for everyone? This would be nice, e.g. for using PHP7 type hints in entities.


Hi Eli,

I'm certain you would have completed more work since last class, but I'm not able to provide any feedback on your work, as it wasn't committed to GitHub.

Please do this more often (both for backup purposes and for code sharing with other students – and to receive feedback).

If you having technical difficulty with GitHub, don't hesitate to ask for help. It's a given that you are learning, so it is not a bad thing to ask.

Take care,

Mr. Gordon


Well done 👍


Sorry, my mistake, internal `generateRandom` function is now sync, not the public `generateRandomKey`


Hallo Manuel,
ja Ubuntu Mate funktioniert gut und schnell mit dem Raspi 2.
Simon läuft auch und führt die Befehle auch aus, allerdings passiert es auch das Simon einfach crashed,
da bin ich noch nicht weiter gekommen. Werde vielleicht noch warten bis Ubuntu Mate 16.04
fertig ist und dann damit probieren.
Wenn !! Simon denn läuft, dann läuft es gut.... bis zum crash.

Gruß und viel Spass im Urlaub
Thomas


Well, not at boot, it takes too long running a proper rankmirrors.


Would you mind also updating the documentation at:
https://github.com/BlackEdder/ggplotd/blob/master/source/ggplotd/gtk.d#L80
for the new behaviour?


Okay,

If I create a BlobCollection with `Mutable(FileReader(true))` I can't store new blobs into it - basically FileMeta's file name will be ignored...

Is there a way to see which file is an array Mmaped from or if it's mmapped at all?


yes, how stupid of it...


However, `Crypto` is not in the public namespace, this should be changed to `Ably.Realtime.Crypto`


thanks for fixing these typos.


I would rather this isn't sitting around in the production code.

The best way to do this is with unittest.mock if we can figure it out.  But if that takes too long, cut and past this code into the unit-test code itself.  It is not ideal because it is not testing anything in the main code, but neither is this.  It is not really a unit-test because it is not testing any of the production code.


Oh, this is why it broke. Remove the entrypoint.sh replacement :smile:


Yes, I am not sure if we may find a way to do it with DM session script.


Minor nits in the changelog:
 s/componable/modular/ ?
 "for certain experiments" ?
 "use generate workload" -> generate workloads ?


This is really cool, haven’t thought of it.


Ah, that's stupid mistake, I'm glad that you've noticed it.


Добавила ссылку на наставника


maybe we must add a toast element inside the demo pages to show the notifications


ok


this can silently lose data since `Base.Mmap.sync!(rand(10,10))` doesn't complain


changed.


spacing?


@bryk Could you check that? We could move later on some other validation logic to this module. Is this ok or is it too much? ;)


Thanks for updating this. It'll make user support much easier.


right, thanks, will fix it.


Also, thus just calls pacman-mirrors -g


Как вы это делаете?)


if you comment this out, how will the the bt object be loaded on a restart?


Maybe give them default values:

```D
void run( string title, int width=470, int height=470 );
```


no troubles, just a lot of spam.
thanks!


random test


Yes, that's what I understand. But I wonder why `SERVERNAME` works as it doesn't appear anywhere else in this repo.


I don't think there's any way. Probably shouldn't use with `mmap=true` in such cases. Also don't see much benefits of `mmap` in your use case, since it involves copying.


Close #132


This commit should fix #3


Yes, but ugly part is in live-session, that pamac or octopi sync with very slow mirror after login.
If we were to have a pure netinstall, this would also be relevant.
Perhaps we should just set live-session to German mirrors only?


Thanks!


you mean httpsPort here?


same here?


Quick note about the index. It is a file text, but it will behave much like a specialized table in the database. It is going to store pairs of user_id / wiki_id, so you can picture a table with only two columns: user_id and wiki_id.

Granted you have some wikis in your database, each one of them bound to a user, that index table would look like this:

```
1 44
2 677,56,78
3 null
4 12,34
…
```

Meaning the user with an id of 1 has only one wiki, with id 44, while user 2 owns three wikis, and user 3 owns none.

Now, what's the point of having this index file? Well, let's say you want to find all the wikis owned by user 2.

Using the wikis table, whose record are kept in the order they were created (basically, from id 1 to id n, growing), you would have to scan *all the records* and check whether the user_id field contains the value 2. Using the index_wikis_on_user_id file, you can go straight for the user_id value you are interested in, 2, and access the matching list of wiki ids: 677, 56 and 78. With that list of wiki ids, you can go back to the wikis table, and very quickly select only those records you are looking for: 677, 56 and 78.

The index file transforms an o(n) operation (having to scan through the entire list of records, of length n) into an o(1) operation: no matter how many wikis a user owns, accessing that list of owned wikis is a single operation, and retrieving those wikis from the table is a single operation as well (even though the user may own, say, 100 wikis, or even all the wikis, computing-wise, it is going to be a faster operation than scanning through the entire list of wikis).

> Index == search efficiency while searching for bound records/resources

Each time you are going to add or delete a record from the wikis table, the index text file will be updated.


Changed to use ui-sref! Thanks :)
https://github.com/adhorrig/cp-zen-platform/commit/39f2bd717e1f1a6ff20db5ba018774cc2c15b17e


Ok. For the SPI core I think we can have it behave so that if it is unable to ack a write immediately (because an xfer is being executed and a chained xfer is pending), it will bus.err. Seems to be allowed WB behavior still. Then we don't need the timeout and the WB transaction length for SPI is deterministic. And we get RT violation reporting. FWIW both collisions and bus.err can be reported as the same.


We should consider the Matrix team here. But I believe, as it will be deprecated, `pip install` will just work, too?


minified


Hi Tim,

My comments apply to your entire Robots.pde file at this location:

https://github.com/rsgc-morland-t/robot-panoply/blob/34866ff2487eaca89568a309c3fcb2045f212b4f/Robots/Robots.pde

Good use of whitespace to make your code readable. Your use of functions to draw each scene and handle repetitive code (like the text blocks) anticipates the next suggestion I am going to give the class (today) – nice work in organizing your code in this way.

A suggestion for improvement – your code has almost no comments – some brief comments to describe intent are needed.

Keep up your very strong effort!

Mr. Gordon


typo here?


Could we chop the npm-debug.log out of the commit ;-)


@vilchik-elena Do we need `ParsingErrorCheckTest`? There's no logic anyway in `ParsingErrorCheck`. The real logic which creates issues from parsing errors is in `JavaScriptSquidSensor` and it seems it's already tested. I would drop `ParsingErrorCheckTest`.


T-Pot will always try to acquire its external IP address to distinguish alerts send to EWS from RFC1918 networks. The interfaces' DHCP lease however is always bound to the network configuration of your DHCP server / segment. If you want to apply static network settings you can follow this [guide](https://help.ubuntu.com/lts/serverguide/network-configuration.html).


@vilchik-elena We should set the scope to "test".


delete file


maybe before we merge, you want to unify all of them having colons, or not?


I like the fact you committed right after using the generator, and prior to adding any custom logic :+1:


It seems it would be useful to cache the internal names of those types as well. Each call returns a new object.


Instead of having a separate ProgressReport / StartRevision, can this use `CompactRevision` and `Canceled=false`? As far as I can tell, a progress report is equivalent to a compaction that doesn't cancel the watcher.


I don't really understand this logic.  Does it not work if you just do
``area_det = glbl.AREA_DET``
since you have already imported ``glbl``?  What is the significance of calling it ``main`` ?

I guess then ``__name__`` is ``__main__`` which has some significance in python, it makes it look like the import is being executed, but this strikes me as a rather dramatic workaround (I know that it came from Tom).

Another way to handle the globals which may be less radical is to yaml them after instantiating them during the load process, then we can yaml.load them here.  That worked for me when I needed ``bt`` in ``beamtime.py``.

I am not sure the best design, but the latter has the advantage that I understand it, so I also  know what else it is doing to the code.....


Argh, I only saw this now because it went to the wrong e-mail address. I’ll try ASAP!

From: Bert Frees [mailto:notifications@github.com]
Sent: Tuesday, March 1, 2016 09:17
To: daisy/pipeline-mod-braille <pipeline-mod-braille@noreply.github.com>
Cc: Davy Kager <mail@davykager.nl>
Subject: Re: [pipeline-mod-braille] bugfix in process-catalog (82c941e)

Hi Davy. With the epub3-to-pef branch merged now, you should use two
attributes at the same time: px:script and px:extends (note it's not
extends-script anymore). Can you try that? All the latest snapshots should
have been uploaded yesterday.

Op dinsdag 1 maart 2016 heeft Davy Kager <notifications@github.com <mailto:notifications@github.com> > het
volgende geschreven:

> It seems that we also need to catch extended-script, as discussed on IRC
> yesterday.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/daisy/pipeline-mod-braille/commit/82c941e9ac28a77ea67d8ceb6377a4f6b1c939f4#commitcomment-16410543>
> .
>

—
Reply to this email directly or view it on GitHub <https://github.com/daisy/pipeline-mod-braille/commit/82c941e9ac28a77ea67d8ceb6377a4f6b1c939f4#commitcomment-16411036> .  <https://github.com/notifications/beacon/AC9y8Ypy4wcsLU3wydZW_QZCgAmcT5-fks5po_XhgaJpZM4HmN4d.gif>


RedisErrorMock is undefined.


e-mail address added


Are you sure? The `.filter(Machine.name == machine)` is incorrect because we're quering from the `Task` table here, not from the `Machine` table (something that could have been mentioned in the commit message).


That's equivalent to adding a 1-transaction buffer in the SPI core which is not strictly needed, but that's acceptable. Otherwise, I would just make the SPI core ack the transaction only when it's done toggling its pins, and have RT2WB report an error when you attempt a new transaction while a current one isn't acked yet.


LGTM :+1:


git-wise, this is excellent! One commit per feature (here, displaying the collection of wikis). I would simply have named the commit "Display wikis index", using an imperative tone and better describing the feature as a whole: customizing index.html.erb was just one aspect of crafting the feature.


Hi Ethan,

My comments apply to all the code in your Robots.pde file at this location:

https://github.com/rsgc-peterson-e/robot-panoply/blob/98586f52b21d4fc6c7375248744b90b343132907/Robots/Robots.pde

What a lot of effort you have put into this!  That is certainly evident.

You have made use of functions to organize your code – anticipating the next steps we'll take in class – great work to get ahead of the curve.

You have nicely commented the intent of your code throughout.

Your code is well structured; my one suggestion is to increase the use of whitespace lines in a few places. I will create a pull request to illustrate.

On the whole, your strong effort is clear!  You have made good use of source control and are going beyond expectations for program functionality – excellent work.

Mr. Gordon


it's origami.support


looks good


Your fix added https://github.com/practiceprovider/openemr/commit/639942dbdbfd3016147975248e3c3ec678d555d8


@vilchik-elena Why this package for `JavaScriptCheckVerifier `?


you don't configure a hostname for http connections?


Думаешь получится? Протестишь, ок? Потом скажешь, как: получилось, иль нет. Или у тя на компе/ноуте нету майна?


When we discussed the `repr()` of a dictionary versus an actual serialize format like JSON, both of which were being used only coincidentally compatibly before, the intent was to use the actual serialization format JSON. Python literals are not a serialization format. Are you sure this is what you want to do?


Thanks!


This commit has too many unrelated changes that makes it hard to follow. Everything related to DemuxStream should be based on https://github.com/xbmc/xbmc/pull/9238


Right. I will try that today. Thanks!

On Tue, Mar 1, 2016 at 4:09 AM, Anthony Romano <notifications@github.com>
wrote:

> Instead of having a separate ProgressReport / StartRevision, can this use
> CompactRevision and Canceled=false? As far as I can tell, a progress
> report is equivalent to a compaction that doesn't cancel the watcher.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gyuho/etcd/commit/9abebaa441a3c8c497b9fda3bde36a78522e5454#commitcomment-16415477>
> .
>

--
Sincerely,
Gyu-Ho Lee


Il manque le bootstrap avant ./configure


Regarding the section mismatch, from a quick glance (at work), does it help if you remove the __exit_p() around "octonet_remove" at https://github.com/herrnst/dddvb-linux-kernel/blob/linux-4.4.y-dddvb/drivers/media/pci/ddbridge/octonet.c#L221 ?

Re hard lock, I assume you're using octonet.ko? Not sure if and how I can help since I don't have such hardware (my CineCTv6+DuoFlex CT2 works stable since literally ages, before this tree and now with this tree, with and without CONFIG_PCI_MSI), nor I'd call myself kernel/driver developer ;) However, I'm willing to bring any driver fixes upstream to @DigitalDevices.

Does the problem also happen with the official driver package from https://github.com/DigitalDevices/dddvb/releases/tag/0.9.22 ? Does the kernel do a "proper" panic and throws a stacktrace so one can at least assume what/where things go wrong?


Looks nice. Add documentation and go for it :)


greate scripts, it works fine as i tried.
just wondering if there is anyone to manage users preferably UI interface that provides capacity of managing users/data usages and other functions

thanks


A quick refresher, and you may actually have read that already: http://guides.rubyonrails.org/routing.html#creating-paths-and-urls-from-objects

It will explain how you could generate the exact same link using different syntaxes:

``` ruby
link_to wiki.title, "/wikis/#{wiki.id}" # 1
link_to wiki.title, wiki_path(wiki.id) # 2
link_to wiki.title, wiki_path(wiki) # 3
link_to wiki.title, wiki.id # 4
link_to wiki.title, wiki # 5
```

Using a path helper (here, `wiki_path` or `wiki_url`) is convenient, but Rails is actually able to do it for you when provided by a resource (here, `wiki`). Because `wiki.class` is `Wiki`, Rails will infer it should use `wiki_` helper. By default, it will use the helper generating a link to the resource, `wiki_path` in that case. Essentially, syntax # 5 is a shortcut to syntax # 3. Using # 1, # 2 or # 4 would be considered bad practice.

Btw, if you want to play with a path helper `rake routes` mentions, just fire up `rails c` and access it on `app`:

``` irb
iirb(main):001:0> app.wiki_path(Wiki.first)
  Wiki Load (0.1ms)  SELECT  "wikis".* FROM "wikis"  ORDER BY "wikis"."id" ASC LIMIT 1
=> "/wikis/1"
irb(main):002:0> app.wiki_url(Wiki.first)
  Wiki Load (0.3ms)  SELECT  "wikis".* FROM "wikis"  ORDER BY "wikis"."id" ASC LIMIT 1
=> "http://www.example.com/wikis/1"
irb(main):003:0>

```


Why? This is `private[ir]` anyways...


sounds good!


Ты про отступ?


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


From a UX POV, it would be better to accumulate all errors cases.


I think `nix-prefetch-*` and similar scripts are a fine way to prefetch private or protected source locations.


Hi Frank
Thx for your suggestion..
I used Netty 4.1.0.CR3 and bouncycastle 1.54 on my Samsung S5. But still iam getting the gateway timeout error. Is there any additional 3rd party library dependancy involved here?

Request sent out
 CONNECT www.google.com:443 HTTP/1.1
 Host: www.google.com
 User-Agent: Dalvik/2.1.0 (Linux; U; Android 5.0; SM-G900I Build/LRX21T)
 content-length: 0
 Via: 1.1 localhost

 Response Received
 HTTP/1.1 504 Gateway Timeout
 Content-Length: 15
 Content-Type: text/html; charset=UTF-8


In messages, when you are referring to methods or functions please add parentheses right after the name ("prepare_core" => "prepare_core()"), it allows to more easily know what we are talking about when reading them.

The shortlog is a bit too vague and the content of the message is maybe a bit too verbose, what about:

> retro-runner: Remove module_path and game_path attributes
>
> Removes these attributes and pass them to prepare_core() as parameters.
>
> This makes the class simpler.

Here the shortlog states what the changes really are in a brief way, the beginning of the log explain more in depth the changes and we finish by explaining why we want these changes.

Looks good otherwise.


@pynicolas thanks


@psychon
If I have to decide, I'd take the `property::visible` approach.
But the `__index` magic could be done additionally.


ah, if i understand correctly SYNCSKIN excludes *.jpg / *.png files.

@phil65 adding the 'extras' folder and 'fanart.jpg' should be all that's needed then.
also make the same changes in the CopyRootFiles-ios.command file.


When following these instructions, does modifying the kernel cause other recipes to be recompiled during the "generating an image with all the changes" step?

Obviously the image itself needs to be recreated, but the rest should be more or less unchanged. I'm worried that this is not the case currently because recipes might depend on output produced when compiling the kernel (for example, kernel headers). If my concern is valid, then we need a different approach, for example, have one kernel as base for the OS and another that can be modified freely and then gets copied into images.


`gulp-tslint` complines about `!=` not being `!==`


@pynicolas to have it in `api` package. But it doesn't work in custom rules, this class is not available for some reason.


I forgot this was based on a ChromeDriver bug. Could you log a TE ticket for us to remove this logic when we do upgrade the driver?


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


added more comments, messed up commit message :P


@vilchik-elena We only need to have it available when running tests, not when running SonarQube.


There is no `Index` alias.


Hi Owen,

Can you submit this change for review. I also don't see any unit tests on the commit?


The reason for the problem that you report is not the column order. The column order that you specify in your patch is the same that it is used now. Your issue is that one or more tables were not loaded yet.
If you load the tables using employees.sql, you won't have this issue.
I can add the explicit list of columns (it won't hurt) but it won't solve your problem.


@alex2534alex я изменил версию протокола. но етого еще мало для поддержки 0.14


indexアクションまで消してしますと、root_pathにてこのアクションを設定しているからエラーが起きちゃうのでは？


I'm still unsure why you'd get duplicate ids, since the ids were taken from keys of a map:

```
defs = [elementRender k e | (k, e) <- M.assocs $ _definitions doc
                              , isElementNotNone e]

```

What should be the method to get an Id on the 'maker' elements?


Hello Thomas,

This is a server side part of  implementation for report generation status tracking and first page feature.
For status tracking I use aproach with ThreadLocal proposed by you - I set listener to processor before processing and remove it in finally block. Also I clean up ThredLocal when the  job is finished.

I've added new property for the "first page mode" - if enabled I return 1 in org.pentaho.reporting.platform.plugin.output.CachingPageableHTMLOutput#paginate and rely on UI logic to update the value later.
Also in org.pentaho.reporting.platform.plugin.output.CachingPageableHTMLOutput#produceCacheablePages
I update listener from JobManager to work in a "first page mode" and add my own listener to populate cache page by page ( I believe we need to optimize how is it actually storing the pages).

When status is CONTENT_AVAILABLE UI should request a new job - it will be finished immediately because we have pages in cache, at the same time we should continue polling first job status and update UI, when it is finished we should stop polling.

My changes rely on #201 so I can't open a pull request now and just provide you code for prelimitary code review.

@thomas


Hi Jamie,

Have you finished up more than the code that is shown here?

Please be sure that you are committing often to GitHub.

Mr. Gordon


Excellent.

Worth reading: http://tosbourn.com/difference-between-redirect-render-rails/ — In today's session, I will further demonstrate what is happening when using `redirect_to`.


part of iqss/dataverse#2934


`CheckByClickInput` is not renamed.


hrefはすべてlink_toに変更しましょう。
見つけ次第すべて直してくださいね！


@ticheler, when you update submodule commit, you also need to check that this commit is first pushed in the submodule repo. In  https://github.com/geonetwork/doc/commits/develop, commit bcf2649740deed6e58c22a8bbcbd796b03fd21f7 is missing. Could you push it ? You probably only have it locally. Thanks


In this function you can actually use your previously implemented function is_english_vowel() to not write the same logic twice


File name do not correspond to name of the class and namespace


Hi Nick,

Your code is very well organized and commented. You've made good use of whitespace and comments to describe your intent.

Did you finish up scene 3?  If so, please be sure that it gets committed to GitHub.

Can't wait to see your work in class!

Mr. Gordon


ну потому что так задумано в дизайне
нужно раскоментить


Point 1: have you tried to run the regression tests (both from `test/*.uts` and `scapy/contrib/*.uts`)?
Point 3: the `__slots__` also limits the objects' size, which was the original motivation of this change.


General comment about git: try to make commit messages informative, like what exactly you updated?
Perhaps something like "added 3 more functions" or "added vowel and dna functions"


We are patching symptom here rather that the cause splitSql() that joins the comment line with the next real code line.

By anchoring the # to the beginning of the line we mis out on any indented # comments, and at the same time we are ignoring the '--' comment, not to speak of multiline /* */ (which may or may not be affected by splitSQL() joining )

using an pattern starting with /^\s*#
would at least improve the robustness

http://dev.mysql.com/doc/refman/5.7/en/comments.html


edit画面にてpassword,password_confirmationはなんのために設置していますか？


@lorefnon I don't see it either. Because `case` calls `===` rather than `include?` under the hood, it would appear that the example is not relevant. In fact, it works for me on Rails master.


Hi Mike,

Have you made further progress on your assignment?  I'm guessing yes but that you haven't yet committed to GitHub – please be sure to do this ASAP.

See you in class,

Mr. Gordon


Could you please point me to the relevant documentation for your review process

Sincerest Regards,


этот гем nokogiri нужно добавить в Gemfile.
все gems что мы используем в проекте должны быть добавлены в Gemfile.


@Kozoriz  redundant asert


This is an interesting approach to do this, but not a very optimal one ) You don't really have to count all the occurrences here, just return False the first time you encounter that a character is not in "nuc". That would mean iterating over the string only once. Right now you are iterating 8 times. You can also lowercase the string before checking further.


почему поменян порядок?


@Kozoriz +1


That command to load the script was executed as the last script which means I had already executed all other scripts and did not encounter any issue until the last one.

I was following this [Tutorial]. Moreover, once I explicitly added the column ordering the command worked without any issues.

[Tutorial]:https://dev.mysql.com/doc/connector-python/en/connector-python-example-ddl.html


改行ね


again, use lower() functiong to lowercase before, it will make defining the "dic" variable cleaner


мы такие методы обычно пишем через точку: self.entity.titleForNewBookmark


`PEP-9`? :)


same here, you can use upper() to uppercase all the characters, this way you will only have to call replace for T to U.


4 поудобнее будет или к 2м уже привык?


I see we are taking things into our hands. We do not start kafka as a service but we spawn a thread. The problem with this is that we have a service status check in kafka actions (host.service_available()) so right now actions are failing


we use +the+ Github issue


@markfisher first request for this was in #110. as michael mentioned, it's really needed for deployers to store enough runtime info(which is only available when actual app has been submitted to yarn/cf, etc) to properly handle undeploy/cancel and status.


need a splace after this, it breaks the formatting

https://github.com/Harvey-OS/harvey/blob/41599c9b4d7a92a45ba81e9ef65d7d84b7985133/CONTRIBUTING.md


Thanks!


Remember: `form_for` is a Rails helper, and as such, it is merely a method generating a string containing some HTML. Essentially what `form_for` returns is something like this:

``` ruby
'<form action="/some/path/to/POST/to">…</form>'
```

Nothing really fancy, as you can see. Providing some arguments and a block to `form_for` though you may adjust the string's content.

Here, the `@wiki` argument instructs `form_for` the form should handle a Wiki resource. It means that in the generated string, some input fields' attributes will be prefixed with `wiki`. It comes in handy when the POST request (which will be triggered upon clicking the submit button) is received by the Rails application: all the data coming from the form will be wrapped under a `wiki`hash within the `params`, instead of being a giant mess of key/value pairs at the root of `params`. We can demonstrate that during our next session.

The block is obviously where you are specifying what the content within the `<form>…</form>` tag is going to be. The block parameter, `f`, is a way for the block to be made aware of the `@wiki` context I was talking about. Through the usage of `f`, you are able to adjust the generate input fields and what not to contain the expected wiki_ prefix which will in turn wrap the data under a `wiki` within `params`, etc.


@syohex ,
コメントありがとうございます．
PRに関して，承知致しました．
それでは今回はそのままPRを出そうと思います．
今後は今回のようなことが無いように気をつけます．
お手数をお掛けしまして，申し訳ございませんでした．


없어요 완전히 같아요
Unity에 정의한 App변수가 제이쿼리에 의존하고 있어서 수정했습니다.


Here we can use the function get_complement you have already implemented  and then just reverse the result of it


これ2回目の投稿できる？
form_forの引数がcreateアクションの時に上書かれてる気がするんやけど...


I believe you don't actually need this if, what happens if you call replace with substring that is not in the string?


"to create one, or create one" could be better said without the first part. just: "in the [Translators](url) channel, or create one yourself and share it at..."


Have you tried to reproduce the bug serializing the svg in #9? I think the problem is that you can specify an id in an object inside a _definitions map. So you will get two id attributes.


Looks like incorrect merge.


Hrmmm... This doesn't seem to reflect the decision we made to move towards top-down models and away from bottom-up, templated models. Please let me know if we should discuss this further. I don't believe this code should go in as it reflects an outdated understanding of the modeling layer.


いけます！何回も行けます！


@rolfbjarne Previously this would've caused a NullReference, now it just redirects to /dev/null. Is there a better way to redirect the output of `KillTree` to a log file or stdout or something?


空けます！


This patch has been included in python-sh-1.11-1 RPM:
https://bugzilla.redhat.com/show_bug.cgi?id=1306405

http://pkgs.fedoraproject.org/cgit/rpms/python-sh.git/commit/?id=7acde2031825e46e3587050ef9a3079e7fe562d9


This is more a personal concern than an actual issue: do we really need to specify it's Brazilian? will the translation be so localised, people from other Portuguese speaking places will not understand it well enough?


The table definition in that tutorial is different from the one in the repository. That's why you had the problem.
The tutorial refers to version 1.0.5 of this repository. There have been several bug fixes since then. We're now at version 1.0.7.
Please use the table definition in employees.sql, and you won't have any problem.


Do I need to give a PR for this?


Ready for fix #2


when i change the value from 1000 to 0, the output value is also 1000, no change, what is the prblem


@borzamircea we should assess later on if we need some authorization rule to be able to update the `project.Author`.


I'm already using @thuydx55 's fork, thank you ;)
`pod 'EZAudio', :git => 'https://github.com/thuydx55/EZAudio'`


Ahhh... here you go. You cleaned up all the other crap and replaced with simple models that expose interfaces.

:+1:


копипаста зло. Спасибо!


remove unnecessary usings


@borzamircea Please refactor repeated code into its own helper function.


Please simplify your fix as you suggested :-)


What is the difference between event_type and event_action? And--where is user_id?

My belief about event streams is that they must have event action, date, user id.


I think it's better to keep it simpler as a for loop.


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


Ну да......


Probably **ProductTittlesList ** would be more convenient in this case


Doing that in a backward compatible way would require adding something like `return_simplex` boolean flag, so I'd rather not start changing the signature here --- it's the semi-deprecated user interface in any case.


@rodrigopavezi I don't think there should have an `ng-binding` class, should it?


I think the is no need for additional members, because this class is meant to hold all data about Node.
where Node is (taken from doc string)
```
class Node(object):
    """ base class for Collector and Item the test collection tree.
    Collector subclasses have children, Items are terminal nodes."""
```
the data you want to store are more like session related,  and Session is child of Collector, so it should fit you.
only problem I see here is that you need to be able to distinguish between Item and Session report.
but it shouldn't be problem either.


Don't be afraid to create new folders for organization. UserManager might feel like a controller, but it's the main gateway for site auth and security. I'd would rather create a new folder called Security or Authorization and place it there than muddle it with Controllers.


This is a self-destruct mechanism that triggers when the builder detects that itself is hung (not when a step hangs), so this should very rarely (if ever) trigger.

IOW: don't worry about this code :smile:


这是什么呢？


Just pushed a version that implements this -- let me know what you think (and thanks for the input).


The double buffer is needed to support chained transactions in a nice way. This is how Freescale does it e.g. and it it is really nice because you don't need to think about framing signals for cs handling and you don't have to worry about the pesky half cycles at the beginning/end of an spi xfer anymore.

We can also delay the ack of a data write when a chained xfer is already pending. This is a bit ugly because it makes the `busy` collision that may happen eventually occur much later than its original cause due to successively accumulating ack delays for a longer chained xfer. But that is also acceptable to me.


Why was this reverted? That wasn't a hack... The GK1 interpreter does have an extra case with 3 parameters. You could have easily changed the SCI version requirement instead of reverting the whole thing


@rodrigopavezi Same for this with `ng-valid` and `ng-pristine`. When Mat creates the UX, I think he copies and pastes from existing Widget settings, so we have to strip out those classes that he erroneously includes. Thx.


Would be good to comment this a bit for readability.


насколько я помню, порядок, который был до изменения, тоже был важен. ты тестировал? там все работает?


Remember: `params` is a hash (containing key/value pairs), prepared by Rails and made available in any controller action. Because `show` is handling routes of shape /wikis/:id (cf. `rake routes`), whatever value :id has been set to when typing the url is now exposed as an `:id` key within `params`.


tests 🎉


OK thx for the information!


Fixed here https://github.com/mono/monodevelop/pull/1283


Ok, it's on my todo-list. :-)


@nk0leg, change log to next: `"Bluetooth device not found. Error: " << `


Yeah there is an `id` attributes in the drawAttributes, there should be an overwriting, I'll try to fix that tonight


10:00 PM?!


a


OK


в любом месте программы я могу сделать
```objc
[[MWMFrameworkListener listener] setMyPositionMode:whatever]
```
это очень неправильно с т/з архитектуры. зачем так делать?


@com9712 I haven't checked closely, but wanted to double-check that _you_ have been careful not to post your own parse credentials to GitHub as part of this code. (There are ways to avoid doing that, if need be -- or to undo it. Let me know if we need to look at them.)


Oh yes. Didn't notice that ^^ I thought `Tags` was public. I'll revert that change.


@nk0leg, what is 10? Add please the named const.


It is worth inspecting the generated link: it has a `data-method="delete"` attribute. And when clicked, it actually displays an alert window for confirmation. I want to dedicate 5 mn to this in our next session.


к чему тогда нам вообще MWMFrameworkListener? :)


Yep


No, removing the __exit_p() does not help. It's the __init-less &octonet_driver reference that uses the __init annotated octonet_probe. I can fix it by annotating octonet_driver declaration using __initdata, but then exit_octonet complains, so the best solution seem to be un-annotated octonet_probe.

I'm not using octonet, I'm just not sure there's no dependancy whatsoever. I have a CineCT2 v7 without CI, that's what I use dvbloopback for (legal where I live). So, the section fix will probably not relief the hard lockups (there's no panic, no logging, just a complete lock-up. I can't SSH into the machine and only powercycle resolves the problem).

I tried many trees and combinations: media_build_experimental and DigitalDevices ddbridge, all eventually end up locking up my system after several hours of watching/channel switching on kernel 4.2 and above.

bisect would be the only way to find out, but since it takes several hours to provoke the lockup it's a tedious task and this is the family production HTPC, so they're not too happy with my tinkering.


I see [1] this comment in the sip definitions which might be of relevance here:

    /** Sets the composition for the view. If the composition is being set manually and not by a QgsComposer, then this must
     * be set BEFORE adding any items to the composition.
     */
    void setComposition( QgsComposition* c /KeepReference/ );

[1]


fbtft work in sun6i https://github.com/gusenkovs/BPI-Mainline-kernel/commit/9c5bc201294d954f92b02209989d90e83f7daba9. I enable DMA and write patch


Да, конечно. Эти исправления специально вынесены в отдельный коммит.


Done.


Не сможешь. Дальше будет коммит, который это выпиливает.


Вопрос риторический?


This fix could (should?) have been registered in its own commit.


A lock icon is displayed against the apps that cannot be stopped which cannot be clicked and hence can neither be deleted or disabled.


Fixes #1?


Task 3: did you download the file and run it through the command line?


I had to remove lines 10 and 11 to get the `db:seed` command to work. Line 10 is not a problem, but as you never use `wikis`, it could be removed. Line 11 however is problematic: seeding will fail with an error as `rails` is not defined. I am not sure what you attempted to do, though.


I understand that this check `shouldIndex` was done once when job instance was created. Now is every time whenever `run` is invoked?


Typo here "ready your writes" instead of "read your own writes"


Sorry for the merge commit, guys!


Replaced this with something equally accurate but non-racing.


There's a way to do it. See the end of configure.ac where it writes to mcs/build/config.make.


We could do a little better with this docstring -- right now we're just restating the name of the module -- can you add some more description?


It's the other way around: before these changes `checkIndex` was called each time the job run (that's every 5s, because at the end of each run the job re-schedules itself to run again).

After these changes, there's a boolean flag and the check is done only once, even though the job re-schedules itself just as before.


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


Transtation needed :gb:


I don't have any EXTRA_ARCH resolving deadlocks with this:

```
user@f838688b8aa2:~/manifest$ oe bake autotools-config
Adding recipes to cookbook: 236 / 236 [100 %]
Calculating task metadata hashes: 177 / 177 [100 %]
The following will be build:
  autotools-config(12)
Do you want to continue [Y/n/?/??]?
```


Ended up in wrong location.


So this is the way to successful streaks on github :D


I think we should add some comments to these lines explaining exactly what is happening in the conditional.  For example, why in some cases would a 'callable' be returned, whereas in others would not.


sdf


Oo. Wow. :+1:


remove unnecessary usings


Nice catch, i think this is no longer necessary


нет. зачем нам класс, который делает подписки и за них отвечает, если мы делаем это в обход?


instead of disabling SC2001, i would suggest:
```
if [ "${PART_SIZE[$i]}" != "all" ] && [ ! -z "${PART_SIZE[$i]//[0-9]}" ] || [ "${PART_SIZE[$i]}" = "0" ]; then
```


@xgqfrms2015

#6


instead of disabling SC2001, i would suggest:
```
if [ "$lv_size" != "all" ] && [ ! -z "$(lv_size//[0-9])" ] || [ "$lv_size" = "0" ]; then
```


At some point, you will probably want to describe your policy on the very website, so end users know what to expect. I guess good locations would be /wikis/new and wikis/:id.


All right, good catch :)


Ah, **res** is not set at line 1512, so this makes no sense. Besides, the ban() by PM is only for mods, so you don't need this check. Otherwise, if you want to disable mod banning by PM, just removed the True, so:
`res = room.ban(target)`


instead of disabling SC2010 we could use the following, but i don't know if it's really better to use GLOBIGNORE:

```
  GLOBIGNORE="*.sig"
  IMG_FULLNAME=$(ls -1 "${IMAGESPATH}/${1}*")
  unset GLOBIGNORE
```


Now, I am a stick armer (stupid me, I know), but I would also like some kind of assurance that I don't disarm during wild acro.

A suggestion:
    CLI variable for "dont disarm with stick if airmode is on"...

/Z


Мы смотрели то, что было, чтобы какой-то был хотя бы минимальный ответ с нашей стороны до занятия. Чтобы если будут какие-то вопросы, их можно было бы на занятии обсудить. Мы будем видеть по Вашим коммитам и изменениям и будем учитывать сроки, которые вам выдали. У нас нет задачи занизить Вам оценку. Скорее даже наоборот :)


Is this not needed here?


If you set the m_partialMicroseconds to t1.tv_nsec/1000, you should subtract that value from the one that you assign to m_partialNanoseconds, shouldn't you?


Nice !


gh


А зачем использовать какой-то сторонний объект для подписки, если подписывается именно этот конкретный класс? При чем подписаться он должен именно в момент инициализации. И текущая архитектура не подразумевает для него момента отписывания. Использование каких-то других классов для конкретно этой задачи выглядит как большой оверхед. И ранее уже привело к багу с неявной инициализацией (я уже несколько раз подробно это описывал).


@lochmueller Is this not needed here?


This mechanism is probably no longer appropriate because "upstart" replaced the start_spinnaker script, and it does not consult the configuration to obtain these credentials.

I sent you a different PR that removes the mechanism and updates the comments. I'll wait for you to approve that change and let you close this one out.


this fix helped me a lot thanks!


Many changes are due to automatic format from Android Studio


надо удалить, если это уже не нужно


@ironfroggy I may misunderstand your point, but here is the issue. Django model_to_dict() returns {'_value', unicode_coverage_dict_string}, which is being passed in to create this Coverage element. The unicode_coverage_dict_string  needs to be converted to a dict object to be used in this create() function. I originally used json.loads() which can convert a unicode dict string to dict object but was problematic as you pointed out. So I am using ast.literal_eval() to convert it to a dict object. The function later used json.dumps() to dump this dict to json serialized string which is then loaded later with json.loads(). Here I only need to convert a unicode dict string (created by Django model_to_dict() function to a dict object, so I thought I don't need json serialization here. Let me know If there is still something I missed or not do correctly.


This should be replaced by "Advanced upload" in one single language variable. It doesn't make sense anymore to call it "simple" (it's not)


I removed the postfix in RELWITHDEBINFO mode in commit 97314e0dba0630cb9a3008e7b5d90a964430fbb7

I did keep the version number however.


Veldig bra


## Task 1

Write a program that includes 5 different functions each of which demonstrates one of 5 **different** string methods. As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses 5 functions to show 5 string methods and functions correctly (2 pts total; 0.4 points each function)

## Task 2

Write a program that processess these data to split them by line ending, replaces the tab separations with commas, and prints out the result - line-by-line. As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly processes data - it prints the data line-by-line, reformatted from tab-delimited to comma-delimited (3 pts.)

## Task 3

Write a program that takes the given list (using argparse) as input, and passes the list to a function.  Within the function, use iteration to remove every even-numbered item from the list, and return only the list containing odd numbers to the `main` loop.  Print the list of odd numbers from the main loop.  As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly removes even-numbered elements from the list using iteration, returns this list, and prints this list from the main loop (3 pts.)

## Task 4

Why do you see these different behaviors between `function1` and `function2`?  Answer the question for each function by copying the code below to `task5.py` within your github username directory and adding your explanation to each function as a **function description**.

- [ ] Correct answer for function 1, as a function description (2 pts)
- [ ] Correct answer for function 2, as a function description (2 pts)


@wluyima How does this look? Can we go ahead with this?


god kok


"expiriment" minor typo


Is there a reason that we are deleting self.unwrap?


武汉大学 霍达


Looks good. One mine typo, which I'll grab later.

Reminder to self.

> Cone on!


added null check since timeseries spec contains null check


not possible directly


Yep you're right -- i was reading into it incorrectly.


It must be $TARGET_KERNEL_ARCH.
Same also three more times below.


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.resourceeditor/6/violations
```
plone/resourceeditor/browser.py:84:16: P002 found "hasattr", consider replacing it
plone/resourceeditor/browser.py:372:1: C901 'FileManagerActions.__call__' is too complex (13)
plone/resourceeditor/browser.py:462:1: C901 'FileManager.__call__' is too complex (13)
plone/resourceeditor/tests.py:43:52: P103 other string does contain unindexed parameters
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Okay, now I understand. Once again you are right! Will fix it for next commit.


participating -> participated


Using theh `@Test` decorator means we could drop the `test_` prefix on these methods, right? Seems redundant :)


Actually this fixes #3, not #4. My bad.


Byta namn då? Innan stod ju `Lagrange 1 Line` för linje med linjära basfunktioner, nu behövs ju en siffra för dimension och en för basfunktionspolynomgrad. Nu finns ju två olika `Lagrange 1 Square`


1.
>>если подписывается именно этот конкретный класс
ну, подписывается тот класс, где ты вызываешь метод Framework::SetMyPositionModeListener, верно? вот конкретно ниже, у тебя подписывается MapViewController, а использует MWMBottomMenuViewController.
2.
>> зачем нам класс, который делает подписки и за них отвечает, если мы делаем это в обход?
ты не ответил на этот вопрос? зачем нам поддерживать два варианта подписки: универсальный и тот, в котором подписка в "конкретном классе" ?


LGTM


![Uploading QQ截图20160228164836.png…]()


What do you suggest? increase minor version number too?


Yes, this was an oversight from me, thanks!


I'm using tortoisegit, which I've never had a problem with in the idpf repositories but it's the second time it's done unexpected things here. I'm going to switch to the git command line and get a fresh clone. Thanks, and apologies for the mess up!

From: Joanmarie [mailto:notifications@github.com]
Sent: February 29, 2016 10:01 PM
To: w3c/aria <aria@noreply.github.com>
Cc: Matt Garrish <matt.garrish@bell.net>
Subject: Re: [aria] Merge branch 'master' of https://github.com/w3c/aria (9b5bf79)

Things are back to normal modulo your two commits. The footnotes/endnotes patch doesn't apply 100% cleanly. Plus you have whitespace issues. Plus it's not my spec. ;)

Under the circumstances, what might be worth doing is manually applying your patch from the command line rather than whatever you did before. After you pull master and have the correct top-of-tree, you can download this file: https://github.com/w3c/aria/commit/4e625b99f38898698ee7d528d6d701ff2b2edd6b.patch

In the top-level "aria" directory, do patch -p1 < 4e625b99f38898698ee7d528d6d701ff2b2edd6b.patch

1 of 23 hunks should fail. That one thing can be manually edited. Then you should be able to push your changes. Should you have any difficulties or weird merge conflict error messages, please ping me.

Thanks!

—
Reply to this email directly or view it on GitHub <https://github.com/w3c/aria/commit/9b5bf797d7427c4881544c89630d98cb67c805cc#commitcomment-16408211> .


We might want to omit these files and generating the Android app (right now, all files in `resources` will be added to iOS/Android).

Maybe we could do something like this?

- When building for iOS: add files from `resources/ios/**/*`
- When building for Android: add files from `resources/android/**/*`
- Always add files from `resources/*` (except `ios` and `android` folders)


"Typos" is my middle name... :cry:


Why the `break`? Wouldn't it be possible to have a standalone invoice (with unrelated subscriptions) in-between two invoices for that subscription and phase?


we should use capitalized OVS


yeah, but if we just do *id* then it messes up when we merge upstream/develop into poco, we could get rid of the github command i suppose.


did you submit the game


good catch!


@nk0leg, add space after the `:`


Great! I guess it compiles, so I'll merge


Yes.  For URLConnection, we just set the Host header, which is enough.  For SSL, the headers don't get passed until after the handshake is complete, so the hostname needs to be set through the SNI parameters.


now included in #120, should close


I think we should drop "How Bridge Foundry works from this section, and close the history with "Many Bridges"


ovs -> OVS


зачем этот врапер сейчас? вызывай прямо


SourceCylindricalProjectionEmission slipped into this commit


@nk0leg, are you sure that heap allocation is required. By the way. I see no delete. Leak? Same for `ConnectError` on line 114


mass ah yedho pandreenga!!!!


Me too...


Japp, du har rätt.


а почему нельзя обойтись vector<NSString *> или vector<std::string> ?


"There are plans for Android too."  -> The first Android workshop was held in August 2015.


should have put some screenshots too :-)


Also, in Summer 2015,...


You only add something to the NOTICE if it has its own NOTICE:
http://www.apache.org/dev/licensing-howto.html#mod-notice

This is the best case scenario where it is ASLv2 with no NOTICE, so nothing to do.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1342.b8e8316b2c2c1f3fd93c45d710a58684b3455252](http://build.palaso.org/viewLog.html?buildId=69776&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1020, ignored: 12 Build time: 00:08:24


нужно удалить наверное?


What do you mean by `into their own domains`


@ajithcj I intentionally used 2moves since Stefano80 asked to use this book last time instead of 8moves.


@nk0leg, add braces


@nk0leg, add braces here too


Good point, we should be able to do that since each batch its own session and can complete itself outside of onTrigger.


1. Код ниже подписывает два глобальных объекта, которые тоже никогда не отписываются. При чем у второго из них явно декларирован вызываемый снаружи метод, что удобно для понимания происходящего. При этом я не настаиваю, что это самый лучший подход, поэтому и написал TODO. И предлагаю вернуться к этому вопросу позже.

2. Потому что комбинирование двух разных подходов более эффективно решает задачу, чем использование только какого-то одного подхода.

К тому же в текущей реализации listener достаточно своих проблем, до сих пор не решенных.

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


new


With this change do we still need to `clone()` the generated `iv` ?


seems like we are getting product with specified title **from ** Cart


@nk0leg, again magical array sizes. Do we really need arrays? If so, then use named consts at least.


лишний TODO, исправить нельзя!


:bug:


faild -> failed


remove empty space


@nk0leg, why have you changed the `SOCKET` to int? This is win native socket, thus here should be `SOCKET`


тут все таки true нужно, вроде как.
иначе синий(желтый) пин останется на карте


Line 145: I read the scoring rules incorrectly.


This was probably my bad. I added the format array stuff in a later PR, and failed to update this, or correct all the naming changes. Thanks for fixing @averyvery.


@artbear ага


Hm, looking at other in-kernel drivers, they also don't neccessarily have the __init/__exit annotations in conjunction with platform_driver, so seems save to drop. Will apply - thanks!

Re hard lock, the CineCT2 v7 certainly won't depend on octonet, so fiddling with the annotations won't do anything. Also, all mentioned trees have the same driver code in common, just packaged up differently, so that's quite obvious the results are similar.

Quickly google'd for that dvbloopback thing, looks like that taps the kernel DVB subsystem somewhat. Maybe dvbloopback is at issue here (author even states discontinuation in the README), or ddbridge and dvbloopback just don't like each other a lot. Can't give any clear advice on that scenario/combination (running TVHeadend with it's built-in card client for such purposes, never had any issue with the CineCTv6), might try without patching dvbloopback in for testing, even though you'll loose access to subscribed channels, but this way you can rule out one or the other component.


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


Should this also work for floats?


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


тут тоже true (см выше)


тут тоже true (cм выше)


Можно, но надо будет слегка поправить код уже готового и рабочего диалога. У меня на все не хватило времени...


@nk0leg, update comment next way please
```
 BluetoothSdpGetAttributeValue requires non const input params,
 therefore we use const_cast to remove constness from the sdp_bytes
to prevent array redundant copy.
```


Shouldn't we init arrays with 255 instead of 256 (as we start a 0, not 1)


I build each platform using Jenkins. I kicked off a RPi2 build manually from my forked repo with the changes to the three lines. I suspect that all builds would fail from the OpenELEC.tv master branch.


It can be stay unchanged, because  everyone does not have bavarian model ofc.


I understand your point... I guess my feeling is that since this is a TCP socket connection and we are just writing bytes over a channel/stream, pretty much the only thing that can go wrong here is an IOException due to something with the connection.

The most likely scenario would be the processor is running and has already created a sender which opened a channel, and then Splunk goes down and then we keep getting IOExceptions until Splunk comes back up. I think it would be unlikely that a single FlowFile would fail and then the next one would immediately work, but I guess it is possible.


Кажется в следующих коммитах этот блок удаляется.


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


Wow I was about to push the same solution. Nice work


Good call


Соответственно цикл тут вообще не нужен


v2.0.1 commit


Это кажется лишнее


Add missed summary (FoundProducts, HomeProdReference, HomeProdTitle)


setconf is specifically for python 2.

For using with Python 3, one can use the "2to3" utility, like I do [here](https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=setconf-py3), but it's not ideal, since there are still problems with ISO-8859-1, which are not present with Python 2.


敵同士が重ならないようにする #271


To be completely honest, I am not 100% sure how that works, I was under the impression that Java was doing it. We already have several other tests (ListenUDP) which do this so I was assuming we were safe in terms of building in different environments.


Add in @publicode number 3
**See** [more](https://github.com/publicode/JavaTraining/blob/master/test_01)


Ahh, that is much better, thanks! I've updated the PR.


Старайся именовать классы не в camelCase. Все буквы - в нижнем регистре. разделитель слов тире.
section-image
img-el


The image in the master branch is missing


last parameter should be true, so file is selected within project tree, so:
ProjectView.getInstance(project).select(psiFile, target, true);


Can you implement with Qt native APIs?


Tem que rolar uma divisão.

this.pe = cm / 5.0799;
this.jarda = cm / 91.4382;
 this.milha = cm / 160931.232;
this.polegada = cm / 2.53995;


Классы, которые используешь в javascript коде лучше предварять суффиксом js-, чтобы было очевидно, что они используются в js. Соответственно на такие классы нельзя навешивать стили


test


Makes sense... I think we want to test both though because while the processor is running the batch won't be completed until the next execution of onTrigger, so doing runner.run(2) is essentially testing the code at the beginning of onTrigger that completes previous batches, but I agree we also want to test the scenario where we run once and then shutdown and verify that also complete the batch.


Есть специальная конструкция - $APPLICATION->AddHeadScript(). Но это устаревший вариант, лучше использовать для работы с ресурсами класс Bitrix\Main\Page\Asset


It was a spur of the moment. my lock file kept autogenerating and
conflicting with a badly done merge. At the time , it looked like some
`/var/blabla.lock` auto-generated file so I thought it was irrelevant,but
please do put it again, I haven't really thought too much on it.

On Tue, Mar 1, 2016 at 5:01 AM, Carlos Escribano <notifications@github.com>
wrote:

> @JoeZ99 <https://github.com/JoeZ99>
>
> Why have you done this? The lock file is the snapshot of which libraries
> work right together to run the app and makes bundler work more efficiently
> installing all the stuff.
>
>
> http://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/
>
> When developing an app, check in your Gemfile.lock, since you will use the
> bundler tool across all machines, and the precision enforced by bundler is
> extremely desirable for applications.
> ...
> When a gem lists a dependency, it lists a particular name and a range of
> version numbers. Importantly, it does not care where the dependency comes
> from.
> ...
> As a result, the application author will want to specify the precise gem
> servers that the deployment environment should use.
> ...
> Now, the application developer must be able to demand that the gem comes
> from that particular source. As you can see, there are critical differences
> between the desires of the gem developer (which are focused around
> longevity and flexibility) and the app developer (which are focused around
> absolute guarantees that all of the code, including third-party code,
> remains precisely the same between environments).
>
> From Bundler docs: http://bundler.io/v1.11/rationale.html
>
> This is important: the Gemfile.lock makes your application a single
> package of both your own code and the third-party code it ran the last time
> you know for sure that everything worked. Specifying exact versions of the
> third-party code you depend on in your Gemfile would not provide the same
> guarantee, because gems usually declare a range of versions for their
> dependencies.
>
> I can see two reasons for removing the Gemfile.lock file:
>
>    - Differences between platforms: bundler/bundler#635
>    <https://github.com/bundler/bundler/issues/635> (at the end of the
>    issue someone says that Heroku and other main providers remove the
>    Gemfile.lock if it was not generated in UNIX so it's like not having
>    Gemfile.lock at all).
>    - Generating a different list of dependencies based on an external
>    file, like databases.yml: http://stackoverflow.com/a/5360120/316414
>
> I think this needs more discussion...
>
> @peillis <https://github.com/peillis> @ecoslado
> <https://github.com/ecoslado> @serguitus <https://github.com/serguitus>,
> please, give your opinion.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/siwapp/siwapp/commit/fb9a07d2c0443235ba2bce073170b328f72e7d9c#commitcomment-16412983>
> .
>

--
uh, oh <http://www.youtube.com/watch?v=GMD_T7ICL0o>.

<http://www.defectivebydesign.org/no-drm-in-html5>


## Task 1

Write a program that includes 5 different functions each of which demonstrates one of 5 **different** string methods. As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses 5 functions to show 5 string methods and functions correctly (2 pts total; 0.4 points each function)

## Task 2

Write a program that processess these data to split them by line ending, replaces the tab separations with commas, and prints out the result - line-by-line. As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly processes data - it prints the data line-by-line, reformatted from tab-delimited to comma-delimited (3 pts.)

## Task 3

Write a program that takes the given list (using argparse) as input, and passes the list to a function.  Within the function, use iteration to remove every even-numbered item from the list, and return only the list containing odd numbers to the `main` loop.  Print the list of odd numbers from the main loop.  As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly removes even-numbered elements from the list using iteration, returns this list, and prints this list from the main loop (3 pts.)

## Task 4

Why do you see these different behaviors between `function1` and `function2`?  Answer the question for each function by copying the code below to `task5.py` within your github username directory and adding your explanation to each function as a **function description**.

- [ ] Correct answer for function 1, as a function description (2 pts)
- [ ] Correct answer for function 2, as a function description (2 pts)


Попробуй использовать не абсолютный путь до файла, а относительный. Компонент работает в нескольких режимах. Если не использовать полные пути до файлов, то появляется больше возможностей для кастомизации


Well in this case you should probably fix your .travis.yml. It gives the impression that Python3 is supported.

I may have a look at the ISO-8859-1 issue when I have time, it's nice when a tool is both Python2/Python3 compatible :)


Yes, I fear dvbloopback violates some internal kernel dvb handling above 4.2. That's why I think a bisect is the only way to find out what patch causes the lock-ups :(


You should mention the motivation and benefit of the new method.


Достаточно указать только CATALOG_GROUP_#, чтобы выбрались все данные из каталога.CATALOG_PRICE_# тут вроде бы лишний, но могу ошибаться


@sco0ter The issue with presenceUpdated is a mismatch of CRLF, for some reason, that section of code does not have DOS endings, whereas the rest of the file does.


It is macro.


Don't leave extra space lines.


Т.е. останется? Параметр отвечает за то, дергать ли колбэк в UI, а не за отрисовку пина на карте.


Нельзя задавать ссылки на элементы в шаблоне. Генерацией ссылок должна заниматься система ЧПУ. В битриксе есть механизм генерации ссылок, нужно использовать именно его


does this mean when you shake your device notifications get cleared? that's what I was able to understand from this...


Don't leave the dead code.


Ты уверен? См. мой комментарий выше.


fixed in combined commit


Why? You should mention the motivation in GIT commit messages.


false означает "не вызывать колбэк onMapObjectDeselected" в UI. Нам точно нужно вызывать этот колбэк?


authenticate 함수 내부에서 user.backend 를 지정해 주기 때문에, 굳이 backend 를 지정해줄 필요가 없습니다.


fix: correct missing "." in join statement.  this was inadvertently deleted when de-linting.


The native resolution of this image higher.


The native resolution of this image is higher.


and added connection to progressbar in controller


## Task 1

Write a program that includes 5 different functions each of which demonstrates one of 5 **different** string methods. As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses 5 functions to show 5 string methods and functions correctly (2 pts total; 0.4 points each function)

## Task 2

Write a program that processess these data to split them by line ending, replaces the tab separations with commas, and prints out the result - line-by-line. As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly processes data - it prints the data line-by-line, reformatted from tab-delimited to comma-delimited (3 pts.)

## Task 3

Write a program that takes the given list (using argparse) as input, and passes the list to a function.  Within the function, use iteration to remove every even-numbered item from the list, and return only the list containing odd numbers to the `main` loop.  Print the list of odd numbers from the main loop.  As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly removes even-numbered elements from the list using iteration, returns this list, and prints this list from the main loop (3 pts.)

## Task 4

Why do you see these different behaviors between `function1` and `function2`?  Answer the question for each function by copying the code below to `task5.py` within your github username directory and adding your explanation to each function as a **function description**.

- [ ] Correct answer for function 1, as a function description (2 pts)
- [ ] Correct answer for function 2, as a function description (2 pts)


fix: PEP8 checks implemented.


version1.0 使用HttpClient
version2.0 pull解析xml


@tshort Hi, if you could run unit tests of your downstream dependencies having this commit installed it would be valuable, as there were no tests related to `as.chron` included, and I believe you are the author of that parts of data.table. Thanks


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


fix: correct incorrect path specified for catalog_populator.py


@jalavik what was this "improvement" doing? :-)


PatchApplier.mustache line 29-ish:  Needs to output warning/error not supported.
Codegen.scala line 93-ish:  Got better var names for jj and kk?


Is this for correct ??


Test


notable example is the user: **_Flamestatics_**


What is the Place


You forgot to update the `getTopicAnswers` itself in order to add the query results to the passed 3rd parameter.


fix: formatting improvements


Is it really neccessary to create array of array?


Поправь отступы (2 пробела)


Program in Task 3 doesn't use iteration.


Not sure I understand your comment - from my understanding, there are 256 grey levels available, starting from 0 and up to 255.

My thought was that lut[] could represent the grey levels from 0 to 255, then we loop through the array applying the threshold value


We could replace this with [Module::Runtime](https://metacpan.org/pod/Module::Runtime).


Ya it is needed


fix: formatting improvements and additional documentation


Yes it is required


Two things (repeated in other places)
1. The state extraction
2. IsConnecting + IsBinding were combined into a general IsBusy flag that simplifies things.


ok


i think coala does this, though i have no idea why it hates this line in particular and no other, it's the PEP8Bear which is doing this


I wish these were not "tt"-specific.


feature: initial checkin of the README for project #3


I've moved some of the code in the controller into the state object - specifically logic that deals with bound projects (more about that below).


fix: delete superfluous file


ok. it is moved simply.


> Great idea, done. Can you transfer the repository? You might need temporary admin status, let me know in that case.

It seems so. Let me know when I can make the transfer. I made the changes you pointed and squashed all in one commit.
I will open a PR where for the pinning.


that feature is called switch arming.

It is really not possible to have this for stick arming. People need to be
able to disarm in crashes without doing 1000 actions.
Just go for switch arming and you will get full freedom.
Op 1 mrt. 2016 14:05 schreef "MasterZap" <notifications@github.com>:

> Now, I am a stick armer (stupid me, I know), but I would also like some
> kind of assurance that I don't disarm during wild acro.
>
> A suggestion:
> CLI variable for "dont disarm with stick if airmode is on"...
>
> /Z
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/borisbstyle/betaflight/commit/94e54728172fdf64c13ee689efc6abd72b3fe580#commitcomment-16416838>
> .
>


Are we introducing [Moose](https://metacpan.org/pod/Moose) as a dependency with this?


For history in the commit message please mention Git IDs of commits that modified those options in Elf32 plugin.


Commit message: `mainly` is duplicated.


ok. end of func


i am working on designing a svpwm based invrter using IR2130 can u help me in this regard


This improvement (improving support for python 3) should go to a different commit.


Good point - do you have a suggestion of how we could make this template agnostic?
I mean, we could change the name, but the templates for the builtin engine would have to be in a specific templating system anyway...


Fix for a pylint warning, different commit


i am working on designing a svpwm based inevrter using IR2130 can u help me in this regard


Sounds good, I didn't know about that!


I think you should change `rename-head` to be `make-rename-transformer`, and eliminate the currying and the `stx` argument from `typed/untyped-renamer`. You'll want the resulting code to look like something like the code here: https://github.com/racket/typed-racket/blob/ca46d80189058d612d0ecbae650dba59d8dc7546/typed-racket-lib/typed-racket/typecheck/renamer.rkt (although that uses `prop:rename-transformer`).


Je parlais de ca par ex, je pense que théoriquement le load-balancer de nginx peut marcher avec le cluster si on connait les ports, maintenant est-ce vraiment utile d'avoir le double-emploi ?


Huec


Don't think so, I'll remove it. (But then TBH I don't see why we needed to clone the generated iv before this change either, #concat doesn't mutate the object it's called on...?)


We are using #pragma once instead of this type of guards.