Documentation incomplete

The documentation describes how to describe and ingest collections, but there is no explanation as to how the collections can be used in other files (such as in an index.jade).

theqwertman is right. Any examples of this?

+1. An example of simply listing the posts in a collection would be extremely helpful.

Without an overview of the added properties this plugin is basically useless...

Greatly appreciate a PR for this.
On Sat, 8 Jun 2019, 01:03 Jan Katzer<EMAIL_ADDRESS>wrote:

Without an overview of the added properties this plugin is basically
useless...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/segmentio/metalsmith-collections/issues/30?email_source=notifications&email_token=AAOQL4FJHHGJL6T6FA6XXBDPZLZOPA5CNFSM4A6GGHH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXHIMSI#issuecomment-500074057,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOQL4BIYYY4PQ5SYCL3C3TPZLZOPANCNFSM4A6GGHHQ
.

Solved with release 1.2.0


CAMEL-9602 - ProducerTemplateBuilder

This is a first cut of a ProducerTemplateBuilder which covers some basic cases.
It does not support exchange, processor and exchange patter.
Any feedback would be appreciated.

@davsclaus do you think this is good enough and matches your requirements ?

Yeah I wonder if you should have a method that returns the Exchange as result? But maybe you just use request(Exchange.class) as that variation?
Also if you want the Message as return, you use request(Message.class). Otherwise it would be the message body only being returned.
We should then also support this builder with dependency injection so you can do
@Autowited
ProducerTemplateBuilder template

And also with the @Produces etc
@Produces("activemq:queue:foo")
ProducerTemplateBuilder template

Then the template has been pre-configured to send to that uri.
But all that is a different ticket we can look at after having a builder.

And you may want to explain what that with/without event notifier means. We may want to remove that, eg keep the builder simple but for  the 95% use-cases.

I've amended the code so now:
a) advanced functionalities like events, cache, executors can be customized by providing a consumer
ProducerTemplateBuilder.on(context)
    .withTemplateCustomizer(t -> t.setExecutorService(myExecutor))
    .withBody("the body")
    .to("direct:start")
    .request();

b) request can return an Exchange or a Message according to the type requested
Notes:

the ProducerTemplate does not have an asyncRequest which returns an Exchnage so point b above is not applicable (but I may be wrong)
I'm unsure about the name ProducerTemplateBuilder as a builder is supposed to "build" something here instead we are adding a DSL on top of  ProducerTemplate but well, I do not have any nicer name in my mind :-)
should we provide also a clear method to cleanup headers/body ?

The asyncXXX methods is not much in use, so its okay not to have em.
Yeah the naming is always tricky. Its more a fluent producer style. FluentProducerTemplate ?
When do you want to use clear? What would the code/use-case be ?

+1 for FluentProducerTemplate
The usage of clear could to reuse the same fluent template i.e. you want so have a different set of headers. Maybe it is better to have a contextualized clear like clearHeaders(), clearBody() etc. I'm unsure about this, just thinking aloud.

Ah yeah such as in a for loop and you want to send to the same endpoint but change maybe the body/headers in the loop.
You could have the fluent dependency injected with the endpoint and other settings, and thus would like to reuse the instance but clear the headers/body in the loop.
But also we should try to keep the api small so the fluent do not have a ton of methods like the producer template got over time.

Or maybe add withHeaders(Map) and you can set it to null to clear it
template.withHeaders(null)
        .withHeader("header1", "value")
        .withHeader("header2", "value")

Not very beautiful indeed.

Yeah setting null is not a good way. I suggest to go without the clear etc at first, and then we can see later if we need them.

renamed to FluentProducerTemplate
added clear headers/body
the underlying template is now cached and reused among calls

Should we remove the sync* stuffs from the fluent variant?

Looks much better.
The fluent builder has not so many methods so maybe its fine for now, we have until 2.18 is released to make up the api ;)

So ok to merge ?

+1

Is the package org.apache.camel.builder ok or better to move to org.apache.camel.impl ?

btw, send ha two variants in ProducerTemplate:

one throws an exception sendBody(ednpoint, body)
one does not send(endpoint, processor)

Should we mimic that behavior in send() ?

I think builder is a good package, that is where the route builder is also.

If they return an exchange then they should not throw an exception, but set the exception on the exchange instead.
And if they throw an exception then that should be runtime based exceptions such as CamelExecutionException

I see, the difference is that sendBody is not supposed to return a value but in the fluent builder such possibility does not exists, should we add it ? I'd say no and one should eventually call ExchangeHelper.extractResultBody(result, null) , make sense ?

The methods are using this naming style
send = InOnly    (no response)
request = InOut    (response)

That's not what the ProducerTemplate does: send(endpoint, processor) does return but send Body just ignore the return result (it calls send under the hoods) and eventually throws and exception, what pattern should I follow
for the fluent template?

We could also make send/request return the same. The difference is essentially if they set the MEP as InOnly or InOut.
And then they return by default the message body. So you need to specify (Exchange.class) if you want exchange returned. But that should be more seldom. As people often need only the body, or if not, then they can get the Message or Exchange by specifying those as the type.
Then its more consistent. And then send/request is only about setting the MEP.

I think I'll leave as it and we may change the APIs later on according on how people will use it.


[action] sonar: in Sonar analysis sonar branch_name is not working

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject
I am trying to provide the sonar_branch in sonar method in fastlane and it is not able to recognize branch_name option
Question Description
Here is the detailed error log of the issue I am trying to
Could not find option 'branch_name' in the list of available options: project_configuration_path, project_key, project_name, project_version, sources_path, project_language, source_encoding, sonar_runner_args, sonar_login

Please post your fastlane env output and the command you are running.

You are using fastlane 2.70.0 which is super old. Update, and you will get all the new features that are in the documentation.


Add Payload for phpinfo()

This PR moves the implementation of Ray::phpinfo() to its own payload class for consistency with other methods.  The functionality of phpinfo() has not been changed.

Nice, thanks!


[Bug]: version 12.0.0 – Error: Unexpected extraInfo events for request on page.goto

Bug description
Steps to reproduce the problem:

run a browserless/chrome docker image (https://hub.docker.com/r/browserless/chrome)

docker run --rm -it -p 3000:3000 -e DEBUG='browserless*' -e PREBOOT_CHROME=true -e MAX_CONCURRENT_SESSIONS=10 --name browserless browserless/chrome:latest

try to load a page with puppeteer (puppeteer-core):

  const browser = await puppeteer.connect({ browserWSEndpoint: 'ws://localhost:3000' });
  const page = await browser.newPage();
  await page.goto('https://google.com', {
    waitUntil: 'networkidle2',
  });
  await page.screenshot({ path: 'puppeteer.png' })
  await browser.close();

Getting an exception (see the "Relevant log output"):
Error: Unexpected extraInfo events for request 8574FCDD373436FD23BAE6CB1FF3CAED

Added a console.log to print the extraInfos (see the "Relevant log output"):
With puppeteer v11.0.0 this doesn't happen – same code, same browserless, only chaning the version in package.json and running yarn to install – the page loads and I'm getting the screenshot in the file.
Puppeteer version
12.0.0
Node.js version
v16.4.0
npm version
yarn 1.22.5
What operating system are you seeing the problem on?
macOS
Relevant log output
Puppeteer exception:

/Users/yurique/.../node_modules/puppeteer-core/lib/cjs/puppeteer/common/NetworkManager.js:347
            throw new Error('Unexpected extraInfo events for request ' + responseReceived.requestId);
                  ^

Error: Unexpected extraInfo events for request 8574FCDD373436FD23BAE6CB1FF3CAED
    at NetworkManager._emitResponseEvent (/Users/yurique/.../node_modules/puppeteer-core/lib/cjs/puppeteer/common/NetworkManager.js:347:19)
    at NetworkManager._onResponseReceived (/Users/yurique/.../node_modules/puppeteer-core/lib/cjs/puppeteer/common/NetworkManager.js:370:14)
    at /Users/yurique/.../node_modules/puppeteer-core/lib/cjs/vendor/mitt/src/index.js:51:62
    at Array.map (<anonymous>)
    at Object.emit (/Users/yurique/.../node_modules/puppeteer-core/lib/cjs/vendor/mitt/src/index.js:51:43)
    at CDPSession.emit (/Users/yurique/.../node_modules/puppeteer-core/lib/cjs/puppeteer/common/EventEmitter.js:72:22)
    at CDPSession._onMessage (/Users/yurique/.../node_modules/puppeteer-core/lib/cjs/puppeteer/common/Connection.js:245:18)
    at Connection._onMessage (/Users/yurique/.../node_modules/puppeteer-core/lib/cjs/puppeteer/common/Connection.js:117:25)
    at WebSocket.<anonymous> (/Users/yurique/.../node_modules/puppeteer-core/lib/cjs/puppeteer/node/NodeWebSocketTransport.js:13:32)
    at WebSocket.onMessage (/Users/yurique/.../node_modules/ws/lib/event-target.js:199:18)

console.log(extraInfos):
[
  {
    requestId: '8574FCDD373436FD23BAE6CB1FF3CAED',
    blockedCookies: [],
    headers: {
      status: '301',
      location: 'https://www.google.com/',
      'content-type': 'text/html; charset=UTF-8',
      'bfcache-opt-in': 'unload',
      date: 'Sun, 28 Nov 2021 01:24:14 GMT',
      expires: 'Tue, 28 Dec 2021 01:24:14 GMT',
      'cache-control': 'public, max-age=2592000',
      server: 'gws',
      'content-length': '220',
      'x-xss-protection': '0',
      'x-frame-options': 'SAMEORIGIN',
      'alt-svc': 'h3=":443"; ma=2592000,h3-29=":443"; ma=2592000,h3-Q050=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,quic=":443"; ma=2592000; v="46,43"'
    }
  },
  {
    requestId: '8574FCDD373436FD23BAE6CB1FF3CAED',
    blockedCookies: [],
    headers: {
      status: '200',
      date: 'Sun, 28 Nov 2021 01:24:14 GMT',
      expires: '-1',
      'cache-control': 'private, max-age=0',
      'content-type': 'text/html; charset=UTF-8',
      'bfcache-opt-in': 'unload',
      p3p: 'CP="This is not a P3P policy! See g.co/p3phelp for more info."',
      'content-encoding': 'gzip',
      server: 'gws',
      'content-length': '41595',
      'x-xss-protection': '0',
      'x-frame-options': 'SAMEORIGIN',
      'set-cookie': '1P_JAR=2021-11-28-01; expires=Tue, 28-Dec-2021 01:24:14 GMT; path=/; domain=.google.com; Secure; SameSite=none\n' +
        'NID=511=HNG5DW5oAvTGM3MxEDI5w0cbS7BUiLM0i7vCtut0VnjatrLZPA0WG9nzN1CiIn3I72ZLtL4OXeO4msFkUZ5zYyX42rGAUgQPbbwkk_LqjVnbyxImeq-Nsrb9f3zAln1DjuF0WoSUTqR1iav_DqsLms_6rFD_aPnF5gQXW7so-cw; expires=Mon, 30-May-2022 01:24:14 GMT; path=/; domain=.google.com; HttpOnly',
      'alt-svc': 'h3=":443"; ma=2592000,h3-29=":443"; ma=2592000,h3-Q050=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,quic=":443"; ma=2592000; v="46,43"'
    }
  }
]

Experienced the same problem in GitHub Actions: https://github.com/DIYgod/RSSHub/runs/4343296274#step:6:76
Source code: https://github.com/DIYgod/RSSHub/blob/master/test/utils/puppeteer.js

I'm staring at this too, unsure where to start to fix...

Same problem

having the same issue on my arm mac.  no issue with the same image on intel mac.

Same problem. driving me nuts.

A temporal solution that prevents the application from exiting, is ignoring HTTTS errors like this:
const browser = await puppeteer.launch({
    headless: true,
    executablePath: '/usr/bin/chromium-browser',
    args: [
        '--no-sandbox',
        '--disable-setuid-sandbox',
        '--disable-dev-shm-usage',
    ],
    ignoreHTTPSErrors: true
});

@yurique What version of Chrome does browserless/chrome:latest contain?

@OrKoN I think it's 93.0.4577.0:
docker exec -it ed08a9d58726 /bin/sh

blessuser@ed08a9d58726:/usr/src/app$ /usr/bin/google-chrome --version
Chromium 93.0.4577.0

Version 12.0.0 is only compatible with the newer Chromium versions (e.g., 97.0.4692.0 or later) because of some breaking CDP changes.

Oh! Thanks, @OrKoN! And sorry for the extra noise.
(I kind of assumed browserless/chrome:latest would have a more-or-less up-to-date version, didn't think to double-check that, I think I've lost track of Chrome versions long time ago :) ).

