Recreate shares.json in case of deletion

To be able to clear shares when testing, we need to be able to reload
the json files after they have been deleted.
Later on, in a real-world scenario, we need to be able to rebuild the
json file based on information stored in the storage. (see inline TODO)
@butonic
Test steps:

Create a folder "test"
Share "test" with "Marie"
Refresh the page, see that the share exists
Delete "shares.json"
Refresh the page, see that the shares are gone

Before this fix: the shares would still be there until the service is reloaded
After this fix: the shares are gone directly and the file is recreated empty.
TODOs:

[ ] get preliminary review for approach
[ ] remove print statements
[ ] apply the same approach for publicshares.json

maybe instead of overriding Lock/Unlock, could simply set the flag in the Save() method before and after writing to catch the events that were triggered by us

@PVince81 as this is mainly for testing: why don't you recreate the daemon again?

@individual-it it might be possible to restart the "reva-sharing" service alone, might be faster to boot up then restarting everything ?

hmm, the problem might be that the container where the services run is not the same.
they might have shared volumes but not shared processes.
restarting the service would require having some kind of API in the container that runs reva to be able to tell it to restart services.
@individual-it correct ?

yes we would have to talk somehow across the container to restart the service.
Anyway how would an admin do it? If a user is deleted it would be impractical to restart the complete service.
Would it make sense to recreate the shares if a change in the user is detected, e.g. the id. I guess even if a user is recreated in LDAP with the same name, email etc. the ID should be different, or?

won't be continued here


Evaluate Cloud Computing Services

We have to choose a main provider for VMs and maybe services similar to Google BigQuery (i.e. scalable and performant data storage). This decision has to be documented.
Possible candidates:

Google Cloud (actually personally my preferred option, as it has worked well for the Trisma project at QUT)
AWS
Heidelberg Cloud

Feel free to add more providers.


Feature/key callback

 (3) Track what saml key is used for decrypting assertions

@mhickman looks great!

unrelated: we should consider submitting a PR from our fork into instructure's repo since this is generic enough IMO.


Demo is broken (disables drag) after a search

Demo at https://geoapify.github.io/leaflet-address-search-plugin/ seems broken.
Steps to reproduce:

Open webpage
Drag the map around. Map drags as expected.
Click on search field
Enter name of a city
Click on a search result. Map fits bounds as expected.
Drag the map around. Map dragging has been disabled without any user notification, nor any explicit function call nor configuration setting about it.

Tested on Firefox 95 and Chromium 93 on a Debian linux box. Using trackpad (i.e. not touchscreen, not digitizer tablet). Not sure if the bug is reproducible in other platforms.

Demo at https://geoapify.github.io/leaflet-address-search-plugin/ seems broken.
Steps to reproduce:

Open webpage
Drag the map around. Map drags as expected.
Click on search field
Enter name of a city
Click on a search result. Map fits bounds as expected.
Drag the map around. Map dragging has been disabled without any user notification, nor any explicit function call nor configuration setting about it.

Tested on Firefox 95 and Chromium 93 on a Debian linux box. Using trackpad (i.e. not touchscreen, not digitizer tablet). Not sure if the bug is reproducible in other platforms.

Nice catch! Thank you! I've found the problem and fixed it.
It should be OK now. Nevertheless, I believe there will be improvements and more plugin versions. The plugin is not a part of the Leaflet repository, so it does not affect the library itself.


enable priority and fairness for kube-apiserver

This is a new feature for kube-apiserver.  We want soak time.  If it performs well, we may well keep this on.
/hold
holding for proof

proof in https://github.com/openshift/cluster-kube-apiserver-operator/pull/818

/hold cancel
proof is good. Manual upgrade tests passed at about existing flake rates.

/lgtm


Better method of downloading reference dbs

Description of feature
For example, the amplicon mode lets you download from ncbi, embl, or bold, but you can't download from multiple dbs in one run. It could be worth letting the user download from multiple sources, then concatenate the reference fasta files.

I've got two possible ideas.

In the Amplicon workflow, we can run CRABS_DBDOWNLOAD for each db the user want to download from, then run CRABS_DBMERGE.

We can have a subworkflow DOWNLOAD_DBS_AND_MERGE() that prepares the DBs before the amplicon_workflow. The output of this subworkflow can also be used as input to other simulation modes (e.g., metagenome simulation)

I'm not a fan of CRABS_DBDOWNLOAD so I might try different options. This nf-core module seems like a possible alternative https://nf-co.re/modules/mitohifi_findmitoreference

The other idea I have is a module or subworkflow at the start of the pipeline that downloads genomes based on a list of species provided by the user to create a reference fasta file. This fasta file could be input for downstream modes in the pipeline like the --metagenome or --amplicon mode.

ncbi-genome-download has been added


Mini Configured with LuCI?

Not actually an issue, but a question.
Is LuCI supposed to be functional after booting a mini build?
I installed OpenWrt-AO-21.02-20211214.2321-NanoPi-r4s-ext4-mini and after all LEDs were on I see that I do pull an IP via DHCP, but the web interface is just a directory listing showing a single folder that can't be entered.
Thanks for your work, this is a great system for OpenWRT

Mini version is command-line only. You'll want the full version for LuCI.

If I catch it at the right time, I may try a CLI install of LuCI.
I just didn't want the docker functionality in the build, so, chose mini.
So, both you and looking at the seed answered my question.
Thanks


TV Ver. Crashing!?

I can't open it, well... I can it just waits a sec, I can't do anything, I did once do something, and after a sec it crashes...

I'm seeing the crash in crashlytics. I'll look into it.

I should have fixed it with https://github.com/jakepurple13/OtakuWorld/releases/tag/v30.0.2


weaver_semconv_gen: Fix typos/differences for event snippets

See: https://github.com/open-telemetry/semantic-conventions/pull/917#discussion_r1576539838

Closing this as won't fix.  Instead, we're moving all the snippets to jinja templates, see: https://github.com/open-telemetry/semantic-conventions/pull/1000


[Bug]: Unable to delete Legacy UA connection

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Upgrade DNN site to 9.13.0
Went to connections
Tried to delete Legacy UA connection
Does not delete connection
Console error

Steps to reproduce?

Go to settings
Go to connections
Try to delete existing Legacy UA connection

Current Behavior
Unable to delete connection
Expected Behavior
Delete legacy UA connection
Relevant log output
jquery.js?cdv=595:2  Uncaught Error: Syntax error, unrecognized expression: #connector-GoogleAnalytics(LegacyUA)
    at I.error (jquery.js?cdv=595:2:11769)
    at Y (jquery.js?cdv=595:2:17234)
    at re (jquery.js?cdv=595:2:20008)
    at Function.I [as find] (jquery.js?cdv=595:2:7520)
    at ce.fn.init.find (jquery.js?cdv=595:2:21991)
    at new ce.fn.init (jquery.js?cdv=595:2:22483)
    at ce (jquery.js?cdv=595:2:1077)
    at Connectors.js?cdv=595:181:45

Anything else?
No response
Affected Versions
9.13.0 (latest release)
What browsers are you seeing the problem on?
Microsoft Edge
Code of Conduct

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

Good catch, the element ID is probably driven by the module name and parenthesis can't be used for an element id.

Also cannot update an existing Connector.  I cant change the value for Google Tag Manager

Ah - great catch!  I'll see if I can look into this during tomorrow's co-coding session on Discord if nobody else has addressed it by then.

Closing per #5972


Hide sub-navigation on mobile to stop screen flickering while scrolling

Fixes https://github.com/sourcegraph/handbook/issues/3666
The sidebar appears above the content on screen sizes below 900px wide.
As we scroll down the page, sub-menus in the sidebar become active.
This pushes content down which deactivates the sub-menu, causing a loop.
An alternative solution would be to always display sub-navigation on screens below 900px, but this pushes content way down pages with many nav items, which is probably not ideal.

I confirmed this fixes the issue for me locally.

Thank you so much, @nielthiart!


Re-embed and index without document titles

Maybe we would be better off if the embeddings were based only upon the page_content and not the titles? For this task you will need to:

cd to the data directory, download the splits from: https://www.dropbox.com/scl/fi/xg5vvhyt20zhqf23plj5d/split.tgz?rlkey=amfa17ibbddo0v0tqx4mvejel&dl=0 and unpack them using tar -xzvf split.tgz This will create 17 subdirectories under split. You can ignore the model subdirectory.
Make sure the .env file in your models repo contains your personal PINECONE_API_KEY and PINECONE_ENV! You want to index this files into your personal pinecone index, not the live production index. Don't start notebook 35_indexer until you have done this.
In the embed_documents function in index_utils.py, only pass the page_content to the embedder
Run the 35_indexer notebook for each of the subdirectories under split (except model) to index the files using just the page_content. Make sure that the index_stats say that the index contains 0 entries when you run the notebook for the first time. It will take a while to run the notebook for each of the subdirectories.
Install and run the server following the instructions in the README.md file in the server repo. Make sure that your personal PINECONE_API_KEY and PINECONE_ENV values are in the .env file in the server repo.
Follow the instructions in the Experiment Process Explained task to determine if using just the page_content to generate the embedding is better.

Indexing without document titles dropped the NDCG score two points.


[BUG] 'app_log_path' is undefined

Description
In the task "Ensure skeleton log paths", two vars are used for : app_log_path and app_log_file. Only app_log_file is defined in the defaults, and it seems these two vars are in fact the same.
Steps to Reproduce
Run cookiecutter. Remove comments of app_log_file var and converge.
Expected behavior:
Clean converge.
Actual behavior:
TASK [app_role : APP | Ensure skeleton log paths] ******************************
fatal: [app]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'app_log_path' is undefined\n\nThe error appears to be in '/home/ablopez/Dev/cookiecutter-ansible-role/app_role/tasks/install.yml': line 34, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: APP | Ensure skeleton log paths\n  ^ here\n"}
Reproduces how often:
100%.
Environment
N/A
Additional Information
N/A

Fixed in 2.8.0 release


Queries in the SDF Loss Computation

Hi
First of all thank you for the wonderful work and for releasing the code-base. I had some queries regarding the implementation of the SDF loss which seems slightly different than that mentioned in paper. It would be great to know the reasoning behind this-

https://github.com/HengyiWang/Co-SLAM/blob/dfb69c80beff76492309ffa626c5865ba8d8f0be/model/utils.py#L142
In this case while computing the free space loss, it seems like the loss is being computed with respect to 1 and not w.r.t the truncation value. Is there a multiplication with the truncation value missing in the equation ?

