Possible concurrent version

Hey I didn't see a discussion section, so opened an issue.
Just wondering if a concurrent version of the art tree could be implemented with this library, and if a guide/hint/ref as to how to do so could be given by anyone if possible.

I think a good spot to start for a concurrent implementation of ART would be The ART of practical synchronization by the same authors of the original ART paper.
I think you could also approach the problem with some other strategies as well, like fine grained r/w locks per node.

https://github.com/declanvk/blart/issues/27#issuecomment-2160258569 is also a previous comment talking about this subject. Cc @Gab-Menezes

IMO the concurrent version is kinda hard to implement, or at least I wasn't able to fully understand the paper/other implementations.
Basically we need to do all operations in a loop a check at all times for stale version, which is kinda estrange.
If I had to use blart in concurrent version I would do something similar to DashMap and keep multiple locks.
Of course this has a lot downside, like memory and using locks, but at least is a starting point.

Or maybe do something similar to MVCC, but I don't know about the subject

This stuff is a little out of my league I assume 😅
But if I had to give it a try, out of curiosity, I was wondering which would be easier to try to modify between trying to turn blart into a concurrent version or trying to make congee allow variable size keys?

Probably change blart to be concurrent. Congee requires fixed len keys because it does a lot of optimization with it.

As I said in the comment probably what makes it hard to implement are operations that hold a reference for an extended period of time, like iterator/entry api. Specially the iterators, since IMO iterators should work in sane way where the contents of the tree don't change while iterating over it. That's why congee doesn't have iterators.
I know that Congee uses the crossbeam epoch as a GC mechanism.

If you find it difficult, I think I better look elsewhere haha.
Really wanted to use a lock free concurrent art tree for a multi column routing table, I think it would have been the best choice, as far as data structures go.
If you don't mind, could u give your opinion on using a lock free concurrent b+ tree (concurrent_map  (sled's internal index)) or a lock free concurrent skiplist (crossbeam skipmap) vs blart as is.

If you don't mind, could u give your opinion on using a lock free concurrent b+ tree (concurrent_map (sled's internal index)) or a lock free concurrent skiplist (crossbeam skipmap) vs blart as is.

I'm sorry, I haven't used or researched those much, so I can't really give an opinion.

Not sure if I should leave the issue open, feel free to close it if this is outside the scope currently.

Feel free to leave this open, I think its useful to track as a feature request.

What a feature it will be...one day 🍻

@MirroredLens I have a few thoughts even though I haven't used concurrent_map nor crossbeam_skiplist.

If you need some of the blart features, like fuzzy, prefix, entry... And you know that your workload is read heavy or at least writes can be buffered and batched to be written in a single shot and the memory consumption of cloning the tree is not a problem I would use arc_swap with blart. This allows you to basically have a lock free version with the drawback of using a little more memory when you want to write to the tree.
Of course locks slow things down, but the biggest problem with a lock is hitting the same cache line everytime, that's what causes contention in a lock. DashMap solves this by sharding the data and keeps a lock around each shard, this allows for the locks to be almost "free", since you are spreading the load. Hence the almost linear scaling with threads.
And it goes without saying, benchmarking for your specific case is the best, I don't know about the performance characteristics of this 2 crates. But it's not always true that a lock free version will be faster than a locking one. AFAIK DashMap is one of the fastest implementations because of this, because is basically rust hashmap (which is one of the fastest in the world) with a small perf loss because of the locks.

'arc_swap' looks interesting for a read heavy or analytical workload. But this wide column routing table I'm working on is also expected to be updated by multiple writers in real time. So that's not really viable for me.

Yeah it's true that most "lock free" designs aren't truly lock free, especially under contention. I did look into dashmap as well but prefix search and range scans are terrible with hash maps in general.

But I did do some research on methods of scaling with shards - partitioning by key hashing, key range or custom. And I'm not sure how to best approach it for blart.
• I ruled out key hashing partitions, even though it would result in an even, unbiased distribution across the cores, as it would lose locality of reference and be quite terrible for range scans and prefix searches.
• Key range partitioning initially felt great for range scans and prefix search, but I was concerned about the distribution of keys across the cores being uneven. Which I presume will result in underutilized cores and maybe overwhelming certain shards more (based on "hot" keys)?
• A custom partitioning strategy might be the best way to go. But not sure how to get the best of both worlds - ordered keys and even distribution across the cores.
Overall though, I think key range partitions might be relatively easy to implement a sharding strategy with compared to optimistic lock coupling and may be much faster and more scalable as well. I think what you said about the iterator apis could be ported well to this model too.
What do you think about all of this?

Key range partitioning initially felt great for range scans and prefix search, but I was concerned about the distribution of workload over the keys across the cores being uneven. Which I presume will result in underutilized cores and maybe overwhelming certain shards more (based on "hot" keys)?

I think you're right, key partitioning would be a natural way to shard the TreeMap based on key. Without prior knowledge of the key distribution/access distribution, I also think it would be difficult to assign key ranges to shard at construction.
Maybe you could devise some way to periodically re-balance key ranges across shards? If you had some simple statistics per-key (or more aggregated), maybe you could periodically lock some adjacent shards and do the re-balance with a dedicated thread? You could re-balance only based on shard size or you could attempt to re-balance based on heat (number of read/write accesses). Not really sure, this is more just brainstorming.
I definitely want to echo @Gab-Menezes's point that some good profiling/benchmarking would be very important to trial different ideas.

I think what you said about the iterator apis could be ported well to this model too.

With the sharding model, I think the iterators would be pretty natural. Just lock each shard in key-range order and iterate over the contents.
The trouble comes when the (hypothetical) application expects that the iterator represents a snapshot view of the data-structure, which I think is pretty difficult to accomplish (probably why congee leaves it out). As long as the application is able to comprise on that point, I think iterators will still be viable.

So,

Without prior knowledge of the keys, we have no way to pre-order the key ranges across the shards.
A re-balancing of key ranges might be required to spread the load across the cores. Like you said perhaps based on some heuristics like shard size or read/write access or more/multiple.
To make iterators consistent across shards may require some kind of consistency mechanism like acid semantics or ssi or logical/vector clocks.

Given these requirements and limitations, maybe a kind of "adaptive key range sharder" would work where keys and key ranges are re-balanced dynamically based on several heuristics,

Key range distribution across shards. Dynamically adapting to the keys as they fill the tree, and/or assign ranges to shards based on optional user specifications (?).
Shard size and read/write access over time can provide more info on when or how to re-balance shards.
Not sure which consistency mechanism is the best here, but I'm assuming a vector/logical clock might be the most performant. Especially if we're watching read/write access, we could use that info to time the iterations with an internal clock to maintain snapshot like consistency across the shards.

I hope I'm right at least theoretically, but even if this is a very scalable architecture, the complexity seems pretty high to get it all working correctly 🤯

Let me know what you guys think of this strategy. And if you guys feel it's worth a try.
Currently on the frontend of things and will focus on the backend in a while. I'd be happy to attempt it in a few weeks.

You're right that I'm focusing on my use case more than a general version (I have a known ordering of keys) 😅
I'd love to share more in the future and thanks for your time!


[Bug Report] ROS2 Gazebo Simulation Physics Issue

Describe the bug
When running in ROS2 environment, the robot will suddenly jump up after a few seconds.
In simulation, the robot dog is able to walk a few steps, but then will suddenly jump up (around timestamp 00:20), as shown in the video. This is observed in multiple simulations under different commands.
https://github.com/user-attachments/assets/349348df-c8bd-47cc-af72-a54eb1a0b293
I don't think the ROS1 version has the same issue.
Steps to reproduce
ros2 launch rl_sar gazebo.launch.py rname:=a1 framework:=isaacgym

and then
ros2 run rl_sar rl_sim.py

System Info
System environment:

Commit: 64fc72d
OS: Ubuntu 20.04
ROS Version: ROS2 Foxy
Python Version: Python 3.8 (from Ubuntu system)

Checklist

[x] I have checked that there is no similar issue in the repo (required)

It seems that there is an error in the angle quaternion. I will check this, thank you for your feedback!

fixed in 4124f78


Asp.Net MVC Core: “Error unprotecting the session cookie” exception

From @skorunka on Tuesday, November 29, 2016 6:02:13 AM
I have an Asp.NET MVC application with this Authentication setup:
ConfigureServices():
services.AddSession()
services.AddAuthentication(sharedOptions => sharedOptions.SignInScheme = CookieAuthenticationDefaults.AuthenticationScheme);

Configure():
		app.UseCookieAuthentication(new CookieAuthenticationOptions());
		app.UseOpenIdConnectAuthentication(new OpenIdConnectOptions
		{
			ClientId = "xx",
			Authority = "xx",
			Events = new OpenIdConnectEvents { OnRemoteFailure = this.OnAuthenticationFailed }
		});

		app.UseSession();

When hosted in IIS, some users get this exception:
Microsoft.AspNetCore.Session.SessionMiddleware,
      Error unprotecting the session cookie.
System.Security.Cryptography.CryptographicException: The key {9ec59def-874e-45df-9bac-d629f5716a04} was not found in the key ring.
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.UnprotectCore(Byte[] protectedData, Boolean allowOperationsOnRevokedKeys, UnprotectStatus& status)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.DangerousUnprotect(Byte[] protectedData, Boolean ignoreRevocationErrors, Boolean& requiresMigration, Boolean& wasRevoked)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.Unprotect(Byte[] protectedData)
   at Microsoft.AspNetCore.Session.CookieProtection.Unprotect(IDataProtector protector, String protectedText, ILogger logger)

I have run this on the hosting server https://github.com/aspnet/DataProtection/blob/dev/Provision-AutoGenKeys.ps1
Web has only HTTPS binding, SSL certificate is ok and signed. What might cause this issue? What actually is that "key" value?
Copied from original issue: aspnet/DataProtection#189

From @skorunka on Thursday, December 1, 2016 1:51:57 AM
I can also see a lot of these:
11/30/2016 18:10:28:warn: Microsoft.AspNetCore.Session.DistributedSession[2]
Accessing expired session, Key:21d8f9ed-4013-2e65-d884-651a6f0060a0

From @kalluri-nb on Wednesday, December 14, 2016 7:22:31 PM
Any update on this. This is happening in cluster a lot. A session cookie value created on one instance is unable able to be unprotected by a different instance. Is there any basic setup for DataProtectionOptions for session management in cluster.

