failed to rename backups, aborting upload: archive/tar: invalid tar header

The problem
My first cron-scheduled backup did not run, and there's a log entry as shown in the title.
What version of Rclone Backup has the issue?
3.1.0
What version of Home Assistant are you running?
2024.3.1
What type of installation are you running?
Home Assistant OS
If you're running HA OS/Supervised, what version of the Supervisor are you running?
2024.03.0
Example YAML snippet
- name: Sync Daily Backups
  schedule: 10 4 * * *
  command: sync
  sources:
    - /backup
  destination: Proton:Backup/Home Assistant/
  include: []
  exclude: []
  flags: {}

Anything in the logs that might be useful for us?
s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
cont-init: info: running /etc/cont-init.d/00-banner.sh

-----------------------------------------------------------
 Add-on: Rclone Backup
 Automatically sync backup to remote locations using rclone
-----------------------------------------------------------
 Add-on version: 3.1.0
 You are running the latest version of this add-on.
 System: Home Assistant OS 12.1  (aarch64 / green)
 Home Assistant Core: 2024.3.1
 Home Assistant Supervisor: 2024.03.0
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums or the Discord chat.
-----------------------------------------------------------
cont-init: info: /etc/cont-init.d/00-banner.sh exited 0
cont-init: info: running /etc/cont-init.d/01-log-level.sh
Log level is set to DEBUG
cont-init: info: /etc/cont-init.d/01-log-level.sh exited 0
cont-init: info: running /etc/cont-init.d/init.sh
cont-init: info: /etc/cont-init.d/init.sh exited 0
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
services-up: info: copying legacy longrun rclone (no readiness notification)
services-up: info: copying legacy longrun scheduler (no readiness notification)
s6-rc: info: service legacy-services successfully started
[13:49:43] INFO: Installed rclone version:
[13:49:43] INFO: Starting Rclone Web GUI...
2024/03/19 13:49:44 NOTICE: Serving files from "/usr/share/rclone-webui-react"
2024/03/19 13:49:44 NOTICE: Serving remote control on http://[::]:8099/
2024/03/19 13:49:44 NOTICE: Web GUI is not automatically opening browser. Navigate to http://[::]:8099/ to use.
[13:49:44] INFO: rclone v1.65.0
- os/version: alpine 3.16.0 (64 bit)
- os/kernel: 6.6.20-haos (aarch64)
- os/type: linux
- os/arch: arm64 (ARMv8 compatible)
- go/version: go1.21.4
- go/linking: static
- go/tags: none
[13:49:44] INFO: Starting Scheduler...
[2024-03-19] [13:49:44] [INFO]: rclone config found
[2024-03-19] [13:49:44] [INFO]: checking job configs...
[2024-03-19] [13:49:44] [INFO]: scheduled jobs:
10 4 * * * sync "Sync Daily Backups"; /backup -> Proton:Backup/Home Assistant/
failed to rename backups, aborting upload: archive/tar: invalid tar header

Additional information
No response

Hello, i can confirm the above. I had the same issues with invalid tar header. This was due to the fact i forgot to exclude Frigate from my backup. And each backup would fill up atleast 55gb. After a few backups my whole drive did a booboo. I excluded Frigate, Removed all backups via SMB /backup, and retested with new backups. All good now.

Glad you got to the solution, if I have some time I might make a troubleshooting section for this issue, it seems to trip people up quite often, and maybe add more info to the error message 👍
I'm going to close this as it seems resolved.


Should object pool base class be Abstract?

Question:
Shouldn't public class ObjectPool<T> be an abstract class? Why/Why not?
Thank you for your time.

Hey, thanks for your question!
I think that could go either way; it's not absolutely necessary for ObjectPool to be abstract. It's true that abstract indicates that a class is intended to be a base class of other classes, but this is also true of a generic class. A generic class must be a base class of some other class. Just like an abstract class, you can't instantiate a generic class (without giving it a type).
If you look at the source for List<T> in the .NET Framework you'll see that it's not abstract. I like to use the .NET Framework as a guideline so that's another reason to not make it abstract.

That makes perfect sense! I overlooked the fact that it was generic and jumped into wondering why wasn't it an abstract class. But you can't instantiate T by itself, you'd need to specify it's type achieving in the process exactly what I wanted. Thanks for the answer, we are now officially two happy nerds. Cheers!


Supporting Raspberry Pi

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

Thank you for your contribution! We will review it and merge it once we have time.


wasm loading maps broke

There's a usize somewhere that's overflowing u32. I thought https://github.com/a-b-street/abstreet/pull/582 was the culprit, because it adds 100,000 to some contraction hierarchy costs. Except, serialize_usize catches the overflow at serialization time, and rewinding past this commit and regenerating doesn't help.
So I'm not sure what's up yet, using this issue to track debugging

Everything's fine as of b6c5ee38c24a7bdcaa249bd507ce63686ed916a1. Some of my tests earlier might've been wrong, because Firefox was still caching something, despite frantic refreshes.

OK at e672256828fda4c1a45db1f8669cb53b225339eb

Yeah, the elevation commit somehow broke things: invalid value: integer<PHONE_NUMBER>, expected usize at line 1075 column 43
All it did was add percent_change: f64 to Road, so... wat?

Oh my. If I read the stack trace, I'd realize what's breaking is abstutil::from_json(&include_bytes!("../../data/MANIFEST.json").to_vec()).unwrap(). That commit added the absolutely massive Seattle lidar geotiff, which expresses the uncompressed bytes as a usize.

Fix deployed


Discovery: Drop search results from previous queries

This patch adds a new optional parameter to the searchChannels function
to be able to drop search results for old searches.
The SearchPage component increments the searchQueryIndex for every
search, so if there's a race condition and a search query done before
ends after a second search query has been launched, it will be dropped
because the searchQueryIndex returned will be different from the one
specified in the new search query.
https://phabricator.endlessm.com/T33202

This is customizing the API that we have already upstreamed. Please mention this in the code as a comment, and try to do it in a way that leaves the original API intact (see getRandomNodesReal for an example).

This commit doesn't change the Kolibri API, the change is done in the kolibri-explore-plugin custom function searchChannels that's only used in our discovery search page. This problem only happens here because we've the component as keep-alive, so when going back to a different view, the old searches are running in the backend.
In any case, the index modifies the function extending it, but the interface is the same, because the new parameter is optional, if it's not provided, the result will be the same, because the searchQueryIndex will be always 0, so no results will be dropped.


Add support for discord replies

Allow for bridged messages to minecraft and IRC to also show the message which a user on discord replied to

implemented in commit cfdd1f482da710fe70e9dc3a21e94368ad8e3a96


Impossible to unpause a game using a gamepad

This Issue is about (delete as applicable)

