feat: detach TheHeader from the left and insert to the left of the SwapSection

Place the TheHeader inside the SwapSection in a new div and adjust its layout to have the same characteristics as the other components on the screen.
I also made an adjustment to the arrangement of the items so that they adjusted to the screen size, always occupying the entire screen, whereas before the component had a fixed size starting at the screen size of 1024px, however, if you do not agree, it is easy to return. it to its initial size.
TheHeader is also used on the Offers screen and with the changes made, it ended up disconfiguring this screen as well, I made some adjustments so that TheHeader would fit better on the screen.
Lastly, I made an adjustment according to Issue #106 so that the mouse cursor is changed to the standard format when it is positioned above the Tooltip text box.

I've already updated my branch with the changes made to Tooltip, awaiting review of my PR

We are using the opportunity of this PR to include more changes in the sizes of all components. We were lacking communication with the design on Figma, so we are adjusting all in here to facilitate future commits. @Hardsys-Andre you might see a lot of unexpected changes in the following commits but do not get overwhelmed!!

This issue closes #157 #82 #133


feat: shouldRemoveOrphanedPartInstance

This PR adds a showstyle blueprint method shouldRemoveOrphanedPartInstance. This method is called whenever the part associated with whatever is set as next is deleted.
If the blueprints want the part instance for next to be removed, and a new next to be found, they call a method on the context object passed to this function.
This function itself does not select a new next, as it is not expected to be used in isolation but as part of the commit phase of database operations, which ensure a safe next point before finalizing the commit.

Just came to my mind - shouldn't this be respecting isTooCloseToAutonext?

Just came to my mind - shouldn't this be respecting isTooCloseToAutonext?

Absolutely, I've added that check https://github.com/tv2/tv-automation-server-core/commit/b653b647c9aed0767fe200f8d17a9090dbf266d2


Refactored Target Custer Code, Added Test case and Basic workflow for platform

What this PR does / why we need it:
Refactored Target Custer Code, Added Test case and Basic workflow for platform
Which issue(s) this PR fixes (optional)
Closes #DS-8530
Special notes for your reviewer:

This contains targetCluster changes from Arun's branch. cc: @aravindranath14


Add failing tests for #13

Failing test case for valid interpolations
Real world failing test case (BidiButton)

This was added in #70


Table 'GRCh37.chromInfo' doesn't exist // ERROR: chrom "14" not found in genome file. Exiting.

Description
Seems that Outrigger needs .chromInfo file but can't find it
There is also a conflit from Ensembl vs UCSC chr naming convention
Steps to Reproduce

Alignwith STAR (OK) using Ensembl GRCh37 primary assembly and gtf
outrigger index with the same gtf used by STAR (OK)
outrigger validate with the same .fa used by STAR

Expected behavior:
This is my first run, so I'm not sure..
But based on the error message it seems that it's looking for the chromInfo file and can't find it. Then it tries to get the chromInfo file from UCSC genome browser, but since the Ensembl naming convention uses "14" instead of "chr14" it can't find it.
I also tried to put the correct version of chromInfo with the .fa or my STAR output or Outrigger output, but didn't fix the problem.
Actual behavior: [What actually happened]
Time Stamp	Creating folder ./outrigger_output ...
Time Stamp		Done.
Time Stamp	Creating folder ./outrigger_output/index ...
Time Stamp		Done.
Time Stamp	Creating folder ./outrigger_output/index ...
Time Stamp		Done.
Time Stamp	Creating folder ./outrigger_output/index/gtf ...
Time Stamp		Done.
Time Stamp	Creating folder ./outrigger_output/junctions ...
Time Stamp		Done.
Time Stamp	Finding valid splice sites in Skipped Exon (SE) splice type ...
Time Stamp	Creating folder ./outrigger_output/index/se/validated ...
Time Stamp		Done.
Time Stamp		Finding valid splice sites for isoform2 of Skipped Exon events which match GT/AG or GC/AG or AT/AC...
Time Stamp			Finding splice sites for exon1 and exon2 ...
b"ERROR 1146 (42S02) at line 1: Table 'GRCh37.chromInfo' doesn't exist\n"
Commands were:
mysql --user=genome --host=genome-mysql.cse.ucsc.edu -A -e "select chrom, size from GRCh37.chromInfo"
Traceback (most recent call last):
File "/P/A/T/H/anaconda2/envs/outrigger-env/bin/outrigger", line 6, in
sys.exit(outrigger.commandline.main())
File "/P/A/T/H/anaconda2/envs/outrigger-env/lib/python3.5/site-packages/outrigger/commandline.py", line 1071, in main
cl = CommandLine(sys.argv[1:])
File "/P/A/T/H/anaconda2/envs/outrigger-env/lib/python3.5/site-packages/outrigger/commandline.py", line 344, in init
self.args.func()
File "/P/A/T/H/anaconda2/envs/outrigger-env/lib/python3.5/site-packages/outrigger/commandline.py", line 352, in validate
validate.execute()
File "/P/A/T/H/anaconda2/envs/outrigger-env/lib/python3.5/site-packages/outrigger/commandline.py", line 834, in execute
exonA, exonB, splice_abbrev)
File "/P/A/T/H/anaconda2/envs/outrigger-env/lib/python3.5/site-packages/outrigger/commandline.py", line 787, in exon_pair_splice_sites
exonA, splice_abbrev, 'downstream')
File "/P/A/T/H/anaconda2/envs/outrigger-env/lib/python3.5/site-packages/outrigger/commandline.py", line 800, in individual_exon_splice_sites
exon_bed, self.genome, self.fasta, direction)
File "/P/A/T/H/anaconda2/envs/outrigger-env/lib/python3.5/site-packages/outrigger/validate/check_splice_sites.py", line 57, in read_splice_sites
flanked = bed.flank(l=left, r=right, s=True, genome=genome)
File "/P/A/T/H/anaconda2/envs/outrigger-env/lib/python3.5/site-packages/pybedtools/bedtool.py", line 806, in decorated
result = method(self, *args, **kwargs)
File "/P/A/T/H/anaconda2/envs/outrigger-env/lib/python3.5/site-packages/pybedtools/bedtool.py", line 337, in wrapped
decode_output=decode_output,
File "/P/A/T/H/anaconda2/envs/outrigger-env/lib/python3.5/site-packages/pybedtools/helpers.py", line 356, in call_bedtools
raise BEDToolsError(subprocess.list2cmdline(cmds), stderr)
pybedtools.helpers.BEDToolsError:
Command was:
bedtools flank -s -r 2 -l 0 -i ./outrigger_output/index/se/exon1.bed -g /tmp/pybedtools.ghiu8lqg.tmp

Error message was:
ERROR: chrom "14" not found in genome file. Exiting.
Versions
Ubuntu 14.04
Anaconda 2
Outrigger version 1.1.1 installed with anaconda
Outrigger calls Python 3.5.5 in virtual env created with anaconda

I found the solution..
After the --genome I put the location of my .chromInfo file and it worked.

HI! What is chrominfo file ,how can i get it

I found the solution..
After the --genome I put the location of my .chromInfo file and it worked.

Hi，would you give the details for getting the .chromInfo? thanks a lot!

chromInfo

I suppose that after -g, wo should input a genome.chrom.sizes file.


parser: set UnionStmt and ShowStmt as readonly (#755)

Conflicting files:
util.go
util_test.go

What problem does this PR solve?
Fix https://github.com/pingcap/tidb/issues/15050.
UNION statement and SHOW statement are not treated as readonly statement. These statements are executed again in retrying transactions.
What is changed and how it works?
Set UnionStmt and ShowStmt as readonly.
Check List
Tests

Unit test

Code changes

Has exported function/method change

Side effects
N/A
Related changes

Need to cherry-pick to the release branch
Need to be included in the release note

Release Note

Treat UNION statement and SHOW statement as readonly statements.

/run-tests

/run-all-tests

LGTM


Increase upper bound on interfaces to 128

We hit the limit of 64 interfaces when running a large number of
docker containers, each of which created a bridge interface.  This
doubles the limit as a temporary work-around, although a real fix
would be to change the code to handle an arbitrarily large number of
interfaces as memory permits.


fixed disk/nic fragment syncing,

these fragments (plus snaphots) also sync only on fragment creation now

fixed


Pause Button Team (Team HoldUp)

Make a pretty pause screen! Work with the Cooler Math team to stop/start the timer when the pause screen is opened or closed.
Afterwards, use AppState from Expo to automatically pause the game if you leave the app (or get a phone call):

@shopaniuk is this task still in progress?


Includes folder not found

Hi Team,
After setting up and running the basic map service commands i get the following error
cd chart-installation/generate_map_files/scripts
python3 ./generate_map_config.py -rules ../resources/layer_rules/rules/ -basechartdata /home/user/SMAC-M/ENC_ROOT
cd: no such file or directory: chart-installation/generate_map_files/scripts
Existing color tables found at /home/user/SMAC-M/chart-installation/generate_map_files/resources/layer_rules/rules/color_tables
Existing layers found in /home/user/SMAC-M/chart-installation/generate_map_files/resources/layer_rules/rules/layer_rules
Traceback (most recent call last):
  File "./generate_map_config.py", line 160, in <module>
    main()
  File "./generate_map_config.py", line 141, in main
    args.force_overwrite, args.debug, point_table, area_table, displaycategory, chartsymbols)
  File "/home/user/SMAC-M/chart-installation/generate_map_files/scripts/basechart.py", line 118, in generate_basechart_config
    map_path, fonts_path, debug, shapepath)
  File "/home/user/SMAC-M/chart-installation/generate_map_files/scripts/basechart.py", line 67, in create_capability_files
    d = get_dictionary(theme, map_path, fonts_path, debug_string)
  File "/home/user/SMAC-M/chart-installation/generate_map_files/scripts/basechart.py", line 45, in get_dictionary
    theme),
  File "/home/user/SMAC-M/chart-installation/generate_map_files/scripts/basechart.py", line 24, in generate_includes
    includes_dir) if inc.startswith(theme)]
FileNotFoundError: [Errno 2] No such file or directory: '/home/user/SMAC-M/ENC_ROOT/map/includes'

I've tried creating the folder but after running the above command, the folder is deleted.

Hello Kevinwochan,
Can you rerun command from base directory (without cd command) like in example:
python3 chart-installation/generate_map_files/generate_map_config.py ...
Of course with all relative paths adjusted.

python3 chart-installation/generate_map_files/generate_map_config.py configfile.toml
Existing color tables found at /home/SMAC-M/chart-installation/generate_map_files/resources/rules/color_tables
Existing layers found in /home/SMAC-M/chart-installation/generate_map_files/resources/rules/layer_rules
/home/SMAC-M/chart-installation/generate_map_files/mapgen/symbol.py:412: UserWarning: Pattern command not implemented: CI
  warnings.warn('Pattern command not implemented: ' + command)