https://github.com/HengyiWang/Co-SLAM/blob/dfb69c80beff76492309ffa626c5865ba8d8f0be/model/utils.py#L143
In this case while computing the sdf loss, why is there a multiplication with truncation ?

While computing the masks for sdf and freespace, is there any particular reason why only the points in the front are considered for free-space loss and not the points in the back ? This seems a bit different then the paper where everything which is not in the truncation region is considered as free space.

Thank you
Best
Manthan

Hi @manthan99,
Thank you for your interest in our work. I'll address each of your points below:
1&2. The reason behind this difference is that we aim for the output of the MLP to be in the range of [-1, 1] instead of [-truncation, truncation]. In other words, the network predicts a scale factor or percentage rather than the truncation value itself. Therefore, for the free space loss, we use a value of 1. And for the truncation region, we need to multiply the truncation.

This is because Co-SLAM focuses on indoor scene reconstruction. In such scenarios, a single ray might pass through multiple rooms or objects. If we consider points in the back as free space, it might conflict with the actual scene geometry. Therefore, we only consider the points in the front as free space.

I hope these clarifications address your concerns. If you have any further questions or need more information, please feel free to ask :)

Hi @HengyiWang
Thank you for the quick response. This makes it quite clear.
I had an additional question regarding the computation of the weights  (fs_weight and sdf_weight). Could you please provide some intuition behind this.
https://github.com/HengyiWang/Co-SLAM/blob/dfb69c80beff76492309ffa626c5865ba8d8f0be/model/utils.py#L106
Let's say for example in the case where all points are lying in the truncation region, then the sdf_weight will be zero in this case. So we won't get any SDF loss in this case ?
Thank you
Best
Manthan

Thanks for your question @manthan99. The computation of the weights (fs_weight and sdf_weight) aims to address the issue of imbalance between free space and SDF samples during training.
Let's consider an example where we shoot a ray of 5m in a room-scale scene, with a truncation value set to 10cm. In such a scenario, the majority of the sampled points would likely fall within the free space region. This significant imbalance between free space and SDF samples can cause the network to bias towards predicting all points as free space, neglecting the SDF information.
Regarding your concern about the SDF weight being 0, it is right. However, this will not happen in our setting as our near-far range (usually 3-8m) is significantly larger than truncation.

Thank you for clearing my doubts.

Hi @HengyiWang,
Following up on your earlier comment on the truncation value

1&2. The reason behind this difference is that we aim for the output of the MLP to be in the range of [-1, 1] instead of [-truncation, truncation]. In other words, the network predicts a scale factor or percentage rather than the truncation value itself. Therefore, for the free space loss, we use a value of 1. And for the truncation region, we need to multiply the truncation.

My understanding is that the SDF value outputted from the network is equivalent to (actual SDF value * truncation). If this is the case, shouldn't the equation for obtaining the weights be sigmoid(SDF) instead of sigmoid(SDF/truncation) in this line of code to follow the equations of the paper?
https://github.com/HengyiWang/Co-SLAM/blob/49cf9e6f947b7e13d453eb8070dee3974d959a50/model/scene_rep.py#L74
I tried removing the division by the truncation value, but the output mesh is completely wrong, so I would hope you help my understand why.

Hi @iAhmedMaher, that is a really good question, in the rendering pipeline of neuralRGBD, the SDF is always in the normalized space. You can probably check the response of the author of neuralRGBD here.


Clean up STM32 examples

Remove old-PAC usage for RCC if RccPeripheral is implemented enough.
So we can stop copy/pasting stuff that isn't needed into new examples.

@timokroeger Would you consider this "done" for now?

