Images showing the results of the codes (again)

For the French FAQ that, at GUTenberg, we are currently trying to modernize (relying on Sphinx-doc, BTW), it would be helpful to provide the feature of testing online the examples. I managed to make it work (many thanks, BTW!) but, for most of the cases, it would be nice for the reader to immediately see the results of the codes with images attached to each code blocks.
This subject has been extensively discussed about learnlatex.org and it was decided to not provide such a feature for such a LaTeX tutorial. Okay but, IMO, such a feature does make sense for a FAQ.
Would it be possible for latexcgi to provide, in addition to the PDFs obtained on request by compilation of the code blocks, static images automatically and by default attached to each of these code blocks?

@dbitouze that depends... for initial faq examples you know the code and so you could host the image. For dynamically edited code, you need to submit to say you have edited, at which point I generate and show the pdf, I could in principle generate a png from the pdf and show that as well but that seems a bit weird.
I suspect that the way I phrased the above comment means I have not really understood your intended workflow.
What might be possible is for examples to fire the PDF generation on page load so you initially see the output. To be honest I can't remember if that is allowed or not (browsers block some events on page load and force manual interaction for security reasons) If that isn't blocked you can probably do that already without change here. I can check, if that would be useful.
(If generating a pdf is blocked, same will be true of an image, which would come back to the question of when the images are generated and where are they hosted)

You can of course cause the pdfs to be generated on page load, eg
https://texlive.net/run?hello

@dbitouze that depends... for initial faq examples you know the code and so you could host the image. For dynamically edited code, you need to submit to say you have edited, at which point I generate and show the pdf, I could in principle generate a png from the pdf and show that as well but that seems a bit weird.
I suspect that the way I phrased the above comment means I have not really understood your intended workflow.

Indeed, maybe it was not that clear in my mind as well :smile:: probably (already displayed) PDFs instead of images would be preferable.

What might be possible is for examples to fire the PDF generation on page load so you initially see the output. [...]

See below.

You can of course cause the pdfs to be generated on page load, eg
https://texlive.net/run?hello

It's nice but:

I don't see how this works: what is hello? is it a file? how does it work in case of several paragraphs and several codes (complete examples, snippets/document fragment examples, etc.)?
it would solicit the server each time a page is visited and hence would not be energy-friendly: more Internet traffic, more servers requested, etc.

Maybe a better option would be for the PDFs to be generated just once when the pages of the FAQ are built, and regenerated only when the visitor edits the code and submits what he has edited.

I don't see how this works: what is hello?

on that demo it is an arbitrary string, try https://texlive.net/run?bonjour a more realistic example with two inline documents is attached here (as .txt for github, save as .html ad view locally)
run2.txt

Maybe a better option would be for the PDFs to be generated just once when the pages of the FAQ are built, and regenerated only when the visitor edits the code and submits what he has edited.

Well as I say I can't host the images or pre-generated PDF.  There is no login, no session ID and no financing so I can't offer to save and store images for ayone, and as I don't really know if one request is coming from the same session as the last I can't serve images made previously either, by design there are no session cookies or any user data stored.
So anything displayed on the page has to come from your site (you could make images of the initial faq examples, and lose them on edit) or be generated on the fly in a single call to texlive.net so I can generate and return to the same browser.
Of  course this mostly relates to the texlive.net hosted latex, if you clone the whole mechanism and serve your own generated files, you could choose to keep as many images as you have space for, and manage user data so users can or can not see the data of others.
Basically to preserve data between sessions (where regenerating after an edit is a new session) you need to be more like overleaf than texlive.net with user accounts that can host project edits not visible to other users but preserved and restored if that user logs in again.
this is basically a stateless service, so yes stuff gets re-done every time anyone looks at the page, but that's the price for a login free instant run service:-)

I don't see how this works: what is hello?

on that demo it is an arbitrary string, try https://texlive.net/run?bonjour a more realistic example with two inline documents is attached here (as .txt for github, save as .html and view locally)
run2.txt

Does it mean it is enough to add to each of the HTML pages of the FAQ the script:
<script>
    function preloadPDF (){
        var p=0;
        while (editors["pre"+ p]) {
            latexcgi("pre" + p);
            p=p+1;
        }
    }
    window.addEventListener('load', preloadPDF, false);
</script>

to make the PDFs displayed each time a page is visited?

Maybe a better option would be for the PDFs to be generated just once when the pages of the FAQ are built, and regenerated only when the visitor edits the code and submits what he has edited.

Well as I say I can't host the images or pre-generated PDF. There is no login, no session ID and no financing so I can't offer to save and store images for anyone, and as I don't really know if one request is coming from the same session as the last I can't serve images made previously either, by design there are no session cookies or any user data stored.
So anything displayed on the page has to come from your site (you could make images of the initial faq examples, and lose them on edit)

Yes, that's what I had in mind (sorry to have been not precise enough): in order to reduce server solicitation and bandwidth, making and storing locally the images beforehand would be a good solution.

or be generated on the fly in a single call to texlive.net so I can generate and return to the same browser.

Do you mean each time a single page of the FAQ is visited, texlive.net generates all the images? Well, our FAQ currently contains 1295 pages (okay not all of them contain code blocks but many of them contain more than one code block).

Of course this mostly relates to the texlive.net hosted latex, if you clone the whole mechanism and serve your own generated files, you could choose to keep as many images as you have space for, and manage user data so users can or can not see the data of others.

Maybe that's a solution. But in the long run...

Basically to preserve data between sessions (where regenerating after an edit is a new session) you need to be more like overleaf than texlive.net with user accounts that can host project edits not visible to other users but preserved and restored if that user logs in again.

Well, it would be too complicated, I guess.

This is basically a stateless service, so yes stuff gets re-done every time anyone looks at the page, but that's the price for a login free instant run service:-)

So you don't mind if (for the moment) all the PDFs of our FAQ's pages are generated thanks to texlive.net when they are visited?

to make the PDFs displayed each time a page is visited?

yes , but this puts some load on the server, you might consider hosting your own copy of latexcgi in that case.
human readers typically don't generate all the examples and if they do, they do it minutes apart not all at the same time.

Yes, that's what I had in mind (sorry to have been not precise enough): in order to reduce server solicitation and bandwidth, making and storing locally the images beforehand would be a good solution.

I would use PDF not images. That is fine and probably what you should do but doesn't require any change here. If you have a page with initial tex examples and initial pdf display from pdfs on your site, your copy of runlatex.js can easily make the initial pdf be replaced by a generated one.  Either a small edtit  to the javascript or it probably works unchanged if you give the iframe holding your original pdf the same id as the ids used here: it will be updated just as re-clicking the button updates a generated pdf.

Do you mean each time a single page of the FAQ is visited, texlive.net generates all the images?

all the pdf on that page would be generated on page load by the loop you showed (not all the pdf for your whole site)

So you don't mind if (for the moment) all the PDFs of our FAQ's pages are generated thanks to texlive.net when they are visited?

well we'll see..  If all the forums running this grind to a halt, and Stefan's server starts running at maximum load, then we may start to mind. But currently it typically serves 3 to 4 thousand document requests a day, so if this adds a few hundred a day more, it is probably not an issue.  If it adds thousands a day more, then it may be.

@dbitouze  I guess this is closest to what you are looking for
https://davidcarlisle.github.io/latexcgi/test-preload.html
view source or look in git view for the source: all the files are in this commit
e0eb872477251f0a2be1d8678c22d913ceee0585
That shows two tex examples each followed by an iframe showing PDF output. The pdf are pre-generated and statically served by the site hosting the page, so no interaction with texlive.net unless you click on the button
The iframes have id pre0ifr, pre1ifr, ...  so if you do click on a texlive.net button runlatex.js replaces the iframe with one hosting a newly generated pdf.

to make the PDFs displayed each time a page is visited?

yes , but this puts some load on the server, you might consider hosting your own copy of latexcgi in that case. human readers typically don't generate all the examples and if they do, they do it minutes apart not all at the same time.

Okay. Hosting our own copy of latexcgi is probably currently beyond my scope.

Yes, that's what I had in mind (sorry to have been not precise enough): in order to reduce server solicitation and bandwidth, making and storing locally the images beforehand would be a good solution.

I would use PDF not images.

Sorry, I meant PDFs.

That is fine and probably what you should do but doesn't require any change here. If you have a page with initial tex examples and initial pdf display from pdfs on your site, your copy of runlatex.js can easily make the initial pdf be replaced by a generated one. Either a small edtit to the javascript or it probably works unchanged if you give the iframe holding your original pdf the same id as the ids used here: it will be updated just as re-clicking the button updates a generated pdf.

Nice!

Do you mean each time a single page of the FAQ is visited, texlive.net generates all the images?

all the pdf on that page would be generated on page load by the loop you showed (not all the pdf for your whole site)

Okay.

So you don't mind if (for the moment) all the PDFs of our FAQ's pages are generated thanks to texlive.net when they are visited?

well we'll see.. If all the forums running this grind to a halt, and Stefan's server starts running at maximum load, then we may start to mind. But currently it typically serves 3 to 4 thousand document requests a day, so if this adds a few hundred a day more, it is probably not an issue. If it adds thousands a day more, then it may be.

I see. Thanks.

@dbitouze  sorry errors in that pre-loaded pdf example (still had remnants of the example calling latexcgi on page load) corrected at 8ce3c97

@dbitouze sorry errors in that pre-loaded pdf example (still had remnants of the example calling latexcgi on page load) corrected at 8ce3c97

Thanks for digging into this! I'm not sure it will be easy to proceed like this with our current workflow (migrating from Dokuwiki to Sphinx-doc) but I'll think about it.

@dbitouze  well it's there as a backup if generating pdf on page load ends up overloading us at texlive.net


Louis/script-node

Purpose
A simple node for outputting the state of the Python script at the time of execution. This node can be particularly useful for debugging, as it makes it easy to view the script at various points throughout a flow.
A script node can be hooked up at various points in a flow and it will output the state of the script at the time the input node finishes. As in, when a node finishes executing it will trigger the script node, which then returns the script as output.
As shown in the image below, a script node is linked to the hadamard and measure nodes. As each node finishes executing the script node is triggered, which displays the state of the script at that point in the flow. As seen, the first output is the script with only hadamard gates, whereas the second also has the measurements.
The second script node (at the end of the flow) shows how the script can be sent to a file. In this example the script is sent to the file node, which saves the script to a file of the users choosing. This is one of the standard nodes for Node-RED, so all users will be able to easily save the script by default without needing to install nodes from other libraries.

