Can you add crossorigin to the img tags? [feature]

What problem does this new feature solve?
I am building a website which requires COEP to be set to 'require-corp' since I need to use SharedArrayBuffer.
However, because of this all the images in the web3modal are not loading. Adding 'crossorigin' to the img's would fix this.
Describe the solution you'd like
add the attribiute 'crossorigin' to all the imgs

@dataexcess Can you expand on your issue and what exactly you'd like to be added to the image tag? My understanding is that crossorigin="anonymous is applied to all image tags by default. If you are using strict CSP policies, you will need to allow our image endpoint in your configuration, which should allow them to be loaded?

I have a script that's using a worker to load a wasm file. I need SharedArrayBuffer to be able to make all of this work. As far as I know (maybe I don't understand it fully) in order to enable SharedArrayBuffer the COEP needs to be set to 'require-corp':
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Cross-Origin-Embedder-Policy
The problem is that setting this in my app (nuxt 3) also blocks all other resources - except if they are marked with crossorigin

I see, so I assume other bits like our websocket relay / data fetching via explorer api work fine? I can look into this for you, but in that case, could you give me a concrete example of what needs to be added to the img tag for it to work? (ideally if you could test it on your end as well first).

All the rest is working perfectly! yes.
I just literally added the word 'crossorigin' inside the img tag in google chrome dev and the images start appearing...
Please have a look and it would be awesome if you can make this work :). Somebody else at the nuxt-security github told me to contact the owners of the 3th party code (you) and had this to say:
"Having a crossorigin prop in the image is a good practise in general so I recommend to do that."
So, I believe it can do no harm and is good practice anyway? But I'm no expert.
Thank you!

Awesome, ty for extra details, I will mark this as a task 👍

Awesome, ty for extra details, I will mark this as a task 👍 Meanwhile you can probably write a small script to loop over all images within our web-compoennt w3m-modal and add it, so things work while we add on our end

Haha yes that's what I'm trying to do now but I'm kind of stuck 😂. Is there a callback available that I can use when the modal opens? Since I cannot add the script onpage load since obviously the images are not there yet since the modal is not there.

I'm trying but it's difficult because it is a shadow-element? I cannot use getElementById() it seems.
Anyways I will keep searching

Let me know if there's a specific branch I could test on to help you once you get to it :)


IndexError

Not sure if this was already fixed. (I'm creating a ticket, just not to forget about this.)
Traceback (most recent call last):
  File "/usr/lib/python2.7/threading.py", line 552, in __bootstrap_inner
    self.run()DebtorSubscription
  File "/home/jonathan/git/prompt_toolkit/prompt_toolkit/eventloop/base.py", line 43, in run
    callback()
  File "/home/jonathan/git/prompt_toolkit/prompt_toolkit/__init__.py", line 399, in run
    completions = list(line.completer.get_completions(document))
  File "/home/jonathan/git/prompt_toolkit/prompt_toolkit/contrib/ipython.py", line 61, in get_completions
    for c in super(IPythonCompleter, self).get_completions(document):
  File "/home/jonathan/git/prompt_toolkit/prompt_toolkit/contrib/python_input.py", line 461, in get_completions
    script = get_jedi_script_from_document(document, self.get_locals(), self.get_globals())
  File "/home/jonathan/git/prompt_toolkit/prompt_toolkit/contrib/python_input.py", line 441, in get_jedi_script_from_document
    namespaces=[locals, globals])
  File "/home/jonathan/env/mobilevikings/local/lib/python2.7/site-packages/jedi/api/__init__.py", line 642, in __init__
    interpreter.create(self._evaluator, namespaces[0], self._parser.module())
  File "/home/jonathan/env/mobilevikings/local/lib/python2.7/site-packages/jedi/api/interpreter.py", line 107, in create
    for something in getattr(parser_module, attr_name):
  File "/home/jonathan/env/mobilevikings/local/lib/python2.7/site-packages/jedi/parser/fast.py", line 38, in __getattr__
    return getattr(self.parsers[0].module, name)
IndexError: list index out of range

Closing this. We upgraded to a new Jedi version.


Demo app: Add carousel component

This change adds a carousel component to the demo app, partly for demonstration purposes - but also to enable us to cheaply perform UI tests and manual verification for a component with children (the demo app previously didn’t have any components with children).
The “Pretty pictures” feature of the demo app is also updated to display a carousel on the top, as well as adding tappable links to the pictures that it displays.

@spotify/objc-dev

Current coverage is 93.15% (diff: 100%)

Merging #101 into master will increase coverage by 0.32%

@@             master       #101   diff @@
==========================================
  Files            57         58     +1
  Lines          4027       4294   +267
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3738       4000   +262
- Misses          289        294     +5
  Partials          0          0

Powered by Codecov. Last update 9b01f01...40bac77


Multiple navbars steps on each other

(I will elaborate later)

I have almost forgotten what this was about, and we have stopped using react-materialize. Leaving this here in case I want to take a stab at it at some later time.


Enhance/migrate view all units component

Description
Component review document and the progress on the task so far.

@AusBruce will review this PR.

Brian and I worked on this assignment for a while, putting in a lot of effort and time to study and conduct research. We have a few meetings to talk about the progress we have made so far. In my opinion, Brian is a highly cooperative team member who provides me with a wealth of tech expertise and insightful references.

Brian and I worked on this assignment for a while, putting in a lot of effort and time to study and conduct research. We have a few meetings to talk about the progress we have made so far. In my opinion, Brian is a highly cooperative team member who provides me with a wealth of tech expertise and insightful references.

@BrianJCal99

Thanks for the work guys,
The one thing to add to this is that the View All Units new version already exists in that other section I showed you, so there is no need to redesign this particular component. This one just needs migrating as per how the other view all units was already done. Its an interesting read though.

This pull request encompasses the migration and enhancement of the f-all-units-list component. The changes aim to migrate the component, improve performance, and align with updated coding standards.
For the review:
1.The refactoring from v1 to v2 shows a clear improvement in code structure and readability. The use of Angular Material in v2 enhances the UI's consistency and modern feel.
2.The added sorting and filtering functionalities in v2 are a significant improvement.
3.There are mentions of terminal errors in the current state of the component. Could you provide more details on these errors? It’s important to resolve these before merging the pull request.(it merged now!)
Here are some recommendations I would make:

Resolve the mentioned terminal errors if it is possible.
Add or update tests for the new functionalities.
Provide additional documentation or comments where necessary.
Overall，
This pull request represents a significant step forward in the application’s UI component development. The enhancements in v2 are promising, but attention to the mentioned aspects is necessary to ensure a smooth transition and integration. I recommend addressing these concerns and providing the additional information requested before proceeding with the merge.

Looking forward to the revised version and the continued improvement of our application's user  interface.
@BrianJCal99


Improve getting started guide, add note about compatible arrow versions

Which issue does this PR close?
N/A
Rationale for this change
At least two people reported challenges getting started with DataFusion related to mismatched arrow versions: https://discord.com/channels/885562378132000778/885562378132000781/1112039133687324742
I would like to reduce this point of friction if possible
What changes are included in this PR?

Add note/ hint for mismatched arrow versions
Update landing page of github repo and docs site to make the getting started information easier to find

Are these changes tested?

Are there any user-facing changes?

Thanks again @TimKotowski  for the help


[5.1][Reflection] Switch Remote Mirror back to uintptr_t for pointers on watchOS.

Cherry-pick https://github.com/apple/swift/pull/24529 to 5.1.
Existing Remote Mirror dylibs are compiled this way, and changing it breaks binary compatibility. We want to use uint64_t everywhere else, since the target's pointer size may not match ours.
rdar://problem/50279443

@swift-ci please test

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - bbc0396c6be643906bfb1297626f75b79590fdb8

Build failed
Swift Test OS X Platform
Git Sha - bbc0396c6be643906bfb1297626f75b79590fdb8


[1.12] Replace artisanal docer version detect script

Backports: https://github.com/dcos/dcos/pull/5993

High-level description
Fix problems with docker version > 1.19 which added new entries to docker version output.
Latest docker hase more entries in docker version output. See folowing examples
Client: Docker Engine - Community
 Version:           19.03.1
 API version:       1.35 (downgraded from 1.40)
 Go version:        go1.12.5
 Git commit:        74b1e89
 Built:             Thu Jul 25 21:21:07 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server:
 Engine:
  Version:          17.12.1-ce
  API version:      1.35 (minimum version 1.12)
  Go version:       go1.9.4
  Git commit:       7390fc6
  Built:            Tue Feb 27 22:20:43 2018
  OS/Arch:          linux/amd64
  Experimental:     false

Client: Docker Engine - Community
 Version:           19.03.1
 API version:       1.40
 Go version:        go1.12.5
 Git commit:        74b1e89
 Built:             Thu Jul 25 21:21:07 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.1
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.5
  Git commit:       74b1e89
  Built:            Thu Jul 25 21:19:36 2019
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.2.6
  GitCommit:        894b81a4b802e4eb2a91d1ce216b8817763c29fb
 runc:
  Version:          1.0.0-rc8
  GitCommit:        425e105d5a03fabd737a126ad93d62a9eeede87f
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

