New middleware structures, and how we manage

I want to renew middleware structures for v2.0.0. I'll tell you my plan roughly.
There are two types of middleware:
1. Built-in middleware
Basic. It does not depend on other libraries. Such as basic-auth, cors, etag, jsx, and so on. Included in hono packages. So, we can use hono/{name} syntax like this:
const { cors } from 'hono/cors'

2. Third-party middleware
Extended. It depends on other libraries. Such as graphql-server, mustache, or firebase-auth. These are not included in hono package but distributed by @honojs/{name} package. like this:
// Install
npm i graphql
npm i @honojs/graphql-server

// Code
const { graphqlServer } from '@honojs/graphql-server'

And, these middleware are managed in under honojs org on GitHub. Each middleware has its own repository. Such
https://github.com/honojs/graphql-server

honojs org in npm
Already, I've got honojs org in the npm repository.
https://www.npmjs.com/org/honojs
We can use @honojs/{name} namespace.
Privileges
We have to think about the privileges of managing third-party middleware.
Basically, I keep managing github.com/honojs/hono for hono package including built-in middleware.
And, the GitHub repository and npm package of third-party middleware is managed by the author of it. To do it I'll give privileges to access GitHub and npm repo to the author of the middleware.
For example. If @Code-Hex wants to make firebase-auth middleware, I will give the privileges to manage GitHub github.com/honojs/firebase-auth and npm @honojs/firebase-auth. The author is free to use the repository for development and release it to npm freely.
How to
For example:

Remove graphqh-server middleware from honojs/hono repository. And move it to honojs/graphql-server.
Publish as @honojs/graphql-server on the npm repository.
Make honojs/firebase-auth repository.
Add @Code-Hex to GitHub honojs org and add privileges to manage honojs/firebase-auth repository.
Make @honojs/firebase-auth
Add @Code-Hex to the npm repository honojs and add privileges to release and manage@honojs/firebase-auth.

It's just an idea. We have to consider it.

Or we can compose third-party middleware by monorepo pattern like the project @Code-Hex taught to me.
https://github.com/dotansimha/graphql-code-generator/tree/master/packages/plugins/typescript
In monorepo pattern, all third-party middlewares are in one repository. Such as github.com/honojs/middleware. And these will be placed under the packages directory such as packages/graphql-sever.
The nice thing about this pattern is that it allows for common test programs and packages. Instead, middleware authors will not be free to configure the project.

Thanks for a lot of writing down your thinking.
Indeed, it may be better to have separate repositories for each package if you give privileges rights to each package (This is my opinion).
Summary of Pros, Cons of management.
Non-monorepo
Pros

Ownership can be transferred in an easy way that is independent of @yusukebe -san.
Issues and pull requests are easier to organize as they are held in their own repositories.
Release flow is essentially independent of honojs framework.

Cons

May be hard to keep up with honojs framework breaking changes.
If the repository owner is not available, they need to think about how to maintain it.

Monorepo
Pros

When honojs framework is updated (or happened breaking changes), it is easy to follow the content.
Can always be reviewed by one of the maintainers involved in honojs org.

Cons

Release flows may become more difficult.
The maintainer needs to know everything.

However, CODEOWNERS can be used to assign reviewers on a directory basis.

Hi @Code-Hex ! Thank you for summarizing.
I think the monorepo pattern is good. It's easy to make settings of projects to be common. But, if we think about Deno, it will be difficult to use monorepo.
On Deno, the version is specified by the GitHub project's tag name. For example, if the latest tag is v0.1.0, Deno's modules' version will be v0.1.0. If all third-party middleware is in the monorepo, we can't make versions independent. We want to publish middleware A as v0.1.0 but the project tag may not be v0.1.0.
I don't think all modules need to be Deno modules, but this is important.
So, I'm planning to use no-monorepo pattern. In the non-monorepo pattern, it's difficult to follow the version up of Hono as you said and it's treble to set up projects for each middleware. However, I think it can't be helped. We can try and find improvements such as using Dependabot/Renovate to follow updates of Hono and making the example project for reference.
If you know of a better solution, please let me know.

Thanks @yusukebe
I see. I don't have release strategy for deno.
This is agree me.

I don't think all modules need to be Deno modules, but this is important.

I'm fine non-monorepo pattern 😄

@Code-Hex Thank you!
I'm making the template project for third-party middleware.
https://github.com/honojs/middleware-template
It's just PoC status. We might use this template after releasing v2.0.0 to develop third-party middleware.

@yusukebe Great work!

https://github.com/honojs/middleware-template

I understand. but I have one question. What do you think about the release flow for npm?
Currently, you considering releasing by using honojs org. I think we should support deployment via GitHub Actions. Because you can share npm token via secrets for an organization. I believe this is a safe way (maybe).

@Code-Hex Thank you!

What do you think about the release flow for npm?

I was just thinking about it!

I think we should support deployment via GitHub Actions.

I think so, it's a better solution. So, I'll try to set up GitHub Actions with middleware-template repository later.
I've had a quick look, and this GitHub Action looks good.
https://github.com/marketplace/actions/publish-to-npm

I think it's great to manage the third-party middleware in individual repo, which benefits,

development
it's flexible to try new techs.
maintenance
easer code review and simple CI/CD pipeline.

Going to #407


Add support for Ubuntu bash in Windows 10

#24 - any workaround with PowerShell or CMD doesn't work

+1 I feel like this might be an issue with Atom itself; it can't even see bash.exe

+1 PowerShell can't see bash.exe inside platformio-atom-ide-terminal the way it can outside.
From powershell.exe run from the Windows 10 Start Menu:
PS H:\> dir C:\Windows\System32\bash.exe

    Directory: C:\Windows\System32

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----       2016-09-13     08:54          70656 bash.exe

From platformio-atom-ide-terminal's default shell C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe:
PS H:\> dir C:\Windows\System32\bash.exe
dir : Cannot find path 'C:\Windows\System32\bash.exe' because it does not exist.
At line:1 char:1
+ dir C:\Windows\System32\bash.exe
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:\Windows\System32\bash.exe:String) [Get-ChildItem], ItemNotFoundException
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.GetChildItemCommand

+1

Hey guys, I was having the same problem... Try entering this in "Settings" > "Shell Override":
C:\\WINDOWS\\sysnative\\bash.exe

It worked for me. Got the idea from Jessica Deen on Channel 9 when she did the same for Visual Studio Code... Hope this helps.
http://www.jessicadeen.com/tech/svcc-getting-started-with-bash-on-windows-10-recap-and-channel-9-video/

Thanks, it works! I'm closing the issue. Btw. thanks for the video it seems interesting.

This works, but does not make the project directory current. Instead, the user home directory is current.


Difference from original paper: regularization term of TOP1 Loss

Hi.
Thank you for sharing this code.
I found difference from original paper in regularization term of TOP1 Loss.
According to the original paper, the regularization term should be calculated over only negative samples^1.
However, this repo calculates it over all samples^2.
It might not be significant difference, but I just pointed it out.

@bilzard Thank you for your comments!


docs: there is no kube-aws describe subcommand

I'm following the tutorial on https://coreos.com/kubernetes/docs/latest/kubernetes-on-aws.html and the command kube-aws describe is mentioned twice. However, using release 0.1.0, this subcommand does not exist:
$ kube-aws describe
Error: unknown command "describe" for "kube-aws"
Run 'kube-aws --help' for usage.

$ kube-aws --help
Manage Kubernetes clusters on AWS

Usage:
  kube-aws [command]

Available Commands:
  destroy     Destroy an existing Kubernetes cluster
  render      Render a CloudFormation template
  status      Describe an existing Kubernetes cluster
  up          Create a new Kubernetes cluster
  version     Print version information and exit
  help        Help about any command

Flags:
      --aws-debug[=false]: Log debug information from aws-sdk-go library
      --config="cluster.yaml": Location of kube-aws cluster config file
  -h, --help[=false]: help for kube-aws

Use "kube-aws [command] --help" for more information about a command.

$ kube-aws version
kube-aws version v0.1.0

I'm not sure if this command was renamed to kube-aws status or if it simply does not exist.
I'd be happy to submit a PR to fix the docs changing describe to status or to remove the mentions altogether depending on what happened.

Resolved by: https://github.com/coreos/coreos-kubernetes/pull/91


Smoke tests does not depend on build stage

Currently smoke tests rely on the Build stage to complete, that's slow and unnecessary.
Ensure Smoke tests run immediately in the pipeline (faster PR pipeline)

Codecov Report

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

@@            Coverage Diff             @@
##           master    #7819      +/-   ##
==========================================
- Coverage   59.25%   58.87%   -0.39%
==========================================
  Files         498      498
  Lines       22293    22293
  Branches     3580     3575       -5
==========================================
- Hits        13209    13124      -85
- Misses       8262     8338      +76
- Partials      822      831       +9

Impacted Files
Coverage Δ

src/client/testing/serviceRegistry.ts
48.64% <0%> (-46.85%)
:arrow_down:

src/client/testing/codeLenses/main.ts
44.44% <0%> (-33.34%)
:arrow_down:

...t/datascience/jupyter/jupyterDataRateLimitError.ts
66.66% <0%> (-33.34%)
:arrow_down:

src/client/formatters/serviceRegistry.ts
75% <0%> (-25%)
:arrow_down:

src/client/common/utils/platform.ts
64.7% <0%> (-23.53%)
:arrow_down:

src/client/common/utils/icons.ts
83.33% <0%> (-16.67%)
:arrow_down:

src/client/api.ts
78.57% <0%> (-14.29%)
:arrow_down:

src/client/datascience/cellFactory.ts
75.86% <0%> (-13.8%)
:arrow_down:

src/client/providers/providerUtilities.ts
83.33% <0%> (-5.56%)
:arrow_down:

src/client/common/platform/pathUtils.ts
66.66% <0%> (-5.56%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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


Fix grafana perf context panel id duplicate issue

Signed-off-by: Xintao<EMAIL_ADDRESS>
What problem does this PR solve?
Problem Summary: Fix grafana perf context panel id duplicate issue
What is changed and how it works?
Just change a new id
Check List
Tests

No code

Release note

Fix grafana perf context panel id duplicate bug

/rebuild


Expose passwordOpt via parameters when configuring a RedisCluster connection.

#55 added support for specifying a password when connecting to a cluster, however this option is only accessible via config. This change exposes password as a param as well.

thanks! :+1:

Released in 2.2.4

Thanks so much for the fast turnaround!


[Event Hubs] ETW fixes

Summary
The focus of these changes is to fix several mistakes related to ETW logs.  Some messages that included links to the troubleshooting guide were improperly annotated with [Event] instead of [NonEvent], causing ETW to log a warning for "too many parameters" when writing the messages.  Also corrected a parameter ordering issue for an event batch processing error, causing the structured argument parameters to be ordered differently than they appear in the message template.

/azp run net - eventhub - tests


Update JoinMarket to v0.9.5 with API support

[x] update from joininbox
[x] new install
[x] recovery

Tested OK as above.

OK will be part of v1.7.2 - closing issue.


Feature/obstacle minigame merge with main game

Fixed some bugs. SpaceGameScreen and ExtractorMinigameScreen have some extra
bugs:
-TerrainFactory map path changed
-asteroid json extra semicolon
-extractor minigame map replaced by earth game map
-adding diaologcomponent after creation of entity, which is a no no. changed to adding them in create() methods of
npc factory and enemy factory.
Unresolved bugs, wont crush the game:
-Unknown animation "stand" and "left" not added into atlas
Extra features:
-Space minigame has some extra stuff, mostly just aesthetics, no other method of exit map yet, just the exit button on top right

Extractor minigame has background with fire. No interactions yet

Added minigame popup window feature when clicking "F" key on keyboard near extractor with X on it in EarthGameArea.

@Foref you have merge conflicts to fix...

@Foref you have merge conflicts to fix...

I will reopen and check

@Foref you have merge conflicts to fix...

Merge conflict resolved


Read-only methods

How to get read-only methods from contracts?
for example from ethers docs
await daiContract.name()
// 'Dai Stablecoin'

// Get the ERC-20 token symbol (for tickers and UIs)
await daiContract.symbol()
// 'DAI'```

I guess that's what you are looking for: https://usedapp.readthedocs.io/en/latest/guide.html#reading-from-blockchain


Add function allowlist to coverage example

Enable the use of a function allow/denylist when testing locally.

Codecov Report

Merging #2830 (2477569) into main (0bc22a8) will decrease coverage by 3.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2830      +/-   ##
==========================================
- Coverage   29.93%   26.80%   -3.13%
==========================================
  Files         324      125     -199
  Lines       38991    13234   -25757
==========================================
- Hits        11671     3548    -8123
+ Misses      27320     9686   -17634

Impacted Files
Coverage Δ

src/agent/onefuzz-task/src/tasks/merge/generic.rs

src/agent/coverage-legacy/src/pe.rs

src/agent/win-util/src/handle.rs

src/agent/onefuzz-agent/src/commands.rs

src/agent/coverage-legacy/src/elf.rs

...rc/agent/onefuzz-task/src/tasks/coverage/dotnet.rs

src/agent/coverage-legacy/src/block/windows.rs

src/agent/onefuzz-task/src/local/coverage.rs

src/agent/debuggable-module/src/lib.rs

...agent/onefuzz-task/src/tasks/regression/generic.rs

... and 189 more

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


Add documentation for custom file names.

Looks good to me. // cc @vlucas.

Yep, looks good. Thanks @kayladnls ! :+1:


Issue #3168 Correction on link

I've corrected the link as mentioned on issue #3168

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@j0rt3g4 Can you please change author to Ken? Thanks!

@scanum  Done

@kenwith are you are to review and merge this content? (thank you!)

Updated with #3420


Fixed #306: Build and use static Proton library in RPM build

Would be good if Mike Cressman had a look at this once.

@ganeshmurthy You'd need this to get green build for your TLS PR. The Proton TLS library is not available in RPM-packaged Proton, it is necessary to build our own.

@ganeshmurthy Thanks, I think I will merge this. @mcressman pretty much saw this; it is not dissimilar from his .spec for the TP3. I think I did the Proton compilation slightly better than what his PR does, without requiring any patches, but that's probably just because I have newer Proton to work with. And I have the -common package that Justin suggested.


Some enhancements to take into consideration

@stickM4N Please, could you move those reviews to separate PRs, one per file? Also, it seems some changes are done by some automated system and are not required or do not follow raylib coding conventions, please, could you review it? Thanks.


[Suggestion] - jira chart values.yaml is missing schema option for Postgres

Suggestion
The helm chart includes the following configurable items for the database
database:
  type:
  url:
  driver:
  credentials:
    secretName:
    usernameSecretKey:
    passwordSecretKey:

Which, is nearly everything needed to populate the dbconfig.xml file, but, for Postgres it is missing the schema (see below block) and, possibly, because of this, the DB config has to be re-configured on first install (and install of subsequent cluster pods (see https://github.com/atlassian/data-center-helm-charts/issues/456)
<jira-database-config>
  <name>defaultDS</name>
  <delegator-name>default</delegator-name>
  <database-type>postgresaurora96</database-type>
  <schema-name>public</schema-name>

Adding schema as an option for Postgres would resolve this issue (and, if this is an option, it's not in the chart values.yaml)
Product
Jira
Code of Conduct

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

@mumbley thanks for this finding! Indeed, ATL_DB_SCHEMA_NAME env var can be added to container env, grabbing its value from values.yaml. Are you keen to raise this simple PR?

@mumbley having looked at it closer, it turns out schema is automatically set to public for postgres:
  <delegator-name>default</delegator-name>
  {% set schema_names = {
    "mssql": "dbo",
    "mysql": "",
    "mysql57": "",
    "mysql8": "",
    "oracle10g": "",
    "postgres72": "public"
  } %}
  <schema-name>{{ atl_db_schema_name | default(schema_names.get(atl_db_type, '')) }}</schema-name>

While this env is indeed missing in values.yaml this should not cause any issues, and schema should make it to dbconfig.xml.


dynamic-mappings loads env from environment, not config file

The the dynamic mappings example project doesn't actually load the env value from the config files, instead it loads from the environment variable directly. The simplest way to see this to run sbt -Denv=prod run in the terminal and note that the output This package runs in prod is not what would be expected as prod.conf sets env = production and not env = prod.

Thanks for the issue @warrenronsiek :smile:
The idea behind this example is not that you can change the application.conf mapping for the resulting package, not during sbt run. These are two very separate things.
This example only changes the mappings in Universal, but nothing inside the run command.
What are you trying to achieve? The example is pretty old and I would rather recommend shipping all configurations and start your application with a different config.resource parameter in different environments.


def find_trigger(model, data):     pass

The function to find the model's backdoor trigger is not concretely implemented. I feel that this function is the most critical part of the project. Could the author provide the source code? And I would be grateful if you do that.

Which function are you referring to?

This function is not used. So you see the placeholder "pass". Please check the main.py file where the generator produces the trigger: trigger = G(z). If this is what you are looking for.


Change URLs for image download based on new rammb server configuration

It took surprisingly long for me to notice that my desktop backgrounds had stopped updating. :)  When I finally did notice, I did some digging and found that there were some server updates:
https://rammb2.cira.colostate.edu/slider-release-notes/
Looks like they disabled HTTP-only access, and playing around with their web interface it seems like now the data comes from a slightly different path: instead of the directory yyyymmdd/, the data is now accessed in yyyy/mm/dd/.
So, with this simple patch my scripts are working again, so I thought I'd post it here too. :+1:

thank you so much :)


xlife: init at 6.7.5

Motivation for this change
There are a few different branches of xlife development. This is the 6.7 (2013) branch, documented in the development history on the homepage:

http://litwr2.atspace.eu/xlife/XLIFE-DEVELOPMENT.html

It appears to be the most recent development branch available.
Things done

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

[X] 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"
[X] 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)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

I've also added the hpndSellVariant license that this application uses. I've done the same in a previous PR: #103518

Fails to build for me.
configuring
configuring with imake: ''
imake -DUseInstalled -I/nix/store/xj92pflviva2kd0frrq6ya40pcnryhgq-imake-1.0.8/lib/X11/config
/nix/store/xj92pflviva2kd0frrq6ya40pcnryhgq-imake-1.0.8/bin/xmkmf: line 53:    13 Aborted                 (core dumped) imake $imake_defines $args

@SuperSandro2000 I couldn't reproduce this build error.
I wonder if there is an issue in the Imakefile: https://sourceforge.net/p/xlife-cal/xlife/HEAD/tree/trunk/Imakefile
The closest I could find to that error was this Red Hat Bugzilla thread: https://bugzilla.redhat.com/show_bug.cgi?id=1605031

Result of nixpkgs-review pr 103884 run on x86_64-linux 1

1 package built:

xlife

Result of nixpkgs-review pr 103884 run on x86_64-darwin 1

Builds now. No idea why it failed earlier.


Uncaught ReferenceError: moment is not defined

When I run my import 'angular-timezone-select'. I get this error:
Uncaught ReferenceError: moment is not defined
I already solved
Uncaught ReferenceError: _ is not defined
by having this before
import 'lodash'
I tried
import 'moment'
but this issue still appears.

I was injecting them incorrectly into my module.
I just need
import 'lodash'
import 'moment'
import 'angular-timezone-select'
angular.module('myapp', [
'angular-timezone-select'])

I was trying to do this
angular.module('myapp', [
'lodash',
'moment',
'angular-timezone-select'])

and getting unhelpful error messages.


Remove @types/protractor

Per my npm install...
npm WARN deprecated<EMAIL_ADDRESS>This is a stub types definition for Protractor (https://github.com/angular/protractor). Protractor provides its own type definitions, so you don't need @types/protractor installed!
I can look into it later but it may involve some other upgrades too. Anyone know if the tsLint issues that prevent people from upgrading have been resolved?

@jrgleason — Thank you<EMAIL_ADDRESS>has been removed in v5.5.0 and tslint and codelyzer have been updated to the latest versions.
Feel free to re-open this issue if you experience any other issues. Thanks again!


Improve default mpd.conf

The buffer_time option throws a warning, I'm not sure if it's my mpd version (0.23.5) that doesn't understand it or if it's valid if the pipewire output option is commented out. I suggest commenting it out in the mpd.conf template. If it's not valid at all it could certainly be removed entirely:
Jan 22 19:35:02 veery mpd[17417]: config: option 'buffer_time' on line 47 was not recognized
Jan 22 19:35:02 veery systemd[1083]: Started Music Player Daemon.

I'd like to include another fix to the mpd.conf, but I'm not sure where in the mppinit code is a good place for installing it. To fix this warning, it's sufficient to just install the timidty package - at least on Debian-ish this would work. A clumsy workaround would be to just touch an empty file there but that would lead to an apt-install prompt to the user whether they would like to overwrite the existing conf (the empty file), which is kind of ugly and misleading. So, I'm a little lost in finding out where in mppinit the mpd package is actually installed. A quick hint and I'll include it in this PR :-)
Jan 22 19:12:38 veery mpd[17020]: decoder: Decoder plugin 'wildmidi' is unavailable: configuration file does not exist: /etc/timidity/timidity.cfg

I think buffer_time is only used by a couple of MPD audio outputs (alsa and sndio). I've used it with alsa to good effect. Commenting it out may be the best solution for now. However, reducing it to 50000 for ALSA output does improve throughput synchronization with mpcplus and mppcava. See https://wiki.archlinux.org/title/ncmpcpp
I will confirm which audio outputs support this setting and think about how to enable it selectively.
I've seen the wildmidi decoder plugin message before but never figured out how to disable it. I will give it another look.

FYI, mpd is installed as a dependency during package installation. The MusicPlayerPlus MPD configuration file is copied in during mppinit if no previous configuration is detected. This happens around line 2828 in mppinit. There is some further editing of mpd.conf when syncing changes and some effort is made to preserve user customizations in sync_sys_mpd_conf() and sync_music_dir(). I do not think any changes to mppinit will be required to address the issues you raise here. We should be able to simply modify config/mpd/mpd.conf as you have done in this pull request and perhaps additionally for wildmidi config.

Try adding the following to your mpd.conf and restarting mpd to see if this avoids the wildmidi config file not found message:
decoder {
    plugin "wildmidi"
    enabled "no"
    #config_file "/etc/timidity/timidity.cfg"
}

I am going to merge this pull request as I think commenting out the buffer_time setting is best for now since that is not supported by some commonly used audio outputs. If you wish to continue discussing wildmidi decoder configuration, let's open an issue.


Created a splash scree

Issue Number
fixes #
ISSUE #52
Describe the changes you've made
Created a splash screen
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[ ] I have added my name in the contributors list at the end of README.md file.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.
[x] Open Source Program names LGMSOC

Resolve conflicts


[PE-D][Tester C] Documentation on birthdays is inaccurate.

From UG: If there is no birthday in the month provided or the birthday is past the current date in the same month, return No employees have birthdays in this month.
It's unclear what "past the current date" means - but I interpreted it as "past the time in my system's clock". This interpretation is not supported by the program, as David with a DOB of 1 Nov still shows up after typing birthday.
Perhaps remove the line entirely?

Labels: severity.Low type.FunctionalityBug
original: cbj252/ped#5

Edit parsing of birthday to obtain the current date, not month


Link Mismatch

The link format used in this page (example "http://localhost:5000/Movies/Edit/2") does not work in my tutorial implementation. I use "http://localhost:5000/Movies/Edit?id=2" notation instead.
Perhaps it will be OK to fix this in this page or am I missing something?

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

ID: f71b0095-5175-f2e8-1db2-a800e0f77ce2
Version Independent ID: 6886d687-d6bf-2652-4583-5de7b7353e45
Content: Update the generated pages in an ASP.NET Core app
Content Source: aspnetcore/tutorials/razor-pages/da1.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

You missed
Update the Edit, Details, and Delete Razor Pages to use the "{id:int}" route template.


[Security] 3.3.0 has several fixable security vulnerabilities.

Search before asking

[X] I searched in the issues and found nothing similar.

Read release policy

[X] I understand that unsupported versions don't get bug fixes. I will attempt to reproduce the issue on a supported version of Pulsar client and Pulsar broker.

Version
3.3.0
Minimal reproduce step
A security scanner will reveal several issues.
What did you expect to see?
Ideally, no security issues are reported against Pulsal (Uber Jar), or at least nothing Critical or High.
What did you see instead?
Several Major issues were reported.
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

I'm about to start the release process for Apache Pulsar 3.3.1, so all already fixed issues will be addressed with this release.
A lot of security vulnerability scanner alerts are false positives or don't apply to Apache Pulsar. It would be useful to mention what scanner you are using and what your requirements are ("Several Major issues were reported." isn't addressable).
Apache Pulsar is maintained by volunteers. Active contributions to Apache Pulsar are more than welcome for this type of issues.  Releases are typically made every 1-2 months for actively supported versions.
If you need quicker response to security vulnerabilities, I'd recommend subscribing to a commercial distribution of Apache Pulsar (I work for StreamNative which offers such services).

Contributions are welcome. Please submit separate PRs for each dependency. Sometimes a dependency update requires extra work depending on changes that may have occurred which may be unrelated to their security fix. (I work for DataStax which also offers such services.)

Indeed, I was looking for #22819 and #22826. I will validate, thanks.

It looks like JARs are not deployed in Maven Central (with rc1 or something similar), so I will have to wait for the official release. However, the POM changes look good; thanks.

It looks like JARs are not deployed in Maven Central (with rc1 or something similar), so I will have to wait for the official release. However, the POM changes look good; thanks.

The release vote email contains the URL of the staging repository, it's https://repository.apache.org/content/repositories/orgapachepulsar-1304/ for 3.3.1 release candidate 1. The version is already 3.3.1 in the jars since the staging repository will be published as 3.3.1 version if the release vote passes.

3.3.1 has been released. Closing this issue.


fix link

Following this checklist to help us incorporate your
contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed
for the change (usually before you start working on it).  Trivial changes like typos do not
require a JIRA issue.  Your pull request should address just this issue, without
pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [MNG-XXX] - Fixes bug in ApproximateQuantiles,
where you replace MNG-XXX with the appropriate JIRA issue. Best practice
is to use the JIRA issue title in the pull request title and in the first line of the
commit message.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will
be performed on your pull request automatically.
[ ] You have run the Core IT successfully.

If your pull request is about ~20 lines of code you don't need to sign an
Individual Contributor License Agreement if you are unsure
please ask on the developers list.
To make clear that you license your contribution under
the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

[ ] In any other case, please file an Apache Individual Contributor License Agreement.

thanks


On view switch: clear table before doing request for new data

When switching view using the dropdown: When it takes a long time to retrieve the data for the table of the new view it gives the impression that the Ladybug isn't working correctly as you see the data from the old view with the columns of the new view (until the data for the new view is returned).
To reproduce this add the following to the end of the buildMetadataQuery() method in the ProofOfMigrationStorage.java and switch to the Proof of migration view in the GUI:
	try {
		Thread.sleep(10000);
	} catch (InterruptedException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	}

Fixed in f6cf05abf5361a43b745f2c7148055b8e5f4534b


DIP-6 Adaptation

Test looks so far so good, so tag latest commit of this PR for a production patch:
https://github.com/darwinia-network/collator-staking-ui/releases/tag/v0.12.6-patch

Good but the votes for Darwinia is all zero now, could we do some adapt here?


Saving the code is not formatting anymore in 1.11.0/1

VSCode Version: 1.11.1
OS Version: OS X El Capitan (10.11.2)

In previous version, when I saved the file (cmnd+s) automagically fixed the formatting for me, but it is not happening anymore.
Steps to Reproduce:

Install VSCode 1.11.1 on OS X 10.11.2
Create a code without formatting

  tag {
    key     = "terraform"
    value          = true
    propagate_at_launch = true
  }

Try Saving the file (cmnd+s)

What language are you formatting? Which extensions do you have installed (code --list-extensions)?

Hey @chrmarti ,
I am using Terraform (HCL) code. Before the upgrade, I was able to save the code and it used to format everything for me automagically.
Here is the list.
🍺  /Applications/Visual Studio Code.app/Contents/Resources/app/bin$ ./code --list-extensions
JerryHong.autofilename
PeterJausovec.vscode-docker
RoscoP.ActiveFileInStatusBar
alefragnani.project-manager
alexdima.copy-relative-path
donjayamanne.githistory
donjayamanne.listFiles
eamodio.gitlens
emilast.LogFileHighlighter
felipecaputo.git-project-manager
gizak.shortcuts
groksrc.ruby
haaaad.ansible
jtlowe.vscode-icon-theme
marcostazi.VS-code-vagrantfile
mauve.terraform
misogi.ruby-rubocop
mohsen1.prettify-json
ms-vscode.sublime-keybindings
mushan.vscode-open-iterm2
naumovs.color-highlight
qinjia.view-in-browser
rebornix.Ruby
robertohuertasm.vscode-icons
secanis.jenkinsfile-support
timonwong.ansible-autocomplete
tomoki1207.pdf
waderyan.gitblame
wholroyd.HCL
wholroyd.jinja
ybaumes.highlight-trailing-white-spaces
🍺  /Applications/Visual Studio Code.app/Contents/Resources/app/bin$

Hello @chrmarti ,
I have found the cause of the issue. Somehow after the upgrade, my terraform files (*.tf) got mapped to HCL files. I had to fix this, by placing below lines in WORKSPACE SETTINGS
{
    "files.associations": {
        "*.tf": "terraform",
        "*.tfstate": "json"
    }
}

My code was formatted automatically on clicking save as I have this in my settings.json
  // Run 'terraform fmt' on save.
  "terraform.formatOnSave": true,

Closing this issue as it is not related to VSCode, apologies for the unwanted noise. I am fairly new to VSCode; switched from Sublime Text recently.
Cheers.


Installation of argocd operator stuck on status "UpgradePending" in a fresh openshift cluster

Steps to reproduce
Step 1:
Spinup a fresh 4.6 openshift cluster
Step 2:
Login to the cluster as kubeadmin
Step 3:
Apply the sealed-secrets-operator subscription through oc command
oc create -f - <<EOF
  apiVersion: operators.coreos.com/v1alpha1
  kind: Subscription
  metadata:
    labels:
      operators.coreos.com/argocd-operator.argocd: ""
    managedFields:
      -
        apiVersion: operators.coreos.com/v1alpha1
        manager: olm
        operation: Update
      -
        apiVersion: operators.coreos.com/v1alpha1
        manager: catalog
        operation: Update
    name: argocd-operator
    namespace: argocd
  spec:
    channel: alpha
    installPlanApproval: Automatic
    name: argocd-operator
    source: community-operators
    sourceNamespace: openshift-marketplace
    startingCSV: argocd-operator.v0.0.13
EOF

Step 4:
Check the operator status from the openshift console

The steps i have mentioned above works fine after one round of creation and deletion of sealed secrets operator through web console.

Even after removing the white spaces from the subscription yaml, i am hitting the same issue on a fresh cluster.
Steps to reproduce:

Create a fresh cluster (Use cluster-bot)
Fetch pr redhat-developer/kam#54
login to the cluster
create namespace cicd and argocd

$ oc create namespace cicd
$ oc create namespace argocd

Run the operator script

$ ./scripts/setup-operators.sh

Details log - https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/redhat-developer_kam/54/pull-ci-redhat-developer-kam-master-v4.5-integration-e2e/1315961750822064128/artifacts/integration-e2e/integration-e2e-steps/container-logs/test.log
NOTE: The steps i have mentioned above works fine after one round of creation and deletion of sealed, pipeline and argocd operator through web console.

Reference - https://docs.openshift.com/container-platform/4.5/operators/admin/olm-adding-operators-to-cluster.html#olm-installing-operator-from-operatorhub-using-cli_olm-adding-operators-to-a-cluster


Part of workspace folder name appears in link from autocompletion

🐛 Describe the bug
Hi, so when I use nvim-cmp to complete either an md link or wikilink with prepend_note_path enabled, it leaves an unwanted "m/". I found out that this is the from the name of the workspace folder which I set it to is named "Random".

(Minimal) obsidian and nvim-cmp config:
opts = {
    workspaces = {
        {
            name = "Random",
            path = "~/Documents/T-20/Random",
        }
    },

    detect_cwd = false,

    completion = {
        nvim_cmp = true,

        min_chars = 2,

        new_notes_location = "current_dir",

        prepend_note_id = false,
        prepend_note_path = true,
        use_path_only = false,
    },
}

return {
    "epwalsh/obsidian.nvim",
    version = "*",
    lazy = true,
    ft = "markdown",
    dependencies = {
        "nvim-lua/plenary.nvim",
    },
    config = function()
        local obsidian = require("obsidian")

        obsidian.setup(opts)
    end
}

return {
    'hrsh7th/nvim-cmp',
    dependencies = {
        "hrsh7th/cmp-nvim-lsp",
    },
    config = function()
        local cmp = require("cmp")
        local cmp_select = {behavior = cmp.SelectBehavior.Select}

        cmp.setup({
            sources = {
                {name = 'path'},
                {name = 'nvim_lsp'},
                {name = 'nvim_lua'},
                {name = "obsidian", option = opts},
                -- {name = "obsidian_new", option = opts},
            },
            mapping = cmp.mapping.preset.insert({
                ['<C-p>'] = cmp.mapping.select_prev_item(cmp_select),
                ['<C-n>'] = cmp.mapping.select_next_item(cmp_select),
                ['<C-y>'] = cmp.mapping.confirm({ select = true }),

                ["<C-b>"] = cmp.mapping(cmp.mapping.scroll_docs(-4), { "i", "c" }),
                ["<C-f>"] = cmp.mapping(cmp.mapping.scroll_docs(4), { "i", "c" }),
            }),
        })
    end
}

Versions
nvim --version
NVIM v0.9.4
Build type: RelWithDebInfo
LuaJIT 2.1.1696883897
   system vimrc file: "$VIM\sysinit.vim"
  fall-back for $VIM: "C:/Program Files (x86)/nvim/share/nvim"

lua require("obsidian").info()
[obsidian.nvim (v2.5.3)] Commit SHA: 9a6eb2afb3218b22ac0026065f4f437745a851f6
[plenary.nvim] Commit SHA: 55d9fe89e33efd26f532ef20223e5f9430c8b0c0
[nvim-cmp] Commit SHA: 538e37ba87284942c1d76ed38dd497e54e65b891
[telescope.nvim] Commit SHA: d90956833d7c27e73c621a61f20b29fdb7122709
ripgrep 13.0.0 (rev af6b6c543b)

Huh... that's odd. Can you run this from a note in your vault and tell me what's printed:
lua print(require("obsidian").get_client():vault_relative_path("~/Documents/T-20/Random/300 Other/A Note.md"))

It prints: m/300 Other/A Note.md

Is your vault behind a symlink?

No, this issue still happens even when I change the workspace directory to its parent folder (~/Documents/T-20)

Do you use the Obsidian app? Do you have a .obsidian folder?

Yes, I do have the app and have the .obsidian folder, but it's with default settings.

Hmm, okay I'm still not able to reproduce so this might be an OS-specific bug (likely with plenary.nvim). Can you step through the Client:vault_relative_path function to debug?
https://github.com/epwalsh/obsidian.nvim/blob/5ac4951a62facac6f2399c92a2c1518d07be499f/lua/obsidian/client.lua#L171-L189
I would just throw print statements after each line. That way we can see where it goes wrong.

The issue is the j variable in line 180. The solution for me is to increase the variable by 2 in order to account for the last character of the workspace name (m) and the directory. (/) This would mean "m/" is removed from the link. Thanks for the help!

Thanks for figuring that out @Tetra-20! I'll add that fix in #322


uniform cost search algorithm in python

uses the lowest cumulative cost to find a path from the source to the destination.
i.e: What the given algorithm do and i.e. Type of algorithm array,stack
1-Insert the root node into the priority queue
2-Repeat while the queue is not empty:
3-Remove the element with the highest priority
If the removed node is the destination, print total cost and stop the algorithm
Else, enqueue all the children of the current node to the priority queue, with their cumulative cost from the root as priority
Type it is searching using graph
Have you read the Contributing.md and Code of conduct

[ ✓] Yes
[ ] No

Other context

(Type here)

@ming-tsai
Yes, why my pr not accept?

@ming-tsai Yes, why my pr not accept?

Nop @sridhar-5 will be validate the pull request.

@ming-tsai  why my algorithm not merge, @sridhar-5  is reviewed it and he says it good and I fixed the problem in grammar

Hi @GAbdulWahab,
History said after approving it, @sridhar-5 found some spelling errors, so he make a request change for you.
After you push the change he has to review it again.

@ming-tsai  ok, sorry  , thanks


Locale not generated after creation of the docker

Installation information

jumpscale version: 9.0
operating system: Linux

Detailed description
After the creation of the js9 development docker. I notice the ays CLI tools doesn't work cause it can't find the configured locale.
I had to manually uncomment the en_US.UTF-8 line in /etc/local.gen then run locale-gen to solve the problem.
I think this should be done during docker creation

My mystake , I was in the docker using docker exec. when using ssh, no problem.


Add support for sqlx 0.8.0

Recently sqlx 0.8.0 got released. Changelog: https://github.com/launchbadge/sqlx/blob/main/CHANGELOG.md

I am on it. looks like it shouldn't take to long

sqlx 0.8 version is up and merged onto main. I plan on doing a bit more verification, and it should be out for the release in the morning.


macOS Big Sur 11.0.1 (20B29) - Battery Service Reccomended

Hello,
I recently reinstalled macOS on my ThinkPad T460s. Everything is working fine, including the DW1820a and bluetooth strangely enough!
The only issue I am facing is that the macOS Battery Applet says that my battery needs replacing. I checked it on HWMonitor but it seems to be in good conditions still, with 293 cycles out of 1000 and a max capacity of 3105 mAh. I should also note that neither Windows or Linux have previously told me that the battery needs replacing, nor has Lenovo's Vantage utility.
Any ideas why I am getting this weird issue?

You can just ignore it, i also have this message but all working fine (i have 2500~mAh ) on my battaries

This is because your battery health percentage is below a certain threshold. The design capacity on my machine is 4349 mAh. The warning will go away if you replace the batteries.

Weared battery (have same notification) it is not the EFI issue

Weared battery (have same notification) it is not the EFI issue

It isn't, not in my case. I changed my batteries in Sept 2020. Installing Linux I noticed that the battery information sent to the OS was completely wrong. Recalibrating fixed it. Maybe it isn't an EFI issue but somewhere down the line something is messing up the battery info.

any solutions to this? @mhl221135

any solutions to this? @mhl221135

Only batteries change- but you can also live with this message its just talking that batteries max capacity lower than design capacity


Enable assertions on gradle run

Codecov Report

Merging #142 (bbbd6a9) into master (1e8687f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #142   +/-   ##
=========================================
  Coverage     78.51%   78.51%
  Complexity      670      670
=========================================
  Files            90       90
  Lines          2001     2001
  Branches        266      266
=========================================
  Hits           1571     1571
  Misses          351      351
  Partials         79       79

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 1e8687f...bbbd6a9. Read the comment docs.


"DEPRECATION: Accessing 'template' in <dummy@component:sl-modal-header::ember831> is deprecated. To determine if a block was specified to <dummy@component:sl-modal-header::ember831> please use '{{#if hasBlock}}' in the components layout.

@notmessenger working on this, please assign it to me.

@notmessenger please mark as done, this has been merged.


Nushell Support

This PR adds support for Nushell.
whattheshell::Shell already supports Nushell, therefore this PR only needs to modify shell/mod.rs.
Since Nushell works great on Windows and Linux, some conditional compilation was necessary. Also Nushell doesn't support eval, therefore goup env will simply return valid Toml and the script for goup env -p will parse that output and load the env vars accordingly.

Awesome, thank you very much for the contribution! <3


Add forwardRef support to WebView in @datadog/mobile-react-native-webview

Here's the TS error Im getting
Type '{ ref: LegacyRef<WebView>; originWhitelist: string[]; contentMode: "mobile"; scrollEnabled: false; style?: StyleProp<ViewStyle>; containerStyle?: StyleProp<...>; onMessage?: ((event: WebViewMessageEvent) => void) | undefined; source: WebViewSourceHtml; isLoadingFields?: boolean | undefined; }' is not assignable to type 'IntrinsicAttributes & IOSWebViewProps & AndroidWebViewProps & WindowsWebViewProps & { ...; }'.
  Property 'ref' does not exist on type 'IntrinsicAttributes & IOSWebViewProps & AndroidWebViewProps & WindowsWebViewProps & { ...; }'.ts(2322)

I think the fix is to add forwardRef support to the datadog component.
Here's how it's being done in the react-native-webview package
https://github.com/search?q=repo%3Areact-native-webview%2Freact-native-webview forwardRef&type=code
Thanks!

Hi @leggomuhgreggo,
I realise the previous issue you opened (https://github.com/DataDog/dd-sdk-reactnative/issues/497) got automatically closed once our fix (https://github.com/DataDog/dd-sdk-reactnative/pull/500) was merged, even though it's not been released yet.
Sorry about that, I'll let you know here once it is merged :)

Thank you for the considerate note!
Oh wow -- I actually wasn't aware of that other issue (Cool to see the community so active!)
Wonderful to hear that the fix is already on its way. And regrets for the extra noise.
Y'all rock 🤘

Hi @leggomuhgreggo,
Sorry for assuming that previous issue was from you :)
The fix has been released under 1.8.0-rc1.
You can use it as such:
import React from 'react';
import WebView from '@datadog/mobile-react-native-webview';
import {WebView as RNWebView} from 'react-native-webview';

export const WebviewSite = () => {
  const ref = React.useRef<RNWebView>(null);

  return (
    <WebView
      source={{uri: 'https://my-site.com'}}
      allowedHosts={['my-site.com']}
      ref={ref}
    />
  );
};

Few caveats:

the initial value in useRef should be null
if you use typescript, the type of the ref has to be WebView from react-native-webview to ensure you the type matches your version of react-native-webview.

If that duplicated import is too much of a hassle for you let us know and we'll look into removing it :)

Beautiful. That all makes sense.
Thank you for the context!


This warning indicates that the suffixCls prop is being passed to a DOM element, but React does not recognize it as a valid prop

BUG: Warning: React does not recognize the suffixCls prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase suffixcls instead. If you accidentally passed it from a parent component, remove it from the DOM element.
section
./node_modules/antd/es/layout/layout.js/BasicLayout<@http://localhost:7001/static/js/bundle.js:134344:63
Layout
StyleProvider@http://localhost:7001/static/js/bundle.js:76731:18
FormProvider@http://localhost:7001/static/js/bundle.js:345503:26
ProviderChildren@http://localhost:7001/static/js/bundle.js:124628:18
ConfigProvider@http://localhost:7001/static/js/bundle.js:124750:52
App@http://localhost:7001/static/js/bundle.js:1497:142
I18nextWithTranslation@http://localhost:7001/static/js/bundle.js:413103:26
C@http://localhost:7001/static/js/bundle.js:417193:31
Router@http://localhost:7001/static/js/bundle.js:416648:30
BrowserRouter@http://localhost:7001/static/js/bundle.js:416162:35

@Abhijrathod just warning, no need to fix


feat: add action for cleaning 90 days old image

https://github.com/secureblue/secureblue/pull/738


How to reduce the memory use of image operations

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 1809 x64
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.14.0
Python version: 3.7.3 x64
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: cuda_10.0.130_411.31_win10 & cudnn-10.0-windows10-x64-v<IP_ADDRESS>
GPU model and memory: GTX 1070 Max-Q 6376 MB

Describe the current behavior
I want to do some operations like:

Split the image into 12*12 squares.
For each sub square, split it into 6*(212) horizontal & 6(12*2) vertical rectangles.
For each rectangle, calculate it's pixels horizontal & vertical gradient's mean and standard deviation.
So, for each sub square, i want 48 values, which is
6*(hori_gx_mean,hori_gy_mean,hori_gx_std,hori_gy_std)----24
and
6*(vert_gx_mean,vert_gy_mean,vert_gx_std,vert_gy_std)----24

Now I have a 1920 * 1280 image with 3 chanels.
I read image by
tf.Session().run(tf.image.decode_image(tf.read_file(imgDir)))/255.0
and calculate image by
`
for y in range(int(srcImg.shape[0]/12)-1):
for x in range(int(srcImg.shape[1]/12)-1):
sub12_12 = tf.image.crop_to_bounding_box(_src, y12, x12, 12, 12)
# computes every 212 and 122 subSet in sub12_12
for div in range(6):
# height=2 width=12
sub12_2 = tf.image.crop_to_bounding_box(
sub12_12, div * 2, 0, 2, 12)
# height=12 width=2
sub2_12 = tf.image.crop_to_bounding_box(
sub12_12, 0, div* 2, 12, 2)
            # computes gradient, return (dy, dx)
            _dy_12_2, _dx_12_2 = tf.image.image_gradients(
                tf.image.rgb_to_grayscale(sub12_2))
            _dy_2_12, _dx_2_12 = tf.image.image_gradients(sub2_12)

            # computes standard deviation
            _stDy12_2 = tf.math.reduce_std(_dy_12_2)
            _stDx12_2 = tf.math.reduce_std(_dx_12_2)

            _stDy2_12 = tf.math.reduce_std(_dy_2_12)
            _stDx2_12 = tf.math.reduce_std(_dx_2_12)
            # compute gradient's mean
            _uDy12_2 = tf.math.reduce_mean(_dy_12_2)
            _uDx12_2 = tf.math.reduce_mean(_dx_12_2)

            _uDy2_12 = tf.math.reduce_mean(_dy_2_12)
            _uDx2_12 = tf.math.reduce_mean(_dx_2_12)

            # 6 set of horizontal and vertical sub-rectangle feature
            feaBuf.append([_uDy12_2, _uDx12_2, _stDy12_2, _stDx12_2,
                            _uDy2_12, _uDx2_12, _stDy2_12, _stDx2_12])
        _retExp.append(feaBuf)
        feaBuf=[]

`
and cost about 0.5g of memory per line of image, and it is‘t affordable.
Describe the expected behavior
Reduce memory cost to about 500kB per line.

This is not Build/Installation or Bug/Performance issue. Please post this kind of support questions at Stackoverflow. There is a big community to support and learn from your questions. GitHub is mainly for addressing bugs in installation and performance. Thanks!


B2B Components: Price display “gross/net” in shopping lists is incorrect.

PHP Version: 8.3   | Shopware Version: <IP_ADDRESS>  | Affected area / extension: Platform(Default)

Expected behaviour:
If you configure net price for a customer group, this should also be displayed correctly in the shopping lists.
Actual behaviour:
The net price is shown under the total amount, but with VAT added, which doesn't make sense, since it's the net price already.

How to reproduce:

Activate shopping lists for a cusstomer
Change the price representation from gross to net for the "customers" customer group.
Put an article in a shopping list
Take a look at the price representation in the shopping list

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-39271
[created from NEXT-39271, comment 506948]

[public] The issue is already covered by the following ticket: https://github.com/shopware/shopware/issues/5279
[created from NEXT-39271, comment 507026]


🚨 refactor(examples): Home

Related issue
https://github.com/DivanteLtd/storefront-ui/issues/903
Scope of work
refactor homepage example
Screenshots of designs
mobile:

desktop:

Checklist

[x] No commented blocks of code left
[x] Run tests and docs
[x] Self code-reviewed

If applicable:

[ ] I followed composition rules for my component
[ ] I tested the component in most common device sizes (can be tested in Storybook from viewport addon in top menu)
[ ] I accept Individual Contributor License Agreement

@AdamPawlinski and please remove unused .jpg and .png :)

