Keychain Prompts on CI/CD Runner

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

CI: GitLab
Issue 1: Every time our build is executed on the Mac Mini a prompt is displayed asking for keychain access to fastlane_tmp_keychain_db. I believed this is caused by setup_ci, so I am not sure why we need this.
Issue 2: This issue has already been talked about, but the solution seems ridiculous and I don't understand why we need to create a keychain in the first place https://github.com/fastlane/fastlane/issues/10589.
Command executed

bundle exec fastlane build

Complete output when running fastlane, including the stack trace and command used

Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
12.3

Ruby
3.0.2

Bundler?
true

Git
git version 2.33.1

Installation Source
~/.../app/ios/App/vendor/ruby/3.0.0/bin/fastlane

Host
macOS 12.3 (21E230)

Ruby Lib Dir
~/.rbenv/versions/3.0.2/lib

OpenSSL Version
OpenSSL 1.1.1l  24 Aug 2021

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
13.0

Swift Version
5.5

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
`./fastlane/Fastfile`
before_all do
  update_fastlane
  setup_ci
end

default_platform(:ios)

platform :ios do
  desc "Get certificates"
  lane :certificates do
    sync_code_signing(
      type: "development",
      app_identifier: ['...'],
      force_for_new_devices: true,
      readonly: true
    )

    sync_code_signing(
      type: "appstore",
      app_identifier: ['...'],
      readonly: true
    )
  end

  desc "Generate new certificates"
  lane :generate_new_certificates do
    sync_code_signing(
      type: "development",
      app_identifier: ['...'],
      force_for_new_devices: true,
      readonly: false
    )
    sync_code_signing(
      type: "appstore",
      app_identifier: ['...'],
      force_for_new_devices: true,
      readonly: false
    )
  end

  desc "Build binary"
  lane :build do
    increment_build_number(build_number: ENV['...'])
    certificates
    build_app(workspace: "App.xcworkspace", configuration: "Release", scheme: "...")
  end

  desc "Push a new beta build to TestFlight"
  lane :beta do
    api_key = app_store_connect_api_key(
      key_id: "...",
      issuer_id: "...",
      key_filepath: ENV['...'],
      duration: 1200, # optional (maximum 1200)
    )
    certificates
    upload_to_testflight(
      api_key: api_key,
      skip_waiting_for_build_processing: true
    )
  end

  desc "Release binary"
  lane :release do
    api_key = app_store_connect_api_key(
      key_id: "...",
      issuer_id: "...",
      key_filepath: ENV['...'],
      duration: 1200, # optional (maximum 1200)
      in_house: false, # optional but may be required if using match/sigh
    )
    certificates
    upload_to_app_store(
      api_key: api_key
    ) # deliver
  end
end

`./fastlane/Appfile`
app_identifier("...") # The bundle identifier of your app
apple_id("...") # Your Apple email address

team_name("...")
itc_team_name("...")
team_id("...") # Developer Portal Team ID
itc_team_id("...")  In any case, it is best to use your own script so that you can adjust the keychain parameters as you wish.
#10589 is not ridiculous 😄
Fastlane keeps private key and certificates in the keychain. If you do not define your keychain entry or use seup_ci (which does this in the background), then everything will be saved in the default keychain.
In case someone gets access to your mac mini or knows the login password, they can enter the default keychain and export private keys and certificates.

When the mac mini goes into sleep mode, the keychain locks, maybe that is the reason. In any case, it is best to use your own script so that you can adjust the keychain parameters as you wish.
#10589 is not ridiculous 😄
Fastlane keeps private key and certificates in the keychain. If you do not define your keychain entry or use setup_ci (which does this in the background), then everything will be saved in the default keychain.
In case someone gets access to your mac mini or knows the login password, they can enter the default keychain and export private keys and certificates.

So, are you recommending to use setup_ci and then store the keychain password (I think there is a keychain env variable for Fastlane) in Gitlab variables?

I would rather use the ensure_temp_keychain from jgavris. But timeout parameter is not Bool anymore, so you can set it to 0.
You can define the password as a gitlab secret, but you do not have to. You can use a random password as well (e.g. SecureRandom).
Both approaches have pros and cons 😄

With the solution from jgavris, how does the keychain get unlocked with the password (or random password) without having to access the machine?

Since you mentioned gitlab, I guess you use gitlab ci. So you have installed a gitlab runner directly on the machine (that is why the runner has access to the machine). With this script you create a new keychain that is locked with a key defined in the function. Did this help you?

I have not tried it quite yet, just trying to understand how the keychain will get unlocked.


Improve Login Form

I would like to improve the current login from . Can  you assign me  ...

Hey @Mahich123 , sure I'd love to see your contributions. But before that, could you please tell me what changes you're planning to implement? :+1:

@arpanaditya  I want to align those text's and design that container

Made pr you can check the updates


bug: Attempt to index field 'foldupdate_range' (a nil value)

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.10.0
Operating system/version
MacOS 14.5
Describe the bug
When lsp adding missing imports in TS files I occasionally get these

Trying to understand if this is a lazyvim error or neovim error
Steps To Reproduce
Open TS file try removing and re-adding imports via lsp organize imports
Expected Behavior
No error
Repro
No response

Looks like a Neovim bug

the error message says that the error happened when processing bufwritepre, which means that nvim was saving the file (and trigger auto formating). were you actually saving the file when this happened? can you reproduce this by doing some edits in insert mode and concurrently trigger auto formatting without leaving insert mode?

@tomtomjhj
yes I do use mostly ctrl-c instead of esc as much as possible. Do you think it can affect this?
This happens randomly. For example this happens when I try to exit fzf-lua window when I am checking for lsp references and kill it with ctrl-c. But I noticed it mostly happens with list of new references that I haven't visited yet.
Or this happens when I try to complete the auto imports from normal mode with tsserver or typescript-tools add imports functionality with lsp code actions. And again happens with new imports mostly.
@tomtomjhj can you tell me how to trigger autoformatting without leaving insert mode?

yes I do use mostly ctrl-c instead of esc as much as possible. Do you think it can affect this?

yes this seems to be the root cause. leaving insert mode eith ctrl-c doesn't trigger insertleave autocmd, and this confuses treesitter fold stuff. i missed this point when i made changes to it.
the fix is simple: just add a nil check. i was initially hesitant to do this because it didnt seem necessary, but it turns out it's indeed necessary. i'll make pr soon.

Can confirm this is happening in 0.11 as well. This occurs in Typescript, Lua, Java files.
Almost 70% time this occurs when I organize imports in Typescript files or use vim.lsp.buf.code_action() to import something. And then sometimes when I use treesitter.

@s1n7ax do you use ctrl-c to exit insert mode?

this was definitely a ctrl-c issue in my case

@tomtomjhj Yes but how is that related to error occurring when vim.lsp.buf.code_action() is used? Anyway, just added a remap for <c-c> to <esc>

Here's the scenario that I guess you were running into.

In insert mode, do some edits that update the syntax tree.
Leave insert mode with ctrl-c.
Make edits in normal mode that update the syntax tree. This happens to be the change made by code_action.
Enter insert mode, and leave insert mode with ESC, without making changes.
nil error

I have fixed this and explained why this happens in https://github.com/neovim/neovim/pull/29581

@tomtomjhj Ah. Cool. So far no errors with nmap <c-c> <esc>.

nmap <c-c> <esc> don't fix the issue. Hopefully PR will be merged soon

inoremap <c-c> <esc>

@tomtomjhj of course dafuq was I thinking XD

