Add support for AprilTag bundles

Hello,
We are developing an application that requires highly accurate localization and we would like to leverage the capabilities of our NVIDIA Jetson Orin NX to achieve this using AprilTags.
We would be deeply interested in leveraging AprilTag bundles, as used in the apriltag_ros ROS package (https://github.com/AprilRobotics/apriltag_ros). In other words, we would like to be able to define a set of AprilTags with known individual properties (size, id, relative position to bundle reference point, etc.) and receive as output a refined localization estimate with respect to the reference point of said bundle.
We believe that this feature would have a very positive impact on the adoption of isaac_ros_apriltag, and we would highly appreciate it if you could consider adding it in the future.
Thank you for your time!

Thank you for your feedback! We don't currently have this feature on our roadmap but we'll keep your suggestion in consideration for the future.
In the meantime, I think you should be able to write your own postprocessing node that collects the single-tag detections from the existing Isaac ROS AprilTag node and identifies bundles. Taking a look at the original apriltag_ros source code, the implementation of bundle matching just uses the single-tag detecting function in a loop anyways.
Hope this helps!


Slightly More Secure

The fix splits the command used in the call of Open3.capture3 on each space and inputs each item into seperate arguments of the call. This does not completely fix the problem brought forward by Brakeman.
After getting curious it seems like the variables used are not set by anything other than hard coded variables set by the environment, (development / test / production) , I believe there is no way to change those values into something malicious because they are hard coded into the environment, @david-yz-liu ?

@PkHutch I'm not sure there's a way to tell brakeman about that, would have to look into it. But for now I don't think you should worry about it.


why sum(0)?

Thank you for this code!
I got a problem with this at line 33 and line 36. Why sum(0) to add all batch dimension? I noticed that it add the timestep dimension from t=1 to T, so maybe it should be sum(2) here.

Hey, thanks for the catch / fix. I'll look into this.

The patch has been made. Appreciate it :)


Create experiment with multiple objects on a large table.

Requirements:

table large enough for several objects (3-4?)
but not too large that the robot needs to move its mobile base to reach
robot spawns in front of table to minimize time waste in getting to room, table.
experiment prompts are nice to have

We can use Joey's experiment setup as a starting point

No experiment prompts, but all other requirements complete.


unciv: 4.6.5 -> 4.6.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/unciv/versions.
meta.description for unciv is: An open-source Android/Desktop remake of Civ V
meta.homepage for unciv is: https://github.com/yairm210/Unciv
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.6.7 in filename of file in /nix/store/krxa29i8h4mvgbm2xwqb9n1lmvgmqwzk-unciv-4.6.7

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
unciv

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/krxa29i8h4mvgbm2xwqb9n1lmvgmqwzk-unciv-4.6.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A unciv https://github.com/r-ryantm/nixpkgs/archive/320b165c863a3061074fce1fd56fec8b27965d8e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/krxa29i8h4mvgbm2xwqb9n1lmvgmqwzk-unciv-4.6.7
ls -la /nix/store/krxa29i8h4mvgbm2xwqb9n1lmvgmqwzk-unciv-4.6.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

unciv

Maintainer pings
cc @tex for testing.

Result of nixpkgs-review pr 228847 run on aarch64-darwin 1

1 package built:

unciv


WIP: Initial container definition and build process

I'm fairly new to this, so going to try to take it slow, add references, post my learnings in this PR thread as I go, etc.
Work

[x] Add container image
[x] Add labels
[x] Add arguments that need to be populated to populate labels
[x] Create build to test container creation
[ ] Figure out how to extract tags
[x] Ensure build populates labels as expected.
[ ] Build on commit to main?
[ ] Build on tag/release?
[ ] Set up a matrix to support multiple ruby & node versions in the future?

References

Build, tag, push action: https://github.com/marketplace/actions/docker-build-tag-push

Or this one, which might be more official: https://github.com/docker/build-push-action

Template for CI build: https://github.com/excellalabs/blog-in-a-box-container/new/main?filename=.github%2Fworkflows%2Fdocker-image.yml&workflow_template=docker-image
Publishing workflow that extracts metadata etc: https://github.com/excellalabs/blog-in-a-box-container/new/main?filename=.github%2Fworkflows%2Fdocker-publish.yml&workflow_template=docker-publish
Docker metadata action: https://github.com/marketplace/actions/docker-metadata-action

OK, it's now showing the labels as I'd expect.
Next step, I'll try to see if I can start a matrixed build to get some combinations..

OK great, so matrixed builds work now, and the tag is being created successfully.


Vue DevTools recommendation only shows for Chrome, not Firefox

Version
2.5.17
Reproduction link
http://localhost:8080 (any local dev setup)
Steps to reproduce

Run any dev build
Open the page Firefox without vue devtools installed
Open devtools console

What is expected?
Devtools extension gets recommended
What is actually happening?
Recommendation only shows for Chrome, not in Firefox

Logic is in https://github.com/vuejs/vue/blob/master/src/platforms/web/runtime/index.js#L51
Twist is that isChrome is only used in this one code; so maybe it could be changed to hasCompatibleExtension and include a Firefox UA detection. Alternatively UA detection can be dropped, so the link is always shown and users can figure out that they might switch browsers to use the devtools.

I know we can show it for Firefox and Safari as well. @znck @Akryum is it possible to always display it once we have the remote devtools or does something else need to be changed?

Can I take this up? :)

Closed via #8638


How to parse json string from srml module?

you can check how they are using hash in runtime (former srml module), basically just runtime_interface


Debug console not working on Windows

Command doesn't execute.

Fixed in https://github.com/zcoinofficial/zcoin/commit/c47d84cfc7219e1e1b377805e42788f2af0d8f24


Added The Heritage School

School Name: The Heritage School
School Website: https://theheritageschool.org/
Mail pattern<EMAIL_ADDRESS>

@EldoDvae Please provide us with:

the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@EldoDvae Please provide us with:

the school street address, including city and country -
994, Chowbaga Rd, Anandapur, East Kolkata Twp, Kolkata, West Bengal 700107
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
https://theheritageschool.org/newacadamic.php
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@EldoDvae Thank you for the clarification and your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


[PE-D][Tester B] The Filter command does not show that the group does not exist.

After clearing the address book, i tried to use the filter command and it returned 0 students listed. This may become problematic when there are lots of groups to keep track of and using a filter command like this might mislead the user into thinking that the group has already been added but there are no students assigned to that group.
Given input: Filter TUT01 (The group TUT01 does not exist)
Output: 0 students listed!

Labels: type.FeatureFlaw severity.Medium
original: TheodoreKooo/ped#10

patched in UG


Updated UNMASK and ALTER ANY MASK with SQL DB

Added SQL Database as a supported platform for UNMASK and ALTER ANY MASK permissions

#sign-off

@giladmit : Thanks for your contribution! The author, @edmacauley, has been notified to review your proposed change.

@giladmit Thank you for this addition. I'm signing off on this.


Give a bit more informative error message related to deleting isle_generated_code/

PR #4143 added this error if a isle_generated_code/ directory exists and an "isle-in-source-tree" feature isn't enabled:
#[cfg(not(feature = "isle-in-source-tree"))]
{
    if explicit_isle_dir.is_dir() {
        eprintln!(concat!(
            "Error: directory isle_generated_code/ exists but is only used when\n",
            "`--feature isle-in-source-tree` is specified. To prevent confusion,\n",
            "this build script requires the directory to be removed when reverting\n",
            "to the usual generated code in target/. Please delete the directory and\n",
            "re-run this build.\n",
        ));
        std::process::exit(1);
    }
}

It says to delete the directory. However, for users who are not familiar with the inner workings of cranelift, it's not clear where this directory is. It took me a long while to find it:
~/.cargo/registry/src/index.crates.io-{hash}/cranelift-codegen-{version}/isle_generated_code

It may be helpful to add the explicit_isle_dir value to the error message so that users are informed exactly which directory they are supposed to delete.

Hmm, something is going wrong here -- you shouldn't ever need to delete a directory in ~/.cargo. The intent of this feature was to allow Cranelift developers to see a version of the code in-tree when developing Cranelift, and should not be used when Cranelift is pulled in as a dependency from crates.io. Could you say more about your environment and how you're using Cranelift?

@cfallin I don't use cranelift directly so not sure what's supposed to happen... I use wasmer. At wasmer 5.0.0 my program compiles with no problem. Once bumped to wasmer 5.0.1, compiling my program results in this error:
error: failed to run custom build command for `cranelift-codegen v0.110.2`

Caused by:
  process didn't exit successfully: `/Users/[...]/target/release/build/cranelift-codegen-725296997bf7694e/build-script-build` (exit status: 1)
  --- stdout
  cargo:rerun-if-changed=build.rs

  --- stderr
  Error: directory isle_generated_code/ exists but is only used when
  `--feature isle-in-source-tree` is specified. To prevent confusion,
  this build script requires the directory to be removed when reverting
  to the usual generated code in target/. Please delete the directory and
  re-run this build.

I followed the instruction and deleted:
~/.cargo/registry/src/index.crates.io-{hash}/cranelift-codegen-{version}/isle_generated_code

Now the program compiles just fine.

@larry0x OK, thanks for the additional info. This error does not happen when using Cranelift by itself, or with Wasmtime (in this repo). We do not provide support for Wasmer -- that is a different project. I would recommend asking them for help if the issue occurs again; they may be setting options in some way that is causing this issue.


Add Pyramid Pattern

Issue resolved and submitted a PR #6

Hey,Thanks for contributing..It would be great if you add an image of the pattern in the decription


feat(frontend): explain verbose

the PostgreSQL supports explain verbose which will give more information about the plan.

Display additional information regarding the plan. Specifically, include the output column list for each node in the plan tree, schema-qualify table and function names, always label variables in expressions with their range table alias, and always print the name of each trigger for which statistics are displayed. The query identifier will also be displayed if one has been computed, see compute_query_id for more details. This parameter defaults to FALSE.

And in our system, especially for streaming operator, we can give more information about the stateTable.

dev=# explain select max(v2) as max_v2, min(v3), v3 from test,test2 where v3>v6 and v1=v3 group by v3;
QUERY PLAN
HashAggregate  (cost=100.76..100.80 rows=4 width=96)
Group Key: test.v3
->  Nested Loop  (cost=0.00..92.26 rows=1133 width=64)
Join Filter: (test.v3 > test2.v6)
->  Seq Scan on test2  (cost=0.00..18.50 rows=850 width=32)
->  Materialize  (cost=0.00..22.77 rows=4 width=64)
->  Seq Scan on test  (cost=0.00..22.75 rows=4 width=64)
Filter: ((v1)::numeric = v3)
(8 rows)
dev=# explain verbose select max(v2) as max_v2, min(v3), v3 from test,test2 where v3>v6 and v1=v3 group by v3;
QUERY PLAN
HashAggregate  (cost=100.76..100.80 rows=4 width=96)
Output: max(test.v2), min(test.v3), test.v3
Group Key: test.v3
->  Nested Loop  (cost=0.00..92.26 rows=1133 width=64)
Output: test.v3, test.v2
Join Filter: (test.v3 > test2.v6)
->  Seq Scan on public.test2  (cost=0.00..18.50 rows=850 width=32)
Output: test2.v4, test2.v5, test2.v6
->  Materialize  (cost=0.00..22.77 rows=4 width=64)
Output: test.v2, test.v3
->  Seq Scan on public.test  (cost=0.00..22.75 rows=4 width=64)
Output: test.v2, test.v3
Filter: ((test.v1)::numeric = test.v3)
(13 rows)

