table-input is not a modal

On this issue’s comment field
Click the table-input button
Click anywhere on the page

The dropdown should close, but it doesn't.

Modals like the "canned replies" automatically closes by design, without any JavaScript. table-input should be modified so it natively closes like the rest of the modals, it should probably never be explicitly closed via JS.

I may be wrong, but I remember it working as supposed to when the feature was first added. It was probably broken by GH updates. I'll take a look at it.

Seems like adding a tooltip on the button disrupts the modal functionality. The "canned replies" input also doesn't have a tooltip, so my guess is that GitHub doesn't implement this in the first place.
I tried a few things, including moving the tooltip to the <details> element itself, to no avail. Unless someone more familiar with the GitHub DOM has a solution, we'll just have to drop the tooltip. It's not perfect, but imo having a functioning modal is more important than having a tooltip.

Seems like adding a tooltip on the button disrupts the modal functionality

You're right, that's because both use :before.
It won't be accessible, but we can wrap the SVG in a div with the tooltip attribute. Better than  nothing at all.

(For record, the tooltip is added in #3615)
So this is why the table-input button has somehow "awkward" focus, and only shows tooltip when hover the icon instead of the larger button.

The "canned replies" input also doesn't have a tooltip

Did you mean "Insert a reply"? It does have a tooltip now:

But it doesn't open a modal, you have to click the icon or press Esc to close it.
Just my random thoughts, doesn't necessarily mean I want a change.

button has somehow "awkward" focus

What are you referring to?
Anyway I noticed that our buttons don't handle the focus the same way:

It's a little hard to see but at first I press tab, where you see the focus going between the first and table-input. Then I move the focus with the arrows, which proceed to skip over table-input and the details button.
They probably need some DOM work to match the other buttons.

only shows tooltip when hover the icon instead of the larger button.

This is because the button has margins inside the summary. Like above, DOM work is welcome.

The "canned replies" input also doesn't have a tooltip

Did you mean "Insert a reply"? It does have a tooltip now:

Yes. You mention that and it made me notice that that button is not a modal either. Ew. I definitely prefer the modal over the tooltip.


Does pandler handler nested VCS references?

I have three different projects in my private VCS(bitbucket), which have nested dependencies like the following. Also, Let's say project A, B and C depend on the same pypi lib but different versions.
A => B => C
I'd like to have whole dependencies of all the projects, listed in frozen.txt of project A, and find any conflicts among dependent pypi libs of them. Would it be possible to handle this with the current version of Pundler? thanks

So, it is in install_requires field, yes? I did not try it, but what I can see from code it should work. Not sure about this, need to create test projects.

It should have form like git@repo.

install_requires field, yes?

My setup.py looks like this.
setup.py
setup(
    ....
    dependency_links=[
        'git+ssh://git@bitbucket.org/szkmp/foo.git@master#egg=foo',
    ],
    install_requires=[
        'foo'
    ]
)

Sorry I flowed from Ruby and am new in python.
Okay, good to hear that. I will let you know if I get any trouble. You could close this issue for now. Thank you


Configure which "backend" to use when searching

I recently installed this and just want to say that this a really amazing plugin! Incredible simple yet clever idea 👍
Background

I want to add a pattern to match semantic versions (the pattern, if you are interested, looks like this: ([0-9]+)\.([0-9]+)\.([0-9]+)(?:(\-[0-9A-Za-z-]+(?:\.[0-9A-Za-z-]+)*))?(?:\+[0-9A-Za-z-\-\.]+)?);
grep doesn't think this is a valid regex;
ripgrep however, handles the regex just fine.

Now I could debug why grep doesn't like the regex but in general I prefer to use ripgrep over grep whenever possible. Which leads me to the question:
What do you think about having the option to configure which "backend" to use:

either by defining which command to run (default: grep but in my case configured to rg);
or a list of backends where the first one available is chosen (i.e. rg ag grep, will check if rg exists, otherwise fallback to ag etc)?

If you are interested I can see if I can throw a pull request together.

That's a good question!
I'm afraid right now it'd be very difficult to implement switchable regexp backends.
tmux-fingers uses gawk ( beware, shitty script ahead :trollface: ) under the hood to match, render and build a lookup table in a single pass.
This is done for two reasons mainly:

Performance: just one script doing all the heavy lifting in a single pass is quickier. From previous experience ( see how it was done in version 0.1.6 ), relying on a bash script has lots of process overhead and poor performance.

Rendering: grep does not provide column info about the match, making it very hard to render tags at the correct position.

So, having a single awk script doing all the stuff makes it very hard to plug in an alternative regexp backend. I suggest you try to debug your regexp with grep -E, which is what gawk uses underneath.
 Right now I think it's very complicated 💀
If you think of an alternative approach let me know 🙏

Fair enough! I'll just have to debug why grep -E doesn't like the regexp then. Thanks for the quick response :)


Is it possible to persist cards?

Hi, I love your card maker. Is it possible to persist cards so that we can later reference them again? Thanks a lot!

there's a save card button under the card preview. I think that's what you're thinking right?

Hi I'm talking about revisiting older card builds, how would that be
possible? Save seems to only cache the things in browser temporarily?
Muggy Ate @.***> schrieb am Di., 4. Jan. 2022, 5:45 AM:

there's a save card button under the card preview. I think that's what
you're thinking right?
—
Reply to this email directly, view it on GitHub
https://github.com/muggy8/lor-card-maker/issues/2#issuecomment-1004525874,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEYF7DZRO6GDEMXDVQ2OLN3UUJ3PJANCNFSM5LF6ICCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm not sure what you're talking about. you can click on saved cards to re-edit them. yes cards will be saved in the browser cache but I'm not paying for a server so there's nowhere else to store it...

Ok thanks for the clarification!
Am Di., 4. Jan. 2022 um 08:29 Uhr schrieb Muggy Ate <
@.***>:

I'm not sure what you're talking about. you can click on saved cards to
re-edit them. yes cards will be saved in the browser cache but I'm not
paying for a server so there's nowhere else to store it...
—
Reply to this email directly, view it on GitHub
https://github.com/muggy8/lor-card-maker/issues/2#issuecomment-1004581867,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEYF7D5DJ3SEVEO6DXOH2UTUUKOVZANCNFSM5LF6ICCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


How can we stop vulcanize to  verify external scripts path?

In my case,some components is very sample,just a normal fragment with old html tags,I design them not just for Polymer and other web-components base projects,they can be in any projects.
Most of my project are pre-process,so vulcanize is the best solution for me,so I can keep all components html file and require them by <link rel="import">.
This maybe beyond what vulcanize do,I am just thinking about if I cant do some trick without hack the codes.

vulcanize verify all external scripts path,can we turn that off?I've try the excludes option,not working for me.
vulcanize replace all resources link,can we turn that off too?

This should be addressed by the new polymer-analyzer based design of polymer-bundler.


Doesn't work when sppkg file is deployed to site collection app catalog

During my experimentation today (May 13th 2018) on a targeted release dev tenant, I've found the custom web part only appears in the full-width column web part picker if I've uploaded the SPFx package with includeClientSideAssets equal to "false" in ./config/package-solution.json.

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

ID: 4825eeab-f123-c0fb-0d30-6d634c821bd2
Version Independent ID: 55d7e710-4faa-4cfa-772d-f83962d9fe52
Content: Use web parts with the full-width column
Content Source: docs/spfx/web-parts/basics/use-web-parts-full-width-column.md
Service: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

The above seems to be true for the SPFx package being uploaded to both the tenant's app catalog, and also to a Site Collection app catalog

It was a webpart created from the latest yo generator 4 days ago.  Here's a gist of the important files:
https://gist.github.com/mpowney/3c0d8285208b11dc9cd388d600c60d06
I'll try building the ColumnFormatter solution later today and let you know what I find

Following up on this - I've tried with SPFx 1.5.1 - I think the problem I've found occurs when I deploy the full bleed web part in a solution added to a site collection app catalog instead of the tenant app catalog.  The steps to reproduce this behaviour are as follows:

Use the latest yo generator to generate an SPFx 1.5.1 solution with a React based web part, auto deploy to tenant remains as default value (no)
Update the web part's manifest file - add the "supportsFullBleed" property below "requiresCustomScript"  and specify as true
Ensure the package-solution.json file has "includeClientSideAssets" set to true
Create a new modern site (Blank communications site)
Use the office365-cli or PnP PowerShell cmdlets to enable an app catalog on the new comms site
Wait a minute or two for the app catalog to complete its provisioning
Load the SPFx solution in to the site's app catalog - drag and drop in to App Catalog library, trust the solution
Add the app to the site collection's root web
Edit the site's home page
Add a full width section to the page

Observations

At this point, the icon for the new web part displays in the page's edit mode in the full width column
Clicking on the web part's icon to add it to the page results in a "Loading" spinner, but nothing further happens
Performing a browser refresh of the page and going back in to Edit mode, the web part is no longer shown as an option in the full width section

If I perform the same steps above and deploy the same sppkg file to the tenant app catalog, the full bleed web part behaves as expected.
Can anyone else confirm this behaviour?

When you include includeClientSideAssets="true" in the documentation it says

If you would not have CDN enabled in your tenant, and the includeClientSideAssets setting would be truein the package-solution.json, the loading URL for the assets would be dynamically updated and pointing directly to the ClientSideAssets folder located in the app catalog site collection. In this example case, the URL would be https://sppnp.microsoft.com/sites/apps/ClientSideAssets/. This change is automatic depending on your tenant settings and it does not require any changes in the actual solution package.

However, if you look at that URL that is the application catalog for the tenant not the app catalog for the site collection.  So maybe this settings is not supported for site collection app catalog?  If I find out anything else I will let you know.
https://docs.microsoft.com/en-us/sharepoint/dev/spfx/web-parts/get-started/hosting-webpart-from-office-365-cdn

