Move MIDI's methods and constants into InputEventMIDI

Draft it will not compile.
Closes https://github.com/godotengine/godot-proposals/issues/8761.
Related to https://github.com/godotengine/godot/pull/86693.
This PR moves OS' methods about MIDI, and the global constants into the InputEventMIDI. The new methods are static.
The previous methods and constants still exist, but are marked as deprecated.
This means the following:

Edit: These methods are currently commented out, because they refuse to compile. Will ask for help later.

Currently
This PR

OS.open_midi_inputs()
InputEventMIDI.open_inputs()

OS.get_connected_midi_inputs()
InputEventMIDI.get_connected_inputs()

OS.close_midi_inputs()
InputEventMIDI.close_inputs()

Currently
This PR

MIDI_MESSAGE_NONE
InputEventMIDI.MESSAGE_NONE

MIDI_MESSAGE_NOTE_ON
InputEventMIDI.MESSAGE_NOTE_ON

MIDI_MESSAGE_NOTE_OFF
InputEventMIDI.MESSAGE_NOTE_OFF

MIDI_MESSAGE_AFTERTOUCH
InputEventMIDI.MESSAGE_AFTERTOUCH

MIDI_MESSAGE_CONTROL_CHANGE
InputEventMIDI.MESSAGE_CONTROL_CHANGE

MIDI_MESSAGE_PROGRAM_CHANGE
InputEventMIDI.MESSAGE_PROGRAM_CHANGE

MIDI_MESSAGE_CHANNEL_PRESSURE
InputEventMIDI.MESSAGE_CHANNEL_PRESSURE

MIDI_MESSAGE_PITCH_BEND
InputEventMIDI.MESSAGE_PITCH_BEND

MIDI_MESSAGE_SYSTEM_EXCLUSIVE
InputEventMIDI.MESSAGE_SYSTEM_EXCLUSIVE

MIDI_MESSAGE_QUARTER_FRAME
InputEventMIDI.MESSAGE_QUARTER_FRAME

MIDI_MESSAGE_SONG_POSITION_POINTER
InputEventMIDI.MESSAGE_SONG_POSITION_POINTER

MIDI_MESSAGE_SONG_SELECT
InputEventMIDI.MESSAGE_SONG_SELECT

MIDI_MESSAGE_TUNE_REQUEST
InputEventMIDI.MESSAGE_TUNE_REQUEST

MIDI_MESSAGE_TIMING_CLOCK
InputEventMIDI.MESSAGE_TIMING_CLOCK

MIDI_MESSAGE_START
InputEventMIDI.MESSAGE_START

MIDI_MESSAGE_CONTINUE
InputEventMIDI.MESSAGE_CONTINUE

MIDI_MESSAGE_STOP
InputEventMIDI.MESSAGE_STOP

MIDI_MESSAGE_ACTIVE_SENSING
InputEventMIDI.MESSAGE_ACTIVE_SENSING

MIDI_MESSAGE_SYSTEM_RESET
InputEventMIDI.MESSAGE_SYSTEM_RESET

This would also be a great opportunity to discuss the usefulness of these constants, and the names associated with them:

...

I apologize for the many, many incoming merge pushes that will ensue.

Given the above issues with the property I'd suggest removing the enum related changes and keep just the moving of the MIDI driver management, that part I think makes a lot of sense to move out of OS, and is smooth to do and doesn't break anything

I could split the PRs in two and keep this one for the Messages. They took me way longer than the methods to transfer over, and there's some discussion to be had for how useful some of them are.

Sounds like a good idea, but unsure how to handle that one without breaking compat at all, so wouldn't be feasible for the short term or even the medium term, but worth saving it

As of currently, this does not compile specifically because of the enum referencing the other enum. So indeed, they are being problematic.

Oh yes that's because of the difference in class-ness, should be solved by just doing MESSAGE_NONE = (int)MIDIMessage::NONE,, but that does have some consequences

This was an accidental removal of a branch but it's no big deal, we are splitting it in two anyway.


Gaster just waits

gaster just waits like that --- the other pwndfu's working perfectly.

run futurerestore's usb fix


Created the newsletter with proper functioning and validation pop up

I've added the proper pop up and proper functioning of the newsletter
i enhanced the design of the search bar as well
Now on submitting the user will see proper message in proper pop up that will looks amazing and also indicate the user about the current state of his/her request .
closes : #668
https://github.com/mdazfar2/HelpOps-Hub/assets/118350936/59b76ac8-3b32-49f6-a690-d68dd6c4f1e6

Kindly make sure that you have added the popup card for this also- #690
It should show You are already subscribed to the Newsletter! in popup as well

Bro i did discussion with afzar aslam for the same

yes so add your popup for that message as well since its just added

@shreyash3087  bro i know that i have to add  it was creating conflict that's why i will do that after merging this
Bro i know what i am doing @mdazfar2  he know

@Ayushmaanagarwal1211  Alright thats fine then


Fix new flake8 errors (#471)

This is against master...

Need to merge this with #483 and a version change to 3.3.4 at the same time, otherwise we'll get 3 separate builds.

Hmmm, not really understanding that?  what is it that makes "3 separate
builds"?
On Wed, Nov 22, 2017 at 10:12 PM, Adam Haile<EMAIL_ADDRESS>wrote:

Need to merge this with #483
https://github.com/ManiacalLabs/BiblioPixel/pull/483 and a version
change to 3.3.4 at the same time, otherwise we'll get 3 separate builds.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/484#issuecomment-346474221,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdspzJOF2hlbx9sbTZMeSdu4g9tFQKks5s5I5YgaJpZM4Qn71e
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com

@rec Travis will build and push to PyPi whenever a new commit goes to master. 3 PRs, new PyPi builds. Technically you could do the version change last and the first two would fail because v3.3.3 already exists. But that seems messy.

Er, hmm, and why do we have that feature?
The idea of folding together three unrelated changes doesn't make me
happy.  Can we turn it off, at least temporarily?
Another possibility is that I prepare a master branch with all these three
commits in it and push it all at once...?
On Wed, Nov 22, 2017 at 11:30 PM, Adam Haile<EMAIL_ADDRESS>wrote:

@rec https://github.com/rec Travis will build and push to PyPi whenever
a new commit goes to master. 3 PRs, new PyPi builds. Technically you could
do the version change last and the first two would fail because v3.3.3
already exists. But that seems messy.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/484#issuecomment-346490285,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsiXdmK2bcKT_2HeHwtvnqlxgvwOtks5s5KBogaJpZM4Qn71e
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com

Wait... nevermind. I lied. It's on tags. Not on master.

har
har
har
On Wed, Nov 22, 2017 at 11:55 PM, Adam Haile<EMAIL_ADDRESS>wrote:

Wait... nevermind. I lied. It's on tags. Not on master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/484#issuecomment-346494789,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsmzBHFn28fa7QCxQYwOB2ti8rhxSks5s5KZGgaJpZM4Qn71e
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com

Feel free to merge all. I'll make a tag and push
On Nov 22, 2017 5:56 PM, "Tom Swirly"<EMAIL_ADDRESS>wrote:

har
har
har
On Wed, Nov 22, 2017 at 11:55 PM, Adam Haile<EMAIL_ADDRESS>wrote:

Wait... nevermind. I lied. It's on tags. Not on master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/pull/
484#issuecomment-346494789>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsmzBHFn28fa7QCxQYwOB2ti8rhxSks5s5KZGgaJpZM4Qn71e>
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/484#issuecomment-346494946,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6j-vCnUpZwOfBssz5aQpXf6gDrG7ks5s5KaFgaJpZM4Qn71e
.

I set it up forever ago... I can't be expected to remember these things :P
On Wed, Nov 22, 2017 at 5:56 PM, Tom Swirly<EMAIL_ADDRESS>wrote:

har
har
har
On Wed, Nov 22, 2017 at 11:55 PM, Adam Haile<EMAIL_ADDRESS>wrote:

Wait... nevermind. I lied. It's on tags. Not on master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/pull/
484#issuecomment-346494789>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsmzBHFn28fa7QCxQYwOB2ti8rhxSks5s5KZGgaJpZM4Qn71e>
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/484#issuecomment-346494946,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6j-vCnUpZwOfBssz5aQpXf6gDrG7ks5s5KaFgaJpZM4Qn71e
.

Wilco, but I'm wiped - it'll be tomorrow.
On Wed, Nov 22, 2017 at 11:57 PM, Adam Haile<EMAIL_ADDRESS>wrote:

Feel free to merge all. I'll make a tag and push
On Nov 22, 2017 5:56 PM, "Tom Swirly"<EMAIL_ADDRESS>wrote:

har
har
har
On Wed, Nov 22, 2017 at 11:55 PM, Adam Haile<EMAIL_ADDRESS>wrote:

Wait... nevermind. I lied. It's on tags. Not on master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/pull/
484#issuecomment-346494789>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsmzBHFn28fa7QCxQYwOB2ti8rhxSks5s5KZGgaJpZM4Qn71e>
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/pull/
484#issuecomment-346494946>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA6a6j-
vCnUpZwOfBssz5aQpXf6gDrG7ks5s5KaFgaJpZM4Qn71e>

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/484#issuecomment-346495114,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsnWqw4ucAi7wqTKqKX5oN6vPMD3yks5s5KbEgaJpZM4Qn71e
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com

OK!  I merged the two pull requests with no problems.  Do I just push a new
tag to master to push a new release?
On Thu, Nov 23, 2017 at 12:21 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

Wilco, but I'm wiped - it'll be tomorrow.
On Wed, Nov 22, 2017 at 11:57 PM, Adam Haile<EMAIL_ADDRESS>wrote:

Feel free to merge all. I'll make a tag and push
On Nov 22, 2017 5:56 PM, "Tom Swirly"<EMAIL_ADDRESS>wrote:

har
har
har
On Wed, Nov 22, 2017 at 11:55 PM, Adam Haile<EMAIL_ADDRESS>wrote:

Wait... nevermind. I lied. It's on tags. Not on master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/pull/
484#issuecomment-346494789>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsmzBHFn28fa7QCxQYwOB2ti8rhxSks5s5KZGgaJpZM4Qn71e>
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/pull/484#
issuecomment-346494946>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA6a6j-vC
nUpZwOfBssz5aQpXf6gDrG7ks5s5KaFgaJpZM4Qn71e>

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/484#issuecomment-346495114,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsnWqw4ucAi7wqTKqKX5oN6vPMD3yks5s5KbEgaJpZM4Qn71e
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com


Fixing broken link to azureml-examples recipes

The AzureML-Examples recipes link got broken when the folder structure changed.

Bypassing CI since this is docs only PR.


Feature/saaa 9116 atv audio focus

Thanks for opening a PR!
Since this is a volunteer project and is very active, anything you can do to reduce the amount of time needed to review and merge your PR is appreciated.
The following steps will help get your PR merged quickly:
Update the documentation
If you've added new functionality, update the README.md with an entry for your prop or event.
The entry should be inserted in alphabetical order.
Update the changelog
After you open the PR, update the CHANGELOG.md file with an entry pointing to your PR.
Provide an example of how to test the change
If the PR requires special testing setup provide all the relevant instructions and files. This may include a sample video file or URL, configuration, or setup steps.
Focus the PR on only one area
Testing multiple features takes longer than isolated changes and if there is a bug in one feature, prevents the other parts of your PR from getting merged until it gets fixed.
If you're touching multiple different areas that aren't related, break the changes up into multiple PRs.
Describe the changes
Add a note describing what your PR does. If there is a change to the behavior of the code, explain why it needs to be updated.

Hi @joudard-wyplay 👋
Thanks for submitting your first PR.
This will need some work before it can be landed including a rebase. Most work right now is happening on master for version 6, this looks like it was forked from an older copy of version 5.
Can you please explain what this fixes or adds and we can decide if it should be merged to v6 or v5 or both.
There seems to be unrelated work in the PR too, some dependencies and an analytics prop that would need to be removed.

Ho sorry, I think I just messed up the repo  / master branch
I was just making an update on the old fork we work with
only the two last commit could be of general interest

I think youbora support can make sens ! :) Even if It would need some rework

