[Cardv2] forward ref to inner element so it can be focused

Detailed description

Describe in detail the issue you're having. Is this a feature request (new component, new icon), a bug, or a general issue?

I'm implementing a grid of expandable cards (similar to the Orient Me homepage), and want to be able to easily reference each card's root element so I can focus them.

Is this issue related to a specific component?

Cardv2

What did you expect to happen? What happened instead? What would you like to see changed?

ref is a special property in React, if you pass a ref to a react component, React uses it internally instead of passing it as a prop to the component itself (much like key in elements rendered in an array). ref also acts differently when passed to a plain html element like a span than when passed to a react component like Cardv2: the latter happens to be mostly useless in this case, I'd like the ref to be passed through to the appropriate dom element that the card renders

What browser are you working in?

What version of the Carbon Design System and carbon-addons-ics are you using?

What offering/product do you work on? Any pressing ship or release dates we should be aware of?

Connections Community Catalog https://github.ibm.com/connections-incubator/connections-catalog-components
Steps to reproduce the issue

Step one
Step two
Step three
etc.

Additional information

Screenshots or code
Notes

Add labels
Please choose the appropriate label(s) from our existing label list to ensure that your issue is properly categorized. This will help us to better understand and address your issue.

@j1mie is this a reasonable thing to add? adding it as a higher-order component would allow the same functionality to be included in other components without much headache


RYA-51 Temporal Indexing mongo support

Added support for mongo backed rya temporal indexing.
Added integration tests
Repackaged mongo secondary indexing
Cleaned up the storage strategies.
Indexing is now consistent across all secondary indexers (mongo only).
renamed StatementContraints to be spelled correctly.
Jira

hey andrew can you fix the merge conflicts?  Then I will review and likely merge.

ok other than needing to put in the delete call in the dao, it looks good.  please adjust your eclipse settings so you're not auto-adding the final statements


Create 0000-01-02-JanBrzez.md

I created a branch, created a file and made a commit, and opened a pull request.

Why?

I need more info.


[Antrea] Disable UDP tunnel offload option

What this PR does / why we need it
Latest Antrea has a configMap enabling/disabling the tunnel offload configuration, this PR adds a setting enabling
configuration the data via YTT variables.
Details for the Release Notes (PLEASE PROVIDE)
NONE

Which issue(s) this PR fixes
Fixes: #
Describe testing done for PR
Unit tests for render added in the PR

/lgtm
/approve


Update GraphQL Cycle Detection, Use TypeInfo

The GraphQL cycle detection code should be re-written to use the TypeInfo data structure instead. This way it can be easily ported into new client languages, and no longer relies on the GraphQL AST.

Also see #509

Relatedly, @Niraj-Kamdar has pointed out that allowing GraphQL cycles can make additional data structures possible, such as linked lists, trees, and graphs.

This has been fixed in this PR @Niraj-Kamdar https://github.com/polywrap/monorepo/pull/685


Introduce StampedLock maps and make FakeValuesService thread safe

The PR assumes that most of the time there is no writes to the maps and only reads.
In this case there could be used optimistic locks most of the time and cheap StampedLock 's
based on measurements there is overhead only about 10% for some of methods, for others there is no noticeable difference
e.g.
Datafaker_KotlinFakerBenchmark.kotlinFakerTest
before PR
Benchmark                                       Mode  Cnt    Score   Error  Units
Datafaker_KotlinFakerBenchmark.kotlinFakerTest  avgt   10  390.813 ± 2.512  ms/op

after (10% slower)
Benchmark                                       Mode  Cnt    Score    Error  Units
Datafaker_KotlinFakerBenchmark.kotlinFakerTest  avgt   10  431.599 ± 13.887  ms/op

DatafakerSimpleMethods.firstname
before
Benchmark                              Mode  Cnt      Score     Error   Units
DatafakerSimpleMethods.firstname      thrpt   10   6016.154 ± 247.178  ops/ms

after (about 10% slower)
Benchmark                              Mode  Cnt      Score      Error   Units
DatafakerSimpleMethods.firstname      thrpt   10   5330.902 ±   67.988  ops/ms

DatafakerSimpleMethods.fullname
before
Benchmark                              Mode  Cnt      Score     Error   Units
DatafakerSimpleMethods.fullname       thrpt   10   2524.751 ±  26.856  ops/ms

after (about 10% slower)
Benchmark                              Mode  Cnt      Score      Error   Units
DatafakerSimpleMethods.fullname       thrpt   10   2235.404 ±   67.011  ops/ms

Codecov Report

Merging #770 (c06466e) into main (8cbb8b9) will decrease coverage by 0.56%.
The diff coverage is 69.36%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main     #770      +/-   ##
============================================
- Coverage     92.86%   92.31%   -0.56%
- Complexity     2633     2646      +13
============================================
  Files           284      285       +1
  Lines          5202     5269      +67
  Branches        537      544       +7
============================================
+ Hits           4831     4864      +33
- Misses          243      273      +30
- Partials        128      132       +4

Impacted Files
Coverage Δ

.../net/datafaker/internal/helper/StampedLockMap.java
59.32% <59.32%> (ø)

.../java/net/datafaker/service/FakeValuesService.java
82.85% <80.39%> (-1.12%)
:arrow_down:

...net/datafaker/internal/helper/SingletonLocale.java
76.19% <100.00%> (+1.19%)
:arrow_up:

... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

that's still a tricky part to create a 100% reproducible test for failing non thread safe  FakeValuesService...
will see this week if i able to make it or not

Would this be something @rwong-gw could maybe help validate?

For the past couple days, I have been using CI to run my Gatling tests every half hour with this version, and everything looks good, with no hanging or deadlocks.
One suggestion: StampedLockMap does not implement several methods in Java's Map API, and just throws an UnsupportedOperationException for them. This causes it to interact poorly with debuggers (see attached image from IntelliJ). You should implement at least the size and entrySet methods to get it to work nicely with IntelliJ, and it probably wouldn't hurt to implement at least the remaining non-mutating operations.

makes sense, added implementation

Thanks, I can now drill down into StampedLockMaps in IntelliJ's debugger:

BTW, you might want to add @Override annotations to the methods putIfAbsent, computeIfAbsent, and merge.

Funny thing is that I finally adapted the test from #759 attached by @rwong-gw
And once I decreased sleep time it starts failing with IllegalMonitorStateException since it can not unlock locks ...
At the same time another option I had in mind was CopyOnWriteMaps (COWMap) which for some reason is not provided by java core.
So I replaced it with COWMap and the problem has gone.
Another good thing that it is a bit faster than with stamplockmaps (almost same speed as with unsafe option).
Probably at the beginning it could consume a bit more memory.
@kingthorin so at least one test is added
@rwong-gw could you please once again double check your case if it is still ok? I'm asking since as mentioned above I replaced it with COWMap because of IllegalMonitorStateException

@rwong-gw could you please once again double check your case if it is still ok? I'm asking since as mentioned above I replaced it with COWMap because of IllegalMonitorStateException

@snuyanzin I'll give it a try.

@rwong-gw could you please once again double check your case if it is still ok? I'm asking since as mentioned above I replaced it with COWMap because of IllegalMonitorStateException

@snuyanzin I'll give it a try.


Druid连接池初始化时支持对驱动类遍历处理，避免ExceptionSorter空的问题

问题说明：

当Driver类如果是用户自定义实现时，由于内部equals判断导致exceptionSorter为空
此问题会导致错误的连接被返回到连接池，从而引不期望的错误

问题Fixed：

遍历驱动类，与期望驱动类比较，通过此方式来设置exceptionSorter

request代码

类路径：

修改类： com.alibaba.druid.pool.DruidDataSource
测试类： com.alibaba.druid.bvt.pool.DruidDataSourceTest_exceptionSorter_extend

 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.萧赢 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Win 1809 supports process isolation mode for Win Srv 2019

Win 1809 supports process isolation mode for Win Srv 2019
Look: https://github.com/moby/moby/pull/38000

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

ID: c47037c4-4b2e-b41c-64a3-d82ec79c18c4
Version Independent ID: 947b1d95-3cd5-53d4-042f-71afe8bb3bdf
Content: Windows Container Version Compatibility
Content Source: virtualization/windowscontainers/deploy-containers/version-compatibility.md
GitHub Login: @PatrickLang

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


SDR loss sensitive to nan

Hello,
I'm trying to use SI-SDR and/or SD-SDR loss for a model for music source separation.
I'm working with the well-known open-unmix model (https://github.com/sigsep/open-unmix-pytorch).
The original model is as follows:

Xmag = magnitude spectrogram of input waveform (mixed song)
Ymag_hat = network prediction of magnitude spectrogram of the source being separated (one of drums, bass, vocals, other)
Ymag = magnitude spectrogram of ground truth of the source being separated
Loss = MSE(Ymag_hat, Ymag)

I wanted to see if I could see any differences by using SDR (which is actually the real evaluation metric of the full source separation task) within the training loop:

X = complex spectrogram of input waveform (mixed song)
Xmag = magnitude(X)
Xphase = phase(X)
Ymag_hat = network prediction of magnitude spectrogram of source being separated
Ycomplex_hat = Ymag_hat * Xphase (combine source magnitude + mix phase for source complex spectrogram)
y_hat = istft(Ycomplex_hat)
Loss = auraloss.SISDR(y_hat, y), loss on SDR of waveforms

In the first iteration, the network's prediction is so bad that in a few places, the SDR value is NaN. After this, the gradients get in a bad state and the next prediction from the network is entirely nan.
Here's some print statements I inserted in the body of the SI-SDR code to help pinpoint the issue.  I tried "torch.nan_to_num" (without any arguments, so using the default substitutions: https://pytorch.org/docs/stable/generated/torch.nan_to_num.html), and it's basically the exact same behavior (except the loss is 0 instead of nan).

Hey, thanks for checking out auraloss!
I have seen this issue before, I traced it to an instability issue in the computation of SI-SDR. My guess is that this is caused when the target contains values of all zeros. Here is a minimal code example to demonstrate that:
import torch
import auraloss

x = torch.rand(4, 2, 100)
y = torch.zeros(4, 2, 100)

sisdr_loss = auraloss.time.SISDRLoss()

print(sisdr_loss(x, y))  # nan
print(sisdr_loss(y, x))  # 80 dB