Is this issue still present? I saw where a fix was rolled out in a recent version of SPFx / SPO that may have addressed this...

I haven't been paying a lot of attention lately, but in an sppkg we use across a number of tenants (including first release tenants), I've found our full bleed web parts to intermittently show or not show. My general perception is the full bleed web part selection, in page edit mode, is still a bit buggy.
Can you provide a specific reference to the fix @andrewconnell?
I'll see if I can be more specific about the circumstances it doesn't show...

@mpowney said:

Can you provide a specific reference to the fix @andrewconnell?

Sorry, I can't... I don't have a reference to point to.
I have seen two other issues that might be related... can you review and see if they are related? #1779 & #1428


Improve instructions

It would be much easier if installation was one liner and required sample.yaml was included in github repo.

Also add something to README about troubleshooting - for example apply works but it fails ...
(base) aslom@m cloud-functions-operator (master) $ kubectl apply -f sample.yaml
function.ibmcloud.ibm.com/greetings created
(base) aslom@m cloud-functions-operator (master) $ ibmcloud wsk action invoke greetings -br
error: Unable to invoke action 'greetings': The requested resource does not exist. (code efabc39430c040b264aafbd7f24095d8)
Run 'wsk --help' for usage.


Add README for API Gateway

API Gateway lacks a readme with an overall explanation of the microservice

Done. Commit here https://github.com/airavata-courses/Team-Rocket/commit/e3c5c2c37d8f13222ace0749a6245078a52fe73c


Add note for installing neuroconv datainterfaces with zsh

Motivation
I'm using zsh and noticed that installing neuroconv as pip install neuroconv[extract] did not work for me. The solution was to use quotes, like pip install 'neuroconv[extract]'. This is added as a note in the User Guide, as it may be helpful for others.
Checklist

[x] Have you thoroughly read our Developer Guide?
[x] Have you ensured the PR description clearly describes the problem and solutions?
[x] Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?
[x] If this PR fixes an issue, is the first line of the PR description fix #XXX where XXX is the issue number?
[x] Did you update the CHANGLOG.md file on your branch to describe your changes?

Small rephrasing otherwise LGTM, thanks for catching and adding this.

Thanks @CodyCBakerPhD!

On the edge of this one, if this is something that we can use in the release notes and users should know it should be there. Things for devops and testing that is mainly for us less clear to me. Documentation probably should go in.


[WORKAROUND] test_mdgraph: skip unsupported algo for gfx1030

workaround for issue #763
skip unsupported fusion algorithms for gfx1030

Converted to draft to avoid spoiling docker image cache on CI. Please change this back to "normal" PR only after ending of CI moratorium (see mail). Then first merge from develop. Push the [Ready for review] button only after that. Thanks.


get rid of delay for existing tab checking

when I set up existing tab checking I put in 500ms timer to give it extra buffer to catch tabs...
now that we have extra instance detection from the cloud I'm removing the delay
apparently cloud detection doesn't help this case. other two points remain:
this 1. speeds up startup times, 2. doesn't seem to miss any existing tabs in my testing anyway so maybe the buffer never helped at all

@TonyGiorgio alright please re-review, I ended up refactoring to simplify and clarify this logic. we had two onmounts before for no good reason, and some unecessary abstraction.
in my testing it now catches dual tabs every time and it does it before even trying to setup


Support for Counter-based Authentication?

Hi,
Can you please add the functionality to add a key with Counter-based Authentication?
At the moment your addon seems to only work with time based authentication but a large number of systems require counter based.
Thanks...

Hi @reni10,
We will support HOTP in the next version.
Li Zhe

Done in version 4.10.


next/font/local issue on versions greater than 1.29.0

Current Behavior
When using next/font/local I receive a "Tamagui Webpack Loader Error:" on versions greater than 1.29.0.
The breaking code is located here.
This should not work for you as the version is set to 1.29.1. You can then downgrade to 1.29.0 to see that the application works
dont forget to install after downgrading the package and also to remove the cache files at apps/next/.tamagui
Tamagui Webpack Loader Error:
   createCSSVariable expected string: undefined
Error: createCSSVariable expected string: undefined
    at createCSSVariable (/Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/@tamagui/static/node_modules/@tamagui/core-node/dist/index.js:5850:17)
    at variableToCSS (/Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/@tamagui/static/node_modules/@tamagui/core-node/dist/index.js:40172:63)
    at /Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/@tamagui/static/node_modules/@tamagui/core-node/dist/index.js:40643:78
    at createTamagui (/Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/@tamagui/static/node_modules/@tamagui/core-node/dist/index.js:40715:9)
    at loadTamagui (/Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/@tamagui/static/dist/cjs/extractor/loadTamagui.js:67:55)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Object.load [as loadTamagui] (/Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/@tamagui/static/dist/cjs/extractor/createExtractor.js:88:42)
    at async extractToClassNames (/Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/@tamagui/static/dist/cjs/extractor/extractToClassNames.js:78:3)
    at async Object.loader2 (/Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/tamagui-loader/dist/cjs/loader.js:69:23)

Tamagui Webpack Loader Error:
   Converting circular structure to JSON
    --> starting at object with constructor 'Object'
    |     property 'Consumer' -> object with constructor 'Object'
    --- property '_context' closes the circle
TypeError: Converting circular structure to JSON
    --> starting at object with constructor 'Object'
    |     property 'Consumer' -> object with constructor 'Object'
    --- property '_context' closes the circle
    at JSON.stringify (<anonymous>)
    at parseWithConfig (/Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/@tamagui/static/dist/cjs/extractor/createExtractor.js:184:18)
    at Object.parse (/Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/@tamagui/static/dist/cjs/extractor/createExtractor.js:107:16)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async extractToClassNames (/Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/@tamagui/static/dist/cjs/extractor/extractToClassNames.js:93:15)
    at async Object.loader2 (/Users/stlouis/code/src/github.com/tamagui/next-font-issue/node_modules/tamagui-loader/dist/cjs/loader.js:69:23)

Expected Behavior
Expected next's not to crash on boot
Tamagui Version
1.29.1

Reproduction
https://github.com/vstlouis/tamagui-next-font-local-repro/tree/main

System Info
System:
    OS: macOS 13.3.1
    CPU: (10) arm64 Apple M1 Max
    Memory: 4.24 GB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 16.20.0 - ~/Library/Caches/fnm_multishells/1785_1685976473508/bin/node
    Yarn: 3.5.0 - /opt/homebrew/bin/yarn
    npm: 8.19.4 - ~/Library/Caches/fnm_multishells/1785_1685976473508/bin/npm
  Browsers:
    Chrome: 114.0.5735.133
    Safari: 16.4
  npmPackages:
    @babel/runtime: ^7.18.9 => 7.22.5
    @manypkg/cli: ^0.19.1 => 0.19.2
    check-dependency-version-consistency: ^3.0.3 => 3.3.0
    eslint: ^8.21.0 => 8.43.0
    node-gyp: ^9.3.1 => 9.4.0
    prettier: ^2.7.1 => 2.8.8
    turbo: ^1.10.3 => 1.10.3
    typescript: ^5.1.3 => 5.1.3

the very latest tamagui outputs a better error message, if you can run with DEBUG=tamagui as well and put the logs here


Authorization result : invalid on V<IP_ADDRESS>

Hello,
I have same issue with this link https://github.com/Lone-Coder/letsencrypt-win-simple/issues/515
When i run letsencrypt-win-simple.V<IP_ADDRESS> to create new certificate and i got [INFO] Authorization result : invalid.
PS. I already try this --webroot C:\inetpub\wwwroot when start it.

It seems like your web server is telling Let's Encrypt it's not allowed to see the response file (403).
This can have any number of reasons (IP restrictions, Windows authentication, file system rights, etc.). You should first confirm that with some service like http://www.webconfs.com/http-header-check.php to see if you can reproduce the 403 and then figure out the cause.

@WouterTinus Thanks for answer.
Here is what i got.
HTTP/1.1 200 OK => Cache-Control => no-cache, no-store, must-revalidatePragma => no-cacheContent-Type => text/html; charset=utf-8Expires => -1ETag => 6432dfce052049d181ebe9258fa72741Server => Microsoft-IIS/10.0X-Generator => OrchardX-AspNetMvc-Version => 5.2X-Frame-Options => SAMEORIGINX-AspNet-Version => 4.0.30319Set-Cookie => __RequestVerificationToken=meF2svORQt82_yEAShShAQs8o8WzVYJmOxqc9r-dBX_KpOEPTrCWtC-XQdQQXjHGFuDbwGvJ1FaH2novLtOVzgRbKf0trsD4JOCV5jprP4Y1; path=/; HttpOnlyX-Powered-By => ASP.NETDate => Mon, 16 Oct 2017 03:05:56 GMTConnection => closeContent-Length => 23554
--

Anything wrong ?

That looks like the homepage, not the answer file, so there's a small chance different settings apply in the subfolder where the answer files are stored, but my next step would be to test it on IPv6: http://ipv6-test.com/validate.php

@WouterTinus OK, This time i try with :
http://legendorc.test.xxxxx.com/.well-known/acme-challenge/dYR1qNhTF-SLhM2JGz9apTREvNwDSb_mMthw67YbxVk
And here is what i got :
HTTP/1.1 403 Forbidden =>
Server => Microsoft-IIS/10.0
X-Powered-By => ASP.NET
Date => Tue, 17 Oct 2017 07:28:54 GMT
Connection => close
Content-Length => 0

Then i try on http://ipv6-test.com/validate.php and put my url to check it.
Here what i got :

So there we have the 403, which is the reason why your certificate can't be authorized. Is there any chance that the root application (Orchard?) is intercepting the request and denying access?

@WouterTinus Oh! Thanks you very much.
Finally, I can do it.
Just follow this link : https://flowerbunch.org.uk/using-letsencrypt-with-orchard-cms