@freeboub maybe a COOKBOOK.md could be added with a tutorial on how to integrate with an analytics platform?

@evoactivity I agree about maintenance, I already integrate something like 5 differents analytics library and each one have completly different API ... so the cookbook will be a nightmare to maintain :) let's wait and see


Add Cargo compatibility.

I'd like to like to depend on this from screenshot-rs for Windows support. I'd also need GetDC() and SRCCOPY for taking screenshots.

+1.
Also we need to split rust-windows crate into "nice wrapper" and "*-sys" ones.

Oh, even better if you can provide a safe API for what I'm trying to do.
What would it be like, this:
let null = Dc {
    raw: 0,
};
let other_null = GetDC(0);
let hdc = MemoryDc::new(null).unwrap();
let hbmp = hdc.raw.create_compatible_bitmap(1920, 1080);
hdc.select_object(hbmp);
hdc.bit_blt((0, 0), (1920, 1080), other_null, (0, 0), SRCCOPY);

?
I'm honestly not sure why CreateCompatibleDC takes a NULL but BitBlt and CreateCompatibleBitmap take a GetDC(NULL) in my attempt, or whether that's necessary at all.

Ah, also need GetSystemMetrics.

I already have a separate crate that handles the raw bindings (https://github.com/retep998/winapi-rs) so moving all the raw definitions over there and keeping this project as a nice wrapper would be a good idea.

wow, thats a nice crate you have there @retep998.

@retep998 Would you be amenable to PRs?

@alexchandel I love PRs.

@retep998 sent one for winapi-rs and one for user32-sys.

I think we can close this one now.

Thanks all :D


Animation window: creating new animation beyond bottom of scrollview doesn't auto-scroll to bottom

Godot version
v4.0.stable.official [92bee43ad]
System information
Linux Ubuntu 20.04 with Unity desktop
Issue description
In the Animation window of the AnimationPlayer, when you create a new animation after many have been created, and scrollbar is not at the bottom, the new animation ends up in the hidden area at the bottom.
Usually in apps, this would auto-scroll the scroll view to the bottom, but currently it only focuses the new animation, so you still have to scroll down to the bottom manually (a trick is to press keyboard Up arrow to focus the element above, scrolling you down then press Down arrow to select the new animation). This is the trick I use in the GIF below:

Steps to reproduce

Create new Animation resource with many animations (enough to cover the whole height of the scroll view)
Add yet another animation
Bonus: press Up and Down arrow key as a trick. Or just scroll down with the mouse.

Minimal reproduction project
N/A

This should be easy to do by enabling autoscroll on the ItemList (I don't know if this exists), or by scrolling to the bottom when creating a new animation.


Organize imports error if run in one project in workspace with two projects

Note: I copied one project to another folder, and in VSCode added that folder to workspace.
Output panel after file clean up:

Could not create compilation unit for file (final fallback): c:\dev\flex\qtp\src\Main.mxml

Output panel without file clean up:

Could not create compilation unit for file (final fallback): c:\dev\flex\qtp\src\Main.mxml
svi 01, 2018 6:15:43 PM org.eclipse.lsp4j.jsonrpc.RemoteEndpoint fallbackResponseError
SEVERE: Internal error: java.lang.reflect.InvocationTargetException
java.lang.RuntimeException: java.lang.reflect.InvocationTargetException
at org.eclipse.lsp4j.jsonrpc.services.GenericEndpoint.lambda$null$0(GenericEndpoint.java:63)
at org.eclipse.lsp4j.jsonrpc.services.GenericEndpoint.request(GenericEndpoint.java:116)
at org.eclipse.lsp4j.jsonrpc.RemoteEndpoint.handleRequest(RemoteEndpoint.java:252)
at org.eclipse.lsp4j.jsonrpc.RemoteEndpoint.consume(RemoteEndpoint.java:181)
at org.eclipse.lsp4j.jsonrpc.json.StreamMessageProducer.handleMessage(StreamMessageProducer.java:183)
at org.eclipse.lsp4j.jsonrpc.json.StreamMessageProducer.listen(StreamMessageProducer.java:92)
at org.eclipse.lsp4j.jsonrpc.json.ConcurrentMessageProcessor.run(ConcurrentMessageProcessor.java:95)
at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source)
at java.util.concurrent.FutureTask.run(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.lang.Thread.run(Unknown Source)
Caused by: java.lang.reflect.InvocationTargetException
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at org.eclipse.lsp4j.jsonrpc.services.GenericEndpoint.lambda$null$0(GenericEndpoint.java:61)
... 11 more
Caused by: java.lang.NullPointerException
at com.nextgenactionscript.vscode.utils.ImportTextEditUtils.organizeImportsFromStartIndex(ImportTextEditUtils.java:98)
at com.nextgenactionscript.vscode.utils.ImportTextEditUtils.organizeImports(ImportTextEditUtils.java:175)
at com.nextgenactionscript.vscode.ActionScriptTextDocumentService.organizeImportsInUri(ActionScriptTextDocumentService.java:6141)
at com.nextgenactionscript.vscode.ActionScriptTextDocumentService.executeOrganizeImportsInUriCommand(ActionScriptTextDocumentService.java:6164)
at com.nextgenactionscript.vscode.ActionScriptTextDocumentService.executeCommand(ActionScriptTextDocumentService.java:1138)
at com.nextgenactionscript.vscode.ActionScriptLanguageServer$1.executeCommand(ActionScriptLanguageServer.java:309)
... 16 more
[Error - 18:15:43] Request workspace/executeCommand failed.
Message: Internal error.
Code: -32603
java.lang.RuntimeException: java.lang.reflect.InvocationTargetException
at org.eclipse.lsp4j.jsonrpc.services.GenericEndpoint.lambda$null$0(GenericEndpoint.java:63)
at org.eclipse.lsp4j.jsonrpc.services.GenericEndpoint.request(GenericEndpoint.java:116)
at org.eclipse.lsp4j.jsonrpc.RemoteEndpoint.handleRequest(RemoteEndpoint.java:252)
at org.eclipse.lsp4j.jsonrpc.RemoteEndpoint.consume(RemoteEndpoint.java:181)
at org.eclipse.lsp4j.jsonrpc.json.StreamMessageProducer.handleMessage(StreamMessageProducer.java:183)
at org.eclipse.lsp4j.jsonrpc.json.StreamMessageProducer.listen(StreamMessageProducer.java:92)
at org.eclipse.lsp4j.jsonrpc.json.ConcurrentMessageProcessor.run(ConcurrentMessageProcessor.java:95)
at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source)
at java.util.concurrent.FutureTask.run(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.lang.Thread.run(Unknown Source)
Caused by: java.lang.reflect.InvocationTargetException
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at org.eclipse.lsp4j.jsonrpc.services.GenericEndpoint.lambda$null$0(GenericEndpoint.java:61)
... 11 more
Caused by: java.lang.NullPointerException
at com.nextgenactionscript.vscode.utils.ImportTextEditUtils.organizeImportsFromStartIndex(ImportTextEditUtils.java:98)
at com.nextgenactionscript.vscode.utils.ImportTextEditUtils.organizeImports(ImportTextEditUtils.java:175)
at com.nextgenactionscript.vscode.ActionScriptTextDocumentService.organizeImportsInUri(ActionScriptTextDocumentService.java:6141)
at com.nextgenactionscript.vscode.ActionScriptTextDocumentService.executeOrganizeImportsInUriCommand(ActionScriptTextDocumentService.java:6164)
at com.nextgenactionscript.vscode.ActionScriptTextDocumentService.executeCommand(ActionScriptTextDocumentService.java:1138)
at com.nextgenactionscript.vscode.ActionScriptLanguageServer$1.executeCommand(ActionScriptLanguageServer.java:309)
... 16 more

@neminovno So, as I understand it, you've created a multi-root workspace, and you've added two root folders to the workspace. Is the file where you're running the Organize Imports command in your first root folder or is it in the second?

Not sure about terminology here. I wanted to work on two unrelated projects in same Workspace. I used right click in File Explorer then "Add Folder to Workspace":

It was file in second project where Organize imports command is run.
I didn't use Actionscript source path:

Currently, the extension supports only one root folder in a workspace. Some things, like build tasks and debugging, work in additional root folders already, but code intelligence does not yet. I need to do some serious refactoring to make the extension support multiple projects in the same workspace. It's been on my todo list for several months, but other things have taken priority.
I'm going to close this issue, in favor of the existing issue: #147


Add React loading spinner to product cards

Add React loading spinner to all of the product cards in Tuotteemme.tsx file. It should show the spinner while the product image is loading.

@nikohoffren I would like to work on this. please assign to me.


:inherited-members: does not work for attributes

I am not sure if this is a bug in Sphinx, but stuff like the root_path and static_url_path attributes/properties from _PackageBoundObject end up not documented in the API page of the docs. api.rst just contains this:
.. autoclass:: Flask
   :members:
   :inherited-members:

Sphinx moved their issue tracker to GitHub and marked the above linked issue as a duplicate, so the new link to the upstream issue is https://github.com/sphinx-doc/sphinx/issues/741


exclude using regex not working

Describe the bug
i'm trying to hide pages using regex (which should be possible as mentioned in docs.
this does not seem to work for a page called hide-me.vue:
  sitemap: {
    exclude: [/.*hide-me.*/],
  }

am i missing something or is this a bug?
Reproduction
https://stackblitz.com/edit/nuxt-starter-zqvdmm?file=nuxt.config.ts
System / Nuxt Info
------------------------------
- Operating System: Linux
- Node Version:     v18.18.0
- Nuxt Version:     3.8.2
- CLI Version:      3.10.0
- Nitro Version:    2.8.1
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      devtools, modules, nitro, site, sitemap
- Runtime Modules<EMAIL_ADDRESS>- Build Modules:    -
------------------------------

@PabloGBarcelo' has pushed up a fix that is available in v4.1.11.
Let us know if you have any issues with it :)

@PabloGBarcelo @harlan-zw
many thanks for the fix you two!
this might be a bit of an edge case, but strangely it doesn't seem to work, if the site.url includes a # character. e.g.:
  site: {
    url: 'https://site.url#foo',
  },

here is an updated reproduction:
https://stackblitz.com/edit/nuxt-starter-zqvdmm?file=nuxt.config.ts
if you remove #foo from the site.url the exclude is working.

Uh yes I'd avoid putting a hash in your site URL, this is meant to be the canonical site URL :grimacing:
I've made an issue here to track as the module should normalize the hash away https://github.com/harlan-zw/nuxt-site-config/issues/13


Migrate CloudBeesConfigStore to @configu/integrations

Suggestion
Migrate the CloudBeesConfigStore from its current setup to the new @configu/integrations module as part of the major restructuring. The goal is to move all third-party integrations, including stores, expressions, and libraries, to @configu/integrations.
This migration requires to migrate also the BaseStore of OpenFeatureConfigStore to integrations/src/utils same as the FileConfigStore, ORMConfigStore etc.
Motivation
This migration is part of a larger effort to better organize and modularize the product. Moving the CloudBeesConfigStore will align with the new structure, making the system cleaner and more maintainable.
Context
You can refer to the already migrated stores for guidance:
Migrated stores in integrations.
Base stores implementations from @configu/ts can be found here:
Base stores in TS.
The old node-based stores were previously in @configu/node:
Node stores.
The old browser-based stores were previously in @configu/browser:
Browser stores.

Please assign me this issue

All yours @Prathamesh010


Added FieldTitle tokenization for calculated fields

Hi Erwin,
I ran into this problem with calculated fields some time ago and reported it. Glad to see that there is a fix for it. I just generated a new template and I can see the tokenization for site field but not for list fields. Does this fix only apply to site fields?
Thx.

This implementation introduces a major bug. If applying the provisioning template twice the calculated field will not work anymore.
I guess this is because of some internal SharePoint manipulation when changing the SchemaXML. The provisioning template contains the field title (display name), but when provisioned to SharePoint, SharePoint translates it into the internal name and adds field references for each field. When applying the template for the second time it appears that SharePoint does not perform the translation of field title to field internal name.
The formula looks like this after second deploy when I edit the field in the browser: =#NAVN?


transformers do not work on file targets with matchpattern/replacepattern

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I have a target like so:
targetID:
    name: bump  version
    sourceid: sourceID
    kind: file
    spec:
      file: path/to/file.txt
        matchpattern: '(packagename)-([\d\.]+)(-snapshot(\.\d+)?)?'
        replacepattern: '$1-{{ source "sourceID" }}'
    transformers:
      - trimprefix: 'v'

Say the source returns v15.2.0 and I wish to trim off the "v" part so that "15.2.0" is used in the replacement operation. This does not seem to work.
However, if I use line: n instead of matchpattern/replacepattern, the transformer works.
Expected Behavior
I expect the source value without the trimmed prefix to be returned and not the whole source value.
Steps To Reproduce
No response
Environment
- OS: Ubuntu 22.04.1
- updatecli: 0.68.0

Anything else?
No response

Hi @oderayi,
The problem you highlight is something I couldn't find a good way to solve because it's hard to identify at the target level that you want to use the syntax {{ source "sourceid" }}
So I usually disable the default sourceinput using disablesourceinput set to true  and move the transformer rule at the source level such as:
sources:
  sourceID:
    name: Get source information
    kind: xxx
    spec:
      ...
    transformers:
      - trimprefix: v

targets:
  targetID:
    name: bump  version
    disablesourceinput: true
    kind: file
    spec:
      file: path/to/file.txt
        matchpattern: '(packagename)-([\d\.]+)(-snapshot(\.\d+)?)?'
        replacepattern: '$1-{{ source "sourceID" }}'

Thanks @olblak. I resorted to using an additional source with the trimmed prefix. I am happy to close this issue if there is no plan to support transformers in targets using matchpattern/replacepattern spec.


Add CI test coverage on AppVeyor (Windows)

Looks like the sniff test is working! https://ci.appveyor.com/project/kevmoo/angular/build/1.0.66


continue with -c yes even with .terraform directory is missing

when using -c yes on a brand new account or region, i get this message
There doesn't appear to be a previous run for aws2tf
missing .terraform directory in<PHONE_NUMBER>24
exiting ....

my ask is to instead of exiting, to merely print this non fatal warning and continue
why?
when working on regions with a large amount of resources, the script may need to be run multiple times, and with a human, the previous command in history may actually be the original without the -c yes meaning that we are actually starting over from scratch

You can't use "-c yes" on a new account and region - it can only be used when you have had a previous run.
The "-c yes" mean - continue from a previous run.
That is what it was looking for by checking for the ".terraform" directory existing.

my bad.
now that aws2tf has determined that -c yes wont work. continue on by disabling the switch

cool


running status is not updated

I edit my project on a nodejs-runtime, press run, the status should say "running", but it doesn't.
I reload the page, the status says "running".
I stop the graph, the status should say "not started", but it doesn't.
I reload the page and the status got updated.

Seems to be not reproducable today.

@ensonic we released a new version yesterday, might've been fixed by that


Parsing for all integer types + tests.

Squashed commits from #698. See that PR for full info.

Hi @jonfortescue, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@dotnet-bot test this please

@dotnet-bot test this please

@dotnet-bot test this please

@dotnet-bot test this please

@jonfortescue, I fixed the nuget feed issue. But there seem to be more failures unrelated to it.

@dotnet-bot test this please

@KrzysztofCwalina fixed the main bugs and squashed the commits. Everything's working on my machine, so we should be ready to merge assuming the server builds succeed.

Looks good.


[LINT] Turn on eslint-plugin-eslint-plugin/require-meta-docs-url

Continuation of #209
Should use a utility function so we don't have to have the URL in every single rule def.

@bradzacher can you close this one?


Improvements to blink

Great work @sjjhsjjh for implementing everything so far!
As @gavinhenderson  and you mentioned in #2  there is I think a bit of wiggle room to improve blink maybe doing your own custom blink detection using landmarks. Definitley needs some playing with but maybe not much better than the inbuilt one..
But anyways heres some random thoughts

The reason for me thinking of tweaking is the user cases where blinks are really subtle (like not fully shutting the eye) or eyes are really nearly closed to start with (we have seen this on Brain Injury units in hospitals..)
I think we could do with a Blink L or R and Blink L & R - so two options somehow.. Not sure neatest way of doing that
We do have some people who want to do an inverse "open eyes" for the action (I know it can sound niche - it is - but we do have some who are blind using this (I think Gavin might have mentioned this)

I think we need to implement a similar blink detection algorithm to the one we put together in EyeCommander. The current one just triggers as soon as we hit a threshold. Whilst this works its hard to setup for a user because if you go to low on the level it will trigger for involuntary blinks and too high will trigger for no blinks. Accuracy is very important in this sort of application.
The way we got around this was to add a timer in.
So in EyeCommander, there are two sliders. One for 'gesture size', exactly like we have in FaceCommander. The other was a threshold time, which is the time that we have to be over the gesture size for before the action is triggered.
So for example if you set it to 1 second, you have to have you eyes closed for a second before it triggers. In reality users usually put it around 500ms, which although can slow down usage, we found the all users prefer 100% accuracy over speed.
Also it would be nice to have a both eyes blink rather than just 'wink' like we have at the minute


Ci workflow

Originating Project/Creator

Affected Component

Affected Architectures(s)

Related Issue(s)

Has Unit Tests (y/n)

Builds Without Errors (y/n)

Unit Tests Pass (y/n)

Documentation Included (y/n)

Change Description
Added fprime-gds tests to workflow
Rationale
To ensure fprime-gds is not broken by future updates
Future Work
Create tests to ensure backwards compatability

@aidan-wagner what tests does this run?  Do we have pytests in GDS that aren't integration tests?

@aidan-wagner i stand corrected.  Looks good!


service: license schema should validate URL

The service schema apparently doesn't validate correctly links.
Filling in a custom license with a bogus URL doesn't report any errors:


Add global phpcs and drupal/coder to ddev in custom Dockerfile

Describe the bug
Following the examples in https://www.ddev.com/ddev-local/customizing-ddev-local-images-with-a-custom-dockerfile/ and the example Docker file:
# You can copy this Dockerfile.example to Dockerfile to add configuration
# or packages or anything else to your webimage
ARG BASE_IMAGE=drud/ddev-webserver:v1.13.0
FROM $BASE_IMAGE
RUN npm install --global gulp-cli

I tried to composer global require drupal/coder.
This didn't work as expected, as it installs for root

However the user that will later try to invoke it, isn't root.
To Reproduce
Add Dockerfile under .ddev/web-build with
ARG BASE_IMAGE=drud
FROM $BASE_IMAGE
RUN composer global require drupal/coder:8.3.5 --verbose
RUN composer global require dealerdirect/phpcodesniffer-composer-installer --verbose
RUN export PATH="$PATH:$HOME/.composer/vendor/bin"
RUN phpcs --config-set installed_paths ~/.composer/vendor/drupal/coder/coder_sniffer

It will error out on phpcs

If Instead I install it manually after the container is up with ddev composer global require... it works as-expected. So I suspect it has to do with the USER used to run the commands.

Absolutely everything in Dockerfile build time (for all Dockerfiles everywhere) happens as root.  composer global require should just be using root's home directory as a temporary storage place.
In your case, phpcs isn't really being installed globally, it's being installed in root's composer cache.
I see you're not the first person to encounter this, see https://unix.stackexchange.com/questions/280846/find-composer-global-install-path-as-root
I think the first thing to try is to symlink into /usr/local/bin

I'll take a look at this in the morning. But the basic issue is that composer's "install global" means "not site-install". It doesn't mean "make available for the machine", like it would everywhere else. But all that should be required is a symlink IIRC.

Thanks. Not sure how to create the symlink (as also the user doesn't have access to root's files), so I'd be happy for some help here :)
Once we figure this out, I'll create a PR to add that to the Dockerfile.example

(btw, in that example I should probably use cgr instead)

Here's what we used to do when we used cgr to install drush (I DO NOT recommend this)
RUN composer global require consolidation/cgr
RUN /home/.composer/vendor/bin/cgr drush/drush:$DRUSH_VERSION && ln -s /home/.composer/vendor/bin/drush /usr/local/bin/drush

So that approach will probably work. But the bottom line is that composer isn't a very good tool for installing things globally.
What I recommend is just using the instructions at https://github.com/squizlabs/PHP_CodeSniffer#installation to install with phar file. So much easier. Install it in /usr/local/bin.  This is how we install drush8 these days:
RUN curl -sSL "https://github.com/drush-ops/drush/releases/download/${DRUSH_VERSION}/drush.phar" -o /usr/local/bin/drush8 && chmod +x /usr/local/bin/drush8

BTW, I don't think the Dockerfile.example is a very good place for a complex item like this. Stack Overflow, ddev-contrib, or even adding it to that article are good places.

@rfay I see. But one more thing I didn't figure out yet - what about getting the specific coder sniffer for Drupal?
So we could do for example:
RUN composer global require consolidation/cgr
RUN ~/.composer/vendor/bin/cgr drupal/coder:^8.3.1
RUN ~/.composer/vendor/bin/cgr dealerdirect/phpcodesniffer-composer-installer

Then would we symlink them? If so what would be the correct place to do that?

BTW, I don't think the Dockerfile.example is a very good place for a complex item like this

:+1:

The install doc provides a lot of options, https://www.drupal.org/docs/8/modules/code-review-module/installing-coder-sniffer - It seems you can manually set the paths, phpcs --config-set installed_paths ~/.composer/vendor/drupal/coder/coder_sniffer, so that means you can composer-install drupal/coder as you wish and then point to it.

Sorry, I've actually tried it - and it doesn't work:
ARG BASE_IMAGE=drud
FROM $BASE_IMAGE

# We try to avoid when possible relying on composer to download global, so in PHPCS case we can use the phar.
RUN curl -L https://squizlabs.github.io/PHP_CodeSniffer/phpcs.phar -o /usr/local/bin/phpcs.phar
RUN curl -L https://squizlabs.github.io/PHP_CodeSniffer/phpcbf.phar -o /usr/local/bin/phpcbf.phar

# If however we need to download a package, we use `cgr` for that.
RUN composer global require consolidation/cgr
RUN ~/.composer/vendor/bin/cgr drupal/coder:^8.3.1
RUN ~/.composer/vendor/bin/cgr dealerdirect/phpcodesniffer-composer-installer

# Register Drupal's code sniffer rules.
RUN php /usr/local/bin/phpcs.phar --config-set installed_paths ~/.composer/global/drupal/coder/coder_sniffer --verbose

I believe the reason it doesn't work, is that when I can as a non-root, it no longer sees those files.  It just sees the default ones:
$ php /usr/local/bin/phpcs.phar -i

The installed coding standards are MySource, PEAR, PSR1, PSR12, PSR2, Squiz and Zend

Sorry, the way to work on this is not to do it in a Dockerfile. It's to do it in the running container. Or a running Debian container might be easiest. It's way to hard to work in a Dockerfile when you can't tell whether what is going to happen. But each of these steps can be taken one at a time inside the container, and it's much more satisfying. I'd be happy to do a zoom or whatever with you and we can work through this together.
Where I'd start is docker run -it drud/ddev-webserver:v1.13.1 bash and proceed from there.  It's a much more experimental setup and lets you figure out what's going on and solve the problems one at a time.

If you want to zoom right now we could.

You were almost there. The path given for the phpcs --config-set installed_paths  in the drupal.org article was incorrect, but there was still the problem of permissions in ~root.  There are many ways to solve this, but here's what I did, using COMPOSER_HOME as /usr/local/composer during the build stage.
What I did was to work interactively in docker run -it --rm drud/ddev-webserver:v1.13.1 bash (or docker run -it --rm drud/ddev-webserver:v1.13.1-<projectname>-built bash which has everything already working in your add-on Dockerfile working in it.
ARG BASE_IMAGE
FROM $BASE_IMAGE

ENV COMPOSER_HOME=/usr/local/composer

# We try to avoid when possible relying on composer to download global, so in PHPCS case we can use the phar.
RUN curl -L https://squizlabs.github.io/PHP_CodeSniffer/phpcs.phar -o /usr/local/bin/phpcs && chmod +x /usr/local/bin/phpcs
RUN curl -L https://squizlabs.github.io/PHP_CodeSniffer/phpcbf.phar -o /usr/local/bin/phpcbf && chmod +x /usr/local/bin/phpcbf

# If however we need to download a package, we use `cgr` for that.
RUN composer global require consolidation/cgr
RUN $COMPOSER_HOME/vendor/bin/cgr drupal/coder:^8.3.1
RUN $COMPOSER_HOME/vendor/bin/cgr dealerdirect/phpcodesniffer-composer-installer

# Register Drupal's code sniffer rules.
RUN phpcs --config-set installed_paths $COMPOSER_HOME/global/drupal/coder/vendor/drupal/coder/coder_sniffer --verbose
# Make Codesniffer config file writable for ordinary users in container.
RUN chmod 666 /usr/local/bin/CodeSniffer.conf
# Make COMPOSER_HOME writable if regular users need to use it.
RUN chmod -R ugo+rw $COMPOSER_HOME
# Now turn it off, because ordinary users will want to be using the default
ENV COMPOSER_HOME=""

@rfay this works great, thank you. Would you like me to add it in Stackoverflow or ddev-contrib?

Yay! Yes, please. ddev-contrib has an empty section for this kind of thing, https://github.com/drud/ddev-contrib#ddevweb-builddockerfile-examples-to-customize-web-container
Stack Overflow is fine and a little easier for a quick thing. Either is fine. Thanks!

Glad it's working! Closing this for now.

btw, here's a PR on drupal-starter that now uses this -- https://github.com/Gizra/drupal-starter/pull/7

Finally got to add it to StackOverflow -- https://stackoverflow.com/questions/61870801/add-global-phpcs-and-drupal-coder-to-ddev-in-custom-dockerfile/61870802#61870802

Thanks!

I recently had to set this up and a few things have changed, mainly:

references to phpcs
replaced the use of cgr for default composer global require
Updated installed_paths as phpcs kept reporting some were missing

Here's the latest config that worked for me:
## Install PHPCS Globally
ENV COMPOSER_HOME="/usr/local/composer"

# We try to avoid relying on Composer to download global, so in `phpcs` case we can use the PHAR.
RUN curl -L https://phars.phpcodesniffer.com/phpcs.phar -o /usr/local/bin/phpcs && chmod +x /usr/local/bin/phpcs
RUN curl -L https://phars.phpcodesniffer.com/phpcbf.phar -o /usr/local/bin/phpcbf && chmod +x /usr/local/bin/phpcbf

# Allow installation of known package plugins
RUN composer global config --no-plugins allow-plugins.drupal/coder true
RUN composer global config --no-plugins allow-plugins.dealerdirect/phpcodesniffer-composer-installer true

# Require composer packages globally
RUN composer global require drupal/coder
RUN composer global require dealerdirect/phpcodesniffer-composer-installer

## Register Drupal's code sniffer rules for phpcs.
RUN phpcs --config-set installed_paths $COMPOSER_HOME/vendor/drupal/coder/coder_sniffer,$COMPOSER_HOME/vendor/slevomat/coding-standard/SlevomatCodingStandard/,$COMPOSER_HOME/vendor/sirbrillig/phpcs-variable-analysis/VariableAnalysis/ --verbose
## Make Codesniffer config file writable for ordinary users in container.
RUN chmod 666 /usr/local/bin/CodeSniffer.conf
## Make COMPOSER_HOME writable if regular users need to use it.
RUN chmod -R ugo+rw $COMPOSER_HOME
## Now turn it off, because ordinary users will want to be using the default
ENV COMPOSER_HOME=""

@cballenar could you please edit your post to say what version of DDEV you changed for? (v1.23.0?)
Could you also add a comment with a link on https://stackoverflow.com/questions/61870801/add-global-phpcs-and-drupal-coder-to-ddev-in-custom-dockerfile/61870802#61870802 ? (Or edit it if you have permissions)

@cballenar your snipped not working
PHP Fatal error:  Uncaught Error: Class "Symfony\Component\Yaml\Yaml" not found in /usr/local/composer/vendor/drupal/coder/coder_sniffer/Drupal/Sniffs/InfoFiles/AutoAddedKeysSniff.php:59

@rfay Are where general way to install composer package globally ?
For example how can I utilize composer autocomplete for magento ?

Hi @sergiirepin - Please open a new issue when you have a new issue. But the approach would be the same as here, a custom Dockerfile that does
RUN composer global require bamarni/symfony-console-autocomplete

https://ddev.readthedocs.io/en/stable/users/extend/customizing-images/#adding-extra-dockerfiles-for-webimage-and-dbimage

@rfay thanks!
I will add an issue, but I am not sure if it is a ddev issue :)
Need some guide probablly, it is not clear how to configure the autocomplete: shell itself in WSL but PHP in docker.

You'll be using it inside ddev ssh, inside the web container, not on the WSL2 shell.

@rfay  Thanks!
The question in  SO now. :)


perf(interpreter): improve i256 instructions

Split off from #582
Summary:

i256_sign:

is split into the two possible variants as before (with the const bool), so we can use it with an immutable reference, avoiding copies in i256_cmp (see slt, sgt impls). i256_sign_compl gets optimized away like i256_sign::<true> before
by sorting  Sign  variants we can leverage derived Eq, Ord impls, which are can be further optimized by the compiler. Also, by using -1, 0, 1 it will be the exact same cmp::Ordering and the signum function result, which helps in the sign function (where we transmute num != 0 directly to Sign)

i256_div: if condition generates better code than a fully exhaustive match (branchless vs multiple branches). I don't know why, I would assume the two to be identical ... godbolt: https://godbolt.org/z/939obPrsj
other small changes are for readability: overflowing().0 == wrapping(), >> == wrapping_shr

Noticed one more thing (a774b294e50c0c970822105449d8552275b61cb3), we can drop the zero check in div and mod after doing the operation, since it allows for more optimizations at function return and results in better codegen overall.


Issue when analyzing code

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Clone https://github.com/osate/osate2.git
Run jscpd on it

You get the error:
(node:5198) UnhandledPromiseRejectionWarning: TypeError: name.replace is not a function
    at sanitizeLangName (/usr/local/lib/node_modules/jscpd/build/tokenizer/prism.js:62:21)
    at /usr/local/lib/node_modules/jscpd/build/tokenizer/prism.js:90:109
    at Array.forEach (<anonymous>)
    at createTokens (/usr/local/lib/node_modules/jscpd/build/tokenizer/prism.js:90:27)
    at /usr/local/lib/node_modules/jscpd/build/tokenizer/prism.js:60:137
    at Array.forEach (<anonymous>)
    at Object.tokenize (/usr/local/lib/node_modules/jscpd/build/tokenizer/prism.js:60:83)
    at JSCPD.<anonymous> (/usr/local/lib/node_modules/jscpd/build/jscpd.js:302:46)
    at step (/usr/local/lib/node_modules/jscpd/build/jscpd.js:43:23)
    at Object.next (/usr/local/lib/node_modules/jscpd/build/jscpd.js:24:53)
(node:5198) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3)
(node:5198) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Expected behavior
Program should terminate without error
Desktop (please complete the following information):
Ubuntu
JSCPD 2.0.13