Check geometry of all layers...
Loading DUSK.csv
Loading DAY_BRIGHT.csv
Loading DAY_WHITEBACK.csv
Loading NIGHT.csv
Loading DAY_BLACKBACK.csv
Traceback (most recent call last):
  File "chart-installation/generate_map_files/generate_map_config.py", line 151, in <module>
    displaycategory, chartsymbols)
  File "/home/SMAC-M/chart-installation/generate_map_files/mapgen/basechart.py", line 108, in generate_basechart_config
    map_path, fonts_path, debug, shapepath)
  File "/home/SMAC-M/chart-installation/generate_map_files/mapgen/basechart.py", line 62, in create_capability_files
    d = get_dictionary(theme, map_path, fonts_path, debug_string)
  File "/home/SMAC-M/chart-installation/generate_map_files/mapgen/basechart.py", line 40, in get_dictionary
    theme),
  File "/home/SMAC-M/chart-installation/generate_map_files/mapgen/basechart.py", line 19, in generate_includes
    includes_dir) if inc.startswith(theme)]
FileNotFoundError: [Errno 2] No such file or directory: '/home/SMAC-M/data/map/includes'

Can you dump contents of configfile.toml, please?

debug = false
displaycategory = "Standard"
area_table = "Plain"
point_table = "Paper"

[paths]
map = "/home/SMAC-M/data/map"
chartsymbols = "/home/SMAC-M/chart-installation/generate_map_files/resources/chartsymbols/chartsymbols_S57.xml"
chart = "/home/SMAC-M/ENC_ROOT"
data = "/home/SMAC-M/data/shape"
ruleset = ""

Can you post output from:
ls -lR /home/SMAC-M/data/shape, please?
Also what is the output of:
ogrinfo --config S57_PROFILE iw -ro <any_s57_input_file>

 ogrinfo --config S57_PROFILE iw -ro AU130120.000
INFO: Open of `AU130120.000'
      using driver `S57' successful.
1: DSID (None)
2: Point (Point)
3: Line (Line String)
4: Area (Polygon)
5: Meta (None)

Drivers seem to be fine
and the folder is empty
/home/SMAC-M/data/shape:
total 0

It is not fine. Did you follow the step (your gdal path can be different):
$ ogrinfo --version
GDAL 2.2.2, released ...
$ find / -name s57objectclasses_iw.csv
/usr/share/gdal/2.2/s57objectclasses_iw.csv

$ wget https://raw.githubusercontent.com/OpenCPN/OpenCPN/master/data/s57data/s57objectclasses.csv
$ wget https://raw.githubusercontent.com/OpenCPN/OpenCPN/master/data/s57data/s57attributes.csv
$ cp s57objectclasses.csv /usr/share/gdal/2.2/s57objectclasses_iw.csv
$ cp s57attributes.csv /usr/share/gdal/2.2/s57attributes_iw.csv


Add GitLab to home page.

Some people would rather link to GitLab instead of GitHub, or both! This issue would add a gitlab.svg to https://github.com/gundamew/hugo-bingo/tree/master/static/img plus whatever changes would need to be made for it to be picked up by the config file for the home page.
Something like: https://about.gitlab.com/images/new_logo/B.jpg but to match the existing aesthetic.

Done. Please check v1.3.0. 😄


Ignore Comments when resolving Macros

A bug was identified by a user where comments in SPL were resolved as macro names.
Validating would fail since we were unable to find these macros.
Now, we use a regex-based solution to remove comment-text in SPL before attempting.
to resolve all outstanding macros

Tested!

current branch fixes this!


Import BusyModule in my module,all routers loadChildren can not use

it is AppModule
@NgModule({
declarations: [
AppComponent,
LoginComponent
],
imports: [
BrowserModule,
FormsModule,
HttpModule,
CommonModule,
BrowserAnimationsModule,
BusyModule,
RouterModule,
RouterModule.forRoot(indexRoutes)
],
providers: [],
bootstrap: [AppComponent]
})
export class AppModule { }
it is Router
export const indexRoutes: Routes = [
{
path: '',
redirectTo: '/login.html',
pathMatch: 'full'
},
{
path: 'login.html',
component: LoginComponent
},
{
path: 'test',
loadChildren: './test/index.module#IndexModule'
}
];
it is my project

when use     ng serve --prod     start

i find all loadChildren can not use，
How to solve?

my angular version is 4.1.3

We must not provide Compiler as JitCompiler. Instead we must provide and inject the Compiler2 extends Compiler.
Provide: Compiler2, usefactory:

yes @apoterenko that was the solution, I created a new CustomCompiler extends Compiler, and I replaced it in the busy component and the aot compilation works

@Xepe, can you please explain why do you need to Compiler to make JIT compiler work in AOT?

@ipassynk I am not totally sure, the issue that I found is because the JIT Compiler is set for the Compiler and for JIT is ok but for AOT it does not work, I got some error like "the compiler is not loaded",  for that reason I created a extends of Compiler and I setup that customCompiler where the component need it.

I am having the same issue. will it be fixed in near future?

@Xepe maybe you can add here the customCompiler example ?


[sentry/stable] Improve the configurability of jobs

Please review #10144 first
What does this commit/MR/PR do?

Improve the configurability of jobs
Wait until postgresql is ready before doing migrations
Add resources options to the migrations job as it can be resources
intensive
Change naming from "{{ Release.Name }}-" to "{{ template "fullname"
.}}-"

Why is this commit/MR/PR needed?

To improve the jobs definitions

Special notes for your reviewer:

This Pr builds on top of #10144, please review that PR first. If necessary, this can be refactored into a completed independent PR but this is only necessary if #10144 is being rejected.
I have a couple of more feature/change requests coming, I'd prefer to keep them in separate PRs for now.

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

mentions:
@rothgar

Will reopen when #10144 is reviewed (just want to have this ready in advance)


Fixes invalid warning for options

Warning was firing when a correct option was specified, instead of the other way around.

Duplicate of #1478

aah, sorry about that!

no problem, thanks for doing the PR.


made compatible with current ArangoDB versions

please merge and create new tag for the app, finally add the tag to https://github.com/arangodb/foxx-apps/blob/master/applications/profile.json

Thanks a lot for the fix.


Add Ian to the contributors list

@all-contributors, please add @IanSSenne for code and ideas

@all-contributors, please add @IanSSenne for code, ideas

@all-contributors, please add @IanSSenne for code and ideas


Add search methods to intro

Depends on https://github.com/rizinorg/book/pull/93
Adds a description about the most essential search methods.
This is not meant to be merged but merely here for feedback. If you like the style I can combine it with https://github.com/rizinorg/book/pull/79.

shouldn't this go to dev?

Ah yes. It should.

by search methods you mean searching commands, don't you?

Yes. But you are right. Those are not really search methods.

Rebase, please


How can guid primary key auto generated just like identity attribute?

Maybe this has no ralation with serenity,I just want to know how can I write  a Migration wich can  generate sql  likes below:
create table tguid
(
gid uniqueidentifier not null default(newid()),
val varchar(12)
)

Try this
http://stackoverflow.com/questions/7530221/is-it-possible-to-set-column-default-to-sql-servers-newid


[APPM-1017] [Postgres] WSO2_AM_DB tables creation script error

ERROR: relation "am_api_sequence" does not exist                    Error **********                    ERROR: relation "am_api_sequence" does not exist SQL state: 42P01Reference: https://wso2.org/jira/browse/APPM-1017

Not related to IoT server, hence closing the issue.


Security Error

Describe the bug
When i upload the msapp File i get the following Error Message:

Is this a known Error?
To Reproduce
Steps to reproduce the behavior:

Powerapp
Save as msapp File
Launch Powerdocu
Upload msapp File

Somebody else reported this issue before (https://github.com/modery/PowerDocu/issues/43), but I haven't figured out yet what causes this. I'll try to find out a bit more to see how we can test this properly

I have this very same issue when running it on a Windows 10 VM in Boxes in a Fedora machine. I get it for both the self-contained one and the other one. I attempted to install the graphviz module from graphviz.org with no avail...


deps: Update dependency com.google.auth:google-auth-library-credentials and google-auth-library-oauth2-http to v1.16.1

1.16.1 has been available since start of April 2023. Hopefully a patch bump is not too contentious. Happy to bump to 1.17.0 too, although that seems to only have been available for about 10 days at the time of writing.

[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)

Output of mvn dependency:tree from the gax-java folder from master:

Output of mvn dependency:tree from the gax-java folder from this branch:

Closing this one - made a hash of it

See https://github.com/googleapis/sdk-platform-java/pull/1748


Initial version 0.1.0 is ignored

TL;DR
When I use the manifest to set the initial version to 0.1.0 it is ignored when a new PR is created even though I have bump-minor-pre-major: true and bump-patch-for-minor-pre-major: true
Expected behavior
Expected next version is v0.1.1
Observed behavior
Next version in the PR is v1.0.0
Action YAML
name: Release
on:
    push:
        branches:
            - master

jobs:
    release-please:
        name: Prepare Release notes
        runs-on: ubuntu-latest
        outputs:
            released: ${{ steps.release-please.outputs.release_created }}
            tag: ${{ steps.release-please.outputs.tag_name }}
        steps:
            - id: release-please
              uses: GoogleCloudPlatform/release-please-action@v3
              with:
          # A repository with a version.txt and a CHANGELOG.md
                  changelog-types: '[{"type":"feat","section":"Features","hidden":false},{"type":"fix","section":"Bug Fixes","hidden":false},{"type":"chore","section":"Miscellaneous","hidden":false}]'
                  release-type: simple
                  package-name: package1
                  bump-minor-pre-major: true
                  bump-patch-for-minor-pre-major: true

Log output
Run GoogleCloudPlatform/release-please-action@v3
  with:
    changelog-types: [{"type":"feat","section":"Features","hidden":false},{"type":"fix","section":"Bug Fixes","hidden":false},{"type":"chore","section":"Miscellaneous","hidden":false}]
    release-type: simple
    package-name: package1
    bump-minor-pre-major: true
    bump-patch-for-minor-pre-major: true
    config-file: release-please-config.json
    manifest-file: .release-please-manifest.json
    token: ***
    fork: false
    clean: true
    monorepo-tags: false
    draft: false
    draft-pull-request: false
✔ Looking for latest release on branch: master with prefix:
⚠ Could not find releases.
✔ Building releases
✔ Building strategies by path

✔ Looking for latest release on branch: master with prefix:
⚠ Could not find releases.
✔ Building pull requests
✔ Building strategies by path
✔ Collecting release commit SHAs
⚠ Could not find releases.
⚠ Expected 1 releases, only found 0
✔ Collecting commits since all latest releases
✔ Splitting 20 commits by path
✔ Building candidate release pull request for path: .
⚠ No latest release pull request found.
✔ Considering: 14 commits
✔ Looking for open release pull requests
✔ found 0 open release pull requests.
⚠ file version.txt did not exist
✔ Starting GitHub PR workflow...
✔ Successfully found branch HEAD sha "12345678".
✔ Successfully created branch at https://api.github.com/repos/test-repo/git/refs/heads/release-please--branches--master
✔ Got the latest commit tree
✔ Successfully created a tree with the desired changes with SHA ---
✔ Successfully created commit. See commit at https://api.github.com/repos/org/test-repo/git/commits/12345678
✔ Successfully updated reference release-please--branches--master to 12345678
✔ Successfully opened pull request available at url: https://api.github.com/repos/org/test-repo/pulls/30.
✔ Successfully opened pull request: 30.
✔ Successfully added labels autorelease: pending to issue: 30

Additional information
When I installed the release-please locally and run it with same values, it created the expected version:
release-please --version
13.4.4

release-please release-pr --token=$PAT_TOKEN --repo-url=org/test-repo

.release-please-manifest.json
{
  ".": "0.1.0"
}

release-please-config.json
{
  "release-type": "simple",
  "bump-minor-pre-major": true,
  "bump-patch-for-minor-pre-major": true,
  "changelog-sections": [
    {
      "type": "feat",
      "section": "Features",
      "hidden": false
    },
    {
      "type": "fix",
      "section": "Bug Fixes",
      "hidden": false
    },
    {
      "type": "chore",
      "section": "Miscellaneous",
      "hidden": false
    }
  ],
  "packages": {
    ".": {
      "packageName": "test-repo",
      "draft": false,
      "prerelease": false,
      "bumpMinorPreMajor": true,
      "bumpPatchForMinorPreMajor": true,
      "changelogPath": "CHANGELOG.md",
      "versioning": "default"
    }
  }
}

I'm currently also running into the same issue and have not found a way to work around it.

@oatovar @maprokes I believe this may have been fixed by:
https://github.com/googleapis/release-please/pull/1267
Are you still bumping into issues?

I didn't have a chance to re-test yet, will come back here, once I give it a try. Thanks!

Yes, I tested that this is ocurring on the latest version for me but will only do so if I set the starting version to 0.0.0. If I bump this up to 0.0.1 it will work as intended.

I found a work around for this, since we're using Backstage to scaffold microservice repos. We're using release-type: simple:

Create the repo with version.txt containing 0.0.0.

This is because by time the tag is pushed, release please could have already created the PR with v1.0.0.

Run git push origin :release-please--branches--main to remove, and close, the open PR.
Bump version.txt to 0.0.1 locally.
Create a chore: bump initial release version commit.
Tag this with v0.0.1.
Push the commit and tag.

This will create a release that is v0.0.2, but that is better than going right to v1.0.0.

I observe the same behavior as OP but where I have only set bump-minor-pre-major to true, while leaving bump-patch-for-minor-pre-major to its default value of false.
Only workaround I know to work is adding Release-As: 0.1.0 in commit message.

Closing as stale. We highly advise using the manifest for configuration options.


Remove unused pytest skip

I originally included the pytest.skip because in rare cases (I think debug builds of Cpython??) docstrings can return None instead of their contents. I think this is rare enough that we don't need to care about it, and it's better just adding a # type: ignore comment instead of a pytest.skip.
I'm hoping this gets code coverage back up to 100% again to fix https://github.com/zarr-developers/numcodecs/issues/646

Thanks David! 🙏
This seems reasonable
If we think we do need this condition, we could also add a # pragma: no cover


Validate parameters of randi_range()

Fixes #29688

I believe we should allow from > to. This require a simple inequality check.
The inverted situation only requires to invert the two numbers and to make sure the "from" is still included while the "to" is not.

Now is better? Reversing the order produces the same results.
Also

make sure the "from" is still included while the "to" is not.

Why? The method is inclusive.

Why? The method is inclusive.

Ah yes, you are right, I though it was not :)