I changed the title to make this easier to find for other Orchard users. Good to hear that it's fixed!


Adding sendEvent method to the page object

Proposed changes in this pull request
Adds the sendEvent method to the page object, and requisite test
Checklist

[x] New tests have been added
[x] npm test passes successfully - see below
[ ] Documentation has been updated

Note on the tests for me I was getting 2 failures with the existing tests (i.e. before adding any new code)
Failures:
1) Phantom #create([]) execute with no parameters
1.1) Expected spy spawn to have been called with [ 'C:\wwwroot\phantomjs-node\node_modules\phantomjs-prebuilt\lib\phantom\bin\phantomjs.exe', [ 'C:\wwwroot\phantomjs-node\lib\shim.js' ] ] but actual calls were [ 'C:\wwwroot\phantomjs-node\node_modules\phantomjs-prebuilt\lib\phantom\bin\phantomjs.exe', [ 'C:\wwwroot\phantomjs-node\lib/shim.js' ] ].

2) Phantom #create(["--ignore-ssl-errors=yes"]) adds parameter to process
2.1) Expected spy spawn to have been called with [ 'C:\wwwroot\phantomjs-node\node_modules\phantomjs-prebuilt\lib\phantom\bin\phantomjs.exe', [ '--ignore-ssl-errors=yes', 'C:\wwwroot\phantomjs-node\lib\shim.js' ] ] but actual calls were [ 'C:\wwwroot\phantomjs-node\node_modules\phantomjs-prebuilt\lib\phantom\bin\phantomjs.exe', [ '--ignore-ssl-errors=yes', 'C:\wwwroot\phantomjs-node\lib/shim.js' ] ].

Looks like its an OS pathing issue (I'm on a windows machine), updated the the pull request to fix that issue (normalizing the path in shim.js)
@amir20 to review

Going to bring this locally and merge and squash the commits.

Merged manually! thanks!

Hi, I visit here from #408 , can anyone tell me how to use the sendEvent method? because it is undocumented...

It's pretty simple. It just works like all other functions. What I've you tried?
This doesn't like an issue. Try searching in older issues. Someone else provided an example.
Sent via my iPhone

On Aug 30, 2016, at 12:14 AM, Calvin Chan<EMAIL_ADDRESS>wrote:
Hi, I visit here from #408 , can anyone tell me how to use the sendEvent method? because it is undocumented...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


[glib] *.pc files are not being installed

I just started using vcpkg on macOS 10.13. Well, tried to. I did:
./vcpkg install glib

This installed glib and all its deps into installed/x64-osx/lib. But the pkgconfig directory is missing and the *.pc files of glib and its deps are not being installed. As a result, I can't build any of my software against the installed dependencies.
I have not found anything in the vcpkg documentation, and web searches turn up nothing.

Using GLib's Meson build system (#10389) would probably fix this.

I haven't used vcpkg for a long time (since the bug report, actually, due to the bug.) So I have no interest in this bug. Feel free to close this.

+1

Close this issue via #11884 merged.


Changed the name of the placeholder

Description
Changed the name of the placeholder form 'Share Your Thoughts' to 'Task Description'.
Fixes #33
Type of change

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

This image shows the change .
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules

@Shreyanshi210205

Please open another PR after closing this as we are nott able to review .
It does not shows the co-responding changes you did.


Do you support newer Python versions?

For example Python3.11 & 3.12 ?

I tested with python3.12 and it does work.


Wrap machete file IO operations in ReadAction/WriteAction

uiTests with headless flag don't work on Mac, @PawelLipski could you test them on Linux?

uiTests with headless flag don't work on Mac, @PawelLipski could you test them on Linux?

Passing CI is basically a passing test on Linux (or in Linux-like Docker environment, to be precise) ;)

uiTests with headless flag don't work on Mac, @PawelLipski could you test them on Linux?

Passing CI is basically a passing test on Linux (or in Linux-like Docker environment, to be precise) ;)

But CI has a virtual display instead of running the test in headless mode, right? And headless mode is what might not work as I've found [here].

But CI has a virtual display instead of running the test in headless mode, right? And headless mode is what might not work as I've found [here].

Huh we've never even meant to run UI tests in the actual headless mode (with -Djava.awt.headless=true). I don't think these tests were ever even close to working in such setup, and there isn't much value in that (as users, well, aren't likely to run IDE headlessly)

Btw... move the PR to draft? until #1303 is addressed

Stumbled upon an interesting thing in this file. I think this makes the usage a little bit easier and explicitly states that there are locks imposed on Read and Write actions as well as that they can be acquired from any thread.


Geolocation field plugin

Many map-mesh web applications are being created, and there's a common need for geolocation fields that are not provided by default in form creation. This enhancement request aims to address this gap, making Liberaforms an even more versatile tool for developers working on such projects.
Related Issue: https://gitlab.com/liberaforms/liberaforms/-/issues/135
Check out this video demonstrating the need for geolocation fields in action (this is the Wagtail plugin used for that field)
https://gitlab.com/liberaforms/liberaforms/uploads/c933e1bc6e1fef614c45558494070e55/2023-12-20T11_27_22.101Z.webm
Optional: Street search.
Obviously it would make more sense to have a MapBox or OpenStreetMap map using leaflet for example and use Nominatum or Mapbox streetsearch API service.
Context:
Liberaforms has been utilizing FormBuilder since its inception.
How this enhancement benefits users:
Simplifies the creation of map-mesh web applications.
Addresses a common need in the development community.
Enhances the overall functionality and appeal of Liberaforms and FormBuilder.

Steps to Reproduce:
Not applicable for this enhancement request.
Expected Behavior:
Geolocation fields should be available in the form builder to cater to the specific requirements of map-mesh web applications.
Additional Information:
Feel free to explore the related GitLab issue for more context and discussion.

Hi @numeropi
I have done a similar thing for a client previously using leaflet.js to do the map selection and GeoJSON to encode the features. Streetsearch would be an extension to that.

At the current time I would be looking for sponsorship from the community to code such a plugin and maintain it.


tesstrain.sh continues even if text2image crashed.

see comment: https://github.com/tesseract-ocr/tesseract/issues/1999#issuecomment-431032850
Problem seems to be that "check completion status" in tesstrain_utils.sh is wrong/does not work as expected. Here is short test:
#!/bin/bash

# Logging helper functions.
tlog() {
    echo -e $* 2>&1 1>&2 | tee -a ${LOG_FILE}
}

err_exit() {
    echo -e "ERROR: "$* 2>&1 1>&2 | tee -a ${LOG_FILE}
    exit 1
}

run_command() {
    local cmd=$(which $1)
    if [[ -z ${cmd} ]]; then
      for d in api training; do
        cmd=$(which $d/$1)
        if [[ ! -z ${cmd} ]]; then
          break
        fi
      done
      if [[ -z ${cmd} ]]; then
          err_exit "$1 not found"
      fi
    fi
    shift
    tlog "[$(date)] ${cmd} $@"
    "${cmd}" "$@" 2>&1 1>&2 | tee -a ${LOG_FILE}
    echo "Tested return error is:" $?
    # check completion status
    if [[ $? -gt 0 ]]; then
        err_exit "Program $(basename ${cmd}) failed. Abort."
    fi
}

run_command text2image --?
read -p "Everything is fine. Press any key..."

IMO according design logic run_command should exit script (because text2image --? returns exit code 1) , but it continue  and user see "Everything is fine. Press any key..."

@zdenop There are other issues with text2image and tesstrain.sh also. Many times it fails to create box file for a certain font but continues on any way and fails at the next step when it does not find the box file.

@Shreeshrii : did you understand description above and I test I made?

@zdenop I needed to test it out to understand it.
The exit status/code is based on the last command run.
    "${cmd}" "$@" 2>&1 1>&2 | tee -a ${LOG_FILE}
    echo "Tested return error is:" $?
    # check completion status

So completion status displays status of echo command and if that is removed then of tee command.
https://unix.stackexchange.com/questions/14270/get-exit-status-of-process-thats-piped-to-another
suggests ways by which we can get the status of text2image completion.
The following works, using pipefail:
#!/bin/bash

# Logging helper functions.
tlog() {
    echo -e $* 2>&1 1>&2 | tee -a ${LOG_FILE}
}

err_exit() {
    echo -e "ERROR: "$* 2>&1 1>&2 | tee -a ${LOG_FILE}
    exit 1
}

run_command() {
    local cmd=$(which $1)
    if [[ -z ${cmd} ]]; then
      for d in api training; do
        cmd=$(which $d/$1)
        if [[ ! -z ${cmd} ]]; then
          break
        fi
      done
      if [[ -z ${cmd} ]]; then
          err_exit "$1 not found"
      fi
    fi
    shift
    tlog "[$(date)] ${cmd} $@"
    set -o pipefail
    "${cmd}" "$@" 2>&1 1>&2 | tee -a ${LOG_FILE}
## echo "Tested return error is:" $?
    # check completion status
    if [[ $? -gt 0 ]]; then
        err_exit "Program $(basename ${cmd}) failed. Abort."
    fi
}

run_command text2image --?
read -p "Everything is fine. Press any key..."

output:
bash -x ./test.sh
+ run_command text2image '--?'
++ which text2image
+ local cmd=/home/ubuntu/tesseract/src/training/text2image
+ [[ -z /home/ubuntu/tesseract/src/training/text2image ]]
+ shift
++ date
+ tlog '[Fri Oct 19 02:51:13 UTC 2018] /home/ubuntu/tesseract/src/training/text2image --?'
+ echo -e '[Fri' Oct 19 02:51:13 UTC '2018]' /home/ubuntu/tesseract/src/training/text2image '--?'
+ tee -a
[Fri Oct 19 02:51:13 UTC 2018] /home/ubuntu/tesseract/src/training/text2image --?
+ set -o pipefail
+ /home/ubuntu/tesseract/src/training/text2image '--?'
+ tee -a
ERROR: Non-existent flag --?
+ [[ 1 -gt 0 ]]
++ basename /home/ubuntu/tesseract/src/training/text2image
+ err_exit 'Program text2image failed. Abort.'
+ echo -e 'ERROR: Program' text2image failed. Abort.
+ tee -a
ERROR: Program text2image failed. Abort.
+ exit 1

