Dealing with alleles and multiple gene calls

I want to report some kind of bug/issue that I recognized during my work in Scirpy and has to do with scirpy.tl.define_clonotype cluster (but likely also with scirpy.tl.define_clonotype, although this was not tested). Scirpy considers any string inside v_call (same problem with j_call) as a unique V-gene assignment, and this is perfectly fine for working with Cell Ranger annotation. However, if we are working with re-annotated data, which is done with IgBlastn or IMGT/Highv-quest this is not true any more. First, the annotation contains alleles, which are depicted like this IGHV3-33*001. The problem is that if another cell would have IGHV3-33*002 these two cells would always be separated by Scirpy (if v_gene = True), because Scirpy thinks that those are totally different genes, although they only differ in their alleles, even if everything else even the junction sequence can be quite similar.
Another issue with using  IgBlastn or IMGT/Highv-quest re-annotation is that they often leave multiple possible gene assignments into the v_call column if they are all similar likely.
What I did so far with my datasets was that I manually manipulated the v_call and j_call column prior to loading the datatset into an AnnData object so that they only contain the first v/j_call and without the allele information.
My idea here would be to adapt the clonotype cluster function so that it automatically ignores multiple v_call's/j_call's i.e. only considers the first one and also ignores the allele information for clustering, but doesn't manipulate the call itself. Immcantation has a own parameter on how to work with multiple calls for a gene (see "parameter first= FALSE": https://scoper.readthedocs.io/en/stable/topics/hierarchicalClones/).
Actually I encountered this problem already some time ago and discussed it with @felixpetschko but eventually we both forgot about it until now. Either way, I think it's good if @grst can also have a look on this problem and help with a solution, because if I remeber correctly it's not that trivial to "fix" this. Maybe there is some elegant workaround available?
Originally posted by @MKanetscheider in https://github.com/scverse/scirpy/issues/542#issuecomment-2309825187

@MKanetscheider, I made this a separate issue.

@zktuong, would be curious if and how you have dealt with this before?
From a technical perspective I could see two solutions:

Simple workaround: Provide a "preprocessing" function that maniuplates the v/j_call string to strip away the allele information and only keep the first gene (what @MKanetscheider has been doing manually so far)
The define_clonotype_cluster function already uses a distance matrix to deterime distance between genes. For now, that's a boring identity matrix. This matrix could be replaced with something like

IGHV3-33*002
IGHV3-33*001
IGHV27
...

IGHV3-33*002
1
1
0
0

IGHV3-33*001
1
1
0
0

IGHV27
0
0
1
0

...
0
0
0
1

yea i just only do:
https://github.com/zktuong/dandelion/blob/7a86c211da0a3eab6863921d2dfbbd562feeeaa9/dandelion/tools/_tools.py#L1660C13-L1660C79
but i don't select the first one:
https://github.com/zktuong/dandelion/blob/7a86c211da0a3eab6863921d2dfbbd562feeeaa9/dandelion/tools/_tools.py#L1669C4-L1669C55
can be easily changed to just v.split(",")[0] in my case

Hi @grst, I just took a look at my improved implementation of the v/j-gene matching and I think the matrix solution you suggested should work fine. If this implementation is not that urgent I would actually tackle this problem in a few weeks after completing my thesis if that's fine.

sounds good, thanks!


Chrome >121 does not seem to work

I have tried re-installing everything, clean directory too, run as root or not, I always get this error (linux).
2024-08-28T09:38:17.441Z ERROR [hero-core/connections/ConnectionToHeroClient] ConnectionToClient.HandleRequestError { context: {}, sessionId: undefined, sessionName: undefined } Error: Failed to launch chrome 125.0.6422.141:

Please re-install the browser engine:
-------------------------------------------------
-------------- NPM INSTALL ----------------------
-------------------------------------------------

 npm install @ulixee/chrome-125-0

-------------------------------------------------

    at BrowserEngine.verifyLaunchable (/home/user/hero/program/plugins/default-browser-emulator/lib/BrowserEngine.ts:56:13)
    at Browser.launch (/home/user/hero/program/agent/main/lib/Browser.ts:136:43)
    at Object.cb (/home/user/hero/program/agent/main/lib/Pool.ts:179:21)
    at Queue.next (/home/user/hero/program/node_modules/commons/lib/Queue.ts:188:44)
    at Queue.run (/home/user/hero/program/node_modules/commons/lib/Queue.ts:72:10)
    at Pool.getBrowser (/home/user/hero/program/agent/main/lib/Pool.ts:143:44)
    at Agent.open (/home/user/hero/program/agent/main/lib/Agent.ts:116:34)
    at async Session.openBrowser (/home/user/hero/program/node_modules/core/lib/Session.ts:276:5)
    at async Function.create (/home/user/hero/program/node_modules/core/lib/Session.ts:957:7)
    at async ConnectionToHeroClient.createSession (/home/user/hero/program/node_modules/core/connections/ConnectionToHeroClient.ts:196:30)
------BROWSER_CREATION_Q--------------------------
    at Queue.run (/home/user/hero/program/node_modules/commons/lib/Queue.ts:63:19)
    at Pool.getBrowser (/home/user/hero/program/agent/main/lib/Pool.ts:143:44)
    at Agent.open (/home/user/hero/program/agent/main/lib/Agent.ts:116:34)
    at async Session.openBrowser (/home/user/hero/program/node_modules/core/lib/Session.ts:276:5)
    at async Function.create (/home/user/hero/program/node_modules/core/lib/Session.ts:957:7)
    at async ConnectionToHeroClient.createSession (/home/user/hero/program/node_modules/core/connections/ConnectionToHeroClient.ts:196:30)
    at async CommandRunner.runFn (/home/user/hero/program/node_modules/core/lib/CommandRunner.ts:36:14)
    at async ConnectionToHeroClient.executeCommand (/home/user/hero/program/node_modules/core/connections/ConnectionToHeroClient.ts:258:12)
    at async ConnectionToHeroClient.handleRequest (/home/user/hero/program/node_modules/core/connections/ConnectionToHeroClient.ts:66:14)
Automatically shutting down Hero Core (Browser Closed)
2024-08-28T09:38:17.449Z ERROR [hero-core/index] UnhandledRejection { context: {}, sessionId: null, sessionName: undefined } Error: The socket was closed while data was being compressed
    at /home/user/hero/program/node_modules/ws/lib/sender.js:510:21
    at /home/user/hero/program/node_modules/ws/lib/permessage-deflate.js:326:9
    at PerMessageDeflate.cleanup (/home/user/hero/program/node_modules/ws/lib/permessage-deflate.js:143:9)
    at WebSocket.emitClose (/home/user/hero/program/node_modules/ws/lib/websocket.js:267:57)
    at Receiver.receiverOnFinish (/home/user/hero/program/node_modules/ws/lib/websocket.js:1209:20)
    at Receiver.emit (node:events:520:28)
    at finish (node:internal/streams/writable:953:10)
    at node:internal/streams/writable:934:13
    at process.processTicksAndRejections (node:internal/process/task_queues:90:21)
