HTTP endpoint to trigger Docker service sync

One possible solution to #22 without introducing an explicit heartbeat (a feature called out in the third paragraph of the README) is to allow users to trigger a resync of one or all Docker containers via the existing HTTP interface. Users would have the choice to either manually trigger resync after container renames (or other container metadata changes) or to automatically trigger the resync via a cron-type scheduled mechanism.
Some notes:

The sync does not update the list of containers, just the metadata of containers known to exist.
Sync will overwrite any manual changes to the service.
General sync (the /sync endpoint) should ignore any services that are not tied to containers.

So there is no rename event in the docker events stream, right? Is there an issue for this in the Docker repo? If not then we should probably create one. Seems like this info should be exposed by docker events.

I raised docker/docker#14407 to have rename events emitted by the Docker events stream. For what it's worth, I thought of two other reasons why this function might still be warranted:

Users may want to 'revert' any manual service changes they have made.
I wasn't sure if Docker will eventually allow you to dynamically change other metadata fields such as environment variables or image names, and I thought a general sync mechanism was the simplest way to cover any such future possibilities.

Users may want to 'revert' any manual service changes they have made.

Maybe reload would be a better action then. And we need to clear previous containers. Don't think that is happening at the moment.

if Docker will eventually allow you to dynamically change other metadata

I don't think we need to worry about that. If they do that they would probably add something like a change event also.

I can see how 'reload' would make more sense. Before I update the PR though, can you elaborate on the 'clear previous containers' requirement? I want to make sure I understand whether it fits better here or with the reconnect logic.

can you elaborate

Current logic for sync/ seems to be very rename-specific. It just updates the values of the containers already in dnsdock cache. I'm more thinking about making a fresh reload from current docker state. So that if container does not exists any more but we failed to get a die event it would still go away. I think this is best we can do until we start separating automatic and manual containers.
We could also setup HUP signal handler for this action. Its usually used for reloading from fresh conf.

Looks like rename will be included in Docker 1.8.0. It's not in the docs yet but I was able to see the event when running on the experimental install:
ubuntu@ip-172-17-7-50:~$ sudo docker version
Client:
 Version:      1.8.0-dev
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   71f8ca3
 Built:        Thu Jul  2 20:43:59 UTC 2015
 OS/Arch:      linux/amd64
 Experimental: true

Server:
 Version:      1.8.0-dev
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   71f8ca3
 Built:        Thu Jul  2 20:43:59 UTC 2015
 OS/Arch:      linux/amd64
 Experimental: true

ubuntu@ip-172-17-7-50:~$ sudo docker events &
[1] 29339
ubuntu@ip-172-17-7-50:~$ sudo docker run -d --name one httpd
2015-07-06T18:05:16.000000000Z 28119e9057b1a53c5b189d0209cbf020794df175d68fd4ba6520eb7a6672653d: (from httpd) create
28119e9057b1a53c5b189d0209cbf020794df175d68fd4ba6520eb7a6672653d
2015-07-06T18:05:16.000000000Z 28119e9057b1a53c5b189d0209cbf020794df175d68fd4ba6520eb7a6672653d: (from httpd) start
ubuntu@ip-172-17-7-50:~$ sudo docker rename one two
2015-07-06T18:05:21.000000000Z 28119e9057b1a53c5b189d0209cbf020794df175d68fd4ba6520eb7a6672653d: (from httpd) rename

I'll still update this PR with the feedback you provided, just in case this feature would be useful.

Would it make sense to reload automatically every n seconds?

I didn't want this PR to get more complex by adding a separate scheduling thread and writing a bunch of defaults (reload on or off, reload period, etc.). It should be easy for someone to add their own mechanism into the dnsdock container that periodically calls curl http://localhost/reload according to the schedule they prefer.

Don't think this PR shall be merged anymore. Rename event is now supported by docker.
I am going to add to the code to the code as dnsdock is now using the offical docker api


Add thrift type of be

Proposed changes
Issue Number: close #xxx

Add configuration of thrift_server_type_of_be, supporting change thrift type via be.conf.

run buildall

Could anyone please add the appropriate labels to this PR? Thank you!


nelson.cli in other cotainer

Hello Roman,
I'm again trying to build a docker-compose file.
My setup:

docker server IP 10.0.0.x

4 containers:

iri
nelson.cli with hostname=nelson.cli
nelson.mon with hostname=nelson.mon
nelson.gui with hostname=nelson.gui

nelson.mon is connecting to nelson.cli:18600 and works fine.
nelson.gui runs on 10.0.0.x:5000 and can ping nelson.cli if I'm inside the cotainer, but is not able to connect to nelson.cli:18600. I've also tried: http://10.0.0.x:5000/nelson.cli/18600, with no success.
From what I understand (please bear with me, I'm not a developer), the nelson.gui is trying to connect to localhost:18600 through my browser, therefore cannot see the server. If i go to http://10.0.0.x:5000/nelson.cli/18600 it's still my browser trying to connect to nelson.cli.
My browser is running on my notebook, while nelson.cli is running on my server and my notebook doesn't know anything about nelson.cli.
This means that the nelson.gui is not talking to nelson.cli, but my notebook is, this may never work for similar setups. Please tell me if I'm wrong with this assumption.
To make it work, I had to give nelson.cli a public hostname like nelsoncli.mydomain.com and modify /src/redux/nelson-api.js to point to this url.
At this point the nelson.gui started working.
Is this by design or am I just hacking around in the blind and there is an easier way?
Thanks,
Antonio

Hi Antonio, you have to run nelson.cli with "--apiHostname <IP_ADDRESS>" so that it listens to non-local connections.
Then you run nelson.gui and access it in your browser like:
http://://

Hi,
my nelson.cli is running on <IP_ADDRESS> inside a docker container with hostname nelson.cli, nelson.mon is working with this setup.

Then, instead of "nelson.cli" you have to use your server ip: http://SERVERIP:5000/SERVERIP/18600

OMG, of course.. that's clear! Thanks again!

Sorry, have to reopen it.
Curl is working, the nelson.gui is not:
https://imgur.com/a/0wn7w

This IP address looks like an internal/local address. Is your browser running on a machine in the same network so it knows how to access that IP?

Yes, the browser has access to that IP.
Today after updating the docker images it works like a charm!

Hi,
I've tried with the new version and added username and password.
Tried the following links:
http://antonionardella.eu:5000/antonionardella.eu/18600/#/nelsonuser:nelsonpass

gives:
Cannot GET /antonionardella.eu/18600/

This one:
http://antonionardella.eu:5000/#/nelsonuser:nelsonpass/antonionardella.eu/18600/

Gives a blank page.


Deploy check plugins fatal error when running --check

When re-running the playbook in --check mode I am getting the following error:
TASK [external/sensu-ansible : Register available checks] ********************************************************************************************************************************************************************
skipping: [apps-test.redacted.com]
skipping: [sensu.redacted.com]

TASK [external/sensu-ansible : Deploy check plugins] *************************************************************************************************************************************************************************
fatal: [apps-test.redacted.com]: FAILED! => {"msg": "The conditional check 'sensu_available_checks is defined and item in sensu_available_checks.stdout_lines' failed. The error was: error while evaluating conditional (sensu_available_checks is defined and item in sensu_available_checks.stdout_lines): Unable to look up a name or access an attribute in template string ({% if sensu_available_checks is defined and item in sensu_available_checks.stdout_lines %} True {% else %} False {% endif %}).\nMake sure your variable name does not contain invalid characters like '-': argument of type 'StrictUndefined' is not iterable\n\nThe error appears to have been in '/Users/michaelporter/redacted/Ansible/roles/external/sensu-ansible/tasks/plugins.yml': line 39, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Deploy check plugins\n  ^ here\n"}
fatal: [sensu.redacted.com]: FAILED! => {"msg": "The conditional check 'sensu_available_checks is defined and item in sensu_available_checks.stdout_lines' failed. The error was: error while evaluating conditional (sensu_available_checks is defined and item in sensu_available_checks.stdout_lines): Unable to look up a name or access an attribute in template string ({% if sensu_available_checks is defined and item in sensu_available_checks.stdout_lines %} True {% else %} False {% endif %}).\nMake sure your variable name does not contain invalid characters like '-': argument of type 'StrictUndefined' is not iterable\n\nThe error appears to have been in '/Users/michaelporter/redacted/Ansible/roles/external/sensu-ansible/tasks/plugins.yml': line 39, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Deploy check plugins\n  ^ here\n"}

I added a debug to check the value:
TASK [external/sensu-ansible : debug] ****************************************************************************************************************************************************************************************
ok: [apps-test.redacted.com] => {
    "sensu_available_checks": {
        "changed": false,
        "failed": false,
        "msg": "skipped, running in check mode",
        "skipped": true
    }
}
ok: [sensu.redacted.com] => {
    "sensu_available_checks": {
        "changed": false,
        "failed": false,
        "msg": "skipped, running in check mode",
        "skipped": true
    }
}

Released in https://github.com/sensu/sensu-ansible/releases/tag/2.7.0, thanks for submitting the issue! Let me know if you run into anything else!


Clojure global "def" not global

In closure the "def" statement should be "global" but it isn't.
http://clojuredocs.org/clojure.core/def
"def: Creates and interns or locates a global var with the name of symbol and a
namespace of the value of the current namespace (ns)."
CodeCombat
(def bar 6)
(defn foo [a b] (def bar b) (* a b))
(.say this (str (foo 2 3) " " bar))

should say: 6 3
but says: 6 6
http://www.tryclj.com/
> (def bar 6)
#'sandbox21848/bar
> (defn foo [a b] (def bar b) (* a b))
#'sandbox21848/foo
> (print (str (foo 2 3) " " bar))
6 3nil

This one appears to be in Closer.js (as the demo site also outputs "6 6")
@vickychijwani

Closing for now since we aren't currently supporting Clojure with new interpreter in Aether.


[META] - Enhancing and refactoring testing procedures for Nebari (WIP)

Context
This meta-issue is focused on revamping Nebari’s testing framework to improve the reliability of the development process. It aims to thoroughly assess current testing practices, identify deficiencies, and strengthen the testing approach.
The plan includes a comprehensive analysis of existing tests, identifying critical user journeys, and providing adequate tools for contributors to facilitate local testing and validation.
The issues listed below will hopefully elaborate and help with better approaching those changes:

[ ] #2395
[ ] #2396
[ ] #2397

Value and/or benefit
Our current testing infrastructure, while standard, has yet to keep pace with changes to our most recent structural changes, nor does it provide a satisfactory/reliable understanding of the different components within Nebari. This has led to a need for more confidence in the tested infrastructure. Enhancing our testing approach will address these issues, improve reliability and confidence in our infrastructure, and give us a better understanding of the current state of our test capabilities.
Anything else?
This issue is part of our current priority scoping board and is subject to change in the future.

@marcelovilla I will add more details to this as an organization bord for future issues/tasks. The general discussion will be held as part of an RFD with the same name.

@viniciusdc we should define what we need to do to get this RFD across the line.


Due to a configuration change you must use MFA?

Hi,
I don’t think I’ve changed anything, but as of today HA isn’t able to connect to M365. When I look at the error logs I see the following.
oauthlib.oauth2.rfc6749.errors.InvalidGrantError: (invalid_grant) AADSTS50076: Due to a configuration change made by your administrator, or because you moved to a new location, you must use multi-factor authentication to access '00000003-0000-0000-c000-000000000000'.
Trace ID: d0e28622-399b-47af-84c2-7ed305518500
Correlation ID: 112c90c6-ea57-4a23-9c3b-3886c39a469b
Timestamp: 2023-01-20 23:51:21Z
Any ideas what I need to do?

Okay - I deleted the token in HA, and was able to trigger a re-auth process. Now back and working. Just not sure why I needed to do that.


feat(sgHeader): Update links

As per card https://trello.com/c/n1UiZwIz/697-js-srp-5-header-design-update

http://seek-asia-style-guide--c14e5c848dcb45d17f46e475af331a4b29445904.surge.sh/jobstreet-header


DynamoDB errors lose original message

Expected
The error output of a DynamoDB node to add msg.err to existing msg object.
Actual
The error output of a DynamoDB node is a new msg with only msg.err and a new msg._msgid property.
Impact
This causes issues using DynamoDB in an HTTP endpoint flow because it loses the msg.res object. Additionally, mitigations by stashing msg.res in flow context aren't possible because even the msg._msgid is lost.

This affects all nodes, not just DynamoDB. It is a problem for me, too, as I run these nodes from flows that are exposed as web services. Without the msg.res and msg.req data in the error, it is impossible to send a response to the original http request, so it must time out instead.


Motivation behind github/ organization and docker/python?

This is more of a general question than an issue, but I'm curious why this lives in github/ instead of actions/: https://github.com/actions? It seems most of the 'official' actions live there, and I think this would make a good addition there!
I also thought I'd ask in this ticket - what was the motivation behind using docker/python? Most of the major/official actions seem to use typescript, and I think many people are very familiar with the syntax to reference an action like this:
uses: actions/checkout@v3

I've used GitHub actions for quite a few years and I honestly can say I've never seen this syntax, so it was a bit strange to me at first:
uses: docker://ghcr.io/github/stale_repos:v1

Certainly not saying anything is wrong, but curious to learn about why this path was taken for this action. It seems like there is more complexity with making/pushing a docker image that gets referenced instead of just referencing a GitHub repo directly. It also seems a bit strange to me that an official GitHub action is using a 3rd party python library for GitHub instead of something like octokit.
Thanks in advance!

👋🏻 Thanks for the questions! We put this in the github repo instead of in the actions repo because of the team that created and maintains it. It was created by the GitHub OSPO and the GitHub actions team manages the actions org. That being said, I take your point that maybe it belongs in the actions org. I can chat with those folks and see if they want it over there!
Using Docker and Python was a choice I made because I am much more familiar with those tools. I've done a little javascript and typescript and found debugging to be pretty difficult for me which made maintaining those repositories quite a chore. 😓
It is definitely a more paved path for actions to use typescript and then reference the action directly, so I understand the raised eyebrows. I would guess that taking docker out of the equation would also make the action more performant, which probably doesn't matter for what this action does but interesting (to me) to note.
Given that choice of using python, I was stuck with going with a 3rd party python library since octokit does not have python support.
Hope that all makes sense!

@zkoppert Thanks for the response! All of that definitely makes sense. From an outside perspective, it is definitely nice to see consistency in location, technology, and implementation of actions, especially when they come directly from the company that created Actions! Ideally there would be some collaboration between OSPO and the Actions team when it comes to OSS actions from GitHub - I would imagine most outside users don't really differentiate specific teams within GitHub. I own the implementation/support for Actions in an organization of ~250 engineers and I know that when there are weird differences it can often confuse people. I think that would be a few orders of magnitude larger for GitHub itself.
Hopefully some of this can help steer this towards the more 'standard' GitHub Actions path, but implementation aside the functionality of this action is awesome - definitely something we love to track in our organization. Thanks again for the thorough response!


context.req.parseBody(): option for support dot notation

What is the feature you are proposing?
For now this method has only option all for array support. It is awesome!
In my opinion, it would be very convenient to add an option to support dot notation (dot: boolean), which would greatly simplify the processing of nested objects.
Example:
let formData = new FormData();
formData.append('obj.key1', 'value1');
formData.append('obj.key2', 'value2');

// processed without dot notation support, like `await context.req.parseBody()`
{
	"obj.key1": "value1",
    "obj.key2": "value2",
}

// processed with dot notation support, like `await context.req.parseBody({ dot: true })`
{
	"obj": {
		"key1": "value1",
		"key2": "value2"
	}
}

Hi @vanodevium
Are you talking about the "FormData"?
If so, it's standard Web API, and it's not Hono matter. There is nothing we can do about it.

It's something from the Browser API. For Hono, I think you can build a custom middleware for it.
app.use(async (c, next) => {
    const json = await c.req.parseBody();

    // do the dot notation logic

    c.req.bodyCache.parsedBody = parsedBody;
  } catch (_err) {
  } finally {
    await next();
  }
});