explain vvv

How about hide distribution keys, PKs, output_index, etc. in plain explain and show these info in this explain verbose?

How about hide distribution keys, PKs, output_index, etc. in plain explain and show these info in this explain verbose?

we can print them as a part in the internal table catalogs.

(except output_index)

(except output_index)

I think it will give all the field name of the output columns of the plannode, so the output_index might be different?

https://github.com/singularity-data/risingwave/issues/3690

Can we have the distribution key & distribution type in verbose explain? 🥰

Can we have the distribution key & distribution type in verbose explain? 🥰

It can be described in the Exchange operators.

How about hide distribution keys, PKs, output_index, etc. in plain explain and show these info in this explain verbose?

we can print them as a part in the internal table catalogs.

This seems infeasible because when you do explain, the materialized view has not been created yet, so does these internal tables 😇

Can we have the distribution key & distribution type in verbose explain? smiling_face_with_three_hearts

It can be described in the Exchange operators.

further more, I think we should print the leaf plan nodes' distribution such as BatchScan and StreamScan in explain(even in normal explain)

Hi, to summarize the above discussion, for explain verbose,

We want to show some detailed info about the plan node, including stream_key (aka. PK of output) and output columns, and some operator-specific information (if have)
We want to show some info about the internal state, including the columns, primary keys and distribution keys.

Here is an example proposed by me. Please comment if have any suggestions.
psql> explain verbose ...

StreamMaterialize { columns: [o_orderkey, count(hidden), distinct_customers], pk_columns: [o_orderkey] }
  StreamHashAgg { group_key: [orders.o_orderkey], aggs: [count, count(orders.o_custkey) filter((flag = 0:Int64))], stream_key: [orders.o_orderkey], output: [orders.o_orderkey, count, distinct_customers] }
    StreamExchange { dist: [orders.o_orderkey] }
      StreamHashAgg { group_key: [orders.o_orderkey, orders.o_custkey, flag], aggs: [count], stream_key: [orders.o_orderkey, orders.o_custkey, flag], output: [orders.o_orderkey, orders.o_custkey, flag, count]}
        StreamExchange { dist: [orders.o_orderkey, orders.o_custkey, flag] }
          StreamExpand { column_subsets: [[orders.o_orderkey, orders.o_custkey]], stream_key: [orders.o_orderkey, orders.o_custkey], output: [orders.o_orderkey, orders.o_custkey] }
            StreamTableScan { table: orders, columns: [orders.o_orderkey, orders.o_custkey, orders._row_id], pk_indices: [2], key: [orders.o_orderkey], output: [] }

State #1 (StreamMaterialize):
  columns: [ o_orderkey, count, distinct_customers ]
  key: [ o_orderkey ]
  dist_key: [ o_orderkey ]

State #2 (StreamHashAgg):
  columns: [ o_orderkey, count, distinct_customers ]
  key: [ o_orderkey ]
  dist_key: [ o_orderkey ]

...(more state info)...


PR template introduction

With this PR, I am introducing a PR template to keep a standard on our development process and make things easier for possible future contributor friends.

Thank you for your detailed comment @jni !
I will be updating with a new commit.
I was also planning/writing a contributing guideline, will drop it in another PR.


Add Apple visionOS support

Motivation
Add the support of Apple VisionOS  121419

Thanks, but this project does not track nightly, nightly is far too unstable and targets can change frequently before being stabilized (if they ever are). If you want to support that target in your own project you can add it as an option to list of targets by filling out a TargetInfo.


Stunt smoke?

I'm working on DE now, and I found that in DE Mobile, Stuntplane's smoke has been fixed. And it used HornJustDown() .
In SA 2.00, HornJustDown checks your vehicle's type and it doen't allow helicopters and planes. We can hook this thing and allows Polmav, Stuntplane and Cropduster.
Besides, CPlane::ProcessControl called CHID::IsReleased which is unavalible on touch screen. Just Redirect it to HornJustDown, and everything is okey.

Pedro Gamer's mobile fix pack includes a script named StuntSmoke, but it just re-implemented the smoking code, lol

Thanks for the info, I'll check that later


E03 S03 Doner confirmation.

Once Donors are notified, a Donor can accept the request. If you accept any new donation requested, you will be able to accept. Once the request is accepted, a notification will be sent to the request owner. The notification will have the following information.

Doner profile information.
Contact information.

Acceptance criteria:

request-owner notification once the request is accepted.
Notification to Doner with request-owner's information.
If the previously notified owner tries to accept an already accepted donor, they will see messages saying the request is complete.
Doner confirmation History.

story points 5

story points 7

Story point 6.

SP 6


After the initial render, any direct url visits to pages result in 404s

Bug report
Describe the bug
in require.js#getPagePath, the pages-manifest.json file is read using:
const pagesManifest = require(join(serverBuildPath, PAGES_MANIFEST))

But every time a new on-demand entry is added to the build, the pages-manifest.json file changes.
Using require though returns a cached version.
This causes requirePage to say module could not be found, which causes a 404.
Fix
Strangely, deleting the require.cache entry didn't resolve it.
delete require.cache[join(serverBuildPath, PAGES_MANIFEST)]

But using the following resolved my issue:
  const pagesManifest = JSON.parse(require('fs').readFileSync(join(serverBuildPath, PAGES_MANIFEST), 'utf8'))

System information

OS: [e.g. macOS, Windows] macOS
Browser (if applies) [e.g. chrome, safari] Chrome
Version of Next.js: [e.g. 6.0.2] v7.0.2-canary.20 / https://github.com/zeit/next.js/commit/b46d7e8f6fc31445801eab08d42e1c361bdaaf28

I wonder what's happening here, as it's definitely working fine, all tests are passing. Are you using a custom server / custom webpack config?

Are you using a custom server / custom webpack config?

Yeh, its quite custom.

We already flush the require.cache in the nextjs-require-cache-hot-reloader.js webpack plugin, and that has been working pretty well so far

Ah, this is probably the source of the problem. I will dig into it.

Don't know if it is a cause but nextjs-require-cache-hot-reloader.js exports class called ChunkNamesPlugin.

Don't know if it is a cause but nextjs-require-cache-hot-reloader.js exports class called ChunkNamesPlugin. A typo I suspect. There is also another plugin called ChunkNamesPlugin.

Changed the name, good catch! it won't affect this issue though

delete require.cache[X] is not working for me inside getPagePath.
So seems like a require hook or something is preventing this.

Found the problem.
The issue was to do with my output dir being inside a symlinked dir. Node caches realpaths, but NextJsRequireCacheHotReloader doesn't resolve realpaths before clearing from cache.
join(serverBuildPath, PAGES_MANIFEST)
/xxx/packages/xxx-symlink/pages-manifest.json

require.resolve(join(serverBuildPath, PAGES_MANIFEST))
/xxx/packages/xxx/pages-manifest.json

Fix
NextJsRequireCacheHotReloader must require.resolve before clearing cache.

@timneutkens Would you accept a PR for the fix I suggest above?

Sounds like the correct fix, however you don't have to use the sync version, the tap is async.


RequireConfirmedEmail deprecated?

It appears there is a new configuration setting in 3.0 called "RequiredConfirmedAccount" that is replacing "RequireConfirmedEmail".  Is that correct?  When I scaffold the Register page, that is the property it checks before deciding whether to login a user upon registration.  Oddly, I don't find a reference to "RequireConfirmedAccount" anywhere in the ASP.NET Core 3.0 documentation, and "RequireConfirmedEmail" is still a valid setting (not marked as obsolete).  Unless I am missing something, I believe the 3.0 version of this doc needs to be updated to use the new property.

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

ID: fe1bea60-d173-f6d2-3559-b147699e295a
Version Independent ID: 98e7aacb-2a01-d581-9b03-bfef2893163e
Content: Account confirmation and password recovery in ASP.NET Core
Content Source: aspnetcore/security/authentication/accconfirm.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

I'll be working on this next week. Follow #12990 to get your questions answered.

I don't think #12990 answers the question. Is there maybe a different place where the question is answered? I can't understand or find any relevant documentation on what is the purpose of "RequireConfirmedAccount", how it is used and how it differs from "RequireConfirmedEmail"

RequireConfirmedAccount is a new higher level option that if set will use the IUserConfirmation service https://github.com/aspnet/AspNetCore/blob/master/src/Identity/Extensions.Core/src/IUserConfirmation.cs to determine if a user is confirmed and hence allowed to logged in.  The default implementation of this, just uses RequireConfirmedEmail, but the idea is this is the new extensibility point for developers to whether a user is considered confirmed and able to log in.

When I set RequireConfirmedAccount to True, no mail is being sent if I'm authenticating via an external login provider. When it is set to False, a confirmation mail is being sent, but I can already sign in without confirming. Could this be a bug or am I missing something?

It would be helpful if this page mentioned the need to remove the code block identified by the comment in RegisterConfirmation.cshtml.cs after the email sender is operational or did I miss that.

When I set RequireConfirmedAccount to True, no mail is being sent if I'm authenticating via an external login provider. When it is set to False, a confirmation mail is being sent, but I can already sign in without confirming. Could this be a bug or am I missing something?

Is there any solution available for this issue?

@gp2-gp2

When I set RequireConfirmedAccount to True, no mail is being sent if I'm authenticating via an external login provider. When it is set to False, a confirmation mail is being sent, but I can already sign in without confirming. Could this be a bug or am I missing something?
Is there any solution available for this issue?

I had the same issue. It looks like you must add role in order to make it work:
.AddRoles<IdentityRole>()
For example:
services.AddDefaultIdentity<IdentityUser>(
	options => {
		options.SignIn.RequireConfirmedAccount = false;
		options.SignIn.RequireConfirmedEmail = true;
	})
	.AddRoles<IdentityRole>()
	.AddEntityFrameworkStores<ApplicationDbContext>();

Found the solution on StackOverflow for me:  I needed to change RequireConfirmedAccount to false, but then set RequireConfirmedEmail to true.  Here's the code fix, and a link to the SO article:
services.AddDefaultIdentity(options =>
{
options.SignIn.RequireConfirmedAccount = false;
options.SignIn.RequireConfirmedEmail = true;
})
.AddRoles()
.AddEntityFrameworkStores();
https://stackoverflow.com/questions/58824729/problems-with-using-external-login-and-sending-confirmation-mail-in-asp-net-core

