Record OAuth client type at registration

The OAuth spec defines two types of client, confidential and public. Previously Gitea assumed all clients to be confidential.

OAuth defines two client types, based on their ability to   authenticate securely with the authorization server (i.e., ability to
maintain the confidentiality of their client credentials):
confidential
Clients capable of maintaining the confidentiality of their      credentials (e.g., client implemented on a secure server with
restricted access to the client credentials), or capable of secure      client authentication using other means.
public
Clients incapable of maintaining the confidentiality of their      credentials (e.g., clients executing on the device used by the      resource owner, such as an installed native application or a web      browser-based application), and incapable of secure client      authentication via any other means.
The client type designation is based on the authorization server's   definition of secure authentication and its acceptable exposure   levels of client credentials.  The authorization server SHOULD NOT   make assumptions about the client type.

https://datatracker.ietf.org/doc/html/rfc8252#section-8.4

Authorization servers MUST record the client type in the client registration details in order to identify and process requests
accordingly.

Require PKCE for public clients: https://datatracker.ietf.org/doc/html/rfc8252#section-8.1

Authorization servers SHOULD reject authorization requests from native apps that don't use PKCE by returning an error message

Fixes #21299

This change adds a new Confidential field. The old behaviour is Confidential=true. Is this a problem for API compatibility? Also I tried to write a migration to set existing apps to Confidential=true. How can I test it?

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@f82f1d0). Click here to learn what that means.
The diff coverage is 89.47%.

:exclamation: Current head 0d3a5ce differs from pull request most recent head bff1c25. Consider uploading reports for the commit bff1c25 to get more accurate results

@@           Coverage Diff           @@
##             main   #21316   +/-   ##
=======================================
  Coverage        ?   47.07%
=======================================
  Files           ?     1017
  Lines           ?   138560
  Branches        ?        0
=======================================
  Hits            ?    65233
  Misses          ?    65360
  Partials        ?     7967

Impacted Files
Coverage Δ

routers/web/user/setting/oauth2.go
0.00% <0.00%> (ø)

services/forms/user_form.go
45.36% <ø> (ø)

models/auth/oauth2.go
61.12% <100.00%> (ø)

modules/convert/convert.go
72.30% <100.00%> (ø)

routers/api/v1/user/app.go
66.97% <100.00%> (ø)

routers/web/auth/oauth.go
20.30% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I am testing it locally. Some questions:

The checkbox is misaligned?
For public clients, there are still secrets and re-generate secret button?
Is it fine to allow  switching the client type? (and related to 2, how about the secret?)

I'll return to this after https://github.com/go-gitea/gitea/pull/21351

The checkbox is misaligned?

I see the problem but I don't know how to fix. Any ideas?

For public clients, there are still secrets and re-generate secret button?

GitLab requires secrets for all clients but according to the OAuth spec https://datatracker.ietf.org/doc/html/rfc8252#section-8.5 it's not recommended:

it is NOT RECOMMENDED for authorization servers to require client   authentication of public native apps clients using a shared secret,   as this serves little value beyond client identification which is   already provided by the "client_id" request parameter.

Is it fine to allow  switching the client type? (and related to 2, how about the secret?)

Fine to switch.

For 1 the checkbox: I could take a look later.
For other questions: it looks good to me now.
I would suggest to wait for #18084 and even #21335 first, to avoid the code being copied 2 or 3 times in the future.

OK, the checkbox has been fixed. The Fomantic UI's checkbox is different from others.

The force push reverted the renaming of ConfidentialClient?

I like the idea to rename, and in https://github.com/hickford/gitea/commit/11326912477cfa4b8544f286b52ce4fd9b78903d the tests pass, but the edit form no longer works, not sure why, I must have missed something.

Do you have the commit? I can take a look.

Please resolve the conflicts

@lunny  resolved

Moving checkbox under redirect uri input box is better.

make L-G-T-M work


NETStandard version

Any plans for a NETStandard version?

I've created a pull request that solves it (#5).

@giacomelli Can you please publish this new HelperSharp version that supports netstandard2.0 on NuGet?
I'm needing to use https://github.com/giacomelli/SalesforceSharp to support in dotnet core and the first step is to get this dependency published.


Fix qa tag to accordion arrows

:shipit:

Comment accepted. Will treat it ASAP

Squashing and checks in progress.


Exporting to PDF with numbers causing issues

Hi,
I am a newbie user of Obsidian. I was using the plugin to auto number equations and referencing. But when I export to PDF this causes issues.
Following error message in console:
" Cannot read properties of undefined (reading 'setAttribute')
at preprocessForPdfExport (plugin:math-booster:3267:15)
at eval (plugin:math-booster:3240:5)
at t.postProcess (app.js:1:1297325)
at t. (app.js:1:2354196)
at app.js:1:237138
at Object.next (app.js:1:237243)
at a (app.js:1:235961)"

I'm having a similar issue.
I can export a PDF just fine, but no equation numbers are shown. I've tried on a new sandbox vault as well with only this plugin and mathlinks installed and the same behavior occurs.
Is anyone else able to export PDFs with equation numbering?

I have the same issue and the error code I received in the console says:
plugin:math-booster:3275 TypeError: Cannot read properties of undefined (reading 'setAttribute')
at preprocessForPdfExport (plugin:math-booster:3267:15)
at eval (plugin:math-booster:3240:5)
at t.postProcess (app.js:1:1297472)
at t. (app.js:1:2354364)
at app.js:1:237138
at Object.next (app.js:1:237243)
at a (app.js:1:235961)


Handle SUBGHZSPI as async

For STM32WL, simplify configuration for the use of SUBGHZSPI to perform LoRa operations.  Use Rx/Tx DMA on SPI to enable async functionality.

LGTM


Stateful table: Any settings to disable data table toolbar section?

Summary
As I don't have any toolbar items for the stateful table component that I am using, so I don't want the table toolbar section to be shown. Please suggest if there is any porp to be set to get rid of it. Below image has the highlighted section which needs to be removed, and this screen capture is from the storybook. https://carbon-addons-iot-react.com/?path=/story/watson-iot-table--simple-stateful-example

Hi @kalyanixraut, our Table component currently does not have a prop to achieve this. However, we are exporting the Carbon DataTable which will allow you to remove this header area.

@lukefirth I imagine this is something we should support in our Table component. Does the PAL spec have any issues wit this?

@davidicus ,  I don’t find carbon’s exported datatable in the storybook nor available here https://github.com/IBM/carbon-addons-iot-react/blob/master/src/index.js
Can you please point it out for me. Thank you!

Any update on this?

PR is open related to this: https://github.com/IBM/carbon-addons-iot-react/pull/902

:tada: This issue has been resolved in version 2.40.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add login sign-in page link to log-in-create-and-join-organizations.md

Closes https://github.com/MicrosoftDocs/Viva/issues/376.

@ms-vikashkoushik PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@ms-vikashkoushik PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@ms-vikashkoushik PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@ms-vikashkoushik PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

closing due to inactivity


retry for LF grant_permissions

Feature or Bugfix

Bugfix

Detail

Add retries to grant_permissions boto3 call in lakeformation client
timeout is increased by 10% every retry

Relates

#1572

Security
Please answer the questions below briefly where applicable, or write N/A. Based on
OWASP 10.

Does this PR introduce or modify any input fields or queries - this includes
fetching data from storage outside the application (e.g. a database, an S3 bucket)?

Is the input sanitized?
What precautions are you taking before deserializing the data you consume?
Is injection prevented by parametrizing queries?
Have you ensured no eval or similar functions are used?

Does this PR introduce any functionality or component that requires authorization?

How have you ensured it respects the existing AuthN/AuthZ mechanisms?
Are you logging failed auth attempts?

Are you using or adding any cryptographic features?

Do you use a standard proven implementations?
Are the used keys controlled by the customer? Where are they stored?

Are you introducing any new policies/roles/users?

Have you used the least-privilege principle? How?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@petrkalos  have look)

After reflecting and thinking about it a bit I think it would be better if we use the client's ConcurrentModificationException...
Two ways I thought of doing it...

Decorator+inner method (for example...)    class RetryTest:
        def __init__(self):
            self._client = boto3.client('lakeformation')
            self.counter = 0
        def _retry_if_concurrency_error(self, exception):
            print(f'retry callback {self.counter=}')
            return isinstance(exception, self._client.exceptions.ConcurrentModificationException)
        def test(self):
            @retry(
                retry_on_exception=self._retry_if_concurrency_error,
                stop_max_attempt_number=5,
                wait_random_min=1000,
                wait_random_max=3000,
            )
            def __call():
                print(f'call... {self.counter=}')
                self.counter += 1
                raise self._client.exceptions.ConcurrentModificationException({}, '')
            __call()

Use of Retrying directly without decorator    Retrying(
        retry_on_exception=self._retry_if_concurrency_error,
        stop_max_attempt_number=5,
        wait_random_min=1000,
        wait_random_max=3000,
    ).call(self._client.grant_permissions, **grant_dict)


Graphics: Ignore borderless parameter unless windows are full-screen.

I do agree that borderless fullscreen as a default is OK, but it does not make much sense for windowed apps. How about we do something like this?

Does borderless even work now?
How to create "fullscreen" window without requesting native fullscreen from OS?

Borderless windows are supposed to replace clumsy native fullscreen, and this PR (from the look of it) does the opposite.

We probably just should use enum here.

Good question.. Tbh i have no idea whether this works, because both borderless and actual full screen are identical on my OS, i did not check on windows. However, on windows difference can be spotted when alt+tabbing. Exclusive full-screen takes longer while borderless is immediate. You must know how borderless works in general: just a window without borders, that covers entire screen space.

Borderless windows are supposed to replace clumsy native fullscreen, and this PR (from the look of it) does the opposite.

Huh.. Maybe i misunderstand something, but my intention was precisely for borderless to replace actual full-screen. Problem i am trying to fix here is setting borderless=true for windowed apps causes a non-full-screen window to have no borders, so we can not drag such windows and what not.

Huh.. Maybe i misunderstand something, but my intention was precisely for borderless to replace actual full-screen.

AFAIK our "fullscreen" maps directly onto "native OS fullscreen" and takes precedence.
That's why I should probably just make a enum and clean all this mess xD


Fix syntax highlighting for serious-python README

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

It should be fixed in src/serious_python/README.md instead.

Hi, fixed everything

Thanks a bunch


Data Science with Python : Support Vector Machines

Description 📜
Introduction to Support Vector Machines (SVM) #732

Type of change 📝

[x] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)

Domain of Contribution 📊

[x] Datascience with #Python

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Updated @prathimacode-hub do please check

Your README needs more elaboration. It has no details. Kindly elaborate everything about topic you had chosen.
What's the audio length? @BhaswatiRoy

updated an elaborative version of readme and the audio is of 3min 21sec

@prathimacode-hub do please review this

Have you noticed it? Both of your PR's turned out to be a single PR. WHY because earlier PR wasn't merged yet. So go one after the other. And whatever issue you work on, make sure your README.md file is elaborate and detailed. @BhaswatiRoy

