Redshift client

Hello, I've just been through the amazing pain of using the pg client to talk to redshift.
I notice there isn't a redshift client for knex, or maybe I haven't noticed. Since the differences are very fresh in my mind I'd love to help on that.

Yes please! The more the merrier. At some stage we're going to have to break these into different modules... @tgriesser?

@krainboltgreene have you filed specific issues for pg + redshift? I'm using knex with redshift very successfully.

Here's some examples:

knex.fn.now() uses CURRENT_TIMESTAMP, but only leader nodes can use that.
uuid isn't a data type on redshift, but it still tries
text is varchar(256), which is kinda small
index(), unique(), primary() are essentially no-op, but it still tries to do them.
increments() doesn't work, no SERIAL data type.
timestamp() tries to do timestamp with zone, but that doesn't exist on redshift
Not sure where this one goes wrong, but: with Bookshelf.js it tries to use returning {{idAttribute}}, but redshift doesn't support this.

I had do quite a bit to get my setup to work, so I'm not sure how you're getting by @tjwebb unless you're just running SELECTs.

Honestly if someone pulled pg out into a module I would totally immediately write the redshift one with the changes I detailed here.

It should be fairly straight forward to add a new client extending or
copying the pg dialect in this repo. We can split them out later if
necessary.
On Tue, Mar 8, 2016, 5:50 PM Kurtis Rainbolt-Greene <
<EMAIL_ADDRESS>wrote:

Honestly if someone pulled pg out into a module I would totally
immediately write the redshift one with the changes I detailed here.
—
Reply to this email directly or view it on GitHub
https://github.com/tgriesser/knex/issues/1255#issuecomment-193631179.

I'm not that adept yet at contributing to node projects D:

Doesn't seem "solvable" with the current system.
Changed to character varying(65535)
Changed to character varying(65535)
No idea how
No idea what to do
Fixed
Removed
Switched to character varying(65535)
???

See: https://github.com/tgriesser/knex/pull/1263

The current postgres client utilizes pg-cursor to handle streaming. Unfortunately due to a bug in this dependency streaming doesn't behave properly and always downloads the full result set before the stream begins. Please see the related isse for details.
I believe that instead of extending the current pg client to create a redshift client, a redshift client should be built mostly from scratch using Java Database Connectivity (JDBC) as Amazon recommends in their documentation. I've outlined an implementation in a fork here.
In order to use it:

There is an added dependency on node JDBC.
The Java JDK (Version 8 or higher) must be installed. OpenJDK works fine.
The Amazon Redshift JDBC Driver must be made available. Currently in the fork I've hard-coded a version and path for this (JDBC 4.2–compatible driver) but it could easily be set as configurable. I did not include the driver in the repository due to terms in the user-license agreement and I believe it ought to be managed as a dependency another way.

My fork still needs tests and such, but I believe the concept is there. Thoughts?

Java as a dependency would be a no-go for my usage.

What about the ODBC option? There is a node module here which looks promising. It uses unixODBC which would have to be available in the local environment. No Java though!

https://github.com/tgriesser/knex/pull/2233


JS: Get scroll to play working

This seems like it's working now, at least on desktop. I'm a little worried about if it's going to work on mobile, because it might be more strict there about requiring a touch event to trigger play.

I notice that when I scroll out, the text disappears, and then the page jerks up. Maybe to fix that, we should change it to not trigger onScreen = false on x-intersect:leave.


Quarkus platform auto-updater

Changes

:gift: Added periodic task that checks Quarkus platform version in templates and creates PR with update if needed.

BTW: I'll be on PTO until August 15th.