When the target y is a vector of zeros we get a NaN for the loss. We would expect the loss to be symmetric, so setting x or y as the target should produce the same 80dB error (The error is bounded to 80dB due to the choice of eps=1e-8).
Ideally you would not have a vector of all zeros when training, but the code ought to handle this situation. I have made a quick fix for this. You can try it by installing the latest auraloss version through the GitHub repo.
pip install git+https://github.com/csteinmetz1/auraloss.git
Let me know if this works for you. I will push a new release shortly afterwards.

Thanks for the reply - I'll be able to test this within a few days (waiting for a replacement GPU, currently).

I'm definitely having a better time with it @csteinmetz1 :

I installed the latest from github like you suggested.

Great! Thanks for testing it out.


feat: Qdrant sparse vector retriever

Description
This PR intends to add support for Qdrant's new sparse vector retrieval by introducing a new retriever class, QdrantSparseVectorRetriever.
Necessary usage docs and integration tests have been added for the retriever.

We did have a thought about this. But that would require a major refactor of the vectorstore implementation and would be challenging to maintain backward compatibility.
As a separate retriever, that won't be a problem and we could also support hybrid searches using the sparse and dense vectors via the ensemble retriever.

We did have a thought about this. But that would require a major refactor of the vectorstore implementation and would be challenging to maintain backward compatibility.
As a separate retriever, that won't be a problem and we can support hybrid searches using the sparse and dense vectors via the ensemble retriever.

We did have a thought about this. But that would require a major refactor of the vectorstore implementation and would be challenging to maintain backward compatibility.
As a separate retriever, that won't be a problem and we can support hybrid searches uaing the ensemble retriever.

We did have a thought about this. But that would require a major refactor of the vectorstore implementation and would be challenging to maintain backward compatibility.
As a separate retriever, that won't be a problem and we can support hybrid searches using the ensemble retriever.

We did have a thought about this. But it would require a major refactor of the vectorstore implementation and we could run into backward compatibility issues.
As a separate retriever, that won't be a problem and we can support hybrid searches using the ensemble retriever.

I've removed the re-export.


About lw in SphereFace2

The lw in SphereFace2 didn't appear in the paper, what does it meaning?

same question here
this line
Dose multiplying a constant and dividing a constant make a difference to the optimization problem?
Thanks a lot

I think the author use 'lw' to increase the gradient.

The author answered this question at here
https://github.com/ydwen/opensphere/issues/18#issuecomment-1508452996, lw is short for loss weight.
I guess it is used to increase the gradient, as @tangzhongliang explained.


File Get Blank

VSCode Version: 1.36
OS Version:Ubuntu 16.04

Steps to Reproduce:
1.Open GIT Conflict  file.
2.Solve it and save. Full file get blank

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

@zem2464 since you reported that the issue does not occur when all extensions are disabled I suggest you re-enable extensions one at a time (or half of them at a time) to discover which extension is causing your problem.

@zem2464 Can you reproduce with code --disable-extensions?

@joaomoreno getting the same issue reported by @zem2464  after reproduce with code --disable-extensions.

@krunal9421 Can you share a screen recording?


non-C.H.U.M. chum items are no-pickpocket

From information on the wiki, as well as my own testing via autumnaton
https://kol.coldfront.net/thekolwiki/index.php/C._H._U._M.

Aren't they actually conditional (based on the image of the C.H.U.M. - i.e. holding a knife or holding a lantern)? That by itself makes them no pickpocket.
It also means that since there are 5 images, if the lantern (1/5 images) and knife (3/5 images) are base 1% each, the knife is effectively 0.6% and the lantern is effectively 0.2%.

Awesome, just checked with Grey Goose to confirm that all the other items aren't duped by drones either.
(I think this is the best way to determine both conditional and no-pickpocket. Autumnaton will get items that are conditional, but not nopp. Goose will only copy non-conditional items)
I'm not 100% sure of the syntax for items that have multiple types, I don't seem to see others that have multiple so maybe this is not the correct way for me to identify them. According to CF40 on the ASS discord they are seperate : here


