PROCESS GENIE

Mobile Testing App

Business Requirements Definition

V0513th November 2014

Table of Contents
1	Background	3
2	Purpose	3
3	Objectives	3
4	Business Requirements	3
Requirements Introduction	3
4.1	Development, Hosting and Branding Requirements	4
4.2	Home Screen	5
4.3	Favourites	6
4.4	Test Management Templates	6
4.5	Quick Reference Guides: Process	7
4.6	Quick Reference Guides: Testing Methodologies	7
4.7	Consultancy Support	8
4.8	Quality Assurance CAST Tools	9
4.9	Training	9
4.10	News and Events	10
4.11	Jobs	11

1	Background

Our research shows that there is no dedicated app that supports Software Testing and Quality Assurance.
There are over 100,000 testing and QA resources across multiple continents.
App download and usageshows sustained growth from year to year.
Revenues from apps also continues to rapidly grow.
Reference and education apps comprise 11% of the market.
We see a significant opportunity in the market to provide an app to support Software Testing and Quality Assurance.
The App will be named “ProcessGENIE”.
2	Purpose

The purpose of this document is to define the business requirements that will need to be met by the developer of the App.
Delivering a solution that meets these requirements is necessary for the App to achieve its objectives.
3	Objectives

To provide a “one stop” information service to the QA and Testing community worldwide of processes, tools, services, training, news, events and jobs.
To generate revenue for <<COMPANY>> Consulting.
To generate new business for <<COMPANY>> Consulting’s partners.
4	Business Requirements
Requirements Introduction

The tables in section 4.2 onwards capture the business requirements. The column definitions for these table ares:
Reference:
This reference uniquely identifies each requirement and should be used in all other documents when referring to requirements in order to provide full traceability.

Description:
An unambiguous statement of the requirement.
Priority:
The following priorities will be used with their meanings:
o	(E)ssential – the App cannot go live without this requirement being met
o	(H)igh – the App could go live without this requirement being met but achievement of the objectives is likely to be significantly impaired.
o	(D)esirable – while going live without this requirement being met is unlikely to impact objectives, certain cosmetic and aesthetic qualities will be compromised.
In the descriptions below we use the following terms to indicate who is using the App and who is adminstering the App:
o	User: A person who has downloaded the App and is using it.
o	Admin: A representative of <<COMPANY>> Consulting who is able to adminster the App and modify content.
We alsodistinguish between categories and sub-categories that can be tapped:
o	Category: Fixed and only modifiable via an App update. A category may be reached from another category e.g. News, Scheduled Training Dates.
o	Sub-category: Reachable from a Category and modifiable by Admin.

4.1	Development, Hosting and Branding Requirements
Reference	Description	Priority
1.01	The App is to be available on the following platforms:
o	Apple (E)
o	Android (E)
o	Blackberry (D)
o	Windows (D)
Phone gap >> uses HTML5 and only disadvantage is slightly low response.
1.02	Development of the App must be on a cross-development platform to minimise cost and future maintenance/enhancement
	E
1.03	The App must be installable by Users via each platform’s application store (e.g. Google Play, Apple’s App Store).
Yearly licence fee, blackberry need to check the fees.

	E
1.04	The App’s content should be hosted on an industry recognised cloud based platform such as Microsoft Azure.
Hosting on cloud based platforms…
	E
1.05	New content will consist of two components: A short “abstract” which will always be pushed onto the device and the main “body” which will only become available following purchase and can either be downloaded to the device by the user or accessed via a link (see 1.09).
We need more clarity on which of the categories or sub categories content should be displayed? Or is it 	H
1.06	It must be possible for Admin to release updates and contenton any of the developed platforms either through the relevant Stores or the Cloud.	E
1.07	Users must be given a notification when new content has been pushed or if there is an update available.

If the new content is added by default, the content will be displayed when the app is open next time.
Does the requirements talks about notifying user when admin make the changes
Does the flag need to be displayed? Who long it should be displayed?

Push notifications will be provided through GCM & APNS…
	E
1.08	Updates should also be notified via the standard notification method used by each platform.	E
1.09	Admin should be able to decide whether content is to be streamed by users from the Cloud or downloaded by users to their devices.	E
1.10	Content will be streamable wherever possible e.g. video and audio.
Can admin Customize whether the content can be customizable?
	E
1.11	Content will be non-interactive.	E
1.12	Sub-categories reachable from tapping a category on the Home Screen to be configurable by Admin.	E
1.13	There should be no <<COMPANY>> branding although <<COMPANY>> will be referred to as a consultancy provider. Therefore the design and colour scheme of the app should be modern and in vogue.	E

4.2	Home Screen
Reference	Description	Priority
2.01	Starting the App will first present the User with a home screen. 	E
2.02	The home screen must always just be a single screen which can be scrollable.	E
2.03	Navigation to other App content must be initially from the home screen.	E
2.04	The following categories should be available from the home screen by pressing a button which will then display the associated screen:
o	Favourites
o	Test Management Templates
o	Quick Reference Guide: Process
o	Quick Reference Guide: Testing Methodologies
o	Consultancy Support
o	Training
o	Quality Assurance CAST Tools
o	Subscriptions	E
2.05	If the number of categories exceeds the size of the home screen then we expect the home screen to scroll.	E
2.06	If any of the categories contain new items then a meaningful indicator should alert the User.	E

4.3	Favourites
Reference	Description	Priority
3.01	A “Favourites” list should be accessible from the home screen and displayed on a separate screen. 	E
3.02	The “Favourites” list can include any content that has been flagged by the User as a favourite. This therefore includes all base, free, subscribed and paid content.	E
3.03	Items in the “Favourites” list should be titled in the same way as they were listed when originally purchased.	E
3.04	If the number of items on the “Favourites” list exceeds the size of the screen then we expect the screen to scroll.	E
3.05	Tapping on any of the items listed on the “Favourites” screen should take the User directly to the start of that content.	E
3.06	It should be possible to unmark a favourite item	E

4.4	Test Management Templates
Reference	Description	Priority
4.01	“Test Management Templates” contentis accessible by a User on payment of a fee via the relevant app store.	E
4.02	Admin will determine the content to be listed.	E
4.03	The fee amount for each content item will be set by Admin and can be different.
If they are buying through the appstore, there are some challenges. Today we have 1 $ and tomorrow	E
4.04	It must be clear on the screen whether there is a fee to pay and how much it is or if the fee has already been paid.	E
4.05	An optional preview button will give the User a pre-determined view of a small part of the associated content.	D
4.06	Admin will determine which content items will have an associated preview.	E (contingent on 4.05)
4.07	Admin will determine the part of the content that will be displayed as the preview.	E (contingent on 4.05)
4.08	It should be possible for the User to purchase the content directly from the preview screen.	E (contingent on 4.05)
4.09	All content should be sorted alphabetically.	D

4.5	Quick Reference Guides: Process
Reference	Description	Priority
5.01	All content in the “Quick Reference Guides: Process” is base content.	E
5.02	The sub-categories that must be displayed are:-
o	Quality Assurance Definition

o	Programme Roles

o	Requirement Types

o	Test Planning

o	Test Scripting

o	Test Scheduling

o	Test Data

o	Test Execution

o	Defect Categorisation

o	Management Information	E
5.03	Within each of the sub-categories in ref 5.02 will be content that should be listed and viewable by the User. 	E
5.04	Content can be a mixture of text, image, audio and video.	E
5.05	When a User taps on any of the content then that content should be displayed.	E
5.06	The User should be able to mark any of the content as a Favourite which should then be processed as defined in section 4.3	E
5.07	Admin should be able to add, modify, delete sub-categories.	E

4.6	Quick Reference Guides: Testing Methodologies
Reference	Description	Priority
6.01	All content in the “Quick Reference Guides: Testing Methodologies” is base content.	E
6.02	The sub-categories that must be displayed are:-
o	Waterfall

o	V-Model

o	Agile

o	Rapid Application Development

o	Test Driven Development

o	Exploratory Testing	E
6.03	Within each of the sub-categories in ref 6.02 will be content that should be listed and viewable by the User. 	E
6.04	Content can be a mixture of text, image, audio and video.	E
6.05	When a User taps on any of the content then that content should be displayed.	E
6.06	The User should be able to mark any of the content as a Favourite which should then be processed as defined in section 4.3	E
6.07	Admin should be able to add, modify, delete categories.	E

4.7	Consultancy Support
Reference	Description	Priority
7.01	All content in the “Consultancy Support” is base content.	E
7.02	The sub-categories that must be displayed are:-
o	<<COMPANY>> Consulting (Change Management & QA Services)

o	Perfecto (Web & Mobile Automation)

o	Neotys (Web & Mobile Performance)	E
7.03	Within each of the sub-categories in ref 7.02 will be content items that can be tapped by the User in order to be viewed.	E
7.04	Content can be a mixture of text, image, links, audio and video.	E
7.05	When a User taps on any of the content then that content should be displayed.	E
7.06	The User should be able to mark any of the content as a Favourite which should then be processed as defined in section 4.3	E
7.07	Admin should be able to add, modify, delete sub-categories.	E

4.8	Quality Assurance CAST Tools
Reference	Description	Priority
8.01	All content in the “Quality Assurance CAST Tools” is base content.	E
8.02	The sub-categories that must be displayed are:-
o	ALM

o	TFS

o	Perfecto

o	UFT

o	Visual Studio
o	Test Manager

o	Khameleon

o	Neoload

o	Performace Studio

o	Selenium	E
8.03	Within each of the sub-categories in ref 8.02 will be content items that can be tapped by the User in order to be viewed. 	E
8.04	Content can be a mixture of text, image, audio, video and external links.	E
8.05	When a User taps on any of the content then that content should be displayed.	E
8.06	An additional button should be available that takes the User to the information that can be seen under “Training” for that item.
8.07	The User should be able to mark any of the content as a Favourite which should then be processed as defined in section 4.3	E
8.08	Admin should be able to add, modify, delete sub-categories.	E

4.9	Training
Reference	Description	Priority
9.01	All content in the “Training” is base content.	E
9.02	The sub-categories that should be displayed when Training has been tapped on the Home Screen are:-
o	ALM

o	TFS

o	Perfecto

o	UFT

o	Visual Studio
o	Test Manager

o	Khameleon

o	Neoload

o	Performace Studio

o	Selenium 	E
9.03	The following categories should be displayed when one of the sub-categories in 9.02 is tapped:-
o	About the Vendor
o	Bespoke Workshops