/lgtm
/hold (it's a draft at the moment)

/unhold


OpWithOffsetSizesAndStrides with only leading offset/sizes/strides overflow.

See: https://reviews.llvm.org/D108617
Note, all  OpWithOffsetSizesAndStrides may take only a subset of
leading values (and auto complete the remaining ones to the canonical expected offsets/sizes/strides); as a consequence this may overflow.
Sending a bugfix for this but FYI there may be other occurrences.
One easy way to track them down could be to force the verifier to specify everything and run tests (but that will catch false rank-reducing positives; still it should help).
@springerm if you get to it first.

Wasnt very hard to fix actually. https://reviews.llvm.org/D115677

Note my comment on the tensor versions; those ones are also blocking me to get the e2e parallel stuff working.

Ok, I am looking at that too. Will update the patch.

Updated the patch removing the issues with tensor.s as well. No failures in core. If you have a failure somewhere let me know.

thanks much will try tomorrow!
On Mon, Dec 13, 2021 at 11:31 PM MaheshRavishankar @.***>
wrote:

Updated the patch removing the issues with tensor.s as well. No failures
in core. If you have a failure somewhere let me know.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/iree-llvm-sandbox/issues/103#issuecomment-992972207,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACNNU5D36445I2OSI5YS7WTUQZX4NANCNFSM5JZ66IYQ
.

--
N


CDN: explore ways of making content load faster outside of HK

Hi @rdunk -- as discussed logging this task here for investigation.

@micahwalter this is probably one that you're best placed to answer. I know we discussed this briefly before. My main concern is with image assets at the moment, if a CDN could help cut down loading times for those I think that would be a huge help.

@micahwalter can you let @rdunk and I know what you think? If we're promoting the site globally it'd obviously be good that it's fast.

@micahwalter — 💭?

@micahwalter 🐧

OK - So we are still serving static assets ( images ) directly from the CMS server. To move this to a CDN we would need to do the following:

[ ] Create an S3 bucket on AWS to store images.
[ ] Copy all existing images to S3
[ ] Create a CDN mirror of the S3 bucket with CloudFront
[ ] Set the new CDN domain name somewhere in Processwire so it serves images from there instead of the CMS server.
[ ] Set up the CMS server to sync newly added images to S3 upon upload.

The first three are pretty straightforward and I can do these on Friday. The last two will require some intervention from @rdunk. I imagine there is a Processwire plugin that can handle the last one, but I'm not familiar.
-micah

Point 4 I imagine will be relatively straightforward to do. Point 5 is a little more involved, just due to the way Processwire generates image variants. I'm sure it's doable, dumping some reference links for myself here:
https://github.com/mirzadabeer/ManageImagesOnAmazonS3Bucket
https://github.com/nmendes/amazons3cloudfront

We could also just set up a cron job to sync with S3 every couple of
minutes or something ...
M
On Thu, Nov 30, 2017 at 5:48 AM ʞunp ʇɹǝdnɹ<EMAIL_ADDRESS>wrote:

Point 4 I imagine will be relatively straightforward to do. Point 5 is a
little more involved, just due to the way Processwire generates image
variants. I'm sure it's doable, dumping some reference links for myself
here:
https://github.com/mirzadabeer/ManageImagesOnAmazonS3Bucket
https://github.com/nmendes/amazons3cloudfront
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mplusmuseum/mplusmuseum-stories/issues/122#issuecomment-348152532,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC5O8YWVxUaNWs4m28vFUbPTwBAl72Mks5s7of9gaJpZM4QkC-1
.

@micahwalter @rdunk — wanted to loop back and emphasise this issue. Been speaking to some people in Oz. Loading times are varying a bit, but when they're slow, they're really pretty slow. It would be great to look into this side of the website soon.

@micahwalter Seems sensible that I try and implement this as a module in ProcessWire, would it be possible to set up an S3 bucket to test it with?

@rdunk @micahwalter — fantastic. Let me know how things go with the font files.

ok, I've done this for both production servers. I think you may just need to redeploy the site @rdunk .

Great, fonts now being served by the CDN.


Is there a tree-sitter grammar for RSX/RSTML?

It seems the only attempt at a tree-sitter grammar for RSX/RSTLM was: https://github.com/tjdevries/tree-sitter-rsx
Are there any plans to extend/fork the tree-sitter-rust grammar to add support for RSX?
What editor/extension do you use for actually building things with RSX?
Context: https://github.com/leptos-rs/leptos/issues/2108

Hi!

What editor/extension do you use for actually building things with RSX?

Currently during proc macro expanding - rstml preserve original spans, so any IDE with "semantic highlight" and proc-macro support should work well.
I tested it with rust-analyzer (on vscode) and IDEA Rust plugin.

Is there a tree-sitter grammar for RSX/RSTML?

I dont think so.

Are there any plans to extend/fork the tree-sitter-rust grammar to add support for RSX?

I had no plans for this, and see the following problems with it:

The needs to support two sets of rules (one in rstml code itself, and one specific for tree-sitter)
as you mention in original issue: js and ts parser are both embed jsx/tsx implementation, which is good for new users, since they don't need to install any specific grammar/plugin for html in js.
But rstml is just a syntax for macro, and it differ with yew's implementation, so it no chance that Rust original grammar will embed it at some time, is there a way in tree-sitter to extend existing grammar dynamically?
Also rstml is library for parsing, not a macro implementation, so other libraries could name entrypoint in different ways.


NativeMessaging manifest not installed on macOS

See https://github.com/jelmervdl/translatelocally-web-ext/issues/53. I can reproduce this using the build linked there. No manifest files are written on start-up, nor when I manually call translateLocally with the --update-manifests option.

It seems the macOS version is run from a sandbox, so manifests go into the translatelocally container.
I need to figure out a way to ask the user for consent to escape that container. There are APIs for this (.e.g full disk access, or asking access to the user's home folder should be sufficient really) but I'm not sure how to access those from Qt yet.
One thing that then should definitely change is that we need to not install manifests on launch, but after the user clearly indicated intent. Related to #107 and #108 (which has a mock-up)

Another thing to try (combined with the button): Can I use getExistingDirectory to trigger the macOS Sandbox to ask permission to the ~/Library/Application Support folder? And then write all the manifests I want to write?


test: convert var to const

Converted var to const. Refactored assignment statements
of the form  var var1 \n , var2 to const var \n const var 2
to better maintain consistency with other test files.

Since this change is confined just to the tests (and so doesn't introduce the slowness that old Node.js runtimes experience to running code), it seems fine. It can catch reassignments in versions of Node.js that actually enforce const-ness for the tests I suppose. Thoughts from other collaborators? Best to land this type of churn in master instead of 5.0 for merge-ability.

@sebasmurphy you might want to throw a ; on the end of the require lines you changed from using ,.

Originally I was 👍 on this and even went through resolving the conflicts and went to merge it, all the while thinking why didn't we do this before (and why this PR was only in the tests) until I realized: this won't work in Node.js 0.12 and 0.10 when 'use strict' is added to the files -- it fails with SyntaxError: Use of const in strict mode.

Aren't those versions deprecated now? I thought the LTS finished already.

Express.js 4.x supports those versions.

Express.js 4.x supports those versions, as they were the versions around when 4.0 was first released. There is a thread regarding bumping the minimum version for 5.x, of course. I would like to merge this into the 4.x code ideally because even after 5.0 is released 4.x will continue to be supported for at least a year if not longer, and this would make the backportibg of fixes a bit harder. We can hold the PR for 5.0 though, or at least until the minimum version for 5.0 is decided.

Mucked up my rebase force push.

Should be revisited after #3560 lands.

Makes sense @kibertoad 👍 Right now this PR has a lot of conflicts too, which is probably expected since this touches a lot of the files and lines.

I hope there is no hard feelings, just trying to trudge through all these old pull requests. This one has so many conflicts, it is difficult to resolve and changing to const is now a tool command away. So I'm going to close this to use a tool instead of trying to resolve all the conflicts.

Sure, no worries!


[BEAM-10482] Fix CoGroup for schema-aware POJO PCollections

Fixes issue with schema-aware CoGroup transform. It doesn't work for any schema-aware POJO PCollection and only works with PCollection<Row> because they have identity toRow function.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run Java PreCommit

Run Java PreCommit

':sdks:java:container:pullLicenses' consistently fails for a reason unrelated to this PR, looking into it

Run Java PreCommit

Run Java PreCommit

R: @reuvenlax

Run SQL PostCommit

Thanks for the ping @aaltay, I'll take a look


IDM Crack with Internet Download Manager 6.42

Overview of Internet Download Manager IDM
Internet Download Manager or IDM is an advanced download manager software that makes it easier to manage your downloaded files with the intelligent system, this program will speed up the downloading of files with its new technology, and according to the manufacturer, It can download up to 5 times faster than usual.
Download Link  ✅
Video downloading from almost all video streaming platforms is the best feature of Internet Download Manager. IDM lets you download videos from websites like YouTube, Daily Motion, Vimeo, etc. The video download button will appear automatically when you watch videos; click on it and quickly grab the video in your required SD or HD quality.
Features of Internet Download Manager IDM

Full compatibility with Internet Explorer, Netscape, Google Chrome, AOL, Opera, Mozilla Firefox, Mozilla Firebird, Avant Browser, MyIE2, and other popular browsers.
Download FLV video files from YouTube, Google Video, and MySpaceTV.
Increase download speed up to 5x by Smart File Splitting System
Limiting download speed for specific files
Preview the contents of zip and RAR files before downloading them
Advanced group scheduling and file queuing to suit your needs
The ability to continue downloading to stop and continue downloading at another time without losing downloaded values

✅▶▶DOWNLOAD Internet Download Manager
✅▶▶MIRROR IDM Crack LATEST VERSION


Add values to define extra volumes and mount text content into container.

What this PR does / why we need it:

I need to mount ca-certificates to the file system, so I created the little pr.

Pre-submission checklist:

[x] Did you explain what problem does this PR solve? Or what new features have been added?
[x] Have you updated the readme?
[x] Is this PR backward compatible? If it is not backward compatible, please discuss open a ticket first

Hi,
Thanks for the PR,
i do not quite understand the  customFileSets section.
In the example that you provide it’s exactly the same as customLdifFiles

With customFileSets, we can override any file in the container.
For example, without this, we can not disable memberOf overlay because olcOverlay={1}memberof,olcDatabase={1}mdb,cn=config can not be deleted by ldif with customLdifFiles.
Now we can easily overwrite the 03-memberOf.ldif content to remove the olcOverlay={1}memberof part.


GitHub dependency graph issue tracking

Discovered during #277
Reported a support issue to GitHub.
This ticket is to track the updates.
Dependency graph refers to a strange repository
Hello! I like the dependency graph feature.
However some part of it works not perfect.
Please take a look at
https://github.com/lrozenblyum/chess/network/dependencies

The project uses org.apache.maven.plugins:maven-surefire-plugin.
While other apache plugins are resolved correctly, for this one GitHub shows a link to https://github.com/kiranpuram/Git_Maven_Jenkins
instead of
https://github.com/apache/maven-surefire
Please improve the dependency graph.

Solved, received letter from GitHub on the 1 Mar 2019


Configure Email Alert Channel

Hello,
I’m encountering an issue while configuring the Alert channel type email in Signoz. In the documentation provided here https://signoz.io/docs/alerts-management/notification-channel/email/#configuring-alertmanager , it mentions that I need to configure a few environment variables. However, in the UI, when configuring it, it only asks for the recipient mail address. Where do I need to configure those values to use the Email Type alert channel?
I deployed Self-Hosted Signoz using the Helm chart in Kubernetes.

The doc is updated.


If an error is thrown while calling the WS middleware - next that

First take on fixing error handling is to send any errors thrown along the list of middleware using the next function.
This fixes at least some of the variants of https://github.com/HenningM/express-ws/issues/19

Hi @kraenhansen,
I think this looks like a good remedy for the bad error swallowing habit this library has struggled with, so thanks a lot! I'll release a new version of the library next week, containing your fix + a few other ones.


support kubeconfigs with more than one document

While we now support stacked kubeconfigs (see #132), and have all the merging logic for it, we crash on multi-document kubeconfigs.
Create two k3d clusters, and save their outputs in a single file:
k3d cluster start promstack
k3d cluster start kube
k3d kubeconfig get --all > ~/.kube/k3d
export KUBECONFIG="$HOME/.kube/k3d"

You'll have a file with two ---\n delimited yaml objects, which could be merged.
kubectl handles it, kube crashes:
Error: Error loading kubeconfig: Failed to infer config.. cluster env: (Error loading kubeconfig: Unable to load in cluster config, KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT must be defined), kubeconfig: (Error loading kubeconfig: Failed to parse Kubeconfig YAML: deserializing from YAML containing more than one document is not supported)

Caused by:
    0: Failed to infer config.. cluster env: (Error loading kubeconfig: Unable to load in cluster config, KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT must be defined), kubeconfig: (Error loading kubeconfig: Failed to parse Kubeconfig YAML: deserializing from YAML containing more than one document is not supported)
    1: Error loading kubeconfig: Failed to parse Kubeconfig YAML: deserializing from YAML containing more than one document is not supported
    2: Failed to parse Kubeconfig YAML: deserializing from YAML containing more than one document is not supported
    3: deserializing from YAML containing more than one document is not supported

We could probably the yaml objects together and flatten the reads to get the same result in file_config.rs.
AFAIKT: It might be enough to change: Kubeconfig::read_from to:

have serde_yaml::from_reader call return Vec<Kubeconfig>
lift remapping  path inside a loop over configs
fold the kubeconfigs together using Kubeconfig::merge
return an already merged Kubeconfig

Fold might be awkward, since you need to fold over an existing Kubeconfig, but fold_first is almost stable!
Last point; this isn't a bad lack. You can tell k3d to give you delimited path output with multiple config files (k3d config merge - just don't use it with --output) and fallback to stacked kubeconfigs. It would just be nice to not crash on what kubectl handles.

Merging looks ok with a manual fold_first that has simplifies error handling. In the new lifted loop:
        let configs: Vec<Kubeconfig> = serde_yaml::from_reader(f).map_err(ConfigError::ParseYaml)?;

        // Remap all files we read to absolute paths.
        let mut cfg = None;
        for mut config in configs {
            // OLD REMAP LOGIC HERE INDENTED
            if let Some(c) = cfg {
                cfg = Some(Kubeconfig::merge(c, config)?);
            } else {
                cfg = Some(config);
            }
        }
        Ok(cfg.expect("Need at least one yaml document in KUBECONFIG file"))

But looks slightly more complicated due to https://www.reddit.com/r/rust/comments/ccpeh4/is_there_a_way_to_deserialize_a_multiple/
might need to split on --- ...

Opportune pr: https://github.com/dtolnay/serde-yaml/pull/189

Released in 0.51.0


Does this issue occur when all extensions are disabled?: yes

Please retry without extensions enabled. Select 'Extensions View > ... > Disable all Installed Extensions' and try again. Thanks.

Thanks @jrieken, you're right it's the fault of an extension. I'll narrow down which one.

Edit: weirdly just disabling and enabling the plugins (one by one as I was testing) fixed the issue


Inconsistent code formatting

https://github.com/bcgit/bc-java/blob/a66e904a431d992281c8378064fed4cbc4ab1d99/prov/src/main/java/org/bouncycastle/x509/X509V1CertificateGenerator.java#L47-L50
The variable sigOID is not aligned vertically with the other variables.
This means that there is no enforced formatting style.
This in turn means it is possible to have misleading indentation, like in the popular goto fail bug.

A good unit test (i.e. providing good coverage of the CUT) should prevent bugs of the "goto fail"-category.


No annotations 2017

Not able to see any annotations in my vs2017 scroll bar

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

ID: 9440cd51-f00a-2a66-c895-0c093c6c883d
Version Independent ID: 77cf076b-949f-ce9a-882f-7a0c02d268d0
Content: How to: Track your code by customizing the scrollbar - Visual Studio
Content Source: docs/ide/how-to-track-your-code-by-customizing-the-scrollbar.md
Product: visual-studio-dev15
GitHub Login: @gewarren
Microsoft Alias: gewarren

@bharath-ram Perhaps you need to switch to "bar mode" instead of "map mode". Can you check in Tools > Options > Text Editor > All Languages > Scroll Bars?

Same issue here. VS 15.8.6. Map and bar mode doesn't work. After disabling and enabling annotations for vertical scroll bar, annotations a short time visible after OK click in Options window.

@ermajo89 @bharath-ram If you are still having scroll bar issues, please report the issue on Developer Community (which will ask you to use the Help > Send Feedback > Report a Problem menu in Visual Studio so that diagnostic information is captured along with the report).


Cant decode

I could not decode

this image.

Huh, this is an odd one, will take a look. Thanks!

Thanks for reporting this! It turns out that QR code had ECI encoding which we didn't handle. We added support for ECI in #71 and published it on npm as 1.1.0. Let me know if you have any additional issues.


Memory

When I use like this，everytime i went into secondController, the app memory has grow and will not release :
`#import <FluentDarkModeKit/UIImage+DarkModeKit.h>
@interface SecondViewController ()
@property (nonatomic, strong) UIImageView *imageV;
@end
@implementation SecondViewController

(void)viewDidLoad {
[super viewDidLoad];
// Do any additional setup after loading the view.
self.view.backgroundColor = [UIColor whiteColor];
UIImage *lightImage =  [UIImage imageNamed:@"ad_owner_header"];
UIImage *darkImage = [[self class] image:lightImage byApplyingAlpha:0.5];
UIImage *image = [UIImage dm_imageWithLightImage:lightImage darkImage:darkImage];
self.imageV.image = image;
[self.view addSubview:self.imageV];
}

(UIImage *)image:(UIImage *)image byApplyingAlpha:(CGFloat) alpha {
UIGraphicsBeginImageContextWithOptions(image.size, NO, 0.0f);
CGContextRef ctx = UIGraphicsGetCurrentContext();
CGRect rect = CGRectMake(0, 0, image.size.width, image.size.height);
CGContextScaleCTM(ctx, 1, -1);
CGContextTranslateCTM(ctx, 0, -rect.size.height);
CGContextSetBlendMode(ctx, kCGBlendModeMultiply);
CGContextSetAlpha(ctx, alpha);
CGContextDrawImage(ctx, rect, image.CGImage);
UIImage *newImage = UIGraphicsGetImageFromCurrentImageContext();
UIGraphicsEndImageContext();
return newImage;
}

(UIImageView *)imageV {
if (!_imageV) {
_imageV = [[UIImageView alloc] initWithFrame:CGRectMake(100, 200, 40, 20)];
}
return _imageV;
}`

Hi there. can you create a sample project and upload it here for us to test, that would make it easier for us to locate the issue.

here is the demo : DYDemo2.zip

Please follow the guide https://github.com/microsoft/FluentDarkModeKit#get-started to correctly set up FluentDarkModeKit before using it. (It might not be working with Objective-C as of now since the setup functions are swift only, but should be easy to change it to Obj-C compatible).
I tested with your project without using FluentDarkModeKit,  replaced
UIImage *image = [UIImage dm_imageWithLightImage:lightImage darkImage:darkImage];
self.imageV.image = image;

with
self.imageV.image = lightImage;

there is still the memory growth that did not go down after going back to the first VC.

if you use like this
self.imageV.image = lightImage;
the memory will grow up when went to second VC only on first time.
but, if use FluentDarkModeKit the memory will growth everytime when i go to second VC

This is done without using FluentDarkModeKit but the memory is increasing.
https://user-images.githubusercontent.com/17924287/118607344-0a200400-b7eb-11eb-84b8-6aa4319b265e.mov

oh, thank you, i find that the memory growth is bacause of  "UIGraphicsBeginImageContextWithOptions" .

hi,  i tried like this, app memory are not  increasing:
UIImage *darkImage = [[self class] image:lightImage byApplyingAlpha:0.5];      // UIImage *image = [UIImage dm_imageWithLightImage:lightImage darkImage:darkImage];     self.imageV.image = lightImage;

i find that if i create image with UIGraphicsBeginImageContextWithOptions and register the image in to UIImageAsset, the memory  will never release.

do you have any good advice?

I think the question is out of the scope of current library. It's more of how your app manage images.

it sounds like it is related to how UIKit manages image assets, so we cannot solve the issue where images are not released as a library.
But we do offer an option to not use image asset.
‘’’
configuration.useImageAsset = false
‘’’
Please see to the detail in get started section


Add ToolACE handler

This PR adds the handler of the ToolACE model, which finetunes LLaMA-3.1-8B-Instruct model with ToolACE dataset, obtaining wonderful points in functional calling.
Here are the results on the latest (0824) version evaluated in our machine:

Rank
Overall Acc
Model
AST Summary
Exec Summary
Simple AST
Multiple AST
Parallel AST
Parallel Multiple AST
Simple Exec
Multiple Exec
Parallel Exec
Parallel Multiple Exec
Irrelevance
Relevance

1
85.91%
ToolACE-8B (FC)
80.24%
92.21%
70.76%
85.78%
84.00%
80.42%
96.86%
94.00%
88.00%
90.00%
81.50%
87.80%

Thanks for your efforts in holding such a wonderful leaderboard. We need your help (@HuanzhiMao, @CharlieJCJ ) in adding our model to the leaderboard. Thanks a lot～

Thank you for the PR and welcome! We’re currently busy with a new dataset release. I’ll review your submission later this week and aim to provide feedback by next Monday. Apologies for the delay.

Hey @XuHwang, one quick question: Where can I find the documentation of this model? Eg, it's input schema, prompt format, etc.

Hey @XuHwang, one quick question: Where can I find the documentation of this model? Eg, it's input schema, prompt format, etc.

Thanks for the reply.
The format of prompt is actually given in the handler, which is similar to the prompt of LLaMA3. The format should be:
<|begin_of_text|><|start_header_id|>system<|end_header_id|>\n\n{SYSTEM_PROMPT}{TOOL_DESC}<|eot_id|><|start_header_id|>user<|end_header_id|>\n\n{QUERY}\n{LANGUAGE_SPECIFIC_HINT}<|eot_id|><|start_header_id|>assistant<|end_header_id|>\n\n

So why don't we just use the LlamaHandler instead? The ToolACEHandler has the exact same _format_prompt, decode_ast, decode_execute, etc as the LlamaHandler.

So why can't we just use the LlamaHandler instead? The ToolACEHandler has the exact same _format_prompt, decode_ast, decode_execute, etc as the LlamaHandler.

Actually the decode_ast, decode_execute are consistent with the LlamaHandler, but the _format_prompt and process_input are slightly different, and we find it would cause obvious performance degradation.

Compared with the current latest LlamaHandler from main branch commit eca516a, your ToolACEHandler differs in the following two ways:

Your _format_prompt has two more \n after each <|end_header_id|> tag. This is an oversight on our end; we missed it for the Llama models. I have just fixed it in [BFCL] Fix Llama Handler #626. After the fix, they should be the exact same.
You choose to add the language_specific_hint to the end of the last user prompt instead of as part of the function description in the function doc. This is not allowed. The func_doc_language_specific_pre_processing is not model-specific (on the other hand, functions like convert_to_tool are model-specific); we perform it for all models regardless, you can think of it as part of the dataset transformation process (before we enter the model-specific format transformation phase)

Thanks for the explanation and sorry for the late reply and updates.
We follow your suggestions for using LlamaHandler as our model. And our model is updated, maybe you need to update the local copy if you have always downloaded it before.

We close this PR because we have updated the ToolACE handler to make it compatible with the latest BFCL in #653 .


Strict compile for queryResultsView

This is needed for fixing sql/workbench/contrib/queryHistory - splitting it into separate PRs for easier reviewing.

Pull Request Test Coverage Report for Build 267856994

1 of 40 (2.5%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 38.083%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/contrib/query/browser/queryResultsView.ts
0
39
0.0%

Totals

Change from base Build 267751368:
-0.03%

Covered Lines:
19129

Relevant Lines:
45842

💛  - Coveralls


Maintained?

Question

It seems like this repository has not been touched in a while.
Is this repo in need for maintainers?
This repo was mentioned in https://github.com/react-native-community/discussions-and-proposals/issues/176#issuecomment-562354454 as one of the repos having trouble being maintained.
If it's something I can help, I'd like to stand up to be a maintainer help out the react-native-community.
Any thoughts?

This package is using some deprecated APIs since iOS 10

It looks like https://github.com/react-native-community/push-notification-ios/pull/66 addressed some of this! Not sure if there are other cases, though. 🙂


network: delegate file event creation to io handle

Avoid exposing the fd and allow the io handle to build custom file events.
Signed-off-by: Florin Coras<EMAIL_ADDRESS>Risk Level: Low
Testing: unit tests
Docs Changes: n/a
Release Notes: n/a

cc @antoniovicente would be good to get your review on this!

/assign @antoniovicente


Remove CMS dependency (just framework suffices)

This module doesn't seem to actually require CMS(?)
If correct, the dependencies should be changed to just framework so it can also be used on framework-only projects.

Fixed with #2


Can't build udisks

Trying to build locally spaceFM and having udisks 1.0.5 as a dependency.
Can't build this pkg due to error. Indeed that one https://bugs.freedesktop.org/show_bug.cgi?id=90778
What do we need to do? Patch udisks in nix?

Yes, patch udisks 1.0.5

https://github.com/NixOS/nixpkgs/commit/1af969f8f326511c08ac7e05a9b9cad049b18df4


babel 7 support

I am using babel7.0.0-beta.34 but got below error
Module build failed: TypeError: Invalid attempt to destructure non-iterable instance
The error was throw from this line
https://github.com/michalkvasnicak/babel-plugin-css-modules-transform/blob/master/src/index.js#L103
thisPluginOptions = options.plugins.filter(
      ([plugin]) => plugin.manipulateOptions === pluginApi.manipulateOptions
)[0][1];

The options.plugins object was changed in Babel7, can we consider fix it like this?
https://github.com/michalkvasnicak/babel-plugin-css-modules-transform/compare/master...kovenliao:fix?expand=1

yes please ^^

please review
https://github.com/michalkvasnicak/babel-plugin-css-modules-transform/pull/73/files


Migrate UWP App to Windows SDK: Error after update with MSIX

Describe the bug
I currently have a UWP App. I want to update this to WindowsAppSDK project. I am testing deployment of the app. I created a sample app and installed msixupload bundle file. After I migrate to windows app sdk and then installing the updated version of the app using sideload, I get deployment errors.
Steps to reproduce the bug

Generate a sample UWP app
Install the app with app version X
With upgrade assistant tool, migrate the app to windows sdk
Make sure the certificate remains same
Check if the windows sdk project builds
Update the version (greater than X) of the windows sdk generated using upgrade assistant and try to install

After installing the msix-Update, an error appears:

AppX Deployment operation failed for package 36148Vagisha.BasicUWPApp_<IP_ADDRESS>_x64__4mc14ffpd8dt0 with error 0x80073CF3. The specific error text for this failure is: Deployment Add operation with target volume C: on Package 36148Vagisha.BasicUWPApp_<IP_ADDRESS>_x64__4mc14ffpd8dt0 from:  (BasicUWPApp_<IP_ADDRESS>_x64_Debug.msix)  failed with error 0x80073CF3. See http://go.microsoft.com/fwlink/?LinkId=235160 for help diagnosing app deployment issues

Interesting this is that when the UWP app is not installed (that is I try to do a fresh install of the app) - I get no issues with installation of windows sdk project
Also when I try to do the opposite, that is 1st install the SDK app and then the UWP, that works fine as well, no issue with update.
When I update from SDK (old version) to SDK (new version) => this also works well.
Only case not working is UWP (old version) to SDK (new version) which I need for migration from UWP to SDK.
I have looked at event viewer and don't have any information.
Is there some kind of setting I am missing?
Expected behavior
The app should update without deployment errors when updating from UWP to SDK.
Screenshots
No response
NuGet package version
Windows App SDK 1.2.2: 1.2.221209.1
Packaging type
Packaged (MSIX)
Windows version
Windows 10 version 22H2 (19045, 2022 Update)
IDE
Visual Studio 2022
Additional context
No response

Microsoft's "Package Deployment Manager" Wizard @DrusTheAxe knows better.

0x80073CF3=ERROR_INSTALL_RESOLVE_DEPENDENCY_FAILED
That means your package's appxmanifest.xml declares a <PackageDependency> that could be satisfied by the packages available on the system when you tried to install.
At a guess, you have WinAppSDK 1.2.2 installed and the package you're trying to install manifests <PackageDependency Name="Microsoft.WindowsAppRuntime.1.2" ... MinVersion="...version..."> where version is the MSIX package version for 1.2.3+?
Your manifested package depednencies can be satisfied by what's currently on the machine and additional packages (if any) passed to the PackageManager API via the DependencyPackageUris parameter. When you 'run' a .msix[bundle] Windows will run the app associated with the .msix[bundle] file type passing your package as a parameter to the app. Windows includes the 'App Installer' app which defines file type associations for MSIX file types (.msix, .msixbundle, more) and the AppInstaller app has no additional packages to pass to PackageManager.AppPackageAsync(...DependencyPackageUris...).
You can get more details about the failure (e.g. the dependency that couldn't be satisfied) if you review the eventlog's Microsoft-Windows-AppxDeploymentServer/Operational channel. You'll see some half-dozen plus events for your requested operation. Check the 1st error between the event starting the request for your package and reporting it failed.
If you're manifesting a dependency on a newer package than present on the machine and the Store installs your package the Store knows about dependencies hosted on the Store and will update them as necessary along with your package to satisfy your dependency. If you install via AppInstaller that doesn't automagically resolve your dependencies against the Store's content. AppInstaller calls PackageManager APIs (e.g. AddPackageByUriAsync) so control goes from the AppInstaller app to Windows' MSIX Deployment pipline. OTOH installing via the Store app control goes from Store to MSIX Deployment pipeline. As AppInstaller doesn't involve Store in the install flow AppInstaller doesn't get the benefit of Store hosted packages automagically installed if/when needed to satisfy dependencies. In this case you need to install your dependencies before installing your package via AppInstaller.


Add/improve CI tests and checks

These are suggestions from a client and @kiniry during the QSAFE project.
For all Literate Cryptol specs, ensure that each:

[ ] type checks
[ ] compiles properly with its relevant document processor (e.g., some version of LaTeX, Pandoc, etc.)
[ ] all properties check, and
[ ] if there is a interactive Cryptol script, it completes properly.

We do not, in general, guarantee all properties prove, thus the use of the scripts.  (Though, for these specs, nearly all properties do, indeed, prove.)

Blocked by #49 . @eddywestbrook @bboston7 this would be a good PDI task, I think.

It sure would be

This should be addressed by GaloisInc/cryptol#1334


Improve handling of GC managed array buffers

Specifically, some of the noGC guards had issues that could lead to panics during exception throwing, and some of the loops over multiple buffers where overly complex.

I'll change this to target main before landing, and will wait for the fetch-rework branch to land before doing so.


please add the ensure_git_status_clean action to the beginning of your lane, or if you're absolutely sure of what you're doing then call this action with the :force option

New Issue Checklist

[X] Updated fastlane to the latest version
[X] I have read the Contribution Guidelines

Hi,
I've two lanes.
The first run unit tests and automation (called 'test_lane') and just before it ends I push to git the changes.
The second lane (called 'hockeyapp_lane') uplaods to hockeyapp the .ipa files.
when I try to switch to the second lane I get the exception to add 'ensure_git_status_clean', so i've added it and still got the same error (I've added the force option as well).
When I try to run one lane that contains the two - everything works...
What am i missing ?
Complete output when running fastlane, including the stack trace and command used
This is a destructive and potentially dangerous action. To protect from data loss, please add the `ensure_git_status_clean` action to the beginning of your lane, or if you're absolutely sure of what you're doing then call this action with the :force option.```

### Configuration Files

Please copy the complete content of your `Fastfile` and any other configuration files you use below:

Fastfile:
```ruby
# INSERT CONTENT HERE

Environment
fastlane version (run fastlane -v):1.9.0
Do you use bundler to execute fastlane (i.e. bundle exec fastlane)?no
Do you use a Ruby environment manager (e.g. chruby, rbenv, rvm)?no

Can you please share your Fastfile and the full log? Thanks!

It looks like there hasn’t been any recent activity on this issue, so I’m closing it. If this is still a problem, feel free to re-open and I’d be happy to help :+1:


Autostart not working

When I open the auto start and attempt to log in it always says the account is invalid each time

Yes, it's a known issue, autostart will not work until we understand the token Nexon launcher passes to the client and how to generate it. For now you must start Kanan without autostart, and then launch mabi through either Steam or the Nexon launcher.

ik this isnt the place but, how to fix this?

autostart has been fixed.


some small fixes + command-line tool HzdTextureCli

removed magic check as game textures have different magic values.
fixed null pointer exception in arraySize check, when DDS file has no DX10 extension header.
fix DXT compression selection for BC3 format.
improve UITexture handling.
moved UI independent code into separate library to share with command-line tool.
added command-line tool to be used in scripts and can build for Linux OS.

Attached the new version + the Linux version:
HzdTextureExplorer-v0.60p4.zip
HzdTextureCli-v0.60p4.linux64.zip

Reminder: would you please merge this pull request


click and rich prompts interweave into system command prompt

System: Windows 10 (have not tried on Linux yet).
Terminal: Windows Terminal set to Command Prompt.
When using @click.option with a prompt, the command prompt appears to interweave with the running program.
Also occurs when using click.prompt() or rich's Prompt.ask() inside of the function definition.
Will cause the command prompt (such as "(venv) C:\py>") to be displayed throughout click prompts while program is running.
Sometimes will treat response to prompt as if trying to run a system command, sometimes will accept prompt input from the command prompt when the click prompt is not the active display.
As an example, the attached screenshot shows one run of a program through trogon, asking for 2 prompt inputs (a staff name, and an email address).  The greenish-yellow squares show the command prompt appearing inside of the running trogon program.
No such problems occur when directly running the click program without trogon.  No issues occur if provide the option values in trogon and/or bypass prompts inside program.  Have tried on several different functions.

EDIT:
Minimal viable product:
mvp.py:
import click

from trogon import tui

@tui()
@click.group()
def cli():
    pass

@click.command()
@click.option('--name', prompt="Name")
def minimal(name):
    """Prompt for a name."""
    pass

cli.add_command(minimal)

if __name__ == '__main__':
    cli()

setup.py:
from setuptools import setup, find_packages

setup(
    name='mvp',
    version='0.1',
    py_modules=['mvp'],
    install_requires=[
        'Click'
    ],
    packages=find_packages(),
    entry_points='''
        [console_scripts]
        mvp=mvp:cli
    ''',
)

Example application run:

Thanks for the report
I haven't been able to reproduce this on MacOS - so it may be a Windows issue.
Does it only happen on command prompt?

Thanks for the report
I haven't been able to reproduce this on MacOS - so it may be a Windows issue.

That was all from my work computer.
Just tried on windows 10 on home computer and experienced all the same issues.
Tried from a Linux install and all the problems disappeared.
Can confirm it appears to be a Windows issue.

I'm getting same issue on Windows 10 with command line prompt run both as CMD and in Windows terminal.
I've checked in Powershell too and this behavior completely freezez the PS.

I can confirm that Windows 10 running Powershell in Windows Terminal has this issue for me too.


Numeric input with 2 fixed decimal

Hi,
May I know how to make the input filter so that it will only accept digit input but the display always has fixed 2 decimal place?
Eg.
The text input will have default value of => 0.00
When user enter 1, it become => 0.01
User enter 2, it become => 0.12
User enter 3, it become => 1.23
Is this possible?
Many thanks in advance.

Here is my solution, hope it helps.
class FilteringMoneyFormatter extends TextInputFormatter {
  final digitOnly = RegExp(r'\d+');
  final _decimalFormatter = FilteringTextInputFormatter.allow(RegExp(r'\d+'));
  @override
  TextEditingValue formatEditUpdate(
    TextEditingValue oldValue, // unused.
    TextEditingValue newValue,
  ) {
    var value = _decimalFormatter.formatEditUpdate(oldValue, newValue);
    var digitOnly = value.text.replaceAll('.', '');
    var money = ((int.tryParse(digitOnly) ?? 0) * 0.01).toStringAsFixed(2);
    return TextEditingValue(
        text: money, selection: TextSelection.collapsed(offset: money.length));
  }
}


feat: make the chart path configurable

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


Run npm in pip install, remove gulp

@minrk I think this is what you were asking for earlier..  Mostly stolen from NB.

The missing piece here is now implying css, jsdeps before sdist, develop, and build_py.

@jdfreder I've rebased this and added a commit in requirements.txt, and the tests are passing. Can you reset this to the state of https://github.com/minrk/ipywidgets/tree/say-no-to-gulp
?

I see you already opened a PR :)

I just wanted to be sure the tests were passing, since I didn't trust it on my own branch. Either way, gulp is gone, yay!

Either way, gulp is gone, yay!

Hazzah!


How to skip decomposition of dequantize_affine and quantize_affine custom ops in inductor?

I want to use the torch.ops.quant.quantize_affine (Q) and torch.ops.quant.dequantize_affine (DQ) to represent a quant model DAG in QDQ style, and do quant fusion using inductor's pattern matcher, for instance:
x(i8)       w(i8)      b(i32)      x(i8)       w(i8)      b(i32)
 |            |          |           |           |          |
DQ           DQ         DQ           |           |          |
  \           |         /             \          |         /
torch.ops.aten.linear.default   ->    my_q_linear_triton_impl
              |                                  |
              Q                                  |
              |                                  |
            y(i8)                               y(i8)

However, since torch.ops.quant.quantize_affine and torch.ops.quant.dequantize_affine are registered to inductor's decomposition table, as well as with CompositeImplicitAutograd flag, they are decomposed in aot_autograd.
I wonder how to preserve the original Q-DQ  ops after aot_autograd? I noticed that the torch's built-in custom Q-DQ ops, such as torch.ops.quantized_decomposed.quantize_per_tensor and torch.ops.quantized_decomposed.dequantize_per_tensor, can be preserved after aot_autograd, and there are pattern rewrites based on these Q-DQ ops. (BTW, what's the relationship between torchao and torch.ao module, will torchao be merged into torch.ao in the future?)

I see, the reason why we want these ops to be decomposed in inductor is because we want the q/dq ops to be optimized by the compiler (triton).
I think pytorch/pytorch#139102 might be something you are looking for here, but API for registering dequant op is not developed yet

I'm not familiar with torch's inner data structures or implementation details, to my understanding, this InvokeQuant is a generic quantize op that user can register their own implementation as a subgraph function? You mentioned "dequant registering is not developed yet", does this mean that we also need a InvokeDequant op?


raw: Make from_std public

This method is just a wrapper around already public APIs, and it's convenient to use directly; in my case I have a socket already passed via systemd socket activation.

Makes sense but I wonder if we should document under what circumstances it can fail. It's a bit odd that these return results when under normal circumstances they can't fail. As far as I remember they are returning results because of some tokio internal abstraction.

As far as I remember they are returning results because of some tokio internal abstraction.

Yeah, it's registering the fd with e.g. epoll() I believe.
Added some docs.


export: no results produced

steps to reproduce:

construct table manually as illustrated (or any other table i've created today using EasyTable import feature)
"Export EasyTable"
click in output text box (which appears blank; see issue #9)
Ctrl-A
Ctrl-C
paste elsewhere (like new buffer in notepad++)

expected behaviour: see exported contents of table
actual behaviour: see nothing pasted

Thanks for the feedback.
Looks like 0.8 has changed some data that EasyTable expected to read.
Fixed, will release in next update.


Fix a bug in ConvertCSVToImages.cxx that is generating build errors on the dashboard

Looks good to me!

thanks. Will merge as soon as i see green.

@aylward there is a CMake configure error that it cant find HDF5.

Hi.  Please do a blame and see where that line comes from.  I disabled it
for win32 in a commit last night, but I don't know where it came from
originally.  Is it necessary for other platforms?    The comment suggests
that it is guessing that it will be needed I  the future, and that suggests
that it's required use isn't appropriately conditioned

Sent via phone.
On May 11, 2016 5:37 PM, "Deepak Roy Chittajallu"<EMAIL_ADDRESS>wrote:

@aylward https://github.com/aylward there is a CMake configure error
https://open.cdash.org/viewConfigure.php?buildid=4361436 that it cant
find HDF5.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/KitwareMedical/TubeTK/pull/726#issuecomment-218598058

@fbudin69500 you seemed to have added this as a requirement. Could you take a look at this  error on the dashboard?

@aylward @thewtex @fbudin69500 Looks like this error would go away if we turn TubeTK_USE_PYQTGRAPH off in CMake/CircleCI/CircleCI_slicer_Docker.cmake.
Is TubeTK_USE_PYQTGRAPH required for something?

PYQTGRAPH seem to be used in a few places:
here, here, and here

@cdeepakroy There is some of the functionality for visualizing the NumPy representations conversion of the TubeSpatialObject. Since Slicer does not have the pyqtgraph Python package, +1 for disabling it in CircleCI_slicer_Docker.cmake

Hi,
I think we should remove the find package call for HDF5.  Most platforms
can install pyside without it.   If pyside needs HDF5 on some platform,
then pyside should request it - TubeTK is making a questionable assumption
here.
s
On Thu, May 12, 2016 at 2:28 PM, Deepak Roy Chittajallu <
<EMAIL_ADDRESS>wrote:

@aylward https://github.com/aylward @thewtex
https://github.com/thewtex @fbudin69500 https://github.com/fbudin69500
Looks like this error would go away if we turn TubeTK_USE_PYQTGRAPH off in
CMake/CircleCI/CircleCI_slicer_Docker.cmake
https://github.com/KitwareMedical/TubeTK/blob/master/CMake/CircleCI/CircleCI_slicer_Docker.cmake#L79.
Is TubeTK_USE_PYQTGRAPH required for something?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/KitwareMedical/TubeTK/pull/726#issuecomment-218844525

--
Stephen R. Aylward, Ph.D.
Senior Director of Operations, North Carolina, Kitware, Inc.
http://www.kitware.com and http://www.aylward.org
(919) 969-6990 x300

Looks good to me.  Please merge or abandon as soon as possible.

@cdeepakroy  After checking what tests require pyqtgraph, it seems that the only one that needs it is this test and it is disabled if TubeTK_USE_PYQTGRAPH is disabled. So you can go ahead and turn that option to OFF.

Turned TubeTK_USE_PYQTGRAPH off and removed find_package for HDF5. Will merge as soon as CircleCI shows the green flag.

Let's keep the USE_PYQTGRAPH on.
Is that causing trouble with any tests?   All tests use to pass with it on,
right?
On Thu, May 12, 2016 at 3:08 PM, Deepak Roy Chittajallu <
<EMAIL_ADDRESS>wrote:

Turned TubeTK_USE_PYQTGRAPH off and removed find_package for HDF5. Will
merge as soon as CircleCI shows the green flag.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/KitwareMedical/TubeTK/pull/726#issuecomment-218855650

--
Stephen R. Aylward, Ph.D.
Senior Director of Operations, North Carolina, Kitware, Inc.
http://www.kitware.com and http://www.aylward.org
(919) 969-6990 x300

Yes i think it might build fine with it turned on.
Lets see the CircleCI build result for the those two commits to be sure. It will evaluate both cases.
-- remove find_package HDF5 and keep USE_PYQTGRAPH on
-- remove find_package HDF5 and turn USE_PYQTGRAPH off

Perfect - thanks!
On Thu, May 12, 2016 at 3:30 PM, Deepak Roy Chittajallu <
<EMAIL_ADDRESS>wrote:

Yes i think it might build fine with it turned on.
Lets see the CircleCI build result for the those two commits to be sure.
It will evaluate both cases.
-- remove find_package HDF5 and keep USE_PYQTGRAPH on
-- remove find_package HDF5 and turn USE_PYQTGRAPH off
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/KitwareMedical/TubeTK/pull/726#issuecomment-218861224

--
Stephen R. Aylward, Ph.D.
Senior Director of Operations, North Carolina, Kitware, Inc.
http://www.kitware.com and http://www.aylward.org
(919) 969-6990 x300


Error on macOS: Cannot write headers after they are sent to the client

Repost of this bug report:
FYI when trying this on macOS I had some issues with the `hyper-gateway`
program, it basically fails and quits after most hyperdrive.el commands.
For example, when I call `hyperdrive-hyper-gateway-version` it prints:

--8<---------------cut here---------------start------------->8---
Request: {
  method: 'GET',
  url: '/',
  headers: {
    host: 'localhost:4973',
    'user-agent': 'curl/8.1.2',
    accept: '*/*',
    'accept-encoding': 'deflate, gzip'
  }
}
Requested: {
  method: 'GET',
  url: 'hyper://',
  status: 500,
  responseHeaders: { 'content-type': 'text/plain; charset=utf-8' }
}
Error [ERR_HTTP_HEADERS_SENT]: Cannot write headers after they are sent to the client
    at new NodeError (node:internal/errors:406:5)
    at ServerResponse.writeHead (node:_http_server:345:11)
    at Server.<anonymous> (file:///usr/local/lib/node_modules/hyper-gateway/src/index.js:131:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  code: 'ERR_HTTP_HEADERS_SENT'
}
node:internal/errors:497
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_HTTP_HEADERS_SENT]: Cannot write headers after they are sent to the client
    at new NodeError (node:internal/errors:406:5)
    at ServerResponse.writeHead (node:_http_server:345:11)
    at Server.<anonymous> (file:///usr/local/lib/node_modules/hyper-gateway/src/index.js:139:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  code: 'ERR_HTTP_HEADERS_SENT'
}
--8<---------------cut here---------------end--------------->8---

What version of the gateway of this was it?

What version of the gateway of this was it?

Likely 3.4.2

This particular error fixed in 3.6.0.


integrate KSQL/Schema Registry and Control Center

the setup currently doesn't integrate KSQL/Schema Registry with Control Center, that should happen for 5.0+

BTW, I have a solution for this - https://github.com/cricket007/cp-ansible/commit/ceed2a36ae62abdc73261a0a778b21c23dbdf9ad


OGC and others: Document method to indicate unnumbered option

In OGC sometimes authors wants to specify a source code block that is not numbered as a figure. The [%unnumbered] options used but there is no example on how to specify that.
We need to add an example like this:
[source%unnumbered,json]
----
{
  my_json: "hello"
}
----

Some source code:

[source,json]
----
{
  my_json: "hello"
}
----

Renders as:

This is requested by @ghobona as well.

Closing in #501


remove maxRetries for queueConfig

Prometheus removed the property max_retries since a while from the remote_write properties (see https://github.com/prometheus/prometheus/pull/5649/files). This PR reflects this removal in Prometheus.

Thanks for the contribution!  Looks like this needs a make generate to update all the generated files and pass CI.
In our compatibility matrix chart we do still support older versions of Prometheus (https://github.com/coreos/prometheus-operator/blob/master/Documentation/compatibility.md#prometheus), so I'm not sure we want to remove it at this point, @brancz WDYT?

Okay, maybe otherwise a deprecation notice would be helpful. Ran into that problem today and prometheus crashed with active "maxRetries" setting.

We can't remove it from the CR, that's a breaking change to the v1 API. We can exclude the config option though in the generated config if the version of the prometheus server is >=2.11.

Okay, make sense. Fully understand to support that for near future.


Cross compile on Alpine targeting x86_64-unknown-linux-gnu : undefined reference to `__res_init'

I am setting up GitLab CI and I have to run cross within Alpine dind(Docker-in-Docker) image.
It successfully builds my project targeting x86_64-pc-windows-gnu with Mingw64
But when I build my project targeting x86_64-unknown-linux-gnu, it failed with error.

undefined reference to `__res_init'

cross +stable build --target x86_64-unknown-linux-gnu --release --verbose
Please find full build.log
The following is my Dockerfile of Alpine running cross
FROM docker:stable-dind

RUN apk add --no-cache ca-certificates gcc mingw-w64-gcc libc-dev

# Copied from https://github.com/rust-lang/docker-rust/blob/6314c6bc3d54d5b9284458c6a9061ef5766c9607/1.44.1/alpine3.12/Dockerfile
ENV RUSTUP_HOME=/usr/local/rustup \
    CARGO_HOME=/usr/local/cargo \
    PATH=/usr/local/cargo/bin:$PATH \
    RUST_VERSION=1.44.1

RUN set -eux; \
    url="https://static.rust-lang.org/rustup/archive/1.21.1/x86_64-unknown-linux-musl/rustup-init"; \
    wget "$url"; \
    echo "0c86d467982bdf5c4b8d844bf8c3f7fc602cc4ac30b29262b8941d6d8b363d7e *rustup-init" | sha256sum -c -; \
    chmod +x rustup-init; \
    ./rustup-init -y --no-modify-path --profile minimal --default-toolchain $RUST_VERSION; \
    rm rustup-init; \
    chmod -R a+w $RUSTUP_HOME $CARGO_HOME; \
    rustup --version; \
    cargo --version; \
    rustc --version;

RUN cargo install cross
RUN rustup target add x86_64-pc-windows-gnu
RUN rustup target add x86_64-unknown-linux-gnu

ENV CROSS_DOCKER_IN_DOCKER=true

RUN mkdir ~/.cargo/ && \
    printf "\n[target.x86_64-pc-windows-gnu]" >> ~/.cargo/config  && \
    printf "\nlinker = \"/usr/bin/x86_64-w64-mingw32-gcc\"" >> ~/.cargo/config

Note that due to the time having passed, this can't compile using cross v0.2.1 (the release version) and the initial Rust version, due to many dependencies requiring newer Rust versions. I'll be trying it using a more modern Rust version and the latest cross commit and see if this is still an issue.

Run Command
docker run -it --rm  -v /var/run/docker.sock:/var/run/docker.sock -v "$PWD":/project -w /project alpine:local sh

Cross.toml
[target.x86_64-pc-windows-gnu]
image = "ghcr.io/cross-rs/x86_64-pc-windows-gnu:main"

Project: Cargo.toml
[package]
name = "res_init"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
libc = "0.2.126"

Project: main.rs
fn main() {
    unsafe {
        libc::res_init();
    }
}

Command
# cross build --target x86_64-pc-windows-gnu

   Compiling res_init v0.1.0 (/project)
error[E0425]: cannot find function `res_init` in crate `libc`
 --> src/main.rs:3:15
  |
3 |         libc::res_init();
  |               ^^^^^^^^ not found in `libc`

For more information about this error, try `rustc --explain E0425`.
error: could not compile `res_init` due to previous error
# cargo build
   Compiling libc v0.2.126
   Compiling res_init v0.1.0 (/project)
    Finished dev [unoptimized + debuginfo] target(s) in 2.92s

In short, I think this is not-an-error, just a confusing error message, since res_init isn't a Windows function, and is only available on UNIX-like OSes. I believe whatever libraries you depend on hopefully have fixed this.
libc itself only tests for res_init on unix and fuschia.


VSCode does not update on Multiuser Server, even when installed in User-Mode

Does this issue occur when all extensions are disabled?: Yes/No
Yes

VS Code Version: 1.90.2
OS Version: Server 2022

Steps to Reproduce:

Install an older version of VSCode with User Setup for two users on a server 2022 (Non-Admin-Mode UAC)
Start VSCode with both users and keep them both running an logged in (Non Admin Mode UAC)
Check for new version with one of the users and press update
The Install will fail, as it seems to detect the process of the other user, which it cannot even kill (and should even not try)
Expected behaviour would be to only look for processes of the current user and NOT all users on the server!

Thank you Mr-Bot (hihi), but that is exactly what I wanted to do ;-)

Any news on this?

Hello? Somebody listening????
This is driving us nuts.
All the updates dont work and lets face it, this problem cannot be that difficult.
If I run the installer, it will install, so it is the autoupdate process which does the wrong check.

@joaomoreno Any news on this?
This is really a problem in corporate environments, where you use admin-servers or even on multiuser terminalservers.
Cannot be difficult in my eyes?

@joaomoreno Would be great to get a feedback on this

Duplicate of https://github.com/microsoft/vscode/issues/76641


Enhance OM: Non-exploding item APIs

Some item operations explode the list. We have to provide alternative behavior that does not explode list.
The following comment is copy pasted from the evaluator:
///A ProjectItemElement could have resulted in several items if it contains wildcards or item or property expressions.
///Before any edit to a ProjectItem (remove, rename, set metadata, or remove metadata) this must be called to make
///sure that the edit does not affect any other ProjectItems originating in the same ProjectItemElement.
Project.SplitItemElementIfNecessary(ProjectItemElement itemElement)

Project.RemoveItem(ProjectItem) (1-2 weeks)

Scenarios

User deletes file via file system or via VS

CPS reacts by calling RemoveItem on all ProjectItems with that value

The current Project.RemoveItem(ProjectItem)

Removes item inside the Include attribute
Explodes the item tag if it evaluated to multiple items

Modifications to not explode

If Include contains only (globs and literal strings)

If item is covered by glob, then don't do anything
If item is covered by literal list, remove all the literals that have the same value as the item to remove

Extra credit for removing literals that have properties inside of them (Include="foo$(prop)")

Else (items and properties present: Include="*.a;@(AnItem);$(AProperty);3.a;2.a;3.a")

Option1

Apply previous behavior (explode)
Justification

If you don't explode and add an exclude, csproj becomes bloated over time as the user deletes elements
CPS should call RemoveItem on the correct item tags (e.g. call RemoveItem on tags that reference the item directly via globs / literals, and not on the tags that indirectly reference that item)

Option2

Split the item tag in 2 tags:

A tag with just the globs and the literals, and then recursivelly call RemoveItem on this tag
A tag with the indirect references and add the item to remove in the exclude

Option3

Split the item tag in more tags:

A tag with just the globs and the literals, and then recursivelly call RemoveItem on this tag
Explode the items coming from indirect references

Add Exclude for Item (1 week) - CPS currently has own implementation. We could add our own so they don't have to implement it

Scenarios

User wants to exclude a file from the build
CPS wants to add an exclude to implement project model semantics

Inputs

Overload1

Item - the ProjectItem whose underlying ProjectItemElement needs to have the exclude applied to

Overload2

Element - the ProjectItemElement to append the exclude on
directoryToIgnore - full path to a directory to ignore. The exclude attribute gets appended with "directoryToIgnore/**/."

Returns

Same implementation as RemoveItem:  true if item was present in the project

Remarks

Edits the underlying xml for the specified item to exclude the item's evaluated value / directory
Does not do anything if the exclude tag already has an exclude containing it

//exclude a single file, and you know it's corresponding ProjectItem
bool ExcludeItem(ProjectItem item)

//exclude a whole directory, and you know the corresponding ProjectItemElement where to add the exclude
bool ExcludeItem(ProjectItemElement element, string directoryToIgnore)

ProjectItem.(Set|Remove)metadata (TODO)
In the presence of globs, the new Item Update attribute should be used instead of these.
ProjectItem.Rename (~1-2 weeks)

Modifications to not explode

If Include contains only (globs and literal strings)

Same as remove, but rename literals instead of delete.
[?] what to do if old name is in glob but new name is not in glob?

Option1: Add the new name that is not in glob as a literal
Option2: don't do anything

[?] what to do if literal incorporates a property

Don't do anything
Raise exception
Explode the item out and rename, potentially breaking the build logic

Else (items and properties present: Include="*.a;@(AnItem);$(AProperty);3.a;2.a;3.a")

Option1 from RemoveItem

Apply previous behavior (explode)
Justification

If you don't explode and add an exclude, csproj becomes bloated over time as the user deletes elements
CPS should call RemoveItem on the correct item tags (e.g. call RemoveItem on tags that reference the item directly via globs / literals, and not on the tags that indirectly reference that item)

Option3 from RemoveItem

Split the item tag in more tags:

A tag with just the globs and the literals, and then recursively call Rename on this tag
Explode the items coming from indirect references and rename

Project.AddItem (TODO)

Only works (does not explode) when:

the existing item or item group has no condition
the existing item tag has no excludes
neither the item nor the considered item tag have metadata

Flag to propagate item operations instead of exploding (2 weeks)

[?] do we want this?
Recursively propagate the above operations on indirect references

act on glob and literal lists
Recursivelycall the same operation on referenced items
explode referenced properties

Do string magic inside the properties. Recursively, since properties can reference each other.

I'd say that the minimum work we need to do here is provide an option to just fail if the requested operation would explode the glob--then a caller could reconsider and do something else. That wouldn't require any additional MSBuild language concepts, but would allow smarter behavior. Of course, it also makes the caller more complex.

First implementation: throw instead of explore
Next: do something smarter


Finalised message (optional)

I have some thoughts on a finalized string. Right now the Spinner completes and erases all that has been written.
In my code I am dealing with this myself, but it is something that might fit within this package.
Old behaviour:
⣯ Encrypting data...

s.Stop()

New optional Behaviour:
s.FinalMessage = "✔Data encryption successful"
⣯ Encrypting data...

s.Stop()
✔ Data encryption successful

This is useful as a checklist at the end of a long CLI command:
✔ Data encryption successful
✔ File uploaded to s3
⣯ Publishing...

Do you believe this is useful for Spinner or is something I should continue to roll my own?

This is an interesting idea.  I'll have to play with it some.

Did a quick mock up.  This is doable and easy. I'll push up a patch shortly.

Feature complete via #27
Example in _examples directory and below with as well as README
s := spinner.New(spinner.CharSets[9], 100*time.Millisecond)
s.Finally = "Complete!\nNew line!\nAnother one!\n"
s.Start()
time.Sleep(4 * time.Second)
s.Stop()

Output
Complete!
New line!
Another one!

Please let me know if this will work for you.

Code and readme is fine. I'll have to update the example app.


adding partial index for uncommitted entries

Codecov Report

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

@@           Coverage Diff           @@
##           master     #816   +/-   ##
=======================================
  Coverage   42.51%   42.51%
=======================================
  Files         134      134
  Lines       10572    10572
=======================================
  Hits         4495     4495
  Misses       5491     5491
  Partials      586      586

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 6b46312...1d8fde3. Read the comment docs.


When browser close, it will reset the cache.

We have implemented ngx-cacheable for our website https://www.matrimonyone.com. Its works perfectly fine until we close browser tab or browser.
When we open website again, it will reset the cache.
To reproduce this issue please follow below steps.

Open  https://www.matrimonyone.com, in footer, click on contact us then feedback.
Click again on Contact us, data will load from caching.
Close the browser and again open the website.
Click on contact us in footer, you will see data is loading from database. (Expected Behaviour: It should load from Cache)

We find out the solutions and closing this issue. Below is the solutions in app.component.ts
import { GlobalCacheConfig, DOMStorageStrategy } from 'ngx-cacheable';
constructor() {
GlobalCacheConfig.storageStrategy = DOMStorageStrategy;
}


Hardcode ips of ESPhome devices

Unifi decided to rotate some ips which resulted in some pretty weird automation failures 😅. Let's hardcode the ips for all the unifi, esphome's and servers to prevent this from happening again.

Still need to update all the integrations within home assistant!

Removed all the ESPHome integrations and re-added them using the UI.

Restarting home assistant.

I think everything is working as expected 🎉


Merge into master

This change is

Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Update greetings.js

Modified replies

thanks for these @HackermanArjun - unfortunately I don't think you're improving the language with the changes - in some cases you're actually making things more confusing.
For the time being I'm going to close this.  If you really want to make some of these changes, open them as specific issues and we can discuss the changes.
Note that none of the open issues indicates that these changes need to be made:
https://github.com/AgileVentures/project_greeter_bot/issues
Ideally if you want to submit a PR you should open an issue first or submit something related to an existing issue


Cannot save models in some cases

Might be a cause of the names of the layers not being set properly so it has repeated names and cannot save the model.

At least by now, this is fixed by putting the tf.Variable creation into the build method. By doing so, the name of the variables get prepended with the layer name


Support documentation files outside of working-dir path

Description of your changes

This PR includes changes to logic of staging files using git add. Currently logic does not support adding files changes if those files are outside of working-dir. Logic behind is changed to list all files that have been modified and then stage only files that are defied within output-file parameter of this GitHub action.
It also includes update to README.md and fixes to docker-entrypoint.sh script to pass Shellcheck scans.

https://github.com/terraform-docs/gh-actions/issues/93
I have:

[x] Read and followed terraform-docs' contribution process.

How has this code been tested

Tests have been performed locally to make sure all strings are parsed correctly. Also all GitHub Actions workflows are passing.

@khos2ow sorry for direct tag can you please release this feature its blocking us from using this action


Button focus styles are overlapping with the border

Having an inconsistent width border has issues like below, this needs to be fixed in both Button and Button Icon components.

Possible solutions here: https://stackoverflow.com/questions/38165317/dont-affect-positioning-of-other-elements-on-border-width-changes

@SimeonC In buttonGroup @SashaShostyr has used box-shadow inset for the button borders, and a standard border for the focus which seems to work nicely.
Just tried this on iconButton and it keeps a consistent total height of buttons regardless if they have borders or not, and seems to solve this issue.
Any issue with using this approach? If not I'll update the button to follow suit

.

I think there may have been some issues with the button group, but apart from that yea I think this should be OK.
It'd be great if we could just globally apply this to :focus-visible rather than have to add it to each component 🤔


Failure in Redirecting to Verifier URL Manually

Issue
Once you establish a set of VCs in the "Holder", you have the option to manually input a "verifier" URL to send them to by clicking the scanner button in the bottom-left.

I was attempting to navigate to this libs example verifier, when I came across the following issue.
Attempting to input a manual URL (and likely via QR code too), doesn't do anything except throw a console error.

You see this error when you paste in a did siop request URL like
openid://?response_type=id_token&scope=did_authn&request_uri=https%3A%2F%2Fc19.cards%2Fapi%2Fsiop%2FyMowgX09nH5bUM4r2Qc7aA&client_id=https%3A%2F%2Fc19.cards%2Fapi%2Fsiop

? (I got this example by "copy link" on the QR from your venue link above.) This is working for me.

You see this error when you paste in a did siop request URL like
openid://?response_type=id_token&scope=did_authn&request_uri=https%3A%2F%2Fc19.cards%2Fapi%2Fsiop%2FyMowgX09nH5bUM4r2Qc7aA&client_id=https%3A%2F%2Fc19.cards%2Fapi%2Fsiop

? (I got this example by "copy link" on the QR from your venue link above.) This is working for me.

Yep I did not understand DID SIOP when I made this. What you have above works for me too!
I think I was trying to ask the wallet to be a relying party when I made this... I understand what's happening here now.

Yep I did not understand DID SIOP when I made this. What you have above works for me too!
I think I was trying to ask the wallet to be a relying party when I made this... I understand what's happening here now.


Create (adjust) endpoint for accepting data flow metadata

Background
The REST API endpoints of the Server
Feature
Create (or adjust if already exists) API endpoint accepting metadata (= additional information) for the given partitioning.
Depends on #37

This item depends on:

AbsaOSS/atum-service#37
By Dependent Issues.

This item depends on:

AbsaOSS/atum-service#37
By Dependent Issues.

This item depends on:

AbsaOSS/atum-service#37
By Dependent Issues.

This item depends on:

AbsaOSS/atum-service#37
By Dependent Issues.


Superluminal profiling

Profile the game using superluminal. Who knows what lurks in the shadows.

Profiling revealed performance issues with logging system, was fixed.


Remove validation

Fixes https://github.com/embeddings-benchmark/mteb/issues/83

LGTM! Interesting, I thought I had double checked all eval_splits in all tasks. Do you happen to know why we missed this one @Muennighoff? If so, could there be other issues like this one?

Not sure, I also thought we had removed all of them 🧐