CSS import error

I can't get rid of this warning
Failed to inline stylesheet: Could not find asset bwu_datagrid|example/src/composite_editor_item_details/asset/examples.css. null (more details)
example/composite_editor_item_details.html:405:5
<link rel="stylesheet" href="src/composite_editor_item_details/asset/examples.css">

It is this import
https://github.com/bwu-dart/bwu_datagrid/blob/master/example/src/composite_editor_item_details/composite_editor_view.html#L10
This is the entry page
https://github.com/bwu-dart/bwu_datagrid/blob/master/example/composite_editor_item_details.html
the app-element is https://github.com/bwu-dart/bwu_datagrid/blob/master/example/src/composite_editor_item_details/app_element.html
I tried adding a few ../ but I wasn't able to find a way to point it to the correct directory.
The same import works in all other examples. The difference here is that the import is in an element composite-editor-view which is imported by app-element. In the other examples the examples.css is only used in the app-element.

I think this is possibly because of the asset folder in the path?

What do you mean with "asset folder in the path"?

Nvm, that wasn't the issue :). I have narrowed this down the the import inliner, its definitely just a bug.

https://codereview.chromium.org/993423004/ is out to fix this, thanks for the report! This looks like a pretty nasty one.

Thanks a lot for the fast fix!

this is published now, web_components 0.10.5+3


Request for some minimal examples that show performance benefit

After having done some experiments to see when these lookup tables may be beneficial for performance, I haven't really found much. What I did find is that if the compiler can inline a function and / or use conditional move instead of conditional jump, chances are it'll be much faster than using a lookup table: https://godbolt.org/z/ndo6jqbWr
I'd like to request some minimal examples where these lookup tables are measurably faster than code without them, if possible.

Sorry I overlooked this issue. Will add proper example


Update count query options docs

Related to this PR https://github.com/OData/odata.net/pull/1658
We have added an IncludeTotalCount(bool countQuery)method.
We have also added IncludeCount() and IncludeCount(bool countQuery) methods and deprecated the IncludeTotalCount since TotalCount is a V3 term.
For GET https://host/service/EntitySet?$count=true:
var people = context.People.IncludeCount();

var people = context.People.IncludeCount(true);

For GET https://host/service/EntitySet?$count=false:
var people = context.People.IncludeCount(false);

Docs Build status updates of commit 86091de:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Odata-docs/includes/appliesto-odatalib-v6.md
:warning:Warning
View
Details

Odata-docs/webapi/add-nextpage-link-and-count-for-collection-property.md
:warning:Warning
View
Details

Odata-docs/includes/appliesto-webapi-v7.md
:warning:Warning
View
Details

Odata-docs/includes/appliesto-webapi-v6.md
:warning:Warning
View
Details

Odata-docs/webapi/batch.md
:warning:Warning
View
Details

Odata-docs/webapi/convention-model-builder.md
:warning:Warning
View
Details

Odata-docs/webapi/custom-url-parsing.md
:warning:Warning
View
Details

Odata-docs/webapi/filter-in-expand.md
:warning:Warning
View
Details

Odata-docs/webapi/function-parameter-support.md
:warning:Warning
View
Details

Odata-docs/webapi/model-builder-abstract.md
:warning:Warning
View
Details

Odata-docs/webapi/model-builder-nonconvention.md
:warning:Warning
View
Details

Odata-docs/webapi/operation-parameters-in-untyped-scenarios.md
:warning:Warning
View
Details

Odata-docs/webapi/path-segment-change.md
:warning:Warning
View
Details

Odata-docs/webapi/skipToken-for-server-side-paging.md
:warning:Warning
View
Details

Odata-docs/webapi/stream-entity.md
:warning:Warning
View
Details

Odata-docs/includes/appliesto-odatalib-v7.md
:warning:Warning
View
Details

Odata-docs/includes/appliesto-odataclient-v6.md
:warning:Warning
View
Details

Odata-docs/includes/appliesto-odataclient-v7.md
:warning:Warning
View
Details

Odata-docs/spatial/define-property.md
:warning:Warning
View
Details

Odata-docs/webapi/attribute-routing.md
:warning:Warning
View
Details

Odata-docs/webapi/built-in-routing-conventions.md
:warning:Warning
View
Details

Odata-docs/webapi/capabilities-vocabulary-support.md
:warning:Warning
View
Details

Odata-docs/webapi/customize-formatter.md
:warning:Warning
View
Details

Odata-docs/webapi/date-and-timeofday-support.md
:warning:Warning
View
Details

Odata-docs/webapi/getting-started.md
:warning:Warning
View
Details

Odata-docs/webapi/httpRequest-message-extension-method.md
:warning:Warning
View
Details

Odata-docs/webapi/navigation-property-binding-with-complex-type.md
:warning:Warning
View
Details

Odata-docs/webapi/routing-abstract.md
:warning:Warning
View
Details

Odata-docs/webapi/Scaffolding.md
:warning:Warning
View
Details

Odata-docs/webapi/work-around-for-singleresult-create.md
:warning:Warning
View
Details

Odata-docs/includes/appliesto-webapi-core-v7.md
:warning:Warning
View
Details

Odata-docs/resources/support.md
:warning:Warning
View
Details

Odata-docs/webapi/action-parameter-support.md
:warning:Warning
View
Details

Odata-docs/webapi/basic-auth.md
:warning:Warning
View
Details

Odata-docs/webapi/concurrencymode-and-etag.md
:warning:Warning
View
Details

Odata-docs/webapi/continue-on-error.md
:warning:Warning
View
Details

Odata-docs/webapi/DependencyInjection.md
:warning:Warning
View
Details

Odata-docs/webapi/enable-no-dollar-queryoptions.md
:warning:Warning
View
Details

Odata-docs/webapi/max-expansion-depth-in-enable-query-attribute.md
:warning:Warning
View
Details

Odata-docs/webapi/merge-complex-and-entity-read-writer.md
:warning:Warning
View
Details

Odata-docs/webapi/open-type-in-untyped-scenarios.md
:warning:Warning
View
Details

Odata-docs/webapi/Set-namespaces-for-operations.md
:warning:Warning
View
Details

Odata-docs/webapi/simplified-uri-convention.md
:warning:Warning
View
Details

Odata-docs/resources/roadmap.md
:warning:Warning
View
Details

Odata-docs/webapi/abstract-entity-types.md
:warning:Warning
View
Details

Odata-docs/webapi/autoexpand-attribute.md
:warning:Warning
View
Details

Odata-docs/webapi/caseInsensitiveAndQualifiedFunctions.md
:warning:Warning
View
Details

Odata-docs/webapi/ComplexTypeWithNavigationProperty.md
:warning:Warning
View
Details

Odata-docs/webapi/debugging.md
:warning:Warning
View
Details

Odata-docs/webapi/DeltaFeed_support.md
:warning:Warning
View
Details

This comment lists only the first 50 files in the pull request.

Odata-docs/includes/appliesto-odatalib-v6.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/add-nextpage-link-and-count-for-collection-property.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/includes/appliesto-webapi-v7.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/includes/appliesto-webapi-v6.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/batch.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/convention-model-builder.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/custom-url-parsing.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/filter-in-expand.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/function-parameter-support.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/model-builder-abstract.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/model-builder-nonconvention.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/operation-parameters-in-untyped-scenarios.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/path-segment-change.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/skipToken-for-server-side-paging.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/stream-entity.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/includes/appliesto-odatalib-v7.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/includes/appliesto-odataclient-v6.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/includes/appliesto-odataclient-v7.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/spatial/define-property.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/attribute-routing.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/built-in-routing-conventions.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/capabilities-vocabulary-support.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/customize-formatter.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/date-and-timeofday-support.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/getting-started.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/httpRequest-message-extension-method.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/navigation-property-binding-with-complex-type.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/routing-abstract.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/Scaffolding.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/work-around-for-singleresult-create.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/includes/appliesto-webapi-core-v7.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/resources/support.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/action-parameter-support.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/basic-auth.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/concurrencymode-and-etag.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/continue-on-error.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/DependencyInjection.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/enable-no-dollar-queryoptions.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/max-expansion-depth-in-enable-query-attribute.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/merge-complex-and-entity-read-writer.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/open-type-in-untyped-scenarios.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/Set-namespaces-for-operations.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/simplified-uri-convention.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/resources/roadmap.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/abstract-entity-types.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/autoexpand-attribute.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/caseInsensitiveAndQualifiedFunctions.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/ComplexTypeWithNavigationProperty.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/debugging.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

Odata-docs/webapi/DeltaFeed_support.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 78439cc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Odata-docs/client/v6/query-options.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Show dev/release version on frontend

https://stackoverflow.com/questions/58384179/syntaxerror-cannot-use-import-statement-outside-a-module


Added functionality to define header bindings

Added functionality to define header bindings to Add-RabbitMQQueueBinding.ps1

Thanks again! Definitely hit me up if you would be interested in access to this repo!


[WIP/QRCoder2] How do we handle changes during the 2.0 development?

Note: This issue is part of the planning of version 2 of the QRCoder. The meta-issue can be found here. These comments have been copied into this issue here, with the comments marked as such. If comments on the topic of this issue already exist in the meta thread, they have been copied here, naming the authors.
Topic
Do we do a code freeze of the current 1.X version or do we allow the further development of the old version in parallel (with the disadvantage that we have to constantly update these developments again in 2.0)?