Yeah, no problem, thanks for reporting! It took us a while to update the Chromium version bundled with Puppeteer (e.g., we do not have any releases for Chromium versions 94 - 96) due to the aforementioned CDP changes. If you encounter any other issues using v12 with Chromium 97+, let us know.

Apologies in advanced if I've done something wrong but I'm receiving these errors on Chrome 97
/project/node_modules/puppeteer/.local-chromium/linux-938248/chrome-linux/chrome --version
Chromium 97.0.4691.0

/project/node_modules/puppeteer/lib/cjs/puppeteer/common/NetworkManager.js:346
            throw new Error('Unexpected extraInfo events for request ' + responseReceived.requestId);
                  ^

Error: Unexpected extraInfo events for request 131165.117
    at NetworkManager._emitResponseEvent (/project/node_modules/puppeteer/lib/cjs/puppeteer/common/NetworkManager.js:346:19)
    at NetworkManager._onResponseReceived (/project/node_modules/puppeteer/lib/cjs/puppeteer/common/NetworkManager.js:369:14)
    at /project/node_modules/puppeteer/lib/cjs/vendor/mitt/src/index.js:51:62
    at Array.map (<anonymous>)
    at Object.emit (/project/node_modules/puppeteer/lib/cjs/vendor/mitt/src/index.js:51:43)
    at CDPSession.emit (/project/node_modules/puppeteer/lib/cjs/puppeteer/common/EventEmitter.js:72:22)
    at CDPSession._onMessage (/project/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:245:18)
    at Connection._onMessage (/project/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:117:25)
    at WebSocket.<anonymous> (/project/node_modules/puppeteer/lib/cjs/puppeteer/node/NodeWebSocketTransport.js:13:32)
    at WebSocket.onMessage (/project/node_modules/ws/lib/event-target.js:199:18)

I have the same problem

the problem is with the latest puppeteer version (12.0.0).
Downgrading to version 11.0.0, if there is no way to update chromium, solves the problem:

yarn add<EMAIL_ADDRESS>

@rhld16 Hm, interesting... https://omahaproxy.appspot.com tells me that r938248 was released as 97.0.4692.0. Although it seems I can replicate the issue on Linux. The mac build seems to be alright... Let me investigate.

same... with this issue!

below error happening on vercel after passing payload destructure object to .hbs template file
2021-11-28T20:51:45.372Z 8fa04871-f14d-4fe4-a5fd-aa190679f653 ERROR Unhandled Promise Rejection {"errorType":"Runtime.UnhandledPromiseRejection","errorMessage":"Error: Unexpected extraInfo events for request 24.1","reason":{"errorType":"Error","errorMessage":"Unexpected extraInfo events for request 24.1","stack":["Error: Unexpected extraInfo events for request 24.1"," at NetworkManager._emitResponseEvent (/var/task/node_modules/puppeteer/lib/cjs/puppeteer/common/NetworkManager.js:346:19)"," at NetworkManager._onResponseReceived (/var/task/node_modules/puppeteer/lib/cjs/puppeteer/common/NetworkManager.js:369:14)"," at /var/task/node_modules/puppeteer/lib/cjs/vendor/mitt/src/index.js:51:62"," at Array.map (<anonymous>)"," at Object.emit (/var/task/node_modules/puppeteer/lib/cjs/vendor/mitt/src/index.js:51:43)"," at CDPSession.emit (/var/task/node_modules/puppeteer/lib/cjs/puppeteer/common/EventEmitter.js:72:22)"," at CDPSession._onMessage (/var/task/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:245:18)"," at Connection._onMessage (/var/task/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:117:25)"," at WebSocket.<anonymous> (/var/task/node_modules/puppeteer/lib/cjs/puppeteer/node/NodeWebSocketTransport.js:13:32)"," at WebSocket.onMessage (/var/task/node_modules/ws/lib/event-target.js:199:18)"]},"promise":{},"stack":["Runtime.UnhandledPromiseRejection: Error: Unexpected extraInfo events for request 24.1"," at process.<anonymous> (/var/runtime/index.js:35:15)"," at process.emit (events.js:412:35)"," at process.emit (/var/task/___vc/__sourcemap_support.js:587:21)"," at processPromiseRejections (internal/process/promises.js:245:33)"," at processTicksAndRejections (internal/process/task_queues.js:96:32)"]} Unknown application error occurred

// import the necessary node libraries
// import { puppeteer } from 'puppeteer';
const chromium = require('chrome-aws-lambda');
const puppeteer = require('puppeteer');
import { createInvoiceTemplate } from "./functions/invoice-template";
import { CONSTANTS as C } from "../lib/constants";

export default async (req, res) => {

    if (req.method !== "POST" && process.env.API_ACCESS_SECRET_TOKEN) return res.status(403).send(res.send("not allowed"));

    const { generateType } = req.body; // && JSON.parse(JSON.streq.body);
    const templatePayload = {
        ...req.body,
        assetsBaseURL: process.env.NEXT_PUBLIC_ASSETS_BASE_URL,
        siteName: C.SITE_NAME,
        logoPath: C.LOGO_PATH,
    }; // && JSON.parse(req.body);

    // console.log({ body: req.body, __dirname, createInvoiceTemplate });

    try {

        // compile the file with handlebars and inject the customerName variable
        const html = createInvoiceTemplate("invoice", templatePayload);

        // console.log('createInvoiceTemplate html output ', puppeteer);

        // simulate a chrome browser with puppeteer and navigate to a new page
        const browser = await puppeteer.launch({
            args: chromium.args,
            defaultViewport: chromium.defaultViewport,
            executablePath: await chromium.executablePath,
            headless: chromium.headless,
            ignoreHTTPSErrors: true,
        });
        const page = await browser.newPage();

        // set our compiled html template as the pages content
        // then waitUntil the network is idle to make sure the content has been loaded
        await page.setContent(html, { waitUntil: 'networkidle0' });

        // convert the page to pdf with the .pdf() method
        let response;
        if (generateType && generateType === 'pdf') {
            const pdf = await page.pdf({ format: 'A4' });
            response = pdf;
        } else {
            const screenshot = await page.screenshot({ fullPage: true });
            response = screenshot;
        }

        await browser.close();

        // // send the result to the client
        res.statusCode = 200;
        res.send(response);

    } catch (err) {
        console.log(err);
        res.status(500).json({ message: err.message });
    }
};

