Is it really necessary to "prepare likelihoods for annotation"?

In SomaticGenotypingEngine::callMutations and HaplotypeCallerGenotypingEngine::assignGenotypeLikelihoods there is a line of code after the call is made but before the variant is annotated:
ReadLikelihoods annotationLikelihoods = prepareReadAlleleLikelihoodsForAnnotation(likelihoods...)

Is this really necessary?  It seems quite defensible to annotate using the same likelihoods from which the variant call is derived.  As far as Mutect is concerned, the standard will be whether our validations are better or at least no worse without it.

Issue moved to broadinstitute/gatk #3022 via ZenHub


Cert-manager http01 challenge 404 authorization error

I have a single node k3s cluster on a VPS. Traefik Ingress routes are succesfully tested on the server using curl. However my https lets encrypt config does not work with the following config.  Sorry in advance, but if this is not the right place to ask, maybe put me in the right direction? Although I saw some similar issues here, so I have my hopes up!
The challenge fails with the following output:
Accepting challenge authorization failed: acme: authorization error for example.com: 403 urn:ietf:params:acme:error:unauthorized: 2a03:b0c0:2:d0::14bf:4001: Invalid response from https://example.com/.well-known/acme-challenge/q-5BZ4t4ucyE5gLiIrMQccmhzOlqnTbtxxyofOH9r7c: 404
What I have tried so far and thinks I think are remarkable:

added to /etc/hosts/ the external ip and domain (example.com)
go to domain in browser and I see a default webpage of the DNS management provider
looked into the cert-manager config (installed with helm) and maybe I need to add something here of my domain? I copied it in to be sure...
Curling myself to the acme-challenge URL also returns 404