From @skorunka on Saturday, December 17, 2016 5:32:11 AM
The machine is not in a cluster or farm. Each requested is server by the same machine, by the same IIS site.

From @kalluri-nb on Saturday, December 17, 2016 3:00:44 PM
We resolved it by adding the redis as persistence store using DataProtectionBuilder.

From @skorunka on Sunday, December 18, 2016 3:54:36 AM
@kalluri-nb Nice, how can I resolve it? What should I change in the Setup.cs?

From @warrenfp on Tuesday, April 25, 2017 5:07:06 AM
@kalluri-nb: Can you share your implementation?

From @blowdart on Friday, April 28, 2017 12:36:41 PM
@skorunka Where are your keys being persisted? This is strange, it looks like the keyring can't get created at all. Have you tried, as a stop gap, persisting the keyring to the file system anywhere?

From @skorunka on Tuesday, May 2, 2017 1:34:17 AM
@blowdart Hi, I'm using defaults. How can I setup where the keys are persisted? Or how can I know where they are actually persisted?

From @blowdart on Tuesday, May 2, 2017 10:43:04 AM
It tries to figure out the right place depending on machine configuration. The log files will tell you, if you're logging at informational.

From @1amirjalai on Thursday, May 11, 2017 10:12:58 AM
@kalluri-nb what do you mean by

We resolved it by adding the redis as persistence store using DataProtectionBuilder.

could you please show an example?

From @1amirjalai on Thursday, May 11, 2017 10:16:37 AM
@blowdart may this issue an issue #228 are related

Closing due to lack of response.
Please reopen if you have further comments or questions on this issue.

This error started happening to me as well after my app was deployed in a docker container.
Running from VisualStudio17 was fine but my container is throwing these errors:
System.Security.Cryptography.CryptographicException: The key {416a607b-290f-44ef-83b3-2a9914859fb4} was not found in the key ring.
--
  | at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.UnprotectCore(Byte[] protectedData, Boolean allowOperationsOnRevokedKeys, UnprotectStatus& status)
  | at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.DangerousUnprotect(Byte[] protectedData, Boolean ignoreRevocationErrors, Boolean& requiresMigration, Boolean& wasRevoked)
  | at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.Unprotect(Byte[] protectedData)
  | at Microsoft.AspNetCore.Session.CookieProtection.Unprotect(IDataProtector protector, String protectedText, ILogger logger)
  | warn: Microsoft.AspNetCore.Session.SessionMiddleware[7]
  | Error unprotecting the session cookie.

Startup.cs
        public void ConfigureServices(IServiceCollection services)
        {
            services.AddMvc();
            services.AddMemoryCache();
            services.AddSession();
            services.AddTransient(provider => new RestClient(AppConfig.ApiGatewayUrl));
        }
        public void Configure(IApplicationBuilder app, IHostingEnvironment env)
        {
            app.UseDefaultFiles();
            app.UseStaticFiles();
            app.UseSession();
            app.UseMvc(routes =>
            {
                routes.MapRoute(
                    name: "default",
                    template: "{controller=Home}/{action=Index}/{id?}");
            });
        }

+1 I see it too in random AppInisight traces. The app is similar to the one above, uses Session variables (no authentication though) and runs on mulitple VMs in Azure App Service.  Seems like some configuration needs to be fixed ?

System.Security.Cryptography.CryptographicException: The key {09794bfd-5325-487e-a7b0-3cc6c51fd40b} was not found in the key ring.  at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.UnprotectCore(Byte[] protectedData, Boolean allowOperationsOnRevokedKeys, UnprotectStatus& status)  at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.DangerousUnprotect(Byte[] protectedData, Boolean ignoreRevocationErrors, Boolean& requiresMigration, Boolean& wasRevoked)  at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.Unprotect(Byte[] protectedData)  at Microsoft.AspNetCore.Session.CookieProtection.Unprotect(IDataProtector protector, String protectedText, ILogger logger) |
-- | --

Your nodes need to share the same data protection settings. See https://docs.microsoft.com/en-us/aspnet/core/security/data-protection/configuration/overview?tabs=aspnetcore2x

Thanks. Will try out the changes

I only have one node and still get this error

We also see exceptions like this in Application Insights. I'm not able to replicate the issue, but we see it happening in the logs. We only have 1 instance and we use in memory sessions. Might it be because we swap staging with prod during releasing? Would moving session data out of the web process help? (Redis for example)
System.Security.Cryptography.CryptographicException: The key {e02033c0-d797-47ae-b6e6-958e54a5bb10} was not found in the key ring.
at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.UnprotectCore(Byte[] protectedData, Boolean allowOperationsOnRevokedKeys, UnprotectStatus& status)
at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.DangerousUnprotect(Byte[] protectedData, Boolean ignoreRevocationErrors, Boolean& requiresMigration, Boolean& wasRevoked)
at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingBasedDataProtector.Unprotect(Byte[] protectedData)
at Microsoft.AspNetCore.Session.CookieProtection.Unprotect(IDataProtector protector, String protectedText, ILogger logger)

Well it can still indicate a problem, like an attacker trying to highjack a session, so it's not unreasonable for it to get logged.

Yes, but we can't do much about it and it generates noise. Logging as a warning is fine, but I think that Application Insights shouldn't treat it as an exception.
Just wondering, is that warning reported when the user tries to make a request with a stale session id? For example, session expiration is 2h, but the user makes another request after 3h of inactivity, but without closing the browser? Such cases shouldn't be reported at all I think.

@mfronczyk A) we don't track closed issues, please open a new issue.
B) This error is not caused by session storage or expiration issues, it's caused by Data Protection mis-configuration or cross site cookie contamination. E.g. If your staging and production sites do not share encryption keys, or if you encryption keys aren't being persisted across application restarts / redeploys.

@verydarkmagic Hi, I meet the same error when I deployed my application in a linux docker container, do you have any workaround on this issue?

@kusey @verydarkmagic
I haven't had this problem in a container, but FWIW when this error popped up running on localhost, my startup.cs looked very similar to what @verydarkmagic posted, particularly this method

public void ConfigureServices(IServiceCollection services) {
services.AddMvc();
services.AddMemoryCache();
services.AddSession();
services.AddTransient(provider => new RestClient(AppConfig.ApiGatewayUrl));
}

However I looked in some other projects that I did have working and found that .NET Core MVC template provides options for services.AddSession like so:

services.AddSession(options =>
{
// Set a short timeout for easy testing.
options.IdleTimeout = TimeSpan.FromHours(1);
options.Cookie.HttpOnly = true;
// Make the session cookie essential
options.Cookie.IsEssential = true;
});

These specific options may not fix your problem but there may be some explicit configuring you can do there that works

I fixed these exceptions by following the @jrad87 advice. Thanks !

If you are using a SSL add options.Cookie.SecurePolicy = CookieSecurePolicy.Always; to the Session options.


Fix FlexPRET program size issue

Compiling examples/flexpret/src/Smoke.lf results in a binary needing  21KB RAM and 54KB ROM. This is too much, more than twice as much compiling for Zephyr (10KB and 21KB).
Here is some more info:
nm build/fp-smoke --size-sort --print-size | tail -30
0000a864 0000016c T _fclose_r
20000018 0000016c D __global_locale
0000a9d0 00000174 T __smakebuf_r
00003730 00000178 T EventQueue_heapify
0000adfc 00000188 T _wcsnrtombs_l
00008118 00000190 t global_stdio_init.part.0
00008a1c 000001a8 T __swsetup_r
00002f24 000001b0 T Scheduler_acquire_and_schedule_start_tag
00001f0c 000001ec T Reactor_validate
20004828 00000200 B contexts
00003ad4 0000020c T fp_exception_handler
00000fdc 00000214 T FederatedConnectionBundle_connect_to_peers
00007d54 00000258 T __sflush_r
00003f90 000002c8 W Reset_Handler
00009650 000002f8 T _free_r
00004988 00000318 t print_integer
200002a8 00000408 D __malloc_av_
20003ea8 000004c0 B __ee_jmp_buf
20004b88 000004c0 b envs
20004368 000004c0 B __ie_jmp_buf
00008330 000004e8 T __sfvwrite_r
20002500 00000540 B main_reactor
0000a1dc 000005bc T _realloc_r
0000276c 000005d8 T Scheduler_run
00005a3c 00000650 T __umoddi3
00004ca4 00000660 t vsnprintf_impl.constprop.0
00005388 000006b4 T __udivdi3
00009948 0000082c T _malloc_r
20002a70 00001438 b scheduler
00006238 000019e4 T _vfiprintf_r

Do you have any insight here, @magnmaeh?

Hmm print and malloc seem to have the most potential for exploring. But the ROM/RAM usage has always been an issue. But here it looks a little like the std lib printf has gotten into the binary. I might be able to look a little into it this evening.

Does reactor-uc even need malloc?

No it doesn't, but I think the FlexPRET SDK uses it for a couple of things


Add support for dialing edge routers over websocket

Dials ws and tls by default
Dials only ws when built for WASM
Updated edge-api to v0.25.21

Do we want regular go apps dialing ws endpoints? Should we make protocol selection configurable?

I thought it would be good to have it exercised if someone has configured it, and it makes working with our quickstart single ER deployment a bit easier.  But I can pull it out when not building for js/wasm


[Feature] Support Standalone Deployment Mode

Currently KubeVela relies on k8s for deployment. We should support standalone deployment mode which is to run inside a single Docker container or a VM. Similar to Dapr Self-hosted Mode.
For a bit more technical details, we could package KubeVela with kcp, and be able to deploy KubeVela anywhere as a control plane.

Supported now https://kubevela.net/docs/installation/standalone, it relies on k3s.


fix(internet-header): set focus to the related element on flyout toggle

Preview environments are getting posted here, as soon as they are ready!


add support for events

Hi @denisbabineau thanks for the pull request!
@nkuttler Maybe you could take a look  at this PR (review and possibly merge)?

This functionality has been added in the refactor branch.


Code completion doesn't offer class-specific properties/functions when set with $NodePath

Godot version
v4.0.dev.20211015.official [f113dc986]
System information
Windows 10
Issue description
When I assign a node to a variable via $NodePath, that variable gets only suggestions for Node's properties and methods, but not more specific (for example, Node3D specific). Explicitly defining class doesn't help.
Steps to reproduce
Add AnimationPlayer in scene and this script on root node:
var anim1 : AnimationPlayer = $AnimationPlayer
var anim2 : AnimationPlayer
anim1.p # doesn't suggest play
anim2.p # suggests play