Template file code:
`

Invoice

@rhld16 Hm, interesting... https://omahaproxy.appspot.com tells me that r938248 was released as 97.0.4692.0. In any case, it seems I can reproduce the issue on Linux. The mac build seems to be alright... Let me investigate.

I'm seeing in puppeteer v12.0.0 on Linux and Windows, which both install 97.0.4691.0.

The same error, macOS 12.0.1, puppeteer 12.0.0, Chrome 97.0.4691.0.

Same error on Windows 10 and Node v16.12.0. Just upgraded to Puppeteer 12.0.0.

Rolling back to Puppeteer 11.0.0 solves this as expected.

I'm having a hard time fully understanding... is this only happening with old versions of chromium and the new version of puppeteer?
If this is occurring with up to date chromium and up to date puppeteer, can someone provide a minimal puppeteer script which reproduces the problem? That way we can find out what's going wrong in chromium and fix it in chromium.

@josepharhar

is this only happening with old versions of chromium and the new version of puppeteer?

It's happened with actual puppeteer and Chrome.

a minimal puppeteer script which reproduces the problem?

https://github.com/zloirock/core-js/blob/5b39046ee5f57186122bc00ee2ef752be07496df/scripts/usage.mjs
How to reproduce:

Clone core-js
npm i
npm run usage-1000

Ok, I see that the issue description has the minimal repro that I want and it occurs with up to date puppeteer and up to date chromium. Thanks!

I filed a chromium bug to fix the bad ExtraInfo event behavior: https://bugs.chromium.org/p/chromium/issues/detail?id=1274813
In the meantime, @OrKoN's change should make puppeteer stop throwing errors.

I just released v12.0.1 that removes the failing assertion. Please give it a try and thanks everyone for reporting issues!

I can confirm that it's fixed for my case.

Jumping in here to say another discussion is happening in #7802. I think the issue is that Puppeteer never expects a Network.responseReceivedExtraInfo to arrive first, which is totally valid according to CDP docs.

I just released v12.0.1 that removes the failing assertion. Please give it a try and thanks everyone for reporting issues!

@OrKoN When the requestReceived event finally comes in, will it attach the previously received extraInfo? I think that is the correct way to handle it.

I think the issue is that Puppeteer never expects a Network.responseReceivedExtraInfo to arrive first

Puppeteer will work fine if a responseReceivedExtraInfo comes before or after the corresponding responseReceived. Are you seeing any strange behavior with the newest commit/release that was added today?

@josepharhar Yes I think it is still not quite correct due to the CDP bug:
https://github.com/puppeteer/puppeteer/blob/6ee2feb1eafdd399f0af50cdc4517f21bcb55121/src/common/NetworkManager.ts#L444
      if (event.redirectHasExtraInfo) {
        redirectResponseExtraInfo = this._requestIdToResponseExtraInfo(
          event.requestId
        ).shift();
        if (!redirectResponseExtraInfo) {
          this._requestIdToQueuedRedirectInfo(event.requestId).push({
            event,
            interceptionId,
          });
          return;
        }
      }

See above that it only looks up extraInfo if redirectHasExtraInfo===true, but we know that won't happen due to the CDP bug. I think a better approach is to remove the if() altogether. In other words, if this._requestIdToResponseExtraInfo has extraInfo, we should honor it even if redirectHasExtraInfo===false

I think a better approach is to remove the if() altogether. In other words, if this._requestIdToResponseExtraInfo has extraInfo, we should honor it even if redirectHasExtraInfo===false

If we look for ExtraInfo events that happened to come before responseReceived without following the hasExtraInfo flags, then we will end up with race conditions and flaky behavior. I don't think we should look at ExtraInfo events when !redirectHasExtraInfo or !hasExtraInfo.

@josepharhar I don't know much about the redirects but I recall something about them reusing request IDs, is that the race condition you have in mind?
I suppose worst case we wait for CDP to fix their bug and then everything is fine :)

Downgrading worked!

Closing this issue because the error was suppressed in v12.0.1. The investigation and potential further fixes can be tracked via crbug.com/1274813


Cleaner Show instance for SpanInfo

This helped with debugging #237, so maybe it's worth keeping.

Makes a lot of sense - the original Show instance is clearly a quick hack.


homepage-dashboard service is automatically disabled (start script broken)

Describe the bug
I added services.homepage-dashboard to my config with a minimal config for bookmarks and services.
systemd automatically disables the service after a start attempt.
Steps To Reproduce
Steps to reproduce the behavior:

...
...
...

Expected behavior
The created systemd service homepage-dashboard.service should be startable
Screenshots
N/A
Additional context
The script invoked from homepage-dashboard.service exec seems invalid
❯ sudo cat /nix/store/9vxkymwdpzxhbihf430q7cqbss3gnjz1-homepage-dashboard-0.8.11/bin/homepage
#! /nix/store/a1s263pmsci9zykm5xcdf7x9rv26w6d5-bash-5.2p26/bin/bash -e
export PORT=${PORT-'3000'}
export HOMEPAGE_CONFIG_DIR=${HOMEPAGE_CONFIG_DIR-'/var/lib/homepage-dashboard'}
exec "/nix/store/9vxkymwdpzxhbihf430q7cqbss3gnjz1-homepage-dashboard-0.8.11/share/homepage/server.js"  "$@"

The prefix HOMEPAGE_CONFIG_DIR- seems invalid.
The line should just be
export HOMEPAGE_CONFIG_DIR='/var/lib/homepage-dashboard'
Notify maintainers
@jnsgruk (I hope it's ok to ping you for this)

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
- system: `"x86_64-linux"`
 - host os: `Linux 6.6.27, NixOS, 24.05 (Uakari), 24.05.20240416.5672bc9`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - nixpkgs: `/nix/store/3kwj19dbdfxnjbcns4hw307ylhz3wgrm-source`

Add a :+1: reaction to issues you find important.

Hey!
I think the script is correct, and it's been working fine for me for a few months.
Can you post the logs from the journal for the service please?

I'm very sorry. This was my own fault.
I didn't know the default-value syntax for bash variables, so this code confused me.
In the end it was a listen port collision.


server connect - pump Status with pump on hangs

repl console output
Pump - 22: added Pump Status to loop <bound_method>
Traceback (most recent call last):
File "main.py", line 79, in
File "lib/uasyncio/core.py", line 265, in run_forever
File "lib/uasyncio/core.py", line 181, in run_forever
File "WaterPumps/pumps.py", line 106, in pumpStatus
TypeError: function takes 2 positional arguments but 1 were given

Fixed in a7851cb


Urdu translation doesn't exist

I was trying to add Urdu translation but it says it doesn't exist.
https://hosted.weblate.org/projects/adaptive-lighting/adaptive-lighting/#translations

How do I add the Urdu translation?

https://hosted.weblate.org/projects/adaptive-lighting/adaptive-lighting/ur/
how to proceed now? 99% done

Translation added.


Not working when requiring NPM package

When requiring the flexibility package require('flexibility') I get a function that I am not sure how to use:
function(e) {      t.fillNodes(e), t.computeLayout(e) }
The only way I seem to be able to get this to work is by loading in flexibility as a script tag.
When I use a script tag the flexibility object is attached to the window and I can use the window.flexibility.walk(domElement) to apply the flexibilty polyfill to the correct nodes that need flexbox.
How can I correctly use the npm package in order to use the walk function to flexize the dom element needed in ie9?
Also should I be even using the flexibility.walk function. In the example in the readme it seems like it works without it though this was not the case for me.

Same issue here! Any solution for this?

Same issue!

Help?

@ekeric13 @mkoehler2911 @sunnyzhouy @tnrich  this should be resolved with the latest commit. Still needs an npm publish but git head should be working at least.

Why isn't there instructions on npm on the readme?


Make ProgressBar type available in public interface

mypy considers function click.progressbar() as generic with type parameter V: https://github.com/pallets/click/blob/ca5e1c3d75e95cbc70fa6ed51ef263592e9ac0d0/src/click/termui.py#L287-L303
In most cases type parameter V will be inferred from iterable function parameter:
from typing import reveal_type

import click

with click.progressbar([1, 2, 3]) as bar:
    reveal_type(bar) # Revealed type is "click._termui_impl.ProgressBar[builtins.int]"

However,  if iterable parameter is not specified (or if it's empty list), type parameter V cannot be inferred:
import click

with click.progressbar(length=1) as bar:
    pass

$ mypy mypy_click/__init__.py
mypy_click/__init__.py:3: error: Need type annotation for "bar"  [var-annotated]

This issue can be solved with manually specified type parameter:
import click

with click.progressbar[int](length=1) as bar:
    pass

Unfortunately, this doesn't work for functions:
$ mypy mypy_click/
mypy_click/__init__.py:3: error: Type application is only supported for generic classes  [misc]

Another way to fix this is specify type for variable bar:
from typing import cast

import click
from click._termui_impl import ProgressBar

with cast(ProgressBar[int], click.progressbar(length=1)) as bar:
    pass

# or

with click.progressbar(length=1) as bar: # type: ProgressBar[int]
    pass

But in this case ProgressBar is imported as not a part of public interface.
So, I think ProgressBar type should be a part of the public interface. Maybe it should not
be exported directly like class, because users should use click.progressbar() function instead.
Public ProgressBar type can be defined as protocol and click.progressbar() just returns its implementation.
Environment:

Python version: 3.11.5
Click version: 8.1.7
Mypy version: 1.6.1

Also possible to use function overloading to solve the problem above with inference of type parameter V:
@overload
def progressbar(iterable: Optional[Iterable[V]], ...) -> ProgressBar[V]: ...

@overload
def progressbar(length: int, ...) -> ProgressBar[int]: ...

Anyway, return type of  the function should be public.

Another way to work around this:
bar: click._termui_impl.ProgressBar[int]
with click.progressbar(length=1) as bar:
    pass


Trap focus when connected in <details open>

When the <details-dialog> element is connected to the document as a
child of a <details> element rendered with [open], trap focus
immediately.
In support of this change, extract the trapFocus and releaseFocus
functions, then call trapFocus within the connectedCallback if the
ancestor detail is [open].

hey @seanpdoyle just a heads up, code wise this looks good, but I want to check with our accessibility team if the change is good for them too :)


Add features to new profile screen

Description

Fix issue with Edit button taking supporters to the organization edit screen
Add actions for editing and posting updates in campaign action sheet
Fix issue breaking issue preventing posting of campaign updates
Fix profile screen state management to handle multiple profile screens in navigation stack without affecting previous screens
Improve ProfileHeader random image behavior by using useMemo hook to prevent unnecessary image switching

Extras:

Fix behavior of Add New Campaign button in FeedScreen.js to remain within stack

This iteration ready to merge


make reattaching durable subscribers transparent to clients

i.e. avoid the applications having to handle the link redirect as they do now

Hi @grs was this part of the durable subs work?


Basic Diagram Dialog

This PR adds a basic dialog to the diagram tool, accessible by pushing the v= button in the tile toolbar. The dialog just displays a cute message for now.
Tests are very basic here because the dialogs will be changing a lot in future PRs.

I'd this one will build properly once master is merged into it which now includes the updated diagram-view library.


Fix iframe-resizer "not found" warnings

Fixes iframe not found warnings described in: https://github.com/davidjbradshaw/iframe-resizer/pull/626

@zeroasterisk Any chance you could review this? Or is this repo abandoned?

Sorry, I haven't used this in a long time... Looking now

Great fix, I can't believe I missed that... Oops.

@zeroasterisk Thank you very much. When should we expect a release?

npm notice name:          react-iframe-resizer-super
npm notice version:       0.2.2
npm notice package size:  7.0 kB
npm notice unpacked size: 18.5 kB
npm notice shasum:        6838b52f1bdc85f004b71090aaca7fde8f26da1d
npm notice integrity:     sha512-iWPEkYsNjQYi7[...]P6hPFIxO0Du8A==
npm notice total files:   15

I'm currently not using this anywhere - would you please test it for me and confirm it's happy?

@zeroasterisk @khangiskhan I think this is working fine. However, if you load the same iframe more than once you end up with this each time its loaded is there anything I can change to prevent this?
iframeResizer.js:132 [iFrameSizer][Host page: iFrameResizer8] Ignored iFrame, already setup.

@dmason30 see comment here: https://github.com/zeroasterisk/react-iframe-resizer-super/issues/7#issuecomment-442078099


Whep support

Support WHEP player to subscribe special participant's stream.
TODO:

Trickle ICE/ ICE restart
Server sent events extension
Video Layer Selection extension

TBD


Code to load/train and run demo for word embedding

Hi @stevenbird,
Here is the code for Word Embedding. Look at demo() of word2vec.py. I also committed the pruned model file which is part of the model trained on 100 billions words (and the code to get the pruned model prune_word2vec_model.py). Could you please put file word_embeddings.zip to the corpora list ?

Thanks @longdt219, I'll take a look

I'm not sure it is OK to import gensim because itis LGPL while NLTK uses Apache license.
Returning -1 for similarity value may cause problems if similarity is used in some formula. For example, if similarities are multiplied, the result can silently become negative, or change the sign semi-randomly (if many words are missing). With an exception or None value users will get an error, and it'll require them to handle this case explicitly.
Another question is what is the value of Word2Vec wrapper? gensim is also Python, and its API is also easy to use.

Hi @stevenbird ,
I remove the package and convert it to the doctest. The pruned model path is also updated. The code for visualization using t-SNE and extracting pruned model are also included.

Hi @stevenbird ,
I changed the code to use t-SNE from scikit-learn instead of adding another file to nltk. Thanks @kmike  and @dimazest for pointing this out.

Thanks @longdt219. Also thanks @kmike and @dimazest.

Oh dear, this PR contained a 45MB binary file! I need to try and rewrite history to avoid bloating this repository.

Oh sorry @stevenbird , I should have removed it.

No problem, fixed now.


Merge pull request #2034 from grendello/nuget-update

Update NuGet to version 5.10.0

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

This is opened against an old bugfix branch - I am not sure what is the goal of this?
Please open the PR against the master branch if you would like to contribute something to the main development stream - thanks!


Support partial results for select/pairs

Closes #117

I think this patch is quite close to merge.
So consider several final comments:

Please add a changelog note
Let's extend our examples - add some info about this feature.
Squash all commit into singl and add good description of this feature (motivation, restrictions, ...)


docs: add disable instructions

What does this pull request do?
To assist support, we're adding disable instructions to each Agent's troubleshooting docs.

Related issues
For https://github.com/elastic/observability-docs/issues/35.

:green_heart: Build Succeeded

Expand to view the summary

Build stats

Build Cause: [Pull request #1319 opened]

Start Time: 2020-07-29T20:44:45.222+0000

Duration: 3 min 30 sec


No way to unsubscribe

I'm not sure why subscribe sends a promise (I guess to show the status of the subscribe ping to the backend?). It may make more sense to return an unsubscriber function or something. I have no idea how to unsubscribe at this point. Alternatively, if the promise response is absolutely necessary, create an unsubscribe method on the sailssocket that i can give it something... Iunno.

If you'd like, you can open a pull request and add the functionality.  This is still a very young project and we are always looking for contributors :)

+1 , is there now a way to unsubscribe ?


Exported dictionaries have their keys sorted alphabetically

Tested versions

Tested with Godot 4.1.2 and Godot 4.3 dev 5

System information
Godot v4.3.dev5.mono - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3070 Ti (NVIDIA; <IP_ADDRESS>92) - AMD Ryzen 7 5800X 8-Core Processor (16 Threads)
Issue description
When an exported dictionary is created, they keys which are set in the editor, are sorted at runtime, which is incredibly frustrating for my use case.
Steps to reproduce

Create an exported dictionary (@export var dict := {})
Add some values into the dictionary through the editor. - ({"b": 0, "c": 1, "a": 2})
Print the dictionary, it will output: "{"a": 2, "b": 0, "c": 1}"

Minimal reproduction project (MRP)
N/A

This is by design, for stability, as far as I know
See also:

https://github.com/godotengine/godot/issues/74941

should it happen with exported dictionaries though? I get why it needs to happen for dictionaries written to files and such, but i dont see why it should need to be the same with exported dictionaries?

That's what exported dictionaries are though? They are stored and loaded

but why should it be? Other than for stability as you mentioned, which seems only needed for file reading, i dont see any other reason why exported dictionaries should be sorted without the users knowledge. A built in sort function for dictionaries should be used instead if the user or the engine REALLY has to have the dictionary sorted

But they are exported, so they're stored, so they're sorted
Are you printing it from the editor? Or from running project? If it's from the running project you are loading it, you have to load the scene?

im running it from the project, but why then can it print the dictionary if its a hard coded variable, even a constant dosent have this issue, and wouldnt that be stored too?

Keeping insertion order is a feature according to the documentation:

Dictionaries will preserve the insertion order when adding new entries.

I don't think the editor should sort them.

It also effects storing and recovering a dictionary via ConfigFile

The cause of this issue is this line: https://github.com/godotengine/godot/blob/master/core/variant/variant_parser.cpp#L2248. I tried tracking down the original commit, but couldn't quite make it far back enough.
It definitely feels... load-bearing, but I removed it on my build a few weeks ago, and it has even fixed similar issues as well (one of my exported dictionaries kept re-sorting its keys on every resource save, which would have caused endless merge conflicts for my team).
I assume making this change would break existing test cases, but I agree that sorting dictionary keys at this stage of serialization should be considered a bug.


Project Red & CC: Tweaked issues with updating

I am currently running

SpongeForge version: spongeforge-1.12.2-2768-7.1.6-RC3629.jar
SpongeVanilla version: -
Java version: 1.8.0_201-b09
Operating System: Linux - Ubuntu 16.04/16.02 (not sure) LTS

Plugins/Mods:

Advanced Machines
Advanced Solar Panels
AE2 Stuff
AppleSkin
Applied Energistics 2
Aroma1997's Dimensional World
Aroma1997Core
asielib
AutoRegLib
Baubles
BD Lib
BiblioCraft
Binnie's Botany
Binnie's Design
Binnie's Extra Bees
Binnie's Extra Trees
Binnie's Genetics
Binnie Core
Blockcraftery
BuildCraft
BuildCraft Builders
BuildCraft Compat
BuildCraft Energy
BuildCraft Factory
BuildCraft Lib
BuildCraft Robotics
BuildCraft Silicon
BuildCraft Transport
CC: Tweaked
Charset
Chisel
Chisels & Bits
CodeChicken Lib
CoFH Core
CoFH World
Compact Solar Arrays
Computronics
Crafting Tweaks
CraftTweaker2
CraftTweaker JEI Support
Diet Hopper
EnderStorage
Energy Converters
Fancy Fluid Storage
FastWorkbench
Flat Colored Blocks
FoamFix
FoamFixCore
Forestry
Forge Microblocks
Forge Mod Loader
Forge Multipart CBE
FTB Library
FTB Utilities
FTB Utilities: Backups
Gravitation Suite
GravityGun
Guide-API
Harvest
iChunUtil
Immersive Cables
Immersive Engineering
IndustrialCraft 2
Industrial Foregoing
Integration Foregoing
Inventory Tweaks
Iron Chest
JEI Bees
Just Enough Energistics
Just Enough Items
Long Fall Boots
MachineMuse's Modular Powersuits
MagicBees
MCMultiPart
Minecraft
Minecraft Coder Pack
Minecraft Forge
Minecraft Multipart Plugin
MineTogether
MineTogetherServer
Mod Tweaker
Modular ForceField System
Modular Routers
Morpheus
Morph-o-Tool
MrTJPCore
MTLib
Mystical Lib
Netherending Ores
Numina
Placebo
Player Plates
PortalGun
projectred-compat
ProjectRed Core
ProjectRed Expansion
ProjectRed Exploration
ProjectRed Fabrication
ProjectRed Illumination
ProjectRed integration
ProjectRed Relocation
ProjectRed Transmission
ProjectRed Transportation
ProjectX 2
Railcraft
RandomPatches
Reborn Core
Redstone Flux
Reliquary
Retro Exchange
Shadowfacts' Forgelin
Silverfish
Simple Corn
Smooth Font Core
Soul Shards Respawn
Sponge
SpongeAPI
SpongeForge
Steve's Carts 2
Tech Reborn
Tech Reborn Mod Compatibility
Tesla Core Lib
Tesla Core Lib Registries
Thaumcrafft Inventory Scanning
Thaumcraft
Thaumic Energistics
Thaumic Tinkerer
The One Probe
Thermal Dynamics
Thermal Expansion
Thermal Foundation
The Twilight Forest
TOP Addons
TorchMaster
Traverse
UniDict
WanionLib
WirelessRedstone-CBE
World Control
YABBA

Issue Description
Workaround that fixes at least this issue atm: spongeforge-1.12.2-2768-7.1.6-RC3616.jar
Mods that didn't work correctly:

CC: Tweaked
Project Red
Maybe some more, didn't test all.

What didn't work:

CC: Tweaked

Turtles and Computers would reset after 2 seconds of running their programs

Project Red

Timer:

It works, but doesn't output signal to any adjacent cables or redstone dust

With Workaround

I can confirm this is an issue with the latest spongeforge build as well.
Besides the timer from ProjectRed not outputting I have also witnessed redstone output breaking for many of its redstone gates and redstone wire not updating.
An easily repeatable way to reproduce the weird behavior is to hook up:
Timer -> minecraft repeater -> NOT gate -> anything switching by redstone
The initial cycle works fine but the NOT gate will cease to output after the initial redstone signal disables it. Breaking redstone that is connected to the output or input of the not gate and replacing it causes the redstone to revert to correct behavior.


set-mark is incompatible with snippets

Atom's snippets with "$1, $2, ..." have the problem that blanks are not properly selected when once atomic-emacs:set-mark called.  We can recover by calling atomicEmacs.keyboardQuit(event) with Esc key now.

Hi @ashiato45,
Apologies for the slow response here. Is this still happening?
I was convinced there was a problem earlier, but now I can't seem to find the problem (with Atom 1.2.4), so it might have been incidentally fixed. If you're still seeing this, can you please provide the key sequences to demonstrate the problem?
Thanks!

Hi, as you say the problem is already fixed.  Thank you very much!


feat: fixing snap connect issues

Explanation
Profile sync SDK was not exporting any method for connecting to snap. This PR exports a method to connect to snap after initializing the SDK

@metamaskbot publish-preview

Please provide a changelog entries time! We request this in the PR template.


Crash in default_login

On a fresh Airflow install, when default_login.py is used, we are seeing a crash on the webserver. The issue is that User class is used after deletion.
Also, organizing .gitignore alphabetically as that list is growing.

I thought I had fixed that, must be in a lost branch somewhere...
This PR looks funky (lots of commits), I think that's what happens when you merge instead of rebasing, but I think it's fine [I'm still adapting to github].

Lots of commits due to merges and a couple features that were reverted. Creating a cleaner PR here https://github.com/mistercrunch/Airflow/pull/175, closing this one, and changing my git mode to rebase to avoid future commit pollution.


CNV-32959: Move InstanceTypes and Preferences under Templates in menu

📝 Description
Fixes:
https://issues.redhat.com/browse/CNV-32959
Move InstanceTypes and Preferences menu items under Templates in the Virtualization menu, as those are just other resources related to VM creation.
🎥 Screenshots
Before:
InstanceTypes and Preferences menu items placed under MigrationPolicies:

After:
InstanceTypes and Preferences menu items placed under Templates:

/lgtm


Create a command line utility

As a user of sentinel, I want to be able to run it on the command line, so that I do not need to install maven, or an IDE to run tests.
Requires #23 to be complete.

https://github.com/pcj/google-options For command line options.


[network] move hardcoded constants to configurable variables

These constants could be configurable that would allow us not having to make a full code deployment to change things like timeouts and limits.  Having these configurable later can be helpful for operations if they are found to be less than ideal values.
https://github.com/libra/libra/blob/277115b85137f65ef7da03c823572b17f613b8d8/network/src/validator_network/network_builder.rs#L50-L66

These are all mostly now in config! https://github.com/libra/libra/pull/6428


"To run this application, you must install .NET Core. Would you like to download it now?" Error Issue

A decent 2 months - ish ago i tried to download this. Then, I got the "To run this application, you must install .NET Core. Would you like to download it now?" message. I clicked Yes. Nothing Happened. I clicked around downloading the correct .NET Core versions needed of the website. Nothing happened. Please help.

Good news: IT WORKS!!!!!
Thank you SO much for the help. I've been trying to get this to work for
months now but its finally working. Your actually a legend, thank you soooo
much :)))
On Thu, 29 Jun 2023 at 17:27, dfggrfe @.***> wrote:

Hello! To download .NET Core, you can visit the following website:
https://dotnet.microsoft.com/en-us/download/dotnet/3.1
[image: image]
https://user-images.githubusercontent.com/76404065/249840332-8d6b1d59-9572-418d-baaa-075ed10e8259.png
—
Reply to this email directly, view it on GitHub
https://github.com/kemo14331/Particle-Converter/issues/25#issuecomment-1613494522,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A77UPO7O5SWQ5MGBDZLGNTTXNWUFLANCNFSM6AAAAAAYI6FMDA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Have tried this multiple times now, still not working

Does it have to be the 3.1 version? Or can I get a newer version instead?

Guys, make sure you unzip .NET core once you download it. I think thats where I went wrong.


dart analysis server has terminated frequently [VS Code]

Analyzer Feedback from IntelliJ
Version information

IDEA IU-191.6707.61
2.2.0-edge.0a7dcf17eb5f2450480527d6ad1e201fb47f1e36
IU-191.6707.61, JRE 1.8.0_202-release-1483-b44x64 JetBrains s.r.o, OS Mac OS X(x86_64) v10.14.4, screens 1920x1080, 2560x1440  Retina

console displayed: nw_path_close_fd Failed to close guarded necp fd 8 [9: Bad file descriptor]

When I opened the project with VSCode, it shows the log:
[18:03:12 GMT+0800 (CST)] [General] [Info] Activating Flutter extension for Flutter project...
[18:03:27 GMT+0800 (CST)] [General] [Error] [Analyzer] Dart_NewStringFromUTF8 expects argument 'str' to be valid UTF-8.
[18:03:43 GMT+0800 (CST)] [General] [Error] Cannot call write after a stream was destroyed
Error [ERR_STREAM_DESTROYED]: Cannot call write after a stream was destroyed
at doWrite (_stream_writable.js:406:19)
at writeOrBuffer (_stream_writable.js:394:5)
at Socket.Writable.write (_stream_writable.js:294:11)
at Analyzer.sendMessage (/Users/siyuaniosdeveloper/.vscode/extensions/dart-code.dart-code-2.25.1/out/src/services/stdio_service.js:72:28)
at Analyzer.sendMessage (/Users/siyuaniosdeveloper/.vscode/extensions/dart-code.dart-code-2.25.1/out/src/analysis/analyzer.js:109:19)
at sendRequest.Promise (/Users/siyuaniosdeveloper/.vscode/extensions/dart-code.dart-code-2.25.1/out/src/services/stdio_service.js:64:18)
at new Promise ()
at Analyzer.sendRequest (/Users/siyuaniosdeveloper/.vscode/extensions/dart-code.dart-code-2.25.1/out/src/services/stdio_service.js:57:16)
at Analyzer.editGetFixes (/Users/siyuaniosdeveloper/.vscode/extensions/dart-code.dart-code-2.25.1/out/src/analysis/analyzer_gen.js:528:21)
at provideCodeActions.Promise (/Users/siyuaniosdeveloper/.vscode/extensions/dart-code.dart-code-2.25.1/out/src/providers/fix_code_action_provider.js:18:27)
at new Promise ()
at FixCodeActionProvider.provideCodeActions (/Users/siyuaniosdeveloper/.vscode/extensions/dart-code.dart-code-2.25.1/out/src/providers/fix_code_action_provider.js:17:16)
at define.provideCodeActions.a.asPromise (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:587:847)
at t.asPromise.Promise (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:49:430)
at new Promise ()
at Object.t.asPromise (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:49:402)
at D.provideCodeActions (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/ex…

It looks like this exception is coming from VS Code itself - the dart plugin.
Through this error (from the analysis server and the underlying VM?) looks suspicious '[Analyzer] Dart_NewStringFromUTF8 expects argument 'str' to be valid UTF-8.'.

/move to Dart-Code/Dart-Code

cc @DanTup

Adding the analyzer label for now until we move the issue.

/move to Dart-Code/Dart-Code


FreeBSD: unable to try with npx

Describe the bug
From https://github.com/sebhildebrandt/systeminformation#give-it-a-try-with-npx:
npx systeminformation info
To Reproduce
Steps to reproduce the behavior:

npx systeminformation info

Current Output
% npx systeminformation info
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y
┌─────────────────────────────────────────────────────────────────────────────────────────┐
│  SYSTEMINFORMATION                                                      Version: 5.21.4 │
└─────────────────────────────────────────────────────────────────────────────────────────┘
/usr/home/grahamperrin/.npm/_npx/e200578d6e6cda24/node_modules/systeminformation/lib/osinfo.js:275
          const logofile = util.getLogoFile(distro);
                                ^

TypeError: util.getLogoFile is not a function
    at /usr/home/grahamperrin/.npm/_npx/e200578d6e6cda24/node_modules/systeminformation/lib/osinfo.js:275:33
    at ChildProcess.exithandler (node:child_process:413:7)
    at ChildProcess.emit (node:events:514:28)
    at maybeClose (node:internal/child_process:1105:16)
    at Socket.<anonymous> (node:internal/child_process:457:11)
    at Socket.emit (node:events:514:28)
    at Pipe.<anonymous> (node:net:337:12)

Node.js v20.5.1
%

Expected behavior

info :-)

Environment (please complete the following information):

systeminformation package version: not applicable
OS: FreeBSD
HP EliteBook 8570p

% date ; uname -aKU
Tue 12 Sep 2023 06:34:51 BST
FreeBSD mowa219-gjp4-8570p-freebsd 15.0-CURRENT FreeBSD 15.0-CURRENT amd64 1500000 #11 main-n265135-07bc20e4740d-dirty: Sat Sep  2 19:40:08 BST 2023     grahamperrin@mowa219-gjp4-8570p-freebsd:/usr/obj/usr/src/amd64.amd64/sys/GENERIC-NODEBUG amd64 1500000 1500000
%

Additional context
Add any other context about the problem here.

@grahamperrin ... thank you for reporting. I need to try this ... will come back with a solution.

I've just hit on this one,
The solution is to remove the util., at row 273. the function getLogoFile is defined locally.
const logofile = util.getLogoFile(distro); becomes const logofile = getLogoFile(distro);

@ovenpasta ... thank you ... sorry, sometimes I am to confused so I have nit seen this simple fix for the issue... Thank you once again. Fixed, version 5.21.11 just released. Can you test It on your side?


content preview input field doesn't allow relative urls

I want to use a relative url /livePreview as content preview url and the collection settings page complains about it because it has an input type="url". If I change it to type="text" in my browser or if I change the pages.collection.php manually, my content preview works as expected.
But now I'm not able to save changes via admin ui because the preview input field complains in a not selected/not visible tab without a visible error message. If I change it back to type="text", I can work with it.
The reason for using a relative url is to keep my local dev setup in sync with the production server without having to change http://localhost/livePreview to https://example.com/livePreview all the time.

@artur Thanks for the fast update, but that doesn' work for me. A quick check in my console gave me:
SITE_URL
"http://localhost/data/cp"
App.base('/')
"/cockpit/"

So site:// and base:// won't work. What I need is http://localhost or an empty string.
My setup:
My frontend CpMultiplane is in the root. Cockpit as the backend is in a subfolder.
cockpit/
    defines.php --> COCKPIT_ENV_ROOT => '../data/cp'
    ...
data/
    cp/
        addons/
        config/
        storage/
    mp/
        ...
defines.php --> MP_ENV_ROOT => './data/mp'
index.php
...

The frontend provides an endpoint /livePreview, which is available under http://localhost/livePreview.
And as I wrote, it works if I change it manually, but then the collection settings page becomes unusable because the form won't submit with an invalid url.

@raffaelj not sure if that helps, but in the past I had a similar issue, I solved that with an override mechanism for content preview urls - https://github.com/pauloamgomes/CockpitCMS-Helpers#content-preview-url-override

@pauloamgomes Thanks for the hint, but I want to avoid overwriting javascript with partials. I used too much hacks and it gets confusing over time.
Also I found a way with the new mechanic: base://../livePreview?token=xx, which translates to /cockpit/../livePreview?token=xx.
My problem is solved, but it feels a bit counterintuitive. So I leave the issue open. I still think, that pure, relative urls should be allowed in a browser-validated url input field.

@raffaelj you can now also do root:// which should resolve to the document root

@aheinze Thanks. This feels more intuitive.


Adding row selection

What type of PR is this? (check all applicable)

[ ] Refactor
[X] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update

Description
Starting work on adding drop down option to select a picker's row.
Related Tickets & Documents

Closes #19

For the attention of @Iranaphor and @amieo-ra

This is only a start, with the GUI element added.

We need to include a callback function onto the dropdown box change. (similar to: https://github.com/LCAS/CallARobot/blob/da0be7a690383626971f37bfbacd576c11418cb0/call_a_robot/www/orders.html#L249C82-L253C74).
This should trigger a function similar to: https://github.com/LCAS/CallARobot/blob/da0be7a690383626971f37bfbacd576c11418cb0/call_a_robot/callarobot.py#L131
For the data to be processed through. I dont think the orders page actually sends gps information through the system so this may also need to copied over from callarobot.html too.

I'd probably just set a cookie with that information and send it through with every GPS update

I'd probably just set a cookie with that information and send it through with every GPS update

that was non-sense... won't work with a cookie, will have to be stored on the server.

OK, I have completed my work here. It appears to be working. The only thing that's not quite working yet is that when the orders page reloads it doesn't show the previous selection. As before, it uses the environment variable CAR_ROWS, so e.g. CAR_ROWS='A1 A2' python call_a_robot/callarobot.py offers 2 rows to select from.
With every position (GPS) update from a client, the row is published with it, so that @nikolauswagner should be able to get it from the regular message sent out along with the latitude and longitude.
The devcontainer should allow to use as indicated above. with ports 8127 and 8128 needed for testing (http and websocket).

I have merged, if there are any issues with it, please open a new ticket.


tweet hidden behind filter indicator

Fixed this issue. Please wait while I fix up some other small issues, and I'll sync it with the rest of the source tree.


Update version of Newtonsoft.Json in sample

OPS Build status updates of commit ce8e2e0:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: OPS Build is busy, click here to report any potential performance downgrade issues, or you could view OPS Portal for trend of build queue length.

OPS Build status updates of commit ce8e2e0:
:clock10: Preparing: average preparing time is 39 sec(s)

OPS Build status updates of commit ce8e2e0:
:clock10: Incremental building: average incremental building time is 21 sec(s)

OPS Build status updates of commit ce8e2e0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/quickstart/install-and-use-a-package-using-the-dotnet-cli.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Chores/plot distance visualization

Review some of the examples.

Adopt the convention "noplot" for example which are not ran on CI.
Plot distance visualization on the doc (as we now have pyRiemann 0.6)
Fix labels for the toys dataset comparison

Thx @qbarthelemy !


Missing sync between Riak and Traffic-Ops on DS CDN change

Version: On TC 2.1
I created an https delivery service in cdn "x".
It was then moved to cdn "y".
However, even when creating new ssl keys for the DS, the keys did not appear in
https://ops.ocn-tc3.tc-dev.cqloud.com/api/1.2/cdns/name/y/sslkeys.json
Looking into the riak, I see that the keys are related to CDN "x".
As a workaround we can block the option to change DS cdn

Another option is to add the cdn to the key used for holding the sslkeys in riak.
However, this will create issues for old deployments

seems like making cdn immutable on ds update is the easiest fix. doesn't seem like something you'd need to do very often - change the cdn for a ds but i could be wrong.


"Bulgăruș" and "Grabaț" are listed in the drop-down menu

Description:
"Bulgăruș" and "Grabaț" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Bulgăruș" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Bulgăruș, TIMIȘ (Lenauheim)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Grabaț" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Grabaț, TIMIȘ (Lenauheim)".


Add .gitignore to examples

Problem
I use create-next-app with many examples, initialize git and add everything to start working, turns out node_modules got committed as well.
Almost every example here doesn't have .gitignore.
Solution
Add .gitignore to every example, ignoring node_modules, .next, and out.

@lukyth Hi, thank you for the PR, I don't think examples should contain a .gitignore too, they use the gitignore of the repo and it's enough 👍
@Timer Adding a .gitignore to the example generated by create-next-app can be useful, wdyt?

We'd want to start doing it with https://github.com/zeit/next.js/pull/9077!

Didn't know the work on create-next-app is on the way. That'd solve my issue nicely! Thank you so much :)


UI-Router 1.x

After upgrading to angular-ui-router 1.x it seems that angular-permission is no longer working properly.  I'm also using polyfill that re-enables the $stateChangeStart angular events but I still have problems when I'm using $transitions and redirectTo is not working anymore. Is there any chance of migrating to angular-ui-router 1.x?

It works now for me :), in my case I was working with version 5.1 of angular-permission, and I upgraded it to the latest version, there was no need to include the polyfill to re-enable the $stateChangeStart event.

If it is working for @valentina91p  I guess it will work for me too then. I will check it today, but you can close the issue then. Thank you.


blockexplorer

hello there, when do you have the block explorer for this?

Hi jjhesk. No block explorer at the moment. We will have a one ready by the time that the public testnet is launched later during the year! We are really looking forward to this.


Update Blade & Sorcery save parser to 1.0

Fixes errors in Blade & Sorcery when parsing saves. Didn't bothered with old saves because they are not supported by the game anymore.

This needs to be fixed for the linting issues; feel free to bump the version as well

Not sure to what version to bump to, made it 0.5.1


Bump IAVL version from v0.19.4 -> v0.19.7

Closes: #XXX
What is the purpose of the change
Bumps IAVL version from v0.19.4 -> v0.19.7.
Also updates sdk commit hash accordingly to the commit that uses the bumped version of IAVL
Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?
[ ] Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

[ ] Specification (x/{module}/README.md)
[ ] Osmosis documentation site
[ ] Code comments?
[ ] N/A

devbot help

Hi! I'm DevBot, a bot that helps with common tasks in the development process. Commands:

devbot fix: Fix basic errors in the PR. (e.g. imports, changelog conflicts, go mod conflicts)
devbot merge base: Merge the base branch into the PR branch.
devbot add changelog [misc/feat/bug/api/sec] [message]: Add a changelog entry to the PR. (e.g. devbot add changelog feat Added a new feature)

If message is blank, defaults to PR title.

devbot re-pr: Re-PR the PR, useful for external contributors where we have no edit perms.

devbot add changelog misc bump IAVL version to v0.19.7

nicejob devbot

Currently running a node on v19 release line

The nodee synched over epoch - merging


Include build options for Protobuf_Runtime

Include options for Compile_Checks, Runtime_Checks, Style_Checks and Contracts_Checks (enabled or disabled).
Include option for Build_Mode (optimize or debug).
Based on what Alire generates as initial GPR file and flags already present (-gnat12 and -gnatW8).

Alire uses different build modes:
   type Build_Profile_Kind is ("release", "validation", "development");
   Build_Profile : Build_Profile_Kind := "development";

In fact I got that from mgrojo/ASFML  and that in turn was obtained from the Alire init template from that time, but I suppose meanwhile Alire has evolved to the three modes.

Probably, we should reuse alire generated configure project.

It makes sense. Again, I took inspiration from an older Alire output and config project was not invented then, but it is cleaner what Alire is doing now.

Or/and move protobuf runtime to a dedicated crate. What do you think?

I have only used Alire for pure lib or pure application repositories, so I wouldn't be able to give an informed answer.


feat(lightbox-media-viewer): add carousel functionality

Related Ticket(s)
Closes #6372
Description
This PR adds a new <dds-lightbox-media-viewer> component that is capable of replacing the <dds-lightbox-image-viewer> and <dds-lightbox-video-player> components, however those components have not been removed and should continue working. Additionally, the carousel's styles have been updated to allow it to be used within a modal.
Changelog
New

Added <dds-lightbox-media-viewer> which can show either a video file or an image

Changed

Carousels should now display properly within the expressive-modal component

The pause functionality should be there, but I'm assuming didn't get deployed because of the ci failures. I'll try to iron those out

@proeung the videos should now be pausing when the slide away or when the modal opens

@andy-blum Looks good! The video pauses when advancing to the next slide. One more thing, can you update the README file as mentioned here - https://github.com/carbon-design-system/carbon-for-ibm-dotcom/pull/8757#discussion_r862995303

@kennylam I'd love to get your review for this PR. Also, @oliviaflory let me know what you think about the second line item that I've noted here (https://github.com/carbon-design-system/carbon-for-ibm-dotcom/pull/8757#pullrequestreview-959283526). Thanks!

@andy-blum oh that was meant to be a general comment since I don't think I was seeing them at either breakpoint

@proeung very sorry I missed your comment, this got lost in my emails 😞

This might not be part of the scope of work for this feature issue, but we'll need to be able to pass some of these lightbox attributes to components like Tabs extend with media so that this carousel feature is available for use. cc: @oliviaflory

By passing into tabs extended with media, does that mean

each tab might have multiple videos/images that the user can carousel through
the carousel would take them through the different tabs content within the lightbox?

@oliviaflory No worries! I know that you have a lot on your plate right now.

each tab might have multiple videos/images that the user can carousel through
the carousel would take them through the content of the different tabs within the lightbox?

I believe it's more of scenario number 2 where the carousel would take them through the content of the different tabs. However, I'll defer to @jwitkowski79 since I believe she has more insight on the functional needs of the component mentioned here - https://github.com/carbon-design-system/carbon-for-ibm-dotcom/pull/8757#pullrequestreview-959283526.

@emyarod I'm struggling a bit with the edge case where a video is taller than its container in a modal. The video component is using the absolute-positioned-over-a-container-with-56.25%-padding trick.
Do you think there's the possibility of a style refactor for the video components to utilize the aspect-ratio property so the positioning issues could be resolved?

@emyarod @kennylam
Looks like tests are failing on types ResizeObserver and ResizeObserverCallback. I can see we've used // @ts-ignore in other areas of the codebase, should we open a follow up issue to import the @types/resize-observer-browser package and remove the TODO's in the codebase?

@oliviaflory @jwitkowski79 I just opened a new feature request issue for enabling the Media Lightbox Media Carousel to the "Tab Extended with Media" component. I figured that we'll need to review and define this feature before the work can be moved to the C4IBM's dev pipeline. https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/8826
In the meantime, the baseline carousel feature in this PR is ready for more design QA and code approval. @emyarod @kennylam @oliviaflory Can you all help take a look? Thanks!

@andy-blum this is looking good overall from design perspective.
1. Fix medium and small breakpoint horizontal scroll
I would have expected the image and video to be contained within the width of the viewport on the medium and small breakpoints, matching the lightbox media viewer behavior.
Currently the Lightbox media viewer with carousel has a horizontal scroll and the user cannot view the whole image at once.
Current

Expecting

2. Keyboard navigation
I'd love to get some accessibility representation to look at the keyboard navigation for screen readers. I was able to access everything in the first page, but switching between the other carousel pages and accessing that content was more challenging. I'm probably not the best person to be testing/making recommendations for that behavior though!

@proeung

I believe it's more of scenario number 2 where the carousel would take them through the content of the different tabs. However, I'll defer to @jwitkowski79 since I believe she has more insight on the functional needs of the component mentioned here - https://github.com/carbon-design-system/carbon-for-ibm-dotcom/pull/8757#pullrequestreview-959283526.

That makes sense, thanks for sharing! I think we will need to consider where the user lands once they close the carousel. I'm assuming if they closed the carousel at slide 3/5, they should land on tab 3/5 within the page. I can comment on the issue you tagged above as well.

Thanks for the horizontal scroll fixes @andy-blum I will keep an eye out for the Percy checks so we can get this in!

@andy-blum seeing a change in Percy, would anything changed in the lightbox media viewer have affected the Expressive modal? I think the lightbox media viewer used the expressive modal component.

@oliviaflory yeah, that's possible. I'll look into it and see what might be causing that

@oliviaflory Oh, yeah!
I fixed this typo in the code, but didn't update the corresponding style. Fix incoming.
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/pull/8757/files#diff-d7e1e2cc7aa32e854a58956a75e1969893fc24c3008fbc7c1d27f46111fccc85R315

@andy-blum last comment but otherwise looks good! Let me know if you don't have time to get this in before code freeze and we can open a bug to fix after instead.
Can we add 16px of padding below the carousel controls so they buttons are not touching the bottom of the modal? Similar to how the Expressive modal component is styled.

it looks like those react e2e tests are failing on main too when I run it locally so it may be fine to ignore in this PR. other than the spacing issue mentioned above, this PR should be good to go!

This looks good to me as well. The only fault I can find is when there's vertically overflowing content at the small breakpoint - the scrollbar overlaps with the modal's close button, but IMO that's a followup issue

@andy-blum ah I see, that could be why the position rule was originally added. we could also consider updating the e2e test so that the width constraint accounts for the scroll bar now

@emyarod Thanks for following up on this PR! I guess the only thing that's blocking this PR from being merged down is the e2e test. Is this something that you can help out with? I'd love to include this in the code freeze period for v.1.35.0 release.

@proeung currently taking a look! react e2e was fixed but the same error is now appearing in web-components instead e2e :sob:

@proeung currently taking a look! react e2e was fixed but the same error is now appearing in web-components instead e2e 😭

@emyarod I'm sorry! I hate when this happens. @andy-blum If you have time today, can you help resolve this e2e error so that we can get this work out for the next RC? @kennylam is on stand-by and holding the RC until this PR is merged.

I've been looking into the test failure, and there doesn't seem to be an issue locally. Do we have access to the screenshots taken by this same test?

@andy-blum I overlooked that the test assertions were changed earlier in the PR so I just reverted the assertions. I think it makes sense to leave them as is given the most recent spacing changes requested by Olivia too. I also reverted the position rule change to hopefully address the scrollbar overlap you mentioned previously


uv torch installation breaking in cpu-gpu multiplatform scenario.

First of all thanks for the tool, UV is simplifying immensely the package management.
I was following the new document you provided for using UV with PyTorch, and I think I could have found an edge case that breaks the package resolution. Some of the last issues (#9306 and #9307) are possibly related, but I will report my specific case since it is a "cleaner" situation and could provide you with a new data point.
I'm using the last version (0.5.4).
I have this scenario:
I need a project that installs torch and torchvision ("torch==2.5.1", "torchvision==0.20.1"), and I need to account for three different scenarios: Darwin os (obv always without GPU), and Linux os (both with and without GPU).
From my understanding following the guide, I should be able to use uv sync --extra cpu and uv sync --extra gpu, installing the proper torch version with this pyproject.toml:
[project]
name = "test"
version = "0.1.0"
description = "Test pytorch dependencies"
readme = "README.md"
requires-python = ">=3.11"
dependencies = []

[project.optional-dependencies]
cpu = [
  "torch==2.5.1",
  "torchvision==0.20.1",
]
gpu = [
  "torch==2.5.1",
  "torchvision==0.20.1",
]

[tool]
[tool.uv]
conflicts = [
  [
    { extra = "cpu" },
    { extra = "gpu" },
  ],
]

[tool.uv.sources]
[[tool.uv.sources.torch]]
index = "pytorch-cpu"
marker = "platform_system != 'Darwin'"
extra = "cpu"

[[tool.uv.sources.torchvision]]
index = "pytorch-cpu"
marker = "platform_system != 'Darwin'"
extra = "cpu"

[[tool.uv.index]]
name = "pytorch-cpu"
url = "https://download.pytorch.org/whl/cpu"
explicit = true

However, it works well locally on Darwin (both --extra CPU and --extra GPU), but when I try to install it on Linux with --extra CPU I get this error:
$ uv sync --extra cpu
Using CPython 3.11.9 interpreter at: /usr/local/bin/python3
Creating virtual environment at: .venv
Resolved 29 packages in 2ms
error: Distribution `torch==2.5.1 @ registry+https://download.pytorch.org/whl/cpu` can't be installed because it doesn't have a source distribution or wheel for the current platform

