Failed to Run Benchmark for llama-3-8b-instruct and llama-3.1-8b-instruct Models.

I attempted to run benchmarks for the llama-3-8b-instruct and llama-3.1-8b-instruct models using both CPU and GPU, but the process failed.    (I successfully tested the llama2-7b-chatbot model)
I followed the instructions in openvino_notebooks/llm-chatbot.ipynb to download the models and ensured that all necessary files (including the required tokenizer.model) were included. I am using the latest version of OpenVINO (2024.3.0) and have also upgraded the transformers library.
The command I executed is:
python benchmark.py -m {path}/openvino_notebooks/notebooks/llm-chatbot/llama-3-8b-instruct/INT4_compressed_weights -n 2 -d CPU -p "What is large language model (LLM)?"
And I received the following error output:

Adding -ic 512 option should work around this issue.
We have a PR to fix this issue, but there is a performance regression, WIP on the analysis.

Thank you for your response. Adding the -ic 512 option indeed resolves the issue.
When can we expect the full solution to be available?

@tim102187S It has been fixed on master branch, please try with latest commit ID of master branch.

Fixed by https://github.com/openvinotoolkit/openvino.genai/pull/801, further default configuration update for OOB performance https://github.com/openvinotoolkit/openvino.genai/pull/1049


get_viewport().size returns (2, 2) in the editor

Godot version
v4.1.1.stable.official [bd6af8e0e]
System information
Godot v4.1.1.stable - Windows 10.0.22621 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3090 Ti (NVIDIA; <IP_ADDRESS>68) - Intel(R) Xeon(R) CPU E5-2689 0 @ 2.60GHz (16 Threads)
Issue description
get_viewport().size returns (2, 2) inside the editor. I expect it to return the viewport size.
There were two instances where it worked properly for me (in both cases the script was reloaded in an already loaded scene) but I couldn't reproduce that. In all my attempts the script didn't work without reloading the scene, after which the issue appeared.
Steps to reproduce
In the included project the scene (node_3d.tscn) should reproduce the issue ((2, 2) is printed in the console)
Minimal reproduction project
test.zip

get_viewport() returns the 2D editor viewport. If you never switched to 2D, the size will be wrong.

Thanks


Question: Identity of the Dataset

Hi,
I would like to know,

What are the datasets used for the graph below?
What version of Scikit-Learn is used of comparison?

Duplicate of https://github.com/IntelPython/scikit-learn_bench/issues/135
Configuration was shown below graph:
Configurations:

HW: c5.24xlarge AWS EC2 Instance using an Intel Xeon Platinum 8275CL with 2 sockets and 24 cores per socket
SW: scikit-learn version 0.24.2, scikit-learn-intelex version 2021.2.3, Python 3.8


Seems like general reference complex variable doesn't work

I go to dynamodb module only to validate (as I had lots of issues finally :-)), there is definitely one AWS related bug in terraform at moment:
https://github.com/hashicorp/terraform/issues/21330
It requires to define in main.tf to have:
provider "aws" {
  region = "${var.region}"
}

and define default via variables.tf
The real issue here is the reference to complex types:
local_secondary_index       = "${var.dynamodb_table_local_secondary_index[count.index]}"
global_secondary_index      = "${var.dynamodb_table_secondary_index[count.index]}"
ttl                         = "${var.dynamodb_table_ttl[count.index]}"

I get:
rror: Unsupported argument

  on main.tf line 19, in resource "aws_dynamodb_table" "table":
  19:   local_secondary_index  = "${var.dynamodb_table_local_secondary_index[count.index]}"

An argument named "local_secondary_index" is not expected here. Did you mean
to define a block of type "local_secondary_index"?

Error: Unsupported argument

  on main.tf line 20, in resource "aws_dynamodb_table" "table":
  20:   global_secondary_index = "${var.dynamodb_table_secondary_index[count.index]}"

An argument named "global_secondary_index" is not expected here. Did you mean
to define a block of type "global_secondary_index"?

Error: Unsupported argument

  on main.tf line 21, in resource "aws_dynamodb_table" "table":
  21:   ttl                    = "${var.dynamodb_table_ttl[count.index]}"

Not sure what is the precise issue you are having but they may also be related to HCL 0.12 changes. You could either try using Terraform 0.11.x or try doing terraform 0.12upgrade to convert it all to 0.12 HCL. PR welcome ;)

@crisboarna - yeah in general I fixed all issues, so will push them as PR

Added updates for Terraform 0.12.


undefined: background.Set

This fails to build for me.
I'm on OS X (El Capitan). Same error with Go versions 1.5.3 and 1.6rc2.
$ go build github.com/avinashbot/himawari
# github.com/avinashbot/himawari
github.com/avinashbot/himawari/main.go:55: undefined: background.Set

Looks like https://github.com/avinashbot/himawari/pull/1 is a fix for this.


Missing few settings options on your github page

http://kenwheeler.github.io/slick
waitForAnimate and zIndex options are missing.
Did you think about open sourcing your github page so that we can also send you pull requests and keep it up to date?

There was a repo for the github page @kenwheeler will create it again.


04/16/2018 23:52 : Create Pull Request for branch prTesting_2018041623 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[Do not merge] Add temporary images that include pipeline helper v2

Required to test the new pipeline helper release without breaking existing pipelines.
https://github.com/uni-passau-artemis/pipeline-helper/releases/tag/v2.0.0

Change applied to main branch.


feat: Feature/azure service principal

PR fulfills these requirements

[ ] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[ ] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[ ] Backend (Database)
[ ] Backend (API)
[ ] Frontend

Describe the reason for change
(link to issue, supportive screenshots etc.)
What does this fix?
(if this is a bug fix)
What is the new behavior?
(if this is a breaking or feature change)
What is the current behavior?
(if this is a breaking or feature change)
What libraries were added/updated?
(list all with version changes)
Does this change affect performance?
(if so describe the impacts positive or negative)
Does this change affect security?
(if so describe the impacts positive or negative)
What alternative approaches were there?
(briefly list any if applicable)
What feature flags were used to cover this change?
(briefly list any if applicable)
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[ ] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[ ] integration
[ ] unit

Which logical domain(s) does this change affect?
(for bug fixes/features, be as precise as possible. ex. Authentication, Annotation History, Review Stream etc.)

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 83.52941% with 56 lines in your changes missing coverage. Please review.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Flag
Coverage Δ

pytests
76.89% <83.52%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

...l_studio/io_storages/azure_serviceprincipal/api.py
100.00% <100.00%> (ø)

..._storages/azure_serviceprincipal/openapi_schema.py
100.00% <100.00%> (ø)

...studio/io_storages/azure_serviceprincipal/utils.py
100.00% <100.00%> (ø)

label_studio/io_storages/base_models.py
75.26% <100.00%> (ø)

label_studio/io_storages/functions.py
100.00% <100.00%> (ø)

label_studio/io_storages/urls.py
100.00% <100.00%> (ø)

label_studio/core/settings/base.py
75.23% <10.00%> (ø)

.../io_storages/azure_serviceprincipal/serializers.py
84.09% <84.09%> (ø)

...tudio/io_storages/azure_serviceprincipal/models.py
80.81% <80.81%> (ø)


Update Snapchat

Site name
Snapchat
Site URL
https://snapchat.com
Update reason
Other (please describe below).
Additional information
Snapchat is still rolling out passkey and isn't currently available everywhere (tested Australia).
Issue Eligibility

[X] The issue I'm creating is not a duplicate of an existing issue.
[X] The issue I'm creating is not a duplicate of an existing pull request

Hi @stevene1919,
Thanks for raising the request for this one to be added. I have actually added this one to our Passkey directory via this PR. I'll close this one off now.


fix: prepare 8.27.0 hotfix

This PR prepares 8.27.0 for submission
Description
#nochangelog

Warnings

:warning:

✅ No changelog changes

  Generated by :no_entry_sign: dangerJS against e6abf4040f791034aad76814066b0e99f8dc7a2d


decode: pick unsupported decoding profile out

driver only supported profile BASELINE, MAIN, HIGH; so profile CAVLC 4:4:4,EXTENDED, HIGH10, HIGH 4:2:2, HIGH 4:4:4 will be picked out to stop its decoding, and those decodings will end up with an error information.

+1


Frontend web development html css js

Introduction to functions in javascript #3228📜
I have Explained what functions are in javascript with its Syntax and rules along with a sample code showing the working of function with output.

Topic Covered📝

What is Javascript Function?
Why Functions?
Rules & Syntax
Sample Code
Output

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[x] Frontend Dev HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

/assign documentation

Please check my PR for this issue.


Go 1.18 version support

Hi again,
When i tried to run your example in for this library on go1.18.10
i got this error
../../../go/pkg/mod/github.com/speedata/baseline-pdf@v0.0.0-20230621085521-8a602a1d42b8/writer.go:60:18: undefined: strings.CutPrefix
note: module requires Go 1.20

is it expected because i use go1.18.10?

I will take a look, thank you!

I am not sure that we have the same version. I've cloned the library
git clone https://github.com/speedata/bagme-examples.git

and in the directory bagme-examples/basic/simple I run these commands
go version
go version go1.18.10 darwin/arm64

and
go run main.go

which produces the document I'd expect
How do you clone/run this example?

BTW: I will update all the bagme stuff in the next few days (I am currently moving the CSS handling to the boxesandglue library).
I will have to switch to go version 1.20 (and later this year to go version 1.21) since I am now using the slog structured logging module which will be part of the 1.21 standard library. In 2024 I will stop using the latest go versions and be a bit more conservative.

I am not sure that we have the same version. I've cloned the library
git clone https://github.com/speedata/bagme-examples.git

and in the directory bagme-examples/basic/simple I run these commands
go version
go version go1.18.10 darwin/arm64

and
go run main.go

which produces the document I'd expect
How do you clone/run this example?

Idk what i did but when i tried the steps you mentioned the issue is missing :"D
So i guess the issue resolved itself haha, thanks for helping and answering!

BTW: I will update all the bagme stuff in the next few days (I am currently moving the CSS handling to the boxesandglue library).
I will have to switch to go version 1.20 (and later this year to go version 1.21) since I am now using the slog structured logging module which will be part of the 1.21 standard library. In 2024 I will stop using the latest go versions and be a bit more conservative.

