context deadline exceeded despite high timeout

I'm seeing an issue where scrapes of this exporter are failing with error context deadline exceeded.
I can't see how this is happening when:

My timeout is high.
The error occurs very soon after the scrape GET.

SQL Exporter logs
Mar 22 14:20:20 host sql_exporter[3668]: I0322 14:20:20.893037    3668 main.go:52] Starting SQL exporter (version=0.4, branch=master, revision=6dfbe982f1e42340e2a87e0d30599383c02f0bed) (go=go1.9.1, user=alin@wheat, date=2017 1128-15:26:05)
Mar 22 14:20:20 host sql_exporter[3668]: I0322 14:20:20.923174    3668 config.go:18] Loading configuration from /etc/sql_exporter/sql_exporter.yml
Mar 22 14:20:20 host sql_exporter[3668]: I0322 14:20:20.937702    3668 config.go:131] Loaded collector "foo-collector" from /etc/sql_exporter/collectors/foo-collector.collector.yml
Mar 22 14:20:20 host sql_exporter[3668]: I0322 14:20:20.937795    3668 collector.go:67] [, collector="foo-collector"] Non-zero min_interval (30s), using cached collector.
Mar 22 14:20:20 host sql_exporter[3668]: I0322 14:20:20.937849    3668 main.go:67] Listening on :9399
Mar 22 14:22:09 host sql_exporter[3668]: I0322 14:22:09.040451    3668 sql.go:88] Database handle successfully opened with driver postgres.
Mar 22 14:22:09 host sql_exporter[3668]: I0322 14:22:09.040451    3668 sql.go:88] Database handle successfully opened with driver postgres.
Mar 22 14:22:09 host sql_exporter[3668]: I0322 14:22:09.108490    3668 collector.go:124] [, collector="foo-collector"] Collecting fresh metrics: min_interval=30.000s cache_age=9223372036.855s
Mar 22 14:22:18 host sql_exporter[3668]: I0322 14:22:18.986887    3668 promhttp.go:38] Error gathering metrics: [from Gatherer #1] [, collector="foo-collector"] context deadline exceeded
Mar 22 14:22:52 host sql_exporter[3668]: I0322 14:22:52.166343    3668 promhttp.go:38] Error gathering metrics: 4 error(s) occurred:
Mar 22 14:22:52 host sql_exporter[3668]: * [from Gatherer #1] [, collector="foo-collector", query="query1"] context deadline exceeded
Mar 22 14:22:52 host sql_exporter[3668]: * [from Gatherer #1] [, collector="foo-collector", query="query2"] sql: Rows are closed
Mar 22 14:22:52 host sql_exporter[3668]: * [from Gatherer #1] [, collector="foo-collector", query="query2"] context deadline exceeded
Mar 22 14:22:52 host sql_exporter[3668]: * [from Gatherer #1] [, collector="foo-collector", query="query3"] context deadline exceeded

Postgres logs:
2019-03-22 14:22:18.779 GMT [3824] LOG:  duration: 9660.997 ms  execute 1: query4
2019-03-22 14:22:19.090 GMT [3825] LOG:  duration: 9971.390 ms  execute 1: query 1
2019-03-22 14:22:19.333 GMT [3826] LOG:  duration: 10206.832 ms  execute 1: query2
2019-03-22 14:22:52.032 GMT [3822] LOG:  duration: 42916.951 ms  execute 1: query3

SQL Exporter config:
global:
  scrape_timeout_offset: 3s
  min_interval: 30s
  max_connections: 16
  max_idle_connections: 16

Prometheus config:
  scrape_interval: 2m
  scrape_timeout: 115s

SQL exporter logs show that the database queries fail after 9 seconds. How can this happen when SQL exporter has 112 seconds to return in?

Ouch, that would appear to be my fault.
Initially SQL Exporter used a configuration-provided query timeout and when I found out about Prometheus' X-Prometheus-Scrape-Timeout-Seconds header (which informs targets about its scrape_timeout) I decided to use that instead (in order to allow Prometheus to control the timeout).
But while I did remove the field from the sample config, I still kept the old scrape_timeout configuration field around (in order to allow further limiting of the timeout) and (the cause of your issues) its default value of 10 seconds. So unless one specifies a global.scrape_timeout value in the configuration, the actual timeout SQL Exporter uses is:
min(prometheus_scrape_timeout - scrape_timeout_offset, 10s)

Hence the ~10s timeouts you're seeing.
The workaround is to explicitly provide a global.scrape_timeout in your sql_exporter.yml and set it to some very large value. Or at least larger than 120s, in your case.

Thank-you for the fast reply.
I can confirm setting global.scrape_timeout higher than Prometheus' scrape_timeout solves it.
 Perhaps having it default to unlimited, so Prometheus' scrape_timeout is the master would be more intuitive?

Perhaps having it default to unlimited, so Prometheus' scrape_timeout is the master would be more intuitive?

That's what I thought too, but it may not be the best idea ever. If something else than Prometheus queries SQL Exporter's /metrics page, it may hang forever, together with the DB connection. So I'm thinking to maybe just set it to some large value (5 minutes, as Prometheus doesn't deal well with scrape intervals over 5 minutes) and make sure to document it again (at least in the sample config).

Good point, that sounds reasonable

@free same issue and take the WA to resolve it. So could you add below config into default sql_exporter.yml in tarball? Thx.
scrape_timeout: 10s

10 seconds is actually the default hardcoded into SQL Exporter. There's no need to put it in the YAML config.


🐛 Comments are removed when updating from rome-ignore to biome-ignore

Environment information
This is far too long for me to review, and I do see things I shouldn't put in. I can provide a subset of information if needed.

I did get this error when running --apply-unsafe

Biome encountered an unexpected error

This is a bug in Biome, not an error in your code, and we would appreciate it if you could report it to https://github.com/biomejs/biome/issues/ along with the following information to help us fixing the issue:

Source Location: /Users/runner/work/biome/biome/crates/rome_rowan/src/ast/mod.rs:230:13
Thread Name: rome::worker_4
Message: Tried to cast node with kind JS_STATIC_MEMBER_EXPRESSION as `"rome_js_syntax::generated::nodes::AnyJsAssignmentPattern"` but was unable to cast

What happened?
Comments before // rome-ignore statements are removed.

Expected result
It should leave these comments in.
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

I just did a find and replace to update rome-ignore, but I still get the error above.

Could please share the code that generates that error please? We're trying to hunt down the rule/code that triggers that for days!
Also:

I believe we removed the code action of that rule in one of the latest patches
the error you see was generated by some rule that emitted invalid code

I believe it's this code in combination with complexity/useLiteralKeys

// Add optimizely service to global window; this allows us to debug feature flag flakiness in localhost a bit better
if (typeof window !== 'undefined' && Boolean(window)) {
  ((window as unknown) as Record<string, unknown>)['optimizelyService'] = optimizelyService;
}

const IS_RUNNING_IN_CYPRESS =
  typeof window === 'undefined' ? false : Boolean((window as never)?.['Cypress']);

happens on 1.2.2

I am going to close this.

the issue described here was closed in  https://github.com/biomejs/biome/pull/269
the code action rome-ignore to biome-ignore was removed in 1.2.1

@yckbilly1929 subscribe to https://github.com/biomejs/biome/issues/98, which is a broader issue that doesn't involve a specific rule, but a how code actions are applied to the code, which causes these issues.


prefer <IP_ADDRESS> over localhost

Standalone startup can sometimes hang on a mac if its trying to connect to ipv6.  Updating startup script to use <IP_ADDRESS> instead of localhost

LGTM. Travis issue looks unrelated.


Add User Lifetime Statistics

Added a UserStatistics model & relevant methods to track a user's lifetime statistics across all classes
Currently includes:

lifetime number of questions asked
lifetime number of questions answered
lifetime amount of time spent receiving help
lifetime amount of time spent helping
lifetime number of students helped

and I tested them too :)

Added a UserStatistics model & relevant methods to track a user's lifetime statistics across all classes
Currently includes:

lifetime number of questions asked
lifetime number of questions answered
lifetime amount of time spent receiving help
lifetime amount of time spent helping
lifetime number of students helped

and I tested them too :)

backend god @rachllee

Nevermind, false alarm. Just that 1 nit and linting then we are good!


missing option to extract data from paginated results

Currently I was unable to retrieve any information about how to handle cases where the provided data Does not fit in a single response. though there is the option to retrieve the "odata.nextLink", there is no option to request information from the nextLink besides calling the api manually. I imagine to solutions for that:
a) Including the paginated results automatically (possibly with a parameter, to prevent unnecessary requests when not needed)
b) An additional "next()" Method for the iterative request of paginated results.
I am sorry if there is already an existing solution which is not mentioned in the documentation, but I was unable to identify a regarding possibility within the code by myself.

This is now available in the pageterator work closed at https://github.com/microsoftgraph/msgraph-sdk-python-core/issues/126 with examples at https://github.com/microsoftgraph/msgraph-sdk-python-core/pull/479.


'generate plugin' and 'generate command' commands, from the plugin-dev package, fail with a TS error

Summary

Commands to generate the plugin and a command for the plugin fail at the compile step:
In this example, I overwrote the list command, which is why you see the questions about overwriting the files. But the compile error is the same for generating a new command or plugin.
➜  myplugin git:(master) ✗ sf dev generate command --name list
 ›   Warning: Plugin @salesforce/plugin-dev (2.4.2) differs from the version specified by sf (2.3.1)
Adding a command to myplugin!
Updating package.json
? Overwrite src/commands/list.ts? yes
Overwriting src/commands/list.ts
? Overwrite messages/list.md? yes
Overwriting messages/list.md
? Overwrite test/commands/list.nut.ts? yes
Overwriting test/commands/list.nut.ts
? Overwrite test/commands/list.test.ts? yes
Overwriting test/commands/list.test.ts
yarn run v1.22.19
warning ../../../../package.json: No license field
$ wireit
Analyzing
✅ Ran 0 scripts and skipped 1 in 0s.
Done in 0.31s.
yarn run v1.22.19
warning ../../../../package.json: No license field
warning From Yarn 1.0 onwards, scripts don't require "--" for options to be forwarded. In a future version, any explicit "--" will be forwarded as-is to the scripts.
$ wireit --fix
Analyzing
  0% [0 / 1] [1 running] lint
✅ Ran 1 script and skipped 0 in 2.5s.
Done in 2.76s.
yarn run v1.22.19
warning ../../../../package.json: No license field
$ wireit
Analyzing
  0% [0 / 1] [1 running] compile
