clang-format: disable regrouping and sorting of includes

Disable regrouping and sorting of includes which can cause issues due to
order requirements in some subsystems.
Fixes #48520
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

blocking until further discussion

Please provide some reason or 'request changes'. PRs shall not blocked like this without any reason.

First, further discussion is a reason.
I can understand that this option may cause issues when dealing with subsystems that have non-self-contained headers. However, it must be clear that this IS a bug that needs to be fixed, not something that "may be difficult, if possible at all". IMO the reality is that, unless it is one day enforced, few people will take care of fixing this issue, which means this clang-option will need to stay disabled K_FOREVER.
So, I'd propose the following:

Finally, write header usage guidelines (I volunteer to do that).
Allow some time for subsystems to follow the guidelines (2-3 releases?).
Re-enable this option. If by then some subsystems have not fixed their header bugs, it's really their problem. People doing their job properly can't stay behind forever.

Note that there are some tools that may be useful to enforce/check https://github.com/include-what-you-use/include-what-you-use (I haven't tried them, though).

I can understand that this option may cause issues when dealing with subsystems that have non-self-contained headers.

clang-format or any other tool used for applying style changes MUST NOT have any side effects on the code. We should run this on code with enough confidence that it only applies style changes and does not impact the generated binary.

However, it must be clear that this IS a bug that needs to be fixed

If so, we should have bugs open for this, if not already.

IMO the reality is that, unless it is one day enforced, few people will take care of fixing this issue, which means this clang-option will need to stay disabled K_FOREVER.

That is what I said here: https://github.com/zephyrproject-rtos/zephyr/issues/48520#issuecomment-1213231479
I am all for fixing the issues, using clang-format for this is not the right path we should take however.

So, I'd propose the following:
Finally, write header usage guidelines (I volunteer to do that).

Great.

Allow some time for subsystems to follow the guidelines (2-3 releases?).

Sounds good.

Re-enable this option. If by then some subsystems have not fixed their header bugs, it's really their problem. People doing their job properly can't stay behind forever.

Enabling the header regrouping/sorting option should be reconisdered at a later time, I agree.

@gmarull
The issue makes clang-format useless for many areas of Zephyr. I'm not sure if that is where community wants to go. At least not me. If there is a feature that is not working and fix can't be implemented in a reasonable short time, please do not block possibility to use the tool.
The clang-format configuration should not be changed to current one, because it started to beak code.
The change should have been applied in multiple steps when all formatting issues caused by change had been fixed before that.

@gmarull The issue makes clang-format useless for many areas of Zephyr. I'm not sure if that is where community wants to go. At least not me. If there is a feature that is not working and fix can't be implemented in a reasonable short time, please do not block possibility to use the tool.
The clang-format configuration should not be changed to current one, because it started to beak code. The change should have been applied in multiple steps when all formatting issues caused by change had been fixed before that.

clang-format is still incompatible with a few checkpatch rules, so for now, it can't be used without manual tweaks anyway. It is also unable to format some macrobatics without assistance, see e.g. https://github.com/llvm/llvm-project/issues/56083. If you discover bugs in your headers, please, fix them.

clang-format is still incompatible with a few checkpatch rules, so for now, it can't be used without manual tweaks anyway. It is also unable to format some macrobatics without assistance, see e.g. llvm/llvm-project#56083. If you discover bugs in your headers, please, fix them.

It is something else if formatting is broken and requires manual tweaks to pass checkpatch, than broken source code that does not pass compilation... In this case (Bluetooth Controller) it breaks builds. Hence the change should not have been introduced in first place.
I can't fix headers because it is the Bluetooth Controller and that would be quite long and demanding.

I can't fix headers because it is the Bluetooth Controller and that would be quite long and demanding.

So that means we'll never be able to use that feature because Bluetooth Controller headers "can't be fixed"?

So that means we'll never be able to use that feature because Bluetooth Controller headers "can't be fixed"?

I didn't say never. I said that will take time, hence it blocks community working on Controller from use of the tool.

You can adjust your clang-format locally before formatting until BT headers are fixed.

It is not a solution but a temporary workaround that may be painfull -> every rebase I would have to stash and apply the change.

@gmarull You change request blocking this PR makes the tool useless at least for part of Zephyr's community.
If you have other suggestion than "fix your headers", please state it. In other case, it would be nice if you remove the change-request.

@gmarull You change request blocking this PR makes the tool useless at least for part of Zephyr's community. If you have other suggestion than "fix your headers", please state it. In other case, it would be nice if you remove the change-request.

This is not blocking anything. If you try clang-format as-is today, you'll find many other issues (checkpatch clashes, macrobatics formattic issues, etc). If your subsystem has bugs in headers, just disable the sorting option on your end until these are fixed. And remember, clang-format is not enforced, so my -1 stands.

As I already stated, this PR fixes what shouldn't have been done in first place. If I was aware of the problem I wouldn't have let to merge commit that enabled this clang-feature until all modules has got headers fixed.
If you need this feature enabled now, then you can enable it locally and let community use the tool without any additional assistance.
Please do not block the PR because I see no argument against blocking it.

@nashif @stephanosio are you also ok with @gmarull statement?

@nashif @stephanosio are you also ok with @gmarull statement?

No I am not OK. All of this is contrary to the initial statement by @gmarull  here https://github.com/zephyrproject-rtos/zephyr/pull/49293#issuecomment-1222241729.
Asking contributors to locally change the .clang-format to not apply this rule is absurd. We can ask you the same, to apply the rule locally when you make changes instead.

@gmarull

Why don't we disable many rules that clash with checkpatch, or that are incompatible then? Because remember, .clang-format as-is today can't be used without getting many checkpatch warnings. Do we need to disable a nice feature because some subsys has issues in its headers? Without a plan to fix that? Really?

checkpatch? Why are we talking about checkpatch?  This rule in .clang-format does changes to the code that can break stuff, right now visible in one subsystem, but can basically break any part of the code. If I, as a unsuspecting developer want to apply style and I end up introducing issues, this is a problem.
Automatically applying code styles to an existing code should not have any side-effects. If you think the results of this rule in .clang-format should be enforced and should be make a rule that everyone follows, then it needs to clearly be part of the style guidelines of the zephyr project, right now it is not and it is not enforceable.

@gmarull

Why don't we disable many rules that clash with checkpatch, or that are incompatible then? Because remember, .clang-format as-is today can't be used without getting many checkpatch warnings. Do we need to disable a nice feature because some subsys has issues in its headers? Without a plan to fix that? Really?

checkpatch? Why are we talking about checkpatch? This rule in .clang-format does changes to the code that can break stuff, right now visible in one subsystem, but can basically break any part of the code. If I, as a unsuspecting developer want to apply style and I end up introducing issues, this is a problem.

It is clearly states above: because you won't be able to pass compliance checks in many cases.

Automatically applying code styles to an existing code should not have any side-effects. If you think the results of this rule in .clang-format should be enforced and should be make a rule that everyone follows, then it needs to clearly be part of the style guidelines of the zephyr project, right now it is not and it is not enforceable.

If shuffling headers has side effects on your code, that means there's a bug somewhere that needs to be fixed.

If shuffling headers has side effects on your code, that means there's a bug somewhere that needs to be fixed.

if there are bugs in the code, a code style tool should not be the tool used for this, there need to be a campaign of fixing those issues before it is enabled in a tool that has the sole puropse for applying style.

If shuffling headers has side effects on your code, that means there's a bug somewhere that needs to be fixed.

if there are bugs in the code, a code style tool should not be the tool used for this, there need to be a campaign of fixing those issues before it is enabled in a tool that has the sole puropse for applying style.

Include sorting can be considered part of a coding style, and so a formatter is the right place to have this setting (or is clang-format wrong?).

Include sorting can be considered part of a coding style, and so a formatter is the right place to have this setting (or is clang-format wrong?).

Yes but when it is announced and approved by community.
I fully support @nashif and his points outwheigh @gmarull's


Remove web stuff

It hasn't been updated in years and it clogs up grep results when grepping for function names.

Edited https://github.com/rizinorg/sdb/pull/9#issue-550263048.

Edited https://github.com/rizinorg/sdb/pull/9#issue-550263048.


Update blog-hero gradients + image

Sling has changed their gradients and the image in the top banner of the 1st level category pages. See https://www.sling.com/whatson/announcements for a new example.
Task: please update these gradients based on the live site the way they look in Mobile view, because I noticed you can't see the full gradient in wider screens. For example, we should capture the orange and black:

Also, the image is only "whats on", so new desktop + mobile images need to be used. See https://main--sling--aemsites.aem.page/whatson/announcements for example where I have updated it already, copy + paste to the others.
blog-hero.css

After looking at the new colors, let's do this.
/whatson = "red", DONE
/whatson/announcements = "cyan", UPDATE EXISTING VALUES (in progress)
/whatson/sports = "green", ADD NEW CLASS/STYLE
/whatson/entertainment = "dark-orange", ADD NEW CLASS/STYLE
/whaston/movies = "blue", UPDATE EXISTING VALUES
/whatson/international = "yellow", ADD NEW CLASS/STYLE
/whatson/freestream = "orange", ADD NEW CLASS/STYLE
You can leave the other colors in the css file for now as other options.

for yellow i have to go with more color points to match sling otherwise it was not at all matching. it was looking very dull compared to sling.


Enters math description as aria label.

Tested by @pkra.

LGTM.

Is speech generation activated?

Nevermind I see #85 is what I'm after :bow:


My binder badge fires tracking cookies

We recently incorporated mybinder badges into https://github.com/sphinx-gallery/sphinx-gallery. This is a great enhancement for us. But at the same time I noticed that the badges would not load on my browser.
Do you really need all this aggressive tracking of your users installed badges?

How could I test this? I just installed privacy badger and visited https://github.com/scikit-optimize/scikit-optimize which has a binder badge. As far as I can tell there was no alarm from the extension (green zero on the badger logo). Same for visiting the badge image.
Do I have to somehow configure the extension? (Only just installed it)
We have GA installed on mybinder.org but if you don't visit it you should be fine.

Yes you can test. We integrated a binder badge for every example in in Sphinx-Gallery docs. Take as an example:
https://sphinx-gallery.github.io/auto_examples/plot_seaborn.html#sphx-glr-auto-examples-plot-seaborn-py
Which is what I add from the screenshot.

You will see that privacy Badger says there is a potential tracker and associates it to mybinder.org, thus it blocks it.
I don't know why it shows up as a tracker in our docs website, but it doesn't say that it is a tracker when the page is on github(at least from the scikit-optimize website you just showed)

