Exceptions from Sphinx-Needs when clean builds are not used

This is a user experience problem while using Sphinx-Needs that, when a Sphinx incremental build is used, Sphinx-Needs often raises exceptions. Most of the time this happens when there is a version upgrade of Sphinx-Needs or when the doctrees are in some inconsistent state. Most of the time, the solution is to do a clean Sphinx build. However, many users struggle until they realize that a clean build is the solution to the problem.
I see two possible solutions.

Every time Sphinx-Needs detects inconsistencies in cached doctrees, it automatically does a full build. If Sphinx-Needs can detect this early and request a full build from Sphinx, this would be ideal. I am unsure if this is technically feasible from an extension as they come in during "later" stages of a Sphinx build. Else we could create a PR for Sphinx to support this. To start with, a check could be as simple as, "is there a version difference of Sphinx-Needs that was used for the last build"
Sphinx-Needs catches all its exceptions and throws an error to the user with a hint to do a clean build.

wouldn't be so easy to detect a problem.
We could store the used Sphinx-Needs version in the Sphinx env object, and compare with one from the currently use Sphinx-Needs package.
But I'm afraid of a false-positive check and Sphinx-Needs requested a full build to often.

Could be doable. We could register a function for build-finished and check for exceptions.
If this the case we can write some short text at the end of the log.

