Intellisense in 5.0 is struggling with some packages from the SciSharp ecosystem

Describe the bug
Intellisense works for regular F#, but it does not work for the tensorflow related code.
To Reproduce
#r "nuget: TensorFlow.Net"
#r "nuget: TensorFlow.Keras"
#r "nuget: SciSharp.TensorFlow.Redist"
#r "nuget: NumSharp"

open Tensorflow
open Tensorflow.Keras
open NumSharp

let tf = Binding.New<tensorflow>()
let W = tf.Variable(-0.06f,name = "weight")
tf.Variable()

See here for the comparison to the behavior in visual studio, where intellisense works (but fsi does not):
https://github.com/SciSharp/TensorFlow.NET/issues/656
Environment (please complete the following information):
Environment (please complete the following information):
Version: 1.52.0-insider (user setup)
Commit: 5e350b1b79675cecdff224eb00f7bf62ae8789fc
Date: 2020-12-04T10:15:27.849Z
Electron: 9.3.5
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19041
Ionide version: 5.0.0-preview2
dotnet SDK version: 5.0.100

This is probably because the version of the #r handler for nuget we have in FSAC is behind the officially-released one. We can't do anything about that until the next version of the FCS package is published (which should be very soon, as Kevin Ransom is working on that right now.)

Got it, thank you.

@baronfel, perhaps this is still the FCS package thing affecting ionide 5.0, but I just discovered that everything (intellisense and code) works correctly on stable VS Code and ionide 4.17.0 with --langversion:preview enabled.
Version: 1.51.1 (user setup)
Commit: e5a624b788d92b8d34d1392e4c4d9789406efe8f
Date: 2020-11-10T23:34:32.027Z
Electron: 9.3.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19041

This is probably the same issue as https://github.com/ionide/ionide-vscode-fsharp/issues/1441, so to help with housekeeping I will close this for now and then reopen it if the impending FCS fix doesn't fix this too.


Playback does not continue to end when reaching silence or near silence.

Bug description
Audio is not played completely until the end.
If the end of the audio is silence or near silence, the cursor does not advance.
If the sound near the end of the is near silence, the cursor does not advance AND the audio is not heard.
Given that I am only playing the audio, I'm not sure what a minimal code snippet is other than using the playPause().
I dont know if it matters but...

I am loading the audio as base64 encoding from my local machine.
wavesurfer is embedded in a Streamlit application.

The audio plays and all functionality works but I've noticed the pause at the end of each audio, since my audio segments include the silence at the beginning and end of each file.
Today, I noticed the playback cursor did not advance and the audio was not heard at a certain point, while there was still visible audio (not silent) just before the cursor reached the end.  This is more evident in the video showing the second audio file "Combined_Audio_Scripts_48.wav"
My loading is one of two types,

Direct conversion of the wav audio to base64
A conversion of a wav to MP3 in memory and then converted to base64

For simplicity, I'm showing the direct conversion
Environment

Browser: Firefox, Linux

Minimal code snippet
# Function to convert audio to Base64 encoding for embedding -- python in Streamlit
def audio_to_base64(file_path):
    with open(file_path, "rb") as audio_file:
        return base64.b64encode(audio_file.read()).decode('utf-8')

                document.querySelector(`.play-pause[data-index="{idx}"]`).addEventListener('click', () => {{
                    wavesurfer{idx}.playPause();
                }});

                base64_audio = audio_to_base64(row['full_audio_path'])
                wavesurfer{idx}.load('data:audio/wav;base64,{base64_audio}');

Expected result
playback of the complete audio, whether silence or near silence.
Obtained result
The audio appears to pause or stop playing entirely near the end when reaching silence or near silence.
Screenshots
https://github.com/user-attachments/assets/233a01a5-deb2-4f84-ac3b-e37a7c4c0347

I think something might be wrong with your base64 encoding. When I load and render the attached files directly as wav's, it renders them correctly. Here's a screenshot of the second clip, it ends right after "confidence". So that "silence" segment from your video is some artifact most likely caused by base64.


"Load PSBT from file..." does not work when no wallet is loaded

Linux Mint 20.1 (Qt 5.12.8), the master (ad89812c70a9b88be642fceffb970859f6fdcba2):

opens no dialog.
From IRC:
<hebasto> achow101: does "File" -> "Load PSBT from {file|clipboard}" make any sense when no wallet is loaded?
<achow101> hebasto: yes, for finalize and sending

cc @achow101

I think these two list items may change to disabled state when no wallet is loaded. Such as the close wallet button,  backup wallet button.... above
When no wallet is loaded. Even if I press it, there is no any GUI response (Expected)
The enabled status is no meaningful when no wallet is loaded.


See a few broken links and typos in the readme file and tutorial

At the top of the readme.md file it says Android.

This repository contains a pre-release for Android developers:
the Blockstack Android SDK (/blockstack-sdk)

The link /blockstack-sdk is broken

In step 6 of the tutorial is says 'Navigate to https://localhost:8080 with your browser to display the application.' while the previous step shows that the 'Project is running at http://localhost:8080/'

So step 6 should be http not https

The link to download the final project code for the tutorial does not do anything.  Just reloads the page.

If you prefer, you can skip working through the tutorial all together. Instead, you can download the final project code and import it into XCode to review it.

Thanks for the issue @petec029 I'm working on this and also building the zip. Apologies for the time frame, I couldn't find the original ZIP so I am needing to recreate it

Closing this as it was merged. Reopen if you are still seeing issues.


chore(in-page-wizard): add componentId tagging

👋 Thanks for creating a pull request! 🚀 Checkout the storybook we've created for it: angular


COMMUNICATION LATENCY ON HTTP SERVER doesn't go down below 30 ms. (IDFGH-9281)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
In tcp server, the average of communication latency is less than 7 ms.
I am using http server.
I call on the local network, but it doesn't go down below 30 ms.
What options do I need to see?
I checked ...
esp_wifi_set_ps(false);
CONFIG_ESP32_WIFI_AMPDU_TX_ENABLED = y -> false;
CONFIG_ESP32_WIFI_AMPDU_RX_ENABLED = y -> false;

checked...
CONFIG_HTTPD_ERR_RESP_NO_DELAY= false

Hello @Inverter-Jo
This might help you improve the performance: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-guides/performance/speed.html#improving-network-speed
Closing this issue. Please feel free to reopen.


May I use Cifar10 to train the generator? Why haven't I been converging? If you can, please provide the code. Thank you.

May I use Cifar10 to train the generator? Why haven't I been converging? If you can, please provide the code. Thank you.

Hi @tanlingp , we haven't use CIFAR-10 to train our generator. Have you tried to reduce the learning rate during the training phase? I'm guessing the batch size is set relatively large.

#I am trying to use the CIFAR10 dataset as the source domain to train the generator, but it has been unable to converge. If you can, please provide the code. Thank you.

#I am trying to use the CIFAR10 dataset as the source domain to train the generator, but it has been unable to converge. If you can, please provide the code. Thank you.

What's the learning rate and batch size in your implementation?

the learnning rate set 0.00001，batch size set 128
| |
tlp6378
|
|
@.***
|
---- Replied Message ----
| From | Qilong @.> |
| Date | 05/17/2023 10:00 |
| To | @.> |
| Cc |<EMAIL_ADDRESS>|
| Subject | Re: [Alibaba-AAIG/Beyond-ImageNet-Attack] May I use Cifar10 to train the generator? Why haven't I been converging? If you can, please provide the code. Thank you. (Issue #8) |
#I am trying to use the CIFAR10 dataset as the source domain to train the generator, but it has been unable to converge. If you can, please provide the code. Thank you.
What's the learning rate and batch size in your implementation?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

the learnning rate set 0.00001，batch size set 128 | | tlp6378 | | @.*** |
…
---- Replied Message ---- | From | Qilong @.> | | Date | 05/17/2023 10:00 | | To | @.> | | Cc |<EMAIL_ADDRESS>| | Subject | Re: [Alibaba-AAIG/Beyond-ImageNet-Attack] May I use Cifar10 to train the generator? Why haven't I been converging? If you can, please provide the code. Thank you. (Issue #8) | #I am trying to use the CIFAR10 dataset as the source domain to train the generator, but it has been unable to converge. If you can, please provide the code. Thank you. What's the learning rate and batch size in your implementation? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

Try to resize CIFAR-10 from 32x32 to 224x224 beforehand. This may be helpful.


📊 Data Collection for Project 3 - Nigerian Student's Year One Performance Prediction

📊 Data Collection for Project 3 - Nigerian Student's Year One Performance Prediction
Project Overview
In Project 3, we are working on building a Tabular Machine Learning model that predicts the Year One GPA (Grade Point Average) of Nigerian students. This model will rely on various features, including Jamb scores, Waec grades, experiences during the first-year session, reading patterns, consistency of class attendance, and more.
Our goal is to create a predictive tool that helps Nigerian students anticipate their Year One GPA and make informed decisions about their education.
What's Needed
We need your valuable input to collect the necessary data for this project. Your contribution involves filling out a survey form that captures essential information. Please note that:

Authentic Information: It's crucial to provide authentic and accurate information as this will contribute to creating a reliable and useful predictive model.

Data Anonymity: Rest assured that the survey form collects data anonymously and does not record any personal information. Your privacy is our priority.

How You Can Contribute

Survey Form: Access the survey form using the following link: Survey Form Link.

Complete the Survey: Fill out the survey form by providing the requested information honestly and accurately.

Contribution Acknowledgment: Your participation in this data collection process is a valuable contribution to our project, even though it can't be recorded as a merged pull request on GitHub. This is a significant part of open-source contribution.

Let's Build a Predictive Tool Together!
By participating in this data collection process, you are helping us gather the data needed to train and test our predictive model. Your contribution plays a pivotal role in the success of our project.
Note: If you have any questions or need assistance with the survey form, please feel free to ask in the comments. Your engagement is greatly appreciated.

Since those who filled out the form will not be recorded as a merged pull request on GitHub.
I think a Txt file can be created and they add their names/usernames line by line and make a pull request, what do you think?

Since those who filled out the form will not be recorded as a merged pull request on GitHub.
I think a Txt file can be created and they add their names/usernames line by line and make a pull request, what do you think?

Thank you @Tiamiyu1, this would have been a nice option. But because it is a very sensitive data, we want to try and keep those who fill it as anonymous as possible. The form itself doesn't even collect names or any biodata. Thank you very much.


Remove outdated note in README.md

As current releases support both overlay and standalone mode.

Thanks :smiley:


Add helix instructions

I don't use Helix but I trust you on this

Makes sense. Didn't touch anything else, just using helix's default highlights. This is how it looks with the "bogster" theme.


deployer to recognise a project with deployer.config file

Deployer to have a command for using a project instantly as the current project, maybe deployer use?

Merged in master


Lalimas/browsercachingupdates

Codecov Report