fix(#2018) added exit code in build.js file

Added process.exit(1) to build.js catch block.  Fixes issue #2018

@dwieeb  you all marked this as closed and I'm still seeing the issue. It's changed now so when a build fials its actually now returning -bash:
echo $0
-bash

@ncapito During ionic cordova build ?

@dwieeb  Yes.  ionic cordova build browser --release --prod and it echo $0 returns -bash  . Build ios does the same.

Yes another example of ionic developers closing tickets without fixing them. Shame.

Yes another example of ionic developers closing tickets without fixing them. Shame.

@loban Please keep comments constructive. Review our Code of Conduct. This PR and its corresponding issue were closed automatically by a script. This PR is for the CLI v2 code base, which is no longer maintained. If the issue is reoccurring in the latest CLI version, please create a new issue with a recent ionic info as I invited anyone to do so in my automatic reply.


Health Check dont invalidate node

Do you want to request a feature or report a bug?
Bug

What did you do?
Create 3VM, 1 Swarm Master 2 Worker.
deploy a stack:
docker stack deploy swarm -c docker-stack.yml

curl the swarm:
watch -n 0,5 "curl -m 1 http://whoami.example.com/"

What did you expect to see?
Traefik dropping the backends not in its net since it can not connect to them.
What did you see instead?
Timeout on frontend since backed is not accessible.
Output of traefik version: (What version of Traefik are you using?)

{
 "LifeCycle": {
  "RequestAcceptGraceTimeout": "0s",
  "GraceTimeOut": "0s"
 },
 "GraceTimeOut": "0s",
 "Debug": false,
 "CheckNewVersion": true,
 "SendAnonymousUsage": false,
 "AccessLogsFile": "",
 "AccessLog": null,
 "TraefikLogsFile": "",
 "TraefikLog": null,
 "LogLevel": "ERROR",
 "EntryPoints": {},
 "Cluster": null,
 "Constraints": [],
 "ACME": null,
 "DefaultEntryPoints": [
  "http"
 ],
 "ProvidersThrottleDuration": "2s",
 "MaxIdleConnsPerHost": 200,
 "IdleTimeout": "0s",
 "InsecureSkipVerify": false,
 "RootCAs": null,
 "Retry": null,
 "HealthCheck": {
  "Interval": "30s"
 },
 "RespondingTimeouts": null,
 "ForwardingTimeouts": null,
 "Web": null,
 "Docker": null,
 "File": null,
 "Marathon": null,
 "Consul": null,
 "ConsulCatalog": null,
 "Etcd": null,
 "Zookeeper": null,
 "Boltdb": null,
 "Kubernetes": null,
 "Mesos": null,
 "Eureka": null,
 "ECS": null,
 "Rancher": null,
 "DynamoDB": null,
 "ServiceFabric": null,
 "Rest": null,
 "API": null,
 "Metrics": null,
 "Ping": null,
 "ConfigFile": "//traefik.toml"
}

compose file:
version:             '3.4'
services:
  traefik:
    image:           'traefik:alpine'
    command:
      - '--api'
      - '--docker'
      - '--docker.swarmmode'
      - '--docker.domain=example.com'
      - '--docker.watch'
      - '--accesslogsfile=/dev/stdout'
      - '--logLevel=DEBUG'
    volumes:
      - '/var/run/docker.sock:/var/run/docker.sock'
      - '/dev/null:/traefik.toml'
    networks:
     - 'swarm'
    ports:
      - '80:80'
    deploy:
      placement:
        constraints:
          - node.role == manager
      labels:
         - 'traefik.frontend.rule=Host:traefik.example.com'
         - 'traefik.port=8080'
         - 'traefik.docker.network=swarm_swarm'

  whoami_swarm:
   image:          'emilevauge/whoami'
   networks:
    - 'swarm'
   deploy:
      labels:
       - 'traefik.backend=whoami'
       - 'traefik.frontend.rule=Host:whoami.example.com'
       - 'traefik.port=80'
       - 'traefik.docker.network=swarm_swarm'
      mode:        replicated
      replicas:    2
      placement:
        constraints:
         - node.role != manager

  whoami_flock:
   image:          'emilevauge/whoami'
   networks:
     - 'flock'
   deploy:
     labels:
      - 'traefik.backend=whoami'
      - 'traefik.frontend.rule=Host:whoami.example.com'
      - 'traefik.port=80'
      - 'traefik.docker.network=swarm_flock'
     mode:        replicated
     replicas:    2
     placement:
       constraints:
        - node.role != manager

networks:
  swarm:
    driver: overlay
    attachable: true
  flock:
    driver: overlay
    attachable: true

If applicable, please paste the log output in debug mode (--debug switch)
time="2018-03-11T00:37:39Z" level=debug msg="vulcand/oxy/forward/http: Round trip: http://<IP_ADDRESS>:80, code: 500, Length: 21, duration: 997.072069ms"
time="2018-03-11T00:37:39Z" level=debug msg="vulcand/oxy/forward/http: completed ServeHttp on request" Request="{"Method":"GET","URL":{"Scheme":"http","Opaque":"","User":null,"Host":"<IP_ADDRESS>:80","Path":"","RawPath":"","ForceQuery":false,"RawQuery":"","Fragment":""},"Proto":"HTTP/1.1","ProtoMajor":1,"ProtoMinor":1,"Header":{"Accept":["*/*"],"User-Agent":["curl/7.55.1"]},"ContentLength":0,"TransferEncoding":null,"Host":"whoami.example.com","Form":null,"PostForm":null,"MultipartForm":null,"Trailer":null,"RemoteAddr":"<IP_ADDRESS>:44234","RequestURI":"/","TLS":null}"
time="2018-03-11T00:37:39Z" level=debug msg="vulcand/oxy/forward: completed ServeHttp on request" Request="{"Method":"GET","URL":{"Scheme":"http","Opaque":"","User":null,"Host":"<IP_ADDRESS>:80","Path":"","RawPath":"","ForceQuery":false,"RawQuery":"","Fragment":""},"Proto":"HTTP/1.1","ProtoMajor":1,"ProtoMinor":1,"Header":{"Accept":["*/*"],"User-Agent":["curl/7.55.1"]},"ContentLength":0,"TransferEncoding":null,"Host":"whoami.example.com","Form":null,"PostForm":null,"MultipartForm":null,"Trailer":null,"RemoteAddr":"<IP_ADDRESS>:44234","RequestURI":"/","TLS":null}"
time="2018-03-11T00:37:39Z" level=debug msg="vulcand/oxy/roundrobin/rr: competed ServeHttp on request" Request="{"Method":"GET","URL":{"Scheme":"","Opaque":"","User":null,"Host":"","Path":"/","RawPath":"","ForceQuery":false,"RawQuery":"","Fragment":""},"Proto":"HTTP/1.1","ProtoMajor":1,"ProtoMinor":1,"Header":{"Accept":["*/*"],"User-Agent":["curl/7.55.1"]},"ContentLength":0,"TransferEncoding":null,"Host":"whoami.example.com","Form":null,"PostForm":null,"MultipartForm":null,"Trailer":null,"RemoteAddr":"<IP_ADDRESS>:44234","RequestURI":"/","TLS":null}"
<IP_ADDRESS> - - [11/Mar/2018:00:37:38 +0000] "GET / HTTP/1.1" 500 21 - "curl/7.55.1" 1094 "Host-whoami-hal-fritz-box-1" "http://<IP_ADDRESS>:80" 997ms
time="2018-03-11T00:37:39Z" level=debug msg="vulcand/oxy/roundrobin/rr: begin ServeHttp on request" Request="{"Method":"GET","URL":{"Scheme":"","Opaque":"","User":null,"Host":"","Path":"/","RawPath":"","ForceQuery":false,"RawQuery":"","Fragment":""},"Proto":"HTTP/1.1","ProtoMajor":1,"ProtoMinor":1,"Header":{"Accept":["*/*"],"User-Agent":["curl/7.55.1"]},"ContentLength":0,"TransferEncoding":null,"Host":"whoami.example.com","Form":null,"PostForm":null,"MultipartForm":null,"Trailer":null,"RemoteAddr":"<IP_ADDRESS>:44238","RequestURI":"/","TLS":null}"
time="2018-03-11T00:37:39Z" level=debug msg="vulcand/oxy/roundrobin/rr: Forwarding this request to URL" Request="{"Method":"GET","URL":{"Scheme":"","Opaque":"","User":null,"Host":"","Path":"/","RawPath":"","ForceQuery":false,"RawQuery":"","Fragment":""},"Proto":"HTTP/1.1","ProtoMajor":1,"ProtoMinor":1,"Header":{"Accept":["*/*"],"User-Agent":["curl/7.55.1"]},"ContentLength":0,"TransferEncoding":null,"Host":"whoami.example.com","Form":null,"PostForm":null,"MultipartForm":null,"Trailer":null,"RemoteAddr":"<IP_ADDRESS>:44238","RequestURI":"/","TLS":null}" ForwardURL=http://<IP_ADDRESS>:80

I believe that Traefik exclusively uses Docker's HEALTHCHECK when determining container state in a Docker Swarm. Containers are added Traefik only when they pass a check and removed when they fail.

From the docs:

A health check can be configured in order to remove a backend from LB rotation as long as it keeps returning HTTP status codes other than 200 OK to HTTP GET requests periodically carried out by Traefik.
The check is defined by a pathappended to the backend URL and an interval (given in a format understood by time.ParseDuration) specifying how often the health check should be executed (the default being 30 seconds). Each backend must respond to the health check within 5 seconds.
By default, the port of the backend server is used, however, this may be overridden.
A recovering backend returning 200 OK responses again is being returned to the LB rotation pool.

You can read it as "default enabled" or "enable if needed" so i tried it with:
[backends]
  [backends.backend-whoami]
    [backends.backend-whoami.healthcheck]
    interval = "5s"

but no Cigar.
I only started with traefik a few days ago so it could just be a misunderstanding on my side or my inability to make sense from the docs.

@Escaflow you need to define a path:
[backends]
  [backends.backend1]
    [backends.backend1.healthcheck]
    path = "/health"
    interval = "10s"

https://docs.traefik.io/basics/#health-check
Without a path the health check doesn't work.

@brodkin Traefik doesn't use Docker's HEALTHCHECK but you can use the command traefik healthcheck in the Docker's HEALTHCHECK

The issue tracker is for reporting bugs and feature requests only.
For end-user related support questions, refer to one of the following:

Stack Overflow using the "traefik" tag
the Traefik community Slack


null extends object inconsistent between ts and d.ts

TypeScript Version:  3.5.3
Search Terms: null extends object
Code
export type SchrodingersCat = null extends object ? 'alive' : 'dead';

Expected behavior:
SchrodingersCat is either 'dead' or 'alive'.
Actual behavior:
If the code is in a file named box.ts, SchrodingersCat is 'dead'. If the code is in a file named box.d.ts, SchrodingersCat is 'alive'.
Playground Link: N/A
Related Issues: N/A

I can not reproduce this behavior with TypeScript 3.5.3 or 3.6.2. It makes no difference whether it's in a .ts file or a .d.ts file. But it does make a difference whether you have strictNullChecks enabled or not.
Can you make sure that strictNullChecks is enabled in both cases?

The behaviour as described by @MartinJohns is correct.
Toggling strictNullChecks in the tsconfig.json compilerOptions  changes the value of null extends object for .ts files, but for .d.ts files null always extends object.

I could only repro this at first, but it turned out that my tsconfig with strictNullChecks on only included my .ts file. It didn't include my d.ts file, which caused the type to be different. Can you check to see if that's what is happening? Once I got both files covered by the same tsconfig, they started behaving the same.

box.ts
In the TS file, SchrodingersCat is always 'dead'.
export type SchrodingersCat = null extends object ? 'alive' : 'dead';

box.d.ts
In a DTS file which has a matching TS file, SchrodingersCat is always 'alive'.
export type SchrodingersCat = null extends object ? 'alive' : 'dead';

box2.d.ts
In a DTS file which has no matching TS file, SchrodingersCat is always 'dead'.
Even creating an empty file named box2.ts, turns SchrodingersCat to 'alive'.
export type SchrodingersCat = null extends object ? 'alive' : 'dead';

package.json
{
  "dependencies": {
    "typescript": "3.6.3"
  }
}

tsconfig.json
Setting "strictNullChecks" to false, turns SchrodingersCat to 'alive' everywhere.
{
  "compilerOptions": {
    "strictNullChecks": true
  }
}

issue.zip

I'm pretty sure that's because when you don't explicitly list files in your tsconfig, and you have box.ts and box.d.ts, only box.ts is part of the compilation.
@sheetalkamat can you confirm?

@sandersn yes. .ts file takes precedence over .d.ts
c:\temp\test4>dir
 Volume in drive C is OSDisk
 Volume Serial Number is F41A-DBC9

 Directory of c:\temp\test4

09/25/2019  09:24 AM    <DIR>          .
09/25/2019  09:24 AM    <DIR>          ..
09/25/2019  09:23 AM                69 box.d.ts
09/25/2019  09:23 AM                69 box.ts
09/25/2019  09:24 AM                69 box2.d.ts
09/25/2019  09:23 AM                51 tsconfig.json
               4 File(s)            258 bytes
               2 Dir(s)  798,106,771,456 bytes free

c:\temp\test4>node c:\TypeScript\built\local\tsc.js --listFiles
c:/TypeScript/built/local/lib.d.ts
c:/TypeScript/built/local/lib.es5.d.ts
c:/TypeScript/built/local/lib.dom.d.ts
c:/TypeScript/built/local/lib.webworker.importscripts.d.ts
c:/TypeScript/built/local/lib.scripthost.d.ts
c:/temp/test4/box.ts
c:/temp/test4/box2.d.ts

um, shouldn’t it be dead everywhere because null doesn’t extend object?

Despite my love for cats, I agree logically this one should be 'dead'.
However, most of the ecosystem is non-strict. By this I mean all type definition files published to npm. This means it could be a massive breaking change. Making the cat always 'alive' would only affect unpublished source code where strictNullChecks is enabled. DefinitelyTyped is probably a good test case to see the real impact of changing such behaviour.
This could also affect the decision to eventually enable strictNullChecks by default per https://github.com/microsoft/TypeScript/issues/54500#issuecomment-1585189794.

@ljharb null: any when strictNullChecks is off; it's only null: null with it on. So the type system doesn't know that null is actually null and gives wrong answers here. Advanced types often don't work as expected with strictNullChecks off.
@remcohaszing I checked this again and the difference in .d.ts files is also whether strictNullChecks is true or false, not whether it's in a .d.ts file or not. I'm going to close this bug.

@sandersn ok, but when strictNullChecks is on, null extends object should be false, right?

Yep, it's 100% "dead" with strictNullChecks on, because null and undefined are not subtypes of object (they are siblings with {} right at the top).


Use {{ bin_dir }}/etcdctl instead hardcoded /usr/bin/etcdctl

We have an inconsistency in symbolic links creation.

https://github.com/kubernetes/contrib/blob/master/ansible/roles/flannel/tasks/github-release.yml#L26

    dest: "{{ bin_dir }}/{{ item }}"

https://github.com/kubernetes/contrib/blob/master/ansible/roles/master/tasks/download_bins.yml#L33

    dest: "{{ bin_dir }}/{{ item }}"

And we have hardcoded bin_dir in etcd-install-github-release.yml:
    dest: /usr/bin/{{ item }}

This change is

@k8s-ci-robot check my CLA

/lgtm
/approve


ruby.rubygems: fix delete binstub lock files

Description of changes
A change introduced with this PR https://github.com/rubygems/rubygems/pull/7797 does not delete the binstub lock files after the binstub file is created.
This change was introduced in rubygems 3.5.15, and this version causes Hydra builds to fail, in particular mastodon.
A resolution is delete these binstub lock files after the binstub file is created to prevent lock files from ending up in the bin folders of the various derivations which will cause the build to fail.
Introduced in nixpkgs with these PRs:

https://github.com/NixOS/nixpkgs/pull/321040
https://github.com/NixOS/nixpkgs/pull/328638

See the failing Hydra build for mastodon: https://hydra.nixos.org/build/267248005

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build ruby.rubygems mastodon


Invoker id assignment path in zookeeper uses wrong id format

Invoker supports using zookeeper for id assignment logic. There it constructs the a path like /invokers/idAssignment/mapping/<invokerName>. Due to. the way name is constructed its coming out to be like /invokers/idAssignment/mapping/Some(<IP_ADDRESS>). Here invoker was launched with ip as the name.

Above is the state as seen via exhibitor. This happens because invokerName below is an Option
https://github.com/apache/incubator-openwhisk/blob/9c05de4494d6edbd198fd494d458b908bbc0cd0a/core/invoker/src/main/scala/whisk/core/invoker/Invoker.scala#L133

yep, by the time we get to line 133, we know that the invokerName is a non-empty String.  it should be unwrapped.  @chetanmeh do you want to do a PR with the fix?


Contract Renewals

Was going to add a commit that improves fetchUsedContracts but I see you are doing something similar in #602 so I'll wait for that. I would personally do both, but I agree that it's mostly useful in fetchUsedContracts so we might be able to drop this in favour of #602 . It's more correct though to do it in both the uploader and the bus I think 🤔


Shutdown the project Wiki

Wiki pages on GitHub contain the outdated information, which is heavily duplicated with README. The documentation should be merged into the in-REPO docs, and then Wiki can be disabled

It can be done once https://github.com/kohsuke/winsw/pull/129 gets intergrated


update accessing-the-api.md

remove invalid description

This change is

@hzxuzhonghu :wave: Thanks for opening this PR. Please check the box to allow edits from maintainers.


how can i add a self-define button on the grid (but  Add ,Delete, Save ,Edit )

how can i add a self-define button on the grid (but  Add ,Delete, Save ,Edit )

Please post only bugs here. All questions to stackoverflow.


Cancel a fortify order by bringing up a window

As seen from the below screenshot.
Pressing the window should cancel the order.
Pressing away from the order should do nothing (and the unit will be fortified in the next turn).

Duplicate of #70


User Password Hashing

i want to use bcrypt library to hash user passwords and avoid any information leak. Please assign me this issue under hacktoberfest2024 and gssoc

It's already being done in the backend @Cgarg9

please assign this issue to me


Provide basic style guide for more efficient contributing experience

This repository would benefit from having a simple style guide or best practices page. For example, lately, I have seen the usage of :::image [Attributes]::: over the traditional markdown image syntax ![Alt Text](Image Path). I'm not familiar with this "new"(?) approach, and I don't believe it is documented anywhere publically, so I continue to use the traditional markdown syntax when contributing. I have seen others do this as well and have it overwritten by an internal Microsoft reviewer.
Example: https://github.com/MicrosoftDocs/windows-itpro-docs/blob/fa22addf4a38b38a79b532d6f0b38c2bd3ff367e/windows/client-management/administrative-tools-in-windows-10.md?plain=1#L29
Providing a very basic style guide stating: images should be added using :::image::: or X over Y or whatever would improve the contributing experience. This would reduce the load on Microsoft reviewers as well, they would have less syntax to correct/overwrite and spend more time ensuring content is correct. It would go a long way in improving efficiency for contributors and Microsoft reviewers as well as provide consistent syntax across documentation.
Please correct me if this is documented somewhere. Thanks!

I found this https://github.com/MicrosoftDocs/memdocs/blob/main/template.md template in another Docs repo, but I'm unsure if this should be used for this repo as well.

The answer is here: https://docs.microsoft.com/en-us/contribute/!


tests fail

Something about icons

Icons have moved. Forgot to change automated testing to reflect that.

yes I figured :)

I think it would be best you Ralf fixed this since you made the icon change.

Hadn't committed it. Fix now in repository.


Consider dropping the IO capability

The IO capability was originally introduced to make signatures more "truthful". It requires any methods that perform I/O directly or indirectly to include IO in scope.
However, after testing this in Bootzooka, I'm not sure it carries its weight. True, the project is small, but even there the necessity to often add using IO was more annoying, than useful. All the places where this was needed where predictable, and it's usually obvious from the method name that I/O is happening anyway.
That's why I propose to remove the feature (along with the compiler plugin) from Ox entirely. Similar functionality might be implemented by a separate project, without the integration e.g. with Ox's streams.

Related to: https://x.com/adamwarski/status/1836713537898430966

If'd you have some opinion regarding this, a 👍 / 👎 would be great - or a comment even more so :)

The correct question in this regard is - what effects are actually worth tracking this way. John De Goes thinks effect tracking is worthless. Martin thinks Async is worth it. My opinion is that for practical purposes well implemented, relatively fast coroutine runtime like Loom or Golang makes it mostly obsolete as to know whether some call will suspend execution brings no value to me - for example: of course grpc call will perform side effects and "suspend" (if that means anything with something like Loom, anyway), that's why I'm performing it!  The whole thing boils down to an information about "this method call might be slow(er)". But so can a call to a SAT solver! IO capability is kinda pointless without referential transparency achieved by suspending monad. There are however things that do seem to be worth tracking - resource scopes - and they do, in fact, require capture checking. I'm not sure this is what was what Martin had in mind - to have a Connection "capability" that is tracked so that it can't be mistakenly exfiltrated and used-after-close.

Yes, I agree about suspensions, IO was designed to track I/O only - so side-effecting operations which might fail.
We do have resource scopes which remain unchanged in how they are tracked


Re-enable group tests 2/?

This is a follow-up PR to #157.
It addresses the following:
PrivateTree refactoring

split secret generating functions
compute commit_secret automatically after secret path generation
remove leaf_secret field
combine generation of path secrets and keypairs
adapt tests

Fixes

Fixes #165 by no longer derinving path secrets from the HPKE private key
Fixes #166 by changing the group API
Fixes #181 by no longer cloning the RatchetTree and only cloning the leaf_secret for KeyPackageBundle

Tests

Add a test for ProposalQueue filtering

@franziskuskiefer I think everything is addressed, can you approve?


Link missing in "interactive AI lab web demo" on "Generate a Responsible AI insights with YAML and Python" page

The goal of this issue is to point out there's a missing link in the RAI demo page link. This link is found at the end of the docs in Generate a Responsible AI insights with YAML and Python.
The missing link can be found in:

Explore the features of the Responsible AI dashboard through this interactive AI lab web demo.
When clicking in the link you get:

We are sorry, the page you requested cannot be found.
The URL may be misspelled or the page you're looking for is no longer available.

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

ID: 5952b4f0-934c-290a-b1a7-3203c814537c
Version Independent ID: 4051b100-a281-723b-8de5-eeca3f9bb274
Content: Generate a Responsible AI insights with YAML and Python - Azure Machine Learning
Content Source: articles/machine-learning/how-to-responsible-ai-insights-sdk-cli.md
Service: machine-learning
Sub-service: enterprise-readiness
GitHub Login: @minthigpen
Microsoft Alias: mithigpe

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

@blclo
Thank you for bringing this to our attention. An interactive AI lab web demo link is working fine Let me know If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


0.5.2 contains break change

I notice the 0.5.2 version remove the ownsplit mod, it breaks program which use 0.5 version io_uring

It is unstable, it may be changed or deleted at any version.

ok but in docs.rs it doesn't mark it is unstable
for now if i want to share the ring like oensplit did, what is the best practice?

submission_shared and completion_shared have been stable, you can refer to ownedsplit.rs code to implement it yourself.


fix: invalid syntax in ServiceMonitor helm template

helm chart version: 1.0.3
I enabled the serviceMonitor in helm values:
metrics:
  serviceMonitor:
    enabled: true

But when I upgrade the helm release it failed due to invalid syntax in servicemonitor template:
Error from server (BadRequest): error when creating "servicemonitor.yaml": ServiceMonitor in version "v1" cannot be handled as a ServiceMonitor: strict decoding error: unknown field "spec.selector.app.kubernetes.io/instance", unknown field "spec.selector.app.kubernetes.io/name"

The ServiceMonitor "release-name-kubelet-csr-approver" is invalid: spec.endpoints[0].port: Invalid value: "integer": spec.endpoints[0].port in body must be of type string: "integer"

good point! I'm not using ServiceMonitor myself so I didn't catch this error

can you test with the new Helm chart (1.0.4) that it works correctly ?

can you test with the new Helm chart (1.0.4) that it works correctly ?

Helm chart 1.0.4 works. I can see the kubelet-csr-approver metrics from Prometheus console.

perfect, I just created a new release!
thanks for your contribution 👍🏼


add a temperature convertor in the basic convertor section.

Is your feature request related to a problem? Please describe.

A temperature convertor from degree celsius to fahrenheit and vice-versa.

Describe the solution you'd like
if a temperature is given in degree celsius we can convert it to fahrenheit and vice versa is also possible.

Please assign this issue to me.

Temperature converter is already there, you can explore the project, or have a look to other Open Issues, even though they labelled as GSSOC, still they are open to work


[Workspace Feeds] Loading Circle Displays Indefinitely When Queuing Offline Card Update

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @joekaufmanexpensify
Slack conversation (hyperlinked to channel name): #quality
Action Performed:
Prerequisite:
Members and company cards added in workspace

Go to staging.new.expensify.com
Choose the workspace from settings
Click company cards >Card Details
Go offline and click update card

Expected Result:
Grey out the "Update card" button offline
Actual Result:
Loading circle is displayed infinitely when you queue an offline card update.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence
https://github.com/user-attachments/assets/9aa1ac19-1211-440a-a152-80217f3bbd9a

View all open jobs on GitHub

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

BZ team member, Callstack will work on this issue, please wait for them to comment before moving the issue forward.

Added to the project

Hey, I'm Agata from Callstack. I would like to work on this issue 🙂

Assigned

PR was merged

🚀

I think these tests will be handled by testing added by @joekaufmanexpensify for the project. I am going to close this as we will handle the payments together later for the project. Thanks for the rest of the checklist


". . .Consider marking event handler as 'passive' to make the page more responsive"

Receiving the following error:
Added non-passive event listener to a scroll-blocking 'touchmove' event. Consider marking event handler as 'passive' to make the page more responsive

From a cursory glance, I'm not sure which items specifically are responsible, but the issue may be related to the following issue report:
https://github.com/leongersen/noUiSlider/issues/785

Is this still an issue?

hello i'm having the same problem, i'm using Coreui Angular


Reduce reported bug count to zero

For more information on this see the link below.
https://github.com/ballerina-platform/ballerina-standard-library/blob/main/docs/library-development-process.md#zero-bugs

Closing as the bug count is zero to the date


Fix off-by-1 buffer overflow in HexToString

Newer dev tools (gcc v13.2.0) add code that detects buffer overflows:
$ ./ch347prog-sram blinky_10.bit
sram
Open On-Chip Debugger 0.12.0+dev-01192-g3a4f445bd-dirty (2024-10-05-19:08)
Licensed under GNU GPL v2
For bug reports, read
	http://openocd.org/doc/doxygen/bugs.html
Info : only one transport option; autoselecting 'jtag'
jtagspi_program
Info : clock speed 10000 kHz
*** buffer overflow detected ***: terminated

The buffer overflow needs to be fixed to be able to program the device through CH347.
Originally, during every iteration 4 characters are written to the output buffer = hex digits (2) + space (1) + NUL (1).
In subsequent iterations the past iteration space and NUL  are overwritten by the current iteration digits.
In the last iteration the NUL lands 1 character past the end of the buffer.
The fix removes the unused/overwritten space.

OK，merged，Thanks


checkbox doesn't show errors

The checkbox should be able to display errors. This will become handy when dealing with the requirement to agree to use terms / agreements for example.

Let me know if you can find the appropriate prop to pass. I don't see one.


cert-manager-plugin should operate at namespace scope only

cert-manager-plugin error messages are variants of the following:
reflector.go:138] sigs.k8s.io/controller-runtime/pkg/cache/internal/informers_map.go:262 Failed to list/watch *v1.Secret, *v1.Certificate,*v1.Issuer is forbidden: User "system:serviceaccount:ns-vcluster:vc-ns-vcluster" cannot list resource "secrets" in API group "" at the cluster scope: Azure does not have opinion for this user.

Hi @stan-yago , thanks for the issue. We are currently moving to a new version of plugins, so if we implement it it will most likely be done there.


drop overload groups from curation if the overloads were manually curated

Bug/issue #, if applicable: rdar://126273913
Summary
To allow for a smoother transition when enabling the overloads feature for the first time, this PR avoids automatically curating an overload group if all its constituent overloads were manually curated. This way, if a documentation author had already tried to manually curate all the symbols in a package or framework, enabling the overloads feature doesn't spuriously add any new symbols that would disrupt that manual curation.
Dependencies
None
Testing
Use this doc bundle to test this feature:
The symbol graph in the bundle was generated from the following code: asdf.docc.zip
The symbol graph contained in this bundle was generated from the following code:
/// This is a cool class.
///
/// ## Topics
///
/// - ``myFunc(param:)-633ch``
/// - ``myFunc(param:)-3u65v``
public class MyClass {
    public func myFunc(param: Int) {}
    public func myFunc(param: String) {}
}

Steps:

swift run docc preview /path/to/asdf.docc (The bundle's Info.plist automatically enables the overloads feature, so the command-line flag is not required.)
Navigate to MyClass.
Ensure that there are two myFunc(param:) methods shown on the page, with specific type signatures in the curated title.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[ n/a ] Updated documentation if necessary

@swift-ci Please test

@swift-ci Please test


Add YAML files to deploy container-provider for cloud volumes

Signed-off-by: Shiva Krishna, Merla<EMAIL_ADDRESS>

LGTM


Naming changes dev

https://github.com/MinaProtocol/mina/pull/14228

!ci-nightly-me

!ci-build-me


update sqlalchemy

Pull Request
Fixes #79
How Has This Been Tested?
ran make lint and fixed errors to pass the tests

[ ] Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

[ ] Yes

Checklist:

[ ] My code follows OCF's coding style guidelines
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

@peterdudfield sir this fails some tests , I ran make lint in forecast_inference and database_cleanup and fixed errors based on error result but for that I have to set some values to None not sure of that and also it passed locally but fails after pushing gives error : The currently activated Python version 3.10.12 is not supported by the project (^3.11). to solve this I changed  some versions to 3.11 still issue persist
If  I am running pytest using poetry run pytest tests it shows timeout no test ran .. please check what changes I should make?

Could you update to the most recent branch, and then the CI will run. I can then have a look at the failing tests

Thank you, ive included these changes in #84. Ill close this for now


Mounting with PJAX and Webpack

Hello, I love your library but I am having trouble getting some thing working.
I am using Barba.js as the PJAX library and ideally I would like to import modules that are on the page, call the init() method and on pageLeave call the detroy() method on the module.
Here is what I have so far:
// This is loaded by Barba.js
export class Components {
    constructor() {
        conditioner.addPlugin({
            moduleSetName: name => `./${name}/index.js`,
            moduleGetConstructor: module => module.default,
            moduleImport: name => import(/* webpackMode: "lazy" */ `${name}`)
        });
    }

   onPageEnter(DOM) {
        // lets go!
        this.hydrate = conditioner.hydrate(DOM);
    }

    onPageLeave() {
      // Need to destroy the loaded components, unless the are on the new page, in which case I need to reinit them with the new el
    }
}

