RND Rework [Death to Techweb]

About the PR

Based off of this design doc.
The TL;DR
All technologies are now sorted into (as of right now) 4 categories called "Disciplines." Each discipline roughly corresponds to a gameplay loop (engie/salvage, medical, science, service) and contains a variety of technology that unlocks lathe recipes.
Each tech, like before, has a cost that must be paid to unlock it. However, technologies aren't unlocked linearly. Each tech is assigned a "tier," and researching a significant portion (75% as of right now) of a discipline's techs of a tier will unlock the next one.
Furthermore, tier 3 is a "lockout tech." Unlocking a tier 3 tech will set it as your main discipline, preventing you from researching other tier 3 techs of other disciplines.
The last point of note is the card system. Instead of having all available techs be researchable, you have to pick from 4, 1 from each discipline, to unlock. After unlocking one, 4 new cards will be drawn that you have to choose from.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: The autolathe catalog has been expanded.
add: You can now see what recipes a technology unlocks.
tweak: The research console can now only select technologies from the specific options given to the player.
tweak: Technologies are now sorted into disciplines. Unlock all the techs in a discipline to unlock higher tiered tech.

for the record this design approved by me at least

My absolute favourite part about this rework is the fact that it actually tells you which items each technology unlocks, as someone who's new to the Science tech tree it's been infuriating trying to find out what I need to research.
Admittedly it's also been difficult to figure out what I've already unlocked. Is there anyway to view the items of previously unlocked technologies, e.g. mousing over them in the tab?

Admittedly it's also been difficult to figure out what I've already unlocked. Is there anyway to view the items of previously unlocked technologies, e.g. mousing over them in the tab?

that functionality is in the ui. Mousing over any of the available/already unlocked researches will show a tooltip with all of the recipes they unlock.

Ready for review

I have a question. Is specialization tied to a game round or to RnD server?

I have a question. Is specialization tied to a game round or to RnD server?

It's per-server.

This looks really awesome Emo, keep up the good work!

review finished.

 Like, really?

If you want to provide useful feedback please do it without sarcasm.

Did you really have to make bluespace storages THAT cheap? Like, really?

I dont think half a stack of plasma and steel is that cheap.

Did you really have to make bluespace storages THAT cheap? Like, really?

I dont think half a stack of plasma and steel is that cheap.

Considering how easy it is for cargo to farm a morbillion space credits, yes, it is cheap. I've had this convo under the Liltenhead video. The fact that it requires "a lot of research points" and "is an endgame research" are kinda bad points. Even on LRP servers with 1-2h shifts it is rly easy to speedrun the whole tech tree (artifacts have like 5 conditions to activate, and only 2 are actually hard (high pressure and radiation)). So ur average science boy will be able to get his hands on the desired tech 3 research in no time + he can get all of tier 3 research via printing disks. After that, he just goes to cargo, after they acquired more money than some countries have IRL (frezon is still easy to make), orders 10 crates of steel, plasma and plastic and mass produces OP bluespace bags. After that everyone on the station is capable of having LITERALLY EVERYTHING on them, starting from the whole freaking medbay, materials enough to build another station from scratch and ending with local armory in their pockets.  Mfs can have 10 guns and 20 ammo boxes on them and shoot n scoop nonstop. Imagine flukies faces when they arrive at the station and every single secoff has like the whole military base in their pockets as well as 10 medkits. So I thinks it's a REALLY BAD idea to make bluespace bags so easy to make. And I'm not even going to mention MRP servers with 3h+ long shifts, lol.


Use of ignore_changes via advanced block

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

terraform: Terraform v1.3.7 on windows_amd64
azure provider: AzureRM v3.38.0
module: v3.1.2
Description
First off THANK you for this awesome module.  I have watched the youtube videos and been living in the wiki for the past several weeks learning the ins and outs of getting this up and running.
Second, I chose the bug issue as it seemed the most appropriate but maybe this is not a bug but possibly a feature request or more so maybe just a question on functionality so sorry if this is not the appropriate place.
We have policy in place that assigns a created tag to Resource Groups and resources that support them. Subsequent runs of terraform obviously do not track that since it was applied via policy and terraform wants to remove the tag for previously created tags.  It looks the lifecycle --> ignore_changes option is my go to fix to get around this.

Since this may not look like there is a fix anytime soon in the azurerm provider, I am wondering is it possible to leverage the lifcycle block in the advanced block for custom settings?  I gave it a good old college try but I think that maybe this is not supported at this point due to the error I received.

Describe the bug

Tags modified outside my Terraform state are flagged for removal because they are applied as policy. I have searched around the issues here and found several mentions that seemed to be related to other resource problems. I think it was on here that perhaps this should be handled at the provider level, so I spent quite a bit of time searching around there as well.
I found some posts regarding the AWS provider, and it looks like it was implemented there, or at least a defult_tags option at the provider level. I did find several posts on the azurerm provider github and looks like some form of functionality, perhaps the tag_keys_to_ignore flag, may be included in the v4 milestone based on this issue: https://github.com/hashicorp/terraform-provider-azurerm/issues/13776
Steps to Reproduce

Deploy the CAF-ES  in some form that includes a resource
Once deployed add a created tag, or any tag other than the ones provide via TF to simulate policy or some other automated tag addition.
Re-run a plan on the CAF-ES Terraform configuration.  The resource(s) that had tags added outside the TF will be listed as changes since the tag is not managed by TF

Screenshots

Azure management RG

TF Plan showing management RG group

Here is my attempt at using the advanced block - Seem to be getting a linting error here so I tried with an = which I didn't think would work and didn't. But the advanced block documentation is empty on the wiki for now so I thought I would ask out here if there was maybe something I am missing of if it is just not possible at this time?

Results of running plan with lifecycle block with =

Additional context

It might be nice, if possible, to have something at a global module level that you could add to ignore tags on resources that support them.  I know that is a generic type of question as what I gather tags on the surface don't seem too bad but based off what I have been reading, at least in the azure spaces there is quite the cobwebs under the covers with not all APIs etc. doing things the same in regard to tags and many resources do tags different etc.
It seems like TAGs are the main culprit when it comes to the need for ignore_changes.  Hoping there is a way to implement this as a workaround here in the module even if at the advanced block level.
Thanks so much for any help

@krowlandson Thanks for such a timely reply. I'll be honest I had not seen the time_static option as I initially was not looking to do the timestamp tagging in TF.  That looks like it actually might be an option.  I was aware of the azure resource creation, but it seems a little clunky in our use case.  I wish that was made available in the portal and not sure why it is not, atleast to my knowledge. While we try to do as little as possible actually in the portal, I think it is a nice and quick feature to just have the tag as well right there in your face without having to look elsewhere etc.
Let me poke around with the time_static provider and see how that works.  In the end the Policy is only writing the tag if it doesn't exist so if we can populate those in the TF when we create them, I think this problem might be solved.  Going to kick the tires on this right now and will let you know.

@krowlandson Using the time_static provider seems to be working like a champ. But I may have discovered a bug. I set the value at the root module in the default_tags section which seems to have deployed the tags as expected down to all of the resources, or at least the ones I am deploying with the exception of the Azure Firewall Policy.  It looks like it is only excepting the tags from the custom settings otherwise it sets it to null while many others are using the local.tags.
Not sure if this is by design for the Firewall policy?
https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/blob/main/modules/connectivity/locals.tf#L1058
Would you like a separate bug ticket if this is actually a bug?

@MCrank I believe this was due to the documented constraints around PATCH operations.

Azure IP Groups and Azure Firewall Policies don't support PATCH operations, which means they don't support updating tags through the portal. Instead, use the update commands for those resources. For example, you can update tags for an IP group with the az network ip-group update command.

I can look into this again to see whether this actually causes issues when managing tags through Terraform.
If not, we can get the "default" tags added to these resource too.

Would you like a separate bug ticket if this is actually a bug?

Happy to track via this issue 👍🏻


Write lyric sprite text's own TextBuilder and

Because:

Draw rectangle is not able to be assigned in the TextBuilderGlyph.
Some of the params might not be use in the TextBuilder.

Also, it will be hard to adjust the main or position text position if use TextBuilder.

Another solution:
Use TextBuilder to generate the ruby and romaji, but will make another class for generate the real position.

https://github.com/ppy/osu-framework/pull/5123/files#diff-0b308524e0451e8e3bf4ba9b12b89870b72bbc9a507a4591a5ba87aace37a95d
Also, see this change.
I guess it might cause the api broken.

Implemented.


Dead mail server on active company email

Using this email_verifier to verify an email that uses a company domain name causes an out of mail server error. So when a user puts in<EMAIL_ADDRESS>they get the "Email appears to point at a dead mail server" error message. This shouldn't happen since it is a real, actively used account.

Are you using the gem locally on your computer or from some publicly visible server? The SMTP server in most cases needs to be able to verify it can connect back. That may be the issue in this case.

I'm having the same problem one of my macbook pros. So yes I'm using it locally on my computer. Only, I don't see this behavior on my iMac at work.


Missing json files about train split

Hi,
Thanks for your excellent work and contributions of open-sourced code.
I think there should be a bunch of json files claiming the train/test splits of downstream datasets. I met this issue when I run
    cd task_adapting
    CUDA_VISIBLE_DEVICES=1 python main.py \
        --base_dir ../data \
        --pretrained_model vit-b-1k \
        --adapt_method prompt_wo_head \
        --test_dataset 'cub200' \
        --epochs 50 \
        --lr 50000 \
        --weight_decay 0.0 \
        --checkpoint_dir ../meta-training/checkpoints/vit-b-1k-wo-head.pth

The error seems lie in a class named 'JSONDataset', which needs 'train.json' to continue training.
Thanks for your efforts.

Hi,
We use the same data split as VPT. You can find these json files at the data resources provided in VPT repo.

Thanks. I will close the issue.


Summary - Basic Concepts

Link

https://netflix.github.io/conductor/intro/concepts/

Workflow Definition

Workflows are defined using a JSON based DSL
includes a set of tasks that are executed as part of the workflows
control tasks

fork
conditional

application tasks

executed on a remote machine

At reviews

use examples to help people understand well

Check it

is rabbitmq or kafka supported as a EVENT output?

Lifecycle of a Workflow Task


Web app in mac

Hi all
I'm trying to test a web app built with ionic, which is using Chromium in the background.
This are the capabilities I've set up:
            caps.setCapability("platformName", "mac");
            caps.setCapability("platformVersion", "12.2.1");
            caps.setCapability("automationName", "mac2");
            caps.setCapability("bundleId", "com.company.id");