Let’s keep it open until we have embassy::main for all series (e.g. #319). I will work on that the coming days.
Moving DbgMcu::enable_all() to embassy::main would be another outstanding cleanup task. But I can track this in another issue.

Right on, no worries at all. Much appreciation for what you're doing!

I think this issue can be closed now

:ok_hand:


lcov report not showing all files

When I open lcov report some files are missing from it.
It's very strange because I can also see other files which are being imported from those files which are missing.
Is there some common case when people are experiencing this sort of an issue?
.babelrc
{
  ...,
  "env":
  {
    "test":
    {
      "plugins": ["istanbul"]
    }
  }
}

package.json
  "scripts": {
    "test-nyc": "cross-env NODE_ENV=test nyc mocha --require ./test/setup.js \"source/**/*.test.js\" \"test/**/*.test.js\"",
  },
  "nyc": {
    "require": [
      "babel-register"
    ],
    "reporter": [
      "lcov",
      "text-summary"
    ],
    "include": [
      "source"
    ],
    "exclude": [
      "**/*.test.js"
    ],
    "sourceMap": false,
    "instrument": false
  },

Update: I even removed exclude and added all: true - still meta.js file is missing. Seems that it ignores specifically meta.js file.
Screenshots:

Update:
I renamed meta.js to meta2.js and it sees meta2.js. But it doesn't see meta.js.
Also I left it as meta.js and renamed the containing folder from meta to meta2 - and it started seeing meta.js.
Seems that nyc is having issues ignoring files when their enclosing folder has the same name as the file itself.

Strange...
After renaming the file and its enclosing folder back and forth the issue disappeared.
Now it sees meta.js inside the meta folder.
Don't know what kind of bug that was.
Closing then.


[docs] fix: Add missing transifex resources

What's the issue?
There are some Transfiex resources missing in the configuration.
What's the fix?
Add missing Transfiex resources entries.

This is similar to #852. Take a look and we should merge them I guess. Not sure why we didnt check this in thinking The PR looks fine

@Wayonb yes, I noticed. #852 contains some entries that should not be added (vision should be not translated). So I decided that quicker will be did this one.


[DOCS] Typo in "Local development with Docker Compose" doc

Description
In the fourth point, there should be an "are" in the sentence to make it gramatically right.
Screenshots

Additional information
No response

i don't think so nothing wrong in that sentence

"If you making" is incorrect. It should be- "If you are making"

@zaidahmad25 are you working on this ?

No

@kumarsonsoff3 plz work after getting Assigned to the issue next time !
Anyways thanks for the contribution 😄

Hey @HarshDeep61034, You might get it wrong. Being a maintainer, I'm aware of this. 😄
Here, @GauravBurande has already created a PR for this issue. I just linked this issue with that PR to avoid multiple PRs for the same issue. And Gaurav, I recommend you to always get assigned first before working on an issue.

Sure, I'll keep that in mind.

Hey @HarshDeep61034, You might get it wrong. Being a maintainer, I'm aware of this. 😄
Here, @GauravBurande has already created a PR for this issue. I just linked this issue with that PR to avoid multiple PRs for the same issue. And Gaurav, I recommend you to always get assigned first before working on an issue.

Ohh my bad 😅
But i don't think so he was assigned either.
No problem it was just a typo 👍

Yeah, I didn't work on this since I wasn't assigned 😅


'Restore Default' is misleading

Using the restore defaults button doesn't return your registry to the state it was in before the programme was used, it simply sets things to the opposite of what they were, this is a deal breaker for me, is there any possibility of this changing?

Using the restore defaults button doesn't return your registry to the state it was in before the programme was used, it simply sets things to the opposite of what they were, this is a deal breaker for me, is there any possibility of this changing?

Where did you see this restore default option?

The successor Winpilot handles things completely differently and is a breeze to use.


Xcode 10 / Swift 4.2 ABI change

//
//  NibResource+UIKit.swift
//  R.swift.Library
//
//  Created by Mathijs Kadijk on 06-01-16.
//  From: https://github.com/mac-cain13/R.swift.Library
//  License: MIT License
//

import Foundation
import UIKit

public extension NibResourceType {
  /**
   Instantiate the nib to get the top-level objects from this nib

   - parameter ownerOrNil: The owner, if the owner parameter is nil, connections to File's Owner are not permitted.
   - parameter options: Options are identical to the options specified with -[NSBundle loadNibNamed:owner:options:]

   - returns: An array containing the top-level objects from the NIB
   */
  public func instantiate(withOwner ownerOrNil: Any?, options optionsOrNil: [AnyHashable : Any]? = [:]) -> [Any] {
    return UINib(resource: self).instantiate(withOwner: ownerOrNil, options: optionsOrNil)
  }
}

Error:
return UINib(resource: self).instantiate(withOwner: ownerOrNil, options: optionsOrNil)
Cannot convert value of type '[AnyHashable : Any]?' to expected argument type '[UINib.OptionsKey : Any]?

This is updated in the library, there just isn't a release yet.
This commit fixes the issue: https://github.com/mac-cain13/R.swift.Library/commit/38252b2c5a54b619b362127db6b2d22c43fe1f20
For now you can use the master version of the library if you want to use R.swift with Xcode 10.

That is correct, R.swift 4 is compatible with Xcode 9, R.swift 5 has the changes needed for Xcode 10.
We we'll drop support for Xcode 9 with R.swift 5.

the error still occurs. swift 4.2

Modify the Podfile:
pod 'R.swift', '~> 5.0.0.alpha.2'

the error still occurs. swift 4.2 / Xcode 10.1 / Rswift 5.0.0

just for reference for anyone finding this issue: I solved this by just setting swift 4.2 to the R.swift.Library dependency (before upgrading I had to set it to 4.1)


yarl (0.8.1) not compatible and not able to add http headers

yarl (0.8.1)  https://github.com/aio-libs/yarl/issues/9
File "/usr/local/lib/python3.5/dist-packages/aioes/connection.py", line 41, in perform_request
    url = self._base_url.with_path(url)
AttributeError: 'URL' object has no attribute 'with_path'

In connection.py, line 41, higher version of yarl,  yarl.URL seems deprecate the with_path method
And
I am not able to add http headers when I perform http request in es operation
I want to pass header as argument when instantiate the es client
headers = {"Host": u"***", "key": "***"}
es = Elasticsearch(['localhost:8888'], headers=headers)

TypeError: __init__() got an unexpected keyword argument 'headers'

with elasticsearch==5.1.0, I can do that
Now
I need to reserve header argument in connection.py->Connection->perform_request, and reserve for every method/class using it

aioes delete_by_query use something like this DELETE twitter/_query
not compatible with latest document
https://www.elastic.co/guide/en/elasticsearch/reference/current/docs-delete-by-query.html
delete_by_query should be like POST twitter/_delete_by_query


Question: Relationship between operator versions and argocd versions

This is a question about how/whether the operator is intended to support different versions of argocd.
The operator has a default docker image tag for the argocd images. This is a very specific sha tag. But there's also a version field in the spec for me to provide my own version.
However, looking at the code, it looks like the intention is that operator version 0.0.13 is only intended to be able to install Argo 1.6.1.
So my question is what the design goal is here, what the philosophy is. My guess is that a version of the operator is only intended to be able to support one (at least one minor) version of argocd. Maybe in future the operator might branch so that 0.0.x would support only the 1.6 argocd line, and 0.1.x would support the 1.7 argocd line. But the intention wouldn't ever be to try and write one version of operator source with conditional logic in it that says "if argocd1.7 is required, do this, else do that"?
Apologies if this is documented somewhere, I couldn't find anything.

That's fine. That was what I was expecting, I just wanted to confirm.


Avoid the need of a fixed dictionary for the mutex system

Having a fixed dictionary for mutex keys can limit what we can do with mutexes.
Let's avoid this limit.

:+1:


Remove non-existent _.last overloads, add _().last chainable form

_.last has a whole bunch of overloads that aren't present in the real API: https://lodash.com/docs#last
In addition, it didn't have a chainable version, which is present in the real API.
Both now fixed.

@bczengel @chrootsu could you review this PR?

@vvakame I think it's really unnecessary signatures :+1:
https://github.com/lodash/lodash/blob/3.10.0/lodash.src.js#L5206

@chrootsu thanks!
@pimterry thanks mate!


helm CLI incorrectly splits on commas even when quoted

When using the helm command and overriding values using --set, if the key value contains a comma, the value is halted there.
I'm assuming the entire argument is being split on commas, regardless of escaping or quotes.
Testing:
$ helm install kubernetes-charts/mysql --set mysqlUser="test_mysqluser,blah,dee,dah" --dry-run --debug | grep test_mysqluser

RETURNS
          value: "test_mysqluser"

I can confirm got cut by this bug yesterday too

If someone can point me in the right direction (where in the code is this handled?) - I'll take a look at creating a PR.  To me, it looks like an issue @ https://github.com/kubernetes/helm/blob/master/cmd/helm/install.go#L268
If someone can confirm if I'm at least thinking correctly.

@aegixx That is the correct location. That whole parser needs to be rewritten to be more robust. There's another issue with dots being unescapable. A PR would be awesome.

Instead of writing a better parser for handling the splitting accounting for quotes why not allow to specify --set multiple times? I realise this is a breaking change but would be cleaner and easier to maintain IMO.

Unfortunately this won't help for my issue. I'm setting environment variables that have a longer string that just happens to contain a comma. Multiple sets won't help with that specific issue.
--Bryan
On Fri, Nov 18, 2016 at 5:40 PM -0600, "Fabian Ruff"<EMAIL_ADDRESS>wrote:
Instead of writing a better parser for handling the splitting accounting for quotes why not allow to specify --set multiple times? I realise this is a breaking change but would be cleaner and easier to maintain IMO.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

The unfortunate thing, @databus23, is that we can't break backward compatibility, and Option 2 would break the existing behavior.

we can't break backward compatibility

helm already breaks compatibility with the tiller, just bump major and fix this

For anyone else coming up on this issue, there is a workaround by escaping the commas with a backslash:
https://github.com/kubernetes/helm/blob/master/docs/using_helm.md#the-format-and-limitations-of---set
Just as a side note, we still believe that this a very unintuitive behaviour that needs a fix. Our use case right now was LDAP credentials, which naturally contain commas, but it also seems very problematic for autogenerated passwords, that might accidentally contain a comma.

Just been banging my head against this with helm 2.8.2. You can now have multiple '--set' arguments, so the old comma syntax can (or should) be deprecated. It was always clunky and I stopped using it as soon as multiple '--set' worked. However it is still the case that as soon as I have a comma in the value helm splits it and borks the deployment. No amount of quoting seems to work around it.
Could we ensure we deprecate the commas (maybe with a warning message) in 2.9.x, and not include it in 3.x?

Although I couldn't find any mention or example in the document, reading the improved parsing code, you can 'escape' a comma, e.g. pass\,word and bypass the splitting. So --set 'my.password=pass\,word' will work until the comma splitting can be deprecated.

About the comma-separated key value pairs, having the option to turn comma-separation off would be nice, as we need to specify strings with multiple commas in them (eg. mongo connection strings), and escaping these from within shell scripts/CD tooling/environment variables can be a pain.

@dsample ®

Trying to define the cron schedule on the command line but this is making things more difficult than it needs to be. :(

This is not a good user experience.  For an escape, I had to do something like this:
helm install dev dgraph/dgraph \
  --set alpha.extraEnvs[0].name=DGRAPH_ALPHA_WHITELIST \
  --set alpha.extraEnvs[0].value='<IP_ADDRESS>\/8\,<IP_ADDRESS>/12\,<IP_ADDRESS>\/16'

This is not a good user experience. For an escape, I had to do something like this:
helm install dev dgraph/dgraph \
  --set alpha.extraEnvs[0].name=DGRAPH_ALPHA_WHITELIST \
  --set alpha.extraEnvs[0].value='<IP_ADDRESS>\/8\,<IP_ADDRESS>/12\,<IP_ADDRESS>\/16'

Just spent 4 days beating my head against the desk to find this. I will try this next. I ended up writing out to a temp file then telling helm to use a second --values file. This does seem like it's still an issue.

Still in the same situation.

Still in the same situation.

you should put your contents into a YAML file and provide it as an extra --values file. See: https://github.com/helm/helm/issues/1556#issuecomment-928177543

Still in the same situation.

you should put your contents into a YAML file and provide it as an extra --values file. See: #1556 (comment)

Thanks a lot. I've escaped comas and this helped. It would be too harsh to create a separate values file just for one parameter.
But I will be glad if someone will add the mention of this behavior to docs. Because I didn't find specific info on this.

Not quite sure how https://github.com/Nordix/helm/commit/54eb73b4239a2ec144085f569774aea14a6c6674 is supposed to fix this and I still encounter this issue when using helm v3.14.2.
Is it just me?

Not quite sure how Nordix@54eb73b is supposed to fix this and I still encounter this issue when using helm v3.14.2.
Is it just me?

No, it’s not just you. I’ve just spent good few hours pulling my hair out in attempt to troubleshoot this. Everything worked in dev environment where I deploy using Docker Compose, but once I’ve pushed to Test AKS using helm… it stopped. Thought there was a problem with connectivity to LDAP server but it was a coma in a user DN. omg I can’t believe it

I just spent a while fighting this one myself, the best solution I found that I've not seen suggested yet is to echo out the YAML values and pipe it into Helm, e.g.
echo -e "
hello:
   world: earth,mars,jupiter
" | helm install . --values -

Annoying for setting a single values but if you're setting lots it's probably actually clearer than many --set flags.


Telegram, Whatsapp do not fill the space, following last update

Describe the bug
Telegram and Whatsapp used to fill the space of the window.
Restarted for the update, now they don't fill and stretch across the screen.
Expected behavior
Services used to fill the entire window.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Win10

happen the same to me with Telegram service, i'm on macOS


Advancing 0.15.0-m4 release to status: admiral

https://github.com/submariner-io/subctl/pull/612 will fix the dry-run validation.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1459 ms

Resolved: Bofrost.at performance has improved in e2d694e.


chore: move mocha options to mocha.opts

Codecov Report

Merging #195 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #195   +/-   ##
=======================================
  Coverage   97.18%   97.18%
=======================================
  Files          11       11
  Lines        2237     2237
  Branches      145      145
=======================================
  Hits         2174     2174
  Misses         34       34
  Partials       29       29

Continue to review full report at Codecov.

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


No music in Deathkings of the Dark Citadel

More detailed information here.
Granted, this is not an issue with HXDD specifically, but even if I put in the patched HEXDD.WAD, there's still no music in this episode.
Is there any way you could incorporate the patch into HXDD?

Hey, I have resumed the project. HexDD is a currently supported but unfinished as far as gameplay goes and needs a preprocessor before its fully functional. Keeping open to track this in the long term.

Added via https://github.com/Lemon-King/HXDD/commit/e135c62d41ab8e8c5359d94bb355d1b31f754262 https://github.com/Lemon-King/HXDD/commit/f46f7d88e1e22c3547588d0ae82ceceaa0f61dee

Deathkings now is working in the latest commits.


[Feature][subtask] Add implementation of backend logic for Yarn queue management feature.

Search before asking

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

Description
Add implementation of backend logic for Yarn queue management feature.
Usage Scenario
No response
Related issues
#2324
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

folded it into #2416


Excluding "devDependencies" in Lambda Function's built version

Note: If your question is regarding the AWS Amplify Console service, please log it in the
official AWS Amplify Console forum

Which Category is your question related to?
amplify push command
Amplify CLI Version
4.5.0
What AWS Services are you utilizing?
AWS Lambda Function, AWS Lambda Layer
Provide additional details e.g. code snippets
I'm using AWS Lambda Layer in my projects and I'm handling the deployment part manually and only include it in the Lambda CloudFormation file under the "Layers" section and it works fine. However, because I moved my dependencies to my layer, I have also moved them to "devDependencies" in Lambda's package.json. Why? Because I have tests inside the src folder of my lambda functions and in order to run those tests I need my packages to be around.
The problem is that I noticed my lambda size is still too big and when I checked, amplify is installing my "devDependencies" and it's basically doubling the total size because of the layer I have.
Finally, my question:
** isn't amplify supposed to install npm packages with --production flag? If not, how I can set it to do so?**
I'd really appreciate any input or thoughts.
P.S. I checked and same thing is happening in Amplify Console Build and it makes sense because it's using the same command amplify push --simple

Just to give you an update, I've also taken a look into the amplify-function-plugin and tried to manipulate "amplify:build" & "amplify:push" in my own function's script to create a production build, but no success in that. I know these parts are pretty new, but some documentation would be really appreciated for these scripts and how to use them.
The only part of document that I found related to this topic was in aws-amplify website regarding the "amplify:generateReport".
Anyway, if you guys think I can contribute in any way, just let me know, I'd be happy to work on a fix for this problem.

One workaround could be to create an additional package.json in the root folder of your function (above /src) which only holds your dev dependencies and also to add your test files to the root folder, as the deployment package is built upon the /src folder

One workaround could be to create an additional package.json in the root folder of your function (above /src) which only holds your dev dependencies and also to add your test files to the root folder, as the deployment package is built upon the /src folder

Hey @foxfl,
Thank you for your comment. It seems like a good solution to me. I'll try it and post the result in this issue.

One workaround could be to create an additional package.json in the root folder of your function (above /src) which only holds your dev dependencies and also to add your test files to the root folder, as the deployment package is built upon the /src folder

Worked like a charm for me. I'm gonna close this issue. Thanks again @foxfl


DO NOT MERGE: CDK - Overview, getting started, concepts

CDK - Revert 549 revert 504 jw/cdk docs ... capturing previously missed reviewer's suggested commits.

The changes from cdk/index.md are missing from the original PR

Can we please delete this PR - I replaced it with https://github.com/0xPolygon/polygon-docs/pull/552 which addresses all of your changes and the comments in this PR here.


Clarification: can SignatureHelp.signatures be an empty array?

The interface for SignatureHelp (as of 3.15) says that signatures contains "one or more", but then activeSignature talks about signatures.length === 0.
export interface SignatureHelp {
	/**
	 * One or more signatures.
	 */
	signatures: SignatureInformation[];

	/**
	 * The active signature. If omitted or the value lies outside the
	 * range of `signatures` the value defaults to zero or is ignored if
	 * `signatures.length === 0`. [...]
	 */
	activeSignature?: number;

	[...]
}

It also says "Signature help represents the signature of something callable" which suggests that when the Position is not something callable, the correct response is null, not { signatures: [] }.
If that's the case, I think the docs for activeSignature should not mention signatures.length === 0. Further, why would activeSignature ever be outside the range of signatures? It's confusing that it talks about all these invalid states.

The reason why the spec talks about this is that clients receiving such a result has some reasonable hints what to do if it receives something that where active signature is outside the array.
I do agree if that this can be made more clear. I rephrases it a little.


Update configure-dynamic-emergency-calling.md

Support for web has been released.

According to @roykuntz , web client is still not supported--except for some ability for the user to enter an address.  Will follow up with Roy about addressing the latter, but closing this PR so that dot get merged erroneously.


getRouteName() is no longer reactive after visit an incorrect route

The issue is simple.
When you visit a route who not exists, the function FlowRouter.getRouterName() is no longer reactive.
The function works always, but the reactive capability is lost.

Hum... After verification, there is no issue, in fact...
Sorry :)


Support schema auto-generation for Composite partition keys

Referring the issue on kundera-discuss : https://groups.google.com/forum/#!topic/kundera-discuss/8aKaMX9G4gc
Add the support to auto-generate schema for composite partition keys.

Alternatively, he should be able to fire native CQL3 ddl command(via Kundera) for such schema related changes.
-Vivek

Fixed with Kundera 3.0 release.


[5.5] Allow to use a temporary storage during testing

The TemporaryStorage trait creates a temporary directory
following the naming scheme
'/tmp/laravel-ENV-TESTCLASS-TESTFUNTCION.XXXXXXXXXX'
for each test run.
If the TESTING_KEEP_TEMP_STORAGE env is set to true the
temporary storage won't be removed afterwards for inspection.

Have you tried using Storage::persistentFake instead?

To be honest I wasn't aware of Storage::persistentFake.
persistentFake would require cleaning up the files in tearDown and would not benefit from using the in memory tmpfs, which is nowadays used on Linux systems.

Ah. There is also Storage::fake :)