The previous script instead of Engine: Version: took docker-init: Version: and then took lowest version from client and server. This PR removes AWK script and replace it with sort -V which can correctly sort versions and docker version -f which allow us to use go template to get version in desired output`. Here we are interested only in client and server versions and choosing the lowest.
Corresponding DC/OS tickets (required)

DCOS-56831
DCOS-57154

Related tickets (optional)

DCOS-ID JIRA title / short description.

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot request-review @drewvanstone @br-lewis @makkes


This is my web server packaged based on httpapi.dll, Vanara.PInvoke.HttpApi 3.4.9 version works fine now.

https://github.com/williamlzw/HttpApiServer.Net

Thank you for the link. I'm glad the library has helped and is working.


ArrayIndexOutOfBoundsException when run shell task with output param

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
The version of dolphinscheduler is 3.1.7.
The shell script is:
http_code=`curl -s -o result -w "%{http_code}" -X GET  'http://url of our service'`
if [[ $http_code -eq 200 ]]; then
    echo "task execute success."
else
    echo "task execute failed."
    cat result
    exit 1
fi
param=`cat result`
echo $param
echo  '${setValue(paramOut='${param}')}'

"paramOut" is set as a task param with OUT direction.
The task is scheduled to run every minute and all was work well when the task is settled, but ArrayIndexOutOfBoundsException is throw after about ten hours  and it's never recoverd.
And there is another problem: most of the time  the param passing is wroking well, the next task node can obtain the correct param passed by the first shell task, but some time the next task obtain the param as blank  and there is no error log. This situation happend occasionally.
The task log:
[INFO] 2024-08-28 02:23:02.470 +0000 - task run command: sudo -u dolphinscheduler -E bash /data1/dolphinscheduler-data/exec/process/dolphinscheduler/11465962145888/14603302834144_23/281095/325874/281095_325874.command
[INFO] 2024-08-28 02:23:02.471 +0000 - process start, process id is: 7742
[INFO] 2024-08-28 02:23:02.528 +0000 - process has exited. execute path:/data1/dolphinscheduler-data/exec/process/dolphinscheduler/11465962145888/14603302834144_23/281095/325874, processId:7742 ,exitStatusCode:0 ,processWaitForStatus:true ,processExitValue:0
[ERROR] 2024-08-28 02:23:02.530 +0000 - shell task error
java.lang.ArrayIndexOutOfBoundsException: 1
	at org.apache.dolphinscheduler.plugin.task.api.parameters.AbstractParameters.getMapByString(AbstractParameters.java:191)
	at org.apache.dolphinscheduler.plugin.task.api.parameters.AbstractParameters.dealOutParam(AbstractParameters.java:144)
	at org.apache.dolphinscheduler.plugin.task.shell.ShellTask.handle(ShellTask.java:99)
	at org.apache.dolphinscheduler.server.worker.runner.DefaultWorkerDelayTaskExecuteRunnable.executeTask(DefaultWorkerDelayTaskExecuteRunnable.java:49)
	at org.apache.dolphinscheduler.server.worker.runner.WorkerTaskExecuteRunnable.run(WorkerTaskExecuteRunnable.java:174)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at com.google.common.util.concurrent.TrustedListenableFutureTask$TrustedFutureInterruptibleTask.runInterruptibly(TrustedListenableFutureTask.java:131)
	at com.google.common.util.concurrent.InterruptibleTask.run(InterruptibleTask.java:74)
	at com.google.common.util.concurrent.TrustedListenableFutureTask.run(TrustedListenableFutureTask.java:82)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:748)
[ERROR] 2024-08-28 02:23:02.531 +0000 - Task execute failed, due to meet an exception
org.apache.dolphinscheduler.plugin.task.api.TaskException: Execute shell task error
	at org.apache.dolphinscheduler.plugin.task.shell.ShellTask.handle(ShellTask.java:108)
	at org.apache.dolphinscheduler.server.worker.runner.DefaultWorkerDelayTaskExecuteRunnable.executeTask(DefaultWorkerDelayTaskExecuteRunnable.java:49)
	at org.apache.dolphinscheduler.server.worker.runner.WorkerTaskExecuteRunnable.run(WorkerTaskExecuteRunnable.java:174)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at com.google.common.util.concurrent.TrustedListenableFutureTask$TrustedFutureInterruptibleTask.runInterruptibly(TrustedListenableFutureTask.java:131)
	at com.google.common.util.concurrent.InterruptibleTask.run(InterruptibleTask.java:74)
	at com.google.common.util.concurrent.TrustedListenableFutureTask.run(TrustedListenableFutureTask.java:82)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.ArrayIndexOutOfBoundsException: 1
	at org.apache.dolphinscheduler.plugin.task.api.parameters.AbstractParameters.getMapByString(AbstractParameters.java:191)
	at org.apache.dolphinscheduler.plugin.task.api.parameters.AbstractParameters.dealOutParam(AbstractParameters.java:144)
	at org.apache.dolphinscheduler.plugin.task.shell.ShellTask.handle(ShellTask.java:99)
	... 9 common frames omitted
[INFO] 2024-08-28 02:23:02.531 +0000 -  -> task execute success.
	cat: result: No such file or directory

[INFO] 2024-08-28 02:23:02.531 +0000 - FINALIZE_SESSION
[INFO] 2024-08-28 02:23:02.531 +0000 - cancel process: 7742
[INFO] 2024-08-28 02:23:02.531 +0000 - Get a exception when execute the task, will send the task execute result to master, the current task execute result is TaskExecutionStatus{code=6, desc='failure'}
[INFO] 2024-08-28 02:23:03.471 +0000 - FINALIZE_SESSION

What you expected to happen
param passing working well
How to reproduce
as described above
Anything else
No response
Version
3.1.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

The ArrayIndexOutOfBoundsException seems is caused by the result of the curl result is blank .
But the second problem is still there:
The next task obtain the param as blank even the param passed by the first shell task is not blank and there is no error log. This situation happend occasionally.


fix: respect interactions select state

There are two interactions state, active and select, which should turn off selections. This PR fixes so that select now also works.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


12/6/2022 Publish

Learn Build status updates of commit 5fb37d0:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

2017-03-09-profile/docs-ref-autogen/account/management-group.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/account/management-group/hierarchy-settings.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/ad/app.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/ad/sp.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/arcappliance/createconfig.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/arcdata/dc/config.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/automation/hrwg.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/automation/python3-package.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/baremetalinstance.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/blueprint.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/config.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/connectedmachine/private-link-scope.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/connectedvmware/vm.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/containerapp.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/containerapp/github-action.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/containerapp/revision.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/csvmware/vm.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/datadog/sso-config.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/datadog/tag-rule.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/datafactory/dataset.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/datafactory/linked-service.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/datafactory/managed-private-endpoint.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/datafactory/managed-virtual-network.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/datafactory/pipeline.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/datafactory/trigger.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

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

2017-03-09-profile/docs-ref-autogen/account/management-group.yml

Line 167, Column 93: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'indexToRemove' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 171, Column 116: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'value' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

2017-03-09-profile/docs-ref-autogen/account/management-group/hierarchy-settings.yml

Line 112, Column 93: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'indexToRemove' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 119, Column 116: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'value' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

2017-03-09-profile/docs-ref-autogen/ad/app.yml

Line 115, Column 319: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'application-client-id' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 347, Column 319: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'application-client-id' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 377, Column 93: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'indexToRemove' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 384, Column 116: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'value' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

2017-03-09-profile/docs-ref-autogen/ad/sp.yml

Line 173, Column 93: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'indexToRemove' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 177, Column 116: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'value' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

2017-03-09-profile/docs-ref-autogen/arcappliance/createconfig.yml

Line 75, Column 42: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'URL' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 75, Column 48: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'PORT' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 78, Column 42: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'URL' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 78, Column 48: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'PORT' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 235, Column 42: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'URL' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 235, Column 48: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'PORT' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 238, Column 42: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'URL' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 238, Column 48: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'PORT' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 366, Column 42: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'URL' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 366, Column 48: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'PORT' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 369, Column 42: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'URL' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 369, Column 48: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'PORT' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

2017-03-09-profile/docs-ref-autogen/arcdata/dc/config.yml

Line 54, Column 88: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'cwd' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 60, Column 88: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'cwd' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

2017-03-09-profile/docs-ref-autogen/automation/hrwg.yml

Line 166, Column 93: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'indexToRemove' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Minor: Make IOTypeHints a real NamedTuple

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Out of curiosity, what prompted this change?

It's just a silly cleanup that I noticed while working on some adjacent code.

Run Python_PVR_Flink PreCommit

PythonDocs issue is BEAM-14172


Memory usage monitoring improvements

This mimics the memory usage from Oink ( https://github.com/noahd1/oink/blob/master/lib/oink/rails/memory_usage_logger.rb ) which improves things in a few ways:

monitor vsz instead of rss: https://github.com/noahd1/oink/issues/closed#issue/1
faster: https://github.com/noahd1/oink/issues/closed#issue/3
win32 support

If you isolate your pull requests out to the pertinent commits, instead of grouping them all in each pull request (use a branch based off of the main repo master, so each branch only has the needed change when you make the PR), and I'll look over them and start pulling them in.
Thank you for the effort, and I will try to be responsive about merging in after this.


[Docs] Added Meshery CLI to the list of architectural components

Adding Meshery CLI to the list of architectural components.
Notes for Reviewers
This PR fixes # 9623
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit c9eeeae38432d1bbaf5314ee1543f08ff0a22175 at: https://657a0d4ceed5550396703f48--meshery-docs-preview.netlify.app

🚀 Preview for commit 11f8b6f78841732bb92d8a5e814ce46ae0fc4a50 at: https://657abdcbc61ad97520b957df--meshery-docs-preview.netlify.app

🚀 Preview for commit b6c5e0c7d59d7768ec2a0168f1c031d000cf1239 at: https://657bfd0e256e874375be3023--meshery-docs-preview.netlify.app

🚀 Preview for commit 8d4a9a0170856b66556e792550d3fcfe648e7997 at: https://657bfe3192e1ef4209686b73--meshery-docs-preview.netlify.app

🚀 Preview for commit 43b16da0964776516aef4354afa2e79da17487d5 at: https://65818768254f040088b86c8e--meshery-docs-preview.netlify.app


Implement further UserPage testing

Add further tests to the UserPage.

[x] Check that the user can edit their own profile information (and it remains)
[x]  Check that the user can actually see the club data of clubs they can edit
[x]  (after adding a leave club button to the ClubCard) Check that the user can leave a club

Additionally, all UserPage tests should be invariant.

After lots of blood, sweat, and tears we got the tests working. For some reason, the tests were passing locally, but failing on GitHub Actions:

I narrowed the problem down, seeing that  after a certain point, the test started looping from the start over and over again:

I suspected some sort of memory error? Perhaps the test was too large. I chipped away at the UserPage tests, to test as much as I could without stalling out the TestCafe test.


Make cross join overridable

Fixes #1932

An added test that overrides crossJoin?
Just to see if it is inserted in the right place.
But looks good to me. :+1:


Refactors the Trainer API and adds function for cleaner train ops

Optimizers no longer have to be named as optimizer_generator, their name can be pased as an optional argument.
Same for schedulers.
Train Ops naming constraints have been relaxed.

@Aniket1998 This has been tested and is good to go from my side. Let me know if any modification is needed.

Codecov Report

Merging #62 into master will decrease coverage by 20.05%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #62       +/-   ##
=========================================
- Coverage   20.05%     0%   -20.06%
=========================================
  Files          37     32        -5
  Lines        1516   1201      -315
=========================================
- Hits          304      0      -304
+ Misses       1212   1201       -11

Impacted Files
Coverage Δ

torchgan/losses/loss.py
0% <0%> (ø)
:arrow_up:

torchgan/metrics/classifierscore.py
0% <0%> (ø)
:arrow_up:

torchgan/losses/auxclassifier.py
0% <0%> (ø)
:arrow_up:

torchgan/trainer/trainer.py
0% <0%> (ø)
:arrow_up:

torchgan/losses/boundaryequilibrium.py
0% <0%> (ø)
:arrow_up:

test/test-models.py

setup.py

test/test-metrics.py

test/test-losses.py

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 39d31f5...d98cb9c. Read the comment docs.

Apart from the two very minor changes, this looks great!


Alignment of destination section!!!!

Describe the bug
So ,on the destination section  heading and its describing paragraph aligned in left.
Expected behavior
But this destination section should aligned in the center.
Screenshots
AFTER

BEFORE

JWOC,2024.

Name : vishal malyan
I have read the issue and go through the code and i have some idea to solve the problem.
I eagerly want to work on it .
please assign it to me under jwoc'2024...

Name : vishal malyan
I have read the issue and go through the code , I have an idea to solve the problem.
I eagerly want to work on it .
please assign it to me under jwoc'2024...

@Lifewithmuskan Already someone is assigned with the same issue #70 .
Sorry to close this issue

Name : vishal malyan I have read the issue and go through the code , I have an idea to solve the problem. I eagerly want to work on it . please assign it to me under jwoc'2024...

This issue is already present. You can make any other new issue on which you want to work on


phpize not in default debian repository, nor in the dependency list

see title...
please add a link to were I can find it

sudo aptitude install phpize
Couldn't find any package whose name or description matched "phpize"
Unable to apply some actions, aborting

phpize && ./configure && make
zsh: command not found: phpize

found fix
http://stackoverflow.com/questions/3108937/ddg#8982179
please add this link or maybe the provided commands to the readme


307 error downloading Files as non-owner

As a non-owner, I am trying to back up the group https://groups.yahoo.com/neo/groups/solzhenitsyn-l/
I backed up the messages okay, but not the files.
I can see that the group has files when logged into Yahoo as a member, but I get:
$ python ./yahoo.py -v -f -ct T-cookie.txt -cy Y-cookie.txt solzhenitsyn-l
2019-11-07 11:58:14.043 EST INFO requests.packages.urllib3.connectionpool Starting new HTTPS connection (1): groups.yahoo.com
2019-11-07 11:58:14.119 EST DEBUG requests.packages.urllib3.connectionpool "GET /api/v2/groups/solzhenitsyn-l/files HTTP/1.1" 307 None
2019-11-07 11:58:14.120 EST ERROR archive_files Couldn't access Files functionality for this group

I have logged out of Yahoo, logged back in and updated the T and Y cookie files, but get the same problem. Please let me know how I can help debug this. Thanks!

@mdecerbo The group solzhenitsyn-l returns an Error 404 Not Found for me. Is it a secret group?
-ct and -cy don't take files — they take cookie data directly. You want to pass
-ct 'z=longstringhere' -cy 'v=longstringhere'.

I didn't think the group was secret! Here's the link: https://groups.yahoo.com/neo/groups/solzhenitsyn-l/info
But thanks very much for clearing up my confusion about how to pass the cookies! So of course:
$ python ./yahoo.py -w -v -f -ct `cat T-cookie.txt` -cy `cat Y-cookie.txt` solzhenitsyn-l

worked just fine.


TEditor Issue with IOS

I'm using TEditor for cross platform app, its working fine for android but In IOS when i try to write using Rich Text Editor(TEditor) that time virtual keyboard is not showing.

I cannot reproduce this, did you toggle software keyboard in your simulator?

Simply virtual keyboard showing in Android but it's not showing in IOS, Should i have to follow some other code or what i should do for this so that it will work in IOS. Please guide me if you have any idea.

Hi @AcusisKumar , you shouldn't do anything. It should work on iOS, did you try on a device?

Hi @jessejiang0214  , I'm using your TEditor for cross-platform , i also be error like @AcusisKumar . I Try use it on emulator but i can use it . Do you try it again on iOS ?


Clarify [EventHandler] tag helper discovery logic and avoid exception

This fixes an issue that @tmat pointed out to me over email. In a recent change (#10720), I added a call to Assumed.Unreachable() when [EventHandler] tag helper discovery encounters an attribute with invalid constructor attributes. However, throwing an exception during tag helper discovery is usually the wrong approach. Normally, if Roslyn symbols aren't in the proper shape during tag helper discovery, Razor will simply not produce a tag helper. (We do support diagnostics for tag helpers, but those are usually reserved for warnings and errors that are related to a tag helper's data that would make it unusable, such as a name containing whitespace.)
It turns out that "unreachable" condition wasn't actually all that unreachable and @tmat was hitting it while working on hot reload tests. So, I've changed the code to make the success conditions clearer, i.e., the attribute data must match one of the two constructor calls. And, I changed the logic to simply skip [EventHandler] attributes that don't meet the success conditions.

Thanks everyone!


The alignment about entity2desc

Hello, thank you for the corpus! I'm following your work and find some problem aligning the golden entity in dataset to entity2desc file. To be specific, with 6312 entries and 13205 mentions in test.json, there are 672 mentions which can't find corresponding description in entity2desc file using the last section of its URL. How can I get those descriptions to reproduce your work?
Apart from this, there are 931 mentions with a "NIL" url description. I'm wondering how do you handle these mentions.
Thanks!

Hello, there're another few questions regarding the data alignment.
When improving the code above to filter the mention title, there're 664 OOV entities. Though the number of OOV entities has dropped, the statistic still can't match the 623 that you reported. Do you have any further processing of the test.json file or entity2desc file?
Secondly, noticing that you haven't published the mentions visual data, only their urls. Using the same approach to get image urls from entity2imgs file, there're 1477 OOV entities in the train+test+valid dataset. Are these entities also not considered during the training and inference? Additionally, can you release the entities' images in same way of the mentions? Cauze reproducing your work need all the data. Thanks!

Hi, the following is my reply:

I use the same data in the cloud drive to find out the 623 OOV entities.
And the mentions' images can be found here:

For entities that cannot find the images, they are assigned to an all-zero image tensor during training/inference.
Sorry, we are working on publishing the entities' image info. But so far it needs 3-5T space for storage. So I am seeing if there are some alternative solutions.

Looking forward to your entity image collection!

Hi, I noticed in train_w_10cands.json, many retrieved entities are also not appear in entity2desc file, which are OOV and NIL. I'm wondering how do you retrieve those entities given that "they are not considered during the training and inference"?


Feat/added skip for empty

I refactor also the binds in this PR, for now, we have noId when is not passed. I have to change noId to empty string.
close #62

Never mind, I still had in mind the old version of that code that didn't use the decorators. We just need to update the README to comunicate that now a routeId is required in order to track metrics for that route.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Verify and iron out base install and update

Some users have reported that either the initial install, or when updating the plugin, isn't particularly smooth.
We need to make sure that this setup and update flow is iron-clad.
And maybe switch to the agent pre-built binaries while we're at it.

This should be all set now -- have added a few more checks (and fixed for windows I think!)


CocoaPods near-term followup

Working from the list at https://github.com/zulip/zulip-mobile/pull/3987#issuecomment-608628240 and following.
This does not include:

moving the Start Metro script (opened as https://github.com/zulip/zulip-mobile/issues/4027)
finding out exactly what's going on (and what should be going on) with libz — our latest knowledge is at https://github.com/zulip/zulip-mobile/issues/4026#issuecomment-610696643, but I think action can wait till we're on RN v0.60.
unimodules (that's https://github.com/zulip/zulip-mobile/pull/4016)

Of course the main thing I'm hoping to learn is the many ways I'm sure the postinstall script could be improved. 🙂

(Marking as P1 because CocoaPods was marked P1, pinging @gnprice)

@andersk Do you know of another good resource to point to for the key points of writing shell scripts? I linked to two below but I feel like there are a few other good things to mention that they don't cover.

Yes! Run ShellCheck on all of your shell scripts and get them down to zero diagnostics. Run it every time you commit. Run it in CI. Run it automatically in your editor. ShellCheck knows things that you didn’t know you needed to know. ShellCheck knows things that I didn’t know I needed to know. ShellCheck is excellent. ShellCheck is magical. ShellCheck is the one shining beacon of hope in the vast fiery hellscape that is the world of UNIX shell scripting. Run ShellCheck.
Run it.

Nice! That led me to the ShellCheck "gallery of bad code", via https://github.com/zulip/zulip/blob/7ff9b2250047fc83ad3600f4b39521409d7e0eda/docs/testing/linters.md#shell-scripts

OK, I've pushed the latest revision. I also noticed that we were using four-space indents in our shell scripts, so I adapted to that for the postinstall script.

Thanks for the review! I just pushed these tweaks, including one more: I was having the postinstall script print that it was running, so people would know where the output that followed was coming from. But yarn does that for us, so, no need.

And merged -- thanks again!


System unavailable: Xvfb seems nonresponsive on test-osuosl-ubuntu1804-ppc64le-2

Please put the system name in the title of this issue.

Link to any log file showing the problem: https://ci.adoptium.net/job/Test_openjdk22_hs_extended.system_ppc64le_linux/3/

Please describe the issue:

Exception: java.lang.IllegalStateException: No display name received from Xvfb within 30 seconds

This has been happening for while on this platform, though I cannot prove that it was definitely this machine.

Seen twice more:
test-osuosl-ubuntu2004-ppc64le-1: https://ci.adoptium.net/job/Test_openjdk11_hs_sanity.system_ppc64le_linux/826/console
test-docker-ubuntu1804-ppc64le-1: https://ci.adoptium.net/job/Test_openjdk11_hs_extended.openjdk_ppc64le_linux_testList_0/118/console

Likely a duplicate of https://github.com/adoptium/aqa-tests/issues/4930

A good number of these issues are on a single dockerhost, dockerhost-osuosl-ubuntu2004-ppc64le-1  , this host has 3 containers test-docker-fedora39-ppc64le-1 , test-docker-ubuntu2004-ppc64le-1 & test-docker-ubuntu2204-ppc64le-3

I'll test by stopping a single container ( test-docker-ubuntu2004-ppc64le-1 ), to see if performance is improved.
Rerun in grinder on test-docker-ubuntu2204-ppc64le-3 with above container & node stopped https://ci.adoptium.net/job/Grinder/10788/

I've identified and corrected a permissions issue on test-docker-ubuntu2204-ppc64le-3 ( /tmp/.X11-unix was owned by Jenkins and it must be owned by root )..  new grinder run is here...  #3171 which seems to be proceeding correctly. Once this run is finished, I'll test it on the other docker containers.

Running same tests on test-docker-ubuntu2004-ppc64le-1 here : https://ci.adoptium.net/job/Grinder/10792/

And passed on the fedora container here. Im going to close this, please let me know if this recurrs and what machine, as I believe the underlying issue has been resolved.


support script type="text/html"

fix https://github.com/microsoft/vscode/issues/77977

Thanks @aliasliao !


ToInt method implementation

Hello.
I saw a  ToInt() method in StringExtensions and it has raised me a question:

Why int.Parse() is being used instead of int.TryParse()? This way, if a string is not a valid number, we wouldn't get an exception.
Also, I think we should handle the cases where a string is a number that is out of bounds (int.MaxValue and int.MinValue) and throw exceptions or something else.

@GintasS
Yes, sure int.TryParse() is better than int.parse() then no need also for your 2nd check.

Sometimes an exception is what you want.
If the value is out of range of an int, an exception will be thrown by int.Parse already.
To my mind there are a few considerations here:

Is a non-numeric value (or out of range) an exceptional circumstance?
If not, what should be returned?

Zero?
null?
False?

If we return int and the use case is simple. If it works, you get an int. If it doesn't, you get an exception.
If we don't throw, then we have more decisions to make.
If it works, you get an int.
If it's empty or null, you get 0. (Should you, though?)
If it doesn't work, you get null (or false; or zero).
You might end up having to return a Maybe<int> or a Result<int> or at least an int? in the non-exception case.
Personally, I rarely use TryParse and if I call int.Parse it's because I'm confident the string should be an int. If it's not, that's exceptional. But I'm open to supporting other use cases, either on the same extension or more likely on a separate one.
Example:
int result = "123".ToInt();
int? result2 = "234.34".ToMaybeInt(); // returns null if it cannot parse to int

@ardalis
ToMaybeInt added on PR #7

Sometimes an exception is what you want.
If the value is out of range of an int, an exception will be thrown by int.Parse already.
To my mind there are a few considerations here:

Is a non-numeric value (or out of range) an exceptional circumstance?

If not, what should be returned?

Zero?
null?
False?

If we return int and the use case is simple. If it works, you get an int. If it doesn't, you get an exception.
If we don't throw, then we have more decisions to make.
If it works, you get an int.
If it's empty or null, you get 0. (Should you, though?)
If it doesn't work, you get null (or false; or zero).
You might end up having to return a Maybe<int> or a Result<int> or at least an int? in the non-exception case.
Personally, I rarely use TryParse and if I call int.Parse it's because I'm confident the string should be an int. If it's not, that's exceptional. But I'm open to supporting other use cases, either on the same extension or more likely on a separate one.
Example:
int result = "123".ToInt();
int? result2 = "234.34".ToMaybeInt(); // returns null if it cannot parse to int

Your remarks are to the point! But I was writing this in regards of having explicit exceptions, and having all the validation/ exception logic in the extension method itself, with the help of the XML comment that would explain all possible exceptions.
But I guess int.Parse is also a valid choice.


GitHub Application Status Update Failure

Summary
During the EasyCLA application check status which is called upon a PR open, reopen, update callback, the EasyCLA backend encounters an error which it doesn't recover from. This error stops the processing of the EasyCLA status check.
Background
Upcon reviewing the backend python logs, the following error was observed:
[DEBUG] 2019-09-14T00:54:24.108Z c5f7676c-a1b1-4708-8c67-62b28d9d1243 updating github pull request for repo: 27729926, pr: PullRequest(title="Adding s390x support to grpc-java", number=6152) with signed authors: [] with missing authors: [('61e32c618940853d552ff62d041fc8fc5f755fe7', GitCommit(sha=None))]
00:54:24
2019-09-14 00:54:24,429 INFO cla: Getting comment body for repository type: github
00:54:24
[INFO] 2019-09-14T00:54:24.429Z c5f7676c-a1b1-4708-8c67-62b28d9d1243 Getting comment body for repository type: github
00:54:24
2019-09-14 00:54:24,429 INFO cla: <IP_ADDRESS> - - [2019-09-14 00:54:24.429423] POST /v2/github/activity 200 OK - GitHub-Hookshot/5a04bc9
00:54:24
[INFO] 2019-09-14T00:54:24.429Z c5f7676c-a1b1-4708-8c67-62b28d9d1243 <IP_ADDRESS> - - [2019-09-14 00:54:24.429423] POST /v2/github/activity 200 OK - GitHub-Hookshot/5a04bc9
00:54:24
unhashable type: 'GitCommit': TypeError Traceback (most recent call last): File "/var/task/wsgi_handler.py", line 108, in handler return serverless_wsgi.handle_request(wsgi_app, event, context) File "/var/task/serverless_wsgi.py", line 173, in handle_request response = Response.from_app(app, environ) File "/var/task/werkzeug/wrappers/base_response.py", line 287, in from_app retur
unhashable type: 'GitCommit': TypeError
Traceback (most recent call last):
File "/var/task/wsgi_handler.py", line 108, in handler
return serverless_wsgi.handle_request(wsgi_app, event, context)
File "/var/task/serverless_wsgi.py", line 173, in handle_request
response = Response.from_app(app, environ)
File "/var/task/werkzeug/wrappers/base_response.py", line 287, in from_app
return cls(*_run_wsgi_app(app, environ, buffered))
File "/var/task/werkzeug/test.py", line 1119, in run_wsgi_app
app_rv = app(environ, start_response)
File "/var/task/hug/api.py", line 497, in api_auto_instantiate
return module.__hug_wsgi__(*args, **kwargs)
File "falcon/api.py", line 274, in falcon.api.API.__call__
raise
File "falcon/api.py", line 269, in falcon.api.API.__call__
responder(req, resp, **params)
File "/var/task/hug/api.py", line 356, in version_router
request, response, api_version=api_version, **kwargs
File "/var/task/hug/interface.py", line 930, in __call__
raise exception
File "/var/task/hug/interface.py", line 901, in __call__
self.call_function(input_parameters), context, request, response, **kwargs
File "/var/task/hug/interface.py", line 823, in call_function
return self.interface(**parameters)
File "/var/task/hug/interface.py", line 123, in __call__
return __hug_internal_self._function(*args, **kwargs)
File "/var/task/cla/routes.py", line 1485, in github_app_activity
return cla.controllers.github.activity(body)
File "/var/task/cla/controllers/github.py", line 192, in activity
result = service.received_activity(body)
File "/var/task/cla/models/github_models.py", line 71, in received_activity
return self.process_reopened_pull_request(data)
File "/var/task/cla/models/github_models.py", line 508, in process_reopened_pull_request
return self.process_opened_pull_request(data)
File "/var/task/cla/models/github_models.py", line 260, in process_opened_pull_request
self.update_change_request(installation_id, github_repository_id, pull_request_id)
File "/var/task/cla/models/github_models.py", line 361, in update_change_request
missing=missing)
File "/var/task/cla/models/github_models.py", line 708, in update_pull_request
signed, missing)
File "/var/task/cla/utils.py", line 689, in assemble_cla_comment
comment = get_comment_body(repository_type, sign_url, signed, missing)
File "/var/task/cla/utils.py", line 738, in get_comment_body
if author not in committers:
TypeError: unhashable type: 'GitCommit'

Expected behavior
The EasyCLA backend would place the PR commit authors into a pass or fail bucket and update the GitHub status.
Screenshots
GitHub application failure on the backend causes the EasyCLA application on the GitHub page to disappear:

Environment
Please complete the following information:

Environment: PROD
Browser: Chrome
Version: EasyCLA v0.1.23

Acceptance Criteria
The "done" criteria when this feature or problem is resolved.

Add defensive checks for the exception
Test case developed, implemented, confirmed
Verified fix in STAGING environment

*References
PR: https://github.com/grpc/grpc-java/pull/6152
Log Entry: https://console.aws.amazon.com/cloudwatch/home?region=us-east-1#logEventViewer:group=/aws/lambda/cla-backend-prod-githubactivity;stream=2019/09/13/[$LATEST]17382ce85fd547a7bf0321ed7eebb1cb;refid=34976989735724049610440285724874976476274677318113427505;reftime=1568422464108

@dealako, @manikantanr how are we able to validate that this has resolved the issue? I don't have my head totally wrapped around what the root cause is.

We potentially have another PR with this issue on ASWF: OpenCue: https://github.com/AcademySoftwareFoundation/OpenCue/pull/456


staking: add missing effect for Defined <> Inactive

There is a bug in the current v0.65.0 release. Defined validators are not transitioning out of the precursor state into Inactive. The reason is that we are not actually effecting the validator state. This had been previously fixed in a follow-up #2921, but somehow slipped back in.

Great find, unfortunate bug. Since this would change the execution of previous blocks, I'm not sure we can easily merge this without causing newly syncing nodes to diverge. I suspect the best course of action here (cc @conorsch) is:

Merge, rewrite the git tag, nuke the existing network and redeploy (it's only a day old)
Discuss in a retro of how we could (a) mitigate root causes of the bug and (b) how we could make mitigation less painful.

I think this will be a useful learning experience.

Merge, rewrite the git tag, nuke the existing network and redeploy

Slight change after discussion: we'll push up a fresh tag, to avoid breaking repos in the wild, v0.66.0. Then we'll indeed fully deploy, nuking all chain state and resetting to 0. We'll still use the same chain id, so the name "Deimos" still refers to the batch of changes we tried to ship yesterday.

Post-merge, I will deploy an ad-hoc validator on preview and confirm it can reach the active set.


Buy function's math does not work for all recurring scenarios

Hello,
https://github.com/alfonsogoberjr/gdax-lambda-buyer/blob/master/handler.js#L62
This is something I found when setting up this project. Currently the buy function takes the DAILY_DEPOSIT and divides that by 24, I assume this was done because in the Readme the Cron job was set to trigger every hour. This would then mean you want your purchases to be calculated like this.
The issue comes if someone (me) wants to do a daily buy rather than an hourly buy. The reason for this usecase is say you want to buy only $5 a day of each crypto ($15 total), if you divide this by 24 that hourly value becomes too small and GDAX will not accept the call.
For myself I simply removed this division since I am using a daily cron task rather than hourly.
My proposal would be:

enforce that all currencies are purchased at the same rate by moving into a single "buy" function. This idea is due to the extra complications in calculating recurring purchase amount in cases where someone sets BTC to buy hourly, but ETH to buy daily for example.
in the buy function somehow detect the cron/rate expression (hourly/daily/weekly?) and do the proper calculations based on that. Because these can be set by cron the permutations can be pretty complex so I am unsure how to best handle the calculations for all cases.

Oh I am happy to mess around with adding this myself in a PR, I'd appreciate some thoughts on how to handle the 2nd bullet point. Possibly have checks forcing only certain reasonable types of CRON expressions, and supporting those? Then in the Readme add the various types of supported cron expressions which are handled cleanly?


Update the Algolia indexer to include artifacts and collections

This could ideally be combined with #610 (indexing our docs)

This was done last sprint


Don't update old bugs that are touched and closed straight away

I've seen this a few times now, most recently on bug 1362775:

Bug has sat for a few years without activity.
Someone finally triages the bug and resolves it (in this case WFM).
A day or so later, Bugbug comes along and comments that it might be a regression.

Since the bug has been resolved, we shouldn't need to be making sure the regression flag (and others?) are set correctly.
The main pain point here is the extra bugmail & bug noise that is generated and has to be ignored by those watching.

Since the bug has been resolved, we shouldn't need to be making sure the regression flag (and others?) are set correctly.

This is useful to have correct past information in bugs, so that we can be more correct when analyzing the development history and so on.

The main pain point here is the extra bugmail & bug noise that is generated and has to be ignored by those watching.

This is true, there is a risk that people will just ignore the updates and not revert them if they are wrong.
@calixteman @sylvestre because of this risk, I'd make the scripts only perform changes on recent bugs, unless they are 100% sure.

Maybe we could have a superpower to avoid to send mails for old bugs only.
@kyoshino do you know if with a special keyword (and maybe special rights) we could change a bug without sending an email to everybody in cc ?

The problem is that if the people don't see the change, they can't fix it in case it's wrong. That's why I was thinking we could do the change only when we're absolutely sure (that is, where we don't need confirmation).


2/4 - Implement native compilation - Implement boilerplate.

Please review the previous PR first - this builds off that.

Codecov Report

Merging #112 into master will decrease coverage by 0.97%.
The diff coverage is 30.12%.

@@            Coverage Diff             @@
##           master     #112      +/-   ##
==========================================
- Coverage    68.9%   67.93%   -0.98%
==========================================
  Files          33       34       +1
  Lines        3345     3421      +76
==========================================
+ Hits         2305     2324      +19
- Misses        783      837      +54
- Partials      257      260       +3

Impacted Files
Coverage Δ

wasm/operators/op.go
75% <0%> (ø)
:arrow_up:

exec/func.go
76% <100%> (+0.48%)
:arrow_up:

exec/native_compile.go
4.54% <4.54%> (ø)

exec/vm.go
82.78% <58.33%> (-4%)
: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 a76146c...39c312d. Read the comment docs.

:smile: :airplane:

please rebase on top the latest master. thanks.

Okay, I think I got that to work at the expense of the commit history (I think that can be squashed at merge?), LGTU?


core: Add if-match support in OpRead/OpWrite/OpStat

cc @STRRL, more detailed discussion could happen here.

Please assign to me, I can get this done and others could move forward implementing if-match on other services.

And by the way, why we want conditional header in OpWrite?

I believe this issue has been taken by @STRRL

I believe this issue has been taken by @STRRL. Hi, do you have time to finish this issue?

Sorry, my bad. I thought that is only related to S3.

And by the way, why we want conditional header in OpWrite?

user can only write to a path when its etag matches to avoid concurrent write conflict.

We could also add this support to OpCopy.

We could also add this support to OpCopy.

I prefer to split this feature in another issue.

I believe this issue has been taken by @STRRL. Hi, do you have time to finish this issue?

Yeah! I am working on it! ❤


[SR-7165] Excise iostream from reflection

Replace iostream with raw_ostream to follow LLVM Coding Standards.
With this PR, we still use iostream for reading in tools/swift-reflection-dump/swift-reflection-dump.cpp.

Resolves SR-7165.

Thanks! I'm not sure we can get away with using raw_ostream either, though, since some of this code gets linked into runtime libraries that don't and shouldn't link against LLVM either. It's probably best to use C printf/sprintf in most of these situations.

@jckarter I don't think any part of Reflection gets linked into the runtime at the moment, and there are no plans to do so.

If it's not linked into the runtime, it shouldn't live in stdlib/.

Oops, it looks like we forgot about this one. I'm going to close the PR because the libraries have changed a lot since last year, but the bug's still available.


Memory copy: calldata <-> storage

Handles copies of reference types from calldata to storage and storage to calldata

Looks good, does this support nested reference types, obviously referring to the ones that are possible to pass in and out of functions using call data:

Structs of structs
Dyn/Static arrays of Structs?


Vanilla <IP_ADDRESS>

https://terraria.org/system/dedicated_servers/archives/000/000/046/original/terraria-server-1423.zip

Thank you for letting me know, I have updated the repo and the docker containers.


Make ursa an optional dependency.

'ursa' is a binary module which is required when WSSecurityCert is used. For users who don't make use of X509 certificate it isn't required.
Introducing binary module brings extra requirement to the build environment. Some users report that unable to installed 'ursa'.
This PR makes 'ursa' an optional dependency. Fail to install it won't break other functionality of node-soap.
Update and restructure the README as well.

Coverage decreased (-0.04%) to 92.179% when pulling 6cd71c19182b97494a461c95e19e5e9d8f870f79 on aleung:optional_ursa into caa11505a968914f94340d538c1d2e657d49a19a on vpulim:master.

Your contribution is really amazing @aleung - thanks a ton man :+1:
I guess we can live with the optional solution, but I do also think that we should be aware that this does not solve the problem of binary modules when working with ursa... so, I'm just referencing the matching ursa issue report https://github.com/quartzjer/ursa/issues/133 so we can probably be updated whenever they come up with a solution for Windows/node-gyp


Require apply-refact <IP_ADDRESS>

Fixes #389, fixes #465, fixes #499, fixes #528.

I have no idea how to make the CI use apply-refact <IP_ADDRESS>. Neither one currently does.

Thanks!

I think we only install apply-refact on Appveyor. It uses whatever version is implied by the stack.yaml - I've just given a shot at bumping that.

This now works on Appveyor.

Thanks for updating CI!


fix(elgato-key-light): Fix error when toggling light

Description
Resolves error when using the latest Elgato firmware.

Replaces service.host with service.referer.address
Updates @raycast/api and axios dependencies

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Related Issues
closes #3700, closes #3487, closes #3326

Hi @zach-fuller 🔥
Thanks for your contribution 💯
@thomaspaulmann can you review this?

Oh thanks for the fix! My keylights stopped working recently and I couldn't figure out why.

Mmh, just checked again and unfortuantely this didn't solve my problem. Looks like I need to dig a bit deeper.

Thanks, this solves the issue I had! #3487

Thank you! Confirm that this fixes #3326 and the issues I reported.


'isLocaleLoading' store not available in svelte adapter

Version
5.24.3
Describe the bug
In the docs of the svelte adapter it says there is a isLocaleLoading function. However, it does not seem to be implemented/available.
Reproduction
Install latest package in a svelte(kit) app, import 'isLocaleLoading' like in the docs.
Logs
No response
Config
{
   "adapter": "svelte",
   "$schema"<EMAIL_ADDRESS>}

Additional information
No response

Thanks for letting me know. I have removed this outdated section from the readme.
If you need this behaviour, you can implement it yourself. The function loadLocaleAsync returns a promise and you should be able to know whenloading a locale has completed and manually set a variable.


Wrong caller checking in nx_dhcp_state_change_notify

https://github.com/azure-rtos/netxduo/blob/939bae5c4dafde45988d593285f444702ed9be0c/addons/dhcp/nxd_dhcp_client.c#L3602
Cannot set the DHCP's notify function from the initialization. The documentation says that it is allowed to call this function from the initialization.  See: https://learn.microsoft.com/en-us/azure/rtos/netx/netx-dhcp-client/chapter3#allowed-from-24
Furthermore, all other functions that set notify functions are allowed from the initialization.
It should be: NX_INIT_AND_THREADS_CALLER_CHECKING

Hello, thank you for your suggestion. We will take a look at it.

Closing. The caller checking has been corrected in this commit: https://github.com/azure-rtos/netxduo/commit/5b5c8c580684cd2f85001ff1763a62e0fdac9452#diff-474e87983b8ff48b783265e25bf5b6f5320d2460339a1bd29e22ae3e452b1cf8


Suggestion to allow ES6 import style syntax

Thoughts on allowing something like this in the future?
import { groupBy } from 'angular-filter'

I could also help out. Although your guidance may be needed since this would be a pretty big change right?


Set intersect query with []string and .In()

I'm working with a model that includes a tags []string kind of thing, and I'd like to know if it's possible to do some specific intersections for queries.
For reference, the feature I'm working on is open source so it may help to check the PR: https://github.com/openmultiplayer/web/pull/420/files
The relevant part of the schema is here: https://github.com/openmultiplayer/web/blob/forum-categories/prisma/schema.prisma#L94-L128 notably the list of Tags in each Post model.
It works great with these queries: https://github.com/openmultiplayer/web/pull/420/files#diff-3c9c79132aaec9a342a9567721f83d4d8b968069440955ebdbf80e89bd31b1b1R155-R183 if you specify one or many tags in the arguments, you get any post that has any of the specified tags.
So, let's say I have 3 posts

tagged "Food"
tagged "Music"
tagged "Food" and "Music"

When I query "food,music" I get post 1 and 2 but not post 3
When I query "food" I get post 1 but not post 3
When I query "music" I get post 2 but not post 3
So it seems posts with multiple tags cannot be queried with my approach.
Is there a good way of doing this with the current Go API? Or do I have to go to raw SQL for this.
Thanks!

I can confirm this, something is wrong internally here as the filters for in should actually not have In and array-specific methods (e.g. HasSome). I will try to pick this up and fix it soon.
In the meanwhile, I think you could try using an .Or() and then .Equals(tag) for each tag. Something like this:
queries := []db.PostWhereParam{
	db.Post.First.Equals(true),
	db.Post.CreatedAt.Before(before),
}
var tagQueries []db.PostWhereParam
for _, tag := range tags {
	tagQueries = append(tagQueries, db.Post.Tags.Some(db.Tag.Name.Equals(tag)))
}
queries = append(
	queries,
	db.Post.Or(
		tagQueries...
	),
)
// ...FindMany(queries)...

I know, this is ugly, but it'll save you doing two queries though :P

Hey, thanks for the response! This looks like a good alternative, it makes sense, but I just tried it and it doesn't seem to work.
I even tried hard-coding the query, just to be sure:
				FindMany(
					db.Post.First.Equals(true),
					db.Post.CreatedAt.Before(before),
					db.Post.Or(
						db.Post.Tags.Some(
							db.Tag.Name.Contains("open.mp"),
						),
					),
				).

This returns two items, which only have the tag open.mp but it doesn't return the item that has two tags, open.mp and forum:

Actually, I'm just going to remove the other filters (before date and first post) to make sure those aren't filtering it out accidentally.

Awesome thanks! I will give this a try when I get time and let you know how it goes 👍

Are you sure you're using the main branch version? There is no release yet.

Ah. That's a relation field and not an array/list field. You need to a relation query with .Where and then do .HasSome on the field of type string[]. Basically the same query as before except of doing .In you write .HasSome

Hm did your schema change? I'll check again tomorrow when I'm back on my computer

Ohh because my tags list is a relation not a simple list of strings? I completely forgot this detail while writing the issue!

Yeah exactly. These operations only exist for list/array operations. In your case it seems you just have a another model (Tag). You still should be able to query for what you want either way, e.g. with .Some and then compare the Tag.name with .Equals or something.

You might want to think about if maybe a string arrayed  would be sufficient, but that obviously depends on your use-case. I think it would fit considering your current schema but would not work anymore if e.g. users could have their own tags on posts.

Yeah the reason for this is to show all posts for a particular tag etc. also we might add colours to tags too or other metadata so it won't fit a []string.
I'll try what I originally tried which was db.Post.Tags.Some(db.Tag.Name.In(tags)), or the earlier suggestion you posted here https://github.com/prisma/prisma-client-go/issues/594#issuecomment-927948361


Broken Image

There's a broken image link in the readme. Just so you guys know.

Fixed, thanks @nathaniel-miller !


Integrated nodemailer-sendgrid-transport as a package

Fixes
Closes nodemailer-sendgrid-transport#67.
Checklist

[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the master branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation about the functionality in the appropriate .md file
[x] I have added inline documentation to the code I modified

Short description of what this PR does:

Integrates the then nodemailer-sendgrid-transport package as an sub-package among others (while having it match and follow the structure/convention used by other sub-packages).
Help with PRs such as NST#67, NST#64 and issues such as NST#65 and NST#25.

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.
Note: (as discussed in the PR mentioned above): the npm test fails due to the usage of the former sendgrid package in this codebase with some API changes and some sub-packages not available yet on NPM (causing all require('@sendgrid/xxx') to fail).

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: ImgBotApp:x: Berkmann18You have signed the CLA already but the status is still pending? Let us recheck it.

Whoops, a tiny detail I that I let pass by, I'll try removing the bot's commits and perhaps close this PR and open a new one for cleanness sake.


lifepo4?

Can be used for lifepo4 with out any changes in a code?

regarding to the datasheet, the MAX17055 uses a versatile algo to distingish between different chemistries. So yes, LiFePo can be used without changing anything.

https://pdfserv.maximintegrated.com/en/an/AN6358.pdf
MAX17055 USER GUIDE
Page 12 section 1.2.8 says :
1.2.8 Lithium Iron Phosphate Support (LiFePO4)
The MAX17055 supports lithium iron phosphate (LiFePO4) batteries with a special model configuration. To produce good state-of-charge
(%) accuracy performance, it is necessary to characterize and model the specific LiFePO4 cells being used. The MAX17055 provides
additional algorithm support specifically for the challenges associated with LiFePO4 and other flat OCV chemistries.
The OCV curve of LiFePO4 is much flatter than conventional lithium cobalt chemistries, which produces a greater sensitivity to the
algorithm's interpretation of cell voltage and open-circuit voltage. A LiFePO4 battery has two voltage plateaus. Using the ScOcvLim register
as a keep-out window, the fuel gauge limits OCV interpretation within a specified OCV voltage range. The default keep-out window is 3.275V
to 3.350V.


Update Quarkus and Mandrel versions in CI workflows

CI needs fixing...

@zakkak Those 23.0     DebugSymbolsTest failures are addressed in #177


Update view-only-meeting-experience.md

Changing from will to might because when the banner is thrown it just means the infrastructure is being spun up.

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

@bwilsonms The "will" is explaining what the user experiences once the meeting has hit capacity and matches what the banner says. Is there a technical nuance that you want to call out?

@bwilsonms The "will" is explaining what the user experiences once the meeting has hit capacity and matches what the banner says. Is there a technical nuance that you want to call out?

The banner can be sent to producers/co-producers as soon as the infrastructure spins up to support view-only, will seems to indicate it will happen immediately when this banner is shown, this is not the case.

@mkbond007


Feefighters Payment processing support

Add support for Samurai. The new gateway from feefighters.com.

Initial samurai support is out. Needs a little work as referenced in #24 and an issue with samurai.js

Close this and track on #24?


Add doesCastPreserveOwnershipForType() to SIL/Utils/DynamicCasts.cpp

Add doesCastPreserveOwnershipForType() to SIL/Utils/DynamicCasts.cpp
Added helpers in the AST Types.h

isNotPotentiallyAnyObject()

isPotentiallyBridgedValueOrExistentialType()

This formalizes logic for when cast operations forward
ownership. Various OSSA optimization rely on this for
correctness. This may fixes latent bugs throughout the optimizer.
I was compelled to fix this now because we want to make OSSA
optimizations across dynamic casts more aggressive. For example, we
want to optimize retain/release across enum formation.

This is the first step in fixing the OwnershipForwardingMixin. I'm not sure how to test this step yet.

@tbkka @jckarter I pretty comfortable with this logic now. Just need to add tests.

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test source compatibility

@swift-ci benchmark

@swift-ci test

@swift-ci benchmark

@swift-ci smoke test


OSError: [Errno 8] Exec format error: '/home/homeassistant/homeassistant/rtsp2webrtc_v5_freebsd'

Receiving error with WebRTC
Home Assistant 2021.12.3
WebRTC v2.2.0
Logger: root
Source: custom_components/webrtc/utils.py:175
Integration: WebRTC Camera (documentation, issues)
First occurred: 10:35:08 AM (1 occurrences)
Last logged: 10:35:08 AM
Uncaught thread exception
Traceback (most recent call last):
File "/usr/local/lib/python3.9/threading.py", line 973, in _bootstrap_inner
self.run()
File "/home/homeassistant/homeassistant/custom_components/webrtc/utils.py", line 175, in run
self.process = subprocess.Popen(
File "/usr/local/lib/python3.9/subprocess.py", line 951, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "/usr/local/lib/python3.9/subprocess.py", line 1821, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
OSError: [Errno 8] Exec format error: '/home/homeassistant/homeassistant/rtsp2webrtc_v5_freebsd'

I've performed a rollback to:
Home Assistant 2021.11.5
WebRTC v2.1.1
Regained full functionality, will remain at these levels for the time being

@AlexxIT
FWIW
I have moved to the HASS Linux image and abandoned my over-engineered FreeBSD instance.  Running latest versions and working perfectly --amongst many other integrations and having access to addons.
Wish I did it sooner...


Document walkthrough of running pipeline with different config

Based on Teams chat with @zsusswein.
Following merge of #131 then the pipeline will be easier to run based on current main branch as the pool will already be set-up.
This means that it should be possible for someone e.g. me to run the pipeline relatively easily.
It would be an instructive task to try doing this, including say with a different configuation file for different parameter estimates from cfa-parameter-estimates:

Make an update to prod.parquet parameter estimates
Use https://github.com/CDCgov/cfa-config-generator to generate config (button is in actions tab to generate configs)
Try running pipeline pointing at new config

As with previous issues, I think tasks like this should be documented in package vignettes, so would suggest trying to do this and putting as package vignette to close.
(This issue is possibly duplicate of others.)

See #116 as possible dupe?


[Autocomplete] properties multiple, disableCloseOnSelect and filterSelectedOptions make virtualized list of autocomplete scroll up

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
For my virtualized Autocomplete I use your example from your documentation
When I try to do virtual Autocomplete component with properties: multiple, filterSelectedOptions, disableCloseOnSelect and choose an element in the middle of list, after choosing list scrolls up to the top.
Expected Behavior 🤔
Virtual List stays at the same place and don't scrolls up after choosing
Steps to Reproduce 🕹

Example is here: Example
Context 🔦

It's not comfortable, when there are about 100 elements in list
Your Environment 🌎

Tech
Version

Material-UI
latest

React
latest

TypeScript
latest

@galkadaw Thanks for opening this issue, this is interesting. The origin is in
https://github.com/mui-org/material-ui/blob/861498c05fc27d794b10803928b8164709772409/docs/src/pages/components/autocomplete/Virtualize.tsx#L60
At some point, it would be great to abstract this demo, it starts to be too complex. In the meantime. What do you think of this patch of the demo? Do you want to work on a pull request? :)
diff --git a/docs/src/pages/components/autocomplete/Virtualize.tsx b/docs/src/pages/components/autocomplete/Virtualize.tsx
index fc731622f..09c419739 100644
--- a/docs/src/pages/components/autocomplete/Virtualize.tsx
+++ b/docs/src/pages/components/autocomplete/Virtualize.tsx
@@ -26,6 +26,16 @@ const OuterElementType = React.forwardRef<HTMLDivElement>((props, ref) => {
   return <div ref={ref} {...props} {...outerProps} />;
 });

+const useResetCache = (data) => {
+  const ref = React.useRef();
+  React.useEffect(() => {
+    if (ref.current) {
+      ref.current.resetAfterIndex(0);
+    }
+  }, [data]);
+  return ref;
+};
+
 // Adapter for react-window
 const ListboxComponent = React.forwardRef<HTMLDivElement>(function ListboxComponent(props, ref) {
   const { children, ...other } = props;
@@ -35,6 +45,8 @@ const ListboxComponent = React.forwardRef<HTMLDivElement>(function ListboxCompon
   const itemCount = itemData.length;
   const itemSize = smUp ? 36 : 48;

+  const gridRef = useResetCache(itemCount);
+
   const getChildSize = (child: React.ReactNode) => {
     if (React.isValidElement(child) && child.type === ListSubheader) {
       return 48;
@@ -57,7 +69,7 @@ const ListboxComponent = React.forwardRef<HTMLDivElement>(function ListboxCompon
           itemData={itemData}
           height={getHeight() + 2 * LISTBOX_PADDING}
           width="100%"
-          key={itemCount}
+          ref={gridRef}
           outerElementType={OuterElementType}
           innerElementType="ul"
           itemSize={(index) => getChildSize(itemData[index])}
@@ -84,6 +96,7 @@ function random(length: number) {

 const useStyles = makeStyles({
   listbox: {
+    boxSizing: 'border-box',
     '& ul': {
       padding: 0,
       margin: 0,

Thank you very much, it helped me

What do you think of this patch of the demo?

I think that it may be helpful in some cases, but I agree with bvaughn. in case, when we don't need to recalculate size on every data change, it will be just hard and expensive operation and not useful. For my case it is not useful.
but I'll check it more closely later

Do you want to work on a pull request?
what should pull request change? an example of documentation?

@galkadaw I hope this helps:
https://github.com/mui-org/material-ui/blob/master/CONTRIBUTING.md


Function with type argument specified cannot be parsed in as cast

The following code is rejected by both CFE and analyzer
int foo<T>(T t) => 42;

main() {
  var v1 = foo; // Ok
  var v2 = foo<int>; // Ok
  var v3 = foo as int Function(int); // throws, I know, but statically Ok
  var v4 = foo<int> as int Function(int); // Error. The operator '<' isn't defined for the type 'int Function<T>(T)'
}

Tested on Dart SDK version: 3.4.0-edge.3f91beb64f8af4705e8e2e6aec1bd78f947b8a31 (main) (Mon Jan 15 23:41:29 2024 +0000) on "linux_x64"

This would be a parser issue, so I'll use 'area-fe-analyzer-shared'. Note that foo<int> as Object fails, too.

cc @jensjoha

The issue exists for  var v5 = bar<int> is Object; as well

IIRC, parsing of recordings of the form expr<...> is disambiguated based on the following token. If the token is on a list of specific tokens, the <...> is parsed as s type argument, otherwise it's parsed as less-than, ..., greater-than.
Most likely is and as are not on that list. The main reason is that the syntax was originally for types, and there is no need to years year the type of a Type object. It's also usable for instantiating a generic function. Again, if you can do that, it's very likely that you'll also know the type of the result already. So not a big loss.
The reason the words are not included anyway, is that as is a built in identifier, not a reserved word, so
foo(a < b , c > as(d, e))

is a valid invocation of a function with two arguments, and a case which requires the disambiguation. (I'm sure there is a version without something looking like a record type too.)

@lrhn, I can't spot any disambiguation rules about terms of the form <primary> <typeArguments>. I don't see any ambiguities that aren't resolved by the basic priority rule that Dart always uses: An alternative that occurs earlier in a derivation rule is used if the parse according to that derivation rule succeeds. This corresponds to the situation where alternatives can be given a priority, and then we give every alternative a priority such that textually earlier ones have higher priority than later ones. For example:
<selector> ::=
  '!' |
  <assignableSelector> |
  <argumentPart> |
  <typeArguments>;

In the particular case of foo(a < b , c > as(d, e)), < b , c > as does not parse as any of the earlier alternatives, and hence the parser will consume < b , c > as a <selector> which is derived via <typeArguments>.
The generic-function-call disambiguation rule does not kick in at < because the matching > isn't followed by (.
To me it looks likely that we don't have any special disambiguation rules about this construct (by the way, the fewer special rules the better ;-), but I'd need to take it into account in the spec parser if we do have it.

I'm referring to the "Grammar Ambiguities" section of the constructor tear-off specification .
We decide whether "primary<identifier, identifier >" is a type literal/function-instantiation based on the next token, and is and as e not on the list that allows the text to be type arguments.
The spec states that the grammar is effectively written as having positive/negative lookaheads, which I believe means it also applies when there is no ambiguity possibly, fx because there is only one possible type argument ("primary < identifier >", each precludes it being successfully parsed as less-than/greater-than, but if suggest going so for constancy.
We do use the rule eagerly, without checking if a later token would disambiguate.

I'm referring to the "Grammar Ambiguities" section

Ah, I forgot that this kind of generic function instantiation was included as part of the constructor tear-off feature. Thanks!

I notice that the spec grammar didn't actually include these lookaheads

That's true, they are not needed in order to handle any ambiguities, because they are already handled by the built-in priorities of every grammar rule of Dart. Also note that the actual parser implementation will naturally implement this kind of prioritization because the basic structure of the parser is recursive descent parsing.
We could add disambiguation rules in, probably, dozens of places in the grammar, but we do also have that overall disambiguation mechanism, so the question is whether or not we should have such rules
Anyway, it's specified like that so we should look into it: https://github.com/dart-lang/language/issues/3588.

Closing: The specification says that the <typeArguments> alternative of the <selector> derivation rule is only applicable in the case where the following token belongs to a set that does not include as or is.
This means that the text foo<int> as int Function(int) (or foo<int> as Object, it doesn't matter which type is the target of the cast) is not a Dart expression, it is a syntax error. Hence, there is no implementation effort: it is already treated as an error at compile time.
If we want to generalize the grammar to derive this kind of expression then we can handle it in a different issue (in particular, it would be in the language repo, not the sdk repo).


NPE during upload with 1.113.12

com.google.cloud:libraries-bom:19.0.0 (com.google.cloud:google-cloud-storage:jar:1.113.12), we sometimes see:
com.google.cloud.storage.StorageException: java.lang.NullPointerException
	at com.google.cloud.storage.StorageException.translateAndThrow(StorageException.java:81)
	at com.google.cloud.storage.BlobWriteChannel.flushBuffer(BlobWriteChannel.java:209)
	at com.google.cloud.BaseWriteChannel.close(BaseWriteChannel.java:151)
	at com.google.cloud.storage.StorageImpl.createFrom(StorageImpl.java:265)
    ...snip...
Caused by: java.lang.NullPointerException: null
	at com.google.cloud.storage.spi.v1.StorageRpc$Option.get(StorageRpc.java:85)
	at com.google.cloud.storage.spi.v1.StorageRpc$Option.getString(StorageRpc.java:89)
	at com.google.cloud.storage.spi.v1.HttpStorageRpc.setEncryptionHeaders(HttpStorageRpc.java:242)
	at com.google.cloud.storage.spi.v1.HttpStorageRpc.getCall(HttpStorageRpc.java:439)
	at com.google.cloud.storage.spi.v1.HttpStorageRpc.get(HttpStorageRpc.java:455)
	at com.google.cloud.storage.BlobWriteChannel.getRemoteStorageObject(BlobWriteChannel.java:80)
	at com.google.cloud.storage.BlobWriteChannel.access$500(BlobWriteChannel.java:33)
	at com.google.cloud.storage.BlobWriteChannel$1.run(BlobWriteChannel.java:174)
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
	at com.google.api.gax.retrying.DirectRetryingExecutor.submit(DirectRetryingExecutor.java:105)
	at com.google.cloud.RetryHelper.run(RetryHelper.java:76)
	at com.google.cloud.RetryHelper.runWithRetries(RetryHelper.java:50)
	at com.google.cloud.storage.BlobWriteChannel.flushBuffer(BlobWriteChannel.java:154)
	... 31 common frames omitted

Same call succeeds when retried.

Thanks @n12c
Does your usage look similar to:
 BlobId blobId = BlobId.of(bucketName, key);
 BlobInfo blobInfo = BlobInfo.newBuilder(blobId).build();
 storage.createFrom(blobInfo, ...);

Indeed.
public long put( String name, InputStream stream, long length, SecretKey key ) {
    var blobInfo = BlobInfo.newBuilder( bucket, name ).build();
    var bufferSize = length == -1 ? uploadMaxBufferSize : (int) Math.min( length, uploadMaxBufferSize );
    var options = key == SentinelKey.PLAINTEXT ?
        new BlobWriteOption[]{ BlobWriteOption.disableGzipContent() } :
        new BlobWriteOption[]{ BlobWriteOption.disableGzipContent(), BlobWriteOption.encryptionKey( key ) };
    return Unchecked.supplier( () ->
        gcs.createFrom( blobInfo, stream, bufferSize, options ).getSize() ).get();
}

Thanks @n12c, I'm working on PR to resolve this. I can't release today (Friday today). Earliest will be Monday.
How often does this failure occur and how many objects are you uploading in your workflows?

That's fine, this code is not in production.
Not sure precisely how many objects, but I see a few thousand PUT and POST calls to GCS in my APM. I see the NPE in my logs two times, but I only adopted 1.113.12 a couple of days ago.

Thanks for clarifying, could you clarify APM?
Also do you know how big are your objects on average?

https://www.datadoghq.com/product/apm/
Objects sizes are bi-modal, one population is very small (hundreds of bytes) but these are not encrypted so probably not of interest here. The other population is encrypted and spans a wide range of sizes from kilobytes to hundreds of megabytes. Metrics explorer in the GCP console gives me a present bucket size of 799.486GiB and 16.565M objects (split evenly between populations) for an average size of ~47KiB across the whole bucket.

Thanks @n12c, I cut a release with the fix in version 1.113.13. PLMK if this has addressed the issue for you. From my experiments locally, this resolved potential NPE from passing in a null to a Get request in the library.


Add a Troubleshooting subsection to dev install docs

Based on discussion on gitter and #662, let's expand the notes subsection into a Troubleshooting subsection to https://github.com/ipython/ipywidgets/blob/master/docs/source/dev_install.md
While it might not solve the underlying issue, it will at least provide greater possiblity for users to find a solution when install is 😢
cc @fperez @SylvainCorlay

+1, excellent idea. Thanks!!

👍


falcoctl

What would you like to be added:
A new CLI tool called falcoctl.
This would be a new repository.
Here is a proof of concept
Why is this needed:
Think of falcoctl as a .go program that is statically linked so that

operators can enjoy Falco

I totally agree. If it's OK for others, I propose to migrate your POC under falcosecurity/falcoctl and move forward. It makes more sense either if we think about #809.

It should be called falconer as it’s purpose is to tame and control Falco.

Can we name it falconer and pronounce it as falcon E - R in honor of kube C - T - L?

Agree ! 🦅 🦅 🦅

Notes from slack:
1 - How do we install the Kubernetes resources needed for Falco? Do we have a new operator? Do we put this logic in the CLI tool? Do we parse YAML dynamically from local disk or URL? Obviously we need to allow folks to change bits in their specific Falco install (for instance kernel module vs eBPF probe)

2 - What are we going to do with rules/outputs and using those as composable building blocks for Falco? How will we store them? How will we maintain them?

3 - What on earth is going to take responsibility of reconfiguring and restarting the API server. This is a bold task.

4. Vendoring in Go still sucks, this isn't new

5. We shouldn't use init()

Another thought
 6. How do we handle `falconer`/`falcoctl` interacting with different versions of Kubernetes?

Version flags
We could have a flag to specify which version of Kubernetes to use (this might be a dependency nightmare)
Example
falconer --k8s=1.10.8 install falco # Installs this concrete version
falconer --k8s=1.15 install falco # Installs the most recent 1.15.* version

Release different binaries
This would allow us some freedom of building in custom logic for each release (relevant to #809) as we push this to GA.
We could branch our code for various versions of Kubernetes, but again, this would be a pain to keep up with. Maybe we could only support Kubernetes LTS?
falco1.14 install falco
falco 1.13 install falco

We also need to think about managed k8s, for example, we can't load kernel module on GKE, eBPF becomes mandatory. Same for auditsink (for k8s events), it's not available in EKS/GKE. If k8s versionning nigthmare is hell, it's still only top of iceberg.

@fntlnz RUNBPF

@kris-nova this feature looks very interesting, I was a bit skeptic about it initially but after looking at your poc code and testing it out I feel like we need it to improve our user experience by far.
RUNBPF contest

Send me an email `lo at linux.com` with your full name and address for the sticker! (I also accept encrypted emails if you have privacy concerns. You can get my public key here https://fntlnz.wtf/downloads/pubkey-0xD624DE73B2400EE4.asc)

Also please review:
https://github.com/falcosecurity/falcoctl/pull/3


Metrics are not going to specified metrics index

What happened:
metrics are not going to metrics index (my-metrics-index) in splunk.
What you expected to happen:
metrics logging goes to metrics index.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Confirmed that my metrics index my-metrics-index is of type metric.
Environment:

Kubernetes version (use kubectl version):
1.11.0
Ruby version (use ruby --version):
OS (e.g: cat /etc/os-release):
Splunk version:
7.0
Others:

values.yaml contents:
global:
  logLevel: info
  splunk:
    hec:
      token: my-token
      host: my-host.com
      port: 8088
      protocol: http
  kubernetes:
    clusterName: "my-cluster"

splunk-kubernetes-logging:
  splunk:
    hec:
      indexName: my-logging-index

splunk-kubernetes-objects:
  splunk:
    hec:
      indexName: my-objects-index

splunk-kubernetes-metrics:
  splunk:
    hec:
      indexName: my-metrics-index

Hi @MarkPare!
How are you deploying? Helm? Manifests?
We need to check the rendered configs. One good way to do that is to get the first 1000 lines of your metrics collector pod, as we dump the config there.
kubectl -n splunk logs -f <yourMetricsPod>
Or check in Splunk for your pod name. (search like this pod::my-pod if you have not installed app for infra or a fields.conf)
If you can share that I can usually spot the necessary actions needed.

@MarkPare closing issue due to inactivity. Please reopen/create a new one if you need further support.

@matthewmodestino thanks for your help on this and sorry for not replying sooner. The issue had to do with my noob-ness using splunk interface. The logging was working correctly.
Thanks to all contributors on this project for awesome work. This library is awesome!


Include installation via CDN in readme

With #37 the user can install the library via CDN. This should be mentioned in the installation section of the readme.
Example CDN link: https://unpkg.com/<EMAIL_ADDRESS>

Hi! 👋
I'd like to help, can you assign to me this issue?

Sure @MrDoomy!

Hello again @Fubinator 👋
I just create at PR right now.
Let me know if it doesn't fit, I'll change that quickly 😉


Support for OpenID Connect ID Tokens

I am trying to find a way to create an OIDC provider using dart since it is not recommended to use OAuth for authentication and only for authorization. since OIDC is build on top of oauth2, its possible to add the additional features to make this a OIDC packages instead.

If this is functionality you need, you’re more than welcome to send in a PR, but OpenID Connect is not necessarily on the priority list right now. If you are familiar with the OpenID specification, though, it should probably be pretty to implement on top of the AuthorizationServer class.


Migration of collections between communities

Currently it is not possible for the administrator to move collections between communities. It would be very beneficial to have this possibility.

Please check the related issue at #3198

Backend endpoint(s) necessary to implement this feature seems to be mostly described in https://github.com/DSpace/DSpace/issues/9824


Feat: Adding Register button at home page

Describe the feature
There is no Register option at home page...login is available but when we go to login then the register button is available...so to make it more user friendly lets add register button at home page itself
Assign me this issue @Satyam1923
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

This is not needed @mehrun171

I would like to work on this issue. Please assign this task to me under @GSSoC24


Toggle empty block comment doesn't work well in Python

VSCode Version: Code 1.16.1 (27492b6bf3acb0775d82d2f87b25a93490673c6d, 2017-09-14T16:38:23.027Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

python
don
0.7.0

cpptools
ms-
0.12.4

Steps to Reproduce:

Press the key binding Shift+Alt+A (Toggle block comment) when editing a python file.
Press again. The comment will not be removed, but a weird leftovers will be left.

Reproduces without extensions: Yes/No

@tomer-gavish Can you share an example? Screenshots and actual code will help

Yes.
Now I notice that it isn't reproduced if any text is selected, but only when the keyboard shortcut is pressed when no text is selected.
Try to insert the following line in a python script, with the cursor in the following position:

Press Alt+Shift+A (Without marking anything), and then again - and you'll get the following state:

It seems that, when no selecting nothing, enabling and disabling block comments doesn't work. It's worth mentioning that this works in other editors, and this feature is pretty useful if you want to start a block comment with no text inside.

I see something similar in javascript as well
When inside an empty block comment, toggling it just keeps adding more empty block comments
So this must be a generic problem.

Caused by 5c58b8cb2d59e1151c87cf2fbe46f69b41a3620f


Release scheduledFlush under a synchronized block

Since the write to scheduledFlush is neither volatile nor written under a synchronized block I don't believe it is guaranteed to be visible to all threads.
This could have the following effects:

Automatic flushing for BulkMutation is stuck, no more writes get scheduled since no threads observe a null value for scheduledFlush.
Another flush could be scheduled before send has returned.

Waiting for new release of bt-client-core with this fix :)

Is this something that needs to be done today?  If so I can do it.  If not, can this wait until Monday?  We're working through new HBase2 async features.

@sduskis we can wait till Monday. Thank you

I just released 1.2.0 earlier today.  I had some trouble with some other changes that went into the build.  Sorry for the delay


Use it with Nuxtjs

Hi everyone,
Have a nice day,
Firstly, I would really thank you for this library , it saves my days during work.
Secondly, I would like to know how can I use it with nuxt js in a proper way. if is there any specific requirement ?
Thanks in advance.

@allaghi Here's a guide for adding Tailwind CSS to Nuxt: this or this
Then you can install and add daisyUI like this
Let me know if you have any questions


Test3

Related Issue
Closes #856
Description
I have added the Unstop coding platform tool
Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

Hello, contributors! This is Jason speaking. Due to some workflow issues, we will be closing all the PRs today. Therefore, I kindly request all of you to create new PRs tomorrow, as the currently open ones will be closed by our team. I apologize for any inconvenience caused. Also, please make sure to check the titles when creating the PRs. You can refer to the README.md for more details. Thank you.