We have GA installed on mybinder.org but if you don't visit it you should be fine.

Indeed, you should not be tracked unless you visit the website. Also from what I see in privacy badger is mybinder.org that is considered the third party tracker not Google analytics

Some ideas why this happens:

visiting mybinder.org sets a cookie with GA=blah in it so I assume this is related to google analytics (my requests contained a DNT: 1 header), the domain of the cookie is mybinder.org
if I turn on "tracking protection" in Firefox the cookies don't get set
when I load the sphinx example when the cookies are set I get a warning like @Titan-C (for me it isn't automatically blocked, preference in privacy badger??)
when I load the sphinx example without the cookies set then there is no warning

We have some custom JS that tries to respect the DNT preference of the user. So in principle you should not end up with a cookie if DNT is enabled. However, in the webdev console the requests I send when visiting mybinder.org have a DNT: 1 header (even though in my FF preferences I did not enable DNT, "Send websites a “Do Not Track” signal that you don’t want to be tracked" is set to "Only when using Tracking Protection").
We use navigator.doNotTrack to check if DNT was enabled by the user. If I enable DNT explicitly then this is set to "1", if I only send  the DNT header then this is undefined. The MDN docs make me believe that it should be set to "1" when the DNT header is set to 1.
Action: should we check the request headers when deciding to include the GA JS?

For the scikit-optimize example I linked the badge doesn't actually get served by mybinder.org but by github.com. Which is why there is no tracking alert.

Thanks for reporting this because we do care about respecting people's privacy. Which is why we exclude the GA JS when we think the user doesn't want to get tracked but apparently the check is incomplete.

Action: should we check the request headers when deciding to include the GA JS?

Is this the main thing to be done here? Agree we should be respecting whatever privacy levels people specify

Thank you for reporting it!
I think the core problem here is that because mybinder.org is our website and the image is also being served from there, any cookies we set (now or in the future) will be sent along to image requests (if I understand it correctly) - making it 'tracking'.
I think the 'real' fix for this is to use a different domain for just serving the image. This should be fairly easy to set up, and should prevent any cookies we set to mybinder.org from being sent when requesting the image.
We can do this in mybinder.org-deploy by:

Adding an additional domain that just points to binderhub (static.mybinder.org, say)
Change the code that generates badges to point to images from static.mybinder.org rather than mybinder.org
(Optional) redirect mybinder.org/badge.svg to static.mybinder.org/badge.svg (if we want), but not required for original work.

This should separate our badge images from mybinder.org enough to satisfy all privacy-enabling addons I'm aware of. Note that switching to shields.io doesn't solve any of these problems, since they can already track you even more efficiently than we can :)
Long term I'd also like to move away from Google Analytics to a self hosted Piwik, but that's not anytime soon.
As others have said, we take privacy seriously. Thank you for reporting this, @Titan-C!


Customize VTS/Prometheus metrics

This PR add ability to customize the vts vhost key. The example below show how to count requests by grouped by server, path
Changes (breaking changes?)
The previous country var are renamed to key

Examples to customize
NGINX custom configuration ( http level )
  apiVersion: v1
  kind: ConfigMap
  data:
    enable-vts-status: "true"
    vts-default-filter-key: "$server_name"
...

Customize ingress, key by path (NGINX $uri)
 apiVersion: extensions/v1beta1
  kind: Ingress
  metadata:
    annotations:
      ingress.kubernetes.io/vts-filter-key: $uri $server_name
    name: ingress

Result

This change is

Coverage increased (+0.02%) to 44.226% when pulling 72b025915124d127a9e22aa43dcb438ea469343e on gianrubio:customize-vts-filter-key into b2ad9e7078a1ab0f843ec7962b80703efd15e978 on kubernetes:master.

Coverage increased (+0.04%) to 44.23% when pulling e37ea14e7eb4efaae065213be550c6e467c15502 on gianrubio:customize-vts-filter-key into 9126ec481368de5f2e21fb43d9763e50fb7d5816 on kubernetes:master.

Coverage increased (+0.04%) to 44.23% when pulling e37ea14e7eb4efaae065213be550c6e467c15502 on gianrubio:customize-vts-filter-key into 9126ec481368de5f2e21fb43d9763e50fb7d5816 on kubernetes:master.

/lgtm

@gianrubio thanks!


[Improvements] Superfluid Link should open in new page

The image isn't displaying and i'm not sure what this issue means. Could you clarify?

Okay, I understand. That will be out of scope for the MVP because it's a little bit complex to handle in React Native.
It is complex because React Native is made to work with Web, Android, iOS. Since Android and iOS apps don't have the intrinsic concept of tabs, React Native doesn't support opening a link in a new tab/window out of the box.

@L03TJ3

@krisbitney you can use the Link component from the native-base library
Example implementation here: https://github.com/GoodDollar/GoodWeb3-Mono/blob/6d4b2d1b9e4335cdeed84b896c2dd90676777e33/packages/good-design/src/core/web3/modals/RedirectModal.tsx#L17

@krisbitney sorry to ping you twice. Misclicked.
You can use the Link component from the native-base library for this.
Reference: https://docs.nativebase.io/link
Example implementation here: https://github.com/GoodDollar/GoodWeb3-Mono/blob/6d4b2d1b9e4335cdeed84b896c2dd90676777e33/packages/good-design/src/core/web3/modals/RedirectModal.tsx#L17

It is fixed in this PR: https://github.com/GoodDollar/GoodCollective/pull/117
Thanks @L03TJ3 . That component was already being used, but I realized there was a bug due to it being wrapped by a text element instead of the other way around. (i.e. it was <text><Link></Link></text> instead of the text being nested within the Link). Your example helped me see the issue.

Ah yeah now I see that link is used everywhere already :nerd_face:
but ok nice!
are you going to use that pr for other issues as well? or can I merge
@krisbitney

Ah yeah now I see that link is used everywhere already 🤓 but ok nice! are you going to use that pr for other issues as well? or can I merge @krisbitney

You can merge if you'd like

ye just cleaning up the board a bit.
Confirmed this is working so closing


WIP:Adding multiple dynamic clusters in a domain

The test added creates 2 dynamic clusters in a domain in inimage using WDT model.

All of the tests passed.
http://wls-jenkins.us.oracle.com/job/weblogic-kubernetes-operator-javatest/1673/consoleFull

update USECASES.md

In Jenkins all tests passed.
http://wls-jenkins.us.oracle.com/view/weblogic-operator/job/weblogic-kubernetes-operator-javatest/1717/consoleFull

All IT tests passed.
http://wls-jenkins.us.oracle.com/view/weblogic-operator/job/weblogic-kubernetes-operator-javatest/1725/consoleFull


Installation

Hi sir
How to installation this script?
Thanks

use composer

@jetonr thanks again

after installation what am i must doing ?
:~# use InstagramScraper\Instagram;
bash: use: command not found

where i must use use InstagramScraper\Instagram;  ??


Automatic rotation of cached JWKS

Automatic rotation of cached JWKS.

Implemented in https://github.com/Neoteroi/GuardPost/releases/tag/v1.0.1
No change in needed here, since that feature is automatically enabled and guardpost dependency is pinned with ~. Those who already have guardpost installed only need to upgrade it. pip install -U guardpost.


As an admin, I want to include book excerpts and articles as well as full volumes, so that I can include material that is specifically about prosody from longer works about other subjects.

dev notes
change digitized work db model as follows:

[x] add item type (full, excerpt, article); all current content should default to full
[x] add new fields for book/journal article, volume, page range (actual, for display/citation), image sequence range (for importing the correct portion of a volume from an external source)
[x] make new fields editable in admin
[ ] include new fields in the list view if appropriate (include type at minimum?)

@mnaydan what label should we use for the item type of current content, non-excerpts? volume? work? full? (It will only be visible to administrators, I don't think we're using this in the public interface anywhere)

@rlskoeser full makes most sense to me.