../../../../node_modules/@types/phoenix/index.d.ts:73:31 - error TS2304: Cannot find name 'CloseEvent'.

73     onClose(callback: (event: CloseEvent) => void | Promise<void>): MessageRef;
                                 ~~~~~~~~~~

Found 1 error in ../../../../node_modules/@types/phoenix/index.d.ts:73

❌ [compile] exited with exit code 2.
❌ 1 script failed.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Steps To Reproduce
Install the plugin extension with sf plugins install @salesforce/plugin-dev, run the generate commands.
Expected result
Should have exited without issues.
Actual result
Command exits with issues.
Additional information

System Information
Using bash on macOS.
Salesforce Plugin version: @salesforce/plugin-dev (2.4.2)
CLI information: @salesforce/cli/2.44.8 darwin-arm64 node-v18.19.0
NPM: 10.8.3
{
  "architecture": "darwin-arm64",
  "cliVersion"<EMAIL_ADDRESS>  "nodeVersion": "node-v18.19.0",
  "osVersion": "Darwin 23.6.0",
  "rootPath": "/Users/renatosilva/.nvm/versions/node/v18.19.0/lib/node_modules/@salesforce/cli",
  "shell": "zsh",
  "pluginVersions": [
    "@oclif/plugin-autocomplete 3.0.18 (core)",
    "@oclif/plugin-commands 3.3.4 (core)",
    "@oclif/plugin-help 6.0.22 (core)",
    "@oclif/plugin-not-found 3.1.10 (core)",
    "@oclif/plugin-plugins 5.1.3 (core)",
    "@oclif/plugin-search 1.0.24 (core)",
    "@oclif/plugin-update 4.2.14 (core)",
    "@oclif/plugin-version 2.1.2 (core)",
    "@oclif/plugin-warn-if-update-available 3.0.19 (core)",
    "@oclif/plugin-which 3.1.10 (core)",
    "@salesforce/cli 2.44.8 (core)",
    "apex 3.1.13 (core)",
    "auth 3.6.17 (core)",
    "data 3.4.3 (core)",
    "deploy-retrieve 3.8.3 (core)",
    "dev 2.4.2 (user) published 9 days ago (Mon Aug 26 2024)",
    "info 3.3.3 (core)",
    "limits 3.3.9 (core)",
    "marketplace 1.2.8 (core)",
    "org 4.1.14 (core)",
    "packaging 1.22.0 (user) published 398 days ago (Thu Aug 03 2023) (latest is 2.8.2)",
    "schema 3.3.9 (core)",
    "settings 2.2.7 (core)",
    "signups 2.3.5 (user) published 103 days ago (Sat May 25 2024) (latest is 2.5.12)",
    "sobject 1.4.6 (core)",
    "source 3.3.18 (core)",
    "telemetry 3.3.11 (core)",
    "templates 56.2.9 (core)",
    "trust 3.7.3 (core)",
    "user 3.5.11 (core)",
    "sfdx-git-delta 5.42.1 (user) published 10 days ago (Mon Aug 26 2024) (latest is 5.43.0)"
  ]
}

Updated the CLI and the error still happens.
  "architecture": "darwin-arm64",
  "cliVersion"<EMAIL_ADDRESS>  "nodeVersion": "node-v18.19.0",
  "osVersion": "Darwin 23.6.0",
  "rootPath": "/Users/renatosilva/.nvm/versions/node/v18.19.0/lib/node_modules/@salesforce/cli",
  "shell": "zsh",
  "pluginVersions": [
    "@oclif/plugin-autocomplete 3.2.2 (core)",
    "@oclif/plugin-commands 4.0.13 (core)",
    "@oclif/plugin-help 6.2.10 (core)",
    "@oclif/plugin-not-found 3.2.18 (core)",
    "@oclif/plugin-plugins 5.4.6 (core)",
    "@oclif/plugin-search 1.2.7 (core)",
    "@oclif/plugin-update 4.5.5 (core)",
    "@oclif/plugin-version 2.2.11 (core)",
    "@oclif/plugin-warn-if-update-available 3.1.13 (core)",
    "@oclif/plugin-which 3.2.12 (core)",
    "@salesforce/cli 2.57.7 (core)",
    "apex 3.4.5 (core)",
    "api 1.2.1 (core)",
    "auth 3.6.51 (core)",
    "data 3.6.3 (core)",
    "deploy-retrieve 3.12.0 (core)",
    "dev 2.4.2 (user) published 9 days ago (Mon Aug 26 2024)",
    "info 3.4.2 (core)",
    "limits 3.3.27 (core)",
    "marketplace 1.2.24 (core)",
    "org 4.5.4 (core)",
    "packaging 1.22.0 (user) published 398 days ago (Thu Aug 03 2023) (latest is 2.8.2)",
    "schema 3.3.26 (core)",
    "settings 2.3.15 (core)",
    "signups 2.3.5 (user) published 103 days ago (Sat May 25 2024) (latest is 2.5.12)",
    "sobject 1.4.32 (core)",
    "source 3.5.16 (core)",
    "telemetry 3.6.10 (core)",
    "templates 56.3.14 (core)",
    "trust 3.7.25 (core)",
    "user 3.5.26 (core)",
    "sfdx-git-delta 5.42.1 (user) published 10 days ago (Mon Aug 26 2024) (latest is 5.43.0)"
  ]
}

This isn't reproducible in a new plugin I just generated and then generating a list command.  Looks like the TS compile is including more than just your plugin so I'd modify your tsconfig.json so it doesn't do that.  From the paths, it seems to be looking outside your plugin dir.

True, after resetting my local install of the global modules (including removing the node_modules in my home folder - which is where the error was pointing to - and the nvm one) the installation of the CLI, the dev plugin and yarn worked to make this error go away.


What is the right way to measure readability - where do we segment text?

If we turn : into a period, we might lower the readability scores. But is that more "correct"?
One thought: we might want to err on the side of having higher grade levels when we have a debatable choice, so people aren't mislead into thinking that the form is really readable but it isn't.

Early experiments suggest that this might not make a big difference either way

Another thought: we want to also make sure our readability scores are similar to the ones that people would obtain in other tools, like Microsoft Word.


CU-862kp1nrk_Create-Project-related-designs

CU-862kp1nrk - Create Project related designs

Task linked: CU-862kp1nrk Create Project related designs


Fix callstack support, add callstack support to allocator

This commit does the following:

Fixes callstack support by disabling the UB sanitizer in the Tracy build. This is taken from the Zig source and reflected in other issues like https://github.com/wolfpld/tracy/issues/602. Added as a C flag and not specifically through sanitize_c to ensure that we explicitly disable the UB sanitizer and nothing else.

Adds callstack support to the allocator so that allocations can be stack traced.

@neurocyte not too sure if you've had luck with callstack stuff or not but this is what I needed to do to fix callstack support with these bindings. I was using sanitize_c before but switched it to a flag before putting this up to ensure that no other sanitizers were disabled (the build option version looks opaque but I haven't dove into the build system to see what exactly it disables).
Just wanted to pass this along since I'm working on using these bindings with z2d's acceptance tests as well. Cheers!


Remove 'Save and preview' button

Because nobody actually uses it.
https://trello.com/c/BtidAaIX/271-remove-save-and-preview-button-and-add-a-dialog-if-previewing-when-the-current-draft-has-not-been-saved

:+1:


testing: format of datasource.yaml files

What problem are you facing?
In some example manifests, we need to inject some values e.g tenantID, accountID, objectID. This injection values usually corresponds to Data Sources of Terraform. Please see the following example:
tenantId: ${data.azurerm_client_config.current.tenant_id}
This convention is from the Terraform registry documentation. So we use a regular expression to extract the key of the tenantID value. For this example ,the key is: azurerm_client_config.current.tenant_id
And our source yaml file has the following format:
azurerm_client_config.current.tenant_id: ***
azurerm_client_config.current.object_id: ***

@muvaf suggested two formats for data source yaml in this PR: https://github.com/upbound/official-providers/pull/299#discussion_r910804037
data:
  azurerm_client_config:
    current:
      object_id: 847c7043-b0a5-4fdf-9285-ee084c94a206

azure:
  tenantId: b9925bc4-8383-4c37-b9d2-fa456d1bb1c7
  objectId: 847c7043-b0a5-4fdf-9285-ee084c94a206

How could Official Providers help solve your problem?
Let's discuss the possible formats and decide that a best one.

Because of switching the different data sources and also supporting the repo secrets, this issue was resolved. Closing.


standard_init_linux.go:175: exec user process caused "permission denied"

I can run other images successfully but when running the hello-world, it fails:
[root@vm ~]# docker run hello-world
standard_init_linux.go:175: exec user process caused "permission denied"
And my docker info is like below:
[root@vm ~]# docker info
Containers: 9
Running: 1
Paused: 0
Stopped: 8
Images: 3
Server Version: 1.12.0-rc5
Storage Driver: btrfs
Build Version: Btrfs v3.17
Library Version: 101
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: host bridge null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp selinux
Kernel Version: 4.1.12-37.6.1.el7uek.x86_64
Operating System: Oracle Linux Server 7.0
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 7.135 GiB
Name: slc00bpv.us.oracle.com
ID: QY4D:Q26Y:RHTS:QNUR:NEKC:LSBB:LPR7:ECKG:DRID:EQPJ:3MHG:Z5K6
Docker Root Dir: /scratch/docker
Debug Mode (client): false
Debug Mode (server): false
Http Proxy: http://www-proxy.us.oracle.com:80
Https Proxy: http://www-proxy.us.oracle.com:80
No Proxy: localhost,<IP_ADDRESS>/8,.us.oracle.com,.oraclecorp.com,/var/run/docker.sock
Registry: https://index.docker.io/v1/
WARNING: bridge-nf-call-ip6tables is disabled
Insecure Registries:
dte.docker.oraclecorp.com
<IP_ADDRESS>/8
And my vm info is
[root@vm ~]# uname -a
Linux vm 4.1.12-37.6.1.el7uek.x86_64 #2 SMP Thu Jul 28 10:59:49 PDT 2016 x86_64 x86_64 x86_64 GNU/Linux
Could you please help me on this?

Usually when I see that error it's AppArmor or SELinux or something like that blocking the executable from starting up.

Thanks @tianon.  Do you have any suggestion that I can do it accordingly to fix this problem?

I'm seeing the same issue with Arch Linux, and I'm not using AppArmor or SELinux.
$ docker info
Containers: 26
 Running: 0
 Paused: 0
 Stopped: 26
