Dependency Conflict

Hi,
After running the following commands:
!git clone https://github.com/BSAkash/NLP4GOV
%cd NLP4GOV/src
!pip install -q -r ./constituent_coreferences/requirements.txt
from constituent_coreferences import corefs
%cd /content/

I encountered an issue at the line:
from constituent_coreferences import corefs

The error message is:
ModuleNotFoundError: No module named 'huggingface_hub.serialization'

I updated huggingface-hub, but it seems like requirements.txt installs an older version via one of the other dependencies. It might help to update the requirements.txt file in the repository to resolve this conflict. If there's another solution, any assistance would be much appreciated.
Thank you!

Hello @NirKosti !  First, is this a local install or via colab? Are you able to get the notebook working from within colab?

Hi @enfascination, I am using Colab, not my local installation.

Thanks for bringing this to attention. Yes, the dependency resolver does run into trouble because one of the libraries (AllenNLP) is now legacy and also somewhat ambiguous on version upper bounds. I will add a hard pip install --upgrade huggingface-hub to the notebook. Please keep us posted on other issues so we can fix them as you go along

Thank you, @Mahasweta-usc. I am still encountering an issue with huggingface_hub. I ran this again:
!git clone https://github.com/BSAkash/NLP4GOV #cloning the github repo, you can visit this url for the entire directory
%cd NLP4GOV/src
!pip install -q -r ./constituent_coreferences/requirements.txt # installs all required dependencies in one go!
from constituent_coreferences import corefs
%cd /content/

I got the error message:
ImportError: cannot import name 'split_torch_state_dict_into_shards' from 'huggingface_hub' (/usr/local/lib/python3.10/dist-packages/huggingface_hub/__init__.py)

This error occurs at the same point in the code:
from constituent_coreferences import corefs

Could you please assist in resolving this issue?

You will still need to pull the master for the new changes to reflect. You will know it has worked when the updated cell looks like:
https://github.com/Mahasweta-usc/NLP4GOV/blob/master/Policy Toolkits/ABDICO_coreferences.ipynb
https://docs.github.com/en/get-started/using-git/getting-changes-from-a-remote-repository


Rename calendar/datepicker mixin

F7 React was having trouble consuming v0.9.0 of F7 Vue because of the naming of the calendar/datepicker mixin. I renamed it to CalendarDatePickerMixin everywhere it appears, and I renamed the file to calendar-date-picker.vue (so the pascal-case conversion would result in CalendarDatePickerMixin instead of CalendarDatepickerMixin).
Fixes #140.

Great, thanks!


Supporting non-fpm packaging methods

Hi,
I've been looking around at using docker for packaging - its a great way to provide a clean, disposable build environment - in a few different use cases.  Berth is fantastic when you don't already have a spec or debian directory, but I was thinking that its current design might be tweaked to allow for alternate packaging methods beyond fpm.  A section under 'package' could potentially specify the type of package build you're doing (say spec:), specify some parameters such as spec file location, macros, pre-build steps and environment variables.
There's a few projects out there that sort of cover this use case but they seem a bit more complex (eg, they require a dockerfile that just builds a package).  Berth does a great job of wrapping the volume and container management already so I feel like this is a space it could excel at.  Thoughts?

Hello Travis, and thank you for the feedback.
As you can tell, Berth has been made with FPM in mind from the beginning, although we only have used it for DEB packages so far. The benefit of FPM is that there's only one tool to tailor Berth towards, and it basically behaves the same regardless of which type of package you are trying to build.
Adding support for another type of packaging tool would likely make the Berth code base quite bigger. As it is right now, you could potentially just use the build step, skip the packaging step, and do your own packaging inside of the build step with the shell script you can run. Alternately, you could split your package building into two different Berth jobs, again only making use of the build step, where the first Berth job would create the compiled/prepared files, and the second job then would handle whatever type of packaging you would like.
You do of course loose some convenience with having to maintain the build configuration in more of a shell script rather than a YAML file, but comparing it against the work it would need to add support for other packaging methods, it probably is the more reasonable choice (at least how I see it :)).
That being said, I'm not too familiar with the spec packaging format, it's requirements and how well it would fit into a YAML file. Could you perhaps provide some example of that? Then it would be easier to give some feedback on how much work it would be to add support for.

Ah I may not have made my intention clear.  FPM is for when you don't already have a spec file (what RPMs are built from) but some projects have one and may benefit from them when the build process is complex.  What I'm suggesting is rather than only knowing how to call fpm, call to the right build tool as specified by the user - so rpmbuild, for instance.  berth wouldn't need to understand the format of the build files as it wouldn't be generating the file.  It would just need to accept some common options similar to how it does so for fpm.
Just dropping the commands into the 'build' section and making the package section optional (it presently is required without at least a small change) is an option but as I was thinking about it I saw no reason one couldn't treat the various build tools as pluggable and fpm is just one implementation.  It would allow developers to use berth for any type of project and reap the workflow benefits even when fpm doesn't make sense.
A spitball version of what the yaml might look like:
environment:
  VERSION: 1.0.0
  SPECFILE: myproject.spec

package:
  image: myrepo/centos-rpmbuild:7
  volumes:
    .: /data

  rpmbuild:
    spec: $SPECFILE
    srpm: false
    buildprep: true
    prebuild:
        #!/bin/bash
        echo "Do some extra non standard stuff before running rpmbuild"
        touch /tmp/foo
    with:
        - libzot
        - libfoo
    without:
        - libtoast
        - libspam
    define:
        - buildver: $VERSION
        - buildrelease: 1

This would run something like this:
echo "Do some extra non standard stuff before running rpmbuild"
touch /tmp/foo
yum-buildprep myproject.spec
rpmbuild -bb  --with=libzot --with=libfoo --without=libtoast --without=libspam --define "bulidver $VERSION" --define "buildrelease: 1" myproject.spec

Unless you're completely opposed to adding other packaging commands beyond fpm I'm happy to take a stab at a PR or two.  I mostly work with rpm packaging so I'd tackle that first.

Thinking about it, having a plugin like structure for the packaging part would probably not be too much work. Berth could then come with two plugins by default, an FPM version which basically is just a copy of the current functionality, and probably the default, and a shell plugin which works more like the current build part.
I would have to think about how the integration between Berth and the packaging plugin would work, I am thinking something like:

Berth imports a Python module provided in the YAML file.
The Python module has a predefined variable which tells Berth the default Docker image to use for this plugin, but it can be overridden in the YAML file as it is right now.
The Python module also has a method with a set name, which is called by Berth with the package part of the YAML file provided as an argument.
The method return either a command to run inside the container or a script like the one provided in the build section to run in the container, and any environment variables that has to be provided besides those in the environment section.
Berth execute the container and the result should be a package file.

There could potentially also be a method in the plugin which can be used to verify the configuration has the necessary settings, which are needed in order for the packaging step to work. This part could however be optional.
Does that sound reasonable to you?

I think so.  Its a bit more abstracted than I was thinking of but it seems perfectly reasonable.  I was really just envisioning a set of modules that ship with berth and a generic call to whichever is specified by the yaml but more flexibility to call any arbitrary tool doesn't seem like a bad thing if the interface is light enough.
I'm already picturing using berth to build an rpm of the berth-rpmbuild plugin :-)

I'll try to see if I can find some time to split the current FPM functionality into a separate Python module within the next couple of weeks. I won't promise too much though, as I'll be relocating to another country and starting on a new job in the same period, but it could be a good task to work on when I have some down time :)


Translate concepts/storage/persistent-volumes in Korean

This is a Feature Request

What would you like to be added

Translate concepts/storage/persistent-volumes in Korean
Why is this needed
No translation with concepts/storage/persistent-volumes in Korean.
It is issued on May already, but it didn't merged.
So I try one more time :)
Comments

https://kubernetes.io/docs/concepts/storage/persistent-volumes/

/language ko
/assign

/priority backlog

/close


TypeError: this.idAwaiter[data.id] is not a function error when running AI GitHub developer

When running AI GitHub developer some users experience an error in the JS SDK.

I keep getting an error from the sandbox when trying to run this. I've tried it a couple ways, both manually putting in the gh token and putting it as a env var. It get's thru login and asks for the repo. Sometimes it get thru to ask for a task, sometimes not. In either case the error seems to be related to TypeError: this.idAwaiter[data.id] is not a function. Any ideas what's going on with this? I tried it in the js-sdk that it initially came with and also tried upgrading to 0.13.0, both fail around the same spot.

https://github.com/e2b-dev/e2b-cookbook/blob/main/examples/ai-github-developer-js/index.ts

The new beta SDK does not use websockets. We also removed this example from the cookbook.


Fix indents

@changeworld : Thanks for your contribution! The author, @navyasric, has been notified to review your proposed change.

Confirmed formatting changes in code syntax.


Move DEFAULT_DISPLAY_NAME from app to entity

Build failure looks like it's caused by infrastructure problems:
Modules changed, recalculating dependency graph
Build timed out (after 60 minutes). Marking the build as aborted.
Build was aborted
Putting comment on the pull request
Finished: ABORTED

I've done a git commit --amend && push -f to try to trigger another jenkins build.

Looks like jenkins gave up on this PR entirely. Merging.

Compilation of Brooklyn core fails after pulling this:
[INFO] -------------------------------------------------------------
[ERROR] COMPILATION ERROR :
[INFO] -------------------------------------------------------------
[ERROR] /Users/sam/code/brooklyn.io/brooklyn/brooklyn-server/core/src/test/java/org/apache/brooklyn/core/entity/EntityNameTest.java:[65,40] error: incompatible types: Entity cannot be converted to TestEntity
[INFO] 1 error
[INFO] -------------------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 28.458 s
[INFO] Finished at: 2016-06-15T16:21:26+01:00
[INFO] Final Memory: 26M/686M
[INFO] ------------------------------------------------------------------------