Yeah there's a bug in the flow here, the product bug will be tracked via https://github.com/aspnet/AspNetCore/issues/18140


如何使用呢

您好，感谢您的工作作出的贡献。我有两个问题想请教您：
1 所选的目标是不限的吗？
2 我应该如何使用您的项目 在自己的计算机上进行推理呢？

Dear @sssssshf ,
We greatly appreciate your interest. We are pleased to announce that our code has been recently released and is now available at https://github.com/Anything-of-anything/Anything-3D/tree/main/AnyObject3D. We kindly invite you to take a look.
For your question:

We intend to make it for any object in the wild, but we do find some corner cases that fails. You can have a try, but no full guarantee on the results.
Please see the code.

Best regards,


Implement end to end tests that attempt to propose invalid blocks

With the introduction of ABCI++ #231, we are adding checks to proposal blocks. while we have tests to ensure that invalid proposals get rejected, we don't have an end to end test for this specific case.

one strategy to accomplish this in both an integration or e2e test is to simply wrap the application and overwrite the prepare proposal method

can close as completed with the addition of the malicious package, which adds the ability to propose and sign invalid blocks.
ensuring that ProcessProposal is always called is a different issue, and the rest of the test cases are covered by existing correctness testing


[AIRFLOW-184] Add mark success to CLI

Dear Airflow Maintainers,
Please accept this PR that addresses the following issues:

(replace with a link to AIRFLOW-184)

Reminders for contributors:

Your PR's title must reference an issue on Airflow's JIRA. For example, a PR called "[AIRFLOW-1] My Amazing PR" would close JIRA issue #1. Please open a new issue if required!
You must add an Apache License header to all new files.
Please squash your commits when possible and follow the 7 rules of good Git commits.

Looks like tests are unhappy:
TypeError: mark_success() got an unexpected keyword argument 'confirm_propt'

Typo? :)
Also saw this error, which looks a little more serious:
assert "Wait a minute" in response.data.decode('utf-8')
AssertionError

I think the overall approach needs to be updated. Ie. you do not mark a "dag" as a success but your mark a "DagRun" and/org its underlying tasksinstances as a success. As such I dont like "dag.mark_success".
Creating TIs outside a dagrun is becoming a big no-no and I would say it is even worse to do so in a "view" component, besides it is indeed not required anymore since #1506 as tasks are eagerly created.
All in all, I really like the functionality but please make it DagRun based.

Agree with @bolkedebruin's comments about DAG run vs dag.

Took another pass and left some comments.
Make sure to take a look at the failing test. Looks like a legit failure after your changes.
Also, we should block on @bolkedebruin for this one, I think, since it's touching DagRun, and he's refactoring stuff there.

Just curious to see if you still intend to refactor this, because I think there is a definite use case for this to be enabled. But maybe this can wait for a REST API.

@artwr  @criccomini  @bolkedebruin yeah, definitely still intend to refactor this. The current mark success in the UI has a few bugs, (using mark_success in conjunction with past, upstream, or downstream flag throws the error page in some cases).  This pr will address those issues as well. I'm ok with waiting for REST API if it's coming up soon, otherwise I think it may be more beneficial to have this merged (once I clean it up =P ).

I think a mark success feature is a good one.  I have been looking for one and don't like running a backfill to achieve it. There is one bug/issue I noticed on the current releases.
I was 3 days behind on an hourly run. I was hoping a single backfill -m would suffice to mark everything success till current. However, since DAG Runs/Task Instances are generated in little groups by scheduler cycles, that was not the case.... I had to continuously run mark success.. for an hour.

I'm ok with waiting for REST API if it's coming up soon, otherwise I think it may be more beneficial to have this merged (once I clean it up =P ).

I think it's worth getting this PR done. The REST API is likely a long ways away.

@bolkedebruin I noticed that subdag's TIs are created independently rather than via DagRun (there are currently no dagrun being created for subdags). Was wondering if this is intentional? imho it would make sense to create dagruns for subdags as well if we want to support mark success / clear a subdag via dagrun.

@jgao54 that is indeed so, but old style. I have a PR outstanding that does add dag runs to backfills (that is how subdags run) but it needs reviewing/testing

@bolkedebruin ah I see it! In that case I'll hold off this PR until yours is merged. I will need to make a bit of changes accordingly, but it will be cleaner.

If you can please review the other pr and +1 it. Thanks!

@bolkedebruin what's the PR?

@criccomini  https://github.com/apache/incubator-airflow/pull/1667

@criccomini  @bolkedebruin  @jlowin  Updated this PR to reflect the changes from #1667

cc @aoen

Seems like all tests are failing?

@criccomini thanks! fixed

Current coverage is 64.75% (diff: 79.51%)

Merging #1590 into master will increase coverage by 0.01%

@@             master      #1590   diff @@
==========================================
  Files           127        127
  Lines          9498       9528    +30
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           6149       6170    +21
- Misses         3349       3358     +9
  Partials          0          0

Powered by Codecov. Last update 821bdb5...7c5199e

@jgao54 Resurrecting this.. please rebase and prepare for review or request to close.

@jgao54 Do you still want to merge this? If not, I will close for inactivity.

@jgao54 closing for inactivity. Please reopen when ready to submit.


LND not ready yet ... waiting another 60 seconds.: report Problem<PHONE_NUMBER>

Hi, I'm launching ./70initLND.sh and keeps getting several errors:
Thanks for the help.
*** Checking bitcoin ***
OK - bitcoind is running

*** LND Config ***
OK - exists

*** Starting LND ***
Starting LND ... give 120 seconds to init.
1555643184
LND not ready yet ... waiting another 60 seconds.
If this takes too long (more then 10min total) --> CTRL+c and report Problem
1555643244
LND not ready yet ... waiting another 60 seconds.
If this takes too long (more then 10min total) --> CTRL+c and report Problem
1555643304
LND not ready yet ... waiting another 60 seconds.
If this takes too long (more then 10min total) --> CTRL+c and report Problem
1555643364
LND not ready yet ... waiting another 60 seconds.
If this takes too long (more then 10min total) --> CTRL+c and report Problem
1555643424
LND not ready yet ... waiting another 60 seconds.
If this takes too long (more then 10min total) --> CTRL+c and report Problem
^C

Sounds like a network problem.

Improving in this situation (when LND is not getting ready during setup) is something I will add for future realease to consider.

Any suggestions to check or resolve this? On the screen I got this:
Public <IP_ADDRESS>:18333 Not reachable
But I did port forwarding and my node is reachable when checking on bitnodes:
https://bitnodes.earn.com/#join-the-network

@Jeans5 can you open a new issue on this?

@rootzoll , it resolved by just rebooting my node.

I'm having the same issue. It worked in my initial setup but I was unable to SSH on that setup after completing the entire setup except syncing the blockchain so I flashed the SD card again and was able to SSH and begin the process. However, LND will not sync.
`OK - chainnetwork is working
*** Check LND Config ***
OK - exists
*** Init LND Service & Start ***
stopping lnd ..
TOR was not selected
Starting LND Service ...
Created symlink /etc/systemd/system/multi-user.target.wants/lnd.service → /etc/systemd/system/lnd.service.
waiting .
waiting ..
waiting ...
      ┌────────────────────────────────────────────────────────┐
      │  Starting LND - please wait ..                         │
      │  ┌──────────────────────────────────────────────────┐  │
      │  │                         0                        │  │
      │  └──────────────────────────────────────────────────┘  │
      ├────────────────────────────────────────────────────────┤
      │               <  OK  >        <Cancel>                 │
      └────────────────────────────────────────────────────────┘

1607804538
LND not ready yet ... waiting another 60 seconds.
If this takes too long (more then 10min total) --> CTRL+c and report Problem
1607804598
LND not ready yet ... waiting another 60 seconds.
If this takes too long (more then 10min total) --> CTRL+c and report Problem
1607804658
LND not ready yet ... waiting another 60 seconds.
If this takes too long (more then 10min total) --> CTRL+c and report Problem
1607804718
`
Not expecting tech support but you might find this useful and if anyone has any tips, I would love to hear them. :)

pls close this.


Are user tracks protected user to user?

Tracks are loaded with the endpoint /editor/<int:track_id> . Could a user use a track_id of non-own track and access to it?

After testing this, using same session url with different logged users, it is confirmed that this problem exists. This is label as a major bug.
Data needs to be protected to its owner user.


Unable to run on Windows 7 64 - bit systems

I am in ningbo users in China, there are software is quite good

You are using QQ or WeChat chat? My mailbox 442324760 @qq.com

我记得carnac使用基于squirrel的自动部署、升级决方案
I remember carnac is based on squirrel auto deployment, update solution.

不要直接在压缩包中启动安装程序
Do not launch the setup.exe from zip archive directly

由于软件会做键盘钩子，监听键盘事件，这个行为和键盘记录器病毒相似，所以有的杀毒软件会误杀掉。
This software uses kerboard events hooks, which is similar to some malware behavior. Some antivirus software may "clean" it.

Thank you very much for your guidance, exotic friends! This software works with adobe captivate 2019 to record instructional videos, very good!
非常感谢您的指导，异国朋友！  这个软件配合adobe captivate 2019 录制教学视频，非常的的好！


Crashing at startup

https://pastebin.com/fkEcVATc

This error is caused by the mod betterblockoutline unable to load something in malilib. Should be unrelated to VMP.
Please attach client full logs.

Youre, right! Sorry.


Swarm mode load balances between 2 instances when only 1 is running

Description
I am trying to setup a Docker in Swarm mode having 1 manager and 2 workers. They run in Digital Ocean's cloud and the nodes communicate via private networking.
I am having an issue when one service connects to another and is apparently having its requests being round robin'ed between 2 endpoints when only 1 instance is running.
Steps to reproduce the issue:

Setup a Docker in Swarm mode having 1 manager and 2 worker in Digital Ocean's cloud having the nodes communicate via private networking.
Setup 2 services based on fx. nginx.

Describe the results you received:
The swarm is apparently working correctly:
$ docker node ls
ID                           HOSTNAME  STATUS  AVAILABILITY  MANAGER STATUS
5yy45mhvjwe6y8p19h91kzh1w    engine2   Ready   Active
759bq2mdovhtcdklvditk3kd4 *  engine1   Ready   Active        Leader
cnzv7b5zcoiecubzjz4vcgk0z    engine3   Ready   Active

Also my services seems to be running just fine - one instance of each service:
$ docker service ls
ID            NAME           REPLICAS  IMAGE                          COMMAND
179qx1ab00z0  previsto-site  1/1       codezoo/previsto-site:latest
7wx5t6ftj2xu  proxy          1/1       codezoo/previsto-proxy:latest

