monitor type ping stopped working

Info
Uptime Kuma Version: 1.3.2
Using Docker?: Yes
OS: Windows (client) Docker with uptime-kuma runs on Debian 10
Browser: Chrome
I installed uptime-kuma a few days ago and setup a few monitors, most of them https and one as ping type. The ping type one stopped working yesterday evening. I know the target is up, even if I do a docker exec -ti uptime-kuma sh  and do a manual ping to the same target the ping works so why does it keep failing for a full 24h inside uptime-kuma?
Also, uptimerobot reports no issues pinging this particular host.
Open for any suggestions.

Are there any error? You can over the bar, check docker log or event list

Every single ping fails since yesterday. docker-compose down followed by docker-compose pull and docker.compose up -d does not show any errors or changes besides the failed ping.
Not sure if it will help but here you go:
docker-compose  logs -f
uptime-kuma | Monitor #6 'MzansiRace': Successful Response: 9245 ms | Interval: 60 seconds | Type: http
uptime-kuma | Monitor #19 'PBS External': Failing: PING ict.zfs.rent (<IP_ADDRESS>): 56 data bytes
uptime-kuma |
uptime-kuma | --- ict.zfs.rent ping statistics ---
uptime-kuma | 1 packets transmitted, 0 packets received, 100% packet loss
uptime-kuma |  | Type: ping
uptime-kuma | Monitor #7 'pacura.ru': Successful Response: 12581 ms | Interval: 60 seconds | Type: http

I've shortened the logs output but as you can see, the ping is failing.
docker exec -ti uptime-kuma sh
 ping ict.zfs.rent