Line in question:
    public void testDefaultDisplayNameUsesDeferredSupplier() {
        TestEntity entity = app.addChild(EntitySpec.create(TestEntity.class)

Tried both Oracle JDK 1.7 and 1.8.

Thanks for pointing out @sjcorbett. Fixed in https://github.com/apache/brooklyn-server/commit/3bb69b6080fbde17279753ba562f4a5008bbd0f6.
The Eclipse compiler is too smart sometimes, especially around generics and losing type info, able to figure out things which leave the java compiler wondering.


@@unique block attribute fails for non-scalar types (regression)

Based on the following snippet from the PSL spec, @@unique should work fine for non-scalar types:
@@unique([recipe, ingredient]) would mean that the combination of fields must be unique in the table:
model RecipeIngredient {
  recipe                            Recipe
  ingredient                      Ingredient
  amount                          Float
  quantitativeDisclosures String
  comment                       String
  @@unique([recipe, ingredient])
}

The semantic meaning seems intuitive here and implies a comparison based on the @id field of the Recipe and Ingredient objects. As of 2.0.0-preview017.2, this was working but some time after that point started failing with a panic as described in #1352. #1352 was closed in favor of #1377, but @janpio mentioned #1377 covers improving the error message but not the regression.
I know nothing about Rust, but https://github.com/prisma/prisma-engine/pull/241 looks like a likely root cause to me. The timing is about right and it seems like all of the tests expect that fields passed to @@unique are scalar.

This is now cleaned up with additional issue https://github.com/prisma/prisma2/issues/1534 which is marked as a regression.


Ensure parser errors are properly returned

This fixes the issue of parser rejection errors not being correctly propagated back to Webpack.
In addition, the asset relocator has a filter regex it uses to determine if a full parse and analysis is needed or not. Due to the lack of the use of word boundaries, this is still overclassifying. So I've restricted the word boundaries a little that should aid in performance as well.
Fixes #7

Codecov Report

Merging #9 into master will decrease coverage by 0.15%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##           master      #9      +/-   ##
=========================================
- Coverage   70.46%   70.3%   -0.16%
=========================================
  Files           7       7
  Lines         711     714       +3
=========================================
+ Hits          501     502       +1
- Misses        210     212       +2

Impacted Files
Coverage Δ

src/asset-relocator.js
76.9% <66.66%> (-0.34%)
:arrow_down:

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 db7e1d3...188c67b. Read the comment docs.

@guybedford Can we add a test for this?

Sure, I've pushed up a simple test case.


[TRACKING] v4.1 - Magician CleanUp

[x] #119
[x] #120
[x] attributify
[x] global keyframes https://svelte.dev/docs#script
[ ] debug util [wait for upstream]
[ ] error handling, e.g. missing config
[ ] cleanup
[x] devtools
[ ] improve devtools
[x] safe list
[ ] compile
[ ] bundle
[ ] clean refactor of user comment removal
[x] WINDI EXPRESSION
[ ] support windi FullConfig
[x] update readme
[x] update docs
[x] update integration guide
[x] #141
[x] nested @apply
[ ] tests
[ ] evaluate https://github.com/sveltejs/svelte-preprocess/pull/104/files

7482f9a1f58dc003c1484eb80cc07e5d49d5b476

1bd87f26a18e75467b6b7690498ca92174c53171


Editor creates red NL characters on Windows when typing [ or {

Thanks. Merged as e5e2aefd. I took out the call to repeat since we're still targeting older JS runtimes that don't have this method.


[SUBMARINE-237] workbench-daemon.sh script return wrong status of workbench

What is this PR for?
We start workbench server successfully,  but when  run "workbench-daemon.sh status"  , it returns message "Submarine Workbench is not running"
What type of PR is it?
Bug Fix
What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-237
How should this be tested?
manual test

start workbench server
run “workbench-daemon.sh  status ” command and see if the return message is correct

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Can you shorten the title of the PR?

Can you shorten the title of the PR?

Can you shorten the title of the PR?

Done, thanks for review


Lazy load not rendering components when list is reversed

It seems that when you reverse the list that lazyload is rendering, it does not detect the change, so the items that were on the bottom of the list that are now at the top are not loading at all.

You can import forceCheck like import { forceCheck } from 'react-lazyload'; and invoke this method whenever you made some changes.

It's undocumented feature, maybe I should add some docs :p
https://github.com/jasonslyvia/react-lazyload/blob/7b282db4f811871c43d31d2e1e81c7e387210abd/src/index.js#L272

Thanks @jasonslyvia ! That works seemlessly, we are calling forceCheck in the componentWillUpdate() function. I can provide more details to our implementation if you want.


Scrape from soundcloud

Add an option whether to scrape from SoundCloud or YT. If the option is set to SoundCloud, and user tries to play YT links, the actual scraping will still happen from SoundCloud. This way, we don't need to bother about YT shutting our bots down.

You can prevent the GuildMusicManager#play recognize Youtube links by set the ClientMusicManagerOptions.enableService.youtube to false. Also there's actually an under development SoundCloudUtils class to handler search from SoundCloud, but since lack of experience of dealing with SoundCloud, I haven't successfully made it work.


Release 2.1.59

On the docket:

[ ] PEX_INHERIT_PATH+default shebangs don't interact well with virtualenvs #1344
[ ] Pex interpreter cache is not upgrade / downgrade safe. #1176
[ ] Decouple PEX runtime interpreter selection from buildtime interpreter selection. #1020
[ ] Provide a better resolve error message for a Platform DistributionTarget when --resolve-local-platforms. #1070
[ ] Remove PEX_PYTHON_PATH from build time interpreter selection. #1075

Split off from "Release 2.1.58 #1535" to make way for a fix for relative --cert paths for --isolated builds with setup_requires.


Update mip-dbcs-relnotes.md

@chrfox - This issue has been reported by our Japanese team and needs to be published as a known issue. Kindly review and acknowledge.


CI Pipeline unable to run for main

The Pipeline after last pull request was unable to run and show the following error:

Unable to create pipeline

deployment: one job can only need 50 others, but you have listed 54. See needs keyword documentation for more details

The interesting thing is that the CI run without any problems inside the PR and had only 40 jobs.

The deployment step failed as it is requiring all other steps to be finished first. And these are indeed 54 jobs. A solution could be to create a dummy job for this situation that awaits the parallel matrix and does nothing with the results. deployment is then only waiting on one job and no longer for all of them.

I've changed the limit in our gitlab instance to 500 according to https://docs.gitlab.com/ee/administration/cicd.html#set-the-needs-job-limit. Can you check that it works now?

The newest builds on main are working and the previous failed validation is working again. So it's fixed.


dockerize tests execution

Lets use docker to execute acceptance tests

No need, pyenv is pretty up to date with python releases and its easier to test multiple pythonv versions than with docker


[21.05] linux_5_13: drop

Motivation for this change
Backport for #139665.
5.13.19 was the last 5.13 release and the version is now EOL[1].
[1] https://lwn.net/Articles/869747/
(cherry picked from commit 01eb8ec98a52c1e7317fd62f5bc58db30951e716)
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

@ofborg eval


Start new installer project

Create a new project in Inno Setup for this application's installation

Done and released.


Remove requirement in GetSDFiles.py that script has to be executed on ArcGIS Server machine

To support ops server automated build process, which runs on a different server then the development server:

[x] Remove the requirement in the GetSDFiles.py script that it has executed on the ArcGIS Server machine.
[x] Add try/except/finally code block to catch unexpected errors and return appropriate exit code.
[x] Remove note when the parameters are printed to display that script needs to run on the ArcGIS Server machine.
[x] Add NOTE if not run on the server that the ArcGIS Server Directories have to configured with UNC paths and OS access granted appropriately.

Fixed and sync'd with elinz\ops-server-config v.next.

Verified.


Onboard Healthcare Integrations sender

Problem statement
Need to onboard Healthcare Integrations as an HL7 API sender

Sample HL7 messages from sender
Detected.txt
inconclusive.txt
NotDetected.txt


System logo boxes appears blank

Hi! This theme is absolutely awesome. Just have some issues with it... maybe you could help me:

NeoGeo Pocket/Color games appears under "DOS" system, and there is no boxed image of system (it appears blank)

Scummvm and Wonderswan systems has no logo box, it appears blank also.

Many thanks!

Hey, that's probably a metadata problem, check the Pegasus metadata file, I did a new metadata pack: https://github.com/dragoonDorise/pegasus-android-metadata and a new automatic installer: https://www.pegasus-installer.com


esp32 wifi crash (brownout issue)

Hello,
I've been struggling for the past few hours with this. Running a simple wifi snippet as shown below results in a crash with no backtrace when wifi.start() is called.
fn main() {
    esp_idf_svc::sys::link_patches();
    esp_idf_svc::log::EspLogger::initialize_default();

    let peripherals = Peripherals::take().expect("Failed to take peripherals!");
    let sysloop = EspSystemEventLoop::take().expect("Failed to take sysloop!");
    let nvs = EspDefaultNvsPartition::take().ok();

    let mut wifi = WifiDriver::new(peripherals.modem, sysloop.clone(), nvs)
    .expect("Failed to init WifiDriver!");

    let wifi_config = Configuration::Client(ClientConfiguration {
        ssid: "foo".try_into().unwrap(),
        auth_method: AuthMethod::WPA2Personal,
        password: "bar".try_into().unwrap(),
        ..Default::default()
    });

    wifi.set_configuration(&wifi_config).expect("failed to set config");
    wifi.start().expect("failed to start driver");
}

I (60) boot: ESP-IDF v5.2.3 2nd stage bootloader
I (60) boot: compile time Dec  1 2024 17:17:56
I (60) boot: Multicore bootloader
I (65) boot: chip revision: v1.1
I (69) qio_mode: Enabling default flash chip QIO
I (74) boot.esp32: SPI Speed      : 80MHz
I (79) boot.esp32: SPI Mode       : QIO
I (83) boot.esp32: SPI Flash Size : 4MB
I (88) boot: Enabling RNG early entropy source...
I (93) boot: Partition Table:
I (97) boot: ## Label            Usage          Type ST Offset   Length
I (104) boot:  0 nvs              WiFi data        01 02 00012000 00004000
I (112) boot:  1 nvs_keys         NVS keys         01 04 00017000 00001000
I (119) boot:  2 otadata          OTA data         01 00 00018000 00002000
I (127) boot:  3 factory          factory app      00 00 00020000 00200000
I (134) boot: End of partition table
I (139) boot: Defaulting to factory image
I (143) esp_image: segment 0: paddr=00020020 vaddr=3f400020 size=23534h (144692) map
I (189) esp_image: segment 1: paddr=0004355c vaddr=3ffb0000 size=03d48h ( 15688) load
I (194) esp_image: segment 2: paddr=000472ac vaddr=40080000 size=08d6ch ( 36204) load
I (206) esp_image: segment 3: paddr=00050020 vaddr=400d0020 size=9c8bch (641212) map
I (370) esp_image: segment 4: paddr=000ec8e4 vaddr=40088d6c size=0e644h ( 58948) load
I (400) boot: Loaded app from partition at offset 0x20000
I (400) boot: Disabling RNG early entropy source...
I (412) cpu_start: Multicore app
I (421) cpu_start: Pro cpu start user code
I (421) cpu_start: cpu freq: 160000000 Hz
I (421) cpu_start: Application information:
I (424) cpu_start: Project name:     libespidf
I (429) cpu_start: App version:      6c6ad98-dirty
I (434) cpu_start: Compile time:     Dec  1 2024 17:17:31
I (440) cpu_start: ELF file SHA256:  000000000...
I (446) cpu_start: ESP-IDF:          v5.2.3
I (451) cpu_start: Min chip rev:     v0.0
I (455) cpu_start: Max chip rev:     v3.99
I (460) cpu_start: Chip rev:         v1.1
I (465) heap_init: Initializing. RAM available for dynamic allocation:
I (472) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
I (478) heap_init: At 3FFB74C8 len 00028B38 (162 KiB): DRAM
I (484) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (491) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (497) heap_init: At 400973B0 len 00008C50 (35 KiB): IRAM
I (504) spi_flash: detected chip: generic
I (508) spi_flash: flash io: qio
W (512) pcnt(legacy): legacy driver is deprecated, please migrate to `driver/pulse_cnt.h`
W (521) timer_group: legacy driver is deprecated, please migrate to `driver/gptimer.h`
I (530) main_task: Started on CPU0
I (540) main_task: Calling app_main()
I (570) wifi:wifi driver task: 3ffbdf9c, prio:23, stack:6656, core=0
I (580) wifi:wifi firmware version: 82f54d2
I (580) wifi:wifi certification version: v7.0
I (580) wifi:config NVS flash: enabled
I (580) wifi:config nano formating: disabled
I (580) wifi:Init data frame dynamic rx buffer num: 32
I (590) wifi:Init static rx mgmt buffer num: 5
I (590) wifi:Init management short buffer num: 32
I (590) wifi:Init dynamic tx buffer num: 32
I (600) wifi:Init static rx buffer size: 1600
I (600) wifi:Init static rx buffer num: 10
I (610) wifi:Init dynamic rx buffer num: 32
I (610) wifi_init: rx ba win: 6
I (610) wifi_init: tcpip mbox: 32
I (620) wifi_init: udp mbox: 6
I (620) wifi_init: tcp mbox: 6
I (630) wifi_init: tcp tx win: 5760
I (630) wifi_init: tcp rx win: 5760
I (630) wifi_init: tcp mss: 1440
I (640) wifi_init: WiFi IRAM OP enabled
I (640) wifi_init: WiFi RX IRAM OP enabled
I (650) phy_init: phy_version 4840,a40f0ac,Aug 16 2024,16:48:06
W (650) phy_init: failed to load RF calibration data (0x1102), falling back to full calibration
�ets Jun  8 2016 00:22:57

rst:0x3 (SW_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)

Everything in sdkconfig is set to the defaults, so I struggle to see why this would crash... It's most likely due to phy_init but beyond that I have no clue.
Any ideas?

Brownout (not enough power) via USB?

 Many thanks. I was looking for a software issue, no wonder..
For anyone ever coming here, the issue was that starting wifi takes a lot of power and I was supplying the board with 3.3V. Switching to 5V fixed the issue.

 Many thanks. I was looking for a software issue, no wonder..
For anyone ever coming here, the issue was that starting wifi takes a lot of power and I was supplying the board with 3.3V. Switching to 5V fixed the issue.

Keep in mind that supplying the board with 5v should only be done on the 5v pin, or else you'll burn it!!
The chip itself works on 3.3v.
The 5v pin is connected to a step-down converter to 3.3v
Also you are missing amperes not voltage...


Terminal suggest: Several unknown completions for "cd ." in pwsh

only folders should be present in the list


fix: 修复导出 CSV 时分隔符错误导致的展示格式错误 close #2701

👀 PR includes

🐛 Bugfix

[x] Solve the issue and close #2701

📝 Description

BREAKING CHANGE: Export 组件 和 asyncGetAllPlainData, copyToClipboard 的是否异步导出参数统一为 async
修复普通表格导出 CSV 分隔符错误.
修复趋势分析表未接受自定义分隔符导致的导出错误.

🖼 Screenshot

Before
After

🔗 Related issue link

close #2701
🔍 Self-Check before the merge

[ ] Add or update relevant docs.
[ ] Add or update relevant demos.
[ ] Add or update test case.
[ ] Add or update relevant TypeScript definitions.

:tada: This PR is included in version @antv/s2-v2.0.0-next.20 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v2.0.0-next.19 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


open_mfdataset fails on variable attributes with 'list' type

Using open_mfdataset on a series of netcdf files having variable attributes with type list will fail with the following exception, when these attributes have different values from one file to another:
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

    ncf = xarray.open_mfdataset(files)
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/backends/api.py", line 658, in open_mfdataset
    ids=ids)
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/core/combine.py", line 553, in _auto_combine
    data_vars=data_vars, coords=coords)
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/core/combine.py", line 474, in _combine_nd
    compat=compat)
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/core/combine.py", line 492, in _auto_combine_all_along_first_dim
    data_vars, coords)
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/core/combine.py", line 510, in _auto_combine_1d
    for id, ds_group in grouped_by_vars]
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/core/combine.py", line 368, in _auto_concat
    return concat(datasets, dim=dim, data_vars=data_vars, coords=coords)
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/core/combine.py", line 122, in concat
    return f(objs, dim, data_vars, coords, compat, positions)
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/core/combine.py", line 307, in _dataset_concat
    combined = concat_vars(vars, dim, positions)
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/core/variable.py", line 1982, in concat
    return Variable.concat(variables, dim, positions, shortcut)
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/core/variable.py", line 1433, in concat
    utils.remove_incompatible_items(attrs, var.attrs)
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/core/utils.py", line 184, in remove_incompatible_items
    not compat(first_dict[k], second_dict[k]))):
  File "/home/ananda/jfpiolle/miniconda2/envs/cerbere/lib/python2.7/site-packages/xarray/core/utils.py", line 133, in equivalent
    (pd.isnull(first) and pd.isnull(second)))
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