Minimal reproduction project
No response

This has been fixed for some time already, closing...


Add latency measurements for ping pong sample

code merged into develop to add latency on becnhmarks, thanks !


Add Future Date

Hi.
Python's faker lib has future_date
It's just shortcat for getting future time, instead of doing smth like:
time.Now().Add(time.hour * rand())

Don't you mind if I'll make PR?

If you want to add a pr I would be more than happy to add it. Just make sure it has the examples and benchmarks, lookup and readme.

examples

I think it'd not to so easy, since result of FutureDate depends on current time.


feat(tracing): Add instana header support in propagation

Summary

Checklist

[x] The Pull Request has tests
[x] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix #[12494]

In addition to my previous comment, this needs a rebase.


Can set Concurrent::Executor for returned Future

Actually, this change permits overriding any of the constructor args on
the returned Concurrent::Future when running an RPC, including
providing an :executor.
When constructing the Elementary::Connection, can can provide a hash of
:future_options that are passed down into the Concurrent::Future
constructor.  This makes it easy to get these operations off of the
global thread pools if needed.

This has been released to rubygems.org


Better documentation for Settings Field Visibility

Problem Description
I have a 'select' type field that allows the user to pick image, icon, or text. I want to only show a set of text controls on the design tab is the value for that select field is 'text'. It is currently unclear if it the advanced fields can be controlled in this manner. Updating the documentation to clarify this point would be helpful.

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[ ] Detailed description

Description should usually be 150+ characters.

[ ] Steps to reproduce included
[ ] Support Ticket URL included
[ ] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.

The field visibility system doesn't know anything about the tabs. All fields are considered regardless of which tab they are in :smiley:


build: adding an disable_exceptions (which does not yet work)

The new build option simply compiles out all try/catch code, while leaving in the exceptions.   This can not yet be successfully used as it turns up fno-exceptions which chokes on throw statements.  This is by design as if we compiled out throw as well, config failures would be fatal instead of gracefully handled.
Risk Level: low
Testing: manual testing
Docs Changes: n/a
Release Notes: n/a
Part of https://github.com/envoyproxy/envoy/issues/27412

raven I seem to remember the parity of build options sometimes biting y'all.  This is a no-op by default which I think is what you need but throwing it your way to confirm :-)

I think this is unproblematic, because it's both defaulting to disabled and also behaving disabled if the config value is entirely unset (which is where we were bitten before). Thanks for thinking of me. :)

/wait I think format is broken =P

/wait again


Update README.md

update link text and URL
thanks for sharing!

thanks, sorry did not see your update, I rarely check this repo.


cargo test --doc can't find crate in dev-dependencies

It works when the crate is in the [dependencies] section.
Rust version: 1.15.0-nightly (8f02c429a 2016-12-15)
Cargo version: 0.16.0-nightly (ddb5c32 2016-12-16)
Platform: Windows
Output:
running 1 test
test _0 ... FAILED

failures:

---- _0 stdout ----
        error[E0463]: can't find crate for `num_cpus`
 --> <anon>:2:1
  |
2 | extern crate num_cpus;
  | ^^^^^^^^^^^^^^^^^^^^ can't find crate

error: aborting due to previous error

thread '_0' panicked at 'Box<Any>', C:\bot\slave\nightly-dist-rustc-win-msvc-64\build\src\librustc_errors\lib.rs:460
note: Run with `RUST_BACKTRACE=1` for a backtrace.
thread '_0' panicked at 'couldn't compile the test', C:\bot\slave\nightly-dist-rustc-win-msvc-64\build\src\librustdoc\test.rs:261

failures:
    _0

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured

Doc:
//! # Examples
//!
//! ```
//! extern crate num_cpus;
//!
//! fn main() {
//!     ...
//! }
//! ```

Cargo.toml:
[dev-dependencies]
num_cpus = "^1.2"

I have similar symptoms with the following:
Linux
rustc 1.15.0-nightly (8f02c429a 2016-12-15)
cargo 0.16.0-nightly (ddb5c32 2016-12-16)
lib.rs:
//! ```
//! extern crate rand;
//! # fn main() {
//! use rand::random;
//! let r = random::<u32>();
//! # }
//! ```

Cargo.toml:
[dependencies]

[dev-dependencies]
rand = "0.3.14"

Running cargo test runs the doc-test just fine, while cargo test --doc, which should only run the doctests, complains:
---- _0 stdout ----
	error[E0432]: unresolved import `rand::random`
 --> <anon>:3:5
  |
3 | use rand::random;
  |     ^^^^^^^^^^^^ no `random` in the root