@mnaydan let me know if you have any ideas about the field order you'd like or which of the new fields you want on the list view. I was thinking of adding the new item type* field (full/article/excerpt), but it looks like the list view is already pretty full.  Is there anything you don't use that you want to drop? Or should I shorten the width for the date fields?
Also, let me know any preferences for the field order/organization on the edit page.  (If it's easier to think about order/layout after seeing an initial version, we can wait.)

Wondering if we can come up with a better name than "item type", which doesn't seem quite accurate (at least, not for "full"); extent, maybe? or does item type make sense to you?

@mnaydan I'm also wondering if the item type (full/article/excerpt) should be read only – I'm not sure if it will work to change the extent after an item has been indexed.... Although, I guess it if it does work then we might have a solution for the items you said that are already included in full and need to be converted to segments.  I'll leave it editable for now and we can test whether or not it works to change the status and page range of an existing record.  (But let me know if you have any opinions on this.)

@rlskoeser I don't think page count is essential to see on list view... we could swap that column out?
For placement: On list view, item type would make sense to me maybe between author and collections columns? And on detail view maybe toward the top after source URL (so that you're primed to know what you're looking at when you look at title and author info below). Do those placements make sense to you?
I agree that "item type" technically doesn't make sense for "full", but I think that's an intuitive label for me and Meredith at least. I can't think of anything better.

Thanks @mnaydan, your suggestions make sense to me, I'll try those out.
If item type makes sense to you two, then it works for me. How do you feel about relabeling "full" to "full volume" ? (If it doesn't bother you I'll leave it; you're the ones who will be working with it!)

@rlskoeser what about "full work"? "Volume" makes me nervous because of its use in enumcron and multivolume runs. Might just be me, though...

Yes, right — volume has other meanings, doesn't it! Full work makes sense to me.

@rlskoeser the majority of the article/book excerpt works are works that are currently contained in the PPA in full (maybe 300?). Would it be easier for you to take those works out of the PPA, then reinput them as articles/excerpts whose status and page range is uneditable? Even being able to do just that, since I have all the work done already in a spreadsheet, would be hugely helpful and totally change the PPA (for the better). Capacity for adding new excerpts NOT currently in the PPA will be primarily for rolling copyright items via small-batch each year.
It would be nice for it to be editable I suppose (in case there's a mistake we need to correct for instance, or we decide we actually want the full work or actually want just an excerpt). But not absolutely essential if we don't have time. It sounds like, though, testing to see whether it works or not at this point would be helpful?

the majority of the article/book excerpt works are works that are currently contained in the PPA in full (maybe 300?).

I was not aware of this! I knew you had some but I wasn't clear on the scope. This is helpful to know. I'll think about what it means in terms of making the page range editable and adjusting item status for content that's already in the database — it might actually give us a way to test and implement part of the excerpt functionality before building out full import support for excerpts.

@mnaydan do you want any validation on the new fields? e.g., should we not allow type = full work if there are pages specified, should we require book/journal title if not a full work? I don't know how much you'll edit this in the admin so I'm not sure how much impact there it to not having it

@rlskoeser hm. Is the initial import/metadata corrections going to happen en masse via a CSV upload, or will we be fixing the metadata record by record? If we're fixing record by record, it might be important to have the validation scenarios you mention.

@mnaydan you have enough of them that we should do the initial import and correction in bulk, but I'm planning to give you a chance to preview/test the excerpt functionality by editing records individually so you can see how it's working so far.

@rlskoeser gotcha, sounds great! I don't think the validation is essential in that case if it would simplify things for you.

@mnaydan thank you for thinking this through with me! Yes, it would be some additional work to implement and test the validation, so I'm happy to skip it. We can always add it later if we discover it is important.

Tested with chi.19606141 for single range article. Tested with ucm.5326809190 for single range and multirange book excerpt. Tested with hvd.32044098641343 for article with multiple ranges. Everything works as expected, and item type shows on list view & defaults to full.
Invalid page ranges are correctly rejected for non-numeric and for out of sequence. One thing it doesn't catch is page ranges outside of the possible page ranges of the volume. For instance, hvd.32044103001129 has HT pages 1-448. If you put in 500-588, it allows you to save it (but yields no keyword search results, since there's nothing to search! Which makes sense). I don't think this is super important to validate, necessarily, if there's no time -- I trust that most of our data work is accurate, but I thought I'd note it. Would like your opinion on this @rlskoeser but then we can close if you think it's fine.

I think it would be hard to validate if the page ranges are outside the actual range, since we only have those numbers once we've calculated the total number of pages (the total isn't provided in the metadata). And only actual pages that exist that are in range would be included in the content that's indexed, so I think it would only result in an inaccurate page count — which doesn't seem terrible to me. (And also probably not very likely to happen)

Thanks for thinking through it with me! Agreed & makes sense to me.


[WIP] New websocket-to-socket proxy

I reimplemented the transmitter thread using nio4r as a separate gem. This is a generic socket-to-socket proxy implemented using a lazy-threaded event loop and in the future it will be used instead of socat and in other projects too.
To maintain the compatibility between websockets and the Nio4r::Selector, it was necessary to rewrite the wrapper responsible for WebSocket en/decapsulation. As the websocket-driver is using an event-based approach of emitting decapsulated websocket frames, I had to drop it and use the more flat websocket gem.
The wrapper itself has been implemented by redefining the receive_nonblock and send_nonblock methods of the socket hijacked from Rack. Maybe it's ugly and I want to apologise in advance :beers: @Fryguy :beers:, but after thinking it through it seemed as the best idea.
@martinpovolny @jrafanie what do you guys think about this?

@skateman : it looks good to me, but I think that you should spend some time figuring some way to stress-test this. I'd love to prevent hitting into some edge cases in production. Maybe you can script putting this on an appliance and then script some intense client activity. Some cmd-line websocket client? Writing some javascript and run it from several browsers?

@martinpovolny I'm continuously doing the intense testing for the external gem but for now manually only. It would be nice to have some automated benchmarks. However, there is still an issue with SSL sockets when using SPICE console, that's why the PR is marked as WIP.

@skateman : what's the status of this PR?
Thx!

@martinpovolny stalled on the external gem, but it's in my queue.

Closed in favor of https://github.com/ManageIQ/manageiq/pull/16490


Update PluginToolbarButton.yaml

Changes
Added additional information for developers who need to track the PluginToolbarButton's active state in cases where the button's Enabled property is being toggled on and off.
Checks
By submitting your pull request for review, you agree to the following:

[X] This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
[X] I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
[X] To the best of my knowledge, all proposed changes are accurate.

Anime game


fix(node_exporter): correct indent in systemd template

Before:

After:

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@e66e609). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff            @@
##             main      #163   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         1
  Lines           ?        42
  Branches        ?         4
========================================
  Hits            ?        42
  Misses          ?         0
  Partials        ?         0

Flag
Coverage Δ

integration
100.00% <0.00%> (?)

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

Looks like this breaks the startup of the alternative test. The exporter fails to run.


📝 Make darkmode logo work on github

This PR makes the new dark mode logo introduced in https://github.com/tiangolo/typer/pull/907 available in the GitHub README by using <picture> conditionally.
Note: This got quite ugly, because mkdocs material allows to override the system setting for darkmode, while on github we can only use the media query. Feel free to ignore and close.

Thanks for the PR, @pypae! We'll review this and get back to you 🙏

@tiangolo : I think it's probably better if you review this one directly 🙏

Thanks @pypae! I tried a looot of things to make it compatible with both MkDocs and GitHub :sweat_smile:
As I want to be able to keep both files in sync.
For now, I would prefer to leave it as is, but thanks for the effort! :coffee:


Debian / ruby2.5 / mail-gpg 0.3.2: Failing tests

