API for post returning 200

Had working code and now every cwl workflow I try with the post API just returns a 200.  Using the API demo code.

`mport requests
import time
import os
import glob
BASE_URL = 'https://view.commonwl.org/workflows'
#workflowurl = 'https://github.com/curoverse/l7g/blob/master/cwl-version/clean/cwl/tiling_clean_gvcf.cwl'
workflowurl = 'https://github.com/curoverse/l7g/blob/master/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl'
HEADERS = {
'user-agent': 'my-app/0.0.1',
'accept': 'application/json'
}
#workflowurl = "https://github.com/common-workflow-language/workflows/tree/master/workflows/compile/compile1.cwl"
addResponse = requests.post(BASE_URL,
data={'url': workflowurl}, headers = HEADERS)
print(addResponse.status_code)
`

After some more digging it looks like for some reason it is returning the workflow response instead of the standard post response.  Not sure why -- maybe if it goes through the queue too fast and/or already exists or a bug?
{"retrievedFrom":{"repoUrl":"https://github.com/curoverse/l7g.git","branch":"master","path":"cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl","packedId":null,"url":"https://github.com/curoverse/l7g/blob/master/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl","rawUrl":"https://raw.githubusercontent.com/curoverse/l7g/master/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl","type":"GITHUB"},"retrievedOn":1557257157834,"lastCommit":"c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc","label":"Creates a FastJ file for each gVCF by path","inputs":{"aidx":{"label":"Assembly index file","type":"File"},"ref":{"label":"Reference genome","type":"String"},"seqidM":{"label":"Mitochondrial naming scheme for gVCF","type":"String"},"refdirectory":{"label":"Directory of input gVCFs","type":"Directory"},"afn":{"label":"Compressed assembly fixed width file","type":"File"},"reffaM":{"label":"Reference mitochondrial genome in FASTA format","type":"File"},"afnM":{"label":"Compressed mitochondrial assembly fixed width file","type":"File"},"aidxM":{"label":"Mitochondrial assembly index file","type":"File"},"tagdir":{"label":"Compressed tagset in FASTA format","type":"File"},"reffa":{"label":"Reference genome in FASTA format","type":"File"},"refM":{"label":"Mitochondrial reference genome","type":"String"}},"outputs":{"out2":{"label":"Indexed and zipped gVCFs","type":"fb65ad7a692c763547e0a11a65560121[]"},"out1":{"label":"Directories of FastJs","type":"Directory[]"}},"steps":{"step2":{"run":"convertgvcf.cwl","runType":"COMMANDLINETOOL","sources":{"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl#step2/afnM":{"sourceIDs":["afnM"]},"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl#step2/reffa":{"sourceIDs":["reffa"]},"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl#step2/seqidM":{"sourceIDs":["seqidM"]},"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl#step2/ref":{"sourceIDs":["ref"]},"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl#step2/refM":{"sourceIDs":["refM"]},"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl#step2/gffDir":{"sourceIDs":["step1"]},"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl#step2/reffaM":{"sourceIDs":["reffaM"]},"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl#step2/aidx":{"sourceIDs":["aidx"]},"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl#step2/tagdir":{"sourceIDs":["tagdir"]},"step2":{"sourceIDs":["step1"]},"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl#step2/afn":{"sourceIDs":["afn"]},"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl#step2/aidxM":{"sourceIDs":["aidxM"]}}},"step1":{"label":"Create list of gVCF directories to process","run":"getdirs.cwl","runType":"EXPRESSIONTOOL","sources":{"step1":{"sourceIDs":["refdirectory"]}}}},"dockerLink":"https://hub.docker.com/r/arvados/l7g","cwltoolVersion":"1.0.20180525185854","visualisationDot":"digraph workflow {\n  graph [\n    bgcolor = "#eeeeee"\n    color = "black"\n    fontsize = "10"\n    labeljust = "left"\n    clusterrank = "local"\n    ranksep = "0.22"\n    nodesep = "0.05"\n  ]\n  node [\n    fontname = "Helvetica"\n    fontsize = "10"\n    fontcolor = "black"\n    shape = "record"\n    height = "0"\n    width = "0"\n    color = "black"\n    fillcolor = "lightgoldenrodyellow"\n    style = "filled"\n  ];\n  edge [\n    fontname="Helvetica"\n    fontsize="8"\n    fontcolor="black"\n    color="black"\n    arrowsize="0.7"\n  ];\n  subgraph cluster_inputs {\n    rank = "same";\n    style = "dashed";\n    label = "Workflow Inputs";\n    "afn" [fillcolor="#94DDF4",label="Compressed assembly fixed width file"];\n    "reffa" [fillcolor="#94DDF4",label="Reference genome in FASTA format"];\n    "tagdir" [fillcolor="#94DDF4",label="Compressed tagset in FASTA format"];\n    "reffaM" [fillcolor="#94DDF4",label="Reference mitochondrial genome in FASTA format"];\n    "ref" [fillcolor="#94DDF4",label="Reference genome"];\n    "aidx" [fillcolor="#94DDF4",label="Assembly index file"];\n    "afnM" [fillcolor="#94DDF4",label="Compressed mitochondrial assembly fixed width file"];\n    "aidxM" [fillcolor="#94DDF4",label="Mitochondrial assembly index file"];\n    "refdirectory" [fillcolor="#94DDF4",label="Directory of input gVCFs"];\n    "refM" [fillcolor="#94DDF4",label="Mitochondrial reference genome"];\n    "seqidM" [fillcolor="#94DDF4",label="Mitochondrial naming scheme for gVCF"];\n  }\n  subgraph cluster_outputs {\n    rank = "same";\n    style = "dashed";\n    labelloc = "b";\n    label = "Workflow Outputs";\n    "out1" [fillcolor="#94DDF4",label="Directories of FastJs"];\n    "out2" [fillcolor="#94DDF4",label="Indexed and zipped gVCFs"];\n  }\n  "step1" [label="Create list of gVCF directories to process"];\n  "step2" [label="step2"];\n  "refdirectory" -> "step1" [label="refdirectory"];\n  "step1" -> "step2" [label="gffPrefix"];\n  "afn" -> "step2" [label="afn"];\n  "reffa" -> "step2" [label="reffa"];\n  "tagdir" -> "step2" [label="tagdir"];\n  "step1" -> "step2" [label="gffDir"];\n  "ref" -> "step2" [label="ref"];\n  "afnM" -> "step2" [label="afnM"];\n  "refM" -> "step2" [label="refM"];\n  "reffaM" -> "step2" [label="reffaM"];\n  "seqidM" -> "step2" [label="seqidM"];\n  "aidx" -> "step2" [label="aidx"];\n  "aidxM" -> "step2" [label="aidxM"];\n  "step2" -> "out1";\n  "step2" -> "out2";\n}\n","licenseLink":"https://raw.githubusercontent.com/curoverse/l7g/master/LICENSE","visualisationXdot":"/graph/xdot/github.com/curoverse/l7g/blob/master/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl","visualisationPng":"/graph/png/github.com/curoverse/l7g/blob/master/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl","visualisationSvg":"/graph/svg/github.com/curoverse/l7g/blob/master/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl","roBundle":"/robundle/github.com/curoverse/l7g/blob/master/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl","packed":false,"permalink":"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl","@id":"https://w3id.org/cwl/view/git/c1ccad9f81cd3fc44a9f6e8d3b27acb154ba03cc/cwl-version/convert2fastj/gvcf_version/cwl/tiling_convert2fastj_gvcf.cwl"}


Adding new tests

These tests are simplified versions of some tests from Test Framework

Coverage increased (+0.9%) to 48.249% when pulling edcbf71971f61a7a0b374f2337a99131832ff69e on yitam:addNewTests into 12e6535a7f5fbc7c94a348ccad063ee7ede207d2 on Microsoft:dev.

Coverage increased (+0.9%) to 48.249% when pulling a336aaff54d8f97f336b54746d9baaa463620316 on yitam:addNewTests into 12e6535a7f5fbc7c94a348ccad063ee7ede207d2 on Microsoft:dev.

Coverage increased (+4.6%) to 51.943% when pulling 6343f9b0016423392e43eebb661648c2b7582072 on yitam:addNewTests into 12e6535a7f5fbc7c94a348ccad063ee7ede207d2 on Microsoft:dev.

Coverage increased (+2.0%) to 51.943% when pulling 805cd3393c2951388f3da77c3e884719f9faa13d on yitam:addNewTests into e854a9ca53de896eff51db58fcf40b114e67ae54 on Microsoft:dev.

Coverage increased (+2.0%) to 51.943% when pulling 805cd3393c2951388f3da77c3e884719f9faa13d on yitam:addNewTests into e854a9ca53de896eff51db58fcf40b114e67ae54 on Microsoft:dev.

Coverage increased (+2.4%) to 52.417% when pulling 9b22f0c3fcf050acc76ed2a8bd010adf595d6f02 on yitam:addNewTests into e854a9ca53de896eff51db58fcf40b114e67ae54 on Microsoft:dev.

Coverage increased (+2.4%) to 52.425% when pulling ee1be419beab8fb68367c0f01b8e5d9b3af4eef4 on yitam:addNewTests into e854a9ca53de896eff51db58fcf40b114e67ae54 on Microsoft:dev.

Coverage increased (+2.4%) to 52.425% when pulling efc52c6be65879ac80fb789a8de3b155c1031e93 on yitam:addNewTests into e854a9ca53de896eff51db58fcf40b114e67ae54 on Microsoft:dev.


How to get user information after verifying token

How can I add user information to verifying view if token is valid?
I created custom serializer as subclass of TokenVerifySerializer and custom view as subclass of TokenVerifyView. How to add user information in response returned from this view?
I want to do something like this
# In serializers.py

class CustomTokenVerifySerializer(TokenVerifySerializer):
    token = serializers.CharField()

    def get_user_from_token(token):
        # somehow get user information here

    def validate(self, attrs):
        UntypedToken(attrs['token'])

        return self.get_user_from_token(attrs['token'])

# In views.py

class CustomTokenVerifyView(TokenVerifyView):
    serializer_class = CustomTokenVerifySerializer

Ok, I found the solution. I'm new at using JWT Authentication.
I figured it out with using decode method from jwt package

@arif, how did you do it? If you please explain it a bit.

@SamsadSajid
I installed PyJWT package from here
Then I created CustomTokenVerifySerializer class in serializers.py and override the validate method like this
from jwt import decode as jwt_decode

class PandaTokenVerifySerializer(TokenVerifySerializer):
    token = serializers.CharField()

    def validate(self, attrs):
        UntypedToken(attrs['token'])
        data = jwt_decode(attrs['token'], settings.SECRET_KEY, algorithms=['HS256'])

        data = {
            'id': data['user_id'],
            'username': data['username'],
            'first_name': data['first_name'],
            'last_name': data['last_name'],
            'email': data['email'],
            'email_confirmed': data['email_confirmed'],
            'email_confirmation_date': data['email_confirmation_date'],
            'phone': data['phone'],
            'phone_confirmed': data['phone_number_confirmed'],
            'phone_confirmation_date': data['phone_number_confirmation_date'],
            'picture': data['picture'],
        }

        return data

@SamsadSajid
I installed PyJWT package from here
Then I created CustomTokenVerifySerializer class in serializers.py and override the validate method like this
from jwt import decode as jwt_decode

class CustomTokenVerifySerializer(TokenVerifySerializer):
    token = serializers.CharField()

    def validate(self, attrs):
        UntypedToken(attrs['token'])
        data = jwt_decode(attrs['token'], settings.SECRET_KEY, algorithms=['HS256'])

        data = {
            'id': data['user_id'],
            'username': data['username'],
            'first_name': data['first_name'],
            'last_name': data['last_name'],
            'email': data['email'],
            'email_confirmed': data['email_confirmed'],
            'email_confirmation_date': data['email_confirmation_date'],
            'phone': data['phone'],
            'phone_confirmed': data['phone_number_confirmed'],
            'phone_confirmation_date': data['phone_number_confirmation_date'],
            'picture': data['picture'],
        }

        return data

@arif Where do you put the CustomTokenVerifySerializer class? The serializers.py in rest_framework_simplejwt folder?

@kumjShid4
No, it's not a good idea editing module files. I created serializers.py in my own app. So create file named serializers.py in your own app and import like this
from your_app.serializers import CustomTokenVerifySerializer


SMC 6.2.3 (vpn_mapping)

smc.api.exceptions.UnsupportedEngineFeature: Resource requested: 'vpn_mapping' is not available on this element.
fatal: [localhost]: FAILED! => {
"changed": false,
"module_stderr": "Traceback (most recent call last):\n  File "/tmp/ansible_Kj4hwX/ansible_module_engine_facts.py", line 577, in \n    main()\n  File "/tmp/ansible_Kj4hwX/ansible_module_engine_facts.py", line 574, in main\n    EngineFacts()\n  File "/tmp/ansible_Kj4hwX/ansible_module_engine_facts.py", line 553, in init\n    is_fact=True)\n  File "/tmp/ansible_Kj4hwX/ansible_modlib.zip/ansible/module_utils/stonesoft_util.py", line 429, in init\n  File "/tmp/ansible_Kj4hwX/ansible_module_engine_facts.py", line 564, in exec_module\n    if engine.name == self.filter]\n  File "/tmp/ansible_Kj4hwX/ansible_module_engine_facts.py", line 524, in to_yaml\n    return yaml_cluster(engine)\n  File "/tmp/ansible_Kj4hwX/ansible_module_engine_facts.py", line 480, in yaml_cluster\n    policy_vpn = get_policy_vpn(engine)\n  File "/tmp/ansible_Kj4hwX/ansible_module_engine_facts.py", line 492, in get_policy_vpn\n    vpn_mappings = engine.vpn_mappings\n  File "/usr/lib/python2.7/site-packages/smc_python-0.6.1_dev.201802262117-py2.7.egg/smc/core/engine.py", line 785, in vpn_mappings\n    resource='vpn_mapping'))\n  File "/usr/lib/python2.7/site-packages/smc_python-0.6.1_dev.201802262117-py2.7.egg/smc/base/mixins.py", line 28, in make_request\n    raise ex(e)\nsmc.api.exceptions.UnsupportedEngineFeature: Resource requested: 'vpn_mapping' is not available on this element.\n",
"module_stdout": "",
"msg": "MODULE FAILURE",
"rc": 0

Hi Remy,
Thank you for posting this - I will address backwards compatibility for policy VPN on the engine in next commit (this evening).

Hi David,
When do you plan to commit ?
Rémy.

Just finishing doc and examples, will be today, hopefully in next 2 hours. For OSPF, i've added a module to add/modify/remove OSPF areas and OSPF profiles. I can add the OSPF domain settings and OSPF interface settings (not related to adding OSPF to an engine) after but at least this would enable you use the normal engine JSON and customize an area and profile and add/remove from engine interfaces. Does that work?

This has been posted. You will want to grab from the develop branch and update to latest smc-python also.

Resolved in develop branch commit 3955f45


Slotted style for li elements being ignored

When styling slotted li elements using the ::slotted(li) selector, slotted styles appear to be ignored.
Live Demo
https://stackblitz.com/edit/lit-element-example-vlkjbm
Steps to Reproduce

Create my-element with slotted content and slotted styles that apply separately to li and ul elements.
Append my-element to document.body
Create ul with li elements.
Append ul to my-element

Expected Results
Slotted li style should apply.
Actual Results
Slotted li style is ignored.
Browsers Affected

[x] Chrome
[x] Firefox
[ ] Edge
[x] Safari 11
[ ] Safari 10
[ ] IE 11

Versions
$ npm ls
<EMAIL_ADDRESS>/Users/ricardo/Projects/test-nested-slotted-lists-lit
├──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
$ npm --version
6.10.1

According to the spec, ::slotted only works for direct child nodes assigned to the slot.
So it works like that by design.

Ah is there no way to style indirect child nodes?

Ah is there no way to style indirect child nodes?

Only via global styles.


Weird blurring

Hey, @PuruVJ I know this isn't meant to be a serious thing. But I found that the screen randomly blurs

Kinda like that

Can you tell the browser and its version please?

Firefox Developer Edition 92.0b5 (64-bit)

backdrop-filter is still experimental in Firefox, so these rendering bugs are inevitable. Only to fix these would be to wait for backdrop-filter to become stable.

Ok, Thanks


Update scala 2.13.0-RC2 to 2.13.0-RC3

Can you go straight to 2.13.0 that got released today ? https://github.com/scala/scala/releases

Closed in favour of #202


Fix swipeout width on device rotation

Detect device rotation and render with newly-established device width. I'll need to look into how to detect rotation gesture from within react-native.

Maybe passing the screen orientation as a property would be enough for start.

https://facebook.github.io/react-native/docs/dimensions.html might do the trick as well..


refactor(transformer/class-properties): reduce visibility of method

Follow-on after #7831. transform_super_call_expression_arguments is only used within this file, so does not need to be pub(super).

#7859
#7858  👈 (View in Graphite)
main

How to use the Graphite Merge Queue
Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 13, 10:18 PM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


feat: Add headless option to nuxtIcon config

Adds a headless option to remove .icon class from the icon component.
Ref - https://github.com/nuxt-modules/icon/issues/24

Looks good thank!
Can you update the readme as well to include this option?

Since #38 is merged, you can set nuxtIcon.class: '' to have the headless option. Updated your PR and merging so you count as contributors ❤


Parsley breaks (and submits an invalid form) when an errormessage is missing

When looking up a nested errormessage, parsley breaks when trying to look that up in an undefined object.
ParsleyValidator.addCatalog('fr', {}, true);

No errormessages are set (obviously not what you want, but you might have overlooked or mistyped one), this doenst break when looking up 'required' for example (error message is just empty), but it does when looking up 'type.email', since type is undefined.

getErrorMessage: function (constraint) {
      var message = '';
      var defaultMessage = this.catalog[this.locale].defaultMessage || '__MISSING_DEFAULT_MESSAGE__';
      // Type constraints are a bit different, we have to match their requirements too to find right error message
      if ('type' === constraint.name)
        if(this.catalog[this.locale][constraint.name])
          message = this.catalog[this.locale][constraint.name][constraint.requirements];
      else
        message = this.formatMessage(this.catalog[this.locale][constraint.name], constraint.requirements);
      return '' !== message ? message : defaultMessage;
    }

Fixed, thanks


[EfficientNet] Understanding architectural implementation decision for the Squeeze and Excitation phase

The following question refers to "Squeeze and Excitation" block from the EfficientNet implementation (https://github.com/tensorflow/tensorflow/blob/v2.3.0/tensorflow/python/keras/applications/efficientnet.py#L517-L540), lines 478 - 498.
Concretely, "Squeeze and Excite" current implementation makes use of two Conv2D blocks: one for filters reduction at line 483 and the other one for filters expanding at line 491.
Giving the fact that the input in this two Conv2D layers is just an array of features reshaped into a 3D tensor (of shape (1, 1, nr_features) according to line 482), the operations that this layers are doing is just a standard one-shot matrix multiplication followed by and activation (as in exactly what a Dense layer does by default).
Therefore, my question is why the implementation uses a Conv2D instead of a regular Dense layer? Is it because of easier code understanding/maintenance? I guess the reason could not be speed execution since both Dense and Conv2D make use of optimized matrix multiplication.
Thanks, would really appreciate if this question can be answered as I can't find any docs that address this question.

@AlexandruPurdila Squeeze and Excitation phase is a method to give weightage to each channel instead of treating them all equally. You can get indepth understanding here.


✨ Adding a dependency injection module

💉 DI Module
This PR adds a dependency injection module to the colibri sdk.
Module features:

Automatic dependency injection
Automatic identification of interface implementation
Disambiguation through metadata

For more information, see the README file in the module directory.

LGTM
Resolve conflicts before merge.


manifest-tool: fix static build and update 2.0.6 -> 2.1.5

Description of changes

Updates manifest-tool from 2.0.6 -> 2.1.5
Fixes static build

Supersedes https://github.com/NixOS/nixpkgs/pull/225468.

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build pkgsStatic.manifest-tool


Feat: Multimoose

Currently MOOSE is running on server configuration. So there is a good chance that the user is using a DGX or so. In that case, it would make sense to fully utilise the capabilities of the hardware. Similar to falcon, moose should run in parallel based on the hardware capabilities.

Nope, this is only convenience, I think. MOOSE is, by default, capable of running in parallel when started in multiple terminals on multiple datasets, right? So multimoose is, in theory, a "wrapper" for that.
Would you like to have it? I would say that's a very low priority still.

Nope, not really. Let’s close this. We will not work on this :)

On 28.06.2022, at 10:21, Sebastian Gutschmayer @.***> wrote:

Nope, this is only convenience, I think. MOOSE is, by default, capable of running in parallel when started in multiple terminals on multiple datasets, right? So multimoose is, in theory, a "wrapper" for that.
Would you like to have it? I would say that's a very low priority still.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were assigned.


Licence question

So the official rules states the following to get started:

this already has a licence file. If I am to build on top of some fundamental concepts in this example app Do i NEED to leave this licence file?
later in the instructions it states that I must add a licence that meets the following:

This will create a copyright under my name. what licence is appropriate to use?

Yes copyright all files you change in your name...

thnaks!!!


Usage with FontAwesome 5 Pro

How do I use FontAwesome 5 Pro into my project? It seems like I can't use the regular or light icons on the header.

        Configure

            Determine the columns and rows for the seat layout. Then, label the seat accordingly.

The problem with the code above is that it doesn't display the FA5's regular icon, it displays FA5's solid icon instead.
Example output:

Help, thanks!

In my new PR I have done a report of FA5 using some new scripts I write. I have the ability to ship the scripts and allow users to port FA into semantic themselves. This will allow users to add FA5 Pro to semantic without leaking the icons.
Currently I need to proof check the scripts and rule out any bugs with them before I make them public.

Hey @hammy2899! Just checking in to see if the scripts you had written to help users port FA5 Pro into Semantic are ready for primetime. Was about to dig into doing this manually but figured I'd check here first ;) Thanks!

@sinalet They are "ready" but I am not happy with them, I was just talking about them the other day with someone and was talking about rewriting them, I will give you a ping when I make them public 😃


RFC igluctl: Should igluctl contain functionality to run SQL/DDL statements?

Should igluctl contain some functionality (igluctl static create) to run generated SQL statements?
My main motivation is that this reduces the UX friction in using igluctl to deploy the schemas/JSON paths to Iglu but then having to switch tooling to run a SQL statement.
Initially this makes sense for Redshift but could also be extended to other data sources (Postgres, SQL Data Warehouse etc). This may(?) mean that igluctl can reconsume the configuration file for emr-etlr-runner.

Hey @miike - it's a really interesting idea. Our plan was to "jump" to the DDL being auto-applied inside the Snowplow storage sinks, but thinking about it, that code has to live somewhere and arguably it c/should live in an Iglu component. One thing to be careful of here is that suddenly we are bundling database drivers into Iglu...
Needs some further rumination. Assigning to @chuwy for his thoughts.

Agreed - there's definitely the potential of introducing a bit of bloat into an otherwise lightweight tool by introducing drivers.

One to ponder! In general though, yes, we are committed to making things which sit somewhere on the Snowplow-Iglu border as generalized as possible, and thus landing them in Iglu territory...

Migrated to https://github.com/snowplow-incubator/igluctl/issues/36


Can't we directly encode pcm？

Must I enter wav for encoding？

Does your tittle "Can't we directly encode pcm?" mean you want process audiostream's input samples in realtime?

Yes, there are such real-time scenarios

Ok, I'm interested in that too. I was inspired by Lyra voice quality vs bitrate, and would like to make transceiver prototype in Gnuradio, to transmit and receive Lyra encoded digital voice over air.


⚠ Bofrost.de has degraded performance

In c870b25, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1366 ms

Resolved: Bofrost.de performance has improved in e314251 after 6 minutes.


AP: Automatically allow instance actors

I discovered just now that we're rejecting activities from instance actors because they (of course) haven't enabled the bridge. 😕 Example: https://mastodon.social/actor:
{
  "id": "https://mastodon.social/actor",
  "type": "Application",
  "inbox": "https://mastodon.social/actor/inbox",
  "outbox": "https://mastodon.social/actor/outbox",
  "preferredUsername": "mastodon.social",
  "url": "https://mastodon.social/about/more?instance_actor=true",
  "manuallyApprovesFollowers": true,
  "publicKey": {
    "id": "https://mastodon.social/actor#main-key",
    "owner": "https://mastodon.social/actor",
    "publicKeyPem": "..."
  },
  "endpoints": {
    "sharedInbox": "https://mastodon.social/inbox"
  }
}

Activity:
{
  "id": "https://mastodon.social/52e59802-c8c0-4f64-a6c4-b45da5c8c9fe",
  "type": "Flag",
  "actor": "https://mastodon.social/actor",
  "content": "",
  "object": [
    "did:plc:mgx2xbqznfs4sphhczkty2tm",
    "at://did:plc:mgx2xbqznfs4sphhczkty2tm/app.bsky.feed.post/3kwettolck225"
  ]
}

Of course, the question is how to identify instance actors? Currently undefined, at least in any standard. FEP-d556 (discussion) is the closest we have. I'll see if that applies to Mastodon instance actors, and if not, I can special case them.

FEP-d556 is great, and more or less works with Mastodon, but the difficulty is that it describes how to go from server to instance actor id, not from actor id to whether it's the instance actor. Not a big deal though, I can always just look up a server's instance actor and see if it matches the actor id.

Webfinger:
{
  "subject"<EMAIL_ADDRESS>  "aliases": ["https://mastodon.social/actor"],
  "links": [
    {
      "rel": "http://webfinger.net/rel/profile-page",
      "type": "text/html",
      "href": "https://mastodon.social/about/more?instance_actor=true"
    },
    {
      "rel": "self",
      "type": "application/activity+json",
      "href": "https://mastodon.social/actor"
    },
    {
      "rel": "http://ostatus.org/schema/1.0/subscribe",
      "template": "https://mastodon.social/authorize_interaction?uri={uri}"
    }
  ]
}

Done!


Unexpected error response for duplicate users

A call to HTTP API POST /_api/user with a duplicate user now returns:
{"error":true,"code":400,"errorNum":1702,"errorMessage":"duplicate user"}

I did expect a code of 409 to be in line with creating a database with a duplicate name.

A call to HTTP API POST /_api/user with a duplicate user now returns:
I am slightly confused by the now. Does that mean the behavior has changed when comparing it to previous versions of ArangoDB, or does that mean that the behavior is simply inconsistent (returning HTTP 400 instead of HTTP 409 as in other parts).

Changed the return code from HTTP 400 to HTTP 409 in this case.

ArangoDB 3.1.13 is available for download.


Investigate the feasibility of a spark streaming example

Implemented with:
https://github.com/snowch/biginsight-examples/commit/78183a076cde38277ffc58f0975a20d1e880e274
https://github.com/snowch/biginsight-examples/commit/8fbf022d26b74c864d33886510bbb72a62c99100


Hook to allow modify faceted seach query

Questions
Answers

Description?
I've seen that currently is not possible modify the query from faceted search module. This will allow external modules to modify the query to achieve its functionality.

Type?
improvement / new feature

BC breaks?
no

Deprecations?
no

Fixed ticket?
https://github.com/PrestaShop/PrestaShop/issues/32951

How to test?
Execute this hook and modify the array you need.

This change is

Hi @Hlavtox
Maybe it's not the best hook name, but by modifying these tables, we can allow to modify the conditions without a major update....
By modifying the array "joinCondition" we can add an additional where for example.
It is possible to modify the getQuery function to use the DbQuery class (for example), but that is a major update....
Currently i can't see other way to modify this query...
Please check it.
Thanks!

For me the MySQL adapter is not the right place to add filtering / changes to the queries. It's designed to be as generic as possible and provide basic functionalities to allow other adapter to be created, like for example a ElasticSearch adapter.
If you want to add new filtering, you should do this at a higher level.

Since we have had no news from you for more than 20 days, I'm closing this PR. Feel free to re-open it if you find the motivation to finish it. Don't hesitate to ping me if you have any questions or doubts.


Adiciona informações faculdade sirius

@Karinaesparza96 Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Code completion plugin setup

Project Setup for Code Completion Plugin.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @pabloem

Addressed the  comments above, @pabloem

Renamed the plugin and updated Readme, @pabloem

LGTM thanks @sultanalieva-s


Select in Modal does not show dropdown list correctly

Current Behavior
dropdown list is hidden behind the backdrop of the modal

Expected Behavior
it should show up over the modal

Your Environment

Tech
Version

Base UI
v6.6.0

React
v16.5.0

browser
chrome

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

I found that if I change zIndex: 100000 for Body comp in react portal for Dropdown menu, it shows up.

I found the duplicates


feat: add  --xml-whitespace-sensitivity ignore to xml hooks

https://github.com/prettier/plugin-xml#whitespace

/tmp/autoware_launch main
❯ pre-commit try-repo https://github.com/tier4/pre-commit-hooks-ros prettier-launch-xml -a --ref add-xml-whitespace-sensitivity
===============================================================================
Using config:
===============================================================================
repos:
-   repo: https://github.com/tier4/pre-commit-hooks-ros
    rev: add-xml-whitespace-sensitivity
    hooks:
    -   id: prettier-launch-xml
===============================================================================
[WARNING] The 'rev' field of repo 'https://github.com/tier4/pre-commit-hooks-ros' appears to be a mutable reference (moving tag / branch).  Mutable references are never updated after first install and are not supported.  See https://pre-commit.com/#using-the-latest-version-for-a-repository for more details.  Hint: `pre-commit autoupdate` often fixes this.
[INFO] Initializing environment for https://github.com/tier4/pre-commit-hooks-ros.
[INFO] Initializing environment for https://github.com/tier4/pre-commit-hooks-ros:@prettier/plugin-xml<EMAIL_ADDRESS>[INFO] Installing environment for https://github.com/tier4/pre-commit-hooks-ros.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
prettier launch.xml......................................................Failed
- hook id: prettier-launch-xml
- files were modified by this hook

autoware_launch/launch/autoware.launch.xml
autoware_launch/launch/logging_simulator.launch.xml

/tmp/autoware_launch main* 6s
❯ git diff
diff --git a/autoware_launch/launch/autoware.launch.xml b/autoware_launch/launch/autoware.launch.xml
index b09380c..9b6271d 100644
--- a/autoware_launch/launch/autoware.launch.xml
+++ b/autoware_launch/launch/autoware.launch.xml
@@ -107,5 +107,4 @@
     <!-- Web Controller -->
     <include file="$(find-pkg-share web_controller)/launch/web_controller.launch.xml"/>
   </group>
-
 </launch>
diff --git a/autoware_launch/launch/logging_simulator.launch.xml b/autoware_launch/launch/logging_simulator.launch.xml
index e775156..c632d0a 100644
--- a/autoware_launch/launch/logging_simulator.launch.xml
+++ b/autoware_launch/launch/logging_simulator.launch.xml
@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <launch>
-
   <!-- Essential parameters -->
   <arg name="map_path" description="point cloud and lanelet2 map directory path"/>
   <arg name="vehicle_model" description="vehicle model name"/>


Update supercollider to 3.8.0

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

I’ve merged it, but when the change is just the checkpoint, please don’t submit it. There’s no point.

@vitorgalvao sure


[prim_ram] Add reset signal to prim_ram_1p/prim_ram_2p?

prim_ram_1p/prim_ram_2p currently don't have a reset signal input, as they don't need one. (Note that the _adv wrappers have a rst signal; this discussion is about the underlying primitives.)
For writing assertions, a reset signal, however, is helpful. Other use cases might come to mind.
I don't have a strong feeling either way, WDYT @tjaychen @msfschaffner @sjgitty @sriyerg?

Another option might be to stick the assertions in a "protocol checker" module which you instantiate whenever you actually use the primitive (maybe with a macro or wrapper module; maybe not). That might make it a bit more obvious which bits are part of the actual design and which bits are just for sanity checks.

i'm not against adding a reset for assertion purposes.
If the reset really ends up being unloaded, in the final round of synthesis
it will be pruned anyways.
is this related to Michael's PR where he just uses a hardcoded 0 for one of
the memory assertions?
On Mon, Jun 15, 2020 at 2:55 AM Rupert Swarbrick<EMAIL_ADDRESS>wrote:

Another option might be to stick the assertions in a "protocol checker"
module which you instantiate whenever you actually use the primitive (maybe
with a macro or wrapper module; maybe not). That might make it a bit more
obvious which bits are part of the actual design and which bits are just
for sanity checks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/2494#issuecomment-644028930,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSQR3D47CUEGDFPIIX3RWXVYZANCNFSM4N57ZWWQ
.

is this related to Michael's PR where he just uses a hardcoded 0 for one of
the memory assertions?

Yes, I filed this issue to discuss if we want resets, or not, and to unblock #2487 in the meantime.

I am also ok with adding a reset signal to these wrappers for assertion purposes, since as Tim mentioned, it will just be pruned away if unused. The protocol checker would also be an option, but for this one assertion I think we need not necessarily introduce another module...

This is now related to #12602
In addition to bringing in a reset, in general it would be nice to have an init function so that software would not need to do it manually in the future.

Related issue #2263


common/gpu/intel/opencl: add opencl

Description of changes
Updates intel libraries as per https://github.com/NixOS/nixpkgs/pull/205648
Adds module for intel opencl currently only supporting GPU
It is possible to add CPU OpenCL support, but only using an older version of the driver: https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=intel-opencl-runtime
Things done

[ ] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

The intel-ocl package is unfree and not required for newer processors, should it be gated behind an unfree/legacy option?

The intel-ocl package is unfree and not required for newer processors, should it be gated behind an unfree/legacy option?

How old? hardware.enableRedistributableFirmware could be used potentially used if the license allows for redistribution.

Potentially makes sense to have the opencl option in nixos itself: https://github.com/NixOS/nixpkgs/pull/219657

The intel-ocl package is unfree and not required for newer processors, should it be gated behind an unfree/legacy option?

How old? hardware.enableRedistributableFirmware could be used potentially used if the license allows for redistribution.

Both the intel-media-driver and intel-compute-runtime support starts with Broadwell, aka 5th gen intel and newer. The licence does not include any provisions for redistribution.
In addition I realized that intel-ocl is a very old version of the CPU OpenCL driver. It also includes a GPU driver but that isn't actually packaged correctly.


[Issue]: Butcher Station don't count baby critters

Description of the bug/crash
It's not a bug/crash, is more like a improvement.
Inside a ranch room (max of 96 blocks) the maximum number of critters to br comfortable/happy and well reproduce are 8.
I put the Butcher Station inside the ranch with Auto-Wrangle Surplus set to 8.
Butcher Station do not consider baby critters when counting (baby critters are not in the list). So the room limit is be over the limit impacting the reproduction speed.
To fix the issue, the Butcher Station needs to count both adult and the baby critters (in my case I had both of the same type).
Thanks.
Steps To Reproduce
Create a ranch room, put 6 adult critters and 4 baby critters inside. Set Auto-Wrangle Surplus to 8. Nothing will happen.
If you put a Critter sensor with both adult and baby critter selected, the account will be 10.

Player.log
Player.log

it works as intended

Update: 23 Feb  8:21am
Compatibility with the new U51 beta version of the game.
Now only the Creatures selected in the filter are counted, Similar to the new Critter Drop-Off.
Minor corrections to the codex and the build menu.

to return to the old behavior, turn off this checkbox

It worked!
I would suggest add a small note in steam workshop to help noob players like myself. :)
Thanks!!!


fix: Potential file inclusion via variable

Signed-off-by: slayer321<EMAIL_ADDRESS>

This PR

Potential file inclusion via variable G304
Related Issues

No issue . related to bugbash
lift bugbash
Notes

Follow-up Tasks

How to test

Hey @bacherfl  I merge the latest state of your master branch into my master branch after that what needs to be done?

Hey @bacherfl I merge the latest state of your master branch into my master branch after that what needs to be done?

Hey, thanks for the update - Sorry, I was not clear enough - I meant you should merge our master branch into your feature branch (slayer321:fix-G304-errors). Then your branch should have the required changes to be able to execute the unit test :)


