add ownerference of alluxioruntime, delete dataset will delete alluxioruntime with the same name

Ⅰ. Describe what this PR does
This PR edits the deleting logic of dataset and alluxioruntime , so that delete dataset can cascading delete runtime.
The alluxioruntime needs to query the cache through the dataset when it is deleted. Now , it can be deleted without a dataset.
During the deleting logic, I have found a bug : when deleting alluxioruntime, the pvc can be deleted if the pvc name is not same as the volume name defined in pod.Spec.Volumes
So , I changed the calibration condition from "if pvc.Name == pvcName" in function GetPvcMountPods to "pvc.PersistentVolumeClaim.ClaimName == pvcName".
Ⅱ. Does this pull request fix one issue?

#405
#420
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
when a pod is using the pvc ,delete the dataset, it cannot be deleted.
When deleting a dataset , the alluxioruntime will be deleted later.
Ⅴ. Special notes for reviews
the return values in cache.go are not consistent， I change them to "states".
the pr was closed because some reason, I create it again.

Codecov Report

Merging #444 (7867dbb) into master (9508bf6) will increase coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #444      +/-   ##
==========================================
+ Coverage   18.18%   18.23%   +0.04%
==========================================
  Files          53       53
  Lines        2271     2265       -6
==========================================
  Hits          413      413
+ Misses       1819     1813       -6
  Partials       39       39

Impacted Files
Coverage Δ

pkg/ddc/alluxio/cache.go
0.00% <0.00%> (ø)

pkg/ddc/alluxio/shutdown.go
0.00% <0.00%> (ø)

pkg/utils/crtl_utils.go
39.28% <0.00%> (-4.72%)
:arrow_down:

Continue to review full report at Codecov.

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


Jupyter Notebook progress bar is broken