error: use of unstable library feature 'rand': use `rand` from crates.io (see issue #27703)
 --> <anon>:1:1
  |
1 | extern crate rand;
  | ^^^^^^^^^^^^^^^^^^
  |
  = help: add #![feature(rand)] to the crate attributes to enable

Executing both commands with --verbose yields:
cargo test --verbose:
   Doc-tests testdep
     Running `rustdoc --test /home/sascha/testdep/src/lib.rs --crate-name testdep -L dependency=/home/sascha/testdep/target/debug/deps --extern testdep=/home/sascha/testdep/target/debug/deps/libtestdep-3c91bddf1516a6f6.rlib --extern rand=/home/sascha/testdep/target/debug/deps/librand-26cfd8d095643d5f.rlib`

And cargo test --doc --verbose:
   Doc-tests testdep
     Running `rustdoc --test /home/sascha/testdep/src/lib.rs --crate-name testdep -L dependency=/home/sascha/testdep/target/debug/deps --extern testdep=/home/sascha/testdep/target/debug/deps/libtestdep-3c91bddf1516a6f6.rlib`

It's not related to using the rand crate, when other crates are used (tested with nalgebra) exclusively in doctests (being declared in the [dev-dependencies] section) the same behavior occurs -> the .rlib is not included when running only the doctests.


Add "A Guide to Training Models Using tf.keras and tf.estimator"

Stage: https://colab.research.google.com/github/WilliamHYZhang/examples/blob/master/community/en/A_Guide_to_Training_Models_Using_tf_keras_and_tf_estimator.ipynb
Does this add something that's not in the current guide/tutorials? https://www.tensorflow.org/tutorials/estimator/keras_model_to_estimator

@lamberta thank you for taking the time to review my PR. To answer your question, this guide provides

How to create model estimator by itself
How to create same model from keras by itself
How to create estimator from keras (covered in the link you provided)
Compare/contrast advantages/disadvantages and use case scenarios.

I don't believe there are any comparison guides between estimator and keras yet. Please let me know what you think!

Hey, @WilliamHYZhang  it would be great if you add licence and colab/github buttons on top of your notebook. You can take reference from my notebook : https://github.com/tensorflow/examples/blob/master/community/en/Tensorflow_Datasets_Example.ipynb
Also, the from_future things  are unnecessary. We've cleaned those lines from the existing notebooks, So kindly delete those lines.

Thanks for the contribution but this community/ directory has been deprecated.  We're no longer accepting pull requests here.
Thanks


Fix some minor benchMark bug and Refactor Master max throughput

fix  https://github.com/Alluxio/alluxio/issues/14654
Major changes:
Remove alluxio.stress.cli.suite.Suite, which duplicates with the Benchmark class.
Fewer classes make the structure clearer.
The base methods of MasterMaxThroughput and JobServiceMaxThroughput are merged.
AbstractMaxThroughputResultboth interfaces with Summary, TaskResult to adapt with theBenchmark`
some bug fix :

some unit test  may output some interference logs (For example, some warn logs indicating that the list file is too long).So I added an identifier to the benchmark output(otherwise, json deserialization to get the result of each unit test will make an error).

The CREATE_FILES function in StressJobServiceBench cannot be used (the number of successes is not cumulative, and the delete file step also fails). The warmup function is also missing

When the file system is the HDFS file system, the parent dir can be deleted to delete the children files. It takes too long to delete the child files one by one (this is necessary on s3).

@jja725 could you help review this PR please? thanks.

let me know when it's ready, thanks for the contribution!

@jja725 @yuzhu PTAL, thanks

@jja725 PLAT, thanks!

@jiacheliu3 @shiqi1122 For your awareness, since you guys are also working on hardening stressbench

Leave some minor request, we are close! Could you help me do some manual testing to make sure everything is fine after refactoring?
@jja725 Recently the benchmark modified something and I have resolved the conflicts. I will retest my code~

@jja725 I had retest
bin/alluxio runClass alluxio.stress.cli.suite.MasterMaxThroughput  --operation ListDir --warmup 5s
bin/alluxio runClass alluxio.stress.cli.suite.MasterMaxThroughput  --operation OpenFile --warmup 5s
bin/alluxio runClass alluxio.stress.cli.suite.MasterMaxThroughput  --operation RenameFile --warmup 5s
bin/alluxio runClass alluxio.stress.cli.suite.JobServiceMaxThroughput --operation Noop

@jja725 done~

@yuzhu any suggestion? can this PR be merged?

ping @yuzhu ~

alluxio-bot, merge this please


Tickscript sp0

rewrite of TICKscript introduction and syntax sections, with small reorganization.
Fixes #1244 - Section Numerical precision
Fixes #1238 - Section Summary of variable use between syntactic domains
Fixes #589 - Section Gotchas - Circular rewrites
The built documentation has been staged at AWS.
http://ec2-54-153-106-230.us-west-1.compute.amazonaws.com:1313/kapacitor/v1.3/tick/

I'd like to get one other person to give this a read through before the final merge. One last set of eyes I think would be useful.

I would like to see this form the basis of our Kapacitor 1.4 docs.  Essentially, when we cut over to 1.4 -- these are the docs.  Can you please restructure the root folder for the content into v1.4 director?
Thoughts?

@desa and ( @nathanielc ): The kapa-1.4 branch is up-to-date, 1.4 release ready and includes all the changes made in this pull request. Can you make sure kapa-1.4 will be part of the release, please?

#1325 obsoletes this PR


Where to disable favicons?

As a non-developer, it wasn't clear to me where I add this code snippet. To the plugin settings in VS Code (when I just copied and pasted the snippet, it didn't seem to do the trick)? To the config.edn file? Somewhere else?
Any guidance would be helpful, thanks!

You can copy the code into custom.css, reference: #1 .


tests: loosen overly-restrictive test patterns

Admit different path separators in DebugInfo tests.  These will be used
in subsequent changes that enable the directory module layout on
non-Darwin platforms.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test


'ECONNREFUSED' running dev on<EMAIL_ADDRESS>

I'm behind a corporate firewall and have to have a .npmrc setup with a proxy and https-proxy.
There appears to be something in<EMAIL_ADDRESS>that tries to connect to npm on launch and seems to ignore my .npmrc proxy settings as I'm seeing what look like proxy problems when I start my next app in dev mode.
Expected Behavior
running yarn dev or npm run dev should bring up the server.
Current Behavior
λ yarn dev
yarn run v1.3.2
$ next
{ Error: connect ECONNREFUSED <IP_ADDRESS>:443
    at Object._errnoException (util.js:1024:11)
    at _exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1182:14)
  code: 'ECONNREFUSED',
  errno: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 443 }
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

n.b. this does not occur when using yarn build & yarn start.
Steps to Reproduce (for bugs)
This appears to affect all projects, but I used this one:
https://github.com/zeit/next.js/tree/canary/examples/hello-world
You need to be relying on a custom .npmrc with a proxy - everything is fine if you don't have a proxy.
Context
This has only broken since upgrading to<EMAIL_ADDRESS>- maybe related to update-check?
Your Environment

Tech
Version

next
6.0.0

node
8.9.1

OS
Windows 7 x64

Duplicate of #4237


Dashboards Will Not Load

Fresh install of superset 1.0.1.  Dashboards will not open.  I originally had GLOBAL_ASYNC_QUERIES enabled, but have sinced disabled, as it appeared that the client side poll never resolved.
Expected results
Dashboard opens ;)
Actual results
Attempting to open a dashboard leaves you stuck on the loader.  There is no error recorded in the browser console, or the server log.
Screenshots
https://user-images.githubusercontent.com/14878744/114721552-689b2380-9d0f-11eb-80b6-962096051500.mp4
Environment
(please complete the following information):

superset version: 1.0.1
python version: 3.8.8
node.js version: not installed

Additional context
superset_config.py
from celery.schedules import crontab
from superset.utils.logging_configurator import DefaultLoggingConfigurator

FLASK_APP = 'superset'
APP_NAME = 'superset'

LOG_LEVEL = "INFO"
ENABLE_TIME_ROTATE=True
SILENCE_FAB = False

# Superset specific config
ROW_LIMIT = 5000

# Superset webserver port
SUPERSET_WEBSERVER_PORT = 8088
# Superset webserver address
SUPERSET_WEBSERVER_ADDRESS = '<IP_ADDRESS>'
# Superset upload folder
UPLOAD_FOLDER = '/srv/superset/upload'
IMG_UPLOAD_FOLDER = '/srv/superset/img'

# Flask App Builder configuration
# Your App secret key
SECRET_KEY = ''

# The SQLAlchemy connection string to your database backend
# This connection defines the path to the database that stores your
# superset metadata (slices, connections, tables, dashboards, ...).
# Note that the connection information to connect to the datasources
# you want to explore are managed directly in the web UI
SQLALCHEMY_DATABASE_URI = 'mysql://root:@localhost/superset'

# Flask-WTF flag for CSRF
WTF_CSRF_ENABLED = False
# Add endpoints that need to be exempt from CSRF protection
WTF_CSRF_EXEMPT_LIST = []
# A CSRF token that expires in 1 year
WTF_CSRF_TIME_LIMIT = 60 * 60 * 24 * 365

APP_NAME = "DDX Enterprise BI"
APP_ICON = "https://www.ddxdental.com/images/marketing/logo-ddx-small.png"
APP_ICON_WIDTH = 70
MAPBOX_API_KEY = ""
FAVICONS = [{"href": "https://www.ddxdental.com/wp-content/uploads/2017/05/DDXOS_Favicon2.jpg"}]

CACHE_CONFIG = {
    'CACHE_TYPE': 'redis',
    'CACHE_DEFAULT_TIMEOUT': 60 * 60 * 24, # 1 day default (in secs)
    'CACHE_KEY_PREFIX': 'superset_results',
    'CACHE_REDIS_URL': 'redis://localhost:6379/0',
}

DATA_CACHE_CONFIG = {
    'CACHE_TYPE': 'redis',
    'CACHE_DEFAULT_TIMEOUT': 60 * 60 * 24, # 1 day default (in secs)
    'CACHE_KEY_PREFIX': 'superset_results',
    'CACHE_REDIS_URL': 'redis://localhost:6379/0',
}

class CeleryConfig:  # pylint: disable=too-few-public-methods
    BROKER_URL = "redis://localhost:6379/0"
    CELERY_IMPORTS = ("superset.sql_lab", "superset.tasks")
    CELERY_RESULT_BACKEND = "redis://localhost:6379/0"
    CELERYD_LOG_LEVEL = "DEBUG"
    CELERYD_PREFETCH_MULTIPLIER = 1
    CELERY_ACKS_LATE = False
    CELERY_ANNOTATIONS = {
        "sql_lab.get_sql_results": {"rate_limit": "100/s"},
        "email_reports.send": {
            "rate_limit": "1/s",
            "time_limit": 120,
            "soft_time_limit": 150,
            "ignore_result": True,
        },
    }
    CELERYBEAT_SCHEDULE = {
        "email_reports.schedule_hourly": {
            "task": "email_reports.schedule_hourly",
            "schedule": crontab(minute=1, hour="*"),
        },
        "reports.scheduler": {
            "task": "reports.scheduler",
            "schedule": crontab(minute="*", hour="*"),
        },
        "reports.prune_log": {
            "task": "reports.prune_log",
            "schedule": crontab(minute=0, hour=0),
        },
        'cache-warmup-hourly': {
            'task': 'cache-warmup',
            'schedule': crontab(minute=0, hour='*'),  # hourly
            'kwargs': {
                'strategy_name': 'top_n_dashboards',
                'top_n': 5,
                'since': '7 days ago',
            },
        },
    }

CELERY_CONFIG = CeleryConfig  # pylint: disable=invalid-name

#GLOBAL_ASYNC_QUERIES_JWT_SECRET = ""

FEATURE_FLAGS = {
#    "GLOBAL_ASYNC_QUERIES": True,
    "DASHBOARD_NATIVE_FILTERS": True,
    "ALERTS_REPORTS": True,
    "DYNAMIC_PLUGINS": True
}

EMAIL_NOTIFICATIONS = True
SMTP_HOST = "smtp.sparkpostmail.com"
SMTP_STARTTLS = True
SMTP_SSL = False
SMTP_USER = "SMTP_Injection"
SMTP_PORT = 587
SMTP_PASSWORD = ""
SMTP_MAIL_FROM =<EMAIL_ADDRESS>

to confirm, dashboards don't load when GLOBAL_ASYNC_QUERIES is enabled then disabled?

@junlincc Sorry, I wasn't clear.  Dashboards don't load with GLOBAL_ASYNC_QUERIES enabled or disabled.  I originally had it enabled, and dashboards did not load.  Then tried to disable it, thinking that was maybe the culprit.

Strange that there are no errors logged. I'm assuming that you have a SECRET_KEY value set, and it's just redacted here.
Other than that, I would suspect that the Redis cache instance perhaps isn't connecting, though I would think that would log an error. Can you confirm access via redis-cli?

@robdiciuccio yes, secret is redacted.  Can access redis via redis-cli.  Can create and load individual charts.
How does one rebuild the frontend assets?  Google seems to be failing me

Updated to 1.1.0   This resolved the issue


Feature: Use Background to avoid repeating the same steps in the subsequent scenarios

I have a question related to the Background when you don't want to repeat the same statements at the beginning of each scenario
Example
Feature: Counter

  Scenario: starts with initial value of 0
    Given I visit the {'/counters'} page
    Then I see {"0"}

  Scenario:  3 times
    Given I visit the {'/counters'} page
    When I tap {Icons.add} icon {6} times
    Then I see {"6"}
    And I see {"Trick!"}
  #

Would it be possible to simply add a background?
Feature: Counter

  Background:
     Given I visit the counter page

  Scenario: starts with initial value of 0
    Given I visit the {'/counters'} page
    Then I see {"0"}

  Scenario:  3 times
    Given I visit the {'/counters'} page
    When I tap {Icons.add} icon {6} times
    Then I see {"6"}
    And I see {"Trick!"}
  #

Check also a similar effort in this package flutter_test_ui 1.0.0

Implemented in version 0.6.0, now this will work:
Feature: Counter

  Background:
    Given I visit the {'/counters'} page

  Scenario: starts with initial value of 0
    Then I see {"0"}

  Scenario:  displays 'Trick!' for the multiple of 3
    When I tap {Icons.add} icon {6} times
    Then I see {"6"}
    And I see {"Trick!"}

Thanks Ole!


Remove downloader host

We faced issues with synchronize module and using 'copy' had too much constraint.
I propose that each nodes download their dependencies, it will then remove the need to copy/rsync binaries from node to node.
I'll probably work again on the 'downloader' host later as it can be usefull in somecase (later)

merged.


Update msv3-mdsv3-medium-series.md

Added 4TB sizes

@ayshakeen

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


VSCode crashes when the search field in the extensions list has text in it

Does this issue occur when all extensions are disabled?: No (But read why :))

Version: 1.83.0
Commit: e7e037083ff4455cf320e344325dacb480062c3c
Date: 2023-10-03T16:12:16.321Z
Electron: 25.8.4
ElectronBuildId: 24154031
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
This also happened with 1.82.1 and I am pretty sure it will happen with the insiders as well.
Steps to Reproduce:

Go to the extension's list/tab (You must have at least 2 extensions in the list)
Select an extension
push Ctrl+F to search
Write anything you want in the text box (If it stays empty it won't crash)
Go to a different extension in the list
VSCode crashes

Maybe a duplicate of #177046

/duplicate

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Heightgraph map marker hidden by route quality coding

Hi The popup on the map with the gradient percentage is now hide behind  the track.
I think it is only a matter of changing the z-index to place it on top of a track?

Regards Jan

As the heightgraph code is already forked anyway it can be patched further to fix this.
As already mentiond by rkflx the marker just has to be moved from the overlay pane to the more appropriate overlay pane
Here is an approach for dist/L.Control.Heightgraph.js:
@@ -5104,14 +5104,15 @@
         var normalizedY = layerPoint.y - 75;

         if (!this._mouseHeightFocus) {
-          var heightG = select(".leaflet-overlay-pane svg").append("g");
+         var svg = L.svg({pane: 'tooltipPane'}).addTo(this._map);
+          var heightG = select(svg._container).append("g");
           this._mouseHeightFocus = heightG.append('svg:line').attr('class', 'height-focus line').attr('x2', '0').attr('y2', '0').attr('x1', '0').attr('y1', '0');
           this._mouseHeightFocusLabel = heightG.append("g").attr('class', 'height-focus label');
           this._mouseHeightFocusLabelRect = this._mouseHeightFocusLabel.append("rect").attr('class', 'bBox');
           this._mouseHeightFocusLabelTextElev = this._mouseHeightFocusLabel.append("text").attr('class', 'tspan');
           this._mouseHeightFocusLabelTextType = this._mouseHeightFocusLabel.append("text").attr('class', 'tspan');
           var pointG = this._pointG = heightG.append("g").attr("class", "height-focus circle");
-          pointG.append("svg:circle").attr("r", 5).attr("cx", 0).attr("cy", 0).attr("class", "height-focus circle-lower");
+          //pointG.append("svg:circle").attr("r", 5).attr("cx", 0).attr("cy", 0).attr("class", "height-focus circle-lower");
         }

         this._mouseHeightFocusLabel.style("display", "block");

About the pointG: It's duplicate with the fix so I've disabled it.
Interested in a pull request?


Cannot install from zip within Kodi

@henryjfry , it seems that with one of your latest commits, you removed the addon.xml file from the root folder of the addon.  Without that, you cannot do an "install from zip" of the addon inside Kodi.  Please restore this.  Thanks.

Install it from my repo:
https://henryjfry.github.io/repository.thenewdiamond/index.html
https://github.com/henryjfry/repository.thenewdiamond/blob/main/zips/script.artwork.beef/script.artwork.beef-0.28.6.zip

@henryjfry , thanks, I will do that.


Move the principal Tatin server into a Docker

Move data to proper location
Fix the right/owner problem
Run as runtime ?!
Ride with certificates?


Upgrade to SQLAlchemy 1.0.0 (close #97)

In local test, there is no bug or error.

Coverage decreased (-2.58%) to 70.94% when pulling c98b38362368d546e5fae8ce979e1828485a4a16 on item4:upgrade/sqlalchemy into 14f3399fc63e85d08fb7ce3d2f9f9af983810bc7 on clicheio:master.


mpfs/px4io_serial.cpp: Remove unnecessary critical section

Masking all system interrupts in this case is not necessarily a good idea as the csection can be relatively long due to accessing a low speed peripheral domain, potentially writing a lot of data via the slow BUS.
I presume the original code tries to prevent any interrupt from executing during the FIFO fill, but this will also starve all other interrupt sources for the duration of the FIFO fill, so it should not be considered a valid way to ensure atomicy of the px4io message.
The best solution for this issue would be to use DMA for the transfer (don't know if DMA is available).
Mitigate the aforementioned issue by enabling the px4io RX interrupt only after the TX FIFO has been fully written.

Are you sure about this? If the code is pre-empted during fifo fill, the data might not be sent as a one burst causing the reception at IO end to fail. I added this critical section to prevent that from happening. Of course, if filling the fifo fill takes too long, it is better to not have the critical section here. I somehow assumed that it is a fast operation

Yes I understood what the csection is for. I think this needs DMA as as you said, this trades one problem for another.
Writing to the uart device can be abysmally slow as it runs with a relatively slow clock as far as I know. Also, this change seems to fix the random CRC error but I did not test RC/PWM control which is more important and it runs at the faster baudrate.
So this patch will be abandoned, we need DMA for this..

But there is no dma available for it...? If it works better without the CS, let's keep this patch! "px4io status" should show if errors occur during run.

Are you sure about this? If the code is pre-empted during fifo fill, the data might not be sent as a one burst causing the reception at IO end to fail. I added this critical section to prevent that from happening. Of course, if filling the fifo fill takes too long, it is better to not have the critical section here. I somehow assumed that it is a fast operation

Yes I understood what the csection is for. I think this needs DMA as as you said, this trades one problem for another.
Writing to the uart device can be abysmally slow as it runs with a relatively slow clock as far as I know. Also, this change seems to fix the random CRC error but I did not test RC/PWM control which is more important and it runs at the faster baudrate.
So this patch will be abandoned, we need DMA for this..

But there is no dma available for it...? If it works better without the CS, let's keep this patch! "px4io status" should show if errors occur during run.

Are you sure no DMA is available? Is the PDMA only memory to memory or does it work with peripherals ? The mmuart reference actually mentions DMA, but not sure if this is just some kind of remnant from some related documentation (it mentions Cortex-M3 too :D)
Like I said, this must not be merged before it is tested with RC and PWM (flight tested). We don't get the CRC error during boot anymore but that is not as important as the flight controls.

I did profiling on the FIFO fill and it takes 1-4us to perform. So perhaps the critical section is not an issue here.
I did not have RC or PWM attached if that makes a difference. I'll try with RC next.

With RC it takes 1-5us to fill the FIFO.

But there is no dma available for it...?

Yes you are right, PDMA does not exist on Polarfire, it is some kind of copy&paste remnant from some other MCU / SoC documentation (something with a Cortex-M3 core) https://onlinedocs.microchip.com/oxy/GUID-199548F4-607C-436B-80C7-E4F280C1CAD2-en-US-1/GUID-95DBAF5E-0123-4434-9F74-FD7477DE45E6.html

The fix doesn't work:
WARN  [px4io] check CRC failed: -1, CRC:<PHONE_NUMBER>
[PX4IO] using firmware from /etc/extras/px4_io-v2_default.bin
[PX4IO] bad sync 0xff,0xff
[PX4IO] found bootloader revision: 5
[PX4IO] erase...
[PX4IO] programming 40140 bytes...
[PX4IO] verify...
[PX4IO] update complete
Abandoning


Fix test timeouts in CI

Resolves #893

Codecov Report

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

@@           Coverage Diff           @@
##           master     #894   +/-   ##
=======================================
  Coverage   34.77%   34.77%
=======================================
  Files          57       57
  Lines        3919     3919
=======================================
  Hits         1363     1363
  Misses       2396     2396
  Partials      160      160

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 a2daf1a...7a5fe90. Read the comment docs.


Export Space Metadata

This adds the ability to export space metadata labels from cf-mgmt export-config. This is useful when you want cf-mgmt apply to import the same space labels.

Switched from using @calebwashburn fork of go-cfclient back to the latest community version (already merged the requisite APIs used by cf-mgmt).
Removed the hardcoded metadata prefix of cf-mgmt.pivotal.io. This might be a breaking change, but as implemented there was no way to not set a prefix (e.g. empty string).
Only copying across labels, should probably also include annotations.

This is almost 1 year old.
Closing this PR as we are not planning in adding new features for now. We might revisit this decision in the future.


astc-encoder: 3.4 -> 3.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ARM-software/astc-encoder/releases.
meta.description for astc-encoder is: "An encoder for the ASTC texture compression format"
meta.homepage for astc-encoder is: "https://github.com/ARM-software/astc-encoder"
meta.changelog for astc-encoder is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/cxxc4jxyssi289k6mgnrafksh1m407zv-astc-encoder-3.5/bin/astcenc-sse2 had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/cxxc4jxyssi289k6mgnrafksh1m407zv-astc-encoder-3.5/bin/astcenc had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 3.5 with grep in /nix/store/cxxc4jxyssi289k6mgnrafksh1m407zv-astc-encoder-3.5
found 3.5 in filename of file in /nix/store/cxxc4jxyssi289k6mgnrafksh1m407zv-astc-encoder-3.5

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

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
astc-encoder

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cxxc4jxyssi289k6mgnrafksh1m407zv-astc-encoder-3.5 \
  --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 astc-encoder https://github.com/r-ryantm/nixpkgs/archive/7c26be24f3aa61bd35f4258c2c85f93e1be5bd24.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/cxxc4jxyssi289k6mgnrafksh1m407zv-astc-encoder-3.5
ls -la /nix/store/cxxc4jxyssi289k6mgnrafksh1m407zv-astc-encoder-3.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

astc-encoder

Maintainer pings
cc @dasisdormax for testing.

works great


Can't create a container with an ontology

Hi, we've just spooled up Deep Lynx to have a 'play' and we've hit the first stumble. I first tried to the diamond ontology from https://raw.githubusercontent.com/idaholab/DIAMOND/master/diamond.owl just by downloading it and pointing to it. When I do, the error I get is

When I just point to the above URL, I get a different error

Am I doing something silly? Should I be doing something different to get the OWL file to be recognised?

That should be working @makeclean I don't think you did anything wrong. I'll take a quick look, I know we had made some changes to both importer and ontology but nothing that should have broken it.
Are you using the docker compose setup?

@makeclean I took a look by spinning up a new instance based on master using the docker compose setup we provide. I then cloned the Diamond repository and selected the owl file to upload. I was unable to recreate the first error and successfully created the ontology.

The second error is because we're not getting the proper Content-Type header from the Github URL. I'll look into correcting that so we avoid doing that check where it's not possible.
If you're not using the docker compose setup then my next guess is something is wrong with your setup itself, probably just an incorrect configuration value or something along those lines. If you like we could get a call together to debug for you.
And sorry you're running into issues with the program :(

So I 'wgetted' the owl file to my local machine, and thats what lead to the first error. I've just cloned, in case that should make a difference, and I get the same error as below.

Should/would it matter that I have DeepLynx running on a cloud server?

It shouldn't matter, that file gets loaded completely into memory so isn't hitting a disk at any point. I'm interested in seeing your cloud configuration though. Were you seeing any errors in the logs?

Forgive the ignorance, where would the logfiles end up? A specific file in the system in a particular place, i searched the wiki but didnt find too much to help?

They should be output to stdout and stderr - as well as two files at the root of the directory where it's running (combined.log and exceptions.log)

Nothing helpful I'm afraid, looks like pages and pages of
022-08-17T14:15:52.985Z ^[[32minfo^[[39m: Worker for job "data_target_emitter" online
2022-08-17T14:15:54.399Z ^[[32minfo^[[39m: Worker for job "export" online
2022-08-17T14:15:54.698Z ^[[32minfo^[[39m: Worker for job "data_target_emitter" signaled completion done
2022-08-17T14:15:56.114Z ^[[32minfo^[[39m: Worker for job "export" signaled completion done
2022-08-17T14:16:07.819Z ^[[33mwarn^[[39m: Error: Job "data_staging_emitter" is already running
2022-08-17T14:16:07.831Z ^[[33mwarn^[[39m: Error: Job "processing_queue" is already running
2022-08-17T14:16:07.832Z ^[[33mwarn^[[39m: Error: Job "events_queue" is already running
2022-08-17T14:16:07.832Z ^[[33mwarn^[[39m: Error: Job "edge_queue_emitter" is already running
2022-08-17T14:16:07.833Z ^[[33mwarn^[[39m: Error: Job "data_source_queue" is already running
2022-08-17T14:16:07.837Z ^[[33mwarn^[[39m: Error: Job "data_target_queue" is already running
2022-08-17T14:16:07.838Z ^[[33mwarn^[[39m: Error: Job "edge_item_queue" is already running
2022-08-17T14:16:07.934Z ^[[32minfo^[[39m: Worker for job "data_source_emitter" online
2022-08-17T14:16:09.999Z ^[[32minfo^[[39m: Worker for job "data_source_emitter" signaled completion done
2022-08-17T14:16:22.987Z ^[[32minfo^[[39m: Worker for job "data_target_emitter" online
2022-08-17T14:16:24.902Z ^[[32minfo^[[39m: Worker for job "data_target_emitter" signaled completion done
2022-08-17T14:16:52.991Z ^[[32minfo^[[39m: Worker for job "data_target_emitter" online
2022-08-17T14:16:54.936Z ^[[32minfo^[[39m: Worker for job "data_target_emitter" signaled completion done
2022-08-17T14:17:07.820Z ^[[33mwarn^[[39m: Error: Job "data_staging_emitter" is already running
2022-08-17T14:17:07.831Z ^[[33mwarn^[[39m: Error: Job "processing_queue" is already running
2022-08-17T14:17:07.832Z ^[[33mwarn^[[39m: Error: Job "events_queue" is already running
2022-08-17T14:17:07.833Z ^[[33mwarn^[[39m: Error: Job "edge_queue_emitter" is already running
2022-08-17T14:17:07.833Z ^[[33mwarn^[[39m: Error: Job "data_source_queue" is already running
2022-08-17T14:17:07.837Z ^[[33mwarn^[[39m: Error: Job "data_target_queue" is already running
2022-08-17T14:17:07.838Z ^[[33mwarn^[[39m: Error: Job "edge_item_queue" is already running
2022-08-17T14:17:07.937Z ^[[32minfo^[[39m: Worker for job "data_source_emitter" online
2022-08-17T14:17:09.728Z ^[[32minfo^[[39m: Worker for job "data_source_emitter" signaled completion done
2022-08-17T14:17:22.995Z ^[[32minfo^[[39m: Worker for job "data_target_emitter" online
2022-08-17T14:17:24.847Z ^[[32minfo^[[39m: Worker for job "data_target_emitter" signaled completion done

The exceptions.log file is empty
There are no new 'special' error messages from around the time i was trying to at the ontology.

Yeah all those look good. Can you look at the network tab in your browser's dev tools and see the actual response to the POST to the /containers endpoint? I'm wondering if an error is coming through and it's just not displaying right. If it's not that, and you're up to date on your deployment, I can only think something in the configuration is going wrong.

I wonder if you're running this with nginx or something in front - a 413 typically signals that the filesize is larger than the server's allowed file upload size. Default DeepLynx's limit is large enough to handle Diamond just fine, so I'm wondering if something in front of it is throwing that error?

Closing due to lack of activity.


how to modify popup detail ?

I see in previous post that to modify the detail popup we need to modify scheduleDetailPopup.hbs. But i don't find this file, and when i want to modify scheduleDetailPopup.hbs section in tui-calendar.js the code is not the same. So how can i modify scheduleDetailPopup.hbs and how to integer it ?

You have to make fork of the repo. Then in src/js/view/popup and src/js/view/template you will find both scheduleDetailPopup.js and scheduleDetailPopup.hbs. Those are files resposible for template and login behind template.
Then you have to run 'bundle' script located in package.json, this will create new dist directory, which will contain your changes.
This is the way i did this, but to be honest i'm not sure if it's the best way to do this. I hope someone will correct me if i'm wrong.

You can't modify default popups by any option. Instead, you can make own your popups. https://github.com/nhn/tui.calendar/blob/master/docs/getting-started.md#customize-popups


Test with cori  bigmen and gpu nodes

They may have a faster connection.

probably not needed anymore. PM is the preferred option going forward.

this can be closed.


[Question]: VectorStoreInfo is not being used PGVectorStore._build_filter_clause

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I'm following an example from the Advanced RAG: Combining RAG with Text-to-SQL video.
https://github.com/run-llama/llamacloud-demo/blob/main/examples/advanced_rag/llamacloud_sql_router.ipynb
I have vector store defined using MetadataInfo:
vector_store = PGVectorStore.from_params(
    ...
)

storage_context = StorageContext.from_defaults(vector_store=vector_store)
vector_index = VectorStoreIndex([], storage_context=storage_context)

vector_store_info = VectorStoreInfo(
    content_info="Articles from my blog",
    metadata_info=[
        MetadataInfo(
            name="created_at", type="str", description="Article creation timestamp in ISO 8601 format"
        )
    ],
)

vector_auto_retriever = VectorIndexAutoRetriever(
    vector_index, vector_store_info=vector_store_info
)

retriever_query_engine = RetrieverQueryEngine.from_args(
    vector_auto_retriever, llm=OpenAI(model="gpt-4o")
)

vector_tool = QueryEngineTool.from_defaults(
    query_engine=retriever_query_engine,
    description=(
        f"Useful for answering semantic questions about articles from my blog"
    ),
)

And I run my workflow for query which needs to filter by my Metadata:
wf = RouterOutputAgentWorkflow(tools=[sql_tool, vector_tool], verbose=True, timeout=120)
result = await wf.run(message="Show me technologies discussed in articles about AI from 2024 ")

What I notice is that generated query ignores my MetadataInfo defined and try to force float type conversion on the string type:

WHERE (metadata_->>'created_at')::float >= 2024.0 ORDER BY distance asc

Which is failed:

DBAPIError: (sqlalchemy.dialects.postgresql.asyncpg.Error) <class 'asyncpg.exceptions.InvalidTextRepresentationError'>: invalid input syntax for type double precision: "2024-11-01T00:41:07.216103"

I think it needs to respect data types defined via VectorStoreInfo. Am I missing something?
Here is the relevant code:
https://github.com/run-llama/llama_index/blob/35906a1b8c1efc8fc649dd49a12e8939dedb662b/llama-index-integrations/vector_stores/llama-index-vector-stores-postgres/llama_index/vector_stores/postgres/base.py#L505

Relevant issue: https://github.com/run-llama/llama_index/issues/16871


[SPARK-19603][SS]Fix StreamingQuery explain command

What changes were proposed in this pull request?
StreamingQuery.explain doesn't show the correct streaming physical plan right now because ExplainCommand receives a runtime batch plan and its logicalPlan.isStreaming is always false.
This PR adds streaming parameter to ExplainCommand to allow StreamExecution to specify that it's a streaming plan.
How was this patch tested?
The updated unit test.

Test build #72909 has started for PR 16934 at commit 3b6c86a.

Test build #72909 has finished for PR 16934 at commit 3b6c86a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72909/
Test PASSed.

This solution is okay. I'm just not sure why we use lastExecution.logicalPlan instead of logicalPlan directly. In that case, we wouldn't need the streaming = true option for ExplainCommand

Good question. Make me realize my fix was wrong. We cannot use logicalPlan here because it's not the runtime batch plan. The purpose of adding StreamingQuery.explain is to allow the user to see the runtime plan.

Test build #72957 has started for PR 16934 at commit ca268b9.

@zsxwing Left one comment for tests and a question.
It would be great if you could also post a snippet of the plans after this fix.

Test build #72962 has started for PR 16934 at commit 20eae3d.

LGTM pending tests

Test build #72957 has finished for PR 16934 at commit ca268b9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72957/
Test PASSed.

Test build #72962 has finished for PR 16934 at commit 20eae3d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72962/
Test PASSed.

Merged build finished. Test PASSed.

Thanks! Merging to master and 2.1.


Predict just the classes you like

It would be really useful to make a feature that could predict the classes that you like, for example just cars and people. You just have to write --classes 0 2
YOLO has this feature, which is just one line of code to not detect everything in the image but only what you like from the pre-trained image. I would love to know when this feature gets added, I think that would definitely benefit many.
I have been trying to do this, and it is just a lot of hassle and I still did not figure it out. I would love to know if anyone knows how to do this. Thank you for the help! :)

This what I did, I hope it helps you
def filter_chosen_classes(result, chosen_classes):
    """
    chosen_classes list of the classes you want to predict
   Coco classes are numbered from 1 to 80
   """
    result = list(result)
    for cls in range(80):
        if cls + 1 not in chosen_classes:
            result[0][cls] = np.zeros(shape=(0, 5))
            result[1][cls] = []
    return tuple(result)


iPhone stuck a bootlogo after trying to pair

Hi guys,
I'm not sure if it's an Apple issue or if homebridge, so I just let you know about it:
http://forums.macrumors.com/threads/iphone-stuck-at-apple-logo-corrupt-setting.1919018/
(My iOS 9.0 iPad paired without any problems).

That looks really bad... Any chance you can get the log from the device during the boot loop (using Xcode) before another reboot happened?

Yeah, I'm not very lucky about that.
I'll try to get a log tonight. (First have to google, how to do that :))

@Kolbi You can try download Apple Configurator (https://itunes.apple.com/us/app/apple-configurator/id434433123?mt=12) and see if you can get log from device...
Given "So I connected it to my Macbook and for my surprise I could make an actual backup of my phone. So I created the backup, fully restored my phone tried iOS 9.1 PB1 and iOS 9.0 and restored the backup", this isn't a boot-loop, it's more like Springboard keep crashing so if you can get the log, I'll see what I can do there ^^

Thank you in advance!!

You should have got some mails :)
Thank you!

I'd suggest you to keep the device on and perform a reset HomeKit from another device (that signed into same iCloud account) which may help Notification Center on current device to recover... If that doesn't work, I guess you'll either have to reset the whole device, manually edit the backup file (remove files about notification center history) and then restore that backup.

Hi, thank you for your advice.
I can test it later this week (the iPhone is on the way to Apple to be replaced).
Restoring the backup need a lot of time... but I'm first trying to restore it. Delete HomeKit from another device (my current iPhone 4S) and if that doesn't help I have to make a new backup from the restored backup but this time unencrypted. At the point manually editing backup file I might need your help again :)
I hope my backup is not damaged already, I have run https://deciphertools.com/decipher-backup-repair.html and it showed me some file size errors within my backup...

My phone is back, I'm now trying to restore the backup. I connected my iPhone 4S with homebridge.
After restoring I'm going to delete all HomeKit data from the 4S and after that I try to delete the app via iTunes from my iPhone 6.
I hope it reboots then...
Could you see froms Log which App produces the issue in Springboard?

Based on the crash report you sent, Notification Center itself crashes…

Yesterday I tried to restore the backup on my iPhone 6 with 9.0.1 and after an hour or so it's stil stuck at the last point (step 6) on synching ("Auf das Anwenden der Änderungen warten" / "Waiting for apply the changes"). By the way copying the apps also needs a lot of time like everything (I guess because the Springboard reboots).
Tonight I'm going to try to restore the backup on iOS 9.1 Public Beta 2 (maybe the springboard issue is fixed in PB2?).
My problem is that my backup is encrypted and I guess I'm not able to modify it - or do you know a method? Creating an unencrypted backup is not possible because it can't fully restore the encrypted.

@Kolbi yeah, it's not possible to modify an encrypted backup and also not possible to create an unencrypted backup without first restoring the encrypted backup to iOS device... It's really strange that misbehaved notification can cause issue like this. If iOS 9.1b2 doesn't fix the issue, I guess you'll have to setup iPhone without restoring backup...

@Kolbi btw, if you have TimeMachine enabled on your computer, you may be able to recover by restoring an older backup.

@KhaosT yeah I might have an backup from August (?) available but I'm not sure if I'm going to restore it to my iPhone 6.
Sadly I'm doing to less backups (Time Machine and iOS) to have an actual backup in the backhand. Yeah I know I'm always telling to friends to do regularly backups as an IT expert I might should do it as well :) but as a fact that I'm using an external usb disk for TimeMachine backups I'm to lazy to do it regularly. Local Time Machine backups has been disabled because of to less space on my SSD. But I'm going to have a look at it tonight. Maybe "El Captain" has enabled it again.
Maybe I'm going to restore the old Backup to my iPhone 4S and going to build up everything from scratch on my iPhone 6 with the 4S as template.
But my last hope is that PB2 is able to boot with the latest backup. That would be great.

Since there have been multiple ios and homebridge updates and no more activity on this issue, I will assume your issue has long been resolved.


CLI generates code with invalid imports on Windows

Describe the bug

Running mesh build generates JS code with invalid require paths on Windows. I tried to generate client code for MS Graph and run a script using the generated code. This throws an error the following error: Error: Cannot find module './sourcesMicrosoft Graphmetadata.json.js'. As you can see the path to the metadata is invalid, it should actually be ./sources/Microsoft Graph/metadata.json.js. But it seems the path separates have somehow been lost.
To Reproduce
Steps to reproduce the behavior:

Clone the repo, navigate to the examples-odata-msgraph-programmatic sample
Run yarn start or npm start
You will get an error like:

Error: Cannot find module './sourcesMicrosoft Graphmetadata.json.js'
Require stack:
- <redacted>\graphql-mesh\examples\odata-msgraph-programmatic\.mesh\index.js
- <redacted>\graphql-mesh\examples\odata-msgraph-programmatic\index.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:902:15)
    at Function.Module._load (internal/modules/cjs/loader.js:746:27)
    at Module.require (internal/modules/cjs/loader.js:974:19)
    at require (internal/modules/cjs/helpers.js:92:18)
    at Object.<anonymous> (<redacted>\graphql-mesh\examples\odata-msgraph-programmatic\.mesh\index.js:3572:74)
    at Module._compile (internal/modules/cjs/loader.js:1085:14)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:14)
    at Module.require (internal/modules/cjs/loader.js:974:19) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '<redacted>\\graphql-mesh\\examples\\odata-msgraph-programmatic\\.mesh\\index.js',
    '<redacted>\\graphql-mesh\\examples\\odata-msgraph-programmatic\\index.js'
  ]
}

Further, if you open the generated JS module in .mesh/index.js and search for "Microsoft Graph", you will see the following statement (in my case it's on line 3572):
const sourcesMicrosoft_Graphmetadata_json_js_1 = tslib_1.__importDefault(require("./sourcesMicrosoft Graphmetadata.json.js"));

Notice the call:
require("./sourcesMicrosoft Graphmetadata.json.js")

Expected behavior

The generated code should not have import errors. The correct require call should look like:
require("./sources/Microsoft Graph/metadata.json.js")

Environment:

OS: Windows 10
@graphql-mesh/cli: 0.34.3
NodeJS: 14.7.3

Additional context

I investigated this issue and found that it's related to Windows paths using backslash as the separator. The error seems to result from the generateTsArtifacts function in the packages/cli/src/commands/ts-artifacts.ts module. The path to the metadata.json.js file is among the imported modules in the generated TS code (see the importCodes variable) that will be passed to the ts compiler. When the path is a Windows-style path, it leads to the following import statement in the TS code:
"import ExternalModule_3 from './sources\\Microsoft Graph\\metadata.json.js';"

I'm not sure at which point the backslashes are removed, probably by the ts compiler.
I've found that if you change the importPath to a posix path, then you get the expected results, something like this:
const importedModulesCodes: string[] = [...importedModulesSet].map((importedModuleName, i) => {
  let moduleMapProp = importedModuleName;
  let importPath = importedModuleName;
  if (importPath.startsWith('.')) {
    importPath = join(baseDir, importPath);
  }
  if (isAbsolute(importPath)) {
    moduleMapProp = relative(baseDir, importedModuleName);
    importPath = `./${relative(artifactsDir, importedModuleName)}`;
  }

+ if (path.sep === path.win32.sep) {
+   importPath = importPath.replace(new RegExp(`\\${path.sep}`, 'g'), path.posix.sep);
+ }
  const importedModuleVariable = pascalCase(`ExternalModule$${i}`);
  importCodes.push(`import ${importedModuleVariable} from '${importPath}';`);
  return `  // @ts-ignore\n  [\`${moduleMapProp}\`]: ${importedModuleVariable}`;
});

