ScenarioFn uses common testing interface

Resolves https://github.com/form3tech-oss/f1/issues/278

Adds a new interface TF which exposes common testing methods
Adds f1.Register to replace f1.New using ScenarioFunc and RunFunc which each use testing.TF rather than testing.T

Most of the complications are due to preserving backwards-compatibility with the existing f1.New method.
We had to remove t.Parallel() from an existing (previously-unique) test after adding a new one to the same module since this triggered race-detection. We have confirmed that this was an previously-existing issue (by simply duplicating the existing test). It may be worth raising an issue on this.

@sirockin would you mind updating the PR again to pull #281 so that we can run tests on the PR please 🙏

@sirockin would you mind updating the PR with master again to pull #281 so that we can run tests on the PR please 🙏

@nvloff-f3, thanks for the feedback and sorry about the lint errors. I've merged from main and attempted to fix lint errors as best as I can. Unfortunately there seem to be some conflicting linting rules when using //nolint:
I get:
==> Running golangci-lint...
pkg/f1/f1_deprecated_scenarios_test.go:5:1: directive `// nolint: paralleltest // running tests in parallel reveals a pre-existing a race condition` should be written without leading space as `//nolint: paralleltest // running tests in parallel reveals a pre-existing a race condition` (nolintlint)
// nolint: paralleltest // running tests in parallel reveals a pre-existing a race condition
^
pkg/f1/f1_scenarios_test.go:5:1: directive `// nolint: paralleltest // running tests in parallel reveals a pre-existing a race condition` should be written without leading space as `//nolint: paralleltest // running tests in parallel reveals a pre-existing a race condition` (nolintlint)
// nolint: paralleltest // running tests in parallel reveals a pre-existing a race condition
^
pkg/f1/testing/t.go:40:1: directive `// nolint: interfacebloat // we are deliberately implementing a subset of the testing.TB interface` should be written without leading space as `//nolint: interfacebloat // we are deliberately implementing a subset of the testing.TB interface` (nolintlint)
// nolint: interfacebloat // we are deliberately implementing a subset of the testing.TB interface

But when fix the space I then get:
==> Running golangci-lint...
pkg/f1/f1_deprecated_scenarios_test.go:5: File is not `gofmt`-ed with `-s` (gofmt)
//nolint: paralleltest // running tests in parallel reveals a pre-existing a race condition
pkg/f1/f1_scenarios_test.go:5: File is not `gofmt`-ed with `-s` (gofmt)
//nolint: paralleltest // running tests in parallel reveals a pre-existing a race condition
pkg/f1/testing/t.go:40: File is not `gci`-ed with --skip-generated -s standard -s default -s prefix(github.com/form3tech-oss/f1) (gci)
//nolint: interfacebloat // we are deliberately implementing a subset of the testing.TB interface
make: *** [lint] Error 1

@nvloff-f3 also I haven't replicated all the tests currently implemented against New() for Register(). Happy to do so though.

Hi @sirockin I've been thinking about this and I have some concerns I want to share.
I don't think the deprecated APIs can ever be removed. We have thousands of f1 scenarios at Form3, we can't realistically change all of them. This would mean that we must forever keep two separate APIs for f1, which is not something we're comfortable maintaining, especially given how we have no use case for the new APIs.
Looking at go's testing package the testing.TB is just defined, but concrete types are passed to user defined functions. Where the interface is there just to allow users to write helpers that can be reused between tests and benchmarks. In that way I'm even more reluctant to introduce the new APIs, as they are not really necessary.
Let's take the following use case:
package main

import (
	stdtesting "testing"

	"github.com/form3tech-oss/f1/v2/pkg/f1"
	f1testing "github.com/form3tech-oss/f1/v2/pkg/f1/testing"
)

func main() {
	f1.New().
		Add("emptyScenario", emptyScenario).
		Execute()
}

func emptyScenario(*f1testing.T) f1testing.RunFn {
	runFn := func(t *f1testing.T) {
		myTest(t)
	}

	return runFn
}

func TestEmpty(t *stdtesting.T) {
	myTest(t)
}

func myTest(t stdtesting.TB) {
	t.Error("test")
}

This does not compile due to testing.TB forbidding interface implementations:
benchcmd/main.go:18:10: cannot use t (variable of type *"github.com/form3tech-oss/f1/v2/pkg/f1/testing".T) as "testing".TB value in argument to myTest: *"github.com/form3tech-oss/f1/v2/pkg/f1/testing".T does not implement "testing".TB (missing method private)

Now if we introduce a copy of testing.TB interface in f1 that does allow users to implement it - it would mean they must use that interface across the test suite. Which would make f1 a generic testing library, and we're not looking to take f1 in that direction.
This now compiles
package main

import (
	stdtesting "testing"

	"github.com/form3tech-oss/f1/v2/pkg/f1"
	f1testing "github.com/form3tech-oss/f1/v2/pkg/f1/testing"
)

func main() {
	f1.New().
		Add("emptyScenario", emptyScenario).
		Execute()
}

func emptyScenario(*f1testing.T) f1testing.RunFn {
	runFn := func(t *f1testing.T) {
		myTest(t)
	}

	return runFn
}

func TestEmpty(t *stdtesting.T) {
	myTest(t)
}

// all the shared test code must now use this interface
func myTest(t f1testing.TF) {
	t.Error("test")
}

However, this interface doesn't have to be in f1, I think it's better to leave it up to users to decide.
All of that being said, I think if we can get around the breaking change of Error and Fatal signature changes - I'm happy to make f1's testing.T compatible with testing.TB. However, I'm reluctant to add the generic interfaces in f1, as I don't think it's a good idea for users to tie their test suites to this package.

The best idea I have to avoid the API breakage is to introduce t.TBCompatible() that would return a struct embedding the t and implementing the extra methods.

Thanks for the useful and extensive feedback, @nvloff-f3. Your points make very good sense.
Making t.Error() and t.Fatal() compatible with testing.TB would definitely unblock our use-case
I would still argue that the .TF interface could be usefully included in the package but happy to let that one slide.
Closing this in favour of: https://github.com/form3tech-oss/f1/pull/286


Not enough details on the unexpected errors occur at jvm-backend-codegen phase

Description:
When theres an unexpected error occurred at the jvm-backend-codegen, it only logs a message saying failed running jvm code gen phase.. But this is not enough to figure out the root cause.
Ideally there should be a bad sad error with complete stacktrace, since this is an unhandled error, that causes the compiler to crash.

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/20373


fix: funfetti cake link

Fix link on funfetti page

https://mynameisyeh.com/blog/2014/9/funfetti-cake
https://nicholaswilde.io/recipes/desserts/cake/funfetti-cake/

Fixed with bfd85a0886461f561d43fa5f1951ea15d914cf76


Adapt M3 Backend

It seems the code produced by the current backend is not suitable for the state of M3. For example the current backend instantiates VPE::new_child_vpe(...) while there is no such method in the current Rust interface of M3.

There's a separate branch implementing a simplified API for M3 that implements the interface used by the current Backend integration


Config Screen Minecraft 1.15 and ClothAPI

With the config screen being blocked with Minecraft 1.15 do we HAVE to have the ClothAPI installed as well? Seems pointless since we can't actually adjust any configs from in game in 1.15.

you can edit the configs, that's what cloth config is for.

Running Fabric yes you can edit them in game, but running Forge you have to edit configs manually from the config folder for Minecraft version 1.15

Cloth doesnt add a screen, but it provides a screen for light overlay to add a screen.
Open up the mod list added by forge and edit the screen.

Ah, I see it now. Idk how I missed that before. Now I wish more mods used
Cloth API lol Sorry to bother you.
On Sun, Jun 7, 2020, 1:19 AM shedaniel<EMAIL_ADDRESS>wrote:

Cloth doesnt add a screen, but it provides a screen for light overlay to
add a screen.
Open up the mod list added by forge and edit the screen.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/shedaniel/LightOverlay/issues/42#issuecomment-640159012,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKCQDO3CML3FVWKZEQNITLRVMPNPANCNFSM4NWS3SRQ
.

Don't worry, I also think it is a pity that most mods don't have a config screen because forge doesnt have one yet, I think when forge finally adds their config screen mods will start to use them.


mobilenet v3 object detection model?

any plans to implement mobilenet v3 for object detection?

@ghylander I don't currently have plans to, as the pytorch-ssd code I use for training doesn't support it.  Perhaps in the future I can look into it, or if it gets added on the pytorch side in the training code.

do you want me to leave the issue open or  close it?


is tcg voyager usable with lumen?

Laravel Version: #.#.#
Voyager Version: #.#.#
PHP Version:
Database Driver & Version:

Description:
Hi i wanna use tcg voyager with lumen  for admin pannel
I can create a separate project , but i prefer not to. so can you give me a manual for using voyager inside lumen ?
Steps To Reproduce:

I can't give you a manual to do so, because we don't officially support Lumen.  You're welcome to give it a try and report back.


Types for MessageEvent and ReceivedEvent

This was done a couple of commits ago, and seems to work nice. Will add some explicit tests for all props, but done enough for now.


Pr/1

Sorry about the pull, it is a misconfiguration. I would like to update the example in my private repo.


Drone testing

A minor technical glitch still, but we have a fully running unit test suite!!

http://nipy.bic.berkeley.edu:8080/github.com/stefanv/mltsp/drone_test/28fcb13f4bfce909a6a2fdbbad224cb5c1d20594
:dancers:

Wow, that is GREAT!! So awesome!!
On Fri, May 15, 2015 at 2:21 AM, Stefan van der Walt <
<EMAIL_ADDRESS>wrote:

http://nipy.bic.berkeley.edu:8080/github.com/stefanv/mltsp/drone_test/28fcb13f4bfce909a6a2fdbbad224cb5c1d20594
[image: :dancers:]
—
Reply to this email directly or view it on GitHub
https://github.com/mltsp/mltsp/pull/22#issuecomment-102339288.


Add CamVid dataset for segmentation

Because of the request #60, I implemented this Camvid dataset class for people who are interested in doing image segmentation.
Since both the input and target should go through the same transformation, I added the file join_transform.py which allows us to do the same image transformation on a list of images.
Sample usage:
import torch
from torchvision.datasets import camvid
from torchvision import transforms, joint_transforms

normalize = transforms.Normalize(mean=camvid.mean, std=camvid.std)
train_joint_transformer = transforms.Compose([
    joint_transforms.JointRandomCrop(224),
    joint_transforms.JointRandomHorizontalFlip()
    ])
