Can intellisense expand column names?

In almost all SQL intellisense, when you type Tab after * or sometable in the following example, all column names can be auto filled.  But I can't figure out where this is.  Can someone give a hint?  Thanks.
SELECT * FROM sometable
INSERT INTO sometable

Azure Data Studio Version: 1.2.4

Any news here? This is a MAJOR adoption blocker...

Bumping this, Azure Data Studio is unusable without this feature.

Any news here ?

Asking for the same feature, could be awesome to implement!

Bumping to keep this alive... please implement!!!

I too will bump this to ask you to implement

While I am using ADS I feel like I shouldn't simply because star expansion is missing. Please, implemen this.

Bump.

Bumping, this feature would be insane.

Please add this. The lack of this feature slows me down a lot. It is biggest reason why I'm not moving over to ADS.
Thank you.

To all who have been holding your breath. This feature as well as many many others that Azure Data Studio are lacking are available in IntelliJ IDEA / Datagrip.
I did not make the transition earlier because I needed to log into my MSSQL DB on my Mac through Kerberos and I thought ADS was the only way to do it. I figured it out with IntelliJ and now I am permanently getting rid of ADS.

Really need it, please implement it asap.

This missing feature is one major reason why use other software to work with sql server. For example SSMS, where I can use the graphical design editor to expand, or some SSMS extensions doing this job.

It will be a nice addition.  I just need a way to bring all the columns into the query without typing all of them.

Seems like the more people that ask for this feature, the less likely MS are to implement it lol.
Yup, definetly need this or else SSMS is never gonna get uninstalled.

I still would like to see this feature, has anyone from MS commented on this?

Hi, just bumping again. @alanrenmsft is work being done on this feature?

@aasimkhan30   Hi, thanks for the update.  SELECT * appears working.  What about Insert?  Is it possible to do something like this?
insert into sometable ( ... ...  ctrl+space here, column list here, probably excluding identity column

@ThisNoName can you please file a new issue for that feature? Thank you!

@robolize @henkers, this feature is added in the insider's version of ads. Please give that feature a try and let us know if you have any feedback. It can used by placing the cursor at '*" and pressing ctrl+space (ads shortcut for intellisense suggestion).

@aasimkhan30 👍 Thank you so much for this. I can confirm it is working for me on Windows and Mac.
(Apologies for the late reply)

I'm on version 1.36.1. This feature is not working.


Freeing garbage collected objects does not call drop

Due to the use of the Array type it is not possible to simply use ptr::read on the data in AllocPtr to drop the value. drop_in_place would allow the value to be dropped correctly but it is currently unstable.
https://github.com/rust-lang/rust/issues/27908

Fixed in f9722cd1a03c1948fdfce06319a5f5e1db831157 :tada:


Do we need the crisis module?

I don't think there are any invariants registered, so it has no value at the moment

We don't have invariants registered. But there are some registered by the band, distribution, staking, and gov modules. But I'm not sure why they are useful, they seem to be unremoved runtime assertions. I believe that when the invariant is broken, it's a bug in the the module. It can be caused by a bad upgrade for example. The crisis module halts the chain so that not a lot is built up upon a bad state of the chain. And then it can be hard-forked or upgrade-skipped with proper fixes.
An example of a check for an invariant is: threefold_hubd tx crisis invariant-broken bank total-supply --chain-id threefold-hub-testnet --node https://threefoldhub.test.gridtesting.xyz:26657 --gas-prices 80TFT --from placeholder --keyring-backend test --broadcast-mode block --gas-adjustment 1.3 --gas auto --home ~/.threefold_hub_devnet
Should it be removed or not?

thanks, just leave it in then

Re-opened because of my question to add the findings to docs/crisis.md .

Updated https://github.com/threefoldtech/threefold_hub/blob/development/docs/crisis.md


infra: add SQLAlchemy to min version testing

preventing issues like #22546
Notes:

this will only affect release CI. We may want to consider adding running unit tests with min versions to PR CI in some form
because this only affects release CI, it could create annoying issues releasing while I'm on vacation. Unless anyone feels strongly, I'll wait to merge this til when I'm back

Since this was switched to draft - what needs to be done before merging?


Support streaming list results

Created upon request from https://github.com/turbot/steampipe-plugin-github/pull/167.
Next to d.streamListItem it would be great to have a d.streamList functionality when multiple items need to be streamed.
A naive implementation could look something like:
func (d *QueryData) streamList(ctx context.Context, item interface{}) {
	  for _, item := range item {
		  if item != "" {
			  d.StreamListItem(ctx, item)
		  }

		  // Context can be cancelled due to manual cancellation or the limit has been hit
		  if d.QueryStatus.RowsRemaining(ctx) == 0 {
			  break
		  }
	  }
}

Hi @japborst . Sorry for taking a while to respond to this one.
So as I understand it, this would just be a shortcut function to stream multiple items with once sdk call?

@kaidaguerre thanks for taking a look.
That's indeed correct.

@japborst got an even simpler solution for this - StreamListItem will be updated to being a variadic function:
func (d *QueryData) streamListItem(ctx context.Context, items ...interface{})

so to pass an array of items you just need to expand it:
d.StreamListItem(ctx, rows...)

This will be in the upcoming v4.0.0 sdk release


Allow usage in a CMake-based build

Hi (cross-linking from https://github.com/espressif/rust-esp32-example/issues/34)
thanks so much for your work on embedded rust, and in particular on ESP chips :)
We're a company using ESP on our hardware, and we'd love to move our codebase to Rust. Because that move will be incremental, we'd like to compile our C/C++ codebase and switch some modules to Rust incrementally. For this, it seems more natural to use a "CMake-based" build that would compile our Rust modules as a library, and link our code to it. And of course, we'd love to use esp-idf-svc :)
For now, we saw https://github.com/ivmarkov/rust-esp32-std-hello which uses esp-idf-svc, but uses a "cargo-first" build which seems less natural for us.
Would it be possible to allow building esp-idf-svc using a CMake build so that it can be included in a CMake project like for instance https://github.com/espressif/rust-esp32-example ?

@igrr, @georgik - to make the CMake-first build compatible with esp-idf-sys (and this with esp-idf-svc and friends), we really only need to define three additional environment variables and pass them down to Cargo here. The names of the env variables are completely arbitrary:

CARGO_CMAKE_BUILD_INCLUDES - this should be a delimited list of all include directories of all ESP-IDF components. If that makes it easier, having additional - user-components - on that list will not be a problem, I think. Btw, I tried with get_target_property(CARGO_CMAKE_BUILD_INCLUDES ${COMPONENT_TARGET} COMPONENT_INCLUDES)
CARGO_CMAKE_BUILD_SDKCONFIG - this should be the full path to the sdkconfig file used during the build. A less ergonomic alternative would be for this environment variable to directly contain all CONFIG_* options - somehow delimited - from the sdkconfig
CARGO_CMAKE_BUILD_COMPILER - the full path to the compiler (as in GCC) used during the build. We need this so as to derive a proper sysroot for the Bindgen utility.

Any help (in terms of CMake syntax) as to from where to get the content of these three variables would be greatly appreciated!

@georgik Can you help with ^^^? I have the (minimal) changes to the esp-idf-sys crate ready, but unless we figure out the above problem, there is no way to test these...

That would be really nice if we could do it, we'd love to be able to start hacking on Rust with esp-idf-svc :)


Improving the cleanup instructions

Currently, this issue is only addressed at the bottom of the FAQ, which is not very visible for new users.

/lgtm


benchit API uses the groqflow cache dir instead of mlagility cache dir by default

@danielholanda @ramkrishna2910 this is why we cant find out cached builds


Use of dispatch requires explicit use of hop(to:)

Currently, when you dispatch a job inside a route handler or another context you can run into a NIO precondition that the dispatch method is not returning on the same event loop. This happens due to _QueuesRedisDriver using a client property, which is basically RedisConnectionPool which could be on any of app.eventLoopGroup's event loops.
Even though _QueuesRedisDriver conforms to RedisClient and has a eventLoop property, it still uses client to send the request, which means it ends up on the pool's event loop, causing the precondition.
An easy fix is adding .hop(to:) whenever you use .dispatch() for example. However, this can get tedious to remember. So, this package or RedisStack should handle hopping to the context.eventLoop automatically

@madsodgaard very good catch here, thanks for bringing it up. Just to be clear, are you talking about lines like this https://github.com/vapor/queues-redis-driver/blob/master/Sources/QueuesRedisDriver/JobsRedisDriver.swift#L106 should be using hop(to:)?

@jdmcd Yup! The subscription methods should probably hop to the context event loop as well.

@madsodgaard returning to this since I think this is going to start affecting my app soon. When you say hop(to:) needs to be used what would we be hopping to? Something like .hop(to: client.eventLoop)? Or something else?

@jdmcd Right now it's returning on the client.eventLoop which could be any event loop of the group, so we need to hop to context.eventLoop which for example is the Request event loop. I actually think we could fix this by just overriding RedisClient.eventLoop on _QueuesRedisQueue, otherwise we'd add .hop(to: context.eventLoop) to _QueuesRedisQueue.send() (not sure if we need the other helper functions as well)
extension _QueuesRedisQueue: RedisClient {
  ...

  var eventLoop: EventLoop {
    self.context.eventLoop
  }
}

@madsodgaard Yea this would definitely work. All the redis commands use the client/pools event loop to do its work so overriding the _QueuesRedisQueue to do it would just switch all the event loops being used to be the context's loop since it's conforming to the protocol. Just went through to verify that all commands had an event loop etc.


The SSL connection could not be established

