Whereami not displaying istio headers properly

Hi,
I deployed whereami:v1.1.1 today behind istio. Enabled headers but i don't see them in the output
My deployment manifest and configMap look liek this
apiVersion: apps/v1
kind: Deployment
metadata:
  name: whereami
spec:
  replicas: 2
  selector:
    matchLabels:
      app: whereami
  template:
    metadata:
      labels:
        app: whereami
    spec:
      containers:
      - name: whereami
        image: gcr.io/google-samples/whereami:v1.1.1
        ports:
          - name: http
            containerPort: 8080
        readinessProbe:
          httpGet:
            path: /healthz
            port: 8080
            scheme: HTTP
        env:
          - name: METADATA #An arbitrary metadata field that can be used to label JSON responses
            valueFrom:
              configMapKeyRef:
                name: whereami-configmap
                key: METADATA
---
apiVersion: v1
kind: ConfigMap
metadata:
  name: whereami-configmap
data:
  METADATA: "echo_headers_enabled"
  ECHO_HEADERS: "True"

The output is missing the headers
➜  istio-ingress-e2e-tls git:(master) ✗ curl https://gke1.abdel.cloud -s | jq .
{
  "cluster_name": "gke-1",
  "host_header": "gke1.abdel.cloud",
  "metadata": "echo_headers_enabled",
  "node_name": "gke-gke-1-default-pool-94ebed87-q2z9.c.lbg-project-278414.internal",
  "pod_name": "whereami-8454b9bbf8-5czfx",
  "pod_name_emoji": "👩❤👩",
  "project_id": "lbg-project-278414",
  "timestamp": "2020-12-02T09:46:44",
  "zone": "us-west1-a"
}```

Even if the metatadata is set echo_headers_enabled

What i expect is something similar to the echoserver, deployed the same way, the output looks like

curl https://gke1.abdel.cloud -s
CLIENT VALUES:
client_address=<IP_ADDRESS>
command=GET
real path=/
query=nil
request_version=1.1
request_uri=http://gke1.abdel.cloud:8080/
SERVER VALUES:
server_version=nginx: 1.10.0 - lua: 10001
HEADERS RECEIVED:
accept=/
content-length=0
host=gke1.abdel.cloud
user-agent=curl/7.72.0
via=1.1 google
x-b3-parentspanid=f43fa3a13fd64a8c
x-b3-sampled=1
x-b3-spanid=67c62c3cedef3d7b
x-b3-traceid=1b2af99f21ce5da6f43fa3a13fd64a8c
x-client-data=CgSL6ZsV
x-cloud-trace-context=7a4858982a3a0949ff73adfc32cc202e/4550108360082774779
x-envoy-attempt-count=1
x-envoy-external-address=<IP_ADDRESS>
x-forwarded-client-cert=By=spiffe://cluster.local/ns/default/sa/default;Hash=15b8e7eb324184a1aa7ae9598db6e500eaedee9caa7e5da6120499c3dbd95775;Subject="";URI=spiffe://cluster.local/ns/istio-system/sa/istio-ingressgateway-service-account
x-forwarded-for=<IP_ADDRESS>, <IP_ADDRESS>,<IP_ADDRESS>
x-forwarded-proto=https
x-request-id=136c88d9-cbe2-95d3-afd3-e0084eda08e6
BODY:

Ok let me take a look.

@boredabdel you're missing an env definition in your deployment spec. You only have the METADATA entry. That's just used as a string, but there's another env var named ECHO_HEADERS:
- name: ECHO_HEADERS
  valueFrom:
    configMapKeyRef:
      name: whereami-configmap
      key: ECHO_HEADERS

{"cluster_name":"asm-test-01","headers":{"Accept":"*/*","Content-Length":"0","Host":"<IP_ADDRESS>","User-Agent":"curl/7.64.1","X-B3-Sampled":"0","X-B3-Spanid":"a8c4ef49fb63d4a3","X-B3-Traceid":"8a240cd17a6bff8ea8c4ef49fb63d4a3","X-Forwarded-Proto":"http","X-Request-Id":"3d3fd5f0-dbd6-491e-977f-a6556ea27e34"},"host_header":"<IP_ADDRESS>","metadata":"echo_headers_enabled","node_name":"gke-asm-test-01-default-pool-f9ad78c0-4qe7.c.alexmattson-scratch.internal","pod_name":"whereami-abdel-c4bf8f85b-dz84q","pod_name_emoji":"🥗","project_id":"alexmattson-scratch","timestamp":"2020-12-04T05:03:30","zone":"us-central1-f"}```


Fatal Error on Linux during getAuthToken