Is there a way to call a Class because when I do it gives me an error saying I can't call it as a function?
export default ExampleModule {
    constructor() { }

    init(slider) {
        // Init Slider
    }

    destroy() {
        // Destroy Slider
    }
}

Some help on this would be amazing!!

This is the default set up.
export default (slider) => {
    // Init Slider

    return () => {
        // Destroy slider
    }
}

You could do this:

class ExampleModule {
    constructor() {
        this.init();
    }

    init(slider) {
        // Init Slider
    }

    destroy() {
        // Destroy Slider
    }
}

export default (slider) => {
    // Init Slider
    const _myModule = new ExampleModule();

    return () => {
        // Destroy slider
        _myModule.destroy();
    }
}

Or override the moduleGetConstructor and moduleGetDestructor hooks.

@rikschennink Thanks for that.
How would I call the destroy() / init() method on pageleave/enter? I don't see any example of that in the docs.

The hydrate method returns an array of modules. You can call destroy on each module when the page is unloaded.

Sorry for the back and forth, and I appreciate you helping me out with this!
on init I do:
init(){
this.hydrate = conditioner.hydrate(DOM);
}

then on destroy, this.hydrate returns an array of modules. do I have to loop over them all and call the destroy method for each?
Is there a way to check if these module are on the new page, without using the hydrate method again? I'm trying not to reimport the JS just reuse the module.
destroy() {
  console.log(this.hydrate);
  // returns [{module}, {module}, {module}]

   this.hydrate.forEach(module => {
       module.destroy()
    });
}