Add patitioning info to describe table

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

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

Please upload report for BASE (exp-multitable-bulkupsert@34bdb88). Learn more about missing BASE report.

Files with missing lines
Patch %
Lines

scheme/scheme.go
30.00%
6 Missing and 1 partial :warning:

Additional details and impacted files
@@                     Coverage Diff                      @@
##             exp-multitable-bulkupsert    #1594   +/-   ##
============================================================
  Coverage                             ?   52.36%
============================================================
  Files                                ?      356
  Lines                                ?    36143
  Branches                             ?        0
============================================================
  Hits                                 ?    18925
  Misses                               ?    16003
  Partials                             ?     1215

Flag
Coverage Δ

go-1.21.x
51.82% <30.00%> (?)

go-1.22.x
51.98% <30.00%> (?)

go-1.23.x
52.07% <30.00%> (?)

integration
52.36% <30.00%> (?)

ubuntu
52.36% <30.00%> (?)

ydb-23.3
47.56% <30.00%> (?)

ydb-24.1
52.20% <30.00%> (?)

ydb-24.2
52.07% <30.00%> (?)

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

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


Document intersection of "secret" env vars + partial parsing

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates

https://docs.getdbt.com/reference/parsing#known-limitations
https://docs.getdbt.com/reference/dbt-jinja-functions/env_var#secrets

What part(s) of the page would you like to see updated?
Env vars prefixed DBT_ENV_SECRET_ can be used in profiles.yml + packages.yml. Their values will be scrubbed from logs.
We should make clear that changes to secrets used in profiles.yml do not trigger full re-parses (unlike changes to other env vars used in the active profile), because dbt will not be storing their values (once https://github.com/dbt-labs/dbt-core/issues/4650 is fixed). This can be a limitation or a feature, depending on your point of view.
Additional information
See https://github.com/dbt-labs/dbt-core/issues/4642#issuecomment-1026016974

In discussing the fix proposed in https://github.com/dbt-labs/dbt-core/issues/9795 with @gshank, we are thinking that while this is an outcome of the current implementation of (secret) env vars + partial parsing, it's not actually the desirable behavior longer-term. The right way to avoid full parses when inputs haven't changed is to more granularly track where those inputs are being used — that is, extending what we've done for env vars to --vars and {{ target }} values.


Crash due to usage of KVO

Users of my app are encountering the following crash in production, which I'm not able to reproduce locally.
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'An instance 0x7fb1db06be30 of class MKMapView was deallocated while key value observers were still registered with it. Current observation info: <NSKeyValueObservationInfo 0x60000023d560> (
<NSKeyValueObservance 0x60000044ae30: Observer: 0x60000023d820, Key path: delegate, Options: <New: YES, Old: NO, Prior: NO> Context: 0x0, Property: 0x600000251ee0>
)'
*** First throw call stack:
(
	0   CoreFoundation                      0x000000010ebfad4b __exceptionPreprocess + 171
	1   libobjc.A.dylib                     0x000000010e65c21e objc_exception_throw + 48
	2   CoreFoundation                      0x000000010ec642b5 +[NSException raise:format:] + 197
	3   Foundation                          0x0000000109e61ba1 NSKVODeallocate + 293
	4   libobjc.A.dylib                     0x000000010e6711d1 _ZN12_GLOBAL__N_119AutoreleasePoolPage3popEPv + 715
	5   CoreFoundation                      0x000000010eb47676 _CFAutoreleasePoolPop + 22
	6   CoreFoundation                      0x000000010eb84385 __CFRunLoopRun + 2213
	7   CoreFoundation                      0x000000010eb83884 CFRunLoopRunSpecific + 420
	8   GraphicsServices                    0x0000000111ad5a6f GSEventRunModal + 161
	9   UIKit                               0x000000010bd54c68 UIApplicationMain + 159
	10  MYAPP                             0x00000001034405e2 main + 770
	11  libdyld.dylib                       0x000000010fb2e68d start + 1
	12  ???                                 0x0000000000000003 0x0 + 3
)

It surely is related to some magic done in CCHMapViewDelegateProxy when adding and remove the observer for the "delegate' keyPath.

Mmmh, without a way to reproduce this issue, it's going to be difficult to fix. Maybe you are using an unusual way of initializing/destroying the map view?

You're indeed right, I tried everything but couldn't reproduce. It happens on all devices/OS. This is how I allocate it:
self.mapClusterController = [[CCHMapClusterController alloc] initWithMapView:self.mapView];
self.mapClusterController.delegate = self;
self.mapClusterController.cellSize = 60.0f;
self.mapClusterController.minUniqueLocationsForClustering = 5;
self.mapClusterController.maxZoomLevelForClustering = 15.5f;

I appreciate it'll be hard to figure this out, thought it was worth to open the issue though.

Hi,
same issue described by Alberto appears
"An instance 0x17285e00 of class MKMapView was deallocated while key value observers were still registered with it. Current observation info: NSKeyValueObservationInfo 0x16f1db30 (NSKeyValueObservance 0xb14a5b0: Observer: 0x16f2ff60, Key path: delegate, Options: New: YES, Old: NO, Prior: NO Context: 0x0, Property: 0xb14a5d0)"
Also from my side, only users are encountering it.
From description seems that "mapView" property of CCHMapViewDelegateProxy is deallocated while an instance of CCHMapViewDelegateProxy is still observing it.
It will be great if you can have a look.

Same here. I was never able to reproduce it myself though.


support queries/filters

Implemented filterLocal

filterLocal was removed in favor of the notifier where extension, this is used to filter models that pass through the notifier. For example:
  @override
  DataStateNotifier<List<Inspection>?> watchAllNotifier({
    bool? remote,
    Map<String, dynamic>? params,
    Map<String, String>? headers,
    bool? syncLocal,
    String? finder,
    DataRequestLabel? label,
  }) {
    return super
        .watchAllNotifier(
          remote: remote,
          params: params,
          headers: headers,
          syncLocal: syncLocal,
          finder: finder,
          label: label,
        )
        .where((i) => i.isVisible);
  }

Since at the moment FD uses a regular Hive box for persistence (which keeps all values in memory), use findAll and filter what you need in Dart.

Thanks @frank06 ,
I'll continue using the findAll now.

As long as a Hive regular box is local storage no, because it makes no sense. If we incorporate Isar then sure, its query mechanism will be exposed for clients to use

Ok, I understand. Thanks @frank06


Fix localization issue

Now the labels at the top are localized depending on the preferred language in the settings. Previously, the system language had no effect on this.

Hi, I've tested this on device with and without your fix, couldn't find the difference e.g. when switching to German language. Could you please explain the issue more in detail?


Enable utf-8 mode for python

Fix #92

Hi, could you help test if the issue is fixed with https://github.com/laike9m/Python-Type-Challenges/commit/84551ad9c1d668a9aecfa73ebac25faa81a9cb3c ?

Hi, could you help test if the issue is fixed with 84551ad ?

Everything well


Label is not visible when using Select or Input components

Steps to reproduce
Link to live example: https://mui-label-problem.vercel.app/
Steps:

click input or select

Current behavior
The label is not visible when it should be displayed above or inside the Select or Input components.
Expected behavior
The label should be visible and properly positioned above or inside the Select or Input components, providing clear context and usability for users.
Context
Repository: https://github.com/takecchi/mui-label-problem
ScreenShot:

Your environment