Hi 👋 ! I am running into an issue issue when using vscode 1.66 : when running pymc models there is a progress bar output which was working fine before the update and now the progress bar simply prints down :( See pic below

This is how it prints in Jupyter notebooks (anaconda) and it was looking so in vscode 1.65.2

Here is the info about my session
Version: 1.66.0 (Universal)
Commit: e18005f0f1b33c29e81d732535d8c0e47cafb0b5
Date: 2022-03-30T05:50:04.419Z
Electron: 17.2.0
Chromium: 98.0.4758.109
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 21.4.0

Here is a reproducible example:

import numpy as np
import pymc3 as pm

x = np.random.normal(loc=0, scale=1, size=100)

with pm.Model() as model:

    mu = pm.Normal("mu", mu=0, sd=1)
    sigma = pm.HalfNormal("sigma", sigma=3)
    likelihood = pm.Normal("likelihood", mu=mu, sigma=sigma, observed=x)

    trace = pm.sample(1000)

Thank you for working so fast on this one! Looking forward for the next update/release 🚀 !

@rebornix I couldn't verify this fix:

Works for me too.


Suppress error message when setting status-utf8

Fixes #53. If you’re using a tmuxline.conf snapshot file, you’ll have to recreate it.

This seems to be a better solution than #57 for backward compatibility.

How can I help to merge this PR (or #57) in master? Its a little bit uncomfortable when you set up a new tmux and get error instead of a prompt.

:+1:
This looks like the best option for backwards compatibility while resolving the issue for those on a newer version of tmux.

@edkolev Would you please review this pull request? Looks like the best option to fix this bug

I think it's better to just set[w] -gq everything, without singling out status-utf8. If another option becomes obsolete again, we're still covered. Unknown option warnings from a plugin give zero value to the plugin users.

Personally, I'm not a fan of gratuitous error suppression. Tmux's option API is changing and there's a good chance some other option will disappear. When that happens I'd like to know about it so I can investigate whether it would warrant any adjustments other than suppressing its error.

@zeorin Right. If you're the plugin writer/maintainer, you would want to do that. But if you're only a plugin user, the errors are only annoyance.

Ideally, a maintainer is running/testing on pre-release versions of tmux and that shouldn't happen to users even if he/she doesn't suppress all error messages: this script would be updated before a tmux release.
The maintainer of this script seems to have abandoned it, so that's not happening.
At this point in time, I'm simply using my pull request branch in my dotfiles, but I'm not attempting to maintain this script.
You can always fork it yourself and develop it the way you want.

Sorry, I'm not able to spend time on side projects at the moment. If anyone is willing to become a collaborator, please let me know. Note that backward compatibility is really important.

Closing due to age.


AKS Learn feedback: we should reminder customer pay attention of azure disk zone limit when resize nodepool

Type of issue
Missing information
Feedback
Hi team,
During nodepool resize, if customer has azure disk in use. We should mention that the new nodepool needs to be same availability zone as old nodepool. Incase azure disk cannot be mounted to new node:
For example, previous node is created in eastus2 az01. but new nodepool is created without zone selection.
" 2 node(s) had volume node affinity conflict. preemption: 0/2 nodes are available: 2 Preemption is not helpful for scheduling."
Page URL
https://learn.microsoft.com/en-us/azure/aks/resize-node-pool?tabs=azure-cli
Content source URL
https://github.com/MicrosoftDocs/azure-aks-docs/blob/main/articles/aks/resize-node-pool.md
Author
@schaffererin
Document Id
8f3a2dfe-0619-3b4b-aa96-6a9f5938d690

#assign:schaffererin


Use gunzip -c instead of zcat [phab:D786]

Created by zarko at 2016-03-21 23:02:52:
Fixes #1024.
On OS X, zcat always appends a ".Z" to whatever file
it's passed. gunzip -c doesn't do that.

shahms accepted this revision at 2016-03-21 23:18:28

zarko closed via commit 573179fb4c5149e0fcc6eb3aeb2f95a3e751b418 at 2016-03-21 23:28:00


LibVCX state machines update make too strong assumptions

There's this common pattern across libvcx objects (credential/proof) where upon creation, or on next expected step after creation, the user passes in connection handle. Let's take issuer's side of credential for instance.

Credential instance is created by vcx_issuer_create_credential - all good so far. Creates state machine in IssuerState::Initial state.
The next expected step is to call vcx_issuer_send_credential_offer which takes credential_handle and connection_handle as parameters. It moves state from IssuerState::Initial to IssuerState::OfferSent and embeds provided connection_handle into the state machine. It's responsibility of the caller to provide correct connection_handle.
Now the issuer should call vcx_issuer_credential_update_state or vcx_issuer_credential_update_state_with_message. This tries to move state machine from IssuerState::OfferSent to IssuerState::Finished and in the process of doing so, it uses connection_handle provided from the step 2.

The problem here is that IssuerCredential statemachine (and others too) assume that the connection handle reference they keep within their state machine will be valid on the next call, but that assumption can be broken easily if you:
Run step 1, 2, serialize IssuerCredential, kill process, run new process, load up and deserialize the IssuerCredential instance. Next updateState() will fail because the state machine refers to connection handle which is not valid anymore.
This is a bit hairy example, but the same thing can easily happen in distributed environment when steps 1, 2 happens on process A and 3 happens in process B, both reading/writing serialized vcx objects from/to shared storage.
Solution proposal 1
For sake of avoiding breaking changes, add methods (specific method naming can be discussed):

vcx_issuer_credential_update_state_safely
vcx_issuer_credential_update_state_with_message_safely
which should take extra parameters of connection_handle so no assumptions on memory state would be made.

So for example:
pub extern fn vcx_issuer_credential_update_state(command_handle: CommandHandle,
                                                 credential_handle: u32,
                                                 cb: Option<extern fn(xcommand_handle: CommandHandle, err: u32, state: u32)>) -> u32 {

would have safe alternative
pub extern fn vcx_issuer_credential_update_state_safely(command_handle: CommandHandle,
                                                 credential_handle: u32,
                                                 connection_handle: u32,
                                                 cb: Option<extern fn(xcommand_handle: CommandHandle, err: u32, state: u32)>) -> u32 {

which would use provided handle, instead the one stored in SM IssuerSM.state.OfferSent(OfferSentState).connection_handle
Alternative proposal 2
The proposal number 1 would mean adding quite a few methods on API level as well as internally. Possible solution requiring less changes would be something like vcx_*_update_connection_handle and so caller would be responsible to call this method after doing vcx object deserialization and passing correct connection handle.
@Artemkaaas @jovfer @ryMarsh44 what do you think? Is this valid concern or am I possibly missing something?
Props to @airoasis for pointing this out!

Here is a temporary workaround for this issue

Thanks, Patrik
We also came across this problem recently.
In Evernym fork we did breaking change by the embedding of the whole required connection-related data into Issuance / Presentation state machines. I think it's the most proper way, but it makes not completed state machines broken but keep API and steps the same.
We did that breaking change because there are no production uses for aries protocols yet for Evernym products and we consider VCX aries protocols as in progress.
There is also the option of using indy Wallet to persist connection information by corresponding thread id. But that case we need to care about freeing data on completion and releasing. And as far LibVcx doesn't manage the persistence of the state machine we declined that option to be relevant with the flow.
Describing your options:

In that case, we need to deprecate the existing update_state methods and force users to migrate to a new one. That actually almost equals to doing breaking change now.
The flow getting more complex and unclear.

@Artemkaaas thanks for feedback!
I am not sure if I like the idea of embedding all connection data (probably a DDO?) into credential/proof state machines (do I get it right?) I think this must conflate them a lot, turning them inbto quite deep data structure..? Also connection can change within its life time, the parties might rotate keys, change endpoints (although this is not in VCX yet and I am not sure if there's even RFCs for this, I presume some "pairwise connection update" protocols will exists). So if connection might change, it could be cumbersome to keep track which objects needs to have their embedded information updated.
I agree the proposal 2 would be error prone and developer unfriendly, but wouldn't require of deprecation of update_state. Instead, it would be strongly recommended to adjust existing code and always update connection handle before calling update_state.
I'd prefer the proposal 1 from my first the most. The connection can have its own lifetime and be injected to other objects on demand whenever needed. Also it can be implemented as nonbreaking change by adding methods. And I agree that the current update_state should be deprecated, but seem to me like it would be done so for a good reason.

I agree that option 1 (passing connection_handle to every depending function) is the most proper way. I'm only concerned about new APIs.

Do you think it can't be implemented without doing breaking changes? I think there can be several ways of implementing that with varying amount of breaking changes. Correct me if I am missing something, I think completely non-breaking without be:

As proposed, addition of following methods:

vcx_*_update_state_safely

everything can stay as is. Calling the currently existing methods vcx_*_update_state, it will use the cached connection_handle, everything stays as is. However, if user calls vcx_*_update_state_safely, it will do extra "preparation step"

Update connection_handle record in the state machine
Proceed same way as vcx_*_update_state did before.

@Patrik-Stas
I have read your proposal and discussion with @Artemkaaas
It is a good change to make connection handle updatable in the connection but the implementation does not seem correct to me.
I am voting for the option 1 (make a new call) but with a slight tweak.
LibVCX API is not really convenient for development purposes. I suggest to start making a API v2 and the first change would be to introduce a connection handle to *_update_state functions.
We will mark this API as experimental so if we will feel that we made a mistake we will just break it. When we will feel that it is a good API that is comfortable to use we will deprecate the old calls and make the new API stable. However, due to SemVer we will not be able to drop the old API until we release a new major release (this should be contacted with Hyperledger)

@KitHat
Yeah I think have experimental V2 API can be alright, it would be nice to have an API with only 1, clear, proper way of updating state machines. Since you say

LibVCX API is not really convenient for development purposes.

Do you have other changes in mind you would consider introducing with V2 API?

2nd question - do you oppose extending current API layer with update methods taking additional parameter?

@Patrik-Stas

Do you have other changes in mind you would consider introducing with V2 API?

Not yet. But I think we need to form all flows into a more homogenous style -- they should have the same naming convention and not to have some helpers that allow to do switch machines without some params.

2nd question - do you oppose extending current API layer with update methods taking additional parameter?

I personally do not like the naming. _safely suffix does not tell you anything about the difference between this function and I had hard time to express it in the function name. So I thought that the old way of updating is erroneous and we should just deprecate it.

@KitHat

I personally do not like the naming. _safely suffix does not tell you anything about the difference between this function and I had hard time to express it in the function name. So I thought that the old way of updating is erroneous and we should just deprecate it.

Yeah the _safely suffix was just first thing that came up in my mind to illustrate my point. Rather the question is/was whether you'd be okay with adding alternative version of vcx_*_update_state with additional connection parameter to current VCX API, rather than to V2 API. Naming can be different. Maybe simply vcx_*_update_state_v2? Trying to be "descriptive" names like vcx_*_update_state_safely or something like vcx_*_update_state_using_connection is getting quite long and only arguably accurate.

I think that vcx_v2_*_update_state would be the best variant. Also I would consider changing names of entities we operate with -- some of them are really misleading, like disclosed_proof

@KitHat
Cool! Just to make sure we are on the same page - so do you agree on:

not yet adding new, full blown, V2 experimental API layer,
but rather extending current LibVCX API addition of methods named vcx_v2_*_update_state alongside existing vcx_*_update_state?

@Patrik-Stas
Yes, but I would suggest to create a separate directory in api of libvcx


Create GVE performance test

Formatting will be fixed in the next commit. I have my VIM tab set to 2 spaces, which doesn't match the tab characters being used here.

/hold

/unhold

/hold

/unhold

/hold

Weird issue where the networks aren't getting cleaned up correctly due to them being in use by a firewall rule at the time of deletion. Investigating. But otherwise good to go.

/unhold

/lgtm


New submission page infinite loading

When adding a new submission, the page is stuck on loadingthe form.
How to reproduce:

Log in to DSpace 8 sandbox
From the left admin menu, click on New
Click on item
Select the collection (example Articles)
The page will continuously load.

The submission page should load with no issue and show the collection form to deposit an item.

@GraziaQuercia : Thanks for reporting.  However, I can no longer reproduce this issue on the Sandbox, so I suspect it was a temporary issue. It's possible either the sandbox was restarting itself or temporarily overwhelmed.  Please try again and see if this is working.
I'm able to get this to work on my end:

Login as an Admin<EMAIL_ADDRESS>on sandbox.dspace.org
Go to "New" -> "Item"
Select "Articles" collection
Submission form loads in a few seconds.

If it's still not working for you, please let me know on this ticket. Let us know what browser you are using, or anything else you feel you might be doing different than I am.  It's always possible I've overlooked something, and perhaps with more details we'll be able to figure out what is going on.
For now, I'll close this. But we can reopen if it is found to still be reproducible.  Thanks!

Good afternoon, just to let you know I just tested this and it works perfectly! Best regards.


Make email verification sending async

The function _sendEmailVerification now returns immediately. This removes the successful external dependency requirement for actions such as creating an account and resending the email verification.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

-2 of 7 (71.43%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.06%) to 93.65%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/domain/account/account.repository.ts
5
7
71.43%

Files with Coverage Reduction
New Missed Lines
%

src/routes/transactions/entities/tests/human-description.builder.ts
2
60.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
6118

Relevant Lines:
6321

💛  - Coveralls


Check for Xcode 11 and Xcode 12 directory names in build_ios_framework_module_test

Description
Xcode 12 changed a generated XCFramework directory from ios-armv7_arm64->ios-arm64_armv7.  Check for the existence of both instead of an explicit Xcode version check.
Related Issues
Fixes https://github.com/flutter/flutter/issues/60043.
Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

So Xcode's change won't break anything in the tool?

So Xcode's change won't break anything in the tool?

No, the tool doesn't care about this structure, it calls xcodebuild to generate the XCFramework, and then Xcode consumes them in an add-to-app host app.  The test is just checking that the files actually gets produced.
https://flutter.dev/docs/development/add-to-app/ios/project-setup#option-b---embed-frameworks-in-xcode

Google testing passed!


Update fast-safe-stringify?

fast-safe-stringify is now on 2.0.7 and it would be nice to update to use it

This does not depend on fast-safe-stringify. Am I missing something?

It used to but apparently that dependency has been removed:
https://github.com/davidmarkclements/quick-format-unescaped/commit/cad2849a129fa57bd92e4ba316479aeabc1f6eb6#diff-b9cfc7f2cdf78a7f4b91a753d10865a2L17


More improvements.

Hi again!

The WinLeave split bug is a bit strange, though:

Might possibly be caused by an issue with nvim_win_set_config()?
WinEnter is triggered before the file is loaded, but after the window is split when splitting windows, which maybe causes that issue somehow, but the same thing does not happen if nvim_win_open() is used instead...

Awesome, thanks :)


adding license scan

Check for licensing info headers, fossology and scancode tests

/test presubmit-kbp-fossology

/lgtm

/approve


Fix regex to be compatible with ruby 1.8.7

Current code uses some crazy regex stuff which doesn't exist in ruby 1.8.7, so I changed it to be simpler and compatible with 1.8.7.

Is there a reason this PR is stuck in limbo?


outboundIpAddresses is null for ACA with ACA environment with Consumption + Dedicated tier

Hi,
We are currently exploring the Azure Container App (ACA) Environment Consumption + Dedicated tier (preview) because we would like to have support for user defined routes to be able to route all internet traffic to our Azure firewall.
We noticed that the outboundIpAddress for the ACA within this ACA environment is null where it is always filled for ACA's in the ACA environment that have the Consumption tier.
Is this by design or a bug since it is in preview? If it is by design how should I get the outboundIpAddress for such a ACA?
Below a screenshot of part of the Json view:

@kendallroden Could you explain this one since issue is related to #355 ? Thank you!

Hi,
Unfortunately @kendallroden is not working at Microsoft anymore :(
Is your Container App environment defined as Internal? If so, then the you won't see an outgoing IP and it's up to you to set up Azure Firewall or NAT gateway. Any egress traffic will be routed through these services IP address.

Hi, Unfortunately @kendallroden is not working at Microsoft anymore :( Is your Container App environment defined as Internal? If so, then the you won't see an outgoing IP and it's up to you to set up Azure Firewall or NAT gateway. Any egress traffic will be routed through these services IP address.

@torosent  Yes it is defined as internal. But if the Consumption plan is used as internal the outgoing IP is filled so it might have somthing to do with the plan. However we are now using the solution with the Azure Firewall and use that PIP for whitelisting purposes and it works. This issue can be closed.

Closing per customer feedback.


Improve std.range.radial documentation.

Add more examples to show exactly what happens if one end of the range is reached first.

Auto-merge toggled on


Godot 4 Beta 17 brings a type error in dialogue_line.gd

Describe the bug
Trying to trigger dialogue causes this error to fire at line 33 of dialogue_line.gd:
Trying to assign an array of type "Array" to a variable of type "Array[Dictionary]".
It seems related to the many breaking changes in array type hinting. Some other variable assignments further down the script will also fire the same error, and even using workarounds seems to reveal different errors in other files.
Affected version

Dialogue Manager version: v2.8.7
Godot version: Godot 4 Beta 17

To Reproduce
Steps to reproduce the behavior:

Just trigger any dialogue or use the example balloon.

This should be fixed now in v2.8.8.


Support for regex flags

Implement support for regex flags
This change adds support for re.IGNORECASE, re.DOTALL and
re.UNICODE regex flags. Flags are extracted from compiled
regex. Also, it supports subpattern flags introduced in
Python 3.6.
re.IGNORECASE controls literals: if it is enabled, instead
of generating static literal, it generates either
lowercase or uppercase character.
re.DOTALL: if enabled, "." matches newlines.
re.UNICODE controls character categories (e.g. "\d" or
"\w"): if enabled, categories include unicode characters.
In Python 2 default is to generate ASCII only data, in
Python 3 - unicode. Python 3 also has reverse flag -
re.ASCII to disable unicode character generation.
Fixes #2, #3 and #5

@Zac-HD Could you take a look at this PR. It is full of humiliation and suffering (thanks to Unicode)

Addressed all PR comments (unless otherwise stated), added a lot more documentation and code comments.

Ok, I didn't know about this given trick (although it doesn't look elegant), so I used the reverse: using find() to search for examples that do not match. Obviously, those two variants can be used interchangeably. I added an extra "positive check" (the one that you're concerned about) just to be safe.
I will look into given trick.

New update, changed the way tests are run (using given). Also added filter by matching generated examples with real regex. Added support for "groupref_exists" contruct ("(?(id)yes-pattern|no-pattern)") and a smarter generation for "$".


Properly merge RContextForwardableStuff

This is a quick fix for #4. Previously ctx.insides were replaced after executing a tag, instead of merged. I have confirmed that this works, but won't have the time to do proper performance testing for some time. Better working with potentially bad performance than broken I guess.
Closes #4

Thanks for the suggestions!

Yes, I think this terminology makes sense :+1:

Codecov Report

Merging #12 (315befa) into main (dd7edda) will increase coverage by 0.91%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main      #12      +/-   ##
============================================
+ Coverage     33.58%   34.49%   +0.91%
- Complexity      624      951     +327
============================================
  Files           127      147      +20
  Lines          6620     8871    +2251
  Branches       1065     1662     +597
============================================
+ Hits           2223     3060     +837
- Misses         4138     5315    +1177
- Partials        259      496     +237

Impacted Files
Coverage Δ

org.lflang/src/org/lflang/graph/TopologyGraph.java
0.00% <0.00%> (-88.00%)
:arrow_down:

...ng/src/org/lflang/generator/ParameterInstance.java
30.76% <0.00%> (-24.24%)
:arrow_down:

...lflang/src/org/lflang/generator/TimerInstance.java
57.14% <0.00%> (-9.53%)
:arrow_down:

...lang/src/org/lflang/generator/ReactorInstance.java
47.66% <0.00%> (-7.38%)
:arrow_down:

...g.lflang/src/org/lflang/generator/TargetTypes.java
63.82% <0.00%> (-5.94%)
:arrow_down:

org.lflang/src/org/lflang/JavaAstUtils.java
47.27% <0.00%> (-4.73%)
:arrow_down:

...ang/src/org/lflang/generator/rust/RustGenerator.kt
65.51% <0.00%> (-2.91%)
:arrow_down:

org.lflang/src/org/lflang/generator/Validator.java
22.22% <0.00%> (-1.78%)
:arrow_down:

org.lflang/src/org/lflang/ModelInfo.java
31.57% <0.00%> (-1.76%)
:arrow_down:

org.lflang/src/org/lflang/util/XtendUtil.java
0.00% <0.00%> (ø)

... and 55 more

Continue to review full report at Codecov.

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

Great! Thanks!


Tab bar size problem

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Framework7 Version: EXACT RELEASE VERSION OR COMMIT HASH, HERE.

Platform and Target: CORDOVA, ANDROID, ETC.

What you did
i am trying to use tabs but toolbar-inner is not working when call multiple tab tageg in index.
I will explain in briefly:-
Steps:-

I have created index.html
create new page with tabs  items.html
create another page with tabs favorite-items.html
4 Navigation:- index to items.html(tabs working fine) then items.html to favorite-items.html (tab not working properly if swipe the tab)

Expected Behavior
table bottom line in same way as first time click
Actual Behavior
table bottom line in size problem when swipe on second

It is hard to say like that. Would be really good to see somekind of live example or JSfiddle/Codepen with the issue

Open this link on iphone or Chrome PC  version >= 56. FF works fine.
http://bomlugarclubedasartes.com/app/
Notice that the tab does not fit the screen causing a second scrollbar to appear. Seems to me it was caused by some webkit update. It did used to work, I checked with older versions of my app and all are having the same bug.
I will post a fix if I manage to fix it.

Issue is closed because of outdated/irrelevant/not actual
If this issue is still actual and reproducible for latest version of Framework7, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


init_apartment - option not to throw

There are situations in which a thread is already CoInitialize'd - not under app control, say a framework library.
init_apartment should have an option to ignore - rather than throw - already initialized thread.

The init_apartment function simply calls CoInitializeEx - you can call that functions directly to avoid the exception.


[C++][Parquet] Writing wrong format version when using ParquetVersion::PARQUET_1_0

I'm not sure if this is related to PARQUET-838, but I ran into this while debugging something
Reporter: Wes McKinney / @wesm
Assignee: Wes McKinney](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=wesm) / @wesm
Note: This issue was originally created as PARQUET-841. Please see the migration documentation for further details.

Wes McKinney / @wesm:
PR: https://github.com/apache/parquet-cpp/pull/225

Wes McKinney / @wesm:
Issue resolved by pull request 225
https://github.com/apache/parquet-cpp/pull/225


feat(icons): added rose icon

What is the purpose of this pull request?

[x] New Icon
[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other:

Description
This comes from the lab branch.
Icon use case
Anything and everything romantic, i.e. Valentine's Day.
Alternative icon designs

Icon Design Checklist
Concept

[x] I have provided valid use cases for each icon.
[x] I have not added any a brand or logo icon.
[x] I have not used any hate symbols.
[x] I have not included any religious or political imagery.

Author, credits & license

[ ] The icons are solely my own creation.
[x] The icons were originally created in #1398 by @danielbayley
[ ] I've based them on the following Lucide icons:
[ ] I've based them on the following design:

Naming

[x] I've read and followed the naming conventions
[x] I've named icons by what they are rather than their use case.
[x] I've provided meta JSON files in icons/[iconName].json.

Design

[x] I've read and followed the icon design guidelines
[x] I've made sure that the icons look sharp on low DPI displays.
[x] I've made sure that the icons look consistent with the icon set in size, optical volume and density.
[x] I've made sure that the icons are visually centered.
[x] I've correctly optimized all icons to two points of precision.

Before Submitting

[x] I've read the Contribution Guidelines.
[x] I've checked if there was an existing PR that solves the same issue.

The smallest circle would need to be on the grid for it to be crisp, also, the bottom right path feels a bit wonky:

A bit more geometric redesign that's crisper at 1x:

Open lucide studio

Yeah, I guess, you're right. I can fix it, but we'll need cubic curves:
Open lucide studio

I feel like the leaf is a bit off-kilter.
Maybe we can steal something from https://lucide.dev/icons/sprout

I feel like the leaf is a bit off-kilter.
Maybe we can steal something from https://lucide.dev/icons/sprout

Maybe, the generic shape is OK, but pixel perfection would need to be improved.

Do you mean for better sharpness?
I actually feel like your admittedly sharper version feels harsh and reduces the soft feel which I like.

Yeah, tastes aside, we have "I've made sure that the icons look sharp on low DPI displays" in our PR checklist for a reason, I actually recall it was in fact none other but yourself who wrote this one. 😄

Do you mean for better sharpness?
I actually feel like your admittedly sharper version feels harsh and reduces the soft feel which I like.

Yeah, tastes aside, we have "I've made sure that the icons look sharp on low DPI displays" in our PR checklist for a reason, I actually recall it was in fact none other but yourself who wrote this one. 😄

Fair enough.
If you center the center and keep the outer parts, they are not centered on the grid tho.
Open lucide studio

If you center the center and keep the outer parts, they are not centered on the grid tho.

I'm not sure what point you're trying to make here, these paths are from your original design, aren't they?
Yeah, that's the whole reason I did the redesign.

Icon use case
Anything and everything romantic, i.e. Valentine's Day.

Yeah I was playing around with an animated version for something related, at the time.
There is also rose-2, to match flower-2, which maybe should be included here?

Maybe we can steal something from https://lucide.dev/icons/sprout

@jguddas Sure, it was of course already based on leaves from existing floral icons sprout and flower-2…

the reduced roundness of @danielbayley version

Yeah, I think bloating it like that distorts the elegant appearance of a rose.

If you center the center and keep the outer parts, they are not centered on the grid tho.

@jguddas @karsa-mistmere I would have ran it though the optimisation step, which although a very useful tool, was sometimes nudging things slightly off-grid… But for reference, here are the grid-aligned circles it was based on, when I originally designed it in Illustrator: Lucide Studio link…

 Is that not allowed?

Unfortunately the smallest circle is off grid, hence @jguddas's little red marker

Nice feature, but yes exactly .5 off… Is that not allowed?

If you have are .5 off you will fill the pixel to only 50% which will look not so sharp.
We could make the center circle smaller, than it's more than 50% there.
Open lucide studio

@jguddas @karsa-mistmere We could do like this…

@jguddas @karsa-mistmere We could do like this…

Having the circle at 1.75 radius still means we have a pixel that is 25% filled.

I think this is the best of all the versions so far, which is just the current suggestion with the center of the circle snapped.

I disagree, the 8px circles are also super blurry at 1px.

I think this is the best of all the versions so far, which is just the current suggestion with the center of the circle snapped.

I disagree, the 8px circles are also super blurry at 1px and would need to alight to the grid more:

And again thin or not (although I'd very much prefer a rounder real rose, not a plastic one), this wobbly path would need to be fixed:

I think this is the best of all the versions so far, which is just the current suggestion with the center of the circle snapped.

I disagree, the 8px circles are also super blurry at 1px and would need to alight to the grid more

@karsa-mistmere @jguddas How about this?
As an aside, it would be super useful if we could upgrade ‘Studio’ to also (asynchronously) generate the relevant previews at various stroke-widths, and 100% scale pixel preview… Along with the ability to add temporary ‘guide’ layers (maybe with a .guide class or something) that aren’t forced to render at 2px stroke-width, and with a contrasting colour…

I think it looks sharp enough to merge.

Open lucide studio
With the aid of a couple of horizontal/vertical lines to bridge the gaps, this version has all of the arcs be grid-aligned.


Create ATM10 example

I'm already maintaining it locally, might as well share it.

Yeah, was thinking the same thing.
Also, while doing this it was the first time I needed to exclude the overrides/shaderpacks (due to an invalid character thing) but was wondering if excluding the overrides/shaderpacks should be a default thing? Would a server ever need the shaderpacks folder?


BWT-134: List all advisory types in pulldown and make pulldown searchable

Jira Ticket:
BWT-134
Jira Ticket URL:
https://bcparksdigital.atlassian.net/browse/BWT-134
Description:
List all advisory types in pulldown and make pulldown searchable

Plz squash commits before requesting review

Plz squash commits before requesting review

Done!


webhook mutant

/retest

/retest

Everything looks good to go to me.  I tested it on my local minikube, and it seemed to work.
I remembered one last thing: We need a NetworkPolicy installed in OSD to allow the k8s API to make calls to our webhook service.  Should we rely on app-interface to install this, or should we have Clowder create it?  I think my preference is to have Clowder create it. wdyt?

I agree with you - I don't know the format of this NetworkPolicy - do you have an example of it?


Rename Tock APIs? (to be discussed)

Observation:

Tock provides more and more APIs
Some of them have no "easy name", in particular the recent Web Connector API - in my opinion the most useful and most used in the future (by front developers, integrators, orchestrators...)
Some have names refering to concepts, that not everyone is confortable with (eg. NLU / NLP for non-technical profiles)
It is sometimes difficult to understand the roles of those APIs, specially for newcomers (cf Gitter)

I propose to rename the APIs (and consistently documentation, Tock Studio menus/tabs, maybe Bot-Api artifactIds...) - even if it requires a bit of refactoring...

Probably for a major Tock release, since it possibly breaks things (at least names).

If we change artifactIds, it might be redirected automatically at Maven repo level (anyway we got through this with the ai.tock refactoring)

Draft:

NLP / NLU (for Tock Studio as well as APIs) => Model and Model API
Admin API => don't change (in my view: not restricted to Tock Studio frontend so Admin seems OK for the API)
Bot API => misleading since people may think it is an API to talk to the bot / integrate with the bot ; for the moment it is only adding Built-in Stories to an existing bot, so... what about Story API ? If we think the API should extend to more than configuring stories (but less than the Admin API), maybe name it (Bot) Builder API ?
Unnamed "Web Connector API" => in my view the most important, most obvious and useful for newcomers, frontend developers (alternative to tock-react-kit), integrators... to integrate with bots, for orchestration, etc. => since Bot Api already exists, name it Chat API ?

To be discussed.
Please feel free to express another opinions or propose other names.

Also see #850

Hi François,
It's (always) a good idea to challenge again naming :)
I'm not really comfortable with "Model", NLP ou NLU is more meaningful for me. If our aim is to make it significant to more people, maybe we just need to remove abbreviation and name it : "Language Understanding API"


Decide what direction to take for a 'job system' in duality

Summary
Currently there is no defined way of doing things when it comes to doing work on a different thread. One could use the TPL and async await but it behaves differently outside the editor compared to inside (see #592). There has also been some talk about creating custom duality job system to fill this need.
This issue is about deciding what direction to take here. What will duality's conventions be around this subject? Will we fully support TPL and async await or choose a different direction? Hopefully we can aggregrate the opinions and insights of ppl here and eventually come to a decision.
Analysis

Will make it clear what the standard way of offloading work to another thread is.
Should make it easy to offload work to a different thread.

I believe there are 2 major features that we want here:

A way to spread out operations over multiple updates like unity's coroutines do. This involves just the mainthread. No additional threads are required. Could be done with a IEnumerator approach.
A way to easily run something on a different thread and have it post back to the mainthread when its done. Will most likely involve a custom synchronization context or something similar.

It should be possible to implement these independently as these features are really different in nature and use cases.

It should be possible to implement these independently as these features are really different in nature and use cases.

Agree completely. I would also say it makes sense to limit this issue to item two:

A way to easily run something on a different thread and have it post back to the mainthread when its done (if we use the task library you could opt out of this behaviour if you want). Will most likely involve a custom synchronization context or something similar.

While discussing item one in a separate issue, as soon as someone needs it enough to open one.

Currently working on a coroutine infrastructure as part of the community plugin

Sample on Unity's new ECS interface for its internal job system:

https://github.com/keijiro/Voxelman
https://github.com/Unity-Technologies/EntityComponentSystemSamples

Don't think it should serve as direct inspiration to our use case for a few reasons, but doesn't hurt to take a look their approach either.

Scanned through it for a bit but it seems they are restructuring gameobjects altogether with this new entity component system. For instance they are moving data to structs and they also pack together all these structs of a certain type together in memory. This allows them to use linear memory access instead of random access. Combined with parallel execution in batches this allows for some serious performance gains.
Interesting indeed but it goes quite a bit futher than just a 'job system'

Most straightforward would be to implement a cusotm synchronizationcontext that posts continuations to the mainthread.
Since the editor already has a synchronizationcontext due to winforms (https://referencesource.microsoft.com/#System.Windows.Forms/winforms/Managed/System/WinForms/WindowsFormsSynchronizationContext.cs,c7dfb662bbd6227d,references) I think we only need to install this custom context if there is no context already.
This should make it quite straightforward to use the task library in duality


Create FastStream template

[ ] Build application and run tests
[ ] Deploy CI scripts
[ ] Script to generate AsyncAPI docs
[ ] Write hello world example

Is it the same with #569?

Yes, it is the same. I'll close #569 as duplicate.


metricsstore: Small refactor and cleanup

This is a small refactor to the metrics store.
In this PR:

Use GetMetricsStore() function to initialize instead of DefaultMetricsStore variable with init()
MetricsStore struct moved to pkg/metricsstore/types.go

Use GetMetricsStore() function to initialize instead of DefaultMetricsStore variable with init()

Could you describe the reason for this change so it's easier to review?

Handing this off to @jaellio


"Fill match arms" assits is corrupts the match block when it has syntax errors

For
match Some(()) {
    Some(())
    None => self
}

This currently only appends an extra None => {} arm, so it doesn't corrupt the match block anymore.


[feature request] Support xargs -I <replace_str>

The xargs command is super useful but is too constrained to use for invocations that depend on positional parameters. This feature request asks for the traditional -I/-i/--replace flags to be added.
Eg:
$ echo -e 'a\nb\nc'|xargs -I arg echo 1 arg 2
1 a 2
1 b 2
1 c 2

Unfortunately, -I always constrains the maximum number of lines processed at a time to 1 and approximates the following loop:
$ for i in a b c; do echo 1 $i 2; done
1 a 2
1 b 2
1 c 2

Although both looping and xargs -I provide similar functionality, I think they serve different uses. Simple loops are wanted for simple lists and xargs is wanted for pipes.
Use case
In the absence of this support, task users are at the mercy of the subcommands they call. For example, here's how you would have to do copy with the current version of xargs:
$ echo -e 'a\nb\nc'|xargs cp -t dist

However, many tools don't support non-positional arguments like the target parameter so this isn't an option. Even Deno's own cp command doesn't support -t so this also fails:
{
  "tasks": {
    "hello": "echo -e 'a\nb\nc'|xargs cp -t dist"
  }
}

$ deno task hello
Task hello echo -e 'a
b
c'|xargs cp -t dist
cp: unsupported flag: -t

References

Briefly mentioned on the original xargs issue
https://github.com/denoland/deno_task_shell/issues/78

It would be good to split these out into two issues. Looping probably won't be implemented because if you are doing loops then you probably shouldn't be writing that in a task and instead use JavaScript for that (deno run ...), but having feature parity with xargs is desirable.

Thanks, I've made the split. Feel free to close the loop issue if it doesn't make sense--I opened it because I thought it was likely to come up again and it'd be nice to centralize that discussion.

@niedzielski thanks! I'll keep it open for now as a "suggestion" to see what other use cases come up.


Simplify variable initialization

This allows to see what the real default are using pydoc and simplifies
the code.

Nop, I actually missed a few if after.


[FEATURE REQUEST]  Access QuickAdd Append workflow from outside Obsidian

I love the workflow of opening Quick Add, and then being able to quickly select a Note from Obsidian to append content on.
Has anyone heard of a way to achieve this from outside of Obsidian? For an example, from a phone!
I know there are plenty of ways to add notes from outside of Obsidian, but being able to select a note and then append would be a dream!
Right now, I leverage Todoist and Integromat to create my own scrappy version, but lacks the flexibility of QuickAdd.
Has anyone been able to find something? Or is this unlikely with how Obsidian was made?
I saw Fleeting Notes but seems it can only really add notes.

Achieved this via Advanced URLs and Tasker on mobile.
May be worth adding to Documentation.
So happy ❤


Deployment issue with azure function:Trying to deploy zip file through azure cli

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az functionapp deployment source config-zip
Example: azfunctionapp deployment source config-zip -g azureGeoGroup -n \azureGeo --src https://drive.google.com/drive/u/0/folders/1OiWIOVTDAQE87U3z5yOizLyUjKFWoZ6J/hlsAPIGeocoder.zip
Errors:
[Errno 22] Invalid argument: 'C:\\Users\\Sanjay\\https:\\drive.google.com\\drive\\u\\0\\folders\\1OiWIOVTDAQE87U3z5yOizLyUjKFWoZ6J\\hlsAPIGeocoder.zip'
Traceback (most recent call last):
pip-install-_1mlo87m\knack\knack\cli.py, ln 215, in invoke
cli\core\commands\__init__.py, ln 625, in execute
cli\core\commands\__init__.py, ln 689, in _run_jobs_serially
cli\core\commands\__init__.py, ln 682, in _run_job
Temp\pip-install-_1mlo87m\six\six.py, ln 703, in reraise
...
cli\command_modules\appservice\custom.py, ln 367, in enable_zip_deploy
OSError: [Errno 22] Invalid argument: 'C:\\Users\\Sanjay\\https:\\drive.google.com\\drive\\u\\0\\folders\\1OiWIOVTDAQE87U3z5yOizLyUjKFWoZ6J\\hlsAPIGeocoder.zip'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az functionapp deployment source config-zip -g {} -n {} --src {}

Expected Behavior
Environment Summary
Windows-10-10.0.17134-SP0
Python 3.6.6
Installer: MSI

azure-cli 2.3.1 *

Additional Context

functionapp

from the error, the command expecting a local file path instead remote url

That is correct. Currently, only local files are supported.
Closing as "by design", please let us know if you have any additional questions.


tostring

I'm honestly just sort of fishing here.
It works in mGBA, so the problem might not be in BPCore-Engine.
However, on my GB300, I get a crash if I either call tostring or string.format. The overlay turns blue, and processing freezes.
I have no real idea where or why this happens. Because I do think both of those are included in BPCE, because it does work under an emulator. I do know the GB300's emulators are not well respected, because they do have faults.
Kinda hard to output a highscore, and I don't particularly want to rewrite tostring in Lua.

I think it's a memory problem. Probably?
I managed to stop the crashes with this insane little bit o' code:
do
	local std_tostring = tostring
	tostring = function(i)
		 if i == 0 then
			return "0"
		 elseif i < 0 then
			return "-" .. tostring(-i)
		 else
			local str = ""
			while i > 0 do
				local digit = i % 10
				str = std_tostring(digit) .. str
				i = math.floor(i / 10)
			end
			return str
		 end
	end
end

Afraid the only real thing I know about the GB300's GBA is it uses this. It is a bit hacky.
I did actually use Lua's tostring - just sharded the value so it was smaller, so I'm assuming it was some sort of memory fault in the GB300's implementation. Maybe it has some sort of limit on the size that can be allocated at any one time?
Anyways, it's probably way out of scope for BPCore-Engine. Just me fooling around to get stuff on a physical device, because that's fun.


Unable to access CompletionCreateParams from Groq.Chat.CompletionCreateParams

When trying to access Groq.Chat.CompletionCreateParams or Groq.Chat.Completions.CompletionCreateParams (which my VSCode auto complete suggests as valid imports) I get the following error:
Namespace '"/Users/bracesproul/code/lang-chain-ai/langchainjs/node_modules/groq-sdk/resources/chat/chat".Chat' has no exported member 'CompletionCreateParams'.ts(2694)

TSConfig
groq-sdk version: 0.2.1 (latest)
Is there a different recommended way to access this interface?

Hi, Brace! Thanks for reaching out. The team is currently looking into this and I'll try to post an update soon. 🤞

Hi, Brace! Thanks for reaching out. Could you please try again with groq-sdk version 0.3.0?

still getting the same error :(

Hi @bracesproul - if you're looking to implement an interface, then Groq.Chat.CompletionCreateParams is a namespace, not an interface. Maybe you mean Groq.Chat.CompletionCreateParams.Message (which is an interface)?
something like:
class MyParams implements Groq.Chat.CompletionCreateParams.Message {
  get role() {
    return 'somerole';
  }
  get content() {
    return 'somecontent';
  }
}

If you want just the type of the whole params object, that would be ChatCompletionCreateParams which you can import like so:
import { ChatCompletionCreateParams } from 'groq-sdk/resources/chat/completions';

function makeParams(): ChatCompletionCreateParams {...}

It might help to look at completions.ts to see how these things are defined more clearly. Does that help?

Hi @bracesproul - if you're looking to implement an interface, then Groq.Chat.CompletionCreateParams is a namespace, not an interface. Maybe you mean Groq.Chat.CompletionCreateParams.Message (which is an interface)?
something like:
class MyParams implements Groq.Chat.CompletionCreateParams.Message {
  get role() {
    return 'somerole';
  }
  get content() {
    return 'somecontent';
  }
}

If you want just the type of the whole params object, that would be ChatCompletionCreateParams which you can import like so:
import { ChatCompletionCreateParams } from 'groq-sdk/resources/chat/completions';

function makeParams(): ChatCompletionCreateParams {...}

It might help to look at completions.ts to see how these things are defined more clearly. Does that help?

Importing from the path you listed fixed it, I appreciate it!

I'm troubled by the same problem. I think this can be temporarily updated to the readme.


#1243 Navbar Responsive with Navbar Button

Codecov Report

Merging #39 (e167c20) into dev (a4f0386) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##              dev      #39   +/-   ##
=======================================
  Coverage   63.17%   63.17%
=======================================
  Files          13       13
  Lines         687      687
  Branches      207      207
=======================================
  Hits          434      434
  Misses        224      224
  Partials       29       29

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


Added Spotify Clone

Description
My project is the Spotify cLone which gives you idea of designing the interface of a website.
Checklist

[yes ] I've been assigned an issue related to this PR.
[yes ] I've used beautifiers.
[no ] I've added my Project's name and description to Index.md
[ no] I've made a README.md file for my Project.
[ no] The README.md file of my projrct contains Project title, Description, Use of project, Set up, Stack used and Output (Screenshots).

Related Issues or Pull Requests number

(Write your answer here.)

please review it.


Tree based explanations

Hey there,
I was thinking about adding Tree based explanations to LIME.  It seems like a lot of the explanations assuming that we're getting linear results.  I was curious how you would tackle refactoring this to make the format of explanations more general as opposed to assuming it was linear.  I was going to add subclasses to explanations.py for TreeExplanations and another for LinearExplanations, which I think is likely uncontroversial.
However, I'm unsure of the preferred way of changing the format for explain_instance in lime_tabular for example.  There are a couple of options

Return dummy intercept and local_exp entries, and augment with additional entries for tree explanation.  The interface stays consistent for old implementation, and new clients have to differentiate the results.
Rather than returning intercept and local_exp as top level entries in the response, nest them under explanation_params which are conditional given the type of explanation that's generated (I don't know if there are plans or desires to support explanations of more than linear + tree types, but I suppose it's possible).  This future proofs it to make it easier to add new types of explanations.  This has the drawback of being a break change, but more consistent for clients implementing the results.

I also thought it might be nice to add some examples longer term, visualizing it, using something like http://bl.ocks.org/pprett/raw/3813537/.
Thoughts?

Yeah, subclasses to explanations.py sounds like the right approach. We should have done this already when regression was added.
Note that lime_tabular returns an explanation.Explanation - as long as linear explanations remain as the default, the interface can remain the same as long as LinearExplanations implements the current functions in the Explanation class. Thus, there is no need to have dummy intercept and local exp.
Before refactoring LIME code, I would suggest trying out some tree explanations. I have tried them multiple times and in multiple ways and they always seem too confusing, and hard for the user to parse. I have yet to see a case where one would prefer them (I'm happy to be shown otherwise).

What if you built a decision tree visualization using the max_depth and max_features and the top N( where N=max_depth) features? Does that make sense?

It does make sense to use max_depth, or max_leaf_nodes, that is precisely what I did.


How to use proxy with ConnectOverCDPAsync?

Your question
Hello:
I can use the following C# code to connect with running Chrome browser:
private void Form1_Load(object sender, EventArgs e)
{
    Process proc = new();
    proc.StartInfo.FileName = Chrome_Binary;
    string remote_chrome_browser = string.Format("{0} {1}",
        "https://bing.com/", Chrome_Running_Parameter);
    proc.StartInfo.Arguments = remote_chrome_browser;
    proc.Start();
}

private async Task BTN_ConnectChrome()
{
    int Port = 9222;
    using HttpClient httpClient = new ();
    string? url = null;
    while (url == null)
    {
        try
        {
            var version =
                await httpClient.GetFromJsonAsync<JsonElement>($"http://localhost:{Port}/json/version/");
            url = version.GetProperty("webSocketDebuggerUrl").GetString();
            Debug.Print(url);
        }
        catch
        {
            await Task.Delay(1000);
        }
    }
    using IPlaywright? playwright = await Playwright.CreateAsync();
    await using IBrowser? browser =
    await playwright.Chromium.ConnectOverCDPAsync(url, new() { SlowMo = 100 });
    var page = browser.Contexts[0].Pages[0];
    await page.GotoAsync("https://www.google.com/");
}

But for some web site, I have to use proxy due to geographic limits.
So I want to launch browser to visit https://bing.com/ first, then attach playwright to the running Chrome browser with proxy, which needs user name, password to login to the proxy.  Finally, I want to let Chrome browser to visit https://www.google.com/ with proxy settings.
I can’t find helpful example.
Please advise,
Thanks,

You can either set the proxy when launching the browser, as a CLI argument, see here: --proxy-server.
or when connecting to a the browser via CDP, you create a new context, there you can also pass the proxy options in the context options.

Hello:
Thanks for your reply.  I studied the first advice, but I can't see any user name/password for Windows.  Do I have to input user name, password by hand if I use CLI argument?
For the second advice, do you have any code example or some web page with detailed information?
Thanks,

This is not supported by Chromium, see here: https://chromium.googlesource.com/chromium/src/+/HEAD/net/docs/proxy.md#proxy-credentials-in-manual-proxy-settings
Instead of using CDP, you can use Node.js and browserType.launchServer({proxy: {...}}) which supports credentials. There you can connect using BrowserType.ConnectAsync then see here: https://playwright.dev/docs/api/class-browsertype#browser-type-launch-server

Hello:
Let me know if it is supported by MS edge, if yes, then I can use MS edge, I don't care if I can use Chromium.
Thanks,

yes, you can use any browser there, for edge you can use the edge channel.


.osfcli.config should be checked in parent directories when it is not found in current dir

https://github.com/dib-lab/osf-cli/issues/109#issuecomment-328462903 =>
ls is set as an alias for list, so it should work, or we don't understand how to use argparse. Definitely something to fix.
@theoutlander could you open a new issue regarding only checking the current directory for .osfcli.config. Would indeed be useful to also search "upwards". Though knowing how/when to stop is a bit tricky so best to discuss in a new issue.
The password can't be set in a file, you should be prompted for it when osfclient needs it. You can also set the OSF_PASSWORD environment variable to contain your password which will then be used instead of prompting you for it.

Does anyone have any wisdom how far up the tree it is reasonable to search for a .osfcli.config? Or do tools like git, hg and friends search until they reach /?

I don't know how git or hg do it, but I'd guess that as long as you properly handle the possibility of nested .osfcli.config files, it shouldn't hurt to go all the way up to /, although if there's one in / they probably made a mistake and maybe should be warned.
Is there a use case for having multiple config files, or would we expect that if we're traveling up and we hit one, we won't hit another one if we keep going?

I can't think why there should be multiple ones really. At least as a starting point I would stop as soon as we find the first one and assume that is the one.

Alternatively read at least from $HOME in addition to the current directory.

As far as I know hg and git do not search for config files upwards until /:

hgrc
git-config

Instead they look in specific locations such as: <repo>/.hg/hgrc, $HOME/.hgrc or /etc/mercurial/hgrc.

I believe the .osfcli.config is meant to be project-specific config file rather than a global config file. Haven't used hg, but in git, you can operate on your git repo from anywhere within the directory tree. I.e. you can do
mkdir git-repo
cd git-repo
git init
mkdir -p 1/2/3/4/5
cd 1/2/3/4/5
git status

and the status will tell you about your git repo you initialized in git-repo

Ah, right. Repository directories. Indeed both hg and git search upwards for repository specific .hg or .git directories.
And yes, perhaps a glocal config is not a good idea: too dangerous, may forget a project specific file in $HOME and mess with other projects.


Update postgres log file pattern

@millerdev you'll want this before you deploy to enikshay
@snopoke

deployed to NIC


Search through table join

Hey guys!
Thank you very much for that project, but I wonder if it supports search with join tables.
For example, I've got a Book object, which has_many :business_models
Then each BusinessModel has_many :territories, they have a related table as well, each territory has a code value inside
Basically, I want to write a sphinx_scope, which will allow to do the following:
Book.by_territory('US').search(params[:query])

The issue is that I'm not sure where to find the docs for such join example
I was trying to mimic the AR request, which looks like that
scope :for_territory, -> (territory) { joins(business_models: :territories).where(territories: {code: territory}) }

but no luck so far.
Would really appreciate your help!

Sorted it out :)

What did you do?

What did you do?

that's pretty easy
ThinkingSphinx::Index.define :book, with: :active_record do
  # fields

  indexes book_subscriptions.territories.code, as: :code, type: :string
end


[BUG] https://api.quotable.io/ no longer working

Describe the bug
The plugin is no longer working is the source https://api.quotable.io/ is no longer available.

help same problem here too!!!

Here to report the same issue!

Same here.

Yes, checked and it seems to be an issue on their end. I managed to find a solution. Will update ASAP

Thought the solution would be easier. It seems the quotable API is down.
I'm working on creating a new api that would perform the same tasks.
WIP

Issue resolved, replaced by my own enhanced api


feat: fuels cli

What did you understand about all these commands?

I think I understand what's described in the screenshot itself. e.g., fuels deploy deploys the contract onto a targeted network. Isn't that the case?

Yes, but there is so much more to it. My suspicion is based on the fact that you suggested keeping init and dev but are concerned with the rest when, in reality, these two are there to support the rest and are useless without them.
The fact is that this screenshot is a poor placeholder while I finish the docs and add a proper description; it was my miserable attempt to keep @luizstacio from going hopeless about all his efforts in this direction from almost one year ago. Meanwhile, you can check his demo video for a more in-depth explanation; it is outdated but gives a good idea of the whole.
I am confident that we will inevitably use this whole CLI in one way or another very soon because the scripts that Estacio brought to the table provide the best possible experience for developing Fuel dApps, and that's not even an exaggeration.
Please allow me to conclude the final little bits and tests and finish the docs; everything will become clearer.
Then, we can also have an alignment call to ensure we're all on the same page.
As usual, thank you for your careful input and attention to detail.

Sounds good, then; let me know when it's ready for review.
It does look like a good experience for building dapps on fuel in general.
I only bring these issues up because forc, that's managed by a different team that doesn't have this context here, is trying to build something similar. This (natural) lack of communication between teams is sometimes the reason we end up with similar products with different names, causing bloat throughout the full-stack experience and confusing users. Let's just make sure to have this alignment chat to figure out the best, most vertically integrated way to release this tool!

@arboleya when installing this PR locally, I had to use pnpm add<EMAIL_ADDRESS>

@danielbate I agree; the config is noisy. I'll trim it down.
You're right—template presets for scaffolding new projects will be handled by https://github.com/FuelLabs/fuels-ts/issues/1330.
I believe sway-applications could be refactored to have a fues.config.ts and use the CLI.
That doesn't mean we couldn't consider having two different options:

-t, --template — npm create fuels@latest -t quickstart
-d, --demo — npm create fuels@latest -d sway-swap

We must focus on recommending the CLI/fuels.config.ts usage but I'd keep the fuels typegen command because it offers more flexibility for potentially less conventional setups and further user customizations.

@danielbate I brought the config file to a minimum, this is what you'll see now:
https://github.com/FuelLabs/fuels-ts/blob/59c3a42b0b3492b50b77d7c644f84dc4fd551e54/apps/demo-fuels/fuels.config.ts#L1-L11

@arboleya I am having this issue when trying to run pnpm fuels dev / npx fuels dev. The same error happens when executing build.
I am using fuels: 0.0.0-pr-1291-20231023140201.

@arboleya I am having this issue when trying to run pnpm fuels dev[...] with 0.0.0-pr-1291-20231023140201

@Torres-ssf I can't reproduce it with the last version: 0.0.0-pr-1291-20231023200800
Could you please try again and let me know if it worked?

@Torres-ssf I can't reproduce it with the last version: 0.0.0-pr-1291-20231023200800
Could you please try again and let me know if it worked?

@arboleya I am still having trouble with build and dev and deploy, init working well now.
Still displaying the following message:
$ pnpm fuels dev                                                                                [11:51:50]
node:internal/streams/writable:287
  const state = stream._writableState;
                       ^

TypeError: Cannot read properties of undefined (reading '_writableState')
    at _write (node:internal/streams/writable:287:24)
    at Writable.write (node:internal/streams/writable:337:10)

Node.js v18.16.0
FAIL

Only issue with init, still displaying the complete error message above as well as Error: Workspace not supplied., may be good to trim the actual error? Also provide info on how to supply the workspace?


NBC shows requiring authentication fail with HTTP 401 when using Comcast_SSO adobepass

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.09.01 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[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
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
United States - Eastern Pennsylvania
Provide a description that is worded well enough to be understood
NBC shows requiring authentication fail with HTTP 401 when using --ap-mso Comcast_SSO (Xfinity as linked provider)
Some example URLs that fail include:
https://www.nbc.com/the-real-housewives-of-beverly-hills/video/we-need-to-talk-about-kathy/9000249196
https://www.nbc.com/saturday-night-live/video/october-1-miles-teller/9000258299
https://www.nbc.com/below-deck-down-under/video/somethings-fishy/9000265442
https://www.nbc.com/dateline-unforgettable/video/under-the-desert-sky/9000256501
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
debug] Command-line config: ['-vU', '--ap-mso', 'Comcast_SSO', '--ap-username', 'PRIVATE', '--ap-password', 'PRIVATE', 'https://www.nbc.com/dateline-unforgettable/video/under-the-desert-sky/9000256501']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.09.01 [5d7c7d6] (zip)
[debug] Python 3.10.4 (CPython 64bit) - Linux-5.15.0-46-lowlatency-x86_64-with-glibc2.35 (glibc 2.35)
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2020.06.20, secretstorage-3.3.1, sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1670 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.09.01, Current version: 2022.09.01
yt-dlp is up to date (2022.09.01)
[debug] [NBC] Extracting URL: https://www.nbc.com/dateline-unforgettable/video/under-the-desert-sky/9000256501
[NBC]<PHONE_NUMBER>: Downloading JSON metadata
[NBC]<PHONE_NUMBER>: Downloading JSON metadata
[NBC]<PHONE_NUMBER>: Downloading Provider Redirect Page
[NBC]<PHONE_NUMBER>: Logging in
[NBC]<PHONE_NUMBER>: Retrieving Session
ERROR: [NBC]<PHONE_NUMBER>: Unable to download webpage: HTTP Error 401:  (caused by <HTTPError 401: ''>); 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 "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 670, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/nbc.py", line 142, in _real_extract
    query['auth'] = self._extract_mvpd_auth(
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/adobepass.py", line 1715, in _extract_mvpd_auth
    session = self._download_webpage(
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 1079, in _download_webpage
    return self.__download_webpage(url_or_request, video_id, note, errnote, None, fatal, *args, **kwargs)
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 1030, in download_content
    res = getattr(self, download_handle.__name__)(url_or_request, video_id, **kwargs)
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/adobepass.py", line 1361, in _download_webpage_handle
    return super(AdobePassIE, self)._download_webpage_handle(
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 864, in _download_webpage_handle
    urlh = self._request_webpage(url_or_request, video_id, note, errnote, fatal, data=data, headers=headers, query=query, expected_status=expected_status)
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 821, in _request_webpage
    raise ExtractorError(errmsg, cause=err)

  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 803, in _request_webpage
    return self._downloader.urlopen(self._create_request(url_or_request, data, headers, query))
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3656, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.10/urllib/request.py", line 525, in open
    response = meth(req, response)
  File "/usr/lib/python3.10/urllib/request.py", line 634, in http_response
    response = self.parent.error(
  File "/usr/lib/python3.10/urllib/request.py", line 563, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.10/urllib/request.py", line 496, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.10/urllib/request.py", line 643, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 401:

@wxolp I believe this is different, since that issue did not involve SSO, and my issue with the previously mentioned URLs still exists.

The URLs you mentioned not need SSO:
POST http://access-cloudpath.media.nbcuni.com/access/vod/nbcuniversal/9000256501
GET https://east.manifest.na.theplatform.com/m/HNK2IC/IeSM1ihtyGA8,h9L82AiZfyLP,a3QSdBW04gyi,1jj6HdGW8QIv,vZNbnH3FnqLY,jFy2kU89d6If.m3u8?sid=a57521c0-3b9b-4bd8-8551-df13ec5f1dee&policy=188569627&date=1667051218171&ip=<IP_ADDRESS>&schema=1.1&manifest=M3U&tracking=true&switch=HLSServiceSecure&am_sdkv=null&p2=null&_fw_did=&nw=169843&f1=&am_extmp=default&uuid=optout-919864457317695&am_appv=null&mode=on-demand&uoo=1&sfid=9244572&s3=null&am_buildv=null&am_abvrtd=null&s4=poc&_fw_ae=&debug=false&csid=oneapp_phone_android_app_ondemand&metr=1023&am_cpsv=4.0.0-2&bundleId=&userAgent=Go-http-client%2F1.1&e1=default&afid=200265138&prof=nbcu_android_cts_bl&c3=null&a2=4.0.0-2&a3=null&a5=null&a6=0&am_crmid=null&rdid=android&am_playerv=null&did=optout&am_stitcherv=poc&am_abtestid=null&sig=369ecdf2d629673ac604902147be3ef9e72b8fbc611ec92aa71c76f88ec70178
GET https://east.manifest.na.theplatform.com/m/HNK2IC/IeSM1ihtyGA8,h9L82AiZfyLP,a3QSdBW04gyi,1jj6HdGW8QIv,vZNbnH3FnqLY,jFy2kU89d6If/4.m3u8?sid=a57521c0-3b9b-4bd8-8551-df13ec5f1dee&policy=188569627&date=1667051218576&ip=<IP_ADDRESS>&schema=1.1&cid=a7dc58df-2da9-4122-9852-b6153f9e1c5a&host=vod-lf-oneapp-prd.akamaized.net&meta=false&manifest=M3U&switch=HLSServiceSecure&am_sdkv=null&p2=null&_fw_did=&nw=169843&f1=&am_extmp=default&tracking=true&uuid=optout-919864457317695&am_appv=null&mode=on-demand&uoo=1&sfid=9244572&s3=null&am_buildv=null&am_abvrtd=null&s4=poc&_fw_ae=&csid=oneapp_phone_android_app_ondemand&metr=1023&am_cpsv=4.0.0-2&bundleId=&userAgent=Go-http-client%2F1.1&e1=default&afid=200265138&prof=nbcu_android_cts_bl&c3=null&a2=4.0.0-2&a3=null&a5=null&a6=0&am_crmid=null&rdid=android&am_playerv=null&did=optout&am_stitcherv=poc&am_abtestid=null&sig=25d85f429b2e3f4ceb99c330387f5a823306dd424849a9db30a74d891f95ffad
1.04%   19.84 megabyte   19.81 megabyte/s

I dont understand this response.  My problem as documented here: https://github.com/yt-dlp/yt-dlp/issues/5123
still exists with the listed example URLs:
https://www.nbc.com/the-real-housewives-of-beverly-hills/video/we-need-to-talk-about-kathy/9000249196
https://www.nbc.com/saturday-night-live/video/october-1-miles-teller/9000258299
https://www.nbc.com/below-deck-down-under/video/somethings-fishy/9000265442
https://www.nbc.com/dateline-unforgettable/video/under-the-desert-sky/9000256501

see #5742 for a possible workaround

Possibly same issue as https://github.com/yt-dlp/yt-dlp/issues/4038, https://github.com/yt-dlp/yt-dlp/issues/516

Closing as dupe of https://github.com/yt-dlp/yt-dlp/issues/516. Try the posted workaround and let us know it it doesn't work


Use "@/" imports

Update tsconfig.json and update imports everywhere needed, for example:
- import { ViewsCounter } from "components/ViewsCounter";
+ import { ViewsCounter } from "@/components/ViewsCounter";

Makes the import ever-so-slightly-more explicit 😄

Can I take this one @sreetamdas under hacktoberfest

Hey, @AnirudhPanda  sorry to say this but already been working on it and am almost done!

Hey, @AnirudhPanda sorry to say this but already been working on it and am almost done!

I thought good practice is we should work only when assigned!
Nevermind.

Umm had a word with sreetam about this long back just forgot to update it here sorry for the inconvenience.

@AnirudhPanda sorry the delay and misunderstanding, I've been travelling :D
I'll be waiting a bit longer to open the repo for contributions towards hacktoberfest for a couple of reasons, but please don't let that stop you from working on the issues :)
@vvaibhavdesai sorry but to clarify I'd said: "Yeah I think it's best to atleast mention that you're working on it — but I'll accept whichever PR comes in first to close the issue :)"
So in this case, @AnirudhPanda I'll gladly accept your PR as well — there's also more issues that you can work on, possibly even together :D

Closed with https://github.com/sreetamdas/sreetamdas.com/commit/30594b9b7ab69c01d7cd8a351a5e5c606256399c


Updated symfony/http-foundation dependency

Hi.
I've changed symfony/http-foundation dependency which now allows using any new symfony/http-foundation version up to (but not including) 3.0 - previous dependency was limited by 2.3.x version, so allure couldn't be used with modern projects using fresh http-foundation versions. Semantic versioning (major.minor.patch with no backwards-incompatible changes between major versions) is adopted as a standard in composer-based projects, and Symfony is a leading framework which doesn't break such rules, so it should be completely safe (Allure uses it only for mime-type guessing, and i guess this interface won't break even in 3.0).
Tests were ok with http-foundation v2.6.3.

Thank you for contribution.


Missing Instructions for Windows Admin Center on Server Core

This article instructs users to launch the Windows Admin Center from the Start Menu after rebooting. It should also include how to start it from the command line so it can be used on Windows Server Core environments.

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

ID: f168bf93-cc54-2a7e-2c1c-7c649a8458d2
Version Independent ID: 0edf20e5-268c-b5df-93a6-85186eb4ba86
Content: Windows Admin Center Common Troubleshooting Steps
Content Source: WindowsServerDocs/manage/windows-admin-center/support/troubleshooting.md
Product: windows-server
Technology: windows-admin-center
GitHub Login: @jwwool
Microsoft Alias: jeffrew

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Unexposed vcd client retry parameters

While we were coding the previous PR we noticed that the retry parameters in the VCD client were not exposed in the cpi job spec (retry_max, retry_delay).
We thought about making some changes to make these values configurable as right now they are hard-coded (10, 20), but we were confused by some parts of the code. There seem to be nested retries (one level in cloud.rb, the other one the send_request method in vcd_client.rb). There are comments in the code that suggest that those parts of the cpi are going to be refactored.
In order to optimise our BOSH configuration and see what settings result in the quickest successful deploy, we would like to play around with these values.

Can you please provide stack traces you have seen when actions time out?
Sent from my iPhone

On Sep 7, 2015, at 9:35 AM, Bence Monus<EMAIL_ADDRESS>wrote:
While we were coding the previous PR we noticed that the retry parameters in the VCD client were not exposed in the cpi job spec (retry_max, retry_delay).
We thought about making some changes to make these values configurable as right now they are hard-coded (10, 20), but we were confused by some parts of the code. There seem to be nested retries (one level in cloud.rb, the other one the send_request method in vcd_client.rb). There are comments in the code that suggest that those parts of the cpi are going to be refactored.
In order to optimise our BOSH configuration and see what settings result in the quickest successful deploy, we would like to play around with these values.
—
Reply to this email directly or view it on GitHub.


Java files are hidden

After installing this plugin I need to update java file for more ease but there is no java file on display against different folders.. WHY ?

Normally I would not respond if I see WTF, but I feel you frustration here.
What you marked yellow is actually quote from https://developer.android.com/topic/libraries/support-library/setup.html

The support libraries are now available through Google's Maven repository. We no longer support downloading the libraries through the SDK Manager, and that functionality will be removed soon..

The requirements table now just show default support library plugin will use by default.


[Storage] Fix unstable access policy SAS cases.

According to https://docs.microsoft.com/en-us/rest/api/storageservices/set-container-acl and https://docs.microsoft.com/en-us/rest/api/storageservices/set-share-acl, "When you establish a stored access policy on a container, it may take up to 30 seconds to take effect. During this interval, a shared access signature that is associated with the stored access policy will fail with status code 403 (Forbidden), until the access policy becomes active.".
This PR fix the unstable access policy accordingly.

Is it possible to check whether it's ready every seconds to stop earlier than 30 seconds?

Could we check whether it's ready every seconds to stop earlier than waiting for 30 seconds everytime?

That's much better, @ljian3377 !
Something like below.
export async function checkWithTimeout(
  predicate: () => boolean,
  delayBetweenRetriesInMilliseconds: number = 1000,
  maxWaitTimeInMilliseconds: number = 10000
): Promise<boolean> {
  const maxTime = Date.now() + maxWaitTimeInMilliseconds;
  while (Date.now() < maxTime) {
    if (predicate()) return true;
    await delay(delayBetweenRetriesInMilliseconds);
  }
  return false;
}

Reference - HarshaNalluru/servicebus/data-plane#L392-L403

Thanks for review! Polling is one option for live test, while might be a problem for recording\playback, as random count of requests might be sent during recording which depends on service side cache status, so I chose a stable pass mode. @HarshaNalluru @ljian3377 and @jeremymeng.

I like polling, but I can also see it's valid to argument about more requests thus more cost.
If we go with straight delay, could you please add some comments before the delay calls?

Have similar problem in datalake.

1) DataLakeServiceClient
       getUserDelegationKey should work:
     RestError: <?xml version="1.0" encoding="utf-8"?><Error>AuthenticationFailed<Message>Server failed to authenticate
 the request. Make sure the value of Authorization header is formed correctly including the signature.
RequestId:6300fec9-701e-0088-08df-f6fd47000000
Time:2020-03-10T13:26:14.2039858Z</Message><AuthenticationErrorDetail>Signature validation failed. Signature key not found.</AuthenticationErrorDetail></Error>
      at new RestError (C:\Users\VMadmin\src\azure-sdk-for-js\sdk\core\core-http\src\restError.ts:27:5)
      at C:\Users\VMadmin\src\azure-sdk-for-js\sdk\core\core-http\src\policies\deserializationPolicy.ts:176:21
      at processTicksAndRejections (internal/process/task_queues.js:97:5)

  2) Shared Access Signature (SAS) generation Node.js only
       GenerateUserDelegationSAS should work for filesystem with all configurations:
     RestError: <?xml version="1.0" encoding="utf-8"?><Error>AuthenticationFailed<Message>Server failed to authenticate
 the request. Make sure the value of Authorization header is formed correctly including the signature.
RequestId:6302d094-701e-0088-08df-f6fd47000000
Time:2020-03-10T13:28:58.0870659Z</Message><AuthenticationErrorDetail>Signature validation failed. Signature key not found.</AuthenticationErrorDetail></Error>
      at new RestError (C:\Users\VMadmin\src\azure-sdk-for-js\sdk\core\core-http\src\restError.ts:27:5)
      at C:\Users\VMadmin\src\azure-sdk-for-js\sdk\core\core-http\src\policies\deserializationPolicy.ts:176:21
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:97:5)

  3) Shared Access Signature (SAS) generation Node.js only
       GenerateUserDelegationSAS should work for filesystem with minimum parameters:
     RestError: <?xml version="1.0" encoding="utf-8"?><Error>AuthenticationFailed<Message>Server failed to authenticate
 the request. Make sure the value of Authorization header is formed correctly including the signature.
RequestId:832a36a7-e01e-0063-1edf-f603bb000000
Time:2020-03-10T13:28:58.1123473Z</Message><AuthenticationErrorDetail>Signature validation failed. Signature key not found.</AuthenticationErrorDetail></Error>
      at new RestError (C:\Users\VMadmin\src\azure-sdk-for-js\sdk\core\core-http\src\restError.ts:27:5)
      at C:\Users\VMadmin\src\azure-sdk-for-js\sdk\core\core-http\src\policies\deserializationPolicy.ts:176:21
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:97:5)

  4) Shared Access Signature (SAS) generation Node.js only
       GenerateUserDelegationSAS should work for file:
     RestError: <?xml version="1.0" encoding="utf-8"?><Error>AuthenticationFailed<Message>Server failed to authenticate
 the request. Make sure the value of Authorization header is formed correctly including the signature.
RequestId:b606dcd2-e01e-000e-26df-f6a995000000
Time:2020-03-10T13:28:58.1443357Z</Message><AuthenticationErrorDetail>Signature validation failed. Signature key not found.</AuthenticationErrorDetail></Error>
      at new RestError (C:\Users\VMadmin\src\azure-sdk-for-js\sdk\core\core-http\src\restError.ts:27:5)
      at C:\Users\VMadmin\src\azure-sdk-for-js\sdk\core\core-http\src\policies\deserializationPolicy.ts:176:21
      at runMicrotaks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:97:5)


waiting for selector "#progressIndicator" failed: timeout 30000ms exceeded

for almost 8 month I was able to download from SAP learning hub. But now it is for me not possible I got the following error.
Entering login: SXXXXXXXXX
Entering password: ***
Submiting...
Redirected to: https://account.sap.com/core/login-as/autologin?reqId=https%253A%252F%252Fwww.successfactors.eu%252Flearninghub&samlContext=eu1_184876214373_b61e62bd-47f0-4b03-a6bf-bd8a432cf1fb
Connected :)
Initialize file
URL Template: https://saplearninghub.plateau.com/icontent_e/CUSTOM_eu/sap/self-managed/Assessment/HATD20_EN_Col17/xml/topic$$$.svg
File path: C:\tmp/HATD20_EN_Col17.pdf
a: waiting for selector "#progressIndicator" failed: timeout 30000ms exceeded
at new u (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:1093139)
at n.DOMWorld._waitForSelectorOrXPath (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:1091809)
at n.DOMWorld.waitForSelector (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:1091208)
at l.waitForSelector (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:848085)
at l. (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:7115)
at x.waitForSelector (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:85644)
at c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:20:842165
at u (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:1035204)
at Generator._invoke (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:1034957)
at Generator.next (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:1035567)
-- ASYNC --
at l. (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:7092)
at x.waitForSelector (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:85644)
at c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:20:842165
at u (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:1035204)
at Generator._invoke (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:1034957)
at Generator.next (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:1035567)
at t (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:13931)
at c (c:\Users\Rudi\Downloads\saplearninghub-pdf-downloader-master\dist\index.js:1:14142)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
Whenn using the --debug=true. Every thing seems to be ok, because, the browser started, the login ist correct and the HATD20_EN_Col17 will open. After a while I get the Info time execeeded. And nothing happens.
Any Idea
Kind Regards
RKu2201

Same issue. It used to work: maybe it needs to be adjusted


Fix overwriting of dataset in erroranalysis

Description
Fix overwriting of dataset in erroranalysis
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #2050 (f337733) into main (553f64f) will increase coverage by 2.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2050      +/-   ##
==========================================
+ Coverage   92.33%   94.41%   +2.08%
==========================================
  Files          98       18      -80
  Lines        4957      412    -4545
==========================================
- Hits         4577      389    -4188
+ Misses        380       23     -357

Flag
Coverage Δ

unittests
94.41% <ø> (+2.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 80 files with indirect coverage changes


Various improves on flex-grid

Add flex-grid wrap option
Add a flex-grid-row wrap option, allowing to create a row which wrap (by default) or not.
Use "expand" explicit value for default flex-grid-row size
I often have in my semantic Scss many column definition aligned, and I think its behaviours could be more understandable with an explicit value (like expand) for columns which take the remaining place, instead of null.
expand would be always the default behaviour
Something like this, similar to what foundation-apps purpose :
.my-row {
  .col-a { @include flex-grid-column(1); }
  .col-b { @include flex-grid-column(expand); }
  .col-c { @include flex-grid-column(shrink); }
}

Instead of
.my-row {
  .col-a { @include flex-grid-column(1); }
  .col-b { @include flex-grid-column; }
  .col-c { @include flex-grid-column(shrink); }
}

@andycochran @brettsmason Can you take a look at this?  Do you think this change of defaults makes sense?

ping @andycochran @brettsmason thoughts on this change of defaults?

I've personally never used the flex grid as I need to support ie9 for my projects, so I'm not familiar with it. I'll try and take a look but hopefully @andycochran will have more knowledge on this.

The changes seem logical, but I too have little experience with Flex Grid (also thanks to ie9). @kball, do we have a Yetinaut with flexbox expertise? @JeremyEnglert?
My one question is the change from null to expand. Is this just semantics? What happens when it's left null—e.g. if I use this mixin:
.col-b { @include flex-grid-column; }

@andycochran Yes, it is only semantical.
With this PR, we have :
// .col-a is equal to .col-b
.col-a { @include flex-grid-column(expand); }
.col-b { @include flex-grid-column; }

// but .col-c is not accepted
.col-b { @include flex-grid-column(null); } // because a "null" value is meaningless

Should we accept when a null value is explicitly passed to the mixin ?

If a null value can currently be passed to the mixin, we shouldn't drop support for that. Don't want to break apps.

@andycochran Fixed (and rebased)

@andycochran can you take a look at this and merge if it seems good to you?

Having used flexbox and flex grid more now, digging into this it makes perfect sense and I think is good to go.


tumangaonline source is broken

It does not load any manga, the reload button appears in the center of the screen but does not do anything.

Same. You can browse and search, no issues, but you can’t open any manga. First it was reported that this happened only with manwha, but now it doesn’t matter what manga you try to open, it’s always a blank screen with the reload button.

as far as I could check, the images can't be displayed now because of some nginx rule, apparently it's because of the "Referer".
curl without referer:
curl https://img1.japanreader.com/uploads/20231114/4400daa5e7840d55a216c7cda7c1c95b/7b2f084e.webp -I
HTTP/1.1 403 Forbidden
Server: nginx
Date: Wed, 15 Nov 2023 16:07:01 GMT
Content-Type: text/html
Content-Length: 146
Connection: keep-alive
Vary: Accept-Encoding
Strict-Transport-Security: max-age=31536000

curl with Referer:
curl 'https://img1.japanreader.com/uploads/20231114/4400daa5e7840d55a216c7cda7c1c95b/7b2f084e.webp' \
  -H 'Referer: https://visortmo.com/viewer/4400daa5e7840d55a216c7cda7c1c95b/paginated/2' \
  -H 'User-Agent: Mozilla/5.0 (iPhone; CPU iPhone OS 13_2_3 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/13.0.3 Mobile/15E148 Safari/604.1' -I
HTTP/1.1 200 OK
Server: nginx
Date: Wed, 15 Nov 2023 16:07:43 GMT
Content-Type: image/webp
Content-Length: 377712
Connection: keep-alive
Last-Modified: Tue, 14 Nov 2023 15:31:36 GMT
ETag: "655392d8-5c370"
Image: s204
Strict-Transport-Security: max-age=31536000
X-Frame-Options: SAMEORIGIN
X-Content-Type-Options: nosniff
X-XSS-Protection: 1; mode=block
Cache-Control: public, max-age=31536000, s-maxage=31536000
X-Cache-Status: MISS
Accept-Ranges: bytes