I think the answer depends on the quantity of breaking changes (and ease of developers to upgrade).  Due to the popularity of the QRCoder project (27 million downloads and counting), I personally would attempt to release 2.0 with only removal of System.Drawing dependencies, as 1.4.3 and newer do not contain these missing classes anyway.  Essentially, I would attempt to have version 2.0 be 100% backwards compatible for the net6.0 tfm.  If that was the case, I think there is no reason to continue development on 1.x.
On the other hand, if a complete rewrite of the exposed API is desired, then I would maintain bugfixes for 1.x if/when requested by a contributor.  It is quite easy to set up GitHub to allow for this.  At GraphQL.NET we can easily release bugfixes for old versions if needed, although we very rarely receive a request to do so; typically users need help upgrading if anything.  Having excellent migration documentation really helps in these cases.
Originally posted by @Shane32 here.

On the other hand, if a complete rewrite of the exposed API is desired, then I would maintain bugfixes for 1.x if/when requested by a contributor. It is quite easy to set up GitHub to allow for this. At GraphQL.NET we can easily release bugfixes for old versions if needed, although we very rarely receive a request to do so; typically users need help upgrading if anything. Having excellent migration documentation really helps in these cases.

My vote would be to clean up the public API and better organize the structure of the main project. Maybe have separate namespaces for the core logic, for payloads and for renderers at least (for better discoverability).
At present there are at least a few ways to create a QR Code. There are QrCodeGenerator.CreateQrCode instance methods (1) which delegate to the QrCodeGenerator.GenerateQrCode static method (2), and then there are also the GetQrCode static helper methods (3). Maybe just pick one preferred way moving forward?
That said, I had also been experimenting with a more fluent-like extension method approach:
  public static TRenderer RenderAs<TRenderer>(this QRCodeData data) where TRenderer : AbstractQRCode, new()
  {
      TRenderer renderer = new TRenderer();
      renderer.SetQRCodeData(data);
      return renderer;
  }

Usage:
string graphic = QRCodeGenerator.GenerateQrCode(textBoxQRCode.Text, eccLevel)
    .RenderAs<AsciiQRCode>()
    .GetGraphic(20);

There would be some factoring out of IDisposable needed to make it work. ~Not sure why the MemoryStream in PngByteQRCode needs to be a class variable. Seems as though the MemoryStream lifetime could be managed in the GetGraphics methods (plumbing the Stream through the helper methods).~ I also wasn't sure of the value of Dispose functionality in QRCodeData? Maybe @codebude could clarify if it could be removed.
Edit: Disregard the bit about the MemoryStream implementation in PngByteQRCode. I was looking at the PngBuilder IDisposable implementation by mistake. All *QrCode implementations implement IDisposable by way of AbstractQrCode's Dispose method, which simply disposes the QRCodeData.
Here is QRCodeData.Dispose (not actually releasing unmanaged resources, just setting to default values):
public void Dispose()
{
    this.ModuleMatrix = null;
    this.Version = 0;

}

Other random thoughts:

Rename GetGraphics to Render ?
Maybe leverage some abstraction like RenderOptions or RenderSettings to clean up some of the parameter lists that have gotten a bit unwieldy.
Maybe add an abstract method or two to AbstractQRCode to enforce a minimal contract for all subclasses. E.g. Render(RenderOptions options)
Rename various other classes and methods for clarity and consistency

Originally posted by @csturm83 here.


Add basic blog (v.1)

address #9
Blog with basic flexbox layout
Sidebar with 4 recent posts and 4 recent releases

Blog index only shows recent releases
Post detail shows both

@jglovier
Few tweaks over mockups to keep design consistent. Link styles in sidebar for example.
Wanted to see if you think green really doesn’t work on a live site. White didn’t feel quite right. Compare below.
Blog Detail White:

Blog Detail Green:

Blog Index:

@jglovier should I open an issue for this todo list?

Tweaks
Add responsive design
Add Tags
Add category pages
Add author pages
Add syntax hightlighting

@jglovier maybe I should shorten the measure to increase legibility? What do you think?

maybe I should shorten the measure to increase legibility? What do you think?

Sorry, I'm confused as to what you are referring to? Measure of what? Line height? Line width?

Few tweaks over mockups to keep design consistent. Link styles in sidebar for example.
Wanted to see if you think green really doesn’t work on a live site. White didn’t feel quite right. Compare below.

I think the green links are fine. I liked @danielgolden's mock with the lighter blue links too, but the green does help it feel consistent with the default link style on the rest of the site.

Hi @jglovier !
I was referring to the length of a line of text. Thought the measure of it could be shorter for paragraphs…
Increasing the font-size a tiny bit might do a fine job. Didn’t want to mess with your defaults right away. :)

Exactly my thoughts about the green links. Better for consistency.

@jglovier
Thanks for the code review. I learned something new. Getting on it asap!

@jglovier Alrighty. Looks ready now!

@jglovier
Cool, I got it! ❤

@jglovier
Sure, no biggie! :champagne:
It was a pleasure to work with you!
Love the Halloween profile pic!
What about integrating a build tool for minification, gzipping, prefixing and stuff?
Should I give that a go before continuing to the next tasks?

@jglovier Just realized that your gh-pages branch wasn’t rebuilt after the merge. As a result, the blog is 404-ing…

@vis-kid actually the build succeeded, it's just that there were some issues with this PR that I didn't spot because I failed to check it out locally and actually test it.
Also, because this is an organization page, it uses the master branch for jekyll, not a gh-pages branch (more on that here).
I've got a branch going to fix things up and I'll open a PR and cc you for reference.
I'm also seeing that the styling doesn't appear to match your screenshots above. I'm trying to figure out why, and the only thing I can think is perhaps your screenshots above are not actually screenshots but design mocks from the Sketch file? Did you run it locally and take screenshots, or just screenshots from the Sketch file?

What about integrating a build tool for minification, gzipping, prefixing and stuff?

Lets hold off on that for now.

@jglovier the screenshots were taken in the browser—def not a design mock.

The mismatch is weird. Was it broken, slightly different…? Didn’t want to cause you extra work with this 😞

Hmmm...not sure what happened then. Many of the styles were very different, and I had to rework the structure to get the blog to render.

Didn’t want to cause you extra work with this 😞

No worries! It wouldn't have gotten this far at all if you had not jumped in to tackle it. :heart:
I merged in some fixes in #41.

@jglovier That is nice to say of you but after seeing your refactorings I realized what I left on the table 😞 . Have some work to do. Will learn from this! Think I got a good idea what I do better for the next PR.
Btw, what would be most convenient to focus on next. The team page?
Love the new loop for recent releases. So much simpler! :metal:

Btw, what would be most convenient to focus on next. The team page?

Well making the blog responsive would be a great next step. Right now it still looks bad on mobile.

On it!

@jglovier for future contributions here, is this code guide by mdo what I should follow?

@jglovier for future contributions here, is this code guide by mdo what I should follow?

@vis-kid that's a great starting point, yes. I'm not sure we follow all of those patterns (like declaration order), but if you adhere pretty closely to that guide you'll def be in good shape. 👍

@jglovier
Cool!
What’s your preferred solution for navigation on smaller devices?

What’s your preferred solution for navigation on smaller devices?

@vis-kid I think a hamburger menu (similar to what I'm using on http://joelglovier.com) would be appropriate. You can even repurpose the code that I'm using on my site  via https://github.com/jglovier/jglovier.github.io

@jglovier
Done! Just need to do some refining.
Is it cool if I hide the footer for small devices?

Is it cool if I hide the footer for small devices?

Yeah, that's fine for now. I think the footer needs some work too, but we can focus on that later and just get the site responsive for now.

my thoughts exactly :)
PR is already open.

@vis-kid @jglovier I just noticed the footer missing on small devices which I think is ok on pages that have the menu available, but the home page doesn't have the menu available, so there is no way to navigate to the the blog etc from the home page on mobile devices.

@jkleinsc alrighty, on it!

@jglovier @jkleinsc
here is a quick fix that keeps the footer on the landing page. #45
Will think of something better when I have a free minute. :)

FYI can try to we keep discussions scoped to the title of the thread? For new issues, we should open a new issue. That just makes it easier to find and reference later. :heart:


Strange messages from sensor

After dcef35236774244fd0d2ee001ce75940d166b50b commit when building go application there appear a lot of messages:

Is the only way to remove these messages is to upgrade SDL2 to 2.0.9 version?

The default package on Ubuntu for libsdl2-dev has not made it to 2.0.9. As a result, if you attempt to use go-sdl2 on Ubuntu, you are greeted with lots of #pragma messages.
Is there a way to just disable and make unavailable the things that are not supported, such that if you try to use them, you get an error then, instead of just for attempting to use anything in the library?

Hi @3auris and @FriskyDev, I've disabled the warnings by default with this commit e081255f. You can see the commit message if you want to know how to re-enable it.
If people think errors is a good idea, I'll probably think about how to add that option too! For now though, I think warnings are enough.

Awesome, thanks! I'll take a look as soon as I can get back to that environment. 👍

For me, it's a great solution, thank you very much @veeableful

This works for me. Thanks for doing this!

Yay! I'll close this for now but let me know if there's any issue!


Add Decorator for Computing Multiplicative Inverse over Quadratic Extension Field