in the case above, the module.destroy() doesn't do anything

The destroy method returns an empty function. Where do I set that?
ƒ () {}

Should use unmount instead => https://codepen.io/rikschennink/pen/bzzbJN?editors=1010
I'll look into this when I have some time

Okay thanks for the help.
Could you point me in the right direction with overriding the moduleGetConstructor and moduleGetDestructor hooks to accept a Class module?

I think something like this, you'd have to set some debug statements to be sure.

conditioner.addPlugin({

  moduleGetConstructor: module => {
    return new module.default();
  },

  moduleGetDestructor: instance => {
    instance.destroy();
  }
});

module returns undefined in moduleGetConstructor for some reason.
This is the class:
Class ExampleModule {
   constructor() {}
   init() {}
   destroy() {}
}

export default ExampleModule;

It took a couple tries but I got a working example: https://codepen.io/rikschennink/pen/wNOvJB?editors=1010

Nice, I had to update moduleGetConstructor to include .default(element) to get it working.
thanks for the help! I think it's working now 🎉
moduleGetConstructor: (module) => {
   return (element) => {
       return new module.default(element);
   };
}

Fantastic! :) If you want you can close the issue 👍

Sorry I'm late to the party.
I wanted to share how I've been handling a similar situation with a "pjax" library of my own.
I have all my modules in a modules folder. There is an index.js file that configures conditioner and exports a hydrate and an unmount function that I call on every page transition. I looks a bit like this:
import * as conditioner from 'conditioner-core'

let boundModules = []

// Webpack
conditioner.addPlugin({
  moduleSetName: name => name,
  moduleGetConstructor: module => module.default,
  moduleImport: name => import(`./${name}`)
})

// other plugins here, removed for brevity

export function hydrate () {
  boundModules = conditioner.hydrate(document.documentElement)
}

export function unmount () {
  boundModules.forEach(x => x.unmount())
}

Thank you @rikschennink for such this amazing library.


any possibility to add email alert function?

Thanks for the nice tool. It's much easier to monitor kafka with this tool. In the meanwhile, I feel that actually monitoring is not enough for kafka, since people normally won't sit there and stare at the statistics all day. So, I wonder if you could add an email alert functionality(or tell me how to do that), which takes a configuration file and send emails when that topic doesn't emit messages for a specified time. In the configuration file, we should be able to configure topics, email addresses and time intervals to check the status. Is that possible for this project? Thanks!

We internally thought about this at Quantifind, and came to the conclusion that we did not want to reinvent the wheel. Our solution is to use an nrpe/nagios plugin that will notify us when the lags of particular queues becomes worrisome or when a queue should be growing but is not.
Check out these nrpe plugins
and Nagios doc on how to set it up.

you could probably implement this with the new report plugin system, check #58.


Added different parameters for ascent and descent rates

It's useful to have different parameters for maximum ascent and descent rates which can then be set individually.

I think it is good. I want to write the same function too.

Rebased and applied.


model duplicate attribute issues [ migrated from xml-database ]