If I set up this capabilities in the appium inspector, the app will will be launched but I can't inspect anything, even if I change the context.
Can I use this driver for webapps? if so, Am I missing something?
Thanks

So for example in the appium inspector, if I select:

This area will be selected

There is nothing there, there is not a hidden view or anything

Do you get any error like me when you just run appium from terminal ?
https://github.com/appium/appium-mac2-driver/issues/128

This driver does not support WebView context.
This driver works only when the app under test can work with XCTest framework by Apple. (Probably native apps?)

Oh that makes sense why it's not working.
Not sure which driver I can use in this case then?
Thanks

Oh that makes sense why it's not working.
Not sure which driver I can use in this case then?
Thanks

If the webview provides any possibilities of debugging then it would be theoretically possible to use chromedriver to connect to it and automate using vanilla selenium. Otherwise 🤷

I guess the ionic project address the possibility of testing with existing framework if it could like how to run tests with chromedriver.

I guess that means that Appium doesn't support this kind of Apps
Thanks both for the answers,


How can I use config file with MpvPlayer

I have added
        public void LoadConfig(string absolutePath)
	{
		mpv.LoadConfigFile(absolutePath);
	}

to Mpv.NET.Player.MpvPlayer and it worked.

Hello @Fantoom,
I will include this feature in the next release of Mpv.NET. In the meantime, a workaround that does not require re-compilation of the latest release is:
player.API.LoadConfigFile("my/absolute/path/to/config");

The "API" property on the player is a reference to an instance of the "Mpv" class that is being used by the player.


Implement Countable where possible

KeyValueStore implementations should implement Countable, where possible.

Here are what I have in mind:

ArrayStore and JsonFileStore are easily countable ;
NullStore has the method and it should return zero ;
CachedStore is countable and call the delegated store count() ;
PhpRedisStore and PredisStore call LLEN (http://redis.io/commands/LLEN) ;
RiakStore count keys using a stream (http://lists.basho.com/pipermail/riak-users_lists.basho.com/2011-November/006636.html) ;

Perhaps could the CachedStore store the count result in its cache with a special key. What do you think?

On the same idea than https://github.com/webmozart/key-value-store/issues/8#issuecomment-119250804, we could have a CountableDecorator for uncountable repositories (like Riak). What do you think?

Yes, let's do that!

I think we can close that now :) .


Preview module

Add preview module to the webapp that will show last few entries of a notebook for all the users.

Epic #2

The initial version of the preview app is available at:
https://github.com/pipalacademy/pipalhub/tree/master/preview
This needs to be cleaned up and moved over here.


Change Identifier name

Contact Details
No response
Is your feature request related to a problem?
Misstyped the identifier name
want to change the name
does not work
Describe the solution you would like!
retype / correct the identifier name
save and have it accepted, integrated in the ARC locations where needed
Additional context!
No response
Host
ARCitect
Host Details
ARChitect 0.0.31
swate1,0,0-beta.03

In my opinion this should be fixed on ARCitect side. With right click context menu


Update forced method to accept Closure type

This PR adds support for closures, allowing us to do something like this:
->forced(fn() => ! app()->isLocal())

Nice, thanks!


Fixed build using Zig 0.7.0 on Gentoo Linux.

Regarding build.zig, I found that andrewrk had
fixed another project using the changes that
I imported.
https://github.com/baverman/sdl-zig-demo/commit/bfc3644b8928d76ed0fee941b9af95e4b7ea1ff7
Per Reddit, from what I gathered, all strings are now
null terminated, so the 'c' prefix before a string
is unnecessary.
https://www.reddit.com/r/Zig/comments/fq504d/cstring_literals_not_working_zig_05_on_windows_10/
Will try to get zig 0.7.0 build into main gentoo tree.

Closed due to lack of interest

Thanks, I fixed it myself just now.

Thank you.  I wish to apologize for how I spoke on hackernews about this PR a while back.  I had closed the PR after a week, in my head thinking you were just ignoring a simple fix, which was impatient of me.  Again, I apologize, it looks to me like you're doing great work, and I hope you have a great day!

Didn't even notice, don't worry :slightly_smiling_face:


ci: remove not needed workflow

This PR removes not needed golang workflow.
This change will optimize github actions runtime.
Thanks to this line the file I just removed will not be created here by bot again.

/rtm

/au

/rtm


No matching version found for<EMAIL_ADDRESS>

hey try 2.0.0-alpha.34, lmk if that doesnt work

hey try 2.0.0-alpha.34, lmk if that doesnt work

I encountered the same problem.
when i want to install the latest version:
> npm update -g extension
npm error notarget No matching version found for<EMAIL_ADDRESS>
so i try to create a new project with my local version:
>  extension create my-new-extension --template=init

 - Starting a new browser extension named my-new-extension..
 - Ensuring my-new-extension folder exists...
 - Checking if destination path is writeable...
 - Scanning for potential conflicting files...
 - Installing  my-new-extension...
 - Writing package.json metadata...
  - Installing dependencies... (takes a moment)
✖✖ Can't install dependencies for my-new-extension:
✖✖ Command npm install --silent failed with exit code 1

the version that work for me is:
  "devDependencies": {
    "extension": "2.0.0-alpha.29" //work
    "extension": "2.0.0-alpha.30" //work
    "extension": "2.0.0-alpha.31" //work
    "extension": "2.0.0-alpha.32" //work
    "extension": "2.0.0-alpha.33" //fail
    "extension": "2.0.0-alpha.34" //fail
    "extension": "2.0.0-alpha.35" //fail
    "extension": "latest" //fail
  }

Normalized in<EMAIL_ADDRESS>

<EMAIL_ADDRESS>works


[Bug] Command execution by mouse click does not restore focus in editor

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:  1.70.1 (Universal)
OS Version: MacOS

Steps to Reproduce:

Start with command palette by press F1 or CMD+Shift+P
Execute command by mouse click at command text

Actual (Problematic) Behavior:
Command palette disappear and focus left from editor
Expected Behavior
Command palette disappear and focus keep in editor

Execute command by mouse click at command text

Which command?

Execute command by mouse click at command text

Which command?

any command

any command

Ok, cool. I choose  "Focus on Output View" which works correctly. Closing, right? @BusinessDuck So, please be precise

any command

Ok, cool. I choose "Focus on Output View" which works correctly. Closing, right? @BusinessDuck So, please be precise

For example you can use "Add cursor above" , i also change it in main description of this thread

Ok, thanks for clarifying. @TylerLeonhardt I can reproduce with the aforementioned command


Missing documentation for [assembly: PreserveToolkit]

There is no mention in the current documentation regarding this.

Microsoft support for Xamarin will end on May 1, 2024 for all Xamarin SDKs including Xamarin Community Toolkit. In preparation for this, all issues and PRs in this repository are being closed.
.NET MAUI Community Toolkit is the replacement for Xamarin Community Toolkit and its documentation can be found here.


Option to add "Play FM with AngelLoader" or similar to Explorer context menu

Someone asked for this. Optional of course, so we don't clutter up the menus if the user doesn't want us to.

So it looks like this is way harder than it sounds. It's true we can easily add an item like "Play in AngelLoader" to the context menu through simple registry additions, but:

If n files are selected at once and the option is clicked, it simply starts the associated app n times, passing one file for each app instance. There's no way to have it bundle them all up and send them all at once to one instance (with the simple registry-based extension association method).
If we want to handle multiple files then we need to write a shell extension, and then it's not recommended to use managed code so we would want to write it in native C++. And it looks hellish to do.
If we did write a shell extension we would have to make sure it doesn't contribute towards making the context menu take literal seconds to load as it does on my system (I need to trial-and-error to see what exactly is being so rude as to cause that...)
In any case, we would need to pass the received filenames to the first instance of the app in case we're already running. We can't do it with our current minimalistic "PostMessage with HWND_BROADCAST and a couple of IntPtrs" approach. We'd need to use pipes, or the new VB-like single-instance event-based communication thing that new .NETs have.


Local cluster stuck on waiting for cluster agent to connect

Rancher Version: 2.1.1
Installation: Single Install
Deployed rancher into an existing cluster and set k8s-mode=embedded and add-local=true. When rancher starts up the local cluster will show "Waiting for API to be available" and the logs will continue to show [ERROR] ClusterController c-xyz [user-controllers-controller] failed with : failed to start user controllers for cluster c-xyz: failed to contact server: Get https://x.x.x.x:443/version?timeout=30s: waiting for cluster agent to connect
Deployment:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: rancher
  namespace: rancher
  labels:
    k8s-app: rancher

---

apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  name: rancher
  namespace: rancher
  labels:
    k8s-app: rancher
subjects:
- kind: ServiceAccount
  name: rancher
  namespace: rancher
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: cluster-admin

---

apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: rancher
  namespace: rancher
  labels:
    k8s-app: rancher
  annotations:
spec:
  replicas: 1
  selector:
    matchLabels:
      k8s-app: rancher
  template:
    metadata:
      name: rancher
      creationTimestamp:
      labels:
        k8s-app: rancher
    spec:
      serviceAccountName: rancher
      containers:
      - name: rancher
        image: rancher/rancher:v2.1.1
        imagePullPolicy: IfNotPresent
        args:
        - "--k8s-mode"
        - "embedded"
        - "--add-local"
        - "true"
        - "--http-listen-port=80"
        - "--https-listen-port=443"
        - "--no-cacerts"
        volumeMounts:
        - mountPath: "/var/lib/rancher"
          name: data
        ports:
        - containerPort: 80
          protocol: TCP
        - containerPort: 443
          protocol: TCP
        livenessProbe:
          tcpSocket:
            port: 80
          initialDelaySeconds: 60
          periodSeconds: 30
        readinessProbe:
          tcpSocket:
            port: 80
          initialDelaySeconds: 5
          periodSeconds: 30
      volumes:
      - name: data
        persistentVolumeClaim:
          claimName: rancher-persistentvolumeclaim

Looks like this is related to #15973 & #13129. If I revert back to v2.0.8 I do not experience this issue.

Hey @nathankodilla there’s a new installation method using rancher 2.1 onwards. Can you check the latest docs and see if that helps.

@aemneina could you provide more specifics on what changed? I looked through the documenation about single node and did not notice anything different from what I have performed.

waiting for cluster agent to connect indicates the cluster-agent not being able to connect, the logging of the cattle-cluster-agent pod should reveal why it cannot connect.

Seeing the issue on our cluster as well @superseb here are the logs you requested:

INFO: Environment: CATTLE_ADDRESS=<IP_ADDRESS> CATTLE_CA_CHECKSUM=d0dce71b6f3f62a82d176dd86516b6e8c82d500947af66702c19b5abaadac9dd CATTLE_CLUSTER=true CATTLE_INTERNAL_ADDRESS= CATTLE_K8S_MANAGED=true CATTLE_NODE_NAME=cattle-cluster-agent-68587d9894-crptb CATTLE_SERVER=https://rancher.test.com CATTLE_SERVICE_PORT=tcp://<IP_ADDRESS>:80 CATTLE_SERVICE_PORT_443_TCP=tcp://<IP_ADDRESS>:443 CATTLE_SERVICE_PORT_443_TCP_ADDR=<IP_ADDRESS> CATTLE_SERVICE_PORT_443_TCP_PORT=443 CATTLE_SERVICE_PORT_443_TCP_PROTO=tcp CATTLE_SERVICE_PORT_80_TCP=tcp://<IP_ADDRESS>:80 CATTLE_SERVICE_PORT_80_TCP_ADDR=<IP_ADDRESS> CATTLE_SERVICE_PORT_80_TCP_PORT=80 CATTLE_SERVICE_PORT_80_TCP_PROTO=tcp CATTLE_SERVICE_SERVICE_HOST=<IP_ADDRESS> CATTLE_SERVICE_SERVICE_PORT=80 CATTLE_SERVICE_SERVICE_PORT_HTTP=80 CATTLE_SERVICE_SERVICE_PORT_HTTPS=443
INFO: Using resolv.conf: nameserver <IP_ADDRESS> search cattle-system.svc.cluster.local svc.cluster.local cluster.local sys.comcast.net cable.comcast.com ula.comcast.net options ndots:5
INFO: https://rancher.test.com/ping is accessible
INFO: Value from https://rancher.test.com/v3/settings/cacerts is an x509 certificate
time="2018-11-05T19:59:51Z" level=info msg="Rancher agent version v2.1.1 is starting"
time="2018-11-05T19:59:51Z" level=info msg="Listening on /tmp/log.sock"
time="2018-11-05T19:59:51Z" level=info msg="Connecting to wss://rancher.test.com/v3/connect/register with token "
time="2018-11-05T19:59:51Z" level=info msg="Connecting to proxy" url="wss://rancher.test.com/v3/connect/register"
time="2018-11-05T20:00:16Z" level=error msg="Failed to connect to proxy" error="websocket: close 1006 unexpected EOF"
time="2018-11-05T20:00:26Z" level=info msg="Connecting to wss://rancher.test.com/v3/connect with token "
time="2018-11-05T20:00:26Z" level=info msg="Connecting to proxy" url="wss://rancher.test.com/v3/connect"
time="2018-11-05T20:00:36Z" level=error msg="Failed to connect to proxy" error="websocket: close 1006 unexpected EOF"
time="2018-11-05T20:00:46Z" level=info msg="Connecting to wss://rancher.test.com/v3/connect with token "
time="2018-11-05T20:00:46Z" level=info msg="Connecting to proxy" url="wss://rancher.test.com/v3/connect"
time="2018-11-05T20:01:07Z" level=error msg="Failed to connect to proxy" error="websocket: close 1006 unexpected EOF"
time="2018-11-05T20:01:17Z" level=info msg="Connecting to wss://rancher.test.com/v3/connect with token "
time="2018-11-05T20:01:17Z" level=info msg="Connecting to proxy" url="wss://rancher.test.com/v3/connect"

Seeing the issue on our cluster as well @superseb here are the logs you requested:
Update:
Switched to the helm chart deploy and we were able to get rid of that error message, but it's been replaced with another.

In the logs for rancher/rancher:cattle we are seeing the following:
And in rancher/rancher:rancher we are seeing the following:

Hi, What is the network mode do U used? flannel or calico?

I have the same issue and opened different ticket 16687
Any news about this?
#16687

Update to 2.1.2 resolved my problem

@penevigor Did that solve the underlying problem or just get rid of the error message? You’re able to get rid of the error by destroying the cluster but that doesn’t truly prevent the issue from occurring in the future.

Update to 2.1.2 resolved my problem

@penevigor Did that solve the underlying problem or just get rid of the error message? You’re able to get rid of the error by destroying the cluster but that doesn’t truly prevent the issue from occurring in the future.

That solved the problem. I can see my clusters in the UI now, they are not Unavailable. 2.1.1 brought the issue and 2.1.2 resolved it. Unfortunately, in the meantime I spent weeks in troubleshooting without luck.

Got the same error, when our rancher instance crashed on memory exceeded. This instance was "busy" by a couple of hours.
Rancher logs shows
failed with : failed to start user controllers for cluster c-xyz: failed to contact server: Get https://x.x.x.x:443/version?timeout=30s: waiting for cluster agent to connect
so we checked @superseb advice

waiting for cluster agent to connect indicates the cluster-agent not being able to connect, the logging of the cattle-cluster-agent pod should reveal why it cannot connect.

so the next step was to login via ssh to our controller instance, check running Docker containers and find k8s_cluster-register_cattle-cluster-agent-7d87f84646-bdqtp_cattle-system_c82febcf-f652-11e8-94fe-06b939b2213e_5
this container stuck on trying to connect to rancher server 2h ago, docker restart ... and everything back to normal

waiting for server-url setting to be set is Rancher waiting for you to login and set the server-url, and unrelated to this issue.
The cluster-agent should be the first target of root causing the original issue, it either can;t connect to the configured server-url (see the container logs) or it can't be scheduled/was evicted from a node.

installed rancher created project, namespace, deployment, restarted docker and now i get 404 error instead of rancher web.
"waiting for cluster agent to connect..."
What to do ?


build-profile: syntax-check: bump panc compiler version to 10.7

@jrha required so we can use new syntax rules from new compiler (e.g. error() function with more than one argument)


Issue installing software: "UserWarning: An issue occurred while importing 'torch-sparse'. Disabling its usage."

I would like to install a package called KGWAS and run its demo code https://github.com/snap-stanford/KGWAS/blob/master/demo/kgwas_101.ipynb. I have installed it and all its dependencies , however, when I test it, it always returns me with an error message (attached below). Would you please help me with this issue?
Thank you very much!
python
Python 3.10.14 | packaged by conda-forge | (main, Mar 20 2024, 12:45:18) [GCC 12.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys
>>> sys.path.append('../')
>>>
>>> from kgwas import KGWAS, KGWAS_Data
/home/.conda/envs/kgwas_gpu/lib/python3.10/site-packages/torch_geometric/typing.py:124: UserWarning: An issue occurred while importing 'torch-sparse'. Disabling its usage. Stacktrace: /home/.conda/envs/kgwas_gpu/lib/python3.10/site-packages/torch_sparse/_convert_cpu.so: undefined symbol: __kmpc_fork_call
warnings.warn(f"An issue occurred while importing 'torch-sparse'. "
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/home/.conda/envs/kgwas_gpu/lib/python3.10/site-packages/kgwas/_init_.py", line 2, in <module>
from .kgwas import KGWAS
File "/home/.conda/envs/kgwas_gpu/lib/python3.10/site-packages/kgwas/kgwas.py", line 22, in <module>
from .model import HeteroGNN
File "/home/.conda/envs/kgwas_gpu/lib/python3.10/site-packages/kgwas/model.py", line 7, in <module>
from .conv import GATConv
File "/home/.conda/envs/kgwas_gpu/lib/python3.10/site-packages/kgwas/conv.py", line 7, in <module>
from torch_sparse import SparseTensor, set_diag
File "/home/.conda/envs/kgwas_gpu/lib/python3.10/site-packages/torch_sparse/_init_.py", line 18, in <module>
torch.ops.load_library(spec.origin)
File "/home/.conda/envs/kgwas_gpu/lib/python3.10/site-packages/torch/_ops.py", line 1350, in load_library
ctypes.CDLL(path)
File "/home/.conda/envs/kgwas_gpu/lib/python3.10/ctypes/_init_.py", line 374, in _init_
self._handle = _dlopen(self._name, mode)
OSError: /home/.conda/envs/kgwas_gpu/lib/python3.10/site-packages/torch_sparse/_convert_cpu.so: undefined symbol: __kmpc_fork_call
>>> data = KGWAS_Data(data_path = './data/')
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
NameError: name 'KGWAS_Data' is not defined
>>> data.load_kg()
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
NameError: name 'data' is not defined

Hello, just pinging this issue to see if there are any updates? Cheers.

Hello! Yes I can confirm I followed those linked instructions. This is what I get when I import torch-sparse. And yes, I double checked and openMP is enabled.
python -c "import torch; print(torch.__version__)"
2.5.1+cu124

i see - can you try from torch_sparse import coalesce?

Hi, doing so returns
python -c "from torch_sparse import coalesce"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/username/rds/hpc-work/miniconda3/envs/pytorch2/lib/python3.10/site-packages/torch_sparse/__init__.py", line 39, in <module>
    from .storage import SparseStorage  # noqa
  File "/home/username/rds/hpc-work/miniconda3/envs/pytorch2/lib/python3.10/site-packages/torch_sparse/storage.py", line 7, in <module>
    from torch_sparse.utils import Final, index_sort
  File "/home/username/rds/hpc-work/miniconda3/envs/pytorch2/lib/python3.10/site-packages/torch_sparse/utils.py", line 5, in <module>
    import torch_sparse.typing
  File "/home/username/rds/hpc-work/miniconda3/envs/pytorch2/lib/python3.10/site-packages/torch_sparse/typing.py", line 2, in <module>
    import pyg_lib  # noqa
  File "/home/username/rds/hpc-work/miniconda3/envs/pytorch2/lib/python3.10/site-packages/pyg_lib/__init__.py", line 34, in <module>
    load_library('libpyg')
  File "/home/username/rds/hpc-work/miniconda3/envs/pytorch2/lib/python3.10/site-packages/pyg_lib/__init__.py", line 31, in load_library
    torch.ops.load_library(spec.origin)
  File "/home/username/rds/hpc-work/miniconda3/envs/pytorch2/lib/python3.10/site-packages/torch/_ops.py", line 1350, in load_library
    ctypes.CDLL(path)
  File "/home/username/rds/hpc-work/miniconda3/envs/pytorch2/lib/python3.10/ctypes/__init__.py", line 374, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: /lib64/libm.so.6: version `GLIBC_2.29' not found (required by /rds/user/username/hpc-work/miniconda3/envs/pytorch2/lib/python3.10/site-packages/libpyg.so)

yes, looks like the pyg is not installed correctly. can you try re-install the pyg and related package? you can use the pip version, which i personally found it more bug-free.
For example, you will need to make sure that the pytorch, OS, and cuda version match with what you have.
For example, for pytorch 2.5, linux and cuda = 12.4, install pytorch first and then do:
pip install torch_geometric
pip install pyg_lib torch_scatter torch_sparse torch_cluster torch_spline_conv -f https://data.pyg.org/whl/torch-2.5.0+cu124.html

I also ran
nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2021 NVIDIA Corporation
Built on Wed_Jul_14_19:41:19_PDT_2021
Cuda compilation tools, release 11.4, V11.4.100
Build cuda_11.4.r11.4/compiler.30188945_0

And
gcc --version
gcc (GCC) 8.5.0 20210514 (Red Hat 8.5.0-22)
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

as well as
ldd --version
ldd (GNU libc) 2.28
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
Written by Roland McGrath and Ulrich Drepper.

Let me know if any of this helps.


Add delete all classes functionality to ClassDeleter

The python client has a function to delete all existing classes from a schema. This is helpful if you want o clean things up before wrinting a test.
It can be done the following way:

Get existing schema with all classes
Delete all those classes one by one

Python example, Go example from the weaviate benchmark

already exists


NOISSUE - Update edge docuementation

What does this do?
updates export test example
Which issue(s) does this PR fix/relate to?
Relates to https://github.com/mainflux/export/pull/37 and https://github.com/mainflux/agent/pull/56
List any changes that modify/break current functionality
Have you included tests for your changes?
Did you document any new/modified functionality?
Notes

Can this be changed to draft up until https://github.com/mainflux/export/pull/37 and https://github.com/mainflux/agent/pull/56 are merged or add DRAFT -  prefix


Friend Requests - Sending invitations

Is your feature request related to a problem? Please describe.
Systematically we are solving the problem of accountability using the Friend-related features.
Describe the solution you'd like
The user can send friend requests to anyone available in the "Friends" toggle tab of the Friends menu.

I think there's some bug in the Friends module. I'm not able to add friends.

Same problem here, maybe we need firstly located where is the problem, frontend, backend or database?

I think it is possibly an issue of the click elements not linking with the backend. I'll look into the scripts.js to check the click functions.


health check blocked when update or upgrade container

Rancher versions:
rancher/server: v1.6.10
rancher/agent: v1.2.6
Infrastructure Stack versions:
healthcheck: v0.3.3
ipsec: rancher/net:v0.11.9, rancher/net:holder
network-services: rancher/metadata:v0.9.4, rancher/network-manager:v0.7.8
scheduler: v0.8.2
**Docker version: 17.06.2-ce **
**Operating system and kernel: ubuntu:16.04 **
Type/provider of hosts: aliyun
Setup details: HA rancher, external DB)
Environment Template: Cattle
Steps to Reproduce:

container health check info:
"healthCheck": {
"type": "instanceHealthCheck",
"healthyThreshold": 2,
"initializingTimeout": 60000,
"interval": 1000,
"name": null,
"port": 8989,
"recreateOnQuorumStrategyConfig": null,
"reinitializingTimeout": 60000,
"requestLine": "GET "/" "HTTP/1.0"",
"responseTimeout": 1000,
"strategy": "none",
"unhealthyThreshold": 3
},

1.in container : curl  localhost:8989/sleep (sleep 100s)
2.stop container on rancher UI

3.look at Processes health check blocked.

4.traefik not remove graceful-shutdown container

I want graceful-shutdown my app.
What should i do ?

We recently made a first effort to do draining on the included haproxy loadbalancer here: http://rancher.com/docs/rancher/v1.6/en/cattle/adding-load-balancers/#draining-target-connections-for-a-load-balancer
Regarding traefik, this is something you have to take care of. e.g. watching for the stop event and telling traefik to lower the weight to drain connections.


Move the demos to a different repo

Resons
Better organisation
This makes it easier to find demos and adds the benefit of being able to test updates without changing the project
Creating a clean project containing (for example) the FPS demo and having the addon as a dependency (maybe even a Github Submodule? See here) would make things considerably more accessible.
One of the main reasons people avoid great addons like these is the lack of documentation. Add to it the fact that the demos are hidden deep within the project; it is easy to see why people turn away, no matter how great this addon is.
Engine behavior
Godot currently loads the demo projects and their classes, which might not be desired. This is especially bothersome when class names override each other. (ie. class_name Player )
Add to this the fact that inexperienced Godot users may try to use this addon to simplify their project, the confusion of why things break, how to use it, etc. they're more likely to turn away from the addon or even the engine.
Packaging size
Having the demos in a different repo avoids packaging all the demo files in the final product. I'm aware that it is possible to exclude files from being packed, but due to the engine behaviors mentioned above, this could and most likely will break the project in unexpected ways.

I agree 100% with this approach, I was currently analyzing with Ark2000/PankuConsole about mirroring the repository to be used as a submodule.

Created (https://github.com/expressobits/inventory-system-demos) and sent to the asset lib as a project, the addon will only be updated once the project is approved in the asset lib

Move addons/inventory system contents to root folder (submodule purposes) in main and develop branch


CLOUDP-70010: iam users describe

Proposed changes

Jira ticket: CLOUDP-70010

Checklist

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation (if appropriate)
[x] I have run make fmt and formatted my code

Further comments
Usage:
  mongocli iam users describe [flags]

Aliases:
  describe, get

Examples:

  Describe a user by ID
  $ mongocli iam users describe --id <id>

  Describe a user by username
  $ mongocli iam users describe --username <username>

OM:
./bin/mongocli iam users describe --username<EMAIL_ADDRESS>--profile om
ID                         FIRST NAME   LAST NAME   USERNAME
5f4579d04bba012f25a03212   rob          carlan<EMAIL_ADDRESS>
Atlas:
./bin/mongocli iam users describe --username<EMAIL_ADDRESS>ID                         FIRST NAME   LAST NAME   USERNAME
5c80f75fcf09a246878f67a4   Rob          Carlan<EMAIL_ADDRESS>

@gssbzn ready for review

I will address those comments here https://github.com/mongodb/mongocli/pull/389


Incorrect convert equation to an expression set to zero

Equation y=4x^2+xy
KAS.parse('y=4x^2+xy').expr.asExpr().print() = y+-4*x^(2)+-1;
[should be y+-4x^(2)+-1x*y]
problem there https://github.com/Khan/KAS/blob/master/kas.js#L3852

While not fixed, we can do:
var compare = expr.left.eval({x: x, y: y}) - expr.right.eval({x: x, y: y});
// Math.sin((Math.PI)) === 1.2246467991473532e-16
if (compare >=-1e-15 && compare <= 1e-15)
And about 1e-15, maybe implement it in library to?

Hmm, I'm not seeing that behavior. Here's what I get:
> KAS.parse('y=4x^2+xy').expr.asExpr().print();
"y+-4*x^(2)+-1*x*y"

Could you check to make sure that you're using the latest version of KAS?

I'm using last version from master branch.
Another question:
sin(pi)!=0 so how can I compare equation y=x*sin(pi) so that equation was true: 0=1234567*sin(pi)

Same here - I'm using the current master branch. Perhaps it's a browser issue then? Any information about how you're executing this could be helpful.
As for your second question, we actually already do what you suggest (compare with a tolerance, instead of requiring an exact match):
> KAS.parse('sin(pi)').expr.eval()
1.2246467991473532e-16
> KAS.compare(KAS.parse("sin(pi)").expr, KAS.parse("0").expr).equal
true

The relevant code for this can be found here: https://github.com/Khan/KAS/blob/master/kas.js#L1825-L1830

I was using lodash instead of underscore, when switched to "underscore" all works fine. Thanks!

KAS.compare(KAS.parse("12341234*sin(pi)").expr, KAS.parse("0").expr).equal === false, But sin(pi)=0?

@zagovorichev Like @alopatin said, evaluating sin(pi) does not give exactly 0, because of floating point accuracy. (It's the same thing that JavaScript gives: Math.sin(Math.PI) gives 1.2246467991473532e-16) It's pretty close to 0, and when we do comparisons we use a tolerance (of 10e-9 it looks like: https://github.com/Khan/KAS/blob/master/src/nodes.js#L301), so when numbers are very close, they will compare equal. 12341234*sin(pi) is just not close enough.

@xymostech Oh, I understand it all. But, for me, it is incorrect behavior. Can be used, as solution, static library, where {'sin(pi)' : 0}? Or replace Math.sin in Math.sin1 with approximation...
Thank you for help, sin(pi) incorrect, and it is sad

It might be interesting to include trigonometric knowledge in the evaluator.  Since we have the parse tree we could determine if the argument to sin simplified to a multiple of pi and return 0 in those cases.

This will be great! nobody did it for now. (At least I have not found solutions)

Hello, all
I am a very beginner in JavaScript and I want to use KAS locally on my browser to test it. Would anyone help me by mentioning how to include it in an html file step by step?


trigger when scheduled task / USB backup happens?

Is there a way to trigger an automation based on events in OMV? I'm thinking primarily about start/finish of scheduled tasks, and e.g. finish of an USB Backup job.
If that's not possible currently, how could it be implemented?

no, it is not possible.
cron jobs are unlikely to be ever possible.
usb backup is a plugin, that is not supported at all by the integration

Can't something regarding this be coordinated with th OMV developers? I've found them quite responsive and welcoming.

not really, its not in their scope.
cron jobs are lx, those are not monitored this way and they dont have persistant states.
for plugins, there would have to be significant interest and willing contributors for me to add plugin support and extend framework.
I do not use OMV for years already and I'm keeping this integration maintained just because there is nothing else for HA. It would be a loss for people to not have any way to integrate OMV. However, adding features that require significant research and work is a different story. While it is not something I'm against, it is also something I'm not willing to do as a solo integration developer.

OK, fair enough. I'll have to roll with what we have then.


Search for text nodes with {p#} as title and auto update text

It would be amazing to allow for the text node name to be labeled as "{p#}", allowing for the designer to reorder the frames and re-run the plugin, which updates the number (and order).
I'm assuming this filter needs to be updated.
https://github.com/dgngulcan/pager/blob/3c54fb3ca2094743d0fe0b3446bdd2b6b28f439d/src/code.ts#L15

hi @karbassi , the label you suggested looks like the same as the existing label -- {p#}
can you elaborate please? thanks!

Hello @dgngulcan, it's been a while since I wrote this issue. I honestly don't know what I meant by this.
I should have written a better issue. Go ahead and delete this.


cri: validate if sandbox task is in correct state during creating container

When getting shim's state times out, method Task() returns with success and an unknown state task response. The client should be aware of the unknown task state and process the situation very carefully.
If we don't validate the task response, the sandbox task pid is ZERO which is later passed to the business container and requests it to join the namespace like /proc/0/ns/ipc
An error message has been caught:
306-2d08-4325-b876-79112929737f)"), skipping: failed to "StartContainer" for "cp-cmd" with RunContainerError: "failed to create containerd task: OCI runtime create failed: container_linux.go:364: creating new parent process
s caused: container_linux.go:2005: running lstat on namespace path \"/proc/0/ns/ipc\" caused: lstat /proc/0/ns/ipc: no such file or directory: unknown"

If we don't validate the task response, the sandbox task pid is ZERO which is later passed to the business container and requests it to join the namespace like /proc/0/ns/ipc.

Thanks for sending this patch. Totally, agree. There is mismatch between s.Container.Task and s.sandboxStore. The s.sandboxStore caches the status which isn't updated in time. It's good to have a validation on the status. If the task(shim) isn't in health status, the t.Pid() is not reliable. And it can be worse case about pid recycle. The sandbox pause process exits after creating oci spec. We should also check the task status during StartContainer API, which ensures that pause pid is valid.

If we don't validate the task response, the sandbox task pid is ZERO which is later passed to the business container and requests it to join the namespace like /proc/0/ns/ipc.

Thanks for sending this patch. Totally, agree. There is mismatch between s.Container.Task and s.sandboxStore. The s.sandboxStore caches the status which isn't updated in time. It's good to have a validation on the status. If the task(shim) isn't in health status, the t.Pid() is not reliable. And it can be worse case about pid recycle. The sandbox pause process exits after creating oci spec. We should also check the task status during StartContainer API, which ensures that pause pid is valid.

Thanks @fuweid for pointing it out. I will improve the PR to address the concerns.

Hi guys we are encountering similar issue, in our case, shim will indeterminably fail. (works fine for sandbox and init container but fails for main container)
my concern is that what if shim timed out when filling in the pid but recovered and workes fine for ss.Status.
So, instead of introducing another shim call (ss.Status), maybe we should expand the task struct

to include the status

Hi guys we are encountering similar issue, in our case, shim will indeterminably fail. (works fine for sandbox and init container but fails for main container)
my concern is that what if shim timed out when filling in the pid but recovered and workes fine for ss.Status.
So, instead of introducing another shim call (ss.Status), maybe we should expand the task struct  to include the status

Thanks, it's a good point. Furthermore, we might need a more sophisticated mechanism to detect the shim's liveness and a framework to retry requesting Task service,


No way to set order for custom rules

Rules ordering is done on validation and custom rules adding occurs before. If so, custom rules always have more priority and that's not true.
Maybe, it's better to store order inside the Rule? So developer could add custom rules at runtime with proper priority.

Hi,
it`s possible to define the validation order with v2? If yes, how?

@gmarques33, Just add the @Order annotation on all your views and add a quick rule to a view using validator.put(View, QuickRule...)

before : validator.put(editText, Rules.required(Messages.ES_REQUERID_FIELD, true));
after : validator.put(View, QuickRule...)   ??? :|

public class V_RadioGroupButton extends Rule {
/**
 * Creates a new validation Rule.
 *
 * @param failureMessage The failure message associated with the Rule.
 */
public V_RadioGroupButton(String failureMessage) {
    super(failureMessage);
}

@Override
public boolean isValid(RadioGroup view) {
    view.setBackgroundColor(Color.WHITE);
    return view.getCheckedRadioButtonId() > 0;
}

}
How to QuickRule? :( documentation please

@jhonvidal You need to use v2 of the library to use ordering with QuickRules.

Thanks for responding, I'm using the v2, the problem is how to interpret the rules

@NotEmpty(sequence = 1)
@Email(sequence = 2)
// We are going to add a new QuickRule with a sequence number '3'
@Order(1)
private EditText mEmailEditText;

@Password
@Order(2)
private EditText mPassword;

@ConfirmPassword
@Order(3)
private EditText mConfirmPassword;

// ----------------------------------------------------------------------------------------------------------
// Possibly inside your onCreate(Bundle) method....
validator.put(mEmailEditText, AllowMyDomainOnlyQuickRule(3));

// ----------------------------------------------------------------------------------------------------------
// This is your quick rule, you can also instantiate an anonymous one
static class AllowMyDomainOnlyQuickRule extends QuickRule {
    // Code ...
}

Please check this comment for a complete idea. Ordering and sequenceing is different in v2 from v1's order attribute.

Great help, thank you very much !!!  @ragunathjawahar

@jhonvidal You're welcome :)

a detail :
these lines :
@NotEmpty(sequence = 1)
@Email(sequence = 2)
@Order(1)
private EditText mEmailEditText;

How is interpreted programming in code? please an example of native rules
:) Thank

I did this for required field :
public class AllowRequiredQuickRule extends QuickRule {

    private static AllowRequiredQuickRule instance = null;

    public static AllowRequiredQuickRule getInstance(int i) {
        if (instance == null) {
            instance = new AllowRequiredQuickRule(i);
        }
        return instance;
    }

    public AllowRequiredQuickRule(int i) {
        super(i);
    }

    @Override
    public boolean isValid(View view) {
        return false;
    }

    @Override
    public boolean isValid(Object o) {
        EditText editText = (EditText) o;
        return !editText.getText().toString().isEmpty();
    }

    @Override
    public String getMessage(Context context) {
        return Messages.ES_REQUERID_FIELD;
    }
}

:+1:


[Debug filter] View gets empty when the input is empty

#105185
Issue Type: Bug

Type something into the filter
Clear the filter (delete all text)
The debug console is empty now

VS Code version: Code - Insiders 1.49.0-insider (6384e3246884694cbc34eacc70281ada15e2c7f2, 2020-08-28T16:02:44.602Z)
OS version: Windows_NT x64 10.0.18363

/assign isidorn

I also can not reproduce this after my latest refactorings. Do you still see this with latest?
And thank you very much for testing it.

f66741d No difference. Still reproduces.

I can not reproduce with latest out of source. Also can not repro when I build insiders from latest locally. I believe the next insiders will have the fix.
If you still see it tomorrow please let me know. Thanks a lot! For now closing

❌ Not fixed.

Ok, let's reopen.

Repro steps:
make sure to have editor.accessibilitySupport: "on" in your settings.json


[Feat]: Auto Commenting when an Issue is Closed

This feature aims to solve the problem of poor communication and lack of acknowledgment when issues are closed. By automatically commenting on closed issues, it ensures that contributors are informed about the closure and are thanked for their efforts. This helps maintain a positive and engaging community atmosphere, encouraging continued contributions and fostering good relationships between maintainers and contributors.

@sunny0625 please assign me this issue

@sunny0625 please assign me this issue

@sunny0625 please assign me this issue

Does this feature close Genuine issues automatically if we delay an assignment or reply?


Fix CQL port to expose to external clients

rpc_address must be set to <IP_ADDRESS> in order to enable port forwarding.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Fix provisioning engine crash on-prem

Web.GetProvisioningTemplate crashes for on-prem blank site, because AlternateCSS and SiteLogoUrl properties are not initiated

Hi @ivanvagunin, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Thanks Ivan. Appreciate you're testing in on-premises....the team has not yet had time to do a lot of on-premises validation. Keep the fixes coming ;-)


update application to 0.0.48

UpdateBot pushed helm dependency: application to: 0.0.48

/test this
/approve


Examples?

Probably put some use examples in as well to keep this page consistent with the rest of the documentation.

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

ID: fbdace9d-6fa0-0aec-25b4-df6f414001d7
Version Independent ID: 77eff122-b9a5-2b1f-3a68-df302cb46194
Content: Set Operations (C#)
Content Source: docs/csharp/programming-guide/concepts/linq/set-operations.md
Product: dotnet-csharp
Technology: csharp-linq
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

I think I can make some sample for this
It looks like code is managed in here, but how can I link with Set Operations (C#) article??
Should I just make duplicate code for article and dotnet/sample?

Thanks for writing this @NickPea We should add examples here. I've added this to our backlog, and added the "up for grabs" label.
@SuperNova911 Thanks for asking about contributing to fix this issue. I've got a few thoughts on the samples:

These could be good candidates for the interactive experience.
Most of our samples in this area were migrated from an internal system, and don't match how we're creating samples now. I'd recommend adding samples in a new LINQ folder under the [snippets/csharp/(https://github.com/dotnet/samples/tree/master/snippets/csharp) folder in the samples repository.

Fixing this would be two PRs, one in the dotnet/samples repo, and one in this repo to include the new examples.
@ - mention me with any questions.


Docker container for the project

Allow deploying the bot via a Docker container (MongoDB may need to be used instead of the current filesystem)
Links of interest:

https://code.visualstudio.com/docs/containers/overview
https://www.vultr.com/docs/how-to-run-a-discord-js-bot-on-a-docker-application/
https://medium.com/devschile/dockerizando-nodejs-y-mongodb-ad24fedab8f2
https://stackoverflow.com/questions/50053313/how-to-make-ffmpeg-available-inside-my-docker-container
https://docs.docker.com/develop/develop-images/multistage-build/#use-multi-stage-builds
https://www.infoq.com/articles/docker-executable-images/

Is being worked on the new-backend-system branch.


Add multi-stage Dockerfile to Kraken

Add new Dockerfile that uses multi-stage build features and techniques to support building multiple Alpine-based Kraken
containers of various persuasions, from a full Go dev/build environment all the way down to just the /sbin/kraken executable
itself.

Also add a .dockerignore file (analogous to .gitignore) to prevent Git and CircleCI metadata trees from being inadvertantly
included in the containers/repositories.

Add default config.yaml and state.json config files to the .gitignore list to help facilitate building containers #4 and #5
(those which are pre-populated with the aforementioned files from the top-level directory) without having them appear untracked.

Add script-generated "kitchen sink" config file to .gitignore for much the same reason (and to avoid accidentally committing it).

Note:  The simplest and most straight-forward way to build and tag all the repositories correctly is to copy-and-paste this command and run it from the top-level kraken tree:
(for STAGE in "-t kraken-dev-go${GOVER:-1.15}-alpine:latest --target kraken-build" "-t kraken-alpine:latest --target kraken-alpine" "-t kraken-exe --target kraken-exe" ; do docker build --pull ${STAGE} . ; done ; if [[ -s config.yaml && -s state.json ]]; then for STAGE in "-t kraken-dev-go${GOVER:-1.15}-alpine-cfg:latest --target kraken-build-cfg" "-t kraken-alpine-cfg:latest --target kraken-alpine-cfg" ; do docker build --pull ${STAGE} . ; done ; fi)

Here's the same shell command minus the subshell and broken out for readability:
for STAGE in "-t kraken-dev-go${GOVER:-1.15}-alpine:latest --target kraken-build"
             "-t kraken-alpine:latest --target kraken-alpine"
             "-t kraken-exe --target kraken-exe"
 do
    docker build --pull ${STAGE} .
done
if [[ -s config.yaml && -s state.json ]]; then
    for STAGE in "-t kraken-dev-go${GOVER:-1.15}-alpine-cfg:latest --target kraken-build-cfg"
                 "-t kraken-alpine-cfg:latest --target kraken-alpine-cfg"
    do
         docker build --pull ${STAGE} .
    done
fi

@mej Can you address the linting issues?

Linting comes back clean now (grumble), so you should be good to merge at your discretion. 🙄 😆

This gives a good basis to iterate on.  Merging.  Note: the location of the Dockerfile will probably change in the future, but this is fine for now.


Proposal: --wip-tags TAG_EXPRESSION

This is a proposal to allow for "work in progress" scenarios to fail.
Context: Occasionally, a team will work on a scenario that depends on other functionality (possibly out of their control) to be implemented before the scenario can pass. For example, a UI needs some new data from a backend. The UI work is completed, but the backend hasn't yet been updated to provide the data.
In this context, the scenario could be marked with a tag (for example @wip for "work in progress") and run with --wip-tags @wip.
This would allow developers to commit the half-finished scenario and allow the build to stay green. As soon as the scenario fails (when the missing functionality is implemented), the build would fail. This is an indication that the @wip tag should be removed from the (now completed) scenario.
It should be noted that some Cucumber implementations already have a --wip option, which is similar. The drawback of this option is that it requires a separate invocation of Cucumber. If this proposal is accepted, we should deprecate (and ultimately remove) the --wip option.

If this proposal is accepted, we should deprecate (and ultimately remove) the --wip option.

I don't quite follow this. It would appear to me that --wip and --wip-tags are different work flows.

Could we stream line this to --wip "@Unfinished" and have --wip imply that everthing is unfinished?

I'm not sure WIP features should have to fail - for instance they may be flakey. perhaps a second category of result / some sort of warning would be better?
Essentially, something unexpectedly working shouldn't be a suite failure to me.

Could we stream line this to --wip "@Unfinished" and have --wip imply that everthing is unfinished?

Yes, I think that's a good idea.

I'm not sure WIP features should have to fail - for instance they may be flakey. perhaps a second category of result / some sort of warning would be better?

So when a WIP scenario suddenly starts passing, print a warning? Yes I think that might be better. Would this mean a new status for a scenario? wip-success? And then print scenarios with this status as yellow and still pass the build?

Would this mean a new status for a scenario? wip-success? And then print scenarios with this status as yellow and still pass the build?

Yes, something along those lines. I was picturing frustrated devs trying to merge their PRs and not being able to because 'it works more than it's meant to!'

Nobody reads build logs though. If the feature was implemented and now works but the wip tag not removed it is a failure to update the documentation.

It could fail by default and pass if —no-strict is specified.

https://github.com/cucumber/cucumber-jvm/pull/1381
The interaction between wip and strict is already complicated. :D

I’m sure it can be simplified

Could we stream line this to --wip "@unfinished" and have --wip imply that everthing is unfinished?

On second thought I don't think this is possible. Consider the following argument lists:
--wip @foo
--wip features/foo.feature

For the first one it's clear that @foo is an argument to --wip. For the second it's clear that features/foo.feature is not an argument to --wip.
For the optparse code however, building this knowledge in is tricky. It would have to interpret the 2nd argument and decide whether it's a tag expression or a file, and I don't think an option parser should have that kind of knowledge.
For this reason, I don't think options should have optional arguments. We have the following alternatives:

Change --wip to expect a mandatory argument (this would break backwards compatibility and lead to some confusion)
Introduce a new --wip-tags option with a mandatory argument and deprecate --wip.

I think the latter is better.

I kind of see a true "work in progress" may mean that I'm writing steps within my feature file, but I may have undefined step definitions. Would a wip-tag also be the appropriate tool to effectively "skip" the entire scenario? Perhaps I'd still like knowledge of the scenario's existence within my report.. as if it were a "TODO" in code.
I say this as in my experience, we have occasions where someone else is writing the feature file than the person implementing the step definitions. We implement the feature file (which has undefined steps) as a '.future' file such that it can be baselined, but for obvious reasons would not execute. It'd be interesting to explore the idea of having a WIP tag that'll "print" the scenario to the report but not impact any test evaluation (build errors, etc.). This could be very effective when coupling cucumber with other tools such as cucumber reporters for Jenkins. Thoughts?

I think that use case is already partially covered by the PENDING state (achieved by throwing a pending exception). Depending on the --strict and --wip settings that may fail the build.
Currently the truth table of scenario result x strict x wip looks like:
           |       --wip            |       --no-wip         |
           | --strict | --no-strict | --strict | --no-strict |
PASSED     |   fail   |  fail       |   pass   |  pass       |
SKIPPED    |   pass   |  pass       |   pass   |  pass       |
PENDING    |   pass   |  pass       |   fail   |  pass       |
UNDEFINED  |   pass   |  pass       |   fail   |  pass       |
AMBIGUOUS  |   pass   |  pass       |   fail   |  fail       |
FAILED     |   pass   |  pass       |   fail   |  fail       |

It's up to the implementation of a reporter how they display each scenario.


Change structure project for "cazanw" binary at installation of cazanw with "npm install"

Anything else @hamirmahal 😄😄?

I'll implement the prebuild before merging the PR


fix Module component to support HMR

Closes #17
I was able to get HMR working by restructuring the componentWillMount() method of the Module component. Lints properly and all tests pass.

Thanks for digging in the code to fix this! 🎉 Feel free to contribute more.


if the server whitch run testcases is not very strong and use default settings the most testcases will fail

i think the default settings witch run 8 cases paralleled without set --parallel_running_case_number , container will not work well on the limited mermery resource

The default setting is based on our test vm.

i just want help somebody else who write correct code but run testcases failed ~

Yes. Do you have any thoughts to improve this repo? I am glad to hear.

for short term i think we can update doc to tell developer the parameter --parallel_running_case_number,
for long term,i think the default parallel_running_case_number can be dynamically by cpu num and free memery size

Make sense. Please do.


DropIn accepts credit cards payments that are not enable in our merchant account

We had a issue over the weekend where someone tried using there amex rather than visa the DropIn-UI accepted / validated there card but it only failed server side. Is there away to configure the dropin ui to not accept specific cards?

Unfortunately, Drop-in does not have the ability to reject specific card networks.
If you use the onPaymentMethodReceived callback, you can see the card network entered by the user under details.cardType and advise the user to enter a different payment method.
braintree.setup(TOKEN, 'dropin', {
  onPaymentMethodReceived: function (payload) {
    if (payload.details.cardType === 'American Express') {
      showAmexNotAccepted();
    }
  }
});

Hi Eric, thank you for your response - would you please add this capability ASAP?  It has caused a very poor customer experience where the customer is only informed that the card type they tried is not accepted AFTER going through the full payment process - it would be a significantly better customer experience for the Drop-In to advise the customer as soon as the customer starts to enter their card based on the first 4 numbers being entered - before the customer attempts to process the payment.

While we won't be adding this feature to the current iteration of Drop-in, this is an oft-requested feature that we're considering for the next version.

Hi, we are planning on adding this feature to Drop-in. We will keep this GitHub issue updated when we can commit to a specific release date.

Quick update on this issue, the new version of Drop-in shows your accepted card type icons in the UI and will not accept numbers of card types not enabled in your merchant account.


How can we read and write to database from TagUI?

Raising this from an user email for sharing, as other users might benefit from this question.

Do you think you can provide very simple script samples on how TagUI can make the connection directly to DB?

Yes this should be possible using the run step to run specific DB command for your database.
It depends on your variant of database for different types of CLI, for eg mysql, sqlcmd. Let me put together a sample for mysql for a database on my server, to read some data and write some data.

TagUI automation file
// form sql query to read from database and save to os script
// on mac/linux be sure that read_db file has execute permission
sql_query = 'mysql DATABASE_NAME -e "SELECT NAME,EMAIL FROM TABLE_NAME"'
dump sql_query to read_db

// use run step to run the script and retrieve results
run /full_path_to_flow/read_db
db_result = run_result.split('\n')
for row from 1 to db_result.length
{
        db_row = db_result[row-1].split('\t')
        echo db_row[0] ',' db_row[1]
}

// form sql query to write to database and save to os script
// on mac/linux be sure that read_db file has execute permission
sql_query = 'mysql DATABASE_NAME -e "UPDATE TABLE_NAME SET NAME=\'KS\' WHERE<EMAIL_ADDRESS>dump sql_query to write_db

// use run step to run the script to write
run /full_path_to_flow/write_db

// repeat the reading of database showing new data
run /full_path_to_flow/read_db
db_result = run_result.split('\n')
for row from 1 to db_result.length
{
        db_row = db_result[row-1].split('\t')
        echo db_row[0] ',' db_row[1]
}

Execution result
START - automation started - Sat Mar 31 2018 22:02:20 GMT+0800 (SGT)

dump sql_query to read_db
run /full_path_to_flow/read_db
NAME,EMAIL
Ken<EMAIL_ADDRESS>Starbucks<EMAIL_ADDRESS>dump sql_query to write_db
run /full_path_to_flow/write_db
run /full_path_to_flow/read_db
NAME,EMAIL
<EMAIL_ADDRESS>Starbucks<EMAIL_ADDRESS>
FINISH - automation finished - 0.7s

Above shows an example of forming SQL query in TagUI automation flow. It first reads from the database and then write something to it, before reading and displaying back the new result.
Notice that NAME value has been changed from 'Ken Soh' to 'KS' after the database write operation. Closing this issue for now unless there is new queries on this.
For now, it might not make sense to create a sql step to do DB operations like these because it is very specific to what type of DB and what type of operations to be general enough for a new helper step.

Hi @kensoh ,
I 'd like to consider Mysql Connection to database to automatize processes on own website. But in your above axample, I didn't anderstand how to configure connection with MySQL like: Host: .... Username=... ; passwords: ..., Database: .... Could you please indicate me the correct command line to insert in read_db file?
Thanks

Hi @adegard that's a good question, thanks! I've updated example above - https://github.com/kelaberetiv/TagUI/issues/112#issuecomment-377695716
The example assumes using root user from server. If database credentials are needed, something like below should be used. (reference - mysql command line options)
sql_query = 'mysql DATABASE_NAME --user=username --password=\'password\' -e "SELECT NAME,EMAIL FROM TABLE_NAME"'

I have some issues with fetching data from mysql database.
i make another folder 'sql_test' in samples.
Above Mention Code I write in a file and when i run this, it does not show any output.
My Path of script file is ->/home/pericent/tagui/src/samples/sql_test
path of read_db file is ->/home/pericent/tagui/src/samples/sql_test
sql_query = 'mysql temp_info --user=root --password=\'\' -e "SELECT NAME, ADDRESS FROM EMP"'
dump sql_query to read_db
run /read_db
db_result = run_result.split('\n')
for row from 1 to db_result.length
{
       db_row = db_result[row-1].split('\t')
       echo db_row[0] ',' db_row[1]
}

I recheck the existence of database and table. both are exist.
Username='root' and password =''

Hi, I am using this line but it only show myql options. it does not fetch data.
What should I do ???
sql_query = 'mysql temp_info --user=root password=\'\' --execute = "select name from emp"'

@kensoh I am trying to run the script for fetching data from mysql database.
but getting below error on command prompt.
START - automation started - Fri Oct 04 2019 17:45:25 GMT+0530 (India Standard Time)
dump sql_query to read_db
run /C:/app/read_db
Fatal Windows exception, code 0xc0000005.
PhantomJS has crashed. Please read the bug reporting guide at
http://phantomjs.org/bug-reporting.html and file a bug report.
Below query written in my automation file (read_db)
// form sql query to read from database and save to os script
// on mac/linux be sure that read_db file has execute permission
sql_query = 'mysql taguidemo --user=****--password='********' -e "SELECT MobileNo FROM tbluser"'
dump sql_query to read_db
// use run step to run the script and retrieve results
run /C:/app/read_db
db_result = run_result.split('\n')
for row from 1 to db_result.length
{
db_row = db_result[row-1].split('\t')
echo db_row[0] ',' db_row[1]
}

read_db.log
I am attaching log file and command prompt output. Please suggest and help

Hi @brahmanath this error is because the filename you specified is invalid -
/C:/app/read_db
In Windows, there is no / before the drive letter. Also, I think you have to name the read_db as read_db.bat so that it can be run by Windows. Something like below -
sql_query = 'mysql taguidemo --user=****--password='********' -e "SELECT MobileNo FROM tbluser"'
dump sql_query to read_db.bat

// use run step to run the script and retrieve results
run C:/app/read_db.bat

Hi Ken,
Thanks for response, I have change the thing that you have suggested, but still getting issue for the same. Kindly look command prompt screen shot and attached file for the same and it will be great appreciation if you suggest on the same.
Below query written in my automation file (read_db)
sql_query = 'mysql taguidemo --user=root --password='mysql@123' -e "SELECT MobileNo FROM tbluser"'
dump sql_query to read_db.bat
// use run step to run the script and retrieve results
run C:/app/read_db.bat
db_result = run_result.split('\n')
for row from 1 to db_result.length
{
db_row = db_result[row-1].split('\t')
echo db_row[0] ',' db_row[1]
}
read_db.log

Hi Ken,
Thanks for response. I have tested both scenario. When I running the query from command prompt, it is working fine and provide result but when I am running read_db.bat file then its generate error "ERROR 1045 (28000): Access denied for user '=root'@'localhost' (using password: YES)" while I have given full right to read_db.bat file. Please see the screen shot and suggest.

read_db.log

Hi Brahma, this is useful information, but I'm afraid it is related to the way Windows batch file works and I could not advise on how to troubleshoot this problem (running the command from prompt works but from batch file does not work).
You can try providing password as part of the command or google to see if other people have problem running mysql command from batch file. For my example in this issue, it is coded and tested to run on macOS and Linux using their shell scripts.


The hpi download link is broken

Jenkins and plugins versions report

Environment

Paste the output here

What Operating System are you using (both controller, and any agents involved in the problem)?
Mac and Centos
Reproduction steps
1.Download the hpi file using the link from the main Githu page here
Expected Results
File should be downloaded
Actual Results
The link is broken
<Error> NoSuchBucket <Message>The specified bucket does not exist.</Message> </Error>
Anything else?
The download link is broken

same here

https://github.com/jenkinsci/google-compute-engine-plugin#installation is just wrong. The plugin should be obtained from the Jenkins update center like any other.


Cloud-proxy container fails to startup and throws an error. May be fixed by https://github.com/pixie-io/pixie/pull/2016

Describe the bug
Currently the nginx.conf expects the a kubernetes dns service named kube-dns.kube-system.svc.cluster.local to be existing in the cluster. If this dns service does not exist the cloud-proxy pod gets terminated on startup.
To Reproduce
Steps to reproduce the behavior:

Go to https://docs.px.dev/installing-pixie/install-guides/self-hosted-pixie/
On executing step 12 to "Deploy Pixie Cloud",
kustomize build k8s/cloud/public/ | kubectl apply -f -
The cloud proxy pod fails to start with error
"host not found in resolver kube-dns.kube-system.svc.cluster.local in /usr/local/openresty/nginx/conf/nginx.conf:43"

Expected behavior
Cloud proxy Pod should startup successfully with existing kube dns if any.
Every kubernetes cluster can have different kube dns service names. Also some clusters can have core-dns. So the cloud proxy service should look up for the dns service specific to the cluster in which its being setup. The control for defining this this should be given to the user.
This might be fixed in https://github.com/pixie-io/pixie/pull/2016

This is covered by https://github.com/pixie-io/pixie/pull/2018 and will be available in the next cloud release.


Added descriptive Host Error messages to PortMidi backend

In the case of a PortMidi host error, uses PmGetHostErrorText, instead of just PmGetErrorText. There might be a better way to deal with functions asking for buffers using ctypes, but I need a solution quickly so I just did it straight up.

Thanks! This will make error messages a lot more helpful.
I have merged your changes to my develop branch and refactored it a bit. I also upped the max length from 80 to 256 since that's the value used in portmidi.h.


[COPYWRITING] Write default texts for key notifications

Is your feature request related to a problem? Please describe.
Currently, we're lacking good texts to communicate key events via emai.
Describe the solution you'd like
Write defaults for these emails we will send out:

The texts should be short, crisp and clear. It's easy to write a lot, it's a craft to be short :)
And always keep in mind when writing: First be clear, then be clever.

Generally, the wording in the app is very inconsistent. As we develop it further, we'd need a clear guideline for how we call things and actions. If you have experience doing this and are interested to help us set this up, please reach out.
Thanks!!!
Describe alternatives you've considered
No response
Additional context
No response

/assign


docs(cmd/rpccli): ui/ux improvements

Originally posted by @rach-id
Rachid has shared his main pain points when interacting with our API/docs:
* getting the token docs need to be more appearent in the docs. I always struggle to find the right commands
`Error: cant access the auth token: token/node-store flag was not specified: no opened Node Store found (no node is running)`

I don't understand what this error means. The bridge node is running but still it complains. Should I run a light/full node? also, if the fix is always the same, would it make sense to add it in the error message?

--url string     Request URL what is this URL? what does it mean?
--token string    Authorization token  I would add hints on how to get it. something like: if you still haven't generated one, use celestia <node_type> admin auth to get it.

account-address             Retrieves the address of the node's account/signer.
  balance                     Retrieves the Celestia coin balance for the node's account/signer and verifies it against the corresponding block's AppHash.
  balance-for-address         Retrieves the Celestia coin balance for the given address and verifies the returned balance against the corresponding block's AppHash.

I don't exacly get what they do, the difference and I guess the verifies it against the corresponding block's AppHash is too specific to be exposed to users.
I guess that setting the CELESTIA_CUSTOM to run against a custom network needs to be documented in a clear place in the docs. I struggled and had to ask node to know how to run it for a custom node. Alternatively, you could provide a CLI flag --trusted.hash -chain.id  that need to be set when setting p2p.network custom
I propose to stick to the same names in comsos-sdk for subcommands and even stick to the tx|query subcommand to organise the subcommands. For example, begin-redelegate can be redelegate simply and even copy the description as it's clear, concise and anyone in the space is already familiar with.

I propose to discuss here how we can improve it 🙂

getting the token docs need to be more appearent in the docs. I always struggle to find the right commands

Just reemphasising this point. I also struggled a little with understanding how to use the auth tokens. I asked Josh to try make it more prevalent in the docs (and included a note on it in the Node API page).
I also don't think it needs to be enabled if your node's API is only exposed on the localhost.
I also had the problem where rollkit required it so even if I used the flag it had to be set in rollkit. I think they have since fixed this problem


Add OMPIterator to a few switch cases

OMPIterator was recently added to upstream clang and thus a few
switch statements need to account for this addition.

@swift-ci please smoke test


Don't crash node and show better error on invalid subgraph transitions

When a subgraph is indexed by two or more graph nodes that share a database, we're currently crashing nodes when they run into from/to block conflicts. What we should do (and what this PR does) is only make the subgraph fail and also print a message that points out the likely cause of the issue: that multiple nodes are indexing the same subgraph.

@lutter Totally. Let me update.

@lutter Updated. How's this?
https://github.com/graphprotocol/graph-node/pull/1903/files#diff-02527213a13f32ef41afb0e0e7e7df5dR743-R748


[Feature]: Response pane and debugger (reduce context switching)

Is there an existing issue for this?

[X] I have searched the existing issues

Summary

User should not have to move to debugger to decode what is happening

Show the error tab’s error on the response pane itself
Show a collapsed error user can see without navigating
Sniping mode + suggested widget

Summary
Job stories:

[ ] When I open the full page API, full page canvas, or the split pane layout, I want my experience to be consistent across all 3, so I don’t have any friction while switching from one layout to the other
[ ] When I get an error, I want the error to be formatted in the best way possible, so I can easily understand, and debug my application
[ ] When I navigate across appsmith, I want the debugger tabs to remain consistent, so I can easily understand errors, responses, and results (number and positioning of tabs)
[ ] When I run a query, api, or JS function (action), I want to have access to the response and errors in an easily accessible way, so I can easily resolve the error without having to navigate on the platform
[ ] When I open the list of errors, I want to see the list of all errors in a compact way, so I can easily find the relevant errors
[ ] When I am focussed on an action/pane, I want to see the relevant errors focussed, so I can identify the errors and address them without the noise of all other errors in the app
[ ] When I am binding data from a query to a widget (or one action to another), I want to see the response of the query as I bind it, so I can easily match the output, and write JS functions like map functions.

Desired outcomes of the project:

[ ] create a new layout for the debugger which works with the side by side layout
[ ] Include errors in the response
[ ] Bring consistency across the debugger across appsmith

Why should this be worked on?
Reduce the need to switch around when debugging errors.

won't fix - project cancelled


numYLabels doesn't affect number of labels

It seems like number of labels on Y axis for line chart is set at 5 and isn't affected by this option.
P.S.: Where can I look up all possible options for objects?

Hi, please try again now with<EMAIL_ADDRESS>(just published with the fix).
The goal is to have all settings in the README in each component section. Sometimes I may have missed some settings which you can usually find in the ctor of the relevant component (e.g. widgets/charts/line.js). If you see something missing please let me know.
Good luck!

Thanks, that works now.


API coverage statistics

We need a way to figure out how much of our API is covered by our e2e tests. @brendanburns suggested this was possible with log scraping today. We also need a way to get it back to the test harness itself so that we can report on it. (Per-test? Per-run?)

I started writing an api proxy that can also collect this info, its just a
question of plugging it in...
Brendan
On Feb 18, 2015 6:41 AM, "Zach Loafman"<EMAIL_ADDRESS>wrote:

We need a way to figure out how much of our API is covered by our e2e
tests. @brendanburns https://github.com/brendanburns suggested this was
possible with log scraping today. We also need a way to get it back to the
test harness itself so that we can report on it. (Per-test? Per-run?)
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/4529.

Where is it?


Reorganisation of files and a few bugfixes

Hi,
I know this is a big pull request, but I needed all these functionnalities at the same time and could not make a lot of small pull requests. I will explain what I did in details here.
General

PSR-2 Coding style is now PSR-2, this eases the collaboration between projects
PHP 5.4 is a minimum, it makes the code more readable in some places. and PHP 5.3 is unsupported for a long time now

Code Structure

Uses PSR-4 Standard, easier autoloading.
Separated "Generator" and "Server" make a much clearer code.
Also separated "Tree" to generate the structure.

File structure

Images are now directly in the docs folder, make a more contained documentation : generator/server on one side, documentation on the other
resources folder contains js files, themes and img. makes it easier to configure the copy on generation.

Templates

Templates are now generated with http://platesphp.com/
Templates are much cleaner and overridable by creating a templates folder inside the theme and naming the template the same way as the original one. the engine does the rest.

Bugfixes

in generated mode, the name of the documentation links to index instead of index.html
if there are no links, the documentation doesn't show the "show/hide code blocks" toggle
a _index.md page in a sub folder now uses the default template and not the one for homepages.
Page.md is exactly the same as Page/_index.md, the first Page calculation takes it in account.

Don't hesitate if you want me to fix/change something before you accept the pull request.

Excellent pr - this corrected the issues we had previously patched and more.  You did remove however the ability to override a template without modifying the core files.  What would be a good place to have template override files? I'm thinking they can be packaged with a resources/themes/ folder.

This looks, I hope it get merged soon, and, in the meantime why not switch to CommonMark ?

Migrating to commonmark was one of my ideas also, but this is already a huge pull request and I wanted it merged in before I go further in my customisations.
I have also other ideas that I already explore in other branches in my fork like being able to export in multiple different formats. My first planned alternative is exporting to atlassian's confluence, but after that why not also do ebooks or pdf or anything like that?

I switched to Commonmark, it was fairly simple (3 changes, the instanciation, the method call and the class import).
In the other hand, even with your modification, it's impossible to switch to another highlighting library easily, there are still too much hard coded assets. But it's quite a good start.
This repository is not very active, I may have to directly fork your contribution.

Added an abstraction layer in pull request #265, added CommonMark package, and set CommonMark as the demo option.
Should solve this moving forward.

Super nice.
It's better than just replacing the default parser knowing that CommonMark does not support some important github flavored markdown (mainly tables).

Heys guys, just a quick heads up, are you interested in my changes ?
If not I will remove my pull request and rename the project. (with all due references to the original work)
I have a lot of things I still want to contribute, but I don't want to contribute to a project that isn't supported.

@onigoetz This looks amazing! I just checked out the latest version of this PR, and it looks like things are working nicely except the landing page images look to be broken links now. Once those are fixed, I think this is ready to merge.
Would you be interested in joining the project as a core contributor? We could use the help getting this project back in order. :) There are a lot of outstanding PRs and issues. Just let me know and I would be happy to add you to the project. You would be able to contribute whenever and on whatever you saw fit.
I know @dkisselev and @Gautham would love to have you on the core team.

For the error, it's an easy one, I just pushed a fix, we don't have the same workflow, I don't use Grunt to run the embedded web server
Because of the images served by php, we need to define the router when starting the server.
I would be very happy to join the core team :)
I can't guarantee a lot of time on the project as I have other projects going on but for the next weeks I will work a lot on documentations and will find new bugs / features to add

Oh Btw, as there are really a lot of changes and I forgot to specify it in the big changelog at the top of the page.
I removed the text editor.
The way I see it, the main feature is documentation generation in different formats, being able to watch your changes live is a plus, but it's a side feature.
Plus there are a lot of nice markdown editors out there.
I hope we're on the same page.

I have added you as a contributor :) Thanks for your willingness to help!
I think we are on the same page regarding the editor. Was the reason for removing it just to simplify the refactor? If so it's possible someone might come along and add it again someday under the new coding standards.
A quick note about the project setup since you are now on the team. All commits to master will auto-deploy from my continuous integration server to daux.io. If you want to build in a staging environment, we have a development branch that is way behind master that could be brought back to life. Let me know if you have any questions. Otherwise feel free to clean up any old PRs or issues that you would like.
And go ahead and merge this PR when you are ready :)