PING ict.zfs.rent (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=57 time=171.232 ms
64 bytes from <IP_ADDRESS>: seq=1 ttl=57 time=171.437 ms
64 bytes from <IP_ADDRESS>: seq=2 ttl=57 time=171.664 ms
^C
--- ict.zfs.rent ping statistics ---
3 packets transmitted, 3 packets received, 0% packet loss
round-trip min/avg/max = 171.232/171.444/171.664 ms

from my docker host:
ping ict.zfs.rent
PING ict.zfs.rent (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from ict.zfs.rent (<IP_ADDRESS>): icmp_seq=1 ttl=58 time=171 ms
64 bytes from ict.zfs.rent (<IP_ADDRESS>): icmp_seq=2 ttl=58 time=171 ms
64 bytes from ict.zfs.rent (<IP_ADDRESS>): icmp_seq=3 ttl=58 time=171 ms
64 bytes from ict.zfs.rent (<IP_ADDRESS>): icmp_seq=4 ttl=58 time=171 ms
^C
--- ict.zfs.rent ping statistics ---
5 packets transmitted, 4 received, 20% packet loss, time 8ms
rtt min/avg/max/mdev = 171.257/171.322/171.406/0.418 ms

Really strange, as Uptime-kuma is also using the builtin "ping"
Could you also try this? This is how Uptime Kuma executes the ping command
ping -n -w 2 -c 1 ict.zfs.rent

Really strange, as Uptime-kuma is also using the builtin "ping"
Could you also try this? This is how Uptime Kuma executes the ping command
ping -n -w 2 -c 1 ict.zfs.rent

That does indeed fail. Need to check the manual what all those switches for the ping command do.
docker exec -ti uptime-kuma sh
/app # ping -n -w 2 -c 1 ict.zfs.rent
PING ict.zfs.rent (<IP_ADDRESS>): 56 data bytes

--- ict.zfs.rent ping statistics ---
1 packets transmitted, 0 packets received, 100% packet loss

from inside your container, the '-n' option seems unknown.
ping -h
ping: unrecognized option: h
BusyBox v1.31.1 () multi-call binary.

Usage: ping [OPTIONS] HOST

Send ICMP ECHO_REQUEST packets to network hosts

        -4,-6           Force IP or IPv6 name resolution
        -c CNT          Send only CNT pings
        -s SIZE         Send SIZE data bytes in packets (default 56)
        -i SECS         Interval
        -A              Ping as soon as reply is recevied
        -t TTL          Set TTL
        -I IFACE/IP     Source interface or IP address
        -W SEC          Seconds to wait for the first response (default 10)
                        (after all -c CNT packets are sent)
        -w SEC          Seconds until ping exits (default:infinite)
                        (can exit earlier with -c CNT)
        -q              Quiet, only display output at start
                        and when finished
        -p HEXBYTE      Pattern to use for payload
/app # ping -w 2 -c 1 ict.zfs.rent

I upped the seconds to wait to 5 and the ping works all of a sudden:
ping -w 5 -c 1 ict.zfs.rent
PING ict.zfs.rent (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=57 time=172.820 ms

--- ict.zfs.rent ping statistics ---
1 packets transmitted, 1 packets received, 0% packet loss
round-trip min/avg/max = 172.820/172.820/172.820 ms

Might be internet problems at home where this NAS runs? I'm currently travelling so can'T physically check but I can access my services at home just fine.

Still a bit strange as your the response time is within 2 seconds.

My guess: Your dns server is slow. It takes more than 2 seconds to resolve a domain.
You can ping the IP directly.

@louislam Seems like DNS is still a huge problem. Looking around curiously I found that many people reported having DNS issues on Alpine with Docker, and some reported issues fixed by switching to Debian or other OS. Might be worth switching to Debian despite the larger image size.

@louislam Seems like DNS is still a huge problem. Looking around curiously I found that many people reported having DNS issues on Alpine with Docker, and some reported issues fixed by switching to Debian or other OS. Might be worth switching to Debian despite the larger image size.

Omg, that is surprising me...
@ovizii Could you please help me try again the command in your host machine? Thank you.
ping -n -w 2 -c 1 ict.zfs.rent

If your host machine is working, than I believe that Alpine Docker is having a huge problem.

Please let me know if you think its my local system and not due to the alpine base image, I can do further tests.

Please let me know if you think its my local system and not due to the alpine base image, I can do further tests.

I have built a debian base image, I think it will eventually fix your problem.
uptime-kuma:1.5.2-debian

Thanks, I have switched to :1.5.2-debian and everything seems fine. I guess I'll check in a few weeks later and figure out how you are handling tags.
For now I'll keep using 1.5.2-debian - maybe later you'll have a generic :debian or whatever.

Thanks, I have switched to :1.5.2-debian and everything seems fine. I guess I'll check in a few weeks later and figure out how you are handling tags.
For now I'll keep using 1.5.2-debian - maybe later you'll have a generic :debian or whatever.

If Debian is better for most users, I will use Debian as main tag in the future. For example:

1.6 (Debian)
1.6-alpine (Alpine)

Just wanted to let you know, that I have now added fixed DNS servers to uptime-kuma's docker-compose.yml file
    dns:
      - <IP_ADDRESS>
      - <IP_ADDRESS>
      - <IP_ADDRESS>

Response times have gone seriously down compared to before. Before this, the docker host used to use a locally (Docker run) DNS (adguard home).

Hi is there a fix in WIP? also the docker containers here are not been read anymore.
Thanks alot! Love this tools for a lon period now

the docker sock does get read ibut the statusses are offline

as while those containers are up and running smooth

This is wait i get when i try to clean the database file


Add ZMechanical for CTX 500

While CTX 500 has dynamic control on Z, there are some cases where the target is too close, so adding mechanical spacing (as done for the single-element Tx) can help these cases.

Added at caf329f


[Question] how to extract gene and omics data from Wikidata

Hello there! Excuse me for this trivial question but I'm quite new to the Wikidata life sciences world (your work on *-omics is great!).
I'm trying to extract relevant data related to life sciences from Wikidata compliant to w3c specifications. I think that using Wikidata as central HUB is a more effective and concrete way to leverage Linked Data for knowledge exploration. What I need is a collection of RDF statements describing Human Genome, diseases, proteins, and related annotations with compliant RDF(S)/OWL statements. I found genewiki.owl and I guess that is the ontology describing life sciences data models on Wikidata.
I'm trying to download data via the Wikidata SPARQL enpoint through a bunch of custom queries but I'm wondering if that is "the way". Is there a more proper and faster approach to download a dump of data related to the genewiki.owl model?
Thank you!
Edit:
I've just noticed that classes URIs in genewiki.owl are note the same of wikidata, but I think they are the same classes. Besides, in that file there are not all properties used in Wikidata (e.g., genomic start/end). Isn't there a comprehensive OWL model?
Bye!

Hi @superciccio14, Thanks for your interest! As Wikidata is very "open-world", we do not have a strict ontology describing the relationships used. genewiki.owl is an attempt at that, but as you saw, is not complete. Are you looking for information about specific concepts, item, or properties? Or do you want 'everything' related to biomedical sciences in Wikidata?
If you are asking specific questions, SPARQL queries are "the way". Here is a little "map" of this biomedical space as of June 2017, that was made using SPARQL queries and some manual filtering, that should help you write the targeted queries. Here are some example queries to guide you also.
If you'd like to download everything, you can either go through the Wikidata dumps are write "traversal" sparql queries that just get everything connected to an item (for example).
Let me now if this helps!

Another example "traversal" query. This gets all statements on a set of items (for one property), including references and qualifiers.
select ?item ?qval ?pq ?sid ?v ?ref ?pr ?rval where {
  values ?item {wd:Q14911732 wd:Q21129787}

  ?item p:P644 ?sid .

  ?sid ps:P644 ?v .
  OPTIONAL {
    ?sid ?pq ?qval .
    FILTER(STRSTARTS(STR(?pq), "http://www.wikidata.org/prop/qualifier/P"))
  }
  OPTIONAL {
    ?sid prov:wasDerivedFrom ?ref .
    ?ref ?pr ?rval .
    FILTER(STRSTARTS(STR(?pr), "http://www.wikidata.org/prop/reference/P"))
  }
}

Hi @stuppie and thanks for your kind replies! I'll definitely try your hints to extract relevant biological Knowledge from Wikidata.
If you are interested in the reason of my question, here there is the long story short. My goal is building a semantic annotation platform for biological entities related to genomic and drug-resistance data. I explored other semantic/linked data effort like this or this one, but they seem outdated and I think they arrived to a dead end. Also the LOD had seemed to me a good starting point but, in practice, it's very hard to manually integrate such datasets. Finally, I came across your paper, and that's the reason of my first question :smile:
Probably, "the way" - as you suggested - is getting data from Wikidata and link'em to my own ontology, eventually adding locally new classes or properties. In that way, I can always refer back to Wikidata and to its xrefs, leveraging a very huge and active KB.

@floatingpurr If you are using existing name spaces in your local data, mapping properties like those from SKOS can help in providing links to Wikidata.
Another approach I am using more and more is to actually use the Wikidata namespace as local namespace. The Wikidata properties can be used as predicates in linked data

Hello @andrawaag, yes the SKOS vocabulary can be definitely used to map my local URIs to their equivalent in wikidata. I've recently performed some tests with the second approach (i.e., directly using wikidata resources to tag my data). In both cases, federated queries have to be taken into account to enrich the local knowledge, as you described here.
A little off topic: are new data contributions for life sciences welcome in wikidata?

A little off topic: are new data contributions for life sciences welcome in wikidata?

Absolutely! Data licenses tend to be the key sticking issue. But if you have a CC0-licensed resource in mind that you'd like to load, we'd love to share our knowledge/experience/tools with you.

Hello @andrewsu, thank you very much for your answer. At the moment, I have not clear options in mind. Maybe COSMIC could be a good dataset, but it does not seem explicitly CC0-licensed. Another option could be to insert transcripts as entities, thus encouraging further annotations about them. Indeed, if I am not wrong, in the actual data landscape they are just listed as external IDs and the predicate/property encodes (P688) has proteins as objects but not RNAs.
I'll definitely get back to you as soon as I have a clear proposal in mind. Thank you once again for your kind support and for your great work : )


Running the tests makes the package dirty, preventing it from being updated or removed

The tests modify the file test/Chinook_Sqlite.sqlite. This is very bad, as the package then won't be updated or removed by Pkg unless you manually reset the change. Either that file should be added to a gitignore, or some other scheme involving making a copy and comparing separate files would be better.

This is very annoying indeed. I suggest a git checkout at the end of the tests. I´ll send a PR with this suggestion.

Ok, this is fixed now by this line

would be good to gitignore the test version in case tests ever fail

Good idea; change made.


Kubernetes modules hard coding of service account names leads to max char errors for IAM roles

Description
Please provide a clear and concise description of the issue you are encountering, and a reproduction of your configuration (see the examples/* directory for references that you can copy+paste and tailor to match your configs if you are unable to copy your exact configuration). The reproduction MUST be executable by running terraform init && terraform apply without any further changes.
If your request is for a new feature, please use the Feature request template.

[x] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Versions

Module version [Required]: v4.16.0
Terraform version: 1.3.4
Provider version(s):

✗ terraform version
Alias tip: tf version
Terraform v1.3.4
on darwin_arm64
+ provider registry.terraform.io/gavinbunney/kubectl v1.14.0
+ provider registry.terraform.io/hashicorp/aws v4.39.0
+ provider registry.terraform.io/hashicorp/cloudinit v2.2.0
+ provider registry.terraform.io/hashicorp/helm v2.7.1
+ provider registry.terraform.io/hashicorp/kubernetes v2.15.0
+ provider registry.terraform.io/hashicorp/null v3.2.0
+ provider registry.terraform.io/hashicorp/random v3.4.3
+ provider registry.terraform.io/hashicorp/time v0.9.1
+ provider registry.terraform.io/hashicorp/tls v4.0.4

Reproduction Code [Required]

Steps to reproduce the behavior:

module "addons" {
  source = "github.com/aws-ia/terraform-aws-eks-blueprints//modules/kubernetes-addons?ref=v4.16.0"

  eks_cluster_id       = module.eks.cluster_id         # 30 char string name
  eks_cluster_endpoint = module.eks.cluster_endpoint
  eks_cluster_version  = module.eks.cluster_version
  eks_oidc_provider    = module.eks.oidc_provider

  enable_aws_load_balancer_controller      = true
  aws_load_balancer_controller_helm_config = {}
}

Expected behaviour

Successful terraform apply
Actual behaviour

Error: expected length of name to be in the range (1 - 64), got <cluster-name>-aws-load-balancer-controller-sa-irsa

  with module.addons.module.aws_load_balancer_controller[0].module.helm_addon.module.irsa[0].aws_iam_role.irsa[0],
  on .terraform/modules/addons/modules/irsa/main.tf line 38, in resource "aws_iam_role" "irsa":
  38:   name        = try(coalesce(var.irsa_iam_role_name, format("%s-%s-%s", var.eks_cluster_id, trim(var.kubernetes_service_account, "-*"), "irsa")), null)

Terminal Output Screenshot(s)

N/A
Additional context

I believe the bug https://github.com/aws-ia/terraform-aws-eks-blueprints/issues/333 is still present for the aws-load-balancer-controller.
It's failing for the irsa module's aws_iam_role which is controlled by var.irsa_iam_role_name.
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/f233a46e63f000cba786dc397975e7ac6e998954/modules/irsa/main.tf#L16-L19
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/f233a46e63f000cba786dc397975e7ac6e998954/modules/irsa/main.tf#L35-L38
The irsa module is consumed by the helm_addon module
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/f233a46e63f000cba786dc397975e7ac6e998954/modules/kubernetes-addons/helm-addon/main.tf#L63
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/f233a46e63f000cba786dc397975e7ac6e998954/modules/kubernetes-addons/helm-addon/main.tf#L71
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/f233a46e63f000cba786dc397975e7ac6e998954/modules/kubernetes-addons/helm-addon/main.tf#L74
The helm addon does not expose the irsa_iam_role_name input
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/f233a46e63f000cba786dc397975e7ac6e998954/modules/kubernetes-addons/aws-load-balancer-controller/main.tf#L1-L8
The local irsa_config has a hard coded kubernetes_service_account
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/f233a46e63f000cba786dc397975e7ac6e998954/modules/kubernetes-addons/aws-load-balancer-controller/locals.tf#L2-L3
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/f233a46e63f000cba786dc397975e7ac6e998954/modules/kubernetes-addons/aws-load-balancer-controller/locals.tf#L46-L52
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/f233a46e63f000cba786dc397975e7ac6e998954/modules/kubernetes-addons/main.tf#L180-L186
What's also interesting is how the aws_iam_policy uses a shorter name to most likely avoid the same max char issue that the aws_iam_role role hits.
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/f233a46e63f000cba786dc397975e7ac6e998954/modules/kubernetes-addons/aws-load-balancer-controller/main.tf#L10-L11
Options
So the only way I can currently solve without forking is to use a smaller cluster name (cluster name cannot exceed 27 chars) but I'd rather not do this.
Option 1: allow overriding local.name or local.service_account_name with helm_config
We could do this var.helm_config using something like helm_config["name"] or helm_config["service_account_name"] or both
# modules/kubernetes-addons/aws-load-balancer-controller/locals.tf
locals {
  name                 = try(local.helm_config["name"], "aws-load-balancer-controller")
  service_account_name = try(local.helm_config["service_account_name"], "${local.name}-sa")

and then we can do this
module "addons" {
  source = "github.com/aws-ia/terraform-aws-eks-blueprints//modules/kubernetes-addons?ref=v4.16.0"

  # ...

  enable_aws_load_balancer_controller = true

  aws_load_balancer_controller_helm_config = {
    # either of the below would avoid the 64 char max limit
    name = "aws-lb"

    service_account_name = "aws-lb-sa"
  }
}

Option 2: allow overriding the irsa_config
We do this already with local.helm_config which is a merge of local.default_helm_config and var.helm_config.
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/f233a46e63f000cba786dc397975e7ac6e998954/modules/kubernetes-addons/aws-load-balancer-controller/locals.tf#L16-L25
We could do the same with the local.irsa_config but then we would have to expose a new input variable across all submodules e.g. aws_load_balancer_controller_irsa_config
After some thought on this, it's probably not the best idea since the local.service_account_name is used in many places so overriding the local.irsa_config would not impact the other places.
Thoughts
I think Option 1 would be easier.
I also think that doing a validation for the IAM role name to ensure it's not over 64 chars during plan-time would be good too. That way we do not have to wait until we deploy our cluster to know that we have to shorten the IAM role name.
The error could be as simple as this where the error is thrown if the length of a string is fed as the index of the array.
> range(0, 65)[64]
64
> range(0, 65)[67]
╷
│ Error: Invalid index

Edit: on second thought, the.above check is covered by tflint and its aws plugin.

@bryantbiggs can we keep this open until I can resolve it across all modules ?
ref: https://github.com/aws-ia/terraform-aws-eks-blueprints/pull/1184#issuecomment-1320200319

closed in #1193


[Question] Paranoid version

Is there anyway of having a paranoid web version?
Something that doesn't even check for updates, doesn't support any cloud storage, ...
No internet at all. A very off-line version.

Yes, it's called desktop and can be downloaded here: https://github.com/antelle/keeweb/releases/latest

doesn't even check for updates

but you should disable it in settings, by default it checks and shows update bubble

doesn't support any cloud storage

what? doesn't SUPPORT? It supports but will never connect without your command. simply don't click Dropbox button if you don't want it.

No internet at all.

Absolutely. The internet is not required for it.

OKAY!!
Calm down!
I can't use the desktop version because I want to use it on Chromebook.
In any case it was just an suggestion based on a need.
On Feb 26, 2016 9:12 PM, "antelle"<EMAIL_ADDRESS>wrote:

Yes, it's called desktop and can be downloaded here:
https://github.com/antelle/keeweb/releases/latest
doesn't even check for updates
but you should disable it in settings, by default it checks and shows
update bubble
doesn't support any cloud storage
what? doesn't SUPPORT? It supports but will never connect without your
command. simply don't click Dropbox button if you don't want it.
No internet at all.
Absolutely. The internet is not required for it.
—
Reply to this email directly or view it on GitHub
https://github.com/antelle/keeweb/issues/126#issuecomment-189578274.

Then you can save HTML file and just open it to launch (or it's not possible on chromebook as well?). Or upload to your server and update only when required.

Yeah that was what I asked for in the first place. (A web-only version that
doesn't connect to the Internet)
I don't understand why you are acting like this! I'm just expressing my
need!
On Feb 26, 2016 9:37 PM, "antelle"<EMAIL_ADDRESS>wrote:

Then you can save HTML file and just open it to launch (or it's not
possible on chromebook as well?). Or upload to your server and update only
when required.
—
Reply to this email directly or view it on GitHub
https://github.com/antelle/keeweb/issues/126#issuecomment-189579557.

I don't understand why you are acting like this!

I'm sorry if the answer seems impolite to you, I didn't mean that.
To me, uploading to your own server, or opening offline file, or the fact that web version will check for updates is obvious. Also, as it's maybe not written clearly, there are very many users writing me that desktop version would be better, that's why the first thing I done is mention that completely offline desktop version do exists. If you know about desktop but cannot use it, it's another case but I cannot guess. I thought, you were one of them, who didn't see 'we have desktop builds' warnings nearly on each page.

Good that there isn't any confusion any more.
Back to business!
Is there any way to make an off-line version of the web version?
I'm more thinking about this:
https://play.google.com/store/apps/details?id=keepass2android.keepass2android_nonet
in comparison to:
https://play.google.com/store/apps/details?id=keepass2android.keepass2android&hl=en
Thanks

:+1:

cloud storages
I've had a brief look on both of these apps: so the main difference between them is that offline version saves file locally and online version saves the file in cloud.
In contrast to that, we always let the user decide, whether to upload your file to the cloud or not, so this app is a combination of that two those apps: if you want, you activate dropbox and save to it; if not, you just open a local file. The Dropbox module is even not activated before you click Dropbox button. And if you click it, you will see Dropbox login and link approval; so unless you linked the app to Dropbox, the module code is here but it can do (and it will not try to do) nothing without your permission.
So I think you should be ok with this.

updater
Auto-updating of the web app is currently not optional and, to be honest, I don't see a way to disable it for the web app. Ways you may find useful for you:

upload the certain version to your server and use from it
use tagged builds from gh-pages branch, for example: I want version 1.0.0, I can find it here: https://github.com/antelle/keeweb/commits/gh-pages (but it's not possible to just open the web app from it, as github gives you the file as text content, not html), so re-upload is necessary here
save the web page and use it: I don't know whether saving is possible in cloud OSes, like ChromeOs, so it might not fit you well

Thank you very much.

@antelle

save the web page and use it: I don't know whether saving is possible in cloud OSes, like ChromeOs, so it might not fit you well

I've been playing with a new Chromebook and saving https://app.keeweb.info/ locally as keeweb.html.
it seems to be working but (unless I'm mistaken) this arrangement puts the onus is on me to keep track of new releases, correct?
would it be very involved to have the javascript saved in keeweb.html check for a newer version of the web app whenever the machine is online and the browser loads keeweb.html? that way at least I would know when to download and overwrite the keeweb.html file with the latest version.
n.

@nopivnick yes, you're right. But you can automate this, i.e. create some sort of download script to check the manifest and download html, if it's not the same.
Checking updates is a bit difficult thing because of privacy issues: if I have downloaded the html, probably I don't expect it to send requests. So, I don't know if it would be useful for anyone outside ChromeOs.
// I suggest that you don't read comments in this issue, because I see that op has removed some comments (the most "interesting" parts of them) including the first one, so my answers here could be irrelevant.

@antelle

(the most "interesting" parts of them)

yeah, I figured as much.
n.


Changing exported Windows Icon doesn't work

Godot version:
Godot 3.2.1 -> 3.2.2 tested
Classic Version (No C#)
OS/device including version:
Windows Export
Issue description:
I exported a 32x32 .ico from Gimp and add it to the export Settings "Icon" for windows export but the Icon doesn't appear.
We tested it with some of our Community Members and everyone has the same Problem, the Icon change doesn't work.
Godot Icon stays after Export.
Steps to reproduce:
Export a Windows Project with setting a .ico on the Icon export Settings.

Do you have rcedit installed and configured? See Changing application icon for Windows for details.

Do you have rcedit installed and configured? See Changing application icon for Windows for details.

Isn't it enough to Export a .ico File with Gimp?

Isn't it enough to Export a .ico File with Gimp?

No it's not, Godot relies on external tool (rcedit) to modify executable and add icon and information strings.

I will ask the Community If they has do it, because i hasn't do it 😔

same with the Community, no install of rcedit.
So i will Close it, because of stupid users, including me 😅
Sorry for that.


QEMU on macOS 15.2 on Apple M4: qemu-system-aarch64: Property 'host-arm-cpu.sme' not found

limactl start --vm-type=qemu fails with {"level":"debug","msg":"qemu[stderr]: qemu-system-aarch64: Property 'host-arm-cpu.sme' not found","time":"2024-12-18T12:34:14+09:00"} in ha.stderr.log.

Host: macOS 15.2, on MacBook Pro 2024 (M4 Max)
Lima: v1.0.2
QEMU: v9.2.0

Similar issues:

https://github.com/canonical/multipass/issues/3842
https://gitlab.com/qemu-project/qemu/-/issues/2665
https://gitlab.com/qemu-project/qemu/-/issues/2721

This issue seems specific to Apple M4 (introduced SME) and macOS 15.2
Workaround
limactl edit --set '.cpuType.aarch64="cortex-a57"' INSTANCE

Attempting to cherry-pick UTM's patch to Homebrew

https://github.com/Homebrew/homebrew-core/pull/201613


Vagrant saltstack failing to provision.

Running make test_e2e was working fine just last night, but now appears to be failing to provision.
*[lots of normal debug info precedes......]*

==> master:  *  INFO:  sh -- Version 2015.05.07
==> master:
==> master:  *  INFO: System Information:
==> master:  *  INFO:   CPU:          GenuineIntel
==> master:  *  INFO:   CPU Arch:     x86_64
==> master:  *  INFO:   OS Name:      Linux
==> master:  *  INFO:   OS Version:   3.11.10-301.fc20.x86_64
==> master:  *  INFO:   Distribution: Fedora 20
==> master:  *  INFO: Installing master
==> master:  *  INFO: Found function install_fedora_deps
==> master:  *  INFO: Found function install_fedora_stable
==> master:  *  INFO: Found function install_fedora_stable_post
==> master:  *  INFO: Found function install_fedora_restart_daemons
==> master:  *  INFO: Found function daemons_running
==> master:  *  INFO: Found function install_fedora_check_services
==> master:  *  INFO: Running install_fedora_deps()
==> master:  *  INFO: Adding SaltStack's COPR repository
==> master: http://mirror.pnl.gov/fedora/linux/updates/20/x86_64/repodata/repomd.xml: [Errno -1] repomd.xml does not match metalink for updates
==> master: Trying other mirror.
==> master: http://fedora.mirror.lstn.net/updates/20/x86_64/repodata/repomd.xml: [Errno -1] repomd.xml does not match metalink for updates
==> master: Trying other mirror.
==> master: ftp://mirror.lstn.net/fedora/updates/20/x86_64/repodata/repomd.xml: [Errno 14] curl#7 - "Failed to connect to 2607:ff68:1:42::10: Network is unreachable"
==> master: Trying other mirror.

*[lots of failed mirrors follow.......]*

Basically it runs through every mirror for repomd.xml failing on each one. I've run it a few times on two machines both with working internet access and each time it appears to try every mirror before giving:
==> master: http://mirror.csclub.uwaterloo.ca/fedora/linux/updates/20/x86_64/repodata/repomd.xml: [Errno -1] repomd.xml does not match metalink for updates
==> master:  * ERROR: Failed to run install_fedora_deps()!!!
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.
2015/05/09 14:03:48 Error running up: exit status 1
2015/05/09 14:03:48 Error starting e2e cluster. Aborting.
exit status 1
make: *** [test_e2e] Error 1

I'm not familiar with saltstack so if anyone could point me in the right direction to solving this I would appreciate it greatly.

I saw this too.  Not sure the cause yet.  Have you checked if you can ssh into the box and just run a yum update to see if it can find any repo?  Basically, the salt stuff in its early steps is just trying to invoke yum
Sent from my iPhone

On May 9, 2015, at 2:34 PM, Benjamin Elder<EMAIL_ADDRESS>wrote:
Running make test_e2e was working fine just last night, but now appears to be failing to provision.
[lots of normal debug info precedes......]
==> master:  *  INFO:  sh -- Version 2015.05.07
==> master:
==> master:  *  INFO: System Information:
==> master:  *  INFO:   CPU:          GenuineIntel
==> master:  *  INFO:   CPU Arch:     x86_64
==> master:  *  INFO:   OS Name:      Linux
==> master:  *  INFO:   OS Version:   3.11.10-301.fc20.x86_64
==> master:  *  INFO:   Distribution: Fedora 20
==> master:  *  INFO: Installing master
==> master:  *  INFO: Found function install_fedora_deps
==> master:  *  INFO: Found function install_fedora_stable
==> master:  *  INFO: Found function install_fedora_stable_post
==> master:  *  INFO: Found function install_fedora_restart_daemons
==> master:  *  INFO: Found function daemons_running
==> master:  *  INFO: Found function install_fedora_check_services
==> master:  *  INFO: Running install_fedora_deps()
==> master:  *  INFO: Adding SaltStack's COPR repository
==> master: http://mirror.pnl.gov/fedora/linux/updates/20/x86_64/repodata/repomd.xml: [Errno -1] repomd.xml does not match metalink for updates
==> master: Trying other mirror.
==> master: http://fedora.mirror.lstn.net/updates/20/x86_64/repodata/repomd.xml: [Errno -1] repomd.xml does not match metalink for updates
==> master: Trying other mirror.
==> master: ftp://mirror.lstn.net/fedora/updates/20/x86_64/repodata/repomd.xml: [Errno 14] curl#7 - "Failed to connect to 2607:ff68:1:42::10: Network is unreachable"
==> master: Trying other mirror.
[lots of failed mirrors follow.......]
Basically it runs through every mirror for repomd.xml failing on each one. I've run it a few times on two machines both with working internet access and each time it appears to try every mirror before giving:
==> master: http://mirror.csclub.uwaterloo.ca/fedora/linux/updates/20/x86_64/repodata/repomd.xml: [Errno -1] repomd.xml does not match metalink for updates
==> master:  * ERROR: Failed to run install_fedora_deps()!!!
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.
2015/05/09 14:03:48 Error running up: exit status 1
2015/05/09 14:03:48 Error starting e2e cluster. Aborting.
exit status 1
make: *** [test_e2e] Error 1
I'm not familiar with saltstack so if anyone could point me in the right direction to solving this I would appreciate it greatly.
—
Reply to this email directly or view it on GitHub.

ssh'ing and running sudo yum update gives the same issue with failing to find repomd.xml.

This: (https://github.com/GoogleCloudPlatform/kubernetes/issues/1085) seems to suggest that it may be an issue with the Fedora mirrors.
Specifically this comment: https://github.com/GoogleCloudPlatform/kubernetes/issues/1085#issuecomment-53791984

@puiterwijk is there a problem with Fedora infrastructure?  We are having problems doing yum update on Fedora 20 VM in Kubernetes.  #1085 makes me think you may know how to help.
Sent from my iPhone

On May 9, 2015, at 6:02 PM, Benjamin Elder<EMAIL_ADDRESS>wrote:
This: (#1085) seems to suggest that it may be an issue with the Fedora mirrors.
Specifically this comment: #1085 (comment)
—
Reply to this email directly or view it on GitHub.

This makes me want to move to Fedora 21 this week.  I know we have OpenShift running on Fedora 21 now in Vagrant as an option so Kubernetes should be fine there, quicker to pick up Docker changes as well.
Sent from my iPhone

On May 9, 2015, at 6:02 PM, Benjamin Elder<EMAIL_ADDRESS>wrote:
This: (#1085) seems to suggest that it may be an issue with the Fedora mirrors.
Specifically this comment: #1085 (comment)
—
Reply to this email directly or view it on GitHub.

I hopped on #fedora on freenode last night, didn't find any solution but found two users with out of date fedora 20 installs with the same issue. Looks like it probably is the mirrors acting up. I'm going to be out for a bit this morning but I'll check if the situation has changed later.

I was trying to setup a cluster to play around with kubernetes but I ran into this very same issue.

From #fedora today in a conversation by yoanis and Zord:
https://bugzilla.redhat.com/show_bug.cgi?id=1220056

@BenTheElder I tried switching from fed20 to fed21 but that does not seem to be a one shot operation:
The following SSH command responded with a non-zero exit status.
Vagrant assumes that this means the command failed!

service network restart
Stdout from the command:
Restarting network (via systemctl):  [FAILED]
Stderr from the command:
Job for network.service failed. See "systemctl status network.service" and "journalctl -xe" for details.

Looks like we're stuck until the fedora guys fix their repo.

@yoanisgil Alright. I wound up kind of busy today. If someone else doesn't get to it first (I imagine they will though) I'll take a stab at getting vagrant working before tomorrow night.

My fix is this: Save the contents of this gist as ~/fedora-updates.repo https://gist.github.com/wshearn/ad2d5e133297ab3ece70
Then in ~/kubernetes/Vagrantfile on line 118 add
    config.vm.provision "file", source: "~/fedora-updates.repo", destination: "fedora-updates.repo"
    config.vm.provision "shell", inline: "mv ~vagrant/fedora-updates.repo /etc/yum.repos.d ; restorecon /etc/yum.repos.d/fedora-updates.repo"

Then you should be able to do ./cluster/kube-up.sh

@wshearn Thanks! Going to give it a try and let you know.

@wshearn after applying your fix I'm able to run ./cluster/kube-up.sh, so this will do until F20 repo is fixed.

The Fedora repo issues appear to be resolved.  I was able to kube-up the vagrant based cluster.

Hello,
I was trying to deploy kubernetes with vagrant yesterday as per the below link:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/getting-started-guides/vagrant.md
But, I getting the following error:
[vagrant@kubernetes-master ~]$ sudo yum update
File contains no section headers.
file: file://///etc/yum.repos.d/saltstack-salt-fedora-20.repo, line: 1
'\r\n'
Any idea?
Thanks.

Salt bootstrap was I hope temporarily failing last night.
https://github.com/GoogleCloudPlatform/kubernetes/issues/9241
It had started to mess up around 6EST after working well so I am hoping its a temporary flake with bootstrap of Salt.  Will verify again today

Salt repositories appear to be functioning again for me now.

Has anyone tested with k8sV1.0? Similar failure occurring.
==> master:  *  INFO: Found function daemons_running
==> master:  *  INFO: Found function install_fedora_check_services
==> master:  *  INFO: Running install_fedora_deps()
==> master:  *  INFO: Adding SaltStack's COPR repository
==> master:  * ERROR: Failed to run install_fedora_deps()!!!
repo errors further up in the script:
==> master: Installing, enabling prerequisites
==> master: http://lug.mtu.edu/fedora/linux/updates/21/x86_64/repodata/50536d92e16bf49919a28a94db65dacbb34483c452b5373eab69e9c02f812c0b-primary.sqlite.xz: [Errno 14] HTTP Error 404 - Not Found
==> master: Trying other mirror.
==> master: http://mirror.chpc.utah.edu/pub/fedora/linux/updates/21/x86_64/repodata/50536d92e16bf49919a28a94db65dacbb34483c452b5373eab69e9c02f812c0b-primary.sqlite.xz: [Errno 14] HTTP Error 404 - Not Found
==> master: Trying other mirror.

I've run it since the 1.0 merge but not on the latest master, i'll update and check back.

Seeing this as well.

It now seems to be hanging on:
==> master:  *  INFO: Adding SaltStack's COPR repository
```
This has happened twice now. I'll take another look in the morning.

From https://github.com/saltstack/salt-bootstrap/blob/develop/bootstrap-salt.sh#L2827, its trying to fetch the repos.d file from copr.fedoraproject.org, which is not resolving for me now.

I was just wondering if we could either use a different (possibly mirrored) repo? The truth is that over the last couple of months this has been a source of issue for some users, so maybe we should stop try to find a more permanent/stable solution.

For me at least, this appears to be functioning correctly again today.

working for me too (today).   stabilizing the repos would be a big sanity booster for newcomers like me

Following instructions, using:
Mac OSX 10.10.5
Vagrant 1.7.4
Parallels Desktop 10.2.2
When I run
export VAGRANT_DEFAULT_PROVIDER=parallels
export KUBERNETES_PROVIDER=vagrant
./cluster/kube-up.sh
master and minion-1 windows pop up, apparently fully functional (I can ssh into them and everything seems to work) but the command line shows following problems for both master and minion:
==> master: Checking for Parallels Tools installed on the VM...
==> master: Parallels Tools installed on this VM are outdated! In most cases
==> master: this is fine but in rare cases it can cause things such as shared
==> master: folders to not work properly. If you see shared folder errors,
==> master: please update Parallels Tools within the virtual machine and
==> master: reload your VM.

master: Running: /var/folders/ng/8c3_08cd5nb8y8414tffrb980000gn/T/vagrant-shell20150904-29622-1bot2j5.sh
==> master: Job for network.service failed. See 'systemctl status network.service' and 'journalctl -xn' for details.
==> master: An error occurred. The error will be shown after all tasks complete.
I have applied @wshearn fedora repo fix above but it didn't help.
Any help appreciated.

Just in case you see an error like
File contains no section headers.
file: file://///etc/yum.repos.d/saltstack-salt-fedora-21.repo, line: 1

since today when provisioning salt.
The reason is that copr.fedoraproject.org moved to copr.fedorainfracloud.org.
Pull request is here https://github.com/saltstack/salt/pull/30534.
A temporary workaround for vagrant is to use this gist: https://gist.github.com/rmohr/43941d0881f0682b99a7#file-bootstrap-salt-sh instead of http://bootstrap.saltstack.com/
by applying this diff:
diff --git a/cluster/vagrant/provision-utils.sh b/cluster/vagrant/provision-utils.sh
index c3a3d83..8875983 100755
--- a/cluster/vagrant/provision-utils.sh
+++ b/cluster/vagrant/provision-utils.sh
@@ -94,7 +94,7 @@ function install-salt() {

   if ! which salt-call >/dev/null 2>&1; then
     # Install salt binaries
-    curl -sS -L --connect-timeout 20 --retry 6 --retry-delay 10 https://bootstrap.saltstack.com | sh -s
+    curl -sS -L --connect-timeout 20 --retry 6 --retry-delay 10 https://gist.githubusercontent.com/rmohr/43941d0881f0682b99a7/raw/5b5e075e373cd81689e991eaef6cd95beb7b2d2a/bootstrap-salt.sh | sh -s
   fi
 }

@rmohr Thanks! Worked for me

seems this issue exist on version 1.1.7.

+1 on 1.1.7

@rmohr Thank you for the workaround.  I edited both provision-master.sh and provision-minion.sh and replaced a total of three references to bootstrap.saltstack.com.

@rmohr Thanks!

worked for me - replaced the salt bootstrap with the gist.
when is this going to be committed to a branch?


Evaluate moving the partialCache lock

See: https://github.com/gohugoio/hugo/blob/41e9e9fe14eb53f830404c9c76893e01c34fa034/tpl/partials/partials.go#L222
The above does not optimal and should be easy to benchmark. The above reduces lock contention, but it may not be optimal for this use case.
This was brought up in https://discourse.gohugo.io/t/how-to-ensure-partial-is-executed-just-once/35411/5 -- but note that even with the above change, a cache is not a lock.

good job


Feature/remove requirement versions

Upgrade to flask 2.0

@ampuerin I need yo to review and test the new script to deploy

@ampuerin Now you can test and review it

Test result ok


Allow executing other programs while Konsole is running

For example, maybe an app will want to call git to sync, or some other function that returns output.
Right now, Konsole has taken over System.in, System.out, and System.err - so we may need an escape hatch for when the user wants to say "I'm not in a Konsole block right now, let someone else print stuff for a whiel"
Maybe
konsoleApp {
   konsole { ... }.run()
   konsoleExec("git clone")
   konsole { ... }.run()
}

?

So I experimented with this a bit. It's not hard to do but it's probably not worth putting this into the library itself, because it's easy enough to just exec the process yourself, capture the input stream, and then just add each line into, say, a konsoleList
val output = konsoleListOf<String>()
konsole {
   output.forEach { line -> textLine(line) }
   textLine()
}.run {
   val process = exec("...")
   val reader = BufferedReader(InputStreamReader(process.inputStream))
   while (true) {
       val line = reader.readLine() ?: break
       output.add(line)
  }
}

We're doing this in Kobweb. See here: https://github.com/varabyte/kobweb/blob/main/cli/kobweb/src/main/kotlin/com/varabyte/kobweb/cli/run/Run.kt#L97
It ends up being easier to just let the application do its own custom thing. Above, we need a reference to the process handles so we can interact with them, which makes me lean towards not trying to wrap this behavior inside Konsole.


Exposing ports from docker container?

I'm trying to run a docker container (in a boot2docker Parallels vm) that runs an app that listens on http://localhost:5000, but I can't connect to it from my mac. I'm exposing the port in the Dockerfile with EXPOSE 5000/tcp, and then running docker run -p 5000:5000 -t whatever. But when I hit localhost:5000 on my mac, it fails to connect.
I suspect I'm missing something obvious - any ideas what's going wrong? Do I need to manually set up port forwarding from my mac's localhost:5000 to the docker host? And if so, how do I do that?

docker run -p 5000:5000 -t whatever will map port 5000 on you docker-machine vm to port 5000 on the container. It should work if you on your mac connect to <docker-machine ip>:5000. You find the ip of your docker machine VM by running docker-machine ip <name>.

Thanks. I've got it to work, and rather embarrassingly, it was of course, my fault :) I was running a web server that by default only accepted incoming local connections, so hitting <docker-machine ip>:5000 would fail. Once I configured it correctly, it works great. Thanks for the help, and for the driver too - aside from my mistake, it Just Works.


Add test that uses openssl app

I think the runner would be as simple as:
openssl s_client -capath ../certificates/root.crt -connect test.nameconstraints.bettertls.com:8244

Good call. I needed to do a little bit of futzing to add a "-verify_hostname x" or "-verify_ip y" parameter as needed, but otherwise worked well enough. On first pass it was showing some false positive test results, but on review I saw that it was applying DNS name constraints against the CN even when the CN was an IP. Since using an IP in a CN isn't to-spec anyway I adjusted the test result expectations. With that sorted, OpenSSL has a 100% pass!


Deprecate --monitorxxx options in favor of --consolexxx

Options --monitorcolors, --monitorwidth and --monitormarkers are badly named. They affect how execution is reported on the console and should be called --consolecolors, --consolewidth and --consolemarkers instead. This is especially important when we added new --console option (#317) to control the overall console output style.
The change is backwards incompatible and thus we cannot just remove the old options but need to deprecate them first. We can keep old short options -C, -W and -K to ease transition to new options.

Implemented, tested, and documented.


Line 374: [Warning] Invalid file link:(~/payroll/media/1b7a7d0d2105d38b4f108fcbcae8a9ea.jpg).
Line 427: [Warning] Invalid file link:(~/payroll/media/3682febe7f8380e092213bf04723673a.jpg).
Line 576: [Warning] Invalid file link:(~/payroll/media/38faa911c7b27828dae0557600159f32.jpg).
Line 674: [Warning] Invalid file link:(~/payroll/media/1859a5d77be20b8a7241e4ea2624640f.jpg).
Line 714: [Warning] Invalid file link:(~/payroll/media/48e1813c069cc3809e85f08cf7e45bcb.jpg).
Line 843: [Warning] Invalid file link:(~/payroll/media/1496d6c0f31bccc37099c5590b1b3092.jpg).
Line 866: [Warning] Invalid file link:(~/payroll/media/7977806af2f0ec351776e5cb71069fb1.jpg).
Line 1168: [Warning] Invalid file link:(~/payroll/media/10d43b370e55069491f108e962d29310.jpg).
Line 1197: [Warning] Invalid file link:(~/payroll/media/0a40d3be32d5491e1636f612d457a794.jpg).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Clicking schema refresh doesn't work since 1.5

Clicking the refresh button doesn't have any effect:

Fixed in 1.5.1.


Fail to generate the cluster blueprint

When I deploy ambari (one master, 2 slave) on Ubuntu 16.04 and ansible stop at TASK [ambari-blueprint : Generate the cluster blueprint]
"An unhandled exception occurred while running the lookup plugin 'template'. Error was a <class 'ansible.errors.AnsibleError'>, original message: template error while templating string: expected token 'end of statement block', got '.'
Is there anything wrong in the template file: blueprint_dynamic.j2?

Hi, this sounds like your Jinja2 version is too old, like with #29 #38 and #43.
In short, upgrade your jinja2 and ansible first (pip install ansible Jinja2 --upgrade) as I'm using a feature available only in Jinja2.10. Although this version has been released couple of years ago, if you use the OS packages (which are very slowly upgraded) instead of a virtualenv, you'll get those templating issues.
Please do let me know if you have the same issue after upgrading jinja.
(I'll try to remove that dependency at some point)

Hi, thanks for your quick fix, now it works!
I really appreciate it


Fixed the playlist cmd

self explanatory for the title
I did however modify all the lang files to no longer have the title of the playlist or whatever to the it was for the name spot

I'm not going to lie ben I'm gonna leave the thumbnail fixing to you as I tried for a bit and couldn't get it to work for the playlists so I removed it as a whole

lol that's fine; does this fix all the playlist commands to work as expected?

from what I can tell yes

it loaded my spotify playlist and successfully viewed it loaded it and such


Declare Webpack as a peer dependency?

First, thanks for creating and maintaining this project!
I'm using Webpack 3 and I installed this plugin as yarn add --dev purgecss-webpack-plugin ... which installed the Webpack 4 compatible plugin and resulted in a hard-to-debug issue (see https://github.com/symfony/webpack-encore/issues/330). I know that your installation instructions show different commands for Webpack 3 and 4 ... but I missed that.
The people that helped me solve this problem wondered why this plugin declares Webpack as a dependency instead of a peer dependency so I created this issue to ask the same in case it helps this project.
Thanks!

Thanks for bringing this up, this is embarrassing but I actually didn't know about peer dependencies :sweat_smile:. I will change this as soon as possible!


small typo in logs

I see Finshed removing SCSI devices in version 3.1.3 during a /csi.v1.Node/NodeUnpublishVolume on nodes

:tada: This issue has been resolved in version 3.1.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


UX

The webpage demonstrating the functionality of the library is not self-explanatory.
What do I enter where? Where do I see the result? What is the meanin of the numbers?

Dear @Echsecutor ,
Good point, it may be beneficial to include an illustrative example to better explain how to use th UX.
Do you also think that I it was beneficial to map the field names with the variable names I used in the javascript code?

Addressed with https://github.com/gs1-germany/palletBayCalculator/pull/16


Using Eloquent find* methods with string UUID

Hi,
Currently we can only use the find* methods (find, findOrFail, etc) with binary UUIDs.
It would be nice to get those methods to work when given a valid UUID string too.
Would you be ok for a PR which would override those find methods in the trait to make use of the withUuid scode instead of the withKey scope when a UUID string is detected?

Here is some code that works, that I could submit in a PR with some tests if you agree:

    public static function find($id, $columns = ['*'])
    {
        if (\is_array($id) || $id instanceof Arrayable) {
            return static::findMany($id, $columns);
        }

        return static::withUuid($id)->first($columns);
    }

    public static function findMany($ids, $columns = ['*'])
    {
        if (empty($ids)) {
            return static::newModelInstance()->newCollection();
        }

        return static::withUuid($ids)->get($columns);
    }

Overriding find methods goes a bridge too far, as it touches too deep into default Laravel behaviour. I'd be ok with some custom scopes.

Ok. However that felt much more coherent with the rest of the features which allow to work seamlessly with string or binary representations.
And MyModel::find($myUuidString) is way more pleasant to write than the corresponding scoped query.

Agree with @vpratfr 🤝
This package already hacked in some internal behaviors that Laravel has.
I know it's better not to touch the original method, but for the find method, just a little tweak can improve better usability and more importantly, can provide better integration ability with other packages which use the find method. And even if the original business logic of the find changes, it's just an easy catch up after all :)
@brendt  Please consider having the find accept string UUID.

@yaquawa we probably will: https://github.com/spatie/laravel-binary-uuid/pull/53
Though I won't be abel to merge it before september.

@brendt Thanks!😻

Closing this. Should be good now.


Add idSiteResult explanation to docs

Need to add an idSiteResult explanation to the express docs that explains how/when to use it.

https://github.com/stormpath/express-stormpath/pull/587


in import create two empty row

Prerequisites

[ ] Able to reproduce the behaviour outside of your code, the problem is isolated to Laravel Excel.
[ ] Checked that your issue isn't already filed.
[ ] Checked if no PR was submitted that fixes this problem.

Versions

PHP version:
Laravel version:
Package version:

Description

Steps to Reproduce

Expected behavior:

Actual behavior:

Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

Thanks for submitting the ticket. Unfortunately the information you provided is incomplete. We need to know which version you use and how to reproduce it. Please include code examples. Before we can pick it up, please check (https://github.com/Maatwebsite/Laravel-Excel/blob/3.1/.github/ISSUE_TEMPLATE.md) and add the missing information. To make processing of this ticket a lot easier, please make sure to check (https://laravel-excel.maatwebsite.nl/docs/3.1/getting-started/contributing) and double-check if you have filled in the issue template correctly. This will allow us to pick up your ticket more efficiently. Issues that follow the guidelines correctly will get priority over other issues.

3.1
laravel 5.5
php 7


Json Arrays have inconsistent nesting levels across different OSs

Description
When creating a json array, the nlohmann library adds an extra level of nesting on Windows but not on Mac when the exact same code is run.
Reproduction steps
I am creating an array of 1 using a json object that I have already defined. Run the code on Mac, then run the same code on Windows.
This causes my_json to be double nested in square brackets on Windows and single nested on Mac:
nlohmann::json my_json;
...
j["key"]= nlohmann::json::array({my_json});

Whereas this creates a single-nested array on Windows but on Mac no array is created and "my_json" is added directly:
j["key"]= {my_json};
Expected vs. actual results
Expected:
"key": [
                {
                    "foo": "bar",
                    ...
                }
        ],

Actual:
"key": [
[
                {
                    "foo": "bar",
                    ...
                }
]
        ],

Minimal code example
No response
Error messages
No response
Compiler and operating system
CMake, Windows, Mac
Library version
3.9.1
Validation

[ ] The bug also occurs if the latest version from the develop branch is used.
[ ] I can successfully compile and run the unit tests.

This is not due to the OS, but due to the compiler's different behavior for braces initialization. See #2583.


ComposerLinter submodule

See #522

Current coverage is 13.64% (diff: 23.91%)

Merging #528 into master will increase coverage by 0.31%

@@             master       #528   diff @@
==========================================
  Files            10         11     +1
  Lines          3060       3151    +91
  Methods           0          0
  Messages          0          0
  Branches        717        735    +18
==========================================
+ Hits            408        430    +22
- Misses         2640       2709    +69
  Partials         12         12

Powered by Codecov. Last update 657c425...21ad18e

It would be great if you could add some more context to your commit messages. Yes we have context in the issue but that doesn't actually live with the code in the same way that commit messages do. The issue is part of GitHub not the repo itself.

To expand on my previous comment. Your code is hopefully self evident and has comments where needed to cover the "what" of your changes. Your commits automatically cover the "who" and "when". The commit messages and pull request should cover the "why" of the changes. Why were the changes necessary? Why/how do they improve the code? Why were they made now rather than before or at a later time? You don't have to literally answer these questions but this is the kind of stuff that makes up a good commit message and provides value to any developer that comes after you.

No I understand, I'll add some more context to the above commits.

Also your commit 21ad18e should be squashed into a previous commit, whichever one introduced the typo. If you were fixing a typo that has been in the code for a long time and was not part of your changes it would be a valuable commit. But since you introduced the typo yourself in a commit that is part of this PR it doesn't really do anything except to highlight your mistake.

Updated and squashed commit messages - you'll have to bear with me I'm pretty use to my one man repos!

I hope this gets in soon so I can do a linter for Psalm which cannot currently be installed globally - it must be installed using Composer in order to work.

@groteworld @jawshooah the guy went through all this effort, responded to feedback in a timely manner, even cleaned up his commit messages... You can’t leave him hanging like this!

This is missing:

changes to create_linter_info.json adding the ComposerLinter
Documentation for Composer Linter
adding it to ignore list in `lint/linter.py

Since this has been waiting a while. I'll do 1 and 3 for you, but it would be helpful if someone with knowledge of it could do the documentation page

nvm looks like 3 was done, and 1 has now been done. should be in next release. ETA 2 days
My apologies for the slow response time and thank you everyone for your work!


add --start-paused,--debug-port flags

Some work towards https://github.com/flutter/flutter/issues/1457.

add a --debug-port flag to allow the user to select the observatory port. This will let multiple apps run and be debuggable at the same time. flutter_tools will auto-select an available port on an arg of 0.
add a --start-paused flag. This will cause the VM to start isolates in a paused mode. We pass the flag to the engine currently - there is not yet a corresponding impl in the engine.

Android is wired up; iOS has TODOs.

LGTM


Server: Add Sentry support

Motivation
Sentry is a useful error-reporting service
Solution
This adds two new config values - sentry_dsn and sentry_environment. When sentry_dsn is provided, we enable Sentry reporting at the provided DSN with the supplied environment name.
Currently, we report both panics (handled by default by Sentry), and all warnings/errors logged through tracing

Open source review:
sentry is the official Rust SDK for sentry (https://github.com/getsentry/sentry-rust). It has an Apache 2.0 license


Revert "Add ChatGLM for llm and chat_model by using ChatGLM API (#9797)"

@etveritas reverting for now until this is resolved https://github.com/langchain-ai/langchain/pull/9797/files#r1330795585, apologies for merging too eagerly!

@baskaryan I just solved this it. Because the output of GLM will add ' at both ends, the result is like "'hello'". So we need eval to get the answer, and the GLM model output can be parsed normally in subsequent applications.

@baskaryan Hi, In the latest version, the use of the eval function to parse the output has been removed, and the return_type parameter in the ChatGLM API is used to qualify the output format.


[sigh] add keychain_path option

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.
[x] I've added or updated relevant unit tests.

Motivation and Context
Sometimes (right now for us) there's a need to use 'non-default' keychain for downloading apple provisioning profiles.
This PR adds a possibility to pass it using '--keychain_path' or '-k' flag.
Description
The code in sigh action was ready for passing external flag 'keychain_path', but there's was no way to actually pass it in CLI.
Testing Steps
Tested in CI and local environment.

Hey @rogerluan , thank you for your review.
Those are indeed nice points, so regarding

I indeed didn't take into consideration passing of password, but now thinking about it, looks like in most cases keychain should already be unlocked before executing sigh action. I assume this is by design and additionally implementing unlocking of keychain should probably be a separate feature.
Honestly I didn't had a chance to use match as we're using sigh for years now without any issues.  I think this is due to our product being built solely in CI environment and because of no real issues with certificates and profiles match was not considered. Just now we're starting migration to a more strict environment.


Seems like the diff-gaussian-rasterization repo is not available yet?

Using the https way to access the repo seems to work:
git clone --recurse-submodules https://github.com/xg-chu/diff-gaussian-rasterization.git

Same thing happens to the submodule.
https://github.com/xg-chu/GAGAvatar_track.git


feat: bash completion

fixes: #2707

I forgot to note - we should somehow reuse the kongpletion completion for readline.

I figured, I am investigating how to do it. We may also want to provide more advanced completers at some point for things like verb names etc


feat: add setting to always display the first item in FluentOverflow

🙋 Feature Request
FluentOverflow hides items if there aren't enough room to display them. However, sometimes you might want to display the first item, even if it is cutoff.
For example, given a list of names, seeing the first name, but having it cut off with an ellipsis (someone can add CSS to add ellipsis) so the user can interact with it is preferrable to just showing the overflow button and no content.
🤔 Expected Behavior
The first item is displayed is the space available, then the overflow button.
This is what it could look like:

😯 Current Behavior
FluentOverflow only shows the first item if there is room for it. If there isn't then just the overflow button is displayed.
This is what it looks like today:

💁 Possible Solution
Add a setting to FluentOverflow that controls whether the first item is always rendered inline.
🔦 Context

💻 Examples

The script used to determine which items overflow actually already accounts for this...
By supplying a fixed attribute to an item, it will be excluded from the calculations (we already use that internally with the AppBar). We just did not expose this in any way in the FluentOverflowItem. I'm creating a PR to add this as a parameter.


Failed install Windows 10 vsc 1.35.0

Install says it cannot complete installation after offering to update during edit session.
vscode-inno-updater-1560127695.log](https://github.com/microsoft/vscode/files/3270194/vscode-inno-updater-1560127695.log)
Tried to install from download and Explorer target says contents do not exist.

I have the same problem. The update to 1.35.0 fails and refers to the vscode-inno-updater-log...

When I retry to install the update, the following error dialog appears:

VS Code Version Info:

Finally, after the third try (restart VS Code - update 2nd try failed - restart Windows - update 3rd try failed - install Windows updates - restart Windows again), the update eventually succeeded...

@larrym Can you try to fully reinstall VS Code? Maybe use the user setup?

I did reinstall from  https://code.visualstudio.com/. But as I said
earlier, the shortcut created on the desktop pointed to '%windir%\Progam
Files' when in fact it was installed at
'c:\users{username}\AppData\Local\Programs\Microsoft VS Code'. I had to
reset the icon too.
On Wed, Jun 12, 2019 at 2:16 AM João Moreno<EMAIL_ADDRESS>wrote:

@larrym https://github.com/larrym Can you try to fully reinstall VS
Code? Maybe use the user setup?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/75166?email_source=notifications&email_token=AAKMH3T3Q2R2B3BPRL2QZBDP2C5FNA5CNFSM4HWPVZY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXPYUHY#issuecomment-501189151,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKMH3UOXVCRR6IFEMO6URDP2C5FNANCNFSM4HWPVZYQ
.


Support session variable

pls add some description @lonng

You can reference MySQL session variable syntax. https://dev.mysql.com/doc/refman/5.6/en/set-variable.html

We need to add a new SET syntax
Add a session variable struct in the package stmtctx to store all session variables.

You can reference MySQL session variable syntax. https://dev.mysql.com/doc/refman/5.6/en/set-variable.html

We need to add a new SET syntax
Add a session variable struct in the package stmtctx to store all session variables.

ok, by the way, Please close the completed issue


Add a horizontal scroll bar to code blocks in help center when needed

Some code blocks in the help center have lines that are too long to fit, and wrap onto the next line. This can look awkward, and may cause confusion (is it still the same line or a new line?).
For example, on https://zulip.com/help/import-from-mattermost:

To improve this, instead of wrapping long lines in code blocks, we should add a horizontal scroll bar when a line is too long to fit.
CZO discussion

@zulipbot claim

i am a beginner at contributing to open source can you please help me out with some
beginner friendly issues

@akarsh-jain-790 do you need  any help with this?

Hi, plz assign me this issue I want to work on this.

@zulipbot claim

@zulipbot claim


[modify] stop accessing to database with @cur_site

Coverage decreased (-3.3%) to 72.923% when pulling 25f4a1ce186f33960033a1128c0ba4f301efba74 on feature/remove-db-access into 08f0f2e3f63a151fe0925c53cde0181100d055b8 on master.


Job shows as passed if a post-job action fails the job

I have jobs where I use the console parse plugin to potentially fail the job. This happens after the job has returned. If a post job action fails the job is still shows passed on the dashboard. Any tips to fix this?

I have the same issue. Many of my job have post build actions which may or may not fail, and it is critical for me to be able to report the status using the plugin.

We need to get the plugin to run last and I suppose it might be possible to have a re-evaluate button on jobs that update things after completion - like the Claim plugin.


Fix how bundles to be tracked are selected for DSIstudio autotrack

Changes proposed in this pull request
The get_bundle function was updated to select parent bundles  by default and only select sub bundles if their name was explicitly stated in the track_id parameter in the recon spec.
This PR addresses the following issue: #120
Documentation that should be reviewed

Codecov Report
Attention: Patch coverage is 0% with 12 lines in your changes missing coverage. Please review.

Project coverage is 33.68%. Comparing base (bd3f918) to head (da3849c).

Files with missing lines
Patch %
Lines

qsirecon/interfaces/dsi_studio.py
0.00%
12 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
- Coverage   33.74%   33.68%   -0.06%
==========================================
  Files          56       56
  Lines        6852     6863      +11
  Branches      901      905       +4
==========================================
  Hits         2312     2312
- Misses       4435     4446      +11
  Partials      105      105

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Question about 796.com wallet

I've tried to transfer BTC to any of the wallets available on 796.com, but in the log file I see: 796.com: 0.00 USD 0.000000 BTC. Which wallet should I use?

796.com has been implemented but never tested. So it might be a bug. Are you familiar with C++ so you can help troubleshooting?
That being said, I'm personally not familiar with 796.com and I see three different balances we can get from the API:

btc: wallet balance
mri: balance of 796Xchange-MRI
asicminer: balance of ASICMINER-796

Can you explain the differences? I guess your BTC can be retrieve with btc. What about your USD?

796.com has these wallets which are used for BTC trading:

Spot account for spot BTC trading
Margin account for futures BTC trading

Unfortunately I'm not familiar with C++, however I'll try to look into the code, maybe I'll find a way on how to fix it.

Which one is the spot account? mri or asicminer?

Not sure what is mri in 796.com, but here are the wallets you can transfer money between:

Main Wallet
Spot Account
Quanto Contracts(USD) Margin Account
XinHua A50 Margin Account
E-mini Dow Margin Account
DAX Margin Account
NK225M Margin Account
USD/CNH Margin Account
EUR/USD Margin Account
USD/JPY Margin Account
AUD/USD Margin Account
GBP/USD Margin Account
XTI/USD Margin Account
XAU/USD Margin Account
XAG/USD Margin Account

Though according to API docs all you can retrieve are:

Main Wallet
margin account (with get_assets)

So I'm completely out of idea what is 796Xchange-MRI

I contacted 796.com to get more details about their API. I'll keep you posted.

Thanks, waiting for reply. However withdrawal doesn't work on their page and their support stopped replying, so it could be that it's not very reliable exchange to trade on :)

Finally support contacted me and they found out that in order to withdraw - you have to enable 2-fa and add phone number. So it's pretty safe, I'd say to use.

The goal on 796 is to use the futures with leverage to initiate and unwind the arbitrage.
The coins should be in your margin account.
When you go long on per say bit bitfinex you would then initiate the open sell order on 796 futures
When you sell the bitfinex coin you would then execute a close sell order.
I would recommend setting this up on bitvc.com (huobi) same contract pretty much but way more liquid


fix (#4199): let postcss pass-through sourcemap

resolve issue #4199 CSS Sourcemap is empty.

@asnowwolf I think this was fixed in https://github.com/angular/angular-cli/pull/4222. If that's incorrect, let me know.

@filipesilva I think the root cause is post-css-loader eat the sourcemap which generated by sass-loader. I'm not sure that PR is resolved the issue. I'll test it later.

@filipesilva
I just tested the latest master branch and it still lost the sourcemap (I turned on the sourcemap and extractCss options)

Can you give me some repro steps to see that? I added a few tests in #4222 and they verify mappings are being generated.

see https://github.com/asnowwolf/t1
the step is:

pull the latest master branch of angular-cli
npm link it as global package
ng new t1 --style=scss --extract-css
npm i --save<EMAIL_ADDRESS>add @import '~bootstrap/scss/bootstrap'; to style.scss
ng serve
in chrome, I can't see the sourcemap correctly.

Sorry, I've test it again, It's right now. I think there are some mistakes on my first test.

I was just testing myself and your demo seemed to give me the sourcemaps. I think you didn't npm link angular-cli to have the master version in the project.
Anyway, glad it's working for you!

Sure, I think that's the mistake. thank you.


Add a confirmation message to contractor on delivering contract stage

It would be great to have a confirmation message to notify contractor that he/she in a danger zone and this operation cannot be reverted. We want to let them know that after delivering results customer will be able to choose between approving contract or declining contract and if they didn't provide their best value for the customer, then they will not be able to get money from the contract.
How it can be implemented:

Add a new component ConfirmationMessage in src/components/ConfirmationMessage/index.jsx
Provide three props: description (string), onClose (function), onConfirm (function)
Use description in Modal.Description
Pass onClose and onConfirm callbacks to Modal component
Call this component when contractor clicks on "Deliver" button

Acceptance criteria:

When contract was started, contractor will see a "Deliver" button on a screen
When contractor clicks on "Deliver" button, the confirmation message will appear on the screen
If contractor clicks "Close" button, nothing will happen and contract should not change its state
If contractor clicks "Confirm" button, contract sends a new status on blockchain (using existing code)
Please suggest descriptive message to contractor. Like "At this stage we notice you that you will not able to change contract status after delivering result. If customer will decide to decline your contract, you will not be paid" or something else.

References:

https://github.com/optriment/optrispace-frontend-v2/blob/master/src/forms/NewJobForm/index.jsx#L39
https://github.com/optriment/optrispace-frontend-v2/blob/master/src/forms/NewJobForm/index.jsx#L340
https://github.com/optriment/optrispace-frontend-v2/blob/master/src/components/UnsavedChangesDialog.jsx
https://github.com/optriment/optrispace-frontend-v2/blob/master/src/components/ContractCardForContractor/index.jsx#L97
https://github.com/optriment/optrispace-frontend-v2/blob/master/src/components/ContractCardForContractor/index.jsx#L366

When this feature will be implemented, then we reuse this dialog on another steps: "Approve" and "Decline" on customer's side.

https://github.com/optriment/optrispace-frontend-v2/pull/28


Catching errors

What's the best way to rescue a Errno::ECONNRESET error? This error can be caused from the mixpanel api being down.
Right now I'm doing something like the following in a controller which doesn't feel quite right:
def track
  MetaEvents::Tracker.new(
    distinct_id,
    request.ip,
    implicit_properties: implicit_properties
  )
rescue Errno::ECONNRESET => error
  MetaEvents::Tracker.default_event_receivers = [MetaEvents::TestReceiver.new]
end

If the Mixpanel API is down, you want to avoid losing the events that are firing by sending them to the TestReceiver.
Take a look at performance-and-asynchronousness, in the documentation. If you enqueue the job inside delayed_job for example, the job will fail to complete until the Mixpanel API is back up. -- However, unmentioned in the documentation, is that when using background jobs to send events, be sure to inject the timestamp for the event as a property.

Makes sense. Thanks!


Get Data from existing realtime database object

I have a realtime database set up and I am trying to get data from it. I could only find new_data_updated and patch_data_updated signals, how do I get existing data?

It should be in the new_data_updated signal.

var fstore
fstore = Firebase.Database.get_database_reference('Users/01ojEi6zjaNR82CxvBc0Zfvt9pa2', {})
fstore.connect("new_data_update", self, "on_received_new_chat")
and I never get a signal with the data that is there in the database, am I doing something wrong?

@Zarasan when new data is added, can you see it in the console at that path?
Are you sure the path you have connected to is correct?

Can I see the definition of the on_received_new_chat function?

@BearDooks I don't need to wait for new data like in the chat example, I have user information already defined there and I need to access it
@WolfgangSenff its just a simple print(data) function

This problem was on my end. Sorry for disturbing you.
Thank you so much for helping

It's cool - it's what we're here for. :) Good luck!

@Zarasan - by the way, I recently fixed a dumb bug with the authorization stuff. The fix is already in latest, but if you're using the branch from the Godot Asset Library, you likely won't have it. It's an easy fix though and since it's the latest thing to be merged, very easy to see what you have to do for the fix.


connect() hangs if stream ends before SFTP ready

Normally, one expects SFTPClient.connect to either connect or fail (reject) after a given time. However, it could happen that the connection closes between establishing the actual SSH channel and getting an SFTP subsystem. Indeed, given the following sequence of events, we get a "hung Promise" that never fulfills or rejects:

call SFTPClient.connect
the underlying SSH client connects and emits ready
the connection is broken (for example by TCP RST) - the client emits end, but no error (!)
the promise returned from SFTPClient.connect (1) is now hung, as it will never reject (because we got no error)

It would be best to also listen for SSHClient event end as long as we're connecting and reject as if we got an error.

thanks for pr. and merged


apktool rebuild - unresolved class , unresolved super class

Information

Apktool Version (apktool -version) - 2.2.0
Operating System (Mac, Linux, Windows) - Linux
APK From? (Playstore, ROM, Other) - Playstore

Stacktrace/Logcat
Include stacktrace here

Steps to Reproduce

apktool decode
make modifications to manifest and code
apktool build
sign apk
install apk
run app , everything working great. sometimes it crash due to unresolved class access which is something i didnt touch or is any relevant to the code i touched.

Frameworks
If this APK is from an OEM ROM (Samsung, HTC, LG). Please attach framework files
(.apks that live in /system/framework or /system/priv-app)
None
APK
If this APK can be freely shared, please upload/attach a link to it.
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? Yes
If you are trying to install a modified apk, did you resign it? Yes
Are you using the latest apktool version? 2.2.0

I localized an app, and it works most of the time, and it crashes in some other times because of "unresolved class ...." , "unresolved super class"... of classes i never touched.
i use apktool decode, then make modifications very carefully, apktool build , sign and install.
the app works great ( for months.. ), but sometimes it crashes with the error above.

Do you have an adb logcat of the host device during this crash/failure? Seems smali related, but with no binary or log nothing really I can do here.

Closing due to no response in 1+ months. I need either an apk or adb logcat of the failure to investigate. I can re-open and re-investigate if provided.


fix bpfctl help command

Enable clap default features which automatically creates the help target for bpfctl.

@Billy99 Was actually doing this right and writing a test 👑


quivering cursor and graphic effect on click boxes

When hovering the mouse over the bottom edge of the More Information or Install TW/Leap boxes, the cursor suddenly has a seizure and starts dancing. The text in the paragraph above also displays shaking symptoms.
This occurs on my oS 13.1 64-bit install with FF40.0.3 and the nVidia proprietary driver. I don't see the same symptoms in Konqueror.

lol that is a bit funny. I havent seen the effect in Chrome, Firefox and IE versions I have installed, but I will have a look at it soon :)

Reproduced with FF 40.0.3 on openSUSE Tumblweed (I have Intel Graphic card on my laptop). There are no 'dancing' symptoms in Google Chrome 45.0.2454.93 :)


Error passed to callbacks is not an instance of "Error" class on connection refused

When redis connection is refused the client callback is invoked with something that is not an instance of Error. This has been tested with 0.7.1, dunno if it changed lately (but I saw no sign of that in the changelog.md)

@strk this should have been fixed on master (only the js parser still had this error) and will be fixed in the next release.


Error

https://mclo.gs/kuklERi
I don't know the circumstances to reproduce.

Why do you think its pl3xmap issue?


Issue on page /vis-common-plots.html

You may be wondering why Lets-Plot isn’t featured here: its functions have almost exactly the same names as those in lets-plot, and we have opted to include the latter as it is currently the more mature plotting package.

Did you mean

You may be wondering why plotnine isn’t featured here..

?

I did, thanks so much for pointing this out!
Arthur
On 7 Aug 2023, 20:14 +0100, Igor Alshannikov @.***>, wrote:

You may be wondering why Lets-Plot isn’t featured here: its functions have almost exactly the same names as those in lets-plot, and we have opted to include the latter as it is currently the more mature plotting package.
Did you mean
You may be wondering why plotnine isn’t featured here..
?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

closed in 564f667


Incorrect (potentially) indentation of function arguments spanning multiple lines

From @nomaed on October 14, 2016 10:40

VSCode Version: 1.6.1
OS Version: macOS Sierra 10.12

Steps to Reproduce:

Write a function (or a method) with several arguments (I did it with JavaScript and TypeScript files, but I believe any would work).

function myFunc(arg1, arg2, arg3, ...args) {
}

Split arguments to several lines by hitting Enter before arguments

function myFunc(arg1,
arg2,
arg3,
...args) {
}

Auto-formatting produces this result:
function myFunc(arg1,
    arg2,
    arg3,
    ...args) {
}

I would expect to see this result instead though:
function myFunc(arg1,
                arg2,
                arg3,
                ...args) {
}

Also, when manually formatting the arguments to appear in the same column (note: this is also the default/recommended setting in tslint and maybe other linters), then further lines will start with wrong indentation:
function myFunc(arg1,
                arg2,
                arg3,
                ...args) {
                    console.log('huh...');
}

Copied from original issue: Microsoft/vscode#13748

@bowdenk7 if you have a thought on this too that would be helpful. Don't know if this has come up much in your user studies...

@nomaed thank you for opening this issue.
@dbaeumer can I get some other eyes on this? I think I actually prefer this result with auto formatting:
function myFunc(arg1,
    arg2,
    arg3,
    ...args) {
}

But that might simply be my preference. Possibly there should be an option for OP to customize the auto formatting for his preference?

Moving this to TS language service to address adding the setting. Thank you for your feedback @nomaed. It is valued very highly.

From @dbaeumer on October 14, 2016 15:4
@waderyan I prefer yours as well. Agree that this should be a setting in the TS formatter. The good thing is if you formatted it once it will leave it as is.

From @nomaed on October 14, 2016 15:8
It definitely shouldn't be the only option. I personally grew to like it a lot (it's the default setting in JetBrains' IDEs) and since it's a default for TSLint (https://github.com/palantir/tslint/blob/master/src/rules/alignRule.ts), I am thinking that it's not uncommon to use this setting.

What about the returned type? It should be indent, in next line, or how?
class Example {
    public method(arg1: string,
                  arg2: number,
                  arg3: boolean)
                  : ReturnType {
        const result = doSomething();
        return new ReturnType(result);
    }
}

Return type (at last the semicolon) is immediately following the args
closing parenthesis, so to make it appear in a new line, the closing
parenthesis should be moved down too.
On Sat, Oct 15, 2016, 14:54 19majkel94<EMAIL_ADDRESS>wrote:

What about the returned type? It should be indent, in next line, or how?
class Example {
public method(arg1: string,
arg2: number,
arg3: boolean)
: ReturnType {
const result = doSomething();
return new ReturnType(result);
}
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/TypeScript/issues/11629#issuecomment-253979847,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJHAMSYtSdVIa0PaSaRHcw4JW1YkQ7ZLks5q0L7agaJpZM4KXHg9
.

@19majkel94 Option no. 2 probably.
Return type (at last the semicolon) is immediately following the args closing parenthesis, so to make it appear in a new line, the closing parenthesis should be moved down too.
I think that these should all be valid:
/**
* #1
 * If first arg is on the same line as the function decl/name,
 * all args are lined up with the first one.
 */
function funcname(arg1: string,
                  arg2: number,
                  arg3: any): ReturnType {
    // ...
}

/**
 * #2
 * If first arg is newlined, it is double-indented so it remains 1 indent
 * level in front of impl., and all args are lined up.
 */
function funcname(
        arg1: string,
        arg2: number,
        arg3: any): ReturnType {
    // ...
}

/**
 * #3
 * In case first arg is newlined like in #2, allow closing parenthesis
 * to be newlined too (outdent once) with return type following it
 */
function funcname(
        arg1: string,
        arg2: number,
        arg3: any
    ): ReturnType {
    // ...
}

The reason why I don't like the way it is now is that the argument aren't aligned and it's decreasing readability.

In case no. 3, the return type indent once is decreasing readability. So I prefer my option no. 2 but it's
sensitive to function rename - we have to indent all again.

My prefered style for large method/function signatures:
class Example {
    public method(
        arg1: string,
        arg2: number,
        arg3: boolean
    ) : ReturnType {
        const result = doSomething();
        return new ReturnType(result);
    }
}

I'd like to add that the current indentation rule appear to work incorrectly when combined wth multiple nesting levels. In a naked VSCode install, auto format produces this:
this.client
      .getContinents()
      .subscribe(
      x => this.continents = x,
      e => this.errorMessage = e
      );

instead of:
this.client
      .getContinents()
      .subscribe(
          x => this.continents = x,
          e => this.errorMessage = e
      );

Maybe I am missing something, but my project has the following tsling.json file:
{
    "defaultSeverity": "error",
    "extends": [
        "tslint:recommended"
    ],
    "jsRules": {},
    "rules": {
        "quotemark": [
            "single"
        ],
        "no-console": [
            false
        ],
        "arrow-parens": false,
        "no-string-literal": false,
        "no-namespace": [
            "allow-declarations"
        ],
        "interface-name": [
            "never-prefix"
        ],
        "no-trailing-whitespace": [true, "ignore-comments"]
    },
    "rulesDirectory": []
}

And TSLint does complain about parameters not being aligned after I auto format (using default settings, I am pretty sure since I am not overriding anything with typescript.format).
I know I can turn off TSLint's rule, but I would prefer to control settings for this...

It would be best to provide a configuration so a user can choose which way to go.

This is still an open issue in VS code 1.30.1 - the TS autoformatter removes leading whitespace, even if the tslint config is:
 "rules": {
    "align": [
      true,
      "parameters",
      "arguments",
      "statements",
      "members",
      "elements"
    ],

The comments above RE "I prefer a single tab indent when continuing the parameter list on a newline" aren't particularly useful - this is still a bug.

What about this issue?
Same problem here.

+1 for adding this formatting option in settings

+1 for this. Just migrating from Intellij to VS Code and this is one of the things that bothers me a lot. There are so many ways to configure the editor, but this simple logic requirement is not configurable. Any progress about that?

FWIW, we have been using prettier now for a couple of years. Really helped with this and other code formatting consistency issues across our teams/products.

+1 for adding this formatting option in settings

+1. I would like to be able to set it the same way as my TSLint is configured.

FWIW, we have been using prettier now for a couple of years. Really helped with this and other code formatting consistency issues across our teams/products.

Prettier moved all the args on the same line... So it's not helpful at all with this.

FWIW, we have been using prettier now for a couple of years. Really helped with this and other code formatting consistency issues across our teams/products.

prettier is opinionated. We need to fix this in vscode if possible.

Related Stack Overflow question: VS Code document formatter to indent multi line arguments vertically aligned to the opening bracket/brace of the argument list

I'd also like to +1 this. Our team uses parenthesis aligned parameters for all other programming languages and it's a pain to work with TS due to this issue. We need a way to control parameter alignment on newlines.

hi, did this issue fixed?

Has anyone figured out a workaround until this can get resolved?
The biggest issue, IMO, is the fact that following lines also get their indentation messed up. So even if you indent things manually, you end up fighting the editor for the next line.


fix: Revised COVID-19 Google Mobility dataset

Description
This DAG is failing in production and so this PR attempts to resolve the issues by both correcting the code and making the code consistent with existing implementations on this project.
Dataset: covid19_google_mobility
pipeline/s: mobility_report
Checklist
Note: If an item applies to you, all of its sub-items must be fulfilled

[x] (Required) This pull request is appropriately labeled
[x] Please merge this pull request after it's approved
[ ] I'm adding or editing a feature
[x] I'm adding or editing a dataset

[x] The Google Cloud Datasets team is aware of the proposed dataset
[x] I put all my code inside  datasets/covid19_google_mobility> and nothing outside of that directory

[x] I'm refactoring or cleaning up some code

@adlersantos Changes complete.  Please review.


feat: Add yay provider

Hey! Very cool project! :smile:
Was setting up an old laptop at home and came across this very promising project. My wife and I are using Arch/Manjaro and having support to yay/pacman would be great!

Hey. Thanks for the PR. I'll get that code coverage action disabled on forks, that's never going to work because of the secret.
The bootstrap function uses pacman to install Yay, but it's only available in the AUR (maybe different on Manjaro?). I'm actually OK with the bootstrap returning an error for now and leaving a message that says to manually install yay, as it's not easy to coordinate multiple steps with command execution yet, or raise privilege; though this will be ready by tomorrow #50
Thanks again 😀

Closes #16

You are right! Forgot that in a pure arch box  you have some extra steps to have an AUR helper available :sweat_smile:
I added more commands if you want to merge this one before the command privilege escalation PR. I can also wait for the other one to get merged and rebase it here if you prefer :)
Tested this in a pure Arch machine and it seems to work.

Saw that you merged it just now, already rebased it here and started using the new command approach

Damn, that was fast! The new run_command helper should be much easier for supporting new providers like this. I'll get this merged when you're happy with it 😀

Sorry, I didn't realise you were new to Rust. I've gone through and tried to identify all the little pesky mistakes.
Let me know if you want me to push any fixes instead 👍🏻

Sorry, I didn't realise you were new to Rust. I've gone through and tried to identify all the little pesky mistakes.
Let me know if you want me to push any fixes instead 👍🏻

Thanks! Yeah, only touched Rust before in Advent of code challenges :smile: Gonna eat something and fix all the details!

Also sorry for Pushing code with compilation errors. I am pulling this in a fresh vm to test in a place without the AUR helper 😅 gonna tidy up in a bit

Also sorry for Pushing code with compilation errors. I am pulling this in a fresh vm to test in a place without the AUR helper 😅 gonna tidy up in a bit

Not a problem, I'm here if you need any help. Have fun!

Are you happy with this now?

Are you happy with this now?

I am happy :) I think that after CI we can merge. Thanks for reviewing @rawkode!

Perfect. I'll dig in to the sudo thing this evening.  Sorry about that!
dabfb77
@knelasevero

😹 no worries! Cool that you found it!

Btw, @rawkode , if you need anything tested on arch I can happily do that here ;)


Images in UI app does not show up

Once the ui-app is spun up in reads information just fine from the album-api but the image links such as https://aka.ms/albums-daprlogo, https://aka.ms/albums-containerappslogo are broken due to public access not allowed (see below for error):

PublicAccessNotPermitted
Public access is not permitted on this storage account. RequestId:483c018f-801e-0015-3f98-7635d6000000 Time:2022-06-02T15:52:45.9635581Z

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

ID: e1779a5c-d2f0-d5c6-ef58-238db5a0ff4e
Version Independent ID: 077bc418-7885-5bd6-1304-03b48947ee07
Content: Tutorial: Communication between microservices in Azure Container Apps
Content Source: articles/container-apps/communicate-between-microservices.md
Service: container-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@lekkimworld Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Should be working and updated. @lekkimworld if your refresh the app it will pick up the new backing images

#reassign:cebundy

Closing the issue.  Please feel free to reopen the issue if you still encounter this problem.
#please-close


Support for RTSP

Does your great player support an RTSP or an RTSPH stream?
If not, this would be a feature request :) Do you think you can support something like that?
thx

Plyr is using HTML5 video. As long as browsers don't support RTSP (which will likely never change since it's a legacy format), neither will Plyr. To support RTSP you either need to use flash, or you can convert the stream with your own server and ffmpeg.