@stweil Is using pipefail the best way of handling this?

@Shreeshrii : can you try to add set -euo pipefail at the beginning of tesstrain_utils.sh and run the training?
See this blog for explanation what it should do...

@zdenop  actually setting -u in tesstrain_utils.sh leads to unbound variable errors in tesstrain_utils.sh and also

/usr/local/bin/language-specific.sh: line 889: FLAGS_webtext_prefix: unbound variable

This appears to fix the former ones at least in my short test:
diff --git a/src/training/tesstrain_utils.sh b/src/training/tesstrain_utils.sh
index 220688dd..b173e21c 100644
--- a/src/training/tesstrain_utils.sh
+++ b/src/training/tesstrain_utils.sh
@@ -213,12 +213,9 @@ parse_flags() {

     # Take training text and wordlist from the langdata directory if not
     # specified in the command-line.
-    if [[ -z ${TRAINING_TEXT} ]]; then
-        TRAINING_TEXT=${LANGDATA_ROOT}/${LANG_CODE}/${LANG_CODE}.training_text
-    fi
-    if [[ -z ${WORDLIST_FILE} ]]; then
-        WORDLIST_FILE=${LANGDATA_ROOT}/${LANG_CODE}/${LANG_CODE}.wordlist
-    fi
+    TRAINING_TEXT=${TRAINING_TEXT:-${LANGDATA_ROOT}/${LANG_CODE}/${LANG_CODE}.training_text}
+    WORDLIST_FILE=${TRAINING_TEXT:-${LANGDATA_ROOT}/${LANG_CODE}/${LANG_CODE}.wordlist}
+
     WORD_BIGRAMS_FILE=${LANGDATA_ROOT}/${LANG_CODE}/${LANG_CODE}.word.bigrams
     NUMBERS_FILE=${LANGDATA_ROOT}/${LANG_CODE}/${LANG_CODE}.numbers
     PUNC_FILE=${LANGDATA_ROOT}/${LANG_CODE}/${LANG_CODE}.punc

thanks!


Coveralls thinks upgrade() functions in migration scripts are not covered.

As discovered in the discussion on #971. Probably this is due to how alembic is being invoked somehow.

Fixed, since we just kicked out coveralls.


test/e2e: don't use internal versions of api extensions

The internal versions of the API Extensions client objects are what client calls get converted into for processing within the server. This mechanism allows many different clients to use many different outward-facing versions, while the code within the server only ever runs against one type. There is no utility to these types outside of the server, especially for code that's making client calls. We can simply use the client-facing types and not require coversion anywhere.

/approve


Fix .std to .var in layer & instance normalization

Mentioned in errata here https://github.com/fastai/course-v3/blob/master/files/dl-2019/notes/notes-2-10.md but not fixed in the notebook?

Many thanks.


Created a loading component and added loading component styles to the…

Hey Ovi,
Let me know if there's anything else I can help out with or fix in the future.

Awesome, thank you for your contribution!


Move single email not working

Hello there,
so i installed n8N with n8n-imap node :

si i create a node imap move mail andi push the UID of the mail :

but when i lunch it, all the mail of the folder mailbox is moved on the same folder and the selection of the mail is not working
how fix this problem please

Can you provide more details? Please share info from the previous nodes and the result of this one. What did you expect to happen, and what actually happened? Attach some screenshots if possible.

Hello,
this my graph :

the problem come from here :

Like u see in 1 i got only mail i want moved with a selection by from in the swtich case :

this switch case get 3 out, with select on form
looks like working
but the problem is the  imap 2 node move détails :

In the first using, all mail from inbox are moving and not only the mail with UID  i got on the node
this a my bug, i can't move only one mail, the node move all mail from Source Mailbox From LIST on the destination Mailbox
why i can't move only one mail i select and not all mail from the source folder
thanks

u need more information ?

I couldn't reproduce the problem on my end; I tested on my server, and emails are moving as expected.
However, I noticed that you have the IMAP node output looped back into its input. Could you clarify why this setup is in place? Please try removing that connection and testing again—it might be causing the issue.


There is some uncertainty about the units of antenna positions

It is unclear whether the various file formats have antenna positions that are relative to the telescope location as specified on pyuvdata.
This needs to be checked for all the various file formats at the read and write and there need to be checks on according to the various file standards.

This just came up for measurement sets while working on addressing issue #177 (on pull request #179). I think it's fixed for measurement sets (by subtracting off the telescope_location), but that was a relatively clean test because we were comparing measurement sets with the uvfits files that CASA produced from the measurement sets. Meaning that we now at least agree with CASA for the conversion of antenna positions between uvfits and measurement sets.

Fixed in PR #222


Should fix #142 build branch from fork

Adding ignore branch on CircleCI front & back build
Co-authored-by: jeffladiray<EMAIL_ADDRESS>

https://discuss.circleci.com/t/create-separate-steps-jobs-for-pr-forks-versus-branches/13419/7
We've just changed only to ignore, and rewrote the regex in order to be more logical. $CIRCLE_BRANCH should contains pull/:id when a PR based on a fork is created.
We've tested locally using circleci config validate & circleci local execute, but since this commit mostly impact branches, it need proper testing on master


the Replacing the SSL certificate for the Web Application Proxy PS missing agurment

correct one: Set-WebApplicationProxySslCertificate -Thumbprint ''

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

ID: d10177c6-b241-0a3e-d6ba-3619c094812a
Version Independent ID: 7a42691f-7f28-eee2-0c69-294e24cccdbe
Content: Managing SSL Certificates in AD FS and WAP in Windows Server 2016
Content Source: WindowsServerDocs/identity/ad-fs/operations/manage-ssl-certificates-ad-fs-wap.md
Service: unspecified
GitHub Login: @jenfieldmsft
Microsoft Alias: billmath

Resolved in commit https://github.com/MicrosoftDocs/windowsserverdocs/commit/9622422acc1b6f4bf134b1c8793c248198961f58 (Pull Request #2800 The "Thumbprint" parameter is required for Set-WebApplicationProxySsl…)

#please-close

#please-close


coreutils: disable tests which fail in the sandbox (no uid==1000)

Description of changes
Several coreutils tests assume there exists a user with uid=1000.  We need to disable these tests in the sandbox.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux

[ ] Fits CONTRIBUTING.md.

Why hasn't this been failing before?

Why hasn't this been failing before?

I have two theories:

This only happens on powerpc64le-linux, in which case it is not a nixpkgs issue
Not all of my builders are running the latest nix, and some lack this commit: https://github.com/NixOS/nix/commit/2fde7e0108d70bcba64ebecc5e5c7ee2863e3446

I'm running some builds now to find out which one it is.  In either case I've draftified this PR and will remove it from the inclusion-set of https://github.com/NixOS/nixpkgs/pull/209870.
Does Hydra have the uid-range feature enabled?

Ooh, yeah.  From just before that commit to nix, which wasn't included until 2.12:
https://github.com/NixOS/nix/blob/40911d7dec75541a400fe8f556d4c70a7f845fac/src/libstore/build/local-derivation-goal.hh#L102

Not all of my builders are running the latest nix, and some lack this commit: NixOS/nix@2fde7e0

Does Hydra have the uid-range feature enabled?

This hasn't been an issue even on old nix versions so I think these  2 are irrelevant

These haven't been failing on nix versions before 2.12 🤔

Yeah both of my build clusters (x86_64-linux and powerpc64le-linux) have made it past one build of coreutils without this, so I'll probably just close this and call it a heisenbug.

Btw do you have matrix? Would be cool to have you in
#stdenv:nixos.org
https://matrix.to/#/!OqhvaDMJdKYUicLDiE:nixos.org?via=matrix.org&via=nixos.dev&via=nixos.org

Btw do you have matrix?

I'm amjoseph on hackint.de (#nix-rfc-133) and libera.chat (#nixos, #nixos-dev, and #spectrum).  Both are bridged with matrix and we have plenty of |m users in channels on both of them.
Did nixorg ever manage to figure out how to bridge IRC into their matrix server?

Yep, it's a heisenbug.  Everything built fine without this.  WTF.
PS, it sometimes takes me a while to respond to mentions on IRC.  I deliberately don't have any kind of audio or desktop notification for synchronous chat.  I appreciate the usefulness of IRC/tox/etc, but from a productivity/willpower standpoint I have to be really careful with it.    I just can't let myself do interrupt-driven chat, it doesn't work for me.


Refactor - Responsive cleanups

This PR adds the following refactorings:

Style the cookie Banner with tailwindcss utility classes instead of theme-ui
Improve the header menu UX (closing the menu when clicking on a link, etc.)
Improve the docs' sidebar UX (scrolling through the sidebar on mobile, etc.)

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update UUID fuction with Hasura

Closes #1271

Hey @Smartmind12,
Your changes are not valid. So, I have to close this PR.


[Enhancement] on console close via clicking on the x leads to end of sound

On Windows 11 22H2 on command prompt, closing the console while the rain is still playing leads to that sound persisting even though console is closed.
Suggestion to end the sound like quit when user clicks on the x button.

Btw, this is an absolutely bomber of a project! I love it~
:D

Hey, thanks a lot! ✨ I started this project but I didn't like the the result that I got at first hahahahaha I need to improve the rain sound itself to make more vivid and seamless loopable.
About the issue, I think I can work on this quit Python routine to make sure the sound stops, thanks for the suggestion :)