Not sure if that's the most desirable solution, or whether that's the ideal place to make the code changes. But that seems to fix the issue.

I ran into the same issue and your suggestion, @habbes, worked for me as well.

@habbes Were you able to get everything to work after putting in your change? I ran into the issue where one of my modules wasn't being recognized. After debugging I realized I had to add
const importedModulesCodes: string[] = [...importedModulesSet].map((importedModuleName, i) => {
  let moduleMapProp = importedModuleName;
  let importPath = importedModuleName;
  if (importPath.startsWith('.')) {
    importPath = join(baseDir, importPath);
  }
  if (isAbsolute(importPath)) {
-  moduleMapProp = relative(baseDir, importedModuleName);
+  moduleMapProp = `./${relative(baseDir, importedModuleName)}`;
    importPath = `./${relative(artifactsDir, importedModuleName)}`;
  }

+ if (path.sep === path.win32.sep) {
+   importPath = importPath.replace(new RegExp(`\\${path.sep}`, 'g'), path.posix.sep);
+   moduleMapProp = moduleMapProp.replace(new RegExp(`\\${path.sep}`, 'g'), path.posix.sep);
+ }
  const importedModuleVariable = pascalCase(`ExternalModule$${i}`);
  importCodes.push(`import ${importedModuleVariable} from '${importPath}';`);
  return `  // @ts-ignore\n  [\`${moduleMapProp}\`]: ${importedModuleVariable}`;
});