But if I enter the proxy container and make requests to the previsto_site service I will only hit the previsto_site service on every 2nd request:
$ docker ps
CONTAINER ID        IMAGE                           COMMAND                  CREATED             STATUS              PORTS               NAMES
22131f95e686        codezoo/previsto-proxy:latest   "nginx -g 'daemon off"   14 minutes ago      Up 14 minutes       80/tcp, 443/tcp     proxy.1.3joi50x8amiw3znoo6grixs77

$ docker exec -ti proxy.1.3joi50x8amiw3znoo6grixs77 bash

root@22131f95e686:/# curl http://previsto-site

root@22131f95e686:/# curl http://previsto-site
curl: (7) Failed to connect to previsto-site port 80: No route to host

root@22131f95e686:/# curl http://previsto-site

root@22131f95e686:/# curl http://previsto-site
curl: (7) Failed to connect to previsto-site port 80: No route to host

Describe the results you expected:
I would expect the load balancer to round robin to the running instances only.
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
Client:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   23cf638
 Built:        Thu Aug 18 05:33:38 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   23cf638
 Built:        Thu Aug 18 05:33:38 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 8
 Running: 1
 Paused: 0
 Stopped: 7
Images: 7
Server Version: 1.12.1
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 26
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: null host bridge overlay
Swarm: active
 NodeID: 759bq2mdovhtcdklvditk3kd4
 Is Manager: true
 ClusterID: evem90a6vnmgzpgvyp0kcimrc
 Managers: 1
 Nodes: 3
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
 Node Address: <IP_ADDRESS>
Runtimes: runc
Default Runtime: runc
Security Options: apparmor seccomp
Kernel Version: 4.4.0-36-generic
Operating System: Ubuntu 16.04.1 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 992.5 MiB
Name: engine1
ID: K6VW:TLMS:3G4J:2GJR:VUJQ:ZOJD:7BJD:QGFW:N7C4:2BGT:MXZO:PXXW
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Insecure Registries:
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
Digital Ocean, Ubuntu 16.04

ping @mrjana

@michaelkrog Can you show the output of docker service ps previsto-site and cat /proc/net/ip_vs inside the container from where you did the curl?

@mrjana I had to get my swarm working so I removed the 2 workers I had – which fixes the issues I had.
But it is very easy to reproduce an unstable swarm for me. In fact I have not been able to produce a stable swarm using Ubuntu 16.04 in Digital Oceans cloud via private networking.
Would it be of any help if I build up an unstable swarm and give you SSH access via your public SSH key?

@michaelkrog Yes that would be helpful. Thanks.

@mrjana I am of to London next week. I'll see if I can make it before then. Otherwise I'll have to wait  till 4th of October.
Meanwhile, how do I get hold of your public key?

I think this should work https://github.com/mrjana.keys

@michaelkrog 1.12.2 is out. If you got a chance can you try it out and let me know if this is fixed?

@mrjana I will.
I am sorry I did not back to you on this as promised.

@mrjana It finally happened! :)
Recreated the setup I had earlier and update to 12.2. And it works. 👍
I did have a minor bump in the road though. I had to scale my proxy-service to 0 and up again in order for it to be able to contact new instances. After that if seems to work just fine.
Awesome work!


prefer native libraries

This will check for WebCrypto API in browser environment or for Crypto NodeJS module in node environment to replace AES encrypt and decrypt methods with the pure javascript implementation from aes-js
It will also make use of secp256k1 library which makes use of bitcoin-core bindings in NodeJS enviroment and will fallback to pure javascript elliptic library in Browser enviroment

replaced by #3


On error, the editor is focused and the WASD keys are typed in the code

Godot version
4.0 rc5
System information
Windows 10
Issue description
On code error, the editor is focused and the WASD keys are typed.
The code editor shouldn't be focused in this case.
Steps to reproduce
I'm testing the game, moving the character with the WASD keys.
If an error happens, the game goes in background and the editor is focused.
Because a movement keys is pressed, it gets written in the code.
Nine times out of ten I quickly edit the unwanted characters, but sometimes I don't notice it and in the next run I get an error.
Annoyingly, the error is not about the typo, but some more generic issue in another script.
For example something like "Parser Error: Could not resolve super class inheritance from "Bullet"." where that script is perfectly ok.
Thank you.
Minimal reproduction project
N/A

On code error, the editor is focused and the WASD keys are typed.
The code editor shouldn't be focused in this case.

This is done by design, as many people have requested the ability to be able to immediately edit their code after encountering a script error.

I see.
Would be great to have an option to manually focus before writing code.
Thank you.

Duplicate of https://github.com/godotengine/godot/issues/21812.


Add a tip on best pratices for Taipy/Template

Description
Templates are not put in front of Taipy documentation even if they are extremely useful. It can answer a lot of concerns. See doc and issue.
Expected change
Create a tip in the Knowledge base section on how to use template and why with other best practices.
You should make the changes of the documentation in this taipy-doc repo.

I set a blocked label cause we need to re-discuss the template concept in Taipy.


[BUG] Custom shader in SKCanvasView crashes .NET MAUI App

I am building a .NET MAUI app that's designed to run on iOS, specifically on an iPad Pro. In it I need to be be able to sample sections of an image based on a number of parameters and display that on screen, so I thought the best way to do that would be through a shader and the SkiaSharp SKRuntimeEffect and SKShader classes seemed like a good way to do it. I made a custom control that inherits from SKCanvasView and wrote an SkSL shader. However, whenever the code reaches the DrawRect() call in the OnPaintSurface method of the class, the app crashes immediately.
I have made a bare bones app that has only a custom control that inherits from SKCanvasView with a trivial shader that just sets each pixel colour to red and the crash still happens, which can be found here: https://github.com/LFLumirithmic/SkiaShaderTest
The most relevant part of the code is this section in the OnSurfacePaint method:
var src = @"
    float4 main(float2 fragCoord) {
        return float4(1,0,0,1);
    }";
using var effect = SKRuntimeEffect.Create(src, out string error);
using var shader = effect.ToShader(true);

using (SKPaint paint = new SKPaint() { Shader = shader })
{
    try
    {
        canvas.DrawRect(info.Rect, paint);
    }
    catch (Exception e)
    {
        Debug.WriteLine($"Exception {e.GetType()}: {e.Message}");
    }
}

The call to canvas.DrawRect() instantly crashes the app. I ran it on an iPad Pro and in an iPad simulator, where it crashes with no exception thrown and no console output provided. I ran it on the Mac Mini used to build to the iPads and there it crashes without throwing an exception, but gives me the following output: https://github.com/LFLumirithmic/SkiaShaderTest/blob/main/mac-console-output.rtf
I also ran it on a Windows 11 laptop, where it crashes in the same place, but throws the following exception:
System.Runtime.InteropServices.SEHException
  HResult=0x80004005
  Message=External component has thrown an exception.
  Source=<Cannot evaluate the exception source>
  StackTrace:
<Cannot evaluate the exception stack trace>

If I create one of the default shader effects as an SKShader object using the provided C# API, e.g.:
using (SKPaint paint = new SKPaint())
{
    SKShader shader = SKShader.CreateLinearGradient(
        new SKPoint(0, 0),
        new SKPoint(1000, 1000),
        new SKColor[]
        {
            new SKColor(0,0,255),
            new SKColor(0,255,0)
         },
         null,
         SKShaderTileMode.Repeat
     );
     paint.Shader = shader;
     canvas.DrawRect(info.Rect, paint);
}

That works perfectly. But even the trivial custom shader shown above doesn't work.
I am using version 2.88.3 of SkiaSharp inside Visual Studio through the NuGet package manager, which is the latest as far as I can tell.
I see that issue #2230 mentions the same problem, but it's phrased as a feature request suggesting a specific solution, so I figured I would open this as a broader question.
Am I doing something wrong here and, if so, what? Or is this an issue in SkiaSharp with .NET MAUI? In a few other issues, I've seen mentions of SKSL not being able to run on CPU in the current version of SkiaSharp, but I want it to run on GPU. Is there some sort of flag or setting or registration step I'm missing?

As of today, SKRuntimeEffect is not supported on a raster/bitmap canvas and only on the GPU.

@mattleibow do you have plans to add support in near future? Or it's out of the scope currently?

You should be able to run shaders with a GPU view, but I have not yet ported them over to Maui as the new versions of the various platforms seem to have dropped OpenGL support. Adding the new versions will probably only happen once I finish the update of the skia native library.


Change default branch to "main" instead of "master"

Just a suggestion to keep up with the times.
Great plugin btw, thanks for making it available!

thanks for the suggestion, renamed default branch to main for both the repo and the action itself


New content date is invalid

- Do you want to request a feature or report a bug?
bug
- What is the current behavior?
Creating new collection item results in an ambiguous date being created in the YAML front matter. It should be an ISO date but is instead "date: 03/22/2010 7:06 PM"
- If the current behavior is a bug, please provide the steps to reproduce.

Create a collection with a field like {label: "Date", name: "date", widget: "datetime"}
Load the CMS and create a new item
Select a date in the datetime field
Save the new item

- What is the expected behavior?
An ISO date should be used so that Hugo can build the pages generated by the Netlify CMS.
E.g. "Date: 2010-22-03T19:06:00+00:00"
- Please mention your node.js, and operating system version.
<EMAIL_ADDRESS>Chrome 60
- Workaround
Use a standard string widget instead and manually manage the ISO format dates. Not a big problem for a technical user but a typical CMS user may struggle more.

Thanks for reporting!

My current workaround is adding a format property to the widget config.
- { label: "Date", name: "date", widget: "datetime", format: "Y-MM-DDThh:mm:ssZ"}

@BlueChili , there is only Y and not YYYY ?
Maybe this is the problem I have. I tried it with:
- {label: "Conference Start", name: "begin", widget: "date", format: "YYYY-MM-DD", required: true}

This is addressed in 0.7.6 - moment.defaultFormat which is ISO, is now used if no format is provided.
Note: prior to 0.7.6, the formatting field wasn't actually working, but it seems setting any format value at all at least caused the default format to be used. At any rate, formatting now works correctly, also as of 0.7.6.

I am still getting this bug


[stable/ambassador]: allow get,list,watch on CRDs

@Flydiverny @kflynn @nbkrause
What this PR does / why we need it:
Currently, with chart version 2.6.1, ambassador can't see the CRDs it created (a new check was added with 2.6.1, this was not done before). As a consequence, ambassador disabled CRD support entirely.
Working with the CRDs itself works, but the new check needs get/list permission on CRDs in general. This PR adds this required permission
There was an official patch for ambassador reference YAMLs, but it didnt make it into the Chart it seems. Look here: https://github.com/datawire/ambassador/commit/c0ccd8ecd56f8d03a509d46fbbd0a2c8c5f53b43
Which issue this PR fixes
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test
Thanks!

/lgtm


Add SignMessageObject to signMessage function