Thanks!


OIPA IDs are shown in various places, but usually named "ID". These could easily be confused with other identifiers

Describe the bug
There’s justification for exposing OIPA identifiers to users, but it should be made clear that that’s what they are. "ID" as a field name could reasonably be confused with iati-identifier. It would be clearer to namespace these by referring to oipa-ids or datastore-ids everywhere.

id is generated by Django. We can not change this. Discussed option to hide it, but we use it. So therefore won't do.

id is generated by Django. We can not change this.

It’s true that it’s generated by django, but it’s not true that you cannot change it. Changing it is documented here:
https://docs.djangoproject.com/en/2.0/topics/db/models/#automatic-primary-key-fields
Specifically:

If you’d like to specify a custom primary key, just specify primary_key=True on one of your fields. If Django sees you’ve explicitly set Field.primary_key, it won’t add the automatic id column.

Sure, but we are not too sure on the impact and not sure about a graceful renaming of that 'id' at this moment. Perhaps you could test it and let us know @andylolz ?

Perhaps you could test it and let us know @andylolz ?

Sorry – I won’t have time to do that.

Sure, but we are not too sure on the impact and not sure about a graceful renaming of that 'id' at this moment.

Probably far easier to change before launch than after launch. (After launch, it will be extremely difficult to change.)
Impact totally depends on what else is using that attribute.

We'll probably just remove it from the end-point.


endpointKey is populated with the subscriptionId value in .bot file using Azure login

Version
4.2.1
Describe the bug
When adding QnA as a service via Azure login the endpointKey is populated with the subscriptionId value. Subsequently, the actual endpointKey key:value is not being stored in the .bot file.
To Reproduce
Steps to reproduce the behavior:

Open your .bot file in Emulator
Select 'Add QnA Maker' under Services and login via Azure
Select a knowledge base and add it
Observe added JSON in .bot file

Expected behavior
Both the endpointKey and subscriptionId keys should have the appropriate values
{
  "type": "qna",
  "endpointKey": "subscriptionId",
  "hostname": "hostname",
  "id": "id",
  "kbId": "kbId",
  "name": "name"
}

[bug]

@stevkan I have not been able to reproduce this issue following the steps provided. The value I see for endpointKey corresponds to a key in my cognitive services azure resource.

@stevkan I see it now. I was unaware of the endpoint key on the QnA side. Let me dig into this a little more.


Use a the more generic name "jenkins/google" for Jenkins commit status

The Google-run PR Jenkins is now starting to run verification checks
and unit/integration tests in addition to the e2e tests, so we should
make the name more generic.
This also sets precedent for other folks (like RedHat) who are
interested in setting up PR Jenkins bots.
@kubernetes/goog-testing @kubernetes/rh-cluster-infra @brendandburns

Assuming we merge this, we'll need to coordinate updating the munger and submit queue at the same time as the Jenkins config. (I haven't changed the name of the commit status on Jenkins yet.)

I love it. LGTM as soon as you can get @brendandburns or @lavalamp to build a new one.

We haven't turned the submit queue on yet (right @brendandburns?) so change jenkins, let us know, and we'll build before we turn it on.

This affects both the munger and the submit queue. Even if the submit queue isn't running at the moment, I think the munger still is.

rebased after the major refactoring by @eparis. PTAL?

submit-queue_test.go needs updating. still LGTM.  But you're going to need @lavalamp or @brendandburns to coordinate/restart the munger. I can't help there and don't want to merge...

Fixed test.


[Tour of Beam] [Task] Load example metadata in ToB to use precompiled objects

What needs to happen?
Tour of beam should load not only snippets but also all example metadata to show precompiled objects if any
Issue Priority
Priority: 3
Components:
tour-of-beam

.take-issue


TTS feature request!

Could you please add a text to speech feature as this could prove helpful to users with reading disabilities or handicaps

#244
Please stop opening duplicate issues or ill be forced to block you. Spamming issues will not make it come faster. TTS is in the works/


add functions for converting magnitudes to counts, getting noise levels for images, etc.

cc @esheldon

Sounds like a set of useful functions related to pure survey numbers. Thanks for suggesting :)
You were mentioning that these are already implemented somewhere. Is this in https://github.com/LSSTDESC/WeakLensingDeblending ?

There are functions in there, but IDK if they are correct for the adopted units / conventions here:
https://github.com/LSSTDESC/WeakLensingDeblending/blob/master/descwl/survey.py#L125

There is also lots of stuff done in the init here: https://github.com/LSSTDESC/WeakLensingDeblending/blob/master/descwl/survey.py#L47
Some of that appears useful.

Maybe this: https://github.com/LSSTDESC/WeakLensingDeblending/blob/master/descwl/survey.py#L110

Right, so for a given survey you'd like

[ ] a method that takes an AB mag and returns the number of counts
[ ] a method that takes a sky_brightness and returns the mean_sky_background
Am I correct ?

@esheldon should confirm, but I think this is the whole thing.


Compatibility with pip 10+

updating setup.py and requirements.txt to be compatible with pip>=10

setup.py install worked on my local machine, trying to fix make commands.


Citations

Carbon
Memory effects of climate and vegetation affecting net ecosystem CO2 fluxes in global forests
https://journals.plos.org/plosone/article?id=10.1371/journal.pone.0211510
Exploring the Potential of Long Short-Term Memory Networks for Predicting Net CO2 Exchange Across Various Ecosystems With Multi-Source Data
https://agupubs.onlinelibrary.wiley.com/doi/abs/10.1029/2023JD040418
Photosynthesis
Understanding and Modeling Climate Impacts on Photosynthetic Dynamics with FLUXNET Data and Neural Networks
https://www.mdpi.com/1996-1073/13/6/1322

LSTM for soil respiration
https://www.frontiersin.org/journals/environmental-science/articles/10.3389/fenvs.2022.1035540/full


Santize some inputs and clarify docs

This PR sanitizes a couple of inputs and adds some docs clarifications.

@mbien thanks for the review


infowindows on obs search don't show translated common name

...even though the snippet does, e.g. http://www.inaturalist.org/observations?place_id=10870&quality_grade=research&locale=it

the Rails part of the fix is 23e6388 and the API part in inaturalist/iNaturalistAPI@28ac5040 . I'll update the production API servers then this is ready for deploy. It's running on Staging now


[error]: 5次重试后依然出错:response.text='',Expecting value: line 1 column 1 (char 0)

今天翻译第一个的时候正常，但是第二次就这样：
[error]: 5次重试后依然出错:response.text='',Expecting value: line 1 column 1 (char 0)

Obviously, this error does not come from DeepLX. Please provide the error from DeepLX, as I cannot make judgments on errors from any other clients.


feat: add public authorization

Add authorizations to public endpoint.
We will have to create another policies, but this is just the start.

Coverage increased (+0.04%) to 85.433% when pulling ad3d6a0bcc853d87cd1ecd19c6570fb7bdd2e63b on feat/public-authorization into 6f99a1d62e065b4a6f2ae4d3c90d03a33209d926 on master.


parent for GO:1990565 HSP90-CDC37 chaperone complex?

On Paola's advise, I'm adding it here. To be discussed on Editors' call.
GO:1990565 HSP90-CDC37 chaperone complex currently hasn't got a logical parent apart from 'protein complex'. I would like to add something more specific but we only know the process rather than the function of the complex so we can't use our usual complex_by_activity inference.
[Term]
id: GO:1990565
name: HSP90-CDC37 chaperone complex
namespace: cellular_component
def: "A protein complex involved in stress-induced mitophagy (mitochondrial degradation). In mammals it consists of heat shock protein HSP90 and its co-chaperone CDC37. Experimental evidence suggests that the complex enables the autophosphorylation of ULK1. Phosphorylated ULK1 in turn phosphorylates ATG13 which is an essential step in mitophagy. HSP90-CDC37 binding prevents rapid ubiquitin-dependent proteosomal degradation of its targets. " [GOC:bhm, IntAct:EBI-9990131, IntAct:EBI-9990540, PMID:21855797, PMID:22939624]
comment: An example of this is HSP90AB1 in human (P08238) in PMID:21855797 (inferred from direct assay).
subset: termgenie_unvetted
is_a: GO:0043234 ! protein complex
is_a: GO:0044444 {is_inferred="true"} ! cytoplasmic part
relationship: capable_of GO:0019887 ! protein kinase regulator activity
relationship: capable_of_part_of GO:0032435 ! negative regulation of proteasomal ubiquitin-dependent protein catabolic process
relationship: capable_of_part_of GO:0045737 ! positive regulation of cyclin-dependent protein serine/threonine kinase activity
relationship: capable_of_part_of GO:1903599 ! positive regulation of mitochondrion degradation
relationship: part_of GO:0005737 ! cytoplasm
I would like to create a term that describes its pos reg action in the phosphorylation chain that leads to mitophagy initiation. 'regulatory complex' is too generic and 'protein kinase complex' slightly out of scope as the paper does not describe the actual kinase activity, just that the inhibition of HSP90 leads to a drastic reduction of phosphorylated downstream targets.
Please discuss!
Birgit
Reported by: bmeldal
Original Ticket: geneontology/ontology-requests/11356