not: we should do the fix for product-cards shadows on the carousel


Header background shading

Create an option to change the background shading / color of the widget header.
Sometimes the white or black header text blend into the background.

I also suffer from this problem as I like to use photos as a wallpaper. There is light sky just where the header is...

@ghost Implemented in v.3.0, please check it in Beta now, see #320


pythonPackages.hydrus: 426 -> 431

Motivation for this change
Version bump
Fixes gif/video previews using mpv
Things done

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

[x] 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"
[x] 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
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 115164 at 7fc9d60f run on aarch64-linux 1

1 package built:

hydrus

1 suggestion:

warning: unused-argument
Unused argument: qt5.
Near pkgs/applications/graphics/hydrus/default.nix:4:3:
  |
4 | , qt5
  |   ^


adding some higher level methods in settings.Provider

Coverage decreased (-8.6%) to 91.385% when pulling be51f1363a8ff78a50eba173acd97948a8d1a03e on api-improvements into a3fdbf5d885c8555963a30c0dcd0891788e39d6c on master.

I'm closing this pull request, the aim of setting is really to provide a simple way of loading environmental settings into a service, mainly through the use of tags on a struct.
The provider + setting interfaces are a little clunky for this reason, as they are intended to be used by a filler.
The changes that you are proposing change the very unambiguous states of:

Key exists but is unable to be represented in the desired type
And Key does not exist.

Into just getting the 0 value for that type, at best and possibly a panic at worst.
The general use case for this for loading settings into structs, with the config for other packages, e.g. Database settings, Key/Cert locations, not for loading single settings, especially boolean settings, which may end up littering the codebase with conditionals, giving vastly different behaviour based on boolean settings.
I, and others do not see any reason to add additional functionality to this tool at this time.
You have also dramatically reduced test coverage.
It is for these reasons, and the reasons I have left in comments that I have decided to close this pull request.

Thats ok I understand.
Basically the package is being used for the wrong purpose if we use it to load single settings.
Fair enough, thanks for the explanation.


Help with image flip

Hey, thank you for figuring this all out!
I am a bit lost enabling the image to flip - do you add a command to "hostname" or is there a way through telnet?
Also, is there a way to enable the timestamp?
[edit]
Ok figured it out
Connect via Telnet
vi /mnt/config/flash_config.ini
under [video] highlight 0 value for flip -> press r on your keyboard and then 3 to swap the value to 3
if you want to enable the timestamp change OSD to 1
Scroll to the bottom, press i on the keyboard and delete the line with the checksum
press esc, then type :wq
generate a new checksum using md5sum /mnt/config/flash_config.ini
Open the file again: vi /mnt/config/flash_config.ini
At the bottom press i on the keyboard and add a new line checksum = your MD5HASH