Maam I submitted the 1st pr and then only I worked on the 2nd pr meanwhile you told me to change my 1st pr that is why both of them got submitted together............. And for both of the files, I was as elaborative as possible from my side.............. I have added everything I could have added there is nothing more I can add from my side............. what should I do now? submit pr one by one?

@prathimacode-hub please tell me what else should I do to get my pr merged.............I am stuck on this pr and unable to shift to any more issues.................

I have deleted one pr work for now and submitted.....................please review this @prathimacode-hub so that I can go to new issues now................. Every possible stuff which I could add I have added from my part.............there is nothing more left from my side to add.................. both in audio file and readme file


Show upgrade notification if node protocol version is higher than wallet protocol version

https://zpl.io/a7d5GRj
@alexandrashelenkova @sasha-abramovich could you please check the text? and let me know if it is not good. TX

Moving this back to backlog, not critical for fork and problematic from implementation view. @beam-mw  will check whether this task should be closed.


Added new use case examples

Added examples for the following

Update a class/object
Patch a class/object
Add a message to a class/object
Expire an object
Create a JWT with new classes/objects
Create a JWT with existing classes/objects

Fixed various minor formatting issues
Made function/method names more general (ease of replication across different languages)
Added checks for class/object presence
Clarified some docstrings and comments

<EMAIL_ADDRESS>

LGTM

احصل على Outlook for Androidhttps://aka.ms/AAb9ysg

من: Nick Alteen @.>
تاريخ الإرسال: الجمعة، ٢ ديسمبر، ٢٠٢٢ ٥:٥٥ م
إلى: google-pay/wallet-samples @.>
نسخة: Subscribed @.***>
الموضوع: Re: [google-pay/wallet-samples] Added new use case examples (PR #51)
Merged #51https://github.com/google-pay/wallet-samples/pull/51 into main.
—
Reply to this email directly, view it on GitHubhttps://github.com/google-pay/wallet-samples/pull/51#event-7941401129, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3IIOIUP7PFJH5FDBIO4AR3WLIEVFANCNFSM6AAAAAASQATGXY.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


BUG: service jams each day around 12:30

The monitoring of my service fails each day around about 12:30 PM.  The failure is a timeout and I suspect something within the code is blocking all available processing at the time.  I suspect I am getting friendly "DOS'd" around that time, so reviewing the access logs I find nothing very suspicious :(
My next guess is there's logging that would explain this, but those have expired.  Will check again today at the time.

OK, this is the time when the daily CFS update takes place, so I bet the file is locked and unavailable for access, causing the timeout.  Will write an intermediate file to help this situation.

OK, this is the time when the daily CFS update takes place, so I bet the file is locked and unavailable for access, causing the timeout.  Will write an intermediate file to help this situation.


chore: bdd-sbhg-1583643417 to 0.0.1

chore: Promote bdd-sbhg-1583643417 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


about the memory

Hi, thanks for your source code. However, when I run the 1-shot code in two GTX 1080 Ti parallel, it still occurs "CUDA error: out of memory", what I can do is to change the numbers of "train-way" and "query", but I am afraid the results would be dropped. So, could you give me some advice when I just have two GTX1080 Ti? And how many of your GPUs when you run this source code? Thanks a lot!

Hi. I trained the model with one GTX-1080 Ti, if I remember correctly.
I have just improved the memory usage in train.py and test.py, you could have a try if it works on 1 GTX-1080 or parallel on 2.

Thanks for your replay! yes, you are right, and I run your source code again, and it doesn't occur any error. So I am wonder if you could tell me the details of "how you improved the memory usage" ? because my code is just imitated yours, but when I run it, it always occur "cuda error: out of memory", and I know the bug is in the for loop of
"for i, batch in enumerate(val_loader, 1):"
but when I have checked my code with yours, I can't discover any differences. I would be very appreciated if you could offer me more information of how to improve the memory usage. Thank you again!

You are welcome.
I just added proto = None; logits = None; loss = None to the end of each loop, so that gradient graph will be cleared before next loop.

Sorry to disturb you, but I just want to share you with my bug. I have found that what caused my GPU memory surged is in this part of my own code :
https://github.com/cyvius96/prototypical-network-pytorch/blob/master/train.py#L117
I mistakenly wrote "vl.add(loss)" without the ".item()"

it still occurs "CUDA error: out of memory" and the bug is in this line.i insert print(f'batch : {i_batch}') in this line,when it reach i_batch = 15,it will occur "CUDA error: out of memory". and my device is TITAN Xp with 12G memory
thank you in advance


Fix integration test - Division of different units is not supported

Describe the bug

To Reproduce
Steps to reproduce the behavior:

Run following integration test UseofHighRiskMedicationsintheElderlyFHIR -> test case 3201fe4a-d1b0-4296-a60e-8680247e8128
See error

Version used:

Version: develop-2.0 (commit https://github.com/FirelyTeam/firely-cql-sdk/commit/25584e496722c12894d706b6757fd4cd0f52c074)

Confirmed in latest develop-2.0 after mering in cql-to-elm. Need to confirm after merging in scopes branch as well


Add kafka data recorder dd metrics

Codecov Report

Merging #178 into master will decrease coverage by 0.42%.
The diff coverage is 36.36%.

@@            Coverage Diff            @@
##           master    #178      +/-   ##
=========================================
- Coverage   91.13%   90.7%   -0.43%
=========================================
  Files          25      25
  Lines        1399    1410      +11
=========================================
+ Hits         1275    1279       +4
- Misses         91      98       +7
  Partials       33      33

Impacted Files
Coverage Δ

pkg/handler/data_recorder_kafka.go
81.11% <36.36%> (-6.24%)
: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 8c5b3a5...fdcb664. Read the comment docs.


fix: cant't display search icon in iterm2

You only changed this for alpha dashboard. Do the rest of the dashboards appear fine in iterm2 (I would guess not, since this seems to be a code point issue)? If not, maybe you could do the same for the other dashboards, since users might use different dashboards that come with LazyVim?

fix: https://github.com/LazyVim/LazyVim/issues/1139#issuecomment-1806858433


Incorrect parsing of empty strings as object member names

Version <IP_ADDRESS>
For fun, I unflattened the convoluted document from my comment in #26:
{
	"": {
		"": [
			{
				"": "",
				"txDefinition": {
					"repeatRate": 0,
					"canPort": 0,
					"messageId": 0
				}
			}
		]
	}
}

When I re-flatten this again, I get:
[
        [
        [
                        [
                "",
                {
                    "repeatRate":0,
                    "canPort":0,
                    "messageId":0
                }
            ]
        ]
    ]
]

Objects have been transformed into arrays.
Also, notice the weird indentation of the innermost and 2nd-innermost arrays at the output.

The output is correct if I replace the empty names ("") with non-empty ones.
See https://stackoverflow.com/questions/33254385/can-a-json-name-value-pair-have-an-empty-name -- Empty strings are valid names for object members.

For what it's worth, I think we've resolved the "weird indentation" issue #31


Remote Workers NOT Polling for Tasks

I have the below application setup.

Conductor Server - Cloud Container - No Security
Java Worker (Spring Boot) - Local Machine

Task is registered with Conductor no issues.
Task Def is created by calling API programatically no issues.
Workflow created by calling API from Postman.
Task Type is SIMPLE
task name is matching the registered task name.
Worker is NOT polling the Task.
It is forever in Scheduled status.
If I try to replicate the same by having the Conductor Server run in local docker container, there seems to be no issue.
There are no task domain used. A dummy worker which prints a line in console. A workflow without any input , just to have that one task.
Just a side note, a Python worker on the other hand is able to poll from local docker container as well as from cloud conductor container.
Worker Registration Code
logger.info("Registering Sub Workers ");
	TaskClient taskClient = new TaskClient();
	logger.info("Pointing to Conductor Server " + CONDUCTOR_SERVER_URL);
	taskClient.setRootURI(CONDUCTOR_SERVER_URL);		//Point this to the server API

	int threadCount = 2;			//number of threads used to execute workers.  To avoid starvation, should be same or more than number of workers

	Worker urlCheckerSubWorker = new URLCheckerSubWorker(TASK_CHECK_URLS, WORKER_NAME);
	Worker emailSenderSubWorker = new EmailSenderSubWorker(TASK_SEND_EMAIL, javaMailSender, WORKER_NAME);

	//Create WorkflowTaskCoordinator
	logger.info("Creating WorkflowTaskCoordinator " + urlCheckerSubWorker + threadCount + taskClient);
	WorkflowTaskCoordinator.Builder builder = new WorkflowTaskCoordinator.Builder();
	WorkflowTaskCoordinator coordinator = builder.withWorkers(urlCheckerSubWorker,emailSenderSubWorker,controlMWorker).withThreadCount(threadCount).withTaskClient(taskClient).build();

	//Start for polling and execution of the tasks
	logger.info("Initializing Coordincator ");
	coordinator.init();
	logger.info("Completed Coordinator Initialization "+ coordinator.getWorkerNamePrefix());

Workflow Definition is as below
{
"ownerApp": "Test_5_Control_M_Test_5_Control_M",
"createTime":<PHONE_NUMBER>352,
"createdBy": "AutoPoP Workflow Generator",
"name": "Test_5_Control_M",
"description": "Test_5_Control_M",
"version": 1,
"tasks": [
{
"name": "TASK_CHECK_URLS",
"taskReferenceName": "TK-05_09_19_02_55_11_005",
"description": "TK-05_09_19_02_55_11_005"
},
"type": "SIMPLE",
"startDelay": 0,
"optional": true,
"rateLimited": false
}
],
"schemaVersion": 2,
"restartable": false,
"workflowStatusListenerEnabled": false
}

Looks like you should check network issues

Can you change your Worker's log4j settings so that it prints at a DEBUG level? The conductor worker code should print out what it's doing behind the scenes (like each polling call)

you can enabled logger com.netflix.conductor.service.TaskServiceImpl on conductor side to see for sure


Adding the into_inner and set_blocking  make async-process more flexible.

Adding the into_inner and set_blocking api allows the user to easily get the raw standard input and output.
And to chain a series of processes-child together so that the standard output of the previous process-child can be the standard input of the next process-child.

@taiki-e Hi, my friend, I have a PR about async-process .
I hope you can combine it if you are free, or please give me some suggestions for improvement.
I'm sorry for contacting you on comments because I didn't see your public email address on GitHub.

Hi @taiki-e  , I'm sorry to bother you, I updated my pr hope you have time to review it, if there are unreasonable places I promptly correct, very much hope to support this feature.

@BinChengZhao
I tend to think it is not preferable to expose the set_blocking function. So, I think the main options here are either of the following.

Add APIs that converting from async_process::ChildStd* directly to Stdio. During the conversion process, std::process::ChildStd* is set to blocking mode.
Add APIs that converting from async_process::ChildStd* to std::process::ChildStd*. Unlike the current implementation of this PR, during the conversion process, std::process::ChildStd* is set to blocking mode.

I have no very strong opinion as to which is better, at this time. What do you think?

Unlike the current implementation of this PR, during the conversion process, std::process::ChildStd* is set to blocking mode.

Well, this may confuse the user as it does not match the behavior of async_io::Async::into_inner...

@BinChengZhao
I tend to think it is not preferable to expose the set_blocking function. So, I think the main options here are either of the following.

Add APIs that converting from async_process::ChildStd* directly to Stdio. During the conversion process, std::process::ChildStd* is set to blocking mode.
Add APIs that converting from async_process::ChildStd* to std::process::ChildStd*. Unlike the current implementation of this PR, during the conversion process, std::process::ChildStd* is set to blocking mode.

I have no very strong opinion as to which is better, at this time. What do you think?

@taiki-e
Yes, I was thinking the same as you, that if we do this choice for the user (set blocking directly - during the conversion process), this is not necessarily what the user wants most.
So, I see that we can keep the API into_inner and add a new API to support this implementation: Add APIs that converting from async_process::ChildStd* directly to Stdio. the conversion process, std::process::ChildStd* is set to blocking mode.
What do you think?
To summarize.
Keep into_inner
Remove set_blocking.
Add new API to go directly to Stdio

It seems we don't have an actual use case for into_inner, so I'm starting to feel that only having API to go directly to Stdio is ok for now.

It seems we don't have an actual use case for into_inner, so I'm starting to feel that only having API to go directly to Stdio is ok for now.

Yes I agree with you on this point, I will have perfected the function and may need to bother you again : ) .

@taiki-e
Thank you for your valuable suggestions, I have optimized all the code with the suggestions.

I followed up on the recent changes to the master branch.

LGTM aside from a nit.

Thank you. I got rid of the last problem.

@taiki-e :
Excuse me , I would like to ask when this change will go to the master branch and be published to crates.io?

Published in 1.1.0.

LGTM. Thanks @BinChengZhao
P.S. In general, I don't recommend pinging at intervals shorter than a week in OSS. I understand your feeling that you want the PRs to merge as soon as possible, but the maintainers have lives of their own and are not always able to be involved in OSS as often.

@taiki-e
I am sorry for my eagerness and for disturbing your life.
Thank you very much for guiding me during this time, I have learned a lot after this experience and after that I will learn this more to contribute with patience and persistence.
I talked to my friends in the Chinese community about you yesterday and they all admire your engineering skills and I urge everyone to get involved in smol-rs sometime.
Finally, thank you again!


[QUESTION] Will the release 0.3.0 be published to Maven Central?

Hi,
Thanks for release v0.3.0. Do you have plans to publish it to Maven Central?
Thanks!

The corresponding GitHub Action was going to. But the build failed for no reason. It's really annoying. Will retry again...

@rmberne @javmarina
The version 0.3.0 has been published to the central repository (listing).


Add interpolation support selector.

Update phpunit 7.
Fix compile code like:
@at-root a#{&} {
}

