intelliSenseOptions and additionalCompilerArgs documentation is completely unclear

The documentation on the cmake presets json mentions the possible use of intelliSenseOptions and additionalCompilerArgs to feed intellisense information (supposedly about include paths and defines?) But it is completely unclear how this works. It is also unclear if it is assumed that linux/mac/windows and clang is expected for this to work at all.
I'm currently attempting to use emscripten (clang-based) from Visual Studio and I have no idea how to configure these fields to give intellisense the information that it needs (mostly just include search paths).
There are no examples, no context. It plainly states which "additional arguments" can be given, and if it should be the defaults? What does that mean? Arguments to what? The compiler, or Intellisense itself? What if the compiler provides no such functionality? Can I also just provide paths to intellisense myself? There is no mention of this.

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

ID: 64d317ae-6e03-8bb6-2b2c-4d1b9ecf3080
Version Independent ID: cab1289c-b480-e674-106f-f37d3a3f198f
Content: CMakeUserPresets.json
Content Source: docs/build/cmake-presets-json-reference.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

@robindegen
Sorry you're not finding the information you're looking for. As far as I can tell, the additionalCompilerArgs value allows arbitrary compiler arguments. There's no schema for those, and the values depend on the platform and compiler in use. If you're using cl.exe, presumably you could specify /I or /D arguments to specify specific include paths and preprocessor macros.  @sinemakinci1 may be able to elaborate further, or correct me if I'm wrong.

@corob-msft Thank you for your response. I think this already kinda indicates how unclear it is. Because according to this blog post it's something completely different:
https://devblogs.microsoft.com/cppblog/configure-intellisense-with-cmake-toolchain-files-in-visual-studio-2019-16-9-preview-2/
additionalCompilerArgs can be used to explicitly add arguments to be used when Visual Studio queries the compiler for IntelliSense. useCompilerDefaults can be used to disable the use of the GCC/Clang compiler for IntelliSense configuration (defaults to true). These options apply to Windows, Linux, and WSL configurations.

It mentions that it's for feeding information to Intellisense. Not the compiler itself. This would be done by Cmake I would expect. I can imagine there being a system where Intellisense would query the compiler used to figure out what all the system paths are (sysroot), so that it knows where the stl headers are etc. But if that is true, then there is no information about how it normally queries this information, nor what kind of output it expects from the compiler when this is called. I can imagine most clang/gcc versions will give a rather standardized output; but this is not always the case.

@robindegen
The IntelliSense system in Visual Studio is driven by a separate compiler invoked by the IDE. When IntelliSense runs, there's no "querying of the compiler" taking place. The IntelliSense engine is told the include path and preprocessor definition values directly by the IDE. The IDE normally passes it the same values for the include search paths and preprocessor definitions that it passes to the regular compiler toolchain. The form these parameters take is an internal detail that's not usually exposed. My assumption was that these values would take the same form as whatever you'd pass to your regular toolchain. I might have been wrong about that, given the statement in the useCompilerDefaults section about using gcc-style arguments. So, you'd need to use the -I and -D forms, instead of /I and /D, as a common form which will work for cl, clang, and gcc. Something like -isystem won't work with cl, but presumably will work with gcc and clang.
Just to clarify when to use this property: If you want to send the IntelliSense engine different values than your normal toolchain uses, then additionalCompilerArgs is the way to do it. If you want it to use the default values set for your project, you don't need to set anything in additionalCompilerArgs at all.
There are other ways the IntelliSense display can go wrong. If your builds succeed but IntelliSense isn't working, it's probably not because you're setting additionalCompilerArgs incorrectly. For example, if your toolchain references environment variables such as CPLUS_INCLUDE_PATH that aren't set as part of the CMake project configuration, you might find they also aren't picked up by the IntelliSense engine.

The documentation on the cmake presets json mentions the possible use of intelliSenseOptions and additionalCompilerArgs to feed intellisense information (supposedly about include paths and defines?) But it is completely unclear how this works. It is also unclear if it is assumed that linux/mac/windows and clang is expected for this to work at all.
I'm currently attempting to use emscripten (clang-based) from Visual Studio and I have no idea how to configure these fields to give intellisense the information that it needs (mostly just include search paths).
There are no examples, no context. It plainly states which "additional arguments" can be given, and if it should be the defaults? What does that mean? Arguments to what? The compiler, or Intellisense itself? What if the compiler provides no such functionality? Can I also just provide paths to intellisense myself? There is no mention of this.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 64d317ae-6e03-8bb6-2b2c-4d1b9ecf3080
Version Independent ID: cab1289c-b480-e674-106f-f37d3a3f198f
Content: CMakeUserPresets.json
Content Source: docs/build/cmake-presets-json-reference.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

Were you ever able to get this working? I'm facing the same issues. I've dumped the cflags using em++ --cflags and have configured the settings as follows, doesn't seem to actually do anything:
"microsoft.com/VisualStudioSettings/CMake/1.0": {
  "intelliSenseMode": "windows-clang-x64",
  "intelliSenseOptions": {
    "useCompilerDefaults": false,
    "additionalCompilerArgs": [
      "-target",
      "wasm32-unknown-emscripten",
      "-fignore-exceptions",
      "-fvisibility=default",
      "-mllvm",
      "-combiner-global-alias-analysis=false",
      "-mllvm",
      "-enable-emscripten-sjlj",
      "-mllvm",
      "-disable-lsr",
      "--sysroot=<path/to/upstream/emscripten/cache/sysroot>",
      "-DEMSCRIPTEN",
      "-Xclang",
      "-iwithsysroot/include/fakesdl",
      "-Xclang",
      "-iwithsysroot/include/compat"
    ]
  }
}

I never did. The fact that Microsoft decided to have a product called "Visual Studio Code" with a json file that has the exact same name as the regular Visual Studio one, but in a slightly different format; REALLY isn't helping all this. Googling for issues in regular Visual Studio has become a nightmare. Everything is poluted with tons of people proposing vscode solutions.

I never did. The fact that Microsoft decided to have a product called "Visual Studio Code" with a json file that has the exact same name as the regular Visual Studio one, but in a slightly different format; REALLY isn't helping all this. Googling for issues in regular Visual Studio has become a nightmare. Everything is polluted with tons of people proposing vscode solutions.

I agree with the sentiment. The issue seems to be the --sysroot= flag, using -isystem or even -isysroot it works. Same  when using clangd.


aap-sfizz needs changes for Android

aap-sfizz is in master now, but it lacks file loading support...kind of. It should load from assets instead of local files.
Since there is no UI to specify local sfz files and convert it into Atom sequence, it needs some extra steps to make it fully usable.
Possible solutions:

(1) support only a set of particular sfz files, load at instantiation time. sfz loader from assets is still needed.
(2) add UI support , perhaps using https://github.com/atsushieno/aria2web/ instead (but it does not support sfz).

This changeset I made to fluidsynth is a reference change. https://github.com/FluidSynth/fluidsynth/commit/daa037b

It needs Worker extension implementation (as host).

Worker is implemented in af1ac0b. Now sfizz does not crash, but there seems to be some problem loading sfz:

2020-07-28 23:04:43.236 18416-18553/org.androidaudioplugin.aap_sfizz I/AAPHostNative: [sfizz] Number of voices changed to: -1089748792

It was just due to lame logging, not an actual issue. Needs more investigation on how it results in no sound.

There is an ongoing work in sfizz-assets branch. It successfully loads sfz file itself so far.
*.sfz files are just metadata that further link to other local file resources, which seem to be loaded by other code paths. Those loaders also have to be replaced.

SndfileHandle sndFile(file.string().c_str());

We will have to add support for Android asset to libsndfile.

I'm changing my mind; there are too many things that depends on local filesystems now: lbisndfile, sfizz/src/external/ghc filesystem, sfizz itself. It is now much easier if aap-sfizz just downloads sfz files from web and stores it in the app storage.

Aaand sfizz has been changing the way how it uses SndFileHandle, which now aggregates most of the file loading part as createExplicitAudioReader() which is very comfortable for me to make necessary changes.

Converting filepath to AAsset fd via AAsset_openFileDescriptor() and passing it as fd to SndfileHandle.ctor() does not seem to work... it then fails to initialize sfinfo in the SndfileHandle and fails at getFileInformation() (if (channels != 1 && channels != 2) evaluates to false).

Back to this issue. Now FilePool.cpp needs some changes.

As a side step, I dumped assets effort so far and got sfizz working from aaphostsample (as of fad0ace). We need resource path settings in sfizz internals though.

These days sfizz port has been working with "copy sfz assets into local app storage (directory)" strategy. It coulod be an enhancement to existing app, but so far we don't "need" changes for Android anymore. Also, sfizz made libsndfile dependency optional, so chances are that we may be able to implement asset-based stream resolver.
aap-sfizz has moved to aap-lv2-sfizz repository, so I'm closing this anyways.


Bug - Cant install and load bash

