Simplify stack-to-stack moves

Most of the changes here are simply refactoring.
Resolving stack-to-stack moves first fills in the scratch register for the parallel moves, if one is needed. So we can split that into two phases, inlining the stack_to_stack function which was not used anywhere else. That lets us establish the invariant that we only continue into the second phase if there is at least one stack-to-stack move.
Because compute consumes self, we know that the lazily-allocated stack-to-stack scratch register has not been allocated yet at entry to the loop, but because the loop only runs if there are stack-to-stack moves, we know the scratch register will definitely be allocated. So we can remove the corresponding state from the MoveVecWithScratch struct and keep it in locals with shorter names instead, eagerly initialized before the start of the loop.
The above changes should have no effect on the output. However I then added state tracking for whether the scratch register or the save slot already contain the right value and therefore don't need to be the target of another move. This should remove the need for the redundant-move eliminator to clean up after parallel moves.
My first approach assumed the correct value was in exactly one of these places, but both allocations can be correct, and we can save more copies by acknowledging that.

(Ah, and I just saw the review request for Trevor as this tab was stale, sorry -- feel free to take over if you'd prefer otherwise I'm happy to review followup!)

Yeah, I thought I'd give you a break, but your review is welcome anyway Chris. :laughing:
I felt like the two bools were the most clear way to express their state: 1) do we need to restore the scratch register after trashing it? and 2) do we need to save the scratch register's contents to the save slot? It might be nice to encode that they must not both be dirty at the same time, which would mean we've lost the value that was supposed to be in the scratch register and can't recover. I'm not completely sold that it's worth defining an enum for that, but I could be convinced if you feel strongly.
I do find it a little unsatisfying that the presence or absence of the save_slot only guards one of the blocks. I've convinced myself (and fuzzing confirms) that the other if statements can only be entered if that first block runs, but that's not obvious at a glance.
So I've just pushed a little rework to make it more clear how save_slot and scratch_dirty are related. Does that help?

Other thoughts: It would be nice to not require the caller to specify a "victim" register, partly for convenience and partly because I don't want to think about what to rename that to since I don't like the current terminology. So when find_free_reg fails it would be nice to pick a register from those already used by the moves.
However we can't require that there are any registers used in moves. It could happen that all the current register contents need to be preserved in-place, so there are no free registers but also no register moves. So the caller needs to provide a last-resort register in any case. Sigh.
There is potentially some room for reducing the number of generated moves though. For one thing, if we sort the moves so that moves into registers usually come after moves to the stack (except when dependencies prevent that), then we improve the odds that our selected scratch register will be overwritten after the last stack-to-stack move and we won't have to restore it from the save-slot.
If the selected register is not used as a source between the first stack-to-stack move and being used as a destination, we don't actually need the save-slot at all, because the needed values are all live in other places when we need them. Finding a register in the set of moves that satisfies this property would be a nice optimization, though I don't know how often it would come up.
A way to think about this is one could make the optimal choice in linear time by simulating the stack-to-stack move-resolution algorithm for each choice of register, counting the number of moves that would be inserted, and picking the register with the minimum count. No, I'm not proposing that we actually do that, but it does mean you can fairly easily evaluate ideas for heuristics by simulating the algorithm in your head.

Cool, yeah, I don't feel super-strongly about reworking into a formal state machine; this is clear enough, thanks!
Re: finding a scratch register (and I agree, I didn't think hard enough about the current naming, sorry; let's call it "borrowed scratch register" or somesuch maybe?) -- I agree in principle it'd be nice to not require the caller to make the arbitrary choice; allowing the resolver the freedom to use more complex heuristics as you say. It strikes me that at this point we're several niche conditions into a rabbit hole though, and I might suggest seeing how often the case in question occurs before trying to optimize further. At least the thinking when originally building the current solution was: (i) not requiring a dedicated scratch in the MachineEnv was a potentially significant impact, because register pressure; but (ii) everything that happens when we hit the last-resort logic is a rare case that just needs correctness, not performance. Happy to change my mind with data of course!

"Borrowed scratch register" turns out to fit nicely, done!
To be clear, I wasn't suggesting we optimize this any further now; I just wanted to note down my thoughts from making sure I understood how this works. Data indicating whether this has any impact would absolutely be good to have; I think the starting point is to look for the frequency of trace log messages saying "scratch resolver: stack-to-stack borrowing" (or prior to this PR, "scratch resolver: stack-to-stack using victim"). I would have guessed that running out of free registers is pretty common though; is it that stack-to-stack moves are rare?

That's my intuition, yeah, though to be honest it's at best anecdata; I never did a detailed study. I'm satisfied of course that this PR maintains the status quo so there's no immediate need to collect better data but it would be interesting!


chore: add missing content about renamed plugins; fix links to the new books and remove hardcoded link to 1.2 docs (RHIDP-4293)

What does this PR do?
chore: add missing content about renamed plugins; fix links to the new books and remove hardcoded link to 1.2 docs (RHIDP-4293)
Signed-off-by: Nick Boldt<EMAIL_ADDRESS>Screenshot/screencast of this PR
N/A
What issues does this PR fix or reference?
https://issues.redhat.com/browse/RHIDP-4293
How to test this PR?
N/A
PR Checklist
As the author of this Pull Request I made sure that:

[x] Code produced is complete
[ ] Code builds without errors
[ ] Tests are covering the bugfix
[ ] Relevant user documentation updated
[ ] Relevant contributing documentation updated

Reviewers
Reviewers, please comment how you tested the PR when approving it.

/cherry-pick release-1.3


GridView column sortLinkOptions

I want to put a custom sorting label like this:
'columns' => [
    [
        'attribute' => 'name',
        'label' => 'Player',
        'format' => 'raw',
        'value' => function ($model) {
            return Html::a(Html::encode($model->name), $model->getUrl());
        },
        'enableSorting' => true,
        'sortLinkOptions' => [
            'class' => 'orderby',
            'label' => 'Player ↓',
        ],
    ],
],

But it doesn't work - attribute label overwrites it. Is this an expected behavior or not?
If I swap array_merge values in https://github.com/yiisoft/yii2/blob/master/framework/grid/DataColumn.php#L142 - everithing is ok for me.

you have to set the label of the column, as you did already above, not in sortLinkOptions.


Show transaction history for user

Add a page that displays all the historical actions performed by the current user (or for any user).

Stock transaction history
Part history
etc etc

@Marbles87 This is an interesting idea, it would take some consideration to implement... Maybe the django-simple-history library could help? What do you think @SchrodingersGat, should we take a crack at it?

django-simple-history looks like a good candidate for this feature. We would need to consider what models need history tracked, and how exactly that works for complex data operations.
For example, when a build is completed and subsequently new stock items are created, this is a multi-faceted step that might not be able to be "tracked" with such a simple approach?
I'm not sure. Research needed!

Sounds good, not sure either. I find it very compelling (transparency is dear to me) so I may give it a shot soon and report!

@SchrodingersGat

For example, when a build is completed and subsequently new stock items are created, this is a multi-faceted step that might not be able to be "tracked" with such a simple approach?

In that case, the historical table would need to be enabled for stock items. If each stock item has a link to the build they originated from, there can be tracked against that build.

Related to #537 #377

@SchrodingersGat is this still on the roadmap or wontfix?

I think the scope requirements for this are pretty prohibitive, and it hasn't received much user attention.


Fixed unit test that failed with the incorrect URLError result

Issue Link :link:
Certain unit tests were failing.
Goals :soccer:
Allow failing unit tests to pass
Implementation Details :construction:
On my system, the results from the unit tests were producing a different error status than what was  checked for in the tests. I modified the tests to check for the expected error status.
Testing Details :mag:
Ran the unit test to make sure they now pass.

Closing in favor of #3003.


.getUsdBalances() & .getCollectibles() were both removed...

Context / issue
Why is this feature needed?
It was very useful & our application was relying on it. If it won't come back, how can I get these same results elsewhere?
Proposed solution
Add it back.
Alternatives
Provide me with a guide to use it else where

Those were not efficient endpoints in our service and were removed.
There are better services to retrieve this information like the APIs from Coingecko, Zerion, etc.

Hey German, I took a look at those 2 options you shared, but they don't provide the same functionality that .getUsdBalances() & .getCollectibles() provided.
Do you know of any third-party APIs/libraries that allow you to get all ERC20 tokens & NFTs (ERC721/1155) from a given wallet address?
Or are you able to link me to where you previously provided that API so that I can fork it or study the code on how it was implemented?