Purpose
The browser wallet can now sign bytes as well. Expanding the library to enable passing an object (with schema) to be signed.
Checklist

[x] My code follows the style of this project.
[x] The code compiles without warnings.
[x] I have performed a self-review of the changes.
[x] I have documented my code, in particular the intent of the
hard-to-understand areas.
[x] (If necessary) I have updated the CHANGELOG.

I can't get the call with a primitive string type to work with the mobile wallet: The parameter serialization fails as boiled down to the test
const { Buffer } = require('buffer/');
const { serializeUpdateContractParameters } = require('@concordium/common-sdk');

test('serialize plain string parameter', () => {
    const contractName = 'test';
    const receiveName = 'string';
    const parameters = 'xyz';
    const schema = 'FgI';
    const schemaVersion = 2;
    const res = serializeUpdateContractParameters(
        contractName,
        receiveName,
        parameters,
        Buffer.from(schema, 'base64'),
        schemaVersion
    );
    console.log({res});
});

which fails with the error
unable to deserialize parameters, due to: Parse error

@concordium/common-sdk is at version 6.4.1 which is the current version.
To run the test, add the test file to the project root, install jest (yarn add --dev jest) and run jest.
The test is based on the following example contract written by @DOBEN (deployed on testnet with index 4239):
use concordium_std::*;
#[derive(Serialize)]
struct State;
/// Init function that creates a new contract.
#[init(contract = "test")]
fn init<S: HasStateApi>(
    _ctx: &impl HasInitContext,
    _state_builder: &mut StateBuilder<S>,
) -> InitResult<()> {
    Ok(())
}
pub type Number = u64;
#[receive(
    contract = "test",
    name = "number",
    parameter = "Number",
)]
fn number<S: HasStateApi>(
    ctx: &impl HasReceiveContext,
    _host: &impl HasHost<State, StateApiType = S>,
) -> ReceiveResult<()> {
    let _ = ctx.parameter_cursor().get()?;
    Ok(())
}
pub type Bool = bool;
#[receive(
    contract = "test",
    name = "bool",
    parameter = "Bool",
)]
fn bool<S: HasStateApi>(
    ctx: &impl HasReceiveContext,
    _host: &impl HasHost<State, StateApiType = S>,
) -> ReceiveResult<()> {
    let _ = ctx.parameter_cursor().get()?;
    Ok(())
}
pub type St = String;
#[receive(
    contract = "test",
    name = "string",
    parameter = "St",
)]
fn string<S: HasStateApi>(
    ctx: &impl HasReceiveContext,
    _host: &impl HasHost<State, StateApiType = S>,
) -> ReceiveResult<()> {
    let _ = ctx.parameter_cursor().get()?;
    Ok(())
}

which produces the schema
{
  "contractName": "test",
  "entrypoints": {
    "bool": {
      "parameter": "AQ"
    },
    "number": {
      "parameter": "BQ"
    },
    "string": {
      "parameter": "FgI"
    }
  }
}

@shjortConcordium @abizjak is there something wrong with the way I call serializeUpdateContractParameters? What should be done to make the test pass?

If you want to use serializeUpdateContractParameters you need to use the schema for the entire module. (Not the schema for the parameter itself)
Otherwise you can use serializeTypeValue(parameters, Buffer.from(schema, 'base64')) .

If you want to use serializeUpdateContractParameters you need to supply the schema for the entire module. (Not the schema for the parameter itself, like "FgI" is) Otherwise you can use serializeTypeValue(parameters, Buffer.from(schema, 'base64')) .

How does the implementation tell which kind of schema is supplied? What does the browser wallet do? Try both and see which one works?

@DOBEN I'm suggesting a different approach in https://github.com/Concordium/concordium-dapp-libraries/pull/35 which is consistent with #34, which was just merged.

Closing thiis MR:
Changes that were implemented in
https://github.com/Concordium/concordium-dapp-libraries/pull/34
https://github.com/Concordium/concordium-dapp-libraries/pull/35
were tested in and satisfy the requirements so that this MR is obsolete.


DS 12.4.0: Angular app cannot start anymore

Detailed description
After upgrading from DS 12.3.0 to DS 12.4.0 our angular app has a compile error during startup and therefore is not functional.
Here is the error message as screen dump:

What did you expect to happen? What happened instead? What would you like to see changed?

Application starting and working without errors

What browser are you working in?

chrome
Steps to reproduce the issue
Let me know in case you need an example-app branch to verify this problem.

@svene fixed with 12.4.1

Thanks for the quick fix !


Allow relative paths for --extensions

also misc cleanup

To test in development, add  --extensions ../paranext-extension-template/dist (or similar) to the end of the start:main script in package.json. Otherwise build the app (npm run package) and test with that.

This change is

src/extension-host/services/extension.service.ts line 113 at r1 (raw file):
Previously, tjcouch-sil (TJ Couch) wrote…
Ok great; that works for me!

https://github.com/paranext/paranext-extension-template/wiki/Debugging-Your-Extension-in-the-Production-Application has been updated now. The CWD is always where the app is run from in development or production so paths are relative to where it is run from.


learn-ocaml build -j 2 is broken

Bug description
It seems there's a race condition in learn-ocaml build that prevents us from safely using the --jobs option.
To Reproduce
Steps to reproduce the behavior:

Clone learn-ocaml (git clone https://github.com/ocaml-sf/learn-ocaml.git)
Run cd learn-ocaml
Docker-pull learn-ocaml master (docker pull ocamlsf/learn-ocaml:master)
Run docker run --rm -it -v "$PWD/demo-repository:/repository" ocamlsf/learn-ocaml:master -j 2

Then, there is high probability that we get:
Learnocaml v.0.12 running.
Updating app at ./www
Cannot process exercises: Unix.Unix_error(Unix.EEXIST, "mkdir", "./www/static")

if the repository contains 2+ exercises (which is now the case of demo-repository).
Expected behavior
No error should happen.

I have examined this a bit, and it attempts to use Lwt_unix.fork() which just can't be made to work reliably (see e.g. https://github.com/ocsigen/lwt/issues/812 and there are others like it) ; duplicating the Lwt state in the child process is just going to result in a mess, I am even puzzled that there appears to have been an Lwt version where it worked at some point.
So the option should just be disabled at this point. The more reasonable implementation could be:

define a dedicated CLI command to compile a specific exercise
run a whole process with that command inside the Lwt_list.map_p on each exercise, using e.g. Lwt_process.exec. The difficulty is to make sure that all relevant options in the current context are properly passed to the child process.

Or, of course, we could wait for multicore which should make it fairly trivial :rocket:

Hi @AltGr, thanks a lot for investigating, and for these pointers.

Or, of course, we could wait for multicore which should make it fairly trivial

Yes, that sounds a good trade-off (and replacing the learn-ocaml build -j option with a -j 1 fallback + warning, for now :)
BTW, I've just skimmed the open issues, and it seems this had been spotted some time ago;
(just putting the URL here to link both issues: https://github.com/ocaml-sf/learn-ocaml/issues/158)


Document method/provide module to access pyclowder on Roger and elsewhere

Description
Environmental Logger (EL) now uses pyclowder.geostreams which is not provided in any module on roger. Formerly EL could be fully exercised from command line after module load gdal-stack-2.7.10.  Would be great to have a generic module that allowed EL to run with geostreams from command line for testing purposes.

Would also be nice (for the purposes of modularity / flexibility) if EL continued to produce the netcdf file even if geostreams / pyclowder are not available (e.g. try-catch) so that @czender's original workflow does not fail.

This should be handled as a virtualenv in Python. Basically, create a
virtualenv for EL and have a requirements.txt file with the following:
wheel==0.24.0
pika==0.10.0
requests==2.11.1
urllib3==1.19
https://opensource.ncsa.illinois.edu/bitbucket/rest/archive/latest/projects/CATS/repos/pyclowder2/archive?format=zip
Then 'pip install -r requirements.txt'. If that sounds like gobbly-gook,
let me know and I can write-up better explanation for developing/running
extractors on ROGER.
On Thu, Apr 13, 2017 at 4:39 PM David LeBauer<EMAIL_ADDRESS>wrote:

Would also be nice (for the purposes of modularity / flexibility) if EL
continued to produce the netcdf file even if geostreams / pyclowder are not
available (e.g. try-catch) so that @czender https://github.com/czender's
original workflow does not fail.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/terraref/computing-pipeline/issues/299#issuecomment-294028666,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACdK7JNGYJxURAMPwO0yfUfHa-44Yrbwks5rvpX5gaJpZM4M89YZ
.

@czender can this issue be closed?

yes


Add threshold option

This adds a threshold option to override the default 128 to switch between white and black.
Note that as side-effect on the compiler level, this also makes options.white and options.black independently optional. I would've liked to keep null values at the perimeter, but that would only have been possible with a breaking change such as:
interface YiqOptions {
  colors?: {
    white: string;
    black: string;
  }
  threshold?: number;
}

I don't mind releasing a new major version in this situation. I also like this approach better:
interface YiqOptions {
  colors?: {
    white: string;
    black: string;
  }
  threshold: number;
}

What about we take this possible new major version opportunity to rename the black and white options to dark and light, respectively?

Thanks for the pull request, that is indeed a nice addition! :-)

@lffg Thank you for considering my PR! I think both things you propose make a lot of sense, so I've made both changes.

Thank you! :)

@m1010j, I've just released it!
https://www.npmjs.com/package/yiq/v/4.0.0
https://github.com/lffg/yiq/releases/tag/v4.0.0

@m1010j, I've just released it!

https://www.npmjs.com/package/yiq/v/4.0.0
https://github.com/lffg/yiq/releases/tag/v4.0.0

Awesome!


Tweak behaviour of NanoWidget as subwidget of another NanoWidget

This would fix the weird behavior I've described in https://github.com/DISTRHO/DPF/issues/68.
I really don't know if this is a proper fix, but it works with Wolf Shaper and the rectangles example I've shown in the issue.
It makes the subwidgets pass through this OpenGL code, which sets the correct viewport: https://github.com/DISTRHO/DPF/blob/6d35b690e5399ceaf52b42443b0b3a94587122ac/dgl/src/WidgetPrivateData.hpp#L81-L88
So, calling rect(0, 0, getWidth(), getHeight()); should draw a rectangle equal to the widget's viewport.
Also, in onNanoDisplay(), you can freely use NanoVG's translate(), reset(), etc. It doesn't seem to affect the OpenGL viewport.

I rather not do this until I am 100% sure it is the right thing.
Sorry it is taking long.

No worries, take your time. I'm not sure this is OK either.

I don't understand why skipDisplay is needed. Why not draw all widgets directly, without special treatment for subwidgets?
In pseudocode:
for each widget in window:
    if widget is visible:
        setup the glViewport
        call widget->onDisplay()

The pugl-upstream-v2 branch appears to fix my original issue. Closing this PR, as the code is probably not relevant anymore.


