Fix: Update Tendermint RPC Broken Link

Error 404:
https://docs.tendermint.com/master/rpc/#/Info/header
https://docs.tendermint.com/master/rpc/#/Info/header_by_hash
Fix:
https://docs.tendermint.com/main/rpc/#/Info/header
https://docs.tendermint.com/main/rpc/#/Info/header_by_hash
Description
Please add a description of the changes that this PR introduces and the files that
are the most critical to review.
If this PR is non-trivial/large/complex, please ensure that you have either
created an issue that the team's had a chance to respond to, or had some
discussion with the team prior to submitting substantial pull requests. The team
can be reached via GitHub Discussions:
https://github.com/celestiaorg/celestia-core/discussions
If the work in this PR is not aligned with the team's current priorities, please
be advised that it may take some time before it is merged - especially if it has
not yet been discussed with the team.
See the project board for the team's current priorities:
https://github.com/orgs/celestiaorg/projects/24
-->

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use
unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

This inconsistency may also exist upstream in CometBFT

CometBFT use https://docs.tendermint.com/v0.34/...which return status 200. Their links to Tendermint is Ok.


[Question] Some questions about namespace bundle splitting

According to the definition of bundle and relationship between a topic and a bundle, once a topic is created the hash of its name will be constant. If there is a splitting happens on a bundle, the hash of some of the topics that belong to the bundle will fall into the new bundle. If the new bundle is assigned to another broker, then the load for those topics will be shed to another broker. But if there is always heavy load on one certain topic, splitting on the bundle does not help to spread the load on that topic, is it correct?
If that's the case, how to solve it?

Yes, my question is about this. If the bundle only contains one partition and the partition is still overloaded, is there a way to solve it?

Currently partition is the finest granularity in Pulsar. We can't split it now.

Increasing partition does not work for me cause I use key hash as the routing mode while producing the messages.

Does the new subscription mode key_shared address your problem?


Timeout for hung requests

Sometimes when downloading a lot of data from NasdaqDataLink, I get a hung connection during the ssl handshake. The requests library, which is being used to make requests, can take a timeout parameter, but this is not exposed through the API as far as I can tell.
What's the best way to retry after a timeout?

-- Please reply above this line --
Dear Nasdaq/data-Link-Python,
Thank you for contacting the Nasdaq Data Client Success Team. Your
email is very important to us and we will respond to you within 1 to 2
business days.
To efficiently support your request, please include relevant details
such as the data product/feed you are inquiring about.
Kind regards,
Nasdaq Data Client Success
[1]
Links:
[1] https://www.nasdaq.com/

-- Please reply above this line --
        Hello,

Thank you for contacting us.
We are looking into your request and will get back to you soon.
Thanks,
VAIBHAV TEMBEY
Nasdaq Data Client Success
How would you rate my reply?
Great [1]    Okay [2]    Not Good [3]
Links:
[1]
https://secure.helpscout.net/satisfaction/647704996/record/7066484955/1/
[2]
https://secure.helpscout.net/satisfaction/647704996/record/7066484955/2/
[3]
https://secure.helpscout.net/satisfaction/647704996/record/7066484955/3/
    > On Wed, Sep 13, 2023 at 3:04 PM EDT, Nasdaq/data-Link-Python<EMAIL_ADDRESS>wrote:

WARNING - EXTERNAL EMAIL; EXERCISE CAUTION.
Sometimes when downloading a lot of data from NasdaqDataLink, I get a
hung connection during the ssl handshake. The requests library, which
is being used to make requests, can take a timeout parameter, but this
is not exposed through the API as far as I can tell.
What's the best way to retry after a timeout?

Reply to this email directly, view it on GitHub [1], or unsubscribe
[2].
You are receiving this because you are subscribed to this
thread.Message ID:
@.***>
Links:
[1] https://github.com/Nasdaq/data-link-python/issues/39
[2]
https://github.com/notifications/unsubscribe-auth/AWYQ644AWJSEYMALXQGWKI3X2H7QVANCNFSM6AAAAAA4W5LXP4

-- Please reply above this line --
        Hello,

Thanks for your patience.
Can you please share with us the exact error message you receive? It
would also be helpful if you could share with us the code as well.
Thanks,
VAIBHAV TEMBEY
Nasdaq Data Client Success
How would you rate my reply?
Great [1]    Okay [2]    Not Good [3]
Links:
[1]
https://secure.helpscout.net/satisfaction/647704996/record/7076792905/1/
[2]
https://secure.helpscout.net/satisfaction/647704996/record/7076792905/2/
[3]
https://secure.helpscout.net/satisfaction/647704996/record/7076792905/3/
    > On Thu, Sep 14, 2023 at 12:50 PM EDT, Client Success - Nasdaq Data<EMAIL_ADDRESS>wrote:

Hello,
Thank you for contacting us.
We are looking into your request and will get back to you soon.
Thanks,
VAIBHAV TEMBEY
Nasdaq Data Client Success

On Wed, Sep 13, 2023 at 3:04 PM EDT, Nasdaq/data-Link-Python @.***> wrote:
WARNING - EXTERNAL EMAIL; EXERCISE CAUTION.
Sometimes when downloading a lot of data from NasdaqDataLink, I get a
hung connection during the ssl handshake. The requests library, which
is being used to make requests, can take a timeout parameter, but this
is not exposed through the API as far as I can tell.
What's the best way to retry after a timeout?

Reply to this email directly, view it on GitHub [1], or unsubscribe
[2].
You are receiving this because you are subscribed to this
thread.Message ID:
@.***>
Links:
[1] https://github.com/Nasdaq/data-link-python/issues/39
[2]
https://github.com/notifications/unsubscribe-auth/AWYQ644AWJSEYMALXQGWKI3X2H7QVANCNFSM6AAAAAA4W5LXP4


Replace serially-numbered configuration properties with array-valued properties

Describe the bug
Not really a bug, but an annoyance and an opportunity for improvement.  We have several groups of configuration properties such as webui.browse.index.N which were invented before we adopted Commons Configuration.    Nowadays we could simply list multiple instances of the key webui.browse.index and get them all, in order, as an array from ConfigurationService.getArrayProperty() instead of trawling through the keyspace by iteratively composing numbered keys until we generate a miss.  In most cases the numbers are insignificant.
In the case of parallel property groups (of which I am sorry to say there are a nonzero number) we really should look at moving these array-of-structures configurations into Spring, which can handle them properly.  In their current form these cases are particularly fragile both to manage and to parse.  These get my vote for the second most heinous abuse of Properties after java.util.logging.

Agreed, this would be nice.  However, it'd definitely take some careful code refactoring that'd need to happen in a major release (8.0, 9.0, etc).  Will pull this over to "Help Wanted" board for now until we can find a volunteer.


Fixed typo (genral -> general)

@prokopst : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks @prokopst


Bootload the program through serial UART/I2C

Tailing to the Issues posted on the Secure-loader Git page
https://github.com/MaximIntegratedAI/secure-loader/issues/3#issue-1618428238
https://github.com/MaximIntegratedAI/secure-loader/issues/4#issue-1621104811
I am completely struck in this issue since a week and I am totally confused.
1)Can we bootload the code to the MAX78000 using  UART/I2C/SPI? Is there a clear documentation regarding that?
2) Can we use the MAXDAP(MAX32625PICO) to use the MAXIM bootloader tools?
I tried to upload the latest dap firmware [max32625_max32666fthr_if_crc_swd_v1.0.4_pico2.bin] from BT/daplink/ but I cant
as the MAXDAP cant enter maintenance mode at all.
I get this

some times it enables and sometimes it can't.
is there a solution for this?
3)How to enter the bootloader through the application program?

I want a stable example like using 2 MAX78000 to program one MAX78000 as target and one as host, To clearly understand how the bootloader programming works.

5)Next use PICO to program MAX78000 using the serial interface.
Please someone help me.

Hi @chaitanya-smartrotamac, I've been reviewing our documentation.  I agree that it is not clear at all.  I'm tracking down some resources to support you.  Theoretically we can port the Bootloader_Host example to the MAX78000, but I am confirming internally exactly which version of the bootloader is on the MAX78000.  There are some mismatches in our documentation about that as well.
Apologies for the confusion, I will keep you updated as I uncover some more info.

HI, @Jake-Carter Thank you for your response.
Is there a possibility that I can get the source code for this bin file?
So, I can develop my own bootloader to communicate with MAX78000?
https://github.com/MaximIntegratedAI/secure-loader/blob/main/MAX78000_Bootloader_v3_4_4.bin

@chaitanya-smartrotamac  Sure, I'll provide it if I can track it down.
Here's what I've been able to clarify so far:  There are two types of bootloaders that we have.

The first type are ROM-based bootloaders.  On startup, these ROM-based bootloaders look for an activation condition that is documented in the microcontroller's User Guide.  When that condition is met, it offers a serial interface (usually UART or I2C) for loading programs into flash.

The other type are Flash-based bootloaders.  These usually do not come loaded into the device by default.  They must be loaded over SWD first.  Afterwards, they present their own activation conditions and bootloader interface over I2C/SPI/UART.  The secure-loader is this type of flash-based bootloader.

Additionally, the two types of bootloaders each come in two types of variants:

Non-secure (No extra security features)
Secure (Validates and protects the flashed application with extra layers of security)

Therefore we have 4 total possibilities:

Non-secure ROM-based bootloader
Secure ROM-based bootloader
Non-secure flash-based bootloader
Secure flash-based bootloader

From what I can gather from my colleagues the MAX78000 comes with type 1 (Non-secure ROM-based bootloader) by default.
Its usage is covered by the MAX78000 User Guide Section 26.  Its activation is covered by Sections 26.4 and 26.5, and the command-set is documented in section 26.8 (general commands).  Any documentation labelled with secure is not applicable for it.
My colleagues have also provided some host scripts for driving the type 1 bootloader in the MAX78000 from a host PC.  They're fairly minimal and undocumented, but I have not been able to find any microcontroller example host code yet.  Hopefully they're a somewhat useful reference until I can pull together some better resources.