Looking forward to the improvements you will make!
This project is clean bro! Clean! I love the UI and how neat and organized the readme is!

Gonna close this issue because after a restart, I couldn't reproduce the issue. Might be just me.


Exclude RequestStatus from the returned values of get_modifiable_properties

Fixes #12107
Status
Ready
Description
with the current change we remove the RequestStatus key from the returned values of get_modifiable_properties. this should avoid the conflict created between the set of values checked during the validation and the set of values supported in the WEB interface (the number of visualized keys in the web interface).
Is it backward compatible (if not, which system it affects?)
YES
Related PRs
A Fix of a bug eventually introduced  with: #12077
External dependencies / deployment changes
No

Hi @amaltaro  I just tested the fix - it's doing the job properly. we can safely merge this one

@todor-ivanov it looks like we missed the the failing unit test:
    WMCore_t.ReqMgr_t.Utils_t.Validation_t.ValidationTests:testValidateRequestAllowedArgs changed from success to failure

Please provide a new PR with the relevant fix (fixing the original issue 12107).

Because I have no idea why would jenkins claim it is failed... since this was exactly the test  that this PR fixes and I have tested even before I have created  the PR.
----------------------------------------------------------------------
Ran 3 tests in 0.093s

OK

So I have no idea why jenkins reports this as  a failure.

and you can also see this  very same test have changed from failure to success in the PR which @anpicci is working:
(which was actually the initial request, because of which we noticed we have missed this one here):
https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-WMCore-PR-test/15239/artifact/artifacts/PullRequestReport.html#pylintpy3
Here is the report from Andrea's PR jenkins actions, which I triggered upon merging this test:
WMCore_t.ReqMgr_t.Utils_t.Validation_t.ValidationTests:testValidateRequestAllowedArgs changed from failure to success


Cask: check if xattr works before quarantining

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same change?
[X] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[X] Have you successfully run brew tests with your changes locally?

xattr doesn't work in systems which have set their system Python version to < 2.7. I've added a check to check_quarantine_support to account for this.
Fixes Homebrew/homebrew-cask#52128.

Ping @Homebrew/brew for merging.


[TESTING] docs: output

PLEASE IGNORE
Signed-off-by: Ryan Northey<EMAIL_ADDRESS>
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/docs


Release/19.09 - Update Brave 0.69.128 -> 1.4.95

Motivation for this change
This is browser, so I believed that it required to be updated in the case that there would have been any updates related to the security.
Things done
This PR backports PR #73400 but requires changes from PR #79688 and PR #79974

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

I'm not sure about the process here, but shouldn't the new commits on release-19.09 be cherry-picked from master instead of having a custom-made commit?

It seems that it should be backported from other PR which I did. I can be wrong tho! @rht

@JeffLabonte  See https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#backporting-changes.
Comment from IRC was:

the commit header in the backport PR is different from the commit on master
(the backport PR) nixos: Update brave from 1.3.118 to 1.4.95 (on master) brave: 1.3.118 -> 1.4.95
the latter commit message is correct per the contribution guidelines (pkgname: old_verison -> new_version)

The above can be done by just doing git cherry-pick -x (e also if you want to edit the message with a reason).

Is this really a pure cherry-pick?
There is a sub-commit message
Update brave from 0.69.128 to 1.4.95
which means the commit that updates 0.69.128 to 1.3.118 was squashed into the commit that updates 1.3.118 to 1.4.95

Is this really a pure cherry-pick?
There is a sub-commit message
Update brave from 0.69.128 to 1.4.95
which means the commit that updates 0.69.128 to 1.3.118 was squashed into the commit that updates 1.3.118 to 1.4.95

Oh you're right. I guess it's too late now :rofl Extra cherry-picked commits in that case (it would have been two) would have been fine.

@worldofpeace will the mistake be resolved manually during the merge conflict resolution with unstable? How is this usually resolved?