But becareful about modifying data in middleware, while modifying data in middleware is a common and often necessary practice in web development, it must be done responsibly, securely, and in compliance with legal and ethical standards.

Ah, I see. I miss understanding. This is proposing adding a dot option to c.req.parseBody()
That is a good proposal. I'd like to know if others want this feature.

@yusukebe
I discussed this functionality with colleagues and programmer friends.
The answer is the same for everyone - it is a very convenient and useful thing.
Moreover, by default this thing will be turned off, so there will be no breaking changes for existing systems.
Of course, the decision is yours. Thank you!

Ah, I see. I miss understanding. This is proposing adding a dot option to c.req.parseBody()
That is a good proposal. I'd like to know if others want this feature.

I think it's something great to add :)

Thanks to both. Let's add the feature. I'll do it, but now I'm busy working on other matters. Does anyone implement it?

Fine, I will do it. I am thinking about the API cause {dot: true} isn't expressive enough, maybe something like {json: true}?

Hi @MathurAditya724 actually I'm already building it, only unit tests left

@fzn0x Great! Do let me know when the PR is ready, maybe I can add some pointers

PR is ready! 👍

Hi @yusukebe I think we can close this issue :)

Yes. Closing:)


ui Printf functions: handle errors by dying

The ui Printf and related functions return an error to indicate their status. But none of the call sites that are calling them are catching that returned error. Go linters see this as an un-handled error and complain about it.
What do you think about locally handling errors on stdout/stderr by just panicking and dying, removing the returned error from their signatures? This will get rid of the go linter warnings, and seems a little cleaner to me.

Makes sense, thank you!


feat: add half modal sheet

Hi,
Thanks for providing a great package.
I really identified with this idea https://github.com/vanshg395/intl_phone_field/issues/182 💡
It would be even easier to use if a bottom sheet was added to the dialogues.
Please confirm if you wish.

@all-contributors

https://github.com/all-contributors please add @kno3a87 for Code 💻

@all-contributors please add @kno3a87 for Code 💻

@all-contributors please add @vanshg395 @marcaureln for Code 💻


How to create a Mesh using Vertices and Triangle indices ?

I am creating a visualization tool using GzRendering which runs convex hull decomposition and I want to create a mesh using Vertices and Triangle Indices provide by these tools and visualize. How can this be accomplished without writing to a file and loading?

I don't think we have a tutorial for this but you can fine example code here for creating a box mesh: https://github.com/gazebosim/gz-common/blob/gz-common5/graphics/src/MeshManager.cc#L458
After that you can pass the mesh object to gz-rendering like so: https://github.com/gazebosim/gz-rendering/blob/gz-rendering7/examples/mesh_viewer/Main.cc#L59-L68
In the above example, you'll just need to set the mesh directly to the one you created instead of asking the MeshManager to load the mesh for you, i.e.
change
descriptor.mesh = meshManager->Load(descriptor.meshName);

to
descriptor.mesh = your_mesh_obj

Closing since the feature is already available.

@iche033 is it required to provide normal's?

It appears normal's are required for things to visual correctly, but after looking at the provided links and the API for the SubMesh I was able to get things working. @iche033 Thanks for the support.


Spek breaks coverage reports

When I'm selecting "Run Tests in <package_name> with Coverage" in Android Studio.
The test coverage report works as long as I don't run any tests written with Spek.
Including Spek tests the report is showing 0 % coverage everywhere.
I fear loosing coverage reports is a K.O. criteria for using Spek for me. Do you know how to fix that?

Sorry, this is an Android Studio bug and I experienced that issue too with JUnit4 tests written in Kotlin. I don't know the exact problem. Anyway, simply cleaning the project helps.


[WIP] Bug/skopt0 7 1 compatibility

Pull Request Test Coverage Report for Build 896

5 of 5 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 94.616%

Totals

Change from base Build 879:
0.003%

Covered Lines:
4692

Relevant Lines:
4959

💛  - Coveralls


"npx remix routes" issue with Vite and latest Remix

Hello,
it seems that after migrating to Vite (unstable) with latest Remix (2.3.1), with latest remix-flat-routes that npx remix routes does not return the routes detected by this package.
I have a complicated folders structure of routes, and they all work fine in the browser, but  calling npx remix routes returns just what it sees with the standard Remix routes lookup system:
<Routes>
  <Route file="root.tsx">
    <Route path="*" file="routes/$.tsx" />
  </Route>
</Routes>

... while the actual routes are (just a sample):
app/routes
├── $.tsx
├── _public+
│   ├── _layout
│   │   ├── FancyFooter.module.css
│   │   ├── FancyFooter.tsx
│   │   ├── layout.module.css
│   │   └── layout.tsx
│   ├── about-us
│   │   ├── ContactIcon.module.css
│   │   ├── ContactIcon.tsx
│   │   ├── _about_us.tsx
│   │   └── about-us.module.css
│   ├── index
│   │   └── page.tsx
│   ├── signin.tsx
├── dashboard
...

All routes work fine in the browser, the issue is only with npx remix routes and Vite in combination.
Just in case it might be useful here is my vite.config.js :
import { unstable_vitePlugin as remix } from "@remix-run/dev";
import { defineConfig } from "vite";
import { flatRoutes } from "remix-flat-routes";
import tsconfigPaths from "vite-tsconfig-paths";
import mdx from "@mdx-js/rollup";
import remarkFrontmatter from "remark-frontmatter";
import remarkMdxFrontmatter from "remark-mdx-frontmatter";