Now I'm running into
Mesh: Cannot read property 'getQueryType' of undefined
Mesh: {
  name: 'TypeError',
  stack: "TypeError: Cannot read property 'getQueryType' of undefined\n" +
    '    at ...\\node_modules\\@graphql-mesh\\merger-stitching\\index.js:92:57\n' +
    '    at Array.map (<anonymous>)\n' +
    '    at StitchingMerger.getUnifiedSchema (...\\node_modules\\@graphql-mesh\\merger-stitching\\index.js:91:38)\n' +
    '    at Object.getMesh (...\\node_modules\\@graphql-mesh\\runtime\\index.js:105:46)\n' +    '    at processTicksAndRejections (internal/process/task_queues.js:93:5)\n' +
    '    at async Object.handler (...\\node_modules\\@graphql-mesh\\cli\\bin.js:868:17)',
  message: "Cannot read property 'getQueryType' of undefined"
}

but don't know what the root of this is.

@Davenporten In my case the code gets generated correctly, I also get an undefined-related error when I try to run my script based on the generated code:
TypeError: Cannot read property 'Edmx' of undefined
    at ODataHandler.getMeshSource (...\node_modules\@graphql-mesh\odata\index.js:128:38)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async ...\node_modules\@graphql-mesh\runtime\index.js:82:24
    at async Promise.all (index 0)
    at async Object.getMesh (...\node_modules\@graphql-mesh\runtime\index.js:78:5)
    at async start (...\index.js:6:20)