Duplicated issue https://github.com/kucherenko/jscpd/issues/223


New button to enable SAR scaling

It may be a nice feature to have a button in the Misc menu that enables SAR scaling. What this would do is use the kernel chosen in the settings window for resampling and use that same kernel to stretch the output image following the SAR props. Mostly useful to give you a better idea of what the video will look like at the display resolution.

Fixed in #166


Automatic import causes compiler to fail Introduction / Nullable Types

When I add the if expression brackets, it automatically imports com.sun.security.ntlm.Client, which causes the compiler to fail. This doesn't happen when I copy and paste the solution into the IDE (Android Studio)

Task :Introduction-Nullable_types:compileKotlin FAILED
e: /Users/ifox/AndroidStudioProjects/Kotlin Koans/Introduction/Nullable types/src/Task.kt: (1, 30): Symbol is declared in module 'java.base' which does not export package 'com.sun.security.ntlm'
e: /Users/ifox/AndroidStudioProjects/Kotlin Koans/Introduction/Nullable types/src/Task.kt: (4, 13): Symbol is declared in module 'java.base' which does not export package 'com.sun.security.ntlm'
e: /Users/ifox/AndroidStudioProjects/Kotlin Koans/Introduction/Nullable types/src/Task.kt: (6, 25): Unresolved reference: personalInfo
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':Introduction-Nullable_types:compileKotlin'.