In return.models:
pf_part_vi has two attributes named AppldTESTxAmt that refer to Part VI Line 6c and Part VI Line 11 of the PF respectively.
SkdASpprtdOrgInfrmtn has two attributes named EIN that refer to filer EIN and Part I Line 12g Column (supported org EIN).
I suspect this is actually an issue with the 990 concordance file / variables sheet as the duplicate issue can be found there as well. I wasn't able to get the default models to migrate to MySQL until I changed db_name for one of the two duplicated fields in the variables.csv file that IRSX uses and the corresponding attribute in return.models. The parser runs fine after the fix - I was mostly curious whether you'd encountered this and how you'd addressed it if you had.
(also thanks for this thing it's amazing)

It seems like this is probably a case sensitivity issue. This hasn't been a problem on postgres, but I believe mysql handles differently. There should be a solution.
Here are the var names and the original xpaths--note that these refer to totally different variables. The xpaths are identical except for capitalization. I believe this is "fixed" in the 2017 schemas so the names are more different. But I think this needs a manual fix for the years where we get collisions in case-insensitive contexts.
AppldTESTxAmt | /IRS990PF/ExciseTaxBasedOnInvstIncmGrp/AppliedToESTaxAmt
AppldTEsTxAmt | /IRS990PF/ExciseTaxBasedOnInvstIncmGrp/AppliedToEsTaxAmt

So, is this issue more complicated than applying revised db_names? I've got a version of variables.csv without duplicate field names I can send over. I'd like to use this repo as a submodule on some infrastructure I'm building around our 990 db but I'm concerned about our current field names differing from the upstream docs. Of course, I could always just write some band-aid code for the time being.

Hey @thirawr , send your fix over, simplest thing is to use that. There are a few other minor changes I've been meaning to add as well.

Here it is:
variables-unduped.csv.zip
/IRS990PF/ExciseTaxBasedOnInvstIncmGrp/AppliedToESTaxAmt was renamed to AmtCrdtNxtYr from AppldTESTxAmt
/IRS990ScheduleA/SupportedOrgInformationGrp/EIN was renamed to SpprtdOrgEIN from EIN

Thx @thirawr. Rereading I think I removed the dupes already actually, but I can take yr fix for appliedtotestaxamt. I assume ein has the same name-collision-in-MySQL issue.


How to export to SVG?

It would be great to be able to export the QrCode to SVG.
How would that work?

@flosse For now you could draw a square for every bit of the QrCode... We would add this the next version.

...ok so extending Renderer with a to_svg_string function could be a first approach?

I got stuck with https://github.com/PistonDevelopers/image/issues/563, so maybe you could answer my question?

0.4 refactored the Renderer structure to support SVG export. See https://github.com/kennytm/qrcode-rust#svg-generation for example.


React-query plugin: generate query response types

Is your feature request related to a problem? Please describe.
In my projects, we're generating our TS types using the react-query plugin which is super convenient when it comes to generating the required hooks (queries/mutations). However, we discovered that apollo:codegen is doing a much better job at generating the types.
Example
GraphQL Query
query UserProfile {
  merchantCenter {
    userProfile {
      data {
        email
        firstName
        id
        lastName
      }
    }
  }
}

GraphQL Code Gen with React Query plugin
export type UpdateUserProfileMutation = (
  { __typename?: 'Mutation' }
  & { merchantCenter?: Maybe<(
    { __typename?: 'MerchantCenterMutation' }
    & { updateUserProfile: (
      { __typename?: 'UpdateMerchantUserProfileResponse' }
      & Pick<UpdateMerchantUserProfileResponse, 'errors'>
      & { data?: Maybe<(
        { __typename?: 'MerchantUserProfile' }
        & Pick<MerchantUserProfile, 'email' | 'firstName' | 'id' | 'lastName'>
      )> }
    ) }
  )> }
);

Apollo GraphQL Code Gen
Whereas with apollo:codegen we're getting the following result:
export interface UpdateUserProfile_merchantCenter_updateUserProfile_data {
  __typename: "MerchantUserProfile";
  email: string;
  firstName: string;
  id: string;
  lastName: string;
}

export interface UpdateUserProfile_merchantCenter_updateUserProfile {
  __typename: "UpdateMerchantUserProfileResponse";
  data: UpdateUserProfile_merchantCenter_updateUserProfile_data | null;
  errors: UpdateMerchantUserProfileError[];
}

As you can see from the above example, the last result is much clearer, cleaner and allows us to access the returned type from the query (ie: UpdateUserProfile_merchantCenter_updateUserProfile_data) as this is referenced and not just coded inline like the 1st example with the react-query plugin.
Also, another interesting point is the codegen is generating all possible types whereas apollo:codegen is only generating the types for the declared queries/mutations.
Describe the solution you'd like
We have tried multiple configuration options but all of them are unsatisfactory. We're wondering if there would be other config options to get as close as possible to the apollo generated types.
Describe alternatives you've considered
We're considering to move away from the react-query codegen plugin and just use apollo:codegen. This involves a lot of refactoring and potentially some manual work where individual queries/mutations have to be created manually (not very efficient).

@nextglabs Have you find a workaround? This is creating a big headache combined with DatoCMS schema which defines even mandatory content fields as nullable :persevere:.
Making use of fragments helps a bit but it would help more if the fragments were used on the response itself.

@ristomatti We're still looking for some solutions... Using fragments isn't a bad idea but still doesn't fully achieve what we're looking for 🤔

@ristomatti We're still looking for some solutions... Using fragments isn't a bad idea but still doesn't fully achieve what we're looking for 🤔

For your given use-case, we would recommend using Fragments that would express your data requirements.
We've taken notes of your request for future roadmaps.
Thank you!

@charlypoly Yes, they have indeed been a ton of help. Recently I also discovered inlineFragmentTypes: 'combine' option which could work for us. We've noticed the caveat on the documentation about potential issues with deeply nested fragments. It would be helpful to know how many levels of nesting does the documentation refer to with "deeply nested fragments". E.g. is a fragment within a fragment already "deep" analogously to how nested JS object properties are commonly referred to?

@ristomatti after checking with the team, it actually seems that the modern version of GraphQL Code Generator generates an output similar to what you expect.
If you go to https://www.graphql-code-generator.com/, scroll to the live demo, and select the "Operations types" demo, you will see that the generated code is close to what you want.

@charlypoly Uh sorry! I just found the email notification when going through my flagged emails. I can confirm the output looks exactly what I was after when setting inlineFragmentTypes: 'combine'. :+1:
I had forgotten this completely and we haven't yet experimented with it more than I just tried running it now. With it the fragment types are indeed used as return values. Awesome!


Rename setters/getters/builders for Storage classes to meet proto conventions

This is the first of a series of PRs that update our naming conventions to meet the ones of proto/gRPC/VeneerToolkit.
Changes are:

property(Type property) setters renamed to setProperty(Type property)
property() getters renamed to getProperty()
builder() factory methods renamed to newBuilder()

Previous methods are kept and deprecated for now.

Coverage increased (+0.04%) to 84.744% when pulling ac2921b43324156884edc00bab2c862f709118f4 on mziccard:storage-rename into 88344a2769f2ff44afdaa236763dbc310ac221ac on GoogleCloudPlatform:master.

/cc @lesv

Is that a google standard or a vKit thing?

No Google standard that I know of (Guava in fact uses our current style). As per title, this is used in protos and VKIT. In order to be able to expose protos and VKIT classes without breaking Veneer look-and-feel we need to obey to their conventions on this.

Sigh - the shorter form works better in IDE's.

Coverage decreased (-0.06%) to 84.652% when pulling 6670b707d62856025ef53dc0b2a271c068bf2412 on mziccard:storage-rename into 88344a2769f2ff44afdaa236763dbc310ac221ac on GoogleCloudPlatform:master.

Coverage decreased (-0.06%) to 84.652% when pulling ada166ff4f88717940c48244b1da6166fc13e66c on mziccard:storage-rename into 88344a2769f2ff44afdaa236763dbc310ac221ac on GoogleCloudPlatform:master.

@lesv I added 4 commits that do the following:

Avoid deprecating package private methods, just rename them
Update TESTING.md readme
Update storage usage in ITBigQueryTests
Update google-cloud-nio code to use renamed methods

Coverage decreased (-0.06%) to 84.652% when pulling a0b87e04175e968672f4b3dc7954653fa992a987 on mziccard:storage-rename into 88344a2769f2ff44afdaa236763dbc310ac221ac on GoogleCloudPlatform:master.

LGTM - I won't be using approve / commit again.   Not quite the behavior I was looking for.
BlobInfo seems public not "package private".  But I defer to your judgement.

BlobInfo seems public not "package private". But I defer to your judgement.

Yes classes are public but methods I renamed without deprecating them should all be package-private. If that is not the case please point me to the methods so I can fix it.

Looked closer, never found a public, so I'm happy.
LGTM

Coverage decreased (-0.06%) to 84.652% when pulling ab64fc63d614d50c26b59ee0d053467d58059125 on mziccard:storage-rename into 88344a2769f2ff44afdaa236763dbc310ac221ac on GoogleCloudPlatform:master.

LGTM

Coverage decreased (-0.03%) to 84.56% when pulling fad6e567a37d1caaf01e95c0e66bd294a0c8cc26 on mziccard:storage-rename into 7bcfe3e9f95cc13ed21ecf38c7c3f44ad88cab96 on GoogleCloudPlatform:master.

LGTM


Simplest way to integrate facebook native ads?

What is the best way to integrate native ads?
Can we please have a sample with facebook native ads?
I find it very hard to implement native ads with fast adapter and a sample would be great.

and if we have a sample with facebook native ads, same could be used with Admob native ads in future.

@TheMaxCoder that's a unrelated question.
You simply need to create a item for these. And then you have the usual ViewHolder for them, which will be displayed in the list.

Hi Mike!
I'm sorry if my question seems completely noob but actually I couldn't get my head around it.
I have a working FastAdapter with ItemAdapter for let's say BlueItem
Now I want to add let's say an AdItem (Native Ad)
My question was, which adapter we can use to add these two different items to the list?
I tried using AbstractWraperAdapter but it was very confusing so I thought maybe you can show some simpler way to achieve this?
Thanks
Max

Yeah, it makes sense.
Thank you so much for the explanations :)
Have a great day!


STOMP Plugin is not enabled on RabbitMQ

I've been trying to setup the Spring websocket broker in PCF dev but only the amqp ports seem to work. I need STOMP enabled but from the service JSON it doesn't seem to be there (only amqp/5672 and http to management 15672) Is there a way to manually do this?
Also tried ssh to vagrant and logging in as vcap but couldn't run the rabbitmq-plugin enabled stomp command as the env wasn't set correctly (could find a profile to fix this).

Hi @daveburnsLIT,
STOMP is not enabled by default but you are correct that you can use the rabbitmq-plugins command to enable it from inside of the VM. You can log in as root and run the following:
root@agent-id-pcfdev-0:~# export PATH=/var/vcap/packages/erlang/bin:/var/vcap/packages/rabbitmq-server/bin:$PATH
root@agent-id-pcfdev-0:~# rabbitmq-plugins enable <plugin>

I haven't tested it out, but you'll probably need to enable rabbitmq_stomp and/or rabbitmq_web_stomp.

@daveburnsLIT Did this address your issue?

Closing for now, feel free to re-open if this doesn't fix your issue. We've prioritized a story about enabling default PCF RabbitMQ plugins on boot.


dnstwist: 20240116 -> 20240812

Diff: https://github.com/elceef/dnstwist/compare/refs/tags/20240116...20240812
Changelog: https://github.com/elceef/dnstwist/releases/tag/20240812
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 334196 run on x86_64-linux 1

2 packages built:

dnstwist
dnstwist.dist


 highlight in Atom

Is there any way to highlight GraphQL queries within <page-query> and <static-query> in Atom? It is possible in VS Code with ESLint and Vetur extensions (described here).