notifications = PushNotifications(...
notifications.getAuthToken { ...

Somewhere in the getAuthToken call, this happens:
Fatal error: value(forKey:) is not supported on this platform: file Foundation/NSDictionary.swift, line 36
This does not happen on the Mac, only on Linux (and thus when deploying to the cloud.)
Do I need to include something I'm not including?

@kenlaws Hi, You can use the latest version 1.0.0. I have fixed the issue. 👍

Confirmed! Thanks!


About input c

Sorry to bother you with naive questions, but I don't figure out it after two days (stylegan2 ada is also not clear), I will really appreciate it if you could reply to me!
I wrote like below to input a image with label into the Discriminator of your pretrained model
import torch
import dnnlib
import pickle
import legacy
import PIL
import numpy as np
network_pkl ='./pretrained/imagenet64.pkl'
device = torch.device('cuda')
with dnnlib.util.open_url(network_pkl) as f:
        D = legacy.load_network_pkl(f)['D']
        D = D.eval().requires_grad_(False).to(device)
image =torch.from_numpy(np.array(PIL.Image.open( './data/01.JPEG'))).float().permute(2,0,1)
image = image.reshape((1,3,64,64))
image = image.to(device)

c = torch.randn([1,1000,3,3],dtype=torch.float).cuda()
print(D(image,c))

But I still don't know how to write the label , could I ask if you could give an example of input c, (suppose here is class 0)?
Thank you very much!

Maybe from the README:

For class-conditional models, you can pass the class index via --class.
A index-to-label dictionary for Imagenet can be found here.

see above :slightly_smiling_face:


Add 9Slice schema

This PR adds 9Slice schema for textures.

Closing this for now, no idea what I need to do with it 😬


Clicking Submit results in no maxWalkDistance parameter

When OTP.js creates an itinerary from loading the page, it includes a maxWalkDistance parameter and works fine.
http://maps.cherriots.org/planner/#plan?fromPlace=44.94728163729739%2C-123.07718009708354&toPlace=44.90988708140662%2C-123.0484593438651&date=06%2F21%2F2016&time=9%3A46 AM&maxWalkDistance=8046&optimize=QUICK&mode=TRANSIT%2CWALK&numItineraries=3&wheelchairAccessible=true
When the itinerary is request through the "Submit" button, no maxWalkDistance parameter is supplied, and no itineraries are found.
Thanks!
Ed

This has been solved by @ed-g in https://github.com/ed-g/otp.js


Protecting Substrings During Translation

Hi MMT team,
In issue https://github.com/ModernMT/MMT/issues/286 regarding protecting substrings that are part of the sentence i.e. using <dnt> tags alone won't work, the solution suggested is to use a single placeholder, with a tag.
I've trained an en-es engine using content with a placeholder for protected strings say XXXX. When translating the string:
A window that displays metrics for the entire XXXX<dnt 3 /> is available to XXXX<dnt 2 /> administrators with the XXXX<dnt 1 /> permission.
The translated text is:
Una ventana que muestra métricas para todo el XXXX<dnt 3 /> está disponible para<dnt 2 /> los administradores XXXX con<dnt 1 /> el permiso XXXX.
As you can see this method doesn't work, and performs even worse when the source string contains other XML elements.
The solution I have been working involves a fixed set of placeholders to represent a piece of non translatable text i.e. XXXA, XXXB, XXXC etc.
So the source string:
A window that displays metrics for the entire XXXXA is available to XXXXB administrators with the XXXXC permission.
gets correctly translated to:
Una ventana que muestra métricas para toda XXXXA está disponible para los administradores XXXXB con el permiso XXXXC.
This method however, doesn't work with en-zh or en-ja as the placeholders get translated, tokenized, or their case is changed i.e
XXXXA becomes XXXxa or XXX Xa or Xxxxa and sometimes XXXXC  etc.
This make it's quite difficult/impossible to replace the placeholders. I should add however, the translated sentence is linguistically correct, the only problem is the modification of the placeholders.
For reference I tested this same approach against a MMT 2.3 engine, which was trained without these placeholders, and it handled them correctly, there was a negligible amount of placeholders that were modified.
Do you have any ideas why the issue is worse with MMT 2.4?
Thanks,
R.

Hi @robertBrnnn
let's make a test. Engines EN <> ES are backward compatible from 2.4 and 2.3. Can you please run the following test: translate the exact source file with the exact same engine (don't perform any re-training of any sort, use exactly the same engine folder not only the decoder).
Do the 2 files differ? If yes, can you please provide me the output of the two system on a single line with parameter verbose=1? I would like to inspect if the alignment information is the same.

Hi @davidecaroselli
I assume you mean for my first method i.e.
A window that displays metrics for the entire XXXX<dnt 3 /> is available to XXXX<dnt 2 /> administrators with the XXXX<dnt 1 /> permission.?
So the translations are the same.
On MMT 2.3 I get
{     "status": 200,     "data": {         "decodingTime": 1916,         "translation": "Ventana que muestra las métricas para toda la XXXX<dnt 3 /> disponible para<dnt 2 /> los administradores de XXXX con<dnt 1 /> el permiso XXXX.",         "sourceWordCount": 19,         "targetWordCount": 20,         "translationTokens": [             [                 "Ventana",                 " "             ],             [                 "que",                 " "             ],             [                 "muestra",                 " "             ],             [                 "las",                 " "             ],             [                 "métricas",                 " "             ],             [                 "para",                 " "             ],             [                 "toda",                 " "             ],             [                 "la",                 " "             ],             [                 "XXXX",                 ""             ],             [                 "<dnt 3 />",                 " "             ],             [                 "disponible",                 " "             ],             [                 "para",                 ""             ],             [                 "<dnt 2 />",                 " "             ],             [                 "los",                 " "             ],             [                 "administradores",                 " "             ],             [                 "de",                 " "             ],             [                 "XXXX",                 " "             ],             [                 "con",                 ""             ],             [                 "<dnt 1 />",                 " "             ],             [                 "el",                 " "             ],             [                 "permiso",                 " "             ],             [                 "XXXX",                 ""             ],             [                 ".",                 ""             ]         ],         "sentenceTokens": [             [                 "A",                 " "             ],             [                 "window",                 " "             ],             [                 "that",                 " "             ],             [                 "displays",                 " "             ],             [                 "metrics",                 " "             ],             [                 "for",                 " "             ],             [                 "the",                 " "             ],             [                 "entire",                 " "             ],             [                 "XXXX",                 ""             ],             [                 "<dnt 3 />",                 " "             ],             [                 "is",                 " "             ],             [                 "available",                 " "             ],             [                 "to",                 " "             ],             [                 "XXXX",                 ""             ],             [                 "<dnt 2 />",                 " "             ],             [                 "administrators",                 " "             ],             [                 "with",                 " "             ],             [                 "the",                 " "             ],             [                 "XXXX",                 ""             ],             [                 "<dnt 1 />",                 " "             ],             [                 "permission",                 ""             ],             [                 ".",                 ""             ]         ],         "alignment": [             [                 1,                 0             ],             [                 2,                 1             ],             [                 3,                 2             ],             [                 4,                 3             ],             [                 4,                 4             ],             [                 5,                 5             ],             [                 7,                 6             ],             [                 8,                 7             ],             [                 8,                 8             ],             [                 11,                 10             ],             [                 12,                 11             ],             [                 13,                 15             ],             [                 13,                 16             ],             [                 15,                 13             ],             [                 15,                 14             ],             [                 16,                 17             ],             [                 18,                 21             ],             [                 20,                 19             ],             [                 20,                 20             ],             [                 21,                 22             ]         ]     } }
On MMT 2.4 I get:
{     "status": 200,     "data": {         "decodingTime": 1850,         "translation": "Ventana que muestra las métricas para toda la XXXX<dnt 3 /> disponible para<dnt 2 /> los administradores de XXXX con<dnt 1 /> el permiso XXXX.",         "sourceWordCount": 19,         "targetWordCount": 20,         "translationTokens": [             "Ventana",             "que",             "muestra",             "las",             "métricas",             "para",             "toda",             "la",             "XXXX",             "<dnt 3 />",             "disponible",             "para",             "<dnt 2 />",             "los",             "administradores",             "de",             "XXXX",             "con",             "<dnt 1 />",             "el",             "permiso",             "XXXX",             "."         ],         "sentenceTokens": [             "A",             "window",             "that",             "displays",             "metrics",             "for",             "the",             "entire",             "XXXX",             "<dnt 3 />",             "is",             "available",             "to",             "XXXX",             "<dnt 2 />",             "administrators",             "with",             "the",             "XXXX",             "<dnt 1 />",             "permission",             "."         ],         "alignment": [             [                 1,                 0             ],             [                 2,                 1             ],             [                 3,                 2             ],             [                 4,                 3             ],             [                 4,                 4             ],             [                 5,                 5             ],             [                 7,                 6             ],             [                 8,                 7             ],             [                 8,                 8             ],             [                 11,                 10             ],             [                 12,                 11             ],             [                 13,                 15             ],             [                 13,                 16             ],             [                 15,                 13             ],             [                 15,                 14             ],             [                 16,                 17             ],             [                 18,                 21             ],             [                 20,                 19             ],             [                 20,                 20             ],             [                 21,                 22             ]         ]     } }
Also FYI, I had to build a new Docker image to run the new 2.4 version of MMT and the compile script for vendor directory failed when downloading Kafka with this error:
Installing Kafka <IP_ADDRESS>...

Traceback (most recent call last):
  File "./compile", line 350, in <module>
    run_all()
  File "./compile", line 315, in run_all
    install_kafka()
  File "./compile", line 300, in install_kafka
    install_apache_lib(kafka_apache_path, kafka_home)
  File "./compile", line 177, in install_apache_lib
    folder = untar(tar_destination, work_directory)
  File "./compile", line 61, in untar
    tar = tarfile.open(filename, 'r:gz')
  File "/usr/lib/python2.7/tarfile.py", line 1691, in open
    return func(name, filemode, fileobj, **kwargs)
  File "/usr/lib/python2.7/tarfile.py", line 1749, in gzopen
    raise ReadError("not a gzip file")
tarfile.ReadError: not a gzip file

I fixed it by downloading Kafka myself, and running compile script again, and it worked. So I think your compile script is broken, perhaps the URL is dead? It worked once Kafka was archive was there, so I assume it's an issue with downloading Kafka.
Thanks,
R

Hi @robertBrnnn
so looking at the two translations, they look exactly the same (including tokenization and alignments). Let me try to explain what I'm trying to achieve. You said:

Do you have any ideas why the issue is worse with MMT 2.4?

In order to answer your question I need to start from an example where with the same model the two systems behave differently (if you change the model doing a re-train, than it is possible that the new content made the translation worse and not MMT version).
So to answer you question I need you to:

Setup the old MMT 2.3.
Setup the latest version in master branch freshly pulled.
Load the same exact engine with the two versions.
Find an example of translation where the two versions output is different.

From there, we can start debugging and see why the two versions have different behaviour.
Thanks,
Davide

"do-not-translate" feature has been finally introduced, please have a look at issue #385 for more details.
Thanks for the support @robertBrnnn ! I'm closing this issue in order to not create duplicates of the same problem. For further discussion please refer to issue #385


[youtube] More than one link return for a youtube url

Hi,
I'm trying to extract some info from a youtube link using youtube-dl. When I run the following command,
youtube-dl -o "%(title)s.%(ext)s" --get-url --get-filename "https://www.youtube.com/watch?v=a0oQy7jgH_w"

it returns two different links and video title with a warning message,
WARNING: [youtube] a0oQy7jgH_w: Skipping DASH manifest: ExtractorError(u'Cannot decrypt signature without player_url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)

https://r6---sn-5hn7ym7s.googlevideo.com/videoplayback?source=youtube&clen=598736472&mv=s&fexp=900720%2C907263%2C924637%2C934954%2C936118%2C938028%2C9406987%2C9408142%2C948124%2C952612%2C952637%2C957201&mt=1430386530&ms=pm&signature=97DE9DCE12DEB48C895A54D08451DF4F1BA75DE6.B5F4C9A1BF217E40D0EAF669288D677AADC4B711&gir=yes&mm=35&sparams=clen%2Cdur%2Cgir%2Cid%2Cip%2Cipbits%2Citag%2Ckeepalive%2Clmt%2Cmime%2Cmm%2Cms%2Cmv%2Cnh%2Cpl%2Crequiressl%2Csource%2Cupn%2Cexpire&id=o-ABxOk8SmGW5B-mWnEIU2Oak1Cd9FVF4bzCRcQrR7ndqa&sver=3&itag=299&requiressl=yes&keepalive=yes&ip=<IP_ADDRESS>&lmt=1429945206484166&pl=20&ipbits=0&key=yt5&nh=EAE&upn=jSzrylh5REY&expire=1430412467&mime=video%2Fmp4&dur=1045.533&ratebypass=yes

https://r6---sn-5hn7ym7s.googlevideo.com/videoplayback?source=youtube&clen=16786506&mv=s&fexp=900720%2C907263%2C924637%2C934954%2C936118%2C938028%2C9406987%2C9408142%2C948124%2C952612%2C952637%2C957201&mt=1430386530&ms=pm&signature=A60B8EB14A9516E382061B6962B319578C4CF321.251291D90FF8F26FA233EC8392AD736026F5D940&gir=yes&mm=35&sparams=clen%2Cdur%2Cgir%2Cid%2Cip%2Cipbits%2Citag%2Ckeepalive%2Clmt%2Cmime%2Cmm%2Cms%2Cmv%2Cnh%2Cpl%2Crequiressl%2Csource%2Cupn%2Cexpire&id=o-ABxOk8SmGW5B-mWnEIU2Oak1Cd9FVF4bzCRcQrR7ndqa&sver=3&itag=140&requiressl=yes&keepalive=yes&ip=<IP_ADDRESS>&lmt=1429937765857607&pl=20&ipbits=0&key=yt5&nh=EAE&upn=jSzrylh5REY&expire=1430412467&mime=audio%2Fmp4&dur=1045.594&ratebypass=yes

Ubuntu Phone - Is It Worth It (Aquaris E4.5).mp4

It should return only one link with a video title. I guess youtube have changed something and it breaks youtube-dl.
Here is my configuration:
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v']
[debug] Encodings: locale ANSI_X3.4-1968, fs ANSI_X3.4-1968, out ANSI_X3.4-1968, pref ANSI_X3.4-1968
[debug] youtube-dl version 2015.04.26
[debug] Git HEAD: 6090285
[debug] Python version 2.7.3 - Linux-3.2.0-69-virtual-x86_64-with-Ubuntu-12.04-precise
[debug] exe versions: avconv 0.8.16-4, avprobe 0.8.16-4, ffmpeg 0.8.16-4, ffprobe 0.8.16-4
[debug] Proxy map: {}

youtube-dl now defaults to downloading DASH video and audio and then merges them (they give a better quality result). If you only want to get normal formats, you can use -f best.
Note that for doing anything serious with the output of youtube-dl you should use the --dump-json option.


[Improve][TubeMQ] set error code 0 when it works

Description
TubeMQ Master will respond error code 200 in the retValue when it works, however the wrapped error code is 0.
The two is not consistent and I think making error code 0 both will be a good idea.
InLong Component
InLong TubeMQ
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

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

This issue is duplicated with https://github.com/apache/inlong/issues/8716, I'll close this


Add the Protocol attribute to the Redis configuration

The go-redis v9 has added support for RESP3 and communicates with the redis server using RESP3 by default. However, many cloud server manufacturers’ redis cloud services do not yet support RESP3. Therefore, when using the v9 client to connect to a redis server that does not support RESP3, some redis commands may fail to process the return value from the server, such as ‘zset range’. In order to be compatible with the RESP2 protocol, it is necessary to add configuration for the RESP version.

这个PR大概什么时候能进版本？我们项目也遇到了这个问题，腾讯云的全线redis产品都只支持RESP2

@tianxing113 Awesome! Thanks for your contribution!

这个PR大概什么时候能进版本？我们项目也遇到了这个问题，腾讯云的全线redis产品都只支持RESP2

It now merged into branch master, and will be release in next version. You can use go get -u github.com/gogf/gf/v2@master if you're urgent.


Update to Async HTTP Client

This looks great! Thanks for taking this further  🙏

@Yasumoto thanks for your feedback. I just improved the search method a bit, added some routes/methods, some commands to the cli tool and updated the readme.
Do you have anything else in mind, that I should add to the project right now? Otherwise I'd like to merge this branch to master and create a new version/release.


[Removal]: Slow Golden Panda

Hotspot Name
Slow Golden Panda
Hotspot b58 Address
11xLUKmBLLE6nuVZ3dEerazCKmSMqN1C8vAwt4raRmbZkjKqtdg
Discord Handle
No response
Hotspot Manufacturer
PantherX
Removal Reason
I will deploy correctly and sell the excess hotspot to other users.
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
Hello, admin. Let me first describe how I joined the HNT project. I read about the HNT project on the Internet, but it is not clear that there are restrictions on deployment. And contacted a hot seller, the seller asked to buy 10 devices at one time, and buy his router, he will provide remote configuration networking, and manage it, I just need to power on the device. According to the seller's claim that this can achieve faster revenue, I don't know that doing so will violate HNT's work requirements, and I don't know how he operates in real time. After deployment, HNT lost revenue and the online seller was unable to contact When I arrived, and the cost I invested was almost completely lost, I later learned that the hotspot device was added to the rejection list. I would like to ask if my hotspot device can still be removed from the rejection list? In the future, I will sell redundant hotspots, and self-deployed hotspots will strictly abide by the HNT deployment rules.

If you can provide photos of your installation also with GPS coordinate.  Admin may be considered to remove the hotspot from the denylist.

OK, I now have 10 hotspot devices, but I don't have 10 locations to install. If I want to sell the redundant hotspot, I can only go online with one hotspot and wait for it to be unblocked, then sell the hotspot (I can provide a screenshot of the sale on the website), and then go online and unblock the next one? Or is there a more direct way?

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Update Readme

Your example uses the docker image veqryn/kube-cloudwatch-autoscaler:1.1
which doesn't work in k8s 1.10.x, or match the repo.
You should update the readme to use the image veqryn/kube-cloudwatch-autoscaler:1.2.

thx


fixcreate view init bug

Proposed changes
the body of create view stmt is parsed twice.
in the second parse, we get sql string from CreateViewStmt.viewDefStmt.toSql() function, which missed selectlist.
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run compile

run p0

run buildall


django 2.0 liveserver/runserver uses HTTP 1.1 and breaks playback

I have a system where I use vcrpy for behave tests through livetestserver, it was working fine until django 2.0 (it was the last issue)
So I decided to debug it, and it seems that the issue was HTTP 1.1 keep-alive pipelining is getting confused, as it would expect the same connection, but sometimes gets a non-open connection.
By applying this monkeypatch to force LiveTestServer/runserver to drop to HTTP 1.0 compliance it all started working again:
from django.core.servers import basehttp
basehttp.ServerHandler.http_version = '1.0'

Could you please update the docs to mention the caveat of HTTP 1.1 pipelining and selective playback, and either document the hack or make with my_vcr.use_cassette() apply that monkeypatch conditionally?

Thanks for doing the research.  Would merge a PR that adds that caveat to docs.

A lot of changes have happened to VCRpy since this ticket was opened. As this ticket has become stale and we have a lot of old tickets that need to be groomed, I'm closing this for now to make it easier to see what is still relevant.
However if it is still needed, please feel free to re-open or create a new ticket.
Thanks! 🙏


[CP] Update dartdoc to provide chips for class modifiers on class or mixin pages.

Commit(s) to merge
f193d91ba50fbb4972bfe68d57e6db8ea9e6c466
Target
beta
Prepared changelist for beta/stable
https://dart-review.googlesource.com/c/sdk/+/299500
Issue Description
The motivating issue is that it is hard to see class modifiers on dartdoc generated pages, here: https://github.com/dart-lang/dartdoc/issues/3392.
What is the fix
https://github.com/dart-lang/dartdoc/pulls/3401 is the fix, to add chips to the top of class and mixin pages to indicate class modifiers, show a small tooltip, and link to descriptive documentation.
Why cherry-pick
Users may miss that class modifiers apply to a part of a Dart API without this change, as the old way of displaying them in the breadcrumb is easy to miss.  As class modifiers are a core feature of Dart 3.0 it makes sense to have them show up prominently in documentation.  @mit-mit requested that we make an effort to get this in for the initial release.
Risk
low
Issue link(s)
https://github.com/dart-lang/dartdoc/issues/3392
Extra Info
This cherry pick pulls in all deltas for dartdoc, including one other bugfix and some housekeeping.
Release notes:  https://github.com/dart-lang/dartdoc/releases/tag/v6.2.2
Github diff between dartdoc revisions:  https://github.com/dart-lang/dartdoc/compare/1a7952b..d01ddc5

Requesting another look from @mit-mit as this is a usability issue.
Requesting a look from @vsmenon and @leafpetersen to verify if this meets the bar for a cherry-pick late into the cycle.

@mit-mit @itsjustkevin Ping to elevate this cherry pick.   The equivalent for Flutter (https://github.com/flutter/flutter/pull/125661)  has already merged; it would be unfortunate if this one didn't make it.

This hasn't been lost in the ocean @jcollins-g, we will be reviewing the cherry-picks today, will report back the status on the issue later in the day 🙂

Hold for next stable hotfix.

@mit-mit do we have the stamp of approval on this cherry-pick now that Dart 3 is released?

@jcollins-g could you rebase this against the stable branch?  Seeing the results would make it easier to verify the safety of this cherry-pick.

@itsjustkevin https://dart-review.googlesource.com/c/sdk/+/302841 is the rebased change.  The delta is exactly the same, though of course the context of other DEPS changes may be different.

@jcollins-g I've edited the issue to point to the new cherry-pick, however there are merge conflicts.

@itsjustkevin I apologize, my git-fu is not very strong apparently.   I have attempted once again to rebase and have restarted the bots.  Will keep an eye on it.


Implement service_account.Credentials.to_jwt_credentials()

(Context: #29)

Sorry for the late follow. Allowing easy "promotion" like this is good and what we did in a couple of other languages. One thing about JWT credentials is that they have a much larger request header footprint, and so are better suited to HTTP 2.0 which can cache large headers.

Good to hear we made the right choice. Thanks!
On Mon, Oct 31, 2016, 4:39 PM anthmgoogle<EMAIL_ADDRESS>wrote:

Sorry for the late follow. Allowing easy "promotion" like this is good and
what we did in a couple of other languages. One thing about JWT credentials
is that they have a much larger request header footprint, and so are better
suited to HTTP 2.0 which can cache large headers.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-auth-library-python/issues/35#issuecomment-257451443,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUc2mNittlj-aF_01hhiAiW4glaF0Fks5q5nwsgaJpZM4KdooJ
.


Add line for switching to the project folder in Readme.md

closes #54

I will replace this pull request with #56 because the commit contains a wrong e-mail address


[rtl,flash_ctrl] Fix ecc errors

Description
We are still getting ecc errors in flash_ctrl. Running the flash_ctrl_rw with seed<PHONE_NUMBER>3256424400418660460856055802845297581904883038901129981482688983152 shows a read fatal fault, but there was a prior write of the same location and apparently the data is identical to what was written. This may indicate an incorrect write. Also, the same error happens with the flash_ctrl prior to sharing the scrambler.

One case I reviewed seems to be due to non-overlapping half-word writes with ECC enabled. This is almost certain to result in ECC errors since ecc bits cannot change from 0 to 1. We need to prevent these cases from being generated in tests.

Thanks for creating the issue @matutem . What just came to my mind is this issue here:  #13429
Can you check if it's related? Maybe we "just" need to constrain DV properly.

We have determined all faults we get are due to incorrect tests, which either do half-word updates to both halves of a word with scrambling or ECC enabled, or tests that simple overwrite locations also with ecc or scrambling enabled: both cases result in ECC errors.
Closing this issue :)


./miri: nicer error when building miri-script fails

@RossSmyth @ChrisDenton if you could do the same for miri.bat that would be great :)

@bors r+

:pushpin: Commit 50b9c514fba2ce2ebc753d8686505c1a8ce1d6f8 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 50b9c514fba2ce2ebc753d8686505c1a8ce1d6f8 with merge 96c760a81502d76dc4fc2d68ff438bc253e45b46...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 96c760a81502d76dc4fc2d68ff438bc253e45b46 to master...


Fix modifying of frozen strings via config files and tests

Fixes https://github.com/fastlane/fastlane/issues/8548

          1 Error

:no_entry_sign:
Tests have failed, see below for more information.

Tests:

Name
File

FastlaneCore FastlaneCore::ConfigurationFile Properly loads and handles various configuration files properly loads boolean values
configuration_file_spec.rb

  Generated by :no_entry_sign: Danger

Oh wow, interesting, so turns out in
Ruby 2.3.0
irb(main):003:0> false.dup
NoMethodError: undefined method `dupe' for false:FalseClass
```
**while in Ruby 2.4.0**
```
irb(main):003:0> false.dup
=> false
```

Alright, updated to work with all versions of Ruby 👍

By the way in current CI environments, the tests run only on Ruby 2.3.1.
Why don't we run specs on the multiple versions?
(We should run specs on Ruby2.3.x, 2.4.x and 2.0.0(Apple system default))

we had that topic a few times i think right now circle does not support matrix runs, @DanToml  still right?

@hjanuschka still correct


Not possible to set working directory for a template task

I have a repository with different folders for different Ansible playbooks:
folder 1:
  ansible.cfg
  collections/requirements.cfg
  roles/requirements.cfg
folder 2:
  ansible.cfg
  collections/requirements.cfg
  roles/requirements.cfg

Every folder has different files as described as above.
When I run a tasks I get the following output:
No collections/requirements.yml file found. Skip galaxy install process.
No roles/requirements.yml file found. Skip galaxy install process.

What I need is a way to specify the working directory for a templates to set the root folder for Ansible and all its configurations files.
E.g. for folder 1 I need to set working directory to the folder 1 so ansible can find the ansible cfg file.
How can I do this? Is it possible as of now and how should i do it?

Today I need to prefix all path to playbook with folder 1, folder 2 etc.
It would be nice to set working directory and the use relative paths from there.


Develop

It might be cool to declare the scheme, once.
import UIKit
import Crossroad

let scheme = "pokedex"

@UIApplicationMain
class AppDelegate: UIResponder, UIApplicationDelegate {

  private typealias Predicate = (Context<OpenURLOption>) -> Bool
  private typealias Route = (String, Predicate)

  private struct Creatures {
    enum Query: String {
      case type = "type"
    }
    let database = "pokemons"
    var route1: Route {
      return (database, predicate1())
    }
    private func predicate1() -> Predicate {
      return {
        if let type: Type = $0.parameter(for: Query.type.rawValue) {
          // post type using NotificationCenter
          return true
        }
        return false
      }
    }
  }

  var window: UIWindow?

  private let router = DefaultRouter(scheme: scheme)

  private let creatures = Creatures()

  func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplicationLaunchOptionsKey: Any]?) -> Bool {
    router.register([creatures.route1])
    return true
  }

  func application(_ app: UIApplication, open url: URL, options: [UIApplicationOpenURLOptionsKey: Any]) -> Bool {
    return router.openIfPossible(url, options: options)
  }
}

