Random 'address already in use' error on restart

If I run: yarn esmon src/api/server.ts
I get random errors when I modify files:
events.js:291
      throw er; // Unhandled 'error' event
      ^

Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:3001
    at Server.setupListenHandle [as _listen2] (net.js:1316:16)
    at listenInCluster (net.js:1364:12)
    at GetAddrInfoReqWrap.doListen [as callback] (net.js:1501:7)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:68:8)
Emitted 'error' event on Server instance at:
    at emitErrorNT (net.js:1343:8)
    at processTicksAndRejections (internal/process/task_queues.js:84:21) {
  code: 'EADDRINUSE',
  errno: 'EADDRINUSE',
  syscall: 'listen',
  address: '<IP_ADDRESS>',
  port: 3001

I wasn't able to reproduce this on a minimal example, and I wasn't able to 'reduce' my code reliably (the error simply appears less frequently). I believe there is a race condition which has something to do with how long does the compilation and/or resource deallocation take. Could it be for instance that you don't actually wait until the 'old' process is fully gone before spawning a new one?
This is macOS 11.3.1, node 16.

If I add 500 ms delay here after cmd.kill():
https://github.com/egoist/esmon/blob/62102236591ff133f6add023175ea68f7f874e47/src/index.ts#L89-L95
then it works fine. I guess you need to call waitpid somehow (if that is possible on Node at all)?

Unfortunately the fix didn't help. Reload worked for the first few times then stopped (probably because everything had been cached and the compilation became too fast). And honestly I don't see how it should have helped - I don't see tree-kill waiting for processes to shut down: https://github.com/pkrumins/node-tree-kill/blob/cb478381547107f5c53362668533f634beff7e6e/index.js#L53

For comparison, estrella watch mode works 100% reliably. I like the simplicity of esmon though so would really prefer you to figure this out ;)

I think that's what does the trick: https://github.com/rsms/estrella/blob/e5b985bd3292ddaa87022b97062be2f18b15b182/src/exec.ts#L477
They attach a listener to p.on("exit", ...) and only proceed after it's called (or after 500ms timeout + SIGKILL).

that sounds like a solution, PR welcome 😄
On Fri, May 28, 2021 at 1:57 AM Ilya Semenov @.***>
wrote:

I think that's what does the trick:
https://github.com/rsms/estrella/blob/e5b985bd3292ddaa87022b97062be2f18b15b182/src/exec.ts#L477
They attach a listener to p.on("exit", ...) and only proceed after it's
called (or after 500ms timeout + SIGKILL).
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/egoist/esmon/issues/5#issuecomment-849827832, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDAWSGJ745WJT66MZR4UYLTP2B2DANCNFSM45RV4WAA
.


Can the canvas operate and display according to the width and height of the loaded image size

Now the width and height of the painter node are consistent, and the read image will be distorted

Yeah I was waiting for someone to bring that up.... The ratio seems not changable atm. locked at 1:1

When adding a bg image, I added a window for agreeing to change the canvas size equal to the image. Until I actually made the canvas in accordance with the aspect ratio, I don’t really understand the work of the resize node. =) But the output image has the correct ratio, it is only in the window that it is not distorted if the height and width are not equal. Sorry for my English.

我不太清楚当我在画布涂抹一个区域后点击生成依然是改变了整张画？

Now I have added that only when loading the backround does the canvas change. But I’m going to add the ability to change the canvas, I’m struggling with the proportions when drawing the canvas.

Thank you, I’ll take a look, and you can move the preview of the mask ;)

Add suppot change canvas size. There may be problems with the size of the node in height, but these are minor things.


[FLINK-30859] Sync kafka related examples from apache/flink:master

There is only one example related to Kafka -- StateMachineExample.

Discussed offline to refactor to remove Kafka dependency


Simple Date rule is not working. Says "TypeError: parseISO is not a function"

rule is simple {date: 'required|date'} and data is {date: '05-12-1999'}
version {    "validatorjs": "^3.18.1" }
I have also tried other formatting like '05/12/1999', but it seems no good. Am I misusing? thanks

Can you please try installing date-fns package directly.  This dependency should have been installed along with validatorJS package.

I was able to reproduce the error. Installing the date-fns package manually did not solve the problem for me.
My versions:
validatorjs: "^3.17.1", installed: 3.17.1
date-fns: "^2.14.0", installed: 2.14.0

@drthoben hmm, ok I will have a look at this further.  This has been a reported issue in the past, but the install of date-fns manually cured the problem.  I won’t be able to look at this until the weekend.

No worries @mikeerickson. Is there anything I can help you with solving the issue?

@drthoben i let you know as soon as it is ready so you can test against your implementation.

Same problem here. Manually installing date-fns didn't solve the issue.
"validatorjs": "^3.18.1"
"date-fns": "^2.14.0"
date format: 2020-07-23
Recently I've updated validators to the latest version because of some issues with translations. I think that date validation worked before the update, but can't be 100% sure about it.
react-dom.production.min.js:21 Uncaught TypeError: parseISO is not a function
    at isValidDate (rules.js:22)
    at Rule.after_or_equal [as fn] (rules.js:379)
    at Rule._apply (rules.js:487)
    at Rule.validate (rules.js:469)
    at eval (validator.js:114)
    at Validator.checkAsync (validator.js:137)
    at Validator.fails (validator.js:506)
    at eval (AddEvent.js:153)
    at Object.da (react-dom.production.min.js:14)
    at ka (react-dom.production.min.js:15)

This issue will be resolved in the upcoming 3.19 release

@multiojuice I have not been able to reproduce this issue locally.  Would you mind create a small test application and either email to me<EMAIL_ADDRESS>or post it on git or somewhere where I can download and try.

@multiojuice Hey there, just pinging you again to see if you can provide the above information.  I am ready to push out a new release, but I want to make sure this issue is resolved as I am not able to reproduce it locally.

This just happened to me. I am using react/mobx/mobx-react-form. I have form with input type date, and rules for that field: 'required|date'. I have set the default value(using moment) and it works as expected, but when I change the input value, then it breaks with the above error.
Here are my deps:
    "mobx": "5.15.4",
    "mobx-react": "6.2.2",
    "mobx-react-form": "2.0.8",
    "moment": "2.27.0",
    "oidc-client": "1.10.1",
    "react": "16.13.1",
    "react-app-polyfill": "1.0.6",
    "react-dev-utils": "10.2.1",
    "react-dom": "16.13.1",
    "react-router-dom": "5.2.0",
    "validatorjs": "3.18.1"

This is a pretty difficult edge case to debug. Since it works locally on new installations (using node)
Any chance you can create a demo which shows the issue?

@mikeerickson I will try tonight

@mikeerickson here repro

@Nodios great, thank you so much I will have a look as soon as possible, likely on Saturday

@mikeerickson Any update on this? I have crated this sandbox with date-fns (latest) manually installed and I had no luck.

@GaborGyorgy Two items

I dont have TypeScript setup locally for immediate testing, thus I will have to task that for another day
Using on a Vue test application, if I use

import * as Validator from 'validatorjs'

All works as expected.  Can you attempt to format as above in your TS application
I will get babel-node setup locally so I can test ES6 import from node, but as stated it works in a VueJS application which is using import format as outlined above.

@GaborGyorgy
OK, I have just created a quick and dirty app using node / babel and have tried your code and all works fine without any modification
After reviewing your codebox, I see you are using some typing file @ v3.15. I am not sure if this is the culprit, but I can say, the offending code was new as of 3.17 so it may bet related to the typings file.
I am trying to extract the referenced typings file from the sandbox, but not having much success. If you don't mind, please send me a copy of what you have so I can analyze it locally.
Finally, if you attempt to use the library as a commonJS format, does it have any positive impact?
const Validator = require('validatorjs')

I am not sure if the Rules access will work within TypeScript, if for any other reason than I am not familiar with TypeScript
import {Rules} from 'validatorjs'

vs

const {Rules} = require('validatorjs')

I was able to replicate the issue above.
"date-fns": "^2.16.1",
"validatorjs": "^3.18.1"

I was using import * as Validator from 'validatorjs'; to import the package and the data to validate contains a simple date string "2020-09-23".

@nethzsumang can you please test with latest version 3.19.2 as this is the version I am testing against.
Also, are you using with TypeScript?

@mikeerickson I updated the package to the latest version 3.19.2 and the issue persists.
No, I'm not using TypeScript.

@nethzsumang I am trying in vein to reproduce this issue.  I am sure it exists as it has been reported numerous times, but it always works here.
Would you be able to create a simple test app which exhibits the issue?

@mikeerickson I have created this sandbox for the reproduction of the issue.

@nethzsumang alas I see the issue. The date-fns module is not imported correctly when using in browser.
All this time I have been testing against a node application.
I will get a fix created and post a new version as soon as possible.

@nethzsumang @GaborGyorgy @Nodios @drthoben @multiojuice
Issue finally resolved in 3.20 release! 🎊
https://github.com/skaterdav85/validatorjs/releases/tag/v3.20.0


Add Yandex DNS provider for Let's Encrypt DNS-01 challenge

Do you want to request a feature or report a bug?
Feature
What did you expect to see?
Easy way to get wildcard certificates for domains delegated on Yandex DNS.
References:
Yandex API:

https://tech.yandex.com/domain/doc/reference/dns-add-docpage/

Projects with implemented API:

https://github.com/AnalogJ/lexicon/tree/v3.2.7
https://github.com/actionm/certbot-dns-pddyandex

Hello,
Traefik uses lego for the ACME features.
Could you open an issue on lego ?
Take a look here https://github.com/go-acme/lego/pull/278
Thanks for your time.


[Cloud Scale] partner information

**NOTE: This is an issue template for new partner pages we are building. Existing partners can use them to submit information needed as we build out enhanced pages. New partners, please continue to use the process outlined here for the time being: https://opensearch.org/new-partner.html
Organization name
CLOUD SCALE INNOVATION PRIVATE LIMITED
Brief description of your business and what it does (50 words or less)
Cloud Scale® is on a mission to simplify the development, deployment, and scaling of complex applications and to bring the full power of multi-cloud to developers and enterprises, everywhere.
Name of your relevant product(s) or service for the OpenSearch audience
Consulting and managed services, Cloud Scale Service Mesh Platform which leverages OpenSearch as the end-to-end observability tool.
What OpenSearch technologies do you specialize in
search, analytics, observability, security
Type of business
MSP, Technology Solutions and Services Provider
Home/main office location or address
Chandigarh, India - 160062
How can people contact you (email and phone)
<EMAIL_ADDRESS>Regions you serve
North America, South America, Europe, Middle East, Africa, Asia Pacific, Australia
Industry specialization
business services, consumer services, education, energy and utilities, financial services, healthcare, media and entertainment, public sector, retail, software and technology
Logo