Changes

Add script node (f0172a9c7dee53304cc2ddeed63928bdbb2ea645 & dae178b3620903c349f358fb4d34ad1c33d63692).
Remove script output from simulator node as it is now redundant (8bc78a1862b6b68eeab2f11e56e7b8f229fcd0e1).
Remove execute node as it is now redundant (1e79b4c8c299640c7fb5aee409460f63773ece9b).

Hey Louis,
Great work on the script node, thank you!
Very happy about how simple the 'saving' feature is!
However, I have a concern about the script node.
I never thought about using it as a debug feature by connecting it to various nodes in the circuit but I think that is a great idea, good initiative!
The issue with this design is the fact that we cannot expect a certain number of qubits before running the script code because the node could be connected to any number of nodes, depending on what the user is debugging...
To illustrate the issue, if the node sits at the end of the circuit to output the full script, it will output the script as soon as it receives the 1st qubit, while the rest of the qubits could still have a few more gates to flow through...
In this situation, the user will never be able to output the full script, except if he works out the last qubit to arrive and connects the node to this specific qubit, but I don't think this is user-friendly at all.
At the beginning of the simulator node, there is a whole code block that's quite complex, just for this purpose of 'waiting' for all the qubits in the circuit. This code could be copy-pasted to the script node and should work there.
That would mean losing the debug feature... Not sure if we can include both features at the same time...


Continue - Update ruby version

Because

This pull request links to my prev pull request
I have bumped it in ruby_testing.
This PR

Bump ruby version to 3.3.5 in ruby_testing

Pull Request Requirements

[x] I have thoroughly read and understand The Odin Project Contributing Guide
[x] The title of this PR follows the location of change: brief description of change format, e.g. String spec: Update instructions for clarity
[x] The Because section summarizes the reason for this PR
[x] The This PR section has a bullet point list describing the changes in this PR
[x] If this PR addresses an open issue, it is linked in the Issue section
[x] If this PR includes changes in the spec folder, they are also updated in the corresponding file in the spec_answers folder (with passing tests).

@JoshDevHub I did mention that it ran without a problem a few days. So its fine, no error was returned


[Android] Creating A Custom Spelling Checker Using Spelling Checker Framework in Android with Kotlin

Creating A Custom Spelling Checker Using Spelling Checker Service in Android using Kotlin
Introduction
The Spelling Checker framework improves the text-input experience on Android by helping the user quickly identify and correct spelling errors. When an app uses the spelling checker framework, the user can see a red underline beneath misspelled or unrecognized words so that the user can correct mistakes instantly by choosing a suggestion from a drop-down list.
The Spelling Checker APIs let you create your own spelling checker service with minimal steps. In this article, I will explain to the reader how to create and implement spelling checker service in an android app.
Key takeaways

Understanding what is Spelling Checking Service
How to create own spelling checker using the Spelling Checker Framework
How to implement your own spelling checker to your android app

Article quality
I am going to focus on creating a custom spelling checker service. This will be purely in kotlin programming language. I will use a simple approach which will be beginner friendly for those looking to venture in trying to implement this feature in their app.
References
https://developer.android.google.cn/reference/kotlin/android/service/textservice/SpellCheckerService

Thank you for submitting your topic.
After some careful consideration, it struck us that this topic may be a bit over-saturated throughout other blog sites and official documentation.
We believe this is the best way for students to build a great portfolio (for potential employers) is by building what does not exist and what can provide the most value.
Please feel free to suggest an alternate topic to explore. 🚀


Invalid hook call when using react class component in MDX

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
When you try to declare react components using React.Component in MDX files you end up with an Invalid hook call. error.
Reproducible demo
No response
Steps to reproduce

Setup an empty Docusuarus with latest dependencies

npx create-docusaurus@latest my-website classic --typescript && npx ncu -u && npm i && npm start
Open https://docusaurus.new/stackblitz-ts

Open the docs/tutorial-basics/markdown-features.mdx
Change the file content to

---
sidebar_position: 4
---

import React from 'react'
export class Test extends React.Component {
	render() {
		return (
Title
);
	}
}

<Test />

Open the page in the browser.
Experience the error.

