Support AWS IAM Authentication

In order to support IAM Authentication, revamped existing
profile creation. Now user can provide type of auth as flag to
profile creation command.
For IAM, if odfe-cli profile contains AWS profile name, it will be used.
if left bank, it is expected to get credetials from Environment variables.
once it is identified as AWS IAM Authentications, cli will sign the request
before execute.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

profile create command preview
@aetter Can you review the text? Thanks
$ odfe-cli profile create
Error: required flag(s) "auth-type", "endpoint", "name" not set
Usage:
  odfe-cli profile create [flags]

Flags:
  -a, --auth-type string   Authentication type; permitted values are disabled, basic and aws-iam.
                           If security is disabled, provide --auth-type='disabled'.
                           If security is configured to basic HTTP Authentication, provide --auth-type='basic'.
                           If security is configured to use AWS IAM ARN as user, provide --auth-type='aws-iam'. The rest of the options are asked from the tty based on this value
  -e, --endpoint string    Create profile with this endpoint or host
  -h, --help               Help for create
  -n, --name string        Create profile with this name

Global Flags:
  -c, --config string    Configuration file for odfe-cli, default is /Users/balasvij/.odfe-cli/config.yaml
  -p, --profile string   Use a specific profile from your configuration file

Just minor recommendations for --auth-type:
$ odfe-cli profile create
Error: required flag(s) "auth-type", "endpoint", "name" not set
Usage:
  odfe-cli profile create [flags]

Flags:
  -a, --auth-type string   Authentication type. Options are disabled, basic and aws-iam.
                           If security is disabled, provide --auth-type='disabled'.
                           If security uses HTTP basic authentication, provide --auth-type='basic'.
                           If security uses AWS IAM ARNs as users, provide --auth-type='aws-iam'.
                           odfe-cli asks for additional information based on your choice of authentication type.
  -e, --endpoint string    Create profile with this endpoint or host
  -h, --help               Help for create
  -n, --name string        Create profile with this name

Global Flags:
  -c, --config string    Configuration file for odfe-cli, default is /Users/balasvij/.odfe-cli/config.yaml
  -p, --profile string   Use a specific profile from your configuration file

Update usage as per review thanks @aetter
Usage:
  odfe-cli profile create [flags]

Flags:
  -a, --auth-type string   Authentication type. Options are disabled, basic and aws-iam.
                           If security is disabled, provide --auth-type='disabled'.
                           If security uses HTTP basic authentication, provide --auth-type='basic'.
                           If security uses AWS IAM ARNs as users, provide --auth-type='aws-iam'.
                           odfe-cli asks for additional information based on your choice of authentication type.
  -e, --endpoint string    Create profile with this endpoint or host
  -h, --help               Help for create
  -n, --name string        Create profile with this name

Global Flags:
  -c, --config string    Configuration file for odfe-cli, default is /Users/balasvij/.odfe-cli/config.yaml
  -p, --profile string   Use a specific profile from your configuration file

@aetter one more dialog we added. Can you provide feedback?
3c22fb3bf9b7:odfe-cli balasvij$ ./odfe-cli profile create -a "aws-iam" -n "test123" -e "endpoint"
AWS profile name (Leave blank if you want to provide credentials using environment variables): iam
AWS Service name where your cluster is deployed (example: for AmazonElasticsearch, value is 'es', for EC2, value is 'ec2'): es
profile created successfully
3c22fb3bf9b7:odfe-cli balasvij$

AWS profile name (leave blank if you want to provide credentials using environment variables): iam
AWS service name where your cluster is deployed (for Amazon Elasticsearch Service, use 'es'. For EC2, use 'ec2'): es
Profile created successfully.

incorporated Andrew's feedback.


Gracefully handle issues with reconstructing the stack trace from failures

When the exception/throwable object can't be reconstructed form the information in the failure payload then the an exception is sent up the stack and ultimately causes the failure object to be unusable (you can't retrieve it using the DAO). This makes it impossible to use the DAO for things like user interfaces or automated processing/reporting on failures.
This can occur for instance when the job class is not in the same VM as the admin UI or thing processing the failure queue.
I don't know the best way to handle this, but for now what has worked is storing the raw information about the failure as Strings and trying to reconstruct the exception, but not bubbling up the exception when problems occur.
Let me know what you think.

If you can you add those 3 properties to the equals and hashCode method for JobFailure, then I think you covered it! Thanks for taking the time to fix the issue!

Sorry, I missed that. I've added the fields to equals and hashcode (and the constructor that clones an existing job failure) and then fixed up a serialization test.

Thanks!


Problem with select and onchange

Select doesn't seem to work when I use onchange="this.form.submit()". I can't expand it and if I check traffic it keep on posting the same value over and over again to the server.

create pen on codepen.io


Update CONTRIBUTING.md to reflect "help wanted" label requirement

PRs should have an accompanying "help wanted" issue that they are targeting

1 suggestion and 1 nitpick, otherwise LGTM!

Also added a comment on this


Ao 14676 graphql 2nd PR with a few tweaks

all execute layers are called graphql.execute with a :Key to specify

only server errors are reported

errors are reported as error events and not Errors KVs

multiplex requests have a transaction name like graphql.multiplex.

correction to case in AppOpticsAPM in grpc and some other files

correction to the index args in the AppOpticsAPM.backtrace method

Looks two commits look good.


TD-86 TD-78 Dashboard Back End & Detail News

I finished implementing the dashboard back end.

I also added the news and a summary of the company to the details page.
The dependencies have been changed, so you will need to update yours from the requirements.txt file.

ok so I've check out your pull request. Everything looks good except for 2 things.

When I enter an incorrect stock symbol in the search bar I get an exception, I updated the database and install the required dependencies from reqirements.txt so Im not sure if I'm missing something.

Other than that everything else works fine. The info for the stock is accurate. The summary is there along with news articles.

I implemented your second change for the scroll button to remain on or off when the page refreshes along with some other minor error handling.
Your first request for when an incorrect symbol is entered into the search bar and an error occurs is the TD-81 task that we asked Jacob and Ben to address. I didn't work on this to prevent conflicts. Murgolo said we can fix bugs after the code freeze, so if they don't get it done by tomorrow, I'll fix the bug.

I have tested the website and looked over the code, everything looks good, there is a conflict though but it looks like it will be an easy fix.


Preparing for 5.0.0 release

Signed-off-by: Nate Koenig<EMAIL_ADDRESS>

I'm waiting for the other CI jobs to finish, but it has been quite a while. I think I'll merge this sometime soon.

@nkoenig , I see the release hasn't been triggered, so I'll do that now

Thanks.


iqua softener

Hey, anyone knows if this project could be anyhow used for IQUA based softener?
My softener works with HA using this integration -> https://github.com/arturzx/homeassistant-iqua-softener
..and by looking at the source code I found reference to https://apioem.ecowater.com/v1
https://github.com/arturzx/iqua-softener/blob/d9e315e416d8c5a424a92b6d193f46b044201428/iqua_softener/iqua.py#L16
This makes me thing Viessmann aqua smart relays on iqua for it's smartness and that later relays on Eco Water somehow.
Wondering if those are just 2 completely different APIs from the same vendor or so.

Viessmann appears to be using the Eco-Water infrastructure. I have a Viessman Aquahome 20 Smart device, so I would be able to verify this.

@bbilas I've got this one too, I assume it uses some other Eco-Water api (https://apioem.ecowater.com/v1) than the one used here. This one here uses Ayla IoT. So I believe the first one is home made one by Eco Water while the other one is sort of vendored API from Ayla which seems to be generic API provider.


Invalid Reference Format Error

After installing pdks with make pdk-with-volare command and openlane with make openlane command, I run
make mount inside the openlane directory which stops at this error. Maybe the command inside the makefile has format error since docker cannot recognize it.
cd /merledu1/tools/openlane-mpw6/openlane && \ 	docker run --rm -v /merledu1/tools/openlane-mpw6/openlane:/openlane -v /merledu1/tools/openlane-mpw6/openlane/designs:/openlane/install -v /merledu1/tools/openlane-mpw6/pdks:/merledu1/tools/openlane-mpw6/pdks -e PDK_ROOT=/merledu1/tools/openlane-mpw6/pdks -e STD_CELL_LIBRARY=sky130_fd_sc_hd --user 1000:1000 -e DISPLAY=:3 -v /tmp/.X11-unix:/tmp/.X11-unix -v /home/merledu1/.Xauthority:/.Xauthority --network host -ti efabless/openlane:-dev docker: invalid reference format. See 'docker run --help'. make: *** [Makefile:101: mount] Error 125
I am on mpw-6c branch

@usmnzain Is there a specific reason why you need to run make mount? This feature is not yet supported, but there's a work around, you need to add the directory which has your design in the openlane make file.


Determine Open data for CWL workflows

Describe the bug
Open data calculations are not correct; see #5339. In PR #5347, we've improved the check for open data for WDL workflows. This ticket is to extend the logic for CWL workflows.
Expected behavior
Open data calculations for CWL should correctly figure out open data test parameter files.
Notes
This looks more complicated than the WDL work; we already had logic in WDLHandler that determined file inputs. For CWL, it looks like that logic is only in the CLI.  Ideally move the CLI code to the common module in the web service to avoid duplication.
┆Issue is synchronized with this Jira Story
┆Fix Versions: Dockstore 1.14
┆Issue Number: DOCK-2331
┆Sprint: 107 - Eridanos
┆Issue Type: Story

Currently we get the file inputs for file provisioning in the CLI using CWL Avro, which we don’t use in the web service. We've also been trying to move away from CWL Avro, although CWL Java  turned out not to be the answer (at least not yet). So we have these options:

Move or copy, preferably move, the code from the CLI to dockstore.common.
Build on top of the current CWLHandler, which "just" uses a Map, but on the other hand handles CWL "imports" better.

FYI @svonworl as well.
Just to understand where we are at, if the parameter file has links to non-open data, then we deem it closed and vice versa.
The exception is if the workflow doesn't have any input files in the first place, then it is open. So we need to look at the descriptors for this last case?

As an example, a workflow has 3 input parameters, file1, file2, and cpu. We need to parse the descriptor to know that file1 and file2 are of type file.
Then we look in the test parameter files for input parameters file1 and file2, and we see if they exist and are open data. We don't even bother to look for file the parameter cpu, because it's not a file input.
The previous approach was to not look at the descriptors at all, and only look at the values in the test parameter file. If the value looked like an URL, we would see if it was open or not. That would cause false positives for something like "file1": "path/localfile", because path/localfile doesn't look like an url, so we wouldn't check it, and the data would be considered open.

D'oh! I didn't realize until now that the test parameter file in CWL actually has the type! I think I can just go off the test param file; I don't need to read the descriptor at all. We can just punt, for now, on incomplete test param files (workflow defines 2 file parameters, test parameter file only has one of them).
CWL test param file example:
example_flag: true
example_string: hello
example_int: 42
example_file:
  class: File
  path: whale.txt