Additional image

Links
https://cloudscaleinc.com/topics/cloud-computing/simplified-multi-cloud-by-cloudscale-experts/
https://cloudscaleinc.com/topics/tech-articles/cloud-native-security-and-compliance/

@nandita-cloudscaleinc  - Thank you for the information.

@pajuric Would you like to assign this to me so I can add the data?


Implement NSAffineTransform.transform{Point,Size} with tests

As part of this change:

The addition and multiplication operators have been implemented for CGFloat so we can compute dot products.
A private helper function for matrix-vector products has been implemented.
Tests for the identity transformation have been added. Once we support more operations (translations, rotations, and scales), we can add more tests.

Cool, thanks.


Image Caption not aligning to center when added using latest Koenig Editor

Image Caption not aligning to centre when added using latest Koenig Editor in Chrome Browser.
Steps to Reproduce

Add image with caption
Image caption not centred

Technical details:
Adding style in code injection solves the issue
<style>
    figcaption {
        display: block;
        text-align: center;
    }
</style>

Are you using casper 2.3.0?

I am using Casper Version 2.2.1

Styling for Koenig is on Casper 2.3+

@lbguilherme As per your suggestions, I upgraded to Casper 2.3, and adding the styling only centers the text.

Hey @vCillusion :wave: could you please post this to the catch-all forum topic	(there's also some info on reporting issues w/ Koeing!)


fftw3 needs double precision

ISMRMRD only needs single precision libraries for FFTW3 but Gadgetron needs both single and double.
This might need 2 builds for fftw (so in different locations). See
http://www.fftw.org/fftw2_doc/fftw_6.html
However, not sure if the type-prefix is what we need.

easiest solution might be as follows:
create 2 external projects: FFTW3single, FFTW3double. Identical aside from the "--enable-float" flag. Let ismrmrd depend on FFTW3single, and gadgetron on both.
@bathomas, @paskino what do you think?

I've checked that the *.h files are identical in both installations, so that's ok.
It implies downloading the package twice, but I don't care...

agreed, I'm doing it.

fixed in fa02925d50593df917b1a09c4731bc4a53207669


ask permissions before adding default extensions as double click handlers

On windows when we install the app right now we are using tauri to define file types we can open. For the non-kcl file types (stl/obj/etc) we need to ask permission first or we come across as rude jsut doing it
we likely need to patch tauri for this.

Been looking to see if this was part of their config stuff at https://v2.tauri.app/reference/config/#wixconfig but it's not. I believe this is how other apps like VS Code do it right? Making you tick a box if you want the app to open those files?
Would assume that wix supports it under the hood, would have to dive in more to see how we could plug that in


how to use custom shadow color

i want to change the color of the  shadow，any solution ?

same problem

Any update?

https://github.com/879479119/react-native-shadow ?

I know how to customize shadow color and border in this module?
Please help me.


Crash on death with --disable-rune-loss

With the new update of the game when you are killed the game crashes if --disable-rune-loss is present but if I remove it the game doesn't crash, I have also already tried the latest release of er-patcher

@gurrgur

Can confirm. Game goes into boot loop / crash when die if option --disable-rune-loss is ON

Hello,
Try using my pr, see if it works!
https://github.com/gurrgur/er-patcher/pull/70

Hello,
Please check this pr, it might fix the crash
https://github.com/gurrgur/er-patcher/pull/71

@polsrepo someone has to merge it and relase.. so we can download it

@polsrepo someone has to merge it and release.. so we can download it

you can just edit your er-patcher with my changes too :)

@polsrepo @gurrgur Tested the PR. Working on my end.

@polsrepo someone has to merge it and release.. so we can download it

you can just edit your er-patcher with my changes too :)

Thank you. It works for me

Thanks again to @polsrepo for fixing this and to everyone testing as well!


Update Docker volume driver spec file to JSON

See https://github.com/docker/docker/blob/master/docs/extend/plugin_api.md where the SPEC file has been updated for JSON.

No longer needed. See #451


Type narrowed to never, incorrectly

TypeScript Version:  latest
Search Terms: n/a
Code
function f(a?: {}) {
    if (!a) {
        // filters out falsy values:
        //  - `null`
        //  - `undefined`
        //  - `false`
        //  - `NaN`
        //  - `0` / `-0`
        //  - `''`
        return {};
    }

    // here, the type of `a` has been narrowed from "optional {}"
    // to just "{} minus `object`" (object-coercible, ie, anything but null/undefined, but not `object`)
    // effectively, this is `string | number | symbol | boolean | Function`

    if (typeof a !== 'object') {
        // `a` here could be the following values:
        //   - a function
        //   - `true``
        //   - a non-zero non-`NaN`` number
        //   - a non-empty string
        //   - a symbol
        return a; // however, the type inferred here is `never`
    }

    // here, `a` has a verifiably certain type of `object`.
    // because of the type guard above.
    return a; // however, the type is reported as `{}`
}

Expected behavior: The type of a inside the if block will be either {}, or better, string | number | symbol | boolean | Function.
Actual behavior: The type of a inside the if block is never, which is flat out incorrect.
Playground Link: link
Related Issues: not sure how to search for it

This may be addressed by #27157; it can be closed or left open til the next release, as you like.


The TLS certificate everytime refuses the connection

Hey,
First of all I want to thank you for putting so much work into such a great module. Keep the great job on!
Describe the bug
I'm having an issue with TLS certificates. It's maybe because I'm not familiar with those.
I made the certificate without any ext, installed succesfully on the host, but I got this exception:
urllib.error.URLError: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: Hostname mismatch, certificate is not valid for 'localhost'. (_ssl.c:1002)>
Next time I tried to add one ext which is localhost, with the following command:
keytool -genkey -keyalg RSA -alias selfsigned -keystore cacert.jks -storepass password -validity 730 -keysize 2048 -ext SAN=ip:localhost
That's great, now the Gateway is established, but I can't reach it from my computer.
So my question is, how can I skip those IP restrictions or what do you recommend to resolve this issue?
IMO I could use DigitalOcean Firewalls for security so I don't need those restrictions.
But the server will send response for only who provides the correct pem file, doesnt it?
Thank you for helping me fix this, I've been working on this for 1 day and I can't get it fixed and sorry for my basic question!

Hey @jhsznrbt many thanks for checking out IBeam and for your kind words 😊
As for your question: as far as I'm aware localhost is a DNS, not an IP. So you'd need to specify either SAN=ip:<IP_ADDRESS> or SAN=dns:localhost. Let me know if that helps. Here's the wiki just in case: https://github.com/Voyz/ibeam/wiki/TLS-Certificates-and-HTTPS#generate-jks

Hey @Voyz,
Sorry for my late answer I was kinda lack of motivation due to several failures, but now I'm back. :D
I've made GitOps pipeline for deploying the image to a kubernetes. I have locked every external connection, but a vpn connection is enabled. So my concept is that my bot and the gateway can communicate with each other inside the network, but if I want to connect to the gateway from my computer I have to log in to my vpn.
If you are interested in that pipeline, I would be happy to share this to you. It's not a big thing but a big support in the deployment imo.

Hey @jhsznrbt I'm happy you've sorted it out 👍 I'll keep your offer in mind, thanks!

I'm going to close this issue due to inactivity. Thanks for your contribution and please feel free to request a reopen if you'd like to continue the discussion 👍


Sidebar tour throbber can become orphaned

Issue Summary
The tour icon that is attached to the sidebar nav toggle can become orphaned in certain circumstances:

the sidebar is collapsed
fullscreen split pane editor is opened

Steps to Reproduce

Start with a fresh database so that all tour throbbers are visible
Open the admin area
Click the sidebar toggle - note that the nearby throbber now hovers in empty space
Re-enable the fixed sidebar
Navigate to the editor and click the split-screen button - note that again the throbber hovers in empty space

The throbber should move with the sidebar or be disabled when the split screen view is opened
Technical details:

Ghost Version: 1.0.0-beta.1
Browser/OS: Chrome / macOS

I also saw, that the 'Upload theme' throbber gets temporary orphaned, when adding or deleting themes. It's fixed again after a refresh.

I also saw, that the 'Upload theme' throbber gets temporary orphaned, when adding or deleting themes. It's fixed again after a refresh.

Likewise with adding/removing navigation items, I've changed the issue title and added TODOs to reflect.


Does not generate output for --quantization-param-path

Hi thanks for the library! I am following https://github.com/vllm-project/llm-compressor/tree/main/examples/quantization_kv_cache to create a quantized model and the performance is quite bad. When looking at doc,
https://docs.vllm.ai/en/latest/serving/openai_compatible_server.html says:

--quantization-param-path
Path to the JSON file containing the KV cache scaling factors. This should generally be supplied, when KV cache dtype is FP8. Otherwise, KV cache scaling factors default to 1.0, which may cause accuracy issues. FP8_E5M2 (without scaling) is only supported on cuda version greater than 11.8. On ROCm (AMD GPU), FP8_E4M3 is instead supported for common inference criteria.

So I guess we should provide this file. However, it seems the quantization_kv_cache example does not output such a file. Thus I wonder whether to get it?

Hey @fzyzcjy, --quantization-param-path in vLLM is a legacy pathway. The example you reference is the right one for making quantized KV cache with VLLM
When you say the performance is bad --- is this accuracy or speed?

Im going to close this for now. Please reopen if you have further questions about KV cache quantization.

Hi thanks for the reply.

When you say the performance is bad --- is this accuracy or speed?

Bad accuracy. But weirdly, one model gets very bad accuracy, while another model is pretty good and has no degrade at all!

Hi thanks for the reply.

When you say the performance is bad --- is this accuracy or speed?

Bad accuracy. But weirdly, one model gets very bad accuracy, while another model is pretty good and has no degrade at all!

Hm - can you share anything more?
We know that KV cache compression is not lossless --- which is why we do not really recommend it other than for research so far. We are currently working on integrating some new "transform-based" methods that help with the accuracy of KV cache / attention compression:

https://arxiv.org/pdf/2404.00456
https://arxiv.org/abs/2405.16406

@robertgshaw2-neuralmagic Sure!

Qwen2.5-Math-7B-Instruct on math questions: When using fp8 kv cache, accuracy of my internal math benchmark drop from 41.4% to 14.9%
Llama3.1-8B-Instruct on the same benchmark: No accuracy drop at all


