Try to use saml authentication with AD FS

Hello,
I try to configure SAML with AD FS.
I followed the instruction here  to configure env file with saml but I always get this error :
Invalid issuer in the Assertion/Response (expected 'https://adfs-server/adfs/services/trust/artifactresolution', got 'http://adfs-server/adfs/services/trust')
here is part of my .env config :
AUTH_METHOD=saml2 SAML2_NAME=SSO SAML2_EMAIL_ATTRIBUTE=email SAML2_EXTERNAL_ID_ATTRIBUTE=uid SAML2_DISPLAY_NAME_ATTRIBUTES=firstName|lastName SAML2_IDP_ENTITYID=https://adfs-server/adfs/services/trust/artifactresolution SAML2_AUTOLOAD_METADATA=false SAML2_IDP_SSO=https://adfs-server/adfs/ls/IdpInitiatedSignOn.aspx SAML2_IDP_x509=<cert x509>
I don't understand how I got this url http://adfs-server/adfs/services/trust
Thanks

Did you try to use SAML2_AUTOLOAD_METADATA=true ? Had a similar issue with Keycloak as IDP and with enabling  the autoload function this problem was solved

Yes I tried and I get this error :
Error on parseRemoteXML. The requested URL returned error: 400 Bad Request

https://flareapp.io/share/VmeXpq5Q#F49

Does this url https://adfs-server/adfs/services/trust/artifactresolution provide the endpoint configuration in xml format ?

No didn't, I replaced it with /FederationMetadata/2007-06/FederationMetadata.xml url and put SAML2_AUTOLOAD_METADATA=true and it's OK now :)
Thanks for your help

Thank you @finnwessel  for helping out on this one!

Hello,
I try to configure SAML with AD FS.
Tell me how to solve the problem
I set everything up, but the error keeps popping up
My config  .env:
AUTH_METHOD=saml2
AUTH_AUTO_INITIATE=false
SAML2_NAME=BOOKSTACK
SAML2_EMAIL_ATTRIBUTE=email
SAML2_EXTERNAL_ID_ATTRIBUTE=id
SAML2_DISPLAY_NAME_ATTRIBUTES=firstName|lastName
SAML2_IDP_ENTITYID=https://data-centre.siecom.pro/FederationMetadata/2007-06/Fe$
SAML2_AUTOLOAD_METADATA=true
SAML2_IDP_SSO=https://data-centre.siecom.pro/adfs/ls/
#SAML2_IDP_SLO=https://data-centre.siecom.pro/adfs/ls/
SAML2_IDP_x509="-----BEGIN CERTIFICATE-----
MIIFZTCCBE2gAwIBAgITFgAAAA/znQbmfG8ocwABAAAADzANBgkqhkiG9w0BAQsF
ADA8MRMwEQYKCZImiZPyLGQBGRYDcHJvMRgwFgYKCZImiZPyLGQBGRYIdXNlcmdh
dGUxCzAJBgNVBAMTAkNBMB4XDTIyMDkxNjA2Mzg1N1oXDTIzMDkxNjA2Mzg1N1ow
ADCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBAKrSC+w1KKqIjKRqqB+O
..........................................................."
SAML2_IDP_AUTHNCONTEXT=true
On the AD FS:
The file with metadata https://wiki.sicom.pro/saml2/metadata in AD FS was installed well, there were no problems with this
Set up the attributes as follows:
LDAP                   Party trust wizard
E-Mail-Addresses       -   email
Given-Name        -          lastName
User-Principal-Name -    id
Surname        -               firstName
Gives error on debug:
Invalid ACS Response: invalid_response
Debug:

#0 /var/www/bookstack/app/Http/Controllers/Auth/Saml2Controller.php(119): BookStack\Auth\Access\Saml2Service->processAcsResponse()
#1 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Routing/Controller.php(54): BookStack\Http\Controllers\Auth\Saml2Controller->processAcs()
#2 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Routing/ControllerDispatcher.php(45): Illuminate\Routing\Controller->callAction()
#3 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Routing/Route.php(262): Illuminate\Routing\ControllerDispatcher->dispatch()
#4 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Routing/Route.php(205): Illuminate\Routing\Route->runController()
#5 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Routing/Router.php(721): Illuminate\Routing\Route->run()
#6 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(128): Illuminate\Routing\Router->Illuminate\Routing{closure}()
#7 /var/www/bookstack/app/Http/Middleware/CheckGuard.php(27): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#8 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): BookStack\Http\Middleware\CheckGuard->handle()
#9 /var/www/bookstack/app/Http/Middleware/Localization.php(45): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#10 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): BookStack\Http\Middleware\Localization->handle()
#11 /var/www/bookstack/app/Http/Middleware/RunThemeActions.php(26): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#12 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): BookStack\Http\Middleware\RunThemeActions->handle()
#13 /var/www/bookstack/app/Http/Middleware/CheckEmailConfirmed.php(47): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#14 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): BookStack\Http\Middleware\CheckEmailConfirmed->handle()
#15 /var/www/bookstack/app/Http/Middleware/PreventAuthenticatedResponseCaching.php(21): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#16 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): BookStack\Http\Middleware\PreventAuthenticatedResponseCaching->handle()
#17 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/VerifyCsrfToken.php(78): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#18 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): Illuminate\Foundation\Http\Middleware\VerifyCsrfToken->handle()
#19 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/View/Middleware/ShareErrorsFromSession.php(49): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#20 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): Illuminate\View\Middleware\ShareErrorsFromSession->handle()
#21 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Session/Middleware/StartSession.php(121): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#22 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Session/Middleware/StartSession.php(64): Illuminate\Session\Middleware\StartSession->handleStatefulRequest()
#23 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): Illuminate\Session\Middleware\StartSession->handle()
#24 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Cookie/Middleware/AddQueuedCookiesToResponse.php(37): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#25 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse->handle()
#26 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Cookie/Middleware/EncryptCookies.php(67): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#27 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): Illuminate\Cookie\Middleware\EncryptCookies->handle()
#28 /var/www/bookstack/app/Http/Middleware/ApplyCspRules.php(33): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#29 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): BookStack\Http\Middleware\ApplyCspRules->handle()
#30 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(103): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#31 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Routing/Router.php(723): Illuminate\Pipeline\Pipeline->then()
#32 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Routing/Router.php(698): Illuminate\Routing\Router->runRouteWithinStack()
#33 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Routing/Router.php(662): Illuminate\Routing\Router->runRoute()
#34 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Routing/Router.php(651): Illuminate\Routing\Router->dispatchToRoute()
#35 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(167): Illuminate\Routing\Router->dispatch()
#36 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(128): Illuminate\Foundation\Http\Kernel->Illuminate\Foundation\Http{closure}()
#37 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Http/Middleware/TrustProxies.php(39): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#38 /var/www/bookstack/app/Http/Middleware/TrustProxies.php(41): Illuminate\Http\Middleware\TrustProxies->handle()
#39 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): BookStack\Http\Middleware\TrustProxies->handle()
#40 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/TransformsRequest.php(21): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#41 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/TrimStrings.php(40): Illuminate\Foundation\Http\Middleware\TransformsRequest->handle()
#42 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): Illuminate\Foundation\Http\Middleware\TrimStrings->handle()
#43 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/ValidatePostSize.php(27): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#44 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): Illuminate\Foundation\Http\Middleware\ValidatePostSize->handle()
#45 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/PreventRequestsDuringMaintenance.php(86): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#46 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(167): Illuminate\Foundation\Http\Middleware\PreventRequestsDuringMaintenance->handle()
#47 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(103): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline{closure}()
#48 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(142): Illuminate\Pipeline\Pipeline->then()
#49 /var/www/bookstack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(111): Illuminate\Foundation\Http\Kernel->sendRequestThroughRouter()
#50 /var/www/bookstack/public/index.php(53): Illuminate\Foundation\Http\Kernel->handle()
#51 {main}

Tell me what I'm doing wrong?


Actual folder or file path in html title

First: many many thanks for the file manager. Great work.
At the moment, if I open filemin in different browser tabs to be able to switch easily between different folder views and/or file editors, I always have the html titles <title>Filemin</title> or <title>Edit file</title>.
Would it be possible to add the actual path to the title, e.g. <title>/path/to/folder - Filemin</title> or
<title> /path/to/folder/filename - edit file - Filemin</title>, like here on Github?
It would make it easier to find the right tab while hovering over them.

Nice idea, will add this.

Thanks. :-)

It turns out to be not easy task, maybe Jamie can advice something?

This is only really possible if the theme doesn't use frames. Otherwise a module (like Filemin) can only set the title on it's inner frame, which doesn't appear in the browser.

Not sure where is the problem here? I will add this to my todo and try to embody this feature.