partio as a Conan-only package

Convert partio as a Conan-only package.
Appears to be used by OSL.

Addressed in 2024.0 releases


Next release due date?

Hi Cristian, great work as always on this powerful tool.
Curious, the upgrade steps - does this require you to have pushed the latest release to PyPi? I see there have been some commits but pulling from PyPi with an install doesn't give me the latest changes.
Thanks

Hi @lakhvirss, thank you!
Only the git-bash utils component had recent more changes. And that is already published to pypi https://pypi.org/project/alpha-build-git-bash-utils.
There was another commit for a simplified version of alpha-build/core, which offloads most formatters and static analysis tools to pre-commit


Issue [Feature] Add Book Loader #1282  fixed

Fixes Issue🛠

Closes #1282
Description👨💻

I have included the Book Loader under loaders
Type of Change📄

[x] New feature (non-breaking change which adds functionality)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@Rakesh9100 Please check and merge the PR


responsiveness fixed

Thanks for your valuable contribution, please raise the PR with correct format, we will accept it.
Reference - https://github.com/SwanandD121/FeatherPerfect_fe/pull/156

Hi, bro I am new to open source contribution.
So, I have to explain what changes I did?
Or I have to send the pull request again?
I'll appreciate your help.
Thank you
On Tue, 31 Dec 2024, 9:29 pm Vedansh Jain, @.***> wrote:

Thanks for your valuable contribution, please raise the PR with correct
format, we will accept it.
Reference - SwanandD121/FeatherPerfect_fe#156
https://github.com/SwanandD121/FeatherPerfect_fe/pull/156
—
Reply to this email directly, view it on GitHub
https://github.com/vedanshjainvj/Job-Hive/pull/17#issuecomment-2566556187,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZJRGNY2YWNNZUFX2JK67AD2IK5M7AVCNFSM6AAAAABUNTAWVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRWGU2TMMJYG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fix repository name to fix broken issues link.

The link to "GitHub" issues at the top of the spec is broken; I believe this will fix it.

Oops, thanks!


Question: Does AzCopy v10 support resuming of failed uploads

Which version of the AzCopy was used?
10.3.2
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
azcopy copy
azcopy jobs resume
What problem was encountered?
When for some reason the upload is interrupted, and resumed using azcopy jobs resume, it seems like it's restarting from the beginning. Is this the expected behavior?
How can we reproduce the problem in the simplest way?

Copy a large file from local HDD to an azure storage blob.
Cancel upload in between.
run azcopy jobs resume .

Have you found a mitigation/solution?
No. Maybe use AzCopy 8.x instead?

Hi, Neno!
Currently, this is intended behavior. We are discussing how we would go about resuming a transfer, at present.