Cool! I'm waiting WWDC keynote now. So I'll check this PR tomorrow 😅

Me too! Enjoy.

Have made the changes @giginet suggested 👍

You can ignore type_body_length violation.
But, please fix trailing_whitespace violations.
You will fix them by only executing swiftlint autocorrect


Warn about scoped names

Is your feature request related to a problem? Please describe.
I'm building from a scope mono repo and when the package name starts with @ tauri build fails with little context and the following message (Error: No such file or directory (os error 2))
Describe the solution you'd like
Warn during init that scoped names are invalid or remove the scoping and snake case the result

probably better for us to be smarter about that than to offer workarounds.

I'm not sure the scoped name is the problem here. Can you create a repo with a project to reproduce it?

I think #2114 fixes it and the problem is when the product name has slashes. Let me know if you still face any issues.


There is a bug in the n-input

Environment Info

Naive UI version: 2.11.7
Vue version: 3.0.11
Browser Info: Chrome(89.0.4389.128)
System Info: Mac OS 11.2.3

Reproduction link
<n-input
      v-model:value="form.functionalExpectations"
      type="textarea"
      placeholder="请输入"
      :autosize="{ minRows: 2, maxRows: 5 }"
    ></n-input>

Steps to reproduce
When I set the minRows and maxRows properties at the same time, entering a lot of text will stretch the textarea horizontally
What is expected?
What is actually happening?

It works in the demo page. A reproduction link is needed.

I'm sorry I didn't explain the root cause before. After checking again, I found that the above problems only occurred after using < n-form-item >
Reproduction link:
https://github.com/amadeus711/issue-demo
The code in src/components/Demo.vue
Reproduction steps:

clone
install
npm run dev

I'll fix it


Client crash: Tinkers' workbench item rendering

Version: Minecraft 1.10.2 - Abridged B1
Issue Description: Client crash. it's from putting Mekanism Energy Tablet on the tinkers' version of the crafting table. A client rendering issue trying to render the item on top of the table.
Steps to Reproduce:
Place Energy Tablet on tinkers' crafting table
Crash report: http://pastebin.com/5kc2PUjE
AND
CrashReport: crash-2016-10-08_19.39.03-client.txt

Removed one of our 3 IE windmills and the problem dissapeared. Whathefuck.

Updated OP

duplicate of https://github.com/elytra/Unabridged/issues/501


TextureLoader return different data between node-three-gltf and threejs on transparent pixel

I load a png image with transparent pixel
threejs is 0,0,0,0

// three_r148/src/extras/ImageUtils.js after line 49
const context = canvas.getContext('2d')
const imageData = context.getImageData(0, 0, canvas.width, canvas.height);
console.log(imageData)
for (let i = 0; i < imageData.width * imageData.height; i++) {
	const r = imageData.data[i * 4]
	const g = imageData.data[i * 4 + 1]
	const b = imageData.data[i * 4 + 2]
	const a = imageData.data[i * 4 + 3]
	console.log(i, '--', r, g, b, a)
}

node-three-gltf is 255,255,255,0

const textureLoader = new TextureLoader()
const metalnessMap = await textureLoader.loadAsync('./assets/textures/metalness.png')
// metalnessMap.encoding = sRGBEncoding

if (metalnessMap?.source?.data) {
    const imageData = metalnessMap.source.data
    for (let i = 0; i < imageData.width * imageData.height; i++) {
        const r = imageData.data[i * 4]
        const g = imageData.data[i * 4 + 1]
        const b = imageData.data[i * 4 + 2]
        const a = imageData.data[i * 4 + 3]
        console.log(i, '--', r, g, b, a)
    }
}

image.zip

I try node-canvas
the result is same as html-canvas

The ImageLoader here uses sharp to load any images/textures. Did you also test your file directly with sharp? It should be something like this:
const data = await sharp('./assets/textures/metalness.png').raw().toBuffer();

for (let i = 0; i < width * height; i++) {
    const r = data[i * 4]
    const g = data[i * 4 + 1]
    const b = data[i * 4 + 2]
    const a = data[i * 4 + 3]
    console.log(i, '--', r, g, b, a)
}