Nice, thanks :D
Yes, I removed the editor during the refactoring of the tree. It simplified a few things.
Okay no problem for the integration and deployment. I had the idea that one of the generators could be a static jekyll site, would you find that interesting ? With this the website could be hosted on github and remove the fear of breaking it when we push to master.
I will merge the branch tomorrow morning (I live in switzerland, here it is the middle of the night)
Apart from the integration system, is there anything I should know? Is there a  communication process or something like that between team members?

When Gautham and I were working on Daux we shot emails back and forth for help and to co-ordinate who was working on what. Feel free to email either of us (and/or Justin) if you need a hand, but as you can tell, we've been too busy to actively work on Daux in the last few months.

@dkisselev Okay, I will use Git flow for my development. workflow.
I don't like to see projects as competing, I have a more "best tool for the job" philosophy. If it is more convenient for the user to edit and create a template with daux but he wants to deploy it on github.io for hosting reasons, and also create a PDF/ Epub from his documentation, with Daux he would be able to do it, but not with jekyll.
But okay, If you don't see it a good idea, I will not do it ;-)


Add tracing, stderr logging, and parallel image alterations

Apologies for the multiple features in a single diff, let me know if you want me to split the diff. This is what has changed:

Use tracing instead of log as it's more powerful. It allows you to add instrumentation and has a bunch of crates to make telemetry better.
Use stderr for logging. At the same time changed it so when using --stdout, it does not conflict with --debug.
In various locations use threads to do operations in paralllel leading to the below ~33% performance improvement for 3 screens. This primarily affects image stuff.