Support dark launch governance

Coverage increased (+0.8%) to 64.817% when pulling 8583d3a4a2ddcc0f3a9e5d08d0e51de4a6607f90 on humingcheng:master into 9ae39b5930aefee2e2c3da7e47be20e3293318cd on ServiceComb:master.

re-org the package the translate.go should be in a package like adaptor/cse, it adapt to CSE governence, event listener will call adaptor/cse to translate cse config into route rule


Actions version updated and pretty format workflow

Description
Actions' version updated for the following:
# .github/workflows/check-md-link.yml
actions/checkout@master -> actions/checkout@v2
# .github/workflows/stale.yml
actions/stale@28ca1036281a5e5922ead5184a1bbf96e5fc984e -> actions/stale@v9

Pretty format workflow files to use consistent layout.
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Documentation update
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Hardware:
Toolchain:
SDK:

Checklist:
Please delete options that are not relevant.

[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] If adding new functionality, I have discussed my implementation with the community using the linked GitHub issue

Should we add workflow_dispatch so people can run those tests within their own repo without merge into their own main repo?

Should we add workflow_dispatch so people can run those tests within their own repo without merge into their own main repo?

Isn't this a potential security risk? Afaik letting untrusted code checkout the project and build it should be disallowed.


where to add s3 sccess and secret key..??