Images: 3
Server Version: 1.12.3
Storage Driver: btrfs
 Build Version: Btrfs v4.8.2
 Library Version: 101
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.8.4-1-ARCH
Operating System: Arch Linux
OSType: linux
Architecture: x86_64
CPUs: 20
Total Memory: 125.9 GiB
Name: op
ID: I7AS:7QQG:AOAF:V7WJ:YI6N:3BU6:VXFM:QGNT:SVDW:G62T:RASE:OIJO
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Insecure Registries:
 <IP_ADDRESS>/8

$ uname -a
Linux op 4.8.4-1-ARCH #1 SMP PREEMPT Sat Oct 22 18:26:57 CEST 2016 x86_64 GNU/Linux

I'm not sure if this is related to https://github.com/docker/docker/issues/26495.

Hi. I had the same problem. Running sudo docker run hello-world fixed my problem. Perhaps it could solve yours too.
The link below holds the solution.
https://askubuntu.com/questions/477551/how-can-i-use-docker-without-sudo
Hope I'm able to help :)

It seems that this was an old issue affecting docker, typically being SELinux related or mounting a filsesystem with the NOEXEC flag


ch.qos.logback/logback-core1.1.1

Type: Incomplete
Summary:
ch.qos.logback/logback-core1.1.1
Details:
Adding EPL-1.0 to the Declared License
(EPL-1.0 OR LGPL-2.1)
Resolution:
https://repo1.maven.org/maven2/ch/qos/logback/logback-core/1.1.1/logback-core-1.1.1.pom
Affected definitions:

logback-core 1.1.1

No License in Files https://clearlydefined.io/definitions/maven/mavencentral/ch.qos.logback/logback-core/1.1.1/1.1.1
pom license is EPLv1.0 and LGPLv2.1 - https://repo1.maven.org/maven2/ch/qos/logback/logback-core/1.1.1/logback-core-1.1.1.pom
jar file manifest License is EPLv1.0 and LGPLv2.1
Logback License is EPLv1.0 and LGPLv2.1 - http://logback.qos.ch/license.html
Curate to EPL-1.0 and LGPL-2.1-only

Agree on EPL-1.0 OR LGPL-2.1-only


Set HeadTitle APPEND, PREPEND, and SET is not working properly

If we write in layout :
echo $this->headTitle('ZF2 Skeleton Application')->setSeparator(' - ')->setAutoEscape(false);

In view :
 $this->headTitle('Other', 'APPEND');

Expected : ZF 2 Skeleton Application - Other
Actual ( currently) : Other - ZF 2 Skeleton Application
The APPEND, and SET is not working.

SET is not working in my controller i am doing this-
$this->getServiceLocator()->get('ViewHelperManager')->get('HeadTitle')->set('Create Test Case',"SET");
still its prepending the title to the old one,
I also changed this in file - Zend\View\Helper\HeadTitle line no 74 to 80, for all three cases i called the set but still its prepending :(
if ($setType == Placeholder\Container\AbstractContainer::SET) {
$this->set($title);
} elseif ($setType == Placeholder\Container\AbstractContainer::PREPEND) {
$this->set($title);
} else {
$this->set($title);
}

@vivex I managed it, I made a blog post about it https://samsonasik.wordpress.com/2013/07/14/zend-framework-2-reset-headtitle-position-from-view/


Purge generate-changelog.sh and deal with CHANGELOG.mds

deisrel is now the preferred tool for generating CHANGELOG-style diffs, and the release process involves adding those to GitHub release notes for a tag. This orphans both the _scripts/generate-changelog.sh script and the CHANGELOG.md files in numerous repositories.
It's confusing to have a dead script and an aborted CHANGELOG everywhere. The scripts should be removed and the CHANGELOG content should be...somehow dealt with.

See https://github.com/deis/workflow/issues/464

@helgi oops, missed that. I'll restrict this issue to the purging of the generator script then.


Fix regex to catch HTML tags

This PR fixes the CI for https://github.com/huggingface/course and in particular ./es/chapter5/5.mdx.
Related to this slack thread (internal) and this failing CI.
The failing doc had this form:
10K<n<100K

<Tip>

This is a tip.

</Tip>

in which the _re_lt_html regex detected <n<100K    \n<Tip> as a single HTML tag. The _re_lt_html is meant to detect the < characters that are not part of a HTML tag.
I fixed the regex for this use case + added a regression test for it. All previous unit tests are still passing so hopefully it doesn't break something in the wild. I also took the liberty to add verbose mode to explain a bit more what the regex is doing (too me a bit of time to remember :roll_eyes:).
cc @mishig25 @MKhalusova @xenova
(also related to https://github.com/huggingface/doc-builder/pull/373 and https://github.com/huggingface/doc-builder/pull/394 which introduced and modified this regex)

CI was failing on an un-related test:
tests/test_autodoc.py:283: AssertionError
=========================== short test summary info ============================
FAILED tests/test_autodoc.py::AutodocTester::test_document_object - AssertionError: '\n<d[221 chars]ters>[{"name": "*args", "val": ""}, {"name": "[462 chars]\n\n' != '\n<d[221 chars]ters>""</parameters></docstring>\n\nBase class[401 chars]\n\n'
Diff is 1304 characters long. Set self.maxDiff to None to see it.
========================= 1 failed, 83 passed in 4.21s =========================

It is due to this commit on transformers that added arguments to the ModelInfo object.
I fixed the expected result in https://github.com/huggingface/doc-builder/pull/398/commits/ec14a49dc9916ec8f0e26148783bc22d72908bc2.

Thank you for fixing this!


Add more examples to documentation

It would be awesome to have more ready to use recipes for common tasks, such as starting a server with scheduled tasks. E.g.

Create digdag.conf file with a secret:

digdag.secret-encryption-key = MDEyMzQ1Njc4OTAxMjM0NQ==
# example
echo -n '16_bytes_phrase!' | openssl base64
MTZfYnl0ZXNfcGhyYXNlIQ==

Start server

digdag server -o data -b <IP_ADDRESS> -c digdag.conf

Make sure the data folder is backed up
3. Create/update project:
digdag push --project workflows scheduled-transactions

Create secret for project's pg datbase:

secrets --project scheduled-transactions --set pg.password

Thank you for your feedback!
It seems that it is good receipts for beginners.
I'll a take a look it later.
You are welcome to contribute as well.
The default configuration path is ~/.config/digdag/config by the way.
You will find this path digdag usage for example.


Yarn.lock install

JIRA link
PAY-150323Yarnfixv2
Change description
Yarn.lock install update
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ x ] No

Closed


Dropout on Attention

Hi, I find you apply dropout to attention weight in ScaledDotProductAttention. It seems that is not in the original paper?

It is used in the tensorflow implementation.
Hope it helps.


doc(kmac): Update kmac_entropy test plan desc

This commit updates chip_sw_kmac_entropy test plan descrition. It was
outdated (wait_timer, operation mode etc)

I don't think changing testplan description and stage affect CI. Let me merge this after resolving conflicts for the sake of alleviating CI burden.


Feature request: Tick rate slider, "advance one step" button when tick rate slider is on 0

implemented with fc8e087


new function to apply LSF

This introduces a function that calculates the matrix which maps spectra to LSF-convolved spectra, assuming the LSF is a gaussian with width set by a constant R = lambda/Delta lambda.  It assumes that the wavelength grid is locally-linear and dense, or it will not generally be normalized correctly.  In the future, I plan to automate the process of synthesizing on a dense grid, convolving with the LSF, and downsampling automatic, since it involves a few steps.
Here's the Ca triplet (what the Gaia RVS uses), with a realistic-ish LSF imposed.

What exactly is a LSF-convolved spectra? Does this accounting for the fact that an instrument has finite resolution?

This doesn’t account for the pixels on the detector, but it account for the line spread from the instrument optics, which generally has this functional form to first (second?) order.
Dealing with the detector pixels is also important, but we aren’t doing this at the moment.


bug: vat/vit yit/yat etc don't work as intended

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[x] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.1
Operating system/version
Archlinux
Describe the bug
vat/vit  and other at/it combination  don't work like normally in neovim, if the tags are spread into multiple lines they seem to select every parent nodes and not just what's inside the tags
Steps To Reproduce
try the vat motion on a fresh lazyvim config
Expected Behavior
Normal it/at behavior
Repro
Here are two screen recording with the vat motion
With stock lazyvim :  https://github.com/LazyVim/LazyVim/assets/68331131/653a4849-4a4b-469c-a12a-a86ad7677d65
With stock neovim :  https://github.com/LazyVim/LazyVim/assets/68331131/20886e4c-6501-47e2-939b-71b52730baea

I use the following overrides in my mini.ai config, they have worked well for me in the past:
{
  custom_textobjects = {
    t = { "<(%w-)%f[^<%w][^<>]->.-</%1>", "^<.->%s*().-()%s*</[^/]->$" },
    T = { "<(%w-)%f[^<%w][^<>]->.-</%1>", "^<.->().*()</[^/]->$" },
  },
}

right, report to mini.ai if you think this is a bug

This is a known limitation of mini.ai, as the maintainer himself has also admitted (because mini.ai uses lua patterns and some false positives are expected to exist). He recommends the [answer])https://github.com/echasnovski/mini.nvim/issues/110#issuecomment-1212277863) in this thread. You could take a look and see if it works for you. The answer is for .html files, but you could customize it for other filetypes you work with as well I imagine.

LazyVim already has that treesitter mapping, so instead of vit, one could do vif I think

This is a known limitation of mini.ai, as the maintainer himself has also admitted (because mini.ai uses lua patterns and some false positives are expected to exist). He recommends the answer in this thread. You could take a look and see if it works for you. The answer is for .html files, but you could customize it for other filetypes you work with as well I imagine.

Thanks for the indications, I'm however just gonna disable the plugin since I don't use its functionalities anyway


Move the required field under the properties

Summary
Moving the required field beneath the properties
Details
There was confusion about a TS error around a required field that wasn't actually a parameter yet. We haven't figured out the right way to give specific error messaging for this kind of situation with our types, but it makes more sense to define the properties that are available before we say which are required.
Convo: https://slack-pde.slack.com/archives/C03BS52QLPJ/p1649958372816539
Relevant PR: https://github.com/slackapi/deno-reverse-string/pull/24

Changes seem fine, I'm not sure I follow what the advantage is to moving required below properties in all of these places. Certainly seems fine though. I can see how shifting it in our app code, like the reverse string app, is helpful in the clarity it provides though. Is the change here mostly just for consistency?

💯 just for consistency and we could have gotten away without these changes to the SDK


Update Web

Codecov Report
Patch coverage has no change and project coverage change: -0.13 :warning:

Comparison is base (d334885) 42.34% compared to head (fcdd50c) 42.21%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
- Coverage   42.34%   42.21%   -0.13%
==========================================
  Files          21       22       +1
  Lines         333      334       +1
==========================================
  Hits          141      141
- Misses        192      193       +1

Impacted Files
Coverage Δ

lib/model/models/create_card.dart
0.00% <ø> (ø)

lib/model/models/image.dart
0.00% <0.00%> (ø)

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

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


Crash on older Windows versions due to missing DPI scaling capabilities

i install osu!lazer and it crash
here is my log:
2020-12-05 11:51:21 [verbose]: ----------------------------------------------------------
2020-12-05 11:51:21 [verbose]: runtime Log for ShadowPC (LogLevel: Verbose)
2020-12-05 11:51:21 [verbose]: osu 2020.1204.0.0
2020-12-05 11:51:21 [verbose]: Running on Microsoft Windows NT 6.1.7601 Service Pack 1, 2 cores
2020-12-05 11:51:21 [verbose]: ----------------------------------------------------------
2020-12-05 11:51:21 [verbose]: Unhandled exception has been allowed with 0 more allowable exceptions .
2020-12-05 11:51:21 [error]: An unhandled error has occurred.
2020-12-05 11:51:21 [error]: System.DllNotFoundException: Unable to load DLL 'SHCore.dll' or one of its dependencies: The specified module could not be found. (0x8007007E)
2020-12-05 11:51:21 [error]: at osu.Framework.Platform.Windows.WindowsWindow.SetProcessDpiAwareness(ProcessDpiAwareness awareness)
2020-12-05 11:51:21 [error]: at osu.Framework.Platform.Windows.WindowsWindow..ctor()
2020-12-05 11:51:21 [error]: at osu.Framework.Platform.Windows.WindowsGameHost.CreateWindow()
2020-12-05 11:51:21 [error]: at osu.Framework.Platform.GameHost.Run(Game game)
2020-12-05 11:51:21 [error]: at osu.Desktop.Program.Main(String[] args)
PLZ HELP

how can i fix it?

You can't, this is something for us to deal with. Please roll back to an older release for now.

what version is good?

2020.1128.0 should run.

THX

Please don't close issues that need fixing on our side if they haven't actually been fixed. This will be closed once we've resolved the underlying issue (which should be next release).


Add weighted sampling in InterleaveDataset

Update InterleaveDataset to support weighted sampling of training datasets, configured through the yaml config file, e.g.,
stop_strategy: "last_exhausted"
data_sets: []
data_dicts:
  - path: "fixie-ai/librispeech_asr"
    name: "clean"
    splits:
      - "train.100"
      - "train.360"
    user_template: "Continue the following text using less than 50 words:\n\n<|audio|>"
    assistant_template: "{{ continuation }}"
    transcript_template: "{{ text }}"
    weight: 2
    num_samples: 100_000
  - path: "fixie-ai/librispeech_asr"
    name: "other"
    splits:
      - "train.500"
    user_template: "Continue the following text using less than 50 words:\n\n<|audio|>"
    assistant_template: "{{ continuation }}"
    transcript_template: "{{ text }}"
    weight: 1
    num_samples: 100_000

Three stop strategies are available: last_exhausted (default), first_exhausted, and never_stop. The first two strategies are similar to the logic used in the Hugging Face dataset, while the third option allows the maximum number of training steps to be determined by max_steps.
The training datasets are sampled based on their weights (default: 1.0) specified in the configuration file. In the example above, samples from the clean/other subsets are sampled at a 2:1 ratio.

@juberti The following test unit failures need your attention.
======================================================================================================== short test summary info =========================================================================================================
ERROR inference/infer_test.py::test_infer_16kHz - json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
ERROR inference/infer_test.py::test_infer_48kHz - json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
ERROR inference/infer_test.py::test_infer_16kHz_stream - json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
ERROR inference/infer_test.py::test_infer_text_only - json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

@juberti The following test unit failures need your attention.
======================================================================================================== short test summary info =========================================================================================================
ERROR inference/infer_test.py::test_infer_16kHz - json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
ERROR inference/infer_test.py::test_infer_48kHz - json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
ERROR inference/infer_test.py::test_infer_16kHz_stream - json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
ERROR inference/infer_test.py::test_infer_text_only - json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Never mind, it seems to be a version issue with my local copy. All checks passed on github.

@juberti @farzadab Please check if all concerns have been addressed.


Add support for parallel tests

Context
For faster test runs, we want to be able to run our test suite in parallel using multiple cores.

Changes proposed in this pull request

Add the parallel_tests gem
Update config for Postgres, Capybara, and Redis
Document how to run the test suite in parallel locally

Guidance to review

Per-commit gives a good breakdown of the changes
Does the doc make sense?
Run it locally! Are you seeing any issues?

Link to Trello card
https://trello.com/c/ySrLogAF

Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

@davidgisbey @duncanjbrown Excellent, thank you both for testing.

I only have 8 cores btw so if someone has 16 it might be worth them following that section of the doc.

Had to do this locally, can confirm the fix works nicely.

I liked having the same output in and out of parallel_tests so I copied the --format line in .rspec to .rspec_parallel — would you consider adopting that?

I haven't quite figured out how to collate test failures from each process and print them all at the end. With progress formatting, there's much less scrolling back to identify what's failed. Happy to change over to documentation if we can find a nicer way to output errors.

@davidgisbey @duncanjbrown Excellent, thank you both for testing.

I only have 8 cores btw so if someone has 16 it might be worth them following that section of the doc.

Had to do this locally, can confirm the fix works nicely.

I liked having the same output in and out of parallel_tests so I copied the --format line in .rspec to .rspec_parallel — would you consider adopting that?

I haven't quite figured out how to collate test failures from each process and print them all at the end. With progress formatting, there's much less scrolling back to identify what's failed. Happy to change over to documentation if we can find a nicer way to output errors.


Email validation message

Add more specific validity message for what is required as a valid email address in signup.js

Resolved in commit f593fdee040f4c8de81fd624c413d03ba7730e2d


Noise threshold for 0D observers?

We currently have 2D frame samplers for setting the noise threshold and controlling the accuracy level for 2D pipelines. Unfortunately there is no equivalent yet for the 0D observers.

This was intentional as a 0D pipeline error can be tricially checked, unlike a frame. A typical evaluation loop would simply be:
while pipeline.error > threshold:
observer.observe(samples=1000)


Unicode characters inserted incorrectly

To Reproduce
Steps to reproduce the behavior:

Edit a text field or use the cell replace transformation, insert an Unicode character, like this smiley 😀.
Notice that this Unicode character is displayed as an invalid/placeholder character instead (). As the manually inserted Unicode character still looked fine (when replacing or before saving), the font, browser or OS is not the reason for the Unicode characters becoming invalid ones.

Current Results
Invalid/placeholder character () when inserting an Unicode character.
Expected Behavior
Proper editing and retrieving of Unicode characters.
Versions

Operating System: Windows 10 x64
Browser Version: Chrome
JRE or JDK Version: Windows distribution
OpenRefine: Version 3.4.1 [437dc4d]

@wetneb Hi, I tried to reproduce this issue in OpenRefine 3.5-beta [4aef758] and it seems correct. Maybe someone has fixed it.
SE_SUSTech

Hi @Warpeas, thanks very much for investigating this! Did you try with Chrome on Windows specifically?

Hi, @wetneb Yes, I'm using Chrome version 90.0.4430.85 and Windows 10.

Thanks, let's close this as not reproducible then.

Well, I will download the latest release and retry again then...

@strarsis happy to reopen if you find out that this persists of course!

Hi, @wetneb This issue can be reproduced in the latest release v3.4.1. And seems to be fixed before the 3.5-beta [4aef758]. So @strarsis maybe you can try to use the beta version or wait until the v3.5 release?


copy multiple file path with file nesting

https://user-images.githubusercontent.com/41773861/186768652-41968665-18eb-4d9c-9142-7a03bd7feced.mov
Version: 1.71.0-insider (Universal)
Commit: 9529e11f6481ae53bba821b05e34549491b9415e
Date: 2022-08-25T13:02:01.718Z
Electron: 19.0.12
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 21.6.0
Sandboxed: Yes

This issue may be same with https://github.com/microsoft/vscode/issues/155766. When run the command on file which using file nesting, passed multiple file paths.


⚠ Jenkins Plugins Artifactory has degraded performance

In ec4ecef, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1561 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 98f13a0 after 12 minutes.


Swift 2.3 support

Hello is there any support for Swift 2.3?

Looks like older versions would support it, not sure where the cut-out is though.
But this one could work:
https://github.com/teodorpatras/EasyTipView/tree/0.1.4

ok, nevermind :P


Android 12 issue

The android dependency needs to be updated, see https://github.com/facebook/facebook-android-sdk/issues/997

@Willham12 more than that, the whole library needs a forward port to the new libraries. It does need to be updated, do you have the time to do so? We'd love a PR #55


VS Code extension icon assets type and encoding

For now, vs code extension's icon use the image/png type and gzip encoding(like this python extension icon).
The gzip encoding is not useful for png format(and basically every bitmap format), icon size even bigger after gziped which only increase useless cpu load.
And the png format is a little outdated, how about using webp format (same quality but smaller)?  or using svg format since most extension icon could be vector graphics(much smaller and effective for gzip encoding)?

Correct. I think there are no plans to change this on the Marketplace side.
fyi @prashantvc


MultiTouchScreen.executeAction() methods should be public

While trying out the multitouch test examples (at http://selendroid.io/gestures.html), I keep getting the following error back from the java compiler:
Error:(159, 37) java: executeAction(io.selendroid.client.MultiTouchAction) has protected access in io.selendroid.client.MultiTouchScreen
This suggests that either the executeAction() methods should be made public, or that the examples need to be fixed.

@lukeis
@EvilOverlordX
I was looking into this issue. And the example in the documentation is incorrect. You need to use TouchActions's perform() method. Looking at the other examples this is the correct way to execute a TouchAction.
TouchAction ta = new TouchActionBuilder().pointerDown(0, 0).pointerMove(x, y).pointerUp().build();
ta.perform(driver);
I will update the documentation

@lukeis How do i update the documentation at http://selendroid.io/gestures.html

Ok.

@Blackbaud-KyleMartinez update this file:
https://github.com/selendroid/selendroid.io/blob/master/gestures.html.haml
and send a PR to the "selendroid.io" repo


"Start from Helm Chart" new project option

I want to start a new project from a Helm Chart in a Helm repository - this option would prompt me for

the Helm Chart (not sure how we can reuse the Helm Browser)
to which folder it should be extracted locally

Alternatively we (only?) have a "Create local project" option in the Helm Chart Browser when a Chart has been selected - which would only prompt for the local folder?

@olensmar so the user should be able to , sequentially: 1) browse for a helm repo 2) select a chart from that repo? Or do we provide a default repo?

