listener: rewrite passthrough filter chain filter chain

Part of #22217
The goal is to decouple

the timing and the creation of an pass through filter chain(FilterChainMatch), and
what a passthough filter chain looks like (Filters)

The follow up is to simplify the merge filter chains which no longer consider merge a passthough filter chain to passthrough filter chain (low priority, not in this PR)
A pass through listener in virtual outbound listener cannot be implemented by a single passthrough filter chain.
Due to the impl of envoy filter chain match, greedy match and never back track.
If the virtual outbound listener :15001 has a :9090 listener(filter chain in #22217), basically we need at least 2 passthrough filter chains for

EMPTY port
port 9090 but any ip not matching .
Otherwise envoy cannot match :9090

To reviewers:
The parameters of OnOutboundPassthroughFilterChain is probably not ideal: it actually require the plugins to mutate the filters in filter chain. We can evolve it gradually.
Mostly only mixer plugin would touch it so IMHO it's not too bad to change the signature in the future.
Signed-off-by: Yuchen Dai<EMAIL_ADDRESS>

Is this just a refactoring? I am not 100% sure I understand what this is doing

@rshriram envoy is not ready yet.
Pilot change is not a trivial change: we cannot merge all per service listeners for two reasons

Merge per svc listener would cause envoy validation failure when building decision tree from vectors of filter chains in proto message.
If we do merge chains, the tcp traffic on http defined port (e.g. <IP_ADDRESS>:443) is not automatically fixed. See the blue line in @howardjohn 's traffic flow

The conclusion is that We need a new approach to build the passthrough filter chains.

@howardjohn in #22217 the passthrough filter chains will be different.

The FilterChainMatch in passthrough filter chain is not an empty match
There will be many passthrough filter chains each with different match, such as match EMPTY port, match "all traffic of dst port 80 but not http protocol", even more.

This PR allow we create a FilterChainMatch independently and then create a filter chain based on that filterchainmatch

@lambdai what I am saying is that we (myself, @howardjohn / @ramaraochavali / @hzxuzhonghu ) can collectively take care of fixing up the filter chains on Pilot end but we need to get some idea of how the mechanism will work on Envoy side. there is no point in adding changes to pilot right now without knowing how the filter chain reloads are handled in envoy. If there are some semantics to follow, it would be good to know.

Put another way, give us the requirements on how you want the filter chains to be ordered

@rshriram https://github.com/envoyproxy/envoy/pull/10662#discussion_r411725272
The assumption is if the listener config proto message only has change in FilterChains field, the the connections could survive from being drained.
Under above new listener config, if the connection belonged to a filter chain config which is byte2byte equivalent, the connection will survive.

@lambdai: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Put another way, give us the requirements on how you want the filter chains to be ordered
The ordered of filter chains doesn't matter. Envoy will compare each filter chain, decide which filter chains are equivalent before and after the listener config update.
Currently the "equivalent" is defined by the corresponding proto message.

@hzxuzhonghu @howardjohn #23132 is the follow up. The filter chain match of pass through filter chains are complicated so we need to extract the passthrough filterchain out of the per svc filter chain build

@lambdai do we need to merge this before 1.6 code freeze? If not, I would like to not add changes to a sensitive code area so close to the release.

@nrjpoddar  yes... it is required by #22217 which we want to deliver in 1.6
2 more needed. This one is low risk. Unfortunately the last one is not as the #22217 is not a trivial change.

I thought single outbound listener was for 1.7?
On Thu, Apr 23, 2020 at 1:26 PM Yuchen Dai<EMAIL_ADDRESS>wrote:

@nrjpoddar https://github.com/nrjpoddar yes... it is required by #22217
https://github.com/istio/istio/issues/22217 which we want to deliver in
1.6
2 more needed. This one is low risk. Unfortunately the last one is not as
the #22217 https://github.com/istio/istio/issues/22217 is not a trivial
change.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/23071#issuecomment-618648037, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXNG3N435UD2UEYQVSLROCP7DANCNFSM4MMHU35A
.

It's on my 1.6 plate.
We definitely want an option for the user to turn on in 1.6, if not all the tools are ready by 1.6.0

ping @howardjohn @rshriram @hzxuzhonghu for networking change

@yangminzhu The new callback is trivial. security plugins doesn't do anything on outbound passthrough filter chains before and after this PR

@mandarjog mixer plugin has a little change at onOutboundListener and the new onOutboundPassthroughFilterChain
It's passthrough filter chain so I feel it is not that risky.

will review tomorrow morning certainly. sorry for delay

gentle ping @rshriram

ping @istio/wg-security-maintainers for callbacks of security plugin on outbound passthrough filter chain. Implemented as empty since outbound passthrough should do nothing (learnt from existing OnOutboundListener().
ping @istio/wg-policies-and-telemetry-maintainers For mixer plugin

Please make sure to handle https://github.com/istio/istio/pull/23071/files#r416289557. I think it is handled implicitly since all tests are passing?

Passthrough filter chain is not added if there is catch all filter chain so we are good :)

@lambdai: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@lambdai: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
2cae55dca5ab094838f3ac5a3f91bc058e43bd76
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@lambdai: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
34a78a3ade088ec324acd5e6f876a95e33992804
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@mandarjog Mixer filter is deprecated, right? This should lessen the risk of this PR I guess

Every time I look at the listener code, takes me like 15 mins to retrace all the paths, thanks @lambdai for handling all the different cases!

@nrjpoddar
Just an except from <IP_ADDRESS>_80 which introduce passthrough filter
Stripping the pod info
182,188d182
<                 "metadata": {
<                     "filterMetadata": {
<                         "pilot_meta": {
<                             "fallthrough": true
<                         }
<                     }
<                 },

The full listener config of <IP_ADDRESS>_80
with this PR:
http://paste.ubuntu.com/p/DWryycscXX/
master
http://paste.ubuntu.com/p/T3pF3nb2yt/

@lambdai the diff looks good! It looks like the FC chain is broken as of now in master as it's not adding mixer filter.

@nrjpoddar
The virtual inbound listener was a little bit different. It surprised me because this PR is supposed to mutate outbound.
Eventually I debugged and realize my two clusters are different...
Now I get exactly the same 15001 outbound listener.
And close 15006 inbound listener after the sort.
master: http://paste.ubuntu.com/p/wDK9dVxywS/
this PR:http://paste.ubuntu.com/p/wJgYq7YZFR/
$ diff mixer.pt.master.15x.yaml pr.15x.yaml
534c534
<                                                     "stringValue": "kubernetes://details-v1-775bf85c6d-5f95b.default"
---
>                                                     "stringValue": "kubernetes://details-v1-67f47c744d-bp6z8.default"
549c549
<                                                     "stringValue": "kubernetes://details-v1-775bf85c6d-5f95b.default"
---
>                                                     "stringValue": "kubernetes://details-v1-67f47c744d-bp6z8.default"
768c768
<                                                     "stringValue": "kubernetes://details-v1-775bf85c6d-5f95b.default"
---
>                                                     "stringValue": "kubernetes://details-v1-67f47c744d-bp6z8.default"
783c783
<                                                     "stringValue": "kubernetes://details-v1-775bf85c6d-5f95b.default"
---
>                                                     "stringValue": "kubernetes://details-v1-67f47c744d-bp6z8.default"


Stack creation failing when using quickstart to create cluster on existing VPC

We are trying to create a cluster on existing vpc, but it was failing when the kube-master instance is created.
Error:
14:19:11 UTC-0500 	CREATE_FAILED 	AWS::EC2::Instance 	K8sMasterInstance 	Received FAILURE signal with UniqueId

found the resolution from previous issues.


hugo-apps-theme repo is transferred

Hi, I transferred Hugo apps theme repository to different account.
https://github.com/gonapps/hugo-apps-theme
https://github.com/gonapps-org/hugo-apps-theme
Could you merge this change?

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello @gonapps,
thank you for notifying us about the new url for your theme. I've closed this issue by creating the corresponding commit myself because the Hugo project established the policy, that all pull requests submitters must sign the CLA above.
Happy Hugoing.

Oh, I didn't know about that, thanks!


When we switch from currency converter to volume then narrator is first announcing the currency info and then volume.

A team member will update this issue with more details soon.

This is your friendly Microsoft Issue Bot. I created this issue automatically as requested by a team member.


enable user to use any model generated here

change two files:
1 . cli.py:  add a piece of code to ask to enter the config the user used to generate his/her expert model

serial_pipeline_sqil.py: as we load the expert config, we could use this config to load expert model and collect

issues:

sample serial collector is not compatible with create_policy (only enable collect mode)
running the code will generate two files: one for sqil and one for the one the user chose. If we use both sqil, the total config will be a little bit messy

Codecov Report

Merging #44 (687d54c) into main (0023438) will decrease coverage by 0.12%.
The diff coverage is 92.30%.

:exclamation: Current head 687d54c differs from pull request most recent head 8af892f. Consider uploading reports for the commit 8af892f to get more accurate results

@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
- Coverage   90.05%   89.92%   -0.13%
==========================================
  Files         339      339
  Lines       24839    24555     -284
==========================================
- Hits        22369    22082     -287
- Misses       2470     2473       +3

Flag
Coverage Δ

unittests
89.92% <92.30%> (-0.13%)
:arrow_down:

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

Impacted Files
Coverage Δ

ding/entry/serial_entry_sqil.py
79.06% <91.66%> (+0.58%)
:arrow_up:

ding/entry/tests/test_serial_entry_sqil.py
89.47% <100.00%> (-0.53%)
:arrow_down:

ding/utils/data/structure/cache.py
94.23% <0.00%> (-5.77%)
:arrow_down:

ding/entry/tests/test_parallel_entry.py
45.45% <0.00%> (-4.55%)
:arrow_down:

ding/entry/tests/test_serial_entry.py
77.06% <0.00%> (-2.36%)
:arrow_down:

ding/utils/tests/test_system_helper.py
80.00% <0.00%> (-1.82%)
:arrow_down:

ding/rl_utils/tests/test_ppg.py
27.50% <0.00%> (-1.77%)
:arrow_down:

ding/utils/data/tests/test_dataloader.py
31.25% <0.00%> (-1.68%)
:arrow_down:

ding/utils/data/dataloader.py
45.07% <0.00%> (-1.04%)
:arrow_down:

ding/torch_utils/tests/test_data_helper.py
73.85% <0.00%> (-0.99%)
:arrow_down:

... and 117 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 0023438...8af892f. Read the comment docs.


Refactoring truncate function

I don't know why it uses array casting.

I don't know why it uses array casting.

I guess this was because we needed to count Emojis correctly,
but there could be a better way.
str.length and [...str].length make different when str includes Emojis.
const str = '🍣🍣🍣🍣🍣';
cosole.log(str.length); // => 10
cosole.log([...str].length); // => 5

Thank you.
I'll be extreamly careful about an input string...


strange loss for restored model

Hi,  @pvoigtlaender
For training, I have changed the "num_epochs" in config_file to 15, and set "freeze_batchnorm" to false, and the loss is about 0.2 after 15 epochs finished.
Then I changed the "num_epochs" to 16 and train again, the model is restored from conv3d_sep2-00000015, however, the initial loss is very large, which is about "1e6 to 1e7".

Hi,
why do you set freeze_batchnorm to false? I cannot recommend that.
Especially if you use a model trained with frozen batch norm and then unfreeze batch norm, the input to the layer will be completely unexpected and then the output should be kind of random which can very well be the cause of such a high loss value.

Hi, @pvoigtlaender
Thanks for your prompt reply.
When the freeze_batchnorm is true, the loss  falling slowly, so I set it to false. And it seems that unfreezing the batch norm may be the cause.

Hi, @pvoigtlaender
For training, I have changed the "num_epochs" in config_file to 15, and set "freeze_batchnorm" to false, and the loss is about 0.2 after 15 epochs finished.
Then I changed the "num_epochs" to 16 and train again, the model is restored from conv3d_sep2-00000015, however, the initial loss is very large, which is about "1e6 to 1e7".

I have found the reason, if I keep the "freeze_batchnorm" false after restoring the model, then the initial loss is still small. The switch of "freeze_batchnorm" determines the final computation graph.


[BUG] #36 added favicon icon

closes #36
Title: [A short summary of the pull request]
Added favicon icon
Description
Added a favicon icon
Screenshots (if applicable)

Hey @Amarjha01
can you create a new logo, like amazon?
and also use the first character of scrollme?
What do you think about this ?

Hey @Amarjha01 can you create a new logo, like amazon? and also use the first character of scrollme? What do you think about this ?

This sounds good. Off course I'll create some awesome favicon icon like amazon .

@saurabhbakolia sir is this okay now ?

@saurabhbakolia  I would like to work on this issue. Could you please assign me this task under the GSSOC-ext and Hacktoberfest-accepted tags? I’m excited to contribute.
i can add the logo some want like amazon sating with s with more interactive
Thanks so much!

@Amarjha01 can you provide the video link

@Amarjha01 can you provide the video link

Video link of what sir ?

@Amarjha01 favicon icon is added showing proper or not

https://github.com/user-attachments/assets/b81e66b6-3876-4394-a352-8cf2390a0777
@thakuratul2 avoid the that's case sensitive error. I will only push favicon icon

@saurabhbakolia @thakuratul2  Sir whenever I push the corrected path vercel fails to deploy. I can't view details of why Vercel was unable to deploy to solve the error.

Hey @Amarjha01,
I think the logo made is okay, but let's give it a try @vikas83pal.
@vikas83pal can you create an attractive logo with a minimalistic design?
@vikas83pal  assigned!
Now that we've clarified the requirements, I’d like both of you to discuss and collaborate on creating a new logo. Please share your ideas and mockups once you’ve come to a consensus. Looking forward to seeing your creative designs!

Hey @Amarjha01,
I think the logo made is okay, but let's give it a try @vikas83pal.
@vikas83pal can you create an attractive logo with a minimalistic design? @vikas83pal assigned!
Now that we've clarified the requirements, I’d like both of you to discuss and collaborate on creating a new logo. Please share your ideas and mockups once you’ve come to a consensus. Looking forward to seeing your creative designs!

Till now I was only working on creating favicon icon but now as you mentioned logo I will look forward. @vikas83pal https://discord.com/invite/wG8CcwKN please join discord

@vikas83pal branch having conflicts

@thakuratul2 in my repo it is working very fine

@vikas83pal branch having conflicts

merge with bug-99

@thakuratul2 please reopen the issue
and add label and tags


test(dtslint): add zipAll

Description:
This PR adds dtslint tests for zipAll.
Related issue (if exists): #4093

Pull Request Test Coverage Report for Build 7753

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.805%

Totals

Change from base Build 7749:
0.0%

Covered Lines:
5242

Relevant Lines:
5415

💛  - Coveralls

Yeah, that looks wrong, too. I'll have a look at this later - after I've had a coffee. Perhaps the signatures are there due to issues that arise with early versions of TypeScript? Anyway, I think the any signatures should be removed.

Is this good to merge? Or is there something I should address?


[5.0] Package Config : Update mergeConfigFrom use array_replace_recursive

using array_merge will only merge the first dimension array and not take default value from second level of dimension array, I think using array_replace_recursive is better so that user can only override certain value from multidimensional array.
Example :
Default config :
return [
     'first' => [
             'second' => [
                     'param1' => 'value1',
                     'param2' => 'value2'
             ]
     ]
];

User Config :
return [
     'first' => [
          'second' => [
               'param1' => 'value3'
          ]
     ]
];

array_merge result :
return [
     'first' => [
          'second' => [
               'param1' => 'value3'
          ]
     ]
];

array_replace_recursive result :
return [
     'first' => [
          'second' => [
               'param1' => 'value3',
               'param2' => 'value2',
          ]
     ]
];

If using array_merge, param 2 won't included, if using array_replace_recursive it will be included.

This is not the ideal way to handle, it was suggested before and introduced quite a few bad side effects.
If you want param2 to be included, you'll need to handle the merging yourself.
You can read more here (search on issues also brings more):

https://github.com/laravel/framework/issues/6088
https://github.com/laravel/framework/pull/7141

Ah you're right, it would be a trouble if the config use number index, so the merging should be handled by the package if not the package user has to write all configuration?

We're not using recursive merging on purpose. :)

Hi @GrahamCampbell, so the configuration file for package should be only one file?
Because I've used laravel package that is migrating from laravel 4 to 5, in laravel 4 it has 3 configuration files in 1 folder, but using laravel 5, LoadConfiguration use filename as index in config so the config can't be grouped as 1 config but become 3 indexes.
And then I try to merge the configuration become 1 file as multidimensional array, is this the correct way to write package configuration file? Please advice.
Thank you.

so the configuration file for package should be only one file?

You can have as many as you want. That's unrelated to this?

Example :

laravel-stapler

filesystem.php
stapler.php
s3.php

When the configuration loaded it becomes :
$config['filesystem'];
$config['stapler'];
$config['s3'];

To make it more organize I merge it into 1 file with multidimensional array :
config/laravel-stapler.php
return [
     'filesystem' => [
     ],
     'stapler' => [
     ],
     's3' => [
     ]
];

so the config become
$config['laravel-stapler.filesystem'];
$config['laravel-stapler.stapler'];
$config['laravel-stapler.s3'];

But the side effect user need to override all config inside the laravel-stapler.filesystem / laravel-stapler.stapler / laravel-stapler.s3
Or do you have suggestion to make the config more organize and not conflict with another package configuration (I'm afraid if using index that is too common such as filesystem / s3, it will conflict with another package).

Ah, no. I see what you mean. We don't support doing that, and I wouldn't recommend it.

Yes, so do you have guideline for writing package configuration? Is it better using one file with multidimensional array or multiple files with package name as prefix such as :

config/laravel-stapler-filesystem.php
config/laravel-stapler-stapler.php
config/laravel-stapler-s3.php

Yes, so do you have guideline for writing package configuration?

My recommendation is not to prefix things. As long as the config file isn't more than 200 lines, I'd recommend just having it all in one file.

Ok so the package configuration better in one file and using multidimensional array if I want to make it more organize.
Thank you for your help @GrahamCampbell.


How to limit one port only accept one connection?

How to limit one port only accept one shadowsocks connection?

I have the exact same need.  In the wiki I have found this

Limit connections

iptables -A INPUT -p tcp --syn --dport ${SHADOWSOCKS_PORT} -m connlimit --connlimit-above 32 -j REJECT --reject-with tcp-reset

But it doesn't work for me. I have change the  ${SHADOWSOCKS_PORT} to one of my port 8334 and change 32 to 1 because I want to limit this port with only one connection.
iptables -A INPUT -p tcp --syn --dport 8334 -m connlimit --connlimit-above 1 -j REJECT --reject-with tcp-reset

but it doesn't work for me

did anyone find any working solution for this problem?
i have the same problem and i couldn't find any solution for that!


feat: added logout functionality

The logout functionality of the app should be working now.

Everything seems fine closing :)


Use offset_fetch api_version when fetching messages

When switching to the new kayrock client, we noticed that the api_version for the auto_commit of the fetch message is not taken into account.
Therefore, even if using the API version 2 for the offset commit with kayrock, the next fetch will always use the api_version 0
This introduces a divergence in the way the new client can be used compared to the old implementation. As it's up to the user code to provide the correct offset to fetch.

Closed in favor of https://github.com/kafkaex/kafka_ex/pull/450/files


source code of rocm-core

Hello,
I would like to compile ROCm totally from source code on RISC-V, but I can not find all the source codes, e.g. rocm-core. I just found https://repo.radeon.com/rocm/apt/4.5.2/pool/main/r/rocm-core/ for amd64 as .deb, apparently it can not work.
Can anyone help me?
It will also be great if anyone can give me the relative works on ROCm for RISC-V.
Thanks.

There is only a rocm-version.h in the rocm-core.
https://github.com/xuhuisheng/rocm-build/blob/master/meta/rocm-core_<IP_ADDRESS>000-49_amd64/include/rocm/rocm_version.h

There is only a rocm-version.h in the rocm-core. https://github.com/xuhuisheng/rocm-build/blob/master/meta/rocm-core_<IP_ADDRESS>000-49_amd64/include/rocm/rocm_version.h

Thanks for your reply.
Your repo is de facto guide to me.
Have you ever thought of a compilation for ROCm on an unsupported GPU architecture? Or can you give me some tips for the key source code components related to heterogeneous architecture?

@xlindo
Yes, I use my building scripts to support my RX580 - aka. gfx803 with latest ROCm-5.0.2.
https://github.com/xuhuisheng/rocm-gfx803
The mininium components should be ends with MIOpen, they are required by tensorflow and pytorch.

@xlindo Yes, I use my building scripts to support my RX580 - aka. gfx803 with latest ROCm-5.0.2. https://github.com/xuhuisheng/rocm-gfx803
The mininium components should be ends with MIOpen, they are required by tensorflow and pytorch.

So it means once I compiled MIOpen successfully on my platform, the upper framework will work well?
How about ROCm runtime, which components are necessary and hardware dependent?
🤣

@xlindo
Yes, you need 00.rocm-core to 35.miopen, all of them need for tensorflow or pytorch to build and run.
There are three steps:

when you properly build HIP, you can run some hip samples, like square, that make you using GPU to run some real samples.
when you properly build miopen, you can use miopen to run some plan, something like cnn
when you properly build tensorflow/pytorch, you can use python.

Bascly this is my path.

@xlindo Yes, you need 00.rocm-core to 35.miopen, all of them need for tensorflow or pytorch to build and run.
There are three steps:

when you properly build HIP, you can run some hip samples, like square, that make you using GPU to run some real samples.
when you properly build miopen, you can use miopen to run some plan, something like cnn
when you properly build tensorflow/pytorch, you can use python.

Bascly this is my path.

Great! Thanks a lot.
I will try it now.

Hi @xlindo
Hope you got the suggestions/inputs. Please let know if you need anything more, request to close this ticket if you do not need anything more.
Thank you.

@ROCmSupport
Please close it, thx

Thanks @xlindo
I am closing this

Hi, Arch Linux package maintainer here! Any progress on the source code for rocm-core @ROCmSupport? It's required to build pytorch with ROCm backend.


raise error if bibtex fields for Extra are not properly processed

if my hypothesis for the behavior in #664 is correct (that ZBB silently chokes on ac3.bib), then it'd be nice if ZBB could raise a ruckus in such an event.
It might be disorienting for the user (as it was for me) when there is (among other things) no Extra field, though that would be expected of a successful ZBB run.

This was a different error - but the error handling facilities of zotero translators are incredibly limited. All a translator can do is reject the whole import with only the literal message "an error has occurred", which doesn't give the user any info to correct the error.


make scenegraph outliner scrollable

Fixes https://github.com/aframevr/aframe-editor/issues/16

Perfect! thank you


tfds backend fails when env is reset with no steps

If you take the examples/tfds_random_agent_catch.py sample and call break right the env.reset() call, before any actions are taken, then the tfds backend will fail with error message:
Exception has occurred: ValueError
Failed to encode example:
In <Dataset> with name "steps":
In <Tensor> with name "reward":
Dtype object do not match float64
{'steps': [{'action': array(0, dtype=object), 'discount': array(0, dtype=object), 'is_first': True, 'is_last': False, 'is_terminal': False, 'observation': array([[0., 0., 0., 1., 0.],
       [0., 0., 0., 0., 0.],
       [0., 0., 0., 0., 0.],
       [0., 0., 0., 0., 0.],
       [0., 0., 0., 0., 0.],
       [0., 0., 0., 0., 0.],
       [0., 0., 0., 0., 0.],
       [0., 0., 0., 0., 0.],
       [0., 0., 0., 0., 0.],
       [0., 0., 1., 0., 0.]], dtype=float32), 'reward': array(0, dtype=object), 'timestamp':<PHONE_NUMBER>.9312475}]}

Hi @buckleytoby , sorry for the massive delay, I somehow missed this bug.
@sabelaraga , do you know what's wrong here? Is this expected?
Daniel

From the error, I would say that the reward is initialized wrongly with an int instead of a float. Are you still experiencing the error? Thanks!

Hi! Since there was no response from OP, I'll close this issue for now.


[Solved] Edit Task

How to get the hum trigger replay value , when recursion for weekly?
Originally posted: 2015-03-23T13:41:10

Can you clarify please? I'm not sure what a "hum trigger" is nor a "replay value" nor "recursion for weekly".
Originally posted: 2015-03-23T16:52:14

Descupe Google Translate was not as helpful this time .
Basically what I need is to get the value of the highlighted field in the image http://i.imgur.com/vYfxKG5.jpg
I tried using the command "MyTask.Definition.Triggers" and saw that the values are stored in it but could not find the property that show me this value.
Originally posted: 2015-03-23T17:05:15

You are looking for a WeeklyTrigger (see Documentation here) where you will set the DaysOfWeek property for all the days of the week needed and the WeeksInterval property for the interval.
Originally posted: 2015-03-24T13:36:32

Setting this value I found in the documentation , but I need is go the other way .
How to rebuild value of a task already created ?
Originally posted: 2015-03-24T18:47:49

Task t = taskService.GetTask("MyTaskName");
// Assume there is only a single trigger
WeeklyTrigger wt = t.TaskDefinition.Triggers[0] as WeeklyTrigger;
if (wt != null)
   return wt.WeeksInterval;

Originally posted: 2015-03-24T19:58:35

That's it!
You helped me a lot . Thank U.
Originally posted: 2015-03-24T20:04:05


The painting panel is great, but I can't seem to use it as a mask conveniently

After changing the image to the input port, I can't read the generated picture, if I can read the generated image, make a mask on it and output it, I think, it will be great, maybe I didn't find the right way?

If you change the image to input port, then it implies receiving a list with the names of images in the folder as input (in my case, temp, for LoadImage input). I tried to make a mask, but there is confusion in the paths and I have not figured it out yet. Maybe in the near future I will be able to deal with the mask :)
You can change the code to save images for PainterNode to input folder:
painter_node.py
import hashlib
import os
from PIL import Image
import torch
import numpy as np
import folder_paths