Compilation error. See log for more details

I wasn't able to reproduce this

I could not reproduce it either. The import of ntlm.Client does indeed break the compilation, but AS in my case did not suggest it automatically (though it is a high-priority guess for the Client auto-complition)

@mariyavdovina , please, take a look at this

This could only happen if class Clientdeclaration is missing from Task.kt for some reason. Accoridng to error message the initial text of task file was changed so the screenshot is needed to find out what really happens.

Sorry, mistakenly closed the issue:)


Update docs for Jib Core 0.19.0

@chanseokoh need a small review on this.

Oh, and just so you know, note that we deprecated the distroless-related methods, but it doesn't mean we removed them. The methods are there during the deprecation period and there are no breaking changes. It's just that they are marked deprecated potentially with warnings at runtime until they will be decommissioned in a major release bump if it were GA.

ah, thanks for clarifying!


@required checks miss calls to super constructors

For example:
import 'package:meta/meta.dart';

class A {
  A({@required int x});
}

class B extends A {
  B() : super();
}

fails to flag the call to super().

On closer inspection, it appears we're missing redirecting constructors as well:
class C {
  C({@required int x});
  C.named() : this();
}


61docker-kitematic

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

这是公司电脑61，docker管理界面kitematic

No information provided or steps to replicate - Closing 😞


