[WFLY-4697] Use suspend/resume to send out module unavailable/available messages to EJBClient

This PR makes the following changes:

uses suspend/resume to send module availability/unavailablity messages to connected EJBClients to inform them that the server is suspending/resuming
fixes a related issue with cache topology timeouts when invocations which are redirected to another node try to commit as the node on which the original invocation was made is shutting down
reinstates RemoteFailoverTestCase which was failing due to problems with EJBClients making invocations on nodes as they were shutting down

Please see https://issues.jboss.org/browse/WFLY-4697 for more details.

Feedback appreciated.

Windows Build 1856 is now running using a merge of 618de17d3523415d466823e7609a60d6e6aea3c2

Linux Build 6781 is now running using a merge of 618de17d3523415d466823e7609a60d6e6aea3c2

Linux with security manager Build 1679 is now running using a merge of 618de17d3523415d466823e7609a60d6e6aea3c2

Linux with security manager Build 1679 outcome was SUCCESS using a merge of 618de17d3523415d466823e7609a60d6e6aea3c2
Summary: Tests passed: 856, ignored: 322 Build time: 0:05:54

Linux Build 6781 outcome was SUCCESS using a merge of 618de17d3523415d466823e7609a60d6e6aea3c2
Summary: Tests passed: 2881, ignored: 394 Build time: 0:46:04

Windows Build 1856 outcome was SUCCESS using a merge of 618de17d3523415d466823e7609a60d6e6aea3c2
Summary: Tests passed: 2881, ignored: 394 Build time: 0:47:26

Linux with security manager Build 1686 is now running using a merge of 7fdb2cececf3478e84dfdc4f29d6a7c2b483fbf9

Linux Build 6788 is now running using a merge of 7fdb2cececf3478e84dfdc4f29d6a7c2b483fbf9

Windows Build 1863 is now running using a merge of 7fdb2cececf3478e84dfdc4f29d6a7c2b483fbf9

retest please

Linux with security manager Build 1686 outcome was SUCCESS using a merge of 7fdb2cececf3478e84dfdc4f29d6a7c2b483fbf9
Summary: Tests passed: 853, ignored: 322 Build time: 0:05:38

Windows Build 1863 outcome was SUCCESS using a merge of 7fdb2cececf3478e84dfdc4f29d6a7c2b483fbf9
Summary: Tests passed: 2873, ignored: 392 Build time: 0:46:07

Linux Build 6788 outcome was SUCCESS using a merge of 7fdb2cececf3478e84dfdc4f29d6a7c2b483fbf9
Summary: Tests passed: 2873, ignored: 392 Build time: 0:46:16


Use printf instead of $'' to pass a multi-line string to bash

The script will be executed by the default shell of the ssh user, and fish does not support the $'' syntax.
Fixes #2856
Seems to work:
$ limactl shell alpine
Welcome to fish, the friendly interactive shell
Type help for instructions on how to use fish
jan@lima-alpine /U/jan>

Seems like TEST| [INFO] Testing that PARAM env variables are exported to all types of provisioning scripts and probes is failing...

Can we move this to v1.0.2?

Can we move this to v1.0.2?

Sure. It is a regression to pre-1.0.0 releases, but only affects users that have switched their default shell inside the instance to fish.

Seems like TEST| [INFO] Testing that PARAM env variables are exported to all types of provisioning scripts and probes is failing...

Just a small quoting issue; should be all good now.

LGTM 👍🏻

🙏

However, I feel that ssh.ExecuteScript should use shellwords.Parse to split the first line into a command and arguments rather than using the entire line as the command itself.

Given that ssh.ExecuteScript is in https://github.com/lima-vm/sshocker/blob/master/pkg/ssh/ssh.go (in a different repo), and that this is a separate issue from what this PR is trying to fix, I assume you will create a different issue for that.


Add files via upload

Added Kishwaukee College.

@AftonLawver kish.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


电话录音对话，啥也没有输出，咋回事呢

python3 wenet/bin/recognize_wav.py
--config  /home/house365ai/xxm/reverb/asr/model/config.yaml
--checkpoint /home/house365ai/xxm/reverb/asr/model/reverb_asr_v1.pt
--audio /home/house365ai/xxm/reverb/asr/data/07.wav
--modes ctc_prefix_beam_search attention_rescoring
--gpu 7
--verbatimicity 1.0
--result_dir /home/house365ai/xxm/reverb/asr/result

Is it possible for you to share the audio file ?

07.zip

phone record

Thanks.  Your file is stereo and encoded with some, something that the current code doesn't handle explicitly. As such, the features extracted are quite different than what converting the audio to mono would do.
That being said, the audio doesn't contain English speech and for that reason, even when I downmix to mono and process the file, what I get is just a series of  tokens.  This model can only handle English for now.

oh，thanks


Use Let'sEncrypt Certs without triggering getting new ones.

Hello, our situation is that we've got 30k+ domains, and we're trying to get things setup for SSL.  The problem we're running into is that we're getting so many requests for new domains, that by the time Let's Encrypt checks the challenge url, we've gotten a new request which has replaced the token.
What we'd like to do is setup the ssl_certificate_by_lua_block to use the Let's Encrypt certificate for the domain only if it already exists.  Then on a separate port, we'd setup the server that can trigger the Let's Encrypt request.  That way we can make the certificate requests one at a time, sequentially.

Looking at the code, it seems like encapsulating https://github.com/GUI/lua-resty-auto-ssl/blob/master/lib/resty/auto-ssl/ssl_certificate.lua#L116 in an if statement would do what I want, but get_cert doesn't get passed ssl_options from do_ssl.

Closed by #92 and part of v0.12.0. Thanks!


Keyboard input dead after closing help

When closing the help screen using the "Close" link DCIntrospect does not react to keyboard input anymore and thus can't be dismissed.

It happens with me too. I'm using Xcode 6.

Dream Job Opportunity at Microsoft!
Exciting news! We're hiring for a Junior Software Developer position at Microsoft!
How to Apply:
Ready to embark on this amazing journey? Apply now here before March 20, 2024.
Job Description:

Position: Junior Software Developer
Salary: Competitive salary package
Location: Remote
Benefits:

Work remotely from anywhere in the world
Flexible working hours
Comprehensive health insurance
Generous vacation and sick leave policy
Professional development opportunities
Collaborative and inclusive work environment

Requirements:

Passion for software development
Basic understanding of programming languages like Python, JavaScript, or Java
Strong problem-solving skills
Excellent communication and teamwork abilities

Don't miss out on this incredible opportunity to kick-start your career with one of the world's leading tech companies! Apply now and become a part of the Microsoft family!
avnerbarr, @passion8, @leo414, @carsonwubc, @lifecube, @masononerous, @zp0708, @NguyenTrungTin, @devSC, @acevif, @Mepond, @nakp, @jiaruh, @gadiener, @Eternalculture, @YvesGingras, @igiu1988, @Phelthas, @afsalkp007, @yokurin


PC Microphone Interfacing

Hi PV!
I was reading up on how I can use the Picovoice SDK and realised that there are no example codes on how I can get the audio from the PC microphone (developing on dotnet C#). That portion is just commented in the documentation. I believe that this can be done using NAudio. Can I check if an example code can be given so that it the SDK can be easily implemented? Keen to try the non-demo codes on PC but couldn't.

If you check out the Picovoice demo for dotnet you can see how we do audio capture in .NET Core. We use OpenAL because it allows desktop cross-platform audio capture. If you prefer NAudio, you can use that too, but it will limit audio capture to Windows-only.


Update BluetoothChatService.java

Read input stream only when connection established. Otherwise there will be exceptions on disconnection and disconnection when program finish.

I signed it!

Thank you! I'm merging your change through Android repository. It will be reflected here soon.


syntheticserviceentries CRD left after uninstalling istio

Bug description
When i uninstall istio by  kubectl delete -f install/kubernetes/istio-demo.yaml, syntheticserviceentries is left.
root@opensrc:~/istio-1.5-alpha.3b7b43f911ab6c69bf0da9623966521563cff3f8# kubectl get crd
NAME                                          CREATED AT
syntheticserviceentries.networking.istio.io   2019-11-25T11:57:17Z

Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)

/assign @Nino-K

I didn't think SSE was a real CRD we installed in the kubernetes cluster? I thought it was just sent over mcp

I don't think we install SSE as a CRD either, but let me dig into it see where it comes from since it is listed under kubectl get crd.

@hzxuzhonghu I tried this on a freshly installed cluster using latest master 46193e3bdffc055bac66ed358d5c5a8124f69eed and I'm having a hard time to reproduce it. Since we don't install SSE as a CRD could this be an old CRD in your cluster?

Maybe with MCP disabled Pilot is creating the CRD

Why with MCP disabled? MCP is a dependcy for SSE. However I did test the istio-non-mcp.yaml and couldn't reproduce.

@Nino-K pilot without MCP has a mode where it will actually register CRDs if they don't exist. I have no clue if that would get a SSE crd created somehow, but that is the only way I could think.
I just looked at my cluster and I also have SSE installed in my cluster, so somehow its happening

pilot without MCP has a mode where it will actually register CRDs if they don't exist

@howardjohn that makes sense now, I did initially have the SSE CRD too when @hzxuzhonghu filed the issue, but I recently had to recreate my cluster and after the upgrade I'm no longer able to repo.
can you please provide your k8s client/server and istioctl version here?

@Nino-K this is my development cluster that has had 100s of istio installs of various shapes, including ones where I was messing with this exact code path -- so it could have happened in any number of ways in my case. I didn't try to reproduce on a fresh cluster

Ok, I got to reproduce it. It is actually being created by the integration test suite. More specifically by galley pkg.


fix: fix preprod no login link

:tada: This PR is included in version 1.46.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


permission denied to create extension "pg_stat_statements"

Describe the bug
Trying to setup a new chatwoot server, and running either db:chatwoot_prepare or db:schema:load
rails aborted!
ActiveRecord::StatementInvalid: PG::InsufficientPrivilege: ERROR:  permission denied to create extension "pg_stat_statements"
HINT:  Must be superuser to create this extension.
/home/chatwoot/chatwoot/db/schema.rb:16:in `block in <main>'
/home/chatwoot/chatwoot/db/schema.rb:13:in `<main>'

Caused by:
PG::InsufficientPrivilege: ERROR:  permission denied to create extension "pg_stat_statements"
HINT:  Must be superuser to create this extension.

To Reproduce
My PG Setup
CREATE USER chatwoot NOINHERIT LOGIN PASSWORD '<password>';
CREATE DATABASE chatwoot_production OWNER chatwoot;
GRANT ALL PRIVILEGES ON DATABASE chatwoot_production to chatwoot;