For this last item I tried to use rayon first but there wasn't much difference with a scoped thread implementation besides that we could potentially use it for some work stealing. Though the wins would be minimal for the bigger dependency imo (<10 milliseconds on 300ms) if not it might be worse due to it being optimised for CPU bottlenecks and not IO bottlenecks (we have mostly IO, but the image stuff is CPU).

Example output with tracing

Before:
➜ hyperfine "target/release/wayshot --stdout > /dev/null"
Benchmark 1: target/release/wayshot --stdout > /dev/null
  Time (mean ± σ):     310.7 ms ±  16.2 ms    [User: 144.5 ms, System: 105.3 ms]
  Range (min … max):   283.1 ms … 333.4 ms    10 runs

After:
➜ hyperfine "target/release/wayshot --stdout > /dev/null"
Benchmark 1: target/release/wayshot --stdout > /dev/null
  Time (mean ± σ):     204.7 ms ±   9.0 ms    [User: 154.6 ms, System: 100.8 ms]
  Range (min … max):   188.2 ms … 216.5 ms    14 runs

Iirc there are some slight improvements as well for the part where screenshots are taken, though I don't have stats for that anymore as I tested this on Monday. I was generally doing some performance analysis everywhere. Below is the before and after for a single screen.
Before:
➜ hyperfine "target/release/wayshot --stdout --output DP-1 > /dev/null"
Benchmark 1: target/release/wayshot --stdout --output DP-1 > /dev/null
  Time (mean ± σ):      24.2 ms ±   2.1 ms    [User: 8.5 ms, System: 4.6 ms]
  Range (min … max):    21.7 ms …  32.6 ms    104 runs