Notice that if I use --extra GPU, instead, not having an explicit extra index and falling back to pypi works correctly, and the installation of GPU torch with Nvidia packages is solved correctly:
$ uv sync --extra gpu
Using CPython 3.11.9 interpreter at: /usr/local/bin/python3
Creating virtual environment at: .venv
Resolved 29 packages in 2ms
Prepared 25 packages in 58.28s
warning: Failed to hardlink files; falling back to full copy. This may lead to degraded performance.
         If the cache and target directories are on different filesystems, hardlinking may not be supported.
         If this is intentional, set `export UV_LINK_MODE=copy` or use `--link-mode=copy` to suppress this warning.
Installed 25 packages in 15.22s
 + filelock==3.16.1
 + fsspec==2024.10.0
 + jinja2==3.1.4
 + markupsafe==3.0.2
 + mpmath==1.3.0
 + networkx==3.4.2
 + numpy==2.1.3
 + nvidia-cublas-cu12==<IP_ADDRESS>
 + nvidia-cuda-cupti-cu12==12.4.127
 + nvidia-cuda-nvrtc-cu12==12.4.127
 + nvidia-cuda-runtime-cu12==12.4.127
 + nvidia-cudnn-cu12==<IP_ADDRESS>
 + nvidia-cufft-cu12==<IP_ADDRESS>
 + nvidia-curand-cu12==<IP_ADDRESS>[47](https://gitlab.pepita.io/core-aide/watergate/temp/-/jobs/3465971#L47)
 + nvidia-cusolver-cu12==<IP_ADDRESS>
 + nvidia-cusparse-cu12==<IP_ADDRESS>
 + nvidia-nccl-cu12==2.21.5
 + nvidia-nvjitlink-cu12==12.4.127
 + nvidia-nvtx-cu12==12.4.127
 + pillow==11.0.0
 + sympy==1.13.1
 + torch==2.5.1
 + torchvision==0.20.1
 + triton==3.1.0
 + typing-extensions==4.12.2

The problem is not with the extra index, since if I accept the situation where I only install cpu on Linux (like on Darwin), the extra index works correctly with this .toml:
[project]
name = "test"
version = "0.1.0"
description = "Test pytorch dependencies"
readme = "README.md"
requires-python = ">=3.11"
dependencies = ["torch==2.5.1", "torchvision==0.20.1"]

[tool]
[tool.uv]
[tool.uv.sources]
[[tool.uv.sources.torch]]
index = "pytorch-cpu"
marker = "platform_system != 'Darwin'"

[[tool.uv.sources.torchvision]]
index = "pytorch-cpu"
marker = "platform_system != 'Darwin'"

[[tool.uv.index]]
name = "pytorch-cpu"
url = "https://download.pytorch.org/whl/cpu"
explicit = true

Notice, however, that this toml only works after I delete and re-generate (with uv sync) the uv.lock, otherwise I have the same error.
I think that the diff of the two locks could show the problem, since the freshly generated lock has the correct x.y.z+cpu version for torch and torch vision, while the old one did not.

Thanks in advance for the help (:

This is a duplicate of #9295 and should be fixed in the next release

sorry, missed that.
Thanks again for all the good work


bug: Floating Labels and Numeric Inputs

Type: bug
Ionic Version: 2.x
Platform: mobile webview
Greetings!
There seems to be a problem combining a floating ion-label with a numeric ion-input. When starting with an empty value, the ion-label moves to the top when the ion-input is focused to allow room for a value to be entered, and stays on top when a valid numeric value is entered. That works as expected. However, when entering a value like 0-0-0, which is an invalid numeric value, the ion-label goes back to its original position and overlaps the value that was entered.
Here's a sample template to reproduce the issue:
<ion-list>
	<ion-item>
		<ion-label floating>Label</ion-label>
		<ion-input type="number" inputmode="numeric" pattern="[0-9]*"></ion-input>
	</ion-item>
</ion-list>

By the way, thank you for all of your hard work in creating such a wonderful framework!

Thanks for bringing this issue up, we will look into this!

I've been able to reproduce the issue using Ionic 3.1.1. When invalid value is provided into number input the floating labels overlaps the text.

I'm also having this issue with Ionic 3.5.2

I'm also having this issue with Ionic 3.7.1


Ionic text field clear button

Sorry if this is a duplicate, but it would be awesome if there were an ionic module for adding a text field clear button. This is a great mobile heuristic and very useful.
example:
http://stackoverflow.com/questions/2803532/how-do-i-put-a-clear-button-inside-my-html-text-input-box-like-the-iphone-does

Here's a good discussion that relates to this: http://forum.ionicframework.com/t/cannot-tap-click-a-clear-search-icon-inside-an-input-label/4237/8
Use the same technique: Either rename the enclosing <label class="item-input-wrapper> to be a , or use the "on-touch" directive instead of "ng-click".


Reduce code duplication between String.match and search

Moved the common preparation code for 'search' and 'match'
to a separate function.
JerryScript-DCO-1.0-Signed-off-by: Mátyás Mustoha<EMAIL_ADDRESS>

How much is the gain?

About 35 lines.

Good reply :D
I mean binary size reduction. Since this is an optimization patch I am curious about its effect.

Ah ok, here's what I see using the default build parameters:
before:

File
Size

build/bin/jerry
217880

build/lib/libjerry-core.a
3301510

build/lib/libjerry-ext.a
141244

build/lib/libjerry-libc.a
63484

build/lib/libjerry-libm.a
139790

build/lib/libjerry-port-default.a
32248

build/lib/libjerry-port-default-minimal.a
28948

after:

File
Size

build/bin/jerry
217880

build/lib/libjerry-core.a
3302310

build/lib/libjerry-ext.a
141244

build/lib/libjerry-libc.a
63468

build/lib/libjerry-libm.a
139758

build/lib/libjerry-port-default.a
32264

build/lib/libjerry-port-default-minimal.a
28956

Seems there's no real size gain :(

No problem. It is still a good patch.

Rebased to master to fix the Travis build.


gradle-plugin: add task to merge sarif reports

What's done:

Added task to merge SARIF reports

This pull request closes #1452

Please add readme, no need to think about tests :)
We will test it manually


Improve --json behaviour with interactive prompts

What steps did you take:
Create an example.yaml file containing:
apiVersion: v1
kind: ConfigMap
metadata:
  name: xxx
data:
  xyz: abc

Run:
kapp deploy -a website -f example.yaml --json

What happened:
It appeared to crash and generate a Golang stack trace.
{
    "Tables": [
        {
            "Content": "",
            "Header": {
                "age": "Age",
                "conditions": "Conds.",
                "kind": "Kind",
                "name": "Name",
                "namespace": "Namespace",
                "op": "Op",
                "op_strategy": "Op st.",
                "reconcile_info": "Ri",
                "reconcile_state": "Rs",
                "wait_to": "Wait to"
            },
            "Rows": [
                {
                    "age": "",
                    "conditions": "",
                    "kind": "ConfigMap",
                    "name": "xxx",
                    "namespace": "lab-carvel-ytt-01-w01-s002",
                    "op": "create",
                    "op_strategy": "",
                    "reconcile_info": "",
                    "reconcile_state": "",
                    "wait_to": "reconcile"
                }
            ],
            "Notes": [
                "Op:      1 create, 0 delete, 0 update, 0 noop",
                "Wait to: 1 reconcile, 0 delete, 0 noop"
            ]
        }
    ],
    "Blocks": null,
    "Lines": [
        "Target cluster 'https://<IP_ADDRESS>:443'"
    ]
}panic: Cannot ask for confirmation in JSON UI

goroutine 1 [running]:
github.com/cppforlife/go-cli-ui/ui.(*JSONUI).AskForConfirmation(0xc0003daa00, 0x1535358, 0xc000cad1d0)
        github.com/cppforlife/go-cli-ui@v0.0.0-20200716203538-1e47f820817f/ui/json_ui.go:114 +0x39
github.com/cppforlife/go-cli-ui/ui.(*ConfUI).AskForConfirmation(0xc0003ebd80, 0xc0001be140, 0xc0000791a0)
        github.com/cppforlife/go-cli-ui@v0.0.0-20200716203538-1e47f820817f/ui/conf_ui.go:105 +0x33
github.com/k14s/kapp/pkg/kapp/cmd/app.(*DeployOptions).Run(0xc00012cc80, 0x0, 0x0)
        github.com/k14s/kapp/pkg/kapp/cmd/app/deploy.go:171 +0xc8f
github.com/k14s/kapp/pkg/kapp/cmd/app.NewDeployCmd.func1(0xc00013f340, 0xc00007df90, 0x0, 0x5, 0xc000374580, 0x1527ef0)
        github.com/k14s/kapp/pkg/kapp/cmd/app/deploy.go:54 +0x2a
github.com/k14s/kapp/pkg/kapp/cmd.NewKappCmd.func1.1(0xc00013f340, 0xc00007df90, 0x0, 0x5, 0x0, 0x0)
        github.com/k14s/kapp/pkg/kapp/cmd/kapp.go:126 +0xb8
github.com/cppforlife/cobrautil.WrapRunEForCmd.func1.1(0xc00013f340, 0xc00007df90, 0x0, 0x5, 0x0, 0x0)
        github.com/cppforlife/cobrautil@v0.0.0-20200514214827-bb86e6965d72/misc.go:45 +0xb2
github.com/cppforlife/cobrautil.WrapRunEForCmd.func1.1(0xc00013f340, 0xc00007df90, 0x0, 0x5, 0x0, 0x0)
        github.com/cppforlife/cobrautil@v0.0.0-20200514214827-bb86e6965d72/misc.go:45 +0xb2
github.com/cppforlife/cobrautil.WrapRunEForCmd.func1.1(0xc00013f340, 0xc00007df90, 0x0, 0x5, 0x0, 0x0)
        github.com/cppforlife/cobrautil@v0.0.0-20200514214827-bb86e6965d72/misc.go:45 +0xb2
github.com/spf13/cobra.(*Command).execute(0xc00013f340, 0xc00007df40, 0x5, 0x5, 0xc00013f340, 0xc00007df40)
        github.com/spf13/cobra@v1.1.1/command.go:850 +0x472
github.com/spf13/cobra.(*Command).ExecuteC(0xc00013e580, 0xc00013e580, 0xc0003ebd80, 0x7f73f8ce0108)
        github.com/spf13/cobra@v1.1.1/command.go:958 +0x375
github.com/spf13/cobra.(*Command).Execute(...)
        github.com/spf13/cobra@v1.1.1/command.go:895
main.nonExitingMain(0x0, 0x0)
        github.com/k14s/kapp/cmd/kapp/kapp.go:42 +0x167
main.main()
        github.com/k14s/kapp/cmd/kapp/kapp.go:21 +0x26

What did you expect:
Am not sure. I was trying to work out what the --json option actually did when it occurred, so not sure if this is expected and normal, but it sort of looks wrong.
Anything else you would like to add:
Nope.
Environment:
$ kapp --version
kapp version 0.37.0

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.
cc @jorgemoralespou

--json is incompatible with interactive UI (that's what this error is trying to message panic: Cannot ask for confirmation in JSON UI). you either have to use -y for the full deploy or --diff-run since it ends before confirmation prompt pops in.

If the flags are incompatible, or -y is required, the correct thing to do is to show that message and not a json rendered output and a panic.
Cannot ask for confirmation in JSON UI. Either use `-y` flag or remove `--json` format

This should be the only output. It should not even try to deploy and fail.

We may have look at making changes generically in the library used: https://github.com/cppforlife/go-cli-ui


Rate Limited

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

📝 Description
Not sure if I'm popular, or what, but it seems I keep hitting the Rate Limited page pretty often
Is there a way to expand the limit?
I have a token dedicated specifically to Gitify
🪜 Steps To Reproduce
unknown
Gitify Version
5.9.0
Operating System
Windows
GitHub Account
GitHub Cloud
📸 Screenshots

@pedro-mass - not until we implement #1124
Until then, I would suggest:

disabling the Detailed notifications setting
reducing the amount of unread notifications in your inbox
ensuring this token is only used on one device


docker-machine restart (with generic driver) removes machine access

Or at least docker-machine ls says it's in a Stopped state and I also can't start it any more...
generic driver does not support start

I'd say docker-machine restart should not work with the generic driver and output an error, like start.
docker-machine version 0.8.1, build 41b3b25o

Are you sure the machine got in a stopped state after docker-machine restart and not something else?
With the generic driver restart should work fine (cf. https://github.com/docker/machine/blob/008283733e9be4c89844ce1ccbee0f944bbbde3d/drivers/generic/generic.go#L174).

Thanks for the link, this might be related to the underlying Vagrant VM in my case.
I just tried a manual reboot from within the VM and ran into similar problems.
I'd expect something like sudo /etc/init.d/docker restart here (for the generic driver)


Catching GetAllVersions Exception

Codecov Report

Merging #399 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #399   +/-   ##
=======================================
  Coverage   91.34%   91.34%
=======================================
  Files         139      139
  Lines        3015     3015
  Branches      137      136    -1
=======================================
  Hits         2754     2754
  Misses        261      261

Impacted Files
Coverage Δ

...ain/scala/hydra/avro/registry/SchemaRegistry.scala
92.85% <100.00%> (ø)

Continue to review full report at Codecov.

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


Inconsistent K in Latin and Cyrillic

In Roboto 2015 latin letter K was updated, but the cyrillic one wasn't. They should be the same.

It should look like that:

This should be addressed as well as К-related Cyrillic letters like, i.e. Ж. Please release hinted version because release v2.138 never received hinting treatment.

Why should they be the same? They are not the same in almost all fonts. And I believe they shouldn't cause they are different letters of different alphabets, and the more they differ, the better.
https://fonts.google.com/?preview.text=KК&preview.text_type=custom

Correct. This is not a bug. For this particular letter, it is expected, and advisable even, that a homoglyph is avoided by having different constructions in the Latin, the Greek and the Cyrillic renditions.


fix(secondary-button): define active text color

Overview
Resolves #228
SCSS fix that defines an :active button text color for secondary-button.
Testing / Reviewing
Click and hold on a Secondary Button without releasing.
Or in Chrome dev tools, select both the :focus and :active pseudo-classes

👍 ✅

👍 ✅


File change not recognized by VFS on Windows when entire watched directory is removed

Similar to https://github.com/gradle/gradle/issues/11833 we don't recognize when a watched directory is removed in its entirety (e.g. when its parent directory is deleted).

cc: @gradle/build-cache

This should now be fixed in https://github.com/gradle/native-platform/pull/131. When the whole watched directory is deleted, upon re-watching via ReadDirectoryChangesW() we get an ERROR_ACCESS_DENIED error code. Then we check if the directory is still there, and if not, we report it gone. Simple.


Table Sort / Bulma / Mobile style

I'm using @next and bulma, however I've not able to achieve the same style as Buefy with the mobile table sort.

vs.

Is this possible? Oruga generates html, with a number of elements that are not required/expected (?) and cannot be edited from vie the component.
eg.

Thanks,
Bob

@bobmcallan we are working on Bulma integration as you can see https://github.com/oruga-ui/oruga-bulma
Probably the Buefy style has to edited in order to work well with Oruga as I did with other components (steps).
At this point we came discuss about in the other repository.


Integrate the K8s rules controller in testing frameworks

The K8s rules controller needs to be properly integrated in testing frameworks and run tests for every PR

@elevran: is this with respect to using Third Party Resources to store rules in our tests?

P1 - Needed for Armada

Created PR https://github.com/amalgam8/amalgam8/pull/510 for helloworld automation.

PR https://github.com/amalgam8/amalgam8/pull/513 for bookinfo automation has been merged.


Pass epoch to committee & validate on connection.

I've tried a few different directions but I found my self having to do some bigger refactoring to work (ex like wrapping the network message on a wrapper type with metadata to pass down epoch). There are alternatives here but found that the least intrusive without having to also touch part of the Network component.

Let's discuss this little bit
We have to include epoch to the StatementBlock, because otherwise it's a vulnerability from BFT perspective - malicious validator can claim he is on new epoch and feed signed blocks from old epoch to other validators, thus breaking the protocol
Now after we do this, I think for practical point of view things will just work...
Then we could some time later just change code a bit to include epoch to the handshake (so we avoid extra round trip during connection introduced by message in this PR). This I think can be done with much lower priority, perhaps when we decided more firmly on which network stack to use
What do you think?

We have to include epoch to the StatementBlock, because otherwise it's a vulnerability from BFT perspective - malicious validator can claim he is on new epoch and feed signed blocks from old epoch to other validators, thus breaking the protocol

100% agreed, this would be done as separate PR as we want to do anyways for BFT reasons as you said.

Now after we do this, I think for practical point of view things will just work...
Then we could some time later just change code a bit to include epoch to the handshake (so we avoid extra round trip during connection introduced by message in this PR). This I think can be done with much lower priority, perhaps when we decided more firmly on which network stack to use
What do you think?

My intention was to minimise any possible aftereffects from having any sort of communication with validators of different epoch during reconfig and get us to a close parity with NW. Sure we can do things more efficiently but for now just wanted to have something in place, that's I chose the path of least changes. If you believe that we can live without this PR I am happy to drop it.

Closing for now. Talked offline about it as there is not immediate need for it for functional level.


Update README.md: Adds SirixDB

SUMMARY

Describe your changes, including rationale and design decisions
If your pull request includes new functionality, then you must have corresponding new tests.

Automated Checks

[x] I have run ./gradlew test and made sure that my PR does not break any unit test.
[x] I have run ./gradlew checkstyleMain or the equivalent and corrected the formatting warnings reported.

Fantastic !
Merging.


📃 add detekt-generator module to generate documentation and default config

Resolves #343
This PR:

Creates a new module detekt-generator which is a new CLI module that compiles sources from the detekt-rules module and scans them for relevant content to be put into the documentation.
Multiple collectors collect data on RuleSetProviders, Rules and MultiRules to compose a list of all necessary data to print the documentation and the default-content.
Printers then take this data and print a *.md file for the documentation of each Rule Set Provider and it's rules and their configuration options.
Also a default-detekt-config.yml is generated.
I touched a couple of rules and added the necessary KDoc for them to get at least some documentation generated.

Currently I still included the generated documentation and the default config. But they're mostly useless at the moment as we only have very few rules documented in the correct style.
Next steps would be adding the necessary KDoc so that we get a full documentation generated from the script.
Currently missing: Tests (working on it, but wanted to share this first for feedback)

@schalkms Yep, you're totally correct that the printing part could be a bit nicer. For this first version I mostly focussed on gathering all the necessary data with the collectors.
After the collection process we already have a list of RuleSetPage which we could use to also generate other formats. The necessary data should already be there. We would only need some different printers.

Yep, I like it! For an indepth review I need more time but looks fine so far! I like the rule template which we can mention in contributing.md so every new rule must confirm to it. An internal dsl sounds also good. More to learn and play with :).

Gonna add a small yaml DSL as well to clean up the config printer, then some tests and then this is good to go 👍

👍

@schalkms I think this is good to go for now. We can still improve it later on.
Only question is where to put the generate documentation and the default detekt config this generates for now.
Next steps are getting all the KDoc ready so that we get rid of all the TODOs in the documentation 🙃

Right. I agree.

Awesome work looks good with all the dls and stuff :D
Will check it out locally too ^^

One thing:  The rule sources path is hardcoded to private const val RULES_SOURCES_PATH = "./detekt-rules/src/main/kotlin". This makes it only possible to run the generator from the project path and not the generator sub project. Should this be configurable? -> You've created an Args object which for now has only the help parameter?

Yeah, I hardcoded it because it only really makes sense at the moment to run the generator over the rules module.
We can make it a CLI arg (the Args class is mostly a leftover from when I copy pasted the -cli module) but I don't think the generator module needs to be super configurable as it's mostly an internal utility.

Yep, then lets delete the Args class + add a require and check if we are in the top project and detekt-rules exist?

@arturbosch Sounds good. Will do that.
WDYT about the directory where the generated files should live? Currently I just threw them into the generator module for debugging. But I guess they should live somewhere else.

I think it is ok for now. Well we could move it to top level so all github repo visitors see a documentation folder and can take a look at it. Later we will integrate the doc files into jekyll/jbake or something.


Uncaught TypeError: Cannot read property 'body' of undefined

When dragging nodes in the graph, after a while it produces this error in barnes_hut.coffee at line 36 where root is undefined.

Thank you for the report! Actually, I really need to improve the force-directed graph. Right now, it is merely a toy, and it is the only chart that I do not consider usable in a real world setting.
This is top on my priority list, but unfortunately I did not have the time to work on Path recently - I hope to change this soon


Azure CLI install on tinycore question.

Environment summary
The machine is running Mac OSx, with Docker toolbox that starts up a VM in virtual box.
VM is a tinycore default VM.
How can I install Azure CLI on the VM?
Tried https://docs.microsoft.com/en-us/cli/azure/install-azure-cli-linux?view=azure-cli-latest. But I must be doing some pre req step wrong since I cannot get it to work.
Any help would be greatly appreciated.

@RavSalgado, mind providing more information about the problem you're seeing? What error pops up when you try to install? Also, what OS is the VM running?

Sure, The VM runs a custom Linux build, tinycore. I run through the python, libffi and OpenSSL install, then the script "curl -L https://aka.ms/InstallAzureCli | bash" cmd.
Am I missing something here?
I'v atached a screen shott.

Thanks.

Ah, got it. This is actually a dup of https://github.com/Azure/azure-cli/issues/8353 and will get fixed when the next version drops. In the mean-time you should be able to work around it by download the install script from https://aka.ms/InstallAzureCli and modifying the line that uses mktemp to:
install_script=$(mktemp -t azure_cli_install_tmp_XXXXXX) || exit

Note: the differences is two extra X characters in the directory name template.

Once that is done, you should be able to install the CLI by running the following from the directory where you downloaded the script:
. install

Thanks, @marstr I had to do a few extra steps and set the $PATH, but now AZ is running without issue. Thank you for your help. closing issue.


Add Ext2Fsd

Ext2Fsd  is an ext2 file system driver for Windows under GPL2.

The latest executable is at
https://github.com/matt-wu/Ext3Fsd
This app requires admin rights to install.


s3:HeadObject is not a valid AWS IAM policy Action

Problem description
Deploying titiler with CDK gives a warning that s3:HeadObject is not an S3 Action. This is because it is not per the documentation here: https://docs.aws.amazon.com/service-authorization/latest/reference/list_amazons3.html
The fix is to simply remove this Action from the policy definition(s), as s3:GetObject also allows the head operation.
Expected Output
No warning about policy actions.
Environment Information
n/a

@philvarner I don't see any mention of HeadObject in titiler code https://github.com/developmentseed/titiler/blob/2168c9284b39a46c4d1a095542c77addc690a738/deployment/aws/cdk/app.py#L168-L177
we updated this 3 month ago https://github.com/developmentseed/titiler/pull/330/files#diff-ce02ebe86b6992fd0e07623a204ba559d8654e02518b6d6da2b98a6000e7ab25L172

Sorry! I was looking at an old version of the code.