I'm not sure if this is related to your issue. But after investigating, in my case it's based on a bug in the cache proxy used by GraphQl-Mesh. Specifically the proxy.getWithSet() method which is supposed to fetch the OData schema and cache it. However, it does not store the fetched schema and returns an unefined value instead.

Probably we broke something after those changes :)

Haha, story of my life

Ok I've setup a Windows environment then try to reproduce and fix the issue. Now it seems working. I released a new version with the fix. Could you try it again?
A note for me;
We should definitely have tests for Windows and POSIX environment that covers this flow.

I will give it a try. Hopefully won't have much delay in getting to npm this time.

https://github.com/Urigo/graphql-mesh/runs/3377639976
You can track the publish progress here :)

Yeah, so my coworker just told me that npm might not get updated through our proxy for 24 hrs. I'll test this right when it comes through.

@ardatan it works well for me now. Even the cannot read property 'Edmx' of undefined error seems to be gone now. My script is able to make queries to MS Graph.

Things seem to be working for me well, though I can't fully confirm yet. cli is looking for @graphql-mesh/store v0.1.7 and that has not made it through to me yet. Once that's through I'll be able to be sure, but all other issues seem to be resolved.

Cool! Let's close this issue then. We can always reopen it later

@ardatan I'm not getting any errors on build or start up, but when I try to hit http://localhost:4000/graphiql I get
Cannot GET /graphiql