Multi tenancy gitops cp4s

Adding print statements for troubleshooting cp4s recipe function call in main method.

Had to fix a bunch of differences between the two branches that were causing conflicts.


New package: Microstructure v1.0.0

Registering package: Microstructure
Repository: https://github.com/Tinggong/Microstructure.jl
Created by: @Tinggong
Version: v1.0.0
Commit: f412bb1f1cf5c60140248ec027b1ef2d25eec17e
Reviewed by: @Tinggong
Reference: https://github.com/Tinggong/Microstructure.jl/issues/2#issuecomment-2048107099
Description: Julia Toolbox for microstructure imaging

Please don't auto-merge this pull request

Superseded by #104654.


unable to run non docker method

Hi I cloned the repo then built and ran npm run start in both ui and src, i'm able to see it running, but then when i ask a question, i see this error: error: Error in discover route: connect ECONNREFUSED ::1:4000? does anyone know why? i also have searxng running using docker

Did you already have a pre existing instance of SearXNG? It shall start with the compose file

yes i have searxng and i ran docker-compose up and had it running in the background. And i ran npm run dev in both src and ui, but i kept getting errors, sometimes it's 409 sometimes it's 403

yes i have searxng and i ran docker-compose up and had it running in the background. And i ran npm run dev in both src and ui, but i kept getting errors, sometimes it's 409 sometimes it's 403

You need to use the SearXNG file we provide and need to change the config for the SearXNG URLs as well.

and run npm run dev for ui and src then searxng in the background right?


Project Wide Search in New Tab

Is there a way to open project wide search in a new window instead of the file browser pane?
It's terrible to have to resize that pane wider, and then back to a tree-list size when you're done.

VSCode Version: 1.19
OS Version: Mac OS High Sierra

Steps to Reproduce:

Search for anything project wide
Pull your eyes out because the results are in the file browser pane

Oops, duplicate of https://github.com/Microsoft/vscode/issues/9435


Adding Support for Workspace Settings like tasks.json

Visual Studio Code Version : 1.7.0
Code Sync Settings Version : N/A
Operating System : OSX 10.11
Occurs On: N/A
Proxy Enabled: N/A
Hi,
Would it be possible to sync tasks.json somehow.  I know they are workspace specific but I find most tasks I have I use across multiple projects and its a pain copying them into each new project. Having to sync them would be great.
Thanks

Yes i will work on support workspace settings not only task.json but other files in workspaces, but for now there are much work to do for supporting the current defined process for syncing the default settings from which some features are limitations by vs code, code team is already acknowledged off and will work on that
Once the default syncing features complete , i will start work on the work space settings :)

Adding this issue in the https://github.com/shanalikhan/code-settings-sync/issues/146 and closing this.


Unban

Hello, I used to play on this server for a while with a lot of my friends and got banned because they gave me their money before they quit and got banned for using alts to funnel money into one acc, i would like to get unbanned so i could comeback and play with some new friends since oblique is like the only good non ptw server with amazing quests and stuff. ty very much

What is your ign?

What is your ign?

CoachTrump

I can't seem to find any record of your ban. I will unban you, however any further server rule-breaking will quickly have this ban reinstated.


feat(review): Added Review Responses

Please Review
@skhtet
@seenusuvarna
@lsemesky
@rkenyon1969
@mleeBoeing

:+1:
Able to created review responses, cannot create duplicate reviews, can see the parent review with children responses, and able to respond to multiple distinct reviews.

Getting Review
GET /api/listing/1/review/ HTTP/1.1
Host: <IP_ADDRESS>:8001
Authorization: Basic d3NtaXRoOnBhc3N3b3Jk

Creating Review
POST /api/listing/1/review/ HTTP/1.1
Host: <IP_ADDRESS>:8001
Content-Type: application/json
Authorization: Basic d3NtaXRoOnBhc3N3b3Jk

{
        "rate": 5,
        "text": "Air mail crashes all the time - it doesn't even support IE 6!",
        "review_parent":3
}

:-1:
I am able to respond to my own original review
bigbrother review
-> bigbrother response

:+1:
Review Responses do not affect the rating of the listing

:+1:

From Brian,

Need to allow users to respond multiple times to reviews, in conversation form.
Editing a response should not move it's order in the conversation.

Updated code to accommodate Brian's criteria. Please review again.

:+1:

So, what about Delete for responses?  Are we not doing that?

tied with: https://github.com/aml-development/ozp-center/pull/849
:+1:


PPPP7.  7555?. .~!!!!:   | |_) / ____ \ |____ / ____ \| |____| |  | |/ ____ \ |__| |
:JPPP5!   .^!!!!:  .?YJJJ^     ...  :!!!!~.    |____/_/    \_\_____/_/    \_\______|_|  |_/_/    \_\____/
:^^^^:      :!!!!~.               .~!!!!:
             .~!!!!:             :!!!!~.
               :^^^^:           :^^^^:

Distributed Compute over Data

Bacalhau Repository
https://link.cod.dev/bacalhau-repo

Please file an issue if you encounter any problems!
https://link.cod.dev/bacalhau-new-issue

===============================================================================
Your system is linux_amd64
No BACALHAU detected. Installing fresh BACALHAU CLI...
Getting the latest BACALHAU CLI...
Installing v1.2.2 BACALHAU CLI...
Downloading https://github.com/bacalhau-project/bacalhau/releases/download/v1.2.2/bacalhau_v1.2.2_linux_amd64.tar.gz ...
Downloading sig file https://github.com/bacalhau-project/bacalhau/releases/download/v1.2.2/bacalhau_v1.2.2_linux_amd64.tar.gz.signature.sha256 ...
Verified OK
Extracting tarball ...
NOT verifying Bin
bacalhau installed into /usr/local/bin successfully.


pull request for e2e test at 12/31/2016 19:15:12

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


Added more logging for Stripe webhooks so we can get more data on attacks

So we can figure out how to preempt more of them.
Conditionally enabled donationWithStripe, does not 404 if fetch_stripe_processing_enabled_state() is true Added a column to wevote_settings in case we want to add a simple configuration page, in the meantime, added the new configuration to /stripe_donations/suspects_list/

Thank you @SailingSteve!  👍
I'd like to discuss before pushing live to the API server. I'm working today and am pretty flexible on times to speak.


Error on mix server

I am following the getting started guide. When i run mix server i get the following error
** (FunctionClauseError) no function clause matching in Keyword.put_new/3
(elixir) lib/keyword.ex:320: Keyword.put_new(nil, :ref, Router.HTTP)
lib/plug/adapters/cowboy.ex:35: Plug.Adapters.Cowboy.args/4
lib/plug/adapters/cowboy.ex:120: Plug.Adapters.Cowboy.run/4
lib/mix/tasks/server.ex:27: Mix.Tasks.Server.run/1
(mix) lib/mix/cli.ex:55: Mix.CLI.run_task/2

I am running this on Mac Yosemite and Elixir is v1.0.2

Hey @zabirauf, I think I know what the issue is, but I have a couple of questions:

Did you pull down Sugar as a Hex or git dependency?
To verify, you used mix sugar.init to scaffold your project, correct?

In the meantime, make sure you have something like this in your config/config.exs file:
use Mix.Config

config :sugar,
  router: Simple.Router

config :sugar, Simple.Router,
  https_only: false,
  http: [ port: 4000 ]

PS, https://sugar-framwork.github.io/ is currently out of sync with the latest code. :frowning: For the most part, I try to keep @doc comments up to date with code changes, so when in doubt, check those at http://hexdocs.pm/sugar/.

Am also able to reproduce with OSX Yosemite and Elixir 1.0.2 (after following the official docs to-the-letter - including using mix sugar.init to scaffold the project - and making some corrections to config.exs per your first comment).  Occurs regardless of whether Sugar is a Hex or git dependency.
On a whim, I decided to change my own app's lib/my_app/router.ex to reflect a module name of MyApp.Router (like what's implied in your comment and as described using the sample app) instead of simply Router (what mix sugar.init defaults to).  This resulted in a new error (again, regardless of whether Sugar was obtained via Hex or git):
** (UndefinedFunctionError) undefined function: Router.run/0 (module Router is not available)
    Router.run()
    lib/mix/tasks/server.ex:27: Mix.Tasks.Server.run/1
    (mix) lib/mix/cli.ex:55: Mix.CLI.run_task/2
    (elixir) lib/code.ex:316: Code.require_file/2

