Via: revert change to protocol element of VIA string.

The protocol stack changes to the VIA string are broken and can't really be fixed. Such support will have to be provided by a plugin. This reverts the VIA string changes from #1534. This replaces #1988.

When you said, "the changes to the VIA string are broken", do you mean the ipv6 and response via problem you mentioned in #1988 ?
I think we should fix those problems instead of reverting it back.

That was one aspect. #2024 fixes that. The more significant issues are

Without format flexibility this change cannot actually be deployed in a real CDN. The CDN clients will never upgrade in synchronization. Therefore without the additional configuration support from #1998 it basically breaks ATS for CDN use.
Everyone seemed to agree this should be done with a plugin, and not in the core. This PR removes it from the core so it can be done by a plugin.

I see your point. can you close this and reopen #1988 ?

Well, fwiw, being the primary whiner on #1988, I'll leave it to amc and shukitchan to decide. I'm ok with #1988.

#1988 was chosen instead of this.


IDSEQ-913 Add error message when min contig size is too small.

This reduces confusion for the user. Right now, when the user types an invalid value, the value resets to the old one, but there is no error message to explain what is going on.

idseq_web doesn't load contigs less than 4 from the pipeline. (See db_load_contigs). I believe this was a choice by YF because due to paired reads, a contig of just 2 reads could just be the two paired reads assembled together. So a contig with less than 4 reads is very unreliable.


chore: bdd-nh-1587754056 to 0.0.1

chore: Promote bdd-nh-1587754056 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[Issue]: Enclosed Telescope Tune-Up