I want to close this issue because I see it more as something that should be solved in Sphinx itself.
What is described for Sphinx-Needs (inconsistent data, version upgrade, config change) is something all other Sphinx extensions have as well.
So for me Sphinx should be aware of if an extension raised its version between two builds. All extension must report their current version to Sphinx, so it should be doable.
Or if not done by Sphinx, then this check should be part of another extension.
We have made some tests for this with Sphinx-Needs (PR #936) and one of the problems was, that our implementation fetches problems of other extensions as well and Sphinx-Needs is unable to tell if these are a problem for the user or not.

a check could be as simple as, "is there a version difference of Sphinx-Needs that was used for the last build"

@twodrops @danwos This is already possible with env_version: https://www.sphinx-doc.org/en/master/extdev/index.html#extension-metadata
You can see here, that it simply excepts if this version has changed since the last incremental build: https://github.com/sphinx-doc/sphinx/blob/3ffde92c54c6e9d594b2a41a365a90dc0dcbac6c/sphinx/environment/init.py#L226
Obviously you would need to decide when to increment env_version, which should only be if something stored on the environment has changed


2.3.2 Write a Dockerfile - text version omits RUN line

labs/beginner/chapters/webapps.md
The text version omits the following line from the Dockerfile creation steps;-
RUN pip install --upgrade pip
Without this line, the following error message is received at Step 4/7:-
You must give at least one requirement to install (see "pip help install")

Even after adding this line to the Dockerfile, and installing Python (which I hadn't previously, as I usually work in R), I still get the same error message, as follows;-
You must give at least one requirement to install (see "pip help install")
What am I missing out here, please?

I have found one mistake within my files and rectified it, so that the error message is no longer being received.  However, I still can't get the website to load.

I have found one mistake within my files and rectified it, so that the error message is no longer being received.  However, I still can't get the website to load.

I have identified the mistake in the code, which was all my mistake  Apologies!


Automatic package generation improvements

In #67, I listed some future work.
Important:

[x] Also upload npm package.

Should do:

Improve tag detection/matching in azure-pipelines.yml (currently, version match is partially hardcoded as v0.*).
rename the azure feed (I called it "python-packages", but it's not only python packages...)

Optional:

Consider having the versioning update automatically (e.g. based on git, or build number, or date, or whatever you prefer). Would allow nightlies, for example (but this doesn't seem important for celltests, really).

I just tested uploading an auto-generated pre-release package to pypi, and it seems fine.

You don't see the pre-release unless you specifically request it (either through pip or via the web ui). E.g. if you request jupyterlab_celltests on the web ui, you get the current release:


#81 Add modified-at-latest parameter to eq.instances.relative()

Closes #81 and is part of energyquantified/core#235

@fredriksvendsen But in general you approve?
@jonmd Can you keep in mind to merge this once it's deployed to the API? I'll try too.


(PUP-1391) puppet device cannot create certs when run as root

Prior to this commit, puppet device could not write to the privatekeydir and
certdir for devices.
drwxr-x---.  3 root      root        18 Jun 28 22:55 devices
With this commit, the $vardir/devices directory’s group is set to pe-puppet,
so that the puppet user can traverse the directory via the group permissions.
drwxr-x---.  3 root      pe-puppet   18 Jun 28 22:55 devices

Travis is failing because of this commit, which is unrelated to mine:
Updating POT file for 550ce1ff1584b6a2a5563371a7fc043618fdc0eb
Advice on how to resolve would be greatly appreciated!

Sometimes Travis incorrectly calculates the range of commits it needs to check. I rekicked it, but if it still doesn't resolve, we'll just ignore it.

Thx!

For files that are shared with puppet/pe-puppet, we usually set both owner and group to service. Can we do the same here?

Checking owner...

Owner changed and tested!

Excuse my ignorance, but do these changes get picked up during an upgrade or only new installs?

If I recall correctly, they are not enforced per run. I had to delete the directory during testing and verify after the next tun that it was recreated with the expected attributes.

@shermdog is that something you would like to see added to this PR, or as a follow-up feature?

With this commit, these are the directories and files whose ownership changes:
/opt/puppetlabs/puppet/cache/devices

from root:root to pe-puppet:pe-puppet
for just the directory
/opt/puppetlabs/puppet/cache/devices/<device>/preview
/opt/puppetlabs/puppet/cache/devices/<device>/ssl

from root:pe-puppet to pe-puppet:pe-puppet
for the directory, and all subdirectories and files
I also do not know how that works during upgrade, but I cannot find any of the above as File resources in the catalog of a proxy puppet agent. I would like to see correcting ownership for existing directories and files be a follow-up feature, as this bug was first reported in 2012, and the feature would might end up in another repo.

@shermdog

Do these changes get picked up during an upgrade or only new installs?

Not during an upgrade ... but I did some more research (wth help from finch) and the permissions defined in defaults.rb are enforced during the next 'puppet device' run, since the setting is a device setting:
define_settings(:device,
    :devicedir =>  {

[root@pe-20173nightly-master puppet]# chown nobody: /opt/puppetlabs/puppet/cache/devices

[root@pe-20173nightly-master puppet]# ls -al /opt/puppetlabs/puppet/cache/
total 44
drwxr-x---. 12 root      root      4096 Sep 20 04:59 clientbucket
drwxr-x---.  3 root      root      4096 Sep 20 05:10 client_data
drwxr-x---.  2 root      root      4096 Sep 20 04:57 client_yaml
drwxr-x---.  3 nobody    nobody    4096 Sep 20 05:31 devices
drwxr-xr-x.  2 root      root      4096 Sep 20 04:56 facts.d
drwxr-xr-x.  7 root      root      4096 Sep 21 05:58 lib
drwxr-xr-x.  9 root      root      4096 Sep 20 04:59 pe_concat
drwxr-x---.  2 pe-puppet pe-puppet 4096 Sep 20 04:56 preview
drwxr-xr-x.  2 root      root      4096 Sep 22 18:19 puppet-module
drwxr-xr-x.  3 root      root      4096 Sep 20 05:00 reports
drwxr-xr-t.  3 root      root      4096 Sep 22 21:44 state

[root@pe-20173nightly-master puppet]# puppet device --noop

[root@pe-20173nightly-master puppet]# ls -al /opt/puppetlabs/puppet/cache/
total 44
drwxr-x---. 12 root      root      4096 Sep 20 04:59 clientbucket
drwxr-x---.  3 root      root      4096 Sep 20 05:10 client_data
drwxr-x---.  2 root      root      4096 Sep 20 04:57 client_yaml
drwxr-x---.  3 pe-puppet pe-puppet 4096 Sep 20 05:31 devices
drwxr-xr-x.  2 root      root      4096 Sep 20 04:56 facts.d
drwxr-xr-x.  7 root      root      4096 Sep 21 05:58 lib
drwxr-xr-x.  9 root      root      4096 Sep 20 04:59 pe_concat
drwxr-x---.  2 pe-puppet pe-puppet 4096 Sep 20 04:56 preview
drwxr-xr-x.  2 root      root      4096 Sep 22 18:19 puppet-module
drwxr-xr-x.  3 root      root      4096 Sep 20 05:00 reports
drwxr-xr-t.  3 root      root      4096 Sep 22 21:44 state

Same for the device's preview and ssl directories, which also intersect this:
[root@pe-20173nightly-master puppet]# chown nobody: /opt/puppetlabs/puppet/cache/devices/bigip.example.com/preview

[root@pe-20173nightly-master puppet]# chown nobody: /opt/puppetlabs/puppet/cache/devices/bigip.example.com/ssl

[root@pe-20173nightly-master puppet]# ls -al /opt/puppetlabs/puppet/cache/devices/bigip.example.com/
total 32
drwxr-x---. 2 root      root      4096 Sep 20 05:31 clientbucket
drwxr-x---. 3 root      root      4096 Sep 20 05:32 client_data
drwxr-x---. 2 root      root      4096 Sep 20 05:31 client_yaml
drwxr-xr-x. 2 root      root      4096 Sep 20 05:31 facts.d
drwxr-xr-x. 7 root      root      4096 Sep 20 05:32 lib
drwxr-x---. 2 nobody    nobody    4096 Sep 20 05:31 preview
drwxrwx--x. 7 nobody    nobody    4096 Sep 20 05:32 ssl
drwxr-xr-t. 3 root      root      4096 Sep 22 21:56 state

[root@pe-20173nightly-master puppet]# puppet device --noop

[root@pe-20173nightly-master puppet]# ls -al /opt/puppetlabs/puppet/cache/devices/bigip.example.com/
total 32
drwxr-x---. 2 root      root      4096 Sep 20 05:31 clientbucket
drwxr-x---. 3 root      root      4096 Sep 20 05:32 client_data
drwxr-x---. 2 root      root      4096 Sep 20 05:31 client_yaml
drwxr-xr-x. 2 root      root      4096 Sep 20 05:31 facts.d
drwxr-xr-x. 7 root      root      4096 Sep 20 05:32 lib
drwxr-x---. 2 pe-puppet pe-puppet 4096 Sep 20 05:31 preview
drwxrwx--x. 7 pe-puppet pe-puppet 4096 Sep 20 05:32 ssl
drwxr-xr-t. 3 root      root      4096 Sep 22 21:58 state


How to select multiple columns in many-to-many relationship

The description says "Select tables and columns that are related," but I can only select one column per table.  Can I actually selet two keys for the relationship?

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

ID: 477d54a5-8a27-ae63-9887-9801639962e8
Version Independent ID: 8a2c56c0-cf25-bd2d-8de9-207ccc1e83d1
Content: Many-to-many relationships in Power BI Desktop (Preview) - Power BI
Content Source: powerbi-docs/desktop-many-to-many-relationships.md
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @yzhong94 . We’ve forwarded your feedback to the writer, who should respond to this thread shortly. Thanks!

Hello,
I would like to understand what will be the preformance delta in the new M:M VS using a star schema model with a key_table the middle.
Wont it be less efficient to have this snowflak model instead of a star model ?
I would wonder why not the have a feature that creates a star schema for you..

Sorry for the confusion - I believe that's suggesting you select one column per table (thus, 2 tables is 2 columns). I'll see if I can make that clearer - thanks!


CustomPromptTemplate._getPromptType - error Not implemented

CustomPromptTemplate._getPromptType

error Not implemented
Ways to reproduce:
"langchain": "^0.0.95",
and followed the exactly example: https://js.langchain.com/docs/modules/agents/agents/custom/custom_agent_chat
It was working fine before. I just updated because of the new openAi models and then it started to happen

Works on  "langchain": "0.0.91",
But if I try to use gpt-4-0613, gpt-3.5-turbo-16k or gpt-3.5-turbo-0613 I get the error
` Chain run errored with error: "Invalid stopping method: generate"

error node_modules\langchain\dist\agents\agent.js (36:0) @ LLMSingleActionAgent.returnStoppedResponse

error Invalid stopping method: generate`
const executor = new AgentExecutor({
agent,
tools,
earlyStoppingMethod: "generate",
maxIterations: 5,
verbose: true,
});
I case I remove the earlyStoppingMethod the error is: Got output Agent stopped due to max iterations.

I have the same problem


helm.sh/resource-policy: are quotes really required?

In Tell Helm Not To Uninstall a Resource, the documentation says:

kind: Secret
metadata:
  annotations:
    "helm.sh/resource-policy": keep
[...]

(Quotation marks are required)

But... are they really? I just tried it without quotes, and it seems to be working fine. I can't think of any reason why quotes would be required here. Kubernetes has many well-known annotations with similar names, and none of them need quotes either.

The quotes aren't required. The quotes force the parser to use a string but it should identify it's a string without them.

Then perhaps this misleading wording should be removed from the documentation?

It's not "wrong" though. Yaml will treat it as a string either way. It's a style choice that's up to the user.

Adding quotes is not wrong, but claiming that they are required is wrong.

@joejulian I think the issue is with the phrase "(Quotation marks are required)" that's found in the docs.

A PR to improve/soften the verbiage on the docs would be appreciated please (https://github.com/helm/helm-www/blob/main/content/en/docs/howto/charts_tips_and_tricks.md#tell-helm-not-to-uninstall-a-resource)

Would you mind transferring this issue to helm-www? I didn't realize the docs were in a separate repository.

I actually don't have the priv's to transfer. But a PR either way would be great.

@mattfarina Could you transfer this issue to the helm-www repository?

Fixed by helm/helm-www#1552.


Just some Questions

Hi Jed,
I'm trying to wrap up a beta-package for my new oembed plugin. Got a few questions about how to interface with Bob.
https://github.com/joshuafontany/TW5-oembed

How would I check to see if Bob in the browser has an active open websocket? Right now I'm testing for if($tw.Bob) to queue up a message, but I would like to test for an open connection as well, and offer people a way to manually input the oembed JSON data if Bob is not available with an open connection.

I seem to be getting multiple messages received by the server for each one I send from the browser. I have setup some debugging to catch and report this, and I always see the "this URL is already being processed" message pop up, which means my message handler functions are being triggered multiple times.

I'm not sure I am saving the new data-tiddlers to the server the correct way in my message handler. I see multiple "tiddler saved" messages.

Could this point to the race conditions bugs in the save-queue that is preventing saving Drafts/etc?

Best,
Joshua F

I am running (check any that apply, put an x inside the [ ] to check a box, like this: [x]):

[x] Windows
[ ] OSX
[ ] Linux
[ ] Other

and using

[x] The nodejs version
[ ] The single file executable

Before posting I read issue guidelines and:

[x] I am using the newest version
[ ] The answer to my question isn't listed in the documentation or this isn't
a question
[ ] This is not a duplicate issue
[ ] I have not done anything that required me to set acceptance to
I Will Not Get Tech Support For This

1 - you can directly access the web socket state in the browser with $tw.connections[0].socket
the best check for if it exists and is active is probably
if($tw.connections[0] && $tw.connections[0].socket && $tw.connections[0].socket.readyState === 1) {
// code stuff
}
2 - On the node side are you using $tw.Bob.SendToBrowser ? Using just $tw.Bob.Shared.SendMessage doesn't do some checks that can prevent multiple messages sent to the same browser.
On both sides is the first thing the handler does $tw.Bob.Shared.sendAck(data);? if not the ack message may take longer than the retry timeout depending on your settings. if you don't have it sending an ack at all it will keep trying to resend.
There may be other things to check, but those are the easy ones I can think of now.
3 - The safest option for saving is $tw.syncadaptor.saveTiddler(data.tiddler, prefix, data.source_connection); There are a bunch of places where I need to ensure that I don't have redundant functions but that should be the best option.
4 - drafts aren't saved on the server mainly to prevent spamming the server with messages and because until one of the more recent versions one persons drafts would overwrite everyone else's. The saving delay for editing draft tiddlers should handle the trouble with draft tiddlers without problems.
The bugs with directly editing a tiddler outside of draft mode seem to be handled with the fix in #142 now the only problem is that the internal syncer is still much slower than I want it to be.

Mahalo (thanks), just what I needed to keep testing.

2- I am only sending a message from the browser to the server. The messageHandler immediately sends the (ack) message. I am then using the saveTiddler to refresh my widget and load the embed data once it appears. The widget's native refresh handling does this well, so no need to send a separate "done" message with the data. The saveTiddler method pushes the new data tiddler to all connected browsers.

Satisfied with this one. Closing.


❓ [Question] How to set three input tensor shape in input_shape?

❓ Question

I have three input tensor:src_tokens, dummy_embeded_x, dummy_encoder_embedding
In this case, I don't konw how to set input_shape in compile_settings's "input_shape"
Who can help me? Thank you!
encoder_out = model.forward_encoder([src_tokens, dummy_embeded_x, dummy_encoder_embedding])
...
script_encoder = torch.jit.script(encoder)...
compile_settings = {             "input_shapes": [[2, 16]],             "op_precision": torch.float32         }

You set the input shapes as a list of lists or a list of dicts if you are using dynamic shape.
So for your 3 inputs you would do something like this:
"input_shapes": [[2,16], [1,10], [1, 15]]

or something similar where the entries correspond to src_tokens, dummy_embeded_x, dummy_encoder_embedding in the order they will appear in the function signature for your forward i.e. in this case maybe something like forward(src_tokens, dummy_embeded_x, dummy_encoder_embedding).
To do dynamic shape it would look like this:
"input_shapes": [
  {
    "min": [2, 16],
    "opt": [2, 20],
    "max: [2, 25],
   }, {
    "min": [1, 10],
    "opt": [1, 20],
    "max: [1, 25],
   }, {
    "min": [1, 20],
    "opt": [1, 25],
    "max: [1, 35],
   }
]


Numbered chapters for quick reference

In many situations, it is useful to refer to chapters by number. eg. Chapter 3.1

@k12ish

I don't see any benefit of this for web urls.
Btw I am considering Hugo these days due to the issues I faced with Node ecosystem past years. May be I will move this project to Hugo after some time. And there, I am using /docs/section-01/article-01 url pattern.

However, I believe article title should be unique and should focus on that.

@k12ish

I don't see any benefit of this for web urls.
Btw I am considering Hugo these days due to the issues I faced with Node ecosystem past years. May be I will move this project to Hugo after some time. And there, I am using /docs/section-01/article-01 url pattern.

However, I believe article title should be unique and should focus on that.


Manage a pool of connections

Currently the BeanstalkConnection object maintains a single connection to Beanstalk. That means it can process messages one at a time, with no cancellation. For example, a "reserve" command blocks all other commands until a job is reserved or the timeout expires (this is how Beanstalk appears to work under the hood).
There could be a new top-level BeanstalkPool object which opens new TCP connections to Beanstalk as necessary. Considerations:

If different users of the pool want to use/watch different pools, does every combination need to get its own connection? One connection can't use multiple tubes, but it would be possible to always preceed a command with a use command for the tube it was meant for. The equivalent of alawys doing this:
await prod.UseAsync("my-tube");
await prod.PutAsync(...);
await prod.UseAsync("my-tube");
await prod.PutAsync(...);

Consumers need to be isolated from producers, or else a "reserve" followed by a "put" could deadlock the connection.

hi, why consumer need to be isolated from producers?
beanstalkd not support that scenario or other reasons?

Hi @chrcao!
Beanstalk does support that scenario, but you have to be careful with it.
It's been a while since I worked with Beanstalk, but my recollection is: Beanstalk responds to commands in the order they were sent. Let's say a client tried to be a consumer and producer at the same time, and sent these commands:
reserve
put 1 0 10 foobar

If there are no jobs in the tube for the reserve, then the put will be ignored until the reserve finally receives a job.
That's probably not a great outcome, and it would be easy to make that mistake. So, that's why I feel like a good client library should help the developer avoid accidentally causing that problem.

@jennings thanks for you answer, i got the point.
and another question when I review beanstalkd code. the data type of  job id from beanstalkd is uint64.
in c#, the same capacity data type is ulong. Wil these  cause problem when beanstalkd insert new job and return a job id more bigger than int(using in this client)?

Probably! 😄
If you want to create a pull request for that, please do.

I no longer think this issue makes sense to do. The caller is perfectly capable of managing the number of connections they create.
But it does make sense to make BeanstalkConnection handle reconnection logic. Right now, if you create a worker and the connection dies, the worker dies. That isn't good.


Line numbers are not keyboard/screen reader accessible

monaco-editor version: 0.8.3
Browser: Any
OS: Windows 10
During Accessibility testing, it was found that users cannot get information regarding the current line number with a screen reader while using the up/down arrow keys to navigate through lines of code in Monaco.  It correctly reads the code text, but there is no obvious way to hear the line number.
Personally, I don't believe that users would always want to hear the line number while navigating text, but perhaps a polite, slightly delayed, aria-live region that announced the line number after focus remains on a line of text would be good.  Or perhaps additional line information could be read from a selection on the F1 command palette (and/or shortcut key) if the user desires it.
Shown below is what is read using NVDA while arrowing down, though other screen readers are similar.

Why do we need to address line numbers? Isn't that more 'meta-content' than actual 'content'? For example, I wouldn't expect different colored text to be handled and differently than plain text in a screen reader

IMHO, I don't think the current line number should be read at all when navigating the text. Perhaps there should be a keyboard bound command that announces the current position / selection state via an aria alert.
@zersiax since this applies also to VS Code, what do you think?

There is another related accessibility behaviour, which is there in Monaco. While tabbing into the editor in side by side diff mode, the cursor stays on the first line, in case of the left hand side diff, but moves to the first change on the right hand side.
The focus on the left hand side is often not visible, I cases, where the first change requires the view to scroll down. Posting it in this thread as it seems relevant to me

@sorosoro2 The diff editor is today exposed to screen readers as two separate editors, the left-hand-side (or original), which is read-only, and the right-hand-side (or modified), which, depending on whom integrates us, is read-only or editable. We need to change this and expose the diff editor to screen readers in a different way. This work is tracked in https://github.com/Microsoft/vscode/issues/17263

This  issue exists in Azure Steam Analytics VSO ,
Bug Link : https://msdata.visualstudio.com/Azure Stream Analytics/_workitems?id=39899&_a=edit
Same issue exists in Azure Portal VSO,
Bug Link : http://vstfrd:8080/Azure/RD/_workitems/edit/9286067?fullScreen=false

Hi VS Code PM here,
I wanted to provide some more details about why screen readers are not reading line numbers for user navigates to each line:

Reading a line number at the start of every line might be noisy for some users, and would degrade the experience. Namely, we invest a lot in making sure that the information is read in the most concise way possible.
There is a limitation to our HTMLTextArea that we are using that does not give us full control over decorating lines with additional context information like line numbers that the Screen Readers would read. So there is not technically good way to fix this with our current approach.
There are two easy workarounds to find out the line number:

First is ctrl+g that opens the command palette, reads the line number and allows the user to navigate to any other line number
Second is focusing on the status bar (F1 > focus status bar, or F6), and the status bar will read the current line number

I hope that helps and provides some clarity why this issue is still not tackled. Thank you for understanding 🙏


[SPARK-35907][CORE] Instead of File#mkdirs, Files#createDirectories is expected.

What changes were proposed in this pull request?
The code of method: createDirectory in class: org.apache.spark.util.Utils is modified.
Why are the changes needed?
To solve the problem of ambiguous exception handling in traditional IO creating directories.
What's more, there shouldn't be an improper comment in Spark's source code.
Does this PR introduce any user-facing change?
Yes
The modified method would be called to create the working directory when Worker starts.
The modified method would be called to create local directories for storing block data when the class: DiskBlockManager instantiates.
The modified method would be called to create a temporary directory inside the given parent directory in several classes.
How was this patch tested?
I have provided test cases as much as possible.
Authored-by: Shockang<EMAIL_ADDRESS>

Thank you for making a PR, @Shockang . Could you enable GitHub Action on your Spark fork?

https://spark.apache.org/developer-tools.html (Testing with GitHub Actions workflow)

ok to test

Can one of the admins verify this patch?

Thank you for your first contribution, @Shockang . I left a few comments.

Thanks a lot.I will revise it according to your opinion.

@dongjoon-hyun It has been revised.could you please verify this patch?

Test build #140364 has started for PR 33101 at commit 435d125.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44895/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44895/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44895/

Test build #140364 has finished for PR 33101 at commit 435d125.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140364/

Test build #140450 has started for PR 33101 at commit b367ea6.

Test build #140450 has finished for PR 33101 at commit b367ea6.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140450/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44964/

Test build #140456 has started for PR 33101 at commit 983bb36.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44964/

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44969/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44969/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44964/

Test build #140456 has finished for PR 33101 at commit 983bb36.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140456/

Test build #140608 has started for PR 33101 at commit a958a5e.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45120/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/45120/

Test build #140608 has finished for PR 33101 at commit a958a5e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140608/

Test build #140611 has started for PR 33101 at commit 6d123ba.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45124/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45124/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/45124/

Test build #140611 has finished for PR 33101 at commit 6d123ba.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140611/

Looks fine. The tests may be overkill but OK :)
Are there any other instance of .mkdirs() out there that should use the utility method, while we're here?

Thank you for your attention  (｡･∀･)ﾉﾞ  In the Spark source code, there are two user-facing pieces of code and more than 30 test cases using mkdirs. Do you mean I need to replace all these places?

I wouldn't worry about the test cases, but if there are places in the source code that look like they probably should leverage the Utils code to make a dir, in order to get all the same behavior, yeah we could change that too here all at once. Up to your judgment - if in doubt post here and we can figure it out.

Merged to master


Unit of the Absolute Depth

Thank you very much for your awesome work first. I am just wondering, what is the unit/metrics of the predicted absolute depth? Is it in meter? Thanks!

Thank you very much for your awesome work first. I am just wondering, what is the unit/metrics of the predicted absolute depth? Is it in meter? Thanks!

Did you solve it? How should I get the actual distance from the estimated depth map? thanks ！
if model_type == "dpt_large":     net_w==net_h=384     model=DPTDepthModel(         path=model_path,         bachbone="vitl16_384",         non_negative=True,         enable_attention_hooks=False,     )

Please, refer to #62. Hope it helps

Please, refer to #62. Hope it helps

Thank you!

Thank you very much for your awesome work first. I am just wondering, what is the unit/metrics of the predicted absolute depth? Is it in meter? Thanks!

Did you solve it? How should I get the actual distance from the estimated depth map? thanks ！
if model_type == "dpt_large": net_w==net_h=384 model=DPTDepthModel( path=model_path, bachbone="vitl16_384", non_negative=True, enable_attention_hooks=False, )

By setting the flag of "--absolute_depth", the output will be the absolute depth without normalization. But I am not sure about the definition of "absolute depth" compared to the normalized depth

Please, refer to #62. Hope it helps

Thank you!

Thank you very much

Please, refer to #62. Hope it helps

Please, refer to #62. Hope it helps

Thank you very much for your reply! If the output is scale and shift invariant and isn't linked to any real distance, then what is the difference between the absolute depth and the depth after normalization? Both of them should be scale and shift invariant, why dont we keep only the normalized output? Thanks!!

Please, refer to #62. Hope it helps

Thank you very much for your reply! If the output is scale and shift invariant and isn't linked to any real distance, then what is the difference between the absolute depth and the depth after normalization? Both of them should be scale and shift invariant, why dont we keep only the normalized output? Thanks!!

The difference is that the absolute depth map preserves the real ratio between different depths (to the model accuracy, though), while normalized depth does not.
Knowing correct scale α and shift β, you can estimate the real depth   in your unit system from the absolute depth map  applying . On the other side, after applying normalization to your absolute depth map you lose the ability to estimate the real depth from this normalized map, because normalization is a non-linear operation.
You may need to normalize depth in order to display it as a picture, like
cv::normalize(D_abs, D_normalized, 0.0, 255.0, cv::NORM_MINMAX, CV_8U);
cv::imshow("D_normalized", D_normalized);

Please, refer to #62. Hope it helps

Thank you very much for your reply! If the output is scale and shift invariant and isn't linked to any real distance, then what is the difference between the absolute depth and the depth after normalization? Both of them should be scale and shift invariant, why dont we keep only the normalized output? Thanks!!

The difference is that the absolute depth map preserves the real ratio between different depths (to the model accuracy, though), while normalized depth does not. Knowing correct scale α and shift β, you can estimate the real depth  in your unit system from the absolute depth map  applying . On the other side, after applying normalization to your absolute depth map you lose the ability to estimate the real depth from this normalized map, because normalization is a non-linear operation. You may need to normalize depth in order to display it as a picture, like cv::normalize(D_abs, D_normalized, 0.0, 255.0, cv::NORM_MINMAX, CV_8U); cv::imshow("D_normalized", D_normalized);

Thanks a lot for the explanation! That is very helpful. So it is saying that, even though we might not now the real distance based on absolute depth without scale and shift, but we can still compare the depth of 2 images by comparing their distance from the absolute foreground depth to the absolute background depth, right?

Thanks a lot for the explanation! That is very helpful. So it is saying that, even though we might not now the real distance based on absolute depth without scale and shift, but we can still compare the depth of 2 images by comparing their distance from the absolute foreground depth to the absolute background depth, right?

I'm not sure about 2 (different) images. Different scenes may have different scales and shifts. Given an image of some scene, you can calculate absolute distance from one object (at the front) to another object (in the back) in that scene and that will be proportional to the real distance between these two objects. You can also calculate the real distance if you know the underlying scale and shift for this particular scene.
When you move to another image of another scene, there will be the second scale and shift, more likely different from the first ones.


cleanup(virtctl): Minor cleanups in virtctl ssh and scp functests

What this PR does
Apply minor cleanups to the virtctl ssh and scp functests:

Import order
Format of passed arguments
Use testsuite.GetTestNamespace(nil) where applicable
Use NewRepeatableVirtctlCommand instead of manually constructing cmd with CreateCommandWithNS

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

None

Same, great job!
/approve

/hold
@0xFelix the sig-compute test failures look relevant to me.

Reverted the change that caused the issue.
/hold cancel

Nice catch @brianmcarey!
/lgtm

/hold Unfortunately, looks like there are more issues

/hold cancel
Cannot replace CreateCommandWithNS with NewRepeatableVirtctlCommand. Added comments explaining why.

@0xFelix there's flakiness on the virtctl test going on: https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/12804/pull-kubevirt-check-tests-for-flakes/1833873011611013121
I'll create an issue for this.

@dhiller All tests failing in a recent run of check-tests-for-flakes have the exclude-native-ssh decorator.
https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/12804/pull-kubevirt-check-tests-for-flakes/1834166102834286592
These can only pass when the binary was built with the excludenative flag.

Happy to see further improvement points.
/lgtm


Requires gpg keys to use buildcache as a normal user

==> Installing libiconv-1.16-uaerdnaqvy2hrkk2gkiirw5mqkq7cg44
==> Fetching file:///shared/ucl/apps/spack/0.20/buildcache/build_cache/linux-rhel7-broadwell-gcc-11.2.1-libiconv-1.16-uaerdnaqvy2hrkk2gkiirw5mqkq7cg44.spec.json.sig
gpg: keyring `/scratch/home/uccagop/apps/spack/0.20/kg_site_micheal_ucl/spack/opt/spack/gpg/pubring.gpg' created
gpg: Signature made Wed Aug  2 16:22:38 2023 BST using RSA key ID 72643654
gpg: Can't check signature: No public key
==> Warning: Failed to verify: file:///shared/ucl/apps/spack/0.20/buildcache/build_cache/linux-rhel7-broadwell-gcc-11.2.1-libiconv-1.16-uaerdnaqvy2hrkk2gkiirw5mqkq7cg44.spec.json.sig
==> Warning: Skipping build of diffutils-3.8-ehpjj76pln2dakw5zahbo7eztvsb3jha since libiconv-1.16-uaerdnaqvy2hrkk2gkiirw5mqkq7cg44 failed
==> Warning: Skipping build of binutils-2.40-y5z2x63iwl6lski65ohvslxc56clv3nx since diffutils-3.8-ehpjj76pln2dakw5zahbo7eztvsb3jha failed
==> Warning: Skipping build of gcc-12.2.0-kut6cpbompf56zcvwf4vkipaifgmre7u since binutils-2.40-y5z2x63iwl6lski65ohvslxc56clv3nx failed

It looks like we just need to make sure that ccspapp exports the public gpg key when creating the buildcache, then third parties can use it.
https://spack.readthedocs.io/en/latest/command_index.html#spack-gpg-export
spack buildcache keys https://spack.readthedocs.io/en/latest/binary_caches.html#spack-buildcache-keys
List public keys available on Spack mirror (eg our buildcache).
(To read https://spack.readthedocs.io/en/latest/signing.html in case anything else we need to know).

Current code for creating a spack site says:
        self.run_command(['spack', 'gpg', 'create', 'ARCHPCSolutions'<EMAIL_ADDRESS>        # 2 steps to use the common build cache - thw first, a config copy, links it into this site as spack mirror
        shutil.copy(initial_mirrors_yaml, os.path.join(self.yaml_dir, 'mirrors.yaml'))
        self.run_command(['spack', 'buildcache', 'keys', '--install', '--trust'])```

So this says it is short term fix. The goal was to get the packages signed somehow (so a new key is created is created for the site), because if I recall correctly, it is much nicer to have the packages signed when it comes to the downloading step. Not sure why it was failing, when the steps are only to create an entirely new key and they say use it. There is nothing here dependent on a particular user id.

See also issue #28

It isn't failing to create the site, it is failing to import the existing packages from the buildcache - because it doesn't have the public gpg key they were signed with. If we export the gpg key into the buildcache with ccspapp (and make sure that the third party user then has that show up in spack buildcache keys) this should solve it.

At the moment, spack buildcache keys for Krishna has no entries, so this is not installing anything:  self.run_command(['spack', 'buildcache', 'keys', '--install', '--trust'])

Interesting.
==> Warning: Failed to verify: file:///shared/ucl/apps/spack/0.20/buildcache/build_cache/linux-rhel7-broadwell-gcc-11.2.1-gmake-4.4.1-3yeveltufgmue74npuzemeaxfiivvt6k.spec.json.sig
==> Error: Failed to install gmake due to NoVerifyException: Spack found new style signed binary packages, but was unable to verify any of them.  Please obtain and trust the correct public key.  If these are public spack binaries, please see the spack docs for locations where keys can be found.

But it still manages to install gcc 12.2.0 somehow?!
==> Error: m4-1.4.19-wiuk7k4zngqyewodi4yl4tsn3nyxytmi: Package was not installed
==> Error: gawk-5.1.1-pokguyuq6ahml5hbgn36776rfqbei4s7: Package was not installed
==> Error: pkgconf-1.8.0-rrab7kijpl5g2acssdn24cszudxvi3xn: Package was not installed
==> Error: zlib-1.2.13-gsyujwxfnx6vvfw46b4t3x63mc5xh3a5: Package was not installed
==> Error: gmp-6.2.1-scvmooxiyejglzu6fchh7atngtxphmyf: Package was not installed
==> Error: readline-8.1-jtrpaefvakqxyzbo3m72q6hcnuywt4uu: Package was not installed
==> Error: ncurses-6.3-6nrbosy4yebkogx2wiyom6ez36wxiz6g: Package was not installed
==> Error: texinfo-6.5-5xsqgvusoaq73mcu33a3fexgex4lhp4f: Package was not installed
==> Error: bzip2-1.0.8-mq5csufsb6jurd7zxg7x3hevqmps2pl3: Package was not installed
==> Error: diffutils-3.8-ehpjj76pln2dakw5zahbo7eztvsb3jha: Package was not installed
==> Error: gcc-12.2.0-kut6cpbompf56zcvwf4vkipaifgmre7u: Package was not installed
==> Error: libsigsegv-2.13-qw2elykliqxp2xlix4zmkrun2lmmiklv: Package was not installed
==> Error: libtool-2.4.7-5a6s7ssqxozebmitnnuvoteoehhorlqw: Package was not installed
==> Error: berkeley-db-18.1.40-6pqhjoddsyu4zfo4xyweynn5s2wsr7dh: Package was not installed
==> Error: autoconf-archive-2022.02.11-rmhkp3hczdktiv6co2buqt7gunykg7pb: Package was not installed
==> Error: zstd-1.5.2-myexac2eokkvlqpva26kef25kjwie4ju: Package was not installed
==> Error: autoconf-2.69-gphegdoewpjvxw5h45ufhqn7n4utlmjm: Package was not installed
==> Error: libiconv-1.16-uaerdnaqvy2hrkk2gkiirw5mqkq7cg44: Package was not installed
==> Error: perl-5.36.0-che7ezb5nsqiffbauv336ctwf3vty4xd: Package was not installed
==> Error: automake-1.16.5-3fsyut7o4pobb6e65if5yuofbk2ghxcn: Package was not installed
==> Error: mpc-1.2.1-t6o5yxx5bumgalqhdls7rzcjdj7iax2z: Package was not installed
==> Error: mpfr-4.1.0-43lsbjo5m3pxgpubn7aoxnc5tjjnxgue: Package was not installed
==> Error: Installation request failed.  Refer to reported errors for failing package(s).
# SPACKSITES: Now calling: #######
 source /scratch/scratch/uccagop/spack/0.20/hpc-spack/spacksites/process-env-scripts/spack-deps-rhel-7.8.sh

source  /home/uccagop/apps/spack/0.20/kg_spacksite_michael_ucl/spack/share/spack/setup-env.sh
spack config --scope=site get packages
# SPACKSITES: ####################
# SPACKSITES: found first compiler spec in packages.yaml<EMAIL_ADDRESS># SPACKSITES: Now calling: #######
 source /scratch/scratch/uccagop/spack/0.20/hpc-spack/spacksites/process-env-scripts/spack-deps-rhel-7.8.sh

source  /home/uccagop/apps/spack/0.20/kg_spacksite_michael_ucl/spack/share/spack/setup-env.sh
spack -e first_compiler_spack_env config get
# SPACKSITES: ####################
# SPACKSITES: Now calling: #######
 source /scratch/scratch/uccagop/spack/0.20/hpc-spack/spacksites/process-env-scripts/spack-deps-rhel-7.8.sh

source  /home/uccagop/apps/spack/0.20/kg_spacksite_michael_ucl/spack/share/spack/setup-env.sh
spack load<EMAIL_ADDRESS>spack compiler find --scope=site
# SPACKSITES: ####################

Ok, the buildcache's public key exists in /shared/ucl/apps/spack/0.20/buildcache/build_cache/_pgp/9EDD6764E6A3D5145ABF8E6EFD20819772643654.pub on Michael already, it appears.

https://spack.readthedocs.io/en/latest/getting_started.html#trusting-keys
Additional keys may be added to the keyring using spack gpg trust <keyfile>. Once a key is trusted, packages signed by the owner of they key may be installed.


Update azure-stack-sql-resource-provider-deploy.md

space in cert-name leads to error during deployment...

@Paxilein : Thanks for your contribution! The author, @mattbriggs, has been notified to review your proposed change.


Scrape Modules!

Hey,
I could add the Modules under every Social Media Tool site.
Alternatively, I could make a Scrape Modules Site, for more advanced researchers and simply link all Social Networks Modules in one page. ofc. with descriptions and more.
The argument for a new site is, that we would not overload the toollist and keep it simple. On the other hand, less sites are more lucid.
Whats your opinion :)?

I'd put them at the bottom of the existing pages with a clear hint that this is for advanced users only.


SLIM-1222 Fix end-to-end FirmwareUpdate

Changes made for FLEX-3580 caused the test with unavailable firmware to
fail. This adds a FirmwareFileExists key to the scenario step data, and
makes sure no attempt is made to read in firmware file bytes in case the
firmware file is not available.

retest this please


updating cwlogs collection docs

PLEASE READ
Following a recent back-end update, all contributors with a local clone or fork of our repository are required to run yarn install. This does not apply to direct page edits.

[x] Yes, I've run yarn install
[ ] No, does not apply to me

Purpose of this pull request
This pull request...

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions, .clabot
[ ] Update Content - Revisions, updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

@himsharma01, I found a few things to update.

Thanks for your review @jpipkin1 . Changes looks good to me.

@himsharma01, I'll go ahead and merge this PR.


Ocaml runtime files not up to date

The Ocaml runtime files appear to be out of date, the commentstring isn't set in vim's runtime files while it is in the vim-ocaml repo.

Then the maintainers of the Ocaml runtime files should send Bram an update.

Oh, sorry, I assumed they did send an update, given how old the change was. After taking a deeper look at the repo I linked, it doesn't seem to be maintained by one of the maintainers listed in the runtime files. Would it be fine if I just sent a single-line patch to add a set commentstring=(*%s*) to ftplugin/ocaml.vim? It's the one thing I'm really interested in.

Oh, sorry, I assumed they did send an update, given how old the change
was. After taking a deeper look at the repo I linked, it doesn't seem
to be maintained by one of the maintainers listed in the runtime
files. Would it be fine if I just sent a single-line patch to add a
set commentstring=(*%s*) to ftplugin/ocaml.vim? It's the one thing
I'm really interested in.

The problem with including patches from various sources is that once the
maintainer sends me a new version, I have no idea if those patches are
included or not.  If not, then we'll fall back to old behavior.
That is why I very much prefer for the maintainer to take care of
patches.  Also to do a check that it works for everyone.
--
From "know your smileys":
:-X	My lips are sealed
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

My problem is that I can't find the "official repo" for these runtime files. The link in the current runtime file is dead, and digging a little deeper only leads to a personal repository that does more that just aggregating ocaml runtime files. The repo I linked to doesn't belong to anybody on the maintainer list of the current runtime file.
I'll try to send an email to each person listed in the file and see what they answer, but I get the (maybe misguided) feeling that the ocaml runtime files aren't maintained by them anymore. I'll close the issue for now.


Add Zefenify

Website: https://zefenify.com/about.html
Distribution Repository: https://github.com/Zefenify/Wolf-Cola
Currently, only macOS build is available with Windows build in the pipeline.

Proof for Electron via screenshots

Thank you


feat: Engine K8S ingress for REST API reverse proxy routing

Description:
Create engine K8S ingress for the REST API port so we can interact with the API via Traefik.  The Host header is set to engine.
Tested with k3d locally:
curl -i http://localhost:9730/engine/info -H "Host: engine"
HTTP/1.1 200 OK
...
{"engine_version":"ae7579"}

Is this change user facing?
NO
References (if applicable):
#1941

left few comments but LGTM

Thanks @lostbean for the review!


Could not download 'pyannote/segmentation-3.0' model.

Hello, I recently tried to spin up this program again, but it seems like there's some problem with speaker diarization.
It worked okay month ago, and I have not changed anything in the config file.
I tried to fix it generating new token and making sure I was gated in all three programs like in YouTube tutorial, but it's still failing.
`Could not download 'pyannote/segmentation-3.0' model.
It might be because the model is private or gated so make
sure to authenticate. Visit https://hf.co/settings/tokens to
create your access token and retry with:

Model.from_pretrained('pyannote/segmentation-3.0',
...                       use_auth_token=YOUR_AUTH_TOKEN)

If this still does not work, it might be because the model is gated:
visit https://hf.co/pyannote/segmentation-3.0 to accept the user conditions.
Traceback (most recent call last):
File "c:\Users\arkad\Desktop\ai\audiosplitter_whisper\venv\Scripts\whisperx-script.py", line 33, in
sys.exit(load_entry_point('whisperx==3.1.1', 'console_scripts', 'whisperx')())
File "C:\Users\arkad\Desktop\ai\audiosplitter_whisper\venv\lib\site-packages\whisperx\transcribe.py", line 211, in cli
diarize_model = DiarizationPipeline(use_auth_token=hf_token, device=device)
File "C:\Users\arkad\Desktop\ai\audiosplitter_whisper\venv\lib\site-packages\whisperx\diarize.py", line 19, in init
self.model = Pipeline.from_pretrained(model_name, use_auth_token=use_auth_token).to(device)
File "C:\Users\arkad\Desktop\ai\audiosplitter_whisper\venv\lib\site-packages\pyannote\audio\core\pipeline.py", line 136, in from_pretrained
pipeline = Klass(**params)
File "C:\Users\arkad\Desktop\ai\audiosplitter_whisper\venv\lib\site-packages\pyannote\audio\pipelines\speaker_diarization.py", line 128, in init
model: Model = get_model(segmentation, use_auth_token=use_auth_token)
File "C:\Users\arkad\Desktop\ai\audiosplitter_whisper\venv\lib\site-packages\pyannote\audio\pipelines\utils\getter.py", line 89, in get_model
model.eval()
AttributeError: 'NoneType' object has no attribute 'eval'
Traceback (most recent call last):
File "c:\Users\arkad\Desktop\ai\audiosplitter_whisper\split_audio.py", line 183, in
main()
File "c:\Users\arkad\Desktop\ai\audiosplitter_whisper\split_audio.py", line 180, in main
process_audio_files(input_folder, settings)
File "c:\Users\arkad\Desktop\ai\audiosplitter_whisper\split_audio.py", line 148, in process_audio_files
diarize_audio_with_srt(audio_file_path, srt_file, speaker_segments_dir)
File "c:\Users\arkad\Desktop\ai\audiosplitter_whisper\split_audio.py", line 77, in diarize_audio_with_srt
subs = pysrt.open(srt_file)
File "C:\Users\arkad\Desktop\ai\audiosplitter_whisper\venv\lib\site-packages\pysrt\srtfile.py", line 151, in open
source_file, encoding = cls._open_unicode_file(path, claimed_encoding=encoding)
File "C:\Users\arkad\Desktop\ai\audiosplitter_whisper\venv\lib\site-packages\pysrt\srtfile.py", line 292, in _open_unicode_file
encoding = claimed_encoding or cls._detect_encoding(path)
File "C:\Users\arkad\Desktop\ai\audiosplitter_whisper\venv\lib\site-packages\pysrt\srtfile.py", line 279, in _detect_encoding
file_descriptor = open(path, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'C:\Users\arkad\Desktop\ai\Pudzian\output\plik.srt'`

Nevermind, I just found the issue.
In YouTube video we are supposed to accept conditions for:
https://huggingface.co/pyannote/segmentation
but the script is requiring
https://huggingface.co/pyannote/segmentation-3.0
which looks like is never version of segmentation.
I agreed to 3.0 conditions, but it's getting stuck on >>Performing diarization...

C:\whisperX\1\audiosplitter_whisper\venv\Lib\site-packages\whisperx\diarize.py
https://hf.co/pyannote/speaker-diarization-3.1 for me ...


ServiceFabricDeployV1 doesn't work with new ServiceFabric structure

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: ServiceFabricDeployV1
Environment
Azure Pipelines

Agent - Hosted or Private:
Hosted Agend

Issue Description
As described in https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-startupservices-model there is a new Service Fabric Structure with StatupServices.xml. With this change the Deploy-FabricApplication.ps1 was also adapted.
Within Deploy-FabricApplication.ps1 there a plenty new changes like:
.PARAMETER StartupServicesFile
Path to the file containing the startup services for debugging.

...

Param
(
    [String]
    $PublishProfileFile,

    [String]
    $StartupServicesFile,

...

    $publishProfile.StartupServiceParameterFile = [System.IO.Path]::Combine($PublishProfileFolder, $publishProfileXml.PublishProfile.StartupServiceParameterFile.Path)

...

if ($IsUpgrade)
{
    if ($StartupServicesFile)
    {
        $PublishParameters['StartupServicesFileMode'] = $true
    }

    $Action = "RegisterAndUpgrade"
    if ($DeployOnly)

...

  # Pass the path to the Startup Services File if it was provided
    if ($StartupServicesFile)
    {
        $PublishParameters['StartupServicesFilePath'] = $StartupServicesFile

        if (-not [string]::IsNullOrEmpty($publishProfile.StartupServiceParameterFile))
        {
            $PublishParameters['StartupServiceParameterFilePath'] = $publishProfile.StartupServiceParameterFile
        }
    }

These changes are currently not available in Publish-NewServiceFabricApplication.ps1 and Publish-UpgradedServiceFabricApplication.ps1.
Due to that fact the Services are not created during the deploy via Azure Pipelines and have to be created manually in the Service Fabric Explorer.
As long as these changes are not available in that Task, the new Service Fabric Structure is unusable for our application.

This is a big problem as we migrate to the latest Service Fabric project files. We have found that if we manually establish the services, and then use upgrade tasks with package diff from then on, subsequent deployments works fine, but the initial creation does not setup the services and they have to be manually created.

I don't know if it changed with more recent versions of the SDK, but upgrades via package deploys no longer work and simply leave the services completely uncreated (which is an even bigger problem in stateful scenarios). Any idea on a timeline for a fix here? This basically makes moving to the new startup parameters impossible, or I have to deploy without using our typical release pipelines.

is there any update or plan to create a new version of the task to support the new structure format?

Is there an update on this issue?

Is there any update on this issue?

I am pretty sure this issue is still existing.

The issue exists and it would be interesting to provide a way to package the Services with a Default Services conversion included


Figure out test settings for TravisCI to be able to spin it up

I'm going to fiddle with this for a minute to see if I can it to line up.

I think I got this.


M3-3127 Improve charts display

Improve charts display
Charts needed to have different styles and color matching, as well as fixing some consistency with responsive styles.
This includes the charts in both the Linodes and NodeBalancers summaries.
Type of Change

Non breaking change ('update')

@martinmckenna fixed as well


Would you be interested in consolidating with Andrew-Chen-Wang/github-wiki-action ?

If we were to fix the following problems with https://github.com/Andrew-Chen-Wang/github-wiki-action that this repo https://github.com/spenserblack/actions-wiki has advantaged over it, would you @spenserblack be interested in consolidating the ecosystem?
Here's a list of things that https://github.com/Andrew-Chen-Wang/github-wiki-action CANT DO NOW but this repo can (you could advertise these on the readme!):

[ ] push to an arbitrary repo https://github.com/spenserblack/actions-wiki/blob/main/action.yml#L7
[ ] work on windows? i don't know that this is an issue since I haven't tested this. I'm going off of what was remarked on in https://github.com/spenserblack/actions-wiki/issues/4#issuecomment-1444982155
[ ] use plain git commands to smartly avoid using rsync or similar

Would you be interested in collaborating with me to combine/interfold/whatever this repo, the git subtree trick from https://github.com/jcbhmr/publish-wiki-action-OLD and mash those changes into the https://github.com/Andrew-Chen-Wang/github-wiki-action ?
https://github.com/Andrew-Chen-Wang/github-wiki-action/discussions/19
If not, that's OK too. It's good to have competition in an ecosystem! ❤

Actually, I think I'm going to close this (for now at least). If you want to do this at some point to consolidate the ecosystem, great! But for now, I think the goals between this and the Andrew-Chen-Wang/github-wiki-action don't quite align perfectly.
Andrew-Chen-Wang/github-wiki-action: Be the "all in one" wiki action? I think? Not really entirely clear yet....
This repo: Push things to a wiki. That's it; but do it well. Is this correct?

Push things to a wiki. That's it; but do it well. Is this correct?

Yup, I think that summarizes it well. In the workflow that I imagined when making this, .github/wiki would basically be the source of truth. The wiki would just be a method of displaying these documents.


Ubuntu issue

Hi there ... I did all the steps and double checked everything ... maybe I'm missing something I donw know. Whenever I try to use the proxpn command (as root) and choose any server from the list, I get this message:
Running:
/usr/sbin/openvpn --config "/etc/proxpn/proxpn.ovpn" --remote "mfl1.proxpn.com" 443 --auth-user-pass "/etc/proxpn/login.conf" --auth-nocache
Options error: In [CMD-LINE]:1: Error opening configuration file: "/etc/proxpn/proxpn.ovpn"
Use --help for more information.
Any ideas of what could it be?
Thanks!

Hmmmm, could you paste the output of sudo cat /etc/proxpn/proxpn.ovpn and sudo ls -la /etc/proxpn/

Sorry about that, I figured out the issue and just pushed a hotfix to master in #20

Thanks a lot Matt! I'll test it today when I get home :+1:


Add FK from assets to users

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


Feature/tri 306 orchestration asynchronous calls over irs api

Asynchronous Job Orchestration

You should fix the problem with code-formatter as soon as possible. Right now we need to review multiple files, instead only few with real changes.


Fix currently broken goldsky sources

What is it?
Some goldsky sources have stopped delivering new data. We need to have it all restarted for now.

Working for now.


Output Gallery can't parse metadata from the .CSV when VAE and LoRA included

Eyeballing 44a8f2f8db469a99ee018cfc727a2f8ccf2d9866, the output gallery parameter info will be broken in the above circumstance when embedded .PNG data or a matching .JSON file is not available. I haven't actually tested it, but it should be broken. Don't know whether it will be an exception, or it will just not find any parameter info.
Ugly fix:

Move VAE and LoRA fields before the Output field in the .CSV so it's the last again
Define a new shape for the .CSV parsing code.

Proper fix:

Actually write the .CSV data with headers so we don't have to guess based on number of fields.

It was not an exception, it was just not finding the parameter info.


how to access a partially visible text with its -actual value

I have a list box which contains year from - 2023 til 2028.
But in the UI the list box is shown in a compact form so only first 3 digits of the above values are shown. e.g. 202 instead of 2023 and 202 instead of 2028.
in my automation scenario how will i handle such a case where i want to select 3rd or 4th value in the list and access it by its value 2025 or 2026 while the UI still shows 202 for all the values. Refer the attached screen shot below

await actor.attemptsTo

Appreciate if feed back if any on this from the @playwright team

@msabya just so you know, this isn't the official Playwright project. Regardless, this issue needs more information. What have you tried to do to select the element? Are you using playwright-testing-library to do so?

See: https://github.com/testing-library/playwright-testing-library/issues/597#issuecomment-1441248688


docs: Add rkt, Docker, and Kubernetes bootcfg deployment notes

@colhom Does infra have a NodePort selection policy? I've used  32401 in the bootcfg example, a random-ish choice. Asking because you chose a port in https://github.com/coreos/docker-nginx-https-redirect/blob/master/contrib/k8s/https-redirect-service.yaml#L11

@dghubble no real rule for assigning node ports. I tend to use 31XXX for production services, and 32XXX for their staging analogues. Just a "pattern", not really a rule.
I expect that we'll be transitioning away from nodeports as we can make better use of load-balancer services anyways. Maintaining this mapping out-of-band is hideous.

I'll just use the randomly selected 31488 NodePort for the generic example then. Folks who have k8s compatible load balancers can edit how they like. Thanks Colin!

BTW, you should be able to not specify the node port, and k8s will pick for you, alternatively, I'd actually say you should just leave it at type: ClusterIP as the default.

Hm, maybe I've optimized for my typical use cases. I choose a fixed NodePort per service globally (rather than letting k8s decide) to support load balancing across nodes in multiple Kubernetes clusters (baby ubernetes). The Service IP is determined at the load balancer level.
What's your aim for using ClusterIP in the example?

Well, I guess using a clusterIP doesnt make much sense, since nodes wont be on the k8s network, thus they wouldn't be able to use the clusterIP to access it. Never mind.

+1 for fixed ports in examples


Mixed Inputs and Output pins in a single IO Expander

Hi! First of all amazing driver!
I have a question! In on of the PCF8575 IC I have mixed inputs and outputs. The first 8 (IO0 to IO7) pins are inputs and the last 8 (IO8 to IO15) pins are outputs. How I can define that. I was trying to use the setButtonMask() function but without any luck.
Any help here?

Thanks for the question,
Rather busy so might take a few days

You do not have to set a data direction register with these PCF857x devices.
You can do a
uint16_t  inputs = PCF.read16() & 0x00FF;. // Select lower 8 bits / lines
uint16_t outputs = b10011101;. // Set some bits high and some low
PCF.write16(outputs << 8);. // Shift them to upper bits.

Should do the trick.

I was hoping for a direction register but I can see on the datasheet that there isn’t one!
I will try the code you send me, hopefully it will solve the problem.
It seems that if I have mixed inputs and outputs on one IC the interrupt pin is always firing! And I can’t solve that somehow…
at least for now

You could keep the last / previous value of read16() and compare with current one.
mask the output pins and you would see which input triggered the interrupt.

The problem is that somehow I cannot reset the interrupt! Since it goes active...  then I can't set it again to the inactive state! And I don't know why is doing that!

Ok, it seems that is working now... a read from all pins seems to solve the problem!

Ok, if solved you may close the issue.


[top_darjeeling, otp_ctrl] Implement SW partition test

Adds a simple test that checks selected SW partitions in OTP_CTRL that have different attributes.
This closes #599.

The only nit would probably be to use the new opentitan_test rule, but if you prefer to punt that to someone else, we can take care of that later.

This is for the integrated_dev branch, where I think we haven't transitioned to the new rules just yet.


possible to include examples of libsignify usage?

would it be possible to get examples, in the readme or dedicated examples/ directory, of using libsignify to validate files using a given pub key please? digging through the source is ok, but a concise example could be helpful too.

Thanks for the idea :) This sounds reasonable to me. I actually intended for the signify CLI here to serve as that example but looking back on that idea now seems weird.
I'll try and put an example up that does "the whole flow" for inspiration.

thanks! if it helps, context from my case: i was trying to use ureq to grab say the SHA256.sig file or a set and verify it as a simple test

I'm super sorry this one got dropped onto the floor. I kept meaning to put something up but always got sidetracked. Does #55 contain what you think you'd need to start again from scratch?

@BlackHoleFox no worries on the delays! that looks good to me yeah and makes sense. i did have to look up why you had a _ in parenthesis but it makes sense to me. thanks for adding the basic example to the docs too


Is there a way to use an ng-required in the schema?

I have several controls where validation depends on previous responses to questions. I use the required attribute in the schema but was wondering if there was an ng-required.

Basically I have some controls that are required only if others are answered a specific way. I am currently using ng-required. Can this framework schema definition support an required condition??

@ngabello Currently not really since we don't support oneOf or anyOf schema definition yet. (we're working on it).
What you can do is to force it in the form definition instead with a combination of condition and required. I.e. you can say that a field is required in the form defintion, and by listing a field twice, but with different conditions you can selectively give the user a required field.
There is an example on the demo page, http://textalk.github.io/angular-schema-form/examples/bootstrap-example.html
Choose: "hack: conditional required"

I solved it using a custom form type, not ideal but effective.
div class="form-group {{form.htmlClass}} schema-form-select"
     ng-class="{'has-error': form.disableErrorState !== true && hasError(), 'has-success': form.disableSuccessState !== true && hasSuccess(), 'has-feedback': form.feedback !== false}">

    {{form.title}}

  <select ng-model="$$value$$"
          ng-model-options="form.ngModelOptions"
          ng-disabled="form.readonly"
          sf-changed="form"
          class="form-control {{form.fieldHtmlClass}}"
          schema-validate="form"
          ng-options="item.value as item.name group by item.group for item in form.titleMap"
          name="{{form.key.slice(-1)[0]}}"
          ng-required="{{form.requiredCondition}}">
  </select>

```


Android: Sample not working when turning on accessibility features on Samsung devices

When turning on accessibility features on Samsung android devices, popups don't display.
Tested using the current sample.
Settings > Accessibility > Visibility enhancements > Remove animations = turn on
Settings > Accessibility > Visibility enhancements > Reduce transparency and blur = turn on
https://github.com/LuckyDucko/Mopups/assets/131879389/1e224ca4-7929-4adb-af23-702ae4cfab30

Huh, thats unusual. Will keep this open as accessibility is important. Will clause #68 as i believe this is a more general tracker


D  issue #49

In response to issue #49, D_ was added which defines the zero-field splitting in terms of D and E/D. The update includes tests and documentation.

Potential dependencies: zfsframes - would need a rewrite to to include the option to analyze a spin system. But I'm not sure how much sense that makes; DStrain - would it make sense to introduce E/D strain? or does strain in E cover this adequately. Aside from those two, given this merely translates D_ to D in validatespinsys I can't think of any other dependencies.


ENS loading issues

If the network is slow, users may submit a transition before ENS load up. It will end up with Invalid Address error message.

[ ] need an Indication to tell users, the address is loading up.

Once triggered Invalid Address error message, the wallet will stop loading the ENS. Means once users click the send before ENS load up, the ENS will never load up again, unless you kill the app and restart it.

[ ] need to continue loading.

@vladyslav-iosdev Help to check if still applicable, and fix or close

@vladyslav-iosdev Help to check if still applicable, and fix or close

ok


feature/add-required-text

My attempt to add required to save/publish text to the form labels.
I took a stab at this, but I'm sure it's not there yet. Specifically, I wasn't sure if I should be trying to render the label again or do a replace to add the new required text to the existing label.

Resolves https://github.com/punchcard-cms/punchcard/issues/404 (will still need to submit PR in Punchcard for CSS update)
DCO 1.1 Signed-off-by: Nick Tilden<EMAIL_ADDRESS>

As you can see from the failing test, you've got some issues off the bat; namely that label and inp aren't defined in the scope you're working in. Try starting there.

Hi @ineedsubstance
I didn't realize you wanted to tackle actual code!
So, once you have the dev environment working and testing correctly...the actual place you want to make a change like this is in the addRequired function. You'll need to dig into the regex to find the end of the label tag and tack on your mark there.
For testing, you'll want to look into /tests/form.js and /tests/html.js for getting your changes tested. For instance, in this tested result the system checks that the proper className is added to the label. You would want another check there to make sure that mark got properly added.
Hope this helps. Scott

Thanks, @scottnath that helps. Yeah, I was hoping to do the styling and code if possible. My javascript is pretty weak, but I'm trying to figure it out so I can contribute where I can.

@scottnath + @Snugug - Updated my approach based on Scott's recommendation and also got testing to work locally which helped immensely. Closer?

@Snugug @scottnath - OK, let's try this again. 😁

@Snugug - in regards to this comment, I added a test for a repeatable and an object and included required to publish in those. I wasn't sure if this was exactly what you were asking for, but here you go.
@scottnath - I commented on your comment. Basically, it didn't pass the test, so I changed the first part but didn't remove the conditional statement.


Content is reset at unexpected times

I'm sure there's a simpler repro than this, but I'm making a note of what I observed just now for further investigation later.

Go to https://learn.svelte.dev/tutorial/textarea-inputs
Hit 'solve'
Change - lists to - [links](https://example.com)
Click the newly-rendered link in the output section
Close the tab

The content will have been reset (this happens for me in Firefox and Chrome).
Weirdly, you have to actually close the tab — if you leave it open and navigate back to the tutorial, the content won't have been reset. And if you drag the newly opened tab into a separate window and then close it, nothing will happen. It's... strange.

I found that the iframe was reloading.
https://user-images.githubusercontent.com/29677552/227553967-32d3c1bf-6e21-45b1-821e-287595f9b2ee.mov

The relevant code is here.
https://github.com/sveltejs/learn.svelte.dev/blob/6736ff71af56c3c6cada90e0ad13f299e387d962/content/tutorial/common/src/__client.js#L112-L124
https://github.com/sveltejs/learn.svelte.dev/blob/6736ff71af56c3c6cada90e0ad13f299e387d962/src/routes/tutorial/[slug]/Output.svelte#L54-L62

This is caused by throttling of setInterval and setTimeout.
https://developer.chrome.com/blog/timer-throttling-in-chrome-88/
The ping in __client.js runs every 100ms with setInterval and sends the message. The handle_message in Output.svelte receives that message. If the message does not come for more than 1000 milliseconds, the iframe is reloaded.
However, when the tab goes to the background, setInterval and setTimeout are throttled to run approximately every 1000 - 1500 milliseconds. This means that sometimes the setTimeout delay time in the handle_message is shorter than the ping delay time, at which time set_iframe_src is called and the iframe is reloaded.
I have found the cause, but I don't know how to fix it.

Thanks for the advice, I'll try it.


Split creation and runtime effects

The idea of this change is to have the ability to use different effect types for resource creation and its runtime.
For example, one can want to use Kleisli, as the "creation" effect, but IO as the runtime effect.
Also, it would be great if we could lower the boundaries of the "creation" effect. Currently, it's Concurrent: ContextShift because of the FutureLift requirements.
WDYT?

Build failed on Install nix step 🤔

Not sure I follow what you're trying to do there, got a usage example in mind?


add sparksql wirte mysql support update ,the design from replace into…

The modification points are: Spark SQL writing MySQL supports update
Background and purpose In the current big data scenario, when writing to the MySQL relational database, the redundancy of data needs to be updated according to the primary key or unique key, so this operation is required
Technical point: use MySQL's replace into syntax for technical transformation

The modification points are: Spark SQL writing MySQL supports update
Background and purpose In the current big data scenario, when writing to the MySQL relational database, the redundancy of data needs to be updated according to the primary key or unique key, so this operation is required
Technical point: use MySQL's replace into syntax for technical transformation


Comparison report privado branch change

Changed the default privado-core-enterprise head branch to dev

Closing this PR as the fallback should be the main branch.


XIONE-12754 : added support to fetch sky model name

Closing older PRs. Pls bring it to latest sprint/main branches if required.


fix: use TinaProvider instead of Tina

Following #877

@all-contributors please add @DirtyF for code

should this be chore and not fix?


Duplicated text in README.md

There is duplicated text in the README.md

Thanks for fixing Merve!


Fixed CI issues caused by moving OSPERF Bench to apps/benchmarks

Summary
Fixed CI issues caused by moving OSPERF in https://github.com/apache/nuttx-apps/pull/2346
Error Log:
Configuration/Tool: qemu-armv8a/nsh_smp
  Cleaning...
  Configuring...
  Building NuttX...
On branch master
Your branch is up to date with 'origin/master'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   boards/arm64/qemu/qemu-armv8a/configs/nsh_smp/defconfig

no changes added to commit (use "git add" and/or "git commit -a")

Impact
move CONFIG_TESTING_OSPFER to CONFIG_BENCHMARK_OSPERF
Testing
None

let's merge both patch and wait the report from next ci run.


Receiver fallbacks

Description
Receiver fallbacks are implemented by making getInvoiceableWallets aware of previously failed wrapped invoices. If the id of such a failed invoice is passed to it (predecessorId), it will skip any wallets for which a forward was already attempted.
For this, a column predecessorId was added to the Invoice table which points to the previously failed invoice.
With this link back to the "predecessor invoice", we can check which wallet was already tried and skip that wallet in this recursive query.
Since we want to retry with the same sender wallet on receiver errors, the client is made aware of receiver errors via the existing invoice polling. If status is FAILED_FORWARD or the invoice is held + cancelled, the invoice polling will throw WalletReceiverError now. We need to check both since it's dependent on timing what the client will see first (FAILED_FORWARD transitions almost immediately to FAILED).
If there is no more receiver wallet to try, we fallback to an invoice that pays SN and credits the receiver with CCs.
close #1494 ~based on #1679~ based on #1700
TODOs

[x] don't log failed forwards as sender errors
[ ] make sure we avoid infinite retry loops if there ever is a bug?

Tests

[x] sender with 1 good wallet and no CCs pays receiver with 2 bad wallets: falls back to CCs
[x] sender with 1 bad and 1 good wallet and no CCs pays receiver with 2 bad wallets: falls back to CCs
[x] paying with CCs (even though both have wallets attached)
[x] sender with 1 bad and 1 good wallet and no CCs pays receiver with 2 good wallets: p2p zap success to first good receiver wallet

Additional Context

I am testing failed forward using this patch:

diff --git a/worker/paidAction.js b/worker/paidAction.js
index 70396ddb..9d711b0f 100644
--- a/worker/paidAction.js
+++ b/worker/paidAction.js
@@ -268,7 +268,7 @@ export async function paidActionForwarding ({ data: { invoiceId, ...args }, mode
     payViaPaymentRequest({
       lnd,
       request: bolt11,
-      max_fee_mtokens: String(maxFeeMsats),
+      max_fee_mtokens: String(0),
       pathfinding_timeout: LND_PATHFINDING_TIMEOUT_MS,
       max_timeout_height: maxTimeoutHeight
     }).catch(console.error)

The properties wallet, message, reason and invoice  of the custom errors can probably be moved into the parent class but that should be refactored in a separate PR imo.

https://github.com/stackernews/stacker.news/pull/1688#discussion_r1878331597

Checklist
Are your changes backwards compatible? Please answer below:
yes, this only adds a nullable database column
On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:
7. See tests above and I looked very closely at the code for extended periods of time
For frontend changes: Tested on mobile, light and dark mode? Please answer below:
n/a
Did you introduce any new environment variables? If so, call them out explicitly here:
no

Won't this cause unwanted sats -> CC conversion?
If all the receivers fail, i think it should try to pay with CC -> CC, not  LN -> CC

I think it should prioritize p2p if the receiver has attached wallets.

I think zaps should be paid in the interest of the sender by default. Therefore, without a setting where the sender can express their preference that depends on if the receiver can receive real sats, I think we should try CCs first.
I think it would also be confusing if we try CCs first for posts and replies but not for zaps.[^1] Preferring sats over CCs should be opt-in imo so this would need a setting which I think we can add later if there's enough demand from users. We shouldn't assume too much about user preferences and I think receiver fallbacks without this setting are valuable enough by themselves already.
So currently, I think it's not worth it to add the complexity of two different payment flows (one for SN fees, one for zaps[^2]) where one even now also includes a sending dependency on receiving next to the already existing dependency of receiving on sending  (a receiver cannot avoid to get paid in CCs if the sender only has CCs). The dependency of the receiver on the sender is already tricky UX-wise (need to explain to them that their wallet isn't broken but what they receive depends on the sender etc) so I'd rather not make sending also depend on receiving unless there is enough user demand for such a thing.
[^2]: was this handled in your PR?
Thanks for flagging though, this wasn't on my mind! I'll look into which changes would be required in theory to accommodate this if/when we need it while I review my implementation.
I am also sorry if I missed some existing discussion between @huumn and you about this. I am pretty sure I did. Could be that I am also disagreeing with @huumn regarding this here :eyes:
[^1]: I assume for posting fees it's clear that everybody will want to try CCs first and we won't need a setting?

otherwise i think the system will end up becoming almost CC exclusive.

30% of each zap goes into rewards (which become sats) so the half life of a CC is almost exactly two zaps: 1 * 0.7 * 0.7 = 0.49 < 0.50.
Therefore, I think how users zap each other doesn't matter much for the amount of CCs in the system, it matters more how many use attached wallets. If everybody has an attached wallet, CCs will cease to exist quickly enough.

#1678 tries p2p first if the sender and receiver has an attached wallet. If the receiver wallet fails, then I think the sender should just do CCs (either those they already have, or jit purchasing them).
Either way, it shouldn't require a massive overhaul to change the priority of payment methods. Sender and receiver fallbacks should be pretty well encapsulated from payment priority.

Converted to draft because this requires a rebase on #1700

Okay, superficially tested again based on #1700 and worked as expected, this is ready for review again.

I fixed the sender error checking by returning fowardStatus which exposes the status of the withdrawal to the sender.

Should we also use that in the recursive query so client and server definitely have the same view about receiver errors?

Should we also use that in the recursive query so client and server definitely have the same view about receiver errors?

Up to you


Autodetect pahntomjs location

The note is that it will fail on non UNIX system.

No worries, I have changed it to be cross plattform of course :P

Well done. I missed that commit. Thanks.


Check for Serverless resources without Transform

Previously cfn-lint accepted resources of Type AWS::Serverless::* unconditionally. With this commit it only accepts them with the AWS::Serverless-2016-10-31 Transform.
Let me know if this is the wrong file to define the test in, or if there's some false positive edge case I haven't considered.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Nice find 👍
We indeed "ignore" serverless on certain locations. I'm just not sure if this is right place. One example is what happens when you add multiple resources:
../cloudformation.yaml:30:5

E3001 Serverless Transform required for Type AWS::Serverless::Function for resource AgeFunction in us-east-1
../cloudformation.yaml:53:5

Maybe this could be a new rule? Since the check basically is a "if Serverles and no transform" check
🤔
@kddejong Any thoughts on this?

Ah interesting, so you're saying instead of adding a rule match for each Serverless resource just add one rule match on the Transform if there are any Serverless resources. I'll see if I can put that together.

@orez- ,
Yup! I'd advice to create a new rule to handle this specific use case. Keeps it clean, can be disabled and is basically a “feature” on it’s own (don't mix rules)
If you need need help on creating rules you can take a look at the documentation or one of the other rules 👍

Alright, I moved this check into its own rule. Let me know any more notes or issues, or if I've forgotten anything!

I see that my change is failing a code coverage check, I think because the new rule has conditionals to stop checking when the template is misconfigured that aren't being exercised. Should I add test cases for those paths?

Thanks for updating this @orez-


Remove deprecated arguments and functions

[ ] Closes #xxxx (Replace xxxx with the Github issue number)
[ ] Tests added: Please use assert_type() to assert the type of any return value

This is for the 1.4 deprecations

Opened https://github.com/pandas-dev/pandas/pull/48599 to add deprecate flags in the docs


[nick] Automate geo-restriction bypass

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
This makes youtube-dl automatically bypass geo-restrictions for nick.com, without having to use --geo-bypass-country US. The other sites in this extractor do not appear to have geo-restrictions, I tried playing full episodes from them with a Hungarian IP.
Example URL: http://www.nick.com/the-thundermans/videos/parks-t-rex-s4-ep409-full-episode/

Post example URL.

Oops, I thought I did. Added.


feat: added repository settings UI component

What does this PR do?

Fixes # (issue)

How should this be tested?

Test A
Test B

Checklist

Required

[ ] Filled out the "How to test" section in this PR
[ ] Read How we Code at oss.gg
[ ] Self-reviewed my own code
[ ] Commented on my code in hard-to-understand bits
[ ] Ran pnpm build
[ ] Checked for warnings, there are none
[ ] Removed all console.logs
[ ] Merged the latest changes from main onto my branch with git pull origin main
[ ] My changes don't cause any responsiveness issues

Appreciated

[ ] If a UI change was made: Added a screen recording or screenshots to this PR
[ ] Updated the Formbricks Docs if changes were necessary

Hey @Chigala - thanks for the PR :)
I have tidied it up a bit and asked @pandeymangg to have a look at the upload functionality. Will do a deeper review once he is done with it to get this PR merged.
Thanks a lot for your work so far 💪

/award 400


How can i convert junit.xml generated using xcpretty after the test run to a pretty cucumber report as in cucumber-reporting plugin for java

XCPrerry has few formatters like rspec , tap and knock .. Refer xcpretty README here

On other note, its related to xcfit at all ..


feat(#355): Add More Informative Exception Messages

Add context if some opcode wasn't recognized.
Closes: #355.

History:

feat(#355): add careful exception handling if opcode is not recognized
feat(#355): add const instructions

PR-Codex overview
This PR focuses on improving error handling and exception messages in the BytecodeClass and BytecodeMethod classes.
Detailed summary

Added a catch block to handle UnrecognizedOpcode exception in BytecodeMethod class
Added error handling and exception messages in BytecodeClass class
Added a new UnrecognizedOpcode class to represent the exception for unrecognized opcodes
Updated the tests to cover the new error handling and exception messages

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@yegor256 Could you take a look, please?

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 4min)


[Paper Confusion] Figure 3

I have read this so many times and one thing i can not make sense is that with an edge (o3, r2, o2) the output vector v'3 should be calculate by function gs but in this graph it was calculated by go. Is there any mistake or am i misleading something?

One more small thing that the output of vr2 should be v'r2 also but it was v'r1, i'm so confused right now.

Hey..
I was also confused at this point.
I guess it's just a labeling error. The correct order should be g0, gp, gs.

Yes this figure has typos. Sorry for the confusion!


remove invalid Go code

Issue #, if available:
n/a
Description of changes:
Removed a line that was obviously copied from a diff, and that does not compile.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

lol this is a dupe of #34


Make sure format() returns string.

I am getting a strange failure when testing some old Drupal code:

TypeError: Behat\Behat\Output\Printer\Formatter\ConsoleFormatter::format(): Return value must be of type string, null returned in Behat\Behat\Output\Printer\Formatter\ConsoleFormatter->format() (line 33 of /usr/share/devshop/src/DevShop/Control/vendor/behat/behat/src/Behat/Behat/Output/Printer/Formatter/ConsoleFormatter.php).

Looking into it, I discovered format() calls preg_replace_callback, which returns null|string|string[].
This change returns an empty string if preg_replace_callback returns null.
Should we check for an array as well?

Can you share a reproducer for a case making the regex fail ?

Should we check for an array as well?

No. preg_replace_callback returns an array when its subject argument is an array, which is not the case here.

I ran into this when the response body (which was checked) was generated by https://github.com/filp/whoops due to an exception being thrown. Unfortunately I cannot share the content because it's full of internal information.
The HTML was 1 MB large with a 6 level stack trace and roughly a thousand environment variables.
The error for preg_replace_callback was PREG_BACKTRACK_LIMIT_ERROR with pcre.backtrack_limit being 1000000 (the default).
I believe issue #1431 is about the same problem.
Maybe this helps to craft a test case.

Just hit the same issue! And this fix is working fine!

It'd be good to get this updated & merged. Ideally with a test to prove it's working as expected.

@acoulton I updated this to the latest master, added an error message if the preg function fails and added a unit test (we can now safely use preg_last_error_msg as we are now using PHP8)


Add xauth support

We should consider what will happen in case of a local console session connection as well.
One user might use .Xauthority file for some local session using lightdm for example.
Than connect with xrdp and overwrite this .Xauthority file
What will be the implications?

Nobody suggests overwriting .Xauthority file. xauth operates on records. If would not remove records for other displays.

#545 will be much better :)


NOWFULL Izzy Green Viral Leaks Video On Social Media X Twitter[1810K]

⭐⭐⭐⭐⭐
26 sec ago Izzy Green Leaked Video Viral On Social Media. NOWFULL Izzy Green Viral Leaks Video On Social Media X Twitter[1810K]

Izzy Green, a young and talented digital creator, recently captured the public's attention with an intriguing video that has gone viral across various social media platforms. The video, known as the "Izzy Green Spiderman," has drawn in countless viewers and sparked discussions among users on Instagram, Twitter (X), and Discord. This remarkable success not only highlights Sophie’s ability to create engaging content but also demonstrates how social media can serve as a powerful tool for promoting creative works. Many people are eager to watch the video and learn more about the individual behind it.
For those interested in watching the full video of Izzy Green, numerous links have been shared across these platforms, making it easy for fans to access the content that has taken the internet by storm. This phenomenon illustrates how quickly information and entertainment can spread in today’s digital age. As more individuals engage in conversations about the video, Izzy Green is not only gaining popularity but also opening up new opportunities in her career as a content creator.


Import with error

the message is:
Error loading extension:res//Godot-MTerrain-plugin-0.9.3/libs/MTerrain.gdextension
my godot vesion is: v4.0.2

The GDExtension library has been build for Godot 4.1, and if you look the MTerrain.gdextension file it's said minimum version required is 4.1
I think this will work with Godot 4.2, But if you want to run this on Godot 4.0, You need to change the Godot-Cpp version in GDExtension folder to 4.0, and remove minimum version required in MTerrain.gdextension file, and build the library by yourself, That may work and may not work, I am not sure


when doing global rollbacking ,may cause an endless loop

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Ⅰ. Issue Description
if there is a table like
CREATE TABLEt_test(  uridint NOT NULL,  name varchar(64) DEFAULT NULL,   PRIMARY KEY (urid),   UNIQUE KEY t_test_name (name) ) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_0900_ai_ci;
name is a uniquekey.  and There is a piece of data in the table. (1,1)
1、global Transaction execute the sql "delete from t_test where id = 1", and this transaction need rollback
2、when rollbacking,other thread execute a sql like "insert into t_test values (2,1)"
3、that will cause the rollback thread  endless loop.
Ⅱ. Describe what happened
that's because the range of the code "catch (SQLIntegrityConstraintViolationException e)" is too wide  in AbstractUndoLogManager#undo
and rollback of business will be catched by this.
that should just catch "insertUndoLogWithGlobalFinished"

JDK version(e.g. java -version):
Seata client/server version:  1.6.X
Database version: mysql 8.X
OS(e.g. uname -a): centos7
Others: jdk8

If it is confirmed that it is not specially designed like this, I can mention pr to fix it.

When data needs to be restored to a state before deletion, and its unique index is being used by other rows, do you think this data is unrecoverable? Should such data be restored in an incomplete state?

It should be unrecoverable, at least from the point of view of seata. I think it can be treated as a dirty write operation, so that seata-server can keep the retry at a fixed frequency.

It should be unrecoverable, at least from the point of view of seata. I think it can be treated as a dirty write operation, so that seata-server can keep the retry at a fixed frequency.

I agree with this approach. In fact, it is unrecoverable.

@funky-eyes Please assign to me, I will fix it.

ok


How could I get the score of the attention while translate?

I'm trying to analyse the alignment of the model, and I need the attention score, but I not sure whether this line code local softmaxOut = self.decoder.softmaxAttn.output was the output of the attention module.
Thank you for your help.

It is but the attention is directly outputted by the translator. See:
https://github.com/OpenNMT/OpenNMT/blob/master/onmt/translate/Translator.lua#L279

Thank you for your help, then I know how to get it. @guillaumekln


Please updated 'Enable Application Insights integration' content

Please update 'Enable Application Insights integration' content.
'Application Insights' setting UI on Create has been updated and 'APPINSIGHTS_INSTRUMENTATIONKEY' is set automatically after created a new function app enabling Application Insights.  Thanks.

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

ID: 13672b67-22ea-3dda-8b6b-3332c24e6030
Version Independent ID: fdc60aeb-9971-1d9f-a6ba-b3ad818cf0d5
Content: Monitor Azure Functions
Content Source: articles/azure-functions/functions-monitoring.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hi @kazshi Thank you for your feedback! We will review and update as appropriate.

@kazshi Yes. The UI when creating a new function app has indeed changed but setting the key is still required if the Application Insights is created separately from the function app.
Then again, the new UI does support linking an existing Application Insights resource while creating the function app too but I believe its best to still mention how the resources are linked nonetheless.

We are assigning the issue to the content author to review and update accordingly.

@PramodValavala-MSFT Thanks for your explanation. I understand your point but I couldn't understand that from the current content. I'm expecting to see up-to-date content anyway.  Thanks.

@kazshi thanks for the feedback I will try to clarify this.

This should now be resolved. #please-close