I've noticed in Sugar's lib/mix/tasks/server.ex that the offending line (27) is based on calling the run/1 method of a router variable, which is populated by a call to Sugar.Config.get(:placid, :router, Router).  Having no idea what :placid is, I temporarily forked and adjusted that line to read Sugar.Config.get(:sugar, :router, Router), which makes more sense and gets past that error (Pull request submitted, unless there's some legitimate reason why :placid is the thing being checked; perhaps an old module name or something?).
However, doing that (or even going the other way and just calling everything Router instead of MyApp.Router, with or without the fix in my fork) now results in a new error:
** (ArgumentError) could not start Cowboy adapter, to use relative certificate with https, the :otp_app option needs to be given to the adapter
    lib/plug/adapters/cowboy.ex:204: Plug.Adapters.Cowboy.fail/1
    lib/plug/adapters/cowboy.ex:167: Plug.Adapters.Cowboy.normalize_ssl_file/2
    (elixir) lib/enum.ex:1261: Enum."-reduce/3-lists^foldl/2-0-"/3
    lib/plug/adapters/cowboy.ex:135: Plug.Adapters.Cowboy.normalize_options/2
    lib/plug/adapters/cowboy.ex:37: Plug.Adapters.Cowboy.args/4
    lib/plug/adapters/cowboy.ex:120: Plug.Adapters.Cowboy.run/4
    (my_app) lib/my_app/router.ex:1: MyApp.Router.run/1
    lib/mix/tasks/server.ex:27: Mix.Tasks.Server.run/1

That error (and, now that I think about it, the original error) seems to be referencing an attempt to spin up a HTTPS server, even though :https_only was set to false and no :https key was defined.  Digging through Sugar.Router.run/1 revealed that run/1 does a check on :https via a normal if statement; forcing :https to be set to false (in config.exs) fixed this last issue, and now mix server works properly (returning a "Hello world" as expected).

Long story short: this issue can be worked around by explicitly setting :https to false, like so (in config/config.exs):
use Mix.Config

config :sugar,
  router: Router

config :sugar, Router,
  https_only: false,
  http: [port: 4000],
  https: false

@YellowApple Thanks for running through all of this! It definitely needs some tweaking with how HTTP/HTTPS listeners are ran, along with some additional tests for those and the server Mix task.
To explain about the :placid bit, I have another project (Placid) that started with the router from Sugar. I made a bunch of improvements to the router in Placid, and recently, I brought those changes back into Sugar and managed to forgot to change a couple references to Placid. I'm thinking about creating a shared library to share most of the router macros with the two projects and anyone else who might find a use for it (it's fairly portable), but I haven't decided quite yet.

No problem, @slogsdon; I'm just glad my code works now ;)  A documentation update would probably be useful for future folks pulling out their hair over Cowboy whining about HTTPS.
Good to know about Placid; I was looking for a good way to adapt Sugar to route API requests, and it looks like Placid is well-optimized for that while still having plenty of Sugary goodness.  I would definitely abstract as much shared code between the two as possible into a shared library, if only to make development easier (since they seem to share a lot in common).

I think I will be extracting that code into a separate library. I just diff'd them, and the differences should be able to be wrapped in configurations, merged, or removed just nicely. I just need to be sure that functionality remains the same and that the resulting module can be extended easily.

Consider this closes as of #66


New version: Microsoft.devtunnel version 1.0.1293+4a61ac5730

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

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

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\Microsoft.devtunnel__DefaultSource\devtunnel.exe returned exit code: 1
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\devtunnel.exe returned exit code: 1

(Automated response - build 878.)


GitHub actions on macos-latest

Maybe #4 might affect this proposal?