A bug in the API (always say which version you're using!)

I feel like this would take too long to set up in the sandbox, and I think the issue is self-explanatory: If you're controlling a game using the gamepad, and you use Phaser's game.paused bool to pause the game, you can't unpause it with controller input because Phaser's pause also stops polling all gamepad input. This applies up to the latest official version of Phaser 2, 2.6.2.

+1

+1

Thank you for taking the time to open this issue. However, official support for Phaser 2 has now ended as we focus on preparing Phaser 3 for beta release. In a bid to keep the project tidy, and avoid confusion, we are closing all v2 related issues.
This does not mean your issue won't be looked-at. In November 2016 the Phaser CE (Community Edition) project began. The aim of Phaser CE was to allow the community to continue v2 development. Since then it has gained significant traction, with many updates and releases.
If you are still interested in having your issue investigated we strongly recommend testing the latest Phaser CE release. And only if the problem persists, opening an issue on the Phaser CE repo.


Fix user resource usage metrics

Coverage decreased (-0.09%) to 91.537% when pulling 5ddb07167387e5e687dcf4b53193e10198578aa7 on mprimi:master into 9d7cb6b74580545e76bcee9e3786f131489de5d6 on Netflix:master.


defaultSettings is mutated in loadSettingsAsync

https://github.com/yuanqing/create-figma-plugin/blob/1a276ea04c35a37494babcc964b62dd3cdf6b383/packages/utilities/src/settings.ts#L19
The fix is:
 return Object.assign({}, defaultSettings, settings)

Fixed in 1.8.3


How to access the noreopen projection data?

The website added a -noreopen option on May 13, such as https://covid19-projections.com/us-noreopen. The projection is very valuable for analyzing the policy effect. I am wondering how can I access the csv files corresponding to those noreopen projection?
I guess the noreopen projection is made based on the historical projection at someday past? So that I can just go back to download the projection at someday past, say May 13, the projection will be corresponding to the noreopen projection? Am I right?
If you got time, it will be great if you could say more about how does the noreopen projection is made comparing to the normal projection?

I didn't upload the data for the no-reopen scenarios but if you fill out the Contact form (https://covid19-projections.com/contact), I'll see if I can send it to you.

Great. I have just filled the contact form.


Rename modules to reflect JVM association?

Should we rename the current modules to reflect their association with Java / the JVM?
Background:

We had a discussion about the issue of supporting other programming languages in the future.
It came up again in PR #52

Suggestion:

livingdoc-jvm-engine
livingdoc-jvm-fixture-api
livingdoc-jvm-fixture-converter

The only module which does not need a rename is livingdoc-junit-engine since JUnit is explicit enough.

Decision: Don't rename modules - JVM is assumed as default.


additional llvm version compile switches

We're currently not supporting LLVM 4 and 5 because of a bug in the code generator. You can find additional details in #2061. If you only have access to LLVM 4 or 5 but still want to compile ponyc from source, you can use @kulibali's WIP branches for LLVM 4 and 5. I want to stress that using ponyc with LLVM 4 or 5 can cause invalid machine code generation, so you shouldn't use these versions for production use.
I'm going to close this PR but if you'd like to push the support for LLVM 4 and 5 forward, feel free to step in in #2061.

Okay, thanks, the info is much appreciated.


Transport creation enhancement

Hi Florian,
thanks a lot for a great tool!
Could you please enhance transport creation option?
I assumed that when you use "transport creation with user match", it would check all open transports of given user searching for the same transport text, as provided in grunt task configuration.
However, it just picks up first opened transport of the user.
Could you please enhance it?

Determination functionality works as designed -> no enhancement regarding text comparison will be done.


refactor(elixir): replaces deprecated Logger.warn

The Logger.warn macro is deprecated in favour of the Logger.warning one.
(https://hexdocs.pm/logger/1.15.6/Logger.html#warn/2)

Current behavior
When running some Elixir example, the following warning is printed:
$ elixir 01-worker.exs
warning: Logger.warn/1 is deprecated. Use Logger.warning/2 instead
  (ockam 0.1.0) lib/ockam/worker.ex:278: Ockam.Worker.schedule_idle_timeout/1

...

Proposed changes

Update Elixir code to replace Logger.warn by Logger.warning.
Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] There are no Merge commits in this Pull Request. Ockam repo maintains a linear commit history. We merge Pull Requests by rebasing them onto the develop branch. Rebasing to the latest develop branch and force pushing to your Pull Request branch is okay.
[x] I have read and accept the Ockam Community Code of Conduct.
[x] I have read and accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam repository. The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a separate Pull Request, this will mark the commit as verified.

@paulozulato Thank you for spending time on this and sending this pull request. While we review please:

[ ] Accept the Ockam Contributor License Agreement by adding your Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate new pull request to the build-trust/ockam repository.

The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a PR, this will mark the commit as verified.

Hi, @mrinalwadhwa, how are you?
I had already accepted the Ockam Contributor License Agreement before (https://github.com/build-trust/ockam/blob/develop/.github/CONTRIBUTORS.csv#L261). Should I do anything else on that?

No need to do anything else, we'll review the PR shortly

I'm doing well 😀

Good to hear! :smile:

No need to do anything else, we'll review the PR shortly

OK. Thank you very much :blush:

thanks for your contribution @paulozulato ! 👍 ,  accepted and going to be merged soon.


aeson-typescript generates incorrect type definitions for Data.Map

This code:
https://github.com/codedownio/aeson-typescript/blob/16d6c2a40730fdc72784ca70391b68c5086d78a1/src/Data/Aeson/TypeScript/Instances.hs#L114-L116
seems to assume aeson generates maps for Data.Map.
This does not appear to be the case. Since JavaScript maps must be keyed by strings, aeson instead generates key-value pairs, see:
https://github.com/haskell/aeson/blob/65fca856304c0b323d538cc783493d8d7a145114/src/Data/Aeson/Types/ToJSON.hs#L1684
and the discussions at:
https://github.com/haskell/aeson/issues/259
https://github.com/haskell/aeson/issues/79
It seems aeson-typescript should instead generate key-value pairs.

It certainly encodes to a JSON object for every case I'm aware of. Those are some pretty old issues also. Do you have an example of this?

Oh, I guess this can happen when you use a ToJSONKeyValue function, see https://hackage.haskell.org/package/aeson-<IP_ADDRESS>/docs/Data-Aeson-Types.html#t:ToJSONKey

Ah indeed, in our example we have a:
newtype Name (sym :: Symbol) = Name { name :: Text }
  deriving (Eq, Ord, Show, Generic, NFData)

that has a ToJSONKeyValue instance and is used as the key of a Map Name ...!

Yep, sounds like we need to modify the instance to handle this case. Wouldn't you rather just use ToJSONKeyText? IIRC you can even auto derive it from a newtype like this.

Sorry I mistyped, just a ToJSONKey* instance. I don't think we call either of toJSONKeyValue or toJSONKeyText explicitly, but I'm not certain as I did not write those instances.


Added UUIDConverter

Possible fix to allow UUID-based keys. Fixes issue https://github.com/hakandilek/play2-crud/issues/60

That's great. Thanks!


Remove ufs cleaner

What changes are proposed in this pull request?
Remove UfsCleaner and supporting code
Why are the changes needed?
FileSystemMaster/BlockMaster functionalities are no longer needed. This is one step in removing them.
Does this PR introduce any user facing changes?
Related property keys are removed

@Jackson-Wang-7 PTAL if there's something you want to keep for this feature, for example PropertyKey.UNDERFS_CLEANUP_ENABLED is still used by ObjectLowLevelOutputStream and ObjectMultipartUploadOutputStream. Thanks!

alluxio-bot, merge this please


Notifications via email and teams for Legion environments that  are alive more than configured period of time

Add regular job that will check and send notifications via email and slack for Legion environments in AWS which are alive more than configured period of time. By default it should be 8 hours.

Moved to the ci-cd


Improve request.js

Used an early return if there is no cookie for making code more readable.

Not a big fan of early exit wounds in most cases. I don't think this change clears the bar of improvement on that. Appreciate you taking a swing regardless!


BUG Can't add nodes using instructions from documentation.

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
BUG REPORT

Environment:

Cloud provider or hardware configuration:
Scaleway
OS (printf "$(uname -srm)\n$(cat /etc/os-release)\n"):
It's an issue with the playbook logic or ansible
Version of Ansible (ansible --version):

ansible@rundeck:~/.kubespray$ ansible --version
ansible <IP_ADDRESS>
  config file = /home/ansible/.kubespray/ansible.cfg
  configured module search path = [u'./library']
  python version = 2.7.12 (default, Nov 19 2016, 06:48:10) [GCC 5.4.0 20160609]

Kubespray version (commit) (git rev-parse --short HEAD):
b495d36
Network plugin used:
Copy of your inventory file:
[kube-master]
master01
[all]
node01          ansible_ssh_host=<IP_ADDRESS> ip=<IP_ADDRESS>
node02          ansible_ssh_host=<IP_ADDRESS> ip=<IP_ADDRESS>
node03          ansible_ssh_host=<IP_ADDRESS> ip=<IP_ADDRESS>
master01        ansible_ssh_host=<IP_ADDRESS> ip=<IP_ADDRESS>

[k8s-cluster:children]
kube-node
kube-master

[kube-node]
node01
node02
node03

[etcd]
master01

Command used to invoke ansible:
ansible-playbook -i inventory/inventory.cfg cluster.yml -u root --limit node02
Output of ansible run:

TASK [kubernetes/preinstall : set_fact] ***************************************************************************************************************
Saturday 05 August 2017  09:40:07 -0400 (0:00:00.041)       0:00:14.579 *******
fatal: [node02]: FAILED! => {"failed": true, "msg": "the field 'args' has an invalid value, which appears to include a variable that is undefined. The error was: 'dict object' has no attribute 'ansible_default_ipv4'\n\nThe error appears to have been in '/home/ansible/.kubespray/roles/kubernetes/preinstall/tasks/set_facts.yml': line 14, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- set_fact:\n  ^ here\n"}
        to retry, use: --limit @/home/ansible/.kubespray/cluster.retry

PLAY RECAP ********************************************************************************************************************************************
node02                     : ok=14   changed=0    unreachable=0    failed=1

Saturday 05 August 2017  09:40:07 -0400 (0:00:00.051)       0:00:14.630 *******
===============================================================================
bootstrap-os : Gather nodes hostnames ----------------------------------- 3.43s
bootstrap-os : Remove require tty --------------------------------------- 2.72s
bootstrap-os : check if atomic host ------------------------------------- 2.65s
adduser : User | Create User Group -------------------------------------- 1.31s
adduser : User | Create User -------------------------------------------- 1.29s
kubernetes/preinstall : check bin dir exists ---------------------------- 1.10s
bootstrap-os : Assign inventory name to unconfigured hostnames ---------- 0.32s
kernel-upgrade : Rebooting server --------------------------------------- 0.10s
kernel-upgrade : upgrade kernel ----------------------------------------- 0.09s
kernel-upgrade : set_fact ----------------------------------------------- 0.09s
kernel-upgrade : wait for bastion to come back -------------------------- 0.08s
kubespray-defaults : Configure defaults --------------------------------- 0.08s
kernel-upgrade : Wait for some seconds ---------------------------------- 0.07s
kubernetes/preinstall : set_fact ---------------------------------------- 0.07s
kernel-upgrade : install elrepo repository ------------------------------ 0.06s
kernel-upgrade : install ELRepo key ------------------------------------- 0.06s
kernel-upgrade : change default grub entry ------------------------------ 0.06s
kubernetes/preinstall : set_fact ---------------------------------------- 0.05s
bootstrap-os : Install required python modules -------------------------- 0.05s
kernel-upgrade : set_fact ----------------------------------------------- 0.05s

Anything else do we need to know:

Looks like ansible is trying to do this.
- set_fact:
    first_kube_master: "{{ hostvars[groups['kube-master'][0]]['access_ip'] | default(hostvars[groups['kube-master'][0]]['ip'] | default(hostvars[groups['kube-master'][0]]['ansible_default_ipv4']['address'])) }}"

some reason it fails?
ansible@rundeck:~/.kubespray$ ansible localhost -m debug -i inventory/inventory.cfg -a "var=hostvars[groups['kube-master'][0]]['ip']"
localhost | SUCCESS => {
    "changed": false,
    "hostvars[groups['kube-master'][0]]['ip']": "<IP_ADDRESS>"
}

Works for me but I can't remember what all those pipes do.
It would appear it is looking for a hostfact that was not set by Ansible do to the --limit flag.
replacing with this
- set_fact:
    first_kube_master: "{{ hostvars[groups['kube-master'][0]]['access_ip'] | default(hostvars[groups['kube-master'][0]]['ip']) }}"

works but again fails farther down
- set_fact:
    etcd_access_addresses: |-
      {% for item in groups['etcd'] -%}
        https://{{ hostvars[item]['access_ip'] | default(hostvars[item]['ip'] | default(hostvars[item]['ansible_default_ipv4']['address'])) }}:2379{% if not loop.last %},{% endif %}
      {%- endfor %}

same var ansible_default_ipv4
and it continues to fail when referenching that variable.

Outdated


chore(ci): adapt add.tokens.erc20 script to new ENV files format

Motivation
After PR #1707 , the format for the ERC20 token env files changed. So, this PR aims to adapt the add.tokens.erc20 script to that.

Since PR #1707 is in standby, I will close this one and re-open when and if necessary.


Add note indicating that changing Gravatar images might take a while to appear

Because the user page is cached, if the user changes their Gravatar image, it might take a while before it actually shows up on their profile page.
We should add a note below the "Change Image on Gravatar.com" link informing users that they may not see the change right away.

Good First Issue: This issue is good for first time contributors. If you've already contributed to Warehouse, please work on another issue without this label instead. If there is not a corresponding pull request for this issue, it is up for grabs. For directions for getting set up, see our Getting Started Guide. If you are working on this issue and have questions, please feel free to ask them here, #pypa-dev on Freenode, or the pypa-dev mailing list.

In today's Warehouse developers' meeting we decided to pare down our near-future milestones on our development roadmap so they really only contain the essential bugfixes and features we need to launch, replace legacy PyPI, and shut down the old site. So I'm moving this issue into a milestone further in the future.

This is effectively fixed with #3368.


MdTabGroup inside #elseTemplate causes browser's tab to crash silently.

Bug, feature request, or proposal:
Bug
The bug may be related to the MdTabGroup/MdTab implementation or to the new implementation of ngIf
What is the expected behavior?
It should work.
What is the current behavior?
Browser's tab crashes silently (IE, Chrome, Edge)
What are the steps to reproduce?

Which versions of Angular, Material, OS, browsers are affected?
Material 1.0.0-beta-2
Angular 4.0.0-rc2

I'm not able to replicate this issue with Angular 4.0.0-rc2. I did however run into some other bugs with templating that the core team is currently working through for the next release.

This seems to be related:
#3461
angular/angular#14873
In my real application the boolean is modified via button click event.
This may be the reason why you can't reproduce, so the MdTabGroup may not be the problem there.
I will close this issue and re-open it if it's still present after angular/angular#14873 fix.


Removing border from calendar

I would like to remove the border from FSCalendar in order obtain a result similar to this.
At the moment it looks like this:

Which parameter should I modify?

Hi, let me see your code.
Because I don't have that border by default.

Or maybe set calendar.pagingEnabled to YES

Same issue for me:

calendar.pagingEnabled doesn't work.
Thanks for your help

You need to modify the source code in FSCalendar.m @squall2022

Thanks @Seanwong933 it works !

it works, thanks a lot.

I have just noticed that removing the bottom border will make the handle disappear:

Instead of removing the views I change the color externally. In some header:
#define ASK_GETTER(OBJECT, GETTER, DEF) \
    _Pragma("clang diagnostic push") \
    _Pragma("clang diagnostic ignored \"-Wundeclared-selector\"") \
    ((![OBJECT respondsToSelector:@selector(GETTER)]) ? (DEF) :     \
        ([OBJECT performSelector:@selector(GETTER)])) \
    _Pragma("clang diagnostic pop")

Then loading the controller:
    UIView* border = ASK_GETTER(self.calendar, topBorder, nil);
    border.backgroundColor = [UIColor clearColor];
    border = ASK_GETTER(self.calendar, bottomBorder, nil);
    border.backgroundColor = [UIColor clearColor];

Reading again the docs I found the official way to get rid of those top/bottom borders.

calendar.clipsToBounds = YES

Where do you declare calendar.clipsToBounds = true?

Thanks @Seanwong933


[Popover] Allow to pass repeated props to modal

Allowing to pass repeated props to modal. For example, it's impossible to pass the classes prop to the modal.

PR updated.
@oliviertassinari could you help me with this error please Should not have any git not staged?
Not sure what's causing it.

@davibq It's a great first pull request on Material-UI 👌🏻. Thank you for giving it a shot!


Added support for Cisco AireOS Wireless LAN Controllers

Offering up my work to add support for Cisco AireOS WIreless LAN Controllers (WLC).

This looks awesome and I appreciate you for putting this together and contributing it to this repository.  I plan to dig into this more later tonight.   I realized I had a few things in the devel branch that  were never merged into master, so I did that and it caused a merge conflict on one file (sessions.py).   I think I can deploy this PR into a new branch and sort that out, or if it looks easy on your side to change the destination to the devel branch, that would be helpful as well.
In short, I just want to make sure I can deploy this somewhere that I can make sure all of the Sphinx auto-generating documentation stuff is in place before merging into master.
Thanks again!

Not a programmer at heart, so I am learning.  Had to sort of hack in how to detect it is a Cisco AireOS WLC.
BTW, actually my first experience with github, so hope I did it correctly.
  Have more coming on this front, but what I submitted was a solid chunk to get started.
.:|:.:|:. Gordon Rogier, CCIE-Wireless #26419 | Technical Leader, Customer eXperience |<PHONE_NUMBER>mailto:972.358.1580 | grogier<EMAIL_ADDRESS>From: Jamie Caesar<EMAIL_ADDRESS>Sent: Monday, May 4, 2020 2:18 PM
To: jamiecaesar/securecrt-tools<EMAIL_ADDRESS>Cc: Gordon Rogier (grogier<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [jamiecaesar/securecrt-tools] Added support for Cisco AireOS Wireless LAN Controllers (#49)
This looks awesome and I appreciate you for putting this together and contributing it to this repository. I plan to dig into this more later tonight. I realized I had a few things in the devel branch that were never merged into master, so I did that and it caused a merge conflict on one file (sessions.py). I think I can deploy this PR into a new branch and sort that out, or if it looks easy on your side to change the destination to the devel branch, that would be helpful as well.
In short, I just want to make sure I can deploy this somewhere that I can make sure all of the Sphinx auto-generating documentation stuff is in place before merging into master.
Thanks again!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jamiecaesar/securecrt-tools/pull/49#issuecomment-623654005, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APBEUSASIVLQN4IIDOKQNKDRP4IIFANCNFSM4MYOYFQQ.

Also, please let me know if you have any problems accessing the new branch.

Made some fixes to the AireOS branch, based on testing against my home WLC, and merged it into the master branch.  @AirManic -- please let me know if you find any issues with your scripts after pulling master.

Will check out a bit later and let you know.  Sounds great.  .:|:.:|:. Gordon Rogier
From: Jamie Caesar<EMAIL_ADDRESS>Sent: Tuesday, May 26, 2020 3:06 PM
To: jamiecaesar/securecrt-tools<EMAIL_ADDRESS>Cc: Gordon Rogier (grogier<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [jamiecaesar/securecrt-tools] Added support for Cisco AireOS Wireless LAN Controllers (#49)
Made some fixes to the AireOS branch, based on testing against my home WLC, and merged it into the master branch. @AirManichttps://github.com/AirManic -- please let me know if you find any issues with your scripts after pulling master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/jamiecaesar/securecrt-tools/pull/49#issuecomment-634250229, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APBEUSBNXIO5FWPQ4UNMJMTRTQOJBANCNFSM4MYOYFQQ.


Verify Go Version

It would be helpful if the CLI could notify the user that they are potentially using an outdated version Go.
For example, in this lesson I spent a lot of time trying to debug why the tests were failing only to find a hint in the Discord that maybe my Go version was the issue. I was using 1.21 when upgrading it to 1.22.5 resolved the issue.
I think it would be beneficial to offer a command to check this, and then potentially update the lessons to guide learners to use this command.

Now the CLI forces people to upgrade for minor versions. Thank you for reporting this.


Add path

closes https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/3330

@mkbond007 fyi.

@mkbond007 PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


yafaray-core: switch to opencv4

Motivation for this change
opencv2 is essentially EOL and has security concerns
OpenCV is used for image denoising in yafaray-core
Tested this by rendering the extremely convenient, denoising-enabled test scene provided in the yafaray source https://github.com/YafaRay/Core/tree/master/tests/test01 , which incidentally would probably be fairly easy to turn into an installCheck in case anyone wanted to have a go - the only tricky thing would be to allow a tolerance of difference against the reference image, due to e.g. image noise.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @Hodapp87

Thanks for the patch @risicle, this was done on https://github.com/NixOS/nixpkgs/pull/86808 .


How do I dynamically register translated languages?Such as http params

[ ] I have looked at the documentation here first?
[ ] I have looked at the examples provided that may showcase my question here?

Package version eg. v9, v10:
v10
Issue, Question or Enhancement:
for example:
http params
{
transLanguage:"en"
}
Code sample, to showcase or reproduce:
//virtual code

https://github.com/go-playground/validator/blob/master/_examples/translations/main.go#L43-L45


App crashes after entering invalid string

Everything works great until you enter a bad mention 😞
Steps:

Go to the example page and type in @xxx or some other string that has no valid choices.
Press enter to select.

Result:

A new-line is created
A null is saved as a "mention"
The module stops working.
Every keystroke produces the following error:

angular.js:11655 TypeError: Cannot read property 'first' of undefined

Expected:

The module continues to work and @xxx is rendered as normal text with no new-line.

It seems there are several areas that need null checks for invalid entries.

@ProLoser i think, there is only one place to fix needed.

+1

+1

I am no longer active on this repo. @biggieman I added you as a collaborator so that you can resolve your outstanding PRs, although I would recommend considering my note on #41

+1

Please use reactions to +1 the original post instead of adding new comments

@ProLoser ok!

Thanks for fixing this so quickly @biggieman! Could you push a new version to bower? That would be great!

New release with last fixes will be soon.

Bump on the release please! I currently have my bower referencing your commit hash haha


Code highlighting in assert macro usage example makes blue color out of variable name "string".

Actual:

The text "string" is highlighted as blue for a keyword.

Expected:

"string" should be regular text color.

Maybe:

also "stdout", strlen, printf?

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

ID: 84710ecd-e868-8f06-7e95-7f68538a4f07
Version Independent ID: b737afb6-cb68-810d-5a99-b696f54d2640
Content: assert Macro, _assert, _wassert
Content Source: docs/c-runtime-library/reference/assert-macro-assert-wassert.md
Product: visual-cpp
Technology: cpp-ucrt
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

@TilKenneth
Thanks for the report. This defect exists in the third-party code colorizer used by Microsoft Docs, and unfortunately, the C++ docs team doesn't have any way to fix it from within the C++ docs repo. It's annoying, but I'm going to resolve this as something we can't fix. I'll try to pursue it internally with the team that maintains the Docs infrastructure. Sorry I don't have a better solution for you.


touch-action problem with Android browser

I have isolated a problem down to hammer-time on the stock Android browser. I mainly tested it on BrowserStack (HTC One M8 emulator) but can confirm it also happens on real HTC phones. Not sure if it affects the stock Android browser on other phones.
With just the below, tapping on the select box does not do anything. However, tapping on the space surrounding the select box triggers the native select dialog.
I'm not sure where to begin looking into this – does anyone have any ideas?
<!DOCTYPE html>
<html>
<head>
  <meta name="viewport" content="width=device-width, initial-scale=1">
  <meta charset="utf-8">
  <title>Hammer Time</title>
  <script src="hammer-time.js"></script>
</head>
<body>
  <select style="touch-action: manipulation;">
    <option value="1">One</option>
  </select>
</body>
</html>

Which version of Android Browser / Android?
On most android devices, hammer-time will be a no-op.

Many if not most devices using the stock browser and not chrome as default browser don't have touch-action.
The problem here is that you can't open a selectmenu with javascript except in chrome and Safari. Since hammer time uses a generated event it will not work on select elements. This should be in the docs / readme.

Thanks for the response! I'm ok with hammer-time being a no-op, however the e.preventDefault(); on dropHammer() is breaking default functionality, i.e. select menu not opened on tap
Or are we suggesting that hammer-time should not be used on select elements at all?
In which case, @runspired I'm actually facing this problem because of ember-hammertime automatically applying touch-action to select elements – I can open issue there if it's more appropriate

Sorry for the slow response yeah i dont think hammer-time should be used on selects at all really ill discuss this with @runspired.


Update Agora submodule to the latest commit

Includes changes related to Flash Layer. Tested locally and works.
Since there are more incoming merges, we could also keep this PR and push changes to match the merge.
Compatible with commit 382c3a

Can you pull in the latest change ?

Can you pull in the latest change ?

The latest changes have been pulled.
This needs a rebase though, because commons_budget is now a required field.

You can just pass PublicKey.init


Mock for testing

Is there any lib or code to use as mock for the call .ipcRendered.send('node-function', data)?

@tadeork you can use Observables to mock ipcRendered.send command.  Check out this gist, I hope it helps.

Hmm, 🤔 providing it in the library may work for that particular scenario buy I think the fake heavily depends on the requirement you've to meet...  I see potentially ending up in providing fake impl. for every electron API 😀
Basically, that was exactly the reason why I added the abstract base class.
However, if you come up with something generic in PR, I would be really thankful 👍


theta estimation

hi, thanks a lot for your work. I want to figure out whether the hard negative sampling implementation accords to the paper.

Hi, thanks for raising the concern. You may find that we have a class namely Efficient_Hard_negative_generator() in the FNE.py file. Given that there is almost no duplicated products appearing within a batch, the summation operation of Eq. (6) can be omitted for the efficiency purpose without any tradeoff of effectiveness. Therefore, you can regard the Efficient_Hard_negative_generator() function as the version we would like to recommend, even though the implementation is slightly different.


Tasks category ordering

Requirement

Have Active category first and keep it open
Order doesn't matter for the rest of the categories
Keep the rest of the categories closed

I would like to take this issue


Error: Subscription class not found: "ChatChannel"

Describe the bug
I followed the installation to set up:

yarn add actioncable-vue
app.js

import ActionCableVue from 'actioncable-vue';

Vue.use(ActionCableVue, {
  debug: true,
  debugLevel: 'all',
  connectionUrl: 'ws://localhost:3000/cable',
  connectImmediately: true,
});

Live.vue component

export default {
  name: "Live",
  components: {
    Header
  },
  data() {
    return {
      message: "hello"
    };
  },
  mounted() {
    this.$cable.subscribe({
      channel: "ChatChannel",
      room: "public"
    });
    this.sendMessage();
  },
  channels: {
    ChatChannel: {
      connected() {
        console.log("--- connected");
      },
      rejected() {
        console.log("--- rejected");
      },
      received(data) {
        console.log("--- received");
      },
      disconnected() {
        console.log("--- disconnected");
      }
    }
  },
  methods: {
    sendMessage: function() {
      this.$cable.perform({
        channel: "ChatChannel",
        action: "send_message",
        data: {
          content: this.message
        }
      });
    }
  }
};

Result =>

On Browser console:

Performing action 'send_message' on channel 'ChatChannel'.
Performed 'send_message' on channel 'ChatChannel'.

On backend logs

Finished "/cable/" [WebSocket] for <IP_ADDRESS> at 2020-03-19 12:59:59 +0800
Started GET "/cable" for <IP_ADDRESS> at 2020-03-19 12:59:59 +0800
Started GET "/cable/" [WebSocket] for <IP_ADDRESS> at 2020-03-19 12:59:59 +0800
Successfully upgraded to WebSocket (REQUEST_METHOD: GET, HTTP_CONNECTION: Upgrade, HTTP_UPGRADE: websocket)
Subscription class not found: "ChatChannel"

Expected behaviour
I should see the logs on the console:
--- connected
--- received

Edit
My url is: http://localhost:3000/live
router.js:
...
{ path: '/live', name: 'Live', component: Live }
....

Can I see the ChatChannelclass from your backend?

Can I see the ChatChannel class from your backend?

Hi, thanks for your checking.
I didn't know that I need to create a ChatChannel class. Can you teach me how to do it? Where should I store the class?

Kindly follow this tutorial > https://edgeguides.rubyonrails.org/action_cable_overview.html. It'll explain how ActionCable works.
ActionCable-Vue is just a simple Vue wrapper that makes working with your ActionCable server much more delightful. You'll still have to do the work to set up your server to correctly respond to connections and messages.

It works well. Thank you very much. Very awesome and simple to use. <3


Add order type & status ID mappings in Python toolbox modules

Expected Behavior
Be able to have an accurate mapping for missing order related status and type mappings in pure python
Potential Solution
Add a new order.py module file that contains these mappings and future order related code
Checklist

[X] I have completely filled out this template
[X] I have confirmed that this issue exists on the current master branch
[X] I have confirmed that this is not a duplicate issue by searching issues

I don't know exactly what functionality you need, but it may already be available (using OrderStatus as example):

We can use Extensions.GetEnumString(OrderStatus.UpdateSubmitted, OrderStatus) and Enum.Format(OrderStatus, OrderStatus.UpdateSubmitted, "g") to convert enum ints to strings.
We can use Enum.GetValues(OrderStatus) to get the int values of all the constants in an enum.
We can use Enum.GetNames(OrderStatus) to get the string names of all the constants in an enum.

If you need something else, chances are the Enum class in C# has a method for it.

I am looking for similar mappings for a pure python (not .net python interpreter) environment to process results data. The PythonToolbox right now is written in such a way that the QA modules can be used outside of a .Net or QC environment.


[DRAFT] TF change antialias to true

This PR:

changes antialias also for TF to True

NOTE tests:

super minimal difference (< 1e-6) in the expected output  (e.g.: 5.00001 instead of 5.0  /  1.00002 instead of 1.0) -
I don't think there's much we can do about that (the results are correct and close to PT see below)

10 x abs
both with antialiasing:
opencv_tf_pil_tf_diff:  108826.1
opencv_torch_pil_torch_diff:  1859450
opencv_tf_opencv_torch_diff:  6922.742
pil_torch_pil_tf_diff:  7202.461

only pt with antialiasing:
opencv_tf_pil_tf_diff:  428563.7
opencv_torch_pil_torch_diff:  1859450
opencv_tf_opencv_torch_diff:  470293.66
pil_torch_pil_tf_diff:  183107.75

observation over 30 images:
straight pages: no visible changes
rotated pages: narrow text is detected somewhat less well (should be fixed if we do the next run with rotated augmentation)
@charlesmindee @odulcy-mindee wdyt ?

@felixdittrich92 I'm not sure to understand the benchmark. Why do you compare tf and torch with PIL and OpenCV ? Is there also antialiasing enabled by default in these frameworks when you downsize an image ?

@felixdittrich92 I'm not sure to understand the benchmark. Why do you compare tf and torch with PIL and OpenCV ? Is there also antialiasing enabled by default in these frameworks when you downsize an image ?

Correct after this PR we use in TF and PT antialias so the images are now much more close also for training runs

Add the end what we want is to work on the same images in both frameworks in the whole pipe inference and training
and with antialias it's the closest we can get atm (in pytorch we use antialising already since 0.7.0)


use maps instead of structs to be able to push zero values and send only fields that are actually configured

Jira task -
Use maps instead of structs to be able to push zero values and send only fields that are actually configured
Release Notes Description (public)

Implementation details (internal)

Information for QA

[ ] Is QA testing required?
[ ] Does PR contain unit tests?
[ ] Should QA create E2E tests for the change?

Additional QA Procedures (Optional)

Screenshots (if appropriate):

I would consider using IMPORT instead of PATCH operation to avoid nondeterministic behavior

As discussed, both methods have their pros and cons, so it is configurable now.


Wrangle the data in preparation for machine learning

Our data currently covers all of NSW, but say that our client wants us to create a machine learning model to predict rainfall over Sydney only. There's a bit of wrangling that needs to be done for that:
We need to query our data for only the rows that contain information covering Sydney
We need to wrangle our data into a format suitable for training a machine learning model. That will require pivoting, resampling, grouping, etc.
To train an ML algorithm we need it to look like this:

| model-1_rainfall | model-2_rainfall | model-3_rainfall | ... | observed_rainfall
-- | -- | -- | -- | -- | --
0 | 0.12 | 0.43 | 0.35 | ... | 0.31
1 | 1.22 | 0.91 | 1.68 | ... | 1.34
2 | 0.68 | 0.29 | 0.41 | ... | 0.57

Get the data from s3 (combined_model_data_parti.parquet and observed_daily_rainfall_SYD.csv)
6.2) First query for Sydney data and then drop the lat and lon columns (we don't need them).
syd_lat = -33.86
syd_lon = 151.21
Expected shape (1150049, 2).
6.3) Save this processed file to s3 for later use:
Save as a csv file ml_data_SYD.csv to s3://mds-s3-xxx/output/ expected shape (46020,26) - This includes all the models as columns and also adding additional column Observed loaded from observed_daily_rainfall_SYD.csv from s3.
This should be done in the appropriate section of the milestone 2 ipynb

Done! https://s3.console.aws.amazon.com/s3/object/mds-s3-22?region=us-west-2&prefix=output/ml_data_SYD.csv


Update README to latest state

PR Description
Update README:

update descriptions

add latest events

update links

remove unused links

[x] I thought about documentation and added the documentation label to this PR if updates are required.

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


Sound Not Working

Hi .
I have implement the StandardPlayer like as in SimplePlayer but video sound not working
My dependencies
implementation 'com.shuyu:gsyVideoPlayer-java:7.1.6'
implementation 'com.shuyu:gsyVideoPlayer-armv7a:7.1.6'

any solution for this?
thanks

Could you provide a  test video link？


Assorted bugfixes in the ICache UVM test code

These fix various bugs that I ran into when trying to get the scoreboard working properly. The most interesting patch is probably the second: it turns out that my previous plan to spot fetch requests is fragile and doesn't take the non-determinacy of delta-cycle scheduling into account. Oops!

Well, it's a less impressive catch if you know that I spent half an hour staring at uvm_info traces going "huh?!", but at least it works now :-)


Taz Wanted Character models fail to render

Describe the bug

Character models do not render.
remix-0.4.1-release
RTX 4090
GPU driver version: 551.61
Game version 1.0 installed off an official retail CD using Elishacloud's directx wrapper: https://github.com/elishacloud/dxwrapper

Attach files!
Log and Config FIles.zip
To Reproduce
Steps to reproduce the behavior:

launch the game
skip the intro video's
the character models in the opening cutscene will not render (also the game will shortly crash if you let the cutscene run on too long but I will make separate bug report for this issue)

Expected behavior

The character models should be visible

Screenshots
-Remixed screenshots:

Rasterized screenshot:

Thanks for posting. For bring up of RTX Remix with a particular game I would recommend reaching out to the community discord server https://discord.gg/rtxremix

Hi @OmenFox, I've noticed that in some games player/actor models are disappear because they write outside of the buffer bounds that they claim. Can you please open the bridge.conf in the .trex folder and change # alwaysCopyEntireStaticBuffer = False to alwaysCopyEntireStaticBuffer = True, and then retry to reproduce the issue? If bridge.conf doesn't already exist, take the example bridge.conf from the bridge repository here: https://github.com/NVIDIAGameWorks/bridge-remix/blob/main/bridge.conf

Hello! Will you please test this on the latest release and let us know if the issue persists? Thank you!

Hello i have tested the newest runtime and have a few things i can report back that i have noticed:
The character models still do not render and are not selectable in the texture's menu but i have managed to select the skybox this time so i can show much better examples of what i am trying to illustrate:

Here is taz's model traveling to the spot he stands and it renders the textures of the spinning but not the arms and under it you can see a shadow texture poorly rendering.

And here is where he should be standing.
Another strange behaviour to note is when these character models should be rendering they lower the framerate by almost half.

Ok it just black screen crash's on start up, it shows the remix overlay telling you to press Alt+X for a second then just crash's to desktop i cannot get ingame at all anymore.

Thank you for confirming. Will you please attach logs and any dump files from an attempt to run the game with 0.5.1?

Yes of course here they are:
Remix 0.5.1 TazWanted First launch Log.zip

Thank you for the update! We've created REMIX-3372 for tracking.


Puzzle mode should have a pause menu

When solving a puzzle, the 'esc' key currently ends the puzzle, but there is no way to pause.
There should be a way to bring up a menu which interrupts the puzzle and gives the player a few options:

Give up
Continue
Quit

'Quit' should be disabled during the initial tutorial.
'Give up' should be disabled if the player isn't in the middle of a puzzle. Actually, the menu probably shouldn't even be pop-uppable if they're not in a puzzle, since it's completely redundant with the puzzle's hud.

When working this issue, we should also be careful about cheaters. The pieces and next queue should be hidden when the player pauses. The player should not have control for 0.5s after unpausing.

Duplicate of #123


changed update task to use email instead of pi name

https://www.pivotaltracker.com/story/show/185031869

@amcates could you push this up and re-run the update_use_department task?


STUCK/MISSING FUNDS AFTER TRYING TO ADD LIQUIDITY

I tried adding 1.98 of BNB to the BNB.RUNE pool.. going 50/50 with Rune. The transaction didn't go through however half of my BNB per the dex explorer has been sent to an unknown address.. and the other half is still in my wallet. Are my funds missing? i also cannot tell if the Rune side of the transaction went through.
It is worth to note that i tried withdrawing the other half in my wallet but that didn't go through
Tx hash--D5F057D28863849DB6EA3ECA74E3D144931D740B3B6E4EA243A616223155AEF6
address my bnb was sent to--bnb1ksxqxurvejkndenuv0alqawpr3e4vtqk855aal

Your funds are SAFE.
https://thornode.thorchain.info/thorchain/tx/D5F057D28863849DB6EA3ECA74E3D144931D740B3B6E4EA243A616223155AEF6
Team will get back to you with the resolution. Thanks.

has been sent to an unknown address
It has been send to a pool address.

Are my funds missing?
No, your funds are still safe, but set as "pending". You can check it here by looking for your address and state of "pending_asset".

Your issue is similar to #1334, where you will find a workaround to fix it manual. We've already started with the team to fix it in ASGARDEX desktop - sorry for the trouble!

https://thornode.thorchain.info/thorchain/tx/D5F057D28863849DB6EA3ECA74E3D144931D740B3B6E4EA243A616223155AEF6
THOR: https://viewblock.io/thorchain/address/thor1g489e8n3894paumdfntr06r3lzucln8srdfwjv
BNB txn: https://explorer.binance.org/tx/D5F057D28863849DB6EA3ECA74E3D144931D740B3B6E4EA243A616223155AEF6
According to tx history, You tried symm deposit for BNB pool but rune tx didn’t go out.
According to the memo ADD:BNB BNB:thor1g489e8n3894paumdfntr06r3lzucln8srdfwjv, Add tx is still in pending until RUNE is added symmetrically.
Potential Resolutions:

You should deposit rune with proper memo - ADD:BNB BNB:bnb1gzvnmk8auvmxj9fjz93qvqgf6w4q0swqksqvlq.
You should send withdraw tx.

Help wanted? Join discord (https://discord.gg/ZJHtzGJB)

for clarity when you say I should deposit rune symmetrically  does that
mean i should add just Rune usd equivalent of what is pending in the BNB
pool??
Thus adding liquidity to the BNB pool but just adding Rune equivalent of
what is already in the pool i.e BNB??
Thanks for the asistance i know you guys are busy thanks for the support
Kind Regards
On Thu, 15 Apr 2021 at 11:39, Chris @.***> wrote:

https://thornode.thorchain.info/thorchain/tx/D5F057D28863849DB6EA3ECA74E3D144931D740B3B6E4EA243A616223155AEF6
THOR:
https://viewblock.io/thorchain/address/thor1g489e8n3894paumdfntr06r3lzucln8srdfwjv
BNB txn:
https://explorer.binance.org/tx/D5F057D28863849DB6EA3ECA74E3D144931D740B3B6E4EA243A616223155AEF6
According to tx history, You tried symm deposit for BNB pool but rune tx
didn’t go out.
According to the memo ADD:BNB
BNB:thor1g489e8n3894paumdfntr06r3lzucln8srdfwjv, Add tx is still in pending
until RUNE is added symmetrically.
Potential Resolutions:

You should deposit rune with proper memo - ADD:BNB
BNB:bnb1gzvnmk8auvmxj9fjz93qvqgf6w4q0swqksqvlq.
You should send withdraw tx.

Help wanted? Join discord (https://discord.gg/ZJHtzGJB)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thorchain/asgardex-electron/issues/1343#issuecomment-820356480,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATVYZZDHWRUZL3SHDUVFLYLTI3F7NANCNFSM4266DUFQ
.

Thus adding liquidity to the BNB pool but just adding Rune equivalent of
what is already in the pool i.e BNB??

@mastosis Yes, but you do need to do it by adding RUNE asymmetrical to the BNB pool. An asymmetrical deposit is currently not supported in ASGARDEX desktop. However, there is a workaround for doing it (similar to a asymmetrical deposit). Or just jump to https://discord.gg/ZJHtzGJB to get some support for doing that.

Thanks Man
I have been able to sort it out
Cheers
Sent from my iPhone

On 15 Apr 2021, at 14:45, veado @.***> wrote:

Thus adding liquidity to the BNB pool but just adding Rune equivalent of
what is already in the pool i.e BNB??
@mastosis Yes, but you do need to do it by adding RUNE asymmetrical to the BNB pool. An asymmetrical deposit is currently not supported in ASGARDEX desktop. However, there is a workaround for doing it (similar to a asymmetrical deposit). Or just jump to https://discord.gg/ZJHtzGJB to get some support for doing that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Better handling of (de)serialization of circular and forward references with natural keys

This PR addresses #26552 and #31051, both of which address problems with (de)serializing data involving forward or circular references and natural keys.

Thanks for your quick review. I've responded inline to everything, most things I will need to change. I'm still hammering out the last few kinks to keep the full testsuite working, once I manage that and fixed your remarks, I'll update the PR.

In the meanwhile, I was writing some more considerations wrt this PR. I was planning on posting these after running the full testsuite, but since I ran into some problems and people started reviewing, I'll post this comment already.
With regard to the actual fixes, I am pretty confident that these are ok. The only issue is about backward compatibility for django.serializers.core.sort_dependencies(). This function used to throw an exception when a circular dependency involving natural keys was present (e.g. when it could not produce a perfectly valid ordering), but now also allows doing a best-effort sorting while still allowing cyclic dependencies using a allow_loops argument. This argument defaults to False, for compatibility with existing callers. However, with this PR, only one caller is present within django, that sets allow_loop to True. I guess the bottomline is: Is this functoin considered public API and should it keep compatibility? The function is not mentioned in the docs AFAICS.
With regard to the testcases, I think they work but there is some room for improvement.
In particular, with the added models to the fixtures tests, when the problem that is tested for occurs, it is likely that this problem occurs in other testcases too (since part of the problem prevents serializing of any app containing a model, not even an instance, with circular references with natural keys). This is just a minor problem, when the bug does come back, it will certainly be detected by some test(s). It could be fixed by moving these models into a separate app that is only used by this particular test, but that seems a hassle for little gain.
With regard to the new test_runner test, it re-initialized the default database, which could slightly influence other tests (in practice, it should leave a flushed database with some extra data in the serialized copy stored on the database connection, which should be ok?).
I tried using a mock for django.utils.connections (as seen elsewhere) to make the testrunner use a new, separate, database, but that seemed to properly call create_test_db for the mocked database, but then the subsequent migration data would not show up in the serialized contents, leading me to believe that the migrate done by create_test_db happened on the original default database (since that uses db.connections). I then tried mocking django.db.connections instead, leading to a no such table: django_migrations. Then mocking both lead to database table is locked, so I'm not quite sure what the right approach is here...
Here's what I tried (in an older version of the test, but I think the same should hold for the test as it is now):
{{{
class FooTest(unittest.TestCase):
def test_setup_databases(self):
tested_connections = db.ConnectionHandler({
'default': {
'ENGINE': 'django.db.backends.sqlite3',
'NAME': ':memory:',
},
})
with mock.patch('django.db.connections', new=tested_connections):
with mock.patch('django.test.utils.connections', new=tested_connections):
runner_instance = DiscoverRunner(verbosity=0)
old_config = runner_instance.setup_databases()
runner_instance.teardown_databases(old_config)
print(tested_connections['default']._test_serialized_contents)
}}}
Any pointers on how to approach this, or is there perhaps an existing test that does something similar that I overlooked?

With regard to the new test_runner test, it re-initialized the default database, which could slightly influence other tests, which might be harmless enough. However, on the postgres, postgis and mysql backends, this tries to recreate the same database, giving e.g. psycopg2.errors.DuplicateDatabase: database "test_django_38_pull-requests-bionic" already exists (see djangoci output).
I tried using a mock for django.utils.connections (as seen elsewhere) to make the testrunner use a new, separate, database

Thinking a bit more about this, I guess it will not be generally possible to actually create a new database. For sqlite, creating a new file or in-memory database could work, but for external database servers, there is no guarantee that you actually have permission to create another database (and if so, what name that should have).
When running multiple parallel testrunners, cloning already makes additional databases, but those have predictable names and if the test environment does not support additional databases, you can disable that with --parallel=1.
An alternative fix for this would be to pass keepdb to the test runner, so it would not error out on an existing database, but instead just reuse it. I've just force pushed this (along with fixes to most of @charettes comments) and the first mysql tests seem to have passed, so that's promising, but postgres is still failing with "psycopg2.errors.UndefinedTable: relation "postgres_tests_testjsonmodel" does not exist
LINE 1: ...gres_tests_testjsonmodel"."jsonmodel_ptr_id" FROM "postgres_..." for some reason...

I had another look at this and could reproduce the JSON-related error locally. What I suspect happens is that some database content from earlier tests remains in the database, which then causes a problem with the serialization in CircularRefTestRunnerTest, or something like that.
What I also noticed, is that both Postgresql and Mysql locally try to access the test_test_django_main database, so the test_ prefix is added twice (probably once by the main test runner and once by the test runner inside CircularRefTestRunnerTest).
In other words, reusing the same database is tricky as well. This leads to another solution to this problem: Add a third database to the test settings. IOW, next to the "default" and "other" database, add e.g. a "testrunnertest" database (name up for suggestions). This database would not be declared in the databases=... of CircularRefTestRunnerTest, so it will not be automatically created on startup of the testrunner, leaving it free to be created from scratch and teared down by CircularRefTestRunnerTest (and maybe other tests that want to test database creation and/or teardown). Of course, this requires changes to the test settings modules used by people, so this test must only run if such a database is actually available.
I have not tried setting this up yet, but how does this sound?

''
'
Le mer. 18 déc. 2019 à 21:35, Matthijs Kooijman<EMAIL_ADDRESS>a écrit :

I had another look at this and could reproduce the JSON-related error
locally. What I suspect happens is that some database content from earlier
tests remains in the database, which then causes a problem with the
serialization in CircularRefTestRunnerTest, or something like that.
What I also noticed, is that both Postgresql and Mysql locally try to
access the test_test_django_main database, so the test_ prefix is added
twice (probably once by the main test runner and once by the test runner
inside CircularRefTestRunnerTest).
In other words, reusing the same database is tricky as well. This leads to
another solution to this problem: Add a third database to the test
settings. IOW, next to the "default" and "other" database, add e.g. a
"testrunnertest" database (name up for suggestions). This database would
not be declared in the databases=... of CircularRefTestRunnerTest, so
it will not be automatically created on startup of the testrunner, leaving
it free to be created from scratch and teared down by
CircularRefTestRunnerTest (and maybe other tests that want to test
database creation and/or teardown). Of course, this requires changes to the
test settings modules used by people, so this test must only run if such a
database is actually available.
I have not tried setting this up yet, but how does this sound?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/12166?email_source=notifications&email_token=AACUDLMU3Y627NPB7RGK6G3QZKCSRA5CNFSM4JTOTSNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHHNEVI#issuecomment-567202389,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACUDLLVMLTMD5JTHJME33TQZKCSRANCNFSM4JTOTSNA
.

LM
nkwkwkwkxk‹‹'f''g'f'f'f'jjbbbbb
Bnjjjlkklkkkkk
Le mer. 18 déc. 2019 à 21:35, Matthijs Kooijman<EMAIL_ADDRESS>a écrit :

I had another look at this and could reproduce the JSON-related error
locally. What I suspect happens is that some database content from earlier
tests remains in the database, which then causes a problem with the
serialization in CircularRefTestRunnerTest, or something like that.
What I also noticed, is that both Postgresql and Mysql locally try to
access the test_test_django_main database, so the test_ prefix is added
twice (probably once by the main test runner and once by the test runner
inside CircularRefTestRunnerTest).
In other words, reusing the same database is tricky as well. This leads to
another solution to this problem: Add a third database to the test
settings. IOW, next to the "default" and "other" database, add e.g. a
"testrunnertest" database (name up for suggestions). This database would
not be declared in the databases=... of CircularRefTestRunnerTest, so
it will not be automatically created on startup of the testrunner, leaving
it free to be created from scratch and teared down by
CircularRefTestRunnerTest (and maybe other tests that want to test
database creation and/or teardown). Of course, this requires changes to the
test settings modules used by people, so this test must only run if such a
database is actually available.
I have not tried setting this up yet, but how does this sound?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/12166?email_source=notifications&email_token=AACUDLMU3Y627NPB7RGK6G3QZKCSRA5CNFSM4JTOTSNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHHNEVI#issuecomment-567202389,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACUDLLVMLTMD5JTHJME33TQZKCSRANCNFSM4JTOTSNA
.

@felixxm good suggestion, I created #12453 for that. It includes the fix commit from this PR, and I added a new testcase, since the testcase I had only worked when both issues were fixed. I've also rebased this PR on top of master and to include that new PR, but let's focus on the other PR first.

Not related to the latest changes but I wonder how much time is actually spent by deserialize_db_from_string to perform the JSON deserialization given data = serializers.serialize('python', get_objects()) could be used to store the object representation while avoiding the continued round trip.

@charettes I'm not sure what you mean there? Are you suggesting to change the implementation of serialize_db_to_string to use python serialization instead of JSON as a performance improvement? Or is this about this PR directly?

@matthijskooijman it isn't directly about this PR.

Are you suggesting to change the implementation of serialize_db_to_string to use python serialization instead of JSON as a performance improvement?

I was wondering if that would make a difference. It's surely faster to operate from a list of Python object than converting a string to it before hand. The question is how significant that is given all the Model.save involved anyway.

I was wondering if that would make a difference.

Hm, I'm not so sure. I guess it also depends on the size of the data - AFAIU this is only about data generated during migration and post-migrated, so usually just contenttypes and permissions, I think.

@felixxm Thanks for merging. I rebased the fix for ticket-31051 on top of your improvements that are now in master, fixing some trivial conflicts and simplifying the testcase a bit (it also needs no new connection object AFAICS). I pushed that to a separate PR for clarity (#12458), making this PR obsolete.


Share NUMFocus balance sheet

There is a balance sheet for Astropy that NUMFocus produces and this should be shared to the finance committee with an idea of how this can be shared with the wider community community

This has been done.


Remove email from AuthConfig

This change is part of a larger PR on docker engine that will remove email from login, and remove the ability to register from the docker cli.
Related PR: https://github.com/docker/docker/pull/20565
Signed-off-by: Ken Cochrane<EMAIL_ADDRESS>

LGTM 🐰
/ping @calavera

this makes me very happy! :tada:
LGTM

this is breaking go get for ppl using docker and engine-api though since docker still use Email (but this isn't vendored)

@runcom I think @dmcgowan is going to add it back in, with an omitempty flag, to fix backwards compatibility issues and such.

@kencochrane alright, that's correct


Hexdump returns error even when successful

If the hexdump command succeeds, it'll still end up returning an error as the success case will end up falling through to filesystem_error1: where it gets remapped to SJA1105_ERR_FILESYSTEM before returning.
https://github.com/openil/sja1105-tool/blob/085d42c72c4e9508aa7bc9bfbbb5fe41082fcdec/src/tool/staging-area.c#L122-L132
As a result, the sja1105-init startup script will always find the config to be invalid and create it anew:
https://github.com/openil/sja1105-tool/blob/085d42c72c4e9508aa7bc9bfbbb5fe41082fcdec/etc/sja1105-init#L25-L28

Hi,
It looks like you found a bug. Would you mind sending a pull request with a patch?
Thanks,
Vladimir


docs: Add information about autotools

This change is

LGTM

As discussed in #11 - maybe this PR is a good moment to introduce information about how to setup GOPATH and directories?

Is it still needed after recent fixes? @nhlfr @jellonek

nope


fix: Remove null element from the list of social links

Checklist

[x] I have read the Contribution & Best practices Guide.
[x] My branch is up-to-date with the Upstream development branch.
[x] I have added necessary documentation (if appropriate)

Short description of what this resolves:
Deserializer while deserializing event json appends null in the social-links which was not handled while working on each link object.
Changes proposed in this pull request:

Use filter function to remove null elements from the array of social-links

Fixes #2389

.filter(Boolean) should work.
Please share which social link has null, it shouldn't be possible on the server

@iamareebjamal actually the problem is in the backend api, it returns 4 objects in relationships['social-links']['data'] out of which 3 objects are only returned in the included, to be specific social-link with id 805 is missing. Right now I have handled the case on the frontend by filtering the social-links array. Let me know if this thing needs to be corrected on the backend?
You can refer the relationships below

and included key below doesn't contain object with id 805

But why does it matter, just use what included is returning


Rename this constant name to match the regular expression '^[A-Z][A-Z0-9](_[A-Z0-9]+)$'.

Description:
The constant named VARIABLE_NAME does not follow the established naming convention of ^[A-Z][A-Z0-9](_[A-Z0-9]+)$. This inconsistency reduces code readability and maintainability.
File Location: src/main/java/de/storchp/opentracks/osmplugin/dashboardapi/TrackPoint.java
Impact:
Reduced code readability and maintainability for developers unfamiliar with the specific naming convention.
Increased potential for future bugs and confusion due to inconsistent naming.
Difficulty in automating certain code management tasks due to non-standard naming.
Proposed Solution:
Rename the constant to NEW_VARIABLE_NAME which adheres to the standard naming convention.

Technical information
-Device: Emulator - Pixel 6a
-OS: Android 14 - upside down cake
-OpenTracks version: v4.0.0

Issue:Rename this constant name to match the regular expression '^[A-Z]A-Z…
…0-9$' https://github.com/HWJFish/OSMDashboard-Winter-2024-SOEN-6431/issues/1
Impact: Reduced code readability and maintainability.
Solution: Rename to ID (as per standard convention).
Benefit: Improved clarity and reduced confusion.


[5.4] Fix SessionGuard.php recaller method when request is null

recaller() method throws a non-object exception when $request is null.  Null appears to be allowed behavior for $request as the constructor default is null.  recaller() method is called by the user() method on line 140 the response is then checked for null on line 142.  I believe in current form the recaller() method will never return null if $request is null.

How did you create this error in a fresh application?

Cleaned it up a bit.

@taylorotwell use of the Auth Facade, specifically Auth::user() in a blade template which is accessible to both authenticated users and guest.  My user model has a group attribute.  When evaluating content for conditional display based on the user group @if(Auth::user()->group == 1), if the user is guest then I believe the request will be null.
This is probably better handled with a user check in the controller but as long as the Facade is accessible in the blade template I believe this scenario would be technically possible.
For quick reproduction, in a fresh install with a user created and logged out a call to Auth::user() in welcome.blade.php:
@if(Auth::user()->id == 1)
Hello World
 @endif


New config item for seed

RIght now seed files won't run in production.  This was good for awhile but we want to rethink this, so we can run the seed command in staging.

This is done


How to test org.takes.ts.fork.Target implementation?

I'm faced with a problem trying to test this code
public final class TkGetFile implements Target<RqRegex> {
...
    @Override
    public Take route(final RqRegex req)
        throws IOException {
        return new TkHTML(
            this.storage.get(
                req.matcher().group("path")
            )
        );
    }
}

In test to call route I need to provide a RqRegex. Unfortunately, I can't mock it, because RqRegex.matcher() returns java.util.regex.Matcher which is final and can't be mocked.
So, the only way I've found is to create real Matcher (yees, using Pattern) but this is really overkill :(
Is there any other possible way?

Well, there are two options to test this class. First one:
final RqRegex req = new RqRegex() {
  @Override
  public Matcher matcher() {
    return Pattern.compile("(?<path>.*)").match("/a/b/c.txt");
  }
};
final Take take = new TkGetFile().route(new RqRegex());
// your test code

Second one:
final Take take = new TsFork(
  new FkRegex("(?<path>.*)", new TkGetFile())
).route(new RqFake("GET", "/a/b/c.txt"));
// your test code

Whichever you prefer :)

The first approach is not very suitable, as we also have to implement 2 methods from Request interface :(
Second one is also not a solution, because Take.route() expects RqRegex, not a Request. So, RqFake can't be used there ...

Take.route() can't expect anything except Request... what do you mean?

My fault. I'm talking about Target.route(), not Take.route()
Please, take a look at the code in the issue description.

@longtimeago yes, I noticed that. that's why I'm suggesting to wrap it into FkRegex for this test:
final Take take = new TsFork(
  new FkRegex("(?<path>.*)", new TkGetFile())
).route(new RqFake("GET", "/a/b/c.txt"));
// your test code

FkRegex will convert your RqFake into RqRegex required by TkGetFile

@yegor256 looks like overkill ... anyway, thanks!


Add ROR ID for Funder

use BaseName to add a ROR ID for Funding agencies in data description schema

@Sun-flow this might be worth doing as we're fixing the funding information for the U19 data.

done alongside U19 data, closes with that PR


Add block reward destination

Have you considered separating out the sighash serialization from the actual hashing?

It occurred to me. But the performance hit is not really worth it. No need to do extra allocations. Unless I misunderstand what you're envisioning.


Improve layout of 404 error page

Is your feature request related to a problem?
When visiting a non-existing link within stdlib.safeds.com, such as https://stdlib.safeds.com/en/stable/thispagecertainlydoesnotexist, one gets shown a 404 error page with a quite messed up layout. Images are way too large, and it looks like no CSS is loaded.
Desired solution
Create a proper 404 error page.

Is this actually fixed?
I still get the strange layout when visiting the above link, both on a normal monitor with firefox and on my (android) smartphone with chrome.

:tada: This issue has been resolved in version 0.15.0 :tada:
The release is available on:

v0.15.0
GitHub release

Your semantic-release bot :package::rocket:


Intercepting the Graphql requests and responses

In Apollo, I think we can use Apollo Link to intercept the requests and responses.
Does graphql-request provide this feature out-of-the-box? Looks like not.. could anyone pl. suggest the right way to accomplish this when we graphql-request?
Thanks

There is middleware now: https://github.com/jasonkuhrt/graphql-request/blob/main/examples/other-middleware.ts.


Vim9: strict white space check is not developer friendly

Problems
Currently (v8.2.2029), Vim9 checks white spaces strictly. It enforces white
space everywhere; around binary operators, assignment operators, function
arguments, dictionary keys and comment symbol. However, this smells like an
ancient language and is frustration for me. Most modern languages have lexers
and skips white spaces while scanning the source code (indentations are
exception; I couldn't find a language confiders spaces between tokens, maybe
TeX is one of them, but legacy language anyway). Enforcing white spaces is
not developer friendly and cannot be considered as a modern language IMO.
Some languages have linters or formatters which defines the number of white
spaces, but in language specification, white spaces are optional between tokens.
Some language formatters prefers omitting spaces due to precedence; a*b + c*d.
Some people like spaces all the way, and others don't. Formatters often suggest
the standard style for that language, but they fix valid code to valid code.
Specification changes suggestion

Allow omitting white spaces around binary operators (+, -, .., *, /,
%, ==, ==#, >, =~, etc) and assignment operators (=, +=, ..=,
=<< etc).
Allow omitting white spaces after commas in function arguments (both :def
and :call), arrays and dictionaries.
Keep enforcing spaces after : in dictionaries and around # for comments.

: conflicts with scopes and # conflicts with autoload function names.

White space is very useful to make it easy to read back code.  Many projects enforce usage of white space with linters and other tools.  This has nothing to do with lexers.  In fact, very old languages such as C did not enforce white space, leading to obfuscated code contests.
We don't have a separate tool to check the format of Vim script, and including the checks in the interpreter works well.  We can debate over where exactly white space should be optional, but this very quickly becomes a personal preference. And it depends on what code you are writing.  I have seen enough ugly Vim script that I tend to want more white space.  And I like that when the script runs it means it won't be too ugly.
Using single character variables seems uncommon, your example of "ab + cd" is not something I often see in Vim script.  When we would allow this we would also allow "somenamesomelist[index]+verylongnamethedict.member" and you can see that doesn't read easily.  If you want to indicate precedence you can add parens or use double spacing, but I already find "a * b + c * d" quite readable.

I agree your opinion about code without spaces been ugly, but I don't argue about readability. I don't say a*b + c*d is often appears in Vim script; I also think it's uncommon. But some language formatters prefers this style and thus some people like this style.
Making a code readable for humans and writing a code for interpreters/compilers are totally different things. And there are common understandings in most programmers on how modern language parsers handles white spaces. Language parser checking white spaces strictly is surprising for many programmers including me. Making a code readable is a task for linters and formatters, not enforced by parsers.

This is a slightly old thread and closed due to no response from Bram, but recently I've seen some complaint on this topic in Japanese Vim community so reopened. If anyone agree on making white spaces in Vim9 script optional, please comment below with reasonable explanation. Thank.


Add CRYSTAL_CONFIG_CC compiler config

This environment variable allows to bake into the compiler a default path for CC pointing to a specific linker.
Currently, the fixed default default is cc (cl.exe on Windows). Changing this when compiling the compiler allows taking into account a linker designed for a specific build environment. An example for this would be homebrew, where the compiler should use the linker installed in homebrew because it knows where to find libraries installed via homebrew.
An alternative would be patching the source when necessary, but that's more fragile. An environment variable is more reliable. It serves as a reference that this value can be configured.
Originally suggested in https://forum.crystal-lang.org/t/crystal-installation-using-linuxbrew-is-not-working/6559/10?u=straight-shoota

I'm wondering about the discoverability of this value. IMO crystal build --verbose shouldn't be the only way to find out the default compiler / linker

It should probably be in the output of crystal env. But then it should be the actual value, taking runtime configuration (i.e. CC) into account.
That's however not trivial due to the complexity of Compiler#linker_command. We should think about how to extract the static linker information.

For compiler developers, there's a catch: This approach doesn't work when using the makefile because it defines $CC and thus overrides the compiler default.
https://github.com/crystal-lang/crystal/blob/2b0b50602b9aeffc509cae053b109d751ed1092a/Makefile#L51


Adding square icon and iOS assets

All necessary assets in the media/iOS directory.

Thanks @willyg302!  Do you have any interest in also wiring the logo through the Rails app, like on the dashboard page?

Sure I could take a look at it!

Thanks!  I'll do the apple-touch-icon part.

Just pushed #835.

Login page design WIP is below, please let me know your thoughts.
Also for the main dash, do you want to keep the quote and image of Odin or remove?

Nice!  Maybe more padding above and below the image, and it could be a bit smaller.  I'm open to changes on the main dash too, if you have ideas.


Resource.Add doesn't add a new resource

This test fails in 2.2.18:
[Test]
public void TestAddResource()
{
    var calendar = new Calendar();
    var evt = calendar.Create<Event>();

    evt.Resources.Add("Resource1");

    Assert.That(evt.Resources.Count, Is.EqualTo(1));
}

It's a bug. evt.Resources.UnionWith(new[]{"SomeResource"}) doesn't work either.

I had a quick look at the source. The getter returns a new detached hashset. Any change on the hashset is never written back into the event object:
        public ISet<string> Resources
        {
            get { return new HashSet<string>(Properties.GetMany<string>("RESOURCES")); }
            set { Properties.Set("RESOURCES", value); }
        }

fbd90977305a609fbff3ced17e44cb479aae9a24
Fixed published in nuget version 2.2.26


MINOR: Move parseCsvList to Csv in server-common

All callers are still in core but we expect to move a bunch of them to other modules soon, so it makes sense to consolidate all the Csv parsing logic into the Csv class in server-common.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

None of the test failures are related, merging to trunk


Notification without popups in background

Hello, i have a problem with notification when app is in background or killed.
I can receive notification from firebase but i can see only the icon on icon try, the popup never appears when app is in background or not running.
Can some one help me?

componentWillMount(){
		// If user isn't logged in goto LoginScreen
		if( this.props.Auth.token === null ) {
			this.props.navigation.navigate('AuthScreen');
		}

		FCM.requestPermissions()
			.then((garanted) => {
				if(garanted) {
					this._setToken();
				}
			})
			.catch(()=>console.error('notification permission rejected'));

		this._notificationListener = FCM.on(FCMEvent.Notification, async (notif) => {
			// optional, do some component related stuff
			console.log('ON Notification', notif);
			// there are two parts of notif. notif.notification contains the notification payload, notif.data contains data payload
			if(notif.local_notification){
				//this is a local notification
				console.log('local notification');
			}
			if(notif.opened_from_tray){
				console.log('opened_from_tray');
				//iOS: app is open/resumed because user clicked banner
				//Android: app is open/resumed because user clicked banner or tapped app icon
			}
			// await someAsyncCall();

			if(Platform.OS ==='ios'){
				//optional
				//iOS requires developers to call completionHandler to end notification process. If you do not call it your background remote notifications could be throttled, to read more about it see https://developer.apple.com/documentation/uikit/uiapplicationdelegate/1623013-application.
				//This library handles it for you automatically with default behavior (for remote notification, finish with NoData; for WillPresent, finish depend on "show_in_foreground"). However if you want to return different result, follow the following code to override
				//notif._notificationType is available for iOS platfrom
				switch(notif._notificationType){
					case NotificationType.Remote:
						notif.finish(RemoteNotificationResult.NewData); //other types available: RemoteNotificationResult.NewData, RemoteNotificationResult.ResultFailed
						break;
					case NotificationType.NotificationResponse:
						notif.finish();
						break;
					case NotificationType.WillPresent:
						notif.finish(WillPresentNotificationResult.All); //other types available: WillPresentNotificationResult.None
						break;
				}
			}
			if(notif.data) {
				//console.log( JSON.parse(notif.data));
				//this._sendRemote(notif);
			}
		});

		FCM.getInitialNotification().then(notif => {
			console.log('initials: ', notif);
			//do something with notification
		});
	}

	componentDidUpdate() {}

	componentWillUnmount = () => {
		this._notificationListener.remove();
	}

	_setToken = () => {
		FCM.getFCMToken().then(token => {
			this.props.setFCMToken(token);
		});
	}

	_sendRemote = notif => {
		FCM.presentLocalNotification({
			title: notif.fcm.title,                     // as FCM payload
			body: notif.fcm.body,                    // as FCM payload (required)
			sound: "default",                                   // as FCM payload
			priority: "high",                                   // as FCM payload
			click_action: "ACTION",                             // as FCM payload
			auto_cancel: true,                                  // Android only (default true)
			vibrate: 300,                                       // Android only default: 300, no vibration if you pass 0
			wake_screen: true,                                  // Android only, wake up screen when notification arrives
			lights: true,                                       // Android only, LED blinking (default false)
			show_in_foreground: true                                  // notification when app is in foreground (local & remote)
		});
	}

Env:
RN 0.48.4
react-native-fcm: 11.2.0
Im using a Samsung Galaxy S7 and app is in background;

Ok guys, how switch off notification-reciever if app not active. Because i write on java my listener for background notifications and when my listener executed and showed my notification i've as a result two notifications one my and one default

@qDanik  would you share the listener code


Add execution_model field

As discussed in the meeting, this adds the execution_model field to specify custom prediction procedutes.
Let me know if you agree with the description here and if we want to stick wit the name or still change it.
(I am just putting a veto on runner, because it's ambiguous with our other usage of this term ;))
@oeway @frauzufall @fjug @esgomezm @FynnBe @m-novikov

Thank you!
Now, reading the description (which is good for me), I do agree with @FynnBe that seems more descriptive the "how to use" name. Otherwise, I would maybe write "model_execution".

I am fine with execution_mode, but 😇 still want to throw how_to_run as a self-explanatory field name in the race, pointing at the runner that it is directly related to, without actually including runner, which clearly distinguishes it. Alternatively run_mode makes more sense to me (or we rename the infamous runner to executor, which has that slightly morbid/generic ring to it).

Ok, I made a poll with all the options listed here:

(I think you can vote multiple times, but please vote only once ;))

only once per option, though?!?

(I think you can vote multiple times, but please vote only once ;))

only once per option, though?!?

To be more precise: although the simple polling implementation from https://github.com/apex/gh-polls allows you to vote as much as you want, please vote only one time (in total) ;).

aftter discussion in the bioimage.io meeting `run_mode


Failed to parse invalid UTF8

The following comment cannot be parsed:
syntax = "proto2";

message Type {
  // An example of strange characters in a comment: attachment���s
  TYPE_UNSPECIFIED = 0;
}

I could write up a test case when I have a little more time, but thought I'd include the issue here for now.

What is the issue? Are you seeing an error or are you seeing something else? I would expect it to convert the invalid UTF8 characters to the Unicode replacement character.
Also, FWIW, I think GitHub did the same to your example input above: it is valid UTF8 with the Unicode replacement character. In order to provide invalid UTF8 in a comment, you'd have to hex-encode it or upload it to a gist as a binary file.
I would prefer it to fail on invalid input (and if we add a "strict mode", it definitely should). But protoc does not. On the other hand, protoc happily preserves invalid UTF8. I filed a bug for this several months ago: https://github.com/protocolbuffers/protobuf/issues/9175

Yea, there's an error when trying to parse a file from buf.build/googleapis/googleapis - this line in particular.
For example,
google/cloud/compute/v1/compute.proto:630:453: invalid UTF8 at offset 36302: ef

Presumably this works in protoreflect because we use it to parse the same file just fine.

I would prefer it to fail on invalid input (and if we add a "strict mode", it definitely should)

I agree - it's a shame that this is even allowed in googleapis, but it'd be great if we could support this use case so that we can fully transition from protoreflect to protocompile.
I'm happy to discuss this more though - ideally we could get GoogleAPIs to remove that culprit line (it's the only case I've found so far), but I'm not sure how realistic that is.

This will be fixed in the new home for this repo (in bufbuild): https://github.com/bufbuild/protocompile/pull/34


added new company "clusus" to companies.json

What kind of change does this PR introduce?

[ ] Bug Fix
[ ] Documentation change
[x] Added new company
[ ] Other(s)

Why was this change needed?

Other information

@susanhsrestha @thenishantsapkota requested changes have been made


Add support for a Completed status (Fixes: #6434)

Per #6434 (filed by my colleague at Altispeed), this is the user interface and backend state. This does piggyback off of some existing shipped functionality, but does work.

@tsimonq2 looks like you know what you are doing! LMK if you need any specific pointers :)

@tsimonq2 any progress on this? I'd like to see this integrated and not get lost to the sands of time :)

@tsimonq2 any progress on this? I'd like to see this integrated and not get lost to the sands of time :)

I know this is still on @tsimonq2  list of dev projects. He is currently assigned a pretty big project of developing a parcel tracking software so this may take a while to get to but we will make sure it gets added as once we get this feature added to Inventree we will be implementing it at our company.

@SchrodingersGat How does this look? It works :smile:

@tsimonq2 are you able to address the failing QI issues as above?

@tsimonq2 if you were able to get back to this soon, it could still make the next release cycle. Please let me know.

@SchrodingersGat Should be all set now :)

@tsimonq2 many thanks for the contribution :)

@SchrodingersGat this needs to be called out as breaking for plugins that call complete_order in the release notes as the effect changes


#r nuget loads unlisted packages

Describe the bug
I did not expect that #r "nuget: ..." with an unqualified version to load unlisted packages. Does it make sense to load unlisted versions?
For example, the current maximum listed version of FSharp.Data.Fred is 0.1.0, but there is an unlisted version 1.0.0 that was accidentally published. It's unlisted, so I wouldn't expect #r "nuget: FSharp.Data.Fred" to load it. I'd expect it to load the maximum listed 0.1.0 version. But it loads 1.0
Please complete the following:
Version: 1.0.446104+34cc09c68f37568da9c8151ba5c51d264d0375d5
Library version: 1.0.0-beta.23461.4+34cc09c68f37568da9c8151ba5c51d264d0375d5
Build date: 2023-09-17T09:55:38.4483953Z
Screenshots

That is a bit surprising but I don't think this is something that .NET Interactive can control, as it's just running dotnet restore on a (hidden) project file using a package reference without a version specified.
When a package is unlisted in NuGet it's hidden from search but can still be restored.

Ok, thanks for looking into it.
It's a low probability edge case regardless. I'll close given it's not under your control.


Off annoying beep sound

When I using the VS Code with "terminal.integrated.shell.windows" set to powershell.exe and deleting characters with backspace there is a high-pitched, old-school sound warning when there are no more characters to delete back. It would be wonderful if there is a setting to disable that sound by default. Because it's very annoying and difficult to off.
You can lisen this sound
VSCode Version: Code 1.20.1 (f9d0c68, 2017-02-09T00:26:45.394Z)
Shell version: Shell 1.7.9
OS Version: Microsoft Windows [Version 10.0.16299.248]
Under macOS all is perfect in VS code.
My solution after 15 minutes of googling:
run cmd.exe as administrator
echo Set-PSReadlineOption -BellStyle None >> C:\Windows\System32\WindowsPowerShell\v1.0\Microsoft.Powershell_profile.ps1
powershell Set-ExecutionPolicy RemoteSigned

This is a duplicate of #422. You can install 2.0.0-beta1 for the fix.

@lzybkr Why we need to install some beta? This must be default option in stable version.
Just check. Most populat text editors like Vs code and Vim use your beeping PSReadline

Best programming text editors
What is the best javascript IDEs or editors

Here can find issues related to silent behavior and “visual beep”:

https://github.com/Microsoft/vscode/issues/21441
https://github.com/Microsoft/WSL/issues/715
https://github.com/noseglid/atom-build/issues/291
https://intellij-support.jetbrains.com/hc/en-us/community/posts/205806309-How-to-disable-completely-audible-beeps-

You don't need to install anything - you've found the configuration option, I just suggested an alternative. Note that the beta is pretty stable, there are a couple bugs that might be blocking some folks, but it doesn't hurt to try it.
I considered a visual beep - in fact it's one of the options you can configure, though unfortunately it's not implemented. I find this a little surprising, but I think you're the first to mention it.


specific golint case is breaking into 2 lines

repos/src/github.com/osrg/gobgp/server/server.go:1136:2:warning: can probably use "var seqs []struct { (golint)
repos/src/github.com/osrg/gobgp/server/}" instead

Is this a bug since normally you'd see (golint) at the end there?

What does the raw golint output look like?

repos/src/github.com/osrg/gobgp/server/server.go:1136:2: can probably use "var seqs []struct {
	seq	int
	isLocal	bool
}" instead

I've never seen that before, though I'm a bit surprised the second line shows up... I would expect the regex to ignore that line. Very odd.

@alecthomas thanks for the quick fix


Notebook integration tests depend on window being visible

I cannot run the integration tests on macOS and leave the window in the background, there is a ton of exceptions and failures:
    1) notebook open
  notebook undo redo
    2) notebook open
    - execute and then undo redo
  notebook working copy
rejected promise not handled within 1 second: Error: Not Implemented: document->Symbol(Symbol.toStringTag)
stack trace: Error: Not Implemented: document->Symbol(Symbol.toStringTag)
    at Object.get (/Users/bpasero/Development/Microsoft/vscode-electron-7.0.x/out/vs/base/common/file:/Users/bpasero/Development/Microsoft/vscode-electron-7.0.x/src/vs/base/common/types.ts:278:12)
    at Proxy.toString (<anonymous>)
    at type (/Users/bpasero/Development/Microsoft/vscode-electron-7.0.x/extensions/vscode-notebook-tests/node_modules/vscode/node_modules/mocha/lib/utils.js:269:6)
    at Object.canonicalize (/Users/bpasero/Development/Microsoft/vscode-electron-7.0.x/extensions/vscode-notebook-tests/node_modules/vscode/node_modules/mocha/lib/utils.js:437:26)
    at /Users/bpasero/Development/Microsoft/vscode-electron-7.0.x/extensions/vscode-notebook-tests/node_modules/vscode/node_modules/mocha/lib/utils.js:481:45
    at Array.forEach (<anonymous>)
    at /Users/bpasero/Development/Microsoft/vscode-electron-7.0.x/extensions/vscode-notebook-tests/node_modules/vscode/node_modules/mocha/lib/utils.js:480:12

Also often I see this error:
Error invoking remote method 'ELECTRON_GUEST_VIEW_MANAGER_CALL': Error: Invalid guestInstanceId: 26: Error: Error invoking remote method 'ELECTRON_GUEST_VIEW_MANAGER_CALL': Error: Invalid guestInstanceId: 26
    at EventEmitter../lib/renderer/ipc-renderer-internal.ts.exports.ipcRendererInternal.invoke (electron/js2c/renderer_init.js:2525:19)

Commands used in the notebook integration tests which depend on window focus

workbench.action.files.revert (won't finish if window has no focus)
workbench.action.revertAndCloseActiveEditor (won't finish if window has no focus)
default:type (might affect editorFocus)

Fixed via https://github.com/microsoft/vscode/pull/103054:

avoid above commands
use workspace edit other than default:type