train_dataset = camvid.CamVid('path/to/CamVid', 'train',
                      joint_transform=train_joint_transformer,
                      transform=transforms.Compose([
                          transforms.ToTensor(),
                          normalize,
                      ])
train_loader = torch.utils.data.DataLoader(train_dataset, batch_size=10, shuffle=True)

Currently, the download function is not implemented yet.
It requires the users to download the data from here by themselves.

So, I'm not sure about the Joint Transforms. We were thinking about factoring out the random number generation from the transforms, so that the same random transform can be applied to different inputs (and eventually to inputs from different modalities, such as images and bounding boxes).
I'll send a PR with these transforms tomorrow.

OK. I'll modify the code based on the new version of transforms.

Hi @fmassa,
I am wondering what the conclusion of joint random transformations is.
In my opinion, making all the transform class being able to take either a image or a list of images as inputs, and providing a joint_transform parameter the dataset object like my code could be a solution.
Also, I made the function private. However, the other two class should be public. They should be able to be used by the users. LabelToLongTensor can be passed by as a transform class to the Camvid class.
More importantly, LabelToPILImage can used to visualize the predicted labels.

Hi @felixgwu,
In a local branch I factored out the random parameter generation from the transforms, and I'm using it now in a project for semantic segmentation.
The drawback of this approach is that we need to pass an extra argument to the constructor of the dataset (let's call it generators). The generators are a list of object that when called generate the random parameters for the transforms.
Thus, in the __getitem__, if the generators were provided in the constructor, we need to call them in order to generate the parameters for the transforms, that can be used for both the inputs and the targets.
This requires passing a new argument to the constructor, but I think it's better than having joint transforms, because we light want to apply some individual transforms before / after the joint transforms, which would not be possible in this setup.
I'm not yet 100% hapoy with my refactoring, but it's time to send a PR to get some feedback. I'm out of my computer this weekend, but when I create the PR I'll tag you on it.

Hi @felixgwu sorry for the delay on this, we are discussing a refactor to transforms in #230 and should have something merged soon after which using the random transformation parameters between transforms should be much easier.

Did it succeed?

Hello @felixgwu and sorry for taking that long to get back at you.
As you might know, there is a new dataset API being designed and existing datasets will be ported to it. Here is a thread explaining the logic behind it: https://github.com/pytorch/vision/issues/5336.
Also, there is this thread that discusses adding the CamVid dataset: https://github.com/pytorch/vision/issues/60. You are probably aware of it.
As far as my understanding goes, it would be better to wait a bit until the new design is stable. Then, it would be best to port the code here to the new design. Someone can help you doing it of course and proper attribution will be given to you @felixgwu of course or you can do it yourself if you want to. What do you think @felixgwu?
Also, since this PR is a bit old maybe you don't need it anymore and/or found an alternative. Any feedback is welcome @felixgwu.
Thanks again and sorry for the long delay.

@yassineAlouini This PR adds two things:

The image dataset CamVid (I erroneously thought this was a video dataset in #60)
Joint transformations for images and segmentation masks.

The second part is well covered by the transforms rework that is currently going on. If I'm not mistaken we already have transformations for everything proposed here in torchvision.prototype.transforms.
As for the dataset, the implementation seems pretty straight forward. Porting it to the new API should be simple. As of now, we haven't we looked mostly into classification and detection datasets, but segmentation datasets will follow soon. We should wait at least until then before we have a go at this.


Conda env create stalls and loops indefinitely

Current Behavior
Conda install hangs and loops indefinitely
Steps to Reproduce
It stalls out when executing the following command:
conda env create -f env.yaml
It runs correctly if you comment out conda-forge in env.yaml but it doesn't create the environment.
Expected Behavior
Conda installs all packages successfully and activates the environment without errors
Environment Information
conda info
  active environment : base
  active env location : /home/shawn/anaconda3
     	shell level : 1
	user config file : /home/shawn/.condarc
populated config files :
   	conda version : 23.1.0
 conda-build version : 3.22.0
  	python version : 3.9.13.final.0
	virtual packages : __archspec=1=x86_64
                   	__glibc=2.35=0
                   	__linux=5.19.0=0
                   	__unix=0=0
	base environment : /home/shawn/anaconda3  (writable)
 conda av data dir : /home/shawn/anaconda3/etc/conda
conda av metadata url : None
    	channel URLs : https://repo.anaconda.com/pkgs/main/linux-64
                   	https://repo.anaconda.com/pkgs/main/noarch
                   	https://repo.anaconda.com/pkgs/r/linux-64
                   	https://repo.anaconda.com/pkgs/r/noarch
   	package cache : /home/shawn/anaconda3/pkgs
                   	/home/shawn/.conda/pkgs
	envs directories : /home/shawn/anaconda3/envs
                   	/home/shawn/.conda/envs
        	platform : linux-64
      	user-agent : conda/23.1.0 requests/2.28.1 CPython/3.9.13 Linux/5.19.0-32-generic ubuntu/22.04.1 glibc/2.35
         	UID:GID : 1000:1000
      	netrc file : None
    	offline mode : False

Please let me know if you need any further information to help reproduce this problem.
Thanks!
Shawn

Thank you @Dene33 !
Plus, following worked for me to create the enviroment.
(1) 'apt install cuda-11-6' (maybe needed 'apt install gcc-10 g++-10')
(2) add "gyp" to env.yaml's channels.
(3) rename directory name, "third-party" to "third_party"
I googled and fixed other bugs like numpy downgrade, 'export EGL_DEVICE_ID=1', etc., but I don't remember exactly. Sorry.
Anyway, the code in this repository works!

We have updated the installation instructions, so that the installation is faster. If there are still issues, we give more options for trouble-shooting in the Readme. Please let us know if you still face problems with the installation.


Add peer and local socket addrs, as well as Request::remote

this is the first PR of three to address https://github.com/http-rs/tide/issues/462
This also adds an initial implementation of Request::remote and Request::forwarded_for. Parsing the Forwarded header value probably would be more elegant with a proper parsing or regex library, but I didn't want to add an additional dependency just for this.

Addressed all changes, thanks for the review!


Undefined references in documentation

" under VC++ Directories," ...  Where is that ?

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

ID: b46cf831-faea-6357-0e06-3284f114c879
Version Independent ID: 0345affe-ce33-c438-d3e4-d134f905d084
Content: Common macros for MSBuild commands and properties
Content Source: docs/build/reference/common-macros-for-build-commands-and-properties.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

"To display all of the currently available macros, in the Property Pages dialog, under VC++ Directories"
I do not see the "Property Pages dialog, under VC++ Directories"

@dfisk
Thanks for including the screenshot. It shows me that you are in fact looking at what's under VC++ Directories. We're using "under" as a preposition in sense 3a on Merriam-Webster's dictionary site, as in: "under the heading VC++ Directories". From this screen, just follow the remainder of the directions: choose the drop-down arrow at the end of a property row.  (I'd pick Executable Directories just because it's first.) Click on Edit and then in the edit dialog box, choose the Macros button.


feat: replace stringContains with String.includes

replacing the helper function stringContains with the build-in string function .includes()

Thanks!


Add SimpleReport Sender transform for missing data

This PR adds simple report sender transforms for missing fields, so that receivers can get data in the same format they've have been receiving it in the covid pipeline.
Test Steps:

./gradlew clean package
./gradlew testIntegration

Changes

Include a comprehensive list of changes in this PR
(For web UI changes) Include screenshots/video of changes

Checklist
Testing

[ ] Tested locally?
[ ] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[ ] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Linked Issues

Fixes #issue

To Be Done
Create GitHub issues to track the work remaining, if any

#issue

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

new endpoint A
new endpoint B

Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

I think this was what got merged into that other branch?

@oslynn Do we still need this PR?
Closing because it seems old and everything seems to be working fine.


Initial cluster context command

PR introduces a new command. osdctl cluster context which takes in either an internal clusterID, external clusterID, or clustername and returns whether the cluster is fully supported or in limited support.
Moving forward we will be adding more functionalities to this command such as reporting on triggered alerts, Sevicelogs, and more.

@sam-nguyen7 Could this be folded into osdctl cluster health?

@fahlmant this is intended to be a quick-reference of state based on external tools.  We're planning to extend it over time.


Proofreading and editing after the translation

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

Closed and Reopened this PR in an attempt to attempt to break out of the Travis build pending status.


Add systemd module to metadata deps

Adds the systemd module to the metadata
dependencies.

Ah, didn't see it there, no worries then just thought it was a miss in documentation. I wasn't able to use it without that module which I was hoping for.


Document flag 21

https://github.com/Discord-Datamining/Discord-Datamining/commit/23f62a24a9ab6b2f12d19c274646802307448dc6

As far as I'm aware, that's toggled client-side. There's no guarantee that it isn't used for something else internally.

As far as I'm aware, that's toggled client-side. There's no guarantee that it isn't used for something else internally.

There is no guarantee but client reefers it like that also the naming of the function which is added that uses this flag indicates it's general use and i don't think flags have double uses so i am sure that this is the correct way of describing this flag.

I think I'd take @MeguminSama's client side for now until we have more info.

I think I'd take @MeguminSama's client side for now until we have more info.

As far as I'm aware, that's toggled client-side. There's no guarantee that it isn't used for something else internally.

Changing this setting in-app sends a PATCH request to /users/@me with the flag, so it's not just a client-side flag.

Updated review

@ShockTr any chance you could make the requested changes so we can merge this? thanks

@ShockTr any chance you could make the requested changes so we can merge this? thanks

which requested changes i can't see one 🤔

@ShockTr Make the description more absolute and then we can merge. Since this is confirmed there's no point having a possibly.

The review requested this change.
description: 'Forcefully disables Nitro features.'

Thank you for your contribution.


continuos network error on app (see image)

Whenever I load the webpage I get a continous stream of red blocks that say monster fail. See image.

can you please do a right click on your screen, click inspect, go to the network tab, press esc to also show the console in the bottom half of the screen and print what's there? try to get the last errors

Hi,
I hope this is right, if not let me know what i'm doing wrong.

Same thing here. My screenshot:

I can also see these error messages:
Mixed Content: The page at 'https://monstereos.io/' was loaded over HTTPS, but requested an insecure resource 'http://mainnet.eoscalgary.io/v1/chain/get_info'. This request has been blocked; the content must be served over HTTPS.
Mixed Content: The page at 'https://monstereos.io/' was loaded over HTTPS, but requested an insecure resource 'http://mainnet.eoscalgary.io/v1/chain/get_table_rows'. This request has been blocked; the content must be served over HTTPS.

@felipemartim nice debug my friend!!! Try to access http://monstereos.io instead of httpS://monstereos.io
we are not supporting https for now, but you guys are safe because everything is signed in your local Scatter. https/http does not matter. Can you let me know if it also works for you @rreeve

@leordev thanks, http works but Google redirects to https automatically.

@leordev Yes, it works perfectly now. Thanks.


chore(master): release 2.0.5

:robot: I have created a release beep boop
2.0.5 (2023-11-08)
Bug Fixes

unable build (5731cab)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/keq-request/keq/releases/tag/v2.0.5 :sunflower:


(DO NOT MERGE) fix(redis): Moving read message logic to lua

Moving locks & read message behavior into lua.
We're experiencing race conditions in higher message volumes that is exacerbated by more aggressive polling and more orca instances. We've gotten to the point in Orca where Keiko isn't able to pull messages off of the queue fast enough to saturate a larger group of Orca worker pools, and this lock contention issue causes performance to regress and slow Spinnaker down.
This moves all read message logic into lua scripts, so that lock and reads are handled as an atomic unit.

Early PR because I'm having some trouble tracking down why message attempts (and thus dead lettering) isn't working. I'll be working on moving retry logic into lua as well, but I'm kind of confused by seemingly unrelated test failures.


feat(extras): add snakeskin extra

Description
Added snakeskin template language extra. Snakeskin template language developed by Andrey Kobets and maintained by v4fire framework community.
Screenshots

Checklist

[x] I've read the CONTRIBUTING guidelines.

@folke could you advice why tests are failed?


Modus Tooltip gets cut when used inside an element with overflow:hidden

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS 16]
Browser [e.g. Chrome, Safari]

Additional context
Add any other context about the problem here.

This will be fixed in #1445


CRC 1.27 fails to start VM - Unable to determine VM's IP address

General information

OS: Linux
Hypervisor: KVM
Did you run crc setup before starting it (Yes/No)? Yes
Running CRC on:  VM

CRC version
# Put `crc version` output here
CodeReady Containers version: 1.27.0+3d6bc39d
OpenShift version: 4.7.11 (embedded in executable)

## CRC config
# Put `crc config view` output here
- consent-telemetry                     : yes

## Host Operating System
NAME="Red Hat Enterprise Linux"
VERSION="8.4 (Ootpa)"
ID="rhel"
ID_LIKE="fedora"
VERSION_ID="8.4"
PLATFORM_ID="platform:el8"
PRETTY_NAME="Red Hat Enterprise Linux 8.4 (Ootpa)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:redhat:enterprise_linux:8.4:GA"
HOME_URL="https://www.redhat.com/"
DOCUMENTATION_URL="https://access.redhat.com/documentation/red_hat_enterprise_linux/8/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"

REDHAT_BUGZILLA_PRODUCT="Red Hat Enterprise Linux 8"
REDHAT_BUGZILLA_PRODUCT_VERSION=8.4
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux"
REDHAT_SUPPORT_PRODUCT_VERSION="8.4"

### Steps to reproduce

  1. crc setup
  2. crc start

### Expected
State running without errors

### Actual
State running with errors

### Logs
INFO Starting CodeReady Containers VM for OpenShift 4.7.11...
DEBU Updating CRC VM configuration
DEBU (crc) Calling .GetConfigRaw
DEBU (crc) Calling .Start
DEBU (crc) DBG | time="2021-06-02T15:43:55-03:00" level=debug msg="Starting VM crc"
DEBU (crc) DBG | time="2021-06-02T15:43:55-03:00" level=debug msg="Validating network"
DEBU (crc) DBG | time="2021-06-02T15:43:55-03:00" level=debug msg="Validating storage pool"
DEBU (crc) DBG | time="2021-06-02T15:44:00-03:00" level=debug msg="GetIP called for crc"
DEBU (crc) DBG | time="2021-06-02T15:44:00-03:00" level=debug msg="Getting current state..."
DEBU (crc) DBG | time="2021-06-02T15:44:00-03:00" level=debug msg="Waiting for machine to come up 0/60"
DEBU (crc) DBG | time="2021-06-02T15:44:03-03:00" level=debug msg="GetIP called for crc"
DEBU (crc) DBG | time="2021-06-02T15:44:03-03:00" level=debug msg="Getting current state..."
DEBU (crc) DBG | time="2021-06-02T15:44:03-03:00" level=debug msg="Waiting for machine to come up 1/60"
DEBU (crc) DBG | time="2021-06-02T15:44:06-03:00" level=debug msg="GetIP called for crc"
DEBU (crc) DBG | time="2021-06-02T15:44:06-03:00" level=debug msg="Getting current state..."
.
.
DEBU (crc) DBG | time="2021-06-02T15:46:57-03:00" level=debug msg="Waiting for machine to come up 59/60"
DEBU (crc) DBG | time="2021-06-02T15:47:00-03:00" level=warning msg="Unable to determine VM's IP address, did it fail to boot?"
DEBU Making call to close driver server
DEBU (crc) Calling .Close
DEBU Successfully made call to close driver server
DEBU Making call to close connection to plugin binary
DEBU (crc) DBG | time="2021-06-02T15:47:00-03:00" level=debug msg="Closing plugin on server side"
Error starting machine: Error in driver during machine start: Unable to determine VM's IP address, did it fail to boot?

Before gather the logs try following if that fix your issue
$ crc delete -f
$ crc cleanup
$ crc setup
$ crc start --log-level debug

Please consider posting the output of `crc start --log-level debug`  on http://gist.github.com/ and post the link in the issue.

Can you try running with the user network mode?
https://github.com/code-ready/crc/wiki/VPN-support--with-an--userland-network-stack

Check https://github.com/code-ready/crc/issues/2214 if you have recently updated or have docker clients.

Can you try running with the user network mode? Yes, I performed  the steps for Linux and it did not work. (log attached)
https://github.com/code-ready/crc/wiki/VPN-support--with-an--userland-network-stack

crc-start.log
crc-deamon.log

@jmajulian21 This looks like vm is  failed to start, not sure why. Can you check virsh -c qemu:///system list after crc start failed? And also look in journalctl -u libvirtd, and in /var/lib/libvirt/qemu/crc.log

@jmajulian21 This looks like vm is failed to start, not sure why. Can you check virsh -c qemu:///system list after crc start failed? And also look in journalctl -u libvirtd, and in /var/lib/libvirt/qemu/crc.log

virsh -c qemu:///system list

journalctl -u libvirtd

"/var/lib/libvirt/qemu/crc.log": The file does not exists

Same problem here, cant find a way to workaround this.

Do you know another way to make a locally raised openshift? with CetOs or with windows? I need to know urgently :'(

@jmajulian21 try with a centos vagrant box. I managed to execute the setup and start commands successfully but did didn't work as expected. Next week i might keep debugging this.

@jmajulian21 I can see you are running it on a VM can you tell us what is the base OS and hypervisor? In the past we did see some issue with CRC when users tried to run it in nested  https://github.com/code-ready/crc/issues/2082

@praveenkumar i guess so. Tryed using arch but libvirt is not recognized by the binary even when  im sure that it is installed. Then tryed use a centos vagrant box since its a supported distro but faced the problem described in this issue.

@praveenkumar i guess so. Tryed using arch but libvirt is not recognized by the binary even when im sure that it is installed. Then tryed use a centos vagrant box since its a supported distro but faced the problem described in this issue.

@Xplouder centos vagrant box directly on the host which have arch?

@praveenkumar yes, Manjaro to be more precise.

Hi, any workaround? Thanks.

@johnwmail This is a closed issue, can you create a new issue using template and provide us more info around it?


Update package version to 1.11.0

Codecov Report

Merging #487 (fa55394) into master (c377b82) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #487   +/-   ##
=======================================
  Coverage   98.06%   98.06%
=======================================
  Files          22       22
  Lines        2532     2532
=======================================
  Hits         2483     2483
  Misses         49       49

Impacted Files
Coverage Δ

robocop/version.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


index.html: Typo?!?!

Make use of correct preposition (on instead of by)
GrammarNazi
By should be used when OVH is hosting it "by" themselves
But you guys are hosting it so...

Thanks for this pull request :)
I used by because previously the API was hosted by Bothosting.
I'd say that on is a much better alternative.

haha alright :dancer: Thanks for all the great work.


Helm & Jenkins integration

Helm & Jenkins integration + other things...

Coverage decreased (-25.0%) to 55.882% when pulling 854c455bebf20c28c8dacae5817aad779241720d on deploy-on-push/k8s/helm-migration into fa121a3a182e4ac233cdeba18c05d4443a03ac73 on master.

Coverage decreased (-25.0%) to 55.882% when pulling c41de200ce0ac338518c1cc8017d06d84609781e on deploy-on-push/k8s/helm-migration into fa121a3a182e4ac233cdeba18c05d4443a03ac73 on master.


Added missing deprecated doctag for mysql_num_rows

Missing @deprecated 5.5 PhpDoc tag for mysql_num_rows
http://php.net/manual/en/function.mysql-num-rows.php

@neuro159 I don't understand. Every other phpdoc for mysql functions has it above the description...


DOC: setup Read The Docs

It's becoming increasingly clear that the scope of documentation for Onyo would be better suited for something like Read The Docs rather than a single README.
See also #125 , #119 , #105 , #20 , #17 , and the templating/validation system.

Done! There's a link to the docs in the README via the RTD bagde.


Very very poor performance on MudSelect in a MudTable with 30+ rows and 70+ options

Bug type
Component
Component name
MudSelect
What happened?
Expanding and collapsing a dropdown with more than a few items has a massive delay.
Expected behavior
Dropdown expands in a reasonable time frame (almost instantly).
Reproduction link
https://try.mudblazor.com/snippet/mkQbYtQKyEHDEDmC
Reproduction steps
See reproduction link.
This appears to be an issue with the amount of rendering load that MudBlazor is adding with each MudSelect on the screen and with each MudSelectItem being rendered. As it is, it is unusable for the relatively common workload of an editable table.
Relevant log output
No response
Version (bug)
5.1.4
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

I had the same issue started the use the following and i saw great performance
https://docs.microsoft.com/en-us/aspnet/core/blazor/components/virtualization?view=aspnetcore-5.0

Tested it with the new Popover and scraped portal, PR: https://github.com/MudBlazor/MudBlazor/pull/2833
https://user-images.githubusercontent.com/10367109/134819744-c8cf57dc-be3e-44c7-af42-493eda32235a.mp4

@ekjuanrejon I'm already using pagination to limit the table to 30 lines per page.
That's much better! Might also fix some of the errors raised in my other issues. Feel free to close this issue.

Duplicate of #2301


Cannot read properties of undefined (reading 'length')

Summary
Receiving "Cannot read properties of undefined (reading 'length')" error when running command
sfdx force:org:shape:list
Steps To Reproduce:
Run from a SFDX Project
sfdx force:org:shape:list
System Information
Running Mac M1 Pro with the following version of sfdx-cli. I already uninstalled all plugins per other similar github issues:
{   "cliVersion": "sfdx-cli/7.165.0",   "architecture": "darwin-x64",   "nodeVersion": "node-v16.17.0",   "pluginVersions": [     "@oclif/plugin-autocomplete 1.3.0 (core)",     "@oclif/plugin-commands 2.2.0 (core)",     "@oclif/plugin-help 5.1.12 (core)",     "@oclif/plugin-not-found 2.3.1 (core)",     "@oclif/plugin-plugins 2.1.0 (core)",     "@oclif/plugin-update 3.0.0 (core)",     "@oclif/plugin-version 1.1.2 (core)",     "@oclif/plugin-warn-if-update-available 2.0.4 (core)",     "@oclif/plugin-which 2.1.0 (core)",     "alias 2.1.0 (core)",     "apex 1.2.0 (core)",     "auth 2.2.3 (core)",     "community 2.0.0 (core)",     "config 1.4.17 (core)",     "custom-metadata 2.0.0 (core)",     "data 2.1.2 (core)",     "generator 2.0.2 (core)",     "info 2.0.1 (core)",     "limits 2.0.1 (core)",     "org 2.2.0 (core)",     "schema 2.1.1 (core)",     "signups 1.2.0 (core)",     "source 2.0.12 (core)",     "telemetry 2.0.0 (core)",     "templates 55.1.0 (core)",     "trust 2.0.3 (core)",     "user 2.1.0 (core)",     "@salesforce/sfdx-plugin-lwc-test 1.0.0 (core)",     "salesforce-alm 54.8.1 (core)"   ],   "osVersion": "Darwin 21.5.0",   "shell": "bash",   "rootPath": "/Users/samirawuapara/.local/share/sfdx/client/7.165.0-65b4bb0" }
Feel free to attach a screenshot.

Hey @sawuapara, thanks for reporting this issue. Unfortunately I am not able to reproduce this. I've tried the following with sfdx from the Installer and npm. I am using version 7.165.0 with npm: '8.15.0' and  node: '16.17.0'

sfdx force:org:shape:list (with no shapes) -> No org shapes found.
sfdx force:org:shape:create -u myDevHub -> shape creates
sfdx force:org:shape:list -> lists created shape
sfdx force:org:shape:delete -> deletes created shape
sfdx force:org:shape:list -> No org shapes found.

Can you provide any additional details on how to reproduced this issue? Thanks

Re-authenticating resolved issue for me. Thank you.


[1/2] Refactor assert functions in class

In the close future, we'll have to write smarter assert functions with finer rules.
Example: this rule was not trivial to express with the previous representation:

If a distribution is a FileObject, then it should define contentUrl and encodingFormat.
Else if a distribution is a FileSet, then it should define includes and encodingFormat.

The PR allows to write such rules more elegantly in a more concise way.

Thanks for the review, @pierrot0!


force infer storage on backward pass (#1)

This PR allows inferstoragetype in backward pass. For type and shape, we can skip backward inter pass because it's always identical, but it's not the case for storage type (e.g. For sparse embedding operator, we'll output RowSparse weight gradients).

@piiswrong merging this helps me enable some CI tests on PR https://github.com/dmlc/mxnet/pull/5800

test failed

@piiswrong updated unit test


Add support for multiple root folders

Currently, OmniSharp requires a single root folder or solution to be specified at start up with the -s command line switch. Soon, VS Code will provide support for multiple root paths. To account for this, it should be possible to launch OmniSharp without setting -s and then use end points to add and remove root paths that project systems would respond to.

For reference, here is the issue tracking the new API on the VS Code side: https://github.com/Microsoft/vscode/issues/28526.

Any progress on this front? On a moderately complex repo that has several sln files, it's annoying to have to open up each sub-folder in the file tree to get Omnisharp working correctly.

Any idea when this will be addressed?

Any idea when this will be addressed?

There are currently no definite plans.

There are currently no definite plans.

Internal Tracking
devdivcsef 412378

Nearly 4 years on from when this issue was opened, is the advice in 2021 to structure folders so that you can open one vs code instance (and therefore one omnisharp instance) per solution?

This really needs to be fixed at some point...

Please.

There are currently no definite plans.

Why not? Your process seems broken.

+1


Release/dev

files ok

ok

@bablubharath closing this PR. In case you want to include changes please consider the suggestions in this thread! Thanks!


middleware/cache minimum cache TTL is now 0 should be 5?

Setting "cache" before had the effect, that records could be cached using the record TTL with a max of 3600 or 1800. This seems to now default to 0.
"cache" or "cache 0" result in all TTLs on answers being 0.
This should at least be the minimum of 5 or else it might break other DNS software not allowing TTLs under 5s.

[ Quoting<EMAIL_ADDRESS>in "[miekg/coredns] middleware/cache mi..." ]

Setting "cache" before had the effect, that records could be cached using the record TTL with a max of 3600 or 1800. This seems to now default to 0.

I'm not seeing this.

"cache" or "cache 0" result in all TTLs on answers being 0.
This should at least be the minimum of 5 or else it might break other DNS software not allowing TTLs under 5s.

Yes, that could be bad.
/Miek
--
Miek Gieben

It seems this was another going down the rabbit hole of strange usage to really get the bugs out.
So I have a zonefile for example.com and a symlink to example.com from example.net.
With my config this basically serves example.com entries at example.net.
Example.com queries are fine and have the right TTL.
Example.net queries are fine except the TTL is always 0.
This works fine without the cache directive and with the auto directive, but it seems it falls over with setting the cache to "cache" or "cache 0".
Sidenote: This is done due to the issue with using the rewrite middleware and chromium based browsers. #383

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/coredns] middleware/cach..." ]

It seems this was another going down the rabbit hole of strange usage to really get the bugs out.
So I have a zonefile for example.com and a symlink to example.com from example.net.
With my config this basically serves example.com entries at example.net.
Example.com queries are fine and have the right TTL.
Example.net queries are fine except the TTL is always 0.
This works fine without the cache directive and with the auto directive, but it seems it falls over with setting the cache to "cache" or "cache 0".

Yes, "cache 0" should be doing something more sensible than setting TTLs to 0,
using the minTTL of the message seems more sensible (i.e. no max).
With just using "cache" I can't get this "ttl 0" behavior btw.
/Miek
--
Miek Gieben


Show Already Attached Vols

This patch updates the behavior of the volume attach command. If a volume is provided that is already attached, it will simply be emitted in the output as if it were being attached for the very first time, effectively making this command idempotent.
Because @cduchesne said "pretty-please" and put a cherry on top :)

Current coverage is 8.27% (diff: 0.00%)

Merging #651 into release/0.6.3 will decrease coverage by 0.01%

@@           release/0.6.3      #651   diff @@
==============================================
  Files                 16        16
  Lines               2196      2199     +3
  Methods                0         0
  Messages               0         0
  Branches               0         0
==============================================
  Hits                 182       182
- Misses              2007      2010     +3
  Partials               7         7

Powered by Codecov. Last update 4bd8304...ea62e7a


update to Julia 1.0

Hello, is there any current plan in updating this package to Julia 1.0?

No immediate plans. But I'll be happy to review PRs.

Ok I will submit one soon, thanks!

closed by #4


Add prometheus exporter to kafka (as side-car container)

Is this still needed? can this be closed ?


don't hardcode clear values

Ok this is a proposal to avoid hardcoding the buffer clearing in the engine (currently with color 0.2, 0.2, 0.2, 1.0).
It's quite verbose as is, I'm not sure what you want to be enforced by the engine. Typically I've never used stencil.
but being able to choose the clearing color and whether you clear color and/or depth buffers seems important.

Haha, that clear code i was just copy and paste from other place, and never touch it again. So thanks alot and look good for me.


handle different unity versions

especially when start to remove packages.. or other recent features.

now tested on 2017.4 and later (package removal currently only works on 2020.3 and later, due to missing newtonsoftjson earlier)


Inform about ST3's feature

I would suggest adding the information to the repo description too.

Thanks! :smile:

Thanks! :smile:


AE2 Visual Bug

When I put item in slot such as ME crafting terminal's crafting slot the item amount appear twice (The big number overlap the small one):

The same happened to the ME (Extended Processing) Pattern Terminal (both crafting and processing mode), Quartz Grindstone, Sky Stone (Block) Chest, ME Interface:

When I create new instance and install AE2 and its addon, the bug doesn't occur:

At first I though the bug appears because of some additional mods I installed, but when I download another modpack instance, the bug still appears (and I can't find the mod that cause the problem)

I have the exact same issue. Thought it was because of my modifications but appaerently it isn't.

That looks super annoying. Would you mind testing if updating AE extended life works to fix it?

Unfortunately, updating AE2 and its addon don't fix the problem. I remember the first time I saw the bug is in v3.2.1 when I use the Quartz Grindstone, but I thought its from AE2 and don't think much

I just found out that the encoded pattern also has this problem. The ME Interface and ME Interface Terminal show the texture of the encoded pattern, not the output item

Ok, I figured out what causing this, it's Arcane Archive
To fix this, open the config menu of Arcane Archive and set Disable non-mod slot highlighting to false


[typespec-next] React to azure/typespec-azure#3391

Reacts to azure/typespec-azure#3391

@timotheeguerin: Can you please confirm all 8 changed files are expected based on https://github.com/Azure/typespec-azure/pull/3391?

@markcowl, @johanste: Can you comment on the failures of these checks?
Swagger ModelValidation is failing with Expected type but found type object.  I would not expect the check to fail on swagger generated from valid TypeSpec.  Is TypeSpec generating invalid Swagger, or is the check wrong?
Swagger BreakingChange is failing due to the new additionalProperties element.  Is the check being too aggressive, and this should not be considered a "breaking change"?


Updated tensorflow-datasets to 1.3.0

https://github.ibm.com/mldlppc/tracker/issues/12317

@jayfurmanek - Please review this one too.


Visual Studio Python環境について

This issue has been moved from a ticket on Developer Community.

Python環境について
PythonのAnacondaがPython3.9.12にアップグレードしたことに際し、通常の公式pythonを3.9,3.10のみ残して3.8はアンインストールしました。
しかしVisual StudioのPython環境というところにはPython3.8が線で消えていますが表示としては残っています
完全に3.8を表示しない方法はありますか

Original Comments
Feedback Bot on 6/5/2022, 08:17 PM:
(private comment, text removed)

Original Solutions
(no solutions)

Translation:
About the Python environment
When Anaconda of Python upgraded to Python 3.9.12, we uninstalled 3.8 leaving only the usual official python 3.9 and 3.10.
However, in the Python environment of Visual Studio, Python 3.8 is disappeared with a line, but it remains as a display.
Is there a way not to show 3.8 completely

Hi, sorry for the issue you are experiencing, but I was not able to repro what you saw. The environment is shown as "missing" after I manually remove it, and the missing environments can be removed by right click -> remove. Could you provide us a more detailed repro steps or a screenshot/gif of the issue? That would be really helpful. Thanks. 😊

Going to close this out. If the issue still persists, please reopen with the information requested. Thanks.


Version bump

Changes since release 0.43.2:

Improve ToolCollector to differentiate crashes and user_errors
[fastlane_core] Updated changelog loader to follow redirects (#4582)

👍  💎


Feature/4.4/centos7

Got an internal request to package 4.4 for centos7, hence...

against wrong branch :(


Competitive programming question

Task completed ! Please confirm if any more changes are needed.
On Fri, Oct 2, 2020, 11:57 PM Chandrika Deb<EMAIL_ADDRESS>wrote:

@chandrikadeb7 commented on this pull request.
In Task 1/Time conversion/Question.md
https://github.com/chandrikadeb7/Hacktoberfest_2020/pull/22#discussion_r498983659
:

@@ -0,0 +1,19 @@
+Given a time in 12-hour AM/PM format, in a string, convert it to military (24-hour format) time.

Add headings and subheading in markdown format
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chandrikadeb7/Hacktoberfest_2020/pull/22#pullrequestreview-501357159,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARG5L7L4RDFN7G77KTNBJGDSIYLQFANCNFSM4SB4WPLQ
.

Hey @chandrikadeb7  Can you please accept my contribution.  I am participating in Hacktoberfest2020. Please merge this file with hacktoberfest-accepted label.


Add Anchor.toml setting to specify the output directory of all typescript types in target/types/

This will speed up UI development

Do you want to make it as a CLI argument also?
I tried setting the idl field in the Anchor.toml however it seems like it doesn't produce the IDL to the expected path.
But setting it explicitly using CLI does produce the output to the expected directory. Reading from the code, it seems like the CLI only reads the command line input when build?
Anchor.toml as followed:
[programs.localnet]
wrapper = { address="Fg6PaFpoGXkYsidMpWTK6W2BeZ7FEfcYkg476zPFsLnS", idl = "~/Projects/wrapper/new_idl" }

We could add a new types to [workspace], for example,
[workspace]
types = "target/types"

I'm working on it 🙂

Resolved with #1032

Hi 👋
Thanks for this awesome feature! I just wanted to point out that the current implementation will not include the metadata object containing the program ID.
Running anchor build generates a new IDL without a metadata object and running anchor deploy later on will update that IDL file to add the metadata object.
Whilst this is the case with the IDL file generated inside the target folder, it is not the case for the IDL copied in the custom directory.
I'd happily do a PR for this but my rust is very... rusty... 🥁

hi @lorisleiva,
The IDL copied does not come target/idl/<idl>.json which contains the ID program but target/types/<idl>.ts I think it would be necessary to add the metadata in target/types/<idl>.ts file


Failed publication of more msgs

Hello, I'm experiencing some issues while trying to publish on the OCB two messages: basically I noticed that a message ("RobotAgentProperties") had never been published on the OCB, even if the rostopic echo on the topic ("/robot_opil_v2/description_channel") printed properly the message. This message contains a UUID, as does another message ("ActualState") on another topic ("/robot_opil_v2/status_channel").
To isolate and investigate I killed my node and I manually published the messages through some tests.
First test
After starting OCB, roscore and firos, I made this rostopic pub:
rostopic pub /robot_opil_v2/description_channel mars_agent_physical_robot_msgs/RobotAgentProperties "robot_id: {uuid: "[251,61,117,161,168,37,80,255,179,230,228,178,188,174,72,46]", description: ''}
type: {vehicle_type: 0}
footprint:
  header:
    seq: 0
    stamp: {secs: 0, nsecs: 0}
    frame_id: ''
  polygon:
    points:
    - {x: 0.0, y: 0.0, z: 0.0}
min_height: 0.0
max_height: 0.0
payload: 0.0
max_pos_x_vel: 0.0
max_neg_x_vel: 0.0
max_pos_x_acc: 0.0
max_neg_x_acc: 0.0
max_pos_y_vel: 0.0
max_neg_y_vel: 0.0
max_pos_y_acc: 0.0
max_neg_y_acc: 0.0
max_pos_ang_vel: 0.0
max_neg_ang_vel: 0.0
max_pos_ang_acc: 0.0
max_neg_ang_acc: 0.0
velocity_control_sensitivity: 0.0
min_turning_radius: 0.0
batt_capacity: 0.0
batt_max_voltage: 0.0
weight: 0.0
vendor: ''
action_capability:
- category: 0
  action: 0
  attributes:
  - {type: '', name: '', value: ''}
  description: ''"
publishing and latching message. Press ctrl-C to terminate

Checked the entyty on the OCB through GET localhost:1026/v2/entities/ and the related entity was published.
So, I runned another rostopic pub with a second message, as it follows:
rostopic pub /robot_opil_v2/status_channel mars_agent_physical_robot_msgs/ActualState "header:
  seq: 0
  stamp:
    secs: 0
    nsecs: 0
  frame_id: ''
robot_id:
  uuid: "[251,61,117,161,168,37,80,255,179,230,228,178,188,174,72,46]"
  description: ''
MoveBaseSimpleState: 0"
publishing and latching message. Press ctrl-C to terminate

Firos shell showed me this message:
[ERROR] [1562341387.645877]: bad callback: <function _publishToCBRoutine at 0x7f9a1e662848>
Traceback (most recent call last):
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/rospy/topics.py", line 748, in _invoke_callback
    cb(msg, cb_args)
  File "/home/walter/OPIL/mod.iot.ran/src/firos/firos/include/ros/topicHandler.py", line 158, in _publishToCBRoutine
    CloudPublisher.publishToCB(robot, topic, data, ROS_TOPIC_AS_DICT[topic])
  File "/home/walter/OPIL/mod.iot.ran/src/firos/firos/include/contextbroker/cbPublisher.py", line 99, in publishToCB
    completeJsonStr = ObjectFiwareConverter.obj2Fiware(self.posted_history[robotID], ind=0, dataTypeDict=definitionDict,  ignorePythonMetaData=True)
  File "/home/walter/OPIL/mod.iot.ran/src/firos/firos/include/FiwareObjectConverter/objectFiwareConverter.py", line 46, in obj2Fiware
    return clsself._json(en, ind)
  File "/home/walter/OPIL/mod.iot.ran/src/firos/firos/include/FiwareObjectConverter/objectFiwareConverter.py", line 68, in _json
    return json.dumps(obj.__dict__, default=clsself._complex_handler, indent=ind)
  File "/usr/lib/python2.7/json/__init__.py", line 251, in dumps
    sort_keys=sort_keys, **kw).encode(obj)
  File "/usr/lib/python2.7/json/encoder.py", line 209, in encode
    chunks = list(chunks)
  File "/usr/lib/python2.7/json/encoder.py", line 434, in _iterencode
    for chunk in _iterencode_dict(o, _current_indent_level):
  File "/usr/lib/python2.7/json/encoder.py", line 408, in _iterencode_dict
    for chunk in chunks:
  File "/usr/lib/python2.7/json/encoder.py", line 443, in _iterencode
    for chunk in _iterencode(o, _current_indent_level):
  File "/usr/lib/python2.7/json/encoder.py", line 434, in _iterencode
    for chunk in _iterencode_dict(o, _current_indent_level):
  File "/usr/lib/python2.7/json/encoder.py", line 408, in _iterencode_dict
    for chunk in chunks:
  File "/usr/lib/python2.7/json/encoder.py", line 408, in _iterencode_dict
    for chunk in chunks:
  File "/usr/lib/python2.7/json/encoder.py", line 443, in _iterencode
    for chunk in _iterencode(o, _current_indent_level):
  File "/usr/lib/python2.7/json/encoder.py", line 434, in _iterencode
    for chunk in _iterencode_dict(o, _current_indent_level):
  File "/usr/lib/python2.7/json/encoder.py", line 408, in _iterencode_dict
    for chunk in chunks:
  File "/usr/lib/python2.7/json/encoder.py", line 408, in _iterencode_dict
    for chunk in chunks:
  File "/usr/lib/python2.7/json/encoder.py", line 443, in _iterencode
    for chunk in _iterencode(o, _current_indent_level):
  File "/usr/lib/python2.7/json/encoder.py", line 434, in _iterencode
    for chunk in _iterencode_dict(o, _current_indent_level):
  File "/usr/lib/python2.7/json/encoder.py", line 390, in _iterencode_dict
    yield _encoder(value)
UnicodeDecodeError: 'utf8' codec can't decode byte 0xfb in position 0: invalid start byte

The entity has not been generated on the OCB and with the usual GET localhost:1026/v2/entities/ there was only the "status_channel".
Second test
Same as before, but I published first the "status_channel" message, then the "description_channel" one. Same error printed on the shell. This time the only entity on the OCB was the "status_channel" one.
Third test
Removed the UUID from the "description_channel" message, then published through rostopic pub the "status_channel" message, then published the new "description_channel" one.
Same outputs of Second test.
Fourth test
Firstly published the "description_channel" message (without the UUID), then the "status_channel" one (with the UUID). Everything correctly published.
Maybe this behaviour has something to do with the UUID conversion, but I can't exclude this is somehow related to my workspace. On the ROS side, everything looks fine. I attach you the messages, the bag files and the GET responses of the first two tests.
Thank you for your time.
Tests.zip

Hi,
I cannot reproduce the errors from the first and second test. I tested them also via rostopic pub and needed to change the uuid (because rostopic pub strangely suggests using an string instead of an  array of bytes) to [251,61,117,161,168,37,80,255,179,230,228,178,188,174,72,46]
Did you notice, that you use " in your rostopic pub to set the uuid? Could this cause the error?
A call should look like follows:
rostopic pub /robot_opil_v2/status_channel mars_agent_physical_robot_msgs/ActualState "header:
  seq: 0
  stamp:
    secs: 0
    nsecs: 0
  frame_id: ''
robot_id:
  uuid: [251,61,117,161,168,37,80,255,179,230,228,178,188,174,72,46]
  description: ''
MoveBaseSimpleState: 0"

I did not tried your third or fourth test. But it seems that the uuid can only be converted once in firos independent of the Message.
Strangely i can either publish ActualState or RobotAgentProperties at first with no issues. But publishing the next Message  causes the error you mentioned above.

I also tracked this strange behaviour down and noticed a simple error of the implementation in cbPublisher.py. Here we are not sending all information we can retrieve to the conversion part of firos, which leads to this error.
The fix will be avialable later today. Thanks for mentioning the error!
I


[INTERNAL] Sample settings dialog: emphasize primary action

This change is required to align with Fiori Design Guidelines (For descriptions, see https://github.com/SAP/openui5/issues/2254)

forwarded to dev team #1980358964

Hi @boghyon ,
Sorry for the late response, can you rebase your change so we can approve it?
Regards,
Nayden

@nnaydenow I just rebased my branch to this upstream master.


WIP dfn.js-style reference links

This is running into styling troubles trying to get the popup to correctly size itself and appear in the right place and toggle on/off. Since I hear you are making changes to related style areas anyway, i.e. the .utils class, I thought instead I'd just upload what I have. Most of the build-side code is ready for review, although the client-side code is not.
On top of #71 since I use some new jsdom features (classList). Best to review the last commit only.

Also, this only works on things with .utils descendants right now, i.e. emu-clauses. I think we should make it work for anything with an ID. There are a few approaches to this, but I think the best is to add a span.utils in an appropriate place for everything that has an ID. dfns, emu-tables inside their caption, and anything else we can think of.

Find references is implemented in 3.10.


Use a Pod for SSHing into nodes

When starting an ephemeral debug container with kubectl, its preferred to start a Pod rather than a Deployment. Wen kubectl run is invoked with --restart=Never it provisions a Pod and not a Deployment, this can also save the extra step to get the pod id, since the given name will be used as the exact pod name.

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

Hi, @bergerx. Thanks for the suggestion. You certainly can create the pod directly like this, however the recommendation from engineering is that you shouldn't create long-running pods with SSH access to the nodes. You still need kubectl access to reconnect the terminal session anyway. You're certainly welcome to do so, there's nothing necessarily wrong with that, but we'd prefer to leave the doc as it is.
That said, I do have a doc update due to go live tomorrow around this SSH piece which hopefully cleans up the steps a little. The default restart policy is left on the pod creation though.
For now, #please-close


socks5 vs authentication support

In Playwright docs I found that it does not support Socks5 with authorization.
https://github.com/microsoft/playwright/issues/2425#issuecomment-775310736
But in your examples you show one: https://github.com/CheshireCaat/browser-with-fingerprints#proxy-usage
Is your plugin adds support of proxy5 with authentication?

Yes, all plugins use custom logic for proxy support. You should use plugin methods instead of built-in playwright options.
In such a case, you can use a proxy with authorization.


Feature: Any plan to support Embeddings?

Embeddings also is a common api to build services, any plan for that? Thanks, :)

Hey, thanks for taking interest in the project. I am sorry your issue kind of went under my radar. I plan to add more endpoints soon, so I might add Embeddings. However feel free to make contributions.

Added in https://github.com/LevitatingBusinessMan/openai-rust/commit/50fc6a04de667b7c8954472fe1e84ee4d9ad3f8a


Adding rhel ai version

Adding rhel ai version
Set github hash by defautl as image version.
Add RHEL_AI_VERSION into /etc/os-release in order to use it in insights
NAME="Red Hat Enterprise Linux"
VERSION="9.20240630.0.4 (Plow)"
ID="rhel"
ID_LIKE="fedora"
VERSION_ID="9.4"
PLATFORM_ID="platform:el9"
PRETTY_NAME="Red Hat Enterprise Linux 9.20240630.0.4 (Plow)"
ANSI_COLOR="0;31"
LOGO="fedora-logo-icon"
CPE_NAME="cpe:/o:redhat:enterprise_linux:9::baseos"
HOME_URL="https://www.redhat.com/"
DOCUMENTATION_URL="https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/9"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Red Hat Enterprise Linux 9"
REDHAT_BUGZILLA_PRODUCT_VERSION=9.4
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux"
REDHAT_SUPPORT_PRODUCT_VERSION="9.4"
OSTREE_VERSION='9.20240630.0'
RHEL_AI_VERSION='4538df1'

This strikes me as a change we should add downstream rather than here in the upstream, but not entirely sure.

This strikes me as a change we should add downstream as a new containerfile or a patch rather than here in the upstream, but not entirely sure. cc @fabiendupont @lmilbaum

Just found your comment in slack, the above is resolved by this but going to post it here for posterity:

We should set real version in downstream build, just wanted to have some default

The OS can be something else than RHEL, so not sure that RHEL in the variable name is relevant.
/cc @romfreiman @n1hility

@fabiendupont do we have some more generic name for it?
I mean i can change to make it ${OS_ID}_AI_VERSION or just something hardcoded that will match better, just don't know what.

@Gregory-Pereira i think setting git hash in upstream or provide the way for dev to setup their own versioning can be helpful too and in downstream we will set release versions

I agree this should be done downstream, not upstream.

Currently downstream uses upstream containerfile. I added verification that it is rhel image in order to set this field only for rhel

/hold till we will decide which exact var we should change in /etc/os-release

/hold

/unhold
Seems like we have decision

Joining late to the party. What does this variable used for? If we agree that it is needed, should it be populated via a builds arg file?


Strange error message with features on dependencies

Problem
cargo build produces a bizarre error message in a workspace with many generated internal dependencies, with features that activate features on their internal dependencies.
I've actually had this issue for a long time (a year?), but my prior attempts to minimize the code ended up causing the bug to disappear!  Today I finally succeeded by making a very conservative minimization.
Steps

Download: cargo-strange-error.tar.gz.
The minimize branch will already be checked out.  Run cargo build:

error: failed to select a version for `rsp2-array-types`.
    ... required by package `rsp2 v0.1.0 (/home/lampam/cpp/throwaway/cargo-strange-error)`
versions that meet the requirements `= 0.1.0` are: 0.1.0

the package `rsp2` depends on `rsp2-array-types`, with features: `rsp2-assert-close` but `rsp2-array-types` does not have these features.

Observe that rsp2-assert-close never appears as a feature name in any Cargo.toml file.  The error message is clearly nonsense!

rg '["/]rsp2-assert-close"'

Bonus round!
Here is an additional thing you can do that may help debug the issue:

Additional repro case
Now try checking out the master branch.  This is the code prior to minimization.  Observe the following:

On this branch, cargo build successfully begins building.  (Just ctrl-c it; it will take a long time)
...but now cargo build --features=nightly produces the exact same error message as above!

The only differences between minimize and master  that I would imagine should affect cargo are the removal of build scripts, binaries, integration tests, and the changes listed here (which are mostly just removing external dependencies):
git diff master..minimize -- $(find . -name Cargo.toml)

Notes
The Cargo.toml files in this workspace contain some intimidating-looking [features] that look like this:
nightly = ["beta", "rsp2-array-types/nightly", "rsp2-array-utils/nightly", "rsp2-assert-close/nightly", "rsp2-minimize/nightly", "rsp2-newtype-indices/nightly", "rsp2-structure-io/nightly", "rsp2-structure/nightly", "rsp2-util-macros/nightly"]
beta = ["rsp2-array-types/beta", "rsp2-array-utils/beta", "rsp2-assert-close/beta", "rsp2-minimize/beta", "rsp2-newtype-indices/beta", "rsp2-structure-io/beta", "rsp2-structure/beta", "rsp2-util-macros/beta"]

These lines are programmatically generated.  beta activates the /beta feature of every path dependency.  nightly activates beta and the /nightly feature of every path dependency.
Cargo version
Tested on:
cargo 1.32.0 (8610973aa 2019-01-02)
cargo 1.36.0-nightly (759b6161a 2019-05-06)

I think this is the same as #6915.  There's something weird about passing a feature through to a dev-dependency.

Yep.  I tried moving all of my dev-dependencies to regular dependencies and the error vanishes.
(well, a different error appears, but one that is correct;  "rsp2-structure" mistakenly activates "rsp2-array-types/serde" instead of "rsp2-array-types/serde-support")


CI fails because -Zscript now needs a hashbang

I think the nightly chains now require the following to run?
#!/usr/bin/env -S cargo +nightly -Zscript

Not sure, but CI is failing:
https://github.com/opendp/opendp/actions/runs/9020660022/job/24786298591?pr=1597

For reference, the error begins with:
Run cargo +nightly -Zscript getting-started/code/quickstart-framework.rs
error: unknown start of token: `
 --> quickstart-framework.rs:2:1
  |
2 | ```cargo
  | ^^^
  |
  = note: character appears 2 more times

So my assumption is that something changed on nightly, but I haven't found a change log about it. The example in the docs hasn't changed, but it fails the same way locally.
Some options:

Get the existing script working. Next step might be to ask a question either on github or reddit. But nightly is going to be unstable, and maybe the lesson is not to rely on it.
Can we pin to a particular nightly version? Maybe, but seems involved.
Or we can change this to a standalone mini cargo.toml. If we had lots of rust examples, that would be clutter, but for one, I think it's fine. (And if we had lots of rust examples, we'd also have the cost of compiling each one separately, so I don't think we're going that direction.)


An origin state root can be reported to auxiliary

An AuxiliaryCore contract tracks state roots of Ethereum at their
respective heights.
The cost to report a state root must be paid in the base token of the
auxiliary chain. This should prevent noise in the reports.
A state root can only be reported once per height.
Fixes #258

Also, it is probably a good idea to make the same change here that we did on OriginCore, where we track a mapping of the hash to the state root struct, if we can assume that a collision is unlikely.

the value sent must be exact amount
I would opt for a similar pattern for all three processes (report, vote, commit) for all blocks (origin, aux, meta)

@benjaminbollen  I addressed all comments. Please see the changes and the commit message.
The state roots are now stored in OriginBlock structs.

When I see this correctly, the reward of the miner is part of the state of the block that is mined. Thus, duplicate state roots at the same height should be impossible. Duplicate state roots at different heights astronomically unlikely. Do you see it the same way @benjaminbollen ?


Implement periodic backups of our datasets to azure

we need an automated way to backup our database. the suggested approach is using sanity export.

We need an upgrade to the cross dataset duplicator plugin so secret project reference is not used in main project. Created this card #1971

The default github runners offer 14GB of disk space, the global dataset is about 13GB, we need larger runners to export the dataset. Waiting for #sdpteam to get info on larger runners for our repo.

Used larger runners

@fernandolucchesi Added info to the runbook


Remove newline at end of README

Remedy critical issue.

do this plz

It is convention to end plaintext files with a newline (\n). See https://unix.stackexchange.com/questions/18743/whats-the-point-in-adding-a-new-line-to-the-end-of-a-file


Question, possible feature request concerning SSHFS user/group

Hi,
I've been using iNotifyScan for some time. It works flawlessly. It's amazing! I switched from using SAMBA to SSHFS. I don't want all my users (family members) using the same SSH key for user www-data. Example, lets say my SSH username is david123. When I create the SSHFS mount to my NextCloud files and I transfer files they are written as user david123 and group david123.  iNotifyScan picks up the files and adds them to NextCloud database but in I can't access them in NextCloud UI. I get permission denied due to the user/group naming.
When using iNotifyScan, is there a possibility to be able to update the user/group of all of the scanned files for any and all users to www-data:www-data during the scan?
Thank you for your time and hard work creating iNotifyScan. Very much appreciated!

Hi,
I'm very glad that you find this project useful :)
Regarding your feature request, let me first confirm if I understand it correctly:

You have a server running Nextcloud as user www-data; the same server also serves files as user david123 via SSH;
Files uploaded via SSH/SSHFS is not accessible by www-data because they have user/group david123. Therefore, the Nextcloud web server cannot access them;
Your solution is to run nextcloud-inotifyscan as root so that it not only adds files to the Nextcloud database, but also updates the user and group so that the web server can access them as www-data.

I think this is a valid and probably very common use case, but I wouldn't recommend your solution, for two reasons: 1) updating user/group is not intended usage of nextcloud-inotifyscan, especially given the fact that it requires privileges, and 2) there is a simpler and (arguably) better solution. In fact, I have exactly the same use cases and have been using the following solution for years, but never was able to document it anywhere. This might be a good time to do it. The idea is to attack the fact that

Files uploaded via SSH/SSHFS is not accessible by www-data because they have user/group david123.

This is doable using ACL to make new files and directories inherit group from their parent directory. Step-by-step example:

Select or create a group whose members are sharing (read & write) accesses, e.g., family;
Add users to the group, e.g., usermod -a -G family www-data; usermod -a -G family david123;
Change the group of files that you intend to share, e.g., chgrp family -R /path/to/shared/dir/;
Make the files writable by the group, e.g., chmod g+w /path/to/shared/dir/;
Make new files/directories inherit the group, e.g., chmod g+s /path/to/shared/dir/;
Make new files/directories inherit the mode so that new files/directories remain writable by the group, e.g., setfacl -dm g::rwx /path/to/shared/dir/.

New files and directories under /path/to/shared/dir/ will now have group family even if they are created by the www-data user via Nextcloud or by the david123 user via SSH. Both users are able to read & write files owned by group family, too. New subdirectories will inherit the same property. Existing files and directories will need to be updated in a similar way as /path/to/shared/dir/, possibly with the help of find -type d/f -exec.


tiflow-verify-go1.19-pipeline-master: Switch to Go 1.19 image for tiflow-lint

Related to https://github.com/pingcap/tiflow/pull/7240

/cc @purelind


[Azure Storage Queue Binding] Add option to decode base64 payloads #232

Description
Allow Azure Storage Queues binding to decode base64 payloads.
If you create and event trigger from an storage account (blob created, deleted, directory created, deleted) you receive base64 payloads.
By configuring the component option decodeBase64 to "true" the component binding will decode the payload for you:
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: mybindingname
spec:
  type: bindings.azure.storagequeues
  metadata:
  - name: storageAccount
    value: name
  - name: storageAccessKey
    value: key
  - name: queue
    value: test1
  - name: decodeBase64
    value: "true"

Issue #232
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly

PD: I want to create the Azure Storage Queue binding documentation that is missing, but I am waiting to see if this is merged to add the documentation updated with the change

Let me fix tests and add a new one

@yaron2 the actual existing storagequeues are not working.
func (m *MockHelper) Read(ctx context.Context, consumer *consumer) error {
	return nil
}

Is not calling the consumer callback, so the assertions inside all the handlers are not being executed, the tests sleeps for a while and passes.
I've fixed the tests and added a new one with the included logic but none of them are being executed.
Maybe you already knew it.
Thanks

@artursouza, extending a little bit more my previous comment, it looks like the same signature is used in the runtime repository:
https://github.com/dapr/dapr/blob/d8aa4008346735aa8e4746626c0a7a5eb4e9a9fa/pkg/runtime/runtime.go#L637
https://github.com/dapr/dapr/blob/d17e9243b308e830649b0bf3af5f6e84fd543baf/tests/config/dapr_redis_state.yaml#L18


Prep for release after Eve Online merge

Fixes to EveOnline plugin

Fix invalid data declaration
Fix -Wall compilation warnings

Unused imports
Unused functions
Unused arguments
Shadowed function name

Incorporate plain link into WidgetType, remove Maybe
Use only one function, always accept WidgetType

Add/fix documentation, change argument order

Document the difference between widget and non-widget functions
Pass the widget as the first argument to take advantage of partial
application

/cc @Drezil

The were being parsed out of the JSON but otherwise unused (hence the warning). I've removed them entirely now.

Ping @jferris any reason not to merge this?

Nope - looks good to merge.


feat: use k8s.website.tabs to show different installation types

For #3131

before:

after:

show:

thanks @yuluo-yx, added some comments !

/retest

/retest

if i'm not mistaken this pr will not only apply tabs to the quickstart, but more generally enable the use of tabs in the documentation.  once it's merged i'll work on using this feature in additional contexts.

if i'm not mistaken this pr will not only apply tabs to the quickstart, but more generally enable the use of tabs in the documentation. once it's merged i'll work on using this feature in additional contexts.

yep, This is mentioned in #3133.

i just checked out this PR, the tabs render correctly, but the functionality (switching between tabs) doesn't appear to work:  clicking on the currently-selected tab has this undesirable scroll effect, while clicking on the other tab (to switch to it) does nothing.

i just checked out this PR, the tabs render correctly, but the functionality (switching between tabs) doesn't appear to work: clicking on the currently-selected tab has this undesirable scroll effect, while clicking on the other tab (to switch to it) does nothing. has anyone else tried it and gotten it to work?

my bad, let me check

i just checked out this PR, the tabs render correctly, but the functionality (switching between tabs) doesn't appear to work: clicking on the currently-selected tab has this undesirable scroll effect, while clicking on the other tab (to switch to it) does nothing. has anyone else tried it and gotten it to work?

hi, I fixed , pls review again. tks 🫡

hey @yuluo-yx can we call the first tab "Without LoadBalancer Support" ?

hey @yuluo-yx can we call the first tab "Without LoadBalancer Support" ?

done!

hey @yuluo-yx you will need to rebase
site/content/en/v1.0.0/user/quickstart.md has been moved to site/content/en/v1.0.1/user/quickstart.md

hey @yuluo-yx you will need to rebase site/content/en/v1.0.0/user/quickstart.md has been moved to site/content/en/v1.0.1/user/quickstart.md

done!

/retest

Hi @yuluo-yx it works now.  thanks for the quick fix.
separately, i agree with @arkodg 's comment about making the "with loadbalancer support" tab the left/default tab. so.. could you please switch the order of the tabs?
otherwise it looks good to me.

Hi @yuluo-yx it works now. thanks for the quick fix.
separately, i agree with @arkodg 's comment about making the "with loadbalancer support" tab the left/default tab. so.. could you please switch the order of the tabs?
otherwise it looks good to me.

Is that right? I'm not sure I understand what you're saying. 🥲🥲

Is that right? I'm not sure I understand what you're saying. 🥲🥲

yes, that's right.  sorry for the late reply.

/retest

Is there another implementation of this feat or are there any other problems with it, if there is an alternative I can close.
@arkodg @eitansuez

this PR is far away than my expected, PTAL https://github.com/envoyproxy/gateway/pull/3208

maybe you can rebase your work base on https://github.com/envoyproxy/gateway/pull/3208 ?

It is merged, I will close it.

It is merged, I will close it.

you can rebase with main branch, you did a lot of around better display/html/css, those are very valuable.

It is merged, I will close it.

you can rebase with main branch, you did a lot of around better display/html/css, those are very valuable.

current pr is too confusing, I will resubmit a pr to update the existing file style, thank you for recognizing my work! 🫡


Use new updateRecommendations API for RM demo

Bulk update results
+
Update Recommendation

Code related to the --days="" parameter wasn't merged.


Executing auto skipped extra test while using test_run_rules parameters

Test code:

@Test
@TestPriority(Priority.P1)
public void t1(){
    System.out.println("1");
}

@Test
@TestPriority(Priority.P2)
public void t2(){
    System.out.println(2);
}

@Test
@TestPriority(Priority.P1)
public void t3(){
    System.out.println(3);
}

@Test
@TestPriority(Priority.P3)
public void t4(){
    System.out.println(4);
}

test_run_rules=PRIORITY=>P1
result in the screenshot

Appears only in integrated development environment at the end of program, when report is generated correctly, won't fix


Mac OSX app support

Feature Request : automation for humans should support automation of Mac OS apps.

Experimental support : https://github.com/intuit/automation-for-humans/pull/7/commits/dc74fb7e1d43883843826b3ed956a7d06f34c92d

This feature still uses hardcoded AXPath's for finding elements as Appium does not support search using relative paths. See https://github.com/appium/appium-for-mac/issues/27
There is a way by which we can get the AXPath's of all the elements in the screen by using the accessibility api.  https://github.com/intuit/automation-for-humans/ needs to support that.


has_field? is broken, falling back to rspec's matchers

I have this empty template which I was writting tests for and here is what I got :
require 'spec_helper'

describe UserController do
    render_views

    describe "Creating new users :" do
        it "should have 'new' action" do
            get 'new'
            response.should be_success
        end

        it "should have signup_path" do
            visit signup_path
            page.status_code.should == 200
        end

        it "should have signup form" do
            visit signup_path

            page.has_field? '#login'
            page.has_field? '#password'
        end

        # This here is to compare capybara's matchers with rspec's ones
        it "should have signup form" do
            visit signup_path

            page.should have_selector '#login'
            page.should have_selector '#password'
        end
    end

    describe 'Authentication' do
        it "should sign the user in" do
            #post :create, :session => { :login => @user.login, :password => @user.password }
            #controller.current_user.should == @user
            pending "Tests are to be done for this controller !"
        end
    end
end

The so called template :

And the results :
Running: spec/controllers/user_controller_spec.rb
...F*

Pending:
  UserController Authentication should sign the user in
    # Tests are to be done for this controller !
    # ./spec/controllers/user_controller_spec.rb:34

Failures:

  1) UserController Creating new users : should have signup form
     Failure/Error: page.should have_selector '#password'
       expected css "#password" to return something
     # ./spec/controllers/user_controller_spec.rb:29:in `block (3 levels) in <top (required)>'

Finished in 0.54628 seconds
5 examples, 1 failure, 1 pending

Failed examples:

rspec ./spec/controllers/user_controller_spec.rb:25 # UserController Creating new users : should have signup form

Maybe I did something wrong because when I dig deeper I see that these has_field? methods return false so there is nothing wrong with this value ...
I'm quite lost with the understanding of this behaviour. (If this is just a noob related problem, sorry for bothering. I'm learning)