max78000_rombootloader_host.zip

Thank you @Jake-Carter for your efforts.
I will check this https://github.com/Analog-Devices-MSDK/msdk/files/10996632/max78000_rombootloader_host.zip
I am developing a secondary bootloader to load the code through UART but the code size is pretty huge to fit as a bootloader.
Are there any documents points to the assembly coding in MAX78000 and how to create linker files for the MAX78000 compiler will be very much useful.

The ROM bootloader for the MAX78000 should use UART by default.  I think for your use-case it's probably the best option, since it does not occupy any additional space in flash.

Are there any documents points to the assembly coding in MAX78000 and how to create linker files for the MAX78000 compiler will be very much useful.

For assembly coding you can refer to the Cortex-M4 instruction set documentation from Arm.
To add an assembly file to a build, save it with the .S file extension.  You must use capital .S for our build system to pick it up.  You'll also need to add it to the SRCS and VPATH build configuration variables.
Our startup file is written in assembly and could be used as an example.
For linker files you can refer to the GNU ld manual and use our default linker file as an example.  The LINKERFILE build config variable can be used to change the linkerfile for the build.  I haven't really found a great reference for Arm-specific linkerfiles, but there are a few articles online.

Hi @chaitanya-smartrotamac, I will close this issue for now.  Please let us know if there's anything else we can help clarify

Hi @Jake-Carter , I am trying to use an external MCU to load code to MAX78000 through UART.
I can reliably activate the bootloader through UART. I can send the command "i\r\n" to get back "USN:..".
I can also send the load command "l\r\n" and get back the "Ready to load SREC", however I am not sure in what format and timing should I send the data. I get a "Load success..." straight away.
On the PC to MAX78000 Python implementation you shared it seems like the pages must be erased before loading the code?
It would be great to have more documentation on how to reliably load the file, which format to use when loading, and the timing of the communication between MAX78000 and an host MCU, or if you have any implementation in C of the python code you shared.
Thank you in advance!


Sass: Use @for to Create a Sass Loop - Doesn't accept spaces

Describe your problem and - if possible - how to reproduce it
The test will only accept:
@for $j from 1 through 5 {
  .text-#{$j} {font-size:10px*$j;}
}

But not:
@for $j from 1 through 5 {
  .text-#{$j} {font-size: 10px * $j;}
}

Add a Link to the page with the problem
https://learn.freecodecamp.org/front-end-libraries/sass/use-for-to-create-a-sass-loop
Tell us about your browser and operating system

Browser Name: Chrome
Browser Version: ?
Operating System: macOS

If possible, add a screenshot here

This is also tracked in #12699

bro same problem

@abenjamin765 it works fine now. Please close the issue

Close


AT set up cmds

Hi All
In the section
static esp_at_cmd_struct at_custom_cmd[] = {
I am defining some custom setup up commands, now im am using only integers, for example
at.xxx = 1
OK
and it is working, but now I would like if the setup function would receive also strings?
for example at.xxx = "community_name"
Regards
Juan

Hi @santanapablo1975 ,
You can call esp_at_get_para_as_str to parse string, please refer to the fuchtion
at_setupCmdCipupdate in main/at_task.c.

Ok thanks, I am going to check it, we can close then the isuue!


feat(types): allow typed access to properties added to entry

Summary

The helpers in the esm entry file were attached to the "prototype" of instantsearch instead of exported, which makes them non-treeshakeable. Removing it now would be a breaking change, but that form can at least already be encouraged.
Result

move the code in main.ts to index.ts for consistency with esm entry file
deprecate things attached to index.es.ts (for /helpers https://github.com/algolia/instantsearch.js/discussions/4749)

The thing is we'll likely update/clean our exports during the React InstantSearch migration, so I'm not sure that's the right time for this change (also definitely an improvement) because it will make it harder to not introduce breaking changes.

Changing exports is always a breaking change @francoischalifour, right? I know for sure that we don't want to continue exposing the helpers as a property on the es entry point (it's not documented), the deprecation message mentions /es/helpers, as I don't think it's useful to discourage use without explaining what a right way of using it is.
This PR mainly focuses on having the right types for something that already is the case, whether it's typed or not, removing those entries is a breaking change :)

Sorry I got confused and forgot that the PR changed from updating the exports to only updating typings—all good then!


Update version from 3.1 -> 3.2

Thanks! This was already taken care of in #274 so I'm going to go ahead and close this out.


Add support for MSVC

Fixes #59

I tested this building glib-sys and it seems to work properly.

Looks good to me and should work, at least better than what we have currently. As @nirbheek said in #59, --msvc-syntax had some issues in the past in some corner cases but we can worry about that when it happens.

@alexcrichton Thanks! Can you prepare a new release with that change?

Sure thing, done!


Error: connect is not a function

I setup a database at MongoLab, and I can connect to it via the CLI using mongo just fine, so I know the credentials are just fine, but when I try to connect using mongojs, I get an error:

TypeError: require(...).connect is not a function

The script itself is about as basic as it gets, it's only got one line of JS:
var db =
require('mongojs').connect('my_user:secret@ds047365.mongolab.com:47365/my_database')

Admittedly, this is my first attempt at working with MongoDB, but this is copied straight out of a few examples Ive found online. Am I doing something wrong?
Heres the full error/stacktrace
$ node connect.js
/Users/me/Documents/scripts/js/node/test/mongodb/test_connection/connect.js:3
var db = require('mongojs').connect('my_user:secret@ds047365.mongolab.com:47365/my_database');
                            ^

TypeError: require(...).connect is not a function
    at Object.<anonymous> (/Users/me/Documents/scripts/js/node/test/mongodb/test_connection/connect.js:3:29)
    at Module._compile (module.js:399:26)
    at Object.Module._extensions..js (module.js:406:10)
    at Module.load (module.js:345:32)
    at Function.Module._load (module.js:302:12)
    at Function.Module.runMain (module.js:431:10)
    at startup (node.js:141:18)
    at node.js:977:3

P.S.: I had a hard time installing mongodb via npm, as it was throwing errors saying I didnt have kerberos installed, and kerberos was failing due to something related with xcode, (same error as in this thread). Eventually I just installed mongodb via brew... Would mongodb being installed via brew instead of npm be the problem? I see mongodb is a dependency of mongojs..

mongojs does not expose a connect function anymore (since 1.x release). Simply use
var db =
require('mongojs')(URL)
To connect.

Ah! ok, that makes sense...
var databaseUrl = "my_user:secret@ds047365.mongolab.com:47365/my_database";
var mongojs = require('mongojs')
var db = mongojs(databaseUrl);
var test = db.collection('test')

test.find(function (err, docs) {
    if(err)throw new Error(err)
    console.log('DOCS',docs)
})

Perfect, thank you!


[TS] Return unsub() from on()

This is another take on the proposed behavior from #42, where on() returns a function that can be called to remove the added listener.
My concerns remain, however, and I don't think this should be landed. Any implementation that provides this convenience makes it easier to create memory leaks, because there are now two separate owners of a given handler reference - Mitt itself, and the app code that added the listener.

@developit i think the feature would find itself very convenient if you consider:
1. wrapping many events in some qualified functions:
function someSpecificFunction() {
  const handler = () => console.log(arguments)

  const unsubs = ['event1', 'event2', 'event3'].map(ev => {
    mitt.on(ev, handler)
    return () => mitt.off(ev, handler)
  })

  return () => unsubs.forEach(fn => fn())
}

which, with this PR could be made more concise like:
function someSpecificFunction() {
  const handler = () => console.log(arguments)

  const unsubs = [
    mitt.on('event1', handler),
    mitt.on('event2', handler),
    mitt.on('event3', handler),
  ]

  return () => unsubs.forEach(fn => fn())
}

2. use in modern frontend libraries such as React:
A typical use could be:
useEffect(() => {
  const listener = mitt.on('event', setStateSomething)
  return listener
}, [someDep])

which, with this PR could be made more concise like:
useEffect(
  () => mitt.on('event', setStateSomething),
  [someDep],
)

In the end people could just also do:
const on = mitt.on
mitt.on = (ev, fn) => on(ev, fn) ?? () => mitt.off(ev, fn)

...but it's such a common use nowadays that it just doesn't make sense not to have it. (i personally look forward to this PR to start using mitt into my company's sdk)

@developit hi. this looks what I am looking for. are you still not considering merging it?


Support Checker Framework for nullability verification

https://checkerframework.org/

Maybe find alternative solution that's easier to use?


Signal SIGXCPU on Linux when I build it myself

Describe the bug
After compiling the program myself, I get a SIGXCPU signal (CPU time limit exceeded) whenever I try to play a track. The program then crashes. This did not happen when I used a prebuilt binary.
To Reproduce
I presume this does not happen on your device, which makes it rather difficult to reproduce. I simply ran:
sudo apt-get install libssl-dev libgtk-3-dev libcairo2-dev libasound2-dev
cargo build
cargo run --bin psst-gui

Of course that makes the process seem a bit cleaner than it really was since at first I missed out on the apt-get install step. Nevertheless I don't believe I did anything particularly out of the ordinary, and after properly ensuring all packages are installed (only libssl-dev wasn't already installed) a cargo clean && cargo run --bin psst-gui yields the same results.
Environment

OS: Linux Mint 21.2 Cinnamon (with kernel 5.15.0-122-generic)

This issue isn't very important to me by the way, but I thought it was good to make record of it.

Heya, I'm pretty sure that this happens as a protection for your computer not to crash, so only the program crashes. This is because you are only building the "developer" build, add --release after your command, then it will remove this issue ;)
So try: cargo run --bin psst-gui --release

This is still a little bit weird that this happens. I don't know exactly what the resolution here is.

Ah ok so I have been looking through documentation, I think the way to fix this is through checking your current CPU time limit:
ulimit -t
Then you can set it to unlimited though this:
ulimit -t unlimited
This may be wrong I haven't tested it yet.

No this isn't the answer, mine is already set to unlimited... wierd!

I tried a couple of things just now:

Release builds work just fine for me. No problems at all.
All of the following commands give "unlimited" for me:

ulimit -R
ulimit -SR
ulimit -HR
ulimit -t
ulimit -St
ulimit -Ht

I also injected the following bit of code into psst-gui (specifically route_widget()):

let realtime_limit = rlimit::getrlimit(Resource::RTTIME).unwrap();
let cputime_limit = rlimit::getrlimit(Resource::CPU).unwrap();
println!("Realtime: {realtime_limit:?}");
println!("CPU time: {cputime_limit:?}");

In a debug build this gives:
Realtime:<PHONE_NUMBER>3709551615,<PHONE_NUMBER>3709551615)
CPU time:<PHONE_NUMBER>3709551615,<PHONE_NUMBER>3709551615)