[20.03] thunderbird, thunderbird-bin: 78.4.0 -> 78.4.1

Motivation for this change
#103069 for 20.03.
This branch may not be built because 20.03 use Rust 1.54 while master use Rust 1.55.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

It works fine.

This branch may not be built

It does build.

Obsoleted by PR #104481.


Is it possible to use multiple frame blocks/custom portal textures?

I’m looking to make a mod using this api, and was wondering if it would be possible to use multiple different blocks for the frame, and if it was possible to set a different texture as the portal itself.

Right now no unfortuntly. I was also hoping that this could be update to allow for more then one frame block, or accept a blocktag instead


New tab with most visited sites (like Firefox)

Congratulations on your work on Thorium. I just found out about it and I think it looks like a well-thought out project.
I am testing Thorium coming from decades of Firefox and I miss the perfect New Tab layout and functionality.
Please add configuration options so the users can recreate it in Thorium, specifically:

the possibility to choose between specific links and most visited sites (already done)
the number of rows
the number of columns
the size and shape of buttons

If you can implement this to simulate Firefox' Net Tab perfectly, I think many users like me will feel at home in Thorium and that would increase the chance of adoption.

Heyo! Kinda of an aside for this, but there are browser plugins that have much the same functionality as firefox's new ran page. However. Alex does have his own Firefox fork in the same thread of reason (compiler optimization flags, patches from other popular forks and more!) that thorium was created with. Check it out at link

Thanks for the info @CryoRenegade, I didn't know about Alex's Firefox fork, Mercury. I downloaded it and ran JetStream 2 browser test and it scored worse than official Firefox and way worse than Thorium:
115992 - mercury_115.4.0_win64
131866 - Firefox 119.0
181506 - Thorium_AVX2_117.0.5938.157
So maybe a good strategy would be to transfer over to Thorium some good parts of Firefox, like customizable new tab with most visited sites or separate search bar.

@gz83 Thank you, in the new page we now have 8 icons that automatically fill with the most used sites


[Feature Request] RPC Version of tdlib

AFAIK, it's hard to call tdlib in several language (e.g.: Node.js), so I think it's a good idea to create an RPC version. If does so, cross-platform developing would be MUCH easier.
Ref: http://www.jsonrpc.org/specification

TDLib provides a JSON interface. Why it's hard to call it from Node.js?

Our JSON requests/responses format in fact is very similar to jsonrpc.

Thank you for replying. However, what I want is remote procedure call, instead of calling tdlib w/ JSON interface.

Then you can look at https://github.com/vysheng/tdbot, which seems to provide TDLib JSON interface over TCP/Unix socket.

It uses TDLib as subproject, as a part of its source tree. It should be built as any other CMake project:
mkdir build
cd build
cmake ..
cmake --build .

@septs try this https://github.com/nodegin/tglib


Change camera arm to fully autonomous w/ option to change mode

default camera is set to left arm
default control arm is set to right arm
while not actively controlled, left arm is autonomous

ScreenPointToRay

Completed default camera is set to left arm, default control arm set to right arm.
Working on triggering autonomous behavior using input key "X".


Merging of model field docstrings isn't rendering 2nd order directives

Hi, I'm trying to use this package in concert with another one that defines other directives (sphinx-needs).
I have a model field that has help text and its own docstring. This packages merges them as expected, but does not allow the directive in the previously-existing docstring to get rendered as a directive.
input:
    description = models.JSONField(
        blank=True,
        null=True,
        help_text="The detailed description of the requirement. Should be concise, singular, and written in the form of a 'shall'.",
    )
    """
    .. impl:: Store description as JSONField for richtext purposes
        :links: R_REQ_RICH_TEXT

    |
    """

Current result:

Desired result:
Something more like:

@ntouran Thanks for reporting the problem.
I think I fixed it in 6869e76e7c6ac9f44415d8093f42df26081eb42d and released a new version 2.1. Could you check whether this fixes the issue for you?

Hey, very nice, thanks! that absolutely works great for 'regular' directives like .. note:: as you've shown. That works nice and is a great improvement.
There is still an issue with my particular use case, which is related to how some of the directives in sphinx-needs work. Somehow I think the docstring is getting processed twice, which is causing an error in sphinx-needs related to seeing the same directive with the same ID twice.
Sphinx error:
A need with ID I_DOC_IMPL_WOO already exists! This is not allowed. Document api/documents[998]

Maybe the merged docstring that you make is getting left in two places somehow?
Anyway I wouldn't blame you for closing this issue since you did make a good improvement. If there's anything you see that might render the directive in the docstring twice that'd be great to remove.
My input is
    description = models.TextField(
        blank=True, help_text="Description of what this doc type is for"
    )
    """description field.

    .. impl:: Use description for descriptoin.
        :id: I_DOC_IMPL_WOO

        It's a model field.

    """

Ah, interesting, I wasn't aware of such limitations.
My guess would be that the fields is included once in the model's parameter documentation:

and once as individual field:

I'm not sure if there is something I can do on my side, maybe an option to disable the inclusion of docstrings in the parameter documentation of models?
In your case, I think it would be the easiest to make use of the autodoc-skip-member signal by putting this into your conf.py:
from django.db.models.query_utils import DeferredAttribute

def skip_model_field(app, what, name, obj, skip, options):
    return (
        name == "description"
        and isinstance(obj, DeferredAttribute)
        and obj.field.model.__name__ == "YourModelName"
    ) or None

def setup(app):
    app.connect("autodoc-skip-member", skip_model_field)

Interestingly, I just found a small bug in this extension which does prevent this code to work, but I will fix this asap.

I've released version 2.2 now, could you test whether the snippet I provided does the job?


Cannot parse StringFormat={0:N2}

As mentioned in headline i get following error message:
Error in StylerPackage:

Cannot parse markup extension string:
"{Binding Value,
StringFormat=\{0:N2\}}"

This was never a problem until update to version 2.
Regards
Steffen

Even though it works, this is actually not properly formatted XAML. Try this:
StringFormat={}{0:N2}

Maybe you are right. But this is the way Visual Studio Properties-Toolwindow generate StringFormat. So XAML-Style should parse it. As mentioned, it was never a problem until update.

fixed in version 2.1 (not in gallery yet)
I'll keep this issue open until we push the new version out, so you'll get notified

Thx for reply. Glad to here from you.
Regards
Steffen

I can confirm the issue. Escaping the braces is correct XAML in my opinion too. Thanks for the work around!

fixed in 2.1


182601230 table text wrapping

This large PR introduces text wrapping to the table tile. Text wrapping is added to table cells with strings in them, column headers, and the table title. Previously the columns would automatically size based on the length of the column name, but this has been removed.
Major changes:

rowHeight, which is required by react-data-grid, has been changed from a number into a function.
A new function, measureTextLines(), is used by rowHeight and approximates the number of lines a given string of text will take to render in a div of a given width.
Many of the hooks used by the table tile have been broken up and shifted around to ensure that required dependencies are defined before they are needed.
Comments have been added to the table tile to help describe what the many different hooks do.
The table now rerenders and requests its document row to resize more aggressively to keep the size correct.

It's worth noting that this PR does not save the column widths in the MST model, so column widths do not persist between page loads and do not spread to copies of the document. These changes should be addressed in a future story, and I've left comments that I hope will be helpful when that project is tackled.

Current state and requested feedback:

Setting or clearing an expression results in the header size being incorrect until the table is modified.
Cypress tests are failing. I haven't looked into these yet.
Somehow, the double click event on a column header started getting eaten up by the single click event. I solved this by using a setTimeout on the effect of the single click event, but it's a bit wonky.
I couldn't find a way to measure how many lines a given string of text will take in a fixed width element, so I wrote my own algorithm to do it. It works pretty well, but it breaks down around when line breaks occur. I believe this is a result of the width I'm using for the algorithm not quite being what the width is on the rendered page, but I'm not sure and welcome feedback.
Padding and other layout issues still need to be fixed.

I couldn't find a way to measure how many lines a given string of text will take in a fixed width element, so I wrote my own algorithm to do it. It works pretty well, but it breaks down around where line breaks occur, and becomes less accurate as the width gets narrower. I believe this is a result of the width I'm using for the algorithm not quite being what the width is on the rendered page, but I'm not sure and welcome feedback.