class PainterNode(object):
    @classmethod
    def INPUT_TYPES(self):
        # temp_dir = folder_paths.get_temp_directory()

        # if not os.path.isdir(temp_dir):
        #     os.makedirs(temp_dir)

        temp_dir = folder_paths.get_input_directory()
        files = [f for f in os.listdir(temp_dir) if os.path.isfile(os.path.join(temp_dir, f))]
        return {"required":
                {"image": (sorted(files), )},
                }

    RETURN_TYPES = ("IMAGE",)
    FUNCTION = "painter_execute"

    CATEGORY = "AlekPet Nodes/image"

    def painter_execute(self, image):
        # image_path = os.path.join(
        #     folder_paths.get_temp_directory(), image)
        image_path = folder_paths.get_annotated_filepath(image)
        i = Image.open(image_path)
        image = i.convert("RGB")
        image = np.array(image).astype(np.float32) / 255.0
        image = torch.from_numpy(image)[None,]

        return (image,)

    @classmethod
    def IS_CHANGED(self, image):
        # image_path = os.path.join(
        #     folder_paths.get_temp_directory(), image)
        image_path = folder_paths.get_annotated_filepath(image)
        m = hashlib.sha256()
        with open(image_path, 'rb') as f:
            m.update(f.read())
        return m.digest().hex()