@adreed-msft OK, that's good to know. Maybe we can make that more clear in the docs?
"azcopy jobs resume" sounds quite like "continue where you left off" and less about: let's restart and override the file. :)
So I guess the workaround is to go back to AzCopy V8 then?

Jobs resume does continue where you left off on a per-file basis, rather than on a per-block basis. Perhaps the wording should be more clear, yes.
Regarding the workaround, yes, if you need resuming at the granularity you're requesting, V8 does support it, I think. I'm not personally familiar with V8's features since I came along well after V10 got started.

Thanks, @adreed-msft - Indeed, I'm uploading a single, large .VHD file and was surprised by the behavior. :)
For now, AzCopy V8 should work for me. Thanks again for making the docs more clear!

I hit the same issue on v10.3.2

AzCopy resume job will continue the last job from wherever it left and retry the files which failed during transfer last time.

AzCopy resume job will continue the last job from wherever it left and retry the files which failed during transfer last time.

@vibhansa-msft But will it resume the entire file, or can it also resume, let's say at 80% of that file and only do the rest (20%)? That's what AzCopy V8 supports.


Fixing fatal error in celeryev for delayed tasks

If the host is None on a task, celerev will crash.

P.S. The delayed task has an eta of 60s.


Not working in Samsung note 10

I got an exception here. Activity Not found exception
 try {                 startIntent(context, PACKAGE_SAMSUNG_MAIN, PACKAGE_SAMSUNG_COMPONENT)             } catch (e: Exception) {                 e.printStackTrace()                 return false             }

Hey @Luisstevie thanks for the notice, this library depends on crowd sourcing very heavily since one person doesn't have all the phones that can be tried and tested to figure out which package works, it would be great to have a pull request for the version of samsung phone that you are currently testing it on.

Hey @judemanutd !
See my comment to #33 it seems the activity name is wrong or was changed.

Hey @judemanutd !
See my comment to #33 it seems the activity name is wrong or was changed.

This should be resolved by https://github.com/judemanutd/AutoStarter/pull/69 but feel free to open this issue back up in case it does not


Blazor Components: Remove ParameterAttribute and allow fields.

Hi,
Started to port some React components to play with Blazor.
First thing is stumbled upon is Parameter attribute.
Here is a current snippet from my code:
@functions {
[Parameter]
double? left { get; set; }
[Parameter]
double? top { get; set; }
[Parameter]
double? right { get; set; }
[Parameter]
double? bottom { get; set; }
[Parameter]
double? width { get; set; }
[Parameter]
double? height { get; set; }
[Parameter]
double? star { get; set; }
[Parameter]
double? basis { get; set; }
[Parameter]
bool auto { get; set; }
[Parameter]
bool column { get; set; }
[Parameter]
bool absolute { get; set; }
[Parameter]
bool absoluteLeft { get; set; }
[Parameter]
bool absoluteRight { get; set; }
[Parameter]
bool relative { get; set; }
[Parameter]
bool @static { get; set; }
[Parameter]
bool @fixed { get; set; }
[Parameter]
bool fixedLeft { get; set; }
[Parameter]
bool fixedRight { get; set; }
[Parameter]
bool hidden { get; set; }
[Parameter]
bool wrap { get; set; }
[Parameter]
bool autoScroll { get; set; }
[Parameter]
bool crop { get; set; }
[Parameter]
RenderFragment ChildContent { get; set; }
}
I imagine that ParameterAttribute indicates properties (and only properties) which could be specified from outside. Exactly for the same reason C# has public property and field modifiers.
So why couldn't we for the sake of simplicity indicate parameters with public properties and fields? Each public property and field should be a parameter by default.
So the code above could look much simpler and take less time to type:
@functions {
public double? left, top, right, bottom, width, height, star, basis;
public bool auto, column, absolute, absoluteLeft, absoluteRight, relative, @static, @fixed, fixedLeft, fixedRight, hidden, wrap, autoScroll, crop;
public RenderFragment ChildContent;
}
If we want internal fields for state, just use private or protected members like we normally do.

I'm a fan for doing such thing in RazorPages, to make public properties bound without BindAtribute