Hey,
I installed the add-on using the instructions. After sign in by the microsoft device login I´m getting an Error:
`01/31/2024 15:39:19 [18] Error: Error getting free space: Status Code: 0
Microsoft.Graph.ServiceException: Code: generalException
Message: An error occurred sending the request.

 ---> System.Net.Http.HttpRequestException: The SSL connection could not be established, see inner exception.
 ---> System.Security.Authentication.AuthenticationException: Cannot determine the frame size or a corrupted frame was received.
   at System.Net.Security.SslStream.EnsureFullTlsFrameAsync[TIOAdapter](CancellationToken cancellationToken)
   at System.Runtime.CompilerServices.PoolingAsyncValueTaskMethodBuilder`1.StateMachineBox`1.System.Threading.Tasks.Sources.IValueTaskSource<TResult>.GetResult(Int16 token)
   at System.Net.Security.SslStream.ReceiveBlobAsync[TIOAdapter](CancellationToken cancellationToken)
   at System.Net.Security.SslStream.ForceAuthenticationAsync[TIOAdapter](Boolean receiveFirst, Byte[] reAuthenticationData, CancellationToken cancellationToken)
   at System.Net.Http.ConnectHelper.EstablishSslConnectionAsync(SslClientAuthenticationOptions sslOptions, HttpRequestMessage request, Boolean async, Stream stream, CancellationToken cancellationToken)
   --- End of inner exception stack trace ---
   at System.Net.Http.ConnectHelper.EstablishSslConnectionAsync(SslClientAuthenticationOptions sslOptions, HttpRequestMessage request, Boolean async, Stream stream, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.ConnectAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.AddHttp2ConnectionAsync(QueueItem queueItem)
   at System.Threading.Tasks.TaskCompletionSourceWithCancellation`1.WaitWithCancellationAsync(CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.HttpConnectionWaiter`1.WaitForConnectionAsync(Boolean async, CancellationToken requestCancellationToken)
   at System.Net.Http.HttpConnectionPool.SendWithVersionDetectionAndRetryAsync(HttpRequestMessage request, Boolean async, Boolean doRequestAuth, CancellationToken cancellationToken)
   at Microsoft.Graph.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at Microsoft.Graph.RetryHandler.SendAsync(HttpRequestMessage httpRequest, CancellationToken cancellationToken)
   at Microsoft.Graph.CompressionHandler.SendAsync(HttpRequestMessage httpRequest, CancellationToken cancellationToken)
   at Microsoft.Graph.AuthenticationHandler.SendAsync(HttpRequestMessage httpRequestMessage, CancellationToken cancellationToken)
   at System.Net.Http.HttpClient.<SendAsync>g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
   at Microsoft.Graph.HttpProvider.SendRequestAsync(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationToken cancellationToken)
   --- End of inner exception stack trace ---
   at Microsoft.Graph.HttpProvider.SendRequestAsync(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationToken cancellationToken)
   at Microsoft.Graph.HttpProvider.SendAsync(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationToken cancellationToken)
   at Microsoft.Graph.BaseRequest.SendRequestAsync(Object serializableObject, CancellationToken cancellationToken, HttpCompletionOption completionOption)
   at Microsoft.Graph.BaseRequest.SendAsync[T](Object serializableObject, CancellationToken cancellationToken, HttpCompletionOption completionOption)
   at Microsoft.Graph.DriveRequest.GetAsync(CancellationToken cancellationToken)
   at hassio_onedrive_backup.Graph.GraphHelper.GetFreeSpaceInGB() in /src/Graph/GraphHelper.cs:line 232`

Has anyone an idea?
Thanks for Help.

Seems like either something transient or a networking issue. Do you have any proxies / firewalls outbound ?
Do all connections fail? In the logs, getting the online backups, uploading etc?

No, there ist no proxy and no Firewall.
If I try to upload a backup, I get on the WebUI:
An error has occurred. This application may no longer respond until reloaded
In the log I see always the same error.

Something on the network is interfering with the connection. Quick search showed a couple of people experiencing this error (not with this addon but again this is something general). See here : https://github.com/Jackett/Jackett/issues/11963
Perhaps something in the router configuration.

In my Setup HomeAssistant is running as a VM (HaOS) with Proxmox.
When I run your Software in VS the Connection to OneDive is sucessful and there is not SSL-Error. The Freespace is showing and I can Upload Backups.
I tried to Add the Microsoft Graph Certificate to Homeassistant, but that doesn´t fix the Error.
My Router is a Fritzbox and there are no setting, which would prevent that. I have no idea.

How is the VM connected to the main network ? Is there another virtual switch / firewall settings in Proxmox?

I could solve the Issue by myself.
The installation of Proxmox seams to be faulty. After reinstallation everything works fine.


Specified geo fields, changed type of md_ fields

@CucumisSativus Please check changes. Now we have specified Geolocation fields. I've also changed type of all dynamic fields with md_ prefix to text_gen_sort. It will be better in faceting.


Issue with baesian optimizer for custom dataset.

When using Bayesian Optimizer (BaysOptimizer) with a custom dataset I'm having problems with a "ValueError: Unknown dataset: [ds name]" error.
According to the Traceback, the error is thrown from the "kgcontroller.py" module (line 160). The only hypothesis I had was that the custom dataset_path variable was Null, which I ended up confirming.
This is the snippet i use for the calling the optimizer:
args = KGETuneArgParser().get_args(['-mn', model_name, '-ds', dataset_name, "-dsp", dataset_path])
bays_opt = BaysOptimizer(args=args)
bays_opt.optimize()
best = bays_opt.return_best()

I already verified both the dataset_name and dataset_path exist since i use them in the KG creation and it works just fine:
args = KGEArgParser().get_args(['-mn', model_name, '-ds', dataset_name, "-dsp", dataset_path])
knowledge_graph = KnowledgeGraph(dataset=args.dataset_name, custom_dataset_path=args.dataset_path)

After diving  bit into the code, it seemed to me that on the "bayesian_optimizer.py"
module, the kge_args object passed to the config_obj is not given the path for the dataset (only the name) on line 51. Which seems that in consequence not passing it to its KnowlegeGraph object, raising the error.
Note: Adding this to the BaesianOptimizer init function seems to solve the issue for me.
Full Traceback:
Traceback (most recent call last):
  File "train.py", line 104, in <module>
    main()
  File "train.py", line 72, in main
    bays_opt = BaysOptimizer(args=args)
  File "/usr/local/lib/python3.6/dist-packages/pykg2vec-0.0.51-py3.6.egg/pykg2vec/utils/bayesian_optimizer.py", line 51, in __init__
    self.config_local = self.config_obj(self.kge_args)
  File "/usr/local/lib/python3.6/dist-packages/pykg2vec-0.0.51-py3.6.egg/pykg2vec/config.py", line 65, in __init__
    self.knowledge_graph = KnowledgeGraph(dataset=args.dataset_name, custom_dataset_path=args.dataset_path)
  File "/usr/local/lib/python3.6/dist-packages/pykg2vec-0.0.51-py3.6.egg/pykg2vec/data/kgcontroller.py", line 160, in __init__
    raise ValueError("Unknown dataset: %s" % dataset)
ValueError: Unknown dataset: GOT

OS: Windows 10

Thank you for the reply!


FEAT: Added responsive Grid layout for team members

@haxzie Please check[test] for the last item(last member width) it is currently set to take 1fr.

@haxzie  Squash the commits if possible.

This is Beautiful :heart:


call_method: JavaScript Error not a function

I can't seems to use any godot function in this latest version of this module (alpha15)
My code was simply is this:
export default class SomeNode extends godot.Node2D {
// There is some code above here, but its not important

	_ready() {
		// Same thing happens with this.get_node('Sprite')
		this.sprite = this.$("Sprite");
		godot.print("hello world");
	}
}

And I get this error:

@zw7u9tru  Can you check this ?
I didn't compile and test your code before merge.

Yes I will work on this right now

Same problem

@Terria028 @antharuu  let me know if the current release works better for you!

@Terria028 @antharuu let me know if the current release works better for you!

Oh, thank you, it's good now !

@Terria028 @antharuu  let me know if the current release works better for you!

Yes, it is now fixed! Thank you, might close the issue now


Godot 4: Scenes Don't Run (Condition "!process_map->has(p_pid)" is true)

Godot version:

Godot Engine v4.0.dev.calinou.160ad23e5 (Originally spotted in)
Godot Engine v4.0.dev.custom_build.339b64697 (Last checked build, as of 1st december 2020)
OS/device including version:

Windows 10
Intel Pentium G3258<EMAIL_ADDRESS>with 16gb@1333mhz memory
Probably not related but...
Nvidia Quadro K2000, Driver: 441.28
Issue description:

When I try to run a scene (F6 or the button) Godot just spikes to nearly 100% CPU Usage, no window appears and all log files created are empty, while what was expected is a Window, with a splash screen and the scene appearing when it loaded.
Steps to reproduce:
Run a Scene, Doesn't matter if it's 2d or 3d, full or totally empty, new or old project..
If the scene is killed it returns this error in the output

platform/windows/os_windows.cpp:485 - Condition "!process_map->has(p_pid)" is true. Returning: FAILED

Minimal reproduction project:

Test.zip
The MRP doesn't help a lot, as it doesn't rly matter, however something i did notice is, When a Window node is created and the camera is attached to it.. the renderer works on both 3d and 2d so that isn't the issue... and also the window has no impact on CPU usage (well almost none)

I have a similar behavior when running the project:

The CPU spikes;
Takes a while, but the window appears;
When quitting the application, the same message outputs: platform\windows\os_windows.cpp:485 - Condition "!process_map->has(p_pid)" is true. Returning: FAILED.

I haven't noticed any critical points related to this error yet. It simply notifies by assert that there was an attempt to terminate an already killed process. So far, I just replaced the line ERR_FAIL_COND_V(!process_map->has(p_pid), FAILED); with if (!process_map->has(p_pid)) { return OK; } so that it doesn't hurt my eyes every time.

Update (8e514bb):
Window now Does appear almost instantly now (used to not show up at all, CPU still spikes and the window is Blank white (regardless of anything in the scene or not, and well, no splash screen so..))

Update (8e514bb):
Window now Does appear almost instantly now (used to not show up at all, CPU still spikes and the window is Blank white (regardless of anything in the scene or not, and well, no splash screen so..))

Update (tested on same branch as above):
Managed to Get it to Run, by setting the main scene to run in project settings, quitting the editor and pressing run in Project Selector, Instantly Ran, just fine, no CPU spike
@felaugmar Could you please try it and see if that solves the CPU spike for you too?

Update (tested on same branch as above):
Managed to Get it to Run, by setting the main scene to run in project settings, quitting the editor and pressing run in Project Selector, Instantly Ran, just fine, no CPU spike
@felaugmar Could you please try it and see if that solves the CPU spike for you too?

#47076 fixed the CPU spike and now runs as expected. Error msg provided is Not related to the issue, and is a Separate issue, that doesn't seem to have any effect on performance.

Im getting this now with the Steam build of 3.4.4 exporting to android via debug. I get a black screen that hangs sometimes, but when I reload it and reinstall the debug app on my developer phone, it works half the time just fine. Should I change something that I haven't likely seen yet since this is closed already? Thanks if you can help :)

Im getting this now with the Steam build of 3.4.4 exporting to android via debug. I get a black screen that hangs sometimes, but when I reload it and reinstall the debug app on my developer phone, it works half the time just fine. Should I change something that I haven't likely seen yet since this is closed already? Thanks if you can help :)

I can see one instance of process_map in an Android issue here: https://github.com/godotengine/godot/issues/38278
However, this error message only means that the child process couldn't be killed by Godot. It's not related to the export specifically (and is not emitted by the exported project).


Invalid monitoring.coreos.com/alertmanagerconfig_v1alpha1.json

Issue
kubeconform throws this error
Check JSON formatting: jsonschema: '/spec' does not validate with https://raw.githubusercontent.com/datreeio/CRDs-catalog/main/monitoring.coreos.com/alertmanagerconfig_v1alpha1.json#/properties/spec/additionalProperties: additionalProperties 'route', 'receivers' not allowed
Description
AlertmanagerConfig does not reflect prometheus-operator-crd monitoring.coreos.com_alertmanagerconfigs.yaml.
Possibly introduced today with PR where schema monitoring.coreos.com/thanosruler_v1.json has been mistakenly placed inside the AlertmanagerConfig schema example/prometheus-operator-crd/monitoring.coreos.com_alertmanagerconfigs.yaml.

@dcardellino24 @eyarz

good catch
open a PR and I will merge it

https://github.com/datreeio/CRDs-catalog/pull/246

Is there a reason why this hasn't been merged yet? We've been waiting on it.

@eyarz please review https://github.com/datreeio/CRDs-catalog/pull/246

please review #246


WARN  %3|1710839746.266|FAIL|rdkafka#producer-1| [thrd:broker:9092/bootstrap]: broker:9092/bootstrap: Failed to resolve 'broker:9092': nodename nor servname provided, or not known (after 214ms in state CONNECT).

I have installed kafka on docker. The docker-compose.yml file is as follows.
version: "3.7"

services:
 zookeeper:
   image: zookeeper:3.7
   container_name: zookeeper
   ports:
     - "2181:2181"
   environment:
     ZOOKEEPER_CLIENT_PORT: 2181
     ZOOKEEPER_SERVER_ID: 1
     ZOOKEEPER_SERVERS: zookeeper:2888:3888

 kafka1:
   image: wurstmeister/kafka
   container_name: kafka1
   ports:
     - "9092:9092"
   environment:
     KAFKA_ADVERTISED_HOST_NAME: kafka1
     KAFKA_ZOOKEEPER_CONNECT: zookeeper:2181
     KAFKA_BROKER_ID: 1
     KAFKA_OFFSETS_TOPIC_REPLICATION_FACTOR: 1
   depends_on:
     - zookeeper

 kafka-ui:
   image: provectuslabs/kafka-ui:latest
   container_name: kafka-ui
   ports:
     - "8080:8080"
   environment:
     KAFKA_CLUSTERS_0_NAME: "kafka-local"
     KAFKA_CLUSTERS_0_BOOTSTRAP_SERVERS: "kafka1:9092,kafka2:9092,kafka3:9092"

My code

But I get the following error.

How can I fix it? Where am I going wrong?

Hey! I think the broker address should be localhost:9082 instead of broker

whoops, that was my mistake while typing from my phone. The point is that you are using broker as the broker address, while it should be localhost:9092

Closing due to lack of updates


"No path to node" error

I am using the latest OSMNX with python 3.9. When I try to find the shortest path between two nodes, I receive an error of "No path to 'No path to 204349928'. I am looping through points in Chittenden county, Vermont. I have nearly 65k points need to run, and this error has occurred multiple times
Is there a way to resolve this issue? Potentially skip the nodes and move on to the next on in the loop.
Thanks!!

Is there a way to resolve this issue? Potentially skip the nodes and move on to the next on in the loop.

If this is a "how-to" usage question (it appears to be), please ask it on StackOverflow, per the contributing guidelines. If this is a bug report for a bug you have discovered in the OSMnx codebase, please completely fill out the new issue template with the necessary details for troubleshooting (per the new issue instructions you saw when opening this issue), and we can re-open.


Use macOS 11 in CI

Signed-off-by: Patrik Stas<EMAIL_ADDRESS>

Codecov Report

Merging #527 (3c3b067) into main (08395ba) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #527   +/-   ##
=======================================
  Coverage   55.07%   55.07%
=======================================
  Files         168      168
  Lines       14562    14562
  Branches     4027     4027
=======================================
  Hits         8020     8020
  Misses       2338     2338
  Partials     4204     4204

Flag
Coverage Δ

integration-aries-vcx
42.92% <ø> (ø)

unittests-agency-client
23.61% <ø> (ø)

unittests-aries-vcx
51.80% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us.


feat: scroll to top on progress screen change

We probably need to smoothen/animate the scroll to top. any feedback on this experience, @wbobeirne ?

The current behavior here is very annoying. Also defer to @wbobeirne about how to fix.

I can make it smooth I preferred just jumping to top, fine with either

I'll look at this one today, sorry for the delay.

Typical web behavior is that new pages jump to the top immediately on navigation, not smoothly, so that's what I'd advocate for. I think Kody's initial behavior was closer to a normal web experience.


Cancel the API Key when an environment is deleted

Is your feature request related to a problem? Please describe.
It is more of an oversight. Whenever an Environment is deleted within a project the API still returns calls if the key is provided.
Describe the solution you'd like
We should delete the API key along with the environment.
Describe alternatives you've considered
Maybe we want to wait for a period before deleting the API key or do a soft delete of sorts in the case the Env needs to be restored.
Additional context
NA

Related: https://github.com/Flagsmith/flagsmith/issues/1875

Related https://github.com/Flagsmith/flagsmith/issues/3060

#3060 is caused by soft delete; unrelated to this.


c3-lsp: init at 0.3.2

Description of changes
Closes #343745

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 343801 run on aarch64-darwin 1

1 package built:

c3-lsp

Result of nixpkgs-review pr 343801 run on x86_64-linux 1

1 package built:

c3-lsp

looks good to me! good use of sourceRoot to avoid all the vscode stuff.

Result of nixpkgs-review pr 343801 run on x86_64-linux 1

1 package built:

c3-lsp

Result of nixpkgs-review pr 343801 run on x86_64-darwin 1

1 package built:

c3-lsp


feat: add dependency checks to PRs and issues

GitHub does not (yet) have the ability to block PRs / issues as dependencies of other PRs issues. This commit uses gregsdennis/dependency-action as a simple workaround.
Now, wording like "blocks #123" or "depends on #123" will add a check for the completion of the dependent PR / issue.
New GH action used: https://github.com/marketplace/actions/pr-dependency-check
GH feature request with the currently available workarounds: https://github.com/orgs/community/discussions/4477#discussioncomment-1039627

This sounds nifty.
Will this cover only the initial issue / PR message, only the title, or any comment?
@RedXanadu Where do we have to document this?

If it happens via a first comment update, that's cool. And I do not mind either way. We just need to know - and document it.

Yes. Although I see this primarily as an tool for us core devs, so putting it in the Wiki is probably enough.

Would not it be part of the documentation that also covers the CI/CD testing?

Is there such a thing? I couldn't find anything in the docs...

I thought @RedXanadu would find something...
If not, it may be time to start a new section in one of the docs.

Sorry, late to the party.
This should be documented as an option under "Making Changes" in the "Contribution Guidelines", e.g.:

If an issue or PR is dependent on another then this relationship can be expressed, in a way that our GitHub repository will understand, by writing "depends on #1234" or "blocked by #1234" in the opening comment.

Is that a correct understanding of what has been proposed here?
Separately, it would be good for someone to document all of the GitHub Lang stuff that underpins CRS in a wiki page, as I would bet that a majority of CRS devs don't have a clue about it… (I certainly don't). Some internal-facing reference material would be very welcome. Does anyone have both the knowledge and the time to write that?
@theseion How does this work for issues, as they have no checks to fail? I tested it on the first comment of an issue and couldn't see a change.