whereas a WDL test param file is:
{
  "wf.t1.s": "some_string",
  "wf.t2.s": "some_string",
  "wf.int_val": 3,
  "wf.my_ints": [5,6,7,8],
  "wf.ref_file": "/path/to/file.txt"
}


Flags are not visible in Safari browser

The TwemojiMozilla font does not work in the Safari browser. Check out your demo.

Hey @pedrogarciyalopez I was verify your case in my mac (usying my snack example)...and here worked (prints on end).
My Mac: MacOS13 (Ventura) with Safari v16.1.
🚀 AstrOOnauta 🚀

@pedrogarciyalopez this is so insane haha...thanks for your investigation! We found a guilty...
I will search about this..if there are any solutions.
If you have any ideia how fix it...please, share with us =)
🚀 AstrOOnauta 🚀

@AstrOOnauta, i think the quick solution is to use TwemojiMozilla for Windows only. Only there is it really necessary. Other platforms support emoji (flags) natively. Check out this snack. Downside is inconsistent look across platforms.

@pedrogarciyalopez yes, this is a nice idea...but with RN I cannot detect if it is web windows or web macOS. Only web, usying the Platform.OS native method...

@AstrOOnauta you can do something like this
const isWindowsWeb = Platform.OS === 'web' && navigator.userAgent.includes('Win');

@pedrogarciyalopez Nice! Thanks for this insight...I will do this and return with any news =)

@AstrOOnauta Great, thanks a lot!


[2.0.8] Custom Project Owner Role with Disabled OOTB Owner wont let me edit a project "There must be at least one member with the Owner role."

We have customized the owner role by creating our own that just had the Create Namespace permission removed. The issue is Rancher UI gives the error that at least one member needs to have the owner role but that role is disabled per our requirements.

@westlywright - Need to fix in edit project as well

The current logic is only checking if it has a owner in edit page. So it will not show the message in new project.
I guess the only thing we can do is just removing the hasOwner check.

Version - 2.1 master 9/20
Verified fixed


Create Team Programming(TP) Logo

Any logo right now will do as long as it is not derogatory.  If you or someone knows how to design logos, have a try at it!

I have created using Canva by editing some details.

sorry for openning and reopening since I am not so familar with GitHub. Very sorry about the inconvenience

see if that is okay or any amendments could be made. Thanks!

No worries, we'll get some processes written up on how we want to do it.  This whole platform is for learning.  That logo looks great, can you create a smaller one of the original as well?  The size of the red box would work great.

That's a good piece of advice. Would that be nicer?

Yes, that would be great.  Both look good and will have their respective uses.  The smaller, for logos and the larger, for web pages

Great Logo, it looks clean and classy


Set govet again

Housekeeping

Enabled go vet check again in hack testing scripts and in appveyor tests.
Regen examples from latest master (brings some minor differences in embedded spec codegen, stems from updated go-openapi/spec dependency)

Contributes #1724
Signed-off-by: Frederic BIDON<EMAIL_ADDRESS>

Added some guard in new scanner test suite: some unstable behavior did cause tests to panic when it hits the problem. This is still brittle, but it does not panick...


linux-16 lost external connection phone device.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
linux-16 lost external connection phone device.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Rebooting phone device to restore connection.


Documentation Clarification: Rebuilding Daily

In the documentation, you say:

I needed to rebuild the mapping once a day and then access it in read-only mode.

From this it makes me wonder whether pogreb is intended to be used that way, or if it was intended to solve the problem of having to do that.

Pogreb was created with a single use case in mind - building entirely new database once a day and accessing in it read-only mode after. That was the requirement of the project it was built for, but you don't have to use it this way.

Updated design document https://github.com/akrylysov/pogreb/blob/master/docs/design.md.


feature request: CLI option to disable hot reloading

It would be handy if plugin hot reloading could be disabled via the CLI e.g. npx platformatic db start --hot-reload=false.
Example use case: Disabling hot reloading when debugging a Platformatic application. If hot reloading is enabled setting breakpoints in the plugin script(s) doesn't work.

Hey guys, I'm learning about code and trying to understand how things work.
Isn't hot reloading per plugin? https://github.com/platformatic/platformatic/blob/f5623ab4c311803393f9b0a529419d832dcd9dea/packages/service/index.js#L111
npx platformatic db start --hot-reload=false
Does this mean this would be a CLI "global" setting for all plugins or I am completely wrong?

Exactly that! Let us know if you have further questions, we know this is a complex project.


Multiple tabs open when clicking a link

I've been using this for a Flowdock app and when clicking on an external link, it opens up multiple (2) tabs pointing to the link's destination, rather than only one. I have not seen this behavior in an app version of YouTube, but I'm not sure if it's widespread or not. This is not critical by any means, but I wanted to see if other people were also running into this problem.

Could you update nativefier and see if this is still a problem? There are a number of under the hood changes that might have fixed this.

I just tried updating to the latest version ( 6.5.6 at time of writing) and the bug is still present. If there's anything you suggest I look at to try and reproduce or look for specific console messages, just let me know! Thanks

Could you give me a series of steps to reproduce this bug?


Radio selects better UX for parameter values?

Radio selects for parameter values would be a single click rather than two to dropdown select then choose value.  Better UX?

The drop down is a little clunky on touch devices IMO

Agreed on gitter to leave select in place, after trialling radios.  Radio code is still in place and could be offered as a user option at a later date.


Create script to generate batch transactions for vesting of BSX crowdloan rewards

[ ] parse https://github.com/galacticcouncil/Basilisk-crowdloan-ui/blob/develop/scripts/data/rewards.json
[ ] generate batch call
[ ] test run

Let's move this to the basilisk-crowdloan-ui


Improve end-to-end encryption

Replace pgp hooks by a native solution like rPGP or sequoia
Verify signatures according to the RFC
Sign emails according to the RFC

This needs to be thought well:

Set up a key manager (add, remove, trust public keys from recipients)

Moved here: https://todo.sr.ht/~soywod/himalaya/15.


build(all): switching to PNPM as the package manager, and PNPM workspaces

DO NOT MERGE. TESTING IN PROGRESS
All Submissions:

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] Does your submission pass tests?

@platypii @hrgdavor please try the changes on this branch. you should clone the repository, and checkout this branch. you should use a new new file system as this will use PNPM to manage the packages.

FYI, lerna bootstrap in no longer necessary. And will throw an error. ;)
I updated the documentation.
The installation steps are now:

pnpm install
pnpm test

I do have one question about PNPM. Is the installation of NPM required?

do you already have a favorite for using instead of lerna (I remember you mentioning moving from lerna)?

Not really.
The next PR will move to Lerna-Lite which has been put together from pieces of Lerna, converted to Typescript, and reworked to support PNPM workspaces. Lots of functionality is now optional.
https://github.com/ghiscoding/lerna-lite
The basic requirements for the automated publishing are:

change determination
independent versioning
automatic change logs

Lerna lite sounds nice and is I am guessing a simpler move. I randomly picked rush and am sticking with it in my project to learn it. I am in favor of lerna-lite for jscad because it is most likely less hassle to move, and it sounds nice.
who knows maybe after some time using both I will also prefer lerna-lite, or be undecided.

@z3dev I had issues runnung tests until I remove global lerna install that I had.
I am not 100% sure it was related, but it is wort mentioning to users about migration
https://github.com/ghiscoding/lerna-lite#migration-for-existing-lerna-users

@z3dev I had issues running tests until I remove global lerna install that I had. I am not 100% sure it was related, but it is worth mentioning to users about migration.

Good point. I added V3-CHANGES.md to document the changes between V2 and V3.

@hrgdavor thanks for the nice feedback.
@platypii any chance that you have some band width?


use new certs for service network and localhost

The load balancer has not yet been switched.
/assign @sanchezl

looks like it mostly worked.  I need to update some e2e tests.

this moves us closer.  Starting here.


[GHSA-3rq8-h3gj-r5c6]  .NET Denial of Service Vulnerability

Updates

Affected products
Description

Comments
I think that the current version of Microsoft.Owin / AspNetKatana is 4.2.2. This security advisory states that the issue is present in 4.21 and fixed in 4.22 when it probably is present in 4.2.1 and fixed in 4.2.2.
Can a link to the actual commit and issue/bug fixing the vulnerability be added to the vulnerability report so that it can be researched more easily?

Hi there @rbhanda! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory


Allow vertical downscaling so that the full map fits the available height

If find this a very useful feature in similar plugins for Visual Studio. It enables one to easily see what part of a file you're currently looking at.

This a super useful piece of functionality.  If you have ever used a map scroll bar in "full map mode", you will know that after a short time you can navigate the code file much faster because you intuitively understand the SHAPE of the code in your file and so moving to the section you intend to is instant.

This would be really nice to have, possibly as an option.
Having the minimap scroll kind of defeats the purpose of giving an idea of current position within the file.


Soften linter