Hi @ValWood: @bmeldal proposes a NTR "chaperone complex" defined as
A protein complex required for the non-covalent folding or unfolding, maturation, stabilization or assembly or disassembly of macromolecular structures. Usually active during or immediately after completion of translation. Many chaperone complexes contain heat shock proteins.

Comment: An example of this is HSP90AB1 in human (P08238) in PMID:21855797 (inferred from direct assay).

is_a: GO:0043234 ! protein complex
relationship: capable_of GO:0044183 protein binding involved in protein folding

@paolaroncaglia mentioned you were part of the 2008-2009 chaperone discussions; would you be able to weigh in as to whether this term is ok or not in the light of those?

Yes. This supports my request that
GO:0044183 protein binding involved in protein folding
be renamed to
"protein folding chaperone activity" to be more understandable to biologists....

Looking at metallochaperone complex - should they infer under chaperone complex?

I created chaperone complex and changed the definition of HSP90-CDC37 chaperone complex/
I moved the existing chaperone/chaperonin under chaperone complex.
Question remains for metallochaperone complexes, currently defined as 'macromolecular complex' and ('capable of' some 'metallochaperone activity'). If needed, please open a separate ticket.

Actually I think there could be a problem. Chaperone isn't a single activity, its a collection of activities.
So the def:
A protein complex required for the non-covalent folding or unfolding, maturation, stabilization or assembly or disassembly of macromolecular structures. Usually active during or immediately after completion of translation. Many chaperone complexes contain heat shock proteins.
Is a problem to logically define using
GO:0044183 protein binding involved in protein folding
which refers specifically to a "protein folding chaperone"
I'm not completely sure what the solution is for chaperones.
I always feel we need more precise terms for the activities of various flavours of chaperone (of which metallochaperone is one).
This one isn't going to be an easy one....

I think that when we say "chaperone complex capable of GO:0044183 protein binding involved in protein folding" we don't imply that it is the only thing it is capable of, so this is at least not wrong. We could add different activities if needed?

OK go with it. The rest is for another day ;)

well I still would like it to be clean if possible ;)
If we want to keep the general chaperone complex term we should make sure that, as you mention,  other flavours of chaperone infer properly under it - at least for those we already have - thus the question for the metallochaperone. I'll create a new ticket to not confuse this one. Thanks for the feedback @ValWood !


Add Owen Campbell-Moore to contributors

This change is

I signed it!

I think we need to be alphabetical order.

Yah, technically it's supposed to be alphabetical by last name.

I'll handle now so we can get the push out quickly, replaced by #2860


Update to Jedi 0.17.1

This brings completions for Django (via django-stubs, which is now included within Jedi) as well as support for Python 3.9 and various bugfixes (mostly around generic type annotations).
As Jedi now includes django-stubs I've included that project's license in the third party distribution file, but I am by no means a license expert so please do check that that's the right way to handle this.

@PeterJCLaw Can you update the new item name to 12486.md? I created this item to match the enhancement #12486


Port ARMS auton selector to VOSS

The GUI autonomous selector from ARMS needs to be ported to VOSS. It should be thread safe and only able to be instantiated once in the project.

[ ] Create Selector singleton class
[x] Port existing selector code
[x] Ensure thread safety using Mutex

Selector does not use Singleton class, as this conflicted with liblvgl's API in multiple ways. The port of the auton selector remains similar in usage to what it was in ARMS.

Tested single screen and it works properly


Middlewares and Logging support

Introduction
This PR includes middleware functionality for routes and includes an initial implementation of a logging middleware.
Linked Issues
resolves #17
Dependencies
Test

[x] Tested Locally
[x] Documented

Review

[x] Ready for review
[x] Ready to merge

Deployment

resolves #14


{Monitor} az monitor autoscale: Upgrade monitor autoscale version and support predicative metric show

Related command

az monitor autoscale
Description

Support for predictive autoscale
Testing Guide

History Notes

[monitor] az monitor autoscale: Upgrade monitor autoscale api version
[monitor] az monitor autoscale: Add predictive metric show cmd

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

closed https://github.com/Azure/azure-cli/issues/22794

Monitor


Running Invoke-SSHCommand in background fails

Hi,
I need to run several ssh commands to different hosts in parallel. Thats why I'd like to run them in background, but that fails.
I run:
#$mySession = New-SSHSession -ComputerName $myHostname -Credential $Credentials
#start-job -scriptblock {Invoke-sshcommand -SSHSession $using:mySession -Command "some nice command"}

It starts a job in the background and when I check it with receive-job I see:
Cannot process argument transformation on parameter 'SSHSession'. Cannot convert value "SSH.SshSession" to type "SSH.SshSession[]". Error: "Cannot convert value
"SSH.SshSession" to type "SSH.SshSession". Error: "Cannot convert the "SSH.SshSession" value of type "Deserialized.SSH.SshSession" to type "SSH.SshSession"."

Is there a way to run Invoke-SSHCommand in background ?
The As-Job parameter does not seem to exist.

Because a Job is in its own PS Runspace, you would need to load the module in to each Job, make the connection and run the command I believe.

On Jan 31, 2023, at 10:53 AM, made1990 @.***> wrote:
Hi,
I need to run several ssh commands to different hosts in parallel. Thats why I'd like to run them in background, but that fails.
I run:
#$mySession = New-SSHSession -ComputerName $myHostname -Credential $Credentials
#start-job -scriptblock {Invoke-sshcommand -SSHSession $using:mySession -Command "some nice command"}
It starts a job in the background and when I check it with receive-job I see:
Cannot process argument transformation on parameter 'SSHSession'. Cannot convert value "SSH.SshSession" to type "SSH.SshSession[]". Error: "Cannot convert value
"SSH.SshSession" to type "SSH.SshSession". Error: "Cannot convert the "SSH.SshSession" value of type "Deserialized.SSH.SshSession" to type "SSH.SshSession"."
Is there a way to run Invoke-SSHCommand in background ?
The As-Job parameter does not seem to exist.
—
Reply to this email directly, view it on GitHub https://github.com/darkoperator/Posh-SSH/issues/507, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD7IHU7E77LFIZTIP2LYFTWVEROBANCNFSM6AAAAAAUMRBHBQ.
You are receiving this because you are subscribed to this thread.

@darkoperator  Thanks for your response.
The problem is that I need to execute a ssh command on several target hosts at once. Thats why I prepare the SSH-Session for all targets in a first step, because setting up the ssh session varies in time for the number of targets. I only wanted to run the ssh command as a job to ensure (more or less) simultaneous execution. That is not possible if I have to setup the ssh session in every single job.


azure-cli-telemetry depends on outdated version of portalocker

The Python package portalocker was just updated upstream to version 3.0.0 [1] while azure-cli-telemetry still depends on versions earlier than version 3:
    install_requires=[
        'applicationinsights>=0.11.1,<0.12',
        'portalocker>=1.6,<3',
    ],

Could you please update azure-cli-telemetry to the latest version of portalocker and release a new version?

[1] https://pypi.org/project/portalocker/#history

Thank you for opening this issue, we will look into it.


collate

Coverage remained the same at 82.759% when pulling 69466b030a1da41841eb31f443cd86d17273442d on sidhmads:collate into 339ad92cf0b2b8bc8ca1a5e62f448b1eaeea70cc on CS2103AUG2017-T11-B1:master.


RFXtrx Sensor not showing all sensors

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version): 0.36-DEV
Python release (python3 --version):
Component/platform: rfxtrx sensor
Description of problem:
Maverick BBQ Thermometer not visible in HASS.
It is using protocol Hideki and visible in RFXMngr
Expected:
Food/BBQ temperatures shown as sensors, preferably battery level too
Problem-relevant configuration.yaml entries and steps to reproduce:
   platform: rfxtrx
   automatic_add: True

log:
17-01-01 18:29:52 homeassistant.loader: Loaded sensor.rfxtrx from homeassistant.components.sensor.rfxtrx
17-01-01 18:29:53 homeassistant.components.sensor: Setting up sensor.rfxtrx

Traceback (if applicable):

Additional info:

@Danielhiversen I think you have good insight in this?

@Danielhiversen I think you have good insight in this?

Is it working with other rfxtrx sensors?

Is it working with other rfxtrx sensors?

@Danielhiversen Unfortunately this is the only sensor I have that is supposed to work with rfxcom, the main reason I´ve started the move from Tellstick.
But I think I read in the community other people having issues with rfxcom sensors.

Your config should be like this:
rfxtrx:
  device: PATH_TO_DEVICE

sensor:
  platform: rfxtrx
  automatic_add: True

If you sure your setup is correctly, then your sensor is probably not supported by the library we are using.
Then you have to add support for it here: https://github.com/Danielhiversen/pyRFXtrx

Probably not my units but I guess these are RFXCom sensors but the value is "Motion"?
What do you mean by that?

Hi @Danielhiversen
My config looks like that yes.
Ok, how can I help to add support for Hideki protocol? I´m not a developer.
I also got a couple of Rising Sun switches (Lightning4) that are not working.
NEXA switches are fine though.

Running your receive.py I can see that I receive some codes:
Recv: 0x0a 0x4e 0x01 0x06 0xfc 0xd8 0x00 0x13 0x00 0x13 0x79
Is it possible to see what unit that is sending these?

Then it seems that we currently do not support your sensor.
Then someone have to add support for it to the rfxtrx library.

@Danielhiversen  with the rfxtrx library you mean your repository?
https://github.com/Danielhiversen/pyRFXtrx
Should I add a request in our community?
I can see that both OpenHab and Domoticz have support so hopefully HASS can support it too
https://github.com/openhab/openhab/wiki/RFXCOM-Binding
http://www.domoticz.com/wiki/Hardware

Yes, my library.

OK!
Would it be possible for you to do it?
If it can make it easier I can check how much it will cost to send the thermometer to Norway.


splitTransition elements aren't removed after 1 sec

TransitionSignal fires a handler after 950ms that should remove elements in the TransitionStatus.Removing state.  They aren't removed in that handler because refireMemoized is called before they are removed from the data structures.  The fix seems to work in my tests.  See this change:
https://github.com/mattjacobus/animus/commit/6989b995dc8cc61c186ee71a017ed588ca4ca1c6

Thank you! I merged the changes :)
I need to rework the transitioning signal at some point :D


protobuf/encoding/protojson.Marshal outputs different comma spacing in between runs/on different 1.16.* versions