@kingyond mentions this fork in https://github.com/leafo/scssphp/issues/616#issuecomment-455481858 but for some reason hasnt opend a PR. I think there's a couple issues with the fix, but I figure a PR will at least better organize discussion about it. @mind-bending-forks also notes that they believe this fixes #616 as well https://github.com/leafo/scssphp/issues/616#issuecomment-464900930

Merged #643


Syncing from upstream zalando/patroni (master)

bt_gitbot

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.009%

Covered Lines:
11528

Relevant Lines:
11546

💛  - Coveralls


Update index - Copy (10).md

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

realtimeinsighttest/1/index - Copy (10).md
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

realtimeinsighttest/1/index - Copy (10).md
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

realtimeinsighttest/1/index - Copy (10).md
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

realtimeinsighttest/1/index - Copy (10).md
:white_check_mark:Succeeded
View

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

OPS Build status updates of commit 688aa14:
:clock10: Preparing: average preparing time is 1 min(s) 15 sec(s)

OPS Build status updates of commit 688aa14:
:clock10: Incremental building: average incremental building time is 29 sec(s)

OPS Build status updates of commit 688aa14:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

realtimeinsighttest/1/index - Copy (10).md
:white_check_mark:Succeeded
View

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


Find metadata

@mszinte:

Figure out if the following parameters can actually be changed by the researcher when using EyeLink or if pre-configured and always the same for EyeLink. If changeable, figure out where to find information about it:

[ ] EnvironmentCoordinates
[ ] SampleCoordinateUnit
[ ] SampleCoordinateSystem

Figure out if SR Research stores the following information somewhere and if yes, where to find it:

[ ] StimulusPresentation.ScreenSize
[ ] StimulusPresentation.ScreenDistance
[ ] EyeTrackerDistance
[ ] ScreenAOIDefinition
[ ] SoftwareVersions

Let @julia-pfarr where to find StartTime and StopTime in events asc file.

I looked at the other testfiles to find more information about StartTime and StopTime:

The edfs from OpenNeuro Dataset only have one "START" and one "END" entry in the asc-events file, so that would make sense as StartTime and StopTime.
Another testfile emg.edf has 3 starts and 3 ends.
The file lt_02.EDF_events.asc which I used first has 192 starts and ends/stops.

So my question is still if the lines "START	9199339 	LEFT	EVENTS" and "END  3856047  EVENTS  RES  38.57  38.52" in the asc file correspond to StartTime and StopTime in BIDS. If yes, we need to change the Data Type from number to array of arrays or both.

Regarding RawDataFilters there are two possible entries in the events asc file:

EVENTS    --> under  you can find “FILTER” for the filter level used (0=off, 1=standard, 2=extra)(from EyeLink User Manual)

ELCL_PCR_PARAM which "relates to RAW data processing"(=answer by SR Research) and which contains two numbers but I don't know what they mean

Does one of the above correspond to the RawDataFilters intended by BIDS?

I looked at the other testfiles to find more information about StartTime and StopTime:

The edfs from OpenNeuro Dataset only have one "START" and one "END" entry in the asc-events file, so that would make sense as StartTime and StopTime.
Another testfile emg.edf has 3 starts and 3 ends.
The file lt_02.EDF_events.asc which I used first has 192 starts and ends/stops.

So my question is still if the lines "START 9199339 LEFT EVENTS" and "END 3856047 EVENTS RES 38.57 38.52" in the asc file correspond to StartTime and StopTime in BIDS. If yes, we need to change the Data Type from number to array of arrays or both.

--> put warning in code when there is more than one START and END in the asc file just keep the first START and the last END and give warning output that we made the decision to only keep first START and last END and that it's not good practice to have multiple starts and ends


Not able to load the LocalFileSystem class

When we are running the code for converting parquet file to csv locally on eclipse, it runs successfully and convert the parque file to csv but when we are running in our UI environment then it is failing saying
Caused by: java.lang.RuntimeException: java.lang.ClassNotFoundException: Class org.apache.hadoop.fs.LocalFileSystem not found.

Please help us on it
Reporter: Persistent NGP
Note: This issue was originally created as PARQUET-1231. Please see the migration documentation for further details.

Qinghui Xu / @qinghui-xu:
It sounds like a classpath/packaging problem. The org.apache.hadoop.fs.LocalFileSystem is not in your runtime classpath. Try to add hadoop-common into your runtime classpath, this might solve your problem.
This is not a problem related to parquet.


minCleanUpDelay setting isn't applied during the first run of the EvictionTask

Redis version
any
Redisson version
any
Redisson configuration
any
What is the Expected behavior?
No response
What is the Actual behavior?
During the first run of the EvictionTask delay is always 5 seconds.
Additional information
No response

Fixed


SDL2 #include path issue

I noticed bgfx is including SDL2 headers by #include<SDL2/SDL_*.h>.
But I think #include <SDL_*.h> is better. Because SDL2 compiled library for windows is not have include/SDL2 directory and pkg-config returns /usr/include/SDL2 for include path. In addition, README.md and scripts/example-common.lua seems created for #include <SDL_*.h>.
:sushi:

I thought about this before, but Windows users can just rename SDL2 directory or create copy.

OK thx for reply

I'm hoping SDL team will make this consistent with OSX and Linux on their side and then it all will just work. :)


Add spanish locals

I'm not sure if this is intended, but I found it useful while working on a project.
Let me know if you want me to include this into devise_invitable generators some way, or if you think it could be useful for someone else as reference.

Thanks, but translations are in wiki: https://github.com/scambra/devise_invitable/wiki/I18n
You can update wiki if some translation is missing or wrong
devise_invitable doesn't include translations, as devise doesn't


updated contributor

Good work. I am merging your PR.
Do give us a star by hitting the star button of this repo!


[FEA] AQE and DPP work together in rapids

Is your feature request related to a problem? Please describe.
want DPP and AQE can work together in rapids
@jlowe  @revans2

What exactly do you mean/want?  Spark itself limits the use of DPP with AQE.  Are you expecting the plugin to override this functionality that is built into spark? I personally think that if we have a solution for this it should go into spark itself, unless there is some part of running on a CPU that would make it impossible compared to doing it on the GPU.

What exactly do you mean/want? Spark itself limits the use of DPP with AQE. Are you expecting the plugin to override this functionality that is built into spark? I personally think that if we have a solution for this it should go into spark itself, unless there is some part of running on a CPU that would make it impossible compared to doing it on the GPU.

@chenrui17 here

What exactly do you mean/want? Spark itself limits the use of DPP with AQE. Are you expecting the plugin to override this functionality that is built into spark? I personally think that if we have a solution for this it should go into spark itself, unless there is some part of running on a CPU that would make it impossible compared to doing it on the GPU.

well , in spark3.0 now , DPP and AE can not work together , we want to add a rule of AQE  based https://github.com/apache/spark/pull/29224/files to make it work together .
and now I met a question  : i found that GpuBroadCastHashJoinExec , GpuBroadCastExchangeExec , etc is in shims project ,  but I want to modify Plugin.scala ,and it is in sql-plugin ，and  i need call GpuBroadCastHashJoinExec and GpuBroadCastExchangeExec in sql-plugin , but when I add pom dependencies in sql-plugin/pom.xml，it occurs error like this:

what should i do ?

What exactly do you mean/want? Spark itself limits the use of DPP with AQE. Are you expecting the plugin to override this functionality that is built into spark? I personally think that if we have a solution for this it should go into spark itself, unless there is some part of running on a CPU that would make it impossible compared to doing it on the GPU.

well , in spark3.0 now , DPP and AE can not work together , we want to add a rule of AQE based https://github.com/apache/spark/pull/29224/files to make it work together .
and now I met a question : i found that GpuBroadCastHashJoinExec , GpuBroadCastExchangeExec , etc is in shims project , but I want to modify Plugin.scala ,and it is in sql-plugin ，and i need call GpuBroadCastHashJoinExec and GpuBroadCastExchangeExec in sql-plugin , but when I add pom dependencies in sql-plugin/pom.xml，it occurs error like this:

what should i do ?

Error:java: Annotation processing is not supported for module cycles. Please ensure that all modules from cycle [rapids-4-spark-shims-spark300_2.12,rapids-4-spark-shims-spark301_2.12,rapids-4-spark-sql_2.12] are excluded from annotation processing

https://github.com/apache/spark/pull/29224 was done to allow AQE to work with the rapids plugin which went into spark 3.0.1 and 3.1.0.
This means that if you use a newer version of the plugin 0.2 + against spark version 3.0.1+ or 3.1.0+ then AQE will work.
What I was referring to is that there are checks inside the AQE code that when it detects that a stage has DPP processing enabled it will not enable AQE for that stage too.
The problem you are running into is how our shim system works. @tgravescs please correct me if I get something wrong.
We have a common core for code that works against all version of spark that we support.  Then we have a separate shim layer for each specific version of spark so we can add operators to the base.  The proper implementation of the shim layer is detected at run time and dynamically loaded.
The base provides an abstract API SparkShims that lets the core/base part of the plugin interact with they shims as needed. This allows us to avoid the issue you are running into.  If you need to get access to something that is only in the shim layer (because in this case the join base class changed between releases in an incompatible way) then you would need to go though the abstract API to get access to it.

What I was referring to is that there are checks inside the AQE code that when it detects that a stage has DPP processing enabled it will not enable AQE for that stage too.

if we want AQE and DPP to co-work,  does these exists some rapids-specific code need change? @revans2

those aren't supported together yet in Spark itself - see https://issues.apache.org/jira/browse/SPARK-30186
Generally we wouldn't support it until Spark itself supports it.

Given that Spark does not support AQE and DPP together, I am marking this closed.


Corrected description of document images

Make sure set the target branch to develop
What is the purpose of the change
Corrected the incorrect description of the Producer module in this picture (Incorrectly introduced Consumer in the Producer module)
Brief changelog
Graphic correction
Verifying this change
https://github.com/apache/rocketmq/issues/2855#issue-873498812

Pls resolve conflict and rebase your commits.

Please... make your pr request to develop and do not submit repeated PRs:-(

Sry for that，I will pay more attention next time.
发自我的iPhone
------------------ Original ------------------
From: von gosling @.>
Date: Fri,May 21,2021 10:34 AM
To: apache/rocketmq @.>
Cc: b1n90 @.>, Author @.>
Subject: Re: [apache/rocketmq] Corrected description of document images (#2857)
Please... make your pr request to develop branch and do not submit repeated PRs:-(
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


'Page environment' builtin data source

Great documentation, very useful! However, would it be possible to get more information about the builtin data source like "Page environment" especially the "Query String/*** Query Parameters" and  "Query String/*** URL fragment" properties? What I see for now is these properties are only retrieved on the first page load and not dynamically. It is useful of course, but a dynamic behavior would be even better. More explanations on the behavior would be appreciated!

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

ID: 79444829-20bc-3b07-8820-29b55798761a
Version Independent ID: 546cff82-d871-917e-10ba-55df6f18e5eb
Content: Connect SharePoint Framework components using dynamic data
Content Source: docs/spfx/dynamic-data.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Thanks for your post. Due to many docs and maintained by the community, if possible, feel free to submit any pull request for contribution.

What scenario are you seeing the query string get updated but not the dynamic property?  They should be getting updated (and a change triggered) along with the other properties like item id, etc.

@patmill The value is only retrieved during the initial page load. If I try to change this value dynamically (just by adding a '#' in URL or manipulaing browser history), the data source value is not updated. I use this sample for my test  where I connect the Search Box WP component to a query string or a URL fragment value from 'Page environment' source, for instance, to get and execute a query coming from another page. It works well during the first page load for this specific scenario and the value is retrieved as expected.
However, I was thinking about a scenario where the user could update the query afterwards, for instance by clicking on predefined links from another component (i.e. prebuilt search queries in a 'verticals' WP) without refreshing the whole page every time.
I did similar feature for one of my clients using an SPFx extension as data source and by manipulating the browser history but I think it is quite overkill since the value is already provided by the default 'Page environment' source so my question.
Maybe also you would know a better way to implement this scenario.

Update: subscribing to the 'hashchange' event manually when the source property is set 'PageContext:UrlData:fragment' does the job for my use case.
private _bindHashChange() {

    if (this.properties.defaultQueryKeywords.tryGetSource()) {
        if (this.properties.defaultQueryKeywords.reference.localeCompare('PageContext:UrlData:fragment') === 0) {
            // Manually subscribe to hash change since the default property doesn't
            window.addEventListener('hashchange', this.render);
        } else {
            window.removeEventListener('hashchange', this.render);
        }
    }
  }


Prepare inbox API

According a https://github.com/wultra/enrollment-server/issues/493 (still a draft)
This should be standalone POD (as push and operations are)

Closed by #100


Create new Invitation - v2

Created by: @yetizer Oct 19 2020 at 07:38 CET
Last updated: Jul 06 2021 at 17:25 CET
ClickUp ID: IK-283
Application Agent calls the SDK to invite a new User. From the agent credential the CustomerID, AppSpaceID and ApplicationID is known.
The Agent sends the following gRPC request to the server:
message CreateInvitationRequest {
bytes tenant_id = 1 [(google.api.field_behavior) = REQUIRED];
//External reference identifier to correlate the lifecycle events with.
string reference_id = 2;
//Human readable name of invitee.
string display_name = 3 [(google.api.field_behavior) = REQUIRED];
id.indykite.jarvis.objects.MapValue attributes = 4;
google.protobuf.Timestamp create_time = 5;
google.protobuf.Timestamp expire_time = 6;
oneof invitee {
// Address where email will be sent.
string email = 7;
// Mobile number where sms will be sent.
string mobile = 8;
}
bytes admin_token = 9;
}
We MUST make sure the TenantID is within the given AppSpace!
The admin_token can be used to linked the person who created the invitation.
Same person may receive to the same email multiple different invitation and could use the same OAuth2 provider to register. There is no safe way to find out who is really invited. It should be part of the configuration to force uniqueness of email or mobile.
We don't know if the invitee has already a digital_twin or not so we may not create one upfront but the invitation should be somehow visible to pre-assign some permissions so when user is onboarded and digital_twin id became fix it can inherit those permissions.
Code Documentation - Need
CreateInvitationTokenCypher
Input Parameters:
- digital_twin_id
- tenant_id
- create_time
- expire_time
- token_id
- invitee_hash (email / mobile - hashed value for search )
- digital_twin_id (??? shell we create something)
//These can be stored as one byte blob:
- display_name
- invitee (email / mobile - encrypted value )
- attribute_type (email or mobile )
- data (binary blob to store custom attributes )
- data_type (type URL for blob to de-serialize )
Output Object:
success (non empty output) or fail (empty)
Cypher:
Sample parameters
:params {app_space_id: '696e6479-6b69-4465-8000-010000000000', tenant_id: '696e6479-6b69-4465-8000-030000000001', digital_twin_id: '671c1690-2d4b-47a9-9b1b-281c1b9316a2', credential_id: '2a32df2e-54ab-435e-8692-0cfde014a8aa', email: {email: 'xxxx', email_hash: 'yyy', confirmed: false}, token_id: '12f67030-5958-4010-84f5-24a5368b3acc', invitation: {create_time: 'today', expire_time: 'next week', display_name: 'John Doe', reference_id: 'unknown'}}
Query
Check if user is already registered with this email.
OPTIONAL MATCH (apps:AppSpace {id:  $ app_space_id})-[:HAS*2]->(t:Tenant {id:  $ tenant_id})
OPTIONAL MATCH (e:Email:Credential {email_hash:'yyy'})
OPTIONAL MATCH (t)<-[:REGISTERS]-(d:DigitalTwin)-[:CONTROLS]-(e)
RETURN  {digital_twin_id:d.id, credential_id:e.id, tenant_id:t.id}
Create an Invitation
MATCH (apps:AppSpace {id:  $ app_space_id})-[:HAS*2]->(t:Tenant {id:  $ tenant_id})
CREATE (d:DigitalTwin {id:  $ digital_twin_id})-[:REGISTERS]->(t)
MERGE (c:Credential:Email {id:  $ credential_id})
ON CREATE
SET c +=  $ email
CREATE (d)-[:CONTROLS]->(c)<-[:WITH]-(inv:Invitation {id:  $ token_id})-[:INVITEE]->(d)
SET inv +=  $ invitation
RETURN {app_space_id:apps.id}
Read invitation from DB:
MATCH (apps:AppSpace)-[:HAS*2]->(t:Tenant)<-[:REGISTERS]-(d:DigitalTwin)-[cr:CONTROLS]->(c:Email)<-[:WITH]-(inv:Invitation {id:  $ token_id})-[:INVITEE]->(d)
RETURN {app_space_id:apps.id, tenant_id:t.id, digital_twin_id:d.id, credential_id:c.id, credential_type:labels(c), credential:properties(c), credential_control:properties(cr) , invitation:properties(inv)}
--Ignore
MATCH (d:DigitalTwin)<-[:INVITEE]-(n:Invitation {reference_id:'unknown'})-[:WITH]->(c) DETACH DELETE d,n,c
Change History:
Checklist

[ ] Create the cypher query
[ ] Implement and apply the cypher query

Commented by: @marvinkite Oct 19 2020 at 07:38 CET
@yetizer I created these cyphers for testing the invitation. What do you think?

Commented by: @yetizer Oct 19 2020 at 07:38 CET
/customer_id, app_space_id and application_id can be replaced with agent_id

Commented by: @yetizer Oct 19 2020 at 07:38 CET
we don't know credential fo invitee, it may not exist yet and even if it exists email doesn't have to be credential even if we use it to invite person

Commented by: @yetizer Oct 19 2020 at 07:38 CET
we shouldn't store email in plain text, it should be encrypted


Merge pull request #1 from OfficeDev/master

Updating from the master

Hi @iddycol, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


Update qb-core-qb-multicharacter.md

Fixed by https://github.com/iLLeniumStudios/docs.illenium.dev/commit/7930ab7dc92e38930c947574ff3842ab71cc1aca


[bn] Localize(updated) content/bn/releases/_index.md

Localize _index.md in Bengali language.
fixes issue:#44993

@saifulshark Thanks you for your contribution.
Could you please sign the CLA before the PR can be reviewed.
You can follow the steps documented here: CLA Steps

Successfully signed the agreement as an individual.
@asem-hamid did the work based on your review.
Thanks.

@asem-hamid please check out now,did this.

/lgtm


release: v0.9.3

Description: 概要

Changes: 変更内容

Changes

chore: tweaks (6aefc8807bc9012ef4ccd3ee51b80ff170658ab9)
chore: disable button while sending (9dae517b87019d94e5c955aae94fe6b22f7490e0)

Expected Impact: 影響範囲

Operating Requirements: 動作要件

Additional context: 補足

Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/bin

yarn update
yarn run v1.22.5
$ yarn update:core && yarn workspaces run update && yarn install && yarn upgrade && yarn audit
$ npx npm-check-updates -u --timeout 100000
Upgrading /home/runner/work/frourio-demo/frourio-demo/package.json

 @types/node  ^14.14.22  →  ^14.14.24

Run npm install to install new versions.

> @frourio-demo/constants
$ npx npm-check-updates -u --timeout 100000
Upgrading /home/runner/work/frourio-demo/frourio-demo/shared/constants/package.json

All dependencies match the latest package versions :)

> @frourio-demo/types
$ npx npm-check-updates -u --timeout 100000
Upgrading /home/runner/work/frourio-demo/frourio-demo/shared/types/package.json

All dependencies match the latest package versions :)

> @frourio-demo/utils
$ npx npm-check-updates -u --timeout 100000
Upgrading /home/runner/work/frourio-demo/frourio-demo/shared/utils/package.json

All dependencies match the latest package versions :)