[x] Take quadratic field element as input, compute multiplicative inverse, put result on advice tape
[ ] Ensure functional correctness

This PR partially addresses requirement described in https://github.com/0xPolygonMiden/miden-vm/issues/591 . To be more specific it solves the first point of https://github.com/0xPolygonMiden/miden-vm/issues/591#issuecomment-1341887052.

I think the only thing left is integrating this into the std::math::ext2 module and adding some tests for this procedure.

I'll be sending another PR, on top of this PR, where I'll address remaining portion.

In PR #597 , I try to address second requirement of https://github.com/0xPolygonMiden/miden-vm/issues/591#issuecomment-1341887052

All looks good! Thank you!

Thanks for the review.


Random Pack Option

Feature request for option to select 'Random' as your pack choice in draft.  Each pack distributed would be randomly selected from all sets.

that's chaos draft
dupe of #1


Versioning issue?

Hi,
Apologies - I prefer not to use Github issues for help requests but I can't find a relevant forum or community.  I'm attempting to set this up and don't appear to be able to run setup.py.  The error it responds with is:
Traceback (most recent call last):
  File "setup.py", line 16, in <module>
    container_template = Template(container_template).render(config)
  File "/usr/local/lib/python2.7/site-packages/jinja2/environment.py", line 964, in render
    vars = dict(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/config.py", line 474, in __getitem__
    return self.evaluate(rv)
  File "/usr/local/lib/python2.7/site-packages/config.py", line 407, in evaluate
    item = item.evaluate(self)
  File "/usr/local/lib/python2.7/site-packages/config.py", line 1025, in evaluate
    rv = lhs - rhs
TypeError: unsupported operand type(s) for -: 'int' and 'str'

This seems like it'd be a package version issue of some kind but Im relatively inexperienced with python and would appreciate any assistance.  Thanks.

Issue was in fact an error in the configuration file. Closed.


How to use getReactElement() when property's name  in props contains "-"

such as 'data-dash-is-loading':

Hi @CNFeffery,
Apologies for taking so long to respond to this. You should be able to just use quotes around the property name, e.g. "data-is-loading"
I tested that out and it works, e.g.:


Fancy Zones layouts by resolution

Description of the new feature / enhancement
Fancy Zones do not currently allow automatically changing the layout selected on a monitor when its resolution changes to a different size. That would be particularly useful when switching a monitor from a horizontal position to a vertical position.
In the current behaviour, the layout that is set when horizontal will just be stretched to fit the vertical orientation but that results most of the time in not-very-useful zones. If the same monitor with 1920x1080 would be considered different from the same monitor with 1080x1920, then we could set a new layout and that would then be automatically changed between the two.
Scenario when this would be used?
The most common scenario would be when a monitor is switched from horizontal to vertical. (but it can be in any situation in which a new resolution is set)
Supporting information

Resolution dependency was removed/changed some time ago because of issues. But this is another perspective.

Could you please bring it back somehow?
You can also maybe do a new checkbox in the FancyZones settings and allow for the behaviour to be turned on and off. By leaving it off by default you could mitigate whatever issues it had.

+1 on this. I was on 0.57 for a while, recently updated to 0.60 and immediately noticed that my different layouts set for landscape vs portrait mode were not working.
Stuck on 0.57.2 now.

i, too, just realized recently that this "feature" disappeared, so I will be downgrading to 0.57.2 as well (thanks @fishstik ) until this feature is brought back.. Everything was working fine for me back in the good ol' days of 0.57.2... heh... in my case, i often switch between resolutions because i have an extra wide monitor, which is normally at 3440x1440 for my day-to-day work, but sometimes, when i'm presenting in a Teams meeting, i "dumb down my resolution" to 2560x1080 so that when other team members are looking at my screen, everything isn't so tiny for them.. i dont mind doing it, but when i do, the zones i have configured for 3440x1440 don't work anymore, so i created a different layout for "Teams meetings", which always used to automatically get chosen whenever i switch resolutions (for which i use a neat little tool called HRC.exe, which is GREAT). anyway, i'll be much happier back in 0.57.2 when everything i use fancyzones for just "worked"! 🤷♂

when i'm presenting in a Teams meeting, i "dumb down my resolution" to 2560x1080 so that when other team members are looking at my screen, everything isn't so tiny for them

Why noy change the scaling (or dpi) in stead? Just wondering...

when everything i use fancyzones for just "worked"!

This might apply for you, but not for many others. There's a reason things change(d)!

when i'm presenting in a Teams meeting, i "dumb down my resolution" to 2560x1080 so that when other team members are looking at my screen, everything isn't so tiny for them

Why noy change the scaling (or dpi) in stead? Just wondering...

good question. i hadn't tried that, but changing the resolution via hotkey (thanks to HRC) works pretty well for me. i can look into the scaling idea, i just never had a good experience with scaling.. things looked funky, but maybe it's gotten better over the years. i'll try scaling to 125% next time i have a meeting, and see how that works. 🤷♂ is there a kb hotkey for that? 😛

when everything i use fancyzones for just "worked"!

@eseinbinder This might apply for you, but not for many others. There's a reason things change(d)!

oh, i absolutely love when things are updated and change (for the better), but recently, when an update came thru and broke the layout editor, that was a bummer. i was actually one of the ones who helped them find and fix the issue. but then when i noticed this issue with the current layout reverting back to a built-in one when i changed resolution (or upon reboot, as someone mentioned, which i hadn't tried yet), i figured i'd just revert to the last known version that "worked" until i hear they fixed it, at which time, i'll give it another try. that's all.. i dont think there's anything wrong with downgrading until a fix is available, now that i've identified the issue and submitted my 2 cents. heh.

when everything i use fancyzones for just "worked"!

@eseinbinder This might apply for you, but not for many others. There's a reason things change(d)!

That why I'm suggesting the compromise of allowing both behaviors to happen.
As mentioned you could bring it back as a toggle. With the toggle OFF layouts would associate to displays based on their ID as it is post PR, with the toggle ON they would use the identifier as ID+resolution as it was before the PR.
Power toys is a tool for power users in the first place, the more options the better

I see. I think, in the end, the most sophisticated thing would be manual input, where the user can define px or %.

Any update on this?

+1, on this issue, it not only applies to rotating monitors, but also for PBP (switch monitor from full screen to PBP mode, which tend to dramatically affect aspect ratio)

Is this ever going to be looked at?


Some search query examples are broken in the docs

The documentation has examples of building different queries throughout. Some work and some do not.
On the responses page: https://mapzen.com/documentation/search/response/
This query
https://search.mapzen.com/v1/search?api_key=search-XXXXXXX&text=YMCA&size=25
results in 'search-XXXXXXX' is not a valid key.
Some other queries do work, like this one:
https://search.mapzen.com/v1/reverse?api_key=search-XXXXXXX&point.lat=48.858268&point.lon=2.294471
Perhaps it's a caching thing, where the cached locations work? Either way, let's fix.

Yeah, if someone else with a valid API key has run exactly the query listed with the XXXXXX api key, it will work. It also requires that the query was run from the same general area, or enough time has elapsed for Fastly to propogate the cached response to all its points of presence. So it's something that's quite hard to prevent from happening.
The solution would be something like a script that runs all the queries from the documentation every time we clear the Fastly cache.

What if we made an API key specifically for documentation? It could probably have a low rate limit, since all the queries in the docs would (likely) be cached.

Yes, I think this a new, real, limited the way to go (other than having an actual key with xxxxx!). There are some other discussions about future options for keys, but in the meantime, I will chat with ops folks on proper rates for search and turn-by-turn.


Close the modal window on click outside

Would be nice to have the possibility  to close the modal window on click outside

You can always bind the overlay click event:
document.getElementById('modal').addEventListener('click', function() {
    modal.close();
}, false);

To take this one step further and prevent children elements from being triggered also use the following:
document.getElementById('modal').addEventListener('click', function(e) {
    console.log(e);
    if( e.target !== e.currentTarget ) {
        e.stopPropagation();
        return;
    }
    modal.close();
  }, false);


Design - Do detail charts need to be full-screen width to understand the trend?

From @johnrobbjr and Kemal's great feedback here.
Do detail charts need to be full-screen width to understand the trend?

@johnrobbjr I agree!  I don't think they particularly need to be full screen.  Any suggestions of what would we would do to improve this?  Were you thinking of collapsing to reduce the vertical scrolling?
Also, for more context these additional detail views are less-frequently used just by nature of them requiring a drill-in step, so less of a priority for now too as you noted in the slides.

I'm closing this now since there's nothing actionable - please comment if you have other thoughts here and I'll happily re-open :)


Bug: Wrong abbreviation used

The correct abbreviation for
Gigawatt is GW not gW
Megawatt is MW not mW which is Milliwatt
Kind regards,
Stefan

You are absolutely right, thanks for the hint! I will fix this when I find time :)

No hurry :-)
Thanks.


Int and VarInt lose sign information when reading -0

Int and VarInt represent the value of the signed integer being read in as an i64, which cannot represent -0 on its own. Int and VarInt are already structs, so we can add a sign field to these and check for it when necessary.

This functionality was added to VarInt in #312.


[BUG] files_ignore used with files not ignoring as expected

Is there an existing issue for this?

[X] I have searched the existing issues

Does this issue exist in the latest version?

[X] I'm using the latest release