apiVersion: [cert-manager.io/v1](http://cert-manager.io/v1)
kind: ClusterIssuer
metadata:
name: letsencrypt-staging
namespace: default
spec:
acme:
email: *
privateKeySecretRef:
name: staging-issuer-account-key
server: https://acme-staging-v02.api.letsencrypt.org/directory
solvers:
- http01:
ingress:
ingressTemplate:
metadata:
annotations:
[kubernetes.io/ingress.class](http://kubernetes.io/ingress.class): traefik

apiVersion: [networking.k8s.io/v1](http://networking.k8s.io/v1)
kind: Ingress
metadata:
name: traefik-ingress
annotations:

#nginx.ingress.kubernetes.io/rewrite-target: /
kubernetes.io/ingress.class: "traefik"
cert-manager.io/cluster-issuer: letsencrypt-staging
spec:
tls:
- secretName: example-com
hosts:
- [example.com](http://example.com/)
rules:
- host: "[example.com](http://example.com/)"
http:
paths:
- pathType: Prefix
path: "/api/"
backend:
service:
name: example-backend
port:
number: 8080
- pathType: Prefix
path: "/"
backend:
service:
name: example-ui
port:
number: 80

apiVersion: [cert-manager.io/v1](http://cert-manager.io/v1)
kind: Certificate
metadata:
name: example-com
namespace: default
spec:
secretName: example-tls
issuerRef:
name: letsencrypt-staging
kind: ClusterIssuer
commonName: [example.com](http://example.com/)
dnsNames:
- [example.com](http://example.com/)

apiVersion: apps/v2
kind: Deployment
metadata:
  annotations:
    deployment.kubernetes.io/revision: "1"
    meta.helm.sh/release-name: cert-manager
    meta.helm.sh/release-namespace: cert-manager
  creationTimestamp: "2022-12-29T21:12:37Z"
  generation: 1
  labels:
    app: cert-manager
    app.kubernetes.io/component: controller
    app.kubernetes.io/instance: cert-manager
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/name: cert-manager
    app.kubernetes.io/version: v1.10.1
    helm.sh/chart: cert-manager-v1.10.1
  name: cert-manager
  namespace: cert-manager
  resourceVersion: "430862"
  uid: fdd006c5-6bf0-4ea2-bf85-1fc5d45c36ee
spec:
  progressDeadlineSeconds: 600
  replicas: 1
  revisionHistoryLimit: 10
  selector:
    matchLabels:
      app.kubernetes.io/component: controller
      app.kubernetes.io/instance: cert-manager
      app.kubernetes.io/name: cert-manager
  strategy:
    rollingUpdate:
      maxSurge: 25%
      maxUnavailable: 25%
    type: RollingUpdate
  template:
    metadata:
      annotations:
        prometheus.io/path: /metrics
        prometheus.io/port: "9402"
        prometheus.io/scrape: "true"
      creationTimestamp: null
      labels:
        app: cert-manager
        app.kubernetes.io/component: controller
        app.kubernetes.io/instance: cert-manager
        app.kubernetes.io/managed-by: Helm
        app.kubernetes.io/name: cert-manager
        app.kubernetes.io/version: v1.10.1
        helm.sh/chart: cert-manager-v1.10.1
    spec:
      containers:
        - args:
            - --v=2
            - --cluster-resource-namespace=$(POD_NAMESPACE)
            - --leader-election-namespace=kube-system
          env:
            - name: POD_NAMESPACE
              valueFrom:
                fieldRef:
                  apiVersion: v1
                  fieldPath: metadata.namespace
          image: quay.io/jetstack/cert-manager-controller:v1.10.1
          imagePullPolicy: IfNotPresent
          name: cert-manager-controller
          ports:
            - containerPort: 9402
              name: http-metrics
              protocol: TCP
          resources: {}
          securityContext:
            allowPrivilegeEscalation: false
            capabilities:
              drop:
                - ALL
          terminationMessagePath: /dev/termination-log
          terminationMessagePolicy: File
      dnsPolicy: ClusterFirst
      nodeSelector:
        kubernetes.io/os: linux
      restartPolicy: Always
      schedulerName: default-scheduler
      securityContext:
        runAsNonRoot: true
        seccompProfile:
          type: RuntimeDefault
      serviceAccount: cert-manager
      serviceAccountName: cert-manager
      terminationGracePeriodSeconds: 30
status:
  availableReplicas: 1
  conditions:
    - lastTransitionTime: "2022-12-29T21:12:40Z"
      lastUpdateTime: "2022-12-29T21:12:40Z"
      message: Deployment has minimum availability.
      reason: MinimumReplicasAvailable
      status: "True"
      type: Available
    - lastTransitionTime: "2022-12-29T21:12:37Z"
      lastUpdateTime: "2022-12-29T21:12:40Z"
      message: ReplicaSet "cert-manager-5958fd9d8d" has successfully progressed.
      reason: NewReplicaSetAvailable
      status: "True"
      type: Progressing
  observedGeneration: 1
  readyReplicas: 1
  replicas: 1
  updatedReplicas: 1```

The error that you are seeing is from LetsEncrypt trying to reach the challenge resource served by the cert-manager created solver pod. This is where the error comes from in cert-manager and this in LetsEncrypt.
You probably first want to look at cert-manager controller logs (increase log level with --v=5) to verify that the error you posted is not a one off and it is actually the case that the controller does get past it on following reconciles.
Then you'd want to look into why LetsEncrypt is not able to reach the solver pod in your cluster.
For reference
https://cert-manager.io/docs/troubleshooting/acme/

I have same setup as above, and similar problem. From what I see there is nothing exposing endpoint /.well-known/acme-challenge/xxxxx which I assume should point to cert-manager-webhook? In my case when calling /.well-known/acme-challenge it hits service which I want to expose, and because that service does not have such endpoint it results in 404... In what way that challenge endpoint should be exposed?

Sorry for not returning to @irbekrm , I totally forgot about this support issue after a quick holiday. I'll try make up to it by trying to help @piotrjaromin by telling what my solution was.
First what helped me is debugging the challenge better by 'kubectl describe' (cluster)Issuer and in the events you can see that you can go step by step deeper into the Orders and then Challenge as I remember correctly.
Second I started reading about the challenge between my pod en DNS Management. It came to understand that LetsEncrypt wants to connect firstly to IPv6 (AAAA) and then IPv4 (A). I checked my records and saw that my IPv6 was wrong and this was the answer why the challenge got a 404.

/close


Fixes 'teams report directroutingcalls' command. Closes #2680

Fixes teams report directroutingcalls command. Closes #2680
Updated to use the Microsoft Graph v1.0 endpoint.

Thank you @nanddeepn, we'll review it shortly.

Merged manually. Thank you! 👏


Question: tvOS support

Is there any plan to support Apple's tvOS anytime soon?

Moved to the forums :smile:
http://community.openfl.org/t/question-tvos-support/6561


scaffold landing page

@ALRubinger let me know if you need anything else for this.
@inoxx03 mind giving this a +1 if this looks good?

+1 on the PR, LGTM:)


Set /Os for some 32 bit MS compilers (fixes #3769)

Signed-off-by: Stefan Weil<EMAIL_ADDRESS>

@BJungmann, @krzysiekj94, does this patch work for you? I tried to fix the issues reported in #3283 and #3769, but cannot test it myself.

My suggestion was a patch in simddetect.cpp after line 199. This would execute a little faster than your suggested patch. I did not check the _MSC_VER limits, though.
`      if (max_function_id >= 7) {
_cpuid(cpuInfo, 7);
avx2_available = (cpuInfo[1] & 0x00000020) != 0;
if defined(_MSC_VER) && _MSC_VER >= 1925 && _MSC_VER <= 1929
    // forbid usage of AVX2 hardware for some
    // versions of the 32 bit Visual Studio compiler which generate buggy code.
    avx2_available_ = false;

endif
    avx512F_available_ = (cpuInfo[1] & 0x00010000) != 0;
    avx512BW_available_ = (cpuInfo[1] & 0x40000000) != 0;
  }

`


Test update command within test_e2e.go

We should be testing ocdev update since we're testing ocdev push already within our end-to-end tests.

there is PR opened for update #437
but we should add more update scenarios

PR open / merged / this issue can be closed.


Add welcome screen page and component

To do:

[x] Tests for welcome page / actions
[x] Wire up 'Get started' button to navigate to Explore

Issue #105
I used the application icon I arbitrarily chose as the logo placeholder.
Missing (TBD if within scope of AC):

the "is first time running application" persisted value
the related "no language selected" state in the language picker

Tests passing, thanks @trisapeace!

I'm fine with both raised issued being out of scope for this task


Release Testing: resilience example steps not clear

Ref #2422
The documentation at https://wallaroo-docs-rc.netlify.com/book/running-wallaroo/resilience.html has a Recovery in Practice section for directing users on how to run an example with resilience.
This document does not follow our conventions for trying to minimize confusion when folks try to run the myriad of commands (see https://wallaroo-docs-rc.netlify.com/book/getting-started/run-a-wallaroo-application.html or one of the other "Run a Wallaroo Application" pages in the book for our conventions).
The document does not tell users to start the Metrics UI nor does it tell users which directory they need to be running the commands from. Also doesn't tell users how to shut down the cluster.

I would like to add that under 'Recovery in Practice' the documentation states:

For example, if we were running the bundle Python example word_count,

But then the machida invocation wants to use word_count_with_dynamic_keys, which means we need to be in a different directory when invoking machida.


Optimize Guacamole docker image

What is being addressed
Update Guacamole docker image to make it smaller
How is this addressed

Each archive is unpacked and deleted in a single layer
OAtuth proxy unpacking moved to docker file to be run a single time in the build phase rather the runtime phase
Update JRE version to 11.0.15_10
Move builder (Maven) image source to regular rather openj9 we had before since we're not using this special JVM at runtime anyway
Use certain values as ENV vars inside the docker file
Pin tomcat version to 9.0.62 (current) so that it will not change in future builds without us knowing about it
Add dockerfile linter

/test

/test

/test-destroy-env

/test


[DOCS] Image is Not Visible in Readme File

Description
Image is not visible in Readme
Screenshots

Additional information
Due to Syntax error Image is not visible in Read.MD file  I have fixed  this issue  Please assign me to this issue
Code of Conduct

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

Resolved


Support for Blackmagic WDM Device

Hi all,
first of all great library! I experimented a little bit and found an issue with the infinite pixel formats that are out there. I am testing with a BlackMagic SDI input card, their driver includes a DirectShow filter (Blackmagic WDM Capture). It did not work out of the box, so I checked with the debugger and added a fourcc code:
HDYC = 0x43594448
which is basically "PixelFormats.UYVY". So adding a few more switches around the code worked for me.
I forked the repos and pushed my mods here:
https://github.com/flat-eric147/FlashCap
Would be great if this little patch could make it into the main repos!

@flat-eric147 Thank you reaching out FlashCap!
I don't have BMSDI, but that means the format is fine with UYVY, so I will merge it when I review it!

Merged.


Inspect/Fix performance regression

In the old editor/visualizer, the distribution would update almost instantaneously.
https://playground.squiggle-language.com/dist-builder
In the newer one, there seems to be some lag.
https://www.squiggle-language.com/docs/Language
My guess is that this is due to using Vega. If this is the case, it's possible we should just use D3 or similar. (Vega could still be a fallback).
With some simple performance monitoring, it seems like the old version takes me ~11ms, and the new version takes ~100ms.
It could also be the case that it's just Docusaurus that's slow (which would also be interesting)

It seems that the performance regression is half caused by Squiggle-lang and the other half by vega... I may have accidentally built a dev version of the website, which could also be an issue.

I think what I'll do is dial the default settings down to improve performance. I think they are a bit excessive

Ok, dialling down the settings improves performance and creates no issues with results. Currently, the biggest performance loss is loading so many ace instances, and considering it only takes 1 second to load (on dev), I think I have what I'd call a fix for this.

It seems pretty good to me, nice work with this!


Fix EnumBaseConverter

Reference Issue

What does this implement/fix? Explain your changes.

When EnumBaseConverter is using directly it fails as its abstract class and does not have default empty constructor:
https://github.com/vvvv/SVG/search?q=EnumBaseConverter&unscoped_q=EnumBaseConverter
Any other comments?

@wieslawsoltes

https://github.com/vvvv/SVG/search?q=EnumBaseConverter&unscoped_q=EnumBaseConverter

In Filter Effects, you use base class directly. . .
(I wanted to add a test, too.)
This class seems to be designed to be inherited.
(Base is included in name, etc.)
@mrbean-bremen
How much do we want to change ?

@wieslawsoltes
SvgBlendMode is kebab-case.
https://developer.mozilla.org/en-US/docs/Web/CSS/blend-mode

I will  update PR with proper converters for each enum and leave the base class abstract.

Do you continue to follow current enum converter design or not ?

I haven't been following closely lately due to other tasks, though I think you (@H1Gdev and @wieslawsoltes) can handle this just fine together. I'll probably invest more time into the lib some time later... I'm also waiting for @tebjan to do the transition of the library, and after that probably have a core dev group with both of you included.

@H1Gdev Updated the PR with the feedback notes. Ready for another review.

And please update title.

And please update title.

Changed

I think this is ready to be merged.


check_parent performs a recursive search, but it shouldn't ?

The tool moves and rearranges files from only the current directory, yet the check_parent function will look for the .videolog file in the current working directory and all all of its sub-directories recursively. This is what os.walk does in src/video_logging.py :
for root, dirs, files in os.walk("./"):
        if ".videolog" in files:
            raise SudoException()

I do not think that there is an equivalent to os.listdir but for files in python. One way to circumvent this is to take advantage of the fact that os.walk returns a generator object (and not a list). This means that instead of getting all the files recursively at once, os.walk (very probably) gets those in the current working directory, then those in the first sub-directory, etc...
Hence, one way to only get the files in the current working directory without going through every sub-directory would be something like :
for root, dirs, files in os.walk("./"):
        if root == '.':
            # Do your stuff
            break
        # else continue

Since the top directory is likely the first on to be walked through, this shouldn't be too inefficient.
That should also solve the issue mentioned in src/data.json in the sudo section :

[B]y default, the tool checks if the current directory contains the 'video-logging' folder in its arborescence by looking for the '.videolog' file. This process is usually quite fast but it may take a while when dealing with big directories.

On most modern file systems, I think that file look-ups in a given directory should be of almost constant time (correct me if I am wrong). Hence this issue is probably linked with the os.walk thing.

Yes, I did want to check the current working directory and each of its subdirectories, and not only the cwd. Unless I didn't get your point? 🧐
Actually, as the folder command does not perform a recursive sort of the files and only moves the files that are in the current working directory, sorting a directory that contains .videolog could have 2 different consequences:

if cwd is video-logging directly, cli.py and src/ would be put in separated folders and thus one could no longer use the tool after the folder command;
if cwd contains the video-logging folder, then the whole folder will be moved and the tool will still be functional, but in another location on the computer.

So in a way, it may be useless to check all of the subdirectories, but I chose to do this so that the project folder doesn't move.
But I'm thinking about dropping the recursive check and only reading the content of the current directory; some people may want to sort their repos folder, which may contain this one.
I was thinking about using
os.path.isfile(".videolog")

for this.
Anyway, what do you think about this method, i.e. having an empty file that is only used for checking if we are in a specific directory? Another idea - that could be worse - would be to keep in memory the path in which the script was executed, and to compare it with the cwd each time.

Yes, we are talking about the same thing :smile:
I feel that my initial issue lacked some important information and might as such have been confusing :dizzy:, allow me to correct that :wink:
In the current implementation, you have chosen to trade usability (the tool takes time to perform the os.walk out of the box) for safety (the tool will not break itself out of the box) :balance_scale:
But, as you might have noticed, not many tools do that:exclamation:

Python lets you delete the python interpreter :snake:
Windows lets you delete registry keys :wink:

In fact, almost no tool will check whether what you are doing makes sense, so long as it can perform what you requested:

You can open and edit a video with a text editor :pencil: There is little point in doing so (from a human perspective) but for as far as the editor is concerned this is perfectly valid manipulation :heavy_check_mark:
Windows will let you move :arrow_right: its internal libraries (even if that will break the OS on reboot) because it can.

There is no deep philosophical reason for that, it is a limitation inherited from earlier days of computer science (see the C programming language) :laughing: This is sometimes (rarely in fact) referred to as a best effort policy, i.e. the tool will try to perform what the human requested to the fullest of its abilities.

This is opposed to dynamically deterministic tools that twill always produce a reliable result, but might restrain themselves. A python interpreter :snake: that would only execute a script if it terminates would be an example of this: it cannot execute all scripts, but if it does run one then we have a reliable guarantee that the execution will eventually end.
If you are interested in that sort of things I would advise you to investigate functional programming with languages such as Haskell or Rust.

Historically, the reason why most tools didn't do that sort of verification is because it would have been to complex (or impossible). But the whole programming environment evolved to account for this weakness :bulb:, and in particular the tools :wrench: (applications, utilities) are never stored along user data :book: in the computer.
For instance on Windows, tools :wrench: can be found in C:\Program Files\ or C:\Program Files(x64)\ (for system-wide installs) or C:\Users\<username>\AppData\<some more things> (for user-specific tools). On the other hand, user data :book: would be found in C:\Users\<username>\Documents for instance.
Thus, my issue with this trade off is that you are giving up on usability (slower on large directories out of the box) to account for an edge case that should never happen. Plus it is kind of surprising when you read it :smile:
With that being said, there is at least one notable exception to this: when you are debugging the tool, you will almost always do this next to its source code. Thus the problem you are tackling is a real issue for development, and I guess you wrote it because you had some misfortune in the first place :wink:
But as such I believe this verification could be opt-in, and not opt-out as it is now. So instead of asking the tool to remove this safeguard :cop: with "sudo", you could instead add a debug flag that would raise it. That is just an example of how to do that, probably you will find a more suitable way.
About how the verification should be performed, I think that all the ways you have mentioned and tried are valid, although I like the one where you dynamically find the path of the executable :heart: more, because it should have no false-positive :heavy_check_mark:


add more documentation about endpoints

done in https://github.com/stac-utils/titiler-pgstac/commit/aa80c21d0c9e58fcad754669ead63414f1b89269


[BUG] Background For 2015 Escape Menu Black

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[ ] My answer given in the checkbox above is a lie.

What problem did you encounter?

I Have No Idea How To Make This Work Normally, Moderators, Please Fix This Or Help Me

The Background is black please help me

i am unable to adjust the background transparency

Upload your configuration please: https://github.com/pizzaboxer/bloxstrap/wiki/Uploading-your-Bloxstrap-configuration

I can't adjust the background transparency
On Wed, Aug 7, 2024, 12:26 PM pizzaboxer @.***> wrote:

Upload your configuration please:
https://github.com/pizzaboxer/bloxstrap/wiki/Uploading-your-Bloxstrap-configuration
—
Reply to this email directly, view it on GitHub
https://github.com/pizzaboxer/bloxstrap/issues/2540#issuecomment-2274193846,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKHVCIUZNNDNLGK7C2WQ67TZQJYE7AVCNFSM6AAAAABLZHQF3SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZUGE4TGOBUGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can you please do what I've asked?

Yes, I can. I Can't change the background transparency
On Tue, Aug 13, 2024, 2:07 PM pizzaboxer @.***> wrote:

Can you please do what I've asked?
—
Reply to this email directly, view it on GitHub
https://github.com/pizzaboxer/bloxstrap/issues/2540#issuecomment-2287138965,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKHVCIUIIFEHSTVVBHBDZJTZRJYPNAVCNFSM6AAAAABLZHQF3SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBXGEZTQOJWGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Stop replying by email and look at the GitHub thread so you can properly see what I am asking you to do.
https://github.com/pizzaboxer/bloxstrap/issues/2540
Upload your configuration please: https://github.com/pizzaboxer/bloxstrap/wiki/Uploading-your-Bloxstrap-configuration

Closed due to inactivity from OP.


Update README.md

Uncomment no_solo_prefix option in example since it's more useful to have enabled
Uncomment global option in example but keep it disabled; also add inline comment explaining when to set to true
Format filepaths and usage options
Add instructions to "unpair"

:telescope: anyone out there?
If this PR isn't considered useful I'm happy to close it, just let me know.

Sorry - I think generally this repo isn't actively monitored. Thanks for the pull request! Merging. :)


isAnimatedFeature.js

Coverage remained the same at 100.0% when pulling 798026eb9a537c2c722741fdf98956aaaf553801 on Oleg-docent:master into b9e9f298a9ca84ba3be5b3a2da3727d96f584258 on ben-eb:master.

@Oleg-docent Read about git rebase and commits squashing to not recreate PRs again and again.

Hi Bogdan)I know. Just created a new PR mechanically)
08 авг 2016 г. 0:00 пользователь "Bogdan Chadkin"<EMAIL_ADDRESS>написал:
@Oleg-docent https://github.com/Oleg-docent Read about git rebase and
commits squashing to not recreate PRs again and again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ben-eb/css-values/pull/13#issuecomment-238108197, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AK-3zKPJn4ei7tM9MVf9M_z7PUCyMmXrks5qdkeGgaJpZM4JekE_
.

Coverage remained the same at 100.0% when pulling 1766e3e4e1fd6997af635468063519c095751566 on Oleg-docent:master into b9e9f298a9ca84ba3be5b3a2da3727d96f584258 on ben-eb:master.


Does not reflect changes on the fly - React Native

Issue Type: Bug
This is a serious bug after last update.
The react native app is connected to packager on ip:8081.
After saving the changes in code it displays refreshing caption on phone but does not actually update/reflect the change. Also, logs old values on console.
It needs to always restart the app on phone or emulator to get the changes. Which is absolutely not desired.
VS Code version: Code 1.60.0 (e7d7e9a9348e6a8cc8c03f877d39cb72e5dfb1ff, 2021-09-01T10:43:02.136Z)
OS version: Linux x64 5.11.0-34-generic
Restricted Mode: No

Looks like issue with extension that provide RN functionality or watchers.
Could you specify how exactly it's related to VS Code editor itself? Does it update files on disk when they are saved?

I have been working on development as usual, in between came this vs code update and then this happened. As I said, it requires to restart the app to see the code changes, so yes it does update file on disk. Will try to find out the cause and get back. Thanks.

Looks like issue with extension that provide RN functionality or watchers.
Could you specify how exactly it's related to VS Code editor itself? Does it update files on disk when they are saved?

Lets close this issue. Now it's working as expected.


Want to add Ad blocker extension

Hello @ridsuteri,
I want to add an ad blocker extension to this repository.
Thanks and regards,
Anvesha Jain,
A GSSOC Contributor.

@anveshajain19 go ahead , I am assigning you the issue :)

Okay, Thankyou.
I will start my work from today itself :)

hello ,i am so much interested  in your project, and i have  familiar with Html,css and java script so so can you please assign to  me this project.

@Khushbu-kumari-87090 issues are assigned on first comes first served basis :)

Hi there,
I'm a GSSOC22 contributor and I would like to contribute to this issue.
If the issue is still not completed then please assign it to me.

@anveshajain19 since your latest PR is closed, do you want to give it another try or should I assign it to someone else?

@Khushbu-kumari-87090 or @HarshKumarChoudary are you interested in taking this issue? If yes then please keep in mind not to submit a PR copied from here

I could work on it. Please assign it to me.

ok @HarshKumarChoudary go ahead, I am assigning you the issue now

any updates @HarshKumarChoudary ?

I got busy with exams. Will do it within two days

Can I work on this Issue ?? please assign me then.

ok go on @Sulagna-Dutta-Roy, I am assigning this to you


whereEqualTo Pointer returns no results

DB :
https://i.ibb.co/RQsn3CG/Untitled.jpg
when i try to query with

var query = QueryBuilder<SubscriptionParse>(SubscriptionParse())
  ..whereEqualTo('operator', 'E5um4KqOMD');
var response = await query.query();

i get no result found ,, shouldn't i get 5 results ?
SubscriptionParse Class :
import 'dart:core';

import 'package:AppName/screens/home/tele-services.dart';
import 'package:flutter/material.dart';
import 'package:parse_server_sdk/parse.dart';

class SubscriptionParse extends ParseObject implements ParseCloneable {
  SubscriptionParse() : super(_keyTableName);
  SubscriptionParse.clone() : this();

  @override
  clone(Map map) => SubscriptionParse.clone()..fromJson(map);

  static const String _keyTableName = 'Subscription';
  static const String _keyObjectID = 'objectId';
  static const String _keyOperator = 'operator';
  static const String _keyName = 'name';
  static const String _keyDescription = 'description';
  static const String _keyLogo = 'logo';

  String get name => get<String>(_keyName);
  set name(String name) => set<String>(_keyName, name);

  String get description => get<String>(_keyDescription);
  set description(String name) => set<String>(_keyDescription, description);

  String get logo => get<String>(_keyLogo);
  set logo(String logo) => set<String>(_keyLogo, logo);

  String get objectId => get<String>(_keyObjectID);
  set objectId(String objectId) => set<String>(_keyObjectID, objectId);

  String get operatorID => get(_keyOperator).toString();
  set operatorID(String operator) => set<String>(_keyOperator, operator);
}

I've encountered the same issue when trying to query with pointers.  I used the parse console to test out some queries and this is what works ok in the console :
where={"owner": {"__type": "Pointer","className":"_User", "objectId": "vKstDRHhfr"}}
Comparing this to what is happening in the parse server sdk, it appears that because the value is being encoded into a string,  in ParseQuery @ line 233 it then adds additional quotes around the value
var value = convertValueToCorrectType(columnAndValue.value);
So it ends up as where={"owner": "{"__type": "Pointer","className":"_User", "objectId": "vKstDRHhfr"}"}
I changed the code at line 233 to :
var value;
if (columnAndValue.value is String && columnAndValue.value.contains("__type")) {
value = columnAndValue.value;
} else {
value = convertValueToCorrectType(columnAndValue.value);
}
and this made the query work ok. So I think that is the issue.
Hope that helps, and many thanks for developing the SDK, a great job !

Noticed another minor thing, when objects are being returned from queries the className field is being set to "className" instead of the object type.  It's being caused when the parse objects are cloned, in ParseObject the className field is always set to 'ClassName' when cloned.
class ParseObject extends ParseBase implements ParseCloneable {
ParseObject.clone(String className) : this**('className'**);
Not sure if this was meant to be this way for some other reason, but if the className is passed through it fixes this.
class ParseObject extends ParseBase implements ParseCloneable {
ParseObject.clone(String className) : this(className);
Only minor things, otherwise it's working really nicely !

@CodeGenie1
It's working. You can test with last release.
     final localUser = await ParseUser.currentUser();
      var queryBuilder = QueryBuilder<ParseObject>(ParseObject('Product'))
        ..whereEqualTo("owner", localUser.toPointer());

Thanks @rodrigosmarques.
This was resolved in version 1.0.15.


Support Python 3.7

There are unlikely to be many issues.
Test with Tox.

This issue will be obsolete if/when https://github.com/condense9/hark-lang/issues/23 gets implement.


RunPageView and SubPageView where suggests fields from wrong (parent) table

reopening https://github.com/microsoft/AL/issues/6339
Describe the bug
RunPageView and SubPageView's where clause suggests fields from the table of the parent page when pressing Ctrl+Space or otherwise invoking the Intellisense auto-complete list. This is obviously wrong, since the page being run probably shows a different child table, and we are filtering its fields, not those of the calling parent page. Thus Intellisense suggests wrong fields, and ones that will give syntax errors once accepted...
To Reproduce
Just have an action or part that use RunPageView or SubPageView respectively, and try creating the where clause and using Ctrl+Space to suggest fields within its ().
pageextension 50100 Test extends "Vendor List"
{
    actions
    {
        addlast(navigation)
        {
            action(Payments)
            {
                ApplicationArea = All;
                Caption = 'Payments';
                RunObject = Page "Vendor Ledger Entries";
                RunPageLink = "Vendor No." = field("No.");
                RunPageView=where() // <- Ctrl+Space in here
            }
        }
    }
}

Expected behavior
Suggest fields from the child table to be shown by the triggered page/subpage, not parent.
Screenshots
This shows a list page build on the Vendor table, trying to add an action to show Vendor Ledger Entries of type Payment for the current vendor, and getting suggested to filter on fields from the parent Vendor, when they are actually applied to the child Ledger Entry.

5. Versions:
Name: AL Language
Id: ms-dynamics-smb.al
Description: AL development tools for Dynamics 365 Business Central
Version: 8.3.570094
Publisher: Microsoft
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=ms-dynamics-smb.al

You are right. Makes RunpageView "where expressions" quite useless. Looks like it is not the most used property though, since I guess this bug has been there since its original implementation.
Nevertheless we should fix this asap.


8413.4 Add a status route for RabbitMQ

Shippable/pm#8413
Add a route that returns the status of RabbitMQ

also move the location of rabbitmqadmin

use the adapter to 15672
use node amqp to check 5672
make some simple call to test the credentials


ci: allow skip_vrt to build preview site

Description
When VRT was skipped, the storybook build was not being kicked off.  This attempts to fix that issue.
How and where has this been tested?
Please tag yourself on the tests you've marked complete to confirm the tests have been run by someone other than the author.
Validation steps

Add the skip_vrt label.
Check that the VRT run was skipped.
Check that the publish task runs the Storybook build.
Check that the storybook build is linked in the PR's published docs site.

To-do list

[x] I have read the contribution guidelines.
[x] ✨ This pull request is ready to merge. ✨

https://github.com/adobe/spectrum-css/actions/runs/7877414434/job/21493685185?pr=2516

Blocking #2501


docs: document SPDK dependency

We maintain an spdk branch for use by SPDK, until we have
implementation live migration v2 in both qemu and SPDK.
Signed-off-by: John Levon<EMAIL_ADDRESS>Change-Id: Ie27c4373dbac043395c2719858a4af7b4d77f37c

cc @swapnili

I am not sure how the arch build failure is related to this change?

it's not, arch is perma-broken and should be removed


Missing open source license

The github page for this repository displays no license for the project; also there is no LICENSE file inside the repository itself. An explicit open source license will enable anyone to freely and uniformly use any combination of files from this repository. This will become even more important as we start to collect contributions from various third parties.

Ditto to the above. I'm interested in trying Cryptol out and using these ready-made specs would be quite beneficial, however right now some files don't have a license at all (e.g. HKDF), many say "Use freely for educational purposes", which effectively prevents any production use.
Is there any plan with regard to the license clarification and unification for the whole repo?

Thanks for reminding me of this issue. I'll opt to close this now that #68 has landed, but please re-open this is there is still something unclear license-wise.


fix: return type for findMany* should be non-nullable

fixes prisma-labs/nexus#223

Internal Note: Find out original motivation for behaviour

Hey @sameoldmadness 👋 ,
sorry for getting back so late to you. Thanks a lot for the PR 🙏 . I had to check internally whether we can actually change this, because we once had very important reasons to do it this way. But it turns out that this is not needed anymore. Could you fix the merge conflict please? Then i can merge it right away :-)

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

@mavilein done, thanks.


Filtering places by Opening times

As an unregistered user I want to filter places in the sidebar list by opening times so that I can find the place which is opened in time when unregistered use is serching.
Acceptance criteria:

"Filter by" button in available above the sidebar list with places for Unregistered user.

WHEN an unregistered user clicks "Filter by" button, panel with filtering options is opened.

Panel with filtering contains second option "Is opened now".

WHEN un unregistered user chooses option "Is opened now" AND click "Apply" button, THEN system shall fixes current day and time and display in the sidebar list all places, which are opened from current moment and during the next hour. (For ex. user filters data at 8:34, then system shall display all places, which is opened during 8:34-9:34 current day).

This user story is related to #135

Not relevant.


update transcript endpoint and data response

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 3 (33.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 58.404%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

compass/views/api/student.py
1
3
33.33%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
622

Relevant Lines:
1065

💛  - Coveralls


Checkbox

Almost identical to Radix:
https://www.radix-ui.com/docs/primitives/components/toggle-group

But instead of <Checkbox.Root> it should be .
There should be a native prop on the root that on the web will just output to a plain old web checkbox.

Requirements:

No using Radix libraries, Tamagui has equivalent libraries for most.
Add docs page to the site.
Simple tests would be appreciated.

Assigned to @alitnk

I believe the link in this task is incorrect, should be: https://www.radix-ui.com/docs/primitives/components/checkbox


GUI layout driven flow improvements

Thanks to flayout and Thomas' PR we can now link gdsfactory inside klayout and instantiate all the generic cells.
This allows to convert any code driven gdsfactory PDK into a klayout GUI editable layouts.
(Disclaimer: we still recommend a pure python code driven flow for complex layouts.)

[x] modify Pcell values without getting a strange error due to deleting cells klayout issue
[ ] convert gdspy paths into klayout compatible paths

[x] remember gdsfactory packages path. So it does not ask you for their location everytime
[x] Test on Linux
[x] Test on Windows
[ ] Test on MacOs
[x] show in menu and group all gdsfactory macro menus inside a gdsfactory menu PR
[x] name is automatically computed, instead of entered, so we can live update cell parameters without having to change the name

@thomasdorch
@flaport
@lukasc-ubc

Can someone test it on Windows and MacOs?

It doesn't work on Windows, I'm getting 'circular import' errors when trying to import gdspy.

I cannot get it to work on mac either :/
I would assume I am doing something wrong:

Selecting the python3.9 folder that comes with gdsfactory. When I run import_generic_PCells subsequently I get the following warnings/errors.

I installed ubcpdk in ~/gdsfactory/lib/python3.9/site-packages but it doesn't make a difference.
Probbly I am doing something wrong. Let me know, if you have suggestions.

It seems that somehow the numpy integration wit KLayout got damaged in the process

Try uninstalling the klayout python package in your python environment. That should fix the name gfc is not defined error.

Running into this error on Ubuntu 22 WSL. Not sure if something is wrong with one of the files, or if I did something wrong.

Hi @joamatab,
I think most of the stuff on the checklist are completed by my latest PR, except for test on MacOS and test on Windows. We know it does not work on Windows, but idk if anyone has got it to work on MacOS yet. It wold be nice if someone with access to a MacOS computer can test it with the latest release.

I can do that. Could you tell me exactly what to test?

install new release
the try the gdsfactory menus in layout
or anything else?

Thanks!
1, install new release
2. gf tool install in your terminal
3. Open KLayout -> gdsfactory 6.2.1 -> import generic PCells
4. Open new layout -> try importing random PCells from the gdsfactory library

release: 6.2.3
running gf tool install once everything seems fine
running it again gives:
NotADirectoryError: [Errno 20] Not a directory: '/Users/j.fischbach/.klayout/macros/gdsfactory'

import generic PCells prompts a file selector dialog

An I somehow have duplicate menu entries

After canceling the file dialog I now get
No module named 'flayout'

I think the duplicate must be because the macro files must've been duplicated elsewhere, I will open a PR for this right away. They both should do the same thing.
The dialog box is expected - it wants you to select the directory where the site-packages directory sits in. This site-packages should also have gdsfactory and flayout in it. After the appropriate directory is selected, it should import the PCells automatically, displaying a progress bar.

I think I have selected the wrong path the first time. To reset it I have to delete the config json?

Yes, it should be under ~/.gdsfactory

Hm it was installed...

Ah ok: I had the wrong directory: one level too deep...
now I get
No module named 'gdstk'

even though it is installed

sorry for the trouble

sorry for the trouble

No worries!

It seems that gdstk is not linked to klayout
I tried lygadgets_link gdstk
but it only gave me:
Did not recognize the klayout relevance of /User/<path to gdstk>

Ok now I only have the menu entries once: seemed to have been a double install also in salt.

It must've been from your previous installation of the older version

So I cleaned everything
It still complains about gdstk missing

also in the macro ide:

Are you able to import gdsfactory in python? If not, it must be an issue with the gdstk installation. If yes, then it must be an issue with KLayout's Python similar to this.

Yes I am able to run gdsfactory from my terminal

Then it is the same issue as on Windows KLayout. I am glad we tested this, or else we could've overlooked this!

I might try a different gdstk install: instead of mamba directly pip. give me a sec

Ok, so tried pip and compile from source. Both didn't help...

I think you need to install gdsfactory on your MacOS python
@lukasc-ubc

I managed to install klayout-gui on Windows using @thomasdorch's conda install. I can import gdstk now 🎉:

But, it only is available on python3.11. The issue is gdsfactory does not support python3.11 because one of our dependencies does not support python3.11 https://github.com/lxml/lxml/pull/362 Once they make a new release, we can add support for python3.11, and then get gdsfactory on KLayout working on all 3 OS. @lukasc-ubc

@Jan-David-Black can you try this on MacOS too, and see if gdstk can be imported there too?

I think you need to install gdsfactory on your MacOS python

I seem to not fully understand: do you mean system level python? Should it not work with conda environments?

@Jan-David-Black can you try this on MacOS too, and see if gdstk can be imported there too?
I will. (I'll edit as soon as it's done)

Due to relatively poor adaption of 3.11 so far I would prefer having the opportunity to stay with 3.10. Which will be possible as soon as @thomasdorch's build is up.

Can you folks help me get this running on Mac OSX?  Below is what I tried, and the error.    It would be helpful if there was a document that had the installation instructions.    Including which versions, which tools, etc.
Thank you!
Lukas

Installed v6.2.5 with the 1 GB installer.
Installed in ~/gdsfactory
~/gdsfactory/bin/conda update GDSfactory
Downloaded KLayout, “Anaconda3-environment based” LW 0.27.12

Copy and rename klayout to klayout-0.27.12-macOS-BigSur-1-qt5Ana3-Rana3Pana3.app

Copy and open the KLayoutAna3.app using Automator, and edit: export PYTHONHOME=/Users/lukasc/gdsfactory, and myklayout=/Applications/klayout-0.27.12-macOS-BigSur-1-qt5Ana3-Rana3Pana3.app

Can't save, error: The document “KLayoutAna3” could not be saved. A code signing error occurred.  /Applications/KLayoutAna3.app.sb-39bc65bd-FHY3tL: replacing existing signature
/Applications/KLayoutAna3.app.sb-39bc65bd-FHY3tL: unsealed contents present in the bundle root

Manually running in the terminal:

export PYTHONHOME=/Users/lukasc/gdsfactory

myklayout=/Applications/klayout-0.27.12-macOS-BigSur-1-qt5Ana3-Rana3Pana3.app

myconfig=$HOME/.klayout/klayoutrc

open -n -a $myklayout --args -e -c $myconfig -style=fusion $@

Errors: Termination Reason:    Namespace DYLD, Code 1 Library missing
Library not loaded: @rpath/libpython3.8.dylib
Referenced from: /Applications/klayout-0.27.12-macOS-BigSur-1-qt5Ana3-Rana3Pana3.app/Contents/MacOS/klayout
Reason: tried: '/Applications/klayout-0.27.12-macOS-BigSur-1-qt5Ana3-Rana3Pana3.app/Contents/MacOS/../Frameworks/libpython3.8.dylib' (no such file), '/Applications/klayout-0.27.12-macOS-BigSur-1-qt5Ana3-Rana3Pana3.app/Contents/MacOS/../Frameworks/libpython3.8.dylib' (no such file), '/Applications/klayout-0.27.12-macOS-BigSur-1-qt5Ana3-Rana3Pana3.app/Contents/MacOS/../Frameworks/libpython3.8.dylib' (no such file), '/Applications/anaconda3/lib/libpython3.8.dylib' (no such file), '/Applications/klayout-0.27.12-macOS-BigSur-1-qt5Ana3-Rana3Pana3.app/Contents/MacOS/../Frameworks/libpython3.8.dylib' (no such file), '/Applications/klayout-0.27.12-macOS-BigSur-1-qt5Ana3-Rana3Pana3.app/Contents/MacOS/../Frameworks/libpython3.8.dylib' (no such file), '/Applications/klayout-0.27.12-macOS-BigSur-1-qt5Ana3-Rana3Pana3.app/Contents/MacOS/../Frameworks/libpython3.8.dyli

Hi Prof. Chrostowski @lukasc-ubc,
By "Anconda3-environment based", do you mean the conda install by Thomas Dorch that I mentioned? Afaik, that is only meant for Windows as of now.

Hi Skandan,
any changes I make on the Pcell I get a NoneType error
does it work well on your end?

Hi @joamatab,
It works well on my end, could you post a screenshot, or the error message so I can debug it?

yes,
https://github.com/gdsfactory/gdsfactory/pull/957
whenever i change any parameter this is what happens

@nikosavola
I think you had a way of making this work


Update an existing toast?

Is there any way to get a reference to an existing toast if visible and update it directly?
If someone saves several times, I'd love to show some kind of feedback in the existing toast instead of showing 4 in a row of the same thing.

In general, if it's possible in react-hot-toast, it should be possible here. Each of the toast functions returns a toast ID that you can use to update or dismiss it. You should be able to do this (straight from the react-hot-toast docs):
const toastId = toast.loading('Loading...');

// ...

toast.success('This worked', {
  id: toastId,
});

Let me know if that doesn't work for you!


Text selection with handler

Hi,
I saw in your image demo in the very end that you are able to select the text using the handler....
https://camo.githubusercontent.com/9f55e388c704bff5bc49499edf06b156b160cbfc/68747470733a2f2f6d656469612e67697068792e636f6d2f6d656469612f336f366e554f316c574d6b657948356e66572f67697068792e676966
How you do it? right now I'm trying but I can't select the text inside.
Thanks

By default a draggable element text is not selectable (depends on the browser you are using).
Simply add user-select: auto to the css of the element

If further help is needed please reopen the issue


Added smart import shortcuts for some submodules

As observed in https://github.com/vanna-ai/vanna/pull/259#issuecomment-2027242687, some classes like Ollama can not be accessed as wanted.
Currently, to access it, the user would need to write:
from vanna.ollama.ollama import Ollama

But the ideal behaviour is:
from vanna.ollama import Ollama

This PR adds support to allow both (with and without smart import shortcut).
I added the same to some other relevant submodules such as Mistral, Anthropic, OpenAI, ZhipuAI.

EDIT: I also now added the same to the vector DB classes, such as MarqoDB, VannaDB, and ChromaDB.

Would you be open to some modifications to tests/test_vanna.py to test the imports?
I haven't gotten up to speed with running the vanna build locally but was thinking of something like the following could work?  untested of course but hopefully the spirit comes across lol
import os

from vanna.anthropic.anthropic_chat import Anthropic_Chat
from vanna.openai.openai_chat import OpenAI_Chat
from vanna.remote import VannaDefault
from vanna.vannadb.vannadb_vector import VannaDB_VectorStore

def test_ollama_module():
    try:
        from vanna.ollama import Ollama
    except ImportError:
        assert False, "Failed to import 'Ollama'"
    else:
        assert True, "Successfully imported 'Ollama'"

def test_mistral_module():
    try:
        from vanna.mistral import Mistral
    except ImportError:
        assert False, "Failed to import 'Mistral'"
    else:
        assert True, "Successfully imported 'Mistral'"

..etc
Thanks for jumping on this so quickly! This project is awesome

I could, but it should not be necessary. Both imports, with and without shortcuts should work.
I think there should rather be some simple tests to verify that both imports work, but I did not want to add this into the tests.py.

Would you be open to some modifications to tests/test_vanna.py to test the imports?

I made a PR to address this now: https://github.com/vanna-ai/vanna/pull/323
Thanks for the great suggestion, @skoschik! :]


remove alias Swoosh.Mailer from quote block

having
alias Swoosh.Mailer

inside the quote/2 in the Swoosh.Mailer module is bad.
I have an application called Mailer which have a bunch of submodules, and if I
use Swoosh.Mailer, otp_app: :mailer

I can not alias any of my submodules afterwards
alias Mailer.HTML

Because it is trying to get the Swoosh.Mailer.HTML instead of my Mailer.HTML
** (UndefinedFunctionError) function Swoosh.Mailer.HTML.render/3 is undefined (module Swoosh.Mailer.HTML is not available)

Sorry this would be a breaking change. I cannot merge it in, at least not before 2.0
There is a simple workaround - place your alias above use:
iex(1)> defmodule X.C do
...(1)>   defmacro __using__(_) do
...(1)>     quote do
...(1)>       alias X.C
...(1)>     end
...(1)>   end
...(1)> end
{:module, X.C,
 <<70, 79, 82, 49, 0, 0, 5, 52, 66, 69, 65, 77, 65, 116, 85, 56, 0, 0, 0, 145,
   0, 0, 0, 14, 10, 69, 108, 105, 120, 105, 114, 46, 88, 46, 67, 8, 95, 95, 105,
   110, 102, 111, 95, 95, 10, 97, 116, ...>>, {:__using__, 1}}
iex(2)> defmodule C.D do
...(2)> end
{:module, C.D,
 <<70, 79, 82, 49, 0, 0, 3, 188, 66, 69, 65, 77, 65, 116, 85, 56, 0, 0, 0, 129,
   0, 0, 0, 13, 10, 69, 108, 105, 120, 105, 114, 46, 67, 46, 68, 8, 95, 95, 105,
   110, 102, 111, 95, 95, 10, 97, 116, ...>>, nil}
iex(3)> defmodule M do
...(3)>   alias C.D
...(3)>   IO.inspect D
...(3)>   use X.C
...(3)>   IO.inspect C
...(3)> end
C.D
X.C
{:module, M,
 <<70, 79, 82, 49, 0, 0, 3, 184, 66, 69, 65, 77, 65, 116, 85, 56, 0, 0, 0, 127,
   0, 0, 0, 13, 8, 69, 108, 105, 120, 105, 114, 46, 77, 8, 95, 95, 105, 110,
   102, 111, 95, 95, 10, 97, 116, 116, 114, ...>>, X.C}

It's ok! Yeah I'm currently using this workaround, thanks for the detailed comment by the way.
Let me ask you, but what is the purpose of this alias inside the quote?  I guess I threw it in 3 years ago just so that I didn't have to type Swoosh.Mailer on those lines. Now we look at it, it shouldn't be there. Back then, 🤷♂ I didn't think this through.

 I can't see why this PR would break it by moving the alias out of the quote.

The fact that it was aliased inside the quote, means users could access the alias, which is also why caused the issue to you. And if users have written code that in the same module that utilized the alias, moving it outside the quote means user code would break because of the removal of the alias. It is a breaking change, albeit an easy fix.

Alright, I got you!

Thanks for understanding


PackagesNotFoundError error on Windows

Hi, thanks for making this code publicly available.
Have you tried running this code in Windows? I'm getting PackagesNotFoundError while trying to install the conda packages.

View anaconda log
(base) C:\Users\centr\OneDrive\Documents\GO-SLAM>conda env create -f environment.yaml
Channels:
 - pytorch
 - conda-forge
 - defaults
Platform: win-64
Collecting package metadata (repodata.json): done
Solving environment: failed

PackagesNotFoundError: The following packages are not available from current channels:

  - _libgcc_mutex==0.1=conda_forge
  - _openmp_mutex==4.5=1_llvm
  - brotli==1.0.9=he6710b0_2
  - brotlipy==0.7.0=py37h27cfd23_1003
  - bzip2==1.0.8=h7f98852_4
  - ca-certificates==2022.3.29=h06a4308_0
  - certifi==2021.10.8=py37h06a4308_2
  - cffi==1.15.0=py37hd667e15_1
  - cryptography==36.0.0=py37h9ce1e76_0
  - cudatoolkit==11.3.1=h2bc3f7f_2
  - dbus==1.13.18=hb2f20db_0
  - embree==2.17.7=ha770c72_1
  - expat==2.4.4=h295c915_0
  - ffmpeg==4.3=hf484d3e_0
  - fontconfig==2.13.1=h6c09931_0
  - freetype==2.10.4=h0708190_1
  - giflib==5.2.1=h7b6447c_0
  - glib==2.69.1=h4ff587b_1
  - gmp==6.2.1=h58526e2_0
  - gnutls==3.6.13=h85f3911_1
  - gst-plugins-base==1.14.0=h8213a91_2
  - gstreamer==1.14.0=h28cd5cc_2
  - icu==58.2=he6710b0_3
  - jpeg==9b=h024ee3a_2
  - kiwisolver==1.3.2=py37h295c915_0
  - lame==3.100=h7f98852_1001
  - lcms2==2.12=h3be6417_0
  - ld_impl_linux-64==2.36.1=hea4e1c9_2
  - libffi==3.3=he6710b0_2
  - libgcc-ng==11.2.0=h1d223b6_14
  - libiconv==1.16=h516909a_0
  - libnsl==2.0.0=h7f98852_0
  - libpng==1.6.37=h21135ba_2
  - libstdcxx-ng==11.2.0=he4da1e4_14
  - libtiff==4.2.0=h85742a9_0
  - libuuid==1.0.3=h7f8727e_2
  - libuv==1.43.0=h7f98852_0
  - libwebp==1.2.0=h89dd481_0
  - libwebp-base==1.2.0=h27cfd23_0
  - libxcb==1.14=h7b6447c_0
  - libxml2==2.9.12=h03d6c58_0
  - libzlib==1.2.11=h166bdaf_1014
  - llvm-openmp==13.0.1=he0ac6c6_1
  - lz4-c==1.9.3=h295c915_1
  - matplotlib==3.4.3=py37h06a4308_0
  - matplotlib-base==3.4.3=py37hbbc1b5f_0
  - mkl==2021.4.0=h8d4b97c_729
  - mkl-service==2.4.0=py37h402132d_0
  - mkl_fft==1.3.1=py37h3e078e5_1
  - mkl_random==1.2.2=py37h219a48f_0
  - ncurses==6.3=h9c3ff4c_0
  - nettle==3.6=he412f7d_0
  - ninja==1.10.2=h4bd325d_1
  - numpy-base==1.21.2=py37h79a1101_0
  - openh264==2.1.1=h780b84a_0
  - openssl==1.1.1n=h166bdaf_0
  - pcre==8.45=h295c915_0
  - pillow==9.0.1=py37h22f2fdc_0
  - pyembree==0.1.6=py37h0da4684_1
  - pyqt==5.9.2=py37h05f1152_2
  - python==3.7.11=h12debd9_0
  - pytorch==1.11.0=py3.7_cuda11.3_cudnn8.2.0_0
  - qt==5.9.7=h5867ecd_1
  - readline==8.1=h46c0cb4_0
  - setuptools==61.2.0=py37h89c1867_3
  - sip==4.19.8=py37hf484d3e_0
  - sqlite==3.37.1=h4ff8645_0
  - tbb==2021.5.0=h4bd325d_0
  - tk==8.6.12=h27826a3_0
  - tornado==6.1=py37h27cfd23_0
  - xz==5.2.5=h516909a_1
  - zlib==1.2.11=h166bdaf_1014
  - zstd==1.4.9=haebb681_0

Current channels:

  - https://conda.anaconda.org/pytorch/win-64
  - https://conda.anaconda.org/conda-forge/win-64
  - https://repo.anaconda.com/pkgs/main/win-64
  - https://repo.anaconda.com/pkgs/r/win-64
  - https://repo.anaconda.com/pkgs/msys2/win-64

To search for alternate channels that may provide the conda package you're
looking for, navigate to

    https://anaconda.org

and use the search bar at the top of the page.

Hello, I have also tried to deploy under windows, but it will be very difficult and there will be all kinds of errors, I recommend you to deploy under ubuntu, it will save a lot of time

Thanks for saving me the trouble, I will try it on Ubuntu first.


[HOWTO] Bind evil-substitute to s in normal mode.

What are you trying to achieve?
I want to bind s to evil-substitute which is vim's s key in normal mode.
What have you tried?
(map! :n "s" 'evil-substitute)
(map! :gn "s" 'evil-substitute)
(define-key evil-normal-state-map "s" 'evil-substitute)
(define-key! evil-normal-state-map "s" 'evil-substitute)
(map! "s" 'evil-substitute)
(map! :leader :n "s" 'evil-substitute)

The first four do not work. The fifth one doesn't work, and it breaks "s" inside the M-x prompt. The last one works, however, it is not a simple s bind, it's a <leader>-s bind which is not what I wanted.
Additional information
I'm new to emacs and even more so for doom emacs.  Also, the place where I wrote those previous lines is ~/.doom.d/config.el.
System information

emacs   version    26.3
        features   XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND GPM DBUS GSETTINGS GLIB NOTIFY ACL GNUTLS LIBXML2 FREETYPE M17N_FLT LIBOTF XFT ZLIB TOOLKIT_SCROLL_BARS GTK3 X11 XDBE XIM MODULES THREADS LIBSYSTEMD LCMS2
        build      Aug 29, 2019
        buildopts  (--prefix=/usr --sysconfdir=/etc --libexecdir=/usr/lib --localstatedir=/var --with-x-toolkit=gtk3 --with-xft --with-modules 'CFLAGS=-march=x86-64 -mtune=generic -O2 -pipe -fno-plt' CPPFLAGS=-D_FORTIFY_SOURCE=2 LDFLAGS=-Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now)
        windowsys  x
        daemonp    server-running
doom    version    2.0.9
        build      HEAD -> develop, origin/develop, origin/HEAD e3d7b2662 2020-02-18 00:20:35 -0500
        dir        ~/.doom.d/
system  type       gnu/linux
        config     x86_64-pc-linux-gnu
        shell      /usr/bin/fish
        uname      Linux 5.5.3-zen1-1-zen #1 ZEN SMP PREEMPT Tue, 11 Feb 2020 15:35:45 +0000 x86_64
        path       (~/.ghq/github.com/flutter/flutter/bin ~/go/bin ~/.config/scripts ~/.config/nvm/10.19.0/bin ~/go/bin ~/.fzf/bin ~/.cargo/bin ~/bin ~/.local/bin /usr/local/bin /usr/bin /bin /usr/local/sbin /usr/lib/jvm/default/bin /usr/bin/site_perl /usr/bin/vendor_perl /usr/bin/core_perl /var/lib/snapd/snap/bin /usr/lib/emacs/26.3/x86_64-pc-linux-gnu/)
config  envfile    envvar-file
        elc-files  0
        modules    (:completion company ivy :ui doom doom-dashboard hl-todo modeline nav-flash ophints (popup +all +defaults) vc-gutter vi-tilde-fringe window-select workspaces :editor (evil +everywhere) file-templates fold multiple-cursors rotate-text snippets :emacs dired electric ibuffer vc :checkers syntax :tools (eval +overlay) (lookup +docsets) magit :lang data emacs-lisp markdown (org +dragndrop +present) sh :config (default +bindings +smartparens))
        packages   ((lsp-mode) (rust-mode) (web-mode) (typescript-mode) (typescript-mode) (treemacs) (treemacs-projectile))
        elpa       (n/a)
        unpin      (n/a)

Solved by disabling evil-snipe with
(after! evil-snipe
  (evil-snipe-mode -1))

For posterity, this is documented in the :editor evil module's documentation: https://github.com/hlissner/doom-emacs/blob/develop/modules/editor/evil/README.org#restoring-old-substitution-behavior-on-ss


Clarify allowed settings in Conda configuration file

The docs say you can use a custom conda yaml configuration file to set workspace packages, and refers to the conda documentation.
However, we have noticed that not alle possible settings in the config file are respected, for example I get this:
Unsupported attribute removed from yml file: name
Unsupported attribute removed from yml file: ssl_verify
Unsupported attribute removed from yml file: variables

Please document what settings are supported (and any differences in a DEP enabled workspace)

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

ID: fdb01791-8357-9f89-556b-f63f2e03ea14
Version Independent ID: 3aecbccc-9b23-7d80-7022-2c30af46b59c
Content: Manage Spark pool level libraries for Apache Spark - Azure Synapse Analytics
Content Source: articles/synapse-analytics/spark/apache-spark-manage-pool-packages.md
Service: synapse-analytics
Sub-service: spark
GitHub Login: @shuaijunye
Microsoft Alias: shuaijunye

@w0ut0
Thanks for your feedback! We will investigate and update as appropriate.

@w0ut0
Thank you for bringing this to our attention. You are correct that not all possible settings in the custom conda yaml configuration file are respected in Azure Synapse Analytics workspaces.
The documentation should be updated to reflect this limitation and provide a list of supported settings. I will make sure to pass this feedback along to the appropriate team.
In the meantime, here is a list of supported settings in the custom conda yaml configuration file for Azure Synapse Analytics workspaces:
channels
dependencies
pip
pip.dependencies

@shuaijunye
could you please review this and update as appropriate.


Follow-up of checking against multiple types

improves failure message when testing against multiple types
ignores keyschema when not a mapping
ignores schema when not a sequence
adds a failing test concerning validating dict and list as allowed value
in conjunction with keyschema and schema
some minor code fixes

sadly there's still an issue that is documented as a test. shall i add this documentation?
since i actually don't need this, i will not fix it. unless the universe stops expanding, maybe i find the time then.
also, i seem to miss a point about rebasing. so you should just cherry-pick 875e8f6

i will solve this manually.


Update links in addons.md

Change Huawei-PaaS/CNI-Genie to cni-genie/CNI-Genie
The Huawei-Paas version is forked from cni-genie. We can see This branch is 62 commits behind cni-genie:master. notice in cni-genie/CNI-Genie repo.
Last commit time for Huawei fork was February 2019, while the cni-genie one was October 2021. And cni-genie/CNI-Genie have 460 stars, while huawei-cloudnative/CNI-Genie only have 100.
It's reasonable to keep forked version if it's newer / more popular. But I don't see any reason to use Huawei version here. Not sure why PR #3857 (5 years ago) was added Huawei version instead of original one.
Change to new links

The document of Canal is moved to new website.
The NSX-T page was linked to a pdf file for v2.0, updated to index page instead.

Misc fixes

Add k8snetworkplumbingwg/multus-cni link for Multus.
Make Contrail and OVN4NFV-K8S-Plugin fit current format.

This PR has some overlapping changes with #32937.

Thanks @JohnTitor for letting me know! I've rebased this PR.

For CNI-Genie, it was linked to forked Huawei-PaaS/CNI-Genie repo instead of cni-genie/CNI-Genie.

Why? We should only accept this kind of change where we understand why we are making it.

We can see This branch is 62 commits behind cni-genie:master. notice in cni-genie/CNI-Genie repo. Last commit time was February 2019, while the cni-genie one was October 2021.
And cni-genie/CNI-Genie have 460 stars, while huawei-cloudnative/CNI-Genie only have 100.
It's reasonable to keep forked version if it's newer / more popular. But I don't see any reason to use Huawei version here.

/approve


macOS window represented filename

See https://www.electronjs.org/docs/latest/tutorial/represented-file
My proposition for an API is WindowMac#setRepresentedFilename(String filename, boolean iconOnly). There is an option to let the window determine an appropriate name based on the file (just the filename as far as I can tell), otherwise it just sets the icon and allows expanding the dropdown.

What does iconOnly corresponds to? We’ll probably also need getRepresentedFilename

iconOnly=true

iconOnly=false

It's certainly optional since the user could just set the title themselves. I have not found anything on how it chooses a title, it may always just be the filename.
Good point on a getter.

There is both setRepresentedFilename and setTitleWithRepresentedFilename. The description from the docs for what the title is set to is

Sets a given path as the window’s title, formatting it as a file-system path

I am not sure if there is any case it is not simply the file name or whether that case is important.

I think WindowMac#setRepresentedFilename(String filename) would be enough


Reconsider implementation of resource_destructor()

I find the following source code useless.
    if (info == nullptr) {
        delete info;
    }

:crystal_ball: How should this function implementation evolve further?

Please improve the software another bit.


Fix: Web -Default avatar is changed when we select/unselect a member in Workspace invitation page

Details

Web -Default avatar is changed when we select/unselect a member in Workspace invitation page
Fixed Issues

$ https://github.com/Expensify/App/issues/22085
PROPOSAL: https://github.com/Expensify/App/issues/22085#issuecomment-1620110078
Tests

Go to any Workspace
Click the Invite button in the top header
Type an email or phone number to invite a member that you haven't chatted with before
Verify that every email address should have one unique avatar associated with it
Select a member in the search result list
Unselect the above selecting member
Verify that the avatar is consistent

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Go to any Workspace
Click the Invite button in the top header
Type an email or phone number to invite a member that you haven't chatted with before
Verify that every email address should have one unique avatar associated with it
Select a member in the search result list
Unselect the above selecting member
Verify that the avatar is consistent

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Workspace

https://github.com/Expensify/App/assets/129500732/5e398cda-e71e-4679-b2f8-9ce916afa648

Search

https://github.com/Expensify/App/assets/129500732/3614f448-b602-47d2-a02b-b77284b207eb

Assign task

https://github.com/Expensify/App/assets/129500732/efa2e1f6-f64b-4220-bf24-aef3a83c251a

Mobile Web - Chrome

Workspace

https://github.com/Expensify/App/assets/129500732/c91e0fa0-afd1-4502-a874-3abd84a98453

Search

https://github.com/Expensify/App/assets/129500732/8fa75aa6-0667-42c2-bebb-f01e7034de9a

Assign task

https://github.com/Expensify/App/assets/129500732/f62341f7-312a-4658-a75d-325950002021

Mobile Web - Safari

Workspace

https://github.com/Expensify/App/assets/129500732/c24127da-17e8-4930-85c5-f2c1c000e930

Search

https://github.com/Expensify/App/assets/129500732/ae3c81f0-a7b6-4338-b47b-44df7cbd94b0

Assign task

https://github.com/Expensify/App/assets/129500732/4222920f-9e2e-46d1-9b33-b5f41cd3c427

Desktop

Workspace

https://github.com/Expensify/App/assets/129500732/ba2a34ee-44e6-46ce-b338-f032208fb793

Search

https://github.com/Expensify/App/assets/129500732/5ff73348-112d-4a3d-a38e-579b4d70433a

Assign task

https://github.com/Expensify/App/assets/129500732/6ff90afe-e8f5-4757-a104-44023e1228a3

iOS

Workspace

https://github.com/Expensify/App/assets/129500732/eb1520b0-c42f-46e3-b892-f05d30e1d598

Search

https://github.com/Expensify/App/assets/129500732/c8b6924f-3fa0-4537-ac08-85d6067a6cf9

Assign task

https://github.com/Expensify/App/assets/129500732/eeb62fe5-6553-4788-8bc0-22f5be36f40e

Android

Workspace

https://github.com/Expensify/App/assets/129500732/d6de4d82-76f3-4121-82bf-8bb40893068f

Search

https://github.com/Expensify/App/assets/129500732/bcf58f65-f968-4d9c-916d-148a9a8b3b4c

Assign task

https://github.com/Expensify/App/assets/129500732/e9eecca8-f8e6-4832-af47-01850f51ed2e

Sorry, I missed that previous notification somehow.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/Julesssss in version: 1.3.43-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/Julesssss in version: 1.3.43-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/Julesssss in version: 1.3.44-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.43-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Background playback

I would LOVE if the stream could be played in background while I respond to a mail or check the weather. 👍

the audio stream, of course

for everything else there is https://github.com/cemrich/zapp/issues/60
Also implemented in NewPipe, BTW.

Of course :) Thanks for clarification. I use NewPipe too and would love to have this feature for Zapp too :)

This is a nice feature! Also NewPipe implements this quite nicely. I'll label this for Hacktoberfest as I'll not be able to implement this for quite a while.

This will be implemented for Mediathek shows in the next release. Streams will follow soon.


Update for generate_stacktrace return type change

There has been a new release of compiletest, I will use that instead.


validate node ids when fetching remote manifests

currently there is no validation which lets a malicious node assume the identity of a new node in the network, if they are able to submit their manifest before the victim node.
i was unable to write a test for this, looks like the netlibc changes broke the suite. validated it manually by removing ~/shoggoth/node/dump.rdb and seeing that shoggoth.systems, node1... and node2... were successfully added into my local node.

Fixed the test compilation issues though tests have mostly been broken and WIP.
You mentioned you have a dump.rdb file. I would expect that file to be named node-data.rdb. Can you conirm what name it is?
Will review the PR and merge ASAP.
Thanks for the fix!

You mentioned you have a dump.rdb file. I would expect that file to be named node-data.rdb. Can you confirm what name it is?

This is possibly a future issue to be filed which is unrelated to this one -- I had issues attempting to build shoggoth from source and deviated from standard procedures, that included using an empty redis.conf.

You mentioned you have a dump.rdb file. I would expect that file to be named node-data.rdb. Can you confirm what name it is?

This is possibly a future issue to be filed which is unrelated to this one -- I had issues attempting to build shoggoth from source and deviated from standard procedures, that included using an empty redis.conf.

That's interesting. If you have time, please create an issue so we can fix it.


Open YT Command dialog from UI button

Just discovered this plugin and already love it. Steady progress is encouraging.
The recent addition of executing commands on YT issues using the YT Command dialog is great! Can you also add a button to the panel UI to open the dialog? i had to come here to find out it's available from a keyboard shortcut. i found no other access to it from the panel or menus.
Thanks!

Thank you for suggestion, it sounds like a reasonable and useful proposal to me.
Although the command dialog launcher is also available under "Tools" menu, I would agree that it is not an immediately evident feature.


Multiple vitessce components on same page share state

Describe the bug
When there are multiple vitessce components on the same webpage, the view configs become shared because the useViewConfigStore hook is created globally.
To Reproduce
Steps to reproduce the behavior:

Create two vitessce widgets using vitessce-python: https://github.com/vitessce/vitessce-python/issues/29

OR

Modify src/demo/index.js and demo/index.html to render two Vitessce components on the page, then interact with one.

Expected behavior
We expect separate Vitessce components to use independent zustand stores to manage their state.
Possible solutions
Use zustand createContext to initialize the stores per-component rather than globally

Potentially also related to https://github.com/vitessce/vitessce-python/issues/68


Editing ui someday task display

Will be tested when necessary classes are in place. Travis won't be happy abt this. So ignore him haha

Coverage decreased (-0.7%) to 83.143% when pulling 58b7564d19a6f41194faea5775dbc71554a10317 on editing-ui-someday-task-display into 0ccb0b57cc7c2b29edcad197874e9eb8c6813d36 on master.

Seems like there's a compile error on your last few commits

Seems like there's a compile error on your last few commits


HDWallet: re-add mnemonic validity check

Description
Re-add mnemonic validation checks (partial revert of #1601).  Non-English mnemonics are not supported.
Fixes #1607.
The important changed line: https://github.com/trustwallet/wallet-core/pull/1616/files?file-filters[]=.cpp&file-filters[]=.h&file-filters[]=.swift#diff-70145d862894261986b9cbc1be92f3af9e7cb4aeb7bd5c6790bf8ac4e8138e75L54
Testing instructions
Unit tests, HDWallet.
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

[x] Prefix PR title with [WIP] if necessary.
[x] Add tests to cover changes as needed.
[x] Update documentation as needed.
[x] I have read the guidelines for adding a new blockchain
[x] If there is a related Issue, mention it in the description (e.g. Fixes #<issue_number> ).

Hey there, I think you can safely add support for foreign language mnemonics by doing extra validation to ensure that the string is using canonical mapping.
In my swift code I have a guard ahead of HDWallet creation that first santizes the input using
https://developer.apple.com/documentation/foundation/nsstring/1409474-decomposedstringwithcanonicalmap?language=objc
let mnemonicString = mnemonic.joined(separator: " ").decomposedStringWithCanonicalMapping

This ensures that accents, kanjis, chinese characters all have their unicode normalized before generating the wallets. Although BIP39 has ceased reccommending to use non english mnemonic word lists there are legacy mnemonics out there that have been created with foreign languages, so it would be unfortunate to not support those if we have a safe way of doing the key derivation.
Can we push the canonicalMapping sanititaion into the CPP code in order to keep foreign language support

Non-English mnemonic is a feature not needed by Trust Wallet, and by only a small portion of other projects, so it will not be added (at least for now), sorry.  #1613 is a low-impact compromise, but it was scrapped now too.

The main issue here is that it was supported although indirectly previously as you could pass foreign language mnemonics and generate keys with it safely. So unfortunately this would break forwards compatibility for all projects who rely on this functionality
Even if it was unintended, it should maintain support if possible.
If this check is moved into wallet-core I'll need to maintain a separate fork and merge from upstream which I'd prefer not to do.
It's also a bit unfortunate that I reported a bug/issue and it resulted in API I used to basically be closed off 😅


HHH-18190 - Remove @LazyCollection

HHH-18190 - Remove @LazyCollection

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

Fixed conflict locally and pushed


[YOM-451] [YOM-452] show node label only on hover

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
YOM-452

this issue is done


Letter and Word Spacing Values

Type of installation
Portable (.zip)
Type of reader (if reader related)
epub
Steps to reproduce

Open any epub.
Activate Word Spacing and Letter Spacing.
Change their values.

✔ Expected Behavior

For the value boxes of letter and word spacing to be changed, ranging from 0 to n.nn.
And the value boxes of letter and word spacing to be more spacious as they would follow two digits after period and their current boxes are not legible.

❌ Actual Behavior

Increasing / decreasing the value causes it to go to minus somehow.
Box are not spacious.

https://github.com/mienaiyami/yomikiru/assets/116664905/9c8b7ab7-0a8d-4713-92a3-97978e4bce81

Somehow its looping.
https://github.com/mienaiyami/yomikiru/assets/116664905/65770d5c-1214-455f-8108-75c95d8a312a

Some suggestions.

Line height,

Minimum value = 1.
Value Range = 1 to n.

Example = 1, 2, n.

Increments / Decrements of 1 when using arrows.
As 0 means, the lines of the same paragraph would merge forming a single line, making it unreadable.

For Paragraph Spacing,

Minimum value = 0.
Value Range = 0 to n.

Example = 0, 1, 2, n.

Increments / Decrements of 1 when using arrows.

When setting the limits initially I did consider this same, but later i had to lower them a bit because it was needed.
Increment of 1 is too much in my opinion, after testing and using it for a long time of course.
And in word spacing 0.01 is too small.

There have been cases when I needed word and letter spacing in negative, it totally depends on the font you use. While noone will need it as low as -1 but I have kept it there for specific cases.


RACSignal+Operations: implement non-recursive +combineLatest:.

If this is still of interest to anyone, our fork contains this change and additional bug fixes and performance optimizations.


Typo in TS types on function whereILike

Environment
Knex version: 1.0.1
Database + version: @vscode-sqlite3 5.0.7
OS: Windows
If issue is about TypeScript definitions, tag @lorefnon 👋
Bug

Explain what kind of behaviour you are getting and how you think it should do

The whereILike function is unknown in TS types, instead there is a whereIlike (notice the lowercase l).

Error message

With whereIlike we get an error at execution time : TypeError: db.table(...).whereIlike is not a function
With whereILike we get an error at compile time : Property 'whereILike' does not exist on type 'QueryBuilder<unknown, DeferredKeySelection<unknown, never, false, {}, false, {}, never>[]>'. Did you mean 'whereIlike'?

Reduced test code, for example in https://npm.runkit.com/knex or if it needs real
database connection to MySQL or PostgreSQL, then single file example which initializes
needed data and demonstrates the problem.

import { knex } from "knex";

(async function() {
  const db = knex({
    client: "sqlite",
    connection: ":memory:",
    useNullAsDefault: true
  });

  await db.schema.createTable("test_table", table => {
    table.text("test_column").primary();
  });

  console.log(await db.table("test_table").whereILike("test_column", "test%"));

  await db.destroy();
})();

Fixed here : https://github.com/knex/knex/pull/4941

Released in 1.0.2


[GCE] Adding note on how to create an ILB without blocking ingress controller

This change is

/lgtm

Coverage remained the same at 46.297% when pulling 0df5f79da52151e53ba568762bf4e448d8c8299b on nicksardo:gce-ilb-docs into 067b388d85c806ae5d315a087800d190345b943b on kubernetes:master.


New version: Splashtop.SplashtopBusiness version <IP_ADDRESS>

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi
PackageVersion is  <IP_ADDRESS>
But , Installer file name is Splashtop_Business_Win_INSTALLER_v<IP_ADDRESS>.exe
<IP_ADDRESS> ?
<IP_ADDRESS> ?
which is correct?
thank you.


⚠ dev-deces.matchid.io website has degraded performance

In 6f385e1, dev-deces.matchid.io website (https://dev-deces.matchid.io) experienced degraded performance:

HTTP code: 200
Response time: 4489 ms

Resolved: dev-deces.matchid.io website performance has improved in 8bc3da4 after 6 minutes.


Affective Features code

Hello,
I read this paper recently and wanted to try my hands on it. Thank you for providing the code for the same.
How to obtain the affective features of a custom dataset?
Regards
Abhishek

Hello Abhishek,
Thanks a lot for your question! I have added a set of scripts to generate affective features for 16-joint pose sequences. You can go through compute_features.py to check how the pose sequence is used. You can either modify your pose sequence to match compute_features.py, or modify compute_features.py to match your pose sequence (I would recommend the latter since the former can introduce a lot of unintended noise in computations).

Thank you for providing immediate assistance. I will look into the provided code.
Regards
Abhishek


Use PAT to get higher github rate-limit

Goes from 60/hour to 5000/hour for authenticated sessions.
This should hopefully avoid serious deb-getters (and those testing repeatedly :raised_hands: ) from running into rate-limiting issues too early and often.

Thanks for working on this. Please can you add a few words to README.md about how to user this feature?

I have added a section to the README.  I also spent a day trying to work out why the test workflow is failing (including when essentially the same PR previously passed. (see https://github.com/philclifford/deb-get/actions/runs/2871542877 and friends)
Some of that was what triggered me to work on this (testing failing because all the tests all come from one/very few sources unless you use the provided secret token) .  Found some other things that could do with tweaking along the way, which I'll put in another issue, but could not get their ubuntu-latest (?focal !!) runner to pass. Maybe a chat on discord with @OhMyMndy will bear fruit there.

Somewhere along the line I broke it - I fried my brain with the quoting and inserting
 '--header "Authorization: token $DEBGET_TOKEN"'

into the wget to the api only if set and without  the whole lump rejected as an unknown parameter .... resorting to the array seemed to fix it but now It looks like I've been using old cache entries from previous tests/installs and silent wgets have masked the fact that they're not running at all ....
echoing the array and running it by hand works a treat so it is close, but no ${\""\"'cigar}

replaced by #457


scb: Add a type parameter for contracts

Add a type parameter for the type of contract identifiers to the SCB
In the actual application this is instantiated to ContractExe, ie. the
location of the executable in the file system.
In the tests we can use a type like data TestContracts = Game | Currency which is nicer than working with magic strings.

Contract instances are identified by UUIDs already. This will become clearer in the new change that I'm working on.
What I like about this approach is that the lookup functions in the test case can be total and in theory we don't have to worry about them failing. (Yeah I know it does actually fail for Currency the way it's defined right now, but supporting that contract, and additional hooks in general, is a bigger change which is why I separated this bit out).


URLs in E-Mails Not Working

Hi @clems4ever,
I am experimenting version 3.15.0 of Authelia, and it looks like the URLs embedded in e-mails point to invalid locations: they result in an HTTP status 404.
Sample affected URLs – parameters are truncated:

https://login.example.com/one-time-password-registration;
https://login.example.com/security-key-registration;
https://login.example.com/reset-password.

For your reference, I am using the following Nginx configuration.

This is weird, this URL is produced by https://github.com/clems4ever/authelia/blob/8478216e5d392a379ff1b4a2f22b097aae16d391/server/src/lib/IdentityCheckMiddleware.ts#L118. We clearly see that /#/ is a constant in that URL.
Can the reverse proxy strip it in some way?

It's weird, this URL is normally produced by https://github.com/clems4ever/authelia/blob/8478216e5d392a379ff1b4a2f22b097aae16d391/server/src/lib/IdentityCheckMiddleware.ts#L118.
Can the reverse proxy strip it in some way?

Directly fetching the link received via e-mail without going through any reverse proxy:
$ wget -qSO- 'http://<IP_ADDRESS>:4221/one-time-password-registration?token=Ch0RyTELuPF10Os9uKCPrbrVhYiVjUONmZqj5qkSOqiXiNriWYJP7lD8DTucACf1'
  HTTP/1.1 404 Not Found
  X-Request-Id: 7427a2ca-8c41-4f1b-8fb8-429938937f1e
  X-DNS-Prefetch-Control: off
  X-Frame-Options: SAMEORIGIN
  Strict-Transport-Security: max-age=15552000; includeSubDomains
  X-Download-Options: noopen
  X-Content-Type-Options: nosniff
  X-XSS-Protection: 1; mode=block
  Content-Security-Policy: default-src 'self'
  Content-Type: text/html; charset=utf-8
  Content-Length: 169
  Date: Thu, 09 May 2019 09:32:52 GMT
  Connection: keep-alive

The corresponding log entry is:
May 09 11:32:52 abricot index.js[2417]: debug: date='Thu May 09 2019 11:32:52 GMT+0200 (CEST)' method='GET', path='/one-time-password-registration' requestId='7427a2ca-8c41-4f1b-8fb8-429938937f1e' sessionId='lawNimWYhFU1KChVe6-FpvPS3b3ov-wv' ip='::ffff:<IP_ADDRESS>' message='Headers = {"user-agent":"Wget/1.19.4 (linux-gnu)","accept":"*/*","accept-encoding":"identity","host":"<IP_ADDRESS>:4221","connection":"Keep-Alive"}'

I think I'm getting this too (with 3.15.0 and 3.16.0),
https://undefined/#/one-time-password-registration?token=sbLsKsNesZZoJNDcDode0M9GbEAhe4grH4DDDpMR8EfswqY3eirBeQouxAXHjb4j
If I manually change  undefined to the correct address everything works perfectly.

The X-Forwarded-Host and X-Forwarded-Proto are missing. Those headers needs to be provided in requests reaching Authelia. That's what's used to build the URL.


WeMo PHP soap requests

using this class to see if i can control my wemos with php instead of python2 because python2 libraries are getting really hard to find..... all these wemo repos are sooooo old.....

as far as i can tell it's working... at the very least i'm able to control the wemos with php curl... and i've successfully turned a light on and off using the web interface stuff.


Recommended chunk size

I see the default chunk is 5MB, when working with an rclone buffer of 500MB. What would be the best chunk size
my average size of two libraries are as follows;
AVG MB= 2925.73
AVG MB= 2328.14

This varies depending on your network speed.
For me e.g. I have a 100mbit/s connection. 5MB seems to be perfect for me.
A lower chunk size helps you starting a movie more quickly but could make the movie buffering while it's playing. A bigger chunk size makes the initial load time slower.
If you have a 1gbit/s connection I think you can increase the chunk size to around 20mb, because you will have a much faster download speed, so the movie should start even quickly than with my 100mbit/s connection.
This is just an explanation... you have to find the best settings for your hardware/connection yourself ;)


Updated default.md

Removed not necessary first line.
Updated description of the default value expression to the one used in the C# Operators topic.

@BillWagner tagged as changes-addressed for you to verify your feedback was taken care. Thx!

Thanks for making the update @pkulikov  I'll :shipit: now.
Thanks again.


Registering commands

Do you need to do anything to register the command before running the script? If not, how does the script manage registering?

There's argument options, and functions to do that.
Issues are for bug reports/feature requests/etc about the library's code, for questions and help join the discord server 🙂
Invite is in the readme


docs: annotate

Added description of S.Annotate to README file

Fixed aforementioned issues

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2306

Relevant Lines:
2306

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2306

Relevant Lines:
2306

💛  - Coveralls


Improve UserSession

A few changes to user_session.py following Joel's feedback as well as some refactoring due to other RTL changes

actually, I'm gonna merge this in now because I'm trying to glue some if this together and I need the updated authenticate method


Update YMHC domains

Pull Request Template for .ngo.us Domain Registration

Checklist for Domain Registration Application
1. Zone File Edit

[x] Alphabetical Order: Confirm your domain block is added in alphabetical order by your organization’s name.
[x] Two Comment Lines:

First line: Full organization name.
Second line: Organizational contact email (must be different from the .ngo.us domain applied for).

[x] Nameserver Entries:

At least two NS (Nameserver) records with a TTL value of 86400.
Correct format for the domain block (see README for examples).

Example:
; Happy Cat Foundation
; Submitted by networking team<EMAIL_ADDRESS>happycat 86400 IN NS ns1.example.com.
happycat 86400 IN NS ns2.example.com.

2. PR Description
a. Intended Use of Domain

School phobia project websites. YMHC is a not-for-profit, charitable community-based organization, offering essential, accessible, and meaningful resources to support the most important role of all –encouraging the healthy growth of young people. YMHC provides the tools needed to meet the challenges of education, health, and life.

b. Organization Online Presence URLs for Verification

As previously discussed

c. Proof of Domain Mention

As previously discussed

3. Agreement to Terms

By submitting this pull request, I agree to the following:

My organization is a non-commercial entity that is not affiliated with any government body.
My organization operates legally and in good faith according to the stated purpose.
I have provided accurate and truthful information throughout this application.
Other terms outlined in the Registry's Terms of Service.

4. Final Checklist Before Submitting

[x] Zone file edited correctly, following format and alphabetical order.
[x] At least two valid nameservers with correct TTL (86400).
[x] Domain block includes required comments with organization name and contact email.
[x] Intended use of the domain clearly explained.
[x] URLs for verification included with the exact domain name mentioned on the public profile.
[x] Agreement to terms signed.

Legacy domain.
All checks have passed.


Terry update basic interaction diagram

Created a new version of the whiteboard basic interaction diagram

This is cool! ReserveAssetCredit was renamed to ReserveAssetDeposit AFAICT, though. We should probably adjust it?

Current version of the image for quick reference:
https://github.com/paritytech/xcm-format/blob/d673c5eab2eb476d8378b8f5fdf24948dee8d65c/XCM_basic_interaction_diagram.png