fix log muell

Scope & Purpose
Remove debug log muell introduced by https://github.com/arangodb/arangodb/pull/10745

[x] Bug-Fix for a released version (did you remember to port this to all relevant release branches?)
[x] The behavior in this PR can be (and was) manually tested (support / qa / customers can test it)

Testing & Verification
This change is a trivial rework / code cleanup without any test coverage.
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/8452/

Tests blue


gam print gal for longer works due to Google change

gam print contacts prints the Domain Shared Contacts
gam print gal print the Global Address List
Both of the these commands use the Domain Shared Contacts API: https://developers.google.com/admin-sdk/domain-shared-contacts/overview

Sometime recently Google made a change and gam print gal returns this error:
Temporary error: 612 - Contacts API is being deprecated. Migrate to People API to retain programmatic access to Google Contacts. See https://developers.google.com/people/contacts-api-migration., Backing off: 16 seconds, Retry: 4/10
gam print peopleprofile appears to print the User GAL but there is no way as fas as I know to print the Group GAL

The GAL API endpoint was never officially documented and supported afaik (the url does not exist in the developer docs you linked to).
It has been turned down at this point. I'd suggest a replacement command rebuild the GAL based on Directory API and Groups APIs. Groups settings API can be used to determine if a given group should be listed in the directory/GAL or not.
What was the use case of pulling this info with GAM?

It's easy to use the Directory API to list users and get the includeInGlobalAddressList field
For groups you use the Directory API to list groups but then you have to make an additional Group Settings API call per group to get the includeInGlobalAddressList field
That's a lot of API calls if you have N thousnad groups.
Ross Scroggs
@.***

On Oct 28, 2024, at 3:17 PM, Jay Lee @.***> wrote:
The GAL API endpoint was never officially documented and supported afaik (the url does not exist in the developer docs you linked to).
It has been turned down at this point. I'd suggest a replacement command rebuild the GAL based on Directory API and Groups APIs. Groups settings API can be used to determine if a given group should be listed in the directory/GAL or not.
What was the use case of pulling this info with GAM?
—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1715#issuecomment-2442778237, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYLYURLTUIH3VVFAVF3TZ52ZYNAVCNFSM6AAAAABQYNAXW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBSG43TQMRTG4.
You are receiving this because you authored the thread.

It may be possible to batch the calls and get 1000 group settings at a time.
It'd definitely help to understand the use cases for the GAL command though.
Jay
On Mon, Oct 28, 2024, 6:28 PM Ross Scroggs @.***> wrote:

It's easy to use the Directory API to list users and get the
includeInGlobalAddressList field
For groups you use the Directory API to list groups but then you have to
make an additional Group Settings API call per group to get the
includeInGlobalAddressList field
That's a lot of API calls if you have N thousnad groups.
Ross Scroggs
@.***

On Oct 28, 2024, at 3:17 PM, Jay Lee @.***> wrote:
The GAL API endpoint was never officially documented and supported afaik
(the url does not exist in the developer docs you linked to).
It has been turned down at this point. I'd suggest a replacement command
rebuild the GAL based on Directory API and Groups APIs. Groups settings API
can be used to determine if a given group should be listed in the
directory/GAL or not.
What was the use case of pulling this info with GAM?
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1715#issuecomment-2442778237>, or
unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYLYURLTUIH3VVFAVF3TZ52ZYNAVCNFSM6AAAAABQYNAXW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBSG43TQMRTG4>.

You are receiving this because you authored the thread.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1715#issuecomment-2442791900, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIZMCKNUIBCRFJJ3UFJSTZ5227BAVCNFSM6AAAAABQYNAXW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBSG44TCOJQGA
.
You are receiving this because you were assigned.Message ID:
@.***>

People want to verify what groups are in the GAL.
Ross Scroggs
@.***

On Oct 28, 2024, at 4:57 PM, Jay Lee @.***> wrote:
It may be possible to batch the calls and get 1000 group settings at a time.
It'd definitely help to understand the use cases for the GAL command though.
Jay
On Mon, Oct 28, 2024, 6:28 PM Ross Scroggs @.***> wrote:

It's easy to use the Directory API to list users and get the
includeInGlobalAddressList field
For groups you use the Directory API to list groups but then you have to
make an additional Group Settings API call per group to get the
includeInGlobalAddressList field
That's a lot of API calls if you have N thousnad groups.
Ross Scroggs
@.***

On Oct 28, 2024, at 3:17 PM, Jay Lee @.***> wrote:
The GAL API endpoint was never officially documented and supported afaik
(the url does not exist in the developer docs you linked to).
It has been turned down at this point. I'd suggest a replacement command
rebuild the GAL based on Directory API and Groups APIs. Groups settings API
can be used to determine if a given group should be listed in the
directory/GAL or not.
What was the use case of pulling this info with GAM?
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1715#issuecomment-2442778237>, or
unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYLYURLTUIH3VVFAVF3TZ52ZYNAVCNFSM6AAAAABQYNAXW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBSG43TQMRTG4>.

You are receiving this because you authored the thread.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1715#issuecomment-2442791900, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIZMCKNUIBCRFJJ3UFJSTZ5227BAVCNFSM6AAAAABQYNAXW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBSG44TCOJQGA
.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1715#issuecomment-2442891509, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL4TDJM47X5CJCMBUD3Z53FPVAVCNFSM6AAAAABQYNAXW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBSHA4TCNJQHE.
You are receiving this because you authored the thread.

gam config batch_size 1000 print groups fields includeinglobaladdresslist
You hit query limits failrly quickly.
Ross
Ross Scroggs
@.***

On Oct 28, 2024, at 4:57 PM, Jay Lee @.***> wrote:
It may be possible to batch the calls and get 1000 group settings at a time.
It'd definitely help to understand the use cases for the GAL command though.
Jay
On Mon, Oct 28, 2024, 6:28 PM Ross Scroggs @.***> wrote:

It's easy to use the Directory API to list users and get the
includeInGlobalAddressList field
For groups you use the Directory API to list groups but then you have to
make an additional Group Settings API call per group to get the
includeInGlobalAddressList field
That's a lot of API calls if you have N thousnad groups.
Ross Scroggs
@.***

On Oct 28, 2024, at 3:17 PM, Jay Lee @.***> wrote:
The GAL API endpoint was never officially documented and supported afaik
(the url does not exist in the developer docs you linked to).
It has been turned down at this point. I'd suggest a replacement command
rebuild the GAL based on Directory API and Groups APIs. Groups settings API
can be used to determine if a given group should be listed in the
directory/GAL or not.
What was the use case of pulling this info with GAM?
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1715#issuecomment-2442778237>, or
unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYLYURLTUIH3VVFAVF3TZ52ZYNAVCNFSM6AAAAABQYNAXW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBSG43TQMRTG4>.

You are receiving this because you authored the thread.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1715#issuecomment-2442791900, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIZMCKNUIBCRFJJ3UFJSTZ5227BAVCNFSM6AAAAABQYNAXW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBSG44TCOJQGA
.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1715#issuecomment-2442891509, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL4TDJM47X5CJCMBUD3Z53FPVAVCNFSM6AAAAABQYNAXW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBSHA4TCNJQHE.
You are receiving this because you authored the thread.


#166 - Added a bounding box query test

#166
Added a bounding box query test based on our tests framework.
It basically tests 3 aspects:

Are the resulted markers in accordance with the given latitude/longitude parameters
Whether the markers where correctly filtered given inaccurate parameter
Whether they were filtered correctly given the severity different parameters

Nice, it's a good test. However, whenever we run it on an empty database, which is the case in Travis CI automated tests, it will all pass vacuously. Can we make it load some sample data before testing it? Perhaps the 2013 data, since it's the sample data that's on Git? In that case, the year should be 2013 rather than the current year.
Regardless, I'll merge this once you handle the line comments.

Not sure what do you mean by making it load the example data.
I changed the default year to 2013 and commented the previous lines for future usability.
Fixed all test calls to assertTrue / assertFalse where required
Changed to <= the bounding box filters

I mean to run process.py so that the DB isn't empty when the test is run.
It would mean a lot more time to run the tests, though, so I'm not sure it's worth it. Need to think about it.

Done.


why is there a dependency to angular-route?

see .bower.json

Did you mean angular-animate? There shouldn't be a dependency on angular-route. The module does have animations in it; though it was a long time when I first did it. There's probably a more recent way to do them.


sometimes when opening the console for the first time in Chrome as Chrome shows the cert warning and you click Advanced ...

... before you get time to click on the "Proceeed to fabric8.vagrant.f8' link, it refreshes and starts to display the fabric8 console. You then have to click reload again

Following the steps in this thread seem to help immensely with the cert prompts BTW -> https://productforums.google.com/forum/#!topic/chrome/FilamLOL72E

TL; DR; goto chrome://flags and search for Remember decisions to proceed through SSL errors for a specified length of time, set to 3 months.


added missing ! on link for extending sails doc

added missing ! on the link for extending sails doc

Nice catch, thanks @luislobo


Jlandi pact cli

Adding matcher support
Adding support to control fileWriteMode
Updating versions to be compatible with recent @pact-foundation changes.

Codecov Report

Merging #350 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #350    +/-   ##
======================================
  Coverage     100%   100%
======================================
  Files          65     52    -13
  Lines        1631   1338   -293
  Branches      253    173    -80
======================================
- Hits         1631   1338   -293

Flag
Coverage Δ

#builder
100% <100%> (ø)
:arrow_up:

#runtime
?

#srcapp
?

Impacted Files
Coverage Δ

config/karma/pact.karma.conf.js
100% <100%> (ø)
:arrow_up:

runtime/style-loader.ts

runtime/directives/sky-app-link.directive.ts

runtime/params.ts

runtime/i18n/resources.pipe.ts

runtime/i18n/locale-provider.ts

runtime/auth-token-provider.ts

src/app/app.component.ts

runtime/bootstrapper.ts

runtime/window-ref.ts

... and 4 more

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 ef8eb3d...bc4377b. Read the comment docs.


Pusher php sdk