had the same issue with custom InsertLeave autocommand.
fixed with validating the existing changes:
  pattern = { "*" },
  callback = function()
    if vim.bo.modified then
      vim.cmd("silent! write")
    end
  end,
})```


Add custom error handling for react-native projects with no bundle identifier set

This PR also moves out 2 method calls that shouldn't be part of the begin rescue block

It would be good to have tests to cover this new functionality, and ensure it stays working :+1:

@mfurtak added tests for the 2 cases I added in this PR 👍

@mfurtak Addressed the feedback 👍

Also resolved merge confclits


OpenML API

OpenML.org API integration, the OpenML.load() function will download a dataset from the OpenML platform given its ID number, and return it as a Vector of NamedTuples.
For example, the iris dataset has the ID 61 in OpenML. The named tuples can be converted to tabular format such as a DataFrame.
iris_named_tuples = OpenML.load(61)
df = DataFrame(iris_named_tuples)

Codecov Report

Merging #182 into dev will decrease coverage by 4.97%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##              dev     #182      +/-   ##
==========================================
- Coverage   84.78%   79.81%   -4.98%
==========================================
  Files          30       31       +1
  Lines        1854     1863       +9
==========================================
- Hits         1572     1487      -85
- Misses        282      376      +94

Impacted Files
Coverage Δ

src/MLJBase.jl
100% <ø> (ø)
:arrow_up:

src/openml.jl
0% <0%> (ø)

src/parameter_inspection.jl
50% <0%> (-50%)
:arrow_down:

src/composition/arrows.jl
25% <0%> (ø)
:arrow_up:

src/hyperparam/one_dimensional_ranges.jl
82.5% <0%> (-12.63%)
:arrow_down:

src/measures/measures.jl
89.47% <0%> (-3.51%)
:arrow_down:

src/composition/networks.jl
67.94% <0%> (-1%)
:arrow_down:

src/resampling.jl
87.3% <0%> (-0.02%)
:arrow_down:

src/equality.jl
90% <0%> (-0.91%)
:arrow_down:

src/utilities.jl
74.32% <0%> (-0.68%)
:arrow_down:

... and 13 more

Continue to review full report at Codecov.

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

@tlienart Thanks for reviewing!

@darenasc Looks like there's just the one suggestion of @tlienart about testing the network is online. Did you want to do something about this?

Thanks @ablaom, I'll leave that one for an update @tlienart, we don't want to generate a DoS of the discourse page from MLJ :)
There are error messages are printed for all the statuses and that should work for now.


2.0.0 wallet struggles to connect to node on linux

User runs ubuntu 18.04, deleted their wallet713.toml and regenerated it as below, but still fails to connect to the hosted node.

My network working fine, several cryptos are synking.
This happens with grin after an update to 2.0.0.

It's officially become very strange. I use an phisical machine (against virtual), make clean install of Ubuntu 18.04, install rustc and dependies (include openssl). And nothing!
I thought that my local network make me trouble and connects to a 4G and nothing!
Than i connects to another 4g and also nothing!
And in all of the attempts pinging is successfully. Can it be banned in my country?

Have the very same issue. Running wallet713 on Mac, and grin node on Ubuntu. It worked fine until at least end of 2019.

My bad, I temporarily closed ports on my server so the wallet couldn't connect to the node.


[SPARK-41865][INFRA][3.2] Use pycodestyle to 2.7.0 to fix pycodestyle errors

What changes were proposed in this pull request?
branch-3.2 has been broken for a while with false-positive failures.

https://github.com/apache/spark/actions/runs/3833643461/jobs/6525303096

pycodestyle checks failed:
./python/pyspark/broadcast.py:102:23: E275 missing whitespace after keyword
./python/pyspark/sql/tests/test_context.py:166:19: E275 missing whitespace after keyword
./python/pyspark/sql/tests/test_pandas_udf_grouped_agg.py:512:15: E275 missing whitespace after keyword
./python/pyspark/sql/pandas/conversion.py:174:20: E275 missing whitespace after keyword
./python/pyspark/sql/readwriter.py:662:60: E275 missing whitespace after keyword
./python/pyspark/sql/readwriter.py:694:60: E275 missing whitespace after keyword
./python/pyspark/ml/tests/test_tuning.py:203:23: E275 missing whitespace after keyword
./python/pyspark/worker.py:583:27: E275 missing whitespace after keyword
Error: Process completed with exit code 1.

Why are the changes needed?
To fix pycodestyle.  Apache Spark expects 2.7.0 like the following.
https://github.com/apache/spark/blob/ad2d42709abfc8f8ad27f836c811a4b75ef32ee9/dev/lint-python#L25
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass pycodestyle.

Thank you, @HyukjinKwon . Although I checked this locally, I'll merge this after pycodestyle passes in CI.
After this PR, I'll fix mypy failures too in a separate PR.

Oh, in CIs, all linters passed. I'll update the PR description and merge this.

It seems that SparkR has new issues.
R doc generation failed (RuntimeError)


Implement "feed" command support

This implements a method to expose the feed command for use with Home Assistant (I'll open a PR there shortly as well). This would allow Home Assistant to provide the ambient temperature value to the HVAC based on HA-controlled sensors (in my case I'm using Bluetooth sensors which allows me to better position it than the built-in sensor of the HVAC thermostat).
Also, it rounds provided temperature values (to both "feed" as well as the existing "set temperature") to 1 decimal place which is the maximum precision supported by CoolMasterNet. This makes sure it doesn't break if someone accidentally provides a decimal or something with lots of digits.

Thanks @Rjevski!
New version at: https://pypi.org/project/pycoolmasternet-async/0.1.3/


'class BLEClient' has no member named 'setMTU' in version 1.0.6

Hi,
due the limitation with other library I have to stick with version 1.0.6, and when I compile my code with this version I facing the following issue: https://ayunts.com/att/mtu.png
the same code is been complied fine with the latest version as of today, but I must use version 1.0.6, can anyone guide me how to fix this issue manually without updating my esp32 core.
please find the code below:
#include "BLEDevice.h"

static boolean doConnect = false;
static boolean connected = false;
static boolean doScan = false;
static BLERemoteCharacteristic* pRemoteCharacteristic;
static BLEAdvertisedDevice* myDevice;
String Message = "Welcome";
int HRM = 1;
char buffer[10];

static void notifyCallback(
  BLERemoteCharacteristic* pBLERemoteCharacteristic,
  uint8_t* pData,
  size_t length,
  bool isNotify) {

    char hexString[3];
    snprintf(hexString, sizeof(hexString), "%02X", pData[5]);

    // Convert the hexString to decimal using strtoul
    unsigned long decimalValue = strtoul(hexString, nullptr, 16);
    if (length>5){
      Serial.println(decimalValue);
      Serial.println();
    }

}

class MyClientCallback : public BLEClientCallbacks {
  void onConnect(BLEClient* pclient) {
  }

  void onDisconnect(BLEClient* pclient) {
    connected = false;
    Serial.println("onDisconnect");
  }
};

bool connectToServer() {
    Serial.print("Forming a connection to ");
    Serial.println(myDevice->getAddress().toString().c_str());

    BLEClient*  pClient  = BLEDevice::createClient();
    Serial.println(" - Created client");

    pClient->setClientCallbacks(new MyClientCallback());

    // Connect to the remove BLE Server.
    pClient->connect(myDevice);  // if you pass BLEAdvertisedDevice instead of address, it will be recognized type of peer device address (public or private)
    Serial.println(" - Connected to server");
    pClient->setMTU(517); //set client to request maximum MTU from server (default is 23 otherwise)

    // Obtain a reference to the service we are after in the remote BLE server.
    BLERemoteService* pRemoteService = pClient->getService(serviceUUID);
    if (pRemoteService == nullptr) {
      Serial.print("Failed to find our service UUID: ");
      Serial.println(serviceUUID.toString().c_str());
      pClient->disconnect();
      return false;
    }
    Serial.println(" - Found our service");

    // Obtain a reference to the characteristic in the service of the remote BLE server.
    pRemoteCharacteristic = pRemoteService->getCharacteristic(charUUID);
    if (pRemoteCharacteristic == nullptr) {
      Serial.print("Failed to find our characteristic UUID: ");
      Serial.println(charUUID.toString().c_str());
      pClient->disconnect();
      return false;
    }
    Serial.println(" - Found our characteristic");

    // Read the value of the characteristic.
    if(pRemoteCharacteristic->canRead()) {
      std::string value = pRemoteCharacteristic->readValue();
      Serial.print("The characteristic value was: ");
      Serial.println(value.c_str());
    }

    if(pRemoteCharacteristic->canNotify())
      pRemoteCharacteristic->registerForNotify(notifyCallback);

    connected = true;
    return true;
}

class MyAdvertisedDeviceCallbacks: public BLEAdvertisedDeviceCallbacks {
  void onResult(BLEAdvertisedDevice advertisedDevice) {
    Serial.print("BLE Advertised Device found: ");
    Serial.println(advertisedDevice.toString().c_str());

    // We have found a device, let us now see if it contains the service we are looking for.
    if (advertisedDevice.haveServiceUUID() && advertisedDevice.isAdvertisingService(serviceUUID)) {

      BLEDevice::getScan()->stop();
      myDevice = new BLEAdvertisedDevice(advertisedDevice);
      doConnect = true;
      doScan = true;

    } // Found our server
  } // onResult
}; // MyAdvertisedDeviceCallbacks

void setup() {
  Serial.begin(115200);
  Serial.println("Starting Arduino BLE Client application...");
  BLEDevice::init("");

  BLEScan* pBLEScan = BLEDevice::getScan();
  pBLEScan->setAdvertisedDeviceCallbacks(new MyAdvertisedDeviceCallbacks());
  pBLEScan->setInterval(1349);
  pBLEScan->setWindow(449);
  pBLEScan->setActiveScan(true);
  pBLEScan->start(5, false);
}

void loop() {

  if (doConnect == true) {
    if (connectToServer()) {
      Serial.println("We are now connected to the BLE Server.");
    } else {
      Serial.println("We have failed to connect to the server; there is nothin more we will do.");
    }
    doConnect = false;
  }

  if (connected) {
    String newValue = "Time since boot: " + String(millis()/1000);

    pRemoteCharacteristic->writeValue(newValue.c_str(), newValue.length());
  }else if(doScan){
    BLEDevice::getScan()->start(0);  // this is just example to start scan after disconnect, most likely there is better way to do it in arduino
  }

  delay(1000); // Delay a second between loops.
} ```