npx @mui/envinfo
  System:
    OS: macOS 14.4.1
  Binaries:
    Node: 18.20.4 - ~/.volta/tools/image/node/18.20.4/bin/node
    npm: 10.7.0 - ~/.volta/tools/image/npm/10.7.0/bin/npm
    pnpm: Not Found
  Browsers:
    Chrome: 126.0.6478.183
    Edge: Not Found
    Safari: 17.4.1
  npmPackages:
    @emotion/react: 11.13.0 => 11.13.0
    @emotion/styled: 11.13.0 => 11.13.0
    @mui/core-downloads-tracker:  5.16.4
    @mui/material: 5.16.4 => 5.16.4
    @mui/material-nextjs: 5.16.4 => 5.16.4
    @mui/private-theming:  5.16.4
    @mui/styled-engine:  5.16.4
    @mui/system:  5.16.4
    @mui/types:  7.2.15
    @mui/utils:  5.16.4
    @types/react: 18.3.3 => 18.3.3
    react: 18.3.1 => 18.3.1
    react-dom: 18.3.1 => 18.3.1
    typescript: 5.5.4 => 5.5.4

Browser: Chrome

Search keywords: label

if you read the docs when you're using inputs that arent TextField input you need to use a seperate component called InputLabel along side wrapping it all in FormControl like this
<FormControl>           <InputLabel>TEST INPUT</InputLabel>             <OutlinedInput label={'TEST INPUT'}/>    </FormControl>

@Brondont

if you read the docs when you're using inputs that arent TextField input you need to use a seperate component called InputLabel along side wrapping it all in FormControl like this

You're right...! My apologies for the mistake.
I should have wrapped it in FormControl. The documentation indeed shows it that way.
Thank you!


fix(ci): correct the main branch url

:tada: This PR is included in version 1.0.0-rc.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[GOCA] make address range representation more generic