Hi,
There is no sidecar  container in statefull set of server its only in deployment.??
Also where to give s3 access key etc.. itn not mention anywhere in value.yaml under s3 config.
Please suggest

Hey there! I'm sorry that you ran into problems, we can always do better with the documentation. I don't use K8s much myself but for Sidecar and other components we have the flags which start with: --objstore.*. By using them, you can specify the file with the bucket configuration or the actual configuration itself :smile: I hope that helps.

This doc should help : https://github.com/thanos-io/thanos/blob/master/docs/storage.md#s3


NodeJS: Add routing header

Corresponding gax: googleapis/gax-nodejs#155

Codecov Report

Merging #1694 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1694   +/-   ##
=========================================
  Coverage     84.98%   84.98%
  Complexity     4383     4383
=========================================
  Files           425      425
  Lines         17452    17452
  Branches       1822     1822
=========================================
  Hits          14831    14831
  Misses         2004     2004
  Partials        617      617

Continue to review full report at Codecov.

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


Calculate foorprint area

Useful to automatically calculate the instrument footprint somewhere. Should be a function of beam width and distance.
Beam width cited by Brenner as 2.9 degrees.

Implemented in calc.footprint


Need overflow-y to be an option

When I disable overflow-y: auto; in the inspector, all my troubles go away.  I have been trying to remove them myself using .style.overflowY = 'pleaseStop', but they keep getting set.
Have I missed something obvious again, or where can I remove overflow-y from angular-split?

I made it work with
split-area{
	overflow-y: hidden !important;
}

Hi @mfor
If you are still interested in this feature (I know it's been long time!), check this stackblitz:
https://stackblitz.com/edit/angular-andtvw?file=src%2Fapp%2Fapp.component.html
The 2.x is still beta, I need to test it on major browsers before release.