Please use the mailing list for question. Usually it's quite a lot quicker to get a response there. Just fyi, has_field? returns a boolean, you need to use this with some kind of assertion if you want it to work in a test. In RSpec that would be page.should have_field('..'). But again, use the mailing list!

Oh snap !
I'm sorry ! This was a noobie mistake, thank you for taking the time to answer me !


[NCL-4647] Remove dockerfiles, yaml config

Also remove obsolete requirements.txt from the venv folder
All Submissions:

[ ] Have you added a note in the CHANGELOG.md for your change?
[ ] Have you added unit tests for your change?

retest this please


[Dupe Bug] Rocket Loader

If we try to load any item in the rocket while the rocket is still full the item in the first slot of the rocket will be duplicate by the number of item we try to insert.

I don't quite follow.  I did try filling a rocket up with stone and putting stone into the rocket loader.   I did notice some odd behavior where the stone disappeared on the client (reopening the loader a few times made it reappear in the loader and attempting to load more stone into the load just swapped the invisible stone stacks with the one you try to drag in) but the total number of items in the world remained the same.

you seem to fix something so i gonna tell you if its fix in the next update or not :)

Have you got a chance to test the recent builds?

gonna try this evening i keep you up to date

i have test it the same ways as before and the dupe is still here
To recap i put in each slot of my rocket 1 stone and in the first slot of the first storage says a diamond block when the loader try to insert an item thait is not the stone or the diamond block this item is converted to a diamond block if its not clear enough i gonna make a video ^^