Mate,
public double? left defines a C# variable of public scope and nuallable double. It is not a definition of a property. And it has nothing to do with Blazor. This is the way to go in .NET from the very beginning. The Parameter attribute is necessary in order to distinguish between ordinary property and otherwise. However, I believe you can define a class, annotated with the Parameter attribute, and containing your properties...

Mate,
public double? left defines a C# variable of public scope and nuallable double. It is not a definition of a property. And it has nothing to do with Blazor. This is the way to go in .NET from the very beginning. The Parameter attribute is necessary in order to distinguish between ordinary property and otherwise. However, I believe you can define a class, annotated with the Parameter attribute, and containing your properties...

In this case I will make separate class with all this properties and declare it as parameter.
It will give us more clean code.
[Parameter]
MyClass myclass  { get; set; }
MyClass will have all necessary properties inside.
I think parameter and public property are different and we do not need to mix.
Parameter we use for markup (razor to recognize with properties can be set in markup) and public property we can use from code behind anytime.

Perhaps the case is different little bit from RazorPages, because in RazorPages public properties are intended to be used from within the view, so that's why I see all the samples and demo decorate the public properties as BindAttribute

Component parameters have different semantics than ordinary .NET properties, hence the additional ceremony. Parameters are the principle way that data should flow into a component that affects rendering. This allows the Blazor runtime to reason about when the component should be rendered. I.e. if the parameters change then the component should be rendered, otherwise it doesn't need to be. So we want the definition of parameters to be an explicit gesture. That's why the [Parameter] attribute is required.
Parameters also should not be mutated outside of the normal component life cycle. Parameters should be set through the IComponent.SetParameters method. That's why we recommend that parameters not be public. This guidance is implemented as an analyzer, but you can disable it if you want.
We prefer properties to using fields because it gives component authors the opportunity to insert some code when the parameter is set. Ex you want to do some conversion logic.
To make it easier to create parameters we provide a convenient snippet. Just type "para" and then hit TAB twice. We also considered adding a Razor directive to make defining parameters easier, but then you have the problem of having to replicate normal C# features (like attributes) for the new syntax.
I hope this helps explain why we landed where we did.


Broken download links for images

Hi,
Was using this script on windows - both with native Python installation and with the Cygwin package. In both cases images did not download properly - no error messages, but the files where all empty and 1kb in size.
Found the reason in def link_replace(self, link): ... link = urljoin(self.base_url, link). Here the link values actually begin with '../', so when joining it eats up the last part of the base_url.
Sorry, for not suggesting a proper patch - I didn't investigate for the root cause and just made a local dirty patch with string.replace.

yes, this issue fixed at #200,
or, if you don't mind you can try a fork version of mine. .. test if the problem solved.

Your version works fine with images located just in the images directory at the server. But there are books (for example<PHONE_NUMBER>936) that have the OEBPS directory in their image path. Your version doesn't download those.
On the other had your script downloads CSSs. I didn't realise that those were skipped in the original version :)

hmm...interesting,
try it by now, I purpose new method to collecting image links

yes, now everything seems ok

Please get the latest version, this issue should be fixed by https://github.com/lorenzodifuccia/safaribooks/commit/d339d9dd0ff4b80855b806b7265105bd5ec79112

Please get the latest version, this remaining css or image downloading issue should be fixed by d339d9d

Nope, the above mentioned problem with images located in OEBPS directory exists in this commit.

One workaround (or solution?) could be add
                    if urlparse(link).path.split('/')[0] == '..':
                        link = urlparse(link).path.split('/', 1)[1]

into link_replace function, just above
                   link = urljoin(self.base_url, link)

One workaround (or solution?) could be add
                    if urlparse(link).path.split('/')[0] == '..':
                        link = urlparse(link).path.split('/', 1)[1]

into link_replace function, just above
                   link = urljoin(self.base_url, link)

Mhh this might be related to the observation in https://github.com/lorenzodifuccia/safaribooks/issues/212#issuecomment-623030088
Could you please provide the book-ids where this issue can be observed? As of now i would guess it would be a similar issue.


[SP-6278] Backport of PDI-19290 - "Ping a host": When you type a stri…

…ng in "Nr. packets to send", an execution is successful. (9.3 Suite)

:white_check_mark: Build finished in 24m 19s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine,ui

:exclamation: No tests found!
:information_source:  This is an automatic message


az login throws an error: json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

This is autogenerated. Please review and update as needed.
Describe the bug
az login --use-device-code throws an error after timeout. Once executed it halts for a while with no visible progress, then exists with the error below.
Command Name
az login
Errors:
The command failed with an unexpected error. Here is the traceback:
Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
  File "/opt/az/lib/python3.10/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/profile/custom.py", line 139, in login
    subscriptions = profile.login(
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/_profile.py", line 152, in login
    user_identity = identity.login_with_device_code(scopes=scopes, **kwargs)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/auth/identity.py", line 148, in login_with_device_code
    flow = self._msal_app.initiate_device_flow(scopes, **kwargs)
  File "/opt/az/lib/python3.10/site-packages/msal/application.py", line 1641, in initiate_device_flow
    flow = self.client.initiate_device_flow(
  File "/opt/az/lib/python3.10/site-packages/msal/oauth2cli/oauth2.py", line 328, in initiate_device_flow
    flow = json.loads(resp.text)
  File "/opt/az/lib/python3.10/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/opt/az/lib/python3.10/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/opt/az/lib/python3.10/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

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

Put any pre-requisite steps here...
az login --use-device-code

Expected Behavior
Environment Summary
Linux-4.15.0-20-generic-x86_64-with-glibc2.27, Ubuntu 18.04.6 LTS
Python 3.10.5
Installer: DEB

azure-cli 2.40.0

Extensions:
azure-firewall 0.14.1

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

@jiasli for awareness

https://github.com/AzureAD/microsoft-authentication-library-for-python/blob/38001a4d087e5d3ec9b77f4096d52d76ab359c46/msal/oauth2cli/oauth2.py#L323-L327
        resp = self._http_client.post(self.configuration[DAE],
            data={"client_id": self.client_id, "scope": self._stringify(scope or [])},
            headers=dict(self.default_headers, **kwargs.pop("headers", {})),
            **kwargs)
        flow = json.loads(resp.text)

@rayluo, looks like resp.text is empty. How can it be?

https://github.com/Azure/azure-cli/issues/23805

https://github.com/AzureAD/microsoft-authentication-library-for-python/blob/38001a4d087e5d3ec9b77f4096d52d76ab359c46/msal/oauth2cli/oauth2.py#L323-L327
        resp = self._http_client.post(self.configuration[DAE],
            data={"client_id": self.client_id, "scope": self._stringify(scope or [])},
            headers=dict(self.default_headers, **kwargs.pop("headers", {})),
            **kwargs)
        flow = json.loads(resp.text)

@rayluo, looks like resp.text is empty. How can it be?

If this is reproducible, we will need to inform the service team.


PLEASE add Anaheim, CA

email to<EMAIL_ADDRESS>(City Council)
subject: DEFUND APD
Body:
To the Anaheim City Council:
My name is [YOUR NAME] and I am a resident of [YOUR NEIGHBORHOOD/CITY]. I am writing in deep concern for the health of the greater Anaheim community. It has become more than clear that a radical shift in our concept of policing and community health must take place at the local level. Police violence in Anaheim has disproportionately affected Black and Brown residents and that must come to an end.
The United States does not have a national healthcare system. Instead, we have the largest military budget, and some of the most well-funded and militarized police departments in the world. Anaheim is no different: the 2019-2020 budget allocated $153.8 million to law enforcement and only $1 million to community development.
Anaheim is the 9th most violent police department in the U.S. From 2003-2016, Anaheim Police Department killed 33 people during the process of arrest, and nearly 40% of them were unarmed. Since 2014, the rate of arrest-related deaths caused by Anaheim PD exceeds that of LAPD, NYPD, and San Fran PD and is 74% higher than the average for police in California.
WE DEMAND that the city council not approve “Resolution 19” that would spend $700,000 dollars of taxpayer money on surveillance technology of anti-police brutality protests and that any future projects related to surveillance be halted. We also DEMAND that the council not approve "Resolution 5" that will use $100,000 to improve 10 police vehicles. Instead, invest that money in creating programs that benefit and enrich the community especially during a global pandemic.
We are calling on our elected officials to stop criminalizing our community members. We as a concerned community refuse to remain silent on the use of technology to further terrorize our community.
[YOUR NAME]

email to<EMAIL_ADDRESS>(City Council)
subject: DEFUND APD
Body:
To the Anaheim City Council:
My name is [YOUR NAME], and I am a resident of [YOUR NEIGHBORHOOD/CITY]. I am writing in deep concern for the health of the greater Anaheim community. It has become more than clear that a radical shift in our concept of policing and community health must take place at the local level. Police violence in Anaheim has disproportionately affected Black and Brown residents and that must come to an end.
The United States does not have a national healthcare system. Instead, we have the largest military budget, and some of the most well-funded and militarized police departments in the world. Anaheim is no different: the 2019-2020 budget allocated $153.8 million to law enforcement and only $1 million to community development.
Anaheim is the 9th most violent police department in the U.S. From 2003-2016, Anaheim Police Department killed 33 people during the process of arrest, and nearly 40% of them were unarmed. Since 2014, the rate of arrest-related deaths caused by Anaheim PD exceeds that of LAPD, NYPD, and San Fran PD and is 74% higher than the average for police in California.
I demand that the city council not approve “Resolution 19” that would spend $700,000 dollars of taxpayer money on surveillance technology of anti-police brutality protests and that any future projects related to surveillance be halted. I also demand that the council not approve "Resolution 5" that will use $100,000 to improve 10 police vehicles. Instead, invest that money in creating programs that benefit and enrich the community especially during a global pandemic.
I am calling on our elected officials to stop criminalizing our community members. We as a concerned community refuse to remain silent on the use of technology to further terrorize our community.
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

on it


CLI is unable to authenticate when using apiUrl

Starting from some recent CLI version, using npx @tolgee/cli pull was not sufficient to authenticate to the self-hosted server when apiUrl is present in .tolgeerc.yaml.
...
projectId: 20
apiUrl: "https://server"
format: JSON_TOLGEE
...

> npx @tolgee/cli pull
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y

🔴 No API key has been provided. You must either provide one via --api-key, or login via `tolgee login`.

Instead, I must do npx @tolgee/cli --api-url https://server pull
> npx @tolgee/cli --api-url https://server pull
🐭✅     Fetching strings from Tolgee...
🐭✅     Extracting strings...
✅ Done!

Hey, sorry for late reaction. And you are using env variable, or login command?

I did logged in with command and it requires apiurl arg at that time. Let me try once again later...

Hmm tried once again with npx @tolgee/cli login without --api-url and now it works. Dunno what's the deal with the undefined thing.
> npx @tolgee/cli login xxxxx
 ✅ Logged in as *** on undefined. Welcome back!
> npx @tolgee/cli pull
 🐭✅     Fetching strings from Tolgee...
 🐭✅     Extracting strings...
 ✅ Done!

Looking at the authentcation file it is stored in a separated entry than that stored with --api-url. Is this expected?


Setup/logic -- Try to convert full morse sentence to the English sentence

The things that we did in this PR:

Install linters
Approach modulus base
Build a method to convert morse character to English character
Build a method to convert Morse word to English word
Build a method to convert Morse sentences to English sentences

@zeff96 Thank you for the CR


PlayerSourceParser entryPoint RegExp fix

I fix RegExp of entryPoint in PlayerSourceParser class.
and it works :)

Thanks! Been really busy lately, and due to some changes on how the SoundByte mobile app works, this library is no longer needed. Will look for a better path in the future soon.

I'll test and deploy a new version tomorrow.

Wow! It's my first time to be a contributor :)
Thanks 👍
I had made another pull request since it had another issue occurred 😊


The bacardi.cc should be generated by code generator.

The bacardi.cc and bacardi.h files are in examples directory currently. The file is used to initialize bridge classes but we are not sure what is added newly in the future. Therefore, we should make it be generated automatically.

I will handle this issue.

Okay, @daehyunjimmy  Go ahead!

@daehyunjimmy Are you working on this issue?
If not so, I'll take it?

I am working on it. if have problems, I will let you know.
Thanks


[SQL][TEST] Re-run collation benchmark

What changes were proposed in this pull request?
Re-running the collation benchmark with two modifications:

UTF8_BINARY_LCASE has been renamed to UTF8_LCASE in https://github.com/apache/spark/pull/46924
UTF8_BINARY should appear first in the collation benchmark results, so performance is relative to it

Why are the changes needed?
We've changed the meaning of LCASE collation in Spark, and also modified how equality checks / hashing/ expressions work with this collation, so we need to re-run the benchmarks and identify areas of improvement.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Rxisting tests.
Was this patch authored or co-authored using generative AI tooling?
No.

also adding @dbatomic and @cloud-fan to review

thanks, merging to master!


Add static list of curated syllabi to the Classroom view

We need to add the curated syllabi to the user account Classroom view:

use content from @pandasa123
use SyllabusCard components

Good afternoon Team. I was wondering if this is the design you expect to see from this issue. I am a bit uncertain of what to do after filling up the information provided by Sanket, as there was no visual attachments to the issue. I asked Eddy to invite me to the Curated Syllabus project on Figma since I didn't have previous access to it.
If this is the final design, does anyone know where I could find a sample of the final cards? Anywhere they might have been used previously, or a similar version of them.
Thank you :)