o	Scheduled Training Dates
Any specific format like table format
9.04	Content can be a mixture of text, image, audio, video and external links.	E
9.05	When a User taps on any of the content then that content should be displayed.	E
9.06	An additional button should be available that when tapped takes the User to the information that can be seen under “Quality Assurance CAST Tools” for that item.	H
9.07	The User should be able to mark any of the content as a Favourite which should then be processed as defined in section 4.3	E
9.08	Admin should be able to add, modify, delete sub-categories.	E

4.10	News and Events
Reference	Description	Priority
10.01	Latest relevant news and events should be shown here.	E
10.02	The categories to be listed are:-
o	News
o	Events	E
10.03	News and Event items are base content.	E
10.04	Admin will provide all content for these categories.	E
10.05	In the News category the content items should be sorted in “published to the app” order with the most recent item at the top.	E
10.06	In the Events category the content items should be sorted in time order of when the events take place with the next event being at the top	E
10.07	The User should be able to set a preference for how local the event information should be. The possible values should be National, Continental or Global.	H
10.08	Admin should tag each event with the country that the event is taking place in.	H
10.09	If a User has set a preference of National then only display events where User country = Event Country	H
10.10	If a User has set a preference of Continental then only display events where User continent = Event continent
10.11	If a User has set a preference of Global then display all events
10.12	In both the News and Event categories, a short Admin determined abstract should appear on screen together with the sorting date and which is tappable. 	E
10.13	On tapping the abstract the User should be shown the full content for that item.	E
10.14	If any of the content is multimedia then it should be played in the default player for the User’s device.	E
10.15	The User should be able to mark any content item as a Favourite which should then be processed as defined in section 4.3	E

4.11	Jobs/ Subscriptions
11.01	Content in the Jobscategory is subscribed content.	E
11.02	Users need to take out a recurring monthly subscription to access jobs information
It was not given how user should be subscribed.	E
11.03	The email address used to access the relevant app store will be used by the app – see 11.07	E
11.04	Jobs will be listed under the following Admin maintainable sub-categories:-
o	Test Analyst
o	Senior Test Analyst
o	Test Lead
o	Senior Test Lead
o	Test Manager
o	Test Architect	E
11.05	When a user taps on a sub-category then they’ll be shown the available jobs within that sub-category.
We need a clarification on how the jobs need to be posted. Is it a table format or free text	E
11.06	An “Apply” button will allow a User to apply for the currently viewed job.	E
11.07	Tapping the “Apply” button should generate an email to the registered address (see 11.03) that provides application instructions.	E
11.08	Job details will be supplied by an <<COMPANY>> partner and they will be stored by Admin in the Cloud.
What details of the job and what format they will store?	E
11.09	Admin will tag the jobs according to the sub-categories in 11.04
Can a job under more than one sub category?	E


phonebook.o
phonebook
etc~


I am happy to know how to use this awesome extention!


Свой код. И предположительно код будущих разработчиков РДО, кто захочет функцию поюзать. Насчет двух параметров попробую, может получится ничего.


sorry, that was the wrong change.  You should instead rename ActionManager in action_manager in the file addons/purchase_requisition/static/src/js/web_addons.js


Autodesk_Performance (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (750 of 750 tests passed)

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


@adrianschroeter It doesn't seem to deep to me yet and with indentation it seems more readable to me. Do you want me to remove the indenation?


Maybe change this to
```
const char *libs[] = { "foo", "bar", NULL };
```

And then just use for loop to try load in order. Fail if nothing got loaded.


Между селектором и фигурной скобкой должен быть пробел (Use a space between the last selector and the declaration block. (https://wiki.itechart-group.com/display/codstandarts/Coding+Standard+CSS))


If want to accelerate the delete ops from this sequential operation, how about using go routines on delete ops if each delete is irrelevant?


Fixed at https://github.com/openmrs/openmrs-module-xforms/commit/62b933000980b9cb9977720d3c366b28a3efe4ff


>Свой код.

Тогда оставь. `IResourceDelta` живет в сериализации или это более системный класс ?


There are other places where this change makes sense, so function that access array of pointers to char arrays and returns the loaded dl handle would be useful.


We really have to get rid of these assertXX methods in the ApplicationDriver interface. They belong in the steps layers. Otherwise ApplicationDriver is just going to explode with endless methods. The methods in ApplicationDriver should either query the application or execute a discrete action.

This makes it possible to implement both UiApplicationDriver and ApiApplicationDriver. We need ApiApplicationDriver in order to capacity-test the application, and right now we are building up a larger and larger debt of methods that have not been implemented on ApiApplicationDriver.


`Pormise.join()` is a more performant Bluebird method to use when we have a fixed number of promises to resolve


I guess this must go one line up (above the `switch` statement)


You're commenting on a commit... ;)


In Rspec please use let: https://www.relishapp.com/rspec/rspec-core/docs/helper-methods/let-and-let


Sure looks like it!


I enabled the robot tests and they fail: https://travis-ci.org/plone/plone.app.contenttypes/builds/76055949

It seems that the collection in the latest version of the 1.1.x branch of p.a.contenttypes is completely broken. I ran into that problem yesterday, this is why I looked at the robot tests in the first place. Any idea what causes those problems?


Оскільки тут йде **конкретна** привязка до корня проекту, а це треба уникати.


Where it's used?


We should probably mark all of these classes with `InterfaceStability.Unstable` as they will be removed once the old consumer is removed (the current plan is to deprecate the old consumer after 0.8.3 is out and a few people are using the new consumer).


in case of {'C:12@5x5', 'P:2x2,max', 'C:24@5x5,sc6'}, 192sec->103sec for one epoch


Replace with getFilterValues()


Yes, not good. I will remove this one.


abd


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1272 of 1274 tests passed)

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


We don't usually include `()` at the end when accessing methods that don't perform side-effects.


This commit implicitly adds a dependency on Sprockets *3*, which added the `DigestUtils` class.


Travis failed the build yesterday but has passed it today - maybe a glitch?


Copy this code in a file, name it test_spezialist.php and put it in your openDCIM folder. Then go to the URL [your openDCIM address]/test_spezialist.php in your browser. Tell me if it works.

<?php
require_once "db.inc.php";
require_once "facilities.inc.php";

$dotCommand = $config->ParameterArray["dot"];

$graphstr = 'digraph G {
        node[shape=plaintext]
        splines=ortho;
        {
                rank=sink;
                Legend[label=<

Legend

Electrical Measure Point	E

Cooling Measure Point	C

Air Measure Point	A

>]
         }
        test->{Hello; Spezialist;}

        "P10"[label=<

P10

E

> color=mediumblue];
        "Emerson_S23"[label=<

Emerson_S23

ECA

> color=mediumblue];

        Hello->"P10":IN;
        "P10":OUT->"Emerson_S23":IN;
}';

$dotfile = tempnam('/tmp/', 'dot_test_');
$graphfile = tempnam('/tmp/', 'graph_test_');
if(file_put_contents($dotfile, $graphstr)) {
        $graph = array();
        $retval = 0;

        exec("$dotCommand -Tpng -o$graphfile $dotfile", $graph, $retval);

        if($retval == 0) {
                header("Content-Type: image/png");
                //unlink($dotfile);
                print file_get_contents($graphfile);
                unlink($graphfile);
                exit;
        }
}
?>


Is this construct found anywhere in the code? I mean having the code right after the if or else and not in the next line?


Непонятно, в цикле создаются объекты, а удаляется один последний только объект:
```
              if _FileName > '' then
              begin
                SL := TStringList.Create;
                try
                  SL.CommaText := _FileName;
                  for K := 0 to SL.Count - 1 do
                  begin
                    Attachment := TIdAttachment.Create(Msg.MessageParts, SL[K]);
                    Attachment.DeleteTempFile := False;
                  end;
                finally
                  SL.Free;
                end;
              end;

              IdSMTP.Send(Msg);
              if gdNotifierThread <> nil then
                gdNotifierThread.Add('Сообщение отправлено: ' + _Subject, 0, 2000);

              if GetEmailAndLock(_ID, ES) then
              try
                ES.State := emsSent;
              finally
                FEmailCS.Leave;
              end;
            finally
              Attachment.Free;
              Msg.Free;
            end;
```


first like


Code style: unnecessary braces.


Для тэга ```<img>``` закрывающийся тэг не требуется:
```<img src="img/orange1.png" class="carousel-image"/>```. Также тэг ```<img>``` требует указание обязательного атрибута ''alt''


Все, связанное с `IResource` это эклипсовское апи.


Actually, I'd probably have a single call to `checkRequiredArgs` and have an `if/else` earlier to determine what the third parameter to the call was. Less duplication that way.


Style: braces are not required for single expression `try`


Style: braces are never needed in case blocks.


Yes that should be enough.


Можно ещё использовать новые семантические тэги, такие как header, footer, nav, section.


Why override if we just call `super`?


@ismooch once the flatfile is loaded everything is stored in a `Map`, that's why you need to reload configurations every time you make a change. Though this does prevent unnecessary `Map` lookups. :+1:


Shouldn't this be `IUserDialogs`?


We now use the `JUnitSuite` consistently in our codebase (this was recently merged). That is, based on JUnit 4 instead of JUnit 3.


Yup, i got it confirmed with all my users from SlimSaber, this fix resolved the issue. 100% confirmed.


Thanks for the feedback.


Небольшая опечатка: не нужен ";" в конце


sdgsdg


How is this multiplayer support supposed to work ? I've been trying to setup multitap on pocketsnes-libretro with no success.


This is probably the most confusing code I've seen yet :)
I understand what it's doing but don't tell me it can't be written much more simply and without any labels.
Pseudo code:
```
for each branch:
  if (branch.commits are fine) then append branch to bs else log "skipping"
```
and `are fine` is a pure function taking just the commits, storyByTag and allowedStates to implement the check done here with `CommitLoop`.


I added a test folder to the project. Go to the URL [your openDCIM folder]/test/test_Spezialist.php and tell me if it works.


sdgaa


2 Async here


Hi,
in my opinion,
instead of removing the sentence  "Deis runs on CoreOS version 494.5.0 or later in the Stable channel",
can you specify which precise version need to be installed in the Stable channel, and with which coreos-install command (with the user_data argument)?

best regards,

Charles.


Added test methods for testing HttpClient


Typo, `awe` -> `we`?


Style: space missing after `if`


No, do you have a traceback or just the failure in robot-tests? I quickly tried them (by hand) using the 1.1.x-buildout and they seem to be working just fine.


in rails_helper.rb and next in specs to use `create(:company)` instead of `FactoryGirl.create(:company)`


also adapt line 9


Do we need this? Why don't we do an orderly shutdown? The old code didn't seem to need a `System.exit(0)` for the normal case.


Don't test factories - they already throw errors when invalid


Oh please don't do that, setting it to krait enables some optimizations that improves performance without any side effects!


Thank you
![album sample_08](https://cloud.githubusercontent.com/assets/13358139/9326833/9e7b4e38-4551-11e5-924a-888d47ab4180.jpg)


There is ```be_invalid```


it is not too deep here, but when we use the same rule in lib/xpath_engine.rb for example it might become to deep.

However, I have no strong opinion about it, it was more a remark.


A slightly nicer way to write this is `println(s"Processed a total of $messageCount messages"))`


Тогда надо кидать другое исключение, например, `LogicError`.


If you run this evolution twice, will the badge_assignment_pool not be detected as not `IBadgeable` itself and be wrongly assigned this interface?


I like that you take the date format out of the view. Not sure though that is should be in the Application Helper. This one is for me! I am going to move it in a date format file.


Unnecessary space at the beginning will result in assigning to `attrsCfg['']` - `split()` keeps spaces:
```
 ~ $ node
> ' 1 2 3'.split(' ');
[ '', '1', '2', '3' ]
```


nope, better is to select subnodes idependly of node names, if in the future there would be more cameras.


yup, @augustt spotted this. Thanks for fixing


We should not test already tested rails features. You could test if the company responds_to users


This line will have no effect as it is after a return statement.


Please check shoulda-matchers gem https://github.com/thoughtbot/shoulda-matchers
You could use it to simplify tests.


I have now made a PR to use version 1.5.1 which seems to be the one recommended in the repository, short of changing to version 2.


need also update README.md under usecase-android-tests/embeddingapi/src/org/xwalk/embedded/api/sample/ and usecase-embedding-android-tests/embeddingapi-async/src/org/xwalk/embedded/api/asyncsample


excludes


given


Ссылка на email обычно оформляется через mailto:
```<a href="mailto:info@@johndoe.com">@johndoe</a>```


@aab-odoo  Done.!


need to add the UT result in the commit message
format http://otcqa.sh.intel.com/wiki/Crosswalk_Test_Suite_Commit_Message_Guide


You mean that we shouldn't test relations? A what about validations?


I spotted this, as all of the tests were failing and I'm in two minds about the best way of fixing it.

I already switched all the SpOcc calls in the example file and tests in `zoon` to match the expected `spocc` order (though not the vignettes). I think I'd rather leave it as a wrapper around `spocc` and keep the arguments the same.

I think it would be better to keep that approach, but to update the module docs to make it clear that `spocc` does things differently.

Could you undo this so I can accept the rest?


Now that you have the error message in place, why don't bring back the min length for the title?


`do_prediction` isn't used anywhere, maybe it should've been removed?


Not currently relevant for the CAD feed integration - this simple one is totally good enough for that purpose.


viewing first time


We need a `NOTE:` comment here with the description of the problem and the mention of the issue on github in GH-#issueNumber format


Don't need finally here.


That's not a "no" - it's just beyond my current needs ;)


This could be merged with line 1054. Been trying to reduce our selector count, and next up is deciding what is still redundant :)