DidService does not update the cache consistently on store

... neither does it store the encoded did consistently (i.e. not always pretty)

PR: https://github.com/walt-id/waltid-ssikit/pull/267


Revert "Revert "[BinderTransport] Allow finding Java class using custom class finder (#28348)" (#28496)"

Was missing <functional> and <string>, for some reason it didn't fail in bazel build

Hi Alisha sorry I didn't check the CI careful enough last time, this time it looks fine. Could you help stamp the reland? Thank you!


Add RSA SecurID

I think the build failure is a transient failure with the source site.

Thank you for the contribution. It needed some fixes, so they were made in 140a0dc85b0f1b4aec14b023edd41bf21554293a. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


[TfLite] unresolved TfLiteGPUDelegateV2Create with Visual Studio

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.13
Custom code
No
OS platform and distribution
Windows 10 Pro
Mobile device
No response
Python version
3.11.4
Bazel version
No response
GCC/compiler version
Microsoft Visual Studio 2022 C++ compiler
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
I built TfLite with -DTFLITE_ENABLE_GPU=ON.
I tried to test TfLite with GPU, but the minimal-working C++ example for the GPU from https://www.tensorflow.org/lite/android/delegates/gpu_native does not work under Windows 10 with Visual Studio 2022.
I receive the linker error error LNK2001: unresolved external symbol __imp_TfLiteGpuDelegateV2Create.
I ran dumpbin on tensorflow-lite.lib and it says a static fuction TfLiteGPUDelegateV2Create does exist.
I tried a different Windows 10 machine with Visual Studio 2019, but I receive the same linker error.
I got the example working under Ubuntu Linux 23.04 with gcc12 using the same build commands (except from replacing the Windows specifics with Linux specifics of course).
Standalone code to reproduce the issue
Build commands (in Command Prompt):
git clone https://github.com/tensorflow/tensorflow tensorflow_src
mkdir tflite_release_x64
cd tflite_release_x64
cmake -G "Visual Studio 17" -A x64 -DTFLITE_ENABLE_GPU=ON ..\tensorflow_src\tensorflow\lite
cmake --build . -j 16 --config Release

C++ code (https://www.tensorflow.org/lite/android/delegates/gpu_native):
#include "tensorflow/lite/model.h"
#include "tensorflow/lite/interpreter.h"
#include "tensorflow/lite/kernels/register.h"
#include "tensorflow/lite/delegates/gpu/delegate.h"
#include <iostream>

using namespace tflite;

int main() {
	// Set up interpreter.
	auto model = FlatBufferModel::BuildFromFile("C:/Users/bartp/source/lite-model_deeplabv3_1_metadata_2.tflite");
	if (!model) return false;
	ops::builtin::BuiltinOpResolver op_resolver;
	std::unique_ptr<Interpreter> interpreter;
	InterpreterBuilder(*model, op_resolver)(&interpreter);

	auto* delegate = TfLiteGpuDelegateV2Create(/*default options=*/nullptr);

	std::cout << "Done\n";
	return 0;
}

Relevant log output
1>------ Build started: Project: MweTfLite2.13Gpu, Configuration: Release x64 ------
1>Main.cpp
1>Main.obj : error LNK2001: unresolved external symbol __imp_TfLiteGpuDelegateV2Create
1>C:\Users\bartp\source\MweTfLite2.13Gpu\x64\Release\MweTfLite2.13Gpu.exe : fatal error LNK1120: 1 unresolved externals

Hi @misterBart, I'm in the process of replicating but am running into Windows installation restrictions, just letting you know I am looking into this.

Update: After removing macro TFL_CAPI_EXPORT (= __declspec(dllimport)) before the function declaration TfLiteGpuDelegateV2Create the program works.
I never heard of __delcapsec(dllimport) before, and after some Googling I get the impression that it is a performance optimization for loading a dynamic library.
The first question is of course why this caused an unresolved external error in Windows / Visual Studio (and not in Linux with gcc).
The second question is whether my fix by removing TFL_CAPI_EXPORT is a proper fix.
Do you/anybody have thoughts about this?

Hi @terryheo, can you please take a look? Thanks.

TFLITE_ENABLE_GPU isn't supported for Windows.
https://www.tensorflow.org/lite/guide/build_cmake#available_options_to_build_tensorflow_lite

In my experience, the documentation often lags behind the implementation.
The GPU code contain Windows-specific code (e.g. opencl_wrapper.cc), that wouldn't make sense if Windows is not supported.
The GPU code contains a delegate for iOS' Metal, also unmentioned in the documentation.

Hi, @misterBart
Thanks for raising this issue. Are you aware of the migration to LiteRT? This transition is aimed at enhancing our project's capabilities and providing improved support and focus for our users. As we believe this issue is still relevant to LiteRT we are moving your issue there. Please follow progress here: https://github.com/google-ai-edge/LiteRT/issues/161
Let us know if you have any questions. Thanks.


fix: improper footer logo blending with background 🛠

Related Issue
Close #1094
Description

Fixed the Logo image blending issue with the background color of the Footer component

Screenshots

BEFORE
AFTER

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@0xabdulkhalid how can your raise PR even before, it's assigned?

Oops sorry for that @SiddhantPawar03.


Add categories for nodes

The list of actions is going to get awfully long. Add some categories to split them up a bit. Should only need one level of categories.

There are categories now but the list could really use a tree view


[<IP_ADDRESS>0] Bug: Transaction fails transferring NFT

I ran across this while trying to do an NFT transfer from Boom wallet.
Bug found testing Stacks Wallet for Web.
Error
Loadable: transaction.signedTransaction
Message: TypeError: t.value.toArrayLike is not a function
Environment
Version: <IP_ADDRESS>0
Branch: release/canary
Commit: df21468

duplicate of #935 ... closing

duplicate of #935 ... closing


about subgraph Gs

Thanks for your amazing work! But where does the code reflect the process of obtaining subgraphs (Gs) through sampling.
If you see this issue, please tell me the answer.Thanks in advance!

Hi Zongzhen, thanks for your questions!
In our implementation, we'll first obtain (unnormalized) node/edge attention weights by the line here, where sigmoid(att_log_logits) essentially makes it the $p$ in our paper. To make the sampling process differentiable, we utilize the gumbel-softmax trick. So, att obtained here is the $\alpha \sim \operatorname{Bern}(p)$ in our paper.
But in practice, we don't yield $G_S$ by doing $\alpha \odot A$, because based on the gumbel-softmax trick it's non-trivial to make this operation differentiable for message-passing-based neural networks (MPNNs). Instead, no matter if att is node-level or edge-level attention, we first lift it to edge-level to obtain edge_att. After that, edge_att will act on the message of the corresponding edge, see here. And this is like an approximation of $\alpha \odot A$ when using the gumbel-softmax trick for MPNNs.
Hope this could resolve your question. But feel free to let us know if you have other questions!

Thanks for your detailed reply, which helped me a lot.


Proposal: return direct link for download

Is it possible to return direct for downloading uploaded build?
I mean that it would be very easy to have direct link integrated with Jira or smth else to download certain build for certain PR in case of CI/CD

+1

It isn't possible right now, but it's a feature that the team would like to work on in the future.

@cvaghela ok, thanks


Pasting images from Clipboard into Beaker

I have a lot of situations where it would be really useful to be able to put a pre-existing image into a Beaker notebook. The most common situation is that I want to grab a screen shot of something - a snippet from a paper I'm reading, a plot someone else made, or a web page with relevant content. These often add extremely useful / important contextual information about the problem I'm working on in the notebook. Currently I have some workarounds but they are all quite awkward.
I notice that tools like GitLab have got an extremely nice ability to capture and upload images straight from the clipboard. This would be an incredibly nice feature to have in Beaker!

Beaker supports drag-and-drop of images into the notebook.  But you are right, doing it from the clipboard with be good too.

actually paste already works as well!
it will be released in v1.7

needs to be documented

Thanks, this is going to be awesome!

you can run 1.7pre1 from the prerelease docker container:
https://hub.docker.com/r/beakernotebook/beaker-prerelease/
it has this feature.

OK actually I found a bug:
https://github.com/twosigma/beaker-notebook/issues/4737

https://github.com/twosigma/beaker-notebook/pull/4738


Blog Post:  Writing my first custom CICD action

Blog Post
I recently wrote my first custom CICD action (checking the tags on blog posts) and would like to share the process and my learnings.

Way cool!!


mtl: Use pointers for temporary state

Partially fix to #2161
PR checklist:

[ ] make succeeds (on *nix)
[ ] make reftests succeeds
[ ] tested examples with the following backends:

This replaces buffer/texture/sampler object usage with pointers instead. This allows us to bypass Objective-C's reference counting (i.e. retain/release). The idea is to create the same objects as before, but purposely leak them and acquire a pointer to them instead. When we're finished with the objects, we take back ownership and release as usual. We should be able to do this either through drop on the owning struct, or through relevant destroy_* functions.
Still heavily WIP (but works with quad/Dota 2) because there are a few places that will cause leaks or use-after-free in its current state (need to fix cases where the same image is returned for an image view, verify drop exists on all the correct owners, etc.)

How does this approach ensure safety? It seems like it would be error prone.

@jrmuizel It would be nice to rely on Objective-C to reference count these correctly, but we shouldn't really need it. The objects we expose externally have well-defined lifetimes matching the Vulkan spec. That's why we could decide to use the destroy_* functions from Vulkan (which no-op currently) instead of drop.
We only have to be really careful while handling temporary objects that we create internally. We already have to work around this issue in some other places, for example we have retained_buffers and retained_textures to hold onto our own temporary buffers/textures until the owning command buffer has been reset or dropped.

I'm going to think about this some more and try to restrict pointer usage to high frequency retains/releases as @kvark mentioned on Gitter

Updated the commit – slightly safer now with less pointer usage for low frequency calls

@kvark thanks, updated to remove SamplerPtr changes to internal

Bors r+

On Jun 24, 2018, at 22:56, Josh Groves<EMAIL_ADDRESS>wrote:
@kvark thanks, updated to remove SamplerPtr changes to internal
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Add MySQL connection

Add a connection to limberframework.database.connections to allow connecting to a MySQL database using SQLAlchemy, i.e. using the mysql driver.

Commit 7d08e6fa3a63273c9f27008bff35725537b152ea


feat: add support for Bot API 6.6

Adds support for Bot API 6.6. Migrates to Deno tooling. Gets rid of proxy types. Prepares for the next major version.
BREAKING CHANGE: Adjustments for InputFile instances now have to be done directly on the exported objects.

Merging without review after exhaustive testing. Will wait for confirmation on the core repo, though.


83 manage biocurations

Codecov Report

Merging #2781 (cfc2b7f) into development (ab3f287) will decrease coverage by 4.78%.
The diff coverage is n/a.

:exclamation: Current head cfc2b7f differs from pull request most recent head d0b643f. Consider uploading reports for the commit d0b643f to get more accurate results

@@               Coverage Diff               @@
##           development    #2781      +/-   ##
===============================================
- Coverage        87.14%   82.36%   -4.79%
===============================================
  Files             1877     1729     -148
  Lines            65953    55916   -10037
===============================================
- Hits             57477    46055   -11422
- Misses            8476     9861    +1385

Impacted Files
Coverage Δ

app/models/protonym/becomes.rb
16.43% <0.00%> (-79.46%)
:arrow_down:

lib/catalog/description_from_observation_matrix.rb
13.22% <0.00%> (-78.52%)
:arrow_down:

lib/batch_load/import/otus.rb
24.24% <0.00%> (-75.76%)
:arrow_down:

lib/queries/concerns/date_ranges.rb
41.26% <0.00%> (-55.56%)
:arrow_down:

lib/utilities/dates.rb
33.26% <0.00%> (-55.24%)
:arrow_down:

lib/queries/concerns/tags.rb
35.71% <0.00%> (-53.58%)
:arrow_down:

lib/catalog/otu/entry.rb
38.09% <0.00%> (-52.39%)
:arrow_down:

app/models/collecting_event/geo_locate.rb
28.57% <0.00%> (-52.39%)
:arrow_down:

lib/catalog/timeline.rb
50.00% <0.00%> (-50.00%)
:arrow_down:

lib/queries/citation/filter.rb
42.42% <0.00%> (-48.49%)
:arrow_down:

... and 253 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 9bc88ce...d0b643f. Read the comment docs.


Figure code include in the paper

Dear Daniele,
Thanks for the beautiful work, I learn a lot.
I was wondering is the figure's in the paper also uploaded in to this github page?
Best,
Yu.

Dear @YuWang07,
Thank you for your interest in our work. I have included @DavideMaspero who can help you with your issue.
All the best,
Daniele

Dear Yu,
I uploaded a new directory called figures, where you can find the 'generate_plots.R' script. Simply move to that directory with R/Rstudio and run/source the script.
Required dependencies are listed in the first lines of the code.
Please, let me know if you encounter any problems.
Best,
Davide


"error 301: Application descriptor missing/cannot be opened" when packaging AIR desktop release (shared runtime)

error 301: Application descriptor missing/cannot be opened

asconfig.json
{
	"compilerOptions": {
		"output": "bin-debug/ExcelReadtoJson.swf",
		"source-path": [
			"src"
		],
		"warnings": false
	},
	"mainClass": "ExcelReadtoJson",
	"application": "src/ExcelReadtoJson-app.xml",
	"copySourcePathAssets": true,
	"additionalOptions": "-locale en_US",
	"airOptions": {
		"output": "bin-debug/Main.air",
		"signingOptions": {
			"storetype": "pkcs12",
			"keystore": "Tidus1.p12"
		}
	},
	"config": "air"
}

when package to desktop released(shared runtime) will got this error. Of course the ExcelReadtoJson-app.xml is there.

Does ExcelReadtoJson-app.xml exist in both src and bin-debug?

Does ExcelReadtoJson-app.xml exist in both src and bin-debug? I'm just checking that it was properly copied to the output folder during compilation.

Yes,both there . I try to change the 'filename' and 'name' in the xml to pure English words, because I found some chinese word in it will get this error too, but dosen't work..

Can you create and a simple project that reproduces this issue and attach it here?

Σ(⊙▽⊙"ha....I found it . I have a chinese word in the xml's annotation...delete it and every thing is ok.

    <manifestAdditions><![CDATA[
    <manifest android:installLocation="auto">
            <!-- sdk的权限 -->
            <uses-permission android:name="android.permission.INTERNET"/>

I have the same problem, but can't find any Chinese word.

error 301: Application descriptor missing/cannot be opened

@danielkotzer This is an error from Adobe AIR's adt command. I doubt it's a vscode-as3mxml issue.
You either have a wrong path somewhere, or maybe you have a different invalid Unicode character in the AIR descriptor file. It doesn't necessarily need to be a Chinese character specifically. Maybe emoji or other languages cause the same issue.
To help with debugging, you might want to set as3mxml.asconfigc.verboseOutput to true in settings. If you run the build after that, you'll see the full adt command, which might give you a clue if any paths are wrong.


Could we support Fuyu-8B, a multimodel llm?

Hi,
Fuyu 8B is a multimodel llm model, could we support it in vllm?
https://www.adept.ai/blog/fuyu-8b
It seems to me current vllm only could support pure text, so for this kind of multimodel mixing with image, how could we handle it?
Thx~

now we added supported for llava, this is welcomed!

I would like to work on this model. But it seems that the persimmon used as language model in Fuyu-8B hasn't been supported.
Maybe we can support it first?


Publishers for Windows/Linux DSC/CustomScript Extensions are different

[Enter feedback here]

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

ID: 26b29b2f-783a-63a0-a1af-e2ce4d321387
Version Independent ID: 97597ce9-dc46-960f-6ad7-047bfe936d48
Content: VM extension management with Azure Arc for servers
Content Source: articles/azure-arc/servers/manage-vm-extensions.md
Service: azure-arc
Sub-service: azure-arc-servers
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@raghushantha Thank you for sharing the feedback , our team will look into it and get back to you at the earliest.

This feedback is directed to an article that is currently under development and has not been published.  Raghu mistakenly filed a GitHub issue and I'm addressing that via internal channel.  #please-close


PHPLIB-954: Add return types to all methods

PHPLIB-954
This PR changes phpcs to require native return types, adding current types using phpcbf. In addition, all ReturnTypeWillChange attributes were removed.
This PR also includes changes for PHPLIB-1114, replacing all Cursor types with CursorInterface.

Updated the PR. This currently includes changes from #1427, but I'll wait until that PR has been merged up to 2.x and rebasing this PR to preserve proper history.


Show hints

When input takes longer
When wrong letters are input repeatedly

Example
Big
Hint: It's the key left of o
Something like that?

I'm not sure my kid would understand "key left of O"... but we could show an image of the keyboard with the key highlighted?


Console Application protocols section grant types has word references not matching for on prem

How to reproduce:

Access console
Develop > Applications > Console (preview)
Go to Protocols tab
Click on Allowed grant types > Implicit and password
Read the hint provided. Hint has refered as Asgardeo. As this is on prem product we may need to change the wordings accordingly

Environment information (Please complete the following information; remove  any unnecessary fields) :
Is 5.12.0 alpha 19

Observing the same issue in the Myaccount application as well

Fixed with https://github.com/wso2/identity-apps/pull/2956


Scanner for English version

Seems like the've fixed text overlapping for English version. Do you plan to update the scanner tool?

Yes, I do realized that they've solved the problem, but I don't expect them to solve it by simply adding new lines🤦 (tbf it's probably the right way to do it), I'll look into it and see if it's easier to do the recognition using English OCR


TypeError: softmax() got an unexpected keyword argument 'axis'

Hello!
I am trying your tensorflow version code(crfasrnn_keras), but I met an error,the following contents are the errors:
(tensorflow) bruce@ubuntu:~/master_thesis/crfasrnn_keras$ python3 run_demo.py
/home/bruce/anaconda3/lib/python3.6/site-packages/h5py/init.py:36: FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type.
from ._conv import register_converters as _register_converters
Using TensorFlow backend.
Traceback (most recent call last):
File "run_demo.py", line 48, in
main()
File "run_demo.py", line 38, in main
model = get_crfrnn_model_def()
File "./src/crfrnn_model.py", line 111, in get_crfrnn_model_def
name='crfrnn')([upscore, img_input])
File "/home/bruce/anaconda3/lib/python3.6/site-packages/keras/engine/topology.py", line 619, in call
output = self.call(inputs, **kwargs)
File "./src/crfrnn_layer.py", line 91, in call
softmax_out = tf.nn.softmax(q_values, axis=0)
TypeError: softmax() got an unexpected keyword argument 'axis'
and my computer configuration are:
ubuntu 16.04 on VMware player 14, host system is win 10,CPU core I5
Thank you very much for your answer!
Bruce

@dfki-bruce What's your Tensorflow version? I think the cause of this error is an older version of Tensorflow (see https://www.tensorflow.org/api_docs/python/tf/nn/softmax - 'axis' is the right keyword argument for newer versions of Tensorflow).
Updating your Tensorflow version (see https://www.tensorflow.org/install/) should solve this problem. Please let me know if it doesn't.

@sadeepj
Thank you so much for your fast reply!
My TF version is 1.2.1.
I will try to upgrade my TF and let you know the results.
Best,

@dfki-bruce Alternatively, as a workaround for older Tensorflow versions, you can change the parameter name 'axis' to 'dim' in Line 91 of crfrnn_layer.py (https://github.com/sadeepj/crfasrnn_keras/blob/4819142eaf3788f9ce6945edfcc5d73aa0aa7f70/src/crfrnn_layer.py#L91).

@sadeepj ,
Thank you for your  advice!
I just upgraded my TF to 1.7.0,
and the errors are:
(tensorflow) bruce@ubuntu:~/master_thesis/crfasrnn_keras$ python run_demo.py
/home/bruce/anaconda3/lib/python3.6/site-packages/h5py/init.py:36: FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type.
from ._conv import register_converters as _register_converters
Using TensorFlow backend.
Traceback (most recent call last):
File "run_demo.py", line 27, in
from crfrnn_model import get_crfrnn_model_def
File "./src/crfrnn_model.py", line 28, in
from crfrnn_layer import CrfRnnLayer
File "./src/crfrnn_layer.py", line 28, in
import high_dim_filter_loader
File "./src/high_dim_filter_loader.py", line 28, in
custom_module = tf.load_op_library(os.path.join(os.path.dirname(file), 'cpp', 'high_dim_filter.so'))
File "/home/bruce/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/load_library.py", line 58, in load_op_library
lib_handle = py_tf.TF_LoadLibrary(library_filename, status)
File "/home/bruce/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/errors_impl.py", line 516, in exit
c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.NotFoundError: ./src/cpp/high_dim_filter.so: undefined symbol: _ZTIN10tensorflow8OpKernelE
So maybe I will downgrade my TF and try your new advice, change the axis to dim.
Thank you!

@dfki-bruce You'd have to remake the high_dim_filter.so file after upgrading Tensorflow:
$ cd crfasrnn_keras/src/cpp
$ rm *.so
$ make

@sadeepj ,
Thank you again for your solution!
I tried and it partially works,following the error:
Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX AVX2,
however ,strictly speakings, it is not an error according to here
so anyway I will try again,
thanks!

@sadeepj
Good morning!
Finally, the result "lables.png" was generated, following the error
"I tensorflow/core/platform/cpu_feature_guard.cc:140] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA"
and the output pic is not as exactly  same as "sample.png",see attach 1.
So I think it should be the right result as it would be,right?
THanks!
Attach 1

@dfki-bruce That's the correct result. What you're seeing is a Tensorflow warning message (not an error) that you can safely ignore. It has nothing to do with the code in the repo, hence closing this issue.


merge typescript branch

Closing in favor of: https://github.com/sclorg/testing-farm-as-github-action/pull/97


Reactotron app not connecting (cli works fine)

I'd like to give a detailed bug report but tbh I have no idea how to debug the desktop app, it's just not connecting.
I'm using the ignite starter kit, this is my ReactotronConfig:
import Immutable from 'seamless-immutable'
import { StartupTypes } from '../Redux/StartupRedux'
const Reactotron = require('reactotron-react-native').default
const errorPlugin = require('reactotron-react-native').trackGlobalErrors
const apisaucePlugin = require('reactotron-apisauce')
const { reactotronRedux } = require('reactotron-redux')
const sagaPlugin = require('reactotron-redux-saga')

if (__DEV__) {
  Reactotron
    .configure({
      // host: '<IP_ADDRESS>' // default is localhost (on android don't forget to `adb reverse tcp:9090 tcp:9090`)
      name: 'Onboarding' // would you like to see your app's name?
    })

    // forward all errors to Reactotron
    .use(errorPlugin({
      // ignore all error frames from react-native (for example)
      veto: (frame) =>
        frame.fileName.indexOf('/node_modules/react-native/') >= 0
    }))

    // register apisauce so we can install a monitor later
    .use(apisaucePlugin())

    // setup the redux integration with Reactotron
    .use(reactotronRedux({
      // you can flag some of your actions as important by returning true here
      isActionImportant: action => action.type === StartupTypes.STARTUP,

      // you can flag to exclude certain types too... especially the chatty ones
      // except: ['EFFECT_TRIGGERED', 'EFFECT_RESOLVED', 'EFFECT_REJECTED', 'persist/REHYDRATE'],

      // Fires when Reactotron uploads a new copy of the state tree.  Since our reducers are
      // immutable with `seamless-immutable`, we ensure we convert to that format.
      onRestore: state => Immutable(state)
    }))

    // register the redux-saga plugin so we can use the monitor in CreateStore.js
    .use(sagaPlugin())

    // let's connect!
    .connect()

  // Let's clear Reactotron on every time we load the app
  Reactotron.clear()

  // Totally hacky, but this allows you to not both importing reactotron-react-native
  // on every file.  This is just DEV mode, so no big deal.
  console.tron = Reactotron
} else {
  // a mock version should you decide to leave console.tron in your codebase
  console.tron = {
    log: () => false,
    warn: () => false,
    error: () => false,
    display: () => false,
    image: () => false
  }
}

nevermind, app & terminal restart for some reason solved it…

Weird.  Another vote for #63 though.


How to limit the cpu core number in Tensorflow 2.0

I am using tf.keras to train a model with cpu and I want to limit the cpu usage of this program. I found this article(https://www.kaggle.com/c/porto-seguro-safe-driver-prediction/discussion/43383):
from keras import backend as K
import tensorflow as tf

config = tf.ConfigProto(intra_op_parallelism_threads=args.jobs, \
                        inter_op_parallelism_threads=args.jobs, \
                        allow_soft_placement=True, \
                        device_count = {'CPU': args.jobs})
session = tf.Session(config=config)
K.set_session(session)

But in Tensorflow 2.0, ConfigProto is deprecated and Session is gone. What's the correct method to limit cpu usage?

tf.config.threading.set_intra_op_parallelism_threads(1)
tf.config.threading.set_inter_op_parallelism_threads(112)

Have a try of this setting @fancyerii ?

@fancyerii, Please take a look at this link. Thanks!

I tried
tf.config.threading.set_intra_op_parallelism_threads(1)
tf.config.threading.set_inter_op_parallelism_threads(1)

using tensorflow.keras. It doesn't work on Windows 10, Windows 10 WSL and Ubuntu. On Ubuntu and WSL it uses all CPUs, no matter the number I set. On Windows 10 it seems to regulate something, but not exactly the number of CPUs. And what is the difference between intra and inter op?

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

#29968

It's apparently a bug discussed in #29968

@benni93 on the difference between intra and inter op, may have a look at this StackOverflow answer.

This issue still persists. Please Re-Open this issue.
https://github.com/tensorflow/tensorflow/issues/29968#issuecomment-572492815

Tensorflow 2.0 :headphones: :1st_place_medal:
config = tf.compat.v1.ConfigProto(device_count={"CPU": 2})
K.set_session(tf.compat.v1.Session(config=config))

Was able to replicate the issue in TF v2.5,please find the gist here..Thanks !

Was able to replicate the issue with TF 2.6.0-dev20210606,please find the gist here ..Thanks!

hello all,
for me it seems to work with:
tf.config.threading.set_intra_op_parallelism_threads(1)
tf.config.threading.set_inter_op_parallelism_threads(1)

using CentOs and tensorflow 2.5.0 and python 3.6.
I think one crucial factor is finding a commonly agreed way of testing this. I was using: htop -p <pid> and I was validating that without the settings I've got >100% CPU usage and with the settings <=100%, which was my goal, albeit I agree that this test might not be the most sophisticated one :)
In https://github.com/tensorflow/tensorflow/issues/29968, it's discussed that there are many threads and context switches. Which I can confirm, however for my use-case was not relevant and from the CPU utilization point of view - I think - is irrelevant.
If there are 100 threads but only one is running or ~1core can serve all active threads, I'm fine.
@sushreebarsa both of your notebooks have syntax error which is independent of tensorflow. At the same time it does not really matter, because what you try to do is anyhow not supported in tensorflow 2, session is not really supported anymore: https://www.tensorflow.org/guide/effective_tf2#functions_not_sessions

In #29968, it's discussed that there are many threads and context switches. Which I can confirm, however for my use-case was not relevant and from the CPU utilization point of view - I think - is irrelevant.

Many threads and context switches cause cache trashing and inefficiencies, i.e. 100 % CPU load does not translate into 100 % efficient CPU use. You can analyze this further with performance analysis tools such as perf to estimate how much actual CPU cycles are used and how many are lost due to the threading overhead.

I had the same problem. Make sure the first lines in your code are:
import os
os.environ["OMP_NUM_THREADS"] = "1"

And then import everything else. If you import tensorflow or even numpy before that, it won't work. Hope that solves the issue!

@fancyerii please refer to the above comment https://github.com/tensorflow/tensorflow/issues/34415#issuecomment-909201783 and let us know if you still need any help here. If it is resolved then please feel free to move this issue to close status ? Thanks!

Seems TF2 still does not respect these options for simple linear algebra tasks. For example,
inputA = tf.random.uniform([batch_size, problem_size, problem_size])
output = tf.linalg.inv(
            inputA, adjoint=False, name=None
        )
output = tf.reduce_sum(output)

would still use all cores it can find no matter what is set in config.
It seems the only thing that actually works as intended is
os.environ["OMP_NUM_THREADS"] = f"{N}"
os.environ['TF_NUM_INTEROP_THREADS'] = f"{N}"
os.environ['TF_NUM_INTRAOP_THREADS'] = f"{N}"

Maybe would be nice if programmatic change would work too, starting new processes not always convenient.

Tried above solutions, still all cores were used by Tensorflow.

This is still an issue in 2023. Neither the os.environ setting nor the Tensorflow config settings work.


removing portworx_24_1_2 to fix missing annotation

Thanks for submitting your Operator. Please check the below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Are you familiar with our operator pipeline?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want to use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Your submission should not

[ ] Add more than one operator bundle per PR
[ ] Modify any operator
[ ] Rename an operator
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description of the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

Relates: #5330

@acornett21 it failed on the same test.

@mborodin-px That's fine I have merge access here, so once the bundle is removed from the catalog images we can proceed with this, then another PR to re-add with the proper changes in the CSV.


[RFE] connect last-run with a terminal

Currently, the  last-run is user-bound. The problem appears if I run two tmt commands from two terminals to a execute multi-host test.
Use-case:

prepare machines

[terminal1]> tmt run -vvvddd --until prepare test --name . provision --how connect --guest host1
[terminal2]> tmt run -vvvddd --until prepare test --name . provision --how connect --guest host2

run test

[terminal1]> tmt run -e SERVERS=host1 -e CLIENTS=host2 --last test --name . discover -f execute -f --how tmt --interactive
[terminal2]> tmt run -e SERVERS=host1 -e CLIENTS=host2 --last test --name . discover -f execute -f --how tmt --interactive

This should be probably optional so additional switch could be introduced to addintion to --last or completely independent switch --last-tty.

When I tried to workaround it by manually setting --id instead of --last I got following traceback for the second run of the test:
/var/tmp/tmt/run-111

/plans/default
Traceback (most recent call last):
  File "/usr/bin/tmt", line 8, in <module>
    tmt.cli.main()
  File "/usr/lib/python3.9/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python3.9/site-packages/click/core.py", line 1290, in invoke
    return _process_result(rv)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 1224, in _process_result
    value = ctx.invoke(self.result_callback, value, **ctx.params)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/lib/python3.9/site-packages/tmt/cli.py", line 283, in finito
    click_context.obj.run.go()
  File "/usr/lib/python3.9/site-packages/tmt/base.py", line 1025, in go
    plan.go()
  File "/usr/lib/python3.9/site-packages/tmt/base.py", line 570, in go
    step.wake()
  File "/usr/lib/python3.9/site-packages/tmt/steps/execute/__init__.py", line 87, in wake
    super().wake()
  File "/usr/lib/python3.9/site-packages/tmt/steps/__init__.py", line 121, in wake
    self._workdir_cleanup()
  File "/usr/lib/python3.9/site-packages/tmt/utils.py", line 480, in _workdir_cleanup
    shutil.rmtree(directory)
  File "/usr/lib64/python3.9/shutil.py", line 718, in rmtree
    _rmtree_safe_fd(fd, path, onerror)
  File "/usr/lib64/python3.9/shutil.py", line 655, in _rmtree_safe_fd
    _rmtree_safe_fd(dirfd, fullname, onerror)
  File "/usr/lib64/python3.9/shutil.py", line 655, in _rmtree_safe_fd
    _rmtree_safe_fd(dirfd, fullname, onerror)
  File "/usr/lib64/python3.9/shutil.py", line 655, in _rmtree_safe_fd
    _rmtree_safe_fd(dirfd, fullname, onerror)
  [Previous line repeated 4 more times]
  File "/usr/lib64/python3.9/shutil.py", line 659, in _rmtree_safe_fd
    onerror(os.rmdir, fullname, sys.exc_info())
  File "/usr/lib64/python3.9/shutil.py", line 657, in _rmtree_safe_fd
    os.rmdir(entry.name, dir_fd=topfd)
PermissionError: [Errno 13] Permission denied: '.ssh'

apparently this is connected to copying the beakerlib backup directory (https://github.com/psss/tmt/issues/502)