Hey @mariacloehb, sorry about that. Here's how it looks on the Qiskit.org (and the expected order)
The items in here are placeholders:

Merged, thank you @mariacloehb


Add custom sending domain for Nova Scotia

Create a new custom sending domain for the Nova Scotia government. Will pass DNS records to their team and update services on Platform Admin after.
See client request https://cds-snc.freshdesk.com/a/tickets/7468

@jimleroyer Not having the plan check is fine, will bypass when merging


docs(guides): multi-tenant scenario the gitops-way with flux2

Closes #528

Moreover, I think we should provide a final PoC repository, migrating and cleaning up the one of mine flux2-capsule-multi-tenancy.

@maxgio92 the detailed tutorial can be part of this dedicated PoC repo and leave the short one in the main Capsule documentation. Anyway, let's to link repo each other so that the reader of main Capsule documentation can access the detailed tutorial.

Thanks @bsctl and @prometherion. I'm going to fix all the parts.

I stripped down introductions to make it a real guide. I left diagrams to help understand the setup, but they can be cleaned up too if you want to further lighen it @prometherion @bsctl.


Using 5'GEX /TCR libraries

Would it be possible to use this tool to demultiplex long cDNA reads generated through 5' TSO 10x library generation?

Do you have a cartoon showing the library structure?

Do you have a cartoon showing the library structure?

Sure. It is 5' GEX amplified library before fragmentation and then end-repair with ligation of the 1D sequencing adapters. cDNA insert should have anchor oligo-dT priming.

Sure. It is 5' GEX amplified library before fragmentation and then end-repair with ligation of the 1D sequencing adapters. cDNA insert should have anchor oligo-dT priming.

Yes. I think ampBinner_10X.py should work on the 5' TSO 10x library.
You can supply this sequence

to --barcode_upstream_seq.
Please note that ampBinner_10X.py wouldn't work well if the supplied barcode list is the complete list of all possible barcodes (a few million), as many of the barcodes have very similar sequences and are indistinguishable given the error rate of the long reads. Usually, only a subset of the barcodes was used in one experiment. So it's better that you narrow down the barcode list as much as you can to improve the accuracy.
If you encounter any issues, please let me know.

Yes. I think ampBinner_10X.py should work on the 5' TSO 10x library.
You can supply this sequence

to --barcode_upstream_seq.
Please note that ampBinner_10X.py wouldn't work well if the supplied barcode list is the complete list of all possible barcodes (a few million), as many of the barcodes have very similar sequences and are indistinguishable given the error rate of the long reads. Usually, only a subset of the barcodes was used in one experiment. So it's better that you narrow down the barcode list as much as you can to improve the accuracy.
If you encounter any issues, please let me know.

I have a whitelist of barcodes from parallel MiSeq run. Thank you I will give it try and let you know how it goes.

I have a whitelist of barcodes from parallel MiSeq run. Thank you I will give it try and let you know how it goes.

That's great!

That's great!

Is this issue solved?


Add images: Android Basics with Compose

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-add-images?authuser=1&continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-1-pathway-3%3Fauthuser%3D1%26hl%3Den%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-add-images&hl=en#1
In which task and step of the codelab can this issue be found?
Add images to your Android app, Step 2 Set up your app
Describe the problem
Broken GitHub link to image required for task: https://github.com/google-developer-training/basic-android-kotlin-compose-birthday-card-app/blob/M3_migration/app/src/main/res/drawable-nodpi/androidparty.png
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Hi @jpphl
Thank you for writing in with this issue.
The lesson has been updated to address it and has recently been republished.
Hope you are enjoying the course!
Regards.


System permissions not respected

Version
2023.3.0-dev.5.1
Description
Role permissions
As a specific role that should not list roles, I am able to see them (applications and auth clients as well).
What I found was that whatever you set in the System component (first screenshot), there will be at least these shown:
Users
Roles
Applications
AuthClients
Reproduction steps

Go to admin
Click on SYSTEM > Permissions in the sidebar
First screenshot shows the settings of the permissions
Second screenshot shows the list of roles (which should not be seen).

I.

II.

Expected behavior
The user with this role should be able to see roles.
Tested on:
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/111.0

It is expected as List and search role is inherited to grant permission, to explicitely deny it ->  we need to alt + click it to deny it's permission to the role.

Expected behavior. Moving the card to done.


Investigate modular content needs

Problem statement
As our project grows to have more channels to communicate with users, we need to investigate more scalable and transferable systems to create and manage content.
What you need to know
Acceptance criteria

[ ] Options evaluated and proposal for next steps reviewed with Design Lead and PM
[ ] Follow-on ticket and ACs created as needed

FYI @chris-kuryak @jillian-hammer

Evaluation complete, will review and plan next steps before sprint end


Bluetooth: Cannot connect if extended advertising is enabled in prj.conf

Describe the bug
A Zephyr program compiled with the following in prj.conf:
CONFIG_BT=y
CONFIG_BT_CENTRAL=y
CONFIG_BT_PERIPHERAL=y
CONFIG_BT_EXT_ADV=y
CONFIG_BT_PER_ADV=y

Calling bt_conn_le_create after discovering a peer returns -5 (-EIO) when compiled with the above config. Removing or setting to n the CONFIG_BT_EXT_ADV option and recompiling corrects the behaviour.

CONFIG_BT_EXT_ADV
CONFIG_BT_PER_ADV
Return value of bt_conn_le_create

n
n
0

n
y
0

y
n
-5

y
y
-5

To Reproduce
Steps to reproduce the behavior:

Compile the code in this gist
Flash to the boards; I tested with nucleo_wb55rg, disco_l475_iot1 and nrf52840_mdk; which is central and which is peripheral doesn't matter
Use serial console to send A to one board to tell it to advertise and S to the other to tell it to scan
The error is shown on the serial console of the scanning board when it attempts to connect

Expected behavior
It should be possible for a single program to support both connect & periodic sync.
Impact
Showstopper
Logs and console output
*** Booting Zephyr OS build zephyr-v2.5.0-2959-g9a14f627426b  ***
Initialising...
Press A to start advertising or S to start scanning
Scanning for 10 seconds
Connecting to Issue (80:e1:26:1b:76:02)
bt_conn_le_create: error -5

Environment (please complete the following information):

OS: (e.g. Linux, MacOS, Windows) Windows
Toolchain (e.g Zephyr SDK, ...): Zephyr / West
Commit SHA or Version used: 9a14f627426ba1e25e75d8ea3ebbdcccc4d9b2a7

Additional context
N/A

Support for extended advertising needs to be enabled in the zephyr controller.
CONFIG_BT_CTLR_ADV_EXT=y

@chrisswinchatt-arm if you objective is to come up with a power consumption comparison between different RTOS, may I point out that you need to take into account not only Bluetooth, but also things like other peripherals (the UART in particular) as well as the behavior of the application itself.

Did some digging because it was weird that only creating connections was affected, but advertising and scanning worked fine.
Turns out the feature bit checking was inconsistent, I've created a fix for it.

Thanks for speedy response @all.


Update concept-identity-protection-risks.md

Adding changed description for token detections both Anomalous Token and Token Issuer Anomaly

@elkuzmen : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@MicrosoftGuyJFlo, you can only sign off in the main PR conversation, not in review comments.


Closes MSAL/#418 + Version Rev

See https://github.com/AzureAD/microsoft-authentication-library-for-android/issues/418

Thanks @shoatman -- second half of this fix is here: https://github.com/AzureAD/microsoft-authentication-library-for-android/pull/424


Fix link to Sauce Labs plugin

The Sauce Labs plugin link goes to a module at version 0.0.0 that hasn't been published in 5 years. Update the link to point to the module that is at version 3.4.3 and was published two months ago. In addition, this module is within the karmar-runner GitHub org.