DisconnectedFromCore: This transport has been disconnected (host: This Hero has been disconnected from Core (coreHost: ws://localhost:1818/hero))
    at ConnectionToHeroCore.beforeDisconnect (/home/user/hero/program/node_modules/client/connections/ConnectionToHeroCore.ts:115:11)
    at ConnectionToHeroCore.onConnectionTerminated (/home/user/hero/program/node_modules/net/lib/ConnectionToCore.ts:243:42)
    at WsTransportToCore.finalHandler (/home/user/hero/program/node_modules/commons/lib/EventSubscriber.ts:46:14)
    at Object.onceWrapper (node:events:635:26)
    at WsTransportToCore.emit (node:events:520:28)
    at WsTransportToCore.emit (/home/user/hero/program/node_modules/commons/lib/TypedEventEmitter.ts:158:18)
    at WsTransportToCore.disconnect (/home/user/hero/program/node_modules/net/lib/WsTransportToCore.ts:60:10)
    at WebSocket.finalHandler (/home/user/hero/program/node_modules/commons/lib/EventSubscriber.ts:46:14)
    at Object.onceWrapper (node:events:635:26)
    at WebSocket.emit (node:events:520:28)
------AGENT QUEUE---------------------------------
    at Queue.run (/home/user/hero/program/node_modules/commons/lib/Queue.ts:63:19)
    at CoreSessions.create (/home/user/hero/program/node_modules/client/lib/CoreSessions.ts:34:8)
    at ConnectionToHeroCore.createSession (/home/user/hero/program/node_modules/client/connections/ConnectionToHeroCore.ts:67:38)
    at Hero._Hero_getCoreSessionOrReject (/home/user/hero/program/node_modules/client/lib/Hero.ts:686:10)
    at Hero.get activeTab [as activeTab] (/home/user/hero/program/node_modules/client/lib/Hero.ts:176:33)
    at Hero.goto (/home/user/hero/program/node_modules/client/lib/Hero.ts:477:17)
    at SpawnBrowser (/home/user/hero/program/browse.js:1807:32)
    at async ProcessMessage (/home/user/hero/program/browse.js:3487:26)
    at async MessagePort.<anonymous> (/home/user/hero/program/browse.js:3659:18) {
  host: 'This Hero has been disconnected from Core (coreHost: ws://localhost:1818/hero)',
  code: 'DisconnectedFromCore',
  coreHost: 'ws://localhost:1818/hero'
}
RESULT {
  error: true,
  msg: 'DisconnectedFromCore: This transport has been disconnected (host: This Hero has been disconnected from Core (coreHost: ws://localhost:1818/hero))',
  stats: { start:<PHONE_NUMBER>594, end:<PHONE_NUMBER>451, ms: 857 }
}

packages.json

{
    "scripts": {
        "start": "sudo bash ./bash/chrome_sandbox.sh && node engine.js"
    },
    "dependencies": {
        "@ulixee/chrome-121-0": "^6167.185.8",
        "@ulixee/chrome-122-0": "^6261.129.11",
        "@ulixee/chrome-123-0": "^6312.124.11",
        "@ulixee/chrome-124-0": "^6367.208.11",
        "@ulixee/chrome-125-0": "^6422.142.11",
        "@ulixee/chrome-126-0": "^6478.183.11",
        "@ulixee/chrome-127-0": "^6533.120.11",
        "@ulixee/cloud": "^2.0.0-alpha.29",
        "@ulixee/execute-js-plugin": "^2.0.0-alpha.29",
        "@ulixee/hero": "^2.0.0-alpha.29",
        "@ulixee/hero-core": "^2.0.0-alpha.29",
        "@ulixee/hero-playground": "^2.0.0-alpha.29",
        "@ulixee/hero-plugin-utils": "^2.0.0-alpha.29",
        "@ulixee/net": "^2.0.0-alpha.29",
        "@ulixee/unblocked-specification": "^2.0.0-alpha.29",
        "acorn": "^8.12.0",
        "acorn-walk": "^8.3.3",
        "async-mutex": "^0.5.0",
        "axios": "^1.7.2",
        "better-sqlite3": "^11.1.2",
        "buffer-image-size": "^0.6.4",
        "cheerio": "^1.0.0-rc.12",
        "express": "^4.19.2",
        "expressjs": "^1.0.1",
        "form-data": "^4.0.0",
        "node-fetch": "^3.3.2",
        "node-schedule": "^2.1.1",
        "openai": "^4.51.0",
        "rxdb": "^15.24.0",
        "rxdb-premium": "^15.24.0",
        "rxjs": "^7.8.1",
        "sharp": "^0.33.4",
        "watch": "^1.0.2"
    },
    "accessTokens": {
    }
}

How do we repopulate ~/.cache/ulixee/?

same here
m1
node v20.5.0
chrome 128 (latest)
installed playground and cloud under sudo, ran cloud with sudo
Cannot even run basic example. Immediately after running script I see chrome popup saying it quit unexpectedly.

same here
m1
node v20.5.0
chrome 128 (latest)
installed playground and cloud under sudo, ran cloud with sudo
Cannot even run basic example. Immediately after running script I see chrome popup saying it quit unexpectedly.
// sorry, not the same error, mine says "Failed to launch Chrome! Cancel Pending Promise (Target.setAutoAttach): Target closed.\n"

Sorry you're having issues too. I think this is actually a different issue. Would you mind moving this to a different ticket and giving me a fuller set of logs before and after the error and your basic project setup? Thanks!

Ok! It does seem that it's fine after completely re-installing everything, it's really strange as this happened right after installing the latest Chrome from Ulixee, but it's now solved.
@blakebyrnes do you mind telling me in new Hero() how to specify the /tmp directory, it uses /tmp/.ulixee, I can't seem to find anywhere how to change it, I want each instance to use a different dir.

Ok, I guess that's good?? Haha. Kind of hard to know what went wrong there. Maybe you grabbed chrome 128 before the data files were ready or something?
You can change it per core: https://ulixee.org/docs/hero/overview/configuration#data-dir

Thank you so much for the help, I'm really unsure what happened, really strange, it was all working well.


Update food.css

Description
Fixed the header of the food blog website
Fixes:  #505

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

Checklist:

[x] I have made this from my own
[x] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK
Before

After

@VidhiBhatt01 added the index.html ....please review

Issues fixed
Buttons are on the end

Made it responsive

@VidhiBhatt01 Please review this... thanks :)

@vandana3869 the header is not responsive at all make a new PR when its fully ready

@arpit456jain thanks....sir can please suggest what changes I should make to make it more responsive


Help Me In Understanding It.

Hey There,
can you make me understant about this
Z[i, h, w, c] = conv_single_step(a_slice_prev, W[...,c], b[...,c])
i want to know why W[...,c] and b[...,c] . i want to know why Three dots there.
In CNN step by step

W[...,c] = W[:,:,:,c]
you can just write it :
Z[i, h, w, c] = conv_single_step(a_slice_prev, W[:,:,:,c], b[:,:,:,c])
and that means all values in W at the layer c of the filter

Thanks got it


Creating lamba layers in typescript

Hi, thank you so much for creating this plugin!
I'm very new to typescript and sam in general but was wondering how I would go about creating a lambda layer written in typescript.
For example, I want to be able to share code between many different lambda functions (spread across multiple sam templates).
Some advice on how to do this would be highly appreciated!
Thanks

Additionally, would lambda layers be the best approach to share code? Perhaps using webpack to import common code would work?

My preference is for using private npm packages for shared code.

Try to copy layers folder
webpack.config.js
const CopyPlugin = require('copy-webpack-plugin');

// Add the AWS SAM Webpack plugin
plugins: [
awsSamPlugin,
// @ Copy layers to build dir
new CopyPlugin([
  { from: 'layers', to: 'layers' },
]),

]


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 2.5.17 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


null input reader: be more careful with "full" (display) size

For the "null" reader, we were clobbering any configuration hints about
the "full" (i.e. "display window") size parameters. This had bad effects
when using a fake "null" image along with ImageCache::add_file() in
order to set up an in-memory buffer that behaves like a texture, if that
buffer was a crop region or had overscan.

LGTM


Custom routing syntax

I would like to try implementing URI Templates (RFC 6570) for module routing. There is the IRoutePatternMatcher which according to the wiki and the XML comment should be used to change the route syntax.
However, setting the RoutePatternMatcher in Bootstrapper has on effect. The matcher is never instantiated and never called.
What is the way to change how routes are matched?

How are you setting it in the bootstrapper. Looks like it should be
overriding the InternalConfiguration
On 9 November 2016 at 10:46, Tomasz Pluskiewicz<EMAIL_ADDRESS>wrote:

I would like to try implementing URI Templates (RFC7819) for module
routing. There is the IRoutePatternMatcher which according to the wiki
and the XML comment
https://github.com/NancyFx/Nancy/blob/master/src/Nancy/Routing/IRoutePatternMatcher.cs#L8
should be used to change the route syntax.
However, setting the RoutePatternMatcher in Bootstrapper has on effect.
The matcher is never instantiated and never called.
What is the way to change how routes are matched?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NancyFx/Nancy/issues/2608, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGappyBLeRhfl19oG4qpAgGGwCEXRlHks5q8aShgaJpZM4KtZbK
.

Aye, that's what I'm doing. Something like
public override NancyInternalConfiguration InternalConfiguration
{
  get
  {
    NancyConfiguration.WithOverrides(c =>
    {
      c.RoutePatternMatcher = typeof(MyMatcher);
    );
  }
}

Pinging @thecodejunkie @grumpydev
On 9 November 2016 at 11:09, Tomasz Pluskiewicz<EMAIL_ADDRESS>wrote:

Aye, that's what I'm doing. Something like
public override NancyInternalConfiguration InternalConfiguration
{
get
{
NancyConfiguration.WithOverrides(c =>
{
c.RoutePatternMatcher = typeof(MyMatcher);
);
}
}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NancyFx/Nancy/issues/2608#issuecomment-259389709, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGapqVHJR_xwHRYK9Izh5yIEB0LKQZBks5q8anfgaJpZM4KtZbK
.

Can you please share some code.

@phillip-haydon There really isn't much to it. This is a minimal example. OWIN-hosted module with the matcher set to custom implementation but it's never called nor instantiated.
using System.Collections.Generic;
using Microsoft.Owin;
using Nancy;
using Nancy.Bootstrapper;
using Nancy.Routing;
using NancyCustomRouting;
using Owin;

[assembly: OwinStartup(typeof(AppStart))]
namespace NancyCustomRouting
{
    public class AppStart
    {
        public void Configuration(IAppBuilder app)
        {
            app.UseNancy(options => options.Bootstrapper = new Bootstrapper());
        }
    }

    public class Bootstrapper : DefaultNancyBootstrapper
    {
        protected override NancyInternalConfiguration InternalConfiguration
        {
            get
            {
                return NancyInternalConfiguration.WithOverrides(configuration =>
                {
                    configuration.RoutePatternMatcher = typeof(UriTemplateMatcher);
                });
            }
        }
    }

    public class TestModule : NancyModule
    {
        public TestModule()
        {
            Get[""] = _ => "Home";
        }
    }

    public class UriTemplateMatcher : IRoutePatternMatcher
    {
        public UriTemplateMatcher()
        {
            // never called
        }

        public IRoutePatternMatchResult Match(
            string requestedPath,
            string routePath,
            IEnumerable<string> segments,
            NancyContext context)
        {
            // never called

            throw new System.NotImplementedException();
        }
    }
}

Uuuh. As far as I can see, IRoutePatternMatcher isn't used anywhere 😕 Where did you read about this? In the Legacy Routing section?

First I'd found the actual interface in source code and only later stumbled upon the wiki page.

Okay, so that obsolete interface is out, good.
So, do you have any suggestion on how I could implement an additional routing syntax? Probably one that would be used along the default.

I'm not sure how easy it is to just replace Nancy's routing. I'd start with the IRouteResolverTrie service. That's what's called when resolving the route handler to call.

http://www.philliphaydon.com/2013/04/09/nancyfx-implementing-your-own-routing/
I believe this is still more or less valid.

I think I need to extend Nancy one step earlier. Consider a route using URI Template like
Get["/{collection}{/page}{?firstname,lastname,occupation}"] = GetCollection;

It will match URLs as follows
/cars => {
           "collection": "cars",
		 }

/toys/2 => {
             "collection": "toys",
			 "page": "2"
		   }

/guests/2?firstname=phillip => {
                                 "collection": "toys",
			                     "page": "2",
								 "firstname": "phillip"
		                       }

As you see, URI templates don't match the current routing which assumes segments separated by slashes. Will IRouteResolverTrie handle that?

Oh greedy routes are hard to implement.
Also query strings are not part of the route so you can't pattern match on those.

IMO this is not a good practice anyways. Nancy provides Modules for this
purpose.
On Nov 11, 2016 19:17, "Phillip Haydon"<EMAIL_ADDRESS>wrote:

Oh greedy routes are hard to implement.
Also query strings are not part of the route so you can't pattern match on
those.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NancyFx/Nancy/issues/2608#issuecomment-259993038, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABPNX2d0FnCqWwVPYZS8o8_1js28w_Jhks5q9JT8gaJpZM4KtZbK
.

I'm sorry what is not a good practice? Greedy routes?
Also, I don't understand why you mention modules. I wouldn't want to replace them. Just add a different way to match incoming URLs.

@tpluscode I apologize that was totally BS. I deleted the comment right after I wrote it. :smile:
Been working for too long today.

Didn't refresh lol. No worries ;)

@phillip-haydon I've been doing some experimenting and it seems that the best way for a completely different approach is to decorate the DefaultRouteSelector. I've created a proof of concept at https://github.com/tpluscode/Nancy.Routing.UriTemplates. Please do have a look.
There are a few obvious weak spots like lack of prioritization (akin to Nancy's segment scoring) and a naive way to select matching route . I'm not yet sure how I would build a trie from URI Templates like you do. Also there isn't an official syntax to extend routes with constraints but that could probably be somehow mitigated by a custom extension to the spec.
There are a few details I'd like to ask about:

I created a separate UriTemplateModule base class which holds the URI Template routes separate from built-in routes. Yet I've overridden the NancyModule#Get methods. I think I probably need a separate set like GetByTemplate or shorter GetT?
I'm actually including the query string in the matched route. For me a URL like /user/1 and /user?id=1 are functionally equivalent. I don't see why query strings wouldn't be included in the route. What do you guys think?
Is the InternalConfiguration the only way to set up a custom route resolver?
Also, I already use Nancy 2 preview. Are there any major breaks from 1.x branch if I wanted to support both. Or maybe I shouldn't be bothered with the old stuff anymore?


Fixed failing build

Damn, why did this not show up when I merged the other PR?

I was wondering this too. My only guess would be that the check runs only when the PR is first opened, but in the meantime another merge might break it. Not familiar with this though, could be wrong.

Account number:
8ff92d181cb09f18ef2a1b03badb5bf4a8962f7e5bb55641d0358d99bf7d5655

Hi @xmac11 is this PR associated with any issue?

@itsnikhil nope

@itsnikhil nope

Thanks, you will receive payment for this soon. But from next time please create an issue before submitting any PR as our payment system is designed in that way.


Adding worker upload instructions

Filling in a gap in this doc. I think it'd be helpful to give people instructions on upload and note the syntax to pass environmental variables.

Did not mean to open a PR against my own repo :(


Added cargo build via --bin flag

Content must go through a pre-merge checklist.
Pre-Merge Content Checklist
This documentation has been checked to ensure that:

[ ] The title, template, and date` are all set
[ ] Does this PR have a new menu item (anywhere in templates/*.hbs files) that points to a document .md that is set to publish in the future? If so please only publish the .md and .hbs changes in real-time (otherwise there will be a menu item pointing to a .md file that does not exist)
[ ] File does not use CRLF, but uses plain LF (hint: use cat -ve <filename> | grep '^M' | wc -l and expect 0 as a result)
[ ] Has passed bart check
[ ] Has been manually tested by running in Spin/Bartholomew (hint: use PREVIEW_MODE=1 and run npm run styles to update styling)
[ ] Headings are using Title Case
[ ] Code blocks have the programming language set to properly highlight syntax and the proper copy directive
[ ] Have tested with npm run test and resolved all errors
[ ] Relates to an existing (potentially outdated) blog article? If so please add URL in blog to point to this content.

Thanks, @dipankardas011 we will take a look at this soon and respond.

Thank you @dipankardas011
This has raised a great question about building in debug vs building a release.
Thanks for your contribution, we have tweaked the page and this can be merged now.
Kind regards
Tim.

Welcome🙌


[Feature Request]: Graceful failure if ComfyUI EXIF fails to parse

Thank you for all your efforts to add ComfyUI support, it is very appreciated! The current EXIF parsing works when using ComfyUI nodes but fails when certain custom nodes are used to do sampling. With the current way EXIF data is managed in Comfy, I don't think it is possible to cover custom node implementations so it would be preferable that if the EXIF data fails to parse because it can't locate the KSampler node, it doesn't show an error in the UI. It makes it hard to navigate when the error keeps appearing for looking over ComfyUI images.
I've attached an example PNG where it fails to parse.

Already added

Amazing, works like a charm 😄


Missing period in Create list item HTTP example

In the Create list item example, is there a period missing in the HTTP example for the meta data's type? The type value is SP.Data.TestListItem and I think it should be SP.Data.Test.ListItem.
Also, it would be nice to remind coders that the list name "Test" should use the Unicode escape sequence for spaces and special characters, where a space is encoded as _x0020_.

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

ID: e5a8f160-c2fd-8692-1c3f-2c5c44c7a286
Version Independent ID: 2d7064f8-409c-4465-3fbb-bd6a4c2806f5
Content: Working with lists and list items with REST
Content Source: docs/sp-add-ins/working-with-lists-and-list-items-with-rest.md
Service: sharepoint
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

The period was not part of the item name:
<d:EntityTypeName>Calendar_x0020_DCList</d:EntityTypeName>
so it was unneeded. Thanks


Random - Qoute-Machine code in codepen

Challenge Build a Random Quote Machine has an issue.
User Agent is: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:46.0) Gecko/20100101 Firefox/46.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.
Hi
I am Vaibhav Sharma . Recently while doing Random Quote Machine API based project , in the reference code of the project (which I opened just for the reference)I found the qoutes are missing the right qoute icon.
You can see them in the attached file.
.
I want you to review this bug ( Once you get free) .
Thank you

Please post your frontend related questions in https://gitter.im/FreeCodeCamp/HelpFrontend room
Include the following

A link to your project url (codepen, etc)
Description of the problem

I would say, it's an unique design to present the quote :smirk:

Just a UI design decision to think out of the box.

Visually, it creates a nice effect. It's also true people naturally look for closing quotes as it's sort of basic grammar thing.

It's just a visual style. Lot of quote sites use them and lots do not
For instance:

Closing as bikeshedding.


GCP: Listings show 0 pages

On the GCP listing page, it shows "0 - 0 of 0" for the page count.

Taken a look at it. That issue is caused by the pages file in the S3 bucket. our pages start with the name "0" which means that the content of the pages file is actually completely accurate when it says
{"first": 0, "last": 0, "entries": 50}
But as we parse this file in front-end to get the number of entries / pages we calculate the last page * number of entries which obviously results in 0 entries and 0 pages :p


442 Update SQL statement.

Fix https://github.com/CodeForPoznan/pah-fm/issues/442

Waiting for answer here https://github.com/CodeForPoznan/pah-fm/issues/442

The numbers you've inserted there for rsa key are incorrect. They are not prime and the public exponent differs from the one defined in settings which would mean that every computation, even with correct key, would fail.
I much rather would like to see a proper rsa key there, but that is not needed anyway, because why would we want raw sql queries here to update stuff in the table if we have python, models and factories :)
Anyway, go on and remove this file along with its directory and mention in Readme. It's not needed anymore, we have make populate-database for that purpose.

Good point with RSA elements :+1:
I removed redundant stuff.


Switch testing framework from LCM to YDL

Switch the testing framework from using LCM to YDL, and add do some testing.
Support multiple read targets.

Known issue, ydl on master works better. I don't feel like merging that rn, but someone should checkout ydl from master and make sure it works here...


Add DefaultFormatter.DisableScientific

Issue: https://github.com/gavv/httpexpect/issues/190
Tasks:

[x] Add DisableScientific flag in DefaultFormatter
[ ] Add/update unit test

Coverage: 95.436% (+0.007%) from 95.43% when pulling 501ce3b521a71e4a73780ddb8f336cde7e7c7ae4 on neverbeenthisweeb:190-add-disable-scientific-impl into 69c1ae446081de7b4d305213e459dcea58cad613 on gavv:master.


Tests added for Array to and from Mapping

Added a few tests for main usage of from and to Mappings, to address issue #15.
(The full test failed due to removed method 'range' and the 'WebGuy' class)

@shadowhand :+1:


Disable SpringAmqpBatchIT due to flakyness

What does this PR do?
Unfortunately is looks like SpringAmqpBatchIT still causes flakyness on the CI, see this run for exmaple.
I assume it is due to the RabbitMQ containers not being properly shutdown, because it can be reproduced by running the test in repeated mode locally.

@elasticmachine run elasticsearch-ci/docs


Caused by: com.amazonaws.services.dynamodbv2.model.AmazonDynamoDBException: The provided key element does not match the schema

Hi Team,
While trying to write spark dataframe to dynamodb I am getting the below error message
Error:
Caused by: com.amazonaws.services.dynamodbv2.model.AmazonDynamoDBException: The provided key element does not match the schema (Service: AmazonDynamoDBv2; Status Code: 400; Error Code: ValidationException; Request ID: O1THUCBT474TKDSSON3POOD3DBVV4KQNSO5AEMVJF66Q9ASUAAJG; Proxy: null)
Note: 50 records are successfully inserted into dynamodb while writing after 50 records below error is coming. There are nearly 5.7million rows in my dataframe
DynamoDB table structure:
f_cartid -> primary key datatype:Number
f_cartmenuid -> sort key datatype:Number
Spark write code:
df.write.option("tableName", mytable).option("region",region).format("dynamodb").save()
Spark Dataframe schema structure:
fnDF.printSchema()
root
|-- f_cartid: long (nullable = true)
|-- f_cartmenuid: long (nullable = true)
|-- f_orderid: double (nullable = true)
|-- f_hotelid: double (nullable = true)
|-- items: struct (nullable = false)
|    |-- f_cartmenuid: long (nullable = true)
|    |-- menuDetails: struct (nullable = false)
|    |    |-- f_menuid: long (nullable = true)
|    |    |-- f_qty: long (nullable = true)
|    |    |-- f_menuprice: double (nullable = true)
|-- users: struct (nullable = false)
|    |-- f_custid: long (nullable = true)
|    |-- userDetails: struct (nullable = false)
|    |    |-- F_Name: string (nullable = true)
|    |    |-- F_EmailId: string (nullable = true)
|    |    |-- f_delpostcode: string (nullable = true)
|    |    |-- f_deliveryaddress: string (nullable = true)
|-- f_device_name: string (nullable = true)
|-- f_ipaddress: string (nullable = true)
|-- f_carttotalamt: double (nullable = true)
|-- f_wanted_asap: long (nullable = true)
|-- f_deliverabletime: string (nullable = true)
|-- f_paymentcharge: double (nullable = true)
|-- f_redempoint: double (nullable = true)
|-- f_dellocation: long (nullable = true)
|-- f_delcharge: double (nullable = true)
|-- f_couponcode: string (nullable = true)
|-- f_crdt: timestamp (nullable = true)
|-- f_moddt: timestamp (nullable = true)
|-- updated_at: date (nullable = true)
Could anyone guide me where I am missing, it would be really helpful.
Thanks in advance!!

Hi gopi-t2s
I notice in your schema that your hash and range key fields are nullable.
|-- f_cartid: long (nullable = true)
|-- f_cartmenuid: long (nullable = true)

Can you check if you do in fact have null values in your dataframe? That would expectedly cause an error.

Hi gopi-t2s
I notice in your schema that your hash and range key fields are nullable.
|-- f_cartid: long (nullable = true)
|-- f_cartmenuid: long (nullable = true)

Can you check if you do in fact have null values in your dataframe? That would expectedly cause an error.

Thanks @jacobfi, I will check that, however I handled this error by changing the data type of hash and range keys from long to string.

Thanks @jacobfi, I will check that, however I handled this error by changing the data type of hash and range keys from long to string.


Update CCM/CSI driver to handle ZFS limitation for FS path

Is this a BUG REPORT or FEATURE REQUEST?
Choose one: FEATURE REQUEST

Versions
CCM Version:
1.26.2
Environment:

Kubernetes version (use kubectl version):
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Others:

What happened?
What you expected to happen?
How to reproduce it (as minimally and precisely as possible)?
Anything else we need to know?

We have discussed this over call and email, CSI driver currently supports OCI Block Storage and File Storage Service only. ZFS is a different service and not part of FSS, hence making changes to existing driver to support for ZFS is not the best solution here. The details of why it can't be done this way is shared over email and call.
This is a feature request for adding a new driver supporting ZFS.


Request: Ex Raid Start time

Could we please get the start time added to the Ex-Raid Cards. I have found that users will sometimes think the posted time is start and not end.

Historically, it seems like EX raids have a duration of 45 minutes.  For now I will just use that as an assumption going forward and just derive the start time from the provided end time.


Update scalafmt-core to 2.6.0

Updates org.scalameta:scalafmt-core from 2.5.3 to 2.6.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, semver-minor

Codecov Report

Merging #126 into master will decrease coverage by 6.81%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #126      +/-   ##
==========================================
- Coverage   47.72%   40.90%   -6.82%
==========================================
  Files           1        1
  Lines          44       44
  Branches        1        1
==========================================
- Hits           21       18       -3
- Misses         23       26       +3

Impacted Files
Coverage Δ

.../main/scala/io/circe/bson/BsonCodecInstances.scala
40.90% <0.00%> (-6.82%)
:arrow_down:

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 7da1c65...603efc8. Read the comment docs.


[SPARK-20431][SQL] Specify a schema by using a DDL-formatted string

What changes were proposed in this pull request?
This pr supported a DDL-formatted string in DataFrameReader.schema.
How was this patch tested?
Added tests in DataFrameReaderWriterSuite.

Test build #76034 has started for PR 17719 at commit 8447a6d.

Test build #76034 has finished for PR 17719 at commit 8447a6d.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #76037 has started for PR 17719 at commit a1a2e35.

Test build #76037 has finished for PR 17719 at commit a1a2e35.

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/76037/
Test PASSed.

cc: @gatorsmile

Test build #76061 has started for PR 17719 at commit 5fe5e39.

Test build #76061 has finished for PR 17719 at commit 5fe5e39.

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/76061/
Test PASSed.

Merged build finished. Test PASSed.

@gatorsmile ping

@gatorsmile ping

In PySpark, we have multiple ways to specify the schema. However, it sounds like we silently pick one of the input schemas. Could you submit a separate PR to fix the potential schema conflicts in PySpark?
Thanks!

Test build #76773 has started for PR 17719 at commit cc3afd7.

Test build #76774 has started for PR 17719 at commit 46994fb.

Sure, I'd love to do tough, I probably missed your point. What's the scenario of the conflict you described? The current logic in readwriter.py just checks types, then decides a schema. Could you give me a concrete example? Thanks

Test build #76773 has finished for PR 17719 at commit cc3afd7.

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/76773/
Test PASSed.

Test build #76774 has finished for PR 17719 at commit 46994fb.

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/76774/
Test PASSed.

nvm. I will fix it later. Thanks!

LGTM

Thanks! Merging to master.


Maximum client

I can only have 14 instance running. After than i get the error "Maximum Client".
Someone have a trick to have more instance ?
Is it the driver maximum or the software maximum ?
Thanks !
Ubuntu 14.04
Radon Crimson 15.12 Drivers
Zogminer

I have never tested more than 12 running together. Looks like a stratum limit, when connecting to the server.

I try with 2 different server it's not working ?

Maximum number of clients reachedMaximum number of clients reachedMaximum number of clients reached
Zcash Miner version v1.0.0-beta2-9a64575 (2016-11-02 11:01:47 +0100)
ZcashMinerThread started
ERROR: No suitable GPU found! No work will be performed!

Maybe you can try with 3-4 instance with gpu to have more than 14 ?


Updated script to include support for docker command

Fixes:#1505 https://github.com/keploy/keploy/issues/1505
Description
This PR modifies the script to include support for running Docker commands specifically for the keploy test step. It ensures there is support for Docker Commands.
Changes Made

If the provided command includes "docker", it directly executes the Docker command specified in the $COMMAND variable.
The keploy test command is constructed using the Docker command specified in the $COMMAND variable, along with other parameters.

Reason for Change:
The current support is for only java.js,go,py, hence by adding docker command support, we help in running keploy test run with other frameworks including the above ones.

@Sonichigo Kindly review this


Changes to scss watch is not working

Very easy to use a plugin for compiling SCSS to CSS. One issue I found is that it complies with CSS files fine but later when I modify the SCSS file the CSS file is not reflecting the changes until I run the command npm run start again. BTW in npm start, I have the following command npx @11ty/eleventy --serve. Please let me know how to resolve this issue?

Thanks for reporting.
I'm sorry, but, in my local environment, there is no such issue.
Could you please show me a minimal Eleventy project that doen't recompile SCSS files when modified?

Thanks for your quick response. I have uploaded my project in git repo here

Thank you, too.
I just confirmed it works fine in my environment:
node v17.8.0
macOS 12.3 Monterey

I am trying it on LTS node version 16.14.2 on windows 11.
Just to confirm have you run the command npm run start and changed SCSS file which intern reflected the changes in CSS file?

Yes. I run npm run start, open localhost:8080 in my browser, and changed background-color in scss/site.scss file.
Browser automatically updated the background color.
Could you try the following command:
DEBUG=*EleventySass* npm run start

and modify your scss/site.scss file, for example, and paste the DEBUG output here, please?
It will be a big help for me to figuring it out.

Thanks for giving me your environment info.
I just tested it again on LTS node version 16.14.2 on macOS, and it works.

When you run npx @11ty/eleventy --serve or npm run start prepended with DEBUG=*EleventySass* ,
if the plugin detects a file change, you will see something like the following output:
[11ty] File changed: src/scss/site.scss
  Dev:EleventySass eleventy.beforeWatch queue: [ './src/scss/site.scss' ] +6s
  Dev:EleventySass getCacheKey(contents, ./src/scss/_reset.scss) -> ./src/scss/_reset.scss2022-03-27T15:50:15.077Zinitial +29ms
  Dev:EleventySass getCacheKey(contents, ./src/scss/site.scss) -> ./src/scss/site.scss2022-03-27T15:50:20.805Z +4ms
  Dev:EleventySass Has compiled ./src/scss/site.scss +44ms
  Dev:EleventySass:DependencyMap Dependencies for ./src/scss/site.scss is not changed: [./src/scss/_reset.scss] +6s
  EleventySass:SourceMapStore public/css/site.css.map written.. +6s
[11ty] Wrote 19 files in 0.10 seconds (5.3ms each, v1.0.0)
[Browsersync] Reloading Browsers...

When I try the above command I get the following error
DEBUG=EleventySass: The term 'DEBUG=EleventySass' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

I'm sorry I wrote the script for *nix.
I'm not familiar with the Windows environment, but
if you are using cmd.exe, try the following:
set DEBUG=*EleventySass* & npm run start

if you are using PowerShell:
$env:DEBUG="*EleventySass*"; npm run start

I'm not sure, but this might be related to https://github.com/11ty/eleventy/issues/2183.

Below is what I get doesn't look like plugging is getting the watch event :(
[Browsersync] Serving files from: public
[11ty] File changed: src\scss\site.scss
[11ty] Writing public/sitemap.xml from ./src/sitemap.xml.njk
[11ty] Writing public/feed/feed.xml from ./src/feed/feed.njk
[11ty] Writing public/feed/.htaccess from ./src/feed/htaccess.njk
[11ty] Writing public/feed/feed.json from ./src/feed/json.njk
[11ty] Writing public/css/site.css from ./src/scss/site.scss
[11ty] Writing public/posts/fourthpost/index.html from ./src/posts/fourthpost.md (njk)
[11ty] Writing public/posts/secondpost/index.html from ./src/posts/secondpost.md (njk)
[11ty] Writing public/posts/thirdpost/index.html from ./src/posts/thirdpost.md (njk)
[11ty] Writing public/posts/2018/firstpost/index.html from ./src/posts/2018/firstpost.md (njk)
[11ty] Writing public/404.html from ./src/404.md (njk)
[11ty] Writing public/posts/index.html from ./src/archive.njk
[11ty] Writing public/index.html from ./src/index.njk
[11ty] Writing public/page-list/index.html from ./src/page-list.njk
[11ty] Writing public/tags/index.html from ./src/tags-list.njk
[11ty] Writing public/tags/second-tag/index.html from ./src/tags.njk
[11ty] Writing public/tags/number-2/index.html from ./src/tags.njk
[11ty] Writing public/tags/posts-with-two-tags/index.html from ./src/tags.njk
[11ty] Writing public/tags/another-tag/index.html from ./src/tags.njk
[11ty] Writing public/about/index.html from ./src/about/index.md (njk)
[11ty] Benchmark      9ms   9%   215× (Configuration) "url" Nunjucks Filter
[11ty] Wrote 19 files in 0.10 seconds (5.3ms each, v1.0.0)
[11ty] Watching…
[Browsersync] Reloading Browsers...

Tried again below is the debug logs
[Browsersync] Serving files from: public
[11ty] File changed: src\scss\site.scss
Dev:EleventySass eleventy.beforeWatch queue: [ './src\scss\site.scss' ] +34s
Dev:EleventySass getCacheKey(contents, ./src/scss/_reset.scss) -> ./src/scss/_reset.scss2022-03-27T16:36:15.062Zinitial +77ms
Dev:EleventySass getCacheKey(contents, ./src/scss/site.scss) -> ./src/scss/site.scss2022-03-27T16:36:15.064Zinitial +8ms
[11ty] Wrote 19 files in 0.11 seconds (5.8ms each, v1.0.0)
[Browsersync] Reloading Browsers...

Thanks a lot.
I'll investigate it.

I just published v1.1.5.
Hope this works.

I just published v1.1.5.
Hope this works.

Excellent, I have tested it now and it works, thanks a lot. Please close this issue.
I have one query unrelated to this. In the build script for production, how do we implement cache buster i.e. adding some hash version to the end of the css URL, is there any built-in functionality in this plugin?

Thanks for testing!
Your contributions make this plugin work on Windows. Thank you very much.
I'm sorry the plugin does not have any cache buster functionality.
Thanks for reminding me. I'll think about it

I thought about cache buster, and if I understand correctly, cache buster is how you refer to (CSS) files.
Therefore, it is not so related to this plugin, because eleventy-sass doesn't refer to CSS files, but generates CSS files.
As you may know, all you have to do when implementing cache buster is just add a query parameter at the end of the value for href attribute in the stylesheet link tags.
There seems to be lots of articles on how to implement cache buster in Eleventy projects.
For example:
https://rob.cogit8.org/posts/2020-10-28-simple-11ty-cache-busting/


Rename shininess to glossiness

Shininess is a legacy of the past. It takes a value in the range 0 to 100 which is totally out of step with the other StandardMaterial scalar properties. Also, since a glossMap is essentially per-pixel glossiness, we should rename.
We should:

Create a new property called glossiness (which is essentially shininess mapped to the 0 to 1 range).
Alias shininess to glossiness (dividing the value by 100). The setter/getter should print a warning to the JS console that that property is deprecated and glossiness should be used instead.

Note that a property called shininess will remain in the material JSON format, at least for now.

done in https://github.com/playcanvas/engine/pull/4949

@slimbuck  Really? How come this is still here:
https://github.com/playcanvas/engine/blob/129062e8602e89b05d2d7311828fad619355b27b/src/scene/materials/standard-material.js#L242-L245

Yeah, we should deprecate it .. move this function to deprecated as well.
https://github.com/playcanvas/engine/blob/129062e8602e89b05d2d7311828fad619355b27b/src/scene/materials/standard-material.js#L1113-L1121


Plan to support Pact v4

Hi!  Thanks for this useful library!
Would there be any plan in the near future to migrate support to 4.0?  We are trying to use the Java adapter for the provider side and the validation method seems to default to 4.0 which is causing some incongruence with our process.
Thanks again!

Pact-JVM should support verifying up to and including V4 pact files, so that sounds like a separate issue best raised in the pact-jvm project.
With regards to supporting additional specifications in the adapter, it would be preferable to at some point support V4 in all of our languages by default. We have been making good strides in bringing the clients libraries. The adapters are a bit of a periphery, the main use case is in bi-directional contract testing, where  the generated contracts are statically compared against an OpenAPI document. The tool used for comparison supports V2-V4 Pacts, but doesn't support traditional CDCT techniques such as provider states or matchers (schema matching is performed by default).
That being said, the route to V3/V4 spec in the adapters, is certainly manageable for someone as a community contribution
general adapter guidance:- https://docs.pactflow.io/docs/bi-directional-contract-testing/contracts/pact#converting-mocks-into-a-pact-compatible-format
pact v4 spec compat:- https://docs.pactflow.io/docs/bi-directional-contract-testing/contracts/pact#supported-pact-specification-versions
See caveat about v4 pacts
The pact serialisation function is here
https://github.com/pactflow/pact-cypress-adapter/blob/ac177d75dd75e16bef8c55986125780650357ee7/src/utils.ts#L63-L77
which creates the structure which adheres to a pact v2 specification.
we provide typed schemas for the pact specification here
https://github.com/pactflow/pact-schemas
which can be used to validate the created pact if required (can definitely help during development)
We would want to expose an ability to set the pact specification version, and have it default.

Thanks @swimclan, the purpose of this is to work with PactFlow's BDCT feature (as noted) so we only require a minimum of v2 for this - but v4 support should be fairly straightforward if you'd (or somebody else) would like to add it.


Use relative scale of bounding box in ground truth and detections.

Use relative scale of bound box in both ground truth and detections, so that it is easy to evaluate the performance without scaling again.

coco bbox is scaled to the original image, while each model will probably resize to some other sizes, using the relative position [0, 1] can make the bbox comparable directly from ground truth to detection output


array_to_string panic on dictionary values

Describe the bug
array_to_string function panic's when running a query on data that has dictionary data type. To fix the panic, array_to_string needs to support dictionary type
To Reproduce

Use DataFusion CLI:

# in dir datafusion/datafusion-cli
$ cargo build
$ ./target/debug/datafusion-cli

Write sample data

CREATE TABLE table1 AS VALUES (1, arrow_cast('foo', 'Dictionary(Int32, Utf8)')), (2, arrow_cast('bar', 'Dictionary(Int32, Utf8)'));

Run query

> SELECT array_to_string(array_agg(column2),',') FROM (SELECT column2 FROM table1);
thread 'main' panicked at .../datafusion/datafusion/functions-array/src/string.rs:298:17:
internal error: entered unreachable code
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Expected behavior
> SELECT array_to_string(array_agg(column2),',') FROM (SELECT column2 FROM table1);
+------------------------------------------------------+
| array_to_string(ARRAY_AGG(table2.column2),Utf8(",")) |
+------------------------------------------------------+
| foo,bar                                              |
+------------------------------------------------------+

Additional context

Here are the data types that array_to_string currently supports:
https://github.com/apache/datafusion/blob/59120255916bcd624161ce8f5df255f2cc838406/datafusion/functions-array/src/string.rs#L63-L101

instead of panic, it would be better to throw an error

I think this is a good first issue as it is well documented , has a test case and the desired behavior is well understood

I would like to take this one.

take

take


Run workflow on feature branches

Rolling build workflows now should run on workflow_dispatch and pull_request events on branches matching *feature* or *feature/**.
These jobs fail now, because of #688

Is this something we really need? Usually, maintainers can restart CI quite without problems, just opening the workflow.

Now it would be handy with the jtc-features branch, because the PRs targeting into it aren't checked (can you manually run them on an arbitrary PR?). For this maybe the change of workflow_dispatch is not really necessary, but pull_request would be.
But I don't now if this feature branch is an extraordinary exception, or if that might be a best-practice example for the future.

Should we revert this, or do we want to merge those changes into master too?

I vote for bringing this to master for other new feature branches in the future.


What is the best way to check the permissions of a Gardens?

It would be very convenient :-D

Will be address with #413


Functional components cannot accept classes

Vue.js version
2.1.10
Reproduction Link
http://jsfiddle.net/5sH6A/993/
Steps to reproduce
Render a functional component, and attach a class to it.
What is Expected?
The component should be rendered with the given class.
What is actually happening?
The component is rendered, but without the class.
Additional info
It appears that we cannot accept a prop named class either, as it is always equal to whatever it's default value is.

Functional components do not inherit anything from the placeholder element - the raw bindings are contained in context.data. You need to do something like this to inherit them:
render (h, ctx) {
  return h(
    'div',
    ctx.data, // inherit raw binding data from placeholder node
    [ ... ]
  )
}

Sweet, thanks for the fast response!


fix jsonp in ie8

http://msdn.microsoft.com/en-us/library/ie/hh180173(v=vs.85).aspx
script element in ie have no "load" event, use onreadystatechange instead

Thanks for that


[stable/redis] Using redis as a subchart does not correctly apply the disableCommands

Hi,
I'm using redis as a subchart to my own chart.
If I install the chart directly, it works just fine, but when I use as a subchart with the below config section, the FLUSHDB/ALL commands are still disabled.
redis:
  cluster:
    enabled: false
    slaveCount: 1
  usePassword: false
  exporter:
    enabled: true
  master:
    disableCommands: null
  slave:
    disableCommands: null
  configmap:
    notify-keyspace-events AKE
  networkPolicy:
    enabled: false
  sysctlImage:
    enabled: true
    command:
      - /bin/sh
      - -c
      - |-
        install_packages systemd
        sysctl -w net.core.somaxconn=10000
        echo never > /host-sys/kernel/mm/transparent_hugepage/enabled
    mountHostSys: true

Hi @arrkaye
Could you try with the values.yaml below?
redis:
  ...
  master:
    disableCommands: []
  slave:
    disableCommands: []
  ...


Option to set a working directory

User would be able to set a custom path and would be able to use relative file paths on render functions.
Suggested syntax:
eryn.setOptions({
  workingDirectory: 'my/path/to/views'
});

// eryn will use working directory to find the absolute path to the file passed
eryn.render('home.eryn', context);

// eryn should still be able to detect an absolute path to a file and ignore
// working directory in that case
eryn.render('C:\\projects\\meow\\cat.eryn', context);

Approved, will be added in a future version.


Java throwing exception - databaseName can not be null

Observed behavior
Upon trying to deploy the backend, Java throws an exception when running dt-v3.2.0-773-g495f7c7.jar
See screenshots for details

Expected behavior
Backend deploying
Steps to reproduce the problem
java -jar target/dt-v3.2.0-773-g495f7c7.jar configurations/default/env.yml configurations/default/server.yml
Please describe any applicable config files that were used
standard env.yml and server.yml
Only modifications are for api keys...
Version of datatools-server and datatools-ui if applicable (exact commit hash or branch name)
af00ed
@landonreed would you happen to have a solution to this issue?

Apologies for the delayed response, it looks either MongoDB is not running or the MONGO_DB_NAME in env.yml has not been set.


Request connection Timeout feature

Hi
Does Pyswagger has a feature to set the connection timeout for the client.request() ???
for example in requests we have
requests.get(w, verify=False, timeout=10)

You can provide those parameters via 'send_opt' when init client
from pyswagger.contrib.clients.requests import Client
client = Client(send_opt=dict(
   timeout=10,
   verify=False,
))

@pradeepYogesh  There is no further action needs to be taken, I'll close this issue and please feel free to reopen it if something goes wrong.


[bugfix(CLI)]: check whether  the user has enabled etcd v2 protocol.

Summary
check whether  the user has enabled etcd v2 protocol.
Full changelog
update the apisix_cli_test.sh to provide a friendly error message when  etvd v2 not enabled.
Issues resolved
Fix #1523

@dabue I think you need to add this check in bin/apisix intead of travis CI.

@dabue I think you need to add this check in bin/apisix intead of travis CI.

You are right. It's more reasonable to put it there，I will update.

I have updated it, please review, thx.

@dabue merged right now, many thx.


fix imread deprecation error in Scipy

Fix #9
Use matplotlib.pyplot.imread instead

I see there is a similar fix in #7 , but I think matplotlib.pyplot.imread might be better. And it brings more compatibility.

SciPy's imread used to return a numpy.ndarray. imageio.imread returns imageio.core.util.Array. If you want/need a numpy.ndarray and don't want to convert it, use matplotlib.pyplot.imread, as it also returns a numpy.ndarray.
https://stackoverflow.com/questions/15345790/scipy-misc-module-has-no-attribute-imread


[WIP] Use schema builder

Brings changes from https://github.com/RedHatInsights/frontend-components/pull/190
Merge together with https://github.com/ManageIQ/sources-api/pull/97

@rvsia can you look at the travis failure?

@agrare I was waiting for the release of a new AddSourceWizard release. It should be fine now

Codecov Report

Merging #111 into master will increase coverage by 0.32%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
+ Coverage   74.14%   74.47%   +0.32%
==========================================
  Files          36       36
  Lines         441      427      -14
  Branches       67       60       -7
==========================================
- Hits          327      318       -9
  Misses         87       87
+ Partials       27       22       -5

Impacted Files
Coverage Δ

src/Utilities/SourcesFormRenderer.js
100% <ø> (ø)
:arrow_up:

src/components/SourceEditForm/SourceEditModal.js
25% <25%> (+1.47%)
:arrow_up:

src/components/SourceEditForm/editSourceSchema.js
28.57% <33.33%> (-19.58%)
:arrow_down:

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 f4f18ef...fc1a6d5. Read the comment docs.


Why not empty  the FutureContext?

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Ask your question here
java version: 1.8
dubbo version: 2.7.12
I have a dubbo service and it named "main". main service set 500 for provider->threads, so 500 threads to choose from main service when main service called by a dubbo customer.However main service going to call another dubbo service, and it get some results successfully and there results have large size. I look Dubbo put the result to FutureContext(tips: The relevant code is at org.apache.dubbo.rpc.protocol.AbstractInvoker.java:193), but it not be emptied never. This leads that my part of app memory isn't recycled, i.e 500 threads will hold own FutureContext, and every FutureContext will have the large obeject. My app is OOM.

FutureContext is used after invoke to get result from Dubbo, which means Dubbo have no time to clear it.
You can try update to 3.0.11, which will do not set FutureContext if the invocation is not in async mode.


⚠ 366.ru has degraded performance

In 82216e7, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 1103 ms

Resolved: 366.ru performance has improved in c75e533.


lagarith-lossless-video-codec: Add version 1.3.27

Closes #10724

[x] I have read the Contributing Guide.

/verify


update some outdated documentation

Update some outdated documentation
For example: the default subnet is closed、some dead links
make sure your commit is signed off
Signed-off-by: ty-dc<EMAIL_ADDRESS>

/lgtm

need to update doc/mkdocs.yaml ?


Cannot enable android.permission.SCHEDULE_EXACT_ALARM on Pixel Watch 3 running WearOS 5

Hello!
I am using gluco datahandler on a pixel 9 phone and pixel 3 watch. I have the error message "Schedule exact alarm is disabled!!! GlucoDataHandler may not work correct!!! Press here to go direct to the permission setting" on my watch.
When I press there I get Alarms and Reminders, it shows loading apps for a few seconds and then that disappears. If I go to app permissions through settings I also have no option to grant permission for alarms and reminders for any apps.
It appears this is a known issue, and there are some possible fixes in the google issue tracker thread, but they are beyond my limited capabilities to implement.
Here is the google issue tracker: https://issuetracker.google.com/issues/365493102
My alarms are working most of the time but sometimes they don't work in the night and that is not good. I was using gluco datahandler for about a year with my pixel 1 and it was stable and wonderful.
Please help!  :)

Hi,
yes the only solution is using the adb command as I wrote already on in this issue.
But if you don´t know, how to use adb, it should normally work without this permission as long you are using the phone version for receiving data.
What source are you using? Do you use the watch to receive the data from source or the phone?
As long you are using the phone, the phone should also trigger the alarm, even on watch.
Michael

Thank you for responding so quickly! Yes, I am using juggluco on the phone
as the source and it does seem to work most of the time. Occasionally it
will just vibrate once instead of a sustained vibration, but maybe that is
unrelated to the permissions issue. Thank you for the info, I feel safer
trusting it now.
On Fri, Dec 6, 2024, 7:59 AM pachi81 @.***> wrote:

Hi,
yes the only solution is using the adb command as I wrote already on in
this issue.
But if you don´t know, how to use adb, it should normally work without
this permission as long you are using the phone version for receiving data.
What source are you using? Do you use the watch to receive the data from
source or the phone?
As long you are using the phone, the phone should also trigger the alarm,
even on watch.
Michael
—
Reply to this email directly, view it on GitHub
https://github.com/pachi81/GlucoDataHandler/issues/186#issuecomment-2523436867,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BNQEE2BVWPX5436JJMHFBMT2EG3VTAVCNFSM6AAAAABTEZT7OCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRTGQZTMOBWG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Is the short vibration on watch or phone?
For watch, please go to advanced alarm settings and increase the delay.
You can also test the alarms on watch and phone, just go to the related alert, scroll down and press test. I recommend to do it on phone, this will also trigger the alarm on watch.
And verify, that you have enabled alarms on watch.

Thank you! That was really helpful. I increased the delay and now the
vibration is longer and more likely to alert me. Thanks for making this
great app!
Lisa
On Fri, Dec 6, 2024, 8:52 AM pachi81 @.***> wrote:

Is the short vibration on watch or phone?
For watch, please go to advanced alarm settings and increase the delay.
You can also test the alarms on watch and phone, just go to the related
alert, scroll down and press test. I recommend to do it on phone, this will
also trigger the alarm on watch.
And verify, that you have enabled alarms on watch.
—
Reply to this email directly, view it on GitHub
https://github.com/pachi81/GlucoDataHandler/issues/186#issuecomment-2523549444,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BNQEE2EGCIVHMM2DCRBU5R32EHB3HAVCNFSM6AAAAABTEZT7OCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRTGU2DSNBUGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Great! What delay are you using?
Maybe I have to increase the default value...

I have it on the full 3 seconds. I need the longer vibration to wake me up
at night. I love having the vibration on my watch at work too, I don't have
to be that weird diabetic with all the beeping all the time. I can be
undercover diabetic, lol!
On Fri, Dec 6, 2024, 3:58 PM pachi81 @.***> wrote:

Great! What delay are you using?
Maybe I have to increase the default value...
—
Reply to this email directly, view it on GitHub
https://github.com/pachi81/GlucoDataHandler/issues/186#issuecomment-2524561569,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BNQEE2AT7DAPBQAFXOCUIPD2EITZNAVCNFSM6AAAAABTEZT7OCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRUGU3DCNJWHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

The delay is not related to vibration itself, it is only the time before starting the vibration. Because the watch notification itself has a short vibration, so I can not start it before. If you want to have a longer vibration, you can use the sound delay of the alarm itself, for example 10s vibration before the sound starts...

Close as it is not an issue on my app. Hopefully Google will fix it.


Case Statements don't work with Mongoid document instances

The Object #=== method is defined as "case equality", being the equality that you'd like to enforce in case statements. In mongoid this is now class-equality. This means you can build a case statement in mongoid around classes, but not regular mongoid document instances.
Take the example below:
class A
  include Mongoid::Document
end

a = A.new
b = A.new

puts a == a # Returns true
puts a == b # Returns false
puts a === b # Returns true as mongoid === only checks class equality

case a
  when b
    puts "Printed because A and B are the same class... This doesn't seem intuitive"
  when a
    puts "This should be printed, but isn't."
end

case a.id
  when b.id
    puts "Not printed as they don't match"
  when a.id
   puts "This is printed as the id === id uses the Ruby Object rules"
end

Having a specific class/subclass match does seem useful if one of the objects is a class -- however, if they are both regular instances it would make more sense to use equals.
Current workaround is to use the id insstead. e.g. "case a.id...".

Good catch here, thanks. Flagging for 2.4.7.


operations on ByteBuffer/IntBuffer

Hi,
I'm interested in decoding integers from an off-heap direct buffer, but I currently see no way of doing this without copying them to the heap. Is there a suggested method of doing this? Is there a fundamental (eg. performance) reason why this is not implemented or is it just to avoid the complexity?
Thanks,
Viktor

Is there a fundamental (eg. performance) reason why this is not implemented or is it just to avoid the complexity?

Does the following help?
Java Buffer types versus native arrays: which is faster?
https://lemire.me/blog/2020/11/30/java-buffer-types-versus-native-arrays-which-is-faster/

If someone would like to propose pull requests leveraging Buffer types, I would be eager to review it. However, one has to be mindful of the performance considerations... and it is not trivial...
The complexity is not much of an issue. One could almost entirely automate the conversion process with a script.
However, it would be easy for a user to get bad performance, through no fault of their own, by misusing the Buffer types. So one would need to add good documentation/guidance.

Good insight. I recently implemented a TSDB using DirectByteBuffer to hold int data, because I didn't want a transfer station (and wanted to save memory), so I did not use native arrays .
However, your opinion makes sense. native arrays  may provide better performance compared to ByteBuffer.


Update update-compliance-delivery-optimization.md

Added known issues section.

You don’t. I think Liza will merge it.
vv
J.O.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Charles Inglis<EMAIL_ADDRESS>Sent: Wednesday, April 11, 2018 2:24:05 PM
To: MicrosoftDocs/windows-itpro-docs
Cc: Jaime Ondrusek; Comment
Subject: Re: [MicrosoftDocs/windows-itpro-docs] Update update-compliance-delivery-optimization.md (#695)
How do I merge via the UX?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fpull%2F695%23issuecomment-380600288&data=02|01|jaimeo%40microsoft.com|f633d69cad084017318208d59ff28f18|72f988bf86f141af91ab2d7cd011db47|1|0|636590786492353613&sdata=cxyHoMoz3pow9gkyh2pIkiiVbJf3Lc4jMZcXpNBBI6E%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANddWJCV2VF-iI6R87u_3fqC1ypCmtjXks5tnnR1gaJpZM4TQw5I&data=02|01|jaimeo%40microsoft.com|f633d69cad084017318208d59ff28f18|72f988bf86f141af91ab2d7cd011db47|1|0|636590786492353613&sdata=hhQ3jso0l7Xm6wvqtQFN1kiurdUAZRTZAp3drF5Ftwc%3D&reserved=0.

You'll see the changes live this afternoon. Thanks!


Bulb doesn't come Online

In my system the Hue Bridge is online and working properly. It recongnise 3 bulbs in 2 rooms.
But the 3 bulb doesn't come online in Simpl Debugger.
They work properly with the Hue App.
Any idea?

Not sure, I typically just control rooms not bulbs myself, I’ll test it this weekend and see what’s what.

On Oct 3, 2019, at 10:56 AM, ErwinDukker<EMAIL_ADDRESS>wrote:

In my system the Hue Bridge is online and working properly. It recongnise 3 bulbs in 2 rooms.
But the 3 bulb doesn't come online in Simpl Debugger.
They work properly with the Hue App.
Any idea?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi Monte,
It was my fault. I made a mistake with the bulbname. Now its working perfectly. Thanks for creating this module.
Best regards,
Erwin Dukker
Van: Monte Thyden<EMAIL_ADDRESS>Verzonden: zaterdag 5 oktober 2019 02:55
Aan: mthyden/PhillipsHueSIMPLSharp<EMAIL_ADDRESS>CC: ErwinDukker<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Onderwerp: Re: [mthyden/PhillipsHueSIMPLSharp] Bulb doesn't come Online (#20)
Not sure, I typically just control rooms not bulbs myself, I’ll test it this weekend and see what’s what.

On Oct 3, 2019, at 10:56 AM, ErwinDukker<EMAIL_ADDRESS><EMAIL_ADDRESS>> wrote:

In my system the Hue Bridge is online and working properly. It recongnise 3 bulbs in 2 rooms.
But the 3 bulb doesn't come online in Simpl Debugger.
They work properly with the Hue App.
Any idea?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/mthyden/PhillipsHueSIMPLSharp/issues/20?email_source=notifications&email_token=ANMH6Q7WTCXFUDZK7IMHVCLQM7QXTA5CNFSM4I5GJLL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEANGTBY#issuecomment-538601863 , or mute the thread https://github.com/notifications/unsubscribe-auth/ANMH6QZXKUX5GWWIDJLZ6RLQM7QXTANCNFSM4I5GJLLQ .  https://github.com/notifications/beacon/ANMH6QZ2I5B2ZPC6RCN3ATDQM7QXTA5CNFSM4I5GJLL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEANGTBY.gif

Awesome, glad its working for you. Apologies your issue completely fell off my radar.

On Oct 13, 2019, at 3:50 AM, ErwinDukker<EMAIL_ADDRESS>wrote:
Hi Monte,
It was my fault. I made a mistake with the bulbname. Now its working perfectly. Thanks for creating this module.
Best regards,
Erwin Dukker
Van: Monte Thyden<EMAIL_ADDRESS>Verzonden: zaterdag 5 oktober 2019 02:55
Aan: mthyden/PhillipsHueSIMPLSharp<EMAIL_ADDRESS>CC: ErwinDukker<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Onderwerp: Re: [mthyden/PhillipsHueSIMPLSharp] Bulb doesn't come Online (#20)
Not sure, I typically just control rooms not bulbs myself, I’ll test it this weekend and see what’s what.

On Oct 3, 2019, at 10:56 AM, ErwinDukker<EMAIL_ADDRESS><EMAIL_ADDRESS>> wrote:

In my system the Hue Bridge is online and working properly. It recongnise 3 bulbs in 2 rooms.
But the 3 bulb doesn't come online in Simpl Debugger.
They work properly with the Hue App.
Any idea?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/mthyden/PhillipsHueSIMPLSharp/issues/20?email_source=notifications&email_token=ANMH6Q7WTCXFUDZK7IMHVCLQM7QXTA5CNFSM4I5GJLL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEANGTBY#issuecomment-538601863 , or mute the thread https://github.com/notifications/unsubscribe-auth/ANMH6QZXKUX5GWWIDJLZ6RLQM7QXTANCNFSM4I5GJLLQ . https://github.com/notifications/beacon/ANMH6QZ2I5B2ZPC6RCN3ATDQM7QXTA5CNFSM4I5GJLL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEANGTBY.gif
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/mthyden/PhillipsHueSIMPLSharp/issues/20?email_source=notifications&email_token=ABEW4UX5TN4PWOVT3J7QV5TQOL4PNA5CNFSM4I5GJLL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBCTUEA#issuecomment-541407760, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABEW4URNJRCLATUIXE6DBBDQOL4PNANCNFSM4I5GJLLQ.


Scan the stack for code attribution

You can set attribution window size in the settings
press ctrl-esc or activate Hugging Face Code: Show Code Attribution from command palette
current cursor position +/- attribution window will be scanned for parts of the code in the stack
Option to open up https://huggingface.co/spaces/bigcode/search will be presented if matches are found

@Stillerman the new endpoint (stack.dataportraits.org) is now live. Let me know when you change over and I'll shut down stack-dev

lgtm except those two comments above. I will approve the PR once they are handled

LGTM ! Merging it for now since the announcement will be soon


⚠ Bofrost.de has degraded performance

In a93713f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1521 ms

Resolved: Bofrost.de performance has improved in 417c0c5.


Menubar items not working after app start

If I switch to another app and then back to TogglDesktop the menubar starts working.

Reported with desk case #207627. Logs #207777


Multipart decoder

Implemented as a "helper" function aleph.http.mutlipart/decode-request to be called with ring request inside a handler. Works both with ring and raw stream handlers. Supports disk-based representation for payloads that exceed specified memory limit to ensure that we don't blow up heap processing large file uploads.

Weirdly enough, I could not replicate the same error locally 🤔


rest list DELETE request now parse filter and window correctly

Allow query parameters (limit, skip, page, sort and filter) to take effect on (list) DELETE requests in the rest package. This resolves #127, although an update to the rest-layer-mongo Storer backend (and potentially others) is required for pagination to take effect when that backend is used.
TODO:

[x] Fix the issue in the route parser
[x] ~Merge fix for rest-layer-mem or~ move rest-layer-mem to rest-layer repo (out of vendor) and deprecate the old repo.
[x] Update tests in affected (failing) test-files to use the external interface (HTTP) for testing.

The latter point is nice to do now as I prefer to use the rest_test.requestTest type for any updated tests for testing the external interface rather than the internal one, and since type is private to the rest_test package, all tests relying on rest package internals must either be updated or moved to a different file. Updating them should yield a somewhat better test coverage. E.g. the filter tests for a DELETE request was passing before this PR, but the filter still did not work in an actual application as they where never parsed from the URL in the case of a DELETE HTTP request.

@torie, do you want to do an initial review on this?

FYI: I have been waiting with merging this until I can find some time to implement (rs/rest-layer-mongo#25) and do a sanity check with that backend. If I can't find that time before the end of this weekend, I will just merge this changes; they will still work with the mongo backend, but there will be a bug when using the window parameters.

Sounds good.


FFmpeg error when running controller.py on windows

i have a problem running one of my bots
when i try to run the controller.py file it says in cmd on windows because i'm running it on windows
09:40:41 - controller.py : RemoTV Controller Starting up 09:40:43 - controller.py : Unknown event type 09:40:43 - controller.py : RemoTV Controller Started 09:40:43 - ffmpeg.py : ffmpeg Video : [] 09:40:44 - watchdog.py : Process FFmpegCameraProcess not running, restarting 09:40:44 - ffmpeg.py : ffmpeg Video : [] 09:40:45 - watchdog.py : Process FFmpegCameraProcess not running, restarting 09:40:45 - ffmpeg.py : ffmpeg Video : [] 09:40:46 - watchdog.py : Process FFmpegCameraProcess not running, restarting 09:40:46 - ffmpeg.py : ffmpeg Video : [] 09:40:47 - watchdog.py : Process FFmpegCameraProcess not running, restarting 09:40:47 - ffmpeg.py : ffmpeg Video : [] 09:40:48 - watchdog.py : Process FFmpegCameraProcess not running, restarting 09:40:48 - ffmpeg.py : ffmpeg Video : [] 09:40:49 - watchdog.py : Process FFmpegCameraProcess not running, restarting 09:40:49 - ffmpeg.py : ffmpeg Video : []
i once got that yesterday when it wasn't working on the pi but i just had to bash kill commands
controller.txt

This is a PEBKAC error.


added optimal_coloring

Use PicoSAT to find the optimal proper coloring of a graph.

Codecov Report
Patch coverage: 92.59% and project coverage change: -1.54% :warning:

Comparison is base (8a43175) 98.94% compared to head (72d4652) 97.41%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
- Coverage   98.94%   97.41%   -1.54%
==========================================
  Files           5        5
  Lines          95      116      +21
==========================================
+ Hits           94      113      +19
- Misses          1        3       +2

Files Changed
Coverage Δ

src/coloring.jl
92.59% <92.59%> (ø)

... and 4 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Hey! I have seen the PR and I'm grateful for the contribution, but I don't have time to review rn, hopefully in the coming weeks. Do you need this urgently?

No hurry.  Take your time.

@dstahlke coming back to your PR now: the immediate goal of this package is to leverage the JuMP ecosystem and remain solver-agnostic. Do you think you might be able to translate your implementation into a JuMP formalism instead of PicoSAT?
I think JuMP is starting to support constraint programming but I don't know how far they've come
https://jump.dev/blog/constraint-programming-update/

It look like ConstraintProgrammingExtensions.jl is not compatible with GraphsOptim.jl.  They have conflicting version requirements for MathOptInterface.

Then I suggest we formulate this as an integer program

My concern with this is that it seems unusably slow.  For example queens_graph(6) takes 16.5 seconds with ILP, as compared to 0.00276 seconds with PicoSAT.

Good point. I will look into constraint programming in JuMP, it would be cool to have example of that too

It seems possible with ConstraintSolver.jl if you wanna try it


Test scripts

I've created examples of how to add App Center Test to Xamarin Android || iOS projects. I'm rusty in bash so I might have missed something.

I compared your approach to my working sample here: https://github.com/King-of-Spades/AppCenter-Samples/blob/master/Xamarin.UITest/FormsGallery/FormsGallery/FormsGallery.Android/appcenter-post-build.sh
I'm not too familiar with regex stuff, so I can't evaluate that at a glance, but if the script works, then it should be fine as far as I know.
This really needs a readme though, because some of your environment variables are custom and not defined within the script itself, so users who don't already understand how scripts work would have no way to interpret them.

@King-of-Spades  @DennisPan @maxxkrakoa
Thanks for the reviews, I've updated the scripts to my best understanding

Did a slight tweak of the 'readme' section at the top to make it more accurate & complete. Everything I can evaluate here looks good to me now (I still don't have much regex knowledge, so I haven't QA'd that section.)


ocamlPackages.vchan: 6.0.1 -> 6.0.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/mirage/ocaml-vchan/releases.
meta.description for ocamlPackages.vchan is: Xen Vchan implementation
meta.homepage for ocamlPackages.vchan is: https://github.com/mirage/ocaml-vchan
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.0.2 with grep in /nix/store/v36n5jjmh05y3mbb90n0bx57nz19p3n0-ocaml5.1.1-vchan-6.0.2
found 6.0.2 in filename of file in /nix/store/v36n5jjmh05y3mbb90n0bx57nz19p3n0-ocaml5.1.1-vchan-6.0.2

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v36n5jjmh05y3mbb90n0bx57nz19p3n0-ocaml5.1.1-vchan-6.0.2 \
  --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 ocamlPackages.vchan https://github.com/r-ryantm/nixpkgs/archive/911e99fd32e478ad3c8bf4a6e9739138f472c76a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/911e99fd32e478ad3c8bf4a6e9739138f472c76a#ocamlPackages.vchan

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v36n5jjmh05y3mbb90n0bx57nz19p3n0-ocaml5.1.1-vchan-6.0.2
ls -la /nix/store/v36n5jjmh05y3mbb90n0bx57nz19p3n0-ocaml5.1.1-vchan-6.0.2/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 --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

3 packages failed to build:

ocamlPackages.cohttp-mirage
ocamlPackages.conduit-mirage
ocamlPackages.irmin-mirage-git

1 package built:

ocamlPackages.vchan

Maintainer pings
cc @sternenseemann for testing.

Result of nixpkgs-review pr 309712 run on aarch64-linux 1

Already on master.


Retry npm registry package resolving when 404s

Related to issue: https://github.com/rstacruz/pnpm/issues/44
Some registries do not support filtering metadata requests by adding a semver suffix to the uri. If this is the case we will get a 404, so we can fetch the metadata for all versions of the package and resolve the correct version on the client side - which is what npm does. If it still doesn't match then we error as per npm.
The specific use-case I'm interested in is when using GemFury to host private packages. GemFury will proxy any requests for non-private packages straight to the npm registry - which means that this approach will not result in two requests for every package when using it.
I'll add some unit tests if you think it's worth continuing with this PR

very closely related to #12, but i do like how this uses it as a fallback. there's also 401 that should be retried in this case:
 curl -v http://registry.npmjs.org/@rstacruz%2Ftap-spec/latest
*   Trying <IP_ADDRESS>...
* Connected to registry.npmjs.org (<IP_ADDRESS>) port 80 (#0)
> GET /@rstacruz%2Ftap-spec/latest HTTP/1.1
> Host: registry.npmjs.org
> User-Agent: curl/7.43.0
> Accept: */*
>
< HTTP/1.1 401 Unauthorized

i've gotta head out today, but i do love this, and i'll try to put in some time tomorrow to iron out any remaining issues.

:+1: very impressive work by the way! :beers:

I've made some changes related to your comments :smile:


enable Sprig functions for embedded Go templates

Templating YAML is tedious and error prone, but we can make it less so by introducing the Sprig library (github.com/Masterminds/sprig).
For example, our embedded YAML templates currently have a meaningful number of space characters carefully placed to render with proper indentation. With Sprig, we can instead say "print this string at indentation level 2."
Before:
// note the invisible two spaces preceding "listeners:"
{{ if .Router.IsPrivate }}#{{ end }}  listeners:

After:
{{ if .Router.IsPrivate }}#{{ end }}{{- "listeners:" | nindent 2 }}

personally, i don't see the need for another dependency for this. agree it's annoying but i'm fine with the annoyance to keep yet another dependency out of our code

I found another way to make the template slightly less error prone:
{{- if and (not .Router.IsFabric) (eq .Router.TunnelerMode "tproxy") (.Router.Edge.LanInterface) }}
      lanIf: {{ .Router.Edge.LanInterface }}
{{- end }}
{{- if and (not .Router.IsFabric) (eq .Router.TunnelerMode "tproxy") (.Router.Edge.DnsSvcIpRange ) }}
      dnsSvcIpRange: {{ .Router.Edge.DnsSvcIpRange }}
{{- end }}

This way, the conditionally-rendered lines are left-justified, so indentation errors are more visible at a glance.

I'll revisit this if I have a specific need for Sprig functions.


DayView appointments are positioned correctly and DayView appointment…

I have applied an Issue #16 fix for the positioning issues of appointments on the DayView component. This PR made a few changes to the DayView component.
First, I needed a way to know the time span that the schedule is visually showing to the user. I added a DayViewStart and DayViewEnd property to the component. Both are currently hardcoded to 8 AM and 8 PM respectively.
Second, I slightly changed how the DayView outputs the CSS grid in HTML. In order to use the grid-row-start CSS property for a schedule item, I needed to make sure the grid cells underneath did not move when a item was positioned over it. To do this, every cell in the grid needed a grid-column property. This would make calculating the starting minute of a schedule item much easier.
Third, the starting position of a schedule item is calculated to the minute and displayed accordingly. The height of the box is now calculated by the duration in minutes and row gaps are accounted for. A schedule item is not allowed to have a height greater than the DayView.
Finally, I have added a graphical representation when a schedule item goes beyond the time span of the DayView. Items that start before 8 AM have an open border and a closed border when starting at 8 AM. Items that end after 8 PM and at 8 PM behave the same way.

All the calculations are currently hardcoded in ems. It may not be as responsive as I would like, but it works. We can make it better later.

Thanks for this fix, really great work!


[Vue warn]: injection "Symbol(addLayer)" not found.

Okay. May you post some context to this problem? How shall I recreate the problem if I don't know anything about your project setup?
The best way to receive fast support here would be to create a minimal setup on codepen or some similar website which shows the error. This would give me the opportunity to look into it.
Otherwise I could just answer:

It works on my machine.

@yuhongliang90 can you please give me more information about the issue? If you don't reply I'll assume that the issue was resolved and closed the thread here.


RFile writes should utilize multiple threads

Is your feature request related to a problem? Please describe.
The write operations to an rfile are serialized. When writing large rfiles in map reduce jobs this can produces very large tales to the jobs. The bottleneck is often compression rather than i/o.
Describe the solution you'd like
Utilizing multiple threads to process multiple blocks in parallel could dramatically improve write performance. Having a dedicated thread to write completed blocks in order would still be necessary, but should be possible. This could be scaled based on available memory for buffering.
Describe alternatives you've considered
Adding pipelines to the existing code could be a smaller lift, and have a big performance improvement as well.

Prototyped having another thread write in this branch.  Wrote this little test program and when running it w/o thread seeing it take ~1300ms and with the separate write thread it takes ~760ms.  Setting up the code to use a separate write thread was done via a manual code modification to the RFileWriter class to make it use the new ThreadedFileSKVWriter class.  When it was using a seperate  write thread ran top and noticed the java process was using 200% CPU.


Add option for image tags

This PR provides an option to replace emoji's with image tags, instead of only span's. Useful when using js-emoji in content editable areas, as users can click and type within span's containing the replaced emoji background.

Coverage remained the same at 90.95% when pulling 0851b62988348a534023128e41a2260be3b7d7e4 on planetary:master into cc077ec691dc0a59ee0e77e19f9e38053616fdde on iamcal:master.

Coverage remained the same at 90.95% when pulling 4f2aacfff105eb9d4c2860ae69a33b22f0d70aa6 on planetary:master into cc077ec691dc0a59ee0e77e19f9e38053616fdde on iamcal:master.

it's not in the docs, but you can do this to force image tags:
var emoji = new EmojiConvertor();
emoji.supports_css = false;

i like adding the codepoints as a data attribute though - i'll add that separately

added in ed21f14a5cf719a03bd8a65fd5ef470b7917d2df
i chose to put the (hex) codepoints in so that you can call replace_unified() multiple times without it expanding the emoji found in the attributes themselves. in the case of multiple codepoints (compound emoji, or with skin variations) it looks like data-codepoints="1234-abcd"


Add a jar target

This builds a JAR file of the compiled classes.

Thanks for merging this so quickly!


Option to configure gelf compression

I think the Docker GELF log driver lacks a log-opt parameter to configure GELF compression.
Docker GELF logs are currently gziped.
Unfortunately, all GELF implementations don't support gzip logs.
For instance, heka GELF input only accepts plain-text gelf logs.
The go-gelf library which Docker relies on actually seems to support:

CompressGzip
CompressZlib
CompressNone

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Looks like this is being worked on in https://github.com/docker/docker/pull/19831


Could not convert string to float: 'N/'

Get the following error when ffpmeg outputs speed=N/A
[Better FFmpeg Process] could not convert string to float: 'N/'
Related to #12

Fixed in PR #22


fix: updated the way we fetch currentLang

changes:

replaced useTranslation with localStorage to fetch the current language

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 85.627%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
5568

Relevant Lines:
6048

💛  - Coveralls


1.4.1 Quilt is broken on 1.20.1

Seems something went wrong with this version. Majority of the animals are missing ( I went from 20 pages of animals in 1.4.0 to 4 pages with 1.4.1) and it doesn't show any of the items usable for breeding except for wheat. 1.4.0 works without issue.
I'm using EMI.

What version of EMI and Quilt are you using?
EMI on Quilt did not get any changes from 1.4.0 to 1.4.1

I’m using EMI 1.1.16, and quilt loader 0.26.4
I do have JEI installed as well to make use of EMI’s compatability with its recipes in case that’s important, for that I’m using <IP_ADDRESS>

emi-1.1.16+1.20.1+fabric
jei-1.20.1-fabric-<IP_ADDRESS>
justenoughbreeding-quilt-1.20.x-1.4.1
qfapi-7.6.0_qsl-6.2.0_fapi-0.92.2_mc-1.20.1
Quilt 0.26.4
https://github.com/user-attachments/assets/cdd6e092-f943-46c9-802c-1d4db247c8b0

Looks like it was a compatibility issue with a certain mod.


opnode: Switch to urfave

Description
This removes OpNodeCmd in favor of a separate configuration and
node struct. Command line flag parsing is also done with urfave/cli.
I did this originally to more easily separate OpNode configuration and node management.
Some things that might need to be sorted out:

Where to place flags (was a seperate package in the l2outputsubmitter)
Geth's versioning note relies on upstream, not our versioning.
Maybe some command line info cleanup? I just went with simple things
[ ] Update docs about running the node

Misc. Changes

Removed the run command
No longer defaults L1 and L2 engine addresses (hard to do with slices and urfave/cli)

Are there any changes in behaviour from a user perspective?

Are there any changes in behaviour from a user perspective?

Yep. I initially put them in the section called Misc. Changes, but I've updated it to ``User Facing Changes`

Codecov Report

Merging #200 (2deaf12) into main (3b44d88) will not change coverage.
The diff coverage is 0.00%.

:exclamation: Current head 2deaf12 differs from pull request most recent head 65c2037. Consider uploading reports for the commit 65c2037 to get more accurate results

@@           Coverage Diff           @@
##             main     #200   +/-   ##
=======================================
  Coverage   48.52%   48.52%
=======================================
  Files          12       12
  Lines         781      781
=======================================
  Hits          379      379
  Misses        369      369
  Partials       33       33

Impacted Files
Coverage Δ

opnode/rollup/derive/execution_payload.go
0.00% <0.00%> (ø)

opnode/rollup/derive/payload_attributes.go
58.51% <0.00%> (ø)

opnode/rollup/driver/step.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


[bug]: Calendar Component Width is Not Flexible

Describe the bug
The Calendar component currently has a static size and does not adjust when the width of its container changes. This leads to a suboptimal user experience, especially in responsive designs where the container size might change dynamically based on the viewport or parent component resizing.

Expected Behavior
The Calendar component should dynamically resize its cells based on the width of its container. This ensures that the calendar remains responsive and visually consistent across different screen sizes and container widths.
Affected component/components
Calendar
How to reproduce

Render the Calendar component inside a resizable container.
Resize the container.
Observe that the size of the calendar cells does not adjust according to the new width of the container.

---- or -----

Render the Calendar component inside a flex container with a width of more than 600px or less than 200px.
Try to adjust the calendar with full width.

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
- OS: Manjaro OS
- Borwser: Brave Browser

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

I found a solution that I think might help you out on stackoverflow, so have a look at this:https://stackoverflow.com/questions/78196878/how-can-i-make-my-calendar-occupy-the-half-of-my-website-using-shadcn

@GiftMothusi Thanks for your reply. i have look on the link you provide, seems a good solution. although i had used #3997 solution.