Alright, Thanks!  I've been able to replicate it now.


No action executed when failing due to conditions or /quests quit

What is your Quests version / build number (do not say "latest")?
4.0.5-b246
Describe the problem and how to replicate it. Any console errors to post using a pastebin?
When people quit the quest using /quests quit {quest} they don't get the standard "Quest failed" action, nothing happens. (except leaving the quest). The same thing happens when people fail the quest due to not meeting a condition.
So there is two possible fixes:

Add an action variable in the stages that applies when people quit using either of these two options
Make it so that when "/quests quit" is performed, or when people fail it due to conditions, they get the "fail" action applied on them.

To me personally, the latter seems the easiest. But either works. 😄
If applicable, can you provide an example quest from quests.yml using a pastebin?

By definition, quitting is not the same as failing - you may disable quitting under Edit Options -> Allow quitting during quest in the quests editor. As for conditions, I'll look into it.


formAction$ code is not executed if Form component has "class" attribute

Hey there,
first of all, thank you for spending time on your libraries and providing them to the open-source community! :)
I'm currently using the library for setting up several forms and encountered the following problem, which took me a few days to narrow it down.
When I add the "class" attribute to the Formcomponent provided by the useFormhook and define a formAction$, the code in the action doesn't get executed on the server.
When server-side-validation is added, the validation is executed. Even if the validation succeeds nothing more happens, there are no error logs etc.
export const useFormSubmit = formAction$<TestForm>(async (formValues) => {
    // This is not executed!
    console.log(formValues);
}, valiForm$(TestFormSchema));