/!\ To report a security issue please follow this procedure:
[https://github.com/OpenNebula/one/wiki/Vulnerability-Management-Process]
Description
I closed #6019 for this more generic one
There is several issues that require to update the address ranges structures in the terraform provider  (https://github.com/OpenNebula/terraform-provider-opennebula/issues/192 and https://github.com/OpenNebula/terraform-provider-opennebula/issues/376).
I realized the XML parsing structure lack of flexibility, we could add custom attributes in an address range.
To remains simple and be more consistent with the way we manage other "sub"-resources (for instance disk and NIC for VMs), I propose to remove the address range structure and use something fully dynamic instead.
It breaks for those that would already use the address range structure.
Expected behavior
Be able to parse all attributes of an adress range
Details

Affected Component: [e.g. Sunstone, Scheduler or Storage]
Hypervisor: [e.g. KVM]
Version: [e.g. 5.4.12 or development]

Additional context
Add any other context about the problem here.

Progress Status

[ ] Code committed
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

I introduced regression here, we loosed the capability to retrieve the Lease section of the address range when unmarshalling the vector.
I left a comment in the related PR and opened a new PR #6104


BatchNorm training instability fix

This is in reference to issue 659.
I modified BatchNorm to have two approaches "batch" and "ema".  "batch" just uses the batch statistics during training time.  If approach is not specified it defaults to "batch" with a warning.  It's robust and seems to be the standard choice - it's far less likely to kill a model just by adding it.
"ema" is based of the smooth start method in the above issue.  So keep a running mean and variance but instead of renormalizing Adam style the parts of the running averages that are zeroed are filled with the batch statistics.   The problem is it's still not robust - the momentum parameter is simultaneously specifying a warmup period (when we're expecting the input distribution to change significantly) and how long we want the running average to be.  So I added a linear warmup period.
Now for any choice of momentum there seems to be a warmup_period choice that will give good results.  And validation performance was at least as good as with batch mode for my tests.  However, I don't see a good default for warmup_period.
Some considerations:

having approach="batch" and the common axis_name="batch" is a little awkward
There's an example using BatchNorm - that will start raising a warning and should probably get changed
The current BatchNorm behavior can't be exactly replicated (ema / momentum=0.99 / warmup_period=1) is close but different at the start
There's one more piece of state hence the test_stateful.py change.  Though this could be conditionally removed for approach="batch" if desired

Let me know what you think or if any changes or tests need to be added

All good - I got caught up in other things myself!
From my tests the replication is exact now.  It added another approach that is very similar to "ema" but it seemed like the most reasonable way to organize it.  Let me know if anything isn't clear


PlanBuilder seems to be missing methods for importing a plan.

We have the following in the NodeJS API; however Java API seems to be missing the equivalent method to import an exported plan.
Export the plan
const exportedPlan =
        plan1.joinInner(plan2, [op.on(masterIdCol1, masterIdCol2), op.on(idCol1, idCol2)])
        .orderBy(op.desc(detailNameCol))
        .offsetLimit(1, 100)
        .export();

Import the plan
const importedPlan = op.import(exportedPlan);
const output = importedPlan.result();

In Java API we can export:
ModifyPlan plan1 = p.fromView("opticFunctionalTest", "detail", "myDetail");
		ModifyPlan plan2 = p.fromView("opticFunctionalTest", "master", "myMaster");

		PlanColumn masterIdCol1 = p.viewCol("detail", "masterId");
		PlanColumn masterIdCol2 = p.viewCol("master", "id");
		PlanColumn idCol1 = p.viewCol("master", "id");
		PlanColumn idCol2 = p.viewCol("detail", "id");
		PlanColumn detailNameCol = p.viewCol("detail", "name");
		PlanColumn masterNameCol = p.viewCol("master", "name");

		ExportablePlan exportedPlan = plan1.joinInner(plan2, p.on(masterIdCol1, masterIdCol2), p.on(idCol1, idCol2))
		                                   .orderBy(p.desc(detailNameCol))
		                                   .offsetLimit(1, 100);
		//Export the Plan to a handle.
		exportedPlan.export(exportHandle);

But PlanBuilder interfaces do not have an import method. We have the following method signature in the functional doc.
import(exportedPlan, importOptions) => plan
import({plan:..., options:...}) => plan	Instantiates a plan from the JSON or JavaScript literal object representation of an exported plan.
The only import option at present is a privilegeLevel of restricted, which the REST API sets to require sanitizing of XPaths sent as part of a REST client plan.

Note to QA: Complete testExportAndImportPlan method of TestOpticOnViews class.

We agreed not to support import in MarkLogic 9 during the specification review.  In SJS, we have to support import so we can execute the plan sent from the client.  In Java, however, there's no use case at present for importing an exported plan -- the plan can just be sent to the server for execution.


Incorrect encoding of "i

In biber 2.0
"{\i}
is rendered as
ı̈  (\u8:̈)
instead of
ï

Yes, by default this is true to have consistent accent handling. Is this a problem? The real way to fix this is just use UTF-8 in your .bib and use LuaLatex or XeLatex.

It is a problem introduced in biber 2.0. It worked fine in biber 1.9.
The got the bibtex record from DBLP, and I might share the bibtex file with other projects not using biblatex.
All the best
Guido

On 4 May 2015, at 8:43 am, plk<EMAIL_ADDRESS>wrote:
Yes, by default this is true to have consistent accent handling. Is this a problem? The real way to fix this is just use UTF-8 in your .bib and use LuaLatex or XeLatex.
—
Reply to this email directly or view it on GitHub https://github.com/plk/biber/issues/65#issuecomment-98552684.

I mean, is it really a problem to have it encoded as it is in 2.0? I changed this so that this case is no longer a special case but is a composed char consisting of a diaresis and a dotless i.

The issue just came up again in the texlive list and in golatex. It is a problem for pdflatex if glyphs with accent are encoded as "glyph + (following) combining accent". pdflatex/inputenc can't handle this as the accent can't look back to find the base glyph.

run biber with
biber --output-safechars

I just remembered this rather old discussion about a similar problem: https://sourceforge.net/p/biblatex-biber/feature-requests/23/
Combining accents are a problem for all engines (for pdflatex always, for xelatex/lualatex if the font doesn't has the accent) -- there are regularly questions when copy&paste on the mac has created umlauts with such accents -- and so should imho be avoided if possible.

That is the reason why Biber should be run  with --output-safechars in such cases. Then it doesn't change the TeX commands for accented chars.

Another instance popped up: Upgrading from Biber 1.9 to Biber 2.1 input encoding error


Add passivity module, is_passive function, and passivity_test. Introduces dependencies on lmi_sdp and cvxopt.

I made a discussion topic a couple of weeks ago about making a passivity module, @murrayrm suggested making a draft PR. Here it is!

I need to cut the formation of the LMI via symbolic variables for performance. Did a timing test on a size 20 state, input, output system, 99% of the time is spent constructing the LMI and converting to cvxopt form.

Would it be possible to implement the matrix handling and solving with NumPy and SciPy functions only?

I'm about 60% sure cvxopt is necessary to solve the linear matrix inequality. Sympy and lmi_sdp can definitely be dropped.
Cvxopt only requires BLAS/LAPACK, which it seems that slycot also requires. So this passivity package could be setup to be optional?

Coverage increased (+0.02%) to 94.448% when pulling 649b21ee7e915fc8758413da04f4ec411ba58d61 on Mark-Yeatman:passivity-tools into 3b5f1990bf47180714423c420320f013bdd4dfaa on python-control:main.

@bnavigator I believe I've resolved all the outstanding review comments?

Instead of writing things in the standard SDP notation, you might want to use CVXPY notation (which acts like YALMIP) so you don't have to use the tiresome basis matrices.
Also note that strictly proper systems cannot be checked via positive real lemma via strict inequalities due zero D + D.T block forcing the optimization problem to infeasibilty. You either have to use extended version or nudge D block with epsilon*np.eye(m).

I would definitely be up for using cvxpy, but it looks like it has more dependencies than just using cvxopt (https://www.cvxpy.org/install/#install-from-source vs https://cvxopt.org/install/#building-and-installing-from-source)? That seems like an issue for a project maintainer.
You're right about the D = 0 case, I will nudge it as you suggested.

CVXPY: I don't think it is necessary to employ a full domain specific language with all its dependencies for one hardcoded optimization problem.

I took the liberty of fixing the missing skips. I hope you don't mind.


support for llama3.2 vision

First of all thanks for the amazing work.  It helps us build a very simple yet efficient router within our Java applications.
I was wondering if there is any plan to support LLama3.2 vision models.
--Thanks and Regards
Vaijanath

If you can provide with tensor library, I can take a stab at it. Right now in order to make llama3.2 vision to work with current code i need to make weights to have List<FloatTensor[]> and have identity operations for missing layers.
for example attn_q.weight is not available for all the layers.


Clean up use of atomics and barriers.

Use __atomic rather than __sync.
Remove atomic.S from libec.

Not particularly necessary at this point.


Compile error with OTP 27.0.1

I'm getting the following error when trying to compile a project with otp 27.0.1. I'm not sure if the error is in the erlydtl app or this plugin.
Line 75 in erlydtl_beam_compiler.erl is "-include("erlydtl_ext.hrl")."
default/plugins/erlydtl/include/erlydtl_ext.hrl exists so I'm not sure what causes the error.
No error with otp <IP_ADDRESS> and same rebar3 version.
rebar3 --version
rebar 3.23.0 on Erlang/OTP 27 Erts 15.0.1

===> Fetching rebar3_erlydtl_plugin (from {git,"https://github.com/tsloughter/rebar3_erlydtl_plugin.git",
                                 {branch,"master"}})
===> Fetching erlydtl (from {git,"https://github.com/erlydtl/erlydtl.git",{branch,"master"}})
===> Analyzing applications...
===> Compiling erlydtl
===> Compiling _build/default/plugins/erlydtl/src/erlydtl_beam_compiler.erl failed
_build/default/plugins/erlydtl/src/erlydtl_beam_compiler.erl:75:10: can't find include file "erlydtl_ext.hrl"
_build/default/plugins/erlydtl/src/erlydtl_beam_compiler.erl:152:73: undefined macro 'ERR/2'
_build/default/plugins/erlydtl/src/erlydtl_beam_compiler.erl:190:77: undefined macro 'ERR/2'
_build/default/plugins/erlydtl/src/erlydtl_beam_compiler.erl:221:33: undefined macro 'ERR/2'
_build/default/plugins/erlydtl/src/erlydtl_beam_compiler.erl:247:14: undefined macro 'ERR/2'
_build/default/plugins/erlydtl/src/erlydtl_beam_compiler.erl:256:14: undefined macro 'WARN/2'
...many more errors...

What version of erlydtl is that using? I get no error building on OTP-27.

I'll try to create a simple app to reproduce.
This is the relevant part in my rebar.config:
{plugins, [
    {rebar3_erlydtl_plugin, ".*",
     {git, "https://github.com/tsloughter/rebar3_erlydtl_plugin.git", {branch, "master"}}}
]}.

{erlydtl_opts, [{doc_root, "templates"}]}.

Does ./_build/default/plugins/erlydtl/include/erlydtl_ext.hrl exist?

Ok, so this is wild. I have now reproduced... What changed is instead of how i was testing, which was:
{plugins, [
    {rebar3_erlydtl_plugin, ".*",
     {git, "https://github.com/tsloughter/rebar3_erlydtl_plugin.git", {branch, "master"}}}
]}.

I instead used exactly your config of:
{plugins, [
    rebar3_appup_plugin,
    {rebar3_erlydtl_plugin, ".*",
     {git, "https://github.com/tsloughter/rebar3_erlydtl_plugin.git", {branch, "master"}}}
]}.

Makes no sense.

Thank you for reproducing the error. I'll try to update bbmustache dependency to version 1.12.2 in rebar3_appup_plugin to see if it somehow fixes the errors.

After upgrading bbmustache to 1.12.2 in rebar3_appup_plugin I get no error.
https://github.com/lrascao/rebar3_appup_plugin/pull/72
You can test it here:
{plugins, [
    {rebar3_appup_plugin, ".*",
     {git, "https://github.com/benbro/rebar3_appup_plugin.git", {branch, "bbmustache"}}},
    {rebar3_erlydtl_plugin, ".*",
     {git, "https://github.com/tsloughter/rebar3_erlydtl_plugin.git", {branch, "master"}}}
]}.

Thanks

Interesting.


feat: split reference.json on sub schemas.

Overview

Description: #79
Schema update type: modification
Services affected: none

References

https://json-schema.org/understanding-json-schema/structuring.html

@KillDozerX2, look this. We need to move on this direction. The file has become too large and is difficult to maintain. Do some tests and check if everything is ok. If you have any idea of change, let me know.

Will do @lalcebo I definitely agree with this approach.

@KillDozerX2 It might be a good idea, an action that generates the final schema.

Cool, I'll come up with a build system soon

I don't think we need a build step, I just tested this in both vscode and intelliJ and it worked. I don't know of any other schema validators, but I say we merge this. It seems to be working as if it was one schema.
I say we merge it and start splitting and we'll handle a build step when someone comes asking for it.

Also, @lalcebo can we do Rebase and merge instead of merge commit? Gives out a cleaner git history which is easy to track.

I create a vscode snippet to quickly create sub schema

I don't think we need a build step, I just tested this in both vscode and intelliJ and it worked. I don't know of any other schema validators, but I say we merge this. It seems to be working as if it was one schema.
I say we merge it and start splitting and we'll handle a build step when someone comes asking for it.

ok

Also, @lalcebo can we do Rebase and merge instead of merge commit? Gives out a cleaner git history which is easy to track.

I agree. It is better.

@lalcebo I think is good to go


add directory-specific config files for solhint

this is me shamelessly promoting my work on solhint-community (?)
disabled rules like no-console for most files, but enabled them again in test and script directories where their usage is expected.
This PR sets the version to the release candidate for the major version 4.0.0, the spec for which is still up for debate . But I already got one worthwhile feature working and I want to start getting feedback on it.

Thanks @juanpcapurro.
The main value I used to get out of solhint-community was the no-unused-imports rule, but that is now part of the standard solhint.
Could you explain what the latest benefits of using solhint-community are?

Could you explain what the latest benefits of using solhint-community are?

right now, the main feature is that you can add config files in subdirectories with overrides, as is shown in this PR.
In the near future I have also defined a roadmap (well, just issues taged with v4.0.0) for what changes will be included in the next major release, such as:

a better solhint init-config that can also initialize the integration with prettier and overrides for foundry projects
merging no-unused-import and no-unused-var
re-defining the recommended ruleset
recognize shadowing in no-unused-import
a better UX for running solhint without a config (similar to forge fmt)

But also I want to keep some things implemented in protofire/solhint out, such as:

interactive fixing
fixers that cause semantic changes (eg: renaming event names or removing console.logs)
an extra flag to output the report to a file
manually checking for updates

On top of that, I do not agree with how they're handling versioning, doing things like adding rules on a patch version, adding/removing rules to the solhint:recommended ruleset on a minor version, or publishing a major version without any breaking changes.
In solhint-community OTOH, I make my best effort to follow semantic versioning.
However, I know that most worthwhile changes I implement will be backported into
protofire/solhint 🙃 and I understand if you don't want to switch or prefer to
consider doing so later down the line when more features are implemented
thanks for considering this in either case.

Thanks for the additional color, @juanpcapurro.
As an open-sourcerer myself, I appreciate your open-source development efforts a lot. But, unfortunately, I do not find the maintenance cost of switching to solhint-community worth it when, as you say, most changes will eventually be back-ported to solhint.
The per-directory configuration is an awesome feature, though! I will take it for a spin in separate projects that are less mature than my Foundry template.


need cp312 whl!

need a flash_attn-2.5.8+cu122torch2.3cxx11abiFALSE-cp312-cp312-linux_x86_64.whl , ty :)

Cool, can you make a PR?

Nvm my GPU doesn't support this.


how can we connect the spans in  Multithreading ?

Problem ：
I want to connect the spans in Multithreading, but when I build the child span with tracer, the span of client span-kind has two ,not a one . I can't figure it out what't going on.
I user the dependency 👍 :
 <dependency>             <groupId>io.opentracing.contrib</groupId>             <artifactId>opentracing-spring-jaeger-web-starter</artifactId>             <version>0.1.4</version>         </dependency>
the step in method is :
Result result = restTemplate.getForObject(url, Result.class);
I just do that .

It is completely unclear to me what you're asking. Please revisit and restate your question.

This is probably not related to Jaeger, nor Jaeger Java Client, but to the OpenTracing Contrib's Spring Starter for Jaeger. Please, open the issue there, but in general, the ScopeManager will not work for concurrent transactions in the same thread (like in reactive applications). For such cases, it's recommended to explicitly pass the context (Scope) as part of your application code.

I'm closing as I don't think this question belongs to this project, but feel free to reopen if you think otherwise.

This is probably not related to Jaeger, nor Jaeger Java Client, but to the OpenTracing Contrib's Spring Starter for Jaeger. Please, open the issue there, but in general, the ScopeManager will not work for concurrent transactions in the same thread (like in reactive applications). For such cases, it's recommended to explicitly pass the context (Scope) as part of your application code.

OK， I think you were right .


Are there plans for a Bioconda release?

Yes, it is definitely planned, but I haven't found the time yet to learn how to create a conda package.

Sorry that it took so long. The devel branch has now a folder conda with the receipt. It seems to work, but if you are experienced with creating conda packages you could check if I need to fix or improve it.
I am currently testing the devel on multiple datasets to check if I accidentally added a new bug and probably around end of the month I plan to merge devel into master as version 1.1. This version I also plan to upload to bioconda.

Again much later then expected, but it is available through bioconda by now.

That's awesome! Great stuff!


[oreilly] KeyError('next') when trying to login to O'Reilly

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2022.04.08 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
Im getting an error while visiting some oreillys courses .  "ERROR: 10000MNHV202259: An extractor error has occurred. (caused by KeyError('next'));" Does anyone know about this?
Verbose log
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2022.04.08 [7884ade65]
[debug] Python version 3.10.2 (CPython 64bit) - Windows-10-10.0.19044-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 2022-01-19-git-dd17c86aa1-full_build-www.gyan.dev (setts), ffprobe 2022-01-19-git-dd17c86aa1-full_build-www.gyan.dev
[debug] Optional libraries: brotli, certifi, Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2022.04.08, Current version: 2022.04.08
yt-dlp is up to date (2022.04.08)
[safari:course] Downloading login page
ERROR: 10000MNHV202259: An extractor error has occurred. (caused by KeyError('next')); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "C:\Users\d\AppData\Local\Programs\Python\Python310\lib\site-packages\yt_dlp\extractor\common.py", line 639, in extract
    self.initialize()
  File "C:\Users\d\AppData\Local\Programs\Python\Python310\lib\site-packages\yt_dlp\extractor\common.py", line 544, in initialize
    self._perform_login(username, password)
  File "C:\Users\d\AppData\Local\Programs\Python\Python310\lib\site-packages\yt_dlp\extractor\safari.py", line 44, in _perform_login
    'https://api.oreilly.com', qs['next'][0])
KeyError: 'next'

Extractor SafariBaseIE expects https://learning.oreilly.com/accounts/login-check/ to be redirected to an url having a query with parameter named next in it, i.e. something like .*learning.oreilly.com/home/.*?.*next=.*.

Ok , is there a way I can fix that?

Editing issue title might help someone who can fix it to find this.

Whereas the actual redirect might be https://www.oreilly.com/member/login/ perhaps with a parameter representing the page from which login was redirected. But as OP failed to provide the problem URL, it's not easy to know.

For login, url is irrelevant. Missing account stopped me fixing it.

@sulyi, OP has indicated they are willing to share account details. If you are interested in trying to fix this, just let him know how to privately share it with you (Or I can act as the middle-man if needed)

@Lecharge I've added my gpg key to github account.

Hi you can use
-u<EMAIL_ADDRESS>-p pappagris its a temp mail not connected to me.

Redirect works again, new api of the webpage seems to be settling,
Extractor should be upgraded to fully use that.

THE ISSUE ISNT SOLVED YET

https://github.com/yt-dlp/yt-dlp/pull/3536#issuecomment-1127569597
@g-arjuna, log?

Redirect works again, new api of the webpage seems to be settling, Extractor should be upgraded to fully use that.

THE ISSUE ISNT SOLVED YET

#3536 (comment)
@g-arjuna, log?

[debug] Command-line config: ['https://learning.oreilly.com/videos/apache-kafka-series/9781789342604/', '--username', 'PRIVATE', '--password', 'PRIVATE', '--verbose']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2022.04.08 [7884ade65]
[debug] Lazy loading extractors is disabled
[debug] Python version 3.10.2 (CPython 64bit) - Windows-10-10.0.22000-SP0
[debug] Checking exe version: ffprobe -bsfs
[debug] Checking exe version: avprobe -bsfs
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: avconv -bsfs
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.14.1, brotli-1.0.9, certifi-2021.10.08, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[safari:course] Downloading login page
[safari:course] Logging in
[safari:course] Completing login
[debug] [safari:course] Extracting URL: https://learning.oreilly.com/videos/apache-kafka-series/9781789342604/
[safari:course]<PHONE_NUMBER>604: Downloading course JSON
[download] Downloading playlist: Apache Kafka Series - Learn Apache Kafka for Beginners
[safari:course] playlist Apache Kafka Series - Learn Apache Kafka for Beginners: Collected 102 videos; downloading 102 of them
[download] Downloading video 1 of 102
[safari:api] Downloading login page
ERROR: An extractor error has occurred. (caused by KeyError('next')); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
File "C:\Users\arjun\OneDrive\Desktop\New folder\yt-dlp-test\lib\site-packages\yt_dlp-2022.4.8-py3.10.egg\yt_dlp\extractor\common.py", line 633, in extract
self.initialize()
File "C:\Users\arjun\OneDrive\Desktop\New folder\yt-dlp-test\lib\site-packages\yt_dlp-2022.4.8-py3.10.egg\yt_dlp\extractor\common.py", line 538, in initialize
self._perform_login(username, password)
File "C:\Users\arjun\OneDrive\Desktop\New folder\yt-dlp-test\lib\site-packages\yt_dlp-2022.4.8-py3.10.egg\yt_dlp\extractor\safari.py", line 41, in _perform_login
'https://api.oreilly.com', qs['next'][0])
KeyError: 'next'