:white_check_mark: Build karma 2873 completed (commit https://github.com/karma-runner/karma/commit/ea9c2875d8 by @Trott)

:white_check_mark: Build karma 2873 completed (commit https://github.com/karma-runner/karma/commit/ea9c2875d8 by @Trott)

Please fix the commit message to follow the guidelines and it should be good to go. Something like docs: fix link to Sauce Labs plugin will do.

Please fix the commit message to follow the guidelines and it should be good to go. Something like docs: fix link to Sauce Labs plugin will do.

Please fix the commit message to follow the guidelines and it should be good to go. Something like docs: fix link to Sauce Labs plugin will do.

Sure, fixed.

Please fix the commit message to follow the guidelines and it should be good to go. Something like docs: fix link to Sauce Labs plugin will do.

Sure, fixed.

:white_check_mark: Build karma 2878 completed (commit https://github.com/karma-runner/karma/commit/c711c1b7f1 by @Trott)

:white_check_mark: Build karma 2878 completed (commit https://github.com/karma-runner/karma/commit/c711c1b7f1 by @Trott)


Clear varnish after deploy

No loner needed if we're going to clear Varnish after maintenance was enabled, see #18.


TeeAsync overload awaits in wrong order

Consider this piece of code:
static void Main(string[] args)
{
    var val = GetValue().TeeAsync(Print).Result;
}

static async Task<int> GetValue()
{
    await Task.Delay(1000);
    System.Console.WriteLine("Get value");
    return 0;
}

static Task Print()
{
    Console.WriteLine("Print");
    return Task.FromResult(0);
}

What is the result of running this piece of code? One would expect that the string "Get value" would be printed to the console followed by "Print". However, the string "Print" is printed first, followed by "Get value". Thus, the ordering is not what one would expect.
The problem stems from this method, which awaits the result of the given action before awaiting the result of @this.
It is worth noticing that this is inconsistent when compared to the overloads of TeeAsync taking a Func<TValue, Task> act as the second argument. They need to await @this before they can invoke act. Likewise, this should await @this before invoking act.

Fixed by the man himself :dancing_men:


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_36bd1281-3945-4347-910e-ad39b213bc71/pull/1 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target_36bd1281-3945-4347-910e-ad39b213bc71.git

Docs Build status updates of commit 0d77576:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-10-11-03-13-33 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-10-11-03-13-33

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


Remove SlideListener inside a listener will throw ConcurrentModificationException

I have something like
new SlidingUpPanelLayout.SimplePanelSlideListener() {
            @Override
            public void onPanelStateChanged(...) {
				mSlidingLayout.removePanelSlideListener(this);
			}
}

This will throw ConcurrentModificationException.

Looks like this is because:
private List<PanelSlideListener> mPanelSlideListeners = new ArrayList<>();
Should probably be a CopyOnWriteArrayList http://docs.oracle.com/javase/6/docs/api/java/util/concurrent/CopyOnWriteArrayList.html


Create base VWSException for erroneous responses from VWS

Thank you @laymonage for this draft PR. Please ping me when you would like a review!

I know this is a draft, but would a user not expect UnknownVWSErrorPossiblyBadName to be a VWSException?

@adamtheturtle I wasn't sure if we should only use VWSException for exceptions that are in the table, but you're right, I guess it's better if we just make all of them inherit from VWSException to simplify things.

I based VWSException from BaseVWSException to distinguish exceptions that have a Response object as property and those that don't. If that's not what you want, we can remove BaseVWSException and just use VWSException, and set an empty Response() (or None) as the default value.

Thank you again @laymonage for your continued work on this.

@adamtheturtle I wasn't sure if we should only use VWSException for exceptions that are in the table, but you're right, I guess it's better if we just make all of them inherit from VWSException to simplify things.

In this draft PR I see that MatchProcessing and MaxNumResultsOutOfRange are made to inherit from VWSException.
You will notice that in the table you linked to, there is no MatchProcessing or MaxNumResultsOutOfRange.
There is a naming confusion in this package which stems from Vuforia changing and mixing names. Vuforia Web Services (VWS) mostly refers to the APIs which allow one to add and manage targets. The Vuforia Web Query API (VWQ) allows one to make
I appreciate that this problem is present in the naming of this tool being vws-python and with the exceptions file and the package being vws.
Perhaps we could have VWQException or CloudRecoException.
This may mean that someone has to do except (VWSException, CloudRecoException) but I think I'm OK with that.
This also resolves the "problem" that ConnectionErrorPossiblyImageTooLarge is inherently different to the VWS Exceptions from the table.

I based VWSException from BaseVWSException to distinguish exceptions that have a Response object as property and those that don't. If that's not what you want, we can remove BaseVWSException and just use VWSException, and set an empty Response() (or None) as the default value.

I think that as a user it is hard to know whether I should use VWSException or BaseVWSException.
Looking at your comment on https://github.com/VWS-Python/vws-python/issues/1271#issuecomment-674513881 I see:

I want to catch the exceptions that might arise from this package, but I don't want to handle them individually.

This is not possible with this change as it is with BaseVWSException and VWSException if I want to rely on response when it is there. I have to use:
try:
    ... # vws-specific code
except BaseVWSException:
    ...
except VWSException as exc:
    status_code = exc.response.status_code
    ...

I think that this is confusing and I have to carefully inspect the documentation to understand what is BaseVWSException and what is plain VWSException. I'm happy to discuss if you disagree.
In the current code base, the following is the only place in which UnknownVWSErrorPossiblyBadName is raised:
https://github.com/VWS-Python/vws-python/blob/f3356f9e5629229d8b2834a655affb53123c01c6/src/vws/_result_codes.py#L48-L52
We can see here that response exists - it just is not passed through to UnknownVWSErrorPossiblyBadName.__init__.
I believe that you can therefore make the change to have UnknownVWSErrorPossiblyBadName take a response and have a response property.
This does not resolve your motivation completely as it still leaves ConnectionErrorPossiblyImageTooLarge, but it resolves it for all exceptions raised by Vuforia Web Services (not the Vuforia Web Query API).

In summary, I believe that there are a few options and a few issues.
My guess of the best way forward is:

Add response to UnknownVWSErrorPossiblyBadName.
Instead of BaseVWSException and VWSException, go with CloudRecoException and VWSException, or just exclude the CloudReco exceptions from this unification.
Add a test which would fail if the changes in this PR were reverted (this could be modifications of existing tests).

This is looking really good @laymonage - please let me know when you would like me to have a look.

Hi @adamtheturtle, I think it's ready for review.

Thank you @laymonage ! In particular for your persistence on this.
I appreciate your work and your high quality code. I will merge this and follow up with some minor changes - for example by crediting you in the CHANGELOG and doing a release.
I will also consider moving some things around - perhaps vws.vws_exceptions/vwq_exceptions/custom_exceptions or similar.


New lookup idea: env

This seems pretty obvious, so not sure why we haven't done it before, but it might be useful to have a lookup that lets you look at environment variables in the shell.

Resolved with #418


Adding CUSTOM_CERTIFICATE environment variable

Adding a CUSTOM_CERTIFICATE environment variable for use in environments with self-signed certificates.

Already implemented in master


fix: not possible to drag&drop file with multiple dots

Description

Fixes #485
It is not possible to drag & drop files containing multiple dots in their name when using an "accept" filter.
This PR aims to change the behavior so that first and foremost, we split file names by . and then use the last dot and not the first.
Secondly, we are now treating and dividing accept parameters in a setter on the property so that it is only done once, and we now also support receiving file extensions both with and without a . in the accept property.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)

How to test?

Open Storybook and go to the "Accept" story
Change the 'accept' property to something like "png" and try and upload files with one and two commas.

Checklist

[x] If my change requires a change to the documentation, I have updated the documentation in this pull request.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.

LGTM


Correct EC2 SpotFleet LaunchSpecifications

LaunchSpecifications should be an array, as in the reference documentation
http://docs.aws.amazon.com/fr_fr/AWSCloudFormation/latest/UserGuide/aws-properties-ec2-spotfleet-spotfleetrequestconfigdata.html

Thanks!


updated README.md with reference genomes and alignment documentation

Some comments about the changes I made:

I added alignment.run_star_and_htseq to quick reference table to help users understand the paralleled relationship between alignment.10x_count and alignment.run_star_and_htseq, and the "subsidiary" relationship between alignment.run_star_and_htseq and aws_star. After I complete writing the script that wraps multiple 10x_count runs together, I'll edit the reference table again to leave only aws_star and say aws_10x_count there.
Please help check these two things about genome that I probably made mistakes: 1) In the subsection "How to launch alignment jobs", whether I used the proper genomes in the examples of running 10x_count, run_star_and_htseq, and aws_star; 2) In the subsection "How to choose the right genome (--taxon argument in running the alignment job)", whether I attached correct links for all those genomes.

Some thoughts about possible next steps. Some of them may be trivial and unnecessary, but if they sound good, I could work to fix them:

Make --s3_output_path positional in aws_star (it's now the only argument that's not positional), or make all the arguments not positional (which I think may be clearer).
aws_star uses --s3_input_path & 10x alignment uses --s3_input_dir. Could be a bit confusing. Could make both path or dir.
Delete --num_partitions and --partition_id arguments in run_star_and_htseq since the input folder is a single sample instead of a bunch of samples. This may help prevent confusion.
Update genomes to the latest versions.

Only thing you might want to add is for the "PLUS" genomes. Besides transgenes, they also include sequences for ERCCs (these are a standardized mix of spike-in sequences used as a control).

Why'd you close it?


Question: encoder options for interleaving image sequence

I'm not sure if it is openh264 encoder's fault or if I have to blame myself for a complete lack of understanding of H.264 and the concept of reference frames.
For a certain type of experiment I have two list of images.

A0 -> A1 -> A2 -> A3 -> A4 -> An
B0 -> B1 -> B2 -> B3 -> B4 ->Bn

The visual difference to the next image in each list is extremely small.
If encoding only the A frames everything is fine.
The size of the encoded data between A[x] -> A[x+1] is minimal as expected.
Same for B of course.
However, I have to encode the frames in the following required way:
A[0] -> B[0] -> A[1] -> B[1] -> A[2] -> B[2] -> .... -> A[n] -> B[n]
Now the visual difference between the encoded frame is big of course.
But also the encoded size of the "A interleaved with B movie" is ten times the size of the sum of "A movie" plus "B movie".
I was expecting that openh264 would recognize that e.g. B[2] is very similar to B[1] and use B[1] as a reference frame when encoding [B2].
Even if H.264 is was not designed for that kind of input, are there any encoding options that can be set to optimize the encoded size for this special input sequence?

@sijchen any news regarding this feature ?


RBloomFilter contains async Implementation

I'm using RBloomFilter and I saw there isn't a async variation of the contains method.
Why is that? is there any particular reason it's the only method missing an async variation?
Is it possible to implement it by my self with another redis client?

Implemented in https://github.com/redisson/redisson/issues/5836


Test for squashfs recovery

Instead of bindly doing a search which results in a error if recovery is not squash, just do a test to see if the file exit and set the parameters based on that
This should stop that ugly error message on recovery grub menu selection

lets go and pray that I dont break the whole boot process lol


Way to return view to "all namespaces"

What would you like to be added:
By default, I have all namespaces displayed in Lens. When I select one of the namespaces from the dropdown, I have not found a way to return to the "all namespaces" view without restarting Lens.
Why is this needed:
Having a way to look at all namespaces is very convenient, so it should be possible to single out a namespace and then return to the overview of all ns.
Environment you are Lens application on:

Kubernetes distribution: [EKS, AKS, GKE, Rancher, ...] Kubespray, k8s v.1.15
Desktop OS: Windows 10 1909

You just unselect the namespace by clicking on it again, so it removes the tick next to the namespace..

Yes but if you have lots of namespaces selected this is not easy.
However if you go to "Overwiew", under "Workloads", there is "CURRENTLY APPLIED FILTERS" above circle graph with a reset button which, I think, does what you want.

Thanks, this helps a lot!


M1 perf. tests. Who has M2/M3 could you run tests and publish results?

https://github.com/ivanrylov/mlx-m1-pro-speed-tests/

(.venv) user@box mnist % python main.py
Epoch 0: Test accuracy 0.866, Time 0.097 (s)
Epoch 1: Test accuracy 0.896, Time 0.088 (s)
Epoch 2: Test accuracy 0.917, Time 0.097 (s)
Epoch 3: Test accuracy 0.928, Time 0.091 (s)
Epoch 4: Test accuracy 0.935, Time 0.089 (s)
Epoch 5: Test accuracy 0.938, Time 0.087 (s)
Epoch 6: Test accuracy 0.946, Time 0.088 (s)
Epoch 7: Test accuracy 0.941, Time 0.089 (s)
Epoch 8: Test accuracy 0.950, Time 0.086 (s)
Epoch 9: Test accuracy 0.948, Time 0.088 (s)
(.venv) user@box mnist % python main.py --gpu
Epoch 0: Test accuracy 0.866, Time 0.241 (s)
Epoch 1: Test accuracy 0.896, Time 0.136 (s)
Epoch 2: Test accuracy 0.917, Time 0.141 (s)
Epoch 3: Test accuracy 0.928, Time 0.135 (s)
Epoch 4: Test accuracy 0.935, Time 0.134 (s)
Epoch 5: Test accuracy 0.938, Time 0.152 (s)
Epoch 6: Test accuracy 0.946, Time 0.136 (s)
Epoch 7: Test accuracy 0.941, Time 0.135 (s)
Epoch 8: Test accuracy 0.950, Time 0.136 (s)
Epoch 9: Test accuracy 0.948, Time 0.135 (s)
m2pro 32gb

Thanks @bigsnarfdude!
Transformer_lm % python main.py --gpu:
M1Pro 32BG -  It/sec 1
M2Pro 32BG - It/sec 1.4
Someone with M3 please run the test :)