export default component$(() => {
    const [TestForm, { Form, Field }] = useForm<TestForm>({
        loader: useFormLoader(),
        action: useFormSubmit(),
        validate: valiForm$(TestFormSchema)
    });

    return (
        <Form class="space-y-4"> // "class" attribute seems to be causing this behavior
            <Field name="test">
                {(field, props) => (...)}
            </Field>
        </Form>
    )
})

As my VS Code Intellisense suggests that the "class" attribute is supported for the Formcomponent, I suppose that this is not the intended behavior. Can you confirm this?

Sorry for the late reply? Were you able to resolve the issue?

Hey, I closed the issue, because the class attribute is actually not causing the problem.
Unfortunately, I still have problems in combination with the latest Qwik version. The code in the submit action hook is just not executed. It seems that the form is stuck, because the "validating" property of the form object is still set to true and not updated. This seems to prevent the hook function from being called.

I am sorry to hear that. This could be a signal bug in bug. Does it help to downgrade Qwik?

I already tried to downgrade the same version as the modular forms playground, but without success.
I read that there was a bug with the validating state in the SolidJS version of the package, which you fixed last year. Could it be a similar case here for the Qwik version maybe?

I don't think so. I can investigate it if you send me a StackBlitz link. Here is a template: https://stackblitz.com/edit/modular-forms-qwik?file=src/routes/login/index.tsx