An example of such variable is provided below:
	double sea_ice_fraction(time) ;
		sea_ice_fraction:least_significant_digit = 2LL ;
		sea_ice_fraction:_FillValue = 1.e+20 ;
		sea_ice_fraction:long_name = "sea ice fraction" ;
		sea_ice_fraction:standard_name = "sea_ice_fraction" ;
		sea_ice_fraction:authority = "CF 1.7" ;
		sea_ice_fraction:units = "1" ;
		sea_ice_fraction:coverage_content_type = "auxiliaryInformation" ;
		sea_ice_fraction:coordinates = "time lon lat" ;
		sea_ice_fraction:source = "CCI Sea Ice" ;
		sea_ice_fraction:institution = "ESA" ;
		string sea_ice_fraction:source_files = "ice_conc_nh_ease2-250_cdr-v2p0_199912011200.nc", "ice_conc_sh_ease2-250_cdr-v2p0_199912011200.nc" ;

The exception will occur when the source_files attribute have a different values in the file time series I am trying to concatenate. I had to use the preprocess argument to remove first this attribute to avoid this exception.
This is caused by the equivalent method in xarray/core/utils.py that does not account for this case:
def equivalent(first, second):
    """Compare two objects for equivalence (identity or equality), using
    array_equiv if either object is an ndarray
    """
    # TODO: refactor to avoid circular import
    from . import duck_array_ops
    if isinstance(first, np.ndarray) or isinstance(second, np.ndarray):
        return duck_array_ops.array_equiv(first, second)
    else:
        return ((first is second) or
                (first == second) or
                (pd.isnull(first) and pd.isnull(second)))

We could probably fix this by adding another case to equivalent for lists, e.g,.
elif isinstance(first, list) or isinstance(second, list):
    # verify both first and second are lists of the same length with the same elements
    # (by calling equivalent() on all elements)

Any interest in putting together a pull request for this? See http://xarray.pydata.org/en/stable/contributing.html

I will handle this issue this Saturday if no one else handle it by then.

Thanks @HasanAhmadQ7, that would be fantastic !

@jfpiolle Would you please share the files, or the code generating them?
As far as I understand we cannot have an attribute of type list. The closest thing I can think of is a variable length type as below
f = Dataset("example.nc","w")
    vlen_t = f.createVLType(numpy.int32, "phony_vlen")
    x = f.createDimension("x",3)
    y = f.createDimension("y",4)
    vlvar = f.createVariable("phony_vlen_var", vlen_t, ("y","x"))
    data =[]
    for n in range(0, len(x)*len(y)):
        data.append(numpy.arange(random.randint(1,10),dtype="int32"))
    data = numpy.reshape(data,(len(y),len(x)))
    vlvar[:] = data
    f.close()

Using files generated this way, I did not get an error using mf_dataset.

@HasanAhmadQ7 You can have attributes of type list. See for instance this code:
from netCDF4 import Dataset
import numpy

f = Dataset("example.nc","w")
x = f.createDimension("x",3)
vlvar = f.createVariable("test_var", numpy.int32, ("x"))

# here create an attribute as a list
vlvar.test_attr = ["string a", "string b"]

vlvar[:] = numpy.arange(3)
f.close()

Having files with different values for the test_attr attribute will cause open_mfdataset to fail.
thanks for addressing this!

@jfpiolle Thank you very much for the clarification


⚠ Bofrost.lu has degraded performance

In db28b08, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1388 ms

Resolved: Bofrost.lu performance has improved in 6a6fc82 after 14 minutes.


adds workflow for pull requests

Codecov Report

Merging #30 (8ec3b16) into main (3043513) will decrease coverage by 7.35%.
The diff coverage is 81.76%.

@@            Coverage Diff             @@
##             main      #30      +/-   ##
==========================================
- Coverage   92.84%   85.49%   -7.36%
==========================================
  Files          14       14
  Lines         615      717     +102
==========================================
+ Hits          571      613      +42
- Misses         36       94      +58
- Partials        8       10       +2

Impacted Files
Coverage Δ

client_node_map.go
100.00% <ø> (ø)

node.go
54.54% <ø> (-45.46%)
:arrow_down:

nodemap.go
68.00% <ø> (-32.00%)
:arrow_down:

register.go
49.27% <33.33%> (-50.73%)
:arrow_down:

testing.go
72.22% <40.90%> (-2.60%)
:arrow_down:

courier.go
95.34% <88.37%> (-2.59%)
:arrow_down:

client_node.go
93.50% <89.65%> (-1.14%)
:arrow_down:

client.go
97.33% <93.33%> (+2.50%)
:arrow_up:

channelmap.go
100.00% <100.00%> (ø)

responsemap.go
100.00% <100.00%> (ø)

... and 6 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 5ea08f6...8ec3b16. Read the comment docs.


menu issues

Two issues observed from the following mwe:
using GLMakie

main() = begin
  fig = Figure()
  menu = Menu(fig, options=["choice $i" for i ∈ 1:30])
  on(menu.selection) do s
    @show s
  end
  fig
end

main() |> display

I cannot scroll and select some entries with a high number of choices (unless resizing the window, which is viable only up to a certain point);
click and select feels slower with increasing number of choices (minor issue).

It'd be nice if the menu could be scrollable. Would this be easy to implement with scroll events ?
My application is: selecting fields from physical simulation output (which can be numerous).

Duplicate? #2074


Wrong number of items

