How to download and merge video from "X" and audio from "best" when both files have video and audio?

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
There is no separate audio files. Bigger resolution = better audio.
For example 480p+best download only first.
--compat-options multistreams give file with two video and audio tracks.
Is it possible to take video from one and audio from another using one command or I have download both and remux using selected tracks for example by ffmpeg?
Verbose log
No response

Which site are you talking about exactly? Because depending on the site, video and audio come together, so you can't collect audio from one and video from another.

You have to download both entirely and mux them together

I know I have to download both. I meant muxing selected tracks automatically like youtube-dl does instead of muxing them manually after download.

I meant that you have to mux them manually
There is no in-built way to discard certain streams during merge. It may be possible to do with --ppa, but you will have to experiment with it (I haven't tried)

I meant muxing selected tracks automatically like youtube-dl does

youtube-dl can't do this either

youtube-dl can't do this either

I tried it many times before I moved to yt-dlp and it worked so it's possible there.

Maybe I misunderstood your question. Show the verbose log of youtube-dl doing what you want

you can test it out for example on random YouTube video passing youtube-dl -f 18+22 LINK (both have video and audio) and it will mux it (ffmpeg needed) by taking video from first and audio from second

I’m pretty sure this wasn’t an intended use case of the original upstream project, and it worked only by accident. But in that case, I guess this is a side effect of blackjack4494/youtube-dlc#81 (specifically, https://github.com/blackjack4494/youtube-dlc/pull/81/commits/d03cfdce1bce28284fa0f168d5d278caacd2fa0b#diff-e77497ca8029f8d381974fd189b343311d8f1323f49ebae2203cfaec91e98f8eL521-R526), from before this project was even formed.

I also don't think this was intended. But as Linus once said, a bug that people depend on is a feature. It would be difficult to "fix" this in yt-dlp now for normal operation. But I should be able to make it work atleast with a compat option

There is no in-built way to discard certain streams during merge.

Doesn't -f bestvideo*+bestaudio discard the bestvideo*'s audio stream (if present) when merging with bestaudio ?

on my side I m trying to simply select an average quality format , audio and video being split but it just download the video.
If I don't select format then it's working fine by downloading the best quality audio and video then merging the both
yt-dlp https://www.france.tv/france-5/c-dans-l-air/3015299-covid-scandales-boris-johnson-contre-attaque.html -f hls-1582-1+hls-1582-0
any ideas?

@pukkandan said: There is no in-built way to discard certain streams during merge.

Doesn't -f bestvideo*+bestaudio discard the bestvideo*'s audio stream (if present) when merging with bestaudio ?

no bestvideo*+bestaudio selects best video source even with audio and if there is not audio pressent it gets the best audio source

This took me a while to figure out but some f3 in the read me did some justice.
I have this. Working on windows with ffmpeg installed.
yt-dlp --audio-multistreams -S ext -P "C:\path" "videoLink"
--audio-multistreams            Allow multiple audio streams to be merged into a single file.
This is because everyone knows 1080p youtube has no audio.
If you don't do this, you'll be having a blank audio clip over a good audio clip.
doing this on particular videos will leave you by default with webm extension.
*Format Sorting: The default format sorting
options have been changed so that higher resolution and better codecs will be now preferred instead of simply using larger
bitrate. Furthermore, you can now specify the sort order using -S. This allows for much easier format selection than what is
possible by simply using --format
vext: Video Extension (mp4 > mov > webm > flv > other). If --prefer-free-formats is used, webm is preferred.
aext: Audio Extension (m4a > aac > mp3 > ogg > opus > webm > other). If --prefer-free-formats is used, the order changes to ogg > opus > webm > mp3 > m4a > aac
ext: Equivalent to vext,aext
Hope this answers it for you.

I also don't think this was intended. But as Linus once said, a bug that people depend on is a feature. It would be difficult to "fix" this in yt-dlp now for normal operation. But I should be able to make it work atleast with a compat option

Is this because people are dependent on the other behavior in various settings?   My use case for this in particular is when the video has a particular language audio attached and the alternate language track is provided as audio only.  For my use case it doesn't make sense to mux them afterwards, because I want to be able to watch live streams, so I am also using "-o -" to pipe it into ffmpeg and/or vlc.
Are we against enabling the arbitrary stream merges in general or are we okay with doing it as long as there is some kind of flag to specify that is what is occurring.   To me it's most reasonable if you explicitly specify a video and a separate audio source, that the expected behavior is to use the video from the first and the audio from the second, not just ignore the second if the first has audio, but if this causes compatibility issues it's understandable to have some kind of flag.

@gnuyen Your use-case should be covered by --audio-multistreams, no? OP's issue was that they specifically want to discard the other streams

@gnuyen Your use-case should be covered by --audio-multistreams, no? OP's issue was that they specifically want to discard the other streams

I'll check during the next live event, but as far as I could tell using -o - and --audio-multistreams and piping it to ffmpeg didn't seem to work.  I couldn't access the second audio stream at all and didn't see one, but maybe I was using ffmpeg wrong.  I will check tomorrow as this will for sure come up again

Hello,
I want to add that if you want to select a specific video and audio streams (formats) to combine, you can do the following:

Get the available audio and video streams
yt-dlp --list-formats <YOUTUBE LINK>

Download and combine the formats you want to combine using their IDs
yt-dlp --format <VIDEO FORMAT ID>+<AUDIO FORMAT ID> <YOUTUBE LINK>

Example:
yt-dlp --list formats https://www.youtube.com/watch?v=aqz-KE-bpKQ
yt-dlp --format 17+599 https://www.youtube.com/watch?v=aqz-KE-bpKQ

i have an issue, using  only best gives me  low quality video and audio but when using bestvideo  gives me good quality video but without an audio
and when i combined bestvideo +bestaudio am having an error talking about ffmpeg path
ERROR: You have requested merging of multiple formats but ffmpeg is not installed. Aborting due to --abort-on-error
Failed to download https://youtube.com/shorts/nAD-1ffz6yk?si=gq9e4t2BiO-p9uXl: ERROR: You have requested merging of multiple formats but ffmpeg is not installed. Aborting due to --abort-on-error
PS C:\Users\HP\Desktop\Programming\Python_programming\Django Tutorial\Youtube downloader\youtube_d>
Please how can i reslove it i wanna use it for my django project

@Tboiii-123 I don't think that is related to this issue. You are missing ffmpeg. best means the best pre-merged format. For YouTube, that means 360p. bestvideo means best format that only contains a video. If you have ffmpeg installed, you want to leave everything at the default and yt-dlp will automatically pick what it thinks is the best quality formats. If even after installing ffmpeg you run in to problems, please open a new issue with a complete verbose log.


Map EPICS alarms to JAWS alarms

Currently we forward EPICS CA messages into the alarm system field-for-field, making priority options (severity) alarm type specific.  It might be cleaner to define priority/severity in the registration topic instead.  This would mean that EPICS alarms would need to be split per severity - likely doubling our 13,000+ EPICS alarms to 26,000+ since EPICS has a two severity system: MAJOR and MINOR.    However, when non-EPICS alarms are defined a priority/severity field would be shared / exist.   Our epics2kafka-alarms translation layer would need to be updated.   Client GUIs/HMI would likely be cleaner if all alarms have same set of possible priorities regardless of source and the alarming schema would be simpler - especially if we drop the "STAT" field, as I don't think it is used.

This may need to be divided into two separate issues/questions:

Does an EPICS PV represent one alarm or many when integrated into our alarm system?
How do we handle heterogeneous active alarming messages?

EPICS PV Integration
An alarm is often discussed as a well-defined entity each with a unique name mapping to a fixed priority and logic type.  However, an EPICS PV in the strictest sense represents multiple alarms then as it can have dynamic alarm priorities (severities) and dynamic logic types (STAT field) associated with one PV name.  Currently EPICS messages are passed field-for-field and record-for-record as active alarming messages, but we might actually want to translate them from one EPICS PV update to one of many possible alarms.
Heterogenous Alarm Sources
Note: The above question about EPICS integration impacts this question because if we decide that we'll map EPICS PVs to multiple alarms and alarming and acks become simple messages (without attributes) and all attributes are then consolidated in the registered-alarms topic then this question becomes unnecessary.
We'd like to originate alarms from any source (such as scripts outside of EPICS), but we haven't nailed down exactly what the expectations are for the attributes attached to alarms from various sources.
There are actually two sets of attributes at the moment due to EPICS's way of alarming with attributes attached.  An alarm currently has:

Registered Alarm Attributes
Active Alarm Attributes

Attributes of a registered alarm with likely be a mix of shared items plus source specific (EPICS PV name must be included in registration of EPICS alarm, but not non-EPICS alarm).  The tricky part is the active alarm attributes.
Specifically, can the active alarming attributes vary, or must they be translated to match a single definition of what attributes an alarm must have?   For example, we could say that all alarms, regardless of source, must match the EPICS form - contain a STAT field and a SEVR field.  Or we could say all alarms must match some other single JLAB form that we define (then translate EPICS alarms into it).   Or, we can say that each alarm source can define it's own set of attributes (this is what the current prototype does).
We need to be careful not to duplicate attributes in the active alarm messages and registered alarm messages.  For example, we would like to put a "priority" field in the registered alarms topic (as non-epics alarms would like to set priority there perhaps), but EPICS defines priority (as severity) with its active alarming messages.

Active Alarm Message Attributes
Description
Pros
Cons

Fixed
We must define the one fixed format and then define translation rules for each source
Single schema for clients
Translation rules are difficult to agree on, fields lost in translation

Variable
We simply provide the messaging engine and forward messages field-for-field.
Nothing is lost in translation.
Clients must handle variable types

For completeness we should mention we could force all alarms to just look like EPICS alarms (include STAT and SEVR).   This isn't great because:

The STAT field is EPICS IOC specific, and even in EPICS we don't really use the STAT field right now, but we'd be forcing every alarm source to use it going forward
Attaching attributes to the active alarming messages seems to be the wrong place - each unique alarm should be given a unique name.  With EPICS a single PV name doesn't exactly tell you what the alarm is (I believe alarms are supposed to have a fixed, registered priority and type).

There likely isn't going to be much appetite for dividing each EPICS PV into something like (21 STAT fields) X (2 SEVR fields) = 42 different logical alarms (at least not manually), especially since it sounds like only a handful of STAT field value are even recognizable.   Perhaps we find some middle ground, like divide each EPICS PV into two alarms: one MAJOR and one MINOR?   Or we could come up with some wildcard syntax in the registration database that maps a single registration record to multiple logical alarms?  This could be similar (but also very different than) the Alarm Grouping feature?

Mapping an EPICS monitor update to one of two unique alarms (a MAJOR and a MINOR) is problematic in our current setup because we're using Kafka Single Message Transforms (SMT) - epics2kafka-alarms to serialize messages into the kafka-alarm-system format.     A Single Message Transform can only transform one-to-one for a given message and also is not supposed to maintain any state.   I believe we would need to transform a single EPICS monitor update message to two unique kafka-alarm-system messages and possibly maintain state.   There are two instances where this would be needed:

When a SEVR = NO_ALARM message is received the translation layer would need to write tombstone (no alarm) messages for both the MAJOR and MINOR alarms (OR otherwise would have to track state of which one was alarming).
A single PV cannot be both MAJOR and MINOR alarming simultaneously in EPICS, so the translation layer would need to tombstone the sister alarm when an alarming update is received (or clients would need to tolerate both MINOR and MAJOR at the same time, possibly masking MINOR with MAJOR)

The cost to redo this translation in a Kafka Streams app or to split it between the SMT and a new Kafka Streams app needs to be weighed with the benefit - a potentially cleaner schema.

We're sticking with EPICs alarm mapping to JAWS alarm 1-to-1 for now.   Can be re-opened later if we change our mind.


Service Request: thesixtyone

From Launchpad bug #1010057 by Joshua Kiley:
This is a radio style website that would fit well with Nuvola's app model. There's a sign in, and it plays ike a radio, of sorts.
Homepage: http://www.thesixtyone.com/

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

TheSixtyOne.com is listed for sale!


Update tools.md

Fix the format of line 84-88, Here should be a list in the original doc.
But after translation, the format is wrong.

/assign @dchen1107

/lgtm
/approve


Make atom ids available (and alterable) in biopython.structure

Sometimes it is neccessary to access and/or alter the atom ids in a pdb file. Right now this is not possible because atom ids are not stored when reading from pdb files and are assigned starting from 1when writing pdb files.
Imaging the following case: I want to read a protein structure from PDB and only save its CA atoms to a new file, but without altering their atom ids. Right now atom ids are reassigned starting from 1 and there is no way to alter this.
Possible implementation would be to add another field to biopython.structure.Atom which is initialized from the PDB file. biopython.structure.io.pdb.PDBFile#write could then allow to renumber atom ids or reuse the old ones:
pdb_file.set_structure(stack)
pdb_file.write("1l2y.pdb", renumber=True) # writes atom ids starting from 1 (current default)
pdb_file.write("1l2y.pdb") # keeps atom ids of the AtomArray

Fixed by 667312473c2a00f409481285ec63e49952060073
PDB and PDBx files have now full support of optional annotation arrays. In this particular case you can either load the structure from file with extra_fields=["atom_id"], which creates the atom_id annotation array from file, or you can set atom_id afterwards. When writing the loaded atom array or stack into a PDB or PDBx file, the atom_id annotation is automatically recognized and used instead of continuous numbering.


Prettier handling of generic function types is broken

Your Environment

Prettier version: tried from 2.4.1 to 2.6.2
node version: 14.17.0
package manager<EMAIL_ADDRESS>IDE: VScode

Describe the bug
I get errors with generic function types in .ts files: const fn = <T>() => {}.
Single type parameter T should have a trailing comma. Example usage: <T,>.

Then, if I add the trailing coma, Prettier removes it.
Trivago's original plugin works well, I get this problem only with this fork.
It may be related to Next.js babel config...
To Reproduce
yarn create-next-app@latest --typescript
yarn add @ianvs/prettier-plugin-sort-imports

Then reate a .ts file with the following declaration: const fn = <T>() => {}.

Thanks, can you also share your prettier config?

Hmm yeah, that's strange since we did not change anything related to parsing. If I take a look at the commits at https://github.com/trivago/prettier-plugin-sort-imports/commits/master, there also don't seem to be any additional commits since we created the fork. So I guess we need to take a look at what causes this different. And yes, a prettier config would be helpful.

I'm not able to reproduce this in my own project.  Would you be willing to share a minimal reproduction, @cvolant?

Also when I use the prettier online formatter, it removes the comma after type arguments.
Related:

https://github.com/babel/babel/issues/14361
https://stackoverflow.com/questions/71524043/single-type-parameter-t-should-have-a-trailing-comma-example-usage-t

It seems like maybe this is a babel error? https://github.com/babel/babel/issues/14361

@IanVS @blutorange thank you for your answers, and sorry for the delay before mine : I was off for some time.
I followed the babel lead and finally succeeded to make it work: it was a indeed a babel issue.
For some reason, @babel/parser was resolved to version 7.17.7. I added the following to my package.json to force it to resolve to version 7.17.10 and it worked (I use Yarn) :
"resolutions": {
  "@babel/parser": "7.17.10"
}


Microsoft.AspNetCore.DataProtection is v3 not compatible with azure functions anymore

according to this: https://github.com/Azure/azure-functions-host/issues/5447
DataProtection since version 3 is not compatible with Azure Functions anymore (v2 is fine), I think the documentation should mention it...

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

ID: e7425f4e-80fe-7972-7e7f-83948811c078
Version Independent ID: ec6921d2-b16c-d1af-1267-388e08599a89
Content: Compatibility in ASP.NET Core
Content Source: aspnetcore/security/data-protection/compatibility/index.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hmm @Rick-Anderson I'm confused. Your link states that "DataProtection.AzureStorage uses new Azure Storage APIs" and nothing about Microsoft.AspNetCore.DataProtection being incompatible with AF v3.

according to this: Azure/azure-functions-host#5447
DataProtection since version 3 is not compatible with Azure Functions anymore (v2 is fine), I think the documentation should mention it...
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e7425f4e-80fe-7972-7e7f-83948811c078
Version Independent ID: ec6921d2-b16c-d1af-1267-388e08599a89
Content: Compatibility in ASP.NET Core
Content Source: aspnetcore/security/data-protection/compatibility/index.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

I was able fix this issue while using IWebJobsStartup instead of FunctionsStartup. The issue seems to be IFunctionsHostBuilder DI container


[YUNIKORN-108] Only memory and vcore is displayed on the web UI

What is this PR for?
The WebUI currently displays only "vcore" and "memory" resources. To improve flexibility, I added a new parameter to the SchedulerResourceInfo interface, enabling dynamic retrieval of various resource types. This allows the WebUI to display different resources based on requirements.
Additionally, I included GPU resources from three vendors in db.json to demonstrate handling diverse resource types in the WebUI.
What type of PR is it?

[x] - Improvement

What is the Jira issue?

Open an issue on Jira https://issues.apache.org/jira/browse/YUNIKORN-108/

How should this be tested?
Screenshots (if appropriate)

Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

Added reviewers and kicked of the workflow.
Can you please check the test failure?

Hi @wilfred-s  @wusamzong , sorry for late response.
I have been preparing for my compulsory military service which will spend me four months.
I can keep reviewing codes before august 15th.
Thanks for @wusamzong contributions and I think we can merge the code in the next commit.

Thanks to reviews by @wilfred-s and @0yukali0, I've resolved the test error and now label empty resources as "n/a."

Codecov Report

Merging #127 (2d915e3) into master (91fe955) will not change coverage.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #127   +/-   ##
=======================================
  Coverage   66.66%   66.66%
=======================================
  Files           1        1
  Lines          30       30
=======================================
  Hits           20       20
  Misses          7        7
  Partials        3        3

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

This gives us the base change for displaying more detail in resources.
I created YUNIKORN-1914 to look at a nicer option for the use of the limited screen space we have.


unable to run sql even in the sql tutorial

Describe the bug
just trying to run the marimo sql tutorial doesn't work, I have already install the requirements as pip install 'marimo[sql]', In one of the first cells the notebook says the installation is correct. but its unable to run the tutorial for sql

don't really know what is the problem, I tried by myself as the documentation indicates and gives me the exact same error.
CatalogException

This cell raised an exception: CatalogException('Catalog Error: Table with name df does not exist! Did you mean "pg_depend"?')
See the console area for a traceback.
Traceback (most recent call last):
  File "/Users/danielluque/anaconda3/lib/python3.11/site-packages/marimo/_runtime/executor.py", line 170, in execute_cell
    exec(cell.body, glbls)
  Cell marimo:///var/folders/rp/__ndnbkj7d364mdw_r8l25m80000gn/T/tmp5in93ni2/sql.py#cell=cell-9, line 1, in <module>
    _df = mo.sql(
          ^^^^^^^
  File "/Users/danielluque/anaconda3/lib/python3.11/site-packages/marimo/_sql/sql.py", line 40, in sql
    relation = duckdb.sql(query=query)
               ^^^^^^^^^^^^^^^^^^^^^^^
duckdb.duckdb.CatalogException: Catalog Error: Table with name df does not exist!
Did you mean "pg_depend"?

Environment
{
"marimo": "0.8.13",
"OS": "Darwin",
"OS Version": "22.5.0",
"Processor": "arm",
"Python Version": "3.11.3",
"Binaries": {
"Browser": "128.0.6613.120",
"Node": "v22.2.0"
},
"Dependencies": {
"click": "8.1.7",
"importlib-resources": "missing",
"jedi": "0.18.1",
"markdown": "3.7",
"pygments": "2.15.1",
"pymdown-extensions": "10.9",
"ruff": "0.6.4",
"starlette": "0.33.0",
"tomlkit": "0.13.2",
"typing-extensions": "4.5.0",
"uvicorn": "0.24.0.post1",
"websockets": "12.0"
},
"Optional Dependencies": {
"altair": "5.3.0",
"duckdb": "1.1.0",
"pandas": "2.1.1",
"pyarrow": "11.0.0"
}
}
Code to reproduce
marimo tutorial sql

it looks like the latest release of duckdb today (1.1.0) broke our integration.
can you re-install version 1.0.0. e.g uv pip install -U duckdb==1.0.0 or pip install -U duckdb==1.0.0

awesome!, it was indeed the new version of duckdb, thanks!


SOLR-17041 Make CommitTracker currentTlogSize lazy

https://issues.apache.org/jira/browse/SOLR-17041

Description
Switching to lazy provider for the tlog size
Solution
Please provide a short description of the approach taken to implement your solution.
Tests
Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.
Checklist
Please review the following and check all that apply:

[ ] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[ ] I have created a Jira issue and added the issue ID to my pull request title.
[ ] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[ ] I have developed this patch against the main branch.
[ ] I have run ./gradlew check.
[ ] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

thank you for the review @cpoerschke! I will be merging this shortly


Outdated debug version with known vulnerabilities

Nsp checker reporting that you are using old debug package version which includes outdated ms package with known vulnerability:
https://nodesecurity.io/advisories/46

The version we are using is new. It is the latest 2.x with the ms patch.


Monolog LogPlugin does not work

Getting the following error when trying to add a MonoLog subscriber.
Warning: parse_url() expects parameter 1 to be string, object given
500 Internal Server Error - ContextErrorException
$adapter = new MonologLogAdapter($logger);
$logPlugin = new LogPlugin($adapter, MessageFormatter::DEBUG_FORMAT);

$this->httpClient = new Client([
    'base_uri' => $this->baseUri,
    'timeout'  => 2.0,
    'cookies'  => true
]);

// Attach the plugin to the client, which will in turn be attached to all
// requests generated by the client
$this->httpClient->addSubscriber($logPlugin);

Hi @samsamm777,
What version of Guzzle are you using?

@jeskew  6.0.*

The call to addSubscriber looks like Guzzle 3 code. It's getting caught by __call in a Guzzle 6 client, which assumes that the first argument passed in is a URI.
Guzzle 6 doesn't use an event emitter anymore, as PSR-7 messages are immutable (and most Guzzle events modified existing request or response objects). Instead, you can use handlers and middleware. Let me see if there's any kind of premade monolog middleware.

Ahh my mistake. Was reading the old docs it looks like! I've found the 'debug' option for now which may suffice for the time being. I'll look into finding/writing a monolog middleware in due course. Thanks

There is already a PSR-3 compatible logging middleware in Guzzle: https://github.com/guzzle/guzzle/blob/master/src/Middleware.php#L184


export intensities, regionprops and neighbors folders

HI,
thanks for the great tool. I am trying to run cell segmentation on IMC data (using CP) but the 'intensities','reionprops' and 'neighbors' folders are not being exported to the output folder as expected (even though the corresponding fields in the export tab are selected). Any ideas why this is might happening?
Thanks!

Hi @sokratiag,
if you want to export these measurements directly when segmenting, you need to tick the box "Run Steinbock post-processing" in the Segmentation tab. If you want to extract measurements after segmentation, you need to click on "Run Steinbock postproc" in the Export tab. Did you do any of those? Just ticking the fields in the Export tab is not sufficient. In case something is not clear you can find the procedure here: https://guiwitz.github.io/napari-steinpose/Standard_use.html#post-processing


Ability to disable agent from the self-hosted agent itself

I think that there should be a way to tell Azure DevOps from the agent (to be more specific, from the machine on which self-hosted agent is installed) - "I do not want to accept any more jobs".
Right now I do not see an easy solution for that, svc.sh script allows only to install, start, stop, display status and uninstall.
Use case: I have an autoscaling group of virtual machines. When I scale up, a new VM is created and during its initialization Self-hosted Linux agent is installed on it, configured and started. The agent connects to my pool and everything is fine. Except when I scale down, running builds can be terminated which is not acceptable for me. Before scaling down my agents, I would like to disable them, wait some time and then safely terminate vms. The only way to achieve this now is to use Azure DevOps Services REST API but that requires at least 3 request->response->parse trips to do it in a nice way, without hardcoding any ids (GET pool with a name 'P', take its id -> GET agent 'A' from pool id 'Pid', take its id -> PATCH agent id 'Aid', disable it). It does not make much sense to me as on the vm where agent is running you already have established, authenticated connection with Azure DevOps, so sending there disable command sounds like an easy thing to implement.
Cloud providers in autoscaling groups allow executing lifecycle hooks, wait a certain amount of time and then terminate vm, with ./svc disable in place that would be a great time saver.

Thanks for the suggestion! The product teams aren’t always watching this repository, so to make sure it gets in front of them for triage, please submit your idea here.


Simplify interactions spec

Overview
What we currently call the 'interrogationBundle' can be simplified to an array of 'interactions'. Let's also change 'parameter' to 'name' so we can better match the CLI spec.

Work for this issue will begin begin on branch work-liquid-labs/question-and-answer/102.


Firebase.InstanceId v4.2.1 fails to resolve "Firebase.InstanceID.InstanceIdResult"

Error Given:
MTOUCH: Error MT2002: Failed to resolve "Firebase.InstanceID.InstanceIdResult" reference from "Firebase.InstanceID, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null" (MT2002)

From Build Output:
Target _CompileToNative:
    /Library/Frameworks/Xamarin.iOS.framework/Versions/Current/bin/mtouch<EMAIL_ADDRESS>11-13.2/response-file.rsp" --registrar:static "--gcc_flags=-ObjC -ObjC -lc++ -lsqlite3 -lz -ObjC -lc++ -lsqlite3 -lz -ObjC -lsqlite3 -lz -ObjC -lsqlite3 -Xlinker -sectcreate -Xlinker __TEXT -Xlinker __entitlements -Xlinker \"/Users/schultzjosephj/dev/catholify-xa/Catholify.iOS/obj/iPhoneSimulator/Debug/device-builds/iphone 11-13.2/Entitlements.xcent\""
    MTOUCH : error MT2002: Failed to resolve "Firebase.InstanceID.InstanceIdResult" reference from "Firebase.InstanceID, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null"
Done building target "_CompileToNative" in project "Catholify.iOS.csproj" -- FAILED.

I have tried cleaning, rebuilding, adding and removing the reference, and restarting Visual Studio. Nothing seems to be working.
Any help would be appreciated!

@acupajoe Firebase Instance ID v4.3.0 has been released. Can you update and try again, please? Let us know your results. Best!

We are closing this issue since no response was received.  Please open a new issue referring to this one if you have more information.


lifx-notifier 1.0.2

New Pull Request Checklist
Please mark the points which you did / accept.

[X] I read the Step Development Guideline
[X] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[X] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[X] I read and accept the Abandoned Step policy

1.0.2 version of the step includes https://github.com/vgaidarji/steps-lifx-notifier/pull/4

@viktorbenei https://github.com/bitrise-io/bitrise-steplib/pull/1144 done.
Changed utils to utility.

This PR can be closed in favor of #1144.


Subsequnces recursion

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Python Program to print all the possible subsequences of the string, using recursion.
“abcd” has following subsequences “”, “d”, “c”, “cd”, “b”, “bd”, “bc”, “bcd”, “a”, “ad”, “ac”, “acd”, “ab”, “abd”, “abc”, “abcd”
Example:
Input : abcd
Output:
d
c
cd
b
bd
bc
bcd
a
ad
ac
acd
ab
abd
abc
abcd
Checklist

[Y] I've read the contribution guidelines.
[Y] I've checked the issue list before deciding what to submit.
[Y] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#1152

kindly review this PR @iamrajiv @Kajol-Kumari @SKAUL05

kindly add hacktoberfest accepted label on this PR ,this PR is counted towards hacktobefest. @iamrajiv @plazzy99


chore: adding this to the README

          Thanks @bradmccoydev would you mind adding this to the `README` in the table of available helm values?

Originally posted by @AlexsJones in https://github.com/k8sgpt-ai/k8sgpt-operator/issues/51#issuecomment-1531266623

PR done for this here: https://github.com/k8sgpt-ai/k8sgpt-operator/pull/55


Can I get crontab formated string from compiled schedule ?

var sched = later.parse.recur().on(10).minute().on(8).hour()
is there any way to get cron sting like "10 8 * * *" from shedule above?

+1

No, there has been talk of various people implementing this but I don't think anything has come from it. Being an API for generating cron statements was never a goal (the fact that it supports cron at all is kind of a fluke, personally I've never even used that functionality).

Sad to see this issue is closed. For the life of me I cannot find a library that converts natural language to cron strings. This library is sooo close.

@krazyjakee a few years ago, I managed to get something working using https://github.com/azza-bazoo/prettycron
It was able to take strings like "* * 25 12 *" and convert it to a cron job, and it would display the schedule in human-readable form "Every minute on the 25th in December"
It was able to handle anything thrown at it, and got a lot of love at RSA that year when it was part of the demo for the company I was with at the time.
Using the nextRun function, it could also display to the user when the next run would happen... it was pretty easy if I remember right.
Check it out! Hope this helps!

I'm looking for the opposite. To take a human readable string and output a Cron string like in this issue description.


Agree one name of our product for end user

In different places we are using different name of our product (git-machete, Git Machete, Git-Machete, and so on) esp. in logging messages and exceptions. We must agree one name and change to this name every end user's message.

Decided on Git Machete (Title Case, as recommended in https://www.jetbrains.org/intellij/sdk/docs/basics/plugin_structure/plugin_configuration_file.html)


Adding CheckStyle to CI

These changes make the build fail when the checkstyle poilcy detects that the rules are not followed. The elipse formatter is to accomany and complement the checkstyle rules.

@ibuildthecloud @cjellick This seems to be inline with what we discussed. When the files are formatted on my system there are about 200 errors

And 200 out of 6300 ain't so bad.

Yeah.


ios 15. x support

is there any chance to have iOS app running on 15. x ios? i know it's pretty old but still working

I'm sorry lazyesti,
i can't support iOS 15.x for iGotify, because i use the new Navigation system and the minimum requirement for that is iOS 16.


Dynanimate Camera Position With Fixed Target not working

Strange, because the other tow camera related nodes are working.

Fixed with fe4711e3645cdd88d7f4f27b1767d96e39580777


Glsl editor

It will re-use the editor if you are editing a single effect. if you are editing different effects it will spawn up to 1 editor per effect. I also was thinking about allowing a different text editor to be spawned. It shouldn't be too hard to implement that.

You can merge


Adding Lots to Consumption RP

Adding Lots to Consumption RP
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-js

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Can one of the admins verify this patch?

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-node

Adding @ravbhatnagar for ARM review.
According to @sandeepkatumalla - there is only one new API in the spec to review, as identified by his comment.

Automation for azure-sdk-for-go


Remove added ignore errors

Remove added ignore errors to ensure error detection when running the playbook

Fine, you can merge


Update to AEM WCM Core Components 2.24.6

no code changes needed for 2.24.6, skipping a new release for this


OpenAI just changed the website, paywrigth doesn't detect elements anymore

Oops, we broke!
Testing changes

fixed in 0.1.1


using license as environment variable

Currently the action to use commands making use of the license need to have it passed as an env var even if it is already present as a config arg.
This is not affecting ci/cd since the license is always passed already, but running it manually leads to a cumbersome command:
LICENSE_KEY=${LICENSE_KEY} newrelic-integration-e2e --commit_sha=test-string --retry_attempts=5 --retry_seconds=60 --license_key...

This issue was closed because it has been inactive for a long time despite it can be reopened at any point if you think it still relevant.
Thank you for your contribution!


Fix: make sidekiq instrumentation rake task compatible with TruffleRuby

Context
This PR starts to fix issues related to supporting TruffleRuby (added in https://github.com/open-telemetry/opentelemetry-ruby-contrib/pull/59) by fixing the Sidekiq implementation test issues.
This should fix https://github.com/open-telemetry/opentelemetry-ruby-contrib/issues/46
Fixing Sidekiq Instrumentation issues in testing
This commit fixes the following error from the Rakefile due to using fork for starting
redis:
NotImplementedError: fork is not available

According to the TruffleRuby compatibility docs, fork is not compatible in TruffleRuby:

You cannot fork the TruffleRuby interpreter. The feature is unlikely to ever be supported when running on the JVM but could be supported in the future in the native configuration.

After fixing that issue, I was able to successfully run a passing test suite for the Sidekiq implementation.

It looks like the TruffleRuby tests are also passing in the CI workflow, so PR is ready for review! 👍


UI external signer support (e.g. hardware wallet)

Big picture overview in this gist.
This PR adds GUI support for external signers. It consists of 8 commits on top of bitcoin/bitcoin#16546 (RPC).
The UX isn't amazing - especially the blocking calls - but it works.
First we adds a GUI setting for the signer script (e.g. path to HWI):

Then we add an external signer checkbox to the wallet creation dialog:

It's checked by default if HWI detects a device. It also grabs the name. It then creates a fresh wallet and imports the keys.
You can verify an address on the device (blocking...):

Sending, including coin selection, Just Works(tm) as long the device is present. It's a bit weird though: the device will prompt for signing first and the QT confirmation dialog appears after.
External signer support is enabled by default when the GUI is configured and Boost::Process is present.

The original https://github.com/bitcoin/bitcoin/pull/16549#issuecomment-626652045 has a Concept ACK from @fjahr.
I'll split out the interface changes and PR them to bitcoin/bitcoin later.

Shouldn't the external signer program be per-wallet?

@luke-jr in practice there's only 1 program out there: HWI. It already knows what to do for different wallets based on their fingerprint. So, unless more such tools show up, I prefer to wait until we can store arbitrary settings in the wallet: https://github.com/bitcoin/bitcoin/issues/13044 and then keep -signer as the default.

Hey Sjors,
I could not see the signer script UI in the wallets settings when I compiled.

@Bosch-0 oops, I accidentally lost a commit in my rebase fury. Try again.

Awesome, that worked! Having some issues with HWI though. After installing and setting the path to the hwi.py file I keep getting an exception error when creating a wallet.
How would we overcome having to install HWI separately? Would it have to be something like shipping HWI alongside the GUI. I'm a not too familiar with this process but having the path set by default behind the scenes would be ideal from a UX perspective.

What's the error you're seeing?
Also, can you try from the command-line: hwi.py enumerate to see if HWI works standalone? This PR will certainly need followups to make it more user friendly.

Here is the error, though I think HWI is not installed properly.

Testing is being done on windows 10.
Installation appears to be successful when running pip3 install .

When running ./hwi.py enumerate I get the below error. Tried the other dependency installation methods as well as installing libusb-1.0.dll and including it in HWI directory but no luck yet.

Bitcoin QT shouldn't crash like this, so that's a bug in this PR. But for the problem with HWI itself, I suggest opening an issue in the HWI repository. We don't do a lot of Windows testing, so that's useful (cc @achow101).

Concept ACK.
@Sjors you mention blocking calls, are you going to fix that here?

Bitcoin QT shouldn't crash like this

Agree, I'm still on the base branch though.

I'm not very good at multi-threading stuff, so I might leave that problem to others, either via followup PR or cherry-picking a solution.

I'm not very good at multi-threading stuff, so I might leave that problem to others, either via followup PR or cherry-picking a solution.

A lot of current GUI multithreading code is stupidly complicated. I wrote a utility function AsyncUpdate(async_fn, update_fn) to run slow blocking code and then update the GUI in https://github.com/bitcoin/bitcoin/commit/39c5492fabab52a888e8bc5303877ff6e4cce6f8 (example usage https://github.com/bitcoin/bitcoin/commit/eacd34144c1ee989c9ab43eb439fecca76addc76 and description https://github.com/bitcoin/bitcoin/issues/16874#issuecomment-632424392). AsyncUpdate takes two lambdas. The first lambda runs asynchronously in a thread, and when it finishes, its return value is passed as an argument to the second lambda which runs synchronously. The first lambda can run slow and blocking code and take long as it wants without freezing the GUI, while the second lambda can take the result and update the GUI. Restrictions are straightforward: The first lambda can't access widgets directly (it can only read initial values, return a final result, and send signals if it wants to show progress messages or update a progress bar). The second lambda needs to be fast and not block, but it can directly update and access Qt widgets with the final result.

@ryanofsky your AsyncUpdate is fine when async function doesn't require user input, doesn't give feedback like progress, or both, possibly multiple times. Otherwise, in these cases it would have to call multiple AsyncUpdate and then it starts to be a mess chaining those.
AsyncUpdate is also fine when there's just one outcome, otherwise the async function needs to return a variant with each possible outcome and then the update function would have to check the effective result.
There's also the case where the user wants to interrupt the async function, like quit, and in this case it wouldn't work.
Another interesting problem with asynchronous updates are bursts, and usually most async calls should be avoid/debounced. I think we have a couple of these cases.

Lastly, I'd love to improve those stupidly complicated cases so we can use a consistent approach everywhere, and therefore anyone could feel confident and unafraid of asynchronous tasks.

re: https://github.com/bitcoin-core/gui/pull/4#issuecomment-732290867
@promag a lot of that comment isn't true, and none of it is a reason not to write blocking code instead of calling AsyncUpdate.
Writing an AsyncUpdate call is about as easy as writing blocking code, and it prevents the GUI from freezing while an operation is running. The only major thing it doesn't support currently is cancel notification (but it could take an optional cancel_fn argument in the future). So if the parent widget is deleted right now, GUI freeze will happen at the point of deletion. This is still better than freezing during the entire operation.
Almost everything in https://github.com/bitcoin-core/gui/pull/4#issuecomment-732290867 is misleading. I would not suggest chaining AsyncUpdate calls: the only type of usage that makes sense is to having one AsyncUpdate call per long running operation. If there is a sequence of serial operations, they should all be in the same lambda. Supporting progress messages and progress updates is supported and trivial with signals. Input is possible with atomics, message queues, signals, or any other way you could think of passing data to an asynchronous thread. I don't know what you are suggesting regarding multiple outcomes. Any outcome that can be represented as an object that be handled in the async_fn/update_fn paradigm, and if you are really lazy and don't want to specify an outcome type you can even have the async_fn return a std::function capturing arbitrary state and have the update_fn call that.
There are cases where you might prefer a more complicated approach over calling AsyncUpdate, like if you want a thread pool. But there should be not many cases where freezing the UI thread is preferable to calling AsyncUpdate. And there is existing GUI code that could be simplified and made more responsive with AsyncUpdate.

checking whether to build with external signer support... configure: error: "External signer support requested but requires Boost.Pocess"
Boost.Process?

@RandyMcMillan what operating system are you on? On macOS and Linux Boost Process comes with Boost, but on Windows it's a separate package.

I'm so proud of this program like i was saying I've been trying to find the right wallet but I'm trying to do big transfers you know but I'm trying to do only one address and its a big one its in the billions my friend i still need help with this here wallet buddy thanks .

I'm so proud of this program like i was saying I've been trying to find the right wallet but I'm trying to do big transfers you know but I'm trying to do only one address and its a big one its in the billions my friend i still need help with this here wallet buddy thanks .

Love you guys man I still honestly trying to create a wallet and haven't started any transfers and need to be able to find me a. Wallet but yeah what is tezor t not heard of it thanks for the stop on that one buddy really appreciate that one some thing good let's keep our address safe bud from attackers like that is there way I can report this attacker who uses tenor t

Love you guys man I still honestly trying to create a wallet and haven't started any transfers and need to be able to find me a. Wallet but yeah what is tezor t not heard of it thanks for the stop on that one buddy really appreciate that one some thing good let's keep our address safe bud from attackers like that is there way I can report this attacker who uses tenor t

I'm surprised the master branch passes given this error:

@MarcoFalke does bitcoinbuilds.org not test a build with --with-boost-process (Travis did when I added RunCommandParseJSON)

I'm surprised the master branch passes given this error:

@MarcoFalke does bitcoinbuilds.org not test a build with --with-boost-process (Travis did when I added RunCommandParseJSON)

@jonasschnelli I believe the bitcoinbuilds.org Win64 machine needs to add --without-boost-process --disable-external-signer to its BITCOIN_CONFIG. I don't know if that causes problems when building stuff before this PR. Alternatively it should not ignore ci/test/00_setup_env_win64.sh.
https://bitcoinbuilds.org/?build=7162

@jonasschnelli I believe the bitcoinbuilds.org Win64 machine needs to add --without-boost-process --disable-external-signer to its BITCOIN_CONFIG. I don't know if that causes problems when building stuff before this PR. Alternatively it should not ignore ci/test/00_setup_env_win64.sh.
https://bitcoinbuilds.org/?build=7162

rpc_help test needs updating

I'll rebase again after https://github.com/bitcoin/bitcoin/pull/21467 (which should also fix the rpc_help failure)

On Sun, 11 Apr 2021 at 16:14, Ronnie Sherfey @.***>
wrote:

@RonSherfey requested changes on this pull request.
specified changes added
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bitcoin-core/gui/pull/4#pullrequestreview-633023883,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLQ46TUL6KN2RICTSBDMCDTIG4EXANCNFSM4OCAZK5A
.

Onwards and upwards --
Linda c

Rebased after #21666.

🐙 This pull request conflicts with the target branch and needs rebase.
Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft".

I moved the commit to turn this feature on for GUI builds to https://github.com/bitcoin/bitcoin/pull/21935

Concept ACK, will review

Repeating comment from @ryanofsky here:

Would be nice to drop #ifdef ENABLE_EXTERNAL_SIGNER in the interface here and just return an empty vector in the implementation or throw a not-implemented exception. Cross-process interfaces are trying not to be conditional to avoid build complexity and fragile incompatibilities between builds.
Also maybe in general in this PR, preprocessor is a little more widely used that it needs to be. Preprocessor is great when you need to write platform specific fragments or cut out a significant external or internal dependency like boost or the wallet. But if you just want to act conditionally, normal code is a good way to go. I'd try to avoid conditional compilation except for the parts of the code directly depending on boost.

It's difficult to get rid of #ifdef ENABLE_EXTERNAL_SIGNER, because most of the time the code refers to ExternalSigner which is not defined (since https://github.com/bitcoin/bitcoin/pull/21666/commits/54569cc6d6f54788169061004026e62e1c08440e). That also applies to node.h.

I added a commit to reduce #ifdef ENABLE_EXTERNAL_SIGNER usage to  bitcoin/bitcoin#21935.

Big concept ACK.

Rebased an addressed @meshcollider's comments.

🐙 This pull request conflicts with the target branch and needs rebase.
Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft".

Rebased and slightly clarified the last commit.

Code Review ACK 1c4b456e1a0ccf0397d652f8c18201c3224c5c21
Mostly reviewed the code, did a little bit of testing. I noticed that when HWI is doing something, there is no indication to the user that it is. It would be nice if there was some dialog box that said something was happening, otherwise there can be a minute or two where Core doesn't indicate anything is happening, and nothing appears on the device.

post-merge ACK, wanted to document that I tested this out with a coldcard and the process was smooth.

@Sjors I tried this with my trezor1, I unlocked it on the command line on a wallet with a bip39 password, this is what pops up when I click create wallet:

hwi --stdinpass --device-type=trezor --device-path webusb:001:14 enumerate
[
  {
    "type": "trezor",
    "path": "webusb:001:14",
    "model": "trezor_1",
    "needs_pin_sent": false,
    "needs_passphrase_sent": false,
    "fingerprint": "67e2434a"
  }
]

dunno if I'm doing something wrong

Mmm, I'm not sure how the passphrase flow works. @achow101?
I might be that we need to add support for that, e.g. if needs_pin_sent or needs_passphrase_sent returns true, we prompt for that.
However in your case those are false.

The passphrase should be cached by the device, so if you try again after running HWI manually, it should work. But we do need to implement a passphrase workflow for needs_passphrase_sent: true.

@achow101 I've never been able to get the trezor_1 passphrase cached with hwi, I've always had to do --stdinpass on every command. hmm maybe I need to update firmware or something...

Oh right, Trezor 1 doesn't have that, only Trezor T.

@achow101 fwiw it caches fine when using trezorctl, dunno what they are doing.

@Sjors
Could you add release notes into https://github.com/bitcoin-core/bitcoin-devwiki/wiki/22.0-Release-Notes-draft#gui-changes?

@hebasto done, also for the RPC.


added 0110 balanced binary tree simplified java solution

File(s) Modified: 0110-balanced-binary-tree.java.
Language(s) Used: java
Submission URL: _https://leetcode.com/problems/balanced-binary-tree/submissions/1128571374/_

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

@MHamiid, may I request for your review on this PR ?


Add binary accuracy

This is the function to compute accuracy of binary classification. (typically used with sigmoid_cross_entropy)

I updated the PR based on the review comments.

Could add test case where function correctly raise InvalidType error when the shapes of x and t are not same.

updated

LGTM :+1:


Unable to build image in production

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
Hello everyone,
Our builds have started failing with the message:
I can't figure out which part is failing.
Step 18/18 : COPY --chown=node:node --from=production /home/node/app/build /usr/share/nginx/html/
unable to convert uid/gid chown string to host mapping: can't find uid for user node: no such user: node

> docker version
Client:
 Cloud integration: v1.0.28
 Version:           20.10.17
 API version:       1.41
 Go version:        go1.17.11
 Git commit:        100c701
 Built:             Mon Jun  6 23:09:02 2022
 OS/Arch:           windows/amd64
 Context:           default
 Experimental:      true

Server: Docker Desktop 4.11.1 (84025)
 Engine:
  Version:          20.10.17
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.17.11
  Git commit:       a89b842
  Built:            Mon Jun  6 23:01:23 2022
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.6
  GitCommit:        10c12954828e7c7c9b6e0ea9b0c02b01407d3ae1
 runc:
  Version:          1.1.2
  GitCommit:        v1.1.2-0-ga916309
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Dockerfile
## Base ########################################################################
# Use a larger node image to do the build for native deps (e.g., gcc, python)
FROM node:lts as base

# Reduce npm log spam and colour during install within Docker
ENV NPM_CONFIG_LOGLEVEL=warn
ENV NPM_CONFIG_COLOR=false

# We'll run the app as the `node` user, so put it in their home directory
WORKDIR /home/node/app
# Copy the source code over
COPY --chown=node:node /site/ /home/node/app/

## Development #################################################################
# Define a development target that installs devDeps and runs in dev mode
FROM base as development
WORKDIR /home/node/app
# Install (not ci) with dependencies, and for Linux vs. Linux Musl (which we use for -alpine)
RUN npm install
# Switch to the node user vs. root
USER node:node
# Expose port 3000
EXPOSE 3000
# Start the app in debug mode so we can attach the debugger
CMD ["npm", "start"]

## Production ##################################################################
# Also define a production target which doesn't use devDeps
FROM base as production
WORKDIR /home/node/app
COPY --chown=node:node --from=development /home/node/app/node_modules /home/node/app/node_modules
# Build the Docusaurus app
RUN npm run build

## Deploy ######################################################################
# Use a stable nginx image
FROM nginx:stable-alpine as deploy
# Switch to the node user vs. root
WORKDIR /home/node/app
# Copy what we've installed/built from production
COPY --chown=node:node --from=production /home/node/app/build /usr/share/nginx/html/

Thanks in advance
Reproducible demo
No response
Steps to reproduce

build Dockerfile

Expected behavior
Build success
Actual behavior
Build failed
Your environment

Public source code:
Public site URL:
Docusaurus version used:
Environment name and version (e.g. Chrome 89, Node.js 16.4):
Operating system and version (e.g. Ubuntu 20.04.2 LTS):

Self-service

[ ] I'd be willing to fix this bug myself.

Our builds have started failing with the message:

What changed? 🤷♂

Step 18/18 : COPY --chown=node:node --from=production /home/node/app/build /usr/share/nginx/html/
unable to convert uid/gid chown string to host mapping: can't find uid for user node: no such user: node

This is a Docker copy issue, this error is not directly related to Docusaurus, so I'm closing

Our template doesn't come with a dockerfile, and we never mentioned it in our docs either.

We did provide a Dockerfile but it was a long time ago for Docusaurus v1:

https://github.com/facebook/docusaurus/blob/docusaurus-v1/packages/docusaurus-1.x/examples/basics/Dockerfile
https://v1.docusaurus.io/docs/en/docker

And it doesn't even look like your Dockerfile:
FROM [node:lts](https://hub.docker.com/_/node)

WORKDIR /app/website

EXPOSE 3000 35729
COPY ./docs /app/docs
COPY ./website /app/website
RUN yarn install

CMD ["yarn", "start"]

v2 doesn't provide any Dockerfile. If you want to use Docker, it's your responsibility to make it work

My Try: https://hub.docker.com/repository/docker/zhangkin/docusaurus/general

Run to start container:

with -v to your docusaurus folder, you can download the file codes here: https://github.com/facebook/docusaurus/releases
with -net=host all the port will be same as the host env

docker run -it --net=host -v /home/kin/Documents/Wiki/docusaurus-2.3.1:/docusaurus --name wiki_docs zhangkin/docusaurus /bin/zsh

inside the container run the yarn install and start the server
yarn install & npm start

After successfully build, you can open a browser and type http://<IP_ADDRESS>:3000/ to see the website now!


Update the synchronous work from queue when current work is completed

Description

Fixes issue #27578

Changes

Correct check condition inside queueWork(). If currentWork is not terminated, the new coming work should be place into workQueue.
When REQUESTED == 0L (CurrentWork is completed), update CurrentWork to request new message.

Detailed logic
Below diagram shows when call receiver client to receive message:

Below diagram shows what if current work is completed or timeout:

Note that the changes are in green color.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

Is it possible to have a test for this bug fix?

Is it possible to have a test for this bug fix?

Yes, sure. I'll add one.


Better emscripten support

The first commit makes the following changes:

Includes config.mk after defining SUBDIRS so that config.mk can override it (so that emscripten doesn't build iceprog)
Adds an "EMCC" flag to build with emscripten.
Allows the debug metadata level to be controlled because emscripten does not support -ggdb.
Adds $(EXE) in various places
Adds a hack to the icetime makefile so that it copies over the chipdb files and embeds them into the resulting javascript

The second commit modifies all binaries to mount emscripten's NODEFS on /x if the resulting javascript is run under nodejs. This is useful for testing so that the resulting javascript can interact with files on the host filesystem. By default, emscripten-compiled output can only access its in-memory virtual filesystem. It also isn't possible to mount additional emscripten filesystems over the root, so as a compromise NODEFS is mounted on /x. After this change, it is possible to run commands like
$ nodejs icepll/icepll.js -f /x/test.v
$ nodejs icepack/icepack.js /x/in.asc /x/out.bin

This will interact with files test.v, in.asc, and out.bin in the current working directory.

I've now merged your first commit. Regarding the second one: Maybe we can use a better name than '/x' for the mount point?

Finally getting back to this. When you merged the arachne-pnr PR, you let the  /x path slide. Should we change both that and this? How about /hostfs instead?

Yes, /hostfs is much better. I'm now merging this, and I will also merge an arachne-pnr PR that changes the path to /hostfs there if you create one.

Oh, you already created an arachne-pnr PR. Merged.


Removing error pop-ups checking for event_id

Signed-off-by: Aditya Jindal<EMAIL_ADDRESS>Description
Currently, the frontend code is being checked for an event_id in response to the Send Notifications Response. This PR removes the check for the event_id.
Issues Resolved
https://github.com/opensearch-project/notifications/issues/441
Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #456 (b2518e6) into main (dd8f9f5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #456   +/-   ##
=========================================
  Coverage     70.71%   70.71%
  Complexity      108      108
=========================================
  Files           124      124
  Lines          3923     3923
  Branches        626      626
=========================================
  Hits           2774     2774
  Misses          979      979
  Partials        170      170

Flag
Coverage Δ

dashboards-notifications
86.40% <ø> (ø)

opensearch-notifications
61.21% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd8f9f5...b2518e6. Read the comment docs.

Frontend Screenshots for the error message:


Paper for the code and the instructions for running it

hi Moji,
Hope you are doing well.
Wanted to check if this repo is a code base for the paper "SingNet: A Real-time Singing Voice Beat and Downbeat Tracking System". If yes, is this a complete code?
Further, would you be able to add an example/instructions to run the code?
Help would be much appreciated.
Thanks
Kewlcoder

Hi Kewlcoder,
PLease send the request to @.***
Thank you in advance.
Moji
On Sun, Jul 23, 2023 at 3:49 PM kewlcoder @.***> wrote:

hi Moji,
Hope you are doing well.
Wanted to check if this repo is a code base for the paper "SingNet: A
Real-time Singing Voice Beat and Downbeat Tracking System". If yes, is this
a complete code?
Further, would you be able to add an example/instructions to run the code?
Help would be much appreciated.
Thanks
Kewlcoder
—
Reply to this email directly, view it on GitHub
https://github.com/mjhydri/Singing-Vocal-Beat-Tracking/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWBSZM6QRRWMQGOTCMQOYDXRV54PANCNFSM6AAAAAA2UWUIIA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Moji,
I have received your message with the EMAIL ID shared as REDACTED. Check the attached screenshot.

Gentle reminder @mjhydri  :) !

Hi there,
I hope you are well and apologize for my late reply. I recently got back
from a summer travel. Please find the BeatNet training script below.
Regarding the datasets, please note that some of them such as GTZAN are
freely available and to obtain some others such as Rock-Corpus I signed a
consent to not hand it over. However, you can get in touch with their
maintainers and obtain them after signing a consent. Please feel free to
ask if you have any questions regarding the training process. Good luck
with your training.
Best,
Moji
On Thu, Aug 3, 2023 at 3:54 AM amantanejams @.***>
wrote:

Gentle reminder @mjhydri https://github.com/mjhydri :) !
—
Reply to this email directly, view it on GitHub
https://github.com/mjhydri/Singing-Vocal-Beat-Tracking/issues/1#issuecomment-1663468350,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWBSZM3JNJP3UIZQGNQYB3XTNKJRANCNFSM6AAAAAA2UWUIIA
.
You are receiving this because you were mentioned.Message ID:
@.***>

hi @mjhydri,
Still don't see any attachments. Would you like to add them to the repository directly?
Thanks

[image: Capture222.PNG]
I hope it helps!
On Thu, Aug 3, 2023 at 2:29 PM kewlcoder @.***> wrote:

hi @mjhydri https://github.com/mjhydri,
Still don't see any attachments. Would you like to add them to the
repository directly?
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/mjhydri/Singing-Vocal-Beat-Tracking/issues/1#issuecomment-1664442565,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWBSZIFSHJFTIWSRKETLJTXTPUXPANCNFSM6AAAAAA2UWUIIA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Also. Please note that the zip file contains the source code for training
BeatNet. In order to train the singing vocal model please use the
following code:
vocal - beat - source.zip
https://drive.google.com/file/d/1GTvHjJZ43sD64PJn8Mw-n43l9HfXcKrk/view?usp=drive_web
On Thu, Aug 3, 2023 at 2:54 PM Moji HHH @.***> wrote:

[image: Capture222.PNG]
I hope it helps!
On Thu, Aug 3, 2023 at 2:29 PM kewlcoder @.***> wrote:

hi @mjhydri https://github.com/mjhydri,
Still don't see any attachments. Would you like to add them to the
repository directly?
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/mjhydri/Singing-Vocal-Beat-Tracking/issues/1#issuecomment-1664442565,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWBSZIFSHJFTIWSRKETLJTXTPUXPANCNFSM6AAAAAA2UWUIIA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi Moji,
Still not able to get access.
Your first msg is visible as follows -
"
[image: Capture222.PNG]
I hope it helps!
"
And the drive link does not give me access. I have requested access, though. Waiting for your approval.
Thanks

Also. Please note that the zip file contains the source code for training BeatNet. In order to train the singing vocal model please use the following code: vocal - beat - source.zip https://drive.google.com/file/d/1GTvHjJZ43sD64PJn8Mw-n43l9HfXcKrk/view?usp=drive_web
…
On Thu, Aug 3, 2023 at 2:54 PM Moji HHH @.> wrote: [image: Capture222.PNG] I hope it helps! On Thu, Aug 3, 2023 at 2:29 PM kewlcoder @.> wrote: > hi @mjhydri https://github.com/mjhydri, > Still don't see any attachments. Would you like to add them to the > repository directly? > > Thanks > > — > Reply to this email directly, view it on GitHub > <#1 (comment)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/ANWBSZIFSHJFTIWSRKETLJTXTPUXPANCNFSM6AAAAAA2UWUIIA > . > You are receiving this because you were mentioned.Message ID: > @.***> >

Hello Moji,
Thanks for the repo and I would like to have a try on the code. Is this zip file contains the .pt model and .yaml file? I would like to get access to it if you don't mind. Thanks very much.
Gary


.inputrc breaks ^A and ^E in Python interactive shell

This might be a WONTFIX, since the project does try to vimify everything, but the .inputrc file that comes with YADR breaks the ability to use control-a (beginning-of-line) and control-e (end-of-line) inside of the Python interactive shell.  Given that those keybindings do work for zsh, I thought this might be a bug to be fixed.

Does vim-style navigation works, though? Like 0 and $...
This is probably a WONTFIX indeed as not only breaks the vimification, but also it seems specific to Python REPL, and different REPL could adopt different keymaps, so on and so forth...
Feel free to reopen the issue if need to


Error compiling java header when building aar

I'm trying to build face detection AAR file following official guide, but I'm getting this kind of error:
ERROR: /home/vladislav/mediapipe/mediapipe/examples/android/src/java/com/google/mediapipe/apps/aar_example1/BUILD:3:14: Compiling Java headers mediapipe/examples/android/src/java/com/google/mediapipe/apps/aar_example1/libmp_face_detection_aar_android_lib-hjar.jar (49 source files) failed (Exit 1) java failed: error executing command external/remotejdk11_linux/bin/java -Xverify:none '--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED' '--add-exports=jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED' ... (remaining 105 argument(s) skipped)
And then I get a lot of protobuf errors as in #1679
Code in aar_example1/BUILD:
load("//mediapipe/java/com/google/mediapipe:mediapipe_aar.bzl", "mediapipe_aar")

mediapipe_aar(
    name = "mp_face_detection_aar",
    calculators = ["//mediapipe/graphs/face_detection:mobile_calculators"],
)

I have tried to build some android examples (including face detection) and they all built successfully.
I have also tried setting JAVA_HOME to /usr/lib/jvm/java-11-openjdk-amd64/ and to /usr/lib/jvm/java-8-openjdk-amd64/. Nothing changed.

Ok, I tried building with 0.8.2 version, and it worked...

Hi @smooth-crescendo , can you please help me what is 0.8.2, is it bazel version? plz help m facing the same issue

Hi @smooth-crescendo , can you please help me what is 0.8.2, is it bazel version? plz help m facing the same issue

It's a mediapipe version. You need to clone this repo again and then just do git checkout tags/0.8.2 in mediapipe directory to switch to 0.8.2 version.

@smooth-vlad Thank you for sharing solution, it's work for me!


Fixes improving source ranges and implicit flags of AST nodes

This is a series of fixes intended to improve the quality of references to source code from AST, it includes:
a) fixes in source ranges stored on AST nodes
b) fixes of few missed 'implicit' flags for generated nodes
c) fix for swift::tokenize(...) method to handle tokens split by the parser

Can one of the admins verify this patch?

Committed to master.
65b5f78 Changed TypeChecker::callWitness(...) to use proper location for TupleExpr.
c25321f Adding a location of the var/let/inout to ParamDecl
33c270d Visiting type repr part of the enum element declaration
0a0a026 Don't skip creating CoerceExpr in case forced_downcast_noop warning
f1a9141 Preserving source range of PBD entry's original initialization.
7e2c92f Added getter for full (raw) text of the token
060182d Fixed several AST nodes wrongly marked with implicit/explicit flag
a131eb5 Properly handle tokens split in parser inside swift::tokenize(...)
60fd687 Fixed interpolated string literal end location
b16f40b Fixed source range for ParamDecl with implicit type location


[Windows] Drop the media capture related cases from windows test

Since media capture is not supported, remove the media capture related cases on windows test

BUG=https://crosswalk-project.org/jira/browse/XWALK-6511

LGTM


Fixed many RTL CSS issues

Fixed almost every RTL problem I found,
Some of the problems (before fixation) are shown in these screenshots

Thanks! 👍


Pattern Ontology Development: Card section - with images -ImageCards --> CardSectionImages

We need to make updates to this pattern based on the latest changes to the ontology and functional specs. See acceptance criteria for specifics.
Old name:
ImageCards
New name:
CardSectionImages
Additional information

Functional Specs
Code
Storybook
Design QA Testing Issue (#1286)
Prod QA Testing Issue (#1287)

Acceptance criteria

[x] Update name in code classes, files, docs, component, stable-selectors, ect

Do directory search to make sure we got everything.

[x] Align prop names to data model.
[ ] Validate data based on functional specs
[ ] A comment is posted in the Design QA issue, tagging Wonil and Roberta, when development is finished
[ ] Assign Wonil to the Design QA issue
[ ] A comment is posted in the Prod QA issue, tagging Praveen and Chetan, when development is finished

@RobertaJHahn
This got merged and ready for QA testing.


(Feature request) New parameter to add a margin

As mentioned here, here and here, it's recommended for QR codes to have a margin (a.k.a. quiet zone) of at least 4 units.
This component draws an SVG with 0 margin for the QR code, which is not a big deal when simply displaying it in HTML, given that you can add the margin with CSS.
However, the browser also allows you to save the SVG locally (right click > Save image as), resulting in a PNG with no margin at all. This becomes a UX issue when people try to reuse the image they just downloaded, and some scanners struggle to read it without the margin.
My suggestion is to support a new parameter like below:
<QrCode value="foobar" marginSize={32} />

For backwards compatibility, marginSize should be 0px by default, and it could reuse the colour provided in bgColor as well.
What are your thoughts?

I think you can just customize it with for example Styled comonents or just css

I agree with @wall3n. I would like to keep this package minimal.
@fappaz you can just add a style attribute like this:
<QrCode value="foobar" style={{ padding: 32, backgroundColor: 'same as bgColor' }} />

Yeah I think this type of components should be without style, just functional. For anyone who wants to add style react I think offers a buch of ways to add it

Sorry I think I didn't make myself clear.
Applying CSS to the component is only adding a margin to the HTML element while it's being rendered on the browser, but not to the SVG drawing per se.
Just try your own code above. There is indeed a padding when displaying the QR code on the DOM, but once you right-click and download it as a PNG, there's no margin. As I mentioned in the original post, I've had users trying to scan this QR code they downloaded and failing to do so because there's no margin.
Therefore what I'm suggesting is an option to draw a thick border around the SVG so the QR code can have that quite zone once downloaded.  Unless there's another way I'm not aware of.

@fappaz did you try smth like this https://stackoverflow.com/a/27546213/1308757?

Nevermind, I just noticed that the project I was working on had multiple QR code packages installed, and the imported component (coincidentally also called QRCode) was from a different lib.
I set up another project from scratch and installed just this lib, that's when I realised I can't even right-click on and download it because it's an SVG instead of a canvas. Even if I embedded a child SVG as you suggested, it wouldn't solve the issue because I couldn't download it anyway - unless maybe I plot it into a canvas first with JS, but then the component starts becoming less practical.
Perhaps the lib could have a new param to render the QR code as canvas instead of SVG, allowing users to download it as an image. Then my request would make sense again.
Anyway, thanks for your help.

Hey, sorry to bug you again about this. I think you should consider re-opening this - the quiet-zone is an important part of the QR code and I would suggest putting it in by default. You can see the effects in the screenshots from the readme -  the QR on the phone on the left doesn't scan, due to the black bezel.

If you agree that's it worth doing, I can put in a PR.

@lukesdm you can wrap a qr code component into a white view. Is there any problem with that?

@lukesdm you can wrap a qr code component into a white view. Is there any problem with that?

I think a few of us have run into issues now, around dark mode, or when the QR code touches a dark object. Adding the white spacing around it is a fine work-around but I think it would save a lot of collective head-scratching in the long-run, as dark mode is getting more popular. Maybe just a note in the readme would be ok?

@lukesdm sure, I would be grad to receive a PR 🙂


Specify subject and its synonyms to help spaCy with parsing

Consider the following example:
Sentence: Born to Run is the third studio album by the American singer-songwriter Bruce Springsteen.
Article: Born to Run
We fail to extract definitions because spaCy parses run as a verb.
We can easily pass the subject of the sentence to whoami, and perhaps even its synonyms from Wikipedia. Before we parse with spaCy, we can make a fake noun out of the subject, e.g. Born_to_Run.
This strategy seems to work -- I tested it using the spaCy web demo.

It seems like Spanish words confuse spaCy. This enhancement would also help us for the sentence: Voces a mi Alrededor is a 1993 album by Franco De Vita released on the Sony label.

Closed by #56


Don't throw error message if Jar already present

Addresses https://ucsc-cgl.atlassian.net/browse/SEAB-3147
Upgrading CLI from 1.10.0 to 1.11.0 works but throws error message
Return the stdout and stderr so you can see message from dockstore script
Don't report a failure just because the JAR is already there.

The dockstore script is downloaded automatically.
I did see this in the Intellij console when debugging:
Connected to the target VM, address: '<IP_ADDRESS>:52268', transport: 'socket'
Java HotSpot(TM) 64-Bit Server VM warning: Sharing is only supported for boot loader classes because bootstrap classpath has been appended
Current Dockstore version: 1.12.0-alpha.0-SNAPSHOT
Upgrading to most recent stable release (1.12.0-alpha.0-SNAPSHOT -> 1.11.2)
The self-install jar already exists at /Users/waltershands/.dockstore/self-installs/dockstore-client-1.11.2.jar.
If you wish to re-download, delete it and rerun "/Users/waltershands/bin/dockstore self-install".


idea: Add a 3rd-party-bindings page to doc site

Is your feature request related to a problem? Please describe.
I'm always frustrated when I want to use react-hook-form with an external component library. It feels like it should be a "solved" problem but it ends up needing to re-invent every time due to each libraries quirks.
Describe the solution you'd like
I want a page similar to https://formik.org/docs/3rd-party-bindings showing 3rd party bindings for common libraries (if available.) If it's not available, at least you'll have an idea of how to best wire up RHF with these external component libraries. Note: react-hook-form doesn't need to own them.
Describe alternatives you've considered
Each person individually asks "How do I use 'react-hook-form' with X component library"
Additional context
I created a library that hooks up mantine and react-hook-form using the name prop.
https://github.com/aranlucas/react-hook-form-mantine
https://codesandbox.io/s/thirsty-dream-2ur5y?file=/src/App.tsx
There's also https://www.npmjs.com/package/react-hook-form-mui

I can take it up.
Are you thinking something like
- Articles/Blog
- Newsletter
- Videos
- 3rd party bindings ?

Or would it be merged somewhere in the current sections?

I think it's best to have 4 pages, however, if with time constraining merge into a single page is a great addition as well.

I'll start with single page, and then work on separating so it's easier to consume.
I'm encountering an issue that is making this change take longer than expected.
It seems https://github.com/react-hook-form/documentation/commit/8a322483f76dd4075647da2464d10ef5aea28b43 changed the data source from https://github.com/react-hook-form/documentation/blob/master/src/data/resources.tsx to https://github.com/react-hook-form/documentation/blob/master/src/components/Resources.json
If it's Resources.json, then the "Edit page" at the bottom does not redirect to the right page. If it's resources.tsx, then it's not currently up to date with the latest Resources.json
I'm thinking of removing Resources.json and moving back to using resources.tsx (Replacing resources.tsx content with whatever is in Resources.json ).
Curious if there's something I'm missing here.
Thanks

I have merged it for now, it looks good and additional resource for devs.

Thank you!
Regarding I think it's best to have 4 pages, do you have any mockups of how that might look like?
I think rather than splitting it up, might be worth to paginate the items on there similar to https://cloudscape.design/components/cards/?tabId=playground where the cards can be on the same page but are just limited to 10/page (reducing the amount of scrolling to get to the bottom.)
Could even have some sorting involved.

https://cloudscape.design/components/cards/?tabId=playground

That looks great! 4 page works better for SEO, I think if we can duplicated those pages and turn them into 4 is also a good choice, but what you have done is already excellent, hopefully SEO pick it up quickly.

Here's my current idea for resource page to try and reduce the amount of scrolling on the page.
Just want to make sure we are on the same page.
With this it seems I'm separating them into 4 different containers, so next step to pages shouldn't be too bad.

This looks great. Could we split it into different pages?
/resources/articles
/resources/third-party-bindings
/resources/videos
The only concern is that pagination would not be a great way to help crawlers index content.

What would be in the
/resource page?
or just change the navbar to have a dropdown similar to tools section?

or just change the navbar to have a dropdown similar to tools section?

yes, thanks for looking into this 👍


Could not impersonate the elevated user.  LogonUser returned error code 1326.

I'm getting this error when I try to impersonate an account for File.Copy(..)

Have you read this? http://stackoverflow.com/questions/17332977/logonuser-failed-with-error-code-1326

https://msdn.microsoft.com/en-us/library/windows/desktop/ms681385.aspx

ERROR_LOGON_FAILURE
1326 (0x52E)
The user name or password is incorrect.

There are a variety of reasons this could be occurring.  Unfortunately, I can't really help with that.  This library just wraps the LogonUser APIs in Windows - so if you are passing the correct credentials and it doesn't work, then look into reasons why LogonUser might fail on other sites.
Thanks.


Change message when requesting permission to access photos

I've submitted my app to the App Store and they're asking to change the message that appears the first time you use the camera, and instead display a message that explains why are you requesting access to the camera or what are you going to do with the photo. How can I change this message?

Hi @miguelopezv ,
You can manually set the message in your project's info.plist file
app/App_Resources/iOS/Info.plist
<key>NSCameraUsageDescription</key>
<string>camera description.</string>

I also had my app rejected today because of this. This should be somewhere in the docs 👍

Thanks for the feedback. Docs will be updated - #226


Adding support for Travis CI.

With support for Ruby 2.2, and JRuby.
Updated Readme.md with Gem version badge, and Travis build status.

Thanks!

@sidoh I think you'll need to enable Travis CI access in the repository settings.

Hmm, I thought I did. I created the webhook and enabled it within Travis. Is there something else I need to do?

@sidoh Hmn just what's on here https://docs.travis-ci.com/user/getting-started/ As the Travis page says there are no builds for this repository. 😄

Haha. Right. Was thinking that was because there have been not commits since the .travis.yml was added.

Travis only runs a build on the commits you push AFTER adding the repository to Travis.

Think that was it. Made a whitespace change and it triggered a build :)


feat: 支持同时在 React 16/17/18 下运行

👀 PR includes

✨ Feature

[x] New feature

📝 Description

原本 Tooltip 做了 React 16/18 的兼容, 但在实际运行时 react-dom/client 的方式打包工具会报错

本质上 react-dom/client 是 react-dom 的一个 sub module
https://github.com/facebook/react/blob/main/packages/react-dom/package.json#L51-L59
所以参考 antd 的处理方式, 使用 import * 的方式全部引入, 然后做一个克隆, 跟进版本动态判断是否使用 createRoot
https://github.com/react-component/util/blob/677d3ac177d147572b65af63e67a7796a5104f4c/src/React/render.ts#L6-L13
import * as ReactDOM from 'react-dom';
import type { Root } from 'react-dom/client';

const ReactDOMClone = {
  ...ReactDOM,
} as typeof ReactDOM & {
  __SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED?: {
    usingClientEntryPoint?: boolean;
  };
  createRoot?: CreateRoot;
};

其他:
React 19 已发布 RC 版本, 社区负面反馈较多, 后续兼容视React 19 稳定情况而定。
🖼 Screenshot

Before
After

❌
✅

🔗 Related issue link

🔍 Self-Check before the merge

[ ] Add or update relevant docs.
[ ] Add or update relevant demos.
[ ] Add or update test case.
[ ] Add or update relevant TypeScript definitions.

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

Project coverage is 80.21%. Comparing base (ecdc9c8) to head (67d6fc6).
Report is 398 commits behind head on next.

Files
Patch %
Lines

packages/s2-react/src/utils/reactRender.ts
80.32%
8 Missing and 4 partials :warning:

packages/s2-react/src/utils/invokeComponent.tsx
0.00%
6 Missing and 1 partial :warning:

...s2-react/src/components/tooltip/custom-tooltip.tsx
50.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             next    #2802      +/-   ##
==========================================
+ Coverage   75.77%   80.21%   +4.44%
==========================================
  Files         257      181      -76
  Lines       11994    10384    -1610
  Branches     2464     2344     -120
==========================================
- Hits         9088     8330     -758
+ Misses       1398      612     -786
+ Partials     1508     1442      -66

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

:tada: This PR is included in version @antv/s2-react-v2.0.0-next.24 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How to configure PASV mode inside docker-compose file

Hi @fauria!
When I use docker-vsftpd alogside docker-compose.yml I cannot properly use ftp ls command cause I get such error:

ftp> ls
500 Illegal PORT command.
425 Use PORT or PASV first.

Part of docker-compose.yml file:
ftp-server:
    image: fauria/vsftpd
    deploy:
      replicas: 1
    ports:
      - "20:20"
      - "21:21"
      - "21100-21110:21100-21110"
    environment:
      - FTP_USER=bob
      - FTP_PASS=12345
      - PASV_ADDRESS=my-host
      - PASV_ADDRESS_RESOLVE=YES
      - PASV_ADDR_RESOLVE=YES
      - PASV_MIN_PORT=21100
      - PASV_MAX_PORT=21110
      - PASV_ADDRESS_ENABLE=YES
      - PASV_ENABLE=YES
    networks:
      - webnet
    volumes:
      - "/opt/dockerstorage/volumes/ftpvolume/_data:/home/vsftpd"

But when I start ftp server with below command it works fine:
docker run -d -v /opt/dockerstorage/volumes/ftpvolume/_data:/home/vsftpd -p 20:20 -p 21:21 -p 21100-21110:21100-21110 -e FTP_USER=bob -e FTP_PASS=12345 -e PASV_ADDRESS=my-host -e PASV_MIN_PORT=21100 -e PASV_MAX_PORT=21110 -e PASV_ADDRESS_ENABLE=YES -e PASV_ENABLE=YES --name vsftpd --restart=always fauria/vsftpd

I think there are some troubles with address lookup but I couldn't find how to solve it.

Update: For now I need to use host network.

This unfortunately has nothing to do with PASV mode as it also happens for me with the following entry in docker-compose
ftp:
    image: fauria/vsftpd
    environment:
      - FTP_USER=foo
      - FTP_PASS=bar
      - PASV_ENABLE=NO
      - PASV_ADDRESS_RESOLVE=NO
      - PASV_ADDR_RESOLVE=NO
      - PASV_ADDRESS_ENABLE=NO
    ports:
      - "8020:20"
      - "8021:21"

Result:
ftp> ls
500 Illegal PORT command.

Okay after some deeper research I figured that the linux ftp CLI does apparently not automatically switch to passive mode by somehow communicating with the server. You rather have to explicitly tell it to use passive mode.
So if you go ftp -p hostname port, then it seemed to work in my configuration with the following docker-compose entry:
ftp:
    image: fauria/vsftpd
    environment:
      - FTP_USER=foo
      - FTP_PASS=bar
      - PASV_ADDRESS_RESOLVE=YES
      - PASV_ADDR_RESOLVE=YES
      - PASV_MIN_PORT=21100
      - PASV_MAX_PORT=21110
      - PASV_ADDRESS_ENABLE=YES
      - PASV_ENABLE=YES
      - LOG_STDOUT=1
    volumes:
      - ./support/vsftpd:/home/vsftpd
    ports:
      - "8020:20"
      - "8021:21"
      - "21100:21100"
      - "21101:21101"
      - "21102:21102"
      - "21103:21103"
      - "21104:21104"
      - "21105:21105"
      - "21106:21106"
      - "21107:21107"
      - "21108:21108"
      - "21109:21109"
      - "21110:21110"

my worked solution
version: "3.3"
services:
  ftp:
    image: docker.io/fauria/vsftpd:latest
    container_name: ftp
    environment:
      FTP_USER: user
      FTP_PASS: password
      PASV_ENABLE: "YES"
      PASV_ADDRESS: "ftp"
      PASV_ADDR_RESOLVE: "YES"

@fauria let's close the issue


Text updates  - app types - applications to fleets

change-type: patch
signed-off-by: Laura Alison <@poshrolls>

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@balena-ci rebase


AST extracting the documentation for functions

Hey, I came across Go-App and as a result this project. Really cool - been wanting something to learn about AST for ages. This is perfect.
My goal is to try and build some components using Go-App for generating web assembly apps and then use the functions and the comments to generate the documentation as part of the web assembly site - i.e the documentation auto generates with the webassembly app which would be pretty neat.
I was playing with your application and my first question (I'm very new to AST stuff) is:
If I enter
package main
// Nav
/*
 this function adds two strings together and returns the result
*/
func Nav(str1 string, str2 string) string {
  // add the two passed strings together
   res :=str1 + str2
  return res
}

into your application I get the comments block:
  Comments: []*ast.CommentGroup {
    &ast.CommentGroup {
      List: []*ast.Comment {
        &ast.Comment {
          Slash: 14,
          Text: "// Nav",
        },
        &ast.Comment {
          Slash: 21,
          Text: "/*\n this function adds two strings together and returns the result\n*/",
        },
      },
    },
  },

which is fine. However when I remove the comment within the function, i.e   // add the two passed strings together I get an empty Comments group, i.e   Comments: []*ast.CommentGroup {}, - I was wondering if this is a bug or something to do with how AST is generated? I.e oes the   Comments: []*ast.CommentGroup {}, need a comment in the function to read the comments above the function.
My goal is to somehow (haven't got there yet) run the AST generator over a package, pull out all the comments and then the function and add those to a markdown file which will then be rendered by my webassembly application. I think using AST is a good way to do this (as further down the line I may want more granular control over how the markdown is generated.
Also - in this case, does the AST generated "know" that the comments before the function are "part" of, i.e related to the function specifcally? - Reason I ask is that if I make a second function with leading comments, all commentGroups seem to be at the bottom of the AST generated. Is there a way for it to know those comments are related to that function?
Thanks

Do you really want to learn how ast works?
Otherwise you could just use https://pkg.go.dev/go/doc
Astextract is good for getting a rough ast representation useful for generating the same code independent from current position and documentation.
You don't want to generate new code using ast, but read in existing code and transform that into documentation.
This program isn't particularly good at that, you should most likely use https://pkg.go.dev/go/ast#Print instead as stated in the readme.


comma are nod encoded when used as a path segment

What kind of issue is this?

[ ] Question. This issue tracker is not the place for questions. If you want to ask how to do
something, or to understand why something isn't working the way you expect it to, use Stack
Overflow. https://stackoverflow.com/questions/tagged/okhttp

[x] Bug report. If you’ve found a bug, spend the time to write a failing test. Bugs with tests
get fixed. Here’s an example: https://gist.github.com/swankjesse/981fcae102f513eb13ed

[ ] Feature Request. Start by telling us what problem you’re trying to solve. Often a solution
already exists! Don’t send pull requests to implement new features without first getting our
support. Sometimes we leave features out on purpose to keep the project small.

import okhttp3.HttpUrl;
import org.junit.Test;
import java.util.Objects;
import static org.junit.Assert.assertEquals;
public class CommaPathSegmentTest {
@Test
public void test() {
String methodUrl = "https://opendata-api.stib-mivb.be/OperationMonitoring/2.0/PassingTimeByPoint";
HttpUrl.Builder urlBuilder = Objects.requireNonNull(HttpUrl.parse(methodUrl)).newBuilder();
urlBuilder.addPathSegments("1064,1108");
    String url = urlBuilder.build().toString();
    assertEquals("https://opendata-api.stib-mivb.be/OperationMonitoring/2.0/PassingTimeByPoint/1064%2C1108", url);
}

}


Material Depth output causes a segfaults at intermediate invalid user input

I've crashed Material Maker multiple times today by trying to alter the Depth multiplier on the Material Output node.
I've figured out what I've done was use backspace to remove digits, wanting to insert new ones. As soon as I changed "0.5" to "0." - Material Maker segfaulted.
I guess it should sanitize the value and buffer last known working one in such cases?

Hmm must disable expressions in that node...

Not sure why it crashed, it should just fail to apply the parameter...
@unfa The last dev build should have a fix for this.


[Feature] Modal

Modal

[x] Default modal
[x] Pop-up modal
[x] Form element
[x] Crypto wallet
[x] Sizes
[x] Placement

I'm on it now.

https://github.com/themesberg/flowbite-svelte/pull/189.

@jjagielka I used to have clickOutside function.
What do you think adding it to the Modal component?

This is an ideological question. If you look at https://en.m.wikipedia.org/wiki/Modal_window you’ll find:

Users must interact with the modal window before they can return to the parent application.

Clicking outside it’s not a real interaction. Some tends even to force the focus trap to not allow leaving the modal without interaction.
I did not implemented the click outside on purpose but you can challenge that approach stating that I did implement reaction to the Escape button.

Thanks for the explanation. I go with it.

Current Modal in Flowbite-Svelte implementation seems already different from the Modal examples in Flowbite-Svelte documentation. Any updates on the documentation?

What is different? Can you please show examples?

For example, in documentation, the examples show to use Modal is by importing Modal from flowbite-svelte. But when i try, there is no Modal but there are SmallModal, MediumModal, LargeModal, SignInModal, ExtraLargeModal. There are even an additional ModalButton component. Is it obviously different from the current examples documentation?

Are you sure you use the latest version if flowbite-svelte?
Can see only 'Modal.svelte' in the code.

Completed.
https://github.com/themesberg/flowbite-svelte/pull/220


test pr

Add preview link message
test

You can preview this docs PR at http://326.docs-pr.validatedpatterns.io
Note that they get generated every five minutes, so please wait a bit.


Update credential details in proof request screen to match design

Current iteration of the app shows text on a grey background with white dividers.

The design should match the wireframes. Wireframes have been updated to ensure background colour can remain consistent but with white elements on top.

Looks good


Improve UTF-8 encoder performance in Dart VM

Investigate benchmark provided by @bprosnitz and provide the strategy for improving the score.
Related benchmarks that capture pure Utf8 string encoding performance is committed in d79ef922a5806bf58114ec9c65b6890ae38cd9e0
/cc @mkustermann @askeksa-google

/cc @askeksa-google If you have time, maybe this could benefit dart2wasm and VM AOT at same time.


newTree.optimize is not a function

I'm submitting a...

[ x] Bug report
[ ] Feature request
[ ] Documentation issue or request

In package

[ ] @ng-toolkit/init
[ ] @ng-toolkit/serverless
[x] @ng-toolkit/universal
[ ] @ng-toolkit/pwa
[ ] @ng-toolkit/firebug

Current behavior

When a run this command  ng add @ng-toolkit/universal
This shows 1 error:  newTree.optimize is not a function
Expected behavior

Minimal reproduction of the problem with instructions

Example repository

What is the motivation / use case for changing the behavior?

Environment

Angular version: 9.0.7

- Node version: 13.11.0
- Platform:  Mac

Others:

I have the same problem(

Happening here too

@Tonybonybob @carlos-alonso-al have a look here on this reply if it helps you out.


Fix airgap for fleet

This commit add global registry setting and pass it to the fleet charts
that are installed by controller, since these settings can't be passed
from UI.

https://github.com/rancher/rancher/issues/29143