After:
➜ hyperfine "target/release/wayshot --stdout --output DP-1 > /dev/null"
Benchmark 1: target/release/wayshot --stdout --output DP-1 > /dev/null
  Time (mean ± σ):      23.9 ms ±   3.3 ms    [User: 9.1 ms, System: 3.9 ms]
  Range (min … max):    21.9 ms …  46.6 ms    61 runs

Finally I did some small changes here and there like using a boolean in the library for cursor_overlay instead of i32 (and changing it at the image layer).

If you're interested in the current bottlenecks, they seem to be:

Copying the image from the memory mapped file.
Overlaying the images on one another
Rotating/resizing images
Encoding the image

These are all after taking the screen copy though. The screen copy itself is about ~24ms in total last time I checked for 3 monitors. 🤔 The bottleneck seems to be on the server side, we're waiting for the compositor. This was tested on Hyprland.

Hi! thank you for the patch. I haven't gone through all of it just yet. Just to be sure, rayon was replaced with scoped threads right?
I spotted some rayon artifacts in use and Cargo.lock during a quick overlook. Is that intentional?

Apart from those two tiny nit-picks, everything looks fine and the PR performs really well! Thank you for the PR!


Remove template-local let bindings

Changes from @akrmn to remove template-local let bindings

is let being deprecated? What's the motivation for this? (makes code clunkier)

The context is given in https://docs.google.com/document/d/14z_OOaeqXjfKrQubkk6Gt1bwHIYnzxzd1xQiarZICKk/edit#heading=h.t2p7lpivzaz