M1 Ultra 128GB
transformer_lm % python main.py
Training a transformer with 153.883 M parameters
Iter 10: Train loss 8.963, It/sec 0.347
Iter 20: Train loss 8.379, It/sec 0.354
Iter 30: Train loss 8.075, It/sec 0.354
Iter 40: Train loss 7.915, It/sec 0.355
Iter 50: Train loss 7.818, It/sec 0.354
Iter 60: Train loss 7.747, It/sec 0.354
Iter 70: Train loss 7.671, It/sec 0.354
Iter 80: Train loss 7.592, It/sec 0.354
Iter 90: Train loss 7.550, It/sec 0.354
Iter 100: Train loss 7.529, It/sec 0.354
transformer_lm % python main.py --gpu
Training a transformer with 153.883 M parameters
Iter 10: Train loss 8.880, It/sec 3.711
Iter 20: Train loss 8.277, It/sec 4.034
Iter 30: Train loss 8.052, It/sec 4.043
Iter 40: Train loss 7.922, It/sec 4.040
Iter 50: Train loss 7.737, It/sec 4.031
Iter 60: Train loss 7.799, It/sec 4.035
Iter 70: Train loss 7.627, It/sec 4.036
Iter 80: Train loss 7.658, It/sec 4.042
Iter 90: Train loss 7.589, It/sec 4.037
Iter 100: Train loss 7.578, It/sec 4.035
mnist % python main.py
Epoch 0: Test accuracy 0.866, Time 0.089 (s)
Epoch 1: Test accuracy 0.896, Time 0.080 (s)
Epoch 2: Test accuracy 0.917, Time 0.091 (s)
Epoch 3: Test accuracy 0.928, Time 0.089 (s)
Epoch 4: Test accuracy 0.935, Time 0.097 (s)
Epoch 5: Test accuracy 0.938, Time 0.089 (s)
Epoch 6: Test accuracy 0.946, Time 0.084 (s)
Epoch 7: Test accuracy 0.941, Time 0.092 (s)
Epoch 8: Test accuracy 0.950, Time 0.089 (s)
Epoch 9: Test accuracy 0.948, Time 0.091 (s)
mnist % python main.py --gpu
Epoch 0: Test accuracy 0.866, Time 0.277 (s)
Epoch 1: Test accuracy 0.896, Time 0.184 (s)
Epoch 2: Test accuracy 0.917, Time 0.185 (s)
Epoch 3: Test accuracy 0.928, Time 0.180 (s)
Epoch 4: Test accuracy 0.935, Time 0.182 (s)
Epoch 5: Test accuracy 0.938, Time 0.179 (s)
Epoch 6: Test accuracy 0.946, Time 0.179 (s)
Epoch 7: Test accuracy 0.941, Time 0.182 (s)
Epoch 8: Test accuracy 0.950, Time 0.182 (s)
Epoch 9: Test accuracy 0.948, Time 0.180 (s)

M1Pro 32BG - It/sec 1
M2Pro 32BG - It/sec 1.4
M1Ultra         - It/sec 4.004

Knowing #GPU cores would be useful

Here's silly question; where's main.py ? And which model you used for the test?

Here's silly question; where's main.py ? And which model you used for the test?

https://github.com/ml-explore/mlx-examples/blob/main/transformer_lm/main.py

MacBook-Pro-M3-Max-128:transformer_lm$ python main.py
Training a transformer with 153.883 M parameters
Iter 10: Train loss 9.016, It/sec 0.463
Iter 20: Train loss 8.450, It/sec 0.472
Iter 30: Train loss 8.136, It/sec 0.471
Iter 40: Train loss 7.989, It/sec 0.471
Iter 50: Train loss 7.869, It/sec 0.472
Iter 60: Train loss 7.763, It/sec 0.456
Iter 70: Train loss 7.694, It/sec 0.470
Iter 80: Train loss 7.644, It/sec 0.470
Iter 90: Train loss 7.591, It/sec 0.469
Iter 100: Train loss 7.593, It/sec 0.470
MacBook-Pro-M3-Max-128:transformer_lm$ python main.py --gpu
Training a transformer with 153.883 M parameters
Iter 10: Train loss 8.954, It/sec 2.661
Iter 20: Train loss 8.365, It/sec 2.941
Iter 30: Train loss 8.066, It/sec 2.940
Iter 40: Train loss 7.904, It/sec 2.938
Iter 50: Train loss 7.802, It/sec 2.937
Iter 60: Train loss 7.749, It/sec 2.918
Iter 70: Train loss 7.673, It/sec 2.834
Iter 80: Train loss 7.578, It/sec 2.770
Iter 90: Train loss 7.608, It/sec 2.729
Iter 100: Train loss 7.548, It/sec 2.688
[16-Core / 40-Core]

Note those were on mlx v0.0.5. I just upgraded to 0.0.6 and it's a bit faster:
MacBook-Pro-M3-128:transformer_lm$ python main.py --gpu
Training a transformer with 153.883 M parameters
Iter 10: Train loss 8.982, It/sec 2.902
Iter 20: Train loss 8.337, It/sec 2.957
Iter 30: Train loss 8.084, It/sec 2.957
Iter 40: Train loss 7.940, It/sec 2.952
Iter 50: Train loss 7.821, It/sec 2.943
Iter 60: Train loss 7.711, It/sec 2.937
Iter 70: Train loss 7.684, It/sec 2.937
Iter 80: Train loss 7.585, It/sec 2.938
Iter 90: Train loss 7.564, It/sec 2.909
Iter 100: Train loss 7.571, It/sec 2.847

transformer_lm % python main.py --gpu

M1Pro 32GB - It/sec 1
M2Pro 32GB - It/sec 1.4
M1Ultra    - It/sec 4.004
M3 128GB   - It/sec 2.9

M2Max 64Gb (12 core CPU, 38 core GPU)
python main.py
Training a transformer with 153.883 M parameters
Iter 10: Train loss 8.829, It/sec 0.344
Iter 20: Train loss 8.292, It/sec 0.351
Iter 30: Train loss 8.083, It/sec 0.351
Iter 40: Train loss 7.908, It/sec 0.351
Iter 50: Train loss 7.789, It/sec 0.345
Iter 60: Train loss 7.762, It/sec 0.345
Iter 70: Train loss 7.679, It/sec 0.350
Iter 80: Train loss 7.621, It/sec 0.350
Iter 90: Train loss 7.579, It/sec 0.350
Iter 100: Train loss 7.549, It/sec 0.351
python main.py --gpu
Training a transformer with 153.883 M parameters
Iter 10: Train loss 8.892, It/sec 2.432
Iter 20: Train loss 8.284, It/sec 2.035
Iter 30: Train loss 8.019, It/sec 2.033
Iter 40: Train loss 7.944, It/sec 2.048
Iter 50: Train loss 7.791, It/sec 2.015
Iter 60: Train loss 7.737, It/sec 2.038
Iter 70: Train loss 7.657, It/sec 2.033
Iter 80: Train loss 7.593, It/sec 2.026
Iter 90: Train loss 7.538, It/sec 2.008
Iter 100: Train loss 7.572, It/sec 2.005

transformer_lm % python main.py --gpu

M1Pro 32GB - It/sec 1
M2Pro 32GB - It/sec 1.4
M1Ultra    - It/sec 4.004
M3 128GB   - It/sec 2.9
M2Max 64Gb - It/sec 2

M3 Pro 36 GB (11 Core CPU, 14 Core GPU) - MLX v0.0.6
python main.py --gpu
Training a transformer with 153.883 M parameters
Iter 10: Train loss 8.936, It/sec 1.043
Iter 20: Train loss 8.347, It/sec 1.072
Iter 30: Train loss 8.058, It/sec 1.075
Iter 40: Train loss 7.937, It/sec 1.069
Iter 50: Train loss 7.868, It/sec 1.069
Iter 60: Train loss 7.693, It/sec 1.071
Iter 70: Train loss 7.694, It/sec 1.071
Iter 80: Train loss 7.613, It/sec 1.074
Iter 90: Train loss 7.618, It/sec 1.074
Iter 100: Train loss 7.504, It/sec 1.077

These MLX numbers are looking close to same performance slope here https://github.com/ggerganov/llama.cpp/discussions/4167

These MLX numbers are looking close to same performance slope here https://github.com/ggerganov/llama.cpp/discussions/4167