Hello,
I can't install your package as i get the following error:
Problem 1
- beyondcode/laravel-websockets 1.1.1 requires pusher/pusher-php-server ~3.0 -> satisfiable by pusher/pusher-php-server[3.4.x-dev, 3.0.x-dev].
- beyondcode/laravel-websockets 1.1.1 requires pusher/pusher-php-server ~3.0 -> satisfiable by pusher/pusher-php-server[3.4.x-dev, 3.0.x-dev].
- beyondcode/laravel-websockets 1.1.1 requires pusher/pusher-php-server ~3.0 -> satisfiable by pusher/pusher-php-server[3.4.x-dev, 3.0.x-dev].
- Can only install one of: pusher/pusher-php-server[v4.0.0, 3.4.x-dev].
- Can only install one of: pusher/pusher-php-server[v4.0.0, 3.0.x-dev].
- Installation request for pusher/pusher-php-server ^4.0 -> satisfiable by pusher/pusher-php-server[v4.0.0].
- Installation request for beyondcode/laravel-websockets (locked at 1.1.1, required as *) -> satisfiable by beyondcode/laravel-websockets[1.1.1].
Look like i must use version 3.4 of pusher-php-server, this version is not compatible with my php version(7.3.6).
Any ideas?

PR is already opened for this issue #209. Just needs to be merged.

@fridzema PR https://github.com/beyondcode/laravel-websockets/pull/209 has been merged. Can you please target the dev-master-version (https://packagist.org/packages/beyondcode/laravel-websockets#dev-master) and verify your installation runs successfully?

@ttomdewit everything installed successfully now!

@fridzema Could you please be so kind & close this issue?
Much appreciated!


Support tags

It would be great to be able to configure tags. See PR #7 for an implementation.

This should now be possible in v0.0.24 onwards. The relevant change was done in https://github.com/uktrade/fargatespawner/commit/72b20163825af557f401268a9f7108174e2dd83b


GK FlipBook files added

Description 📜
General knowledge digital flipbook ...happy learning:)
Fixes #1427

Type of change 📝

[x] Add a new project

Domain of Contribution 📊
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Project using HTML, CSS & JS

Checklist ✅
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots / Gif (Recommended) 📸
https://user-images.githubusercontent.com/91609324/167297209-aa2e202b-7705-428e-992c-720c2e4b9b2b.mp4
README (2).md

Add readme file

Add readme file

done

Add readme file

done

where is readme?

 here

bhai aise ni kr, isko apne project folder ke andar daal readme.md file bna vha


Feature Request: Tree Table

Thanks a lot for this great component. I wanted to give you feedback, on how useful treelike/hierarchical display/rendering could be. For an example, please visit https://experience.sap.com/fiori-design-web/tree-table/. Basically this type of table allows you to structure data into categories, allowing the user to choose the level of details they are interested in.

This is a great idea but will be a lot of work. I'm not sure when we can get around to this. But if someone wants to contribute, we're happy to help


replace unsigned_negation

lint unsigned_negation has been removed: replaced by negate_unsigned
feature gate

r? @chandraprakash
(maidsafe_highfive has picked a reviewer for you, use r? to override)

r? @chandraprakash

Please update examples as well.
$ RUST_TEST_TASKS=1 RUST_TEST_THREADS=1 cargo test
Compiling crust v0.2.0 (file:///home/travis/build/maidsafe/crust)
examples/simple_receiver.rs:23:22: 23:39 warning: lint unsigned_negation has been removed: replaced by negate_unsigned feature gate
examples/simple_receiver.rs:23         unsafe_code, unsigned_negation, unused_allocation, unused_attributes,
                                                ^~~~~~~~~~~~~~~~~

examples/simple_sender.rs:22:22: 22:39 warning: lint unsigned_negation has been removed: replaced by negate_unsigned feature gate
examples/simple_sender.rs:22         unsafe_code, unsigned_negation, unused_allocation, unused_attributes,
                                              ^~~~~~~~~~~~~~~~~

examples/crust_peer.rs:25:22: 25:39 warning: lint unsigned_negation has been removed: replaced by negate_unsigned feature gate
examples/crust_peer.rs:25         unsafe_code, unsigned_negation, unused_allocation, unused_attributes,
                                           ^~~~~~~~~~~~~~~~~

 Running target/debug/crust-ca605d2f121f2715

Please update examples as well.

Done.


e2e: Update advance flow pipelineruns Task names

Fixes

https://issues.redhat.com/browse/HAC-4706
Description

e2e: Update advance flow pipelineruns Task names.
advance flow failing in Periodic job due to pipeline runs task name got changed.
Type of change

[ ] Feature
[ ] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[x] Other (please describe): Test

Screen shots / Gifs for design review

How to test or reproduce?

Browser conformance:

[x] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

[test]

[test]

[test]

/lgtm
/approve


Relax dependency on scikit-learn: skore should not forbid using dev version of scikit-learn

Is your feature request related to a problem? Please describe.
skore specifies as its dependency "scikit-learn<1.7". As a consequence, a user cannot have skore and scikit-learn dev version installed jointly. The danger is that the most advanced scikit-learn users, or its developers, are then unlikely to use skore.
To expend a bit, as a developer of a package, I try to work on daily basis with the latest version (the development one) of a package, because actually working with a package helps me spot bugs or other possible improvements. Right now, I must choose whether I am going to apply this policy for skore or for scikit-learn.
The danger is to push away our most agile users, users who would be more likely to be early adopters.
Describe the solution you'd like
Remove the upper limit on scikit-learn's version, at least on the dev version
Describe alternatives you've considered, if relevant
No response
Additional context
No response

@GaelVaroquaux we must ensure compatibility with a range of versions of scikit-learn.
How to inform user that we don't currently guarantee compatibility over the last minor, 1.6?
I don't think documentation is enough to avoid issues.


Allow setting manual threshold in PL inference

Description

This PR makes a few small changes which allows users to set a manual threshold when running lightning inference, even after the model has been trained with adaptive thresholding enabled. To use the manual threshold, the user can simply set the thresholding method to manual in the config, and specify the image and pixel-level thresholds. These values will then overwrite the adaptive threshold values which are loaded from the model weights.

The model loading functionality in LoadModelCallback was moved from on_test_start and on_predict_start, which removes duplication and is more in line with the intended use of setup.

By adding the model loader callback before the post-processing configuration callbacks, the threshold value is first loaded from the model file. Then, when setup is called in post-processing configuration callback, the thresholds are overwritten by the manual values specified by the user in the config.yaml.

Based on my experimentation I could not find any side effects of these changes, but it would be good if the reviewers could double check this on their side.
Related discussions: https://github.com/openvinotoolkit/anomalib/discussions/578, https://github.com/openvinotoolkit/anomalib/discussions/767
Changes

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

Checklist

[x] My code follows the pre-commit style and check guidelines of this project.
[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing tests pass locally with my changes

Thanks. I feel we should document this behaviour somewhere though.

I agree with @ashwinvaidya17. Would be great if we mention this somewhere in the documentation.

Yeah, I think we need an extensive guide in our documentation which explains the thresholding and normalization mechanisms of anomalib, and gives pointers on how to configure this. We're getting many questions in the discussion section on these topics.


[feature] Use jdbc connection pooling

This pull request uses HikariCP
to provide JDBC connection pooling. it fixes #38
It also removes an unused dependency from the pom which doesn't really warrant
it's own pull request.

Integration tests are failing. Here's part of a stacktrace I found in the Tomcat's logs:
Caused by: org.springframework.beans.BeanInstantiationException: Failed to instantiate [com.zaxxer.hikari.HikariDataSource]: Constructor threw exception; nested exception is java.lang.ClassCastException: org.postgresql.Driver cannot be cast to javax.sql.DataSource
	at org.springframework.beans.BeanUtils.instantiateClass(BeanUtils.java:163)
	at org.springframework.beans.factory.support.SimpleInstantiationStrategy.instantiate(SimpleInstantiationStrategy.java:122)
	at org.springframework.beans.factory.support.ConstructorResolver.autowireConstructor(ConstructorResolver.java:267)
	... 46 more
Caused by: java.lang.ClassCastException: org.postgresql.Driver cannot be cast to javax.sql.DataSource
	at com.zaxxer.hikari.pool.PoolUtilities.initializeDataSource(PoolUtilities.java:102)
	at com.zaxxer.hikari.pool.BaseHikariPool.<init>(BaseHikariPool.java:156)
	at com.zaxxer.hikari.pool.HikariPool.<init>(HikariPool.java:60)
	at com.zaxxer.hikari.pool.HikariPool.<init>(HikariPool.java:48)
	at com.zaxxer.hikari.HikariDataSource.<init>(HikariDataSource.java:80)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:526)
	at org.springframework.beans.BeanUtils.instantiateClass(BeanUtils.java:147)
	... 48 more

I suggest you use the more traditional approach by using jdbcUrl for configuring Hikari.

Fixes #38

FYI @dacrome: Closing issues via commit messages only works in commits or in the description of a PR, but not in arbitrary comments. I've tried the same as you did, but ended up editing the description of the PR.

Well, it only fixes #38 when we merge it, right?

Yes. Didn't see that you already mentioned this in the PR description. Anyway, GitHub only recognize this in PR desciptions or commit messages, like @tkrille already descibed.

Can you squash the last 2 commits?

Squashed

LGTM now. Merged.


Hitting "Cancel" on upload song modal, re-opening and hitting "Save" crashes the app displaying "Re-establishing connection" forever

If you start uploading a song, then "Cancel" in the middle, and then if you re-open the modal

The modal is now in some undefined state, having a progress bar from previous upload
You can hit "Save" button, which now breaks something further and "Re-establishing connection" is now forever visible

As a side note, you should probably also not be able to click "Save" while the file is uploading.
https://github.com/fly-apps/live_beats/assets/33274/1164f399-095e-4504-8677-700d416684ec

As a side note, I think something should happen when the upload finishes and progress bar reaches 100%. As it is now, it stays there at 100% without giving user any indication that they can now safely hit "Save'

Adding loading bar on button when you click save would be good as well.

This was a bug on LV main. Thanks!


Models (state container & new JavaScript best practices) (WIP)

Replaces #377 (rebased onto current master)
This also contains #499
This PR contains:

[x] Changelog
[ ] Documentation
[x] Tests

@kusma Indeed, I'll clean it up

@kusma I've gone through and cleaned up the history quite a bit, but I'm not entirely sure how granular you want this to be. Let me know if you'd like i.e. bugfixes squashed as well.

Superseeded by #601


CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide Website
WHEN I view the study guide
THEN I see a dark blue header and a footer, and four boxes with a shadow

Done


Missing support for cmake

I am having problems with integrating this library into my project.
If cmake is 100% supported, it should work seamless with my gitache project; but it doesn't.
Now gitache isn't relevant here; that does just the standard things expected for cmake; I am just mentioning it to make clear that this is a well established, automated, standard process; which doesn't work with imguiwrap ;).
The problems I run into are:
Could not find a package configuration file provided by "imguiwrap" with
 any of the following names:

   imguiwrapConfig.cmake
   imguiwrap-config.cmake