Merging #5482 (0f19924) into dev (cbf8b83) will increase coverage by 1.19%.
The diff coverage is n/a.

Flag
Coverage Δ

*Carryforward flag

msal-angular
96.50% <ø> (?)

msal-browser
86.49% <ø> (ø)

Carriedforward from cbf8b83

msal-node-extensions
76.03% <ø> (ø)

Carriedforward from cbf8b83

msal-react
94.68% <ø> (?)

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

lib/msal-browser/src/internals.ts
100.00% <ø> (ø)

lib/msal-angular/src/msal.navigation.client.ts
92.85% <0.00%> (ø)

...l-react/src/components/UnauthenticatedTemplate.tsx
100.00% <0.00%> (ø)

lib/msal-react/src/hooks/useMsal.ts
100.00% <0.00%> (ø)

...eact/src/components/MsalAuthenticationTemplate.tsx
96.55% <0.00%> (ø)

lib/msal-react/src/hooks/useMsalAuthentication.ts
92.55% <0.00%> (ø)

lib/msal-react/src/error/ReactAuthError.ts
90.90% <0.00%> (ø)

lib/msal-angular/src/constants.ts
100.00% <0.00%> (ø)

lib/msal-react/src/MsalProvider.tsx
96.15% <0.00%> (ø)

lib/msal-react/src/utils/utilities.ts
84.00% <0.00%> (ø)

... and 15 more

Please also update PR title and description


Npm run setup

Looks like is not possible to run npm run setup with the current setup, I have tried to modify the script by adding:
if [ ! -f "$APP_NAME/package.json" ]
then
 ...
elif [ ! -d "$APP_NAME/node_modules" ]
then
    npm install --prefix ./$APP_NAME
fi

cd $APP_NAME &&
npm run setup &&
strapi start &

strapiPID=$!
wait "$strapiPID"

but no luck, the strapi container crash at start time
 exec user process caused "no such file or directory"

Any suggestions?

up!

Hello! Never tried and I haven't the time for now.
Hop the Strapi community can help you on this point.

@MatteoGioioso I tried localy run script with setup command and it's work. Could you share your complete script?
$APP_NAME is defined?

HI,
We just released a full rebuild of our docker image 🎉
We are closing the issues and open PRs to start fresh.
If you feel like something is missing or want to add things feel free to open new PRs ! 💯


Milia and dashing-rails (engine) integration

I'm trying to sort out how to integrate dashing-rails, which provides a Rails engine of Shopify's Dashing dashboard framework, into a Milia-enabled app.
https://github.com/gottfrois/dashing-rails
In general, what should be required to use an engine with Milia?  Setting the current tenant seems like an obvious requirement but I'm unsure of how or where to do that when using an engine.  Any tips?

With engines everything works the same as you are already used to.
In your engine simply setup your tenanted models as usual (adding a tenant reference in your engine migrations to those models and adding acts_as_tenant to your model file). That's it!


refact(hostsensorutils): refactors host sensor exports

Overview
This PR is scaffolded on top of #1141  and brings an extra level of refactoring.
The focus of this PR is to unexport private declarations that are not part of the interface exposed by this package.

cc: @dwertent @matthyx  - follow-up PR, for when the fate of #1141  is decided (I am trying to chunk changes into smaller consistent pieces)

@fredbi ok so this one can be changed into a PR?

@matthyx  yep up to you. You may either review and merge #1141 first, then I rebase this one for merge,
OR you may merge this one: it will embark #1141 and close it automatically.
I've split refactoring actions to relieve a bit the strain of reviewing such PRs.

@matthyx didn't see the dependent PR was merged... :)
Rebased. Ready.

thanks Fred !


This feature has gone over well with our players, and the issues it brings impact gameplay minimally. We'll keep this in for now.


Use Iframe height as trigger

Hi David,
I am trying to use the change of the iframe height (when internal content is loaded) as trigger for an event outside of the iframe.
I tried it like this
jQuery(document).ready(function($) {

  $('iframe#browser').load(function () {

if ($('iframe#browser').height() < 1000) { $('#related-wrapper').show(); }

else if ($('iframe#browser').height() > 1000 )  { $('#related-wrapper').hide(); }

});

});

Tests have shown that I get weird incosistent height values for the height of the iframe when I load content back and forth within the iframe, and therefore can't use it as a trigger
(height values like: 150px (inner 1) > 1400px (inner 2) > 1700px (inner 1 again) > 1400px (inner 2 again)....
But when I look at your example: http://davidjbradshaw.com/iframe-resizer/example/ I can see that with your callback function the height gets properly calculated, when I load content in the iframe forth and back.
Is there a way to use this as trigger? What would be the code? I somehow can't figure it out.
Your help is appreciated.
Kind regards
Jan-Philipp

Ok. I solved it by using the height in the callback function as trigger like this:
iFrameResize({

log                     : true,
enablePublicMethods     : true,

resizedCallback         : function(iframeHeight){ // Callback fn when resize is received

					var iframeheight = iframeHeight.height;
					if (iframeheight < 900) {$('#related-wrapper').show();};
					if (iframeheight > 900) {$('#related-wrapper').hide();};

				}

});

:-)


Load & Save Functions

Add feature to save stroke layout and append it

buttons can be hidden (through the options)
Default folder is with the snapshots
Autosave when stopping inking
Load works in append (existing drawings are not cleared)
Lond click to select file to save/load

beta version available here awaiting testing:
ppInk beta load-save.zip

new beta version
ppInk beta.zip


fix(usb_host): return ESP_ERR_NO_MEM on failed alloc in client register

Description

Fixes issue where ESP_ERR_NO_MEM was being silently discarded after cleaning up after a failed malloc in usb_host_client_register.
Testing

Verified that error is now properly return when forcing a memory alllocation failure.

Checklist
Before submitting a Pull Request, please ensure the following:

[x] 🚨 This PR does not introduce breaking changes.
[x] All CI checks (GH Actions) pass.
[x] Documentation is updated as needed.
[x] Tests are updated or added as necessary.
[x] Code is well-commented, especially in complex areas.
[x] Git history is clean — commits are squashed to the minimum necessary.

Thanks for your contribution.

sha=7962d54c4c59f71c751e772eb172d571a500329d


ControlPlane API

Problem Statement
The purpose of this task is to define a ControlPlane API which can be used to express and manage the lifecycle of Kong Kubernetes Ingress Controller (KIC) Deployments and Services.
Acceptance Criteria

[x] any number of ControlPlane resources can be created which results in KIC Deployments and Services
[x] ControlPlane resources have configuration options to tune each option of the KIC
[x] ControlPlane resources can be cleanly torn down
[x] https://github.com/Kong/gateway-operator/issues/28
[x] https://github.com/Kong/gateway-operator/issues/26
[x] https://github.com/Kong/gateway-operator/issues/27
[x] https://github.com/Kong/gateway-operator/issues/30
[x] https://github.com/Kong/gateway-operator/issues/32

Some of the work for this was done in the initial POC in 16ec8bc093918657fd0bbdc431b9954b0ba425f1. Marking this as resolved with all remaining work now moved into separate follow up issues.


Kitchen Sink : Rock-Paper-Scissors

Describe the feature
I would like to implement a rock-paper-scissors game,an interactive human vs computer game.Can you please assign this issue to me under the label of hacktoberfest?
Additional information

[x] Would you be willing to help implement this feature?

Awesome! Go ahead!


[firestore-bigquery-export] Cloud Function location is always configured as us-central1

Extension name: firebase/firestore-bigquery-export
Extension version: 0.1.40
Configuration values (redact info where appropriate):

LOCATION=europe-west2
DATASET_LOCATION=europe-west2
COLLECTION_PATH=accounts
DATASET_ID=firestore
TABLE_ID=accounts
USE_COLLECTION_GROUP_QUERY=No
DO_BACKFILL=Yes

I'm trying to deploy the firebase-bigquery-export extension using the following CLI command and the config above (located in
accounts.env):
firebase deploy --only extensions --project=project-X
However, the Cloud Function location is always configured as us-central1.
I've manually configured the extension through the Firebase console and I am able to correctly configure it as europe-west2 so I know that it should be possible to configure the location outside of the default us-central1.
I am using a Cloud Firestore database.

This issue has already been solved in #1750
You can reconfigure the cloud function location as follows:
LOCATION=europe-west2
firebaseextensions.v1beta.function/location=europe-west2
firebaseextensions.v1beta.v2function/location=europe-west2

Yup, thanks @marknesh this is the correct solution, here's an additional thread with info:
https://github.com/firebase/extensions/issues/1651#issuecomment-1767943894

Hi everyone! I got a similar issue... I can perfectly choose CF location, but I always face this error:

I would like to deploy my CFs at south america location, I can see that they are properly configured there, but for some reason queues/* is at us-central1.
I've ran a test using my CFs at us-central1 and everything worked, so I guess is something between my south american settings and something adding this us-central1 setting.
My desired location: southamerica-east1


nil check response to avoid SIGSEGV on failure

Fixes #17

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Tony Lee seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


What is difference between zipformer and zipformer_ctc models?

Hello guys,
What is difference between zipformer and zipformer_ctc models? Is there some article describe the differences in terms of WER and in terms of Real Time factor?
Thanks.

Please check the model.py and relevant source code for more details, currently we don't plan to do detailed benchmarking on performance for every recipe.


Dynamic blocks based on input length

While typing in an input field, how to change the blocks dynamically based on user input
for example
if input length = 5, blocks: [2, 3]
if input length = 10, blocks: [2, 3, 5]

Hacking Cleave.js
import React from "react";
import ReactDOM from "react-dom";
import Cleave from "cleave.js/react";

function App() {
  const [value, setValue] = React.useState("");
  const ref = React.useRef();
  const lower5 = value.length < 5;

  function onChange(e) {
    setValue(e.target.value);
  }

  React.useEffect(() => {
    // 6. And trigger manual focus when our condition is meet
    // 7. Problem is that it will trigger a focus at page load
    // ref.current.focus();
  }, [lower5]);

  return (
    <Cleave
      value={value}
      onChange={onChange}
      // 5. So we get a ref to inside input
      htmlRef={hRef => (ref.current = hRef)}
      // 1. Cleave does not track option changes
      // 2. so we need to trigger a rerender when our rule is meet
      key={lower5 ? "lower5.true" : "lower5.false"}
      // 3. above key will rerender Cleave with new options
      // 4. but focus is lost
      options={{
        blocks: lower5 ? [2, 3, 5] : [3, 2, 5],
        delimiter: " ",
        uppercase: true
      }}
    />
  );
}

const rootElement = document.getElementById("root");
ReactDOM.render(<App />, rootElement);

import React from "react";
import ReactDOM from "react-dom";
import Cleave from "cleave.js/react";
function App() {
const [value, setValue] = React.useState("");
const ref = React.useRef();
const lower5 = value.length < 5;
function onChange(e) {
setValue(e.target.value);
}
React.useEffect(() => {
// 6. And trigger manual focus when our condition is meet
// 7. Problem is that it will trigger a focus at page load
// ref.current.focus();
}, [lower5]);
return (
<Cleave
value={value}
onChange={onChange}
// 5. So we get a ref to inside input
htmlRef={hRef => (ref.current = hRef)}
// 1. Cleave does not track option changes
// 2. so we need to trigger a rerender when our rule is meet
key={lower5 ? "lower5.true" : "lower5.false"}
// 3. above key will rerender Cleave with new options
// 4. but focus is lost
options={{
blocks: lower5 ? [2, 3, 5] : [3, 2, 5],
delimiter: " ",
uppercase: true
}}
/>
);
}
const rootElement = document.getElementById("root");
ReactDOM.render(, rootElement);

One more example with dynamic length of blocks
import React from "react";
import ReactDOM from "react-dom";
import Cleave from "cleave.js/react";

function App() {
  const [value, setValue] = React.useState("");
  const ref = React.useRef();
  const delimiter = " ";
  const length = value.split(delimiter).join("").length;
  const blocksOf4 = Math.ceil((length + 1) / 4);
  const blocks = Array(blocksOf4).fill(4);

  console.log(length, blocksOf4, blocks);

  function onChange(e) {
    setValue(e.target.value);
  }

  React.useEffect(() => {
    // 6. And trigger manual focus when our condition is meet
    // 7. Problem is that it will trigger a focus at page load
    if (value.length !== 0) {
      ref.current.focus();
    }
  }, [blocksOf4, value.length]);

  return (
    <Cleave
      value={value}
      onChange={onChange}
      // 5. So we get a ref to inside input
      htmlRef={hRef => (ref.current = hRef)}
      // 1. Cleave does not track option changes
      // 2. so we need to trigger a rerender when our rule is meet
      key={blocksOf4}
      // 3. above key will rerender Cleave with new options
      // 4. but focus is lost
      options={{
        blocks,
        delimiter
      }}
    />
  );
}

const rootElement = document.getElementById("root");
ReactDOM.render(<App />, rootElement);

I did a PR that you may be interested where you can define dynamic blocks. This way there is no need to do workarounds: https://github.com/nosir/cleave.js/pull/572

Thanks for the proposal but dynamically blocks (options) updating is out of the lib's scope, comment added in PR https://github.com/nosir/cleave.js/pull/572 @iamandrewluca 's workaround looks good, thanks.


GitHub action

Add an official GH action so people can quickly wire up deploys

@tj Come back to Open Source. Please.

@duttaditya18 ahah sorry :p have to pay the bills

Started one here https://github.com/apex/actions, need to look into handle branches for staging etc


No support for custom schema name

The existing DB setup/migration capabilities do not offer the possibility to customize the schema name for DTF. So when multiple services are using the same database, you only have the option to use the dt schema in the multitenancy setup, but don't have the option to use a schema per service approach.
Is there a plan for supporting this in the future?

No plans for this currently. The idea is that the existing login-based multi-tenancy architecture should allow multiple services to share the same database with less overhead. Is separate schemas something you need?

Each service is independently deployed and is independently managing its DTF dependencies. Is the multi-tenancy architecture built for supporting multiple versions of DTF running on the same schema or it's built just for data segregation between services that have the same DTF version but share the DB resource? If one service will update their SQL provider dependency, the schema will be upgraded, while the other services remain running with the old SqlProvider version. Do you recommend using the multitenancy feature with this setup?

Yes, the multitenancy feature is designed such that all tenants in a single database have their schema updated together. If you wanted to isolate schema updates, you'd need to put tenants in separate databases (which also enables things like resource isolation, etc.).  Are you in a situation where each tenant could simply have its own database?

@cgillum does durabletask-mssql handle migrations?
What happens in this example where service A and service B share schema

service A upgrades durabletask-mssql to a new version
the new durabletask-mssql changes dt schema when DurableTask.SqlServer.SqlOrchestrationService.CreateIfNotExistsAsync is called
service A schema migration is deployed

At this point, neither service A, nor B were upgraded. (A will have the new version of durabletask-mssql, but not yet deployed).

will service A continue to work? (deployment would happend in a matter of minutes)
will service B continue to work? (next version of service B may not be even planned at this moment)

@mercer the intent is to have all DB schema upgrades be backwards-compatible, allowing old services to continue working with newer schemas. Does this answer your question?

What about forwards-compatible?
Given this scenario:

Service A upgrades schema to v2
Service A gets deployed with v2
Service B still has v1 of library, and downgrades the schema to v1
Service A now has v2 library working with v1 schema

Downgrading schema versions is not supported. They can only move forward. Service B in your scenario would use the existing v2 schema when deployed.

I see.
But in my scenario service B will execute, as a part of schema migration, so before the service is deployed, DurableTask.SqlServer.SqlOrchestrationService.CreateIfNotExistsAsync, with v1.
What will v1 CreateIfNotExistsAsync() do when it sees v2 schema (upgraded by Service A)?

Silently continue (forwards compatibility)?
Fail (does not accept a newish schema)?
Change anything in dt.Versions?

The v1 CreateIfNotExistsAsync code will do nothing if it sees a newer schema version exists in the database. You can find the implementation for this here:
https://github.com/microsoft/durabletask-mssql/blob/24fdfe6979f5af3d57b4f52aaa3eb30a2ba5038f/src/DurableTask.SqlServer/SqlDbManager.cs#L54-L59
In this snippet, DTUtils.ExtensionVersion represents the schema version included in the currently running service ("V1" in your scenario) and currentSchemaVersion represents the version found in the database (in the [SemanticVersion] table - e.g. V2).

Thank you for all the info @cgillum, our concerns were settled, will close this issue now 🥇


JSONTree expands when loaded from localhost, not from external IP address

I'm using react-json-tree 0.6.1 with node 5.3.0, npm 3.3.12 on OSX 10.10.5 and Chrome (recent).
The site works perfectly if I access it from localhost:3000. Going to my computer's external address (say, <IP_ADDRESS>:3000) the app works fine except that react-json-tree items do not expand (!) Clicking the twisty triangle does not change the appearance - it stays collapsed.
I confess I didn't read much of the documentation - I just slapped the <JSONTree... /> component in place, and it worked right away.
Any ideas how to troubleshoot this? Thanks.

Update:

Works with localhost on Chrome 50.0.2661.49 beta (64-bit), doesn't work on external IP address
Does not work on either localhost or external IP with Firefox 45.0.1

Update: Never mind. An update of many things in my package.json (including to<EMAIL_ADDRESS>makes things work whether to localhost or an external IP address. I don't have a specific fix that I can identify, but it's fine now (which is what I would have expected all along...)

Nope. I am wrong in the previous symptom description... I am using a slightly complicated npm/hot loading/etc. setup (with which I am not completely familiar). But I have learned this: the bad behavior is dependent on the port I connect to, not the hostaddress. Specifically:
either-localhost-or-computers-external-ipaddress:3001 - react-json-tree works as expected
either-localhost-or-computers-external-ipaddress:3000 - items don't expand (the original complaint)
I know this sounds loony: but can anyone guess why connecting to a different port would make a difference? (Any guesses? Any thoughts of avenues to investigate?) Thanks again.

That does sound weird. To clarify - are you trying it with example project? Did you try to trace what happens in onClick handler? (one thing I can think of - there are two handlers, so they could conflict somehow, but it's hard to blame ports for that, obviously).

No, this occurs in my own project, which has some fancy proxying going on with both ports 3000 & 3001 (there are a lot of actors involved: webpack hot reloading, webstorm, ....) I was not able to reproduce in the tests/example project, so I don't think it's anything related to react-json-tree itself. I was just hoping to  spark a troubleshooting thought from someone here.
I think react-json-tree is off the hook, and that I need to investigate my build system further. You all can take the rest of the day off :-)

Ok, got it :)


Support for AAD authentication in MSSQL connector

Description
I needed to use this block to be able to login with Active Directory. This is useful if the database uses a passwordless strategy.
authentication: {
    type: "azure-active-directory-default"
  }

Check also https://github.com/timothyhoward/evidence-connector-mssql/pull/1
Screenshots

Checklist

[x] For UI or styling changes, I have added a screenshot or gif showing before & after
[x] I have added a changeset
[x] I have added to the docs where applicable

@mycaule Thanks for this PR
I feel like SQL login is probably not the right name for the username/pw auth flow?
perhaps Username / Password is more descriptive

@mycaule Thanks for this PR
I feel like SQL login is probably not the right name for the username/pw auth flow?
perhaps Username / Password is more descriptive?

I used this because it is the vocabulary used by Microsoft, SQL authentication refer to user/pass created in SQL Server
As opposed to the username/password in Active Directory of your organization.
You can check this screenshot
https://learn.microsoft.com/en-us/azure-data-studio/connect#new-installation-of-azure-data-studio-140-and-higher

Also added the authentication types @timothyhoward supported in his plugin.

Not sure what's going on with the checks here, something related to the Netlify preview URL. @ItsMeBrianD @archiewood any ideas?

@ItsMeBrianD Please review

LGTM

@archiewood / @hughess is this good to go from a product perspective?

@ItsMeBrianD as long as this works, it's good to go

QA done on this

Mega thank you @mycaule - and thanks for your patience while we reviewed


feat: add useHashSearchParams hook

Description
A new hook  useHashSearchParams, support get/set search params in URL hash part, which is implemented based on useHash hook in react-use. It useful to handle SPA(Single Page Application) route params that based on hash mode.
For example：http://a/b/c#/a/b/c?id=1&name=jim
// get specified search param
const [id, setId] = useHashSearchParams('id')  // id: "1"
// set specified search param
setId(2) // http://a/b/c#/a/b/c?id=2&name=jim

// get whole search params
const [params, setParams] = useHashSearchParams()  // params: {id: "1", name: "jim"}
// set whole search params
setParams({...params, name: 'jack'})  // http://a/b/c#/a/b/c?id=1&name=jack

// return default value if specified param absences
const [id, setId] = useHashSearchParams('id2', '2')  // id: "2"

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as before)

Checklist

[x] Read the Contributing Guide
[x] Perform a code self-review
[x] Comment the code, particularly in hard-to-understand areas
[x] Add documentation
[x] Add hook's story at Storybook
[x] Cover changes with tests
[x] Ensure the test suite passes (yarn test)
[x] Provide 100% tests coverage
[x] Make sure code lints (yarn lint). Fix it with yarn lint:fix in case of failure.
[x] Make sure types are fine (yarn lint:types).

@streamich request for comment


Message sent to deallocated instance when repeatedly performing operations that will show the HUD

There's an issue in our app where if the user repeatedly performs a post status operation it will trigger the following crash.
*** -[UIViewAnimationState configureAnimation:forLayer:forKey:]: message sent to deallocated instance 0x174bdae50

The specific flow for this is:

Click post status
Composition modal dismisses
HUD shown attached to window
Status upload succeeds or fails
HUD hidden with 2 second delay

If this sequence is repeated rapidly after a few operations this crash can be reproduced.
This is what our hideHUDAfterDelay helper method looks like:
- (void)hideHUDAfterDelay:(float)delay withMessage:(NSString*)message andCompletionType:(HUDCompletionType)type {
    // if a message has been supplied set it
    if (![NSString isEmpty:message]) {
        HUD.labelText = message;
    }

    // show the spinner for the time minus half a second then switch to checkmark and hide
    if (delay > switchToCheckDelay) {
        switch (type) {
            case kHUDCompletionX: {
                DLog(@"change hud to X delay:%f", delay-switchToCheckDelay);
                dispatch_time_t popTime = dispatch_time(DISPATCH_TIME_NOW, delay-switchToCheckDelay * NSEC_PER_SEC);
                dispatch_after(popTime, dispatch_get_main_queue(), ^(void){
                    [self changeHUDLabelTo:@"×" andHide:YES];
                });
                break;
            }
            case kHUDCompletionCheck: {
                DLog(@"change hud to check delay:%f", delay-switchToCheckDelay);
                dispatch_time_t popTime = dispatch_time(DISPATCH_TIME_NOW, delay-switchToCheckDelay * NSEC_PER_SEC);
                dispatch_after(popTime, dispatch_get_main_queue(), ^(void){
                    [self changeHUDLabelTo:@"✓" andHide:YES];
                });
                break;
            }
            case kHUDCompletionNone: {
                [self performSelector:@selector(hideHUD) withObject:nil afterDelay:delay];
                break;
            }
            default:
                break;
        }
    } else {
        // delay too short to switch to checkmark
        [self performSelector:@selector(hideHUD) withObject:nil afterDelay:delay];
    }
}

I've pasted a portion of the BT below.
(lldb) bt
* thread #20: tid = 0x211a6, 0x0000000181ec63d4 CoreFoundation`___forwarding___ + 968, queue = 'com.airg.imagequeue', stop reason = EXC_BREAKPOINT (code=1, subcode=0x181ec63d4)
    frame #0: 0x0000000181ec63d4 CoreFoundation`___forwarding___ + 968
    frame #1: 0x0000000181dcac4c CoreFoundation`_CF_forwarding_prep_0 + 92
    frame #2: 0x000000018698f7c8 UIKit`-[UIViewAnimationState animationForLayer:forKey:forView:] + 712
    frame #3: 0x0000000186705404 UIKit`-[UIViewAnimationState actionForLayer:forKey:forView:] + 44
    frame #4: 0x00000001866c9da0 UIKit`-[UIView(CALayerDelegate) actionForLayer:forKey:] + 140
    frame #5: 0x0000000186020c9c QuartzCore`-[CALayer actionForKey:] + 104
    frame #6: 0x0000000186016678 QuartzCore`actionForKey(CALayer*, CA::Transaction*, NSString*) + 100
    frame #7: 0x00000001860164e4 QuartzCore`CA::Layer::begin_change(CA::Transaction*, unsigned int, objc_object*&) + 176
    frame #8: 0x00000001860187d0 QuartzCore`CA::Layer::set_position(CA::Vec2<double> const&, bool) + 240
    frame #9: 0x00000001860186d4 QuartzCore`-[CALayer setPosition:] + 48
    frame #10: 0x0000000186018664 QuartzCore`-[CALayer setFrame:] + 456
    frame #11: 0x00000001866c9a3c UIKit`-[UIView(Geometry) setFrame:] + 292
    frame #12: 0x00000001866ddc14 UIKit`-[UIImageView _setViewGeometry:forMetric:] + 188
    frame #13: 0x0000000186728158 UIKit`-[UIActivityIndicatorView _feedTheGear] + 1276
  * frame #14: 0x000000018672a178 UIKit`-[UIActivityIndicatorView initWithActivityIndicatorStyle:] + 184
    frame #15: 0x0000000100228f24 <Redacted>`-[MBProgressHUD updateIndicators](self=0x00000001564f9c20, _cmd=0x00000001003bfeca) + 428 at MBProgressHUD.m:482
    frame #16: 0x00000001002269b0 <Redacted>`-[MBProgressHUD initWithFrame:](self=0x00000001564f9c20, _cmd=0x0000000186ea5004, frame=CGRect at 0x0000000104dbba60) + 1640 at MBProgressHUD.m:204
    frame #17: 0x0000000100226b54 <Redacted>`-[MBProgressHUD initWithView:](self=0x0000000000000000, _cmd=0x0000000186ee0dcc, view=0x0000000154d10f20) + 320 at MBProgressHUD.m:213
    frame #18: 0x00000001002259e0 <Redacted>`+[MBProgressHUD showHUDAddedTo:animated:](self=0x00000001004a0270, _cmd=0x00000001003aae92, view=0x0000000154d10f20, animated=true) + 104 at MBProgressHUD.m:116

Any ideas? Thanks in advance.

Looks like you're doing UI operations on a background thread (thread #20, com.airg.imagequeue). You need to make sure you're always dealing with UIKit (and MBProgressHUD) on the main thread / queue.

You are 100% correct, my bad. I didn't even see that. Thank you so much for the quick response!


Summarize a repository

Can Cody be used to summarize a repository, as ti what it does on a high level, what technologies are used, etc?

@anujkumar-df Hey, For this, just ask Cody the question in an indexed repository, "Can you please summarise this project for me, tell me it's working and the technology used for the development?". This will suffice to get the answers you want.

Interesting, can you share what "indexed repository" actually means? I am using the VSCode extension, but would I need to use the Code app?

Hi @anujkumar-df
You need the Cody app if it isn't publically available on GitHub/GitLab. If the repo is public, we have a Discord bot that can do it.


Minimal support of taking screenshots on ios sim

In scope of #171 added very basic support of taking screenshots when running tests on ios simulator.
It takes two screenshots: one before and after the case.
Quite new to this project, please let me know if something can be improved, thanks!

Hey @maxkomarychev ,
Thanks a bunch for putting the effort to create this PR, however another PR #541 is very close to being merged, and already added all the features mentioned in #171 on both platforms. You're welcome to take a look and see if something you want is missing there, and maybe collaborate.
I will be closing this OR now, thanks again for submitting it, I really appreciate it!


Comparison section

Corrected the stylistic design of the section.

Thanks for your patience, I get to this just now.
Could you rebase? I'll merge it right away :+1:

@TomasVotruba updated

Thank you, so fast


The built-in MMLU prompt adapter doesn't reproduce paper performance

Hi, I found the built-in MMLU prompt adapter has much lower performance, I need to adjust the default prompt to reproduce the llama paper performance.
The built-in MMLU prompt adapter is
The following are multiple choice questions (with answers) about college medicine.

Question: Which of the following is not a true statement?
A. Muscle glycogen is broken down enzymatically to glucose-1-phosphate
B. Elite endurance runners have a high proportion of Type I fibres in their leg muscles
C. Liver glycogen is important in the maintenance of the blood glucose concentration
D. Insulin promotes glucose uptake by all tissues in the body
Answer:

With this default prompt, the performance is much lower and doesn't match the paper performance of llama-13b.
The prompt used in another git is without the "Question: " prefix, which is like
The following are multiple choice questions (with answers) about college medicine.

Which of the following is not a true statement?
A. Muscle glycogen is broken down enzymatically to glucose-1-phosphate
B. Elite endurance runners have a high proportion of Type I fibres in their leg muscles
C. Liver glycogen is important in the maintenance of the blood glucose concentration
D. Insulin promotes glucose uptake by all tissues in the body
Answer:

This prompt can exactly reproduce the paper performance of llama-13b.
Should we adjust the MMLU default prompt adapter?

Thanks for the question.  When designing HELM, we discussed quite a bit about what prompts should be used, since we know that the prompts definitely affect performance.  In the end, because we had so many scenarios and models, it seemed more principled to choose fairly simple/default prompts and we prioritized consistency, as long as the prompt was reasonable.  In this case, putting "Question: " is something that's reasonable, and in some sense, you could say that it's a failing of the LM for not being able to handle that properly.  So I think we'd prefer to keep the default prompt in this case with the knowledge that there are many things you could do to get better accuracy out of any particular LM if you tuned the prompt.

Closing this issue; please re-open if you have follow up questions.


A specific request .

Hey thanks for adding the requested feature to change default download directory !
You have given me the binary file of the code to be able to run on Android . Thats of version 1.4 .
Can you please give the binary file of this version 1.5 to run this on Android .

V1.5 Android binaries are there on the releases page. Check again!

oops  ! got it . closing this
thanks


Deploy Artifactory to speed up deployment

https://www.jfrog.com/open-source/#os-arti - just from running the build a bit - it seems like the parts taking the most time are downloading the artifacts over and over - it's likely worth adding timing when we do #5 as well.

@akarasev This looks good - let's wait to close this until it's live


Managed Identity retries on 410 status

fixes #28568

I have interest in seeing if this PR can be merged, as it has affected my team on several occasions on production clusters.


Add timestamp logic to groovy file

Because we will be running this without checking in new things to this
repo we will not be moving the commit hash that we usually use for a
build number.  Instead we use the logic that we have in the stability
groovy file.

test ci please

test ci please


query time

Currently, DocsGPT use VectorDBQA. According to my local test in Jupyter Note, a query costs 1-2 mins.
   chain = VectorDBQA.from_chain_type(
      llm=llm,
      chain_type="map_reduce",
      vectorstore=docsearch,
      k=4,
      return_source_documents= True ,
      search_type = 'mmr',
      chain_type_kwargs={"question_prompt": p_chat_reduce,
                         "combine_prompt": p_chat_combine}
      )
   result = chain({"query": question})

When I change it to ChatVectorDBChain.from_llm, a query costs 15-25 seconds.
chain = ChatVectorDBChain.from_llm(
    llm=llm,
    vectorstore=docsearch,
    prompt=p_chat_combine,
    qa_prompt=p_chat_reduce,
    top_k_docs_for_context=3,
    return_source_documents=False,
    # kwargs= {
    #     "question_prompt": p_chat_reduce,
    #     "combine_prompt": p_chat_combine
    #     }
    )

Would you please consider to make the query faster?

Done, updated, thank you!

Done, updated, thank you!

hi sire @fjun99  @dartpain please test the answer
it's seem the reply not good and smart now~

Hmm.
I see, its a bit more strict now, but for my mock tests it works well.
I suggest you provide me with your vectors and mock questions so i can try work on that specific case

wow……it's see i need some skill to export this.how can i do this

Hmm. @137-rick I see, its a bit more strict now, but for my mock tests it works well. I suggest you provide me with your vectors and mock questions so i can try work on that specific case

hi, sir i found out why not smart
because map_reduce is tree sum
it s will scan more doc and make sum
make the sum answer the question
it s slow but provide more info


NodeJS 18

Error on install ring at Nodejs 18:
$ iobroker url iobroker.ring --host ioBroker --debug
install iobroker.ring
Installing iobroker.ring... (System call)
npm ERR! code EBADENGINE
host.ioBroker Cannot install iobroker.ring: 1
ERROR: Process exited with code 25

Hello @HeGa59,
Before we can allow Node 18, Ring Api Client has to update:
https://github.com/dgreif/ring/blob/866fcbcdf12561d3d5dd512aa1d4f73134c25490/package.json#L100
Feel free to open an issue there
Best Regards
Thiemo

https://github.com/dgreif/ring/issues/1048#issuecomment-1264675424


Update Angular 4 link

Hey, ngforage author here.
Changed the link to point to the github repo instead of npm as ngforage has been deprecated and there are separate packages up for Angular 4, 5 and 6 :)

Cool thanks :+1:


Request Event, Expand Row

please add event to expand row to give more information about data

Hi, @teguhsatria892 .
Thanks for your request. But I don't know what features you want, more details please.

Is it resolved? If not, reopen please.


SA host, group tab test

replacing https://github.com/ansible/ansible-ui/pull/1901

https://issues.redhat.com/browse/AAP-22176


ThreeJS template project

Hi there,
somte time ago i've setup a ThreeJS template project for playing around and later on i decided to publish it for public usage.
Maybe you want to add this project or i would do when you tell me where in this big list it would fit best:
Here the link to my project: https://github.com/JohnnyDevNull/ng-three-template

you can add it to the readme


[RFC] Always create sandbox init process and remove the field sandbox_pidns

In hyperstart, the pod_init process has its own net/pid/uts/ipc ns.
And in kata, these namespaces are also essential.
Net ns for the sandbox should be always created since the container's
network should be isolated from the vm's initial network to
avoid messing.
Pid ns for the sandbox should be always created since the containers
always join this ns. We can use new pid ns for each container,
but in most cases, they are shared the same sandbox init pid ns.
Uts ns for the sandbox should be always created since the hostname
is set in it without mess it up with the vm's initial UTS.
Ipc ns for the sandbox should be always created since the containers
always join this ns. We can use new ipc ns for each container,
but in most cases, they are shared the same sandbox init ipc ns.
So sandbox init process is better to be always created
in its own net/pid/uts/ipc ns and let the later containers
join in. To make it different from hyperstart, we can use
some config to tell the container not to join it in
some cases later.
Signed-off-by: Lai Jiangshan<EMAIL_ADDRESS>

So sandbox init process is better to be always created
in its own net/pid/uts/ipc ns and let the later containers
join in. To make it different from hyperstart, we can use
some config to tell the container not to join it in
some cases later.

I have 2 concerns with this:

The current k8s plan is to support 3 types of sandbox pid namespaces: host, shared or private but here we introduce a fourth one where some containers would share the sandbox namespace while other would not. There are no use cases for this

There is currently no way to specify if a container wants to use the shared pid ns or not, so we'd always be using the shared pid ns.

@sameo I agree with you, this is represent the shared mode.
We can add other option for private mode. (or sandbox_pidns = false represents for this mode?)
Even pid namespaces is private the sandbox init process should also be created, regardless the container join it or not.

We can add other option for private mode. (or sandbox_pidns = false represents for this mode?)

I think it would be cleaner if we change the pod boolean for pid_ns to be a string that cannot be empty, but either be private or shared.

Even pid namespaces is private the sandbox init process should also be created, regardless the container join it or not.

Yes, I agree with that. We should document it as part of the proto file.

@laijs @sameo This whole thing about shared pid ns is related to k8s concept of pod having its containers sharing or not the same namespaces. First, that means that if k8s creates a pod with this flag, it expects all its containers to share those ns, therefore, there is no need to get this information from the container, it's more appropriate to get it from the pod IMO.
The discussion about shared vs private is exactly the same than having a boolean, or am I missing something ?
Also, it is way more convenient for us to handle this when creating the sandbox. This way, we can reexec and run this reexec inside the expected namespaces. Same way, we know that we can kill this process when destroying the sandbox.

@sboeuf

The discussion about shared vs private is exactly the same than having a boolean, or am I missing something ?

Almost, yes. With a boolean, the default value is false and thus by default we don't share the pid ns. With a string, it has no meaningful value and you can even force the caller to select one.

@sameo ok then I consider this is a detail in this conversation, and I am fine with both ways. But about the pidns flag, I would really prefer to keep this as part of CreateSandbox().

@sboeuf Yes, I fully agree. Being shared by default and having containers opt out of it would be something that can't be expressed by the CRI afaiu.

Ok, we agree to keep the sandbox_pidns as a knob to tell the agent whether all the containers should join the sandbox init process's pidns or not.
I will close this pr, and add a new issue for the remanning requirement in #59

Thank you @laijs


Undefined variable inputFocusOutlineOffset

There appears to be only one theme, the aura-light that provides this variable, but it is required in the base theme.
ERROR  [sass] Undefined variable.
outline-offset: $inputFocusOutlineOffset;
                ^^^^^^^^^^^^^^^^^^^^^^^^
assets/css/theme-base/_mixins.scss 21:21                         focused-input()

flipping through compiled themes, seems like this should default to 0

Fixed internally.
Thank you!

this variable "$inputFocusOutlineOffset" seems to be absent in https://github.com/primefaces/primeng-sass-theme/releases/tag/17.12.0, I cannot make my project work :(


PUB 1269 send media applications report

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/PUB-1269
Change description

Added new endpoint to handle sending media requests
Added new Csv creation service
Added new env variable for the team email which comes from the key vault
Converted maps to be <String, Object> rather than <String, String> (this is required for sending files and does not break existing notifications)

Screenshots of the notify email and the CSV that is being set (link redacted):

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Details

Done :)

Looks good! Needs some integration tests adding for new the endpoint too

Done :)


Vuex change data

hi,
the problem is when changing green updateTree gives them change ex:
[{"id": 1, "text": "Requests", "text": "Requests", "state": {"editable": false}, "children": transforts:
[{"text": "Requests", "data": {"text": "Requests"}, "state": {"selected": false, "selectable": true, "checked": false, " expanded ": true," disabled ": false," visible ": true," indeterminate ": false," matched ": false," editable ": false," dragging ": false," draggable ": true," dropable " : true}, "children":
we see that ID has disappeared

Hi!
Could you please show me a real example? How do you call the updateTree?

yes
the code vuejs:
`import LiquorTree from 'liquor-tree'
import {mapGetters} from 'vuex'
import store from '@/store'
export default {
data () {
return {
  nodeid: 1,
  treeOptions: {
    store: {
      store: this.$store,
      // key: 'tree',
      getter: () => {
        return this.getTypeDate
      },
      dispatcher (tree) {
        store.dispatch('updateTree', tree)
      }
    },
    checkbox: true
  }
}

},
components: {
tree: LiquorTree
},
computed: {
...mapGetters([
'getTypeDate',
'getLoading'
])
},`
this code vuex:
Action:
export const updateTree = ({commit, state}, payload) => {   commit('setTypeTree', payload) }
mutations:
export const setTypeTree = (state, data) => {   state.typeData = data }

Hi. Yes, I've found the bug too. Could you please try story your node in in data property. I will not be changed. I will raise a new issue

ok


[ConstraintSystem] [NFC] add tests for label matching diagnostics

This patch add test cases for label matching diagnostics.
This haven't changes for compiler.
This is a part of following my other project.
https://github.com/apple/swift/pull/30444
I am planning to make many changes of diagnostics there particular corner cases.
It is useful to bake current behavior to repository as test cases
before reviewing my that project.

@swift-ci please smoke test

I will make two new other patches.

(1) I make argument_matching.swift by renamed from keyword_arguments.swift.
And move labeling cases in diagnostics.swift to it.

(2) I break lines expectation in object_lierals_osx.swift.

After these works will be finished, I will adjust this patch to make it only add new tests.

Sounds like a plan!

@LucianoPAlmeida  @xedin I moved test cases to argument_matching.swift.
@swift-ci please smoke test

LGTM! @omochi :)


Hotfix/identity 3.1.2

Packages impacted by this PR
@azure/identity
Issues associated with this PR
This PR merges changes from hotfix branch hotfix/identity_3.1.2 to main
Original issue - https://github.com/Azure/azure-sdk-for-js/issues/24003
Fixed by shipping version<EMAIL_ADDRESS>through the hotfix branch
Describe the problem that is addressed by this PR
Merging hotfix branch release commits back into main branch
Checklists

[x] Added impacted package name to the issue description
[x] Added a changelog (if necessary)

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-identity

This PR doesn't cause the API update reported though. The API update happened in a different PR to main branch earlier today - https://github.com/Azure/azure-sdk-for-js/commit/5706943c19171965d8fd2394a25aaa3e60fbd4fc
I think all subsequent PRs into main will need the API to be approved @xirzec
But how did the CI allow the previous PR to be merged into without the API approval? @ckairen

azp run js- identity

/azp run js - identity

This PR doesn't cause the API update reported though. The API update happened in a different PR to main branch earlier today - 5706943 I think all subsequent PRs into main will need the API to be approved @xirzec But how did the CI allow the previous PR to be merged into without the API approval? @ckairen

We don't block on merges, we only block on releases. As for the API update, there's probably a small window between the pipeline run and the creation of API review ticket where it gets a bit confusing. @praveenkuttappan is looking into the issue


#2214  Vulkan uses gl_InstanceIndex and gl_VertexIndex instead of gl_InstanceId and gl_VertexId as the latter are not supported.

PR Details
Vulkan uses gl_InstanceIndex and gl_VertexIndex instead of gl_InstanceId and gl_VertexId as the latter are not supported.
Related Issue
https://github.com/stride3d/stride/issues/2214
Motivation and Context
gl_InstanceId and gl_VertexId are removed in https://github.com/KhronosGroup/GLSL/blob/main/extensions/khr/GL_KHR_vulkan_glsl.txt
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] I have built and run the editor to try this change out.

I'm guessing those are strictly nomenclature changes then ? Thanks !


Unable to create new service: ChromeDriverService

I would like to run automation tests using Selenide latest version on selenium server(local machine).
CMD command:
java -jar selenium-server-standalone-3.141.59.jar -port 4444
Test Configuration:
@BeforeMethod(alwaysRun = true)
public void tearUp(Method method) {
Configuration.browser = WebDriverRunner.CHROME;
Configuration.remote = "http://localhost:4444/wd/hub";
open("https://google.com");
}
I got an error:
org.openqa.selenium.SessionNotCreatedException: Unable to create new service: ChromeDriverService
I guess i need to add chrome driver?
System.setProperty("webdriver.chrome.driver","path to chromedriver.exe");

@DEEKEEN No. Selenide downloads chromedriver automatically (in most cases).
Please attach a full tracktrace of your error, there should be more details.


fixed key error when adding multiple sector constraints

I wanted to add multiple sector constraints, for example:
mapper1 = {
     "AAPL": "ESG",
     "PG": "NOTESG"
     .
     .
     .
   }
lower1 = {
    "ESG": 0.1
}
upper1 = {
    "NOTESG": 0.1
}

ef.add_sector_constraint(mapper1, lower1, upper1)

mapper2 = {
     "AAPL": "TECH",
     "PG": "DEFENSIVE"
     .
     .
     .
   }
lower2 = {
    "DEFENSIVE": 0.1
}
upper2 = {
    "TECH": 0.2
}

ef.add_sector_constraint(mapper2, lower2, upper2)

Where the main problem was that some assets belonged to more than one sector, and wanted to mix different sector classifications, such as debt can be subdivided between corporate and governmental.
When adding the second constraint, it looks for all registered tickers, where some may not be present in the second mapper, thus by changed the [] dict indexing notation for a .get notation, a None is returned, and therefore not added to the optimization constraint sum in:
       for sector in sector_upper:
           is_sector = [sector_mapper.get(t) == sector for t in self.tickers]
           self.add_constraint(lambda w: cp.sum(w[is_sector]) <= sector_upper[sector])
       for sector in sector_lower:
           is_sector = [sector_mapper.get(t) == sector for t in self.tickers]
           self.add_constraint(lambda w: cp.sum(w[is_sector]) >= sector_lower[sector])

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.12 :warning:

Comparison is base (6f699fd) 99.42% compared to head (3feb53a) 99.30%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #531      +/-   ##
==========================================
- Coverage   99.42%   99.30%   -0.12%
==========================================
  Files          33       33
  Lines        5072     5072
==========================================
- Hits         5043     5037       -6
- Misses         29       35       +6

Impacted Files
Coverage Δ

pypfopt/base_optimizer.py
97.54% <100.00%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Thanks you for the fix.


add the new cask analytics info to brew info <some cask> as well

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

This addresses my comment on #7566 and adds the new brew cask info ... analytics stats added by @tharun208 in #7580 into brew info <some cask> as well.
Had to tweak a few things so that strings are returned at a lower level instead of direct puts.  Also changed some names to hopefully clarify which functions print a string and which return a string.
Here's the output of this working:
$ brew cask info dropbox
==> Analytics
dropbox: latest
https://www.dropbox.com/
Not installed
From: https://github.com/Homebrew/homebrew-cask/blob/master/Casks/dropbox.rb
==> Name
Dropbox
==> Artifacts
Dropbox.app (App)
install: 2,095 (30 days), 6,517 (90 days), 29,650 (365 days)

$ brew info dropbox
Error: No available formula with the name "dropbox"
==> Analytics
Found a cask named "dropbox" instead.

dropbox: latest
https://www.dropbox.com/
Not installed
From: https://github.com/Homebrew/homebrew-cask/blob/master/Casks/dropbox.rb
==> Name
Dropbox
==> Artifacts
Dropbox.app (App)
install: 2,095 (30 days), 6,517 (90 days), 29,650 (365 days)

well the tests pass! but they're wrong. 😭
This breaks brew info --analytics <formula> because table_output still prints things out directly instead of returning strings.  When --analytics is used the "==> Analyticstitle is printed LAST which definitely isn't right. I could fix this by convertingtable_output` to also return a string, but at this point I think I should step back and examine this approach.
Is changing everything here to return string even generally desirable?  It does make certain refactors/abstractions easier but also there's some simplicity in having these functions print to stdout directly and this would strip all of that way.
There's probably some simpler way to accomplish my original goal so I'm inclined just to abandon this approach 🤷♂

well the tests pass! but they're wrong. 😭

Well, incomplete rather than wrong unless there's a direct test for this functionality.

Is changing everything here to return strings even generally desirable? It does make certain refactors/abstractions easier but also there's some simplicity in having these functions print to stdout directly and this would strip all of that way.

I think consistency would be valuable. Either all of these methods should return strings or none of them should (and the names should be clear on whether they will output or not).


storage/sql: Added WITH EXPIRATION option for SPLIT AT

PR: https://github.com/cockroachdb/cockroach/pull/38004
From release notes:

Added WITH EXPIRATION option for SPLIT AT. [#38004][#38004] {% comment %}doc{% endcomment %}

@ericharmeling, since you're working on split stuff, you should take this. Perhaps it's already part of your open PR?

I can add it in there for sure!


Notice: A non well formed numeric value encountered for PHP 7.1

There are few issues with PHPExcel after PHP version update (5.6 -> 7.1) but this one is the most annoying: https://github.com/PHPOffice/PHPExcel/issues/1110
Did you have a chance to review it and adapt ExcelBundle?
Thanks!

As a workaround, i just replaced :
ResponseHeaderBag::DISPOSITION_ATTACHMENT

With the string :
'attachment'

And it worked fine.


When displaying the update candidates, display the name of the channel the update was found in

The example output would be:
Updates found:
  org.yaml:snakeyaml                                    1.29                 ==>  1.33  [EAP 8.0]

Note: ChannelSession in wildfly-channel will have to be modified to expose this information.
Note2: The update candidate would have to preserve this information to display diff correctly.

Hi @spyrkob could you please assign this to me?


Problem with php_pear on Ubuntu?

Hi Folks,
A call like this:
php_pear "odbc" do
    action :install
end

Results in an error:
RuntimeError
------------
Package obdc not found in either PEAR or PECL.

But switching this to package seems to work:
package php5-odbc do
    action :install
end

This is on a Virtualbox Precise64 box (Ubuntu). I have seen some indications that this may be related to Ubuntu. Is there a different in such an environment?
Thanks.

It actually looks like the package name is PDO_odbc in pecl. Trying to install just odbc from the command line fails with your error, but using the full extention name runs a successful installation


WIP: Add support for deleting OCP clusters

Signed-off-by: Andrej Podhradsky<EMAIL_ADDRESS>

@apodhrad Since the PR is WIP making it as a Draft !

Ping @apodhrad !

@apodhrad If its ready, please remove WIP and Draft state !

not ready...I still need to finish some changes and test it

Now, waiting until the wrapanapi's PR is merged
https://github.com/RedHatQE/wrapanapi/pull/455

@apodhrad I have pinged you there as the wrapanapi PR is waiting to update from long time.

Ping @apodhrad , Are you willing to continue on it ?

@ogajduse ^

Can be closed, followed with #134

Good chunk of this PR should go into https://github.com/RedHatQE/wrapanapi for OCP resource identification and deletion.

@jyejare Ack on this review, will be resolved in https://github.com/RedHatQE/cloudwash/pull/134. Thanks

@oharan2 Do you wish to close this PR now since #134 is arrived ?


wivrn: 0.19 -> 0.20

https://github.com/WiVRn/WiVRn/releases/tag/v0.20
Things done
Ported to: https://github.com/WiVRn/WiVRn/releases/tag/v0.20
added glib as a dependency

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

@PassiveLemon

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

I've done some testing and maybe avahi-daemon.service is not being properly started?

@tomodachi94 your wish is my command!

Will fully review in a moment, currently running some tests on my machine

I've been keeping up to date in my flake repository: https://github.com/PassiveLemon/lemonake/tree/master/pkgs/wivrn
Would you mind updating the PR with these? Ignore the missing sources/versions, that's just how the repository is designed.
Also here's a message from Xytovl:
When you plan to update to 0.20, there's quite a few changes related to build:

the default IPC socket path has been changed to $XDG_RUNTIME_DIR/wivrn/comp_ipc, note the change from _ to /, this can be overriden at compile time by setting XRT_IPC_MSG_SOCK_FILENAME but I do not recommend it as it may cause issues with inconsistent docs.
There is now a GUI with WIVRN_BUILD_DASHBOARD, which adds an additional dependency on qt, has a .desktop file. WIVRN_CHECK_CAPSYSNICE compile option adds check in gui for cap_sys_nice, you can safely set it to off if nix takes care of it
Configuration is now smarter: it is possible to omit encoder, codec or group in the encoders array and it will have a sensible value (hardware encoder, best quality codec supported by headset + encoder, grouping encoders of the same type together)
monado patches have changed (make sure your list is up to date), WIVRN_USE_SYSTEMD is no longer necessary for reliable application autostart feature.

The link I provided above covers the dashboard and the monado patches, but the smarter configuration and IPC socket path are not relevant to the package/module so those can be disregarded.

Thanks for the feedback @PassiveLemon I'll work on adjusting the package.

How do I make it a Qt package? I need to call it like described in: https://nixos.wiki/wiki/Qt or else I don't get all the qt dependencies I need.

It should be called with pkgs.qt6Packages.callPackage since the dashboard uses Qt 6

If you would like, I could PR 0.19 -> 0.20 since I pretty much have it all done anways

If you would like, I could PR 0.19 -> 0.20 since I pretty much have it all done anways

Ok I can close this PR then. Thank you!

also, if you don't mind @ me on that new PR please :) @PassiveLemon

Submitted a PR ^^


Bug - Application Skill validator test failing intermittently

Description
Two unit tests fail sporadically causing build issues on PRs and dependency updates:
1) Tests\Unit\Validators\StrategicResponseApplicationValidatorTest::testSkillsComplete

Failed asserting that true is false.

/var/www/tests/Unit/Validators/StrategicResponseApplicationValidatorTest.php:174

-------------------------------------------------------------------------------------

1) Tests\Unit\Validators\ApplicationValidatorTest::testSkillsComplete

Failed asserting that true is false.

/var/www/tests/Unit/Validators/ApplicationValidatorTest.php:120

either both tests are bad, the skill validator is bad, or we have some factory generation that falls outside the expected data structure and throws a failure randomly.
Reproducing locally has been problematic (I ran these tests for over an hour one day without a failure).

I believe the tests fail when they delete a skill criteria and expect application validation to fail, and it does not. We now only require hard skill, so ensure it doesn't delete a soft skill by mistake.


About sdk version

the project now update 1.2.0 is 33, could the sdk verison less than 33.

I suggest to migrate your app to sdk 33 or even 34 because tge minimum sdk to publish on google play store is 33


[Bug] ... Does not show up while the command is already running

Platform

Device: Phone,
OS: Android,
Browser:  Kiwi...
Browser Version:116.0.5845.72
Better xCloud Version:1.13.1

Describe the bug
...Added but didn't run

Screenshots/Videos
If applicable, add screenshots/videos to help explain your problem.

It should work after refreshing several times

Nó sẽ hoạt động sau khi làm mới nhiều lần

When I fake vpn, it shows up.. I don't understand


Support for updating Poetry pyproject.toml, not just poetry.lock?

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
poetry
Package manager version
No response
Language version
No response
Manifest location and content before the Dependabot update
/pyproject.toml, /poetry.lock
dependabot.yml content
Default configuration (no dependabot.yml)
Updated dependency
No response
What you expected to see, versus what you actually saw
Given that the documentation specifies that the default versioning-strategy for the Python ecosystem is auto, I'd expect Dependabot PRs to by default update pyproject.toml as per either theincrease or widen strategies.
Instead, only the poetry.lock is updated by Dependabot. This is of very limited usefulness since poetry.lock only influences what gets installed when working directly in the repository, and has no impact on the dependencies that are specified when packaging and distributing our code via PyPI. As a result, we almost always have to manually "fix up" Dependabot's pull requests (usually by pulling down the branch and manually running poetry add <dependency>~<new-version>) to include appropriate pyproject.toml updates in order to actually affect the packaging of our project.
I'm guessing that this may be because Dependabot states support for PEP 621 compliant pyproject.toml files, and Poetry's pyproject.toml is not currently PEP 621 compliant. But as an end user of Dependabot, it would be useful to have this variant file format supported as well.
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

The docs don't even mention the capability to make updates to poetry.lock, making this even more confusing

This will be solved if https://github.com/python-poetry/poetry-core/pull/708 is merged, which makes poetry PEP 621 Compliant

cc @carlincherry @cmrice

This will be solved if python-poetry/poetry-core#708 is merged, which makes poetry PEP 621 Compliant

This dependency has been merged in - is this ticket completed or just something that can be actively worked on now*?

While python-poetry/poetry-core#708 has indeed been merged, it hasn't been released to public yet (will come as a new 2.*.* poetry release), which is presumably waiting for python-poetry/poetry#9136 to be completed. Until that time, there's no reliable way for poetry end-users to use these PEP-621 compliant pyproject.toml dependencies, and hence this ticket cannot be called completed until then. At the very least, the dependabot docs should be updated to reflect that dependabot only supports poetry 2.* and only when using PEP-621 compliant pyproject.toml.


Support usage in no_std contexts

Sumary
In short, this crate already is no_std compatible, it just doesn't declare no_std in the root lib.rs.  Adding that no_std declaration makes it usable in no_std projects.
Motivation
This crate is used widely in the wasmtime project, among others, which currently doesn't support no_std. However, I along with others are working to port it to no_std environments, and this crate is part of the puzzle.

Hm, I guess bumping to 0.2.2 would bump the msrv (to whatever no_std) is, although at this point it's still old enough not to matter...

Nevermind, published as v0.2.2.

Thanks! I agree, I don't think that no_std would affect the MSRV either, so you should be all good there.


Update the documentation for the new Models page

Update the documentation for the new Models page

@fakela we can discuss more about it

Okay i see it here 👍

@fharper  Here  is the suggested edit: https://dash.readme.com/project/mindee/v1.0/suggested/update/619e304ca5963f000fd4edca


Python: How to set the api base/url of the openai model using OpenAITextCompletion by python

How to set the api base/url of the openai model using OpenAITextCompletion by python?

Hi @Rainismer, if you'd like to provide your own base url for the OpenAITextCompletion class, you may pass in a custom AsyncOpenAI client. Let us know if this works for your use-case. Thanks.

Closing this issue due to no response. Please re-open if you still need help. Thanks.


[TIR][REFACTOR] Update TIR nodes std::string->String.

This PR updates the remaining TIR node's member to use
String instead of std::string.

cc @junrushao1994 @yzhliu @zhiics @wweic


Classes/imports/Typescript/Snowpack/other cleanup

This PR is significantly bigger and messier than I should have made it, particularly since I just did all the changes at once instead of incrementally. I may have been a bit overzealous on style changes as I tried to make some things more idiomatic - in retrospect went a bit too far. If you want any thing changed back (or have any want for changes in general) please let me know and I'd be more than happy to do so!

♻


Mixture of short SampleSets fails

https://preview.squiggle-language.com/playground#code=eNqrVirOyC8PLs3NTSyqVLIqKSpN1QELuaZkluQXwURSUtMSS3NKnPNTUpWslHIrNGLyFIAgODG3ICc1OLVEL60oP9cns7hEA0TolRaE5GsY6hgaaGrqEK0yJk9TqRYAOTIwQg%3D%3D
SampleSetDist.mixture tries to produce a sample of sampleCount length, and assumes that all samplesets are of this length.

It's also not obvious what's the right thing to do here: should we take sampleCount samples, or less?
Taking sampleCount samples would cause discretization; d = SampleSet.fromList(List.upTo(1,10)) is not discrete, but mx(d, d) would be.
Discretization would be unavoidable for mx(SampleSet.fromList(List.upTo(1,10)), normal(5,2)->SampleSet.fromDist), though.
My best guess is to take min(sum(lengths), sampleCount) samples.

The current implementation also keeps track of correlation. So I'm not sure what the mixture of two samplesets of unequal length is supposed to do. Fail with the better error?..


Pydantic V2 support

Pydantic will soon release their V2 rewrite.
This library isn't compatible with it.
This might break some users, depending on how they installed this library.
Then somebody has installed the library with pip install dataclass-mapper[pydantic], it will already force the user to use a version < 2.0.0. However if somebody has installed it with pip install dataclass-mapper pydantic, it will (in the future) install Pydantic V2 and then crash. We should prevent that with some version in the library.
And also we should support the new version.

[x] Add a version check and add warnings
[x] Support new Pydantic library

Implemented with 1.8.0


更新一下微信群的二维码

What problem does this feature solve?
.
What does the proposed API look like?
.

我们更新一下

二维码已更新

更新下二维码@ErKeLost

二维码是最新的呢


GitHub App PoC

A base GitHub App for the redhat-actions must:

Run containerized in OpenShift
Not require administrator permissions or a non-restricted SCC

The app will:

Provide a frontend accessible in-cluster which the user can launch to authenticate with GitHub (https://github.com/redhat-actions/common/issues/36)
Allow the user to select repositories they wish the app to have access to (through github oauth flow)

For authorized repositories, the app:
Must:

Create secrets into the repository which contain the OpenShift server URL and credentials (https://github.com/redhat-actions/common/issues/30)

While a temporary token could be used at first, we will create a serviceaccount and use its token as documented in the wiki

Create a workflow with oc-login step with the secrets matching those created by the app

Should:

Create workflow from starter workflow

Prewritten workflows to build different project types

Inject oc-login step into an existing workflow job
Set up OpenShift integrated image registry

Push (using push-to-registry)
Pull (requires podman login)

Stretch:

Install self-hosted runners (could be a separate app entirely) (https://github.com/redhat-actions/common/issues/29)

working up to starter workflow. See https://github.com/redhat-actions/openshift-actions-connector/issues for next steps


[FEAT]: Create and share Azure account ID and public SSH key

Description
In case you should be granted specific privileges on Azure cloud infrastructure and/or other services please comment adding

The email which you used to register to Azure
A public SSH key (of which only you have the corresponding private key)

Additional Information
No response

I don't have an email registered there yet. Is it mandatory to access the cloud VM ?
My public key is:
ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABgQDd106LHawJhe2TB4h9Mh9HrShamH6UqdsuniIm+7LXjYJjB1Sq5JVaDueEMdNOAeiEOzHcnPyqCfz0QiLoGT97cSI8I+Q+WspCtS/v8via8GkuyAMs8KJVEeSfH9iCEeLjbHGjC0UMVN8GaMBbgBYNSDOS1oF3OJAbX0ebny7nf5NDebEdX2VJcmuIJsUeC88pdp6g09MnnXodRVQkJ3lQtqJHIVz/j+U578qgZAJn14wk5Ojv01+Wpos+uJ2OskUp81N/xbiwZPo2byyl3RjToPOVgR0c6WsVKUNnAGEn1l4PYNOfvAj73ndowJHsoBCPA771n++6Q5rwj2S+696qGyrPd3U+8qfEVM09/UN1FZO89ZGiJ1XYK7Ig+60q9aQsbEGJlhut+tG8Dypf50q1XFfQFQNHXcNgJHeGOjcnetZFnt+GlzyCdjKSeC6+wjMoZEx4RpP5qdaibELcebO8J8P2ZFM+OuSFay54PZhLsDRQ/LeCu4EMZigzFZwBl+M= Mio@DESKTOP-7MJ41MV

The reason why I didn't want to register an account yet (or if it was avoidable) is:

To not lose the 200 USD offered in 30 days.

Assigning to @alv67 asking also to him

The email which you used to register to Azure
A public SSH key (of which only you have the corresponding private key)

Assigning to @gmacario with requested information:

registered Azure with my github<EMAIL_ADDRESS>my public key is:

ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABgQCvsYfICuHJEwSXw61e1fq4uIWDrJI3QeaktLtscULJn2u3LFyd4IuDGVy4QtLhwUZUuKFvxB2AhCeqY7Etj5nUnhaHguWsQnFgQFiP7kZFOfaIP/Q2qCAHdaUkaEmLeV2aC/UZKaMLD3Sbunt+X95+//JPawbZe8DJDYhbN/Rltq4eM2h523ZnAuAugavKB8KKCTpNOB50fpMKeXmqqQuiIgohhFGaGBehum9t5S/hA42tdoE0UELug0fDHf3/1Rb6zxZEC8VNK6UuhweyQa81lV7LIhQM0R5OHkbfrG2CBOlTBoTXsuqjvM86EamknGtSP+v+9hEFTEa2ZEgb049U4G7wC9UEIcys7KDmYr2MREtcXkQdaaOE10U8VsHoT9CmIstTdPM5YOenG+HPG2hOvkm0M7gOeGnJ63pbfgsOhCbBwXKsqhqaNBuNxieeb3a5MuY/yl4zbZJ1VYuElAlu/W/hGbOqjde54EIx6EO213BgCGzAwYmob3EoUlZpajc=<EMAIL_ADDRESS>

Thanks @alv67 and @gteti
We are now set, if you need any Azure resources just create a new issue on https://github.com/B-AROL-O/ARNEIS/issues and assign it to me.
Closing


NixOS has issues with performace on btrfs filesystem.

Describe the bug
I had a year old NixOS installation on btrfs + encryption alongwith a swap partition with zswap enabled. I was and still am using the nix flakes with my configuration. I didn't notice this right away but I had a lot performance issues when the OS booted up because I had enabled auto garbage collection and store optimizations in my configuration. After disabling those performance was reasonable but always came up whenever I used the nix command. I also checked my fstab there were no options like noatime,space_cache present which are supposed to increase the speed, so I did add those to my configuration but it didn't help a lot. I have reinstalled my system with xfs + encryption and these issues don't seem to be present anymore. So my guess would be these issues came up because I had not disabled copy-on-write for btrfs on nix store. So if possible can someone test this out? I might have provided more info by using a virtual machine but I have a data cap and online classes sorry.

I have been using btrfs for the nix store for quite some time without any issues. I do garbage collection manually from time to time though.
There used to be a problem with compression being ignored (https://github.com/NixOS/nix/issues/3550), which should be fixed now. copy-on-write is certainly not an issue (unless there is a bug), since data from the nix store is never modified at all.
Maybe btrfs has issues with the heavy read + delete pattern that occurs when gc happens while booting?
Also while data in the store proper (/nix/store) is never modified, /nix/var/ contains logs and a database, so it might be worth setting nocow on that directory.
Also nix with flakes writes to a cache in the home directory I think, so if that is on btrfs as well, it might also potentially affect performance.

ah other things of interest would be the hardware (ssd or rotating) and the kernel version used (there is still a lot of change in btrfs land even these days).

My entire drive was formatted with btrfs, so yes the home directory was also btrfs.
My drive is rotating.
Kernel Version is :-
Linux X542URR 5.11.11 #1-NixOS SMP Tue Mar 30 12:30:31 UTC 2021 x86_64 GNU/Linux
There was an issue which made btrfs slower in 5.10 I think but this issue was experienced even in the lts(5.4 at the time) and 5.11.

I use btrfs. It works fine but some files do get heavily fragmented overtime, such as sqlite databases and the systemd journal. I have a systemd timer which runs a btrfs defrag on such files. The job runs about 10 minutes after booting and it keeps fragmentation under control. With an SSD fragmentation isn't as big of an issue, in my experience. But, it does consume metadata, thus I still use the aforementioned defrag job.

@emmanuelrosa are these the files you are referencing?
me on X542URR in ~/.cache/nix
at 🕙17:45:50 ❯ ls
binary-cache-v6.sqlite
binary-cache-v6.sqlite-journal
eval-cache-v2
fetcher-cache-v1.sqlite
fetcher-cache-v1.sqlite-journal
flake-registry.json -> /nix/store/qi48i5dkbffrdxc15rhdz1x4mpm6q6n3-flake-registry.json

If that is the case I might try it in a VM.

Does Nix check the integrity of its DBs and caches? I'm not sure how comfortable I'd be disabling checksumming/cow on those if not.

I was referring to sqlite databases in general. But more broadly, I defrag files I know get fragmented due to many random writes.
In some cases that's not enough. For example, for VMs I have a dedicated LVM volume group. For my bitcoin lightning node (clightning), it put its data files on ext4.

@emmanuelrosa then can I close the issue? It can still be used for discussion about the matter.

Yes, I think so.


Compile should move measurements to end of circuit when backend is a simulator

Informations

Qiskit (Python SDK) version:  0.5.5

What is the current behavior?
When compiling a circuit measurements are by default moved as close to the front of the circuit as possible. This is good for running on physical hardware but not optimal for simulations which can sample measurement outcomes for multiple shots if all the measurements are at the end of a circuit.
Since the current simulators can sample measurement outcomes from the final state if all measurements are at the end of the circuit this default behaviour prevents this unless barriers are explicitly inserted into the circuit before measurement.
Steps to reproduce the problem
Consider the following 15 qubit Bernstein-Vazirani circuit:
q = QuantumRegister(16, name='q')
c = ClassicalRegister(15, name='c')
qc = QuantumCircuit(q,c)
qc.x(q[15])
for k in range(16):
    qc.h(q[k])
qc.x(q[15])
qc.cx(q[0], q[15])
for k in range(2, 8):
    qc.cx(q[k], q[15])
for k in range(9, 11):
    qc.cx(q[k], q[15])
qc.cx(q[12], q[15])
for k in range(15):
    qc.h(q[k])
# qc.barrier(q)  # compare with an without this barrier
for k in range(15):
    qc.measure(q[k], c[k])

If we execute without the barrier before measurements and time for the default number of shots we have:
%%timeit
result = execute(qc, "local_qasm_simulator",shots=1024).result()
11.7 s ± 634 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

This is quite slow, and by looking at the compiled qobj you can see the measurements have been pushed forward to follow immediately after each H gate on the same qubit. If we add the barrier before the final measurement we get:
%%timeit
result = execute(qc, "local_qasm_simulator",shots=1024).result()
43 ms ± 6.58 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

What is the expected behavior?
If a circuit is designed where all measurements are at the end of the circuit, they should remain there and the measurement sampling optimization should be enabled on simulations.
Suggested solutions
When the compiled backend is local_qasm_simulator or the online simulator move all measurements as far backwards in the circuit as possible, rather than forwards. This will allow the optimization to be used by default in many use cases.

Should this be the responsibility of the simulator or the compiler to figure out that measurements can be pushed to the end?
Since we are talking about an optimization implemented by the simulator, it seems to me that the simulator should try to discover whether the optimization is applicable or not.
Another reason is that, conceptually, the compile() should not know what "kind" of backend it is targeting; but rather what coupling map, native gate set, etc. it is targeting. Because those are the things that correspond to the compilation passes, not a monolithic thing such as device vs. simulator. But this could be debated.

@ajavadia the compiler is already deciding to move them forwards, which is an optimisation specific for hardware, it could also presumably move them backwards using the same transpiler pass (or not move them at all).  To me seems better to do this in the one location then leave it up to every simulator to include its own mini-compiler.

I would say that hardware-specific passes in the compiler make sense (@atilag what do you think?). That said, for this specific case, what about doing nothing by default and optimize for (our) physical devices instead of optimizing for simulators?

We have a solution to this issue (see the last comment in #691), but it is most likely best to wait for aer to implement as it involves modifying the way terra talks to thelocal_qasm_simulator, and it is not really a bug per say.
As a side note, the transpiler must have hardware specific passes otherwise we can not map circuits to hardware with limited connectivity.

Thank you, @nonhermitian! @ajavadia if you agree we should move this to Aer, we can mark the bug as "wont fix" and close it.

I agree this should be closed.

Closing.


Always Finalize the PKCS11 FindObject Operation

There are 6 pkcs11 operations that require the session handle to be finalized before a session can be reused. These operations are: Encrypt, Decrypt, Sign, Verify, Find, and Digest. The bccsp/pkcs11 package makes use of three of these operations: sign, verify, and find. Each of these operations has an init function, i.e., SignInit, VerifyInit and FindObjectInit, each are part of the cryptoki implementation. If the Init functions fail, the session handle is never initialized and the finalize function do not need to be called.
For SignInit and VerifyInit, the next operation we call are the Sign or Verify functions, which also atomically finalized the session. For the FindObjectInit operation however, we must explicitly call the FindObjectFinal function to release the lock on the session handle.
The current implementation makes a call to FindObject in between FindObjectInit and FindObjectFinal which has an error path. In the current implementation FindObjectFinal is not called on the error path, leaving the session in a state that it can't be used again.
This change ensures FindObjectFinal is always called, even on the error path.
Signed-off-by: Brett Logan<EMAIL_ADDRESS>

@Mergifyio backport release-2.2

It seem like a cryptography problem. Is there any obvious phenomenon if the session is not reusable?

In general we never expected to hit this bug. The log will tell you however if you hit this bug, you'll get an error from PKCS11 that tells you the session was not valid, though I can't remember off the top of my head what that error is.

In general we never expected to hit this bug. The log will tell you however if you hit this bug, you'll get an error from PKCS11 that tells you the session was not valid, though I can't remember off the top of my head what that error is.

thanks for your answer, Does it mean that this bug will be hit when we execute 6 pkcs11 operations like Encrypt, Decrypt, Sign, Verify, Find, and Digest again？


🧟♂ [Anomaly]: broken Google images results page

Issue: there is a layout shift on the Google's search results for images.
URL: https://google.com
Occurrence: only on mobile.

Fixed with https://github.com/igorskyflyer/ad-void/commit/f827e4264a35334cc8d9048b86d874c1d34f13ce.


UI changes to enable test environment which don't require importing cluster

Raising an issue to track the UI changes for #1232

@joylyn
Please follow up the latest change of #1232. The field indicating test environment is now internal.

Closed via #1239


Data Entry - entries from previously entered record appear

There are some "remnants" of the previously submitted record on the new entry screen.  I think it's just time, weather 48, weather now and NAV.  Gave some thought as to whether this was  a bad thing especially for fields like weather where the result is likely to repeat.  Concluded that it might lead to entry errors when a default is accidentally left.  The only place it might work is date.  For everything else it's better to start with a clean slate for each new record.

Agreed - for radioButtons (single choice) I have to reset state to a value - it does not work to reset with nothing selected. For now I have reset to "Not Recorded", however this could be problematic if in reality an entry was omitted by mistake. I can add another default choice of "Nothing Selected" so that it always returns to this value. Sound ok? I cannot figure out a way to get the time to reset to 00:00... I'll keep working on that.


fix!: esm typescript support

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

Comparison is base (816e76c) 100.00% compared to head (e32ac47) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #29   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           75        75
  Branches        13        13
=========================================
  Hits            75        75

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


What is this json_ref?

Can someone please help me understand json_ref? C++ std::initializer_list is generally immutable but this looks like something that lets a JSON value take ownership of elements in an initializer list if they are constructed as rvalues? Is there any documentation on this? Or examples somewhere?

It was introduced here: https://github.com/nlohmann/json/pull/663


[Bug]: ES module - Mock a function of a module

Version
29.7.0
Steps to reproduce
Hello,
i pursue an issue report 2 years ago (#12145 ), closed but no resolved.
It's a project of typescript, but i'm going to illustrate my problem with js.
Create the file "toto.js".
export function main() {
    const res = a();

    return res;
}

export function a() {
    return 10;
}

Create test file "toto.test.js" :
import { jest } from "@jest/globals";

jest.useFakeTimers();

jest.unstable_mockModule("./toto", () => {
    const totoModule = import("./toto");

    return {
        ...totoModule,
        a: jest.fn(() => -2),
    };
});

describe("Suite1", () => {
    test("works", async () => {
        const { main } = await import("./toto");
        expect(main()).toBe(-2);
    });
});

Expected behavior
No error and test "work" pass with result = -2 (expect(main()).toBe(-2);)
Actual behavior
    TypeError: main is not a function

      51 |     test("works", async () => {
      52 |         const { main } = await import("./toto");

Additional context
Thx in advance.
Environment
$ npx envinfo --preset jest
npx: installed 1 in 2.137s

  System:
    OS: Windows 10 10.0.19045
    CPU: (4) x64 Intel(R) Core(TM) i5-6600 CPU @ 3.30GHz
  Binaries:
    Node: 14.18.1 - C:\Program Files\nodejs\node.EXE
    npm: 6.14.15 - C:\Program Files\nodejs\npm.CMD
  npmPackages:
    jest: ^29.7.0 => 29.7.0

const totoModule = import("./toto");
you need to await the import() statement.
But that's the thing you're mocking - we don't have a jest.importActual API at the moment (it's unchecked in the OP of #9430). You'll need to implement the entire mock manually within unstable_mockModule callback

Thx for your quick response !
Well noted. I looked #9430 and subscribe.
I didn't have awareness on all problems about esm modules to test.
"You'll need to implement the entire mock manually" : i understand however i couldn't do that. Many functions, it's too much work.
I continue to use jest for projects without type:module.
For others, search a other solution.
Thx again.