Hi,
While importing the last release into Debian, I did run into the following, which blocks the upload currently:
Loaded suite /usr/lib/ruby/vendor_ruby/rake/rake_test_loader
Started
DEPRECATION WARNING: `#deliver` is deprecated and will be removed in Rails 5. Use `#deliver_now` to deliver immediately or `#deliver_later` to deliver through Active Job. (called from block (3 levels) in <class:ActionMailerTest> at /<<PKGBUILDDIR>>/test/action_mailer_test.rb:117)
.DEPRECATION WARNING: `#deliver` is deprecated and will be removed in Rails 5. Use `#deliver_now` to deliver immediately or `#deliver_later` to deliver through Active Job. (called from block (3 levels) in <class:ActionMailerTest> at /<<PKGBUILDDIR>>/test/action_mailer_test.rb:89)
/<<PKGBUILDDIR>>/lib/mail/gpg/message_patch.rb:41: warning: instance variable @gpg not initialized
.DEPRECATION WARNING: `#deliver` is deprecated and will be removed in Rails 5. Use `#deliver_now` to deliver immediately or `#deliver_later` to deliver through Active Job. (called from block (3 levels) in <class:ActionMailerTest> at /<<PKGBUILDDIR>>/test/action_mailer_test.rb:102)
.DEPRECATION WARNING: `#deliver` is deprecated and will be removed in Rails 5. Use `#deliver_now` to deliver immediately or `#deliver_later` to deliver through Active Job. (called from block (3 levels) in <class:ActionMailerTest> at /<<PKGBUILDDIR>>/test/action_mailer_test.rb:53)
.DEPRECATION WARNING: `#deliver` is deprecated and will be removed in Rails 5. Use `#deliver_now` to deliver immediately or `#deliver_later` to deliver through Active Job. (called from block (3 levels) in <class:ActionMailerTest> at /<<PKGBUILDDIR>>/test/action_mailer_test.rb:67)
.DEPRECATION WARNING: `#deliver` is deprecated and will be removed in Rails 5. Use `#deliver_now` to deliver immediately or `#deliver_later` to deliver through Active Job. (called from block (3 levels) in <class:ActionMailerTest> at /<<PKGBUILDDIR>>/test/action_mailer_test.rb:43)
/<<PKGBUILDDIR>>/lib/mail/gpg/message_patch.rb:41: warning: instance variable @gpg not initialized
.E
===============================================================================
Error: test: DecryptedPart should decrypt and verify. (DecryptedPartTest): NoMethodError: undefined method `size' for nil:NilClass
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:132:in `keys_for_data'
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:14:in `encrypt'
/<<PKGBUILDDIR>>/lib/mail/gpg/encrypted_part.rb:20:in `initialize'
/<<PKGBUILDDIR>>/test/decrypted_part_test.rb:14:in `new'
/<<PKGBUILDDIR>>/test/decrypted_part_test.rb:14:in `block (2 levels) in <class:DecryptedPartTest>'
===============================================================================
E
===============================================================================
Error: test: DecryptedPart should decrypt. (DecryptedPartTest): NoMethodError: undefined method `size' for nil:NilClass
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:132:in `keys_for_data'
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:14:in `encrypt'
/<<PKGBUILDDIR>>/lib/mail/gpg/encrypted_part.rb:20:in `initialize'
/<<PKGBUILDDIR>>/test/decrypted_part_test.rb:14:in `new'
/<<PKGBUILDDIR>>/test/decrypted_part_test.rb:14:in `block (2 levels) in <class:DecryptedPartTest>'
===============================================================================
E
===============================================================================
Error: test: DecryptedPart should raise encoding error for non gpg mime type. (DecryptedPartTest): NoMethodError: undefined method `size' for nil:NilClass
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:132:in `keys_for_data'
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:14:in `encrypt'
/<<PKGBUILDDIR>>/lib/mail/gpg/encrypted_part.rb:20:in `initialize'
/<<PKGBUILDDIR>>/test/decrypted_part_test.rb:14:in `new'
/<<PKGBUILDDIR>>/test/decrypted_part_test.rb:14:in `block (2 levels) in <class:DecryptedPartTest>'
===============================================================================
E
===============================================================================
Error: test: EncryptedPart with email address should resolve email to gpg keys. (EncryptedPartTest): NoMethodError: undefined method `size' for nil:NilClass
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:132:in `keys_for_data'
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:14:in `encrypt'
/<<PKGBUILDDIR>>/lib/mail/gpg/encrypted_part.rb:20:in `initialize'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `new'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `block (2 levels) in <class:EncryptedPartTest>'
===============================================================================
E
===============================================================================
Error: test: EncryptedPart with email address should resolve emails to gpg keys. (EncryptedPartTest): NoMethodError: undefined method `size' for nil:NilClass
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:132:in `keys_for_data'
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:14:in `encrypt'
/<<PKGBUILDDIR>>/lib/mail/gpg/encrypted_part.rb:20:in `initialize'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `new'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `block (2 levels) in <class:EncryptedPartTest>'
===============================================================================
E
===============================================================================
Error: test: EncryptedPart with emails and key data should resolve to gpg keys. (EncryptedPartTest): NoMethodError: undefined method `size' for nil:NilClass
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:132:in `keys_for_data'
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:14:in `encrypt'
/<<PKGBUILDDIR>>/lib/mail/gpg/encrypted_part.rb:20:in `initialize'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `new'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `block (2 levels) in <class:EncryptedPartTest>'
===============================================================================
E
===============================================================================
Error: test: EncryptedPart with key fingerprint should resolve id list to gpg keys. (EncryptedPartTest): NoMethodError: undefined method `size' for nil:NilClass
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:132:in `keys_for_data'
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:14:in `encrypt'
/<<PKGBUILDDIR>>/lib/mail/gpg/encrypted_part.rb:20:in `initialize'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `new'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `block (2 levels) in <class:EncryptedPartTest>'
===============================================================================
E
===============================================================================
Error: test: EncryptedPart with key fingerprint should resolve single id  gpg keys. (EncryptedPartTest): NoMethodError: undefined method `size' for nil:NilClass
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:132:in `keys_for_data'
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:14:in `encrypt'
/<<PKGBUILDDIR>>/lib/mail/gpg/encrypted_part.rb:20:in `initialize'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `new'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `block (2 levels) in <class:EncryptedPartTest>'
===============================================================================
E
===============================================================================
Error: test: EncryptedPart with key id should resolve id list to gpg keys. (EncryptedPartTest): NoMethodError: undefined method `size' for nil:NilClass
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:132:in `keys_for_data'
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:14:in `encrypt'
/<<PKGBUILDDIR>>/lib/mail/gpg/encrypted_part.rb:20:in `initialize'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `new'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `block (2 levels) in <class:EncryptedPartTest>'
===============================================================================
E
===============================================================================
Error: test: EncryptedPart with key id should resolve single id  gpg keys. (EncryptedPartTest): NoMethodError: undefined method `size' for nil:NilClass
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:132:in `keys_for_data'
/<<PKGBUILDDIR>>/lib/mail/gpg/gpgme_helper.rb:14:in `encrypt'
/<<PKGBUILDDIR>>/lib/mail/gpg/encrypted_part.rb:20:in `initialize'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `new'
/<<PKGBUILDDIR>>/test/encrypted_part_test.rb:20:in `block (2 levels) in <class:EncryptedPartTest>'
===============================================================================
......................................................../<<PKGBUILDDIR>>/lib/mail/gpg/verify_result_attribute.rb:10: warning: instance variable @verify_result not initialized
.......................
.........

Finished in 25.665790299 seconds.
------
104 tests, 471 assertions, 0 failures, 10 errors, 0 pendings, 0 omissions, 0 notifications
90.3846% passed
------
4.05 tests/s, 18.35 assertions/s
rake aborted!

Please note: Debian switched recently to ruby2.5, not sure if this could be related. Also, the deprecation warnings aren't a problem (currently), I'm more afraid of the failing tests. Any advice regarding this?
Thanks for your work,
cheers,
Georg

Thank you, I indeed messed that up, these errors are totally unrelated to Ruby 2.5. My local testing environment is currently broken for the same reasons travis CI isn't working right now, making my workflow a bit complicated. I just pushed out 0.3.3

Thanks a lot, highly appreciated, works! :) Upload upcoming...


Investigate an observable internalOptions prop on hoist dropdown fields

When dynamically changing the option underneath one these fields we expect to see the options rerender in the field's menu. This seems to work as expected when using the @observable.ref decorator on set of options we pass in as props, and then reassigning that array to change it's value. However, if we use just @observable and then push to that array we are seeing the outer component that the field is nested in rerender, but not the internal field component itself.
While using the ref for our props is perfectly acceptable and somewhat more clear, there is a concern that we are relying on some implementation detail in mobx, and cannot rely on this always being the case. We should therefore look to somehow rerender these fields with certainty.

At very least we need to declare the internalOptions prop on the basedropdown class

Thanks for filing, Don!
I think it is important that the internalOptions property be observable to ensure that the component re-renders whenever those internal options are changed (as a result of the options passed via props changing, in any way).
Currently the render method of the parent (the container specifying the selectField) can run if those outer options are updated (i.e. pushed onto). but that alone doesn't trigger a re-run of the selectField's own render method. Unless the entire options collection has changed, the component hasn't received any new/different props - it's the same collection, and nothing deep is happening by default at the other level.
The autorun setup internally by selectField will run, updating internalOptions correctly, but without render running it won't reflect changes to those opts.
SelectField does not access its props.options object within its internal method, and even on the individual item level, normalizeOptions strips out the MobX proxy objects that power the "deep" change detection on the individual option (object) level.  Only by making internalOptions observable do we get that change detection back on the data structure that's actually accessed within render().
It might still be a best practice to replace the entire options collection and/or avoid mutating the options directly on the individual object level.  But I don't think we should require that...

Believe this is the minimal incremental change here, based off of the current develop branch.  The change to call the setInternalOptions action method would need to be made in the multiSelect control we have in another active branch.
While making this change, I was wondering if we want this multi-level object hierarchy for dropdown fields. I think it's confusing things more than it might be helping, especially as QueryComboField doesn't follow the same patterns, and we have this awkward naming around dropdown vs. combo base classes (I can never get it in my head which is which).  It would be a distinct ticket, for sure, but I would vote for looking at that change and seeing how much code duplication it would entail vs. allowing a flatter and simpler hierarchy.

I am fine with the idea of flattening the dropdown field hierarchy

And thanks for the analysis Anselm.  Don -- just want to make sure its clear-- changing the options reference passed, actually changes the prop value itself, which is what was causing the re-render when you change the ref, according to the standard react conventions.  The @observable.ref marking on the options passed in was actually doing nothing....

Ah yes that makes perfect sense. Sorry I missed that fact


Fix serenity next breakage due to dm-permission being removed in serenity next

Title

This removes the guild_only parameter entirely, which is not the solution and will break bots at runtime. I will fix this when I have time and energy, but for now you can pin to the serenity commit before the breakage. A reminder that this is what you get when using unstable branches, they break and have no guarantees for even compiling.


[redux-actions v2.x.x] Multiple typed arguments for payload creator not supported

I seems that the libdef for redux-actions v2.x.x don't support multiple typed arguments for payload creators. Here is an example flow.org/try/#... which shows the error.
The current tests are not failing because they don't have type definitions for the arguments: definitions/npm/redux-actions_v2.x.x/flow_v0.34.x-/test_createAction.js#L62.
I tried to fix it by myself (#935) but I don't have enough experience in libdefs - especially generics for rest parameters.

I fixed it now. Would be nice if someone could review it: https://github.com/flowtype/flow-typed/pull/935.

Closing since fixed.


pjax cant loading Hermit X after updata

please provide your domain.
@DIYgod

@liwanglin12 加上 issue 规范吧：https://github.com/DIYgod/APlayer/issues/new

@DIYgod 你是contributor不能加么

cant use when Load the page second time. u blog also like this

@23330 Issue confirm. Please wait.

Please post your error information form the console.
And please provide your website's address if it's possible.

lwl12.com blog also like this .cant Load when Load the page second time.
fist load is normal

fixed. https://github.com/liwanglin12/Hermit-X/pull/36

thx


Sometimes it reports error "Something went wrong! FN is negative". Why does it happen?

Hello~Thanks for your code.It's very nice. When using it , I encountered the above problems.
I saw the source  code,“tmpfn += len(g) - num_associations”，this line，but I don't know why sometimes len(g) is less than num_associations. I want to ask that under what circumstances can it happen. It shouldn't have happened in theory.

Maybe it is because gg may match more than one mask in tt? Can I add a "break" at end of this loop, which means gg only match the first mask meeting in tt.

There should be only one predicted mask having IoU above 0.5 with a ground truth mask if all predicted masks are non-overlapping. How are you calling compute_MOTS_metrics_per_sequence? Are you calling it via eval.py?

Closing this for now, please reopen if needed


[UpdatePlugin]  murx- devi 0.2

Please update my plugin:
Author: Me
Repo name: devi
Release: https://github.com/murx-/devi_binja/releases/tag/0.2

Updated, thanks!


Migration to TypeScript

Hi, I've been noticing runtime errors and regressions that I believe can be prevented to a big extent by migrating this project to typescript and introducing strong-typing everywhere. What do you guys think? Would be happy to contribute.

@ajmalab, we have started supporting ESM and deno. cdxgen could be used as a library with strongly typed wrapper functions in the future. In terms of contribution, we're happy to receive sponsorship or time to work on issues that are labelled Consider Funding.


fix: [#9151] LG for 'choices' field in multi choice input results in Warning

Description
This PR fixes the validation of LG expressions to avoid showing a warning when LG expressions are used.
A trimExpression function was added to remove the ${} characters of the expression allowing it to be correctly parsed.
It also adds a unit test to cover this new function.
Task Item
Fixes #9151
Screenshots
These images show the warning displayed before and the LG expression being properly validated after.
BEFORE

AFTER

Coverage increased (+0.01%) to 54.662% when pulling c448cc1052840de371189bd16caeca3c0c173220 on southworks/fix/lg-expression-validation into a27c3e55332ee6b5a63619f375a5b2fd127923f4 on main.

Should be good to go after related package lock hash updates


add new term: alveolar capillary dysplasia without misalignment

revised label for MONDO_0009934 'alveolar capillary dysplasia with misalignment of pulmonary veins'

Open questions:
@mgriese

would it make sense to create a grouping class for these two terms, eg alveolar capillary dysplasia? Is this term a more general term that could group the with and without misalignment terms together?  https://www.ebi.ac.uk/ols/ontologies/ncit/terms?iri=http%3A%2F%2Fpurl.obolibrary.org%2Fobo%2FNCIT_C98809
Could you help us with the text definition for the without misalignment term? We current have:
A rare form of interstitial and vascular lung disease that presents as severe pulmonary hypertension and refractory hypoxemia early in life.

Thank you!
cc @kallia-p

related to https://github.com/monarch-initiative/mondo/pull/3147

@nicolevasilevsky @mgriese In Pubmed I found only one study reporting ACD without misalignment of the pulmonary veins PMID:9514134.

OK, we have several cases; is difficult to publish without genetics; would just list as a term (with the citation above)

There may be more in pubmed after quick check: 33766794,

Regarding the question above: as I am not a pathologist expert in this field,  I am not able to provide a good histopathologic definition. The definition you gave is fine.

too many conflicts, will redo


Missing objcopy on Miniforge3 Docker

[x] I read the conda-forge documentation and could not find the solution for my problem there.

Issue
Running PyInstaller on condaforge/miniforge3:4.10.3-1 image fails to generate functional binaries. This is due to the assumption that linux-64 platform includes the objcopy command supplied by binutils. This could be remedied by adding binutils as a dependency.
Recreate Steps
## launch docker
docker run --rm -it condaforge/miniforge3:4.10.3-1

## within docker BASH session
conda create -yn pi python=3.9 pyinstaller

## activate 'pi' env
conda activate pi

## create dummy script
echo "import sys; print(sys.path)" > script.py

## test
pyinstaller -F script.py

Result
Throws error message:
FileNotFoundError: [Errno 2] No such file or directory: 'objcopy'

Full Output

172 INFO: PyInstaller: 4.3
172 INFO: Python: 3.9.6 (conda)
175 INFO: Platform: Linux-5.10.25-linuxkit-x86_64-with-glibc2.31
176 INFO: wrote /script.spec
178 INFO: UPX is not available.
180 INFO: Extending PYTHONPATH with paths
['/', '/']
183 INFO: checking Analysis
183 INFO: Building Analysis because Analysis-00.toc is non existent
183 INFO: Initializing module dependency graph...
184 INFO: Caching module graph hooks...
190 INFO: Analyzing base_library.zip ...
2091 INFO: Processing pre-find module path hook distutils from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks/pre_find_module_path/hook-distutils.py'.
2092 INFO: distutils: retargeting to non-venv dir '/opt/conda/envs/pi/lib/python3.9'
4405 INFO: Caching module dependency graph...
4560 INFO: running Analysis Analysis-00.toc
4570 INFO: Analyzing /script.py
4571 INFO: Processing module hooks...
4572 INFO: Loading module hook 'hook-pickle.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
4574 INFO: Loading module hook 'hook-multiprocessing.util.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
4575 INFO: Loading module hook 'hook-difflib.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
4577 INFO: Loading module hook 'hook-sysconfig.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
4583 INFO: Loading module hook 'hook-lib2to3.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
4599 INFO: Loading module hook 'hook-_tkinter.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/opt/conda/envs/pi/lib/python3.9/tkinter/__init__.py", line 37, in <module>
    import _tkinter # If this fails your Python may not be configured for Tk
ImportError: libX11.so.6: cannot open shared object file: No such file or directory
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: libX11.so.6: cannot open shared object file: No such file or directory
4643 ERROR: Tcl/Tk improperly installed on this system.
4644 INFO: Loading module hook 'hook-distutils.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
4644 INFO: Loading module hook 'hook-xml.etree.cElementTree.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
4645 INFO: Loading module hook 'hook-xml.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
4683 INFO: Loading module hook 'hook-heapq.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
4685 INFO: Loading module hook 'hook-encodings.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
4725 INFO: Loading module hook 'hook-distutils.util.py' from '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks'...
4739 INFO: Looking for ctypes DLLs
4770 INFO: Analyzing run-time hooks ...
4774 INFO: Including run-time hook '/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/hooks/rthooks/pyi_rth_multiprocessing.py'
4782 INFO: Looking for dynamic libraries
5205 INFO: Looking for eggs
5205 INFO: Python library not in binary dependencies. Doing additional searching...
5214 INFO: Using Python library /opt/conda/envs/pi/lib/libpython3.9.so.1.0
5219 INFO: Warnings written to /build/script/warn-script.txt
5244 INFO: Graph cross-reference written to /build/script/xref-script.html
5255 INFO: checking PYZ
5255 INFO: Building PYZ because PYZ-00.toc is non existent
5255 INFO: Building PYZ (ZlibArchive) /build/script/PYZ-00.pyz
5592 INFO: Building PYZ (ZlibArchive) /build/script/PYZ-00.pyz completed successfully.
5596 INFO: checking PKG
5597 INFO: Building PKG because PKG-00.toc is non existent
5597 INFO: Building PKG (CArchive) PKG-00.pkg
13615 INFO: Building PKG (CArchive) PKG-00.pkg completed successfully.
13617 INFO: Bootloader /opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/bootloader/Linux-64bit/run
13618 INFO: checking EXE
13618 INFO: Building EXE because EXE-00.toc is non existent
13618 INFO: Building EXE from EXE-00.toc
13618 INFO: Appending archive to ELF section in EXE /dist/script
Traceback (most recent call last):
  File "/opt/conda/envs/pi/bin/pyinstaller", line 11, in <module>
    sys.exit(run())
  File "/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/__main__.py", line 114, in run
    run_build(pyi_config, spec_file, **vars(args))
  File "/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/__main__.py", line 65, in run_build
    PyInstaller.building.build_main.main(pyi_config, spec_file, **kwargs)
  File "/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/building/build_main.py", line 737, in main
    build(specfile, kw.get('distpath'), kw.get('workpath'), kw.get('clean_build'))
  File "/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/building/build_main.py", line 684, in build
    exec(code, spec_namespace)
  File "/script.spec", line 21, in <module>
    exe = EXE(pyz,
  File "/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/building/api.py", line 450, in __init__
    self.__postinit__()
  File "/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/building/datastruct.py", line 160, in __postinit__
    self.assemble()
  File "/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/building/api.py", line 616, in assemble
    retcode, stdout, stderr = exec_command_all(
  File "/opt/conda/envs/pi/lib/python3.9/site-packages/PyInstaller/compat.py", line 461, in exec_command_all
    proc = subprocess.Popen(cmdargs, bufsize=-1,  # Default OS buffer size.
  File "/opt/conda/envs/pi/lib/python3.9/subprocess.py", line 951, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/opt/conda/envs/pi/lib/python3.9/subprocess.py", line 1821, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'objcopy'

Expected Result
It should successfully build the binary.
Workaround
The error can be overcome by installing binutils. That is, running the above recreate steps, but with
conda create -yn pi python=3.9 pyinstaller binutils

enables error-free running of the last step.
Recommended Solution
The feedstock maintainers should consider including binutils as a runtime dependency, at least for linux-64 builds. The recipe should include a end-to-end test that would catch this failure in future builds.

I'm hesitant to do this as it would likely cause issues for packages building in conda-forge relying on pyinstaller. This is because they normally would be getting build utilities (like the compiler and binutils) from the build environment. Including this in pyinstaller would pull this in the host environment. This would then likely lead to clashes between these two environments.
Given this, is there a reason one can't install this themselves or grab it from the system?

@jakirkham thanks for the reply. Yes, one can simply install the binutils to get things working. I created this issue partially to document that workaround for others, partially to spark a discussion as to adding it as an explicit dependency.
If there are non-trivial consequences to the latter, then it sounds better to leave it as is.

Other ways to consider: either vendor a copy of objcopy and patch pyinstaller to use that or urge the developers of pyinstaller to switch to a solution that doesn't require dependency on objcopy (using some ELF library like py-lief or pyelftools). All objcopy is used for is this.

Turns there is already a fallback in place, the path just doesn't get hit. Let's see what upstream devs have to say about https://github.com/pyinstaller/pyinstaller/pull/5991


[Form Recognizer] update readme and troubleshooting

Packages impacted by this PR
@azure/form-recognizer
Issues associated with this PR
#23397
Describe the problem that is addressed by this PR
Adds Troubleshooting guide
Provide a list of related PRs (if any)
Refer #23397

API change check
API changes are not detected in this pull request.


feature_extractor.py

Describe the bug
A  error occurred when I followed the instructions in the readme file to extract features of the ucf-crime dataset.
ERROR：
feature_extractor.py", line 108, in to_segments
temp_vect = data[min(ss, data.shape[0] - 1), :]
IndexError: too many indices for array: array is 1-dimensional, but 2 were indexed

Sorry, I can't download the dataset because the download failed for me; thus, I can't debug it ATM. Where have you downloaded it from?

https://www.dropbox.com/sh/75v5ehq4cdg5g5g/AABvnJSwZI7zXb8_myBA0CLHa?dl=0


Installing as project dependency asks for Ionic version

Version
4.1.3
Browser and OS info
Any / macOS 10.13.3
Steps to reproduce
Install @vue/devtools as devDependency with yarn (yarn add --dev @vue/devtools)
What is expected?
@vue/devtools should be installed with no issues
What is actually happening?
Yarn shows a warning message: Couldn't find any versions for "ionic" that matches "beta". Please choose a version of "ionic" from this list:
Then it just don't install at all, stuck at Building fresh packages. waiting...

Following the Electron installation docs: https://github.com/vuejs/vue-devtools/blob/master/shells/electron/README.md

I think we don't have ionic declared as a dependency anywhere in the devtools.

@Akryum Yep.. That's the weird thing. I did a fresh yarn install before testing this, and running yarn add --dev @vue/devtools asks for the Ionic version. Perhaps a sub-dependency of a direct dependecy?

@Akryum Sorry to bother you guys with this, but I managed to solve this removing ionic from my globally installed packages. It looks like ionic installation broke for some reason. Removing it solved the issue.


Sprint19 time sol pde

closes #206

@jakobsj, @nabriis
I appreciate it if you can take a look at this PR. One point to consider, if the data files are large, I can update the tests to require less data. Many thanks!

Thank you @nabriis for your review! Good point about the term 'final', I made an update to address that.

@nabriis, just to update you I added extra check to make sure to catch the error when the user passes words other that 'final' or 'all' for the value of time_obs argument

Thank you @jakobsj for your review.

Thank you, this looks great, I was particularly pleased with the helpful comments throughout such as for explaining whether to interpolate.

Thank you for your review!


filters don't update immediately in pathway view

If I'm looking at for instance
https://www.patricbrc.org/view/GenomeList/?in(genome_id,(1434164.3,1434165.3,1434163.3))#view_tab=pathways&filter=and(eq(annotation,%22PATRIC%22),or(eq(pathway_class,%22Xenobiotics%20Biodegradation%20and%20Metabolism%22),eq(pathway_class,%22Carbohydrate%20Metabolism%22)))
and I open the filters and change the pathways being filtered, the list does not update until I reload the page.
version 3.2.84

I think I have found the reason for this issue and I would like to "assign" this to myself. I still don't have permission to do anything on the right panel, e.g., assign the issue to myself, add labels, etc.

I think I have fixed it and this is part of pull request #409 now.

tested on alpha.


[KBFS-1515] Pull out Codec into its own package

This is needed before pulling out anything
else into its own package.
Rename functions slightly.
Adjust spacing.

The meat is in kbfscodec, and everything else is just renaming and space adjustments.

Yeap, will send out a server PR soon.

Server PR is https://github.com/keybase/kbfs-server/pull/127


Xamarin IOS failed to install the package

I would appreciate your kind help. I have tried to install your package using Xamarin and it did work fine for Android. But when I have tried to install the package for iOS as per your instruction guideline I have encountered 2 issues.

Error Could not find a part of the path 'C:\Users...t\packages\xamarin.firebase.ios.core\<IP_ADDRESS>\lib\xamarinios10\Firebase.Core.resources\GoogleUtilitiesComponents.xcframework\ios-arm64_i386_x86_64-simulator\GoogleUtilitiesComponents.framework\PrivateHeaders\GULCCComponentContainerInternal.h'.
( I have solved this issue by adding NUGET_PACKAGES and NUGET_PATH to the system environemnt to C:\nu)
After installing the package BarcodeScanner.Mobile.XamarinForms it shows installed but it is not refelected inside the references inside the project (note: it causes other issues as when I try to install other packages they fail afterwards).
When I tried to add  BarcodeScanner.Mobile.XamarinForms.iOS.Initializer.Init(); to the AppDelegate it shows The name 'BarcodeScanner' does not exist in the current context

I am using Visual Studio 2022 17.3

It worked when I have used the console manager directly Install-Package BarcodeScanner.Mobile.XamarinForms instead of the Nuget Manager


HTTP -> HTTPS redirection based on X-Forwarded-Proto Header

Do you want to request a feature or report a bug?
Feature
What did you expect to see?
When running Traefik behind a load balancer that performs SSL termination, I want to be able to send a redirect from HTTP to HTTPS.
The current middleware RedirectScheme uses the schema to send a redirect. With a load balancer that uses SSL termination, the schema, however, is always HTTP for traefik. To indicate whether it originally was an HTTP or an HTTPS request, many load balancers set the X-Forwarded-Proto header accordingly.
Doing the redirection at the load balancer itself is not always supported. The GCP HTTP load balancer, for example, does not support it. IMO it makes sense to have a Middleware that sends redirects based on the X-Forwarded-Proto header.

Hello,
I answered to this issue on the forum:
https://community.containo.us/t/http-https-redirection-based-on-x-forwarded-proto-header/2908/4


Update tools-deps variants to <IP_ADDRESS>5

Routine upstream version bump. Will merge when CI is green.

Official images PR: https://github.com/docker-library/official-images/pull/15999


Must the Application Gateway and the Cluster be in the same resource group?

My existing Application Gateway and AKS clusters are in different resource groups.
If I enable AGIC via the portal as per the link below  my gateway is not listed.
https://learn.microsoft.com/en-us/azure/application-gateway/tutorial-ingress-controller-add-on-existing#enable-the-agic-add-on-in-existing-aks-cluster-through-azure-portal
Please confirm whether this can be configured with the cluster and gateway in different resource groups or whether they need to be in the same resource group.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 4ce7e4ca-0ac4-9d41-9546-b8f5e7ee654e
Version Independent ID: 476b4dc9-46e3-71eb-e08e-67137004b804
Content: What is Azure Application Gateway Ingress Controller?
Content Source: articles/application-gateway/ingress-controller-overview.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@crowne
Thanks for your feedback! We will investigate and update as appropriate.

@crowne
I have responded to your query here. Let's continue our discussion there. Closing this as duplicate. Thank you!


Create Pullrequest from tags

Gitea version (or commit ref):  1.10.0+dev-142-g7ca5dd0
Git version: latest
Operating system: Ubuntu
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Description
We have the usecase to create Pullrequests from available tags
via GIT CLI its the same as for branches
git checkout destination_branch
git merge your_tag
However gitea does not show any tags within the pullrequest control
Do I miss something?

I think even github didn't support this.

@lunny GitHub has this functionality

@lunny
on gate you can create a comparison between branches and tags already, with a workaround by editing the url in the compare-window via replacing the target-branch with the tag you like to merge.
Unfortually the "Create PullRequest"-Button is not there :(

for example
https://try.gitea.io/Butcher/test-repo2/compare/master...<IP_ADDRESS>
with <IP_ADDRESS> is a tag

As far as my understanding is correct, Gitea makes a difference between a branch-head and a tag.
on git level its just a different prefix (ref/head vs ref/tag)
Do get this running, gitea has to threat a tag and a branch-head equally

@lunny  I added a basic tag-pull-request-functionality on my fork:
https://github.com/sharpSteff/gitea
Just added all tags to the branch-selection-control, because I'm not really familier with this go-templates.
I'll try the forked version with my team to see any compatibility problems with old PRs, etc

@sharpSteff, see PR #15723.  It doesn't enable creating a pull request, but it lets you select tags to compare.  So once this gets reviewed, you may want to merge it into your fork.
I also confirmed that GitHub does not currently have a UI to create a pull request from a tag.  It's probably just not a very common workflow.


Canonicalize include paths before emitting

This allows CARGO_MANIFEST_DIR to be a relative path, which can be useful in non-cargo build systems.

Cherry-picked from https://github.com/djc/askama/pull/1107.

Drawback: Before this PR, the crate root could reside in a non-UTF-8 path without issues, any only everything inside it had to have a sane encoding. With the PR, the whole path has to be sane.
But I guess that's okay. As a user, I don't think I would expect things to be working it my path was broken.


[Bug] AttributeError: 'ConfigDict' object has no attribute 'dist_params'

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (master) or latest version (3.x).

Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
master branch https://github.com/open-mmlab/mmdetection
Environment
(deformable_detr) root@workspace:/dfs/data/code_python/detection_2d/mmdetection# python mmdet/utils/collect_env.py
fatal: not a git repository (or any parent up to mount point /dfs)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
sys.platform: linux
Python: 3.8.13 (default, Mar 28 2022, 11:38:47) [GCC 7.5.0]
CUDA available: True
GPU 0,1: Tesla V100-PCIE-32GB
CUDA_HOME: /usr/local/cuda
NVCC: Build cuda_11.3.r11.3/compiler.29745058_0
GCC: gcc (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0
PyTorch: 1.10.0+cu113
PyTorch compiling details: PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) oneAPI Math Kernel Library Version 2021.4-Product Build 20210904 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v2.2.3 (Git Hash 7336ca9f055cf1bfa13efb658fe15dc9b41f0740)
OpenMP 201511 (a.k.a. OpenMP 4.5)
LAPACK is enabled (usually provided by MKL)
NNPACK is enabled
CPU capability usage: AVX512
CUDA Runtime 11.3
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86
CuDNN 8.2
Magma 2.5.2
Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=11.3, CUDNN_VERSION=8.2.0, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -DEDGE_PROFILER_USE_KINETO -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.10.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.11.0+cu113
OpenCV: 4.6.0
MMCV: 1.3.18
MMCV Compiler: GCC 7.3
MMCV CUDA Compiler: 11.3
MMDetection: 2.26.0+
Reproduces the problem - code sample
(deformable_detr) root@workspace:/dfs/data/code_python/detection_2d/mmdetection# CUDA_VISIBLE_DEVICES=1 tools/dist_train.sh configs/deformable_detr/deformable_detr_r50_16x2_50e_coco.py 1
Reproduces the problem - command or script
(deformable_detr) root@workspace:/dfs/data/code_python/detection_2d/mmdetection# CUDA_VISIBLE_DEVICES=1 tools/dist_train.sh configs/deformable_detr/deformable_detr_r50_16x2_50e_coco.py 1
Reproduces the problem - error message
Traceback (most recent call last):
  File "tools/train.py", line 244, in <module>
    main()
  File "tools/train.py", line 172, in main
    init_dist(args.launcher, **cfg.dist_params)
  File "/dfs/data/anaconda/envs/deformable_detr/lib/python3.8/site-packages/mmcv/utils/config.py", line 507, in __getattr__
    return getattr(self._cfg_dict, name)
  File "/dfs/data/anaconda/envs/deformable_detr/lib/python3.8/site-packages/mmcv/utils/config.py", line 48, in __getattr__
    raise ex
AttributeError: 'ConfigDict' object has no attribute 'dist_params'
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 2441) of binary: /dfs/data/anaconda/envs/deformable_detr/bin/python
Traceback (most recent call last):
  File "/dfs/data/anaconda/envs/deformable_detr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/dfs/data/anaconda/envs/deformable_detr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/dfs/data/anaconda/envs/deformable_detr/lib/python3.8/site-packages/torch/distributed/launch.py", line 193, in <module>
    main()
  File "/dfs/data/anaconda/envs/deformable_detr/lib/python3.8/site-packages/torch/distributed/launch.py", line 189, in main
    launch(args)
  File "/dfs/data/anaconda/envs/deformable_detr/lib/python3.8/site-packages/torch/distributed/launch.py", line 174, in launch
    run(args)
  File "/dfs/data/anaconda/envs/deformable_detr/lib/python3.8/site-packages/torch/distributed/run.py", line 710, in run
    elastic_launch(
  File "/dfs/data/anaconda/envs/deformable_detr/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 131, in __call__
    return launch_agent(self._config, self._entrypoint, list(args))
  File "/dfs/data/anaconda/envs/deformable_detr/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 259, in launch_agent
    raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
============================================================
tools/train.py FAILED
------------------------------------------------------------
Failures:
  <NO_OTHER_FAILURES>
------------------------------------------------------------
Root Cause (first observed failure):
[0]:
  time      : 2022-12-09_15:03:20
  host      : workspace
  rank      : 0 (local_rank: 0)
  exitcode  : 1 (pid: 2441)
  error_file: <N/A>
  traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html
============================================================

Additional information
No response

If only one GPU is used, why not use python train.py config

Feel free to re-open this issue if you still meet this error

maybe your config file is wrong path

Feel free to re-open this issue if you still meet this error

Hello, I have encountered this issue, mmdet=2.28.1, mmcv=1.7.1. How should I solve it?

maybe your config path is wrong
---- Replied Message ----
| From | @.> |
| Date | 01/16/2024 11:29 |
| To | open-mmlab/mmdetection @.> |
| Cc | 张野 @.>,
Comment @.> |
| Subject | Re: [open-mmlab/mmdetection] [Bug] AttributeError: 'ConfigDict' object has no attribute 'dist_params' (Issue #9471) |
Feel free to re-open this issue if you still meet this error
Hello, I have encountered this issue, mmdet=2.28.1, mmcv=1.7.1. How should I solve it?
image.png (view on web)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


[Bug] Large series will not render as volumes and will crash (Array Buffer Allocation Failed)

Describe the Bug
If a series is a large enough the array buffer allocation fails and we can't do MPR on it.
Chrome:

Safari can do better but still the render seems odd, I also have to override the cs3d cache first to increase it:

Steps to Reproduce

https://viewer-dev.ohif.org/local
Load study attached
Go into basic viewer
Try to MPR
Check console

The current behavior
The viewer crashes
The expected behavior
Render MPR correctly
OS
MacOS
Node version
18
Browser
Chrome

@wayfarer3130 @sedghi
I can share the data on slack if needed

@IbrahimCSAE  - yes, if you shared the data I would look into this issue.  Do I have time against flexview for it?  I'm also doing some other work that is related.

Hi all, is there a defined strategy on this ?
I think the real way to solve this is to do partial loading of image, like pathology.
Probably estimate a max number of slices that can be loaded and then load / unload the slices depending on the camera location.
In my opinion it will be always a series that will not load in memory, either because browser limitation, hardware limitation, or series anyway too large with thousands of instances

Bill added a nice PR to fix the issue until 4GB for now https://github.com/cornerstonejs/cornerstone3D/pull/1260

That's cool ! (but i think it will have a day when we will have > 4gb images ^^⁾

@salimkanoun - I agree, we will have > 4GB images - we can double the new size on most machines now, by switching to 16 bit textures (instead of float 32).  That seems to have largely been fixed on the GPU.  Getting to larger than 4 gb will require either WebAssembly64, or webgpu implementation, which is in the works inside VTK, but has a LOT of testing and fixes to get things stable.  I'm also looking at splitting volumes out per image, which splits the webgl versus browser memory, and that might resolve things too, but I'm not sure when we will have time for that one.


FIX: mapping of empty point

Fixes https://github.com/shapely/shapely/issues/1807
Empty points coordinates are convert to an empty tuple. This is in line with how GEOS and PostGIS serialize empty points as GeoJSON ({"type":"Point","coordinates":[]}.

@Oreilles could you also update there with latest main (and add a note to CHANGES.txt)?

Should be good 👍

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 87.861%

Totals

Change from base Build<PHONE_NUMBER>7:
0.3%

Covered Lines:
2613

Relevant Lines:
2974

💛  - Coveralls

@Oreilles Thanks!


Merge proxy and router into one single process.

Fixes #1642.

Looking good apart from a couple more nits.

Docs still pending


Convert exit code label (str) to status (int)

Codecov Report
Attention: Patch coverage is 76.19048% with 5 lines in your changes missing coverage. Please review.

Project coverage is 80.69%. Comparing base (5937b88) to head (cdd37d8).
Report is 74 commits behind head on main.

Files with missing lines
Patch %
Lines

aiida_workgraph/task.py
70.00%
3 Missing :warning:

aiida_workgraph/workgraph.py
81.81%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #312      +/-   ##
==========================================
+ Coverage   75.75%   80.69%   +4.94%
==========================================
  Files          70       66       -4
  Lines        4615     5128     +513
==========================================
+ Hits         3496     4138     +642
+ Misses       1119      990     -129

Flag
Coverage Δ

python-3.11
80.69% <76.19%> (+5.02%)
:arrow_up:

python-3.9
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Implement memory_limit with Monte Carlo multiple comparisons correction

Summary

This comes from this NeuroStars post. Basically, MKDAChi2 + FWECorrector has been crashing for at least one user, and I think supporting memory_limit there, even if it limits the method to one core, will help.
Additional details

Reduced memory usage in meta-analyses.
Next steps

We can probably just do it the same way we do this in other classes- with an if statement regarding the number of cores.

I see two possibilities here:

Decorate the Corrector (or FWECorrector) class with use_memmap. I will probably need to grab the relevant attributes (i.e., self.memmap_filenames and self.memory_limit) in Corrector.transform in order to feed them to the Estimator-specific correction method. I will also need to add optional parameters for these two to that method.
Decorate Estimator.correct_fwe_montecarlo with use_memmap. I worry that this would overwrite the attributes from the Estimator as a whole, which would be a problem when re-using the object.

Am I missing any options?


Missing tests for unfold

As the title says.
Unfold is a sort-of in-dataframe .toDataFrame() conversion (https://kotlin.github.io/dataframe/unfold.html), but this behavior isn't tested anywhere, nor do we check it's consistent with our Iterable<>.toDataFrame() overloads.
Related to unfold, we should probably rename AnyCol.isPrimitive() and KClass<*>.isPrimitive() to isValueType() or something, because "primitive" has a different meaning in the JVM world.

Related to https://github.com/Kotlin/dataframe/pull/197


Fixes #791. Print version of AsciidoctorJ when calling asciidoctorj -…

…-version
With this PR the output of asciidoctorj --version looks like this:
AsciidoctorJRuby 2.0.0-SNAPSHOT (Asciidoctor 2.0.6) [https://asciidoctor.org]
Runtime Environment: jruby <IP_ADDRESS> (2.5.3)

I have no clue though if the modifications to the gradle build to create a properties file with the version of AsciidoctorJ is ok, I felt stupid adding this.

I like it, except I don't like "AsciidoctorJRuby". That's not the name of the project, so it's misleading. It should just be "AsciidoctorJ".


Redirect dev.bootstrapcdn.com to https

@jmervine: not sure how to handle this so I'm opening an issue to track it.

This is currently occurring.
http://dev.bootstrapcdn.com does already redirect to https://dev.bootstrapcdn.com, at least when I hit it.

Nope, that is not true.
C:\Users\XhmikosR\Desktop>curl -ILl http://dev.bootstrapcdn.com/
HTTP/1.1 200 OK
Server: Cowboy
Connection: keep-alive
X-Powered-By: MaxCDN
X-Hello-Human: You must be bored. You should work for us. Email<EMAIL_ADDRESS>or @jdorfman on the twitter.
Cache-Control: public, max-age=3600
Expires: Wed, 07 Dec 2016 20:54:36 GMT
Last-Modified: Wed, 07 Dec 2016 19:54:36 GMT
Strict-Transport-Security: max-age=16070400; includeSubDomains; preload
Accept-Ranges: bytes
X-Content-Type-Options: nosniff
X-Frame-Options: DENY
X-Xss-Protection: 1; mode=block
Content-Type: text/html; charset=utf-8
Content-Length: 11742
Content-Security-Policy: child-src 'self' platform.twitter.com syndication.twitter.com ghbtns.com;default-src 'none';font-src 'self' maxcdn.bootstrapcdn.com fonts.gstatic.com;img-src 'self' data: www.google-analytics.com bootswatch.com syndication.twitter.com pbs.twimg.com platform.twitter.com analytics.twitter.com stats.g.doubleclick.net;report-uri https://d063bdf998559129f041de1efd2b41a5.report-uri.io/r/default/csp/enforce;script-src 'self' 'unsafe-inline' maxcdn.bootstrapcdn.com www.google-analytics.com code.jquery.com platform.twitter.com cdn.syndication.twimg.com api.github.com;style-src 'self' 'unsafe-inline' maxcdn.bootstrapcdn.com fonts.googleapis.com platform.twitter.com
Vary: Accept-Encoding
Date: Wed, 07 Dec 2016 19:54:36 GMT
Via: 1.1 vegur

Let alone that it sends CSP without HTTPS...

Odd, in Chrome it's redirecting correctly best I can tell. Let me check Safari and Firefox.

It doesn't redirect with curl as you can see, it doesn't matter if it does elsewhere :P

Okay, it was Chrome thing. Safari and Firefox aren't redirect. It's because NODE_ENV=development is set for dev and https redirection only occurs in "production" mode.
Personally, I don't care about https on a dev site, but if the consensus ( @XhmikosR and @jdorfman ) is that it should be there, I can either set dev to use "production" or make a quick code change to ignore environment and only respect the "FORCE_SSL" variable for turning this on and off.

Personally I think it might be just easier to set the dev site to use NODE_ENV=development. This will be consistent with the main site too.

I'm sure you mean s/NODE_ENV=development/NODE_ENV=production/ and you're probably right, I just haven't had the time to ensure that doesn't do anything else we don't want.
I know it will minify code, which I don't like for debugging purposes, but with review apps, it might not be an issue anymore.

Fixed...
jmervine@laptop:~$ curl -isLl http://dev.bootstrapcdn.com | head -n 30
HTTP/1.1 301 Moved Permanently
Server: Cowboy
Connection: keep-alive
Location: https://dev.bootstrapcdn.com/
Vary: Accept
Content-Type: text/plain; charset=utf-8
Content-Length: 63
Date: Thu, 08 Dec 2016 17:32:15 GMT
Via: 1.1 vegur

HTTP/1.1 200 OK
Server: Cowboy
Connection: keep-alive
X-Powered-By: MaxCDN
X-Hello-Human: You must be bored. You should work for us. Email<EMAIL_ADDRESS>or @jdorfman on the twitter.
Cache-Control: public, max-age=3600
Expires: Thu, 08 Dec 2016 18:32:16 GMT
Last-Modified: Thu, 08 Dec 2016 17:32:16 GMT
Strict-Transport-Security: max-age=16070400; includeSubDomains; preload
Accept-Ranges: bytes
X-Content-Type-Options: nosniff
X-Frame-Options: DENY
X-Xss-Protection: 1; mode=block
Content-Type: text/html; charset=utf-8
Content-Length: 10169
Content-Security-Policy: child-src 'self' platform.twitter.com syndication.twitter.com ghbtns.com;default-src 'none';font-src 'self' maxcdn.bootstrapcdn.com fonts.gstatic.com;img-src 'self' data: www.google-analytics.com bootswatch.com syndication.twitter.com pbs.twimg.com platform.twitter.com analytics.twitter.com stats.g.doubleclick.net;report-uri https://d063bdf998559129f041de1efd2b41a5.report-uri.io/r/default/csp/enforce;script-src 'self' 'unsafe-inline' maxcdn.bootstrapcdn.com www.google-analytics.com code.jquery.com platform.twitter.com cdn.syndication.twimg.com api.github.com;style-src 'self' 'unsafe-inline' maxcdn.bootstrapcdn.com fonts.googleapis.com platform.twitter.com
Vary: Accept-Encoding
Date: Thu, 08 Dec 2016 17:32:16 GMT
Via: 1.1 vegur


feat: add connection_cache_size to Config

Add a feature, to set connection_cache_size opt.

Is there any clue to add the connection cache for the async-h1 client?

You'll need to figure out how to implement it for the async-h1 client. You might need an atomic counter of some sort on the client.


Revert blocking zeroconf IPs

Pipeline tests have revealed that blocking the zeroconf IPs has negative side-effects. These rules apparently need to be more sophisticated than that.

Superseded by #52


docs: add note on how to use without Sanity Client instance and how to use with RSC

See #42.
Not passing in the entire Sanity Client actually significantly reduced the bundle size, in my case from 131kB to 102kB for First Load JS. IMO this should probably be the default recommendation.
I went ahead and removed the npm install --save @sanity/client part from the readme since it actually isn't strictly needed and many projects will probably already have it.
This PR also adds a note on how to use with Server Components in the App Router. The default example in the readme won't work in a Server Component since the loader cannot be set in a Server Component:

Unhandled Runtime Error Error: Functions cannot be passed directly to Client Components unless you explicitly expose it by marking it with "use server"

The Image component, being a Client Component, needs to be wrapped in another Client component in order to pass a custom loader to it.
Thanks for your work on this package!

👍 to having this explained in the readme!
Now that #62 was merged, this could be simplified further into just passing the project details, without having to wrap them in a client-like:
const imageProps = useNextSanityImage({
  projectId: process.env.NEXT_PUBLIC_SANITY_PROJECT_ID,
  dataset: process.env.NEXT_PUBLIC_SANITY_DATASET
}, mySanityData.image);


application_logs: serialize ENVOY_CONN_LOG connection id as tags

Additional Description: Resolves #28307. Following the addition of ENVOY_TAGGED_LOG and it's derivatives, this PR changes ENVOY_CONN_LOG and ENVOY_STREAM_LOG to print connection and stream IDs as tags. Examples below.
Risk Level: Medium
Testing: Unit tests
Docs Changes: None
Release Notes: None
Platform Specific Features: None
When JSON logs are not enabled:
Log line: ENVOY_CONN_LOG(info, "fake message {}", connection_, "val");
Expected log: [Tags: "ConnectionId":"52"] fake message val
When JSON logs are enabled:
Log line: ENVOY_CONN_LOG(info, "fake message {}", connection_, "val");
Expected log: {"Message":"fake message val","ConnectionId":"52"}

/assign @mattklein123
Will appreciate your review for this one

/retest


NPE in WindowsShellHelper

Steps to reproduce:

Open and kill terminal very quickly

This can likely happen when the terminal is killed during a wmic.exe call.
Cannot read property 'on' of null: TypeError: Cannot read property 'on' of null
    at new WindowsShellHelper (file:///C:/Users/Daniel/Documents/dev/Microsoft/vscode/out/vs/workbench/parts/terminal/electron-browser/windowsShellHelper.js:26:24)
    at file:///C:/Users/Daniel/Documents/dev/Microsoft/vscode/out/vs/workbench/parts/terminal/electron-browser/terminalInstance.js:98:90
    at Object.notifySuccess [as _notify] (file:///C:/Users/Daniel/Documents/dev/Microsoft/vscode/out/vs/base/common/winjs.base.raw.js:1170:59)
    at Object.enter (file:///C:/Users/Daniel/Documents/dev/Microsoft/vscode/out/vs/base/common/winjs.base.raw.js:852:30)
    at Promise_ctor._run (file:///C:/Users/Daniel/Documents/dev/Microsoft/vscode/out/vs/base/common/winjs.base.raw.js:1068:29)
    at Promise_ctor._completed (file:///C:/Users/Daniel/Documents/dev/Microsoft/vscode/out/vs/base/common/winjs.base.raw.js:1036:18)
    at file:///C:/Users/Daniel/Documents/dev/Microsoft/vscode/out/vs/workbench/parts/terminal/electron-browser/terminalInstance.js:92:61
    at CallbackList.invoke (file:///C:/Users/Daniel/Documents/dev/Microsoft/vscode/out/vs/base/common/callbackList.js:58:43)
    at Emitter.fire (file:///C:/Users/Daniel/Documents/dev/Microsoft/vscode/out/vs/base/common/event.js:91:40)
    at ChildProcess.<anonymous> (file:///C:/Users/Daniel/Documents/dev/Microsoft/vscode/out/vs/workbench/parts/terminal/electron-browser/terminalInstance.js:527:45)

I can still repro this with the same stack, I don't think the PR fixes the right issue. It fails on this line, because this._xterm is null: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/terminal/electron-browser/windowsShellHelper.ts#L41
A null xterm object is passed into the constructor, which implies that the terminal instance is null, so I think the problem is the terminalInstance being disposed while processReady is running:
https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/terminal/electron-browser/terminalInstance.ts#L164


[BUG] Slow animations on Android

Great job on the pane so far but there seems to be an issue on my ionic 4/5 apps where the pane animates rather slowly and skips a lot of frames.
Steps to reproduce the behavior:

Create a starter app on a real Android device
Install this package and use as usual.
Ensure the app is built into an apk and installed on a real Android device running latest chrome browser and using Chrome webview.
Drag the pane to expand or collapse to see the slow animations
Expected behavior
It used to animate very smoothly while sliding the pane up and down previously, so this is probably due to a recent update to Chrome.

Thank you for report @SumailaBello
What pane version are you using ?

Thank you for report @SumailaBello
What pane version are you using ?

Ok I just updated to the latest version now and the problem seem to be solved plus it seems to be faster now 😁. Great job 👏🏾


On data preprocessing

Hello, I'd like to ask about Max in trantransform class_ What is the role of the labels parameter? Why is my image data transferred into the neural network without normalization parameters, but simply converted into tensor. Thank you.

sorry, I have some problems with the above writing. what's role of max_labels in TrainTransform class?

Yeah, same question, would be nice if someone elaborated...

Ah, they are simply using batch norm before input, and do not use imagenet pretrained weights

max_labels is decided by dataset you are training. Since different images have different number of boxes, all box annotations should be padded into the same size decided by max_label.


Are Adaptive Cards 1.2 supported?

It seems this document hasn't been updated in a few months, and the Adaptive Cards website says Teams supports 1.2. Does it?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3fb4394d-25b1-8752-3996-fe3a2934732d
Version Independent ID: 6d391381-89d0-117c-27cc-00043bd26a36
Content: Cards Reference - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-reference.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

Note
Only version 1.0 of Adaptive Cards is supported for all users. Version 1.2 is currently available only in Developer Preview

This is listed at the top of this section now. Likely as a result of this report.

@technicallynick - Unlikely, because my report was responding to that note

Hi @v-kydela . Yes, Adaptive Cards v1.2 are supported in Microsoft Teams. I have updated the Cards reference article to reflect the latest Adaptive Cards release. Thank you!

Hi @laujan , ActionSubmit inside ActionSet still doesn't work on my side in Teams. Here is my code:
`public static IMessageActivity BuildIntentSelectionCardJustin(ITurnContext turnContext, dynamic data)
{
        var intentRefList = GetIntentRefList();
        var intentCurrentList = intentRefList.Where(kv => ((IList<DispatchLuis.Intent>)data).Any(di => di.ToString().Equals(kv.Key)) || (kv.Key == Intent.None.ToString()));

        var adaptiveCard = new AdaptiveCard("1.2");
        var body = new List<AdaptiveElement>();

        foreach (var intent in intentCurrentList)
        {
            var actionSet = new AdaptiveActionSet();
            var actionSubmit = new AdaptiveSubmitAction()
            {
                Title = intent.Value,
                Data = intent.Value
            };
            actionSet.Actions.Add(actionSubmit);
            body.Add(actionSet);
        }

        adaptiveCard.Body = body;

        var attachment = new Attachment()
        {
            ContentType = AdaptiveCard.ContentType,
            Content = adaptiveCard,
        };
        return MessageFactory.Attachment(attachment, inputHint: InputHints.AcceptingInput);
    }`

@justgfather - Teams does not support string submit actions. You will need to put something in the Data property that can be represented as a JavaScript object. Please read my blog post for more information: https://blog.botframework.com/2019/07/02/using-adaptive-cards-with-the-microsoft-bot-framework/

@

@justgfather - Teams does not support string submit actions. You will need to put something in the Data property that can be represented as a JavaScript object. Please read my blog post for more information: https://blog.botframework.com/2019/07/02/using-adaptive-cards-with-the-microsoft-bot-framework/

Can you show me how to do it? I do not understand

@justgfather - Look in the Adaptive Cards in Teams section to see how to use an imBack submit action in Teams.
 If you still need help, post a question on Stack Overflow.


Added a Marquee

"Defending Freedom: Exposing Unjust Detentions"

your PR has conflicts. please resolve them.

Wanted to add a Marquee on the index.html.
On Wed, 24 Jul 2024 at 15:10, Wesley Kambale @.***>
wrote:

your PR has conflicts. please resolve them.
—
Reply to this email directly, view it on GitHub
https://github.com/wkambale/missingpersons/pull/82#issuecomment-2247750231,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCX3G4XWC2OYAVYYZU3ZQLLZN6KTTAVCNFSM6AAAAABLMKMN62VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBXG42TAMRTGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/marquee marquee are a deprecated component

Thanks
On Wed, 24 Jul 2024 at 18:58, Stacey Ali @.***> wrote:

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/marquee marquee
are a deprecated component
—
Reply to this email directly, view it on GitHub
https://github.com/wkambale/missingpersons/pull/82#issuecomment-2248375298,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCX3G4X3XQVEP6KHH7IIWSLZN7FLBAVCNFSM6AAAAABLMKMN62VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBYGM3TKMRZHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

@Kubomu given @Slam256 's feedback, I suggest we close this PR.
@Slam256 if you are happy with the suggestion, please close the PR.


[Bug]: Convert workflow errors to more gracefully exit

Bug Description
Currently, when the process is exited via an error code, an email goes out that the workflow failed. See https://github.com/seriouslysean/monster-hunter-now-events/actions/runs/6688963642.
This is great, but means an email goes out.
Expected Behavior
What we actually want to happen, is for the job to succeed, but exit gracefully (with a success message maybe) saying that there's just no need for the job to continue and not allowing any future jobs to complete.
This will need to be tested with the various steps, however, to ensure no future steps occur with a non error exit.
Steps to Reproduce (Optional)
No response

Fixed as of 1.5.0.


Documentation

Hello,
Would be nice to have some documentation with examples of some complex methods, like page rules, DNS records, etc.

HI @shadowlmd;
There are some examples on our Knowledge Base: https://support.cloudflare.com/hc/en-us/articles/115001661191-Cloudflare-PHP-API-Binding
Do these work for you, or would you like us to add something else?

Yep, these should do. I wonder how I could miss this link. Sorry, my bad.


Add GIMP

Microsoft Reviewers: Open in CodeFlow

There already is a GIMP package: https://github.com/microsoft/winget-pkgs/tree/master/manifests/GIMP/GIMP
Please review both the manifest directory, and open/closed Pull Requests, to make sure you are not creating duplicates.