After reading all the items (from the 34 categories), I just found 35.39M instead of the reported 48.19M.
items = {}
for category in tqdm(categories, desc="Reading metadata"):
    with open(f"resource/dataset/Amazon-Reviews-2023/raw/meta_{category}.jsonl", 'r') as category_file:
        for line in category_file:
            item = json.loads(line.strip())
            items[item["parent_asin"]] = item["title"]
len(items)
# 35393189

Did I miss something?

Not all items in the dataset have associated metadata. The total item count is based on the distinct number of parent_asin in the user-item reviews. As a result, the dataset includes 48.19M items in total, of which 35.39M have metadata.


Assign Diagnostics with Identifiers

Description
The diagnostics from Typst should be able to be identified or referred via a precise way, e.g. by assigning identifiers, MSVC Error LNK2019.
Use Case
Here are four possible use cases:

Making it possible to attach "Fix Me" code actions to the diagnostics.
Operating the diagnostics programmatically, for example, changing the diagnostics about out of root errors, whose hint is only useful for typst-cli.
Further detailed documentation on specific diagnostics, like MSVC explaining the solutions to LNK2019 errors.
Filtering the unwanted errors by ID.

Similar thing in rust: https://doc.rust-lang.org/error_codes/error-index.html

Hi, this is something we intend to work on, in particular to make features such as warning suppression easier to implement. I have been considering something like Rust's slugs (see https://doc.rust-lang.org/rustc/lints/listing/allowed-by-default.html), which would make a reference to a warning immediately readable (for example, unnecessary_import_rename), but we could also use numeric codes for compiler warnings (e.g. W0123). This can also be expanded to errors. What do you think?

IMHO, it is not necessary to enforce diagnostics to be immediately readable, as the IDs usually come with the diagnostics messages, which explain the meaning of IDs. This is also not quite useful as the English string is not quite readable for non-English users, which means we will have to show them by localization in the end.
But string IDs do have an advantage that I like, that we can have a namespace on diagnostics, e.g. @preview/some_package:some_warning_id, so we can maintain and operate warnings emitted by packages easily.

The idea of using names/slugs is mostly for warnings so you can suppress them, but can be optionally extended to errors as well. While we can use numeric IDs, they would only make sense for the compiler's internal warnings and errors imo - if/when we get package warnings, we don't want to force package authors to keep a consistent list of numeric IDs for their diagnostics.
For instance, I think this (using hypothetical warning suppression syntax)
#let f = {
    @bikeshed::allow(unused_param)
    (a, b, c) => a + b
}

is better / more syntactically coherent and readable than
#let f = {
    @bikeshed::allow(W0932)
    (a, b, c) => a + b
}

Of course, it is fair to think of non-English speakers, but this would generally only be used within code (where all syntax is also in English); the warning messages themselves wouldn't use those slugs, and can be localized in the future.

you can parse the syntax names into numeric codes, which is not a problem. These numeric codes are exactly used by compiler internals and packages may not be able to allocate a code at script level. For example:
#let f = {
    @bikeshed::allow(@preview/modern-thesis, overflow-box)
    (a, b, c) => a + b
}

Then it is mapped to 10086 meaning a general overflow errno for packages.
I don't mean it necessary to use numeric error, but the diagnostics IDs are frequently read by machines but not humans so it is important to keep the efficient memory usage and cost to compare imo.
If your're really worry about readability, you can use string id, but please still limit the length of these ids so that machines can compare them quickly.

The performance/comparison problem can be solved with interning (i.e. PicoStr).

The performance/comparison problem can be solved with interning (i.e. PicoStr).

The interning technique is only useful for rust.

While there isn't a problem in translating the slugs to IDs internally (and thus for World consumers to use as well), that probably can't be done for packages unfortunately (would be hard to enforce consistent IDs in principle).
To better gauge how we can tackle this, could you share the use case you have in mind in which the string identifiers could be a problem (and/or how you plan to use the identifiers)? Worth noting that warn suppression, at least, would be handled by the compiler itself, so I assume there is some other requirement in play.

To better gauge how we can tackle this, could you share the use case you have in mind in which the string identifiers could be a problem (and/or how you plan to use the identifiers)?

I'm not saying identifiers must be strings, but I was required to compare string ids with numeric ids, and my.

To better gauge how we can tackle this, could you share the use case you have in mind in which the string identifiers could be a problem (and/or how you plan to use the identifiers)?

I'm not saying identifiers are necessary to be numeric, but I was required to compare numeric identifiers and string identifiers. My opinions are summarized as follows:

My case is already in listed in the first message. In the use cases, the diagnostic IDs are read by language service servers and clients for identification and filtering, but not directly for users, so it is worth keeping efficiency than readability.
for warning suppression or other scenarios that require readability, we can parse the source code and map readable slugs into diagnostic IDs or map IDs back.
even if we uses string IDs, package writers may not be allowed to allocate diagnostic IDs by themselves, and we may have some government on it.

Is the performance with strings really such a big concern?

For the performance issue on compiler's side, I did not quite think of it in practice, but I can construct some cases that we need to intern and compare frequently, that is raise warnings on arithmetics.
@bikeshed::warn(bad-arithmetics)
#let mul(a, b) => a * b
#for I in range (10000) { mul(int(1e10), int(1e10)) }

Then the diagnostics will be raised by thousands of times. This should be optimized carefully if we use string ids but is probably not a thing if we use numeric ids.

Well, we gotta be pragmatic here. Realistically, your document shouldn't be emitting thousands of warnings. In your particular case, you'd only have a single warning as well in the end, since it'd have the same message and span (diagnostics are deduplicated). Overall, I don't see how this can be that large of a problem; the typical document will have 0-5 warnings, maybe 10-20 suppressed.
With that said, I'm in favor of limiting the length of the slugs. Maybe we can limit it to 32 or 64 bytes just to be sure.

diagnostics are deduplicated
That's happening on compile's side, so there are thousands of diagnostics along with their ids to be duplicated by compiler. That means, after adding diag ids, you'll probably deduplicate them by span, id, and message. I don't see a problem of my description.
With that said, I'm in favor of limiting the length of the slugs. Maybe we can limit it to 32 or 64 bytes just to be sure.
That's what I'm happy to see, that restrict the length of strings. 32 bytes of limit is still a bit excessive for id-like data. If you make use of EcoString's feature, 13~14 bytes is a good limit so that no data is allocated on heap.

Is the performance with strings really such a big concern? I am not really convinced that this is a bottleneck.

+1. iirc clangd, clippy use string id and there doesnt seems to be such a limit. they have very long error strings.

Is the performance with strings really such a big concern? I am not really convinced that this is a bottleneck.

+1. iirc clangd, clippy use string id and there doesnt seems to be such a limit. they have very long error id strings.

I don't know whether you've read these repositories, but IIRC both rustc and clang compiler use numeric ids. clang strictly assigns all linting with unique numeric ids. rustc assigns all linting with a same diagnostics id.
The string ids you see may be the visual name of these diagnostics.

Seems that i've misread. I agree with

These numeric codes are exactly used by compiler internals

I'm not sure with this

In the use cases, the diagnostic IDs are read by language service servers and clients for identification and filtering

IIRC to make a ls, you need embed typst-* crate. i think if numeric codes are exactly used by compiler internals holds, this will hold, too. Am i right?

IIRC to make a ls, you need embed typst-* crate. i think if numeric codes are exactly used by compiler internals holds, this will hold, too. Am i right?

For a server (rust) it is, for a client (lua, javascript, etc.) it isn't.

FWIW, r-a always pass a string id to ls client.

However, rustc assigns all linting with a same numeric id, so r-a will probably only passes a same string id here.

I think I've been a bit tired on discussing whether to use numeric codes or string codes. I think it is not a big problem to always uses string codes for identifying. Currently, it is not a problem to have a slightly slower implementation but a problem to have no idea on comparing them precisely. I'm just usually concerned about how to implement a fast compiler, and throw some points about what I have read and learned from projects. From perspective of performance, I may follow what rustc does, which

uses numeric codes solution.
provides a way to filter linting level diagnostics by configuration or macros, and doesn't provide a way to filter them at client side.

rustc assigns all linting with a same numeric id, so r-a will probably only passes a same string id here.

I did some test and the result is interesting. Seems that for compiler errors, the error code looks like E\d+, for warnings, it is a descriptive string
            "code": "unused_variables",
            "source": "rustc",
            "message": "unused variable: `test`\n`#[warn(unused_variables)]` on by default",

For compiler design, i think using a numeric id(maybe using a enum) is beneficial. I totally agree with Currently, it is not a problem to have a slightly slower implementation but a problem to have no idea on comparing them precisely .
For language server design, in the case of "too many diagnostics", i think the format/length of the code is not the bottleneck. Because when there is a lot of errors, like 10000 of them, the language client/server need to ser/de a big json array and send it to another process using stdio. This can easily takes more time comparing to filtering string ids. I remember I've seen "too many errors, stopping" or similar thing in clangd/r-a. Maybe it is their way to prevent "too many errors" problem.

provides a way to filter linting level diagnostics by configuration or macros, and doesn't provide a way to filter them at client side.

I don't know if i get it clearly. When using vscode, i can always filter diagnostics using the panel. (Actually, i never use this functionality). I don't think you can prevent users do this.

After reading the thread again, I think there was a mystery divergence, which would lead the discussion to become ineffective. First, a diagnostic message may be assigned with different forms of IDs at different stages. We seem to be on different channels when discussing :). Specifically, it might have three levels:

compiler internals
rust tools and language servers
languages clients

Then, it seems like a third approach other than numeric or string ids, having a pointer ID (type id) for comparison during compilation, is even better than the string IDs with interning. This is because you don't even have any cost on interning the IDs when creating diagnostics. It has the same flexibility as assigning string IDs, and it keeps the same performance as clangd which strictly assigns numeric codes to all specific diagnostics.
Lastly, besides efficiency, at language client level, rustc uses short and limited amount of string codes to ensure steady user inference to language clients as well as editor users.

Pointer ID/Type ID sounds cool. What does it looks like?

I deleted the type id to avoid confusing. The clippy has a macro that creates a global lint object at runtime:
declare_clippy_lint! {
    /// ### What it does
    /// Checks for type parameters in generics that are never used anywhere else.
    ///
    /// ### Why is this bad?
    /// Functions cannot infer the value of unused type parameters; therefore, calling them
    /// requires using a turbofish, which serves no purpose but to satisfy the compiler.
    ///
    /// ### Example
    /// ```no_run
    /// fn unused_ty<T>(x: u8) {
    ///     // ..
    /// }
    /// ```
    /// Use instead:
    /// ```no_run
    /// fn no_unused_ty(x: u8) {
    ///     // ..
    /// }
    /// ```
    #[clippy::version = "1.69.0"]
    pub EXTRA_UNUSED_TYPE_PARAMETERS,
    complexity,
    "unused type parameters in function definitions"
}

https://github.com/rust-lang/rust-clippy/blob/76856ffb576fb2dff472481a6389d42ca836db6f/clippy_lints/src/extra_unused_type_parameters.rs#L17-L41