thanks @julioramosest  - LGTM!

as discussed - I think having a "Browse Helm Charts" option in the left menu (taking you to the Helm Panel) is a good start - before we add a dedicated panel.


Using https and username/password fails

When creating an action and uploading to a private repo I get a warning:
Warning: Unexpected input(s) 'repository_username', 'repository_password', valid inputs are ['entryPoint', 'args', 'python_version', 'poetry_version', 'pypi_token', 'build_format', 'repository_name', 'repository_url', 'ignore_dev_requirements', 'allow_poetry_pre_release']
and an error:
ValueError

  There is no pypi-token.REPONAME setting.

  at /root/.pyenv/versions/3.9.6/lib/python3.9/site-packages/poetry/console/commands/config.py:153 in handle
      149│                 self.line(str(value))
      150│             else:
      151│                 values = self.unique_config_values
      152│                 if setting_key not in values:
    → 153│                     raise ValueError("There is no {} setting.".format(setting_key))
      154│
      155│                 value = config.get(setting_key)
      156│
      157│                 if not isinstance(value, str):

The action is defined as:
name: Python package
on:
  push:
jobs:
    build:
      runs-on: ubuntu-20.04
      steps:
         - uses: actions/checkout@v2
         - name: Build and publish to pypi
           uses<EMAIL_ADDRESS>           with:
             python_version: "3.9"
             build_format: "sdist"
             allow_poetry_pre_release: "yes"
             ignore_dev_requirements: "yes"
             repository_name: "REPONAME"
             repository_url: "REPOURL"
             repository_username: ${{ secrets.USER }}
             repository_password: ${{ secrets.PW }}

My best guess is that it has something to do with the if-statement in the .sh, that controls if a token should be used or the user/pw combo.
https://github.com/JRubics/poetry-publish/blob/01628671a54aebc4906c610823ed11cfd628a305/entrypoint.sh#L39

Please try to use<EMAIL_ADDRESS>instead of version 1.8 and check if this still happens :)

Please try to use<EMAIL_ADDRESS>instead of version 1.8 and check if this still happens :)

Thank you, that was an embarrassing mistake. On to my next error.


Ask gpt4 to decide which timestamps require new screenshots

Perhaps you could feed the transcript, including timestamps, to gpt4 and ask it what timestamps seem like they should be accompanied with a new frame from the video, to save on image storage costs. gpt4 would decide based on e.g. if the narrator refers to something in the video. ofc not perfect but may be sufficient to significantly save on image storage costs and make the article text more readable.

I don't think that reliance on an LLM is in the cards for this tool.   I have a new version I'm fiddling with that uses other heuristics to coalesce the transcript, although I don't know if that will see the light of day.


Add createPopup function to config

Useful when we need custom case.
It can be based on provider (window size) or window specs.

seems sane to me. would you mind squashing those commits @merqlove ?

This patch is about custom popup window, for example if we need window with size 600x400

i gotcha, but would you mind rebasing those commits and squashing them into a single commit? will make for cleaner history once it merges.

Yeah, i can force it

thanks @merqlove!

@booleanbetrayal Also will be good if someone try renewed tests :) Maybe Travis

Yeah, seem that it fails  test/test/unit/ng-token-auth/multiple-user-types.coffee
I can fix it in other PR

Thanks @merqlove ... hard to run tests from my phone.  =]


fix packet encoding/decoding which seems to break after 2 billions packets sent

Hello,
Long story short, I've been using the encoding/decoding logic of packet number for another purpose than encoding pn, and observed that it would start decoding incorrect values when reaching 2^31.
I dived into RFC9000, on the Appendix for the algorithmic details of encoding/decoding, and observed that the encoding algorithm in quiche does not match the encoding algorithm suggested by the RFC. I suppose there is a reason for this, but it eludes me.
So, this pull request does an implementation close to what is suggested (hopefully more efficient), and fixes my problem, and the bug with packet numbers too. I've also added test cases for packet encoding/decoding using example values from the RFC.
This passes all unit tests, and a  simple integration test in which I downloaded a 5 GiB file through the H3 module using quiche-[client/server]. However, I am unsure about:

Why the largest acked value on a path can be bigger than the current packet number on that path.
Why largest_acked values were initialized to UINT64::MAX; I changed this to 0, and removed a condition checking for UIN64::MAX. I had to do this, because pkt_num_len() is now using information from the largest_acked value.
Whether the original pkt encoding is a bug, a forgotten todo, or something else. The fact that there were no test cases suggests a forgotten todo?

@LPardue @ghedo I've other pull requests with potential quality of live improvements (nothing big) from my ongoing project using quiche. Are those welcome? If external contributions are not, I prefer not to spam :-)
(e.g., support of recvmmsg() for apps/src/client.rs used in HTTP download)

@frochet also, specifically about recvmmsg() support, @evanrittenhouse has been pulling all that UDP socket logic into its own crate https://github.com/cloudflare/quiche/pull/1764 so that is probably a better place to look into for that change.

I've asked for a quicdev slack invite. Thank you for the guidance :-)
Rebase done! It looks like some changes in Master addressed the first two questions in the first post as well, so great!

Merged, thanks @frochet! And sorry it took so long :disappointed:

And also thanks @vanc for the review and suggestions!


404 page

Can we have a 404 page with the regular header and footer and I will draft the content for the page (suggested links etc).

Templates are https://github.com/CoderDojo/cp-zen-platform/blob/master/web/public/templates/errors/404.dust and https://github.com/CoderDojo/cp-zen-platform/blob/master/web/public/templates/errors/404_no_headers.dust
CSS is here https://github.com/CoderDojo/cp-zen-platform/blob/master/web/public/css/dashboard.less

Hey there, would like to help out with this one - can you let me know what exactly you would like to see when hitting the 404?
Currently it just reads 'PAGE NOT FOUND!'
Cheers

Hi @flakyfilibuster - thanks for your interest in helping out!
I'm open to ideas on this one - we'd love something unique to CoderDojo - maybe something with pixel art ninjas. It'd be great to get some design help on this. I love the animation on the Github 404 page so it'd be neat to have something like that.
Do you have any ideas?

@tangentfairy If you specify it more, I can add tasks for Google CodeIn - You could also become a mentor in google codein to supervise this. It would look something like this:

Description
Requirements:
Expected Outcome:
**Links/Tutorials: **

And it should last about 3-5 hours and can be given to several students.

If you can help the students to set up locally then that sounds good! They
might need help with making contributions.
On Monday, December 21, 2015, Nicco Kunzmann<EMAIL_ADDRESS>wrote:

@tangentfairy https://github.com/tangentfairy I like the ides. If you
specify it more, I can add tasks for Google CodeIn - You could also become
a mentor in google codein to supervise this. It would look something like
this:
Description
Requirements:
Expected Outcome:
Links/Tutorials:
And it should last about 3-5 hours and can be given to several students.
They could for example create pull requests to a certain folder.
—
Reply to this email directly or view it on GitHub
https://github.com/CoderDojo/community-platform/issues/147#issuecomment-166403322
.

--
Ursula Clarke Everett
Technical Lead @ CoderDojo Foundation
S: tangentfairy | T: @CoderDojo | @tangentfairy
--
Volunteer at your local Dojo and inspire the next generation of digital
creators!

Pledge to start a Dojo https://coderdojo.com/pledge-a-dojo/
How to volunteer at your local CoderDojo
https://coderdojo.com/get-involved/volunteer/
Read the CoderDojo Foundation Strategy
https://coderdojo.com/news/2015/08/07/the-coderdojo-foundation-strategy-20152016/

--
Hello World Foundation operating as the CoderDojo Foundation is a company
limited by guarantee not having a share capital with charitable status, CHY
20812. Registered in Dublin, Ireland, No 524255. Registered Office:
Dogpatch Labs, Unit 1, CHQ Building, Custom House Quay, Dublin 1, Ireland.

hey @tangentfairy,
unfortunately I am as bad a designer as I'm an artist - I was under the assumption that there are already some images or a finished design that needs implementation.
You really don't want me drawing something :P

@flakyfilibuster - we are still looking for design help on this one. For the moment, I'd be open to putting something like this on the 404 page and just putting a message that says:
"Oops! This is not the page you are looking for. Maybe you'd like to check out some of the following fun sections of our site:
Find your local Dojo
Community Forums
Kata, our community wiki
CoderDojo community news
"
For me, doing the above would close this issue until we can make a fancier 404 page! :smile:

We can also make a CoderDojo contest out of this and put all the 404 designs in a new folder and each time a random design is chosen. One could put scratch projects there, iframes, images, ..., svgs, gifs, ...

@niccokunzmann - this would be great in the long run!

@flakyfilibuster Let me know if I can help with design!

Hey there,
sorry for the radio silence :(
I was actually looking for royalty free ninja pics, to pixelate, however if you can come up with some ideas or even images/ designs, that would be great.

I think the CSS ninja I mentioned could be used! Then it'd be a really cool example of what you can do with CSS for our young ninjas too! :)

Oh, I didn't know that one was fair game, that we need to honor some licensing rights etc, etc.
Well good then, I'll put that in - thanks.

It's open source afaik!

http://blog.codepen.io/legal/licensing/ Anything public = MIT License
What if we animated the ninja so that it drops down from the "ceiling" on a rope or something?

That's a really cool idea @gisellegk! Maybe if we can get the basic ninja up there first then we can "iterate" on it and improve it to animate :) It would be really cool to use CSS animations to showcase what CSS can do!

Hey there, this is where I am right now:
Ninja stays centered in desktop and mobile view.
Obviously the text needs some padding/margins.

Should the text be centered?
How about moving the "Oops! This is not the page you are looking for." text above the ninja and the rest underneath?
Are there some styleguides as to what type of tags the text should have? h2/h3 for main header? Just paragraphs?

Feedback is appreciated.
Thanks

@flakyfilibuster: I like the text centered! I'd prefer the oops text to stay with the rest of the text as I think it is easier to read.
We don't have a style guide at the moment - this is something we're looking at doing in the first quarter and have a volunteer designer working on it! I think the "Oops" could be in a h2/h3 - I'll leave it up to you to decide which :)
Looks great!

hey @tangentfairy,
added a pull request https://github.com/CoderDojo/cp-zen-platform/pull/794
Can you have a look and let me know if anything is missing/ wrong?
Much obliged

This is live now and looks really cool!
Let me know if you have any questions about other issues @flakyfilibuster - there are a lot at the moment (and most are actually small features). Some are not specced out but I can help with any of them.
If you want to join the Slack development channel ping me a mail on<EMAIL_ADDRESS>!


Secrets management

The most annoying thing about (hypothetically) deploying a new system is secrets management. I have a bunch of secrets floating around my system with no reliable way to back them up.
https://wiki.nixos.org/wiki/Comparison_of_secret_managing_schemes
I'm hesitant to include them in-tree.

Bitwarden Secrets Manager looks promising. Free plan, self-hostable if that changes, per-machine credentials (the only manual step would be creating a machine and adding its key somewhere on the system).
I'm not sure if any pre-existing integration is available, but it should be trivial thanks to the CLI.


Fix issues with local scene init

This makes a few changes.
It pulls the event listeners for OBR.scene.items.onChange and OBR.scene.onMetadataChange out of the initLocalItems function to prevent them being called multiple times and causing leaks.
It ensures that any interaction with the local scene is done after the scene is ready.
It checks to see if the scene is already ready once the extension loads and will call the sceneReady function when it is.

Thx again for you input I reworked The whole updating functionality based on your feedback in #18 So this PR is no longer necessary. But with out it the other changes wouldn't have been possible.


ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: -11) local_rank: 0