啦啦啦


Honestly, are these labels considered a best practice in Go? It feels like assembler, the  code is hard to read and understand since execution control is jumping around by dozens of lines, so you can't just look at few nearby lines to understand what's happening. Unless there is a strong reason why you do it this way, I suggest removing all the labels (it can always be done like the way I suggested above and always leads to a much cleaner&clearer code IMO).


I understand, that these colormatps are more consistent with the default obspy style. Hower, the reason I built these colormaps was to be consistent with the software provided by Kristekova and their papers. The examples in the documentation looked exactly like the examples in their paper - this is now lost.

If you insist on heaving the default consistent with the obspy style, this is fine with me. Still I think instead of deleting the styles we should at least have the option to be consistent with the Kristekova style. Also, for the tfr plots, it is much more appealing to have a colormap that is white at small values.


For the sake of Twitter brevity, would prefer '1 hour' here (yeah, I know my '1 hours' was… laziness :^))


Maybe it's just me but the functions `localBranches` and `remoteBranches` look almost identical and could be merged into one function.


Fails when program is executed from another directory


Not sure that this is necessary.


Move test to `regression` tests section. Add issue number to the test name (`'{testName} (GH-{issueNumber})'`). We need better name for the test, which will describe not only the input of the test, but the expected behavior as well.


Nice!


\r-


你的文件夹命名不规范，请用你的学号，如咨询学号，请发邮件至Michael.qiao@boolan.com


Are we sure that is right?? I went into the code an it looked like the original min lat, max lat, min long, max long was correct...


Do we need this?


It's true, people rather ask in #salsaflow than add read the inline help :)
Still, since you've taken all this care to create what I think is great help pages, it's worth doing it really right.


For the seeds, it doesn't matter which topic goes with which source, and what level the sources are assigned. I think it would be a fun exercise if you iterate over the topic list, and in a second step assign a source and a level to the topic. (Mainly because for upcoming tasks, it is useful to exercise combining things to existing things; (like adding students to teams, supervisors to teams, etc) we don't have to create many totally new things with zips.


No problem then, I'll take your word for it.


Any better idea then?


Any chance to tag each new release with number only? I see libpsl-0.8.1 instead of 0.8.1 in tags which causes trouble to our tracking system. I don't care if it's intended for Debian, but please change back to normal ways.

Thanks.


masz apostrofy wewnątrz apostrofów,
bash tego nie ogarnie


This is breaking on OSX in fact as there is no clock_gettime present in there, i.e. resulting in:
collectd-tg.c:107:7: error: implicit declaration of function 'clock_gettime' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
  if (clock_gettime (CLOCK_MONOTONIC, &ts) != 0)
      ^
collectd-tg.c:107:22: error: use of undeclared identifier 'CLOCK_MONOTONIC'
  if (clock_gettime (CLOCK_MONOTONIC, &ts) != 0)
                     ^
2 errors generated.


Hah, sure.


This may cause issues further down the line. I'm not sure what situation we'll be in when the cache contains nulls. It may cause segfaults in ocaml, but we'll see then. I think we should make an architecture overview, describing what data goes where, what caches contain what data at what point in time, etc. Perhaps a google doc to collaborate on?


These procedures are probably not as efficient as they can be right now. I will spent some more time on this if this pull request gets some traction.


There are so many dependencies that imho for developer a tool for installing all deps would be useful. No one has to use it. But I agree the sudo rows should go to another script.


А почему .featured-block не помещен в див ```<div class="centered content">```?

правильно было бы обернуть весь контент в  ```<div class="centered content">``, который отцентрирован и имеет определенную ширину. Тогда для дочерних элементов, таких как ```.featured-block``` не нужно было бы писать такие стили как ```width: 63%; margin-left:auto, margin-right:auto```


From `?raster::Extent`:

> Slots
> xmin: minimum x coordinate
> xmax: maximum x coordinate
> ymin: minumum y coordinate
> ymax: maximum y coordinate
>
> Examples
> ext <- extent(-180,180,-90,90)

so unless they're getting flipped internally somewhere?


Yup, I reverted because otherwise all the translations would have to be updated. But on second  (or third) thought I’d say let’s get it right now, not later.


@papiyatoma Hello


I would be pretty fine with just more comments in that specific file.


why remove?


test comment


Because the latest Gibbon was (temporarily) removed from rubygems a while ago. Specifying a version in the gemfile helped. No I see, it is back, so we can try removing the version again.


You should include :
- leshan-core
- leshan-client-core
- leshan-client-cf
or
- leshan-all (which contains the client and server jar)

It seems you try to mix version 0.1.11-M1 and 0.1.11-M5, I don't know if it's a typo but this will not work.


Hi @mmhyamin - how's it going? Thanks for forking the Mozilla Study Group repo - let me know if there's anything I can do to help set up!


And another note, as soon as I resume my work, this file will automatically disappear from my repository. ;)


Most probably. Iterative refactoring FTW.

Feel free to jump in: the client is huge and _completely mad_


I'm not sure if we can predict that. I'd rather merge this now and branch off with different ideas on how to frame it than try to make a single version perfect.


nit: in flag description, indicates that the flag is ignored if --container-runtime is not set to rkt.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 358](https://teamcity.labkey.org/viewLog.html?buildId=267219&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5 (2 new), passed: 0 Build time: 00:09:53
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

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

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

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

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

```


It should depend on the grunt task. grunt watch should call wellington watch to take benefit of doing partial sass compiles and file watching. A grunt compile task should do a single compile then die.


101280704_LowTempRad (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1782 of 1797 tests passed)

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


In this unit test, a call to the size routine is in a for loop. The
first time through, an array is deallocated. The second pass through the
call in the for loop crashes.

for ( int iSizingType = DataSizing::None; iSizingType <=
DataSizing::FlowPerCoolingCapacity; ++iSizingType ) {

SizeUnitarySystem( UnitarySysNum, FirstHVACIteration, AirLoopNum );

}

I guess I could split up the unit test, getting rid of the for loop (and
boolNotAUnitTest), so as not to set a precedence.


Dang, it really didn't take much to slim this down for the browser. Is there any reason this couldn't make it into the main Hoek repo?


I'm telling you that's what's going to happen.

On Tue, Aug 18, 2015, 8:02 PM Alex Kern <notifications@github.com> wrote:

> I'm not sure if we can predict that. I'd rather merge this now and branch
> off with different ideas on how to frame it than try to make a single
> version perfect.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/HackathonHackers/code-of-conduct/commit/e2a7bfb43fbadf4d7b4c7f56726639db5784e15a#commitcomment-12773542>
> .
>


THis is a comment


Maybe we should replace some special words with happy words.

Cunt turns into rainbow or something


asd


also added setnx cmd


adsf