Where<PHONE_NUMBER>3709551615 is 2**64 - 1 (presumably unlimited).
I honestly don't think this bug is viable to fix without a kernel developer butting in. If you want to close this as wontfix that would be fine by me.

Thank you so much for your help!

@jacksongoode should we close this now, I do want to note that building with these optimisations does make it take a bit longer to build.

I think it's ok to close, if this needs to be reopened if someone wants to figure out more optimal dev settings I believe that's fine.


Use from_bits_retain for all Termios wrapper fields.

What does this PR do
In case there are any unrecognised bits, they should be kept when setting fields of the underlying libc termios struct in get_libc_termios, rather than being dropped. This ensures that termios can be roundtripped even with unrecognised bits set.
Checklist:

[x] I have read CONTRIBUTING.md
[x] I have written necessary tests and rustdoc comments
[x] A change log has been added if this PR modifies nix's API

Would this PR fix #2071 ?

I think so, can you test it?

Would this PR fix #2071 ?

I think so, can you test it?

No, I can't.  I don't have an Illumos system.  Maybe @nospam3089 can test it?


adding the data loader function and the data load function test noteb…

…ook. there was also a very minor change to the sample_info_data_visualizer notebook. All edited notebooks have had their changes converted to their appropriate modules nbconverted folder

hey @gwaybio I believe that this is ready for review!


Creating docker image from appveyor

appveyor support to launch docker build
changing LICENSE to WTFPL
adding readme with appveyor badge.
creating Dockerfile installing vcbuildtools

Closing for a bug in Dockerfile


[Feature Request] Auto cancel activities after they timeout

Currently Core relies on the server to detect when an activity times out, requiring activity implementations to heartbeat in order to get cancelled.
Core should copy the behavior from the Go SDK and automatically cancel activities that exceed their schedule-to-close or start-to-close timeouts.
Since Core decouples cancellation and heartbeats, this change would improve the DX for all Core based SDKs, especially for languages with built-in cancellation mechanisms such as Python and .NET (Python even more since cancellation is implicit).

automatically cancel activities

To confirm - "cancel" from a lang POV but do not return an activity task completion so we don't race against server timeout in some way.
Also this assumes we'll won't allow activity options to be altered server side (or at least requires that we come back and change this if we do).

Yes, cancel from a lang POV, I agree this requires clarification.
I don’t see a good reason to report back to the server.
Also good point about changing activity options, we can potentially address that when we add the capability.


Registry Dirctory Metrics should interface level

What is the purpose of the change
interface level metrics of Dirctory num
Verifying this change

Codecov Report

Merging #12012 (7c22f97) into 3.2 (d974a54) will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                3.2   #12012      +/-   ##
============================================
+ Coverage     69.16%   69.19%   +0.03%
  Complexity        2        2
============================================
  Files          1559     1559
  Lines         64333    64344      +11
  Branches       9385     9386       +1
============================================
+ Hits          44495    44523      +28
+ Misses        15553    15538      -15
+ Partials       4285     4283       -2

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

Please fix conflicts

Please fix conflicts

done


[Suggestion][Minor] Change window opening mechanic in System Tray

Hey Matthew!  Great project, just found it and will definitely be using it for all my video recording/screenshot taking needs.
When I started messing around with Captura's settings, I tested out the Minimize to System Tray on Startup option and it worked great, but it took me a little while to figure out how to open the window again.  Right now you have it set to open the window when it is double clicked, but I think that's very strange and unintuitive for a tray icon.  I'm used to right clicking on the tray icon for options and left clicking once to open up the window.  It might be different for others, but I don't think I've ever opened a window by double clicking the tray icon.  So, I suggest making it like other applications where a left click opens the window and a right click shows the options.
Thanks for all your work!  Feel free to disregard and close this issue if you disagree.

In all the apps I've used, double clicking on the tray icon is the way to show the main window. I suppose I will leave it as it is.
How about a Show Main Window option in the context menu?

Yeah I don't know, maybe they are both used.  Here's me pressing all of the icons in my System Tray.  They're all one left click to open.
That would be great, ty!

Oh Giphy cut my video short actually.

I'm adding the option for user to select what will happen on left click on tray icon.
It will be set by default to open main window.

Please test dev build: https://ci.appveyor.com/api/buildjobs/kty9dy4umetfwwa5/artifacts/Captura-Portable.zip
Left click action is set to Show Main Window. Go to Settings / Configure Tray Icon ... if you want to customize.

Works perfectly!  Thanks!


IBMQ account adminstration is a bit unclear

What is the expected enhancement?
This is how things are done now:
use_account()   # add to current session
add_account()  # save to disk
remove_account()  # remove single account from current session AND disk
remove_accounts() # remove all accounts from session AND disk
load_accounts() # load from disk into session, potentially filtering by hub/group/project

I see the following problems:
1- I think save_account instead of add_account is more appropriate for saving to disk
2- remove_accounts() and remove_account() should be merged. I think we can just have remove_accounts(), which can take a single credentials or a filter on hub/group/project. Leaving empty removes all accounts. This would mirror load_accounts().
3- I think we need separate functions for add/remove in session and save/delete in disk.
So how about add_account/remove_account only targeting the session, and save_account/delete_account targeting disk. This way there's more control, and the function purposes are more clear.
So I propose the following:
add_account(token, url) # add to current session
save_account(token, url) #save to disk
remove_accounts(**kwargs) #remove from current session
delete_accounts(**kwargs) #delete from disk
load_accounts(**kwargs) #load into sessions

In the above, 'token', 'url', 'hub', 'group', 'project' can be part of **kwargs filtering. The big advantage is that you don't need to supply long tokens and urls (e.g. can just specify a certain hub as shorthand). This is the real advantage over using Qconfig I think, otherwise it would be similar.

@ajavadia your killing me. I have to say I agree with you but lets put more fire on the qiskit 0.6 update.
Are you sold on add vs use as that is all I used in the example? We could make the others and this would not break anything except documentation.
Saying this i think it is clear i need to make a backend example.
so would
use_account(token, url) # add to current session
save_account(token, url) #save to disk
disable_account(token, url)  #remove from current session
delete_accounts(**kwargs) #delete from disk
load_accounts(**kwargs) #load into sessions

The disable just does one like the use and delete is for the harddrive.

^ I concur. Will work towards this.

great. this breaks less and then i can make an example after highlighting the admin

Slight changes to the above after discussions:
enable_account(token, url) # add to current session
disable_accounts(**kwargs)  #remove from current session
save_account(token, url) #save to disk
delete_accounts(**kwargs) #delete from disk
load_accounts(**kwargs) #load into session
active_accounts() # list all accounts active in this session
stored_accounts() # list all accounts saved to disk

Waiting for #1000 to be merged.


Priority Features

[x] #923
[ ] #926
[ ] #927
[ ] #928
[ ] #940
[ ] #941
[ ] #942
[ ] #943

This summary ticket has been moved to Zenhub as an Epic. The work still needs to be done, but this issue can be closed.


Issue #1666 configure metering to also snapshot records from bucky

Connected to https://github.com/SixSq/tasklist/issues/1666

LGTM.


Base editorconfig + format

Description
This is my editorconfig from work, it's far from perfect, it's a little crooked in places.
But it has passed the test of time, we have been using it for about 2 years in combination with the SQ analyzer, I think it will be enough to start with, but it is specific in places.
@VisualBean , do you agree, or will we write a new editorconfig?
Related issue(s)
#202

Most people seem to swear by this one
https://github.com/dotnet/roslyn/blob/main/.editorconfig
Made by the Roslyn team (for Roslyn itself).
Personally I am indifferent, although the Roslyn one is pretty widespread, so might sit better with the majority.

Most people seem to swear by this one
https://github.com/dotnet/roslyn/blob/main/.editorconfig
Made by the Roslyn team (for Roslyn itself).
Personally I am indifferent, although the Roslyn one is pretty widespread, so might sit better with the majority.

It's made on the basis of what you threw off, let's take it, then I'll redo it now and send a new pull request


Global Update Feature for npm Dependencies in Turbo Repo

Which project is this feature idea for?
Turborepo
Describe the feature you'd like to request
Currently, in Turbo Repo, when an npm dependency package gets an update, developers must manually update it in each app or package using the pnpm update command. I propose a feature that allows for global updating of npm dependencies across all apps and packages. This would streamline the update process, saving time and reducing the risk of inconsistencies.
Describe the solution you'd like
I would like to see the implementation of a command or configuration option within Turbo Repo that allows developers to update npm dependencies across all apps and packages simultaneously. This could be a new command such as turbo update or an extension of the existing update functionality with a global option.
Describe alternatives you've considered
An alternative solution could be to create a script that automates the current process of individually updating dependencies in each app or package. However, integrating this feature directly into Turbo Repo would provide a more robust and user-friendly solution, aligning with the tool's goal of simplifying development workflows.