Hi, I am trying to run your code, but I get the Error "ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: -11) local_rank: 0 (pid: 69934)". I cannot solve this problem according to this error report. Could you help me to solve this problem?
In my experiments, I only use 2 GPUs (Tesla V100, 32G), the environments are installed following your instruction. The detailed errors are depicted  in the following:
(stgrounding) yjh@DGX-1:~/workspace1/STCAT$ CUDA_VISIBLE_DEVICES=0,1 bash run_vidstg.sh
/home/yjh/anaconda3/envs/stgrounding/lib/python3.8/site-packages/torch/distributed/launch.py:178: FutureWarning: The module torch.distributed.launch is deprecated
and will be removed in future. Use torchrun.
Note that --use_env is set by default in torchrun.
If your script expects --local_rank argument to be set, please
change it to read from os.environ['LOCAL_RANK'] instead. See
https://pytorch.org/docs/stable/distributed.html#launch-utility for
further instructions
warnings.warn(
WARNING:torch.distributed.run:

Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your appli
cation as needed.

ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: -11) local_rank: 0 (pid: 44291) of binary: /home/yjh/anaconda3/envs/stgrounding/bin/python3
Traceback (most recent call last):
File "/home/yjh/anaconda3/envs/stgrounding/lib/python3.8/runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/home/yjh/anaconda3/envs/stgrounding/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/home/yjh/anaconda3/envs/stgrounding/lib/python3.8/site-packages/torch/distributed/launch.py", line 193, in
main()
File "/home/yjh/anaconda3/envs/stgrounding/lib/python3.8/site-packages/torch/distributed/launch.py", line 189, in main
launch(args)
File "/home/yjh/anaconda3/envs/stgrounding/lib/python3.8/site-packages/torch/distributed/launch.py", line 174, in launch
run(args)
File "/home/yjh/anaconda3/envs/stgrounding/lib/python3.8/site-packages/torch/distributed/run.py", line 710, in run
elastic_launch(
File "/home/yjh/anaconda3/envs/stgrounding/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 131, in call
return launch_agent(self._config, self._entrypoint, list(args))
File "/home/yjh/anaconda3/envs/stgrounding/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 259, in launch_agent
raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
scripts/train_net.py FAILED
Failures:
[1]:
time      : 2023-03-06_10:59:36
host      : DGX-1
rank      : 1 (local_rank: 1)
exitcode  : -11 (pid: 44292)
error_file: <N/A>
traceback : Signal 11 (SIGSEGV) received by PID 44292
Root Cause (first observed failure):
[0]:
time      : 2023-03-06_10:59:36
host      : DGX-1
rank      : 0 (local_rank: 0)
exitcode  : -11 (pid: 44291)
error_file: <N/A>
traceback : Signal 11 (SIGSEGV) received by PID 44291
(stgrounding) yjh@DGX-1:~/workspace1/STCAT$

Sorry, I can not figure out your problem based on this error information. Maybe you can change nproc_per_node=2?

Hello, may I ask you has this problem solved? Is it possible to use two GPUs to achieve the effect of the original paper?


In what projection should I provide geometries for encoding?

Hi! I'm having some trouble with my vector tiles and I think I'm using the wrong projection. The example shows a WKT geometry in what appears to be latitude & longitude, but the vector tiles spec mentions both spherical mercator and pixel values. So here's my question:
When using mapbox_vector_tile.encode(), should I provide geometries in spherical mercator (meters), degrees latitude and longitude, or tile-relative pixels?
Thanks!

Tile relative values. We take mercator meters and then scale them to a range of 4096. The scaled values are what we pass to the encoder.

Thanks. Would you consider a PR that documents this?

Yes, absolutely! :)
I created an issue for it already: https://github.com/mapzen/mapbox-vector-tile/issues/12


Steps in readme make failing tests

Version of infeos
0.1.33
Describe the bug
I ran through the steps in the readme, and when I run infeos test I get:
$ infeos test

  Planet Contract Tests
    1) should create a new planet

  Star Contract Tests
    2) should create a new star
    3) should add a new planet to existing star
    4) should throw when adding a non-existing planet to a star

  0 passing (16ms)
  4 failing

  1) Planet Contract Tests
       should create a new planet:
     TypeError: universeContractInstance.createstar is not a function
      at Context.it (test/PlanetTests.js:43:40)

  2) Star Contract Tests
       should create a new star:
     TypeError: universeContractInstance.createstar is not a function
      at Context.it (test/StarTests.js:43:40)

  3) Star Contract Tests
       should add a new planet to existing star:
     TypeError: universeContractInstance.createstar is not a function
      at Context.it (test/StarTests.js:62:40)

  4) Star Contract Tests
       should throw when adding a non-existing planet to a star:
     TypeError: universeContractInstance.createstar is not a function
      at Context.it (test/StarTests.js:89:40)

To Reproduce
Steps to reproduce the behavior:
Follow the steps in the readme until the end.
Expected behavior
I expected the tests to pass.
Desktop (please complete the following information):

OS: macOS X Mojave
Version 10.14.2

yes this is a problem. adding https://gist.github.com/DimitarSD/9b3af2a3312b78fe4c0826018e7346ce in the src & build folders is solving it.
They just didn't mention you need to do it


Add HTTP/2 Push Support to Caddy