And the fact that nothing got "installed".
When listing the files in the install prefix of a gitache project, I expect to see something like:
>ls /opt/gitache/libcwd_r/49222d132e2085669a45e519a8165087bfbe40eb818eb0743f5f7caf4a89a683/
DONE  include/  lib/  share/

with installed headers in include, the libraries in lib etc.
(Most notably, 'lib' contains (apart from the library itself):
lib/cmake/libcwd_r/libcwd_rConfigVersion.cmake
lib/cmake/libcwd_r/libcwd_rConfig.cmake
lib/cmake/libcwd_r/libcwd_rTargets.cmake
lib/cmake/libcwd_r/libcwd_rTargets-release.cmake
lib/pkgconfig/libcwd_r.pc

For imguiwrap I end up with:
>ls /opt/gitache/imguiwrap/0d4482cb3cc895fda9ade4fa0a5452f208bcc226fd07e7c60f0c10c52816f54d
DONE

a completely empty directory.
Obviously this doesn't allow my project to find anything - no headers and no library.

Ah - that's because it's implemented for either vendoring/embedding or use via Fetch_Content, as opposed to installing as a standalone package (https://github.com/kfsone/imguiwrap#integration-with-your-cmakelists-based-project).

So, imguiwrap does not have an install target? That's basically what I was saying :-/

It's not something I envisage as a distributable, only build from source. I believe the license is permissive enough that anyone who wants to use it can just drop the source in to their own code and throw away the examples/etc.
I'm happy to support anyone who wants to facilitate making it installable, but it honestly didn't occur to me that anyone might use but the non-header portion in earnest: it has main() in it.
I'm happy to maintain an install target if you want to contribute, or be supportive of it if you want to fork.


Volume doesn't work

Hi I've got an issue with the volume which doesn't work/respond. Any idea what is causing this? the rest of the controls are working fine though,
how my code looks like:
type: custom:android-tv-card
remote_id: remote.chromecastzolder
keyboard_id: media_player.chromecast_zolder
rows:
  - - back
    - power
    - home
    - search
  - - volume_down
    - volume_mute
    - volume_up
  - - navigation_buttons
  - - netflix
    - youtube
    - disney
    - spotify
    - nlziet
    - videoland
  - - previous
    - rewind
    - play_pause
    - fast_forward
    - next

I enabled and using the following HA integrations:

Android TV Remote:
remote.chromecastzolder
media_player.chromecastzolder
Android Debug Bridge:
media_player.androidtvzolder
Google cast:
sensor.chromecast_zolder

I've tried all three without success.
What I noticed is that the volume buttons on the entity (at the integration page) is't working either. So what I mean:

If you know a workaround or maybe a way to control the volume with an adb command (?) please let me know.

If it isn't working in the Home Assistant more info popup, then it isn't an issue with this card and shouldn't be labeled as a bug.
How are your speakers setup?

It could be card related right but I tested all media integrations and volume doesn't work anywhere so I need to fix that first
You may close the ticket
Thanks for replying anyway!


devServer open: 'chrome'

Bug report

What is the current behavior?
I am on macOS with Safari as my default browser. I am attempting to open Google Chrome with the following in my webpack config:
devServer: {
    open: 'chrome'
}

This only works is chrome is already open on my Mac. If all browsers are closed it will open Safari if Chrome is open it will open the build in Chrome.
I have also tried
devServer: {
    open: 'Google Chrome'
}

If the current behavior is a bug, please provide the steps to reproduce.

Using macOs set your default browser to Safari.
Make sure you have Google Chrome installed.
Add the following to your webpack config:

devServer: {
    open: 'chrome'
}

While all browsers are closed run a dev build of your code. Safari opens.
Then
Open Chrome and Safari and rerun your build, chrome opens.

What is the expected behavior?
If Chrome is installed, it should open chrome if it is closed.

Other relevant information:
webpack version:  6.9.0
Node.js version: 10.15.1
Operating System: MacOS 10.14.4
Additional tools:

It is os limitations, please read https://github.com/sindresorhus/open#app, we can't do something on our side, maybe you can open issue in https://github.com/sindresorhus/open repo


Use degrees for angles in Arc and Sector

Changes
Closes #868 by switching angle and angle_start to use degrees per discussion in that issue's comments.
Follow-up Work
It may be a good idea to add radians versions of these properties to prevent having to convert between degrees and radians.

This looks good. I'm thinking we might want to target the 2.1 release to avoid breakage.


[Reflection] Add support for imported structs with recorded fields (attempt #2)

Update IRGen to trigger generation of type metadata for foreign
struct types found in fields. And fix TypeRefBuilder to handle
the case where struct has fields but at the same time has opaque
metadata.

@swift-ci please test source compatibility

FYI, I just merged a change to XFAIL GRDB and ObjectMapper as these were failing due to earlier changes: https://github.com/apple/swift-source-compat-suite/pull/127

@xedin Can you add reduced test cases for the two failures that showed up in the source compatibility suite from the earlier attempt at landing this? https://forums.swift.org/t/source-compatibility-bots-in-bad-shape/9526

Absolutely! I am just trying to re-run and figure out what is going on...

@rudkx this run only got GRDB and ObjectMapper, but I haven’t changed anything in my commit...

@swift-ci please test source compatibility

@rudkx I don’t know what is going on by it looks like compat suite passed with my commit and no changes...

@xedin Interesting. It's a bit of a coincidence those failures were related to metadata and some illegal IR. I suppose we can land again and see what happens...

Let me try to re-run couple more times, and see if this is going to make any difference...

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please test source compatibility

Last time and if everything is fine I'm going to merge this back in.

Looks like it’s green again, going to merge

@swift-ci please smoke test and merge

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please smoke test

@swift-ci please smoke test

@rudkx @jckarter So the problem with original PR was that it was recursing to much into the foreign types, it failed only in optimization enabled builds and because it was re-writing ImportedStructs while iterating it, which I fixed. So please let me know if you guys are okay with the changes.

SGTM!


⚠ Minecraft has degraded performance

In f83c3ef, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1166 ms

Resolved: Minecraft performance has improved in 02ee99f.


Release Flatcar Container Linux Alpha 3346.0.0 Beta 3277.1.2 Stable 3227.2.2 LTS-2022 3033.3.5 LTS-2021 2605.31.1

Release Flatcar Container Linux Alpha 3346.0.0 Beta 3277.1.2 Stable 3227.2.2 LTS-2022 3033.3.5 LTS-2021 2605.31.1
The release of Flatcar Container Linux Alpha 3346.0.0 Beta 3277.1.2 Stable 3227.2.2 LTS-2022 3033.3.5 LTS-2021 2605.31.1 is planned for September 1st, 2022.
Tasks

[x] Go / No Go hackmd doc created from previous one for test tracking and release notes: link (@dongsupark / @sayanchowdhury )
[x] All planned changes are merged & blocker bugs are fixed (@dongsupark )
[x] Branch off main (only new major) (@dongsupark )
[x] Tag build branch (@dongsupark )
[x] Generate manifest (@dongsupark )
[x] Test status captured in hackmd doc (@dongsupark )
[x] Create SDK container after Jenkins is done building the flatcar-sdk tarball; upload container to ghcr.io ()
[x] All tests are green (@flatcar-linux/flatcar-maintainers )
[x] Release notes / announcements added to Go/No Go doc with show-changes/show-fixed-kernel-cves.py (@dongsupark / @sayanchowdhury )
[ ] QA: check diff for image file list, packages, and image size in the image job output ()
[x] Matrix Go/No Go meeting (inc. release notes review) (@dongsupark )
[x] Manifest release created on GitHub (@dongsupark)
[x] Update the SDK release metadata for new major Alpha releases with SDK builds (only new major) (@dongsupark)
[x] Images uploaded with copy-to-origin.sh (@dongsupark)
[x] Run the os/release job (@dongsupark)
[ ] Publish EC2 Marketplace Images (@sayanchowdhury)
[ ] Azure offer updated & publishing started (@sayanchowdhury)
[ ] Azure Go Live (@sayanchowdhury)
[ ] GCP offers updated (@sayanchowdhury)
[x] Update payload signed/uploaded (@dongsupark )
[x] Symlink to "current" updated with set-symlink.sh (@dongsupark)
[x] Website updated with ./update-flatcar-versions.sh and PR merged (@dongsupark)
[x] Update CloudFormation templates (@dongsupark)
[x] Release package published in Nebraska (@dongsupark )
[x] Send announcement to Mailing Lists ()
[x] Send Tweet out (@ahrkrak)
[x] Brief version announcement in slack (k8s slack #flatcar) and the Flatcar matrix channel (@t-lo )

FYI, ami-02ea825969d1f499b for Stable arm64 was set to private due to the issue https://github.com/flatcar-linux/Flatcar/issues/840.

Azure marketplace offers flatcar-container-linux-free and flatcar_pro are published; flatcar-container-linux is still in Certification.

@dongsupark  Can we close this issue?

Yeah, we still do not know what was going on with the corrupted AMI.
However, as we now run CI without vmdk, not an issue any more.
Let's close this issue.


Convert README and CONTRIBUTE to rst and include them as part of the documentation

Hello @xuhdev may I work on this?

Hello @xuhdev may I work on this?

@dgabriel123 Absolutely! Any document improvement is welcome. If you feel it is too difficult to place them in the current documentation, simply converting to rst and improving text is appreciated.

@dgabriel123 Absolutely! Any document improvement is welcome. If you feel it is too difficult to place them in the current documentation, simply converting to rst and improving text is appreciated.


Error on PYTHONHASHSEED=0 python3 train_mnli.py DIIN demo_testing_SNLI --training_completely_on_snli

Everything is OK up to loading shared.jsonl but when data_processing.py tries to load it the following error is raised:
[1] Loading data SNLI
550152it [00:09, 55502.23it/s]
10000it [00:00, 58533.31it/s]
10000it [00:00, 64180.29it/s]
[1] Loading data MNLI
392702it [00:07, 50719.45it/s]
10000it [00:00, 61434.57it/s]
10000it [00:00, 60510.07it/s]
9796it [00:00, 57942.87it/s]
9847it [00:00, 56161.09it/s]
../data/shared.jsonl
Traceback (most recent call last):
  File "train_mnli.py", line 68, in <module>
    shared_content = load_mnli_shared_content()
  File "..../Densely-Interactive-Inference-Network/python/util/data_processing.py", line 173, in load_mnli_shared_content
    assert shared_file_exist
AssertionError

It seems like the data downloader downloads shared.json instead of shared.jsonl while the training script tries to load a .jsonl file.
Everything works fine when downloading shared.jsonl without a script.
Also you could rename shared.json (which redirects to downloading the file) to shared.jsonl  in README

Thank you for pointing out. It is updated now.


Not able to scroll in bookmark folders

When the screen height is limited it's not possible to access all the bookmarks in a list because some are rendered off-screen. Need to implement some way of scrolling for access to all bookmarks.

Fixed in next branch.


How to call Snackbar from a ts/js file other than a component?

I have a global request error logic in a ts/js file. If the return statusCode match some condition I want to show a Snackbar/Alert.
So the Snackbar/Alert will not delcared in any componet file，and I call Snackbar/Alert whenever and wherever I need.
for example, in a js file:
Snackbar.error('my message', duration, onClose);
You can reference the Ant Design notification

Duplicate of #18098


Unrecruitable flag specific to the pokemon + others

MultiplyElementInMajorStatusEvent and BladeState
Added AddElementRangeEvent (adds range if the move is of a certain type)
Removed element from MultiplyElementInMajorStatusEvent and the event is now called MultiplyDamageInMajorStatusEvent
Removed AddElementRange since it can also be broken down (ElementNeeded and AddRange)
Added HasStatusNeededEvent, which allows the user to specify exactly which status effects are needed; can have members
Added SetTrapSingleEvent inheriting from SingleCharEvent
Collapsed MultiplyDamageInMajorStatusEvent into MajorStatusPowerEvent and HasStatusNeededEvent into GiveStatusNeededEvent

My mistake for recommending HasStatusNeededEvent be collapsed; I have reverted that change in the follow-up commit; be sure to merge that in as you continue work: https://github.com/PMDCollab/PMDC/commit/1cfce5e8d5de6571bfb68d1b7125a19f41017aa9


ppt and pptx ToPdf not work

issue is awt required but in android 10 or higher awt not found android studio

Hi @dineshchavhan . I hadn't tested this API in android yet.
Will do

this can be now implementable in android since release of 2.0.0


Issue with windows separator

Operating system
Windows 11/10
Helmfile Version
0.147.0
Helm Version
3.10.1
Bug description
Upgrading release=xxx, chart=repo\chart
Note "" in the output.
This bug break helmfile with cygwin/bash.
In powershell + busybox, "" does not break helm/helmfile, but it's still not nice to see ""
helmfile template works but not helmfile sync, if it helps to investigate
Example helmfile.yaml
N/A
Error message you've seen (if any)

Steps to reproduce
use helmfile 0.147.0 under bash/cygwin and run helmfile sync
Working Helmfile Version
0.147.0
Relevant discussion
No response

@itscaro fixed: https://github.com/helmfile/helmfile/pull/451  could you test it on your windows?

@yxxhero Thanks, I will try

@yxxhero Tested on pwsh + busybox and bash + cygwin. all are good ! thanks :-)


[FEATURE]: Add ability to access exported model types

Describe what you want
There doesn't seem to be any way to grab the type of a table. For instance, a findMany ran on the users table should return a type of User[]. The returned data is typed correctly, but the type is not named so I cant import/use it anywhere (for instance like a function parameter type).

You can use InferModel<T> - check this out from the docs:

You can get the type of the user entity by inferring the TS type from the table function result.


Change in stats from last week look broken

None of the projects have any change data.

what maybe issue?

Seems to be the case as of 2017-06-14 as well. Hmmm...

when the site doesn't build every day, the stats are broken until it builds again daily for a week.   The build was broken for a few days til this morning.  Fixes to this behavior are welcome :)

An easy but ugly fix would be a daily cron build...

we do that already.
The site build breaks without constant maintenance - every time a repository that we list moves, the build breaks.  A fix against that is needed.
You can see the fail mode if you change a repo path in a source/projects/x.md file to something old (e.g. revert today's change from spf13/hugo to gohugoio/hugo and try the build and you'll see what fails)

Ahhhhhhh, sh&$. Didn't even think about notifying the good folks at Netlify of the org change for this site. Sorry about that, and thanks for updating, @fool

fear not @rdwatters it wasn't just hugo and someone actually did PR it this morning ;)


feat: Suggesting best routes

Closes #17

Clicking on the find route button is causing an error. This is clearly because you used something related to react-leaflet outside <MapContainer />. However, I can't find where exactly you have used it.
One thing you can try is implementing routing.js as a React component and returning something (worst case: null) from it. And use it inside <MapContainer />


Add option to control zero padding of file numbers in output

Motivation: Scripts which process generated output should not have to perform more work than necessary in detecting the number of generated output files. Currently, a zero padding is added to the file numbers {n} depending on the number of files. Compiling a document with 5 pages, changing it to produce 10 pages and re-compiling it leaves behind both test-5.svg as well as test-05.svg. By forcing no padding to be set, the scripts at least have a consistent way of referring to individual pages in a document.
New flag:
      --zero-pad <ZERO_PAD>
          Zero padding of the file number in the output file name [default: true] [possible values: true, false]

This is turned on by default for backward compatibility. One could pass --zero-pad=false to turn it off.
Thanks!

An option seems a bit overkill to me. We should just settle on one behaviour in my opinion. I wonder what other, similar tools do.

I think there are benefits to allowing customization.
Pros of zero padding

ls lists the output in the correct order by default and makes processing simpler
Image viewers and file browsers display the individual images in the correct order

Pros of no padding

Easier to link to a file from latex because the page numbers don't change
Easy detection of the number of pages from another build script
No junk files left behind (I don't want p-01.svg and p-1.svg leftover in the same directory when only one of them is fresh at any given time)

pdfseparate uses the pattern %d and does not use any padding.

Here's a slightly related take: For PDFs which embed a different logical page number in addition to a physical page number, it would be great if the output pattern can include this information. This might sound overkill - can we allow custom format strings in the output file name?
That would allow the user to specify if they want the physical or logical page number, and also specify the padding (zeros, underscores) themselves. The big negative here is forcing the user to learn a new pattern syntax.

If we add customization, I think I would prefer to encode it in the {n} pattern somehow, as some kind of format string. What do you think?

I like the idea. The natural step would be to allow rust format! style patterns, so I found interpolator from https://lib.rs/keywords/string-formatting. Do you have any suggestions for such a crate? I could not see any existing fmt-like dependencies on https://github.com/typst/typst/blob/main/Cargo.toml.

Can Rust-style format patterns express the page-count-dependant zero padding though?

I could provide them as built-in "variables" and use std::fmt semantics
p // page index
w // floor(log10(page count)) + 1
t // page count

To use them,
page-1 -> page-{p}
page-01 -> page-{p:0w$}
page-01-of-10 -> page-{p:0w$}-of-{t}
https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=bc3d2447a5a5a1fd1b2a313c01ba9a7a
If someone wants to add logical page numbers in the future, it would be another variable.
I have not tried out any third party crates that emulate std::fmt's behavior at runtime.

That does sound nice in principle. The one downside I see is that Rust's formatting syntax is somewhat cryptic in my opinion. Even though I've programmed in Rust for a very long time now, I always need to look more complex things up.

Yeah, I see your point. I guess it would be fine to just adopt Rust's syntax. To an extent, some crypticness is unavoidable in these compact formatting descriptions.

Let's close this PR and I'll make a new one with rust-style fmt syntax?

If someone wants to add logical page numbers in the future, it would be another variable.

Page numbers are already purely physical on the output and should remain so, this is likely unneeded complexity.
I don't think there's any need for something that tries to stay close to Rust/fnmtlib's syntax, since it's a different, much simpler use case. We could likely get away with something as simple as:

{p}: page index
{t}: total page count
{0p}: page index, but zero padded
{0t}: total page count, but zero padded

At this point, it even begs whether Typst should do any formatting here at all? If a user actually cares about the padding, then they should probably add it in post.
I certainly don't ever care about the padding, if I export to non-PDF I only ever care about one image and the padding proved to be annoying.
Backwards compatibility shouldn't be much of an issue here, as it simplifies the few brittle scripts that people had to hack together to handle this. But if you prefer a backwards compatible way to do this, one could use something like:

{p}: page index, but zero padded
{t}: total page count, but zero padded
{p-}: page index
{t-}: total page count
{n}: page index, alias for p
{n-}: page index, alias for p-

Syntax is up for debate, I guess.

I don't think there's any need for something that tries to stay close to Rust/fnmtlib's syntax, since it's a different, much simpler use case.

Rust's syntax is better documented and more flexible. It's also more search-friendly for a beginner. I'm personally not a fan of tools that have their often incomplete formatting syntax I need to look up on a man-page.

At this point, it even begs whether Typst should do any formatting here at all? If a user actually cares about the padding, then they should probably add it in post.

I agree with this. We could just add one more variable for no-padding {x} (insert letter of your choice) and let user scripts process them. We should however keep {n} for backwards compatibility. If @laurmaedje too agrees, I'm inclined to pick this route.

Page numbers are already purely physical on the output and should remain so, this is likely unneeded complexity.

Perhaps this is a discussion for another issue but I recognize the use-case in a presentation framework using typst. Using actual slide numbers (with page-1-step-1 for transitions) would be a better UX compared to using unpredictable page numbers.

Rust's syntax is better documented and more flexible. It's also more search-friendly for a beginner. I'm personally not a fan of tools that have their often incomplete formatting syntax I need to look up on a man-page.

I would generally agree with this if it was for anything more than the output of the page numbers. I don't think it warrants the added syntactic complexity that comes with keeping it close to Rust-like formatting.

Perhaps this is a discussion for another issue but I recognize the use-case in a presentation framework using typst. Using actual slide numbers (with page-1-step-1 for transitions) would be a better UX compared to using unpredictable page numbers.

While that's a use case I haven't considered, this can likely be done with the proposed unpadded page numbers and a little bit of typst query too.

Having {p}, {t}, {0p}, and {0t} sounds fairly reasonable to me. We could keep {n} = {0p} for one version and deprecate it.
However, if we think that formatting is completely out of scope and the padding is just annoying, we can just as well keep only {n} and change it to not pad. That's a pretty small breaking change that will probably affect very few users, so I'd be willing to do it.

That's a pretty small breaking change that will probably affect very few users, so I'd be willing to do it.

Sure! Is there a process for announcing breaking changes? We might not want users' bash scripts to start silently breaking.

It'll just be marked "(Breaking change)" in the changelog.

Thanks for your inputs @tingerrr and @laurmaedje. I made a draft PR at #3933. Feedback welcome :)


Add Support for Delete Location History Endpoint

Add support for the Delete Location History endpoint of the Device API.

Completed in #13


Upgrade httpoison dependency

Is your feature request related to a problem? Please describe.
Hi, I'm trying to install the configcat dependency to my Elixir project, but get the following error
Resolving Hex dependencies...
Resolution completed in 1.007s
Because your app depends on configcat ~> 4.0.0 which depends on httpoison ~> 1.7, httpoison ~> 1.7 is required.
So, because your app depends on httpoison ~> 2.1, version solving failed.
** (Mix) Hex dependency resolution failed

Describe the solution you'd like
Is there a way we can make the dependency more flexible? for example {:httpoison, "~> 1.7 or ~> 2.0"}
Describe alternatives you've considered
Forking this project?!

Hey @petemcfarlane,
Thanks for reaching out and for the suggestion. We have updated the httpoison dependency and will release it tomorrow.

Brilliant, thanks! 🤩


S23 refactor

refactored email validation in src/routes/auth.ts
added docker.dev & docker-compose-dev.yml for local development

Looks good


docs: update readme to show unused reason

What this PR does / why we need it?
PR Checklist

[ ] This PR adds K8s exceptions (false positives)
[ ] This PR adds new code
[ ] This PR includes tests for new/existing code
[x] This PR adds docs

GitHub Issue
Closes [XX-XX]

Notes for your reviewers

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 41.34%. Comparing base (ced2ef2) to head (6ae2f61).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #317   +/-   ##
=======================================
  Coverage   41.34%   41.34%
=======================================
  Files          61       61
  Lines        3212     3212
=======================================
  Hits         1328     1328
  Misses       1671     1671
  Partials      213      213

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


Package upload fails without error.

Customer reported issue:

Hello,
Whenever I try to upload an update (NabtoClientNative.4.1.4.nupkg), nothing happens after the progress bar reaches 100% - no text, no redirect, not anything. I can just try to upload again. The new update does not appear anywhere so it must have failed for some reason. I have tried in Chrome and Safari on a Mac and in Edge on Windows 10, exact same behavior. Any idea about what is going on?
Thanks!
Ulrik

couldn't find traces to investigate. Closing.


api/v1alpha1/zz_generated.deepcopy.go is not compliant with goimport format

Bug Report

What did you do?
Run build script for an operator and notice that command below generates a file non-compliant with goimports standards.
$ /home/fjammes/go/bin/controller-gen object:headerFile="hack/boilerplate.go.txt" paths="./..."
$ git status
On branch tickets/DM-29567
Your branch is up to date with 'origin/tickets/DM-29567'.

nothing to commit, working tree clean
$ goimports -w api/v1alpha1/zz_generated.deepcopy.go api/v1alpha1/zz_generated.deepcopy.go
$ git status
On branch tickets/DM-29567
Your branch is up to date with 'origin/tickets/DM-29567'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   api/v1alpha1/zz_generated.deepcopy.go

no changes added to commit (use "git add" and/or "git commit -a")
$ git --no-pager diff
diff --git a/api/v1alpha1/zz_generated.deepcopy.go b/api/v1alpha1/zz_generated.deepcopy.go
index e55de09..8a38007 100644
--- a/api/v1alpha1/zz_generated.deepcopy.go
+++ b/api/v1alpha1/zz_generated.deepcopy.go
@@ -21,7 +21,7 @@ limitations under the License.
 package v1alpha1

 import (
-	"k8s.io/api/core/v1"
+	v1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	runtime "k8s.io/apimachinery/pkg/runtime"
 )

What did you expect to see?
Line 21 in api/v1alpha1/zz_generated.deepcopy.go should be generated using format compliant with goimports
What did you see instead? Under which circumstances?
Line 21 in  api/v1alpha1/zz_generated.deepcopy.go is not generated using a format compliant with goimports. This might break static code analysis job in CI.
Environment
Operator type:

/language go

Kubernetes cluster type:

Kind
$ operator-sdk version
operator-sdk version: "v1.5.0", commit: "98f30d59ade2d911a7a8c76f0169a7de0dec37a0", kubernetes version: "1.19.4", go version: "go1.15.5", GOOS: "linux", GOARCH: "amd64"
$ go version
go version go1.16.3 linux/amd64

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.0", GitCommit:"9e991415386e4cf155a24b1da15becaa390438d8", GitTreeState:"clean", BuildDate:"2020-03-25T14:58:59Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.2", GitCommit:"faecb196815e248d3ecfb03c680a4507229c2a56", GitTreeState:"clean", BuildDate:"2021-01-21T01:11:42Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}
Possible Solution
Replace line 21 in api/v1alpha1/zz_generated.deepcopy.go by
v1 "k8s.io/api/core/v1"

in the go code generating this file.
Additional context

This import is written by controller-gen so needs to be fixed there. For now, a workaround is just run goimports on the project once controller-gen has been run (add this to your Makefile).
/triage support


modelnet40_ply_hdf5_2048.zip not available anymore

Hello there!
Unfortunately I cannot test your work beacuse the weights are not available anymore under this link:
https://shapenet.cs.stanford.edu/media/modelnet40_ply_hdf5_2048.zip
I've also searched mirrors but did not found anyone currently.
May I ask you to create a mirror and share it with us here or provide an alternative link maybe?
Thank you very much in advance!

The page is online again.
Just in case it will be unavailable again here is a mirror:
https://drive.google.com/file/d/14Gao2KbJeOCyaU67TgH2iKOJNpzzeKHt/view?usp=sharing


fixed issue #9862

Scope & Purpose
Fix Github issue #9862

[x] Bug-Fix for devel-branch (i.e. no need for backports?)
[ ] Bug-Fix for a released version (did you remember to port this to all relevant release branches?)
[x] The behavior in this PR can be (and was) manually tested (support / qa / customers can test it)
[x] The behavior change can be verified via automatic tests

Related Information

[x] There is a GitHub Issue reported by a Community User: https://github.com/arangodb/arangodb/issues/9862

Testing & Verification
This PR adds tests that were used to verify all changes:

[x] Added new integration tests (i.e. in shell_server / shell_server_aql)

https://jenkins01.arangodb.biz/view/PR/job/arangodb-matrix-pr/5963/

did we fix that RocksDB iterators are resettet twice when used from AQL ?

tests blue

No, at least I didn't address this in my PR.
Let me check that later today!


Type issue with custom matcher

I'm running into an issue implementing a custom matcher. I've followed the example in the README but TS complains about the type.
The makeRegexp function passed to makeCachedMatcher (or makeMatcher) is expecting a return of type RegExp, but the example outlines { keys: Key[], regexp: RegExp }.
const convertPathToRegexp = (path: string) => {
  let keys: Key[] = [];

  const regexp = pathToRegexp(path, keys, { strict: true });
  return { keys, regexp };
};

const strictMatcher = makeCachedMatcher(convertPathToRegexp);

results in
Argument of type '(path: string) => { keys: Key[]; regexp: RegExp; }' is not assignable to parameter of type '(pattern: string, keys?: { name: string | number; }[] | undefined) => RegExp'.
  Type '{ keys: Key[]; regexp: RegExp; }' is missing the following properties from type 'RegExp': exec, test, source, global, and 13 more.ts(2345)

Am I missing something here, or is this an issue? I'm happy to raise a PR if it's as simple as changing the return type of makeRegexp 👍🏼
Thanks for your work on this package too! It's really great and a nice light alternative to react-router with a flexible API.

FWIW I can navigate the type issue but I still get console errors:
const strictMatcher = makeCachedMatcher((path: string) => {
  const { regexp } = convertPathToRegexp(path);
  return regexp;
});

I think it gets confused by my custom ProtectedRoute component:
<Router matcher={strictMatcher}>
  <Switch>
    <Route path="/login" component={Login} />
    <ProtectedRoute path="/logout" component={Logout} />
    <Route path="/register" component={Register} />

    {/* Specific document */}
    <ProtectedRoute path="/:slug/documents/:id" component={Document} />

    <Layout>
      {/* List all root folders */}
      <ProtectedRoute path="/" component={Home} />
      <ProtectedRoute path="/teams" component={Teams} />

      {/* Sub-folders */}
      <ProtectedRoute path="/:slug" component={Folder} />
    </Layout>
  </Switch>
</Router>

which looks like this:
export const ProtectedRoute = (props: RouteProps) => {
  const {
    userStore: { isAuthenticated },
  } = useStores();

  return isAuthenticated ? <Route {...props} /> : <Redirect to="/login" />;
};

export default function makeMatcher(
  makeRegexpFn?: (
    pattern: string,
    keys?: Array<{ name: string | number }>
  ) => RegExp // here.
): MatcherFn;

PR welcome, 😂

Released in v2.7.4


Upgrade logging package to 1.2.0

Hi !
Thanks for this package, its very useful.
Could you just update the logging package to 1.2.0?
This causes conflicts in some projects.
Thank you very much

Great, thank you!


Fix mistype in output.md

Thank you @gorshunovr for the PR, but the website's content, specifically `content/'](https://github.com/terraform-docs/website/tree/main/content) folder has to be updated through the main repository [`docs/` folder. And the changes will be automatically pushed here in this repository.
I'm gonna close your PR, as it's already been updated through the main repo.


Feature request: Expose css selector to select flag

v5 made the flag images slightly bigger (26x26px instead of 20*20px).
We'd like to keep the UI somewhat consistent. Fix would be to style the flag using the existing selector
that is used internally (MuiTelInput-Flag)

Could the class name be exported as string constant, to have a fixed reference to the box so that consumers can safely style the flag, without the risk of the reference breaking in between version updates?
Just targeting any img within the component seems a little dangerous on the long run.
MUI helps in those cases by exporting class names, therefore my suggestion.
import { outlinedInputClasses } from "@mui/material/OutlinedInput";

Hello !
Not sure about the solution, please feel free to open a PR ;)


Change ILink's url to be optional

Currently, ILink is defined as an interface with two properties, a range and a url. Both of these properties are not optional. The DocumentLink in the language server protocol however is defined almost identically with a range and an optional target?. This seems reasonable given that the range of text to be clicked on should be identifiable visually. On the other hand, the actual "place" of where the link will lead to is not necessarily needed until the user hovers over it or clicks on it.
https://github.com/Microsoft/monaco-editor/blob/bad3c34056624dca34ac8be5028ae3454172125c/monaco.d.ts#L4790-L4793
If neither of the two properties in ILink are intended to be optional, then what is the point of the resolveLink function? If provideLinks is just going to make links that are fully usable, then what's the point of "resolving" a link?
If there are no plans to make changes to ILink, could some documentation please be added to LinkProvider then?

It is a typing mistake. It should be:
export interface ILink {
	range: IRange;
	url?: string;
}

The ILink will go through resolveLink if it is missing an url.


Error loading the demo site

Hi, when I try to load the demo site many components are failing.

Your app is having trouble loading the streamlit_shadcn_ui.py_components.utils.declare.card component.
(The app is attempting to load the component from ****, and hasn't received its "streamlit
" message.)
If this is a development build, have you started the dev server?
If this is a release build, have you compiled the frontend?
For more troubleshooting help, please see the Streamlit Component docs or visit our forums.

Same error here


generalize fromKeyedList

I didn't try any benchmarks, but I think that allowing multiple parents downgrades the performance guarantees.
Yes, the API will be more general, but you lose the n * log (n) guarantee, because then you allow the scenarios where "everything is a parent of everything" (which is already allowed by the original fromList API). In those cases you have potentially up to n*(n-1)/2 ~ O(n^2) connections between nodes. I think in such cases the linear lookup in the entire list (hidden behind the lazy nodes) is probably the better alternative compared to O(n^2) cost of initial construction.
On the other hand "everything is connected to everything" will probably not arise in practice, so I think it's ok to generalize the API in this way.

My thinking is that in both cases the implementation for cases where each item has just a single parent is Just isomorphism between list and maybe (List.singleton / Just | [] / Nothing). The potential cost for this case is just a function that translates the two (Maybe.withDefault [] << Maybe.map List.singleton) and call of List.foldr on singleton list. For cases where items have multiple parent we expect the function to get list of parrents to be constant (the same way the function to get maybe is) so the overhead here depends on the number of parents (for each of which we need to insert item to dictionary). I believe dictionary insert is log(n) and so yeah in case that everything is children of everything else overhead of this can because significant but it seems rather transparent from the API of the function.
I've ran benchmarks before and after the change and there was some insignificant overhead.
These are becnhamrk results after this change

These are benchmark results before the change

Maybe worth noting that in case of Maybe API this benchmark is just extracting the value from record while in case of List api there is the additional need to translate Maybe to List.
Less important thing for design of this library is that if you want to plan to use it in GWI then you'll definitely need consider items having multiple parents because questions can belong to multiple categories (See starter category and demographic category).
Additional note is that it's always possible to define custom function for construction within the users code because this library exposes all the primitives one needs to do so.

I will no longer have a chance to use this at GWI, as this is my last day and I'm on vacation for the past week :)
But definitely thanks for merging and documenting the improvement, I think it could still come in handy for some users of the library. The expansion of sub-trees definitely wasn't the bottleneck of the intended functionality (add all from category) we were implementing, I just came across this and tried it on the pathological case.

I see. I'll give it few more days before making final call on this. I'm honestly not sure what is the right call at the moment.

I've decided to keep the support for multiple parents in as it seems hard to justify not supporting the use-case which seems legit.


Cannot find module './routes/users/'

After running 'npm install' from the Ogar-client directory, and trying to run 'node app.js':
module.js:340
throw err;
^
Error: Cannot find module './routes/users'
at Function.Module._resolveFilename (module.js:338:15)
at Function.Module._load (module.js:280:25)
at Module.require (module.js:364:17)
at require (module.js:380:17)
at Object. (/home/joshua/Ogar-client/app.js:12:13)
at Module._compile (module.js:456:26)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:312:12)
at Function.Module.runMain (module.js:497:10)

Fixed the above by commenting out the related 'require' line in app.js, and now 'node app.js' runs but immediately exits with no errors.

Try adding a blank "users" file in the /routes/ directory. Worked for me, but I'm still having other problems.

Just starting looking into it. Some of the NPM dependencies have been renamed or are missing.


Compodoc and Angular GUI integrated in OASP4JS/Angular project

Bulgarian translation included.

Added to this PR a commit with a fix of an issue with the route params management from email confirmations. Also some dependencies have been updated.


Stream Compression config field

Overview
In 2.10, the compression field was added to the Stream Config.
It can take value:

"none"
"s2"

Clients and Tools

[x] Schemas @ripienaar
[x] CLI @ripienaar
[x] Terraform @ripienaar
[x] Github Actions @ripienaar
[ ] Go @piotrpio
[ ] Java @scottf
[ ] JavaScript @aricart
[ ] .Net @scottf
[ ] C @levb
[ ] Python @wallyqs
[ ] Ruby @wallyqs
[ ] Rust @Jarema @caspervonb

Other Tasks

[ ] docs.nats.io updated @bruth
[ ] Update ADR to Implemented
[ ] Update client features spreadsheet

Client authors please update with your progress. If you open issues in your own repositories as a result of this request, please link them to this one by pasting the issue URL in a comment or main issue description.

Note the options are lowercased - "none" and "s2" otherwise you get a JSON error.

Hmm.. Usually I do not like strings as enums..


Question about evaluating the pre-trained model for cityscapes dataset

Hello, when I evaluated the pre-trained model for cityscapes dataset, using the command: bash tools/dist_test.sh ./configs/Mobile_Seed/MS_tiny_cityscapes.py ckpt/MS_tiny_cityscapes.pth 1 --eval mIoU, the error occurred as follows:

Have you ever come across this problem? How did you solve it?

Thank you for reporting this error! I have fixed the code for multi-gpu test, please pull the latest code and try again!
tip:  if you are testing with a single GPU, you can also change the --launcher pytorch to --launcher none for single-GPU testing.

The evaluation results on the Cityscapes dataset should be close to this:

Thank you for reporting this error! I have fixed the code for multi-gpu test, please pull the latest code and try again! tip: if you are testing with a single GPU, you can also change the --launcher pytorch to --launcher none for single-GPU testing.

That works, thank you so much!!!!!!!!

Since the problem has been resolved, I will close the issue.


Priority: Remove call-in info for 3/31 meeting

Can this be prioritized? For the upcoming 3/31 Measure R meeting (https://boardagendas.metro.net/event/measure-r-independent-taxpayer-oversight-7159aebed1a8/), can you please delete the following text from the website that references submitting public comments by phone?:

There are no live public comments during the Measure R meetings (neither are there for Measure M and ICAOC).

@shrayshray Should we also disable the live ecomment link that appears during the meeting?

@shrayshray Ah, disregard, I forgot we hid ecomment links a while ago!

@hancush, perfect! Looks great to me!


Tensorflow C API Prebuilt Library for Ubuntu 20

Hi,
Currently, published Tensorflow C API Ubuntu libraries are  built for Ubuntu 16. Is there any plan to built it for Ubuntu 20?
When I try to link it against my Ubuntu 20 projects, I got linker errors such as:
.dynsym local symbol at index 857
...
/usr/bin/ld: Model.cpp:(.text+0x101): undefined reference to `TF_NewGraph'
/usr/bin/ld: Model.cpp:(.text+0x10a): undefined reference to `TF_NewSessionOptions'
/usr/bin/ld: Model.cpp:(.text+0x128): undefined reference to `TF_SetConfig'
/usr/bin/ld: Model.cpp:(.text+0x160): undefined reference to `TF_LoadSessionFromSavedModel'

Thanks.

@fatihkiralioglu
We see that the issue template has not been filled, we ask for the template as it helps us analyse the issue [ ex: tf version]
Can you please refer to these resolved issues and let us know if it helps: link, link1, #41506, #21301.

@Saduf2019 Hi,
The version is 2.4.0. I have downloaded prebuilt library from : https://www.tensorflow.org/install/lang_c
https://storage.googleapis.com/tensorflow/libtensorflow/libtensorflow-cpu-linux-x86_64-2.4.0.tar.gz
The issues you have shared does not resolve my issue. I can successully monitor .so objects with nm but cant link against my Ubuntu 20 project

@Saduf2019 There is a similar issue:
https://github.com/tensorflow/tensorflow/issues/41382

Thanks @Saduf2019, issue is resolved, I just fixed the linkage order in my make file. Best Regards


Support "constants" section in PDDL

See ipc-2004/pipesworld-no-tankage-nontemporal-strips for an example.

Fixed


Calling server function within another server function inside loader alters parameters unexpectedly

Describe the bug
When calling a server function inside another server function, the parameters get altered unexpectedly, causing the outer function to receive modified parameters. This issue prevents the function from being reusable as intended.
Your Example Website or App
N/A
Steps to Reproduce the Bug or Issue

Define two server functions using createServerFn:

export const serverFn = createServerFn('POST', async (params, ctx) => {
  console.log(params)
  await server2Fn(params)
  return null
})

export const server2Fn = createServerFn('POST', async (params, ctx) => {
  console.log(params)
  return null
})

Call the serverFn function with the following parameters:
// inside a loader
await serverFn({ slug: { eq: 'test' } });

Observe the console logs for params within serverFn:
Expected output:
{ "slug": { "eq": "test"} }

Actual output:
{
  "method": "POST",
  "payload": { "slug": { "eq": "test" } },
  "requestInit": undefined
}

Expected behavior
The createServerFn wrapper seems to be adding extra data to the parameters, such as method, payload, and requestInit. This behavior interferes with the intended parameter structure, making it difficult to reuse the server function properly.
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome
Version: "@tanstack/react-router": "^1.43.3", "@tanstack/start": "^1.43.3"

Additional context
No response

I found that simply replacing the following lines
https://github.com/TanStack/router/blob/bf52d5af9c80a99e1141c30ebb67999ecd4b24de/packages/start/src/client/createServerFn.ts#L85-L92
with
  return Object.assign(
    async (payload, opts) => {
      return compiledFn(payload);
    },

Solves my issue and doesn't cause issues with other server functions even when called in the react component. So I'm not sure why it's being called with those extra args.


Pretty examples

Make sure that there is a prettily styled version of all the components created. While it has been done for some of the solutions, it hasn't been done for all of them. This would help users know that it is possible to create custom designed versions of the components.

@alisonhall Can you break this up into a ticket for each component?

Hey @mattslavin , I was talking with Zoltan, and he said that this is related to what you had been discussing. Would you mind taking a look at this tickeT?


Update mobile-sdk-ios.md

If/how we can update page types to accommodate assessments outside of purchase, signup, signin and test?

minor feedback, should we keep consistent wording between "minSdkVersion: 21" for IOS and "Minimum Deployment Target: 12.4" for Android?