On another note: if you edit the language to English in /mnt/config/factory_config.ini it remains after a factory reset.
I wonder if there is a way to swap the CN G2H to EU like someone has done for the Hub: https://kaspa.rs/xiaomi-aqara-hub-cn-on-european-server/ -> this would possibly permanently change the firmwares to the European version.
[edit 2]
To swap to the EU servers you need to edit mentions of lumi.camera.gwagl02 to lumi.camera.gwag03
cd ../
vi /mnt/config/miio/device.conf
vi /mnt/config/hostapd.conf
vi /etc/build.prop

Has anyone had any luck with the image flip?  Followed the steps above but the flip value just reverts back to 0.


Add a way to change the icon

What feature or improvement do you think would benefit Files?
It would give the users more customizability.
Requirements
This will add more customizability
Files Version
Version: <IP_ADDRESS>
Windows Version
Windows 11 Home 21H2 22000.1219
Comments
in the taskbar and stuff.

App icon can't be change due to how Files is built unless you build from source.


Difficulty running any test built on dune's diff action

I am trying to run my codebase's tests (Same project as #28),
which include a lot of things like
(rule
 (alias runtest)
 (action
  (diff pretty.expected pretty.output)))

Where the test produces some pretty.output, and pretty.expected is checked in to the repo
(so-called "gold" tests)
The difficulty is one run of these tests produces all the .output files in the _build folder. This means
that if the first mutation causes a test failure, all the later ones will as well, because they're just comparing
to the same test output.
This is definitely related to #16 #18, but it seems different enough I wanted to open my own issue for it.
I've got my currently-not-working branch here: https://github.com/stan-dev/stanc3/compare/master...WardBrian:stanc3:mutaml?expand=1
The way this currently proceeds if you run make mutation-test is that the first few runs timeout, while they're creating output files, and then the rest all fail because those output files exist and will not compare equal to the .expect files

I got closer by having the command passed to the metaml runner be a .sh file that contains
#!/bin/bash

find _build/default/test -name '*.output' -delete || true
dune runtest test/integration/ --force

Now it seems like my bigger issue is #32

Even after manually upping the timeout, still can't get this style of test to work. I'm confirming that the executable is receiving the environment variable, but it does not seem to really be mutating anything, as the tests all 'pass', but manually applying one of the mutation diffs that gets printed causes them to fail rather quickly

I was eventually able to get this to work by making a copy of the instrumented binary somewhere outside of _build, and then hard-coding that path into my tests. Not something viable for production, but to run this one-off it worked


Changed the minimum number of CylinderMesh rings to 0

Reduced the minimum number of cylinder rings to be 0 to address this issue: https://github.com/godotengine/godot/issues/82850 <!--
Please target the master branch in priority.
PRs can target 3.x if the same change was done in master, or is not relevant there.
Relevant fixes are cherry-picked for stable branches as needed by maintainers.
You can mention in the description if the change is compatible with 3.x.
To speed up the contribution process and avoid CI errors, please set up pre-commit hooks locally:
https://docs.godotengine.org/en/latest/contributing/development/code_style_guidelines.html
-->

I didn't see that this issue had already been addressed. Whoops.


Examples running on EFR32MG21 fail rendez-vous at Spake2p msg1

Problem
EFR32MG21 stalls during the BLE Secure rendez-vous after receiving  Spake2p msg1. It doesn't complete the encryptions and becomes unresponsive.
Summary of Changes

Cleanup up mbedtls defines for efr32 examples.
Remove all MBEDTLS_xxxx_ALT defines "config-device-acceleration.h" defines the available alternates API to accelerate crypto for the given MCU. MBEDTLS_SHA512_ALT and MBEDTLS_SHA512_PROCESS_ALT were causing issues for the MG21
Reduce some task stack size to free up some heap space. MG21 is more RAM limited and could not keep up with mallocs in the crypto.
Add some missing crypto sources files from GSDK3.1
add "mbedtls/check_config.h" to check for conflicts or missing defines.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jmartinez-silabs:x: jepenven-silabsYou have signed the CLA already but the status is still pending? Let us recheck it.


CEL for function inputs

https://github.com/GoogleContainerTools/kpt/issues/3964 is the kpt issue. We should implement that upstream, ideally.

https://github.com/GoogleContainerTools/kpt-functions-sdk/pull/678 is a building block

We should take into consideration as we move forward with the plan laid out in https://github.com/nephio-project/nephio/discussions/450 (it is mentioned there).
There is no deliverable for this for R2.


Parallel commits compare

Another try

Thanks Itai! I have read through this, and though I don't have any substantial comments - it seems like the ListCommits function became very long, and at points very complicated to understand. I think that if there's a way to break this into smaller logical units it would be much easier to read

