emit type errors with<EMAIL_ADDRESS>

Vue - Official extension or vue-tsc version
2.0.29
VSCode version
Version: 1.91.1 Commit: f1e16e1e6214d7c44d078b1f0607b2388f29d729 Date: 2024-07-09T22:07:46.768Z (3 wks ago)
Vue version
<EMAIL_ADDRESS>TypeScript version
5.5.3
System Info
System:
    OS: macOS 14.5
    CPU: (12) arm64 Apple M3 Pro
    Memory: 49.23 MB / 18.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.11.0 - ~/.nvm/versions/node/v20.11.0/bin/node
    npm: 10.2.4 - ~/.nvm/versions/node/v20.11.0/bin/npm
    pnpm: 8.14.1 - ~/Library/pnpm/pnpm
  Browsers:
    Chrome: 126.0.6478.185
    Edge: 127.0.2651.74
    Firefox Nightly: 130.0a1
    Safari: 17.5

Steps to reproduce

Create a new component

<template>
    <form @submit="onSubmit" />
</template>

Run vue-tsc --build --force

What is expected?
Running vue-tsc --build --force should not error on this.
What is actually happening?
The following type error is thrown
src/Repo.vue:6:6 - error TS2345: Argument of type '{ onSubmit: (event: Event) => void; }' is not assignable to parameter of type 'ClassAttributes<HTMLFormElement> & FormHTMLAttributes<HTMLFormElement> & Record<string, unknown>'.
  Type '{ onSubmit: (event: Event) => void; }' is not assignable to type 'FormHTMLAttributes<HTMLFormElement>'.
    Types of property 'onSubmit' are incompatible.
      Type '(event: Event) => void' is not assignable to type 'FormEventHandler<HTMLFormElement>'.
        Types of parameters 'event' and 'event' are incompatible.
          Type 'FormEvent<HTMLFormElement>' is not assignable to type 'Event'.

6     <form @submit="onSubmit" />

Vscode does not display the error inline, which is the reason I suspected vue-tsc instead of vue.

Link to minimal reproduction
No response
Any additional comments?
I having the same problem in different shapes throughout the codebase:
src/components/Link/Link.vue:28:8 - error TS2345: Argument of type '{ href: string; "aria-current": false | "page"; onClick: (e?: MouseEvent) => Promise<void | NavigationFailure>; }' is not assignable to parameter of type 'ClassAttributes<HTMLAnchorElement> & AnchorHTMLAttributes<HTMLAnchorElement> & Record<string, unknown>'.
  Type '{ href: string; "aria-current": false | "page"; onClick: (e?: MouseEvent) => Promise<void | NavigationFailure>; }' is not assignable to type 'AnchorHTMLAttributes<HTMLAnchorElement>'.
    Types of property 'onClick' are incompatible.
      Type '(e?: MouseEvent | undefined) => Promise<void | NavigationFailure>' is not assignable to type 'MouseEventHandler<HTMLAnchorElement>'.
        Types of parameters 'e' and 'event' are incompatible.
          Type 'MouseEvent<HTMLAnchorElement, MouseEvent>' is missing the following properties from type 'MouseEvent': layerX, layerY, offsetX, offsetY, and 16 more.

28       <a

src/components/Input/Input.vue:164:9 - error TS2769: No overload matches this call.
  Overload 1 of 3, '(e: "change", event: Event): void', gave the following error.
    Argument of type 'ChangeEvent<HTMLInputElement>' is not assignable to parameter of type 'Event'.
  Overload 2 of 3, '(e: "blur", event: Event): void', gave the following error.
    Argument of type '"change"' is not assignable to parameter of type '"blur"'.
  Overload 3 of 3, '(e: "keyup", event: Event): void', gave the following error.
    Argument of type '"change"' is not assignable to parameter of type '"keyup"'.

164         emit('change', event)

src/components/InlineNavigation/Item.vue:58:8 - error TS2345: Argument of type '{ href: string; "aria-current": boolean; onClick: (event: Event) => Promise<void>; }' is not assignable to parameter of type 'ClassAttributes<HTMLAnchorElement> & AnchorHTMLAttributes<HTMLAnchorElement> & Record<string, unknown>'.
  Type '{ href: string; "aria-current": boolean; onClick: (event: Event) => Promise<void>; }' is not assignable to type 'AnchorHTMLAttributes<HTMLAnchorElement>'.
    Types of property 'onClick' are incompatible.
      Type '(event: Event) => Promise<void>' is not assignable to type 'MouseEventHandler<HTMLAnchorElement>'.
        Types of parameters 'event' and 'event' are incompatible.
          Type 'MouseEvent<HTMLAnchorElement, MouseEvent>' is missing the following properties from type 'Event': cancelBubble, composed, returnValue, srcElement, and 7 more.

58       <a :href="`#${props.target}`" :aria-current="isActive" @click.prevent="handleNavigation">
          ~

Another interesting instance of this seems to be the following:
<input
    v-else
    :type="type"
    v-bind="$attrs"
    @change="
      (event) => {
        emit('change', event)
      }
    "
    @blur="
      (event) => {
        emit('blur', event)
      }
    "
    @keyup="
      (event) => {
        emit('keyup', event)
      }
    "
  />

In Vscode the type is inferred properly:

but vue-tsc throws an error
src/components/Input/Input.vue:173:8 - error TS7006: Parameter 'event' implicitly has an 'any' type.