I tried to use the fork sulyi:oreilly-login-fix

Try again, that's not a log from 10a2af1 for sure.
It still tries to login using https://api.oreilly.com instead of https://www.oreilly.com/member/auth/login/.
Currently I can't verify if mentioned branch works with specific url since the trial account Lecharge kindly setup for me have already expired, but I was still able to download first minute of the videos.
Please ask help on discord if you need to troubleshoot how to apply patch.

Hey Sorry for the trouble ! I changed the version using python -m pip install -U git+https://github.com/sulyi/yt-dlp.git@oreilly-login-fix
I can confirm it is working fine. Thanks for the help

I can confirm all the links found by extractor are correct and the login works fine but the connection fails somehow.
[debug] Command-line config: ['--config-location', 'D:\\Utils/YoutubeDL/Configs/ytdlp_oreilly.conf', '--playlist-start', '1'] [debug] | Config "D:\Utils/YoutubeDL/Configs/ytdlp_oreilly.conf": ['--cookies-from-browser', 'firefox', '--download-archive', 'archive.txt', '-i', '-c', '-v', '--console-title', '--batch-file=batch-file.txt', '--write-annotations', '--write-description', '--write-info-json', '--write-thumbnail', '--sub-lang', 'en', '--write-auto-sub', '--write-sub', '--add-metadata', '--embed-subs', '--embed-thumbnail', '-o', '_Downloads/%(playlist_title)s/%(playlist_index)s. %(title)s.%(ext)s', '-f', 'bestvideo[height<=720]+bestaudio/best[height<=720]/worst', '--merge-output-format', 'mp4', '--geo-bypass'] [debug] Batch file urls: ['https://learning.oreilly.com/videos/gradle-fundamentals/9781491937266/', 'https://learning.oreilly.com/videos/reactive-spring-boot/9780137831463/'] [debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8 [debug] yt-dlp version 2022.05.18 [b14d52355] [debug] Lazy loading extractors is disabled [debug] Plugins: ['SamplePluginIE', 'SamplePluginPP'] [debug] Python version 3.9.1 (CPython 64bit) - Windows-10-10.0.22000-SP0 [debug] Checking exe version: ffmpeg -bsfs [debug] Checking exe version: ffprobe -bsfs [debug] exe versions: ffmpeg n4.4-4-gacb339bb88 (setts), ffprobe n4.4-4-gacb339bb88 [debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2020.12.05, mutagen-1.45.1, sqlite3-2.6.0 [Cookies] Extracting cookies from firefox [debug] Extracting cookies from: "C:\Users\mh-sh\AppData\Roaming\Mozilla\Firefox\Profiles\8tceuger.default-release\cookies.sqlite" [Cookies] Extracted 2122 cookies from firefox [debug] Proxy map: {} [debug] Loading archive file 'archive.txt' [safari:course] Checking if logged in [debug] [safari:course] Extracting URL: https://learning.oreilly.com/videos/gradle-fundamentals/9781491937266/ [safari:course]<PHONE_NUMBER>266: Downloading course JSON [download] Downloading playlist: Gradle Fundamentals [info] Writing playlist metadata as JSON to: _Downloads\Gradle Fundamentals\0. Gradle Fundamentals.info.json WARNING: There's no playlist description to write [safari:course] playlist Gradle Fundamentals: Collected 25 videos; downloading 25 of them [download] Downloading video 1 of 25 [safari:api] Checking if logged in [debug] [safari:api] Extracting URL: https://learning.oreilly.com/api/v1/book/9781491937266/chapter/video224126.html [safari:api]<PHONE_NUMBER>266/video224126: Downloading part JSON [safari] Checking if logged in [debug] [safari] Extracting URL: https://learning.oreilly.com/videos/gradle-fundamentals/9781491937266/9781491937266-video224126 [safari]<PHONE_NUMBER>266-video224126: Downloading kaltura session JSON [debug] [Kaltura] Extracting URL: https://cdnapisec.kaltura.com/html5/html5lib/v2.37.1/mwEmbedFrame.php?wid=_1926081&uiconf_id=29375172&entry_id=&flashvars%5BreferenceId%5D=9781491937266-video224126&flashvars%5Bks%5D=djJ8MTkyNjA4MXzZHTYFLYKMxC4GcBvQs03Dg4Er_yI6lNg20Q3fMj64p_PSYBeMm_8HitjupoKd7gsXIkQsewtCqrDVQRfqaPJGhjLg15eWmm4liRXe8ABltg%3D%3D [Kaltura]<PHONE_NUMBER>266-video224126: Downloading webpage [Kaltura] 0_2ksee5zw: Downloading video info JSON [Kaltura] 0_2ksee5zw: Checking mp4-4023 URL [Kaltura] 0_2ksee5zw: mp4-4023 URL is invalid, skipping: [WinError 10054] An existing connection was forcibly closed by the remote host [Kaltura] 0_2ksee5zw: Downloading m3u8 information WARNING: [Kaltura] Failed to download m3u8 information: [WinError 10054] An existing connection was forcibly closed by the remote host [debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id [debug] Downloading subtitles: en [info] 0_2ksee5zw: Downloading 1 format(s): mp4-1797 WARNING: There's no video description to write [info] Writing video subtitles to: _Downloads\Gradle Fundamentals\01. Installation and Configuration.en.srt [debug] Invoking http downloader on "http://cdnapi.kaltura.com/api_v3/service/caption_captionasset/action/serve/captionAssetId/1_xxlsz35n" [download] Got server HTTP error: [WinError 10054] An existing connection was forcibly closed by the remote host. Retrying (attempt 1 of 10) ... [download] Got server HTTP error: [WinError 10054] An existing connection was forcibly closed by the remote host. Retrying (attempt 2 of 10) ... [download] Got server HTTP error: [WinError 10054] An existing connection was forcibly closed by the remote host. Retrying (attempt 3 of 10) ... [download] Got server HTTP error: [WinError 10054] An existing connection was forcibly closed by the remote host. Retrying (attempt 4 of 10) ... [download] Got server HTTP error: [WinError 10054] An existing connection was forcibly closed by the remote host. Retrying (attempt 5 of 10) ... [download] Got server HTTP error: [WinError 10054] An existing connection was forcibly closed by the remote host. Retrying (attempt 6 of 10) ... [download] Got server HTTP error: [WinError 10054] An existing connection was forcibly closed by the remote host. Retrying (attempt 7 of 10) ... [download] Got server HTTP error: [WinError 10054] An existing connection was forcibly closed by the remote host. Retrying (attempt 8 of 10) ...

@g-arjuna  is it still working fine in your side ?
i am getting error same which you had face
can you please explain me how can i install this (https://github.com/sulyi/yt-dlp.git@oreilly-login-fix)
i am getting error while installing with command

Hi! I just stumbled over this behavior in 2024.04.09 and see that drop-in-replacing that version's yt_dlp/extractor/safari.py with github.com/sulyi/yt-dlp/blob/10a2af1d54895d1c73c72c12d74af1155ca66a78/yt_dlp/extractor/safari.py allows safari downloads to complete successfully given valid O'Reilly Online Learning credentials. Is there a way to have the diff reviewed by a knowledgable person, merged and released?
--- /usr/lib/python3.12/site-packages/yt_dlp/extractor/safari.py.orig   2024-04-10 00:10:43.000000000 +0200
+++ /usr/lib/python3.12/site-packages/yt_dlp/extractor/safari.py        2024-05-24 17:28:21.160058556 +0200
@@ -3,10 +3,6 @@

 from .common import InfoExtractor

-from ..compat import (
-    compat_parse_qs,
-    compat_urlparse,
-)
 from ..utils import (
     ExtractorError,
     update_url_query,
@@ -22,54 +18,48 @@

     LOGGED_IN = False

-    def _perform_login(self, username, password):
-        _, urlh = self._download_webpage_handle(
-            'https://learning.oreilly.com/accounts/login-check/', None,
-            'Downloading login page')
+    def is_logged_in(self, fatal=True):
+        auth, urlh = self._download_json_handle(
+            'https://api.oreilly.com/api/v2/me/', None, note='Checking if logged in', fatal=False,
+            headers={'Accept': 'application/json'}, expected_status=401)
+        if urlh.status == 401:
+            msg = 'Unable to login: %s' % auth['detail']
+            if fatal:
+                raise ExtractorError(msg)
+            self.write_debug(msg)
+            self.LOGGED_IN = False
+            return self.LOGGED_IN
+        self.LOGGED_IN = True
+        return self.LOGGED_IN

-        def is_logged(urlh):
-            return 'learning.oreilly.com/home/' in urlh.url
+    def _initialize_pre_login(self):
+        self.is_logged_in(fatal=False)

-        if is_logged(urlh):
-            self.LOGGED_IN = True
-            return
+    def _perform_login(self, username, password):

-        redirect_url = urlh.url
-        parsed_url = compat_urlparse.urlparse(redirect_url)
-        qs = compat_parse_qs(parsed_url.query)
-        next_uri = compat_urlparse.urljoin(
-            'https://api.oreilly.com', qs['next'][0])
+        if self.LOGGED_IN:
+            return

         auth, urlh = self._download_json_handle(
             'https://www.oreilly.com/member/auth/login/', None, 'Logging in',
             data=json.dumps({
                 'email': username,
-                'password': password,
-                'redirect_uri': next_uri,
+                'password': password
             }).encode(), headers={
-                'Content-Type': 'application/json',
-                'Referer': redirect_url,
+                'Content-Type': 'application/json'
             }, expected_status=400)

         credentials = auth.get('credentials')
         if (not auth.get('logged_in') and not auth.get('redirect_uri')
                 and credentials):
-            raise ExtractorError(
-                'Unable to login: %s' % credentials, expected=True)
+            raise ExtractorError('Unable to login: %s' % credentials, expected=True)

         # oreilly serves two same instances of the following cookies
         # in Set-Cookie header and expects first one to be actually set
         for cookie in ('groot_sessionid', 'orm-jwt', 'orm-rt'):
             self._apply_first_set_cookie_header(urlh, cookie)

-        _, urlh = self._download_webpage_handle(
-            auth.get('redirect_uri') or next_uri, None, 'Completing login',)
-
-        if is_logged(urlh):
-            self.LOGGED_IN = True
-            return
-
-        raise ExtractorError('Unable to log in')
+        self.is_logged_in()

 class SafariIE(SafariBaseIE):
@@ -86,7 +76,7 @@

     _TESTS = [{
         'url': 'https://www.safaribooksonline.com/library/view/hadoop-fundamentals-livelessons/9780133392838/part00.html',
-        'md5': 'dcc5a425e79f2564148652616af1f2a3',
+        'md5': '592d0e6d0b03d9b981b3f7306ffc5ca9',
         'info_dict': {
             'id': '0_qbqx90ic',
             'ext': 'mp4',
@@ -94,6 +84,9 @@
             'timestamp':<PHONE_NUMBER>,
             'upload_date': '20150724',
             'uploader_id': 'stork',
+            'duration': 149,
+            'view_count': int,
+            'thumbnail': 'http://cfvod.kaltura.com/p/1926081/sp/192608100/thumbnail/entry_id/0_qbqx90ic/version/100000'
         },
     }, {
         # non-digits in course id
@@ -117,6 +110,7 @@
     _UICONF_ID = '29375172'

     def _real_extract(self, url):
+        entry_id = ''
         mobj = self._match_valid_url(url)

         reference_id = mobj.group('reference_id')
@@ -129,12 +123,17 @@

             webpage, urlh = self._download_webpage_handle(url, video_id)

-            mobj = re.match(self._VALID_URL, urlh.url)
+            mobj = re.match(self._VALID_URL, urlh.geturl())
             reference_id = mobj.group('reference_id')
             if not reference_id:
                 reference_id = self._search_regex(
                     r'data-reference-id=(["\'])(?P<id>(?:(?!\1).)+)\1',
-                    webpage, 'kaltura reference id', group='id')
+                    webpage, 'kaltura reference id', default='', group='id')
+                entry_id = self._search_regex(
+                    r'data-entry-id=(["\'])(?P<id>(?:(?!\1).)+)\1',
+                    webpage, 'kaltura entry id', default='', group='id')
+                if not any((entry_id, reference_id)):
+                    raise ExtractorError('Unable to find kaltura "reference id" or "entry id"')
             partner_id = self._search_regex(
                 r'data-partner-id=(["\'])(?P<id>(?:(?!\1).)+)\1',
                 webpage, 'kaltura widget id', default=self._PARTNER_ID,
@@ -145,9 +144,10 @@
                 group='id')

         query = {
-            'wid': '_%s' % partner_id,
+            'wid': f'_{partner_id}',
             'uiconf_id': ui_id,
-            'flashvars[referenceId]': reference_id,
+            'entry_id': entry_id,
+            'flashvars[referenceId]': reference_id
         }

         if self.LOGGED_IN:

Thank you so much!

Prompted by [@D357R0Y3R](https://github.com/ytdl-org/youtube-dl/pull/32812, I made a version of the @sulyi branch for yt-dlp and also for yt-dl.
The common feature of both was that the POSTing to the login URL for username/password authentication times out with no response. This also happens with a wget command like:
wget --body-data '{"email": "PRIVATE", "password": "PRIVATE"}' --method POST --header 'Content-Type: application/json' --user-agent 'Mozilla/5.0' -v -S 'http://www.oreilly.com/member/auth/login/'
Presumably there is a CF-style bitbucket for incoming HTTP requests whose name is not on the door, which might be overcome with curl-cffi.
Or is there a new API that should be used?


Update pci-req9.md

Under 9.3, I made "such as" into a parenthetical phrase, and I changed "etc." to "and so on" instead.
Under 9.4, 9.6, and 9.8, I replace the colon with a period.
Under 9.9.1, I removed a period.

@EdPrice-MSFT,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/compliance/pci-req9.md
:white_check_mark:Succeeded
View

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


Invalid Amiibo Dump when loading Tears of the Kingdom Link Amiibo

Not sure if the TOTK Link Amiibo that's circulating online is an actual bad dump or if this editor doesn't see the new Link Amiibo properly.

make sure the amiibo bin file is registered in game.


refactor: improve memory stats perfomance

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Replace StatsAlloc with tikv-jemalloc-ctl to get memory stats.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)
#5613

cc @lmatz @chenzl25 Could you please help to do the perfomance tests again after this PR?

Of cause, I already test it on my own laptop and the performance has come back.


[BUG] Quadro-quoted string

There is a string literal which starts with 4 quotes instead of 3 in console.py:
https://github.com/willmcgugan/rich/blob/0ea2d45a9527a99fd6a6dbe71dcf06fb9ff1e253/rich/console.py#L104
It breaks syntax highlighting in some poor editors.

People are actually looking at my code?!
Fixed in 9.5.0

Source code is the best documentation.
Thank you!


1.0 Docs for centraldashboard

We currently don't have application docs for the central dashboard.
https://www.kubeflow.org/docs/components/
We should have docs for each application that goes 1.0. So we will need some documentation for centraldashboard. This can probably be pretty simple since the central dashboard is pretty self explanatory.
We do have this documentation about accessing UIs
https://www.kubeflow.org/docs/other-guides/accessing-uis/
But its kind of buried.
We might want to move or include some of that content in the central dashboard docs.

@avdaredevil could you please provide an update? What docs if any are missing for the central dashboard?

Some notes from meeting with @avdaredevil

1 paragraph about centraldashboard
Some description about managing namespaces and adding contributors

Thanks @jlewi! For the bit about namespaces and contributors, we can have a high-level overview and link to this doc for details:
http://kubeflow.org/docs/other-guides/multi-user-overview/


Uot 91839

Sentence transformer code. documentation on confluence.

@rha930 @katerdowdy What is the status of this PR?  Do we want to pull it in, or move it to an experimental section?

@rha930 @katerdowdy What is the status of this PR? Do we want to pull it in, or move it to an experimental section?

i have a task to go through this and other sentence work but it keeps getting pushed back.


kube-rbac-proxy image update

Currently cluster-api uses a quite old kube-rbac-proxy container image version v0.5.0.
The newest kube-rbac-proxy image is v0.16.0.
Is something speaking against an update here? Our CVE scanners are already complaining :-)
The corresponding code is here I guess:
https://github.com/k3s-io/cluster-api-k3s/blob/286bede3a5d3d7a4372d37dfd8f5e34c55b76111/bootstrap/config/default/manager_auth_proxy_patch.yaml#L13

It is not maintained for a while, will bump up the version, thanks!


Pagination

I see you're using PHP_INT_MAX for digital ocean's "per page" value in pagination. I don't think they will support a number that large, and even if they did, we'd likely run out of memory on our server trying to deal with a dataset that large all at once.

I see it is using a static 200 now, but the actual page is not being specific, limiting users to 200 droplets.  Should a page parameter be added to droplet getAll?

Does this issue apply anymore?  The $per_page and $page variables exist for the getAll function.

The issue was rather about automatic pagination, so we could get all, and then the library would iterate over all the pages for us.

This issue appears to still be under discussion, so I thought I'd add some more info.
When querying https://api.digitalocean.com/v2/droplets, the DigitalOcean API returns a meta key with total number of droplets.

Retaining this meta value at the end of the getAll() request would make paginating the API much easier with this library and the package @GrahamCampbell developed (https://github.com/GrahamCampbell/Laravel-DigitalOcean).
If anyone has suggestions for paginating the API rather than setting a high $per_page variable, I'm all 👂.


Model Zoo Report is empty

I have recently seen that the model zoo report is empty. Not sure if it is related to the new organization of models in the model zoo or not. @gongsu832 any thoughts?

The onnx folks moved the old onnx zoo repo under a new subdirectory, I would chat with @cjvolzka who I think found the correct new place for the data.

All the "old" models in the zoo moved to https://github.com/onnx/models/tree/main/validated. So for example if a model path was https://github.com/onnx/models/tree/main/vision/classification/mnist it should become https://github.com/onnx/models/tree/main/validated/vision/classification/mnist

Sigh. First "archive", now "validated". Wonder what's next.

Solved by #2678