> @frourio-demo/admin
$ npx npm-check-updates -u --timeout 100000 -x @date-io/date-fns,next-transpile-modules
Upgrading /home/runner/work/frourio-demo/frourio-demo/apps/admin/package.json

 marked          ^1.2.8  →     ^1.2.9
 @types/node  ^14.14.22  →  ^14.14.24

Run npm install to install new versions.

> @frourio-demo/front
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
info Visit https://yarnpkg.com/en/docs/cli/workspaces for documentation about this command.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

stderr:
npx: installed 318 in 17.811s

npx: installed 318 in 7.271s

npx: installed 318 in 7.326s

npx: installed 318 in 7.493s

npx: installed 318 in 6.98s

error<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
error Commands cannot run with an incompatible environment.
error Command failed.
Exit code: 1
Command: /usr/local/bin/node
Arguments: /usr/share/yarn/lib/cli.js run update
Directory: /home/runner/work/frourio-demo/frourio-demo/apps/front
Output:

error Command failed with exit code 1.

Changed files

Changed 2 files:

apps/admin/package.json
package.json

:octocat: Repo | :memo: Issues | :department_store: Marketplace

Codecov Report

Merging #84 (d302c1b) into main (61972e7) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #84   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          123       123
  Lines         1734      1734
  Branches       173       173
=========================================
  Hits          1734      1734

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 61972e7...d302c1b. Read the comment docs.


Updated README

I have updated the README file to provide more information for new users.
Changes:

usage types
highlighting for examples
godoc reference link
some minor edits.

Thank you


Mismatch between the network annotation in  - K8s plumbing working group documentation and the implementation

kind: Pod
metadata:
  name: my-pod
  namespace: my-namespace
  annotations:
    kubernetes.v1.cni.cncf.io/networks:  netA,netB,other-ns/netC

Implementation: https://github.com/Intel-Corp/multus-cni/blob/122dbfb345ae4a13fd1d592723a0ba1603278dd9/multus/multus.go#L278

@dougbtv @s1061123 : I think, we have to change the documentation for it.

Yep, this is definitely a mistake here, I don't think the version string got attached. Here's the sample pod I was working from:
apiVersion: v1
kind: Pod
metadata:
  name: bothpod
  annotations:
    kubernetes.cni.cncf.io/networks: macvlan-conf,flannel-conf
spec:
  containers:
  - name: bothpod
    command: ["/bin/bash", "-c", "sleep<PHONE_NUMBER>000"]
    image: dougbtv/centos-network
    ports:
    - containerPort: 80

And that pod really needs to be...
apiVersion: v1
kind: Pod
metadata:
  name: bothpod
  annotations:
    kubernetes.v1.cni.cncf.io/networks: macvlan-conf,flannel-conf
spec:
  containers:
  - name: bothpod
    command: ["/bin/bash", "-c", "sleep<PHONE_NUMBER>000"]
    image: dougbtv/centos-network
    ports:
    - containerPort: 80

I'll submit a fix for this one -- thanks for the eyes Kural! (I also left a comment on the spec in response to yours, too.

Thanks @dougbtv

@s1061123 @dougbtv Creating a patch for it along with k8s-client package

Kural, cool if you have it in the dev/nwpg-crd-changes-k8s-client-feature branch -- that's good, otherwise, I'd wait until that change is complete because it moves all that logic to a new file. I just previewed that branch, looks good, and thanks for kicking that off.

#58 Fixed in the PR


updating the low-level test spec for the HDMI input module

Creating pull request for the L2 low level spec for the HDMI input module.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

This is no more required.  All the L2 low-level spec  and L2 Test code are part of different PR.
https://github.com/rdkcentral/rdk-halif-test-device_settings/pull/32


Allow setting a resolver in the OCI registry client

What this PR does / why we need it:
As OCI was moved out of /experimental, third-party developers are using Helm 3.8 as their entrypoint to deal with OCI repositories (instead of dealing with ORAS directly). However, there is still some margin of improvement on the customization capabilities offered to those developers.
This PR is simply to add a way to enable devs to set up a custom containerd resolver for their OCI registry client.
Besides, it also exports a couple of fields, so that it becomes easier for devs to mock the PullResult struct in their unit tests.
closes #10623 (*)
(*) In that issue I mentioned some ways to improve the customization of the registry client, but, rather than adding a myriad of setters or ClientOpts for each aspect (custom headers, custom LoginOpts, etc.) setting a custom resolver makes the trick for us, at least.
Special notes for your reviewer:
After having reviewed the test suite for the client registry, I haven't found any existing code merely testing the ClientOpts, so I just added a ClientOptResolver(nil) to ensure the test coverage. Although the logic is trivial, please let me know if any other testing approach is required on your side.
I can confirm this change works for us: https://github.com/kubeapps/kubeapps/pull/4216
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[x] this PR has been tested for backwards compatibility

@jdolitsky Do you think we can merge this PR for the upcoming 3.10.0 release ?? 🙏

@antgamdia Can you please rebase?

@antgamdia Can you please rebase?

@joejulian Sure thing! Done :) Happy to see we are finally moving forward with this PR 🎉


accessiblity feature: scroll focused suggestion into view

It would be great, if the focused suggestion could use scrollIntoView so that we could ensure it was visible.
Two options I can see for how to implement this:

onSuggestionFocused could pass the full focus event as a second parameter so that users could use DOM APIs to implement this themselves.
Just have this lib call scrollIntoView on focus.

@joeybaker We already have a similar issue.
Thanks a lot for your suggestions! I adapted the scrollIntoView approach.
It's fixed in v1.14.1

Sorry for the dupe! Thank you!


21.11: perlPackages.CompressRawZlib fails to build

Describe the bug
Since a nix-channel --update, perlPackages.CompressRawZlib fails to build. I think this is required by libreoffice.
Steps To Reproduce
$ nix-build '<nixpkgs>' -A perlPackages.CompressRawZlib
these derivations will be built:
  /nix/store/pvm4p451dvfy06qdps8v9bd0pcnrqjqb-perl5.34.0-Compress-Raw-Zlib-2.101.drv
building '/nix/store/pvm4p451dvfy06qdps8v9bd0pcnrqjqb-perl5.34.0-Compress-Raw-Zlib-2.101.drv'...
unpacking sources
unpacking source archive /nix/store/vz6jq6hphynix8y1y31m707hmrp05jia-Compress-Raw-Zlib-2.101.tar.gz
source root is Compress-Raw-Zlib-2.101
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file Compress-Raw-Zlib-2.101/META.yml
patching sources
configuring
patching ./Makefile.PL...
patching ./examples/filtinf...
patching ./examples/filtdef...
Parsing config.in...
Auto Detect Gzip OS Code..
Setting Gzip OS Code to 3 [Unix/Default]
Looks Good.
Up/Downgrade not needed.
Checking if your kit is complete...
Looks good
Generating a Unix-style Makefile
Writing Makefile for Compress::Raw::Zlib
Writing MYMETA.yml and MYMETA.json
no configure script, doing nothing
building
build flags: SHELL=/nix/store/dznl6xxx7bp9xynmsbrjjba9x726m3f9-bash-5.1-p8/bin/bash
cp lib/Compress/Raw/Zlib.pm blib/lib/Compress/Raw/Zlib.pm
Running Mkbootstrap for Zlib ()
chmod 644 "Zlib.bs"
"/nix/store/yws26zrhcm0sfb6766fpbr1zlk497w3d-perl-5.34.0/bin/perl" -MExtUtils::Command::MM -e 'cp_nonempty' -- Zlib.bs blib/arch/auto/Compress/Raw/Zlib/Zlib.bs 644
"/nix/store/yws26zrhcm0sfb6766fpbr1zlk497w3d-perl-5.34.0/bin/perl" "/nix/store/yws26zrhcm0sfb6766fpbr1zlk497w3d-perl-5.34.0/lib/perl5/5.34.0/ExtUtils/xsubpp"  -typemap '/nix/store/yws26zrhcm0sfb6766fpbr1zlk497w3d-perl-5.34.0/lib/perl5/5.34.0/ExtUtils/typemap' -typemap '/build/Compress-Raw-Zlib-2.101/typemap'  Zlib.xs > Zlib.xsc
mv Zlib.xsc Zlib.c
cc -c  -I/nix/store/s8dchzan4ghjkqrgbdsnhwc36imak1d2-zlib-1.2.12-dev/include -D_REENTRANT -D_GNU_SOURCE -fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -I/no-such-path/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -O2   -DVERSION=\"2.101\" -DXS_VERSION=\"2.101\" -fPIC "-I/nix/store/yws26zrhcm0sfb6766fpbr1zlk497w3d-perl-5.34.0/lib/perl5/5.34.0/x86_64-linux-thread-multi/CORE"  -DNO_VIZ -DZ_SOLO   -DGZIP_OS_CODE=3 -DUSE_PPPORT_H Zlib.c
rm -f blib/arch/auto/Compress/Raw/Zlib/Zlib.so
LD_RUN_PATH="/nix/store/9vdc8d0gfmjzb0bqp7xyzjba69azqs9b-zlib-1.2.12/lib" cc  -shared -O2 -L/nix/store/rxjh6jxg7ksisbcxl0cjwcff5iprpdfz-glibc-2.33-123/lib -fstack-protector-strong  Zlib.o  -o blib/arch/auto/Compress/Raw/Zlib/Zlib.so  \
   -L/nix/store/9vdc8d0gfmjzb0bqp7xyzjba69azqs9b-zlib-1.2.12/lib -lz   \