Describe the bug?
Maybe I'm misunderstanding the capabilities of these two levers, but I used files_ignore and files in conjunction, but the outputs of this action is undesirable for my use case. By using files_ignore, I want to exclude files listed in that section from being considered as "changes", but that's not what's happening.
The closest I could find was this issue: https://github.com/tj-actions/changed-files/issues/675 but it looks like that was user error - hoping that's the same here.
To Reproduce

Set up a folder structure as so:

.
├── products/
│   ├── gri/
│   │   └── {package code}
│   └── README.md
└── README.md

Set up actions as so:

- name: Get changed folders
  id: changed-folders
  uses: tj-actions/changed-files@v35
  with:
    json: "true"
    dir_names: "true"
    dir_names_max_depth: 2
    files_ignore: |
      **/docs/**
      **.md
    files: |
      products/**

If I make a change to the file products/README.md and some nested file under products/gri, I expected the output to be a list with one element products/gri, but what I get is a list of two elements: products/gri AND products.

What OS are you seeing the problem on?
ubuntu-latest or ubuntu-20.04
Expected behavior?
I thought by specifying files_ignore: **.md, it wouldn't capture products as a changed directory, since the only change in that directory is the README.md file.
I expected the output to be a list with one element products/gri, but what I get is a list of two elements: products/gri AND products.
Relevant log output
2022-12-29T20:52:51.4277434Z ##[debug]file patterns: packages/**
2022-12-29T20:52:51.4277736Z ##[debug]products/**
2022-12-29T20:52:51.4281309Z ##[debug]excluded file patterns: !**/docs/**
2022-12-29T20:52:51.4281588Z ##[debug]!**.md
2022-12-29T20:52:51.4286486Z ##[debug]file patterns: /home/runner/work/uf-team-data/uf-team-data/packages/**
2022-12-29T20:52:51.4286925Z ##[debug]/home/runner/work/uf-team-data/uf-team-data/products/**
2022-12-29T20:52:51.4287321Z ##[debug]!/home/runner/work/uf-team-data/uf-team-data/**/docs/**
2022-12-29T20:52:51.4287689Z ##[debug]!/home/runner/work/uf-team-data/uf-team-data/**.md
2022-12-29T20:52:51.4288068Z ##[debug]!/home/runner/work/uf-team-data/uf-team-data/.git/**
2022-12-29T20:52:51.4288464Z ##[debug]!/home/runner/work/uf-team-data/uf-team-data/**/node_modules/**
2022-12-29T20:52:51.4288861Z ##[debug]!/home/runner/work/uf-team-data/uf-team-data/node_modules/**
2022-12-29T20:52:51.4289741Z ##[debug]followSymbolicLinks 'true'
2022-12-29T20:52:51.4290738Z ##[debug]matchDirectories 'true'
2022-12-29T20:52:51.4338697Z ##[debug]followSymbolicLinks 'true'
2022-12-29T20:52:51.4339489Z ##[debug]implicitDescendants 'true'
2022-12-29T20:52:51.4340429Z ##[debug]matchDirectories 'true'
2022-12-29T20:52:51.4341159Z ##[debug]omitBrokenSymbolicLinks 'true'
2022-12-29T20:52:51.4343934Z ##[debug]Search path '/home/runner/work/uf-team-data/uf-team-data/packages'
2022-12-29T20:52:51.4357877Z ##[debug]Search path '/home/runner/work/uf-team-data/uf-team-data/products'
2022-12-29T20:52:51.4977328Z ##[debug]followSymbolicLinks 'true'
2022-12-29T20:52:51.4982220Z ##[debug]matchDirectories 'true'
2022-12-29T20:52:51.5116826Z ##[debug]followSymbolicLinks 'true'
2022-12-29T20:52:51.5117718Z ##[debug]implicitDescendants 'true'
2022-12-29T20:52:51.5118742Z ##[debug]matchDirectories 'true'
2022-12-29T20:52:51.5119497Z ##[debug]omitBrokenSymbolicLinks 'true'
2022-12-29T20:52:51.6100393Z Retrieving changes between 9dd49cb6d808f9b3fedca3bbe627d2387ac9f1cc (main) → bd70ae13f7a2c353b21424925f025cafff6f38ac (koshy-ignore-md)
2022-12-29T20:52:52.0907419Z ##[debug]Matching changed files: products|products/gri
2022-12-29T20:52:52.1324336Z ##[debug]Non Matching changed files: [\".github/workflows\"]
2022-12-29T20:52:52.1722034Z ##[debug]Matching modified files: products|products/gri
2022-12-29T20:52:52.2126486Z ##[debug]Non Matching modified files: [\".github/workflows\"]
2022-12-29T20:52:52.6510496Z ##[debug]Added files: []
2022-12-29T20:52:52.6551940Z ##[debug]Copied files: []
2022-12-29T20:52:52.6552610Z ##[debug]Deleted files: []
2022-12-29T20:52:52.6553331Z ##[debug]Modified files: [\"products\",\"products/gri\"]
2022-12-29T20:52:52.6554005Z ##[debug]Renamed files: []
2022-12-29T20:52:52.6554639Z ##[debug]Type Changed files: []
2022-12-29T20:52:52.6555246Z ##[debug]Unmerged files: []
2022-12-29T20:52:52.6555861Z ##[debug]Unknown files: []
2022-12-29T20:52:52.6556600Z ##[debug]All changed and modified files: [\"products\",\"products/gri\"]
2022-12-29T20:52:52.6557424Z ##[debug]All changed files: [\"products\",\"products/gri\"]
2022-12-29T20:52:52.6558199Z ##[debug]All modified files: [\"products\",\"products/gri\"]

Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@koshy1123 I believe this can be configured by setting the match-directories input to false

Thanks! I'll give it a shot and report back.

@jackton1 This isn't quite what I expected, now I get an empty set instead.
My expectation was that products/gri would still be picked up as a change, but with the option match_directories: "false", I just get an empty set. I suspect because it has products/ in it? Unclear to me.
I'll see what else I can do.

@koshy1123 In that case I’ll reopen this and double check why it doesn’t work as expected.

Actually @koshy1123 I believe the bug is with your implementation
Notice you used
- name: Get changed folders
  id: changed-folders
  uses: tj-actions/changed-files@v35
  with:
    json: "true"
    dir_names: "true"
    dir_names_max_depth: 2
    files_ignore: |
      **/docs/**
      **.md
    files: |
      products/**

**.md would only ignore all markdown files in the top level directory as opposed to nested directories to ignore nested directories you'll need to use
- name: Get changed folders
  id: changed-folders
  uses: tj-actions/changed-files@v35
  with:
    json: "true"
    dir_names: "true"
    dir_names_max_depth: 2
    files_ignore: |
      **/docs/**
      **.md
      products/**.md  # OR: **/*.md
    files: |
      products/**

Let me know if that resolved the issue.

Thanks for the pointer - I didn't realize there were different rules compared to Github's file filter documentation. The files_ignore still doesn't work for me. I created an example repo with a very minimal example of the problem if you want to investigate exactly:
Repo: https://github.com/koshy1123/changed-files-illustrate-bug
Example PR: https://github.com/koshy1123/changed-files-illustrate-bug/pull/1
Example action run that was undesirable: https://github.com/koshy1123/changed-files-illustrate-bug/actions/runs/3833710707/jobs/6525429909
See matrix output that has two elements as opposed to just the one I expected. matrix={"folder":["Afolder/ASubFolder","Afolder"]}
All that said, I was able to use the file filter rules to work for my use case, I just changed
files: |
      products/**

to
files: |
      products/*/**

which works well enough for me since I don't expect to capture changes from the top level folder anyway.
Happy to leave this closed, but I leave it to your discretion. Thanks again for your help!

@koshy1123 This should now be resolved


Load image from local pc is imposible, only from an url. How can we do to load an image file from local computer?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Once we finish #803 (which'll be in the next release), you should be able to serve local images using standard Python web frameworks.

#803 is now live. So going to close this issue. Also once we have #960, that should make things slightly easier.

Workaround use base64 to convert image and add  html
  me.html(
                    f'<img src="data:image/png;base64,{image_logo}"  alt="" />'
                )


Retokenize classic

Closes #79 by removing detokenize; should also result in close #23:
Please try to test the social links with whatever crazy nonsense you can think of and any other examples from #79:
I think all examples provided are now fine with pdfLaTeX and XeLaTeX, but a couple independent reviews would be great
Please also check if the combination of pdfLaTeX, classic theme, and social links with underscores all work as in the original issue #23 so that we can finally close that out (again) - it looks to me like things are working now without detokenize, probably due to the overhaul of social icons by @ShadowMitia last year (thank you again).