What version of protobuf and what language are you using?
google.golang.org/protobuf v1.26.0
go 1.16.*
linux
What did you do?
Generated JSON from a protobuf message on go 1.16.6 and got different output strings depending on calling code. See https://github.com/pigfrown/protojsonissue/ for minimal example + output. The JSON strings sometimes have spaces after commas, sometimes they don't.
Although output is still valid JSON the different output breaks my unit tests when they are running on different machines/different versions of go. In particular tests pass on local machines but fail on github actions.
What did you expect to see?
The same JSON output regardless of which patch version of go 1.16 is used.
The same JSON output regardless of the context in which protojson.Marshal is called.
What did you see instead?
Running the example (https://github.com/pigfrown/protojsonissue/) on different go version (1.16.6 and 1.16.7) produces different output. (spaces or no spaces after commas in generated JSON).
When running on 1.16.6: Modifying the example (https://github.com/pigfrown/protojsonissue/) to call protojson.Marshal twice instead of once produces different spacing in the output JSON, even though the input message is identical.
Anything else we should know about your project / environment?
Updating to 1.16.7 appears to have fixed the trivial example but the more complicated example in my unit tests still exhibits this behaviour.

This is intended behavior. From the FAQ:

Why does the output of protojson keep changing?
We make no promises about the long-term stability of Go's implementation of the JSON format for protocol buffers. The specification only specifies what is valid JSON, but provides no specification for a canonical format for how a marshaler ought to exactly format a given message. To avoid giving the illusion that the output is stable, we deliberately introduce minor differences so that byte-for-byte comparisons are likely to fail.
To gain some degree of output stability, we recommend running the output through a JSON formatter.


Fix AggregateRel validate rel without group and functions and function signature not match

LGTM. Just want to make sure if this PR is tested on Gluten CI, or TPC-H/DS workload.

Yes, it has passed gluten CI and test in jenkins TPCH


fix(yaml) - correct escaping behavior in single-quoted YAML strings

Fixes wrong escaping behavior in single-quoted strings
Resolves #4068
Changes
Before, backslashes in single-quoted strings were incorrectly treated as escape characters:

Now single-quoted strings follow YAML specification where only single quotes can be escaped (by doubling them):

Checklist

[x] Added markup tests, or they don't apply here because...
[x] Updated the changelog at CHANGES.md

Lets also make sure we add a single quote escaping to our markup tests - esp if we go with char.escape.


Ogre 2.2.4

Here is the pull request for ogre 2.2.4 compatibility.
Cmake scripts have been update and work with Ogre source.
Some refactoring in textureloader and materialloader has been done.
Let me know if anything has to be changed.

I need to get the build environement for this sorted out, both locally and in CI 😄
I may revew that no sooner than this weekend, gonna be busy with "real" work 😅


null pointer write doesn't break until verification stage

as discussed with jeremy,
[2023-06-13T11:20:23Z TRACE risc0_zkvm::prove::exec] [138325] pc: 0x0701f5ac, insn: 0x00072583 => lw x11, 0(x14)
[2023-06-13T11:20:23Z TRACE risc0_zkvm::prove::exec]   imm=0x00000000, rs1=x14 -> 0x00000000, rs2=x0 -> 0x00000000, Addr = 0x00000000, lowBits = 0, loaded = 0x00000000
[2023-06-13T11:20:23Z TRACE risc0_zkvm::prove::exec]   fillByte = 0x00, extended: 0x00000000

[2023-06-13T11:23:45Z DEBUG risc0_zkp::verify::host] > compute_polynomial
[2023-06-13T11:23:45Z DEBUG risc0_zkp::verify::host] < compute_polynomial
thread 'main' panicked at 'called Result::unwrap() on an Err value: Verification indicates proof is invalid', host/src/main.rs:36:35

"Oh, interesting.  The zero page is a fault to read or write (purposely), but I wonder if the witness generations isn’t tightly constrained enough on that, but the actual proof polynomial is.  That would explain the behavior.  Do you have a repo you could send?"
repo here: https://github.com/stskeeps/nullaccess-bug
uarch_combined.o is rv32im compiled object made from the corresponding c file w/o optimization, forcing a null pointer write

@SchmErik This might be related to #672
Generally we need to improve the executor esp. regarding memory accesses.

@SchmErik This might be related to #672

Yeah, I think this is related but I think this is also a separate issue in that the prover allows a null pointer to be written.

Generally we need to improve the executor esp. regarding memory accesses.

Fixing this in the executor means that it won't pass through the prover if you're using the executor and prover like we suggest in the starter template but I think we may want to look at patching the prover too

The prover currently should prevent null pointer writes, because this is enforced by circuit constraints. It's the executor that's currently allowing it I think.

@jbruestle Is there potentially a bug in the prover here?

Removing the fixes tag from #727 for now


Guild statistics

Is your feature request related to a problem? Please describe.
There is a dire need to replace Statbot in the guild I run the bot instance in.
Describe the solution you'd like
This extends the functionality of the leveling system we have planned (#138) to provide guild message insights. Math skills is a requirement before digging in. The article we provided below is an example of what could be included. Some ideas we also have include:

Displaying top message users
Displaying top message channels
Display top voice channel users
Displaying top voice channels
Diagrammatic analysis on the above
Filter the channels/members that are shown in the stat display
Ability to opt out of being tracked

Traditional level roles are rigid and one-way: users keep their roles forever once obtained, even if they later become inactive. Instead this will be using a member's stats over a range of time to reward and remove roles. They make it easy to reward the members who keep coming back to the community.
This will not be available as a user app.
Describe alternatives you've considered
Not an alternative, but this rather extends the functionality if we do provide it. Support for tracking member activities and statuses and the duration of each individual activity/status over a span of time. Can store this data forever or for a set period of time.
Additional context
Relevant article on what this is: https://support-dev.discord.com/hc/en-us/articles/360043053492-Statistics-Bot-Policy

Image manipulation is also an implicit requirement before making progress on this, which itself should deserve it's own issue because of it's dependency on other issues too.

Realistically speaking, we will not begin working on this until next year most likely.


Replace RocksEngine with KvEngine in lock manager

What problem does this PR solve?
More work on making tikv generic over engines.
https://github.com/tikv/tikv/issues/6402
What is changed and how it works?
This replaces all uses of RocksEngine in tikv::server::lock_manager with either impl KvEngine, or, in test cases, KvTestEngine.
Related changes
Check List
Release note

No release note.

/merge


Provision Jammy VMs

Reasoning
We need to upgrade all our boxes to Jammy; we're starting to be unable to get apt packages, etc.
Boxes to Migrate
These are the names of the existing boxes. We can change the names to use current conventions, suggestions inline.
Staging
dpul-staging1 => migrate to dpul-staging3?
dpul-staging2 => migrate to dpul-staging4?
Production
dpul1 => migrate to dpul-prod1
dpul2 => migrate to dpul-prod2

Existing dpul-staging boxes have:
16 GB of memory
40 GB hard drive
dpul-staging1 has 4 CPUs; dpul-staging2 has 2 CPUs
I made the new staging boxes with 4 CPUs, 16GB mem, and 30GB drives.
Existing dpul production boxes have:
2 CPUs
32 GB of memory
40GB hard drive
I made the new prod boxes with 4 CPUs, 32GB mem, and 30GB drives.
VMs are all done, they just need firewall rules added.

Jammy DPUL boxes are all ready to go. They currently have lib-vm as the hostname, but running the playbooks will fix that. Let me know if you run into any issues.

Hint for Future Me - in each environment, only one VM needs port 6379 open for Redis as well as the usual SSH, HTTP/HTTPS ports.

We're reopening these, but just so we can keep track of doing the provisioning of these now that they're made.


New vk.com design fails scrobbling

Hi!  Seems like after vk.com made new design, scrobbling is not working anymore.

Resolved in #1039.

As for me, VK scrobbling still doesn't work.

Is it should be? These changes will be in next release.

Temporary workaround for those who can't wait for new release:

Go here
Download ZIP

Unpack ZIP somewhere

Go to chrome://extensions/ and enable developer mode

Press Load unpacked extension... and select the unpacked folder

You may now go to vk.com and listen to music.

This is now fixed right?

merge needed #1039


Definition of NashConv (in Mean-Field Games)

Dear authors,
It seems to me that the code for nashconv only works when the initial distribution is uniform. For other initial distributions, this definition, summing up the deviation payoffs over all states, may be improper since states may associate with different probabilities at the beginning. I just want to note this since the current available MFG games in Openspiel seem to start from the uniform distribution. Thanks.
https://github.com/deepmind/open_spiel/blob/e3a597b1d5e34a3130f36fa833702a1ae720a83f/open_spiel/python/mfg/algorithms/nash_conv.py#L53

@perolat, can you take a look?

Hi @wyz2368,
The sum is over population's initial states and not the initial distribution's state which will appear at the next step. It is the analogue of the following line in N-player games.
https://github.com/deepmind/open_spiel/blob/e3a597b1d5e34a3130f36fa833702a1ae720a83f/open_spiel/python/algorithms/exploitability.py#L196
We do have a list of initial states in a multi-population mean field game (one for each population). This initial state (given a population) is a chance node which actions are to choose the first state according to the initial distribution.
So you can define something else than a uniform initial distribution and have the right exploitability. Like in the following game:
https://github.com/deepmind/open_spiel/blob/e3a597b1d5e34a3130f36fa833702a1ae720a83f/open_spiel/games/mfg/crowd_modelling_2d.cc#L37
Hope it helps!

Thanks @lanctot and @perolat !
This helps. I raised this question because my understanding about NashConv is that NashConv is the sum of regrets over players, where regret for one player is defined as the difference between the expected deviation payoff (fixing opponent strategies or players' distributions in MFG) and the expected payoff of the current strategy. Since we calculate the payoffs in expectation, the initial state distribution plays a role. Therefore, calculating NashConv by summing over initial states is correct only when the initial state distribution is uniform (ignoring scaling). For other initial state distributions, NashConv should be calculated through weighted sum following the initial state distribution.

Hi @wyz2368, was it clear to you after Julien's response that this line:
https://github.com/deepmind/open_spiel/blob/e3a597b1d5e34a3130f36fa833702a1ae720a83f/open_spiel/python/mfg/algorithms/nash_conv.py#L59
is a sum over populations, not over states initial states from the distribution?
The expectation over the individual distributions of initial states gets factored at the next step (after you decide on the population).

Oh, I see. Thank you very much @lanctot . I think I misunderstood @perolat 's reply.
Now it makes sense to me. So there is one chance node for every population at the beginning. Then the sum over populations in MFG is analogous to the sum over players in other games.
Thanks again.

Ok, great. I would not call it a chance node over populations, though, because-- as far as I understand-- there is no distribution over populations. It's just a collection of them (like, i.e., the set of players).

I mean the chance node for each population decides the initial states at the next step. So it is a chance node over initial states and every population is associated with such a chance node. It is a no chance node over distributions. Just for clarification. Sorry for the confusion. : )

Haha, no worries, my bad.. sorry! Yes, right :)


IF-343: Asset series mandatory require not empty

Depends on changes in IF-338 and subsequent IF-346, make diff on IF-346 for review

Task linked: IF-346 Non-Mandatory Attributes/Metrics can not be deleted in AssetSeries Implementation

Task linked: IF-338 editing asset series causes error

Task linked: IF-343 Definition of "Mandatory" in ATT has no influence on Attribute or Metric Definition in AssetSeries Implementation


Update to latest filter-obj

The dependency https://github.com/sindresorhus/filter-obj used here is very old. Could this project be updated to use the latest version?

It cannot be upgraded before we target a higher Node.js version, which I don't intend to do yet.


[1.4.0-RC1] ${BASE} in paths reported from compiler breaks hyperlinking to source

steps
use sbt 1.4.0-RC1, compile a project with some warnings or errors
problem
compiler messages are reported like [warn] ${BASE}/modules/magnolia/src/test/scala/A.scala:166:18: which makes tools like intellij unable to recognize that this refers to a file and provide hyperlinks
expectation
compiler messages report full paths to files like [warn] /home/user/modules/magnolia/src/test/scala/A.scala:166:18: so that tools are able to parse the log

@OlegYch Thanks for the report. I've noticed this too.
When I originally implemented virtualization I think I did implement this in https://github.com/sbt/sbt/pull/5417/commits/3ce4d22b84aa6b4140907fcfcffcb521f82c5a81#diff-58c1ab41b5ea609053ceb7f30c2f660d, but maybe it regressed when the reporter moved to https://github.com/sbt/sbt/blob/70111bb6892e333f5427c4146797f39f904ac28d/main/src/main/scala/sbt/internal/server/BuildServerReporter.scala. /cc @adpi2
Maybe we could add a setting (default enabled) that would display the full path like current sbt 1.3.x since some people may prefer the machine-independent path.

@eed3si9n sounds like a good plan

Actually this issue is caused by the Problem value that is built by Zinc. It contains a Position made of sourceFile, the real file, and sourcePath, which happens to be the machine-indepentent path aka the virtual file id. Here is the code.
It would be rather easy to fix this issue on Zinc side by setting the absolute path of the file to sourcePath. It should be safe since it is only consumed by the implementation of Reporter (BuildServerReporter and LoggedReporter).
Then if we want to implement your proposition @eed3si9n we can use the FileConverter.

I have an alternative solution which is to use a sourcePositionMappers that will convert all Position to machine specific depending on a Boolean setting reportMachinePath. I try this out.


Update docker 18.03.0-ce -> 24.0.6

I've upgraded old docker to the newest version
Testing done

docker build . -t docker-agent && docker run -it --entrypoint /bin/bash docker-agent

docker version
docker ps

Running it with Jenkins as a agent with docker:dind and with mounted host's docker socket.

Submitter checklist

[x] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[x] Ensure that the pull request title represents the desired changelog entry
[x] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[x] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Hi, I encountered a compatibility issue when using docker-compose 1.21 with docker 24.0. The compose utilities are not picked up by the docker executable.
To replicate the issue, first enter the spawned container:
docker exec -it $CONTAINER /bin/bash

Inside the container, invoke the docker command, and under "Management Commands", compose* is not there.
If you run docker compose version, it will fail with /bin/bash: docker-compose command not found, despite the executable is under /usr/local/bin and has executable flags set.
Proposed solution:
Can you update the docker-compose version to 2.21.0? What's the reason for keeping it at 1.x?

Good catch @HaoguangYang ! The only reason is that we don't have a lot of users for this image so no one reported this issue earlier (usually Docker in Docker / Docker on Docker are really bad practices in term of security OR people use their own baked image).
Are you ok to contribute to this image to help us?


UI - Validation Updates - QA Feedback

Issue:

https://github.com/bcgov/entity/issues/23743

Description of changes:
https://github.com/bcgov/entity/issues/23743#issuecomment-2438878721
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun

Temporary Url for review: https://strr-ui-dev--pr-227-lj3xp8me.web.app


As an authorized user, I have the right to share the event in the social networks,  so that other people can easily see it and also take part in it.

Date Time
11-23-2020 15:13
Priority
Medium
Pre-requisites
1.The user is authorized and login into https://eventsexpress-test.azurewebsites.net/home/events?page=1&status=active
Description
Having created the event or joined an existing one, a user can share the event in
Facebook, Telegram, Twitter, LinkedIn to show it to other people.
1.The “Profile” tab is implemented on the left sidebar and opens a page with categories data by clicking on it.
2.On the page we can view some brief information about ourselves at the top.
3.Profile information is presented in five menus: 'Future events’, ‘Archive Events’, ‘Visited Events’, ‘Events to go’, ‘Add Event’.
4. In ‘Future Events’ and ‘Events to go’ a user can see a share button.
5. If a user presses a share button, he can share the event on social networks: Facebook, Telegram, Twitter, LinkedIn.
Acceptance criteria
Considered that I’m in the role of an authorized user,
When I am on the ‘Profile’ page,
I choose one out of five menus ‘Future Events’
In the ‘Future Events’ the system shows me the button ‘Share’
Where I can choose four buttons: Facebook, Telegram, Twitter, LinkedIn
And redirect the event announcement as a message to one of these social networks.
Additional context

Additional context: #183 As an authorized user I want to share events through social networks so than I can publish the event on my social page.


chore: bump pnpm version to v7.17.0

In particular strict-peer-dependencies should default to false - see v7.13.5 release notes

Codecov Report

Merging #1168 (616a71f) into main (9159cf4) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main    #1168   +/-   ##
=======================================
  Coverage   94.76%   94.76%
=======================================
  Files          48       48
  Lines        6912     6912
  Branches      445      445
=======================================
  Hits         6550     6550
  Misses        362      362

Impacted Files
Coverage Δ

packages/core/src/storage.ts
42.42% <0.00%> (ø)

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


Argument #4: Padding size should be less than the corresponding input dimension

I am seeing this error:
Argument #4: Padding size should be less than the corresponding input dimension, but got: padding (512, 512) at dimension 3 of input [1, 1, 1, 220]
after trying to train using this command:
python train.py --output_directory=outdir --log_directory=logdir
I am using pytorch 1.0 and python 3.6 with a single Tesla V100 gpu and I am using my own data set which I processed to be identical to the LJSpeech format and have changed the filelists accordingly.
The full log is:
FP16 Run: False
Dynamic Loss Scaling: True
Distributed Run: False
cuDNN Enabled: True
cuDNN Benchmark: False
Epoch: 0
Train loss 0 28.056627 Grad Norm 7.857424 5.01s/it
Traceback (most recent call last):
  File "train.py", line 284, in <module>
    args.warm_start, args.n_gpus, args.rank, args.group_name, hparams)
  File "train.py", line 242, in train
    hparams.distributed_run, rank)
  File "train.py", line 133, in validate
    for i, batch in enumerate(val_loader):
  File "/home/mahdeto/anaconda2/envs/tacotron2/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 637, in __next__
    return self._process_next_batch(batch)
  File "/home/mahdeto/anaconda2/envs/tacotron2/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 658, in _process_next_batch
    raise batch.exc_type(batch.exc_msg)