@kvch , are you familiar with the below error?
env:
    GOROOT: /Users/runner/hostedtoolcache/go/1.17.8/x[6](https://github.com/elastic/elastic-agent-system-metrics/runs/5968534193?check_suite_focus=true#step:5:6)4
package github.com/elastic/elastic-agent-system-metrics: build constraints exclude all Go files in /Users/runner/work/elastic-agent-system-metrics/elastic-agent-system-metrics
Error: Process completed with exit code 1.

:broken_heart: Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-04-11T10:01:59.673+0000

Duration: 3 min 26 sec

Steps errors
Expand to view the steps failures

Mage check

Took 0 min 1 sec . View more details here
Description: mage -v check

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

/test

/test

@fearful-symmetry Does the error on macos say anything to you?

So, most of the MacOS metrics are legacy code, but I suspect that it's either a permissions issue, or we're trying to call Statfs on some kind of invalid filesystem type for that syscall. We might just want to exclude vmhgfs from the filesystem tests.
There's also this, which seems....interesting:
 numcpu_test.go:33: CPU Count: -1

PR here to address some of the issues: https://github.com/elastic/elastic-agent-system-metrics/pull/13

/test

Merging to solve golangci-lint issues in agent PR.


unable to setup?

Ok this is now on my mac OSX .  Just thought I'd set it up here as well.
I used sudo npm install -g urcli and everything was happy
Then I went for
urcli setup token 'blahblahbal' and I get this
Expected behavior
setup my account
Actual behavior
I redacted my token here
Gilads-MBP:~ Amar$ urcli setup token "eyJ0eXAiOiJKV1QiLCJhbGciOi"
You can now add all the languages that you are certified for.
Simply type in each language code and separate them with a space.
Example: "en-us pt-br"
Certified language > 'en-us'
You have entered the following language(s):  [ 'en-us' ]
Was this correct? (y/n) > y
unhandled rejection: TypeError: certifications.body.filter is not a function
    at Interface.<anonymous> (/usr/local/lib/node_modules/urcli/lib/commands/setup.js:65:39)
    at next (native)
    at step (/usr/local/lib/node_modules/urcli/lib/commands/setup.js:20:191)
    at /usr/local/lib/node_modules/urcli/lib/commands/setup.js:20:361
    at process._tickCallback (internal/process/next_tick.js:103:7)

Steps to reproduce the problem
just running setup, certs, revenue anything gets me that answer.
I previously had this working locally before, so I'm not sure what happened.
It's telling me I have urcli 2.7
Your Operating System (e.g., Linux Ubuntu, MacOS, Windows)
mac os x 10.10.5
EDIT:
Tried with sudo -- didn't help

Are you using the exact command: urcli setup token "your-token". If so, you are using the word token as the token, in which case the this would be a natural error. Try using urcli setup "your-token" instead, without the extra word token in there. Hopefully it works :smile:.

oh geez
so embarassing
yup that was the problem.

No worries, I'd much rather you open issues about such things, than feel like you have to be super careful it's not user error. The thing is, there should be an error message telling you that you gave it the wrong token, so it's still valuable info.


feat: Try again in remote launching when ResourceLimitExceeded is caught

I could not directly use your decorater. Maybe there is a better solution.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jgolebiowski I think we should aim to close this. The situation is this:

We have a specific need, namely that a certain exception is thrown, of a certain subtype, when you run out of quota. Having this in benchmarking (or elsewhere in Syne Tune) would be great
I do not have the skills to connect your solution to this need. I don't know how stable it is to match class names of exceptions, in particular if they are from some external dependency. Your solution also does not allow dependence on the exception message

Solutions:

Either we replace your general case with the specific case we need at the moment (I don't really see a broader one)
Or you make sure your general case works for the specific one, hopefully not making it a lot more complex

How about a refactoring of your code like this?
a-b

OK, I think this is it. Please do check it again. I have not tested this (should try and run with instance type for which I have 0 limit)

Tested this:
(st_with_ray) 88665a46461b:syne-tune matthis$ python benchmarking/examples/launch_local/launch_remote.py --method BO --benchmark transformer_wikitext2 --experiment_tag blurb1 --instance_type ml.g5.24xlarge --estimator_fit_backoff_wait_time 10 Failed to import apex. You can still train with --precision {float|double}. coolname is not installed, will not be used Master random_seed = 163844129   0%|                                                                                | 0/1 [00:00<?, ?it/s]blurb1-BO-0 hyperparameters = {'experiment_tag': 'blurb1', 'benchmark': 'transformer_wikitext2', 'method': 'BO', 'save_tuner': 0, 'num_seeds': 1, 'start_seed': 0, 'random_seed': 163844129, 'scale_max_wallclock_time': 0, 'launched_remotely': 1, 'instance_type': 'ml.g5.24xlarge', 'verbose': 0, 'remote_tuning_metrics': 1, 'delete_checkpoints': 0, 'num_gpus_per_trial': 1} Results written to s3://sagemaker-us-west-2-719355911555/syne-tune/blurb1/BO-0/ botocore.exceptions.ClientError[ResourceLimitExceeded] detected when calling estimator.fit. Waiting 0.16666666666666666 minutes before retrying botocore.exceptions.ClientError[ResourceLimitExceeded] detected when calling estimator.fit. Waiting 0.16666666666666666 minutes before retrying

Nice, I have 0 quota for ml.g5.24xlarge. Seems to work


Kotlin support

Currently, lsif-java supports indexing Java and Scala projects. We're working on a SemanticDB compiler plugin for Kotlin in the repo here https://github.com/sourcegraph/lsif-kotlin
Subscribe to this issue for notifications when lsif-java index can also index Kotlin projects in Gradle/Maven.
Subtasks

[x] #338
[x] #339
[ ] #353
[ ] Support auto-indexing for Kotlin package repos on sourcegraph.com

We just released https://github.com/sourcegraph/lsif-java/releases/tag/v0.6.11 with support for Kotlin Gradle builds. Please give it a try by installing lsif-java (instructions https://sourcegraph.github.io/lsif-java/docs/getting-started.html ) and run lsif-java index.

@olafurpg, @scalabilitysolved: I renamed this issue and added it to this dashboard. Let me know if you preferred if we took a different approach.
@scalabilitysolved / @macraig Also, I noticed that this issue is closed. Are we technically considering adding Koitlin done for the purposes of our Q4 goal?

Kotlin hovers are the last item needing to be implemented before we'll consider Kotlin GA for our Q4 goal.

Closing this since lsif-java v0.7.2 now supports hover messages for Kotlin 💪 Issue  #353 is still open, but it's not blocking GA. We can add support for Maven Kotlin projects if there is demand for it.

Congrats team on completing the first Q4 OKR!


Support ECS Tasks

It’d be great if this construct could fully configure ECS tasks for DataDog including:

Adding a Fluentbit router container
Adding the DD agent container
Configuring Docker labels for tags
Configuring the Firelens log driver
Creating a CloudWatch Logs Group for the Fluentbit container

I have code for this in Python if you’re interested

Any news on this? This would make this library massively more useful to us.


Consistency and language issues

An issue for listing minor consistency problems.
The first one: edid_timing.json has width and height instead of frame_width and frame_height.

Closed by #16?

Unfortunately, no, but I created #21.

It's resolved in #21.


OxCGRT data not registered

At which point in the documentation do we register OxCGRT indicators into the Scenario object? Can we register those manually?

Thank you for your question!
Please register OxCGRT data manually by Sceanrio.register(extras=[oxcgrt_data]) before Scenario.estimate_delay().
import covsirphy as cs
# Load datasets
data_loader = cs.DataLoader("input")
jhu_data = data_loader.jhu()
population_data = data_loader.population()
oxcgrt_data = data_loader.oxcgrt()
# Create Sceanrio instance
snl = cs.Scenario(country="country name here")
# Register datasets
snl.register(jhu_data, population_data, extras=[oxcgrt_data])
snl.estimate_delay()

Documentation: https://lisphilar.github.io/covid19-sir/usage_quick.html#Start-scenario-analysis

Great, thanks for your help!

Did you try today argument of Scenario.timepoint()?
Kindly share the script you tried.

Additionally, version information and outputs/errors may be necessary for investigation. Could you create a new issue for that with "Request fixing a bug" template?
https://github.com/lisphilar/covid19-sir/issues/new/choose


Add OVF properties to configure static IP

This allows customers to allow addition of OVF properties
to configure static IP in the template VM or the encrypted OVF.
Primarily design for use to encrypt Ops Manager in PCF, but
is a generic implementation and can be used for any customer
which wants to use static IP.

Update will inherit the properties from encryption. One can directly assign a static ip to the wrap image in brkt-cli and no need for ovf properties to be able to do that.


Allow RoslynHelpers to create generics from complex name syntax nodes

Yes, that's correct. I'd like to move all the Roslyn utility code into a common tools project at some point, so I'd like this to cover all cases. That said, only the overload for QualifiedNameSyntax is unused at this point.

Arg, as soon as I merged this, I realized I forgot one last cleanup step. I'll send another PR for this shortly.


Widevine/Shaka player doesn't play video

Have you read the FAQ and checked for duplicate open issues?
There are a few (old) similar issues, except this doesn't result in an error message that I can see.
If the problem is related to FairPlay, have you read the tutorial?

What version of Shaka Player are you using?

Version: 4.10.2830.0
Can you reproduce the issue with our latest release version?
As far as I can tell this is the latest
Can you reproduce the issue with the latest code from main?
Yeah not my job buddy
Are you using the demo app or your own custom app?
No
If custom app, can you reproduce the issue using our demo app?
What browser and OS are you using?
MacOS Sequoia 15.0.1
Brave v1.71.114
Chrome (older)
Safari 18.0.1
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
What are the manifest and license server URIs?

Beats me.  I'm on Amazon prime, Netflix, then also tried https://shaka-player-demo.appspot.com/
What configuration are you using?  What is the output of player.getNonDefaultConfiguration()?

What?  No, I'm not building this damn thing, I just want to watch netflix.  And chill, potentially.
What did you do?

Go to anyplace that would use widevine to play DRM video.
Attempt to play video.
Get no video.  Only sound.
Be very frustrated because it was working a few days ago.

What did you expect to happen?
Not be frustrated by video not playing.
What actually happened?

Became very frustrated by video not playing.
Only sound.

Are you planning send a PR to fix it?

In order to help solve your problem you must provide the following information:
**What are the manifest and license server URIs?**
<!-- NOTE:
  You can send the URIs to<EMAIL_ADDRESS>instead,
  but please use GitHub and the template for the rest.
  A copy of the manifest text or an attached manifest will **not** be
  enough to reproduce your issue, and we **will** ask you to send a
  URI instead.  You can copy the URI of the demo app to send us the
  exact asset, licence server, and settings you have selected there.
  If you send the URIs to email, the response time and resolution
  will be much higher.
-->


Synchronize took over 48 hours to complete

Hi.
Recently installed Daedalus on Windows 10. Install run ok. When I ran the Wallet for the first time it took over 48 hours to do the synchronize section. My computer was flat out and I couldn't run anything else for that period.  I can provide logs etc if you specify where they are and which ones you need.
Many thanks
Carl

Hi @carlwhiston,
Syncing time depends on many factors such as:

Internet connection speed
hard drive speed
CPU and memory

Usually, syncing should be complete within 3-4 hours. Having spent 48 hours is definitely unexpected. This process involves downloading and processing large number of files which results in a lot of I/O operations and could be impacted by Windows Defender or antivirus software.
Regards,
N.

@carlwhiston please download the latest Daedalus Mainnet 1.0.0 which introduces huge improvements in terms of connecting and syncing performance: https://daedaluswallet.io/en/download/


ARROW-7249: [CI] Release test fails in master due to new arrow-flight Rust crate

@kszucs Can we remove https://github.com/apache/arrow/blob/master/.github/workflows/dev.yml#L56 check to run for each pull request?
(Can we also remove https://github.com/apache/arrow/blob/master/.github/workflows/dev.yml#L92 too?)

@kou The reason why I've removed them from running on each PR to reduce the number of the triggered builds. See the mailing list thread about the queueing concerns. The docs build takes fairly long time, in contrary of the source release test.
@andygrove have you run the source release test? Because the master is still failing.

Sorry. I will look at this as soon as I can but can't get to a computer for
a couple of hours.
On Mon, Nov 25, 2019, 3:39 AM Krisztián Szűcs<EMAIL_ADDRESS>wrote:

@kou https://github.com/kou The reason why I've removed them from
running on each PR to reduce the number of the triggered builds. See the
mailing list thread about the queueing concerns. The docs build takes
fairly long time, in contrary of the source release test.
@andygrove https://github.com/andygrove have you run the source release
test? Because the master is still failing.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/apache/arrow/pull/5889?email_source=notifications&email_token=AAHEBRFDZM65OVKIXO7F6MDQVOTPBA5CNFSM4JRACYOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFB563Y#issuecomment-558096239,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHEBRDMNU6WB2RNHGDYNMLQVOTPBANCNFSM4JRACYOA
.

@kszucs Thanks. The mailing list discussion is<EMAIL_ADDRESS>, right?
I've opened https://issues.apache.org/jira/browse/ARROW-7255 to enable the source release test on pull request.

@kou yes, that is it.


I would like to use CSDeskBand to increase the display size of my cat

https://github.com/dsafa/CSDeskBand#usage
The cat is too small to see clearly.

#93

This won't work on Windows 11 unless using ExplorerPatcher. Patching system element is NOT very ideal for every users I think.


Update license field to correctly identify MIT

Description of the change
Update license field to correctly identify MIT
Type of change

[x] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues
https://pinwheel.atlassian.net/browse/PP-5448
#42
Checklists
Development

[x] Lint rules pass locally
[ ] The code changed/added as part of this pull request has been covered with tests
[ ] All tests related to the changed code pass in development

Code review

[x]  This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] "Ready for review" label attached to the PR and reviewers mentioned in a comment
[ ] Changes have been reviewed by at least one other engineer
[ ] Issue from task tracker has a link to this pull request

cc @jrodbx


Handle deleted discord messages and channels

Some jerks out there keep deleting discord messages or channels that contain SpellBot games in them for some reason. This eventually causes 404 API errors when the bot tries to check up on those games.
So I had to implement some code to watch for these delete events and handle them by deleting their associated games.

Codecov Report

Merging #470 (a042180) into master (60fd6f0) will decrease coverage by 0.71%.
The diff coverage is 7.40%.

@@            Coverage Diff             @@
##           master     #470      +/-   ##
==========================================
- Coverage   96.48%   95.77%   -0.72%
==========================================
  Files          68       68
  Lines        3331     3358      +27
  Branches      465      473       +8
==========================================
+ Hits         3214     3216       +2
- Misses         70       95      +25
  Partials       47       47

Impacted Files
Coverage Δ

src/spellbot/__init__.py
93.20% <7.40%> (-1.28%)
: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 60fd6f0...a042180. Read the comment docs.


Limit page height to 2x screen height

Pagination of table items should start by default when the table is two times the height of the screen (as in Gmail).

Pagination might not be possible.  This was explored during the app refresh.  It is due to how etcd the underlying database works on a cluster.
@luizbafilho I think you helped work on that and we backtracked from it right?  I don't want to go down the same path again.

Pagination is possible, but not in the way we are used to.
For instance, we cannot have page numbers below the table, it will need to be a "Next" button at max. If we decide to enable a "Previous" button, we'll need to cache the data in memory on the page when we fetch more items, that way we can "go back" to the previous page, cause in k8s you can't specify a "page" you wanna fetch, you can only ask more data with a limited number of items.

There is also the issue of sorting/ordering too, right @luizbafilho?  If we were to have this pagination even as you describe above then sorting kind of becomes impossible given how the data is cached, etc.

Yes, sorting on k8s api is not doable at all. Any kind of sorting can only be done on the client side after that data has been fetched.

I'm closing this for now since we are not prioritizing pagination anytime soon.

We're seeing customer performance issues because we're not paginating results.
We can't have "jump to page X" but we can have next and previous buttons.
When you query a List of resources, you do get an approximate number of items remaining.

There's things we can do, but they require that the UI is built around it.
Our current query pattern is:

For each namespace:

For each cluster:

List

Aggregate and return results

So we issue O(N*M) List requests, and sort and filter on the frontend. If we limit each request, we'd have N*M different approximate numbers remaining.
If we were to push down the filters into the backend we wouldn't have to return data that's been filtered out. Then, if we didn't need to sort, we could just stop querying when we'd reached e.g. 25 results, and return a token that represents where we were so we can request next.
But a very large part of our UI is built around sorting. I know most uses of sorting are actually an attempt to use filtering (e.g. nobody wants to sort by status, they want to find objects with non-OK status; nobody wants to sort by name, they want to find a name they either know or vaguely know) but I think we'd need to make the filtering better before we can remove sorting.

🤔 ok.  This does need to be fixed in general.  The last time this was done by an engineer it didn't go well hence my hesitation.
@ozamosi let's set up some time on finding a path forward for this.  @bigkevmcd, is it safe to assume that the work we do here in OSS will makes its way to Enterprise?

Also see https://github.com/weaveworks/weave-gitops-enterprise/issues/1308 for the customer performance issue that triggered these comments.

Multi-cluster paginated results are pretty complex.
For example, if you have a page size of 20, and one cluster...you get 20 results from the one cluster, all's fine.
If you have two clusters, do you get 10 from each, and all's fine?
If you have 20 clusters, do you get 1 from each?
If you have 21 clusters...what do you do?


Using square brackets in get_ports() results in a segmentation fault.

With this code, python just crashes:
import jack
client = jack.Client('test')
print(client.get_ports('[]'))

I also tried different combinations (single bracket, adding text before, etc). The segfault happens on both Python 2.7.12 and 3.4.5.

Thanks for the report!
This seems to be a bug in JACK which was reported to the mailing list in February: http://lists.jackaudio.org/pipermail/jack-devel-jackaudio.org/2019-February/001694.html (link may be for list members only?).
Anyway, this was transferred to a Github issue (https://github.com/jackaudio/jack2/issues/433) and the bug was promptly fixed (https://github.com/jackaudio/jack2/commit/d4ef6800b4fe9ee61d0b9a1b76af1292023a40e7).
There has been no new JACK2 release since then, so you'll have to compile it yourself to get the fixed version.
The same issue is not yet fixed in JACK1: https://github.com/jackaudio/jack1/issues/85

Ok, if I understand it correctly, the fix just checks if the regex pattern is valid, right?
For the time being (since I'm still using jack1), I think I'll override the get_ports method like this:
import re
import jack

def _get_ports_fix(self, name_pattern='', is_audio=False, is_midi=False,
                   is_input=False, is_output=False, is_physical=False,
                   can_monitor=False, is_terminal=False):
    if name_pattern:
        try:
            re.compile(name_pattern)
        except:
            return []
    return jack.Client._get_ports(self, name_pattern, is_audio, is_midi,
                                  is_input, is_output, is_physical,
                                  can_monitor, is_terminal)

jack.Client._get_ports = jack.Client.get_ports
jack.Client.get_ports = _get_ports_fix

Thank you for your answer.
Let me know If you have a better solution.

@MaurizioB Yes, I think in the meantime this is a reasonable work-around.
There might be some edge cases though, because I assume the two regex engines may have slightly different rules.

Not sure if this is helpful, but I have a trace of this thanks to faulthandler:
Current thread 0x00007f7b04ff9700 (most recent call first):
File "/usr/lib/python3/dist-packages/jack.py", line 315 in owns
File "/usr/lib/python3/dist-packages/jack.py", line 1377 in _wrap_port_ptr
File "/usr/lib/python3/dist-packages/jack.py", line 1265 in get_port_by_name
File "/usr/lib/python3/dist-packages/jack.py", line 1361 in _port_list_from_pointers
File "/usr/lib/python3/dist-packages/jack.py", line 1329 in get_ports
File "Event_Carnyx_WebUI Backend", line 4 in _event_action
File "/home/daniel/Projects/KaithemAutomation/kaithem/src/newevt.py", line 590 in _do_action
File "/home/daniel/Projects/KaithemAutomation/kaithem/src/newevt.py", line 440 in _on_trigger
File "/home/daniel/Projects/KaithemAutomation/kaithem/src/newevt.py", line 861 in _check
File "/home/daniel/Projects/KaithemAutomation/kaithem/src/newevt.py", line 547 in check
File "/home/daniel/Projects/KaithemAutomation/kaithem/src/newevt.py", line 195 in _run
File "/home/daniel/Projects/KaithemAutomation/kaithem/src/workers.py", line 92 in workerloop
File "/usr/lib/python3.6/threading.py", line 864 in run
File "/usr/lib/python3.6/threading.py", line 916 in _bootstrap_inner
File "/usr/lib/python3.6/threading.py", line 884 in _bootstrap


Unified suggestions for use, create, update, delete flags (id and name)

Feature or problem description
We have number of commands now that provide set of flags: --name, --id. We can easily provide shell extensions for those flags (as we do in some of the commands)

Can you explain a bit more what you mean by "suggestions" - dynamic completions?

When typing --id= and pressing tab we want to have command line suggestions that are common with filesystem items:

Thanks, that is what I thought, just confirming.


Failed to load resource: the server responded with a status of 404 (NOT FOUND)

superset==0.16.0
Failed to load resource: the server responded with a status of 404 (NOT FOUND)

The path of file should not be the problem...

How did you install it? Looks like static files were not build.

Yep, it could be linked to https://github.com/airbnb/superset/issues/2187

i use
pip install 0.16.1.tar.gz
found the same error.

@asdf2014 ,I have the same problem as you ! have you solved the problem ?

@aurora5161 Not yet. We decide use the superset that version is 0.15.4 in product environment. Newest version 0.16.0 superset's test coverage is too low to use it as formal product. I think if you are not very anxious, you can wait a while for a more stable version of the release. :-)

@xrmx Why do you close this issues, is it already resolved? Please link the involved PR.

@asdf2014 ,thank you very much for your advice,the version 0.15.4 can solve the 404 problem!

@xrmx But why is superset v0.16.0 re-importing this question? Same situation like Gravatar issues (www.gravatar.com), do you know if you have noticed it in superset v0.17.0.

0.17.5 still doesn't fix it.
I uninstall superset 0.17.5 and install 0.16 and it works.

@buptljy can you describe your issue? Somehow your server isn't serving the JavaScript files?

First I installed superset-0.17.5 by following  the official site. But  when I opened http://localhost:8088 and I got exactly the same view as the screenshot shown above, which was a 404 error and no resources file(including css and js files). So I unistalled superset and turned to superset-0.16 and everything went well.

Notice: this issue has been closed because it has been inactive for 364 days. Feel free to comment and request for this issue to be reopened.


speed up blender with L40S

at roughly the same cost (and still under a dollar), run the blender rendering in ~3 minutes instead of ~5 with the new L40S GPUs

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-2e3c8f5.modal.run

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-ae516e4.modal.run


Mff fix

This should fix the reader for the kind of files that I have.

Wait, some other commits are here.
Sorry.


Improvement: Remove vote by '-' icon

Description

issue: #2913
A '-' icon to appear when you hover over the "remove vote" option on a note.
Changelog

Frontend

SVG icon for remove vote.
Additional prop and changed type for DotButton.
state to detect mouse hover.

Checklist

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] The light- and dark-theme are both supported and tested
[x] The design was implemented and is responsive for all devices and screen sizes
[x] The application was tested in the most commonly used browsers (e.g. Chrome, Firefox, Safari)

(Optional) Visual Changes

Before hover

On hover over remove

I think this could be handled using css only
Also, the minus sign is slightly wider than the plus

Thank you for your PR @Quasilius-Starlord! Are you planning on implementing the suggestions made by @blxckOxymoron? In my personal opinion, doing it your way with an svg is fine, but yeah, it would be good if it would be the same width as the + sign.

Yeah, I'll be implementing as suggested.

After I made my new changes the UI looks like following:


Barcode is not scanning

Hi,
We are using capacitor plugin for barcode scanning. We are able to scan qrcodes but barcodes are not getting scanned. QRcodes are working all OK. Any hint will be really appreciated. Thanks in advance
scanIT() {
async checkPermission() {
return new Promise(async (resolve, reject) => {
const status = await BarcodeScanner.checkPermission({ force: true });
if (status.granted) {
resolve(true);
} else if (status.denied) {
alert('Give access to camera');
} else {
resolve(false);
}
});
}
async stopScan(){
BarcodeScanner.stopScan();
}
async prepareScan(){
await BarcodeScanner.prepare({targetedFormats:['EAN_13']})
}
async onScan() {
await this.checkPermission();
await BarcodeScanner.hideBackground();
document.querySelector('body').classList.add('scanner-active');
const result = await BarcodeScanner.startScan({cameraDirection: "back" });
console.log('After Start Scan ' + result.content);
await BarcodeScanner.showBackground  ();
document.querySelector('body').classList.remove('scanner-active');
}

hi,
thanks for the prompt reply. Do you think using ml-kit may work, I can try integrating this(not sure how far can I go). Any leads from you?

Hi @kumarjitendra-epic you are more than welcome to make your own fork and clone it to your local machine. Then you can checkout the branch and test by installing the plugin locally, see the Capacitor guide for more info how to do that. https://capacitorjs.com/docs/plugins/workflow#local-testing

I would assume for 1D barcodse then we'd need to use the phonegap plugin until there is a better capacitor version?

@levarberry I think the ml-kit branch is already good for use, next week we are releasing an app that uses that branch. Workflow is to clone this repository, switch to the ml-kit branch, install the plugin locally (see Capacitor docs how to do that).
Do note that the API changed.

@kumarjitendra-epic unfortunately not, due to unforeseen issues in my own project I had no time this week.
Hopefully I can find time next week to finish things up.

Also, I am trying to use cameradirection as back, it works fine when run in browser but as soon as I use the apk on mobile, it stops working

@JanMisker , by any chance the new plugin has been released that you are discussing about?

you'd have to clone this repository locally and switch to the ml-kit branch.


Constant Crashing When Loading Project due to Missing "item.msgbnd.dcx" File

Greetings,
Smithbox v<IP_ADDRESS> is constantly crashing when loading recent projects.
The crashlog is as follow:
Log 2024-7-03--17-36-37.txt
If I'm not mistaken, the file "item.msgbnd.dcx" is one of the Elden Ring assets that is packed within the game files and must be extracted first. I'm using the "Low Requirement" version of Smithbox since I have neither intention nor interest in modding any of the game assets. I only use it for modifying or increasing graphics/visual fidelity in regulation.bin file.
The last Smithbox I used, v<IP_ADDRESS>, didn't have this issue.

Oh sorry, I've found the issue. The next release will fix this and remove the requirement for the msgbnds to be unpacked.


#80 added an identity reader for the @readers annotation

fixes #80
Review @jcranky

Codecov Report

Merging #83 into master will increase coverage by 0.09%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
+ Coverage   88.88%   88.98%   +0.09%
==========================================
  Files           7        8       +1
  Lines         117      118       +1
  Branches        5        4       -1
==========================================
+ Hits          104      105       +1
  Misses         13       13

Impacted Files
Coverage Δ

...cala/org/zalando/grafter/macros/ReadersMacro.scala
100% <100%> (ø)

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 5b88d96...a1b93cc. Read the comment docs.

:+1:

👍


CVE-2022-36033: Bump version for "jsoup" plugin

Expected Behavior
gradle installs the jsoup-1.15.3.jar plugin. The plugin version should be >= 1.15.3 to patch the vulnerability.
Current Behavior

gradle v7.6 installs the jsoup-1.15.1.jar plugin. According to https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-36033, the vulnerability is patched in jsoup-1.15.3.jar.
ℹ  Looks like the version needs to be bumped in this file 👉  https://github.com/gradle/gradle/blob/e311794a0f75ebec3c1eec83e35b66161ff81435/build-logic/build-platform/build.gradle.kts#L69

Thank you for providing a valid reproducer.
The issue is in the backlog of the relevant team and prioritized by them.

Thanks for the report, this will be fixed in Gradle 8.0


Unable to change source branch

I recently changed the default branch for my project, and would like to update this in the Static Web App as well. This appears to be easily done in the GitHub Action YAML however I couldn't find anything on the Azure Portal side.
Will I need to delete and recreate it, or change the default branch back to a name that matches the Static Web App? Or is there an option somewhere to update that source branch?
Thanks in advance, and also Static Web Apps is great!

Right now delete and recreate is the only way to go, we are working on improving the flexibility around this! We appreciate your feedback and thanks for trying out the product.

OK, and thanks for getting back to me so fast.
Feel free to keep this issue open if it helps for tracking, otherwise I'll keep an eye out for this in the future!

Just as an update, for newly created apps this is fixed, with the exception that the UI will still link to the old branch.

How can we do this for existing apps?

You can reset your api key and update the GH repo secret with the new key. Then you are free to change branches. You can take a look at this issue for instructions on resetting/getting your key:
https://github.com/Azure/static-web-apps/issues/193