Happy to help!
I've played around with some examples, it does fix all the character issues inside the document visually. I'll have to test more to be sure.
The links generated however are completely broken with special characters:
\social[twitter]{shadowmitia} gives https://twitter.com/shadowmitia
\social[twitter]{jdoe\_doé\"o} gives https://twitter.com/jdoe_do\Uffffffff
Different sets of special characters seem to give the same result each time.

Happy to help!
I've played around with some examples, it does fix all the character issues inside the document visually. I'll have to test more to be sure.
The links generated however are completely broken with special characters: \social[twitter]{shadowmitia} gives https://twitter.com/shadowmitia \social[twitter]{jdoe\_doé\"o} gives https://twitter.com/jdoe_do\Uffffffff Different sets of special characters seem to give the same result each time.

Good to know - I didn't think about putting umlauts and similar inside the URL.
Perhaps for now we can proceed with this fix and update the documentation to indicate this.
I guess we as a team need to decide if special characters in the links should be supported at this time (I think it's a recent development that that this is even possible) and, if yes, I can do a bit more research on it, but I suspect it's a problem that's a bit bigger than moderncv.
Thoughts?

I think special characters need to be supported, but unless the fix is easy to do, it doesn't have to be now.
Stackoverflow allows special characters in usernames, which would break any social link pointing to a profile. I'm guessing more websites allow more than just ASCII.
Haven't looked into it yet, but I'm guessing we would have issues with other alphabets (I'm thinking Asia alphabets, but Cyrillic would probably cause issues too, and so would others).
One last thing, if we do merge this, I wouldn't create a new version of moderncv just yet, as it could break previously generated CVs if we do.

@ShadowMitia did those links break with pdfLaTeX?
From https://tex.stackexchange.com/questions/87205/special-character-in-url-link it looks like we only need \usepackage[utf8]{inputenc} and \usepackage[T1]{fontenc}, which are loaded with pdfLaTeX in template.tex (not exactly sure what the behavior is now either with XeLaTeX)

That was with pdflatex yes

From https://tex.stackexchange.com/questions/87205/special-character-in-url-link
I use TeXstudio, and the internal PDF visor don't copy links with valid utf-8 so compile with:
pdflatex -synctex=1 -interaction=nonstopmode document.tex
lualatex -synctex=1 -interaction=nonstopmode document.tex
xelatex -synctex=1 -interaction=nonstopmode document.tex
and view with evince...
compiling with lualatex or xelatex works fine and don't need of imputenc and fontenc.
but with pdflatex as @ShadowMitia say don't work: result in non valid utf-8 character even with use of inputenc and fontenc.
It would be ideal if it were possible... still googling.

I tested more, ...but these broken links don't work even on the "\detokenize" version too.
So I think like @mefuller these is another problem. I can't figure exactly if retokenize_classic version can broke something that... don't work anyway...? so I think this clean version permit more control over the links.
Please correct me I'm wrong.

So how about this: we merge this PR, closing #23 and #79 and open a new issue reflecting that hyperlink addresses with special characters break since that particular problem appears to be beyond the scope of either #23 or #79?

I'm ok with merging.

@gcorrad and/or @stephanlachnit do you also approve the PR / plan to close the two issues and open a new one as described above?

Thank you all


brentq returns solutions outside of the bounds

<<Please describe the issue in detail here, and for bug reports fill in the fields below.>>
The root-finding brentq function sometimes returns solutions outside of the bracketing interval, which is unexpected.
Reproducing code example:
from scipy.special import pbdv
from scipy.optimize import brentq

def f(x):
    z = 10 * (100 - x)
    d1 = pbdv(-1.0002, z)[0]
    d2 = pbdv(-0.0002, z)[0]
    return 0.002 * (x - 0.01) * d1 - d2

print(brentq(f, 100, 130))

Error message:
print(brentq(f, 100, 130)) returns 65.0 while 65.0 does not belong to [100, 130].

Scipy/Numpy/Python version information:
0.18.1 1.11.2 sys.version_info(major=3, minor=5, micro=2, releaselevel='final', serial=0)

I see
In [5]: print(brentq(f, 100, 130))
/home/charris/.local/bin/ipython:5: RuntimeWarning: invalid value encountered in double_scalars
  start_ipython()
65.0

Which makes me suspicious :)

Maybe we should error in that circumstance.

I don't see that warning...
In fact, f(130) is a NaN, so I guess that is what confuses the algorithm. I guess a simple bound-check should be performed.

There is a bounds check, but it won't work with nan :(
    if (fpre*fcur > 0) {
        params->error_num = SIGNERR;
        return 0.;
    }

I suppose we change that to  if (!(fpre*fcur <= 0)) {

I know that working with NaNs is painful in python. It's nice if you found a way to get around this problem.
In that case, NaN just signals that the function is very high, but it still admits a root.

related issue #3089


Fix CI builds

ember-sinon v2.2.x changed the minimum Node requirement to Node 6 without bumping major, so we have to pin it to v2.1.x for now until we drop Node 4 support here too
/cc @elwayman02

Sorry about that @Turbo87! We picked up the node requirement change from ember-cli itself dropping support, and I forgot to mark it as a major upgrade. My bad!


Update flatten_video_tracks.py to use deepcopy.

Link the Issue(s) this Pull Request is related to.
Fixes #811
Summarize your change.
Now the example uses deepcopy when splicing the audio tracks into the output timeline.
Reference associated tests.
There are no tests for the code in the examples/ folder.

Codecov Report

Merging #812 into master will decrease coverage by 6.48%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #812      +/-   ##
==========================================
- Coverage   84.02%   77.54%   -6.49%
==========================================
  Files          74      109      +35
  Lines        3124     5647    +2523
==========================================
+ Hits         2625     4379    +1754
- Misses        499     1268     +769

Flag
Coverage Δ

#py27
?

#py37
?

#py38
?

#unittests
77.54% <ø> (-6.64%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ntimelineio/opentimelineio-bindings/otio_utils.cpp
86.90% <0.00%> (-0.46%)
:arrow_down:

src/opentimelineio/typeRegistry.cpp
84.84% <0.00%> (-0.45%)
:arrow_down:

...entimelineio-bindings/otio_serializableObjects.cpp
95.11% <0.00%> (-0.31%)
:arrow_down:

src/opentimelineio/item.cpp
90.47% <0.00%> (-0.15%)
:arrow_down:

...lineio/opentime-bindings/opentime_rationalTime.cpp
97.36% <0.00%> (-0.07%)
:arrow_down:

src/opentimelineio/serialization.cpp
83.60% <0.00%> (-0.06%)
:arrow_down:

src/opentimelineio/timeline.h
100.00% <0.00%> (ø)

src/opentimelineio/typeRegistry.h
100.00% <0.00%> (ø)

...imelineio/opentime-bindings/opentime_timeRange.cpp
100.00% <0.00%> (ø)

...melineio/opentimelineio-bindings/otio_bindings.cpp
100.00% <0.00%> (ø)

... and 46 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8be8eaa...f694407. Read the comment docs.

@ssteinbach do you have any clue about why the code coverage system is confused by this? The file I changed isn't even run by the unit tests, so I don't see how the coverage could drop by any amount, let alone 6.48%

Thats probably a bug in coverage, which happens sometimes.  I'll try kicking the CI and see if the new report is correct?


Try to fix CI

Description
It looks that this issue https://github.com/apache/incubator-mxnet/issues/16815 is blocking the CI.
Revised based on the error message in http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-16902/4/pipeline
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@marcoabreu I’m also wondering how this would ever be possible. I guest it’s due to some internal update of the CI machines.

The environment is fully dockerized, so changes on the CI machines shouldn't impact it as far as I know

this could be a Docker cache issue. I can't reproduce locally. TVM lib seems corrupted.

If this change is not fixing the failure, could we revert it if the additional deps are not needed?

https://github.com/apache/incubator-mxnet/issues/16478#issuecomment-558771696

@larroy Yes, we should revert it


WIP: Add support for using Jaeger operator, if installed and enabled, to p…

…rovide production deployments
This PR supersedes #9508, using a similar approach to #13407, expecting the operator to have been installed previously.
The operator is enabled using --set tracing.jaeger.operator.enabled. By default an in-memory deployment of Jaeger would be used (equivalent to the non-operator approach).
The Jaeger operator's custom resource is defined under the tracing.jaeger.spec node (suggestion for another subnode name welcome).
For example, a production deployment can be enabled by using --set tracing.jaeger.spec.strategy=production, and alternative storage options can be defined using --set tracing.jaeger.spec.storage.type=elasticsearch.
More information on the operator can be found here, with some examples of custom resources here.
Resolves #8893
Resolves #8965

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: objectiser
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: gyliu513
If they are not already assigned, you can assign the PR to them by writing /assign @gyliu513 in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@objectiser: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
8d381d456bf5ab1ec975ebaca5efe599cc63f5bb
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Further work on this PR is pending decisions in in the Policies & Telemetry WG on how to deal with component operators.
One option is use of operator instead of current templates, but still only install minimal in-memory version only. Then provide docs on how to use operator to configure a more comprehensive Jaeger deployment - so that the complexity of the CR for such a deployment doesn't have to be managed by Istio.
However the point was raised about needing a simple way for users to try out Istio without having to install a set of operators first.

I like the idea of just specifying the CR for the component operators within Istio - we are using the operators of the components and making the CRs manually for now, so as long as I can continue setting the zipkin endpoint (so the envoy proxy traces go to my manually created Jaeger service), and as long as I can optionally continue manually deploying the operators, either option sounds good for us.
Perhaps when the Istio operator is ready, the Istio CR could just be tied to component CRs by name / label / etc in various fields, like (adapted from Banzai Cloud example):
apiVersion: istio.banzaicloud.io/v1beta1
kind: Istio
metadata:
  labels:
    controller-tools.k8s.io: "1.0"
  name: istio-sample
spec:
  version: "1.1.6"
  mtls: false
  autoInjectionNamespaces:
  - "default"
  pilot:
    enabled: true
...
  tracing:
    enabled: true
    kind: jaeger
    cr: my-jaeger-cr

Then Istio could just provide a bunch of CRs for demo purposes.
Then there could be a helm chart for "istio-demo" that has the istio operator as well as all the component operators as "subcharts" or dependencies - that way the official operator doesn't install the component operators, but there is an option available to do all at once. This would also be helpful to show how to configure the component operators manually when ready.


Regenerate rules

This PR cleans up data files and rules as necessary so that all rules are up to date.
@lifeforms please look at 932150, 942290 specifically
@fzipi please look at 932300 specifically

Ugh, sorry I don't have my laptop till the weekend. Why did we removed the !$ on 932300?

Ugh, sorry I don't have my laptop till the weekend. Why did we removed the !$ on 932300?

It's an empty suffix comment. The regex is exactly the same without it.

@lifeforms I think we are waiting on your approval here.

OK from me!

A relevant issue was described in: #2756


Fall through still sets status code when handleError: false

When config is set as handleError: false ecstatic is still setting the status code to 403 or 404 if there is no index or static file.
I would have expected that it would have left the status alone as express.static does.
Is there a way to truly not handle errors?

Sounds like a bug. If you feel like investigating, ecstatic's source code isn't actually all that big and only a little hairy. If you send me a pull request I will merge.

@jfhbrook It looks like it will be a fairly fundamental change.
Status is set during processing and passed on if handleError is false - It looks like it should be ignored throughout status-handlers - but that would be a major refactor.
I think this isn't so much a bug, it's just "expected behaviour"
My expectation of "handleError" was different to design.
The application does in fact not handle the error, but it does generate an error state.

Hmmm
Yeah OK, I think I misunderstood the issue, it's been a minute since I've looked at this. It's coming back to me, though.
I think this issue is captured by https://github.com/jfhbrook/node-ecstatic/issues/154 and you're right, it's a pretty major refactor. Loosely related is https://github.com/jfhbrook/node-ecstatic/issues/183 , which would add restify support. My thinking here is to refactor to have a non-middleware API and then maybe a middleware-based adapter? But you're right, this is a bigger issue than I thought.
I'm gonna close this issue just cause I think #154 captures it pretty well, but if you disagree we can talk.


Occasional HTTPError 401 Unauthorized

Occasionally, the urlopen raise HTTPError with 401 Unauthorized response. I don't know what causes it (maybe too much request in a short time?)
At this time, I tried to catch those exceptions in my calling script and rerun the download after some timed sleep. But this is only a patch solution.
Any suggestions on what causes it?

@c0redumb, I think you are right. According to the HTTP/1.1 standard,  [https://tools.ietf.org/html/rfc7235#section-3.1]

"The 401 (Unauthorized) status code indicates that the request has not
been applied because it lacks valid authentication credentials for
the target resource."

I reckon Yahoo has implemented some kind of time out defense against sever overloading and potential ddos attacks by tracking the number / frequency of requests from a specific ip address.

Perhaps one way to minimize requests is to keep a local database of  retrieved data. Then when a request is made, the function first checks if the requested data exists on the database and downloads only the necessary data. However this assumes the requests are not due to tests / debugging where you have to hit em up a fair bit.

I have another code that actually stores historical data, and only pull EOD data for the last few days that is missed. This way the data query and transmission is minimized. However, I still get this kind of unauthorized errors occasionally.
My guess is that the restriction Yahoo put in place might be related to the number of queries, not the amount of data returned. So if I am pulling one-day data for 500 stocks, it is 500 queries. That is the same number of queries as pulling 10-year data for 500 stocks.

I download the whole historical data ticker by ticker with no problem. But if I use multiple threads, it gives error occasionally. So for the same ip, a minimal interval between 2 downloads should be specified.
On May 30, 2017 5:49 PM, c0redumb<EMAIL_ADDRESS>wrote:
I have another code that actually stores historical data, and only pull EOD data for the last few days that is missed. This way the data query and transmission is minimized. However, I still get this kind of unauthorized errors occasionally.
My guess is that the restriction Yahoo put in place might be related to the number of queries, not the amount of data returned. So if I am pulling one-day data for 500 stocks, it is 500 queries. That is the same number of queries as pulling 10-year data for 500 stocks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/c0redumb/yahoo_quote_download/issues/3#issuecomment-305018770, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGofRPdF38ykjsseiXpDrBtFg2IQRbkjks5r_I7rgaJpZM4No4jX.

My code used to do multiple thread (set to 10) concurrent download with the old API. But I found that doesn't work with the new API.
Any more information on the query intervals? 100 per minute per IP?

Is there anywhere I can modify the useragent?

Is there anywhere I can modify the useragent?

https://docs.python.org/3/library/urllib.request.html#urllib.request.Request

Now claims user agent of Firefox with 9c683a9. Hope that improves the chance for a good response.

It appears to me that this error is not due to user agents.
Here's my try:

I've built a loop in the main function to catch the error and retry five times if needed.
I've then added a function to yqd in order change the user agent (Linux, Mac, Win..) at each of the five loops.

Anyway, in case of unauthorized error the loop continues till the end, but unsuccesfully.
Other ideas?

Sorry I haven't touched this for a while. Let me see if I can do something to make it better.

I think I found a solution, though I still don't know the cause.
The solution is to clear the cookiejar before retry the download. In my previous attempts, the retries are done, but implicitly the library sends the same cookie, which results in the same failure.
With that in the code, I am planning to release a v0.9 soon.


feat: improved proxying for the sap.fe.test library

Added an execute call that mimick the opaTest functionality.
Tests can be written like if they were running within OPA...
Generic proxying to the frontend code.

woah, what a BÄM 🚀
thanks  🙌  a ton for this!


Memory Leak issue in Cluster

Warning: Can't call setState (or forceUpdate) on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in the componentWillUnmount method.     in Cluster (at Clusters.jsx:30)
I have the above warning coming up in the Cluster,
the code i have is as below
class Clusters extends Component {
  clusterMarker = (coordinates, pointCount, getLeaves) => {
    if (coordinates) {
      return (
        <ClusterMarkers
          coordinates={coordinates}
          pointCount={pointCount}
          key={`key${coordinates.join(',')}`}
          category={this.props.category}
          leaves={getLeaves}
        />
      )
    }
  }

  render() {
    return (
      this.props.signs.length > 0 && (
        <Cluster
          ClusterMarkerFactory={this.clusterMarker}
          maxZoom={maxZoom}
          zoomOnClick="zoomOnClick"
          ref={cluster => {
            this.cluster = cluster
          }}
        >
          {this.props.signs.map(
            sign =>
              sign.geometry !== null && (
                <Marker
                  key={sign.properties.id}
                  coordinates={sign.geometry.coordinates}
                  anchor="bottom"
                >
                  <MarkerImage
                    sign={sign}
                    src={parkingIcons(sign.properties.smo_subtype)}
                  />
                </Marker>
              )
          )}
        </Cluster>
      )
    )
  }
}

there's a couple of PR's addressing this issue


Entity Property Type Override

Closes https://github.com/CreateAPI/CreateAPI/issues/176

Implements individual property type override. This allows both general property naming and specific property type, mirroring the usage of rename.properties. I consider this an advanced feature, since this allows such fine control over single properties.
Overriding Optional
Since this is an advanced feature and by definition allows one to override the actual property type, it could be thought that his encompasses the entire type which includes whether it is an Optional.
For example, assume that Pet.tag would originally map to String?, but with the below:
entities:
  propertyTypeOverrides:
    Pet.tag: UUID

this would result in Pet.tag mapping to UUID, therefore making this required to the client. This can also happen the other way, making a non-optional property generate as an optional instead. This would also of course be a responsibility to the developer that the types can properly decode/encode.
I don't do this here (Optional is stable through the type override), however I think this should be a consideration.

FWIW, this looks good to me. 🤷♂


Package requirements for usage

I couldn't find the required packages for running the code in the example. Therefore I created a summary of them to help others.
The following are the packages that I needed to install in order to run the example in the README file:
regex
numpy
scipy
sklearn
jsonlines
tqdm

Thank you @Martin36! We have added a requirements.txt now. More features/examples for reading in FEVEROUS data will be added in the coming days.


Added Sort as of Issue #50

As discussed in issue #50 here is the pull request for sort()
https://github.com/caolan/highland/issues/50
Regards,
Dennis

Since this got mentioned recently - @snnd, can you break this up? You can refer to split/splitBy or uniq/uniqBy for examples. I'd also be glad to take it over if you don't have time/interest :)

@snnd Like #11 there's not very much left on this to do so I was wondering if you minded me finishing this pull request on your behalf? I've pulled in all your changes onto a branch of my own and tidied everything up so everything is basically good to go.

I'm guessing @snnd is busy with other work looking at his Github stats for this year so I'm going to push ahead with this one anyway. The original issue that prompted this PR has been open for more than a year so it'd be nice to close it.

Sorry for being IDLE for a while too long now. I would be glad to see someone continuing this PR for me.
Cheers,
Dennis
On 9 Mar 2015, at 11:35, Stefano Vozza<EMAIL_ADDRESS>wrote:

I'm guessing @snnd is busy with other work looking at his Github stats for this year so I'm going to push ahead with this one anyway. The original issue that prompted this PR has been open for more than a year so it'd be nice to close it.
�
Reply to this email directly or view it on GitHub.

No probs, you still get full credit in the commit history, I just squashed your commits into one with some light changes.

Closing in favour of #245.


Ros2 panther manager

Most of the work is done, can't be merged as it is waiting for plugin PR first. Plugin PR should include merging newest ros2-devel branch.
Things to finish after plugins are added:

remap fan_enable, aux_enable etc. (should be /hardware/name) services controller launch
update bringup launch
remove dummy_scheduler
add shutdown_hosts.yaml
change ExpectThrowWithDesription to EXPECT_TRUE(IsMessageThrown) in tests

* Why do we push to have several independent trees, since they are in the end using the same resources and executing interchangeably (SingleThreadedExecutor)?

My mistake, this should be MultiThreadedExecutor. It's fixed. The idea behind two separate BTs is that safety is a crucial part of system  and should be handled independently, we don't want deadlock or some sleep behavior in lights affect safety features.


[Bug]: Fix the contact page

Description

Please assign this to me @Vaishnavi-Patil2211 . I want to fix the contact also
Expected Behavior

Show us the trouble with screenshots

Which browsers are affected?
No response
Which OS are affected?
No response
Enter the version of your web browser

Device?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

@Vaishnavi-Patil2211 please check this https://github.com/OSCode-Community/OSCodeCommunitySite/pull/1146 so that I can move to next issue


How to run zfp as a c code with gcc

Hello,
I am trying to use zfp on windows to compressed my dataset so I started from the simple.c file that compressed 3D array, I am trying to run this code by using this command
gcc -o a simple.c is that correct ?
I got on these errors:
Please can you help me how to use zfp on windows ?
Where can I found these functions please?

zfp_field_3d
zfp_stream_open
and the other functions in the errors?
Thanks in advance.

simple.c: In function 'compress':
simple.c:39:12: warning: implicit declaration of function 'stream_open'; did you mean 'zfp_stream_open'? [-Wimplicit-function-declaration]
   stream = stream_open(buffer, bufsize);
            ^~~~~~~~~~~
            zfp_stream_open
simple.c:39:10: warning: assignment to 'bitstream *' {aka 'struct bitstream *'} from 'int' makes pointer from integer without a cast [-Wint-conversion]
   stream = stream_open(buffer, bufsize);
          ^
simple.c:66:3: warning: implicit declaration of function 'stream_close'; did you mean 'zfp_stream_close'? [-Wimplicit-function-declaration]
   stream_close(stream);
   ^~~~~~~~~~~~
   zfp_stream_close
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0x43): undefined reference to `zfp_field_3d'
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0x51): undefined reference to `zfp_stream_open'
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0x70): undefined reference to `zfp_stream_set_accuracy'
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0x80): undefined reference to `zfp_stream_maximum_size'
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0xa4): undefined reference to `stream_open'
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0xba): undefined reference to `zfp_stream_set_bit_stream'
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0xc6): undefined reference to `zfp_stream_rewind'
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0x10c): undefined reference to `zfp_decompress'
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0x155): undefined reference to `zfp_compress'
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0x1c9): undefined reference to `zfp_field_free'
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0x1d5): undefined reference to `zfp_stream_close'
C:\Users\hp\AppData\Local\Temp\cceazl20.o:simple.c:(.text+0x1e1): undefined reference to `stream_close'
collect2.exe: error: ld returned 1 exit status

