implement in-place page modification

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#53
#52  👈
#51
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @pepyakin and the rest of your teammates on  Graphite

Merge activity

Apr 3, 12:34 PM EDT: @rphmeier started a stack merge that includes this pull request via Graphite.


Run workflow After all package-level workflows have completed

Currently, each package has it's own workflow. From what I can tell, this is how circletron should be configured. The problem is, workflows cannot depend on one-another, such that you cannot run something after all of the workflows from been completed (after all of the package stuff has been completed).
That said, I would like to run something after all of the packages have been completed. Right now, each package is responsible for versioning and creating change logs, which it commits and tags with the version. When that happens, we end up with lots of extra commits (one for each package changed).

Ideally, after all of the packages have done what they need to do, a single job will create the "release" commit with the change log, add all of the needed tags (for each of the new package version) and push that single commit up.
Not sure how to do that. One thought would be to have a single workflow defined in the root-level circle.yml which just references jobs in the package-level circle.yml (so that those jobs may be turned into SKIP_JOBs if needed. Each of those jobs would have the steps of what would be the workflow for that package. Then, have a job that requires those other jobs to be completed before it runs (this would be the "after all" job.) The downside is that root-level circle.yml would have to change if a new package is added (it has to know about all of the packages itself). Also, not sure if it would even work.
I was wondering if circletron could somehow (not sure how, but it might involve polling the API...) trigger a new workflow after the package-level workflows have completed.
Thoughts?

The problem is, workflows cannot depend on one-another, such that you cannot run something after all of the workflows from been completed

I think this is a general problem with circleci... I'd really like it if circleci would allow the requires field of a job to refer to a job from a different workflow.
The only way to do this in circleci is to place all jobs in the workflow. So perhaps the solution that circletron should take would be to allow multiple circle.yml files to contribute to a single workflow.
So in this part of the code https://github.com/circletron/circletron/blob/main/src/index.ts#L185 instead of using mergeObject (which currently throws an error when two packages define the same workflow), it should use some kind of mergeArraysIntoObject which would concatenate the arrays instead of throwing.
Does that sound reasonable?

Yeah, that sounds like it would solve the issue by creating a single workflow which allows us to require various package "steps" to complete before running additional jobs.


Add some small refactoring changes

What changes are proposed in this pull request?
Functional:

Make capacity command load configurations before execution

Non functional:

Add MasterSelectionPolicy.Type
Add NoopCloseable
Add LOG to some classes, update private to protected to some class member variables so that the class can be overrode
Update RaftJournalServiceHandler constructor to take an additional RaftJournalSystem as an input
Add primary selectory to master context
Extract updateAccessTime in FileSystemMaster
Let notifySyncedPath return SyncState

Does this PR introduce any user facing changes?
N/A

alluxio-bot, merge this please


Safari supports HTTP/3 for all users now

Fixes #7114.
Per https://github.com/Fyrd/caniuse/issues/7114#issuecomment-2448297609

Safari now supports HTTP3 for all users, as of September 2024, no matter which version of Safari they are running.

For now I would argue for y for 18.1+, because that is the first minor version released after 2024-09:
https://developer.apple.com/documentation/safari-release-notes/safari-18_1-release-notes

Released October 28, 2024 — 18.1 (20619.2.8)

18.0, released in 2024-09, could be either way from the start, but could also have been changed while it already was out, hence me playing it safe with 18.1.
While one could argue for y for all versions with the note, I feel like that would not represent the history as a retroactive change, though there's certainly room for debate.

This PR is not accurate, since HTTP3 support did not ship with Safari 18.0. It was enabled in the same month as Safari 18.0 but it applies to all users on Safari 16.0+.
This is a weird feature. I talked to three of our engineers to nail down the historical details, and represent them in this PR: https://github.com/Fyrd/caniuse/pull/7200

@jensimmons PR is also fair:

https://github.com/Fyrd/caniuse/pull/7200
https://github.com/Fyrd/caniuse/pull/7200/files

You decide Fyrd! :)

… aaand the simultaneous comment, haha :)

I'm happy about the excitement!! Go HTTP3!
I was going to create a PR like this, the way @Schweinepriester did... just make Safari 18.0 (or he did 18.1) solid green, while the previous versions stayed they way they've been... since, you know, that's literally how every other "Foobar just shipped!" PR works for Can I Use. But then as I discussed this with our engineers, I realized, no actually the day we made this change, we made it for every version of Safari 16.0+. It's impossible for anyone who is running Safari from the last two years of releases to not have this change. So... that's why I wrote the PR at #7200 the way I did.

Yeah I went with @jensimmons 's PR in this case though certainly both approaches seemed reasonable. Thanks as always @Schweinepriester !


docs(@angular/cli): adds proxy multiple entries how-to

If there is a need to define multiple entries or otherwise more complex configuration a js file can be used to configure the proxy. These changes document how to do so.

@sumitarora Done 👍


Links in markdown open in external web browser

Version: 1.40.0
Commit: 9ca6200018fc206d67a47229f991901a8a453781
Date: 2017-12-15T12:00:00.000Z
VS Code: 1.67.0
Electron: 17.4.5
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621

Open a markdown file with relative links to other files
Click on the link

Instead of opening in ADS it tries to open it in an external web browser with a URL like this : https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/chgagnon/Desktop/sqlserver2019/content/repair/sop017-add-app-deploy-group.ipynb
(I used the SQL 2019 troubleshooting book for this, opened up the folder into the workspace and then use the Notebook viewlet to open the MD file)

I looked into this and it seems that some files in the markdown-language extension are missing, specifically media/index.js and media/pre.js.
The notebook-out directory is also missing from the markdown-math extension which is causing this other issue https://github.com/microsoft/azuredatastudio/issues/20091


Added version compatibility chart

Fixes #6071
Added a compatibility matrix (django CMS/Django/Python) to the home page of the  documentation, covering django CMS 3.0.18 and later versions.

Coverage remained the same at 82.694% when pulling 9898b7afa4e7a071b1f0a89f72cb6ba2929353d8 on evildmp:compatibility-matrix into ca96ebd20ace63958f2fd65c551118ffd01753a9 on divio:release/3.4.x.


[Bug]: Transaction stuck pending with status "Payment canceled" with no indication how to resolve

Describe the bug
Last week, I attempted to open a self-funded channel. The sent on-chain transaction didn't confirm and channel didn't open. I suspect this is because fees spiked and didn't come back down, and BT didn't use a high-enough fee or attempt to replace the transaction. After realising my channel wouldn't open anytime soon, I opened a CJIT channel and used that for some LN transactions. After a few days, the home screen still displays "transferring" balance (see first screenshot) although the self-funded channel has the status "Payment canceled", so the transfer will never complete. The on-chain transaction remains unconfirmed.
At this stage there is no indication to the user what to do. Can I get a refund? What if the transaction eventually confirms?
Reproduce
The reproduction of this depends on how BT handles a volatile fee environment and channel setup transactions that haven't confirmed.
Screenshots / Recording
1

2

Operating system
Android 13 TKQ1.220829.002
Bitkit version
114
Log output
No response

there is a new status now: Refund available.
thanks


Wordpress Permalinks

Hi,
I have wordpress installed in the path C:\Dev\server\www\wordpress. And in domains-enabled folder the wordpress.conf file.
Back in  Host File Manager, I put the address <IP_ADDRESS> and in the name wordpress.dev. And restart the server.

wordpress.dev adress doesn't work. I mean, the browser just download a download file (?);
Permalinks in wordpress not also works.

And thanks for all

Hey

Download Wordpress - https://wordpress.org/latest.zip

Extract /wordpress folder from zip to C:\server\www\

Start Servers & Access localhost/wordpress

You will be redirected to http://localhost/wordpress/wp-admin/setup-config.php

add <IP_ADDRESS> and wordpress.dev to hosts file

add the following wordpress.conf to server\bin\nginx\conf\domains-enabled folder

server
{
    listen       <IP_ADDRESS>:80;
    root         www/wordpress;
    index        index.php;
    server_name  wordpress.dev;

    access_log   logs/wordpress.dev.access.log;
    error_log    logs/wordpress.dev.error.log;

    gzip_static on;

    location / {
        try_files $uri $uri/ /index.php?$args;
    }

    # add trailing slash to "*/wp-admin" requests to make the admin interface work
    rewrite /wp-admin$ $scheme://$host$uri/ permanent;

    # prevent access to any files starting with a dot, like .htaccess
    location ~ /\. { access_log off; log_not_found off; deny all; }

    # do not log access to robots.txt and favicon.ico
    location = /robots.txt { access_log off; log_not_found off; }
    location = /favicon.ico { access_log off; log_not_found off; }

    # deny access to some files
    location ~ /(\.|wp-config.php|readme.html|license.txt) { deny all; }

    # send "expires headers" and turn off "404 error logging".
    location ~* ^.+\.(ogg|ogv|svg|svgz|eot|otf|woff|mp4|ttf|rss|atom|jpg|jpeg|gif|png|ico|zip|tgz|gz|rar|bz2|doc|xls|exe|ppt|tar|mid|midi|wav|bmp|rtf)$ {
        access_log off;
        log_not_found off;
        expires max;
    }

    # use fastcgi for all php files
    location ~ \.php$ {
        try_files      $uri =404;
        fastcgi_pass   <IP_ADDRESS>:9100;
        fastcgi_intercept_errors on;
        fastcgi_index  index.php;
        fastcgi_param  SCRIPT_FILENAME  $document_root$fastcgi_script_name;
        fastcgi_param  PATH_INFO $document_uri;
        include        fastcgi_params;
    }
}

restart nginx
Now: when you browse to http://wordpress.dev/ you will be redirected to a wrong URL http://wordpress.dev/index.php/wp-admin/setup-config.php. This one is already rewritten.
You need to remove the index.php part from it and use http://wordpress.dev/wp-admin/setup-config.php` to install Wordpress.
Follow install instructions...
Frontend: http://wordpress.dev/ & Backend: http://wordpress.dev/wp-admin/

Regards, Jens

Thanks for the support.  Well, did exactly what is above, the difference is WPN-XM installed in C:\Dev\server\
Neither wordpress.dev and permalinnk worked :/
acess.log: http://pastebin.com/wkvx3UbB
error.log: http://pastebin.com/meB2yebz

Thanks for the support.

Glad i could help.
Is the issue resolved? I couldn't find any http://wordpress.dev/ related entries in the logs.

Feel free to post, if the issue is not resolved, yet.


Fix broken link to contributing.md