173       (event) => {

I have installed the vue canary minor version as documented here: https://vuejs.org/about/releases. My package.json now looks like:
{
  "dependencies": {
    "vue": "npm<EMAIL_ADDRESS>  },
  "overrides": {
    "vue": "npm<EMAIL_ADDRESS>    "@vue/compiler-core": "npm<EMAIL_ADDRESS>    "@vue/compiler-dom": "npm<EMAIL_ADDRESS>    "@vue/compiler-sfc": "npm<EMAIL_ADDRESS>    "@vue/compiler-ssr": "npm<EMAIL_ADDRESS>    "@vue/reactivity": "npm<EMAIL_ADDRESS>    "@vue/runtime-core": "npm<EMAIL_ADDRESS>    "@vue/runtime-dom": "npm<EMAIL_ADDRESS>    "@vue/server-renderer": "npm<EMAIL_ADDRESS>    "@vue/shared": "npm<EMAIL_ADDRESS>    "@vue/compat": "npm<EMAIL_ADDRESS>  }
}

I hope that helps. If you need any more help reproducing the issue please let me know 🙏🏼

Maybe another interesting bit to this: I am observing other obscure errors whenever conditional types are in play. For example:

<template>

    {{
      (
        translationInitialSectionsFlat.find(
          (section) => section.type === row.type && section.id === props.row.id
        ) as RowMeasure | undefined
      )?.description?.text ?? t('Missing translation')
    }}

</template>

While `translationInitialSectionsFlat` is properly typed in `script setup` it suddenly looses its type in the template:

Script

Template

Hi @gu-stav you seem to be using vue @ version 3.2 - is that correct?
Please try using just vue@latest (5.4.x). There seems to be something wrong with your canary override

This canary release is outdated. Please use 3.4.x or the 3.5 alpha version.

@davidmatter How is this canary release outdated? This is what the offical documentation about using the canary version is recommending: https://vuejs.org/about/releases
Also: the canary release is from Junly the 15th and since then there hasn't been a new canary release.

Not sure about that, you might want to ask in @vue/core

I've created a PR @ vue/core https://github.com/vuejs/core/pull/11516
This will fix the broken canary release workflow.

Thanks a lot @davidmatter 🙏🏼


Support autoscaling/v2

Now the chart support autoscaling/v1 and autoscaling/v2beta2 for auto scaling.  autoscaling/v2beta2 is deprecated in k8s 1.23 and will be removed in k8s 1.26 (expected to be released this year). So we are considering to support autoscaling/v2 API.

Just wanted to note that to make this work we don't strictly need to drop support for older versions just yet because we can use some of the helm's logic to make it deduce cluster capabilities and use appropriate HPA version this way, just like it's already done e.g. here https://github.com/Kong/charts/blob/632a6f7202fc51ddf2f2b78c88e10b8c456eb543/charts/kong/templates/hpa.yaml#L2


NFT tutorial

This PR profit from this project awesome settings to publish a tutorial on how to recover NFT gift from OKP4.
There's a french and an english version.
It will be available there :

https://docs.okp4.network/docs/nft-tutorial/en
https://docs.okp4.network/docs/nft-tutorial/fr

@tpelliet, feel free to amend this PR :smiley:

@tpelliet I added a note regarding Julien's remark. Is it ok to merge with you?


Support For Vector Migration Changes

adding support for new vector migration changes (per species, only one type of migration). The new vector_migration_example and reporter tests will throw errors until emod-malaria is updated the the latest EMOD code.
see: https://github.com/InstituteforDiseaseModeling/EMOD/pull/36

closing, because combining with fraction-to-ratio PR to make : https://github.com/InstituteforDiseaseModeling/emodpy-malaria/pull/6


Help needed on configuration

Hi,
there is some way to debug what is happening when molecule test command is executed with this driver?
I'm using a simple configuration:
driver:
  name: molecule-proxmox
  options:
     api_host: xxxxxxx
     api_user: root@pam
     api_password: "********"
     node: pve
     ssh_user: root
     ssh_identity_file: /path/to/id_rsa
platforms:
  - name: test01
    template_name: template01
    ciuser: root
    cipassword: root
    ipconfig:
      ipconfig0: 'ip=x.y.z.k/24,gw=x.y.z.1'
    nameservers:
      - <IP_ADDRESS>

but when I try to exec a simple test (using default scenario auto-generated files) nothing happens:
WARNING  Driver molecule-proxmox does not provide a schema.
INFO     default scenario test matrix: dependency, cleanup, destroy, syntax, create, prepare, converge, idempotence, side_effect, verify, cleanup, destroy
INFO     Performing prerun with role_name_check=0...
INFO     Running default > dependency
WARNING  Skipping, missing the requirements file.
WARNING  Skipping, missing the requirements file.
INFO     Running default > cleanup
WARNING  Skipping, cleanup playbook not configured.
INFO     Running default > destroy

PLAY [Destroy] *****************************************************************

TASK [Populate instance config] ************************************************
ok: [localhost]

TASK [Dump instance config] ****************************************************
skipping: [localhost]

PLAY RECAP *********************************************************************
localhost                  : ok=1    changed=0    unreachable=0    failed=0    skipped=1    rescued=0    ignored=0

INFO     Running default > syntax

playbook: molecule/default/converge.yml
INFO     Running default > create

PLAY [Create] ******************************************************************

TASK [Populate instance config dict] *******************************************
skipping: [localhost]

TASK [Convert instance config dict to a list] **********************************
skipping: [localhost]

TASK [Dump instance config] ****************************************************
skipping: [localhost]

PLAY RECAP *********************************************************************
localhost                  : ok=0    changed=0    unreachable=0    failed=0    skipped=3    rescued=0    ignored=0

INFO     Running default > prepare

PLAY [Prepare] *****************************************************************

TASK [Waiting for instance ssh connection.] ************************************

No connections on proxmox server, so no machines are created...

Are you sure that you are able to reach the spun up machine on the default ssh port? It seems like ansible is not able to reach the machine via ssh on port 22. In your other issue you mentioned using an alternative port. Could this be an issue here?
I would furthermore suggest you run molecule for more parameters (-v) to find out what molecule is trying to do there. Maybe molecule uses an (for you at least) unexpected ansible.cfg file and the therein used ssh-port to connect.
You can actually override the connection port (and other parameters) in your molecule config:
driver:
  name: molecule-proxmox
  options:
  [...]
platforms:
  [...]
provisioner:
  name: ansible
  config_options:
    ssh-connection:
      host_key_checking: false

Hi, thanks for reply!
Nope, is not related to other raised issue, after raising that, I updated connection configs and reach directly the pve server using standard 22 and 8006 ports.

You can actually override the connection port (and other parameters) in your molecule config:

How? This is not reported in docs, so for this I asked for adding parameters in other issue.
I'll try with -v option asap.

Hi @IamLunchbox,
I tried with more verbosity and I can see that molecule is waiting to connect to the new VM but no VM are going to start so, I'm pretty sure that something should be configured in create.yml and destroy.yml files, but I can find any documentation on how to compile them with this provider.

I fear I cant help you more specifically, there could be loads of issues why it doesnt work and imho github issues are not the place to do that. You should probably check through tcpdump and on the vm if network works at all.
I personally skip the prepare step (after the connection works) all together. If you check the prepare.yml you'll see, that you cant set sethostname to no to do that.
Aug 11, 2024 08:54:26 bitchecker @.***>:

Hi @IamLunchbox[https://github.com/IamLunchbox],
I tried with more verbosity and I can see that molecule is waiting to connect to the new VM but no VM are going to start so, I'm pretty sure that something should be configured in create.yml and destroy.yml files, but I can find any documentation on how to compile them with this provider.
—
Reply to this email directly, view it on GitHub[https://github.com/meffie/molecule-proxmox/issues/17#issuecomment-2282803582], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ANRA34JEJRTWEBRSPMTEMILZQ6CLBAVCNFSM6AAAAABME5GTZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBSHAYDGNJYGI].
You are receiving this because you were mentioned.
[Tracking image][https://github.com/notifications/beacon/ANRA34II37VOCCBRW65MPZ3ZQ6CLBA5CNFSM6AAAAABME5GTZKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUICDIX4.gif]

The big problem is that pve is not starting a new vm from the template...so it seems that nothing happens.
I'm asking for prepare.yml and destroy.yml because of that.

In that case you could manually edit the create.yml which comes with installing the plugin and see if it works then. Maybe the variable interpolation is off somehow - but i dont know why the create step should report a successful creation then.
Can you post your redacted molecule.yml and the execution log of molecule test -v here?
Aug 12, 2024 02:41:52 bitchecker @.***>:

The big problem is that pve is not starting a new vm from the template...so it seems that nothing happens.
I'm asking for prepare.yml and destroy.yml because of that.
—
Reply to this email directly, view it on GitHub[https://github.com/meffie/molecule-proxmox/issues/17#issuecomment-2283405434], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ANRA34O3EEGFQMYTSMJFZ2TZRBYM5AVCNFSM6AAAAABME5GTZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBTGQYDKNBTGQ].
You are receiving this because you were mentioned.
[Tracking image][https://github.com/notifications/beacon/ANRA34ND7SO4VZKZ3KJ6P5DZRBYM5A5CNFSM6AAAAABME5GTZKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUIDIAHU.gif]

molecule.yml file:
---
driver:
  name: molecule-proxmox
  options:
    api_host: xxxxxxxxx
    api_user: root@pam
    api_password: xxxxxxx
    node: proxmox
    ssh_user: root
    ssh_identity_file: /path/to/ssh/private/key
platforms:
  - name: molecule
    template_name: Alma9-template
    ciuser: root
    cipassword: root
    ipconfig:
      ipconfig0: 'ip=<IP_ADDRESS>/24,gw=<IP_ADDRESS>'
    nameservers:
      - <IP_ADDRESS>
provisioner:
  name: ansible
  config_options:
    ssh-connection:
      host_key_checking: false
      ansible_ssh_common_args: '-p ssh-port'
  lint:
    name: ansible-lint

molecule -v test output:
PLAY [Create] ******************************************************************

TASK [Populate instance config dict] *******************************************
skipping: [localhost] => {"changed": false, "false_condition": "server.changed | default(false) | bool", "skip_reason": "Conditional result was False"}

TASK [Convert instance config dict to a list] **********************************
skipping: [localhost] => {"changed": false, "false_condition": "server.changed | default(false) | bool", "skip_reason": "Conditional result was False"}

TASK [Dump instance config] ****************************************************
skipping: [localhost] => {"changed": false, "false_condition": "server.changed | default(false) | bool", "skip_reason": "Conditional result was False"}

PLAY RECAP *********************************************************************
localhost                  : ok=0    changed=0    unreachable=0    failed=0    skipped=3    rescued=0    ignored=0

INFO     Running default > prepare
Using /molecule/default/ansible.cfg as config file

PLAY [Prepare] *****************************************************************

TASK [Waiting for instance ssh connection.] ************************************

As suggested, all files are at default status, only molecule.yml is updated with connection parameters.

Thanks bitchecker. What did you need to change in create.yml to fix your issue?

Hi @meffie,
in create.yml I'm adding something like that to manage the VMs creation:
- name: Clone cloud-init template
  community.general.proxmox_kvm:
    api_user: root@pam
    api_password: xxxxxxxxx
    api_host: xxxxxxxxxx
    node: proxmox
    vmid: <template-id>
    newid: <new-id> # without other steps fails, some proxmox_kvm module bug?
    clone: Alma9-template
    name: molecule
    storage: local-zfs
    timeout: 90
    net:
      net0: 'virtio,bridge=vmbr0'

- name: Update VM configuration
  community.general.proxmox_kvm:
    api_user: root@pam
    api_password: xxxxxxxxx
    api_host: xxxxxxxxxx
    node: proxmox
    vmid: <new-id> # without other steps fails, some proxmox_kvm module bug?
    cores: 2
    memory: 4096
    ide:
      ide2: 'local-zfs:cloudinit,media=cdrom' # this is not working!
    ciuser: root
    cipassword: root
    nameservers: <IP_ADDRESS>
    ipconfig:
      ipconfig0: 'ip=<ip-address>/24,gw=<gateway-address>'
    tags: molecule
    update: true

- name: Start VM
  community.general.proxmox_kvm:
    api_user: root@pam
    api_password: xxxxxxxxx
    api_host: xxxxxxxxxx
    node: proxmox
    vmid: <new-id> # without other steps fails, some proxmox_kvm module bug?
    state: started

What is missing is that at the end of these steps, the vm should add into the "molecule" inventory, that is this (default code):
  - name: Create instance config
    when: server.changed | default(false) | bool  # noqa no-handler
    block:
      - name: Populate instance config dict  # noqa jinja
        ansible.builtin.set_fact:
          instance_conf_dict: {}
          # instance': "{{ }}",
          # address': "{{ }}",
          # user': "{{ }}",
          # port': "{{ }}",
          # 'identity_file': "{{ }}", }
        with_items: "{{ server.results }}"
        register: instance_config_dict

      - name: Convert instance config dict to a list
        ansible.builtin.set_fact:
          instance_conf: "{{ instance_config_dict.results | map(attribute='ansible_facts.instance_conf_dict') | list }}"

      - name: Dump instance config
        ansible.builtin.copy:
          content: |
            # Molecule managed

            {{ instance_conf | to_json | from_json | to_yaml }}
          dest: "{{ molecule_instance_config }}"
          mode: "0600"

And I'm still trying to understand how to attach new code with default one.
In destroy.yml instead, I added this:
- name: Stop VM
  community.general.proxmox_kvm:
    api_user: root@pam
    api_password: xxxxxxxx
    api_host: xxxxxxx
    node: proxmox
    vmid: 9000
    state: absent
    force: true

As I said in the "code comments" the Cloud-Init directives are failing and no logs are provided.

@meffie , @IamLunchbox any news on this?

Hi, I am sorry I was on vacation for the past weeks and had no time to check back into this issue.
If there is a problem with either molecule or the proxmox_kvm module, we'd need to know their respective version. Could you provide the output of molecule --version', the molecule-proxmox version (pip freezeideally) and the output ofansible-galaxy info community.general`? And what is your proxmox version?
I was not able to reproduce your problem yet and I don't have an immediate idea, why your state check fails.
In addition, I am suspicoius as to why you HAVE to assign a vmid for the new vm. This seems unusual.
Aug 24, 2024 17:32:10 bitchecker @.***>:

@meffie[https://github.com/meffie] , @IamLunchbox[https://github.com/IamLunchbox] any news on this?
—
Reply to this email directly, view it on GitHub[https://github.com/meffie/molecule-proxmox/issues/17#issuecomment-2308431878], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ANRA34L2PO6UHHHQ5ZRL3NLZTCRPRAVCNFSM6AAAAABME5GTZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBYGQZTCOBXHA].
You are receiving this because you were mentioned.
[Tracking image][https://github.com/notifications/beacon/ANRA34J5VNWPXQJ6NHVYZXDZTCRPRA5CNFSM6AAAAABME5GTZKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUJS7QAM.gif]

for molecule:
molecule==24.8.0
molecule-proxmox==1.0.0

for galaxy community.general should not be necessary because everything is already available, but:
community.general:9.3.0

proxmox: 8.2.4

I tried to reproduce your problem with a clean virtualenv, your specific versions and only the packages molecule-proxmox, molecule, ansible, requests. I was not able to reproduce your issue.
I suspect the main issue may be, that molecule does not seem to pick up the molecule-proxmox create.yml-playbook. The conditional check when: server.changed is not reached my execution.
But there was a similar issue in my testrun, which could be related. In one execution, Molecule somehow did not pickup the correct path for the molecule-proxmox library and did try to use and old locally installed molecule-version. This was only fixed after closing the shell and reactivating the virtualenv afterwards. But then it was working reliably.
My successful execution logged:
1 plays in /home/ubuntu/Repos/testrepo/venv/lib/python3.10/site-packages/molecule_proxmox/playbooks/create.yml.
My failed execution logged:
1 plays in /home/ubuntu/.local/lib/python3.10/site-packages/molecule_proxmox/playbooks/create.yml - and it failed because that version did not have the pool-parameter yet (molecule-proxmox 0.9).
If you not did so before, check with more verbosity (-vv) what paths molecule identified for the given create and destroy playbook and use a clean virtualenv for your tests.
Lastly, maybe try to remove all extra config items from your molecule config.Just to make sure a misplaced dict item is not throwing something off -  even though I don't expect this to be the case. When you use the maximum verbosity (-vvvv) you'll see the config items molecule created and used, e.g. to pass them on to proxmox_kvm - maybe this will give you a hint on whats going on.

Hi,
for what I can see, the create.yml is completely skipped (with default file):
PLAYBOOK: create.yml ***********************************************************
Positional arguments: /home/bitchecker/molecule/molecule/default/create.yml
verbosity: 4
connection: ssh
become_method: sudo
tags: ('all',)
skip_tags: ('notest', 'molecule-notest')
inventory: ('/dev/shm/bitchecker/molecule/molecule/default/inventory',)
forks: 50

PLAY [Create] ******************************************************************

TASK [Populate instance config dict] *******************************************
task path: /home/bitchecker/molecule/molecule/default/create.yml:13
skipping: [localhost] => {
    "changed": false,
    "false_condition": "server.changed | default(false) | bool",
    "skip_reason": "Conditional result was False"
}

TASK [Convert instance config dict to a list] **********************************
task path: /home/bitchecker/molecule/molecule/default/create.yml:24
skipping: [localhost] => {
    "changed": false,
    "false_condition": "server.changed | default(false) | bool",
    "skip_reason": "Conditional result was False"
}

TASK [Dump instance config] ****************************************************
task path: /home/bitchecker/molecule/molecule/default/create.yml:28
skipping: [localhost] => {
    "changed": false,
    "false_condition": "server.changed | default(false) | bool",
    "skip_reason": "Conditional result was False"
}

PLAY RECAP *********************************************************************
localhost                  : ok=0    changed=0    unreachable=0    failed=0    skipped=3    rescued=0    ignored=0

So, the main issue starts from here.
For converge.yml he tries to connect to the name that you give to the platform in molecule.yml so I can see in my logs:
sending connection check: [b'ssh', b'-vvvv', b'-C', b'-o', b'ControlMaster=auto', b'-o', b'ControlPersist=60s', b'-o', b'StrictHostKeyChecking=no', b'-o', b'KbdInteractiveAuthentication=no', b'-o', b'PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey', b'-o', b'PasswordAuthentication=no', b'-o', b'ConnectTimeout=10', b'-o', b'ControlPath="/home/bitchecker/.ansible/cp/%h-%p-%r"', b'-O', b'check', b'molecule']

so, if you're not in the same network of the proxmox server, and you haven't a DNS that can solve that name, you'll never reach that guest (that will never be reached because if you don't update the create.yml file, it will never be created)

Yes it is skipped - because server.changed is undefined. This variable is neither used nor set in molecule-proxmox.
Did you by chance copy the default create.yml from the molecule project into the molecule default scenario? The path you pasted implies, that the create.yml resides in that scenario.
 They are pulled from this project - you only need to set a converge.yml and a molecule.yml in the molecule/default directory.
Aug 25, 2024 17:56:56 bitchecker @.***>:

Hi,
for what I can see, the create.yml is completely skipped (with default file):
PLAYBOOK: create.yml ***********************************************************
Positional arguments: /home/bitchecker/molecule/molecule/default/create.yml
verbosity: 4
connection: ssh
become_method: sudo
tags: ('all',)
skip_tags: ('notest', 'molecule-notest')
inventory: ('/dev/shm/bitchecker/molecule/molecule/default/inventory',)
forks: 50
PLAY [Create] ******************************************************************
TASK [Populate instance config dict] *******************************************
task path: /home/bitchecker/molecule/molecule/default/create.yml:13
skipping: [localhost] => {
"changed": false,
"false_condition": "server.changed | default(false) | bool",
"skip_reason": "Conditional result was False"
}
TASK [Convert instance config dict to a list] **********************************
task path: /home/bitchecker/molecule/molecule/default/create.yml:24
skipping: [localhost] => {
"changed": false,
"false_condition": "server.changed | default(false) | bool",
"skip_reason": "Conditional result was False"
}
TASK [Dump instance config] ****************************************************
task path: /home/bitchecker/molecule/molecule/default/create.yml:28
skipping: [localhost] => {
"changed": false,
"false_condition": "server.changed | default(false) | bool",
"skip_reason": "Conditional result was False"
}
PLAY RECAP *********************************************************************
localhost                  : ok=0    changed=0    unreachable=0    failed=0    skipped=3    rescued=0    ignored=0
So, the main issue starts from here.
For converge.yml he tries to connect to the name that you give to the platform in molecule.yml so I can see in my logs:
sending connection check: [b'ssh', b'-vvvv', b'-C', b'-o', b'ControlMaster=auto', b'-o', b'ControlPersist=60s', b'-o', b'StrictHostKeyChecking=no', b'-o', b'KbdInteractiveAuthentication=no', b'-o', b'PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey', b'-o', b'PasswordAuthentication=no', b'-o', b'ConnectTimeout=10', b'-o', b'ControlPath="/home/bitchecker/.ansible/cp/%h-%p-%r"', b'-O', b'check', b'molecule']
so, if you're not in the same network of the proxmox server, and you haven't a DNS that can solve that name, you'll never reach that guest (that will never be reached because if you don't update the create.yml file, it will never be created)
—
Reply to this email directly, view it on GitHub[https://github.com/meffie/molecule-proxmox/issues/17#issuecomment-2308905044], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ANRA34P336G2GLCBZ7DSLYTZTH5ELAVCNFSM6AAAAABME5GTZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBYHEYDKMBUGQ].
You are receiving this because you were mentioned.
[Tracking image][https://github.com/notifications/beacon/ANRA34IXLBGNXIUSMY63TQ3ZTH5ELA5CNFSM6AAAAABME5GTZKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUJT4MFI.gif]

When I created the env, I created a new scenario, and I'm using the default create.yml and others. The only files that I changed were converge.yml and molecule.yml.
In on previous test, I also completely deleted all files and kept only those two.
I still think that if you're not on the same PVE network that uses default ports and with guests on the bridge in the same network, all these stuff will never work...so also if your PVE is hosted (with natted guests) you'll be never able to use this.

In on previous test, I also completely deleted all files and kept only those two.

Good to know! I will test if molecule-docker works with the default files generated by molecule. If it does, this will probably stop this project being ever merged into the molecule-plugins repo and we should check, what keeps the default config from being run with molecule-proxmox.

I still think that if you're not on the same PVE network that uses default ports and with guests on the bridge in the same network, all these stuff will never work...

This is incorrect. You don't need to use the same network, if you use this plugin. I always work through VPN without hostname resolution. Molecule saves the ip associated IP-addresses, not only the hostname, into its cache file:
ubuntu@adm: cat /home/ubuntu/.cache/molecule/test/default/inventory/ansible_inventory.yml
# Molecule managed

---
all:
  hosts:
    test: &id001
      ansible_host: <IP_ADDRESS>
      ansible_port: 22
      ansible_private_key_file: redacted
      ansible_ssh_common_args: -o UserKnownHostsFile=/dev/null -o ControlMaster=auto
        -o ControlPersist=60s -o ForwardX11=no -o LogLevel=ERROR -o IdentitiesOnly=yes
        -o StrictHostKeyChecking=no
      ansible_user: test
      connection: ssh
  vars:
    molecule_ephemeral_directory: '{{ lookup(''env'', ''MOLECULE_EPHEMERAL_DIRECTORY'')
      }}'
    molecule_file: '{{ lookup(''env'', ''MOLECULE_FILE'') }}'
    molecule_instance_config: '{{ lookup(''env'', ''MOLECULE_INSTANCE_CONFIG'') }}'
    molecule_no_log: '{{ lookup(''env'', ''MOLECULE_NO_LOG'') or not molecule_yml.provisioner.log|default(False)
      | bool }}'
    molecule_scenario_directory: '{{ lookup(''env'', ''MOLECULE_SCENARIO_DIRECTORY'')
      }}'
    molecule_yml: '{{ lookup(''file'', molecule_file) | from_yaml }}'
ungrouped:
  hosts:
    test: *id001
  vars: {}

so also if your PVE is hosted (with natted guests) you'll be never able to use this.
I think you can safely assume that nobody tested the case you outlined yet.

But if you think using a different port keeps you from doing so with molecule-proxmox 1.0: This is not the case. Currently, use ansible_port to use the given port for ALL platforms of that scenario.
name: ansible
  connection_options:
    ansible_port: 23

And because you said never: If you want to have an alternative port per platform, please create a PR. I am sure michael is open for improvements, especially when the change is within the spec of the proxmox_kvm module.

I rechecked the behaviour with molecule init scenario test and the podman driver. It works if I just provide a molecule.yml and a converge.yml.
After initiating the scenario, which leads to three default files being created (create, prepare, destroy) the same behavior occurs as described by you.
Therefore, I assume that the skipping of instance creation when using the default create.yml does not seem to be a molecule-proxmox specific error.

@bitchecker could you fix this problem and/or is it ok to close this issue?

Hi,
according to my point of view avoiding configuring the create file should be the default as the tool should directly take care of creating guests on proxmox in order to run tests on the Ansible code, as I showed in the previous comments, it does not do this by default unless major changes are made by the user.
If this is the expected behavior, you can proceed to close the issue.

Sorry for the delay. I was traveling when you replied back in August and missed that reply.  I'll take a closer look at the info you provided.

When I created the env, I created a new scenario, and I'm using the default create.yml and others. The only files that I changed were converge.yml and molecule.yml.

@bitchecker can you confirm your issue, if you only provide converge.yml and molecule.yml in your scenario? Please delete all other files from the scenario directory.

Hi,
yes, I'm using only converge and molecule but I would like to clarify a missing point:
Is this software designed to test Ansible code interacting with proxmox and automatically provision guests, or I'm wrong?
If I'm right, this issue is just reporting that guests are not provisioned, and I provided all the requested logs. If I'm going to delete create and destroy, I'm telling that creation and deletion of guests are not managed by this tool... Nothing else.

Well, id phrase it differently:
Molecule tests ansible roles. To do so, it needs a system to configure. The system is provided by a driver (docker, azure, podman) and each single system is a plattform.
Molecule proxmox is a driver, which provides you an environment to test a role. To do so, it reaches out to proxmox, clones a vm and connects to it via ssh.
So no, this code is not designed to make roles interact with proxmox, but rather with ephemeral vms.
Sep 22, 2024 15:26:46 bitchecker @.***>:

Hi,
yes, I'm using only converge and molecule but I would like to clarify a missing point:
Is this software designed to test Ansible code interacting with proxmox and automatically provision guests, or I'm wrong?
If I'm right, this issue is just reporting that guests are not provisioned, and I provided all the requested logs. If I'm going to delete create and destroy, I'm telling that creation and deletion of guests are not managed by this tool... Nothing else.
—
Reply to this email directly, view it on GitHub[https://github.com/meffie/molecule-proxmox/issues/17#issuecomment-2366788190], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ANRA34P2CBIMMJ4FWU7YLYTZX3ARJAVCNFSM6AAAAABME5GTZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRWG44DQMJZGA].
You are receiving this because you were mentioned.
[Tracking image][https://github.com/notifications/beacon/ANRA34KMTNGDXT6TRI4LPPDZX3ARJA5CNFSM6AAAAABME5GTZKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUNCJJF4.gif]

So no, this code is not designed to make roles interact with proxmox, but rather with ephemeral vms.

Sorry, when I said this?

Molecule tests ansible roles. To do so, it needs a system to configure. The system is provided by a driver (docker, azure, podman) and each single system is a plattform.
Molecule proxmox is a driver, which provides you an environment to test a role. To do so, it reaches out to proxmox, clones a vm and connects to it via ssh.

It's exactly what I've said in my previous comment:

Is this software designed to test Ansible code interacting with proxmox and automatically provision guests, or I'm wrong?

And you're just confirming that this driver should provide the environment, provisioning guests on proxmox. I'm telling that this not happens.

Is this software designed to test Ansible code interacting with proxmox and [...]

It may be a translation error, but this is what I was referring to.
Anyways, all the logs you provided above (1 and 2) reference the default create step from molecule scenario init - not the create step from molecule-proxmox.
But I don't have an idea, why these steps happen in your instance.

I'll close this due to inactivity.

Sorry, I've been inactive on this one. Looks like there is more to do here.


Update build

Hi,

I added arm64 architecture to simulators.
Enabled bitcode
Updated build

To make it work, I pointed to my fork of ios_system release https://github.com/yury/ios_system/releases
After merge, please update url in checksum in xcfs/Package.swift

Both before and after editing xcfs/Package.swift, I get:
error: artifact of binary target 'ios_system' has changed checksum; this is a potential security risk so the new artifact won't be downloaded

Both before and after editing xcfs/Package.swift, I get:
error: artifact of binary target 'ios_system' has changed checksum; this is a potential security risk so the new artifact won't be downloaded

Yes, but now you can use work around.

Comment binaryTarget that you want to update
run swift package update
Uncomment and update checksum
run swift package update

This allows you to update only packages you want to update. Not all packages.

Yes, but now you can use work around.

Comment binaryTarget that you want to update
run swift package update
Uncomment and update checksum
run swift package update

This allows you to update only packages you want to update. Not all packages.

I must have missed something, because I did all this and I still get:
error: artifact of binary target 'ios_system' has changed checksum; this is a potential security risk so the new artifact won't be downloaded

My xcfs/Package.swift refers to the newly built ios_system:
        .binaryTarget(
            name: "ios_system",
            url: "https://github.com/holzschu/ios_system/releases/download/v2.7.0/ios_system.xcframework.zip",
            checksum: "83545ba08c01d0ea813fa48133efd64f734a0c3900e8e123188e9e9d2907ca90"
        ),

I must be missing something, but I don't know what it is.

I must have missed something, because I did all this and I still get:
error: artifact of binary target 'ios_system' has changed checksum; this is a potential security risk so the new artifact won't be downloaded

My xcfs/Package.swift refers to the newly built ios_system:
        .binaryTarget(
            name: "ios_system",
            url: "https://github.com/holzschu/ios_system/releases/download/v2.7.0/ios_system.xcframework.zip",
            checksum: "83545ba08c01d0ea813fa48133efd64f734a0c3900e8e123188e9e9d2907ca90"
        ),

I must be missing something, but I don't know what it is.

After a combination of swift package reset, resolve, update and clean, the framework is now built (I don't know which one made the trick).

After a combination of swift package reset, resolve, update and clean, the framework is now built (I don't know which one made the trick).


Latest phar file is missing the new FCGI vendor

We use cachetool in our deploy scripts and since 1.8.0 we have issues with the phar file:
curl -sO http://gordalina.github.io/cachetool/downloads/cachetool.phar
php cachetool.phar apc:cache:clear all --fcgi=/tmp/.xxx-fpm.sock
PHP Fatal error:  Class 'Adoy\FastCGI\Client' not found in phar:///home/kumaci/cachetool.phar/src/CacheTool/Adapter/FastCGI.php on line 45

Seems like the new Adoy classes are not included in the phar file

Fixed on 1.8.1


Fix/check estimator

scikit-learn-contrib (our PR is https://github.com/scikit-learn-contrib/scikit-learn-contrib/issues/62) requires that all estimators pass a check_estimators test ala https://scikit-learn.org/stable/modules/generated/sklearn.utils.estimator_checks.parametrize_with_checks.html#sklearn.utils.estimator_checks.parametrize_with_checks, which ours were not currently doing. I've been going through everything that inherits from the BaseEstimator class (or should) and making the required changes. @agoscinski would appreciate your input on the linear_models section, as I'm not sure that OrthogonalRegression can pass the estimators, as to my knowledge the predicted values may have a different shape that the fitted y, no? Please correct me if I'm wrong.

:books: Documentation preview :books:: https://scikit-matter--196.org.readthedocs.build/en/196/

@agoscinski I am honestly perplexed by some of the testing errors coming up right now (matmul failing for correctly-shaped matrices). Would appreciate thoughts -- @ceriottm @PicoCentauri @Luthaf and others also welcome to weigh in
Running tests/test_metrics.py, I get things like:
ERROR: test_local_reconstruction_error_train_idx (__main__.ReconstructionMeasuresTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/rca/source_installs/scikit-matter/tests/test_metrics.py", line 141, in test_local_reconstruction_error_train_idx
    lfre_val = pointwise_local_reconstruction_error(
  File "/Users/rca/miniconda3/lib/python3.10/site-packages/skmatter/metrics/_reconstruction_measures.py", line 456, in pointwise_local_reconstruction_error
    - 2 * X_test @ X_train.T
  File "/Users/rca/miniconda3/lib/python3.10/site-packages/numpy/ma/core.py", line 3077, in __array_wrap__
    m = reduce(mask_or, [getmaskarray(arg) for arg in input_args])
  File "/Users/rca/miniconda3/lib/python3.10/site-packages/numpy/ma/core.py", line 1757, in mask_or
    return make_mask(umath.logical_or(m1, m2), copy=copy, shrink=shrink)
ValueError: operands could not be broadcast together with shapes (15,4) (4,5)

I've checked, it's within the @ operator, not the *. Also checked that both matrices are the same type.
For the record, this only happens for tests/test_metrics.py and tests/test_sparse_kernel_centerer.py on my end.

In the StandartFlexibleScaler np.ma.* is used which transforms the mean to  a <class 'numpy.ma.core.MaskedArray'> and thus also the transformed array.c
https://github.com/lab-cosmo/scikit-matter/blob/766e3dabc42f26727b484b37fcd696ad64a9c222/src/skmatter/preprocessing/_data.py#L152
Then in the metrics code the @ operation between masks is executed as an elementwise multiplication which results in the error because of not broadcastability of the shapes
Just replace np.ma.* with regular np.*

Unfortunately, np.average flags an error with scikit learn (scikit learn
does not like the name average, even if we rename upon import). Unless
there's another function we can use, we might need to make a spoof
On Thu, May 18, 2023, 07:09 Alexander Goscinski @.***>
wrote:

In the StandartFlexibleScaler np.ma.* is used which transforms the mean
to a <class 'numpy.ma.core.MaskedArray'> and thus also the transformed
array.c
https://github.com/lab-cosmo/scikit-matter/blob/766e3dabc42f26727b484b37fcd696ad64a9c222/src/skmatter/preprocessing/_data.py#L152
Then in the metrics code the @ operation between masks is executed as an
elementwise multiplication which results in the error because of not
broadcastability of the shapes
Just replace np.ma.* with regular np.*
—
Reply to this email directly, view it on GitHub
https://github.com/lab-cosmo/scikit-matter/pull/196#issuecomment-1552961173,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALKVP3TLPJJBZPYP2QLHIZDXGYGPBANCNFSM6AAAAAAYFTRLLE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I dont fully understand the problem with np.average, could you elaborate. Where does scikit-learn return an error?

Everything passes! Ready for a proper review with one or two outstanding q's:
[ ] technically, the sample selectors inherit from BaseEstimator, but do not follow the estimator behavior to a tee (their transformed objects do not have the same number of rows as their inputs). In order to delay the selector-refactor conversation, I suggest we leave them out of test_check_estimators
[ ] Should orthogonalregression inherit from base estimator? @agoscinski

I should note what many of the necessary fixes were:

Only assigning values to variable_ type attributes outside of the initializer
Making sure input arrays are always validated
Making sure errors match the expected behavior for scikit-learn (the weird additions of Reshape your data)
Making sure to always define n_features_in_ and n_samples_in during fit functions

I have a question here, shouldn't we explicitly add tests with @parametrize_with_checks in our test sets? Or the requirement that it should just be possible to run such tests?

technically, the sample selectors inherit from BaseEstimator, but do not follow the estimator behavior to a tee (their transformed objects do not have the same number of rows as their inputs). In order to delay the selector-refactor conversation, I suggest we leave them out of test_check_estimators

If they are okay with it, sure. We can write in the doc also that it is not compatible with Pipeline yet, if this helps convincing them. In the end we need to mark this somehow programmatically. I am not sure how to do this, since scikit-learn Pipeline never checks for the type of the transformers, so changing the base class does not help here. Maybe the contributors have some useful suggestions.

Should orthogonalregression inherit from base estimator? @agoscinski

Yes it is an estimator so it should inherit from it, what is the problem with it? That it does not always agrees with the shape of the input, right? We can make mark it as private class (_OrthogonalRegression), because it is only use for reconstruction measures, then I think we can ommit it for the estimator check. But it basically has the same problems as the sample selection classes, so I would do the apply the same solution.

@agoscinski and @hurricane642 can you do another formal review?


Make all cred generators sync.

This was causing weird segfault issues, no idea why any of them were this way (rather, why we didn't override them when we noticed they were async). Also added an example of 2factor auth.
https://github.com/nodegit/nodegit/pull/375
https://github.com/nodegit/nodegit/issues/323
https://github.com/nodegit/nodegit/issues/321

Is this good to merge?

Whoops. Yeah, in it goes.


Feathering

Does turning off the feathering turn it off for the entire GUI or just the plot ?

Hi @Resonanz , the feathering is turned off at the egui::Context level. Each Context ctx refers to the same mutable data (interior mutability).
So yes, the following code turnes feathering off for the entire GUI:
ctx.tessellation_options_mut(|tess_options| {
            tess_options.feathering = false;
        });

Containers, such as egui::containers::Window do not have separate tessalation options. A egui::Ui (where you place widgets on) has a reference to the parent ctx Context and no separate tessalation options.
There is a tesselate method in egui::Context, so maybe you can tesselate certain Shapes even though the tesselation is turned off globally.


Test binder InputDestination/OutputDestination only catch one message despite batch-mode true

Versions
Spring boot: 2.6.15
Spring cloud stream: 3.2.9
Description
InputDestination / OutputDestination test binders do not appear to work with batch-mode: true. Function stream only receives a single message instead of the list of batched messages.
Configuration
spring.cloud.function.definition: batchStream;
spring.cloud.stream.bindings.batchStream-in-0.consumer.batch-mode: true
spring.cloud.stream.kafka.binder.consumer-properties:
  max.poll.records: 10
  fetch.min.bytes: 1000
  fetch.max.wait.ms: 500

Test
fun `batchProcessor will accept multiple input messages and join them`() {
    input.send(MessageBuilder.withPayload("hello").build(), "test.batch.in")
    input.send(MessageBuilder.withPayload("world").build(), "test.batch.in")
    assertEquals("hello world", output.receive(10, "test.batch.out").payload.toString(Charsets.UTF_8))
}

Function
@Service
class BatchProcessor {
    @Bean
    fun batchStream(): Function<Message<List<String>>, Message<String>> =
        Function { message ->
            // This line is executed twice, with the `message` being a `String` - "hello" and then "world".
            // Should be executed once, with the `message` being a `List` of ["hello", "world"].
            MessageBuilder.withPayload(message.payload.joinToString(separator = " ")).build()
        }
}

Note that when I invoke the same functionality using kcat, it works as expected:
$ echo 'hello' | kcat -b localhost:9092 -t test.batch.in -P ; echo 'world' | kcat -b localhost:9092 -t test.batch.in -P
$ kcat -b localhost:9092 -t test.batch.out
> hello world

You are using an old and no longer supported version of s-c-stream/function (https://spring.io/projects/spring-cloud-stream#support), so please upgrade and file a new issue if you still see a problem


Suggestion: Add sponsors to READMEs on eslint.org and code-explorer

What problem do you want to solve?
On the main ESLint README, rewrite README, markdown README and js README repos we have sponsors listed but haven't listed in code-explorer and eslint.org repo.
What do you think is the correct solution?
Add the sponsor's info in code-explorer and eslint.org as well.
Participation

[x] I am willing to submit a pull request for this change.

Additional comments
If we have intentionally chosen not to include sponsor information in eslint.org and code-explorer, please feel free to close this issue.

Sounds good to me.

Completed via https://github.com/eslint/eslint.org/pull/654 https://github.com/eslint/code-explorer/pull/69


Differents results between Snapchat AUTO and ParseSnapchat IOS

Hi
When I try the two tools "Snapchat AUTO" and "ParseSnapchat IOS" on the same Snapchat folder I"ve got different result.
The "ParseSnapchat IOS" tool, find more dialogues than the "Snapchat_Auto".
Do you know why ?
Thank you

Hi! ParseSnapchat IOS should no longer present more chats than Snapchat Auto.
There was a known issue that was fixed in the latest release.
If you still see this issue please re-open the issue and provide some examples so I can try to replicate it.


add TW language

Ok
n76<EMAIL_ADDRESS>於 2020年2月4日 週二 上午1:27 寫道：

@n76 requested changes on this pull request.
There are so many untranslated strings that this seems premature. Could
you re-submit with more of the strings translated?
Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/n76/Local-GSM-Backend/pull/129?email_source=notifications&email_token=ANDZ3APQE36GO7RA6GDZJCLRBBHZRA5CNFSM4KPAPW3KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCUA2IPI#pullrequestreview-352429117,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDZ3ALU7SYBWA5WOE2RVW3RBBHZRANCNFSM4KPAPW3A
.


specify service annotation protocol

What is the purpose of the change
For Issue #5049
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.vghy顾海洋 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Drrefactor/root focused view leak

Summary
Please see issue with detailed description:
https://github.com/react-native-tvos/react-native-tvos/issues/291

Changelog
Fix potential focus race related to detached focused view's reference being held in RCTRootView.

[Internal] [Fixed] - Prevent using detached RCTView as preferredFocusEnvironment
Test Plan

Thanks for the PR! I'll test this out. If we are going to do this, we would need a similar change in the view class for the new Fabric architecture.

I applied second commit, regarding detached views, to Fabric RCTRootComponentView.
I see that you already use weak reference in RCTRootComponentView.h -> reactPreferredFocusedView, so this one is already done.


Update lint rules to fix example failures on generated file

Fixes linting failures on generated code in example/lib by excluding the depend_on_referenced_packages option
e.g. https://github.com/MindscapeHQ/raygun4flutter/runs/3993980593?check_suite_focus=true

LGTM!
The CI is failing for unrelated issues, I will have to look into that.


Wrong wording of missing CPU instructions

Have a look at the code at https://github.com/tensorflow/tensorflow/blob/3d3c6db1ca2d50f6f07722cd800144f8f736167c/tensorflow/core/platform/cpu_feature_guard.cc#L150-L157
This outputs e.g.

This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.

Judging from the naming missing_instructions I suppose there is a "not" missing in that warning.

This is correct wording. E.g. if CPU supports AVX2 but tensorflow is built without AVX2 support, AVX2 is in missing_instructions. Close this.

So a TensorFlow build without avx2 enabled which builds oneDNN alongside its build doesn't use avx2, but the oneDNN part does use it?
Intresting...
Then maybe the wording can be improved?

@Flamefire,
As the wording is correct already, Can you suggest how the wording can be improved ?

"The TensorFlow binaries were not build with the following CPU instruction sets: <...> To enable them rebuild ...\n Note that oneDNN is used for various performance-critical operations and may dynamically use those CPU instructions sets nonetheless."
Or so... And maybe make sure that oneDNN is actually used/built because I don't see any switch checking if oneDNN support is enabled around that message.

@Flamefire,
Can you submit a PR for updating this file with the change you want to propose? Thanks!


Cleanup transaction schema

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What

Remove kycVerified
Update transferReceivedAt only for SEP-31
Remove custodialTransactionId

Why

Not used
Was only used by SEP-31
Not used

Known limitations
N/A

Reference Server Preview is available here:https://anchor-ref-pr847.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr847.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr847.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr847.previews.kube001.services.stellar-ops.com/

Was the V6 migration file added for the 2.0 release or did it exist for 1.x releases? If it existed in 1.x releases we should make a new migration file for removing the column instead of deleting the add statement.

It's for v2

Reference Server Preview is available here:https://anchor-ref-pr847.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr847.previews.kube001.services.stellar-ops.com/


Adding 3 new Yamls and also splitting existing Color Cluster Yaml to match the cert test cases

Adding 3 new Yamls and also splitting existing Color Cluster Yamls to match the cert test cases
Test_TC_TM_2_1,Test_TC_TSUIC_2_1,Test_TC_PCC_2_1
Ran the tests locally and all are passing

@saurabhst @andy31415 @Damian-Nordic ?


Add support for brokers separating external and internal traffic (introduced in KIP-103)

Fixes point 1 of Issue #859
Checks if the listener.security.protocol.map property is present in the broker metadata. If it is: it will apply the mapping from listener name to security protocol used before checking whether a listener uses the protocol desired by the kafka-node client.
For point 2 it currently still requires the endpoint to be used by kafka-node to be listed first.

Thanks for your contribution!


f.close should be f.close()

Currently, on this line:
https://github.com/marcmengel/jobsub_lite/blob/c26e3631f7412a6876bc9bc6b99539d51b1428ad/lib/creds.py#L29
there's a call to f.close.  This should be f.close()

I have changed this in my modifications for Issue 12.

On Mar 17, 2022, at 2:34 PM, Shreyas Bhat @.***> wrote:
Currently, on this line:
https://github.com/marcmengel/jobsub_lite/blob/c26e3631f7412a6876bc9bc6b99539d51b1428ad/lib/creds.py#L29 https://github.com/marcmengel/jobsub_lite/blob/c26e3631f7412a6876bc9bc6b99539d51b1428ad/lib/creds.py#L29
there's a call to f.close. This should be f.close()
—
Reply to this email directly, view it on GitHub https://github.com/marcmengel/jobsub_lite/issues/20, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE44QBTWJYBOBTIRYING5LDVAOCMFANCNFSM5Q72L4JA.
You are receiving this because you are subscribed to this thread.

@goodenou This is now fixed - please note that you'll need to rebase before opening a PR for #12.


Jesse: Watch unzipgit.github.io

Jesse requests the watch of the watch_keyword unzipgit\.github\.io. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

unzipgit\.github\.io has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


update copilot-version

btw, new versions don't seem to use bin/copilot-node-server anymore.

Yeah, you can now use M-x copilot-install-server to install the server.

I mean that this version is outdated:
https://github.com/copilot-emacs/copilot.el/blob/733bff26450255e092c10873580e9abfed8a81b8/copilot.el#L147-L148
but the latest copilot-node-server version doesn't have the bin/copilot-node-server file anymore, so copilot.el would need to have some changes.

The command line has been changed to node copilot/dist/agent.js --stdio based on their README page.

Can you try #305? 🤔


Used Gael's code to add to ChocolateyPackages the ability to set Feat…

…ures

This change is

Thanks, I am unable to test it but by just looking at the code this all makes sense.

Thanks, I am unable to test it but by just looking at the code this all makes sense.


Replace quay.io with the new registry in the cluster-cloud-director chart

Task
Replace quay.io with the new registry in the cluster-cloud-director chart.
Links

Epic;
State after initial investigation on CAPA.

done


Missing React.StrictMode introduced in React 16.3

Strict mode introduction.
https://reactjs.org/docs/strict-mode.html
Here's the proof.  This could be a React issue or a Flow issue, but since the error is occurring in Flow, I'm opening it here.
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVBLAtgBzgJwBcwAlAUwEMBjYqfOLMAcn0pqfSrgDsBnYgEYVeGKuWqEACvRxgAvGAAUOGbwCU8gHxLUASAA84mgDoAyoXyjCAWTgATMpt2owYfbzI0MPTS7C7nVwNgDy8fP1d9YCNCMwsrWwdfNSA

There seems to already be a PR open for this: https://github.com/facebook/flow/pull/6078


Added basic functionality for syncing ElastiCache

/ptal @achantavy

Are there any new indices that have to be added?

Update

schema docs: https://github.com/lyft/cartography/blob/master/docs/schema/aws.md
add an index for the id field of your node and any others that you expect to be queried frequently (if any): https://github.com/lyft/cartography/blob/69eda2ed0df9003b6d966547912e27a2a6935015/cartography/data/indexes.cypher


use navigation.navigate inside useffect

Hi, I was trying to redirect to another view when a state changes with useEffect(), but when navigation.navigate('/something') is triggered I got a react warning:
Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in a useEffect cleanup function.

is there something I am missing? maybe call an unsubscribe method for navigation?
here is the code:
import React, { useEffect } from 'react';
import injectSheet from 'react-jss';
import { useNavigation } from 'react-navi';

// Local Modules
// import Modal from '../../components/modal';
import Styles from './styles';

// Stores
import authStore from '../../stores/auth';

// Icons
import logo from './icons/logo.svg';

function Login ({ classes }) {
  const [authState, authActions] = authStore();
  const navigation = useNavigation();
  const { authenticated } = authState;
  useEffect(() => {
    if (authenticated) {
      navigation.navigate('/')
    }

    return () => {
      console.log('unmount');
    }
  }, [authenticated, navigation]);

  return (

Iniciar sesión

          Usuario
          <input {...{
            type     : 'text',
            value    : authState.credentials.email,
            onChange : (event) => authActions.updateField('email', event.target.value),
          }} />

          Contraseña
          <input {...{
            type     : 'password',
            value    : authState.credentials.password,
            onChange : (event) => authActions.updateField('password', event.target.value),
          }} />

Recuperar contraseña

Crear cuenta

 authActions.authenticate()}>
              Iniciar sesión

  )
}

export default injectSheet(Styles)(Login);

thanks in advance!

The issue is that you're thinking of login like a classic client/server webapp where you do an action and then redirect to the logged in page.
You need to think in a different way for single page apps. You have a render loop which is controlled by the router at the top level and you want the screen to render a view based on a state. In this case the state is whether or not the user is logged in or not.
In this case, you perform an action, login, and then the success of that action changes the state. that the user is logged in. This should cause a render loop update and display the new logged in view. No 'redirect' is necessary.
Follow the guide here: https://frontarm.com/navi/en/guides/authenticated-routes/

Another way to think about this separation of concerns. The Login component shouldn't know anything about navigation. All it should do is render the form. The form has a button on it, which when clicked, executes a function that is passed in. The component which is displayed after login is not the login components concern.

thank you!


EventHub support in scripting.

This needs an updated WebJobs.ServiceBus nuget package.
Script.Host.CsProj assembly references are broken while waiting for a nuget package publish.
I'll fix that before committing the pull request.

Also, please add a sample script (like all the other bindings) to the samples dir, as well as at least one E2E test so we have at least one regression test running.

@MikeStall this is all in, right? PR can be closed?

This has been merged


Purge does not remove row from 'docs' table

When a document is purged, its entry is not removed from the SQLite docs table. In fact nothing removes rows from this table, so over time, if lots of docs with UUIDs are created and purged, the table will grow monotonically. (See internal bug report). My commentary from there:

I think I remember what led to this. For speed, every CBLDatabase instance keeps an in-memory cache of some of the mappings from the docs table. That means that removing a row from the table would require some nasty multithreaded cache-invalidation. Instead I punted on that, figuring that the table is small (each row is just the docID string and an integer primary key) so it would take a long time to affect performance, and this was the kind of performance problem that we could hold off on fixing until/unless someone actually hit it in real life. Sounds like that's happened :-/ ...

This affects all three of our platforms. But it will not affect LiteCore or CBL 2.0 as the schema is entirely different.

https://github.com/couchbase/couchbase-lite-net/issues/827

https://github.com/couchbase/couchbase-lite-net/commit/2ec92049049c455d02cff6de1695952b3b142788#diff-1722e90c97a4f64f1d99393330b84705


[8.x] Add inspiring quote by Marie Curie

Adds an inspiring quote by Marie Curie to the Artisan command.

“Nothing in life is to be feared; it is only to be understood. Now is the time to understand more, so that we may fear less.”
- Marie Curie

Thanks for contributing to Laravel! ❤


extra utils portal not working and a fix to it

so i went searching to find out why the extra utils portals arent working and it due to the dimension id's of galicticraft overlapping with the extra utils id's so all that is needed is to change the id's ( i dont know how hard that will be but its the only real option)

Even on the official server right now?

yes ... ive tried going into them in survival and creative in a single player and last time i had tried on the official server it didnt work either


Incompatible with the Universal Windows Platform

Developing a Windows 10 Phone App, using the Universal Windows Platform and installing with NuGet results in this error:
"Install-Package : Analytics 2.0.2 is not compatible with UAP,Version=v10.0."

I aslo want use Segment in a Windows 10 application, and have the same issue.

I just successfully added https://segment.com/docs/libraries/xamarin/ to my UAP app. Should we use Xamarin library instead of this one?

yes, this library is not meant for xamarin's universal app platform.

We were talking about Windows 10 Universal Apps, not Xamarin apps
On Tue, Jan 12, 2016 at 22:05 Prateek Srivastava<EMAIL_ADDRESS>wrote:

yes, this library is not meant for xamarin's universal app platform.
—
Reply to this email directly or view it on GitHub
https://github.com/segmentio/Analytics.NET/issues/31#issuecomment-171118830
.

@f2prateek I am not developing a Xamarin app, but a Windows 10 Universal App. Wich library should I use?


bug: Getting knocked out sets Hp to 46

Knockout by bat, fists, or cars leaves players with a random amount of hp below 50. This is intentional; getting beat up hurts!


Precise selection of balancing Utxos

We used to have a mechanism to precisely select utxos eligible for balancing.
It was merged here: #355
It was removed as part as the commit https://github.com/tweag/cooked-validators/commit/931d1b4353e4c8b740b8cc6b74b19f3d14a1b15a, in file Balancing.hs line 249.
It was removed under the assumption that it was not actually used anywhere in the code. That assumption might have been wrong. We need to investigate it and restore this functionality if needed.

Will be closed by #417


Remove Python version requirement for pre-commit

Description
Fixes #1642

Update release notes, will merge afterwards


Google Groups: unrendering loses original MD

As reported here: https://groups.google.com/d/msg/markdown-here/SnQ4fVtQvQQ/MbzhEOCPKiMJ
Rendering works, but unrendering goes to plaintext instead of original Markdown. What's probably happening is that the wrapper is not being found, so the rendered text is actually being rendered again.

This doesn't seem to still be a problem. Closing.


Sample to show how to create a snapshot.

Fixes #31

Hey @michaelrice. So, I'm seeing this issue which I suppose @landscape-bot  has already commented on. I'll take a look at this in the next few days as I have time.

I fixed the bot issue of unused import. The only thing left is not in my
sample. Its Travis failing on the pep8 step from some of the other samples.
On Apr 17, 2015 9:16 AM, "Shawn Hartsock"<EMAIL_ADDRESS>wrote:

Hey @michaelrice https://github.com/michaelrice. So, I'm seeing this
issue
https://travis-ci.org/vmware/pyvmomi-community-samples/builds/58836230
which I suppose @landscape-bot https://github.com/landscape-bot has
already commented on. I'll take a look at this in the next few days as I
have time.
—
Reply to this email directly or view it on GitHub
https://github.com/vmware/pyvmomi-community-samples/pull/160#issuecomment-93994470
.

@michaelrice hopefully someone can put in some time and fix those other pep8 problems? I'm swamped until Wednesday.

@hartsock I added the pep8 cleanup to #164 if you wan to merge it first Ill rebase this afterwords.

@hartsock I got this rebased but travis failed to activate the virtualenv somehow..

I confirmed with travis-ci support this issue is being caused by their infrastructure problem. They are getting ready to release a status update about it.

@michaelrice thanks for keeping us in the loop on this!

@hartsock there is no way to re-trigger the job unless I push more.. so its kind of stuck like this. The only thing wrong before were pep8 issues with other samples which I fixed in #164 I could close this and reopen it or you could just merge it :dancers:

I found a control to let me retrigger the CI loop... we should see new results here.


Master branch is out of date

I'm not 100% sure on this, but it's difficult to tell which branch is the proper one to use; Branch appears to be out of date with 2.0.2.

My bad, it's ahead of the game. I'll go slink off into the corner quietly.


Do we need MIN_VALIDATOR_REGISTRY_CHANGE_INTERVAL?

Originally MIN_VALIDATOR_REGISTRY_CHANGE_INTERVAL was added to prevent the validator registry from changing literally every slot when we were doing cycles. At this point, however, I'm not as sure that it's necessary, and I don't think it would break anything to remove it, and instead just have a simple condition last_finalized_block > last_validator_set_change_slot.

In phases 0 and 1 we don't allow for re-registrations so I guess we're safe :)
For phase 2 I suggest either adding a withdrawal fee or an equivalent of MIN_VALIDATOR_REGISTRY_CHANGE_INTERVAL, otherwise polluting validator_registry with junk will be close to free.

Turned into a PR https://github.com/ethereum/eth2.0-specs/pull/340

closing in favor of #340.


[BUG] Trying to install v3.58.0 in ES after automated update prompt results in HTTP 500

Issue Description
Logging into an ES instance that does not have v3.58.0 installed yet will ask the user to update ESCU.  When trying to do so, an HTTP 500 error will occur and prevent installation.

Installation can still be done manually with success on Splunk Enterprise (install app from file -> upgrade) but not on Splunk Cloud.  (using Find more apps -> Update also yields a HTTP 500 error).
I've reproduced this on 2 separate instances.

Spoke to @ccl0utier offline, this is an existing known-issue w/ Splunkbase.


JupyterLab smoke test

Run a few of the notebooks in Jupyterlab (esp. stuff like amss, orth_proj, linear_algebra where we changed a lot of the math) and make sure that things are solid.

I just wanted to make sure that you did a few of these with the latest version.  If so, I think we should close.

Thanks for bumping. Still see the occasional math issue (seems stochastic) in the Jupyter downloads and source builds. For ex., the image below built from source, or a \col in the orth_proj download (near "the following result uses the preceding algorithm...") HTML looks fine in both cases.

If it is stochatic, it could just be bugs in jupyterlab and mathjax.  If the code works, I think we are in business.

Perfect, never saw any code issues. So will close now then.


Index defines itself by time, different points per day, index_today's time  Select an Index input box, can you write a time function?

System Details
Versions

Type
Version

Kibana
6.5.3

Elasticsearc
6.5.3

Original Install Method

[ ] Release pkg link
[ ] Gulp
[ ] Source
Other:

Bug Details
Bug Description
Steps to Reproduce

Expected Behavior
Sample of data you have in Elasticsearch

Full Watcher JSON

Screenshots (if relevant)
Errors in Browser Console (if relevant)

Kibana logs and/or server output (attached if relevant)
Any additional information
Index defines itself by time, different points per day, index_today's time
Select an Index input box, can you write a time function?

I'm not sure I understand the question or the final purpose here without context or any example.
I'll assume you're referring to the Alert Indices - those are indeed created as daily indices by Sentinl and cannot be modified in order for the Sentinl Kibana App to use their data consistently. If you prefer to store Alerts your own way, you can disable alert indexing and write using the output functions. I hope this helps.


How it looks right now

Hello @kimrutherford and @ValWood.
I finished, took quite a bit more than I anticipated. How it's looking now:
> auto-fix alleles: 1117
   syntax_error 311
   type_error 58
   type_error_and_syntax_error 20
   unknown_in_description 682
   update_coordinates 46

> need_supervision: 150
   coordinate_change_no_mutation 12
   coordinate_change_creates_error 10
   shift_coordinates 124
   update_and_shift_coordinates 4

> cannot fix alleles: 455
   nucleotide alleles: 183
   aminoacid alleles: 258
   disruption alleles: 14

   sequence_error: 334 of which 134 nucleotide
   pattern_error: 96
   invalid_error: 25

For the non self-evident:

type_error: the allele_type has to be changed
unknown_in_description: The description is unknown. I think it should be left empty instead (we have both instances in the database). unknown is also already in the allele_type
need_supervision: alleles that would need a quick check to decide whether to accept the proposed auto-fix:

coordinate_change_no_mutation: for alleles in which the coordinate of the genes changed, but since they were partial deletion, the original coordinates referred to no aminoacid E.g. before change: deletion of 120-150, after change deletion of 130-160. A decission based on the other alleles in the session has to be made.
coordinate_change_creates_error: for alleles in which the coordinate of the genes changes, and there was no error in the sequence before, and introducing the changes has created an error (very likely they already used the right coordinates).
shift_coordinates: for alleles with sequence errors, in which shifting the coordinate by 1 fixes the error. E.g. there is no V123, but there is V122. Some alleles have all aminoacid shifted by one, clearly indicating that at least for those the fix makes sense.
update_and_shift_coordinates: for alleles in which the coordinate of the genes changes. Updating the gene coordinates + a shift by one as in shift_coordinates would fix the sequence error.

pattern_error: The allele description is not captured by any of the regular expressions in the grammar.
invalid_error: There is some inconsistency in the description. E.g. VAC-123-PP. There should be as many aminoacids in one side and the other if it is a substitution.

We can discuss the implementation and how to go aobut it in one of the next calls.

auto-fix alleles: 1117

Thanks Manu.  This is going be very helpful.


Mapping Section Not Fitting Container

Hello,
I am playing about with this control, which is great by the way.
In my project I am having an issue when using the control and that the interactable part of the control (ie the actual map) is around 200px wide where as the whole map container is set to width and height 0 and correctly fits the page. The result is a large grey space where the map should be, the Map/Satellite toggle does fill the page correctly and appears over on the right hand side of the container. When I inspect element on the page the interactable part of the control changes to fill the page correctly.
Incidentally I have an hidden container on the left hand side of the main content that the map control is included in that is the same size that the map generates, so i suspect the map is being sized off this in some way rather than the container that it is included in. However, the problem persists if i remove the hidden left panel from my project. I am also using the DevExpress controls in this project.
I have been testing in Chrome Verion 45.0.2454.93 m and Maxthon <IP_ADDRESS> to reproduce the problem.
It is not an issue using IE <IP_ADDRESS>.16431 - which I guess implies that there is a specific issue in my project but I thought it was worth raising anyway. I will update should I find out anything new.
Here is a screenshot:
http://imgur.com/psFJRAB
Regards,
Simon

While investigating this I have noticed that this occurs when the div that the map is sitting in has it's wisth set in terms of % rather than in absolute pixels. When using absolute pixels the map is the expected size.
Thanks
Simon

Thanks for your feedback!!!!


(NOT AN ISSUE) How did you make this?

I'm trying to make my own minecraft hacked client for 1.12.2, but I can't figure out how. Do you have any resources to help me?

Watch tutorials on how to create a forge mod

bruh

lol


Sort printed package list in brew upgrade

At the beginning of a brew upgrade run, brew prints the list of packages to be upgraded e.g. as
ncdu 2.2.2 -> 2.3
nmap 7.94 -> 7.94_1
jq 1.6 -> 1.7.1
sdl2 2.26.5 -> 2.30.0
libusb 1.0.26 -> 1.0.27
<EMAIL_ADDRESS>3.11.3 -> 3.11.7_1
file-formula 5.44 -> 5.45

The order is apparently arbitrary (I think maybe FS readdir order?) which makes it difficult to easily eyeball the list to see if some specific package of interest is included. Instead, sort it.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[not yet!] Have you successfully run brew tests with your changes locally?

I think we can just add some headings to the brew outdated output, as long as we don't break brew outdated | xargs and brew outdated --json, possibly by reusing the output_dump_formula_or_cask_report method from brew update-report.

I do brew upgrade wget ack it'll always try to upgrade ack first which is not correct.

Why would this matter? I would not expect this to necessarily follow the argument order.

Why would this matter? I would not expect this to necessarily follow the argument order.

because it's the way things have always been done and I'd rather not change it for no good reason
because of how errors are handled

Passing on this, sorry.


Persist messages to kafka

Hello.
We're considering adding support to persist all MQTT topic matching a certain filter, to Kafka. Not sure if this is relevant for the project. If it is relevant for the project, let me know and we'll work together as we implement it according to your guidelines.
If not, which is perfectly fine, we'll just do it in a private fork.
All the best,
Per.

Bridging messages to kafka is already supported, see plugin/bridge/kafka.
If you find any problems or need to add any capabilities, please report back.

that was funny. thanks for pointing me the right way.


Make new file, translations.yml

[ ] write migration from some current config settings to translations.yml

[ ] your-server-name
[ ] total-server-stat-title ('total on')
[ ] top-list-title ('top')

[ ] add new translation options for:

[ ] cm
[ ] m
[ ] miles
[ ] km
[ ] hp
[ ] hearts
[ ] weeks

[ ] W

[ ] days

[ ] D

[ ] hours

[ ] H

[ ] minutes

[ ] M

[ ] seconds

[ ] S

No need for this


Missing ADA's in transfer !

Hi all,
I have transferred 500+ ADA from my Nami wallet to my Binance account, it has been 7 hours already and the funds haven't shown up, even though the transaction went through normally and the ADA's left the wallet immediately.
Please help!

This repository is for updates to developer portal website. You can check the list of community channels at this forum post for any assistance.
For mentioned case, you'd want to check if your balance is deducted from your source using an explorer , and then follow-up with the binance or nami support depending on where you see descrepancy


Bedrock Edition (MCPE/Win10) in Windows

Trying to open Bedrock edition worlds in MCEdit on Windows
I have been reading through the steps to run from source and have managed to boot to the menu however I am encountering problems loading bedrock worlds. I saw there is a build file for linux only so I can't run this.
Currently getting the following error. Any help or suggestions?
[    INFO][      pymclevel.mclevel.py:200]:Identifying C:\Users\james_000\AppData\Local\Packages\Microsoft.MinecraftUWP_8wekyb3d8bbwe\LocalState\games\com.mojang\minecraftWorlds\-WEDACfOAAA=\level.dat
[   ERROR][                  root.py:1189]:Wasn't able to open a file {file => C:\Users\james_000\AppData\Local\Packages\Microsoft.MinecraftUWP_8wekyb3d8bbwe\LocalState\games\com.mojang\minecraftWorlds\-WEDACfOAAA=\level.dat}
Traceback (most recent call last):
  File "C:\Users\james_000\Desktop\New folder (4)\MCEdit-Unified-master\leveleditor.py", line 1186, in loadFile
    level = pymclevel.fromFile(filename)
  File "C:\Users\james_000\Desktop\New folder (4)\MCEdit-Unified-master\pymclevel\mclevel.py", line 226, in fromFile
    return PocketLeveldbWorld(filename)
  File "C:\Users\james_000\Desktop\New folder (4)\MCEdit-Unified-master\pymclevel\leveldbpocket.py", line 585, in __init__
    self.worldFile = PocketLeveldbDatabase(filename, create=create)
  File "C:\Users\james_000\Desktop\New folder (4)\MCEdit-Unified-master\pymclevel\leveldbpocket.py", line 314, in __init__
    self.options = leveldb_mcpe.Options()
AttributeError: 'NoneType' object has no attribute 'Options'
[   ERROR][                   root.py:677]:Failed to load file C:\Users\james_000\AppData\Local\Packages\Microsoft.MinecraftUWP_8wekyb3d8bbwe\LocalState\games\com.mojang\minecraftWorlds\-WEDACfOAAA=\level.dat: TypeError("object of type 'NoneType' has no len()",)

If you want you can use me to help test bedrock support on windows

To open PE worlds, you have to use the new support, which is not accessible by default. Follow the steps here to activate this new support. It's talking about OSX, but the principle is the same on Windows. (The DLL is already in the sources.)
It will become the default method to open PE worlds soon :smile:

Thanks for the reply. Getting an issue with the LevelDB-MCPE.dll file
[   ERROR][      pymclevel.leveldb.py:107]:[Error 193] %1 is not a valid Win32 application
even trying to load the dll through IDLE is producing the same problem. From a quick google search of that error it could be an architecture issue but I don't know. My PC is 64bit and to my knowledge everything installed should be 64bit as well

Come back with a fresh head and realised the default python installer is a 32bit and you have to dig a little deeper to find the 64bit installer. Reinstalled and redownloaded all the 64bit libraries and it seems closer to working. Getting this error now.
  File "C:\Users\james_000\Desktop\New folder (4)\MCEdit-Unified-master\mclangres.py", line 91, in <genexpr>
    versions = ['%s.json' % '.'.join(map(u"{}".format, c)) for c in sorted((map(int, b) for b in (a.split('.')[:-1] for a in versions)))]
ValueError: invalid literal for int() with base 10: '4_af'

I tried debugging it but didn't get anywhere. Commenting that line out allows the world to open fine

Looks like you're using a snapshot or pre-release game version.
Which one is it?

I am using Minecraft Windows 10 Edition V1.1.0 which is the latest main release. Not a snapshot or pre-release

OK.
I found the reason for this bug.
You have installed a game version (or pre-release or snapshot) which contains '4_af', and this is why MCEdit crashes. (The crash is not related to the PE support, but to another part of the program.)
As a work-around until it's fixed, move all the files in the 'assets/indexes' game sub-directory which do not contain only digits and dots when you want to use MCEdit. Restore them before launching the game.


As a marketing manager, I want to generate a report of the countries in a continent organised by largest population to smallest to identify new market opportunities and develop innovative marketing strategies.

Changed title to match report # 2
All the countries in a continent organised by largest population to smallest

Generates report of countries from largest pop. to smallest pop.


Fix @export_ attributes & add multiline prop support

@export_ attributes:
Fixed parsing of comments associated with @export_ attributes (such as @export_enum and @export_flags
Multiline property support:
Added capability to handle properties defined across multiple lines
It's not particularly well tested but seems to be working - either way, I'd recommend more stringent testing before accepting.
Additionally, from the rest of the code and from what was there, you were using unicode characters - for performance, I presume, but I wasn't familiar enough to use it, sorry!

Interesting. I haven't tried that, good catch! I also didn't notice that quotation marks don't block ( ) [ ] and { } from being detected, which would indeed be the correct behaviour.
I made some changes to the implementation, to avoid changing method structure much: 4b8a05d075216ddd06026a9b26830e4bbf96f653


Crashes on pasting links

I've had Obsidian crash a few times just after pasting a hyperlink.
I also can't narrow it down to specific sites,  ( I recall a link to Zenodo.com, or a shortcode url for example)
I can only imagine that problems  were created by an obsidian update - since this plugin did not update recently.
Maybe others had a similar experience?
edit:
Crashed after pasting this URL: https://www.integromat.com/en
Opened Obsidian again, (the link text was stuck on "fetching title"), pasted again - crashed again.

What version of Obsidian are you on and which OS? I just pasted that URL and the title was resolved without issue.
I'm using Obsidian 0.14.6 and macOS 12.3.1.

Windows 10 and the up2date version of Obsidian at the time of posting (I did update Obsidian and all plugins in search of an easy fix).
Right now I'm on 0.14.6, v1.2.5 of the plugin
But I can still reproduce the crash.
I'm now in my home network but I first reported this from office network, so I can kind of exclude external factors (I think ?)

Encounter the same problem. Windows 10 at version 0.15.9

+1, windows 11, plugin version 1.2.5 with obsidian v1.0.3

Same happens on Mac.  It's happening consistently, everytime when i copy and paste a line with url and checkbox.
About Obsidian
Version 1.1.8 (Installer 1.0.3)

Happens all the time on Windows

Interesting, I use this on both Windows and macOS but haven't had it cause a crash.
Can you give me a way to reproduce it? Is it every link you post or is it in specific circumstances?
If you have a specific line that you're pasting a specific link into that causes a crash, if you are able to post it here so I can recreate that would be great!
Thanks!

Interesting, I use this on both Windows and macOS but haven't had it cause a crash. Can you give me a way to reproduce it? Is it every link you post or is it in specific circumstances?
If you have a specific line that you're pasting a specific link into that causes a crash, if you are able to post it here so I can recreate that would be great!
Thanks!

It's strange but for some reason, plugin stopped crashing Obsidian and started to show "Site unreachable". Work on every link

The crashes stopped for me, eventually - I cannot say what caused the change though.

Okay, hm, I'm going to close this for now but if it starts crashing again please let me know!

Observered crash on this link https://zhuanlan.zhihu.com/p/30135457


Set data in Select2 after insert with AJAX

I'm using the Select2  with AJAX (the code below):
$(".select2-ajax").select2({
        placeholder: "Search user",
        minimumInputLength: 1,
        ajax: {
            url: $('#url-search-client').val(),
            dataType: 'json',
            type: 'post',
            data: function (term, page) {
            return {
                filter: term
            };
            },
            results: function (data, page) {
            return {results: data};
            }
        },
        width : '50%',
        formatInputTooShort: function () {return 'Informe mais caracteres'; },
        formatResult: formatResultSelectAjax, // omitted for brevity, see the source of this page
        formatSelection: formatSelectAjaxValue, // omitted for brevity, see the source of this page
        dropdownCssClass: "bigdrop" // apply css that makes the dropdown taller
 });

Well, if not found client, the user can be use a button to open a modal and add the new client, is possible use the return(json with id and namae) of new client and put the data (like name) into the select2 as selected?
$('.btn-form-client').click(function() {
        $.ajax({
            url: $('#frm-client').attr('action'),
            dataType: 'json',
            type: 'post',
            data: $('#frm-client').serialize()
        }).done(function (data) {
            $('#modal-client').modal('hide')
        });
        return false;
    });

Can you reopen this issue ?
$(".select2-ajax").select2('data')is deprecated now and I'm struggling to add locale data to an ajax select2.
I tried the new recommended way.
var $option = $('<option>test</option>').val(10);
$select.append($option).trigger('change');

The option is added to the html node in the DOM but it doesn't show up in select2 drop down.
The documentation is very thin about how to mis local data and ajax data, it only redirects to old stackoverflow issues which are no longer working with version 4+


The run initialization should be protected with a Lock in order to safely start experiments in a multithreaded environment

When trying to run this code
from sacred import Experiment
from multiprocessing.dummy import Pool as ThreadPool
import itertools

ex = Experiment('toy_experiment')

from sacred.observers import MongoObserver
ex.observers.append(MongoObserver.create())

@ex.config
def config():
    param_a = 0.001
    param_b = 20
    param_c = 100

@ex.automain
def main(_run, param_a, param_b, param_c):
    import random, time
    print("I am id %s with param_a:%s param_b:%s param_c:%s " % (_run._id, param_a, param_b, param_c))
    time.sleep(random.randint(0,10))
    return param_a*param_b*param_c + random.randint(0,10)

getExperiments = lambda params : [dict((x,y) for x,y in zip(params.keys(), param)) for param in itertools.product(*params.values())]

params = {
    "param_a": [0.001, 0.01],
    "param_b": [20, 50]
}

pool = ThreadPool(2)
results = pool.map(
    lambda e : ex.run(config_updates=e),
    getExperiments(params)
)
pool.close()
pool.join()

I came across a non-deterministic behavior from sacred, sometimes throwing random errors and sometimes messing around with the configuration fields of the runs.
Looking in depth into the framework I've found out that the function run(...) of Experiment calls the  _create_run(...)  which is not thread-safe. Since the call to _create_run(...)  is not protected with a mutex running multiple experiments in parallel can lead to all kind of race conditions.
In this PR I've simply added a Lock instance in the Experiment class and protected the call to _create_run(...)  . For my understanding of the framework this can be a feasible solution since the _create_run(..) function is fast to run and protect it is not so costly in terms of performances.
A better way to do that would be protecting only the actually thread-unsafe blocks.

Coverage increased (+0.02%) to 84.158% when pulling fa81aaa3cf0c12b47cd06b0916c44418e6922e61 on galatolofederico:master into 6faaa8bc08fe8f0f8fe31cdfecb42b4e2af6c24c on IDSIA:master.

Hi @galatolofederico. Thanks for engaging! Unfortunately sacred in its current form is fundamentally non-thread-safe and I don't think locking _create_run will solve that. The main problem is that the active configuration is tracked globally for each captured function. That means calling a captured function from different threads will lead to inconsistent configurations.
Fixing this would be substantially more work, and probably include turning all global variables into thread-local ones.
For now I would suggest to not use sacred in a multi-threaded environment, and instead use muliprocessing.

Closing this, since it is an incomplete solution and would require substantially more work.


Remove region references from docs

Remove region reference from docs, we only support us-west-2 right now

/cherry-pick release-0.11

/hold

/lgtm

/unhold
/approve


Update "master" branch references to "main"

Update "master" branch references to "main".
Signed-off-by: David Enyeart<EMAIL_ADDRESS>

@bestbeforetoday Could you review?

/azp run


[Confluent] Upgrade version to 0.3.0 to trigger release pipeline

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

Confluent


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1276 ms

Resolved: Bofrost.at performance has improved in c3b211c after 7 minutes.


Moving ALZ-Monitor to AMBA

Overview/Summary
Within this PR you find the resource PolicyDefinitions and the ALZ pattern. These originate from "/azure/alz-monitor".
This PR fixes/adds/changes/removes

ALZ Policy definitions
ALZ Policy initiatives
ALZ pattern, ARM template and parameter file for deployment

Breaking Changes
Testing evidence
Deployment

Policy Assignments:

As part of this Pull Request I have

[x] Read the Contribution Guide and ensured this PR is compliant with the guide
[x] Checked for duplicate Pull Requests
[x] Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Ensured PR tests are passing
[ ] Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

I went ahead and merged the file as is, we will come back and fix the linking and link errors


Error: cbase.h: No such file or directory  #include "cbase.h"

Hello, I'm trying to build the project but I'm getting this error. I guess it need some SDK for building? If yes - what SDK?


#1356: Add question to generate preview files for modules and blocks.

Description
This adds a question to generate a module preview file and a block rendering file:

Related Issues
fixes #1356

If we keep this for 3.x we do not have to deal with Laravel < 6 errors.

If we keep this for 3.x we do not have to deal with Laravel < 6 errors.

works for me!

Hey @ifox,
Thinking, shouldn't we generate a site/layouts/block.blade.php file by default on install (3.x) as you effectively always need it?


disables field_error_proc for all bootstap generated code

This patch disables default field_error_proc for all code twitter_bootstrap_form_for generates cause before it was breaking bootstrap's layout by wrapping input and label with div.field_with_error
before:
http://dl.dropbox.com/u/8663332/twitter_bootstrap_form_for/before.png
after:
http://dl.dropbox.com/u/8663332/twitter_bootstrap_form_for/after.png

Out of curiosity, were you able to determine why this wasn't working for you before?

Oh, I see. In your original bug report, you hadn't wrapped the field with a fieldset using inputs. Can I ask why?

Pulled. I actually moved this bit of code out into the main twitter_bootstrap_form_for wrapper, so it should only ever need to be called once per form. I've tested this (with and without an inputs wrapper), but can you ensure that it works for you before I release?
609807a029e62cb03d69d4bb356a58b3abc3d31c

I can confirm that this resolves #23

Great. I'll push out a release soon. Have a few more issues that should probably make it in.

Yeah, works for me too.


Is it possible to modify what the php artisan view:cache command does?

Hey,
Is it possible to modify what the php artisan view:cache command does?
I want to add some scripts (JS or PHP) when we launch this command to replace some HTML tags on cached views in production...
Thanks!

This repo is for bug tracking. Use the forums or slack channel for solving your issue

Of course x)


chore(github-template): update issue template

Codecov Report

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

@@          Coverage Diff          @@
##           master    #16   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      4
  Lines         102    102
=====================================
  Hits          102    102

Continue to review full report at Codecov.

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


[fix](invert index) Fixed compatibility issues with tii tis

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall


ExternalTaskLock uses wrong parameter name

The ExternalTaskLock class should be using the name "LockDuration" instead of "Duration".  Currently the External Task lock will fail because it thinks you did not set the lock duration.

I'll fix this while I'm in that area.  PR will be up momentarily.

I belive this is fixed. the PR is merged.