You need to tell gcc where to find the zfp headers and library.  Assuming you're in the examples directory and have already built the zfp library, add -I../include -L../lib -lzfp to the compile line.  If you're using GNU make, then you could also build the examples just by typing make BUILD_EXAMPLES=1 in the top-level zfp directory.

Thanks for your answer, I use this command \ZFP-master\examples>g++ -I../include -L../lib -lzfp simple.c -o a
And I got on this errors!
simple.c: In function 'int compress(double*, int, int, int, double, int)':
simple.c:47:12: error: 'stream_open' was not declared in this scope
   stream = stream_open(buffer, bufsize);
            ^~~~~~~~~~~
simple.c:47:12: note: suggested alternative: 'zfp_stream_open'
   stream = stream_open(buffer, bufsize);
            ^~~~~~~~~~~
            zfp_stream_open
simple.c:74:3: error: 'stream_close' was not declared in this scope
   stream_close(stream);
   ^~~~~~~~~~~~
simple.c:74:3: note: suggested alternative: 'zfp_stream_close'
   stream_close(stream);
   ^~~~~~~~~~~~
   zfp_stream_close
simple.c: In function 'int main(int, char**)':
simple.c:90:25: error: invalid conversion from 'void*' to 'double*' [-fpermissive]
   double* array = malloc(nx * ny * nz * sizeof(double));
                   ~~~