https://caddyserver.com/v1/docs/push
push /index.html {
  /assets/styles.css
  /assets/keepass.js
  /assets/*.js
}

See also https://github.com/caddyserver/caddy/pull/1215

@milgner since we stay on v1 for now, any chance this will make it in?

I had some issues with upgrading Caddy to 2 and am thinking about moving back to Nginx now. And with only minimal research I saw that it would be able to address this issue, too: https://www.nginx.com/blog/nginx-1-13-9-http2-server-push/

Go for it then. Nginx can also handle let's encrypt cert creation


feat(site): add two ways to compile target demo when running start

PR Type
Imporve compiling site.
How to use
Suppose the demo you want to compile is button/demo/ghost.md, then

run command npm run start button/demo/ghost.md

add only: true at the top, which is called metadata, as below,

---
only: true
order: 8
title:
  zh-CN: 幽灵按钮
  en-US: Ghost Button
---

Any progress?

It's not a good idea to write
only: true

Better to make a parameter of command npm run start, like npm run start button.

CI won't pass if any commit contains only, so I think it doesn't matter.

CI won't pass if any commit contains only, so I think it doesn't matter.

CI takes time. And developers still needs to add a commit for remove that line.

OK, I'll add a check before commit.

Now you can run npm start button/demo/ghost.md to compile that one.

Ping~

@hsuanxyz Have time to review this?


possible to copy/sync xattr

Is it possible to sync/copy xattr of files by providing some sort of flag? (similar to rsync --xattr)

Hello,
extended attributes have not been a topic so far.
What I could do for you: add new option --cpOptions.
(there has already been a related interest in the past)
This option would make it possible to specify different cp command line options, e.g.:
--cpOptions='--preserve=timestamps,xattr'
This should preserve the extended attributes during copying (according to the cp manpage). Also a solution for the Local Mode.
Detection of changes of extended attributes is out of scope, however, unless such changes are detectable via file modification times.
Let me know.
F.

Yes, that would work for my use case.

Hi, your Enhancement Request is there. F.

Thanks for implementing this 👍


CRTB status field support

Issue:
See #44663
Problem
This is an enhancement to bring the CRTB CRD nearer to regular kube operation, with a status field which can be queried, waited on, etc.
Solution
A status field is added to the CRD, and the controllers are extended to manage the contents of the field.
Testing

Engineering Testing
Manual Testing
Ran a local rancher cluster and inspected system CRTB to confirm presence of proper status and conditions.
Automated Testing
Added unit tests for CRTB reconciliation functions.
QA Testing Considerations

Regressions Considerations

TODO
Existing / newly added automated tests that provide evidence there are no regressions:

TODO

Looking into the Drone CI failures.
The general error message seen over and over is:

custom_test.go:55: cluster test-custom-one-node creation wait failed on: prov cluster is not ready: timeout waiting condition: context deadline exceeded

The relevant code is
https://github.com/rancher/rancher/blob/2b0b02f11c60397c20989a87d4b09be4ef5a12fd/tests/v2prov/tests/custom/custom_test.go#L53-L55
The referenced WaitForCreate and message is at
https://github.com/rancher/rancher/blob/2b0b02f11c60397c20989a87d4b09be4ef5a12fd/tests/v2prov/cluster/clusters.go#L140-L145
The cluster object is declared at
https://github.com/rancher/rancher/blob/2b0b02f11c60397c20989a87d4b09be4ef5a12fd/pkg/apis/provisioning.cattle.io/v1/cluster.go#L14
There is a kube Watch on this resource, and it is not reported as ready per all the conditions, for more than 5 minutes.
Question is, are we watching the main cluster of Rancher, or local, or something else ?
Where would we find the logs of this cluster ? (To research why it is not getting ready).
Are such logs even available ?

There is a kube Watch on this resource, and it is not reported as ready per all the conditions, for more than 5 minutes. Question is, are we watching the main cluster of Rancher, or local, or something else ? Where would we find the logs of this cluster ? (To research why it is not getting ready). Are such logs even available ?

Ok, there is the test data bundle ... That looks to be base64 ... Copied ... Decoded ... Gzip ... Decompressed ... Looks to be a JSON dump of the Cluster resource in question.
Messages of interest in that json:

"reason": "Waiting",
"message": "non-ready bootstrap machine(s) auto-integration-test-cluster-prkrt-pool-0-fwclg-7lg5m and join url to be available on bootstrap node"

"reason": "WaitingForAvailableMachines",
"message": "Minimum availability requires 1 replicas, current 0 available"

{
"type": "AgentDeployed",
"status": "False",
"lastUpdateTime": "2024-06-03T11:47:08Z",
"reason": "Error",
"message": "Error while applying agent YAML, it will be retried automatically:
exit status 1,
Error from server (Forbidden):
error when retrieving current configuration of: Resource:
"rbac.authorization.k8s.io/v1, Resource=clusterroles", GroupVersionKind: "rbac.authorization.k8s.io/v1, Kind=ClusterRole" Name: "proxy-clusterrole-kubeapiserver", Namespace: ""
from server for:
"./management-statefile_path_redacted": clusterroles.rbac.authorization.k8s.io "proxy-clusterrole-kubeapiserver" is forbidden:
User "u-mutgmwsnsf" cannot get resource "clusterroles" in API group "rbac.authorization.k8s.io" at the cluster scope
[...]

Lets attach this: tdb.json
And a log dump after it, also base64 ... Decoded, decompressed. Attached:

Relevant flaky test: https://github.com/rancher/rancher/issues/42248

@JonCrowther
Trialed the PR incrementally to see when importing a generic cluster breaks.
As you may see I was able to have the entirety of the status mgmt, including conditions, for /rbac, i.e. remote cluster.
It is only when I start adding status mgmt for /auth things go sideways.
Adding the mgmt to the auth Create alone is :heavy_check_mark: (This I have locally and not pushed here).
Adding the mgmt to the auth Update alone is :boom:  (This is here, commit 81aa19c19714774cf807b24206c614f3d21a5e89).
So the issue seems to be something around CRTB status updates.
Things to be tried now:

Remove Update mgmt in the /rbac code, but not in /auth.

IOW maybe the two Updates interfere with each other ?

Remove the new condition at the top, for the early bailout when only status was changed.
???

Status:
The current state of the PR represents a rework of the controllers to carefully handle the shared Status field.
This depends on the Status.Conditions array to be properly shared and visible at the beginning of the controller's Create and Update functions.
Each controller knows which conditions

represent it own success, and bails out without making changes if these are all present.
it is able to set, and removes these, and only these from the incoming set before it performs its reconciliations.
These are then able to add the conditions back as they are encountered.
represent total success, i.e. its own and the others. The status is set to Completed by whichever controller  sees total success.
Any failure conditions set an Error status.
Partial success without failure conditions leaves the status as in In Progress.

Unfortunately the assumption noted at the beginning, i.e. the Status.Conditions array to be properly shared and visible at the beginning of the controller's Create and Update functions, was found to be not true. On each entry into Create or Updated of either controller the Conditions array is found to be empty. This means that the conditions of either controller are not visible to the other, nor itself on the next round reconciliation.
It is currently not clear what the cause of this is. Possibilities currently entertained are:

The kube controller framework set the array to empty.
The rancher controller framework(s) sets the array to empty.
The controllers are not properly saving the accumulated conditions to kube.


Add spikeinterface support for SpikeGadgetsRecordingInterface

As stated in the title. Linked to https://github.com/catalystneuro/nwb-conversion-tools/issues/500.

There is a problem with the doctest as spikeinterface is producing output to standard output. See:
https://github.com/SpikeInterface/spikeinterface/pull/758


Can't figure out how to order functions in the docs using documentation.js seems like it isn't an option based on issue reports from 2017!

Can't figure out how to order functions in the docs using documentation.js seems like it isn't an option based on issue reports from 2017!
THIS IS Supper ANNOYING but most big JavaScript seems to be built on JSDoc which has same limitations.

Is lists of the current order and the desired order?
Are these the steps to duplicate the problem?
How to Make the Docs

Add new markdown files to the docs by making changes to docs/doc_setup.yml or the documentation between /** **/ in the index.js file.
Install locally documentation.js
Run in the home directory of wellioviz:
documentation build ./dist/index.js --config docs/doc_setup.yml -f html -o docs
go to the docs directory. Run: python3 -m http.server Please note, that if you run the server   from anywhere except the docs directory, the image links won't work. The docs page on the github.com repository runs from the docs folder on the master branch as well.
Go to a browser and open http://localhost:8000/index.html . You should see the docs.

Thanks,
DC

Yes, those are the steps to replicate the problem.
Under the header "Wellioviz Function Types", I'd like to insert specific functions under each time. So for example, I'd like to insert the documentation for the curvebox() function under the "plotting functions" header which is under "Wellioviz function types".
The status quo is documentation.js puts the inline documentation for each function at the bottom of the docs in the order of the functions.
From issues in documentation.js it looks like documentation.js might not be able to positive functions if they are treated as module exports, which all wellioviz functions are. However, that's my interpretation as that isn't spelled out exactly anywhere.

Determined this is a limitation with documentation.js


[US4256] refactor(code): Restructure NDM build directory.

This is the change to refactor build directory to accomodate building one more binary related to operator-sdk.
Signed-off-by: satbir<EMAIL_ADDRESS>

Codecov Report

Merging #211 into master will decrease coverage by 0.45%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #211      +/-   ##
==========================================
- Coverage    51.1%   50.65%   -0.46%
==========================================
  Files          43       43
  Lines        1769     1769
==========================================
- Hits          904      896       -8
- Misses        800      806       +6
- Partials       65       67       +2

Impacted Files
Coverage Δ

cmd/ndm_daemonset/probe/eventhandler.go
34.69% <0%> (-12.25%)
:arrow_down:

cmd/ndm_daemonset/probe/udevprobe.go
63.24% <0%> (-1.71%)
:arrow_down:

Continue to review full report at Codecov.

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


add Third-Party Implementations inpaint_wechat

Update README.md

Thank you for your contribution! 🚀 I have locally merged the pull request with some changes and updated the main branch.

Thank you for your contribution! 🚀 I have locally merged the pull request with some changes and updated the main branch.


What about adding a refresh method like in iScroll 4?

First off, I want to say that the scroller behavior is pretty nifty. Thank you.
I would like to know if there is a way to refresh the scrollbar once content has been added dynamically (through ajax for example.)
Cheers

Cheers buddy!


Add: Tile Matching Tkinter Game

Type of change

[X] Added new project
[ ] Bug fix
[ ] New features
[ ] Enhanced documentation

Changes proposed in this pull request

Project name : Tile Matching Game with Time Limit

Short description :

This is a Tile Matching game with a time limit made using Tkinter GUI in python
Hope you like it! If you do kindly merge it in this repo.
Checklist

[X] The code is Python 3 compatible
[X] I have added relevant comments in my file to make code understandable
[X] I have added my project in a folder which includes all the necessary files

Please add the hacktoberfest labels if you do merge 🙌.
Thank you!!


[BACKPORT - 3.0.x] backport #3233

Which issue is fixed by this PR? If not related to an existing issue, what does this PR do?
This backports #3233
Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[x] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR? Please include manual steps or automated tests.
(If no tests are part of this PR, please provide explanation as to why no tests are included.)
Check all that apply

[ ] This PR includes tests
[ ] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3348/
Test PASSed.


Pieces plugin for eclipse IDE

After the IDEA IntelliJ IDE, the most commonly used open source ide is Eclipse, I think it's time that we have what most Developer loved application plugin is for Eclipse.

Going to close this 👍


WORMS lookup - should we include 'unaccepted' or 'superseded combination' data from WORMS?

This relates to the fuzy match algorithm used by AphiaRecordsByMatchNames  at https://www.marinespecies.org/rest/ which includes a 'status' in the response . This status can be 'accepted', 'unaccepted' and 'superseded combination'.

Removed worms from HELCOM handler.


Extend archive module to support TTL and delay

Added new workflow status listener called ArchivingWithTTLWorkflowStatusListener that supports delayed archiving and setting time-to-live for the documents in Redis for now (can be extended to Cassandra later).
Delayed Archiving: It is required in cases where short workflows are indexed after a delay and updateIndex will fail in those cases since the document might not exist. Further, when using async indexing, in case of high throughput scenarios, an asynchronous create document index request could be pending in the indexQueue, and the synchronous update request will fail due to missing document.
Time-to-live: It is helpful in the case of TERMINATED workflows. If the parent workflow is terminated and then sub-workflow is canceled. In those cases, the task worker could still be running the task and when it completes and try to update the status of the task, the task/workflow will be missing in persistence if the workflow is archived immediately after the termination/completion.  So TTL can help in this case.  Also if the parent-workflow need to lookup output from sub-workflow, using TTL can help the data to be available in the memory(Redis case) rather than disk lookup in elasticsearch.

Pull Request Test Coverage Report for Build 4199

26 of 123 (21.14%)  changed or added relevant lines in 11 files are covered.
23 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.5%) to 69.071%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cassandra-persistence/src/main/java/com/netflix/conductor/dao/cassandra/CassandraExecutionDAO.java
0
1
0.0%

contribs/src/main/java/com/netflix/conductor/contribs/ArchivingWorkflowModule.java
0
1
0.0%

mysql-persistence/src/main/java/com/netflix/conductor/dao/mysql/MySQLExecutionDAO.java
0
1
0.0%

postgres-persistence/src/main/java/com/netflix/conductor/dao/postgres/PostgresExecutionDAO.java
0
1
0.0%

core/src/main/java/com/netflix/conductor/core/config/Configuration.java
0
3
0.0%

core/src/main/java/com/netflix/conductor/metrics/Monitors.java
2
6
33.33%

contribs/src/main/java/com/netflix/conductor/contribs/ArchiveWorkflowStatusListenerProvider.java
0
7
0.0%

core/src/main/java/com/netflix/conductor/core/orchestration/ExecutionDAOFacade.java
12
25
48.0%

redis-persistence/src/main/java/com/netflix/conductor/dao/dynomite/RedisExecutionDAO.java
10
29
34.48%

contribs/src/main/java/com/netflix/conductor/contribs/listener/ArchivingWithTTLWorkflowStatusListener.java
0
47
0.0%

Files with Coverage Reduction
New Missed Lines
%

redis-persistence/src/main/java/com/netflix/conductor/dao/dynomite/RedisExecutionDAO.java
2
74.92%

core/src/main/java/com/netflix/conductor/core/orchestration/ExecutionDAOFacade.java
3
54.68%