chmod 755 blib/arch/auto/Compress/Raw/Zlib/Zlib.so
Manifying 1 pod document
running tests
check flags: SHELL=/nix/store/dznl6xxx7bp9xynmsbrjjba9x726m3f9-bash-5.1-p8/bin/bash VERBOSE=y test
"/nix/store/yws26zrhcm0sfb6766fpbr1zlk497w3d-perl-5.34.0/bin/perl" -MExtUtils::Command::MM -e 'cp_nonempty' -- Zlib.bs blib/arch/auto/Compress/Raw/Zlib/Zlib.bs 644
PERL_DL_NONLAZY=1 "/nix/store/yws26zrhcm0sfb6766fpbr1zlk497w3d-perl-5.34.0/bin/perl" "-MExtUtils::Command::MM" "-MTest::Harness" "-e" "undef *Test::Harness::Switches; test_harness(0, 'blib/lib', 'blib/arch')" t/*.t
t/000prereq.t ...... # Running Perl version  5.034000
t/000prereq.t ...... ok
t/01version.t ...... ok
t/02zlib.t ......... 2/349
#     Failed test (t/02zlib.t at line 493)
#          got: 1
#     expected: -3

#     Failed test (t/02zlib.t at line 517)
#          got: 1
#     expected: -3
# Looks like you failed 2 tests of 349.
t/02zlib.t ......... Dubious, test returned 2 (wstat 512, 0x200)
Failed 2/349 subtests
t/07bufsize.t ...... skipped: Lengthy Tests Disabled
t/09limitoutput.t .. ok
t/18lvalue.t ....... ok
t/19nonpv.t ........ ok
t/99pod.t .......... skipped: Test::Pod 1.00 required for testing POD
t/meta-json.t ...... skipped: Test::CPAN::Meta::JSON required for testing META.json
t/meta-yaml.t ...... skipped: Test::CPAN::Meta required for testing META.yml

Test Summary Report
-------------------
t/02zlib.t       (Wstat: 512 Tests: 349 Failed: 2)
  Failed tests:  126, 134
  Non-zero exit status: 2
Files=10, Tests=507,  1 wallclock secs ( 0.05 usr  0.01 sys +  0.41 cusr  0.03 csys =  0.50 CPU)
Result: FAIL
Failed 1/10 test programs. 2/507 subtests failed.
make: *** [Makefile:1053: test_dynamic] Error 255
builder for '/nix/store/pvm4p451dvfy06qdps8v9bd0pcnrqjqb-perl5.34.0-Compress-Raw-Zlib-2.101.drv' failed with exit code 2
error: build of '/nix/store/pvm4p451dvfy06qdps8v9bd0pcnrqjqb-perl5.34.0-Compress-Raw-Zlib-2.101.drv' failed

Expected behavior
the package should build
Additional context
I'm reasonably confident I have no overlays that could be affecting this; examining nix show-derivation /nix/store/pvm4p451dvfy06qdps8v9bd0pcnrqjqb-perl5.34.0-Compress-Raw-Zlib-2.101.drv shows that the only things in inputDrvs are stdenv, perl, zlib, and bash, none of which I have overlays for.
Notify maintainers

@jonringer, since you have the most recent commit to pkgs/development/perl-modules
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.29, NixOS, 21.11 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.16`
 - channels(ash): `"home-manager-21.11, nixos-unstable-22.05pre365028.710fed5a248"`
 - channels(root): `"nixos-21.11.336792.6f88260faaa"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

of course I only manage to find #166910 now, apologies

I think this is required by libreoffice.

Yes. libreoffice fails to build on nixos-21.11 :cry:

Closing because https://github.com/NixOS/nixpkgs/pull/166910 was merged. We'll just have to wait for the channel to update.

That's happened already.


Adding contract interfaces for specifying runtime compilation exceptions...

....
Fixes #12

Updated the contract interface to support multiple compilation failures. We could consider moving these interfaces to KRuntime once we add support for producing exceptions annotated with this type as part of build compilation failures.

:shipit:


Show CI status of master branch in readme.

Show status of master branch, so it doesn't show the status of the last pushed branch,

I've merged that thanks - great idea!


Make curry callable as a python library addition atop of pyquil (fairly easy)

Create an interface that adds the features in curry, and make curry use pyquil Program objects internally

Done!


Refresh node graph when changing tabs

Currently if you open the node graph (node graph button in the top right) and then switch document tabs, the node graph will not update to reflect the new document until the document is interacted with. See PortfolioMessage::SelectDocument and NodeGraphMessage::SendGraph.

I have opened a PR for this. Can I get assigned?


Unable to reuse secret name in different namespaces

When creating multiple secrets with the same name in different namespaces, catalog compilation fails.
Steps to Reproduce the Problem

Use the following configuration

   secrets:
    secrets:
      namespace1/mysecret:
        stringData:
          token: "XYZ"
      namespace2/mysecret:
        stringData:
          token: "XYZ"

Actual Behavior
The catalog compilation fails with error:
Jsonnet error: failed to compile /builds/redacted/syn-tenant-repo/dependencies/secrets/component/main.jsonnet:
 RUNTIME ERROR: duplicate field name: "mysecret"
	/builds/redacted/syn-tenant-repo/dependencies/secrets/component/main.jsonnet:(35:29)-(42:2)	function <namespaced>
	/builds/redacted/syn-tenant-repo/dependencies/secrets/component/main.jsonnet:56:17-52	thunk <secrets>
	/builds/redacted/syn-tenant-repo/dependencies/secrets/component/main.jsonnet:58:49-56	thunk <resources>
	/usr/local/lib/python3.11/site-packages/commodore/lib/commodore.libjsonnet:407:33-42	thunk <o>
	std.jsonnet:1501:24
	std.jsonnet:1501:5-33	function <anonymous>
	/usr/local/lib/python3.11/site-packages/commodore/lib/commodore.libjsonnet:407:16-43	thunk
	/usr/local/lib/python3.11/site-packages/commodore/lib/commodore.libjsonnet:(403:5)-(408:6)	function <anonymous>
	/usr/local/lib/python3.11/site-packages/commodore/lib/commodore.libjsonnet:(403:5)-(408:6)
	/usr/local/lib/python3.11/site-packages/commodore/lib/commodore.libjsonnet:(401:3)-(409:4)	function <anonymous>
	/builds/redacted/syn-tenant-repo/dependencies/secrets/component/main.jsonnet:58:27-70	thunk
	std.jsonnet:1569:20	thunk <v>
	std.jsonnet:32:25
	std.jsonnet:32:16-27	thunk
	std.jsonnet:32:16-38	function <anonymous>
	std.jsonnet:32:16-38	function <anonymous>
	std.jsonnet:1569:8-22	function <anonymous>
	/builds/redacted/syn-tenant-repo/dependencies/secrets/component/main.jsonnet:58:17-71	object <anonymous>
	During manifestation
Compile error: failed to compile target: secrets

Expected Behavior
Successful catalog compilation

Fixed with https://github.com/projectsyn/component-secrets/pull/14


Card does not clip image

$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel dev, v1.2.1, on Mac OS X 10.14.3 18D109, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
[✓] Android Studio (version 3.1)
[✓] VS Code (version 1.31.1)
[✓] Connected device (2 available)

• No issues found!

import 'package:flutter/material.dart';

main() {
  runApp(_MyApp());
}

class _MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        appBar: AppBar(title: Text("Card doesn't clip images")),
        body: Center(
          child: _Card(
            image: NetworkImage(
                "https://dtpmhvbsmffsz.cloudfront.net/posts/2015/05/12/55529027d14d7b1e2f002521/m_55529027d14d7b1e2f002522.jpg"),
            title: "Does not clip upper corners",
          ),
        ),
      ),
    );
  }
}

class _Card extends StatelessWidget {
  final ImageProvider image;
  final String title;

  _Card({@required this.image, @required this.title});

  @override
  Widget build(BuildContext context) {
    return Card(
      clipBehavior: Clip.hardEdge,
      child: Column(
        crossAxisAlignment: CrossAxisAlignment.stretch,
        children: <Widget>[
          Expanded(
            child: Image(
              image: image,
              fit: BoxFit.cover,
            ),
          ),
          Padding(
            padding: EdgeInsets.all(12),
            child: Text(title),
          )
        ],
      ),
    );
  }
}

Seems to work for me

[✓] Flutter (Channel master, v1.2.2-pre.21, on Mac OS X 10.14.3 18D42, locale en-AT)

Resolved in 1.2.2?

I don't know the latest version of dev channel, but I'm sure it will be included in the next dev channel release when it happens.


Class:OrganisationalUnit

Parent
ObjectGroup

Label
Organisational Unit

Definition
A unit within an organisational hierarchy which may be at, above or below the institutional level.

Usage

Repeatable

Relationships

Potential standards/vocabularies/ontologies to adopt
AIISO (http://vocab.org/aiiso/) ; FOAF (http://xmlns.com/foaf/0.1/)

Notes
This class can represent any level of organisational unit, incorporating institutions (e.g. a museum), higher units (e.g. a university to which a museum belongs) and more detailed structures (e.g the departments and divisions within a museum). It can be used to arrange these different units at different levels into a hierarchical structure.

I think this is really out of scope of a collection description standard. It should only point to entities responsible for a collection and/or owning a collection, but I cannot think of any use case why for describing a collection it is necessary to describe how an organisation is hierarchical organised. At least it has not any scientific value. Even for asset management I do not really see the added value.

If anything, shouldn't OrganisationalUnit not have Institution as Parent instead of CollectionDescription?

This is meant to allow the documentation of organisational units responsible for the management of a collection within the institution. For example: at FM we would have separate OrganisationalUnits for Geology, Birds, Herps, Fishes, Botany etc.  Without this we would only have a single record for the whole museum.

Original Definition: An organisational unit within an institutional hierarchy
New Definition: This class provides an oppertunity to indicate where in the institutional hierarchy the collection description belongs and which unit within that institution who is responsible for its management.

Would using the generic foaf:Organisation class work? Seems to be flexible enough to cover a legal 'organization' as well as subunits like department/division (the range of which presumably would be defined by OrganisationalUnitType + a controlled vocab).
There's a few existing extensions to the foaf term in the org ontology as well, which could poss be useful in the future if we want to be able to describe pan-institution collections.

Is there a reason why the class is "OrganisationalUnit" and not "Organization"? See https://www.w3.org/TR/2014/REC-vocab-org-20140116/#overview-of-ontology .
See the description for "Organization":
"Represents a collection of people organized together into a community or other social, commercial or political structure. The group has some common purpose or reason for existence which goes beyond the set of people belonging to it and can act as an Agent. Organizations are often decomposable into hierarchical structures. "
versus for "OrganizationalUnit":
"An Organization such as a department or support unit which is part of some larger Organization and only has full recognition within the context of that Organization. In particular the unit would not be regarded as a legal entity in its own right. "
My suggestion is for the Class to be "Organization" and "organizationalUnit" to be moved and added as a new property.


Is it possible to use wildcards in selectors?

modules:
  - name: moduleABC
    properties:
      VERSION:
  - name: moduleXYZ
    properties:
      VERSION:

I want to set the version for any module in the given yaml file, without having to specify the module name or index. Is this possible?

Seeing lodash' documentation on _.set, I assume it's not possible.
However, lodash does support passing an array of paths.
It's not the same as a wildcard, but at least I could specify a number of selectors like this:
        - file: mta.yaml
          selector:
            - modules[0].properties.VERSION
            - modules[1].properties.VERSION
            - modules[2].properties.VERSION
            - modules[3].properties.VERSION
            - modules[4].properties.VERSION
          value: ${nextRelease.version}

For that to work, the validator needs to allow string or array for the selector:   https://github.com/meesvandongen/semantic-release-yaml/blob/29de253dc008153bf64e0547f2cdb81b296920f5/src/verify-commands.ts#L21
This is what happens atm:
Your configuration for the selector option is [ 'modules[0].properties.VERSION', 'modules[1].properties.VERSION', 'modules[2].properties.VERSION', 'modules[3].properties.VERSION', 'modules[4].properties.VERSION' ].

AggregateError:
    SemanticReleaseError: Invalid `selector` option.

This is my workaround, but it's not ideal because (besides the fact that the selector doesn't support an array) I have to know upfront how many 'modules' there are. If one too many 'modules' index is specified (which doesn't exit), then _.set will create it and corrupt the YAML file.
        - file: mta.yaml
          selector: modules[0].properties.VERSION
          value: ${nextRelease.version}
        - file: mta.yaml
          selector: modules[1].properties.VERSION
          value: ${nextRelease.version}

An API that is maximally flexible would be to somehow specify a function
which takes (contentAsJSObject, config) => void. Where you would then
mutate the first parameter.
Only point is that only serializable values are preferred in the config.
Which a function isn't in principle.
I'll have to check if it is possible to specify a semantic release config
in js as alternative to JSON (like is possible in eslint). Would imply this
function syntax would only be possible in the js file.
Op wo 24 aug. 2022 18:27 schreef Pieter Janssens @.***>:

This is my workaround, but it's not ideal because (besides the fact that
the selector doesn't support an array) I have to know upfront how many
'modules' there are. If one too many 'modules' index is specified (which
doesn't exit), then _.set will create it and corrupt the YAML file.
    - file: mta.yaml
      selector: modules[0].properties.VERSION
      value: ${nextRelease.version}
    - file: mta.yaml
      selector: modules[1].properties.VERSION
      value: ${nextRelease.version}

—
Reply to this email directly, view it on GitHub
https://github.com/meesvandongen/semantic-release-yaml/issues/5#issuecomment-1225957080,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIOEZFLCYZ6FFOSEMQPEDCTV2ZEPZANCNFSM57PVMIFQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

So, there is indeed a .releaserc.config.js which seems acceptable enough to only support for such a feature.


Fixed the example plugin not loading correctly

Sorry about the last pr, I was uneducated. This one actually fixes something :3

I'll just actually fix the issue, been putting it off for long enough lol

Just fixed it, I'll release now.


The version of chromedriver installed by default together with selenium-cucumber-js is too old and unsupported by modern browsers

Just following your getting start guides and installed everything from a scratch.
The version of chromedriver installed is 76.0.1 however my chrome browser is of higher version (81) and the driver is incompatible with it anymore. Whenever I tried to run the example tests shipped together with the npm they gave me this:
SessionNotCreatedError: session not created: This version of ChromeDriver only supports Chrome version 76
       (Driver info: chromedriver=76.0.3809.126 (d80a294506b4c9d18015e755cee48f953ddc3f2f-refs/branch-heads/3809@{#1024}),platform=Mac OS X 10.11.6 x86_64)
         at Object.checkLegacyResponse (/private/tmp/tst/node_modules/selenium-webdriver/lib/error.js:529:15)
         at parseHttpResponse (/private/tmp/tst/node_modules/selenium-webdriver/lib/http.js:509:13)
         at /private/tmp/tst/node_modules/selenium-webdriver/lib/http.js:441:30
         at processTicksAndRejections (internal/process/task_queues.js:97:5)
     From: Task: WebDriver.createSession()
         at Function.createSession (/private/tmp/tst/node_modules/selenium-webdriver/lib/webdriver.js:769:24)
         at Function.createSession (/private/tmp/tst/node_modules/selenium-webdriver/chrome.js:761:15)
         at createDriver (/private/tmp/tst/node_modules/selenium-webdriver/index.js:170:33)
         at Builder.build (/private/tmp/tst/node_modules/selenium-webdriver/index.js:642:16)
         at new module.exports (/private/tmp/tst/node_modules/selenium-cucumber-js/runtime/chromeDriver.js:20:8)
         at getDriverInstance (/private/tmp/tst/node_modules/selenium-cucumber-js/runtime/world.js:54:22)
         at /private/tmp/tst/node_modules/selenium-cucumber-js/runtime/world.js:207:29

I found no way to download Chrome of version 76 to my mac - looks like Google has already removed all the links for downloading it from public (what a shame). Whenever I ran the npm outdated in the directory node_modules/selenium-cucumber-js it gave me this:
$ npm outdated
Package                 Current  Wanted         Latest  Location
chai                    MISSING   3.5.0          4.2.0  selenium-cucumber-js
chalk                   MISSING   1.1.3          4.0.0  selenium-cucumber-js
commander               MISSING   2.9.0          5.0.0  selenium-cucumber-js
cucumber                MISSING   1.3.3          6.0.5  selenium-cucumber-js
cucumber-html-reporter  MISSING   4.0.4          5.2.0  selenium-cucumber-js
cucumber-junit          MISSING   1.6.0          1.7.1  selenium-cucumber-js
electron                MISSING   1.8.8          8.2.1  selenium-cucumber-js
electron-chromedriver   MISSING   1.8.0          8.0.0  selenium-cucumber-js
electron-packager       MISSING   9.1.0         14.2.1  selenium-cucumber-js
eyes.selenium           MISSING  0.0.72         3.8.11  selenium-cucumber-js
fs-plus                 MISSING   2.9.1          3.1.1  selenium-cucumber-js
geckodriver             MISSING  1.19.1         1.19.1  selenium-cucumber-js
merge                   MISSING   1.2.1          1.2.1  selenium-cucumber-js
phantomjs-prebuilt      MISSING  2.1.12         2.1.16  selenium-cucumber-js
require-dir             MISSING   0.3.2          1.2.0  selenium-cucumber-js
selenium-webdriver      MISSING   3.5.0  4.0.0-alpha.7  selenium-cucumber-js
chromedriver             76.0.1  76.0.1         80.0.2  selenium-cucumber-js

So I decided to ugprade the driver. I modified node_modules/selenium-cucumber-js/package.json:
   "chromedriver": "^80.0.2",
... and ran npm update chromedriver. After upgrade I managed to run your example project to see how it works.

I've got a PR in which includes a change to the value in package.json to the chromedriver's git page, so it always gets the latest Chromedriver.
Until this is approved, you can do this change locally:
"chromedriver": "git+https://github.com/giggio/node-chromedriver#master",

Fixed


Get List of Recently Accessed Files

Objective:
Develop a Python script that retrieves a list of recently accessed files on the local system.
Features:

Identify and list files based on their recent access timestamps.
Provide options for specifying a directory or searching system-wide.
Display information such as file names, paths, and access timestamps.

Example Usage:
# Example usage of the script to get recently accessed files
import recently_accessed_files

# Retrieve list of recently accessed files
recently_accessed_files.get_recently_accessed_files()

Difficulty: Beginner/Intermediate
Tags: Python, File Handling, Scripting
Additional Information:

Consider using the os module for file operations and os.path.getatime() to retrieve access times.
Provide options for setting a time threshold for 'recent' accesses.

Contribution Guidelines
The updated guidelines can be found here.
Note:

folder_name: recently_accessed_files
script_name: recently_accessed_files

Hi @bhargavnova in this issue ... we have to access list  of files from specific directory or system-wide?

@onkar-kota,
Well, that will be efficient when it's systemwide, but that will involve more complex code because systems have lots of files.
let's aim for a directory now, because the whole system is itself a directory.
time to get that list will increase if directory size/I mean file count increases,
so work like aiming for the directory,
if you want I can assign this one to you.

@bhargavnova
Yeah, that's true... the code will be bit more complex.
Yes, assign me I'll try to resolve this issue

cool!
happy coding!

Hey @bhargavnova please check my PR #67


Unable to install packages from Azure's package repository

Describe the bug
Looks like Azure put their python package index behind an authorization mechanisms that redirects you through a login portal in such a way that this package gets confused. Pip seems to handle this scheme correctly.
unearth: Collecting links from https://pypi.org/simple/some-package/
unearth: Found index url https://pypi.org/simple
unearth: Skip https://pypi.org/simple/some-package/ because of Client Error(404): Not Found.
unearth: Collecting links from<EMAIL_ADDRESS>unearth: Found index url<EMAIL_ADDRESS>unearth: Skip link <Link https://spsprodcca1.vssps.visualstudio.com/go/profile (from https://spsprodcca1.vssps.visualstudio.com/_signin?realm=pkgs.dev.azure.com&reply_to=https%3A%2F%2Fpkgs.dev.azure.com%myorg%2Fcore%2F_packaging%2Fcp%2Fpypi%2Fsimple%some-package%2F&redirect=1&hid=e8463d24-ca16-4e07-aeb8-7434d592857c&context=eyJodCI6MiwiaGlkIjoiOThkZWFmYjYtYmYzZC00YTE5LWFjMjQtZDk0YzY5NjhjYzYyIiwicXMiOnt9LCJyciI6IiIsInZoIjoiIiwiY3YiOiIiLCJjcyI6IiJ90)>: Not a file: profile
unearth: Skip link <Link https://spsprodcca1.vssps.visualstudio.com/_signout (from https://spsprodcca1.vssps.visualstudio.com/_signin?realm=pkgs.dev.azure.com&reply_to=https%3A%2F%2Fpkgs.dev.azure.com%myorg%2Fcore%2F_packaging%2Fcp%2Fpypi%2Fsimple%some-package%2F&redirect=1&hid=e8463d24-ca16-4e07-aeb8-7434d592857c&context=eyJodCI6MiwiaGlkIjoiOThkZWFmYjYtYmYzZC00YTE5LWFjMjQtZDk0YzY5NjhjYzYyIiwicXMiOnt9LCJyciI6IiIsInZoIjoiIiwiY3YiOiIiLCJjcyI6IiJ90)>: Not a file: _signout

To Reproduce
Add a source entry like this in pdm:
[[tool.pdm.source]]
url =<EMAIL_ADDRESS>verify_ssl = true
name = "azure"

And then adding a package provided by said repository.
System (please complete the following information):

unearth version: master
Python version: 3.9
OS: macOS

The credentials are passed via Basic Authentication. Does pip support that use case though?

Hi vodik,
I had to use this URL scheme (as listed in the guide):
<EMAIL_ADDRESS>I am not using ssl but I don't think it's the issue. I've been downloading packages from Azure index for months with no issues.

@vodik Can you try on the latest stable version?


update dependencies

Hello folks!
I've noticed that some dependencies used in this project are abandoned, so I've updated them. :)

Could you do this again and remove the emails, please? (because of bots scrapping it...)
Thanks!!


please add the code to project images

add the code to project images please

Hi! I hope the author could release the code for projecting the images, too. Thanks a lot!


[Reimplementation]

Prerequisite

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

💬 Describe the reimplementation questions
Hello, are the original images used for testing?
'rawData/Annotations/test_wo_ignore.json'   or  'divData/Annotations/test.json'
Thanks
Environment
Exactly the same environment as the author。
Expected results
No response
Additional information
No response

In ./mmdet/datasets/sodad.py :
ensure data_infos and dets have the same length
    for data_info, result in zip(self.data_infos, dets):
        # print(data_info['start_coord'])
        x_start, y_start = data_info['start_coord']
        new_result = []
        for i, res in enumerate(result):
            bboxes, scores = res[:, :-1], res[:, [-1]]
            bboxes = self.translate(bboxes, x_start, y_start)
            labels = np.zeros((bboxes.shape[0], 1)) + i
            new_result.append(np.concatenate(
                [labels, bboxes, scores], axis=1
            ))

So,i can only use 'divData/Annotations/test.json'

Prerequisite

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

💬 Describe the reimplementation questions
Hello, are the original images used for testing? 'rawData/Annotations/test_wo_ignore.json' or 'divData/Annotations/test.json' Thanks
Environment
Exactly the same environment as the author。
Expected results
No response
Additional information
No response


Report describe/it inside XCTestCases using XCTest

While reviewing #30, I had some alternative ideas that I wanted to test out. This is a simpler approach that adds a describe and it methods to XCTestCase via an extension, and immediately runs those closures and reports failures directly to XCTest.
This explicitly disallows use of the global context an thus if you try to use the global describe or it during an XCTest invocation a fatalError will be raised.
Example tests and report:

Ideas for future extension
It would be nice to be able to inject the full test hierarchy into XCTest along with the skipped tests.
This will allow proper test reporting for describe and context inside Xcode output. Interacting with XCTest run-time is limited and I did not find a way to do this directly to XCTest.
I have in the past done this via injecting NSInvocations to the test invocation list for a class, which would allow reporting each invocations separately. Unfortunately NSInvocation is not permitted in Swift (In Swift 2 I did find some measures to be able to do this (https://github.com/kylef/JSONSchema.swift/blob/0.2.0/JSONSchemaTests/JSONSchemaCases.swift#L63-L81) but that was broken in Swift 3).
This could be possible by including Objective-C sources, however that is problematic with Xcode projects generated with swift package generate-xcodeproj. You cannot link against the XCTest framework from Objective-C via swift pm (https://github.com/apple/swift-package-manager/pull/1422 would allow this in the future).
Closes #2

I'm getting an issue with this when running in Xcode, in that there is always an alert after running the tests.
Running swift test is fine however.


Implement PersonalAudioClassifier for iOS

This PR extracts the iOS specific parts of #3201. This should be merged at the command line to retain authorship on the underlying commits.

Rebased onto latest ucr to resolve merge conflicts.


Direct Play Amazon Music Album per URL

Is it possible to play an Amazon Music Album directly per URL?
exmaple:
i want to play an Amazon Music Album on trigger without having it in my Library:
https://music.amazon.de/albums/B0916RJW6M?marketplaceId=A1PA6795UKMFR9&musicTerritory=DE&ref=dm_sh_YHJDGDJ33YBqNEI5RIufBokFs
i want to build an RFID Card Reader that plays an Amazon Music Album for my daughter. So she can have some cards to play Audiobooks from Amazon.

no response - closed


Add test for "beforeEach" in "test" behavior

#6006 didn't fix the issue introduced in #5885, discussed in #5964. I'm adding a test (that will fail) until fixed.

cc / @niieani

@mjesun does this mean FB's internal test suite is still broken?

Yes :'( I thought I tested it, but can't see where I commented

Fix in https://github.com/facebook/jest/pull/6100

@niieani Now that the test is in master, could you bring back #5885, #6006 and #6100 altogether, without the legacy flag? Having this would be perfect and then I'm happy to merge it again. Sorry for all the churn I've created, I'm not happy with it either 😔

@mjesun this test should either be fixed or skipped on windows

@SimenB looks like this PR passed on AppVeyor: https://ci.appveyor.com/project/Daniel15/jest/build/7913

That's odd, the merge failed: https://ci.appveyor.com/project/Daniel15/jest/build/7914

@mjesun revert https://github.com/facebook/jest/pull/6105 and merge #6100 on top and it should be golden.

@niieani can you remove the legacy flag on #6100?

It's not a flag though, it's config. Why is it an issue adding that to package.json in FB?

@mjesun Given @SimenB's comment on unintuitive ordering here https://github.com/facebook/jest/pull/6105#issuecomment-385878095 can we confirm that we do not want this odd behavior behind a flag?

If we can figure out someone who will clean this internally, I'm happy having the flag // cc @cpojer


Question about the specific meaning of each category in citeseer.

Hi, can you provide the specific meaning of each category in citeseer (we know that citeseer contains 6 categories, ML, AI, IR, DB, HCI, Agent)?
For example, number '1' in  the citeseer_data.y corresponds to which? ML, AI, IR, DB, HCI, or Agent?

Hi, the name of each category can be accessed through the label_names attribute. After loading the pt file into a, you may access it through a.label_names, which is ['Agents', 'ML', 'IR', 'DB', 'HCI', 'AI']


Fix C# grpc_slice leaks

Fixes https://github.com/grpc/grpc/issues/11434.
The leak was not observable for method, host and status_details of length <=15 (on 64bit), because grpc_slice of such lengths are passed by value.

CC @jskeet

Yay, thanks :) Do we have any automated testing against leaks? (Admittedly it would have needed a longer-than-math-service method to catch this one...)

@jskeet  currently we don't have any. We'd like to have some, but they are tricky to setup in a way that would reveal new leaks. For C core/C++ we are running tests with memory sanitizer though which can reveal some of the leaks.

Known failures:
https://github.com/grpc/grpc/issues/11386


A bug when creating entity_strings.txt for wikidata5m

when I creat entity_strings.txt for wikidata5m, it reports a bug.
python /home/zjj/kgt5/data/get_unique_entities.py --dataset wikidata5m 285780it [00:00, 596976.79it/s] Traceback (most recent call last):   File "/home/zjj/kgt5/data/get_unique_entities.py", line 26, in <module>     unique_entities.add(split_sentence[1].strip()) IndexError: list index out of range
And I output this line in train.txt, it is
predict tail: creation | destruction | instance of |     bonus tracks
So how do I solve this? Just skip this line?

Yeah that's a bug since the entity representation here also contains the symbol '|'. This was fixed in a subsequent dataset revision (we will upload that soon)
For now, you can skip this line I think (other option would be to look for entity 'creation | destruction' and remove the symbol '|' from it, but this is not ideal).


refactor(bridge): Refactoring of service screen (#4918) (#5244)

Fixes #4918
Fixes #5244
Note: fromTime can't be implemented for the service states endpoint, because of hasRemediations for each stage. The previous and current value should be checked then. Always returning all service states is less effort (fewer API calls).
Note2: A fromTime parameter for the service states endpoint can actually be implemented without any additional calls. Just need to check if there are any remediation.* events in service.latestEvents with a timestamp higher than fromTime.
Needs discussion if we should implement this (including an update algorithm for service states, which should already exist in the commit history)

Before the refactoring we had a info "Last processed artifact:"

This was removed during the refactoring:

Was this on purpose?

When a sequence in a stage was still running the stage was highlighted green when selected, thus reading the stage name was quite okay:

Now the background of the stage stays gray, while the label is white, which makes reading the label difficult because of little contrast:

Can we fix this?

While staying on the service screen, after the polling worked (which seems to not update the screen immediately), the newer service is added at the bottom.
Before:

After refactoring:

After refreshing the refactored page:

I would expect to have always the same sorting, but not sure which is more correct, the latest on top or the latest on bottom.

Regarding the latest processed artifact: Is the environment variable CONTINUOUS_DELIVERY set? Else the artifact will not show up because it's set to quality gates only environment.

@ermin-muratovic

order of services is now by name, also on update
fixed selection color of stages that are currently running


[feature] Build and freeze dependent .debs - do not rely on termux repo

We need some way to build and freeze packages for ha.
Probably a custom termux package repo.

I've given this some thought.
We can now search the termux-package repo's git log and pull specific package build definitions by commit / package version. This sort-of works, until a dependency of a dependency expects a different ABI to the one that's being built. Then only at runtime do we get segfaults etc. See ./pkg-old.sh --version <version> <package name>
Exploring the idea of a 'frankenstein' package repo; checking out different commits in each ./package/<pkg> dir, then building via Docker. This should fix the ABI issue, but doesn't obviously mean that any dependency combo will be valid. But that's not the objective.
Tasks

[ ] Extend pkg-old.sh to check out commit/dirs and create the 'frankenrepo'.
[ ] Extend pkg-old.sh to build the repo via docker


refactot Route.all_plugins()

My code accurately reproduces original functional, but is it correct?
In original code unique collects plugin names to avoid repeats. If plugin has no name, this check not. So, noname plugin can repeats. Is it right logic?

@SergBobrovsky you do a really cool job on those PRs. Keep up and take the critics as constructive criticism.

@SergBobrovsky you do a really cool job on those PRs. Keep up and take the critics as constructive criticism.


Clear up the "easily" part of cfw in home.txt

I would not exactly call dsiware and hardmod "easy" ;)
Cleared up some misconceptions where people (almost) updated to 11.5

I see you have rephrased it quite well


Bookmarks button add...

Hi, is it possible to add a bookmark button to the left of the address bar? (as done in Cent Browser and Maxthon)
Thank you.
https://www.upload.ee/image/15704915/2023-9-20_9-45-19.png

I also miss this shortcut button.
360chrome for WinXP has it unlike the chrome 109...


Unexpected result of DecodeFrameAsync

In case of submitting maximum number of surfaces to decoder during flushing (working with preallocated hardware memory), when calling DecodeFrameAsync with NULL bit stream, NULL work surface and valid sync point we get not NULL surface with pointer not matching to any of the pointers of preallocated buffers.

Hi @filippenkos ,
Which decoder? Could you provide a reproducer?

@filippenkos , can you please reply?

Hi @dmitryermilov ,
Sorry for late reply. It happens only with MPEG2 decoder. We couldn't provide a reproducer.

Closing this since it's under offline discussion.


[Tooltip] width should match popup default

Create a popup with a lot of text such as this:
<span data-position='right center' data-tooltip='Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat.'>
  <i class='ui tiny circular info icon inverted blue'>
</span>

I'd expect that the text wraps automatically to use the default width that the popup has, but instead it just goes off the edge of the screen.  I'd like to keep this simple and not have to use JavaScript for a simple component that appears in a lot of places in our app.
Version 2.3.1
This behavior can be seen here:
https://next.plnkr.co/edit/Tdk1IwngNXT6djBC

We added optional fixed class support to gain this behavior by https://github.com/fomantic/Fomantic-UI/pull/1216