```~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

This is C, not C++ code.  Try compiling with gcc instead of g++.

Please in which file that I find these functions ?
undefined reference to zfp_field_3d' undefined reference to zfp_stream_open'
undefined reference to zfp_stream_set_accuracy' undefined reference to zfp_stream_maximum_size'
undefined reference to stream_open' undefined reference to zfp_stream_set_bit_stream'
undefined reference to zfp_stream_rewind'  undefined reference to zfp_compress'
undefined reference to zfp_field_free'  undefined reference to zfp_stream_close'
undefined reference to `stream_close'
can find these functions?

They're all declared in include/zfp.h and defined in src/zfp.c (except stream_open() and stream_close(), which you'll find in include/zfp/bitstream.inl).
Did you try compiling with gcc (not g++)?  Some linkers require that libraries appear at the end.  Try gcc -I../include -L../lib simple.c -o a -lzfp.

@mearcla Are you still having issues?  If not, I'd like to go ahead and close this issue.

@mearcla I'm going to go ahead and close this issue for now. If you are still having problems feel free to reopen.


Inconsistent URL path handler (leading slashes)

If the path for a request for example contains leading slashes like: ////slug////slug2///
the getPath() method on the Uri component returns:
/slug////slug2/// (converted the leading slashes to a single slash) which in my opinion is incorrect and should be ////slug////slug2///. According to the code in Uri.php It's done to prevent XSS attempts. However I don't see how?

this is cause of PSR-7 standard, I think you have to create an issue here.


Fix curl check regression and simplify

This fixes #717 by capturing the output into a string for later version extraction.
Additionally, it replaces the subshell calls to head and awk with basic
bash read which is 95+ times faster (in my tests, based on 100000
iterations), simpler and safer (no forking that could fail etc).
NOTE: You'll notice I'm not redirecting stderr anymore. Even the oldest
version of curl I could find (4.8 from 2000) still outputs to stdout so I'm
not sure why this is being captured.
Lastly, it also reuses the curl version information previously obtained for the
command_version() function.

Thanks for your input. I modified the dependency checks a bit and all your issues should be resolved with that. See a07c8d14f62c3b32e4f9e963dabc0691f55fcde6.


Search by release date

Because of some inconsistencies with seasons names and order, I'd like a lookup for release date for a show. The season is that my lookup fails because of the data appears like this:
/shows/some-show/season/2020/episode/3
I've not found a way to map between numerical seasons and year-bases season, however I do have the original air date so if I could do a search based on that I would indeed get a match.

You can call /shows/:id/seasons?extended=episodes to get all episodes for a show, then loop over each episode to find your matches. This returns a ton of data, so if you are able to use other methods that would still be better for the Trakt API and for your app.
A better alternative might be to call /shows/:id/seasons (no extended info) then /shows/:id/seasons/:season to get episodes for each.
There isn't a super clean solution to this unfortunately.


[BUG]  Rofi not working properly when used in a script executed by a button

Checklist before submitting an issue

[X] I have searched through the existing closed and open issues for eww and made sure this is not a duplicate
[X] I have specifically verified that this bug is not a common user error
[X] I am providing as much relevant information as I am able to in this bug report (Minimal config to reproduce the issue for example, if applicable)

Description of the bug
Using a button with the property to execute a script containing rofi/wofi, causes rofi/wofi in dmenu mode to return nothing, so you cannot store content coming from rofi/wofi in variables.
Reproducing the issue
Just create a window and a widget and then put the script directory to execute ej:
(defwindow centered
  :class "centered"
  :monitor 0
  :geometry (geometry
    :anchor "center center"
    :height "25%"
    :width "25%"
)
  :focusable false
  :stacking "fg"
  :exclusive false
  :style "background-color: red;"
  (test_widget)

)

(defwidget test_widget []
  (box
      :style "background-color: red;"
    (button
    :onclick "$HOME/.config/eww/scripts/test"
    (label :text "click me")
    )
  )
)

And then the script to execute should be like this:
#!/bin/bash

wifi_password=$(rofi -dmenu -p "Password: ")
dunstify "$wifi_password" #For debug

Now, try to run the script using the button, and then run the script from the terminal, you will notice that the result is not the same, instead it only works correctly if one runs it with the terminal.
Expected behaviour
The expected result is supposed to store the content in a variable, and then send a notification with the entered information, but that does not happen.
Additional context
No response

solved issue, the solution is put this:
after:
:onclick "$HOME/.config/eww/scripts/test &"

before:
:onclick "$HOME/.config/eww/scripts/test"


Data conversion

Thank you for the code.
How do I convert the standard coco file format into the json format usable by BCNet?
How to get bg_object_segmentation in the json file available on BCNet？