101280704_LowTempRad (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


QSickRage package?


I redid the code per your reference and link: (Note the relationship and ForeignKey mention). Correct?

```
class Assessment(Base):
    __tablename__ = 'assessments'

    assessment_id = Column(Integer, primary_key=True)
    name = Column(String(50))
    text = Column(String(2000))
    created_on = Column(DateTime, default=datetime.utcnow)

    user_id = Column(Integer, ForeignKey('users.user_id'))
    user = relationship('User', backref='assessments')
    video_id = Column(Integer, ForeignKey('videos.video_id'))
    video = relationship('Video', backref='assessments')
    category_id = Column(Integer, ForeignKey('categories.category_id'))
    category = relationship('Category', backref='assessments')
    element_id = Column(Integer, ForeignKey('elements.element_id'))
    element = relationship('Element', backref='assessments')

# many to many
    categories = relationship(
        'Category',
        secondary='assessment_category_link'
    )

    def __init__(self, name, text, user, video, category, element): # OBJECTS !! user...
        self.name = name
        self.text = text
        self.user = user
        self.video = video
        self.category = category
        self.element = element

    def __repr__(self):
        str_created_on = self.created_on.strftime("%Y-%m-%d %H:%M:%S")
        return "<Assessment(name='%s', text='%s', user='%s', video='%s', category='%s', element='%s')>" % (self.name, self.text, self.user, self.video, self.category, self.element)

```


Cool, I was just about to raise an issue for this.


Closes #2


LogPrintf("\n\n\n"); //A bit excessive??? - :)


Is this going to become a Pull Request?


Is it my rewording or this new section in general?


What client/device did you see this in?


What I mentioned in chat.

On Tue, Aug 18, 2015, 8:11 PM Alex Kern <notifications@github.com> wrote:

> Is it my rewording or this new section in general?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/HackathonHackers/code-of-conduct/commit/e2a7bfb43fbadf4d7b4c7f56726639db5784e15a#commitcomment-12773653>
> .
>


Android.


Unless absolutely necessary avoid using !important tags. Also, you can just use table:last-of-type instead of attaching it to body. It's a given that table would be inside the body tag.


Why did you change it to 10?


I think it should be fine since I don't believe that using the random filename method is valid in the browser anyway. I think that is the only thing that would break.


yeah this was a really big win!! :-)


Yes done that...


The : could also be moved against the '. It forces it to a symbol for better memory mgmt. accidental space is accidental. This fix is totally valid though, just explaining why the : was there.


94006714-MultipleTowers (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1801 of 1801 tests passed)

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


Ok. Will try that. Still a haml noob haha.

On Tue, Aug 18, 2015, 8:16 PM Matthew Machuga <notifications@github.com>
wrote:

> The : could also be moved against the '. It forces it to a symbol for
> better memory mgmt. accidental space is accidental. This fix is totally
> valid though, just explaining why the : was there.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coderie/coderie-site/commit/f415955238cb13589c27d3add0e520b8381dda36#commitcomment-12773710>
> .
>


[PEP8](http://legacy.python.org/dev/peps/pep-0008/) (official python styleguide) is quite clear about this: **USE SPACES, NOT TABS!** :angry:


Remove inline styles and put in a .css file or in the head.


What does the 225 * 100 stuff do? Or is it the magic sauce that goes into calculating frecency?


This event is already being triggered in Core/Player/PlayerTracker


I don't see a screenshot, can you elaborate?


It's the magic frecency falloff sauce: the coefficient falls off as the date ages. You'll see I've made some unsupported changes to the local formula. It improves the ranking in my top sites, up weighting local places. (We should plot graphs for these.)


@pavel-kirienko No need for this sleep(). It works fine without it.


ah, I see.


Great commit. This is coming along very well! Thank you!


this commit broke ipfs when using the toolbox on OSX. works fine on remote box.


Remove inline styles from anchor tag and put in head or external .css file


@dotta Change reverted and committed to 2.4.1.  VoltDB users will have to get relief for this issue through a correction in the VoltDB JDBC driver.


101280704_LowTempRad (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1782 of 1797 tests passed)

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


Hi @liwang-pivotal !

I wonder why it is empty, if you say that you are in the cluster with 3 nodes...
Or all of them are on the same machine ?


delete file


Are you sure that you really have permission to write file into the `root` ?


Oh. Line 64 of docker_client.go should refer to the IP address of the docker machine. Get that with `docker-machine url` and massage accordingly. Also will need to exempt Linux from that lookup.


Thanks @kennethormandy for the help. Open-source FTW.


inverse also works


Doh... Never mind. It is a default option, so you can specify it as a module option. Like you have now on your admin:
```
 	myFTP --redisHost=52.21.147.62 | log
```


Or you could report an issue against _Eggs.Variant_ instead, and that would give me a motivation to fix it upstream :)


Does this require anything else?


@alainjobart

Please see the commit message of this commit. Healthcheck doesn't return "Target" when QueryService is not serving. Let's discuss tomorrow how to fix this.


构造函数


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1273 of 1273 tests passed)

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


looks good!


I believe this is the cause of this error:

ERROR: notssdb.test.test.test1
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/Users/ack/code/venv/NotssDB/notssdb/test/test.py", line 14, in test1
    u1 = api.create_user('dance', 'cats', 'Jen', 'C', 'dancegirl@aol.com')
  File "/Users/ack/code/venv/NotssDB/notssdb/api/object.py", line 35, in create_user
    new_user = User(username, password, firstname, lastname, email)
  File "<string>", line 2, in __init__
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/instrumentation.py", line 347, in _new_state_if_none
    state = self._state_constructor(instance, self)
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/util/langhelpers.py", line 747, in __get__
    obj.__dict__[self.__name__] = result = self.fget(obj)
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/instrumentation.py", line 177, in _state_constructor
    self.dispatch.first_init(self, self.class_)
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/event/attr.py", line 258, in __call__
    fn(*args, **kw)
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/mapper.py", line 2792, in _event_on_first_init
    configure_mappers()
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/mapper.py", line 2688, in configure_mappers
    mapper._post_configure_properties()
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/mapper.py", line 1710, in _post_configure_properties
    prop.init()
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/interfaces.py", line 183, in init
    self.do_init()
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/relationships.py", line 1620, in do_init
    self._generate_backref()
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/relationships.py", line 1811, in _generate_backref
    (backref_key, self, m))
ArgumentError: Error creating backref 'assessments' on relationship 'Assessment.category': property of that name exists on mapper 'Mapper|Category|categories'


I meant why support Python 2.6?


Sorry about the whitespace changes, they were made automatically by my editor. This hasn't been tested yet, but I can redo this.


Do we need some enums? :P


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


@yugui although this is good in decoupling configuration from plugins, it adds a coupling of plugins to the config loading. it means if a new plugin is added and needs a config var, it needs to be added to the main config loading function. i suppose it is okay, but i was wondering your thoughts on maintaining the independence of plugins from the main application.


rax retry tests


Typo: `wanings` -> `warnings` in this filename (and all the `#include`s)

Or, IMHO, we should remove this and just `-Wno-float-equal` and open an issue to make the parameters use the correct types. If there are more misuses added, they're going to be with the same parameters array.


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - i386-Windows-7-VisualStudio-12: Tests Failed

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


this seems to break the client and i dont see any error with it uncommented.


Hate to beat a dead horse, but interested so I can understand it a bit more.

https://github.com/clintonskitson/docker/commit/e6e2049e48bae5e583aab93213804e8269466067

The commit here has 9 tests that are passing and seems to be operating as I would expected without container context.. which combination of commands do you think would trigger a problem?


This is a nice construct. I personally find multi-word keywords awkward though. Have you considered the same but with underscores, i.e. `suffices_to_show`? Or maybe even just `suffices`?


+1 to @silverwind's comments here, this is a big deal


helloooooo git hub


The previous wording was correct


How about a select or something for countries - to restrict countries to the WCA countries? Free-form input may lead to some mistakes.


Good separation now :+1:


I didn't bother because this problem goes away once we use WCA accounts for registration.


`String.prototype.normalize()` or `String#normalize()` (idiomatic but perhaps not as recognisable), the `()` being the missing bit


> This is a nice construct. I personally find multi-word keywords awkward though. Have you considered the same but with underscores, i.e. suffices_to_show? Or maybe even just suffices?

Ok, I will change it to `suffices`.