Thanks

an update: I commented // pClient->setMTU(517);
an it is solved, can anyone explain how this might effect the communication between the two terminals?
thanks again

an update: I commented // pClient->setMTU(517);
and it is solved, can anyone explain how this might effect the communication between the two terminals? thanks again

As long as the data packet being send/receive is less than 20 bytes you are good to go. But if it is greater than 20 bytes than you might see incomplete data on the server.


Wont Build for me ?

Thanks for providing this for free.
I followed the instructions here, using latest vs 2022

Clone recursively: git clone --recursive https://github.com/TheCherno/RayTracing
Run scripts/Setup.bat
Open RayTracing.sln and hit F5

I cant get the sln to build, get various files not found, not sure if I have missed something or there is a path missing in the sln setup ?
A selection of the errors I get
1>C:\RayTracing\Walnut\vendor\imgui\imgui.h(60,10): fatal error C1083: Cannot open include file: 'float.h': No such file or directory
2>C:\RayTracing\Walnut\vendor\glfw\include\GLFW\glfw3.h(103,10): fatal error C1083: Cannot open include file: 'stddef.h': No such file or directory
4>C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.31.31103\include\yvals.h(12,10): fatal error C1083: Cannot open include file: 'crtdbg.h': No such file or directory
4>Done building project "RayTracing.vcxproj" -- FAILED.
Any thoughts on how to fix this would be appreciated, thanks.

looks like you didnt installed windows10 sdk. Install it from the Visual Studio Installer.

Thanks  for the feedback, I had already got Win 10 sdk installed. However after your suggestion I did some more tinkering and managed to fix the problem. If anyone else hits this problem here is what I did
Using the visual studio installer, click on the individual components tab, then right at the bottom you see the Win10 sdk versions available, the middle numbered version was the one ticked originally, I cleared this and then ticked the higher number version and then clicked modify.
Then back in visual studio, right click on the ray tracing project in the solution explorer and click retarget solution, and then select Windows 10 sdk "latest installed version".
The project then rebuilt without errors.

hiya
Thanks for the reply, I had got the sdk installed but your suggestion was enough for me to fix the problem.
I have written some notes on what i did to fix it in case anyone else hits the problem. I have closed the issue
Thanks again
Geoff