As always, could be something on my end, but I'm not sure where to start on this.

Not sure how your setup looks like but we use /graphql not /graphql as endpoint and GraphiQL is disabled in production mesh start command which can be enabled with
serve:
    playground:true

That did it @ardatan ! Thanks again for all the help!

I'm experiencing this (I believe the same) issue as well.
The ./mesh/index.js file contains paths to my queries that are incorrect.
it gets generated like this:
const certificates_query_ts_1 = tslib_1.__importDefault(require("./../../../../../resolvers/certificates.query.ts"));
const certificates_mutation_ts_1 = tslib_1.__importDefault(require("./../../../../../resolvers/certificates.mutation.ts"));

But it should look like this:
const certificates_query_ts_1 = tslib_1.__importDefault(require('./../resolvers/certificates.query.ts'));
const certificates_mutation_ts_1 = tslib_1.__importDefault(require('./../resolvers/certificates.mutation.ts'));

My config file looks like the following:
transforms:
  - mock:
      if: true
      initializeStore: ./resolvers/certificates.query.ts#initializeStore
      mocks:
        - apply: Query.certificate
          custom: ./resolvers/certificates.query.ts#certificate
        - apply: Mutation.updateCertificate
          custom: ./resolvers/certificates.mutation.ts#updateCertificate
        - apply: Mutation.createCertificate
          custom: ./resolvers/certificates.mutation.ts#createCertificate

And I build the artifacts with: mesh --dir apps/bd/api/src build

Can you post the versions of the difference packages you're using?

@habbes I haven't been working on graphql-mesh stuff for a while and just updated @graphql-mesh/cli to 0.39.0 and am now getting this issue again. From the comment above it seems like maybe somehow it was introduced easier? Have you updated your version? Are you seeing this issue again?

I have also tested it with graphql-mesh/cli: 0.39.0 and Yarn 3, no change in response.

How about we re-open this issue?

That would work for me, though at this point I'm sorry to say that I'm no longer working with graphql-mesh and won't be able to test it.


create ceduphh.txt

website link: http://www.ceduphh.com.br/home/
courses link: http://www.ceduphh.com.br/cursos/?curso=CURSOS TÉCNICOS

@GRParasky I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host ceduphh.edu.br not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for understanding!

Sorry for the misunderstand. I will make a new pull request with the right
domain.
Em sáb., 14 de mar. de 2020 às 01:01, Philip Torchinsky <
<EMAIL_ADDRESS>escreveu:

@GRParasky https://github.com/GRParasky I have to close this request
without merging, because you requested to add a non-existent domain to this
repository: Host ceduphh.edu.br not found: 3(NXDOMAIN) Please ensure that
you are familiar with this repository rules located at the bottom of this
page https://github.com/JetBrains/swot and submit a new request with
the correct information. Thank you for understanding!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/7918#issuecomment-599007248, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMTLEQRIRYREZTH6VAAVN33RHL6QLANCNFSM4LH2U34A
.


Please provide more flexible gql querying

Currently all queries are effectively AND queries. We need to have things like OR, Contains, Greater than or Less Than, Starts With, Ends With, Not Equal, Ranges, etc.
This is hard but for some workflows really necessary. Perhaps start with Contains and Greater Than and Less Than?

I'm happy to help design individual query, but there's a limitation inherent with OR and NOT_EQ queries which I can't fix. There are good talks and articles describing why this isn't a feature supported in big-data databases and whey they are inherently slow. There should otoh always be a way to query the data in alternative kind of fashion that you get what you are looking for. Ranges are supported (height min height max in blocks query as example), contains is also possible ex. providing list of ids. But each way of querying requires enormous amount of time in design.
In any case, some feature could be missing which we can support from the list of contrains you mentioned. If we find the highest priority queries we need to support on work on support for them one by one, then I think we are on a good track, knowing that still, it's a lot of work making fast queries with huge amount of data (which is getting exponentially bigger these days).

I understand your concern, but even adding one of those options would help
Thanks
Sent from my iPhone
On Aug 21, 2021, at 12:55 PM, Hlöðver Sigurðsson @.***> wrote:

I'm happy to help design individual query, but there's a limitation inherent with OR and NOT_EQ queries which I can't fix. There are good talks and articles describing why this isn't a feature supported in big-data databases and whey they are inherently slow. There should otoh always be a way to query the data in alternative kind of fashion that you get what you are looking for. Ranges are supported (height min height max in blocks query as example), contains is also possible ex. providing list of ids. But each way of querying requires enormous amount of time in design.
In any case, some feature could be missing which we can support from the list of contrains you mentioned. If we find the highest priority queries we need to support on work on support for them one by one, then I think we are on a good track, knowing that still, it's a lot of work making fast queries with huge amount of data (which is getting exponentially bigger these days).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FArweaveTeam%2Fvartex%2Fissues%2F25%23issuecomment-903144413&data=04|01||1f00f9914acb4e7dfc2408d964c468e0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637651617001071050|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=6lXZFp02MudGBupH1Xzr5yQTdq9NSE9yRGH77Hi58Io%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAALIC552PZIVXLIE35UUZM3T57K6DANCNFSM5CR6D7WQ&data=04|01||1f00f9914acb4e7dfc2408d964c468e0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637651617001071050|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=nOJWdQ8VkpGNDkKDDk6jPFctL%2FzkNAeZDSoeLVp4rw4%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01||1f00f9914acb4e7dfc2408d964c468e0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637651617001081044|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Pg7NMaWMAOSLmg1TpJDzzFCpV19d2VUkLFDwqZLl4tg%3D&reserved=0 or Androidhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26utm_campaign%3Dnotification-email&data=04|01||1f00f9914acb4e7dfc2408d964c468e0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637651617001081044|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=0L6BFjAeXgpTp6mFMI49nv2ZWGW%2FZkVdqD%2BDB0egv60%3D&reserved=0.


Fix include_str! compile error in examples/scenes.

I don't think CARGO_MANIFEST_DIR should ever end in "/", though I've only checked on Linux and macOS.

Full error trace, when running cargo run_wasm --package with_winit:
   --> examples/scenes/src/test_scenes.rs:244:28
    |
244 |               let contents = include_str!(concat!(
    |  ____________________________^
245 | |                 env!("CARGO_MANIFEST_DIR"),
246 | |                 "../../assets/Ghostscript_Tiger.svg"
247 | |             ));
    | |______________^
    |
    = note: this error originates in the macro `include_str` (in Nightly builds, run with -Z macro-backtrace for more info)

error: could not compile `scenes` due to previous error```


Mapping environment variables

Is there any way to map environment variable to custom key? Like command line SwitchMappings.

Triage: closing as we don't plan to do this.


[W4.4c][T16-1]Mohd Aslam

Great job again!

Hi, apologies for the spam. After you read our comments you can close the PR at your end. Note that subsequent PR workflows will be the same.

Thank you for reviewing.


Remove do_call method, extend SpoofTransaction to work with unsigned transactions

What was wrong?
do_call method used a surprise sender to accomplish calls and gas estimation when the signing key is not available.
How was it fixed?
SpoofTransaction will inject a sender attribute when used with an unsigned transaction.  This behaviour requires the parameter from_ to be supplied.
intrinsic_gas and get_intrinsic_gas where moved from the transaction class to the transaction_context class.
EDIT: I also made the transaction executor classes stand-alone rather than a mix-in for the state class.  This made it a little simpler to manage the transaction_context, transaction, message and computation objects during the transaction execution steps as instance attributes.
Cute Animal Picture

I made the assumption that having computation.get_gas_used() include the intrinsic gas would be ok, since the only get_gas_used reference I found was to calculate the total transaction gas used.  Is this an ok approach, or am I mixing together separate conceptual domains?

@dylanjw I think inclusion of the intrinsic gas concept within the TransactionContext is appropriate and does not violate any separation of concerns that come to mind 👍


UI Label / Terminology Change: File Share --> Volume

Bug-id: https://github.com/Tendrl/dashboard/issues/424

@cloudbehl Along with text changes, we should also modify the file names.


[feature][catalog] Support for multiplexing connections

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Please help to look #4540 before merging the current pr.


rename admindb package to roomdb

One thing I kept thinking: since this package takes care of persisting all the data, then maybe admindb isn't really descriptive? roomdb might be better; admindb implies being concerned with a database of admins, I think, suggesting the presence of other databases as well.

Originally posted by @cblgh in https://github.com/ssb-ngi-pointer/go-ssb-room/pull/55#discussion_r586387302

related tasks:

[ ] unify receiver names of handlers and database
[ ] move tests to muxrpc/tests


Update New Line Character Used in Line Highlight Spacing

Fixes #41 using @philippkuehn's suggested fix

Thanks


Configuration: default amount scale

Millions Polish zloty doesn't make sense as a search query. Let's set it in the configuration amount_scale=1000000
Then change translations:

[ ] notices.filter.value.placeholder=Value range in millions, e.g. '0-1000' -> notices.filter.value.placeholder.1000000, notices.filter.value.placeholder.1000
[x] filter.email.noreply - noreply is bad habit, it frustrates your clients. You should always provide a feedback link, ie. "In case of any problems please contact us at {contact_email}."

filter.email.noreply - noreply is bad habit, it frustrates your clients. You should always provide a feedback link, ie. "In case of any problems please contact us at {contact_email}."

filter.email.contact is already there and included in the email template. :)

Sorry, I've meant to copy this translaion into #35

[ ] it is also included in helper sql

Which make it harder to make it configurable. Won't mySQL handle such big numbers?

There's no problem with big numbers in MySQL. I put this dividing into the script to optimize the webapp. :)
I think it's possible to move out this dividing from the helper script and place it into the webapp instead, maybe at FTL level. I'll look into this.

:point_right: http://docs.redflags.eu/developer/cookbook/#step-6-review-webapp-settings
Also I modified the structure of the rfwl_barplot table in the helper SQL script.

That's great @juzraai, thanks! :)