The reason for replacing jimp with sharp is that sharp is much smaller in size and faster than jimp. There were also some vulnerabilities with some dependencies of jimp (which may be resolved by now https://github.com/oliver-moran/jimp/issues/1088). The overall maintainance of the sharp package also seems to be better. So, there are several reasons I switched to sharp, and so far, my experiences are pretty well (also in my other projects).

Thanks for your reply.
Yes, I tested sharp directly loading images and the results are also "255 255 255 0"

Also, I use jimp to achieve the result I want, but I don't know how to do it with sharp.
const image0 = await Jimp.read('./assets/textures/metalness.png')

let imageData = image0.bitmap
const background0 = new Jimp(imageData.width, imageData.height, 0x00000000)
background0.blit(image0, 0,0)
imageData = background0.bitmap

for (let i = 0; i < imageData.width * imageData.height; i++) {
    const r = imageData.data[i * 4]
    const g = imageData.data[i * 4 + 1]
    const b = imageData.data[i * 4 + 2]
    const a = imageData.data[i * 4 + 3]
    console.log(i, '--', r, g, b, a)
}

This might solve my problem.
const data2 = await sharp({
        create: {
            width: 4,
            height: 4,
            channels: 4,
            background: '#00000000' // {r:0, g: 0, b: 0, alpha: 0}
        }
    })
    .composite([
        {input: './assets/textures/metalness.png'}
    ])
    .raw()
    .toBuffer()

for (let i = 0; i < 16; i++) {
    const r = data2[i * 4]
    const g = data2[i * 4 + 1]
    const b = data2[i * 4 + 2]
    const a = data2[i * 4 + 3]
    console.log(i, '--', r, g, b, a)
}

Maybe you can open an isssue at the sharp repository. That the channel values of an transparent pixel load differently may not be an intended behavior.

I found a way to solve this problem
let s6 = await sharp('./assets/textures/metalness.png')
    .ensureAlpha()
s6 = await s6.metadata()
    .then(meta => {
        return s6
            .raw()
            .toBuffer()
            .then(buf => {
                return sharp(buf, {
                    raw: {
                        width: meta.width,
                        height: meta.height,
                        channels: meta.channels,
                        premultiplied: true, // !!!
                    }
                })
            })
    })

const d6 = await s6.raw().toBuffer()
console.log('d6')
dump(d6)

I didn't find a direct way, I have submitted the issue in the sharp repository


interactive plot does not render if you cannot access the Google Font API

I've just discovered this as my work firewall don't allow get requests to google APIs, and so I can't render an interactive plot.
I can manually install the Roboto font family though, but the code doesn't look for it - it goes straight for the API and throws an error if you can't connect.

I have used it behind a firewall and it does work, but I think you need to specify a generic font family instead (e.g. "serif"); it has been a while though, and changes have happened to the codebase since then, so I'll have to double check that there are reasonable workarounds.

Thank you! Only impacts interactive plot. I get a lot of warnings on static plot but it does work for that.


Provides specific CAMARA OAS for notification event

Prepared with @patrice-conil
As we are offering “notification event” for a lot of our API we need to clarify the technical assets. As of today, we are defining the notification event in the same OAS than the ‘nominal’ API resource itself. It could trigger confusion as it mixes endpoint provided by server from the notification endpoint that must be on client listener side.
Our proposal is

to provide a separate OAS only describing the notification endpoint. We’ll provided here generic CAMARA event notification structure. This swagger will be available to app developer to generate their listener endpoint.
To describe event notification API per API in the callback structure (as done in QoS).

@bigludo7 : For a generic Camara wide event structure, cloudevents spec (https://github.com/cloudevents/spec/tree/main) could be worth a look

Hi @shilpa-padgaonkar,
Thanx for the link, we will use it if workgroup members adhere to the idea.
Another point to discuss this friday.
BR,
Patrice

For those not familiar with CloudEvents might want to look here : https://github.com/cloudevents/spec/blob/main/cloudevents/primer.md

@shilpa-padgaonkar @patrice-conil To make progress on this topic I've contributed a proposal for a generic notification OAS (#41 ). I keep aligned with the API design guideline document.

Please considere this message as a formal inclusion of the GSMA OGW Product WS request to include Subscription/notification mechanism as part of OGW services. Some examples of those are attached, and will be also requested when and if affected.
The request includes both the formalization of the creation in Commonalities of a subscription/notification mechanism generic for OGW services, as agreed by GSMA OGW Product WS, and also the inclusion of each spscific funcionality in each already existing API. Others may be applied, and others may apply in future drops/releases.
Additional context
More details included in the formal request file, attached:
Camara Submission - Subscription_Notifications v2.docx

@jgarciahospital few suggestions:

application must be able to, at least, terminate its subscription (better adjust it), not just create
MNO should also notify the application about events related to the subscription. For example, if the person closed their contract or changed the number, currently active subscription makes no sense anymore.

Thanks for the feedbacks.
@jgarciahospital
The notification model has been introduced for CAMARA API for some months now and it is already available in few Camara APIs like Qod, Carrier Billing, Device Status. The word document you provided is very good material to update what we have currently implemented or in process of implementing. I propose to take a look & to redistribute this per API CAMARA.
@gregory1g
Capability to terminate subscription & event related to subscription management are already available - This is described in the API Design guideline document and implemented in Device status API.
BTW, I recommend that we stick to the original topic of this conversation: the format of the notification send back to the listener once event occurred to avoid mixing different topic. Of course new issue are awelcome.

Hi all,
@bigludo7
if i understand well, proposal is to define a common /traversal OAS3 API to cover this functionality https://github.com/camaraproject/Commonalities/blob/main/documentation/API-design-guidelines.md#122-event-notification

@PedroDiez  Yes !

Thanks for the feedbacks. @jgarciahospital The notification model has been introduced for CAMARA API for some months now and it is already available in few Camara APIs like Qod, Carrier Billing, Device Status. The word document you provided is very good material to update what we have currently implemented or in process of implementing. I propose to take a look & to redistribute this per API CAMARA.
BTW, I recommend that we stick to the original topic of this conversation: the format of the notification send back to the listener once event occurred to avoid mixing different topic. Of course new issue are awelcome.

Thanks Ludovic, we've already openned or commented to confirm the request per API.

I had a closer look into CloudEvent primer and was impressed by the design thoughts which went into it and the wide adoption – it kept stable since 2019, and there is a complete eco-system of SDKs etc around. It' a CNCF incubated project, so already a kind of de-facto standard.
So, yes, we should definitely evaluate and follow this spec instead of spending further time to improve and formalize ( our own and being just another [incompatible webhook] (https://github.com/cloudevents/spec/blob/v1.0.2/cloudevents/primer.md#normalizing-webhooks) format into the world.
The differences of CloudEvent to our current guideline are not significant, but subtle, and there are also some points which should be reconsidered. E.g. the three layers of eventNotification, event and eventDetails. Also the subscriptionId on level of transport outside the event does not make sense for me (and does not fit what we need for implicit subscriptions in QoD).
And as they are so similar, there is no reason why we should have a different definition for events and its metadata in CAMARA – it wouldn’t be developer friendly to don’t allow developers to use the existing implementations, SDKs, bindings etc of CloudEvents. Also we would jeopardize any attempt to get “Cloud Native” in the sense of LF/CNCF.
Therefore I propose to do first a short evaluation before we rush forward with our "proprietary one". And I hope that we can conclude on this existing standard. It is now the time to do it, before the APIs are leaving the pre-production stage.

Hi @hdamker,
I thought we didn't have time to do this now. But if so then I totally agree with you.
Do you want me to update our proposal (@bigludo7 and me) to reflect these changes in #41?
Could you handle updating the guidelines in parallel to open discussion with all contributors?

@patrice-conil : I see that the PR #41 has in its heading WIP but is not created as a draft PR. If the PR is not ready to be reviewed, would you mind changing it to draft PR instead?

Sorry @shilpa-padgaonkar,
it's my mistake.
You can review it.

As there seems to be now an agreement, for evaluation, I can see this issue include several PRs:
Some of the obvious ones I have listed below:

Original PR https://github.com/camaraproject/Commonalities/pull/41

New PR showing how a cloud-events based solution could look like https://github.com/camaraproject/Commonalities/pull/43

A PR to add description in guidelines document about how we will incorporate the CloudEvents spec for our Camara APIs. Which optional parts in the spec will be relevant for us.

This evaluation will help us decide if our approach is sustainable long term ( where we can imagine messages being delivered not just using http protocol but may be even kafka or other standards).

There is dedicated guideline document for delivering notifications with Cloudevents: Web Hooks for Event Delivery.
Regardless results of evaluation section 4. Abuse Protection provides important security recommendations that should be considered in CAMARA.

Hi all,
Think missing some part of the context.
The point here is for everybody to comment about the preference of which model to follow? Whether based in PR#41 or PR#43?
cc @patrice-conil @shilpa-padgaonkar

Hi @PedroDiez,
Yes, we can discuss/comment both proposals here.

Feedback from TEF:

Currently our model is closer to PR#41 proposed model.
Anyway, move towards CloudEvents model is somtehing it can be achieved.

Then, if there is a preference to go for CloudEvents Model, it is also OK from our side.


Cards are not rendered until I refresh hangouts page

Hi ,
I have created a bot  using Bot url (API Gateway API)which sends message/cards to bot . It used to work properly. Suddenly Response cards are not rendering and once I refresh the page it shows up.
Did anyone face this issue? Or any thoughts on this.
Thank you in advance

we tested this with simple card withouts buttons. It renders properly. But Not with complex cards with buttons

THanks for reaching out, but this issue tracker is only for issues with the samples themselves. See https://developers.google.com/hangouts/chat/support for other support options. Specifically, if you think there's a bug with Hangouts Chat itself you can file an issue on the tracker listed on that page. If you're a G Suite customer, you can also contact support for assistance.


Keep the current save's voting buttons active when a preset is executed [SUGGESTION]

Is your feature request related to a problem? Please describe.
When downloading an executing a preset embedded in a save, the action of clearing the screen results in the vote buttons (and other buttons like tags and name) graying out. You then have to navigate back to the save to vote on it.
Describe the solution you'd like
Keep the current save "loaded" when a preset is executed so that you can vote on it after generating the preset.
Describe alternatives you've considered
Some way of quickly navigating back to the previously loaded save would also make voting easier.
Additional context
N/A

This can be reasonably implemented in the current version using sim.clearRect instead of sim.clearSim. Removed future label.


New Astrazeneca column for 2nd vaccination

Hi @ThisIsBenny,
like expected there is now a new column for Astrazeneca for the 2nd vaccination.
Thx in advance!

Here is the error:
states[state]['2nd_vaccination']['difference_to_the_previous_day'] = row[12].value
Is wrong. Correct would be
states[state]['2nd_vaccination']['vaccinated_by_accine']['astrazeneca'] = row[12].value
states[state]['2nd_vaccination']['difference_to_the_previous_day'] = row[13].value
Please see #28

Done 😊

👍🏻

Thanks


Latest ComfyUI updates break plugin on Windows.

I'm not sure what is causing it but the ksampler is throwing a 'tuple' object has no attribute 'model' error. Maybe caused by changes to the sampler here?:
https://github.com/comfyanonymous/ComfyUI/commit/4eab00e14bc0b52a9c688486d7ee8b392e01020

fixed 84ab26f


Hide problems for files that don't exist in current branch

I often work with multiple branches in parallel, switching between them from time to time. Say in branch A I create a set of new files that make references to each other. When I switch to another branch (say branch B) that does not include those files, the "Problems" tab will start showing errors in those files that exist in branch A but not branch B, because they make references (e.g., import) to other files in branch A that also don't exist in branch B. This is for Typescript development, if it matters, but I've seen a similar behaviour in other languages too.
Is there a way to avoid showing problems for these files, since they are not actually a problem in the current branch? Closing the tabs with the new files removes the listed problems in the problems tab, but I don't want to have to close all tabs every time I switch branches.
I tried searching for related issues and didn't find anything relevant, but I may not be searching for the correct terms, so apologies if this issue already exists.

Do you know about these SCM working sets options?

I'm sorry, but I don't understand how using these options would resolve my problem, could you please elaborate?

One of the options for the first setting would allow you to begin with no files open when you switch from A to B for the first time. Then after you work on B, opening the files you need, a subsequent switch to A would reinstate the set of open files you had when previously on A.

Thank you for the workaround, but this doesn't actually resolve my problem. Note that this implies having a separate working set for every branch, which is not what I want. Also, from what I understand there are some requirements for this to work (e.g., only changing the branch inside VSCode), which I'm not able to stick to due to other requirements. Besides, while closing the file often dismisses the problem list, this does not always happen.

Problems are generated by the extensions and they manage the life cycle of these problems. VS Code just acts as UI for these problems. Therefore, I would request to create the issue against the extension to provide options to not to create problems for non existing files or something like that. Thanks.


Fix Guild Owner and Admin GuildPermissions.All

added bits for new permissions

Perhaps include the underscores to keep formatting consistent and improve readability for future additions/fixes?


Generic structure for "a directory of commands with help and plug-ins"

We have:

commands
patterns
elements
outputs

All of which are similar, all of which need help, all of which would like to accept plug-ins.


Add Streams types

The Streams spec does not provide any Web IDL so it has to be added through the manual way.
cc: @Surma

Alternatively we can craft our own IDL, which should be more readable.

There already is: https://github.com/whatwg/streams/issues/45
The author isn't happy with Web IDL...

Alternatively we can craft our own IDL

Well.. according to https://github.com/Microsoft/TypeScript/issues/25277 we need to make it generic. do not think WebIDL can do that today, so we will have to have a base webIdl with a punch of any's and then a modification layer on top. might be easier to just have a .d.ts file that we include verbatim.

Where do the current definitions for ReadableStream and WritableStream come from?

https://github.com/Microsoft/TSJS-lib-generator/blob/master/inputfiles/browser.webidl.preprocessed.json

I want to give this a try! 😄 I've already started with a few types to see what could work, and I need some suggestions on how to proceed.

Ideally, we'd take the existing .d.ts from DefinitelyTyped and merge it into dom.generated.d.ts. However, it seems that the lib generator doesn't work like that: it only consumes WebIDL files or modifications from {added,overriding,removed}Types.json files. As @mhegazy already mentioned, it seems like we should either:

Rewrite the .d.ts as WebIDL, and add some tweaks in the JSON files for types that can't be expressed in WebIDL. This will not be an official WebIDL file, which might be a bit of a risk, although we'll have to tweak and maintain the types anyway.
Rewrite the .d.ts entirely as JSON. This will probably be quite verbose, but you wouldn't need to go back-and-forth between WebIDL and JSON to figure out where a type is defined.

Any suggestions on what approach is more desirable would be much appreciated.

The read method of ReadableStreamDefaultReader currently has the following type:
read(): Promise<IteratorResult<R>>;

However, IteratorResult is defined in es2015.iterable, which is not available when targeting ES5 or lower.
I think we can get away with defining a separate interface (e.g. ReadResult<R>) that is just a copy of IteratorResult<R>. Since TypeScript uses structured typing, the type checker should treat both as equivalent. Or is there a cleaner way to handle this?

I noticed that many WritableStream types are already defined (because they're implemented in Edge), but some of them are quite awkward. For example, the callbacks in UnderlyingSink are defined as callable interfaces. I think this is because they're generated from a WebIDL callback definition.
Is it okay if we change these to be function types? For example:
// old
interface WritableStreamDefaultControllerCallback {
    (controller: WritableStreamDefaultController): void;
}
// new
type WritableStreamDefaultControllerCallback = (controller: WritableStreamDefaultController) => void;

Or should we keep using callable interfaces, including for the new callbacks (e.g. UnderlyingSource.pull)?


First version of json modifier

This is a very light version of what could be a json modifier, it's missing much helpers to be complete but seems functional for now.
I'm opening pull request to get advices back, if anyone has any please tell them here :)
Note: I couldn't find any way to make array and object modifiers tailrec. That sounds bad for now.

I think i'll need to modify the default Json serializer so JsUndefined are not written as null. This will provide a simple way to "remove" a field from an object.
(see https://github.com/playframework/Play20/blob/master/framework/src/play/src/main/scala/play/api/libs/json/JsValue.scala#L188)


[Question] FlinkCluster stopping after first job execution

Context
We are currently working with the TPC-DS benchmark which uses the Table API to execute multiple SQL queries (around 100) each as a single job.
Problem
When using the operator to deploy a Flink cluster along with the job with the following yaml example:
apiVersion: flinkoperator.k8s.io/v1beta1
kind: FlinkCluster
metadata:
  name: tpcds
...
---
job:
    jarFile: /cache/flink-tpcds-1.14.3-jar-with-dependencies.jar
    className: org.apache.flink.table.tpcds.TpcdsTestProgramS3
...

this successfully creates a job executing the first query but Flink shuts down right after while the job submitter tries to create another job for the second query. When manually running the benchmark on a FlinkCluster, this issue does not appear.
Is this the expected behavior?
Best,
Donatien

Hi, @Donaschmi sorry for the late reply!
So FlinkCluster with a Job will represent a Job Cluster, meaning one job per cluster. Right now, if you want to run multiple jobs within the same cluster you will need to deploy  a FlinkCluster (no job spec) and then have a job that submits your jobs to this cluster;
Not sure about your requirements, but another thing you can do is to split jobs across multiple definitions, using args or different classNames i.e
apiVersion: flinkoperator.k8s.io/v1beta1
kind: FlinkCluster
metadata:
  name: tpcds-1
...
---
job:
    jarFile: /cache/flink-tpcds-1.14.3-jar-with-dependencies.jar
    className: org.apache.flink.table.tpcds.TpcdsTestProgramS3Job1
...

apiVersion: flinkoperator.k8s.io/v1beta1
kind: FlinkCluster
metadata:
  name: tpcds-2
...
---
job:
    jarFile: /cache/flink-tpcds-1.14.3-jar-with-dependencies.jar
    className: org.apache.flink.table.tpcds.TpcdsTestProgramS3Job2
...

No response. Hope you got it going.


A invalid link on core-icons page

This link is invalid.
http://www.polymer-project.org/components/core-icons/demo.html
It should be
https://www.polymer-project.org/0.5/components/core-icon/demo.html

Note
core-icon (correct)
core-icons (wrong)

Which page is the link on? There is a core-icons demo as well: https://www.polymer-project.org/0.5/components/core-icons/demo.html
We've made a fix on the core-icon reference page, which hasn't yet been deployed. But I'm not sure if that's the same link you're reporting here.

On this page, https://www.polymer-project.org/0.5/docs/elements/core-icon.html
You can find the link by searching in page, "See core-icons for the default set of icons."
I think the last link you showed should be the most correct! :D

Closing this issue due to age and the release of version 1 of Polymer - please feel free to re-open if this is incorrect.


Add HF integration

Hi @learn2phoenix,
Thanks for this nice work! I wrote a quick PoC to showcase that you can easily have integration with the 🤗 hub so that you can automatically load the model using from_pretrained (and push it using push_to_hub), track download numbers for your models (similar to models in the Transformers library), and have nice model cards on a per-model basis, and perhaps most importantly, leverage safetensors for the weights in favor of pickle.
Also, this greatly improves the discoverability of your model (as it's currently hosted on Google Drive which is hard to find).
It leverages the PyTorchModelHubMixin class which allows to inherits these methods.
Usage is as follows:
from CSD.model import CSD_CLIP
from huggingface_hub import hf_hub_download
import torch

# instantiate model
model = CSD_CLIP("resnet50", "default")

# equip model with weights
filepath = hf_hub_download(repo_id="tomg-group-umd/CSD-ViT-L", filename="checkpoint.pth", repo_type="model")
state_dict = torch.load(filepath, map_location="cpu)
model.load_state_dict(state_dict)

# push to hub
model.push_to_hub("your-hf-org-or-username/csd-clip")

# reload
model = CSD_CLIP.from_pretrained("your-hf-org-or-username/csd-clip")

This means people don't need to manually download a checkpoint first in their local environment, it just loads automatically from the hub.
Would you be interested in this integration?
Kind regards,
Niels
Note
Please don't merge this PR before pushing the model to the hub :)

@NielsRogge Thanks for suggesting the integration and I agree it is valuable.
The model is available at https://huggingface.co/tomg-group-umd/CSD-ViT-L Could you update the PR to work with this?

Hi, thanks for linking the model.
Sure, although would it be possible to try out the script above using my branch? I've updated it accordingly with loading the weights before pushing to the hub.

@NielsRogge
This seems to work with following changes in the usage you suggested:
Include the following function in the CSD/model.py
def load_model_state(model, state_dict_path):
    # Load the full state dictionary
    full_state_dict = torch.load(state_dict_path)

    # Extract only the model state dictionary
    if "model_state_dict" in full_state_dict:
        model_state_dict = full_state_dict["model_state_dict"]
    else:
        model_state_dict = full_state_dict  # Assume it's already just the model state

    # Load the state dictionary, ignoring missing keys
    model.load_state_dict(model_state_dict, strict=False)

    return model

model = CSD_CLIP("vit_large", "default")
# equip model with weights
filepath = hf_hub_download(repo_id="tomg-group-umd/CSD-ViT-L", filename="checkpoint.pth", repo_type="model")
model = CSD_CLIP(name='vitlarge', content_proj_head='default')
model = load_model_state(model, 'path/to/your/state_dict.pth')

Could you include the usage instructions in README.md?
Could you make these changes and I can then merge the PR

Cool! Ok yes I'll remove the push_to_hub, this was just to show how you could push the weights to https://huggingface.co/tomg-group-umd/CSD-ViT-L.
Would you be able to overwrite the weights of that repo? A model.safetensors file will be pushed in case you call push_to_hub

I don't think we need to overwrite. That repo has limited write access, so all's safe there.

Oh ok, to which repository could you push the weights in that case? Do you have a personal HF profile?

Why do we want to push the weights? I have already pushed the CSD weights to this repo for users to use huggingface. Am I missing something here, I am sorry my experience with latest on huggingface is limited

The reason I would overwrite the model repo (which is the purpose of this PR) is to have an easier integration between the model weights on the hub and your Github repository (by making it possible to do model = CSD_CLIP.from_pretrained("your-hf-org-or-username/csd-clip")).
Moreover, currently there's no model card, metadata tags and download counts aren't working, which are all resolved by this :) (meaning everytime someone calls from_pretrained, the download counter will go up by one).

Fair enough. I'm beginning to understand how HF pages work, thanks for pointing me in this direction. Following are the things that I've done:

Added the model card. Could you check and let me know if it is fine to proceed?
renamed checkpoint.pth to pytorch_model.bin. This makes the from_pretrained  work
added a minimal config.json.  Downloads should have been tracked now but it says 0 (which is an improvement over earlier not being tracked at all) even after I successfully downloaded through pretrained call. Can you help with this?

Overwriting the model repo by the user is still not required. Let me know if you think otherwise

Ok great :) some remarks:

would be great to link the HF model in the README of this repo, which currently links to Google Drive.
it seems like download stats are working indeed for https://huggingface.co/tomg-group-umd/CSD-ViT-L, given that the repo now has a config.json. Nice hack ;)
would be great to update the pipeline_tag of the model card, since I don't think CSD is a text-to-text-generation model? Probably best suited here is pipeline_tag: image_feature_extraction, so that people find it at https://huggingface.co/models?pipeline_tag=image-feature-extraction
Regarding the purpose of this PR, it would allow to automatically load the model from the hub, without requiring users do download a checkpoint separately and putting it in a folder. Could you clarify where you leveraged a pretrained call?

Thanks for the review :)

HF link included in README. Code pushed to your branch
pipeline_tag updated

I did not really leverage from_pretrained in the current code as we have released weights only for ViT-L model and I  am not sure if it makes too much sense to have a special case for just this model

Thanks for the quick updates!
If I understand correctly, you made changes to the existing HF model repo so that this branch works out-of-the-box? I just tried the following:
from CSD.model import CSD_CLIP

# load from the hub
model = CSD_CLIP.from_pretrained("tomg-group-umd/CSD-ViT-L")

but that results in:
ModuleNotFoundError: No module named 'CSD.model'

strange. This works for me:

Any idea what could be happening here?

It's working fine for me, so if you are up for it, this PR can be merged.
The model has been downloaded 28 times last month I see: https://huggingface.co/tomg-group-umd/CSD-ViT-L.
Do note that wikiart is also available here: https://huggingface.co/datasets/huggan/wikiart, which means that generating embeddings can now be done in just the following lines :)
from datasets import load_dataset
from CSD.model import CSD_CLIP
import torch
from torchvision import transforms

transform = transforms.Compose([
            transforms.Resize(256),
            transforms.CenterCrop(224),
            transforms.ToTensor(),
            transforms.Normalize([0.5], [0.5]),
])

@torch.no_grad
def embed_batch(batch):
     images = batch["image"]

     # prepare for the model
     images = [transform(image) for image in images]
     images = torch.stack(images, dim=0)

     # embed
     embeddings = model(images)
     batch["embeddings"] = embeddings.numpy()

     return batch

# load from the hub
model = CSD_CLIP.from_pretrained("tomg-group-umd/CSD-ViT-L")

# load wikiart
dataset = load_dataset("huggan/wikiart")

# embed
dataset_with_embeddings = dataset.map(embed_batch, batched=True, batch_size=2)

This is just a PoC and an FYI :)

Thanks @NielsRogge for this PR. Appreciate the continuous feedback.


[OSF-3622] CSL citation fix

For CSL files with no specified bibliography information, default citation data is provided

I'm not sure if this is the best approach, given the differences between how U Schyłku Starożytności (Polish) looks on the Zotero style page and how it looks with these changes.
Ideally, we'd be able to prevent styles that we can't necessarily render correctly from being options presented to the user, rather than attempting to parse some json that may or may not be indicative of what the style is supposed to look like.
Three main points to achieve this:

Add flag on the CitationStyle model -- indicates presence of bibliography tag in the xml tree
Modify the default query to exclude models without a bib
Modify script to set flag on models.

Also, please follow the default template when submitting a PR:
## Purpose

<!-- Describe the purpose of your changes -->

## Changes

<!-- Briefly describe or list your changes  -->

## Side effects

<!--Any possible side effects? -->

## Ticket

<!-- Link to JIRA ticket, if applicable e.g. https://openscience.atlassian.net/browse/OSF-1234 -->

Furthermore, the build is failing due to syntax errors (see the bottom of the log).
You can run these checks locally with invoke flake and invoke jshint

Pass complete @RomanZWang. Alternate approach proposed, happy to do a demo / talk about concerns touched on in that comment. 🎶

Just a drive-by comment: there's a lot of noise in the history for this PR. @RomanZWang when you finish addressing Matt's review, can you create a new branch and commit only the necessary changes in a single commit or two?

Okay I will make these changes. Thank you for the feedback


Testing: Consider creating a JUnit rule for Jenksinfile.instance

Jenkinsfile.instance is relied upon a lot by terraform-pipeline, to allow access to underlying Jenkinsfile methods
The instance is null by default, and tests often require mocking Jenkinsfile.instance methods.  Eg: environment variables - https://github.com/manheim/terraform-pipeline/blob/master/test/AwssumePluginTest.groovy#L19-L28
There's a lot of copy/paste of mocking code to set environment variables.  Worse, not all tests clean up after themselves, to guarantee that environment variables set in one tests are no interfering in another test.
Consider creating a JUnit @Rule that allows easy and consistent mocking/resetting for Jenkinsfile.instance
The suggestion below is modeled after JUnit's TemporaryFolder - https://junit.org/junit4/javadoc/4.12/org/junit/rules/TemporaryFolder.html

Example:

OLD

class AwssumePluginTest {
    @Before
    void resetJenkinsEnv() {
        Jenkinsfile.instance = mock(Jenkinsfile.class)
        when(Jenkinsfile.instance.getEnv()).thenReturn([:])
    }

    private configureJenkins(Map config = [:]) {
        Jenkinsfile.instance = mock(Jenkinsfile.class)
        when(Jenkinsfile.instance.getStandardizedRepoSlug()).thenReturn(config.repoSlug)
        when(Jenkinsfile.instance.getEnv()).thenReturn(config.env ?: [:])
    }
...
            @Test
            void addsEnvironmentSpecificRoleArnAsPrefixToTerraformInit() {
                String environment = "myEnv"
                configureJenkins(env: ['MYENV_AWS_ROLE_ARN': 'someArn'])
                AwssumePlugin plugin = new AwssumePlugin()
                TerraformInitCommand command = new TerraformInitCommand(environment)

                plugin.apply(command)

                String result = command.toString()
                assertThat(result, containsString("AWS_ROLE_ARN=someArn awssume terraform init"))
            }
...

Potential NEW

class AwssumePluginTest {
    @Rule
    MockJenkinsfile mockJenkinsfile = new MockJenkinsfile()
...
            @Test
            void addsEnvironmentSpecificRoleArnAsPrefixToTerraformInit() {
                String environment = "myEnv"
                // MockJenkinsfile automatically populate Jenkinsfile.instance, with the given environment variable
                mockJenkinsfile.withEnvironment('MYENV_AWS_ROLE_ARN', 'someArn')
                AwssumePlugin plugin = new AwssumePlugin()
                TerraformInitCommand command = new TerraformInitCommand(environment)

                plugin.apply(command)

                String result = command.toString()
                assertThat(result, containsString("AWS_ROLE_ARN=someArn awssume terraform init"))
                // MockJenkinsfile automatically resets Jenkinsfile.instance and environment variables at end of test
            }
...

JUnit 5 replaced Rules with Extensions.  The intent of this Issue is solved by MockJenkinsfile and ResetStaticStateExtension from Issue #271.


[SIL] Remove unused ResilienceExpansion from SILDeclRef

We ended up not using this, so let's not leave it in as cruft. No functionality change.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - ab60a11ff387d0762484c37e00034843b61026d9

@swift-ci Please smoke test


minor issue: convert MapLoader to singleton

There should DEFINITELY never under any circumstances be another MapLoader
we do not need duplicates of our freakin' object pools floating around in memory
this game is going to be so resource intensive AS IT IS. jeez fam

I really don't wanna do this so I'm gonna put off the work on my poor teammates but luckily it's easy just follow the lead of my MapGenerator e.g. private static MapGenerator instance = null;


Add MySQL's SET column type to the Schema Builder

I noticed that there is no SET column type available for the Schema Builder.
http://dev.mysql.com/doc/refman/5.0/en/set.html

Laravel supports custom schemas. If you thin the SET column would be useful to lots of people, then please send a pull request to add it.

Ish...
https://coderwall.com/p/mo1gew/custom-datatype-in-laravel-schema-builder

https://github.com/laravel/framework/pull/24229


Skip writing parquet output files if there were no validation errors

See https://github.com/cal-itp/data-infra/issues/1180 for more context.
This PR fixes the actual underlying cause, but I've already moved the offending files currently in GCS.

Is there any chance that we want to write a file that's called like empty_service_alerts or something just to remove any ambiguity about whether the validator actually ran on the feed? I'm just wondering about a world where we can't tell the difference between "validator didn't run" and "no validations found". Or, as you mentioned in another place, can we make sure that the columns will be written but empty?

Eh I'm hesitant to add dummy files. I think the columns idea is interesting, but right now I'm trying to minimize changes since I want to move to ndjson anyways. Is that fair?

Ok, I'm fine with that as long as we prioritize https://github.com/cal-itp/data-infra/issues/1345 soon

Fixes https://github.com/cal-itp/data-infra/issues/1180


Feed URL
https://mattcool.tech/rss.xml
More Info
No response

closed by https://github.com/murtuzaalisurti/rssed/commit/c25baff75ede687cff8106836969021149757567


Can not processing the binary

In my config.exs, have this config:
use Mix.Config

config :conform_test, test: [
	{:hd_prime,<<
0xC7,0x1C,0xAE,0xB9,0xC6,0xB1,0xC9,0x04,0x8E,0x6C,0x52,0x2F,0x70,0xF1,0x3F,0x73,
0x98,0x0D,0x40,0x23,0x8E,0x3E,0x21,0xC1,0x49,0x34,0xD0,0x37,0x56,0x3D,0x93,0x0F,
0xFA,0x33,0x6F,0x6E,0x0A,0xC9,0x25,0x13,0x95,0x43,0xAE,0xD4,0x4C,0xCE,0x7C,0x37,
0xDC,0x97,0x46,0x51,0x29,0x69,0x32,0x84,0x54,0xF1,0x8F,0xAF,0x8C,0x59,0x5F,0x64,
0x24,0x77,0xFE,0x96,0xBB,0x2A,0x94,0x1D,0x5B,0xCD,0x1D,0x4A,0xC8,0xCC,0x49,0x88,
0x0D,0x81,0x15,0xF6,0x35,0xB1,0x05,0xEE,0x2E,0x4E,0x15,0xD0,0x4B,0x24,0x54,0xBF,
0x48,0x19,0x8A,0x0A,0xA7,0xC1,0x40,0x58,0x22,0x94,0x93,0xD2,0x25,0x30,0xF4,0xDB,
0x07,0x08,0xFA,0x9B,0x37,0x8E,0x3C,0x4F,0x3A,0x90,0x60,0xBE,0xE6,0x7C,0xF9,0xA4,
0x20,0xFD,0x51,0xF6,0x94,0x58,0x70,0x5A,0xC6,0x8C,0xD4,0xFE,0x6B,0x6B,0x13,0xAB,
0x0D,0xBA,0x74,0xD8,0xA8,0x4B,0x2A,0x14,0xB3,0x14,0x4E,0x0E,0xF1,0x28,0x47,0x54
	>>}
]

When i execute the mix conform.new the following errors raised:
➜  conform_test mix conform.new

You already have a schema at config/conform_test.schema.exs.
Do you want to overwrite this schema with a new one?
 [Yn]

** (FunctionClauseError) no function clause matching in Conform.Utils.Code.to_heredoc/3
    lib/conform/utils/code.ex:213: Conform.Utils.Code.to_heredoc(<<199, 28, 174, 185, 198, 177, 201, 4, 142, 108, 82, 47, 112, 241, 63, 115, 152, 13, 64, 35, 142, 62, 33, 193, 73, 52, 208, 55, 86, 61, 147, 15, 250, 51, 111, 110, 10, 201, 37, 19, 149, 67, 174, 212, 76, 206, 124, 55, 220, 151, ...>>, :open, "\"\"\"\n      ")
    lib/conform/utils/code.ex:84: Conform.Utils.Code.format_list_item/2
    lib/conform/utils/code.ex:63: Conform.Utils.Code.format_list/2
    lib/conform/utils/code.ex:84: Conform.Utils.Code.format_list_item/2
    lib/conform/utils/code.ex:62: Conform.Utils.Code.format_list/2
    lib/conform/utils/code.ex:84: Conform.Utils.Code.format_list_item/2
    lib/conform/utils/code.ex:63: Conform.Utils.Code.format_list/2
    lib/conform/schema.ex:289: Conform.Schema.stringify/2

You can try this test project at https://github.com/developerworks/conform_test

Interesting, ok I'll take a look at this ASAP


Implement a routine for calculating growing season from datetime and temperature series.

The seasonal phenology dynamics requires estimates of conditions within the growing season of a site, so we need a routine to identify the growing season within a time series of observations across years.
This basically just takes a year or longer time series of temperature values and their timestamps and the growing season is "just" where  the temperatures within years are  above 0°C, but needs:

Arguments to tweak treatment of cold snaps: how long a snap breaks a season?
We might also need hemisphere to handle array inputs that include northern and southern summers and hence where the season should be measured around winter not summer equinox.
This is going to differ for models on daily data versus subdaily data: is it mean daily temperature or mean daytime temperature.

Relevant comments from this https://github.com/ImperialCollegeLondon/pyrealm/issues/348#issuecomment-2553437754 by @Shirleycwj

It's a little bit more complicated when it comes to the growing season. I think we should open up the option of different methods to define growing season but use the temperature threshold as the simplest, default method. Using the temperature threshold only (above 0 degree), timestep up to monthly is actually acceptable due to temperature oscillation at the beginning and end of the growing season. Currently there are no absolute consensus on how growing season is defined (thermal or radiative or productive growing season, see Körner et al 2023 Ecology letters and Fu et al 2022 Global change biology ) or method to define growing season...so we could ask the user to provide DOY of start of the growning season (SOS) and end of growing season (EOS) to calculate growing season value for input variables. If the user works on phenology then they would have such information, otherwise they could use our simplified way to define productive growning season.


4.10.0 crashes on startup - Linux Mint 20.2 (fresh install)

I get a splash screen before the game crashes. I've tried running in portable mode, using the Preferences.ini file from a Stepmania 5.0.12 install, and launching with pasuspender. The legacy versions create /tmp/crashinfo.txt with a note to file a bug report, the regular versions exit with "Fatal: crash from within the crash signal handler."
Linux MInt 20.2 Cinnamon (fresh install)
Intel Core i3-2105, using integrated graphics, 4G RAM
OutFox-5.3.0-alpha-4.9.10-amd64-date-20210827
OutFox-5.3.0-alpha-4.9.10-amd64-legacy-date-20210827
OutFox-5.3.0-alpha-4.10.0-amd64-date-20211001
OutFox-5.3.0-alpha-4.10.0-amd64-legacy-date-2021100
The attached files were created using the portable mode (portable.ini exists in the OutFox root directory).
4.10.0-legacy-crashinfo.txt
4.10.0-legacy-info.txt
4.10.0-legacy-log.txt
4.10.0-info.txt
4.10.0-log.txt

I will force the GL version for Linux to prevent this error in the next alpha.
Good catch.


rkt: don't errwrap cli_apps errors

We don't print them with the rkt log package, it's cobra/pflag who does
it, so the concrete error messages were missing.

sure, lgtm


Remove ICEES+ API KG endpoints

This issue is to remove the multiple KG endpoints from each ICEES+ API instance, as they are no longer needed given ICEES KG and also outdates (TRAPI 1.2).

@karafecho This issue can be closed now that all ICEES+ production instances have been updated with KG related API endpoints removed.

Agreed, closing issue.


Updated example to reflect changes in scroll.Spec.

Purpose
Updated example in README.md to reflect changes to scroll.Spec structure.
Implementation
Removed line 55.

I can see that you put a lot of thought and effort into this change. Here's your reward: lgtm.


oncompleate set value incorrect action

If in "oncomplete" callback make setVal action with other masked input, you can see incorrect reactions in Chrome and Safary, well work in FF
Demo: http://jsfiddle.net/serGlazkov/nxLzq82o/

@serGlazkov ,
I updated the jsfiddle to the latest version of inputmask.
http://jsfiddle.net/nxLzq82o/1/   I see that the input looses focus.

@serGlazkov ,
Try again with the latest version


PlexRequest type functionality.

I would LOVE for this type of functionality to be added, esp. if it could support emby as well.
Plex Reqest can be found here: http://plexrequests.8bits.ca/

I would love this feature too so that people could request TV shows and/or movies. Similar to the Couch Potato extension but maybe a webpage with the ability to add TV or Movies.

+1


Pie chart / Scatter chart are not available in package

Can you please add more charts and existing charts like Pie chart / Scatter chart. as they are not available in package

I've removed these from the documentation. We have no plans to add further charts to this package at the moment.


Depth of data slicing wrong

Hi
Edit: Added error message
I'd like to ask for some advice regarding the behavior of mpire.
I have a list of Atoms objects: data = [Atoms0, Atoms1, Atoms2, Atoms3, ...]
This Atoms0 = [Atoms0[0], Atoms0[1], Atoms0[2], ... ]
So data is some sort of nested list of objects.
However, when I use the MPIRE package (as a drop-in replacement for multiprocessing) the slicing of data is wrong.
I expect for the workers to behave like:
worker 0 <- function(Atoms0)
worker 1 <- function(Atoms1)
worker 2 <- function(Atoms2)
...

Meanings the depth of slicing is 1 (the primary list)
Instead the behavior (based on a nice error handling):
worker 0 <- function(Atoms0[0], Atoms0[1], Atoms0[2], Atoms0[3], ...)
worker 1 <- function(Atoms1[0], Atoms1[1], Atoms1[2], Atoms1[3], ...)
worker 2 <- function(Atoms2[0], Atoms2[1], Atoms2[2], Atoms2[3], ...)
...

Which suggest that the input arguments being used is of depth=2 (secondary list)
I am not sure what is triggering this behavior.
My current workaround is to iterate over index numbers instead of the list objects like:
def task(data_index):
    return self.time_consuming_function(self.data[data_index])
with WorkerPool(n_jobs=5) as pool:
    self.results = pool.map(task, range(len(self.data)))

This workaround works properly.
For simpler testing, I have prepared a code that can replicate this
from mpire import WorkerPool
import numpy as np
from ase.build import bulk

class MyClass:
    def __init__(self):
        # Make a large atomic structure
        self.system = bulk('Al', 'fcc', a=4.05, cubic=True)
        self.system.repeat((4,4,4))

        # Make a list of 100 copies of the structure
        self.data = [self.system for _ in range(100)]
        ## Self.data = [Atoms_object, Atoms_object, ...]
        ## Atoms objects is a list of Atom objects
        ## Self.data = [[Atom_1_object, Atom_2_object, ...], [Atom_1_object, Atom_2_object, ...], ...]

    def time_consuming_function(self, x):
        # Gets the positions array of the atoms in the structure
        return x.get_positions()

    def run_1(self):
        self.results = []
        for x in self.data:
            self.results.append(self.time_consuming_function(x))

    def run_2(self):
        with WorkerPool(n_jobs=5) as pool:
            self.results = pool.map(self.time_consuming_function, self.data)

    def run_3(self):
        def task(data_index):
            return self.time_consuming_function(self.data[data_index])
        with WorkerPool(n_jobs=5) as pool:
            self.results = pool.map(task, range(len(self.data)))

z = MyClass()
z.run_1() # For-loop implementation
z.run_2() # MPIRE implementation
z.run_3() # MPIRE implementation + workaround

Thansk for the time.
---------------------------------------------------------------------------
Exception                                 Traceback (most recent call last)
Exception: Exception occurred in Worker-0 with the following arguments:
Arg 0: Atom('Al', [0.0, 0.0, 0.0], index=0)
Arg 1: Atom('Al', [0.0, 2.025, 2.025], index=1)
Arg 2: Atom('Al', [2.025, 0.0, 2.025], index=2)
Arg 3: Atom('Al', [2.025, 2.025, 0.0], index=3)
Traceback (most recent call last):
  File "/home/krojas/apps/mambaforge/envs/pyagox/lib/python3.10/site-packages/mpire/worker.py", line 458, in _run_safely
    results = func()
  File "/home/krojas/apps/mambaforge/envs/pyagox/lib/python3.10/site-packages/mpire/worker.py", line 391, in _func
    _results = func(*args) if is_apply_func else func(args)
  File "/home/krojas/apps/mambaforge/envs/pyagox/lib/python3.10/site-packages/mpire/worker.py", line 571, in _helper_func_with_idx
    return args[0], self._call_func(func, args[1])
  File "/home/krojas/apps/mambaforge/envs/pyagox/lib/python3.10/site-packages/mpire/worker.py", line 596, in _call_func
    return func(*args, **kwargs)
TypeError: MyClass.time_consuming_function() takes 2 positional arguments but 5 were given

By default MPIRE unpacks iterables. In other words, the map is actually doing what starmap does in the vanilla multiprocessing package. See https://sybrenjansen.github.io/mpire/usage/map/map.html#circumvent-argument-unpacking for ways to avoid this behavior.
Bottom-line: wrap your data with mpire.utils.make_single_arguments or use
self.data = [(Atoms0,), (Atoms1,), (Atoms2,), (Atoms3,), ...]


Add text for empty add cell

Adds the text No Processed Data and expands button to be Add To Analysis

:+1:


Investigate best dapp / web3 context testing practices

Ask others in the space what their best practices are, specifically how web3 & mobile dapp browsers are handled
Can also be done concurrently on Berlin trip
And research browserstack for device compatibility testing
Also, can we automate tests?
Output is a notion document.

https://www.notion.so/threebox/dApp-Testing-db01e089eac94ad98cb862b85e8e660e


Rename package to jupyter-scheduler

Checklist

[x] Used a personal fork of the feedstock to propose changes
N/A as we are changing the package name. Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Problem
Currently main branch of this feedstock repo has package name jupyter_scheduler while 1.x branch jupyter-scheduler.
Github repo jupyter-scheduler
says "This extension is composed of a Python package named jupyter_scheduler for the server extension and a NPM package named @jupyterlab/scheduler for the frontend extension." and asks to use pip install jupyter_scheduler to install the extension. It's not clear if jupyter_scheduler or jupyter-scheduler is the name of the extension / package.
Proposed solution
Use jupyter-scheduler as a Jupyter Scheduler package name, use jupyter_scheduler to refer to the server component. This would be in line with how package is named at PyPi website, how jupyter-ai package is named at conda-forge.
This PR updates a set name in the recipe which will make 2.1.0 available at conda-forge under jupyter-scheduler.
Next steps after this PR is approved:

Mark conda-forge jupyter_scheduler package as broken following steps outlined in the docs
Update all packages that use conda-forge Jupyter Scheduler as a dependency to use correct package name jupyter-scheduler
Update Jupyter Scheduler Github to use jupyter-scheduler as a package name (for example, when providing pip install commands)

@conda-forge-admin, please rerender


Fix return type for verifyMicrodepositsForSetup

Summary & motivation
Small fix for return type for verifyMicrodepositsForSetup method
Testing & documentation
I think no tests are required in this case, really simple fix.

@jima-stripe please take a look?

Not stale.


Brightness scale?

Hi... sorry for what is probably a dumb question... I'm looking at the brightness setting... and its default of 1.... but what's its scale?  Is it from 0 to 1... or 1 to 10?

0 = Off.
1 = Full Brightness.
So for 50% brightness, set to 0.5

I'm unable to set a brightness of "0.5" and/or "0,5".
Can this be caused by a different regional decimal seperator? In the Netherlands this a comma.

That's a bug.
Type 5, then put a . before it
On Sat, 28 Mar 2020, 14:59 Meow<EMAIL_ADDRESS>wrote:

I'm unable to set a brightness of "0.5" and/or "0,5".
Can this be caused by a different regional decimal seperator? In the
Netherlands this a comma.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ahodges9/LedFx/issues/92#issuecomment-605458599, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHXFVTAKD6PCRARHM44HI3TRJYGFXANCNFSM4LUBPLPA
.


🐛 [Tandoor] all images not found after update to 1.5.11

Description
🐛 [Tandoor] all images not found after update to 1.5.11
Reproduction steps
All images of receipe are not found

Tandoor webpage search here :
https://url.com/media/recipes/f03d365a-0183-46d5-9990-84a49a40b4bf_15.jpeg

It doesn't work either by deleting an image and adding a new one.

I think it's a change with /addons_config/ of hassio.

Addon Logs
Starting...
/etc/cont-init.d/00-banner.sh: executing

-----------------------------------------------------------
 Add-on: Tandoor recipes
 recipe manager
-----------------------------------------------------------
 Add-on version: 1.5.11
 You are running the latest version of this add-on.
 System: Home Assistant OS 11.4  (aarch64 / raspberrypi4-64)
 Home Assistant Core: 2024.1.2
 Home Assistant Supervisor: 2023.12.1
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums
-----------------------------------------------------------
 Provided by: https://github.com/alexbelgium/hassio-addons
-----------------------------------------------------------
/etc/cont-init.d/00-global_var.sh: executing
ALLOWED_HOSTS='<IP_ADDRESS>,<IP_ADDRESS>'
DB_TYPE='sqlite'
DEBUG='0'
POSTGRES_DB='pg_tandoor_db'
POSTGRES_HOST='pg_tandoor_host'
POSTGRES_PASSWORD=******
POSTGRES_PORT='5432'
POSTGRES_USER='pg****'
SECRET_KEY='TA_SUPER_SECRET_CLEF'
externalfiles_folder='/addons_config/tandoor_recipes/externalfiles'
/etc/cont-init.d/01-config_yaml.sh: executing
Setting permissions for the config.yaml directory

Load environment variables from /config/addons_config/tandoor-recipes/config.yaml if existing
If accessing the file with filebrowser it should be mapped to /homeassistant/addons_config/tandoor-recipes/config.yaml
---------------------------------------------------------
Wiki here on how to use : github.com/alexbelgium/hassio-addons/wiki/Add‐ons-feature-:-add-env-variables

... no env variables found, exiting
/etc/cont-init.d/01-custom_script.sh: executing
Execute /homeassistant/addons_config/tandoor-recipes/tandoor-recipes.sh if existing
Wiki here : github.com/alexbelgium/hassio-addons/wiki/Add-ons-feature-:-customisation
/etc/cont-init.d/20-folders.sh: executing
/etc/cont-init.d/32-ingress.sh: executing
[19:07:15] INFO: Ingress disabled
/etc/cont-init.d/32-nginx.sh: executing
/etc/cont-init.d/99-run.sh: executing
ALLOWED_HOSTS=<IP_ADDRESS>,homeassistant.cuscus-pike.ts.net,<IP_ADDRESS>
SECRET_KEY=TA_SUPER_SECRET_CLEF
DEBUG=0
[19:07:16] INFO: Defining database
[19:07:16] INFO: Using a local sqlite database
Creating symlinks
[19:07:16] INFO: Launching app
Checking configuration...
Waiting for database to be ready...
Database is ready
Migrating database
Operations to perform:
  Apply all migrations: account, admin, auth, authtoken, contenttypes, cookbook, oauth2_provider, sessions, sites, socialaccount
Running migrations:
  No migrations to apply.
Generating static files
js-reverse file written to /opt/recipes/cookbook/static/django_js_reverse

1 static file copied to '/opt/recipes/staticfiles', 648 unmodified, 1397 post-processed.
Done
[2024-01-26 19:08:03 -0500] [378] [INFO] Starting gunicorn 20.1.0
[2024-01-26 19:08:03 -0500] [378] [INFO] Listening at: http://<IP_ADDRESS>:8080 (378)
[2024-01-26 19:08:03 -0500] [378] [INFO] Using worker: gthread
[2024-01-26 19:08:03 -0500] [383] [INFO] Booting worker with pid: 383
[2024-01-26 19:08:03 -0500] [384] [INFO] Booting worker with pid: 384
[2024-01-26 19:08:03 -0500] [385] [INFO] Booting worker with pid: 385

Architecture
No response
OS
No response

I have the same problem.
I found that Tandoor is looking for the images on:
https://url.com/media/recipes/ID.jpeg
but in the file system the images are on:
/tandoor/mediafiles/recipes/ID.jpeg
I tryed to change the foldername but this dosen't work. If i upload a new image via Tandoor Webpage, the image will uploaded to /tandoor/mediafiles/recipes/ID.jpeg but can't show on the Tandoor Webpage because on the Webpage it is /media/recipes/ID.jpeg.

⚠ Changed setting GUNICORN_MEDIA default to False. Very old (several years) installations not using a reverse proxy might see media file serving stop working. Set GUNICORN_MEDIA=1 to restore old behavior, see Docs for more explanation. It is not recommended to use gunicorn for media serving.

Thanks, I'll try a version removing my own nginx server so it will use the embedded one

Mmh not working with the upstream nginx server, I'll investigate

Setting DEBUG=1 worked, I'm not sure GUNICORN is right as I got a warning that was ignored.

It's not working out of the box? I had forced gunicorn usage and it works directly on my system


Add a function to make savefiles for games

This I think will make TIC-80 different from others Fantasy Consoles

you can use pmem api or you can store data to sprites/map/sfx/music RAM area and then call sync to copy it to the cartridge, a user will see Save Cart Dialog in this case


Relativize spec paths more when reporting

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions

Description of the Change
First, here's a before/after comparison on Windows:

Before
After

Much nicer.
Changes include:

Updating to the new .<anonymous> syntax
Stripping away file:/// identifiers on Windows paths and converting the rest of the path to use Windows path separators (in order for it to get relativized correctly)
Relativizing paths in parentheses

Alternate Designs
Instead of relativizing the path twice, depending on if its in parentheses or not, a more general line.replace(///#{_.escapeRegExp(spec.specDirectory}#{_.escapeRegExp(path.sep}///, '') could be used.  I am unsure if that would potentially relativize more than we wanted.
Why Should This Be In Core?
Because it fixes already-implemented code in core.
Benefits
See before/after picture above.
Possible Drawbacks
I don't foresee any.
Applicable Issues
None.
/cc @damieng for the Windows portions.

If specs are necessary for this change I can add them.

This looks good to me. I'm okay with keeping windows-style paths for local stuff although for the areas where we report remotely - bugsnag, GA, notification to github issue - I prefer to normalize them to Unix style so that they are correctly combined into a single issue where appropriate.
Thanks for this!