Is that a correct understanding of what has been proposed here?

Yes.

Separately, it would be good for someone to document all of the GitHub Lang stuff that underpins the internal CRS project/repo workings in a wiki page, as I would bet that a majority of CRS devs don't have a clue about it… (I certainly don't). Some internal-facing reference material would be very welcome. Does anyone have both the knowledge and the time to write that?

I can help with that, although I'm not sure what you want covered. Can you give me a list of topics?

@theseion How does this work for issues, as they have no checks to fail? I tested it on the https://github.com/coreruleset/coreruleset/issues/3288#issue-1873131754 and couldn't see a change. (Tested it on a PR and it worked as intended.)

TBH, I only made it work for PRs. But it should only be a small thing to add to issues.

Hmm… I think that, for any time we use a GitHub Workflow/Action/Automation, we should have an internal note (for the attention of CRS devs) stating what it is and what maintenance or modifications it may need. So, I think a list of topics would be everything we're currently doing with GitHub magic, so that the knowledge can be decentralised. I've added this as a workshop suggestion for the retreat, as I think this would be better as an in-person group discussion/workshop, as the aim should be to spread the knowledge of these Workflows and how they function.

Good idea. Definitely better to discuss in a group.


incorporate more info in the embed object

add the time of post
include images

partially solved in commit ca72f7459dd152b5304701c44ba4aef52f16246e


Add full sorting merge join algorithm issue_953

Add full sorting merge join algorithm
#953
This PR is just part1, pls wait..

@nudles will help on review this PR


Tables

Some SetState is missing, I will fix them if this approach is okay. AttackBy() isn't a good implementation, I tried to make "the place things on the table by clicking on it" , but this isn't the right one.