Sure, I copied the important things of the corresponding index.tsx to this fork: https://stackblitz.com/edit/modular-forms-qwik-zrkmwv?file=src%2Froutes%2Flogin%2Findex.tsx
Thank you for spending some time on this :)

Just wanted to let you know, that I found the problematic part here, even though I still don't know why and how to work around it.
But importing Qwik's useNavigate hook and using it in the useTask$ hook causes the submit promise to never resolve. But this seems to be a Qwik related issue any way.

Thanks for the info. Sorry, I am busy at the moment and have not had a chance to investigate it.


feat(cli): Better CLI args validation

Start adding better CLI args validation (#603). Validate log-level to start. This doesn't close the issue.

Yes please. @srawlins could you please do what @vojtajina and just print the warning and not exit. Other than that I'm happy to get this in.

@srawlins ping

Eek, sorry. I'm on vacation for 2 more weeks. Back in late June.
On Jun 3, 2015 9:26 PM, "Friedel Ziegelmayer"<EMAIL_ADDRESS>wrote:

@srawlins https://github.com/srawlins ping
—
Reply to this email directly or view it on GitHub
https://github.com/karma-runner/karma/pull/1234#issuecomment-108587877.

OK changed behavior to exit on unrecognized log levels, and added a test. Ready for review again. Sorry for the absurd delay :worried:

I fixed the lint warnings and there is still a Travis failure that I don't think is related to this PR because it's sporadic. Travis builds #1600 and #1601. It passes for me locally.

Thanks, looks good to me :octocat:


AVX-26322 fix terraform-OCI-module

As discussed, we should have one more final commit , tracked in another ticket for documentation re-write before publishing this module

In aviatrix-controller-initialize/main.tf, use "python" instead of "python3".


Adding tooltips to buttons without text (#347)

Fixes #347.
Changes proposed in this pull request:

Added "title" to icon buttons missing tooltips

Let me know if I missed out on anything.

 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.

Awesome, that sounds great! Thanks!


When there is only a slider in the model, there is a problem with the tooltip positioning of the slider

TuSimple/naive-ui version (版本)
2.34.3
Vue version (Vue 版本)
3.2.45
Browser and its version (浏览器及其版本)
Chrome(105.0.5195.127)
System and its version (系统及其版本)
Windows 10
Node version (Node 版本)
Reappearance link (重现链接)
https://codesandbox.io/s/relaxed-chandrasekhar-zjmel6?file=/src/App.vue
Reappearance steps (重现步骤)
点击按钮打开Modal，拖动Slider，Tooltip才会归位，再次打开依旧会有这个问题
Expected results (期望的结果)
打开Modal不应该显示Slider的Tooltip
Actual results (实际的结果)
打开Modal在卡片外部显示了Slider的Tooltip
Remarks (补充说明)

这个问题是由于组件库对 a11y 的处理造成的，modal 打开后默认会聚焦到里面的元素，而 slider 被聚焦时，会显示 tooltip。当 modal 内只有 slider 一个组件时，默认就会造成 tooltip 的定位出现问题。有两个解决方案：

设置 modal 的 auto-focus  属性为 false。
将 slider 的 tooltip 默认设置为 false，等到 modal 打开后或在合适的时机下再将其设置为 true。
第一种解决方案的参考：https://codesandbox.io/s/sharp-bassi-d2z8bh?file=/src/App.vue

This issue is caused by the processing of a11y by the component library. When the modal is opened, it will default to focusing on the elements inside, while when the slider is focused, it will display tooltip. When there is only one component in the modal, the default will cause problems with the positioning of the tooltip. There are two solutions:

Set the modal's auto-focus prop to false.
Set the slider's tooltip prop to false by default, and wait until modal is opened or at an appropriate time to set tooltip prop to true again.
Reference for the first solution: https://codesandbox.io/s/sharp-bassi-d2z8bh?file=/src/App.vue

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


Intro Notes from winter 2023

INTRO SLIDES

[ ] Date says 2022 - not 2023
[ ] maybe talk about functions being verbs and objects being nouns
[ ] add data to objects description list
[ ] add biocounductor to the "more on Functions and Packages slide"
[ ]  update that Hadley wickham works at Posit instead of RStudio... mayeb add a slide about the connection between these two
[ ] emphasize pull down menu for cran when installing from menus by adding  a sentence to the image slide
[ ] add slide about how installing only needs to be done once (although I feel like I did do something about that somewhere, but seems like it needs to be emphasized)
[ ] remove installation of jhur from slides
[ ] maybe remove the install from menu slide - seems to be causing confusion: A-hah! Thank you! It worked when I copy and pasted one line of code at a time into the console. Before, I was clicking on the Run dropdown and hitting "run selected lines", which wasn't working out. Appreciate it. (I think running it as a chunk is going to be problematic too in the lab cause they need to answer Yes to a question)

RESOURCES

[ ]  Posit is lower case on the resource page

RSTUDIO

[ ] Maybe talk about difference of scripts and Rmarkdown on a slide? Previously I thought ignoring scripts might keep it simple, but maybe we need to describe both (or at least edit all the old images from John to be markdown only)?
[ ] if we do both maybe just preview what each look like when we first talk about opening the editor

LAB

[ ] Maybe we should work through installing the jhur package as part of the lab (like directly include it) and remove it from the intro slides? - in retrospect idk why we didn't do this before :P
[ ] add coment in the lab about this: “install from sources the package which needs compilation” and I had to type in Yes to get it to run the install...
[ ] I think we should tell them to copy paste one line at a time into the console for the jhur installation step
[ ] add some trubleshooting info maybe??(might need to reinstall R)
[ ] Maybe also include discussion about running code in console vs the editor/Rmarkdown

Installation Gude

[ ] Maybe add extra info for troubleshooting installation somewhere.  Was R the right version for their operating system?etc.
[ ] Point them to older versions of RStudio
[ ] Also I think we could make the installation instructions a bit more prominent somewhere on the website? I was struggling to find them :P

HW1

[ ] update about takeaways (called something different I guess)

Reproducibility

[ ] some images are kinda small - like the pyramid - could make it bigger
[ ] session info slide was small
[ ] touch on Posit?

https://community.rstudio.com/t/rstudio-desktop-crashes-on-startup-with-library-not-loaded/130296

Thanks @carriewright11 🤩

no problem :)

https://dailies.rstudio.com/version/2021.09.2+382.pro1/ for older versions of macos @avahoffman  do you recall what Krista had?

New top of the RStudio lab:

https://community.rstudio.com/t/rstudio-desktop-crashes-on-startup-with-library-not-loaded/130296

ooh forgot this... maybe I will add to the help page?

https://dailies.rstudio.com/version/2021.09.2+382.pro1/ for older versions of macos @avahoffman do you recall what Krista had?

Mac OS 10.14.5

I think this is done? What do you think @avahoffman ?

I think so!


Release?

Looks like its been a while since the last release. Maybe using release-drafter to pump out a new release would be a good idea?

Yeah, I would prioritize this. I just tried to build a small docker container using docker, and the version on pip is pretty broken (couldn't load VideoStims).

Done!


sharing PyPi & ReadTheDocs access rights

Hi @grlee77 @aaren @kwohlfahrt, I'll be offline for a month and a half from the 21st on. So I'd like to give some or all of you access to PyPi and ReadTheDocs for this project. If you tell me your username for those sites then I'll do that.

I'm aaren on both pypi and readthedocs.

I'm grlee77 on both.  (just created the ReadTheDocs one)

kwohlfahrt on both, just registered.

Thanks, you all have access on both sites now.

I've also created https://ci.appveyor.com/project/PyWavelets/pywt and removed the config on my personal Appveyor account. If you register on www.appveyor.com I can add you as a collaborator (need the email address you registered with for that, not your username).

Have signed up to appveyor, using github. Didn't put an email in but presume it is same as my commit email.

Yep, that works - you're in @aaren

@rgommers I'm late to the party, but I've also signed up to appveyor (initially using my email, but it is also linked to my github account). Can I be added as a collaborator please?

@kwohlfahrt I've added the PyWavelets Core team with Admin rights to the account, so you should have access now. If that doesn't work, which of your email addresses do I use?

Works, I had to log out and then log back in with github in case anybody else runs into that problem. Thanks!