Sasha, you can easily update title on the parent frame (while being in child's) by doing something like this:
window.parent.document.title = '/home/example.com/public_html - Filemin – Webmin version, hostname'

Before you do so, you can save the current title and it's hostname to localStorage and then use it to append it to the constantly updated path.
Example:
localStorage.setItem("initial_title", window.parent.document.title)

Later use it as such:
/home/example.com/public_html - Filemin – localStorage.getItem("initial_title")

This is a great idea. I will make theme update the path for all modules on next release.
P.S. Don't forget to flush the Filemin title when user switches to the other module. I think it's better done using the theme as Filemin will not be able to return title back to its initial state, unless tricky usage of onbeforeunload() function.

This functionality (Document title for the browser's tab) will be available in Authentic Theme 17.03 and will work for all other modules/pages across Webmin/Virtualmin/Cloudmin + will show path for Filemin (not only module name). In couple of days.
Thanks for the nice advice on this feature.

It's already out in version 17.03. Please update.
I hope it works well for you!

@qooob Thanks a lot. Very good work. It will help me a lot.
@Real-Gecko
I don't know if I'm the only one. But when I open filemin in different tabs, very often I open only the inner frame to be able to use the whole window width.  In this case the tab doesn't have the path in the title.
I made some tests and it doesn't look difficult.
E.g. index.cgi, line 18, this should do it:
&ui_print_header(undef, $path." Filemin", "", undef, 0 , 0, 0, "<a href='config.cgi?path=$path' data-config-pagination='$userconfig{'per_page'}'>$text{'module_config'}");
or edit_file.cgi, line 10, this one:
&ui_print_header(undef, $path.'/'.$in{'file'}.' - '.$text{'edit_file'}, "");
What do you think? Is that overdone?

I have plans to later add tabs support for Filemin (like Nautilus or Double Commander). You will not need extra window then.

Version 1.0.0 show all editable files in named tabs. Go grab it.


chore(package): update zone.js to version 0.7.4

https://greenkeeper.io/

Current coverage is 100% (diff: 100%)

Merging #80 into master will not change coverage

@@           master   #80   diff @@
===================================
  Files           8     8
  Lines         138   138
  Methods        33    33
  Messages        0     0
  Branches       16    16
===================================
  Hits          138   138
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 01f10e8...1021be1


Add option to preserve existing line breaks

Sometimes I have more complex rules that I break up into smaller chunks on each line to make them more understandable.
Would be nice to have an option to preserve that.

That could be tricky, because line break adjustments is the bread-and-butter task of a formatter. It's probably easier to just not format the file then.


Add "then" as alias for "and_then"

Just kidding, then is a keyword just like and. This will have to be limited to the JS port.


feat/secretstore

Description
Please explain the changes you've made
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #43
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

@pravinpushkar With 1.12 now out the door, we should be able to pick this PR back up, right?

@pravinpushkar With 1.12 now out the door, we should be able to pick this PR back up, right?

@philliphoff Yes, I will plan to complete this in coming weeks.

@philliphoff Good to merge?


Container/Pod events with :fieldpath=>"implicitly required container POD"

I thought from the comment "'Created', 'Failed', 'Started', 'Killing', 'Stopped' and 'Unhealthy' are in fact container related events, returned as part of a pod event." those only ever arrive as container events.
But I'm seeing both CONTAINER_CREATED and POD_CREATED, CONTAINER_STARTED and POD_STARTED, CONTAINER_KILLING and POD_KILLING in my log:
https://gist.github.com/cben/2c9784f1c1259318a394314fa5a56aef
Our event catcher classifies the as POD_ events based on fieldpath.
All these cases have :fieldpath=>"implicitly required container POD".  I'm not yet sure what this implies.  Are these actual pod events vs container-events-with-unknown-container?
Originating k8s code: https://github.com/kubernetes/kubernetes/search?utf8=✓&q=ImplicitContainerPrefix
Currently POD_(CREATED|FAILED|STARTED|KILLING|STOPPED|UNHEALTHY) become an EmsEvent but do not trigger ems refresh are not propagated to MiqEvent / policy.  There is no "pod started" event in UI.

[ ] add example to event_catcher_mixin_spec.rb
[ ] understand / ask k8s if these are actual pod events.  If yes:

[ ] make them trigger ems refresh (perhaps even if no?)
[ ] make them trigger policies
[ ] define in miq_event_definitions.csv

@simon3z Is this pressing to understand and/or fix for darga-3 release?
@dkorn Do you know more about these events?
@miq-bot add-label providers/containers

@simon3z Is this pressing to understand and/or fix for darga-3 release?

@cben if the effect is that ManageIQ can catch more (nonexistent) events then we have to fix this but not in a rush.
Is there any other more serious side-effect?

The effect is either:

there exist pod events we didn't cover — e.g. Pod Started — currently not
in Events list, can't define policies for them.
these are actually container events for which kubernetes is failing to
report the container.
no big deal until we support policies for container events.

I suppose I should figure out which one...

there exist pod events we didn't cover — e.g. Pod Started — currently not
in Events list, can't define policies for them.

@cben Pod Started? That doesn't even exists in kubernetes. (Nor we are interested in adding new ATM ones there).

@miq-bot assign cben


add XviD AVI option to --recode-video

Some TVs, on friend's homes and schools, only support DivX or XviD videos. So, would be nice to have this feature on youtube-dl.

#2817

#2910


New package: Rizin.Cutter in version 2.2.1

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

xref: https://github.com/rizinorg/cutter/issues/3056
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/109817&drop=dogfoodAlpha

@microsoft-github-policy-service agree

@wingetbot waivers Add Validation-Executable-Error

@stephengillie / @denelon - I'm not sure the Validation-Executable-Error should have been waived here.
Although it may install and run fine on Windows 11, on Windows 10 machines it has a dependency on MSVCP140 and VCRUNTIME140_1 and VCRUNTIME140 - that is to say, has a dependency on Microsoft.VCRedist.2015+.x64

After installing the dependency on Win10, it runs fine -

@Trenly We validate on Windows 11 by default unless the minimum OS version specifies Windows 10 if I'm not mistaken.

Let me confirm with the team.

Confirmed with @denelon the Pipelines run Win10 by default and Win11 only if specified - It's possible that manual validation uses Win11 instead of Win10

Sorry for the delay. I was offline for some mandatory in person training.
I was wrong.
We default to a Windows 10 VM during validation unless the manifest states a requirement for Windows 11.
It appears not all users who were performing manual validation were also defaulting to a Windows 10 VM. This will be communicated to the team today and added to our process documentation.

Could you explain in brief how this affects future contributions? Is there a way to express the dependency on Microsoft.VCRedist.2015+.x64?
Thanks anyway for pointing it out!

We don't yet have support for dependencies as a stable feature. We're still looking at a few challenges related to reboots that can get triggered by dependencies being upgraded. Yes, the dependency can be identified in the manifest, but we're holding off on accepting packages that fail during install if the dependency isn't present.

We don't yet have support for dependencies as a stable feature. We're still looking at a few challenges related to reboots that can get triggered by dependencies being upgraded. Yes, the dependency can be identified in the manifest, but we're holding off on accepting packages that fail during install if the dependency isn't present.

Shame, but thanks so much for the clarification!

I've got it in the 1.6 milestone so we're aiming for September.

@assarbad @Trenly you might be interested in https://github.com/microsoft/winget-pkgs/issues/115801
BTW, if something is needed from our side - please let us know by opening an issue at https://github.com/rizinorg/cutter/issues


[Frequency] Added an option to set a frequency offset

Some CC1101 are crappy and don't work exactly on 868.3 Mhz although set. Mine e.g. needs a setting of 868.35 to see my sender properly. This adds a parameter to begin() to specify the offset.

@ThomasKeil Thanks for your contribution!


Re-enable tests

We should continue to test the Aiven provider to maintain a quality bar, and monitor undeleted resources or excess spend on the account going forward.

@AaronFriel ping - any plans to move this forward or it moves somewhere on the board? Thank you.

@t0yv0 thanks for the ping. It looks like tests failed, rerunning then will diagnose.


Migrate formatting to use ruff

Description
Use ruff. Don't change much else.
Todos
Notable points that this PR has either accomplished or will accomplish.

[x] Migrate formatting

Status

[x] Ready to go

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

Project coverage is 90.50%. Comparing base (9755af2) to head (e803e65).

Additional details and impacted files


Support multi-architecture images with build-docker-create

Is your feature request related to a problem? Please describe.
Build and push multi-architecture images using buildx and add resulting artifacts to an artifactory build
Describe the solution you'd like to see
build-docker-create will support both traditional and multi-architecture images:
jf rt build-docker-create --server-id=<serverId> --build-name <buildName> --build-number <buildNumber> --project <project> <target-repo> <imageNameAndVersion>
e.g. Using environment variables for serverId, buildName, buildNumber, and project
jf rt bdc temp-docker-local repo.mycorp.net/temp-docker-local/org/project:2.3.1
Describe alternatives you've considered
Current build-docker-create command with --image-file.  Unfortunately, a lot of work for the CI build system to hunt down the required arguments and does not support multi-image.
Additional context
Current --image-file option is inconvenient to use.  Also, it is not necessary to provide the digest to find the layers for a traditional image.  You cannot specify a digest to find a multi-architecture index.

Can anyone please provide a detailed example of using this command? the --image-file is not clear, whatever used I get an error "no such file or directory". thanks

Can anyone please provide a detailed example of using this command? the --image-file is not clear, whatever used I get an error "no such file or directory". thanks
Yeah, it's an extremely silly implementation. This should all be automated and collected properly when using the JFrog CLI tool and while wrapped as a step in Jenkins for ex.

What I'm doing is when you build the image using docker buildx you add --metadata-file to the docker buildx command to output the build metadata to a physical .json file. e.g.
docker buildx build --metadata-file image-metadata.json ...
Then when you run the JFrog BuildInfo step (build-docker-create, or bdc for short) you need to convert this .json data to a format that JFrog CLI requires, i.e. <IMAGE-TAG>@sha256:<MANIFEST-SHA256>
e.g.
jq -r '.["image.name"] + "@" + .["containerimage.digest"]' image-metadata.json > jfrog-image-metadata
then you run the JFrog CLI bdc command:
jf rt build-docker-create <docker_repo> --image-file jfrog-image-metadata
and finally you push this to Artifactory
jf rt build-publish
If you're on a CI system like Jenkins you probably should add --build-name and --build-number variables to the commands as well (should also be implicitly set).


Prevent repeating frame indices

Fix for Issue #1158.
Problem
stream.thread_type.FRAME threading causes an incorrect series of repeating frame indices at the end of most videos. This is because a list of frames is returned by CodecContext._send_packet_and_recv . frame.index corresponds to the most recent frame received from the decoder. Therefore all frames in the list are assigned the same index as the final frame in the list.
Solution
I solved this issue by detecting whenever multiple frames are returned and then correcting the frame.index based on the frame's position in the list.
Other changes
AVCodecContext::frame_number is deprecated. I switched to AVCodecContext::frame_num.
EDIT: The deprecation was issued in ffmpeg 6.0, so I reverted back to using AVCodecContext::frame_number.
Testing
The test iterations may be overkill, but it only increased the test run time from 6 seconds to 6.5 seconds on my machine.
Future considerations
It is important to note that frame.index is affected by CodecContext.skip_frame. Skipped frames will not increment the frame index. I don't know if this is intended or desired behavior. Perhaps we should consider two different frame properties.
1: A property for the number of frames returned by the decoder (this is the current behavior).
2: A property for the frame index relative to the total number of frames in the video. I assume this can be determined by a timestamp?

I would like there to be performance benchmarks ran on decode. Eyeballing the code, it looks this costs more cpu time but the PR is still good if the performance difference isn't measurable.
I'm curious why you choose to extract the logic of _correct_frame_indices to it's own function instead of inlining it in decode. This again feels like a performance thing but I would like measurable results. Actually, since _setup_decoded_frame is only one line now, that could be inlined as well.

I'm very doubtful about this PR, where is the behaviour you describe documented?

Performance Benchmarks
@WyattBlue I ran some rudimentary performance benchmarks using the following code:
import av
from av.datasets import fate as fate_suite
import time

PATHS = (fate_suite("h264/bbc2.sample.h264"), fate_suite("h264/interlaced_crop.mp4"), av.datasets.curated("pexels/time-lapse-video-of-night-sky-857195.mp4"))

def test_decode_frame_indices():
    for path in PATHS:
        for thread_type in ("NONE", "AUTO"):
            for thread_count in range(4):
                for skip_type in ("NONE", "NONKEY"):
                    frame_list = []
                    compare_list = []
                    frame_count = 0
                    #print('\n', path, thread_type, thread_count, skip_type)
                    with av.open(path) as container:
                        stream = container.streams.video[0]
                        stream.thread_type = thread_type
                        stream.thread_count = thread_count
                        stream.codec_context.skip_frame = skip_type
                        for packet in container.demux(stream):
                            for frame in packet.decode():
                                frame_list.append(frame.index)
                                compare_list.append(frame_count)
                                frame_count += 1
                    print(frame_list == compare_list)

start_time = time.perf_counter()
test_decode_frame_indices()
print(time.perf_counter() - start_time)

I did 3 runs on each branch and got the following average durations:
main: 5.705269041995052 seconds
fix_frame_indices: 5.665727951010922 seconds

If you were referring to more sophisticated benchmarking, I'll need some help to get started.
Function vs Inline
The only reason I made a function was to make it slightly more organized. If you prefer that I change to inline, that's perfectly fine with me.

AVFrame
@jlaine I looked through AVFrame Struct Reference and found display_picture_number, which looked promising. However, when I exposed it in pyav it always returned 0.
It is also deprecated in ffmpeg 6.0.
This issue seems to also conclude that display_picture_number is not trustworthy.
Docs
frame.index is found in the pyav.org docs here and here.

If the point is purely to have a running index, what is gained over doing:

for (index, frame) in enumerate(container.decode(video=0)):
   ...

@jlaine Yeah, that's a good point. However, since frame.index is on the pyav.org homepage, I'm sure many people expect it to be available. I would really like to provide a replacement before removing it.
It seems these problems arise from the fact that frame.index is not an actual index. enumerate() and frame.index work correctly only when iterating every frame. They will break if using CodecContext.skip_frame or container.seek.
How about this solution:

The current functionality of "number of frames decoded so far" is moved from frame.index to a new property called CodecContext.frame_num. That way it matches ffmpeg and the original functionality is still available to users.

Create frame.guessed_index which is calculated from frame.pts * stream.guessed_rate * frame.time_base or similar (I still struggle with the intricacies of time base).

The problem is frame.pts, stream.guessed_rate, and frame.time_base are not always available in all media. Example: fate_suite("h264/bbc2.sample.h264")
If frame.guessed_index  isn't available then the original intent of this PR can be implemented by the user like this:
for packet in container.demux(stream):
    frame_list = packet.decode()
    correction = len(frame_list)
    for frame in frame_list:
        corrected_index = stream.codec_context.frame_num - correction
        correction -= 1
        print(f"{corrected_index=}")

Ideally we would be able to get a frame's index from ffmpeg, but I can't find a way to do it at the moment. I think this new solution solves many problems:

The original functionality of frame.index remains (elsewhere).
The repeating frame indices from #1158 can be solved.
Pyav matches ffmpeg's structure and property names.
Provide a way to get the frame index even when CodecContext.skip_frame = "NONKEY".
The name frame.guessed_index makes it clear that the value is not guaranteed to be accurate.

I agree with jlaine. We should deprecate and remove frame.index. There is no such concept in ffmpeg. Putting in a guessed_index would only further complicate an already complex project.


Remove ECS Prometheis

Why I am making this change
We've switched the Prometheis fully over to EC2, so we can remove the ECS config for Prometheus.
For the moment, AlertManager, PaaS Proxy and NGinx are still on ECS, so we can't remove everything.
Paired with @kentsanggds.
https://trello.com/c/DWWeMBr8/662-turn-off-ecs-prometheus-m

I didn't do anything new apart from edit the description slightly and remove a file (we didn't need alert-config.tf as those are the ECS alerts), so I'll merge this with Matthew's review.


snippets draft update

Describe your changes
Change %sqlcmd snippets output into displaying 1. snippet name 2. snippet query 3. snippet table.
Still working on 1. figuring out how to retrieve SqlMagic.displaylimit 2. adding test functions 3. update documentation if needed, but I want to know whether it's on the right direction or not.
Issue number
Closes #648
Checklist before requesting a review

[x] Performed a self-review of my code
[x] Formatted my code with pkgmt format
[x] Added tests (when necessary).
[x] Added docstring documentation and update the changelog (when needed)

:books: Documentation preview :books:: https://jupysql--716.org.readthedocs.build/en/716/

@edublancas
This is the current output of %sqlcmd snippets. Am I on the right track?

Another question is can we assume %sqlcmd snippets will always be called inside a jupyter notebook or do we also think about a user running it in a python script? Asking this question because of figuring out a correct way to get displaylimit if the user specifies one.

@bbeat2782 you don't need to query the tables. instead, you can create a table that displays the snippet names
e.g.
----------------
| some snippet |
----------------
| another           |
----------------

you can use this: https://github.com/ploomber/jupysql/blob/4b938edcdea296dcf1c9585b6ed52fadf50835eb/src/sql/display.py#L10

@bbeat2782 you don't need to query the tables. instead, you can create a table that displays the snippet names
e.g.
----------------
| some snippet |
----------------
| another           |
----------------

you can use this:
https://github.com/ploomber/jupysql/blob/4b938edcdea296dcf1c9585b6ed52fadf50835eb/src/sql/display.py#L10

@edublancas Oh that is what you meant when you said table. Thanks. Will make the change.

Output of %sqlcmd snippets when no snippet is stored.

Output of %sqlcmd snippets when chinstrap and chinstrap_sub are stored.

The above images are screen captured images from snippet_display_improvement.ipynb.zip


Data tab

@viktor-yakubiv I removed react-table as it was not necessary after all. Could you please have a look at this proposal. Originally I was working on variant when only visible rows were rendered, but I had to scratch it as I had troubles with Intersection Observer. I asked a question on StackOverflow. You may find it here.
Currently, table works as an infinite scroll. Rows are added on-demand which may cause memory issues. I will keep it as-is for now. Hopefully, somebody will answer my question on SO and I will be able to improve it.
Missing things:

[ ] ordering
[ ] rows expandable on click

Are you still working on this?

@viktor-yakubiv yes

I forgot to mention that we have to merge this with #25 somehow together in order because there may be conflicts.

Not that many conflicts as I expected. #25 merged.
I would accept all from master if possible. I am working on CSS refactoring.


no import button

I don't have any import button to download the models. I see a details button which works, and link with the Thingiverse id
I am using cura 4.2.1 and v1.1.4 of the plugin

Hi,
On the details page there should be a list of associated files, each with a download button. But this list is filtered to only show STL files, maybe the model you're looking at has no such files? Can you share a screenshot of the page?

stl vs obj filtering is exactly the issue.  Is there a reason you filter out 'obj' files? I am far from an expert but from a functionality standpoint the plugin should be able to support any file type that thingiverse will let you download that can be imported into cura. Is that a possibility? If not maybe I can add a little warning that says hey this file needs to be manually downloaded.

A known stl based model:

Is this line the filter?
https://github.com/ChrisTerBeke/ThingiBrowser/blob/1f87d0943c85c6af11d9adf428e6e565981dca47/views/ThingFilesList.qml#L18
I could try hacking away at this but I need a little help with building the plugin into a .curapackage.

Thanks for testing that. I'll try and see if I can release a new version of the plugin with support for other file types that Cura supports.

PR #10 adds support for other file types (all that Cura can import). I'll submit a new package on the marketplace and wait for it to be reviewed.


Correct serialized property name on AsciiFoldingTokenFilter

Closes #2892

Backported to 2.x and labelled 2.5.7

Backported to 5.x and labelled 5.6.0


try enabling darcs 2.12.1

Thanks

Unfortunately the testsuite is failing... see http://bugs.darcs.net/issue2507

So I disabled darcs again for now.

I thought some of these problem tests were disabled in the release 2.12 branch, but I guess not.
Last time this happened, I believe the darcs tests were disabled in build-constraints.yaml - could we do that for now ?

darcs-2.12.2 is on Hackage, which claims to fix the test suite.

Nice, trying again.

Incidentally @juhp, shouldn't the Travis job have notified me of the problem with tests ? What did you do differently from me, to see it ?

@simonmichael unfortunately not - the travis job only does dependency version changing.
For stackage snapshots (nightly and lts) we actually build packages, but we don't have a story yet for building packages before adding them.
In retrospect i was pondering just to let the darcs fail in stackage since darcs has been in Stackage before, but let's see if the new release is okay.


Create bit ape clup

Bit ape  is an exciting Bitcoin NFT project that combines traditional digital assets with cute ape elements, attracting the attention and enthusiasm of countless people. Each bit  ape has a unique look and features that allow people to find a balance between artwork and collectibles. Their adorable appearance and unique nature lends a unique sense of honor and social value to their holders. This project is not only a collection of digital assets, but also a community where people can share their interests and passions and explore this emerging field together. The uniqueness and promise of the Bit ape makes it an important icon in the field of digital art and assets

close pr. reason: imitation of bayc


Feature/padding utilities

affects: @buildit/gravity-ui-web
adds a suite of padding utility classes
Description
Currently we have a problem with the footer not having enough space above it. After much discussion in person and on this PR #323 I've added a function that will generate a suite of padding utility functions. I've added a class to the main element which will add the padding we need. They should be useful in the future for other cases where extra/less padding is required.
Checklist
SASS

[x] The SASS (and related pattern files) follow the naming conventions, architecture and structure outlined in the documentation.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix issue where username/password are not encoded for basic auth

Fixes #4789
The golang SetBasicAuth function does not escape the params as per the spec - so we need to do this to ensure special characters are handled correctly.

Codecov Report

Merging #4796 (7b4e7d3) into master (90a8ea5) will decrease coverage by 26.13%.
The diff coverage is 50.00%.

@@             Coverage Diff             @@
##           master    #4796       +/-   ##
===========================================
- Coverage   53.77%   27.64%   -26.14%
===========================================
  Files        1122       59     -1063
  Lines       38759     4619    -34140
  Branches     4931        0     -4931
===========================================
- Hits        20844     1277    -19567
+ Misses      17737     3164    -14573
  Partials      178      178


[BUG] Requesting multiple AssetPathEntites at the same time is very slow on iOS

Describe the bug
For our app, we need to display a calendar that marks each day that has a video in the devices gallery.
Therefore, we request an AssetPathEntity for each widget like this:
final albums = await PhotoManager.getAssetPathList(
      type: RequestType.video,
      hasAll: true,
      filterOption: FilterOptionGroup(
        containsLivePhotos: false,
        orders: const [
          OrderOption(type: OrderOptionType.createDate, asc: false)
        ],
        createTimeCond: DateTimeCond(
          min: date.startOfDay(),
          max: date.endOfDay(),
        ),
        videoOption: FilterOption(
          durationConstraint: DurationConstraint(
            min: minDuration,
          ),
        ),
      ),
    );
    return albums.firstWhereOrNull((album) => album.isAll);

Using FutureBuilders for the widgets works like a charm on Android, on iOS however, this process is extremely slow.
Requests for about 20 days simultaneously takes about 40 seconds, then all the Futures complete at the same time. Requesting each day sequentially still takes about 2-3 seconds per day.
On Android, all requests complete virtually instantly, even when requesting AssetPathEntities for a lot of days at the same time.
Expected behavior
Similar speeds on iOS and Android.
Flutter version
[✓] Flutter (Channel stable, 2.10.1, on macOS 12.2.1 21D62 darwin-arm, locale en-DE)
• Flutter version 2.10.1 at /Users/tim/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision db747aa133 (12 days ago), 2022-02-09 13:57:35 -0600
• Engine revision ab46186b24
• Dart version 2.16.1
• DevTools version 2.9.2
Smartphone (please complete the following information):

Device: iPhone 13, iPhone 8 (physical devices)
iOS 15.3.1
Version ^1.3.10 (also reproducible in 2.0.0--dev.11)

Any screen recordings?

Also, no matter how fast the plugin on Android, your implementation seems to be strange...

A more reasonable implementation would be:

Define a Map<DateTime, int> for assets count
Load the isAll path only once
Load assets with paginations of the calendar, then increase the count of each day.

Meanwhile, I'll try to figure out if any latency we can improve.

Performing hot reload...
Syncing files to device Alex’s iPhone...
Reloaded 1 of 899 libraries in 378ms.
collection name = Panoramas, count = 0
collection name = Videos, count = 3
collection name = Favorites, count = 0
collection name = Time-lapse, count = 0
collection name = Hidden, count = 1
collection name = Recently Deleted, count = 3
collection name = Recents, count = 3
collection name = Bursts, count = 0
collection name = Slo-mo, count = 1
collection name = Recently Added, count = 3
collection name = Selfies, count = 3
collection name = Screenshots, count = 3
collection name = Portrait, count = 3
collection name = Live Photos, count = 3
collection name = Animated, count = 0
collection name = Long Exposure, count = 0
collection name = Unable to Upload, count = 0
collection name = RAW, count = 0
collection name = 抖音, count = 2
collection name = 微博, count = 1
collection name = QQ, count = 1
[PhotoManager 📣] 2022-02-22T11:44:13.860127 - Obtain path list duration: 0:00:00.121575

A more reasonable implementation would be:
Define a Map<DateTime, int> for assets count
Load the isAll path only once
Load assets with paginations of the calendar, then increase the count of each day.

On Android, this is is actually less sensible, since at that stage we only care whether there are any assets at all that match the given filter criteria, so we want to use the assetCount getter, which is synchronous once you obtained the AssetPathEntity.

Can you run the latest example and filter logs by PhotoManager 📣?

Another option is to explicitly mark containsModified to false to see if it makes any differences.

Closing, as we don't have a valid further response about the fix.


Add break line on message input

When typing long paragraphs, it appears as a long straight line.
The input text field needs to grow vertically as it overflows / wraps.

Solution in order to support Mac:
if ((e.keyCode == 10 || e.keyCode == 13) && (e.ctrlKey || e.metaKey))


Minimum opset support & better frontend / shape inference error messages

Right now, we get very cryptic error messages, esp when we don't support an op. If it passes the frontend, then we get errors in the shape inference; if it does not pass the frontend, often we get an error about a type being wrong or something like that (often a symptom that the newer opsets use a different type for the same args).
My suggestions are 3 folds

Create a minimum supported opset (maybe 3, 5, 9?) It does not mean that we refuse older ones, it will just trigger better error messages.
When we encounter an op below the minimum supported opset, we give ONCE a warning "warning, this model use operations that are older than opset X, please consider up converting your model"
When we encounter an error for an op below the minimum, we give an error "Error, encountered an error parsing op X with opset Y which is older than the minimum supported opset Z. Please regenerate and/or up convert your onnx model"

In addition, I suggest that we develop a "macro" to form the error messages. Something of the sort
ErrorMessage(user, developper)
and for the debug version and/or when we add the debug flag, we get the full message. That would allow us to have detailed messages for us, and simpler messages for users. Maybe we can just have 2 error commands, might be convenient to have only one.
Fire away your comments

@tungld As seen in #1825, we get errors parsing models with too old opsets. Could you check if we give a warning like suggested here?
Much appreciated


[QUESTION] Is ORM supposed to be Faster than Model Cache?

Before you submit this issue, you has been search all existed issues and search the documentation

[x] I've been search all existed issues
[x ] I've been read all documentation

Describe your question
So, I'm running wrk -t 10 -c 100 http://localhost:9501
 public function index()
    {
        $user = $this->request->input('user', 'Hyperf');
        $method = $this->request->getMethod();
        $userIds = [
            1,
            2,
            3,
            4,
            5,
            6,
            7,
            8,
            9,
            10,
            11,
            12,
            13,
            14,
            15,
            16,
            17,
            18,
            19,
            20,
        ];
        return [
            'method' => $method,
            'message' => "Hello {$user}.",
            'users' => \App\Model\User::findManyFromCache($userIds)
            // 'users' => \App\Model\User::whereIn('id', $userIds)->get()
        ];
}

When I run findManyFromCache - I get 2700 reqs/sec, if I do it the other way I get 3400 reqs/sec.
If the cache is slower, what's the point? wouldn't it be better just to go w/ raw queries/orm?

When i run tests in cache, I get 1169 reqs/sec, If I do it in database, I get 924 reqs/sec.
My mysql and redis all run in docker.


AssertionError

Assertion Error

Trying to replicate the experiment from here, https://github.com/czhu95/ternarynet
a. NameError: name 'StatPrinter' is not defined
2 b. Commenting the StatPrinter() callback gives the following error

Traceback (most recent call last):   File "./tw-cifar10-resnet.py", line 213, in <module>     config = get_config(args.n)   File "./tw-cifar10-resnet.py", line 199, in get_config     max_epoch=400,   File "/home/username/.local/lib/python3.6/site-packages/tensorpack/train/config.py", line 111, in __init__     assert_type(callbacks, list)   File "/home/username/.local/lib/python3.6/site-packages/tensorpack/train/config.py", line 95, in assert_type     assert isinstance(v, tp), v.__class__ AssertionError: <class 'tensorpack.callbacks.group.Callbacks'>
4. Your environment:
+ Python version. 3.6
+ TF version: 1.6
+ Tensorpack version:0.8.5

The ternarynet project used very old tensorpack and tensorflow that are not compatible with what they are now. It has actually included a version of tensorpack in its repo. It's not expected to work with latest tensorpack.

FYI I've included the implementation of Trained Ternary Quantization paper inside tensorpack examples now: https://github.com/ppwwyyxx/tensorpack/tree/master/examples/DoReFa-Net


[Coral-Trino] Modify transformation for CURRENT_TIMESTAMP

What changes are proposed in this pull request, and why are they necessary?

In Hive, CURRENT_TIMESTAMP returns a UTC timestamp, whereas in Trino, it returns a timestamp based on the session's timezone. Therefore, to be compatible with Hive query result, AT TIME ZONE 'UTC' should be added to the translated Trino SQL. For example, SELECT CURRENT_TIMESTAMP is transformed into SELECT CAST(CURRENT_TIMESTAMP AT TIME ZONE 'UTC' AS TIMESTAMP(3)).
How was this patch tested?

Modified unit tests.

Could you evaluate the Spark behavior in case Spark is used on the LHS? What triggered this issue? Is it still required? FYI @phd3.


Disable ClrStack/Walk Events During Rundown

ClrStack/Walk events are unintentionally enabled during rundown if any of the rundown suppression arguments are used.  This can result in a large number of stacks that can't be matched to CLR events and overall slower conversion of an ETL file to ETLX.
An example configuration where this occurs is with /GCCollectOnly traces where the fully expanded commandline includes /NoRundown:
PerfView /RestartingToElevelate:collect /DataFile:PerfViewGCCollectOnly.etl /BufferSizeMB:256 /StackCompression /KernelEvents:Process,ImageLoad /ClrEventLevel:Informational /ClrEvents:GC /NoGui /SessionName:PerfViewGCSession /NoRundown /GCCollectOnly Collect

These events are not needed during rundown, but are enabled because rundown for V2.0 runtimes occurs via the public provider instead of the rundown provider.  The fix is to switch the non-existent starting rundown keywords to be a value that doesn't intersect with anything in the public provider or the rundown provider.
cc: @Maoni0

thanks for fixing this!


Remark comments

Description
Fix https://github.com/Azure/autorest.csharp/issues/2288
update  in reference docs:

Remove <remarks></remarks> block for Error Response.
Add link to REST API documents for the given operation
Update <remarks></remarks> for the Request Body schema to indicate whether the parameter is required or optional.
Update <remarks></remarks> for the Request Body schema to indicate whether the parameter is required or optional.
Update <remarks></remarks> for the Request Body schema with the parameter descriptions
Update <remarks></remarks> for the Response Body schema with the parameter descriptions.
Add Polymorphic types

Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

I re-gened all the RPs in the azure-sdk-for-net to reflect this reference doc modification. The PR is https://github.com/Azure/azure-sdk-for-net/pull/29264. Please take a look. Thanks

@chunyu3, would you please update the PR description so we can read the list of items the PR addresses?

I re-gened all the RPs in the azure-sdk-for-net to reflect this reference doc modification. The PR is https://github.com/Azure/azure-sdk-for-net/pull/29264. Please take a look. Thanks

@chunyu3, would it be possible to create sample reference docs from the updated files in this PR so we can see what the final result looks like after processing in the docs generation pipeline?

@chunyu3, would you please update the PR description so we can read the list of items the PR addresses?

updated

Hi @annelo-msft  @ShivangiReja   'externalDocumentationObject' is not supported in modelerfour now and Timothee is working on this and will be available in next minor autorest version. Now in those test projects, there are no rest-api-doc link in comments. But the feature is implemented in autorest.csharp, after modelerfour release the version, I will update.

But the feature is implemented in autorest.csharp, after modelerfour release the version, I will update.

That makes sense.  If you merge this PR and close https://github.com/Azure/autorest.csharp/issues/2288, would you please create a release-blocking issue to track the feature to generate the REST API links?

File issue https://github.com/Azure/autorest.csharp/issues/2315 to track modelerfour blocking

T

I re-gened all the RPs in the azure-sdk-for-net to reflect this reference doc modification. The PR is Azure/azure-sdk-for-net#29264. Please take a look. Thanks

@chunyu3, would it be possible to create sample reference docs from the updated files in this PR so we can see what the final result looks like after processing in the docs generation pipeline?

The sample doc:
reference-doc.zip

The sample doc:
reference-doc.zip

@chunyu3, it looks like this is only the plain HTML files.  Would you please include .css files and other configuration so we can view the docs as they would appear on the docs site?

The sample doc:
reference-doc.zip

@chunyu3, it looks like this is only the plain HTML files. Would you please include .css files and other configuration so we can view the docs as they would appear on the docs site?

Hi @annelo-msft I uploaded the sample docs for deviceupdate and purviewScanning (attached following). Please take a look. Thanks.
purviewScanning.zip
deviceupdate.zip

Hi @annelo-msft I updated according to the comments, and regenerated the reference doc, please take a look. Thanks.
https://rawcdn.githack.com/chunyu3/azure-sdk-for-net/0f7a0c880a7fb4b51110fe8604c1c244bdfb2ead/doc/ApiDocGeneration/Azure.IoT.DeviceUpdate/docfx-output/docfx_project/_site/index.html
https://rawcdn.githack.com/chunyu3/azure-sdk-for-net/0f7a0c880a7fb4b51110fe8604c1c244bdfb2ead/doc/ApiDocGeneration/Azure.Analytics.Purview.Scanning/docfx-output/docfx_project/_site/index.html

@chunyu3 I was looking at your generated doc here and have a quick question -  If a method takes one of the polymorphic type, then should we add something like
This method takes one of the JSON object below as a payload. Please select a JSON object to view the schema for this. before Request Body:? (see here for more details)

akes one of the JSON object below as a payloa

Hi @ShivangiReja  @annelo-msft  Shall we care about the location of the polymorphic type, in request, in response or both? When there is polymorphic type, we will add above statement before payload schema. Right?

/azp run

/checkenforcer reset

/check-enforcer evaluate

/check-enforcer override

/azp run

All jobs done, but hanging on autorest.csharp - ci Expected — Waiting for status to be reported which is ci issue, so force merge this pr


[Bug]在接收到含有qq表情的消息时报错

描述此Bug
在接收到含有qq表情的消息时报错，并自动重启。
控制台输出
[WARNING] 未检测到GPU,将使用CPU进行推理
C:\Users\chen\.conda\envs\Muice\lib\site-packages\transformers\utils\generic.py:260: FutureWarning: `torch.utils._pytree._register_pytree_node` is deprecated. Please use `torch.utils._pytree.register_pytree_node` instead.
  torch.utils._pytree._register_pytree_node(
C:\Users\chen\.conda\envs\Muice\lib\site-packages\transformers\modeling_utils.py:479: FutureWarning: You are using `torch.load` with `weights_only=False` (the current default value), which uses the default pickle module implicitly. It is possible to construct malicious pickle data which will execute arbitrary code during unpickling (See https://github.com/pytorch/pytorch/blob/main/SECURITY.md#untrusted-models for more details). In a future release, the default value for `weights_only` will be flipped to `True`. This limits the functions that could be executed during unpickling. Arbitrary objects will no longer be allowed to be loaded via this mode unless they are explicitly allowlisted by the user via `torch.serialization.add_safe_globals`. We recommend you start setting `weights_only=True` for any use case where you don't have full control of the loaded file. Please open an issue on GitHub for any issues related to this experimental feature.
  return torch.load(checkpoint_file, map_location=map_location)
[WARNING] Failed to load cpm_kernels:name 'CPUKernel' is not defined
Some weights of ChatGLMForConditionalGeneration were not initialized from the model checkpoint at ./model/chatglm2-6b-int4 and are newly initialized: ['transformer.prefix_encoder.embedding.weight']
You should probably TRAIN this model on a down-stream task to be able to use it for predictions and inference.
F:\ai\Muice-Chatbot\llm\transformers.py:22: FutureWarning: You are using `torch.load` with `weights_only=False` (the current default value), which uses the default pickle module implicitly. It is possible to construct malicious pickle data which will execute arbitrary code during unpickling (See https://github.com/pytorch/pytorch/blob/main/SECURITY.md#untrusted-models for more details). In a future release, the default value for `weights_only` will be flipped to `True`. This limits the functions that could be executed during unpickling. Arbitrary objects will no longer be allowed to be loaded via this mode unless they are explicitly allowlisted by the user via `torch.serialization.add_safe_globals`. We recommend you start setting `weights_only=True` for any use case where you don't have full control of the loaded file. Please open an issue on GitHub for any issues related to this experimental feature.
  prefix_state_dict = torch.load(os.path.join(pt_model_path, "pytorch_model.bin"), map_location='cpu')
INFO:     Started server process [22604]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:21050 (Press CTRL+C to quit)
INFO:     ('<IP_ADDRESS>', 13513) - "WebSocket /ws/api" [accepted]
INFO:     connection open
[INFO] 已链接
[INFO] 收到QQ**********的消息：
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\uvicorn\protocols\websockets\websockets_impl.py", line 244, in run_asgi
    result = await self.app(self.scope, self.asgi_receive, self.asgi_send)  # type: ignore[func-returns-value]
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\uvicorn\middleware\proxy_headers.py", line 70, in __call__
    return await self.app(scope, receive, send)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\fastapi\applications.py", line 1054, in __call__
    await super().__call__(scope, receive, send)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\applications.py", line 123, in __call__
    await self.middleware_stack(scope, receive, send)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\middleware\errors.py", line 151, in __call__
    await self.app(scope, receive, send)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\middleware\exceptions.py", line 65, in __call__
    await wrap_app_handling_exceptions(self.app, conn)(scope, receive, send)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\_exception_handler.py", line 64, in wrapped_app
    raise exc
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\_exception_handler.py", line 53, in wrapped_app
    await app(scope, receive, sender)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\routing.py", line 754, in __call__
    await self.middleware_stack(scope, receive, send)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\routing.py", line 774, in app
    await route.handle(scope, receive, send)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\routing.py", line 371, in handle
    await self.app(scope, receive, send)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\routing.py", line 96, in app
    await wrap_app_handling_exceptions(app, session)(scope, receive, send)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\_exception_handler.py", line 64, in wrapped_app
    raise exc
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\_exception_handler.py", line 53, in wrapped_app
    await app(scope, receive, sender)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\starlette\routing.py", line 94, in app
    await func(session)
  File "C:\Users\chen\.conda\envs\Muice\lib\site-packages\fastapi\routing.py", line 367, in app
    await dependant.call(**values)
  File "F:\ai\Muice-Chatbot\ws.py", line 88, in websocket_endpoint
    for reply_item in reply_list:
TypeError: 'NoneType' object is not iterable
INFO:     connection closed
INFO:     ('<IP_ADDRESS>', 13521) - "WebSocket /ws/api" [accepted]
INFO:     connection open
[INFO] 已链接

复现该Bug的操作
在聊天时发送包含qq表情的消息
额外信息
使用的模型：ChatGLM2-6B-Int4 + Muice-2.4-chatglm2-6b-int4

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx

请问这个“含有qq表情”是只有QQ表情吗？如果可以，请你举一个例子给我

是的，是我表述不对，刚刚试了一下，只有单独的一个表情才会，普通文本里的会被自动忽略

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx

我们已确认这个问题只会在QQ自带的表情上产生并作出了更改，与此同时我建议您使用手机的emoji来替代QQ自带的表情

还请您拉取代码来同步修改，并确认这个问题是否存在

现在在reply_list为None时不会回复,不再报错

谢谢，问题已经被解决


🌟Feature request: allow the option of disabling typeahead

Is your feature request related to a problem? Please describe.
We are attempting to use the dropdown menu for context relating to each table column in our app - for this case we use an input field to search for the facet value you would wish to filter the column to.

However because typeahead is always on if you happen to type something that is actuate in the menu, "sort" for example focus jumps from the input field to the corresponding button.
Describe the solution you'd like
I'd simply like an input for the dropDownMenu that let's me disable typeahead for cases such as this
Describe alternatives you've considered
Another alternative could be to ignore typeahead if an input has focus, but could be a little too "magic"

Closed by #467

@huntabyte Could you do the same for the "Select" builder please?


Fastlane -v takes very long to execute

I've been noticing that fastlane seems to take a long time to execute. The worst case I've seen-- 50 seconds:
➜  ~  time fastlane -v
fastlane 1.2.3
fastlane -v  50.62s user 1.67s system 99% cpu 52.499 total

Coworker machines are also taking long (roughly 10 seconds each time).
Are others experiencing this delay, or is it maybe isolated to our systems? Let me know what information would be useful to provide.

Unfortunately this seems to be related to RubyGems or Bundler, I can't really tell. As far as I know there is not much I can do :disappointed: If you have an idea how to speed this up, please let me know.
It usually runs much faster when running it using bundle exec when you used Bundler.

It usually runs much faster when running it using bundle exec when you used Bundler.

Can confirm this is true.

I'll close this issue for now, as I don't know if it's an issue caused by fastlane or Ruby. Also, probably related https://github.com/KrauseFx/fastlane/issues/420

Same problem here, it takes two minutes to display the version (!):
$ time fastlane --version
fastlane 1.18.0

real	2m2.848s
user	1m54.236s
sys	0m5.715s

Sorry about that @besi, but there is not much I can do except adding notes to the documentation.

@krausefx: not a problem. I just thought I post my case and copy the solution into this issue.


PYTAK_TLS_CLIENT_KEY in Docs is actually in PYTAK_TLS_CLIENT_PASSWORD

PYTAK_TLS_CLIENT_KEY in Docs is actually in PYTAK_TLS_CLIENT_PASSWORD. I assume the Docs are out of date, but just wanted to point this one out. I followed the basic installation instructions for Pytak shown in the Docs.

Hey @SamKiley thanks for the report.
Those two config parameters are different, although the later, PYTAK_TLS_CLIENT_PASSWORD isn't actually included in the documentation.
For reference:
PYTAK_TLS_CLIENT_KEY: Path to the Client Key.
PYTAK_TLS_CLIENT_PASSWORD: PKCS#12 password.

Hey does this mean we can use p12 certificates and key directly or is there still a requirement to convert to a .pem format?

@PeterQFR yes, datapackages (user.zip, et al) are supported. See PREF_PACKAGE at https://pytak.readthedocs.io/en/latest/configuration/

@PeterQFR yes, datapackages (user.zip, et al) are supported. See PREF_PACKAGE at https://pytak.readthedocs.io/en/latest/configuration/

Hi @ampledata it seems that setting "PREF_PACKAGE" in the config (in send_receive.py example) doesnt work.
read_pref_package seems to not get called.
Example usage:
config["mycottool"] = {"PREF_PACKAGE": "test.zip"}


Translated 5 patterns and fixed some typos and mistakes

Hi !
I have translated a few patterns and some expressions that appear many times throughout the document.
I think we can improve it a lot more as he uses some familiar language that we need to reinterpret to have the correct meaning.

English :
Thank you very much for the help.
With your help we shall go twice as fast.
Thank you again for the corrections made.
French :
Merci pour l'aide.
Avec votre aide, nous irons deux fois plus vite.
Merci encore pour les corrections effectuées.


Support building object_store and parquet on wasm32-unknown-unknown target

Which issue does this PR close?

Closes #180.
Rationale for this change
My goal is to get datafusion building and running on the wasm32-unknown-unknown target, and getting object_store and parquet are the first steps.
What changes are included in this PR?
This PR adds some conditional compilation in object_store (mostly around disabling local filesystem access), as well as shifting some dependencies to dev, and removing un-used features in deps (i.e. tokio).
Are there any user-facing changes?
None expected, this is purely a build-support change.

Trying to compile object_store with non-default features returns an error about getrandom which appears related to https://docs.rs/getrandom/latest/getrandom/#webassembly-support
I'm not familiar enough with WASM to know if this is an issue or not?

Yep, wasm32-unknown-unknown is only going to support a limited set of features on parquet and object_store, in particular, the cloud storage features in object_store won't work, and some of the compression features on parquet won't work. I've documented everything in the respective READMEs now, and I also added a compile-time warning for object_store.

and some of the compression features on parquet won't work

As a note on this, I've gotten zstd and lz4-flex to both compile to wasm, so it's possible to have better wasm parquet support if you're interested. For zstd I think it should work out of the box with arrow-rs IIRC (you may need to make sure you're building against the right clang, especially if on Mac, see https://github.com/kylebarron/parquet-wasm/pull/42/files). For lz4, we'd need to update arrow-rs to switch between lz4 and lz4-flex depending on whether building for wasm or not. See https://github.com/jorgecarleitao/parquet2/pull/124 for the relevant code in parquet2. As of that, parquet2 builds to wasm32-unknown-unknown for all compressors


Selecting black and turning opacity to 0 results in alpha being "TRANSPARENT" instead of expected 0

Can be reproduced here: https://casesandberg.github.io/react-color/
When using Sketch Picker, turning the color to black and the opacity to zero results in the hex label displaying TRANSPARENT instead of the inspected #000000. This also impacts many of the other pickers on this page, even though the selection has to be done using the sketch picker

I was looking around for an answer to my own issue. Anyway, apparently this isn't a bug.
https://github.com/casesandberg/react-color/blob/master/src/helpers/color.js#L39


[51677] [51678] [51739] Text updates

@jonasofrich Have you updated the excel sheet as well?

@jonasofrich Have you updated the excel sheet as well?


Spectral rule equivalents

Is your feature request related to a problem? Please describe.
Users with existing Spectral rulesets want to migrate to Redocly, not all rules have a built-in equivalent
Describe the solution you'd like
A list of substitutions to try. Here's a starter set, these need to be tested and added to our docs. Additions, comments and feedback all gratefully received!
I wonder if it would be helpful to include configurable rule equivalents, like:
rules:
  rule/info-description:
    subject:
      type: Info
      property: description
    assertions:
      defined: true

no-eval:
rules:
  rule/no-eval:
    subject:
      type: any
      property: description
    assertions:
      notPattern: 'eval('

no-script
rules:
  rule/no-script-tag:
    subject:
      type: any
      property: description
    assertions:
      notPattern: '<script'

openapi-tags:
rules:
  rule/openapi-tags:
    subject:
      type: Tag # or maybe TagList ?
    assertions:
      defined: true

operation-tags:
rules:
  rule/operation-tags:
    subject:
      type: Operation
      property: tags
    assertions:
      defined: true
      maxLength: 1 # I guess we already have `operation-singular-tag`

(credit to @adamaltman for the list!)
Describe alternatives you've considered

Additional context

Maybe add to the cookbook?


Project Redesign on Veryfi-java

Dear Team/Contributors,
I'm thrilled to share that your project has undergone a major redesign.
Some of the patrons that you should think about are:
Singleton
Adapter
Facade
Iterator
Those are the main things that could make the coding be more smooth.
Best,
Medardo Moran

Perfect thanks for your feedback we will add this to our backlog


Update expression-uses-in-reports-report-builder-and-ssrs.md

@vincenzo-petronio : Thanks for your contribution! The author, @maggiesMSFT, has been notified to review your proposed change.

#sign-off
Good catch, @vincenzo-petronio. Thanks!


llms/googleai: batch embedding calls

The new Batch API significantly speeds up embedding
PR Checklist

[ ] Read the Contributing documentation.
[ ] Read the Code of conduct documentation.
[ ] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[ ] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[ ] Describes the source of new concepts.
[ ] References existing implementations as appropriate.
[ ] Contains test coverage for new functions.
[ ] Passes all golangci-lint checks.

So good to see you contributing! can't wait to get this in!.

Also happy to see you contribute here, @johanbrandhorst  :-)
Re tests, the googleai provider is relatively well-tested with live tests vs. the Gemini API (https://github.com/tmc/langchaingo/blob/main/llms/googleai/shared_test/shared_test.go)
For this test specifically, testing batching vs. no batching is a bit tricky and we don't really do mock testing with fake HTTP backends for now. That said, if the shared live tests pass and you observe the performance improvement, I believe this is good enough.


RCLONE Backup mode with LZ4 - feature request

Considering many people use RCLONE as a backup tool. I was wondering why there is not a backup mode/flag that would allow us to use LZ4 compression? Besides making RCLONE less bandwidth hungry, while preserving speed (up to 400MB/s on LZ4 speed), this would solve some other issues backupers face, as RCLONE not preserving file metadata / date / time on Amazon Cloud, for instance.
It seems to me that the challenge of implementing such a feature would be to keep the capacity of RCLONE on detecting file changes on the source, while comparing the source files with the destination files in the backup location. Since the destination files would be compressed we will not be able to compare uncompressed files with compressed ones. In this sense, I think the use of extracted metadata might be a solution. We could create a database of all metadata of all files copied, save it in a separate file from the compressed files, within the destination location, and than compare this metadata with the metadata of the source files.
Does anyone feels capable / motivated to implement such a feature?
Here is LZ4 source in GO language:
https://github.com/pierrec/lz4
And here is some information about it:
http://lz4.github.io/lz4/
Unhappily I still don't code in Go, but I can try to learn some stuff and give a hand in case someone can guide me. I currently code in C, Python and Javascript.

I think this is mostly a duplicate of #675 - can you add your comments about LZ4 to that ticket and close this one?


Added link to reset user state

Added readerId to the auth response
Added templated markup to link to the reset button on the articles

LGTM


Allow outgoing webhook to add reaction when sending a response without additional calls to the REST API.

Fixes: #12811
Testing plan:
It has been tested both manually and with automated tests.
GIFs or screenshots:
This introduces a new format for response messages for outgoing webhooks:
{
  "reactions": [
    {"emoji_name": "wave"}
  ]
}

It also works with content:
{
  "content": "test",
  "reactions": [
    {"emoji_name": "wave"}
  ]
}

Responding with two emoji reactions along with a message:

When the emoji_name field is missing in a specified emoji reaction:

Updated documentation for the API:

Error messages;

While working on the PR, I found that if the response message is not JSON, it will send an error message after the message that triggers it, but it doesn't notify the user. This might be inconsistent with our error handling logic for outgoing webhooks. Prove me wrong if it's intended this way.
try:
    response_json = json.loads(response.text)
except json.JSONDecodeError:
    fail_with_message(event, "Invalid JSON in response")
    return

Good find; I think we ended up doing just that in b7b1ec0aebbf75c2f66a4c81a42bf4da0ac7c9e1.

A small comment on commit messages: You should only put Fixes:  in one commit.  Also, documentation changes generally belong in the commit that changes reality to match that documentation; so we should squash the documentation commit into the previous one.

Thank you for the feedback!

I merged the first commit, after moving check_add_reaction after do_add_reaction, since generally it's clearer to have functions call functions above them in the file.
I pushed the second commit back here after a few small documentation/comment changes.  It looks correct, but I'd like tests that actually verify the full set of parameters passed to check_add_reaction -- one can imagine some future refactoring bug where we mess up the plumbing of parameters, or something, and so it's worth having a test.  (Probably in test_outgoing_webhook_system.py?)

It looks like that I have broken something with the rebase.

I've rebased this due to a recent change. I'm not sure if a service bot is the only case when a user doesn't have a UserMessage row. It is possible that we can remove the check below?
if (
   user_profile.bot_type == UserProfile.EMBEDDED_BOT
   or user_profile.bot_type == UserProfile.OUTGOING_WEBHOOK_BOT
):

@PIG208 I wonder why the build is failing here. Could you please investigate?

@PIG208 I wonder why the build is failing here. Could you please investigate?

I have rerun the tests several times and I think this might be a nondeterministic error? Will rerun it again.


Adding node coordinates support

PR in order to solve #12

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
:information_source: You can also turn on project coverage checks and project coverage reporting on Pull Request comment
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Add support to custom user Agent

Hi,
as per issue https://github.com/fustom/ariston-remotethermo-home-assistant-v3/issues/362 i notice that if we change a useragent with user agent of browser, I not receive any errors for two weeks.
Thi issue is to ask you to add the possibility to specify user agent on library side, then we can add a form in HA integration, in that way user can customize it with the best option.
Thanks

@fustom  any update on this?


Northern Ireland is in the wrong place

Clearly I am doing something wrong importing the shapefile. If I load directly in QGIS, NI appears to be in the right place.


Custom,minikube, Deploying Prometheus,prometheus-k8s.yaml: error converting YAML to JSON: yaml: line 22: mapping values are not allowed in this context

When execute ./deployment in node with minikube installed,  just like following:
Deploying Prometheus
serviceaccount/prometheus-k8s created
role.rbac.authorization.k8s.io/prometheus-k8s created
role.rbac.authorization.k8s.io/prometheus-k8s created
role.rbac.authorization.k8s.io/prometheus-k8s created
clusterrole.rbac.authorization.k8s.io/prometheus-k8s created
rolebinding.rbac.authorization.k8s.io/prometheus-k8s created
rolebinding.rbac.authorization.k8s.io/prometheus-k8s created
rolebinding.rbac.authorization.k8s.io/prometheus-k8s created
clusterrolebinding.rbac.authorization.k8s.io/prometheus-k8s created
prometheusrule.monitoring.coreos.com/prometheus-k8s-rules created
servicemonitor.monitoring.coreos.com/kube-dns created
servicemonitor.monitoring.coreos.com/kubelet created
servicemonitor.monitoring.coreos.com/prometheus-operator created
service/prometheus-k8s created
servicemonitor.monitoring.coreos.com/kube-apiserver created
servicemonitor.monitoring.coreos.com/kube-controller-manager created
servicemonitor.monitoring.coreos.com/kube-scheduler created
servicemonitor.monitoring.coreos.com/alertmanager created
servicemonitor.monitoring.coreos.com/kube-state-metrics created
servicemonitor.monitoring.coreos.com/node-exporter created
servicemonitor.monitoring.coreos.com/prometheus created
error: error parsing manifests/prometheus/prometheus-k8s.yaml: error converting YAML to JSON: yaml: line 22: mapping values are not allowed in this context

then ,
[root@localhost prometheus-kubernetes]# kubectl get pods -n monitoring
NAME                                  READY   STATUS    RESTARTS   AGE
alertmanager-main-0                   2/2     Running   0          30s
alertmanager-main-1                   2/2     Running   0          20s
alertmanager-main-2                   2/2     Running   0          13s
grafana-6f7b97c8dd-nvxct              1/1     Running   0          28s
kube-state-metrics-6769d4fdcc-ww5mg   2/2     Running   0          27s
node-exporter-mn65d                   1/1     Running   0          30s
prometheus-operator-7846b499d-l299d   1/1     Running   0          34s

but prometheus-k8s-0 and prometheus-k8s-1 missing
and
[root@localhost prometheus-kubernetes]# kubectl get svc -n monitoring
NAME                    TYPE        CLUSTER-IP       EXTERNAL-IP   PORT(S)             AGE
alertmanager-main       ClusterIP   <IP_ADDRESS>    <none>        9093/TCP            10m
alertmanager-operated   ClusterIP   None             <none>        9093/TCP,6783/TCP   10m
grafana                 ClusterIP   <IP_ADDRESS>     <none>        3000/TCP            10m
kube-state-metrics      ClusterIP   <IP_ADDRESS>   <none>        8080/TCP            10m
node-exporter           ClusterIP   None             <none>        9100/TCP            10m
prometheus-k8s          ClusterIP   <IP_ADDRESS>      <none>        9090/TCP            10m
prometheus-operator     ClusterIP   <IP_ADDRESS>   <none>        8080/TCP            11m

I fix this error ,but the result is the same
how to fix above error:
prometheus-k8s.yaml:line 22
change
- {key: k8s-app, operator: Exists}

to
- key:k8s-app
  operator:Exists

before ./deploy

Finally, I solve it !!!!!!!!
something wrong in deploy.sh
change
 sed -i -e '1,8d;32,45d' manifests/prometheus/prometheus-k8s.yaml;

to
 sed -i -e '1,8d;32,49d' manifests/prometheus/prometheus-k8s.yaml;

Fixed


[ZEPPELIN-5195] check JAVA_HOME in check_java_version

What is this PR for?
Make check_java_version in bin/common.sh check JAVA_HOME if set
What type of PR is it?
Bug Fix
Todos
None
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-5195
How should this be tested?

Strongly recommended: add automated unit tests for any new or changed behavior
Outline any manual steps to test the PR here.

Screenshots (if appropriate)
Questions:

Does the licenses files need update?
No
Is there breaking changes for older versions?
No
Does this needs documentation?
No

LGTM

LGTM

I will include this in master and branch-0.9 on Wednesday if no further comments are received.

I will include this in master and branch-0.9 on Wednesday if no further comments are received.

LGTM

LGTM


[BUG] Corrupted Save File

Aged Version: 3.0.2 or 3.0.1, I do not recall.

I believe that some chunks have corrupted and I have no clue what to do in order to fix them. Or this issue, is my world screwed?
latest.log

You added mods to the pack which corrupted your save file
Seems like Immersive Weathering
But you need to figure this out on your own.
Also version 3.x aint compatible with 2.x

Thank you! I managed to fix it by removing immersive weathering, I love your modpack!


TypeDoc -  Generate the api documentation for the application.

Awesome, will merge it soon.

Thank  you @ludohenin
I am new to git. That's why late check-in.
Please link this request id with #174 and make #174 as obsolete.

Just tried it but Isn't missing a kind of serve.docs ?

Yes.I Will do it.

Should the docs folder to be included to .gitignore ?

docs folder contains lots of auto generated files. so we no need to keep it in the repository.
I would like keep this project is very lightweight. What would you say?

@natarajanmca11 this is excellent thank you, lgtm. @mgechev merge?

@NathanWalker I'm not quite in fact.

@natarajanmca11 looks great! Lets merge this PR, although we are planning some drastic changes in the entire seed project.

@mgechev Let me know if I can assist on any of the drastic changes :+1:  Curious what those are.

That's actually quite cool to navigate into angular source

Thank you all. I am really enjoy with this seed project. Learned a lot from here.


Use suggested replacement as refactoring template if any substitution has no loc

Previously foo = typeOf (undefined :: a) fails to refactor, because
template = typeRep (Proxy :: Proxy a)
substs = [ a -> undefined ]

So hlint is telling apply-refact to refactor it into typeRep (Proxy :: Proxy undefined).
The template is not typeRep (Proxy :: Proxy b) because the substitution b -> a has no loc, so b is expanded into a in the template (#895).
This can be fixed by: if any substitution in u has no loc, set u to mempty, and use the suggested replacement (res) as the refactoring template (tpl). In the example above it becomes
template = typeRep (Proxy :: Proxy a)
substs = []

This conflicts with the other patch of yours I just merged. Any chance you can rebase it on master?

@ndmitchell Done. Thanks for merging the PRs, and take good care!


Failed to connect, failed to create session, failed to defeat challenge...

When I start kahoot-rand:
go run kahoot-hack/kahoot-rand/main.go <code> "x" 5
Terminate this program to stop the automatons...
failed to connect: failed to create session: failed to defeat challenge: decode.call(this, 'bYz31iSGfMnUI2kjFaeOAeaDpdu5LswsERumLvqlQxvRwbnmjqcURHdp5lCuHBGoPMNXXMlY4e4o68yySIyS1BEoBE23z5EtqFFy'); function decode(message) {var offset = 22 + 94 * (93 * 38) * (93 * 95 * 47); if (this.angular.isString(offset)) {console.log("Offset derived as: {", offset, "}");}return _.replace(message, /./g, function(char, position) {return String.fromCharCode((((char.charCodeAt(0) * position) + offset) % 77) + 48);});}
exit status 1

Looks like kahoot implemented some anti bot thing?
Go version:
go version
go version go1.9 linux/amd64

Same issue here.

Try again. This tool depends on a web server to decipher kahoot's challenge strings, and that server goes down from time to time.

@unixpickle why not let the user run the deciphering tools?

@unixpickle Now it works. What @shadowbrony33 suggested is a pretty good idea.
I have experienced another issue with kahoot-rand though... I'll open a new issue for that.

closed *


KOGITO-7230: Upgrade dependencies with reported vulnerabilities

I changed following dependencies:
com.fasterxml.jackson -> 2.13.3
com.google.code.gson -> 2.8.6
org.wildfly.core -> 18.1.1
Performed basic sanity check on the webapp, looked good. Let me know @romartin

@domhanak Can you please update the same dependencies in stunner-editor/errai-bom? Thx u!

@yesamer I prefer doing it in separate PR for stunner-editors because I did not test anything on that side. Just in case.


[google compute] image labels

Allow labels on GCE images
Description
Add capability to set and clear labels on GCE images.
Status
Replace this: describe the PR status. Examples:

done, ready for review

Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)
[x] Documentation
[x] Tests
[x] ICLA (required for bigger changes)

/cc @katrielt

Codecov Report

Merging #1138 into trunk will increase coverage by <.01%.
The diff coverage is 88.57%.

@@            Coverage Diff             @@
##            trunk    #1138      +/-   ##
==========================================
+ Coverage   85.48%   85.48%   +<.01%
==========================================
  Files         348      348
  Lines       66443    66474      +31
  Branches     5914     5916       +2
==========================================
+ Hits        56796    56825      +29
- Misses       7239     7241       +2
  Partials     2408     2408

Impacted Files
Coverage Δ

libcloud/test/compute/test_gce.py
97.58% <100%> (+0.01%)
:arrow_up:

libcloud/compute/drivers/gce.py
76.15% <69.23%> (+0.1%)
:arrow_up:

libcloud/test/compute/test_ec2.py
97.77% <0%> (-0.16%)
:arrow_down:

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 5e1972e...53dbe53. Read the comment docs.


ML should handle missing RSA keys better

Description
$ az ml service view realtime -n asdf
[Errno 2] No such file or directory: '/Users/jasonsha/.ssh/acs_id_rsa'
Traceback (most recent call last):
  File "/Users/jasonsha/lib/azure-cli/lib/python2.7/site-packages/azure/cli/main.py", line 36, in main
    cmd_result = APPLICATION.execute(args)
  File "/Users/jasonsha/lib/azure-cli/lib/python2.7/site-packages/azure/cli/core/application.py", line 210, in execute
    result = expanded_arg.func(params)
  File "/Users/jasonsha/lib/azure-cli/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 289, in __call__
    return self.handler(*args, **kwargs)
  File "/Users/jasonsha/lib/azure-cli/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 488, in _execute_command

This should instead direct the user to the command or process to create the acs_id_rsa file.

Shouldn't this be in the ML team's repo?

@JasonRShaver : I dont believe ML is looking at this repo. Can you shoot them a mail?


Tracking issue for the storage research

Our requirements for the storage

Key-Value
JSON-like schema
Simple, Easy, and widely used in other blockchain projects
Rust binding

Tracked issues

[ ] #10
[x] #9

I recommand rocksdb.
Key-Value
RocksDB is an embeddable persistent key-value store for fast storage.
JSON-like schema
Sadly, RocksDB doesn't support JSON-like schema.
Therefore, if we want to handle "Value" as JSON object, I would recommand below two options.

Naively, use serde_json
Define new wrapper for rocksdb

Simple, Easy, and widely used in other blockchain projects
RocksDB is used by solana, near, polkadot and so on.
Rust Binding
You can check this below github / documentation
https://github.com/rust-rocksdb/rust-rocksdb
https://docs.rs/rocksdb/latest/rocksdb/

At the beginning, there were several options for db (leveldb, couchdb, rocksdb).
Although leveldb and couchdb can be good solution for another blockchain project, it seems not to be for our project.
Because of lacking functionality and reliability about those db package for rust lang, it is better to choose rocksdb.
Furthermore, rocksdb is used by solana, near, polkadot and so on.
In this reason, I recommand using rocksdb.

How to Install RocksDB
check this: https://docs.desmos.network/fullnode/rocksdb-installation/
Basic Operations

Get: get value from KVstore with key
Put: put value into KVstore with key
Delete: delete pair (key, value) from KVstore with key

User Custom Operations (Merge)
User defined Operation for "Atomic Read-Modify-Write operation" (they said).
Adventages of it

Encapsulates the semantics for read-modify-write into a simple abstract interface.
Allows user to avoid incurring extra cost from repeated Get() calls.
Performs back-end optimizations in deciding when/how to combine the operands without changing the underlying semantics.
Can, in some cases, amortize the cost over all incremental updates to provide asymptotic increases in efficiency.

More Infomation, check this: https://github.com/facebook/rocksdb/wiki/Merge-Operator
How to store Key-Value Data

Lazy Evaluation
RocksDB takes strategy of lazy evaluation.
In other words, Put or Merge does not necessarily force the value to be computed or the merge to occur immediately.
RocksDB will more-or-less lazily decide when to actually apply the operations
(e.g.: the next time the user calls Get(), or when the system decides to do its clean-up process)

Log, SST, MANIFEST file
log file: candidate key-value to be computed
SST file: actual data storage saving pair (key, value)
MANIFEST file: metadata for current db managing key lists
check this: https://github.com/facebook/rocksdb/wiki/MANIFEST

Checkpoint
If the snapshot is on the same filesystem as the original database, the SST files will be hard-linked.
otherwise SST files will be copied. The manifest and CURRENT files will be copied.
check this: https://github.com/facebook/rocksdb/wiki/Checkpoints

We use git.


Install procedure is not clear

Warning:  I will close the issue without the minimal init.vim and the reproduce ways.
Problems summary
Is not clear how the vimrc should be
I'm trying to setup my vimrc file, but I've this error:
~ » vim .vimrc
Error detected while processing /home/samba/.vimrc:
line   32:
E15: Invalid expression: /home/samba/.vim/plugins})
E116: Invalid arguments for function dein#load_state({/home/samba/.vim/plugins})
E15: Invalid expression: dein#load_state({/home/samba/.vim/plugins})
Press ENTER or type command to continue

Expected

I expect to run the install.sh script and have a dir installed in ~/.vim/plugins or something
I expect to have the vimrc configured or at least an example in the docs about how this should be

Environment Information (Required!)

dein.vim version(SHA1):
how can I get a sha1 ? do you want a sha1sum or what?

OS:
archlinux

Vim/neovim version:

Provide a minimal .vimrc with less than 50 lines (Required!)
" map
map <silent> <C-N> :set invhlsearch<CR>
map <f5> :w<CR>:!bash %<CR>
map <f6> :w<CR>:!python %<CR>
map <f7> :w<CR>:!ruby %<CR>

" from:https://github.com/Shougo/dein.vim
if &compatible
  set nocompatible
endif
set runtimepath+={/home/samba/.vim/plugins/dein.vim}

if dein#load_state({/home/samba/.vim/plugins})
  call dein#begin({/home/samba/.vim/plugins})
  call dein#add({/home/samba/.vim/plugins/dein.vim})
  call dein#add('Shougo/neocomplete.vim')
"  ...

  call dein#end()
  call dein#save_state()
endif

filetype plugin indent on
syntax enable

The reproduce ways from Vim starting (Required!)

vim [filename]
get the error
continue and ignore de dein error

Screen shot (if possible)
Upload the log messages by :redir and :message (if errored)

Please study Vim script.
    set runtimepath+={path to dein.vim directory}

It means:
set runtimepath+=/home/samba/.vim/plugins/dein.vim

{} is just place holder.

And set up a path to the repository directory.
>
	set runtimepath+={path to dein directory}
<
Example:
>
	set runtimepath+=~/.vim/bundle/repos/github.com/Shougo/dein.vim
<

Note: Why have not read deoplete.txt?

how can I get a sha1 ? do you want a sha1sum or what?

Hm.  You can get the sha1 by git rev-parse HEAD


feat(MM-61865): send network request telemetry data to the server

Summary
Add sending network requests telemetry data to the server
Ticket Link

https://mattermost.atlassian.net/browse/MM-61865
Checklist

[X] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[ ] Includes text changes and localization file updates
[ ] Have tested against the 5 core themes to ensure consistency between them.
[ ] Have run E2E tests by adding label E2E iOS tests for PR.

Device Information
This PR was tested on:
iPhone 15 (17.4)
Screenshots

 PASS  app/managers/performance_metrics_manager/index.test.ts
  performance_metrics_manager
    load metrics
      ✓ only load on target (19 ms)
      ✓ only register load once (5 ms)
      ✓ retry if the mark is not yet present (5 ms)
      ✓ fail graciously if no measure is set (12 ms)
    app state changes
      ✓ forces send on app state change to inactive, or anything other than active (5 ms)
    time to interaction
      ✓ measures time to interaction correctly (1 ms)
      ✓ handles missing TTI mark gracefully
    network request metrics
      ✓ collects network request metrics (10 ms)
    other metrics
      ✓ do not send metrics when we do not start them (7 ms)
      ✓ send metric after it has been started (6 ms)
      ✓ a second end metric does not generate a second measure (11 ms)
      ✓ different metrics do not interfere (6 ms)
      ✓ metrics to different servers do not interfere (10 ms)

=============================== Coverage summary ===============================
Statements   : 98% ( 49/50 )
Branches     : 92.85% ( 13/14 )
Functions    : 92.85% ( 13/14 )
Lines        : 97.91% ( 47/48 )
================================================================================
Test Suites: 1 passed, 1 total
Tests:       13 passed, 13 total

was
=============================== Coverage summary ===============================
Statements   : 82.5% ( 33/40 )
Branches     : 71.42% ( 10/14 )
Functions    : 72.72% ( 8/11 )
Lines        : 84.21% ( 32/38 )
================================================================================

 PASS  app/client/rest/tracking.test.ts
  ClientTracking
    ✓ should set bearer token (3 ms)
    ✓ should set CSRF token (2 ms)
    ✓ should get request headers (1 ms)
    ✓ should initialize and track group data
    ✓ should increment and decrement request count (26 ms)
    ✓ should handle request completion (1 ms)
    ✓ should clear completion timer (1 ms)
    ✓ should check if all requests are completed (1 ms)
    ✓ should send telemetry event
    ✓ should build request options
    ✓ should perform fetch with tracking (3 ms)
    ✓ should handle fetch errors (16 ms)
    ✓ should handle non-ok response with error details (3 ms)
    ✓ should handle non-ok response without error details (1 ms)
    ✓ should handle response with bearer token header
    ✓ should handle response with lowercase bearer token header (1 ms)
    ✓ should call increment and decrement the same number of times as doFetchWithTracking, and handleRequestCompletion only once (510 ms)
    ✓ should handle invalid HTTP method (2 ms)
    ✓ should handle server version changes without cache control (2 ms)
    ✓ should not update server version when cache control present (7 ms)
    ✓ should handle zero transfer time in speed calculation (1 ms)
    ✓ should track duplicate requests correctly and log duplicate details (303 ms)

=============================== Coverage summary ===============================
Statements   : 93.5% ( 144/154 )
Branches     : 82.55% ( 71/86 )
Functions    : 100% ( 29/29 )
Lines        : 93.05% ( 134/144 )
================================================================================
Test Suites: 1 passed, 1 total

was:
 PASS  app/client/rest/tracking.test.ts
  ClientTraking
    ✓ should set bearer token (3 ms)
    ✓ should set CSRF token
    ✓ should get request headers (1 ms)
    ✓ should initialize and track group data
    ✓ should increment and decrement request count (9 ms)
    ✓ should handle request completion (1 ms)
    ✓ should clear completion timer (1 ms)
    ✓ should check if all requests are completed
    ✓ should send telemetry event (1 ms)
    ✓ should build request options
    ✓ should perform fetch with tracking (2 ms)
    ✓ should handle fetch errors (14 ms)
    ✓ should call increment and decrement the same number of times as doFetchWithTracking, and handleRequestCompletion only once (306 ms)
    ✓ should track duplicate requests correctly and log duplicate details (104 ms)

=============================== Coverage summary ===============================
Statements   : 87.61% ( 92/105 )
Branches     : 69.44% ( 50/72 )
Functions    : 100% ( 19/19 )
Lines        : 87.12% ( 88/101 )
================================================================================
Test Suites: 1 passed, 1 total
Tests:       14 passed, 14 total
Snapshots:   0 total
Time:        3.082 s
Ran all test suites matching /app\/client\/rest\/tracking.test.ts/i.

Release Note

This PR merges into @enahum PR, so no testing required on this PR.


Include inherited font for form elements; Improve accessibility for motion-reduced browsers

Included in this PR:

Extended font rules in base styles to include form elements.
Added media query to base styles to check for motion-reduction and switch off animations if present.

Why

button, select etc. by default don't share the same font rules as other html elements. There may be cases where we'd want a different font for an input, but I'd argue 99 times out of 100 that's not the case.
Inspiration from CSS-Tricks and Normalize.css.
For some users animations can be dizzying and disorientating at best, and induce epileptic fits at worst. We love our animations, but if we want to be serious about putting a11y at the forefront, this is a necessary step.
Inspiration from a11y-101.

@psimk @pigeonfresh Not sure if this is the sort of thing that should be discussed in the coding standards first? Shall I open an issue there instead?

button, select etc. by default don't share the same font rules as other html elements. There may be cases where we'd want a different font for an input, but I'd argue 99 times out of 100 that's not the case.
Inspiration from CSS-Tricks and Normalize.css.

The Vue skeleton already includes the normalize.css package, does it not work correctly?
However, we are not using normalize.css in the React skeleton, so might be good to add it there.

I wouldn't Nuke all animations in prefers-reduces-motion. This is probably not the way, and lot's of animation is done in JS anyway. Also check this link: https://css-tricks.com/nuking-motion-with-prefers-reduced-motion/

Closing this, as there are comments without clear resolutions and there has not been any activity for 2 months.