For human interface something like this is okay?    ```
virtual InterfaceBase* GetInterface() override { return GetHumanInterface();}
HumanInterface* GetHumanInterface(){return dynamic_cast<HumanInterface*>(&interface_);}

dynamic_cast is not needed here. Just return &interface_; will be enough.

I think every change is done I will work on states later.

Bullets should fly above the table so changing the passable level of them is a good idea?

Yes, it is a good idea.

Small item or small creature?

I think that small item is more suitable for bullets.


SAML authentication: error de-serializing response context with 4.x

branch 4.x

AAC cannot de-serialize authenticationContext from SAML response
https://github.com/scc-digitalhub/AAC/blob/23942dd284fdc0ea23347f8f6a0817ca93992595/src/main/java/it/smartcommunitylab/aac/saml/auth/SamlWebSsoAuthenticationFilter.java#L149
error:
it.smartcommunitylab.aac.saml.auth.SamlWebSsoAuthenticationFilter.attemptAuthentication
org.springframework.security.saml2.provider.service.authentication.Saml2AuthenticationException: Wrong destination for response
at it.smartcommunitylab.aac.saml.auth.SamlWebSsoAuthenticationFilter.attemptAuthentication(SamlWebSsoAuthenticationFilter.java:155) ~[classes/:na]

The linked file does not match current 4.x branch.
After 8783cfa71d9dac776d62867b1b8bd1eaf207d1fe we use another request context.
Please check the version.
See https://github.com/scc-digitalhub/AAC/blob/8783cfa71d9dac776d62867b1b8bd1eaf207d1fe/src/main/java/it/smartcommunitylab/aac/saml/auth/SamlWebSsoAuthenticationFilter.java#L149

that the line where AAC hangs, I also check the commit https://github.com/scc-digitalhub/AAC/commit/8783cfa71d9dac776d62867b1b8bd1eaf207d1fe from git log
that's the line in the 4.x code
https://github.com/scc-digitalhub/AAC/blob/8783cfa71d9dac776d62867b1b8bd1eaf207d1fe/src/main/java/it/smartcommunitylab/aac/saml/auth/SamlWebSsoAuthenticationFilter.java#L149

hoping it will help:
https://stackoverflow.com/questions/72508155/spring-saml2-and-spring-session-savedrequest-not-retrieved-cannot-redirect-to
I try to change SameSite cookie value from Lax to None and it's works.


How do we "actually" train new models?

I read the theory, but are there any scripts or programs that will do this synthetic data generation for us?  Is that something that will be provided in a future release?
Thanks,
Ryan

@ryanlath Thanks for the request, this is indeed something I am preparing to provide in an upcoming release, including a detailed example showing the data preparation and training process.
In the meantime, I've released the synthetic data generation code in a separate repository here: https://github.com/dscripka/synthetic_speech_dataset_generation
Feel free to experiment with that code, and if you run into any problems create an issue on that repo and I'll try to help resolve it.

@ryanlath, first version of an example notebook for training new models is now in the repo: https://github.com/dscripka/openWakeWord/blob/main/notebooks/training_models.ipynb
Going to close this issue, but if you have other questions or feedback about the tutorial and training custom models, feel free to add them to this discussion thread: https://github.com/dscripka/openWakeWord/discussions/11


gitlab-container-registry, codeberg-pages: delete files in postPatch rather than with patch file

Description of changes
Less data in nixpkgs, and the derivation is less sensitive to changes in the deleted files come the next version bump.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 308126 run on x86_64-linux 1

2 packages built:

codeberg-pages
gitlab-container-registry


Please invite me to the Recode-Hive GitHub Community Organization

Name
Srinivas
Discord Username (if applicable)
No response
Additional Context
No response

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


NoMFAConsoleSignin should not trigger with SSO authentication

Currently, when logging into the console via the SAML-authenticated landing page, it will trigger the  NoMFAConsoleSignin
event.
Ideally the currenty alarm should check whether SAML provider is defined (or additionalMFA is used):
"{($.eventName = \"ConsoleLogin\") && ($.additionalEventData.MFAUsed != \"Yes\") && ($.responseElements.ConsoleLogin != \"Failure\") && ($.additionalEventData.SamlProviderArn NOT EXISTS) }"

Related links:

https://github.com/amazon-archives/aws-security-benchmark/blob/master/architecture/create-benchmark-rules.yaml#L1718
https://github.com/MicrosoftDocs/azure-docs/issues/35206

Please open a PR with the required changes.

I don't know really how to proceed with this alert, maybe it makes sense to create 2 different ones to differentiate between these IAM and SSO logins?
For example, login with IAM user with MFA (some generic fields dropped):
{
    "userIdentity": {
        "type": "IAMUser",
        "principalId": "...",
        "arn": "arn:aws:iam::__AWS_ACCOUNT__:user/username",
        "accountId": "__AWS_ACCOUNT__",
        "userName": "username"
    },
    "additionalEventData": {
        "LoginTo": "https://console.aws.amazon.com/console/home?hashArgs=%23&isauthcode=true&state=hashArgsFromTB_us-west-1_...",
        "MobileVersion": "No",
        "MFAUsed": "Yes"
    },

And login with SSO (MFA isn't enforced every time you use it, but is required when you do the initial login to SSO provider):
{
    "userIdentity": {
        "type": "AssumedRole",
        "principalId"<EMAIL_ADDRESS>        "arn"<EMAIL_ADDRESS>        "accountId": "__AWS_ACCOUNT__",
        "sessionContext": {
            "sessionIssuer": {
                "type": "Role",
                "principalId": "...",
                "arn": "arn:aws:iam::__AWS_ACCOUNT__:role/aws-reserved/sso.amazonaws.com/AWSReservedSSO_..._...",
                "accountId": "__AWS_ACCOUNT_",
                "userName": "AWSReservedSSO_..._..."
            },
            "webIdFederationData": {},
            "attributes": {
                "creationDate": "2022-08-16T06:58:58Z",
                "mfaAuthenticated": "false"
            }
        }
    },

So, to filter out IAM users without MFA:
{
  ($.eventName = "ConsoleLogin") &&
  ($.userIdentity.type = "IAMUser") &&
  ($.additionalEventData.MFAUsed != "Yes") &&
  ($.responseElements.ConsoleLogin != "Failure")
}


MVP-ready website scraping

Currently, the oracle can scrape websites and collect a sentiment analysis result automatically (from LLMs) #166 #163 #167 #187.
We want to make sure this feature is mvp-ready before launching testnet, and asses the quality of it before release.
It seems the /data/web endpoint is buggy - see the notes below.
Acceptance item

[ ] Make sure that when depth is set to 0, it defaults actually to 1.
[ ] QA
[ ] Identify bugs and open follow-ups if needed, or if chores, fix the bugs found along the way

Notes

By default,  depth is set to 0, and the scraping request never returns (depth=0 seems to be invalid)

@nolanjacobson are you currently working on this?

@mudler  - yes

@nolanjacobson i assigned you to this task now

@nolanjacobson can you please reference the issues when opening up PRs? thank you!
PR: https://github.com/masa-finance/masa-oracle/pull/355
@nolanjacobson is left now QA and documentation, right?

@mudler - correct.

needs documentation card @nolanjacobson

@Luka-Loncar  - can you create a ticket to do docs for worker scraper?

Follow-up card created in https://github.com/masa-finance/masa-oracle/issues/397. Closing this one


add create-react-app example

this doesn't actually work...

@alexeagle if you don't mind take a look, here is an example repository where I was able to get react-scripts to work with aspect_rules_js.

https://github.com/heartless-clown/bazel_aspect_rules_js_react

Though the solution is far from ideal, maybe it will be helpful in making a proper react-scripts example.

@alexeagle you were probably looking at the wrong branch.

ejected
non-ejected

Yeah that was it, thanks. Do you know what was the critical fix/fixes to get past the webpack syntax error in my branch?

Looking over what you did, it's very useful thanks!

the solution is far from ideal
anything specific you had in mind that should be improved?

I'll take a look at porting the fixes to this repo.

For anyone reading through the history, Greg is really the expert here and should be consulted with any and all questions 😜


Edimax component reports wrong power values

Home Assistant release (hass --version):
$ docker images | grep home-assistant
homeassistant/home-assistant        latest              b2f52230bed7        6 days ago          1.64GB

$ docker exec -ti home-assistant python -m homeassistant --version
0.64.3

Python release (python3 --version):
$ docker exec -ti home-assistant python --version
Python 3.6.4

Component/platform:
edimax
Description of problem:

Power now (attribute current_power_w) is reported in MW instead of W.
Power day (attribute today_energy_kwh ) is reported in MWh instead of kWh.

Expected:

An attribute named current_power_w contains the value in W, not in MW.
An attribute named today_energy_kwh contains the value in kWh, not in MWh.

Additional info:

Firmware 2.03 / 2.08 reports the NowPower value in W, NowEnergy.Day in kWh
homeassistant/components/switch/edimax.py#L86 should not divide by 1M
homeassistant/components/switch/edimax.py#L86 should not divide by 1k

CC @andreipop2005
Hmm, this does seem like an issue, but I don't own any device of this platform, so I can't do a fix. Good use case of https://github.com/home-assistant/architecture/issues/10 though.

I will take a look at the bug and try to fix it.

Indeed, the units were off. Fixed it, I just hope that previous firmware uses the same measures - I own 2 such power plugs but both of them are updated. Anyway, I guess it is better to be compliant with latest firmware.


The "Move symbol to ..." refactoring displaces trailing comments  in the destination module (including type: ignore, etc.)

Overview
The "Move symbol to ..." refactoring displaces trailing comments in the destination module. This is not so bad for whole-line comments, but it also "steals" in-line comments like type: ignore or noqa... from the last code line in the file.
Environment data

Language Server version: 2023.3.30 (pyright 68a8e98a)
OS and version: Windows 10 Pro 22H2
Python version: 3.11.2

Code Snippet
This is reproducible by having the following top-level modules in the working package, say src/myproject and having Pylance recognize this package as being able to be refactored with the "Move symbol to ..." refactoring. See the reproduction below.
src/myproject/source.py
b = 2
<newline at end of file>

src/myproject/destination_with_trailing_comment.py
a = 1  # type: ignore
<newline at end of file>

Repro Steps

Set "python.analysis.refactoring.allowMovingSymbols": true, in settings.json
Try to move a symbol to a file with a trailing in-line comment
The moved symbol will "steal" the trailing in-line comment from its original target

See also in the behaviors below some additional unnecessary newlines, but these are easily managed by formatting on save. The actual relevant structural bug is type-checking/linter-relevant comments are displaced.
Expected behavior
Ideally the resulting files would look like the following. If the underlying implementation for moving symbols is fundamentally incapable of preserving trailing in-line comments (tokenizer/syntax tree funny business, etc.), perhaps a toast/notification should at least be raised in the VSCode interface that says something like "Trailing in-line or full-line comment in destination module may have been displaced" if the destination file has a trailing comment detected by regex.
source.py
<newline at end of file>

destination_with_trailing_comment.py
a = 1  # type: ignore
<blank line>
b = 2
<newline at end of file>

Actual behavior
source.py
<blank line>
<newline at end of file>

destination_with_trailing_comment.py
a = 1
<two blank lines>
b = 2  # type: ignore
<newline at end of file>

Logs
Will provide logs upon request, but I don't think it will be necessary.

Original
source.py
newfile.py

thank you for reporting the issue. working on the fix!

This issue has been fixed in prerelease version 2023.4.11, which we've just released. You can find the changelog here: CHANGELOG.md
It should be noted that it places two blank lines still (that's on purpose at the moment). #4202 was logged to address this.


Pronunciation audio for non-native speakers

Would you consider adding pronunciation help audio for non-native speakers? Something like this

I came across a free Indic TTS engine recently built by IITM (https://www.iitm.ac.in/donlab/tts/demo_tts.php). I think this will be a great add to Olam as well.
Let me know if you guys are up with it, I would love to modify this dictpress theme and raise a PR!

Sorry, missed this issue. Yes please, that would be great.

Ah yes, that's right. dictpress has moved on to v1.1.0 with major breaking changes but Alar is still running an old version. I've to upgrade it at some point.
Please try this release https://github.com/knadh/dictpress/releases/tag/v0.3.0

Running into the same issue with v0.3.0 too :(

Darn, sorry about that. Upgrading Alar to the latest version of dictpress is on my to-do. I'll try to do that when I get a chance.
In the meanwhile, I've been thinking of using one of the many speech text-to-speech APIs out there (the Microsoft Azure ones seem very high quality for Indic languages) for every single entry in Olam and Alar to create an open data voice corpus. That can thus be directly integrated into the dictionary instead of linking to a 3rd party service.

I would love to help in building the voice corpus! Please let me know!
On Mon, 5 Jun 2023 at 12:29 PM, Kailash Nadh @.***>
wrote:

Darn, sorry about that. Upgrading Alar to the latest version of dictpress
is on my to-do. I'll try to do that when I get a chance.
In the meanwhile, I've been thinking of using one of the many speech
text-to-speech APIs out there (the Microsoft Azure ones seem very high
quality for Indic languages) for every single entry in Olam and Alar to
create an open data voice corpus. That can thus be directly integrated into
the dictionary instead of linking to a 3rd party service.
—
Reply to this email directly, view it on GitHub
https://github.com/alar-dict/alar.ink/issues/10#issuecomment-1576154805,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALCVD3J53F4KBUJ5FF5TJ3TXJV7VTANCNFSM6AAAAAAYATTLHY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Aditya D.S.

For Alar, it can be as simple as reading the YML file, passing every Kannada word to one of those speech APIs, collecting the voice response and saving it somewhere. Actually, if this can be made as a standalone CLI or something that connects to the dictpress Postgres database and scans words and does the same, then it can be generally useful.

Cool, will work on it this week!
On Mon, 5 Jun 2023 at 12:53 PM, Kailash Nadh @.***>
wrote:

For Alar, it can be as simple as reading the YML file, passing every
Kannada word to one of those speech APIs, collecting the voice response and
saving it somewhere. Actually, if this can be made as a standalone CLI or
something that connects to the dictpress Postgres database and scans words
and does the same, then it can be generally useful.
—
Reply to this email directly, view it on GitHub
https://github.com/alar-dict/alar.ink/issues/10#issuecomment-1576201914,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALCVD3MC3TCAZ2M3RDUJZT3XJWCP7ANCNFSM6AAAAAAYATTLHY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Aditya D.S.


Setting quantity for subscriptions

I see that there is currently no capabilities for specifying the quantity for subscriptions - would that be something that would be merged in or are you not interested in that functionality?
Any tips on how to go about adding it?

Hi!
I'm not really sure about what you mean by quantity, tell me if I'm completely off the point.
Unfortunately, we're always gonna be limited by the way subscriptions are set up on Stripe. You can refer to Stripe docs for subscriptions for further information on their subscription flow.
They have no concept of quantity whatsoever, which means that using subscriptions with stripe, you have to "pre-make" subscription plans for each quantity you want you are willing to provide.
I'm using subscriptions with quantities as well, but they are preset plans on Stripe.


Extend test coverage for clClonekernel

Add the following test coverage for the clCloneKernel
With args:
Use clSetKernelArg to set args after kernel is cloned. Enqueue and
read the buffer to validate.
With Execinfo:
Use clSetKernelExecInfo after the kernel is cloned and read the buffer
to validate.
With no args:
Clone a kernel with no args and enqueue.
With SVM pointer:
Use clSetKernelArgSVMPointer to set args after kernel is cloned.
Enqueue and read the buffer to validate.
This fixes #1098 and #1234
This PR also replaces test_error with test_assert_error to report the error during failure.

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

I'm seeing
fatal: unable to access 'https://github.com/KhronosGroup/libclcxx.git/': Could not resolve host: github.com
Command exited with code 128

How can we restart this?

@gwawiork, please review.

I'm seeing
fatal: unable to access 'https://github.com/KhronosGroup/libclcxx.git/': Could not resolve host: github.com
Command exited with code 128

How can we restart this?

I had the same issue on another PR. I ended up re-pushing (on the same branch) to retrigger another run. Hopefully there's a more clever alternative with a simple button to press somewhere, but I couldn't find it.

Forced pushed to rebase.

Forced pushed again, CI was stuck

Hi @jainvikas8
I've check your changes on Windows on Intel GPU  and I am getting
ERROR: clFinish failed! (CL_OUT_OF_RESOURCES from C:\NewVPG\Repo\public-github\opensource-qb\OpenCL-CTS\test_conformance\api\test_clone_kernel.cpp:524) test_exec_enqueue_helper failed for cloneKernel_2 on retry. clCloneKernel test_cloned_kernel_exec_info failed. clone_kernel FAILED
The test process is not able to exit looks like hang. Could you double check on your side if everything is ok ? I will analyze the problem as well.

Forced pushed to rebase.

Hi @jainvikas8
I've check your changes on Windows on Intel GPU and I am getting
ERROR: clFinish failed! (CL_OUT_OF_RESOURCES from C:\NewVPG\Repo\public-github\opensource-qb\OpenCL-CTS\test_conformance\api\test_clone_kernel.cpp:524) test_exec_enqueue_helper failed for cloneKernel_2 on retry. clCloneKernel test_cloned_kernel_exec_info failed. clone_kernel FAILED
The test process is not able to exit looks like hang. Could you double check on your side if everything is ok ? I will analyze the problem as well.

Have not observed this in the past, normally it should fail for this too.
 test_exec_enqueue_helper failed for cloneKernel_1 on the retry
Will look into reproducing this issue. Thanks.

Forced pushed to rebase and add 2 new commits to address the review comments.

Hi @jainvikas8
I've check your changes on Windows on Intel GPU and I am getting
ERROR: clFinish failed! (CL_OUT_OF_RESOURCES from C:\NewVPG\Repo\public-github\opensource-qb\OpenCL-CTS\test_conformance\api\test_clone_kernel.cpp:524) test_exec_enqueue_helper failed for cloneKernel_2 on retry. clCloneKernel test_cloned_kernel_exec_info failed. clone_kernel FAILED
The test process is not able to exit looks like hang. Could you double check on your side if everything is ok ? I will analyze the problem as well.

Have not observed this in the past, normally it should fail for this too.
 test_exec_enqueue_helper failed for cloneKernel_1 on the retry
Will look into reproducing this issue. Thanks.

@gwawiork I've been unable to reproduce this issue on MALI-GPU in Linux with OpenCL 2.1 and 3.0. So, please let me know your thoughts on this?

@jainvikas8 Please take a look at my review when you have the time.

[Qualcomm] We tested this and it passed on our 3.0 implementation. Will need more time for a detailed review.

Removing "focused review" until this PR has an updated owner.

@kpet Rebased and CI checks are passing.
@bashbaug Addressed outstanding review comments.


Hactoberfest

Hi @prajnakalpa It looks like there are some conflicts. Please resolve them and the PR will be merge


Bitcoin syncer silent failure using ssl://blockstream.info:993

Issue I described in IRC before already: when using ssl://blockstream.info:993 as the electrum server on testnet, without any other logged failures, start receiving only empty messages from blockstream.info. Thereafter, don't receive any real messages (such as blockheight updates) from the electrum server anymore, so all running swaps stall silently (except for trace logging hints).
extract of logs:
[2022-08-31T11:23:27Z TRACE microservices::peer] Peer message processing complete
[2022-08-31T11:23:27Z TRACE microservices::peer] Awaiting for peer messages...
[2022-08-31T11:23:27Z DEBUG microservices::peer::connection] Awaiting incoming messages from the remote peer
[2022-08-31T11:23:27Z TRACE microservices::esb::controller] Received ESB request from 1 service busses...
[2022-08-31T11:23:27Z DEBUG microservices::esb::controller] loopback -> peerd<020615001ab19d153c15e0c6c91ca3128fb56a91ecdeecd537640c0861464f5532@<IP_ADDRESS>:18750>: protocol_message(pong(..))
[2022-08-31T11:23:27Z DEBUG farcaster_node::peerd::runtime] BRIDGE RPC request: protocol_message(pong(..))
[2022-08-31T11:23:27Z TRACE farcaster_node::peerd::runtime] Got pong reply, exiting pong await mode
[2022-08-31T11:23:27Z TRACE microservices::esb::controller] request processing complete
[2022-08-31T11:23:27Z TRACE microservices::esb::controller] Awaiting for ESB request from 3 service buses...
[2022-08-31T11:23:27Z TRACE electrum_client::raw_client] ==> {"jsonrpc":"2.0","id":299,"method":"server.ping","params":[]}
[2022-08-31T11:23:27Z TRACE electrum_client::raw_client] Starting reader thread with `until_message` = Some(299)
[2022-08-31T11:23:27Z TRACE electrum_client::raw_client] <== "{\"id\":299,\"jsonrpc\":\"2.0\",\"result\":null}\n"
[2022-08-31T11:23:27Z TRACE electrum_client::raw_client] Reader thread 299 received a response for its request
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::conn] Conn::read_head
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::io] received 310 bytes
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::role] parse_headers;
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::role] -> parse_headers;
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::role] Response.parse bytes=310
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::role] Response.parse Complete(160)
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::role] <- parse_headers;
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::role] -- parse_headers;
[2022-08-31T11:23:28Z DEBUG hyper::proto::h1::io] parsed 5 headers
[2022-08-31T11:23:28Z DEBUG hyper::proto::h1::conn] incoming body is content-length (150 bytes)
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::decode] decode; state=Length(150)
[2022-08-31T11:23:28Z DEBUG hyper::proto::h1::conn] incoming body completed
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::conn] maybe_notify; read_from_io blocked
[2022-08-31T11:23:28Z TRACE want] signal: Want
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::conn] flushed({role=client}): State { reading: Init, writing: Init, keep_alive: Idle }
[2022-08-31T11:23:28Z TRACE want] signal: Want
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::conn] flushed({role=client}): State { reading: Init, writing: Init, keep_alive: Idle }
[2022-08-31T11:23:28Z TRACE hyper::client::pool] put; add idle connection for ("http", localhost:38083)
[2022-08-31T11:23:28Z DEBUG hyper::client::pool] pooling idle connection for ("http", localhost:38083)
[2022-08-31T11:23:28Z DEBUG reqwest::async_impl::client] response '200 OK' for http://localhost:38083/json_rpc
[2022-08-31T11:23:28Z TRACE hyper::client::pool] take? ("http", localhost:38083): expiration = Some(90s)
[2022-08-31T11:23:28Z DEBUG hyper::client::pool] reuse idle connection for ("http", localhost:38083)
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::role] encode_headers;
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::role] -> encode_headers;
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::role] Client::encode method=POST, body=Some(Known(466))
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::role] <- encode_headers;
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::role] -- encode_headers;
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::encode] sized write, len = 466
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::io] buffer.queue self.len=116 buf.len=466
[2022-08-31T11:23:28Z DEBUG hyper::proto::h1::io] flushed 582 bytes
[2022-08-31T11:23:28Z TRACE hyper::proto::h1::conn] flushed({role=client}): State { reading: Init, writing: KeepAlive, keep_alive: Busy }
[2022-08-31T11:23:28Z TRACE electrum_client::raw_client] <== ""
[2022-08-31T11:23:28Z TRACE electrum_client::raw_client] <== ""
[2022-08-31T11:23:28Z TRACE electrum_client::raw_client] <== ""
[2022-08-31T11:23:28Z TRACE electrum_client::raw_client] <== ""
[2022-08-31T11:23:28Z TRACE electrum_client::raw_client] <== ""
[2022-08-31T11:23:28Z TRACE electrum_client::raw_client] <== ""
[2022-08-31T11:23:28Z TRACE electrum_client::raw_client] <== ""
[2022-08-31T11:23:28Z TRACE electrum_client::raw_client] <== ""
[2022-08-31T11:23:28Z TRACE electrum_client::raw_client] <== ""
[2022-08-31T11:23:28Z TRACE electrum_client::raw_client] <== ""
[2022-08-31T11:23:28Z TRACE electrum_client::raw_client] <== ""

This failure's distinct from #628 and does not happen with electrs v0.8.9 or v0.9.7, so I'm guessing it's related to blockstream's fork. I will spin up an instance and see if I can reproduce.

Do you still experience this issue or did #735 also fixed it on your side? (On my side it did)

Ping @Lederstrumpf

Yup, I haven't seen this issue crop up since #735 at all, so we can safely assume the bump resolved it.


Full screen button does not work in Google Chrome

Checkout this page:
http://mashable.com/2015/07/17/schumer-thrones-google-emmys/
Chlick on the fullscreen button of the map and nothing happens.
Looking the error console I can see this.
Which says "requestFullscreen() is deprecated on insecure origins, and support will be removed in the future. You should consider switching your application to a secure origin, such as HTTPS. See https://goo.gl/rStTGz for more details."

Basically fullscreen feature is not working under http protocols anymore. I'm looking for a way to know if we could "hide" the fullscreen button when origin is http.

This bug has to be fixed in CartoDB.js -> https://github.com/CartoDB/cartodb.js/pull/674


[C#] Grpc.Tools 2.40.0-pre1 error

This issue - https://github.com/grpc/grpc/issues/26819 - is back in 2.40-pre1.
I think Grpc.Tools needs some kind of automated testing.

Good catch. Looks like https://github.com/grpc/grpc/pull/26896 was merged after the v1.40.x branch was cut.
We do have some test coverage for Grpc.Tools (e.g. here: https://github.com/grpc/grpc/blob/master/test/distrib/csharp/run_distrib_test_dotnetcli.sh, building DistribTestDotNet.csproj also includes Grpc.Tools codegen)
I'm going to investigate a bit why these tests didn't catch the problem.

Ok so I think the reason the distribtests didn't catch the problem is that they do include a codegen with Grpc.Tools, but it's only a simple one and it doesn't include any custom grpc_csharp_plugin options, so the for loop that's impacted by the bug never gets to run:
https://github.com/grpc/grpc/pull/26896/files#diff-a84435c6492be9bfe4fe4663d238de49ffc1caeb462a601f690c2fc0bd049920R49
https://github.com/grpc/grpc/blob/013a45ccc4f0cc99532cb3148ca6ed3f60996711/test/distrib/csharp/DistribTest/DistribTestDotNet.csproj#L30

Closing with a note that we should improve the Grpc.Tools distribtests to catch similar problems in the future.


Updating C implemintation of Gaussian Elimination.

@antonte What questions about the original implementation?

@leios they are minor:

it feel unsafe to use size_t since we are iterating backward, I would prefer to use int
declaring 2d array and casting it to double*
using 3 and 4 instead of declaring constants cols and rows

Screwed up with git a bit, but this is in the AAA: https://github.com/algorithm-archivists/algorithm-archive/commit/f110a471985d3063f97c37b53de10146fddff5fa
Thanks!


Chinese translation

That's it. An afternoon's work.
Some keys are left untranslated because I haven't seen those displaying in the game.
Also I'm confused that some text in the game can not be found in the keys (e.g. the text displayed when trying to place a wind mill without enough room). They might be hardcoded. (Also mentioned #30 )
Anyway, if no problems exist, it should be the translation seen in the game.
EDIT: Many of the translating keys are from the Chinese translation in Better With Mods.

Hey thanks for the contribution!
I'm going to close this issue, but that doesn't mean your contribution is being rejected. It's just a result of how the source code is managed, since the source in this repo is automatically generated from the source we use for development. Because of that, the code needs to be added to the development repo, which for legal reasons cannot be shared publicly (due to the substantial amount of Mojang's code present in BTW).
I've sent you an invite to the organization so you can see the source repo. Feel free to resubmit a PR there with your language file and I'll merge it in.


EXCEPTION: Uncaught (in promise): TypeError: this.once is not a function

Hi,
First, thanks for sharing this! Had no problems cloning and running your code.
The problem:
I would like to incorporate this code on a SPA, in visual studio 2017, I made following this instructions: https://jonhilton.net/2017/02/21/create-a-vs2017-angular-2-and-net-core-site-using-the-command-line/ and https://channel9.msdn.com/Events/Visual-Studio/Visual-Studio-2017-Launch/WEB-103
I’m stuck now on this error “EXCEPTION: Uncaught (in promise): TypeError: this.once is not a function” that happens in “mqtt.service.ts” when creating the connection:
// Create the client and listen for its connection
this.client = mqtt.connect(url, options);

I’m stuck here.
The attached file is the solution I’m using minus .vs, .git and node_modules folders.
Probably it's just a newbie problem but I’m really stuck here for quite some time (shame to say)
monitor.zip
best regards,
Manuel Alves

Sorry for bothering you. Newbie mistake... Please close this “issue”.
My package.json was missing:
    "core-js": "^2.4.1",
    "mqtt": "^1.14.1"

Once again, thank you for sharing this code.
Best regards
Manuel Alves

Thanks for posting your solution, glad you found the issue.


Perform Proxy logic more like cURL

closes #3904
Types of changes
Introduces a helper function getProxyForUrl that computes the proxy to use based on Env Vars and options provided in accordance with the discussion and linked articles in the issue
Proxy Variables Handling:

No proxy variables: The function correctly returns an empty string for various protocols when no proxy variables are set.
Invalid URLs: The helper function returns an empty string for invalid URLs.
Proxy options:

allowUpperCase:false: Prevents *_PROXY values being returned when set to false.
favourUpperCase:false: Causes *_PROXY to be used before *_proxy when set to false. The expectation is that lowercase are preferred and get priority
includeNpm:false: Does not return npm_config_*_proxy env vars when set to false.
legacyMode:false: Processes URLs proxy in node-red <= v3.1 compatibility mode.

Proxy Environment Variables:

Handles environment variables like http_proxy, HTTP_PROXY, https_proxy, HTTPS_PROXY, ftp_proxy, ws_proxy, mqtt_proxy, all_proxy, no_proxy, etc.
Properly handles different configurations of these variables, including lowercase has priority over uppercase, scheme handling, with/without port, default port numbers (e.g. http:80), IP addresses (v4/v6), host suffix, wildcard matching, and case sensitivity.
Takes precedence of some variables over others based on the configurations.

NPM Proxy Configuration:

Works with npm_config_http_proxy, npm_config_https_proxy, npm_config_proxy, npm_config_no_proxy.

Handles precedence of NPM proxy configurations over environment variables like HTTP_PROXY, HTTPS_PROXY, NO_PROXY.

Overall, the "Proxy Helper" function effectively handles various proxy configurations and environment variables, providing consistent and expected results according to the specified configurations.

[x] Bugfix (non-breaking change which fixes an issue)

[x] New feature (non-breaking change which adds functionality)

Proposed changes

HTTP, ws and MQTT nodes will all determine the proxy by calling to this new helper function
HTTP node will compute the proxy for dynamic URLs (currently only computed against the nodes edit panel URL field)
support overriding with system env var NR_PROXY_MODE (setting to legacy reverts functionality)

TODO

[ ] Tests & Docs for NR_PROXY_MODE

Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run npm run test to verify the unit tests pass
[x] I have added suitable unit tests to cover the new/changed functionality

When are the npm config options used? I can't see any reference to them in the original issue other than when you bring them into the discussion and google searches don't really reveal anything about how they expected to be used beyond npm's own use.

When are the npm config options used? I can't see any reference to them in the original issue other than when you bring them into the discussion and google searches don't really reveal anything about how they expected to be used beyond npm's own use.

If node-red is launched via NPM and the npmrc (or CLI) has specified proxy options, I believe the correct thing to do is respect them. In the interest of getting this in, I could disable the npm proxy parts (there is an option excludeNpm we could default to true
There are a number issues and inconsistencies this covers including:

In the http-request node, the proxy URL is only determined when the node is instantiated - meaning when using a dynamic URL, the proxy is NOT re-evaluated per request
no_proxy should not be case-sensitive
no_proxy with protocols that have a default port are not considered
~Does not consider NPM proxy configuration at all~ (we could ignore this for now as discussed per above)
Doesn't consider https_proxy or HTTPS_PROXY
Incorrectly prioritises HTTP_PROXY over http_proxy. HTTP_PROXY is not always supported or recommended
doesn't consider all_proxy or ALL_PROXY

One concern I have is how many settings this is added to the settings file - for a topic that has very rarely caused any questions or issues.
Why can we not just have a standard behaviour in terms of upper-case and lower-case? Document which we will use in preference to the other and not expect users to have to understand all of these flags that have been added to the settings file.

We could be opinionated and expose the finer control at a later if that is an issue.
I mean, we dont really have to add anything at all to the settings file (since it has defaults) but we may wish to keep the mode: "legacy" flag listed out as a formative option.
e.g. settings.js
    // proxyOptions: {
    //     mode: "legacy", // legacy mode is for non-strict previous proxy determination logic (node-red < v4 compatible)
    // },

That way, the user can be least change back to legacy (if needed) and the unlisted options are considered "unsupported"/"subject to change"


ssh: must specify HostKeyCallback

Hi
When trying to upload via C14 CLI I get this error ..
root@vrayo:~# c14 upload /etc/ 6cd0ab4e-xxxx-4fe8-8207-0614fc5f494b
FATAL[0000] ssh: must specify HostKeyCallback
Host runs Ubuntu 16.04.2 LTS
Go version : go version go1.6.2 linux/amd64
Any idea on how to fix this ? :)
thanks
Gonzague

go 1.8.1 on arch linux here.
Can repro :/

You do not have a specific configuration in your ssh? .ssh/config file
@gonzague

@flemzord I do not. I wasn't told to do that.

@agirot can you fix it ?

@flemzord : nope super basic setup

@gonzague Can you pull and try again plz ? (#34)

Confirmed fixed here.

Fixed :) thanks


Transliteration issue with ஜ character not being transliterated

Hello, I opened an issue a few weeks ago about the ந் character not being transliterated (here), which was quickly fixed (Thank you!). However, it looks like I found another character which also does not get transliterated, which is the ஜ character. Here is an example:

As seen in the print statements at the bottom, it doesn't seem to change. Hopefully this is an easy fix as well. Thanks again!

@CRE2525 - thanks for bug report; what is the correct transliteration according to ISO? Is it just "j" or "ja" ?

According to this website, it's 'ja'; however, I do not speak Tamil so I can't verify that.

@CRE2525  - it is JA


Amazing work! Thank you

Just stopping by and saying thank you. This is a very useful and intuitive tool

@gevera Thank you by using it, we appreciate your gratitude. We also are interested on new ideas or issues related to tasks functionality or adapters generation.


document which function calls increment / decrement refcounts for handles

In an earlier meeting, there was some confusion about handle management. I.e. when must HG_Destroy be called? Clearing up this confusion amounts to improving documentation on when handle refcounts are modified. For instance, HG_Forward and HG_Bulk_transfer each increment handle recounts, and the corresponding callbacks are expected to HG_(Bulk_)Destroy (decrement the refcount) when they are finished with it.

Yes that's true the question comes back all the time so that's something we need to clarify. Thanks!

One thing that I think would be good to put down as a "line in the sand" when documenting this, is that the API user should treat HG_Destroy() as a promise that the API user will not touch the handle again.
In other words, the reference counting that's going on is for Mercury's benefit (so that if it needs to live longer to wrap up some in-flight operation it can do so), not something that the caller should rely on as a way to continue poking at the handle after Destroy().


Python 3 support?

The build is not running with Python 3.
Has anyone ever tried running the tests with Python 3? Is there a reason why it's not supported yet?

Its 2015, would be nice to have python 3 support

+1 for Python 3 support

+1

Umm yeah, another request for python 3 support.

+1
└─[$] pip install flanker                                                                                [16:57:38]
Collecting flanker
  Downloading flanker-0.4.38.tar.gz (87kB)
    100% |████████████████████████████████| 90kB 97kB/s
Collecting chardet>=1.0.1 (from flanker)
  Downloading chardet-2.3.0.tar.gz (164kB)
    100% |████████████████████████████████| 167kB 95kB/s
Collecting cchardet>=0.3.5 (from flanker)
  Downloading cchardet-1.0.0.tar.gz (609kB)
    100% |████████████████████████████████| 610kB 853kB/s
Collecting cython>=0.21.1 (from flanker)
  Downloading Cython-0.23.4-cp35-cp35m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl (3.6MB)
    100% |████████████████████████████████| 3.6MB 126kB/s
Collecting dnsq>=1.1.6 (from flanker)
  Downloading dnsq-1.1.6.tar.gz
Collecting expiringdict>=1.1.2 (from flanker)
  Downloading expiringdict-1.1.3.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/sp/yghc86_51z55_7k6_n59s9600000gn/T/pip-build-muknokf8/expiringdict/setup.py", line 2, in <module>
        import md5  # fix for "No module named _md5" error
    ImportError: No module named 'md5'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/sp/yghc86_51z55_7k6_n59s9600000gn/T/pip-build-muknokf8/expiringdict

+1

+1

+1

Hey all--GitHub now supports reactions, so please use that to +1 the issue rather than spamming everyone following it with a text comment of "+1"

Seems like good progress has been made at https://github.com/plq/flanker – would bringing in some of their work be feasible?

My work is mostly based on #106. I think you'd be better off with python3-specific stuff cherry-picked from https://github.com/BayanGroup/flanker/

Hey - it's getting crazy that py3 isn't supported yet.

@punkrokk the bitter truth is that the Mailgun team does not have plans to add support for Python 3. But if somebody wants to contribute the support for Python 3 we are ready to consider it, as long as it does not break 2.7 functionality.

Hey, guys, for those of you looking for a Python 3 email parsing library I implemented addresslib. Unlike flanker, it doesn't implement its own parser. Instead it wraps Python's stdlib parser. There's also some Python 2 support. Cheers!

@alexei good stuff. If I had more time I’d help you and add mimeparsing to it.

Actually the addresslib part of Flanker is already Python 3 compatible.


Unable to locate DeprecationWarning: In future, it will be an error for 'np.bool_' scalars to be interpreted as an index

I'm getting this DeprecationWarning:
DeprecationWarning: In future, it will be an error for 'np.bool_' scalars to be interpreted as an index
    self.cstruct = CStructuredGrid(self.xdim, self.ydim, self.zdim,

I cannot find it in the source code and thus I cannot find what piece of code is triggering the warning. The only thing I can think of is this
VARIABLES = {"U": "uo", "V": "vo"}
DIMENSIONS = {"lat": "latitude", "lon": "longitude", "time": "time", "depth": "depth"}

fieldset = parcels.FieldSet.from_netcdf(
            fnames,
            VARIABLES,
            DIMENSIONS,
            indices=indices,
            mesh="spherical",
            allow_time_extrapolation=True,
)

but I do not see any bools around, since indices is a dict of list of scalars for each dimension. I'm worried that my code will stop working in future versions. Can someone point me in the right direction?

Yes, this is an annoying Warning that has been around for a while. I looked into it earlier, but couldn't find an easy way to fix it. It seems related to these lines
https://github.com/OceanParcels/parcels/blob/0df79e6cea94913e1b1565443e653fe72e87de62/parcels/grid.py#L120-L130
@Ckehl, perhaps you know what's causing this Warning and how to fix it?
For now, it doesn't have an impact on Parcels (yet), but something to fix soon(ish) for sure, before numpy deprecates this feature

Ok, thanks for taking the time to look into this. I thought that it was something that I was doing wrong in my code. I think the change was made in numpy version 1.14 (https://docs.scipy.org/doc/numpy-1.14.0/release.html):

Using np.bool_ objects in place of integers is deprecated. Previously operator.index(np.bool_) was legal and allowed constructs such as [1, 2, 3][np.True_]. That was misleading, as it behaved differently from np.array([1, 2, 3])[np.True_].

It seems it is also happening in other libraries like https://github.com/skyfielders/python-skyfield/issues/486

The issue comes from self.z4d.
Generally for a Grid, we set this variable as an integer value (grid.py, l. 336):
self.z4d = -1  # only used in RectilinearSGrid

But for S-grids (RectilinearSGrid, l. 371; CurvilinearSGrid, l.500), it is set as a boolean value:
self.z4d = len(self.depth.shape) == 4

Therefore, this warning is emitted for SGrids (as far as I can see it now), because of the required implicit-cast from boolean to integer. It could be as easy as writing (for SGrids) instead:
self.z4d = int(len(self.depth.shape) == 4)

or
self.z4d = 1 if len(self.depth.shape) == 4 else 0

Ah thanks @CKehl! I just created a PR with this change #1123

Fixed in #1123


Why do we need to watch in riverpod?

Why do we need to watch dioProvider (https://github.com/bizz84/tmdb_movie_app_riverpod/blob/5f97913891a2aaffd3b49d637d2e49f606fc7e42/lib/src/features/movies/data/movies_repository.dart#L72)
and moviesRepositoryProvider (https://github.com/bizz84/tmdb_movie_app_riverpod/blob/5f97913891a2aaffd3b49d637d2e49f606fc7e42/lib/src/features/movies/data/movies_repository.dart#L86)?
Both are defined as function providers so in practice should never change. Wouldn't we rather read?

According to the official docs:

Whenever possible, prefer using ref.watch over ref.read or ref.listen to implement a feature.
By relying on ref.watch, your application becomes both reactive and declarative, which makes it more maintainable.

Source:
https://docs-v2.riverpod.dev/docs/concepts/reading

Your assumptions may change in the future, so you may as well go with the recommended guidelines.


Lorenz drawing a bit out of scale

Hello there and friendly greetings! I actually don't know if the Lorenz module is supposed to behave like that. It works, don't get me wrong, but the graph as you can see is a bit out of scale and out of module.
I hope this could be useful. Have a great day!

Ha well clearly not.
does it always do that for you? Or just at some settings?
I wonder if that code deals improperly with rack zoom also
thanks


Add Trendy Card Design

Is your feature request related to a problem? Please describe.
Add Trendy Card Design
Describe the solution you'd like
Want to change the basic card's design to a new trendy card design.
Screenshots

Go ahead @aniketsinha2002
Show the love by starring the repo 🌟


Update CONTRIBUTING.md for the fact that Travis is no longer used

Thanks! Good catch

Thanks for making std::simd happen!


Background updates management

Hi,
I'm looking to connect this to some realm db which update from background sometimes. Is there an api to update the layout without disturbing the user (no animations, same selection) ?

I don't understand what you mean. Can you explain your problem? ;-)
Maybe the following code helps:
UIView.performWithoutAnimation {
  // updating the layout
}

In my app I have sometimes background update to my collection data. When this occurs I use collectionView.performBatchUpdates. I'd like to do this without calling animations

UIView.performWithoutAnimation {
// updating the layout
}
is a good start.

The main issue I have is that the selection is not kept when I do an update. I tried to go around with disabling animation the manually selecting the previous selected item but the are still small flash.
My goal is when there is no selection it add a card like in the demo but when there is a selection the selection not changed and the row is added silently

An easy way to see what I'm fighting in the example is to replace :
if(self.colorArray.count == 1 || self.cardCollectionViewLayout!.revealedIndex >= 0) {
self.cardCollectionViewLayout?.unrevealCard(completion: {
self.colorArray.insert(self.getRandomColor(), at: index)
self.collectionView?.insertItems(at: [IndexPath(item: index, section: 0)])
})
} else {
self.colorArray.insert(self.getRandomColor(), at: index)
self.collectionView?.insertItems(at: [IndexPath(item: index, section: 0)])
}
with :
self.colorArray.insert(self.getRandomColor(), at: index)
self.collectionView?.insertItems(at: [IndexPath(item: index, section: 0)])

OK, now i know what you mean. :-)
But it's not so simple to fix.

To achieve the right behavior I have to create my own UICollectionView (extend) and overwrite some methods (insertItems(at indexPaths: [IndexPath])).

Can you post what works for you I'll extend the collection in my project so you don't have to change the structure of your lib

If you only have the problem for inserting items, with the new version you can use the HFCardCollectionView or copy the call of the helper method to your own class.
class HFCardCollectionView: UICollectionView {

    override func insertItems(at indexPaths: [IndexPath]) {
        if let collectionViewLayout = self.collectionViewLayout as? HFCardCollectionViewLayout {
            collectionViewLayout.willInsert(indexPaths: indexPaths)
        }
        super.insertItems(at: indexPaths)
    }

}

I hope this helps. ;-)


Enhance Tensor print

Changes proposed in this PR:

Add tensor address to the print
Handle the case when tensor is too large
Add test

See also: #270
Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #276. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:octocat: cibot: @zhoonit, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.

Other than @wooksong comments, LGTM.


krb5: 1.20 -> 1.20.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/krb5/versions.
meta.description for krb5 is: "MIT Kerberos 5"
meta.homepage for krb5 is: "http://web.mit.edu/kerberos/"
meta.changelog for krb5 is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.20.1 with grep in /nix/store/d6zb92zl1ni3mzbvxq96izl14n7qby3d-krb5-1.20.1
found 1.20.1 in filename of file in /nix/store/d6zb92zl1ni3mzbvxq96izl14n7qby3d-krb5-1.20.1

Rebuild report (if merged into master) (click to expand)

39642 total rebuild path(s)

39642 package rebuild(s)

First fifty rebuilds by attrpath

ArchiSteamFarm
AusweisApp2
BeatSaberModManager
CHOWTapeModel
ChowCentaur
ChowKick
ChowPhaser
CuboCore.coreaction
CuboCore.corearchiver
CuboCore.corefm
CuboCore.coregarage
CuboCore.corehunt
CuboCore.coreimage
CuboCore.coreinfo
CuboCore.corekeyboard
CuboCore.corepad
CuboCore.corepaint
CuboCore.corepdf
CuboCore.corepins
CuboCore.corerenamer
CuboCore.coreshot
CuboCore.corestats
CuboCore.corestuff
CuboCore.coreterminal
CuboCore.coretime
CuboCore.coretoppings
CuboCore.coreuniverse
CuboCore.libcprime
CuboCore.libcsys
DisnixWebService
EBTKS
EmptyEpsilon
Fabric
HentaiAtHome
LAStools
LASzip
LASzip2
LibreArp
LibreArp-lv2
Literate
MACS2
MIDIVisualizer
MMA
NSPlist
OSCAR
OVMF
OVMFFull
PageEdit
PlistCpp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/d6zb92zl1ni3mzbvxq96izl14n7qby3d-krb5-1.20.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A krb5 https://github.com/r-ryantm/nixpkgs/archive/967b87ea6e287814a099f2177f38dbc31211e000.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/d6zb92zl1ni3mzbvxq96izl14n7qby3d-krb5-1.20.1
ls -la /nix/store/d6zb92zl1ni3mzbvxq96izl14n7qby3d-krb5-1.20.1/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings

https://nvd.nist.gov/vuln/detail/CVE-2022-42898


Add Ploi Elephpant

Thank you for contributing to A Field Guide to Elephpants. To speed up approval
of your contribution, please review the following checklist:

[x] I have read the CONTRIBUTING guide
[x] All text and photographs included are licensed under CC:BY-NC-SA or a compatible license.
[x] (If Needed) New subspecies have been added to sculpin_site.yml following the naming convention.

Do you have a description?  Something that briefly describes how the elePHPant came to be and/or how people might acquire them.
Also the Latin name should be different from the actual name.  Fake Latin is OK
And finally the first two photos alone are fine

Do you have a description? Something that briefly describes how the elePHPant came to be and/or how people might acquire them.
Also the Latin name should be different from the actual name. Fake Latin is OK
And finally the first two photos alone are fine

Considering all the open PRs currently, do you think it's worth making these adjustments? I've also seen quite a few PRs from you that are still open and quite old. Has this repository died? 😢

Consider it done @cookieguru 🤗
More information:
https://ploi.io/elephpant
https://twitter.com/ploi_deploy/status/1764986992327077980

Considering all the open PRs currently, do you think it's worth making these adjustments? I've also seen quite a few PRs from you that are still open and quite old. Has this repository died? 😢

It is not dead.  The build process needs to be updated, and in the process of that PHP upgraded as well.  I have encountered a bug in a newer version of Sculpin but haven't had a chance to dig in to it.

Considering all the open PRs currently, do you think it's worth making these adjustments? I've also seen quite a few PRs from you that are still open and quite old. Has this repository died? 😢

It is not dead.  The build process needs to be updated, and in the process of that PHP upgraded as well.  I have encountered a bug in a newer version of Sculpin but haven't had a chance to dig in to it.

Ah, understandable. I'll patiently wait🙌


Add size to created ecdsa ssh keys

When ecdsa keys created with community.crypto.openssh_keypair are used by paramiko, an error is raised (at least with paramiko versions 2.10 and 2.11).
They issue does not occur when they key is generated:

with a non-empty comment
with a size

This could affect tobiko and some tempest plugins (e.g. whitebox-tempest-plugin, which connects to computes).
This patch modifies the ssh keys created by the tobiko role (to connect to the VM instances created by tobiko) and the libvirt_manager role (to connect to the VM it creates) with size=521.
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running
[X] Appropriate documentation exists and/or is up-to-date:

[X] README in the role
[X] Content of the docs/source is reflecting the changes

Maybe we can get a top level param? Like we have cifmw_basedir and some others?

Maybe we can get a top level param? Like we have cifmw_basedir and some others?

@cjeanner do you think it makes sense to reuse the existing cifmw_manage_secrets_ssh_type and cifmw_manage_secrets_ssh_size from the manage_secrets role?

/retest

/approve


New command to set the default path in an adr.config.json

FWIW I'm actually working on this now and will submit a PR at some stage.


Extract CAF 0.17's caf::binary_deserializer into a standalone utility

VAST currently serializes some data using caf::binary_serializer, and then deserializes it using caf::binary_deserializer. The serialization format changed between CAF 0.17 and 0.18, so once we upgrade to CAF 0.18 we will be unable to load some data from older databases. To this end, we need to extract the caf::binary_deserializer from CAF 0.17.6 into a standalone utility that does not depend on CAF at all.
The current binary serializer takes an actor system or a pointer to an execution unit as a first argument to the constructor, but that should be irrelevant for the types we currently serialize.
The interface of the new function could look something like this:
namespace vast::detail {

/// Deserializes a contiguous byte range into an instance of type *To* in
/// the serialization format of CAF 0.17.6's binary serializer.
/// @tparam To The type to deserialize into.
/// @param from The byte range to deserialize from.
/// @returns A valid instance of *To*, or an error.
template<class To>
caf::expected<To> legacy_deserialize<T>(std::span<const std::byte> from);

} // namespace vast::detail

:100: Definition of Done

A new utility exists as a drop-in replacement for the legacy deserializer.
The deserialization does not depend on any CAF types internally.
The utility is tested extensively in unit tests. The following types must be tested, as they are currently written to disk in this format:

vast::legacy_type
vast::qualified_record_field
vast::synopsis_ptr
vast::ids

We no longer use caf::binary_deserializer in libvast.

Commit 71d826299bab0440f3f34bb5145da9e6899dd050 implements all targeted type. Two tasks remain:

The 'macOS Release (Clang)' checks must pass. The 'Debian Release (GCC)' checks are successful, though.
Replace caf::binary_deserializer with vas::detail::legacy_deserializer.


Key Bindings in UI Catalog crashes

I've fixed this in https://github.com/gui-cs/Terminal.Gui/pull/1965
It seems there was a change in behaviour where SelectedItemChanged could be fired before the View had finished construction as a byproduct of OnEnter.  I fixed the issue by moving the event handler registration to the last thing in the constructor and manually called the 'update view' functionality for the first list item after that.


chore: fix typo

Types of changes

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

Description

This PR fixes a typo in the error displayed when a folder doesn't exist when running the serve command.
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Codecov Report

Merging #8674 (7d9f397) into dev (498f855) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##              dev    #8674   +/-   ##
=======================================
  Coverage   66.05%   66.05%
=======================================
  Files          92       92
  Lines        4010     4010
  Branches     1104     1104
=======================================
  Hits         2649     2649
  Misses       1095     1095
  Partials      266      266

Flag
Coverage Δ

unittests
66.05% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/cli/src/utils/serve.js
72.41% <0.00%> (ø)

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 498f855...7d9f397. Read the comment docs.

Codecov Report

Merging #8674 (7d9f397) into dev (498f855) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##              dev    #8674   +/-   ##
=======================================
  Coverage   66.05%   66.05%
=======================================
  Files          92       92
  Lines        4010     4010
  Branches     1104     1104
=======================================
  Hits         2649     2649
  Misses       1095     1095
  Partials      266      266

Flag
Coverage Δ

unittests
66.05% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/cli/src/utils/serve.js
72.41% <0.00%> (ø)

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 498f855...7d9f397. Read the comment docs.