"Module 'GoogleMapsBase' not found" Error with Google Maps SDK for iOS V2.0.0

I just downloaded Google-Maps-iOS-Utils V1.0.0. and GoogleMaps iOS V2.0.0 with CocoaPods:

Two compiler errors "Module 'GoogleMapsBase' not found" then occur:

My code was working with Google-Maps-iOS-Utils V1.0.0 and GoogleMaps V1.13.2.
Please help resolve this issue. Thank you!

Cross reference: https://code.google.com/p/gmaps-api-issues/issues/detail?id=10190

Thank you!!

When I compile my static framework with the Google Maps iOS SDK integrated. It compiles fine, however when I add that framework in my project, it states the error "framework GoogleMapsBase SDK not found for arm64". Same error occur for armv7 architecture also. My custom framework is woking good till I am compiling for dynamic framework but as I change the "Mach-O Type" key in build setting in my framework to "Static". It is generating this error when I add my static framework to my project.

Maybe you want to include GoogleMaps into your project too?

@mountainvat then purpose of my SDK is not fulfilled. My concern is why it is working for dynamic type but not for static. Is there anything needs to be done extra for static framework?

Can you please post some more detailed error messages?

By selecting my target, I got the following errors(image atta

ched)

Just curious are you also linking in the GoogleMaps Utils library or just the GoogleMaps SDK?

Just SDK only.