Expected behavior
It should be possible declare React class components in MDX files without errors.
Actual behavior
The page crashes with:
Invalid hook call. Hooks can only be called inside of the body of a function component. This could happen for one of the following reasons:
1. You might have mismatching versions of React and the renderer (such as React DOM)
2. You might be breaking the Rules of Hooks
3. You might have more than one copy of React in the same app
See https://reactjs.org/link/invalid-hook-call for tips about how to debug and fix this problem.
Error: Invalid hook call. Hooks can only be called inside of the body of a function component. This could happen for one of the following reasons:
1. You might have mismatching versions of React and the renderer (such as React DOM)
2. You might be breaking the Rules of Hooks
3. You might have more than one copy of React in the same app
See https://reactjs.org/link/invalid-hook-call for tips about how to debug and fix this problem.
    at Object.throwInvalidHookError (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:16227:9)
    at Object.useContext (webpack-internal:///./node_modules/react/cjs/react.development.js:1619:21)
    at useMDXComponents (webpack-internal:///./node_modules/@mdx-js/react/lib/index.js:47:64)
    at Test.render (webpack-internal:///./docs/tutorial-basics/markdown-features.mdx:59:73)
    at finishClassComponent (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:19747:31)
    at updateClassComponent (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:19693:24)
    at beginWork (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:21606:16)
    at HTMLUnknownElement.callCallback (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:4164:14)
    at Object.invokeGuardedCallbackDev (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:4213:16)
    at invokeGuardedCallback (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:4277:31)

Moving the same class/code into a separate .tsx under src/ and importing it into the MDX works.
Your environment

Public source code: https://docusaurus.new/stackblitz-ts
Public site URL: not relevant
Docusaurus version used: 3.1.1
Environment name and version (e.g. Chrome 89, Node.js 16.4): Chrome 122, Node 20.9
Operating system and version (e.g. Ubuntu 20.04.2 LTS): Windows 11

Self-service

[ ] I'd be willing to fix this bug myself.

Hi, I think this is an MDX issue (or maybe, design decision). The issue can be reproduced without Docusaurus:
// webpack.config.js
module.exports = {
  mode: "development",
  entry: "./index.tsx",
  output: {
    filename: "my-first-webpack.bundle.js",
  },
  module: {
    rules: [
      {
        test: /\.mdx$/,
        use: {
          loader: "@mdx-js/loader",
          options: {
            providerImportSource: "@mdx-js/react",
          },
        },
      },
      {
        test: /\.tsx?$/,
        use: "ts-loader",
        exclude: /node_modules/,
      },
    ],
  },
};

{/* doc.mdx */}
import React from 'react'
export class Test extends React.Component {
	render() {
		return (
Title
);
	}
}

<Test />

// index.tsx
import React from "react";
import ReactDOM from "react-dom";
import Doc from "./doc.mdx";

ReactDOM.render(<Doc />, document.getElementById("root"));

The output produced contains the following:
render() {
    const _components = {
      div: "div",
      ...(0,_mdx_js_react__WEBPACK_IMPORTED_MODULE_2__.useMDXComponents)()
    };
    return (0,react_jsx_dev_runtime__WEBPACK_IMPORTED_MODULE_0__.jsxDEV)(_components.div, {
      children: \"Title\"
// ...

This is expected output per https://mdxjs.com/guides/injecting-components/. Please reach out to them to see if they want to support this.

The output produced contains the following:
render() {
    const _components = {
      div: "div",
      ...(0,_mdx_js_react__WEBPACK_IMPORTED_MODULE_2__.useMDXComponents)()
    };
    return (0,react_jsx_dev_runtime__WEBPACK_IMPORTED_MODULE_0__.jsxDEV)(_components.div, {
      children: \"Title\"
// ...

Yes, using hooks in a class component is not going to work 😅
How do you get this output? This is surprising because the MDX playground

Wondering what could be the compiled output of a React class component for MDX.
Particularly, if the user uses components provided by context:
import React from 'react'

export class Test extends React.Component {
	render() {
		return <SomeComponent/>
	}
}

Should the output be the following?
import React from 'react'

class Test extends React.Component {
  static contextType = MDXContext;

  render() {
    const {SomeComponent} = this.context;
    return <SomeComponent/>
  }
}

I don't see the MDXContext being exported @wooorm so I thought maybe class support has never been implemented?
https://mdxjs.com/packages/react/

I don’t think many people use class components with MDX 😅 Or at least maybe they do but don’t have issues with it? Also, not everyone uses context-based providers, and together, expect components to be injected from them in class methods.
I think the MDX issue here is explained here: https://github.com/mdx-js/mdx/issues/2444#issuecomment-1962422800, a solution is being thought up here: https://github.com/mdx-js/mdx/pull/2445.
MDX also isn’t tied to React. So if the class story improves, we have to be careful to not mess with classes in other frameworks.

I don’t think many people use class components with MDX 😅 Or at least maybe they do but don’t have issues with it? Also, not everyone uses context-based providers, and together, expect components to be injected from them in class methods.

Agree, that's a niche use case 😅

I think the MDX issue here is explained here: mdx-js/mdx#2444 (comment), a solution is being thought up here: mdx-js/mdx#2445.

I'm not sure @remcohaszing PR will fix this.
I don't see it in the Playground (maybe because it's framework agnostic?) but I see this in the dev tools, and there's no "conditional _createMdxContent" call being involved here.
import {Fragment as _Fragment, jsx as _jsx, jsxs as _jsxs} from "react/jsx-runtime";
import {useMDXComponents as _provideComponents} from "@mdx-js/react";
import React from 'react';
export class Test extends React.Component {
  render() {
    const _components = {
      div: "div",
      ..._provideComponents()
    };
    return _jsx(_components.div, {
      children: "Title"
    });
  }
}

We are simply calling a hook in a class component, which doesn't work.

I don’t think many people use class components with MDX

https://github.com/search?q="export+class"+React.Component+language%3Amdx&type=code
There are some false-positives (code snippets instead of components declared in MDX) in the result but it gives a high level insight data it is used here and there. But there are really not many results
I personally find it interesting that things work if you move the react class components into a dedicated file and import it. One would assume that the callstack and execution flows stay the same on rendering.  Maybe this can give some hints on supporting class components within MDX? I assume MDX is injecting some additional code into the classes which cause the problem.
If class components cannot be supported (anymore) it might be worth to emit some user/dev friendly warnings/errors (configurable?). Even though I raised the issue I'm also fine with not having support for class components within MDX. I refactored my code to pull out the more complex components and things are working fine for me now. 😁

This is different from https://github.com/mdx-js/mdx/issues/2444#issuecomment-1962422800.
I didn’t even realize MDX supports useMDXComponents outside of MDXContent / _createMdxContent. I could see both arguments why it should or shouldn’t. Personally I lean towards it shouldn’t, but changing it would be a breaking change.
Docusaurus could replace @mdx-js/react with a custom import provider, or even with a fallback to @mdx-js/react if the user really wants to, like Next.js.

The playground doesn’t use providers (@mdx-js/react). With that turned on, this behavior happens. Without, it doesn’t.

👍

That is to say, regardless of class components or function components, the provider should not be injected for
. Only if someone puts <This> in a function (whether function or class component), that should happen. And I don’t consider fixing it as a breaking change as it’s the always intended behavior.

That doesn't seem to be a breaking change for me either 🤔 This looks like a regular bug fix because anyway, atm a simple class component doesn't even work 😅 I don't see how you can break it more.

Why is the React context API not allowed in classes?

What do you mean?
Hooks are not allowed in class components, so useMDXComponents/_provideComponents() won' work.
But context remains supported with static contextType and this.context.

<This/>

should probably be compiled as:
import React from "react";
import {MDXContext} from "@mdx-js/mdx";

class Test extends React.Component {
  static contextType = MDXContext;

  render() {
    const _components = {
      div: "div",
      ...this.context,
    };
    return _jsx(_components.div, {
      children: _jsx(_components.This),
    });
  }
}

The problem with contextType is that you can only have one context type per class component. (Unless there’s something I don’t know?)

Yes, otherwise you can use <MDXContext.Consumer> 😅

Hooks are not allowed in class components, so useMDXComponents/_provideComponents() won' work.

If you choose to use something other than @mdx-js/react, there could be anything happening in there. It’s more that @mdx-js/react has a hook (React.useContext(MDXContext)) inside it. Could we read from context, that would work in a class, some other way?


Ansible: Etcd Gets Symlinked to Wrong Location

On Ubuntu hosts, etcd gets installed via a github release; however, the symlink portion of the play links to /usr/local/bin. This seems to be incorrect later in the run where flannel tries to look for etcdctl to exist at /usr/bin, as seen here. I suspect that the github release play simply needs to be updated to link to /usr/bin instead (this seems to fix it in my environment). I'll take this on later this week if someone can assign to me.

Assigned to myself as I can't assign it to you for some reason. But rest assured, I won't be fixing it, I'll be anxiously awaiting to commit your fix!


API docs url is empty and endpoints not accessible (404)

I think this might be more of a setup problem however the api-docs endpoint offers only an empty map and the endpoints for the documentation result in a 404.
The logs for my app show that the controllers are being scanned.
My configuration class:
package com.myorg.enterprise.docs;

import com.google.common.base.Predicate;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;
import org.springframework.web.servlet.config.annotation.EnableWebMvc;
import springfox.documentation.builders.PathSelectors;
import springfox.documentation.service.ApiInfo;
import springfox.documentation.spi.DocumentationType;
import springfox.documentation.spring.web.plugins.Docket;
import springfox.documentation.swagger2.annotations.EnableSwagger2;

/**
 * A Configuration file that allows the generation of Swagger
 */
@Configuration
@EnableWebMvc
@EnableSwagger2
public class MyOrgSwaggerConfig {

    /**
     * Every SwaggerSpringMvcPlugin bean is picked up by the swagger-mvc framework - allowing for multiple
     * swagger groups i.e. same code base multiple swagger resource listings.
     *
     * @return SwaggerSpringMvcPlugin
     */
    @Bean
    public Docket customImplementation() {

        return new Docket(DocumentationType.SWAGGER_2)
                .apiInfo(getApiInfo())
                .select()
                .paths(paths())
                .build()
                .pathMapping("/");
    }

    private Predicate<String> paths() {
        return PathSelectors.any();
    }

    /**
     * A method that returns the API Info
     * @return ApiInfo The Information including description
     */
    public ApiInfo getApiInfo() {
        return  new ApiInfo(
                "My REST API",
                "This REST API allows the user to manage domain objects",
                "1e",
                "",
<EMAIL_ADDRESS>                null,
                null
        );
    }
}

The versions I'm using are:
        <!-- Swagger -->
        <dependency>
            <groupId>io.springfox</groupId>
            <artifactId>springfox-swagger-ui</artifactId>
            <version>2.1.1</version>
        </dependency>

        <dependency>
            <groupId>io.springfox</groupId>
            <artifactId>springfox-swagger2</artifactId>
            <version>2.1.1</version>
        </dependency>

        <dependency>
            <groupId>io.springfox</groupId>
            <artifactId>springfox-spring-web</artifactId>
            <version>2.1.1</version>
        </dependency>

        <dependency>
            <groupId>io.swagger</groupId>
            <artifactId>swagger-annotations</artifactId>
            <version>1.5.0</version>
        </dependency>

        <dependency>
            <groupId>io.springfox</groupId>
            <artifactId>springfox-swagger-common</artifactId>
            <version>2.1.1</version>
        </dependency>
        <!-- End of Swagger -->

The application is running as ROOT context so there shouldn't be any problems with
My logs contain:
Created resource listing Path: /default/my-controller

However this is not accessible on http://localhost:8080/v2/api-docs/default/my-controller as expected.
Accessing http://localhost:8080/v2/api-docs returns:
{}

Please note I have had this working using the old com.mangofactory version and followed the migration documentation
Finally the swagger-ui.html is trying to access undefined:
http://localhost:8080undefined/

Any help or advice would be gratefully received

This was due to me using the GSON library to generate the Json. I switched on the excludeFieldsWithoutExposeAnnotation() which meant that none of the objects would produce any data. To get around this I created several serialisers for the following classes:
.registerTypeAdapter(springfox.documentation.service.ApiListing.class, new SwaggerApiListingJsonSerializer())
                .registerTypeAdapter(springfox.documentation.spring.web.json.Json.class, new SwaggerJsonSerializer())
                .registerTypeAdapter(springfox.documentation.swagger.web.SwaggerResource.class, new SwaggerResourceSerializer())
                .registerTypeAdapter(springfox.documentation.service.ResourceListing.class, new SwaggerResourceListingJsonSerializer())
                .registerTypeAdapter(springfox.documentation.swagger.web.SwaggerResource.class, new SwaggerResourceSerializer())
                .registerTypeAdapter(springfox.documentation.swagger.web.SecurityConfiguration.class, new SwaggerSecurityConfigurationSerializer())
                .registerTypeAdapter(springfox.documentation.swagger.web.UiConfiguration.class, new SwaggerUiConfigurationSerializer());

Ah great! Thanks for reporting your solution back :metal: I'll update the docs to reflect your fix.
In addition since we're not really supporting gson, would you be able to create an adapter for gson such that it always works if gson is found in the classpath? I might attempt it but since your more intimate it would be great if you could shepherd that! I'd be happy to collaborate 🍻

The solution for Gson use in Swagger looks intriguing. However, when I try to use the code, node of the Swagger...Serializer classes are found. I included all the dependencies that were given in the original posting. Is there a particular dependency I need to add to get this to work?


Use something else other than name equality to match Review and Stall

This would be useful to a food critic who visits stalls which have no name and they hence have to write something generic.

We have model Stall storing the Review, we have JsonAdaptedStall storing the JsonAdaptedReview, but our AddressBook stores both Stall and Review, which might be an issue with this.
As this is not a feature, rather a code enhancement - I think it can be postponed to v1.4.


add workflows enterprise endpoints

Codecov Report

Merging #47 (0ef98a8) into main (f4bdfcd) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
+ Coverage   98.62%   98.64%   +0.02%
==========================================
  Files          95       97       +2
  Lines        1953     1981      +28
  Branches      242      246       +4
==========================================
+ Hits         1926     1954      +28
  Misses         21       21
  Partials        6        6

Impacted Files
Coverage Δ

crowdin_api/api_resources/__init__.py
100.00% <100.00%> (ø)

crowdin_api/api_resources/workflows/__init__.py
100.00% <100.00%> (ø)

crowdin_api/api_resources/workflows/resource.py
100.00% <100.00%> (ø)

crowdin_api/client.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


refactor: EOA validate and execute receives a separate encoded tx calldata and signature (in tx_info)

Feature Request
Describe the Feature Request

Currently the EOA receives a payload of eth_sendRawTransaction -> it has to: RLP::decode, extract sig, RLP::encode the calldata without sig, hash initial encoded payload, check signature, then validate.

Describe Preferred Solution
### Tasks
- [ ] Pretend that the payload received in KKT0 is the RLP-encoded calldata of an Ethereum transaction (without the signature)
- [ ] Hash that RLP-encoded calldata, and use the tx_info.signature to do the ecdsa check
- [ ] Decode the RLP-encoded calldata, perform checks (nonce and chain_id)
- [ ] Validate
- [ ] In execute, RLP-decode and call Kakarot eth_send_transaction

Describe Alternatives

Related Code

Additional Context
Check out Kakarot SSJ implementation:
https://github.com/kkrt-labs/kakarot-ssj/blob/main/crates/contracts/src/eoa.cairo
If the feature request is approved, would you be willing to submit a PR? Yes
/ No (Help can be provided if you need assistance submitting a PR)

Oops duplicated here
https://github.com/kkrt-labs/kakarot/issues/831


[SPARK-39266][CORE] Cleanup unused spark.rpc.numRetries and spark.rpc.retry.wait configs

What changes were proposed in this pull request?
This PR cleans up the spark.rpc.numRetries and spark.rpc.retry.wait configs, both of which are unused.
Why are the changes needed?
Since SPARK-19450 / #16790 in Spark 2.2.0, both of these configurations are unused and setting them has no effect. Marking the configs as deprecated and cleaning them up from the docs helps to avoid user confusion.
In addition, this cleanup slightly improves the performance of constructing RpcEndpointRefs because it removes two unused fields that were initialized by reading the deprecated configs.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
n/a

Merged to master. Thank you all!


OS X: find_package() call for hunter packages prefers /System/Library/Frameworks over hunter builds

I was testing a FindGPUImage.cmake file for a hunter GPUImage dependency:
cmake/Hunter/config.cmake
hunter_config(GPUImage VERSION 0.1.6-hunter-1)

CMakeLists.txt
hunter_add_package(GPUImage)
find_package(GPUImage)
message("GPUIMAGE: ${GPUIMAGE_INCLUDE_DIR} ${GPUIMAGE_LIBRARY} ${GPUIMAGE_FOUND}")

When I run this:
build.py --toolchain libcxx --verbose --clear

it prints:
GPUIMAGE: /System/Library/Frameworks/GPUImage.framework /System/Library/Frameworks/gpuimage.framework TRUE

and if I remove the /System/Library/Frameworks/GPUImage.framework
sudo mv /System/Library/Frameworks/GPUImage.framework /tmp;
build.py --toolchain libcxx --verbose --clear

it finds the hunter library:
GPUIMAGE: /Users/dhirvonen/devel/hunter/_Base/xxxxxxx/7bc3715/b2b5fbe/Install/include /Users/dhirvonen/devel/hunter/_Base/xxxxxxx/7bc3715/b2b5fbe/Install/lib/libgpuimage.a TRUE

It would seem we always want the hunter builds to be given preference, but I'm not familiar enough with all of this to know if this is something that should be handled in FindGPUImage.cmake or within the hunter framework itself.  (The FindGPUImage.cmake is using the stock cmake find_library call.)  Posting this as an issue in case it is actually something hunter needs to adapt to.  I believe this is related to some problems I hit earlier where a GStreamer framework was being used to satisfy some hunter dependencies.
This is using a simple FindGPUImage.cmake file:
# Find GPUImage
#
#  GPUIMAGE_INCLUDE_DIR - where to find GPUImage/logging.h, etc.
#  GPUIMAGE_LIBRARY     - List of libraries when using libglog.
#  GPUIMAGE_FOUND       - True if libglog found.
#
# Lineage: 4/21/2015
#   Modified from https://github.com/facebook/hhvm/blob/master/CMake/FindGlog.cmake

IF (GPUIMAGE_INCLUDE_DIR)
  # Already in cache, be silent
  SET(GPUIMAGE_FIND_QUIETLY TRUE)
ENDIF ()

FIND_PATH(GPUIMAGE_INCLUDE_DIR GPUImage/GPUImage.h)

FIND_LIBRARY(GPUIMAGE_LIBRARY gpuimage)

# handle the QUIETLY and REQUIRED arguments and set Libmemcached_FOUND to TRUE if
# all listed variables are TRUE
INCLUDE(FindPackageHandleStandardArgs)
FIND_PACKAGE_HANDLE_STANDARD_ARGS(GPUIMAGE DEFAULT_MSG GPUIMAGE_LIBRARY GPUIMAGE_INCLUDE_DIR)

MARK_AS_ADVANCED(GPUIMAGE_LIBRARY GPUIMAGE_INCLUDE_DIR)

Hunter version of GPUImage is config-mode package. I've added wiki example: https://github.com/ruslo/hunter/wiki/pkg.gpuimage

and if I remove the /System/Library/Frameworks/GPUImage.framework ... it finds the hunter library

Yep, this is because Find<...>.cmake has higher priority then <...>Config.cmake

I see.  I was missing the CONFIG in: find_package(GPUImage CONFIG)
On Tue, Apr 21, 2015 at 3:50 PM, Ruslan Baratov<EMAIL_ADDRESS>wrote:

Hunter version of GPUImage is config-mode package. I've added wiki
example: https://github.com/ruslo/hunter/wiki/pkg.gpuimage
and if I remove the /System/Library/Frameworks/GPUImage.framework ... it
finds the hunter library
Yep, this is because Find<...>.cmake has higher priority then
<...>Config.cmake
—
Reply to this email directly or view it on GitHub
https://github.com/ruslo/hunter/issues/83#issuecomment-94917871.


OpenStreetMap insted of Mapbox

Hello what do you think to use OpenStreet map instead of mapbox

Mapbox maps are based on OpenStreetMap data and their client SDK (at least the older version that we are currently using) is open source. So what exactly are you proposing and what benefit would that bring?

Using a different client library?
Using a different OpenStreetMap tile provider?
Using OpenStreetMap data for charging stations? (this is currently in development, see #97)
Using a different service for geocoding queries (search field)?

I already wrote some comments on #36 and #141 that might help understand why we are currently using Mapbox, which other options are available and why some of them are not currently feasible.

I hope you can understand now why I opened this issue
And thank you for your great work

Hm okay, yeah, so this tool seems to just detect „trackers“ based on package names, not checking if tracking code is actually included and enabled. As stated in #141 all telemetry is disabled, there are just some concerns about the SKU token which is necessary for billing purposes. So I‘ll close this as a duplicate of #141.


Buffers view should show unloaded buffers as well

The plugin does not display open buffers after starting a session.

NeoTree buffers. (Left)
Telescope buffers. (Front float)
Open nvim session, run any this command
(
:NeoTreeShow buffers
:NeoTreeFocus buffers
:NeoTreeFloat buffers
)
only 1 buffer.
Run
:Telescope buffers
you can see that there are many more buffers.

I see what the issue is. I am only showing loaded buffers in this list, while Telescope is including all listed buffers. I think that whether or not to show unloaded buffers should be a configuration option. I'll put this in as an enhancement and work on it soon.
I am going to make the current behavior the default and let you opt-in to showing unloaded buffers as well.

Thanks!


Added a loaded event to the main database class

If this isn't helpful, I'd love to know how I can accomplish what I'm looking for. This allows me to use the Loki constructor with autoload and use an event listener to populate any collections as appropriate. This is especially helpful when the creation of the database and the loading of the collection might be separated for some reason.
db = new loki(dbFile,
	{
		autoload: true,
		autosave: true,
		autosaveInterval: 10000,
	});

db.on('loaded', function() {
	var reactions = db.getCollection('reactions');
});

While we already have an option to pass and autoloadCallback function within the loki constructor, this style may also be useful.  Thanks!

cheers @KodeAndGame  and @obeliskos  for merging.


Added Muted Topic screen in Setting Detail Screen.

Updated.
Fixed travis build.

rebased 👍

Rebased 👍
ready to go in 🏃

Closing this, but feel free to move some of the work to the new topic screen.


Create credential-verification-requirements.md

Here is a basic summary on what to do on verification. I'm a bit time-constraint right now, so I couldn't spend as much time on it as I wished. Feel free to extent

Some light edits


Add testing script, set page size to 0, add cql definition file

Fixes https://github.com/uber/jaeger/issues/81 and https://github.com/uber/jaeger/issues/82

Coverage increased (+0.07%) to 98.361% when pulling 8811f970a837c6596257a8539f697d1bc310a4ac on rooz/cassandra-cql-and-bucket-fix into 5777a94a6fa1dfb859a19b9be9d78c5f57cda860 on master.

Coverage increased (+0.07%) to 98.361% when pulling 8811f970a837c6596257a8539f697d1bc310a4ac on rooz/cassandra-cql-and-bucket-fix into 5777a94a6fa1dfb859a19b9be9d78c5f57cda860 on master.

Coverage remained the same at 98.362% when pulling 43e4b436bdcd18fa55da0d18e4b45660fff7c3c6 on rooz/cassandra-cql-and-bucket-fix into 7bb89a21d71da172dfee26e5b788e44dd4e2542f on master.


xds: include port in VirtualHost domain matching

This matches the Envoy behavior.

Known issues: #21746 #22222 #13379


If you put a URL escaped in the config YAML, it gets unescaped when making request to the real server

/document/10.1007%2Fs00414-006-0114-x
/document/10.1007/s00414-006-0114-x

Also true when using the fake server
Chriss-MacBook-Pro:mockingjay-server qui$ curl -v http://localhost:9090/document/10.1007%2Fs00414-006-0114-x
* Hostname was NOT found in DNS cache
*   Trying ::1...
* Connected to localhost (::1) port 9090 (#0)
> GET /document/10.1007%2Fs00414-006-0114-x HTTP/1.1
> User-Agent: curl/7.37.1
> Host: localhost:9090
> Accept: */*
>
< HTTP/1.1 404 Not Found
< Date: Mon, 18 May 2015 12:11:44 GMT
< Content-Length: 526
< Content-Type: text/plain; charset=utf-8
<
message: Endpoint not found
request:
  uri: /document/10.1007/s00414-006-0114-x
  method: GET
  headers:
    Accept: '*/*'
    User-Agent: curl/7.37.1
  body: ""
endpointsavailable:
- name: Some article on content api
  request:
    uri: /document/10.1007%2Fs00414-006-0114-x
    method: GET
    headers: {}
    body: ""
  response:
    code: 200
    body: <Publisher><meta:Info xmlns:meta="http://www.springer.com/app/meta"><meta:Date>2006-09-01</meta:Date></meta:Info></Publisher>
    headers:
      content-type: text/json
* Connection #0 to host localhost left intact

Example yaml
---
 - name: Some article on content api
   request:
     uri: /document/10.1007%2Fs00414-006-0114-x
     method: GET
   response:
     code: 200
     body: '<Publisher><meta:Info xmlns:meta="http://www.springer.com/app/meta"><meta:Date>2006-09-01</meta:Date></meta:Info></Publisher>'
     headers:
       content-type: text/json

This is fixed for the fake server but not when checking compatability


nodePackages.dockerfile-language-server-nodejs: fix build

Description of changes
When trying to use the dockerfile-language-server, it fails to start with message "Error: Cannot find module '../lib/server".
In package.json there is an additional step to generate this file : "prepublishOnly": "tsc -p ./src".
I don't know a lot JS, so their might be a clever way to fix this issue, but adding tsc -p ./src to the build command solve it for me (tested with Helix editor)
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

1 package built:

dockerfile-language-server-nodejs

@Net-Mist Do you want to add yourself as maintainer?

@Net-Mist Do you want to add yourself as maintainer?

@dotlambda with pleasure!

LGTM, maybe split the commit adding the new maintainer? Quoting CONTRIBUTING.md:

When adding yourself as maintainer, in the same pull request, make a separate commit with the message maintainers: add <handle>. Add the commit before those making changes to the package or module. See Nixpkgs Maintainers for details.

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

1 package built:

dockerfile-language-server-nodejs

@erdnaxe thanks for the tip. I've split the commit into 2

maintainers: add <handle>

That's for when you add yourself to maintainers/maintainer-list.nix

@dotlambda So should I revert to a single commit ?

Arg, my bad. Thanks for correcting me @dotlambda and sorry @Net-Mist !

@erdnaxe no problem,
revert to a single commit

Do you think I should also add
passthru.updateScript = nix-update-script { };

Do you think I should also add
passthru.updateScript = nix-update-script { };

I don't think that's needed. nix-update should work out of the box.


Attempted workaround for temporarily faulty CI tioga runner

Summary

This PR is an attempted workaround for the CI runner of LC's tioga
LC advised explicitly pointing at the tioga10 runner since something appears to be wrong with the tioga11 runner.

Grrfff. Still not working.

Grrfff. Still not working.

It's actually a different sort of error though.  This one is running the CI, and the build is getting stuck at building/linking something at the end. I think spin_bvh_example (?)


Secure the maker's API routes with basic authentication

Resolves #76.
We choose basic authentication because it is simple.
Basic auth allows us to not code any particular login API and instead just trigger the browser's built-in authentication dialog via the WWW-Authenticate header of the 401 Unauthorized response. It looks like this:

The password is derived from the maker's seed.
The username is always maker.
The implementation has one ceveat: Basic authentication is vulnerable to MITM attacks unless some form of transport encryption is in place. I suggest all securing the maker's API additionally with TLS. However, I think we can easily ship this as a feature on top!

Would it not make sense to also have authentication for the taker?

We can trivially reuse this for the taker as well. I thought the taker is most likely run locally and on-demand whereas the maker is probably on a server somewhere due to the need for an open port!


SAS URLS

I have searched but have not been able to find an explanation as to why an array of SAS URLS is returned, May you kindly explain why multiple SAS URLS are returned, every tutorial and example I find only makes use of the first URL.

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

ID: d08723a0-f42c-7a2f-efff-828d745e4ebd
Version Independent ID: 2df45dba-fab1-5cf7-8cc3-212d9e7764d9
Content: Upload files into an Azure Media Services v3 account using REST
Content Source: articles/media-services/latest/upload-files-rest-how-to.md
Service: media-services
GitHub Login: @Juliako
Microsoft Alias: juliako

@tbtmuse Thank you for the feedback. We are actively investigating and will get back to you soon.

@tbtmuse , If the question is specifically why an array is returned instead of a single item, it is because you could potentially have more than one SAS URL to the same asset.  The API in question, listContainerSas, is used for both upload and download.  While not common for uploads, it is moderately common for downloads.

@DashleenBhandari-MSFT, thank you for the explanation as to why an array is returned, my follow up question would then be, what is the purpose of the second url that is returned when calling listContainerSas immediately after creating a new "input" Asset.
The MediaServices Rest API v3 returns an array of two SAS URLS and the documentation just says use the first one.
I'm just trying to understand the API  a bit more.

There are two SAS URLs returned (why an array is returned as David  mentioned).
We return multiple SAS URLs as there are two storage account keys for each storage account. A storage account has two keys because it allows for seamless rotation  of storage account keys (e.g. change one while using the other then start using the new key and rotate the other key). The 1st SAS URL represents storage key1 and second one storage key2.

@tbtmuse , There are two SAS URLs returned.
We return multiple SAS URLs as there are two storage account keys for each storage account. A storage account has two keys because it allows for seamless rotation  of storage account keys (e.g. change one while using the other then start using the new key and rotate the other key). The 1st SAS URL represents storage key1 and second one storage key2.
We are checking internally to update the document.

@DashleenBhandari-MSFT, thank you for the explanation, it all makes sense now that you have explained it.
Yes, that should certainly be in the documentation as that information isn't available anywhere.

@tbtmuse , Thanks for your response. Yes, we are in process of updating the document. I shall go ahead and close the issue for now. Feel free to re-open the issue if you need further help.


feat(ai): add support for TechDocs

This PR adds support for embedding and querying TechDocs to the RAG AI plugin, heavily inspired by Backstage's search functionality.

:heavy_check_mark: Checklist

[x] Added changeset (run yarn changeset in the root)
[x] Screenshots of before and after attached (for UI changes)

:ship:


missing crypto functions in all_default_functions?

Describe the bug
Hello, I'am Arroyo (which is Distributed stream processing engine based on DataFusion) user. Arroyo upgrade datafusion version latest (i.e. 0.40) recently. But I meet below error
2024-08-06T08:28:28.674123Z DEBUG datafusion_functions: /home/zhuliquan/.cargo/registry/src/rsproxy.cn-0dccff568467c15b/datafusion-functions-40.0.0/src/lib.rs:179: Overwrite existing UDF: contains
test arrow::expression_match::test::test_execuator_process_batch ... FAILED

failures:

---- arrow::expression_match::test::test_execuator_process_batch stdout ----
thread 'arrow::expression_match::test::test_execuator_process_batch' panicked at crates/arroyo-worker/src/arrow/expression_match.rs:64:88:
called `Result::unwrap()` on an `Err` value: Execution error: UDF sha256 not found

according to error info, sha256 udf is missing and not registry. Meanwhile, I found other crypto udfs are not available. It's seems that missing some builtin udfs in all_default_functions
https://github.com/apache/datafusion/blob/1c98e6e640090600d36ce51f87919fea7b6eeed2/datafusion/functions/src/lib.rs#L161-L172
I click crypto::functions, then jump to below code (

), but others jump to own package (

).
https://github.com/apache/datafusion/blob/1c98e6e640090600d36ce51f87919fea7b6eeed2/datafusion/functions/src/lib.rs#L127-L129
To Reproduce
No response
Expected behavior
No response
Additional context
No response

Sorry, It's not Datafusion issue, it's other problem.


Add shell.nix for Nix Env Selector 🙏🏻

Hi, thanks for this template! I'm using vscode with the Nix Env Selector and it does not seem to pick up correctly HLS by default, I guess if we had some shell.nix to point the extension to that had all the libs that we already built in flake.nix we could make the integration with this editor quite seamless, but unfortunately I do not know enough Nix to do it myself 😢.
Thanks in advance and do let me know if you think this might be a useful addition to the template! 🙏🏻

@kutyel, This is a good idea! I will add this functionality, and I will ask you to try it once it is completed.
Thank you for trying this!

@kutyel I'm sorry I still haven't gotten to this. If you figured this out for your project, please consider submitting a pull request here!

Nope, although Nix works my editor setup does not work with Hakyll and HLS so I'm a bit blind while touching Haskell in the project unfortunately 😢


centerpiece: 1.0.0 -> 1.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/friedow/centerpiece/releases.
meta.description for centerpiece is: Your trusty omnibox search
meta.homepage for centerpiece is: https://github.com/friedow/centerpiece
Updates performed

Rust 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 1.1.0 with grep in /nix/store/sgnb30p0cdpyfpv14fk3mhj73sziagg5-centerpiece-1.1.0
found 1.1.0 in filename of file in /nix/store/sgnb30p0cdpyfpv14fk3mhj73sziagg5-centerpiece-1.1.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

centerpiece

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sgnb30p0cdpyfpv14fk3mhj73sziagg5-centerpiece-1.1.0 \
  --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 centerpiece https://github.com/r-ryantm/nixpkgs/archive/9946750788b2ce80279f1b75aaca03377787701e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/9946750788b2ce80279f1b75aaca03377787701e#centerpiece

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sgnb30p0cdpyfpv14fk3mhj73sziagg5-centerpiece-1.1.0
ls -la /nix/store/sgnb30p0cdpyfpv14fk3mhj73sziagg5-centerpiece-1.1.0/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

1 package built:

centerpiece

Maintainer pings
cc @a-kenji for testing.

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

1 package built:

centerpiece

Both this and the previous version of the package segfault for me :/

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

1 package built:

centerpiece

@Scrumplex
We are aware of some constellations not working.
It is only supporting wayland systems.
Though there are also reports of some wayland systems not loading libraries correctly.
We believe this might have to do with the underlying library.

I just checked the backtrace and read "MangoHud". It is enabled by default on my system, and setting MANGOHUD=0 to disable the Vulkan layer fixes the segfault for me.

Thank you for checking, this is valuable information!


Add copy artifacts command

The command needs to be added to https://github.com/stuartwdouglas/jvm-build-service/blob/89ec977c3886ef61bfeccf695d2376622fc82d6f/java-components/build-request-processor/src/main/java/com/redhat/hacbs/container/EntryPoint.java#L20

/retest

/retest


Add timestamp and biddercode to event URL - refactored

See parent PR https://github.com/rubicon-project/prebid-server-java/pull/635

And Please refactor ExchangeService method placement according to workflow


MkDocs documentation

Since the pipeline has grown enough to have options and alternative target rules, it has reached the point where I should start documenting those.
I started to put together a MkDocs documentation to eventually hook to Read The Docs.
As far as I understand, once we are happy with the initial contents of this it is a matter of configuring from their side to set up the hooks so the HTML re-renders each time the docs files change.
I will also move part of the content that is now on the README to the Docs, once this set up works and the documentation renders properly.

This is an initial MkDocs documentation for minute so now recent changes and functionalities are addressed. This will probably be subject to more changes soon, and once I merge and fix the hook on read the docs to point to the main branch I will also update the README to point to the ReadTheDocs site.
Since this does not make changes on the actual codebase, I will just go ahead and merge it.


NullPointerException using Select.from

I just received a NullPointerException while building the Query via Select.from:
Select.from(MyModel.class).where(Condition.prop("myprop").notEq(null)).orderBy("ID").list();

Did I get the usage wrong from the (poor) docs? What's the correct equivalent to WHERE prop <> null?

Null isn't supported in queries yet. There's a pending pull request that'll go into 1.4.0, #227. I'll close this issue when #227 gets into master.


[firebase_auth] on ios not using getTokenId refresh bool

Refresh bool is hard coded to YES for ios. Android uses the bool as expected.
https://github.com/flutter/plugins/blob/master/packages/firebase_auth/ios/Classes/FirebaseAuthPlugin.m
  } else if ([@"getIdToken" isEqualToString:call.method]) {
    [[FIRAuth auth].currentUser
        getIDTokenForcingRefresh:YES
                      completion:^(NSString *_Nullable token, NSError *_Nullable error) {
                        result(error != nil ? error.flutterError : token);
                      }];

@bjon
The issue at https://github.com/flutter/flutter/issues/16807 has been closed and moved here. Future collaboration on this issue will be done here.

Hi @bjon
If you are still experiencing this issue
with the latest version of Flutter and FlutterFire plugin
can you please provide your updated  flutter doctor -v
and your and your flutter run --verbose/flutter build --verbose ?
Also, to better address the issue, would be helpful
if you could post a self contained app on github
or the steps to reproduce it.
Thank you

I'm facing the same issue:
getIdToken(refresh: false will not refresh token on Android, but will refresh for iOS.
Here is a simple fix for the issue:
https://github.com/FirebaseExtended/flutterfire/pull/1334

Thank you for your contribution @AlexZaiats
I see that your PR has been merged,
may I close this issue?

@iapicca sure, thanks


Failed to Bypass Cloudflare on Built-in NHentai plugin.

PLEASE READ THIS
I acknowledge that:

I have updated to the latest version of the app (stable is v8.7.0)
I have updated all extensions
If this is an issue with an extension, that I should be opening an issue in https://github.com/inorichi/tachiyomi-extensions

DELETE THIS SECTION IF YOU HAVE READ AND ACKNOWLEDGED IT

Device information

Tachiyomi version: v8.7.1-AZ
Android version: 12
Device: Samsung Galaxy A5 2017

Steps to reproduce

Open App
Try to Open NHentai

Expected behavior
You get to see the ahem stuff, Read manga
Actual behavior
java.lang.exception Failed to Bypass cloudflare
Other details
Additional details and attachments.
https://nhentai.net/g/125582/
This is what im trying to read if that's important
Device is running on a custom rom (LineageOS 19.1)
I followed troubleshotting guides, didn't work
Might be because of User agent, Other Tachyforks have made people able to change User agents which fixed this.

#46


React Native 0.47 no longer supports createJSModules

React Native 0.47 removed createJSModules, so react-native-tcp will no longer compile on Android. See:
https://github.com/facebook/react-native/releases/tag/v0.47.0
I copied the fix from react-native-device-info to remove the @Override declaration, which allows it to work on RN 0.47, but I haven't tested it on older versions.

I created a PR to address this:
https://github.com/PeelTechnologies/react-native-tcp/pull/57

+1 for this. please merge and release it, thanks.

when to release the new version?

In the meantime, you can do:
"react-native-tcp": "git+https://github.com/PeelTechnologies/react-native-tcp.git",
It would be really nice to get a new release though.

sorry for the delay. 3.3.0 has been pushed to npm


Question: Run Generate Release Notes Build Task anywhere

I would like to generate Release Notes not on TFS but somewhere else ( another Windows machine).
I there a way I can run the code of this "Generate Release Notes Build Task"  somehow outside of TFS?
Regards Stefan

It would take some work on your part, but yes. You have a couple of choices

You can us the older Powershell version that uses my own logic to create the document
Or you can use the newer Node version that uses the same calls as the Microsoft UI

In both case you would need to edit the .ps1/.ts file that is the tasks entry point to get the parameters via the command line or config file as opposed to from the build agent via the agent API.
Also you would have to provide a PAT token to access the Azure DevOps API - at present the build agents PAT token is used.
But it is certainly possible. Just needs some editing of entry functions

Thanks for the answer.
I will let you know  If we decide to go the "Use Generate Release Notes Build Task outside of TFS"-route.
If so, I would try to adapt the node or ps version and maybe provide a PR such that others could also benefit from it.

I have decided that I won't ever be porting this PowerShell task to run on any platform other then Windows. This is basically down to the fact that the Azure DevOps agent can only run 'full' PowerShell on Windows. There is no PowerShell Core support and as far as I can see no plans to add it. The direction of travel is all Node.JS base.
However, I can offer an alternative to this task, my Cross Platform Generate Release Notes now, as of 2.17.10, can be run in a build as well as a release. Hopefully that should address your needs
Also so #580


Dynamic object properties name not working

const headers = ['username', 'password']
...

view :
@each(user in users)
  {{ user['username'] }} // Working
  {{ user.username }} // Working

  @each(header in headers)
    user[header]  // Not Working, return undefined value
  @endeach
@endeach

Am I missing something?

Using $parent on nested each. So, the right code is $parent.user[header]

Hey @egodasa! 👋
You need to use $parent.
Please so some search before posting an issue.
See:

https://github.com/poppinss/edge/issues/52
https://github.com/poppinss/edge/issues/32
https://github.com/poppinss/edge/issues/30

@RomainLanz I've read the documentation and forgot about $parent variable. Using search bar isn't helpful since I'm looking for "Dynamic Properties" name, not "nested each".
Thx for Your suggestion. I'm still learning english. Sorry if I have some mistake.


Make the 08 - Number Counter Responsive.

Hey Swapnil, I would like to solve this issue can you please assign it to me

Hey Swapnil, I would like to solve this issue can you please assign it to me

Good Luck

hey, @swapnilsparsh I have added responsiveness to the app, please review my PR and merge it 😊

hey, @swapnilsparsh I have added responsiveness to the app, please review my PR and merge it 😊

Issue resolved by @SK02K1 #51


Add async/await support

Resolves #63. This is hidden behind #if swift(>=5.5) to ensure it still builds on older Xcode versions.

Hey Carson, is there a timeline for when this is being released? Would love to use swift-request in our app, but async/await support is a must for us 😄

@waly-k Unfortunately, I’m not sure when I’ll get back to this. However, you may want to look at request-dl, which a contributor started as a sort of successor to this package: https://github.com/request-dl/request-dl


add write_int_fields to replace write_packed_immediates

This avoids having to explicitly list the types of all fields -- we derive them from the type of the struct instead.
Also add write_int_fields_named, to give the fields by name instead of ordered by index.

@bors r+

:pushpin: Commit 47f8218d0dfcd9f31185f2628d07a993ba6780f5 has been approved by RalfJung

:hourglass: Testing commit 47f8218d0dfcd9f31185f2628d07a993ba6780f5 with merge 9029fa4aa93d2c0eb20a4f69fc39f1a0fb0d0565...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 9029fa4aa93d2c0eb20a4f69fc39f1a0fb0d0565 to master...


Grammar not working with the serverless inference API

System Info
Current inference api servers and custom endpoints.
Information

[ ] Docker
[X] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
Running  the following code:
import requests
from pydantic import BaseModel, conint
from typing import List
import huggingface_hub

class Animals(BaseModel):
    location: str
    activity: str
    animals_seen: conint(ge=1, le=5)  # Constrained integer type
    animals: List[str]

prompt = "Q: How many pupils fit into a bus?\n A: "

data = {
    "inputs": prompt,
    "parameters": {
        "repetition_penalty": 1.3,
        "grammar": {
            "type": "regex",
            "value":"\d+\n"
        }
    }
}

headers = {
    "Authorization": f"Bearer {huggingface_hub.get_token()}",
    "Content-Type": "application/json",
}

API_URL = "https://api-inference.huggingface.co/models/mistralai/Mistral-7B-Instruct-v0.2"
response = requests.post(
    API_URL,
    headers=headers,
    json=data
)
print(response.json())

Gives:
{'error': 'Request failed during generation: Server error: ', 'error_type': 'generation'}

However, running this with custom inference endpoint (same model), works:
...
API_URL = "https://xxxxxxxxxx.us-east-1.aws.endpoints.huggingface.cloud"
...
>>> [{'generated_text': 'Q: How many pupils fit into a bus?\n A: 40\n'}]

Expected behavior
Grammar should work with serverless inference endpoint, too
This is a follow-up to https://github.com/huggingface/text-generation-inference/issues/1717

hi @ggbetz I just tested this request on the serverless inference endpoints and got a correct response.
[{'generated_text': 'Q: How many pupils fit into a bus?\n A: 40\n'}]

This bug was fixed here: https://github.com/huggingface/text-generation-inference/pull/1765 and was a subtle issue with with batching that could cause errors with generation.


[4.2] ClosureLifetimeFixup: We need to insert a None on the backedge of a loop after we destroyed the value

rdar://40221767

@swift-ci Please test


Json runner

It is still an a ugly beast, but give these changes a glance.

It sounds like this isn't yet working, so I'll hold off on merging to master. You can merge if you see fit.

It is working actually. At least on my computer. Might need to do more stringent testing tomorrow.

I may do a pre-release to REL tomorrow to have them find issues.

Ah, I'll get that.
On Tue, Nov 21, 2017 at 11:22 PM, Mark Halonen<EMAIL_ADDRESS>wrote:

@markhalonen commented on this pull request.
In src/ncorr_commandLine.cpp
https://github.com/relinc/ncorr_cpp/pull/1#discussion_r152466635:

@@ -553,92 +424,105 @@ int main(int argc, char *argv[]) {
else{
config_DIC_analysis = DIC_analysis_config::REMOVE_BAD_POINTS;
}

std::vector imgs;
for (json::iterator it = inputFile.find("images")->begin(); it != inputFile.find("images")->end(); ++it) {

	image_names.push_back(it->get<string>());

}
//string directoryInUse = getDirPath(getImagesArray[1]);

The random commented out lines of code. Unless they are meaningful?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/relinc/ncorr_cpp/pull/1#discussion_r152466635, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHfV6F4gG6mwWbWv9ttTS6xr5_s-eQjZks5s46GEgaJpZM4Qle3m
.


feat: Add new Column Management Modal component

Column Management Modal
Associated ticket: RHINENG-3003
What
Implement a modal that a user can use to configure the visibility of individual table columns. The modal supports columns that are hidden by default and also those that should be unhideable.

Screencastfrom2024-04-1216-04-18-ezgif.com-video-to-gif-converter.webm
Why
Since there is a request (see the associated ticket above) to implement column management in tables in all apps across Insights it would make sense to abstract a component that could be reused. Currently, the only implementation of column management is in Insights Vulnerability (hidden inside kebab menu in table toolbar). This PR is based on the Vulnerability's implementation inspired by the PF column management demo.
Example usage in practice with Redux
// Hooks file
export const useColumnManagement = (columns, onApply) => {
    const [isModalOpen, setModalOpen] = useState(false);

    return [(
        <ColumnManagementModal
            appliedColumns={columns}
            applyColumns={newColumns => onApply(newColumns)}
            isModalOpen={isModalOpen}
            setModalOpen={setModalOpen}
        />), setModalOpen];
};

// Table file
const columns = useSelector(({ TableStore }) => TableStore.columns;
const [ColumnManagementModal, setColumnManagementModalOpen] =
    useColumnManagement(columns, newColumns => dispatch(changeTableAction(newColumns)));

return (
    ...
    <Button onClick={() => setColumnManagementModalOpen(true)}>Manage columns</Button>
    ...
    { ColumnManagementModal }
    ...
)

Preview: https://react-component-groups-pr-component-groups-129.surge.sh

Thank you @fhlavac for the comprehensive review, I have (hopefully) addressed all of your and @ andrew-ronaldson's comments.

@andrew-ronaldson can you please take a look? So this one can be merged

Looks good! I think the only comment I have leans more into the design guidelines. The example given includes a primary button as the trigger for this modal. That's fine on its own but if the table has a toolbar with other actions we should only have one primary button on the page. In OCP we had these buttons to trigger the manage column modal.
If I want to use this code in my product, can I easily change that primary button to a secondary or icon button? If that's the case I think we just need a bit of text pointing to design guidelines for buttons.

@andrew-ronaldson I changed the example button variant to secondary and added an icon to the example. The button is not part of the component, since there are various ways to implement it, I think the majority of the time the button will be inside of a kebab menu. Is this okay?


value error creating uncaught exception

ERROR: Uncaught exception in luigi
Traceback (most recent call last):
File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\retcodes.py", line 75, in run_with_retcodes
worker = luigi.interface._run(argv).worker
File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\interface.py", line 211, in _run
return _schedule_and_run([cp.get_task_obj()], worker_scheduler_factory)
File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\interface.py", line 149, in _schedule_and_run
not(lock.acquire_for(env_params.lock_pid_dir, env_params.lock_size, kill_signal))):
File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\lock.py", line 111, in acquire_for
pids = {pid for pid in _read_pids_file(pid_file) if getpcmd(pid) == my_cmd}
File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\lock.py", line 111, in
pids = {pid for pid in _read_pids_file(pid_file) if getpcmd(pid) == my_cmd}
File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\lock.py", line 42, in getpcmd
_, val = lines
ValueError: not enough values to unpack (expected 2, got 1)
Hi The above error was started yesterday and I'm not entirely sure of the cause. Luigi was not updated and was working perfectly fine in the morning. Let me know if more details are required

ERROR: Uncaught exception in luigi Traceback (most recent call last): File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\retcodes.py", line 75, in run_with_retcodes worker = luigi.interface._run(argv).worker File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\interface.py", line 211, in _run return _schedule_and_run([cp.get_task_obj()], worker_scheduler_factory) File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\interface.py", line 149, in _schedule_and_run not(lock.acquire_for(env_params.lock_pid_dir, env_params.lock_size, kill_signal))): File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\lock.py", line 111, in acquire_for pids = {pid for pid in _read_pids_file(pid_file) if getpcmd(pid) == my_cmd} File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\lock.py", line 111, in pids = {pid for pid in _read_pids_file(pid_file) if getpcmd(pid) == my_cmd} File "C:\Users\priestleyg\Anaconda3\lib\site-packages\luigi\lock.py", line 42, in getpcmd _, val = lines ValueError: not enough values to unpack (expected 2, got 1)
Hi The above error was started yesterday and I'm not entirely sure of the cause. Luigi was not updated and was working perfectly fine in the morning. Let me know if more details are required

Did anyone ever figure this out? This just happened to me and seems to be non-deterministic.

So, for more information about this error. I was working with someone on a Remote Desktop Computer, so I wasn't sure if this was caused by them running the same Task. I noticed the error came from lock.py, and when looking at the description of the source code...it was to prevent workers from running the same workflow? I simply renamed my Task ( In this case from RunAll to Run) and it the error went away and my pipeline ran.


Add support for ICMP probes and add job variable

This PR renames some panels to make it more generic so dash can support both HTTP and ICMP probes as well.
It also adds job variable, which is very handy when it comes to grouping probes:
HTTP probe:

ICMP probe:

Hi, and thanks, for looking into this PR.
job label tends to be a good label that super helpful sometimes to 'group'  instances that have something common.
In case of blackbox, it can be a group of endpoints, so it easy much easier to select or find the endpoint you need, instead of just flat list of instances.
Also, Summary panels show group stats for selected job unless you select Job: 'All'.

Hi, and thanks, for looking into this PR.
job label tends to be a good label that super helpful sometimes to 'group' instances that have something common.
In case of blackbox, it can be a group of endpoints, so it easy much easier to select or find the endpoint you need, instead of just flat list of instances. Also, Summary panels show group stats for selected job unless you select Job: 'All'.

Great makes sense!


Add required roles to helm-chart

/area runtime
/area operator
/area docs
What version of Dapr?
1.0.0-rc.1
Expected Behavior
Dapr operator should be fully functional after being installed with helm w/o additional actions needed.
This both serves as ease-of-use and also documents what permissions are in fact needed.
Actual Behavior
operator will fail because of missing roles/permissions.
Steps to Reproduce the Problem
Install dapr with the helm chart in a secured cluster. In my case latest openshift 4.6.4.
Release Note
RELEASE NOTE: FIX Bug in helm-chart. Added required role and role-binding.

Closed as a dup of #2620

Closed as a dup of #2620


How to get hdrp working?

Could you please enlighten me what i am doing wrong? "gaussianSplatRenderSystem" does not exist in the current context.

open GaussianSplatHDRPPass.cs file
Add using GaussianSplatting.Runtime; to the top
then can follow these steps to add volume and pass https://github.com/aras-p/UnityGaussianSplatting/blob/main/docs/render-pipeline-integration.md

Thanks! I got it working.

Closing this

Sorry all, I was not testing HDRP/URP and accidentally added compile errors. Fixed now!


[FEATURE REQUEST] A Temporary Solution for Generating Images Directly without create URL Manualy

Feature requested
Referrance https://skilliconsimagegen.vercel.app/

That's really funny, I've been working on pretty much the same thing, I think a builder like this would be excellent, and its fairly easily do-able!
Here's my version with the markdown and HTML code generation:
https://skill-icons-builder.vercel.app/

Thankyou, Quite good UI ✅ but make sure Add some Addtional Future Like Keywords that i have, i made only for timepass & build my read me temporary fast. i will open pullrequest for Keywords in your repo.
Main Thing My Builder will be Removed after @tandpfun will release new Builder or Promote your site on this repo from vercel.
Issue/Future on Your Site make sure you implement or fix it

Show How much Skills Selected by User & Total Skills Available with Circular Progress (if you can)
Support Key Words Search
Preview Image Casing Lag when Using Thrid Party Library For Slider Insted of use Native Input Type Range
I Know Real Time Slider are good but this seniro Casing Lag when changing Value due to Preview Image Size Changes

@Trixzyy @MeetBhingradiya These are really dope! However, I don’t think Tadpfun has been active on this repo for a while. I’ve been contributing to another repository that stepped in during Tadpfun’s absence and also started the project in Go.
https://github.com/LelouchFR/skill-icons


Unittests

Please prefix your pull request with one of the following: [FEATURE] [FIX] [IMPROVEMENT].
In raising this pull request, I confirm the following (please check boxes):

[ ] I have read and understood the contributors guide.
[ ] I have checked that another pull request for this purpose does not exist.
[ ] I have considered, and confirmed that this submission will be valuable to others.
[ ] I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
[ ] I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows (check one):

[ ] I have never used the project.
[ ] I have used the project briefly.
[ ] I have used the project extensively, but have not contributed previously.
[ ] I am an active contributor to the project.

{pull request content here}

Codecov Report
Base: 83.19% // Head: 83.33% // Increases project coverage by +0.14% :tada:

Coverage data is based on head (f89a61e) compared to base (be29732).
Patch coverage: 80.76% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
+ Coverage   83.19%   83.33%   +0.14%
==========================================
  Files          36       36
  Lines        3653     3672      +19
  Branches      434      433       -1
==========================================
+ Hits         3039     3060      +21
+ Misses        484      475       -9
- Partials      130      137       +7

Flag
Coverage Δ

unittests
83.33% <80.76%> (+0.14%)
:arrow_up:

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

Impacted Files
Coverage Δ

utility.py
46.15% <36.36%> (-53.85%)
:arrow_down:

mod_upload/controllers.py
74.53% <75.00%> (+0.09%)
:arrow_up:

mod_ci/controllers.py
72.26% <80.64%> (+1.80%)
:arrow_up:

mod_sample/media_info_parser.py
85.03% <87.50%> (-0.79%)
:arrow_down:

mod_test/controllers.py
71.71% <88.88%> (-0.29%)
:arrow_down:

mod_ci/cron.py
100.00% <100.00%> (ø)

mod_ci/models.py
93.87% <100.00%> (-0.13%)
:arrow_down:

mod_customized/controllers.py
97.50% <100.00%> (ø)

mod_deploy/controllers.py
86.32% <100.00%> (+0.35%)
:arrow_up:

mod_regression/controllers.py
94.14% <100.00%> (ø)

... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Order the currencies alphabetically

Right now, the currencies are ordered by their usage, according to a comment in currencies.dart.
As the library offers a way to show the favorite currencies in the beginning of the list, don't you think we should show the most used currencies there, like EUR or USD, and then order the rest alphabetically? A user that scrolls the list looking for the ethiopian birr doesn't care about the most traded currencies. I know there's a search text field, but as a user, honestly, you find that search text field when you finished scrolling the list 😅
May we order them alphabetically?
Thanks a lot, by the way!

@rocboronat I don't think adding the most used currencies in favorite is a good idea. Some apps use favorite as it is. Also As far as I know some developers preferred this sorting.
What we can do is to add an alphabetical sort option.

Oh! Yes yes, sorry, you're totally right. I wanted to proprose to let the developer use the favorites option to show the usd and eur there, in case he wants to make that currencies easier to choose. Thanks for the answer, btw!


Added onGoing notification (cannot be dismissed by user)

Hi Eddy,
I added onGoing as an option for the notification. see here for more details. I could only implement this for android, not sure it's currently possible for ios.

Hi! Excellent work 👍
I'll update the TS definition as well, and rename onGoing to ongoing as that's what Android uses.
Really appreciate you took the time to create this and send a PR.
Eddy

No problem ;)
Published this to npm yet?

This plugin was written in JS, but there's a TS definition file. Looking to migrate to TS in the future, just not a priority.


"Show gender and age in caravan form dialog" integration

I just published it as a separate mod but thought it could be integrated here.

Couldnt merge this as it contains the dll-file.
There is no need to include that in PRs since I recompile anything before publishing anyway.
All these changes should be added in 41174a8.
Thank you for the contribution!


xds: Handle NACKs of initial versions of resources

Cilium agent was spinning sending responses when Envoy NACKed the initial response from Cilium.
Signed-off-by: Romain Lenglet<EMAIL_ADDRESS>

test-me-please

test-me-please

In lieu of proper CI build, here's proof that the unit tests are passing, demonstrating that the bug has been fixed:
$ go test github.com/cilium/cilium/pkg/envoy/... -test.v -check.v
...
OK: 16 passed
--- PASS: Test (5.01s)
PASS
ok  	github.com/cilium/cilium/pkg/envoy	5.090s
...
OK: 13 passed
--- PASS: Test (4.32s)
PASS
ok  	github.com/cilium/cilium/pkg/envoy/xds	4.327s


Is there a cleaner way to implement UUIDv4 within PostgreSQL?

Is there a cleaner way to implement UUIDv4 within PostgreSQL via migration? Like if I want to set uuid_generate_v4() as default for my uuid column then it can't be done like this:
$table->uuid()->default( 'uuid_generate_v4()' );

The package has support for UUIDv7 via an expression class which adds a PostgreSQL expression as default value.
For now this is what I'm able to run:
Schema::createExtensionIfNotExists( 'uuid-ossp' );

Schema::create( 'users', function( Blueprint $table ) {
    $table->uuid( 'id' );
    $table->string( 'name' );
    $table->string( 'email' )->unique();
    $table->timestamp( 'email_verified_at' )->nullable();
    $table->timestamp( 'email_verified_on' )->nullable();
    $table->string( 'password' );
    $table->rememberToken();
    $table->timestamps();

    $table->primary( 'id' );
} );

DB::statement( 'ALTER TABLE users ALTER COLUMN id SET DEFAULT uuid_generate_v4();' );

but this exactly as clean as this
$table->uuid( 'id' )->default( new Uuid7() );

Ok so I took the same approach Uuid7 class does & created Uuid4 class.
<?php
declare( strict_types=1 );

namespace App\Support\Expressions;

use Illuminate\Contracts\Database\Query\Expression;
use Illuminate\Database\Grammar;

class Uuid4 implements Expression {

    public function getValue( Grammar $grammar ) : string
    {
        return 'uuid_generate_v4()';
    }

}

Now all that needs to be done is this:
$table->uuid( 'id' )->default( new Uuid4() );

@tpetry Let me know if you would be willing to add this Uuid4 class in the package & I'll open a PR.

A UUIDv4 expression [1] is available in my generic package working for all databases supported by Laravel. But this one emulates the uuid_generate_v4() function to also support older databases - which is slower. The expression you've posted is probably better but I won't add it as it is can also easily be done by new Expression('uuid_generate_v4()') in your migration.
[1] https://github.com/tpetry/laravel-query-expressions


Remove Optional OperatorAddress supplier

Search for all instances of operatorAddressSupplier and make this non-optional.
This issue is related to https://github.com/sappenin/java-ilpv4-connector/issues/302, which has more discussion around this topic, as well as instigating code from both the Rust and Java connectors.

From @adrianhopebailie in Interledger slack:

I would suggest we figure this out and add it to the spec. I think it has come up in all implementation design discussions.I propose we have something like: “When a connector starts up and is configured to get its address from a parent it should use the address self until it is able to get an address. IL-DCP requests to the connector MUST be rejected”


Future: Consider adding static analysis for vulnerabilities (e.g., hit density)

Per section 5.B of the paper:
Perform static analysis on source code to determine the likely number of latent vulnerabilities (e.g., using Coverity scan, RATS, or flawfinder); measures such as hit density could indicate more problematic software.   A variant would be to report on densities of warnings when warning flags are enabled.

This will report many False Alarms.

Sure, but the point would be to see if the density was unusually high.  If you keep walking to the edge of a cliff, eventually you are likely to fall off.


docs: add release badge

Coverage remained the same at 100.0% when pulling 17f9fc18e5d0306d1413b5bd678cc19c201deee2 on add-release-badge into 642a3173dcf753830116453d7c04dbe67e0f9bfe on main.

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


ENH: Add extra "back" button in the uploads page

Add extra "go to study description button at the bottom of the screen in
the uploads page.
Fixes #1754

👍


[basisu, gppanel, msgpack11, nana, rapidcheck, folly] Add missing header file of STL

Describe the pull request

What does your PR fix?
Since an internal change,  doesn't transitively include, then these ports build failed like the following error.
folly\src\46d6707604-d64234cbb5\folly/lang/Bits.h(71): error C2039: 'bit_cast': is not a member of 'std'
gppanel\src\eeef051b25-b2b3e8579b\gpPanel\mpScaleY.cpp(146): error C2039: 'fabs': is not a member of 'std'
For fixing these issues, just need to add the missing header file into the source.

/azp run

I believe this is a 16.6 fix.


ec_deployment: Add config block to elasticsearch

Description

Adds a new config block which allows users to set custom settings to
the elasticsearch.yml file on the elasticsearch resource.
It's in line with the other config blocks which have been added to the
other ec_deployment resources, plus the specific elasticsearch resource
built_in_plugins list called plugins.
Additionally, documentation most of the ec_deployment resource has
been added, but only covering the elasticsearch resource documentation.
The other resource's documentation will be added later on.
Related Issues

Resolves #5
How Has This Been Tested?

$ make testacc
-> Formatting Go files...
-> Formatting Terraform documentation blocks...
-> Done.
-> Running terraform acceptance tests...
=== RUN   TestAccDatasourceDeployment_basic
=== PAUSE TestAccDatasourceDeployment_basic
=== RUN   TestAccDeployment_basic
=== PAUSE TestAccDeployment_basic
=== RUN   TestAccDeploymentTrafficFilter_basic
=== PAUSE TestAccDeploymentTrafficFilter_basic
=== CONT  TestAccDatasourceDeployment_basic
=== CONT  TestAccDeployment_basic
=== CONT  TestAccDeploymentTrafficFilter_basic
--- PASS: TestAccDeploymentTrafficFilter_basic (12.18s)
--- PASS: TestAccDatasourceDeployment_basic (672.30s)
--- PASS: TestAccDeployment_basic (810.06s)
PASS
ok  	github.com/terraform-providers/terraform-provider-ec/ec/acc	810.556s

Types of Changes

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

32f4aa0 addresses the merge conflict


Add .english.md to english rosetta challenges

A number of the Rosetta code challenge files end with .md. They should end with .english.md.

Hey @moT01,
can I take this?
Will open PR tomorrow

Files list

That looks correct @UgRoss, you may want to wait on the PR until https://github.com/freeCodeCamp/freeCodeCamp/pull/38747 gets merged since it's all the same files.

Sure, thanks!

Hey there @moT01!
I'm super new to coding and looking for open projects and teams to join. Any tips or help is greatly appreciated!

Hi @moT01 , I can work on this.I have coding experience in Python and Java but have no experience in open source projects. Please let me know if I can work on this.

I mentioned that there's another PR in place to add something to most of these files, and that it might be a good idea to wait for that one to get merged. Anyone is free to make a PR, but you will likely have to rebase and make the same changes again after that one gets merged.

@moT01 Thanks for that update. I have got another issue to work on.


[Enhancement] Voice channel "live" status checker.

Checks

[x] I confirm that I have searched for existing issues / pull requests before reporting to avoid duplicate reporting.
[X] I confirm that I noted that if I don't follow the instructions, the issue may be closed directly.

Description
Ik that you just can check if the bot is online, but would be cool if voice channels were added to, for example "Server status: Online"
Player Count: 0/20.

What is that? Please add screenshot to explain.

Kinda like this, just with the server metrics.

Oh ok., looks great

Would you consider it?

Will but lower priority. Placeholder API top priority in August

All right, thanks!


Correct the link of .NET 8.0 SDK in README.md

Good catch, thanks!

Ah, I missed that when making PR #21, thank you for noticing and fixing it!


[Bug]: Sentry exceptioni in ActionSagas.ts: actionObj.name APPSMITH-5M7

Is there an existing issue for this?

[X] I have searched the existing issues

Description
https://appsmith.sentry.io/issues/3811373707
Steps To Reproduce
NA
Public Sample App
No response
Environment
Production
Issue video log
No response
Version
ignore

less than 300 events, won't fix


Document event retrieval endpoints in more detail

Rendered: see 'docs' commit status

This also adds a previously-undocumented endpoint: /state_ids
Backfill is technically not part of this section, however it is being left untouched to make the merge with #1469 easier (which moves it out of the file).
Reference material:

Some calls to synapse on these endpoints with a relatively simple private room.

fixes https://github.com/matrix-org/matrix-doc/issues/1451


Fixing Markdown layout of ASCII art tables.

Using code blocks for all such tables.  They should probably be converted to proper markdown tables.

eoinobrien's patch does the right thing.  Ignore this one.


chore: Update documentation for installing EFA device plugin to use official EKS chart

Issue #, if available:
Description of changes:

Update documentation for installing EFA device plugin to use official EKS chart

Ref: https://github.com/aws-neuron/aws-neuron-sdk/pull/850
Ref: https://github.com/aws-samples/aws-efa-eks/pull/22#issuecomment-1995125135
Ref: https://github.com/awsdocs/amazon-eks-user-guide/pull/747
Ref: https://github.com/aws-ia/terraform-aws-eks-blueprints/pull/1904
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@aws-maens / @natemail-aws any updates? we'd like to encourage users to use the official chart and not the dated manifests from a sample repo, thank you!

FYI - the EKS docs have now been updated to reflect this change

Reviewed - LTGM!


Fixed #29822 -- Add a Timezone aware datetime widget for admin

Ticket :- https://code.djangoproject.com/ticket/29822
Timezone conversion is handled on  frontend using DateTimeShortcuts.timezoneOffset.

Thanks for reviewing the PR @carltongibson. I'll add test and release notes for the new widget.

it doesn't look like you've used the suggested Intl API from the discussion — was there a reason for that?

DateTimeShortcuts.timezoneOffset stores the offset between server and user's timezone. So, I thought we can use that to convert the time from one timezone to another. I thought this would be a simpler approach than using the Intl API.

@Shubh1815 Do you have time to keep working on this?

Closing due to inactivity.

New attempt: #15806


[CHORE] Prevent Mainnet forking tests from executing with custom commit message prefix

Overview
To prevent the exhaustion of INFURA api keys, we should have a commit message prefix that prevents the execution of those tests. Something like this would allow us to develop and eventually make CI green on the entire test suite only after we're closer to the finish line for a PR.
Message prefix: [wip]
git commit -m"[wip] this shouldn't execute the mainnet forking tests"

Rather than make use of a commit message prefix, I suggest having a condition to avoid running mainnet forking tests while the PR is in a draft state.
We can still run all other CI jobs as is, once the PR is in a 'ready to review' state the mainnet forking tests  would run accordingly and this would avoid forcing specific semantics on commit messages. Thoughts? @drewstone