RuntimeError: Traceback (most recent call last):
  File "/home/mahdeto/anaconda2/envs/tacotron2/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 138, in _worker_loop
    samples = collate_fn([dataset[i] for i in batch_indices])
  File "/home/mahdeto/anaconda2/envs/tacotron2/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 138, in <listcomp>
    samples = collate_fn([dataset[i] for i in batch_indices])
  File "/raid1/mahdeto/repo/tacotron2/data_utils.py", line 61, in __getitem__
    return self.get_mel_text_pair(self.audiopaths_and_text[index])
  File "/raid1/mahdeto/repo/tacotron2/data_utils.py", line 34, in get_mel_text_pair
    mel = self.get_mel(audiopath)
  File "/raid1/mahdeto/repo/tacotron2/data_utils.py", line 46, in get_mel
    melspec = self.stft.mel_spectrogram(audio_norm)
  File "/raid1/mahdeto/repo/tacotron2/layers.py", line 76, in mel_spectrogram
    magnitudes, phases = self.stft_fn.transform(y)
  File "/raid1/mahdeto/repo/tacotron2/stft.py", line 88, in transform
    mode='reflect')
  File "/home/mahdeto/anaconda2/envs/tacotron2/lib/python3.6/site-packages/torch/nn/functional.py", line 2685, in pad
    ret = torch._C._nn.reflection_pad2d(input, pad)
RuntimeError: Argument #4: Padding size should be less than the corresponding input dimension, but got: padding (512, 512) at dimension 3 of input [1, 1, 1, 220]

Please help. Thanks!

In case someone runs into this. Turns out I had some corrupted wav files (very small but had some speech) that caused this.

Additionally this case is possible, when the wav file is a stereo, not mono. Tacotron is using 16kHz 16bit mono for training.

I had the same problem,now.
Codec:PCM S16LE(s16l)
Type:Audio
Channels:Mono
Sample rate:16000Hz
Bits per sample:16
Maybe is a codec problem,Above info is obtained from VLC,anyone can inform us the working setup?
Quite confusing,I also edited the hparam file respectively does not seem to work.
Originally 22khz,now 16kHz 16bit mono for training.
Does not seems to work.hmmmm..

Oh thanks i also suspected those weird small files it is not corrupted but,something is wrong when used as input.Solved.THANKS.

In case someone runs into this. Turns out I had some corrupted wav files (very small but had some speech) that caused this.

I am also getting this error: RuntimeError: Argument #4: Padding size should be less than the corresponding input dimension, but got: padding (512, 512) at dimension 3 of input 4 when i am doing inference: what do you mean by corrupted wav files? when i use 16khz file , i was getting sample rate error so i had to switch back to 22khz. My inference script: python inference.py --tacotron2 output/checkpoint_Tacotron2_last.pt --waveglow output/checkpoint_WaveGlow_last.pt --cpu -o output/ -i phrases1/phrase.txt
I can run the same inference with prretrained models, but not with my trained models.


Playground: type discovery broken for certain re-export patterns

In some cases when doing export * from ./some-folder type discovery looks for files in the wrong directory.
Simple playground example: https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBAbwIIF84DMoRHA5ACwEMBPA
The module being loaded has 4 files.  Each index file just does a export * from for the next file in the list, with the final file exporting a single type A.

index.ts
types/index.ts:
types/utils/index.ts
types/utils/a.ts

When the playground is discovering types for this module, it looks for the utils file in  at<EMAIL_ADDRESS>and<EMAIL_ADDRESS>instead of looking in the types directory.  I think this example has 1 more level of nesting than is required to show the issue, but is still a fairly simple reproduction.
Full code for the test package is here: https://github.com/hayes/type-discovery-bug

Files in the imported module:

playground loading utils file from the wrong directory:

Repro:
https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBAbwIIF84DMoRHA5ACwEMBPA

Expected behavior:
type discovery should look for /lib/types/util/index.d.ts instead of /lib/util/index.d.ts
Actual behavior:
playground attempts to load /lib/util/index.d.ts

Nice report, looks reasonable to me 👍🏻


Upgrade terraform

See Trello: https://trello.com/c/X4v1uBUh
Just maintenance stuff, ensuring we are using the latest and it's consistent across.
I ran terraform plan against Preview and all is :+1:

@katstevens changed to 10.x, ready for re-review


Animated List Detail Composable

Animated List Detail
This PR covers the creation of the AnimatedListDetail composable and the implementation of it within the Arch Demo.  Once final, we can pull the AnimatedListDetail code into Launchpad library to make it more available.
Note:  Since the Arch Demo did not orginally have a "view" snippet functionality I've left that part undone.  That could be a good work assignment for devs coming onto the bench as an intro to Compose.

@br-jared-stockton - Should have everything for review again.


Layer5 Logo Visibility Issue in Dark Mode