I can see that another issue has been raised on the SDK issue tracker (https://issuetracker.google.com/issues/37727522). Would you mind tracking that bug instead because it's directly related to the SDK. Also it may help if you could provide a minimal reproduction case.
Thanks.


MapGraphQL with dynamic schema name.

Is your feature request related to a problem?
I have a multi-tenant application, where each tenant is able to define attributes. I'd like to be able to define a schema per tenant. However I don't know the tenants up front and the Middleware caches output and reuses Schema's based on the SchemaName.
app.MapGraphQL() currently only accepts a string, sets it in the httpContext.Items but all middleware seems to use the constant provided to the middleware.
The solution you'd like
I'd like to be able to dynamically provide a schema name to the middleware.
If we would be able to pass a labmda to determine the schema name based on the HttpContext all schema caching and optimizations would work. eg:
app.MapGraphQL(httpContext => GetSchemaName(httpContext))
When the model of a tenant is changed it is easy to regenerate the schema using IRequestExecutorResolver.
Product
Hot Chocolate

While it sounds easy this is an epic with many things to consider to get it implemented that it performs well. We will consider something in this area for the version 14 cycle.

We have no plans to implement this for now.


[nodejs][10.prompt validations] Fix typos and improve formatting

Proposed Changes

Minor changes to typos and minimal structural changes.
Change the section Deploy this bot to Azure linking existing documentation

Closed due to Microsoft#741


"BufList" dependency choice

Most of the code I've laid out in quinn-h3 is based on tokio_util::codec and its AsyncRead friend. The problem with them is that it introduces a mandatory copy in it's user buffer for all data (through the &[u8] param).
@seanmonstar's proposal introduces a Bytes-like yielding stream read API:
// src/quic.rs
pub trait RecvStream {
    /// The type of `Buf` for data received on this stream.
    type Buf: Buf;

    fn poll_data(
        &mut self,
        cx: &mut task::Context<'_>,
    ) -> Poll<Result<Option<Self::Buf>, Self::Error>>;
}

Cool, now we can be zero-copy from transport to user! But now, we'd like to decode frames on something like impl Buf for &[Bytes].
Sean pointed out that he already implemented it in hyper. So we could:

Copy the implementation in h3
Depend on hyper
Move that to a crate.

My opinion is that it has its place in tokio_util, and that a FrameRead implementation based on it would be very useful for the whole ecosystem, aside the fact that we wouldn't have to run our own thing in h3.

The implementation in hyper is tiny, I'd just copy it directly to h3 to start with. It would be nice to update tokio_util::codec, but I think a deeper redesign of that module is required, and don't want to block us here.

Closing via #14


Kani regression test expected/dynamic-error-trait/main.rs fails with latest CBMC

Proposed change:
Using diffblue/cbmc@6a7b97cc89 (which is current develop, and between the already released 5.74.0 and the upcoming 5.74.0) results in a regression test failure that I believe is due to a limitation in Kani. Running the regression tests I get:
[...]

------------------------------------------
stderr:
------------------------------------------
warning: unused imports: `Read`, `Write`
 --> /home/ub-backup/tautschn/kani.git/tests/expected/dynamic-error-trait/main.rs:6:21
  |
6 | use std::io::{self, Read, Write};
  |                     ^^^^  ^^^^^
  |
  = note: `#[warn(unused_imports)]` on by default

warning: field `addr` is never read
  --> /home/ub-backup/tautschn/kani.git/tests/expected/dynamic-error-trait/main.rs:11:5
   |
10 | pub struct MemoryMapping {
   |            ------------- field in this struct
11 |     addr: *mut u8,
   |     ^^^^
   |
   = note: `#[warn(dead_code)]` on by default

warning: Found the following unsupported constructs:
             - caller_location (1)

         Verification will fail if one or more of these constructs is reachable.
         See https://model-checking.github.io/kani/rust-feature-support.html for more details.

warning: 3 warnings emitted

thread '<unnamed>' panicked at 'Found property counter that doesn't match number format', kani-driver/src/cbmc_output_parser.rs:172:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

------------------------------------------

failures:
    [expected] expected/dynamic-error-trait/main.rs

test result: FAILED. 159 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 15.16s

Some tests failed in compiletest suite=expected mode=expected host=(none) target=(none)

Looking at the CBMC output, and comparing it to an earlier version, the only difference is:

New: [_ZN4core3ptr42drop_in_place$LT$std..io..error..Error$GT$17hd75fb0a3517a80c2E.recursion] recursion unwinding assertion: SUCCESS
Before: [.recursion] recursion unwinding assertion: SUCCESS

The latter seems bogus, and the new output produced by CBMC strictly seems to be an improvement. Either the output parser needs fixing, or we need to check why this mangled function name is seen in the output in the first place, which then perhaps returns the output to something that is compatible with Kani's expected property format.
Motivation:
diffblue/cbmc@5c4503f2ef2fd altered the way we compute some property names. CBMC reserves the discretion to change property names, as long as they remain unique. On this occasion, however, it seems the previous property name was bogus, and Kani just seemed to apply a workaround.

The "before" case was handled by a special case in the output post-processor (see code below). It's nice that the change didn't go unnoticed. Any reason for "recursion" properties to not have a counter?
https://github.com/model-checking/kani/blob/81a8f8f6a4b22592a884b1d8fc78ac13d968a156/kani-driver/src/cbmc_output_parser.rs#L133-L135

Any reason for "recursion" properties to not have a counter?

We do not currently distinguish the call stacks of recursive calls, so there cannot be multiple instances, just one per function name.


vscode.tasks.executeTask error Unexpected: Task does not exist after task.terminate()

Type: Bug

clone repro from https://github.com/vaclavHala/vscode-workspace-race and open project folder corrupted-tasks
run the repro extension from vscode
open two folders in the workspace (these will not be touched by the repro code, the problem was encounterd in multi-folder workspace and this just reproduces that environment)
run command Run tasks.json race
after some time popup with error Unexpected: Task does not exist. will appear

Expected beahvior:
the loop running the dummy task will keep running indefinitely
This seems to be caused by using TaskExecution.terminate() in combination with changes of the workspace folders. When the while(true) loop in the repro is changed to only keep writing the tasks.json file and running the tasks the problem never appears
VS Code version: Code - Insiders 1.82.0-insider (f1302be1e67e3af5fbeb8bbb2ea784de7bc96150, 2023-09-01T11:05:38.381Z)
OS version: Linux x64 4.19.0-18-amd64
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 7 2700 Eight-Core Processor (16 x 2484)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 1, 1

Memory (System)
31.43GB (12.42GB free)

Process Argv
--crash-reporter-id f8df83a3-a85f-44ab-903a-d2a2cb362dd0

Screen Reader
no

VM
0%

DESKTOP_SESSION
xfce

XDG_CURRENT_DESKTOP
XFCE

XDG_SESSION_DESKTOP
xfce

XDG_SESSION_TYPE
x11

Extensions (0)

Extension
Author (truncated)
Version

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythontb:30258533
pythonptprofiler:30281269
vsdfh931:30280409
vshan820:30294714
vscod805:30301674
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30404738
py29gd2263:30784851
vsclangdf:30492506
c4g48928:30535728
dsvsc012:30540252
pynewext54:30618038
showlangstatbar:30737417
ecj1e332:30687743
pythonfmttext:30716741
fixshowwlkth:30771523
showindicator:30805243
pythongtdpath:30726887
i26e3531:30792625
welcomedialog:30812478
pythonnosmt12:30779711
pythonidxpt:30768918
pythonnoceb:30776497
copilotsettingt:30808721
dsvsc013:30777762
dsvsc014:30777825
diffeditorv2:30786206
pythonmpsinfo:30842935
dsvsc015:30821418
pythontestfixt:30826906
pythonfb280951:30830809
pythonregdiag:30842812
pythonmypyd1cf:30850306
pythoncet0:30848247

I'm not seeing the task you said to run in that folder

The repro steps say to run command Run tasks.json race, not a task. The command runs the task itself via the vscode.tasks API.
Here is the command in the command pallette

And here is the error it produces

Please try the repro steps again and let me know if you still can't see what I show in the screenshots @meganrogge

I get this error, which is interesting because the workspace contains two folders.

From the screenshot it seems like you have one workspace folder containing two file system folders - that still means the workspace is a single root workspace
You need to have a multi-root workspace by calling File > Add Folder to Workspace...
I hope the terminology I use is correct, it is taken from this doc https://code.visualstudio.com/docs/editor/multi-root-workspaces

PR fixing it https://github.com/microsoft/vscode/pull/200872


Disabled on select in repeater not reactive

Package
filament/forms
Package Version
v2.13.24
Laravel Version
9.11
Livewire Version
No response
PHP Version
8.0.2
Bug description
With the following form schema when i try to change the disabled state of the itemable_id with the reactive itemable_type field it stays disabled.
Also the repater is empty even with defaultItems(1).
        return [
            Forms\Components\Repeater::make('items')
                ->schema([
                    Forms\Components\Select::make('itemable_type')
                        ->label('Type')
                        ->options([
                            DockerContainerPort::class => 'Container Port'
                        ])
                        ->reactive()
                        ->required()
                        ->afterStateUpdated(fn (callable $set) => $set('itemable_id', null)),
                    Forms\Components\Select::make('itemable_id')
                        ->label('Item')
                        ->required()
                        ->unique(table: Item::class)
                        ->disabled(fn (\Closure $get) => empty($get('itemable_type')))
                        ->searchable()
                        ->getSearchResultsUsing(fn (\Closure $get, string $search) => $this->searchContainerPorts($get('itemable_type'), $search)),
                    Forms\Components\Toggle::make('active')->default(true),
                    Forms\Components\Toggle::make('new_tab'),
                    Forms\Components\Toggle::make('only_admin'),
                ])
                ->defaultItems(1)
                ->disableItemMovement(),
        ];

Reproduction repository
https://github.com/Walter-Melon/filament-select-example
Steps to reproduce
It appears the disabled closure does not work with searchable select fields.
Relevant log output
No response

Can you provide us a reproduction repository URL and then we'll look at your issue?
That would allow us to download it and review your bug much easier, so it can be fixed quicker. Please make sure to give us some instructions on where to find the bug, and include a database seeder with everything we need to set it up quickly.

I did a bid more testing and it appears the disabled closure does not work with searchable select fields.
Here is my testing repository.
https://github.com/Walter-Melon/filament-select-example

This is caused by a bug in Choices.js. I'm working on implementing a workaround.

I'm not sure what's the best way to "watch" the disabled state of the element, since Choices.js takes over control after initialization. Any ideas, @danharrin?

Here's how we implemented a dynamic min/max for the date picker, using hidden fields. Ugly but works. https://github.com/filamentphp/filament/pull/2314/files#diff-ea8114ac781958e1aea949dc578736fdc36f72f813c7f7395a5fff5a69015f36R36-R37

But how do you watch changes, @danharrin?

Maybe x-model and $watch? Not sure if that would work, we might have to close this as Choices doesnt allow it

Hmm, x-model probably won't update if Livewire just replaces the value.
We could add a dummy input which has the disabled attribute and get that value in a Livewire.hook('element.updating', () => {}) to call enable() or disable() on the Choices.js component.

but then it would call it every time, that sounds inefficient?

We can check the el being updated, and only call disable() when it equals the input element. Or do you have general concerns that the hook gets called on every Livewire response that updates an element?

the latter, general concerns

Yeah I agree, its just a limitation with Choices

I used the next hack in my code:
TextInput::make('stand_id_disabled')
    ->readOnly(fn(Get $get) => $get('project_id') === null)
    ->visible(fn(Get $get) => $get('project_id') === null)
    ->label('Stand')
    ->required()
    ->placeholder('Select an option'),

Select::make('stand_id')
    ->relationship('stand', 'name')
    ->required()
    ->searchable()
    ->visible(fn(Get $get) => $get('project_id') !== null)
    ->options(fn(Get $get) => Project::find($get('project_id'))?->stands->pluck('name', 'id'))
    ->preload(),

It's not the accurate result but it's possible to create something on this base. Maybe later I'll create a plugin

This was my solution to this issue, thanks @njxqlus for the base code.
Forms\Components\Select::make('role')
    ->live()
    ->enum(Role::class)
    ->options(Role::class)
    ->afterStateUpdated(fn(Forms\Set $set) => $set('countries', null))
    ->required(),
Forms\Components\Select::make('countries_disabled')
    ->label(__('Managed countries'))
    ->visible(fn(Forms\Get $get) => $get('role') !== Role::CountryManager->value)
    ->disabled()
    ->multiple()
    ->hint(__('This field is only for country managers')),
Forms\Components\Select::make('countries')
    ->label(__('Managed countries'))
    ->visible(fn(Forms\Get $get) => $get('role') === Role::CountryManager->value)
    ->required(fn(Forms\Get $get) => $get('role') === Role::CountryManager->value)
    ->preload()
    ->multiple()
    ->relationship('countries', 'name')
    ->hint(__('This field is only for country managers')),


Fix GitHub ref

Fixes github ref parsing to set environment variable

The failing tests seem to be unrelated to this PR


[SPARK-21512][SQL][TEST] DatasetCacheSuite needs to execute unpersistent after executing peristent

What changes were proposed in this pull request?
This PR avoids to reuse unpersistent dataset among test cases by making dataset unpersistent at the end of each test case.
In DatasetCacheSuite, the test case "get storage level" does not make dataset unpersisit after make the dataset persisitent. The same dataset will be made persistent by the test case "persist and then rebind right encoder when join 2 datasets" Thus, we run these test cases, the second case does not perform to make dataset persistent. This is because in
When we run only the second case, it performs to make dataset persistent. It is not good to change behavior of the second test suite. The first test case should correctly make dataset unpersistent.
Testing started at 17:52 ...
01:52:15.053 WARN org.apache.hadoop.util.NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
01:52:48.595 WARN org.apache.spark.sql.execution.CacheManager: Asked to cache already cached data.
01:52:48.692 WARN org.apache.spark.sql.execution.CacheManager: Asked to cache already cached data.
01:52:50.864 WARN org.apache.spark.storage.RandomBlockReplicationPolicy: Expecting 1 replicas with only 0 peer/s.
01:52:50.864 WARN org.apache.spark.storage.RandomBlockReplicationPolicy: Expecting 1 replicas with only 0 peer/s.
01:52:50.868 WARN org.apache.spark.storage.BlockManager: Block rdd_8_1 replicated to only 0 peer(s) instead of 1 peers
01:52:50.868 WARN org.apache.spark.storage.BlockManager: Block rdd_8_0 replicated to only 0 peer(s) instead of 1 peers

After this PR, these messages do not appear
Testing started at 18:14 ...
02:15:05.329 WARN org.apache.hadoop.util.NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable

Process finished with exit code 0

How was this patch tested?
Used the existing test

Test build #79883 has started for PR 18719 at commit 3e4438d.

@gatorsmile could you please review this?

Test build #79883 has finished for PR 18719 at commit 3e4438d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79883/
Test PASSed.

cc @cloud-fan

shall we add a afterEach to clean up the CacheManager?

Good suggestion, make sense. done.

Test build #79889 has started for PR 18719 at commit e9d9a33.

Test build #79889 has finished for PR 18719 at commit e9d9a33.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79889/
Test PASSed.

Merged build finished. Test PASSed.

Thanks! Merging to master.

good catch, i will create a follow-up PR today.


Patient list in Figure 3A

Dear Floris,
Thank you very much for making the code and data of the GLASS project open! It is a really exciting and important endeavor.
I am trying to reproduce Figure 3A from the open data, and I encountered some problems when selecting the patients. I wonder if it is possible for you to provide the ordered list of patients in Figure 3A. Thanks!
Best regards,
Quanhua Mu
PhD student
Hong Kong University of Science and Technology

Dear Kevin,
Thank you for your interest and my apologies for the late reply. The figure is roughly sorted by the number of drivers per patient. I cannot provide you with an exact list of patients, but I can refer you to the R code that was used to generate the figures here https://github.com/TheJacksonLaboratory/GLASS/blob/master/R/figures/F1-F3-GLASS-heatmap.R. This R code references SQL queries that were used to build the input data tables and these were subsequently sorted in R. I hope that this helps.
Best regards,
Floris


HDDS-9646. Unknown container from datanode in Recon.

What changes were proposed in this pull request?
This issue  can happen whenever there is a delay in container report of a container that belongs to a closed pipeline. Recon knows about pipeline state through the pipelineSync task that runs every 5 min. If a pipeline gets created and closed within 5 min, pipelineSync task in Recon gets an already CLOSED pipeline.
Whenever a container pertaining to this pipeline is reported to Recon by the DN, it fails to add it in it's memory as there is a check present that prevents adding containers belonging to CLOSED pipelines.
The check was added as part of HDDS-7829 for SCM.
The change here is to ignore the pipeline close check for Recon as recon doesn't create container first but gets to know it from DN and SCM. In case of SCM, it first creates the container and gives blocks to client to write on an allocated pipeline so the check makes sense however for Recon we should be good to ignore it.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9646
How was this patch tested?
Manually.

@devmadhuu could you please take a look?

@dombizita  Pls have a look

Thanks @sadanand48 for the patch, @devmadhuu, @dombizita, @sumitagrawl for the review.


No links in main menu causes an exception

When there are no links in the main menu, except home, there's an NoneType exception generated.
Solution:

Check code

The same check must be copied in the footer menu

Fixed.  Reading empty yaml files now return an empty structure instead of None


Updating conda and singularity with the latest version of leviosam2

Hi @milkschen,
If its okay, could you update Conda or Singularity with the latest version of leviosam2? I'm eager to try out some of the new ONT flag. I've been running into some issues with cmake

Hi @zhemingfan, the ONT mode has been added in the latest version (v0.2.2), which is now available on conda/singularity/docker.


bootstrap-vcpkg.bat failing

Describe the bug
bootstrap-vcpkg.bat faile with error:
..\src\vcpkg\versiondeserializers.cpp(112): error C2440: 'initializing': cannot convert from 'initializer list' to 'vcp
kg::SchemedVersion' [C:\vcpkg\toolsrc\windows-bootstrap\vcpkg.vcxproj]
  ..\src\vcpkg\versiondeserializers.cpp(112): note: No constructor could take the source type, or constructor overload
  resolution was ambiguous

Environment

OS: Windows 10 Pro, x64
Compiler: VS 2015, Update 3

To Reproduce
Steps to reproduce the behavior:

Just run bootstrap-vckpg.bat
Fails with the above error

Expected behavior
It should.. build.
Failure logs
  versiondeserializers.cpp
..\src\vcpkg\versiondeserializers.cpp(112): error C2440: 'initializing': cannot convert from 'initializer list' to 'vcp
kg::SchemedVersion' [C:\vcpkg\toolsrc\windows-bootstrap\vcpkg.vcxproj]
  ..\src\vcpkg\versiondeserializers.cpp(112): note: No constructor could take the source type, or constructor overload
  resolution was ambiguous
C:\vcpkg\scripts\bootstrap.ps1 : Building vcpkg.exe failed. Please ensure you have installed Visual Studio with the
Desktop C++ workload and the Windows SDK for Desktop C++.
At line:1 char:4
+ & {& 'C:\vcpkg\scripts\bootstrap.ps1' }
+    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,bootstrap.ps1

ScriptHalted
At C:\vcpkg\scripts\bootstrap.ps1:411 char:5
+     throw
+     ~~~~~
    + CategoryInfo          : OperationStopped: (:) [], RuntimeException
    + FullyQualifiedErrorId : ScriptHalted

Additional context
None.

@MoonStruckHorrors, could you get latest source and try again? I can't repro it locally with vs2015 update3.
S E:\vcpkg\vcpkg> git rev-parse head
c2329ab26a6e9e06da101d8541c393c102ba6c8c

I have the same issue (VS2015 update3, Win10 20H2) using the latest sources (c2329ab).
  visualstudio.cpp
..\src\vcpkg\versiondeserializers.cpp(112): error C2440: 'initializing': cannot convert from 'initializer list' to 'vcp
kg::SchemedVersion' [D:\projects\vcpkg\toolsrc\windows-bootstrap\vcpkg.vcxproj]
  ..\src\vcpkg\versiondeserializers.cpp(112): note: No constructor could take the source type, or constructor overload
  resolution was ambiguous
  vcpkg.cpp
D:\projects\vcpkg\scripts\bootstrap.ps1 : Building vcpkg.exe failed. Please ensure you have installed Visual Studio
with the Desktop C++ workload and the Windows SDK for Desktop C++.
In Zeile:1 Zeichen:4
+ & {& 'd:\projects\vcpkg\scripts\bootstrap.ps1' -disableMetrics}
+    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,bootstrap.ps1

ScriptHalted
In D:\projects\vcpkg\scripts\bootstrap.ps1:411 Zeichen:5
+     throw
+     ~~~~~
    + CategoryInfo          : OperationStopped: (:) [], RuntimeException
    + FullyQualifiedErrorId : ScriptHalted

Likewise. identical issue, though my output lists vcpkg.cpp just before the first failure. This is from a pull from origin/master not 5 minute ago, so I have the "latest source".
  visualstudio.cpp
  vcpkg.cpp
..\src\vcpkg\versiondeserializers.cpp(112): error C2440: 'initializing': cannot convert from 'initializer list' to 'vcp
kg::SchemedVersion' [C:\Projects\vcpkg\toolsrc\windows-bootstrap\vcpkg.vcxproj]
  ..\src\vcpkg\versiondeserializers.cpp(112): note: No constructor could take the source type, or constructor overload
  resolution was ambiguous
C:\Projects\vcpkg\scripts\bootstrap.ps1 : Building vcpkg.exe failed. Please ensure you have installed Visual Studio
with the Desktop C++ workload and the Windows SDK for Desktop C++.
At line:1 char:4
+ & {& 'C:\Projects\vcpkg\scripts\bootstrap.ps1' }
+    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,bootstrap.ps1

ScriptHalted
At C:\Projects\vcpkg\scripts\bootstrap.ps1:411 char:5
+     throw
+     ~~~~~
    + CategoryInfo          : OperationStopped: (:) [], RuntimeException
    + FullyQualifiedErrorId : ScriptHalted

On a hunch, I picked the 2020.07 tag and gave it another go. No issues:
  versiont.cpp
  visualstudio.cpp
  vcpkglib.vcxproj -> C:\Projects\vcpkg\toolsrc\msbuild.x86.release\vcpkglib.lib
  vcpkg.cpp
  Generating code
  All 25384 functions were compiled because no usable IPDB/IOBJ from previous compilation was found.
  Finished generating code
  vcpkg.vcxproj -> C:\Projects\vcpkg\toolsrc\msbuild.x86.release\vcpkg.exe
  vcpkg.vcxproj -> C:\Projects\vcpkg\toolsrc\msbuild.x86.release\vcpkg.pdb (Full PDB)
  vcpkgmetricsuploader.cpp
  Generating code
  All 11117 functions were compiled because no usable IPDB/IOBJ from previous compilation was found.
  Finished generating code
  vcpkgmetricsuploader.vcxproj -> C:\Projects\vcpkg\toolsrc\msbuild.x86.release\vcpkgmetricsuploader.exe
  vcpkgmetricsuploader.vcxproj -> C:\Projects\vcpkg\toolsrc\msbuild.x86.release\vcpkgmetricsuploader.pdb (Full PDB)

Building vcpkg.exe... done.

Telemetry
---------
vcpkg collects usage data in order to help us improve your experience.
The data collected by Microsoft is anonymous.
You can opt-out of telemetry by re-running the bootstrap-vcpkg script with -disableMetrics,
passing --disable-metrics to vcpkg on the command line,
or by setting the VCPKG_DISABLE_METRICS environment variable.

Read more about vcpkg telemetry at docs/about/privacy.md

So.. whatever broke, it was between those commits.

Likewise. identical issue, though my output lists vcpkg.cpp just before the first failure. This is from a pull from origin/master not 5 minute ago, so I have the "latest source".
  visualstudio.cpp
  vcpkg.cpp
..\src\vcpkg\versiondeserializers.cpp(112): error C2440: 'initializing': cannot convert from 'initializer list' to 'vcp
kg::SchemedVersion' [C:\Projects\vcpkg\toolsrc\windows-bootstrap\vcpkg.vcxproj]
  ..\src\vcpkg\versiondeserializers.cpp(112): note: No constructor could take the source type, or constructor overload
  resolution was ambiguous
C:\Projects\vcpkg\scripts\bootstrap.ps1 : Building vcpkg.exe failed. Please ensure you have installed Visual Studio
with the Desktop C++ workload and the Windows SDK for Desktop C++.
At line:1 char:4
+ & {& 'C:\Projects\vcpkg\scripts\bootstrap.ps1' }
+    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,bootstrap.ps1

ScriptHalted
At C:\Projects\vcpkg\scripts\bootstrap.ps1:411 char:5
+     throw
+     ~~~~~
    + CategoryInfo          : OperationStopped: (:) [], RuntimeException
    + FullyQualifiedErrorId : ScriptHalted

On a hunch, I picked the 2020.07 tag and gave it another go. No issues:
  versiont.cpp
  visualstudio.cpp
  vcpkglib.vcxproj -> C:\Projects\vcpkg\toolsrc\msbuild.x86.release\vcpkglib.lib
  vcpkg.cpp
  Generating code
  All 25384 functions were compiled because no usable IPDB/IOBJ from previous compilation was found.
  Finished generating code
  vcpkg.vcxproj -> C:\Projects\vcpkg\toolsrc\msbuild.x86.release\vcpkg.exe
  vcpkg.vcxproj -> C:\Projects\vcpkg\toolsrc\msbuild.x86.release\vcpkg.pdb (Full PDB)
  vcpkgmetricsuploader.cpp
  Generating code
  All 11117 functions were compiled because no usable IPDB/IOBJ from previous compilation was found.
  Finished generating code
  vcpkgmetricsuploader.vcxproj -> C:\Projects\vcpkg\toolsrc\msbuild.x86.release\vcpkgmetricsuploader.exe
  vcpkgmetricsuploader.vcxproj -> C:\Projects\vcpkg\toolsrc\msbuild.x86.release\vcpkgmetricsuploader.pdb (Full PDB)

Building vcpkg.exe... done.

Telemetry
---------
vcpkg collects usage data in order to help us improve your experience.
The data collected by Microsoft is anonymous.
You can opt-out of telemetry by re-running the bootstrap-vcpkg script with -disableMetrics,
passing --disable-metrics to vcpkg on the command line,
or by setting the VCPKG_DISABLE_METRICS environment variable.

Read more about vcpkg telemetry at docs/about/privacy.md

So.. whatever broke, it was between those commits.

Same issue, resolved by using 2020.07 instead of master.

@BillyONeal, it looks a real issue, however I couldn't repro this issue locally, could you please take a look?

@strega-nil was able to repro this and is taking a look

I could repro this issue with vs2015 update3 now, I just found the toolset was replaced with more higher revision in my test machines. so I didn't repro the issue before.
Thank you @strega-nil for fixing this issue.


updating minitest notebook for 20.4

This PR updates the minitest notebook for the 20.4 software release reference run.
A new config option max_hour_angle was needed in the desisurvey-config.yaml file, and various tweaks were needed to the notebook itself to adapt to the new tiling pattern.
Outputs are currently in /global/cscratch1/sd/sjbailey/minitest-20.4 and will be moved to /global/cfs/cdirs/desi/datachallenge/reference_runs/20.4 once merged and desitest 20.4 tagged.

Eagle-eyed @alxogm noticed that the surveysim here did a PASS=4 dark tile before it did the overlapping PASS=1-3 tiles, which means that I didn't update the minitest notebook surveysim to match the new tile file where the gray layer is PASS=0 instead of 4.  Good catch.  Hold off on merging for me to update that and rerun.

@alxogm I fixed the desisurvey-config.yaml file so that it observes tiles in the correct order now.  I think this is ready to merge and tag for 20.4


Feature/advanced generator selection

Addresses #186
Created issue so it can be referred in the changelog.

Thanks for all these changes! The generator code has been problematic in the past, with probably the most issues (especially with Visual Studio).
A few questions:

Does this fulfill  #186 completely?
Do all the tests still pass?
Have you poked around with this running in the debugger, to see if it all behaves as expected?
I don't think the cmake.preferredGenerators option is used anymore in this PR. Is that correct?

Does this fulfill #186 completely?

Yes. The issue is a placeholder could be mentioned in the changelog.

Do all the tests still pass?

I had issues with running smoke tests because I need to have some environment active, otherwise even if I put ninja into path configure fails because it won't find compiler. Found that I can add it to test_project/.vscode/.cmaketools.json. (or probably running under vcvars would work too).
I have 4 tests failing atm. Will update comment after done fixing them.

Have you poked around with this running in the debugger, to see if it all behaves as expected?

Yes, that's how I did it. Although I prefer to have tests - poking every time is expensive.

I don't think the cmake.preferredGenerators option is used anymore in this PR. Is that correct?

No, it has precedence over 'suggested' generator. Look at getPreferredGenerators() in common.ts. It first takes generators from config and then appends ones from EnvironmentProvider. I decided to pass whole Generator objects around, so both toolset and architecture are not lost.
I can't say that populating toolset and platform is consistent in all scenarios, but I definitely think that platform from EnvironmentProvider should take precedence over one in configuration. Otherwise there will be inconsistency like mentioned in one of recent issues.

ok, test updates:

Configure respects environment overrides - I don't see how it could pass before for Server Mode, because CMake is ran as server before configuration is altered...

Yeah... There are a few flaky tests. I need to work a bit towards making the tests more robust, but they cover a lot ATM (just not as much as I'd like).
Thanks for the catch of configureEnvironment. That's one that's bugged me and I never realized the problem was CMake Server. Oops.
Anyway, this all looks pretty good. I'll get it in and see about releasing it. Hopefully it will clear up some generator problems people have been having...


Update page weights for concepts/overview/working-with-objects

This PR updates the page weights for concepts/overview/working-with-objects
for the 2022 Docs Sprint at Contributor Summit
Related to https://github.com/kubernetes/website/issues/35093
/assign @natalisucks

/lgtm
/approve


Add init section to script operator

Describe the problem you are trying to solve
With more complex pipelines and scripts that make use of the state a repeated pattern is initializing the state. As of now, this has to happen in every invocation of the script matching on state being set or not. This makes the script logic harder to follow and adds often unnecessary performance overhead.
Describe the solution you'd like
A init (or state?) section is called at the start of the script returning the initial state of the script operator. An example would be:
define script apply_rate
script
  match state of
    case null => let state = 0
    case _ => null
  end;
  match event of
    case %{ present update_rate } => let state = event.update_rate, drop
    case _ => null
  end;
  let $class =  "default";
  let $rate = state
end

could become:
define script apply_rate
init
  0
script
  match event of
    case %{ present update_rate } => let state = event.update_rate, drop
    case _ => null
  end;
  let $class =  "default";
  let $rate = state
end

Alternatively, init could process events to build up an initial state until it doesn't drop an event, which would allow seeding a script operator with multiple events to build a state.
Notes
This does likely require an RFC.
If this is an implementation of an RFC provide a URL
to the RFC this enhancement implements.
If this is a major enhancement or contribution an RFC may be required. It is ok to submit an enhancement
first and our core team will assist with major contributions. In general, major contributions should be
discussed with the community before submission.

I really really like that!
I also think this needs an RFC.

Ja I'm writing one already for this and #933 as they kind of touch on the same area

This is the RFC for this: https://github.com/tremor-rs/tremor-rfcs/pull/46

this is implemenmted


Potential issue with OnceCallback

I was recently looking at the Mdns::Publisher implementation and its use of OnceCallback class in callback.hpp.
I think there may be an issue with the implementation, and it may not work correctly with all STL library implementations. The implementation seems to indirectly assume that a move constructor or a move assignment will clear the rvalue reference from which it moves. However, this is not guaranteed by the C++ standard and behavior is tied to specific class and its implementaion.
https://github.com/openthread/ot-br-posix/blob/34c3899c0cb28a22404380b0626bbfe872dcf6b4/src/common/callback.hpp#L82-L89
We are using std::function<>. I found this related to this type which does not guarantee to clear (null) the moved-from rvalue reference:
(3-4) Copies (3) or moves (4) the target of other to the target of *this. If other is empty, *this will be empty after the call too. For (4), other is in a valid but unspecified state after the call.
ref: https://en.cppreference.com/w/cpp/utility/functional/function/function

Should be resolved by #1913.


Inline images to base64

Could this be considered?

Maybe - could you post an example of how you envision the workflow?

I'm using this with gulp to build userscripts (greasemonkey). User scripts don't support local images, so they need to be bundled in the html.

I'd recommend you inline the images before you pass them to smoosher, with something like https://github.com/Wenqer/gulp-base64.

@gabrielflorit That's for stylesheets. I'm after inlining images in html, i.e. <img> tags.

Oh I see. Interesting. There's gotta be a plugin for that. Hmm.

Here's one: https://github.com/zckrs/gulp-css-base64

@jgwconsulting that's for stylesheets.

Oops, it would help if I could read properly! Maybe this one: https://github.com/rabidaudio/gulp-inline-image-html


(Revert): Monaco commented && restored code-mirror

@dmytroshch please revert package-lock.json related to bfx-hf-* modules,
PR check fails and I get same error locally
npm ERR! Could not install from "node_modules/bfx-hf-algo/@bitfinex/lib-js-util-math@git+https:/github.com/bitfinexcom/lib-js-util-math.git" as it does not contain a package.json file.


fix: update color of usernames & boldness of usernames on the fly

Boldness was already covered before
Fixes #2637

This should be ready for a review now @Mm2PL if you want to take a peek
To test, mention a user in a normal & a /me message, then flick the Bold @usernames and Color @usernames settings on and off


Ensure we can fold apply of a differentiable_function_inst.

Also fix one small potential issue while there.
Fixes #65489

@swift-ci please test

Tagging @BradLarson @rxwei @dan-zheng

@swift-ci please test


Readability issues with unchecked checkboxes in light mode

When using this theme and light mode setting (and otherwise everything else default too), unhecked checkboxes seem to blend in completely
The second line has an unchecked checkbox:

Here's checking the checkbox for reference:

Using Origami <IP_ADDRESS>

i just pushed a potential fix, can you let me know if it looked better?

@7368697661 thanks for the heads up, I've checked it out and it seems to be very nicely readable in all the settings combinations I've tried. I think this can be considered closed from my side. Cheers for this! 👏


covid19_cases_map.ipynb: Problem in installation of cartopy

Required attestation

[Y] I have completely and exactly followed the virtual environment setup instructions from the book and have consulted the README for any updates.
[Y] I have cloned either this repository or my fork of this repository so that I have all necessary files on my local machine.
[Y] I have checked that this issue has not already been reported or resolved here.
[Y] I am aware that there is a pre-built Binder environment that I can use, but I want to install locally on my machine instead.

Background information
1. Which OS are you using?
TODO: Provide your OS here – make sure to differentiate between Intel and M1 chip Macs.
Mac M2.
2. Which Python version are you using?
TODO: Provide your Python version here.
python 3.9 for base and python 3.8 for book_env
3. Are you using conda or venv?
TODO: Indicate whether you are using conda or venv.
conda

Commands run and their outputs
Please provide all of the commands you ran as well as the traceback:
Dear Stefanie:
I have some problems in running the "covid19_cases_mpa.ipynb"  in chapter 6 and installation of cartopy. The detail descriotion is in the "details" section.
Any help will be highly appreciated.
Best regards,
Li-Xin Li
(Before posting this message I did a search on google but did not get a clear solution.)

First, I installed cartopy with "base" by running "conda install -c conda-forge cartopy". I found that I cannot import cartopy as I run the jupyter lab in "book_env". As I run "import cartopy.crs as ccrs" in the jupyter notebook "covid19_cases_map.ipynb" I got a message saying that "ModuleNotFoundError: no module named 'cartopy'".
Then, in a terminal with "book_env" environment, I run "conda install -c conda-forge cartopy" again (without uninstallation of the cartcopy in "base" environment). Then in a jupyter notebook in "book_env" I can run "import cartopy.crs as ccrs" with no problem. That is, I went through "covid19_cases_map.ipynb" with out a problem.
However, after I finished "covid_cases_map.ipynb" then started dong the exercises in chapter 6 in a new jupyter notebook, I got another problem:
In a jupyter notebook in "boon_env" I run
#---------------------------------------------------------------------
%matplotlib inline
import matplotlib.pyplot as plt
import numpy as np
import pandas as pd
import seaborn as sns
fb = pd.read_csv('data/fb_stock_prices_2018.csv', index_col='date', parse_dates=True)
quakes = pd.read_csv('data/earthquakes.csv')
covid = pd.read_csv('data/covid19_cases.csv').assign(
date=lambda x: pd.to_datetime(x.dateRep, format='%d/%m/%Y')
).set_index('date').replace(
'United_States_of_America', 'USA'
).sort_index()['2020-01-18':'2020-09-18']
#------------------------------------------------------------------
I got the following mesage:
"<frozen importlib._bootstrap>:219: RuntimeWarning: scipy._lib.messagestream.MessageStream size changed, may indicate binary incompatibility. Expected 56 from C header, got 64 from PyObject"
I never had this problem before installing cartopy.
Does my process in installing cartopy has some problems? By installing a cartopy in "base" then installing another cartopy in "book_env", I realize that there may be two different versions of cartopy on my M2. Does this cause some problem of inconsistency? Should I uninstall the one with the "base" enviroment?
I admit that after installing the cartopy, I updated the "anacondo" by running "conda update -n base -c defaults conda" as being reminded during installation of cartopy. Did this case the problem?
TODO: paste commands and any traceback here

Screenshots
Optionally, include any screenshots that will help diagnose the issue.

You shouldn't install things in the base environment in general. Based on your error, I believe your version of scipy changed. Does your setup still pass the checks from chapter 1 (checking_your_setup.ipynb)?
If anything has changed such that it no longer passes the checks, you will need to downgrade those packages or recreate the environment as you did in chapter 1.

Hi Stefanie:
I reinstalled everthing, but got the same problem. I reinstalled anaconda3 (the same version as before), created an environment book_env with Python 3.8, then reinstalled required packages for your book (pip3 install -r requirements.txt), then installed cartopy (run 'conda install -c conda-forge cartopy' in book_env). Then I still get the message
"<frozen importlib._bootstrap>:219: RuntimeWarning: scipy._lib.messagestream.MessageStream size changed, may indicate binary incompatibility. Expected 56 from C header, got 64 from PyObject"
exactly as before.
While for checking up the set up, I got the result as shown in the figure below so it didn't go through. But this problem existed before. Since it did not affect my working with other notebooks (text and excercises) in your book, I did not report this problem to you. But of course if you can give me somme highlight I will be very grateful.
Best wishes,
Li-Xin

@stefmolin
Update:
I reinstalled anaconda3 with the latest version. Then I create book_env with Python 3.8 (as before). Then, I installed the required python packages for the book as described in chapter 1, in the book_env environment.
Then, I run checking_your_setup.ipynb to check the set up. It passed the check without a problem. Everythong is OK so far.
Then, I installed cartopy by 'conda install -c conda-forge cartopy' in book_env. The installation process went through smoothly. With "conda list" I see that the scipy has the version 1.7.3, both before and after the installation of cartopy. That is, as I run conda list in book_env, I got
..........................
scipy                     1.7.3                    pypi_0    pypi
...........................
both before and after the installation of cartopy).
(But during the installation process I saw the following message:
"
The following NEW packages will be INSTALLED:
...........................
scipy              conda-forge/osx-64::scipy-1.10.0-py38hfb8b963_0 None
...........................
"
and after the installation I did not see scipy-1.10.0.)
Then I run checking_your_setup.ipynb to check the set up again. This time the check did not get through. I stll get the message
<frozen importlib._bootstrap>:219: RuntimeWarning: scipy._lib.messagestream.MessageStream size changed, may indicate binary incompatibility. Expected 56 from C header, got 64 from PyObject
The details are in the follwoing figures.

Any help will be great appreciated!
Best wishes,
Li-Xin

Update again:
I think the problem is solved. By running 'scipy.version' and 'np.version' in the notebook I found that the versions of scipy and numpy had indeed been changed by the installation of cartopy. (I still don't understand why the command 'conda list' in the terminal does not show the change.) After downgrading the versions of scipy and numpy to the versions before installation of cartopy, the problem mentioned in the above post disappears. However, as I ran 'checking_your_setup.ipynb to check' I got two messages saying that the versions of matlibplot and requests also need be adjusted. After installlation of matlibplot and requests with the required versions (a little bit complicated since I had to find the channels for downloading the required packages), everything seems OK now.
Thank you very much for your kind help! (Now I know that I should not install packages in the base environment in general.)
Best wishes,
Li-Xin Li


Add nonpassive modifier to touchmove and touchstart events

Problem
Chrome complains of non-passive event listener violations if certain listeners are not explicitly marked as non-passive.

Solution
Svelte has created a convenient way to add a nonpassive event modifer like so:
  on:touchmove|nonpassive
  on:touchstart|nonpassive

This PR applies the modifier to events in Alert Dialog Content and Dialog Content

@huntabyte Should I generating patch changesets for these PRs?

I'll generate it, no worries @ronkeiser!


feat: update velero version

Automated changes by the update-addons GitHub action

addon images should only be updated in main, not in the k0s version branch