export default defineConfig({
  plugins: [
    remix({
      ignoredRouteFiles: ["**/.*"],
      routes: async (defineRoutes) => {
        return flatRoutes('routes', defineRoutes)
      },
    }),
    tsconfigPaths(),
    mdx({
      remarkPlugins: [
        remarkFrontmatter,
        remarkMdxFrontmatter,
      ]
    })
  ],
});

Thank you in advance for your answers and for your work!

Your ignoredRouteFiles setting is incorrect. That setting applies to the Remix default convention. You need to specify that all files should be ignored.
ignoredRouteFiles: ["**/*"] // ignore all files (remove the .)

Thank you for the reply. I made the change but a new error pops up preventing both npx remix routes from working as well as npm run dev. I tried even having just a single index.tsx route in the routes/ and even an empty routes/ and same thing happens. Returning the ignoredRouteFiles back to standard **/.* fixes the npm run dev startup issue but remix routes returns the same error.
$ npx run dev

><EMAIL_ADDRESS>dev
> vite dev

error when starting dev server:
Error: Cannot add empty string to PrefixLookupTrie
    at PrefixLookupTrie.add (/Users/xxxx/sources/app/node_modules/@remix-run/dev/dist/config/flat-routes.js:38:23)
    at flatRoutesUniversal (/Users/xxxx/sources/app/node_modules/@remix-run/dev/dist/config/flat-routes.js:141:18)
    at Object.flatRoutes (/Users/xxxx/sources/app/node_modules/@remix-run/dev/dist/config/flat-routes.js:97:23)
    at Object.resolveConfig (/Users/xxxx/sources/app/node_modules/@remix-run/dev/dist/config.js:153:33)
    at async resolvePluginConfig (/Users/xxxx/sources/app/node_modules/@remix-run/dev/dist/vite/plugin.js:209:9)
    at async config (/Users/xxxx/sources/app/node_modules/@remix-run/dev/dist/vite/plugin.js:338:26)
    at async runConfigHook (file:///Users/xxxx/sources/app/node_modules/vite/dist/node/chunks/dep-vyqc0_aB.js:67500:25)
    at async resolveConfig (file:///Users/xxxx/sources/app/node_modules/vite/dist/node/chunks/dep-vyqc0_aB.js:66953:14)
    at async _createServer (file:///Users/xxxx/sources/app/node_modules/vite/dist/node/chunks/dep-vyqc0_aB.js:59492:20)
    at async CAC.<anonymous> (file:///Users/xxxx/sources/app/node_modules/vite/dist/node/cli.js:764:24)

I think I know where the issue is but not how to fix it. It seems that npx remix routes ignores the vite.config.ts completely and is using some standard Remix config, which explains the whole mess. Even entering some garbage into  vite.config.ts does not prevent npx remix routes from starting, while it crashed npm run dev, as it should.

That's really odd. I just tried it with Remix v2.3.1 and remix-flat-routes v0.6.2.
Which version of rfr are you using?
❯ tree app/routes
app/routes
├── $.tsx
├── _public+
│   ├── _layout
│   │   ├── FancyFooter.module.css
│   │   ├── FancyFooter.tsx
│   │   ├── layout.module.css
│   │   └── layout.tsx
│   ├── about-us
│   │   ├── ContactIcon.module.css
│   │   ├── ContactIcon.tsx
│   │   ├── _about-us.tsx
│   │   └── about-us.module.css
│   ├── index
│   │   └── page.tsx
│   └── signin.tsx
└── dashboard
    └── _layout.tsx

❯ npx remix routes
<Routes>
  <Route file="root.tsx">
    <Route path="*" file="routes/$.tsx" />
    <Route file="routes/_public+/_layout/layout.tsx">
      <Route path="about-us" file="routes/_public+/about-us/_about-us.tsx" />
      <Route index file="routes/_public+/index/page.tsx" />
      <Route path="signin" file="routes/_public+/signin.tsx" />
    </Route>
    <Route path="dashboard" file="routes/dashboard/_layout.tsx" />
  </Route>
</Routes>

That's really odd. I just tried it with Remix v2.3.1 and remix-flat-routes v0.6.2.

Have you added Vite to the project as per instructions at https://remix.run/docs/en/main/future/vite ?

Which version of rfr are you using?

What is rfr?

Here's my starter repo for Remix + Vite. It also includes Tailwind CSS.
https://github.com/kiliman/remix-vite-2.2.0

Here's my starter repo for Remix + Vite. It also includes Tailwind CSS.
https://github.com/kiliman/remix-vite-2.2.0

Sorry I did not understand this. I see the repo, but there is no remix-flat-routes in it, so not sure how it applies?

I'm having this issue as well. I believe its because the remix cli doesn't know how to look at the vite config. It is working in your example repo @kiliman because your vite config is importing it's remix config from the normal remix.config.js, thus vite and remix cli are sharing the same source of truth.
I followed that same pattern and the issue has been resolved for my app. Ultimately I think this needs to be raised upstream in remix.

Ah, I see. Yes, I don't think the Remix CLI supports Vite yet.

I followed that same pattern and the issue has been resolved for my app. Ultimately I think this needs to be raised upstream in remix.

Yeah, good catch. What I did to resolve this was to leave my vite.config.ts but add remix.config.js with the contents:
// TMP fix for npx remix routes to work with Vite; vite.config.js is the actual config used for the app

import { flatRoutes } from "remix-flat-routes";

/** @type {import('@remix-run/dev').AppConfig} */
export default {
  ignoredRouteFiles: ["**/.*"],
  routes: async (defineRoutes) => {
    return flatRoutes('routes', defineRoutes)
  },
}

And now both npx remix routes (without Vite) and npm run dev (with Vite) work fine.

This is still an issue even with the vite plugin being stable now. Any ideas on how to fix this?

@lennerd Can you show me your vite.config.ts and remix.config.js file?
Compare them to what I have in remix-vite-template. https://github.com/kiliman/remix-vite-template

Yeah that should work because you have both files.
So we always need both for remix routes to work? That sounds non intuitive, no? Maybe something we need to create an issue for in Remix dev tools repo?

A new PR just landed that now will read the Remix config from vite.config.ts for the Remix CLI. So it should be in the next release.
https://github.com/remix-run/remix/pull/8916

The remix PR has been merged into the newest version.

Maybe an updated Readme would be nice.


enabled GPL_TIMING_DRIVEN

Enable GPL_TIMING_DRIVEN. Here are some updated metrics:

TNS (increase): -136,962,912.00 (v6 was -128,422,560.00)
WNS (decrease): -1930.20 (v6 was -1966.33) - note that the WC path from v6 is now -1673)
clock skew (decrease): 108.99 (v6 was 122)
CTS latency (slight increase): 953 (v6 was 940) - clock tree still has long branch connecting to SRAM)

Still seeing "Routing congestion too high" error in GRT. Here's the heat map:

Artifacts exist in GCP through grt stage.

@jeffng-or @tspyrou
GPL_TIMING_DRIVEN=1 alone does not move the needle... miniscule benefit currently or noise even.
It takes an additional 3 hours to run...
I believe we have a few pathologies that have to be addressed and that until those are addressed, it is better to have faster turnaround times... We need a minimum clock period in the neighbourhood of 1000ps to be in the same range as commercial tools(700ps at 28nm in the SonicBOOM tapeout).
This PR:
$ git log -1
commit 7c64256f7ac4c08ed15ed7ae8977996116aa0da2 (HEAD)
Author: Jeff Ng<EMAIL_ADDRESS>Date:   Thu Oct 17 14:51:16 2024 -0700

    enabled GPL_TIMING_DRIVEN

    Signed-off-by: Jeff Ng<EMAIL_ADDRESS>$ bazel run BoomTile_place $(pwd)/tmp
$ tmp/make elapsed
.//logs/asap7/BoomTile/base
Log                            Elapsed seconds Peak Memory/MB
3_1_place_gp_skip_io                       543           6860
3_2_place_iop                               48           4316
3_3_place_gp                             14060          26420
3_4_place_resized                          898           9064
3_5_place_dp                              1855          26476
Total                                    17404          26476

main:
$ git who
$ bazel run BoomTile_place $(pwd)/tmp
$ tmp/make elapsed
tmp/make elapsed
.//logs/asap7/BoomTile/base
Log                            Elapsed seconds Peak Memory/MB
3_1_place_gp_skip_io                       465           6858
3_2_place_iop                               41           4316
3_3_place_gp                              3058          19050
3_4_place_resized                          736           9060
3_5_place_dp                              1498          26477
Total                                     5798          26477

@jeffng-or is this accurate - it seems different than what you showed me today where there was large difference.

@jeffng-or is the tns/wns comparison accurate too?

@jeffng-or is the tns/wns comparison accurate too?

The TNS/WNS stats above are accurate to this run and v6, which was the change to run at 833MHz.

close? It is still available for study after it is closed.

closing per concerns about increasing GPL run time with minor impact to WNS


Second Replicaset Member Does Not Configure But 3+ Does

Using this as reference  https://github.com/edelight/chef-mongodb/issues/61#issuecomment-9531587

I am no longer using chef.


Delete package.in

resolves #700

ohkay.... as far as i figured out is that, as we are done with package.in as of now, we don't need these lines in our code
https://github.com/jcubic/jquery.terminal/blob/b941eca8e82221313829f47f463e895ac83ef56e/templates/Makefile.in#L31-L32
and
https://github.com/jcubic/jquery.terminal/blob/b941eca8e82221313829f47f463e895ac83ef56e/Makefile#L31-L32
if its wrong approach, please suggest the right one.

And to test if it's working. you can use:

create temporary commit so you don't lose the code:
run ./scripts/version <some version> pick any version like 3.0
run make
if package.json is updated with 3.0 it mean that it's ok. and you can run git rest --hard and push the changes from your commit if not you need to modify the code and do everything again.

Ok closing, you probably will not fix the real issue. I can delete a file in a second. This is not valid contribution you need to do real work.


GDScript error messages use word Nil which doesn't exist in GDScript

Godot version
v4.0.alpha8.official [cc3ed63af]
System information
Windows 10
Issue description
There are several error messages which use word Nil, but there are no Nils in GDScript. I suggest that those error messages should be changed to use the correct term null.
Some error messages also somewhat incorrectly talk about Nil/null being a type while it actually isn't. I have made a proposal to fix the null/void types. This bug report is however just about fixing the confusing error messages.
Steps to reproduce
example 1
var a: int = null

Current error message:
Trying to assign value of type 'Nil' to a variable of type 'int'.

This is especially bad error message as a type null/Nil doesn't actually exist. Better error message would be:
Trying to assign 'null' to a variable of type 'int'. Only assigning 'int' is allowed.

example 2
var a = null
a.foo()

Current error message:
Invalid call. Nonexistent function 'foo' in base 'Nil'.

Better error message could be something like this:
'a' is 'null' so its methods cannot be called.

There are surely more error messages which use word Nil, but these are ones I have noticed so far.
Minimal reproduction project
No response

See https://github.com/godotengine/godot/pull/46241.

Nil is technically a Variant type (albeit it's not exposed for use in scripting), with only one value: null. The error message is valid, but I agree it's confusing.

Yes, it is confusing. Neither of these don't work:
	var a: Nil = null
	var a: null = null

Of course, I wouldn't expect the first line to work if Nil is not exposed to scripting, but the documentation says

null is an empty data type that contains no information and can not be assigned any other value.

(apparently incorrectly) that null is a type, so I might expect that the second line would work.

Assigning null to int produces a good error message now (v4.4.dev6.mono.official [1f47e4c4e])

Calling null function still produces a cryptic error message. I have quite a lot of programming experience so I can decrypt the meaning of the message, but for a developer who is new to programming, the error message is not very good.


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes; or I'm proposing the module to be migrated from CARML/TFVM.

Bicep or Terraform?
Bicep
Module Classification?
Resource Module
Module Name
avm/res/compute/gallery
Module Details
Compute Gallery module, migration from CARML
Do you want to be the owner of this module?
No
Module Owner's GitHub Username (handle)
No response
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

@krbar will migrate this module from CARML, without becoming its owner. The module will be marked as orphaned once the migration is complete, unless someone volunteers to own it.
If someone else wishes to become the owner of this module, please indicate your intent in a comment on this issue.

@matebarabas Module merged into the BRM repo: https://github.com/Azure/bicep-registry-modules/pull/752

[x] PR merged; Module published.
[x] GitHub teams are available and assigned to their parents.
[x] CODEOWNERS file updated.

Thank you for your work @krbar! I'm closing this issue now.
Since this module was migrated from CARML without an owner, an orphaned module request was also created: https://github.com/Azure/Azure-Verified-Modules/issues/482


Remove obsolete Xcode 10 usage instructions.

Using Xcode's legacy build system is no longer required.

Happy to address suggestions later! Merging to unblock other documentation improvements (https://github.com/tensorflow/swift/pull/352).


Can't find pcap.h

Hi,
I saw this guide: https://github.com/TooTallNate/node-gyp/wiki/Visual-Studio-2010-Setup
and installed VS2012 express, after this I tried the SDK. After some errors, I uninstalled all redistributable vs2008 and vs2010 and then all is ok with the SDK. No I tried npm install -g pcap and I got this error:
pcap_binding.cc
pcap_session.cc
..\pcap_binding.cc(5): fatal error C1083: Cannot open include file: 'pcap/pcap.h': No such file or directory [C:\Users\ChrisLE\AppData\Roaming\npm\node_modules\pcap\build\pcap_binding.vcxproj]
..\pcap_session.cc(4): fatal error C1083: Cannot open include file: 'pcap/pcap.h': No such file or directory [C:\Users\ChrisLE\AppData\Roaming\npm\node_modules\pcap\build\pcap_binding.vcxproj]
I use Windows 7 64, installed WinPCAP. Did I forget smth?
Cheers,
Chris

+1 cant install on windows!!

Guys can you try once again after the major refactor

@ujjwalt Not sure whether the refactor catered for Windows support. I wouldn't be surprised if it still doesn't work.

I didn't do much to the C++ side, so I'd be surprised if it suddenly worked.
I don't run Windows, but it would be great if somebody who did could help properly windows-ify the installation process.

@mranney Pcap on Windows w.r.t V8 Binding requires a little bit more research.

I have not got it to work on Windows yet, but there are two big pre-reqs to getting it working:

Get the WinPcap Developer Pack
Locate MSBuild's global configuration directory in %LOCALAPPDATA%\Microsoft\MSBuild
Add WinPcap Developer Pack include and lib directories to MSBuild's global configuration.

I have not yet succeeded in achieving this really essential setup. I assumed, it should work by adding something like this to the global configuration file (but it did not):
<ItemGroup>
    <Compile Include="the_dir_here">
</ItemGroup>

I request anyone here with any windows expertise to resolve this. I have none and therefore am unable to help out with windows

See http://stackoverflow.com/questions/2676417/how-do-include-paths-work-in-visual-studio/35591563#35591563 for information on adding the dev pack's include and lib folders to the global includes for windows :)

I will Test it later

Btw It does fix the pcsp dependencies, but shows other missing dependencies/headers that are not available for Windows.
On Jan 29, 2017, at 12:39 AM, Chrizzly <notifications<EMAIL_ADDRESS>wrote:
I will Test it later

You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/mranney/node_pcap/issues/95#issuecomment-275901226, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHmSDDjh7t-4hq3ykugUq3iWiUomDVJnks5rXFBQgaJpZM4B0D7q.

The solution  :
Create folder
.....\Pcap.Net\PcapDotNet\3rdParty
In 3rdParty folder you put everything from the developerkit  ( https://www.winpcap.org/devel.htm )
when you are done you should now find pcap.h in :
....\Pcap.Net\PcapDotNet\3rdParty\WpdPack\Include\
Now you can compile
....\Pcap.Net\PcapDotNet\src\PcapDotNet.sln
In case this is explained somewhere else: apologies.

I already solved it ，please see，
http://gotovpn.blogspot.com/2018/02/solve-most-important-issues-for-nss.html
THANK YOU!

I got same issue when I installed node-dash-button
PS C:\Users\koooo\Documents\dasher> npm i node-dash-button

><EMAIL_ADDRESS>install C:\Users\koooo\Documents\dasher\node_modules\pcap
> node-gyp rebuild

C:\Users\koooo\Documents\dasher\node_modules\pcap>if not defined npm_config_node_gyp (node "C:\Program Files\nodejs\node_modules\npm\node_modules\npm-lifecycle\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" rebuild )
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
  pcap_binding.cc
  pcap_session.cc
  win_delay_load_hook.cc
..\pcap_binding.cc(2): fatal error C1083: Cannot open include file: 'pcap/pcap.h': No such file or directory [C:\Users\koooo\Documents\dasher\node_modules\pcap\build\pcap_binding.vcxproj]
..\pcap_session.cc(2): fatal error C1083: Cannot open include file: 'pcap/pcap.h': No such file or directory [C:\Users\koooo\Documents\dasher\node_modules\pcap\build\pcap_binding.vcxproj]
gyp ERR! build error
gyp ERR! stack Error: `C:\Program Files (x86)\MSBuild\14.0\bin\msbuild.exe` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:126:13)
gyp ERR! stack     at ChildProcess.emit (events.js:214:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)
gyp ERR! System Windows_NT 10.0.17134
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\koooo\Documents\dasher\node_modules\pcap
gyp ERR! node -v v8.11.2
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\koooo\AppData\Roaming\npm-cache\_logs\2018-05-19T12_13_16_898Z-debug.log

extract WpdPack
copy Include and Lib folders to  node_pcap folder.
add Include and Lib folders to include_dirs var in node_pcap\binding.gyp like below.
and try to build again (node-gyp configure build)
{
"targets": [
{
"target_name": "pcap_binding",
"sources": [ "pcap_binding.cc", "pcap_session.cc" ],
"include_dirs": [
"Include",
"Lib",
"<!(node -e "require('nan')")"
],
"link_settings": {
"libraries": [
"-lpcap"
]
}
}
]
}

I will try to use the cap package, it seems more up to date. Just FYI.


How can i change the Blog card's Description text?

How can i change the Blog card's Description text? it is just whatever is written inside my blog posts. Can we set it explicitly?

I have tried adding description in blog post's front-matter but it doesn't change.

This is native behavior with Hugo.
Read more here: https://gohugo.io/content-management/summaries/#manual-summary-splitting
Solution:
in your blogpost.md
Begin the blog post with the box of text you wish to be the description.
"Hey this article is going to be about..."
<!--more-->
"Your actual article here which may include all sorts of formatting like heading etc."


Add links to the Navbar

From MLH-Fellowship/pod-3.1.2-portfolio Issue 7
Add links to the navbar to navigate to the relevant sections (eg. projects, experience) of the page.
Also, it would be great if we can have an option to choose whether we want to make the navbar sticky or not.

@sonalsk i guess we can close this one. right?

Oh no, we haven't added the navigation links yet. I am waiting for the final navbar to get merged.


How do we handle Data and LockTime generically?

Following the refactoring of the DistributeTransaction it was noticed how that type of transaction is no longer created by RUSK. This leads to the question whether the TotalReward fields should be a TransparentValue within the first TransactionOutput carried within the StandardTx method of the ContractCall interface or we should instead add the value to the Data field of generic smart contract which Input encoding is handled by their ABI.
Related to this question is the topic of time-locked transaction, which LockTime information could be handled generically with the Data field as well or would deserve a dedicated field in the embedded StandardTx structure of the ContractCall interface, so to standardize it across all smart contracts, genesis one included

The outcome of the conversation converged on the following points:

DistributeTransaction should be like all the other contract calls and carry an embedded Transaction with empty Fee and with TotalReward as a uint64 in the TransparentValue field (dusk-network/dusk-protobuf#98)
LockTime is not needed as generic field. If a user-defined smart contract will have a timelock, this will be an input parameter encoded in the Data field
We need to add the Data field to the Transaction which is going to be left empty for the genesis contracts (since their anatomy is known apriori) and carry the encoded ABI-specific input data for the user-defined smart contract transactions (dusk-network/dusk-protobuf#99)


Ruby Testing

@stephensli I would love to resolve this 🐛issue.
possibly will find some good 🖋 solution.
It would be kind if you assign this to me.

@Maufa42 thank you for being interested this ticket! I have yet to correctly scope out how I want to test this or have any related documentation to allow you to easily configure the application locally. This means it will be pretty hard to test this in a way that will be good!.
Here is the plan:

I'm going to get some documentation up once I have the sample site #6 complete.
I'll write some structured tests for a language outside of Ruby to allow you to see how I would like tests to be actioned

This should allow you to easily setup the repository locally, write new tests and get them in! If you give me a couple of days I'll let you know.

@stephensli Sounds Great👍 . Let's Do This Together


python311Packages.latexify-py: 0.2.0 -> 0.3.1

Diff: https://github.com/google/latexify_py/compare/refs/tags/v0.2.0...v0.3.1
Changelog: https://github.com/google/latexify_py/releases/tag/v0.3.1
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

4 packages built:

python310Packages.latexify-py
python310Packages.latexify-py.dist
python311Packages.latexify-py
python311Packages.latexify-py.dist


Remove repeated scripts to improve load time

What is the change?
Deleted the repeated scripts in both header and footer.ejs and placed all the script tags in footer.ejs and deleted the body, head, html closing tags in header.ejs.
Related issue?
close: #488
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x] Have you followed the Contribution Guidelines while contributing.
[x] Have you checked there aren't other open Pull Requests for the same update/change?
[x] Have you made corresponding changes to the documentation?
[x] Your submission doesn't break any existing feature.
[x] Have you tested the code before submission?

one extra bootstrap cdn was removed in #505 so there could be conflicts

@innovation-platform can you please resolve this asap

Yes I will do it by today. Thank you.
On Wed, 31 Mar, 2021, 7:27 AM Pranav, @.***> wrote:

@innovation-platform https://github.com/innovation-platform can you
please resolve this asap
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ALPHAVIO/BlogSite/pull/504#issuecomment-810697846,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APTIMSSDICN3WA5SDWXMSILTGJ6QFANCNFSM4Z62W6BA
.

Can you like make a new branch from master and do the changes with one commit ?
PR has 7 commits with same message so can't merge.

Sorry for the delay and those mistakes. I will do it asap. Thank you.
…
On Thu, 1 Apr, 2021, 3:52 AM Pranav, @.> wrote: @Pranav016 requested changes on this pull request. ------------------------------ In views/partials/footer.ejs <#504 (comment)>: > +<script<EMAIL_ADDRESS>+<script @./dist/js/bootstrap.bundle.min.js" integrity="sha384-JEW9xMcG8R+pH31jmWH6WWP0WintQrMb4s7ZOdauHnUtxwoG2vI5DkLtS3qm9Ekf" crossorigin="anonymous"> both popper and bootstrap are already present, repeated scripts clutter the code and cause ambiguity ------------------------------ In views/partials/footer.ejs <#504 (comment)>: > +<script<EMAIL_ADDRESS>+<script @.***/dist/js/bootstrap.bundle.min.js" integrity="sha384-JEW9xMcG8R+pH31jmWH6WWP0WintQrMb4s7ZOdauHnUtxwoG2vI5DkLtS3qm9Ekf" crossorigin="anonymous"> remove the repeated ones, i think this issue should have been resolved in a single commit in one time — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#504 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/APTIMSWNTMCR42VLSI72WDLTGOOD3ANCNFSM4Z62W6BA .

Good work 👍

Yes I will do it. Thank you.


[infra+code] feat: add glue/python ci

@paulswartz regarding the 'archive' directory. I like to keep useful code around for fast access. Tracking it down in git, especially deletions is too slow and takes me out of my element. The 'archive' folder is a stop before getting outright deleted.

@paulswartz regarding the 'archive' directory. I like to keep useful code around for fast access. Tracking it down in git, especially deletions is too slow and takes me out of my element. The 'archive' folder is a stop before getting outright deleted.

If the archive folder is only for your use, do you need to check in it? You can have local ignored paths and keep as much in the archive as is helpful.

Neat! Didn't know about this. I have removed them.


Labels not appearing in the projection do not contribute to the pipeline

Compare the pipelines of these two specifications:
var compoundSpecification = Given<Airline>.Match((airline, facts) =>
    from flight in facts.OfType<Flight>()
    where flight.airlineDay.airline == airline
    from booking in facts.OfType<Booking>()
    where booking.flight == flight
    from cancellation in facts.OfType<FlightCancellation>()
    where cancellation.flight == flight
    select new
    {
        booking,
        cancellation
    }
);

var simpleSpecification = Given<Airline>.Match((airline, facts) =>
    from flight in facts.OfType<Flight>()
    where flight.airlineDay.airline == airline
    from booking in facts.OfType<Booking>()
    where booking.flight == flight
    from cancellation in facts.OfType<FlightCancellation>()
    where cancellation.flight == flight
    select cancellation
);

They produce different pipelines:
simpleSpecification.Pipeline.ToDescriptiveString()
airline: Skylane.Airline {
    flight: Skylane.Flight = airline S.airline Skylane.Airline.Day S.airlineDay Skylane.Flight
    cancellation: Skylane.Flight.Cancellation = flight S.flight Skylane.Flight.Cancellation
}

compoundSpecification.Pipeline.ToDescriptiveString()
airline: Skylane.Airline {
    flight: Skylane.Flight = airline S.airline Skylane.Airline.Day S.airlineDay Skylane.Flight
    booking: Skylane.Booking = flight S.flight Skylane.Booking
    cancellation: Skylane.Flight.Cancellation = flight S.flight Skylane.Flight.Cancellation
}

The pipeline -- and therefore the number of results -- should not change based on which facts are selected.

The pipeline is now generated based only on the set definitions, not the projection. Both specifications yield the full pipeline:
airline: Skylane.Airline {
    flight: Skylane.Flight = airline S.airline Skylane.Airline.Day S.airlineDay Skylane.Flight
    booking: Skylane.Booking = flight S.flight Skylane.Booking
    cancellation: Skylane.Flight.Cancellation = flight S.flight Skylane.Flight.Cancellation
}

Released: https://github.com/jinaga/jinaga.net/releases/tag/v0.2.21


[node-labeller] fix hasTSCCounter nil pointer dereference

What this PR does
Before this PR:
#12652 introduced an issue where calling the hasTSCCounter method of node-labeller with a nil cpu counter would result in a nil pointer dereference.
This is a common occurence in some architectures such as s390x.
After this PR:
This PR relies on Go's short-circuit behavior to assert whether cpuCounter is nil as the first condition.
It also introduces a test to node_labeller_test.go that tests execution behavior when cpuCounter is nil.

Fixes bug introduced in #12652

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/cc @orelmisan
/cc @iholder101

FYI @vamsikrishna-siddu

Change - Amended commit subject.

Nice catch @Acedus! Thanks a lot!
/approve

Change - Moved new test to a describe table for the case where TSC labels shouldn't be added.

Thanks a lot!
Re-LGTMing on the basis of @orelmisan's lgtm.
/lgtm

/lgtm
Thank you @Acedus


Regz patching

This patch shows a minimal example of patching the register schema after it has been loaded into Regz. It was a goal to allow the patches to be written in zig for strong typing and potential for code completion.
We can't instantiate the Regz code directly in build scripts as it has C dependencies, so Regz is built as an executable and it operates on files.
Chip now has a patches field which is a list of patches that are applied to the Database in order. These patches are serialized to a newline-delimited JSON file that is passed to the Regz executable. Regz loads the register schema into its database and then applies the patches. In the future the plan will be to allow to use a newlined-delimted JSON file directly, I have plans to create a graphical program for patching register schemas.
Right now creating enums and assigning that shared type to different fields is all that's supported.

@haydenridd I wasn't able to make firmware.add_patches() without a bunch of rework. BUT I did add a patches field to the add_firmware() options, which should be just as good :)


Warning for filtered words

It'd be nice if Nadeko could warn users when they say a filtered word instead of just deleting the message.

Use custom reactions. Or do you mean like .warn ?
On Thu, Jul 13, 2017, 8:29 AM Katie<EMAIL_ADDRESS>wrote:

It'd be nice if Nadeko could warn users when they say a filtered word
instead of just deleting the message.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Kwoth/NadekoBot/issues/1398, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGadxvYpcyT83-jcu-PN4e6lEH7L8tAks5sNg2ZgaJpZM4OW68h
.

This is not actually the place for suggestions, altho I do remember something already suggested...
Anyway, for suggestions you need to post it here: http://feathub.com/Kwoth/NadekoBot but the site is down for quite a while and we don't know when it will get back up.

Nevermind. I'm dumb. That wouldn't. work
On Thu, Jul 13, 2017, 8:32 AM jon donigian<EMAIL_ADDRESS>wrote:

Use custom reactions. Or do you mean like .warn ?
On Thu, Jul 13, 2017, 8:29 AM Katie<EMAIL_ADDRESS>wrote:

It'd be nice if Nadeko could warn users when they say a filtered word
instead of just deleting the message.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Kwoth/NadekoBot/issues/1398, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGadxvYpcyT83-jcu-PN4e6lEH7L8tAks5sNg2ZgaJpZM4OW68h
.

Post features on feathub, when it's back.


add new interface for hdmap

Add HDMap Interface for ClearArea and SpeedBump

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Update documentation to deprecate Node.js Security WG DB

The Node.js Ecosystem Security Database is no more maintained (it's deprecated). So there is no reason to use it anymore for a production application. We should tag as deprecated in README (and also in the strategy markdown page).

Throw a deprecation warning when we use this strategy (see process.emitWarning)

https://github.com/NodeSecure/vulnera/blob/51bb814087da39e105f37a1538c43ca7b430b09b/src/strategies/index.js#L15-L16
Here a link to an official Node.js article that explain why: https://nodejs.medium.com/node-js-ecosystem-vulnerability-reporting-program-winding-down-591d9a8cd2c7

I'm on it

@all-contributors please add @PierreDemailly for code, doc


Add help command to ingame console

Should be pretty easy

This was opened back when the console still used a custom homegrown language, now it is a full C# REPL. Is this still relevant?

While builtin mod support has been removed the console will come back in a limited capacity for advanced server moderation and development debugging purposes so this is still relevant

Closed by https://github.com/ForLoveOfCats/SkyOfSteel/commit/636647121034c2bedffcef5cbd64c731bb55b05b


Update US19JanTaxR1.md

adding round 4

Docs Build status updates of commit cc1cd4d:
:clock10: Pending: waiting for processors (51 builds ahead of you)
:warning: Docs Build is busy, currently there are 51 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit cc1cd4d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/payroll/US19JanTaxR1.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[DPM] Entities and caches declaration for DPM's cache

Declare required entities and caches for DPM's cache.

Codecov Report

Merging #573 (fcd88de) into master (fa9bf5c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #573   +/-   ##
=========================================
  Coverage     33.45%   33.45%
  Complexity     1237     1237
=========================================
  Files           511      511
  Lines         12747    12747
  Branches       1585     1585
=========================================
  Hits           4264     4264
  Misses         7905     7905
  Partials        578      578

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 fa9bf5c...fcd88de. Read the comment docs.


refactor(utils): add function overload on who have the encode parameter.

add function overload on who have the encode parameter.

Codecov Report

Merging #237 into develop will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           develop     #237   +/-   ##
========================================
  Coverage    95.68%   95.68%
========================================
  Files            9        9
  Lines          278      278
  Branches        69       69
========================================
  Hits           266      266
  Misses          11       11
  Partials         1        1

Impacted Files
Coverage Δ

packages/ckb-sdk-address/src/index.ts
100% <100%> (ø)
:arrow_up:

packages/ckb-sdk-utils/src/address/index.ts
94.44% <100%> (ø)
:arrow_up:

packages/ckb-sdk-utils/src/ecpair.ts
95% <100%> (ø)
:arrow_up:

packages/ckb-sdk-utils/src/crypto/blake160.ts
100% <100%> (ø)
:arrow_up:

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 a392bd1...8ead9b3. Read the comment docs.


Fixed PowerShell parameter notation

Replaced "=" to " "

@tomo-ms : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Add the Muffin/Trash plugin

If we add the Muffin/ prefix now they would be needed on others as well I think.

Right, @ADmad you think we should?
On another front @dereuromark, I think we should start removing from here the ones that we think do not follow the "right way" of doing things - might help us not have to re-write plugins ourselves.. lolol. I say this because the currently available SoftDelete behavior we advertise on this list is just badly designed IMO.
I agree that we can't play control police or sound biased in any way, BUT, I also wouldn't like to encourage developers to use plugins that aren't right. What do you think?

Just my two cents but why not reverse the idea of removing plugins and instead flag plugins that do follow the "right way". Sort of an "core team approved" image, star, whatever. Two benefits:

freedom of choice is maintained
could gently guide people towards the "right way" (maybe some guidelines?)

+1 for the prefixes btw

@dereuromark I think we should update others. It would make it easier to differentiate between similarly named plugins.
@jadb I agree about removing plugins from the list. This should be a curated list, not a comprehensive list of all plugins out there. We already have plugins.cakephp.org for that. I don't see any problem being opinionated and removing/rejecting plugins which we think are not up to standard. What do you say @dereuromark ?

@bravo-kernel long time!!

<3 cake long time :wink:
Was totally unaware of http://plugins.cakephp.org/ so... reversing my reversal suggestion, lol

Was totally unaware of http://plugins.cakephp.org/ so... reversing my reversal suggestion, lol

It is linked in menu on cakephp.org but in general CakePHP team is really bad at advertising / marketing. :stuck_out_tongue:

freedom of choice is maintained

If there are good quality options available for the same task surely we will list multiple.

could gently guide people towards the "right way" (maybe some guidelines?)

There's only so much we can do in our free time :)

I do also think a "(recommended)"  flag would be good!

Let's update the others than.
Yeah, but too much "recommendation" and stuff might reduce visibility and is sometimes hard to "judge".
As for the removing of bad practice stuff, this is also kind of difficult to find opinions on, unless it's super-clear.. And it might draw some bad blood. But we can give it a try for some of those most outstanding in sense of "wrong".

Maybe a rated plugin site makes more sense? I like this as it is.

@jhli Yes please. This should stay basic. The rest is up to plugins.cakephp.org IMO.

There are some guidelines here, if you wish to follow them:
https://github.com/sindresorhus/awesome/blob/master/awesome.md

Research if the stuff you're including is actually awesome. Put only stuff on the list you or another contributor can personally recommend and rather leave stuff out than include too much.

Then you can wear the cool badge they are working on:

I totally agree with https://github.com/sindresorhus/awesome/blob/master/awesome.md#only-awesome-is-awesome, this one just killed me https://github.com/sindresorhus/awesome/blob/master/awesome.md#pay-attention-to-gramar

:+1: for @deizel's suggestion.
Who wants to start submitting PRs to remove plugins? I think those are good reasons to start removing based on:

Unmaintained (yes some are already i believe)
Bad practice or not "the right way".
Missing documentation, test coverage or both.
Poorly written documentation (some of mine may apply here, lol).

Let the culling begin!

Merging this. Prefixs can be added to other plugins later.

@bravo-kernel PR submitted for "gramar". :joy:


Fix up job object options for unit tests

Most of the jobobject package tests we have ask for options that aren't actually needed/used. This change makes it so that any test that doesn't need a named job object doesn't ask for one and any test that doesn't plan on using the iocp messages doesn't flip the notifications field either. This wasn't causing any issues, but it's probably best to filter down what's being tested to only what's needed.

Re-pushing as the CI was stuck on reporting status and re-running didn't seem to do anything..


Scripting failed Cannot read properties of undefined (reading 'serverInfo')

Type: Bug
From the manage database tab, when I script an object i get the following message:
"Scripting failed
Cannot read properties of undefined (reading 'serverInfo'). "
I close the window, reopen and it works. Or I go to a different window and it works.

Azure Data Studio version: azuredatastudio 1.45.0 (8eff468100f02157f2219335f7a566d14451ca49, 2023-07-24T18:20:47.177Z)
OS version: Darwin arm64 22.3.0
Restricted Mode: No
Preview Features: Enabled
Modes:

System Info

Item
Value

CPUs
Apple M1 Max (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
3, 3, 3

Memory (System)
32.00GB (1.62GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Hi @jmetape
Can you provide more information:

How did you create this connection and launch dialog, using connection string/connection dialog?
Please provide masked connection string (if used for creating connection) and connection properties as seen in settings.json (Ctrl+Shift+P > Preferences: Open Settings (JSON) > datasource.connections)


Cannot boot firmware

Not sure if im alone, but i cant boot the firmware provided.
if i try the steps to boot it from RAM i get the error:
Uncompressing Kernel Image ... ERROR: LzmaDecode.c, 543
Decoding error = 1
LZMA ERROR 1 - must RESET board to recover
If i change the bootargs like written, i get the error message:
Bad Magic Number

error.txt

solved it by myself. The error message regarding LZMA error is because of the used u-boot version. You have to copy it to flash before executing.
This will wipe out the old installation and install the new one:
tftp 0x81000000 openwrt-ath79-generic-panasonic_cz-taw1-squashfs-sysupgrade.bin
[...]
done
Bytes transferred = 9175851 (8c032b hex)

erase 0x9f050000 +8c032b
cp.b 0x81000000 0x9f050000 8c032b
bootm 0x9f050000


Inconsistent whitespace sensitivity with numbers

If you type the equivalent of "1.2 00" into the test environment, it renders as '1.200' (i.e. one number). This is consistent with doing "12 00", which renders as '1200'. However, if you try "1. 200", you instead get '1. 200' (i.e two 'numbers').
This raises multiple questions regarding parsing of whitespace and numbers:

Is whitespace significant for numbers or not? The current situation appears to be 'depends', which is unhelpful.
Is it valid to have something like "12." parse as a number? I would argue that this doesn't really make sense.

Furthermore, even though some specification of the grammar exists, how numbers are parsed is not specified: I had to discover myself that, for example, insignificant leading or trailing zeroes remain. This should likely get fixed too, and it relates to this issue, which is why I mention it here.

The short answer is that whitespace breaks tokens.  So "1200" becomes <mn>1200<mn>, while "12 00" becomes <mn>12</mn><mn>00</mn>.   So in the MathML it is two numbers, but it's up the renderer (FireFox or MathJax if you're using that) how to display that, and it appears they display them without a space.  That's not something AsciiMathML controls, so not anything to "fix" here.
Now, interestingly, it does appear "1.2" becomes <mn>1.2</mn> while "12." becomes <mn>12</mn><mo>.</mo>.  So currently "12." does not get parsed as a number, but as a number and an operator, and the rendering engine seems to add a space after operators, which is why "1. 200" is appearing to you as two numbers.  I would argue that parsing is actually wrong, and "12." should get parsed as a number, since an ending decimal point can be used to indicate trailing zeros are significant digits in science fields.  So if anything needs fixing, it'd be getting "12." to get parsed as a single number.

@drlippman Thank you for that - it's very useful information. I feel it would help a lot if all this got added to the description of the grammar on the main AsciiMath site.

Feel free to make a PR  to the website repo with suggested changes.


Update language-configuration.json

Fixes https://github.com/microsoft/vscode/issues/163181
Follow up to https://github.com/microsoft/vscode/pull/160357

@Colengms FYI.


do we support self host saml2 IDP ?

for now, only support keycloak and aliyunIDaas

@shengyuandong what's the SAML IdP you used?


Add PostgreSQL 12 support

Typeorm fixed a bug with support for PostgreSQL 12 in version 0.2.21, this PR upgrades the Typeorm dependency. See: https://github.com/typeorm/typeorm/issues/4573

@colbyeg maintaining a fork over here https://github.com/FauxJira/FauxJira


Next cell is not tappable.

I've build an app which will allow user to like and share quotes by using your library but the problem is that after a swipe next cell action buttons are not capable. We've tried to fix this issue by using your delegate method but it is not working.
Video attached - https://youtu.be/yVZlXM_TXbE

Hey
In the video I see a red background
But you're saying the buttons aren't functional
Anyway, the only thing you need to do is making sure zPosition is provided correctly for each cell
See https://github.com/amirdew/CollectionViewPagingLayout/blob/master/Lib/TransformableView.swift
func zPosition(progress: CGFloat) -> Int
I can help better if you share a simple version of your code
Or use one of the examples to replicate the issue

I'm using StackTransform and I have a red background to see if actual cell is moving but cell is not moving

The cell doesn't move, the transform only applies to the 'cardView',
'cardView' is the first subview of the cel's 'contentView' by default, you can change it though,
See https://github.com/amirdew/CollectionViewPagingLayout/blob/master/Lib/Stack/StackTransformView.swift
But you don't need that you can just move all of the views to a subview

Here is my code which I am currently building.
CollectionViewPagingLayoutTest.zip

Hey
In the video I see a red background
But you're saying the buttons aren't functional
Anyway, the only thing you need to do is making sure zPosition is provided correctly for each cell
See https://github.com/amirdew/CollectionViewPagingLayout/blob/master/Lib/TransformableView.swift
func zPosition(progress: CGFloat) -> Int
I can help better if you share a simple version of your code
Or use one of the examples to replicate the issue

Thanks for your comment. I've played with Z position but it didn't help. Still buttons are not clickable after swipe.

I just checked your code, I noticed you copied the code to your project so it was outdated or modified version.
I changed it to Swift package manager and everything works now.
Sorry for delay, I had a busy week
CollectionViewPagingLayoutTest.zip

I just checked your code, I noticed you copied the code to your project so it was outdated or modified version.
I changed it to Swift package manager and everything works now.
Sorry for delay, I had a busy week
CollectionViewPagingLayoutTest.zip

Hi. Thank you very much! Now everything is working. I've got only few layout issues but i will try to fix them :) Thank you again!


Update CompatHelper plugin

Currently, CompatHelper plugin generates the following CompatHelper.yml:
name: CompatHelper
on:
  schedule:
    - cron: 0 0 * * *
  workflow_dispatch:
jobs:
  CompatHelper:
    runs-on: ubuntu-latest
    steps:
      - name: Pkg.add("CompatHelper")
        run: julia -e 'using Pkg; Pkg.add("CompatHelper")'
      - name: CompatHelper.main()
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          COMPATHELPER_PRIV: ${{ secrets.DOCUMENTER_KEY }}
        run: julia -e 'using CompatHelper; CompatHelper.main()'

but according to the original repo, it seems that now the file needs to be like this:
name: CompatHelper
on:
  schedule:
    - cron: 0 0 * * *
  workflow_dispatch:
permissions:
  contents: write
  pull-requests: write
jobs:
  CompatHelper:
    runs-on: ubuntu-latest
    steps:
      - name: Check if Julia is already available in the PATH
        id: julia_in_path
        run: which julia
        continue-on-error: true
      - name: Install Julia, but only if it is not already available in the PATH
        uses: julia-actions/setup-julia@v1
        with:
          version: '1'
          arch: ${{ runner.arch }}
        if: steps.julia_in_path.outcome != 'success'
      - name: "Add the General registry via Git"
        run: |
          import Pkg
          ENV["JULIA_PKG_SERVER"] = ""
          Pkg.Registry.add("General")
        shell: julia --color=yes {0}
      - name: "Install CompatHelper"
        run: |
          import Pkg
          name = "CompatHelper"
          uuid = "aa819f21-2bde-4658-8897-bab36330d9b7"
          version = "3"
          Pkg.add(; name, uuid, version)
        shell: julia --color=yes {0}
      - name: "Run CompatHelper"
        run: |
          import CompatHelper
          CompatHelper.main()
        shell: julia --color=yes {0}
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          COMPATHELPER_PRIV: ${{ secrets.DOCUMENTER_KEY }}
          # COMPATHELPER_PRIV: ${{ secrets.COMPATHELPER_PRIV }}

Additionally, I noticed that github now implicitly prohibits bots from creating PRs, so I think it is nice to emit some log messages for manually enabling that.

see also https://github.com/JuliaCI/PkgTemplates.jl/issues/380


Investigate getting / importing an MPSC / MPMC queue for logging

It's easy to get one in rust, but our backends aren't all in rust, and it will be best to have something in the meantime, see https://github.com/mozilla/cubeb-coreaudio-rs/pull/180.
@ChunMinChang suggests https://github.com/facebook/folly/blob/fe37daacf861700585d303b277138cb18a92686c/folly/MPMCQueue.h or https://github.com/cameron314/concurrentqueue.

For your consideration:
https://github.com/yuzu-emu/yuzu/blob/master/src/common/bounded_threadsafe_queue.h
I contributed this queue to that project.
Its base design came from:
https://github.com/rigtorp/MPMCQueue


[stable/ambassador] add option to use old label names

What this PR does / why we need it:
When moving the ambassador chart to helm/charts we made a few breaking changes including changing labels which make migration from the old chart a bit of a headache.
Which issue this PR fixes

fixes datawire/ambassador#1268

Special notes for your reviewer:
Adds a bit of a mess to the templates but suppose thats worth it to make migration easier :)
Also removes the s in admin service name which was accidentally added. Not sure if we should leave it be or remove it to match the old chart again :)
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

I would recommend instead of this to just have users delete the helm chart and re-install it. There are other charts that have switched to the new labels and just added a note to the readme to do the above.

@steven-sheehy yeah I was thinking the same, I figured I'd present the option/possibility. But I don't like it :)
/hold

Closing this for now


Fix deployments

TODO:

[x] make sure the devel container still works.

I ran it locally with both podman-compose and docker-compose, so should be good I think :)

@ochosi if you wanna check this out if it's easier to run it now. If you feel like trying out the compose file in the devel/ dir that would be great too :)

Nice work - the deployment works neatly now! (tested with proxy 3.2.2)

Looks nice!
Regarding the dockerfile: I think this will make the caching worse, if I understand correctly: the npm install will now be re run on every change to the working directory

Looks nice!
Regarding the dockerfile: I think this will make the caching worse, if I understand correctly: the npm install will now be re run on every change to the working directory

Ah! I didn't think of that, i'll revert that change


Make Hits widget focusable and let the user navigate with arrows keys

When I do a search, I would like, as a user to be able to browse the results with the arrows of the keyboard. What could be nice is that when I hit the the return key in the search field, the hits are focused and the first element is highlighted.
What do you think?
PS : yes I know, this is not standard / common but most of the websites just fail at doing basic keyboard interaction that I don't consider them as good examples.
PPS : but I would be very interested in using great keyboard experience
PPPS : yes vim bindings would a great easter egg ;)

Google implements that actually:

from the searchbar, hit
up/down :)
enter

And we want to be like google :D

Should be doable but maybe behind an option. Using https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/tabindex?

Yes tabindex will be needed to make a div focusable.

:horse: CLOSING BOT SAYS:

This is very old and not done, reopen if you have the need and want to fix it

Looking for a solution to achieve this.

@Giovanni-Mattucci For now you will have to implement it on your side, via some JavaScript custom coding/tabIndex in your hits. This should be doable

Is there a sollution for this now?

if you are using connectHits, you can add keyboard handling there yourself for now


OSError: [Errno 24] Too many open files

Hi,noahzn
I am currently facing an issue while using training and encountering the following error: "OSError: [Errno 24] Too many open files". I was wondering if you could kindly provide some guidance on how to resolve this issue.

Hi, can you copy the complete error log?

Traceback (most recent call last):   File "train.py", line 13, in <module>     trainer.train()   File "/data/css/project/Lite-Mono-main/trainer.py", line 219, in train     self.run_epoch()   File "/data/css/project/Lite-Mono-main/trainer.py", line 234, in run_epoch     for batch_idx, inputs in enumerate(self.train_loader):   File "/home/css/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 681, in __next__     data = self._next_data()   File "/home/css/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1348, in _next_data     self._shutdown_workers()   File "/home/css/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1455, in _shutdown_workers     self._worker_result_queue.put((None, None))   File "/home/css/data/env/lite2/lib/python3.8/multiprocessing/queues.py", line 88, in put     self._start_thread()   File "/home/css/data/env/lite2/lib/python3.8/multiprocessing/queues.py", line 173, in _start_thread     self._thread.start()   File "/home/css/data/env/lite2/lib/python3.8/threading.py", line 852, in start     _start_new_thread(self._bootstrap, ()) RuntimeError: can't start new thread

Hi, can you copy the complete error log?

Traceback (most recent call last):
File "train.py", line 13, in
trainer.train()
File "/data/css/project/Lite-Mono-main/trainer.py", line 219, in train
self.run_epoch()
File "/data/css/project/Lite-Mono-main/trainer.py", line 234, in run_epoch
for batch_idx, inputs in enumerate(self.train_loader):
File "/home/css/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 681, in next
data = self._next_data()
File "/home/css/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1348, in _next_data
self._shutdown_workers()
File "/home/css/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1455, in _shutdown_workers
self._worker_result_queue.put((None, None))
File "/home/css/data/env/lite2/lib/python3.8/multiprocessing/queues.py", line 88, in put
self._start_thread()
File "/home/css/data/env/lite2/lib/python3.8/multiprocessing/queues.py", line 173, in _start_thread
self._thread.start()
File "/home/css/data/env/lite2/lib/python3.8/threading.py", line 852, in start
_start_new_thread(self._bootstrap, ())
RuntimeError: can't start new thread

Hi, can you copy the complete error log?

Traceback (most recent call last):
File "/data/env/lite2/lib/python3.8/multiprocessing/queues.py", line 239, in _feed
File "/data/env/lite2/lib/python3.8/multiprocessing/reduction.py", line 51, in dumps
File "/data/env/lite2/lib/python3.8/site-packages/torch/multiprocessing/reductions.py", line 359, in reduce_storage
File "/data/env/lite2/lib/python3.8/multiprocessing/reduction.py", line 198, in DupFd
File "/css/data/env/lite2/lib/python3.8/multiprocessing/resource_sharer.py", line 48, in init
OSError: [Errno 24] Too many open files

Hi, can you copy the complete error log?

Traceback (most recent call last):
File "/data/env/lite2/lib/python3.8/multiprocessing/queues.py", line 239, in _feed
File "/data/env/lite2/lib/python3.8/multiprocessing/reduction.py", line 51, in dumps
File "/data/env/lite2/lib/python3.8/site-packages/torch/multiprocessing/reductions.py", line 359, in reduce_storage
File "/data/env/lite2/lib/python3.8/multiprocessing/reduction.py", line 198, in DupFd
File "/data/env/lite2/lib/python3.8/multiprocessing/resource_sharer.py", line 48, in init
OSError: [Errno 24] Too many open files

Please set num_workers to a smaller value.

I tried decreasing num-workers and reducing batchsize, as well as increasing the system thread limit using ulimit -n, but I encountered the following error：
Traceback (most recent call last):
File "train.py", line 13, in
trainer.train()
File "/data/project/Lite-Mono-main/trainer.py", line 219, in train
self.run_epoch()
File "/data/project/Lite-Mono-main/trainer.py", line 234, in run_epoch
for batch_idx, inputs in enumerate(self.train_loader):
File "/home/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 681, in next
data = self._next_data()
File "/home/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1348, in _next_data
self._shutdown_workers()
File "/home/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1455, in _shutdown_workers
self._worker_result_queue.put((None, None))
File "/home/data/env/lite2/lib/python3.8/multiprocessing/queues.py", line 88, in put
self._start_thread()
File "/home/data/env/lite2/lib/python3.8/multiprocessing/queues.py", line 173, in _start_thread
self._thread.start()
File "/home/data/env/lite2/lib/python3.8/threading.py", line 852, in start
_start_new_thread(self._bootstrap, ())
RuntimeError: can't start new thread

Please set num_workers to a smaller value.

I tried decreasing num-workers and reducing batchsize, as well as increasing the system thread limit using ulimit -n, but I encountered the following error：
Traceback (most recent call last):
File "train.py", line 13, in
trainer.train()
File "/data/project/Lite-Mono-main/trainer.py", line 219, in train
self.run_epoch()
File "/data/project/Lite-Mono-main/trainer.py", line 234, in run_epoch
for batch_idx, inputs in enumerate(self.train_loader):
File "/home/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 681, in next
data = self._next_data()
File "/home/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1348, in _next_data
self._shutdown_workers()
File "/home/data/env/lite2/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1455, in _shutdown_workers
self._worker_result_queue.put((None, None))
File "/home/data/env/lite2/lib/python3.8/multiprocessing/queues.py", line 88, in put
self._start_thread()
File "/home/data/env/lite2/lib/python3.8/multiprocessing/queues.py", line 173, in _start_thread
self._thread.start()
File "/home/data/env/lite2/lib/python3.8/threading.py", line 852, in start
_start_new_thread(self._bootstrap, ())
RuntimeError: can't start new thread

Can you try setting num_workers as 0? It seems that your machine doesn't have enough cpu resources

Thank you for your response. I will switch to another GPU for training.

您可以尝试将 num_workers 设置为 0 吗？看来您的机器没有足够的 cpu 资源

Thank you for your response. I will switch to another GPU for training.


Bolt is unable to correct a problem and is using tokens

Describe the bug
The following typescript error cannot be resolved: ReferenceError: typescript is not defined
at /src/components/Tasks/TaskUploader.tsx:18:1

The system has attempted fixes like the above more than 10 times.

Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-snwqto
Steps to reproduce
Click Attempt Fix, and the application should find the problem in the code and correct it
Expected behavior
Regular operation is expected, and a login screen should display.
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 265353664,
  "usedJSHeapSize": 211233336,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = cp5337
Chat ID = 66cd85b660b8

Additional context
No response

Please add project initialization prompt.


v1.0 Milestone 4 Plan

Milestone Lead:
Milestone Buddy:
Deliverables for this milestone

[ ] Release v1.0-rc4
[ ] Addressed customer feedback from previous RC
[ ] Documentation complete
[ ] Samples and quickstarts complete
[ ] "Getting Started" usability review complete
[ ] GTM plan activities complete

Starts - 9th Jan 2021
Endgame starts - 25th Jan 2021
Release date - 28th Jan 2021

Dupe of endgame


feat: expand basic API around Fourier integrals

The statements in this PR are mostly straightforward variations around already existing statements, which have proved useful down the road.

LGTM!
maintainer merge

Thanks!
bors merge


Updated AppContext switches article

Updated AppContext switches article
Contributes to #11630

I've checked this article off from the list I'm tracking for the release at #11630

What other one, @mairaw?

I haven't checked anything off the list at #11630 -- I'll do that now.


Error running Gulp: "gulp null --color " -> Task 'null' is not in your gulpfile

Hey,
unfortunately I get an error running gulp. Gulp seems to be called with "null" as an parameter. Maybe some variable for stetting the default task can't be found.
Thanks for all your work!
Working directory: E:\XXXXXX
Using gulpfile E:\XXXXXX\gulpfile.js
Task 'null' is not in your gulpfile
Please check the documentation for proper gulpfile formatting
Program exited with code 1

fixed with 1.8.8.


Change postion and size of the Video

How I can change the position and size of the video player ?

Did you get any progress on this? I'm interested!


Carbonlink should support all relevant query types supported in mainline carbon

It looks like mainline carbon supports 4 types of queries in the carbonlink protocol (https://github.com/graphite-project/carbon/blob/77e1eb09b9708c91ffbca6e94ffd46e1143b04ee/lib/carbon/protocols.py#L127-L176)

cache-query - returns datapoints coalesced from whisper as well as in memory. This is supported in go-carbon and AFAICT is implemented completely.
cache-query-bulk - same as cache-query but allows requesting an array of metrics instead of a single metric.
get-metadata \ set-metadata - These probably don't need to be added. AFAICT these methods are original to the carbonlink protocol (https://github.com/graphite-project/carbon/commit/7549c5c3cd5917a1deb7fed8b58a28b528b2eb49) but have been unused in any discernable way.

Additionally, I think cache.CarbonlinkRequest type doesn't match the required signature for unpacking all valid carbonlink requests. I /think/ it should be something like:
type CarbonlinkRequest struct {
        Type        string // Required
        Metric      string // Required for cache-query
        Metrics    []string // Required for cache-query-bulk
        Key         string // Required for get-metadata and set-metadata
        Value      string // Required for set-metadata
}

I am not sure where they are used.
webapp/graphite/carbonlink.py does only cache-query requests

webapp/graphite/carbonlink.py does only cache-query requests

Master branch - yes, 0.9.x has cache-query-bulk implemented. But maybe it's not so relevant, because nobody yet ported that to a master, and next release will be done from the master...

cache-query-bulk is not supported by graphite-web 1.0+ :(


issue with sliding time window

Hi, I think I found a small bug when trying to run "sliding time-window" style RSA with spatial_radius=None.
Error:
File "/opt/anaconda3/lib/python3.6/site-packages/mne/evoked.py", line 705, in init
raise ValueError('Data must be a 2D array of shape (n_channels, '
ValueError: Data must be a 2D array of shape (n_channels, n_samples)
Cause:
This is caused by the fact that mne.EvokedArray expects (chan x times) array and lets (time, n)
Fix:
Just making it  a 1 x list here fixes it:
if spatial_radius is not None:
    info = epochs.info
    info = mne.pick_info(info, picks)
else:
    info = mne.create_info(['rsa'], epochs.info['sfreq'])
    **data=[data]**#!!! Here

Fixed in #19
Thanks @Ana-KG for the debugging! Sorry I didn't get to it sooner.


crcVersion: 1.29.1 : Getting issue Failed to renew TLS certificates

Hi,
While installing the crc version 1.29.1 , getting below issue:
[crc@eaas-crc ~]$ crc start --cpus 12 --memory 40000 --pull-secret-file /tmp/pullsecret.json
WARN A new version (1.31.2) has been published on https://developers.redhat.com/content-gateway/file/pub/openshift-v4/clients/crc/1.31.2/crc-linux-amd64.tar.xz
INFO Checking if running as non-root
INFO Checking if running inside WSL2
INFO Checking if crc-admin-helper executable is cached
INFO Checking for obsolete admin-helper executable
INFO Checking if running on a supported CPU architecture
INFO Checking minimum RAM requirements
INFO Checking if Virtualization is enabled
INFO Checking if KVM is enabled
INFO Checking if libvirt is installed
INFO Checking if user is part of libvirt group
INFO Checking if active user/process is currently part of the libvirt group
INFO Checking if libvirt daemon is running
INFO Checking if a supported libvirt version is installed
INFO Checking if crc-driver-libvirt is installed
INFO Checking if systemd-networkd is running
INFO Checking if NetworkManager is installed
INFO Checking if NetworkManager service is running
INFO Checking if /etc/NetworkManager/conf.d/crc-nm-dnsmasq.conf exists
INFO Checking if /etc/NetworkManager/dnsmasq.d/crc.conf exists
INFO Checking if libvirt 'crc' network is available
INFO Checking if libvirt 'crc' network is active
INFO Loading bundle: crc_libvirt_4.7.18...
INFO Creating CodeReady Containers VM for OpenShift 4.7.18...
INFO Generating new SSH Key pair...
INFO Generating new password for the kubeadmin user
INFO Starting CodeReady Containers VM for OpenShift 4.7.18...
INFO CodeReady Containers instance is running with IP <IP_ADDRESS>
INFO CodeReady Containers VM is running
INFO Updating authorized keys...
INFO Check internal and public DNS query...
INFO Check DNS query from host...
INFO Adding user's pull secret to instance disk...
INFO Adding proxy configuration to kubelet and crio service...
INFO Verifying validity of the kubelet certificates...
INFO Starting OpenShift kubelet service
INFO Kubelet client certificate has expired, renewing it... [will take up to 8 minutes]
INFO Kubelet serving certificate has expired, waiting for automatic renewal... [will take up to 8 minutes]
Failed to renew TLS certificates: please check if a newer CodeReady Containers release is available: Temporary error: certificate /var/lib/kubelet/pki/kubelet-server-current.pem still expired (x8)

[crc@eaas-crc ~]$ cat /etc/redhat-release
Red Hat Enterprise Linux Server release 7.8 (Maipo)
[crc@eaas-crc ~]$ crc version
WARN A new version (1.31.2) has been published on https://developers.redhat.com/content-gateway/file/pub/openshift-v4/clients/crc/1.31.2/crc-linux-amd64.tar.xz
CodeReady Containers version: 1.29.1+bc5f4409
OpenShift version: 4.7.18 (embedded in executable)

Please use CRC 1.31.2 before filing an issue. Older versions of CRC aren't supported.
Note; bundles will have to renew certificates 30 after they have been released. This is a necessity and adds to the startup time. For some releases of the bundle this process can not be performed as this is not a scenario that is actually supported for a single node cluster setup.

Note: If another issue is filed without, following the issue template it will be marked as invalid and immediately closed.

Hi,
But CRC 1.31.2  is not stable for RHEL 7 or 8 , as per the Redhat.
So installing this version, Please provide the proper certificate.
Thanks

But CRC 1.31.2 is not stable for RHEL 7 or 8 , as per the Redhat.

Who says? This works on RHEL 7.9 and 8.4, as tested by us.

Please provide the proper certificate.

Certificates are generated INSIDE the VM upon start when needed. The image contains a CSR (signing request), so this is not something we 'provide'.
As per #2677, #2674 and #2628 we know this is based on a nested virtualization setup. Which we have indicated is not supported either. Unless you try this on a desktop/notebook computer or bare-metal, there is not much we can do. Though, my colleague has asked you questions related to the pull-secret, but this has been unanswered. Can you please answer the questions so we can try to resolve some of the problems?

Hi,
As we cannot install on the laptop and we do not have any bare-metal.  Till now we had released the crc 4.7.11 which is stable on the OS version 7.6 .
So now we had OS version 7.8 , but then also crc get failed on the certificate, please provide the pull secret path to get download.
Thanks

please provide the pull secret path to get download

? I am not understanding what you are referring to. You are likely confusing issues I am touching on. Please answer here: https://github.com/code-ready/crc/issues/2677#issuecomment-908352249

As we cannot install on the laptop and we do not have any bare-metal.

It would have been possible to run 1.29.1 on a laptop, and recreate a bundle with crc generate for re-use on the other environment. But since this is not an option, there is not much we can do in this case.

Hi,
I had replied.
Can you provide the latest path for pull secret , we will check once this option.
Thanks

Hi,
Please can you update with pull secret path, to apply for latest crc version 4.7.x
Thanks

No 'pull secret update' is needed, the same pull secret should work for all versions of crc. For what it's worth, I just tried crc 1.29.1 on an up to date EL7 distro, and the certificate renewal was successful. What you are seeing is thus not an issue happening to everyone, but must be related to something specific to your setup. Which is why it is important to provide accurate and detailed debugging information, and not to try too many things at once so that it's possible to narrow down what is causing this issue.
But I'd recommend first using the latest crc release as if this is something we can fix in crc, we'll only be able to fix it on top of the latest release.

Hi,
Thanks, you can close.


Fixes cache use for LinkPreview

I was wondering why the cache never got a hit, we were storing optionals here but looking for non-optional URL later.

@steipete great catch! This is like the nth time I've made this error, thank you for fixing it!

@steipete thanks! :D