I agree. Improved it some more. Breaking it for path / non-paths log-commits usage should make the code clearer at least for the common flow (non-paths). LMKWYT

Tested it locally by adding 200ms delay to block adapter read and zeroing out the cache. For 200 commits we currently log all of them in 78 seconds. This change reduces that number to 6 seconds.

Tested locally again after all changes applied, the above numbers are still valid. Thanks all for the improvements and the review! Pulling..


[bitchute/ffmpeg] Muxing fails due to unsupported timecode stream in MP4 file + -map 0

Copied from https://github.com/blackjack4494/yt-dlc/issues/291
Verbose log
Y:\YouTube Channels\Chass>youtube-dl --cookies "Y:\cookies.txt" --add-metadata --write-description --write-info-json --write-thumbnail --all-subs --sleep-interval 1 -ciw -o "%(title)s.%(ext)s" -v https://www.bitchute.com/video/7KOApju0sI1E/
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--cookies', 'Y:\\cookies.txt', '--add-metadata', '--write-description', '--write-info-json', '--write-thumbnail', '--all-subs', '--sleep-interval', '1', '-ciw', '-o', '%(title)s.%(ext)s', '-v', 'https://www.bitchute.com/video/7KOApju0sI1E/']
[debug] Loading archive file None
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dlc version 2020.11.11-2
[debug] Python version 3.9.0 (CPython) - Windows-10-10.0.19041-SP0
[debug] exe versions: ffmpeg 4.3.1-2020-11-19-full_build-www.gyan.dev, ffprobe 4.3.1-2020-11-19-full_build-www.gyan.dev
[debug] Proxy map: {}
[BitChute] 7KOApju0sI1E: Downloading webpage
[BitChute] 7KOApju0sI1E: Checking video URL
[debug] Default format spec: bestvideo+bestaudio/best
[info] Video description is already present
[info] Video description metadata is already present
[BitChute] 7KOApju0sI1E: Thumbnail is already present
[debug] Invoking downloader on 'https://z-28b3jxzl1og7.bitchute.com/iAhk3bN20i22/7KOApju0sI1E.mp4'
[download] 4chan threads - _x_ - The Desolate Towns Phenomenon.mp4 has already been downloaded
[download] 100% of 22.94MiB
[ffmpeg] Adding metadata to '4chan threads - _x_ - The Desolate Towns Phenomenon.mp4'
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:4chan threads - _x_ - The Desolate Towns Phenomenon.mp4" -map 0 -c copy -metadata "title=4chan threads - /x/ - The Desolate Towns Phenomenon" -metadata "date=20201218" -metadata "description=https://yuki.la/x/17299706
https://archive.4plebs.org/x/thread/22403433/#q22418409
https://yuki.la/x/25030586
If you wish to donate: https://www.paypal.me/thechasseurchannel
Bitchute: https://www.bitchute.com/channel/Chass/
Ultimate Spooky Greentext Playlist: https://www.youtube.com/playlist?list=PLbDSt84tOm2Nm0D8baMW3CfVaaF03LCjB
Music used:
Won't sleep - Kukomitzu
https://youtu.be/vz3KlzgEFm8?t=6
Stalker – Kukomitzu
https://youtu.be/vz3KlzgEFm8?t=297
♫ Epic Horror Synth Trailer Music ♪♬ - Something Wicked
https://www.youtube.com/watch?v=Zuw_O5MU5CE
[Vaporwave] Stevia Sphere - mint
https://www.youtube.com/watch?v=ykT3wtSCpzE
\"Midnight, the Stars and You\"
https://www.youtube.com/watch?v=-fN-Xjpd-qE
My last roadtrip - Kukomitzu
https://youtu.be/vz3KlzgEFm8?t=597
Outro music:
Gothic 3 - Geldern Night
https://www.youtube.com/watch?v=wCtV4DYZkK0" -metadata "comment=https://yuki.la/x/17299706
https://archive.4plebs.org/x/thread/22403433/#q22418409
https://yuki.la/x/25030586
If you wish to donate: https://www.paypal.me/thechasseurchannel
Bitchute: https://www.bitchute.com/channel/Chass/
Ultimate Spooky Greentext Playlist: https://www.youtube.com/playlist?list=PLbDSt84tOm2Nm0D8baMW3CfVaaF03LCjB
Music used:
Won't sleep - Kukomitzu
https://youtu.be/vz3KlzgEFm8?t=6
Stalker – Kukomitzu
https://youtu.be/vz3KlzgEFm8?t=297
♫ Epic Horror Synth Trailer Music ♪♬ - Something Wicked
https://www.youtube.com/watch?v=Zuw_O5MU5CE
[Vaporwave] Stevia Sphere - mint
https://www.youtube.com/watch?v=ykT3wtSCpzE
\"Midnight, the Stars and You\"
https://www.youtube.com/watch?v=-fN-Xjpd-qE
My last roadtrip - Kukomitzu
https://youtu.be/vz3KlzgEFm8?t=597
Outro music:
Gothic 3 - Geldern Night
https://www.youtube.com/watch?v=wCtV4DYZkK0" -metadata "purl=https://www.bitchute.com/video/7KOApju0sI1E/" -metadata "artist=Chass" "file:4chan threads - _x_ - The Desolate Towns Phenomenon.temp.mp4"
ERROR:   Stream #0:2 -> #0:2 (copy)
Traceback (most recent call last):
  File "youtube_dlc\YoutubeDL.py", line 2159, in post_process
  File "youtube_dlc\postprocessor\ffmpeg.py", line 532, in run
  File "youtube_dlc\postprocessor\ffmpeg.py", line 235, in run_ffmpeg_multiple_files
youtube_dlc.postprocessor.ffmpeg.FFmpegPostProcessorError:   Stream #0:2 -> #0:2 (copy)

Description
Commit d03cfdce1bce28284fa0f168d5d278caacd2fa0b introduced -map 0 to the FFmpeg post-processor to copy all streams. Unfortunately, the BitChute MP4 timecode stream I've run into here has an apparently invalid tag and can't be copied. If I type the ffmpeg command without -map 0 it works fine since it simply ignores the extra unsupported stream. The option was added to ensure that extra streams are preserved, but in this particular case, it's causing a muxing failure due to an unsupported extra stream.

This doesn't seem to be an issue with yt-dlc. Other bitchute videos work fine. This particular video has a broken data stream. You can fix it by using --postprocessor-args "-map -0:d?"

This doesn't seem to be an issue with yt-dlc. Other bitchute videos work fine. This particular video has a broken data stream. You can fix it by using --postprocessor-args "-map -0:d?"

This happens with multiple BitChute videos. It is not isolated to that single one.

This happens with multiple BitChute videos. It is not isolated to that single one.

Do you have any ideas how to fix this? I dont want to throw away the data streams of all videos just because of a few videos

Do you have any ideas how to fix this? I dont want to throw away the data streams of all videos just because of a few videos

Is there a way to tell ffmpeg not to throw a warning for unsupported streams?

Is there a way to tell ffmpeg not to throw a warning for unsupported streams?

Found it: -ignore_unknown says "Ignore input streams with unknown type instead of failing if copying such streams is attempted." Closing PR incoming.

Found it: -ignore_unknown says "Ignore input streams with unknown type instead of failing if copying such streams is attempted." Closing PR incoming.

I had also tried -ignore_unknown but didnt know about -dn. Thanks for the fix. I will merge it asap

I had also tried -ignore_unknown but didnt know about -dn. Thanks for the fix. I will merge it asap

Seems -dn is the inverse of -map 0:d for a stream.

Seems -dn is the inverse of -map 0:d for a stream.


Update legacy.json

Changed url of bhbotlist

You need to update legacy.json as part of the same PR that renames the list itself.


no support for fast user switching or multiple active gui logins

followup to #20687
it simply seems impossible to log in more than one user on the graphical interface of nix, no matter the window manager

Fast user switching works fine with KDE 4 (kdm). For KDE 5 (sddm), you need 69bea26ea9a6ec97927f45bea78f2148325c2cd9 which is in master but not release-16.09.

Cherry-picked 69bea26 to release-16.09.

@edolstra whats needed to make something similar work for gdm?

No idea, I don't use gdm. @lethalman may know.

GDM is blocked by #19896 now.

I tested gdm+gnome on master after resolving that issue, and it seems to work smoothly – gdm remains active on ctrl+alt+1 and the user sessions are spawned on 2, 3, ...

tested the same, works on master


[Feature] QR code with voucher

Detailed description
It's possible to add the voucher code to the QR code ?
Examples
No response
Additional context
No response

Übersetzter Text
Unfortunately, this is not possible. Only a passphrase for a secure WiFi can be transmitted in the QR code.
The voucher code must be entered in the Captive Portal.


Cannot Run ImList.of()?

Description
Hello! I am not too sure why I am unable to run this ImList.of() method in jshell. Below is my code.

I followed what the prof did in lecture..

Topic:
Specific Topics this question is related to (If Any)
Screenshots
Screenshots (if any)

I'm guessing that ImList has a dependency on something else... Pair perhaps?

Thank you Prof!


implement read checksums from file

This creates the checksums:
hfile -p . -a sha256 > checksums

To read them with sha256sum:
sha256sum -c checksums

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx


[Issue 7949][Tiered Storage] support aws creds per offload policies

Fixes #7949
Motivation
Provide different s3 credentials per offloadpolicies on each ns.
Modifications
Add awsId/awsSecret in OffloadPolicies.
Does this pull request potentially affect one of the following parts:

The rest endpoints: yes it adds options
The admin cli options: yes it adds options

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs

/pulsarbot run-failure-checks

@gaoran10  would you please also take a look?

LGTM, could you renew the doc file pulsar/site2/docs/reference-pulsar-admin.md about the command set-offload-policies.

@gaoran10 rebased from masted and documented.

/pulsarbot run-failure-checks

@KannarFr Hi, could you rebase code with master again, the block test has been fixed. Thanks.

@gaoran10 done.

/pulsarbot run-failure-checks

@gaoran10?

@gaoran10

LGTM.

@KannarFr Could you please resolve the conflicts?

@codelipenghui done.

@codelipenghui rebased!

@KannarFr Hi, KannarFr, a few days ago, the offload related code had been refined, it seems that the AWS role credentials had been removed, maybe we could repair it in this PR? Refer to the class JCloudBlobStoreProvider.