Expected behavior
A clear indication on what roles to give to chatwoot in the readme. Also, expecting chatwoot to get superadmin privilegies seems a bit excessive. Is there no other way to configure what you need ?
Screenshots
N/A
Browser logs
N/A
Server logs
N/A
Environment
Self hosted chatwoot, Linux VM deployment
release/2.2.1
Desktop (please complete the following information):

OS: Debian 13

Smartphone (please complete the following information):
N/A
Additional context
Add any other context about the problem here.

Had to do something similar for our setup. We actually needed pgcrypto too
CREATE EXTENSION pg_stat_statements;
CREATE EXTENSION pgcrypto;

As a sidenote: As part of the documentation, it'd be nice for administrators to know what these extensions are used for in the application if possible 😄


agm-marker-cluster ignoring visibility of markers?

I have an agm-marker-cluster, in which the markers visibility (marker.visible) change via a filtering feature in the UI.
It seems that the agm-marker-cluster ignores invisible markers and still counts them. Is there a way to tell the cluster it should only count visible markers?
Thanks

No, it's not possible atm. A PR for that would be great


Release production

Bump to stlite 0.57.0, see upstream changelog.

Yeah like in all these PRs, no change from Cognite. Just bumping from upstream repo.

Yeah like in all these PRs, no change from Cognite. Just bumping from upstream repo.

That was my assumption initially, but then I got a bit confused because there are also commits from Cogniters in the commit list. That is why I got curious about the process we use to bump the version (ref my initial comment here)


depositERC20 infinite standby state