Thanks for watching our work. Please refer to process.sh.

Thank you for your answer. I have solved the problem of the data set, but I encountered a new problem during training. The training time of all images is 0 seconds, and then I started to infer. Can I ask what the possible reason is ?

It means there is training error and the models skips this training stage to do evaluation.


⚠ Bofrost.at has degraded performance

In 956616f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1041 ms

Resolved: Bofrost.at performance has improved in ea47213 after 6 minutes.


Support for dbtunnel proxy

chainlit (partially implemented without share)
dash
fastapi
flask
nicegui
shiny-python
solara

"chainlit (partially implemented without share)" - does this mean I can't share the link even if someone has attach permission on my cluster? Because that seems to be the case for me.

@yrwrstnhtmr all the work is done i will close this issue. you can share via the link. You can share links, this was a past limitation that i was able to get around with a custom proxy.


Node build timed out on MacOS, dbg

https://grpc-testing.appspot.com/job/gRPC_master/14209/config=dbg,language=node,platform=macos/console
Many of the warnings are about using old versions of libraries.

Doesn't look like we're seeing this anymore. Closing this as part of buildpony bankruptcy.


Export to docx

Add ability to export to docx

Added to backlog: https://trello.com/c/4ZNWB0SW

I think that's probably getting outside the scope of markpad core. Perhaps an extension framework/mechanism needs to be built up
Sent from my Windows Phone

From: SimonCropp
Sent: 9/03/2012 9:32 AM
To: Paul Jenkins
Subject: [DownmarkerWPF] Export to docx (#136)
Add ability to export to docx

Reply to this email directly or view it on GitHub:
https://github.com/Code52/DownmarkerWPF/issues/136

The ability to export Markdown to another format would be cool, but much like we're seeing with Pretzel we need to get the extensibility API right.

I guess my use is for managing doco. organisations always want docx. but as the person who maintains it I would much prefer to use markdown.


Let shared locks be shared again

This might not be the best solution, but I think this required the least amount of modifications. Any suggestions are welcome.

I'll stop making this mess here, and wait for some instructions on these CI tools :D

lol, well I think you got it except that some things are platform specific. The travis builds are failing because pypiwin32 is not available on linux so that dependency should be platform dependent.
The windows version is mostly ok but it's failing some pep8 compliance checks. They can easily be checked using flake8:
pip install flake8
flake8 portalocker

To locally test everything (although I'm not sure if that works on Windows), you can try tox:
pip install tox
tox

Great, I made platform specific tests. Will work on that.
The windows error is kinda weird though. I verified on my computer, that pip can't install pypiwin32 on 2.6 (it fails with an exception in os.path.join(), which I think is awesome). I could install it with conda, but my tests are not 2.6 compatible :D
I'm starting to like these tools, they really make you write good code.

@matekatona I've pushed out a new release thanks to @TWAC which might fix (part of) this issue already.

@matekatona can you confirm that this issue still exists?

Closed due to inactivity, I believe it's fixed already :)
I'll definitely merge in the tests though

I've merged in all of the changes and made some adjustments so it works on Python 3 as well. Thanks for the awesome work @matekatona and sorry for the slow release ;)

Wow, I completely forgot about this. Unfortunately I haven't used Python for more than a year now, but I'm glad I could contribute a little.
Thanks for cleaning up after me :)

I still appreciate it a lot, it saved me a lot of time not having to debug that part as well :)


Prefund check

Closes #217
Introduced changes

Skip prefund burners when already deployed
Readable default prefund value

Checklist

[x] Linked relevant issue
[ ] Updated relevant documentation
[ ] Added relevant tests
[ ] Add a dedicated CI job for new examples
[ ] Performed self-review of the code
[x] Built packages
[x] Built examples

could you just fix the title for the CI


Change vectors to be generic over element type.

Most important thing to note upfront: despite the pretty significant implementation changes, the API is basically unchanged.
This PR results in changing SimdI32<4> to Simd<i32, 4>, Mask8<2> to Mask<i8, 2>, etc.
This results in a pretty substantial reduction of the API, which now only contains 6 things (not counting type aliases): Simd, Mask, SimdElement, MaskElement, LaneCount, and SupportedLaneCount.
This allows functions such as splat, shuffle, gather_or, lanes_eq, select, etc to be implemented generically once for all vectors, rather than once for each element type (with tons of macros 🤢).  Users can still call these functions in the same way as before this PR--only the implementation is different.
There is one notable downside: rustdoc really does not know how to handle this situation and the page for the Simd type is massive.  This PR does not attempt to unify implementations over similar types, such as all float types or all unsigned integers.  A future PR changing some of the functions to trait functions could reduce this and make those functions a little more discoverable.  Alternatively, there might be some improvements we can suggest/add to rustdoc to help wrangle the extra functions.

i think if you make a type alias and then impl methods on that type alias, rustdoc will collect the methods there.
However, that would halfway undo the PR

Even if that is true, is that desirable?  It may be a temporary "solution" but in some ways I think it's more confusing because we want people to use Simd, not SimdF32.

In practice many things still can't be generic over your element type anyway. Slightly jiggering things to make rustdoc behave a little better doesn't make it any worse.

Given there are multiple routes of exploration for improving the docs I am okay with deferring them somewhat to a later PR, even if they matter.


Support personality quiz type

What does this change?

Adds support for personality quiz
splits out Quiz atom into 2 components: PersonalityQuizAtom and KnowledgeQuizAtom
Adds unit test for PersonalityQuizAtom
Adds reset button for PersonalityQuizAtom

How to test
Try out the component on storybook
Have we considered potential risks?
May be other quiz types that we do not support
Images
Frontend (original/base)

Current implementaion

Future work

use Soure's radio buttons when https://github.com/guardian/source/issues/623 is resolved

I've not dug into it but are we happy this component is accessible?


conda: Cant find executable

Been trying to get to know python and all about virtual environments. Using normal Cpython and virtualenv everything works but trying to use virtual envs defined though conda does not. I am having trouble finding out which package that needs to be updated (or if its even at all possible)
As envs made by anaconda go into the ~/.pyenv/versions/anaconda3-4.4.0/envs dir I changed this:
(setenv "WORKON_HOME" "/Users/****/.pyenv/versions/anaconda3-4.4.0/envs")
Would be better to change the prefix for anaconda but this should show my problem for now.
See: this
So pyenv-workon can find the envs. It can find them but when changing to it I get the following error by anaconda-mode:
/usr/local/Cellar/emacs-plus/HEAD-400934b_2/Emacs.app/Contents/MacOS/Emacs: /Users/****/.pyenv/versions/anaconda3-4.4.0/envs/testtt/bin/python: No such file or directory
This makes sense as looking in the directory these are the only files:
activate
conda
deactivate

Now I really do not know if the problem is from this package, pyenv or anaconda-mode. I also tried using another package conda.el but ended up with the same error. Any ideas where to start looking. I did try editing conda.el to a new exec-path when activating but no luck. I am leaning towards that it is the function pythonic-activate that helps all the packages find the executable but digging there just got me lost. Any ideas?

Hmmm I'm not sure. I haven't used conda much, and my (admittedly limited) understanding is that conda environments work in a completely different way than virtualenvs and that trying to use them together is only going to lead to a lot of pain. The general philosophy with conda seems to be that they don't try to interoperate with the rest of the python ecosystem and have their own separate world of tools. Because I'm a dedicate pip/virtualenv user, I'm not going to make any effort to add conda support to this package. Sorry I can't be more helpful :/ I recommend looking for (or writing!) a conda-specific package that does whatever you're using this one for.


[Profiler] Improve if statement coverage

Fix counters for regions following else ifs, fix the counters for else if conditions, and fix handling of break statements. Also while here, clean up the handling of branch exit regions such that we don't generate multiple overlapping regions for each branch, but a single region at the end of the entire if statement that accounts for all exiting control flow.
rdar://104078910
rdar://104079242

@swift-ci please test source compatibility

@swift-ci please test


AB2D-6108 TIBQ: ab2d-sample-client-bash

🎫 Ticket
https://jira.cms.gov/browse/AB2D-6108
🛠 Changes

Added _until parameter to bootstrap.sh and start-job.sh

ℹ Context
Currently, AB2D API only accepts _since as a date parameter. The goal is for AB2D to be able to support the use case where a PDP could run jobs with time interval based queries so that they could download their data in batches. To achieve that, we will add a _until parameter to define the upper ceiling of the date range.
Both the since and _until parameter will be applied to the lastupdated_Date.

🧪 Validation

Added some error handling so that the script will exit if a job fails to export rather than continuing to try and monitor / download results.


Add support for several forma configurations

Add default configuration that can be extended by adding configuration with given key. You must define default configuration and optionally you can extend it

Need to fix gitignore


One signature per power-up

It could be beneficial to give the option to the user to only allow 1 transaction to be signed per tap (where a tap represents a power-cycle). This would make it easier to detect a compromised client since it would prevent it to sign additional, unrequested transactions when signing yours (it can do it but then on etherescan you wouldn't find your transaction, since it only signed the unrequested one).
The implications on usability have to be evaluated and it should only be implemented as an option, since some use case might require several transactions to be signed at once.

A compromised client could cache the PIN and perform multiple transactions by simply resetting the card in between. Closing  for now, can be reopened if a different use case is fullfilled by this feature