As we discussed, I'm interested in the question of whether rendering to an offscreen <div> or <textarea> and then querying it for its size would work. I don't know if you'll have time to undertake such an experiment, but if it works it would be less code to maintain, presumably more accurately reflect the browser's wrapping algorithm, and hopefully would be performant.

@tealefristoe I started looking at this but I got bogged down in the complexity.
Can you explain why it is necessary to measure the rowHeight?
I'm sure there is a reason why this doesn't work but this is what I'd hope we could do:

set the width of the column.
each cell is a div with this column width, and let the browser figure out the height of the div
each row of cells is in a div that lets the browser figure out its height from the max height of each of its children cells.

I don't remember the css layout properties to make that all happen, but I'm sure it is possible. However I'm guessing there is some reason that simplified approach can't be used here.
I just want to understand the need for the complexity. If you don't know the answer, that is OK.

@tealefristoe I've been looking into this more and now I understand this complexity has to do with how RDG works. I'll keep reviewing it.
In the future, it would be helpful for PRs of this size to summarize the changes the PR is making and why.

@scytacki Sorry for not getting back to you sooner about this. I've added a description to the PR which I hope will be helpful for evaluating the proposed changes.
It sounds like you answered your own initial question: a lot of the complexity, and specifically the need for rowHeight, is because we're using react-data-grid. A lot also comes from a desire to make the table efficient and handle large data sets, though. Unfortunately, that has resulted in very complex code. I tried my best to simplify things and leave descriptive comments when I could, but I'm sure this PR is a net increase to the complexity, especially since I was figuring the whole system out as I was trying to make progress.

@scytacki The only cypress test that's failing for me locally at the moment is teacher_network_problem_spec, which I think is a flaky one about comments. So the variable tests are passing locally.

@tealefristoe the shared_variable tests are failing for me locally.  Is your cypress running using Chrome 105 or perhaps Electron instead. GH Action is using Chrome 105.
I don't think these failing tests are specific to this code. They seem to be failing due to a change in the Text Tile or perhaps just a weird interaction of the new version of Chrome, Cypress, and Slate. When the test "types" in "Hello World" things don't work right. Locally the placeholder text is left there and the Hello World is appended after it. On GH actions the placeholder text also stays there, and "ello World" is prepended before it. (yes the 'H' is missing).


Source Request

Please any books site also
To read books

Open separate issues with website links in the source repository.


add OpenVoice

add OpenVoice notebook
ticket CVS-130638

done

View entire conversation on ReviewNB

removed all noniformative outputs

View entire conversation on ReviewNB

done after sys.path.append(repo_dir) all modules successfully imported

View entire conversation on ReviewNB

hmm., interesting. What openvino version do you have? i didn't face that neigher in openvino==2023.3.0, nor in the latest main

View entire conversation on ReviewNB

done

View entire conversation on ReviewNB

Turns our it was accidentally left by mistake. Fixed

View entire conversation on ReviewNB

openvino is still needed to install here. And add -q to all pip install:%pip install -q package

View entire conversation on ReviewNB

done

View entire conversation on ReviewNB

done

View entire conversation on ReviewNB

Done: when scalars are wrapped as torch.tensor it works fine. Removed todo

View entire conversation on ReviewNB

Done. Now no extra utils/py files are needed, notebooks alone is enough

View entire conversation on ReviewNB

done

View entire conversation on ReviewNB

done

View entire conversation on ReviewNB

@pavel-esir please make linter happy https://github.com/openvinotoolkit/openvino_notebooks/actions/runs/7754919412/job/21149238378?pr=1633


Where can I find old document?

Hello,
The "LoRa Server documentation" website is latest, if I want to look the last year's documentation, where can I get them?
Thanks.

All the documentation is on github. You will find it per component repository (doc/) and https://github.com/brocaar/docs.loraserver.io


Non-unique array items with uniqueItems=True does not fail validation in 4.0.1+

Description
An array with non-unique items does not fail validation when uniqueItems is True.
Steps to reproduce
Run the following test:
import jsonschema
import pytest

def test_unique_items():
    """names has non-unique items and is expected to raise a validation error."""
    instance = {"names": ["foo", "bar", "foo"]}
    schema = {
        "properties": {"names": {"type": "array", "uniqueItems": True}},
        "type": "object",
    }

    with pytest.raises(
        jsonschema.exceptions.ValidationError, match="Failed validating 'uniqueItems'"
    ):
        jsonschema.validate(instance=instance, schema=schema)

Expected behavior
The test is expected to pass, i.e. a jsonschema.exceptions.ValidationError is raised:
jsonschema.exceptions.ValidationError: ['foo', 'bar', 'foo'] has non-unique elements

    Failed validating 'uniqueItems' in schema['properties']['names']:
        {'type': 'array', 'uniqueItems': True}

    On instance['names']:
        ['foo', 'bar', 'foo']

Actual behavior
The test fails, i.e. no exception is raised:
>           jsonschema.validate(instance=instance, schema=schema)
E           Failed: DID NOT RAISE <class 'jsonschema.exceptions.ValidationError'>

Version affected: 4.0.1 and up;
nox > Ran multiple sessions:
nox > * tests(jsonschema='3.2.0'): success
nox > * tests(jsonschema='4.0.1'): failed
nox > * tests(jsonschema='4.1.0'): failed
nox > * tests(jsonschema='4.1.1'): failed
nox > * tests(jsonschema='4.1.2'): failed

Hi. Can you include an example that runs the same thing on both versions? Your schema doesn't set a $schema key to declare what version you're trying to use, so you're comparing behavior across two different versions of the JSON Schema specification.

OK I see, setting that myself, this looks like a duplicate of the unfortunate #866.
A PR is definitely welcome, tests are needed upstream as a first step.

Oh! Sorry for the duplicate, I had looked for this issue but that was a few weeks ago and did not verify again before raising this one.


PR / Readme Feedback

Description
What:
Why:
When:
Where:
Tasks

[ ]
[ ]
[ ]
[ ]
[ ]

Acceptance Criteria

[ ]
[ ]
[ ]
[ ]

Constraints

N/A

References:

References #

Need to look at src/ngsa/readme and src/ngsa/test/readme and determine our strategy. Based on the comments of new devs "clicking around", I like having the test details in the test folder and the app details in the app folder with the base folder providing high level info and a "table of contents".
Invest in the readme.md  -- create great dev experience, help others review code
Add things that help people get started:

What is it?
What it does?
Developer roadmap/architecture swimlanes

Should help people easily onboard

Spikes readme:
Add a readme to the spikes folder that explains what a spike is and sets expectations for completeness and quality


New train

Issue #, if available:
This merge brings all changes from training with the new dataset and LoRA adaptation to the present script:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

It is ok to merge


Unknown MNC?

I've ran the update codes file but keeping getting this issue. I have managed to get information in the past however this is all I see now, I am running with the --sniff command.

I just re-installed. Thanks for the code! You're doing gods work!


RuntimeError: Unsupported workspace file reference type self (running fastlane ios beta)

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
I'm getting [!] Unsupported workspace file reference type self (RuntimeError) when running fastlane ios beta.
Thanks in advance,
Complete output when running fastlane, including the stack trace and command used

/Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/xcodeproj-1.5.7/lib/xcodeproj/workspace/file_reference.rb:81:in `absolute_path': [!] Unsupported workspace file reference type `self` (RuntimeError)
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/xcodeproj-1.5.7/lib/xcodeproj/workspace.rb:195:in `block in load_schemes'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/xcodeproj-1.5.7/lib/xcodeproj/workspace.rb:194:in `each'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/xcodeproj-1.5.7/lib/xcodeproj/workspace.rb:194:in `load_schemes'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/xcodeproj-1.5.7/lib/xcodeproj/workspace.rb:85:in `from_s'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/xcodeproj-1.5.7/lib/xcodeproj/workspace.rb:66:in `new_from_xcworkspace'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane_core/lib/fastlane_core/project.rb:106:in `workspace'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane_core/lib/fastlane_core/project.rb:120:in `schemes'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane_core/lib/fastlane_core/project.rb:133:in `select_scheme'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/gym/lib/gym/detect_values.rb:102:in `detect_scheme'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/gym/lib/gym/detect_values.rb:29:in `set_additional_default_values'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/gym/lib/gym/module.rb:15:in `config='
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/gym/lib/gym/manager.rb:8:in `work'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/actions/build_ios_app.rb:64:in `run'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/runner.rb:257:in `block (2 levels) in execute_action'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/actions/actions_helper.rb:50:in `execute_action'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/runner.rb:236:in `block in execute_action'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/runner.rb:231:in `chdir'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/runner.rb:231:in `execute_action'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/runner.rb:157:in `trigger_action_by_name'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/fast_file.rb:148:in `method_missing'
        from Fastfile:30:in `block (2 levels) in parsing_binding'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/lane.rb:33:in `call'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/lane.rb:33:in `call'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/runner.rb:49:in `block in execute'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/runner.rb:45:in `chdir'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/runner.rb:45:in `execute'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/lane_manager.rb:57:in `cruise_lane'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/command_line_handler.rb:36:in `handle'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/commands_generator.rb:108:in `block (2 levels) in run'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/commander-fastlane-4.4.6/lib/commander/command.rb:178:in `call'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/commander-fastlane-4.4.6/lib/commander/command.rb:178:in `call'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/commander-fastlane-4.4.6/lib/commander/command.rb:153:in `run'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/commander-fastlane-4.4.6/lib/commander/runner.rb:476:in `run_active_command'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:75:in `run!'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/commander-fastlane-4.4.6/lib/commander/delegates.rb:15:in `run!'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/commands_generator.rb:333:in `run'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/commands_generator.rb:42:in `start'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/fastlane/lib/fastlane/cli_tools_distributor.rb:107:in `take_off'
        from /Users/ivmos/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.89.0/bin/fastlane:20:in `'
        from /Users/ivmos/.fastlane/bin/bundle/bin/fastlane:22:in `load'
        from /Users/ivmos/.fastlane/bin/bundle/bin/fastlane:22:in `'

Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
10.13.1

Ruby
2.2.4

Bundler?
false

Git
git version 2.14.3 (Apple Git-98)

Installation Source
~/.fastlane/bin/bundle/bin/fastlane

Host
Mac OS X 10.13.1 (17B1003)

Ruby Lib Dir
~/.fastlane/bin/bundle/lib

OpenSSL Version
OpenSSL 1.0.2g  1 Mar 2016

Is contained
false

Is homebrew
true

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
9.2

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE
en_US.UTF-8
✅

fastlane files:
`./fastlane/Fastfile`
fastlane_version "2.9.0"
default_platform :ios

# we will call some node scripts which are written in ES6 (see below)
nodeExec = '../node_modules/.bin/babel-node'

# load in config generated in the pre-build step (see above)
file = File.read('../appConfig.json')
appConfig = JSON.parse(file)

# iOS
platform :ios do
  desc "Submit a new Beta Build to Apple TestFlight"
  lane :beta do
    # fetch previously generated certificates, but don't generate new ones if none already exist
    match(type: "appstore", readonly: true)
    # ensure we're on the "master" git branch
    ensure_git_branch(branch: "master")
    # set the app build number from our previously generated config
    increment_build_number(
      xcodeproj: "./ios/FooMobileApp.xcodeproj",
      build_number: appConfig["buildNumber"]
    )
    # set the app version from our previously generated config
    increment_version_number(
      xcodeproj: "./ios/FooMobileApp.xcodeproj",
      version_number: appConfig["appVersion"]
    )
    # build the app for app store export
    gym(
      clean: true,
      export_method: 'app-store',
      workspace: "./ios/FooMobileApp.xcodeproj/project.xcworkspace",
      scheme: "FooMobileApp",
      output_directory: "./build-tools/deploy/data"
    )
    # upload to TestFlight and notify testers
    testflight(
      skip_submission: true
    )
  end
end

# Android
platform :android do
  desc "Submit a new Beta Build to TestFairy"
  lane :beta do
    # ensure we're on the "master" git branch
    ensure_git_branch(branch: "master")
    # build the app in release mode
    gradle(
      project_dir: "./android",
      task: "assemble",
      flavor: "",
      build_type: "Release",
      properties: {
        'versionCode' => appConfig["buildNumber"],
        'versionName' => appConfig["appVersion"]
      }
    )
   # Upload to TestFairy
   testfairy(
     api_key: 'XXXXXX',
     ipa: './android/app/build/outputs/apk/app-release.apk',
     comment: "Build #{lane_context[SharedValues::BUILD_NUMBER]}",
   )
  end
end

`./fastlane/Appfile`
app_identifier "XXXXX" # The bundle identifier of your app
apple_id "XXXX" # Your Apple email address
team_id "XXXXX"  # Developer Portal Team ID

fastlane gems

Gem
Version
Update-Status

fastlane
2.89.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

slack-notifier
2.3.2

atomos
0.1.2

CFPropertyList
2.3.5

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.4

xcodeproj
1.5.7

rouge
1.11.1

xcpretty
0.2.6

terminal-notifier
1.7.1

unicode-display_width
1.1.3

terminal-table
1.7.3

plist
3.2.0

public_suffix
2.0.5

addressable
2.5.1

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.6.4

tty-cursor
0.5.0

tty-spinner
0.8.0

babosa
1.0.2

colored
1.2

highline
1.7.8

commander-fastlane
4.4.6

excon
0.55.0

faraday
0.12.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.0

gh_inspector
1.1.3

json
1.8.1

mini_magick
4.5.1

multi_json
1.12.1

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.2.0

bundler
1.14.6

faraday_middleware
<IP_ADDRESS>

naturally
2.1.0

simctl
1.6.3

uber
0.0.15

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
2.1.0

mime-types-data
3.2016.0521

mime-types
3.1

little-plugger
1.1.4

logging
2.2.2

jwt
1.5.6

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

google-api-client
0.13.6

libxml-ruby
3.0.0

generated on: 2018-04-04

@ivmos This is super weird... "self" is not a type of option available 🤔 Does your dropdown look different than mine?

I have the same issue after updating.
It's unclear from the error which file it's having an issue with, but the target I'm running on has the files as Relative to Group.

bump

Has there been any progress on this? thanks

Same its happening here.

Worked with @ivmos workaround: https://github.com/fastlane/fastlane/issues/12233#issuecomment-378884704

This was not fixed. It got no attention then was auto-closed, but it's still broken in 2.102.0.

I'm having a similar issue when trying to generate screenshots. I get Unsupported workspace file reference type SDK`` on every attempt. This lane has worked just fine in the past but no longer works with any of the old applications or my new ones.

Gem
Version
Update-Status

fastlane
2.102.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.0.0

slack-notifier
2.3.2

rouge
2.0.7

xcpretty
0.2.8

terminal-notifier
1.8.0

unicode-display_width
1.4.0

terminal-table
1.8.0

plist
3.4.0

CFPropertyList
3.0.0

public_suffix
2.0.5

addressable
2.5.2

multipart-post
2.0.0

word_wrap
1.0.0

tty-spinner
0.8.0

babosa
1.0.2

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

excon
0.62.0

faraday
0.15.2

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20180417

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.3

gh_inspector
1.1.3

mini_magick
4.5.1

multi_json
1.13.1

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
1.0.0

bundler
1.16.2

faraday_middleware
0.12.2

naturally
2.2.0

simctl
1.6.5

emoji_regex
0.1.1

tty-cursor
0.6.0

tty-screen
0.6.5

json
2.1.0

io-console
0.4.5

libxml-ruby
2.9.0

atomos
0.1.3

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.6

xcodeproj
1.6.0

Hi all, as detailed here, this is an issue with the dependency Gem Xcodeproj being unable to parse embedded xcode workspaces. I submitted a fix here, fingers crossed that causes these issues to go away as well once it's accepted and Gem version is updated https://github.com/CocoaPods/Xcodeproj/pull/606


android [nfc]: Convert NotificationsModule and NotificationsPackage to Kotlin

Thanks for the review! Revision pushed, and I tested on the office Android device (received a notification which, when tapped, brought me to the conversation).

Thanks! Looks good; merging.


Support Boundary Checking for Loop Dependent Iterators

Currently TVM's boundary check avoids some invalid global memory access, it ignores the case when the arguments in reduce_axis requires global memory accessing (to an index tensor, this is common when dealing with sparse tensor/ragged tensors).
Below is a simple example (segment sum) to reproduce the problem, what it did is basically is:

given a data tensor x and a offset(indicates the segment information) tensor offsets (starts with 0 and ends with the length of x).
For each segment index i, compute the sum of elements inside segment in x: sum(x[offsets[i]:offsets[i+1]]), and store the results in out[i].

import tvm
import tvm.te as te

num_elements = te.var('num_elements', dtype='int32')
num_segments = te.var('num_segments', dtype='int32')
x = te.placeholder((num_elements,), dtype='float32', name='x')
offsets = te.placeholder((num_segments + 1), dtype='int32', name='offsets')

def segment_sum(i):
    """Compute sum(x[offsets[i]:offsets[i+1]])"""
    k = te.reduce_axis((0, offsets[i + 1] - offsets[i]))
    return te.sum(x[k + offsets[i]], axis=k)

out = te.compute(
    (num_segments,),
    segment_sum,
    name='out'
)

s = te.create_schedule(out.op)
segment_axis = out.op.axis[0]
segment_outer, segment_inner = s[out.op].split(segment_axis, factor=4)
s[out.op].bind(segment_inner, te.thread_axis('threadIdx.x'))
s[out.op].bind(segment_outer, te.thread_axis('blockIdx.x'))

print(tvm.lower(s, [x, offsets, out]))

Below is the generated code
primfn(x_1: handle, offsets_1: handle, out_1: handle) -> ()
  attr = {"global_symbol": "main", "tir.noalias": True}
  buffers = {out: Buffer(out_2: Pointer(float32), float32, [num_segments: int32], [stride: int32], type="auto"),
             x: Buffer(x_2: Pointer(float32), float32, [num_elements: int32], [stride_1: int32], type="auto"),
             offsets: Buffer(offsets_2: Pointer(int32), int32, [(num_segments + 1)], [])}
  buffer_map = {x_1: x, offsets_1: offsets, out_1: out} {
  attr [IterVar(blockIdx.x: int32, (nullptr), "ThreadIndex", "blockIdx.x")] "thread_extent" = floordiv((num_segments + 3), 4);
  attr [IterVar(threadIdx.x: int32, (nullptr), "ThreadIndex", "threadIdx.x")] "thread_extent" = 4;
  if (blockIdx.x < floordiv(num_segments, 4)) {
    out_2[(((blockIdx.x*4) + threadIdx.x)*stride)] = 0f32
    for (rv: int32, 0, ((int32*)offsets_2[(((blockIdx.x*4) + threadIdx.x) + 1)] - (int32*)offsets_2[((blockIdx.x*4) + threadIdx.x)])) {
      if (((blockIdx.x*4) + threadIdx.x) < num_segments) {
        out_2[(((blockIdx.x*4) + threadIdx.x)*stride)] = ((float32*)out_2[(((blockIdx.x*4) + threadIdx.x)*stride)] + (float32*)x_2[((rv + (int32*)offsets_2[((blockIdx.x*4) + threadIdx.x)])*stride_1)])
      }
    }
  } else {
    if (((blockIdx.x*4) + threadIdx.x) < num_segments) {
      out_2[(((blockIdx.x*4) + threadIdx.x)*stride)] = 0f32
    }
    for (rv_1: int32, 0, ((int32*)offsets_2[(((blockIdx.x*4) + threadIdx.x) + 1)] - (int32*)offsets_2[((blockIdx.x*4) + threadIdx.x)])) {
      if (((blockIdx.x*4) + threadIdx.x) < num_segments) {
        out_2[(((blockIdx.x*4) + threadIdx.x)*stride)] = ((float32*)out_2[(((blockIdx.x*4) + threadIdx.x)*stride)] + (float32*)x_2[((rv_1 + (int32*)offsets_2[((blockIdx.x*4) + threadIdx.x)])*stride_1)])
      }
    }
  }
}

Note that in for (rv_1: int32, 0, ((int32*)offsets_2[(((blockIdx.x*4) + threadIdx.x) + 1)] - (int32*)offsets_2[((blockIdx.x*4) + threadIdx.x)])) {, the memory access to offsets_2 is not protected thus incurring invalid memory access error when ((blockIdx.x*4) + threadIdx.x) is greater then num_elements.
If we change the order of the if-statement and the for-loop, the program should work correctly:
if (((blockIdx.x*4) + threadIdx.x) < num_segments) {
  for (rv_1: int32, 0, ((int32*)offsets_2[(((blockIdx.x*4) + threadIdx.x) + 1)] - (int32*)offsets_2[((blockIdx.x*4) + threadIdx.x)])) {
    out_2[(((blockIdx.x*4) + threadIdx.x)*stride)] = ((float32*)out_2[(((blockIdx.x*4) + threadIdx.x)*stride)] + (float32*)x_2[((rv_1 + (int32*)offsets_2[((blockIdx.x*4) + threadIdx.x)])*stride_1)])
  }
}

The bug was also mentioned in TVM forum.
I think this error is related to https://github.com/apache/incubator-tvm/blob/f13fed55cfe872ba7f40970f6a35f965d186a30a/src/tir/transforms/bound_checker.cc, I wonder how could I change it to be aware of global memory access in reduce_axis?
cc @junrushao1994

Just a kind reminder that #5130 has been there for half a year :(

I see, looking into the case. Both this example and #5130 seems to appear when the iteration is dependent on an outer loop. And we will need to find a detector to insert the condition properly when generating the loops.
The current compute bound insertion assumes the loop bounds to not being iterator dependent. A contribution would be more than welcomed.
It might also be useful to think about what can be done and (cannot be done) in iterator dependent loops. As the original assumption of compute means the axis should be independent from each other for certain scheduling correctness to hold. In this case, a dependent reduction bound would certainly bring different implications, and possible different analysis for the scheduling to hold

Also cc @spectrometerHBH @Hzfengsy since this might bring some fruit for thoughts for the TIR schedule design

The block isolation in TVM's Tensor IR would solve the problem.


Nearable zone is occationally zero even when in range

Problem/Question
The Nearable zone property is sometimes 0 when ranging or monitoring a trigger.
This has been observed when the nearable is very close and zone is 1 (ESTNearableZoneImmediate). Happens quite frequently during ranging and monitoring.
Is this a bug or something the application should compensate for (e.g. by ignoring zone zero values or sampling/filtering values over time)?
What does ESTNearableZoneUnknown mean really? The documentation does not say much.
When the nearable goes out of range it is reported with zone zero for a period, it would be convenient to be able to rely on this.

@divineprog, ESTNearableZoneUnknown is expected value and it happens when beacon was not found in particular time window. Currently stickers are advertising with time interval 2,5 sec, and sometimes bluetooth packet can be lost. So taking that into count it is possible to get Unknown state from time to time. I will try verify if there is no bug there anyway. Thanks for feedback.

Thanks a lot for the update and the clarification! It makes sense that you get unknown zone/proximity now and then, apps will have to take that into account, and it would also be possible to wrap this in a library if one would want to filter out unknown values in some way. Thanks again :)

This issue should be closed out then :)

Good call! Bumping Tx power & reducing the adv interval of stickers should help BTW, same as beacons and CLProximityUnkown.


Fix runtime not found for triton due to wrong default protocolVersion

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #3176
Type of changes
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[ ] Test A

[ ] Test B

Logs

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[x] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

Fix runtime not found for triton due to wrong default protocolVersion if protcolVersion is not specified in the isvc resource

/lgtm
/approve


the graph reasoning module issue

Thanks for your sharing.Great work! But when I read the codes according to the papers, there exists some issues that I can not understand, especially the graph reasoning module.  In the henG.py file , I  can not find the impletation of the formula (6) in your paper.
My first question:
The e_obj variable means the x_middle in your paper,right? But it is not the combination of Y_o and X_m with softmax.Perhaps it should be like this:
e_obj = self.fc_o_(torch.cat([s_obj, o_a_view], -1))    #49th line
My second question:
In formula (6), Y_v is the combination of X_middle and Y_o. But in your impletation,  it seems like the
Y_v is the combination of X_middle and answer_view. So in my opinion,it should be like this:
A_obj = F.softmax(self.w_g_o(F.relu(self.w_s_o(o_a_view) + self.w_s_o_(e_obj))), dim=-2)
Am I right? Look forward to your valuable apply! Thanks a lot!

Yes, I have same confusion and I have a little trouble to align the code with paper equation.

@runzeer Have you understood this reasoning module? I try my best to read it, but I cannot align these codes with the Eq.(3) ~ Eq.(9).


This commit improves the trend plots according to Fri discussion

All these parts are improved and updated according to the discussion:
[X] 1. real life meanings
[X] 2. center the title
[X] 3. mentioning simulation logic and links
[X] 4. x axis rotation
[X] 5. simulate with trend, start with 2010
[X] 6. add a median line

Hi Wenyu, I think it looks great! I'm not sure how difficult it would be to change, but I think it would be clearer if the y-value label was the name of the value (eg. Average Price) instead of 'metric value'.

Hi Wenyu, I think it looks great! I'm not sure how difficult it would be to change, but I think it would be clearer if the y-value label was the name of the value (eg. Average Price) instead of 'metric value'.

Not difficult at all, just revised and committed.


collect method for cones

as discussed at https://discourse.julialang.org/t/ann-clarabel-jl-new-convex-optimization-solver/83915
To filter out the b[i]==Inf cases, I currently need to build the vector cones in a loop, so it has as many elements as length(b). Then, I can index into it with cones[.!isinf.(b)].
It would nice if I could create this vector by collect([NonnegativeConeT(2)]) or similarly to allow the indexing.

all linear inequalities

My typical use case has also linear equalities, which I have handled with Clarabel.ZeroConeT(1). I guess I could reorder the restrictions to get the equalities first and then apply your idea. (Replacing an equality with two inequalities is not a good idea, or?)

We have implemented a feature that will automatically eliminate constraints with very large bounds, which I think means that that the issue described here is no longer relevant.     This is now part of release v0.4.1.
I will close, but feel free to reopen if I have misunderstood.


vkQuake engine built without MP3 support?

Game
Quake
Engine
vkquake
Linux Distribution
Arch Linux
Luxtorpeda Version
v63
Bug description
Hi,
This isn't so much a bug report as it is a question. I was troubleshooting why vkQuake is unable to play MP3 files, which a lot of map packs that use external music make use of, and noticed that in the build script for the package the following flag is passed to the make command: USE_CODEC_MP3=0. That would explain the lack of MP3 support in vkQuake, but this seems to have been a deliberate decision, as Quakespasm-spiked isn't built with this flag and the build instructions in the vkQuake readme mention nothing of this.
Steps To Reproduce

Install the vkquake package
Play any map pack that uses MP3 files for playing external music (Alkaline in my case)
Notice the 'Couldn't find cdrip for track[xx]' when loading into a map that uses an MP3 file as BGM or 'unhandled extension' errors when trying to manually play said track.

Expected Behavior
vkQuake can handle MP3 files
Additional Context
No response
Relevant log output
No response

Thanks for the report!
Not directly related to this, but it seems like vkquake is no longer in active development: https://github.com/Novum/vkQuake/discussions/691
I'm not sure why the mp3 codec is off, that has been there since the beginning, and may have been a compatibility measure for an older runtime we were using. I've updated it, and added the libmad dep, so vkquake should work with mp3s now.
Out of curiosity, are you able to play the way you want in ironwail? I'm not going to remove vkquake, but I'll probably add a notice that it's no longer in active development.


GH-43150: [Docs] Correct documentation of pyarrow.compute.microsecond

What changes are included in this PR?
Corrected a small mistake in the docstring

GitHub Issue: #43150

Thanks for the fix @Aloqeely !


feat: Paged vod output instead of dumping every vod onto one page

Will become increasingly more necessary as vods accrue

https://github.com/hugh-braico/slowtrainroll.in/pull/42


Use of relative attention

Hey all!
While looking through the VPT code I noticed the use of "relative attention logits" in the Self-Attention layers, as seen here:
https://github.com/openai/Video-Pre-Training/blob/main/lib/xf.py#L342
The hypothesis on my end regarding these now was that this R-stream is used as a learnable, data-dependent bias for attention, as also seen in the attention function and the b_nd matrix.
I was also wondering about the use of  nbasis = 10 as the per-head dimensionality for it, and thought of it as a form of bottleneck. But I am not sure how different values for nbasis would affect the network.
I would really appreciate any further insights, corrections and references to other resources regarding this.

Hey. Good catches! I unfortunately do not have insights on why they exactly went with this approach. Your best bet is to try to send email to the paper corresponding author(s), or try asking this question on some other forums where someone might know why you would do this


the redis and hive port type issues

We I start worker I got this imformation.
ERROR/MainProcess] consumer: Cannot connect to redis://ctu-cco-int.ubisoft.org:6379/12: Int or String expected.
I fould it is bcs socket.getaddrinfo request the int type of port fields int python2.7/site-packages/redis/connection.py
I found it is also have convert in init function, however it look like the <class 'future.types.newint.newint'> type could not been convert to int
logging.info(type(port))  # <class 'future.types.newint.newint'>
self.port = int(port)
logging.info(type(self.port))  # still <class 'future.types.newint.newint'>
Another question is also about the port of hive, after I config the hive2 connect,  I test it via the Hot Query. it is not work.
it is bcs the port in socket.getaddrinfo function need a int type, but long type give. So I have to manual  convert it to int in python2.7/site-packages/thrift/transport/TSocket.py
I checked the define in model.py of airflow. the port is defined int. it is very strange

Looks strange, maybe try:
pip install redis --upgrade

I have tried. It look like some package conflict or missing

Sorry, we cannot help with that. Try Googling your Redis issue, ask the Redis community, or defer to RabbitMQ...

I have found the reason. It is bcs the Python 2.7 version issue.  I reinstall python to 2.7.10. it work.


[occm] Lookup ports by SG and not tags

What this PR does / why we need it:
When creating an SG for an LB we tag the ports we attach the SG to in order to be able to find them easily later on when the SG will be getting deleted. We do that to simulate an API allowing us to filter ports by attached SGs, but turns out - such an API already exists in Neutron and Gophercloud v1.6.0 allows to use it.
This commit makes sure that when deleting an SG we're discovering ports to remove it from by querying the Neutron filtering by SG. Moreover tagging the ports with the SGs is removed to as not being necessary anymore.
I've left the code removing the SG tags from ports for backward compatibility.
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:

Release note:

[openstack-cloud-controller-manager] Ports of the nodes will not anymore be tagged with the IDs of SGs OCCM creates for the load balancers.

/approve
/hold
in case @mdbooth you still want to take another look or feel free to unhold

Alright, I'm assuming I can unhold this.
/hold cancel

/cherry-pick release-1.28


add onLoad event for app

It would be nice to add smth like onLoad event for app when it's just being loaded to page (when FRINT_INIT action is dispatched, not when beforeMount is called)
we have now a problem with subscribing for message broker.

if we do happen to add a lifecycle method like that, I think initialize would be a good one. we have that in Service, Factory and Model too.

initialize method supported in App classes. Can be seen in next branch docs, which will land in v1.0: https://github.com/Travix-International/frint/tree/afa394b64a7b94a20ff692beb431c455d85e711d/src/core#arguments


In Grids - Grid overflow fixed when items moved from backward.(Wrapper changed from window to custom container as an optional prop)

Is this related to it? Could provide more text in description and less in title.
https://github.com/clauderic/react-sortable-hoc/issues/577


[Runtime Bug]: Game crashes when reset settings is clicked

Describe the bug
I'm getting a different crash with the game when I try to reset settings in the RTX Remix Overlay. Should I make this a separate issue?
Game crashes when reset settings is clicked. Produces error message that says

Microsoft Visual C++ Runtime Library
Assertion failed!
Program: C:\Users\SDSF\EphineaPSO.trex\d3d9.dll
File: ../src/d3d9/d3d9_swapchain.cpp
Line: 448
Expression: !"invalid vsync enable state"
For information on how your program can cause an assertion
failure, see the Visual C++ documentation on asserts
(Press Retry to debug the application - JIT must be enabled)
Abort   Retry   Ignore
How do you reproduce the bug?
How do you reproduce the bug?
Click the reset settings button on the overlay on the character select screen.
Occurs with both DX9 and DX8 with  d3d8tod3d9.dll
What is the expected behavior?
What is the expected behavior
Settings are reset.
Version
dxvk-remix-fb5dcc9-650-release bridge-remix-3e91886-107-release
Logs
Logs
PsoBB_d3d9.log
d3d9_01.log
NvRemixBridge.log
Crash dumps
Crash dumps
NvRemixBridge.exe_20240929_082710.dmp
Media
No response

REMIX-3649 for tracking


Does not support method each_row_streaming

This is a method that base Roo::Base supports, but this gem does not.

how fix it?

+1

Unfortunately it will not be possible to suppourt this for XLS. XLS is a binary format and we rely on the spreadsheet gem to interpret it. Nokogiri, which we rely on for roo allows us to stream XLSX content as XLSX are XML files.


Bad training results with multiple subjects

I used 4 different subjects, with unique names, and high quality/cured 30 photos each using the recommended settings, however the results with multiple samplers, cgf scale and steps, do not resemble each other at all and do not preserve the facial features and body dimensions corresponding to the input.
It is still better to train individually and with regularization images with the classic method.
I hope we can find a solution.
Keep up the good work, this technology is evolving very fast!

how many total steps ?

1500, fp16 disabled.

do 1500x4, that's 6000k steps

and keep the fp16 checked

you can retrain (continue training) a previously trained model, use the same images as before

you can retrain (continue training) a previously trained model, use the same images as before

But can you do this after losing the collab files, using only the gdrive ones?
I'm lost trying to find out how to resume from the intermediate checkpoints I have saved in the google drive.
Is is possible? Or you should have the collab always open? If you close it, you lose the resume possibility?
Thanks and congratulations for the incredible work @TheLastBen

if you have a final model in the sessions folder, you can simply load the session using its name and it will load the previous model without having to run the "model download" cell.
if  you want to use an intermediate as the initial model, simply copy its path using the colab explorer and paste it in the "model download", don't forget to check the box "resume training.

thanks. Understood.
Em dom., 30 de out. de 2022 às 10:38, Ben @.***>
escreveu:

if you have a final model in the sessions folder, you can simply load the
session using its name and it will load the previous model without having
to run the "model download" cell.
if you want to use an intermediate as the initial model, simply copy its
path using the colab explorer and paste it in the "model download", don't
forget to check the box "resume training.
—
Reply to this email directly, view it on GitHub
https://github.com/TheLastBen/fast-stable-diffusion/issues/251#issuecomment-1296261267,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABUXTGSPYE4X5YPKUHMJ4JLWFZ25BANCNFSM6AAAAAARPBELT4
.
You are receiving this because you commented.Message ID:
@.***>


Support Reference equality.

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1834
Description
Support equals operation on Reference object.
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: Bug fix
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct.
[x] I have read the Contributing page.
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[x] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

https://github.com/google/android-fhir/assets/86107848/a3948242-c450-4e3a-a7d6-a7c0f98fc3d8


Next months arrow disabled even having enabled date in next month in Inline mode.

HI,
Please look at stackblitz link for more clearance
https://stackblitz.com/edit/mydatepicker-inline-disbale-issue
If after disabling the whole calendar using disableUntil and disableSince then
providing enabledDates array. makes next month selector disable even some dates are enabled for next month.

Hi @acharyaks90
I will check that but quick fix is set disableHeaderButtons option to false. Then the header arrows are not disabled in any case.

I fixed this issue. You can try here:
stackblitz.com - angular-mydatepicker
After that install version 0.9.1 from npm.


Overflow Error on Mobile Clients

The "PorkyBlackHolesThatWillDestroyTheEntireUniverseSoYouProbablyShouldntUseThem.com.com.inc" text overflows onto the right margin on mobile devices. This throws off the look of the entire website on mobile clients.
Info

Tested on Firefox iOS (webkit) and chrome Android (chromium)
Not tested on Firefox Gecko

Started working on fix via the pull request in the "hiteacheryouare-patch-1" branch

ISSUE HAS BEEN FIXED
This issue has now been closed. Thank you for your support and continued help with open-source projects like this. 😎


⚠ Bofrost.es has degraded performance

In 600503b, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1699 ms

Resolved: Bofrost.es performance has improved in 625e45b after 6 minutes.


helper.py method "click" has unresolved reference

https://github.com/disinfoRG/FbScraper/blob/master/fbscraper/helper.py#L164
"selector" is not passed in as an argument in click() .

resolve by this PR #38


Tech deep dive on vector search

@vamshin - Please update the blog issue, as it is missing key info. I converted it to this repository, but it appears it may have been opened elsewhere?

@vamshin - Is it fair to assume that this is the same blog and it was published on the AWS side of the house here: https://aws.amazon.com/blogs/big-data/amazon-opensearch-services-vector-database-capabilities-explained/.

@pajuric thats right. Can we create like a parter blog?

Closing this blog. Please reopen if you decided to move this forward at some point.

Closing this out


fix: providers stuck in initializing state

Currently if a specific extension is not avaliable/installed providers get stuck in initializing state
Expected both initializing and initialized flags on providers to be false under those conditions

Thanks for the PR the issue got fixed in commit 4a25b78.


Updates of ocean diagnostics and recipes for GMD-2019-291 (ESMValTool Part II paper)

Before you start, read CONTRIBUTING.md and the guide for diagnostic developers.
Please discuss your idea with the development team before getting started, to avoid disappointment later. The way to do this is to open a new issue on GitHub. If you are planning to modify an existing functionality, please discuss it with the original author(s) by tagging them in the issue.

Tasks

[x] Create an issue to discuss what you are going to do, if you haven't done so already (and add the link at the bottom)
[x] Give this pull request a descriptive title that can be used as a one line summary in a changelog
[ ] Make sure your code is composed of functions of no more than 50 lines and uses meaningful names for variables
[ ] Circle/CI tests pass. Status can be seen below your pull request. If the tests are failing, click the link to find out why.
[ ] Preferably Codacy code quality checks pass, however a few remaining hard to solve Codacy issues are still acceptable. Status can be seen below your pull request. If there is an error, click the link to find out why. If you suspect Codacy may be wrong, please ask by commenting.
[ ] Please use yamllint to check that your YAML files do not contain mistakes
[ ] Update documentation for the recipe to the doc/sphinx/source/recipes folder
[ ] Update provenance information if needed
[ ] Assign the author(s) of the affected recipe(s) as reviewer(s)

If you need help with any of the tasks above, please do not hesitate to ask by commenting in the issue or pull request.

Closes #1620

@ledm Is this ready for review or are you still working on it?

Looks like I've got my branches crossed! I've been working on this in two different branches:
This one (development_ocean_recipes_gmd_2019_291) and
version2_development_ocean_development_for_v2_paper_1166. Unfortunately, most of the work is in the other branch, not this one!

This PR is now ready for review. Most of my plots listed here in GMD-2019-291 were already included.
This PR just adds some graphical improvements, a couple new time series diagnostics, and a new recipe.

Hi Lee,
Could you please clean up a bit by removing commented out lines from recipes and code? It would also be great if you could have a look at the failing tests on CircleCI. Let me know if you need help!

I've been testing recipe_ocean_amoc.yml from this recipe and it looks like something has changed with the extract_named_regions preprocessor. Line 570 of check.py:
cmor_points = [float(val) for val in coord_info.requested]

This line attempts to convert the requested dimensions into floats. This doesn't work with the strings required for this preprocessor and it fails.

@jvegasbsc have a look at @ledm comment above please mate, it also seems a bit off to me at first glance too - there could be coords that have strings as points (like typesi) :beer:

I will take a look. It should not be too dificult

In addition to the issue above, I'm also getting a weird error from the concatenate preprocessor when running the recipe_ocean_example.py:
  File "/home/users/ldemora/workspace/ESMValTool/run/esmvalcore/preprocessor/__init__.py", line 224, in _run_preproc_functi
    return function(items, **kwargs)
  File "/home/users/ldemora/workspace/ESMValTool/run/esmvalcore/preprocessor/_io.py", line 193, in concatenate
    _fix_aux_factories(result)
UnboundLocalError: local variable 'result' referenced before assignment

The result object isn't created in io.concatenate() when there's only one cube in the list.

that's been fixed yesterday (and merged) - can you please merge master in here? :beer:

that's been fixed yesterday (and merged) - can you please merge master in here? 🍺

Cheers - solved it.

I will take a look. It should not be too dificult

Done!. Can you test if ESMValGroup/ESMValCore#657 solves your issue?

@valeriupredoi can you please test this?
It would be nice to have it merged for the final release.

shall do! but I'd be happy if the conflicts get fixed first - they are scientific stuff and would feel uncomfortable fixing them meself :beer:

@ledm I might be able to spend some time in the coming weeks  to revamp this stale branch if you think it is still valuable.
I spotted a previous branch in #1331 that looks to me like a precursor of the present one,  as I saw similar changes e.g. in diagnostic_timeseries.py... is this correct or there are things to rescue also from #1331 ?


Fix extraction of authentication exception in sample.

We now use the WebAttributes.AUTHENTICATION_EXCEPTION key as a replacement for the (deprecated) and in the meantime removed constants from AbstractAuthenticationProcessingFilter.

Choose wrong target branch - closing this PR and creating a new one,...


Correct Sonatype Taxonomy URL

Correcting URL for Sonatype Taxonomy as the target page has been renamed.
Signed-off-by: msymons<EMAIL_ADDRESS>

DCO fail.  Aghhhh!


Break out bringing up GRPC server from consuming a service

For flexible development and testing until this is fully finalized closer to release.

cc @degrammer & @mxsdev: not a significant change just needed the ability to control things individually for targeted testing.


feat(monorepo): use forge install to install monorepo

#221  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jakim929 and the rest of your teammates on  Graphite


[Distributed] IRGen: Fix loading of the witness tables

The witness table is an array of pointers - void **,
which means that the list of the witnesses has type of void ***.
Instead of using result of emitAddressAtOffset accessor
should set the correct type for the list, which is void ***
and emit bit a lot for each of the witness table slots.
Resolves: rdar://87568630
Resolves: rdar://88340451

@swift-ci please test

@swift-ci please test Windows platform

Build failed
Swift Test Linux Platform
Git Sha - 6cbbfd5519f085ff5304bff8034f8ae020c272fd

Forgot to update IRGen tests :/

Build failed
Swift Test OS X Platform
Git Sha - 6cbbfd5519f085ff5304bff8034f8ae020c272fd

@swift-ci please test


isort 5.13.2

Destination channel: defaults
Links

PKG-4934
Upstream repository
Upstream changelog/diff

Explanation of changes:

Set version and sha256
Remove noarch
Small test cleanup
Linter fixes

Notes

Would love to add pytest if there is a straightforward way to do it, but am unsure of which tests to run. I tried adding and running the tests folder with no luck.

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_d7vwaqj5kg/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:49: documentation_specifies_language: Use the generic link, not a language specific one
clone/recipe/meta.yaml:51: documentation_overspecified: Using doc_url and doc_source_url is overspecified

===== ERRORS =====

clone/recipe/meta.yaml:51: http_url: http://isort.readthedocs.io/en/latest/ is not https
===== Final Report: =====
1 Error and 2 Warnings were found

Generally we want to run the unit tests. I took a quick look through the tests to track down which dev dependencies are imported. We don't have pylama so we'll need to skip that test file. The snippet below should get you most of the way there, though my local build had a few failing tests you'd need to investigate.
test:
  source_files:
    - tests/unit
  imports:
    - isort
  commands:
    - pip check
    - pytest tests/unit --ignore="tests/unit/test_pylama_isort.py"
  requires:
    - pip
    - pytest
    - py >=1.11.0
    - hypothesis >=6.10.1
    - colorama >=0.4.6


Highlightjs could do a better job for patches

Please look at this patch in Phabricator: https://reviews.freebsd.org/differential/diff/11400/?revisionID=4603
It is .mk file, therefore all changed by patch areas are in 'make' syntax. Additionally, make's targets are in the 'shell' syntax.
Phabricator knows all this. And highlightjs only shows the added lines in green.

Hightlight.js preserve custom markup, so this can be done by add-on that will wrap any changed line into span.

Yeah, this is way out of scope for a highlighter. The way it could work is by using highlight.js inside some dedicated diff viewer to highlight syntax but we're not going to become a diff viewer itself.


docs: extension readme

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 68.41%. Comparing base (3cdb9b5) to head (102e48f).
Report is 2 commits behind head on main.

@@             Coverage Diff             @@
##             main     #394       +/-   ##
===========================================
+ Coverage   54.34%   68.41%   +14.06%
===========================================
  Files          49       70       +21
  Lines        5025    11838     +6813
  Branches     5025    11838     +6813
===========================================
+ Hits         2731     8099     +5368
- Misses       2245     3482     +1237
- Partials       49      257      +208

see 21 files with indirect coverage changes

You mean adding examples to the readme?

You mean adding examples to the readme?

Looking at the existing code in the testnet runtime, there are a lot of noises aside from the direct configuration of the module.
Something like the code snippet below could be useful for someone that interested to use this in their code.
struct ExampleFunction;

...

pub(crate) type Functions = (
	ExampleFunction<WithFuncId<ExampleFuncId>, Runtime>,
);

#[derive(Default)]
pub struct Config;
impl super::Config for Config {
	type Functions = Functions;

	const LOG_TARGET: &'static str = "pop-chain-extension";
}


i5 tk complains about wrong Newtonsoft.Json library, though latest is installed

Describe the bug
The toolkit complains that it is running in standard mode, as the requires Json lib is not installed, although it is installed (it complained about an earlier version 3 being installed via Unity Package Manager, but still keeps complaining when I updated it to the latest nuget package, v 13.0.3).
To Reproduce
log any xAPI behaviour.
Expected behavior
no warning
Screenshots
[i5.Toolkit.Core.ExperienceAPI.Statement] Running in standard mode since the project does not contain the required Newtonsoft JSON library for advanced features.
In standard mode you can only send straightforward Actor-Verb-Object statements.
To use advanced features, install the Newtonsoft JSON library from https://github.com/jilleJr/Newtonsoft.Json-for-Unity.

Bug Observed on the Following Build Targets
Check applicable build targets by putting an x in-between the square brackets.

[X ] Unity editor
[ ] PC, Mac, Standalone
[X] Universal Windows Platform (UWP)
[X] iOS
[ ] Other (please specify):

Obviously in MirageXR

This occurred since the module still relied on an old open-source mirror for the Newtonsoft library but now, Unity has its own package version of it.
I have updated the soft dependency and now, the advanced version relies on the newer Unity package.
The module will switch to the advanced mode if the package com.unity.nuget.newtonsoft-json is installed via Unity's package manager.


Remove InkToolbar sample reference, add API reference

Description

Removes link to InkToolbar related control because that page doesn't exist
Adds link to InkToolbar API doc

Motivation and Context

Fixes #296
How Has This Been Tested?

Built and run locally
Screenshots (if appropriate):

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

That is one very cute canvas!😃


[5.5] Turn off async test on ios and watchos

This is likely related to the workaround in PR #37939.
See also: rdar://79670222.

@swift-ci Please test


Certificate Refactor: Certificate Response, Renewals, Async Support

This is a PR for the conversation started in issue #11. Unit tests are in progress pending review from @ericchiang.
Details:

NewCertificate now returns CertificateResponse struct
NewCertificate now supports asynchronous certificates (Not yet in Boulder, but based on spec)
If NewCertificate call results in a synchronous certificate, CertificateResponse.Certificate is populated with *x509.Certificate. URI and StableURI are both set. The issuer url is set based on the rel="up" Link header. (see below for more details
If NewCertificate call results in an empty response body, CertificateResponse is populated with RetryAfter and URI.
RenewCertificate method
RevokeCertificate method

Methods on CertificateResponse:

CertificateResponse contains three methods: Retry, RetryPoll, and Bundle
Retry will handle implementation details to retry the call to CertificateResponse.URI. User triggers it based on number of seconds passed in CertificateResponse.RetryAfter. Behavior here is the same as NewCertificate depending on if the certificate is available or requires another try.
RetryPoll will poll retries maxRetries times
Bundle allows for bundling the certificate with the cross-signed Issuer. This is a convenience method and puts the burden on the user to call, but the implementation details are covered in the library.

@ericchiang Any thoughts?

@cristiangraz sorry have been busy. Will look this over today :)

Overall looks really good!
General comment. Would it be possible for you to clean up your git commits? You've added stuff like this.
Since this is really only one change, it seems reasonable to roll it into a single commit.

Thanks for reviewing! I'll make the changes, add unit tests, and put this all into a single commit to clean things up. I needed to get something into production and this branch ended up being a gradual progression. Let me know if you have an easier solution to clean things up, otherwise I'll create a new PR to supersede this one.

This one is superseded by #14