Description
Layer5 Logo Visibility Issue in Dark Mode
Expected Behavior
The layer5 logo should be properly visible in dark mode
Screenshots
Light Mode :

Dark Mode :

Enviroment:

OS: Mac Linux Windows
Browser: Chrome Safari Firefox
Version:
Device: Desktop Mobile

Contributor Guides and Handbook
The meshery.io website uses Jekyll and GitHub Pages. Site content is found under the master branch.

📚 See the Contributing to Meshery.io Website section of the readme.md and other contributing instructions, too.
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

can i work on this issue

I would like to work on this issue. Please assign it to me.

am sorry I deleted my repo by mistake ill regenerate my pr

I would love to work on this issue.

I would love to work on this issue.

Ive already created a pr @Atharv0003

@Chadha93 any more enhancements needed?

On Wed, 24 May 2023 at 7:03 PM, Atharva Shirdhankar <
@.***> wrote:

I think the Layer5 logo is going to be removed from the "Used By" section
in the next Rollout.
#1152 (comment)
https://github.com/meshery/meshery.io/pull/1152#issuecomment-1558713118
May be we can close this issue as of now.
—
Reply to this email directly, view it on GitHub
https://github.com/meshery/meshery.io/issues/1146#issuecomment-1561163206,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7GM7OA6Q46YXYLXBMDEU6TXHYEZPANCNFSM6AAAAAAXUTIZ5I
.
You are receiving this because you were assigned.Message ID:
@.***>

Yes lee had mentioned repeated call of layer5 wasn't necessary so i removed
it

Closing as completed


⚠ St. Lawrence Global Observatory CKAN has degraded performance

In 94e751f, St. Lawrence Global Observatory CKAN (https://catalogue.ogsl.ca/en/) experienced degraded performance:

HTTP code: 200
Response time: 5892 ms

Resolved: St. Lawrence Global Observatory CKAN performance has improved in 1ce7a73 after 8 minutes.


can't build

having trouble building this.
README Is also wrong as main is in cmd now and not root.
go install github.com/invopop/gobl.fatturapa/cmd/gobl.fatturapa@latest works
but if pull the src and build I get:
cd gobl.fatturapa//cmd/gobl.fatturapa && go install .
# github.com/invopop/gobl.fatturapa
../../eu.go:5:26: undefined: l10n.CountryCode
../../eu.go:35:25: undefined: l10n.CountryCode
../../address.go:36:21: invalid operation: addr.Country == l10n.IT (mismatched types l10n.ISOCountryCode and l10n.Code)
../../address.go:49:24: invalid operation: address.Country == l10n.IT (mismatched types l10n.ISOCountryCode and l10n.Code)
../../parties.go:178:20: invalid operation: id.Country == l10n.IT (mismatched types l10n.TaxCountryCode and l10n.Code)
../../transmission.go:63:47: invalid operation: cus.TaxID.Country != l10n.IT (mismatched types l10n.TaxCountryCode and l10n.Code)

It does not make sense ... what am I doing wrong :)

Hi @gedw99! This looks like an issue with the latest GOBL and the changes we had to make recently to differentiate between tax and ISO country codes (we'd falsely assumed that countries would make an effort to keep them the same 🤦) This PR will fix it: https://github.com/invopop/gobl.fatturapa/pull/37


Drop Python 3.2 support

Coverage remained the same at 94.611% when pulling 5363dc339d4750389f9c9e0b57e015e3c8818533 on enhancement/37 into 89d2ad768b6a6f9f7929bdc6b795f2fa287bfd0b on develop.

Coverage remained the same at 94.611% when pulling 5363dc339d4750389f9c9e0b57e015e3c8818533 on enhancement/37 into 89d2ad768b6a6f9f7929bdc6b795f2fa287bfd0b on develop.


Pip version not work in create

The package django-cruds-adminlte==0.0.4 is before than  4a64ca0 so create template has {% extendstemplate_father %}  so create view don't work.

New release 0.0.5 available on the pypi store ;) Thanks for noticing.


Job Update version and create Release PR fails

Issue
The job - https://github.com/q-shift/backstage-plugins/actions/runs/8422073464/job/23060495580, when executed manually using version = patch, fails
Run actions/setup-node@v4
  with:
    node-version-file: [2](https://github.com/q-shift/backstage-plugins/actions/runs/8422216057/job/23060998342#step:3:2)0.x
    cache: yarn
    always-auth: false
    check-latest: false
    token: ***
Error: The specified node version file at: /home/runner/work/backstage-plugins/backstage-plugins/20.x does not exist

Fixed with https://github.com/q-shift/backstage-plugins/pull/64


Update lgtm.yml

Codecov Report

Merging #135 (2977a45) into master (1b8c646) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #135   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files           8       8
  Lines         939     939
======================================
  Misses        939     939

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 1b8c646...2977a45. Read the comment docs.


Zend\Db\Sql\Predicate\Predicate: $nextPredicateCombineOperator + addPredicate() not working

Zend\Db\Sql\Predicate\PredicateSet::addPredicate() doesn't consider $this->nextPredicateCombineOrder, and it actually can't do so because $this->nextPredicateCombineOrder is defined in Zend\Db\Sql\Predicate\Predicate which derives from Zend\Db\Sql\Predicate\PredicateSet.
I created two test cases: https://github.com/zendframework/zf2/pull/6799
Test case 50 succeeds implicitly because the default combine order which is used by Zend\Db\Sql\Predicate\PredicateSet::addPredicate() instead of intended Zend\Db\Sql\Predicate\Predicate::nextPredicateCombineOrder is AND.
Test case 51 however fails, unveiling the observed (mis-) behaviour.

Fixed with #6799


chore(main): release 1.4.3

:robot: I have created a release beep boop
1.4.3 (2024-11-06)
Bug Fixes

era5: invalid short name for 2m temperature variable (1678a5f)
era5: remove init.py content (56f330b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/BLSQ/openhexa-toolbox/releases/tag/v1.4.3 :sunflower:


Added hover feature to Reset button

Added the following properties:
The background color must switch to orangered on mouse over.
The color of the button text must turn white on mouse over.

Excellent!
@Prachi-05 your code matches the issue description I'd put out.
The commit message you've written describes everything you've done.
However, there's one thing I'd strongly recommend: Whenever you're contributing to OpenSource projects, write present-tense, imperative-style commit messages. Here's a quote from Documentation/SubmittingPatches in the Git repo:

Describe your changes in imperative mood, e.g. "make xyzzy do frotz" instead of "[This patch] makes xyzzy do frotz" or "[I] changed xyzzy to do frotz", as if you are giving orders to the codebase to change its behavior.

I'm happy to accept this PR. Kudos and well done!


Might not want to suggest WebOptimizer anymore

The page suggests using WebOptimizer along with a few other progreams. It looks like WebOptimizer has not been touched for a while and last update was to try to get it to work on .net core 3. The current build status is failed so I am not sure suggesting this package to users is a good idea anymore.

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

ID: 36f55e9a-cd3f-83a0-f142-969dc10d3ceb
Version Independent ID: f6845f70-1ac6-2036-f213-a3b665b3d8bf
Content: Bundle and minify static assets in ASP.NET Core
Content Source: aspnetcore/client-side/bundling-and-minification.md
Product: aspnet-core
Technology: aspnetcore-clientside
GitHub Login: @Rick-Anderson
Microsoft Alias: scaddie

Hello @Eaglef90 ... This is a duplicate of https://github.com/dotnet/AspNetCore.Docs/issues/20808.


refactor: in-memory immutable storage state

This is part of the actor-model refactor. We will have a single atomic pointer Arc<State> for the storage engine. All reads will be lock free by getting the pointer, and all writes just update the pointer in the write thread. With this refactor, we can move towards an actor-based storage implementation as well as make it easier to implement compaction.

[x] refactor layer map https://github.com/neondatabase/neon/issues/4392
[x] move all states into a single struct
[x] implement the concurrency control scheme as in https://www.notion.so/neondatabase/Immutable-Storage-State-8f54a54c22654f169db8608cdbd2af61, with a wrapper lock to ensure compability
[ ] correctly implement file deletion and remove wrapper locks https://github.com/neondatabase/neon/issues/4509

file deletion is tracked in another issue and all other items are either done or have linked PRs.


Fix findAndModify tests for MongoDB 6.0.6

What should be done?
findAndModify now allow query to containing $ prefixed key, as long as they not predefined keys.
To reproduce it, make sure to run task env-pull to get the 6.0.6 version of mongodb. see https://github.com/FerretDB/FerretDB/pull/2727
Using $invalid in the query is now allowed, and it actually sets the _id with document containing $ prefixed $invalid key.
test> db.values.findAndModify({query: {"_id": {key: {$invalid: "val"}}}, upsert: true, update: {v: "replaced"}})
null
test> db.values.find({"_id": {key: {$invalid: "val"}}})
[
  { _id: { key: { '$invalid': 'val' } }, v: 'replaced' }
]

Using $eq or other keys that are valid operators return error.
test> db.values.findAndModify({query: {"_id": {key: {$eq: "val"}}}, upsert: true, update: {v: "replaced"}})
MongoServerError: Plan executor error during findAndModify :: caused by :: _id fields may not contain '$'-prefixed fields: $eq is not valid for storage.

Where?
https://github.com/FerretDB/FerretDB/blob/439ec96798e5790ffe61bae03cace6c5333839e5/integration/findandmodify_compat_test.go#L115-L117
https://github.com/FerretDB/FerretDB/blob/439ec96798e5790ffe61bae03cace6c5333839e5/integration/findandmodify_compat_test.go#L122-L124
Definition of Done

unit tests added/updated;
integration/compatibility tests added/updated;
spot refactorings done;

"What should be done?" section describes the current state but not what should be done. What should be done to resolve that issue?

This was not fixed, reopening.


Raw results?

Could you provide the raw results like in your paper?
and do you evaluate the MonTrack on COESOT or CEUTrack on FE108?

@laisimiao Raw results are in here. Actually, we have tried many times to cross-adapt those trackers; however, their performance is unsatisfactory. It may be due to the huge distribution gaps between those datasets. If any further question, feel free to ask.

Thanks. And is the event input form of MonTrack voxel ? If not, it is trouble to re-convert COESOT dataset to its specific form and evaluate.

@laisimiao It takes raw event cloud as input, correponding processing code as here. Here is one of our results for adapting MonTrack on COESOT, code as here.


Optionally ignore style tags in HTML, accept additional argument for css to inline?

👋 Hi there :)
Have you considered allowing css to inline be passed as an additional parameter?
(This might be related to https://github.com/Stranger6667/css-inline/issues/10 , but not quite the same)
The use case I'm thinking of is to possibly replace juice in mjml (I'm not affiliated, just a lib user), where css specified in <mj-style inline="inline"> is collected as a separate string that's currently passed into juice as extraCss, and inlining of other css is disabled (which makes sense because sometimes you do want to tell clients that respect style sheets to do something different from outlook)
https://github.com/mjmlio/mjml/blob/246df840f4d0fcd812e51ca55bd6bef6592cb0e6/packages/mjml-core/src/index.js#L317-L319

Hi @cheapsteak !
I didn't think about it, but having such a feature seems perfectly reasonable to me :) Correct me if I am wrong, but to support your use case, it might be:

A InlineOptions option to disable "style" tag inlining;
A InlineOptions option with extra CSS to inline;

As far as I see, the extra CSS should be in the form of qualified rules as a single string, e.g. h1 {color: green}, is it in this way in mjml?
Also, from looking at the code sample you mentioned I am not sure what insertPreservedExtraCss option does? Could you, please, help me with that?
If the things above make sense, then I can implement that :) And I will appreciate if you can provide some input, expected output samples so I can include them to the test suite :)
Cheers