Okai noted, will be watching this repository.
Good luck!

:D


Cargo build issue

Environment
mac
rustc 1.64.0-nightly (7665c3543 2022-07-06)
Delta-rs version:
Latest main branch

Bug
What happened:
Cargo build get the belowing errors:
error[E0252]: the name tls is defined multiple times
and
error[E0599]: no function or associated item named newfound for structhyper_rustls::HttpsConnector in the current scope
Anybody met this before? Thanks.
What you expected to happen:
How to reproduce it:
More details:

I've encountered this but haven't found a way to solve it. It originates from the aws/delta-checkpoint folder, so I've been working around it by building from the sub-directories that I care about instead of at the top-level (that is, cd rust && cargo build). But it would be nice to get the top-level fixed.
See past discussion: https://delta-users.slack.com/archives/C013LCAEB98/p1654306625879469

It works. Thanks.


Babel doesn't compile node_modules

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

I notice that next.js ignores the node_modules directory from being compiled by babel. It will cause some errors in old version browsers.
For example, after npm run dev in example of data-fetch, it would generate the following code in .next/main.js
/* 347 */
/***/ (function(module, exports, __webpack_require__) {

"use strict";

const ansiRegex = __webpack_require__(348);

module.exports = input => typeof input === 'string' ? input.replace(ansiRegex(), '') : input;

/***/ }),
/* 348 */
/***/ (function(module, exports, __webpack_require__) {

"use strict";

module.exports = () => {
	const pattern = [
		'[\\u001B\\u009B][[\\]()#;?]*(?:(?:(?:[a-zA-Z\\d]*(?:;[a-zA-Z\\d]*)*)?\\u0007)',
		'(?:(?:\\d{1,4}(?:;\\d{0,4})*)?[\\dA-PRZcf-ntqry=><~]))'
	].join('|');

	return new RegExp(pattern, 'g');
};

These code is generated from strip-ansi, which is one of the dependencies of next.js.
However, Chrome browsers whose version < 41 would throw error of "SyntaxError: Use of const in strict mode" according to caniuse. In addition, a few old browsers don't recogize arrow functions.
How can I make babel compile node_modules so that my project can run on these old version browsers?

Babel does not compile node_modules. You should raise this issue with (strip-ansi)[https://github.com/chalk/strip-ansi] . They should provide a provide a compiled version.

I think this is intentional.
And for this specific issue we are dealing it here: https://github.com/zeit/next.js/pull/3403


"The specified task cannot be tracked" while waiting for user input variable in a "preLaunchTask"

VSCode Version: 1.50.0
OS Version: Windows 10

It seems that when starting a launch configuration that has a preLaunchTask that itself uses an input variable, we get "The specified task cannot be tracked" after 10 seconds of waiting for user input. When the same task is run manually via Run Task everything works fine. I believe this behaviour is not correct and that VSCode should never mark a task as unresponsive when in fact it has not been started yet. It should block and wait for user input without a warning. I'm not sure if there are any workarounds to this problem.
Steps to Reproduce:

Create an input variable and a task that references it in tasks.json
Create a launch configuration in launch.json that uses that task as its preLaunchTask
Launch the configuration and wait for 10 seconds

@isidorn, before I try to change this: how would you expect debug to behave in this scenario?

@alexr00 ideally debug would simply not complain in this case and wait indefenitely.
The issue is this timeout here that does not take into account that a user is about to enter input.

Pushed a fix. Note that if you cancel the input then the launch config will still execute. This is consistent with the existing behavior.


Dotnet6 runtime extension v1.1.0 with bug fixes

Dotnet6 runtime extension version v1.1.0 with minor bug fixes for insider build
Changes:
Bug fixes.

@sridharanprasanna next time you can use just one PR to update both extensions. https://github.com/microsoft/azuredatastudio/pull/21340

changes published for both extensions.

@sridharanprasanna next time you can use just one PR to update both extensions. #21340

Sure, will do. Thanks.


not update and not set new value created in popup

after new foreignkey create , need refresh page for use this new value

Sorry I think you are talking about the admin right? That doesn't work and won't work even in the implementation for Django itself that is coming in Django 1.11.
Sorry


Failure to create images directory

{ [Error: ENOENT: no such file or directory, mkdir '.../Library/Application Support/electron-audio-player/images']
  errno: -2,
  code: 'ENOENT',
  syscall: 'mkdir',
  path:
   '.../Library/Application Support/electron-audio-player/images' }
Application is ready

Operating userData is after application launch. Since the constructor of MusicMetadataReader is before ready, it fails


added box office scraper

@sbuschbach @dtomc @rmazumdar This is the scraper code for the box office info. Someone review my PR?

Good to go! @dhanus


enable action cable

Enable action-cable, maybe even broadcast the job info to a channel.
I'd say a simple test of this is to inline and/or add some javascript that simply starts listening to that channel and uses console.log to print the messages.
┆Issue is synchronized with this Asana task by Unito

Fixed in #29


PHP Brackets are red

Type: Bug
Hello,
I just updated to VSC 1.76.0 and now my opening and closing PHP brackets are red, just as if there are no closing brackets. My php is not broken, and my local environment functions fine.
I usually rely on the colors of my characters, to tell me if there is an error, so this is a pretty big deal for me. Also, when I highlight an opening bracket, it does not highlight its relative closing bracket. something is broken with that update. Thank you.
A
VS Code version: Code 1.76.0 (Universal) (92da9481c0904c6adfe372c12da3b7748d74bdcb, 2023-03-01T10:23:45.993Z)
OS version: Darwin x64 22.3.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7500 CPU @ 3.40GHz (4 x 3400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
2, 3, 3

Memory (System)
24.00GB (0.01GB free)

Process Argv
--crash-reporter-id ee40ecba-3463-4fa5-91b4-7657061c1ba0

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

apacheconf-snippets
eim
1.3.0

auto-close-tag
for
0.5.14

vscode-pull-request-github
Git
0.60.0

pro-deployer
hem
1.2.1

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30669237
azure-dev_surveyone:30548225
pyindex848:30662994
nodejswelcome1cf:30587006
3biah626:30602489
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
showlangstatbar:30672706
vsccsb:30677849
pythonb192:30669360
funwalk2:30676043

Is this #175797? Are you able to repro in the latest Insiders?

Seems this is happening when you close your <?php> tags. When the tags are left open, the colorization is correct. This is a new bug from the update.
Closed php tags

Open php tags

Duplicate of #175797


Fix a bug in is-expr with readonly arrays

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

$title
Fixes #34879
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Closing this because a new PR will be sent to patch branch.


Self intersection in hybrid

Describe the bug
Hybrid reconstructs world position from depth buffer, this ends up into a lot of precision loss. This means that bigger scenes have more self intersection.
To Reproduce
Go to emibl with the epsilon value of 0.01 (or something small). Raytracing will look correct, but hybrid won't.
Expected behaviour
Hybrid should look identical to raytracing.
Screenshots
Don't have RTX screenshots, but classic self intersection (black lines and artifacts when moving the screen around, disappear or get less when you get closer).
Desktop (please complete the following information):
Tower
Additional context
N.A.

Closing for now. Its planned and not really a bug since it is expected behavior.


Improve article 1103.4507: \today should be date of submission perhaps

The date in arXiv:1103.4507 has a \today in it, which appears to be getting compiled to an empty string. Shouldn't it be compiled to the date of submission?

Thank you for the report! This enhancement is beyond the current scope of ar5iv as a project.
It may be better-handled in a future where arXiv itself builds the HTML pages - a first version of which is coming by the end of the year.
The technical problem is that the submission date is completely external to the TeX source bundles - so it requires a custom build process to 1) obtain the metadata from arXiv OAI and then 2) override \today via some latexml mechanism. It is possible, but it is much easier from within, as arXiv can leverage their DB information directly.
Let me add that I believe this is currently also unsolved in arXiv's PDF workflow - \today will represent the build date of the PDF, rather than the submission date of the article.
I suspect it is generally better for authors to use a fixed date in such cases, and avoid the \today macro.
Closing here, but I am bookkeeping the issue in the "arXiv integration" milestone, so it is easy to find in the future.

Oh yes, I agree the arXiv should fix this on their end.
I try to get rid of \today whenever I post on the arXiv, but sometimes I forget, and so do many other authors.


2021-07-12-Experience-being-a-data-fellow-at-Code-for-Nepal-an…

…d-internship-at-Open-Knowledge-Nepal.md

we also need to add this photo to go with the blogpost


[table/prometheus-operator] Cannot upgrade after rules change ("prometheusrulemutate.monitoring.coreos.com": the server could not find the requested resource)

Describe the bug
I'm trying to upgrade my prometheus-operator release with  only a few (for time changes) on helm. Every time I run helm upgrade --namespace monitoring prometheus-operator stable/prometheus-operator -f ./values.yaml
UPGRADE FAILED
ROLLING BACK
Error: failed to create resource: Internal error occurred: failed calling admission webhook "prometheusrulemutate.monitoring.coreos.com": the server could not find the requested resource
Error: UPGRADE FAILED: failed to create resource: Internal error occurred: failed calling admission webhook "prometheusrulemutate.monitoring.coreos.com": the server could not find the requested resource

If I keep the values.yml file unchanged I get no error.
Version of Helm and Kubernetes:
helm version
Client: &version.Version{SemVer:"v2.13.0", GitCommit:"79d07943b03aea2b76c12644b4b54733bc5958d6", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.13.0", GitCommit:"79d07943b03aea2b76c12644b4b54733bc5958d6", GitTreeState:"clean"}

Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.6", GitCommit:"ab91afd7062d4240e95e51ac00a18bd58fddd365", GitTreeState:"clean", BuildDate:"2019-02-26T12:59:46Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"12+", GitVersion:"v1.12.10-eks-825e5d", GitCommit:"825e5de08cb05714f9b224cd6c47d9514df1d1a7", GitTreeState:"clean", BuildDate:"2019-08-18T03:58:32Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
stable/prometheus-operator
What happened:
I changed some for keys on my values.yml (git diff bellow):
-          for: 30m
+          for: 10m

-          for: 20m
+          for: 5m

What you expected to happen:
To get the new for keys applied.
I also noticed I can't apply any change to any rule.
How to reproduce it (as minimally and precisely as possible):
Have the version of the chart installed:
prometheus-operator     74              Mon Oct  7 19:48:29 2019        FAILED          prometheus-operator-6.7.3       0.31.1          monitoring

Change alert rules in your values.yml file.
Apply the upgrade.
Anything else we need to know:
I tried setting --set prometheusOperator.admissionWebhooks.enabled=false and even removing or editting ValidatingWebhookConfiguration prometheus-operator-admission and the prometheus-prometheus-operator-prometheus-rulefiles-0 configmap.
I always got the same.

Try the --force flag to let helm recreate necessary resources

@bood It didn't help.
I got the same message.
I tried downloading the chart (my current version, previous version or latest version) removing the webhooks templates hard setting enable: false to prometheus-operator and the webhook. I got the same message every time.

Ok, more information, I deleted --purge it and installed it again (with the same PVCs).
helm upgrade --install --namespace monitoring prometheus-operator stable/prometheus-operator -f ./values.yaml --install  --version 6.13.0

I'm using helm3 for prometheus-operator now. I do remember seeing some kind of error on helm2 when upgrading too (I was using helm v2.11.0 but I didn't record the error message).
I suggest you try: upgrade helm2 to latest version or use helm3 instead.

Hmm. It's still beta, right?
Plus, I would have to upgrade several installations (releases). Anyway, I'm gonna take that into consideration (thank you). But will keep this open for a while.

still happening even upgrade to helm 2.16

if you have run before without disabling the webhook, you must manually delete the following kinds:
kubectl get validatingwebhookconfigurations.admissionregistration.k8s.io   //delete all objects
kubectl get MutatingWebhookConfiguration //delete all objects
and after that run:
helm install --name prometheus-operator stable/prometheus-operator
--set prometheusOperator.admissionWebhooks.enabled=false
--set prometheusOperator.admissionWebhooks.patch.enabled=false
--set prometheusOperator.tlsProxy.enabled=false


Corrected spelling errors in doc/dev/coding-guidelines-and-review.txt

Fixed spellings in doc/dev/coding-guidelines-and-review.txt. Fixes issue #801.

[x] Is it mergable?
[x] Did it pass the tests?
[x] If it introduces new functionality in scripts/ is it tested?
Check for code coverage with make clean diff-cover
[x] Is it well formatted? Look at make pep8, make diff_pylint_report,
make cppcheck, and make doc output. Use make format and manual
fixing as needed.
[x] Did it change the command-line interface? Only additions are allowed
without a major version increment. Changing file formats also requires a
major version number increment.
[x] Is it documented in the ChangeLog?
http://en.wikipedia.org/wiki/Changelog#Format
[x] Was a spellchecker run on the source code and documentation after
changes were made?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

jenkins please test this

jenkins ok to test

jenkins add to test list

Ready to review!

Jenkins, test this please

Ready to review.

LGTM - thanks!


Some commits

Hello,
Please find here two commits.
One is forcing the jenkins to be created as a "system" user.
The second is allowing to disable the download of the .jar file to allow installing it using a system package

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

This looks reasonable but is missing unit tests.
travis didn't run for some reason -- maybe a glitch?

@bigon This PR needs a rebase.  Are you still interested in getting this merged?

Hi,
I've rebased the changes


[Localization]Warning did not translated when created an UWP Application without enable Developer mode

[Environment]
OS: Windows-10-Enterprise-19H1-VSDrop-ja-jp
VS: 16.8 Preview 5(d16.8/30615.102)
Repro steps:

Create a UWP Application

Don't enable Developer mode

[Expected]
Warning should be translated
[Actual]
Warning did not translated

The warning comes from Microsoft.Windows.UI.Xaml.Build.Tasks.dll in the Windows 10 SDK.

Warnings and errors are always presented in English, this is by design for errors in Winui to reduce our package size. I'm not positive about this specific error but it is a pretty common pattern to not locallize these, just like we don't localize our API names.

There are certain expectations from Microsoft products to comply with Accessibility, World Ready, and other requirements. Package size concerns shouldn't trump those.

@SvetBonev there is another issue for that https://github.com/microsoft/microsoft-ui-xaml/issues/2299
As for me, it is controversial topic. A lot of developers do not prefer localized exception messages.

@maxkatz6 Thank you for the pointer to the other issue.
Because this is an error message issued by MSBuild which is part of Visual Studio, the error message needs to follow the Visual Studio localization rules.
It is a matter of providing consistent experience within the Visual Studio product as well as ensuring regulatory compliance with the laws of different countries and regions.

@jevansaks FYI.


feat: Enable dependabot

Signed-off-by: Gábor Lipták<EMAIL_ADDRESS>https://github.com/pepperize/cdk-organizations/pull/17

@gliptak there is only a minor conflict remaining 👍 please resolve


Backend CSV Handling Issues

Your bug (What bug you found):
CSV handling on the backend is causing errors in the code. These errors cause the leaderboard function to not display anything and cause the backend CSV append operation to fail.
Potential fixes (Optional, write a potential fix):
Write a new version of the backend code that is streamlined further than it is. The new code should be able to handle the appending and acquiring.
Operating System (What OS you are using):
Windows 10
Additional Notes (Any additional notes about the issue):
No additional notes.

Working a fix currently, it should solve issues on the backend. If this fix does not work I will go for a redesign.


mprime-primenet: init at 30.19

Description of changes
This package adds official release builds for mprime. mprime is already packaged on nixpkgs but the existing package builds from source which prevents the binary from running workers connected to primenet, which is the easiest way to participate in GIMPS (Great Internet Mersenne Prime Search). I've tested running this with workers connected to primenet on both x86-linux and x86-macos with no issues so far.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] 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.11 Release Notes (or backporting 23.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Rebased and update to use pkgs/by-name

Updated package to latest version


Report a critical error when deploy to java webapp in some cases

Steps to reproduce:
Create a new webapp with java11+linux+Java SE > Deploy jar package to the app > Get the result as below
Also repro with java17+linux+tomcat10+war package
Actual Behavior:
Shows a critical error

But the website is deployed succeed which could be access with browse website

According to @nturinski this is an issue with the linux detectors. @Flanker32 does this still reproduce?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Add mock for kubernetes provider

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/approve


cosmit &  domain name length clarification

Apart from cosmetic changes, there was a bug with domain name length check because Unicode string's len() in fact is not the same as byte slice's len().

Oh wait… I believed it counts characters. Well, my bad.
In such case, it is just "cosmit", I guess.

OK. I'll clean it up later today.


not work with ie11

IE11 will report syntax error in _app.js

Can confirm that the package doesn't work with IE11. There is an ES6 arrow function in browser.js on line 157.

confirmed

As a workaround, you can add debug package to your project


Email sign up forms: Inset email module needs code snippet

Which page is this about?
https://cfpb.github.io/design-system/patterns/e-mail-signup-forms#types-1
What kind of issue is this?
Add one or more of the issue labels below to the right-hand sidebar AND issue title.
(We strongly encourage you to make content changes yourself where possible!)

Needs code snippet

Describe your issue
Add code snippets for both sidebar and inset email examples in the types section
Size this request (1=tiny, 5=enormous)
3
Add any additional background info as needed
Make sure NOT to include internal links.
Thanks. Don't forget to add labels indicating issue type and size before submitting!
Once you've submitted your issue, please add it to the content backlog project board: https://github.com/cfpb/design-system/projects/1

Added second PR that removes duplicate code from CFGov
https://github.com/cfpb/consumerfinance.gov/pull/6749

@brandon-rusk We discussed this during sprint planning, and decided to scope this down to include only the regular email signup piece. Here are the tasks (please edit as needed)

[ ] Remove the inset email from your PR
[ ] @anselmbradford will create a separate issue to preserve comments and next steps from the original PR around the inset email sign up
[ ] Merge and release into the Design System
[ ] Brandon has PR to remove redundant code – update it with the changes in the latest design system release

I think this can be closed. There's an example on https://cfpb.github.io/design-system/patterns/e-mail-signup-forms


chore: update cli setup flow

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #473 (b026491) into main (59e2a0d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #473   +/-   ##
=======================================
  Coverage   93.98%   93.98%
=======================================
  Files          46       46
  Lines        1762     1762
  Branches      393      393
=======================================
  Hits         1656     1656
  Misses        102      102
  Partials        4        4

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 59e2a0d...b026491. Read the comment docs.


dockerfile uses an old image

Description of Issue
The dockerfile is referencing an old (no longer supported by Microsoft) dockerimage:
FROM microsoft/dotnet:2.1.0-preview2-aspnetcore-runtime-alpine

As such, this needs to change to be in line with the current (at the time of creating this issue) supported image fr ASP NET Core:
FROM microsoft/2.1-aspnetcore-runtime-alpine

This issue has been fixed in pull request #16.

Close #16


After multiple/quick dashboard reload top-bar and side-bar disappear

Current Behavior
After multiple dashboard reload (F5 in web browser) top and side bar disappear and does not reappear until browser cache is cleared.
This is happening after upgrade from 1.4.1 to 1.8.1 . After reverting to 1.4.1 problem disappears.
Expected Behavior
Fully functional dashboard
Steps To Reproduce

Open Dashboard
Reload quickly few times

Environment

Dashboard version: 1.8.1
Node-RED version: 3.1.3 - 3.1.9
Node.js version: no idea, bundled with official NR docker image
npm version: no idea, bundled with official NR docker image
Platform/OS: Windows/ Mac
Browser: Firefox, Chrome, safari

Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

I have tried quite hard but have failed to replicate this. I am running on ubuntu and not using Docker.

In my case I can reproduce this issue on Windows/Linux/Mac + Firefox/Chrome/Safari . I don't know how much it matters but the Dashboard is behind an Nginx Proxy with websockets active, it shouldn't change anything since as I mentioned earlier 1.4.1 works without a problem. Something must have changed in the Dashboard 2.0 code between 1.4.1 and 1.8.1.

The main thing that changed between those releases was the configuration option for the sidebar type if I recall correctly. Why clearing the cache affects this though, I have no idea - we don't store anything in session storage or cookies 🤔

Is there any way I can assist you to debug this issue?

a screen recording if you're able to would be ideal

It turned out that somehow when opening the URL Dashboard 2.0 also served data from the native Dashboard in the background, (you can see it in the local storage of the browser console) which causes some kind of garbage in the browser cache. Removing the node-red-dashboard from the palette is not enough, you need to physically remove the directory from disk. More interestingly, in the dockerized version od node-red, after some time this add-on reinstalls itself. I got around the problem by setting the permissions of the node-red-dashboard directory to 000, which is an indirect solution, but it works. Any idea why this is happening?

Yeah you say "native dashboard", what do you mean?

Yeah you say "native dashboard", what do you mean?

Dashboard 1.0 package node-red-dashboard

you need to physically remove the directory from disk

Which directory?  If you uninstall it then that should remove it completely.

you need to physically remove the directory from disk

Which directory?  If you uninstall it then that should remove it completely.

node-red-dashboard from modules dir. Pressing uninstall button on the list in palette manger does not remove this directory. That's just 1 half of the problem. The second half is why is it served together with dash 2.0 under /dashboard url?

node-red-dashboard from modules dir.

Exactly which directory?  Please try and answer questions fully.

even after deleting that dir it magically comes back after a while (few hours or days). Why? How?

Show us your package.json file after you uninstall the dashboard.


[Suggestion] Select multiple photos and videos across different albums and download the selection as .zip file

Hi there,
I would like to suggest an enhancement for selecting and downloading photos and videos.
At the moment I can either select and download one photo/video or a complete album with all its media files. I would like to suggest the enhancement that I could select several photos/videos across different albums and download my selection as a zip file.

Hi,
thank you for the nice suggestions. This is a more complex feature request. That is not on my roadmap for the next few years. I'm happy to collaborate on a design doc for this feature. Can you take the lead on it?
I'm also happy to review PR.

Hi Patrik,
I try to make a user story for this.
As a PiGallery2 user, I want an easy way to select multiple photos and video clips across all my albums and download the selected items as a zip archive. This process would incorporate at least:

While hovering over a photo or video thumbnail, display a checkbox or a circular area on the top left corner where a user can select or deselect the particular assets
Display a counter on the top menu bar:

"n items selected" if >=1 photo or video clip is selected

display a "X" on the right side of the counter for "deselect all"

display an icon for "Download all n items" on the right side of the "X"

After pressing "download all n items"

a .zip archive is downloaded with this naming convention: pigallery2_ archive_YYYYMMDD_HHMMSS.zip

after finishing the download all n items are deselected

Some questions to think about::

What is a suitable archive size?
What happens when the suitable archive size is exceeded?

I second this request: this year again, my family wants to download a selection of photos from the album I shared with them. Unable to do so, they just download the whole album (hundreds of photos with sub-folders) and deleted the unwanted one afterwards...
@wechsler42 it doesn't look like there is any "suitable archive size" concept when download a whole album: it either just works, or doesn't. So I don't think we should bother with that in a first implementation.
Your way of handling the selection looks rather nice; however the selection from multiple folders might make things quite complex: maybe, as a first step, we could consider downloading a selection of photos from a single folder ?

The way that would bring us closer to this feature, I think should start with consolidating the backend code for folder listing and searching.  (making folder listing as a special case of search).
That would allow a lot of things. One of them would be downloading search results.
Downloading search results already a step forward into downloading partial folders.
As I expressed multiple place in Readme and https://github.com/bpatrik/pigallery2/discussions/738, I'm not implementing feature requests.
My current goal is to enable some reasonable extension  #743 and let users extend the app the way they prefer.


Fix: update DynconfigOptions typo

Signed-off-by: Jim Ma<EMAIL_ADDRESS>
Description
Update DynconfigOptions typo
Related Issue

Motivation and Context

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist:

[x] Code compiles correctly.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #390 (a25934e) into main (ce4c5cf) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #390      +/-   ##
==========================================
- Coverage   42.41%   42.34%   -0.08%
==========================================
  Files          73       73
  Lines        5640     5640
==========================================
- Hits         2392     2388       -4
- Misses       2900     2905       +5
+ Partials      348      347       -1

Flag
Coverage Δ

unittests
42.34% <ø> (-0.08%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

scheduler/config/config.go
0.00% <ø> (ø)

client/daemon/peer/peertask_base.go
33.49% <0.00%> (-0.98%)
:arrow_down:


F1 currently requires that you highlight the word...

... would be nice if it could determine the word that the cursor currently resides and utilize that word when launching help.  Primary reason is as one is typing, navigating with cursor keys, etc. it feels more natural (to me, suspect others) to simply be able to press F1 to go to help instead of having to highlight the word first before doing so.

Code is completed Planed fore release 0.3.0

Dine


Need offline UI: Connection interrupted

Original issue is #135, which has been split into multiple issues.
We need UI specs and implementation for:

When the bot connection has been interrupted.

Immediate interruption (including immediate reconnect?)
Non-immediate interruption (up to 2 mins)

What we need

[ ] Design specs

[ ] Acceptance criteria

[ ] T-shirt sizing

Grey text

With spinner (spinner will be grey, small, size of icon in previous Offline UI items)

Text proposal: “Network interruption occurred. Reconnecting…”

This will show up as soon as an interruption has occurred, and either disappear (reconnect successful) or switch to “Web Chat is taking longer than usual to connect” after 15 seconds. (Will eventually turn to “Connection failed” if it fails completely)

Tagged wrong issue :(


Feature request: password change

It seems that Haier makes the account password change once a year.
However, it is not possible to change the password within the integration and we are forced to delete and reconnect. This will involve renaming all the devices all over again.
Is it possible to implement password changing?
Alternatively can anyone tell me if any configuration file containing the password can be changed? possibly the path what is it?
Thank you

You cannot access the password. I will work on this

From one week my devices are no longer working with the integration.
The log report a bad login.
hOn app ask me to change the password, so I did it.
Now I deleted the devices and try to re-sync again but the integration always say: "Your login attempt has failed. Make sure the username and password are correct."
I'm sure the credentials are correct. Checking multiple times.
Are there issues with some special chars in password?
Thank you.

Hello,
Today I change my hOn account password with another one with only "-" (minus) as special char in password string.
It worked!
So, I can confirm that the issue is with some special chars in password (such as "$" or similar chars).


Can I use child or children as the first parameter of the widget? Let all components look like this Text('counter' ).

@zdu-strong
Your question is better suited for support channels like StackOverflow, so please ask the same there.
Closing from here as this isn't an issue or feature request.


[BUG] $black variable is missing from daff-button theme functions in ng11

:bug: Bug report
Current Behavior
Currently, I can't set use the design library on latest Angular 11.
Expected Behavior
I expected the design library to work.
Minimal reproduction of the problem with instructions
I was following the getting started guide and I got the following error when I added the theme:
Error in node_modules/<EMAIL_ADDRESS>(2186:18)
Undefined variable.
╷
2186 │ 0 3px 5px rgba($black, 0.12),
│ ^^^^^^
╵
../node_modules/<EMAIL_ADDRESS>2186:18 daff-raised-button-theme-variant()
../node_modules/<EMAIL_ADDRESS>2381:3 daff-button-theme()
../node_modules/<EMAIL_ADDRESS>3603:2 daff-theme()
../~/src/styles.scss 4:1 root stylesheet

Stackblitz
What is the motivation / use case for changing the behavior?
I tried to use the design library.
Environment

Daffodil version: 0.39.9
Angular version: 11.2.14

Fixed via #2173


Publish events to an Event Grid Domain

In the case of the publishing event using a default input event schema such as the EventGridSchema, the AEG will populate a full path of the topic resource in the event message, like it is expected.
But, in the case of the publishing event using a CloudEventV01Schema for input schema, the topic part of the source property is not populated to the full topic resource path, for example:
"source": "abc#myapp/vehicles/motorcycles",

it should be populated by AEG in the full topic resource path, for example:
 "source": "/subscriptions/xxxxxxxxxxx-xxxx-xxxx-xxxxxxxxxxxx/resourceGroups/myRG/providers/Microsoft.EventGrid/domains/myDomain/topics/abc#myapp/vehicles/motorcycles",

Am I missing something or is this a bug?
The problem is that the event subscription and subscriber didn't receive a full topic resource path.
Thanks
Roman

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

ID: d061a5b9-7841-eff7-73e4-196fd451596a
Version Independent ID: 1c297d9b-fb7a-6206-4ff8-ca85d9c6c443
Content: Manage large sets of topics in Azure Event Grid with event domains
Content Source: articles/event-grid/how-to-event-domains.md
Service: event-grid
GitHub Login: @banisadr
Microsoft Alias: babanisa

@romankiss
Thanks for your feedback! We will investigate and update as appropriate.

@romankiss Yes we are not updating that. Conceptually topic indicates where the event is posted and source is where the event originated. Ideally source should stay unchanged once it set by the event publisher as the event flows from one sub-system to the other.
If someone posts a grid schema event, then we are mapping “topic#subject” as source (if the subscription specified cloud events as the out schema). We applied a similar mapping rule for accepting cloud events. While it works for some scenarios, it is confusing as that interpretation is not part of the cloud event spec. Instead to achieve full compliance with cloud event specification, grid service should instead be letting customers map a specific property as domain topic and not impose any special rules on the source field.
Please note that Cloud event spec is still being finalized and Product group are pursuing addition of source and subject as two separate fields. Based on how it goes Product group will adjust how cloud events are supported by grid.

@DixitArora-MSFT  Thanks for your details. I have agreed, that the topic indicated a "publisher place", where the event is posted. That's working well for posting a grid schema event and the AEG populates the full topic resource path.
Based on that, it should be the topic updated by AEG to the full resource path also for posting a cloud schema event.
Note, that the topic path in the posted event is validated by AEG and must be matched with a full topic resource path of the "publisher place".
Anyway, I have still a question for this issue.
Do we have to populate a full topic resource path (included an azure subscription id, etc.) in the source property of the posted cloud event or the AEG will be made it?
Thanks
Roman
p.s.
Mapping the input cloud event schema to the output grid event schema will generate the following error:
 'Deployment has failed with the following error: The specified EventDeliverySchema value "EventGridSchema" cannot be used in combination with the topic's input schema of CloudEventV01Schema'

@romankiss  Suggestion from Product Group is to use event grid input schema for your domain and also receive events in grid schema as that helps you with getting full ARM id. You can also consider adding it as part of data section if it is needed for event handling.

@DixitArora-MSFT Using an event grid input schema requires to use an array of the events instead of the single event cloud input schema. Also, the custom input schema allows to customize only an event grid schema. It should be nice to have an option for customizing events to cloud event schema. In many cases will help this issue around the source, topic, subject, etc.

@romankiss Raise an User voice for this and do upvote it for the traction. For now we will proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Wrong number of pages created

Hello!
Thank you for this great css!
I have the following issue; when I create 2 A4 sheets, the print preview (option-P) on Mac shows 3 pages, nr 1 and nr 3 are correct, page nr 2 is an empty sheet.
I set the padding to 25 mm which is way larger than the real margins of my printer.

Sorrry for late reply. It looks that it depends on the content.
paper-css defines just the size of the paper. If the content of a sheet is bigger than the hight of the paper specified, your browser break its page automatically.


[<IP_ADDRESS>] very minor bug with SHOW MEASUREMENTS and empty measurements

SHOW MEASUREMENTS returns nothing if all measurements in the database are empty, even if they exist.
I accidentally deleted all points in the database when I hit this (fixed) bug: https://github.com/influxdb/influxdb/issues/4276. As such I have a database with defined measurements but no points. SHOW MEASUREMENTS returns nothing until I write the first point to any measurement, and then it returns the right list of measurements.
> use mydb
Using database mydb
> show measurements
> insert foo value=12
> show measurements
name: measurements
------------------
name
foo
h2o_feet
ifHCInUcastPkts
math

> select * from /.*/
name: foo
---------
time				a	b	location	status!	value	water_level
2015-10-10T00:41:54.213966399Z						12

>

The other columns in the SELECT * FROM /.*/ output are from the other measurements.

@beckettsean I think the measurement itself might be removed if you delete all points from the measurement. If this is the case, you wouldn't expect to see the measurement anymore if you called SHOW MEASUREMENTS.

this is an 0.9 bug anyway, I vote we close until someone sees it with 0.13+

I tested it before commenting. It still happens, but I think it's intended.


Please make an effect for the focus spell Elemental Betrayal

https://2e.aonprd.com/Spells.aspx?ID=797

This is likely not automated because of the complexity. I don't know if there's a way to automate this currently.


implement line/word diff

We don't currently merge two strings if both contents has been modified since the have branched from a common ancestor. To improve that, need to have a look at https://github.com/janestreet/patdiff but it seems to depend on https://github.com/janestreet/core_extended ... @diml any chance to have the patdiff library usable on mirage (ie. pure ocaml code) ?

See also https://github.com/gjaldon/simple-diff


Extra capability annotations for Easse and JSON RPC

These commits provide extra provide-capability annotations for

Easse web resource : to be used when you want to include the Easse javascript angular module
Json RPC web resource: to be used when you want to include the Json RPC javascript angular module
Json RPC extender: to be used to indicate a dependency to Json RPC provider when you mark a service as Json RPC endpoint

By the way, when I add my @JsonWebresource annotation to a type definition this leads to the following Require-Capability in the manifest: osgi.enroute.webresource;filter:="(&(osgi.enroute.webresource=/osgi/enroute/jsonrpc)(&(version>=1.1.1)(!(version>=2.0.0))))". Only when I explicitly add resource={"jsonrpc.js"} this is reflected in the manifest: osgi.enroute.webresource;resource:List<String>="jsonrpc.js";filter:="(&(osgi.enroute.webresource=/osgi/enroute/jsonrpc)(&(version>=1.1.1)(!(version>=2.0.0))))".
Is it normal that the default attributes of the annotation are not reflected in the require-capability header or is this a bug in bnd?

This might be a bug ... please open an issue for it

Ok, opened an issue for that: https://github.com/bndtools/bnd/issues/976


Fix the issue with the tzdata noninteractive setup

Fixing the Github Action failing during build; step freezes when installing tzdata

merged! thanks for your contribution!


Counter in dataviewer

Discussed in https://github.com/microsoft/vscode-jupyter/discussions/11081

Originally posted by KrisvanderMast August  9, 2022
When using the dataviewer in Jupyter notebook I can easily filter. That's great but I can't see how many records remain of the total when filtering. For example if I have 30.000 records displayed and filter on a column first name for Kris I'd love to see how many records have Kris in their first name column.
This discussion started first out on Twitter: https://twitter.com/KrisvanderMast/status/1554898271172272133

Closing this issue as there aren't enough upvotes to warrant this feature.


Secondary language support

Is your feature request related to a problem? Please describe.
As a polyglot, i find it frustrating to see recommendation in only one language. Is it possible to set add a secondary language in the settings.
Describe the solution you'd like
The current recommendation system is curated, and the json returned is divided by languages. It should be possible to query the same category of two different languages.
Additional context
I understand there are maybe other more urgent issues to solve before this enhancement.

+1! Having recommendations based on the used language would be amazing!


Make future_utils a dev-dependency

This dep was used for future::poll_fn, which has been stabilized in rust 1.64.0:
https://blog.rust-lang.org/2022/09/22/Rust-1.64.0.html
Keeping it for tests since they also use future::join.
Thank you for the awesome tls crates!

Thanks!


is not imported by your project etc... Dep claims it added in vendor for me but no

What version of dep are you using (dep version)?

dep:
 version     : v0.4.1
 build date  : 2018-01-27
 git hash    : 37d9ea0
 go version  : go1.9.3
 go compiler : gc
 platform    : darwin/amd64

What dep command did you run?
dep ensure -add github.com/aws/aws-sdk-go-v2

What did you expect to see?
It said I have not imported it yet and added to lock file and vendor folder. It does added into lock file but definitely not in vendor folder
What did you see instead?
It should add it for me.
I managed to fix it if I delete the vendor folder and run dep ensure again.

There are so many bugs introduce when external dependencies change their structs or functions signature. Yet go get obviously can't save developers and dep seems not working fine too. Package management is a big down side to choose to use Golang

I need to run rm -rf vendor/ && dep ensure everytime. I have used PHP composer, JS bower, JSPM and NPM. This is the first package manager with such problem and nobody cares. Yet I pretty much have no alternative as a Go developer.

No I was originally try to add AWS package only. Dep updated the lock and toml file only without installing anything. I need to rm-rf vendor && dep ensure to force dep download it for me.

i'm sorry, but dep's whole design is centered around that specifically not happening, and running dep ensure -add github.com/aws/aws-sdk-go-v2 doesn't demonstrate any such problem for me. i can certainly believe that there was some poor UX that made it appear that this was the outcome, but without more precise reproduction steps, i can't classify this as a bug.

dep ensure --add github.com/101medialab/go-redis-distributed-lock
Fetching sources...

"github.com/101medialab/go-redis-distributed-lock" is not imported by your project, and has been temporarily added to Gopkg.lock and vendor/.
If you run "dep ensure" again before actually importing it, it will disappear from Gopkg.lock and vendor/.

In ./vendor/github.com/101medialab, there is NO go-redis-distributed-lock package installed.

And it presents after I run rm -rf vendor/ && dep ensure. I am going to close this issue if you report it is not reproducible. Just let the pain in my ___. I got used to it already and expected nothing tbh.

If the project you're running the command against is public, it might help with reproduction. But otherwise, sorry - i've gotta close this, as i can't reproduce it 😞

Same exact thing happening to me with a different package. Probably shouldn't have closed this.
dep ensure --add github.com/SAP/go-hdb/driver
Fetching sources...

"github.com/SAP/go-hdb/driver" is not imported by your project, and has been temporarily added to Gopkg.lock and vendor/.
If you run "dep ensure" again before actually importing it, it will disappear from Gopkg.lock and vendor/.

Hello,
Encountered the same issue while trying to add new dependencies on my new fresh project like so :
dep ensure -add github.com/gorilla/mux/ github.com/jinzhu/gorm   github.com/dgrijalva/jwt-go github.com/joho/godotenv
I've found something that might helps :

Delete the vendor folder
Do dep ensure command
The following warning should appear:

Warning: the following project(s) have [[constraint]] stanzas in Gopkg.toml:

  ✗  github.com/dgrijalva/jwt-go
  ✗  github.com/gorilla/mux
  ✗  github.com/jinzhu/gorm
  ✗  github.com/joho/godotenv

However, these projects are not direct dependencies of the current project:
they are not imported in any .go files, nor are they in the 'required' list in
Gopkg.toml. Dep only applies [[constraint]] rules to direct dependencies, so
these rules will have no effect.

Either import/require packages from these projects so that they become direct
dependencies, or convert each [[constraint]] to an [[override]] to enforce rules
on these projects, if they happen to be transitive dependencies.

Maybe it is because you're adding new dependencies that is not imported in any of your files


Incorrect attribution for Faraday

In the JSON, you have:
    {
        "pattern": "Faraday",
        "name": "Laughable",
        "slug": "laughable.com",
        "url": "https:\/\/laughable.com\/"
    },

Faraday is a Ruby library for doing HTTP requests, so you can't determine who is actually making requests using it absent other identifying information in the User-Agent.

Thanks! Deleted.


improved meson files & fixed qt-build and example

I've noticed some issues with your latest changes to the buildfiles.
this should fix them :D
also btw, your changes introduced by 10a0e258c3433ab7d3d2fca6de7b0a694d89ddee have broken the otf2clm script, since it expects --batch or --single as first arg, but passes the first arg (in non-batch mode) as the otf filepath to batch_parse and instead it ignores the third passed param.

also btw, your changes introduced by 10a0e25 have broken the otf2clm script, since it expects --batch or --single as first arg, but passes the first arg (in non-batch mode) as the otf filepath to batch_parse and instead it ignores the third passed param.

fixed by 11dbe01


Preventing abusive signups

I see there is a feature to prevent abusive logins but nothing to prevent abusive number of signups. There is email verification but shouldn't there be a captcha at signup page?

Thanks for opening an issue. :sunglasses:
Althogh we do validate that emails look like valid email addresses, we're not verifying deliverability. We left those design choices up to the developer. If someone wanted to prevent abusive signups they could use a lot of logic/code found in the abusive login detection.
Now if you leveraged a service like like recaptcha, you may not need to manage that extra server side logic.
I'd like to see these types of features become articles/guides that go into the wiki. Drywall actually has an article on this. If you're willing to write one I'd be happy to collaborate and get it up on the wiki.


MIR misses useful "developer" options

built-in disassembler
ability to debug JIT-ed code with GDB (__jit_debug_register_code)
ability to monitor JIT-ed code with sampling profilers (Linux Perf, Oprofile, VTune)

Thank you for the proposals.  I'll put it on my to do list.  Although I don't know when they will it be available.
Currently I debug JITted code with switching on printing debug info where I can see addresses of generated code and can put breakpoint on it and use low level gdb commands, nexti/stepi, disass, etc.  But using names instead addresses would be more convenient.
As for profilers, I am on the stage where I mostly implement infrastructure.  When I do work on generated code tuning, I'll definetely need this.


chart: add sqlite

Closing, because SQLite schemas and corresponding migrations can be used with ejabberd's new automatic schema migration function.


Update forwarding.md

Generally we don’t bother with such small changes. Just something to note going forwards.

My bad, thanks for telling > Generally we don’t bother with such small changes. Just something to note going forwards.


[BUG] pandas 1.5.x _MergeOperation doesn't have copy keyword anymore

PR Description
Please describe the changes proposed in the pull request:

Remove copy from _MergeOperation

This PR resolves #1174.

PR Checklist

Please ensure that you have done the following:

[x] PR in from a fork off your branch. Do not PR from <your_username>:dev, but rather from <your_username>:<feature-branch_name>.

[x] If you're not on the contributors list, add yourself to AUTHORS.md.

[x] Add a line to CHANGELOG.md under the latest version header (i.e. the one that is "on deck") describing the contribution.

Do use some discretion here; if there are multiple PRs that are related, keep them in a single line.

Automatic checks
There will be automatic checks run on the PR. These include:

Building a preview of the docs on Netlify
Automatically linting the code
Making sure the code is documented
Making sure that all tests are passed
Making sure that code coverage doesn't go down.

Relevant Reviewers

Please tag maintainers to review.

@ericmjl

latest ci passed, see https://github.com/pyjanitor-devs/pyjanitor/actions/runs/3255206793/jobs/5344282656

Will it affect users with pandas < 1.5 ?

default copy is True, nothing will be changed. It may create some temporary variables and get a little bit slower.
It wouldn't change the inputs or outputs.

If False, avoid copy if possible.
https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.merge.html

looks ok to merge @ericmjl @thatlittleboy @pyjanitor-devs/core-devs thoughts?


add css clases for scroll bar style in chrome and safari

Nikola, can you take these in, they were not in the master branch yet


Creating PMMLs from lifelines

Following the conversation on https://github.com/CamDavidsonPilon/lifelines/issues/188
Thought I'd create this issue so it can be tracked and a solution can be found.
It would be awesome if we can figure out a way to create PMML/PFAs from lifeline models as that is the standard
Currently, PMML creation using sklearn2pmml does not work on lifelines because of a pickling error.

I spent some time in this today.
I tried sklearn2pmml as that is kind of the standard that is available. It is from jpmml.
In it, you first try to make a PMMLPipeline and then try to dump that pipeline to PMML.
Code:
import pandas as pd
from sklearn2pmml.pipeline import PMMLPipeline
from lifelines.utils.sklearn_adapter import sklearn_adapter
from lifelines import CoxPHFitter

df = pd.DataFrame({
    'T': [5, 3, 9, 8, 7, 4, 4, 3, 2, 5, 6, 7],
    'E': [1, 1, 1, 1, 1, 1, 0, 0, 1, 1, 1, 0],
    'var': [0, 0, 0, 0, 1, 1, 1, 1, 1, 2, 2, 2],
    'age': [4, 3, 9, 8, 7, 4, 4, 3, 2, 5, 6, 7],
})

CoxSklearnRegression = sklearn_adapter(CoxPHFitter, event_col='E')
pipeline = PMMLPipeline([
    ("regressor", sklearn_adapter(CoxPHFitter, event_col='E')())
])
pipeline.fit(df[['var', 'age', 'E']], df[['T']])

print(pd.DataFrame({
    'pred': pipeline.predict(df[['var', 'age', 'E']]),
    'actual': df['T'].tolist(),
}))

Creating the PMML Pipeline worked great.
After this, when I try:
from sklearn2pmml import sklearn2pmml
sklearn2pmml(pipeline, "coxph.pmml", with_repr = True)

It errors:
Standard error:
Oct 04, 2019 10:16:35 PM org.jpmml.sklearn.Main run
INFO: Parsing PKL..
Oct 04, 2019 10:16:35 PM org.jpmml.sklearn.Main run
SEVERE: Failed to parse PKL
net.razorvine.pickle.PickleException: failed to __setstate__()
	at net.razorvine.pickle.Unpickler.load_build(Unpickler.java:409)
	at net.razorvine.pickle.Unpickler.dispatch(Unpickler.java:234)
	at org.jpmml.sklearn.PickleUtil$1.dispatch(PickleUtil.java:77)
	at net.razorvine.pickle.Unpickler.load(Unpickler.java:123)
	at org.jpmml.sklearn.PickleUtil.unpickle(PickleUtil.java:98)
	at org.jpmml.sklearn.Main.run(Main.java:104)
	at org.jpmml.sklearn.Main.main(Main.java:94)
Caused by: java.lang.NoSuchMethodException: net.razorvine.pickle.objects.ClassDict.__setstate__([Ljava.lang.Object;)
	at java.lang.Class.getMethod(Class.java:1786)
	at net.razorvine.pickle.Unpickler.load_build(Unpickler.java:406)
	... 6 more

Exception in thread "main" net.razorvine.pickle.PickleException: failed to __setstate__()
	at net.razorvine.pickle.Unpickler.load_build(Unpickler.java:409)
	at net.razorvine.pickle.Unpickler.dispatch(Unpickler.java:234)
	at org.jpmml.sklearn.PickleUtil$1.dispatch(PickleUtil.java:77)
	at net.razorvine.pickle.Unpickler.load(Unpickler.java:123)
	at org.jpmml.sklearn.PickleUtil.unpickle(PickleUtil.java:98)
	at org.jpmml.sklearn.Main.run(Main.java:104)
	at org.jpmml.sklearn.Main.main(Main.java:94)
Caused by: java.lang.NoSuchMethodException: net.razorvine.pickle.objects.ClassDict.__setstate__([Ljava.lang.Object;)
	at java.lang.Class.getMethod(Class.java:1786)
	at net.razorvine.pickle.Unpickler.load_build(Unpickler.java:406)
	... 6 more

---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
<ipython-input-5-f6552a17c384> in <module>
     26
     27 from sklearn2pmml import sklearn2pmml
---> 28 sklearn2pmml(pipeline, "coxph.pmml", with_repr = True)

~/miniconda3.7/lib/python3.7/site-packages/sklearn2pmml/__init__.py in sklearn2pmml(pipeline, pmml, user_classpath, with_repr, debug, java_encoding)
    250                                 print("Standard error is empty")
    251                 if retcode:
--> 252                         raise RuntimeError("The JPMML-SkLearn conversion application has failed. The Java executable should have printed more information about the failure into its standard output and/or standard error streams")
    253         finally:
    254                 if debug:

RuntimeError: The JPMML-SkLearn conversion application has failed. The Java executable should have printed more information about the failure into its standard output and/or standard error streams

I think I know why it happens.
The library is probably (at some point) pickling the model, and sending it to a JavaRuntimeEnvironment and unpickling it there. Problem is that the class created from sklearn_adapter(CoxPHFitter, event_col='E') doesn't really exist anywhere, its just a temporary class created in https://github.com/CamDavidsonPilon/lifelines/blob/master/lifelines/utils/sklearn_adapter.py#L123
So, as long as the sklearn-adapter method is going to create dynamic classes - this won't work.
@CamDavidsonPilon I'm curious to understand why dynamic classes was chosen for this ? And thoughts on moving it to static classes - while being more verbose, it would be a bit more robust I think.

And thoughts on moving it to static classes

I'm open to this. As you know, current support for sklearn is limited because of these dynamic classes. Can you describe what this might look like?

Could you point me to docs/info on what the sklearn adapter is intended for?
That way we may be able to identify a better architecture. I do have some thoughts, but wanted to check if they solve other needs.
Note: With regard to PMML I think even if they are made static, some more things may be needed. I'll continue my exploration

It's intention is to create an API that a) resembles and b) is compatible with scikit-learn. That is, have lifelines classes that behave like, for example, sklearn.linear_model.LinearRegression. That way, these classes can plug into tools like GridSearchCV to find the best group of parameters for a model.
Known limitations are the ones you've bumped into: serialization: i) because autograd/jax creates anonymous functions, they don't work will with most serialization libraries, ii) creating dynamic classes almost always falls with serialization libraries too.
Docs are here: https://lifelines.readthedocs.io/en/latest/Compatibility with scikit-learn.html
And unit tests are useful: https://github.com/CamDavidsonPilon/lifelines/blob/master/tests/utils/test_utils.py#L917 (and note that xfail tests too)

@CamDavidsonPilon Now that the pickling issues are resolved, I wanted to take a look at PMML too.
I realized that the error I'm seeing here may not be related to pickling/joblib issues it is probably that the java library jpmml-sklearn doesn't have support for lifelines pickles.
Had created https://github.com/jpmml/jpmml-sklearn/issues/118 for this.
I was wondering if you're aware of any library (in python/R/java etc.) which supports PMMLs for similar estimators to the ones in lifelines.
The part that gets me a bit confused is that lifelines returns a prob value for every time T in the timeline - while most PMMLs I see for Regression etc. have a single score value and not an array of scores.
So, I wanted to see the implementation that some other libraries may have.

The part that gets me a bit confused is that lifelines returns a prob value for every time T in the timeline - while most PMMLs I see for Regression etc. have a single score value and not an array of scores.

This is true if you are predicting the survival function. Choosing predict_median or predict_percentile returns a single value per observation.


Unable to compile examples on Linux

I wanted to get acquainted with heaps even though the author doesn't consider it stable enough yet.
I installed heaps with haxelib git heaps https://github.com/ncannasse/heaps.git, hxsdl with haxelib git hxsdl https://github.com/ncannasse/hxsdl.git and other dependencies were available without having to use git.
I then cloned this repo and made obvious modifications to the hxsdl in my path to get it compiling on linux (e.g., SDL.h into SDL/SDL.h). Then going into samples/2d and doing haxe 2d_ogl.hxml it craps out on:
./src/sdl/GL.cpp:63:24: error: ‘GLLoadAPI’ was not declared in this scope
Error: ./src/sdl/GL.cpp: In static member function ‘static bool sdl::GL_obj::init()’:
./src/sdl/GL.cpp:63:24: error: ‘GLLoadAPI’ was not declared in this scope
  bool tmp = (GLLoadAPI() == (int)0);  HX_STACK_VAR(tmp,"tmp");
                        ^
In file included from include/sdl/GL.h:9:0,
                 from ./src/sdl/GL.cpp:4:

It looks like that's in an #ifdef _WIN32 block which makes me think something's spuriously setting that even though I'm not in Windows. If I delete that whole block or set to #if 0, I get:
Error: ./src/sdl/Sdl.cpp: In static member function ‘static Void sdl::Sdl_obj::init()’:
./src/sdl/Sdl.cpp:53:19: error: ‘timeBeginPeriod’ was not declared in this scope
  timeBeginPeriod(1);
                   ^
./src/sdl/Sdl.cpp: In static member function ‘static Void sdl::Sdl_obj::quit()’:
./src/sdl/Sdl.cpp:204:17: error: ‘timeEndPeriod’ was not declared in this scope
  timeEndPeriod(1);

Strangely, if I modify that file, it gets changed back, preventing me from going further, so I can't even do dirty hacks to get it working until I understand how this works better. Is the focus just on Windows and flash for now? Should I wait a couple of months before playing with this?

Sdl.cpp is generated from hxsdl/sdl/Sdl.hx.
Does adding the following in the @:cppFileCode data at the top of the class help ?
#ifndef _WIN32
#	define timeBeginPeriod(_)
#end

But you'll still need a way to resolve the GL API (the block you have #if 0), unless your Linux headers already have support for OGL3+

FYI I have dropped hxsdl support in favor of Heaps/HL and Heaps/Lime (contributed by @pperidont)


altair-express-middleware - options are not used

I used altair-express-middleware like that:
server.use('/altair', altairExpress({
 endpointURL: '/graphql',
 subscriptionsEndpoint: `ws://localhost:4000/subscriptions`
}));

(taken from the README.md)
But the index.js in altair-express-middleware does not do anything with the opts parameter currently.

Hey @thunder7553, the options are not used at the moment. Currently it just creates the app instance on the route. It is still an MVP and depending on the usage, I was going to enable the configuration options.

@thunder7553 See #226 for context.  @imolorhe put altair-express-middleware in response to this issue (I'll add that he was extremely responsive).  Is there a repo for the middleware?  Perhaps one of us can create a PR.

@jrop It's the same repo. It's a monorepo. You're welcome to create a PR.

@thunder7553

@thunder7553 Options are now used.

Added in<EMAIL_ADDRESS>


Add new Leaderboard feature.

Built an insolated LeaderBoard Component which would take single or multiplayer usernames and scores by the props and update it to the leaderboard.

The commit message should be feat: Add Leaderboard rather than feat: Leaderboard.
It should look like a command.


amd64 binaries appear on downloads page

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


New strains to add to pathogen strain list

For https://github.com/PHI-base/curation/issues/180
Please add strain 'H11-20' for pathogen Aspergillus fumigatus.

@CuzickA H11-20 should be available in PHI-Canto now.

For https://github.com/PHI-base/curation/issues/135
Please add strain 'ND158' for pathogen Aspergillus fumigatus.

ND158 added.

For https://github.com/PHI-base/curation/issues/192
Please add strain 'JK19' for pathogen Sclerotinia sclerotiorum

JK19 added.

For https://github.com/PHI-base/curation/issues/199
Please add strain 'KNH-6-95' for pathogen Venturia inaequalis.

KNH-6-95 added.

For https://github.com/PHI-base/curation/issues/193
Please add strain 'DK05' for pathogen Ramularia collo-cygni.

For https://github.com/PHI-base/curation/issues/184
Please add strains 'F55064', 'W73355' and 'H27023' for pathogen Aspergillus fumigatus.

I've added all the above strains: DK05, SD1, F55064, W73355, and H27023.


Sourcery stopped working, cannot click Enable button in side bar

I was using Sourcery in VS Code a few days ago and it was working perfectly.  Today I opened VS Code and noticed it wasn't working any more as it is not even giving suggestions in the "welcome-to-sourcery" file. If I try to click on the blue Enable button in the side bar it doesn't do anything or provide any message/feedback. Tabbing to the button and pressing the Enter key is the same result.

OS: Windows 10 Version 22H2 (OS Build 19045.3324)
Sourcery version 1.7.0
VS Code version 1.81.1
Sourcery Log file:
sourcery.log
Steps I have tried:

Restarting VS Code
Uninstall and reinstall Sourcery extension
Searching and deleting sourcery.yaml / auth.yaml files on my system and logging into Sourcery in VS Code via browser again

This is now released and should be fixed


Fix : remove query count limit

Remove the limit on query count. By default the limit is 100. If you
try to get the count of a collection and the collection has more than
100 rows, the result is always 100.

Nice fix!

@flessard @drew-gross so theres no limit now?


Remove payout route requirement

Right now we require a working payout route in order to move money (ostensibly; in practice we still move money if a participant has a failing payout route). We did this because we wanted to reduce or even eliminate our escrow (#1383), but it's now clear that that is both unnecessary and undesirable. We should allow projects to pool funds in Gratipay, to give us more flexibility to grow our escrow in step with volume (which we need to do in order to be able to keep funds transfer snappy), and also to give projects more flexibility (e.g., https://github.com/gratipay/gratipay.com/issues/4339).

Along with this should be some rejiggering of the payday process so cards aren't charged if there is money to use.


TextInput ref gets re-set with every render

Description
starting in 0.62 (probably related with the hooks-rewrite of textinput), every time textinput re-renders, the ref callback function is executed. That means that for every character you write, ref callback is called with

null and then
new ref

the refs are not referentially equal, even though the _nativeTag field on the ref is the same, and the TextInput is not unmounted in between the renders.
React Native version:
This is reproducible on current master
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.
use code here https://github.com/facebook/react-native/blob/5cde6c5e7d440e62d21ef858d77ed185df8a2da0/RNTester/js/examples/TextInput/TextInputSharedExamples.js#L162 - just put a console log in there
Expected Results
I expect the ref callback to be called when the component mounts / unmounts only, not on every re-render

I'm also seeing this

Same problem.
Here my environment information
System:
    OS: macOS 10.15.5
    CPU: (4) x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
    Memory: 228.73 MB / 8.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 10.20.1 - ~/.nvm/versions/node/v10.20.1/bin/node
    Yarn: 1.22.4 - ~/.nvm/versions/node/v10.20.1/bin/yarn
    npm: 6.14.5 - ~/.nvm/versions/node/v10.20.1/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.4, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
    Android SDK:
      API Levels: 22, 23, 25, 26, 27, 28, 29
      Build Tools: 23.0.1, 25.0.2, 26.0.2, 26.0.3, 27.0.0, 27.0.3, 28.0.2, 28.0.3, 29.0.0, 29.0.2, 29.0.3
      System Images: android-27 | Google APIs Intel x86 Atom, android-27 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: 4.0 AI-193.69<IP_ADDRESS>14223
    Xcode: 11.4/11E146 - /usr/bin/xcodebuild
  npmPackages:
    react: ^16.13.1 => 16.13.1
    react-native: ^0.62.2 => 0.62.2

I can confirm this is happening for us as well.

@vonovak The code you provided to reproduce the error actually is expected to cause the callback to be run on every re-render (see callback ref caveats).
But still, I came across the same error even when the callback ref is not re-created on every render.
You can look at the result here.
The ref callback is only called on every re-render (4 times) when the code is running on mobile (I have only tested iOS).
There is no issue when using React Native Web, or React DOM for that matter.
And I did not find that the values provided are referentially different when using the same callback ref. I used a Set to keep track of how many distinct object references were passed.

I believe (but don’t know) this is still a problem


fmt,sync/atomic: fmt functions should allow integer verbs for atomic integers

What version of Go are you using (go version)?
go 1.19
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
https://go.dev/play/p/jHX7pcphAmV
What did you do?
Format an atomic.Int32 using the %d verb.
What did you expect to see?
Since an atomic.Int32 “is an atomic int32”, I expected that the fmt verbs that apply for int32 should generate the same representation when applied to an atomic.Int32.
Or, if the %d verb is not expected to work in the same way that it does for an int32, I expect the atomic package docs to note that detail.
What did you see instead?
Applying the %d verb to an atomic.Int32 prints it as a struct, not an integer:
{{} 42}

(CC @robpike @martisch)

That's odd, I feel like it was at some point because I've definitely had the noCopy check catch something in the runtime tests before, though perhaps I'm just misremembering.

Apparently not:
https://cs.opensource.google/go/go/+/master:src/cmd/go/internal/test/test.go;l=601-627;drc=8a3d167f5b4b482a230cd9800df9969af60561af

Slightly related to https://github.com/golang/go/issues/54582 I think.


Fix priorityclass missing in loadtest

This PR fix https://github.com/CentaurusInfra/arktos/issues/783
The root cause of this bug is that, priorityclass, a cluster-scope obejct, was created with namespace/tenant set in the dynamic client. The code change fixed it by setting the namespace/tenant correctly according to the scope of the resource.
Verification:

Scale-up 100-node load test finished without daemonset-priorityclass-0 creation error.
Scale-out 100-node load test finished without daemonset-priorityclass-0 creation error.
Scale-up 100-node density test succeeded
Scale-out 100-node density test succeeded

/approve


Freezing emoji

Three is no emojticon to indicate feeling Cold, Shiver or Freezing, but a Snowflake emoji ❄.
Petitions

Emoji Request

The candidate emojis for Unicode 11 include a Freezing Face emoji as U+1F976.
http://www.unicode.org/L2/L2017/17284-emoji-recs.pdf


Why limiting RPC to 1?

Hello,
according to what I see on:
https://github.com/kalmyk/fox-wamp/blob/1997ab27c55517a4168fa512a2e507a199fcc57d/lib/realm.js#L276
you are limiting to one single call per RPC. Any reason to do that? the WAMP protocol does not limit that explicitly, so you are limiting functionality. In any case, should be unlimited by default and with the posibility to limit it by configuration parameters.

Simultaneous tasks limit for wamp session made unlimited


Fix lost FQN in TypeDeclaration::references() and UseStatments::fullyQualifiedNameFor()

This PR addresses https://github.com/MontealegreLuis/phuml/issues/18

Thank you @Amegatron!!
Could you please pin PHP Parser dependency in composer.json (line 21) to 4.11.0. It looks like 4.13 uses incompatible type declarations
"nikic/php-parser": "4.11.0"

Done 👍  Also, found some other bugs in parsing and will make PRs once I figure them out.

Damn, forgot to start new branch for other issues. Should I recreate the PR, or it is OK to merge them all-together? There is still one more bug I wanna fix today concerning that same "references": compound types like array<type1,type2> are not understood well by parser and they go as separate "references".

I'm okay with fixing #19  and #20 in this PR. Could you please also open an issue for array<type1, type2>.
I'm not sure how standard is that notation and how it should look in the diagram.
Could you please elaborate a little bit more in a new issue?
Also not sure if Git hooks are working. I thought they would validate commit format (I've been following the  conventional commits specification)
It's not a big deal, but was wondering if I need to fix the Git hooks set up.

Thank you @Amegatron!

As for array<...> notation, it is pretty common in PhpDoc if you want to specify the contents of an array. For example, array<int> means that this is array of only ints. array<string,MyClass> essentially means a hash-map: keys are strings, and elements themselves are objects of MyClass. For now I'm not yet sure how exactly does it look in "out-of-the-box" diagrams, but since I use this library mostly as an tool to generate my own formats of diagrams, I get those array<...> when I try to get TypeDeclaration::rerefences(). I'll try to check it out today.
Also, this notation is used not only in arrays, but for any kinds of "lists". For example, if you're using any Collection class (implementing ArrayAccess or related), you can also specifcy the types for keys and values in the same format, like Collection<MyClass>. Concerning this last scenario, I also sometimes get strange references, though not yet sure how to reproduce it: sometimes I get such references as Collection<, or MyType> - with those junky brackets added to the type itself. Still need to investigate this issue more.

As for the git hooks, I believe all the tests have passed automatically, but it told me approval was still needed for new contributors, that's why it didn't merge itself. If that was the question :)

You're welcome!
As for array<...> notation, it is pretty common in PhpDoc if you want to specify the contents of an array. For example, array<int> means that this is array of only ints. array<string,MyClass> essentially means a hash-map: keys are strings, and elements themselves are objects of MyClass. For now I'm not yet sure how exactly does it look in "out-of-the-box" diagrams, but since I use this library mostly as an tool to generate my own formats of diagrams, I get those array<...> when I try to get TypeDeclaration::rerefences(). I'll try to check it out today.
Also, this notation is used not only in arrays, but for any kinds of "lists". For example, if you're using any Collection class (implementing ArrayAccess or related), you can also specifcy the types for keys and values in the same format, like Collection<MyClass>. Concerning this last scenario, I also sometimes get strange references, though not yet sure how to reproduce it: sometimes I get such references as Collection<, or MyType> - with those junky brackets added to the type itself. Still need to investigate this issue more.
P.S. it is also worth noting, that this notation allows lots of nested levels, for example array<int,array<string,MyClass>>, etc, which means literally this: array with number-keys, elements of which are arrays with string-keys (hash-map) and values are objects of MyClass :)

I see. Thank you for the explanation. I think this would be a feature request to support "generics" in doc blocks.

As for the git hooks, I believe all the tests have passed automatically, but it told me approval was still needed for new contributors, that's why it didn't merge itself. If that was the question :)

When you committed a few tasks should have been run automatically, and if you ran your commit from a terminal, you should have seen an output similar to the one below.
GrumPHP detected a pre-commit command.
GrumPHP is sniffing your code!
Running tasks with priority 0!
==============================
Running task 1/6: composer...
Running task 2/6: composer_normalize...
Running task 3/6: phpstan...
Running task 4/6: phpunit...
Running task 5/6: phpcsfixer2...
Running task 6/6: phpcsfixer2_test...
Running task 1/6: composer... ✔
Running task 2/6: composer_normalize... ✔
Running task 3/6: phpstan...
Running task 4/6: phpunit...
Running task 5/6: phpcsfixer2...
Running task 6/6: phpcsfixer2_test...
GrumPHP detected a commit-msg command.
GrumPHP is sniffing your code!
Running tasks with priority 0!
==============================
Running task 1/1: git_commit_message...


Map does not show in Android on ObsDetail page

Describe the bug
On android, it appears the map does not show in the ObsDetail page. This was brought up in PR #385
To Reproduce

Select an observation
Click on "Data" tab
See that the map shows on ios but not android

Expected behavior
The map should show
Screenshots

Context (please complete the following information):

Device: Android Pixel 3
OS: [e.g. iOS8.1]
App version: [e.g. 1.0.0]

Additional context
Poking around, I think this line
should be
  if ( !initialLatitude || !initialLongitude ) {

I don't have a google maps api key, but when I make that change an error about not having an api key comes up. So I think it's going in the right direction.
However, I think there's a deeper issue in that the useUserLocation hook only gets the location for iOS, which is why latLng is null in android.

Not happening in v0.17.1+64


My game crashes when right clicking a bamboo chair and I cannot recover the world.

crash-2020-05-29_14.00.23-client.txt

Please use the latest version.


Add example for STM32F411 with HW & SW tasks communicating via MPSC channel

Hi, could you rebase on master to get CI to work again? Thanks!

Hi, could you rebase on master to get CI to work again? Thanks!

Of courtse.

@korken89, rebased!


more modifications

Here, again, you see how we can expose the new FormSpec in the chen data, in order to be more transparent. Apparently, we have all default sin this example, but it is useful to keep in mind, and to learn how to actually trigger different behavior.

Thanks, can merge and resolve the conflicts if that's okay with you?

Merged manually in https://github.com/lexibank/chenhmongmien/commit/7565c3daf9de52e1662a898bda26e048ae7e787f

Sorry @tresoldi saw your comments too late.

I am not sure what happened here, I had the original PR ready 24 days ago (you can tell by the way I am referring to FormSpecs as something new), the only comment I made today was about the progressbar.

I guess you did not press on SEND off of the review. never mind.


Grain "creation_timestamp"

From the NMOS Content Model Specification:

NMOS specifications constrain a Grain used for interchange /to have exactly two TimeStamps (Origin and Sync).

However, the queryapi-subscriptions-websocket.json schema additionally has "creation_timestamp".
Discuss.
;-)

I'd suggest that as the Content Model docs are a work in progress and due for review this can be safely ignored for now. Anything which results in inconsistencies with a released version of the content model should be addressed in a future version, but we'd probably be better doing a more thorough analysis of that when a released version is available.

OK. It would help if the schema didn't describe "origin_timestamp", "sync_timestamp" and "creation_timestamp" identically, or explained why they are identical in the case of websocket grains.


UAI animation behaviour :

https://github.com/user-attachments/assets/fb72ab2a-6da9-494e-bf38-ddbb7468f9d9

I tried multiple times to reproduce this issue, but it was not reproducible on my end.
https://github.com/user-attachments/assets/6f3352c9-1b05-41bf-aa9f-1bcb1a7839d6

Still facing the same issue on android one plus 9R.

Completed worked smoothly at my end


No paging controll in Linux version

Storage Explorer Version: 1.18.1
Build Number: 20210304.5
Platform/OS: arch linux
Architecture: x64
Regression From: not sure
Bug Description
no paging control for queue view
Steps to Reproduce
select queue with more messages than fits to page. try to go to next page. there is no paging control
Expected Experience
to be able to get to next page
Actual Experience
frustrating

Additional Context

@martynas-m sadly this is by design. The APIs available for queues only allows Storage Explorer to peek at the first 32 messages. If Storage Explorer tried to look at messages further in the queue, Storage Explorer would have to dequeue your messages. You can read more about the API here: https://docs.microsoft.com/en-us/rest/api/storageservices/peek-messages
We've asked the service to increase this limit before but that request has not yet been fulfilled. I'm going to close this as by design now.


AOA for Angola

Can you please add AOA for Angola

fixer.io gets it exchange rates from the ECB. You will have to contact them to add the currency you desire, or you can just use another api that provides the currency you want.


Binary Exponentiation

Binary Exponentiation

Binary Exponentiation - Competitive Programming Algorithms
Modular Exponentiation (Power in Modular Arithmetic) - GeeksforGeeks

Codecov Report

Merging #23 (70d6676) into main (bd6b5ea) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
+ Coverage   99.12%   99.16%   +0.04%
==========================================
  Files          59       60       +1
  Lines         342      360      +18
  Branches       22       19       -3
==========================================
+ Hits          339      357      +18
  Misses          3        3

Impacted Files
Coverage Δ

core/src/main/scala/algo/algebra/binexp.scala
100.00% <100.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 bd6b5ea...70d6676. Read the comment docs.