@gaoran10 there are missing dependencies such as org.apache.jcloud.providers:azureblob or org.apache.jcloud.providers:aws-s3`?

/pulsarbot run-failure-checks

will cherry-pick this change to 2.6.2

@wolfstudy so tag for release 2.7.0?

@wolfstudy so tag for release 2.7.0?

Yes, in 2.7.0, we will release this change.

@codelipenghui I saw you commit the changelog for 2.7.0 and no mention of this enhancement, is all ok?


TTS with Node Red

Hey everybody,
thanks for that Node, its very useful for my needs. I have a question about Node Red and TTS, using this Node.
Can anybody explain, how to send my TTS Text in Node Red to the TTS-Ultimate-Node? I tried some ideas and with one opinion, it works fine. But I have to send my TTS messages in a style like this: "Hello world!"
I wouldnt like to use " ... " in the beginning/ end. I only would like to write: Hello World!. But if I do like that, there is an TTS isue from Amazon Polly.
Maybe I need another function? At the moment I am using the "Trigger" event (see snapshot). Maybe I am using the wrong Function?
Thanks for help.

.

Hi Larson
Wy do you use  tag in the text?

Otherwise i bekomme an issue from amazon… exactly that is the Problem - i dont want to use Tags

You don't need to use the tags. Check the TTS node and disable the SSML option.

Disable this, if enebled.

Hey Larson,
are you still there?

Yes. Still at work. Cant check it until tonorrow… hold on…thanks…will answer after my check

Thank u you for help. That was exactly the problem ! Issue fixed:)!


Wrongly shows "please login again" error messages when not logged in

Hi Team,
Great work on the improved source academy! However, I see "Please login again" error messages both upon visiting the front page (even in incognito mode) and after a successful login. Please refer to the attached screenshot. These error messages might confuse new users.
URL: https://sourceacademy.nus.edu.sg/login
Browser: Firefox 68.0.1 / Chrome Version 76.0.3809.100
OS: MacOS Mojave 10.14.6

Have a fix for this; will deploy shortly. Apologies!

Hi Team,
Great work on the improved source academy! However, I see "Please login again" error messages both upon visiting the front page (even in incognito mode) and after a successful login. Please refer to the attached screenshot. These error messages might confuse new users.
URL: https://sourceacademy.nus.edu.sg/login
Browser: Firefox 68.0.1 / Chrome Version 76.0.3809.100
OS: MacOS Mojave 10.14.6

Have deployed the fix in PR #854. Thanks for logging the issue!

Thumbs up for the prompt fix!


Requesting License + input on upstream Keras Semantic Segmentation design

I came across your repository and it looks like good work and for that reason I'm submitting this request.
François Chollet, Keras' author, said he is interested in directly incorporating dense prediction/FCN into the Keras API, so I'm seeking suggestions/feedback/contributions at fchollet/keras#6538.
Also, could you add a license so it is clear how this can be used? I suggest the MIT license which is the same as Keras, it is pretty simple and lets people use it as they would like:
The MIT License (MIT)

Copyright (c) <year> <copyright holders>

Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.

 IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

Thanks for your consideration!

Hi Andrew, thanks for your suggestions. I have added the License. In addition, I am definitely willing to join the discussion and contribute to the Keras API as well.


Github action updated due to open-ce-environment separation

Checklist before submitting

[x] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Fixes #302 .
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

This change is not valid, you can't use actions within actions (I really wish you could).
I opened his to fix it:
https://github.com/open-ce/open-ce/pull/306


Arrow Keys should allow selecting one of the matches

Added in 0.0.5 https://github.com/rosendin/vue-bulma-typeahead/commit/8e45020010f1cd5513ec5ad2a61c06ef7e2caa17


Set default navigation page type tracking dimension

This change closely follows https://github.com/alphagov/static/pull/921
Here, we set a default value of none for the navigation-page-type custom dimension. This is to ease the processing of Google Analytics results, since fields missing this custom dimension are omitted in some reports.
Trello
https://trello.com/c/Eui3OmTB/491-track-navigation-page-type-for-non-taxon-pages

This might be more of a question for Tara, but should none be set on non-navigation pages? Or should it only be set on non-taxon navigation pages like browse and topic pages?

@suzannehamilton I already checked with Tara - she wants "full coverage" (including 'thing's)


Chapter 3 : Cascade evaluate ValueError: The pyarrow library is not installed

Dear authors,
the evaluate component of the pipeline fails due to the lack of pyarrow module.
Solved by changing the module request in the pipeline definition :
dsl.pipeline(
    name='Cascade pipeline on SF bikeshare',
    description='Cascade pipeline on SF bikeshare'
)

def cascade_pipeline(
    project_id = PROJECT_ID
):
    ddlop = comp.func_to_container_op(run_bigquery_ddl, packages_to_install=['google-cloud-bigquery'])

    c1 = train_classification_model(ddlop, PROJECT_ID)
    c1_model_name = c1.outputs['created_table']

    c2a_input = create_training_data(ddlop, PROJECT_ID, c1_model_name, 'Typical')
    c2b_input = create_training_data(ddlop, PROJECT_ID, c1_model_name, 'Long')

    c3a_model = train_distance_model(ddlop, PROJECT_ID, c2a_input.outputs['created_table'], 'Typical')
    c3b_model = train_distance_model(ddlop, PROJECT_ID, c2b_input.outputs['created_table'], 'Long')

    evalop = comp.func_to_container_op(evaluate, packages_to_install=['google-cloud-bigquery[bqstorage,pandas]', 'pandas'])
    error = evalop(PROJECT_ID, c1_model_name, c3a_model.outputs['created_table'], c3b_model.outputs['created_table'])
    print(error.output)

Best Regards
Jerome

SOLVED by  changing the module import used by the evalop :
evalop = comp.func_to_container_op(evaluate, packages_to_install=['google-cloud-bigquery[bqstorage,pandas]', 'pandas'])


Backport: HBASE-24379 CatalogJanitor misreports region holes when there are act…

…ually over laps. (#1741)
Signed-off-by: stack<EMAIL_ADDRESS>

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 10s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ branch-2.2 Compile Tests _

+1 :green_heart:
mvninstall
5m 21s
branch-2.2 passed

+1 :green_heart:
compile
0m 59s
branch-2.2 passed

+1 :green_heart:
checkstyle
1m 22s
branch-2.2 passed

+1 :green_heart:
shadedjars
4m  4s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 38s
branch-2.2 passed

+0 :ok:
spotbugs
3m 25s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 23s
branch-2.2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 46s
the patch passed

+1 :green_heart:
compile
0m 56s
the patch passed

+1 :green_heart:
javac
0m 56s
the patch passed

+1 :green_heart:
checkstyle
1m 18s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m  5s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
26m 21s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
javadoc
0m 36s
the patch passed

+1 :green_heart:
findbugs
3m 35s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
166m 22s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 36s
The patch does not generate ASF License warnings.

234m 21s

Subsystem
Report/Notes

Docker
Client=19.03.9 Server=19.03.9 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1785/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1785

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 48d5dbbbb207 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1785/out/precommit/personality/provided.sh

git revision
branch-2.2 / abe5a05bd1

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1785/1/testReport/

Max. process+thread count
4701 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1785/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Seems the test has been done, let me push it.


support relay-runtime v13

This issue is linked to https://github.com/relay-tools/relay-hooks/issues/126.
Even if it is react, many of the logic can be recovered in order to guarantee greater stability.

released with [relay-angular] - v1.0.0


[STYLE] Make IO redirect names more clear

Rename handle_io_redirect() to open_io_files().
Rename stdio_bind.c to stdio_redirect.c because "binding" file descriptors is commonly used with sockets, not with files.

I still think handle_* is more make sense for me

I still think handle_* is more make sense for me

The problem is that f.e. handle_red_out() sounds like it will handle the redirection completely, meaning it will redirect STDOUT to a file. But that's not what that function does. The actual redirection with dup2() is done by the redirect_*() functions.
I never know what the handle_*() functions actually do.


[WIP] Make node.root do magic SQL

Purpose
Remove reverse field for node root and switch it to use a SQL queries.
Changes
Rewrote SQL to get the top parent of a node. Wrote overly complicated test to make sure it works.
Added custom manager to AbstractNode with get_children(root=the_root_node) method that gets a queryset of children, recursively, of the root node.
Side effects
SQL queries could need to be optimized later.

@sloria I think I got it. Have a look when you get a chance.

I just added a thing that if the root comes back null it returns self.


Add iterator support

Updated dependencies; removed pointer analyzer as it panics in go 1.22; updated docs
Updated pipeline
Update to go.123 and update dependencies, this adds support for iterators

Hi, thanks a lot for the PR. These same changes will be part of #202 along with fixes for the rest of the issues. I will close this now.


ISPN-7296 Bounded Off Heap

https://issues.jboss.org/browse/ISPN-7296
This uses a simple LRU bounded cache which is applied to all entries in a given cache.  There is a lru lock that is required to be acquired whenever reading or writing to the lru based attributes or nodes off heap.
The eviction is done by acquiring the said lru lock to test the size and then if it is deemed to needed removal it read the head node of the lru stack and then try to acquire the write lock for the segment of that key.  If it can it will then remove it otherwise it must release the lru lock and try again.  This could be costly if there are a lot of failures to acquire the write lock.  Unfortunately due to deadlocking issues lruLock must be obtained after the write lock of a given segment (thus why a try lock must be used)

Ran with JMH a bit and the performance for the write only seems at a manageable level.
Ran test with eviction of COUNT for object, binary and off heap storage with a size of 500
Average of 3 runs each
Object: 33996
Binary: 36115.5
OffHeap: 32108
For some reason binary was quite fast in my tests.  But as you can see off heap was the slowest but only ~11% slower than the fastest.

Fixed test counter not being reverted :)

Pushed to master, thanks @wburns this is awesome work !


⚠ Eyrth has degraded performance

In 1461ee0, Eyrth (https://eyrth.com) experienced degraded performance:

HTTP code: 200
Response time: 17847 ms

Resolved: Eyrth performance has improved in 008bd19.


Bug: only a table gets deleted on backspace

Lexical version: 0.16.1
Steps To Reproduce

Visit https://playground.lexical.dev/
Insert a table of any dimension
Add paragraphs or images.
Select all by pressing Cmd+A
Press Backspace to delete the selected content.

The current behavior
Only the table gets deleted
https://github.com/user-attachments/assets/5a7f4358-44dc-492f-97ca-32bf031d502f
The expected behavior
All node/content should be deleted.

Only one table get deleted at a time.
https://github.com/user-attachments/assets/20400cfc-ca6e-4032-bdf1-82ac790ade53

This should be fixed in 0.17.0, can not reproduce at https://playground.lexical.dev/


portable plots extremely unlucky or not producing partials

Hello,
I've tested plotter from pool-puzzles branch. Created plots with:
/usr/bin/chia_plot -2 /chia/nvme/ -t /chia/nvme/ -d /chia/dst/ -z txch13eer6vdp8af7kj0f3u8spfqhsf8mad08h5pzd3hgq2w63lzpu3pqhvrzxs -f ae9df026cf05de62e3762ecf0b8995ec6acb61c6d1f272dc00f54c25c26051e2da6631909e2587937334b8219a370caf -r 24 -n 30

Plots are created fine, I've created 8, chia plots check reports they are fine:
2021-06-22T16:30:10.140  chia.plotting.plot_tools         : INFO     Found plot /root/plots/plot-k32-2021-06-22-07-20-06b9b1775c8314e4810548159a96373f9746994b8cb33d459ebe9c0f596aed08.plot of size 32
2021-06-22T16:30:10.142  chia.plotting.plot_tools         : INFO     Found plot /root/plots/plot-k32-2021-06-22-14-28-31d32608010385eb240d26ad5fc09a57f097332b05de732ef027c2cdb474438b.plot of size 32
2021-06-22T16:30:10.168  chia.plotting.plot_tools         : INFO     Found plot /root/plots/plot-k32-2021-06-22-10-11-bd1e0c2cd8fbb2132a1f549936799a7f0ed851b9396b2d1695f767a372f5223f.plot of size 32
2021-06-22T16:30:10.227  chia.plotting.plot_tools         : INFO     Found plot /root/plots/plot-k32-2021-06-22-11-37-d0f2fa13f6eff108e7aeb9adca555473e3943e3ac3e1d2b1be3a8aa3bc2e7308.plot of size 32
2021-06-22T16:30:10.244  chia.plotting.plot_tools         : INFO     Found plot /root/plots/plot-k32-2021-06-22-13-02-e4b53231536f44b0838b8b5f7bcd4d71d1ca18f2c0c2ac7851583cc5cd685129.plot of size 32
2021-06-22T16:30:10.252  chia.plotting.plot_tools         : INFO     Found plot /root/plots/plot-k32-2021-06-22-08-45-678498ea00a742e86b15b03d3c9d784411351abf2c2e3e8e61ca4fb7b0ee4e25.plot of size 32
...
2021-06-22T16:31:30.573  chia.plotting.check_plots        : INFO     Testing plot /root/plots/plot-k32-2021-06-22-14-28-31d32608010385eb240d26ad5fc09a57f097332b05de732ef027c2cdb474438b.plot k=32
2021-06-22T16:31:30.573  chia.plotting.check_plots        : INFO     	Pool public key: None
2021-06-22T16:31:30.579  chia.plotting.check_plots        : INFO     	Farmer public key: ae9df026cf05de62e3762ecf0b8995ec6acb61c6d1f272dc00f54c25c26051e2da6631909e2587937334b8219a370caf
2021-06-22T16:31:30.579  chia.plotting.check_plots        : INFO     	Local sk: <PrivateKey 6743d3fc912e979714331326cff88887f6577cb310de853ff7f03e5a99ed7164>
2021-06-22T16:31:38.186  chia.plotting.check_plots        : INFO     	Proofs 24 / 30, 0.8
2021-06-22T16:31:38.187  chia.plotting.check_plots        : INFO     Testing plot /root/plots/plot-k32-2021-06-22-07-20-06b9b1775c8314e4810548159a96373f9746994b8cb33d459ebe9c0f596aed08.plot k=32
2021-06-22T16:31:38.187  chia.plotting.check_plots        : INFO     	Pool public key: None
2021-06-22T16:31:38.193  chia.plotting.check_plots        : INFO     	Farmer public key: ae9df026cf05de62e3762ecf0b8995ec6acb61c6d1f272dc00f54c25c26051e2da6631909e2587937334b8219a370caf
2021-06-22T16:31:38.193  chia.plotting.check_plots        : INFO     	Local sk: <PrivateKey 50c0fe40fa2cb52116d4d9cc7c3baa11aab4d0dcc88c40b4ce5485f9f06583f2>
2021-06-22T16:31:47.136  chia.plotting.check_plots        : INFO     	Proofs 30 / 30, 1.0
2021-06-22T16:31:47.136  chia.plotting.check_plots        : INFO     Testing plot /root/plots/plot-k32-2021-06-22-11-37-d0f2fa13f6eff108e7aeb9adca555473e3943e3ac3e1d2b1be3a8aa3bc2e7308.plot k=32
2021-06-22T16:31:47.137  chia.plotting.check_plots        : INFO     	Pool public key: None
2021-06-22T16:31:47.143  chia.plotting.check_plots        : INFO     	Farmer public key: ae9df026cf05de62e3762ecf0b8995ec6acb61c6d1f272dc00f54c25c26051e2da6631909e2587937334b8219a370caf
2021-06-22T16:31:47.143  chia.plotting.check_plots        : INFO     	Local sk: <PrivateKey 66575635fe3e45db75414fbab6150830495b92fb06f1e6a42f007ad1bd66ad45>
2021-06-22T16:31:59.598  chia.plotting.check_plots        : INFO     	Proofs 39 / 30, 1.3
2021-06-22T16:31:59.599  chia.plotting.check_plots        : INFO     Testing plot /root/plots/plot-k32-2021-06-22-10-11-bd1e0c2cd8fbb2132a1f549936799a7f0ed851b9396b2d1695f767a372f5223f.plot k=32
2021-06-22T16:31:59.599  chia.plotting.check_plots        : INFO     	Pool public key: None
2021-06-22T16:31:59.605  chia.plotting.check_plots        : INFO     	Farmer public key: ae9df026cf05de62e3762ecf0b8995ec6acb61c6d1f272dc00f54c25c26051e2da6631909e2587937334b8219a370caf
2021-06-22T16:31:59.606  chia.plotting.check_plots        : INFO     	Local sk: <PrivateKey 196775b2ea5b67a677dc74fc0cc0cb0b1318cf1e417295c4cbb5ad3376abb8ea>
2021-06-22T16:32:07.489  chia.plotting.check_plots        : INFO     	Proofs 25 / 30, 0.8333
2021-06-22T16:32:07.489  chia.plotting.check_plots        : INFO     Testing plot /root/plots/plot-k32-2021-06-22-13-02-e4b53231536f44b0838b8b5f7bcd4d71d1ca18f2c0c2ac7851583cc5cd685129.plot k=32
2021-06-22T16:32:07.489  chia.plotting.check_plots        : INFO     	Pool public key: None
2021-06-22T16:32:07.496  chia.plotting.check_plots        : INFO     	Farmer public key: ae9df026cf05de62e3762ecf0b8995ec6acb61c6d1f272dc00f54c25c26051e2da6631909e2587937334b8219a370caf
2021-06-22T16:32:07.496  chia.plotting.check_plots        : INFO     	Local sk: <PrivateKey 06f59b43d5946c0c655803e5f5937a643f2d34aa2732cce5fb9f457d0ed0e73d>
2021-06-22T16:32:17.504  chia.plotting.check_plots        : INFO     	Proofs 33 / 30, 1.1
2021-06-22T16:32:17.504  chia.plotting.check_plots        : INFO     Testing plot /root/plots/plot-k32-2021-06-22-08-45-678498ea00a742e86b15b03d3c9d784411351abf2c2e3e8e61ca4fb7b0ee4e25.plot k=32
2021-06-22T16:32:17.504  chia.plotting.check_plots        : INFO     	Pool public key: None
2021-06-22T16:32:17.511  chia.plotting.check_plots        : INFO     	Farmer public key: ae9df026cf05de62e3762ecf0b8995ec6acb61c6d1f272dc00f54c25c26051e2da6631909e2587937334b8219a370caf
2021-06-22T16:32:17.511  chia.plotting.check_plots        : INFO     	Local sk: <PrivateKey 29ac3682e5423f131fa26b974c6eedfff53f760b996b0e44047f1273d7a72bc3>
2021-06-22T16:32:25.408  chia.plotting.check_plots        : INFO     	Proofs 25 / 30, 0.8333

I've been farming whole day on my testnet pool and have not seen a single partial from them. Other plots (created by chia plots create) are working fine on same harvester.
Any ideas? Maybe the pool-puzzles branch is not "production ready" yet?

Maybe the pool-puzzles branch is not "production ready" yet?

Of course not lol, it's never been tested for real. Maybe the setup of the puzzle hash was wrong? Can you post how you did that?

Sorry, I'm not a native speaker. "Production ready" was meant as a joke :)
I've created plots using command at the top. -z value from (same as for original plotter):
$ chia plotnft show
...
P2 singleton address (pool contract address for plotting):**txch13eer6vdp8af7kj0f3u8spfqhsf8mad08h5pzd3hgq2w63lzpu3pqhvrzxs**

-f value from
# chia keys show | grep Farmer
Farmer public key (m/12381/8444/0/0): ae9df026cf05de62e3762ecf0b8995ec6acb61c6d1f272dc00f54c25c26051e2da6631909e2587937334b8219a370caf

pool contract address != pool puzzle hash
now where to get the puzzle hash, I have no idea

the pool puzzle hash is supposed to be a 32 byte / 64 character hex string, not a BASE64 address...

Yeah ok, I need to convert it internally, see: https://github.com/madMAx43v3r/chia-plotter/pull/529#issuecomment-867211813


Update gemspec for rails5

Could you merge and bump up if there is no problem.

Thanks for submitting this. I decided that I wanted to fix up the travis testing to make sure the bump works. I pushed that out at 2.2.0.


Update windows-build images or WINE version to fix KERNEL32.dll.GetQueuedCompletionStatusEx

Tangentially related to https://github.com/rust-embedded/cross/issues/186 and https://github.com/rust-embedded/cross/issues/85
The WINE version in Ubuntu 18.04 is outdated in the sense that it does not implement KERNEL32.dll.GetQueuedCompletionStatusEx, thus leading to the following error when that function is called:
wine: Call from 0x7bc5b50c to unimplemented function KERNEL32.dll.GetQueuedCompletionStatusEx, aborting

I believe this happens for every tokio application, more description on the issue in general can be found at https://bugs.winehq.org/show_bug.cgi?id=43878
Would it maybe be an option to upgrade just the Windows docker images to Ubuntun 20.04 (now or at release)?

Seems like this was fixed in 3.21, and we use 3.0, and WINE provides binary releases for our images. This should be pretty fixable. It looks like it won't affect the image size either.


Optimise PKEY decoders

The most expensive part of using a PKEY decoder is the OSSL_DECODER_CTX_new_for_pkey() call. This builds up all of the decoder chains, which is a complex and time consuming operation. However, if no new providers have been loaded/unloaded since the last time it was called we can expect the same results for the same parameters. Note that this operation takes place before we event parse the data for decoding so it is not dependent on the parsed data at all.
We introduce a cache for OSSL_DECODER_CTX objects. If we have been called with the same parameters then we just duplicate an existing OSSL_DECODER_CTX. This should be significantly faster than creating a new one every time.
Partially addressed the issue in #15199

I've set the "tests:deferred" label since the test is in https://github.com/openssl/tools/pull/161

Using the test linked to above I've measured the performance in each of the various branches including master plus this patch. Smaller numbers are better.

branch
av time for 100 calls (us)

1.1.1
830

3.0
55123

3.1
12271

master
12813

Master + patch
2345

So this patch seems to be approx. 5 times faster than 3.1 (but still slower than 1.1.1).

I've set the "tests:deferred" label since the test is in openssl/tools#161

Do we do some simultaneous read of pkeys in the multithread test? It would be probably interesting to add it if not as there is some locking involved here and the TSAN CI run could find something.

I've just added one in a new commit.

What about timing  info for this?

A global property query change should also flush the cache but doesn't seem to.

Fixed

What about timing info for this?

Not sure what timing info you are after beyond https://github.com/openssl/openssl/pull/21426#issuecomment-1631044902
Or do you want to see multi-thread performance numbers? The test in openssl/tools#161 supports that but I haven't generated the numbers.

This looks like it's almost duplicating the cache in crypto/property/property.c but at a slightly higher level. I do wonder if we need both.

Any thoughts on this?

Still approved with or without @t8m's suggestions.
Apart from the double ; which should be addressed.

All nits addressed. Please take another look.

Merged to master.