cassandra-persistence/src/main/java/com/netflix/conductor/dao/cassandra/CassandraEventHandlerDAO.java
8
68.42%

cassandra-persistence/src/main/java/com/netflix/conductor/dao/cassandra/CassandraMetadataDAO.java
10
55.44%

Totals

Change from base Build 4194:
-0.5%

Covered Lines:
12408

Relevant Lines:
17964

💛  - Coveralls

@apanicker-nflx Can you please review this? This PR is expected to solve problems related to a couple of use cases in archiving workflows. Thanks for your help!


Error 401 with TLS 1.2 on Alamofire 4.0 with trust policies

I can't set the trust policy for request a server with TLS 1.2 configuration. I have a SessionManager with this configuration:
lazy var sessionManager: SessionManager = {
    let serverTrustPolicies: [String: ServerTrustPolicy] = [
        "ws-boots-uat.clm.comarch.com": .pinCertificates(
            certificates: ServerTrustPolicy.certificates(),
            validateCertificateChain: true,
            validateHost: true
        )
    ]
    let session = SessionManager(configuration: URLSessionConfiguration.default, serverTrustPolicyManager: ServerTrustPolicyManager(policies: serverTrustPolicies))
    session.adapter = AccessTokenAdapter()
    return session
}()

Now I set my info.plist  with allowing insecure http on the key AppTransportSecuritySettings with the follow configuration:
<key>NSAppTransportSecurity</key>
<dict>
    <key>NSExceptionDomains</key>
    <dict>
        <key>ws-boots-uat.clm.comarch.com</key>
        <dict>
            <key>NSExceptionAllowsInsecureHTTPLoads</key>
            <true/>
            <key>NSExceptionRequiresForwardSecrecy</key>
            <false/>
            <key>NSIncludesSubdomains</key>
            <true/>
            <key>NSTemporaryExceptionMinimumTLSVersion</key>
            <string>TLSv1.2</string>
        </dict>
    </dict>
</dict>

Now when I put a request, I use my session manager with the following code:
let urlManager: URLManager = URLManager.sharedInstance

        urlManager.sessionManager.request(URLManager.Router.registerUser(params: params)).validate(statusCode: 200..<300).responseObject(completionHandler: {
            (response: DataResponse<User>) in

            if let user = response.result.value{
                do {
                    let claims: ClaimSet = try JWT.decode(user.access_token, algorithm: .hs256(user.refresh_token.data(using: .utf8)!))
                    user.id = claims["sub"] as! Int
                    user.iss = claims["iss"] as! String
                    print(claims)

                    self.saveUser(user: user, success: nil, failure: { error in
                        reject(error)
                    })

                    let urlManager = URLManager.sharedInstance
                    urlManager.sessionManager.request(URLManager.Router.getUserInformation(userID: user.id, params: nil)).responseObject(completionHandler: {
                        (response: DataResponse<User>) in

                        guard response.result.value != nil else{
                            print(response.response as Any)
                            reject(response.result.error!)
                            return
                        }

                        self.updateUserInfo(userInfo: response.result.value!, user: user, success: {
                            fulfill(self.currentUser()!)
                        }, failure: { error in
                            reject(error)
                        })

                    })

                } catch {
                    print("Failed to decode JWT: \(error)")
                    reject(error)
                }
            }else{
                print(response.response as Any)
                reject(response.result.error! as NSError)
            }

        })

What am I doing wrong?
Note: I'm also using JWT encoding framework and Alamofire Object Mapper Framework.
Best Regards

What errors are you actually receiving? It's likely there will be more detail there.

Well, it seems the API was wrong because I was wrong with the API's endpoints and thats it. The configuration was right. Thanks anyway @jshier


chore(docs): update Appointment Manager documentation

chore(docs): update Appointment Manager documentation

@pmarchini @fredmaggiowski this PR can be merged


There are some question about this code

This code is a very good job, but in the process of my study, I found some problems, so I hope the author can help me to solve them.

In File "Dassl.pytorch/dassl/data/datasets/base_dataset.py", line 25: assert check_isfile(impath),
I think this maybe wrong, do it need to check folders instead of files?

impath is supposed to contain the path to an image rather than a folder

In File "Dassl.pytorch/dassl/data/datasets/base_dataset.py", line 25: assert check_isfile(impath),
I think this maybe wrong, do it need to check folders instead of files?

The path of "class_names" and "class_path" in office31.py may not be right.
There is a subdirectory named "images"  in the directory "domain_dir".

do a little bit of work by moving all class folders to domain_dir
please refer to DATASET.md

This is a very good job, especially when I have carefully debugged M3SDA, but I have a question about how you can sample as much data as possible for each source domain in a dataset with multiple source domains I think this is not an accident because a split_bach is clearly pointed out in your code. Thank you very much.

And there is another problem, because the data of each source domain is not the same. When all the data of the source domain are collected into one data set, when sampling the same data of each source domain, will there be no data available for sampling in the source domain with less data?

@buerzlh please refer to https://github.com/KaiyangZhou/Dassl.pytorch/blob/master/dassl/data/samplers.py#L7

OK, i got it. Thank you very much.


How can I actually control the order of actions during a phase?

I want to run gulp before I run my tests. My first idea was to simply run a script during the prepare phase. But that script is run before bower and node dependencies are restored, so I'm missing those.
I could just run my script in the test phase, but that doesn't seem like the right place.
I assumed that I could control this behavior by re-ordering the plugins for the branch, but that has no effect on the order of these actions.
I can see how some people would like to run a certain action before cached modules are restored and how others would like to do that after they're restored. So, is there a way?

And I think I just got my answer.
When you're using strider.json to control your strider config, the plugins you configure in there are pushed to the top of the plugin stack, so they always run first.

And I just got a tiny bit smarter :P Not using merge_plugins in strider.json will obviously, not merge the configurations.
So you just have to configure all plugins in strider.json and then only the order in strider.json matters.
Now, what is actually the desired behavior here? Should people override their web config with strider.json? Should one extend the other? If so, which one's the root?
I also wrote some code at https://github.com/oliversalzburg/strider/commit/b2a9f5d3dc6bee39c14185163952490d0e72d458, but I'm no longer sure what the correct approach is. Please advise.
Relates to #789


False negative for manuscript hosted on institutional repository "hal"

Dear unpaywall team,
Thank you for developing such a nice tool!
I have found a couple of false negatives:

http://dx.doi.org/10.1016/j.neuroimage.2016.03.054 >> http://www.hal.inserm.fr/inserm-01291748/file/Maumet_NeuroImage_A_Contrario.pdf
https://doi.org/10.1016/j.neuroimage.2013.04.079 >> http://www.hal.inserm.fr/inserm-00816852/file/Maumet_NeuroImage_2013.pdf

I have checked and the source repository "HAL" (both hal Inria & hal Inserm) seems to be available in your data sources.
Is there anything I can do to help with this issue?

Hi!  Thanks for reporting it!  We'll get it fixed.
Best,
Heather
--
Heather Piwowar
cofounder of Impactstory http://impactstory.org/: share the full story of
your research impact.
working from Vancouver, Canada
@researchremix http://twitter.com/#!/researchremix and @Impactstory
https://twitter.com/ImpactStory
On Wed, Apr 4, 2018 at 9:11 AM, Camille Maumet<EMAIL_ADDRESS>wrote:

Dear unpaywall team,
Thank you for developing such a nice tool!
I have found a couple of false negatives:

http://dx.doi.org/10.1016/j.neuroimage.2016.03.054 >>
http://www.hal.inserm.fr/inserm-01291748/file/Maumet_
NeuroImage_A_Contrario.pdf
http://www.hal.inserm.fr/inserm-01291748/file/Maumet_NeuroImage_A_Contrario.pdf
https://doi.org/10.1016/j.neuroimage.2013.04.079 >>
http://www.hal.inserm.fr/inserm-00816852/file/Maumet_
NeuroImage_2013.pdf
http://www.hal.inserm.fr/inserm-00816852/file/Maumet_NeuroImage_2013.pdf

I have checked and the source repository "HAL" (both hal Inria & hal
Inserm) seems to be available in your data sources
https://unpaywall.org/sources.
Is there anything I can do to help with this issue?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Impactstory/unpaywall/issues/36, or mute the thread
https://github.com/notifications/unsubscribe-auth/AANngYLek1wo9tzQ-E3WrQyAFd09jw17ks5tlPC6gaJpZM4THFcX
.

Not sure what the issue was, but we found both of these on HAL Inserm.


Use default Shopware-way to update order payment status

1. Why is this change necessary?
The PayPal Unified plugin currently sets the payment status of an order directly via a SQL query. Therefore other plugins cannot track changes of the payment status because this does not triggers any of the usual hooks (e.g. hooks of the methods in the doctrine model or sOrder class).
Also no order status history entry is written.
2. What does this change do, exactly?
This PR changes the returnAction of the Shopware_Controllers_Frontend_PaypalUnified controller to use the method \Shopware_Controllers_Frontend_Payment::savePaymentStatus to save the order payment status. This method is part of SW and can easily be hooked by any plugins independently of which payment provider is used.
The method \SwagPaymentPayPalUnified\Components\Services\OrderDataService::applyPaymentStatus is removed. Please note, that this is a backward incompatiple change, but it is necessary.

Looks good! Please close this PR once you opened it in the upstream repository.

Closed in favor of https://github.com/shopwareLabs/SwagPaymentPayPalUnified/pull/3/files


ScaledObject doesn't have correct Idle/Min/Max Replica Counts specification

Report
Keda reports an error if all of idleReplicaCount, minReplicaCount and maxReplicaCount are 0, effectively disabling the scaler. It isi desirable to be able to disable a scaler instead of deleting it, analoguos to scaling a deployment to zero instead of deleting it.
Expected Behavior
keda should simply scale its target to 0 if all of idleReplicaCount, minReplicaCount and maxReplicaCount are 0
Actual Behavior
a warning event is logged: "ScaledObject doesn't have correct Idle/Min/Max Replica Counts specification"
Steps to Reproduce the Problem

create a cron scaler with all 3 attributes mentioned above set to zero
notice that a warning is logged

Logs from KEDA operator
example

KEDA Version
2.6.1
Kubernetes Version
1.21
Platform
Amazon Web Services
Scaler Details
cron
Anything else?
No response

Can you elaborate what the scenario is where your max replicas would be 0? Do you want to "pause autoscaling"?


Add test for Log Out options in workbenches

This PR implements ODS-1971
Signed-off-by: bdattoma<EMAIL_ADDRESS>

Jenkins Job: 1524 SUCCESS
Jenkins Job for smoke: COMING SOON