Remove some aggressive linters: godoc, goconst, misspel, nestif, and prealloc. Also remove stylechecksince it is subsumed byrevive`, and the later is more configurable.
Allow tests and build stages to continue even if linter fails.
Signed-off-by: Hamza El-Saawy<EMAIL_ADDRESS>

THANK YOU


chore: remove stderr path

Inherit stderr from the host, so that workers have the chance to log information through their stderr, that will end up in the host side (let it be the WWS binary itself running standalone, or through runwasi, for example).

Will merge on 🟢


tools: Add optional prefix to Initcode

Add an optional executable bytecode prefix to the Initcode class, which will be prepended to the initcode and executed on contract creation.

This is wonderful; works like a charm.


Frontend use backend

Description
Current cookbook haproxy.cfg template outputs the use_backend rules before the http-request rules. These rules per haproxy config validation, should occur after the use_backend rules.
$ sudo haproxy -c -V -f /etc/haproxy/haproxy.cfg
[WARNING] ... : parsing [/etc/haproxy/haproxy.cfg:50] : a 'http-request' rule placed after a 'use_backend' rule will still be processed before.

This PR just moves the use_backend rules after the extra_options in the template erb
Issues Resolved
fixes: #313
Contribution Check List
Seeking help on the tests. I am probably being dense, but haven't been able to run tests locally. Attempted following the linked testing documentation. Could anyone share like a quick list of what it takes to run the tests? The chef exec bundle install complains that I don't have a Gemfile and the delivery local unit complains that I don't have a local file.

[ ] All tests pass.
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable

@damacus What do you think about something like this PR that only sets the http-request and http-response rules above the use_backend which is what the haproxy config is actually only complaining about?
First attempt PR: https://github.com/sous-chefs/haproxy/pull/314

Should http-request always basically come last?  If so this is sweet.

http-request rules should always come before use_backend. Did I get my if unless logic right? I thought it was sweet, much better than attempt #1.

@JohnRoesler sorry for the delay, is it possible to test for this?

@rshade no worries! Good question, I was struggling to get tests to run, so I admit I didn't attempt adding one. I tried following the linked chef testing standard. I'd be happy to add a test, but may need some assistance with where to start.

you could probably do an inspect test: https://www.inspec.io/docs/reference/resources/command/, that checks for that warning. I think that might be good enough to prove it works.

@rshade we probably actually want a chefspec test if we can help it (or move the other generic test to spec tests to speed up the run).

@JohnRoesler if you drop in on #sous-chef on the chef community slack we can talk you through it. :)

@damacus @rshade mind taking a look this PR again?

Usually we'd bump as a seperate process. But this is fine. I don't think that unit test is actually testing the ordering in the file 🤔

Awesome!


docs: fix simple typo, retreives -> retrieves

There is a small typo in osslsigncode/osslsigncode.c.
Should read retrieves rather than retreives.
Semi-automated pull request generated by
https://github.com/timgates42/meticulous/blob/master/docs/NOTE.md

If you rename your repo pedantic, I might consider this.


[BUG] RapidsShuffleHeartbeatManager needs to remove executors that are stale

We only add executors in RapidsShuffleHeartbeatManager but we never remove them.
We need to keep track of the last time we saw a heartbeat, and as we get more registrations check if any executors are stale, and not include in executor update messages. We don't need to tell already registered executors about the loss of a peer, because UCX should be able to figure that out on its own.
We also need to update the logic on how the "new executors" for a peer is decided. Currently it's based on registration order, and it will need to be time based I believe.

This PR https://github.com/NVIDIA/spark-rapids/pull/2587 is related and improves things slightly for 21.06.


auto-deploy latest helm updates to GCS repo

AUTO DEPLOY latest Helm updates.
Test: Buildkite CI
Checklist:

[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them:

AUTO DEPLOY complete: https://console.cloud.google.com/storage/browser/coda-charts;tab=objects?forceOnBucketsSortingFiltering=false&project=o1labs-192920&prefix=&forceOnObjectsSortingFiltering=false


Improve touch screen button description

Improve the touch screen button node description to clarify that it is intended for gameplay use only, and not UI. Closes https://github.com/godotengine/godot-docs/issues/1395

I've added your suggestions. However I changed the example of using it to touching a unit to move them, since jump controls would be UI.

Thanks for your changes!

Thanks!

Cherry-picked for 3.2.3.


Fix remote_repository_url of color_deconvolution tool

Fix remote_repository_url of color_deconvolution tool.
I wasn't sure whether it's worth incrementing the version for such a small change. On the other hand, it's nice to have the remote_repository_url  set correctly.

FOR THE CONTRIBUTOR — Please fill out if applicable
Please make sure you have read the CONTRIBUTING.md document (last updated: 2024/04/23).

[x] License permits unrestricted use (educational + commercial).

If this PR adds or updates a tool or tool collection:

[ ] This PR adds a new tool or tool collection.
[x] This PR updates an existing tool or tool collection.
[x] Tools added/updated by this PR comply with the Naming and Annotation Conventions for Tools in the Image Community in Galaxy (or explain why they do not).

I would not bump. Its a cosmetic change that triggers all kind of updates on Galaxy deployments.

I understand. But it's fine to merge with the old version, so this will not be forgotten in the next substantial update?

Yes, I would merge and stop the CI job. Or merge with [ci skip].

Lets add next time a profile version as well.


com.groupon.sparklint#sparklint;1.0.2: not found (UNRESOLVED DEPENDENCIES)

I am trying to run sparklint in Live mode by simply adding the --conf and --packages parameters to my submit command (as given in the README).
I am getting an unresolved dependency message.
com.groupon.sparklint#sparklint;1.0.2: not found
Is there anything special I need to do to help Spark find this package?
Additional info:
I am running this on a yarn cluster

Hi @weineran I've just made a lot of build system changes these days. If you are using spark1.6.1 ~ 2.0.2, try the new description in readme "com.groupon.sparklint:sparklint-spark161_2.10:1.0.4". You can substitute spark161 with the actual spark version you are using by removing the dots. If your spark is 2.11, replace 2.10 with 2.11 correspondingly.
Please let me know if you still have problem loading the jar.
P.S. There is an off-by-one bug in live listener mode I'm yet to fix #31, hopefully you don't enter that today :)

Should have been resolved now, since maven central has finished synchronizing


[BUG] Tool Agent doesnt have usedTools property in the response.

Describe the bug
OpenAI Tool Agent is depreciating and there is a new Tool Agent in exchange.
OpenAI Tool Agent has usedTools property in the response but Tool Agent does not have it.
To Reproduce
Steps to reproduce the behavior:

Change OpenAI Tool Agent with Tool Agent component
Run the flow and await for the rest API response.
Check the result

There is no usedTools property.

Expected behavior
Tool Agent also must have usedTools property in its response.
Flow

Pretty sure it does:

Thank you for your quick response @HenryHengZJ i will try another function. It seems like there is a chance that my function may have never been called.

I have been using tool calls to get the proper JSON format in the response without returning anything from the function. In the past, this was known as a tricky solution to get formatted JSON in the stream. Right now it is not working as I have to return a result from the function I called and only then it is working as intended otherwise it just ignores the function call.
I should have seen this difference before, I thank you for your precious help @HenryHengZJ
Closing as it is clear to me now.


[WIP] Mnlbox big auth refactor

Can I do this? Not too much experience with this.. Learning tho!!

That should be it. Going to bed. Need to take a new look at this with fresh eyes.

@elitan Why we need separate PR when we have #24 :thinking:  :smiley:

I did not know how to push to that repo/branch :/

But I think it works really good now! 👍


Certificate resource status can't be checked with current pass/fail conditions.

Bug report
Certificate resource status can't be checked with current pass/fail conditions.
Expected behavior: [What you expected to happen]
Adding annotations to the Certificate CRD would allow us to monitor pass/fail conditions during a deploy.
Actual behavior: [What actually happened]
We were attempting to add annotations to the CRDs responsible for Certificate resources so  kubernetes-deploy could test for whether those resources were successfully deployed.
Currently all resources deployed with CRD's are expected to provide an ObservedGeneration status message.
the problem is the cert-manager controller doesn't add those statuses so we are unable to use the pass/fail logic in kubernetes-deploy to check those resources.
We opened an issue on cert-manager to ask about getting ObservedGeneration status added
to the CRD. Please see there for the discussion.
The short version is do you expect every CRD provider to provide an ObservedGeneration status message so it's pass/fail conditions can be monitored during a deploy.
https://github.com/jetstack/cert-manager/issues/1551
Version(s) affected: [run kubernetes-deploy --version]
v0.25.0
Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

The short version is do you expect every CRD provider to provide an ObservedGeneration status message so it's pass/fail conditions can be monitored during a deploy.

Yes, we do.
These checks exist to prevent a race conditions that we slammed into repeatedly where we examined the resource and declared success/failure based on old state. I'm sure we could find a way to relax the constraint, but it would mean the results would occasionally be wrong and we don't think that's the right trade off.

The cert-manager project has declined to implement observedGeneration. Is there anything that we could do in Kubernetes-deploy to help support controllers like this?

We still think its' reasonable for cert manager to update observedGeneration and since we can't give a reliable result without it we don't see any other path forward. Feel free to re-open if you suggestions we might have missed.


cannot add role to the hosts when they exist

I use capistrano (3.2.1) and have the following definition:
role :app, %w{a b}, user: "test", password: "xxx"
role :db,    %{a}, user: "test", password: "xx"

but I cannot run some tasks on role :db which is because roles(:db) is empty.
I read the source code and find the root cause is the following code:
def server(host)
    servers.find { |server| server.matches? Server[host] } || Server[host]
end

def matches?(other)
    user == other.user && hostname == other.hostname && port == other.port
end

Why we need to add the matches?(other) method? I think we should remove this logic.

If I create a new Capistrano project, and put the role definitions from the original ticket in production.rb and the following in deploy.rb:
task :test_roles do
  on roles(:db), in: :parallel do |host|
    puts "DB: #{host.hostname}"
  end

  on roles(:app), in: :parallel do |host|
    puts "APP: #{host.hostname}"
  end
end

And then run cap production test_roles, I get:
DB: a
APP: b
APP: a

I think this is the correct behavior. I'm closing this ticket. Feel free to reopen (or open a new one) if anyone can reproduce the original issue in latest Capistrano.


spec for refresh_parser

testing  a private method is always nasty.
But in this case we just want to make sure, that the expected filter comes in

@durandom thanks a lot !


Error: Expected subtree parent to be a mounted class component.

I'm removing component which contains react-tiny-popover from DOM-tree. Nothing breaks, but this error shows in console:
Error: Expected subtree parent to be a mounted class component. This error is likely caused by a bug in React. Please file an issue.
    at invariant (react-dom.development.js?f8c1:55)
    at findCurrentUnmaskedContext (react-dom.development.js?f8c1:9545)
    at getContextForSubtree (react-dom.development.js?f8c1:19022)
    at updateContainerAtExpirationTime (react-dom.development.js?f8c1:19076)
    at updateContainer (react-dom.development.js?f8c1:19140)
    at ReactRoot.legacy_renderSubtreeIntoContainer (react-dom.development.js?f8c1:19428)
    at legacyRenderSubtreeIntoContainer (react-dom.development.js?f8c1:19555)
    at Object.unstable_renderSubtreeIntoContainer (react-dom.development.js?f8c1:19603)
    at ProxyComponent.Popover.renderWithPosition (index.js?d533:272)
    at ProxyComponent.eval (react-hot-loader.development.js?9cb3:587)

This error causes by Popover component, I checked, when I do the same actions without this component, everything is ok.

There is already fix for this, see https://github.com/alexkatz/react-tiny-popover/pull/24. It just need a new release with this fix.

Thank you, waiting for release too.


METAMODEL-109: CassandraDataContext throws IllegalArgumentException with

proper message instead of NPE when keystore does not exist.

Looks solid to me. +1


Playwright setup and experiences

Thanks for adding Playwright support - several projects I have been involved with lately has considered migrating to Playwright due to limitations with Selenium. This provides a possibility to evaluate playwright without a huge rewrite and even to migrate in steps if needed!
Just started testing this out and have managed to get it running. A couple of things I've noted:
Setup, incorrect docs?
The readme says
sessionConfiguration.Driver = typeof (Playwright);
but I had to use
sessionConfiguration.Driver = typeof(Coypu.Drivers.Playwright.PlaywrightDriver);
Basic authentication
With Selenium webdrivers it was possible to pass the credentials inside the url (like http://user:pass@site.org). It does not seem like Playwright supports this url format. There are proper ways to set HttpCredentials in Playwright, but there is not straightforward way to pass them into Coypu since those parts are private. My current workaround is to perform this before the first Visit call to the site:
var context = (Microsoft.Playwright.IBrowserContext) Browser.Driver.Native;
var authenticationString = $"{username}:{password}";
var base64EncodedAuthenticationString = Convert.ToBase64String(System.Text.ASCIIEncoding.ASCII.GetBytes(authenticationString));
context.SetExtraHTTPHeadersAsync(new Dictionary<string, string> { { "Authorization", "Basic " + base64EncodedAuthenticationString } });

This works, but is kind of clunky. If you have plans to extent the API to allow passing HttpCredentials (and possibly more options) that would be great, otherwise I would be happy to put together a PR for it.
ExecuteScript return type
With Selenium webdriver, the return type for calls to ExecuteScript was castable to the expected type (booleans could be cast to bool, numbers to long). Playwright seems to return strings instead. Although this isn't a breaking change for Selenium users, it would be more convenient if the interfaces match. At least it should be documented.

Great feedback thanks! Agree with all these suggestions. Even partially implemented PR would be very welcome, otherwise I'll try look at them soon as I can.

I don't think I got a notification for this for some reason or I missed it. Unfortunately I haven't even had a chance to test out the last release yet with my test framework using it. I agree with Adrian, this is excellent feedback and these sound like legit issues and improvement for a PR.

Setup, incorrect docs?

Readme corrected and merged.

On the basic auth question, I'm inclined just to parse these from the url in the Playwright driver for consistency rather than introduce a whole new API bearing in mind where we are. @bjowes would that satisfy?

That would be fine @adiel

These points are addressed in #235 and #233
Could I get a quick review from one of you two then we can ship these.

Reviewed

@sbowler would you mind releasing v 4.2.0? It should be ready to go in master now. Thx

I should be able to do that. Not quite sure when. Unfortunately I've been sick and a bit behind on things now.


edit "All-Systems-Operational" to: "All-systems-are-up-and-running"

⚠⚠⚠ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma
Tick the checkbox if you understand [x]:

[x] I have read and understand the pull request rules.

I only changed "All-Systems-Operational" to "All-systems-are-up-and-running"
For our usecase this translation makes a lot more sense thats why we wanted to edit this.
User interface (UI)
Checklist

[x] My code follows the style guidelines of this project
[ ] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[x] I have commented my code, particularly in hard-to-understand areas (including JSDoc for methods)
[x] My changes generates no new warnings
[ ] My code needed automated testing. I have added them (this is optional task)

Please do not use any external image service. Instead, just paste in or drag and drop the image here, and it will be uploaded automatically.

If this is an English only translation change, this should be done in weblate. But I think this is debatable so we can discuss this first here.
If your intent is to invalidate translations for all other languages, I don't think you have provided enough justification for doing so.


Multiple tokens

Hi, thanks for this great library! Everything works great but I have 2 questions:

Is there any way to accommodate for aspects which have more than one word, e.g. task = nlp(sentence, aspects=['social media']) raises an error:

    149             raise ValueError
    150         if len(example.aspect_tokens) > 1:
--> 151             raise ValueError
    152
    153     @staticmethod

ValueError:

Are there any other properties of task.batch, such as the overall sentiment score or the words that constitute that sentiment score?

Thanks so much!

Hey @samuellye98,

we will release the version 2.0 (in the next two weeks) wherein you can define an aspect by few words ☺
this property task.batch is helpful when you preparing the input batch (encode the tokenized examples).


Question about css structure and scope issues

Just wondering if there's a reason you decided to apply css to data tags in several places instead of classes? E.g. data-player has css mapped to it.
Secondly at the moment the css isn't scoped meaning if I happened to have another element on my page with data-player for example it inherits clappr's styles which is undesired behaviour. A simple fix for this would be to add a clappr class to the outer div and then prepend .clappr to the start of all the styles.
https://github.com/webpack/css-loader (with "CSS Modules" enabled) might be a nice solution though to help make each component in the player have its own css scope.

@tjenkinson we originally considered the data tags to add some scope to the markup elements and passing some information around. Since every element had data attributes, we used those in the CSS to make the selectors more specific. Some elements haven't been revisited to add classes to the data attribute selectors, but we dropped the idea of having data attributes in a later moment.
Since the adoption of webpack and css-loader, we are still reworking the scope of the css classes, but we didn't have the time yet to do it so. One of our Clappr-based players already use scoping instead of data attributes, but we still need to change this on Clappr itself.

Ok thanks :)


fix: Update infra-host entityExpirationTime from daily to 8 days

Relevant information
Updating entityExpirationTime for INFRA-HOST entities to 8 days.
Checklist

[x] I've read the guidelines and understand the acceptance criteria.
[x] The value of the attribute marked as identifier will be unique and valid.
[x] I've confirmed that my entity type wasn't already defined. If it is I'm providing an explanation above.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Marking PR as DRAFT until there is approval that EIGHT_DAYS is acceptable from a COGS perspective.


ucp 7143 wait-for-lock-timeout and wake-up-delay-duration readable

UCP #7143
Signed-off-by: ssebo<EMAIL_ADDRESS>What have you changed?
change wake-up-delay-duration and wait-for-lock-timeout from u64 to ReadableDuration
What is the type of the changes?

Engineering (engineering change which doesn't change any feature or fix any issue)

How is the PR tested?

CI
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
no
Does this PR affect tidb-ansible?
https://github.com/pingcap/tidb-ansible/pull/1216
Refer to a related PR or issue link (optional)
https://github.com/tikv/tikv/issues/7143
Benchmark result if necessary (optional)
Any examples? (optional)

@youjiali1995  hi, mentor, I revert my code, the test still not pass, seems that not my change make the server::service::diagnostics::log::tests::test_search case fail ?

@youjiali1995 hi, mentor, I revert my code, the test still not pass, seems that not my change made the server::service::diagnostics::log::tests::test_search case fail ?

Yes. The test is broken...

Is the change backward compatible?

Is the change backward compatible?

No. I want to implement it by myself...😂

Is the change backward compatible?

No. I want to implement it by myself...

@SSebo If you want to do it, you can write a custom Deserialize which parses pure numbers as milliseconds for the configuration of lock manager.

@youjiali1995 @BusyJay I'll add backward compatibility tonight.

@youjiali1995 @BusyJay

Can I change ReadableDuration deserialize implementation? It's common when change u64 to ReadableDuration, and want to add backward compatibility. We can add some warning log to warn user to change the config.
I found ReadableSize accept number without unit, ReadbleDuration does not.

Sorry for the late reply. I prefer changing  the deserializer of configuration to ReadbleDuration. You can try #[serde(deserialize_with = "path")].
What do you think? @BusyJay

@youjiali1995 Ok, I'll revert the change of ReadableDuration, and try  #[serde(deserialize_with = "path")

@youjiali1995 @BusyJay add backward compatibility, PTAL

@youjiali1995 Your suggestion is far more cleaner than mine!  PTAL

@BusyJay PTAL

/test

@NingLin-P dynamic-config-test failed. PTAL, thanks!

/run-all-tests

/run-dynamic-config-test

Block by https://github.com/pingcap/tidb-test/pull/1017.

@youjiali1995 all checks have passwd !

@SSebo Merged. Thanks!


Cleanup finalized vmi once domain is down.

What this PR does / why we need it:
If a vmi gracefully shuts down, we were not cleaning up the local data related to the vmi until the vmi object itself is deleted. This causes the watchdog to timeout and create a bunch of noise in virt-handler's logs.
So, a vmi's guest can gracefully terminate and leave the vmi object around in a finalized state. When this happens we need to clean up the local data related to that vmi or else we'll get a bunch of error messages in the logs.
Release note:

NONE

/lgtm
/approve

/retest


Optional chaining not working

Hey all! 👋
I'm getting errors because of optional chaining:
../common/store/users/in-flight-activity-slice.js 12:37
Module parse failed: Unexpected token (12:37)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|   reducers: {
|     addInFlightActivity: (state, action) => {
>       const addedId = action.payload?.id;
|

Here's my .swcrc file.
{
  "jsc": {
    "parser": {
      "syntax": "ecmascript",
      "jsx": true,
      "dynamicImport": true,
      "exportDefaultFrom": true,
      "decorators": true,
      "decoratorsBeforeExport": true
    },
    "transform": {
      "legacyDecorator": true
    },
    "target": "es5",
    "externalHelpers": true
  }
}

It seems swc should have no problem with optional chaining but haven't had any success here. 😄

Can you retry with the latest version published 0.1.3?


Message Doesn't sending keyboard without errors

Hello, I'm trying to send message with keyboard. Message sending, but keyboard not, and there are no errors.
My code: https://pastebin.com/uvL7CdMk
Logs: https://pastebin.com/Aantcq2n

Only bots can send messages with keyboards.

But I'm authorize like a bot

You are sending this as keyboard:
  reply_markup = replyMarkupShowKeyboard {
    rows = vector[2] {
      vector[1] {
        keyboardButton {
          text = ""
          type = null
        }
      }
      vector[1] {
        keyboardButton {
          text = ""
          type = null
        }
      }
    }

Thank's, i figured it out. Here is example of working keyboard https://pastebin.com/aQFE76ur


[Feature] Keep scroll position while adding comments to scrollable containers / widgets

Initial investigation investigation into the issue points to re-rendering as a prominent cause. Adding the profiling outputs for following case -
User has scrolled into a container and when switches to comment mode.

The first image clearly shows the table widget being re-rendered.
Flame graph in second image shows that all the child components under Table widget components also re-rendered. Which resets the scrollbar position.

Actually this issue has two parts, apart from preventing re-renders we would also like to add the comment pin at the correct offset based on the scroll. That could be a part of another PR


Create a "CLI Quick Start" guide

Create "Getting Started with Zowe CLI" content for advanced users who are already familiar with modern CLI tools and scripting (These users likely need minimal guidance). Content should link to core doc for more info on each topic.
New content will have the the following at minimum:

Install -  Lightweight download and install instructions.
Usage - Describe the various places the CLI is used. Command Terminals, IDEs, Scripts, automation servers.
Issuing your first command - Show how to issue a command to list datasets from a native terminal like cmd prompt or a bash terminal on Linux/mac without using profiles.
Using profiles to store command options - Briefly explain the concept of profiles and show creation of a z/OSMF profile
Scripting - Write a shell script that submits JCL that exists in a PDS member on the mainframe and waits for the job to complete before returning the results to screen.

I suggest creating a new section under Getting Started named "Get started with Zowe CLI" or similar topic in the Using section. We can add more advanced topics iteratively (write a gulp task to perform builds, automate mainframe system tests with console commands/mochajs, create a Jenkins stage for mainframe).

@BrandonJenkins14 - in what branch are you working?

@JamesBauman When we pull it into a sprint (probably next sprint) I will create a new branch off master for the work. Right now, all I have is the plan.


[Feat]: Include Staff Description In Staff Search

Description
I am unable to find staff that match a description

Solution
I want to be able to find staff by their description

This will help to see  all staff with similar description

Anything else?
No response
Code of Conduct

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

I want to work on this, can I proceed?

Sure thing!! @wilfredcloud!

Great. PR created

@tobySolutions Is the issue still available , if yes, iw would like to work on it

Fixed by #766


feat: Improve the error message when you try to query an empty table

Summary
If you create a table, but don't load any data into and then try to query it, you get this error:
FTL error="rpc error: code = NotFound desc = data_token not found"

...which isn't very actionable. It would be much better if it said something like "You have tried to query a table that doesn't contain any data - please load some data before querying" or something.

This error only occurs on a fresh Kaskada system, where zero dataTokens have been created.


details not in listVirtualMachines response

ISSUE TYPE

Bug Report
COMPONENT NAME
API

CLOUDSTACK VERSION
4.11
4.9

CONFIGURATION

N/A
OS / ENVIRONMENT

N/A
SUMMARY

listVirtualMachines does not return details as doc says.
STEPS TO REPRODUCE

cs listVirtualMachines id=d949e443-9bc8-4399-9e09-842bbb395cae --region simulator

EXPECTED RESULTS

the resource details as below should be included in listVirtualMachines response, as the documentation of the API says or the API doc should be updated.
cs listResourceDetails resourcetype=uservm resourceid=d949e443-9bc8-4399-9e09-842bbb395cae --region=simulator
{
  "count": 3,
  "resourcedetail": [
    {
      "fordisplay": true,
      "key": "foo",
      "resourceid": "5",
      "resourcetype": "UserVm",
      "value": "bar"
    },
    {
      "fordisplay": true,
      "key": "Message.ReservedCapacityFreed.Flag",
      "resourceid": "5",
      "resourcetype": "UserVm",
      "value": "false"
    },
    {
      "fordisplay": true,
      "key": "SSH.PublicKey",
      "resourceid": "5",
      "resourcetype": "UserVm",
      "value": "ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAAAgQCalaJdHI7bK4qsCMSqJ1iq7K4YuHyjlkahPuSsbMrZInN2Mo1ehjieILocEITo8ck+rFkC4NHc6mvUDF3gdUZmWKpWYfc5eKsymOHgk05Nv1e+1TjvKGWaZprisUFXD+52PkQ6DcAZUDYw1yufxGfufYuK3WWzDHriSBIiaDEcgw== \n"
    }
  ]
}

ACTUAL RESULTS

{
  "count": 1,
  "virtualmachine": [
    {
      "account": "admin",
      "affinitygroup": [
        {
          "account": "admin",
          "id": "2d41d761-6d4e-400a-9f26-cc0d5ef6f884",
          "name": "ansible-test-titan-10437836-ag"
        }
      ],
      "cpunumber": 1,
      "cpuspeed": 500,
      "cpuused": "10%",
      "created": "2018-03-31T18:07:13+0000",
      "diskioread": 0,
      "diskiowrite": 0,
      "diskkbsread": 0,
      "diskkbswrite": 0,
      "displayname": "ansible-test-titan-10437836-display-1",
      "displayvm": true,
      "domain": "ROOT",
      "domainid": "1f1d6aa1-4b9a-11e7-8fea-0242ac110003",
      "guestosid": "1ef0010d-4b9a-11e7-8fea-0242ac110003",
      "haenable": false,
      "hostid": "41adc5d5-3b6a-4035-9913-7b14fa099c1f",
      "hostname": "SimulatedAgent.db961dea-4e46-48c6-983e-566712333810",
      "hypervisor": "Simulator",
      "id": "d949e443-9bc8-4399-9e09-842bbb395cae",
      "instancename": "i-2-5-VM",
      "isdynamicallyscalable": false,
      "keypair": "ansible-test-titan-10437836-sshkey",
      "memory": 512,
      "memoryintfreekbs": 0,
      "memorykbs": 0,
      "memorytargetkbs": 0,
      "name": "ansible-test-titan-10437836-vm-1",
      "networkkbsread": 148668416,
      "networkkbswrite": 74334208,
      "nic": [
        {
          "broadcasturi": "vlan://untagged",
          "gateway": "<IP_ADDRESS>",
          "id": "8bee0927-6506-4a18-93bc-ccce070714a2",
          "ipaddress": "<IP_ADDRESS>",
          "isdefault": true,
          "macaddress": "06:ce:d4:00:01:f2",
          "netmask": "<IP_ADDRESS>",
          "networkid": "6f4007a0-ddba-4cdf-ac34-c9c0e5f049b8",
          "networkname": "guestNetworkForBasicZone",
          "secondaryip": [],
          "traffictype": "Guest",
          "type": "Shared"
        }
      ],
      "ostypeid": 142,
      "passwordenabled": false,
      "rootdeviceid": 0,
      "rootdevicetype": "ROOT",
      "securitygroup": [
        {
          "account": "admin",
          "egressrule": [],
          "id": "abcdb1a7-4795-4aff-82a9-ca4347b017fd",
          "ingressrule": [],
          "name": "ansible-test-titan-10437836-sg",
          "tags": [],
          "virtualmachineids": []
        }
      ],
      "serviceofferingid": "a9ad69ae-391d-43ba-97a8-5f33b78a955d",
      "serviceofferingname": "Small Instance",
      "state": "Running",
      "tags": [],
      "templatedisplaytext": "CentOS 5.6 (64-bit) no GUI (Simulator)",
      "templateid": "4f4c3063-4b9a-11e7-8fea-0242ac110003",
      "templatename": "CentOS 5.6 (64-bit) no GUI (Simulator)",
      "userid": "1f1daf4d-4b9a-11e7-8fea-0242ac110003",
      "username": "admin",
      "zoneid": "ab54814a-b8b9-4889-ae76-cae19c0c843f",
      "zonename": "Sandbox-simulator-basic"
    }
  ]
}

@resmo did you try that as a normal user or root/domain admin?

I used root

is this to be fixed in documentation? It is tagged for 4.11.1 but no PR is out for it

I am not aware of anyone working on this.

Aren't the details in when root/admin call the api?

the call in the "acutal result" is an root admin

@resmo if I understand correctly, you would like the listVirtualMachines API to include the resource details tag in its response?

yes, I would prefer the API to return the details as well.

@resmo thanks, I will have a look and update the API response with this tag.
/CC @DaanHoogland @rhtyd

sound cool @dhlaluku

there is a flag: details=true

can no longer reproduce. closing.


Can't save model on Colab but can on Kaggle

I have a very simple model and orbax won't let me save it:
https://drive.google.com/file/d/1hM9NN5vY2EO_NvieIFZ_A4uHUG-s5VEE/view?usp=sharing
Same code works fine on Kaggle though:
https://www.kaggle.com/code/windmaple/notebook28117e6799

Never mind. I had to install orbax-checkpoint instead of orbax.


PAYMENT INFORMATION

I wanna know where should i add stripe private and public key for people to then pay me through cards is it inside env file should i just add stripe public and private key there and it will work fine?

No, this application doesn't currently support Stripe


Training on Human3.6M provide Upside-down results

I have trained the model on Human3.6M and UP-3D. However, the model produces resonable results of UP-3D, but have upside-down results for Human3.6M, of which the head keypoint is alway wrong.

I have checked my h36m_train.npz using the following code (h36m_train.npz is contained in class H36MDataset(), similar to FullDataset()).
In the following code, I use batch = dataset[2] to directly get the img and annot batch, which already includes flip, rotation or adding noise.
Notice that I have replaced  self.to_lsp = list(range(14)) with self.to_h36m = list(range(13)) + [18], but this member seems not used in the train code, which may not be the cause(I'm not sure).

class Visualize(object):
    def __init__(self, options):
        self.options = options
        self.device = torch.device('cuda' if torch.cuda.is_available() else 'cpu')
        # Renderer for visualization
        self.smpl = SMPL().to(self.device)
        self.renderer = Renderer(faces=self.smpl.faces.cpu().numpy())
        # LSP indices from full list of keypoints
        self.to_lsp = list(range(14))
        self.to_h36m = list(range(13)) + [18]

    def vis(self, input_batch):
        input_batch = {k: v.to(self.device) if isinstance(v, torch.Tensor) else v for k,v in input_batch.items()}
        rend_imgs = []

        img = input_batch['img_orig'].cpu().numpy().transpose(1,2,0) #(H, W, C)
        gt_keypoints_2d = input_batch['keypoints'].cpu().numpy()
        gt_keypoints_2d_ = gt_keypoints_2d[self.to_h36m]
        gt_pose = torch.unsqueeze(input_batch['pose'], 0)
        gt_betas = torch.unsqueeze(input_batch['betas'], 0)
        gt_vertices = self.smpl(gt_pose, gt_betas)

        vertices = gt_vertices[0].cpu().numpy()

        pred_keypoints_2d_ = gt_keypoints_2d_[:, :2]
        pred_camera = torch.unsqueeze(torch.Tensor([0,0,0]), 0)
        cam = pred_camera[0].cpu().numpy()

        rend_img = visualize_reconstruction(img, self.options.img_res\
            , gt_keypoints_2d_, vertices, pred_keypoints_2d_, cam, self.renderer)
        rend_imgs.append(torch.from_numpy(rend_img))

        rend_imgs = make_grid(rend_imgs, nrow=1)
        plt.imshow(rend_imgs)

        plt.savefig('others/h36m_vis.png')

with open('/hpn/logs/paper_step1/config.json') as f:
    json_args = json.load(f)
    json_args = namedtuple('json_args', json_args.keys())(**json_args)
options = json_args
dataset = H36MDataset(options)

batch = dataset[2] #already includes flip, rotation or adding noise

visulize = Visualize(options)
visulize.vis(batch)

And it seems that gt 2D pose is correct, and data preprocess(like flip and rotation) is also correct.

So, I doubt that problem lies in the training code. Could you help me to locate where the problem may come from? Thank you very much!
Any other detail on ask.

BTW, I haven't figure out why the 3D joints have size of (B, 38, 3) in the model.smpl.py, and what's the difference between J_regressor and J_regressor_extra .
        """
        This method is used to get the joint locations from the SMPL mesh
        Input:
            vertices: size = (B, 6890, 3)
        Output:
            3D joints: size = (B, 38, 3)
        """
        joints = torch.einsum('bik,ji->bjk', [vertices, self.J_regressor])
        joints_extra = torch.einsum('bik,ji->bjk', [vertices, self.J_regressor_extra])
        joints = torch.cat((joints, joints_extra), dim=1)
        joints = joints[:, cfg.JOINTS_IDX]
        return joints

More detail
I did't use
# in config.py
# Indices to get the 14 LSP joints from the ground truth joints
J24_TO_J14 = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 18]

when constructing H36MDataset(). But I find it is used in eval.py. Do you use it in your origin code?

First, you might have to flip the SMPL pose as described here.  This is probably why your are getting flipped shapes. The network tries to both match the output shape with the target shape and align it with the 2D keypoints, that's why you are getting these "spiky" shapes as output.
The size of the joints is 38 because different datasets have different joint definitions, so we take a superset of all those joints and apply the losses only on the relevant joints.
J24_TO_J14 gets the 14 lsp joints from the 24 SMPL joints and is used to evaluate MPJPE on H36M. During training we apply the joint loss on 17 joints, but we evaluate only on 14.

Thank you! It works now!


Syntax Error after cleanup at token '='

Hi,
I've been facing an issue if I use FOR expression with GROUP BY clause. If I comment it, the ABAP Cleaner works fine.
Screenshots are attached below for both the sample code and the issue. Please, share if it can be fixable from my side else check if it is fixable from your side.
Also, thanks for the awesome tool. It's a game changer!

Hi Yash,
thanks a lot for reporting this issue! This seems to be a bug in the cleanup rule "Shorten VALUE statements", which mistakes the group key for a table row. A minimal example to trigger the error would be this:
itab = VALUE #( FOR GROUPS OF <fs> IN itab GROUP BY
                ( plant = <fs>-plant )
                ( plant = <fs>-plant
                  l1    = <fs>-l1 ) ).

Here, the cleanup rule erroneously identifies plant = <fs>-plant as a common assignment and therefore extracts it from both inner parentheses …
itab = VALUE #( FOR GROUPS OF <fs> IN itab GROUP BY
                plant = <fs>-plant
                ( )
                ( l1    = <fs>-l1 ) ).

… which is a syntax error. Will fix this asap!
Kind regards,
Jörg-Michael

Hi Yash,
this bug is now fixed for the next release.
Kind regards,
Jörg-Michael

Hi,
Thank you very much.  Will wait for the next release.

Hi Yash,
thanks again for this bug report – this error should now be fixed with version 1.13.0, which was just released!
Kind regards,
Jörg-Michael

I love learning about syntax possibilities from ABAP Cleaner bug reports 😄


react-flow render all nodes at the first mount

when I set onlyRenderVisibleElements={true},  NodeRenderer will call getNodesInside, and in this line,
it refs to the node.__rf.width,
https://github.com/wbkd/react-flow/blob/41db69e06c22d278657ba810198f371119b4972b/src/utils/graph.ts#L240
but nodes are initialized with width and height = null,  so getNodesInside returns true, and react-flow will render all nodes at the first mount. It takes a lot of time for an application with a lot of complex nodes.
https://github.com/wbkd/react-flow/blob/41db69e06c22d278657ba810198f371119b4972b/src/utils/graph.ts#L156

if use set style width, can we refs to style.width ?  https://github.com/wbkd/react-flow/pull/1590


Numba optimization

Image to ASCII generators speed improved by 250%. To work properly, after server is started, generators should be executed manually for the first compile;
Also, .gitignore fixes.

For some reason, on actual web server (single core, 512 ram) numba's speed is actually slower than without it.


⚠ Bofrost.lu has degraded performance

In 9b6e626, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1351 ms

Resolved: Bofrost.lu performance has improved in 6315bf2 after 14 minutes.


Float distance in mouse scroll.

Expected Behavior
Mouse scroll wheel function should accept float distance values on Windows.
It seems that a simple cast into int on this line https://github.com/pywinauto/pywinauto/blob/bf7f789d01b7c66ccd0c213db0a029da7e588c9e/pywinauto/windows/mouse.py#L321C37-L321C38 would solve the issue. Accepting float values would make sense because the distance is internally multiplied by 120 (one wheel click according to MS). However, I don't know about possible consequences on other systems.
Actual Behavior
Using a float value produces an Exception.
Steps to Reproduce the Problem

Call pywinauto.mouse.scroll(wheel_dist=x) with x a float value

produces
Traceback (most recent call last):
  File "pywinauto\mouse.py", line 260, in scroll
    _perform_click_input(button='wheel', wheel_dist=wheel_dist, coords=coords)
  File "pywinauto\mouse.py", line 177, in _perform_click_input
    win32api.mouse_event(
TypeError: 'float' object cannot be interpreted as an integer

Short Example of Code to Demonstrate the Problem
from pywinauto import mouse
mouse.scroll(wheel_dist=0.5)

Specifications

Pywinauto version: 0.6.8
Python version and bitness: 3.10 x64
Platform and OS: AMD64, Windows

This is interesting feature request. I guess it is possible to implement it for high level method .wheel_mouse_input(wheel_dist=0.5) or .click_input() if the scroll bar range is available at the UI element level. More low level function mouse.scroll(wheel_dist=0.5) has no information about any scroll bars, so the ratio 0.5 is undefined at this level.

Thanks for your answer, indeed I was rather thinking about the low level feature. In my use case, there is no scroll bar (I use the scroll event to simulate a swipe up/down in a mobile like app), and a "wheel click" as defined by MS is indeed too coarse to have a fine control on the movement generated by the event. I tested with the low level win32 api and it accepts (and works) with distances which are not multiples of 120 (WHEEL_DELTA).


LOG_MODE_MINIMAL do not work with USB_CDC for nRF52840

Describe the bug
I was trying to enable a log in console.
To do so, I played around with shell_module example, and I was trying to add another console driver to this example.
When I pick LOG_DEFFERED=y it works perfectly.
When I pick LOG_MINIMAL or LOG_MODE_IMMEDIATE then nothing works.
To Reproduce
Steps to reproduce the behavior:

Take a nRF52840 board
Change LOG_MODE_DEFFERED to LOG_MODE_MINIMAL and add zephyr,console = &cdc_acm_uart0; in usb.overlay file in shell_subsystem sample.
build sample with your changes together with prj_minimal.conf with usb_overlay.proj
See error

Expected behavior
Shell and console is working on usb port.
Impact
Huge for me. I need a solution which gives me fast output.
For others, I guess it will be not a big problem.
Environment (please complete the following information):

OS: Windows
Toolchain NRF SDK 2.2.0,
Commit SHA commit c0689b16ff127d1b71a4cc20310d476e1807e26e (3.2.99)
Board: Particle Xenon

Here is repo with my changes
https://github.com/DuMaM/nrf_submodule_shell_error/commits/master

I don't think going with LOG_MODE_MINIMAL will give you the desired fast output due to the very nature how USB works (host initiates every single transaction). The biggest limiting factor currently is that when CDC ACM is used as console, then every single character output to console tends to get transmitted in its own transfer. That is, on host you get 1 byte per URB which is seriously limiting the throughput.
The console uart driver writes the characters with uart_poll_out:
https://github.com/zephyrproject-rtos/zephyr/blob/85a7d0cc467384be33ce7a6d80f35cefce1d6d1b/drivers/console/uart_console.c#L77-L95
which calls cdc_acm_poll_out:
https://github.com/zephyrproject-rtos/zephyr/blob/85a7d0cc467384be33ce7a6d80f35cefce1d6d1b/subsys/usb/device/class/cdc_acm.c#L988-L1020
USB_WORKQUEUE_PRIORITY Kconfig help mentions:
  By default, USB work queue priority is the lowest cooperative
  priority. This means that any work handler, once started, won't
  be preempted by any other thread until finished.

Therefore, if the console output happens in lower priority (i.e. any preemptive priority) then you'll essentially get one character at a time. You can get more characters at a time (which allows greater throughput, which is essentially what @DuMaM seems to want) by changing USB_WORKQUEUE_PRIORITY to lower priority (e.g. to 1; lowest cooperative priority is higher than any preemptive priority) but that might bring unforeseen issues.
@pfalcon Any idea how to handle it? I think the USB workqueue being cooperative is generally good, however it clearly messes up with the possibility to fill up console buffer.

@DuMaM Could you please check if #53003 gives you enough speedup when used with deferred logging?

Minimal logging is working just fine with CDC ACM console when I disable CDC ACM logging. I think the only way around that is to either remove the CDC ACM logging completely or to fail build if CDC ACM is used as console and CDC ACM log level is not Off (done in #53032).


add fastify logo to landing page header section

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the feature has not already been requested

🚀 Feature Proposal
the fastify logo should be added to the landing page header section in place where we are having text Fastify as is at https://www.fastify.io/

Motivation
No response
Example
No response

@Eomm from first sight i thought this didn't exist, but on inspecting, I realized it was a broken path

@Eomm you may disregard this, its my fork that wasnt upto date


build: add node v18 support

Description

~Remove support for node v12 and v17~ I will split the node v12 support removal into it's own PR (#7833)
Add support for node v18

Additional context

https://nodejs.org/en/about/releases/
https://twitter.com/BethGriggs_/status/1516449244034519041

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Note for myself: need to update https://github.com/vitejs/vite/blob/9c6501d9c363eaa3c1e7708d531fb2a92b633db6/package.json#L40

I'm not 💯 % sure, but maybe the failing node-18-windows job is a jest node-18 globalTeardown bug?

About the failing test on WIndows:

I don't see anything wrong with the specific test case except it's the last one that gets executed.
So I think, even if it's a real issue, it's likely to be a bug with Jest and Node.js 18, we can safely ignore it…
…until someone with a Windows machine can actually help debug it.

Done a little research and found a possible culprit: https://github.com/jprichardson/node-fs-extra/pull/646
Now that we also have rimraf in dependencies, maybe we can replace the fs.removeSync in jestGlobalTeardown.cjs with rimraf and see what happens?
Related commits:

https://github.com/isaacs/rimraf/commit/d53235de863cfb00f691456288fcf3abbced7f6c
https://github.com/jprichardson/node-fs-extra/pull/646/files

rimraf retries 100 times; fs-extra retries for 500ms.
So rimraf is likely to fail on very fast machines and fs-extra may fail on slow machines…

Done a little research and found a possible culprit: jprichardson/node-fs-extra#646
Now that we also have rimraf in dependencies, maybe we can replace the fs.removeSync in jestGlobalTeardown.cjs with rimraf and see what happens?
Related commits:

isaacs/rimraf@d53235d
jprichardson/node-fs-extra#646 (files)

rimraf retries 100 times; fs-extra retries for 500ms. So rimraf is likely to fail on very fast machines and fs-extra may fail on slow machines…

I may do this this evening or weekend, need to be "afk" for a while
In the meantime, I would suggest that we already merge this PR as it fixes already a compatibility log bug 🙂

@Shinigami92 I see that this was marked for 3.0, this is ok to be released in 2.9 though, no?

Yes, I marked it with 3.0 as I also deprecated v12 in one PR
Now I split them into separate :+1:


Api v1 - clients, users v1 REST support

ChangeLog-Entry: This introduces new, consistent handling of keys across
the clients, users, and keys endpoints.  It disables the updating of keys
via the clients and users endpoints - all keys are to be managed via the
keys endpoints after initial creation. This deprecates the previous key
creation and update, though it is still supported for backwards
compatibility.
This PR replaces https://github.com/chef/oc_erchef/pull/139 which was based against the wrong branch.
ping @chef/lob

Generally LGTM. I don't see anything obvious. I looked closer when these were separate PRs. A lot of changes but nothing jumps out as wrong.

Just to clarify this PR is still separated and built on top of the other cumulative PRs.  Almost half of it is new and updated tests, so it's not quite as big as it looks.
PR #141 is the 'one PR to rule them all' that will merge to master once this one is vetted.

I'm fine merging this. Nothing jumps out at me that's worrisome, but I don't think I hit every nook and cranny either.


"KILLED" After Merging Two Big Wordlists

After attempting to merge together two wordlists (143.9MB + 2.9GB), it ended with "KILLED".
Here is the code:
root@kali:~/dymerge# python dymerge.py '/root/Downloads/Wordlists/HashesORG251015.txt' '/root/Downloads/Wordlists/W0RD715T.txt' -s -u -o ~/Desktop/P455W0RD715T.txt
DyMerge 0.2 Nikolaos Kamarinakis (nikolaskama.me)
[+] Starting Dictionary Merge Task
[+] Reading Dictionaries
[+] Merging DictionariesKilled
root@kali:~/dymerge#
So Basically, it broke, i'm guessing from the size of the wordlists.

This occurs because you do not have enough available RAM to generate a successful merging task with dictionaries of such great size. I will probably alter some functions to modify the data stored in the RAM in the near future.

Please read my latest article on DyMerge here. I believe it will help you out. Thanks.


Unable to save updated test

Save test case gives "Please review required fields and save again." after adding/modifying tests.
Even if I change any of the required fields, I get the same message.

Me, too.
https://jsperf.com/membernullchecks/1/edit
https://jsperf.com/member-null-checks/1/edit
:(

Confirm that. Was trying to save like crazy.

Duplicate of #236.

Closing as duplicate of #236.


Limited role for Power Platform integration

The 'Note' section in this article states " a more limited role for this integration will eventually be added to Azure AD ... ".
Can you advise when this new role would be made available, please?

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

ID: ae234693-3a0a-d25c-ecd1-abd01ad7580f
Version Independent ID: ae234693-3a0a-d25c-ecd1-abd01ad7580f
Content: Enable the Microsoft Power Platform integration - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/power-platform/enable-power-platform-integration.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @jaredha
Microsoft Alias: jaredha

@jaredha, please review this feedback. Thanks!

Bug 660438 created to track this issue.

@laneswenka, can you take a look at this?


Add Contribution / Git Workflow Section to README.md

Standardize the contribution process for this project.

All PRs related to issues should have the first line of their body/description include the resolves keyword and what issue it is meant to resolve (e.g. Resolves #2 to resolve issue #2). This auto-links the PR to the issue.
Assign at least 1 other reviewer to your PR and get explicit approval before merging (See issue #7).
Put clear descriptions in PR bodies to help the reviewer understand all context of your changes. Adding testing details if required.
Add an overview of how to do a simple Git workflow for creating a pull request with a new branch for new contributors that may not have used Git before.
Brainstorm any other standards.

Add README info on how to add new commands, what files to create, what modules to declare, and what example event bodies look like.


Typescript 'matches' issue on xstate/react rc.1 and later.

Description
I've got a strange one. Typings for the context of Machines vanish depending on how you do a matches call.
Actual Result

Here I have a pretty typical check to see if I'm in a particular state. However, you can see that the context type is now showing as any, when it's actually defined when I create the machine.
If I wrap that check in a Boolean, the context typings are maintained, and Typescript correctly reports that I've got an error.

Expected Result
Maintain the typings without having to 'wrap' the match function in a Boolean. This only seems to be the case after 1.0.0-rc.1 as well. Releases prior to that seem to work well.
https://codesandbox.io/s/unruffled-bassi-ts45b
Additional context
"xstate": "^4.11.0"
"@xstate/react": "^1.0.0-rc.5"

That is really weird. What version of TypeScript are you using?

Uhh, yeah, it's super weird.
I'm using 3.9.6. It does it in that codesandbox link as well if you comment/uncomment my if's, and it looks like it's using Typescript v3.3.3

This may be related to #1142, #947, #1184. You can find a few workarounds that may work for you there

This actually looks slightly different to me. In all linked issues this conditional type:
https://github.com/davidkpiano/xstate/blob/597cfdc659b03f2095e73faf3f82702ae68a3911/packages/core/src/State.ts#L294
was resolved to never, but in here it seems that the other branch is chosen and because supplied default here:
https://github.com/davidkpiano/xstate/blob/597cfdc659b03f2095e73faf3f82702ae68a3911/packages/xstate-react/src/useMachine.ts#L105
is different than what we default to in the core:
https://github.com/davidkpiano/xstate/blob/597cfdc659b03f2095e73faf3f82702ae68a3911/packages/core/src/State.ts#L87
this is not working properly.
I thought that simply providing the same default would solve this:
https://codesandbox.io/s/mutable-water-gjqm4?file=/src/index.tsx
but, unfortunately, it doesn't - for some reason. However, if I supply manually that default in a place of TTypestate to State then the correct problem gets reported.


Create gettings_started_with_Clojure.md

Still editing

Great work. :+1: Let us know when you're done so we can review and merge it.


register with a different tenant code

Hi James,
Sorry to put this under an "issue" section. I couldnt find a way to contact you. I just needed some directions in knowing how do I register using a different tenant code
Kind Regards
Shrenik

@shrenicshah The solution comes with two example sites showing how to use the MultitenantUserStore.
You need to set the TenantId. In the example it's hard coded, but you need to determine how you distinguish tenants and set it on the MultitenantUserStore to allow it to work.


Get-ForensicAmcache returns an error

When I'm trying to run Get-ForensicAmcache it returns an error:
Get-ForensicAmcache : Cannot find key 'Root\File' in the 'C:\Windows\appcompat\Programs\Amcache.hve' hive because it does not exist.
At line:1 char:1

Get-ForensicAmcache -HivePath C:\Windows\appcompat\Programs\Amcache.h ...

  + CategoryInfo          : NotSpecified: (:) [Get-ForensicAmcache], Exception
  + FullyQualifiedErrorId : System.Exception,PowerForensics.Cmdlets.GetAmcacheCommand

Am I doing something wrong?
The path for Amcache.hve is correct.

Same issue here

any update here? Is it fixed or found any solution to fix it?

I think the repository is just abandoned and old, and these caches change with every windows release, so it's not going to work unless someone goes through a huge modernization effort with the repo @techassassinator


Get rid of dyn trait for connection manager

The dyn trait is no longer needed after the central/peripheral split
refactoring.

/test


Bug when the plugin after create the folder and after stop and start the server the plugin don't load and some times the plugin putting up the plugin for the first time in the server don't load

I install the plugin and the plugin was correctly loading but after i stop and start the server the plugin don't load, even removing the plugin folder after stop the server, the plugin still don't load and the error is this one.
[Server thread/ERROR]: Error occurred while enabling RealSkywars v6-9-2022 Build 1 (Is it up to date?)
java.lang.NullPointerException: Cannot invoke "org.bukkit.configuration.ConfigurationSection.getKeys(boolean)" because the return value of "org.bukkit.configuration.file.FileConfiguration.getConfigurationSection(String)" is null
at josegamerpt.realskywars.sign.SignManager.loadSigns(SignManager.java:38) ~[RealSkywars.jar:?]
at josegamerpt.realskywars.RealSkywars.onEnable(RealSkywars.java:320) ~[RealSkywars.jar:?]
at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:264) ~[purpur-api-1.19.2-R0.1-SNAPSHOT.jar:?]
at org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoader.java:370) ~[purpur-api-1.19.2-R0.1-SNAPSHOT.jar:?]
at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:548) ~[purpur-api-1.19.2-R0.1-SNAPSHOT.jar:?]
at org.bukkit.craftbukkit.v1_19_R1.CraftServer.enablePlugin(CraftServer.java:596) ~[purpur-1.19.2.jar:git-Purpur-1781]
at org.bukkit.craftbukkit.v1_19_R1.CraftServer.enablePlugins(CraftServer.java:510) ~[purpur-1.19.2.jar:git-Purpur-1781]
at net.minecraft.server.MinecraftServer.loadWorld0(MinecraftServer.java:641) ~[purpur-1.19.2.jar:git-Purpur-1781]
at net.minecraft.server.MinecraftServer.loadLevel(MinecraftServer.java:427) ~[purpur-1.19.2.jar:git-Purpur-1781]
at net.minecraft.server.dedicated.DedicatedServer.initServer(DedicatedServer.java:343) ~[purpur-1.19.2.jar:git-Purpur-1781]
at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1142) ~[purpur-1.19.2.jar:git-Purpur-1781]
at net.minecraft.server.MinecraftServer.lambda$spin$1(MinecraftServer.java:310) ~[purpur-1.19.2.jar:git-Purpur-1781]
at java.lang.Thread.run(Thread.java:833) ~[?:?]
¿Is this a bug or a error creating the configuration files?

Hi there! Please check the signs.yml file to see if it contains anything.

The signs.yml its empty. That mean something?

It should have "Signs:"

The signs.yml just contain this without anything else i just delete the realskywars folder so the plugin can create a new folder, but its still coming out the error i show you before.

Try putting in Signs: and see if it does anything different

Still load the same error.

See if the most recent commit fixed the issue

I don't know what are you mean. Can you tell me what i need to do?

 Can you upload a release or a update to the plugin in spigot with this fix version? because i don't know how to download this new version

Please check if this version works for you
RealSkywars.zip

It work! Thank you so much


Picoweb.run error

Using Picoweb on ESP32 devkit v1: one months ago on same board, all ok; tryed on other 2 identhical board, same error.
Here my simple code to test:
`import picoweb
app = picoweb.WebApp(name)
@app.route("/")
def index(req, resp):
yield from picoweb.start_response(resp)
yield from resp.awrite("Hello world from picoweb running on the ESP32")
app.run(debug=True, host="<IP_ADDRESS>")
`
Error after end of booting:
` ...
I (2373) network: CONNECTED
I (5213) event: sta ip: <IP_ADDRESS>, mask: <IP_ADDRESS>, gw: <IP_ADDRESS>
I (5213) network: GOT_IP
:: connecting to network "<<<my_ssid> (try 2)"...
:: WLAN connected to ip <IP_ADDRESS> gateway <IP_ADDRESS>
I (13773) modsocket: Initializing