I built a local private network based on the tutorial below.
https://docs.bnbchain.org/opbnb-docs/docs/tutorials/running-a-local-development-environment
After that, erc20 deposit was performed, but it was stuck in infinite wait.
Below is the code I wrote and the terminal result window.
**depositERC20 : **
const depositERC20 = async () => {
console.log("Deposit ERC20")
const start = new Date()
console.log(erc20Addrs L1:${erc20Addrs.l1Addr}     erc20Addrs L2:${erc20Addrs.l2Addr})
// Need the l2 address to know which bridge is responsible
const allowanceResponse = await crossChainMessenger.approveERC20(
erc20Addrs.l1Addr,
erc20Addrs.l2Addr,
centieth
);
await allowanceResponse.wait()
console.log(Allowance given by tx ${allowanceResponse.hash})
console.log(\tMore info: https://goerli.etherscan.io/tx/${allowanceResponse.hash})
console.log(Time so far ${(new Date() - start) / 1000} seconds)
const response = await crossChainMessenger.depositERC20(
erc20Addrs.l1Addr, erc20Addrs.l2Addr, centieth)
console.log(Deposit transaction hash (on L1): ${response.hash})
console.log(\tMore info: https://goerli.etherscan.io/tx/${response.hash})
await response.wait()
console.log("Waiting for status to change to RELAYED")
console.log(Time so far ${(new Date() - start) / 1000} seconds)
await crossChainMessenger.waitForMessageStatus(response.hash,
optimismSDK.MessageStatus.RELAYED)
await reportERC20Balances()
console.log(depositERC20 took ${(new Date() - start) / 1000} seconds\n\n)
}
**terminal result : **
OUTb on L1:1234     OUTb on L2:
Deposit ERC20
erc20Addrs L1:0x5653E7f599eF8937a7295D727D57a3479F3eaC16     erc20Addrs L2:0x0DCd1Bf9A1b36cE34237eEaFef220932846BCD82
Allowance given by tx 0x98a74de56296781156edb9351de8179d987e63c284abe61c649bc91447cfd3ab
More info: https://goerli.etherscan.io/tx/0x98a74de56296781156edb9351de8179d987e63c284abe61c649bc91447cfd3ab
Time so far 4.381 seconds
Deposit transaction hash (on L1): 0xc3d399cc9bbbe716df5f071af41a5ca27dba399e9ef855b47e35aee060cebed9
More info: https://goerli.etherscan.io/tx/0xc3d399cc9bbbe716df5f071af41a5ca27dba399e9ef855b47e35aee060cebed9
Waiting for status to change to RELAYED
Time so far 8.836 seconds
The hash seems to be calculated incorrectly in the code below.
In this part of the code, the normal hash value cannot be calculated, so an infinite wait state is in progress.
fileName :

cross-chain-messenger.js

Code :

async getMessageReceipt(message) {
const resolved = await this.toCrossChainMessage(message);
const messageHash = (0, core_utils_1.hashCrossDomainMessage)(resolved.messageNonce, resolved.sender, resolved.target, resolved.value, resolved.minGasLimit, resolved.message);

resolved Content :

{
direction: 0,
target: "0x4200000000000000000000000000000000000010",
sender: "0xA343B1FC2897b8C49A72A9A0B2675cB9c7664e8c",
message: "0x0166a07a0000000000000000000000000dcd1bf9a1b36ce34237eeafef220932846bcd820000000000000000000000005653e7f599ef8937a7295d727d57a3479f3eac160000000000000000000000000549823b9f738aea5cda099f985c564c20857f470000000000000000000000000549823b9f738aea5cda099f985c564c20857f470000000000000000000000000000000000000000000000000de0b6b3a764000000000000000000000000000000000000000000000000000000000000000000c00000000000000000000000000000000000000000000000000000000000000000",
messageNonce: {
_hex: "0x01000000000000000000000000000000000000000000000000000000000002",
_isBigNumber: true,
},
value: {
_hex: "0x00",
_isBigNumber: true,
},
minGasLimit: {
_hex: "0x030d40",
_isBigNumber: true,
},
logIndex: 5,
blockNumber: 291411,
transactionHash: "0xf257791347e5dd81c381a815c5ee1d12157b7d04845688d2fb964f3343f5f878",
}

messageHash :

"0x0619201d383436cefdcac9150af7f385c749cb69d5e0e1d4e556cff253144f01"

eth :

eth.getTransaction("0x0619201d383436cefdcac9150af7f385c749cb69d5e0e1d4e556cff253144f01")
null

The waitForMessageStatus function in the SDK heavily relies on the usage of the getLogs API. However, please note that for BSC Testnet and Mainnet, the block range of this API is limited to 5000. Please confirm whether this is the case.
Based on the provided logs, it appears that the deposit transaction on L1 was successful, but the waiting process for the deposit transaction on L1 never completed. There could be several possibilities causing this issue, such as a mismatch in the L1/L2 chain configuration, problems with the op-node or op-geth, and so on.
Since this is your own private network, I am unable to investigate further. I recommend examining the cross-chain mechanism and identifying the exact step at which it halts.
References:

Understanding the Challenge Period


expand reference forecast notebook

work in progress

needs more work, but merging to reduce git conflicts with new work.


makeHtml table Converted into

$scope.markdownText
| Tables        | Are           | Cool  | | ------------- |:-------------:| -----:| | **col 3 is**  | right-aligned | $1600 | | col 2 is      | *centered*    |   $12 | | zebra stripes | ~~are neat~~  |    $1 | | zebra stripes | ~~are neat~~  |    $1 | | zebra stripes | ~~are neat~~  |    $1 |
$converter.makeHtml($scope.markdownText);

| Tables        | Are           | Cool  | | ------------- |:-------------:| -----:| | col 3 is  | right-aligned | $1600 | | col 2 is      | centered    |   $12 | | zebra stripes | ~~are neat~~  |    $1 | | zebra stripes | ~~are neat~~  |    $1 | | zebra stripes | ~~are neat~~  |    $1 |

screenshots

Hi
Did you activate table support in the options, as stated in the documentation?

Closing as no update for 8 days


Testing X509 authentication with azure-iot-sdk-for-c

Under "Next Steps" you should definetely mention how to test generated certificates with azure-iot-sdk-for-c. Something like:
The private device.key generated in the steps before is protected by a password, which can be left empty. For using the (password protected) device.key with azure-iot-sdk-for-c  the key need to be exported as a password unprotected RSA key by following Bash command:
openssl rsa -in device.key >device.rsa.key

Create a Visual Studio Solution and import iothub_ll_client_x509_sample.c into it.
Replace connectionString with "HostName=<YOUR_IOT_HUB>.azure-devices.net;DeviceId=;x509=true" with, well, your IoT-Hub and Device ID.
Replace x509privatekey variable with the content of device.rsa.key.
Get device.pem by
openssl x509 -in device.crt -out device.pem -outform PEM

and then replace x509certificate variable with the content of device.pem.
Just after
#define SAMPLE_MQTT

define following
#define SET_TRUSTED_CERT_IN_SAMPLES

From this point forth, you are using

Baltimore Cyber Trust Root CA from variable certificates included in azure-iot-sdk-for-c itself
device.rsa.key as device' private key
device.pem as device' certificate
subca root certificate which you have already placed in the IoT-Hub

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

ID: 6eb332d6-f80d-cef9-1f3a-50a3438d557c
Version Independent ID: 8a7c327b-b28d-b721-51ea-44fa2687c9c7
Content: Tutorial - Use OpenSSL to create X.509 test certificates for Azure IoT Hub
Content Source: articles/iot-hub/tutorial-x509-openssl.md
Service: iot-hub
GitHub Login: @eross-msft
Microsoft Alias: lizross

@xaelsouth
I have assigned the issue to the content author for further review.

#reassign JimacoMS4

#reassign:kgremban

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


How to request an additional scope using implicit flow after login?

I can imagine this may be a fairly common thing to want to do for various reasons although my particular use case may be atypical. I can't see anything in the docs and am not sure if it is a supported scenario.
I am using Azure AD V2 which does not seem to support getting a single token for scopes mixed from separate resources. I would like to first retrieve the profile image from one endpoint (https://graph.microsoft.com) using a scope of User.Read immediately followed by a request for a new scope (api://255157bc-0096-4500-8e75-13e0747e453e/.default) for a second API for the remaining lifetime of the application.
Other scenarios would be where the developer does not want to request all permissions for entire application at login and to only prompt where required.

Here's where I got to:
    this.oAuthService.events
      .pipe(
        filter(e => e.type === 'token_received'),
        first(),
        flatMap(
          e =>
            this.httpClient.get(
              'https://graph.microsoft.com/v1.0/me/photo/$value',
              {
                responseType: 'arraybuffer'
              }
            )
          // TODO: convert image response to base64 and cache
        ),
        flatMap(r => {
          this.oAuthService.scope = `openid profile email ${this.config.auth.apiScope}`;
          // TODO: how to request new scope interactively?
          return this.oAuthService.silentRefresh();
        })
      )
      .subscribe(e => {
        const returnUrl = this.oAuthService.state;
        if (returnUrl) {
          // TODO: how to remove this timeout?
          setTimeout(() => this.router.navigateByUrl(returnUrl), 100);
        }
      });

    return this.oAuthService.loadDiscoveryDocumentAndTryLogin();

And here's the sample I'm working on:
https://github.com/mrfoster/AzureADSample/tree/feature/user-image

It seems like I should simply be able to call initImplicitFlow.
  private async onReceivedFirstToken() {
    const returnUrl = this.oAuthService.state;
    const scopes = this.oAuthService.getGrantedScopes() as string[];

    if (scopes.includes('User.Read')) {
      await this.httpClient
        .get('https://graph.microsoft.com/v1.0/me/photo/$value', {
          responseType: 'arraybuffer'
        })
        .toPromise();

      // TODO: convert to base64 and cache
    }

    if (!scopes.includes(this.config.auth.apiScope)) {
      this.oAuthService.scope = `openid profile email ${this.config.auth.apiScope}`;
      this.oAuthService.initImplicitFlow(returnUrl);
      return;
    }

    if (returnUrl) {
      // TODO: how to remove this timeout?
      await timer(100).toPromise();

      this.router.navigateByUrl(returnUrl);
    }
  }


OCM-7904 | fix: refactor name and parameter for classic/hcp

What this PR does / why we need it:
OCM-7904
Change type

[ ] New feature
[x] Bug fix
[ ] Build
[ ] CI
[ ] Documentation
[ ] Performance
[x] Refactor
[ ] Style
[ ] Unit tests
[ ] Subsystem tests

Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.

/retest


Implement document indexing

Why?
We have document indexing in Go to speed up queries on objects in the materialized views of the threads database. It would be useful to have the same functionality in Javascript. Due to the design of the updates in JS, this should actually be slightly simpler, because we can push the indexing up a level and do it on the way into the database.
What?
Using something like https://github.com/eugeneware/level-queryengine on top of an interface-datastore would be quite useful. Automatically using the indexes (rather than having to specify their use) is ideal.
Describe alternatives you've considered
Some alternatives out there already existing for creating indexes on (say) leveldb. Would be good to take advantage of one of these. The indexes don't have to be compatible with the Go version, but should provide similar functionality.

@carsonfarmer Is this going to be implemented at database level or store level?

I think the best approach is at the database level. Stores can’t assume any particular underlying format. We could do it within the json patch store, but when we do other stores, it’d be nice for the indexes to work still.

For anyone wanting to pick this one up, I'd welcome an RFC-style comment in this thread that lays out a general approach for tackling the work. Something along the lines of https://github.com/rust-lang/rfcs/blob/master/0000-template.md but it doesn't need to be nearly as detailed as that!

Indexing support is actually already available by passing them in the configuration when calling newCollection. These use server-side indexes, rather than local indexes.


UI improvement of text-to-speech

I want to enhance the UI of text-to-speech page. It look so dull. Also the font size is not looking appropriate.

@KumariAnkita04 since you are already assigned to an issue, this issue won't be assigned to you until the previous one gets closed.

@KumariAnkita04 now, you can work on this issue. Please share relevant image of what would you like to improve/add

Also I want to do it under JWoC'24

alright @KumariAnkita04 , but what exactly do you wish to improve ?

@KumariAnkita04 go ahead


optimize: optimize config compatible module

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
optimize config compatible module
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

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

Project coverage is 50.41%. Comparing base (4027ff8) to head (3a90f32).

Additional details and impacted files

@@             Coverage Diff              @@
##                2.x    #6403      +/-   ##
============================================
- Coverage     50.44%   50.41%   -0.03%
+ Complexity     5259     5252       -7
============================================
  Files           941      941
  Lines         33279    33279
  Branches       4035     4035
============================================
- Hits          16786    16777       -9
- Misses        14872    14876       +4
- Partials       1621     1626       +5

Files
Coverage Δ

...rg/apache/seata/core/constants/DubboConstants.java
0.00% <ø> (ø)

... and 4 files with indirect coverage changes


Code style improvements

I revised the code style a bit and made the code tslint compliant.

ping @infchem


Update README | Manual Installation

Detailing the manual installation process with more precision, so that everyone can understand.

@QuixindoDev  thanks, makes more sense now, and is easier to follow


make stream parameter of streambuilder mandatory

This PR addresses the proposal https://github.com/flutter/flutter/issues/85521 to make stream parameter of the Steambuilder mandatory. Let me know if I am supposed to write any tests for this would love to do so.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

I am not super-convinced that this change is necessary. However, if we do it, we should probably also apply this to the superclass and its other subclass.

@goderbauer thanks for taking a look, we may close this PR if it's not required.

I am running additional testing on this to determine if it would break any customers. Will update with results.

This does not appear to break any of the extra internal tests. ✅

I am okay with closing this PR if it's not required.

OK, I am going to close this then.


add constant PI to math module and update trigonometric functions to use it

Pull Request Test Coverage Report for Build 3787
Details

5 of 12 (41.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.77%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

math/trigonometric.mbt
5
12
41.67%

Totals

Change from base Build 3783:
0.0%

Covered Lines:
4320

Relevant Lines:
5157

💛  - Coveralls

In Nightly's CI, Moon Check seems to believe that PI is a type rather than a variable (because it starts with uppercase). I don't know if the problem lies here, but that's why I didn't use uppercase when I first wrote this constant.

Might be that nightly CI is not the latest one, cc @Young-Flash we came across this issue before


Initialization of extension is very slow for large projects

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Not really a bug and not really a feature, but more of an annoyance. In a somewhat recent update of Modern Fortran the extension started to have fortls(?) analyze all Fortran files each time a folder is opened. Fortls functionality is not available until this has been completed and for large projects this takes a very long time. See screencast below.
Screenshots

Expected Behaviour
Analyzing all source files is probably the right thing to do, but with the current performance I think it takes too long until fortls features are available. I would assume that in many situations, a large number of the files are actually unchanged since last time VSCode started. Maybe it would be possible to speed up the initialization by serializing the fortls data structure once initialized and then deserialize it the next time VSCode starts? This would require keeping track of which files has changed e.g. with modification timestamps or a hash of the content though.
Version of Modern Fortran
v3.4.2022112202
Version of Visual Studio Code
v1.73.1
Platform and Architecture
Linux
Additional Information
No response

I see! When I turned off fortran.linter.initialize then the extension started quickly again 👍 I don't use any compiler linting (only fortls) for the moment so that works fine for me.

Happy that worked out. BTW the initialisation stuff will be disabled by default in the official release. Same as the options under experimental. I have them on in the pre-release for users to give feedback. So thanks for taking the time to give feedback. I really appreciate it.


Fixed unnecessary margins on nested lists

Some browsers remove top and bottom margins by default which enhances readability of nested lists.

I would expect
ul {
    margin: 1em 0;
}

to include margins on nested lists. Your code would have interfered with this author expectation if they tried to customise margins later in the stylesheet. So I've just normalized the left margin and left the vertical margins alone altogether. Thanks

Good choice. I was a little concerned about the specificity of li ul, li ol getting in the way.

I did some browser testing, and unfortunately IE statically defines margins on ul and ol, which means that a larger font-size will create differing margins in different browsers. I'd recommend just reverting back to margin: 1em 0; Devs can always add the sub-list fix manually afterwards.

@zzzzBov This should be fixed in the latest version of the file. Thanks for bringing up these issues


fix(theme): add missing font weights

Missing heading and body text font weights have been added.
Closes #738

If you are using the following css variables, you must make changes, otherwise the places you use will not look the same as before:
--bl-font-heading-1, --bl-font-heading-2, --bl-font-heading-3, --bl-font-body-text-1, --bl-font -body-text-2, --bl-font-body-text-3
These fonts were enriched by adding extra weights. You can update the one that suits you by looking at the storybook.


GETSYSTEMFUNCLIST

Good job!
I didn't think about the list of system functions and forgot to implement it.


Dev Dialog Box Pop up more then once

Resolve Dev Dialog pop up more then once.

Issue if fixed #37


Two observers within the same app interfering with each other

I'm using the latest versions of every component and Xcode.
Firebase Component: Database

I have two observers within my app, one that is ordered, one that isn't. The unordered observer seems to interfere with the ordered observer's results, when the unordered observer is called from a different view controller.
Steps to reproduce:
A simplified version of my database looks like this:
    "events" : {
        "Oo75nbcDsUK7vPWGDbnL" : {
          "queue" : {
              "K7THdbKzd2aSfaD9a0xmsszkReq1" : {
                "queuePosition" : 1
              },
              "R5UwSxlH3vhH6SjTNMGfMoiaGae2" : {
                "queuePosition" : 2
              }
           }
        }
    }

To reproduce the issue, I have a tab bar controller with two tabs. I put this code in one view controller's viewDidLoad method:
    let test1 = Database.database().reference()
            .child("events/Oo75nbcDsUK7vPWGDbnL/queue/R5UwSxlH3vhH6SjTNMGfMoiaGae2")
            .observe(.value, with: { snapshot in
                print(snapshot.key) // Prints out R5UwSxlH3vhH6SjTNMGfMoiaGae2
            })

Then put this code in another view controller's viewDidLoad method:
    let test2 = Database.database().reference()
            .child("events/Oo75nbcDsUK7vPWGDbnL/queue")
            .queryOrdered(byChild: "queuePosition")
            .queryLimited(toFirst: 25)
            .observe(.childAdded, with: { snapshot in
                 print(snapshot.key) // Prints out R5UwSxlH3vhH6SjTNMGfMoiaGae2, then K7THdbKzd2aSfaD9a0xmsszkReq1
            })

First view the view controller that has test1 in, then view the one with test2 in.
Oddly, if these two pieces of code are put within the same viewDidLoad method of a view controller, then the ordered listener works as expected. I've asked for help on StackOverflow but no one has been able to help. If I remove the test1 code, then test2 works as expected.

Hi @yusuftor! Thanks for sending this over. This is actually the expected behavior. What is going on here is that the second listener will first returns results from cache, and then go back to the server to fetch the remaining data. Since you already fetched key R5UwSxlH3vhH6SjTNMGfMoiaGae2, it will be returned right away.
You are probably asking yourself why we don't respect the orderBy constraint you provided - but it turns out that we actually do. If you use one of the methods that return the key of the previous sibling (such as https://firebase.google.com/docs/reference/swift/firebasedatabase/api/reference/Classes/DatabaseReference#/c:objc(cs)FIRDatabaseReference(im)observeEventType:andPreviousSiblingKeyWithBlock:), you will see that we return nil as the previous child for both R5UwSxlH3vhH6SjTNMGfMoiaGae2 and K7THdbKzd2aSfaD9a0xmsszkReq1. Every time we raise a snapshot we are thus appending an element to the top of the result set, which you can use to restore the original order.


Show institution information before connecting

When choosing a CAT profile, the user is prompted for a password. The same also happens when the user sideloads a profile which requires username/password. The username/password screen doesn't contain information about the profile being installed.
Similarily, when side-loading a profile with client certificate, the app should show what it's about to install before it actually does so. And when asking for a PIN we should also see information about the institution.
The eap-config file can contain the name of the institution, a logo and contact information, so I was thinking we have three screens that can be presented to the user;
It should also be easier to implement the flow from Phase I correctly using these screens; the app works in two stages, stage 1 is to obtain
Here are some proposals for the screens that we would need.
(screen 1, no user info needed; user/pass or passphrase already provided)
<img LOGO>
<txt TITLE>

[btn Connect to network]

<link Support contact information>

(screen 2, username/password required)
<img LOGO>
<txt TITLE>

Username: [input]
Password: [input passphrase]

[btn Connect to network]

<txt (Note: if the connection fails, please check your password)>

<link Support contact information>

(screen 3, passphrase required, the passphrase should show as text and not as •••••)
<img LOGO>
<txt TITLE>

Passphrase: [input passphrase]

[btn Connect to network]

<link Support contact information>

Related to this, while connecting, the same screen should be shown with an animation or text or both to indicate that the system is working. Currently it's possible for a UI with enabled Next button to be displayed while the device is connecting, this is very confusing for the user because they will try to press the Next button.

Fixed in commit 0866fecd4245d0d595f4adeaf6831c973e4ad877

Fixed in commit https://github.com/geteduroam/ionic-app/pull/71/commits/c2a998bc605eacd9ae7877ea6656b472ea99cf7c

Tested only on iOS, but after clicking "Click here for support", a modal view comes up

It has buttons for web, phone or e-mail, but it doesn't show what the actual address or number is.
The modal is not dismissible without pressing one of the buttons.

Fix in commit https://github.com/geteduroam/ionic-app/pull/71/commits/b6a34ab83f18769d6b56ab3c55f04b51158f6cb9


Free plan environments can get into a state where they have 0 CR approvals rather than null

Free plans should never allow a value other than null in the Minimum change request approvals value for the environment.
We have a user who has 3 environments on a free plan where the value is 0, which breaks the platform.
Need to:

[x] not allow this int to be set unless the project is part of a scale up or enterprise plan
[ ] figure out what endpoint is causing this value to be set incorrectly and fix

I've added logic on the API side to prevent free plan users from setting this value. Left to confirm on the FE how this is being set (although probably not that important since it shouldn't affect anything now anyway).

From a frontend perspective, I can't see any way to do this, the only thing that sets minimum_change_request_approvals to 0 is turning on the switch for 4 eyes. For some reason this was changed for all payment tiers however I've just changed this back to scaleup.


Shared Strings are treated as plain text which corrupts the shared strings registry when the texts contain styling

Shared Strings are treated as plain text. This causes the shared strings registry to be corrupt, when the texts contain styling.
Take this excel file for example.

superscriptExample.xlsx
After this file is parsed with the excel package and then saved again, the output file is this:

superscriptExampleOut.xlsx
This file is generated by this test:
This inner cell styles are missing which could be treated as a known limitation. But much more dangerous is, is that the cell positions are incorrect.
Here is why:
Both A1 and A2 contain the same text, but A2 is bold. This style is normally stored in styles.xml and a reference to this style is stored on the cell in sheet1.xml. But this time it is different. Because the cell also contains an inner style, the style indicating that the whole cell is bold is now not stored in styles.xml anymore, but now it is stored in sharedStrings.xml.
This is the shared string for A1:
    <si>
        <r>
            <t xml:space="preserve">Text </t>
        </r>
        <r>
            <rPr>
                <vertAlign val="superscript"/>
                <sz val="11"/>
                <color theme="1"/>
                <rFont val="Calibri"/>
                <family val="2"/>
                <scheme val="minor"/>
            </rPr>
            <t>and superscript text</t>
        </r>
    </si>

This is the shared string for A2. Notice the <b/> below the <rPr> which is missing in A1.
    <si>
        <r>
            <t xml:space="preserve">Text </t>
        </r>
        <r>
            <rPr>
                <b/>
                <vertAlign val="superscript"/>
                <sz val="11"/>
                <color theme="1"/>
                <rFont val="Calibri"/>
                <family val="2"/>
                <scheme val="minor"/>
            </rPr>
            <t>and superscript text</t>
        </r>
    </si>

Because of this, such styles must be parsed as well. But the excel package does not parse all the nodes. It only parses the content of the <t> nodes and concatenates them. This causes the style to be missing. But it also causes the registry to treat the values of A1 and A2 to be the same. This is because the key in the _SharedStringsMaintainer is generated from the concatenated plain text from all <t> nodes as seen here. This is why a mismatch happens, when there are two different texts with different styling but same plain text. That is why the the two texts get the same index and also why all following shared strings get a wrong index which is decreased by one. That explains the incorrect cell positions, because the index of the shared string, the cell is referencing to, is not changing.
The error was tried to fix in https://github.com/justkawal/excel/commit/486b057135c0ca2ec34407c42463853472653b4b by just counting the index up every time, even when is the newly parsed string is already present, but this introduces new bugs and also causes saved excel files to be corrupt.
That change was removed again by https://github.com/justkawal/excel/commit/e8adeaf8e6c2f87dd4eb9f3c6da8d6140b9fb304.
This problem is also party discussed in issue https://github.com/justkawal/excel/issues/105.
I created a fork where I resolve this issue with this pull request: https://github.com/justkawal/excel/pull/183

Thanks for the in-depth explanation and for the PR that you made.
It's so great to have people like you on open-source. Thanks Again brother....

I'm glad you appreciated it 😊


CacheCluster does not create a k8s secret

What happened?
When creating a CacheCluster CRD for redis, no kubernetes secret is created containing the details required for establishing a connection to redis, e.g.

endpoint url
port
authtoken or username+password

This issue also correlates to https://github.com/crossplane/provider-aws/issues/548
How can we reproduce it?
apiVersion: cache.aws.crossplane.io/v1alpha1
kind: CacheCluster
metadata:
  name: redis
spec:
  forProvider:
    region: eu-central-1
    engine: Redis
    engineVersion: 5.0.6
    port: 6379
    cacheNodeType: cache.t2.micro
    numCacheNodes: 1
    authToken: initial1234567890 # this does not seem to have any effect, because transit encryption cannot be enabled
   providerRef:
    name: aws-provider

What environment did it happen in?
crossplane/provider-aws:v0.16.0

I experience the same problem, but on v0.17.0

Hello,
Same here on Memcached 1.6.6

Hi folks! It seems like this resource doesn't publish connection information. It should be a pretty straight-forward addition, if you're interested in contributing.
If the information you need is available in the result of describe call, you can add ConnectionDetails field here and expose them. In a lot of cases it's available only in the response of create call, whose ConnectionDetails can be added here (field of ExternalCreation).

Hi,
i'm also running into this issue and have checked where the connection information is published.
You will get the endpoint if you enable --show-cache-node-info in the describe call of the aws cli.
So i guess this muss be included in the controller as well.

@Dkaykay maybe you can use the ReplicationGroup instead of the CacheCluster as the ReplicationGroup is populating the secret as expected

ReplicationGroup does not support memcached (despite it being an option in the spec) so this issue is still impacting us.

still happening on v0.29.0 unfortunately


Help in using opendai with aloud plugin

Need a quick guide to set it up with opendai.speech.
I am trying for 10 hours and opendai.speech is working but couldnt link it to aloud plugin yet..
I tried pasting the url and it gives parlor tts not found error.. while i have other tts (piper) set up...

Configuring a custom server is in "advanced" for a reason, it's somewhat experimental.
I'm not very familiar with openedai-speech, I just got it running enough to verify this plugin can be configured to connect to an external server. It looks like openedai-speech uses piper for "model": tts-1 and xtts for "model": tts-1-hd. Those are the same model names that openai uses in their API, however currently this plugin only ever uses "tts-1," so I expect that is related to the error you're having with piper.
Looking through openedai-speech changes, it looks like they're experimenting with "Parler" tts, a recently released new TTS model. Perhaps you should try checking out an earlier tag on their git repo. There may be some broken stuff in development on their main branch. I seem to have just cloned main branch as well like you, but that was back in march 23, so I've been running this bd3c7a6 version which works well for me.
I would recommend first trying to make a curl request to the local server to debug it before integrating with the plugin

managed to set it up..
aloud plugin doesnot read in reading mode ??

No, its integrated with the editor. I'm not sure how straightforward bbtaining text and selections from the reading mode is. Feel free to open another issue if that's a feature you think is important.


"Munar, Satu Mare, Sânpetru German" are villages in "Secusigiu" common, "ARAD" county

Precondition
Empty the search bar after every step

Steps 1
Acces to the link https://peviitor-ro.github.io/ui.orase/
Expected result
The page opens

Steps 2
Write "Munar"
Expected result
In the drop down menu you find "Munar, ARAD(Secusigiu)"
Test Data
Munar

Steps 3
Write "Satu Mare"
Expected result
In the drop down menu you find "Satu Mare, ARAD(Secusigiu)"
Test Data
Satu Mare

Steps 4
Write "Sânpetru German"
Expected result
In the drop down menu you find "Sânpetru German, ARAD(Secusigiu)"
Test Data
Sânpetru German


Certain IdentifierPatterns in the old parser now parsed as IdentifierExpr

Issue Kind
Parse of Valid Source Produced Invalid Syntax Tree
Source Code
switch x { case let .y(z): break }

Description
This code snippet produces the following parse tree:
SourceFileSyntax children=2
  0: CodeBlockItemListSyntax children=1
    0: CodeBlockItemSyntax children=1
      0: SwitchStmtSyntax children=5
        0: switchKeyword
        1: IdentifierExprSyntax children=1
          0: identifier("x")
        2: leftBrace
        3: SwitchCaseListSyntax children=1
          0: SwitchCaseSyntax children=2
            0: SwitchCaseLabelSyntax children=3
              0: caseKeyword
              1: CaseItemListSyntax children=1
                0: CaseItemSyntax children=1
                  0: ValueBindingPatternSyntax children=2
                    0: letKeyword
                    1: ExpressionPatternSyntax children=1
                      0: FunctionCallExprSyntax children=4
                        0: MemberAccessExprSyntax children=2
                          0: prefixPeriod
                          1: identifier("y")
                        1: leftParen
                        2: TupleExprElementListSyntax children=1
                          0: TupleExprElementSyntax children=1
                            0: IdentifierExprSyntax children=1
                              0: identifier("z")
                        3: rightParen
              2: colon
            1: CodeBlockItemListSyntax children=1
              0: CodeBlockItemSyntax children=1
                0: BreakStmtSyntax children=1
                  0: breakKeyword
        4: rightBrace
  1: eof

For z, the C++ parser instead produced an UnresolvedPatternExprSyntax with an IdentifierPatternSyntax as its child.
The C++ tree distinguishes identifiers that are bound in the pattern from those which aren't or can't be. For example,

if the pattern was let .y(z, someFunction(5)), z would be an IdentifierPattern while someFunction would be an IdentifierExprs.
if the pattern was let .y(z, someFunction(w)), z and w would be IdentifierPatterns while someFunction would be an IdentifierExpr (and the compiler would later emit an error that pattern variable binding cannot appear in an expression).

rdar://100301666


Disable provisional completions for now

Summary of the changes

This fixes the dreaded "Disco Colors". Basically what happens is that when provisional completion fires it makes two edits to the backing C# document (to manually add a . which is otherwise interpreted as HTML). These modifications at programmatic speed seems to cause the document to get out of sync, as evidenced by colors being in the wrong position and by Debug.Asserts about the length of the C# backing document.
This can be reproduced by typing @FirstName.N inside a razor Document with C# under that point.
We don't want to leave Provisional Completion removed but we estimate that having to type @DateTime.N until we figure out a more permanent fix will be minimally invasive, as opposed the very confusing color changes when you're just typing.

@NTaylorMullen did WebTools end up replying about how common provisional completions are?

Ya their feel is that it'd be a pretty big take back and people would feel as though completion was broken which is reasonable. I think this PR is a "low risk" way to fix if we need touse it

This is ready for re-review. I don't guarantee it fixes ALL disco-colors but it DOES fix the one source we've identified in Provisional Completion.


py_any.downcast::<PyIterator>() behaviour is different on windows vs macos and linux

Bug Description
This is an extension of https://github.com/PyO3/pyo3/discussions/2902
On Linux and MacOS, py_any.downcast::<PyIterator>() is only Ok for generators.
On windows it appears to be Ok for strings and anything which is a sequence.
Steps to Reproduce
Should be as simple as
let s = "foobar".to_object().into_ref(py);
let iter: PyIterator = s.downcast()?;

Should fail on linux and macos, but pass on windows.
 Did you use a virtualenv?
pyenv
Additional Info
No response

So I've just run the following test locally on PyO3 main, both Windows and Unix, Python 3.10 and 3.11:
#[test]
fn test_iterator_downcast() {
    Python::with_gil(|py| {
        let s = "foobar".to_object(py).into_ref(py);
        let iter: &pyo3::types::PyIterator = s.downcast().unwrap();
    })
}

In all cases, I get the expected failure:
---- test_iterator_downcast stdout ----
thread 'test_iterator_downcast' panicked at 'called `Result::unwrap()` on an `Err` value: PyDowncastError { from: 'foobar', to: "Iterator" }', tests/test_downcast.rs:7:59
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

So I don't think this is a bug in PyO3. Instead let's go back to the discussion and get some more information about how you're setting up your test environment, so we can understand what's different about your Windows setup.

Actually, with your example Python class in #2902 I can repro this, and will push a fix shortly.

My mistake, it was a subclass of str, sorry created this in a hurry.


Grant proposals for 2024-2025

I would like to direct a portion of the Council's budget to fund additional grants to Rust contributors. The budget for the Fellowship Grants did not cover as many people as we would have liked.
We have four candidates that we would like to fund at $20,000 over 12 months as "Project Grants". This will come out of the $150,000 unreserved Council budget from 2024. We'll need to finalize the exact structure with the Foundation, but we are likely looking at it structured as an initial payment plus a mid-point payment contingent on how things are going (payments are unrestricted, no separate travel/hardware budget). This is also contingent on the candidates accepting the grants.
@Mark-Simulacrum and @jamesmunns and Paul Lenz have reviewed these candidates and agree that we would like to extend the grants to them.
I would like to ask the council to approve the proposed $80,000 for these grants.
Time: ~10 minutes

@rfcbot fcp merge
Proposing to approve this.

Do we have a list of the four proposed grants somewhere?

@eholk not public, as we haven't confirmed with the people whether they would like to accept. @ehuss can probably invite you to the doc shared with me and Mark

@rfcbot reviewed

We discussed this in the last meeting.  People were on board with it, as reflected in the FCP, and that FCP has completed.


Fixing stateless props override on Popover component

Fixing issue reported on: https://github.com/segmentio/evergreen/issues/639
I was taking a look at the PopOver component and I noticed that when I pass a value for the statelessProps props - with a style object - it'll start to override and lost all the previous style properties.
It's because it was just spreading the statelessProps into the component and overriding all the default styling values.
So my change was basically to merge the statelessProps.style with the current style into one single style props.
So, I can do something like this without losing any default style props.
<Popover
          ...
          statelessProps={{
            style: {
              zIndex: 1000
            }
          }}
         ...
/>

@jeroenransijn Can you review it and verify if it makes sense? I'm using this component on my production project and we're needing it as soon as possible

@mshwery Thank you for approving this PR. Do you know when it will be available on NPM (deployed)?

Now! https://github.com/segmentio/evergreen/releases/tag/v4.19.2


Implememt tool use

Langchain is a powerful tool to create agent-based workflow. It will be interesting to implement it in Unity for game AI usage.🥰
I will first create a tool use system for quick verification.

Implement unity version LangChain.  3945b7c
Tool use will be implemented soon.

Already implemented in 306b6a1.


HDDS-7962. Add configuration flag to enable LegacyReplicationManager for RATIS containers

What changes were proposed in this pull request?
Currently all RATIS containers are hard coded to be handled by LegacyReplicationManager. This jira proposes adding a configuration that will decide if these containers should be handled by new RM or legacy. For now, by default, legacy should handle ratis containers.
Note that the new RM does not yet provide a move API. So if the new RM is configured to handle RATIS containers, container balancer is not expected to work.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7962
How was this patch tested?

I've also added commits from HDDS-7847 to this.


Missing explanations

==>    hidden_units = [8, 8]
Hello guys,
Is it possible to add explanations on the reasons for the choices for the hidden units values?
it's not a real issue because everything work perfectly but I don't understand why you had choosen these values..
why not these possibilities :  [7, 7] [9, 9]  [7, 7, 7] [8,8,8] [9, 9, 9]  etc...
(theses exemples are totaly random, sorry if they look to be "dumb values". I'm learning alone, without school, in my house with nobody to help me)
THANKS YOU VERY MUCH!!!!!
Alex

Hi @Guitalex ,
Could you please confirm which example/tutorial you are referring to ? Yeah sometimes some values work best based on lot of trails.Please provide more context to look into the issue.
Thanks!

hello!
for this example :  https://keras.io/examples/keras_recipes/bayesian_neural_networks/
THX!
Alex

Hi @Guitalex ,
Thanks for pointing out to exact tutorial. I have gone through the tutorial and the hidden_units are nothing but hidden dense layers with each unit in hidden_units representing no of neurons. Yeah you can alter these values as per your requirement.You can take of size=3 of you want 3 hidden dense layers etc.
To demonstrate the same I tried with hidden_units = [10, 8, 4] and attached gist for results.

@Guitalex ,
I am not reopening the issue again as it is not an issue and my explanation above is enough to address the issue.
If you still have any query regarding this please let me know.
Thanks!


[Feature] Adding Speed Calculator

Description
Speed Converter
Implement a speed converter which allows users to conveniently convert speeds between different units such as meters per second (m/s) to kilometers per hour (km/h).
One can select the conversion into different units and vice versa.
Please assign me this issue under iwoc2024
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Sir please assign me this issue under iwoc @Rakesh9100

Go ahead and name your folder Speed-Calculator @TheDarkW3b

@Rakesh9100 assign this issue to me...i can enhance this issue


chore(cluster_agent): added trace for republish

added trace for republish method

bors merge

bors merge

bors merge


KAFKA-8942: Document RocksDB metrics

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Call for review: @bbjeck @ableegoldman @mjsax
This PR needs to be cherry-picked to 2.4.

JDK 11/Scala 2.13 and JDK 8/Scala 2.11 failed and logs on Jenkins have been already removed.
In JDK 11/Scala 2.12 the following test failed:
org.apache.kafka.trogdor.coordinator.CoordinatorTest.testTaskRequestWithOldStartMsGetsUpdated

Retest this, please


Laion embeddings and clip output for the same image do not match exactly

The laion website provides embeddings and parquets which tie an embedding at an index in the array to its associated metadata. In theory the clip output and the laion embedding for the same image should be exactly the same, but they're not. Here's how to reproduce:
from io import BytesIO

import clip
import numpy as np
import pandas as pd
import requests
import torch
from PIL import Image
from sklearn.metrics.pairwise import cosine_similarity

# embeddings: https://deploy.laion.ai/8f83b608504d46bb81708ec86e912220/embeddings/img_emb/img_emb_0.npy
# parquet: https://deploy.laion.ai/8f83b608504d46bb81708ec86e912220/embeddings/metadata/metadata_0.parquet

INDEX = 0

laion_embedding = np.load("img_emb_0.npy")[INDEX]
laion_embedding = np.expand_dims(laion_embedding, 0)
url = pd.read_parquet("metadata_0.parquet")["url"][INDEX]

device = "cuda" if torch.cuda.is_available() else "cpu"
model, preprocess = clip.load("ViT-B/32", device=device)

response = requests.get(url)
image = Image.open(BytesIO(response.content))
image = preprocess(image).unsqueeze(0).to(device)

with torch.no_grad():
    clip_embedding = model.encode_image(image).cpu().numpy()

print("Pre-norm difference:", (laion_embedding - clip_embedding).sum())
clip_embedding = clip_embedding / np.linalg.norm(clip_embedding)
print("Post-norm difference:", (laion_embedding - clip_embedding).sum())
print("Cosine sim:", cosine_similarity(laion_embedding, clip_embedding))

This outputs:
Pre-norm difference: -4.79
Post-norm difference: 0.1592
Cosine sim: [[0.91325109]]

I've verified that laion uses the ViT-B/32 backbone as well. I'm wondering what might be causing the discrepancy here. Any ideas?

This issue is related to this issue: https://github.com/rom1504/clip-retrieval/discussions/100 which didn't seem to be resolved. I was able to replicate the results in this issue, but it's not a perfect solution.

Another comparable problem is that the text embedding on the LAION400 does not match the output of the CLIP-ViT-B/32 text-encoder.

Another comparable problem is that the text embedding on the LAION400 does not match the output of the CLIP-ViT-B/32 text-encoder.

@stevebottos The text embedding should not be affected by image resizing ....

@TianRui-Song717 That's strange, any luck discovering why?


The continuous increase in storage

I use the milvus installed by help and the storageClass made by juicefs. Juicefs' metadata is stored in Redis and Object storage are stored in mini
The storage size of mini has been continuously increasing in the Milvus cluster without any use
I have stored 3G for about 12 hours. Is there any good way to stop the increase

Is the milvus running in the meanwhile? The data may be written by milvus, you can monitor the access log of juicefs by cat $MOUNTPOINT/.accesslog.

Only this one reported an error

All other pods are

Sorry, I understand what you're saying. Could you please help me take a look at this log

I just mounted it locally. Do you need me to cat it again after a while

I mounted it locally and saw the accesslog, but it doesn't have any extra logs. My objects and size have been increasing

Hi, there are normally two reasons:

the application (.eg. milvus) continuously writes data to Juicefs. In this case, you should check if there are some access logs while objects are increasing.
if you mount Juicefs with the flag writeback, the data will be written to local disks first, and continuously be put to object storage later.

My Milvus is only installed and not used, which is very strange. Is there any other troubleshooting ideas provided? Thank you

My Milvus is only installed and not used, which is very strange. Is there any other troubleshooting ideas provided? Thank you

Could you enter the Juicefs volume to check if there are unexpected files? Then you can check data leaks by juicefs fsck $metaurl.

juicefs fsck $metaurl log
fsck.zip

It seems there are many files in .trash of Juicefs, and some objects of them (about 3GiB) are still not uploaded to object storage.
The mount option writeback is likely to cause this case, could you provide the yaml config of storageClass?

Thank you for your answer. Here is my configuration file information

k8s.zip

Ok, it seems all chunks are uploaded to object storage. And there are still many files in your Juicefs volume, their chunks cost 1.3GiB capacity of storage. You can clean them as needed.
filelist.log

dump-2023-08-02-014356.json.gz

Can you tell if the logs are being stored based on these data

The filelist.log is excerpted from your fsck.log, and your dumped meta data also shows there are still many files in your Juicefs volume.
So, the objects on minio are expected, please check out which application write these files and if they should be deleted.

closing stale issue.


Hard-coded params in compile script

Please use generic solution for this:
https://github.com/rl-institut/simbev/blob/f7a61ad744c4993fdc2c12675657d0b8883cbb1e/simbev/helpers/helpers.py#L155-L158
Either by

adding a proper argparse or
deleting it and providing the 3 lines as little script example in the readme

code removed with PR52, new example script coming later


Fixing bug on request saving

This bug appeared when saving a request with no collections available.
Fixes #390

#390

Hey @adevr,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: b5db8ee0-1a12-11ea-a744-67689bca182d

Hey @adevr,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 5d519150-1bc7-11ea-b973-2f3c92f96c0e


Admin menu ordering

Hello,
Currently Flask-Admin doesn't provide a way to order the links in the admin menu.
However, such a feature is  extremely important for admin interfaces targeting non-developers as well as libraries built on top of Flask Admin (such as Oy Content Management System.
This PR addresses this issue by adding two new attributes as follows:

sort_order to flask_admin.menu.BaseMenu and all its subclasses  (default to None).
menu_order to flask_admin.base.BaseView and
flask_admin.model.base.BaseModelView and all their subclasses (defaults to 0)

Based on these attributes the BaseView.()_menu attribute is sorted every time a new view is added using BaseView.add_view.
For menu links added using BaseView.add_link, the `BaseView._menu_link* is also sorted whenever a new link is added.
For menu categories, the user can specify category orders on the BaseView.category_menu_orders dictionary the same way they specify category_icon_classes. In this case the category._children list is also sorted whenever a new child is added to the category.

It would be nice to have it!

You might want to clarify some of the terminology. First of all, maybe you should change the first sentence of this pull request to:

Currently Flask-Admin doesn't provide a way to order the links in the admin menu independently of insertion order.

Presumably you have a use case in which menu entries are added and removed as the application runs.
Second, I think the parameter name "menu_order" is subject to misunderstanding. It sounds like something that applies to the menu as a whole. (Like "ORDER BY" in SQL.) Until I looked at the code I expected it to take values such as "alphabetical" or "ascending" and wondered why you would want to do that.
It would be clearer if we referred to the "position" of a menu item rather than to its "order". The "menu_order" parameter could be "menu_pos" instead.


Cross: Test out End-to-End flow

As as a DEVELOPER
I want to ensure that the DMR and participants can be setup easily
So that the system communicates with each other appropriately
Acceptance Criteria

[ ] Ensure all containers are communicating appropriately
[ ] Docker compose file

(Note: Storing the docker file TBD)

For the time being we'll share docker compose in the teams channel. To decide on the repo structure after Milestone 1.

@guypritchard / @shelabd ... if think this issue would benefit from a little more detail if possible. Right now, i would not know how to take this task on myself

PRs are in other repos:
DMR:
https://github.com/buerokratt/DMR/pull/41
MockBot:
https://github.com/buerokratt/Mock-Bot/pull/25


Add boot configuration scripts.

Prepare host keytab file in the treadmill/spool/keytab directory.
Ensure that /etc/ld.so.preload exists, as temp workaround until the
code to handle file absense gracefully.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #70   +/-   ##
=======================================
  Coverage   24.79%   24.79%
=======================================
  Files          42       42
  Lines        1839     1839
  Branches      197      197
=======================================
  Hits          456      456
  Misses       1361     1361
  Partials       22       22

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 35e0241...19a4129. Read the comment docs.


Zoom and pan behavior appears to be off

Zooming in and panning works fine until you zoom out. At this point you can zoom in again but panning no longer works. This doesn't appear to be a problem for the zoom example chart. For me the behavior is occurring on a chart where all the data is loaded after the fact. I'm not sure if that is relevant or not.

It's quite possible that could be a bug, but it's really hard to verify it without more information.
We get a lot of reports and the more info you give us, the more likely we'll be able to address it in a timely fashion.
Please read the contributing guidelines, then add the following information to your issue:

Operating system and web browser (with version)
What version of the library
If you were doing anything unusual, like using C3 in conjunction with a particular library or framework
Ideally, a JSFiddle example depicting the specific problem really helps us out.

Use this JSFiddle template

Thank you for your cooperation — we rely on users like you to help make C3 the best charting library it can be!

No response from submitter


Content Suggestion: Liquiditi

Link to Content: https://liquiditi.io/
Briefly Describe Content: No-KYC swap service between BTC and L-BTC, L-USDT, L-CAD
Suggested Section: Bitcoin Layer 2 Network swap services

What page are you imagining for this? A dedicated layer 2 section?

Hmm, not sure. Are there any no-KYC swap services doing USD to BTC currently?

bisq?

Still unsure where to add this. I think we need a Layer 2 page

If it's a service that gets paid a fee to move funds between sidechains, could it go on "Spend Bitcoin"?

Yeah, IMO a Layer 2 page makes sense now that fee market is developing we will see more Layer 2 stuff going forward.


Add env config

It'd be cool if we let the user merge custom ENV easily from the config.


Building iOS should not see android issues.

Building flutter app for iOS also shows android embedding error. As it adds noise to the output, we should not show android error/warnings during an iOS build.
Steps to Reproduce

Find an app uses Android V1 embedder.
Run flutter build ios
See:

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
Warning
──────────────────────────────────────────────────────────────────────────────
Your Flutter application is created using an older version of the Android
embedding. It is being deprecated in favor of Android embedding v2. Follow the
steps at

https://flutter.dev/go/android-project-migration

to migrate your project. You may also pass the --ignore-deprecation flag to
ignore this check and continue with the deprecated v1 embedding. However,
the v1 Android embedding will be removed in future versions of Flutter.
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
The detected reason was:

  /Users/ychris/flutter/plugins/packages/battery/battery/example/android/app/src/main/AndroidManifest.xml uses
  `android:name="io.flutter.app.FutterApplication"`
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

This app is using a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to migrate this app to the V2 embedding.
Take a look at the docs for migrating an app: https://github.com/flutter/flutter/wiki/Upgrading-pre-1.12-Android-projects

cc @GaryQian

Also building Android but shows some iOS signing info:
Running "flutter pub get" in multiple_flutters_module...         2,173ms
Signing iOS app for device deployment using developer identity: "Apple Development: xxx(HC79TMNY27)"
Signing iOS app for device deployment using developer identity: "Apple Development: xxx(HC79TMNY27)"

Building without sound null safety
For more information see https://dart.dev/null-safety/unsound-null-safety
*

@eggfly Thank you for bringing this up! Do you mind to share the full steps of reproduce? (Ideally in a separate issue)

Was tired of seeing this warning in iOS triaged issues.  Yoink https://github.com/flutter/flutter/pull/96148


Code | Pyramid Pattern 2

@aryashah2k Please, assign this issue to me!

@anjali-kundliya05 , please refer the discord server, for now issues is paused. You have to spend this week finalizing projects to contribute
No PR will be counted towards SWOC Leaderboard this week

Hey, can you please assign issues to me from 2-42 ? @aryashah2k


What is the license?

Hi
I don't see LICENSE.md file in the repository and no license was mentioned in README.md file as well. What is the license of this amazing project?

MIT license, but please put a link to https://wordpress.org/plugins/crelly-slider/ where the user can see it (in the credit sections of your project).

I have added the license file to the repo.


Boolean coerced to String when using .oneOf(String, Boolean)

myProperty: {
    type: SimpleSchema.oneOf(String, Boolean)
}

While the .oneOf() arguments are listed in this order, any Boolean values entering the database are coerced to strings. If you flip the arguments...
myProperty: {
    type: SimpleSchema.oneOf(Boolean, String,)
}

... everything seems to work fine

This is a limitation of the clean function. It currently uses only the first type. Might be fixable but could be tricky.
https://github.com/aldeed/node-simple-schema/blob/master/lib/clean.js#L77


Invalid (or problem causing) default value

I followed the directions here but while it successfully connected to GitHub, my function never showed up. Turns out the value, WEBSITE_RUN_FROM_PACKAGE, defaults to 1. If you set it to 0 and restart/sync/whatever (I clicked a few things) it then works. I think the doc should be updated to note this.

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

ID: 5a34424e-6a23-5517-5755-c9282643876f
Version Independent ID: 1c375649-94d6-863a-203b-cb5ed5e4044e
Content: Continuous deployment for Azure Functions
Content Source: articles/azure-functions/functions-continuous-deployment.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hi @cfjedimaster Thank you for your feedback! We will review and update as appropriate.

@cfjedimaster I have assigned the issue to the content author to evaluate and update as appropriate.

@colbytresness are we missing a step or is WEBSITE_RUN_FROM_PACKAGE getting set to 1 a bug?

If you've deployed from local, this setting will be populated as 1. It won't be set if you created from the portal. So it's definitely worth adding a call out to the document that you'll need to remove this for source control to work, if it's already present. I'll do that at some point.

Tracking this doc issue internally as #1574283.
Thanks!

@ColbyTresness - where in source control would this setting be held? I don't see any config files that hold this (using the VS Code Azure Function integrations).
However we do have the same problem.

Fixed #please-close


Mages and Illusioners cant clone

Their clones just instantly disappear when they spawn, even when using /summon. I'm using all the mods from https://www.curseforge.com/minecraft/modpacks/village-and-pillage-2-0 forge 36.2.20 dungeons mobs beta 2.0.3
EDIT: Tried updating to forge 36.2.34 and going back in versions of the mod, but neither worked.
EDIT 2: Tried just dungeons mobs with only its dependencies and it works there, so it's probably a mod compatibility issue

iBecause the Mage and Illusioner clones do not have nbt control living tick

Mages and illusioners properly clone now


Deploy testcab/cors-anywhere

https://hub.docker.com/r/testcab/cors-anywhere

This container is needed for https://github.com/aeternity/aepp-base/pull/1490. @mmpetarpeshev can you deploy it somewhere the proper way? (like cors-anywhere.aepps.com) It also may be used by Superhero Wallet in future to retrieve aepps details.
Meanwile this PR needs to remain opened 🙃

This container is needed for #1490. @mmpetarpeshev can you deploy it somewhere the proper way? (like cors-anywhere.aepps.com) It also may be used by Superhero Wallet in future to retrieve aepps details.
Meanwile this PR needs to remain opened 🙃

@dincho support this service . If he prefers I do it. WIll take a look, but let's sync with him firstly.

No. This is wrong in so many cases. I won't deploy a (practically) public proxy to our environments.
All the applications that want's to be listed in browser should support CORS configuration, either wildcard but better base app specific.


Dash teleport not working if controller was not already turned on

I've encountered the following problem:
the dash teleport works fine, but if I enter into Play Mode with a controller switched off and then I switch the controller on, the beam of the teleport appears (and apparently the data in the debug inspector are updated), the teleport events are sent, but the character doesn't move at all, as if he was glued to the starting point

I've just tried this in the example scene 038_CameraRig_DashTeleport and I can start with out the controllers turned on, then when I turn the controller on after the scene has loaded I can still dash around correctly.
Using Unity 5.4.4f1 and latest version of VRTK Github master (commit:  https://github.com/thestonefox/VRTK/commit/d533b5e2307eaa9eccd9ceeeacc089c803cb214f)

Mmmh I really do not understand why, but disabling and enabling again the dash teleport script on the cameraRig make things work again. I haven't touched anything in those scripts, but I'm using the AssetStore version. In any case there's no need for further investigation, I imagine that must be my fault.  Thanks for the speed in your answer.

Try upgrading to the latest github master and see if it fixes your issues.


Correcting synchronization problem in last pull to master

Last pull to master was made with master already ahead of develop. This should fix that.

@dabacon is there a problem with the the CLA bot. Why are we still waiting for it to report?

Closing per @babbush's comment on quantumlib/OpenFermion#6. We're going to move the history of the develop branch to master, instead of merging changes (#12).


Crash on Flutter application launch on emulator

Steps to Reproduce
Crashes with 1.22.0-12.1.pre
Works with  1.21.0-9.1.pre

From Xcode, launch the application on Emulator iPad air (4th generation)
The project builds
The application crashes at launch

Logs
2020-09-23 13:29:01.527533+0200 Runner[52852:484760] [VERBOSE-2:shell.cc(210)] Dart Unhandled Exception: Invalid argument (domain): Invalid domain name: "localhost", stack trace: #0      new _DomainNetworkPolicy (dart:io/network_policy.dart:85:7)
#1      _constructDomainPolicies (dart:io/network_policy.dart:181:22)
#2      _EmbedderConfig._setDomainPolicies (dart:io/embedder_config.dart:44:23)
2020-09-23 13:29:01.527542+0200 Runner[52852:484773] [VERBOSE-2:shell.cc(210)] Dart Unhandled Exception: Invalid argument (domain): Invalid domain name: "localhost", stack trace: #0      new _DomainNetworkPolicy (dart:io/network_policy.dart:85:7)
#1      _constructDomainPolicies (dart:io/network_policy.dart:181:22)
#2      _EmbedderConfig._setDomainPolicies (dart:io/embedder_config.dart:44:23)
2020-09-23 13:29:01.527696+0200 Runner[52852:484773] [VERBOSE-3:dart_io.cc(39)] Check failed: !LogIfError(set_domain_network_policy_result).
2020-09-23 13:29:01.527728+0200 Runner[52852:484760] [VERBOSE-3:dart_io.cc(39)] Check failed: !LogIfError(set_domain_network_policy_result).
CoreSimulator 732.17 - Device: iPad Air (4th generation) (7888B562-066B-41D9-A280-0A022398BD1E) - Runtime: iOS 14.0 (18A372) - DeviceType: iPad Air (4th generation)

Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, 1.22.0-12.1.pre, on Mac OS X 10.15.6 19G2021, locale fr-FR)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 12.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.0)
[✓] VS Code (version 1.49.1)
[✓] Connected device (3 available)

• No issues found!

Hi @egodefroy
Can you please provide your flutter doctor -v, your flutter run --verbose and a minimal complete reproducible code sample?
Thank you

Finally solved this after hours, it seems that with the new network_policy.dart, we no longer need to add 'localhost' as NSExceptionDomains

@alvinthen Great ! Sorry but I didn't manage to make a minimal example reproducing the problem

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

I experienced the same issue today after migrating to 1.22.
More specifically, I connect to local Firebase Cloud Functions emulator. In order to do that in iOS I need to add
         <dict>
		<key>NSExceptionDomains</key>
		<dict>
			<key>localhost</key>
			<dict>
				<key>NSExceptionAllowsInsecureHTTPLoads</key>
				<true/>
			</dict>
		</dict>
	</dict>

to the Info.plist. And in my Dart file:
final cloudFunctionInstance = CloudFunctions(region: region).useFunctionsEmulator(origin: "http://localhost:5001")

Unfortunately since I updated to 1.22, I get the same error as above.
Following the suggestion from alvinthen I removed the dictionary key above from the Info.plist, and the app stopped crashing, but it still gave an error.
flutter: PlatformException(-1022, The resource could not be loaded because the App Transport Security policy requires the use of a secure connection., null, null)

Solution
I had to change the configuration of the Cloud Function emulator, adding the host, in firebase.json:
"emulators": {
    "functions": {
      "port": 5001,
      "host": "<IP_ADDRESS>" <-- added this
    },
    "firestore": {
      "port": 8080,
      "host": "<IP_ADDRESS>" <-- added this
    },
    "ui": {
      "enabled": true
    }
  }

and changed the useFunctionsEmulator call to:
final cloudFunctionInstance = CloudFunctions(region: region).useFunctionsEmulator(origin: "http://<IP_ADDRESS>:5001")

Now everything works again.
I do not know if this was intended behaviour, that is way I replied to this issue. @pedromassangocode feel free to close again if so.

In my experience, if I set this key to true it seems to cause the crash
<key>NSThirdPartyExceptionAllowsInsecureHTTPLoads</key>
<true/>

If I set it to false or remove it altogether the app doesn't crash.
More information:

I'm running on iOS 14 Simulator
Seems to happen on any domain

Here's my flutter doctor -v
[✓] Flutter (Channel stable, 1.22.0, on Mac OS X 10.15.7 19H2, locale en-PH)
    • Flutter version 1.22.0 at /Users/zonilyjamepesquera/flutter
    • Framework revision d408d302e2 (4 days ago), 2020-09-29 11:49:17 -0700
    • Engine revision 5babba6c4d
    • Dart version 2.10.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/zonilyjamepesquera/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 49.0.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.49.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.15.0

[✓] Connected device (1 available)
    • iPhone SE (2nd generation) (mobile) • FD9EBBCA-9F59-4B3C-970B-CF66715D628A
      • ios • com.apple.CoreSimulator.SimRuntime.iOS-14-0 (simulator)

• No issues found!


IVSPlayerView's property videoGravity is not working properly.

Describe the bug
IVSPlayerView's property videoGravity is not working properly.
To Reproduce

set  videoGravity to resizeAspectFill
video content is not filling the whole layer, there are black area below and above.

Expected behavior
Video content fill the whole player view's layer.
Screenshots

Hello, thanks for filing an issue.  It doesn't look like this is reproducible when using the default stream in the CustomUI sample application (screenshots below).  Can you confirm that the sample application is running version 1.4.1 of the AmazonIVSPlayer SDK?
playerView.videoGravity = .resizeAspect

playerView.videoGravity = .resizeAspectFill

Thanks!

Closing this due to lack of activity. Please reopen if the issue still persists, thanks!


Failed to start tmux

Ran into another tmux issue with the latest Docker image for REPLbot, however I don't think this one is related to the scripts being in DOS format 😄
root@replbot01:~# docker run --rm -it \
>   -v /etc/replbot:/etc/replbot \
>   -v /var/run/docker.sock:/var/run/docker.sock \
>   binwiederhier/replbot
Unable to find image 'binwiederhier/replbot:latest' locally
latest: Pulling from binwiederhier/replbot
ba3557a56b15: Pull complete
94caa5d1da70: Pull complete
a10a29983da5: Pull complete
3d6ccddb7fa4: Pull complete
ed1bef566495: Pull complete
201a9d1142eb: Pull complete
0806b78d1f4d: Pull complete
2987e14e0ec7: Pull complete
b6e074b08920: Pull complete
Digest: sha256:5766d1634030f8bda85441f471a0a5f566bf24acba515c9373ba50d3ae04cccd
Status: Downloaded newer image for binwiederhier/replbot:latest
2021/10/02 21:32:03 Discord connected as user REPLbot/880575037789323355
2021/10/02 22:03:43 [773039894888579092_893981604693831710] Starting session, requested by<PHONE_NUMBER>18024202
2021/10/02 22:03:43 [773039894888579092_893981604693831710] Started REPL session
2021/10/02 22:03:43 [773039894888579092_893981604693831710] Failed to start tmux: command failed: fork/exec /tmp/replbot_773039894888579092_893981604693831710.tmux.script: no such file or directory
command: /tmp/replbot_773039894888579092_893981604693831710.tmux.script
command output:
2021/10/02 22:03:43 [773039894888579092_893981604693831710] Closed REPL session
2021/10/02 22:03:43 [773039894888579092_893981604693831710] Session exited with error: command failed: fork/exec /tmp/replbot_773039894888579092_893981604693831710.tmux.script: no such file or directory

Let me know if you need any other info! I'll try to dig into this more later when I have some time.

No luck with demo either
2021/10/02 22:17:41 [773039894888579092_893985116660064276] Starting session, requested by<PHONE_NUMBER>01897236
2021/10/02 22:17:41 [773039894888579092_893985116660064276] Started REPL session
2021/10/02 22:17:41 [773039894888579092_893985116660064276] Failed to start tmux: command failed: fork/exec /tmp/replbot_773039894888579092_893985116660064276.tmux.script: no such file or directory
command: /tmp/replbot_773039894888579092_893985116660064276.tmux.script
command output:
2021/10/02 22:17:41 [773039894888579092_893985116660064276] Closed REPL session
2021/10/02 22:17:41 [773039894888579092_893985116660064276] Session exited with error: command failed: fork/exec /tmp/replbot_773039894888579092_893985116660064276.tmux.script: no such file or directory
command: /tmp/replbot_773039894888579092_893985116660064276.tmux.script
command output: ```

I have actually seen this before. I think this is a race condition. I'll investigate tomorrow.

Investigating now.

I was able to reproduce. I should really set up some CI tests with the docker image. I'll have it fixed in no time.

Okay, I fixed it in v0.6.3. It was simple. Bash was not installed inside the docker image :facepalm:
https://github.com/binwiederhier/replbot/commit/e161af3ad3397002c5fb7919ae608315f15166b0#diff-f6dea68331835ca7422a9119cd23989dc9b897966efb202c6426ceaccfe8d2b6L2
I also updated the docker install instructions to make asciinema/recording work:
https://github.com/binwiederhier/replbot#binaries-and-packages
Let me know and re-open if it doesn't work

Oh you know what. I should embed ttyd too to make the web terminal work. I'll make an issue for that.

Sweet, everything's working great! I noticed the ttyd issue as well but totally forgot to document that. Thanks a ton! The new web feature is really a hit.


Added an article for Hugo E-commerce

Hey!
I added an article for Hugo e-commerce but want to be totally upfront that it's from Snipcart (which already has an article posted here, though with different content).
Just wanted to let you know so it didn't come off spammy or anything. Hope you find value in the piece and, if you'd rather just keep it limited to one article from Snipcart, totally understand.
Keep up the great work!

Thanks!


Print usage when parse fails during console producer

Handle OptionException while parsing options when using console producer and print usage before die.

@omkreddy @ijuma @cmccabe

@omkreddy Thanks. Who will merge this? It's a minor change.


Why is there a thread pool at all when it is just used once in a synchronous method?
Plus, the shutdown is not invoked when the method fails anywhere (e.g. due to interruption or when any of the harvesters fails).

Why is there a thread pool at all when it is just used once in a synchronous method?

I didn't write this code, but I assume the goal is to run all the harvesters in parallel.  I guess raw Thread calls could be used rather than ExecutorService, but it doesn't seem that important.

Plus, the shutdown is not invoked when the method fails anywhere (e.g. due to interruption or when any of the harvesters fails).

Ah, good point, I'll change that.

Why is there a thread pool at all when it is just used once in a synchronous method?

I didn't write this code, but I assume the goal is to run all the harvesters in parallel. I guess raw Thread calls could be used rather than ExecutorService, but it doesn't seem that important.

How often does this code run and how big is the tasks list usually? Creating threads is expensive and might be better to have a static thread pool with a limited amount of threads (say 5) than to create them over and over again.
I didn't write this either, but the amount of threads various projects are throwing around is astonishing. Not for this PR, but going over stuff like this (and making it properly async) would probably be a tremendous performance improvement.

How often does this code run and how big is the tasks list usually? Creating threads is expensive and might be better to have a static thread pool with a limited amount of threads (say 5) than to create them over and over again.

It's only called from MappingCandidateHarvesters#initialize(), so it's only called once, the first time MappingCandidateHarvester#getHarvesters() is called.  I assume the parallelization is to make startup fast.
The number of tasks is equal to (number of local addresses) * (number of configured stun servers).

How often does this code run and how big is the tasks list usually? Creating threads is expensive and might be better to have a static thread pool with a limited amount of threads (say 5) than to create them over and over again.

It's only called from MappingCandidateHarvesters#initialize(), so it's only called once, the first time MappingCandidateHarvester#getHarvesters() is called. I assume the parallelization is to make startup fast.

Okay, since this is a one-off thing, don't bother. The other two inline comments might still apply though.


aerospace: init at 0.9.0-beta

Description of changes
Introduces aerospace - an i3 like tiling window manager for macOS. I opted to pull from built binaries as the build process is a complex one with some swift dependencies that I couldn't work through after a full days effort. Please let me know if there are any issues with this approach.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

I opted to pull from built binaries as the build process is a complex one with some swift dependencies that I couldn't work through after a full days effort

Feel free to describe problems you faced in a new issue https://github.com/nikitabobko/AeroSpace/issues/new The process of building shouldn't be complex. GH actions builds AeroSpace without any problems https://github.com/nikitabobko/AeroSpace/actions/runs/8502224571
But besides the issues with the build setup, building the binary in nix has other flaws. I sign the binary with my private key which is needed to retain macOS accessibility permission across updates (unsigned binaries will re-request the accessibility permission on every update)

@lolbinarycat @SigmaSquadron
I made a new PR to replace this one (https://github.com/NixOS/nixpkgs/pull/344015)
Feel free to review it

i no longer work on nixpkgs


Separate development and production environment

Description
Separate development and production environment
Checklist
Add prod environment

[x] : Configure amplify environment

env
prod

[x] : Add amplify pre-signup lambda function in prod environment

Add lambda execution policy to execute lambda function

[x] : Modify api endpoint in prod environment

Related Issue

#72

예에~! 거의 다왔네요!
조금만 더 화이팅!!


Typo cause page.pages not working

https://github.com/codestacks/jekyll-category-pagination/blob/master/paginator.rb#L110

Hi LazyZhu,
can you explain this a little bit? Which kind of typo causes page.pages not working?
Best regards,
Marloto

As 'page.pages' means the maximum number of pages, Thus the line should be pages = (list.length / detail['pagination']).ceil not pages = (detail['pagination'] / list.length).ceil

Thx! Sorry for delay... Issue fixed


Mosquitto-PHP does not build on Ubuntu 22.04 for php8

Am I missing some dependency?
I did:
git checkout php8
git pull
phpize
./configure --with-mosquitto=/usr/include
make
And I am on ubuntu 22.04.1 freshly installed.
~/Mosquitto-PHP$ make /bin/bash /home/user/Mosquitto-PHP/libtool --mode=compile cc -I. -I/home/user/Mosquitto-PHP -I/home/user/Mosquitto-PHP/include -I/home/user/Mosquitto-PHP/main -I/home/user/Mosquitto-PHP -I/usr/include/php/20210902 -I/usr/include/php/20210902/main -I/usr/include/php/20210902/TSRM -I/usr/include/php/20210902/Zend -I/usr/include/php/20210902/ext -I/usr/include/php/20210902/ext/date/lib  -DHAVE_CONFIG_H  -g -O2    -DZEND_COMPILE_DL_EXT=1 -c /home/user/Mosquitto-PHP/mosquitto.c -o mosquitto.lo  -MMD -MF mosquitto.dep -MT mosquitto.lo libtool: compile:  cc -I. -I/home/user/Mosquitto-PHP -I/home/user/Mosquitto-PHP/include -I/home/user/Mosquitto-PHP/main -I/home/user/Mosquitto-PHP -I/usr/include/php/20210902 -I/usr/include/php/20210902/main -I/usr/include/php/20210902/TSRM -I/usr/include/php/20210902/Zend -I/usr/include/php/20210902/ext -I/usr/include/php/20210902/ext/date/lib -DHAVE_CONFIG_H -g -O2 -DZEND_COMPILE_DL_EXT=1 -c /home/user/Mosquitto-PHP/mosquitto.c -MMD -MF mosquitto.dep -MT mosquitto.lo  -fPIC -DPIC -o .libs/mosquitto.o /home/user/Mosquitto-PHP/mosquitto.c: In function ‘zim_Mosquitto_Client_setTlsCertificates’: /home/user/Mosquitto-PHP/mosquitto.c:234:18: warning: passing argument 1 of ‘php_stat’ from incompatible pointer type [-Wincompatible-pointer-types]   234 |         php_stat(ca_path, ca_path_len, FS_IS_DIR, &stat);       |                  ^~~~~~~       |                  |       |                  char * In file included from /home/user/Mosquitto-PHP/mosquitto.c:10: /usr/include/php/20210902/ext/standard/php_filestat.h:44:35: note: expected ‘zend_string *’ {aka ‘struct _zend_string *’} but argument is of type ‘char *’    44 | PHPAPI void php_stat(zend_string *filename, int type, zval *return_value);       |                      ~~~~~~~~~~~~~^~~~~~~~ /usr/include/php/20210902/ext/standard/php_filestat.h:60:20: warning: passing argument 3 of ‘php_stat’ makes pointer from integer without a cast [-Wint-conversion]    60 | #define FS_IS_DIR  13       |                    ^~       |                    |       |                    int /home/user/Mosquitto-PHP/mosquitto.c:234:40: note: in expansion of macro ‘FS_IS_DIR’   234 |         php_stat(ca_path, ca_path_len, FS_IS_DIR, &stat);       |                                        ^~~~~~~~~ /usr/include/php/20210902/ext/standard/php_filestat.h:44:61: note: expected ‘zval *’ {aka ‘struct _zval_struct *’} but argument is of type ‘int’    44 | PHPAPI void php_stat(zend_string *filename, int type, zval *return_value);       |                                                       ~~~~~~^~~~~~~~~~~~ /home/user/Mosquitto-PHP/mosquitto.c:234:9: error: too many arguments to function ‘php_stat’   234 |         php_stat(ca_path, ca_path_len, FS_IS_DIR, &stat);       |         ^~~~~~~~ In file included from /home/user/Mosquitto-PHP/mosquitto.c:10: /usr/include/php/20210902/ext/standard/php_filestat.h:44:13: note: declared here    44 | PHPAPI void php_stat(zend_string *filename, int type, zval *return_value);       |             ^~~~~~~~ make: *** [Makefile:202: mosquitto.lo] Error 1

Ok, wrong repro. I used: https://github.com/nismoryco/Mosquitto-PHP instead and that worked fine.


Detangle delisted "This extension violates the Chrome Web Store policy"

https://chrome.google.com/webstore/detail/nhehghlakghlfjpmjdhihgkldeeiehlh

Should we try to just grab the zip and remove the notifications permission from the manifest for now?  IIRC notifications were removed in favour of raising the window.