You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@edgent.apache.org by dlaboss <gi...@git.apache.org> on 2016/04/05 21:02:58 UTC

[GitHub] incubator-quarks-website pull request: [QUARKS-107] [WIP] new Rang...

GitHub user dlaboss opened a pull request:

    https://github.com/apache/incubator-quarks-website/pull/33

    [QUARKS-107] [WIP] new Range recipes

    In an effort to simplify these recipes and focus on their unique aspects I want them to be able to use a simulated temperature sensor that's defined elsewhere.  Right now the recipes are expecting  ``quarks.samples.utils.sensor. SimulatedTemperatureSensor`` - which doesn't yet exist in the incubator-quarks repo.
    
    @ddebrunner, does that approach / location seem right?  If so, I'll add that class to that repo.  My impl of it was inspired by the code in recipe_value_out_of_range.md.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/dlaboss/incubator-quarks-website quarks107_new_range_recipes

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-quarks-website/pull/33.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #33
    
----
commit 647a2ed5bd732c5b2a312711e109432fdf7c4171
Author: Dale LaBossiere <dl...@us.ibm.com>
Date:   2016-04-05T18:40:02Z

    [QUARKS-107] new Range recipes

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-quarks-website pull request: [QUARKS-107] [WIP] new Rang...

Posted by dlaboss <gi...@git.apache.org>.
Github user dlaboss commented on the pull request:

    https://github.com/apache/incubator-quarks-website/pull/33#issuecomment-205981418
  
    @ddebrunner, this [WIP] was prematurely merged when someone was bulk merging a lot of pending requests.  I'll leave things as is for now rather than back them out and adjust pending your response to the question above.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-quarks-website pull request: [QUARKS-107] [WIP] new Rang...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-quarks-website/pull/33


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---