i liked the environment var for this stuff since I don't have a /users/saurabh...


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 359](https://teamcity.labkey.org/viewLog.html?buildId=267235&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:54
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


I get a complaint here since ENABLE_NEUTRON is a boolean:

```
shell provisioner:
* Shell provisioner `args` must be a string or array.
```

Obviously you are passing in an array, but it still complains. Using Vagrant 1.7.2.


The sidebar/non-sidebar page templates came out of the box with the theme (which I'm assuming they chose for it's freeness and not it's responsiveness). Probably fine to kill this off.


Interesting idea. We should try it out. Would be interested to see how mingw handles 32/64-bit support and support for extended floating-point. It might have better features than Visual Studio actually, which is what I use to build these now.


This needs to be reverted because subtitles is small and wrong positition.
With this reverted it works as it should.


Yes, I think it is acceptable for now because that kind of dependency is already introduced by pivotal.

But we should eventually remove the dependency by adding an etcd directory for plugin configurations
e.g. `/projects/foo/plugins/travis_token`.


Good changes :)

One request: on the old website, I had no idea that the name/wcaid/etc fields were editable. This functionality exists here, but is still not obvious. Can some suggestive styling be added (maybe cursor: pointer and/or a small edit icon or something) for these fields?


:+1: to that.


The else case for the ternary operator should be `val` and not `envName`.


I think the card schema needs to be adjusted.

It seems like there are two types of cards now, "non-permanents"/"spells", e.g. `Laser Blast`, `Errant Blast`, and `EMP`, and "permanents", e.g. the robots. So every card should get the `permanent` key and if it's a spell, it will be `false`. Or, we could use a `type` field instead with a string value of `permanent` or `spell`, leaving room for other types in the future.

Also I don't think there needs to be a unique field for each spell (`kill`, `requiresTarget`, and `emp`). Each spell might instead get the same `target` field, and the values could be `1`, `all`, `random`.


Ah, makes sense.

It may still be beneficial to allow editing these fields, even after user accounts are added. Eg, people mis-caps their names surprisingly often (john doe vs John Doe), mis-type birthdates, etc. We could somehow force user data to be consistent with results data when possible, but this doesn't help for new users.


### Minor nit
I see a warning for this line:
`luigi/luigi/rpc.py: SyntaxWarning: assertion is always true, perhaps remove parentheses?`

This is what worked for me as a local fix:
```py
        assert \
            not (url.startswith('http+unix://') and not HAS_UNIX_SOCKETS), \
            'You need to install requests-unixsocket for Unix socket support.'
```


No funciona porque les falta algo


I'm seeing infinite recursions from this function. I can stop them by adding  "_paths" into the array on line 24.


this should not be specific to a program, can you rename it to something like
``"FirstMapReduceProgram",
"SparkProgram"
..etc``

something generic and easy to understand.


There are few other "error" related issues open that this may address. Just documenting them here until we dig deeper. #695 #480


Why remove option?


测试了一下，注释掉这一句，然后openright，空白页是有页眉的。


Good eye -- nice! :+1:


```bash
cid=$(docker create -v /foo busybox)
cid2=$(docker run --volumes-from $cid busybox true)
docker rm $cid2
docker rm -v $cid
```

This will not be able to delete the volume since the local driver's count is going to be off.


You can see the full traceback on the travis job: https://travis-ci.org/plone/plone.app.contenttypes/builds/76055949


I consider pull requests independent work. One person works on a forked repository at a time and submits pull requests to be merged into others' work. A repo can be collective work, but generally I think pull requests are individual work.

Maybe add "GitHub has a different take on the meaning of fork" to it? I don't know what to add or remove.


101280704_LowTempRad (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (749 of 749 tests passed)

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


So much code to thread lastToken through. It looks like that gets set in the constructor of ```FormattingContext.InitialContextFinder``` where it's called "endToken". How are you sure this represents the last thing the user typed? Couldn't it just be the end of any formatted span?


Agreed. Those kinds of edits will be done over at https://www.worldcubeassociation.org/users/8/edit. Keeping stuff in sync with results data is a whole different beast that I have some ideas for solving (basically only let people edit their information if they're not associated with a WCA id).


should do similar change even in test.


Prefer the ternary if myself, but in this case the original was definitely more readable.


True... Pivotal notifications should also be a plugin in a refactor.

Okay, LGTM!


nada que ver


You're right. I like detail, but perhaps it's unnecessary for the purpose of this glossary. I'll edit it later, if no one else has already done it by then.


@MikePall This is why we need to make a clone of you that does our will. :D


Same as prev comment. The original code was much more readable.


The plan is to add small edit icon that links to pages like https://www.worldcubeassociation.org/users/8/edit once we have WCA accounts being used to register.


Can you add a test for this fix?


xccxcx


So you should be ok, clinton-hall package is not up to date.


Not sure about the intention here - why model_id part was removed? It was representing resource which is exposed by rest api on which you can call get.


1774_add_misc_objects (mbadams5) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Oh noes!!! 😱


ほんとですね :)


Hi, I tried building IoTAgent, I found that  the cmake option MQTT cannot work at my clean ubuntu14.04 .


1774_add_misc_objects (mbadams5) - x86_64-MacOS-10.9-clang: Tests Failed

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


Might be a little nit-picky, sorry... Is this necessary. As the game is set up, it is impossible to move a piece off of the board. It's a matter of how we set up the movement on click of the `div` in the table. I'm just wondering if we are adding extraneous conditionals to our code or if we should be taking these precautionary steps. A good questions for Mike maybe?


I really dislike this word. But I'll allow it.


rax retry tests


Lol you didn't like it in Vagrant either :smile:, but I think it makes the most sense.


TeamCity ATK :: QA :: Integration Per Branch [Build 1887](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546529&buildTypeId=Atk_IntegrationPerBranch) is now running


FourPipeBeamAirTerminal (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1802 of 1802 tests passed)

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


You never know what its going to do. `gem pristine` doesn't "ignore environmental variables", `vagrant pristine` would've done something totally different from gem and this. Its just a weirdly generic word that feels too much like "util" to me.

</rant> :P


Like I said, its fine, but I would've just done `disable_env_inheritance` or something like that.


名詞にしてみました。


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

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


Addresses #2256


Hello burjorjee,

First of all, thank you so much for your contributions to this (and of course ryepdx's as well)!! I'm confident this has dramatically improved my chances of recovering my password and saving myself from my own idiocy. I had a question around the functionality of using the command:

pyethrecover.py -w nameofwallet.json -s password_spec.txt

namely, will it stop running the module as soon as the right password is found? My hope is that it does, as I am having it run through tens of thousands of possible combinations at a time, and it'd be awesome to know that I can just leave it unattended to do its thing. Just wanted to see if you could confirm how it works.

PS: I'm a complete neophyte when it comes to running things out of the command line prompt, so appreciate you bearing with me if this is all very obvious.

Thanks in advance--definitely appreciate any input!


Hi, Could you please specify which linux distro that the code failed to run upon?


Hmm. Isn't ``String`` renamed ``AbstractString`` on 0.4?


@btovar this commit needs to be in 5.1, and hopefully deployed soon. Thanks!


better name it CaretakerNfcActivity to indicate it is an Activity


Where's the jumping-up-and-down emoji when we need it! :+1:


我搞错了，这句确实多余

> On Aug 19, 2015, at 9:09 AM, Dongdong Tian <notifications@github.com> wrote:
>
> 测试了一下，注释掉这一句，然后openright，空白页是有页眉的。
>
> —
> Reply to this email directly or view it on GitHub.
>


1774_add_misc_objects (mbadams5) - i386-Windows-7-VisualStudio-12: Build Failed

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


```jl
julia> String === AbstractString
true
```


``String`` is the old deprecated name


This is an ongoing project to build a website for my family.


Should this be here?


Wow! Nice work.


OK, now I hear you. Thanks for pointing that out.


I think this is just left over garbage that isn't used anymore.  There is no references in the nuspecs or code to this in the active stuff


101280704_LowTempRad (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1782 of 1797 tests passed)

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


Comentando esto..


If you add two spaces at the end of a line it will give you a rtn so the dots appear on the next line


I wonder if the difference is all from converting ``""::ASCIIString`` to ``""::UTF8String`` in the ``Variable`` constructor...


Any particular reason why we need an specifict match case for String?


Could you test? I'd rather have concretely typed strings.


No worries! It's just a weird ruby cheat thing that probably shouldn't exist anyway. Kind of a way to trick the vm into thinking it's a symbol literal.


ここを変えた理由って何ですか？


今の状態だとメソッド内でparams を直接あつかっているので微妙だったのですが、書き方を変えてモデルに切り出してみます :)


`$this->request->data` に再代入した理由は何ですか？


It's usually a good idea to leave white space between the braces and the innards of the hash as well to help isolate syntax bugs like this


テストを組んでるうちに、薄々そんな気がしてました。


ok..

哈哈，其实你也可以自己动手，psd也在上面了


Forgot to comment in the commit, related to #450.


S82_DR7.csv will required you to match the column nyuID to column col0 in result_DR7.csv.  Then after combining the two files, you will have the required values to make the S82 and DR7 image have the correct sizes.


May or may not be helpful: https://github.com/phergie/phergie/blob/master/Phergie/Plugin/Encoding.php

That was what we used in Phergie v2 for handling encoded data.


[icon1_1.png](https://github.com/fritx/markppt/blob/dev/artwork/icon1_1.png)
这个图片本来就是正方形的了😂 512x512


closes #110


what sdid I do?


can you log an issue with code and the eslint output please.


form_sanitizer() defaults checks returns a phpentities converted plain text before going into sql. To reconvert back to html - pull $data out., and wrap in html_entity_decode() after stripslashes(). Same goes to serialized data.


I also don't know why there's a warning. Maybe it's because class_exists (unless the second parameter is set to false) will actually invoke the autoloader and will fail. On the other hand this class might not be loaded add the time so skipping the autoloader will result in not detecting the Enterprise_PageCache module. In any case your solution is much cleaner. Thanks for contributing it!


OK


提交测试代码


101280704_LowTempRad (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1273 of 1274 tests passed)

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


Hola Mundoooo!


visit の段階では ImageUploader は使われてないのでたぶん大丈夫だと思いますが、厳密な挙動がわかっていないので、上に書いた方が安全かもしれないですね。

`ImageUploader.enable_processing = true` をこの background に書くのは適切ではない、ということは意識してたのですが。
セッティングなので scenario に書くのもどうかと思い。

まだ rspec の feature  / scenario の適切な段組方法が掴みきれないこともあり、この書き方になってました。

- visit はscinario内に書く
- ImageUploader.enable_processing = true も今のところ scinario内に書く

としてみます :)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.516](http://teamcity.monogame.net/viewLog.html?buildId=9761&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


TeamCity ATK :: QA :: Integration Per Branch [Build 1895](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=547011&buildTypeId=Atk_IntegrationPerBranch) is now running


release: '/static/${namespace}/pkg/widget.js', // fis.set('namespace', 'home')
应该改成上面这样，逗号放在末尾会出错（被误认为逗号属于注释内容）


TeamCity ATK :: QA :: Integration Per Branch [Build 1895](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=547011&buildTypeId=Atk_IntegrationPerBranch) outcome was **FAILURE**
Summary: Failed to resolve artifact dependency Build time: 00:00:07


release: '/static/${namespace}/pkg/widget.js', // fis.set('namespace', 'home')
应该改成上面这样，逗号放在末尾会出错（逗号会被误认为属于注释内容）


i dont like this


Was this allocated by the BIP maintainer? If not, it should be a placeholder like "XX" or "schism-hardforks"


Missing angled brackets around email address.


No, They still haven't released a newer version.


missing files :cry:


RE: the issue in #1038 where you would like to fix the formatting if both plugins are enabled, you could do something such as:

```javascript
$percentages =  $('#resodetails span.meter-level div')
if ($percentages) // There are already % displayed
   $percentages.after('No. Days')
else
   // Display as normal
```

A simpler test, though it couples you the definition of the reso % plugin would be:

```javascript
if (window.plugin.resoEnergyPctInPortalDetail)
  // There are already % displayed, act accordingly
else
  // No % displayed, act accordingly
```

In addition, plugin scripts are loaded in the order they are added to tampermonkey/greasemonkey. I'm sure while you're testing, the % plugin is loaded before yours, have you considered what happens to the display of that when your plugin is loaded first... you may need to update the % plugin to suit.


It would be great to squash the two commits into one

for the comments to be capitalized

`// css task` to be `// CSS task`

`// javascript task` to be `// JavaScript task`

:)


If the request fails resolve wont be called and the promise will never be fulfilled etc etc. This should be addressed in the future.


add new line here, please


Here is a version with mode code reusesage etc