Running on http://<IP_ADDRESS>:8081/
Traceback (most recent call last):
File "main.py", line 12, in
File "/lib/picoweb/init.py", line 302, in run
File "/lib/uasyncio/core.py", line 129, in run_forever
File "/lib/uasyncio/init.py", line 31, in add_reader
TypeError: function expected at most 3 arguments, got 4
MicroPython v1.11-269-gb29fae0c5 on 2019-09-03; ESP32 module with ESP32
Type "help()" for more information.
`

May be a bug?
Thank you,
Mauro

Not enough information provided, but looks like duplicate of https://github.com/pfalcon/picoweb/issues/45

Ok @pfalcon ,
I can give you information about versions that worked on previous installation.
Versions where all had worked:
Installing picoweb 1.7.1 from https://files.pythonhosted.org/packages/1b/4f/f7d35f90521e95d9d2307f69ff523133d7d4dd6da7ce1ce0c8382e7255fa/picoweb-1.7.1.tar.gz
Installing pycopy-uasyncio 3.1.1 from https://files.pythonhosted.org/packages/5f/24/fb08acdd7ebf1626dcdb3cdaf0d3f463c254a4a3aa2cab70b0ee6562a83d/pycopy-uasyncio-3.1.1.tar.gz
Installing pycopy-pkg_resources 0.2.1 from https://files.pythonhosted.org/packages/05/4a/5481a3225d43195361695645d78f4439527278088c0822fadaaf2e93378c/pycopy-pkg_resources-0.2.1.tar.gz
Installing pycopy-uasyncio.core 2.3 from https://files.pythonhosted.org/packages/6a/96/80a86b1ea4e2b8c7e130068a56f9e8b5bbb28369e48de56a753778e14faf/pycopy-uasyncio.core-2.3.tar.gz
The new versions:
Installing picoweb 1.7.2 from https://files.pythonhosted.org/packages/a7/cb/1b48103a22f8d67985af5e30ee157c195d2a37b12864e6e8744ad8e7e1b6/picoweb-1.7.2.tar.gz
Installing pycopy-uasyncio 3.1.2 from https://files.pythonhosted.org/packages/73/ef/f5c8a4a9ed2bd20e615d1c638b0547ef1487e4cb0c11533b94dead9191ee/pycopy-uasyncio-3.1.2.tar.gz
Installing pycopy-pkg_resources 0.2.1 from https://files.pythonhosted.org/packages/05/4a/5481a3225d43195361695645d78f4439527278088c0822fadaaf2e93378c/pycopy-pkg_resources-0.2.1.tar.gz
Installing pycopy-uasyncio.core 2.3.1 from https://files.pythonhosted.org/packages/35/29/db2c4a67a1e03bf8db4059dcb7cb8697c4b3e0c09535b6882291af13c064/pycopy-uasyncio.core-2.3.1.tar.gz
and for me it's not clear if the problem is in picoweb or uasyncio.
You need other info?

You must use https://github.com/pfalcon/pycopy . Please read the README.

All clear, thank you.
m


Changing direction of a default light

It is possible to change direction of a default light in the scene?
fragment.getArSceneView().getScene().getSunlight().getLight().getLocalDirection())

What I see is that the Sun node has default direction
static final Vector3 DEFAULT_SUNLIGHT_DIRECTION = new Vector3(0.7F, -1.0F, -0.8F);
set with this.setLookDirection(DEFAULT_SUNLIGHT_DIRECTION.normalized());. You can try setting the look direction to another value for the node. Otherwise we have modified the default direction of directional lights (the Sun node contains light of this kind) in our code by manipulating the localRotation of the nodes containing the lights. Hope that this helps.

What I see is that the Sun node has default direction
static final Vector3 DEFAULT_SUNLIGHT_DIRECTION = new Vector3(0.7F, -1.0F, -0.8F);
set with this.setLookDirection(DEFAULT_SUNLIGHT_DIRECTION.normalized());. You can try setting the look direction to another value for the node. Otherwise we have modified the default direction of directional lights (the Sun node contains light of this kind) in our code by manipulating the localRotation of the nodes containing the lights. Hope that this helps.

thanx, this helped


really slow after match

im having issues with REALLY REALLY slow after matches, dont know if this is totsugeki issue or not but look at my server responses

I suspect it's because you have average (or below average) upload speed. Since the server is in Japan, the upload speed needs to be really good to avoid those long times (and also, ASW better organizing their messy API ; like...a win should just be a win uploaded and then maybe the replay but in the background). I have the same problem. While I have amazing download speed and stability, my upload speed is the average french one (1-2Mb/s)

I suspect it's because you have average (or below average) upload speed. Since the server is in Japan, the upload speed needs to be really good to avoid those long times (and also, ASW better organizing their messy API ; like...a win should just be a win uploaded and then maybe the replay but in the background). I have the same problem. While I have amazing download speed and stability, my upload speed is the average french one (1-2Mb/s)

i have a 20 mb/s up speed tho, here are my tests

maybe its the ASW servers that are exploding ?

i guess they're really that bad... they really need to optimize their /set API

try checking your connection to tokyo, since the game has to connect to their servers over there after matches to update your profile

dont know if this is totsugeki issue or not but look at my server responses

Given only the /stats/set is slow, I would echo what @notKamui and @ozne1 are saying, in that it has to do with your upload speed to Tokyo.

maybe its the ASW servers that are exploding ?

Also likely, or the internet between you and Japan is slow. Not all ISP's are built alike, and some have very poor connectivity to other countries (ie Australia to the rest of the world).

Closing this out. Feel free to reopen if it's still happening.


Add unit sub-selection for values in SCSS/CSS

Checklist

[x] I have added tests

@pokey — this one is ready to go again.

:wave: Hey @Will-Sommers, how's this PR going? I'd personally love to see this land; let us know if someone else will need to pick this up.

Heyo @auscompgeek feel free to take it over!
This weekend I should be able to push up my code related to the regex version of this. I need to scroll back in my convos with @pokey to see where we fell on take every unit within a string.

Looks good to me. If there are any details we can discuss them in the meeting.

Looks good to me. If there are any details we can discuss them in the meeting.

well there's the ever-present parser vs regex discussion 😅 https://github.com/cursorless-dev/cursorless/pull/722#discussion_r893104146.  In this case I think parser wins due to the multiple unit from key thing, but curious what you think

Looks good to me. If there are any details we can discuss them in the meeting.

well there's the ever-present parser vs regex discussion 😅 #722 (comment). In this case I think parser wins due to the multiple unit from key thing, but curious what you think

I think that the syntax tree is a better match here. Personally I don't really have the need to support units in any other language anyhow. Not one that I can think of on top of my head.

Personally I don't really have the need to support units in any other language anyhow. Not one that I can think of on top of my head.

Talon script within sleep()?

Personally I don't really have the need to support units in any other language anyhow. Not one that I can think of on top of my head.

Talon script within sleep()?

The amount of times I want to change just the unit in a Talon script sleep is more or less zero. Turning 500ms into 1s sure but in that case I would not target just the unit.


Adding hover effect on navbar

#109
I have added hover effect in navbar
https://user-images.githubusercontent.com/69043944/114294760-26db5600-9a4d-11eb-8dd6-fb275611a78e.mp4
https://user-images.githubusercontent.com/69043944/114295005-dbc24280-9a4e-11eb-8d51-485f78020182.mp4

@kumarishalini6 Resolve the conflicts

Hey! i have solve the conflicts file but don't know about workflow error

Not able to switch on this branch

Not able to switch on this branch

You might have deleted the branch..

@kumarishalini6 Please fix the merge conflicts

Ok

Done


Fixed #31051 -- Fix handling of circular references involving natural keys

This is a slightly cleaned up version of #12166, fixing just ticket-31051.

Rebased.

I tried to move all tests that work without this patch to a separate PR #12680.

Rebased on top of #12680.

@matthijskooijman Thanks again :+1: I pushed minor edits and reorganized commits.


Define how to manage meetings

To manage the backlog and vote on open RFC we need to organize open community meetings. For this reason, is important to define what kind of meetings we want to have, how to organize them, and how to document the action point decided during the meeting

A section on meetings organization has been added to GOVERNANCE.md.
See issue #40

A section on meetings organization has been added to GOVERNANCE.md.
See issue #40


10453 DEV Add functions to clean all the system node components and remove the shared folders.

https://github.com/Shippable/pm/issues/10453

This is complete.
Closing


[question] Why 2 of 3?

Hey @mflaxman - thanks for the guide.
I'm concerned about the probability of simultaneously losing 2 out of three keys in between "health checks". For that reason I think 3 of 5 might be better.
Is that fair or do you disagree with the probability of that happening?

Is this section not clear?

Very open to changes, but closing this github issue for now as it seems there is already a page on this during the setup process.


Done with homepage

https://www.loom.com/share/3133ceb65a4f46b38baaa28a5663ce57?sid=6074625a-3648-4a6b-af89-282a931b8bb5
this is the loom recording


add creative commons c0 license

closes gh-283

/cc @steveklabnik to merge if you feel this is sufficient

:+1:

@tkellen can you rebase please so we can merge before 1.0?

Hows that?

thanks :+1:


Is there a specific set of mime types supported?

When I choose an image from my iPhone gallery, sometimes it is not received by the app but it works for some images. Is there just a specific set of mime types supported?

All images should be supported. Can you try the latest version? It has a fix were sometimes the app was not able to receive images.

All images should be supported. Can you try the latest version? It has a fix were sometimes the app was not able to receive images.

It looks like there's an issue on my end. I am getting the file paths correctly but looks like new File(path) isn't able to get the actual image file for all image extensions.


Added basic kaldi-gstreamer-server support

Very basic support for kaldi-gstreamer-server, using their HTTP API.
A complete implementation would use chunked transfers or websocket to be faster (speech starts processing before it's all been sent) and more usable (timeouts) but I didn't want to add any dependencies.

Hi astropika,
This looks pretty good so far - I'm going to merge this when I have some time.
Just as a heads up, we'll need to make the following changes afterward:

Better setup instructions for kaldi-gstreamer-server on all 3 major supported platforms, includingstep-by-step instructions for how to set it up starting from a fresh OS install.
Plain-English error messages for all of the most common errors.
A recommended Kaldi server configuration.

Is there any update on when this may get pulled in?

An open PR for about 15 months now? So, it will not happen.

Any news on this? Could it be rebased on the last version @astropika ?

@Uberi @astropika any word on this?


Comments with four stars are more are incorrectly parsed

Consider this reduced example from Phobos:
private void formatUnsigned()
{
    /**** Print ****/

    if (1) put(1);
}

@safe pure unittest
{
}

The Unittest block has "Print" has comment attached.

How are you seeing this bug show up? When I ask D-Scanner for a dump of the token information it gives me this as expected:
$ dscanner --tokenDump test.d
text                    	blank	index	line	column	type	comment	trailingComment
<<             private>>	1	0	1	1	148
<<                void>>	1	8	1	9	179
<<      formatUnsigned>>	0	13	1	14	65
<<                   (>>	1	27	1	28	21
<<                   )>>	1	28	1	29	22
<<                   {>>	1	30	2	1	56
<<                  if>>	1	59	5	5	125	Print
<<                   (>>	1	62	5	8	21
<<                   1>>	0	63	5	9	72
<<                   )>>	1	64	5	10	22
<<                 put>>	0	66	5	12	65
<<                   (>>	1	69	5	15	21
<<                   1>>	0	70	5	16	72
<<                   )>>	1	71	5	17	22
<<                   ;>>	1	72	5	18	32
<<                   }>>	1	74	6	1	60
<<                   @>>	1	77	8	1	49
<<                safe>>	0	78	8	2	65
<<                pure>>	1	83	8	7	151
<<            unittest>>	1	88	8	12	176
<<                   {>>	1	97	9	1	56
<<                   }>>	1	99	10	1	60

How are you seeing this bug show up?

IIRC with the --ast dump it was shown as comment of the unittest block, but it seems like I can't reproduce this anymore.
Sorry for the noise and horrible bug report.