For those arriving here from Google: this is currently possible in VS Code by configuring eg.:
"vetur.grammar.customBlocks": {
  "docs": "md",
  "i18n": "json",
  "page-query": "graphql",
  "static-query": "graphql"
}

..and then running the command Vetur: Generate grammar from `vetur.grammar.customBlocks`

@sergeyfilimonov It looks like atom vue is the most common package for adding syntax highighting for Vue in Atom. So assuming you're using that, it doesn't appear that it supports adding custom grammar in the same way Vetur for VS Code does. I think it would make sense to ask for that feature in their repository.


New Generator

Codecov Report
Attention: Patch coverage is 90.00000% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 97.20%. Comparing base (631ac92) to head (4fa2d51).
 Consider uploading reports for the commit e8310b8 to get more accurate results

Files
Patch %
Lines

src/sparkdantic/generation.py
82.60%
4 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##              main     #235      +/-   ##
===========================================
- Coverage   100.00%   97.20%   -2.80%
===========================================
  Files            3        3
  Lines          193      143      -50
===========================================
- Hits           193      139      -54
- Misses           0        4       +4

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


Fixed notification shimmer

Issue Reference
Fixed notification shimmer

@shivendra-webkul
As I have checked and found that it is working fine for LTR and for RTL it is still having issue. Please check the video for reference.
VIdeo 1 - LTR
https://github.com/bagisto/bagisto/assets/92721837/43317769-31c4-4c15-b3c3-b6d7e10950da
VIdeo 1 - RTL
https://github.com/bagisto/bagisto/assets/92721837/5658095a-c342-43cd-8376-ba30e85c2649

Checked and found that the issue has been fixed and working fine for the shimmer effect in RTL and LTR view. Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/1b5c1807-a038-43ef-96d8-55d80a08342e


Update FUNDING.yml

Summary of the PR
A short summary of this PR and what it adds/removes/fixes.
Related issues, Discord discussions, or proposals
Links go here.
Further Comments

/clapass


initial version of training material for small ncRNA clustering using blockclust

Looks much better! Thanks @pavanvidem

thanks a lot @erasche ! There is already a workflow here https://github.com/usegalaxy-eu/workflow-testing/tree/master/blockclust. should I move it under https://github.com/usegalaxy-eu/workflow-testing/tree/master/training/transcriptomics ?

@pavanvidem I'll take care of that, no worries.


BUG: SwapReads Navbar Logo: doesn't switch in Dark Theme

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The SwapReads logo in the Navbar doesn't change while switching the themes, whereas all other components do, making it look odd. A little design edit and change can bring about a better look.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

hey i can fix this issue please assign it to me

Assigned!
Make sure to attach before & after screenshots in your PR. Also, star the repo before working on it. While creating your PR, ensure you pull all existing changes beforehand to avoid conflicts.
Happy open-source contributing!


Websocket connection not done to the right host