```csharp
 [Timing(TimingTypes.ThreadYieldMicroSeconds)]
    public class ThreadYieldMicroSecondsStrategy : YieldThreadStrategy
    {
        public ThreadYieldMicroSecondsStrategy() : base(true) { }
    }

    [Timing(TimingTypes.ThreadYield)]
    public class YieldThreadStrategy : Timing
    {
        private readonly MicroStopWatch watch;
        private long lastWaitEnded;

        public YieldThreadStrategy(bool unitInMicroSeconds = false)
        {
            watch = new MicroStopWatch(unitInMicroSeconds);
        }

        public override void Wait()
        {
            EnsureWatchStarted();

            while((watch.ElapsedUnits - lastWaitEnded) < ThreadExecutionInterval)
                Thread.Yield();

            lastWaitEnded = watch.ElapsedUnits;
        }

        private void EnsureWatchStarted()
        {
            if(watch.IsRunning)
                return;

            lastWaitEnded = 0;
            watch.Start();
        }

        public override void Dispose()
        {
            watch.Stop();
        }

        private class MicroStopWatch : Stopwatch
        {
            private readonly bool unitInMicroSeconds;
            private static readonly double MicroSecPerTick = +1000000D / Frequency;

            public MicroStopWatch(bool unitInMicroSeconds)
            {
                this.unitInMicroSeconds = unitInMicroSeconds;
                if(unitInMicroSeconds && !IsHighResolution)
                    throw new Exception("On this system the high-resolution performance counter is not available");
            }

            public long ElapsedUnits
            {
                get
                {
                    if(!unitInMicroSeconds) return ElapsedMilliseconds;

                    return (long)(ElapsedTicks * MicroSecPerTick);
                }
            }
        }
    }
```