Description of the bug/crash
The game crashes when activating the mod. Tried to force update the mod is installed version 2.1.3 from December 1.
Exception in RegisterBuilding for type ClusterTelescopeEnclosedConfig from Assembly-CSharp
System.NullReferenceException: Object reference not set to an instance of an object
at LargeTelescope.LargeTelescopePatches+ClusterTelescopeEnclosedConfig_ConfigureBuildingTemplate.Postfix (UnityEngine.GameObject go) [0x00047] in <8a308d8e88034754aa244644601aa562>:0
at (wrapper dynamic-method) ClusterTelescopeEnclosedConfig.ClusterTelescopeEnclosedConfig.ConfigureBuildingTemplate_Patch1(ClusterTelescopeEnclosedConfig,UnityEngine.GameObject,Tag)
at (wrapper dynamic-method) BuildingConfigManager.BuildingConfigManager.RegisterBuilding_Patch0(BuildingConfigManager,IBuildingConfig)
at (wrapper dynamic-method) GeneratedBuildings.GeneratedBuildings.LoadGeneratedBuildings_Patch67(System.Collections.Generic.List`1<System.Type>)
at UnityEngine.Debug.LogError (System.Object message, UnityEngine.Object context) [0x00000] in <82f243aab18c4f32918da2df41974365>:0
at Debug.LogError (System.Object obj, UnityEngine.Object context) [0x00000] in <82d98ed04cc642f199bc654910617431>:0
at DebugUtil.LogErrorArgs (UnityEngine.Object context, System.Object[] objs) [0x00000] in <82d98ed04cc642f199bc654910617431>:0
at DebugUtil.LogException (UnityEngine.Object context, System.String errorMessage, System.Exception e) [0x00000] in <82d98ed04cc642f199bc654910617431>:0
at GeneratedBuildings.GeneratedBuildings.LoadGeneratedBuildings_Patch67 (System.Collections.Generic.List1[T] ) [0x00000] in <05233252dc2f4934b51fa8fcc19aaaed>:0    at LegacyModMain.LoadBuildings (System.Collections.Generic.List1[T] types) [0x00000] in <05233252dc2f4934b51fa8fcc19aaaed>:0
at LegacyModMain.Load () [0x00000] in <05233252dc2f4934b51fa8fcc19aaaed>:0
at Assets.Assets.CreatePrefabs_Patch2 (Assets ) [0x00000] in <05233252dc2f4934b51fa8fcc19aaaed>:0
at Assets.Assets.OnPrefabInit_Patch12 (Assets ) [0x00000] in <05233252dc2f4934b51fa8fcc19aaaed>:0
at KMonoBehaviour.InitializeComponent () [0x00000] in <82d98ed04cc642f199bc654910617431>:0
at KMonoBehaviour.Awake () [0x00000] in <82d98ed04cc642f199bc654910617431>:0
at UnityEngine.Object.Internal_InstantiateSingleWithParent_Injected (UnityEngine.Object , UnityEngine.Transform , UnityEngine.Vector3& , UnityEngine.Quaternion& ) [0x00000] in <82f243aab18c4f32918da2df41974365>:0
at UnityEngine.Object.Internal_InstantiateSingleWithParent (UnityEngine.Object data, UnityEngine.Transform parent, UnityEngine.Vector3 pos, UnityEngine.Quaternion rot) [0x00000] in <82f243aab18c4f32918da2df41974365>:0
at UnityEngine.Object.Instantiate (UnityEngine.Object original, UnityEngine.Vector3 position, UnityEngine.Quaternion rotation, UnityEngine.Transform parent) [0x00000] in <82f243aab18c4f32918da2df41974365>:0
at UnityEngine.Object.Instantiate[T] (T original, UnityEngine.Vector3 position, UnityEngine.Quaternion rotation, UnityEngine.Transform parent) [0x00000] in <82f243aab18c4f32918da2df41974365>:0
at Util.KInstantiate (UnityEngine.GameObject original, UnityEngine.Vector3 position, UnityEngine.Quaternion rotation, UnityEngine.GameObject parent, System.String name, System.Boolean initialize_id, System.Int32 gameLayer) [0x00000] in <82d98ed04cc642f199bc654910617431>:0
at Util.KInstantiate (UnityEngine.GameObject original, UnityEngine.GameObject parent, System.String name) [0x00000] in <82d98ed04cc642f199bc654910617431>:0
at LaunchInitializer.Update () [0x00000] in <05233252dc2f4934b51fa8fcc19aaaed>:0
Build: U54-646687-SCR
Steps To Reproduce
Activate mod
Restart the game
Crash
Player.log
Player.log

Click on this button next to the mod to force the update it.

Today I updated this mod forcibly several times and when it was activated the game crashed, now it doesn't crash although the same version 2.1.3 is installed. A few hours ago I got a small update of the game in Steam, maybe that's why the game stopped crashing with this mod.

the current version of the mod is 2.1.4
no idea why it's not being updated for you.
try this, it will reinstall all mods.
https://github.com/aki-art/ONI-Mods/wiki/Download-Failed-error

I have the same did not update one mod from Sgt_Imalas, I tried some options with him, but in the end the problem was on his side, he accidentally targeted the folder of the previous version on uploading.

Never mind I now ran the game and the version of the mod became 2.1.4 perhaps the truth was some kind of failure.


App not generating backup file

Hello,
I'm running the DeFi wallet on Linux, but the founds doesn't show up. Seems lost after your fork update.
Please could you advise?

the above is solved. But the App is not generating a backuf file when I want to backup. Any ideas?

Hi @Folderguion , thanks for your feedback, may I know which version are you on? Do you have a screenshot on how you try to save backup?

Hello! Tnx for your help.
I'm on the version: 2.1.17 .
I just click on Wallet menu > Backup wallet > the folders explorer open up but there's no file to save. The only option is to press on Cancel on the explorer window.
The same happen when I go to Logs > any menu the same, no files generated.
I've gone through the post you linked but I don't understand just the very first step :" unlock the wallet with 'walletpassphrase PASSPHRASE 60' " > How to do this?
Thanks

Ok, as per your indication I've posted on this thread > https://github.com/DeFiCh/app/issues/461


gk-deploy: mention required kernel modules

Signed-off-by: Jose A. Rivera<EMAIL_ADDRESS>

This change is

LGTM.


Incremental harvests shouldn't put all documents on "pending"

Not all documents are set on pending. Only the previously failed documents and this is correct. However there is nothing in harvest_source that retries these documents and they don't come in through the seeders by definition so we end up with pending documents that block progress.
Need to add retries to harvest_source and rewrite the tests to catch this case.


Cannot start local generator instance

When trying to start a local node instance running generator, I am met with this:
node app.js --debug -v -f ../plugins [info:core 13:06:18.166 generator.js:103:17] Launching with config: {} Communication error: 2 [warning:core 13:06:18.181 generator.js:131:25] photoshop communications error: {"error":"Photoshop communication error: 2"} Communication error: 2 [warning:core 13:06:18.182 generator.js:131:25] photoshop communications error: {"error":"Photoshop communication error: 2"} [info:core 13:06:18.184 generator.js:117:25] Photoshop connection closed Exiting with code 0: Generator close event
I am running Node 4.4.0. I am blocked on all development for the updated Apple Layered Image Exporter plugin due to a needed fix in 3.1.1 of generator-core, which has still not been released.
I filed against the issue(about 9 months ago) that was fixed in 3.11 but it was removed from the GitHub issue system..Also the commit where it was fixed by an Adobe developer has also been removed, making it appear the issue never existed! Why?!?! I was never notified it was fixed, and it is STILL broken in CC2017, even though the code has been fixed in the repo...
https://github.com/adobe-photoshop/generator-core/commit/192be7e789f1a787cda891b48bbb9e0f61c16733#diff-8317c373a0e944920aed59122ba0938b
Line 216 *actually originally read as "log.log()" but was corrected to say "logger.log()".
So what happened to this commit? I still see:
[0m[1;31mExiting with code -1: Uncaught exception: Cannot read property 'log' of undefined
In my OS X console logs when trying to use Flite in the most up to date Photoshop.

@pixels4nickels the master branch of generator will not run on Photoshop 2017.  See Forum Post.  That is the cause of your "Communication Error" noted above.
I do not understand the second half of your issue report.  Generator version 3.8.4 is included in PS 2017.  Are you saying there is a bug in that version that has since been fixed?

Thanks for that @mcilroyc! That will help me continue development...whew.
The previous issue I spoke of was 9 months ago and has been fixed in the codebase, yet it still happens in CC2017. The logger argument at one time was referred to as log and caused an exception. It is still happening even though I see in 3.11 the usage of the logger argument was corrected. It is really, really strange. If I set useFlite to true, it blows up on that same logging statement. Thanks so much for the quick response!

Have you solved this problem? I also encountered the same problem.


Separate logging threads for stdout and stderr

Two commits in this PR:

Separate logging threads for each stdout and stderr (log_priority set to DLOG_INFO and DLOG_ERROR respectively)
Code formatting (line breaks)

Changed a base branch to flutter-1.22.6-tizen.


Add support for TP-Link Tapo L900

This is just based on white lighting, as set by the measure script.

What are the supported color modes of this light? You can check that in developer tools.
For the light to be adopted to the library a profile for each supported color mode should be submitted.
I see this light also should have color control. Also a measurement for hs needs to be done in that case.

What are the supported color modes of this light?
supported_color_modes: brightness, hs, onoff

I'm guessing a measure using hs needs to be run in this case?

Btw you don't have to modify manufacturer_device_types and supported_models. These will be automatically generasted after PR is merged.

What are the supported color modes of this light?

supported_color_modes: brightness, hs, onoff
I'm guessing a measure using hs needs to be run in this case?

Yes this light needs brightness and hs. So you only need to run another time for hs.csv.gz

Yes this light needs brightness and hs. So you only need to run another time for hs.csv.gz

Running now, will add to this PR once complete.

Btw alias is not needed when it is exactly the same as the folder name.
The auto discovery logic will first check the folder name, when it doesn't match it will check any of the aliases in model.json

@bramstroker I have added the hs measure csv result to this pull request.

@bramstroker I have added the hs measure csv result to this pull request.

Awesome, everything is ok now. let's merge.

Hmm, I was too early.
I see the validation in CI gives an error invalid color mode combination hs,brightness.
It's because HA docs say if brightness is supported by the light it MUST be the only supported color mode of the light.
So this seems a bug to me in the TP-Link integration.
Probably we can just remove brightness.csv from the PR as it won't be used by HA. But we need to be 100% sure of that.
Could you check what is the active color_mode in developer tools attribute when you control the light? Is it always hs?

If the lights are set to white, then the color mode is brightness, but if it's set to a color, then it's hs.

If the lights are set to white, then the color mode is brightness, but if it's set to a color, then it's hs.

Hmm ok, I think I will loosen the validation check in that case. As this seems to be a legit case in HA.

Is this one... https://www.tp-link.com/au/home-networking/smart-bulb/tapo-l900-10/ or https://www.tp-link.com/au/home-networking/smart-bulb/tapo-l900-5/ ?

Ah good point @CloCkWeRX - the ones I used were 5m.

I did not get any response on HA discord, so I will just remove the validation of combination brightness, hs from the validator.


Is there any plan for edge scheduling of stack deployments?

Hi there!
I have been using portainer for only a couple months now to distribute docker containers using the edge agent to remote managed machines. One thing I have been running into is we typically deploy these releases late night when machines aren't in use.
Question:
Is there any plan for edge scheduling of stack deployments?

I'm not aware of any specific work around this at the moment, but I can flag this one with the team for possible future consideration as a feature.
In the meantime, you could potentially leverage Edge Jobs alongside the Portainer API to do this at the moment, or alternatively, depending on how you deploy your stacks, the automatic update functionality that we added in 2.9.


Print UDP Receive Buffer Size in CLI

When running the CLI, we need to print UDP Receive Buffer Size in the current OS and validate if it doesn't fit the recommended size.

2021/02/22 19:35:09 failed to sufficiently increase receive buffer size (was: 208 kiB, wanted: 2048 kiB, got: 416 kiB). See https://github.com/lucas-clemente/quic-go/wiki/UDP-Receive-Buffer-Size for details.
Code: https://github.com/lucas-clemente/quic-go/blob/b8f07c728cf901da96ad445306a3190a51b2b399/packet_handler_map.go#L84

should provide a readme to indicates how to tune the server for UDP performance for different scenarios, close this issue cause this is not suitable for all.


OidcProvider tag prevent rendering AppRoutes

Issue and Steps to Reproduce
Hello, i might be finding some clues if i open a new issue here. When i run the code, it's got stuck on "loading" page

i had to manually add the "auth" in the address bar for it to redirect to auth page. When i tried to debug and place some breakpoint on it one by one, it seems when the app load the index.tsx, it won't load the AppRoutes when OidcProvider tags are in, am i doing the right things when declaring those? or is there something missing (Already tried to change it from redirect uri from env, it really is redirect the app to the auth page, but it returns invalid request from the backend)
Versions
react-oidc 6.5.10
Screenshots
Configuration code

Error if i change the redirect uri

AppRoutes code

Expected
App start and then redirect to auth page
Actual
Stuck on loading page

Additional Details

Installed packages:

Hi @reggspr , thank you for your issue.
Does your oidc provider provide a .weelknowurl.
You should see an HTTP 404 in the http network and that may be why it stay stuck on the loading page.
If it does not provide that weel know url, you can bypass it by configuration all technical oidc urls. Like 👎

Hello @guillaume-chervet thanks for the response
what should i do to check if the oidc provider provide a wellknowurl? and for additional note, it's stuck not because of 404 error, i thought that as well in the first place, but ruling that out once i know when debugging oidcprovider tag is preventing the AppRoutes to load. Here's the before and after i remove the tag
before:

after:

Thank you @reggspr you right. What is strange is I do not seen any request about well know url.
What happen is you move OidcProvider at the top level of your application? Before QueryClientProvider.

i tried move it on the top level, it doesn't do anything

Did you tried your oidc configuration on the reactoidc demo?

yes, i've tried it but still the same error @guillaume-chervet

@reggspr that very strange.
So with your oidc configuration on the demo you never get any http call to your oidc server from react-oidc library?
I have no idea of what can happen to you.
May you print your configuration (please hide the domain for security reason)?

this is the raw configuration, and don't worry the domain is still in localhost because it's still on debug environment @guillaume-chervet

Thank you @reggspr , which oidc provider do you use?
Thank strange your configuration seem OK.

Which browser do you use?

i use chrome

Thanks for trying to help @guillaume-chervet , i found the core of the problem
i need to add the /auth redirecturi's from duende's identity server so i can set the redirecturi from the configuration to domain/auth

Thank you @reggspr for the feedback :)
Have fun


Support for new network validation protocol(v3)

The parachain core team has been working on some optimizations part of https://github.com/paritytech/roadmap/issues/26 for increasing the number of validators doing consensus work, some changes had been required to the network validation protocol, so a new network validation protocol(v3) will be introduced starting with the upcoming polkadot-v1.6.0.
The implementation is backwards compatible, but the optimisations can not be be enabled untill all the validator switched to polkadot version supporting the new protocol.
Relevant pull requests:

https://github.com/paritytech/polkadot-sdk/pull/1178
https://github.com/paritytech/polkadot-sdk/pull/1191
https://github.com/paritytech/polkadot-sdk/pull/2444

High level description of the changes.
In a nutshell, checking the validity of all assignments and approvals for validating parachain blocks, takes a lot of time, so this optimisation reduce the amount of messages the nodes have to check and forward by merging all assignments from tranche0 in a single message and by opportunistically approving more than one candidate with a single signature.
Polkadot-sdk changes:

A new validation network protocol, to be able
to transmit that more than one candidate was included in the assignment(ApprovalDistributionMessage::Assignments, ApprovalDistributionMessage::Approvals)
Plumbing in approval-distribution, approval-voting, dispute-coordination, so that the new messages are correctly processed and accepted as valid.
Support to translate between v3 and v1/v2 network protocols, so that we can still gossip to nodes with only v1/v2.
Update the approval-voting to produce AssignmentCertKindV2::RelayVRFModuloCompact assignments for tranche0, if the runtime features tells us it is enable.
Update the approval-voting to coalesce opportunistically more than one approval vote, up to the maximum specified in the runtime configuration max_approval_coalesce_count

Let me know if you got questions or any more details are needed.

Heads up, we have initiated the governance process to enable the changes described here ok Kusama: https://kusama.polkassembly.io/referenda/392
CC @kamilsa @iceseer


Error when i put a paper-input inside a paper-drawer-panel

Hi partners!
when it loads the page, the paper-drawer-panel aligned to the right, i should be collapsed, but instead it charges full loaded.
when i call the togglePanel function it bumps and then it midly works.
this only happens when there is a paper-input inside a paper-drawer-panel
<paper-drawer-panel id="dmenu" responsive-width="1980px" right-drawer>
        <div drawer>
            <paper-toolbar>
                Title
            </paper-toolbar>
            <div id="drawerContent">
                <form id="form" action="http://localhost/site" method="post">
                      <paper-input floating-label name="id" label="reporte"></paper-input>
                      <paper-input floating-label name="latitud" label="latitud"></paper-input>
                      <paper-input floating-label name="longitud" label="longitud"></paper-input>
                      <paper-input floating-label name="nombre" label="nombre"></paper-input>
                      <paper-textarea floating-label name="descripcion" label="descripcion"></paper-textarea>
                      <input type="hidden" name="basurero"/>
                      <paper-button raised onclick="submitForm()">Enviar</paper-button>
                </form>
            </div>
        </div>
        <div main>
            <paper-icon-button icon="menu" onclick="document.getElementById('dmenu').togglePanel()"></paper-icon-button>
            hello world
        </div>
    </paper-drawer-panel>

Please, hope you can help me with this

This issue tracker is for the Polymer core library. We'll make sure this gets moved to the appropriate repo.

This issue was moved to PolymerElements/paper-drawer-panel#70


Use JAXBContext.newInstance( String, ClassLoader)

Would it be possible to change
de.kosit.validationtool.impl.ConversionService.initialize(String)
to use
JAXBContext.newInstance(contextPath, ConversionService.class.getClassLoader())
instead of
JAXBContext.newInstance(contextPath)
otherwise I'm getting class loading issues on Java11 in the Equinox OSGi container.

Definitely possible! Changed the code accordingly and will provide a patch release soon.


[Build test fail] make || make mt

Expected Behavior
Not having this:
2154
*** Error in `./build/Cobra': double free or corruption (!prev): 0x0000000000e56970 ***
Makefile:49: recipe for target 'test' failed
make[1]: *** [test] Aborted (core dumped)
make[1]: Leaving directory '/home/maxie/Desktop/Languages/CobraLang'
Makefile:10: recipe for target 'mt' failed
make: *** [mt] Error 2

Actual Behavior
Whenever I run:  make || make mt or even both apart I get this twice (so once for make and once for make mt):
make all
make[1]: Entering directory '/home/maxie/Desktop/Languages/CobraLang'
make buildAll
make[2]: Entering directory '/home/maxie/Desktop/Languages/CobraLang'
make[2]: Nothing to be done for 'buildAll'.
make[2]: Leaving directory '/home/maxie/Desktop/Languages/CobraLang'
make lib
make[2]: Entering directory '/home/maxie/Desktop/Languages/CobraLang'
make[2]: Leaving directory '/home/maxie/Desktop/Languages/CobraLang'
make[1]: Leaving directory '/home/maxie/Desktop/Languages/CobraLang'
make test
make[1]: Entering directory '/home/maxie/Desktop/Languages/CobraLang'
g++  -I ./ test/main.cc -Iinclude build/libcobra.a -o build/Cobra
./build/Cobra
2154
*** Error in `./build/Cobra': double free or corruption (!prev): 0x0000000000e56970 ***
Makefile:49: recipe for target 'test' failed
make[1]: *** [test] Aborted (core dumped)
make[1]: Leaving directory '/home/maxie/Desktop/Languages/CobraLang'
Makefile:10: recipe for target 'mt' failed
make: *** [mt] Error 2
2154
*** Error in `./build/Cobra': double free or corruption (!prev): 0x0000000000e56970 ***
Makefile:49: recipe for target 'test' failed
make[1]: *** [test] Aborted (core dumped)
make[1]: Leaving directory '/home/maxie/Desktop/Languages/CobraLang'
Makefile:10: recipe for target 'mt' failed
make: *** [mt] Error 2
make all
make[1]: Entering directory '/home/maxie/Desktop/Languages/CobraLang'
make buildAll
make[2]: Entering directory '/home/maxie/Desktop/Languages/CobraLang'
make[2]: Nothing to be done for 'buildAll'.
make[2]: Leaving directory '/home/maxie/Desktop/Languages/CobraLang'
make lib
make[2]: Entering directory '/home/maxie/Desktop/Languages/CobraLang'
make[2]: Leaving directory '/home/maxie/Desktop/Languages/CobraLang'
make[1]: Leaving directory '/home/maxie/Desktop/Languages/CobraLang'
make test
make[1]: Entering directory '/home/maxie/Desktop/Languages/CobraLang'
g++  -I ./ test/main.cc -Iinclude build/libcobra.a -o build/Cobra
./build/Cobra
2154
*** Error in `./build/Cobra': double free or corruption (!prev): 0x0000000001638970 ***
Makefile:49: recipe for target 'test' failed
make[1]: *** [test] Aborted (core dumped)
make[1]: Leaving directory '/home/maxie/Desktop/Languages/CobraLang'
Makefile:10: recipe for target 'mt' failed
make: *** [mt] Error 2

Steps to Reproduce the Problem

Clone the repo at the cloned repo of CobraLang (/home/maxie/Desktop/Languages/CobraLang for me)
Run make then make || make mt
See what the terminal display

Specifications

OS: Ubuntu 15.04
Terminal used: gnome-terminal

Solved! In my last update (https://github.com/chaseWillden/CobraLang/commit/00f131a903e0b734422c66d15307c4b454bb9d0f), I increased the default allocation memory size a little. Ubuntu and OSX I guess allocate different sizes in memory. So if you re-pull and make it should work for you now.


Trait with Associated Types

Hello, thank you for the excellent crate. It seems that It does not support Sub trait with associated type. That is, how to cast Super to Sub in the case below?
trait Super{}
trait Sub: Super {
type Item;
}
It would be awesome if you could provide a solution!
Thanks,
AmbitionXiang

Hi!
So note that this crate has quite limited support for generics and associated types. But currently it is possible to get some support with something like:
use cast_trait_object::*;

struct Test;

#[dyn_cast(Sub<Item = Test>)]
trait Super {}
#[dyn_cast(Sub<Item = Test>)]
impl Super for Test {}

trait Sub: Super {
    type Item;
}
impl Sub for Test {
    type Item = Test;
}
#[test]
fn it_works() {
    let a: &dyn Super = &Test;
    let _b: &dyn Sub<Item = Test> = a.dyn_cast::<dyn Sub<Item = Test>>().ok().unwrap();
}

It is possible that the situation could be improved somewhat but I'm not quite sure how much. You could potentially try to implement the DynCast trait yourself for a type and get the convenient dyn_cast methods for free but it might not be worth the trouble.

Hi!
So note that this crate has quite limited support for generics and associated types. But currently it is possible to get some support with something like:
use cast_trait_object::*;

struct Test;

#[dyn_cast(Sub<Item = Test>)]
trait Super {}
#[dyn_cast(Sub<Item = Test>)]
impl Super for Test {}

trait Sub: Super {
    type Item;
}
impl Sub for Test {
    type Item = Test;
}
#[test]
fn it_works() {
    let a: &dyn Super = &Test;
    let _b: &dyn Sub<Item = Test> = a.dyn_cast::<dyn Sub<Item = Test>>().ok().unwrap();
}

It is possible that the situation could be improved somewhat but I'm not quite sure how much. You could potentially try to implement the DynCast trait yourself for a type and get the convenient dyn_cast methods for free but it might not be worth the trouble.

Hi~ Thanks for your tips and I have solved it. I’ll close the issue now.

Hi~ Thanks for your tips and I have solved it. I’ll close the issue now.


Fix publish error (no 'prefix' in str format)

Fixes: ansible/galaxy-dev#163

It's still broken

  |
-- | --
Response payload |
IntegrityError at /api/automation-hub/v3/artifacts/collections/ null value in column "task_id" violates not-null constraint DETAIL:  Failing row contains (null, null, collection_demo, 1.0.26, 1).   Request Method: POST Request URL: http://host.docker.internal:5001/api/automation-hub/v3/artifacts/collections/ Django Version: 2.2.5 Python Executable: /venv/bin/python3.6 Python Version: 3.6.8 Python Path: ['/venv/bin', '/usr/lib64/python36.zip', '/usr/lib64/python3.6', '/usr/lib64/python3.6/lib-dynload', '/venv/lib64/python3.6/site-packages', '/code/galaxy-api', '/venv/lib/python3.6/site-packages'] Server time: Wed, 6 Nov 2019 13:30:17 +0000 Installed Applications: ['django.contrib.admin',  'django.contrib.auth',  'django.contrib.contenttypes',  'django.contrib.messages',  'django.contrib.sessions',  'django.contrib.staticfiles',  'rest_framework',  'django_filters',  'galaxy_api.api',  'galaxy_api.auth'] Installed Middleware: ['django.middleware.security.SecurityMiddleware',  'django.middleware.common.CommonMiddleware',  'django.middleware.csrf.CsrfViewMiddleware',  'django.middleware.clickjacking.XFrameOptionsMiddleware',  'django.contrib.sessions.middleware.SessionMiddleware',  'django.contrib.auth.middleware.AuthenticationMiddleware',  'django.contrib.messages.middleware.MessageMiddleware']   Traceback:  File "/venv/lib64/python3.6/site-packages/django/db/backends/utils.py" in _execute   84.                 return self.cursor.execute(sql, params)  The above exception (null value in column "task_id" violates not-null constraint DETAIL:  Failing row contains (null, null, collection_demo, 1.0.26, 1). ) was the direct cause of the following exception:  File "/venv/lib64/python3.6/site-packages/django/core/handlers/exception.py" in inner   34.             response = get_response(request)  File "/venv/lib64/python3.6/site-packages/django/core/handlers/base.py" in _get_response   115.                 response = self.process_exception_by_middleware(e, request)  File "/venv/lib64/python3.6/site-packages/django/core/handlers/base.py" in _get_response   113.                 response = wrapped_callback(request, *callback_args, **callback_kwargs)  File "/venv/lib64/python3.6/site-packages/django/views/decorators/csrf.py" in wrapped_view   54.         return view_func(*args, **kwargs)  File "/venv/lib64/python3.6/site-packages/django/views/generic/base.py" in view   71.             return self.dispatch(request, *args, **kwargs)  File "/venv/lib64/python3.6/site-packages/rest_framework/views.py" in dispatch   505.             response = self.handle_exception(exc)  File "/venv/lib64/python3.6/site-packages/rest_framework/views.py" in handle_exception   465.             self.raise_uncaught_exception(exc)  File "/venv/lib64/python3.6/site-packages/rest_framework/views.py" in raise_uncaught_exception   476.         raise exc  File "/venv/lib64/python3.6/site-packages/rest_framework/views.py" in dispatch   502.             response = handler(request, *args, **kwargs)  File "/code/galaxy-api/galaxy_api/api/v3/viewsets.py" in post   200.             version=data['filename'].version,  File "/venv/lib64/python3.6/site-packages/django/db/models/manager.py" in manager_method   82.                 return getattr(self.get_queryset(), name)(*args, **kwargs)  File "/venv/lib64/python3.6/site-packages/django/db/models/query.py" in create   422.         obj.save(force_insert=True, using=self.db)  File "/venv/lib64/python3.6/site-packages/django/db/models/base.py" in save   741.                        force_update=force_update, update_fields=update_fields)  File "/venv/lib64/python3.6/site-packages/django/db/models/base.py" in save_base   779.                 force_update, using, update_fields,  File "/venv/lib64/python3.6/site-packages/django/db/models/base.py" in _save_table   870.             result = self._do_insert(cls._base_manager, using, fields, update_pk, raw)  File "/venv/lib64/python3.6/site-packages/django/db/models/base.py" in _do_insert   908.                                using=using, raw=raw)  File "/venv/lib64/python3.6/site-packages/django/db/models/manager.py" in manager_method   82.                 return getattr(self.get_queryset(), name)(*args, **kwargs)  File "/venv/lib64/python3.6/site-packages/django/db/models/query.py" in _insert   1186.         return query.get_compiler(using=using).execute_sql(return_id)  File "/venv/lib64/python3.6/site-packages/django/db/models/sql/compiler.py" in execute_sql   1335.                 cursor.execute(sql, params)  File "/venv/lib64/python3.6/site-packages/django/db/backends/utils.py" in execute   99.             return super().execute(sql, params)  File "/venv/lib64/python3.6/site-packages/django/db/backends/utils.py" in execute   67.         return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)  File "/venv/lib64/python3.6/site-packages/django/db/backends/utils.py" in _execute_with_wrappers   76.         return executor(sql, params, many, context)  File "/venv/lib64/python3.6/site-packages/django/db/backends/utils.py" in _execute   84.                 return self.cursor.execute(sql, params)  File "/venv/lib64/python3.6/site-packages/django/db/utils.py" in __exit__   89.                 raise dj_exc_value.with_traceback(traceback) from exc_value  File "/venv/lib64/python3.6/site-packages/django/db/backends/utils.py" in _execute   84.                 return self.cursor.execute(sql, params)  Exception Type: IntegrityError at /api/automation-hub/v3/artifacts/collections/ Exception Value: null value in column "task_id" violates not-null constraint DETAIL:  Failing row contains (null, null, collection_demo, 1.0.26, 1).

Why aren't we using the same endpoint as before?

had the same issue as David (see nr.2 in "I've tried" section of https://github.com/ansible/galaxy-dev/issues/163 issue)

Why aren't we using the same endpoint as before?

@newswangerd As far as I know, we are using the same endpoint?  What was it before?
Pulp exposes pulp_ansible.app.viewsets.CollectionUploadViewSet at '/ansible/collections'
https://pulp-ansible.readthedocs.io/en/latest/restapi.html#operation/ansible_collections_create

@newswangerd @romanblanco I just updated and rebuilt everything and now I'm seeing the task_id null error as well.
Very verbose logs at http://pastebin.test.redhat.com/811942

Looks like GET to the task resource is setting task_id None, which causes the galaxy-api CollectionImport creation to fail because of the null task_id.
@newswangerd  Wasn't this previous broken and fixed on the pulp end? Feels like a regression.
pulp-api_1               | 2019-11-06 16:08:49,634| django.server basehttp:log_message:154 35<PHONE_NUMBER>26624 INFO| "GET /pulp/api/v3/tasks/6e500795-d56a-4507-a040-5985ca795e93/ HTTP/1.1" 200 489
galaxy-api_1             | DEBUG 2019-11-06 16:08:49,641 galaxy_api.api.v3.viewsets viewsets:post:192 33 Thread-6 task_detail: {'created_at': None,
galaxy-api_1             |  'error': None,
galaxy-api_1             |  'finished_at': None,
galaxy-api_1             |  'id': None,
galaxy-api_1             |  'messages': None,
galaxy-api_1             |  'started_at': datetime.datetime(2019, 11, 6, 16, 8, 49, 510373, tzinfo=tzlocal()),
galaxy-api_1             |  'state': 'running',
galaxy-api_1             |  'updated_at': None}
galaxy-api_1             | INFO 2019-11-06 16:08:49,642 galaxy_api.api.v3.viewsets viewsets:post:195 33 Thread-6 Publishing of artifact alikins-test_collection-0.0.112.tar.gz to namespace=alikins by user=alikins created pulp import task_id=None
postgres_1               | 2019-11-06 16:08:49.643 UTC [98] ERROR:  null value in column "task_id" violates not-null constraint
postgres_1               | 2019-11-06 16:08:49.643 UTC [98] DETAIL:  Failing row contains (null, null, test_collection, 0.0.112, 1).
postgres_1               | 2019-11-06 16:08:49.643 UTC [98] STATEMENT:  INSERT INTO "galaxy_api_collectionimport" ("task_id", "created_at", "namespace_id", "name", "version") VALUES (NULL, NULL, 1, 'test_collection', '0.0.112')

@newswangerd @romanblanco @daviddavis
The similar previous issue:
https://github.com/ansible/galaxy-dev/issues/124
https://pulp.plan.io/issues/5569
https://github.com/pulp/pulp_ansible/pull/228

@newswangerd @romanblanco updated, should work now.
The publish endpoint path introduced in f78082d009398d9415262a51c8b0d0cbd8fd7002 was wrong and broke things.


Issues resuming from background

Still working to identify this.  But when resuming from background, often it does not seem to get latest on the current channel in focus.
For example, if i'm having a one on one conversation and put in the background and I get a notification of a new message and restore from background (not tap notification.  That action does seem to work) the latest messages are not loaded.
This seems to happen more when its been backgrounded longer.  Usually to get around this I swipe it away from recents and re-open.  Most of the time this does the trick.  Others I've had to force stop and then open again.
Its un-clear if it simply hasn't established a connection to the server again, or is just failing to load those missed messages.
Might be related to #117 its hard for me to be certain

It is caused by the state mismatch of ServerConfig model...
By killing the RocketChatService manually (swipe away from recent tasks), the state is reset and works as expected.

I tested this issue (related #117) both demo.rocket.chat server & my own rocket server. Surpringly, the problem didn't happen in demo.rocket.chat server almostly.
But, the problem happen frequently in my own server.

The root cause is that RxWebSocket#isConnected returns true unexpectedly, even after the WebSocket connection is disconnected silently for some reason...
https://github.com/RocketChat/Rocket.Chat.Android/blob/develop/android-ddp/src/main/java/chat/rocket/android_ddp/rx/RxWebSocket.java

We have an issue to redo the connectivity.
The root cause is that DDP MethodCall's Timeout is not handled well.
I will try to fix it.

#196 has added up to fixes here.
Other times will be in conversation and they will send a message and it'll show in sidebar a new message but won't show in channel pane. I have to again switch channels and back to get it to show up.
Should be fixed.

@laggedHero awesome!  I'll test the new builds!

Still having this issue with 1.0.15 from the play store fyi

I believe this is fixed already in 1.0.21.


Add transcoding for video

As a user, I want to view video files that are not native to the browser.
I looked for that in the implementation in stash. Sadly it is implemented in Go, but the general structure will be useful for our implementation:

Add a new API endpoint for requesting a transcode for video with a starting point. Should close the process when not playing the video.
Add two functions for creating a new mp4 file and for transcoding the video via ffmpeg
Add the mp4 option in LightBox, when it's seeking and transcoding, then start a new transcoding process via the API and activate it when the requested file is not in the right format.

An experimental option was added. StreamingHTTPResponse does not seem to stop once you close the light box, but it does when the window is closed. I have to investigate what possible fixes are.
We can use the same endpoint, but add a bit of information like "?s=109&r=1080" or something like that. This will obviously only be relevant when transcoding.
We also have to write a custom ReactPlayer to override the duration, as the video file from ffmpeg does not know while transcoding how long it will be. This should be added to the photo model as an attribute.

Now the process closes reliably.


Adds ability to get data from cloud copy

Overview
This PR adds the ability to get data from cloud copy. However, it's not stable right now due to no stable copy is available. Additionally, this doesn't not have any date checking yet.

Pull Request Test Coverage Report for Build 186

0 of 0   changed or added relevant lines in 0 files are covered.
253 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.5%) to 41.876%

Files with Coverage Reduction
New Missed Lines
%

/home/travis/miniconda/envs/TEST/lib/python3.6/site-packages/yodapy/datasources/ooi/helpers.py
4
32.65%

/home/travis/miniconda/envs/TEST/lib/python3.6/site-packages/yodapy/utils/parser.py
9
76.74%

/home/travis/miniconda/envs/TEST/lib/python3.6/site-packages/yodapy/datasources/ooi/init.py
83
36.07%

/home/travis/miniconda/envs/TEST/lib/python3.6/site-packages/yodapy/datasources/ooi/m2m_client.py
157
31.72%

Totals

Change from base Build 184:
-0.5%

Covered Lines:
366

Relevant Lines:
874

💛  - Coveralls


Add check/alarm for monitoring network throughput

The new MaaS template is used to create a set of checks and alarms, one
per specified NIC, for monitoring network throughput.
As part of this commit, the conntrack monitor is moved to 'kernel'
monitors, so that the network monitors can be for actual network things.


use --ozone-platform-hint=auto flag to allow wayland support on linux executable

ref: https://github.com/microsoft/vscode/issues/109176

@microsoft-github-policy-service agree

For pending discussion in https://github.com/electron/electron/pull/35630 I will hold off from keeping auto as the default. However I would like to support users configuring ozone-platform-hint, if you can modify the PR to support it as a runtime argument similar to https://github.com/microsoft/vscode/pull/186004, we can move this change forward.

Any update on this?


Failed to release to TestPyPI

https://github.com/bugen/pypipe/actions/runs/7209518193

@diazona
The error this time seems to be on the TestPyPI side.  I created release v0.4.1a0 on GitHub without making any changes, and it was successfully released to both TestPyPI and PyPI.
In cases like this, where a release fails for some reason, I would like to manually rerun the release.yml without creating a new release on GitHub. Is this possible?"

Certainly. I think you can just rerun the job using the Github web interface (or the REST API). In that case it will re-attempt the deployment to Test PyPI, then if that succeeds, it will go ahead to the next step and deploy to PyPI.
If you have any trouble with those instructions, let me know, and I can take a closer look and maybe offer some other suggestions.

I didn't want to leave the situation where I couldn't release on PyPI unattended, so I hurriedly dealt with it. As a result, a meaningless version v0.4.1 was created...

If you have any trouble with those instructions, let me know, and I can take a closer look and maybe offer some other suggestions.
Thank you👍 It's a bad habit of mine to try to handle things in my way without fully understanding. From now on, I will make it a point to consult calmly before taking any action.

Makes sense. In general there is rarely a reason to do anything in a rush when it comes to open source software, or at least I think so. But no harm done in this case; it doesn't hurt to bump the version number for no reason.


[BUG] exception occurred

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?
System.Net.Http.HttpRequestException: Response status code does not indicate success: 403 (Forbidden).
Bloxstrap Log
N/A
Bloxstrap_20240925T221256Z.log

Are you lying to us about you using the latest version of Bloxstrap...? This error occurs only when you're on a version that's not the latest (2.7.0)

It happened yesterday and after restarting the computer it just started working back normally (I have auto-update on, so I didn't really think about it)

We don't support exploiters. Bye!


cannot train with ml engine/install it

Found this while trying cloud training. Error can be done locally with:
$ pwd
.../tensorfx/build/dist
$ pip install --upgrade --force-reinstall tensorfx-0.1.4.tar.gz
Processing ./tensorfx-0.1.4.tar.gz
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-qs8P7n-build/setup.py", line 22, in
with open('requirements.txt') as rf:
IOError: [Errno 2] No such file or directory: 'requirements.txt'
----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-qs8P7n-build/

problem: setuptools >= 34.3.1 is needed. I built the tar ball with 28.2.0 the first time and got this error.

sent PR to check for this is build.sh

min setuptools needed is 30

Is this a pecularity of the internal goobuntu setup? I certainly didn't hit it on my mac where I use miniconda (but not a virtual or conda env).

which version of setuptools did you use?

I used whatever comes by standard with miniconda (as far as I remember, I didn't install it explicitly) which happens to be 34.2.0.
I checked on ubuntu. It is 26.1.1, and so this must be an ubuntu thing.
Does it make sense to require using miniconda as a starting point for the development enviornment, so there is a possibility of consistent environment?


General question on how your library was created

Sorry I know this question is not directly related to your project.  Just you are one of the few people I have found that have created any code for this API.  I need to do something similar in Java but I am not finding any official specification for the App Store Notifications v2.  Is there an openAPI spec for this library?

Hi,
I'm not aware of any OpenAPI spec. I used Apple's documentation here: https://developer.apple.com/documentation/appstoreservernotifications/app_store_server_notifications_v2
The notification history and testing APIs are documented here: https://developer.apple.com/documentation/appstoreserverapi

Thanks so much...It's crazy that Apple makes you parse html pages.  I coded
by hand using your classes as a guide.
On Sun, Aug 28, 2022 at 3:02 PM August Heegaard @.***>
wrote:

Hi,
I'm not aware of any OpenAPI spec. I used Apple's documentation here:
https://developer.apple.com/documentation/appstoreservernotifications/app_store_server_notifications_v2
The notification history and testing APIs are documented here:
https://developer.apple.com/documentation/appstoreserverapi
—
Reply to this email directly, view it on GitHub
https://github.com/agisboye/app-store-server-api/issues/19#issuecomment-1229531867,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEQPB4EOAVP2T2UZHZ37D3V3OZTZANCNFSM57ZTKFFA
.
You are receiving this because you authored the thread.Message ID:
@.***>


6.0.0 document issue for config online modify

Bug Report

What version of TiKV are you using?
no matter
What operating system and CPU are you using?
no matter
Steps to reproduce
no matter
What did you expect?
1、add some notes that  configuration cannot be persisted if readonly filesystem in dynamic config document for operator scenes
2、store-pool-size、apply-pool-size：Incorrect scope statement
3、scheduler-worker-pool-size：no maximum value
4、raft-max-inflight-msgs：no scope statement
5、num-threads：no description
What did happened?
some instructions are missing

/type bug
/severity Moderate
/assign glorv

/assign ethercflow

@qiancai Do we need to add a note in https://docs.pingcap.com/tidb/stable/dynamic-config to describe that this feature may not be correctly handled when the config file is mounted read-only(typically in the cluster environment.)


add google analytics

This should "just work" assuming I didn't make a copy-paste error somewhere.
This add  a newline to index.Rmd to make sure the live book rerenders when this is merged.
Somebody with sufficient power will need to merge this - I'm not on this repo.  If you want to hold off until more material is in the book, that's fine too, just wanted to get this PR set up while I'm doing all the others.  Merging this PR should start analytics data collection.

@avahoffman


dev: Refactor stop-timer components / codes #7

Link #7
Description:

Refactored the stopwatch app to different components.
Separated the store, reducers and actions.

Done.


Fixes DayPicker height issue on open

When the initialVisibleMonth has a different height than the current month, the DayPicker does not adjust immediately. I noticed this in prod on airbnb.com. Whoops!
to: @airbnb/webinfra

thanks for fixing this - thought one of my external styles was causing this to happen!

Coverage remained the same at 88.333% when pulling 50d0afefc0de1066a6497ad0c0a4ebe4fbe673d0 on maja-adjust-height-when-initialVisibleMonth-is-set into e4cbdc0dc637c4df9b72022cb58bf93f632aec5e on master.


Update ios support for download attribute

The feature is available for testing in iOS 13 developer beta 1 according to https://bugs.webkit.org/show_bug.cgi?id=167341

Thanks!


template schema importing all the files in a folder

This a section in a schema file
   imports:
    - path: configs/folder1/resources/gcpresource/test/*

I am trying to import all the files in a folder using the template's schema file. I know that this does not work.
My question is, what is a better way to import all the files using * or something that's suitable so that deployment manager can import all of the files in a folder without having to explicitly specify them ?

This is a frequently requested feature which the Deployment Manager team is currently working on. Hold on just a little bit more.

thanks @ocsig

The feature is available for public use:
https://cloud.google.com/sdk/docs/release-notes#cloud_deployment_manager

Added config key which enables globbing in the gcloud deployment-manager deployments <create|update> command. Config files can now use glob patterns in import paths.
This feature can be enabled by setting the config property deployment_manager/glob_imports to true: gcloud config set deployment_manager/glob_imports True.
Imports with a path pattern that match multiple files cannot be named.
When referencing a template in a resource type field, its full name is still required.
More information on globbing is available at http://man7.org/linux/man-pages/man7/glob.7.html.
More information on deployment-manager configurations is available at https://cloud.google.com/deployment-manager/docs/configuration/.

@ocsig  Thanks. We already have started using this feature extensively.
Based your conversation with Vincent, we took some leads and incorporated in our deployments and have been able to successfully import all of the files in folder without hassles. Indeed this is a very useful feature.

The feature is available for public use:
https://cloud.google.com/sdk/docs/release-notes#cloud_deployment_manager

Added config key which enables globbing in the gcloud deployment-manager deployments <create|update> command. Config files can now use glob patterns in import paths.
This feature can be enabled by setting the config property deployment_manager/glob_imports to true: gcloud config set deployment_manager/glob_imports True.
Imports with a path pattern that match multiple files cannot be named.
When referencing a template in a resource type field, its full name is still required.
More information on globbing is available at http://man7.org/linux/man-pages/man7/glob.7.html.
More information on deployment-manager configurations is available at https://cloud.google.com/deployment-manager/docs/configuration/.

@ocsig is is possible to import everything under a resource folder that may also include some sub folders with files?


[CI] Upgrade zephyr-sdk to 0.16.9

Part of #17451
Zephyr SDK 0.16.9 is compatible with Zephyr 3.6, which we use for CI.
https://github.com/zephyrproject-rtos/sdk-ng/wiki/Zephyr-SDK-Version-Compatibility-Matrix

@tvm-bot rerun


WIP: Perform finalize processes when the job is deleting

What type of PR is this?
/kind bug
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #1339
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Oh, this branch isn't my forked branch :(
/close


Update logs to stringify error codes from mbedTLS

Improve error logging for mbedTLS failures
Description
Existing error logs containing mbedTLS numeric error codes are not readable. This PR replaces those with strings by using the mbedtls_strerror utility
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing, we will find a way to upstream a change to mbedtls for const char* log messages from error codes.


AQCU-1706: apply Kathy's fix to legacy branch for missing Qualifiers …

…from table

Coverage remained the same at 97.964% when pulling debe71049f4244b33349218e35e691974fdf0ca5 on mhines-usgs:legacy into 53ff15c1af91053322bdd2dc233a545fa95c2354 on USGS-R:legacy.


Error 403 at Crunchyroll even with cookies

Checklist

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

Question
I'm not sure whether this could be a bug.
I think this could be related with Cloudflare or reCaptcha for some reason.
Yesterday I used youtube-dl to download without no problems a entire non-premium non-login-required anime season from Crunchyroll website ( this anime: https://www.crunchyroll.com/es-es/hundred ). After that, youtube-dl stopped working when I tried to download more videos, giving me HTTP error 403 when I try to download any video from Crunchyroll...
So I took a research and investigated all related about cookies usage and I discovered yt-dlp, which in theory has the Chrunchyroll extractor more updated than youtube-dl, so I'm using yt-dlp right now.
With yt-dlp, on my first try, I only get the same 403 error for any Crunchyroll video url like with youtube-dl.
Because I can access Crunchyroll website with no problem under my local web-browser: Firefox DevEdition.
But yesterday after I finished downloading all the 12 episodes with youtube-dl, and it stopped working giving me error 403, I noticed that Crunchyroll website asked me to solve a reCaptcha in my local Firefox DevEdition web-browser, but it happened only once.
If I delete all cookies from my Firefox DevEdition browser, and I access Crynchyroll website, it asks me to resolve a reCapctha only once.
I also noticed that if I use CefSharp web-browser in a .NET Windows Forms application, the first time that I try to access Crunchyroll website it always ask me to resolve a reCaptcha. This happens once every time that I run a new instance of the application. After I solved the reCaptcha it does not ask me anymore while the application instance is running. This never happened to me before I downloaded the 12 episodes with youtube-dl.
In my Firefox DevEdition web-browser I tried to use cookies.txt extension to generate a cookies file from Crunchyroll website. But when I pass these cookies.txt file to yt-dlp it does not solve the problem, I still get error 403.
I also tried this alternative syntax to take cookies from my web-browser:
yt-dlp.exe --cookies-from-browser "firefox:ElektroStudios.dev-edition" --get-filename -o "test video.%(ext)s" "https://www.crunchyroll.com/es-es/hundred/episode-1-little-garden-702479"
That command execution it just throws error 403 again and again.
I tried other parameters like removing cache dir, check no certificate, setting the same user agent as my Firefox web-browser, and username + password.
I also tried using --user-agent "Mozilla/5.0" as suggested here. It still gets error 403 with or without cookies.
What can I do to avoid this 403 error and finally let me download videos from Crunchyroll again?.
Cookies.txt file content
# Netscape HTTP Cookie File
# https://curl.haxx.se/rfc/cookie_spec.html
# This is a generated file! Do not edit.

www.crunchyroll.com	FALSE	/	FALSE<PHONE_NUMBER>	cf_chl_2	9f3a7a1794ff871
www.crunchyroll.com	FALSE	/	FALSE<PHONE_NUMBER>	cf_chl_prog	x14
#HttpOnly_.crunchyroll.com	TRUE	/	TRUE<PHONE_NUMBER>	cf_clearance	ieG4GNX1pmXKTG6c2LcgKhvEkc8bvWWaisNvaM4zBxg-1652007424-0-250
#HttpOnly_.crunchyroll.com	TRUE	/	FALSE<PHONE_NUMBER>	c_visitor	4acfe514-2de9-4994-880f-36ded63f323e
#HttpOnly_.crunchyroll.com	TRUE	/	TRUE<PHONE_NUMBER>	__cf_bm	D8vSxRBb4t9zm2X2NUwRnFjNzboYdfAE9Th9U4JVv_Y-1652007424-0-AWaAkapHhy9vfAZgOUnPtG3vf0ieECPCZrGyg/MI4DdOni7GKSaYaMNlmzVTh6XUAVYnKvA4o2dLmj7+loj+d8O2+v4tWi+jm2RTvjNYcxei
www.crunchyroll.com	FALSE	/es-es	FALSE<PHONE_NUMBER>	_evidon_consent_cookie	{"consent_date":"2022-05-08T10:57:10.749Z","consent_type":1}
www.crunchyroll.com	FALSE	/es-es	FALSE<PHONE_NUMBER>	_evidon_suppress_gdpr_locale_cookie	accepted
#HttpOnly_.crunchyroll.com	TRUE	/	TRUE	0	__cfruid	c91cf6b0af8161647b92804d2eee06e57bac830f-1652007529
#HttpOnly_.crunchyroll.com	TRUE	/	TRUE	0	session_id	f50cc1151f31662aada2cb5a7a6f0819
#HttpOnly_.crunchyroll.com	TRUE	/	TRUE<PHONE_NUMBER>	etp_rt	d6746056-7de4-41be-92eb-3525679a704e
www.crunchyroll.com	FALSE	/es-es/hundred	FALSE<PHONE_NUMBER>	crunchyroll_beta_hide_banner	1

Verbose log
[debug] Command-line config: ['--cookies-from-browser', 'firefox:ElektroStudios.dev-edition', '--verbose', '--get-filename', '-o', 'test video.%(ext)s', 'https://www.crunchyroll.com/es-es/hundred/episode-1-little-garden-702479']
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "C:\Users\Administrador\AppData\Roaming\Mozilla\Firefox\Profiles\ElektroStudios.dev-edition\cookies.sqlite"
[Cookies] Extracted 2724 cookies from firefox
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2022.04.08 [7884ade] (win_exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19041-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 2021-11-25-git-522f577d7e-full_build-www.gyan.dev (setts), ffprobe 2021-11-25-git-522f577d7e-full_build-www.gyan.dev
[debug] Optional libraries: brotli, certifi, Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
[debug] [crunchyroll] Extracting URL: https://www.crunchyroll.com/es-es/hundred/episode-1-little-garden-702479
[crunchyroll] 702479: Downloading webpage
ERROR: [crunchyroll] 702479: Unable to download webpage: HTTP Error 403: Forbidden (caused by <HTTPError 403: 'Forbidden'>); 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 "yt_dlp\extractor\common.py", line 641, in extract
  File "yt_dlp\extractor\crunchyroll.py", line 422, in _real_extract
  File "yt_dlp\extractor\crunchyroll.py", line 275, in _download_webpage
  File "yt_dlp\extractor\common.py", line 932, in _download_webpage
  File "yt_dlp\extractor\common.py", line 800, in _download_webpage_handle
  File "yt_dlp\extractor\common.py", line 785, in _request_webpage

  File "yt_dlp\extractor\common.py", line 767, in _request_webpage
  File "yt_dlp\YoutubeDL.py", line 3601, in urlopen
  File "urllib\request.py", line 531, in open
  File "urllib\request.py", line 640, in http_response
  File "urllib\request.py", line 569, in error
  File "urllib\request.py", line 502, in _call_chain
  File "urllib\request.py", line 649, in http_error_default
urllib.error.HTTPError: HTTP Error 403: Forbidden

Related: https://github.com/ytdl-org/youtube-dl/issues/30926

Duplicate of #2052

Now I can say that the Cloudflare's block which only affects yt-dlp http requests lasted for about 24 hours.
At this point I only can suggest that users should set interval settings when using yt-dlp (and youtube-dl) with Crunchyroll website, because it seems so severe and it will block you for "excessive" downloading with http error 403 and for a entire day at least.
I'll share a simple batch-script for Windows users to automate url downloads from Crunchyroll.
    PAUSE
    EXIT /B 1
)

ECHO:Download completed.
PAUSE
EXIT /B 0

I hope this helps.


rb-grid gutter attribute

Includes tests and new demo section.
TP https://rockabox.tpondemand.com/entity/9831

flexCells as separate PR?

Added flex-cells attribute to this PR


Trying to Implement the Restore Session part but got Error 401 Unpaired with Cellphone

Hi, I'm currently trying to implement the restore session for this whatsapp-web-reveng. I'm doing it by saving the serverToken and clientToken credentials after the first login withQR code. I check whether there exists a credential stored. If yes, then pass the credentials. Otherwise, generate a QR code. However, I'm getting such error:

Actually the first time response returned is 200 which is OK. However, for the second one, the response is coded 401 which means "Unpaired with phone". So, I assumed that my phone is immediately disconnected after my first login (well actually the WhatsApp backend is disconnected after a few minutes IF there is no activity received). Is there anyway to resolve this issue?

Update: After I implement the Resolve Challenge, it still returns the same thing.

Hello! Did you solved this?

can i get this code for restore session whatsapp

#325


New release soon?

Any chance of cutting a new release? I'd love to start using the toml fix in a project I'm working on.

Sorry for the delay! In that case, I'll have a new release out tonight :),
Thanks!
~Timothy

https://github.com/timothycrosley/isort/releases/tag/4.3.18 Took a day longer than I wanted to get in one last fix, but here you go: https://github.com/timothycrosley/isort/releases/tag/4.3.18 :)
Thanks!
~Timothy


Ldanzinger/authentication view

@michael-tims please review and merge. These are minor UI changes needed after testing. Also, adding the new items to the README

Looks good!


Initial colab notebooks to link in docs

Added colab directory w/ readme
Initial notebook for use-case: service health monitoring
Initial notebook for use-case: infra health monitoring
Initial notebook for use-case: db health monitoring

@zasgar updated the colab to remove all outputs and fixed typos


Update version of com.android.tools.build:gradle

Latest release it 2.3.1

Latest is now 3.3

Single test for #206, #233, #234.  If it builds without errors, just do a sanity test with test-basic, like drawing some features on the map.

Reintegrate


Improve the UX/UI of counters

We should improve the UI of the counters to make more visible the fact that they are selectable
current UI

We need some mockup before proceed.

@allyoucanmap Here's the proposed design of the updated counter card

@tonym3nsa I like the idea. Could we test also with the selection color on hover instead of the color of the border?
I'm worried that too many different colors on hover could be confusing (but maybe not)

@allyoucanmap here's with the selection color as the background.

@tonym3nsa we could also keep the border color to have reference to the specific category. probably this approach needs a slightly different blue


Nightly build against scikit-learn master

@Alexsandruss  - switching to python 3.10 bumped scikit-learn version and we have new failures.
Patched run: {'failed': 70, 'passed': 10318, 'skipped': 736, 'deselected': 17955, 'xfailed': 46, 'xpassed': 11}


Rule Regex doesn't work as expected

Hey, I've got a syntax highlighting rule that should match the first word of each line like this:
<Rule color="CmdOrVar">     ^[+\-_a-zA-Z]([\w]*)?(\s|$) </Rule>
and it works when trying it on regexr:

But in the Editor it looks like this and it feels like it's deciding for each word instead of each line:

I have tried so long for different solutions for this as well as some other things and it feels impossible to do anything more with regex than just highlighting certain keywords, strings or comments...
Is there maybe a way to write my own logic for highlighting text, based on code instead of regex?

You can derive a class from DocumentColorizingTransformer.

Thanks, I managed to make that work, uglily, but it works


[Feature Addition]: Web App for Air Quality Prediction

ML-Crate Repository (Proposing new issue)
:red_circle: Project Title : [Feature Addition]: Web App for Air Quality Prediction
:red_circle: Aim : The aim is to create a web app for the Air Quality Prediction project. Use the best existing model in the project folder. Follow the Web App README template for the same.
:red_circle: Dataset : N/A
:red_circle: Approach : Try to create a Flask/Streamlit app for the existing project.
🔴 Reference Project Folder: Brain Tumor Detection

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
Participant ID (If not, then put NA) :
Approach for this Project :
What is your participant role? (Mention the Open Source Program name. Eg. HRSoC, GSSoC, GSOC etc.)

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Can You Please Assign this issue under SSOC. 2024 Season 3
Shivansh Mahajan
Github:- https://github.com/shivansh-2003
Participation ID:- NA
I will make a pickle file of the Fitted ML model and then deploy it to Streamlit Python Framework to display the feature and predict the quality . I am well versed with making Streamlit App you can check out my linkedin :-https://www.linkedin.com/in/shivansh-mahajan-13227824a/ and Git repository .
My some  recent Project in Streamlit https://www.linkedin.com/feed/update/urn:li:activity:7199784822737682432/(SPAM classifier)
https://www.linkedin.com/feed/update/urn:li:activity:7201206409605091328/ . (NLP APP)
Participation Role:- SSOC Season 3

Contributions will start from June 1, 2024. Till then please have some patience.

I have a question.Do I have to use your Model and create a web app ? Or do I have to make my own model with the dataset and create a web app?

I have a question.Do I have to use your Model and create a web app ? Or do I have to make my own model with the dataset and create a web app?

Use the existing model which is having the highest accuracy.

Shiladitya Mondal
Github:- https://github.com/Shiladityagit
Participation ID:- NA
Approach for this project : I will pick the model with the highest accuracy ,then I will make a pickle file to import the model .I will create a web app using Flask/Streamlit to show the air quality predictions.
Linkedin profile : https://www.linkedin.com/in/shiladitya-mondal-28421b2a3/
Can you please assign me with this issue @abhisheks008 ?
Participation Role:- SSOC Season 3

Armaan Jagirdar
Github: https://github.com/Armaan457
Participating under SSOC'24
After picking the model with highest accuracy, I'll make a pickle file to create a web app with Streamlit/Flask. I am a beginner but I have some experience of working with Flask and bit of Streamlit.

Assigned @Shiladityagit


Benchmarks

would be nice to have them, keep up the good work, thanks !

You are more than welcome to add them, we will be waiting for you pull request.
Thanks for helping us!

I created PR for this issue. I'm not an expert in benchmarking so please add your ideas/comments how it should be done

@janisz Thanks a lot!, we will review it.
On Wed, Feb 3, 2016 at 11:16 PM, janisz<EMAIL_ADDRESS>wrote:

I created PR for this issue. I'm not an expert in benchmarking so please
add your ideas/comments how it should be done
—
Reply to this email directly or view it on GitHub
https://github.com/streamrail/concurrent-map/issues/12#issuecomment-179472810
.


Remove dependency on deleted module cloud-plugin-storage-image-swift

Tests result from the bubble
tested with pull request https://github.com/MissionCriticalCloud/cosmic-core/pull/164
Deploy DC
[root@cs1 cs1]# python marvin/marvin/deployDataCenter.py -i /data/shared/marvin/mct-zone1-kvm1.cfg
[INFO] 2016-05-27 10:24:50,043 - marvin - Parsing Test data successful
[INFO] 2016-05-27 10:24:50,137 - marvin - === Test Client Creation Successful ===
[INFO] 2016-05-27 10:24:50,146 - marvin - === Deploy DC Started ===
[INFO] 2016-05-27 10:24:50,476 - marvin - === Create Zone Successful ===
[INFO] 2016-05-27 10:24:55,594 - marvin - === Creating Physical Network Name : mcct-pnet Id : aadb1f10-b36a-4c67-920b-b45232227fc0 Successful ===
[INFO] 2016-05-27 10:25:20,974 - marvin - === Update Network Service Provider Successfull ===
[INFO] 2016-05-27 10:25:26,068 - marvin - === Update Network Service Provider Successfull ===
[INFO] 2016-05-27 10:25:36,215 - marvin - === Update Network Service Provider Successfull ===
[INFO] 2016-05-27 10:26:44,605 - marvin - === Add Host Successful ===
[INFO] 2016-05-27 10:27:44,886 - marvin - === Creating Storage Pool Successful ===
[INFO] 2016-05-27 10:27:44,962 - marvin - === Add Image Store Successful ===
[INFO] 2016-05-27 10:27:44,996 - marvin - === Enable Zone Successful ===
[INFO] 2016-05-27 10:27:44,998 - marvin - === Data Center Settings are dumped to dc_entries_May_27_2016_10_27_44.obj ===
[INFO] 2016-05-27 10:27:45,012 - marvin - === Deploy DC Successful ===

VM lifecycle tests
[root@cs1 integration]# tail -100f results.txt
Test advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS ===ok
Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===ok
Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===ok
Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===ok
Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===ok
Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===ok
Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===ok
Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===ok
Test migrate VM ... SKIP: At least two hosts should be present in the zone for migration
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status : SUCCESS ===ok

----------------------------------------------------------------------
Ran 10 tests in 1653.464s

OK (SKIP=1)

ping @wilderrodrigues  @remibergsma @borisroman
can someone please merge this pull request?


[GHSA-prjv-jj26-wf8h] ClassLoader manipulation in Apache Struts

Updates

References
Source code location

Comments
Add patch link related to CVE-2014-0112.

Hi @sunSUNQ, https://github.com/apache/struts/commit/2e2da292166adbc78c4cb1e308b30ddb4fba6d3f is appropriate for CVE-2014-0094 but doesn't correspond with CVE-2014-0112 having been fixed in 2.3.20, according to https://cwiki.apache.org/confluence/display/WW/S2-021, so this contribution will not be included in GHSA-prjv-jj26-wf8h. Thank you for your interest in GHSA-prjv-jj26-wf8h!


Add tests for secrets into TFaGA

Signed-off-by: Petr "Stone" Hracek<EMAIL_ADDRESS>

[test]

Hopefully, now bugs/typos/errors will be reported by review :)


Groupby-agg on index with known divisions results in unknown divisions

I would expect ddf.groupby(ddf.index).sum() to have same divisions as ddf, but they seem to be unknown. Not sure if this is a desired behaviour or if I'm doing something wrong.
In[2]: import pandas as pd
In[3]: import dask.dataframe as dd
In[4]: ddf = dd.from_pandas(pd.DataFrame({'A': [1, 2]}), npartitions=1)
In[5]: ddf.divisions
Out[5]: (0, 1)
In[6]: ddf.groupby(ddf.index).sum().divisions
Out[6]: (None, None)

dask 0.16
pandas 0.21.1

After taking a look at dask tests I discovered what follows:
In[2]: import dask
  ...: import dask.dataframe as dd
  ...: import pandas as pd
  ...: import numpy as np

In[3]: ddf = dd.from_pandas(pd.DataFrame({'A': [1, 2]}, index=pd.Series([0, 1], name='idx')), npartitions=1)

In[4]: ddf.divisions
Out[4]: (0, 1)

In[5]: ddf.groupby(ddf.index).sum().divisions
Out[5]: (None, None)

In[6]: ddf.groupby('idx').sum().divisions
Out[6]: (None, None)

In[7]: ddf.groupby(ddf.index).apply(np.sum).divisions
UserWarning: `meta` is not specified...
Out[7]: (None, None)

In[8]: ddf.groupby('idx').apply(np.sum).divisions
UserWarning: `meta` is not specified...
Out[8]: (0, 1)

(warnings were truncated)
So, the expected behaviour occurs only when

I pass index name instead of the index itself to a groupby method AND
I use .apply(np.sum) instead of .sum() as an aggregation (works with built-in sum too)

Right, I think that groupby-aggregations currently always produce a
single-partition dataframe.  There isn't currently any special behavior
when operating on the index.
We do perform special behavior on groupby-apply operations, just because
not doing special behavior here can be so much more expensive (full
shuffles occur).
It seems reasonable to me to also add special handling on
groupby-aggregations as you suggest if we happen to have well-known
divisions (as we do in your case).
On Fri, Dec 15, 2017 at 3:29 AM, Michał D<EMAIL_ADDRESS>wrote:

After taking a look at dask tests I discovered what follows:
In[2]: import dask
...: import dask.dataframe as dd
...: import pandas as pd
...: import numpy as np
In[3]: ddf = dd.from_pandas(pd.DataFrame({'A': [1, 2]}, index=pd.Series([0, 1], name='idx')), npartitions=1)
In[4]: ddf.divisions
Out[4]: (0, 1)
In[5]: ddf.groupby(ddf.index).sum().divisions
Out[5]: (None, None)
In[6]: ddf.groupby('idx').sum().divisions
Out[6]: (None, None)
In[7]: ddf.groupby(ddf.index).apply(np.sum).divisionsUserWarning: meta is not specified...
Out[7]: (None, None)
In[8]: ddf.groupby('idx').apply(np.sum).divisionsUserWarning: meta is not specified...
Out[8]: (0, 1)
(warnings were truncated)
So, the expected behaviour occurs only when

I pass index name instead of the index itself to a groupby method
AND
I use .apply(np.sum) instead of .sum() as an aggregation (works
with built-in sum too)

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/2999#issuecomment-351957345, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszGy9RCDB2iZ8hyje6BHJGK7hDCQKks5tAjvogaJpZM4RCTnZ
.


fix: eliminate interference when other elements are invisible

fix: eliminate interference when other elements are invisible

Codecov Report

Merging #107 (8c47f67) into main (50410cf) will not change coverage.
The diff coverage is 0.00%.

:exclamation: Current head 8c47f67 differs from pull request most recent head 4b1c40b. Consider uploading reports for the commit 4b1c40b to get more accurate results

@@           Coverage Diff           @@
##             main     #107   +/-   ##
=======================================
  Coverage   40.75%   40.75%
=======================================
  Files          64       64
  Lines        2358     2358
  Branches      492      494    +2
=======================================
  Hits          961      961
+ Misses       1049     1047    -2
- Partials      348      350    +2

Flag
Coverage Δ

unittests
40.75% <0.00%> (ø)

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

Impacted Files
Coverage Δ

packages/core/src/lib/element.ts
2.69% <0.00%> (ø)

packages/core/src/lib/helper.ts
24.15% <0.00%> (ø)

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 50410cf...4b1c40b. Read the comment docs.


Save outputs in high precision in CLI prediction

Currently, CLIPredict() saves prediction results in default 6-digit precision which causes precision loss. This PR sets precision to a level so that the conversion back to bst_float is lossless.
Related: #3298.

Codecov Report

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

@@            Coverage Diff            @@
##             master    #3356   +/-   ##
=========================================
  Coverage     45.69%   45.69%
  Complexity      228      228
=========================================
  Files           166      166
  Lines         12973    12973
  Branches        466      466
=========================================
  Hits           5928     5928
  Misses         6853     6853
  Partials        192      192

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 1214081...486c606. Read the comment docs.


Inspector won't stay open to the hint tab

When using the inspector in practice, it keeps jumping to the details tab even when there is no details. Thus, you can't really look at the hints while they go by...

This may not be as relevant once we send all payload output to regular notebook/console output


Workflow commands for GitHub Actions is stale

What article on docs.github.com is affected?

https://docs.github.com/en/actions/reference/workflow-commands-for-github-actions#using-workflow-commands-to-access-toolkit-functions
What part(s) of the article would you like to see updated?

The following table shows which toolkit functions are available within a workflow:

Additional information

The table isn't complete.
At the very least, it's missing:
https://github.com/actions/runner/blob/main/docs/adrs/0280-command-input-echoing.md
which added echo '::echo::on' and echo '::echo::off'.
It is probably missing a number of other things. It'd be nice if someone interviewed the team and got the documents current.

It might not be documented on purpose, as the recommended way for turning command echoing on is to create a secret ACTIONS_RUNNER_DEBUG with a value of true: Enable debug logging
Other commands to potentially document:

::add-matcher and ::remove-matcher

Not to be documented:

::internal-set-repo-path
::add-path and ::set-env (deprecated)

In principle one is supposed to be able to turn toggle echo repeatedly (even w/in a single job step or action stage), which one couldn't do w/ the secret.
Anyway, the ADR is explicit, and I'll quote it:
https://github.com/actions/runner/blob/main/docs/adrs/0280-command-input-echoing.md#context

Echo output in the user logs can be explicitly controlled by the new commands ::echo::on and ::echo::off.

And as secrets apply to all workflows in a repository, it really is a very large hammer for what may be potentially a tiny nail.
That said, if it shouldn't be documented, there should be an obvious place for doc writers to go to determine this.
At present, I'm fairly confident no such place exists.

Thanks for pointing this out, @jsoref!
I agree that we should document echo '::echo::on' and echo '::echo::off' as a new section in "Workflow commands for GitHub Actions"
A new section titled something like "Echoing command outputs" that describes what  ::echo::on and ::echo::off do, along with examples like the existing sections. We should also mention the interaction/overlap with ACTIONS_RUNNER_DEBUG (how ACTIONS_RUNNER_DEBUG is a global setting, while these are more granular) and link to and from its section.
I'll tag this for anyone to open a PR to implement 👍

Apparently I missed?


Alabaster BG

Hi,
Support for custom background color for syntax hightlighting has been recently added to zed, cf. https://github.com/zed-industries/zed/pull/12400
This is thus a feature request to add the Alabaster BG theme here.
Alabaster BG is a variation of Alabaster available for Sublime Text:
https://github.com/tonsky/sublime-scheme-alabaster?tab=readme-ov-file#variations

Hello,
Don't really use the BG variants myself, but will be happy to merge a PR and update extension if someone implements this.


The link in the add-on pane of FF points to the wrong URL

The link in the addon-pane points to
https://retorquere.github.io/better-bibtex/
resulting in 404 error

it should point to
https://github.com/retorquere/zotero-better-bibtex/wiki

Thanks for reporting it, will be fixed in the next release (day or so)


Error after copying the sample code

Hi trying this action to add my ssh keys but it returns an error. I followed the sample code and run it but it returns an error.

Did you used this action with GitHub Actions? Can you show your workflow file? It seems like you are cloning my Repo on branch master and trying to run the main file, which is only available after a npm run build or within the releases/v1 branch.

Hi @yakuhzi , thanks for replying.
I am trying to use this to copy my ssh keys like this

I am using act to run the workflow in my local and this is the result

It seems like this is an issue with either Ubuntu or act, as my workflows using this action are all running on macOS without issues. I tested it with a new repository using Ubuntu as action host and ran into an other issue:
Error: Command failed: ssh-add -K /home/runner/.ssh/id_rsa
Could not open a connection to your authentication agent.

Hi @yakuhzi Let me try this tomorrow on my mac to confirm.

Hi, I have the same issue on mac

with this workflow :

Can you check if it's resolved with the latest release?


ARROW-7495: [Java] Remove "empty" concept from ArrowBuf, replace with custom referencemanager

Related to ARROW-7495.
With the introduction of ReferenceManager in the codebase, the need for a separate ArrowBuf is no longer necessary. Instead, once can create a new reference manager that is used for the empty ArrowBuf. For reminder/review, empty arrowbufs have a special behavior in that they don't actually have any reference counting semantics and always stay at one. This allow us to better troubleshoot unallocated memory than what would otherwise be an NPE after calling ValueVector.clear()

cc @jacques-n

Ping @jacques-n do you think you would have time to review this?  @rymurr are you familiar with this part of the code and can do a first pass?

Hey @emkornfield. I'm happy to take a look. I likely won't get to it till Monday as I am traveling at the moment.

Can we please add a test that tells how this will look like? I mean the proposal is to remove the isEmpty flag from ArrowBuf and have the users (whoever wants an empty ArrowBuf) pass a custom reference manager that is as good as an ArrowBuf.
It will be great if we have some test to do this. I am not sure if a new concrete implementation of ReferenceManager should be written to manage empty bufs or can we use the existing ReferenceManager.NO_OP?

Can we please add a test that tells how this will look like?
IIUC, the proposal is to remove the isEmpty flag from ArrowBuf and have the users (whoever wants an empty ArrowBuf) pass a custom reference manager that is as good as an empty ArrowBuf.
It will be great if we have some test to do this. I am not sure if a new concrete implementation of ReferenceManager should be written to manage empty bufs or can we use the existing ReferenceManager.NO_OP?

Thanks, IMO, one could either use ReferenceManager.NO_OP or write their own ReferenceManager for empty ArrowBuf. I added a simple test case using ReferenceManager.NO_OP.

This looks good to me. Potential nice to have: a static method to generate a default empty buffer of length n

This looks good to me. Potential nice to have: a static method to generate a default empty buffer of length n

Thanks, done.

@tianchen92 , can you please revise thia? Looks like the work is completed

@tianchen92 , can you please revise thia? Looks like the work is completed

ok, revised, thanks!

I think it would be good to have a set of tests ensuring that the behavior of the empty buffer is consistent. For example, refcnt, release/allocate, etc. That way, if someone changes the noop reference manager which this is depending on in the future, we don't break the expected behavior of an EmptyBuf as used by all the vector classes. Other than that, this looks good.

I think it would be good to have a set of tests ensuring that the behavior of the empty buffer is consistent. For example, refcnt, release/allocate, etc. That way, if someone changes the noop reference manager which this is depending on in the future, we don't break the expected behavior of an EmptyBuf as used by all the vector classes. Other than that, this looks good.

Good suggestion, seems empty ArrowBuf was used in vectors before populating data. I have added a set of tests in TestValueVector.

@jacques-n are you happy with the test coverage?  Any other issues?

Good suggestion, seems empty ArrowBuf was used in vectors before populating data. I have added a set of tests in TestValueVector.

The tests you added are all vector based. I'd like to tests against the "empty" ArrowBuf as well.

Good suggestion, seems empty ArrowBuf was used in vectors before populating data. I have added a set of tests in TestValueVector.

The tests you added are all vector based. I'd like to tests against the "empty" ArrowBuf as well.

I see, I added more cases in TestArrowBuf

Looks mostly good. +1
As a follow-on, it'd be nice to include a test which ensures that the Netty Allocator returns an empty-behaving byte buffer when users allocate a zero byte buffer.

include a test which ensures that the Netty Allocator returns an empty-behaving byte buffer when users allocate a zero byte buffer

Thanks jacques, I created https://issues.apache.org/jira/browse/ARROW-8870 to track.
@emkornfield Please help merge this PR when you have time, thanks a lot!

@tianchen92 can you rebase this once? I will merge then.

@tianchen92 can you rebase this once? I will merge then.

Thanks, rebased:)