Describe the bug
I'm currently trying to use vite to serve assets for a backend generated site.
The site is available on http://localhost:8000
Vite is server on http://localhost:3000
ws is established using ws://localhost:8000 instead of ws://localhost:3000
Reproduction
If you inspect /vite/hmr you find this line :
const socket = new WebSocket(${socketProtocol}://${location.host});
System Info

vite version:
Operating System: any
Node version: any

Suggestion / Solution
Could the websocket path using the port / location from vite dev server ?
In the meantime this issue can be solved using a koah middleware
    app.use(async (ctx, next) => {
      await next();
      if (ctx.url === '/vite/hmr') {
          ctx.response.body = ctx.response.body.replace('${socketProtocol}://${location.host}', '${socketProtocol}://localhost:3000')
      }
    })

Seems to be the same use case for #452?

@yyx990803 the issue is pretty close but a bit different, the goal here is not to run ws on a different port / host but having the connection done to the right host / port. Feel free to merge the issue if you think it's the same problem.

@Grafikart can you see this approach https://github.com/vitejs/vite/pull/918? Thanks!


Problems with definition of "educational organization"

According to the current definition, an educational organization is an organization "whose purpose is to provide training or otherwise facilitate learning or the acquisition of knowledge, Skills, values, beliefs, or habits".
There's a problem here due to the phrase "whose purpose". Is this meant to convey that an educational organization must have a singular purpose, or is it instead intended to be compatible with such an organization having multiple purposes? At the very least, this should be clarified.
Either way of clarifying it might be problematic, however. If it is clarified in such a way that an educational organization must have a singular purpose, then it implies that private, for-profit colleges aren't educational organizations, since they have purposes unrelated to education (e.g., making a profit). And that implication seems incorrect.
On the other hand, if it is clarified in such a way that an educational organization may have multiple purposes, then it implies that certain religious organizations (e.g., the Southern Baptist Convention) are educational organizations, since one of their purposes is to facilitate the acquisition of values, beliefs, or habits (of a religious nature). And, again, that implication seems incorrect (though it may be more palatable than the implication that private, for-profit colleges aren't educational organizations).

@gregfowlerphd You might clarify the meaning by changing "whose purpose" to "whose ostensible purpose". Even for-profit institutions don't advertise they're only in it for the money.

Organizations may have multiple purposes, they may have primary purposes and secondary purposes that they hope to achieve while fulfilling the first. Hence, why we have not create more detailed representations of them.
Commercial Organization is an equivalence class such that other organizations may become commercial organizations if they bear a Commercial Role. So, an Education Organization may or may not bear that Commercial Role, just as a Service Provider like Verizon may bear a commercial role but a Waste Collection Service may not bear that role if it's created as a special non-profit service. So, this solves the commercial purpose issue, but may not be totally satisfactory for religious organizations.

@cameronmore: Yeah, it's clear that the definition of 'organization' doesn't require an organization to have a singular purpose. But that's compatible with the definition of 'educational organization' requiring an educational organization to have a singular purpose. This is why I proposed that the current definition, which uses a phrase ('whose purpose') that might be taken to express such a requirement, be clarified. Perhaps 'whose purpose' could be changed to 'one of whose purposes'?
I agree, though, that this does nothing to resolve the issue that some religious organizations will count as educational organizations.
@swartik: Interesting proposal. I agree it might solve the problem involving private, for-profit colleges. However, I might prefer 'one of whose ostensible purposes' (see above for why), given that it seems that an educational organization might have multiple ostensible purposes.
As far as I can tell, though, the resulting definition would still be subject to the problem involving religious organizations.

Thinking about it more, groups of people and organizations are not artifacts, which we have defined as having clear functions. The notion of purpose is certainly not the same or as strict as the notion of function, and because these agents have agency, they can choose to change the purpose of their organization with no problem (so long as they agree and follow the rules which they themselves set). If we were using the term function here, we might get into difficult waters trying to say that organizations only have one function. But agents are organic and not as rigid, we should extend that same flexibility to the purposes that organizations dedicate themselves to.

@cameronmore:
I agree with what you say about the distinction between purpose and function and about the flexibility of an organization's purposes. So it might be on the right track.
On the other hand, I'm not entirely clear on what you're proposing, so I'm not sure it's on the right track. One thing that gives me pause is when you say: "If we were using the term function here, we might get into difficult waters trying to say that organizations only have one function." I may be misreading you, but it sounds like you're here proposing defining organizations as having only one purpose. If so, I think that's on the wrong track: Doing so would simply have the (untoward) consequence that certain groups of agents we'd typically take to be organizations aren't organizations merely because those groups have more than one purpose.
(FWIW, at this point, my preferred response to the issues I raised about the definition of "educational organization" is to simply change "whose purpose" to "one of whose purposes" (or maybe to "whose purpose or one of whose purposes") so as to clarify that an educational organization needn't have a singular purpose. The resulting definition would still imply that certain religious organizations are educational organizations, but I'm not sure that's so bad.)

I'm only trying to say what you just said,

change "whose purpose" to "one of whose purposes" (or maybe to "whose purpose or one of whose purposes")

I would agree with that for a few of these classes.

@cameronmore: Oh, OK. Yeah, I definitely think that's on the right track, then. :)

"whose established purpose," "whose ostensible purpose,""whose initial purpose," "whose primary purpose," (I think saying "one of whose purposes' would be too weak) or perhaps something else? Or just add an elucidation on the parent class noting that organizations do not have to have only one purpose, or that 'purpose' is not intended to be totally exclusive of other functions?

@cameronmore: One option you didn't consider was 'one of whose primary purposes'. You're probably right that 'one of whose purposes' is too weak, but I'm not sure the same is true of 'one of whose primary purposes'. What do you think?
(I do have some thoughts about the options you did provide, if you don't like the option I mentioned.)

Yeah 'one of whose primary' feels too weak, like it implies that an org may have multiple primary purposes but that conflicts with the notion of primary itself, just a personal intuition

@cameronmore:
First, as far as 'one of whose primary purposes' goes, do you think 'main' or 'major' would work better in place of 'primary'? (I'd also like to push back, gently, against the idea that if something has a primary X, it can have only one primary X. Google reveals many instances of the phrases 'two primary purposes' and 'two primary goals', and dictionary.com illustrates the relevant sense of 'primary' with a case involving multiple primary Xs. That said, I suspect 'main' or 'major' are preferable to ‘primary’ anyway.)
Second, ‘whose established purpose’, ‘whose ostensible purpose’, and ‘whose initial purpose’ all seem to be subject to the same problem as ‘whose purpose’—namely, the resulting definition might mistakenly be taken to imply uniqueness of the relevant sort of purpose. Similarly for ‘whose primary purpose’, if I’m right that an organization can have multiple primary purposes. And if I’m wrong, then a definition with ‘whose primary purpose’ implies (I think mistakenly) that an organization with exactly two equally important purposes cannot be an educational organization.
Third, I think the other option you provided—adding an annotation—might work, but the annotation would need to be phrased and placed with care. The issue isn’t whether organizations are required to have only one purpose. (Given the definition and axioms for organization, it’s clear there’s no such requirement.) Rather, the issue is whether ‘whose purpose’ in the definition for educational organization implies (or might be mistakenly taken to imply) that organizations of that particular sort are subject to an additional such requirement.
(Perhaps an analogy would help here. Parents are not required to be male. But the correct definition for one of the subclasses of parent—namely, father—will imply that parents of that particular sort are so required.)
With this in mind, let’s consider the details of the annotation route. Since the issue is with ‘whose purpose’ and only two subclasses of organization have that phrase in their definitions, it seems preferable to place the annotation on those subclasses rather than on the parent class. As far as what the annotation should say, I think something like this might work for educational organization:

The use of the phrase ‘whose purpose’ in the definition is not intended to imply uniqueness of purpose: An organization of this sort might also have other purposes, including non-educational ones.


Unable to add Discography for Van Halen

Describe the bug
Unable to add discography. Just get the "Something is wrong..." error popup,
To Reproduce
Try to add discoraphy.
Expected behavior
Should add the artists discography to playlist
Desktop (please complete the following information):

OS: Xubuntu 22.04
Browser Firefox
Version 121,01

Went back and without refreshing page, it just worked. Maybe something on spotify's side.


querier: Add /api/v1/labels support

Prometheus in 2.6.0 added support for new method: https://github.com/prometheus/prometheus/pull/4835
We may want to add same method to Thanos Querier to stay up to date with Query API. StoreAPI we have supports this, but most of implementations do not implement this:
func (s *ProxyStore) LabelNames(ctx context.Context, r *storepb.LabelNamesRequest) (
	*storepb.LabelNamesResponse, error,
) {
	return nil, status.Error(codes.Unimplemented, "not implemented")
}

This seems to be useful for Grafana adhoc filters feature: https://github.com/grafana/grafana/issues/6038
Let us know if priority of this needs to be increased, currently it does not seem urgent. Also, help wanted (:
Acceptance Criteria:

Thanos Querier exposes /api/v1/labels that will fanout to store API gRPC labelNames method.
Store gateway, sidecar and ruler implements this method.

This has been added in March so closing! Thanks, @jojohappy ❤


accounts: create manage hooks

So that we can create users for example for inveniomanage, e.g.
inveniomanage user createsuperuser --u eamonn --p mypassword --e<EMAIL_ADDRESS>inveniomanage user createuser --u eamonn --p mypassword --e<EMAIL_ADDRESS>

@eamonnmag yes. please re-open using https://github.com/inveniosoftware/invenio-accounts/issues/new

Ah ok, sorry @jirikuncar, didn't see that!


[Trivial] Reject overlong encodings earlier

Codecov Report

Merging #3134 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #3134      +/-   ##
===========================================
+ Coverage    68.95%   68.95%   +<.01%
===========================================
  Files          688      688
  Lines        55690    55692       +2
===========================================
+ Hits         38403    38405       +2
  Misses       17287    17287

Impacted Files
Coverage Δ

src/ripple/protocol/impl/tokens.cpp
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


strange behavior when unminimizing my terminal

When I minimize my terminal and then unminimize it, I get a very strange configuration of windows.

This doesn't happen all of the time, and seems to happen after I've created a new tab in my terminal by using ctrl+t.
Additionally, when I hit ctrl+t in my terminal, that also resizes my windows in somewhat pathological way:

I'm using plain jane OSX terminal running big sur. Pasting in "About This Mac":

Notably I haven't disabled SIP b/c it's a work machine

Tabs are not supported due to lack of macOS APIs. See #68

Aha, thanks. Interesting API shortcomings.


[Windows] Failing process not reliably reported

Process monitoring on windows VMs does not consistently report failing processes.
If a job launches a powershell script that only contains exit 1, then in a windows deployment:
Linux behaviour (expected):

deployment fails
bosh vms and bosh instances -p will consistently report failing

Actual windows behaviour:

the deployment succeeds
bosh vms and bosh instances -p will alternately report running or failing

Dummy release with linux and windows jobs that only exit 1
winexit-release.zip
Sample manifests for dummy release
linexitman.txt
winexitman.txt

Has something been done in that matter?
I'm experiencing the same exact issue.
Tanzu Ops Manager + BOSH Director version: v2.10.39-build.450
VMware Tanzu Application Service version: v2.11.19
VMware Tanzu Application Service for Windows version: v2.11.11


spread metadata of deduped storage

Currently the deduped storage is storing its metadata on a single storage server (defined as the metadataStorage cluster). This makes the metadata a critical point of failure, as the metadata contains the mapping between hashes and block indices, it would mean that the entire vdisk would effectively be deleted once its metadata storage is unavailable.
Therefore it is proposed to forget about the MetadataStorage as a concept altogether, and spread the metadata in a similar fashion as the data is already spread. It would basically make the meta data storage as done in the LBA code very similar to how it is done in the nondeduped storage code.
The simplest solution would simply be turning LBA's MetaRedisProvider interface into:
// RedisProvider is used by the LBA,
// to retreive a Redis Connection
type RedisProvider interface {
	RedisConnection() (redis.Conn, error)
}

and use that one within the LBA struct, rather than the previously defined MetaRedisProvider. The ardb code which creates the LBA Object will already do the correct thing, as the used provider already provides that function, effectively spreading the data using the simple index-modulo algorithm.
With this change the MetaRedisConnection method of ardb.redisProvider can also be deleted, as wel as any other meta-related content within that struct (e.g.: the metaConnectionConfig property).
NOTE that the zeroctl copy and delete commands will also have to be adapted together with this change (e.g.: functions such as copyDedupedVdisk will have to be adapted to work with the multiple data storage servers, instead of a single metadata storage server).
This feature also means that the validateMetadataStorage check in the config module is now obsolete, and should be deleted and validation code that uses it should be adapted accordingly.
Last but not least, unit test code should be adapted where (and only where) required.

how would this provide redundancy? because also today for data of non deduped volumes there is no redundancy?
would metadata data of LBA's still be grouped? (is it grouped in the first place, with that I means do we store metadata per groups of e.g. 256 LBA blocks, if it is grouped what is group size?)

how would this provide redundancy? because also today for data of non deduped volumes there is no redundancy?

There is also no redundancy directly in the ardb code for deduped code either. In fact as deduped data is shared between multiple disks, a failure is even more damaging.
The redundancy (for those vdisk types which support that: VdiskTypeBoot and VdiskTypeDB) is provided by:

Backup Storage Server, kept in sync by the Tlog Server (planned for milestone 7);
The Tlog Server, which stores all transactions logs using erasure coding and replications (already implemented);

would metadata data of LBA's still be grouped?

Yes it is grouped, they are stored per 128 entries. Docs about it can be found here https://github.com/zero-os/0-Disk/blob/master/docs/nbd/lbalookups.md. And yes they'll remain grouped.
It's good you bring this up though. This will impact how the LBA data will be spread, as a simple index module operator wouldn't really work indeed, if we want to keep them grouped, which I think we should. Details of this will have to be worked out as part of this issue. But I think that would have become clear as an issue anyhow, once work on this issue started.

This idea is now superseded by a new proposal, which can be found as issue #474.


For workload clusters supporting OIDC auth, the web UI should support updating kubeconfig

For workload clusters that do not support OIDC authentication for the Kubernetes API, the web UI supports the creation of key pairs. (Also in the gsctl CLI there is the gsctl create kubeconfig command).
For workload clusters with OIDC enabled, the web UI should support users in updating their kubectl configuration (kubeconfig) accordingly. In the simplest form, this could be done by providing a stand-alone YAML file, or by providing the user/server/context snippets to copy and paste.

For the web UI in order to detect whether a workload cluster is configured for OIDC auth, we have several annotations on the Cluster CR:

oidc.giantswarm.io/client-id
oidc.giantswarm.io/group-claim
oidc.giantswarm.io/issuer-url
oidc.giantswarm.io/username-claim

(Source)
If these annotations were present, a MAPI aware web UI could decide to show the workload cluster as OIDC enabled.
Problem: Currently these annotations are not set on all OIDC enabled workload clusters.
Once we have this information available on the Cluster CR, we can also use it in the web UI in order to either

show the kubectl config ... commands needed in order to configure access or
offer a complete, stand-alone kubeconfig file for download.

there's followup issues of this, e.g. #116


Add support for Polygon

This PR introduces partial support for interacting with contracts on Polygon and pulling metadata.

get_token_uri_from_contract_batch() is currently not supported due to a limitation in web3.py
Additional error handling has been added to pulling.py so it falls back on get_token_uri_from_contract, without failing.
an additional argument -chain has been added to pulling.py to switch to another chain. (defaults to ethereum for backwards compatibility)
A new POLYGON_ENDPOINT has to be registered on Alchemy

Example:
python3 pulling.py -contract 0x82bbf7be0eb9a6024b7a641ba179e00812bdae53 -chain polygon

Closes #33


No module named spacy. Try: pip install -U spacy

I got this when I want to instantiate a Spacy object. But I'm sure I have installed spacy and I can run python -m spacy correctly. Cloud you please provide some possible reasons for this situation？

Hi!
Let me first check, is the issue title exactly matching the error message you are seeing (older versions of spacy-cpp had such message, but recent versions suggests user to use pip3)? If yes, please try latest spacy-cpp.
Can your system access Spacy in Python? Like for example the Python code in https://github.com/d99kris/spacy-cpp#example-usage
If yes, I suspect there could be some mismatch between for which Python version Spacy is installed for, and which Python version you are building spacy-cpp for. By default spacy-cpp's CMakeLists.txt uses Python v3.x, so spacy needs to be installed for Python 3 (e.g. using pip3). Alternatively one can change spacy-cpp's CMakeLists.txt to build for Python 2 (not recommended as Python 2 is EOL) by changing:
find_package(PythonInterp 3 REQUIRED)```
to
```find_package(PythonLibs 2.7 REQUIRED)
find_package(PythonInterp 2.7 REQUIRED)```
and rebuild (maybe delete `build` dir first, just in case).

If still no success, perhaps you can share more system details such as version of OS, Python, CMake, C++ Compiler. Thanks!

I believe the above steps should help resolving the issue, so I will proceed to close this issue. Feel free to re-open it for follow-up questions. Thanks!

I'm getting the same error after building the header only example.
I'm running ./exprog1
And get:
libc++abi: terminating with uncaught exception of type std::runtime_error: No module named spacy. Try: pip3 install -U spacy
Abort trap: 6


The "History" changes features does not work in pheditor.min.php

When you edit a file , as the code describes, a history file is created.
On the main pheditor.php, works as described:
-> a folder is created
-> corespondent file is created
But in the pheditor.min.php, the code does not do anything, not even the newfolder.

The minified version was outdated and now is updated.
File history only keeps the old version of the files before saving new changes.


dashboard/startify: fix invalid reference to "neovimPlugins"

For some reason I've used pkgs.neovimPlugins instead of pkgs.vimPlugins. There is no such thing as pkgs.neovimPlugins.
Closes #208

Merging when the CI passes.