From: Soilcsi @.>
Sent: 04 May 2022 20:57
To: TheCherno/RayTracing @.>
Cc: geoffw123 @.>; Author @.>
Subject: Re: [TheCherno/RayTracing] Wont Build for me ? (Issue #2)
looks like you didnt installed windows10 sdk. Install it from the Visual Studio Installer.
—
Reply to this email directly, view it on GitHubhttps://github.com/TheCherno/RayTracing/issues/2#issuecomment-1117819979, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHWPY65R3D5G732MVZO2YCLVILJCXANCNFSM5VB77NCA.
You are receiving this because you authored the thread.Message ID: @.***>


The injected element should be configurable

At the moment it is a requirement of Indigo that an element with an id of indigo-container is present in the HTML. This limits users a little bit when creating their own HTML templates.
Indigo should allow the user to set the ID of the element that should be updated with Indigo, with the default value being the existing indigo-container string

So you can now do the following (in JS or Scala, but there are also new Scala versions of the launch function):
IndigoGame.launch("my-container", {"width": window.innerWidth.toString(), "height": window.innerHeight.toString()})

For a page with this in it:

Will be in the next release.


feat: add Redis RBAC support

PR summary

Adds Redis RBAC support
Changes user PATCH endpoint from ChangeUserPassword to UpdateUser

Integration tests
$ make test-int
go test `go list ./...` -tags=integration
 ok  	github.com/IBM/cloud-databases-go-sdk/clouddatabasesv5	150.672s
ok  	github.com/IBM/cloud-databases-go-sdk/common	(cached)

PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

 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.Alex Hemard 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.

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


How to change tempo throughout track

Is it possible to speed up halfway through the song with setTempo?
I can't tell if setTempo sets the tempo for the entire Midi file or if it's possible to do multiple setTempo at different ticks.

Hi @jeremygottfried,
I've just released v2.1.2 which exports the TempoEvent class (along with others) so you should now be able to add a TempoEvent to your track wherever you'd like the tempo to change.
track.addEvent(new MidiWriter.TempoEvent({bpm: 200}));

I think it will depend on the software you're using to play the MIDI file whether or not tempo events mid-track are respected.  In my testing with GarageBand, it seems to only read the latest tempo event.


Multiple On Clicks run when I click on only one.

Hi @gamcoh,
Thanks for making this package I like it a lot.
I ran into an issue with using the on_click feature of the card.
from streamlit_card import card

import base64

with open('transparent.png', "rb") as f:
    data = f.read()
    encoded = base64.b64encode(data)
data = "data:image/png;base64," + encoded.decode("utf-8")

from streamlit_card import card

hasClicked = card(
  title="Button 1",
  text="Some description",
  image=data,
  on_click=lambda: print('hi'),
  key='first'
)

hasClicked2 = card(
  title="Button 2",
  text="Some description",
  image=data,
  on_click=lambda: print('hi again'),
  key='second'
)

Here is a quick example that reproduces the issue I am having.
When you click on the card that is titled "Button 1" it will print "hi".
Afterwards if you then click on the card titled "Button 2" instead of expected output to be "hi again" it will instead print "hi" then "hi again".
I believe it has something to do with the __init__.py file checking
if clicked:
    on_click()

Which after clicking on a card it is set to true, but also stays true when pressing another card resulting in both on_clicks to execute.
Thanks again for making this super useful just struggling with figuring out how exactly the on_click portion works.

If you don't want this, you could use some python / streamlit tricks to get that out of the way :

Caching with @st.cache_data - only runs your function the first time streamlit is open.
st.session_state - save your state manually.
Use some python condition to see if the function has already been executed.

Thanks for the quick response.
Is there a way to have the card behave like a checkbox? So it is more of a toggle when clicking it then unclicking it essentially?

Hey @zimmer44 ,
had same issue, solved it with st-click-detector from https://github.com/vivien000/st-click-detector and jinja2 html template.


C/C++ project about icloud backup

hi,if there is any C/C++ project about icloud backup.

this project also can not support now , i try to use it , it does not work in login!!


[Mentions v2] Create command ResolveActionableRoomMentionWhisper in Auth

We need to create a new ResolveActionableRoomMentionWhisper comand so we can resolve the new ACTIONABLEROOMMENTIONWHISPER, this command can solve the whisper in two ways:

If the answer is positive we will create the notFound rooms related to this mention and invite everyone mentioned in the original message to the created rooms
If the answer is negative we will mark it as resolved

Details can be found here.

Last PR is merged, gonna be deployed on Monday I think

Deployed https://github.com/Expensify/Auth/pull/10734#issuecomment-2109003842


filter page 1

_ No description provided. _

I guess you forgot to add the dependencies for some widgets here in PR. Its showing error of no such files are there


KAFKA-4894. Fix findbugs "default character set in use" warnings

retest this please

@cmccabe : Thanks for the patch. LGTM


Housekeeping tasks default to current module

Hi,
The housekeeping tasks in search/django/tasks.py rely on a WORKER_MODULE_NAME setting, to use as the module target when running deferred tasks. It would be nice if they just took the current module's name as the target when this setting is missing, save one having to configure the setting or setup modules.
This change wraps the logic for getting the task's target in a helper function, default to the current module.
And then I made changes to the logging so it is possible to configure logging separately from the root logger, and to not use string formatting (lazy formatting means the logger can potentially skip doing the formatting altogether if the log record is never emitted).
David B.

Good call! Thanks David.


series: reorder_levels function implementation

I think dataframes should have the same implementation, so I'll dedupe this one.

Duplicate of #1197


How many tokens are probably consumed or how much it will cost to generate a table page.

How many tokens are probably consumed or how much it will cost to generate a table page. Is there a rough amount / number?

It varies depending:

which version you are on (have to send full history)
how large your screenshot is (affects how long the output will be with max of 4096 tokens)

Here are some sample tokens and costs for GPT-4 vision.

The project is definitely not optimized for token usage and there are no plans to optimize that further since the APIs are pretty cheap in general. If you're concerned about cost, we hope to support open source, local models that will be free but worse quality.
What's your budget?

It varies depending:

which version you are on (have to send full history)
how large your screenshot is (affects how long the output will be with max of 4096 tokens)

Here are some sample tokens and costs (USD) for GPT-4 vision.

The project is definitely not optimized for token usage and there are no plans to optimize that further since the APIs are pretty cheap in general. If you're concerned about cost, we hope to support open source, local models that will be free but worse quality.
What's your budget?

I just want to get a general idea


Pool server for equihash solution <n, k>

This PR, together with those to be applied on equihashverify and node-stratum-poo, allows to install a pool server based on z-nomp able to manage and validate the following equihash solutions:

n = 48, k = 5
n = 96, k = 3
n = 96, k = 5
n = 192, k = 7
n = 200, k = 9

Supported equihash algorithm options:

Equihash48_5
Equihash96_3
Equihash96_5
Equihash144_5
Equihash192_7
Equihash200_9

Dependencies:
https://github.com/ZencashOfficial/equihashverify/pull/2
https://github.com/z-classic/node-stratum-pool/pull/35

Concept ACK, also see https://github.com/NickRHill/zh-nomp

They accept only equihash as coin algorithm parameter. That implementation precludes the use of z-nomp in a generic way for all the variants of equihash (n = 200 / k = 9, n = 144 / k = 4, n = 192 / k = 7, etc etc) because these parameters are hard-coded in the used equihashverify fork and not passed as in my implementation:
z-nomp -> node-stratum-pool -> equihashverify

See also PR https://github.com/ZencashOfficial/node-stratum-pool/pull/1


images: increase max number of open files

Increased the maximum number of open files system limit for jenkins user in the images.

Tested on Jenkins, the system nofile limit has been increased.


Naive attempt at bumping vendor version of AWS SDK

This to match the version in go.mod, cf #377

Fixed by https://github.com/kubernetes-sigs/aws-iam-authenticator/pull/388


display_heatmap error

hello. i just following this tutorial using my local enviroment. At the display_heatmap cell, it always give me this error:
ValueError: X has 28 features, but MinMaxScaler is expecting 1 features as input.
i guess this problem rise since i use last channel format but the minmaxscaler can only apply on first channel format?
the tutorial: https://github.com/christianversloot/machine-learning-articles/blob/main/visualize-layer-outputs-of-your-keras-classifier-with-keract.md

i tested your 'heat_map.py' code (at the examples folder) too, and it give me the same error


[Task]: Deduplicate callout component in CE & EE

Closing it coz of change in release plan. Tracking it in epic https://github.com/appsmithorg/appsmith/issues/20460


[A] Add functions to check if table is empty to the API

Add functionality from https://github.com/seleznevae/libfort/issues/26

Pull Request Test Coverage Report for Build 649

20 of 27 (74.07%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 78.916%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tests/bb_tests_cpp/test_table_basic.cpp
0
7
0.0%

Totals

Change from base Build 641:
-0.03%

Covered Lines:
4353

Relevant Lines:
5516

💛  - Coveralls


JsFile: move getLinesWithCommentsRemoved from comment-helper and 100% code coverage

Coverage increased (+0.59%) to 98.63% when pulling 2aa66552892c2669f65c5e623b04d4ad13191741 on mikesherov:removeCommentHelper into 7711766f21e19f01c3d85bdb4147f4bbac7df8db on jscs-dev:master.

ping @markelog @mrjoelkemp just like token-helper, this is getting of comment-helper in favor of consolidating it back to JsFile where it belongs.

:+1:


Blendshape keys are not correct

Plugin Version or Commit ID
v0.12.0
Unity Version
2021.3.25
Your Host OS
Windows 10
Target Platform
UnityEditor, Android
Description
Hello, I'm working on connecting MediaPipe with Ready Player.me avatars, and I've noticed that the blendshape values I receive don't match the actual blendshapes. They seem mixed up. For example, the "cheekPuff" blend shape only responds when I close my right eye.
Is there a way to change the output values for all blend shapes? Is there an easier way to detect incorrect blendshapes?
Code to Reproduce the issue
No response
Additional Context
No response

Can it be the same issue as https://github.com/google/mediapipe/issues/4436 ?

yes, almost same, but there are much more issues.
I'm currently working on untangling the issue, however, it's very difficult to determine which parts of the face affect specific blend shapes. Can you tell me how to more easily connect the appropriate blend shapes with facial movements (For example, when I select a specific blend shape, I would like to see which points on the face mesh are included in the calculation of the blend shape value.)?
Then I'll send you the exact indexes for each blend shape (or any other data to help solve issue).


심사 5차 리젝

5차 리젝까지 왔네여 ^^..
카톡으로 다 전달 드렸지만... 리젝 기록차 이슈에 올려 놓겠습니다.
Thank you for your reply.
Guideline 2.1 - Information Needed
Before we can complete our review of your app, we need a video that demonstrates the current version, 1.0, in use on a physical iOS device.
Specifically, we need a demo video that shows app’s 초성확인하기 feature.
The demo video should:

Show your app running on a physical iOS device, not on a simulator.
Clearly documents all relevant app features, services, and user permission requests.

Next Steps
Provide a link to the video in the App Review Information section of your app’s page in App Store Connect and reply to this message. You can use a screen recorder to capture footage of your app in use. Note that if your app can only be reviewed with a demo video, you’ll need to provide an updated demo video for every app submission.
Resources
To learn more about providing information to App Store Review in App Store Connect, see App Store Connect Help.
Best regards,
App Store Review
즉, 실기기 데모영상 제출...

회신 완료~


Cannot connect to MQTT SERVER

I cannot connect to MQTT Server because I don't know
#define MQTT_SERVER "YOUR.MQTT.SERVER.IP"
I tried to run and got the message from Serial:
Attempting MQTT connection...	Failed.
Abort called

ctx: cont
sp: 3ffefc30 end: 3ffefe40 offset: 01b0

>>>stack>>>
3ffefde0:  3ffeedec 3ffeebbc 3ffeeae8 4020225a
3ffefdf0:  00000000 00000000 00000000 3fff0dc4
3ffefe00:  0000001f 00000019 3de83a2c 3ffeafef
3ffefe10:  3fffdad0 00000000 3ffeee10 402022ca
3ffefe20:  feefeffe feefeffe feefeffe 402043f8
3ffefe30:  feefeffe feefeffe 3ffeee20 40100a2c
<<<stack<<<

 ets Jan  8 2013,rst cause:2, boot mode:(1,6)

 ets Jan  8 2013,rst cause:4, boot mode:(1,6)

wdt reset

Please help me.

I did it myselft. Just using ifconfig and get local ip address. Thanks @ItKindaWorks


ui: fixed typo

Thanks for creating this project. Just seen a typo on this interface.

@nvibert Thanks for contributing. However it seems like this was updated just three minutes before your pull request, hence the merge conflict 😅

No worries!

No worries!

Feel free to pick up on something else if you'd like. Theres a couple of issues, also just providing feedback is very much appreciated!


Tooltips for dashboard icons not displaying

Minor issue in that the tooltips are not displaying for icons on the dashboard. It looks like they are trying to display but are not being shown because they are under the title.
Steps to reproduce

Mouse over icons on the dashboard. Like the Sent icon.

Fixed. Tooltips now display.


Preserve inline comments

Preserve inline comments such as this:
var = value   ; my comment

Also looks like this gets converted incorrectly:
before
StringTrimLeft x, x, 1           ; leading +x -> x

after
x := SubStr(x, "1           ; leading +x -> x"+1)


Eliminated shared runtime library

Compile times may have gone up ... tests seem to take a bit longer to run. @mateiz maybe you can measure this?
weld_module_free and weld_module_mem_free are now used to free memory (they free all memory allocated by the module, which is what weld_value_free did before), and weld_module_memory_usage is used to determine a module's total memory usage. weld_value_free and weld_value_memory_usage now do nothing. I think we should implement these later once we figure out how to correctly free and determine the memory usage of a single value.

Yep, that's right. I added some comments to the new functions clarifying this. The Python bindings have been updated.

Basically people will want to compile a module once and then call it many times, possibly in different threads. Having only a module-level free function won't make sense for that.

@mateiz @sppalkia fixed up the organization and naming of the runtime library, support for run_id is there as well

@jjthomas for some reason on Travis, the include_bytes! call can't find libweldrt - is it being compiled before the rust stuff?

I'm in the process of debugging this here: https://github.com/weld-project/weld/pull/178. It's because the new runtime code depends on the C++ standard library, but we don't have that library loaded in Python on Linux for whatever reason. I think it's a dynamically linked library.

BTW the reason it did not build is because errno is a macro in C++ (!) so you can't have variables named errno: https://github.com/weld-project/weld/pull/178/commits/c262c9e34e4ed308d6fc8a7fcfa7912c0f924326


The ckpt file would not work

I tried to find-tune the pre-trained model on my own data. I downloaded the pre-trained model from MS-Celeb-1M. The folder contains four files: a .pb file, a .ckpt.data file, a .ckpt.index file, and a .meta file.
I tried to restore the model from the last three files but got the same error:

DataLossError (see above for traceback): Unable to open table file ../models/20170512-110547/model-20170512-110547.meta: Data loss: not an sstable (bad magic number): perhaps your file is in a different file format and you need to use a different restore operator?

Does the .ckpt file not correspond to .pb file?
Thank you very much in advance.

i also encountered the same problem, and how to solve this about you?

I managed to fix it. Basically, my solution is based on this answer. The TensorFlow does not produce .ckpt file but separated three files. The option to run the train_tripletloss.py is --pretrained_model ../models/20170512-110547/model-20170512-110547.ckpt-250000.
Hope it could help you.


Document is in English, are translations intended or at least the possibility to translate?

Currently you only have the binary PDF in the repository. Would you be willing to copy the document in a text-based format, such as Markdown or LaTeX? This allows other people to easily modify the protocol's text or to produce translations.

Hey there.  It's a Google Doc currently and I don't think there's a way to export & re-import to another editable format that retains all of the formatting.
Open to giving commenting permissions to the Google Doc (would give edit permissions, but commenting is the only way to enforce review for security purposes).  However, it's unlikely that we will actually do another release unless a substantial problem arises OR someone takes over maintenance.
Thanks for the feedback!

Hey there. It's a Google Doc currently and I don't think there's a way to export & re-import to another editable format that retains all of the formatting.

There is https://github.com/lmmx/gdocs2md-html at least for exporting, although I haven't tested it myself.
I argue for getting the source file off Google Doc for better editing and higher portability. Currently anyone can copy the full text off the PDF, keeping the source file on Docs doesn't offer much security. A PDF can be generated from markdown with pandos or such, with nice formatting.

Open to giving commenting permissions to the Google Doc (would give edit permissions, but commenting is the only way to enforce review for security purposes).

I'd be open to that. Personally I am interested in

Making a translation myself
Getting the source file off a proprietary service
Minor development of the protocol

However, it's unlikely that we will actually do another release unless a substantial problem arises OR someone takes over maintenance.

Pity. I don't think I am qualified to take over maintenance, at least not alone. However keeping the protocol up to date und developing it further seems like a rather important project for the community to me. What can I do to help?

Thanks for the feedback!

It's my pleasure.

I'm not sure what help to ask for unless you are able to recruit another maintainer, unfortunately!
I am happy to give you commenting permissions on the Google Doc, though, which I think should allow you to "fork" it (i.e. make a copy within Google Docs, or export, etc.). Feel free to PM me on Gitter ("Discuss" link on the website) with your e-mail address.


Basic Documentation for the pyglet.gui module.

This branch adds:

Descriptions of each class and its __init__ method in widget.py
Descriptions of each property of the WidgetBase class in widget.py
A gui.rst file to show these docstrings in the documentation.
"modules/gui"in index.rst.

Please ignore this pull request since I intended the changes to be made on the pyglet1.5-maintenance branch.


[lsp-server] 🐞 code intellisense in Vue SFC is not working properly

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
In Vue SFC, code intellisense is not working and code error is displayed at the wrong place, but in js/ts file it's working.
In Vue SFC file:
https://user-images.githubusercontent.com/14963619/217221080-1dc487d9-4bac-4eba-b51a-0cff514228c4.mov
In JS file:
https://user-images.githubusercontent.com/14963619/217221105-50a0ea45-b2d9-4b54-af25-4674e2e077e9.mov
Expected Behavior
Code intellisense is working and code error is displayed at the right place in Vue SFC.
Steps To Reproduce
No response
Environment

LSP Server Version: latest
OS: macOS 13.0 (22A380)
LSP Client: vscode

Anything else?
No response

hmm ok. this might be different from the existing issue. it's confusing because we're using the vue sfc compiler directly here!

The version of Volar is v1.0.24. I'm not sure it is a Volar issue or a graphql lsp issue. Can you reproduce?

I also triggered this issue. Is there any solution?

This should be resolved as of early this year, sorry everyone!


Please make a new version for ES 6.3.0

See https://github.com/vvanholl/elasticsearch-prometheus-exporter/pulls

This is merged today


Error on opening Spyder in Anaconda

Description
I had issues with Pyinstaller and PyQt5 on windows, which was giving me an error. Firstly, I reinstalled PyQt5 using pip, but this did not do the trick. Next, I followed the advice on this thread: https://github.com/pyinstaller/pyinstaller/issues/3740 . The issue was solved, so hurray.
After rebooting my computer, later that day, I could not open spyder anymore. I receive the error as described over here: https://github.com/ContinuumIO/anaconda-issues/issues/1333 .
$ conda install qt=5 pyqt=5 qtpy=1.1.2 --force did the trick and made Spyder start up again. However, I got the error shown below.
plugin = mod.PLUGIN_CLASS(self)
AttributeError: '_ModuleMock' object has no attribute 'PLUGIN_CLASS'

Naturally I did a google search again and found this thread: https://github.com/spyder-ide/spyder/issues/7488 . This did not solve the issue unfortunately. It seems that $ conda install qt=5 pyqt=5 qtpy=1.1.2 --force made Spyder go back to version 3.2.3. After updating it back to spyder 3.3.1 via conda install spyder=3.3.1 spyder would not boot again, given the first error:
qtpy.PythonQtError: No Qt bindings could be found
So I tried a complete reinstall but that didn't solve the issue.
What has happened here? How can I make spyder 3.3.1 work again?
Version and main components

Spyder Version: 3.2.3
Python Version: 3.5.6
Qt Versions: 5.9.6, PyQt5 5.9.2 on Windows

Dependencies
pyflakes >=0.6.0 :  2.0.0 (OK)
pycodestyle >=2.3:  2.4.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.23.4 (OK)
numpy >=1.7      :  1.15.2 (OK)
sphinx >=0.6.6   :  1.7.9 (OK)
rope >=0.9.4     :  0.11.0 (OK)
jedi >=0.9.0     :  0.12.1 (OK)
nbconvert >=4.0  :  5.4.0 (OK)
sympy >=0.7.3    :  1.2 (OK)
cython >=0.21    :  0.28.5 (OK)
qtconsole >=4.2.0:  4.4.1 (OK)
IPython >=4.0    :  6.5.0 (OK)
pylint >=0.25    :  2.1.1 (OK)

This is fixed in our latest version (3.3.1). Please update by opening the Anaconda Prompt and running there
conda update spyder

I meant, this error
plugin = mod.PLUGIN_CLASS(self)
AttributeError: '_ModuleMock' object has no attribute 'PLUGIN_CLASS'

@ccordoba12 I have updated the issue. The error was the result of one of my intermediate steps $ conda install qt=5 pyqt=5 qtpy=1.1.2 --force. Updating back to spyder 3.3.1 made spyder not functional again. Could you reopen the issue?

We can't help you to debug Pyinstaller stuff, sorry.

I meant that we don't have time to help you debug this problem because it's not directly related to Spyder but to Pyinstaller.

Okay, thanks for the help anyways.


Wait for Firestore auth to finish before executing queries + add security rules

Previously things were working fine, as rules were wide open... So the auth wasn't really necessary and the problem was hidden.
[#169393818]
We don't have a separate repo for Firebase project, so rules are edited in the web console.
Could you please review them too?
https://console.firebase.google.com/project/glossary-plugin/database/firestore/rules
I did little QA myself using a local instance of the plugin and things seem to work fine now.

OK, I've fixed Travis issues, so merging it now.


Remove pod sandoxes in parallel

We now use a classic work-stealing approach to remove the sandboxes in
parallel if multiple are defined.
/cc @ereslibre

what about using AggregateGoroutines?

Thanks for the hint, used the suggested API in the implementation :+1:


Paths with colon causes "Failed to load API definition." error.

Version: 6.2.3
Frequency: Always
Error description
As we want to follow the rules to add colons when the action is making non CRUD operation, due to recommendations here: https://cloud.google.com/apis/design/custom_methods
We are experiencing issue with generation of Swagger documentation. The problem is not occueing in version 5.6.3 and we can successfully use this version.
Example
 [HttpPut("{id}:Example")]
public async Task<IActionResult> Example(string id)
{
...
}

The swagger UI cannot be generated for this:

Dup of #2239 and fixed by #2310. Nuget release on it's way!

was this error resolved?
I'm on the latest version and the problem still persists.

@marekzyla  the issue has been resolved in release 6.3.0

I can confirm it's solved. Thanks!


[#2622] Makes forgot-password link removable

References

Fixes #2622
Requires DSpace/DSpace#9194

Description
Introduces a new check to show the forgot-password link that uses the related AuthorizationFeature
Instructions for Reviewers
Try to enable the additional configuration:
user.forgot-password = false

inside the authentication-password.cfg and check that it is not shown inside the login-component, in both:

dropdown login menu
login page

Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes ESLint validation using yarn lint
[x] My PR doesn't introduce circular dependencies (verified via yarn check-circ-deps)
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[x] If my PR includes new libraries/dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR includes new features or configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

The e2e tests are failing due to the changes introduced in the login-component.
Once the #9194 backend PR will be merged, this test will pass without any problem.

@vins01-4science : Could you resolve the merge conflicts in this PR to make it easier to review?

@vins01-4science : Could you resolve the merge conflicts in this PR to make it easier to review?

Yes, sorry I forgot to address your comment.
I will update it shortly.
Thank you.

@vins01-4science : Gave this a test & review today. Overall it looks good.
I've also verified that the e2e test failure in this PR only occurs because the backend PR isn't merged yet. Once the backend PR & this one are merged, this e2e test should succeed (I've verified it succeeds for me locally with both the frontend & backend PRs installed).
All that said, I noticed a few small issues that would be nice to fix if possible:
* Even when `user.forgot-password = false` is set on the backend, I can still access the Forgot Password page by typing in the "/forgot" path in my browser address bar (e.g. `http://localhost:4000/forgot`)  The good news is that when I submit it, I do receive an expected error.  But, ideally, I should not be allow to access this page.  Can we place some sort of "guard' in front of it that makes it inaccessible when disabled?

* Although it appears to not be related to this PR... I'm noticing that `user.registration = false` no longer works properly (at least via `yarn build:prod; yarn serve:ssr`).  The "Click here to register" link _always_ appears in the login dropdown even if that config is set to false.  (NOTE: This appears to also be a bug on `main`. But, if you find time to fix it in this PR, feel free.  Otherwise, I can log it as a separate bug ticket and look for a volunteer.)

@tdonohue Thank you, everything have been addressed.
For the user.registration issue I was not able to reproduce it, so probably it needs more investigation.
The guard issue has been addressed, so let me know if you find something else!


Fix issue in org-static-blog-with-find-file handling

org-static-blog-with-find-file cannot be used in this case as it erases the buffer. Adding a special case for org-static-blog-render-post-content.

Thank you for your pull request.
Could you explain the issue and your fix in a bit more detail?

Thank you for your pull request.
Could you explain the issue and your fix in a bit more detail?

Yeah sorry for not adding more. My previous changes made org-static-blog-with-find-file insert the provided contents into the current temporary buffer. This was the case for almost all of the uses of org-static-blog-with-find-file. However, org-static-blog-render-post-content was not calling this within a temporary buffer. Instead, it is just exporting html through org mode. This means that it can't use the same logic used for org-static-blog-with-find-file and needs its own handling.

Cool, thank you!


Refactor UI health check

Check that the NextJS app is up rather than the nginx
Refactor responsibility for verifying production has correct secrets

Keep the health endpoint free from things it doesn't care about
Throw an error in config so the app won't start in prod if it has default env parameters

Pulled apart by #1121 and #1120


Missing FormData and Request.formData

I'm building a C@E app which reads a html form submission but I found that FormData and the helper method Request.formData currently are not implemented in the runtime.
Are these planned to be implemented in the runtime?

Yes, we do plan to support them! I unfortunately can't right now give a good estimate of when they'll be available, though, as we haven't yet finalized planning for future feature work for JS.

Yes, we do plan to support them! I unfortunately can't right now give a good estimate of when they'll be available, though, as we haven't yet finalized planning for future feature work for JS.

That's okay, it's good to hear they will be supported. I've included a polyfill for FormData which works, I'm looking forward to when I can delete all the polyfills 😄

any update on this?
My team is in the process of migrating two sizable Remix apps to C@E and removing our reliance on FormData would be a substantial effort.
I've tried several different strategies to polyfill FormData without success.
The polyfill @JakeChampion mentioned above was bespoke for a private project. As a last ditch effort if there are no other paths forward, we could potentially do the same but would really prefer a less involved solution.

Moving to https://github.com/bytecodealliance/StarlingMonkey/issues/116.


Incorrect path for multi-part keys (composite keys)

I believe the property name is required for multi-part keys.
See https://stackoverflow.com/a/12510062/1220627
Currently paths are generated without property names regardless.
Current
{
  "swagger": "2.0",
  // ...omitted
  "paths": {
    "/FooSet('{fooId}')": {
      // ...omitted
    },
    "/MultiKeySet('{fooId}','{barId}')": {
      // ...omitted
    }
  }
}

Correct swagger for OData
{
  "swagger": "2.0",
  // ...omitted
  "paths": {
    "/FooSet('{fooId}')": {
      // ...omitted
    },
    "/MultiKeySet(fooId='{fooId}',barId='{barId}')": {
      // ...omitted
    }
  }
}

My observation has been that the property name is only required for versions of OData prior to version 4 and that in version 4, these fields become optional.

That could be the case. I'm consuming a version 2 OData api.
Does this tool only support v4?
Never the less, if they're optional in v4 adding the property names would enable backwards compatibility :)
BTW are you aware of this https://github.com/oasis-tcs/odata-openapi? It seems like an official tool for converting CSDL to OpenAPI. While it's a bit complicated to run it does a very good job. Perhaps this tool could use that for conversion but make it simple to run via Node. Than all the conversion edge cases are handled by the OData team them self - seems like a win/win :)


Fix typo in AMSI test

PR Summary
Now CI-Windows fail after #8546 : correct typo in cmdlet name in AMSI test.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

[x] Add [feature] if the change is significant or affects feature tests

@SteveL-MSFT Please review to fix night tests.


Added /completions endpoint back to support gpt-3.5-turbo-instruct

This change addresses the issue discussed here. With the current state, it is impossible to use gpt-3.5-turbo-instruct without the completions endpoint or downgrading the gem version.
All Submissions:

[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 update/change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?

@alexrudall

Thanks, @mishranant !

Released in v6.5


Twitter import seemed to fail w/o exception but throwing a backtrace

See import ID: 9bdbc780-8ae3-478a-bec3-1f65d8ccf07c
This is the backtrace:
services/importer/lib/importer/runner.rb:241:in `block (2 levels) in single_resource_import'
services/importer/lib/importer/importer_stats.rb:33:in `block in timing'
shared/bundle/ruby/1.9.1/gems/statsd-client-0.0.7/lib/statsd.rb:26:in `timing'
services/importer/lib/importer/importer_stats.rb:31:in `timing'
services/importer/lib/importer/runner.rb:239:in `block in single_resource_import'
services/importer/lib/importer/importer_stats.rb:33:in `block in timing'
shared/bundle/ruby/1.9.1/gems/statsd-client-0.0.7/lib/statsd.rb:26:in `timing'
services/importer/lib/importer/importer_stats.rb:31:in `timing'
services/importer/lib/importer/runner.rb:230:in `single_resource_import'
services/importer/lib/importer/runner.rb:95:in `run_import'
services/importer/lib/importer/runner.rb:88:in `block in run'
services/importer/lib/importer/importer_stats.rb:33:in `block in timing'
shared/bundle/ruby/1.9.1/gems/statsd-client-0.0.7/lib/statsd.rb:26:in `timing'
services/importer/lib/importer/importer_stats.rb:31:in `timing'
services/importer/lib/importer/runner.rb:87:in `run'
app/connectors/importer.rb:39:in `run'
app/models/data_import.rb:588:in `new_importer'
app/models/data_import.rb:331:in `dispatch'
app/models/data_import.rb:158:in `run_import!'
lib/resque/importer_jobs.rb:9:in `block in perform'
lib/resque/base_job.rb:21:in `call'
lib/resque/base_job.rb:21:in `run_action'
lib/resque/importer_jobs.rb:9:in `perform'

This is a raise CartoDB::Importer2::FileTooBigError error, user is already properly notified, was just the import log the one lacking better context. New format:
2015-06-18 14:18:57 UTC: Starting import for /tmp/imports/20150618-5199-8uegii/twitter_xxxx.csv
2015-06-18 14:18:57 UTC: Errored importing data:
2015-06-18 14:18:57 UTC: CartoDB::Importer2::FileTooBigError: /tmp/imports/20150618-5199-8uegii/twitter_xxxxxx.csv
2015-06-18 14:18:57 UTC: ----------------------------------------------------
2015-06-18 14:18:57 UTC: ["/home/kartones/src/cartodb/services/importer/lib/importer/runner.rb:245:in `block in single_resource_import'"


Proper way to inject Android Context.

Hi, what is the proper way to inject Android Context?

Hi,
it's currently not possible, nor should it be.
By definition, stocking a Context should only be done with extreme care to prevent memory leak.
Furthermore, the Context depends... on the context. Whether you want the Activity, a Service, the Application context, etc. depends on the location and purpose of your code.
At most, you could register the Application object (which is a Context) in your kodein bindings ;)


Error on every second message

File "/opt/render/project/src/.venv/lib/python3.8/site-packages/EdgeGPT.py", line 555, in ask_stream
May 14 07:21:33 PM      response["item"]["messages"][-1]["contentOrigin"] == "Apology"
May 14 07:21:33 PM  KeyError: 'messages'

this is the upstream error ( seems You sent a sensitive message that the bot refuses to reply )


Plotting segments of Orbits in 2D as well

Refactored the OrbitPlotter . Now it can plot segments of Orbits in 2D as well due to the plot_trajectory method. Addresses Issue #274

Awesome! Can you add some screenshot of how the plot_trajectory method looks like? For instance, plot some Hohmann transfer with only the traversed parts of the orbit.

Sure! I will do that in night

@Juanlu001  , Do you have some data about the hohmann transfer?

Do you have any idea on how to plot this?

I'm being able to plot in your branch without problems:

Using this notebook:
https://gist.github.com/Juanlu001/e574886ccf8bff10c8c08459981a674f
Please review the reasons why it does not work for you. Leaving some extra comments.

Rebased! :)

This is looking better :) Now we can improve notebooks like this one without cluttering the final plot:
http://docs.poliastro.space/en/latest/examples/Going to Jupiter with Python using Jupyter and poliastro.html
Merging!

True:)


fixing bug in wakeable loop causing IndexOutOfBoundException

When there is a task in taskOnWakeup that clears the taskOnWakeup list i can cause the following exception:
java.lang.IndexOutOfBoundsException: Index: 1, Size: 1
at java.util.ArrayList.rangeCheck(ArrayList.java:653)
at java.util.ArrayList.get(ArrayList.java:429)
at com.twitter.heron.common.basics.WakeableLooper.executeTasksOnWakeup(WakeableLooper.java:160)
at com.twitter.heron.common.basics.WakeableLooper.runOnce(WakeableLooper.java:89)
at com.twitter.heron.common.basics.WakeableLooper.loop(WakeableLooper.java:79)
at com.twitter.heron.instance.Slave.run(Slave.java:180)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)
This happens in stateful process.  When there is an error and the instance gets a message to restore to  state it will call the following:
https://github.com/twitter/heron/blob/master/heron/instance/src/java/com/twitter/heron/instance/Slave.java#L113
which initiates the restore and is also a task in taskOnWakeup.  The restore process also involves clearing the taskOnWakeup list.

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

@maosongfu @nwangtw This exception doesn't arise from any concurrency situations.  Its just one task in list actually clears the list.  So in the previous for loop when we try to get the next task to run e.g. task i + 1 there will be an index out of bounds exception thrown since their is nothing in the list anymore.  When iterating a CopyOnWriteArrayList, a separate copy is made for the read and thus the clearing of the list will not effect us

Thanks for reply! It makes more sense now.
However I am afraid this fix is still not safe. If my understanding is correct, the clear call should clear all tuples immediately and cancel the pending actions. With CopyOnWrite list, the current content in the lists are still being executed afterwards.

Thanks for the explanation. At the same time, I am also looking at the code and thinking about whether we would have better options for a fix.

@maosongfu @nwangtw I have made the fix simpler and with fewer changes.  Can you please take another look?

@nwangtw I have pushed another commit that adds a locking mechanism for adding new tasks and timers to the wakeable loop. During the state restore, we can lock the looper so that no new tasks can be added and after the state restore is completed we can unlock.  What do you think?

@nwangtw @maosongfu I looked through code more thoroughly to find who is add adding or removing tasks from wakeable loop for the slave thread. Only methods that handle state operation i.e. com.twitter.heron.instance.Slave.handleControlMessage will potentially add tasks as a task exisiting in the queue. There is no concurrency problems since only tasks are add upon initializing the slave thread or by tasks run by the slave looper so everything is in sync.  Thus, we don't really need locks per se but just make sure we handle the situation where tasks add other tasks and tasks clearing the list correctly

@nwangtw I am not sure about changing the Runnable to a custom interface to support the return of a boolean to determine whether we should clear to list.  Seems like a heavy change since a lot things use the wakeable not just the slave thread

@nwangtw I add a boolean that gets set when tasks are cleared to break out of the loop.  What do think about that mechanism?

@nwangtw thanks for reviewing the PR again.  I have addressed your comments.  Do you have any further concerns?

Good comments are definitely helpful for my concerns. PR is acceptable to me after addressing maosong's comments.

@maosongfu @nwangtw I have added the comments requested.  Thanks for the review!


Create a project in the kogito-examples repository that showcases the architecture

Create a project in the kogito-examples repository that showcases the architecture, using the kogito-addons-quarkus-data-index-persistence-postgresql addons and including Jobs Service

PR: https://github.com/apache/incubator-kie-kogito-examples/pull/1833

We need to update the runtime image in the example (right now using the nightly version) so it can properly run in quarkus 3 (https://github.com/apache/incubator-kie-issues/issues/873)

Merged!


Update iis.rst

I didn't create an Issue because this is not for a new article ... it's just to add a section to an existing article: How to use Web Deploy with .NET 5 projects and IIS.
I imagine this section would save folks time trying to figure out what to do until the tooling gets here to make it sweet. I took the key bits that I reported in https://github.com/aspnet/Hosting/issues/141 and https://github.com/aspnet/Home/issues/452 and rolled it into a section.

Hi @GuardRex, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@GuardRex, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

I just saw this open topic: http://docs.asp.net/en/latest/publishing/azure-windows-vm.html
I'd like to volunteer for that ... We can kill this PR, and I can expand and include screenshots and make it really nice.


Add contribution guidelines

Added CONTRIBUTING.md file to outline contribution guidelines
Along the way:

Updated README.md to link to contribution guidelines.
Added DCO bot config to ignore org members
Added .npmignore to not publish .github/ or .eslintrc

LGTM


syslog: fix crash when print localtime by syslog

Summary
syslog: fix crash when print localtime by syslog
test need to enable config:
CONFIG_LIBC_LOCATIME=y
CONFIG_SYSLOG_TIMESTAMP_FORMATTED=y
CONFIG_SYSLOG_TIMESTAMP_LOCALTIME=y
CONFIG_SYSLOG_TIMESTAMP_REALTIME=y
Change-Id: I322830e0818237a7eb65a158a6a0dea8f9da9b6c
Signed-off-by: Jiuzhu Dong<EMAIL_ADDRESS>Impact
output local time in syslog.
Testing
daily test.

@Donny9 What was the problem with this? I tried it, but I didn't have any kind of "crash", it worked OK for me.
I am following the changes, but I don't understand the functional changes this PR brings. It seems to me only "styling" changes are made. Can you please elaborate on the issue and the solution?
Also, to my understanding this PR introduces a bug.
If OSINIT_HW_READY() returns false, then the variable tm will be used uninitialized in line 132.
The same applies for variable ts in line 140.
Also, why ingore the return code of the various calls in clock_gettime(), clock_gettime(), clock_systime_timespec()?   Isn't it better to check for the error, and provide sane values in tsandtm`?

@Donny9 What was the problem with this? I tried it, but I didn't have any kind of "crash", it worked OK for me.
I am following the changes, but I don't understand the functional changes this PR brings. It seems to me only "styling" changes are made. Can you please elaborate on the issue and the solution?
Also, to my understanding this PR introduces a bug.
If OSINIT_HW_READY() returns false, then the variable tm will be used uninitialized in line 132.
The same applies for variable ts in line 140.
Also, why ingore the return code of the various calls in clock_gettime(), clock_gettime(), clock_systime_timespec()?
Isn't it better to check for the error, and provide sane values in ts and tm?

Hello @fjpanag , you need to enable follow config, then use syslog to output log with localtime in origin code base.
CONFIG_LIBC_LOCATIME=y
CONFIG_SYSLOG_TIMESTAMP_FORMATTED=y
CONFIG_SYSLOG_TIMESTAMP_LOCALTIME=y
CONFIG_SYSLOG_TIMESTAMP_REALTIME=y
The root cause of cash is that the localtime_r can't be called before OSINIT_HW_READY, because localtime_r will call sem_wait, but semaphore subsystem is uninitialized.

@Donny9 What was the problem with this? I tried it, but I didn't have any kind of "crash", it worked OK for me.
I am following the changes, but I don't understand the functional changes this PR brings. It seems to me only "styling" changes are made. Can you please elaborate on the issue and the solution?

The key change is move localtime_r/gmtime_r into if (OSINIT_HW_READY()).

Also, to my understanding this PR introduces a bug.
If OSINIT_HW_READY() returns false, then the variable tm will be used uninitialized in line 132.
The same applies for variable ts in line 140.

ts and tm is initialized to zero at line 74-76 and 79-81.

Also, why ingore the return code of the various calls in clock_gettime(), clock_gettime(), clock_systime_timespec()?
Isn't it better to check for the error, and provide sane values in ts and tm?

The sane value(zero) set at the definition.

@Donny9 I now understand the issue, thank you!
BTW, as I see, there is no CONFIG_LIBC_LOCATIME defined anywhere....
@xiaoxiang781216 You are correct.
I followed the code with my debugger, and saw the variables containing garbage. But it was my mistake, they are in fact initialized before being used, as they should.

Yes, CONFIG_LIBC_LOCATIME is enabled in our product to convert UTC to local time correclty, which is seldom selected in the demo project. That's why the problem is found until now.


Replace all BorderRadius.circular with const BorderRadius.all

This PR contains such replacements: BorderRadius.circular(arg) -> const BorderRadius.all(Radius.circular(arg)).
I ran new Dart Code Metrics rule prefer-const-border-radius on flutter repo and left test/painting/border_radius_test.dart unchanged since it tests BorderRadius.circular behaviour.
Pre-launch Checklist

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

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

Isn't there a way to make BorderRadius.circular(arg) a const?

Isn't there a way to make BorderRadius.circular(arg) a const?

Nope, we have to create Radius object during construction.
https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/painting/border_radius.dart#L294

The reason we have the BorderRadius.circular(arg) constructor is to avoid requiring that people create the much more verbose alternative. But certainly for people (such as us!) who prefer the verbosity that allows const, it would be an interesting lint.
Thanks @grafovdenis!

For instance, if the motivation is not to depend on any community package (coz they can be abandoned, contain malicious code, etc.), then it's totally understandable.

Yeah, we started doing that because otherwise any dependency that made a slight change that stopped the tool from compiling would mean that all our users would get stuck on a broken version and the tool would be unable to run to get them fixed. (And also any dependency that made an analyzer error pop up or anything like that on any of our tests anywhere would cause our CI to go red, which was happening at least once a week before we pinned everything.)

Isn't there a way to make BorderRadius.circular(arg) a const?

I can see that Radius.circular is now a const constructor then why can't BorderRadius.circular be a const constructor as well?


Feature Request: Notification for rare drops.

Add an option to send a notification on rare drops
With adjustable value.

+1 I came here to add this. This was commented 2 years ago, I am not sure if this is coming


Support actions other than simple assignment

Inspiration: https://docs.python.org/3/library/argparse.html#action

[x] assign (store; store_const, store_true, store_false can be accomplished with implicit value)
[x] append (also append_const with implicit value)
[x] print_help (help)
[x] count
[x] version

Done for now with this last PR for count: https://github.com/ggabriel96/opzioni/pull/58


[2.0] server render with renderToStream

hi, I try to use renderToString and renderToStream at server-side，it works well with renderToString，but as for renderToStream，I don't know how to deal with the back-end template such as jade、ejs and so on. renderToStream has better performance than renderToString when the component is complex，but in real application it seems not easy to apply on the existed framework such as express, so I hope there is an example to show us how to use renderToStream in real application，thx~

Hi!
Please make sure to ask the question on the forums or the gitter chat. We keep questions out of issues :smile:
The example is here https://github.com/vuejs/vue/blob/next/benchmarks/ssr/renderToStream.js.
About using express, you just need to do stream.pipe(res). Here's is an example https://youtu.be/c9esL3I4IGM?t=1h31m55s