Commit messages can't be changed now; we don't do any history rewriting on these branches (and this isn't really a strong reason).  Conflicts in git are only concerned with file contents and not commit messages, and we don't merge between master/unstable and stable branches... perhaps I misunderstood the question.

@vcunat

Conflicts in git are only concerned with file contents and not commit messages

I am concerned with the jump in the diff since 0.69.128 to 1.4.95 instead of 1.3.118. There will be a conflict during the cherry pick.

we don't merge between master/unstable and stable branches...

OK, I think I don't know the mechanism behind how the stable branches are being updated so as to not fall behind master/unstable. I assume it is a merge operation between a sufficiently mature version of old master/unstable, not a continuous cherry-pick from master/unstable.

If the following cherry-picks use commit starting with 1.4.95, it should be conflict-less.  In any case, these bumps seem to need no other changes, so conflict resolution is trivial.
Stable branches do "fall behind" intentionally.  Only qualifying changes get cherry-picked manually.

Make sense. Thank you for the explanation!


Users statistics

It would be usefull to add the "card-users" next to the app users in the statistics of the clients dashboard.
By example: Lochristi counts 2853 app users and there are 1250 cards that people use but nobody counts them in.
See also feature request to send news immediatly to the cardusers this increases the the reach of each client that pays foor the app.

Hi,
Can we have a ETC of this issue so we can update.
Else If the issue is resolved please put the state_resolved label and close the issue.
Thanks!


DetectNet Camera Error!

Hello,
I am trying to run DetectNet-Camera on my main machine. I've worked it before. It did not work after reinstalling the computer. I'm getting the following errors.
`./detectnet-camera
detectnet-camera
args (1):  0 [./detectnet-camera]
[gstreamer] initialized gstreamer, version <IP_ADDRESS>
[gstreamer] gstreamer decoder pipeline string:
v4l2src device=/dev/video0 ! video/x-raw, width=(int)1280, height=(int)720, format=RGB ! videoconvert ! video/x-raw, format=RGB ! videoconvert !appsink name=mysink
detectnet-camera:  successfully initialized video device
width:  1280
height:  720
depth:  24 (bpp)
detectNet -- loading detection network model from:
-- prototxt    /home/monster/Desktop/jetson-inference/build/x86_64/bin/networks/final/deploy.prototxt
-- model       /home/monster/Desktop/jetson-inference/build/x86_64/bin/networks/final/snapshot_iter_27552.caffemodel
-- input_blob  'data'
-- output_cvg  'coverage'
-- output_bbox 'bboxes'
-- threshold   0.500000
-- batch_size  2
[GIE]  TensorRT version 2.1, build 2102
[GIE]  attempting to open cache file /home/monster/Desktop/jetson-inference/build/x86_64/bin/networks/final/snapshot_iter_27552.caffemodel.2.tensorcache
[GIE]  loading network profile from cache... /home/monster/Desktop/jetson-inference/build/x86_64/bin/networks/final/snapshot_iter_27552.caffemodel.2.tensorcache
[GIE]  platform does not have FP16 support.
[GIE]  /home/monster/Desktop/jetson-inference/build/x86_64/bin/networks/final/snapshot_iter_27552.caffemodel loaded
detectnet-camera: cudnnEngine.cpp:605: bool nvinfer1::cudnn::Engine::deserialize(const void*, std::size_t, nvinfer1::IPluginFactory*): Assertion size >= bsize && "Mismatch between allocated memory size and expected size of serialized engine."' failed. Aborted (core dumped)

@dusty-nv Has anyone been able to solve this issue? I am using custom car detection model on jetpack 3.1 on TX2. On jetpack 3.0, I didnt have this issue. It was running perfectly. But after updating, it threw this error.

@curiouser001 did you update anything when "reinstalling the computer"? I am experiencing the same issue after upgrading Tensor RT and CuDnn.


Enable use of Mbed TLS as a CMake subproject

Remove use of CMAKE_SOURCE_DIR in case mbedtls is built from within
another CMake project. Define MBEDTLS_DIR to ${CMAKE_CURRENT_SOURCE_DIR}
in the main CMakeLists.txt file and refer to that when defining target
include paths to enable mbedtls to be built as a sub project.
Fixes #2609
Signed-off-by: Ashley Duncan<EMAIL_ADDRESS>Signed-off-by: Jaeden Amero<EMAIL_ADDRESS>Description
A few sentences describing the overall goals of the pull request's commits.
Status
IN DEVELOPMENT

Will need to point to a crypto submodule containing https://github.com/ARMmbed/mbed-crypto/pull/152 on the mbed-crypto/development branch

Requires Backporting
NO  - but may be useful in a 2.18.1
Migrations
NO
Additional comments
Any additional information that could be of interest
Todos

[ ] Tests
[ ] Documentation
[ ] Changelog updated
[ ] Backported

Rebased to add information about this newly supported feature to the README and to add a regression test.

Force pushed to update crypto submodule

Rebased to update crypto submodule

Rebased to not require too new of a CMake version.

Rebased to change minimum CMake version to 2.6, to match the version required by the top level project CMakeLists.txt. Also updated the crypto submodule to the latest version of https://github.com/ARMmbed/mbed-crypto/pull/152

Rebased to rebase atop latest development and resolve conflicts in ChangeLog and crypto submodule update.

Rebased to move cleanup of the CMake subproject test to cleanup(), and to update to the latest https://github.com/ARMmbed/mbed-crypto/pull/152

Rebased to update to the latest https://github.com/ARMmbed/mbed-crypto/pull/152

Rebased to update Mbed Crypto to a commit on its development branch containing the merged https://github.com/ARMmbed/mbed-crypto/pull/152 PR.

@Patater when is this PR expected to be merged?
We need the PSA multiple key usage feature - https://github.com/ARMmbed/mbedtls-psa/pull/266 that is part of Mbed TLS 2.18. And currently Mbed TLS 2.18 isn't compiled with PDMC on Linux, so this PR is needed


Prevent duplicates from being posted

It has been a problem since forever. Too often duplicates are posted.

Surprisingly it was done! Finished with PR #71.


Nesting level too deep - recursive dependency?

How do you use Sentry?
Sentry Saas (sentry.io)
Version
3.9.0
Issue
This is a rather strange issue and I'm not quite sure how to reproduce as I don't get much from the stack-trace.
Sentry\Exception\FatalErrorException is being thrown with the error:

Error: Nesting level too deep - recursive dependency?

The stack-trace is showing the call at /srv/www/vendor/symfony/dependency-injection/Compiler/AutowirePass.php:192 which is if ($arguments !== $call[1]) {.
Being that it's a Sentry exception, I can only assume that this lib is trying to get a stack-trace or parse the Symfony DI layer for some reason.
The strange thing about this error is that it's only occurring on AWS when the ELB is hitting the server's internal private IP, as the ELB does.  Normal application requests do not appear to be triggering this error, at least from what I can tell in the Sentry logs.  The errors are occurring across every node in the cluster at a 10 second interval (every ELB request for health-check).
I should further state that rolling back to 3.8.1 resolves this issue.
Any insight into what changed in 3.9.0 that might lead to this issue?

Is your app running Symfony, using our Symfony SDK?
This SDK has no dependency on https://github.com/symfony/dependency-injection so I'm not sure what is going on as we also don't throw a FatalErrorException in the codebase.

@cleptric we are not using that lib.  I also see the exception in this lib:
https://github.com/getsentry/sentry-php/blob/master/src/Exception/FatalErrorException.php
https://github.com/getsentry/sentry-php/blob/master/src/ErrorHandler.php#L328

And here is the shutdown function being registered that throws the Exception
https://github.com/getsentry/sentry-php/blob/master/src/ErrorHandler.php#L186

I meant we do not throw the FatalErrorException explicitly.
This will be tricky to reproduce though. Mind sharing some details about your application, dependencies and your AWS setup (EC2/ECS/?, OS, PHP version)? You can do so via michi (@) sentry.io.

I sent an email with some of our application details.  I think it's best to debug this from the angle of what's changed in the latest version, since downgrading the version resolves the issue.
There aren't any errors in our logs either, from the 3.8.1 version.  So, it's not like 3.9.0 is catching some new error and reporting it where 3.8.1 did not.  3.9.0 appears to be creating this error, itself.

Looks like there is something else going on and the SDK is not the root cause of it.


Update deploy-examples.yml

:tada: This PR is included in version 0.2.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore(lint): prefer const

If a variable is never reassigned, using the const declaration is better.

Thanks, but we're not strict about this.


Extra guidance on determining sourceImageID

In the section "Create an image in one tenant using the source image in another tenant", can I recommend that guidance on getting sourceImageID is provided.
To do this via CLI, the required command would be:
az sig image-version show --gallery-image-definition myImageDef --gallery-image-version 1.0.0 --gallery-name myGallery --resource-group myResourceGroup --query id -o tsv
It would also be helpful perhaps to output an example, as done with the VM ID in the previous section, "Create an image", which, using the example above and being consistent with this page's other examples, would be:
/subscriptions//resourceGroups/myResourceGroup/providers/Microsoft.Compute/galleries/myGallery/images/myImageDef/versions/1.0.0

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

ID: 720b9403-2322-5abd-2edc-7fa80bae90f2
Version Independent ID: 2c33a6b1-43ef-6736-3255-8f8c25feb9cf
Content: Create an image definition and image version - Azure Virtual Machines
Content Source: articles/virtual-machines/image-version.md
Service: virtual-machines
Sub-service: gallery
GitHub Login: @sandeepraichura
Microsoft Alias: saraic

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

Thanks @mosesmosheh for the feedback, I do not think it is possible to query this information in cross-tenant scenario.  @cynthn - FYI..

@sandeepraichura Can you query for the ID across tenants?

@mosesmosheh
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Add tidelift retriever basic test and functionality

Fix #14
Signed-off-by: John Speed Meyers<EMAIL_ADDRESS>

@jmelot, the tests seem to pass, but there is an error with the linting, though it seems to be related to a CI issue, not the code itself. Can you make heads or tails of it?
Also, with this PR I've tried to bring the Tidelift retriever in line with the new schema and how the codebase implements it. Feedback is welcome. It is a Monday morning after all :)

@jmelot, you are too good! Nice work identifying and fixing the issue.

@jmelot - Agreed. I'll merge this plus open a new issue for exactly what you describe.


tokengen - Add Cobra Cli interface description

Add the cobra interface description for init. This description should also include definition for following flags:

github //Plugin for GitHub enables
default-branch //setup default branch or use standard default

In stead of an init command, we will and a function that can generate a token via sign in from the user


Feature request: strong type dsl builder

the olivere/elastic supports follow codes:
q := elastic.NewBoolQuery() q.Must(elastic.NewTermQuery("id", "1544175232779030532"))
It's definitely more readable.

Thank you for the suggestion!
This is definitely in the long term goals and would qualify as a helper to build specific queries.
It would need to be built by hand upon the generated types to carefully chose what to expose which cannot be done as part of the code generation of the whole typed api.
I'm flagging this issue as an enhancement and will keep it open for future reference!

This feature is the biggest gap between olivere and the official lib

I know from previous pull request notes and issue posts as well as the release notes that the typedapi is in alpha.  Is there somewhere to subscribe to news on when it will move past the alpha phase?  Thanks!

Does this include the Filter.query? I rely on this heavily, and am stuck with olivere until I can do this
query := elastic.NewBoolQuery()
if filter.ObserverType != "" {
query = FilterQuery(filter.ObserverType, esFieldMapping["observer_type"], query)
}

@notque I'm not sure what your FilterQuery method does, however you can already do something like this:
s := &search.Request{Query: types.Query{Bool: types.BoolQuery{
	Filter: []types.Query{
		{
			Term: map[string]types.TermQuery{
				"field": {Value: "value"},
			},
		},
	},
}}}

And run that request through the Search endpoint.
If you want to dig further I would advise you to create a specific issue so we can keep this one dedicated to the dsl topic.

@Anaethelion understood, I wasn't thinking clearly and didn't show the full piece. I will create a new issue to show. Thanks.


Add Remark command and accompanying tests

Tutorial 2 - Adding a new command

Codecov Report

Merging #14 (178537b) into master (a0cd6a4) will increase coverage by 1.03%.
The diff coverage is 88.15%.

@@             Coverage Diff              @@
##             master      #14      +/-   ##
============================================
+ Coverage     72.15%   73.19%   +1.03%
- Complexity      399      421      +22
============================================
  Files            70       73       +3
  Lines          1232     1287      +55
  Branches        125      130       +5
============================================
+ Hits            889      942      +53
- Misses          311      313       +2
  Partials         32       32

Impacted Files
Coverage Δ

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
25.00% <12.50%> (+5.00%)
:arrow_up:

...c/main/java/seedu/address/model/person/Remark.java
87.50% <87.50%> (ø)

...java/seedu/address/logic/commands/EditCommand.java
97.10% <100.00%> (+0.08%)
:arrow_up:

...va/seedu/address/logic/commands/RemarkCommand.java
100.00% <100.00%> (ø)

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <100.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
100.00% <100.00%> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
85.71% <100.00%> (+2.38%)
:arrow_up:

...eedu/address/logic/parser/RemarkCommandParser.java
100.00% <100.00%> (ø)

...c/main/java/seedu/address/model/person/Person.java
97.72% <100.00%> (+0.10%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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

Will not be merging this remark command


Migration fails, every remote revision setting lost!

I updated from commit 478d418
# git fetch
remote: Counting objects: 45, done.
remote: Total 45 (delta 17), reused 17 (delta 17), pack-reused 28
Unpacking objects: 100% (45/45), done.
From https://github.com/tleish/redmine_remote_revision_url
   478d418..f0b6c33  master     -> origin/master

Had a look at the commit history and decided it's worth a try to update.
l# git pull
Updating 478d418..f0b6c33
Fast-forward
 README.md                                          |    5 +++-
 .../redmine_remote_revision_url/_settings.html.erb |    9 ++++----
 db/migrate/001_update_saved_settings.rb            |   18 +++++++++++++++
 init.rb                                            |    9 ++++++--
 .../application_helper_patch.rb                    |    8 ++-----
 .../repositories_helper_patch.rb                   |    2 +-
 .../repository_patch.rb                            |   24 +++++++++++++++-----
 7 files changed, 54 insertions(+), 21 deletions(-)
 create mode 100644 db/migrate/001_update_saved_settings.rb

Then I ran the migrations:
# bundle exec rake redmine:plugins:migrate RAILS_ENV=production
(in /var/www/projekte/redmine-3.4)
...
Migrating redmine_remote_revision_url (Remote Revision URL)...
== 1 UpdateSavedSettings: migrating ===========================================
== 1 UpdateSavedSettings: migrated (0.0162s) ==================================

...

Looks good. Then I restarted Redmine.
And all plugin fields in all repositories are empty. Every Git remote is lost!
And I thought there had been proper migrations with #10 ?!

When looking at the database tables I can still see all my settings in the field "extra_info". So it's obviously not completely lost...
How can I restore it?

So I updated the 20 repositories in my DB manually and added a colon : before the extra_info field names. Works again.
However, I still think that this is a regression from previous bugfixes. None of the other extra_info fields are starting with a colon, it's just a plain string token.

I no longer use Redmine and cannot effectively update and maintain this repo.  Let me know if interested in taking it over?


Update README to import IHKeyboardAvoiding

Updated README to add the following line:
import IHKeyboardAvoiding

All CocoaPods need to be imported, this should be obvious


Contribution to the project

Hi,
During my navigation to the tree of the project, I have noticed that there is no CONTRIBUTING file. However, explaining the steps of contribution in that file will increase the number of contributors, enhance the contribution process itself, and save a lot of time and effort for you.
Maybe there are some reasons for that, but I think it's time to create one and explain the contribution steps there.

Good job!
I know that the developers, sometimes, don't have enough time to deal with the documentation, but I think you can find a way to make that step easier. However,  if you would like to accelerate the process, you can use a ready-made template for the CONTRIBUTING.md file. However, I think contributing-template is an excellent place to start.
Thank you.


Debug-Release Difference

Hello,
I am the developer of Happy Can:
https://play.google.com/store/apps/details?id=com.happycenter.happycan.android
After August 1 2019, I developed Happy Can version 15 with 64-bit ABI support.
To do this, I updated LibGDX, Gradle, Android Studio etc.
And for the first time, the game does not run on some devices such as Samsung SM-T113 Tablet and some of Android 4.x devices.
My game is running in debug mode on all devices
but it is not running in release mode on some devices.
https://imgbbb.com/image/LRjHD2
https://imgbbb.com/image/LRj62G

It would be benefits if you added more info.
What does "not running" mean?
Are there any exceptions?
Can you supply a minimal example in which this occurs so we can reproduce it?

I download the game from PlayStore, tab on the icon and it throws that error message:
"Happy Can has stopped":
https://imgbbb.com/image/LRj62G
Exception class: java.lang.verifyError

Get the error log.
Don't defeat the issue tracker template, its there for a reason. If you need personal assistance, please use the forum/irc.


Update get_micro_data.py to comply with TC change in class names

Tax-Calculator PR #1303 changes the names of classes in TC.  OG-USA will need to be updated in it's calls to TC.  These are in the get_micro_data.py script.

Fixed via PR #293


scan should not affect terminal event

Given:
most.create((add, end) => {
  add(2)
  add(4)
  end(6)
})
.scan(
  (acc, x) => {
    return String(acc) + String(x)
  },
  9
)
.observe(console.log)
.then((terminal) => {
  console.log('\nterminal value:')
  console.log(terminal)
})

We get:
9
92
924

terminal value:
924

Why does 9 appear on its own once?
Why does scan never see 6?
Why does then never get 6?

Hey @jasonkuhrt, good questions.  I'll try to tackle them individually:

Why does 9 appear on its own once?

This is how scan is defined.  The initial value always appears in the scan.  For example, here's how scanl is defined in Haskell's Data.List

Why does scan never see 6?

In most.js, the end signal value is not "in the stream".  IOW, the values "in the stream" and the end signal value are two different classes of values, treated differently.

Why does then never get 6?

Scan substitutes the final value it sees "in the stream" as the end signal value.  Looking back on this, it seems like a bug, even though it was intentionally implemented this way.  It's a breaking change, but seems pretty compelling.  We could probably do it for 0.16.
Does that help shed some light?

This is how scan is defined. The initial value always appears in the scan. For example, here's how scanl is defined in Haskell's Data.List

Ok, specifically the q :  cons argument see in https://hackage.haskell.org/package/base-<IP_ADDRESS>/docs/src/GHC.List.html#scanl:
scanlGo f q ls    = q : (case ls of

In most.js, the end signal value is not "in the stream". IOW, the values "in the stream" and the end signal value are two different classes of values, treated differently.

Makes sense, was just checking my intuition was right here. Sounds good.

Looking back on this, it seems like a bug, even though it was intentionally implemented this way. It's a breaking change, but seems pretty compelling. We could probably do it for 0.16.

Yeah lets if possible. I think the reason this should be considered a bug is because of the definition of the terminal event apart from streamed values:

the values "in the stream" and the end signal value are two different classes of values, treated differently.

Thanks for the clarifications! The initial value being appended to the result of scan seems surprising but I'm essentially accepting of it if that's the standard for scan (I'm just not very experienced with scan is all).
I'll rename this issue to be focused on fixing the identified bug.

I think the reason this should be considered a bug is because of the definition of the terminal event apart from streamed values

Agreed

I'll rename this issue to be focused on fixing the identified bug.

Perfect, thanks!
I'll send a PR for this shortly, so you can try it out.


ACT-714 Setup PoC for IPTT report

GIVEN
User has indicators, target period, and results
WHEN
User goes to the Reports section
THEN

User selects Program/Workflow level 1
User selects a target period

Formula for percentage met:
End of program:

if Target > Baseline

actual / target * 100

if Target < Baseline

(baseline - actual) / (baseline - target) * 100

Target periods:

if Target > Baseline

actual / target * 100

if Target < Baseline

(target - actual / target ) + 1) * 100

reference: https://math.stackexchange.com/questions/2215861/calculate-percentage-of-goal-achieved

Hi @Kimaiyo077 and @andrewtpham I would like some clarification on the calculation of the % met (which I believe is the % achieved) and also target values displayed:

The overall target value (End of Programs) only show the aggregate Target of the frequencies whose data has been collected e.g if say an indicator has a Overall target of 100 with a monthly frequency of 10 for 10 months if only 8 months data has been collected the IPTT reports shows an overall target of 80 instead of 100. Is this the intended design?

When we have multiple entries for a certain frequency the target values is added for each entry e.g if we have two entries for the target of say 2, each 1, the target is displayed a 4 with an actual of 2 instead of a target of 2:

When the baseline value is higher that the target, a reducing balance kind of scenario, the % achieved is calculated as expected however I think we will have to analyze how we collect data for this use case. When collecting data with target < baseline we still set the frequencies targets to be a summation of the overall target without the consideration of the baseline value. In this use case the idea is to reduce the baseline value towards the target and therefore the frequencies target should be a summation of the difference between the target and baseline value. Does that make sense?


Changing policy setting interfaces to adopt the new monitor interfaces

…erfaces

This change is

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


allowing slashes in querystring

Hi,
Following url Im using
GET /product/details/:sku_id
in which sku_id can have the "/" forward slash. like GET /product/details/eddd/68-A
I want to consider "edd/68-A" as a string not as a url
but now this taking as a separate URL.
can u help me out.
Thanks.

Hi.
so id must not contain "/" characters, so it's very difficult to parse....
I have 2 solution for you

use query params

GET /product/details
and pass in browser: /product/details?sku_id=edd/68-A. In nodejs: req.query.sku_id.

encode in base64 your sku_id.
You can use nodejs buffer to do this.

> console.log(new Buffer("Hello World").toString('base64'));
SGVsbG8gV29ybGQ=
> console.log(new Buffer("SGVsbG8gV29ybGQ=", 'base64').toString('ascii'))
Hello World

Thank you for the quick smart solution.


chore(protocol-designer): bump to 5.0.0

Overview
Closes #6276
Blocked
Do not merge until we're ready to QA
Though, this may be a prereq to #6183 since migrations need to look at the version number
Changelog
Review requests
Risk assessment

makes more sense to do as part of #6357


Monorepo example does not have the viewsDir option set correctly

Describe the bug
By default, the renderer serves pages from pages/views, however the monorepo example has both views placed directly under pages.
Expected behavior
The example should work out-of-the-box when cloning the repo and trying it out. The solution would be to either place index.tsx and about.tsx under a views subfolder client-side (examples/monorepo/ui/pages), or add the viewsDir: '' option when importing the RenderModule in NestJS (examples/monorepo/server/src/application.module.ts).
Version

next.js: 9.1.6
nest: 6.10.12
nest-next: 9.1.0

Hey thank you for providing a solution, would you be able to create a PR?

Sure, can you grant me the required permissions?

@JHacin You don't need permissions to make a fork and make a PR against this repo.
Anyhow, I just did a PR that fixes the example by modifying the viewsDir option. Thank you for investigating and reporting this. I was aware something is wrong in #40 but didn't have the time to look into it.

Thanks.


RFC: Intrinsics: Define common availability and naming rules

To avoid various naming schemes for intrinsics, let's create
a few rules to streamline them across the RISC-V ISA.

Not related to this PR, but something broader RISC-V wide.

This is not in line with how Intel and ARM intrinsics work. I'd prefer the _ prefixed format that Bitmanip and Scalar Crypto intrinsics used.

I probably should have referenced the origin of this naming convention: https://github.com/riscv-non-isa/rvv-intrinsic-doc/blob/master/rvv-intrinsic-rfc.md#naming-rules
Note, that I have reached out regarding the two naming conventions on tech-toolchain: https://lists.riscv.org/g/tech-toolchain-runtime/message/303

I removed the pull request for cyrpto intrinsics. If you define new ones, please present them to CETG for approval.

I've reworked this RFC to make it compatible with all intrinsics formats that I could find (GCC, LLVM, and scalar crypto).
This requires no modifications to existing specifications but at the same time prevents the definition of new formats.

Should this proposal include the P extension? Looks like the naming in this https://github.com/riscv/riscv-p-spec/blob/master/P-ext-proposal.adoc has __rv_v_ for intrinsics that return packed vectors.

The P extension spec is still not forzen yet, so it could follow the define. @chuanhua

Closing because of lack of consensus.


Add in groups to features feed

Noticed we didn't upgrade the features feed for ajsp when we added the grouping feature.

good catch :+1:

Coverage remained the same at 16.039% when pulling c504e637b458b1d0d8103632f9d31b54d1a278d4 on cleanup-features into bd72544b5d47c01c352c8ee3041e79ec215f86f1 on master.


UHF-6339: Test coverage

Codecov Report

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

@@          Coverage Diff           @@
##             main      #1   +/-   ##
======================================
  Coverage        ?   9.28%
  Complexity      ?      80
======================================
  Files           ?      13
  Lines           ?     237
  Branches        ?       0
======================================
  Hits            ?      22
  Misses          ?     215
  Partials        ?       0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Updated Seaside dependencies.

Use some of the facilities in Buoy

Coverage decreased (-2.0e-05%) to 99.9% when pulling 11d00037b28e8a1c555d5c3df764b5b42ab5817b on update_dependencies into 1ea1a9b9a997f86eef59ff52ac32c1cfa3fd3e55 on release_candidate.


[7.x] Add assertFound method

This PR adds an assertFound method which could be very useful to assert redirection generated by Laravel Validation Logic
It could replace this code in tests:
- $response->assertStatus(302);
+ $response->assertFound();

I'm not sure found is the right word here. It's not really the opposite of "not found".

I used this word to match the http specification, I think it’s easier to understand 🙂

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


fix: Wrong diff for moving a line within a chunk

:tada: This PR is included in version 1.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


boost is broken

Almost certainly linked to my changes... https://godbolt.org/z/hYC4gu

The rsync didn't seem to work, the /celibs directory was empty (but was correctly chown'd). Am manually rsyncing now. See also #1303 and mattgodbolt/compiler-explorer-image#207

May 19 00:50:29 ip-172-30-0-189 compiler-explorer:  rsyncing boost libraries
May 19 00:50:29 ip-172-30-0-189 compiler-explorer:  + rsync -a --chown=ubuntu:ubuntu --exclude=.git '/opt/compiler-explorer/libs/boost_*' /celibs/
May 19 00:50:29 ip-172-30-0-189 compiler-explorer:  rsync: change_dir "/opt/compiler-explorer/libs" failed: No such file or directory (2)
May 19 00:50:29 ip-172-30-0-189 compiler-explorer:  rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1196) [sender=3.1.2]

I bet it's another systemd thing

yeah, it's starting before the file systems are mounted...


Turned off fail-fast for the Rest cross os tests

Turned off fail-fast for the Rest cross os tests
Updated storybook and fixed issue with next.js

https://github.com/storybookjs/storybook/issues/24234

Updated version and depencies for PPaaS

Coverage Tests:

Acceptance Tests:

When you guys ran this did it work for you?

When you guys ran this did it work for you?

It ran on my machine and the PR flow runs the build and tests as well. Locally you can run pr.sh to see if passes.


ci: build tests on windows from now on

@freerdp-bot may I kindly request another test run?

@nfedera great work! Now we can (finally) run tests on all systems \o/.


Installed an app from Windows Store (source) but Winget Upgrade command downloads the update from Winget (source: Github)

Brief description of your issue

Steps to reproduce
Installed Winget from Github (because I'm running Windows 10 20H2 Beta channel and it's not there by default).
I searched for Windows Terminal in Winget using (Winget Search) command.
installed Windows Terminal Preview using (Winget Install) command from Microsoft Store (source)
after 2 days there is a new update for Windows Terminal on Windows Store. I ran (Winget Upgrade) and it identified that there is a new version of Windows Terminal available, but on the source column it shows (Winget) instead of (Microsoft Store)
Expected behavior
I was expecting (Winget Upgrade) command to correctly identify the source of the installed app and download the subsequent updates from the same source, not different source. If I download Windows Terminal from Microsoft Store using (Winget Install) command, (Winget Upgrade) command should update Windows Terminal from Microsoft Store.
Actual behavior
(Winget upgrade) command downloads Windows Terminal from Github (Winget as source)
Environment
[winget --info]
Windows Package Manager v0.2.2941 Preview
Windows: Windows.Desktop v10.0.19042.630
Package: Microsoft.DesktopAppInstaller v1.11.2941.0

Any other software?

Nothing to be conflicting with Winget.

Did I have to use "Winget source update" before doing "Winget Upgrade" ?

As upgrade is a work in progress, we don't yet have the local tracking for the source that a package was originally installed from.  This is coming sooner rather than later, but since it isn't there yet, the correlation is in source order.  So this isn't a bug, so much as a feature not yet implemented.

@JohnMcPMS Thanks for the explanation, I wasn't aware of that.


Add .env.example

Issue
The project currently lacks a .env.example file in both the backend and frontend repositories. This file is crucial for providing contributors with a template of required environment variables, making it easier for them to set up the project locally.
Impact

Setup Difficulty: New contributors may struggle to identify the required environment variables, leading to a frustrating setup experience.
Contribution Barrier: The absence of a .env.example file can deter potential contributors who might abandon the effort due to unclear setup instructions.
Consistency: Having a .env.example ensures consistency in the development environment across different machines and contributors.

Recommended Solution

Create .env.example: Add a .env.example file in both the backend and frontend repositories. List all the required environment variables with placeholder values.

Example Content
Backend .env.example
PORT=3001
DB_URI=database_url
CLERK_SECRET_KEY=client_key

Frontend .env.example
VITE_CLERK_PUBLISHABLE_KEY=publishable_key

Additional Notes
Including a .env.example file will streamline the setup process, making it easier for new developers to contribute to the project. This small addition can significantly improve the developer experience and foster a more inclusive and collaborative environment.

I have added the example .env file to enhance the developer experience.
Thank you for pointing out those issues.


Do we want a timer involved?

A timer can add tension but can also interfere with the game fun by not allowing them time to thing of what happened and the correct answer.  Maybe have one puzzle with a timer.

Do we now include the day API so that they can see the time. how does thta change when they answer. we have to have the current timer.  Then we can blank the timer and then it would be the next day.
This is unnecessary.  We can just keep it to which day it is if this gets complicated.

answer to this generally was no but that will depend on the puzzle.


Does not work with file:// handler

Trying to load a local XML using file://… shows only values as flowed text, i.e. as it does without the extension installed.
Using Chrome 10.0.648.127 on Mac OS X 10.6.6

You should enable "Allow access to file URLs" option in Extensions pane

That should be mentioned in the README :smile:


www/caddy: Domain presentation under Handlers

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[X] The title contains the plugin to which this issue belongs

Describe the bug
Domain under "Handlers" is referencing the Domain description rather than the actual domain name.
To Reproduce
Steps to reproduce the behavior:

Create a Domain with e.g. example.com as domain, and Example as description.
Create a Handler and select Reverse Proxy Domain Example.
In the Handler Overview, the Domain column will have Example rather than example.com.

Expected behavior
Under Reverse Domain Proxy and the Domain column, I would expect to see the Domain rather than the description.
Screenshots

Relevant log files
N/A
Additional context
This might just be by design. If so, feel free to close and I will just re-think about how I go about "Description". 😊
Environment
OPNsense 24.1.4-amd64 with os-caddy version 1.5.2.

This is by design, since you could create:
foo.example.com.443
foo.example.com.8443
foo.example.com.10000
And then there would be just "foo.example.com" 3 times in the handlers.
I tried to show a mix of domain name and port with the model relation field, but I didn't get it to work, so I defaulted for a required description.


json output support for create oidc-provider ... and create operator-roles ...

These commands currently lack support for json output, with #660 + #661 applied, this results in cluster creation --output=json no longer being strictly json. Partially due to no guards on the log statements in the STS block as well as logging in the commands themselves.
https://github.com/openshift/rosa/blob/2eab206e73e074f1150c33a83d3a06ce54e18b6f/cmd/create/cluster/cmd.go#L1803-L1806
https://github.com/openshift/rosa/blob/2eab206e73e074f1150c33a83d3a06ce54e18b6f/cmd/create/oidcprovider/cmd.go#L172
https://github.com/openshift/rosa/blob/2eab206e73e074f1150c33a83d3a06ce54e18b6f/cmd/create/oidcprovider/cmd.go#L226
https://github.com/openshift/rosa/blob/2eab206e73e074f1150c33a83d3a06ce54e18b6f/cmd/create/operatorroles/cmd.go#L329

/remove-lifecycle stale


[Rest API] Change BrokersBase api getActiveBrokers and getLeaderBroker to pure async.

Motivation
Async rest api is more efficient than sync. I think we could refactor all sync api to async.
Modification

Change getLeaderBroker from sync to async.
Change getActiveBrokers from sync to async.

Verifying this change

[x] Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

[x] no-need-doc

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

@Jason918  @liudezhi2098 PTAL :)


New to ReadTheDocs - wish to understand use of Travis CI

Browsing around I see the .travis.yml file. It invokes sphinx-build
My limited understanding is that ReadTheDocs can automatically rebuild from GitHub. Is the use of Travis CI a replacement for that? And if so, is it better? Gives one more control?

No, we just do it for validation.


Supporting wildcard facets like in Algolia

Description
I am trying to migrate InstantSearch-based site from Algolia to Typesense and facing the issue. Algolia supports getting all available facets by using a wildcard:  facets:['*'] (https://www.algolia.com/doc/api-reference/api-parameters/facets/). Typense returns error in this case:
Could not find a facet field named * in the schema.
Any idea how to work around it?
Steps to reproduce

Expected Behavior

Actual Behavior

Metadata
Typsense Version:
OS:

@shasha79 Typesense doesn't yet support facet_by: *, so each facet field should be explicitly listed in the query. See #106.
On a side note, may I know which instantsearch widget this is for?

I am migrating the whole Algolia template, there are many widgets there
https://github.com/algolia/pwa-ecom-ui-template

Dynamicwidgets with facets:[] also fails.  I tried switching the search client between a compiled Algolia version using dynamicwidgets with facets:[] to a TS server, I actually got the TS instantsearch adapter requesting unlisted facets from the TS server.
I'm guessing the  Instantsearch client must have cached some initial facet request and once it was directed at the TS adapter it still worked for the TS server.   I don't know if dynamicwidgets could render, because one of the fields in Algolia sample data was not on the TS server so the adapter tripped over it.

Related to #106

I see that instead of the "*" error, it shows the error 'Field name is not marked as a guy in the schema.'
I using const data = useDynamicWidgets({      guys: ['*']    }) in docker version 0.25.0.rc9
It's not finished, do I need to do something else to make it work?
I'm using react-instantsearch-hooks-web

This is now supported: https://github.com/typesense/typesense-instantsearch-adapter/issues/150#issuecomment-1441201147


Table view shows wrong header properties for commits

It's because of a wrong extra line in the default store

It's because of a wrong extra line in the default store