painter_node.js
Find next line code in painter_node.js:
formData.append("type", "temp");
and remove or comment him
// formData.append("type", "temp");
Then in LoadImage node select Paint_0.png or other Paint_...,png, and change Mask.

Thanks a lot for the confusion and look forward to your plugin updates in the future   :)

Added the ability to use the Mask Editor, and also changed the save image  folder on INPUT (instead of TEMP).


Updated MapLoading.js

Added a text 'Loading' next to the spin icon

@ERS-Long Thank you. Can you make the 'Loading' text configurable (no hard-coding) and also make the addition of the text element as optional so the original functionality is still available?

Will do.
Thanks,
Sent from my iPhone

On Feb 14, 2019, at 4:25 PM, Tim McGee<EMAIL_ADDRESS>wrote:
@ERS-Long Thank you. Can you make the 'Loading' text configurable (no hard-coding) and also make the addition of the text element as optional so the original functionality is still available?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Looks good. Thank you!


Error submitting jobscript, bsub returns exit code 255

Hey!
Not quite sure if this is the right place for my issue, as I suspect it's more of a cluster issue than a problem with the profile. But maybe someone can still help.
I'm getting randomly failing job submissions  on pipelines that usually work fine. The tracebacks are something along the lines of:
Traceback (most recent call last):
  File "/homes/lukasw/.config/snakemake/lsf_short/lsf_submit.py", line 230, in submit
    external_job_id = self._submit_cmd_and_get_external_job_id()
  File "/homes/lukasw/.config/snakemake/lsf_short/lsf_submit.py", line 216, in _submit_cmd_and_get_external_job_id
    output_stream, error_stream = OSLayer.run_process(self.submit_cmd)
  File "/homes/lukasw/.config/snakemake/lsf_short/OSLayer.py", line 40, in run_process
    completed_process = subprocess.run(
  File "[..]/lib/python3.9/subprocess.py", line 528, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'bsub -M 1000 -n 1 -R 'select[mem>1000] rusage[mem=1000] span[hosts=1]' -o "logs/cluster/[..]/jobid140_5dad74cf-fda6-41db-9965-ac8a0c18ec25.out" -e "logs/cluster/[..]/jobid140_5dad74cf-fda6-41db-9965-ac8a0c18ec25.err" -J "[..]" -q short [..]/.snakemake/tmp.4tc5r3ou/snakejob.core_metaquast.140.sh' returned non-zero exit status 255.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/homes/lukasw/.config/snakemake/lsf_short/lsf_submit.py", line 259, in <module>
    lsf_submit.submit()
  File "/homes/lukasw/.config/snakemake/lsf_short/lsf_submit.py", line 236, in submit
    raise BsubInvocationError(error)
__main__.BsubInvocationError: Command 'bsub -M 1000 -n 1 -R 'select[mem>1000] rusage[mem=1000] span[hosts=1]' -o "logs/cluster/[..]/jobid140_5dad74cf-fda6-41db-9965-ac8a0c18ec25.out" -e "logs/cluster/[..]/jobid140_5dad74cf-fda6-41db-9965-ac8a0c18ec25.err" -J "[..]" -q short [..]/.snakemake/tmp.4tc5r3ou/snakejob.core_metaquast.140.sh' returned non-zero exit status 255.
Error submitting jobscript (exit code 1):

So bsub returns exit code 255, which leads the profile to raise a BsubInvocationError. Since this issue appears sporadically, I am wondering if this could be caused by file system latency (which is quite high on this system). I.e. the jobscript is not available yet at the time of trying to run bsub, would that make sense? Any ideas how I might try and debug this?
Lastly, could the profile help with mitigating this by e.g. waiting a few seconds between creating the jobscript and submitting it or an option to retry submission after a few seconds instead of raising an error at the first failure to submit?
Cheers!

Lastly, could the profile help with mitigating this by e.g. waiting a few seconds between creating the jobscript and submitting it or an option to retry submission after a few seconds instead of raising an error at the first failure to submit?

Yeah, this could be implemented here: https://github.com/Snakemake-Profiles/lsf/blob/1bdd36a7041ae6952ec0278cc0200e5a78842bbe/{{cookiecutter.profile_name}}/OSLayer.py#L39-L46
I guess with a third optional parameter timeout that is the time to wait if for a retry if the first invocation fails. Could also have 2 optional parameters, retries and timeout: number of times to retry and timeout. By default is 0 retries and 0 timeout (i.e. just try once immediately). You'd set different values to the defaults when calling bsub: https://github.com/Snakemake-Profiles/lsf/blob/1bdd36a7041ae6952ec0278cc0200e5a78842bbe/{{cookiecutter.profile_name}}/lsf_submit.py#L216
I am wondering if is ok for you to implement to check if this actually solves the issue and possibly submit a PR fixing this?

Sorry I accidently merged a possible fix. Let's hope it works. Did you want to test it out @W-L? I've also been having the same issue on codon recently and it has been driving me crazy. Let's hope this is the fix

Thanks for the quick replies guys! I'm afraid I think my suspicion was incorrect. I had dotted a few Path(self.jobscript).exists() around yesterday and got True even for the randomly failing jobs. But then again, I'm not sure how the filesystem latency comes about and how to test for it. As a workaround it seems that setting restart-times: lets my pipelines finish eventually. At least as long as this issue does not affect the same job multiple times in a row. But the logs are still littered with these tracebacks.. I might do some more digging if I have time

Ah okay. Yeah, I have a big pipeline at the moment that keeps being impacted by this and I generally just set restart times to 2 and it normally gets everything finished eventually

Even with the current tip of master?

yes but I guess problem was on my side - I needed to specify extra group for the HPC I use. However, I used other input repo, old version of this.

Hello, I'm getting a similar error, and it looks like specific jobs aren't submitted and no err / out files are produced for the rule(s)...? I've used the slurm profile with this snakemake workflow successfully, but I had to switch to a new HPC which uses lsf. The workflow currently requires snakemake 7. The mostly default profile parameters (changes were LSF_UNIT_FOR_LIMITS = 3, restart_times = 2, jobs=300) worked with a simple touch test, but threw errors with the more complex workflow. I'm not really sure where to start except to provide the error. Let me know if more information is needed or which parameters are worth experimenting with.
Traceback (most recent call last):
  File "/gpfs_common/share01/stern/mwvandew/jrst_tst/jrst/Stern_5269/UU_Cfam_GSD_1.0_ROSY/lsf.go_wags/lsf_submit.py", line 242, in submit
    external_job_id = self._submit_cmd_and_get_external_job_id()
  File "/gpfs_common/share01/stern/mwvandew/jrst_tst/jrst/Stern_5269/UU_Cfam_GSD_1.0_ROSY/lsf.go_wags/lsf_submit.py", line 228, in _submit_cmd_and_get_external_job_id
    output_stream, error_stream = OSLayer.run_process(self.submit_cmd)
  File "/gpfs_common/share01/stern/mwvandew/jrst_tst/jrst/Stern_5269/UU_Cfam_GSD_1.0_ROSY/lsf.go_wags/OSLayer.py", line 40, in run_process
    completed_process = subprocess.run(
  File "/usr/local/usrapps/stern/mwvandew/conda/envs/snakemake/lib/python3.10/subprocess.py", line 526, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'bsub -M 40 -n 4 -R 'select[mem>40] rusage[mem=40] span[hosts=1]' -W 400 -o "logs/cluster/fastqs_to_ubam/bucket=map_tst.breed=jrst.sample_name=Stern_5269.ref=UU_Cfam_GSD_1.0_ROSY.readgroup_name=Stern_5269_A/jobid13_6456f528-a963-4cc4-8e2f-ce3a3fcf6149.out" -e "logs/cluster/fastqs_to_ubam/bucket=map_tst.breed=jrst.sample_name=Stern_5269.ref=UU_Cfam_GSD_1.0_ROSY.readgroup_name=Stern_5269_A/jobid13_6456f528-a963-4cc4-8e2f-ce3a3fcf6149.err" -J "fastqs_to_ubam.bucket=map_tst.breed=jrst.sample_name=Stern_5269.ref=UU_Cfam_GSD_1.0_ROSY.readgroup_name=Stern_5269_A" -q serial /gpfs_common/share01/stern/mwvandew/jrst_tst/jrst/Stern_5269/UU_Cfam_GSD_1.0_ROSY/.snakemake/tmp.bamk4xzx/snakejob.fastqs_to_ubam.13.sh' returned non-zero exit status 255.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/gpfs_common/share01/stern/mwvandew/jrst_tst/jrst/Stern_5269/UU_Cfam_GSD_1.0_ROSY/lsf.go_wags/lsf_submit.py", line 271, in <module>
    lsf_submit.submit()
  File "/gpfs_common/share01/stern/mwvandew/jrst_tst/jrst/Stern_5269/UU_Cfam_GSD_1.0_ROSY/lsf.go_wags/lsf_submit.py", line 248, in submit
    raise BsubInvocationError(error)
__main__.BsubInvocationError: Command 'bsub -M 40 -n 4 -R 'select[mem>40] rusage[mem=40] span[hosts=1]' -W 400 -o "logs/cluster/fastqs_to_ubam/bucket=map_tst.breed=jrst.sample_name=Stern_5269.ref=UU_Cfam_GSD_1.0_ROSY.readgroup_name=Stern_5269_A/jobid13_6456f528-a963-4cc4-8e2f-ce3a3fcf6149.out" -e "logs/cluster/fastqs_to_ubam/bucket=map_tst.breed=jrst.sample_name=Stern_5269.ref=UU_Cfam_GSD_1.0_ROSY.readgroup_name=Stern_5269_A/jobid13_6456f528-a963-4cc4-8e2f-ce3a3fcf6149.err" -J "fastqs_to_ubam.bucket=map_tst.breed=jrst.sample_name=Stern_5269.ref=UU_Cfam_GSD_1.0_ROSY.readgroup_name=Stern_5269_A" -q serial /gpfs_common/share01/stern/mwvandew/jrst_tst/jrst/Stern_5269/UU_Cfam_GSD_1.0_ROSY/.snakemake/tmp.bamk4xzx/snakejob.fastqs_to_ubam.13.sh' returned non-zero exit status 255.
Error submitting jobscript (exit code 1):

Just from this error message, I'm not sure what causes this. But here are three ideas of things you could check:

Make sure you are using the latest version of the master branch, as suggested above. It contains this waiting mechanism for the jobscript, so maybe this helps already?
Try to run snakemake with --verbose to hopefully get a more detailed error message from the traceback?
Try to run the bsub command manually, maybe with some kind of dummy jobscript that you create. Then you can maybe get a more detailed error message.

Thank you, I think I got it. It was an HPC specific thing. I set a default queue, but that queue only accepts single processor jobs, it threw a hissy fit on rules that asked for > 1 processor. Turns out my HPC recommends no specific queue name and chooses the best queue given the resources requested. The 3rd idea helped me find the problem. Thanks again.


Update configuration file to new Autorest

After PR https://github.com/Azure/autorest/pull/2151, the configuration file needs to be updated.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1082   +/-   ##
=======================================
  Coverage   56.98%   56.98%
=======================================
  Files        2001     2001
  Lines       56024    56024
=======================================
  Hits        31927    31927
  Misses      24097    24097

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 9479cda...ddd982f. Read the comment docs.


etcdctlv3: snapshot/recover design

Snapshot
Snapshot command should get a point-in-time consistent snapshot of the v3 KV space
Snapshot command should not snapshot any configuration related data like clusterID or memberID.
TODO: handle lease?
TODO: support incremental snapshot?
Recover
Recover command should write configuration data provided by user like (--initial-cluster, --initial-token, etc..) and create correct member dir.
Recover command should take a snapshot and put it into the right location in the created member dir.

/cc @heyitsanthony

@xiang90 it looks to me like recover command is roughly what we called set-config command here.
Will recover do integrity verification of the snapshot and return a zero exit code only if it's good? Is that even necessary?

it looks to me like recover command is roughly what we called set-config command here, plus some additional logic to set up the data directory?

Right.

Will recover do integrity verification of the snapshot and return a zero exit code only if it's good? Is that even necessary?

Probably we can add a --verify flag? So etcdctl can help to verify the snapshot. Or the verification will delay to server startup time. On the other side, we do expect user to verify snapshot after they store it somewhere. So a force verification seems to be redundant for some use case.

@xiang90 how much validation is done on etcdctl backup? Assuming that whatever backup creates with exit code 0 is "gauranteed" to be valid for restore on any etcd 2.x, the --verify flag would be redundant. Verifying the snapshot hasn't been corrupted between backup and restore is out of scope, for sure.

@colhom etcdctl backup guarantees to return a good snapshot when exit with 0.

Ok then, imho we don't need a --verify flag for restore.

@colhom I feel so. Thanks!

I think auth and alarms would need special handling too.
Maybe it'd make sense to have a special etcd KV space and use the KV machinery already in place (watchers, get) instead of managing custom buckets?

@heyitsanthony In most of the cases (if not all), users just want to backup the static data not dynamic ones like leases, watcher or alarm I think.

@heyitsanthony Shall we close this one?

@xiang90 yeah seems like it's all hammered out


Refactoring of instrumentation tests

Espresso tests contained quite a lot of repetitive low-level code for setup, I've moved it into helper methods, saved 500 lines of code.

I signed the CLA.


Refactor delegate endpoint signatures to use EIP712

Currently we require to sign a hash, which is really wrong, and we should refactor the delegate signature to use an EIP712, similar to what we do for the delete transactions endpoint.

Duplicate of https://github.com/safe-global/safe-transaction-service/issues/1857#issuecomment-1934416322

Looks good to me after checking with Felipe. I have left the checks for the v1 endpoints to the mobile team as a staging build of the app is needed to verify that delegation is working as expected.
Cheers


[Bug]: Non existing route sylius_admin_ajax_all_product_variants_by_phrase

Plugin version
1.0.0-alpha
Sylius version
1.10
What happened?
This URL /admin/orders/1000600059/edit produces this error on Sylius 1.10
Relevant log output
No response

Now that the route is defined a new exception occurs:
 The parameter "sylius.ajax.product.variant_autocomplete_limit" must be defined.


feat: add lotsByArtistsYouFollow

The type of this PR is: Feature

This PR resolves [CO-]
Description
Adds lotsByArtistsYouFollow on OwnerType enum to start tracking page views of the Lots By Artists You Follow screen in the app.
Slack thread for more context -> here
Eigen PR -> here

PR Checklist (tick all before merging)

[x] If I've added a new file to the tree I've exported it from the common index.ts
[x] I've added comments with examples for any new interfaces and ensured that they're in the docs
[x] No platform-specific terminology has been used outside of click and tap (platform is inferred by the DB storing events)

LGTM 😊

:rocket: PR was released in v4.150.0 :rocket:


Rebase to Go 1.6

Signed-off-by: cyli<EMAIL_ADDRESS>

Thanks!

LGTM
/cc @jfrazelle

oh jeez so old so sorry LGTM

@jfrazelle No worries!  thank you!  :D

next time feel free to just throw something :P
On Thu, Mar 3, 2016 at 11:56 AM, Ying Li<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle No worries! thank you! :D
—
Reply to this email directly or view it on GitHub
https://github.com/docker/go/pull/4#issuecomment-191935417.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

Stealing this thread to ping maintainers: this is now two golang versions behind, has this package been deprecated for something else in the meanwhile? If not, any chance to get a rebase against 1.8+?
Thanks!
(a random observer who landed here via OCI)

@lucab Opened https://github.com/docker/go/pull/6


feat: Resource base class

Added cdk.Resource and cdk.IResource (currently placeholders)
as bases for all constructs that represent AWS resources.
awslint rules:

resource-class-extends-resource
resource-interface-extends-resource

Apply across the library.
BREAKING CHANGE: cloudformation.CustomResource no longer extends CfnCustomResource.

ssm.ParameterProps renamed to ssm.ParameterOptions.

Pull Request Checklist

[x] Testing

Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[x] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated

[x] Title and Description

Change type: title prefixed with fix, feat will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[x] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

@eladb will this be updated https://github.com/awslabs/aws-cdk/blob/master/design/aws-guidelines.md?
Why are ImportedXxx classes not extending Resource (but sometimes well because they extend XxxBase which extends Resource)?
Thanks

Yes, we are finalizing the guidelines and will update the guidelines with all the new goodies, including linters for most of the guidelines.

In the meantime, here's the draft


Apply the header to the documentation site

The documentation site (currently hosted at https://ds-nihr.vercel.app/) does not show the DS's own NIHR-branded header.
Definition of Done

Apply the DS's own NIHR-branded header component to the documentation site

What does DS's "own NIHR-branded" look like?

NIHR logo usage awaits resolving license usage.


Selecting multiple NamespaceNodes throws 'Unexpected: could not determine node order'

const xpath = require('xpath');
const DOMParser = require('xmldom').DOMParser;

const xml = '<book xmlns:book1="http://example.com/book1" xmlns:book2="http://example.com/book2"><title>Harry Potter</title></book>';
const document = new DOMParser().parseFromString(xml);
const nodes = xpath.select('/*/namespace::*', document);

console.log(nodes[0].localName + ': ' + nodes[0].firstChild.data);
console.log('Node: ' + nodes[0].toString());

Throws
Error: Unexpected: could not determine node order
    at nodeOrder (/test/node_modules/xpath/xpath.js:3021:11)
    at AVLTree.add (/test/node_modules/xpath/xpath.js:3029:10)
    at XNodeSet.buildTree (/test/node_modules/xpath/xpath.js:3147:23)
    at XNodeSet.toArray (/test/node_modules/xpath/xpath.js:3188:23)
    at new XPathResult (/test/node_modules/xpath/xpath.js:4391:20)
    at XPathExpression.evaluate (/test/node_modules/xpath/xpath.js:4316:9)
    at Object.exports.selectWithResolver (/test/node_modules/xpath/xpath.js:4738:26)
    at Object.exports.select (/test/node_modules/xpath/xpath.js:4718:17)
    at Object.<anonymous> (/test/test.js:6:21)
    at Module._compile (module.js:652:30)

The issue is that the function nodeOrder is not able to determine in which order it should return multiple NamespaceNodes, and therefore throws an Error.

I'm sorry that it has taken so long to fix this and I imagine the fix is probably no longer relevant to you, but it is fixed now in version 0.0.34. Thank you for reporting it.


Visualize the attentions of the model！

@HaoZhang534 @SlongLiu @FengLi-ust @IDEACVR
Excuse me...Again, I have a question.
How to Visualize the attentions of the model?
1.how to visualize encoder-decoder multi-head attention weights?
2.how to visualize encoder self-attention weights
Thanks very much!

@HaoZhang534 @SlongLiu @FengLi-ust @IDEACVR Excuse me...Again, I have a question. How to Visualize the attentions of the model? 1.how to visualize encoder-decoder multi-head attention weights? 2.how to visualize encoder self-attention weights Thanks very much!

You need to get the attention map you want and normalize it to 0~255 and then you can save it as an image.


Ask permission before enable 'reduce app usage' feature.

launcher3-alpha-debug#30.zip

The permissions & 'reduce app usage' feature worked as expected. But

The email screen was present
The demo screenshot shown before the user was prompted for permissions had the Siempo name and logo. I think this could confuse old Siempo users, so can we replace that image with the screenshot attached (or similar)?

The permissions & 'reduce app usage' feature worked as expected. But

The email screen was present
The demo screenshot shown before the user was prompted for permissions had the Siempo name and logo. I think this could confuse old Siempo users, so can we replace that image with the screenshot attached (or similar)?

Emai request was removed in other PR #.

2  I'll change the screenshot

Uploading launcher3-alpha-debug#30.1.apk.zip…

Uploading launcher3-alpha-debug#30.1.apk.zip…

The ZIP isn't actually attached to this comment. Guessing the upload didn't finish.

launcher3-alpha-debug#30.1.apk.zip

The updated screenshot looks good, thank you.
The reduce usage feature works, but the flow is bound to confuse some users.
The current behavior:

User clicks the 'Reduce overuse of flagged apps' toggle to enable it
User redirected to Usage access screen, clicks Focus Launcher, toggles 'Permit usage access', and clicks back arrow twice
'Focus Launcher needs your permission' appears
User clicks 'ENABLE', then 'GOT IT'
User enables 'Allow display over other apps', hits back arrow, then goes back to home screen

At this point, the user will assume that the reduce app usage feature is enabled, but it's not -- just the required permissions have been gained. The user would actually need to go back into the settings and click the toggle button again.
I'd suggest:

redirecting using back to the 'Reduce app usage' screen and enabling the 'Reduce overuse of flagged apps' . That way the user can verify that it's enabled now

That said, the feature does work currently. Some users will be bound to figure it out, others won't. That's OK for now.  If you don't have time to update the flow, we can push it off until after getting the app on the play store.

Messaged Denis/zed on Discord and agreed to leave as is and tackle the menu flow in another issue.


Remove throws

This is an attempt to make the codebase more functional by removing throw statements as much as possible, and work with Try[_] instead of having a bunch of functions that throw and a Try at the very top.
This PR addresses the decoding part. It changes the signature of some public functions (like all the validate). Most people shouldn't be using them too much anyway but it's still a breaking change that might force users to adapt their code when upgrading.
Encode still throws a lot, that would be a separate work.
As it's an important change, I'm open to feedback and discussion, I won't be merging it right away.

Closing for now, Scala 3 bringing checked exception we might take a different route


Appstudio update test-component-pac-dmut

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-dmut-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
13 seconds
init

✅ Succeeded
40 seconds
clone-repository

✅ Succeeded
35 seconds
sast-snyk-check

✅ Succeeded
2 minutes
build-container

✅ Succeeded
12 seconds
inspect-image

✅ Succeeded
2 minutes
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
8 seconds
deprecated-base-image-check

✅ Succeeded
40 seconds
sbom-json-check

✅ Succeeded
8 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-dmut-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
26 seconds
build-container

✅ Succeeded
7 seconds
sast-snyk-check

✅ Succeeded
39 seconds
sbom-json-check

✅ Succeeded
11 seconds
inspect-image

✅ Succeeded
8 seconds
deprecated-base-image-check

✅ Succeeded
13 seconds
clair-scan

✅ Succeeded
41 seconds
clamav-scan

✅ Succeeded
9 seconds
show-summary

✅ Succeeded
7 seconds
show-sbom


How to hide prev and next button?

Hello,
I want to hide the prev and next button, but when I don't write button tag, cause ERROR TypeError: Cannot read property 'nativeElement' of undefined error. Is there any option or method to hide these button? I didn't found it at documentation.
Thanks for your love and support

For what reason you want to hide the buttons. have you tried to hide with css

Thanks for your reply.
I think mobile screen is too small to show much buttons. I wrote style="display: none" to button tag before open this issue, but no use. I tried to wrap button with div tag just now, and button didn't show any more.

Would be nice if there is a better option to not use the buttons. When put a (hacky) *ngIf="1==2" on the buttons, the points aren't displayed to. If it's possible, make the buttons optional and let me add an element to show the points.

Yes, but thats cheating

How to hide buttons for desktops?
Grid >= md


add comment showing the unrolled loop of the huge-apps Router

I love the huge-apps example. It gives a good idea about building applications larger than the standard todo list. The recursive directory call was easy to understand but I got a lot more value from unrolling the loop to see just what this looks like all put together.

The idea is the folders show the hierarchy, but yeah this is helpful. thanks.


Show failed elements in status area

In GitLab by @tristanvb on May 15, 2017, 10:58
When running a pipeline, it's possible that a build or fetch failed but that the user decided to continue building any non failing elements; since we have space in the status area to count the elements (total, session, built, fetched, etc), it makes sense to also show the number of elements which have failed.

In GitLab by @tristanvb on Jul 26, 2017, 11:17
This is implemented.

In GitLab by @tristanvb on Jul 26, 2017, 11:17
closed


Add a basic GitHub action for CI

The title says it all.
Please review.

Most probvably superseded by #7


Breaks Facebook

First, thank you for this! It would be nice not to have to open Chrome simply to sign into my U2F-enabled services.
When I have the extension enabled in Safari, Facebook experiences issues. Sometimes the page freezes and I get the beach-ball of death, other times it just loads very slowly with Safari consuming significant CPU. In all cases so far, I can simply close the tab and go about my business (that is, it's not bring Safari down entirely). Disabling the extension makes Facebook operation return to normal.
Safari 10.0.1
macOS 10.12.1
Let me know if there is any additional information I can provide.

I can verify that Facebook is still broken on 10.11.6 running Safari 10.1.1, though not so much as to render it completely useless. It no longer loads past the second timeline article, though messenger and all the notifications still work and I can directly view articles.
I'm unfamiliar enough with how the U2F extension works that I wouldn't know where to start as far as digging up why, so if you have suggestions on where I could look on the debug console, I'll be glad to.

Providing some technical details on this one… When navigating to Facebook in Safari with the extension installed, I see the following two errors in Web Inspector:
[Error] SyntaxError: JSON Parse error: Unexpected EOF
	parse (u2f.js:135)
	(anonymous function) (u2f.js:135)
[Error] SyntaxError: JSON Parse error: Unexpected identifier "setImmediate$0"
	parse (bridge.js:6)
	(anonymous function) (bridge.js:6)

The first error generally shows up twice. The second error shows up repeatedly and at some regular interval while on a Facebook page.
The JSON parse error is coming from line 6 of bridge.js and line 139 of u2f.js:
var data = JSON.parse(e.data);

I hope that provides a little bit of extra background and is helpful in debugging the Facebook issue.
Software Versions

macOS Sierra 10.12.6
Safari 10.1.2 (12603.3.8)
Safari-FIDO-U2F 1.0 Beta0

☝ Apologies for the noise. It seems there's an active conversation on #4 around similar errors.

This is also killing a particular Apple ID login page: https://idmsa.apple.com/IDMSWebAuth/signin.html?appIdKey=6ffd461b658f7972a60536ab99ab86365a77801c114fa496fd77c9198c6d2093&language=GB-
EN&iso-lang=prod&path=/uk/en/signIn
It causes a massive number of errors, which the Apple website then helpfully attempts to send to Apple's servers.


Create list of places

A list with all places marked on map.

[ ] Create a model for Places

[ ]  List this places (mocked for while)


[gke benchmarks] Reenable dotnet GKE benchmarks

PR https://github.com/grpc/test-infra/pull/389 updates the docker images to fix benchmarks failing after upgrading to .NET 8 RTM. It needs to be merged first.

Sorry for the delay. If all goes well, you should see new benchmark data in the next 4 hours or so.


Recent Transactions only show sent payments

Tested only on Windows 10, Windows 2012 R2
Recent Transactions card on the home screen only displays recent SENT transactions and not received. If this is deliberate the wording should be changed as it is mis-leading

This section actually shows a number of things:

Outgoing transactions
Incoming transactions
Deleted transaction proposals
Rejected transaction proposals
New transaction proposals
Info on transaction proposal acceptions

That being said, I think it's probably worth changing the title of this section


Node.js versions for CI

Short question, is there a reason for this version instead of using 10?
https://github.com/uber/deck.gl/blob/master/.travis.yml#L25

When we were adding the node.js 10 in June, 10.4.0 was the most latest release, so we just add it at that time.

Well but 10 is always the latest 10.x. So I think we can set it to 10.


Filters of separate tabs changing each other

Describe the bug
When using filters of the same type in multiple tabs of the same project, they affect each other. Changing one filter also changes filters of the same type in other tabs.
To Reproduce
Steps to reproduce the behavior:

Go to datamanager
Create two tabs
Apply a filter to the first tab (e.g. Completed is empty no)
Apply filter of same type but different value to other tab (e.g. Completed is empty yes)
Switch between tabs a few times

Expected behavior
Filters in different tabs should be independent
Screenshots
https://user-images.githubusercontent.com/87806894/130771913-0caa2d72-5df9-4184-b456-0266ee2fdc4a.mp4
Environment (please complete the following information):

OS: LS hosted on Linux and accessed on Windows
Label Studio Version 1.2

this issue was recently fixed in DM repo, expect a fix to be shipped with the upcoming release


React Native Vector Icons issue toolbar-android could not be found

I believe that the react-native-vector-icons dependency needs to be updated to 7.0.0 as react-native-vector-icons have removed Icon.AndroidToolbar which is causing the issue.

Hi,
So sorry for that. I was playing with the GitHub iOS app and managed to get rid of all notifications...so I missed this.
It's merged and will be released today.
Cheers


When is Virtual Dressing coming?

Hi, I was just going through your checklist and I saw virtual dressing in it. So, I am curious to know when it's coming. Are you guys working on it?

+1


Always tunnel Kube API client through master ssh

Fixes #1088
K8s Client is now always tunneled through the SSH connection to primary master host.
Done via making ConfigureClient to actually configure the client, not just load the configuration.
If the master host is localhost, don't try to forward.

Worker up does not use k8s-client.


Can't subscribe to private channel with latest versions of socket.io-client

Can't subscribe to private channel with latest versions of socket.io-client.
With socket.io-client^3.0.0 when i try to subscribe private channel i'm getting error "Bad handshake method". Downgrading client to ^2.3.0 solves the problem.
Solution is to update laravel-echo-server to support socket.io-client^3.0.0.

Thanks for reporting. There are some breaking changes in Socket.io  3.0 so please I will investigate this as time permits. Thanks

I have a fork here that is already ported to v3 and was made to be Docker-ready: https://github.com/renoki-co/echo-server
However, I don't have a bad handshake method, but I do have an issue regarding reconnection: https://github.com/laravel/echo/issues/295

I have a fork here that is already ported to v3 and was made to be Docker-ready: https://github.com/renoki-co/echo-server
However, I don't have a bad handshake method, but I do have an issue regarding reconnection: https://github.com/laravel/echo/issues/295


Device/screen lock

Is there a way to use this plugin to directly check if a device/screen lock is enabled? I see the isDeviceSecure() method in Java, but I don't think it's exposed through the API and I don't see an equivalent in iOS.
The desire is that there would be a way to check if the device has support for securely storing encrypted values. For both Android and iOS, this requires a user-configured device lock to be in place. This API would be used prior to set/get/other calls being used.

Hello @l3ender , please check this section of the Readme for an example: https://github.com/mibrito707/cordova-plugin-secure-storage-echo#users-must-have-a-secure-screen-lock-set


Release/upstream v0.13.5 16

I had to do the following:

Merge upstream changes for our Complement fork in this PR.
Disable 3 of Sytest presence intentionally-failing tests (after they were whitelisted by accident in one of our PRs), which Kozi disabled right after implementing the custom Connnect presence logic in this commit.
Disable helm charts installation in CI which were skipped up until now, and don't work due to env differences (not sure exactly what fails them, but we really don't need them as we have our own CD and we don't use helm charts for Dendrite).
Resolve conflicts from Dendrite upstream v0.13.5, and integrate m.login.application_service flow.

👍


android components are not smooth on GVRSceneObject

i added the recycler view to my GVRViewSceneObject but the components like ImageView and Button edges are not smooth and i can see aliasing(nasty little-jagged edges) all over edges, some kind of lines moving on edges, it's making eys strain very quickly.
Is there any params i can add to minimize this effect ..?

another user had some hints in this thread: https://github.com/Samsung/GearVRf/issues/1155

Hi @spakanati123,
@liaxim may give you some RenderData's configuration.
But could you try
https://developer.android.com/studio/write/draw9patch as background?
9patch may give you a good result as well.

Hi @spakanati123 could try the thread https://github.com/Samsung/GearVRf/issues/1155 mentioned by @thomasflynn and send us any feedback?

@ragner the conclusion on the thread #1155 is to use the layout size in proportion to quad size.
700 pix layout ~ 1F quad size.
But this is not the good idea as far as i know, we can not hardcode layout values which is not a recommended approach.

@spakanati123 While in general Android development that is true, using Android views in VR is a special case which merits considering breaking the rules. Thread #1155 has pretty much everything we have on the topic. You could try applying only the convolution shader and see how that works for you.

@spakanati123 Assuming it is all good. Planning to close in few days.


Is it possible to force supply to save after uploading screenshots and metadata to Google Play between each language upload?

Feature Request
Motivation Behind Feature
When uploading lots of languages and localized screenshots for each language if supply fails after uploading most of the languages, it will ditch/reset all progress.
Feature Description
Add an option, or update its default behavior, to perform a save/upload after each language.

More time added to the overall screenshot/metadata upload process.
Alternatives or Workarounds
Only work around I have right now is:

Create metadata-done and metadata-pending folders.
Moving  5 to 8 languages at a time I want to upload to metadata-pending.
Start the upload.
Pray nothing fails due to connection dropping or Google API errors.
When it's done I move all the language folders in metadata to metadata-done.
Repeat steps 2 through 5 until metadata-pending is empty.

What are the reasons why supply files regularly?
How would you resume after it failed and the first batch of changes were already saved?

@xhairbait What's the error message you are getting and/or why is supply failing?

Sometimes if a screenshot fails to upload due to a time out on multiple attempts or if a screenshot resolution is incorrect, it will fail to save any screenshots or metadata that it has staged an upload to Google.
When I say "staged" what I've seen is that terminal states files are being uploaded, but they aren't saved until the entire screenshot and metadata process has completely been uploaded to Google.

One error I got was that there was a Google API error. Very vague and not specific to what went wrong. Just retrying with only some language folders worked just fine. It eventually worked for every folder (without editing it's contents).

@janpio
For you question about what I would do if there was a failure, would be to only have certain language folders in the metadata folder when trying to upload (~6 with a stable internet connections seems to be safe and not an utter waste of time if there is some sort of failure that occurs).

This is also an issue when working with a team and someone modifies the Google Play Store listing in any way. Which interrupts the upload. Therefor, reverting all progress. :'(

@xhairbait Unfortunately that's a "feature" and the standard behaviour from the Google Play Publishing API. Each edit is only valid for a specific time and changing something on the web overrides it. (Basically creates a new one and the old one is no longer valid)

Oh this was definitely meant as a feature request, rather than a bug in fastlane/supply.


Debug symbols for releases

Hi,
Is the debug symbols for each release available somewhere?
Thanks,
Peter

They are not, the automated builds are cross-compiled on Linux using the mingw compiler suite so I can't provide PDBs.


Remove HMP from optimum-habana

HMP is deprecated in favor of PyTorch autocast

removed hmp usage
removed setting autocast env variables through GaudiConfig
updated tests
updated docs
updated README.md files

@jwieczorekhabana Thanks for opening this PR! I'll review it in the beginning of next week and I'll try to accelerate on merging the autocast PRs.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@jwieczorekhabana will the Gaudi config for Wav2Vec2 be the following?
{
  "use_fused_adam": true,
  "use_fused_clip_norm": true,
  "use_torch_autocast": true
}

Or will there be any specific bf16 ops?

@jwieczorekhabana will the Gaudi config for Wav2Vec2 be the following?
{
  "use_fused_adam": true,
  "use_fused_clip_norm": true,
  "use_torch_autocast": true
}

Or will there be any specific bf16 ops?

That'll be it. There is a hanging PR on HuggingFace https://huggingface.co/Habana/wav2vec2/discussions/2/files that can be merged after 1.13 release.
Similarly for gpt2 there is a closed PR that can be reopened https://huggingface.co/Habana/gpt2/discussions/4/files
Release 1.13 addresses issues both of those topologies had.

Cool, I'll merge them when 1.13 is released then :+1:

@jwieczorekhabana Can you rebase this branch on main? There are merge conflicts in tests/test_diffusers.py so GitHub won't let me merge the PR when 1.13 is released.

@jwieczorekhabana Can you rebase this branch on main? There are merge conflicts in tests/test_diffusers.py so GitHub won't let me merge the PR when 1.13 is released.

@regisss Done

@jwieczorekhabana It seems there are still some merge conflicts in tests/test_diffusers.py

@jwieczorekhabana I pushed a few commits to solve the CI that was not passing


Revisit logic for determining min/max for variables

A team member will update this issue with more details soon.

This is your friendly Microsoft Issue Bot. I created this issue automatically as requested by a team member.


Cant add item

Hi, i cant add item with this tools. And all information is default like level, platime, etc

Which game version are you using? CODEX?

Which game version are you using? CODEX?

steam version 1.0

Latest version of this tool only works with game version <IP_ADDRESS>. If you are running <IP_ADDRESS> latest Gaget version won't work. If that is your case you could update your game or use one of the old releases: Gadget-1.0.0.


add confy.app

add confy.app

a brazilian newcomer, developed by FotonTech - more info about them on linkedin 🇧🇷

idk if they're planning to open their source, but it means something: ConfyApp on GitHub

no worries, assuming commercial for now - shall wait and see


⚠ 366.ru has degraded performance

In b5507a2, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 5391 ms

Resolved: 366.ru performance has improved in 0cbdcfe.


Add PyCon 2019 information

Adding table entry for PyCon 2019 using information from its website https://in.pycon.org/2019/ . Information on Scholarships has been set to No as no information on it is available on the site yet.

Details of the conference:

Website: https://in.pycon.org/2019/
Date:  October 12 - 15, 2019
Venue:  Chennai Trade Center and IIT Madras Research Park
Description: India's premier conference on using and developing the Python programming language

CoC : https://in.pycon.org/2019/code-of-conduct.html
CFP is open now : https://in.pycon.org/2019/cfp.html
Thanks for the repo and all the details :)


Add ALE to the README Installation section

ALE Now supports bash-language-sever out of the box, if installed, and requires no configuration for it to start running.

Nice! Do you have time to submit a PR? :)

I don't have the time myself.

Submitted here https://github.com/mads-hartmann/bash-language-server/pull/64

Fixed in #64


4.x ingress for security list should have source not destination

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version and Provider Version

Affected Resource(s)
The security list for control plane shows the following:
cp_ingress_seclist = [
    {
      description      = "Allow Bastion service to communicate to the control plane endpoint. Required for when using OCI Bastion service.",
      destination      = local.cp_subnet,
      destination_type = "CIDR_BLOCK",
      protocol         = local.tcp_protocol,
      port             = 6443,
      stateless        = false
    }
    ]

This should be source and source_type

Terraform Configuration Files

# Copy-paste your Terraform configurations here - for large Terraform configs,
# please use a service like Dropbox and share a link to the ZIP file.
# Please remove any sensitive information from configuration files before sharing them.

Debug Output

Panic Output

Expected Behavior

Actual Behavior

Steps to Reproduce

terraform apply

Important Factoids

References

closed in #418


fix: lora+: include lr in optimizer kwargs

Playing with the LoRA+ optimizer, I realized that the underlying optimizer will complain about the required positional lr argument if it is not included in the optimizer kwargs.
In the original implementation, this was actually already the case, but somehow this fell out along the way.

Thanks for fixing this issue. But isn't the better fix to add lr to the kwargs? I.e.
- lr = kwargs["lr"]
+ kwargs["lr"] = lr

Then we can leave lr in the function signature, which is more user friendly than raising a KeyError down the line.

Ah, to make it a forced argument to create_loraplus_optimizer. Yes.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


ember-getowner-polyfill is deprecated

DEPRECATION: ember-getowner-polyfill is now a true polyfill. Use Ember.getOwner directly instead of importing from ember-getowner-polyfill
the package is still in the dependencies

This is not coming from ember-cookies, see https://github.com/simplabs/ember-cookies/search?utf8=✓&q=getowner&type=


site not working

Subject: site not working
Description:
nothing
Results:
nothing
Backend version: develop-bcfd1b0a
Frontend version: develop-6d22ac7f
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Page: https://couchers.org/login
User:

Not enough info


Improve test output

Exceptions in tests (for example from partial functions) cause failure without line number which makes the cause difficult to find.  This PR switches to use functions that accurately convey location of failure in the test.
Use byDeadlineM instead of assertByDeadlineM, which allows specifying a period.  This allows a longer poll time to be used so the test output isn't as spammy.

Replaces https://github.com/input-output-hk/cardano-node/pull/3596

bors r+

bors r+

bors r+

bors r+


This does not work when developing in a container.

Im assuming the path its attempting to open in the container does not exist. Is there a workaround for this?

See my comment here: https://github.com/ArnaudBarre/vite-plugin-react-click-to-component/pull/6#issuecomment-2482674073
This seems to be an issue, but it should be fixed at the Vite level so for example opening files from the error overlay works


Problem getting cookies

I have this problem:
When I do this
s = requests.Session()
res = s.get('http://arenavision.cc')
cookies = dict(res.cookies)
print cookies

I get this correct output message:
{'__cfduid': 'db525e1fd170d9f44a8006f18e58688701568808549'}
But when I do this, in the same python program:
cookie_value, user_agent = cloudscraper.get_cookie_string("http://arenavision.cc")
I get this error message
ERROR:root:"http://arenavision.cc" returned an error. Could not collect tokens.
More things to have into account:
1- Is Debian OS in a Google Cloud account LOCATED IN USA
2- It's not desktop OS, which means I can't watch the original web in a browser
3- When I do "sudo wget http://arenavision.cc" I get
--2019-09-18 12:14:52--  http://arenavision.cc/
Resolving arenavision.cc (arenavision.cc)... <IP_ADDRESS>, <IP_ADDRESS>, 2606:4700:30::681f:5e1d, ...
Connecting to arenavision.cc (arenavision.cc)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 403 Forbidden
2019-09-18 12:14:52 ERROR 403: Forbidden.

4- When I do the request.get --not cloudscraper, just python request-- I get this web in text mode, inside python program:

<!DOCTYPE html>

<title>Attention Required! | Cloudflare</title>
<meta name="captcha-bypass" id="captcha-bypass" />
<meta charset="UTF-8" />
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<meta http-equiv="X-UA-Compatible" content="IE=Edge,chrome=1" />
<meta name="robots" content="noindex, nofollow" />
<meta name="viewport" content="width=device-width,initial-scale=1,maximum-scale=1" />
<link rel="stylesheet" id="cf_styles-css" href="/cdn-cgi/styles/cf.errors.css" type="text/css" media="screen,projec
tion" />

                <form class="challenge-form" id="challenge-form" action="/cdn-cgi/l/chk_captcha" method="get">
  <input type="hidden" name="s" value="1b81a0026a7f5cab5247745693c418800963823b-1568809023-1800-ARy/uLUnNkVQWdy52S3
zG29zaqQketOnk8QBTbCRCKCUTllfD5Jt+FekXziolL2hmg72YYzAcO5dlQp0AsFb7GP3gWYSH+0iNc4DSGGqQD4wF6rCQAs3lAyr/USs8WvaWOitSC
+CfXPNVcsz+GsVfjduRWwVluqsB4hP754s4HQhM8zft8z9e9OJdFZPTe4cgEWskGUpzu3sgt8NtwxuOVc="></input>

        <textarea id="g-recaptcha-response" name="g-recaptcha-response" class="g-recaptcha-response" style="width:
250px; height: 40px; border: 1px solid #c1c1c1; margin: 10px 25px; padding: 0px; resize: none;"></textarea>
        <input type="submit" value="Submit"></input>

</form>

Can you please show user-agent, also what python version are you using? 3.5,3.6,3.7 etc??

I don't think you are showing me the full error, are you sure that the error isnt more along the lines of reCaptcha...

Ok, I have detected a little better the error, and I am now pretty sure that is a bug:
import cloudscraper

scraper = cloudscraper.create_scraper(interpreter='nodejs',debug=True)
scraper.recaptcha={  'provider': 'deathbycaptcha',  'username': 'myuser',  'password': 'mypassword'}

web=scraper.get('http://arenavision.cc')

This code goes OK, with python and with python3, and the final page beyond cloudflare displays correctly.
But this other code goes wrong:
import cloudscraper

scraper = cloudscraper.create_scraper(interpreter='nodejs',debug=True)
scraper.recaptcha={  'provider': 'deathbycaptcha',  'username': 'myuser',  'password': 'mypassword'}

cookie_value, user_agent = cloudscraper.get_cookie_string('http://arenavision.cc')

So the bug is finally in the get_cookie_string function, which is not getting the recaptcha provider given in the scraper instance. I know that I call 'cloudscraper.get_cookie_string' and not 'scraper.get_cookie_string' but before using recaptcha providers it worked.

Ok i'll look that tomorrow if you don't mind, as its almost 10pm here and i only just got home

@Fran008 , if you are encountering a reCaptcha via
python
cookie_value, user_agent = cloudscraper.get_cookie_string('http://arenavision.cc')

you need to pass reCaptcha params...
for example..
cookie_value, user_agent = cloudscraper.get_cookie_string(
    'http://arenavision.cc',
    recaptcha={ 'provider': 'deathbycaptcha',  'username': 'myuser',  'password': 'mypassword'}
)

Also check that the reCaptcha isn't releated to 1.1.43, as there was an issue under ubuntu calling reCaptcha &| addtional challenges to trigger
But... ultimately if you are triggering reCaptcha due to whatever reason, you will need to solve the captcha.

@Fran008
Heres a logic flow if you wanna learn whats going on..

Thank you for the explanations. Do I have to update? With the same version, not updating, I got this error message --because it does not recognize the parameter 'recaptcha' in get_cookie_string:

cookie_value, user_agent = cloudscraper.get_cookie_string('http://arenavision.cc',recaptcha={'provider': 'death
bycaptcha', 'username': 'myusername', 'password': 'mypassword'})
ERROR:root:"http://arenavision.cc" returned an error. Could not collect tokens.
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/cloudscraper-1.1.42-py2.7.egg/cloudscraper/init.py", line 295, i
n get_cookie_string
tokens, user_agent = cls.get_tokens(url, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/cloudscraper-1.1.42-py2.7.egg/cloudscraper/init.py", line 263, i
n get_tokens
resp = scraper.get(url, **kwargs)
File "/usr/lib/python2.7/dist-packages/requests/sessions.py", line 501, in get
return self.request('GET', url, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/cloudscraper-1.1.42-py2.7.egg/cloudscraper/init.py", line 123, i
n request
resp = super(CloudScraper, self).request(method, url, *args, **kwargs)
TypeError: request() got an unexpected keyword argument 'recaptcha'

Let me look into that mate 👀

Uhhh....
>>> import cloudscraper
>>> cloudscraper.get_cookie_string('http://arenavision.cc',recaptcha={'provider': 'deathbycaptcha', 'username': 'myusername', 'password': 'mypassword'})
('__cfduid=da1982b0facd8f2b7bb6ae8106a6f72371568962804; cf_clearance=3eac074a6a3caa47cd12ff1139f352ebc4c950c7-1568962804-1800-150', 'Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:60.6.3) Gecko/20100101 Firefox/60.6.3')
>>>

Can you please
import cloudscraper
cloudscraper.__version__

and show me? thanx...

Python 3.5.3 (default, Sep 27 2018, 17:25:39)
[GCC 6.3.0 20170516] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import cloudscraper
>>> cookie_value, user_agent = cloudscraper.get_cookie_string('http://arenavision.cc',recaptcha={'provider': 'death
bycaptcha', 'username': 'Fran008', 'password': '12340007aA@'})
ERROR:root:"http://arenavision.cc" returned an error. Could not collect tokens.
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.5/dist-packages/cloudscraper-1.1.42-py3.5.egg/cloudscraper/__init__.py", line 295, i
n get_cookie_string
    tokens, user_agent = cls.get_tokens(url, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/cloudscraper-1.1.42-py3.5.egg/cloudscraper/__init__.py", line 263, i
n get_tokens
    resp = scraper.get(url, **kwargs)
  File "/usr/lib/python3/dist-packages/requests/sessions.py", line 501, in get
    return self.request('GET', url, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/cloudscraper-1.1.42-py3.5.egg/cloudscraper/__init__.py", line 123, i
n request
    resp = super(CloudScraper, self).request(method, url, *args, **kwargs)
TypeError: request() got an unexpected keyword argument 'recaptcha'
>>> cloudscraper.__version__
'1.1.42'

Look at the last line, including version. I'll make the test with python2 too --see next comment

Python 2.7.13 (default, Sep 26 2018, 18:42:22)
[GCC 6.3.0 20170516] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import cloudscraper
>>> cookie_value, user_agent = cloudscraper.get_cookie_string('http://arenavision.cc',recaptcha={'provider': 'death
bycaptcha', 'username': 'Fran008', 'password': '12340007aA@'})
ERROR:root:"http://arenavision.cc" returned an error. Could not collect tokens.
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/cloudscraper-1.1.42-py2.7.egg/cloudscraper/__init__.py", line 295, i
n get_cookie_string
    tokens, user_agent = cls.get_tokens(url, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/cloudscraper-1.1.42-py2.7.egg/cloudscraper/__init__.py", line 263, i
n get_tokens
    resp = scraper.get(url, **kwargs)
  File "/usr/lib/python2.7/dist-packages/requests/sessions.py", line 501, in get
    return self.request('GET', url, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/cloudscraper-1.1.42-py2.7.egg/cloudscraper/__init__.py", line 123, i
n request
    resp = super(CloudScraper, self).request(method, url, *args, **kwargs)
TypeError: request() got an unexpected keyword argument 'recaptcha'
>>> cloudscraper.__version__
'1.1.42'

And OS version:
uname -r
4.9.0-8-amd64

And OS version:
uname -r
4.9.0-8-amd64

that is the kernel version... not the "os version"

Might want to change your password to dbc... as you posted it..

Thank you, better keep password safe ;)

Can you change https://github.com/VeNoMouS/cloudscraper/blob/master/cloudscraper/init.py#L258 in /usr/local/lib/python2.7/dist-packages/cloudscraper-1.1.42-py2.7.egg/cloudscraper/init.py
to
field: kwargs.pop(field, None) for field in ['allow_brotli', 'browser', 'debug', 'delay', 'interpreter', 'recaptcha'] if field in kwargs

Sorry... OS version:
Distributor ID: Debian
Description:    Debian GNU/Linux 9.6 (stretch)
Release:        9.6
Codename:       stretch

Operating System: Debian GNU/Linux 9 (stretch)
Kernel: Linux 4.9.0-8-amd64
Architecture: x86-64``

Your solution has worked!
Now it goes fine in python2.7 having changed line 258.
Thank you very much!

ok so something weird with attributes vs keys ... its better to use the way i posted, so i'll make that change public.

There's a new related issue...
When I do the cloudscrape with the deathbycaptcha provider it always get the tokens... but when I do without provider it RANDOMLY fails, not always. I have made a loop to retrieve the cookies without failing, but it's a workaround a little bit ugly... specially because the web I want to scrape blocks you after some attempts.
I will try to gather more info... but please don't close this issue until we clarify this.

It seems that the difference is not because of the provider... it's because of the interpreter: it's working well if in my program I change the line
scraper = cloudscraper.create_scraper()
with
scraper = cloudscraper.create_scraper(interpreter='nodejs')

Mmm... no, it fails sometimes too. And it's randomly. With captcha providers works well.

You should be using the same session for your requests, or storing your cookies away and reloading them when needed if you are doing individual requests outside a shared session.
If there was a issue with IUAM challenges, everyone would be reporting the issue as that is the core
function of this library.

kwargs fix pushed in 1.1.44 (e8994348721c373095d5f5409b20218f06408282)
I am closing this issue off

@Fran008 you do understand what i meant by storing the cookies and re-using rather then constantly solving on every single request?

Yes, I do. And knowing that is a great achievement! Thank you for that.
I will get the cookies once a day and store them in my database. Then I will retrieve them everytime I want to access the web. I guess once or twice a day will be enough.

Mmm... I have tried to store and retrieve cookies from my database and it seems that it isn't enought to keep a session alive. Silly me.
Is there a way to persist this session in time? Or do I have to persist the session ONLY in each instance of my executions? If the correct answer is the second one, I have adapted all my programs to use the same cookies in each call, so I guess it's what you recommended.
If it's possible to persist one session by storing it in a database, I still don't know how to do it.

Make sure you are using the same User-Agent with the cookies...

If it's possible to persist one session by storing it in a database, I still don't know how to do it.

With an upcoming release I want to do, you can pickle dump the whole session, http adaptor with ssl context and all..
that should make your life alot easier
For the moment till that is released, dump your cookies using like pickle.dumps and load them via pickle.loads

>>> import cloudscraper
>>> scraper = cloudscraper.create_scraper()
>>> scraper.get('https://japscan.to')
<Response [200]>
>>> import pickle
>>>
>>> x = pickle.dumps(scraper.cookies)
>>> x
b'\x80\x03crequests.cookies\nRequestsCookieJar\nq\x00)\x81q\x01}q\x02(X\x07\x00\x00\x00_policyq\x03chttp.cookiejar\nDefaultCookiePolicy\nq\x04)\x81q\x05}q\x06(X\x08\x00\x00\x00netscapeq\x07\x88X\x07\x00\x00\x00rfc2965q\x08\x89X\x13\x00\x00\x00rfc2109_as_netscapeq\tNX\x0c\x00\x00\x00hide_cookie2q\n\x89X\r\x00\x00\x00strict_domainq\x0b\x89X\x1b\x00\x00\x00strict_rfc2965_unverifiableq\x0c\x88X\x16\x00\x00\x00strict_ns_unverifiableq\r\x89X\x10\x00\x00\x00strict_ns_domainq\x0eK\x00X\x1c\x00\x00\x00strict_ns_set_initial_dollarq\x0f\x89X\x12\x00\x00\x00strict_ns_set_pathq\x10\x89X\x10\x00\x00\x00_blocked_domainsq\x11)X\x10\x00\x00\x00_allowed_domainsq\x12NX\x04\x00\x00\x00_nowq\x13Jz\xf5\x85]ubX\x08\x00\x00\x00_cookiesq\x14}q\x15(X\x0b\x00\x00\x00.japscan.toq\x16}q\x17X\x01\x00\x00\x00/q\x18}q\x19X\x08\x00\x00\x00__cfduidq\x1achttp.cookiejar\nCookie\nq\x1b)\x81q\x1c}q\x1d(X\x07\x00\x00\x00versionq\x1eK\x00X\x04\x00\x00\x00nameq\x1fh\x1aX\x05\x00\x00\x00valueq X+\x00\x00\x00df243f31ee15ef94d69272b7605a0bafe1569060208q!X\x04\x00\x00\x00portq"NX\x0e\x00\x00\x00port_specifiedq#\x89X\x06\x00\x00\x00domainq$h\x16X\x10\x00\x00\x00domain_specifiedq%\x88X\x12\x00\x00\x00domain_initial_dotq&\x88X\x04\x00\x00\x00pathq\'h\x18X\x0e\x00\x00\x00path_specifiedq(\x88X\x06\x00\x00\x00secureq)\x89X\x07\x00\x00\x00expiresq*J\xf0(g_X\x07\x00\x00\x00discardq+\x89X\x07\x00\x00\x00commentq,NX\x0b\x00\x00\x00comment_urlq-NX\x07\x00\x00\x00rfc2109q.\x89X\x05\x00\x00\x00_restq/}q0X\x08\x00\x00\x00HttpOnlyq1NsubssX\x0b\x00\x00\x00.japscan.coq2}q3h\x18}q4(X\x08\x00\x00\x00__cfduidq5h\x1b)\x81q6}q7(h\x1eK\x00h\x1fX\x08\x00\x00\x00__cfduidq8h X+\x00\x00\x00d2db18f9fc994014942a1ecb0d51d36981569060211q9h"Nh#\x89h$X\x0b\x00\x00\x00.japscan.coq:h%\x88h&\x88h\'h\x18h(\x88h)\x89h*J\xf3(g_h+\x89h,Nh-Nh.\x89h/}q;X\x08\x00\x00\x00HttpOnlyq<NsubX\x0c\x00\x00\x00cf_clearanceq=h\x1b)\x81q>}q?(h\x1eK\x00h\x1fh=h X<\x00\x00\x00a21795001527e9dc0ebaefb5dd24bae54daa50df-1569060216-1800-150q@h"Nh#\x89h$X\x0b\x00\x00\x00.japscan.coqAh%\x88h&\x88h\'h\x18h(\x88h)\x89h*J\x90\n\x86]h+\x89h,Nh-Nh.\x89h/}qBX\x08\x00\x00\x00HttpOnlyqCNsubX\x07\x00\x00\x00sessionqDh\x1b)\x81qE}qF(h\x1eK\x00h\x1fhDh X\x1a\x00\x00\x00v6qi410per4jmou1blfuaqjos9qGh"Nh#\x89h$X\x0b\x00\x00\x00.japscan.coqHh%\x88h&\x88h\'h\x18h(\x88h)\x89h*Nh+\x88h,Nh-Nh.\x89h/}qIubusuh\x13Jz\xf5\x85]ub.'
>>> scraper = cloudscraper.create_scraper()
>>> scraper.cookies = pickle.loads(x)
>>> scraper.cookies
<RequestsCookieJar[Cookie(version=0, name='__cfduid', value='d2db18f9fc994014942a1ecb0d51d36981569060211', port=None, port_specified=False, domain='.japscan.co', domain_specified=True, domain_initial_dot=True, path='/', path_specified=True, secure=False, expires=1600596211, discard=False, comment=None, comment_url=None, rest={'HttpOnly': None}, rfc2109=False), Cookie(version=0, name='cf_clearance', value='a21795001527e9dc0ebaefb5dd24bae54daa50df-1569060216-1800-150', port=None, port_specified=False, domain='.japscan.co', domain_specified=True, domain_initial_dot=True, path='/', path_specified=True, secure=False, expires=1569065616, discard=False, comment=None, comment_url=None, rest={'HttpOnly': None}, rfc2109=False), Cookie(version=0, name='session', value='v6qi410per4jmou1blfuaqjos9', port=None, port_specified=False, domain='.japscan.co', domain_specified=True, domain_initial_dot=True, path='/', path_specified=True, secure=False, expires=None, discard=True, comment=None, comment_url=None, rest={}, rfc2109=False), Cookie(version=0, name='__cfduid', value='df243f31ee15ef94d69272b7605a0bafe1569060208', port=None, port_specified=False, domain='.japscan.to', domain_specified=True, domain_initial_dot=True, path='/', path_specified=True, secure=False, expires=1600596208, discard=False, comment=None, comment_url=None, rest={'HttpOnly': None}, rfc2109=False)]>
>>>

^^ as an example

Great, great, GREAT!!
It's working now storing cookies and user-agent. I just needed the user-agent too. I am making this manually in my database, I won't use pickle by the moment but thank you sooo much for the patience and the info. You're awesome.

All good man, knowledge should be shared :)
Drop me a line if you got any more queries :)

VeNoMouS


Using filtering to create angular data grids

Topic Suggestion
Using filtering to create angular data grids
Proposed article introduction
IgniteUI for Angular Grid's filtering options include Quick filtering, Excel style filtering, and Advanced filtering, allowing users to display just entries that fit specific criteria. The Material UI grid component in Ignite UI provides angular filtering features as well as a comprehensive filtering API via the Data Container to which the Grid is linked.
In this tutorial, we will discuss how to Build an Angular Data Grid With Filtering concept.
Key takeaways

Consider the angular grid component.
Filtering

Importing the component first
Calling the component
Configuring the component
Marking up the rest of the UI
Binding the data

Article quality
This will be a unique article which will help the reader understand the concept of creating angular data grids with filtering by the end of the tutorial.
References
N/A

You can only suggest one topic at the moment.


GetHideValidationDialog should be global

I think, GetHideValidationDialog function should be global on Sales Header table and a new function with the same name should be added to Purchase Header table. When subscribing OnBefore/OnAfterValidate events, it's needed to access HideValidationDialog variable from subscriber function.

We have accepted the request. This function shall be soon made accessible through extensions.

GetHideValidationDialog should be also global / external for Sales Line


compiling SOIL2 with mingw64 gcc version 11.2.0

I am trying to compile SOIL2 with mingw64.
I have installed premake5 and used the following commands:
premake5 gmake
cd make/windows
mingw32-make
after which I get the following error:
mingw32-make
==== Building soil2-static-lib (debug_x86_64) ====
Creating ../../obj/windows/x86_64/debug/soil2-static-lib
mkdir: cannot create directory ‘..\..\obj\windows\x86_64\debug\soil2-static-lib’: No such file or directory
mingw32-make[1]: *** [soil2-static-lib.make:153: ../../obj/windows/x86_64/debug/soil2-static-lib] Error 1
mingw32-make: *** [makefile:45: soil2-static-lib] Error 2
Please help, as I do not want to use MSVC for my OpenGL projects.

mkdir: cannot create directory that looks weird. The issue depends on your own configuration of mingw64 and the whole environment. From the error I don't have much information to help you. May be you can try to use premake4 that it seems more friendly to the mingw enviroments.

@BihartStudio try creating directory using mkdir -p instead of just mkdir


Add support for vegcover and albedo

In 4.2 and 5.0 classic, vegcover optionally can be in the veg_lib and both vegcover and albedo can be in the veg_param file.  This needs to be handled in tonic.
This also involves changing the logic for options - currently there is just a flag for global_lai (or lai_index in some cases) but this needs to be replaced by the set {veglib_vegcover, vegparam_lai, vegparam_vegcover, vegparam_albedo, lai_src, vegcover_src, and albedo_src}.  global_lai would correspond to vegparam_lai = true and lai_src = from_vegparam.

This was addressed by PR #40 but for some reason didn't get automatically closed.  It can be closed now.

Wait, never mind. The reason it didn't get closed was because I said it shouldn't be closed yet. The idea being that this issue pertains to the reverse direction (netcdf to ascii) as well as ascii to netcdf, and this hasn't been addressed yet.

The ascii-netcdf conversion aspect of this issue has been addressed in PR #40.  The netcdf-ascii conversion aspect of this issue has been moved into a new issue #44.  Thus, this issue can be closed.