Hi!
I was checking your contributing guide and found a broken link to it in README.md.

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

Thanks, @tkirill. Appreciate you providing the fix!


Fix version error message

This change addresses issue #209
Asdf throws an error message when it fails to read the version map file. The error message includes the version being searched for in the form of an undefined variable, version_string. I added a line defining this variable so that the exception doesn't throw another exception.
Please note that the most likely reason for the version map file not to be found is that the asdf-standard package has not been installed, as that is where the file is found in the distribution.

Coverage decreased (-0.3%) to 93.564% when pulling fb534078b557544973d48722ddd757f4cfe91fe6 on bernie-simon:version_string into faca21758025496bbd6a44224858af9b0970991d on spacetelescope:master.

@bernie-simon This now fails because astropy-helpers as well as .travis.yml were updated. It might be easier to create a new PR with the proper change fo rthe version string.

Superseded by #217.


add option for strict equality of COMMODITY_BALANCE equation

There are certain technologies/levels where an inequality for the COMMODITY_BALANCE (i.e., SUPPLY > DEMAND) is not applicable, like electricity or pollutants.
We need to define a new equation in the GAMS formulation that implements strict equality as an option.
The question at this point is how to "activate" the strict equality. Should this be contingent on the commodity only, or (commodity, level), or also be specific on node, year or sub-annual time.

for now, in our specific application with water technologies, we found it useful to have the equation contingent on commodity,level

Shall we close this?

this will be closed as part of release 1.2 (merged PR #172 into the release-candidate branch)

Closing as this is now in master and 1.2 will be released shortly.


Invited amount is not getting updated in Total Balance section when user B creates the account with the same phone number invited by the user A

Frequency: 80%
Repro on build version: Android Internal Release Build V1.32.2, iOS Test flight Release Build V1.32.2, Android Play store Build V 1.31.0
Repro on devices: Google Pixel 4a(12.0), iPhone 13 mini(15.1.1), Google Pixel 2XL(11.0), One Plus 7t(11.0)
Pre-condition:

For User A:  User A must have invite link to User B with some funds
For User B: User B must have be on confirm screen page while creating the account with same phone number invited by user A

Repro Steps:

For User B : Enter the valid confirmation code
For User B: Wait for sometime and observe

Bug: Invited amount is not getting updated when user B creates the account with the same phone number invited by the user A
Expected Behavior: Invited amount should get updated when user B creates the account with the same phone number invited by the user A
Investigation:

Escrow payment received is seen in the recent activity page and balance is not seen updated.
If the user kills and relaunches the application then also balance is not seen updated.
If the user refreshes the page then also balance is not seen updated.

Impact: Bad User Impact as balance is not getting updated
Attachment: Recent Activity .jpg
Balance.mp4

@ValoraQA can we retest to confirm that it's not a temporary issue with Blockscout.

Hey @MuckT, We have re-verified this issue using Latest Android Internal Release Build V1.32.2, iOS Test flight Release Build V1.32.2 & observed that the issue is fixed now.

Invited amount is getting credited and also updated in Total Balance section when user B created the account with same phone number invited by user A.

Hence we are closing this issue.
Devices: Google Pixel 4a (12.0), Google Pixel 2XL (11.0), iPhone 13 mini (15.1.1)
Attachment: Invites amount credited.mp4
Thanks..!!


Unable to get an interactive shell

I would like to get a shell using sh. I am currently not sure whether this is a feature request, a bug report, or user error :)
import sys
from sh import sudo
out_kwargs = {'_out': sys.stdout, '_out_bufsize': 0, '_tty_in': True}
sudo.bash(**out_kwargs)

This gives a bash prompt, but I'm unable to interact with it. How can I get an interactive prompt? Thanks.

can't be done, currently: https://github.com/amoffat/sh/issues/92

_fg option exists in the 1.2 branch, which will be shipping soon


New Wizard - Create new Object Search & Select LWC

Problem
The current picklist is too large and can time out
Solution

user types 3+ chars
return results that can be picked from
have apex store a static object list and then have LWC search the static and return filtered results
It should allow the user to search by Label or API name of an object.
it needs to optionally accept a parent object, and use this as an additional filter to return objects that are allowed to be children.

Logic Assertions
Steps to test the solution works:

1

QA  Proof it passed QA (steps and pictures)
Implementation

Sandbox
Objects and Fields
Automation
Permissions
Github PR
Documentation

Done and shipped as part of v2.21


[release-0.19] Add patch to revert optionally sourcing /etc/default/kubelet in kubelet service

Description of changes:
This is a manual cherry-pick of https://github.com/aws/eks-anywhere-build-tooling/pull/3850.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/override imagebuilder-1-30-presubmit

/override imagebuilder-1-30-presubmit

/lgtm
/approve


Fix mxp Test

This test uses arrays, amongst other things, which causes various problems.

Still not fixed.


Build time is too long using build_firmware.sh

We are using MicroROS and Zephyr on a custom STM32-based board.  When running the build_firmware.sh script (ros2 run micro_ros_setup build_firmware.sh) it takes a very long time to build the firmware (minutes). It's only marginally faster when using the -f flag.  Building is very slow even if no source files are modified.  We can see that many object files are rebuilt each time.  This seems wrong.  Any ideas on how to make building faster?  This slows rapid code development.
When building zephyr only using west build, everything compiles very quickly and no un-touched source files are recompiled.

Just use the micro-ROS module for Zephyr and customize your build procedure: https://github.com/micro-ROS/micro_ros_zephyr_module
It is true that the micro_ros_setup scripts will rebuild the whole environment in order to handle changes in the micro-ROS installation.


don't unmount the scatterplot on model change if we have it cached

So no loading screen. Benefit to this is there will be an animation between the points if the scatterplot doesn't get created and destroyed every load.

ooo didn't think of the animation part. But we could keep the loading screen while it's preprocessing in the backend on the first run.


run wasm_executor from js

Getting this error
panicked at 'error initializing logger: SetLoggerError(())', src/lib.rs:15:53

Stack:

Error
    at module.exports.__wbg_new_abda76e883ba8a5f (/Users/xiliangchen/projects/acala/chopsticks/executor/pkg/executor.js:500:17)
    at wasm://wasm/0093cb12:wasm-function[669]:0xef365
    at wasm://wasm/0093cb12:wasm-function[1984]:0x11fd93
    at wasm://wasm/0093cb12:wasm-function[1095]:0x10f383
    at wasm://wasm/0093cb12:wasm-function[1205]:0x113fe0
    at wasm://wasm/0093cb12:wasm-function[1658]:0x11e03d
    at wasm://wasm/0093cb12:wasm-function[1466]:0x11af11
    at wasm://wasm/0093cb12:wasm-function[1586]:0x11d1e1
    at wasm://wasm/0093cb12:wasm-function[1273]:0x116566
    at wasm://wasm/0093cb12:wasm-function[120]:0x74c0f

RuntimeError: unreachable
    at wasm://wasm/0093cb12:wasm-function[1095]:0x10f3bd
    at wasm://wasm/0093cb12:wasm-function[1205]:0x113fe0
    at wasm://wasm/0093cb12:wasm-function[1658]:0x11e03d
    at wasm://wasm/0093cb12:wasm-function[1466]:0x11af11
    at wasm://wasm/0093cb12:wasm-function[1586]:0x11d1e1
    at wasm://wasm/0093cb12:wasm-function[1273]:0x116566
    at wasm://wasm/0093cb12:wasm-function[120]:0x74c0f
    at wasm://wasm/0093cb12:wasm-function[1074]:0x10e225
    at wasm://wasm/0093cb12:wasm-function[662]:0xee7d2
    at wasm://wasm/0093cb12:wasm-function[1757]:0x11eea0
[ERROR] 11:47:16 RuntimeError: unreachable

Can you keep the existing rust binary. It is useful for development that I can edit the Rust files and the changes will be applied without restarting the server.

Can you keep the existing rust binary. It is useful for development that I can edit the Rust files and the changes will be applied without restarting the server.

yes we can


[Bug]: Importing python HttpClient requires sqlite3