A InlineOptions option to disable "style" tag inlining;
A InlineOptions option with extra CSS to inline;

yes that should work!

the extra CSS should be in the form of qualified rules as a single string, e.g. h1 {color: green}, is it in this way in mjml?

yup!

insertPreservedExtraCss

that's a "juice" parameter , i'll copy their description here :)

whether to insert into the document any preserved @media or @font-face content from extraCss when using preserveMediaQueries, preserveFontFaces or preserveKeyFrames. When true order of preference to append the <style> element is into head, then body, then at the end of the document. When a string the value is treated as a CSS/jQuery/cheerio selector, and when found, the <style> tag will be appended to the end of the first match. Defaults to true.

for emails, anything that can't be inlined should probably be thrown away, which I guess is why mjml had disabled that :)

Thanks! I implemented the abovementioned options - inline_style_tags & extra_css (in #46 and #48 ). Is there anything else I can do regarding this feature?
I think that a new release could be issued when #49 is done so you can directly use it as a replacement for juice :) What do you think?

those look awesome!!! please feel free to close this issue :)


TypeError on first login page

Bug
When Cypress tries to test the first login page, a TypeError is returned:

Versions used

epinio: main branch (https://github.com/epinio/epinio/commit/c097bd41117d15047d6f9e003f12ef8032875f2d)
rancher latest: 2.6.4
ui-dashboard-index: epinio-dev (https://github.com/rancher/dashboard/commit/3961e89042602cae1d22d57771edcc0a3f2a2a82)

It can also be reproduced without Cypress.

Tested successfully with latest epinio-dev
Thanks Richard.


Enter button to send question

The Strange thing happens when you write the second question of dialog and you press enter. Card opens instead of sending the answer.

When we highlight search field and press enter the question is send


:bug: The CLI no longer raise an unexpected exception when no encoding has been found

Unfortunate ending to the CLI script when no encoding has been found.
Given traceback:
line 192, in cli_detect
    ] if args.alternatives else x_[0].__dict__,
UnboundLocalError: local variable 'x_' referenced before assignment

Rocky mistake indeed...
In addition to the previous case that has been fixed, the CLI will now output a well JSON output even when there is no encoding that matches the content.

Codecov Report

Merging #70 (362f3b1) into master (bdb91cd) will increase coverage by 0.17%.
The diff coverage is 54.83%.

@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
+ Coverage   85.32%   85.50%   +0.17%
==========================================
  Files          11       11
  Lines        1077     1076       -1
==========================================
+ Hits          919      920       +1
+ Misses        158      156       -2

Impacted Files
Coverage Δ

charset_normalizer/cli/normalizer.py
65.21% <54.83%> (+1.77%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix handling adding multiple parametrs for key.

Fixes gh-171.

Assuming the test failed before the fix, LGTM.
@spencergibb yes, when upgrading to newer version of OpenFeign,  I haven't realised that this could now return unmodifiable list.


Fixed 'utf-8' being referenced as 'utf8'

Still getting my local environment going so this should get tested by someone else, but the references to using utf-8 in the file were mixed between 'utf-8' and 'utf8'. One was right, the other was wrong. This is a simple switch to make them all 'utf-8' as they should be. This likely would be an issue in the parent project as well.
fixes #173

Yup that would cause the issue. Python's default reader does not use unicode and the utf-8 library names the encoding function 'utf-8'

This might not be completely finished. I believe the issue is instead about the nature of the test crossing file structure types between windows/wine and linux

Found the issue, will implement and update PR shortly.


[TEST] [MINOR] Eliminate unnecessary output in ut "union an unmasked table"

Why are the changes needed?

Eliminate unnecessary output in ut "union an unmasked table"

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Thanks, merged to master


C++ Compression Level set properly

That is, through the send initial metadata op.
Fixes #6998

Worth checking qps impact before submission
On Thu, Jul 21, 2016, 7:33 PM David G. Quintas<EMAIL_ADDRESS>wrote:

That is, through the send initial metadata op.
Fixes #6998 https://github.com/grpc/grpc/issues/6998
You can view, comment on, or merge this pull request online at:
https://github.com/grpc/grpc/pull/7502
Commit Summary

C++ Compression Level set properly

File Changes

M include/grpc++/impl/codegen/async_stream.h
https://github.com/grpc/grpc/pull/7502/files#diff-0 (27)
M include/grpc++/impl/codegen/async_unary_call.h
https://github.com/grpc/grpc/pull/7502/files#diff-1 (9)
M include/grpc++/impl/codegen/call.h
https://github.com/grpc/grpc/pull/7502/files#diff-2 (12)
D include/grpc++/impl/codegen/impl/async_stream.h
https://github.com/grpc/grpc/pull/7502/files#diff-3 (486)
M include/grpc++/impl/codegen/method_handler_impl.h
https://github.com/grpc/grpc/pull/7502/files#diff-4 (15)
M include/grpc++/impl/codegen/server_context.h
https://github.com/grpc/grpc/pull/7502/files#diff-5 (9)
M include/grpc++/impl/codegen/sync_stream.h
https://github.com/grpc/grpc/pull/7502/files#diff-6 (15)
M src/cpp/server/server_context.cc
https://github.com/grpc/grpc/pull/7502/files#diff-7 (12)

Patch Links:

https://github.com/grpc/grpc/pull/7502.patch
https://github.com/grpc/grpc/pull/7502.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/7502, or mute the thread
https://github.com/notifications/unsubscribe-auth/AJpudfkrq0jR2voJaptpMzjoY45BoWI8ks5qYCvxgaJpZM4JSZAm
.

Done.
Some scenarios:
At master:
RUNNING SCENARIO: cpp_generic_async_streaming_ping_pong_secure
I0722 13:20:14.305978771   30996 report.cc:75]               QPS: 11495.2
I0722 13:20:14.307985246   30996 report.cc:79]               QPS: 11495.2 (11495.2/server core)
I0722 13:20:14.308015055   30996 report.cc:84]               Latencies (50/90/95/99/99.9%-ile): 87.6/90.0/91.7/103.0/136.5 us

At branch:
RUNNING SCENARIO: cpp_generic_async_streaming_ping_pong_secure
I0722 12:48:01.812688886   12232 report.cc:75]               QPS: 11685.9
I0722 12:48:01.814797951   12232 report.cc:79]               QPS: 11685.9 (11685.9/server core)
I0722 12:48:01.814828993   12232 report.cc:84]               Latencies (50/90/95/99/99.9%-ile): 85.2/89.2/91.3/103.2/140.6 us

At master:
RUNNING SCENARIO: cpp_generic_async_streaming_qps_one_server_core_insecure
I0722 13:23:26.412124839   31498 report.cc:75]               QPS: 202993.0
I0722 13:23:26.412972121   31498 report.cc:79]               QPS: 202993.0 (202993.0/server core)
I0722 13:23:26.412985945   31498 report.cc:84]               Latencies (50/90/95/99/99.9%-ile): 31265.1/32659.3/33487.3/36995.6/73239.1 us

At branch:
RUNNING SCENARIO: cpp_generic_async_streaming_qps_one_server_core_insecure
I0722 12:51:13.932141166   12574 report.cc:75]               QPS: 200969.1
I0722 12:51:13.934612692   12574 report.cc:79]               QPS: 200969.1 (200969.1/server core)
I0722 12:51:13.934674187   12574 report.cc:84]               Latencies (50/90/95/99/99.9%-ile): 31407.6/33185.2/34475.6/45716.1/73245.0 us

Other scenarios are similarly equivalent in terms of QPS/Lacencies.

LGTM.

test this please


cosign attach commands support providing a signature or attestation but not certificate

$ cosign attach -h
Provides utilities for attaching artifacts to other artifacts in a registry

Usage:
  cosign attach [command]

Available Commands:
  attestation Attach attestation to the supplied container image
  sbom        Attach sbom to the supplied container image
  signature   Attach signatures to the supplied container image

[...]

Use "cosign attach [command] --help" for more information about a command.

If you attach a signature:
cosign attach signature -h
Attach signatures to the supplied container image

Usage:
  cosign attach signature [flags]

Examples:
  cosign attach signature <image uri>

Flags:
      --allow-insecure-registry                                                                  whether to allow insecure connections to registries. Don't use this for anything but testing
      --attachment-tag-prefix [AttachmentTagPrefix]sha256-[TargetImageDigest].[AttachmentName]   optional custom prefix to use for attached image tags. Attachment images are tagged as: [AttachmentTagPrefix]sha256-[TargetImageDigest].[AttachmentName]
  -h, --help                                                                                help for signature
      --k8s-keychain                                                                             whether to use the kubernetes keychain instead of the default keychain (supports workload identity).
      --payload string                                                                           path to the payload covered by the signature (if using another format)
      --signature string                                                                         the signature, path to the signature, or {-} for stdin

[...]

There's no way to put a signature and accompanying certificate on the image.

When we do this, we need to make sure that the verify commands can also use the attached certificate!

@znewman01 What do we want to verify for the certificate ? Should we expect any precedence over other verifications ?

The overall workflow for external signing using a CA before the feature requested in this issue.

There's a root cert R.
Get a child cert C with keys pk and sk.
Sign the image payload P with sk to get a signature S.
cosign attach S and P to the image in the registry.
Distribute R and C out-of-band.

To verify, you run something like cosign verify --root R --certificate C --certificate-identity<EMAIL_ADDRESS>--certificate-issuer accounts.example.com and S and P are auto-discovered from the OCI registry.
The proposal here is to just stick C in the OCI registry and allow it to be auto-discovered as well. I think the verification piece will "just work" if we stick C in the "signature" object, which is how certs are distributed for keyless signing.

@znewman01
Here actually C should be a cert chain, not a single cert. There is no guarantee that C (if just a leave cert), is signed by R.
So ideally the verify command should look like this:
cosign verify --root R. And everything else (C (chain), S, P) can be auto discovered from OCI registry.
Also for the root R the underlying system's PKI path can be used.
And the admission controller should work on a similar way. Getting the accepted root CA certs from a pre-defined location/set, and everything else from the OCI registry.

@znewman01 Here actually C should be a cert chain, not a single cert. There is no guarantee that C (if just a leave cert), is signed by R. So ideally the verify command should look like this: cosign verify --root R. And everything else (C (chain), S, P) can be auto discovered from OCI registry.

Yeah, I think that's reasonable. Fortunately I think it's compatible with everything else.

Also for the root R the underlying system's PKI path can be used.

I would like that to be opt-in behavior; I think it's somewhat surprising if you don't explicitly want it. But again something you'd want in certain cases

And the admission controller should work on a similar way. Getting the accepted root CA certs from a pre-defined location/set, and everything else from the OCI registry.

Seems like a good idea.

I have created a Pull request
https://github.com/sigstore/cosign/pull/2602
To provide changes required for the enhancement needed in Attach and Verify command. Kindly review the Pull request