TeamCity CommitStatusPublisher :: Github.Original [Build 18](http://192.168.2.85:8111/bs/viewLog.html?buildId=5830&buildTypeId=CommitStatusPublisher_GithubOriginal) is now running


If a new broadcast is created, it automatically removes the previous service and republishes the new one. This happens even during the renaming of the service.


add a comment to explain what this is exactly


TeamCity CommitStatusPublisher :: Github.Original [Build 18](http://192.168.2.85:8111/bs/viewLog.html?buildId=5830&buildTypeId=CommitStatusPublisher_GithubOriginal) outcome was **SUCCESS**
Summary: Success Build time: 00:00:01


@hoholee12 oh :S he'll be on bitches & boats or something xD... @DiamondBond 4 PRESIDENT


/cc @pranavkm unblocking the build. You might want to take a look


SerializerFeature.IgnoreNonFieldGetter :+1:


it's not necessary to erase i_res. I think if find the result
just (*i_res) = result should be ok. else push_back(result)


@Knudah u plz fix @jarlebass 's nooby push?


Don't know if you want to use https://github.com/mysociety/alaveteli/pull/2741 for adding the missing method and make this just reload the association, or just use this commit. I don't have much of a preference.


1. use [rails_routes definition of this path] (http://guides.rubyonrails.org/routing.html)
2. use ```link_to``` helper


TeamCity CommitStatusPublisher :: Github.Original [Build 19](http://192.168.2.85:8111/bs/viewLog.html?buildId=5831&buildTypeId=CommitStatusPublisher_GithubOriginal) is now running


This is a total mess, why did you change the formatting?


TeamCity CommitStatusPublisher :: Github.Original [Build 19](http://192.168.2.85:8111/bs/viewLog.html?buildId=5831&buildTypeId=CommitStatusPublisher_GithubOriginal) outcome was **SUCCESS**
Summary: Success Build time: 00:00:02


adding @pranav91


what the hell is this?
```Rails
= link_to 'Refresh' page_refresh_path
```


new line here


请直接提 PR 修复。


```ruby
visit edit_admin_outgoing_message_path(outgoing_message)
```

https://github.com/bbatsov/ruby-style-guide#no-dsl-parens


if video_ts and res_ts just as sample code. you can comments them out now.


А чего маленькими буквами?


please read [about difference between gsub!, gsub and sub methods] (http://stackoverflow.com/questions/6766878/what-is-the-difference-between-gsub-and-sub-methods-for-ruby-strings)

you should select the most fast & suitable method


I would like to suggest best tool to easily restore emails from an offline OST file. You can download this software from:-

http://www.convertmicrosoftosttopst.org


We won't need network for location in this application because network won't provide NMEA sentences.


@rahulcs are we using unbxdprank currently ?


list 제거


Yes, he didn't do so in http://lists.linuxfoundation.org/pipermail/bitcoin-dev/2015-June/008936.html but @gmaxwell can confirm that he did indeed assigned 99 to this BIP draft.


Thanks, I will correct it.


hello @BrittoAnthony


I don't think you even need that part at all. The Java server will automatically set that value. The "from" is used when sending the message from server to clients, not from a client to the server (as the server knows what socket that sent the message, and knows what username is associated with that socket).


amended, thanks !


To be honest, I think we should write the additional tests to get this back up to 99%. The more tests, the better.


Recommendation: Create a function `main()`, and have it be the *only* statement executed at the module level.
Example:
```
if __name__ == '__main__':
    main()
```


Why do these cs-gpios declarations not have a third parameter?


In the future please use [`npm version [major|minor|patch]`](https://docs.npmjs.com/cli/version)  to increment the package version since it will automatically reset the proper numbers to 0 if a major or minor version has increased.

This is fine for now since you already published :)


Thanks for clean up the style mishaps!


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 9](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=97&buildTypeId=XbimEssentials_XbimEssentials) is now running


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 9](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=97&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:00:43


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 10](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=98&buildTypeId=XbimEssentials_XbimEssentials) is now running


TeamCity ATK :: QA :: Integration Per Branch [Build 1896](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=547023&buildTypeId=Atk_IntegrationPerBranch) is now running


TeamCity ATK :: source code Package [Build 8260](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=547022&buildTypeId=Gao_PackageSourceCode) outcome was **SUCCESS**
Summary: Running Build time: 00:09:50


I see you have made use of you Editor role. Perfect!


TeamCity ATK :: QA :: Integration Per Branch [Build 1896](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=547023&buildTypeId=Atk_IntegrationPerBranch) outcome was **FAILURE**
Summary: Failed to resolve artifact dependency Build time: 00:00:11


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 10](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=98&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:00:44


please add this requirement in the README...


only to send the rank in the ga. unbxdprank is basically the loop index itself.


This part is a good example that before integrating your work we should have the common things separated given there are almost no differences. But this is ok for now


Note you can also keep the window open by running without the debugger (Ctrl-F5). Up to you whether to include that in these docs.


Maybe time to open as a pull request?


long line.


long line


good catch - because I am so far unable to test it...


Is there a reason why we're mixing usage of parens and no parens?


Плохое название для define.


Yep, nice refactor.


The current condition prevents move count based pruning for fresh SpNodes.
It prevents the creation of very short living subtree searches. A subtree search which is directly pruned/terminated after creation because its move count is too high.


fixed in 63e8ec37bb8cbd80244abb61a267f59f3c35eccb


why?


Sorry, this was thanks to ChaseGiunta (previously spelt wrong).


SQLite3  у тебя в девелопмент депенденси =>
это гарантированно упадет если подключишь как гем => вообще ничего работать не будет


@sbrunner How should we do?


ok. cc @pymd


этого тут быть явно не должно


как и этого


Recommendation: Python 3.4+ `pathlib.Path` module.
Example:
```python
from pathlib import Path

input_dir = Path(sys.argv[1])

for year_dir in input_dir.iterdir():
    for day_file in year_dir.iterdir():
        print('Processing: {}'.format(day_file))
        with dayfile.open() as f:
            lines = f.readlines()

```


There's some context about the effectiveness of our coverage reporting here: https://www.pivotaltracker.com/n/projects/790363/stories/72169016


и этого тоже


Awesome! Great work steven!


> We didnt discuss on the need for author master. Is it required ? Considering same author names may appear differently in different books, is it required to master this ? We need to know from Yasho. Also, is it there in Merlin now ?
> Even if required, what is the need for the is_pseudonym and author-sort fields ?
> @KrishnaRamamoorthy

Actually, the reason for this was partly because the current system wasn't working. I didn't realize it hadn't come up in the discussion (or had it? I thought I would have noticed it since it was already in the code). The problem Yasho had *was* that the name was written differently in different books. So it was hard to search by author because there were so many different versions of the names (`Philip Pullman` vs `Pullman, Philip`  vs `Philip P.`) An author master would help because then all the names can be written (and therefore the authors can be referenced) more consistently.

The `author_sort` field is an auto-set, standardised version of the name (eg. `Asimov, Isaac`) which can be used for searches and sorting (sort by author). The name is the human-readable version, eg. `Isaac Asimov`. Actually this hasn't been thought out that well (do we really need a separate field for author as-it's-written-on-the-book?). The `author_sort` idea was based on what Calibre does for its books. On second thoughts,  for our case it might be better to store the name standardised, and then later generate the human-readable version on-the-fly.

Fine, maybe there's no need to have `is_pseudonym` for now (at least, not unless we plan to store more detailed author profiles).

Anyway, if it wasn't discussed, maybe we should mark it for discussion in the next meeting?


это не часть гема


The change only affects the `FrontendUser` class. The `file.php` entry point is back end related.


мусор


    nvd3 graph for # of plans and partner/nonpartner

    - http://localhost:8001/planinfo/HYP displays graph of all plans that
      start with HYP
    - http://localhost:8001/planinfo/partner/HYP displays graph of summed
      amount of all plans that start with HYP grouped as partner/non_partner
    - use vdloo python-nvd3 fork with older d3.js, the upstream python-nvd3
      is borked

      it turned out that the latest python-nvd3, nvd3 and d3.js aren't
      completely compatible. fixing it would cost more time than it's worth.
      forking and hardcoding the latest d3.js (3.2.0) that worked was the
      least time-consuming solution.

      for future reference: alternative workarounds for the error
      https://github.com/novus/nvd3/issues/330 would be to locally host 3.2.0
      in ./bower_components/ (see
      https://github.com/areski/python-nvd3/blob/develop/nvd3/NVD3Chart.py#L46)
      but we don't use bower so that's a dirty hack or write a method for
      chart object that sets the d3 useVoneroi to false.

      this will probably be fixed by someone else upstream at some point and
      right now we aren't missing any features with the 3.2.0 workaround.


Hm, I hadn't meant to check this in, this was for testing only.
I'll revert it.

> On 19.08.2015, at 10:34, Kevin Laeufer <notifications@github.com> wrote:
>
> Shouldn't we create a separate example for this? Like software_timer or systick or something like that?
> The blink example is supposed to be very minimal.
>
> —
> Reply to this email directly or view it on GitHub.
>


тоже лишнее


So far I've practically only received feedback from you.
There's also some parts that people are clearly against (ie activation of all uncontroversial hardforks with a given height + 95% mining confirmation after that height).
That is being discussed in http://lists.linuxfoundation.org/pipermail/bitcoin-dev/2015-July/009731.html without a lot of participation.
I would like that discussion to advance more before opening the BIP and uncontroversial hardfork code proposal as PRs.
Well, for the hardfork, it would also be nice to have the nVersion bits code deployed first too (IsSuperMajority is not expected to exist in 5 years [or whatever time we want to give users to upgrade]).
I can still open both as PRs and just keep them updated, but given that there are 2 threads in the mailing list to discuss it, I don't think that will be very valuable.


running **them** maybe? I mean the point here is that we are running tests with Zeus.


если предусмотрен только sql lite - то хорошо бы привязаться к его ошибкам, а не к общим


@KrishnaRamamoorthy: I've put more replies inline. The `publication` table was from the auto-generated code; I had already removed in the next commit (f0b1cbbf337bf63ce7acaf05bc946750c25f1b07). User Role is being implemented using a framework (Flask-Principal), so I'll have to learn how that works before including it into the table. I'll add genre and item type: thanks for pointing those out.


aah, good point ...


Personally, I think especially since it been allocated a number is should definitely be a PR now. Discussion about the general concepts and ideas can still occur on the list, but nits and grammar review is best done on github.


зачем?


and a model ;)


Now Runkit stably works with all PHP 5.x.


I know it's from the original URI normalization pr, but use of _or_ here seems off to me. Why not use _terms_? Rreturn `{"terms": {"target.source_normalized": [<uri_strings>]}}`  (or when there's only one URI string then just `{"term": {"target.source_normalized": <uri_strings}}`


Hence the first two words in that sentence: "if debugging". :P


ack


зачем принимать блок если ты его не используешь?


Just as I was going to fix this, I realized that it was just recently added support in mariaDB container to run scripts like this..

But I kinda think that not initiating the DB in utf8 is a mariaDB issue, thus a mariaDB specific fix is fine.
Okay, same kind of fix might be needed if using mysql though...


`getAttribute("<XHP attribute")` is it intentional, that there is no closing `>`?


почему просто не вызвать?
в чем смысл тут использовать send ?


Did you mean to change the accepted django-otp versions here? At first glance it seems unrelated to phone number validation, and 0.3.0 added migrations that warranted an [upgrade note](https://pythonhosted.org/django-otp/overview.html#upgrading)


Error handling still TODO


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.516](http://teamcity.monogame.net/viewLog.html?buildId=9761&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:35:25


remove completely if not used anymore


but the "if else" statment is in the for loop, the "else" path may be entered several times. need to add a flag to identify whether the result type was found in the "for" loop, then to decide whether to push_back the result or not out of the "for" loop


And duplication of the Dockerfile is just temporary yes..
The mariadb images on dockerhub has not yet been updated ( do not contain https://github.com/docker-library/mariadb/commit/0196e310a3bd7511ee6090150925a3301c4daaad )


Should this be run through the translations? And is `alert` an override for a nice modal, or are we actually showing basic alerts when we run out of subjects? If we'd have a modal div, we could maybe link directly to Talk in this message too?


Still broken (http://hydra.nixos.org/build/24813270). I think we should revert this patch.


why? you should always use url helpers for emails as it will not work if you just use path


Please follow local style


"K-Hop process" should probably be renamed.


OK, will delete the debug trace.


Can squash in to 1fbb51338395b9403f74f5478066eab4fe88a5f6?


@mattbostock set it to 99% after a discussion with @annashipman in https://github.com/gds-operations/vcloud-edge_gateway/commit/a191fb13d9686e8efb846b7e9283bef514b6d690. I think, if plausible, we should try and stick to that. In that story, Anna previously said:

>  people will aim for making a metric good, instead of the real goal of ensuring the test coverage is good.

I think dropping this to 98% does this, so @surminus and I are going to look at increasing test coverage on this branch. It may be a rabbit-hole; it may not be.


thank you sir!


Great idea. I will merge it to my master branch.


Remplacer `Choisisser` par `Choisir`


Had to go back to versions 4.16.5 and then pull it into 5.0.2 so to go back in time you do what's called cherry picking https://www.kernel.org/pub/software/scm/git/docs/git-cherry-pick.html


Not sure why the website puts the linebreak into the HTML conversion of this. Is it GitHub Flavored Markdown or the line starting with the link?


Should this include requests with hidden comments?


был бы более желателен подход не с метод миссингом а с генерацией методов, как в том же актив рекорде, тк метод миссинг убивает интроспекцию
duck typeинг итп

а вместо возни с "method_name.split('_').drop(2).join('_')" можно было бы просто написать метод find_by который принимает хэшь


Is this using necessary?


Maybe add the equivalent
`$x = isset($A) ? $A : $B;` so it is more obvious.


I doubt most people are even aware of that option, let alone "we all are" using it.


Il y a à la fois du tutoiement et du vouvoiement. Le premier `votre` a été remplacé par `ton`, donc il serait plus logique de remplacer le `Vous participez` par `Tu participes`


нил вернется и так


How does a boolean flag and an `if then else` statement sound?


Maybe link to functions where this is useful (i.e. `usort`).

I don't feel strongly about any of these two though.


Please squash these two commits: the first introduces the line, and the second removes it.


Remplacer `Attender` par `Attendre`


Done.


вместо этого можно было бы использовать блочную структуру
@asdasd || = begin
...
end


So, @annashipman, @dcarley and I discussed this a while back and I think we came to the conclusion that we might not actually care.


Seems to me this fixed the issue #641.


Вторым коммитом удалил файлы проекта


стоит проставить выставление новой строчки в настройках иде


Sur les claviers français, on ne parle pas de bouton "entrer" mais de bouton "Entrée"


Esto no es equivalente a poner `*/5 * * * *`?


Does this cause cyclic dependency issue as utilities wait on appfactory.core etc? Can we fix or revert these changes plz?


Confirmed to be working as expected on light wikis, but I'm not seeing anything that corrects the issue for dark wikis per the original brief.

I'd recommend duplicating and altering the svg files for to use `#fff` (probably locating them in skins/oasis/images/) and some overrides in skins/oasis/css/core/article.scss uitlising `$is-dark-wiki` or, perhaps more appropriately, `$sprite-theme` as found in skins/shared/color.scss


the PRINTFs should also be hidden inside an ifdef __ROUTING_DEBUG__ like with __NEIGHBOR_DISCOVERY_DEBUG__ in 63f01fd225b241666a291a20b709ea2bea6feb9e.


используй собственные исключения
незачем в библиотеке кидать runtimeError


Integration tests are fine, nay, necessary. A lot of the plugin bumps up against the boundaries (VBE, file system, etc.) I wouldn't stress too much over it.


Hi, have you tried upstreaming this fix?


Source of css: https://github.com/xwp/wp-jquery-ui-datepicker-skins


param types are missing, arr


@amrita-jamuura & @gaurav-jamuura

Please note the following:

1. The ref used was incorrect, this commit should reference isue #2 and not the pull request #3
2. The commit message need not use phrases like "i did", "we did", "commited" , etc as these are implicit since the code commit is from your name.


Credit? Lol


IRL, we discussed this, and maybe everyone else is right. Adding additional tests to get this out of the door is arguably excess work. I think I'm happy with 98% coverage, but I think it would be good to include a lot of this conversation - or the outcomes from it - in the commit message.


Yeah, personally, I am indifferent about it being lowered. We could go down to 90 without worrying. Additional tests for the sake of numbers (rather than functionality - which is what Anna was referring to in that comment) aren't necessarily a good thing.


The field is introduced in moonstone, but used in Enyo. How will Enyo code work with Onyx and other?


Oh wait, I already know how to post my codes and explain my issues. :)


Thank you!
Publishing in a bit.


OK


Seems AI uses this to check casting state in scripts and elsewhere:
`Unit::IsNonMeleeSpellCasted`


Is the bug reproduced in Onyx? If not, is it possible to set suppressScroll in Moonstone directly?


OK, fresh *Ubuntu Trusty*. Base system.

    # lsb_release -a
    No LSB modules are available.
    Distributor ID:	Ubuntu
    Description:	Ubuntu 14.04 LTS
    Release:	14.04
    Codename:	trusty

Additionally installed *git* (to clone repository) and *build-essential* (to run make). **Still no good**. Running `make ubuntu` spawns:

    Building dependency tree
    Reading state information... Done
    build-essential is already the newest version.
    python-dev is already the newest version.
    python-dev set to manually installed.
    Some packages could not be installed. This may mean that you have
    requested an impossible situation or if you are using the unstable
    distribution that some required packages have not yet been created
    or been moved out of Incoming.
    The following information may help to resolve the situation:

    The following packages have unmet dependencies:
     python2.7 : Breaks: python-virtualenv (< 1.7.1.2-2~) but 1.7.1.2-1 is to be installed
    E: Error, pkgProblemResolver::Resolve generated breaks, this may be caused by held packages.
    make: *** [ubuntu] Error 100

Any advice?

Also did I get it right, you'll make it sure that all your commits work on Ubuntu only, no portability whatsoever?


It would be cool to inject your own logger which conforms to a certain protocol


зачем???

просто вызови search!

и вообще зачем тут метод миссинг который убивает интроспекцию и дак тайпинг?

создай отдельные методы find и find_by который принимает хэш


Great!


Good question.  That makes perfect sense.


This skips all continuous joints without limit tag (which is not required for them)


Sure - either that or just run one then the other.


так делать не нужно по тем же причинам что и выше


This commit is incomplete :(

It lacks the actual mdss hooks...

Have a look at my shamu or hammerhead kernel.


Fix #5


sdf


Thanks for the fix. I missed that you had to `try... finally` at the beginning with the `@contextlib.contextmanager`. Makes total sense though.


too many `\n`


Llooks like my pre-commit hook to remove unneccessary whitespace had some unintended side effects ;-)


I can tell you didn't test this :smile:, because if you had you wouldn't have punctuated it like this (`perror` adds its own colon).  In fact, what I was really hoping you'd do is use `print_error_errno()`.

A [useful error message](http://www.gnu.org/prep/standards/html_node/Errors.html) here would be:

> samtools: could not write to foo/bar.bam: Errno description

1. When reporting errors due to sys call failures, it's vital to use perror or equivalent (#51)
2. It's also important to say which particular file is failing
3. If several commands are being piped into each other, the user needs to know that it's **samtools** that's complaining, not that there exists an internal function called `readgroupise`
4. In the likely failure scenarios, it doesn't matter to the user whether it was writing the header or writing the read that failed.  At the moment, we have a confusing sentence that looks like it's not sure whether it's reading or writing.

Here endeth the sermon :smile:


Hi Sree:
Maybe this should in range  [(0 - p->luma_log2_weight_denom), (7 - p->luma_log2_weight_denom)] instead [-7,7]


Needs some docs for the new messages here


Can you provide some examples with your regex on http://www.regexr.com/


Basically prove it works there before I start adding tests for it


Yeah it isn't completely obvious at first, but it's in case an error occurs during the yield.


This was an interesting one, as I had animation on the transform of the model. On the first frame, it was valid, but after playblasting it ended up at the last frame and got moved. Ended up publishing a model with non-zero transform.


Yes, exactly. First I assumed it did something 'automagically' so it behaves like the *enter* and *exit* stack of a class-based context-manager. Knowing that it doesn't simplifies the idea on how it works plus keeps it pythonic.


> This was an interesting one, as I had animation on the transform of the model. On the first frame, it was valid, but after playblasting it ended up at the last frame and got moved. Ended up publishing a model with non-zero transform.

Good catch.


Is there a reason you're using `.html()` here instead of `.text()`?


Because I have to update the mapping files all the time https://github.com/github-linker/cache. But I want to get rid of the mapping files and replace it with an API service, because it slows down the initial page load for GitHub by 250ms see https://github.com/github-linker/chrome-extension/issues/38

I'm working hard to fix it very soon.


My comfort with [ADJ_DIST:] is not great, so if folk see anything obviously wrong, say so


Сделал. Хотя теперь другой ворнинг
W: conffile-without-noreplace-flag /etc/freshwrapper.conf


Indeed the code has been adjusted to skip 5.3 (where runkit doesn't compile) and 7.0 (which is not supported by runkit): https://github.com/phpmyadmin/phpmyadmin/blob/master/test/install-runkit


Getting very nitpicky here, but this can be shortened to `.attr('id', 'mw-enhanced-rc-toggle-all')`


```
ERROR:waitress:Exception when serving /logout
Traceback (most recent call last):
  File "/home/ngw_admin/ngw/env/local/lib/python2.7/site-packages/waitress/channel.py", line 337, in service
    task.service()
  File "/home/ngw_admin/ngw/env/local/lib/python2.7/site-packages/waitress/task.py", line 173, in service
    self.execute()
  File "/home/ngw_admin/ngw/env/local/lib/python2.7/site-packages/waitress/task.py", line 392, in execute
    app_iter = self.channel.server.application(env, start_response)
  File "/home/ngw_admin/ngw/env/local/lib/python2.7/site-packages/pyramid/router.py", line 243, in __call__
    return response(request.environ, start_response)
  File "/home/ngw_admin/ngw/env/local/lib/python2.7/site-packages/pyramid/httpexceptions.py", line 287, in __call__
    return Response.__call__(self, environ, start_response)
  File "/home/ngw_admin/ngw/env/local/lib/python2.7/site-packages/webob/response.py", line 1021, in __call__
    start_response(self.status, headerlist)
  File "/home/ngw_admin/ngw/env/local/lib/python2.7/site-packages/waitress/task.py", line 376, in start_response
    'Header value %r is not a string in %r' % (v, (k, v))
AssertionError: Header value u'Basic realm="NextGISWeb"' is not a string in ('WWW-Authenticate', u'Basic realm="NextGISWeb"')
```


die Orginale wären gut. Vielleicht auch die vom Titelbatt.


should I remove this Lazada too?


It'd be nice to have a `cursor:pointer;` here so it's more obvious that it can be clicked.


The issue says: "If StorageService is available, this information should be persisted in a separate Storage, otherwise it should be kept only in memory."
So, there should be an in-memory storage created if the StorageService is not available...


See the 5.3 compile failure: https://travis-ci.org/phpmyadmin/phpmyadmin/jobs/76113776#L294


Looks good. You could squash this into the previous one.


A mistake: I wrote the xenprep_abort function by copying and modifying xenpre_start. Fixed in #2345


Why not use `__DIR__` here as you did below?


the storage must not be set to null!


No, seems to be a  leftover of a ReSharper automatism


I think this is cool!

Make a hybrid of quaternion104 and mandebulb5. Have only the main formula
enabled on both, *no* constants applied. You now have *two concentric
spheres of the same dimension*. A cool place to start. You can then
infinitely hybrid-tweak out to infinity as an animation, then change the
number of formula iterations and start again. LOL

Another potential cool thing i have yet to try is:

Make a hybrid of two standard mandebox103's. On one increase the box fold
start iteration to 1 (or more). Set up two keyframes, with opposite
formula weights, to morph between the two keyframes. Then animate. If it
works then we have just animated between iterations which could not be done
before?  When I was exploring hybrids in v1.18, i had always wished to be
able to animate between hybrids  with different iteration counts.:)

Cheers

On Wed, Aug 19, 2015 at 9:46 PM, carbuncle grim <mclarekin@gmail.com> wrote:

> I am having problems debugging.  Adding an additionConstant to z.w is not
> working, and therefore no other tweaks on z.w will work. BUT it is still
> fun to play with. It does render slower than I had expected, Hmmmm?.
>
> Might be a stupid question, but I am wondering should z4D be in all three
> statements below:
>
> fractal_formulas.hppvoid Quaternion104Iteration(CVector4 &z, const CVector4 &c, int &i, const cFractal *fractal);
>
>
> fractal_formulas.cppvoid Quaternion104Iteration(CVector4 &z, const CVector4 &c, int &i, const cFractal *fractal)
>
>
> compute_fractal.cpp
>
> Quaternion104Iteration(z4D, CVector4(c, 0.0), i, fractal);
>
>
> BTW The very last Quaternion tweak, Z Constant ( z scale factor), works well making hybrids with Mandelbulb formula, because you can change the Quaternions size.
>
> Cheers
>
>
>
>
> On Sun, Aug 16, 2015 at 11:05 AM, carbuncle grim <mclarekin@gmail.com>
> wrote:
>
>> Have recoded draft quaternion with CVector4, and getting this error for
>> everyCVector4 containing parameter variables
>>
>> /home/graeme/git/mandelbulber2/mandelbulber2/src/fractal.cpp:440: error:
>> undefined reference to `CVector4
>> cParameterContainer::Get<CVector4>(QString) const'
>>
>> Do I need to "define" CVector4 in algebra.cpp, I am guessing I need some
>> equivalent to:-
>>
>>
>> double CVector3::itemByName(char item)
>> {
>>     switch(item)
>>     {
>>         case 'x':
>>             return x;
>>             break;
>>         case 'y':
>>             return y;
>>             break;
>>         case 'z':
>>             return z;
>>             break;
>>         default:
>>             return 0.0;
>>             break;
>>     }
>> }
>>
>>
>> But tried that and still got errors, should I commit what I have coded,
>> or wait until I have an answer to get rid of these fatal errors?
>>
>> Cheers
>>
>> G
>>
>>
>>
>>
>>
>>
>> On Sat, Aug 15, 2015 at 11:17 AM, carbuncle grim <mclarekin@gmail.com>
>> wrote:
>>
>>>  Ooops ignore me again
>>>
>>>  CVector4 temp = CVector4( z.x,  z.y, z.x, w);
>>>
>>>
>>>
>>> BTW Quaternion104(3D) and Mandelbox103 make billions of interesting
>>> hybrids, especially with the hybrid formulas Iterations: and Weight:
>>> parameters.  DE can be reasonably good.
>>>
>>>
>>> On Fri, Aug 14, 2015 at 12:15 PM, carbuncle grim <mclarekin@gmail.com>
>>> wrote:
>>>
>>>> How do I rewrite this with CVector4?
>>>>
>>>> void Quaternion104Iteration(CVector3 &z, CVector3 &c, int &i, double
>>>> &w, const cFractal *fractal)
>>>> {
>>>>
>>>>     CVector3 temp = z;
>>>>     CVector3 tempW = z * 0;
>>>>     CVector3 vectW = z * 0;
>>>>     vectW.x = w;
>>>>     vectW.y = 0.0;
>>>>     vectW.z = 0.0;
>>>>
>>>> Cheers
>>>>
>>>>
>>>>
>>>> On Thu, Aug 13, 2015 at 10:10 AM, carbuncle grim <mclarekin@gmail.com>
>>>> wrote:
>>>>
>>>>> Thank you
>>>>>
>>>>> And educational, I learn best by looking at how others do things.
>>>>>
>>>>>
>>>>>
>>>>> On Thu, Aug 13, 2015 at 6:46 AM, Krzysztof Marczak <
>>>>> notifications@github.com> wrote:
>>>>>
>>>>>> CVector4 is a very good idea!
>>>>>>
>>>>>> —
>>>>>> Reply to this email directly or view it on GitHub
>>>>>> <https://github.com/buddhi1980/mandelbulber2/commit/0a61708f78260d45726e6610563cab76a11c594e#commitcomment-12672324>
>>>>>> .
>>>>>>
>>>>>
>>>>>
>>>>
>>>
>>
>


Please disregard the installation part, I've had malformed `sources.list` file and this caused this issue.

I can finally confirm, that your *Makefile* is ok on the *Ubuntu* system.

However the rest of the question remains - *Ubuntu only*?


aaah I see, then yes, a constant with a comment would be nice


cange to $urlRouterProvider.otherwise('/collection/root');


This also needs a further commit squashing into it.


change to
Dropbox


Hmm, actually on second thoughts I've found a different way of approaching my problem - I've personally no need for this change any more.


nice cleanup m8 i r8 8/8

but why did you decide to use double instead of single square bracket?? curious


Sorry, yes, it is that error with a different name.
Once I came to write the code I picked a string that fitted better with the existing conventions.
I shall update the design.


I think [this](https://help.github.com/articles/mentions-on-github-pages/) is what you want.


change to $state.transitionTo('app.collection.content', { coll: 'root'});


That would be more correct Indeed,,but does it fixing some issues for you???


Sure ^^
I didn't find any test on the former, and didn't feel confident enough to write the test myself so there is an exemple:  http://regexr.com/3bk7a

There a case were dotfiles aren't matched, which is the same as with the old one: When the preceding character is'nt a `/` or `\` like in `".dotfile"`


Because is always generated when you compile the document.


add {title: 'Add to Dropbox', cssClass: 'glyphicon glyphicon-floppy-open'}


This needs an additional check to see what state the toggle is in currently. For example, if I open one list of changes and then click expand all, it expands all collapsed changes and collapses all expanded changes. You can hook off the `mw-collapsible-toggle-collapsed` class which is also applied to the toggle :)


Please use more iterations to measure the time consumption.


Done: https://github.com/bitcoin/bips/pull/181


I've filed https://bugzilla.mozilla.org/show_bug.cgi?id=1196199 to get this line fixed.


ah ok


It would be better to not create these short living subtree searches in the first place (instead of doing some (suboptimal) work instead of directly ending the subtree search), but that would require adding pruning condition tests (duplicate code :-/) at the place where we create these new subtree searches. I'll create some tests for that concept later.