using windows 10 enterprise insider preview
build 14366.rs1_release/160610-1700
running bash from start shows windows quickly opening and closing. launching bash from CMD prompt results in error
Cannot launch bash because the LX subsystem has an install, uninstall, or servicing operation pending.
restarting system and removing and adding said feature from windows features control panel doesn't fix issue.

Could you share the contents of the [HKEY_CURRENT_USER\SOFTWARE\Microsoft\Windows\CurrentVersion\Lxss] registry key with me?
After doing that could you run the following from an admin command prompt?

sc stop lxssmanager
bash.exe

ddarvish:
Are you running with Legacy Console Enabled by any chance?
I just fought with an issue mimicking your symptoms (running bash shows window opening but immediately closing, as well as some initial install difficulty) that was caused as I switched over to legacy console mode periodically as required for an Atom plugin.
Open a Command Prompt session, right click the menu bar and select Properties, Options to check.
This fixed my issue of Bash refusing to open, and I suspect that this was also behind the initial difficulties I had installing Bash - the problem seemed to 'fix itself', so I had previously attributed it to an Insider Preview update; but now I suspect it was me adjusting the Command Prompt legacy settings.

I was facing a similar issue. I deleted the registry entry for lxsss. After that I stopped the service and ran bash again. I was able to install at this point. (I had at one time played with bash before removing the feature from Windows again. I'd guess this is where I got bit).

actually i went ahead and just ended task on lxsss and the install went
through. this was a fresh install on windows so im sure it was a bug.
anyway all is well and good. if anyone wants to possibly assist me in
trying to install get get working GmusicFS (
https://github.com/EnigmaCurry/GMusicFS) which is based on the fuse file
system i think that would be great =)
Davis Darvish
On Fri, Jul 15, 2016 at 7:23 AM, Justin Vanderhooft <
<EMAIL_ADDRESS>wrote:

I was facing a similar issue. I deleted the registry entry for lxsss.
After that I stopped the service and ran bash again. I was able to install
at this point. (I had at one time played with bash before removing the
feature from Windows again. I'd guess this is where I got bit).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BashOnWindows/issues/591#issuecomment-232964615,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABerV5G-7N9so1MW2uExhrZBJzrxW7P0ks5qV5fjgaJpZM4I_kUy
.

@indoorowls Thanks! This helped me to install bash.

lxrun /uninstall /full /y
lxrun /install /y

thx Bluscream
this helped me a lot.

ty Bluscream!

np guys, calm it were just two commands thanks :D

@benhillis I was using the Ubuntu bash successfully. Since the past 2 days, I am getting an error 0x8007001f. The Desktop shortcut terminates immediately. The error occurs when I try to execute bash in command line interface. The lxssmanager status is as follows
SERVICE_NAME: lxssmanager
TYPE               : 20  WIN32_SHARE_PROCESS
STATE              : 4  RUNNING
(STOPPABLE, NOT_PAUSABLE, ACCEPTS_PRESHUTDOWN)
WIN32_EXIT_CODE    : 0  (0x0)
SERVICE_EXIT_CODE  : 0  (0x0)
CHECKPOINT         : 0x0
WAIT_HINT          : 0x0

Thanks Bluscream!

@praveengambhir16 - Sorry to hear about that. Can you gather traces using these instructions. And, share the traces out so that we can see where things are going wrong?

lxrun /uninstall /full /y
lxrun /install /y

But between these two remove lxss folder

This was it Bluscream!!

Thank you all for thanking me <3 i feel honored :3

Thanks @Bluscream !!

Fixed with the solution @Bluscream mentioned. Tried to delete the registry entries and stop the service as mentioned by @benhillis and @vanstinator  (I had previously manually deleted the LXSS folder) and it didn't work. Previously I had tried lxrun /uninstall /full /y and it had not worked, so it may have been a combination of the two methods that fixed the problem.


⚠ Bofrost.at has degraded performance

In b0dd3c8, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1300 ms

Resolved: Bofrost.at performance has improved in 5883923.


Fix the emoji picker does not close when editing message is deleted

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/22214
PROPOSAL: https://github.com/Expensify/App/issues/22214#issuecomment-1625230204
Tests

Login with userA’s account
Click FAB => Go to the “New room” section
Create a public room with userA workspace
Go to the room details
Share code > Copy URL to clipboard
Open other browser and login with userB’s account
Join the room by pasting room URL
Send a message in room from userB
Click on “Edit comment” message after sent
Open emoji picker
Switch back to UserA's account and delete the message sent by UserB
Verify that The emoji picker should close when UserA deletes the message that UserB is editing

[x] Verify that no errors appear in the JS console

Offline tests

Could not test when network is offline
QA Steps

Same as Tests

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/9639873/f8401476-cca1-4570-ade6-20966ce7d6d5

Mobile Web - Chrome

https://github.com/Expensify/App/assets/9639873/f869b66f-a58e-4b57-a9f9-0537fbf7da26

Mobile Web - Safari

https://github.com/Expensify/App/assets/9639873/3b8dc14e-7d5e-4117-a0e3-b2ad57b92112

Desktop

https://github.com/Expensify/App/assets/9639873/6e6608d8-f88c-4a4d-ba48-ca1560944004

iOS

https://github.com/Expensify/App/assets/9639873/78da007c-742e-4d4f-b2c5-b5b3ae5a72c3

Android

https://github.com/Expensify/App/assets/9639873/5a693d03-31cb-45da-9c1a-a8bd113fbac8

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/marcaaron in version: 1.3.44-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.45-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Fix for 8995 - Number of times summarizer refreshes snapshot is too high

Create a separate expiration for the summarizer first run.
Note: I'm not sure if this is the best direction but I was thinking we might even propagate this timeout from the SummaryConfiguration through the hostPolicy ?? Or not ?
I did the initial tests and it seems to work fine - will run the ci:test  but wanted to get feedbacks.

Can we add tests that validate this? @tyler-cai-microsoft I don't remember how you tested the cache expiry for the GC work. Are there unit tests / end-to-end tests that can be written for this?

Can we add tests that validate this? @tyler-cai-microsoft I don't remember how you tested the cache expiry for the GC work. Are there unit tests / end-to-end tests that can be written for this?

I made changes to the getVersions.spec.ts file in the tests folder.


Fix how multi-part videos add the .nfo file for compatibility with Emby/Jellyfin

Issue: Emby/Jellyfin are unable to parse the .nfo file for multi-part videos due to how the .nfo is recognized. The .nfo requires at minimum the same name as the -pt1 video to be recognized.

Possible fixes:

Allow user to add a .nfo for each video file
Set the nfo file to be identical to the name of the first part of multi-part videos


Reload NGINX after rolling back NGINX configuration changes

Proposed changes
If an error is detected after a NGINX reload during a config apply, then we need to rollback the config changes and reload NGINX again to revert the changes.
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[ ] I have read the CONTRIBUTING document
[ ] I have run make install-tools and have attached any dependency changes to this pull request
[ ] If applicable, I have added tests that prove my fix is effective or that my feature works
[ ] If applicable, I have checked that any relevant tests pass after adding my changes
[ ] If applicable, I have updated any relevant documentation (README.md)
[ ] If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

Codecov Report
Patch coverage: 40.90% and project coverage change: -0.03% :warning:

Comparison is base (26eb063) 67.60% compared to head (f28271d) 67.57%.
Report is 7 commits behind head on main.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #419      +/-   ##
==========================================
- Coverage   67.60%   67.57%   -0.03%
==========================================
  Files         113      113
  Lines       12766    12764       -2
==========================================
- Hits         8630     8625       -5
- Misses       3566     3570       +4
+ Partials      570      569       -1

Files Changed
Coverage Δ

...dor/github.com/nginx/agent/v2/src/plugins/nginx.go
71.89% <40.90%> (-0.79%)
:arrow_down:

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[Hub Generated] Review request for Microsoft.Security to add version preview/2023-05-01-preview

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki. Note that this doesn't apply if you are trying to merge a PR that was previously in the private repository.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

@talco773, please fix the [must fix] lint error.

@talco773, please fix the [must fix] lint error.

@ms-zhenhua some of the failures seems like false alarms:
NoDuplicatePathsForScopeParameter:
Why these two scopes are duplications?
'/{scope}/providers/Microsoft.Security/customRecommendations/{customRecommendationName}' is a duplicate of path '/{scope}/providers/Microsoft.Security/customRecommendations' that has the scope parameter.'???
One is list and one is get single
DeleteOperationAsyncResponseValidation:
The link doesn't work...
TopLevelResourcesListBySubscription:
All the resources I added have the following path:
/{scope}/providers/Microsoft.Security/ResourceType
and the 'scope' can be subscription

@ms-zhenhua you didn't add any comment. What else should be fixed?

@talco773 I left the following comment. Kindly help confirm it. Thanks.
https://github.com/Azure/azure-rest-api-specs/pull/24090#discussion_r1223788132

@talco773 I left the following comment. Kindly help confirm it. Thanks. #24090 (comment)

I replied the relevant comment

@talco773 I left the following comment. Kindly help confirm it. Thanks. #24090 (comment)

I replied the relevant comment

Please see the latest respond on the comment, thanks

/azp run

this js sdk breaking may be caused by changing default tag from composite-v3 to a new apiversion

Please merge latest main after https://github.com/Azure/azure-rest-api-specs/pull/24457 get merged. and then rerun the pipeline.

What is left to do before the merge? I already got signoff

Hi @talco773 , any specific reason why added "WaitForARMFeedback" label again, it was already ARM SignedOff. Just putting the label again.

/azp run


[Bug]: Push doesnt work on mobile

Describe the bug
So I had to copy my repository onto my phone by hand (via USB on my PC).
The problem is that when I try to push my changes to Github, it says "MissingParameterError: The function requires a 'remote or URL''"
I have tried:

creating a new remote on my phone
switching to remote branch
appending '.git' to the repo URL

Relevant errors (if available) from notifications or console (CTRL+SHIFT+I)
MissingParameterError: The function requires a 'remote or URL'
Steps to reproduce
I do not have the hardware to test this on other devices

Set up obsidian git on desktop
Copy it over to your android device and open it in obsidian
set up authentication to github (Name and Token)
commit all changes
push

Expected Behavior
No response
Addition context
No response
Operating system
Android
Installation Method
None
Plugin version
2.22.0

@MakiRollDev Did you find any workaround?

@kaisersakhi No sadly not I just switched to Google drive

Same error for me in my  Android Phone
MissingParameterError : The Function requires a "remote OR url" parameter . But none was provided

same problem. I would like to give some logs or more info, but don't know how

no, i no idea how to open dev tools from android. ok, android 11 rkq1.200826.002, MIUI 12.5.6, pull works correctly. field "username on your git server does no effect, i changed it to completly incorrect, and all worked after reboot of the phone. Also, the new token i generated in github was never used. There are files that were modifed after last commin inside .git of the phone, some /objects, index, /refs/heads and so on
Also I add .git/config files from windows and android. As I see, username and psw fields do nothing because url was not modified after copying from win, but author name and email were changed. Maybe that's okay for git security, i'm bad in it
config-windows.txt
config-android.txt
Now something I was doing: i used this guide to sync my devices

Step 5: Add the folder as a vault in Obsidian
1. Open Obsidian

2. Tap the sidebar icon in the top left corner

3. Click the vault name in the top left corner, click "Manage valuts..."

4. Click "Open folder as vault" and navigate the the folder you copied over

5. Obsidian Git should try to fetch
(or open command pallette by swiping down -> "Obsidian Git: Pull")

6. Enter your username & access token when Obsidian git asks for it
(using your GitHub password might work, but I did not try it)

7. Congratulations, you now have working sync!

At 6. there was an error which was solved by filling the author email field. After that pull worked correctly
and thats why: i added the .git/config after i click "open folder as vault" - there is no email now
config_after Open folder as vault.txt
I tried to change the access key to classic and finegrained tokens, but this made no difference

here is with incorrect github username (github.com/Irvu/main-obsidian-vault.git)

and here with edited token (url = https://github_pat_21... , 1 changed to 2)
Then I regenerated all tokens, saw a menu with login/password or token. I've filled it with my login and token, but as i know, .git/config rewrites global config. But that's not right in this case, because i broke the .git/config earlier (token is github_pat_1 now)
config broken.txt
i reinstalled the app, but it made no difference, the mistake is still here
then i removed config file from .git, and this is new generated
config andr-obs made itself.txt
so, the mistake

or with writing the exact link in the field of loging<EMAIL_ADDRESS>

I couldn't fix the issue from scratch but here is a workaround. For iPhone, I airdropped the whole vault folder. Moved it to iPhone/Obsidian in Files. Then opened it as a new vault. Git repo was set to<EMAIL_ADDRESS>This way it started to work.


[Feature request] Download HTML file

I've found Mailhog to be invaluable and I'm very glad this project exists, thanks for your efforts :)
One task I often perform is to test an email and then provide the design with sample content for others to review. To help with this I sometimes use a screenshot or copy/paste the HTML from the relevant tab but it would be great to be able to simply download an .html file.

I'm sure I could add a few more download options such as downloading the email HTML or text body (if set) :+1:  I am currently working on some other things in Mailpit, so I'll leave this open and hopefully be able to implement this within the next few days. Thanks for the suggestion.

Thanks that's great, looking forward to it.

Thank you again for the suggestion. This feature has been added and released as part of v1.5.0. When viewing an email, click the "Download" button and select "HTML body". Please can you confirm this works for you as expected?

I just updated and it works perfectly. Thanks very much for implementing it!


Add fibonacci formula

Added code to calculate fibonacci series from a given number using a recursive approach and binet's formula

Hello @darwinz, I have committed changes as you requested, Please Review it.


Check if redisgraph is deployed before running tests

Deployment of RedisGraph will be disabled by default upstream.
This change enables the previous behavior before continuing with the tests.
Issue: https://github.com/open-cluster-management/backlog/issues/10931

/hold


Update nuttx submodule

There are still some nuttx errors so need to pick those commits to the submodule update. This was a bit more work than I anticipated.


Rebuilding with cache changes navigation appearance

At least with the current release version, rebuild the documentation with an existing public folder results in a change in the navigation on the left side. After rebuilding, a line break is added after the > symbols:

Thanks for filing this case. At the moment I have no idea why that would make a difference, but it's likely that this still is an issue with version 1.15.0. I'll look into it as soon as I can.

Ok, I didn't think about that. During development, I basically always rebuild everything with the -a flag. That's why I never see this.
The "collapsible" navigation links are added as a post-build modification to the finished HTML. So without the -a flag, the HTML is already there and effectively modified twice.
43,44c43,44
+

Glossary

---
-

Glossary

An easy fix would be to check, if the "nav-link" is already there. The real solution though would be to get rid of this post-build modification step, and add this when the toctree is built.

Isn't it possible to add those arrows via CSS?

Adding these things with CSS has the limitation that you can't click on the link and/or arrow separately. If I would add them via link::before, I can't expand the navigation tree separately from clicking a link. Pseudoelements can't receive focus separately.

Should be fixed (with the workaround mentioned above) in 1.15.1. To get rid of the post-processing requires me to understand how Sphinx renders the toctree, which I haven't looked at yet.


fix: Update README.md

Pull Request
Description
Fix typos in README file.
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Code cleanup/refactoring
[X] Documentation update
[ ] This change requires a documentation update
[ ] CI system update
[ ] Test Coverage update

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.89%. Comparing base (b90cc88) to head (7d505b8).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #606   +/-   ##
=========================================
  Coverage     93.89%   93.89%
  Complexity      611      611
=========================================
  Files            51       51
  Lines          2112     2112
  Branches        244      244
=========================================
  Hits           1983     1983
  Misses           57       57
  Partials         72       72

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Use YAML front-matter instead of markdown metadata extension

I'd suggest using a front-matter like system such as the one that Jekyll has that parses from YAML (http://jekyllrb.com/docs/frontmatter/) to make the metadata more flexible.
With YAML you get support for lists, dictionaries, and whatnot which would be very useful in templates compared to plain strings.

There is now a plugin system, we can still discuss this if someone wants to work on it.


Solution Extension examples

The SDK on GitHub has a broken link to solution examples. Are there any? I followed the ones in this documentation for solutions and still only ended up with a tool extension. Or was that my mistake somewhere?

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

ID: ac61f326-7fa7-15c7-8b0a-67e50d100dfb
Version Independent ID: fd98eee8-3b3c-65d6-c704-b84b6c1eefac
Content: Develop a solution extension
Content Source: WindowsServerDocs/manage/windows-admin-center/extend/develop-solution.md
Product: windows-server-threshold
GitHub Login: @nwashburn-ms
Microsoft Alias: niwashbu

OK, it was my fault. I thought this was going to add a new connection but it adds my solution in main window in drop down alongside the "Windows admin center". Not very intuitive and they don't really look like tabs.

@NJArcher thanks for the feedback and for letting us know that you were able to unblock yourself. I will pass your UX feedback on to our designer team.


Add release logging to pipeline

As this is already applied to the pipeline I'm going to merge this now.


backups: stop lowercasing file paths (fixes #16591)

This PR changes hot exit to no longer lowercase the file paths on Windows and macOS. This fixes a data loss issue that can be reproduced via:

open a file and make changes
copy the path of the file and use quick open to open this path, but change the file name to upper case
make changes to both files without saving
restart

=> the changes of one buffer are lost
This fix restores both buffers to where they were before exit.
Also fixes https://github.com/Microsoft/vscode/issues/16591

Yeah, I was also surprised that this seems to work :)


CP-337 Add jspm bundle options for minify and sourceMaps

Description
JSPM now offers the ability to minify bundles and generate sourceMaps when bundling.
This PR adds the option to disable or enable each of these features per-bundle.
Example:
    bundles: {
        app: {
            bundler: 'jspm',
            entry: 'build/src/app',
            output: 'dist/CPDemo.js',
            sfx: true,
            sourceMaps: true,
            minify: true
        }
    }

Notice the sourceMaps and minify options at the bottom.
Testing

When sourceMaps is true, jspm bundle should output the following:

     Building the single-file sfx bundle for build/src/app...
ok   Built into dist/CPDemo.js with source maps, unminified.

When sourceMaps is false, jspm bundle should output the following:

     Building the single-file sfx bundle for build/src/app...
ok   Built into dist/CPDemo.js, unminified.

When minify is true, jspm bundle should output the following:

     Building the single-file sfx bundle for build/src/app...
ok   Built into dist/CPDemo.js, minified.

Tested with CPDemo.
@trentgrover-wf
@evanweible-wf
@jayudey-wf
FYI - @guybedford-wf

	When this pull is merged I will not be able to function
	properly because no ticket is included in the title.

Last updated on Thursday, January 15 10:54 AM CST

it would be nice to be able to switch this on and off depending on what type of build you're doing.  maybe people don't want to emit sourcemaps on dist

This is possible since the config is created in JS.
var argv = require('yargs').argv;

var customizedOptions = {
    bundles: {
        app: {
            bundler: 'jspm',
            entry: 'build/src/app',
            output: 'dist/CPDemo.js',
            sfx: true,
            sourceMaps: true,
            minify: false
        }
    }
};
if (argv._[0] === 'dist') {
    customizedOptions.bundles.app['minify'] = true;
    customizedOptions.bundles.app['sourceMaps'] = false;
}
var wGulp = require('wGulp')(gulp, customizedOptions);

good point
+1

+1

@jayudey-wf @charliekump-wf ready for merge

QA Resource Approval: +10

[ ] Testing instruction
[ ] Dev +1's
[ ] Dev/QA +10 with detail of what was tested
[ ] Unit test created/updated
[ ] All unit tests pass

Merging into master.


AttributeError: 'NoneType' object has no attribute 'groups'

File "/root/userfolder/BoT-SORT/fast_reid/fastreid/data/datasets/mot17.py", line 79, in process_dir
pid, camid = map(int, pattern.search(img_path).groups())
AttributeError: 'NoneType' object has no attribute 'groups'
anyone facing this problem please?

Hi, may you provide more information? Are you trying to train the ReID net?
It seems that img_path is None.

yes I'm trying to train the ReID.

I have generated the MOT17-ReID datasets and set the path but still get the same error? I couldnt get it why

The argument of MOT17 was not connected properly, which resulted in the incorrect creation of the ReID dataset. Delete the previously generated MOT17-ReID and try again.

Why do I report an error when I regenerate again


https://github.com/DSpace/dspace-angular/issues/1306 - Implemented ne…

https://github.com/DSpace/dspace-angular/issues/1306 - Implemented new metadata in the dspace schema to store thumbnail descriptions of collections and communities
References
https://github.com/DSpace/dspace-angular/issues/1306
This change was targeted at https://github.com/DSpace/dspace-angular/issues/1306 which now allows storing in a "dspace" schema metadata a detailed description of collection and community logo thumbnail images (also recognized by screen readers). This change is linked to another made in the frontend with the same ticket name.
Description
The "dspace.thumbnail.description" metadata was added to be able to store thumbnail descriptions
Instructions for reviewers
Use a new database and use the "dspace database migrate" command in the "bin" folder of the installed DSpace or perform a new installation of DSpace with this change applied.
List of changes in this PR:
*The "dspace.thumbnail.description" was created for the schema "dspace" in the file "dspace/config/registries/dspace-types.xml"
Include guidance on how to test or review your PR.
When installing DSpace, simply check whether the "dspace.thumbnail.description" metadata was created correctly on the DSpace metadata registration page, in the "dspace" schema.

We have tested but for us the dspace.thumbnail.description metadata does not show up in the metadata registry page.
Checked out the PR branch.
Started backend with the docker compose up -d command
Entered the dspace container with the docker exec -it dspace /bin/bash command
Ran the /dspace/bin/dspace database migrate command

Connected frontend and backend
Checked the Registries → Metadata page, opened the dspace schema, but the new type does not appear there

To me it looks like he new type does not get created with the migration process. Please let me know if I did something wrong during my testing.

@kanasznagyzoltan I redid the test with an empty bank and the metadata was installed correctly. I suggest you retrace your steps, something may have been left behind. If it still doesn't work, you can follow this step in the official DSpace documentation to manually update the metadata (https://wiki.lyrasis.org/display/DSDOC8x/Upgrading+DSpace#UpgradingDSpace-ManuallyupdatingtheMetadataRegistries).

@DanGastardelli
I have tested using a blank postgres db and now there is the new metadata element in the list:

@DanGastardelli Eu testei usando um banco de dados postgres em branco e agora há o novo elemento de metadados na lista:

No meu caso anterior, eu tinha um banco de dados existente e a migração do banco de dados ./dspace talvez não tenha executado todas as migrações relacionadas ao xml. ( https://wiki.lyrasis.org/display/DSDOC8x/Upgrading+DSpace#UpgradingDSpace-ManuallyupdatingtheMetadataRegistries )

I'm glad you made it!


Chemvend rename

About the PR

rename chemvend
Why / Balance

chemvend is a really boring name tbh, nanodrug also fits with the nanomed and it's used in tg
Changelog

Make sure to take this Changelog template out of the comment block in order for it to show up.
:cl:

tweak: Changed ChemVend's name to NanoDrug

Dude

Dude

What

Isnt like controversal?
Cause chemwend called like that to give a chemicals, not already made drugs.

Isnt like controversal? Cause chemwend called like that to give a chemicals, not already made drugs.

raw elements could technically be considered drugs

Technically calling radium as drug
Or potassium.
Or sugar.

what

what

It will be unintuitive. Peps would think that NanoDrug machine will give them already made drugs.
And prior of having pr that actually making some of reagents poisonos, it will be nonsense.

sounds cooler imo and its in tg so thats my reasoning

Call it nanochem instead

dont see a reason to change this, chemvend is fine

Call it nanochem instead

good idea, done

starting to realise how stupid this pr is
idk why i opened it in the first place tbh


Provide test coverage for multiGPU training with Training operator

Description

Added test to verify multi GPU training using Training operator.
Training is using same model as other e2e tests. As a dataset I use Alpaca, which generates some GPU load (and Twitter dataset doesn't work for GPU training for some reason).
How Has This Been Tested?

Manually by running the tests against OCP cluster with a node containing 2L4 GPUs.
Merge criteria:

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

/lgtm


Document architecture

As an architect I need to understand the architecture design for Pathfinder.

https://trello.com/c/3pfDqpcm/45-as-an-architect-i-need-to-understand-the-architecture-design-for-pathfinder


Refactor Azure API Request Limits

In moments of high-load the OneFuzz Service quickly hits it's quota for Azure requests and begins outputting the following exceptions:
Exception while executing function: Functions.agent_can_schedule Result: Failure
Exception: HttpResponseError: (OperationNotAllowed) The server rejected the request because too many requests have been received for this subscription.
Code: OperationNotAllowed
Message: The server rejected the request because too many requests have been received for this subscription.

And then:
Timeout value of 00:15:00 was exceeded by function: Functions.agent_can_schedule

As far as we can tell, these exceptions build, but the service eventually recovers.
Creating this issue to track a better way to control our requests and avoid these quota limits.
AB#41139528
AB#41139533
AB#41139542
AB#41139548
AB#41139552

Do you know if these exceptions were being hit when communicating with a storage account? Or was it something else like scalesets/keyvault/etc.? Essentially I'm trying to understand if the data model refactor is going to solve this issue for us "for free"

Do you know if these exceptions were being hit when communicating with a storage account? Or was it something else like scalesets/keyvault/etc.? Essentially I'm trying to understand if the data model refactor is going to solve this issue for us "for free"

I'm not sure. Yesterday, I was seeing that several of our functions were hitting these exceptions - jobs, containers, agent_can_schedule, agent_commands, queue_file_changes, etc.


Obscure error message when modifying property accessed by UI on timer thread

Describe the bug
Normally, when you access anything that the UI concerns from another thread, you have to go through the dispatcher. A member in the UWP Community Discord ran into this same problem, but was met with a very undescriptive error message, saying System.Runtime.InteropServices.COMException: "".
Though, I do recall seeing the proper error message before when this happened, but I don't remember how long ago that was.
Steps to reproduce the bug
https://github.com/Pdawg-bytes/WASDK-INPC-COMException-repro/tree/main
Clone this repository and run the app. Take note of the comments in App1/SystemViewModel.cs.
Expected behavior
When you attempt to access something like this, the error message should tell you that you need to use the Dispatcher to dispatch it to the UI thread.
Screenshots
Here's the undescriptive error message:

NuGet package version
WinUI 3 - Windows App SDK 1.6.0: 1.6.240829007
Windows version
Windows 11 (22H2): Build 22621
Additional context
No response

This is very annoying.
It's not clear to me why the exception text is empty, as the HResult value is properly set to -2147417842 0x8001010E RPC_E_WRONG_THREAD. The plain old Win32Exception would automatically display a message like The application called an interface that was marshalled for a different thread..


Is it support ipcam on xm550ai chip?

and how to install firmware on ipcam xm550ai?

https://github.com/OpenIPC/openipc-2.1/wiki/install_xm530

Unfortunately i cannot understand. Which program i need to use and how to upload new firmware to ipcam.
Need step by step instruction

You will need UART adapter connected to your computer and camera and then boot to U-boot command line and do commands applicable for your board.
Don't forget about backup of your original firmware

i can't find uart interface on board. Maybe you can help?
Parameters for IVG-85X50PYA-S.pdf

Check top connector with pin marked from 1 to 4, first find GND with your multimeter and then find TX and RX by trial and error method


Fix intermittent panic in notify issue change content

Ensure that issue pullrequests are loaded before trying to set the self-reference.

LoadPullRequest does the assignment already.

Please send backport

Please send backport

I was unable to create a backport for 1.18, please send one manually. :tea:


Add debugging for message processing.  Set DEGUG=framework for extra …

Add debugging for message processing.  Set DEGUG=framework for extra messages about hot the framework is treating incoming messages

Thanks @jpjpjp !


无法正常运行3.6

Traceback (most recent call last):
File "/Volumes/Study/Python/Web/venv/lib/python3.6/site-packages/cffi/cparser.py", line 5, in
from . import _pycparser as pycparser
ImportError: cannot import name '_pycparser'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Volumes/Study/Python/Web/base64.py", line 1, in
import xxtea
File "/Volumes/Study/Python/Web/venv/lib/python3.6/site-packages/xxtea/init.py", line 9, in
ffi = FFI()
File "/Volumes/Study/Python/Web/venv/lib/python3.6/site-packages/cffi/api.py", line 65, in init
from . import cparser
File "/Volumes/Study/Python/Web/venv/lib/python3.6/site-packages/cffi/cparser.py", line 7, in
import pycparser
File "/Volumes/Study/Python/Web/venv/lib/python3.6/site-packages/pycparser/init.py", line 15, in
from .c_parser import CParser
File "/Volumes/Study/Python/Web/venv/lib/python3.6/site-packages/pycparser/c_parser.py", line 11, in
from .ply import yacc
File "/Volumes/Study/Python/Web/venv/lib/python3.6/site-packages/pycparser/ply/yacc.py", line 67, in
import base64
File "/Volumes/Study/Python/Web/base64.py", line 6, in
encrypt_data = xxtea.encrypt(text,key)
AttributeError: module 'xxtea' has no attribute 'encrypt'
Console里面import xxtea也报错

似乎是你的 cffi 包有问题，这个跟 xxtea 这个包没有直接关系。


Fix up for Raspotify

As of Raspotify 0.31.4 a fair amount has changed.

PRETTY_HOSTNAME is unnecessary. Raspotify's default device name is already "raspotify(HOSTNAME)". It has been for a long while now.

The Raspotify config file has moved and changed formats. https://github.com/dtcooper/raspotify/wiki/Migration-from-older-versions-to-0.31.4-and-beyond

Raspotify no longer creates a "raspotify" user, it instead relies on systemd's DynamicUser functionality.

Not setting the initial volume allows Raspotify to remember the volume between starts.

Don't edit the Raspotify config. It has sane defaults that the user can change if they so desire. This also has the benefit of not having to worry about keeping up with librespot options and flags.

Raspotify's default device name is already "raspotify(HOSTNAME)". It has been for a long while now.

This is why I introduced the PRETTY_HOSTNAME functionality and change the configs: I really do not want the device to be visible as "raspotify(HOSTNAME)", but as something less technical like "Living Room". I'll have a look at how this is possible with the new configuration format.

This is why I introduced the PRETTY_HOSTNAME functionality and change the configs: I really do not want the device to be visible as "raspotify(HOSTNAME)", but as something less technical like "Living Room". I'll have a look at how this is possible with the new configuration format.

Why wouldn't you just change the actual hostname? shairport-sync and I believe bluetooth can be configured to pull in the devices hostname.

Why wouldn't you just change the actual hostname? shairport-sync and I believe bluetooth can be configured to pull in the device's hostname as or at least apart of their service's name.

Like I said, the device name in Spotify/AirPlay should not be something technical like the hostname (at least for me). I want the AirPlay device to be visible as "Living Room", while the hostname is e.g. "rpi-living-1" (and I really do not want the hostname to be "Living%20Room").

Both are arbitrary.


Allow multiple managers

previously our promoting manager method was to handoff the 'single torch' of manager role to someone else. However upon further consideration, we gotta fix that. There will be at least two people that have the manager role at the Tower at the same time (Tower Lead and Head Setter), but there are probably more people that we want to have the ability to have this as well (perhaps some Outdoor Adventure higher-ups will want manager permissions in the app)

duplicate issue, see #87


Any way to set the collectionFormat of array properties in viewmodels?

I have a number of array properties but they always get displayed as collectionFormat multi.  I'd like to use collectionFormat csv.
Can this be set in .NET?

If only the Schema had collctionFormt as a property


[3.15.x] Fixes #6747 - opentelemetry - ensure that the sequence of recorded sp…

backports https://github.com/apache/camel-quarkus/pull/6748
cherry-picked: 42eecb3071f1f73048b84e6abeddc1eff9e35e46

The main PR is currently opened https://github.com/apache/camel-quarkus/pull/6748
As for the explanation - it is based on https://camel.apache.org/camel-quarkus/3.15.x/reference/extensions/opentelemetry.html#extensions-opentelemetry-usage-tracing-cdi-bean-method-execution
To ensure that the sequence of recorded spans is correct, you must use the full to("bean:") endpoint URI and not the shortened .bean() EIP DSL method.

ok, many thanks for explanation @JiriOndrusek :+1:


Support for subtitles while streaming

Today you can stream videos like this:
youtube-dl -o - https://www.youtube.com/watch?v=<id> | vlc -

It would be nice to be able to have subtitles when streaming. Something like:
youtube-dl --write-sub-file /tmp/subtitles.srt -o - https://www.youtube.com/watch?v=<id> | vlc --sub-file /tmp/subtitles.srt -

[X] I've verified and I assure that I'm running youtube-dl 2019.01.24
[X] At least skimmed through the README, most notably the FAQ and BUGS sections
[X] Searched the bugtracker for similar issues including closed ones
[X] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser
[X] Bug report (encountered problems with youtube-dl)

$ youtube-dl --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out utf8, pref UTF-8
[debug] youtube-dl version 2019.01.24
[debug] Python version 2.7.15 (CPython) - Darwin-16.7.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 4.0.2, ffprobe 4.0.2, rtmpdump 2.4
[debug] Proxy map: {}

After putting the right value for lang and url, the following shell command
should do what you want.
lang=en; url='https://www.youtube.com/watch?v=<id>'; pwd_sv="$PWD"; tmpdir=`mktemp -d`; cd "$tmpdir" && youtube-dl -o sub --write-sub --skip-download --sub-format best --sub-lang "$lang" "$url" && test -f "$tmpdir"/sub.* && youtube-dl -o - "$url" | vlc --play-and-exit --sub-file "$tmpdir"/sub.* -; cd "$pwd_sv"; rm -rf "$tmpdir"

I suspect the vlc subtitle auto-detection is buggy at the moment because I
cannot make it work. I have to pass --sub-file explicitly in the above
command. I can turn the above into a proper shell script if you really need it.
For myself, I use mpv and because it has youtube-dl integration, the command
is as simple as mpv https://www.youtube.com/watch?v=<id>.

@alexvong1995: Augh! That's a bit long, and probably slow, since you have to request the video twice.
However, I found out that it's better to watch this way, as eliminating the pipe makes it possible to jump in the video, see the video length, etc.
vlc "$(youtube-dl --get-url 'https://www.youtube.com/watch?v=<id>')"

I'll leave the issue open for someone to suggest a way to get subtitles with this approach, or interest from the developers to add supports for this.
PS! Good to know about mpv!

@forthrin
forthrin<EMAIL_ADDRESS>writes:

@alexvong1995: Augh! That's a bit long, and probably slow, since you have to request the
video twice.
However, I found out that it's better to watch this way, as eliminating the pipe makes it
possible to jump in the video, see the video length, etc.
vlc "$(youtube-dl --get-url 'https://www.youtube.com/watch?v=')"
I'll leave the issue open for someone to suggest a way to get subtitles with this approach, or
interest from the developers to add supports for this.

I incorporate your suggestion into my last command:

id=; lang=en; tmpfile=mktemp; wget -O "$tmpfile" "https://www.youtube.com/api/timedtext?v=$id&lang=$lang&fmt=vtt" && vlc --play-and-exit --sub-file "$tmpfile" "$(youtube-dl -f best --get-url "https://www.youtube.com/watch?v=$id")"; rm -f "$tmpfile"

The command is now slightly less long. It uses 'wget' instead of
'youtube-dl' to download the sub, which is faster for me.

PS! Good to know about mpv!

I like the intuitive and minimalistic interface of mpv!

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

This is much better! This should be useful for several people out there. This snippet would be very suitable for a FAQ or the manual pages.

I'll close the issue, as it's probably not feasible to make youtube-dl pass both subtitles and streaming URL on to a video player in a uniform, straight forward way. This can be accomplished using a relatively simple script, as shown above, and in other ways.


problems with doc

it makes no sense the log levels, wrong examples, you really need to to a better job

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

ID: 86aff194-f810-3c21-e322-cdda05dfc734
Version Independent ID: 0864707c-97a7-1166-1bd4-ef9ea212c64e
Content: Configure monitoring for Azure Functions
Content Source: articles/azure-functions/configure-monitoring.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hi @gidziu. Thanks for your comments on this article. If you could please provide more detailed information about what was causing you trouble, it will make it easier for us to improve this article. #assign:ggailey777

@gidziu
I'm going to assign this to the document author so they can take a look at it accordingly.

@gidziu if you can help us by providing more specific examples, that would be great. In the meantime, #please-close


Organization of the MRTK menu with just the Foundation package

Not exactly a bug, but more for having a discussion. When you just pull in the Foundation package, the MRTK dropdown folder's organization looks a bit interesting:

Should Leap be first there, or do we want to have (or already have some other standard for how we choose to sort that?)

@keveleigh @CDiaz-MS thoughts?

Ahh interesting, yeah. I wonder if we want to have a platform/device-specific subsection somewhere. It feels like it shouldn't be first though? Looks like the menu items are currently hardcoded to be first with the 0:
https://github.com/microsoft/MixedRealityToolkit-Unity/blob/55f8afa15fb774d620f3d27ff8634e706e7ecb66/Assets/MRTK/Providers/LeapMotion/Editor/ConfigurationChecker/LeapMotionConfigurationChecker.cs#L328
so we can update it there? I think, if we want the line to show up separating sections, we need at a gap of at least 10 between one menu item and the next.
Configure Unity Project uses 499, but others (like Input Simulation) don't specify anything, so it might come down to load order at that point.
https://github.com/microsoft/MixedRealityToolkit-Unity/blob/e0b9d322d367d33047b7b0f09f87c193b81fbf18/Assets/MRTK/Core/Inspectors/Setup/MixedRealityProjectConfiguratorWindow.cs#L59

Yeah, I always felt like these hardcoded numbers have been kinda goofy - there's no clear single place to understand how the menu is being laid out at a global level.

I always felt like these hardcoded numbers have been kinda goofy - there's no clear single place to understand how the menu is being laid out at a global level.

We've been using the ProfileMenuItemIndices enum for ordering the "create asset" menu. We could do something similar for this menu

I always felt like these hardcoded numbers have been kinda goofy - there's no clear single place to understand how the menu is being laid out at a global level.

We've been using the ProfileMenuItemIndices enum for ordering the "create asset" menu. We could do something similar for this menu

That sounds good - we can add a CI step to check that you aren't using hardcoded numbers as well, which would prevent future folks from messing up.

Removed Leap menu priority in #7877


Are Datomic subscriptions supported?

David Nolen, in one of his videos, claims that Om Next supports subscriptions via its Relay-style query interface. But I haven't been able to find any examples of such. I'm newly a Javascript refufee (coming from Meteor and turning to CLJS), and Re-Frame seems like my new god lol, but I'm worried that it hasn't pursued the Relay-influenced features as much as Om Next.
How do you go about go about Datomic queries and mutations from Re-Frame components and are the queries reactive subscriptions (i.e. they re-render components when server side Datomic state changes)?

@faceyspacey this is not the place to ask for support.  Repo issues are for bugs and feature requests. For support, please use http://clojurians.net/  or newsgroups like  https://groups.google.com/forum/#!forum/clojurescript


tech: legacy and codestyle cleanup

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Typescript Language Died

We have written the needed data into your clipboard because it was too large to send. Please paste.
Issue Type: Bug
When I was renaming and moving some folders inside my Angular project the Typescript extension just died.
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:57:51.166Z)
OS version: Darwin x64 19.0.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-5350U CPU @ 1.80GHz (4 x 1800)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
18, 7, 5

Memory (System)
8.00GB (0.50GB free)

Process Argv
-psn_0_6923930

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

vue-snippets
hol
0.1.11

Angular2
joh
8.3.0

vscode-docker
ms-
0.9.0

vetur
oct
0.22.6

material-icon-theme
PKi
3.9.2

RocketseatReactJS
roc
2.2.0

RocketseatReactNative
roc
2.2.0

vue-vscode-snippets
sdr
1.8.0

nativescript
Tel
0.11.0

(2 theme extensions excluded)

A popup opened with this message:
The TypeScript language service died 5 times right after it got started. The service will not be restarted.
Source: TypeScript and JavaScript Language Features (Exten...

/needsMoreInfo

any updates? I still have this issue


feat: port CategoryTheory.GlueData

TODO:

[ ] gluedIso:

failed to synthesize instance
  HasColimit (MultispanIndex.multispan (diagram (mapGlueData D F)))

find a way to sythesize this instance.

[ ] ι_gluedIso_hom: probably will be fixed automatically when guedIso is fixed.

Thanks!
bors r+


Add some extra details to docs

Hi, Ben! Thanks for creating this project. I heard you on the Syntax podcast and it was great timing as I was in the market for a self-hosted visitor analytics solution.
I hadn't used Cloudflare before and had a few minor stumbles when going through the README, similar to issue #71 .
I've added a little bit more detail to the README and have added some screenshots too.
I created a /docs directory for the screenshots but I also realise that you load existing screenshots from an external URL. I'm happy to follow that pattern if you want to save my screenshots somewhere else.
Thanks!

Hey, thanks for submitting. I just got back from a vacation but hoping to look at this soon.

@ninjaPixel Merged – thanks!


1.9.0 build from Spigot - crashed server on /wb bypass

I've installed the latest version of WorldBorder for 1.13.1 and I'm using PaperSpigot build 262 (1.13.1) with Dynmap-3.0-SNAPSHOT-spigot.jar (1.13.1).  I issued a wb bypass smmmadden from the console and it worked perfectly and updated the config file automatically.  I then went into the game and issued the same command for my wife's admin account and the config was updated, but server crashed immediately afterward.
See crash report here
crash-2018-09-16_23.04.02-server.txt
Where it crashed:

Not clear if this is WB, Dynmap or Paper related.
-Steve

Wouldn't expect it to be anything to do with WorldBorder, and the stack trace contains only NMS code references. Looks to be a problem with Paper/Spigot/Bukkit/Minecraft rather than a plugin, based on that.


SyntaxError: Use of const in strict mode

We are using gulp-babel to build and we are getting the following error since last week:
[INFO] [10:24:18] Requiring external module babel-register
[ERROR] node_modules\web-component-tester\runner\cli.js:11
[ERROR] const chalk = require('chalk');
[ERROR] ^^^^^
[ERROR] SyntaxError: Use of const in strict mode.
[ERROR]     at exports.runInThisContext (vm.js:73:16)
[ERROR]     at Module._compile (module.js:443:25)
[ERROR]     at Module._extensions..js (module.js:478:10)
[ERROR]     at Object.require.extensions.(anonymous function) [as .js] (node_modules\babel-register\lib\node.js:166:7)
[ERROR]     at Module.load (module.js:355:32)
[ERROR]     at Function.Module._load (module.js:310:12)
[ERROR]     at Module.require (module.js:365:17)
[ERROR]     at require (module.js:384:17)
[ERROR]     at Object. (node_modules\web-component-tester\runner.js:11:11)
[ERROR]     at Module._compile (module.js:460:26)
Seems like cli.js was changed 12 days ago.

What version of node are you using?

I had the same problem when I was building my project with node v0.12.0, not with node v5.0.0 though.

I'm getting this issue as well.  Do we know which branch introduced this issue?

The problem occurs when using wct > 4.3.0 on older node versions. I don't know exactly which versions. Our CI server was using 0.12 and that's where we saw the error.
Solution:

Upgrade node or
Downgrade wct

Apologies, our tools are moving to support the latest Node LTS and latest stable. This means Node 4 and 6 right now.

We rolled back to WCT 4.2.2, and it seems to be working.  Thanks, guys.

@u4yk our current plan is to only support the latest LTS and stable. If you're not able to upgrade node, you won't get any new features. It'd be great to have feedback here. Are there things blocking you from upgrading node to a somewhat recent version?

@justinfagnani, we are also using node 0.12 and cannot upgrade it in the near future (we are working on it though...). In the meantime we have downgraded to wct 4.2.2

@justinfagnani we're planning on upgrading node,  but since this was in production,  we would need a change request to upgrade.  Reverting to 4.2.2 is a good stopgap until we do upgrade.


Fixed connections to brokers that share the same host and port

This fixes #1190
The problem was caused by brokers getting overridden by the seed broker, because all nodes share the same host and port in Oracle Cloud's Streaming service. I changed it to update the nodeId and rack (if necessary) before making the list of brokers, which allows us to use a more specific check when returning the seed broker itself.

@Nevon I found one more place that makes the assumption about one node per host/port: brokerPool.removeBroker({ host, port })
It seems that this is only called on KafkaJSConnectionClosedError. Do you have any recommendations here? Would it be correct to remove all brokers with the specified host and port?

It seems that this is only called on KafkaJSConnectionClosedError. Do you have any recommendations here? Would it be correct to remove all brokers with the specified host and port?

That would probably "work", since all requests would be rejected and on subsequent retries we would reconnect, but I'm not sure if it makes sense since in the case of Oracle Streaming service that would apparently mean rejecting all requests and disconnecting all brokers.
One option is to change brokerPool.removeBroker to take a Broker instance instead of a host+port. I don't like this solution because it means that the Connection has to have a reference to the broker, despite being on a lower layer and not being supposed to know anything about brokers (it does have a broker property, but it's just a convenience over host:port - it's not an actual broker). The reason the Connection would need it is to be able to include it as a property in the KafkaJSConnectionClosedError.
Another option is to change how the removal of the Broker from the BrokerPool works. Currently, the Cluster is responsible for removing the broker from the brokerpool, but what if we change it so that the broker itself can deregister itself from the brokerpool? When the broker is instantiated, the brokerpool can pass in a function that allows the broker to remove itself from the brokerpool, which it can call when it gets a KafkaJSConnectionClosedError (currently it just disconnects). We can then remove the corresponding code from the Consumer, Producer and ConsumerGroup.
this.createBroker = options =>
  new Broker({
    allowAutoTopicCreation,
    authenticationTimeout,
    reauthenticationThreshold,
    onDisconnect: broker => {
      this.removeBroker(broker)
    },
    ...options,
  })

I don't know if there's a better solution, but at least it avoids a circular dependency between the broker and the connection, and probably allows us to remove some duplicated code.

all nodes share the same host and port in Oracle Cloud's Streaming service

On another note, do you have an account manager or something you can contact in Oracle Cloud about this kind of issue? While I'm happy and thankful that you're trying to solve your own issue, it grinds my gears a bit that you and I are paying the cost of Oracle doing weird shit (you can put AWS MSK in the same category) instead of them supporting their own product. Especially since now we have to pay the maintenance cost of this forever, without even being able to reproduce the issue since their service is closed-source.


Replication Auto Discovery

We have 2 nodes, 1 primary, 1 secondary.
We added a new node called <IP_ADDRESS> as secondary, the state was unknown in rs.status()
but mongoid picked <IP_ADDRESS> in few minutes and create 200 delayed. I can use these when I print the mongoid log to console.
[1/26/15, 7:10:55 AM] sahin: D, [2015-01-26T15:11:54.859790 #4010] DEBUG -- :   MOPED: <IP_ADDRESS>:27017 COMMAND      database=admin command={:ismaster=>1} runtime: 20010.1679ms
[1/26/15, 7:11:05 AM] sahin: D, [2015-01-26T15:11:54.879640 #4010] DEBUG -- :   MOPED: <IP_ADDRESS>:37017 QUERY        database=movielala collection=movies selector={"$query"=>{}, "$orderby"=>{:_id=>1

@arthurnn

Just to add more info in to this ticket:

This is not a master/slave setup. It is a Primary/secondary!
Moped didnt remove the replica that had state 'UNKNOWN' from the cluster.

am I right @sahin ?

my bad, it is primary/secondary.
We added a new node, and it had some network firewall errors. and while we were working to fix the network setup, we realized that
Moped added the node which had 'UNKNOWN' from the cluster.


Add a setRef version that supports non rw fs

There are some filesystems that do not support opening the files in read
and write modes at the same time. The method SetRef is split in files with
an extra version that only writes the reference. It can be activated with
-tags norwfs on building.

Do not merge. There are still some changes needed to make it fully compatible.

The implementation is ready. Tests also pass with go test -tags norwfs ./....


[Suggestion] Add more notification after turning on in User Setting

OS: All
Build Version: 20190606.16
Repro Steps:

Make sure that there are 3 Linux web app at least.
Try to create a  new Linux web app again.
Pop up a dialog --> Select "Turn on advanced creation".

Actual:
Open the user setting and the action terminates.
Suggestion:
Add more notification after turning on advanced creation in User Setting like "Please reload the VS Code" or anything else.

With the new advanced creation, this does not apply anymore so I'll go ahead and close this


This template depends on flat-map-stream, which is a security concern right now!

Is your feature request related to a problem? Please describe.
(Copied from bot-docs: issue 596. Quoted below.)
Describe the solution you'd like
Remove reference to flat-map-stream package.
Describe alternatives you've considered
Quoted issue from @kraghavk.

The project generated by botbuilder mentioned here has a dependency on flat-map-stream package and this package cannot be trusted anymore. See npm audit output below:

                   === npm audit security report ===

                             Manual Review

         Some vulnerabilities require your attention to resolve

      Visit https://go.npm.me/audit-guide for additional guidance

Critical Malicious Package

Package flatmap-stream

Patched in No patch available

Dependency of nodemon [dev]

Path nodemon > pstree.remy > ps-tree > event-stream >flatmap-stream

More info https://nodesecurity.io/advisories/737

found 1 critical severity vulnerability in 2920 scanned packages
1 vulnerability requires manual review. See the full report for details.

Hope MS will fix this dependency soon.

[enhancement]

a fix for this has been merged into master.  generator-botbuilder v 4.1.20 or higher will have the fix when it is published


FEATURE: Use resumable uploads

This should lower the memory usage when storing into the cloud storage.

Did not test that, but maybe it should work. :)

/cc @Inchie

Now that we're running on the more modern SDK, there's an option for enabling resumable uploads. Let's look into that. Regarding chunked uploads, the SDK source code says that it's "not recommended to use". Well.

Ok, then let's see what the new SDK achieves already.


gem install jets not working without postgresql installed locally

looks like I needed to have postgresql installed locally on my mac (High Sierra 10.13.6 w/Xcode 9.4.1 w/devtools installed) in order to install the jets gem. should probably note this (and other possible dependencies) in the installation docs. I see this note in your gemspec, but having this in the quick start could be helpful until this req is removed :)
https://github.com/tongueroo/jets/blob/master/jets.gemspec#L53
This issue is about http://rubyonjets.com/docs/install/ (source file: https://github.com/tongueroo/jets/blob/master/docs/_docs/install.md)

Thanks for the bug report 👍 Fixed in PR #35
Also added a --no-database option that tells jets new to not insert the pg gem to the generated Gemfile.
Released in v0.8.15 https://github.com/tongueroo/jets/blob/master/CHANGELOG.md


process: missing CONTRIBUTING.md

No CONTRIBUTING.md file is present in the protobuf repo, so it's unclear whether a CLA is required for contributions.  Based on pull requests, it looks like one is.  Please consider adding a CONTRIBUTING.md file to your repository.

https://github.com/golang/protobuf/blob/v1.4.0-rc.3/CONTRIBUTING.md


refactor: move JuliaSIMD deps to extensions

xref https://github.com/EnzymeAD/Enzyme.jl/issues/1983


Add support and tests for URLComponents.percentEncodedQueryItems

rdar://59439703 (URLComponents.percentEncodedQueryItems missing on linux)
resubmission of #2870 against main

@swift-ci test

@spevans should I merge this or is it expected that the code owners will?

Could you just squash this down into 1 commit to remove the merge branch commit  then I think it should be good to go. I think it should also be worth back-porting to 5.3 since its adding missing functionality. Thanks.

@spevans

Could you just squash this down into 1 commit to remove the merge branch commit then I think it should be good to go

done!

I think it should also be worth back-porting to 5.3 since its adding missing functionality. Thanks.

Does this mean I should also raise a pr against release/5.3?

@swift-ci test and merge

@rauhul The final decision on whether to add a pr to the next 5.3 patch release will be up-to the release manager but since this is fixing a bug (missing functionality) and isn't source breaking its probably eligible.
Once its gone into main use git cherry-pick -x <commit sha> to back-port it to the 5.3 branch as this  will generate a nice commit message which Github will render with a link back to this PR. Then raise this as a new PR against the release/5.3 branch - (one of the reasons to squash commits is that it make back-porting easier).

Could this be the cause that the bot is failing in:
https://ci.swift.org/job/oss-swift-incremental-RA-linux-ubuntu-16_04/15227

The following tests FAILED:
	1367 - TestFoundation.TestURLComponents-test_queryItems (ILLEGAL)
	1373 - TestFoundation.TestURLComponents-test_createURLWithComponents (ILLEGAL)
	1376 - TestFoundation.TestURLComponents-test_percentEncodedQueryItems (ILLEGAL)

@spevans @aschwaighofer any idea why this wouldn't have been caught pre-merge? I don't have a linux environment to repro on unfortunately.

It looks like it failed on ubuntu 16.04 and the CI only tests on 18.04 before merge. Im just getting an old 16.04 host up and running again to see if I can see what the issue is.

I retested this PR on Ubuntu 16.04 and could not get it to fail. Given the nature of the test failure (SIGILL) my best guess is the the CoreFoundation part did not get rebuilt since it explicitly adds a +1 retain, presumably that code is currently buggy anyway.
My suggestion is to create a new PR with just the CoreFoundation change and we can merge that. Once a nightly has gone out merge the rest in a 2nd PR. I think that should avoid any stale build cache issues.

@spevans sounds good, I PR'ed the CoreFoundation changes here #2936


[MALA FAMA TEAM] Improve coverage of the fuzzer

Hi!
This one is related to HackerOne reports 725149 and 725151.
The reason behind this PR is that the fuzzer is only fuzzing the ipv6/ipv4 packets (see
https://github.com/aol/moloch/blob/master/capture/packet.c#L1916). If we modify the
linkType to be 1 we are going to get to the ether packet level, meaning that we will fuzz everything. Depending on the type, will select the type of packet that we want and trigger so we will end up with much more coverage of the program. On our tests we got around 15% more coverage and found some new crashes doing so.
License
I confirm that this contribution is made under an Apache 2.0 license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Thanks! Great idea!  I placed in another location, but helpful.


ERROR: Unable to extract uploader id;

Checklist

[ ] I'm reporting a broken site support issue
[X] I've verified that I'm running youtube-dl version 2021.12.17
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar bug reports including closed ones
[X] I've read bugs section in FAQ

Verbose log

[debug] Custom config: []
[debug] Command-line args: ['https://www.youtube.com/watch?v=Huif6_lMhPk', '--verbose']
[debug] Encodings: locale cp1251, fs utf-8, out utf-8, pref cp1251
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.10.11 (CPython) - Windows-10-10.0.22621-SP0
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] Huif6_lMhPk: Downloading webpage
[youtube] Huif6_lMhPk: Downloading MPD manifest
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\TK\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\youtube_dl\YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "C:\Users\TK\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\youtube_dl\YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "C:\Users\TK\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\youtube_dl\extractor\common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "C:\Users\TK\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\youtube_dl\extractor\youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "C:\Users\TK\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\youtube_dl\extractor\common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

I am trying to download Youtube stream https://www.youtube.com/watch?v=Huif6_lMhPk and getting the error "Unable to extract uploader id;"

Same issue, can't download any videos anymore, upgraded to the latest too

I have the same problem on Windows
> youtube-dl https://www.youtube.com/playlist?list=PLW-oQRxLODMcYEFXP4eg5EXBl-0hsTSe1 --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.youtube.com/playlist?list=PLW-oQRxLODMcYEFXP4eg5EXBl-0hsTSe1', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.22621
[debug] exe versions: none
[debug] Proxy map: {}
[youtube:tab] PLW-oQRxLODMcYEFXP4eg5EXBl-0hsTSe1: Downloading webpage
[download] Downloading playlist: Design of Experiments
[youtube:tab] Downloading page 1
[youtube:tab] playlist Design of Experiments: Downloading 118 videos
[download] Downloading video 1 of 118
[youtube] KSOvTHNEtig: Downloading webpage
[youtube] KSOvTHNEtig: Downloading MPD manifest
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 815, in wrapper
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 836, in __extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 534, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\youtube.py", line 1794, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 1012, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

And also on WLS Ubuntu
% youtube-dl https://www.youtube.com/playlist\?list\=PLW-oQRxLODMcYEFXP4eg5EXBl-0hsTSe1
[youtube:tab] PLW-oQRxLODMcYEFXP4eg5EXBl-0hsTSe1: Downloading webpage
[download] Downloading playlist: Design of Experiments
[youtube:tab] Downloading page 1
[youtube:tab] playlist Design of Experiments: Downloading 118 videos
[download] Downloading video 1 of 118
[youtube] KSOvTHNEtig: Downloading webpage
[youtube] KSOvTHNEtig: Downloading MPD manifest
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
olivia% youtube-dl https://www.youtube.com/playlist\?list\=PLW-oQRxLODMcYEFXP4eg5EXBl-0hsTSe1 --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.youtube.com/playlist?list=PLW-oQRxLODMcYEFXP4eg5EXBl-0hsTSe1', '--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.10.6 (CPython) - Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.35
[debug] exe versions: none
[debug] Proxy map: {}
[youtube:tab] PLW-oQRxLODMcYEFXP4eg5EXBl-0hsTSe1: Downloading webpage
[download] Downloading playlist: Design of Experiments
[youtube:tab] Downloading page 1
[youtube:tab] playlist Design of Experiments: Downloading 118 videos
[download] Downloading video 1 of 118
[youtube] KSOvTHNEtig: Downloading webpage
[youtube] KSOvTHNEtig: Downloading MPD manifest
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Same Issue here
>> youtube-dl --extract-audio --audio-format mp3 -k --verbose https://www.youtube.com/watch\?v\=43Lbv2TDyPI\&ab_channel\=MusicWorld
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--extract-audio', '--audio-format', 'mp3', '-k', '--verbose', 'https://www.youtube.com/watch?v=43Lbv2TDyPI&ab_channel=MusicWorld']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: 01e0c20d0
[debug] Python version 3.10.11 (CPython) - macOS-13.3.1-arm64-arm-64bit
[debug] exe versions: ffmpeg 6.0, ffprobe 6.0, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] 43Lbv2TDyPI: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

>> youtube-dl --version
2021.12.17

[ ] I'm reporting a broken site support issue

You are, but you shouldn't be, because ...

[x] I've searched the bugtracker for similar bug reports including closed ones

Learn how by reading #30839 (again?).

[x] I've read bugs section in FAQ

Apply this.


[BUG] CRC installers json in CDN cannot be parsed due to format error

General information
Json file downloaded from CDN cannot be parsed due to format error
SyntaxError: JSON.parse: expected double-quoted property name at line 6 column 2 of the JSON data

{
	"version": {
		"crcVersion": "2.37.1",
		"gitSha": "36d451",
		"openshiftVersion": "4.15.14",
	},
	"links": {
	    "linux": "https://developers.redhat.com/content-gateway/file/pub/openshift-v4/clients/crc/2.37.1/crc-linux-amd64.tar.xz",
	    "darwin": "https://developers.redhat.com/content-gateway/file/pub/openshift-v4/clients/crc/2.37.1/crc-macos-installer.pkg",
	    "windows": "https://developers.redhat.com/content-gateway/file/pub/openshift-v4/clients/crc/2.37.1/crc-windows-installer.zip"
	}
}

CRC version
2.37.1

This has been fixed in git in https://github.com/crc-org/crc/pull/4224
We are working on updating/fixing the file in CDN, but this is taking more time than we hoped :-/

Should be fixed now
curl -L  https://developers.redhat.com/content-gateway/file/pub/openshift-v4/clients/crc/2.37.1/release-info.json|jq .


Initial smoke test for Kubernetes.

This is only create/delete server group.
Kubernetes support for the observer looks awkward because kubectl has
awkward commands and doesnt fully support json. But it is sufficient for
now for this test. Might need to take a different approach when writing
future tests.
@lwander
This can wait for tomorrow. I'm not going to do any more today.

github is confused by this branch and thinks there are conflicts.
I created pull/145 using a git diff master with this branch and it is happy so am closing this one.


Is it possible to detect multiple schemas in one editor?

For context, I want to create a Kubernetes editor that takes in an input that may contain more than one YAML documents/instances (using an array or the --- directives). And provide IntelliSense based on the detected schema(s). My question is, is it possible to load schema(s) on demand? i.e: editor detects a k8s resource type and loads the schema of the detected type. Thanks!

I imagine something like this pseudocode could work:
import * as monaco from 'monaco-editor'
import { configureMonacoYaml } from 'monaco-yaml'

const monacoYaml = configureMonacoYaml(monaco, {
  enableSchemaRequest: true,
  schemas: []
})

const model = monaco.editor.createModel('', undefined, monaco.Uri.parse('file:///kubernetes-resource.yaml'))

let schema;
model.onDidChangeContent(() => {
  const newSchema = guessResourceSchemas(model.getValue())
  if (newSchema !== schema) {
    schema = newSchema
    monacoYaml.reconfigure({
      enableSchemaRequest: true,
      schemas: {
        fileMatch: 'kubernetes-resource.yaml',
        uri: 'https://kubernetes.io/docs/',
        schema
      }
    })
  }
})

There’s also the isKubernetes option. I tried to explain it in https://github.com/remcohaszing/monaco-yaml/issues/54#issuecomment-908653577.