Failed to build json schema for nested Annotated

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
See example MRE below. JSON schema fails, but validation seems to work well.
Example Code
from pydantic import Field, TypeAdapter
from typing import Annotated
from collections.abc import Sequence

# validates if length of sequence is in [1..10], each item is in [0..10]
F = Annotated[Sequence[Annotated[int, Field(ge=0, le=10)]], Field(min_length=1, max_length=10)]

# fails with exception, not OK
TypeAdapter(F).json_schema()

# works, OK
TypeAdapter(F).validate_python([1, 2, 3, 4])

# validation error, OK
TypeAdapter(F).validate_python([1, 2, 3, 20])

# validation error, OK
TypeAdapter(F).validate_python([1, 2, 3, 4, 4, 5, 6, 6, 7, 8, 8, 3, 4, 5])

Python, Pydantic & OS Version
pydantic version: 2.1.1
pydantic-core version: 2.4.0
pydantic-core build: profile=release pgo=false mimalloc=true
python version: 3.11.4 (v3.11.4:d2340ef257, Jun  6 2023, 19:15:51) [Clang 13.0.0 (clang-13<IP_ADDRESS>)]
platform: macOS-13.0.1-arm64-arm-64bit
optional deps. installed: ['email-validator', 'typing-extensions']

Selected Assignee: @Kludex

I'm not sure what commit fixed it, but it's fixed on the main branch. It will be available on the next release.


Lua client can't connect to the server

How are you using the lua-language-server?
Visual Studio Code Extension (sumneko.lua)
Which OS are you using?
Windows
What is the issue affecting?
Annotations, Type Checking, Completion, Diagnostics/Syntax Checking, Hover, Libraries, Formatting, Spell Checking, Plugins
Expected Behaviour
I expect the extension to provide me everything it has.
Actual Behaviour
Instead, I get this error:
[Error - 17:48:21] Lua client: couldn't create connection to server.
Error: spawn UNKNOWN
	at ChildProcess.spawn (node:internal/child_process:412:11)
	at Object.spawn (node:child_process:707:9)
	at c:\Users\mikwee\.vscode-oss\extensions\sumneko.lua-3.2.0\client\node_modules\vscode-languageclient\lib\node\main.js:418:46
	at async LanguageClient.createConnection (c:\Users\mikwee\.vscode-oss\extensions\sumneko.lua-3.2.0\client\node_modules\vscode-languageclient\lib\common\client.js:1018:28)
	at async LanguageClient.start (c:\Users\mikwee\.vscode-oss\extensions\sumneko.lua-3.2.0\client\node_modules\vscode-languageclient\lib\common\client.js:601:32)

Reproduction steps

Start VSCode/VSCodium
See the error

Additional Notes
No response
Log File
No response

Because of some compatibility issues, the last available version on VSCode-OSS is 2.5.3.
3.2.0 is an failed attemption. This version has been reverted. Please try to install the old version manually.


Update dlp-learn-about-dlp.md

Cosmetic change

issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/6950

@JohanFreelancer9 Please copy review

@chrfox Please merge this PR. Thanks


Incorporate Verilog-parser Bug Fix

Changes

Incorporate Verilog-parser bug fix.
Add constant evaluator in Verilog analyzer to compute width from constant expression.

Hi Bo-Yuan, I don't quite understand why the first check failed while the travis-ci didn't...

The fail is not on the new branch, but in the merge commit. Seems that it's due to the new submodule tags... Since the new commit check is successful, will merge and check if the master analysis go through.


Run fade animation each time the Source changes

Platform: Android
Version with issue: <IP_ADDRESS>0

Hi,
we're making a gallery App and we need to display some images in a graceful way. Currently we have just one instance of CachedImage in our XAML and we were wondering whether it's possible to run the fade animation each time we change the Source property because, when the image is already loaded, it changes with no animation. Waiting for a prompt reply. Thank you for your efforts.
Best regards.

It should work with a custom config: https://github.com/luberda-molinet/FFImageLoading/blob/master/source/FFImageLoading.Common/Config/Configuration.cs#L21


Migrate functionality of interceptors from old reconciler

Description
The old reconciler offered support for intercepting the Helm cart content (adjusting resources before they were reconciled).
Existing interception logic has to be migrated to the new reconciliation approach.
AC:

[x] Adding disclaimer annotation (https://github.com/kyma-incubator/reconciler/blob/main/pkg/reconciler/service/annotationsinterceptor.go)
Decision: in template operator to be included
[x] Remove namespaces from cluster-wide resources (https://github.com/kyma-incubator/reconciler/blob/main/pkg/reconciler/service/clusterwideresourceinterceptor.go)
Decision: in responsibility of teams
[x] HPA replica field alignment (https://github.com/kyma-incubator/reconciler/blob/main/pkg/reconciler/service/hpainterceptor.go)
Decision: will be covered by central implementation in module manager
[x] Adding default labels to Kyma managed resources (https://github.com/kyma-incubator/reconciler/blob/main/pkg/reconciler/service/labelsinterceptor.go)
Decision: in template operator to be included
[x] Adding namespace as label (required for backward-compatibility between Kyma 2 and Kyma 1- https://github.com/kyma-incubator/reconciler/blob/main/pkg/reconciler/service/nsinterceptor.go)
Decision: out of scope, see previous bug-issue https://github.com/kyma-project/kyma/issues/13524
[x] Cover PVC edge-cases by blocking an update of immutable fields (https://github.com/kyma-incubator/reconciler/blob/main/pkg/reconciler/service/pvcinterceptor.go)
Decision: covered by https://github.com/kyma-project/module-manager/pull/167
[x] Cover IPService edge-cases by setting required fields (https://github.com/kyma-incubator/reconciler/blob/main/pkg/reconciler/service/servicesinterceptor.go)
**Decision:**probably obsolete after moving away from replace strategy, see https://github.com/kyma-incubator/reconciler/pull/403

@jakobmoellersap: I can see one AC related to the HPA interceptor. I don't see any code for that, though. How is that AC fulfilled? Thanks!

You are right, I purposefully not included it due to https://github.com/kyma-project/module-manager/pull/194#issuecomment-1363970415
TLDR: I think the HPA alignment is a workaround for malformed charts and we should not care about it in the central reconciliation. If people need it it can be implemented individually with Post Transforms


[SPIKE] - Assess UTxO HD impact on DBSync & Make Compatibility Changes

Business Hypothesis
We need to determine whether the UTxO HD implementation will impact DB-Sync. After discussion with Consensus team on 11/1/2022, it is likely that DBSync needs to change types.
Drivers
-UTxO HD LMDB is nearing completion and will require a compatibility upgrade
Functional Requirements

Identify modifications needed to support the UTxO HD LMDB release

NFRs
-Performance should not degrade or be materially different than baseline.
External Dependencies
-UTxO HD LMDB version is stable and available for testing against.
DoD
-QA testing completed
-audits completed
-benchmarks pass, no significant regressions
-merge to mainnet/release branch

My understanding is that the UTxO HD was never designed to be used by db-sync (ie the functionality is not exposed as a library) and therefore db-sync needs to keep on hold the whole of ledger state in memory which is actually kind of :hankey: .
For over 2 years, I have been advocating for ledger and consensus  to be enhanced so that db-sync can provide all the functionality it has now, without db-sync having to maintain its own copy of ledger state (db-sync has to maintain a copy but only actually uses a tiny, tiny portion of it).

Related https://github.com/input-output-hk/cardano-db-sync/issues/1233


the font size becomes samll in the preview window

Hi esm7, u make a perfect plugin i love. but i dont know wht the text in the preview window becomes smaller. It makes me hard to read. Thx.
the font size in preview window

Actual font Size

Does it look the same when you use the Page Preview feature (core plugin) of Obsidian?
Map View just launches the native Obsidian note preview, so I have no control over how it looks like, except applying some global CSS that would effect everything :(

Does it look the same when you use the Page Preview feature (core plugin) of Obsidian? Map View just launches the native Obsidian note preview, so I have no control over how it looks like, except applying some global CSS that would effect everything :(

Thank you for reminding me about it, I found that the problem is caused by plugin Hover editor. Interestingly, it only affects the preview windows font-size of map view.🤣


Check for invalid keys with multiple ":"

:tada: This issue has been resolved in version 1.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[common] API & Path enhancement

Allow to set default combine options per path instance

coverage: 62.027% (+6.3%) from 55.731%
when pulling 2f75776a114125bcf29b758e9476f99d475717aa on api-path-enh
into 6a7ef8b529cb3706f7d7c4fefc0a79561b0e7df7 on main.


Features/start chat btn

This change is marked as an internal change (Task), so will not be included in the changelog.

LGTM


Use with GitHub pages?

Doesn't seem this is possible, but wanted to check. If not possible, can it be a feature request?
GitHub pages can now be served out of a docs directory on master.
I'm looking to set up a user subdomain at js.org, which supports GitHub pages.

Hi,
sorry, but I am not sure I understand the question. What is that prevents you to accomplish your desidered task?


Hide orphan modules in Stats

A module is an orphan if it is not included in any chunk. Orphans modules are now hidden by default in Stats.
What kind of change does this PR introduce?
feature
Did you add tests for your changes?
yes
Does this PR introduce a breaking change?
yes
What needs to be documented once your changes are merged?
New stats option orphanModules, defaults to false.

Thanks


To make copyright visible in voter login

hey @QAZIMAAZARSHAD
As you can see in screenshot attached below that copyright section is not visible for voter login page .
I am interested to fix it.
Kindly assign this issue to me under LGM SOC '21

@puneetkaur2929 this issue has been fixed check again.

But you can do one thing

This is the screenshot of the help page, as you can see padding in the copyright line is more compared to other footers. Fix this.


Deleting the icon of a button doesn't immediately hide the icon

If a button is edited and the icon path is set to empty, it doesn't hide the icon until a restart

Fixed with #49


add link to test result details

@mr-c Thanks! @audrium Let's add similar nicely formatted links to other CWL v1.x conformance test results as well at a convenient location on the page.
@mr-c BTW as you could see there are quite a few failed conformance tests. We have simply published the current status. Please feel free to have a look and if you spot some really urgent troubles, please just open new issues besides the ones we already have for CWL v1.0 conformance.

Codecov Report

Merging #217 (b9bcbef) into master (422bf47) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #217   +/-   ##
======================================
  Coverage    1.28%   1.28%
======================================
  Files          10      10
  Lines         546     546
======================================
  Hits            7       7
  Misses        539     539

You are welcome @tiborsimko ; I've added better formatting and links for v1.1 and v1.2
If you have logs of the failures, I would be happy to look them over. I don't have a setup for running REANA on my laptop.


Add Autoprefixer

Proposed Changes

Add postcss-loader to webpack's .scss loaders. This is needed to use Autoprefixer.
Add postcss-loader to sassThreadLoader's warmup modules.
Add importLoaders: 1 to css-loader. Needed to use CSS Modules with postcss-loader,
source
Add postcss.config.js file.
The browser string used by Autoprefixer is the explicit last 2 versions of all mayor browsers:
"last 2 Chrome versions, last 2 Firefox versions, last 2 iOS versions, ...".
This is done instead of just "last 2 versions" because that pulls browsers with poor css support like "Blackberry Browser". Source
Update Readme.md

Coverage remained the same at 77.886% when pulling 65e8c8205096d23fa7f567021b558b501c0e11f5 on elas7:autoprefixer into 5ebfcfdc2bc0f4277724de05601f505f144b9428 on ModusCreateOrg:master.

Coverage remained the same at 77.886% when pulling 48a5041023a9971a09d6ae8c121e55e77760edaa on elas7:autoprefixer into 5ebfcfdc2bc0f4277724de05601f505f144b9428 on ModusCreateOrg:master.


Loads whole file into memory while uploading to backend

Especially when uploading large files on memory-limited systems, the backend will try allocating a lot of memory for the file and can panic if there is not enough memory, causing the upload to be incomplete and downcat to crash.
For some reason, the file's data is not being freed from memory until the whole file has been written to disk.
Possible solutions for this issue:

Freeing memory while parts of the file is written
Limiting the request so only a certain amount can be in memory at one time.

Unfortunately, I've tried to implement these solutions but I'm too dumb to solve them :)

Okei dokei


Add 'apt' to Command Library

Description
Add 'apt' to snippets.yml to allow detection of Layer changes made using the 'apt' command.
Implementation
Duplicate the snippets.yml entry for 'apt-get'
References
tern/command_lib/snippets.yml, lines 9-14
Super Issues
n.A.

May I work on this issue, please?

@oc37ejuc sure!

I have a question, regarding the Contribution Guidlines-
under 'After making changes' it states I should run prospector, and fix any issues it flags.
I have just cloned the forked repository, without making any changes yet, and prospector is flagging various issues.
Do I have to deal with all of those as well?

I have just cloned the forked repository, without making any changes yet, and prospector is flagging various issues.
Do I have to deal with all of those as well?

Can you paste the steps you took to install the requirements and the output of prospector here? You don't have to fix those, and the CI tests will probably not pick it up.

Steps as follows:
pip install wheel
pip install -e.[dev]
prospector .
The output from prospector is attatched.
prospector_output.txt

Thanks! I'll check it out. Prospector doesn't test yaml files so adding a change to snippets.yml will not affect linting. However, it would be great if you could test your change against a docker image that was created using apt install. I haven't come across one yet.

@oc37ejuc Can you also file an issue about the prospector errors? I'd appreciate it. Thanks!

@nishakm done :)

@nishakm Is this issue available for me to work upon?

Sure! Go for it.

@nishakm Sorry for the delay, today I tried to setup the project on my local using this and it gave some error while running vagrant. The error was 'pwd': Operation not permitted - getcwd (Errno::EPERM)
I googled it and found this Stackoverflow issue
I have never used vagrant before but I think we are using some kind of script to install dependancies which needs to be modified to install gems here - /usr/local/bin as suggested in the Stackoverflow issue.
@nishakm Can you help me with what steps should I take now?

@nishakm  Can i work on this issue?
I  am new here, contributing for the first time.
Please guide me.

@nishakm Can i work on this issue?
I am new here, contributing for the first time.
Please guide me.

Hi! If you are new to contributing to any open source project, I would first ask that you set up your work environment according to https://github.com/tern-tools/tern/blob/master/CONTRIBUTING.md#setting-up-a-development-environment. Also, you can reach out to me on the slack channel. Follow instructions to get access here: https://github.com/tern-tools/tern/blob/master/CONTRIBUTING.md#slack-channel


Mention this guide requires only 1 master node

@nirvanagit  could you please ensure that the update note is in the Before you begin section.
https://deploy-preview-8376--kubernetes-io-master-staging.netlify.com/docs/setup/independent/create-cluster-kubeadm/
Right now the note appears at the end of the page.

@Rajakavitha1 : Could not find Before you begin section

@nirvanagit Before you begin section is here in this file: https://kubernetes.io/docs/setup/independent/create-cluster-kubeadm/

Please address the feedback from @Rajakavitha1. If you are using the Github UI, browse to your fork, select the patch-6 branch, and edit the file /content/en/docs/setup/independent/create-cluster-kubeadm.md. The If you have patch-6 checked out when you do that, the commit will be added to this PR automatically.

@nirvanagit Any updates?

I'll carry this.

The note is now reworded and appears in the right place.
/lgtm
/approve


Update subusers.md

Updating broken link for Teammates article
Currently pointing to a 404: https://sendgrid.com/docs/help-spport/account-and-settings/teammates/
Should be: https://sendgrid.com/docs/ui/account-and-settings/teammates/

Hello @bhammerquist,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Hello @bhammerquist,Thanks again for the PR!                                         You have earned 5 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.                                         Please take a moment to checkout this link to find more issues to get you past the required threshold or to simply continue the celebration.                                         Also, please be sure you have officially registered with us here by November 1, 2018 to qualify.                                         If you have any questions you can email us at<EMAIL_ADDRESS>                                        Thank you and Happy Hacktobering!                                         Team SendGrid


Dispose message channel when pinger detects a failure

When the message channel pinger detects a failure (like the wrong format of the "pong" message or the change of remote channel id), the message channel should be closed and disposed.
The previous code didn't do this really.

lgtm


Show feedback as a question icon on the bottom left

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Done


Vuex for better State-Management and Vue-DevTools

Having some problems with the points in the Header.
Vue-DevTools:
while debugging in chrome, vue-devtools plugin will not detect that i'm working on a vue-project.
Vuex:
After installing "vuex" and "babel-preset-stage-2" i got following error message after import e.g. { mapActions }
ERROR in ./~/vue-loader/lib/selector.js?type=script&index=0!./src/assets/vue/pages/components/add.vue
Module parse failed: C:\phonegap\mytime\node_modules\vue-loader\lib\selector.js?type=script&index=0!C:\phonegap\mytime\src\assets\vue\pages\components\add.vue Unexpected token (34:8)
You may need an appropriate loader to handle this file type.
|
|       methods: {
|         ...mapActions([        <-- Line 34
|           'addAccount'
|         ]),
@ ./src/assets/vue/pages/components/add.vue 3:2-97
@ ./src/routes.js
@ ./src/main.js
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8081 webpack/hot/dev-server ./src/main.js ./webpack/dev_helpers/CordovaDeviceRouter.js
Any suggestions?
Many thanks in advance,

Hello @SgtPepper27
For babel: https://github.com/centrual/cordova-template-framework7-vue-webpack/issues/29
For Vuex:
npm i vuex --save-dev

Create folder structure like that:

| -- src
|----|-- store
|----|----|-- index.js
|----|----|-- actions.js
|----|----|-- getters.js
|----|----|-- mutations.js

Open src/store/index.js
import Vue from 'vue'
import Vuex from 'vuex'
import * as actions from './store/actions'
import * as mutations from './store/mutations'
import * as getters from './store/getters'

Vue.use(Vuex)

export default new Vuex.Store({
  actions,
  mutations,
  getters
})

Also edit the actions, mutations and getters js files like: https://github.com/vuejs/vuex/tree/dev/examples/shopping-cart/store
Open src/main.js
// ...
import store from './store'
// ...
new Vue({
  // ...
  store
  // ...
})

and it's ready.
Have a great day! 😃

Gone through the tutorial #29, but still get the error "Unexpected token" => spread syntax still not working.
vuex syntax you postet above is not the problem, but the spread syntax "..."
...mapActions([
'addAccount'
]),

Upgrade your node.js version. Minimum node.js requirement is 6.5.0.

I already have version 7.6.0. Example worked for me as nomal vuejs project with template webpack-simple, but on this template i struggle.
in my standard vuejs project my .babelrc file looks like that:
{
"presets": [
["es2015", { "modules": false }],
["stage-2"]
]
}
and that´s it to use spread syntax.

Spread syntax about ES6, not about this template. Please check one more time:
import { mapState, mapActions, mapMutations, mapGetters } from 'vuex'

must be added into your file and babel must be configured right.
Also it can be about babel preset. es2015 preset description: Only compiles ES2015 to ES5. Spread is ES6 syntax. Please try with env preset too.
Also check is your presets installed with npm ls --depth=0. If not you can install with this command:
npm install --save-dev babel-loader babel-plugin-transform-runtime babel-preset-stage-2 babel-preset-es2015

or for env:
npm install --save-dev babel-loader babel-plugin-transform-runtime babel-preset-env

So there is so much possibilities, but the template is not inside these possibilities for this error.

Now it work´s - Thanks. Following settings worked for me:
.babelrc:
{
"presets": [
["stage-2"],
["env"]
]
}
webpack.config.js:
module-->rules-->
{
test: /.js$/,
exclude: /(node_modules|bower_components)/,
use: {
loader: 'babel-loader',
options: {
presets: ['env', 'stage-2'],
plugins: ['transform-runtime']
}
}
},

in your webpack add the following
module: {
rules: [
...
...,
{
test: /.js$/,
loader: 'babel-loader'
}


Search

Hi. Thanks for the open source ). This is exactly what is needed to speed up the site.
I wanted to clarify. Why is the search done on the "value" field and not on the "label" field?
And the second. Parameter "closeOnSelect" really doesn't work.

Hi @ARA1985, this library supports react node as label and it will be too complex to search in a react node, also most cases values are the same as label and lowercase so the search feature depends on the value.
I have a target to revamp the library and release a major version with fixes for all issues.


Entity properties are tracked and don't need to call DbSet<>.Update(entity)

In UpdateCustomerCommandHandler.Handle, when you get an entity from DbSet and change the property values of the entity, then you don't need to call
_context.Customers.Update(entity);
Update() invalidates all the fields.
From: https://docs.microsoft.com/en-us/dotnet/api/microsoft.entityframeworkcore.dbset-1.update?view=efcore-2.1

All properties of the entity will be marked as modified.

Just calling await _context.SaveChangesAsync(cancellationToken); is enough here.
Thanks for your example project, learned a lot.

Thanks, I had been meaning to fix this one but kept forgetting! Much appreciated.


docs: use correct reference to collections module in docs

The "collections" module was moved to the utilities package. This needs to be reflected in the documentation.
Resolves #536
Resolves #540

:tada: This PR is included in version 0.45.4 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.97.4 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.0.2-rc.1 :tada:
The release is available on npm package (@v1-rc dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Update README.md

Added Leetcode link to the competitive coding part.

Please add the hacktoberfest-accepted label if you wish to merge the PR


meta: expose availability flags in derivation metadata

Currently it's not possible to determine the reason why a package is
unavailable without evaluating nixpkgs multiple times with different
settings. eg.
nix-repl> :p android-studio.meta
{ available = false; broken = false; unfree = true; unsupported = true; ... }

The following snippet is an example that uses this information to query
the availability information of all packages in nixpkgs, giving an
overview of all the packages currently marked as broken, etc.
{ pkgs }:

with import <nixpkgs/lib>;

let

  mapPkgs =
    let mapPkgs' = path: f: mapAttrs (n: v:
      let result = builtins.tryEval (v ? meta); in
      if !result.success then {} else
      if isDerivation v then f (path ++ [n]) v else
      if isAttrs v && v.recurseForDerivations or false then mapPkgs' (path ++ [n]) f v else
      {}
    );
    in mapPkgs' [];

  getMeta = path: drv:
    if drv.meta ? available then
      let meta = {
        pkg = concatStringsSep "." path;
        inherit (drv.meta) broken unfree unsupported insecure;
      };
      in builtins.trace meta.pkg meta
      else {};

  metaToList = attrs: flatten (map (v: if v ? pkg then v else metaToList v) (attrValues attrs));

in metaToList (mapPkgs getMeta pkgs)

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I totally forgot about this. Should be pretty safe tho.


Split ohno-tests into multiple jobs

Jobs can run in parallel so more braincells can be lost faster

You know I wont fix it, instead im just gonna change all the apt stuff to various github actions as they run significantly faster


fix(TDI-39619): add logic to exclude tablename if necessary

What is the current behavior? (You can also link to an open issue here)
What is the new behavior?
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[x] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:
https://jira.talendforge.org/browse/TDI-39619

Pull request test result: success :star:
Sonar results

@wwang-talend thank you for the review. I reproduce this situation.
Create custom filed in Account table and querying:
"SELECT Account.Account__c FROM Account"
"SELECT Account__c FROM Account"
Both works well for me.


Fix broken link to natural-language rubygems page

Coverage decreased (-0.02%) to 94.072% when pulling 7d12dc75d729fba6349a81acaa5d8a3476ce3a9f on fix-broken-nl-rubygems-link into 89494ec65a6a274096e5cb53e926a0afdc157957 on master.


Dont run version check on main

Why
Because CI fails on the main branch builds
What was changed
CI pipeline was changed to not run the version bump check

Codecov Report

Merging #38 (99cbb7c) into main (d1f825e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #38   +/-   ##
=======================================
  Coverage   98.64%   98.64%
=======================================
  Files          11       11
  Lines         515      515
=======================================
  Hits          508      508
  Misses          7        7

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix StellarWalletAddress

This is not called in a context where it would crash yet. So there's no problem.

skipping ci


Missing entries in the nodes tab

Hi,
I don't know if this is the right place to report my problem because, although the starting point is the behavior of the Puppetboard "nodes" view, it would be more of a Puppet (or PuppetDB) related problem or a misconfiguration on my part.
Context

Puppetboard version: 4.3.0
Puppet version: 6.27.0

I have created an environment for testing purposes called "tests". I have assigned some nodes to this environment (using an ENC). They used to be in "production" environment. I let a few days pass for the nodes to update.
Observed behavior
I select the "tests" environment and go to the "nodes" tab.
Only one node appears.
However, when I visit the "reports" tab, I see that many nodes have come forward and made their updates.
Similarly, the "inventory" tab does contain hosts.
Expected behavior
I assumed that nodes that have applied updates due to a change in environment (presence of changed reports) should appear in the nodes tab of the considered environment.
However
Digging a little deeper, I notice that the "catalog_environment" property of the missing nodes is still set as "production", while the node that appears has "tests". This line seems to confirm that this property determines the presence of the node in the "nodes" tab for the selected environment.
What justifies this choice? Indeed, both "facts_environment" and "report_environment" have the value "tests".
Should I suspect a problem on my devices, which despite several "unchanged" reports, still have a "production" catalog? I am 100% sure they are up to date.
Thank you in advance for your answers,

Hi @cyberm8!
Sorry for a late reply.
Puppet < 6 has a bug a that causes facts_environment to be set to a wrong value in some cases, so we can't use that.
But perhaps it would make sense to switch to use the report_environment... 🤔


Add support for Intuity Mediccal POGO meter (UPLOAD-1223)

Addresses UPLOAD-1223.

Still WIP, and based against in-browser-merged, which is why it's in draft.

Closing this as won't do. See linked ticket for further details.


Changed log. to tflog. package

KNTK - 262
It looks like a terrafrom core look up for TF_LOG environment variable by default, so I only changed the current way of logging to use tflog package.

If TF_LOG environment variable will be set, I think yes.

Is this really all? Will it make all messages related to interactions with Kentik API appear in logs?

If TF_LOG environment variable will be set, I think yes.

Can you, please, show me example debug output? I'd like to see what information we can provide and how are API requests and responses represented in the output.

Here https://pastebin.com/mjzcvf6E You can find debug output for example from kentik-synthetic_test. TF_LOG were set to TRACE.

Thanks for the debug output. I'll approve and merge this PR. However, in order to close ticker KNTK-262 we need to add logging of API requests and responses (at debug level) to the provider.


docs: add compatibility overview

First of all: Thank you for this awesome lib! =)
I have a small improvement request for the docs: It would be super helpful to have a overview about which version of semver is compatible with the respective nx version
I would also be happy to contribute this, I just need some time to dig through the changelog

Good idea @mikelgo

I started to collect the data

@edbzn PR created.
However I need some help there

Close as completed, thank you @mikelgo!


B 3369: pass via STDIN the $TEMPLATE variable when

prefixed with 'STDIN' keyword. for example
VM_HOOK = [
    name      = "net_fw_hook",
    on        = "CUSTOM",
    state     = "ACTIVE",
    lcm_state = "HOTPLUG_NIC",
    command   = "net_fw_hook",
    remote    = "YES",
    arguments = "STDIN $TEMPLATE" ]

Without the keyword current workflow is executed
Signed-off-by: Anton Todorov<EMAIL_ADDRESS>

Hello @atodorov-storpool,
Currently we are redesigning the hook systems, we will take care of that in the new design.


Title: celsius to fahrenheit convertor

@swapnilsparsh
Is your feature request related to a problem? Please describe.
we can add celsius to fahrenheit convertor
Describe the solution you'd like.
we take celsius from user and convert into fahrenheit
Describe alternatives you've considered.
we can add some gui and take 1 input from user
after performing celsius to fahrenheit convertion we can display result
Add any other context or screenshots about the feature request here.

@dhruv8433 Thank you for providing your insight but it is not required as of now


Add note about GetWriteToken() for automated builds

It wasn't trivial the reason for 403 error, might save others the headache ...

Good idea. I also added a comment to the Godocs here:
https://github.com/CenturyLinkLabs/docker-reg-client/blob/master/registry/doc.go#L48-L51


poor performance of page.dispose() call

I'm trying to use this library to identify text in a picture, all fine, but unacceptable performance.
At begining, I got an error message: "Only one image can be processed at once. Please make sure you dispose of the page once your finished with it", like it said I need to destroy page object before processing next one, then I did it, but the performance became very poor:

engine.process(....) only costs 0.021 seconds
page.dispose() cost about 2 seconds...

how page.dispose() cost so much time? my picture isn't bigger than 100k.

I don't think the image size has anything to do is the duration of the disposal of the object. Instead of manually disposing, which I'm sure it implements IDisposable which allows you to use the Using feature, just create a new instance each time. After all, .NET is managed code, which the garbage collector will take care of your memory management.

Agree with Jonathan here. I can confirm that the page object does indeed
implement IDisposable and the using pattern should be used.
My guess is that your not disposing the other resources when you're
finished (pix, engine etc). Leaving the system in an undefined state.
Recommended having a look at the example repo. Would also be a good idea to
try running tesseract exe on an example image to double check tesseract
itself doesn't have any issues with it.
For reference disposing the page object will call TessBaseApiClear to
effectively reset/clear the engine's state so it can process the next
page/image. The following goes into quite a bit of detail if you're
interested
https://stackoverflow.com/questions/51069618/the-semantics-of-tessbaseapiclear
On Sun, 2 May 2021, 19:55 Jonathan Dahan, @.***> wrote:

I don't think the image size has anything to do is the duration of the
disposal of the object. Instead of manually disposing, which I'm sure it
implements IDisposable which allows you to use the Using feature, just
create a new instance each time. After all, .NET is managed code, which the
garbage collector will take care of your memory management.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract/issues/550#issuecomment-830781616,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB7HSGNC77HWS34GLMVQWTTLUOPNANCNFSM425IHE5Q
.


Abs data

Hi there, would it be possible to use this for abs data etc to use when off roading??

Hi @Dieseldog57,
The ABS control unit has its own interface, and is (currently?) not implemented in the application.

Personally, I'm using the version at https://github.com/Td5OpenDiag/Td5OpenDiag-android whenever I have the opportunity to drive my Defender. I use it as a more complete and accurate dashboard (on the version I have, I developed a display to show engine coolant temperature with ranges, RPMs, speed, battery voltage, turbo pressure, and calculated instant fuel consumption.

Have a good day :call_me_hand:


Dumps of Disc games always crash the kernel.

It seems like Window's Data Execution Prevention doesn't like how Obliteration handles Disc based games, or perhaps another underlying cause.
In the GUI, it'll just exit with the textbox: The Kernel Crashed
In the kernel, VSCode returns the exception: Exception 0xc0000005 encountered at address 0x1a0fa42907d: User-mode data execution prevention (DEP) violation at location 0x1a0fa42907d
Persona5Rlog.txt
; Symbol: no symbol info
; Source: unknown
1A0FA42907D: 41 5B                      pop    r11
^ Exception 0xc0000005 encountered at address 0x1a0fa42907d: User-mode data execution prevention (DEP) violation at location 0x1a0fa42907d
1A0FA42907F: 5A                         pop    rdx
1A0FA429080: 48 85 C0                   test   rax, rax
1A0FA429083: 75 0A                      jne    0x1a0fa42908f
1A0FA429085: 48 8B 43 50                mov    rax, qword ptr [rbx + 0x50]
1A0FA429089: 48 8B 53 58                mov    rdx, qword ptr [rbx + 0x58]
1A0FA42908D: EB 04                      jmp    0x1a0fa429093
1A0FA42908F: 49 83 CB 01                or     r11, 0x1
1A0FA429093: 41 5C                      pop    r12
1A0FA429095: 5B                         pop    rbx
1A0FA429096: 5E                         pop    rsi
1A0FA429097: 5F                         pop    rdi
1A0FA429098: 48 B9 9C 1D 39 FA A0 01 >  movabs rcx, 0x1a0fa391d9c
1A0FA4290A2: 4D 31 C0                   xor    r8, r8
1A0FA4290A5: 4D 31 C9                   xor    r9, r9
1A0FA4290A8: 4D 31 D2                   xor    r10, r10
1A0FA4290AB: 45 89 DB                   mov    r11d, r11d
1A0FA4290AE: 41 53                      push   r11
1A0FA4290B0: 9D                         popfq
1A0FA4290B1: C9                         leave
1A0FA4290B2: E9 E5 8C F6 FF             jmp    0x1a0fa391d9c
1A0FA4290B7: 00 00                      add    byte ptr [rax], al
1A0FA4290B9: 00 00                      add    byte ptr [rax], al
1A0FA4290BB: 00 00                      add    byte ptr [rax], al
1A0FA4290BD: 00 00                      add    byte ptr [rax], al
1A0FA4290BF: 00 55 48                   add    byte ptr [rbp + 0x48], dl
1A0FA4290C2: 89 E5                      mov    ebp, esp
1A0FA4290C4: F8                         clc
1A0FA4290C5: 9C                         pushfq
1A0FA4290C6: 41 5B                      pop    r11
1A0FA4290C8: 48 83 E4 F0                and    rsp, -0x10
1A0FA4290CC: 48 83 EC 60                sub    rsp, 0x60
1A0FA4290D0: 48 89 E0                   mov    rax, rsp

Strange. It should not make any different with the current version because the execution is not reached the game code yet.

Apparently, it's handled in some unsafe way if Windows is preventing it.

Strange. It should not make any different with the current version because the execution is not reached the game code yet.

And seeing as I just upgraded my CPU from a 2700X to a 3700X on my 2nd computer, I went to test and... Yeah, same error.

Weird, I tried Sonic Mania - i've got only missing syscall 340, same thing with some applications like file manager.
Maybe - game depending thing?

Weird, I tried Sonic Mania - i've got only missing syscall 340, same thing with some applications like file manager. Maybe - game depending thing?

Is that a disc dump or a PKG dump?

Is that a disc dump or a PKG dump?

PKG, i've got what you're mean.
 But then I will create PKG. And what will happen if I unpack PKG to folders again - it will be PKG DUMP?

Still happening on new logger

Seeing as my new method worked for Tom Clancy The Division...

I'm going to try dumping P5R with the same method I applied to The Division. @ultimaweapon If this works, I'll close the issue and chalk it up to a bad dump.


Add pgAdmin to MIW Helm chart

Add "pgAdmin" to the MIW Helm chart.
Adds access to the underlying database.
In case of data restore (when Postgresql DB backup is enabled) or just data insights.
Should be optional and disabled by default.

Is done


Allow multiple logins of the same user

Problem
Logging in a single user in multiple browsers does not work.
Solution
It seems that there is only one valid session per user. => https://github.com/t2v/play2-auth#stateless-vs-stateful-implementation

[x] broccoli.auth.session.allowMultiLogin (default true): set to false if you want to use the stateful implementation which logs you out when you login on another machine

I cannot logout :(


macOS "The terminal process "/bin/zsh '-l'" terminated with exit code: 1."

VS Code Version:
Version: 1.54.1
Commit: f30a9b73e8ffc278e71575118b6bf568f04587c8
Date: 2021-03-04T22:42:18.719Z (1 day ago)
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.2.0

OS Version:
macOS BigSur (11.1)

Steps to Reproduce:

When I press to open the folder to my new compiled file

After I press the blue folder shown in the picture above, I get an error message saying "The terminal process "/bin/zsh '-l'" terminated with exit code: 1."

Does this issue occur when all extensions are disabled?: Yes

It looks like your report is missing some important information, See this wiki page for more info on how to report actionable terminal issues.
/label "needs more info" integrated-terminal

Ah, sorry did see that.
Here's some from the log file when I press the button.
[2021-03-06 05:07:36.869] [renderer1] [trace] terminalInstance#ctor (id: 25) {"name":"reveal","ignoreConfigurationCwd":true,"env":{"ELECTRON_RUN_AS_NODE":"1","TERM_PROGRAM":"Apple_Terminal","SHELL":"/bin/zsh","TERM":"xterm-256color","TMPDIR":"/var/folders/6/hrm79gmj70z1fgflpnlpt49c0000gn/T/","TERM_PROGRAM_VERSION":"440","TERM_SESSION_ID":"F9CEAA0E-A79B-4613-82B4-0FD0250B25B9","USER":"dariber","SSH_AUTH_SOCK":"/private/tmp/com.apple.launchd.Y9ChKeSPXZ/Listeners","PATH":"/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin","__CFBundleIdentifier":"com.apple.Terminal","PWD":"/Applications/Developent","XPC_FLAGS":"0x0","XPC_SERVICE_NAME":"0","SHLVL":"2","HOME":"/Users/dariber","LOGNAME":"dariber","LC_CTYPE":"UTF-8","":"/Applications/Developent/Visual Studio Code.app/Contents/MacOS/Electron","__CF_USER_TEXT_ENCODING":"0x1F5:0x0:0x2","APPLICATION_INSIGHTS_NO_DIAGNOSTIC_CHANNEL":"true","VSCODE_CLI":"1","ELECTRON_NO_ATTACH_CONSOLE":"1","ORIGINAL_XDG_CURRENT_DESKTOP":"undefined","VSCODE_NLS_CONFIG":"{"locale":"en-gb","availableLanguages":{},"_languagePackSupport":true}","VSCODE_NODE_CACHED_DATA_DIR":"/Users/dariber/Library/Application Support/Code/CachedData/f30a9b73e8ffc278e71575118b6bf568f04587c8","VSCODE_IPC_HOOK":"/Users/dariber/Library/Application Support/Code/1.54.1-main.sock","VSCODE_PID":"42484","VSCODE_AMD_ENTRYPOINT":"vs/workbench/services/extensions/node/extensionHostProcess","VSCODE_PIPE_LOGGING":"true","VSCODE_VERBOSE_LOGGING":"true","VSCODE_LOG_NATIVE":"false","VSCODE_IPC_HOOK_EXTHOST":"/var/folders/_6/hrm79gmj70z1fgflpnlpt49c0000gn/T/vscode-ipc-8ef3f828-1e81-4294-89c8-e428ae6c03f0.sock","VSCODE_HANDLES_UNCAUGHT_ERRORS":"true","VSCODE_LOG_STACK":"false","PLATFORMIO_CALLER":"vscode","LC_ALL":"en_US.UTF-8","PLATFORMIO_IDE":"2.3.0","PLATFORMIO_PATH":"/Users/dariber/.platformio/penv/bin:/Users/dariber/.platformio/penv:/Users/dariber/.platformio/python3/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin","NO_PROXY":"<IP_ADDRESS>"},"extHostTerminalId":"fbc64fb7-9db8-45d6-8591-3722334fe29c"}
[2021-03-06 05:07:37.234] [renderer1] [debug] Terminal process ready (shellProcessId: 44030)
[2021-03-06 05:07:37.605] [renderer1] [debug] Terminal process exit (id: 25) with code undefined
[2021-03-06 05:07:37.605] [renderer1] [debug] Terminal process exit (id: 25) state 5
[2021-03-06 05:07:37.606] [renderer1] [trace] terminalInstance#dispose (id: 25)


Feature request: Support creating signed commits

It doesn't seem like it's currently possible to configure the action to create signed commits.
Would it be possible to add support for this? I would've assumed that running https://github.com/crazy-max/ghaction-import-gpg with git_commit_gpgsign: true before this action would be enough to make the commits signed automatically but that doesn't seem to be working.

Looking at the API again today, it looks like you might be able to add a signature with the commit object you're creating? I'll have to look into this some more.
reference: https://docs.github.com/en/rest/git/commits?apiVersion=2022-11-28#create-a-commit

Oh I see, I wasn't aware of these nuances with GitHub, thanks for looking into this. This feature would be really useful in repositories that require PRs to have signed commits.

There is now a new sign-commits option you can enable, but there's some limitations so I'm going to leave this ticket open for now in the hopes of a better long term solution.
Currenet limitations

No custom author. This makes it harder to integrate securely with Renovate which needs to ignore a specific author (well, as much as authors are 'secure'. This is more a 'prevents accidents' feature)
PAT's can't be used to connect from one personal account to a repo in another personal account. This rules out using a machine user to get a custom author unless the repos are hosted in a Github organisation.


Error during Evaluation of the model

When I run train.py through Electricity.sh script, after training procedure I get this error:
Traceback (most recent call last):   File "/content/CoST/train.py", line 110, in <module>     out, eval_res = tasks.eval_forecasting(model, data, train_slice, valid_slice, test_slice, scaler, pred_lens, n_covariate_cols, args.max_train_length-1)   File "/content/CoST/tasks/forecasting.py", line 70, in eval_forecasting     test_pred_inv = scaler.inverse_transform(test_pred)   File "/usr/local/lib/python3.10/dist-packages/sklearn/preprocessing/_data.py", line 1034, in inverse_transform     X = check_array(   File "/usr/local/lib/python3.10/dist-packages/sklearn/utils/validation.py", line 915, in check_array     raise ValueError( ValueError: Found array with dim 4. None expected <= 2.

Please see the FAQs in the README
Q: ValueError: Found array with dim 4. StandardScaler expected <= 2.
A: Please install the appropriate package requirements as found in requirements.txt, in particular, scikit_learn==0.24.1.


Node version conflict (app/cypress)

I am using the latest Cypress in my project, which uses Node 18.x to test my components. Everything works well locally, but somehow GitHub Actions complains about node version conflicts.  I attached to this post the error message as well as my workflow.
This is my project structure
root -> {src/components, package.json}
    | --> cypress/{solutions, package.json}

Do I need to use a different node version for cypress?
Error Message
Run `npm audit` for details.
/opt/hostedtoolcache/node/18.17.0/x64/bin/npx cypress cache list
┌─────────┬───────────────────┐
│ version │ last used         │
├─────────┼───────────────────┤
│ 12.17.1 │ a few seconds ago │
└─────────┴───────────────────┘

DevTools listening on ws://<IP_ADDRESS>:43627/devtools/browser/18dea1f8-15b8-48e8-960f-399d42068f20
[2101:0730/225806.881368:ERROR:gpu_memory_buffer_support_x11.cc(44)] dri3 extension not supported.
Missing baseUrl in compilerOptions. tsconfig-paths will be skipped
An invalid configuration value was set.

Expected nodeVersion to be one of these values: "bundled", "system".

Instead the value was: "16.16.0"
Test run failed, code 1
More information might be available above
Cypress module has returned the following error message:
Could not find Cypress test run results
Error: Could not find Cypress test run results

Workflow
name: cypress-component-testing

on: push

env:
  APP_NODE_VERSION: 18.x
  CYPRESS_NODE_VERSION: 16.16.0

permissions:
  id-token: write
  contents: read

jobs:
  cypress-run:
    runs-on: ubuntu-latest
    steps:
      - name: Clone Repository
        uses: actions/checkout@v3
      - name: Setup Node
        uses: actions/setup-node@v3
        with:
          node-version: ${{ env.APP_NODE_VERSION }}
      - name: Cypress Run
        uses: cypress-io/github-action@v5
        with:
          # command: npm run cy:run
          component: true
          record: false
          working-directory: cypress

@h2oearth
Your error is:

An invalid configuration value was set.
Expected nodeVersion to be one of these values: "bundled", "system".
Instead the value was: "16.16.0"

This corresponds to the Cypress documentation References > Configuration > Node version and therefore CYPRESS_NODE_VERSION: 16.16.0 used to set nodeVersion is an incorrect usage of this configuration option.

if nodeVersion were to be set to system this would be equivalent to Node.js 16.16.0 set by the GitHub runner from https://github.com/actions/runner/blob/main/src/Misc/externals.sh
if nodeVersion were to be set to bundled for Cypress 12.17.1 this would also be equivalent to Node.js 16.16.0 from https://github.com/cypress-io/cypress/blob/develop/.node-version
nodeVersion is deprecated

Even if the usage were corrected, it would not change anything.
If you simply remove CYPRESS_NODE_VERSION: 16.16.0 from your workflow it should cause your error to go away.
See also README > Node.js > Usage.

@MikeMcC399 Thank you for your quick reply! I was not aware that CYPRESS_NODE_VERSION is a reserved env var. I tried to use a different node version (cypress) than the one set by setup-node; the reason for doing this is because the Bundled Node version in the latest cypress is 16.16.0  and wanted to use the same node version.

@MikeMcC399 Thank you! It works as expected


Drop item config not working (1.19.2)

As described in the title, item loss configuration simply does not work. At least on a dedicated hosting server. Config for item loss has been set to 'true' with the percent of loss at '100' and amount of items to be destroyed is set to a range of 1-10.  However, after a player retrieves their gravestone, no items are lost.
This occurrence is being recorded from a hosting service on Bisect hosting. It is worth noting that other configs have also been shotty on this hosting service, where sometimes the config will simply not work at all vs in singleplayer where it works just fine.
OS is Windows 10 with MC version java 1.19.2

So it works like intended in a singleplayer world? You are using the configs in the server files, and not the client screen thing correct? Have you restarted the server after the config change? There might also be an issue if the config file has invalid syntax, and then it will load default configs. If that's what's happened the game should write Loading default YIGD config due to an error reading the config file. Delete yigd.toml, and a new working config file should generate in the log.
I would assume you've looked on all of these things, but I really don't know what's happened if it works on your singleplayer world, but not on the server. The code should be the exact same on dedicated server as well as a singleplayer world. And it's packaged with the generation of the graves, so if one would fail I'd expect both to do so.
I've not yet taken a look on this due to a problem I'm having with my development environment, but I will as soon as I figure it out, or the issue is resolved somehow in any other way.
I assume you're the one who made a comment on the curseforge page too. Sorry I haven't checked on it or done anything, but as I said, I'm having troubles doing so

Currently, I wouldn't assume that it's defaulting to the original config as some of the settings that I have put in the config work seamlessly. Such setting the chance of a zombie spawning on gravestone retrieval.

Found the issue. I think I somehow made it possible remove items when items were not supposed to be able to be removed, and then items that should've been able to be removed wasn't able to be removed. I'll reopen the issue since a fix is not out to the public yet, but will close the issue as soon as that happens. Thanks for reporting in! I'll make sure all maintained versions are updated, and then I'll release the updates. Hopefully won't take too long

Found the issue. I think I somehow made it possible remove items when items were not supposed to be able to be removed, and then items that should've been able to be removed wasn't able to be removed. I'll reopen the issue since a fix is not out to the public yet, but will close the issue as soon as that happens. Thanks for reporting in! I'll make sure all maintained versions are updated, and then I'll release the updates. Hopefully won't take too long

Absolutely! and thank you so much for your help and commitment man. Same can't be said for some of the other mod makers that have glaring issues with their mods. Keep it up man.

Didn't mean to close. But thank you for all your work man. Keep it up!

So slightly later than I'd hoped, I've released a new version fixing this. Thanks for reporting it, again. Sorry for taking longer than expected


Improve UX flows and validation in expenses and invited expenses

Due to problems with error validation and the form, users can get stuck in unresolvable states of the flow.
The resolution is an aggregation of four well understood issues that have been reviewed  by engineers:

#5020
#5306
#5429
#5222
https://github.com/opencollective/opencollective/issues/5697

update - will start looking at this next week.

Should this issue also fall in the above list? We have entered it as a separate papercut.

@aminakazi The new validation strategy should make it easier to implement https://github.com/opencollective/opencollective/issues/4667. Can't guarantee it will be included in the project for this sprint, but great if you have a design for it :+1:

leave on pause until @kewitz is back.

may get some of this done between work on #5704 but priority is there for this week.

Related: https://sentry.io/organizations/open-collective/issues/3287154074/

Done :rocket:


Create CONTRIBUTING.md document

So that everybody may know that contributions are welcomed =) and they can see what are the basic rules to follow to make our life easier and contributed PRs merged.
Some of the things to mention in the doc:

[ ] Semantic releasing is in use
[ ] Unit tests are run before any commits
[ ] Code coverage check also
[ ] Linting as well
[ ] Commitizen-style commits are essential

Probably it's a good idea to resolve this along with testing investigation so that no one will give up trying to commit something :)

I think that I also will include .clang-format config file in order to promote some code formatting assistance and maybe even run that tool automatically so that all code-style will be the same.


CU-k37ppz: treasury singleton - phase 1

We will not be proceeding with this for now.

Not going to do this for now.


[Bug] Current measurements are missing since 3.9.x

Description of the bug
Since the release of 3.9.x, the current measurements (networks speed, CPU/RAM/Storage percent) are hidden.
Reverting to 3.8.2 bring them back

How to reproduce

Update to 3.9.0 or 3.9.1
Open the dashboard
Downgrade to 3.8.2
Open the dashboard

Relevant log output
No response
Info output of dashdot cli
No response
What browsers are you seeing the problem on?
Firefox, Chrome, Microsoft Edge
Where is your instance running?
Linux Server
Additional context
No response

Hey, thanks for reporting! This is intentional - I have decided, that it takes away from the generally clean look of Dashdot.
If you want to bring them back, there is a new flag DASHDOT_ALWAYS_SHOW_PERCENTAGES "true". I forgot to add it to the docs, but I have fixed that now and the docs will be live in about 30 mins. Thank you!

Here is the documentation, in case someone needs it: https://getdashdot.com/docs/config#dashdot_always_show_percentages


Provide more cross-border relations

In order to demonstrate the harvesting of linkages, there is an immediate need for more relations. Currently, there is one relation encoded in the harvester:
<https://cida-test.er.usgs.gov/chyld-pilot/id/hu/041504081604>
<https://www.opengis.net/def/hy_features/ontology/hyf/lowerCatchment>
    <https://geoconnex.ca/id/catchment/02OJ*CA>.

What other relations are we thinking here? If we are only working in the one watershed, I think this is more or less the only one that's available?

maybe other features? river segments ?

We won't be able to do anything in the immediate. Especially river segments since we have a unique-id conundrum with them still. Also, the catchment topology is the river topology, so that is actually already expressed in the one we have. The only other thing I can think is to have a topological relationship between some aquifers across the boarder really.

It's going to be hard to demonstrate utility without a representative dataset. Can we prototype a single topological relationship to see how we might handle them?

yes.. good point.  We can discuss with Julie-Ann Ishikawa

Figured out how to add a few more. "segments" is hard since we don't have incremental flowpaths per HU12 but we can do total-upstream flowpaths. Stay tuned, I think I'll have something deployed later today.

OK, we now have a few more.
Several catchments (such as this one) overlap an aquifer.
I also added a nexus between the outlet of Lake Champlain and the first catchment in Canada.


Make Binary Dictionary Operations Optional

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
The dyn_cmp_dict and dyn_arith_dict features of arrow enable binary operations involving dictionary arrays and other dictionary arrays or scalar arrays.
They are, however, extremely expensive both from a compilation time, and code size perspective. As the kernels must be generated for the combinatorial explosion of all dictionary key and value types. See https://github.com/apache/arrow-rs/issues/2596 and https://github.com/apache/arrow-rs/pull/2760.
They are also exceedingly rare in practice, as almost all queries instead use the scalar variant, i.e. add scalar value to dictionary, compare dictionary against scalar value, etc...
Describe the solution you'd like
I would like a feature flag, e.g. binary_dict_op, that is not enabled by default, and enables the arrow features. The three or so tests that happen to need this, can then be gated on this feature flag or possibly updated to not need it.
Describe alternatives you've considered
We could not do this
Additional context
The feature was initially enabled as a dev dependency in #3363 by @avantgardnerio
This was then updated as physical-expr dev dependency in #4163 by @isidentical
It was then enabled as a non-dev dependency in #4168 by @retikulum possibly unintentionally?

Hi. I added this on purpose (but without knowing it is extremely expensive) to pass test_dictionary_type_to_array_coersion test case. The following error was generated before enabling it:
Error: ArrowError(CastError("Comparing array of type Dictionary(Int32, Utf8) with array of type Dictionary(Int32, Utf8) requires \"dyn_cmp_dict\" feature"))

It seems great to me for enabling the feature flag rather than enabling it by default. Thanks for noticing me.


[QUERY] Unable to SqlClient instrumentation

Library name and version
Azure.Monitor.OpenTelemetry.AspNetCore 1.0.0
Query/Question
I just upgraded Azure.Monitor.OpenTelemetry.AspNetCore from beta5 to 1.0.0. and noticed the breaking change - classes like SqlClientInstrumentationOptions and AspNetCoreInstrumentationOptions are now internal instead of public.
While on beta, I was using these classes to setup OpenTelemetry in my ASP.NET (.NET 8) project, like this:
public static IServiceCollection AddOpenTelemetryCustomized(this IServiceCollection services, ConfigurationManager configuration)
{
	services.AddOptions<SqlClientInstrumentationOptions>().Configure(options =>
	{
		configuration.Bind("OpenTelemetry:Sql", options); // configuration for SQL statements included in traces
		options.Filter = TelemetryFilters.ShouldCollectSql; // exclude health check dummy queries
	});

	services.AddOptions<AspNetCoreInstrumentationOptions>().Configure(options =>
	{
		// exclude things like healthchecks, swagger, visual studio debug endpoints
		options.Filter = TelemetryFilters.ShouldCollectRequest;
		options.EnrichWithHttpRequest = EnrichWithHttpRequestAction;
		options.EnrichWithHttpResponse = EnrichWithHttpResponseAction;
	});

	// ApplicationInsights connection string is expected in enviroment variable
	services.AddOpenTelemetry().UseAzureMonitor().ConfigureResource(builder =>
	{
		builder.AddService(serviceName);
	});
}

Now that I can't use them, what is the clean/recommended way to configure these things (for example getting value for SetDbStatementForText from environment variable, if present, etc.).
Thanks!
Environment
.NET 8

Asp.Net Core, HttpClients were vendored in, but not anymore, so you should be able to get it working.
For SQL, you need to add explicit dependency to https://github.com/open-telemetry/opentelemetry-dotnet/tree/main/src/OpenTelemetry.Instrumentation.SqlClient and then try again, and it should work.
https://github.com/Azure/azure-sdk-for-net/tree/main/sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore#customizing-sqlclientinstrumentationoptions

@rajkumar-rangaraj the public docs is absent of this information! Need to add more content from this repo to docs for easy discovery.

Thank you @cijothomas but I am still bit confused.
I have to add package references to both OpenTelemetry.Instrumentation.AspNetCore (currently 1.7.0) and OpenTelemetry.Instrumentation.SqlClient (currently 1.6.0-beta.3) to have access to classes SqlClientInstrumentationOptions and AspNetCoreTraceInstrumentationOptions (renamed from AspNetCoreInstrumentationOptions I guess).
But if I understand this docs and code in method AddVendorInstrumentationIfPackageNotReferenced correctly, that will effectively prevent UseAzureMonitor() from adding both AspNetCore and SqlClient instrumentation, and I have to call them explicitly like this:
services.AddOpenTelemetry().UseAzureMonitor()
    .WithTracing(builder =>
    {
        builder.AddAspNetCoreInstrumentation(options =>
        {
            // exclude things like healthchecks, swagger, visual studio debug endpoints
            options.Filter = TelemetryFilters.ShouldCollectRequest;
            options.EnrichWithHttpRequest = EnrichWithHttpRequestAction;
            options.EnrichWithHttpResponse = EnrichWithHttpResponseAction;
        });

        builder.AddSqlClientInstrumentation(options =>
        {
            configuration.Bind("OpenTelemetry:Sql", options); // configuration for SQL statements included in traces
            options.Filter = TelemetryFilters.ShouldCollectSql; // exclude health check dummy queries
        });
    })
    .ConfigureResource(builder =>
    {
        builder.AddService(serviceName);
    });

So basically, the behavior is: if you want all options on defaults, UseAzureMonitor() will add instrumentations, but if you want to customize some options, you need to add instrumentations manually (instead of them being called from UseAzureMonitor and "picking up" on configured options like it was before). Is that correct?
Note: I wanted to link also source code of AddVendorInstrumentationIfPackageNotReferenced, but code here on GitHub does not match what I see when I F12 UseAzureMonitor (from Azure.Monitor.OpenTelemetry.AspNetCore 1.0.0. package in my project). Dictionary look like this, so it disables both AspNetCore and SqlClient instrumentation:
var vendorInstrumentationActions = new Dictionary<string, Action>
{
    { AspNetCoreInstrumentationPackageName, () => tracerProviderBuilder.AddAspNetCoreInstrumentation() },
    { SqlClientInstrumentationPackageName, () => tracerProviderBuilder.AddSqlClientInstrumentation() },
    {
        HttpClientInstrumentationPackageName,
            () => tracerProviderBuilder.AddHttpClientInstrumentation(o => o.FilterHttpRequestMessage = (_) =>
                {
                // Azure SDKs create their own client span before calling the service using HttpClient
                // In this case, we would see two spans corresponding to the same operation
                // 1) created by Azure SDK 2) created by HttpClient
                // To prevent this duplication we are filtering the span from HttpClient
                // as span from Azure SDK contains all relevant information needed.
                var parentActivity = Activity.Current?.Parent;
                if (parentActivity != null && parentActivity.Source.Name.Equals("Azure.Core.Http"))
                {
                    return false;
                }

                return true;
                })
    },
};

Hi @rouen-sk,
You caught us in a middle of a release cycle, so the code in the main branch of this repo doesn't exactly match some of the docs. I'm deeply sorry for that.
To find the source code for a specific release, you can browse the Releases in this repo and click on the commit id. https://github.com/Azure/azure-sdk-for-net/releases?q=AspNetCore&expanded=true
--
Regarding our versions and dependencies, let me try to explain what's happening..

While OpenTelemetry.Instrumentation.AspNetCore was in beta, we had public dependencies of all three Instrumentation libraries which were also in beta. (AspNetCore, Http, and SqlClient).

When OpenTelemetry.Instrumentation.AspNetCore shipped v1.0.0, we couldn't ship with beta dependencies, so the source code for all 3 Instrumentation libraries (AspNetCore, Http, and SqlClient) was privately vendored in our sdk. The UseAzureMonitor() applies only the default options of the Instrumentation libraries. If you need to customize either of these Instrumentation libraries you would need to manually onboard to the Instrumentation libraries to access their public option APIs. In this scenario, our library would backoff to allow your configuration to take precedence. (This statement is missing from our public docs).

I'm currently readying OpenTelemetry.Instrumentation.AspNetCore v1.1.0 which will be released in days. This will include package dependencies to the public stable AspNetCore and Http Instrumentation libraries and you'll be able to use their public options APIs without any extra effort. This won't help you for the SqlClient instrumentation which is still in beta and will still require manual onboarding to configure its options. (This statement is available now in the repo's Readme)

Please let me know if you're still confused and I can explain further.

Hi @TimothyMothra thank you for explanation. I am OK with the code above (calling UseAzureMonitor() and then also AddAspNetCoreInstrumentation and AddSqlClientInstrumentation, no big deal, I was just curious, if I am supposed to do so or I am missing something.
I still think it is unusual design decision and kinda unexpected behavior - ASP.NET itself and many 3rd party libraries which support registration to DI container via extension methods simply just have their Option class public, and you can Configure it into container and that does not mean you suddenly have to call extra service registration methods, because default one is "backing off" just because you have configured options. But I guess there must be reason for it I don't see :)


Open Graph previews not working

Open Graph Social Media previews are not working

This will not be worked on, moving on with version 8.0


Upgrade to sequelize >5.8.11 to fix vulnerability

See : https://www.npmjs.com/advisories/1018

Same issue here

Similiar to this, if you upgrade to the latest version of sequelize, this package will not work when you perform a migration. You will get something like the error below when running the migrate command
node ./node_modules/sequelize-auto-migrations/bin/makemigration.js --name tmp -x
/Users/tjohnson/Public/repos/scottsdale-event/node_modules/sequelize-auto-migrations/lib/migrate.js:452
let options = currentState[tableName].schema[df.path[2]];
                                                                ^
TypeError: Cannot read property 'undefined' of undefined
    at Object.parseDifference (/Users/tjohnson/Public/repos/scottsdale-event/node_modules/sequelize-auto-migrations/lib/migrate.js:452:65)
    at Object.<anonymous> (/Users/tjohnson/Public/repos/scottsdale-event/node_modules/sequelize-auto-migrations/bin/makemigration.js:78:23)
    at Module._compile (internal/modules/cjs/loader.js:689:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
    at Module.load (internal/modules/cjs/loader.js:599:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
    at Function.Module._load (internal/modules/cjs/loader.js:530:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:742:12)
    at startup (internal/bootstrap/node.js:283:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:743:3)
error Command failed with exit code 1.

Is there any alternative for this package?

 I found this package which is pretty nice, it basically transforms your already existing database into models.
Sadly it doesn't auto update the DB when I change the models, but at least it's way better than doing things from ground zero.

I got empty table's column in migration script when run makemigration, occurs in sequelize 5.8.11
        params: [
            "account",
            {

            },
            {}
        ]

@staadecker

@nvcken did you solve this issue ?


KAFKA-2869; Host used by Authorizer should be IP address not hostname/IP

./gradlew test passed locally

Thanks for the patch. LGTM


Improved logging

What:

fix: don't crash when jsdom's document is passed to prettyDOM.
feat(logging): don't log DOM in Cypress, don't highlight in the browser.

Why:

When passing in jsdom's document, pretty-format module crashes. (#44)
When passing in Cypress window.document to prettyDOM, it crashes because document.outerHTML.length results in Cannot read property length of undefined error. (#44)
As discussed, logging the DOM in Cypress isn't really useful, so the DOM logging has been removed in Cypress environment.

How:

Use document.documentElement in prettyDOM to have the actual element with an outerHTML property.
Refactored debugDOM function to determine the environment and how/whether to log the DOM element.

Checklist:

[x] Documentation
[x] Tests
[x] Ready to be merged
[x] Added myself to contributors table

Hey. Nice PR! Waiting on this currently. @kentcdodds yo, how soon can you merge this?
Thanks,
Faizaan.

Looks like were missing some coverage. If it's really difficult to simulate a non-browser environment...

I'm wondering, in what kind of situation would dom-testing-library be running in a pure browser environment? We've handled Cypress' case. Codesandbox runs in a mixed environment and I think it should have the same results as Node for consistency (as explain in this comment). So maybe we don't need to support a browser-only environment and just have an exception for Cypress?
I'll look into the coverage issue, don't know why line 59 isn't being covered by the tests.

Agreed. Changes pushed.

Thanks so much for your help! I've added you as a collaborator on the project. Please make sure that you review the other/MAINTAINING.md and CONTRIBUTING.md files (specifically the bit about the commit messages and the git hooks) and familiarize yourself with the code of conduct (we're using the contributor covenant). You might also want to watch the repo to be notified when someone files an issue/PR. Please continue to make PRs as you feel the need (you can make your branches directly on the repo rather than your fork if you want). Thanks! And welcome to the team :)

:tada: This PR is included in version 2.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(ibmcloud): Explicitly set HCCO controllers

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

thanks! @hasueki can you articulate in the PR desc why this change?

Yup I can definitely fill in the details! One moment, will get some testing done first.

/hold

/unhold

@enxebre @sjenning Description updated!

/approve
/lgtm

/cherry-pick release-4.12


How to define a custom flake attribute for a reusable module

I'm trying to add a custom option lib to my reusable flake module, and here's what the documentation has to say about it (emphasis added):

If your custom output attribute is a one-off because you need to do something special in a single project, all you have to do is define a value in the flake option.
However, if you want it to be reusable and integrate well, you should declare an option for it, and you could perhaps provide a bit of support logic if that makes sense to do.

However, this piece of documentation does not answer the question "how do I declare an option?", nor does it provide a link doing so. Looking through the rest of the documentation also does not provide any obvious answers.
It would be very helpful if a link that answers this question were present in this text, something like:

If your custom output attribute is a one-off because you need to do something special in a single project, all you have to do is define a value in the flake option.
However, if you want it to be reusable and integrate well, you should declare an option for it, and you could perhaps provide a bit of support logic if that makes sense to do.

Or, alternatively, provide a code snippet that demonstrates how to do so.

Why close? Did you find documentation?
I'm kinda blocked as well around this topic of "how to actually do a module"

The next section in the documentation links here, which kinda shows how you can add custom perSystem attributes to a flake, but I've tried multiple different combinations and none of them worked for me. I ended up just defining my own eachSystem utility instead of using flake-parts.
As for why I closed the issue, part of me fears the backlash from attacking the hegemony of those with a deep understanding of nix who have a vested interest in not making nix easier to use for the layman in order to bolster their own power relative to everyone else (not that I squarely place the blame on them).

This shows an example on how to define a custom option that will put things under the root flake.
https://github.com/hercules-ci/flake-parts/blob/main/modules/nixosModules.nix#L14
You can use the mkSubmoduleOptions from flake-parts-lib.


revert dockerfile and only have shell script changes (apt to apt-get)

Summary of Changes
revert dockerfile and only have shell script changes (apt to apt-get) in order to not to get DEPRECATED warning on Linux. Planning to change Dockerfile to multi-stage build that satisfies the use case
Changes in shell scripts remain the same

thank you!

I'd like to ask you a question, how can I make Dockerfile multi-stage build without losing its use case for Agave? @yihau @steveluscher @alessandrod @dmakarov


[Library definitions] Add optional propTypes to React.AbstractComponentStatics

fixes #7484
add test accessing Component.propTypes
add test assigning propTypes to AbstractReactComponent
fix line numbers for other tests due to shift in lib/react.js

This is basically https://github.com/facebook/flow/pull/7486 (except I accidentally deleted the branch on origin #classic :( )
cc @jbrown215  @TrySound

Lookin' good! Thank you!

Thank you! Is there any other action I need to take here?

No, I'll handle the conflicts! Just haven't had time to merge yet.

This introduces a bunch of new errors in our codebases. I'll have to investigate them before I merge.

@jbrown215 is this still revelant?

Probably, but I still haven't looked into the new errors.

Hi @jbrown215 is this PR still planned to be worked on and merged? I'm working on a library that defines propTypes and flow types and would be really keen to use this.

Closing in favor of https://github.com/facebook/flow/pull/8476


Subnetworks list doesn't update after creation

Describe the bug
After creating a subnetwork it does not appear in the list.
To Reproduce
Steps to reproduce the behavior:

Open the file: test/testfiles/zeroline/functions.scd
Click on the Communication Editor.
Click Add Subnetwork.
Add some parameters.
Click "Add +"
The list of subnetworks is unchanged.
However if the user goes to a different editor and then returns to the Communication editor the new subnetwork is shown.

Expected behavior:
After clicking "Add +" the subnetwork list should be refreshed.
I suspect this may be a consequence of a recent refactor.
Desktop:

OS: Linux, Kubuntu 22.04
Browser: Chrome
Version: Version 102.0.5005.61 (Official Build) (64-bit)

Additional context:
None

The same problem is in the IED Editor (and our new 104 Plugin we are buildin)g.
In the IED Editor when you update the name of the IED, the change doesn't show directly, only after switching editors and back again.
Also when you initialize a DA Value in the IED Editor, that also not directly reflected. Only after collapsing the LN and expanding it again.

There is also some strange behavior in the Template Editors. All 4 type lists have the same problem.
When you edit 1 type and change the name it is displayed directly, but you aren't able to edit the same type directly again.
If you first edit a other type and then try editing the first one again it works again.

The same problem is in the IED Editor

Too bad we didn't catch this in review, the fix is already merged as is, so we'll need a second pull request. The original issue (Subnetworks list doesn't update) has been fixed by it, at least. I just confirmed this.
The remaining problem with the IED seems to be (at first glance) that you have a separate selectedIed property there in which you set an Element manually (instead of reactively) and that isn't updated when doc is. It looks like you did this because you needed to remember the currently selected IED in a global variable before the change to how we load plugins. This should no longer be the case.
You can see the symptoms of this by looking at the alphabeticOrderedIeds listed in the dropdown at the top: The list items themselves are updated as soon as you update the IED (new name immediately reflected), while anything rendered based on this.selectedIed stays the same. You only actually requestUpdate('selectedIed') onSelect, where this should be unnecessary as long as the selectedIed reference has actually changed.
I suggest adding a @query to the current mwc-list-item["selected"] and using this.selectedIEDListItem in a reactive private get selectedIED(): string with no setter, turning selectedIED into a fully reactive property, and passing the data down in a way similar to what we do in the Substation editor.
I also suggest closing this issue and creating new issues for any other problems we find with individual plugins' compatibility with the new updating scheme. What do you think?

Duplicate of https://github.com/openscd/open-scd/issues/813, fixed by https://github.com/openscd/open-scd/pull/814 .


Added support for configuration from ENV

Thank you so much for your work on this project! I love it and use it every day.
In an effort to make execution in a docker container simpler and easier, I've added support for configuration via environment variables that can be set by a container orchestrator often more conveniently than command-line options can be set. My current docker container for this app hacks environment support on top of the current release, but I thought adding it to the source would be cleaner, and make it simpler to define a dockerfile for.
I'd love to go ahead and define a Docker container for this project as well, but I wanted to offer this change up first to get your feedback before I start on the dockerfile.

If the environment variables are evaluated in no_config_callback and not in the Click context then type enforcement is lost and need to be manually checked and inserted into the Click context. Is this what you'd prefer to the Click defaults? For reference the lambdas are what the Click docs recommend. I tried using Click's envvar kwarg, but learned that it isn't supported for this mix of arguments and options.

Can't you use the type_cast_value function for casting the environment values?https://github.com/Cronocide/gamdl/blob/733a4bb2c61d9ce06dd186a65ab6c33178aaeb43/gamdl/cli.py#L43

Like this? Env variables can configure the script in conjunction with a config file or CLI parameters.

Yes 👍 that should do the job


Do not retry after a rejected message

Fixes regression from #1251

Codecov Report

Merging #1278 (7500330) into main (5f9afe2) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1278   +/-   ##
=======================================
  Coverage   99.97%   99.98%
=======================================
  Files         308      308
  Lines       20698    20698
=======================================
+ Hits        20692    20694    +2
+ Misses          5        3    -2
  Partials        1        1

Impacted Files
Coverage Δ

internal/events/aggregator.go
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Unable to compile pyinstaller

Alle the times I try to run the setup script end up with this error:
Building wheels for collected packages: pyinstaller
  Building wheel for pyinstaller (PEP 517) ... error
  Complete output from command /usr/bin/python2.7 /usr/local/lib/python2.7/dist-packages/pip/_vendor/pep517/_in_process.py build_wheel /tmp/tmppUVywX:
  running bdist_wheel
  running build
  running build_bootloader
  No precompiled bootloader found. Trying to compile it for you ...
  Setting top to                           : /tmp/pip-install-HSQsFw/pyinstaller/bootloader
  Setting out to                           : /tmp/pip-install-HSQsFw/pyinstaller/bootloader/build
  Python Version                           : 2.7.13 (default, Sep 26 2018, 18:42:22) [GCC 6.3.0 20170516]
  Checking for 'gcc' (C compiler)          : not found
  Checking for 'clang' (C compiler)        : not found
  Checking for 'icc' (C compiler)          : not found
  could not configure a C compiler!
  (complete log in /tmp/pip-install-HSQsFw/pyinstaller/bootloader/build/config.log)
  ERROR: Failed compiling the bootloader. Please compile manually and rerun setup.py

  ----------------------------------------
  Failed building wheel for pyinstaller
  Running setup.py clean for pyinstaller
Failed to build pyinstaller
Could not build wheels for pyinstaller which use PEP 517 and cannot be installed directly
root@DietPi:/opt#

Edit: grammar

Fixed. The error was caused by the compiler because I haven't installed zlib


All classes have logger messages for now

Coverage decreased (-0.09%) to 25.442% when pulling 64e2079f741a7a8527431852abd8a20cca89382b on logging into 1b7d6cd1d1380ad70ed7b3cf99d61a651f4d2fce on master.


chore(docs): pin sphinx-tabs until we figure out dark mode

The issue on the sphinx tab that tracks this is https://github.com/executablebooks/sphinx-tabs/issues/152


Project Properties Updated

-added address fields present in v1.8.0 of the PlanGrid API

This change is

@BRyeGmoney looks good, thanks for updating this! I just have one comment.

@JuliusAlexanderIV @kswoll no problemo. i imagine you guys are doing the squashing as part of the merge?

@BRyeGmoney thanks for the PR, I just merged it into the master branch so it's now available on Nuget!


Updated sensor for 0.92

As this will break older (pre 0.92) in loosing the dependency I decided against merging this in right now. I took the liberty however and making your changes directly in the :dev branch and attributing you there instead so that it can be in the next release (see 419353c)

I missed that there is a dev branch now 😀

I missed that there is a dev branch now 😀

Yeah; I learned from my mistakes the last time I broke stuff for everyone.  Then next time I'll make a PR on dev.


Pin dependencies

Fix #33

@jmelot
This will interest you: https://pypi.org/project/bs4/
That bs4 dependency was actually a typosquatting prevention package that was a redirect to protect beautiful soup users.

@jmelot:
This will interest you: https://pypi.org/project/bs4/
That bs4 dependency was actually a typosquatting prevention package that was a redirect to protect beautiful soup users. That was why that dependency was at 0.0.1, which doesn't make sense given that beautiful soup has been around for many years.
Apropos: https://news.ycombinator.com/item?id=24649729

WHOOPS. That's pretty horrifying, glad you fixed it! Nice article.


WIP: Fix issue - 2110 Adding prometheus metrics

Added goa-middleware to record metric for WIT requests count with (entity, action) labels.
Fix issue: https://github.com/openshiftio/openshift.io/issues/2110

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1903      +/-   ##
==========================================
- Coverage   63.63%   63.48%   -0.15%
==========================================
  Files         144      146       +2
  Lines       13492    13451      -41
==========================================
- Hits         8585     8540      -45
- Misses       4086     4098      +12
+ Partials      821      813       -8

Impacted Files
Coverage Δ

remoteworkitem/jira.go
75% <0%> (-25%)
:arrow_down:

remoteworkitem/scheduler.go
53.65% <0%> (-7.32%)
:arrow_down:

kubernetes/deployments_kubeclient.go
52.52% <0%> (-1.72%)
:arrow_down:

metric/recorder.go
57.14% <0%> (ø)

metric/manager.go
63.15% <0%> (ø)

controller/workitem.go
82.56% <0%> (+0.72%)
:arrow_up:

workitem/expression_compiler.go
79.83% <0%> (+1.05%)
:arrow_up:

workitem/table_join.go
100% <0%> (+10.63%)
:arrow_up:

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 82828f8...c257093. Read the comment docs.


bug: apisix/dashboard don't support array type for the schema in plugins

Issue description
I use array type for schema of the plugn,but i can't edit it in apisix/dashboard
" ip-restriction" pugin has the same problem, it also used array type
whitelist = {             type = "array",             items = {type = "string", anyOf = core.schema.ip_def},             minItems = 1         },         blacklist = {             type = "array",             items = {type = "string", anyOf = core.schema.ip_def},             minItems = 1         }
eg:

Environment
0.8

apisix version (cmd: apisix version):
OS:

Minimal test code / Steps to reproduce the issue
your steps
step1:

the link :  http://apisix.iresty.com/apisix/dashboard/#/schema/routes/list
step2:
edit the rule ,and add  a "ip-restriction" plugin , then you can reproduce the issue

ip-restriction is in the dashboard todo list, welcome PR.

ip-restriction is in the dashboard todo list, welcome PR.

thinks

@moonming @dequanLi  How about this?

Request
{"uri":"i.i","plugins":{"ip-restriction":{"blacklist":["222"],"whitelist":["111"]}},"desc":"test"}

Response
{"error_msg":"failed to check the configuration of plugin ip-restriction err: invalid \"oneOf\" in docuement at pointer \"#\""}

just for sure, does oneOf mean the request will only POST either whitelist or blacklist?

just for sure, does oneOf mean the request will only POST either whitelist or blacklist?

yes

The JSON format for current plugins
grpc tracecode
{
    "proto_id": "1",
    "service": "helloworld.Greeter",
    "method": "SayHello"
}

jwt-auth:
{
    "key": "user-key",
    "secret": "my-secret-key"
}

key-auth
{
    "key": "auth-one"
}

limit-count
{
    "count": 2,
    "time_window": 60,
    "rejected_code": 503,
    "key": "remote_addr"
}

limit-conn
{
    "conn": 100,
    "burst": 50,
    "default_conn_delay": 0.1,
    "rejected_code": 503,
    "key": "remote_addr"
}

limit-req
{
    "rate": 4,
    "burst": 2,
    "rejected_code": 503,
    "key": "remote_addr"
}

openid-connect
{
    "client_id": "kbyuFDidLLm280LIwVFiazOqjO3ty8KH",
    "client_secret": "XXXXXX-XXXXXX-XXXXXX-XXXXXX-XXXXXX-STRINg",
    "discovery": "https://samples.auth0.com/.well-known/openid-configuration",
    "redirect_uri": "https://iresty.com",
    "ssl_verify": false,
    "timeout": 10,
    "scope": "apisix"
}

server_less
{
    "functions" : ["do return true end"]
}

zipkin
{
    "endpoint": "http://<IP_ADDRESS>:1982/mock_zipkin",
    "sample_ratio": 1
}

limit-count-redis
{
    "count": 2,
    "time_window": 60,
    "rejected_code": 503,
    "key": "remote_addr",
    "policy": "redis",
    "redis": {                      -- 是固定项，只不过用一层配置包裹了一下
        "host": "<IP_ADDRESS>",        -- 我稍后把他修改成平铺结构，去掉这个不必要的 table
        "port": 6379,
        "timeout": 1001
    }
}

IP 黑白名单
whitelist = [           -- 二选一的数组
    "<IP_ADDRESS>/24",
    "<IP_ADDRESS>/16"
]

blacklist = [
    "<IP_ADDRESS>/24",
    "<IP_ADDRESS>/16"
]

proxy-rewrite
{
    "uri": "/uri/plugin_proxy_rewrite",
    "host": "foo.com",
    "headers": {
        "X-Api-Engine1": "apisix",    -- key 不是固定的，行数也不固定
        "X-Api-Engine2": "apisix",
        "X-Api-Engine3": "apisix"
    }
}

It will be like this.

It will be like this.

How to add new ones? We need an 'add' button.

create succeed :) Dashboard will be updated today.

Cool, nice job!


LSP_MusicGetPos always returns 0

I call jsr LSP_MusicGetPos once per frame (not inside ad interrupt), I would expect the lower word of d0 to report some number after jsr is done but I always get 0.
Of course when converting I added -getpos at the end of the command as argument and I see the following message after conversion : GetPos enabled (0 bytes)
This is how I am launching the player
  ;Init LSP and start replay using easy CIA toolbox
	lea		            LSPMusic,a0
	lea		            LSPBank,a1
	suba.l	          a2,a2			; suppose VBR=0 ( A500 )
	moveq	            #0,d0			; suppose PAL machine
	bsr.w		          LSP_MusicDriver_CIA_Start

Am I doing something wrong?

What LSP mode are you using? getpos is only supported in "standard" mode ( ie no "insane" nor new "micro" mode ). Did you double check the you're using "-getpos" command when building the data? ( and use the right .lsmusic file ).
If it persists, could you send the .mod file so I can have a look?

this is how i use lsp, dont know if it's the correct way
https://github.com/Ozzyboshi/tunnel/blob/6055c3e4eac49ab1db4dbfa60d704d12d9f43ea8/tunnel.s#L343
the modfile is this
https://github.com/Ozzyboshi/tunnel/blob/lsp/music/ozzyvirgil3.mod
yes i used -getpos when launching lspconvert, i remember this resulted in 4 extra bytes inside lsmusic file or something similar.
I saw you added the micro mode, i will try also this since it's meant to be a 4k prod, if you can help with compression i would be very grateful

but your mod file has only 1 position in the sequence. So yeah getpos will always returns 0. If you add a second position in the mod sequence you'll get 0 then 1 then 0 again when music loop. ( get pos returns the position in the current mod sequence )

ok , let's start to define what is a position in the sequence, for me is the row of the pattern.
this piece of data would be helpful in case you want to sync your demo to the music.
For what I understand this is not what getpos reports, right? Is it the pattern number?
Anyway, in order to sync I added this instruction to the replayer:
https://github.com/Ozzyboshi/tunnel/blob/6055c3e4eac49ab1db4dbfa60d704d12d9f43ea8/lsp.s#L180
this will write a bit at a particular mem location whenever the sample start playing, according to the channel in use.
It's up to my demo to reset the bit after it has been used.
In simple case scenarios it works, at least, it's fine for me.

thx for your suggestion, i think this issue can be closed then


Signing file and include signature

Hello,
I want to sign a PDF file with rpgp as I would do it via gpg --clearsign --output signed.pdf test.pdf. I looked into the tests and tried it already but only came up with concatenating the required parts (signed message header, pdf content, signature) and writing them to the file. Unfortunately the result is not valid.
Thanks for your help.

Cleartext signatures are a mechanism that applies to text files (see https://openpgp.dev/book/signing_data.html#cleartext-signatures for some details). I don't think they can be used with a pdf?
Maybe a detached signature is what you want? (A detached signature is a small file that can be stored separately of the pdf file, and be used to verify the integrity of the pdf)

They might have been thinking about inline signatures (gpg --sign) that have the signed data and the signature in the same output file.
As you've said clearsigned binary files are not really a thing (although I guess technically it's possible to create them).

you can use either StandaloneSignature or the cleartext construction in #328


HR: no need for Firefox normalization

Firefox no longer needs box-sizing and height rules, as the browser's stylesheet now specifies the correct box-sizing value and does not affect height. Tested in Firefox ESR (v. 68) through Firefox Developer Edition (v. 76).
Paste this into Firefox's address bar to view the browser's default styles: view-source:resource://gre-resources/html.css

Completely agree with @mattbrundage!
I can confirm that the box-sizing and height rules are not required for Firefox anymore. Actually I was going to submit the same changes, but @mattbrundage was first 🙂
According to User Agent Stylesheet there is already defined box-sizing: content-box; style for Firefox.
/* <hr> noshade and color attributes are handled completely by
 * the nsHTMLHRElement attribute mapping code
 */
hr {
  display: block;
  border: 1px inset;
  margin-block-start: 0.5em;
  margin-block-end: 0.5em;
  margin-inline-start: auto;
  margin-inline-end: auto;
  color: gray;
  -moz-float-edge: margin-box;
  box-sizing: content-box;
}

hr[size="1"] {
  border-style: solid none none none;
}

Also, I can confirm that height calculates properly in Firefox now.

@necolas up


HDF5 file unable to be created/opened

I met this issue during Batch Processing Multiple Files, using both the example files as well as my own local files (in .avi and .mp4 format)
ERROR:
Unable to open/create file '/home/tierpsy_user/Tmp/MaskedVideos/cam2+ob1-1.hdf5'
Traceback (most recent call last):
File "/tierpsy-tracker/tierpsy/processing/ProcessWorker.py", line 108, in
ProcessWorker(**args, cmd_original = subprocess.list2cmdline(sys.argv))
File "/tierpsy-tracker/tierpsy/processing/ProcessWorker.py", line 38, in init
self.execAllPoints()
File "/tierpsy-tracker/tierpsy/processing/ProcessWorker.py", line 66, in execAllPoints
cmd_original = self.cmd_original)
File "/tierpsy-tracker/tierpsy/processing/trackProvenance.py", line 28, in execThisPoint
func(**argkws)
File "/tierpsy-tracker/tierpsy/analysis/compress/processVideo.py", line 142, in processVideo
compressVideo(video_file, masked_image_file, **compress_vid_param)
File "/tierpsy-tracker/tierpsy/analysis/compress/compressVideo.py", line 243, in compressVideo
save_full_interval = save_full_interval)
File "/tierpsy-tracker/tierpsy/helper/params/get_defaults.py", line 48, in compress_defaults
fps = _read_correct_fps(fname, expected_fps)
File "/tierpsy-tracker/tierpsy/helper/params/get_defaults.py", line 23, in _read_correct_fps
fps_out = read_unit_conversions(fname)[0]
File "/tierpsy-tracker/tierpsy/helper/params/read_attrs.py", line 188, in read_unit_conversions
reader = AttrReader(fname, dflt)
File "/tierpsy-tracker/tierpsy/helper/params/read_attrs.py", line 54, in init
self.field =  self._find_field()
File "/tierpsy-tracker/tierpsy/helper/params/read_attrs.py", line 59, in _find_field
with tables.File(self.file_name, 'r') as fid:
File "/home/tierpsy_user/.local/lib/python3.6/site-packages/tables/file.py", line 778, in init
self._g_new(filename, mode, **params)
File "tables/hdf5extension.pyx", line 492, in tables.hdf5extension.File._g_new
tables.exceptions.HDF5ExtError: HDF5 error back trace
File "H5F.c", line 509, in H5Fopen
unable to open file
File "H5Fint.c", line 1400, in H5F__open
unable to open file
File "H5Fint.c", line 1615, in H5F_open
unable to lock the file
File "H5FD.c", line 1640, in H5FD_lock
driver lock request failed
File "H5FDsec2.c", line 941, in H5FD_sec2_lock
unable to lock file, errno = 11, error message = 'Resource temporarily unavailable'
End of HDF5 error back trace
I have checked the temporary dir box and all other processes have been ceased to prioritise this analysing process.
Thank you very much for the help!

I'm using Windows 11. The docker installation and tierpsy.ps1 creation have been successful by far.

Hi Jing, make sure to use a 'include pattern' that matches your video types :) reading from the text you posted here (thanks for including that) it was *.hdf5, it should work for your Avi videos if you set it to *.avi.

fixed


Binary export type count fix

When exporting to binary typeCount was writing over the first tree's PersistentTypeID.
Now writes empty int to later overwrite it.

Good catch! Looks like this bug has existed from the very start, even back when the project was TypeTreeTools.


Update CI to test supported python versions

Closes: #8
This updates CI to test Python version 3.8 - 3.10

From some searching it looks like the immediate failure and CI never kicking off here means I don't have access to update the workflows.
Maybe it's something else obvious that I am missing, but I don't see anything.