What happened?
Statement from chromadb import HttpClient raises the following error.
File "/usr/local/lib/python3.11/dist-packages/chromadb/__init__.py", line 79, in <module>
  raise RuntimeError(
RuntimeError: Your system has an unsupported version of sqlite3. Chroma requires sqlite3 >= 3.35.0

Why does the http client need sqlite?
Versions
python 3.11, chromadb 0.4.21, ubuntu 20.04
Relevant log output
No response

If you just need the client, use chromadb-client which is a thin client
and does not require sqlite.
On Sun, Dec 31, 2023 at 11:17 AM Joe Maley @.***> wrote:

What happened?
Statement from chromadb import HttpClient raises the following error.
File "/usr/local/lib/python3.11/dist-packages/chromadb/init.py", line 79, in
raise RuntimeError(
RuntimeError: Your system has an unsupported version of sqlite3. Chroma requires sqlite3 >= 3.35.0
Why does the http client need sqlite?
Versions
python 3.11, chromadb 0.4.21, ubuntu 20.04
Relevant log output
No response
—
Reply to this email directly, view it on GitHub
https://github.com/chroma-core/chroma/issues/1603, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGZWEHBPXMQ5MVXO4JGJLTYMG24XAVCNFSM6AAAAABBIOBJ7KVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA3DCMJUHEYDQMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you


Integrate with DIAL stateful API

Required issue: https://github.com/epam/ai-dial-core/issues/33

Loader examples

For input

For modal windows

Animated prototype
https://www.figma.com/proto/lwL5jneveZczAj5RLcLFH2/DIAL-Chat?page-id=0%3A1&type=design&node-id=5834-9838&viewport=-1517%2C-3269%2C0.12&t=9MJUItVIut9BcsLz-1&scaling=min-zoom&starting-point-node-id=2%3A14&mode=design


Add SSH_DISPLAY_USER configuration

This adds a SSH_DISPLAY_USER configuration parameter in order to change what's displayed in clone URL.
You might wonder about the use case:
I'm running gitea in a docker container and have configured SSH Container Passthrough. However the host ssh user is "gitea" and not "git" (as used in the container). Everything works fine but I was unable to customize the ssh clone URL shown in the web ui.

Great, thank you. Then my changes are obsolete.


Adding manifests for SSMS 19..0 and SSMS 19.0.1

Adding manifests for SQL Server Management Studio 19.0 and 19.0.1.
The good news is that I created the tooling to generate these files, so the next time we (I am on the SSMS Team) release a new version, a PR can be created automatically.
There may be updates coming around the format of the files. The one I used as templates were the ones I found for 18.12.1, which seemed fairly accurate.

[x] Have you signed the Contributor License Agreement? Not needed, since I'm a Microsoft FTE.
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Does your manifest conform to the 1.4 schema? It is 1.2.0, as it was the latest that somebody added for SSMS 18.12.1. I can change it if needed but I don't think it would make a difference.

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@erpett , @erinstellato-ms, @dzsquared, @markingmyname, FYI.


Get sources step(TFVC) fails for OSx agent with authentication error

Logs:
2016-04-29T05:52:10.2676450Z ##[section]Starting: Get Sources
2016-04-29T05:52:10.4158000Z ##[command]/Users/administrator/satishAgent/_layout/externals/tee/tf eula -accept
2016-04-29T05:52:11.5926010Z You have already accepted the terms of this End User License Agreement.
2016-04-29T05:52:13.4593810Z ##[warning]Exit code 1 returned from process: file name '/Users/administrator/satishAgent/_layout/externals/tee/tf', arguments 'eula -accept'.
2016-04-29T05:52:16.5751170Z An error occurred: Access denied connecting to TFS server http://satish-destop:8080/ (authenticating as _)
2016-04-29T05:52:16.5770970Z ##[error]Exit code 100 returned from process: file name '/Users/administrator/satishAgent/layout/externals/tee/tf', arguments 'workspaces -format:xml -collection:http://satish-destop:8080/tfs/TestDefault/ -login:,******** -noprompt'.
2016-04-29T05:52:16.5803310Z ##[section]Finishing: Get Sources

The same is failing for Ubuntu agent too with the following log:
2016-04-29T07:22:06.6384200Z ##[section]Starting: Get Sources
2016-04-29T07:22:06.7913110Z Syncing repository: sample (Git)
2016-04-29T07:22:06.8125840Z ##[command]git version
2016-04-29T07:22:06.9301410Z ##[command]git clone --progress --no-checkout http://OAuth:@kaadhina1:8080/tfs/DefaultCollection/_git/sample "/home/ddcloud/kaadhinaagent/_layout/_work/1/s"
2016-04-29T07:22:06.9346990Z Cloning into '/home/ddcloud/kaadhinaagent/_layout/_work/1/s'...
2016-04-29T07:23:21.4798680Z fatal: Authentication failed for 'http://OAuth:@kaadhina1:8080/tfs/DefaultCollection/_git/sample/'
2016-04-29T07:23:21.5606170Z ##[error]Git clone failed with exit code: 128
2016-04-29T07:23:21.5786750Z ##[section]Finishing: Get Sources

Per the readme and release announcement, this preview is VSTS only.  We are doing on prem work in git upstream, tfsvc and the Dev15 server to get the bearer token through on-prem.


IgnorePointered Widgets with elevation overlayed on a scrolling list block scrolling on fuchsia.

See me for more details.

@apwilson it looked like you had a simple repro case when I stopped by.  Can you paste a repro case here?

Somehow we 'fixed' this by using a PhysicalModel instead of a Material but I'm not sure why that fixes the issue.

Any chance you still have the code that originally showed the problem? Otherwise it will be hard to act on this problem...


Documentation

Summary
Writing new documentation for all the modules in the x/ folder. Here's the Readme for x/epochs/ primarily taken from some of Aidan's earlier work.
I'm getting rid of the x/epochs/spec/ folder, but I started the README off by copy pasting the content from all the files in that folder to the new README and editing from there

Good stuff! Helpful to have this all documented. A good next step would be to add an explanation to this README of how our stride specific epochly logic works. Can be a separate PR!


Obfuscated apk failure

I'm running the obfuscator with basic configuration:
python -m obfuscapk.cli -p -o Rebuild -o NewAlignment -o NewSignature original.apk

But I receive this error while trying to install it
Failure [-124: Failed parse during installPackageLI: Targeting R+ (version 30 and above) requires the resources.arsc of installed APKs to be stored uncompressed and aligned on a 4-byte boundary]

This installation error is typically related to a signature issue in the app generated from Obfuscapk on an app that targets an Android version greater than or equal to 30. In detail, it is caused by the "resources.arsc" file being stored in its compressed form inside the .apk file and should not be stored in its uncompressed form.
A way to solve the problem could be to run the script provided from the https://github.com/mathieures/convert-apk repository to unpack the .apk file and align the resources.arsc, rebuild the working .apk, and install it.

I consider this issue closed


Update TPLs and config site for llnl scf toss4

Description
Addresses a bullet in #19343

built the TPLs on SCF toss4.
got rid of SCF toss3 config site files. bye bye jade
Updated config site file for magma (SCF toss4)

Type of change

[ ] Bug fix
[ ] New feature
[ ] Documentation update
[x] Other~~
config site and build stuff

How Has This Been Tested?

successfully completed a build with new config site and TPLs on magma.
Reminders:

Please follow the style guidelines of this project.
Please perform a self-review of your code before submitting a PR and asking others to review it.
Please assign reviewers (see VisIt's PR procedures for more information).

Checklist:

[x] I have commented my code where applicable.~~
[ ] I have updated the release notes.
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have confirmed new and existing unit tests pass locally with my changes.
[ ] I have added new baselines for any new tests to the repo.
[ ] I have NOT made any changes to protocol or public interfaces in an RC branch.

merge w/ develop in #19371


Add validator operand

Creates and watches template-validator resources.

/cc @omeryahud @ksimon1 @vatsalparekh

Now that the placement API is merged, please add it to this PR as well (https://github.com/kubevirt/kubevirt-ssp-operator/pull/226)
I was thinking about something like this:
SSP.spec.templateValidator.nodeSelector
SSP.spec.templateValidator.affinity
SSP.spec.templateValidator.tolerations

Added placement API.

Looks good to me, just one thing. Would be possible to add tests which tests placement api? e.g. create CR with affinity, then remove affinity. Because currently in old ssp we encountered bug, that it can't remove affinity from deploy manifest.

Would the placement values depend on the cluster where the tests are run? I will try to add the tests, maybe the deployment will be created even with incorrect placement values.

I would do it like we do it in old ssp https://github.com/kubevirt/kubevirt-ssp-operator/blob/master/functests/10-test-template-validator-placement.sh (with fake values)

Added the test. Removing placement works fine.

/lgtm
@vatsalparekh can you please review?

/lgtm


sys/linux: syz-describe: auto generate syzlang

Our tool~ (SyzDescribe) could automated static generate basic syscall descriptions~ (syzlang format) for Linux kernel drivers.
Based on our evaluation, SyzDescribe is far more accurate than DIFUZE in identifying kernel drivers~ (no FP and only FN) and their device file names~ (still have a few FP and FN).
We will open source SyzDescribe after the paper is accepted~ (already submitted, hope we can open source it soon).
(Related Issue (#590))

Scope of SyzDescribe:

Character or Block drivers in Linux kernel

syscall handler structure
device file names for open()
command value and argument type for ioctl()

Non-open file descriptor dependency from ioctl()

fd returned from ioctl() and used in new ioctl(), e.g., resource fd_kvmvm[fd] in KVM

Techniques of SyzDescribe:

Static analysis in C++ based on LLVM 13

The generated syscall descriptions could be directly used in syzkaller based on doc
make extract TARGETOS=linux SOURCEDIR=$KSRC
make generate
make

However, those auto generated syscall descriptions are not friendly to humans.
I am working on giving them some meaningful names or file paths now.
Current plan:

file name of those generated ones, e.g., syzdescribe + kernel version + source code relative path + something else？
give resource fd meaningful names
try to use macro instead of value of cmd in ioctl() (not sure whether this is possible for LLVM bitcode)
try to use name in source code of structure and their fields
add some comments in generated syscall description from debug info for human, e.g., source code relative path + line number

The commit is current results directly from SyzDescribe, which are mainly for drivers with existing syscall descriptions~ (will generate results for the whole kernel in the future).
How do you think about this and what else are needed to accept them in syzkaller？

file name of those generated ones, e.g., syzdescribe + kernel version + source code relative path + something else？

Yes, we need some common prefix, or maybe put them into a separate dir. Using kernel version is not good. We will re-generate them periodically and names shouldn't change. We need to capture the revision somewhere else.
We also need some prefix for syscalls (the current "_" looks a bit hacky). Something generic, short and unique. Something like "auto", but "auto" can probably conflict with manual descriptions.

The problem with ioctl values is not just naming, the values are also arch-specific.
We've recently got support for per-file metadata and per-arch descriptions:
https://github.com/google/syzkaller/commit/44a5ca633e186c5836010366c515a4017836121b
So think this should generate files like "sys/linux/*_amd64.txt" and include:

meta arches["amd64"]

also we could add:
meta autogenerated

if we need if for something.

Some issues I've spotted in the generated file names:
There presumably should be turned into syz_open_dev since openat does not substitute "#":

disabling openat$_azx_driver_init_snd_ctl_f_ops_syzgen_3: open(/dev/hwC#D#) failed: no such file or directory
disabling openat$_azx_driver_init_snd_hwdep_f_ops_syzgen_3: open(/dev/snd/hwC#D#) failed: no such file or directory
disabling openat$_chr_dev_init_console_fops_syzgen_5: open(/dev/vcs%u) failed: no such file or directory
disabling openat$_chr_dev_init_console_fops_syzgen_7: open(/dev/vcsa%u) failed: no such file or directory
disabling openat$_chr_dev_init_console_fops_syzgen_9: open(/dev/vcsu%u) failed: no such file or directory
disabling openat$_init_ch_module_changer_fops_syzgen_1: open(/dev/s%s) failed: no such file or directory
disabling openat$_usb_init_usbdev_file_operations_syzgen_1: open(/dev/ep_%02x) failed: no such file or directory

tun is located at /dev/net/tun:
disabling openat$_tun_init_tun_fops_syzgen_1: open(/dev/tun) failed: no such file or directory


generated table does not render the same in the editor and on stackoverflow

The table generated on stackedit is not compatible with stackoverflow markdown.
I'm opening this bug even if this might be more for stackoverflow than here...
Maybe the fix would be a "Stack Exchange" compatibility mode (haven't seen a button).
Problematic Markdown sample:
|                   |  `>` | `\|` |     | `"` | `'` | `>-` | `>+` | `\|-`| `\|+`|
 -------------------|:----:|:----:|:---:|:---:|:---:|:----:|:----:|:----:|:----:|
| Trailing spaces   | Kept | Kept |     |     |     | Kept | Kept | Kept | Kept |

Stackedit and expected output:

Stackoverflow and !@#:

Just by chance, try to name the first column...
El 2/10/2016 6:09 a. m., "Romain Dartigues"<EMAIL_ADDRESS>escribió:

The table generated on stackedit is not compatible with stackoverflow
markdown.
I'm opening this bug even if this might be more for stackoverflow than
here...
Problematic Markdown sample:

>
|

"
'
>-
>+
|-
|+

Trailing spaces
Kept
Kept

Kept
Kept
Kept
Kept

Stackedit and expected output:
[image: stackedit]
https://cloud.githubusercontent.com/assets/393859/19019701/7a9eca2a-8890-11e6-909e-81855a4d56c5.png
Stackoverflow and !@#:
[image: stackoverflow]
https://cloud.githubusercontent.com/assets/393859/19019702/7aa0945e-8890-11e6-8168-09e560dce6c6.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/benweet/stackedit/issues/1027, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA8y5sXARaQk5-I5QaQyV63sodN8aObgks5qv3TBgaJpZM4KL-CA
.

Sorry, should have checked before posting, this is indeed a SE problem : http://meta.stackexchange.com/questions/73566/is-there-markdown-to-create-tables
Awesome job you did, keep up ;^)


Export NeMo FastConformer Hybrid Transducer-CTC Large Streaming to ONNX.

Fixes #790
CC @tempops @sangeet2020
Also CC @titu1994
Exported models can be found at
https://github.com/k2-fsa/sherpa-onnx/releases/tag/asr-models

Will add

[ ] C++ runtime and other API bindings for it
[ ] Android demo
[ ] transducer support

Hi @csukuangfj ,
Thanks for the PR.
If I understand correctly, this PR is only meant for exporting the EncDecHybridRNNTCTCBPEModel model types such as one found in nvidia/stt_en_fastconformer_hybrid_large_streaming_multi.
The decoding implementation needs to be done separately? or can I still use the existing implementation https://k2-fsa.github.io/sherpa/onnx/pretrained_models/offline-ctc/nemo/index.html for these models?

If I understand correctly, this PR is only meant for exporting the EncDecHybridRNNTCTCBPEModel model types such as one found in nvidia/stt_en_fastconformer_hybrid_large_streaming_multi.

Yes, you are right.

The decoding implementation needs to be done separately?

No, we can reuse the existing CTC decoding code.
However, we need to change the C++ code to support the neural network part.
By the way, the C++ decoding part does not depend on the specific neural network model; therefore, it can be reused for any model trained by the CTC loss.


Full text search

It should be relatively simple to implement locally.
Would it be useful to also allow server-side search?

Fixed in #252.


ci/github: Remove debug hack from github publishing

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

pushing through to fix ci


rclone.conf issues

What is the problem you are having with rclone?
cant start over wont let me delete or copy, just stuck with the rclone.conf file can't modify it.
What is your rclone version (eg output from rclone -V)
v1.42
Which OS you are using and how many bits (eg Windows 7, 64 bit)
linux (its on a seedbox can't figure it out)
Which cloud storage system are you using? (eg Google Drive)
google drive
The command you were trying to run (eg rclone copy /tmp remote:tmp)
trying to d a remote, copy, ect gives me this error.
Failed to save config after 10 tries: Failed to create temp file for new config: open /home/user/.config/rclone/rclone.conf920998464: read-only file system
A log from the command with the -vv flag (eg output from rclone -vv copy /tmp remote:tmp)
Using config file from "/home/user/.config/rclone/rclone.conf"
2018/07/12 03:31:55 Failed to create file system for "remote:tmp": didn't find section in config file

Rclone appears to think that the user has a read only file system.
Is that correct?
You can use the --config flag to point the config file to a place where rclone does have read write access.

what would the full command look like?

Just add --config /path/to/new/rclone.config to your rclone command line.
I'm afraid I don't know anything about seedboxes so can't suggest an actual research.
You might want to investigate why/home is read only though - that doesn't seem right.

I'm going to close this since it isn't an rclone issue.  If you need more help I suggest asking in the rclone forum.

I ran into this issue as well.  The problem was that I had previously run rclone as the root user.  Changing ownership to the user running rclone fixed it.


⚠ T:Riza Roblox Proxy has degraded performance

In dadf53d, T:Riza Roblox Proxy (https://rprxy.triza.dev) experienced degraded performance:

HTTP code: 200
Response time: 2409 ms

Resolved: T:Riza Roblox Proxy performance has improved in 34948d9.


STORM-2386 Fail-back Blob deletion also fails in BlobSynchronizer.syncBlobs (1.x)

Please refer STORM-2386 for more details.
I'll submit patches for master soon. If we need to port back to 1.0.x as well, it should be easy to cherry-pick.

For master branch: #1976

+1


not installing

[ ] I have reset Atom to defaults prior to submitting report.
[ ] I have not reset Atom to defaults prior to submitting report.

Description

Steps to reproduce

Expected behavior:
Actual behavior:
Reproduces how often:
Versions

> atom --version

> apm --version

OS name and version:
Platformio-ide-terminal version:
Additional Information

Thank you for reaching out with this bug report.
You are getting this reply because the issue template was not filled in its entirety, overwritten,  or some portions were partially left out.
This issue template contains important information to be able to provide any kind of support, and will help us triage or reproduce the issue. Without reproduction no workaround or fix can normally be derived...
Copy and paste the template located at https://raw.githubusercontent.com/platformio/platformio-atom-ide-terminal/master/.github/ISSUE_TEMPLATE.md and fill in all fields. Logs / Errors / other information should be added at the bottom in the Notes portion of the issue template.
Tickets without this information may be closed and labeled invalid at our discretion.
Thank you for your understanding.


Ts docker

Extract build code out of deploy code.
Allow building specific platforms.
Add Dockerfile which uses linux/amd64 binary
Add Makefile to make managing the above easier

@hoffmabc I think you should review this one because you setup the build stuff. deploy.sh should act exactly as before.

Not sure why the travis errors. It's saying it can't find a package?


Adding support for suppressing event sending

Coverage remained the same at 100.0% when pulling 793edf9db1429f0af581cebec1747eda3b5abe52 on armonge:master into 198970bcf422e8f6928d3d4a35bd9d45f2262e4f on mediarain:master.

Package has been published


[ENG-1627] Add BrandAssets model

Purpose
Add BrandAssets model for later branding for AbstractProviders to hold styling infomation
Changes

adds BrandAssets model with auto mgration
adds new fields to AbstractProvider
adds BrandAssetsFactory

QA Notes

small, low risk migration
not user facing, no QA testing

Documentation
Not user facing, no docs
Side Effects
None that I know of.
Ticket
https://openscience.atlassian.net/browse/ENG-1627

LGTM! 🛥

Turns out that giving a model a plural name will lead to confusion ("a provider has one brand assets") and maybe even technical problems (how to pluralize it in the api, brand-assetses?) -- mind if we revisit that name real quick?
It should be clearly descriptive, not entangled with other model concepts (e.g. "provider"), and (this is new) not plural -- maybe VisualBrand? I still think just Brand is not clearly descriptive, but am open to being shouted down if i'm alone in that.

@Johnetordoff sounds like we're going with naming the model Brand instead of BrandAssets -- sorry for the back-and-forth


🌱 Improve installed package error when bundle can not be found

Description
Follow up for https://github.com/operator-framework/operator-controller/pull/499#discussion_r1381994273
Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

@grokspawn could you please take another look? PR got removed from a merge queue becuase #520 got merged and variablesources/installed_package_test.go needed an update.

@grokspawn could you please take another look? PR got removed from a merge queue becuase #520 got merged and variablesources/installed_package_test.go needed an update.

Checking CI signal, and then I'll approve again if green.


lit: 13.0.0 -> 14.0.0

Description of changes
Move to latest lit (from LLVM 14, ish), but this is from the official
release on PyPI.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Debating including psutil as it's optional but useful re:lit for giving tests timeouts, and at least one (out-of-tree) project expects this.
Leave it out for now.
Can tackle later if useful.  And maybe there's a reason to not include (compat, new bugs, new failures due to tests unlocked by this, dependency changes esp wrt bootstrap, etc.).
It doesn't seem too problematic (added and built some stuff that uses this lit), FWIW, but just bump for now :).


Hi, I'm not logging in to the start button

Hi, I'm not logging in to the start button (I installed all the necessary hardware). When logging in, it also says: "Neither ViGEm nor SCPToolkit is installed, please install ViGEm
(I'll repeat that I've got all the prerequisites for Harv even installing vigem)
Please give me guidance. Thank you

What's your OS?

OS 7 - 64Bit

Can you make screenshot of diagnostic window?
Tools -> Diagnostics
Did you install that ?

Yes, I installed it, but my previous problem has been resolved. Now I start the start button, but when I click the edit button, I'm facing this page.

With the 3.19 and later versions you need to add a Game controller with Add controller button. Then you can configure it almost the same way as before.

Thank you, my problem was solved. I was ashamed to take your time

No problem, we all try to help.


hack/test-cmd.sh can be easier to read if we set PATH

instead of using a replacement var.  Makes cut and paste testing
easier.

LGTM


Assist in Fundraising Campaigns

Use your extensive knowledge to:

[ ] - Collaborate with the fundraising team to plan and execute fundraising campaigns.
[ ] - Identify potential funding sources and grant opportunities from partners.

I’m available to work on this


Add process.env types to vite/client

Description
When including vite/client types, without including node types, process.env won't be defined. Despite Vite defining process.env.NODE_ENV by default and it's use as the defacto standard for libraries in dev vs prod.
Suggested solution
Add appropriate types to vite/client types for it. Those types need to work even when also including node types, e.g. vitest. So they need to make use of declaration merging as appropriate, e.g. (Partial definition, for vite/client we might want a bit more):
declare global {
  namespace NodeJS {
    interface Process {
      env: ProcessEnv;
    }

    interface ProcessEnv {
      NODE_ENV: string;
    }
  }

  var process: NodeJS.Process;
}

Alternative
Define the types for process.env by myself in env.d.ts.
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Duplicate of #8560


Flame.util.initialDimensions() getting wrong size of the screen

Flame version: 0.24.0
Flutter doctor -v
[√] Flutter (Channel master, 1.22.0-2.0.pre.36, on Microsoft Windows [versÃ£o 10.0.19041.450], locale pt-BR)
• Flutter version 1.22.0-2.0.pre.36 at C:\src\flutter
• Framework revision d30e36ba8c (6 days ago), 2020-08-21 22:36:03 -0400
• Engine revision d495da20d0
• Dart version 2.10.0 (build 2.10.0-49.0.dev)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.6.5)
[√] Android Studio (version 4.0)
[√] Connected device (4 available)
After every first flutter run, the method Flame.util.initialDimensions() is returning a Size with wrong dimensions, in my case the width, it happens running on debug or release mode.
void main() async{

  WidgetsFlutterBinding.ensureInitialized();

  Size size = await Flame.util.initialDimensions();

  print("SIZE: $size");

}

flutter run it prints:
SIZE: Size(598.0, 360.0)
After hot restart it prints (The right dimensions):
SIZE: Size(640.0, 360.0)

@d-apps Thanks for filling this, we usually don't support the master channel, could you confirm if this behaviour happens on stable/beta?

@d-apps Thanks for filling this, we usually don't support the master channel, could you confirm if this behaviour happens on stable/beta?

I just tested and it happens on beta too. Exactly the same behavior.
Channel beta, 1.21.0-9.1.pre

@d-apps I have tried your code, and it always returns the correct dimension, are you testing on the device? Emulator? can you share additional information?

I came across this problem 1 week ago, I solved it with resize functionality. My main was like the following:
void main() async {
  WidgetsFlutterBinding.ensureInitialized();

  await Flame.util.fullScreen();
  await Flame.util.setLandscapeLeftOnly();
  Size size = await Flame.util.initialDimensions();

  Game game = Game(initialSize: size);
  runApp(game.widget);
}

and Game class was like this:
class Game extends BaseGame with HasTapableComponents, HasWidgetsOverlay {
  Game({@required Size initialSize}) {
    super.size = this.initialSize;
    this.tileSize = initialSize.height / 9.0;

    this.spawnManager = SpawnManager(this);
  }

  SpawnManager spawnManager;
  double tileSize;
}

and SpawnManager was using tileSize of the Game instance to spawn things.
I was using emulator, not every time but most of the time, the cold start was getting wrong size. When I hot reload everything works as it should. BTW, even though the debugger was showing the correct size, the game was not working properly. Two things come to my mind, either landscape mode of emulator is being realized after first initialization or there is a race condition for setting the size on BaseGame.

I changed the order of methods on main.dart and the behavior is a little different now. If I flutter run in debug mode, the game gets only half of the screen on the width, the first screen of my game is the menu screen , an Overlay Widget, and I get an overflow error, but if I stop the flutter run and open my game directly from the launcher on the device, everything works fine and I don't have this issue.
This error of getting half of the screen, if I hot restart it fixes the problem.
Changed the main.dart to:
void main() async{

  WidgetsFlutterBinding.ensureInitialized();

  await Flame.util.fullScreen();
  await Flame.util.setOrientations(
      [
        DeviceOrientation.landscapeRight,
        DeviceOrientation.landscapeLeft
      ]
  );
  Size size = await Flame.util.initialDimensions();
...

}

Error:
════════ Exception caught by rendering library ═════════════════════════════════════════════════════
The following assertion was thrown during layout:
A RenderFlex overflowed by 73 pixels on the bottom.

The relevant error-causing widget was:
  Column file:///D:/Documentos/FlutterProjects/sky_heroes/lib/src/menus/main_menu.dart:50:13
The overflowing RenderFlex has an orientation of Axis.vertical.
The edge of the RenderFlex that is overflowing has been marked in the rendering with a yellow and black striped pattern. This is usually caused by the contents being too big for the RenderFlex.

Consider applying a flex factor (e.g. using an Expanded widget) to force the children of the RenderFlex to fit within the available space instead of being sized to their natural size.
This is considered an error condition because it indicates that there is content that cannot be seen. If the content is legitimately bigger than the available space, consider clipping it with a ClipRect widget before putting it in the flex, or using a scrollable container rather than a Flex, like a ListView.

The specific RenderFlex in question is: RenderFlex#6e1cd relayoutBoundary=up1 OVERFLOWING
...  parentData: not positioned; offset=Offset(0.0, 0.0) (can use size)
...  constraints: BoxConstraints(0.0<=w<=324.0, 0.0<=h<=324.0)
...  size: Size(324.0, 324.0)
...  direction: vertical
...  mainAxisAlignment: start
...  mainAxisSize: max
...  crossAxisAlignment: start
...  textDirection: ltr
...  verticalDirection: down
◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤
════════════════════════════════════════════════════════════════════════════════

I just realized that starting the game from the launcher it still doesn't gets the correct dimesions, same problem.

This project has the problem, sometimes it retrieves the correct size but most of times it doesn't, I think you you'll be able to reproduce.
https://github.com/d-apps/dont_starve_pumbaa

It seems it was fixed now using flame: ^0.27.0, plus I am using Flame's init method, I am closing this issue.
void main() async{

    await Flame.init(
        fullScreen: true,
        orientation: DeviceOrientation.landscapeLeft,
    );

    Size size = await Flame.util.initialDimensions();
    var config = Config();
    config.size = size;

    runApp(MyGame());

}


Elections > export: Fix display of numbering of candidates

Describe the bug
In the export of an election with >9 candidates, the column with the numbering of the candidates in ballot results is limited to one digit, resulting in linebreaks within numbers >9.
How to Reproduce
Steps to reproduce the behavior:

create an election
add 10 candidates to list of candidates
create new ballot (all voting methods except Y/N/A per list)
start and end voting
export election
position 10 in list of ballot is broken into two line "1" and "0"

Expected behavior
Column width allows for numbers >9

I can see a translation bug here, the code tries to translate undefined. (assignment-pdf.service.ts:265)
I can also see the broken display of 10th or 11th candidate.


Published draft does not load

After seeing confirmation pop up of publishing,m page redirects here: https://test.portal.biochemistry.gwu.edu/objects/view/https/test.portal.biochemistry.gwu.edu/BCO_000006/1.0

react-dom.production.min.js:209 TypeError: Cannot use 'in' operator to search for 'step' in
at ParametricDomain.js:62:19
at Array.map ()
at ParametricDomain.js:61:44
at Array.map ()
at $n (ParametricDomain.js:58:15)
at Go (react-dom.production.min.js:153:146)
at vs (react-dom.production.min.js:261:496)
at lc (react-dom.production.min.js:246:265)
at sc (react-dom.production.min.js:246:194)
at Zs (react-dom.production.min.js:239:172)
es @ react-dom.production.min.js:209
ParametricDomain.js:62 Uncaught (in promise) TypeError: Cannot use 'in' operator to search for 'step' in
at ParametricDomain.js:62:19
at Array.map ()
at ParametricDomain.js:61:44
at Array.map ()
at $n (ParametricDomain.js:58:15)
at Go (react-dom.production.min.js:153:146)
at vs (react-dom.production.min.js:261:496)
at lc (react-dom.production.min.js:246:265)
at sc (react-dom.production.min.js:246:194)
at Zs (react-dom.production.min.js:239:172)

resolved!


Add a better go lint and fix errors

Resolves #35

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.929%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
789

Relevant Lines:
840

💛  - Coveralls


484 Address Incomplete

Describe the bug
Registration  status got Registered but when try to call other register number getting following error
To: sip:1003@hostName
I/flutter (  862): From: sip:1001@hostName;tag=o7qg3j69o4
Showing following error
I/flutter (  862): [2024-07-08 12:54:53.500] Level.debug tcp_socket.dart:160 ::: Received TcpSocket data
I/flutter (  862): [2024-07-08 12:54:53.501] Level.debug socket_transport.dart:274 ::: received binary message:
I/flutter (  862):
I/flutter (  862): SIP/2.0 484 Address Incomplete
System Infomation()
Flutter SDK Version:2.5.0
Target OS and Version:0.5.1
Host OS and Version:iOS14

Any suggestions

484 may be related to the format of the SIP URI you called. Please make sure your URI is a reachable address.


Allow users to view and complete recommendations from within the integration

Is your feature request related to a problem? Please describe.
Not a problem, just requesting additional functionality.
Describe the solution you'd like
Provide a way for users to see recommendations and mark them as complete once they've used the chemicals. The latest recommendations can be viewed by adding a few lines to the me query:
        {
            me {
                id
                firstName
                device {
                    batteryLevel
                    serialNumber
                    temperature
                    cartridgeCharges
                    health
                    coreStatus
                    lidOpen
                    online
                    shouldTakeReadings
                    lastMessage
                    currentFirmwareVersion
                }
                hub {
                    online
                    chargerStatus
                    ssid
                    lastMessage
                }
                pool {
                       ####  New Stuff Below This Line ####
                        latestRecommendations {
                          conflictWarning
                          recommendations {
                            id
                            chemical {
                              behaviour
                              image
                              name
                              types
                              packageSize
                              packageSizeUnit
                              upc
                            }
                            completedAt
                            expiredAt
                            type
                            decision
                            explanation
                            treatment
                          }
                      }
                    ####  New Stuff Above This Line ####
                    latestReading {
                        alkalinity
                        bromine
                        chlorine
                        ph
                        readingTime
                    }
                }
            }
        }

Recommendations can be completed with this mutation:
mutation ($recommendationId: ID!, $completedAt: DateTime) {
  completeRecommendation(recommendationId: $recommendationId, completedAt: $completedAt) {
    success
    completedAt
  }
}

And these params:
{
	"recommendationId": "245043",
	"completedAt": "2021-03-05T00:32:40.000Z"
}

Describe alternatives you've considered
N/A
Additional context
Add any other context or screenshots about the feature request here.

@lukeclopez-al, is there an easy way for me to simulate some of these recommendations for testing purposes? Maybe using a test account of some sort?

@ydogandjiev I could give you access to a testing account that has numerous readings/recommendations generated already

@lukeclopez-al, how does the Sutro app generate these strings:

I don't see them in the GraphQL response for the updated query you shared. Is this string generated client-side based on the type and decisions properties of the recommendation?

Here is an example of them im getting as im opening my pool for the season :)
{
    "data": {
        "me": {
            "device": {
                "batteryLevel": 87,
                "cartridgeCharges": 67,
                "coreStatus": true,
                "currentFirmwareVersion": "2.1.7",
                "health": "GOOD",
                "lastMessage": "2023-05-03T00:08:45Z",
                "lidOpen": false,
                "online": true,
                "serialNumber": "xxxxxxxxxxxxxxx",
                "shouldTakeReadings": true,
                "temperature": 75.9
            },
            "firstName": "xxxx",
            "hub": {
                "chargerStatus": "CHARGED",
                "lastMessage": "2023-05-03T00:08:45Z",
                "online": true,
                "ssid": "xxxxxxxxx"
            },
            "id": "xxxx",
            "pool": {
                "latestReading": {
                    "alkalinity": 150.5,
                    "bromine": null,
                    "chlorine": 1.04,
                    "ph": 6.85,
                    "readingTime": "2023-05-02T23:23:19Z"
                },
                "latestRecommendations": {
                    "conflictWarning": "",
                    "recommendations": [
                        {
                            "chemical": {
                                "behaviour": "saltwater_generator",
                                "image": null,
                                "name": "Saltwater Generator",
                                "packageSize": null,
                                "packageSizeUnit": null,
                                "types": [
                                    "PRIMARY_SANITIZER"
                                ],
                                "upc": "generic_saltwater_generator"
                            },
                            "completedAt": null,
                            "decision": "LOW",
                            "expiredAt": null,
                            "explanation": "It will make your swimming water safe and clean.",
                            "id": "4505579",
                            "treatment": "Adjust your Saltwater Generator to raise your chlorine level.",
                            "type": "CHLORINE"
                        },
                        {
                            "chemical": {
                                "behaviour": "sodium_carbonate",
                                "image": null,
                                "name": "pH up / Soda ash",
                                "packageSize": null,
                                "packageSizeUnit": null,
                                "types": [
                                    "PH_UP"
                                ],
                                "upc": "generic_sodium_carbonate"
                            },
                            "completedAt": null,
                            "decision": "LOW",
                            "expiredAt": null,
                            "explanation": "It will prevent corrosion of your spa surface and equipment.",
                            "id": "4505580",
                            "treatment": "Broadcast 2 cups and 1 half cup of pH up / Soda ash.",
                            "type": "PH"
                        }
                    ]
                }
            }
        }
    }
}

I think we could add this just by putting this as a recommendations sensor, and having the actual recommendations be a well formatted attribute array. Should be easy just to drop that json in there.
If I get some time this week I will look at adding it.

@lukeclopez-al do you have any examples of the conflictWarning?

@lukeclopez-al, how does the Sutro app generate these strings:
I don't see them in the GraphQL response for the updated query you shared. Is this string generated client-side based on the type and decisions properties of the recommendation?

Yes, these are made on the client side based on the properties of the recommendation.

conflictWarning

The only one that exists right now is: "Wait an hour between any chemical additions."

@lukeclopez-al so just a string message if one is needed. thanks.

So looking at this its fairly easy to get the recommendations and setup a binary sensor if there are recommendations and attach the recommends and conflictWarning structure itself to the attributes.
Im not sure about this though, I think there might be a better/other way to handle this.
Putting this structure as an attribute allows a user to grab them and present them how ever they want.
Im not sure we want to try to start dynamically creating entities for a recommendation, as it would really just need to be a sensor with the same structure attached to attributes. Im wanting to get some thoughts on this.
We can easily do a service for completing a recommendation all that really takes is the id for the recommendation itself.  It would allow HA users to do their own setup with a button or something and just pass the id if the recommendation to service.
Thoughts?

@wonderslug, that sounds like the perfect design! I spent a lot of time trying to figure out how to best model this information in Home Assistant; not sure why a binary sensor didn't come to mind. I assume it would be triggered when there is a recommendation and clear when everything is good, right?

Yeah only trigger true as Problem device class when there are recommendations present.  Once they are cleared (I think) or not present it would be false/cleared.

Added support for viewing and completing recommendations in v1.0.0-beta.0:
https://github.com/ydogandjiev/hass-sutro/releases/tag/v1.0.0-beta.0
I modeled it as a todo list. Checking off the recommendations marks them as completed. They can also be uncompleted by unchecking them in the list:

@lukeclopez-al, @wonderslug, please give it a shot and let me know if you have any feedback.


Improve gadgettracermanager -dump

Adding containers, traces, stacks, all options to gadgettracermanager -dump=
This PR addresses a part  of #311. The gadgettracermanager -dump= command is enhanced with the following three options for specifying what item to print:

gadgettracermanager -dump=containers
gadgettracermanager -dump=traces
gadgettracermanager -dump=stacks
gadgettracermanager -dump=all

Testing done
Changes have been tested locally with minikube using: make minikube-deploy. Example of output:

@flyth You had other changes planned in grpc. Does this PR fit with your plans?

The other PR doesn't touch the existing interfaces and also not the dump functionality, so it should be fine. 👍


dfilemaker: define defaults and add option processing

Being worked in https://github.com/hpc/mpifileutils/pull/269


[modify] reduce duplicated codes and stabilize up of head_logs

Coverage increased (+0.004%) to 84.512% when pulling a206258119bf9f89365a611edf7c0f42815378f0 on pr-reduce-head-logs into a601bd57bdda1a81e54e0dd2c46d9b9addbbfdae on master.


Meeting notes from 10/17/23

[ ] Say where sources came from. Defer to CancerInFocus's sources page rather than duplicating data.
[ ] Specify timespans over which rates are calculated.
[ ] Nice to have absolute counts along with % rates (fix: AAC column in dataset?)
[ ] Add cancer centers, etc. into overlays with comprehensive data (address, hours, etc.)
[ ] Radon data?
[ ] Investigate bug where downloading data produces empty sheet.
[ ] Add support instructions for non-technical audience

@vincerubinetti I just responded to the item: "Add cancer centers, etc. into overlays with comprehensive data (address, hours, etc.)" That is available currently at our website.

I created this issue, but it's generally best to split issues up into specific, concrete tasks, rather than a list of unrelated things from a meeting. Usually you'd only do a list of small things in a single issue like this if you were intending to knock them all out in a single PR (pull request, a change to the code).
So, I've split these out into their own separate issues. Now, an issue can contain a discussion specific to just that task, and a PR can close a specific issue.


Explain why the user has to do choose a category in review page copy button

Should say instead something like:
This agent is configured to use a single classification model for all categories. To copy this text as a new saying please specify the category it belongs to:

also, can we increase the width or decrease the dropdown help text.

Closed by https://github.com/samtecspg/articulate/commit/4bbad5162d66df2fe8842319267d3f42baf8de7b


Improve accessibility for screenreader/AT users

I'd make an actual PR, but your current code stumps me a bit...but conceptually, it'd be great to see the same sort of aria-label additions I'm proposing for @mdo's version https://github.com/patrickhlauke/github-buttons/commit/35dab7a66b505ac95de1b4006930a6836c5425d7

@patrickhlauke Done! Go ahead and give it a try.

Lovely...works great. Thanks for the speedy fix :+1:


Expected String error

vue.js:634 [Vue warn]: Invalid prop: type check failed for prop "items". Expected String with value "https://i.wifegeek.com/200426/f9459c52.jpg,https://i.wifegeek.com/200426/5ce1e1c7.jpg,https://i.wifegeek.com/200426/5fa51df3.jpg,https://i.wifegeek.com/200426/663181fe.jpg", got Array
found in
---> at src/lightbox.vue
props: {
items: {
type: String, ---> i changed this to 'Array'. fixed
"default": function _default() {
return [];
}
Copied from smyypkn's issue - this is still a problem.

Thanks @JamesCobbledick
It's fixed!


General fixes

@fteicht

@alvalentini Why have you removed this line? Now, on my side, "SkDecide" is not more recognised as a solver from my notebook.

I removed the line because you should add it in your notebook.
When the skdecide integration will be in a stable and working version, and available on PyPi, we will add it as a default solver.

@alvalentini This change seems dangerous to me since now the skdecide solver and rollout function are using the same domain, which can crash scikit-decide. Also, skdecide's solvers can run multiple solvers in parallel by creating them using the domain factory, so passing here the same domain to the solver is another (independent) reason for crash.

I did this change in order to fix a problem with the rewriting of the plan, but after your comment I redid it in a different (and I think correct also from the skdecide point of view) way.


Publish Azure Function App fails on 2.0.1-beta.38

Attached requirements.txt. Noticed the following error.
There was an error restoring dependencies.ERROR: ['pip', 'download', '-r', '/Users/prananth/Projects/FunctioningPython/Gutenberg/TrainBook/requirements.txt', '--dest', '/var/folders/8_/8m4c263132g8fwq0ql816zkw0000gn/T/azureworker9pk49gcc'] failed with exit code 1
requirements.txt
The app works fine locally.

@elprans can you please investigate this issue?

Your requirements.txt contains a package which is not on PyPI:
  Could not find a version that satisfies the requirement en-core-web-sm==2.0.0 (from -r requirements.txt (line 21)) (from versions: )
No matching distribution found for en-core-web-sm==2.0.0 (from -r requirements.txt (line 21))

Great! Is there a way to find out which don't exist on PyPI? Removed en-core-web-sm and it still fails. Also it would be great if a better error message gets thrown out.

@ahmelsayed Is there a CLI mode which runs packapp with --verbose?
@priyaananthasankar There are multiple reasons why dependency resolution might fail.  Making extra requests to pypi is not necessary, we just need to show better diagnostics.

Removed en-core-web-sm. Checked if all other packages are in PyPI. Still fails.

@priyaananthasankar

ERROR: cannot install wrapt-1.10.11 dependency: binary dependencies without wheels are not supported.  Use the --build-native-deps to try building the binary dependencies using a Docker container.

So sending requests to PyPI doesn't ensure there are wheels. It gave me 200 ok for wrapt. Removed wrapt and still received the same error.

@priyaananthasankar what's the error you're receiving now?

There was an error restoring dependencies.ERROR: ['pip', 'download', '-r', '/Users/prananth/Projects/FunctioningPython/Gutenberg/TrainBook/requirements.txt', '--dest', '/var/folders/8_/8m4c263132g8fwq0ql816zkw0000gn/T/azureworker9pk49gcc'] failed with exit code

Tried deploying with --build-native-deps: Fails. Works fine locally!
Command "/home/site/wwwroot/worker_venv/bin/python -u -c "import setuptools, tokenize;file='/tmp/pip-build-k96eir8q/spacy/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /tmp/pip-pnz5q8lu-record/install-record.txt --single-version-externally-managed --compile --install-headers /home/site/wwwroot/worker_venv/include/site/python3.6/spacy" failed with error code -9 in /tmp/pip-build-k96eir8q/spacy/

Using azure-storage is causing issues, check out this link talking about cryptography library - https://github.com/Azure/azure-storage-python/issues/219

This issue was moved to Azure/azure-functions-core-tools#719


WIP: Implement arcade machine detection

This branch at the time of pulling is not tested with the arcade machine.
The script added to the project should check if a global javascript variable named isArcadeMachine is set to true. If it is true after the game assemblies are loaded.

This pull request was created by mistake from the wrong branch.


TeamsActivityHandler should return BadRequests for unexpected Teams Invoke subtypes

OnTeamsFileConsentAsync
OnTeamsMessagingExtensionSubmitActionDispatchAsync

They currently run throw new NotImplementedException();
They should throw throw new BadRequestException();

OnInvokeActivityAsync
This should handle the BadRequestException and set the status code on the InvokeResponse to a 400.

This was fixed


Correcting the delegate naming

Correcting the delegate Naming

Change KSAlertView.xib to KSDialogView.xib
Change KSAlertDelegate to KSDialogViewDelegate

yuno make corrections

👍


Use old row format for test to fix CI

What problem does this PR solve?
For #1355, use old row format in test.

/run-all-tests

/run-all-tests

/run-all-tests

Test failure due to #1183, ignore it first.

/run-cherry-picker


2.61 to master

See detailed description in the commit message.

@horadla23 why there is no merge commit?

@horadla23 why there is no merge commit?

because 2.61 and master HEAD was on the same commit when I created the branch

because there is no difference between 2.61 and master right now, so the merge was a fast-forward merge


Support client adaptive load balancing

By default, the client uses the AllocateMessageQueueAveragely algorithm for load balancing.
As shown in the figure below, each square represents a queue on the broker side, and r and w respectively represent the actual number of read and write queues.

Under normal circumstances, the read and write traffic is balanced. When some brokers are maintained, a copy group will be in a read-only state. Although the number of read and write queues is the same as before, the actual number of writable queues is 0.

In the case of accumulation, all 4 clients will process the message
In the case of no accumulation, since new messages are only equally divided by 3 clients. Client c will be idle.

When we offline client c, which will cause Broker-3's queue divided equallyto other clients. It cannot be solved.
If there are more consumers in the subscription group or more nodes maintained by the server, the no-load situation will be more serious, with a ratio of 1-sum(writeQueue) / sum(readQueue)

Solution:
Due to RocketMQ does not currently implement an effective data source idle declaration mechanism (condition TopicRouteData Perm = 4 && MessageLag = 0), and the implementation is not simple and intuitive. A trade-off plan is to switch to the load balancing implementation of AllocateMessageQueueAveragelyByCircle, and no-load queues and ordinary read-write queues do not need to be distinguished. For subscription groups that are already online, if you use ByCircle directly, the new and old client load balancing algorithm are inconsistent, which will cause some queues in the publishing process to not consume, which will lead to accumulation
Detailed design:

For a brand new subscription group, it is recommended to directly use AllocateMessageQueueAveragelyByCircle for load balancing, which can significantly improve this no-load situation.

For groups that are already online

Using the default Averagely strategy, you can switch to AveragelyAuto implementation.
During load balancing, the client gets the ConsumerConnection connectionSet from the server and contains the version data numbers of all clients in the subscription group. When the client versions are all greater than a specific value, it will automatically switch to AveragelyByCircle.

For other strategies, no upgrade is required

Performance evaluation and compatibility:

Obtain all client versions MQClientAPIImpl.getConsumerConnectionList() in broker is a pure memory operation for the server
Client load balancing is calculated in pure memory on the client side, which has little impact on performance
This solution only involves the modification of the client side and does not affect the server side. Multi-language SDK can also implement the same logic

Use the version of clients and dynamically switch to a new strategy.
Good idea！Looking forward to the code.

How about setting AllocateMessageQueueAveragelyByCircle as the default rebalance choice?


Methods （cancelInternal／dealloc／reset） crash on  V4.0

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
iOS

Platform Version
8.0 Later

SDWebImage Version
4.0.0

Integration Method
manually

Xcode Version
Xcode 8.3.2

Repro rate
sometimes 0.08%

Repro with our demo prj
does it happen with our demo project?

Demo project link
null

Issue Description and Steps

-[SDWebImageDownloaderOperation cancelInternal]_block_invoke (SDWebImageDownloaderOperation.m:205)

2.[UIView(WebCache) sd_internalSetImageWithURL:placeholderImage:options:operationKey:setImageBlock:progress:completed:] (UIView+WebCache.m:39)
3.[SDWebImageDownloaderOperation dealloc] (SDWebImageDownloaderOperation.m:83)
4.[SDWebImageDownloaderOperation reset]_block_invoke (SDWebImageDownloaderOperation.m:225)
Crash on V4.0.0 More than V3.7.6 version ，I need help ，how to fix them. so much crashed

I believe this is a duplicate of #1883 ?

@skyline75489 how to  fix it ？

Try #1858 and see if it works

I use  #1858 try to fixed it ， better than last version， but there is still a crash ，different error。
you can try used UITableView have 100+ cells to show web image ， move up and down quickly ，will crashed

👀

Should be fixed by #1940 - closing for now


Support RDS Migration Function not running via Aurora Serverless Data API

Currently on deploy the RDSMigrationFunction executes migrations via the Aurora Serverless Data API. This means that it is subject to the Data API limits including the 45 second timeout. This results in an error on running a migration as captured here https://discord.com/channels/983865673656705025/983866416832864350/1055632674661859379.
For longer running production migrations it would be great to have an option to have the RDSMigrationFunction on deploy not use the Data API.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Correct unused variables

Some variables are unused in code.
Maybe error-prone has a detector for that

Add variables in cdi-config-extension,
create method with unused parameter
modify the config of error prone in the root pom until the message appears

Are we using javac ? Because in an exemple I found, there's a tag in the pom like this one (in configuration) : javac-with-errorprone
and this one : true

In fact it's much easier if I copy paste all the config tag here :
<configuration>                 <compilerArgs>                     <arg>-d</arg>                     <arg>${project.build.directory}/classes-ep</arg>                 </compilerArgs>                 <compilerId>javac-with-errorprone</compilerId>                 <forceJavacCompilerUse>true</forceJavacCompilerUse>                 <!-- Erroneously inverted logic... for details, see                 https://jira.codehaus.org/browse/MCOMPILER-209 -->                 <useIncrementalCompilation>true</useIncrementalCompilation>             </configuration>

So far i've tested the plugin with mvn install, is there something better ?

Yes, you can use mvn compile

Still compiles with useless class & variables (mvn clean compile -pl :cdi-config-extension)


Dtype checking for diff

What do these changes do?
Adds dtype checking for diff
Related issue number
Resolves #230

[x] passes git diff upstream/master -u -- "*.py" | flake8 --diff
[x] passes black --check modin/

Blocked on #241

No longer doing dtype checking


Restructure configfile for Nextercism, difficulties 1-2

Our initial stab at the reconfiguration for Nextercism incorporating
elements of #794 and #798. The "comment" field is an experimental
addition to the configuration to make remembering each exercise a
little easier while working on the file.
Analysis of the first two levels of difficulty led to some
re-assigning of those values. Exercises with stubs moved to
difficulty. Exercises without a stub moved to 2.

This is a rather big one, but I thought a lot of these changes were interdependent. After taking a holistic look at the levels I moved a few around. I found that the exercises with stubs, by having stubs were much easier than the non-stubbed exercises and that made a good delineation between the lowest levels.
This is much easier to view in the visualizer.
This is a work in progress for sure.

Also, I have been using a Google Docs Sheet to try and track some of this stuff. If anyone wants the link let me know.

Also, I have been using a Google Docs Sheet to try and track some of this stuff. If anyone wants the link let me know.

Yes, that would be amazing! I've been away, so I'm going to give this some time this week and form a considered reply.

Invite sent

@tleen I've opened #805, I only intended to try and get an idea for what might work as core exercises and got carried away. I'm sure there'll be a lot of moving exercises about, but for me at least it's finally given me a proper grasp of the problem.
I think we should focus on choosing core exercises first, and then we can configure the others around them, what do you think?

As your PR handles the whole track and this one only the level 1&2, it supersedes this PR.


Program to calculate factors of a number

@Hannibal404 Please push the file inside directory "Python/Maths/"

@Kumar-laxmi Done!


Correct comment

Doxygen throws a warning: warning: the name `urf_decode.cpp' supplied as the second argument in the \file statement is not an input file
this is corrected. The information should be correct regardless whether or not it is used.

Thank you for the patch. I have merged it now.


GSP in maintenance mode + Tech Debt

We haven't added new features to the GDS Supported Platform and we have addressed Tech Debt consistently, through the whole quarter.


Parquet reader doc

Codecov Report

Merging #256 into master will decrease coverage by 0.08%.
The diff coverage is 93.29%.

@@            Coverage Diff             @@
##           master     #256      +/-   ##
==========================================
- Coverage   86.98%   86.89%   -0.09%
==========================================
  Files          72       74       +2
  Lines        3825     3990     +165
  Branches      581      549      -32
==========================================
+ Hits         3327     3467     +140
- Misses        399      413      +14
- Partials       99      110      +11

Impacted Files
Coverage Δ

petastorm/tf_utils.py
80.16% <100%> (-0.71%)
:arrow_down:

petastorm/local_disk_arrow_table_cache.py
100% <100%> (ø)

petastorm/test_util/reader_mock.py
82.5% <100%> (+0.44%)
:arrow_up:

petastorm/workers_pool/process_pool.py
93.67% <100%> (-0.61%)
:arrow_down:

petastorm/py_dict_reader_worker.py
92.85% <100%> (ø)

petastorm/reader_impl/reader_v2.py
85.43% <100%> (-2.41%)
:arrow_down:

petastorm/benchmark/throughput.py
82.96% <66.66%> (+0.25%)
:arrow_up:

petastorm/arrow_reader_worker.py
92.1% <92.1%> (ø)

petastorm/reader.py
91.98% <92.59%> (-0.51%)
:arrow_down:

petastorm/pytorch.py
86.88% <0%> (-3.28%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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


feat: add defaults to js-packages plugin options

Blocked by #754 as it includes the osAgnosticPath helper
closes #601

as merged it should close this issue too #646


openstack: volumeType is now required

vendoring: update openshift/api, client-go and cluster-api-actuator-pkg/testutils to latest so we have the new API for RootVolume.VolumeType
openstack: update tests for required RootVolume.VolumeType

/test unit

/assign JoelSpeed
/cc pierreprinetti
/test unit
I want to show you the unit errors I'm having locally, I can't figure it out yet. I'm wondering if ExtractFailureDomain needs update to not be liberal anymore since volumeType is now required.

/test unit

/test unit

/test unit

I'll leave it as draft for now until the API PR merges. Until then, the unit tests are ready and should pass now.

/test unit lint vet vendor generate fmt

/test unit lint vet vendor generate fmt e2e-openstack-operator

gather did not start for pod must-gather-l7t6s: etcdserver: leader changed

Not related to that PR I think.

/test e2e-openstack-operator

This LGTM apart from the dependency on the API fork, is this good to go once that is resolved?

This LGTM apart from the dependency on the API fork, is this good to go once that is resolved?

Yes, I need to re vendor go client once API PR has landed and update it all there.

The PR is now ready for review @JoelSpeed
Tests passed locally for me, let's see how it works in CI.

/uncc elmiko

e2e-openstack-operator failing on a test with Checking that the old machine is removed, I'll investigate what happened.
Let's give it a retry.
/test e2e-openstack-operator
Nutanix failing during IPI install:
/test e2e-nutanix-ovn

/test e2e-openstack-operator

/test e2e-nutanix-ovn

e2e-nutanix-ovn is not passing two times in a row with the same error during IPI install step. I suspect it's not related to this PR and the testing history doesn't have much recent data.

It's ready to be merged I think.

/approve


doc: Add documentation for ZC EM Post API

Original date: 2019-04-26 21:14:50
Original PR: https://charm.cs.illinois.edu/gerrit/5053

This change adds documentation for both p2p and broadcast operations
using the ZC EM Post API.
Change-Id: Ic17133de58a3bf551c3dae17c997390e7f8d4c02

The ZC Post API is in 6.10 right? Assuming that, we should target this at 6.10 as well

Yes, this should be in 6.10.

Only review the Post API commit. The previous commit is being reviewed in this PR (https://github.com/UIUC-PPL/charm/pull/2157)


Final Jpeg Orthomap

Is there a way to write the final output directly in jpg format? Thanks :)

There is no direct way to do that right now.  I will sometimes take a screenshot of the visualizer window zoomed out to show the whole data set.  I know that's not what you are asking about.  At some point I would like to create a geotiff output option, but that requires making choices about which image fragments to value over others and the main point of this project (differentiating from other projects like ODM) is that all the original images are preserved and viewable in place/in context in the visualizer tool.  Hope that helps clarify.
Best regards,
Curt.


feat(core): validate duplicate field names

When you use the same fieldName for two properties in one entity, error will be thrown:
@Entity()
class User {

  @PrimaryKey()
  id!: number;

  @Property({ name: 'custom_name' })
  name!: number;

  @Property({ name: 'custom_name' })
  age!: number;

}

This does not apply to virtual properties:
@Entity()
class User {

  @PrimaryKey()
  id!: number;

  @ManyToOne(() => User, { name: 'parent_id' })
  parent!: User;

  @Property({ name: 'parent_id',  })
  parentId!: number;

}

This validation can be disabled via discovery.checkDuplicateFieldNames ORM config option.

Closes #4359

Seems the virtual entity exception does not work, our entity fails the validation (which it didnt before) with this definition:
  @Property({ nullable: true })
  accountId?: number;

  @ManyToOne({ entity: () => Account, joinColumn: 'account_id', nullable: true })
  account?: Account;

With Error:
return new MetadataError(`Duplicate fieldNames are not allowed: ${names.map(n => `${className}.${n[0]} (fieldName: '${n[0]}')`).join(', ')}`);
               ^
MetadataError: Duplicate fieldNames are not allowed: Entity.accountId (fieldName: 'accountId'), Entity.account (fieldName: 'account')

I've tried multiple ways of rewriting the definition but it doesn't pass validation in any way I write it

Uh, I forgot to add persist: false to the docs, it works correctly, what you do is wrong and exactly the thing that is now validated, you should have persist: false on one of the properties. Relations are not "virtual entities", and this has nothing to do with those, the note in the docs is about "virtual properties" = those marked with persist: false.
https://mikro-orm.io/docs/next/defining-entities#virtual-properties

Okay, that makes more sense, so canonically the only correct approach is keeping only the relation and if we need assignment by id use ref(...)?

Yes, it was always like that, having two properties side-by-side mapping to the same column is possible nowadays (they will be mapped correctly), but it will bring issues down the path, e.g. property validation is triggered for both those properties, so for new entities you need to provide both values (which is of course irrelevant for nullable props), and I would also expect problems in schema generator.

FWIW, I wanted to support this in v6 somehow, not yet sure how. Technically you can already do something like this:
@ManyToOne({ entity: () => Account, joinColumn: 'account_id', nullable: true })
account?: Account;

@Property({ nullable: true }) // getters are automatically marked with `persist: false`
get accountId(): number | undefined {
  return this.account?.id;
}

set accountId(val:  number): void {
  this.account = rel(Account, val);
}


[8.x] Prevent long URLs from breaking email layouts

Long URLs will break the default email notification layout because they don't wrap:

I added a new utility class to the default mail theme and modified the notification layout so that  URLs in notifications are forced to wrap:

I'd consider this a breaking change because I had to update a translation string in the notification layout.

This fix should probably go into 6.x.

I agree with @GrahamCampbell.

👍 thanks for the fast feedback. I’ll get this branch re-targeted

Your modifications brake the caouecs/Laravel-lang/ package because you removed the actionURL, see this issue and this PR.


Fix package grouping for NIO examples and pubsub's autogenerated layer

The visual result is the following (compare it with the current site).

/cc @lesv

LGTM - if you could change to spi.* and examples.*, I'd be happier, but it's a personal bias, not a requirement.

@lesv Fixed according to your comments!

Thanks.


Report for address book

Name: address_book
Query example: https://gist.github.com/laacz/1180765
Do you plan implementing it yourself or we could help?

I don't plan on implementing it myself (at least in the near future), but this is a feature that would be nice to have in the core, I'd be happy to merge a PR that includes a report for this.

Ok, will do.

Added in #19


GetDel Operation

Guys, Would be great to use the GetAndDelete Operation(https://redis.io/commands/getdel/). Do any of you know how much work it takes to do this?

Hey @inponomarev  Are you ok? Can you review my pr?


Tng 1126 image map data mappings

I guess you forogt to create the PR based on the current e-Spirit/fsxa-api from next-release branch?


$form->display('id', 'ID'); is null

$ form-> display ('id', 'ID'); is coming null in editing. Can someone help me?

what's the value of id in your database?

Make sure that you type is $form->display('id', 'ID');
If didn't work, try to debug using your browser is there any Error?


fix(extension): project config not applied

Fix #182

:tada: This PR is included in version 3.27.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Disabled/loading state "Request item" button should be a lighter shade of grey

It currently uses the existing disabled state, which looks a bit too heavy:

Potentially change the background colour of the disabled buttons to #e8e8e8 (smoke) and the text colour to #323232 (charcoal) as shown in the first button, or make the text colour even lighter (#8f8f8f) as shown in the bottom version.
@GarethOrmerod would you be able to double-check this, please?

@jamesgorrie There’s already a ‘not requestable’ message and there is a banner explaining you need to sign in to able to request. Could there be a ‘requestable’ message the same for those that are requestable. I can understand the frustration of displaying a CTA that you can’t interact with.
Otherwise the text on the disable button would need to be #323232 if we are being AA compliant.

@GarethOrmerod ah, OK, so I'll change the disabled button anyway.
I think what we're talking about here is state change and item is either:

requestable
not requestable

Then we need to do a look up to see:

is a user logged in? Make item requestable
Is a user isn't ???


Update lab 05

Module: 00
Lab/Demo: 05
Changes proposed in this pull request:

Updated a formatting step due to changes in the UI

This needs to wait until PL-300 VM is updated as well.


Option to subscribe to a subset of relay(s) in the pool (NDKSubscriptionOptions)

Adds some flexibility and removes complexity when needing to subscribe to only a subset of an existing pool. Necessary for seeding/filling with since and/or until.
minimal non-contextual example
  const edgeSub = await ndk.subscribe(
    { kinds: [1], limit: 1 },
    { relays: relaySubset }
  );

Present workaround is to use nostr-js or nostr-tools in addition to ndk.

use relaySet as the options of the subscribe to accomplish this:
const urls = [ "wss://blah" ];
const relaySet = NDKRelaySet.fromRelayUrls(urls);
ndk.subscribe({ kinds:[1]}, { relaySet });


Converting html to delta - colour issues

Trying to use this package to convert this html text

This is just pink

This is just blue

into delta again but the colour doesn't seem to want to convert?
Using this in my code to convert it.  Test using the same HTML using the tests. This is the test:
 test('Paragraph with colors', () {
 const html =
 '
This is just pink

This is just blue
';

 final converter = HtmlToDelta();
 final delta = converter.convert(html);

 final expectedDelta = Delta()
 ..insert('This is just pink ', {"color": "#F06292FF"})
 ..insert('\n\n')
 ..insert( "This is just blue", {"color": "#4DD0E1FF"})
 ..insert('\n');

 expect(delta, expectedDelta);
 });

And this is the Delta result that it gives:
[
{insert: This is just pink , attributes: {color: #F06292FF}},
{insert: \n\n},
{insert: This is just blue, attributes: {color: #4DD0E1FF}},
{insert: \n}
]

It tells us that the format is correct and that there is not a problem with this package. You could try to report this directly in Flutter Quill, or directly provide me with any errors that the debugger has shown you to have more context of where the error might be.

Since this issue has not response of the author, it will be closed

Sorry didn't get any notifications for this ticket.
So this is what the html is and when I print the delta I get this displayed.
flutter: This is just pink This is just blue
I don't get any errors in the flutter console
flutter: insert⟨ This is just pink  ⟩ + {color: #BA68C8FF}
insert⟨ ⏎⏎ ⟩
insert⟨ This is just blue ⟩ + {color: #42A5F5FF}
insert⟨ ⏎ ⟩
Somehow... even when I use your example code it displays it wrong in the flutter quill text entry screen still. So it could be a bug with flutter_quill display colours wrong I guess?

This is a issue coming from flutter_quill this package just translate

HTML to Quill Deltas

How can I implement this?

@Psycoguana thanks for your suggestion. I'll take care about this part of the code as soon as possible.

Due to my busyness I have not had the time to update the code. As soon as I can I will add it

Just applied the new build and it fixes it in one of the places.
But if the text is bold and has colour it strips it. Not sure if the code is like this in another place?


Add support for project datatype in custom fields

https://github.com/ankitpokhrel/jira-cli/discussions/481

Thank you for your contribution 🎉