For anyone else struggling with the similar issue as mine and wants to update npm dependencies in every app or package of their turbo repo can use this.

Hi @vaibhavmuchandi, thanks for the issue. We've been discussing some more integrations with package managers, so we can definitely add this as a potential feature.

Doesn't syncpack do something similar or am I  misunderstanding what you want to do?
https://jamiemason.github.io/syncpack/

There is syncpack update (https://jamiemason.github.io/syncpack/update).
Examples

Update react-related packages only: syncpack update -f react
Update production dependencies only syncpack update --types prod

Or run with no options to update everything in the whole monorepo, although this would include peerDependencies, overrides/resolutions, etc

Yes, it's pretty awesome too!
Lazy me sometimes wishes it would update versions or add overrides to resolve audit/security vulnerabilities :D

Thanks! There is npm audit --fix but are you looking for something a little different? That but monorepo-wide and/or changes your package.json instead of your lockfile?

We definitely recommend syncpack (in fact we're going to have to use it on this repo soon enough) - but I'm sure there a few tools that solve the same issue out there!

Thanks! There is npm audit --fix but are you looking for something a little different? That but monorepo-wide and/or changes your package.json instead of your lockfile?

Yeah, mainly to change in the package.json instead of lock file. That communicates better and avoids accidentally reverting fixes when not re-running npm audit --fix. Mainly just being lazy to be honest :)

We definitely recommend syncpack (in fact we're going to have to use it on this repo soon enough) - but I'm sure there a few tools that solve the same issue out there!
– @tknickman

Would you be open to me raising a PR to set up syncpack in this repo? Would be a good opportunity for me to use it as a living example to create an onboarding tutorial at the same time. No pressure, just an idea 👍


Refactor the README file once we have a website for the project

Some contents of the README file can be moved to either the wiki or the Portus website (pending to be done). Some examples:

The screenshot of Portus.
Notes on the development environment.
Release
etc.

Done


Update go version to 1.11 in circle tests.

New image with go1.11 had been pushed. This is also done to master in
https://github.com/istio/istio/pull/10186 to resolve some linter
problems.
Also we had recently updated the go version to 1.11 in the dev guide.
https://github.com/istio/istio/wiki/Dev-Guide

@kyessenov
go/fmt problem was due to behavior change across go versions, which this PR is supposed to fix. =)
I have addressed all of them in this PR too, and the lint job is now passing.

/retest

@hklai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
524f4ee4938ac2b8996a52713338625ec153be70
link
/test e2e-simpleTestsMinProfile

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm
㊗

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hklai, kyessenov
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

.circleci/OWNERS [hklai]
bin/OWNERS [hklai]
galley/OWNERS [hklai]
istioctl/OWNERS [hklai]
mixer/OWNERS [hklai,kyessenov]
pilot/OWNERS [hklai,kyessenov]
pkg/OWNERS [hklai]
security/OWNERS [hklai]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
524f4ee4938ac2b8996a52713338625ec153be70
link
/test e2e-simpleTestsMinProfile

prow/e2e-mixer-no_auth-mcp.sh
524f4ee4938ac2b8996a52713338625ec153be70
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Discrepancy between Amplify CLI deployment using local push and Amplify CI/CD deployment for serverless container api

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v18.18.0
Amplify CLI Version
12.10.1
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
no
Describe the bug
Resource access permissions not updated for container based server less REST API when deploying it through Amplify CI/CD but permissions updated when deploying through amplify push from local system.
Expected behavior
Resource access permissions should be updated when deploying  container based server less REST API using Amplify CI/CD
Reproduction steps

amplify init
enable container based deployment using  amplify configure project
amplify add api (Add container based serverless api)
amplify add storage
amplify add function
amplify update api
Do you want to access other resources in this project from your api? Yes
? Select the categories you want this function to have access to.

◉ function
◉ storage
❯◉ testfunction
◉ testfunction2
◉ restserverlessapia6c7abf2
❯◉ create
◉ read
◉ update
◉ delete
You can access the following resource attributes as environment variables from your Lambda function
FUNCTION_RESTSERVERLESSAPIA6C7ABF2_NAME
FUNCTION_TESTFUNCTION2_NAME
FUNCTION_TESTFUNCTION_NAME
STORAGE_TEST6_ARN
STORAGE_TEST6_NAME
STORAGE_TEST6_STREAMARN
STORAGE_USERS2_ARN
STORAGE_USERS2_NAME
STORAGE_USERS2_STREAMARN
STORAGE_USERS_ARN
STORAGE_USERS_NAME
STORAGE_USERS_STREAMARN
permissions updated in "containerapi-cloudformation-template.json" file.
amplify push -y
permissions added in "TaskDefinitionTaskRoleDefaultPolicy282E8624"
update in GitHub or Code Commit repo and trigger Amplify build
permission lost in "TaskDefinitionTaskRoleDefaultPolicy282E8624"
There is no any permission lost issue when we do deployment from local system using amplify push but permissions lost when trigger Amplify build using CI/CD
Project Identifier
No response
Log output

# Put your logs below this line

Additional information
Workaround : custom-policies.json
Tested deployment using CI/CD after adding permissions in custom-policies.json file and it's added to TaskDefinitionTaskRoleFD4-idoWvD9uZrLv with customer inline policy name CustomExecutionPolicyForContainer.
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey,👋 thanks for raising this! I'm going to transfer this over to our Amplify API repository for better assistance 🙂


Svelte landing page

@all-contributors add @Spikeysanju for code

@all-contributors add @abhishekmishragithub for reviewing pull request

@all-contributors add @abhishekmishragithub for review

@all-contributors add @Spikeysanju for infra


How can i transfer drone_branch variable into sonar-project.properties?

i want to transfer DRONE environment variable to specify some of sonar properties. How can i do this ??

use environment to set variable,  and write ${env.xxxx}  in your properties file to use the environment variable.


Add reference() to allow access to global objects in run() even if minified

Doesn't work in all environments.  Use eval() or comment to bypass minification


Add icon "Copy" to TX details

https://zpl.io/2p3Br3l
Add "Copy and close" button in tx details popup (implement after that task https://github.com/BeamMW/beam-ui/issues/590)

Merged with https://github.com/BeamMW/beam-ui/issues/590


How to listen to changes to the database?

I have implemented what you did in the RealtimeStreaming example, but the function is never invoked. Why is that? Is this the method used to listen to changes in the database?

Did you ever manage to figure out what was wrong? I'm running into the same issue.


net core 3.0 configurations

Fixes #
Proposed Changes

Testing

 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.:x: tedesse sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@tedesse thanks for submitting a PR.  we appreciate the help.  Unfortunately I'm not going to take this PR.  We've moved our .NET Core dependency, in the samples, to 3.1.x.  This is the current recommended version from the .NET Core team.  the 3.0 release has been tagged end of life.


Merge Tender, Transaction, and Position Changes for March 2024 Standard with Master

Solidifying progress in Spring 2024 Semester Capstone Project.

Forward version in Master, duplicate in dev-java21.


Fixes for title and contents highlighting

Adds a grammarly accurated title and highlights SQL commands used in article contents. Also fixes some minor grammar errors

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for your contribution, but the guide has been moved here https://github.com/freeCodeCamp/freeCodeCamp/tree/master/guide and this cannot be merged. If you create a new PR targeting the new location we will do our best to get it merged.


Part-Coordinated Transformer train

Sorry for asking so many questions.
CUDA_VISIBLE_DEVICES=0 python train_ParCo_trans.py
--vqvae-train-dir output/00000-t2m-ParCo/VQVAE-ParCo-t2m-default/
--select-vqvae-ckpt last
--exp-name ParCo
--pkeep 0.4
--batch-size 128
--trans-cfg default
--fuse-ver V1_3
--alpha 1.0
--num-layers 14
--embed-dim-gpt 1024
--nb-code 512
--n-head-gpt 16
--block-size 51
--ff-rate 4
--drop-out-rate 0.1
--total-iter 300000
--eval-iter 10000
--lr-scheduler 150000
--lr 0.0001
--dataname t2m
--down-t 2
--depth 3
--quantizer ema_reset
--dilation-growth-rate 3
--vq-act relu
You chose "last" in "select-vqvae-ckpt", but why did you choose "last"? Is there a reason why you didn't train by selecting the best fid values?
Thank you for your kind reply every time.

Of course, the performance was reproduced well even when it was done "last".

We follow T2M-GPT choosing "last" VQ-VAE ckpt for training Part-Coordinated Transformer.
And for testing Part-Coordinated Transformer, we choose "fid" for HumanML3D and "last" for KIT dataset. Because the results on validation set seem to appear more stable when we choose in that way.


Add support for '<millis/milliseconds/micros/microseconds/nanos/nanoseconds>SinceEpcoh' time unit string

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@5344efb). Click here to learn what that means.
The diff coverage is 75%.

@@            Coverage Diff            @@
##             master    #4092   +/-   ##
=========================================
  Coverage          ?   66.78%
  Complexity        ?       20
=========================================
  Files             ?     1037
  Lines             ?    51422
  Branches          ?     7190
=========================================
  Hits              ?    34340
  Misses            ?    14718
  Partials          ?     2364

Impacted Files
Coverage Δ
Complexity Δ

.../org/apache/pinot/common/utils/time/TimeUtils.java
87.87% <75%> (ø)
0 <0> (?)

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 5344efb...873e318. Read the comment docs.

@snleee For completeness in case user wants to use it.


Multiple enhancements

The directive now has a scope variable that is the current item in the photoswipe gallery.  That lets me do angular bindings in my photoswipe template and thus display whatever properties I want to include on my slides
The directive has a two way bind to a variable that lets you open/close the photoswipe by changing the variable in the parent controller.  This is currently used to let me open the gallery immediately when my slides load without needing any open button in my photoswipe template.
The scope now has an on closed delegate so that you can let the parent controller respond to when the photoswipe gallery closes.  I use that so I can do some ui routing updates when the gallery closes.
The scope now has an extensions property.  This is really nice because I can now add functions and event handlers in the parent controller, and wire events up in my photoswipe template.  So my gallery controller has an extensions object that has functions and properties that I can bind to in my photoswipe template.
The combined result lets me have something like the following in my template: {{item.title}} {{item.myCustomDetails}}
I think that you added startOn which behaves similarly to what I did with open on the scope.  The difference being that if you have a bound variable you watch, you can start in response to a later event such as data loading, or the user clicking a button somewhere outside your photoswipe template.

@DannyStaten Thanks, I like your approach.
I'll take a look more in deep to the code to make the demo working again, hope to merge it soon!

I am glad you liked the updates.  I was contemplating writing a photoswipe directive, but decided to google it first which led me to your directive.  So thank you for saving me all that initial work to create the directive.  This is the first time I have contributed to anything on GitHub actually.  You can see my use of it at www.danstaten.com.

@DannyStaten Yeah, I like it.
Keep contributing and submitting your code, it is more than appreciated!


Fix config entry has options check

Description:
When I tweaked the flow options, I added a fallback method that raised a method. However, that caused our supports_options check to be updated, and that ended up not working with partials, which is what webhook flow handlers generate.
This fixes the check.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

I reworked it to no longer use partials.


Show TextureProgressBar radial center cross only when editing the scene

Closes https://github.com/godotengine/godot/issues/46266 Closes https://github.com/godotengine/godot/issues/96859 for real. Nothing more.
Special thanks to @Mickeon

Thanks!


app.info.js is incorrectly detected as having Chrome Apps APIs

This results in TODOs being added to app.info.js.

Resolved in #26.

Resolved in f051c7896d405e36d6dc62d038d24936cb5fec7a.


About continuous_dims

continuous_dims = {
"layer4.0.conv1.weight": [0, 1],
"layer4.0.conv2.weight": [0, 1],
"layer4.0.downsample.0.weight": [0, 1],
"layer4.1.conv1.weight": [0, 1],
"layer4.1.conv2.weight": [0, 1],
}
Above is the definition of continuous_dim in imagnet.py .
The question is, what are the criteria for determining continuous_dim. Described as continuous_dim performing the crop. If so, do I have to choose all the parts that correspond to the last layer before entering the fully connected layer?
Additionally, in the example above, that's how it works
import torch
Bringing a torch to an inn
Get .reset back from torchvision 18
Model = resnet18 (pre-training = true)
Wrapping paper = Inn.All-in-one wrapper (init_from_discrete=True)
Specify continuous dimensions to cut
Continuous_dims = {
"Tier 4.0.conv1.weight": [0],
"Tier 4.1.conv1"Weight": [0, 1]
}
Converting to an integrated model
in_model = rapper (model, (1, 3, 224, 224), continuous_dims)
The choice of continuous_dims is slightly different here. I would appreciate it if you could tell me the difference between these two as well.

Hello @sangfeel ! Please start discussion instead of starting Issues in similar cases.

continuous_dims = { "layer4.0.conv1.weight": [0, 1], "layer4.0.conv2.weight": [0, 1], "layer4.0.downsample.0.weight": [0, 1], "layer4.1.conv1.weight": [0, 1], "layer4.1.conv2.weight": [0, 1], }
Above is the definition of continuous_dim in imagnet.py .
The question is, what are the criteria for determining continuous_dim. Described as continuous_dim performing the crop. If so, do I have to choose all the parts that correspond to the last layer before entering the fully connected layer?
Additionally, in the example above, that's how it works import torch Bringing a torch to an inn Get .reset back from torchvision 18
Model = resnet18 (pre-training = true) Wrapping paper = Inn.All-in-one wrapper (init_from_discrete=True)
Specify continuous dimensions to cut Continuous_dims = { "Tier 4.0.conv1.weight": [0], "Tier 4.1.conv1"Weight": [0, 1] }
Converting to an integrated model in_model = rapper (model, (1, 3, 224, 224), continuous_dims)
The choice of continuous_dims is slightly different here. I would appreciate it if you could tell me the difference between these two as well.

Hello @sangfeel ! Please start discussion instead of starting Issues in similar cases.

Sorry. This activity is my first github. I start discussion.


Update the infra/core modules to AVM modules

Update the infra/core modules to AVM modules for todo-csharp-cosmos-sql.
1. Modules not replaced by AVM.

Monitoring (applicationinsights-dashboard): Missing the feature Microsoft.Portal/dashboards.
AVM issue link: https://github.com/Azure/bicep-registry-modules/issues/1130
APIM: Missing features Microsoft.ApiManagement/service/loggers and Microsoft.ApiManagement/service/apis/diagnostics.
AVM issue link: https://github.com/Azure/bicep-registry-modules/issues/1124
Cosmos SQL: Missing features Microsoft.DocumentDB/databaseAccounts/sqlRoleDefinitions and Microsoft.DocumentDB/databaseAccounts/sqlRoleAssignments and missing outputs of roleDefinition.outputs.id and cosmos.properties.documentEndpoint.
AVM issue link: https://github.com/Azure/bicep-registry-modules/issues/1126

2. Modules replaced by AVM.

App Service
Key Vault
App Service Plan

@jongio for notification.

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
Project: todo-csharp-cosmos-sql
Remote: azure-samples-staging
Branch: pr/3802
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-cosmos-sql -b pr/3802

View Changes | Compare Changes

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
Project: todo-csharp-cosmos-sql
Remote: azure-samples-staging
Branch: pr/3802
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-cosmos-sql -b pr/3802

View Changes | Compare Changes


[Language]Creating Dummy Data In Python Using Faker Package

Topic Suggestion
Creating Dummy Data In Python Using Faker Package
Proposed article introduction
It's important to have fake data for a variety of reasons. It can be difficult to find the data you need in a specific format. The Faker package in Python is used to create dummy data, which is the subject of this article.
Dummy data is a useful tool for testing and evaluating software and hardware. You can use it to see how your code responds to various inputs after you've finished developing it. Faker, a Python package, can be used to create dummy data. An open-source library that generates dummy data of various types is available here.
Key takeaways

Why do we need dummy data
Installing the Faker Package
How to create and initialize a Faker Generato
Providers
Command-line usage of Faker Package

Article quality
The reader will discover how to use Python's Faker library to generate various forms of data. The reader investigates how to generate names, biographical profiles, and currency-related data. The reader will also learn how to replicate the same dummy data and how to generate unique data. We will also look at the providers and discover that it is possible to create data that is specific to a specific location.
References
N/A
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Please fill in the Article Quality section so we can review your article.

Done
@louisefindlay23

Hi. Your article quality section is more about what you're writing about rather than what makes your article unique? Can you tell me what aspects of your article are not covered elsewhere or your unique perspective that you plan to bring?

@louisefindlay23
What distinguishes my article from others is that I will show how to reproduce the same dummy data as well as how to generate the unique data. I will also deeply explore the providers and create locale-specific data. Thanks.


Embedding a sequence without finetuning

Hi,
I'm working on bio_embeddings (poster, talk), a tool that embeds sequences using various methods, and we'd like to include PLUS-RNN as an embedding method.
For bio_embeddings, we generate embeddings from sequence without fine-tuning. Do you have an example of how to turn a sequence of amino acids into a sequence of embeddings with PLUS-RNN? Unfortunately I could only find examples that involve a Trainer.
I'm also working on making PLUS as publishable python package, you can see the current state here.

@konstin, I apologize for the delayed response.
I have updated the codes to support generating protein sequence embeddings without fine-tuning.
You can now use the plus_embedding.py file to turn protein sequences into sequences of embeddings with PLUS-RNN.
Given a FASTA file with protein sequences, the script generates and saves the PLUS-RNN representations as NumPy arrays.
For example, you can use the below command, assuming you have downloaded the pre-trained models.

python plus_embedding.py --data-config config/data/embedding.json --model-config config/model/plus-rnn_large.json --run-config config/run/embedding.json --pretrained-model pretrained_models/PLUS-RNN_LARGE.pt --device 4 --output-path results/plus-large

If you have any other questions or problems, please let me know!


How can I use this extension?

I installed this extension on Automation1111, entered the API key in the settings and that's all.
This extension doesn't show me the tab.
I need instructions on how I can use this extension out of the box.
Help me please.

Same here, i notice it download preview images when launching A1111 for anything i have manually added. Beyond that none of the other functionality appears to be present?

shitty extension don't use it


ARROW-8388: [C++][CI] Ensure Arrow compiles with GCC 4.8

Python failure is unrelated: https://github.com/apache/arrow/pull/6894/checks?check_run_id=577211623#step:6:2480

Unrelated python error: https://github.com/apache/arrow/pull/6894/checks?check_run_id=577557693#step:6:2171

Thanks @bkietz! This particular issue is very tedious. Will gcc 4.8 barf on any return of a unique_ptr or only temperamentally?

@wesm the issue is implicit move on return: 4.8 will only do this if the return type exactly matches the returned expression:
#include <memory>

struct B {};
struct D : B {};

std::unique_ptr<B> works() {
  std::unique_ptr<B> up;
  return up;
}

std::unique_ptr<B> broken() {
  std::unique_ptr<D> up;
  return up; // should work; unique_ptr<B> is implicitly constructible from unique_ptr<D>&&
}

std::unique_ptr<B> workaround() {
  std::unique_ptr<D> up;
  return std::move(up);
}

This failure extends to Result<unique_ptr<B>>, which is implicitly constructible from unique_ptr<D>

Ah good to know, so it's not quite that bad, though still annoying.


JWT Access token does not contain requested claims when using SAML2 Bearer Grant

Describe the issue:
When using SAML2 Bearer Grant, the access token obtained does not contain the requested claims. But the ID token that is subsequently returned contains all the claims. This is because the user is being treated as a federated user when using the SAML2 bearer grant, and claims for federated users are expected to be in the cache when the token is being created [1]. After the access token is created, the claims are added to the cache, hence they are available for the ID token when it is being built.
This is not consistent with other OAuth/OIDC grants as they return all the requested claims even in the access token. Therefore, this disparity needs to be fixed.
How to reproduce:

Enable SAML2 bearer grant in travelocity app.
Register SPs for SAML (Travelocity) and OAuth as a client for the bearer grant.
Add requested claims to the Claim Configuration of the SPs.
Login to travelocity and request OAuth2 Access token.
Decode Access token and observe claims. Compare with claims returned in the SAML response as well.

Expected behavior:
All requested claims should be available in the access token.
[1] https://github.com/wso2-extensions/identity-inbound-auth-oauth/blob/f256fde3c79404704dc829ccc8a134174898641f/components/org.wso2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/openidconnect/DefaultOIDCClaimsCallbackHandler.java#L142

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


[Build] Build fails on Android Studio Electric Eel 2022.1.1

Build #AI-221.6008.13.2211.9477386, built on January 11, 2023
Error:
.../google/ground-android/ground/build/generated/source/kapt/debug/com/google/android/ground/databinding/DataCollectionFragBindingImpl.java:195: error: incompatible types: void cannot be converted to Single<String>
            viewModelOnContinueClicked = viewModel.onContinueClicked();
                                                                    ^

@JSunde PTAL?

This should be resolved after https://github.com/google/ground-android/pull/1435. Please sync to head and try again


Why not configure docker usr/pwd in travis

Seems we are configuring it in makefile at https://github.com/IBM/go-repo-template/blob/master/Makefile#L135-L136
/cc @morvencao

Closing this as we have switched to build multi-arch image with qemu in one platform, see: https://github.com/IBM/go-repo-template/pull/53
To make Travis to be another option, we keep the travis code in another branch: br_use_travis_to_build_multi-arch_image: https://github.com/IBM/go-repo-template/tree/br_use_travis_to_build_multi-arch_image
/close


[BUG] Changing devices with only one device assigned clears default device state

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When adding a facility, an initial device is assigned and set as default. If the user changes that device, it's no longer set as default, even though it's the only device assigned. As a quality of life improvement, if there's only one assigned device, it should always be set as the default.
Note: the user does receive an error message explaining the issue.
Steps To Reproduce
As an org admin

Go to manage facilities page
Add a facility
Fill form with valid data
Go to "Manage Devices" section
Change the assigned default device

You should now see the assigned device's "Set as default" checkbox as unchecked
Expected Behavior
The frontend should not clear the "Set as default" checkbox (and internal default device state) if the device changed is the only one assigned to the facility.
Additional context
No response
Does this issue have a priority label?

[X] I have added a priority label

Has issue been assigned to a team?

[X] I have added a team label

closing this since we dont have the concept of default devices anymore


Updates table.sort description to be clearer

Changes
Updates table.sort description to be clearer about how to create a valid comparison function. This question comes up often enough I think it is worth updating the documentation.

Checks
By submitting your pull request for review, you agree to the following:

[x] This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
[x] I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
[x] To the best of my knowledge, all proposed changes are accurate.

Looks good to me, thanks a bunch @dogwarrior24!


Add PKS mgmt plane node to reserved IPs list

Customer placed BOSH in <IP_ADDRESS>/24 and could not deploy clusters. PKS mgmt plane subnet MUST not use <IP_ADDRESS>/16 subnet.
Also applies to 1.2 and 1.4/master.

Thanks @lparis!


Unit test for TaskListQueryHandler

@mgmccarthy : This is a simple PR to get a feel for your expectations regarding unit tests.

#870

@benjaminettori I'll probably get to the code review Sunday night.

@benjaminettori, you have to rebase this PR b/c of the RTM update. For instructions on how to rebase for AllReady, check out this blog post: http://stevejgordon.co.uk/contributing-to-allready


Microsoft RSA Root Certificate Authority 2017 - Discrepancy between certificate thumbprint

The Microsoft RSA Root Certificate Authority 2017 certificate available on the download link https://www.microsoft.com/pkiops/certs/archived/Microsoft RSA Root Certificate Authority 2017.crt does have a different thumbprint then what is written on the page ee68c3e94ab5d55eb9395116424e25b0cadd9009 vs 1ed397095fd8b4b347701eaabe7f45b3. The real certificate served by your server is the one with thumbprint 1ed397095fd8b4b347701eaabe7f45b3. We had to download it manually to get our connection to work.
This is a problem only on this certificate, other ones are fine.
Thanks,
M.

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

ID: 2f62b7b9-f841-b6b2-4190-b0a589b41972
Version Independent ID: 4b6a351b-35e6-8865-c622-94e82d98d9a7
Content: Azure Certificate Authority details
Content Source: articles/security/fundamentals/azure-CA-details.md
Service: security
Sub-service: security-fundamentals
GitHub Login: @shlipsey3
Microsoft Alias: sarahlipsey

Thanks for letting us know. Apologies for the confusion. I should be able to resolve that error today. Thanks!

@miroslav-staffa
Thanks for your feedback! We will investigate and update as appropriate.
@shlipsey3 Thank for your quick responses.

Fixed the incorrect thumbprint and serial number.
#please-close


hijacked connection

Getting error on the server side.
Using httprouter.
It works correctly with my other client implementation.
Using ws-machine, I got
http: response.WriteHeader on hijacked connection
http: response.Write on hijacked connection

can you please come up with a minimal example to reproduce it?

That was the best request ever.
While trying to prepare it, I managed to sort the problem, on my server - not on ws-machine.
Thanks heaps.


rootfs: fix golang version detection

When the rootfs creation is used for PRs there is
not a match with a kata runtime version, in this
case lets clone the runtime repository and checkout
to the kata branch target. If is already cloned
this mean this was set by depens-on script or the user.
Fixes: #326
Signed-off-by: Jose Carlos Venegas Munoz<EMAIL_ADDRESS>

/test

/test


easy way to also translate pageName in URL

Hi,
is there an easy way to also translate the page itself? Lets say i have a "aboutus.js" in /pages and the generated locale pages should be ueberuns.js or sobrenos.js (my spanish is bad ;-)).
translating content but still having an english url is kind of meaningless isnt it? At least from SEO perspective.

hacked something together on a fork. See the commit
https://github.com/logemann/gatsby-plugin-react-i18next/commit/938c0dc82e7c448773e67480479d5afcd0783d9e
Dont know if its the best way to implement it, but was at least the easiest to implement. Here you can do this in configuration:
{
    resolve: `gatsby-plugin-react-i18next`,
    options: {
        ...
        pages: [
            {
                matchPath: '/aws-beratung',
                urlPageRewrite: new Map([
                    ["en", "/aws-consulting/"]
                ])
            }
        ]
    }
}

This results in a different URL on the english localized-created page. Means, my page in /pages is named aws-beratung.js (which gatsby will create at https://mydomain/aws-beratung/ ) and the localized version will be https://mydomain/aws-consulting/
Happy to hear thoughts about that feature. It would be nice to have the mapping included in the JS page source which gets picked up by the plugin....

hacked something together on a fork. See the commit
https://github.com/logemann/gatsby-plugin-react-i18next/commit/938c0dc82e7c448773e67480479d5afcd0783d9e
Dont know if its the best way to implement it, but was at least the easiest to implement. Here you can do this in configuration:
{
    resolve: `gatsby-plugin-react-i18next`,
    options: {
        ...
        pages: [
            {
                matchPath: '/aws-beratung',
                urlPageRewrite: new Map([
                    ["en", "/aws-consulting/"]
                ])
            }
        ]
    }
}

This results in a different URL on the english localized-created page. Means, my page in /pages is named aws-beratung.js (which gatsby will create at https://mydomain/aws-beratung/ ) and the localized version will be https://mydomain/aws-consulting/
Happy to hear thoughts about that feature. It would be nice to have the mapping included in the JS page source which gets picked up by the plugin....

after digging in deeper, i dont know if i really want to make those changes. First of, the pageUrl rewrite config is one thing. When doing it properly with needed fixes in the area of sitemap.xml serialization, its quite some effort. But even if have my mappings in place in the GraphQL i18n context, then there is a lot to change in the way the useI18next() hook works and probably also fixes in the  component.
Dont know if its worth the effort at this point. Really sad that something like that was not on the table at the beginning. Seems that not many people have SEO in mind when it comes to I18N. I might reopen this thing, if there is some kind demand besides my personal one.

after digging in deeper, i dont know if i really want to make those changes. First of, the pageUrl rewrite config is one thing. When doing it properly with needed fixes in the area of sitemap.xml serialization, its quite some effort. But even if have my mappings in place in the GraphQL i18n context, then there is a lot to change in the way the useI18next() hook works and probably also fixes in the  component.
Dont know if its worth the effort at this point. Really sad that something like that was not on the table at the beginning. Seems that not many people have SEO in mind when it comes to I18N. I might reopen this thing, if there is some kind demand besides my personal one.

I've been also dissapointed about possibility of URL's localization. For me this is the only thing that's this plugin missing. Fortunately my site is just single page but I would like to use this plugin on other sites that are more complex.

I've been also dissapointed about possibility of URL's localization. For me this is the only thing that's this plugin missing. Fortunately my site is just single page but I would like to use this plugin on other sites that are more complex.


Overlay showed wrong info until client restart

I had recently handed in a grown cat for death runes and acquired a new kitten.

My active config had these entries:
$ rg kittenConfig .runelite/settings.properties
116:kittenConfig.catOverlay=true
217:kittenConfig.lastAttentionType=SINGLE_STROKE
533:kittenConfig.secondsNeglected=914
691:kittenConfig.kittenOverlay=true
912:kittenConfig.kittenAttentionOverlay=true
1036:kittenConfig.kittenHungryOverlay=true
1959:kittenConfig.secondsAlive=3194
2013:kittenConfig.secondsHungry=731
2206:kittenConfig.kittenNotifications=true
2576:kittenConfig.felineId=1621

Once the client was restarted it showed the correct info.

https://github.com/runelite/plugin-hub/pull/1990/


Flash List did not update state

Currently, I'm working on a project that is related to a commerce app and using context API so, when I update the state from context API flash list did not render the state that I mentioned in the context API but it's working perfect and producing correct result when I change flash list to flat list.
const { cartArray, setCartArray } = useGlobalContext();
<View style={{ flex: 1, marginEnd: 4 }}>
                <TextInput
                  placeholder={'0'}
                  placeholderTextColor={"lightgray"}
                  mode="outlined"
                  outlineColor='lightgray'
                  keyboardType={'numeric'}
                  numberOfLines={1}
                  maxLength={4}
                  style={{ fontSize: 12, height: 35, lineHeight: 20 }}
                  value={cartArray[id]?.pack}
                  onChangeText={(val) => {
                    onInputChangeValue({ pack: val }, { ...item })
                  }
                  }
                />
              </View>

so when I called the onInputChangeValue method it did not update the cartArray and setCartArray
  const onInputChangeValue = (qty, product) => {
    let tempCart = {
      ...cartArray,
      [product.id]: { ...cartArray[product?.id], ...product, ...qty }
    }
    console.log("🚀 ~ file: PMI.js ~ line 20 ~ onInputChangeValue ~ tempCart", tempCart)
    setCartArray(tempCart);
  }

But if I change Flash List to Flat List It Work Well

just pass extraData={this.state.extraData} in your FlashList
// and update this state when you need.
this.setState({
extraData: !this.state.extraData
)}

FlashList rerenders only iff data or extraData changes. Any other change is ignored.
If you think this is a bug, you can create a new issue with proper reproduction steps that we can follow.


arch: arm: soc: add explanatory comment for kernel headers' inclusion

This commit adds an explanatory comment in all soc.h headers
where kernel_includes.h is included, to stress that it needs
to be included after SOC-specific headers are brought in.
Signed-off-by: Ioannis Glaropoulos<EMAIL_ADDRESS>

@galak FYI

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8629   +/-   ##
=======================================
  Coverage   64.17%   64.17%
=======================================
  Files         429      429
  Lines       41161    41161
  Branches     6926     6926
=======================================
  Hits        26416    26416
  Misses      11587    11587
  Partials     3158     3158

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 0b00493...f9d144d. Read the comment docs.

The reason is the fact that kernel_includes.h needs
the ARM CMSIS definitions, which are brought in by the
SOC-specific header inclusions

From a build-perspective, when one header files depends on another the cleanest way to enforce this is not to require the user to include one header file before the other. But have the header file itself include it's dependencies.
Something like this:
https://github.com/SebastianBoe/zephyr/commit/da7de128d69eed05f0018faff4e7c6ccfea120f0

@SebastianBoe, IMO, your suggestion does not solve the problem of cycle formation. It just moves it inside kernel_includes.h.

As discussed offline, the reason we get a problem is that the "has CMSIS been defined?" check in cmsis.h doesn't work when soc.h indirectly includes cmsis.h.
To work around this, one could omit the "has CMSIS been defined?" check by moving the "generic CMSIS" implementation into a new header file and having the boards that need it include it from soc.h.
Essentially. Simplify by requiring that all (arm) soc.h define CMSIS.

Right, @SebastianBoe, @nashif , #8641  makes this PR obsolete.

Going to add the comment for now, til we figure out how to clean this up further.


Is there any intention to maintain this project?

Hi,
I'd like to make some changes or accept pull requests in this project, is there a chance to resume maintenance or allow others to maintain this project?
Thanks

@khrizt I've been using this library for a while but a few libraries haven't been updated. I created a new library and I've published on Github as well.
https://github.com/miguelbemartin/buffer-sdk-php
Feel free to use or ask me anything :)


Add option to use Winforms embedded browser #547

Added a code receiver, EmbeddedBrowserCodeReceiver,  that uses a WinForms-based dialog with an embedded browser control. Added an optional parameter to GoogleWebAuthorizationBroker authorization calls so the client can pass in the requested code receiver. Default code receiver remains the LocalServerCodeReceiver.

I signed it!

I signed it!

I signed the CLA on behalf of Tribute Inc.

Sorry for the delay in getting to this. @peleyal can you take a look?

I'm not an expert in Winforms, but before I'm jumping into the review, I'm trying to figure it out if we should create a new assembly for WinForms supports or not.
I don't like the fact that we are adding new dependencies like System.Drawing and System.Windows.Forms, but on the other hand it doesn't really harm. Chris and Don, what are your thoughts?
Anyway, I'll try to get into it before the weekend.
Thanks!
Eyal

I had that same concern when I started work on the task. I started out creating a new project, but I was having a lot of difficulty with deciding on an appropriate name, and I was not sure how that would be organized with respect to NuGet packages.
I considered the following facts in finally deciding to add the dialog to the existing assembly.

The existing methods imply that the platform is for installed apps, and WinForms naturally fits into the environment of installed apps.
The project already had two sample receivers.
WinForms is automatically included with the .NET platform.
WinForms assemblies won't be loaded unless the embedded browser code receiver is used. Apps that use the prompt code receiver or local server code receiver won't cause the WinForms assemblies to be loaded.
A WPF control can still be added later without any compatibility issues, with respect to compilation. Users would need to be careful about using the appropriate class for their project, although there are ways to mix and match WPF and Winforms.

Being new to this project, I do not know whether the DotNet4 project is used by Mono-based projects. If it can be used on Mono, then I would have possibly made a different decision, since WinForms is not really a part of the Mono environment.

I signed the CLA again. I think there has been some confusion over associated email addresses because I have multiple valid emails with the company.

I signed the CLA again. I think there has been some confusion over associated email addresses because I have multiple valid emails with the company.

I hear you :+1:
Good summary. I'm happy that you considered all the above. I agree with you that it's more natural to have it in the .NET 4.0 project and not creating a new one.
Regarding Mono, we don't officially support it, but we don't want to limit ourselves. With what I remember WinForm should be built on Mono, it's WPF that doesn't. right?
+@chrsmith as well, any thoughts?

Also @chrsmith, can you take a look in the WinForms specific implementation (since I'm not familiar with WinForms) and I'll review the library's part?

@peleyal Sure thing. I :heart: WinForms.

At least someone...  @chrsmith :+1:
Anyway, @hurcane if you have a sample app that tests it would be great to share it, so Chris would be able to test it himself.
I think that my first comments will be ready by tomorrow.

Sorry, but I have some questions that may seem ignorant to somebody experienced with this process.
In what form would you want a sample app? Since this is part of the authorization process, client secrets need to be provided. I have this integrated into my company's app, but I cannot share that.
Can I just post some sample usage lines, and leave it to Chris to set up a project with necessary references and supply client secrets? Since the change is just to the authorization process, the code to test it won't be too difficult to implement.

You should definitely not share your client secret and id. I thought about some sample app, like the ones we have in our samples repository, https://github.com/google/google-api-dotnet-client-samples. I guess you can even have a PR to change https://github.com/google/google-api-dotnet-client-samples/tree/master/Tasks.WinForms.NoteMgr to work with your new API. Later on, after releasing a new version of the library we will even merge this PR to the samples repository.
We just want to test that the new code works. What happens when the users accept the permissions, what happens when they decline, etc.

And your question is totally valid. Let me know if my suggestion sounds good to you :)

Thanks so much for the links, @peleyal. I had not discovered the samples repo. For sure I can modify the WinForms sample and eventually submit a pull request. For testing purposes, will @chrsmith simply clone my fork of both repos?
In addition to testing the Allow and Deny responses, I had two other use cases:

Closing the authorization window without clicking Allow or Deny.
Switching to a different Google account before authorizing.

Closing the window is the use case that led me to make these changes. The local server code receiver has a bug (issue #508) that I could not accept. I initially planned on fixing that bug by implementing a timeout mechanism, but my lack of experience with async/await programs prevented me from being able to figure it out.
In fact, the embedded browser authorization is synchronous from the perspective of the installed application. The calling code will be unresponsive until the browser dialog has closed. Personally, I like the synchronous style as a user, but I would understand if you want the calling app to still be responsive, and eliminate the modal dialog.
I was wondering how I am going to get the sample to use my locally changed version of the library. It looks like I'll need to educate myself on creating a NuGet package and using a local package source for the samples.
I might do some other experimentation with the other samples targeting the net40 framework. I see there is a VB console app for the calendar, and a WPF task list sample. I think it would be interesting to see how the WinForms dialog works with those samples, too.

First review; looks good.
I'll wait for you to address the comments and for Chris to actually test it before having the next review.
Regarding your comments:
"I was wondering how I am going to get the sample to use my locally changed version of the library. It looks like I'll need to educate myself on creating a NuGet package and using a local package source for the samples."

You don't need to publish\create new NuGet packages. I would just add a local reference to the Google.Apis assemblies you are changing. Then Chris would be able to do the same.
Before releasing a new library, we would be able to actually merge the PR, using the new Google.Apis.Auth version.
I'll leave the details for Chris, but my main point, don't worry about it. Focus about creating the right change and a sample code for it. We will take it from there.

"I might do some other experimentation with the other samples targeting the net40 framework. I see there is a VB console app for the calendar, and a WPF task list sample. I think it would be interesting to see how the WinForms dialog works with those samples, too."

I think that you should change only the sample I mentioned because it's the only one that is for WinForms. The others don't run on WinForms and there is NO point in using a WinForm dialog for them.

@hurcane, what's the company name you used to sign the CLA? I want to verify it... (an alternative is to sign using you email address that you use in GitHub)

The company name is Tribute, Inc. from Hudson, OH USA. I did a couple incorrect submissions using my email as the signer. This past Friday, I submitted the CLA again with the owner's name and email. He signed it, but I don't think we have gotten a confirmation yet.

Should I submit a pull request now for the changes in my samples fork? Or can testing be done using a clone of my fork, and the pull request can be submitted after this pull request is accepted?
My fork of the samples repo: https://github.com/hurcane/google-api-dotnet-client-samples/

We have received a confirmation of a completed CLA for Tribute Inc.

@chrsmith can you please take a look in this one (WinForm wise)?

The WinForms code looks reasonable to me. (I didn't build and see what the .resx file looks like in its full GDI+ enabled glory.) If @peleyal is happy with the rest of the code, I'm cool with giving it a :+1:

@chrsmith Can you run a WinForm sample using the new code? I prefer us to test it before submitting it.

Do we all agree that we should add it to the current Auth assembly and not to a new NuGet package for WinForm extensions? It's a tough call, and setting up a new project + NuGet configuration might be an overkill here (similar approach was taken for ASP.NET MVC, https://www.nuget.org/packages/Google.Apis.Auth.Mvc).
IMO setting up a project just for WinForm is better, because WinForm isn't used by 99% of Google.Apis.Auth users and it's a better separation.
Sorry that I'm bringing it up again, but I want us all to be on the same page regarding this one.

Thanks!
Eyal

Our internal testing has revealed a bug. The project works on machines that have IIS installed, even if it is not active. It is failing on machines that don't have IIS. I will be addressing that, and I could move the WinForms-dependent classes into a separate DLL, following the MVC project model.

@peleyal @chrsmith  @LindaLawton  I have moved the WinForms dependencies into a separate project. In addition, I have a fork of the google-api-dotnet-client-samples project that includes changes to the WinForms Notes sample to allow testing the embedded browser in the WinForms project.
In addition, I corrected the bug in the previous commits. The redirect URL was going to localhost by default, but WinForms projects should use the "out-of-band" redirect URL. The project has been modified to work out-of-band.
Designing NuGet specifications is new to me. I hope that I did it correctly for the new project in the solution.

@peleyal @chrsmith @LindaLawton Is anybody willing to view the updates I have made to this pull request?

@hurcane I am sorry for this taking so long. I swear to  will make the time to review this code tomorrow.
Moving the WinForms bits to a separate project removes most of the concerns I would have had -- chiefly bringing in dependencies that might cause problems on specific platforms. I'll take a final look and let you know in less than 20 hours. (Again, sorry.)

Hey, sorry for taking so long to get to this PR. We (Google) are trying to improve the overall level of support for our client libraries, and so we are still working out things like release process, PR process, etc.
I spoke with some people in the office, and thought more about what we want to do in terms of support. I do not think we should merge this pull request into the repo.
While this PR may improve the client-side app experience when using Google APIs, it also complicates the test/release matrix. e.g. how well is WinForms supported on other platforms if at all? How can we best test this library for future releases? Even if the code is in another csproj, it still adds complexity we need to support in the future.
However, I'm happy to take the refactorings to enable your change. (e.g. exposing an ICodeReceiver parameter in GoogleApis.Auth.DotNet4.) But beyond that I'd encourage you to create your own GitHub repo for GoogleApis.Auth.WinForms portion. I'd be happy to link to it in the documentation for this project.
I will close this PR out. Please open another PR specifically for the refactoring to enable custom ICodeRecivers. I am sorry it took so long to bring this PR to resolution.


Add cffi API level to increase performance

Currently you use cffi at the ABI level, it could be great for performance to use it at the API level.
See : https://cffi.readthedocs.io/en/latest/overview.html

AFAIK, API level is useful when you have some C source code to be compiled to .so. But pyVulkan is just a binding for exising C API of vulkan, so I'm not sure about the potential performance improvements.

Indeed you seems right but I don't know enough CFFI.


Update server

removed the download-language-server.sh as it didn't work, so I just manually downloaded the server and put it the language-server folder.
removed the is_allowed_to_start and resolve_module.js as it wasn't necessary
added dummy handling for the onWorkspaceConfigForDocumentRequest, (if I do not handle this request, code actions will not appear)
enabled code actions to fix spelling mistakes

NOTE: This server is still a work in progress.

Uhh, just realised that node_modules are ignored, and installing the package.json with npm doesn't not work for some, yet unknown, reason.
Will find a fix for that...


Feat/tdkn 126 change title to displayname

Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)
the components are using the property "title" to display their name, it was change to "displayName", so the build is broken.
What is the new behavior?
Add "displayName" property anywhere this is needed
Does this PR introduce a breaking change?

[ ] Yes
[X] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:


Default profile missing from terminal menu

Testing #119427
macOS Catalina
Changing the default profile to ksh does not include it in the terminal's menu. I can add it using settings, but it would be nice if  I didn't have to do that.

It doesn't add it there by design because that's what the + button does. Next iteration it'll look something like this where the + is your primary/default and ^/v is the secondary/custom ones:


etcd systemd unit should wait for certificate files to become available

Otherwise etcd start right after booting and fails several time before we copy certificates over SSH, then it converges.
This produces logs like:
Oct 12 13:14:07 tink-controller-0 96c53009bb40[846]: 2020-10-12 13:14:07.754655 I | etcdmain: etcd Version: 3.4.13
Oct 12 13:14:07 tink-controller-0 96c53009bb40[846]: 2020-10-12 13:14:07.754682 I | etcdmain: Git SHA: ae9734ed2
Oct 12 13:14:07 tink-controller-0 96c53009bb40[846]: 2020-10-12 13:14:07.754692 I | etcdmain: Go Version: go1.12.17
Oct 12 13:14:07 tink-controller-0 96c53009bb40[846]: 2020-10-12 13:14:07.754967 I | etcdmain: Go OS/Arch: linux/amd64
Oct 12 13:14:07 tink-controller-0 96c53009bb40[846]: 2020-10-12 13:14:07.755005 I | etcdmain: setting maximum number of CPUs to 4, total number of available CPUs is 4
Oct 12 13:14:07 tink-controller-0 96c53009bb40[846]: [WARNING] Deprecated '--logger=capnslog' flag is set; use '--logger=zap' flag instead
Oct 12 13:14:07 tink-controller-0 96c53009bb40[846]: 2020-10-12 13:14:07.755325 I | embed: peerTLS: cert = /etc/ssl/etcd/etcd/peer.crt, key = /etc/ssl/etcd/etcd/peer.key, trusted-ca = /etc/ssl/etcd/etcd/peer-ca.crt, client-cert-auth = true>
Oct 12 13:14:07 tink-controller-0 96c53009bb40[846]: 2020-10-12 13:14:07.755756 C | etcdmain: open /etc/ssl/etcd/etcd/peer.crt: no such file or directory

And adds unnecessary noise tot he system.

assets/terraform-modules/controller/templates/etcd.yaml.tmpl has this already added a day after this issue got created:
        ConditionPathExists=/etc/ssl/etcd/etcd/server-ca.crt
        ConditionPathExists=/etc/ssl/etcd/etcd/server.crt
        ConditionPathExists=/etc/ssl/etcd/etcd/server.key
        ConditionPathExists=/etc/ssl/etcd/etcd/peer-ca.crt
        ConditionPathExists=/etc/ssl/etcd/etcd/peer.crt
        ConditionPathExists=/etc/ssl/etcd/etcd/peer.key

It's just a matter of aligning the remaining platforms to use this.

It's just a matter of aligning the remaining platforms to use the common controller/worker modules.

I wouldn't say "just", but yeah :smile:

Ported the conditions over in https://github.com/kinvolk/lokomotive/pull/1511

Hmm, it seems #1511 does not work as expected. See https://yard.lokomotive-k8s.net/builds/4690979 :disappointed:

Those conditions weren't doing anything because they have to be specified in the [Unit] section.

Good find^^ In assets/terraform-modules/controller/templates/etcd.yaml.tmpl they are also under [Service].

I'll file a PR fixing them everywhere. I hope this doesn't break things :sweat_smile:


feat(codemod): add codemods for socket.io 4.0 and react native 75

📚 Description

🔗 Linked Issue

🧪 Test Plan

📄 Documentation to Update

@Janhvibabani We're moving codemod packages to their corresponding owners so it's easier for codemod owners to own and maintain their codemods, and be notified about any issues in their own repos. Do you mind moving this to your own repo and updating the GitHub source URL of the published codemod to point to your github repo?

Create a repo in your own GitHub account and move the source code of your codemods to that repo.

This will drive more traffic to your profile and clarify ownership, increasing visibility and transparency.
You could name the repo something like codemods (or choose a name you prefer).
For the README of that repo, you can use a format similar to what's in the commons repo.

Update the git URL in the codemods’ metadata to your new repo and bump their versions.

Republish the codemods to ensure they are showing the correct git URL in the registry.

Optionally, you can duplicate this GitHub Action for your repo. This way, every time you make a change to one of your codemods and merge the code to the main branch, it will automatically publish the codemod for you.