Roll Engine from ae0401df460a to a818cb1026bd (23 revisions)

https://github.com/flutter/engine/compare/ae0401df460a...a818cb1026bd
2021-08-17<EMAIL_ADDRESS>Roll Skia from 957ed75731e0 to 7020699e3835 (7 revisions) (flutter/engine#28145)
2021-08-17<EMAIL_ADDRESS>Use the suppressions script to source runtime suppressions in run_tests.py (flutter/engine#28140)
2021-08-17<EMAIL_ADDRESS>Revert "[ci.yaml] Move roller to blocking (#28134)" (flutter/engine#28143)
2021-08-17<EMAIL_ADDRESS>Fix dead key handling on Win32 (again) (flutter/engine#28125)
2021-08-17<EMAIL_ADDRESS>Roll Dart SDK from 17c9a00cce20 to fecde133832e (1 revision) (flutter/engine#28142)
2021-08-17<EMAIL_ADDRESS>Roll Skia from 17dc658f29e5 to 957ed75731e0 (4 revisions) (flutter/engine#28135)
2021-08-17<EMAIL_ADDRESS>[ci.yaml] Move roller to blocking (flutter/engine#28134)
2021-08-17<EMAIL_ADDRESS>Roll Skia from 059798f40eac to 17dc658f29e5 (1 revision) (flutter/engine#28133)
2021-08-17<EMAIL_ADDRESS>Roll Dart SDK from 7bd8eb1d95e7 to 17c9a00cce20 (3 revisions) (flutter/engine#28132)
2021-08-17<EMAIL_ADDRESS>Roll Skia from 9ded59bbadb3 to 059798f40eac (12 revisions) (flutter/engine#28130)
2021-08-17<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from ctNKhIDQX... to FJeJwYM81... (flutter/engine#28129)
2021-08-17<EMAIL_ADDRESS>[ci.yam] Add roller dry run to presubmit (flutter/engine#28122)
2021-08-17<EMAIL_ADDRESS>Revert "Add bundletool to DEPS and remove outdated header comment (#28040)" (flutter/engine#28128)
2021-08-17<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from ZRdfbSetp... to n7ocDhOf_... (flutter/engine#28127)
2021-08-17<EMAIL_ADDRESS>Add bundletool to DEPS and remove outdated header comment (flutter/engine#28040)
2021-08-17<EMAIL_ADDRESS>Roll Dart SDK from d39206fb4e25 to 7bd8eb1d95e7 (1 revision) (flutter/engine#28123)
2021-08-16<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from 54i7Z2kLS... to ctNKhIDQX... (flutter/engine#28121)
2021-08-16<EMAIL_ADDRESS>Roll Skia from bd40fb55bb11 to 9ded59bbadb3 (2 revisions) (flutter/engine#28119)
2021-08-16<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from a27NGktH6... to ZRdfbSetp... (flutter/engine#28118)
2021-08-16<EMAIL_ADDRESS>Issues/79528 reland (flutter/engine#28117)
2021-08-16<EMAIL_ADDRESS>added log statement to timerfd_settime failure (flutter/engine#27974)
2021-08-16<EMAIL_ADDRESS>Roll Skia from 7d99ba2503e1 to bd40fb55bb11 (1 revision) (flutter/engine#28114)
2021-08-16<EMAIL_ADDRESS>[fuchsia] [ffi] Basic support for Channels and Handles (flutter/engine#27849)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from 54i7Z2kLSPDo to FJeJwYM81VAV
fuchsia/sdk/core/mac-amd64 from a27NGktH6S6i to n7ocDhOf_t98
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


"brew upgrade" fails with "Error: Version must not be empty"

brew doctor output
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some installed formulae are deprecated or disabled.
You should find replacements for the following formulae:
  neofetch
<EMAIL_ADDRESS>  python-packaging

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
HOMEBREW_VERSION: 4.3.11-3-g38e47ea
ORIGIN: https://github.com/Homebrew/brew
HEAD: 38e47ea9964a6991125d44a373a7de038b70ee10
Last commit: 61 minutes ago
Core tap HEAD: 04fbd71d56ad72b201449d1d1ad96a3510673957
Core tap last commit: 6 months ago
Core tap JSON: 29 Jul 15:37 UTC
Core cask tap HEAD: 1a98cd7bf4117c79d6c79916d06cb9223b084844
Core cask tap last commit: 6 months ago
Core cask tap JSON: 29 Jul 14:47 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_BAT_THEME: ansi
HOMEBREW_CASK_OPTS: []
HOMEBREW_EDITOR: /opt/homebrew/bin/nvim
HOMEBREW_MAKE_JOBS: 12
HOMEBREW_SORBET_RUNTIME: set
Homebrew Ruby: 3.3.4 => /opt/homebrew/Library/Homebrew/vendor/portable-ruby/3.3.4/bin/ruby
CPU: dodeca-core 64-bit arm_blizzard_avalanche
Clang: 15.0.0 build 1500
Git: 2.42.0 => /opt/homebrew/bin/git
Curl: 8.6.0 => /usr/bin/curl
macOS: 14.5-arm64
CLT: <IP_ADDRESS>.1.1708646388
Xcode: 15.4
Rosetta 2: false

What were you trying to do (and why)?
brew upgrade
What happened (include all command output)?
$ brew upgrade --verbose --debug
/usr/bin/env PATH=/opt/homebrew/opt/gh/bin:/opt/homebrew/Library/Homebrew/shims/shared:/usr/bin:/bin:/usr/sbin:/sbin HOME=/Users/llimllib gh auth token --hostname github.com
==> Downloading https://formulae.brew.sh/api/formula.jws.json
/usr/bin/env /opt/homebrew/Library/Homebrew/shims/shared/curl --disable --cookie /dev/null --globoff --user-agent Homebrew/4.3.11-3-g38e47ea\ \(Macintosh\;\ arm64\ Mac\ OS\ X\ 14.5\)\ curl/8.6.0 --header Accept-Language:\ en --fail --remote-time --output /Users/llimllib/Library/Caches/Homebrew/api/formula.jws.json --location --time-cond /Users/llimllib/Library/Caches/Homebrew/api/formula.jws.json --disable --cookie /dev/null --globoff --show-error --user-agent Homebrew/4.3.11-3-g38e47ea\ \(Macintosh\;\ arm64\ Mac\ OS\ X\ 14.5\)\ curl/8.6.0 --header Accept-Language:\ en --fail --compressed --speed-limit 100 --speed-time 5 https://formulae.brew.sh/api/formula.jws.json
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 4287k  100 4287k    0     0  7410k      0 --:--:-- --:--:-- --:--:-- 7404k
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading gh
/usr/bin/env /opt/homebrew/bin/gh --version
Error: Version must not be empty
/opt/homebrew/Library/Homebrew/version.rb:497:in `initialize'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/call_validation.rb:167:in `bind_call'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/call_validation.rb:167:in `validate_call_skip_block_type'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/call_validation.rb:109:in `block in create_validator_slow_skip_block_type'
/opt/homebrew/Library/Homebrew/attestation.rb:86:in `new'
/opt/homebrew/Library/Homebrew/attestation.rb:86:in `block in gh_executable'
/opt/homebrew/Library/Homebrew/extend/kernel.rb:534:in `with_env'
/opt/homebrew/Library/Homebrew/attestation.rb:83:in `gh_executable'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/call_validation.rb:270:in `bind_call'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/call_validation.rb:270:in `validate_call'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/_methods.rb:277:in `block in _on_method_added'
/opt/homebrew/Library/Homebrew/attestation.rb:114:in `sort_formulae_for_install'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/call_validation.rb:270:in `bind_call'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/call_validation.rb:270:in `validate_call'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/_methods.rb:277:in `block in _on_method_added'
/opt/homebrew/Library/Homebrew/cmd/upgrade.rb:137:in `run'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/call_validation.rb:270:in `bind_call'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/call_validation.rb:270:in `validate_call'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/3.3.0/gems/sorbet-runtime-0.5.11495/lib/types/private/methods/_methods.rb:277:in `block in _on_method_added'
/opt/homebrew/Library/Homebrew/brew.rb:95:in `<main>'
Please report this issue:
  https://docs.brew.sh/Troubleshooting

What did you expect to happen?
brew upgrade to succeed
Step-by-step reproduction instructions (by running brew commands)
I don't have an exact repro; @carlocab asked me to file an issue in this discussion. My best guess is:

set brew to this version:

$ brew --version
Homebrew 4.3.11-3-g38e47ea
Homebrew/homebrew-core (git revision 04fbd71d56a; last commit 2024-01-23)
Homebrew/homebrew-cask (git revision 1a98cd7bf4; last commit 2024-01-23)

Install gh from version HEAD-53c36d0 (not sure how to tell brew to do that)
brew upgrade

One feature thought: it seems like homebrew ought to log the formula that failed without needing to run --debug

Thanks @llimllib! This will be fixed by #17899.


Add Page Size Selector to Search Results

The number of results returns per page on the search screen is currently hard coded. The users should have a control that allows them to set it to something else. This is especially important for users with slow connections that want to load less results per page.
Once set, that setting should be stored, and applied every time to the search results.

Also the ability to jump to a specific page number, last page, first page may be useful?


Copy progress indicator is wrong after restart

After restarting graph-node while it is copying a subgraph, the overall progress indication is wrong. It very much looks like after restarting, entities that were copied before the restart aren't counted in the progress. This is only a display issue, it does not affect how copying works.

This is wrong, it needs to be calculated from the information in state.tables.

Fixed by https://github.com/graphprotocol/graph-node/pull/5075


DCOS-8098: Add log link to task table

Add an icon that links directly to logs tab.

Reviewer: @jfurrow

Except for that one extra s, LGTM 🚢

feel free to merge once it goes back to green


Preliminary Num Turker Analysis

Run dataset aggregation script to generate datasets for turkers 1, 2, 3, 4, 5
Use data frame script to generate filtered datasets for is_stressor = 1, is_covid = 0
Run scikit learn script using algorithm = SVC, features = BERT
Save text output, dataset, dataset settings, and models in a folder
Aggregate Macro precision, recall, and f1 scores in an excel file
Generate a graph comparing turker performance using these scores); group P, R, and F1 together for each pool of turks and make it visually easy to look at.

Also compare to original label #20

Example from our conversation today


Kernel 5.15 breaks bcache caches, causing data loss.

Describe the bug
I have root as a bcache device, using an ssd (/dev/nvme0n1p4) as my cache device, and md raid 1 over 2 spinning disks (/dev/md127) as the backing device. ext4 is running on the bcache device (/dev/bcache0) which is my root fs.
Booting into 5.15 breaks the btree on the cache device, such that bcache then removes the cache device. On Sunday (14 Nov), I had the bcache device set in writeback mode. The loss of the cache device caused substantial data loss and I was very lucky to only have to reinstall the whole machine, and not lose any personal data. I now run the bcache device in writethrough mode. Nevertheless, I've just tried again, and booting into 5.15.2 caused the cache device to corrupt and then be ejected from the bcache device.
This shows it all going terribly wrong in 5.15.2:
Nov 17 11:23:46 rocket kernel: BUG: unable to handle page fault for address: ffff9a27e1d537f8
Nov 17 11:23:46 rocket kernel: #PF: supervisor write access in kernel mode
Nov 17 11:23:46 rocket kernel: #PF: error_code(0x0003) - permissions violation
Nov 17 11:23:46 rocket kernel: PGD 6fbc01067 P4D 6fbc01067 PUD 100f85063 PMD 12341b063 PTE<PHONE_NUMBER>d53161
Nov 17 11:23:46 rocket kernel: Oops: 0003 [#1] SMP NOPTI
Nov 17 11:23:46 rocket kernel: CPU: 3 PID: 1655 Comm: kworker/3:36 Tainted: G           O      5.15.2 #1-NixOS
Nov 17 11:23:46 rocket kernel: Hardware name: To Be Filled By O.E.M.
I have searched around and can't find other reports of anything similar from other distributions, though google is pretty useless these days for attempting to find such things. Seems unlikely to me that it should be nixos specific, but difficult to be sure.
Steps To Reproduce
Steps to reproduce the behavior:

(Probably not using 5.15) setup a bcache device. Whether or not it is required that the backing device be raid1 and the cache device be ssd over nvme is unknown
Format it, use it.
Reboot into 5.15 and attempt to use the device. It looks like a panic occurs, and at that point the btree on the cache device is corrupted and reboots will refuse to attach the cache device to the bcache. The cache device is now unusable and has to be rebuilt, causing loss of all data on it.

Expected behavior
The kernel should not panic!
Notify maintainers

Metadata
# nix-shell -p nix-info --run "nix-info -m"
- system: `"x86_64-linux"`
- host os: `Linux 5.14.18, NixOS, 21.05.4116.46251a79f75 (Okapi)`
- multi-user?: `yes`
- sandbox: `yes`
- version: `nix-env (Nix) 2.3.16`
- channels(root): `"home-manager-21.05, nixos-21.05.4116.46251a79f75, nixos-unstable-21.11pre331460.931ab058daa"`
- channels(matthew): `"home-manager-21.05, nixos-unstable-21.11pre331460.931ab058daa"`
- nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
attribute: linuxPackages_5_15

Appears to be a known issue<EMAIL_ADDRESS>that will be fixed in next kernel stable.

Ok, phew! I failed to find those reports. I'll try again then in a few days and close this if it all seems good. Thank you.

https://nixpk.gs/pr-tracker.html?pr=146444

I have lost a btrfs partition of 3.6TB due to this issue with bcache in writebak mode :(
Now I'm struggling to get back anything I can from the corrupt partition. It does not mount and the recovery command did not recover a lot of files.
I would never use bcache in writeback mode again ...

Yeah, it's an awful situation - any data loss always is. FWIW, I've used bcache in writeback mode for years and never had a problem[0]. I don't understand how the upstream kernel developers managed to not spot this in testing but there you go - it's happened. Obviously I was very lucky that the cached, and lost, data was mainly nix-store, and not my personal data.
I do think all 5.15 kernels should be removed from nixos stable right now until this issue is fixed - anything which causes data loss really ought to cause the kernel to be removed until a fix is available, IMO.
[0] But, I've only recently switched from debian unstable to nixos. Watching the two, it seems to me like Debian are rather more conservative with kernels, certainly at the moment.

I do think all 5.15 kernels should be removed from nixos stable right now until this issue is fixed

They already were fixed with https://github.com/NixOS/nixpkgs/pull/146821 and the change is available on all supported channels.

@duzun Might be of interest: I've just written and added this to my configuration.nix:
let
  latestKernel = pkgs.linuxPackages_latest.kernel;
  latestPatch = lib.toInt (lib.versions.patch latestKernel.version);
  latestMinor = lib.toInt (lib.versions.minor latestKernel.version);
  latestMajor = lib.versions.major latestKernel.version;

  sensibleKernel = if latestPatch >= 5
                   then
                     pkgs.linuxPackages_latest
                   else
                     let
                       minor = latestMinor - 1;
                       attr = "linuxPackages_${latestMajor}_${toString minor}";
                     in lib.attrsets.getAttrFromPath [attr] pkgs;
in
{
  ...
  boot.kernelPackages = sensibleKernel;
  ...
}

Yes, this will absolutely go wrong when the major gets bumped. But still, may be useful until then.

@msackman thanks for sharing.
Meanwhile I’ve writen some scripts to ease handling bcache:
bcache-flush
bcache-mode-read
bcache-mode-write
bcache-status
My strategy is to use bcache in writethrough mode by default, but sometimes when I have intensive writes to switch it to writeback for a while, watch the status, flush to backing device when done or before reboot, and switch back to writethrough mode when done.
I can share my scripts if anybody is interested. They served me well during recovery for moving a few TB back and forth between HDDs.

@duzun sounds good. Also, just in case you're not aware, if you are using suspend or hibernate, technically, you should set the cache_mode to 'none' during suspend/hibernate. This is because apparently you're not meant to write anything to any disk during resume and if not in 'none' then even a read from the underlying hdd can cause a write to the sdd. See https://bcache.evilpiepirate.org/FAQ/
There doesn't seem to be any proper support for this in NixOS that I can find. So I've hacked my own up:
  environment.etc."systemd/system-sleep/bcache.sh" = {
    mode = "0540";
    text = ''
#! /bin/sh

OP="$1"

if [ "$OP" = "pre" ]; then
   for cache in /sys/block/bcache*; do
     echo "Putting $cache into cache_mode = none"
     echo none > $cache/bcache/cache_mode
   done

   echo "All bcache block devices set to cache_mode = none"

elif [ "$OP" = "post" ]; then
   for cache in /sys/block/bcache*; do
     echo "Putting $cache into cache_mode = writethrough"
     echo writethrough > $cache/bcache/cache_mode
   done

   echo "All bcache block devices set to cache_mode = writethrough"

fi
'';
  };

@msackman Here are my bcache scripts.
I would use my bcache-flush script for suspend.
Though, during my experience with bcache in writeback mode for the last about a year, my system was going to sleep and coming back with no issue. I'm not sure whether is was going to hibernation (two desktop PCs, one working 24/24).


Homework 1

Homework 1 Completed .
File Log:

Add Student Output:

Search Student Output:

Update Student Ouput:

Metric
Value

alert_status
b:x: ERROR/b

bugs
0

code_smells
33

reliability_rating
A (0 Bugs)br/Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)br/Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
b:x: ERROR/b

bugs
0

code_smells
34

reliability_rating
A (0 Bugs)br/Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)br/Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
b:x: ERROR/b

bugs
1

code_smells
33

reliability_rating
E (at least 1 Blocker Bug)br/Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)br/Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
b:x: ERROR/b

bugs
0

code_smells
32

reliability_rating
A (0 Bugs)br/Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)br/Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
b:white_check_mark: OK/b

bugs
0

code_smells
31

reliability_rating
A (0 Bugs)br/Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)br/Note: A being best, E being worst

vulnerabilities
0


eslint  ts error  warning never refresh, even though i has fixed the problem,  i need to run volar: restart vue server to remove the error tip.

volar: 0.34.3 , i used the take over mode.
#1007 #1001  this error occured again.

I can't reproduce this problem, please provide repro case.

Had this error too, pops up randomly when switching between Vue projects. As I have observed, it only affects Vue projects with Typescript. No errors when used in Javascript.

I still have this problem, but it happens when TS types are passed across files. For example, for component props, the parent component passes data to the child component. If the type is wrong, vscode will report an error. Even if I deal with this error later, the error prompt of vscode will not disappear, and the prompt is still the previous error.

Will be helpful if you can describe reproduce steps on https://github.com/johnsoncodehk/volar-starter.

@johnsoncodehk  clone this demo  https://github.com/WormGirl/volar-eslint-takeOverMode-cache-bug

enable take over mode
open src/views/HomeView.vue, will find an ts error like below

open  src/components/HelloWorld.vue fixed this error by change msg type to string.
return  src/views/HomeView.vue. this error still show.


feat(receiver): update proto to v0.19.0

If this works, we should make a PR (or at least an issue) in the upstream.

Agreed. @mat-rumian can you create an issue in this repo to switch this back once it's released upstream?


coreupdate: document serving packages from s3

Already approved PR. I'm not doing further checks on the docs change, just merging to unblock this fix.


examples crash

Hardware: Raspberry Pi 3B

Library: Installing collected packages: rpi-ws281x
Successfully installed rpi-ws281x-3.1.0

OS: Raspbian Stretch

kernel: pi@raspberrypi:~ $ uname -a
Linux raspberrypi 4.14.34-v7+ #1110 SMP Mon Apr 16 15:18:51 BST 2018 armv7l GNU/Linux

error information:
python3 strandtest.py

Traceback (most recent call last):
File "/home/pi/rpi-ws281x-python/examples/strandtest.py", line 91, in
strip.begin()
File "/home/pi/.local/lib/python3.5/site-packages/rpi_ws281x/rpi_ws281x.py", line 124, in begin
raise RuntimeError('ws2811_init failed with code {0} ({1})'.format(resp, str_resp))
RuntimeError: ws2811_init failed with code -5 (mmap() failed)

I am having the same problem, is there a solution coming up?

@burckaan With a lot of experiments, I found a solution as below:
sudo pip3 install rpi-ws281x
sudo python3  strandtest.py
It works for me now.:)

I've been experiencing this issue too. I want to control three separate  led strips at once. I'm using pins 10 (SPI mode),13, and 18 (PWM mode). I've used sudo to run my python script as iZhangHui suggests, but I think it should be possible to run the script without using sudo. I did some digging and it looks like the function call that fails (mmap) is attempting to map memory which the user does not have the permissions to do.
The keys here are that the user must have permissions to access /dev/mem AND (I think) must have CAP_SYS_RAWIO. Now I'm not entirely sure how to do those things, but this is what I've figures out so far. Perhaps there is a way to add an exception for this specific program? I think there should be a solution that doesn't give blanket memory access to the pi user all the time.
Sorry to dig up a dead bug report like this, but hopefully someone who knows a little more than I can think up a clever and elegant solution.

I am having the same problem trying to control ws2815 string.  Only now, the problem originates on "line 131, in begin . . . (mmap() failed)"
And the solution above: sudo python3 strandtest.py  brings error "No such file or directory"

sudo python3 -m pip install --force-reinstall adafruit-blinka worked for me.


[BUG] Rule ESCU - Windows AD Rogue Domain Controller Network Activity - Rule fails to run

Rule ESCU - Windows AD Rogue Domain Controller Network Activity - Rule fails to run with the following error:

Already planning to be fixed as discussed with @ljstella.


Added ability to enable/disable menu icon.

Thanks for the PR, but this can be done by simply setting navigationItem.leftItem = nil


Support Postgres Declarative Partitioning

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

Summary 💡

[ ] Add support for declaring tables as PARTITIONED BY RANGE/HASH/LIST ("id"), along with creating a default partition.
[ ] Initially support partition key being the ID primary key
[ ] Handle unique constraint limitations that come with partitions.
[ ] Add API method for detaching the default partition, creating a new default, and reattaching the old default with an appropriate range. This will enable creating partitions on demand at a constant interval, without having a partition key be a timestamp.

Motivation 🔦
Partitioning enables us to delete old data in bulk fast. Deleting data from big tables takes a very long time so we partition the data by date (technically by ID as explained above). This way we can delete data from a single partition in a matter of seconds.
Thanks!

Hi, team. This issue has been open for 2 years and it looks there is some interest to it (14 reactions).
My team is also interested in this feature, and we're interested in contributing to it. Is this something you'd be willing to accept as a contribution?


fix: typo of 'redo'

[X] I read the contributing guide
[X] I agree to follow the code of conduct

Summary
typo
How did you test this change?

I'd like to know when will this be released :)

I'd like to know when will this be released :)

Maybe this weekend.


Make returned data extensible

I'm running into a situation where I need another plugin (ngx-datatable) to be able to modify the data being returned from the server. Right now it's unable to do so because the data is not extensible.
ngOnInit(): void {
    this._apollo.watchQuery({
        query: getAllTheThings
    }).subscribe(({data, loading}) => {
        console.log(Object.isExtensible(data['theThings')); // false
    });
}

It would be super helpful if this was mutable out of the box. Failing that, does anyone have a suggestion as to how to maintain the observable state of the data but allow the other plugin to modify it?

Why don't you use lodash cloneDeep for this? Not sure if this is the expected way to go though?

Import * as _ from 'lodash';
...
Let copyOfData = _.cloneDeep(data);

Yeah it can't be mutable because then we have to clone every time we return data which quickly becomes inefficient. If you want it to be mutable cloning it yourself is the right way to do it.

Thanks for the info. I had considered using lodash previously but was concerned about the performance overhead and whether it would maintain its observable state. Appears to be working though. 👍

Yep we wouldn't want to build that performance overhead into the core itself.

Closing this. If you think this still need to be opened, @mention me :)


ARROW-6227: [Python] Apply from_pandas option in pyarrow.array consistently across types

Many sequence converters were not using the from_pandas option in pyarrow.array (which is False by default), so NaN and other pandas null sentinel values were being silently converted to Arrow null. For example as in the original report pyarrow.array(['string', np.nan]) would return an array with 1 null. This is definitely not good.
I made an effort to apply this consistently where it wasn't being applied and add appropriate unit tests.
Some notes:

We don't consistently raise TypeError or ValueError on mixed type input. For example

In [4]: pa.array(['string', np.nan])
TypeError
In [7]: pa.array([0, np.nan], type=pa.time32('s'))
ValueError

We weren't handling integer inputs for time32/time64 types. That's fixed and tested in passing here (though not for correctness...)

I have some code that I forgot to push

Definitely review again. The NaT issue is https://issues.apache.org/jira/browse/ARROW-842

Additonal test looks good

The NaT issue is https://issues.apache.org/jira/browse/ARROW-842

Ah, I even commented myself a while ago exactly about the NaT case .. Will close the new issue as a duplicate then.

There's a failure in arrow-python-test, will fix
[ RUN      ] DecimalTest.TestNoneAndNaN
..\src\arrow\python\python_test.cc(430): error: Failed
'ConvertPySequence(list, {}, &arr)' failed with Type error: int or Decimal object expected, got float

Frankly not sure we need this test. I'll expand the Python unit test I added to include decimals

I'll also add a couple time32/time64 correctness tests per ARROW-4111

I think this is done. Will merge once builds run

Codecov Report

Merging #5150 into master will increase coverage by 1.89%.
The diff coverage is 94.44%.

@@            Coverage Diff             @@
##           master    #5150      +/-   ##
==========================================
+ Coverage   87.36%   89.26%   +1.89%
==========================================
  Files         909      739     -170
  Lines      122252   105445   -16807
  Branches     1437        0    -1437
==========================================
- Hits       106807    94121   -12686
+ Misses      15083    11324    -3759
+ Partials      362        0     -362

Impacted Files
Coverage Δ

python/pyarrow/tests/test_convert_builtin.py
97.25% <100%> (ø)

cpp/src/arrow/python/python_test.cc
100% <100%> (ø)

python/pyarrow/tests/test_array.py
95.47% <100%> (ø)

cpp/src/arrow/python/python_to_arrow.cc
91.97% <90.38%> (ø)

go/arrow/math/uint64_amd64.go

go/arrow/memory/memory_avx2_amd64.go

rust/datafusion/src/execution/filter.rs

rust/arrow/src/csv/writer.rs

rust/datafusion/src/bin/main.rs

go/arrow/ipc/file_reader.go

... and 465 more

Continue to review full report at Codecov.

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


How can I change appearance Icon ?

@Karlen92 you can replace the Toast component with whatever you like https://github.com/jossmac/react-toast-notifications#replaceable-components


metal3/baremetal-operator: switch to 0.3.1 opensuse image

Align with other images so we default to the OBS builds, 0.3.1 being the latest tag which aligns with the upstream release.

Some reviewer notes from discussion with @dbw7:
The upstream Dockerfile installs the binary as /baremetal-operator but the downstream package-based image is /usr/bin/baremetal-operator - therefore I removed the command override in the manifest - this should mean we use the default entrypoint for each image (which will make testing with upstream images for development etc easier)
Also see https://build.opensuse.org/package/revisions/isv:metal3:BCI:baremetal-operator/baremetal-operator-image - the default entrypoint was actually broken, so I had to update it before the command removal would work


Download file problem fix!

Download file problem fix!

Thank you!
It’s not clear what problem exactly you’re trying to solve. Further, onDownloadStart (replacing the call to onDownloadRequested) is certainly not the correct place for this.
However, two (potential) issues have been extracted from your pull request to issues https://github.com/delight-im/Android-AdvancedWebView/issues/277 and https://github.com/delight-im/Android-AdvancedWebView/issues/278. Thanks again!


Open notification when touching any part of the notification item.

Summary:
In Notifications when i touch the text of an item nothing happens. Thought it would be good if It was possible to click in any part of the item and it would do its given action.
Steps to reproduce:
How can we reproduce the issue?
Go to notifications and touch the text in the middle of the item related to the notification.
What did you expect the app to do, and what did you see instead?
Expected the app to open the notification.
Device and Android version:
Zenfone 3 deluxe, Android 8.0.0
Commons app version:
2.8.2
Would you like to work on the issue?
Yes, I would like to give it a try even though I am fairly new to android development and it would be my first contribution to an open source project.

Indeed that would be great!
Good luck with implementation, please let us know :-)

Thanks for reporting this! To be clear, the app would open the notification if you tap the right side or the left side (the icon or date). We did not implement this for the middle part due to conflict with the "show more" link. Is it desirable to have the middle part work the same way as the others?

Just thought it would be good. The first time I used the app I insisted on clicking on the middle part out of habit.

I think it comes down to whether or not the onclick in the "middle text" conflicts with the "show more" as @misaochan mentioned. I will give it a try.
If I have questions about the code where do I ask? In the issue or in the group? Thanks for your attention.

Sounds good @befrou :) Please feel free to ask in this issue if you have any questions.

I used "SpannableString" to setup a 'ClickableSpan" in the "middle text" and it worked without conflict with the "See more". The only thing that bothers me is that the click on the "middle text" doesn't highlight the "Notification" as it does when you click the "date" at the right side or the "icon" at the left side.

I think it should be OK in that case. Could you please submit your PR so we can try it out @befrou ? Thanks!

Fixed in #1872


Broken Link

Link after "Connect to Azure SQL Database V12 via Redirection" is broken.

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

ID: 7fb994ad-e2aa-f7a9-6472-749d257639ae
Version Independent ID: 7f128e8b-8698-08ab-4d2e-47c4d6f60e1a
Content: Ports beyond 1433 for SQL Database
Content Source: articles/sql-database/sql-database-develop-direct-route-ports-adonet-v12.md
Service: sql-database
GitHub Login: @MightyPen
Microsoft Alias: genemi

@andrewatfornax Thanks for you valuable feedback. I have assigned this issue to content author to look into this issue and do appropriate updates in the document.
@MightyPen Could you please look at this issue. The URL provided with "Connect to Azure SQL Database V12 via Redirection" doesn't seem to be correct.

#please-close
Thank you @andrewatfornax . This Issue 25822 is being fixed by PR 25827.

@PRMerger15 Hi,
#please-close


always remove RemoteTimelineClient's metrics when dropping it

I need this to unblock https://github.com/neondatabase/neon/pull/3087

Two familiar sounding failures in one of the regress-tests:
FAILED test_runner/regress/test_remote_storage.py::test_remote_storage_upload_queue_retries[local_fs]
FAILED test_runner/regress/test_remote_storage.py::test_timeline_deletion_with_files_stuck_in_upload_queue[local_fs]


Add ( Get Profile & Change Profile )

Hey there is 2 rpc methods which will be a good way to organize the usernames for each wallet so it will be like a small book ..
Rpc method 1 : Get profile
Rpc method 2 : Change Profile
By adding them you can retrieve old Txs and get each address username and make a label with it
Also you can use it with a quick search for search for an address the sent to the wallet before with its username :)

Now to retrieve old transactions we use api.idena.org. However due its indexing, transactions are processed with a bit of delay. It would be useful to useful to use RPC. A deep dive of how these endpoints work is required.

Hey again .. these endpoint is is already fixed in the source of idena-go ..you will notice that the next update

But also i can help with these endpoints as i already know how each one work


Tidy old Ruby versions

I don't understand why we need to use defined() for this. The rbenv::version type has an absent parameter which purges a package and removes a directory 1.
I think the intention was that new Ruby versions should be added to cleanup at the same time they are installed but in practice that doesn't happen (5014ad4, 6ff24fe).
Instead I think we can set the classes to ensure => absent when we want to remove them like we do for other stuff.

Looks fine and makes sense. Merge away.


Add system to override UI props via AppConfig

First pass at a prop override system allowing any user of @autoarena/ui components to override their default props via the AppConfig context. This is useful to change behavior in different environments (e.g. local, cloud, on-prem deployments) without baking any notion of those environments into the base @autoarena/ui package.
Note that this does require components to explicitly opt into prop overrides by using this pattern:
export function MyComponent(props: Props) {
  const { ...myProps } = usePropOverrides(MyComponent, props);
  ...

The typing is not perfect, and the interface to set the overrides could be improved (Map has pretty poor devx) but the system works as-is and should prove a low-overhead way to customize application behavior.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.87%. Comparing base (b4b0d3a) to head (770664d).

Additional details and impacted files
@@           Coverage Diff           @@
##            trunk      #94   +/-   ##
=======================================
  Coverage   96.87%   96.87%
=======================================
  Files          34       34
  Lines        1439     1439
=======================================
  Hits         1394     1394
  Misses         45       45

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Can the images be cropped?

Hi, is there a chance the image can be cropped automatically? I am in South Africa and use Meteosat-8. It would be nice if I could crop the image so it shows a more zoomed in photo. The image resolution is high so a cropped version still looks good.
Thanks for the great product.

This will be worked on in conjunction with #20, which will allow users to create custom views, including cropped views.
This might not be implemented for a while, as I need to work on a few other issues at the moment. In the mean time though, I'm planning on adding some more cropped views manually. I'll make sure South Africa is one of them and will update this issue once it's added.


[Enhancement] Add documentation for fetching caBundle using secrets in API Calls

Description
Feature: https://github.com/kyverno/kyverno/pull/7985
This PR adds a field called caBundleFromSecret in service of API calls. This secret will be used for fetching CABundle for TLS with the service.
Slack discussion
No response

Closing as it conflicts with Resource Cache KDP.
We can use an API call to fetch the secret as follows
- name: tlscerts
     apiCall:
       urlPath: "/api/v1/namespaces/{{request.namespace}}/secrets/<TLS_SECRET_NAME>"
       jmesPath: "base64_decode( data.\"tls.crt\" )"

The service can use this variable as follows
service:
  url: https://svc.kyverno-notation-aws/checkimages
  caBundle: '{{ tlscerts }}'


twitter auth for okcurl

Early review to see if this fits within the okhttp repo and get comments on the approach etc.
I briefly checked the code against two similar Apache licensed joauth clients
https://github.com/twitter/hbc
https://github.com/yusuke/twitter4j
Works for a few simple GET requests and POST with form bodies.
Currently only reads the credentials from existing https://github.com/twitter/twurl config file ~/.twurlrc
Todo in this pull request

tests
sensible error handling e.g. .twurlrc missing
javadoc
give example supported requests, also useful to find edge cases

Later features

support alternate credentials store e.g. Keychain on osx
Implement authentication flow using "OkHttp Test Client" by default and optionally other clients.  e.g. open browser for user login and prompt user to enter the PIN.

I think this is interesting, but it’s far beyond the scope of okcurl. Maybe create your own GitHub project for this feature?

What about the Authentication Interceptor?  I'm happy to default to putting it into an okhttp-twitter module in a new repo.
Would you be against adding generic hooks to okcurl to allow using authenticators defined by external projects?  There are some improvements I'd like to add to okcurl that are generic, even if you have different backends to talk to.
I'm used to have tools like this for testing APIs, okcurl seems like a promising starting point.

How'd you feel about configuring an OkHttpClient instance just add you like it, then having okcurl use that instance? We'd just need a mechanism to manage the command line parameters with a prepared OkHttpClient instance.

Sounds good.  Some of the other changes I was thinking of adding were support in okcurl for the Cache and putting the right JARs on the boot classpath for SPDY or HTTP/2 support based on your environment.
Essentially I'd like to be able reproduce the behaviour that mobile clients would see from the command line.
Are these significant enough that I should do them in a separate okcurl, let's call it "greatcurl" for now. :)


Disable networking.firewall by default

This got a bit off-topic in #12927 by my comment against having it enabled by default, so let's continue here instead.
As stated in the comment, we really should focus on not opening ports that are not needed to world instead of putting additional netfilter/conntrack modules in front that will increase the attack vector.
After all, if you want a service to be available to the world, the netfilter can't help you either and if you don't want it to be available, why enable the service (or listening to ::) in the first place? Everything that we might want to block or disable can be disabled without relying on netfilter, except a few very specific corner cases (like for example port knocking) which where people can still enable it.
The pull request still lacks documentation in the release notes and is primarily there for discussion, in addition to that I'd like to add a few tests (and module fixes) here to ensure that we don't listen to ports that are unnecessary.

I still wouldn't say I'm a fan of turning it off by default, since I don't really see a strong argument for doing that. I also don't really understand the argument that turning iptables on will increase the attack surface. It seems far easier to trust that one module than dozens of ad-hoc services and individual users sitting on your computer.

The netfilter modules had vulnerabilities in the past, but that's not the point, because every piece of software has its bugs, but adding more stuff instead of removing stuff is the same way as AV vendors are approaching this, by blacklisting stuff that's known to be bad instead of whitelisting stuff that's known to be good.
As for the netfilter, the "dozens" of ad-hoc services are only relevant for desktop systems if users aren't careful enough to disable them in the first place, on dedicated servers however there shouldn't run any "ad-hoc services" which you don't explicitly enable in your NixOS config.
And even if it comes to "ad-hoc services" in terms of nixpkgs, we should rather make listening to these ports optional than make it the default.
For example if you have an IM client and want to transfer files, you may actively want to open ports and the only thing the netfilter is doing is making it even harder for you to actually do what you want to do.

What I meant by dozens isn't that you're simultaneously running dozens of services, but it's a single reasonably well known and tested entity blocking entry vs. a suite of codebases of varying quality, even if you're only ever enabling one of them at a time. If I'm spinning up a service, I don't want to have to trust that your netstat listener catching code catches all cases, or that the service has been configured properly to not listen on my public IP, or that it might start listening on my outbound IP after a few minutes because it wants to index some large data file first. The firewall may not be perfect, but gives me assurance that I don't want to have to get through a handful of other imperfect systems that are likely less well tested.

That netstat listener catching code was only an example intended for example for NixOS VM tests to make sure we have good quality modules that do not bind on ports that are unnecessary.

Then I'm even less confident in this being a good idea. I don't want to trust a whole bunch of module authors to have done this properly and have working VM tests that catch this. Most services in NixOS don't have VM tests, and while I'd like that to improve, this seems like a lot more potential holes in exchange for a bit of convenience when running p2p desktop sharing programs.

Previous discussion: https://github.com/NixOS/nixpkgs/issues/2135, https://github.com/NixOS/nixos/issues/55
I don't really see what changed, so I'm :-1: on this - I don't think we should ship without a firewall by default. People who care can disable it easily enough.

I agree with @edolstra

@edolstra: Still your argument on defense in depth really doesn't hold, because it does not protect services that are already exempt from firewall rules. It only protects you from rogue programs that are randomly binding to everything. Programs like this should not run on your machine and if they're really necessary you'd exempt them anyway... or even sandbox them properly.

Another problem caused by the firewall being on by default: #10101

@aszlig What if such rogue program is some malware?

@astsmtl: If malware is on your system, then you're doomed anyway and it can still use outbound connections as well as bind to ports that are exempt from the rules. And if it has root access it can even disable these rules directly.

:+1: from me. We should have as little as possible enabled by default; on a properly configured system, netfilter doesn't add any security.

This has been discussed before and we've through a lot of trouble at the time to enable the firewall by default in a way that doesn't break people's existing installations. I believe having the firewall enabled by default was the right choice then and it's still the right choice now. :-1: from me on this issue; please don't change the current default setting.

Please just no. :-1:
Is implementing https://github.com/t-woerner/firewalld one way to fix this? (Is it maybe not an ideal fit for NixOS because of it's design?) firewalld integrates well with network manager and allows you to have multiple switch-able zones (which are themselves individual configuration rule sets) which can be toggled without any of the reloading of kernel modules normally required to complete configuration changes.

After all, if you want a service to be available to the world, the netfilter can't help you either and if you don't want it to be available, why enable the service (or listening to ::) in the first place?
Many have often wondered why we have this thing called dbus which does; things.
People are also asking why we need kdbus which as I understand it; does some other things.
Try adding a portcullis and a retractable bridge to each of the gates in your castle analogy (as would be the norm for most medieval castles) if you follow.

Ports in applications or services are dynamic and therefore the port itself does not describe the use case.
From:
https://fedoraproject.org/wiki/FirewallD#Port_metadata_information_.28proposed_by_Lennart_Poettering.29
Whether to enable a firewall is often a question found in an operating system installer and I am yet to  use an installer where the option is disabled by default.
I think that if the consensus was that firewalls did not work then no one would use them anyway.

:+1: from me. If people want a firewall they can just as easily enable it.

Many have often wondered why we have this thing called dbus which does; things.
People are also asking why we need kdbus which as I understand it; does some other things.
Try adding a portcullis and a retractable bridge to each of the gates in your castle analogy (as would be the norm for most medieval castles) if you follow.

In case of dbus, we need it because systemd depends on it, so it is not an unnecessary dependency.
And the reason why most installers of other distros enable it by default is that they tend to enable services on installation (like for example Debian does or at least has done in the past). NixOS however doesn't have a notion of "installation" but a configuration.nix where we can exactly state what we want to run.
But anyway, I guess in respect to #12987, let's at least get rid of nf_conntrack, because that's a module we really don't need for just blocking ports programs may run at random.

I am in favor of disabling the firewall by default.
Back as a new user I was astounded that my python -m http.server &c. were not working. After some search I found out about the firewall, but it induced a considerable amount of stress.
I think the principle of least astonishment should be applied in this case.

We just discovered that the firewall blocks avahi as well. avahi-browse doesn’t find any services for example, without any notice. That is very bad for the end user.

That is very bad for the end user.
I disagree. It is doing exactly what was intended.

I imagine there's a fairly clean split on this issue between people using NixOS as a server vs. those using it as a desktop OS.

I disagree. It is doing exactly what was intended.

So blocking avahi functionality (with services.avahi enabled) is the intended behavior? I think not.
What do you want to do about that? Implicitly disable networking.firewall when enabling services.avahi? That would be the only sensible thing to do in that case. Yet …

So blocking avahi functionality (with services.avahi enabled) is the intended behavior? I think not.

When a firewall is turn on it blocks all traffic by default. This is the intended behaviour.
I am sorry that you are unable to discuss this rationally.
I don't want to do anything about that. I also do not want to discuss this any further.
This thread is becoming silly.

The problem is that this firewall is turned on without the user even knowing about it, it's kind of hard to suspect breakage due to an option you didn't even know exists.

I was going to suggest to make the nixos installer put networking.firewall.enable = $WHATEVER into the initial configuration.nix file, because then I don't think the default value matters as much (the user will become aware of this important setting and set it to his/her liking.
But then I realized that this is what this PR does.
So +1 from me. (I don't care that much about the default setting, but it clearly needs to be more visible, and this PR does that.)

@bjornfor's point is an important one on this topic in my opinion.
:+1: from me also.

FWIW, I agree with the points made by @bjornfor and @aszlig, so :+1:.
Enabled firewalls by default are only useful if you as the user don't know what you're and the package maintainer doesn't know what he's doing, but this is NixOS and not Fedora, so I guess not that many newbies who don't know about firewalls will use it. Enabling it explicitly and by default in the generated configuration.nix for new installs is the way to go because this is be the first thing a new NixOS user will try and thus will become aware of the firewall. But advanced users that build qemu images or deploy via nixops should know if they need the firewall or not and thus set it explicitly.
As a compromise for the people who are against this: Should/could we demand that all services listen on localhost by default except when they can be considered to be secure enough in its default state, e.g. sshd has password authentication disabled so it's quite safe to have it listening on all public IPs.

The change as-is now works fine for people who install a fresh 16.03 system, but people who upgrade to that release from an existing installation might implicitly have their firewall disabled without noticing.

@peti Then let’s put it in a big fat warning box in the release notes.

I don't see a consensus for changing this again with no new arguments (we really don't want to turn the firewall on and off every few releases), and the firewall can be turned off easily for people who don't want it. Since this discussion is likely to go around in circles for a long time I'm going to close this issue. However, we could add a networking.firewall.enable line to the nixos-generate-config template to make it easier to disable.
Aside: this is not an appropriate use of system.stateVersion, which is only intended to preserve legacy behaviour for stateful data on disk (such as SSH host keys or PostgreSQL databases), not to make a NixOS release behave like an older version with respect to general option defaults. Because then you could have situations where destroying and recreating an EC2 instance originally upgraded from 15.09 to 16.03 would suddenly no longer have a firewall. Maybe we need to have a system.configVersion attribute for the purpose of setting some option defaults to legacy values.


Validate method after compilation almost empty

Hello!
I am implementing the library into an Ember app.
For some reason, after adding draft-04/schema, draft-04/hyper-schema and my own schema the validate function returned by the compilation function is almost empty and only check that the schema passed is an object, but it doesn't check each element.
I also tried the validate function and it returns the same.
On a side note, I tried compileAsync and AJV was looping indefinitely over the hyper-schema URL.
Please let me know if you need clarity on any element or if I can be of any help in the testing process.
Thank you very much!
What version of Ajv are you using? Does the issue happen if you use the latest version?
{ "ajv": "^5.2.2" }

Ajv options object

{
    allErrors: true,
    meta: false,
    extendRefs: true
}

Your code

// Ember code, I'm just putting the basics of what could be useful

this.get('ajax').request(
    'http://json-schema.org/draft-04/schema',
    { method: 'GET' }
).then(function(metaSchema){
    self.get('validator').addMetaSchema(metaSchema, undefined, true);
    self.get('ajax').request(
        'http://json-schema.org/draft-04/hyper-schema',
        { method: 'GET' }
    ).then(function(metaHyperSchema){
        self.get('validator').addMetaSchema(metaHyperSchema, undefined, true);
        self.get('validator').removeKeyword('propertyNames');
        self.get('validator').removeKeyword('contains');
        self.get('validator').removeKeyword('const');

        self.get('validator')._refs['http://json-schema.org/schema'] = 'http://json-schema.org/draft-04/hyper-schema';

        var validate = self.get('validator').compile(schema);
        console.log(validate);
    });
});

JSON Schema

The last JSONSchema loaded is:
https://raw.githubusercontent.com/gudTECH/retailops-sdk/master/verify/schema/catalog_get_config_v1.json
// The value of schema is the content of: https://raw.githubusercontent.com/gudTECH/retailops-sdk/master/verify/schema/catalog_get_config_v1.json

var validate = self.get('validator').compile(schema);

Response received from the console log
(function(self, RULES, formats, root, refVal, defaults, customRules, co, equal, ucs2length, ValidationError/*``*/
) {
    var validate = (function(data, dataPath, parentData, parentDataProperty, rootData) {
        'use strict';
        var vErrors = null;
        var errors = 0;
        if ((data && typeof data === "object" && !Array.isArray(data)) ) {
            var errs__0 = errors;
            var valid1 = true;
        } else {
            var err = {
                keyword: 'type',
                dataPath: (dataPath || '') + "",
                schemaPath: '#/type',
                params: {
                    type: 'object'
                },
                message: 'should be object'
            };
            if (vErrors === null)
                vErrors = [err];
            else
                vErrors.push(err);
            errors++;
        }
        validate.errors = vErrors;
        return errors === 0;
    });
    return validate;
}
)

This is the correct behaviour because "links" is not a validation keyword, so everything inside "links" is ignored. If you want to validate against the schema inside "links" object, you can either compile it directly (by passing a sub-schema object to .compile method) or add "id" attribute to your schema and access the sub-schema using JSON pointer with .getSchema method.
Also, there is no need to download draft-4 meta-schema, it's available as the part of ajv package as shown here: https://github.com/epoberezkin/ajv#using-version-5

Thank you very much for your response!
I will look into that!


Unexpected end of JSON input with retrieving emissions records after recent updates

After recent updates, I'm having an "Failed to evaluate transaction: SyntaxError: Unexpected end of JSON input" error after registering the org admin, the user, creating an emissions record, and then trying to retrieve the emissions record.

To test this, you will need to copy egrid-data-loader/emissions-calc.js to /chaincode/node/lib/ before deploying the chain code.

Hi Si, thanks a lot for reporting. I could reproduce the error. The error message occurs because getEmissionsData produces the lookup key with the wrong date. Have a look here
In recordEmission you set dates as follows:
"fromDate": "2020-09-01 00:00"
"thruDate": "2020-09-30 23:59"
in getEmissionsDatayou set dates differently:
"fromDate": "2020-09-01 00:00:00"
"thruDate": "2020-09-30 23:59:59"
Thus, the chaincode returns a SyntaxError.
I recommend putting the date in a standard format like the number of milliseconds between midnight of January 1, 1970 and the specified date.
I close this error since chaincode and REST API are working as expected.

  Here is what it looks like now.  Thanks!


Package test failures

The packages test fail in Linux with undefined symbol in the shared library. (The tests can be run by commenting out link("../test/package") from test/test.sn).
Packages test also require the default Slogan config path to exist. (A workaround for this is to first run make install).
It is better to use a simpler ffi library for tests.

Fixed in commit e496b0bd4aac1223b6b75ab1f643a6498240a1e0


Updating instructions

Removing text describing the option to run nuget pack from root. It doesn't work anymore.
Also, the nupkg file is now created at the same level as the csproj file, updating that line.

dotnet pack on the sln is the preferred way to pack all projects. Do you want to mention that?

Yeah, we should mention that you can do a dotnet pack on the sln file in the root to pack all projects.

My bad. I'm closing the PR. I was using 'nuget pack'. I need more cofee.
Thanks @ViktorHofer @safern

Take @danmosemsft with you 😁


⚠ RPC server - Conflux has degraded performance

In 9651d78, RPC server - Conflux ($CONFLUX_RPC_URL) experienced degraded performance:

HTTP code: 200
Response time: 915 ms

Resolved: RPC server - Conflux performance has improved in 4a8fbac.


Uncaught TypeError: Cannot read property 'downloadFile' of undefined

Steps:

Open file
Right click -> Remote Sync Pro -> Download file

Atom: 1.22.1 x64
Electron: 1.6.15
OS: Mac OS X 10.12.6
Thrown From: remote-sync-pro package 5.1.1
Stack Trace
Uncaught TypeError: Cannot read property 'downloadFile' of undefined
At /Users/macbook/.atom/packages/remote-sync-pro/index.coffee:45

TypeError: Cannot read property 'downloadFile' of undefined
    at handleEvent (/packages/remote-sync-pro/index.coffee:45:28)
    at /packages/remote-sync-pro/index.coffee:108:46)
    at CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app/src/command-registry.js:381:36)
    at CommandRegistry.dispatch (/Applications/Atom.app/Contents/Resources/app/src/command-registry.js:266:23)
    at AtomEnvironment.module.exports.AtomEnvironment.dispatchContextMenuCommand (/Applications/Atom.app/Contents/Resources/app/src/atom-environment.js:1388:34)
    at EventEmitter.outerCallback (/Applications/Atom.app/Contents/Resources/app/src/application-delegate.js:348:31)
    at emitThree (events.js:116:13)
    at EventEmitter.emit (events.js:194:7)

Commands
     -0:28.4.0 remote-sync-pro:download-file (div.line.cursor-line)

Non-Core Packages
atom-autocomplete-php 0.25.6
atom-html-preview 0.2.5
atom-live-server 2.2.0
better-git-blame 0.3.2
busy-signal 1.4.3
coffee-compile 0.28.1
docblockr 0.13.2
file-icons 2.1.14
git-control 0.9.0
goto-definition 1.3.4
intentions 1.1.5
linter 2.2.0
linter-eslint 8.4.0
linter-ui-default 1.6.10
markdown-preview-plus 2.4.16
minimap 4.29.7
nord-atom-syntax 0.9.1
nord-atom-ui 0.11.0
pigments 0.40.2
Remote-FTP 2.0.0
remote-sync-pro 5.1.1

Bug appears when you are not yet connected to remote

How to resolve it? One year go and this bug still appears.

Bug appears when you are not yet connected to remote

Maybe, you lost .remote-sync config file?


[Qustion] How to decode raw h264 stream?

I'm trying to write a go implementation of a scrcpy client where the scrcpy server sends a series of raw h264 data (nalu) to the socket connector. In the official C implementation, they do this by using ffmpeg to decode the NALU packets in memory. Since my C skills are poor, I can't convert their code to a Go implementation, so some advice, help or examples would be appreciated.
Also: any thoughts on opening a "discussion" section in the repository? It would help to have a proper place for help questions like this.
Thanks :)

I'm trying to write a go implementation of a scrcpy client where the scrcpy server sends a series of raw h264 data (nalu) to the socket connector. In the official C implementation, they do this by using ffmpeg to decode the NALU packets in memory. Since my C skills are poor, I can't convert their code to a Go implementation, so some advice, help or examples would be appreciated.

You could start by checking the demuxing_decoding example and replace is.CodecParameters().CodecID() with astiav.CodecIDH264 here. There might be other things to change, but that should be good start 👍

Also: any thoughts on opening a "discussion" section in the repository? It would help to have a proper place for help questions like this.

 I've created the discussions area 👍

You could start by checking the demuxing_decoding example and replace is.CodecParameters().CodecID() with astiav.CodecIDH264 here. There might be other things to change, but that should be good start 👍

Thanks, I will try and keep this issue open until I solve my needs and attach my solution here.
(I used the wrong account when asking this question, sorry)

Hi. I had a similar need, although I am decoding HEVC (which arrives over 0mq within a capnproto container). The advice from @asticode helped a lot. I want to mention that I spent a lot of time fighting with goav first, but it was a breeze with go-astiav and the referenced example.
My code at this point in time with successful decoding routine is linked here, maybe it will help you out @Tryanks https://github.com/comma-hacks/webrtc/blob/f06088234fa29e1aa06a077f5189a5814139cb4b/main.go

Hi. I had a similar need, although I am decoding HEVC (which arrives over 0mq within a capnproto container). The advice from @asticode helped a lot. I want to mention that I spent a lot of time fighting with goav first, but it was a breeze with go-astiav and the referenced example.
My code at this point in time with successful decoding routine is linked here, maybe it will help you out @Tryanks https://github.com/comma-hacks/webrtc/blob/f06088234fa29e1aa06a077f5189a5814139cb4b/main.go

Thanks! It was very helpful for me, I only needed to replace the CodecID and I successfully decoded the bare stream.
I noticed that to wrap the bare data as AVPacket, I need to call the pkt.FromData method, my next step is to encode the decoded AVFrame as MJPEG and provide the bare stream, does it mean I can call the pkt.Data() method of the re-encoded AVPacket to achieve my purpose?

my next step is to encode the decoded AVFrame as MJPEG and provide the bare stream, does it mean I can call the pkt.Data() method of the re-encoded AVPacket to achieve my purpose?

Take a look this function of the transcoding example, it specifically takes an input frame, encodes it as a packet and muxes the packet into a file.
Let me know if you're still having issues.

I took care of everything! Decode, encode, reuse ......
Now the rest of my work is to finish go-scrcpy and to organize all the solutions and write a simple guide.
Thanks for all the help along the way!


Linguist api docs

Hey, I just made a Pull Request!
I've been getting into the API docs functionality that has been added for plugin backends and would like to add API specs for the linguist backend.
:heavy_check_mark: Checklist

[X] A changeset describing the change and affected packages. (more info)
[X] Added or updated documentation
[X] Tests for new functionality and regression tests for bug fixes
[X] Screenshots attached (for UI changes)
[X] All your commits have a Signed-off-by line in the message. (more info)

Hi @bforbis, I've been pretty busy as of late but wanted to share that I have seen this.
I'm currently a bit torn on this one as I often point to this plugin as a good example of patterns. I like that it's very close to the default plugin structure. I also personally take a code first approach to development and this would be a change for me.
I'll keep thinking on this though, I'm not totally on the side of saying no, again just didn't want to keep you in the dark on my thinking.

@awanlin, thanks and no problem on being busy!
I am just copying the patterns that exist for catalog, search, and events right now. I don't have a lot of opinions on whether its spec first vs code first, just wanted the ability to see the linguist API in my stitched together backstage OpenAPI spec.
The API for linguist is simple enough that switching between either approach won't be too much effort. I think openapi tooling for backstage will continue to evolve, so if some better code-genarated API spec tooling is available in the future this could be ported to it.

Dropped the optic.yml file as it is not needed and additionally rebased off main to fix package / lock conflicts.

One more fix for yarn dedupe

@bforbis, if you can fix the conflicts that crept in I'll follow up with approval and merging 👍


Fix some deprecation warnings that were missed in the 6.0.0 release

What does this pull request do?
Happened to notice these DeprecationWarnings weren't updated when we released 6.0.0. This simplifies the warnings and the tests.

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2023-01-26T22:28:27.162+0000

Duration: 19 min 18 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
5081

Skipped
3679

Total
8760

:green_heart: Flaky test report
Tests succeeded.
:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

/test linters : Run the Python linters only.

/test full : Run the full matrix of tests.

/test benchmark : Run the APM Agent Python benchmarks tests.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

:globe_with_meridians: Coverage report

Name
Metrics % (covered/total)
Diff

Packages
100.0% (73/73)
:green_heart:

Files
100.0% (241/241)
:green_heart:

Classes
100.0% (241/241)
:green_heart:

Lines
91.919% (18746/20394)
:+1: 0.002

Conditionals
74.98% (2877/3837)
:-1: -0.013


Custom installation PATH

Is it possible to make the installation PATH in another directory? e.g., /opt/ instead of /usr/local/?
TIA
Joven

For example:
sudo mkdir /opt/pgbadger
perl Makefile.PL DESTDIR=/opt/pgbadger
make
...
sudo make insall
...
Installing /opt/pgbadger/usr/local/man/man1/pgbadger.1p
Installing /opt/pgbadger/usr/local/bin/pgbadger
...


[Feature Request] Write data only with Tag key and without Field Key

Feature Request
Proposal:
Sometimes there's a need to have a measurement with only one tag. Just do the count of events.
Current behavior:
[uisadm@vuis-test ~]$ curl -i -XPOST 'http://vuis-test:8086/write?db=testdb' --data-binary 'LOANREPORT,credittype=IL<PHONE_NUMBER>000000000'
HTTP/1.1 400 Bad Request
Content-Type: application/json
Request-Id: dc2c3a47-bb8a-11e6-98a3-000000000000
X-Influxdb-Version: 1.1.0
Date: Tue, 06 Dec 2016 08:06:15 GMT
Content-Length: 97
{"error":"unable to parse 'LOANREPORT,credittype=IL<PHONE_NUMBER>000000000': invalid field format"}
To have a success write we need to add fake field with value=1
[uisadm@vuis-test ~]$ curl -i -XPOST 'http://vuis-test:8086/write?db=testdb' --data-binary 'LOANREPORT,credittype=IL value=1<PHONE_NUMBER>000000000'
HTTP/1.1 204 No Content
Content-Type: application/json
Request-Id: f044f9d7-bb8a-11e6-98a6-000000000000
X-Influxdb-Version: 1.1.0
Date: Tue, 06 Dec 2016 08:06:49 GMT
Desired behavior:
Write should be successful without definition any Field Key.
Use case:
In my case there's only one tag and we want to do the COUNT(*) where tag_name='tag_value' to see how much of events happens. And just one tag, which is indexed, it's enough.

Oh, I see that COUNT() function work only with fields.
This would work
SELECT COUNT(value) FROM LOANREPORT WHERE credittype='IL'
But doesn't with tag
SELECT COUNT(credittype) FROM LOANREPORT WHERE credittype='IL'

Yea, the storage engine only stores fields with a time. Tags are stored as part of metadata and can't be queried because they are indexing markers rather than fields written.
I don't think we have a way to just write a marker for a time period. You might want to just write a boolean true as a sentinel value instead. Then you can use count() on that field.
I'm going to close this. If you need any help with InfluxDB, ask your question on the InfluxDB mailing list. If you want me to reopen this issue for whatever reason, just comment here and I'll reopen it.


fibonacci_multicore: Fatal error: exception Stdlib.Effect.Unhandled(Domainslib__Task.Wait(_, _))

I was running the fibonacci_multicore example in code/task. I got the following:
/tmp/l/github/parallel-programming-in-multicore-ocaml/code/task $ OCAMLRUNPARAM=b ./_build/default/fibonacci_multicore.exe 1 41
Fatal error: exception Stdlib.Effect.Unhandled(Domainslib__Task.Wait(_, _))
Raised by primitive operation at Dune__exe__Fibonacci_multicore.fib_par in file "fibonacci_multicore.ml", line 15, characters 24-41
Called from Dune__exe__Fibonacci_multicore.main in file "fibonacci_multicore.ml", line 19, characters 12-52

I think this is probably not expected. What is going wrong? Thanks

@Sudha247 can you take a look at this?

Thanks for the report @tomjridge! #16 fixes it.


fix Matador queue display bug

We have queues with : in the queue name, which Matador fails to show correctly and also fails to let you do delete, re-activate, etc operations on. Credit to @d2marcelo for the fixes.

@d2marcelo your fix was merged in to Matador master

@d2marcelo Also, it's live in v1.1.0 on npm, thanks for the PR


feat(openapi): add scalar schema documentation

Add scalar api docs

Closes #2846

im not sure why this is, but i think its causing the examples to fail loading
 Maybe @marclave has an idea 😄

ah! this is a bug when the server url isn't set properly.. we should better handle this edge case.
I'll pull and checkout this branch and see if I can fix it up

awesome - thanks!
Usually we would get this merged in pretty quick but we have had a planned OpenAPI refactor for a little bit so it may be a bit before this goes in. I just saw it on Reddit and got excited 😄

@JacobCoffee got a fix incoming :)
https://github.com/scalar/scalar/pull/644

@JacobCoffee new version released with the fix!✨

@JacobCoffee The pyright failures come from develop. We'll need to investigate and fix it there

Closing this in favor of #2906 and #2893


missing example in the user guide for time constraints

To my mind the Iris user guide skips a step in discussing time constraints. After the example with cube_11 = cube_all.extract(hour_11), an example where you have 2 dates in datetime format and want to constrain the data to be between these two dates seems a really common use case.
something like:

d1 = datetime.datetime.strptime('20070416T0000Z', '%Y%m%dT%H%MZ')
d2 = datetime.datetime.strptime('20070430T0000Z', '%Y%m%dT%H%MZ')

daterange = iris.Constraint(time=lambda cell: d1<= cell.point <= d2)
data_subset = long_ts.extract(daterange)

Do other people think that this would be a useful addition? I think it is probably a fairly common use case and I got a bit confused by the discussion of partial datetime which cool but not needed here. I am happy to give this a go if people want it.

Thanks for taking the time to highlight this @nhsavage.
I think that this would make a nice addition to the user guide. It's a small example, but like you said, is a very common use case. So I think that many users would fine it both useful and helpful.
If you pushed up a PR, I'd merge it :smile:

@stephenworsley Reference 2.3.2 Constraining on Time from the Iris documentation.

A example for constraining to a datetime range has been added in #3352.


How to remove previous | next from pages?

The previous | next link appears on posts as well as pages.

Is there any way to make it appear on posts only?
How can it be completely disabled?

Repository - https://github.com/Kulbhushan-Chand/Kulbhushan-Chand.github.io
Following is the link, what I 'm talking about -

pagination settings in _config.yml -

collections settings in _config.yml -

folder structure -

It's part of the single layout. There is no way to turn it off.
I'd suggest creating a variant of that layout or the post_pagination.html include if you want to omit those links.


[Playground][Task]: Adding the Cloud Datastore client to save playground examples

What needs to happen?
After editing the Python scripts and removing the cloud storage client, we must use the Cloud Datastore client
Issue Priority
Priority: 2
Issue Component
Component: beam-playground

.take-issue


Fleet color theme issue

We noticed that you've been working on a custom theme for JetBrains Fleet on your Git repository. After reviewing the current documentation, we wanted to bring a few important points to your attention:

It looks like the colors in your theme are not in the correct format. According to the official documentation, all color values should be in the #rgba format.
Some of the keys used in your theme are outdated and no longer supported. To ensure your theme works correctly, we suggest checking the latest version of JetBrains Fleet by editing either the Light or Dark theme to view the most up-to-date keys (Edit Color Theme action).
Updating these aspects should help improve the compatibility and performance of your custom theme. If you need any further assistance, please don't hesitate to reach out.

Thank you

@OljaKli, Thank you.  Maybe in future if there is a way how to validate theme by Gradle plugin.


feat: 상세 모달 윈도우 레이아웃을 구현하라

global portal 컴포넌트 구현
장소 상세 모달 윈도우 레이아웃 구현

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.47 :tada:

Comparison is base (9b21cb5) 97.05% compared to head (28a4a08) 97.53%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #84      +/-   ##
==========================================
+ Coverage   97.05%   97.53%   +0.47%
==========================================
  Files          11       14       +3
  Lines         136      162      +26
  Branches       18       22       +4
==========================================
+ Hits          132      158      +26
  Misses          4        4

Impacted Files
Coverage Δ

components/common/GlobalPortal/index.tsx
100.00% <100.00%> (ø)

hooks/useIsMounted.ts
100.00% <100.00%> (ø)

utils/test/renderWithPortal.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


dateformat not working with "require" ?

using import I get this error
SyntaxError: Cannot use import statement outside a module
while using require this error
Error [ERR_REQUIRE_ESM]: require() of ES Module /var/www/html/wikisphere-nodejs/node_modules/dateformat/lib/dateformat.js from /var/www/html/wikisphere-nodejs/fetch.js not supported.
Instead change the require of dateformat.js in /var/www/html/wikisphere-nodejs/fetch.js to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/var/www/html/wikisphere-nodejs/fetch.js:30:18) {
  code: 'ERR_REQUIRE_ESM'

any hint ?
(node v16.13.0
npm 8.1.0)

the article below does a pretty good job of explaining options for consuming ES Modules from within CommonJS.
https://regbrain.com/article/how-to-run-es-packages-in-commonjs-modules
TL;DR, you can:

keep old versions of dependencies
move entire application to ES
tactically integrate ES modules into CommonJS code

Been using this for years. Disappointing change. Not seeing how this benefits things at all.
I do not need to async load my date formatter. I'm gonna toss that in a const and ref it where I need to when I need to. Like at the start of the process. Once. Its already gonna be on disk. I'm not waiting for some http call to get it.
Why would I go through these hoops just to 'use ES modules ooohh aaahhh' like some kind of epeen stroke...
Moving on to some other date formatter that does not require annoying hoops to use.

I am now unable to import a module that uses dateformat from a Jest test, because Jest support for ES6 modules is still experimental and requires even more hoops to jump through. Thank you for providing this package, I would love to keep using it, but finding an alternative that keeps working with current versions of the tools we use is currently more acceptable than switching the whole project to an experimental mode of a critical UT framework we use.
tl;dr While I applaud your effort to support ES6, which is the future, it's still too early to completely drop CommonJS support.

@chase-manning I have opened #183 to add support for cjs

Been using this for years. Disappointing change. Not seeing how this benefits things at all.
I do not need to async load my date formatter. I'm gonna toss that in a const and ref it where I need to when I need to. Like at the start of the process. Once. Its already gonna be on disk. I'm not waiting for some http call to get it.
Why would I go through these hoops just to 'use ES modules ooohh aaahhh' like some kind of epeen stroke...
Moving on to some other date formatter that does not require annoying hoops to use.

I 100% agree this is a rather annoying change and doesn't seem to create any value.

I encountered the same problem. How can this be fixed?
Error [ERR_REQUIRE_ESM]: require() of ES Module /home/max/pool/node_modules/dateformat/lib/dateformat.js from /home/max/pool/lib/logger.js not supported.Instead change the require of dateformat.js in /home/max/pool/lib/logger.js to a dynamic import() which is available in all CommonJS modules.


feat: support environment variable in config file

Sorry, the https://github.com/apache/apisix-ingress-controller/pull/713 closed because I wrongly rebase the upstream code, so now I have initiated a new PR. e2e has been added, can be review again
Please answer these questions before submitting a pull request

Why submit this pull request?

[ ] Bugfix

[x] New feature provided

[ ] Improve performance

[ ] Backport patches

Related issues
#710

@tao12345666333

Please resolve the conflicts, thanks!

Codecov Report

Merging #745 (2daf86c) into master (4a862e2) will increase coverage by 0.08%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #745      +/-   ##
==========================================
+ Coverage   31.70%   31.78%   +0.08%
==========================================
  Files          66       65       -1
  Lines        6640     6651      +11
==========================================
+ Hits         2105     2114       +9
- Misses       4280     4281       +1
- Partials      255      256       +1

Impacted Files
Coverage Δ

pkg/config/config.go
65.00% <85.71%> (+2.50%)
:arrow_up:

test/e2e/e2e.go

Continue to review full report at Codecov.

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

The code LGTM. Thanks!
But I personally prefer to directly support configuration through environment variables or through configuration files
Instead of supporting environment variables in the configuration file.
WDYT? @gxthrj @tokers @lilien1010

The code LGTM. Thanks!
But I personally prefer to directly support configuration through environment variables or through configuration files Instead of supporting environment variables in the configuration file.
WDYT? @gxthrj @tokers @lilien1010
In the case that the user chooses to use file configuration instead of commands of container, the use of environment variables is to write sensitive information through downward api and secret.  So if the program directly uses env, we need to consider the problem of file configuration and env configuration merging,  doesn't make this problem easy.

The code LGTM. Thanks!
But I personally prefer to directly support configuration through environment variables or through configuration files Instead of supporting environment variables in the configuration file.
WDYT? @gxthrj @tokers @lilien1010

Softwares like ETCD and Istio support using environment variables directly, and the precedence is highest, so it's my understanding that we do should support environment variables, configuration file and command line options.

@tao12345666333 @tokers
Should we merge the current feature first, and add more options later?

@tao12345666333 @tokers Should we merge the current feature first, and add more options later?

Agree, or this PR might be too large to review.

Let's move forward, this feature also has some usage scenarios.

ping @gxthrj @tokers  for review & merge.


Tests for ListBlobs + Recordings

Added 8 tests to increase code coverage for ListBlobs as it is impacted by service version bump to 85/86

[x] The purpose of this PR is explained in this or a referenced issue.
[x] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[x] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[x] MIT license headers are included in each file.

/azp run go - azblob


Adding a replacement macro for the snowplow uuid within a redirect url.

In order to perform ID syncing, we needed the ability to inject the snowplow id within a redirect url.
I decided on a replacement key of ${SP_UUID}. Not sure what kind of documentation needs to change.

No need to merge this.


Define OpTransaction

Describe the feature
Define type reth_optimism_primitives::OpTransaction, which should implement reth_primitives_traits::Transaction and reth_codecs::Compact
Additional context
No response

Hey @emhane! Would love to work on this one!

hey @emhane, do you want me to completely implement the OpTransaction? If so, is there another struct I could get some inspiration regarding the fields it should contain?
I've got the skeleton of the implementation done right now with trait methods as todos

hey @emhane, do you want me to completely implement the OpTransaction? If so, is there another struct I could get some inspiration regarding the fields it should contain? I've got the skeleton of the implementation done right now with trait methods as todos

good question! it should just be a wrapper type
pub struct OpTransaction(op_alloy_consensus::OpTypedTransaction)


Can I apply this to depth estimation?

Hello! Thank you for your great work.
I was very impressed by your use of grad-cam for semantic segmentation.
Is it possible to apply to the depth estimation, which is pixel-wise prediction without class?

Yes.
So is the network doing regression of the depth basically ?
You can just use this target:
https://github.com/jacobgil/pytorch-grad-cam/blob/master/pytorch_grad_cam/utils/model_targets.py#L21
Pass category=1 if you want to visualize what causes a larger depth, and category = 0 if you want to visualize what causes a smaller depth.


Need warning about building the container, I think....

I get the impression that the config directory is now included in the orchestrator docker image (as opposed to the orchestrator v1, which mounted it from the host).
But that means that if you change anything you must re-run docker compose build. And the README didn't point this out, it states that docker compose up will build the container if needed.
I'm also not 100% sure whether any containers I build are uploaded to our registry, the name seems to suggest so, but I'm not sure.

Yes it is. I agree, I should probably change that.
Yeah, the name indicates that it will be uploaded to our registry if you do docker push

Hmm. And it seems that the .env file is also used at docker compose build time.

You can override .env values using environment variables (or the environment section of docker-compose.yml. But yeah, .env should probably be added to .dockerignore


Catalog refactor

This refactors the catalog to give a simpler, non-PostgreSQL internal API to the rest of the system.  It will still store data in PostgreSQL's catalog layout for compatibility with admin tools that scan the catalog, but it makes it simpler for other parts of the system to read/write metadata in the catalog by not requiring them to learn the internal catalog schema.
The following items need to be finished before merging this PR:

[x] Finish header files for pg_attribute, pg_constraints, and pg_index.
[x] Finalize API for passing expressions (pointer to AbstractExpression vs serialized representation)
[ ] (not required for TPC-C) Finalize API for altering tables (add/drop columns, column type/default/nullable changes, and constraints).
[ ] Implement the core API (everything except altering tables)
[ ] (not required for TPC-C) Implement alter table API

Supersedes #381.
Fixes #380
Fixes #347
Fixes #323
Fixes #313
Fixes #311

Still need to implement initialization of pg_namespace and pg_type in the bootstrap routine (minimal functionality for user tables) and start implementing interface.  Entry objects probably need a rework due to SqlTable's API change.

Partially blocked on #433 because we will need 'ColumnValueExpression' objects to build the indexes.  However, we could dummy populate them until the fix is in.
@pavlo, should we wait to merge until after the expressions are added, or should I proceed with a throwaway key definition (probably null) and have the real definitions commented out with a TODO left to explain?

@apavlo, this should be ready for review.

Thoughts for review:
@amlatyrngom should review the overall API as comparison and guidance for adapting to this API from his fake implementation.
@mush-zhang should review the tests as an example of usage and possible identification of additional cases
@mbutrovich should review the refactor of TPC-C and parts of storage layer that happened from moving IndexKeySchema.

This PR also introduces a flag in TransactionContext that indicates that a transaction cannot Commit. It's currently tripped by a write-write conflict in the SqlTable or a unique-key conflict in the index wrapper. It cannot be set back, and the TransactionManager will not allow a transaction to commit with this flag set.
We didn't spin it off to a separate PR because there were other catalog-related changes in some of the files and undoing those so as to cherry-pick this feature seemed unnecessary with the catalog close to being ready for review.

@mbutrovich How is the "cannot commit" flag different than an abort? Who is responsible for checking that this flag is set before we go off and do more work (e.g., continue to execute subsequent queries in a batch)? What if somebody sets this flag and then never calls abort?

How is the "cannot commit" flag different than an abort?
Separation of responsibilities. The storage layer can trip a flag that this txn can no longer be allowed to commit, however it's the execution layer's job to actually abort in case there are operations it needs to handle (cleanup, etc.). The storage layer can't call abort directly.

Who is responsible for checking that this flag is set before we go off and do more work (e.g., continue to execute subsequent queries in a batch)?

Currently no one. I proposed having the TransactionContext not allow to stage any more writes based on this flag (since in theory a txn could keep shipping out logs even though it has no possibility of committing now) but @jrolli suggested it wasn't needed at this time. Conversation is still open on this flag right now, it just prevents committing.

What if somebody sets this flag and then never calls abort?
It would be no different any a transaction calling begin and never calling either abort or commit. It would just be hanging out in memory.


What is the correct WETH-USDC Pool on Ropsten?

I have tried both of these for pools:
//address WETHUSDCPool = 0x88e6A0c2dDD26FEEb64F039a2c41296FcB3f5640;
from https://etherscan.io/address/0x88e6a0c2ddd26feeb64f039a2c41296fcb3f5640#readContract -- this looks like the right pool from searching

  address WETHUSDCPool = 0x8ad599c3A0ff1De082011EFDDc58f1908eb6e6D8; //?? linked from uniswap
from https://info.uniswap.org/#/pools/0x8ad599c3a0ff1de082011efddc58f1908eb6e6d -- this one I get from UNISWAP

I always get an error consulting the Uniswap V3 Oracle Library with these pools, it reverts which it can do if there are no reserves.
What is the correct WETH-USDC Pool? Thank you!!

The info I was getting from uniswap was a mainnet pool despite being connected to Ropsten in my metamask. Hm. How do I find the pool for WETH-USDC on Ropsten?

I found https://ropsten.etherscan.io/address/0x1F98431c8aD98523631AE4a59f267346ea31F984#code
and using  values
0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48 (USDC Ropsten)
0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2 (WETH Ropsten)
and 3000 for the pool rate
I got "0x8ad599c3A0ff1De082011EFDDc58f1908eb6e6D8"
but this address on ropsten explorer looks strange. Is this the right pool?


Set up CI with Azure Pipelines

[skip ci]

 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.:x: izabelawlodarska sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


[ZF3] Add two-way hydration for Zend\Db\TableGateway

Currently TableGateway only supports one-way hydration for data reading. I would love to have TableGateway also handle the hydration for writing data (insert/update). Currently we have insert() which expects an array to be passed or insertWith() which expects an Zend\Db\Sql\Insert instance. I would like to get an insertObject() or insertEntity() method which uses an injected hydrator instance to fetch the array data from the object.
Currently, my TableGateway returns my entities when data is read. But I would like TableGateway to handle my entities as well. The hydrator could be picked from the result set prototype or maybe injected directly to TableGateway.
An alternative implementation could be to add a Feature Zend\Db\TableGateway\Feature for this and move to hydration part for the reading to this feature as well.
Any comments? Could it be implemented for 3.0.0 milestone?

@GeeH
I personally still think this issue might be worth to be kept and implemented but should be moved to the Zend\Db repository.

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


feat(Icons): Add new icons to Teams theme

volume-down
volume
volume-up

promote-presenter
demote-presenter

Microsoft Reviewers: Open in CodeFlow

Technically ok.
Visually the demote is ~1px up compared to promote which can cause an icon to jump in scenarios where you switch from one to the other on click (toolbar). Is that ok?

Good catch - will see about getting that fixed

Updated shapes and renamed presenter SVG-s.


Compatibility with AlamoFire Download and internet connections

Hi @mostafaberg
Are you aware of any incompatibility issues between the DFU library and using Alamofire.Download?
(https://github.com/Alamofire/Alamofire/blob/master/Documentation/Usage.md#downloading-data-to-a-file)
Once I connect to the Central Manager. For example within the ViewDidAppear and call the
[Asyncronous] Alamofire.Download task to download a file from the internet , it constantly does not return my download data.
I verified this I am able to download without problem using a sample app which does not perform BlueTooth connections and operations such as the DFU library does.

Did you try setting a different queue in DfuServiceInitiator's init?

We do not have any compatibility issues and I don't see how our library would affect your code.
can you show some code where he issues occur ?


LSv2 is missing

Please add information on the LSv2 series, which I believe is supposed to be GA now?

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

ID: b7323e68-7415-ba43-47de-8d1738f7c52b
Version Independent ID: 1bce47f4-d516-dd38-3e54-8da7b9b7b416
Content: Azure Windows VM sizes - Storage
Content Source: articles/virtual-machines/windows/sizes-storage.md
Service: virtual-machines-windows
GitHub Login: @jonbeck7
Microsoft Alias: jonbeck

Thanks for the feedback! We are currently investigating and will update you shortly.

Duplicate of #15622


[network] Refactor LibraNet for end-to-end QoS fairness and backpressure isolation

🚀 Feature Request
Motivation
Today, due to various queues in the LibraNet stack (e.g. PeerManager -> RPC, etc.) which are shared across peers/connections, it is difficult to guarantee fairness to peers and ensuring that back-pressure from one peer does not affect other peers. We need a way to solve both of these problems.
Pitch
Describe the solution you'd like
We can achieve full end-to-end fairness and QoS by spawning a new stack of network actors (RPC + DirectSend + SubstreamMuxer(new))per peer. In some sense, we already do this in DirectSend by spawning a separate tasks per peer. The proposed solution will just take this notion to the entire stack.
The existing PeerManager actor will be split into 2 new actors: A SubstreamMuxer for opening and listening for new substreams from a particular peer, and a new PeerManager which only dials and listens for new connections.
Describe alternatives you've considered
Another way of achieving this would have been to have all the queues in the stack be aware of all peers and provide fair queueing. Isolating back-pressure in this case would also have been challenging.
Are you willing to open a pull request? (See CONTRIBUTING)
Yes
cc: @phlip9

Does "peer" here mean a specific peerid / account, or a network connection? I could see really subtle bugs pop up if we try to coordinate backpressure across multiple network connections (realistic example: in Linux when you coredump you only reset the tcp connection once the core dump is complete. This frequently causes servers to live on for minutes after they are destined for death, not reading the socket. Imagine while this is happening a new prospective version of this peer comes to life)

@bmaurer : The way this is currently imagined, it would mean connection. If we ever go into a world where we can have multiple connections per peer, I don't think we'd try to coordinate backpressure across all the connections.


Preview window: cursor flickers in preview window, happens more often in tmux than outside of it

[X] I have read through the manual page (man fzf)
[X] I have the latest version of fzf
[X] I have searched through the existing issues

Info

OS

[ ] Linux
[X] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[X] zsh
[ ] fish

Problem / Steps to reproduce
At times while navigating the fzf items with preview window open makes cursor appear & disappear in different locations of the preview window.
Steps:

Open Alacritty window or iTerm2 window.
Launch tmux.
Go to some directory with a lot of files to run fzf from
Open fzf with command fzf --ansi --preview='echo {}', or fzf-tmux --ansi --preview='echo {}
Move up/down arrow key, it might require sprints of pressing and holding the arrow keys, abruptly releasing and repeating the procedure till you see cursor if flickering in the window.

This could have to do with the terminfo extension Sync mentioned in man tmux under section TERMINFO EXTENSIONS, though I don't know for sure (@nicm might be of help on that front). I know for sure its not my terminfo that  has any problem as other program's cursor doesn't flicker anymore...
Demo:
https://user-images.githubusercontent.com/71736629/129471010-3701c49f-f6e4-477c-ba7c-56d77c0f83e3.mp4

Can confirm it still happens with latest fzf on macOS/zsh.
This was really annoying me so I hacked a workaround by wrapping fzf and fully hiding the cursor while fzf is running.
Since fzf is so great at fuzzy matching, I never end up having to edit what I'm typing, so I really don't mind not seeing it while I type.
Keys could also be bound to show/hide the cursor. tput civis; will hide the cursor and tput cnorm; will restore it.
The following can be added to your .zshrc file to hide the cursor while fzf or fzf-tmux is running. A similar workaround can be done with other shells.
Add to ~./zshrc
# Hide cursor while fzf is running: https://github.com/junegunn/fzf/issues/2588
fzf()
{
    tput civis;

    command fzf "$@";
    local exit_code=$?

    tput cnorm;

    return $exit_code
}

fzf-tmux()
{
    tput civis;

    command fzf-tmux "$@";
    local exit_code=$?

    tput cnorm;

    return $exit_code
}

I hope up too soon. The workaround above messes up the output of fzf, rendering it useless if you actually select something.
Sorry if anyone loses time with this.