Missing authentication cookies... But access granted ❌

Hey there,
First of all, amazing middleware that support edge runtine 🔥👌
Got a weird issue (or I might miss-understood something)
1. Calling XHR
curl 'http://localhost:3000/api/users' \
  -X 'DELETE' \
  -H 'Content-Type: application/json' \
  --data-raw '{"collection":"users","uid":"umSF8FT5IChu1HIBco8WhItX1Yi1"}'

2. On a protected route passing throught middleware
From your amazing doc:
import { authMiddleware } from "next-firebase-auth-edge";
import { NextRequest } from "next/server";

const {
  NODE_ENV,
  NEXT_PUBLIC_FIREBASE_API_KEY,
  NEXT_PUBLIC_FIREBASE_PROJECT_ID,
  FIREBASE_CLIENT_EMAIL,
  FIREBASE_PRIVATE_KEY,
} = process.env;

if (
  !NEXT_PUBLIC_FIREBASE_API_KEY ||
  !NEXT_PUBLIC_FIREBASE_PROJECT_ID ||
  !FIREBASE_CLIENT_EMAIL ||
  !FIREBASE_PRIVATE_KEY
) {
  throw new Error("One or more environment variables are undefined.");
}

console.log("NEXT_PUBLIC_FIREBASE_API_KEY", NEXT_PUBLIC_FIREBASE_API_KEY);

export async function middleware(request: NextRequest) {
  return authMiddleware(request, {
    loginPath: "/api/login",
    logoutPath: "/api/logout",
    apiKey: NEXT_PUBLIC_FIREBASE_API_KEY!,
    cookieName: "AuthToken",
    cookieSignatureKeys: ["secret1", "secret2"],
    debug: true,
    cookieSerializeOptions: {
      path: "/",
      httpOnly: true,
      secure: NODE_ENV === "production", // Set this to true on HTTPS environments
      sameSite: "lax" as const,
      maxAge: 12 * 60 * 60 * 24, // twelve days
    },
    serviceAccount: {
      projectId: NEXT_PUBLIC_FIREBASE_PROJECT_ID!,
      clientEmail: FIREBASE_CLIENT_EMAIL!,
      privateKey: FIREBASE_PRIVATE_KEY?.replace(/\\n/g, "\n")!,
    },
  });
}

export const config = {
  matcher: ["/api/login", "/api/logout", "/api/:path*"],
};

3. Successfully logged missing Auth
ⓘ next-firebase-auth-edge: Handle request
         path: /api/users
ⓘ next-firebase-auth-edge: Missing authentication cookies
ⓘ next-firebase-auth-edge: Token is missing or has incorrect formatting. This is expected and usually means that user has not yet logged in
         reason: MISSING_CREDENTIALS
DELETE /api/users
 ✓ Compiled in 38ms (164 modules)

4. But... HTTP function is called anyway
// under /api/users/route.ts, this is called, but should not :'(
export async function DELETE(req: NextRequest) {
  console.log("DELETE /api/users");
}

Explanation
Not providing that function will cause the middleware to do nothing on un-authenticated requests.
By default, should not it cancel the request?
handleInvalidToken: async (reason) => {
      console.info("Missing or malformed credentials", { reason });

      return redirectToLogin(request, {
        path: "/",
        publicPaths: ["/"],
      });
    },

Amazing job again!
Alex

You are perfeclty right, middleware should not presume our willing...
But since this is auth related, I think I will not be the first one miss-interpreting that point :)
Anyway, great tool again, works like a charm.
Thank you for you super-fast answer!
Enjoy the week-end,
Alex


Add ecr get_authorization_token response and tests

Pretty straightforward - not sure what else to add.

Coverage increased (+0.008%) to 93.485% when pulling 025e975e446f0945c53182dbe6aebfe43c07273f on william-richard:add-ecr-get-authorization-token into 57c5c6df2619249910fff0c89c67ac450195d7c2 on spulec:master.

This isn't correct - it should be returning a b64 encoded string of the form AWS:<password.  I'm going to reopen once I have that.

Coverage increased (+0.009%) to 93.486% when pulling aeefc8056d34dc8830342bfe70326700a9df1b67 on william-richard:add-ecr-get-authorization-token into 57c5c6df2619249910fff0c89c67ac450195d7c2 on spulec:master.

This looks great, @william-richard - thanks!


Pixel Metadata Cairo contracts

Description
The news feature in PixelMetadata already started.

[ ] Add the state management of the last pixel placed in the state. Already commented out
[ ] Finish the place_pixel_metadata: place only metadata if you are the owner of the last pixel placed in the state of the contract
[ ] place_pixel_with_metadata: place a pixel and also a PixelMetadata
[ ] function to get the last pixel and pixel metadata in the map.
[ ] Tests for place pixel state, place pixel metadata

Can i work on this?

May I pick this up?

Do you have an ETA for this?

Can you ping me on Telegram too please? So we can discuss how you can contribute to AFK?
@jaykayudo @beeguy74

I'd like to handle this task.

I intend to do a decent job and at least do the draft PR within 48 hours. I ll finish task till sunday evening @MSghais

Hi @MSghais, can i work on this?
ETA - 1 day

Hi @MSghais please can I be assigned to this issue

May I try my hand at this?

May I try my hand at this?
ETA - 1 DAY

Hi @MSghais, may I work on implementing the features? This includes managing last_pixel, enforcing ownership for metadata updates, combining pixel placement with metadata, adding a getter function, and writing comprehensive tests.


Add new icon style "Only Text"

This new icon style just draws the text without any outline or solid background, just like OSX does in its default clock if you show the date.
I changed the preference window so instead of a checkbox "Show Icon Outline" it now shows a popup window with three options: Solid, Outline, Text

Itsycal can already draw text without the outline or background in the style of the system clock. You can set a datetime patten and check 'Hide icon':

Also, in general, if you want to draw text in the menubar, you can just set the title of the NSStatusItem's button. You don't need to draw it into an image.

OMG!! that is so right !!  😂

I do want to acknowledge your effort here though. I appreciate it. I have some ideas for improving and extending the UI here, but they are much bigger than a small tweak to the interface and are for another time.

Thanks for your kind words. I find the app super useful and have replaced the OSX date/time with it. Waiting anxiously for your planned upgrades!


elastalert crash when using timestamp_type: unix_ms

elastalert use search range filter with a range of doubles (the time in milisec with micro sec resolution (e.g:<PHONE_NUMBER>.100) which fails on a date elastic field with epoc_millis format.
when timestamp_type: iso in use, the query passes fine but than the returned payload failed to get parsed (becuase the date field is returned as int ...
ERROR:root:Uncaught exception running rule Demo HD Failure: Parser must be a string or character stream, not int

I think you want unix rather than unix_ms.<PHONE_NUMBER>.100 is a unix timestamp in seconds, compared to<PHONE_NUMBER>100 which would be unix_ms.


Any updates on this?

I love the idea of this gem, and noticed your request at the bottom of the README: "Any ideas or attempts to make this less of a horrible hack would be very much appreciated."
Curious if new releases of ruby might have made a less hacky version of this possible?

Aww, thanks! Unfortunately nothing has changed (and I haven't seen refinements getting a great deal of love in general). Short of releasing a modified version of Ruby there's not much that can be done. :(

Too bad.  Honestly, the refinements feature should have had this baked in from the start.  It makes more sense, and has much wider application possibilities, than the existing implementation.


Re-adjust 1Inch Solver to be slightly more aggressive

More context: https://gnosisinc.slack.com/archives/CC3C5SMB5/p1620798729034000
We may have oversteered in #580 so this PR makes it so that we can split a trade into three main parts and each part into 3 subparts. Each main path can have two connector tokens, upper bounding our solution at 27 interactions if my math is correct (3 main paths with 2 intermediate hope each being split into 3 again).
We keep the limit of 750k gas per trade (excluding GP overhead). This value could in the future be adjusted depending on current gas price and amount sold.
cc @nlordell @koeppelmann open for suggestions.
Test Plan
only a config change, so 🤷

Maybe its already getting a bit out of hand with the run time configuration, but perhaps these parameters could be made part of the LiquiditySource::1Inch enum that is passed in at runtime, with some default values. Something like 1inch(1, 3)

Does this mean that the route can only be split in thirds?

It can be split into three unequal parts as well (e.g. 60, 30, 10).
Looking at the 1Inch API response, I can see that protocols returns a three dimensional array. After some manual trial and error (during which I noticed that max_route_parts should be called main_route_parts and that complexityLevel needs to be escaped as a string not an int 🤦 ) I come to the conclusion that main_route_parts is the max length of the first level array, complexityLevel is the depth of each main route array (number of connectors) and parts the max length of the last array (in how many chunks each hop can be split).
That is
protocol: Protocol[main_route_parts][complexityLevel][parts]

Here is a sample 1Inch route that shows what is what:

This indicates that my original calculation (max 27 interactions) should be correct.

Nice :detective:


Reconnection sample please

Hi there,
Could you please expand the samples with a reconnection sample?
We're having the issue of a broken TCP connection not properly reinitialising the subscription, despite having a recursive subscription function, and as well, the callback for the catch-up-subscription going live firing again.
Thanks,
Henrik

Which type of subscription?

CatchUp. It's a funky error that only happens once in a while, and we can't reproduce it easily by shutting down/restarting ES either... So a sample just on how to reconnect lets us rule out at least that part for sure. :)

Btw, if you'd be open for an hour or two of consulting to reproduce/find the error through the logs I'd be very interested in buying that. Yesterday it stopped our read models from working, and today it stopped our integrations.

Can you post your code?

No because you want it in C# and I only can give you F#. But would it be possible to get a recommended reconnection sample please? The subscriptions cut out today as well and I don't know how to debug it.
Again, is it possible to purchase some consulting hours from you guys?

We are aware of a couple of issues in catchup subscriptions, namely https://github.com/EventStore/EventStore/issues/1127, https://github.com/EventStore/EventStore/issues/929, and this one on google groups which has some sample code on resubscribing.
If these aren't the issues you are seeing, could you post a sample of your code (F# is fine) with a bit more of an explanation on what you are seeing?
For enquiries about consulting, could you please email chris.watt at geteventstore dot com

Did you create the examples?


RunApi get_job_output fails with HTTP 406

RunApi get_job_output fails with HTTP 406
clients.ctm_api_client.rest.ApiException: (406)
Reason: Not Acceptable
HTTP response headers: HTTPHeaderDict({'Cache-Control': 'private', 'Expires': 'Wed, 31 Dec 1969 19:00:00 EST', 'Set-Cookie': 'JSESSIONID=*********************; Path=/automation-api/; Secure; HttpOnly', 'Access-Control-Allow-Origin': '*', 'Access-Control-Allow-Methods': 'GET, POST, DELETE, PUT', 'Access-Control-Allow-Headers': 'Authorization,Content-Type', 'Server': 'Jetty(9.2.z-SNAPSHOT)', 'Content-Type': 'text/json;charset=ISO-8859-1', 'Content-Length': '110', 'Date': 'Sat, 21 May 2022 18:03:28 GMT'})
HTTP response body: { "errors" : [ {"message":"Not Acceptable", "code" : "406", "uri" : "/run/job/***:***/output"} ]
  }

The problem seems to be in file run_api.py line 1845, an actual content-type is not listed

Despite error message says Content-Type': 'text/json;charset=ISO-8859-1', an actual content type is application/text

Hi,
We are releasing a new version of the python client mid/end June which has a more user friendly monitoring of the run that should fix the issue.
I will close the issue only when the new version will be released to let you know.
Thanks

@cmoraes-bmc thank you for the update, that would be nice. Current version is very counter-intuitive and makes more roadblocks than just writing code to call HTTP requests on my own.
As a side note, that would be really nice if the new version adopts async/await calls.

Hi @maxsatula  as i said before, i was waiting for the new version to go out to close the issue. Please check the new version of python client, it has support for basic monitoring actions like getting the output of a job. You can check a tutorial here: https://controlm.github.io/ctm-python-client/notebooks/monitoring.html
Thanks!


Unable to establish connection to application

Does this occur consistently? Yes.
Repro steps:

Right click on my azure app
Click on "Start remote debugging"
I then see  "starting tunnel proxy..." which appears for a few minutes before displaying "Unable to establish connection to application: Timed out"

Action: appService.StartRemoteDebug
Error type: Error
Error Message: Unable to establish connection to application: Timed out
Version: 0.20.0
OS: win32
OS Release: 10.0.18363
Product: Visual Studio Code
Product Version: 1.51.1
Language: en

Call Stack
TunnelProxy.<anonymous> extension.bundle.js:419:121486
fulfilled extension.bundle.js:419:116847

@joshdobbs what kind of app is this? Node.js, Python, etc.? Windows, Linux?

This is a Node.js app using express. @ejizba I'm using an azure app service, How can I find out if the app service is running on windows or linux?

There's a few ways to do that, but if you right click on the app in VS Code and do "View Properties", one of the first few properties should be "kind" which should let us know. For example, mine says "kind": "app,linux",

Thanks for the info. It's running on linux.

I just realized that not only can I not debug my api no longer runs on production. When I tried to deploy it I got the following in docker.log
2020-12-02T23:50:40.659Z INFO  - Initiating warmup request to container mastertrackpro-api_0_ba158686 for site mastertrackpro-api 2020-12-02T23:50:40.684Z ERROR - Container mastertrackpro-api_0_ba158686 for site mastertrackpro-api has exited, failing site start 2020-12-02T23:50:40.684Z INFO  - Initiating warmup request to container mastertrackpro-api_0_ba158686_middleware for site mastertrackpro-api 2020-12-02T23:50:44.905Z INFO  - Container mastertrackpro-api_0_ba158686_middleware for site mastertrackpro-api initialized successfully and is ready to serve requests. 2020-12-02T23:50:44.912Z ERROR - Container mastertrackpro-api_0_ba158686 didn't respond to HTTP pings on port: 8080, failing site start. See container logs for debugging. 2020-12-02T23:50:44.943Z INFO  - Stopping site mastertrackpro-api because it failed during startup.

Sorry for the late response, are you still having issues? When you say "not only can I not debug my api no longer runs on production", I would actually guess you can't debug because the app is failing to run. Are you sure the correct port is open? Is this project by chance in a public github repo so we can take a look?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Reimplement sidebar with better styling

Closes #134
This adds back the sidebar that Jared created with the controls from the main app. I had to set a specific width on the logo image to match the main client, but I think it looks okay.
@waxlamp Should we add the icon for logins here? We could also add the about dialog from the main client, too. Let me know what you think. I also added a section for future widgets, should I remove that from the PR?
@jtomeck I have a couple of questions for you (that are also in the TODO section).

Something is off about the padding, but I'm not certain exactly what it is. Would you mind taking a pass on this code and paying special attention to this?
The font here is different from the main app, should we change the font here or on the main app for consistency?

TODO:

[x] Fix the matrix legend size, it's too big as it is. It might also some alignment (@JackWilb)
[ ] Decide whether we should remove the widget section (@waxlamp)
[x] Fix general padding through the controls (@jtomeck)
[x] Decide whether we still want to use this custom font. If we do it needs to be propagated to the main client, else we could revert it to use the fonts from the main client (@jtomeck)
[x] Alex requested that the menu list items be "more compact" since we'll probably want more controls than we currently have (@JackWilb make an issue) (#154)

@JackWilb - I've made a commit that helps a lot, although I don't have any data loaded so let me know if there's more that you think is needed.
I remember looking at this font a while back.  I'm leaning towards keeping the old because I think it's causing alignment issues in components like buttons.

@JackWilb - I've made a commit that helps a lot, although I don't have any data loaded so let me know if there's more that you think is needed.

Looks great, the styling should not be affected by data loading (minus the "matrix legend" piece, which I'm planning to address). Thanks 😄

I remember looking at this font a while back.  I'm leaning towards keeping the old because I think it's causing alignment issues in components like buttons.

@jtomeck I believe the original font we were using was Avenir, and that's not a free font. The license cost looks to be 300 dollars for the package. We should choose a font that doesn't give us these alignment issues and then roll out the change for the whole app ecosystem

I really love the way this font looks. It's seemingly just the text alignment inside buttons that is an issue. It's a couple pixels below vertical centered. It's not a huge issue, but something I'd like to address when there's more funding (I'm not sure we'll want to spend too much time on this right now, but @waxlamp can correct me if I'm wrong). So I guess we just roll with this font for now and I can create an issue for button text alignment?

I really love the way this font looks. It's seemingly just the text alignment inside buttons that is an issue. It's a couple pixels below vertical centered. It's not a huge issue, but something I'd like to address when there's more funding (I'm not sure we'll want to spend too much time on this right now, but @waxlamp can correct me if I'm wrong). So I guess we just roll with this font for now and I can create an issue for button text alignment?

It sounds like you have a concrete thing you can fix with some targeted time--I'd say file a quick issue, then revisit in January. If it's critical to fix it sooner, go ahead and do it, just let me know how much time you think it will take before you start. (This doesn't sound critical; correct me if I'm wrong.)

It sounds like you have a concrete thing you can fix with some targeted time--I'd say file a quick issue, then revisit in January. If it's critical to fix it sooner, go ahead and do it, just let me know how much time you think it will take before you start. (This doesn't sound critical; correct me if I'm wrong.)

Yeah it's not critical. To be honest, let me know if you guys are even noticing it. It's very slight. Here's two screenshots. One with Roboto and one with Blinker.

Yeah it's not critical. To be honest, let me know if you guys are even noticing it. It's very slight. Here's two screenshots. One with Roboto and one with Blinker.

I wouldn't notice anything off unless you had mentioned it, and even with you mentioning it, I'm not sure I'm seeing it. The one of the left seems to be maybe a bit lower than center (did I get it right?).
FWIW, I really like the font in the left screencap. I feel it fits our project better than the one on the right.

Yeah you got it right, and I agree. I want to use this font, even with the vertical alignment issue. I posted a hacky fix in my comment above if you guys think it's worth.

Yeah you got it right, and I agree. I want to use this font, even with the vertical alignment issue. I posted a hacky fix in my comment above if you guys think it's worth.

Sorry, I missed the suggested fix in my first go-round. If you don't think that fix presents too much technical debt, then I'm ok with it. Need @JackWilb's opinion.

Yeah you got it right, and I agree. I want to use this font, even with the vertical alignment issue. I posted a hacky fix in my comment above if you guys think it's worth.

Sorry, I missed the suggested fix in my first go-round. If you don't think that fix presents too much technical debt, then I'm ok with it. Need @JackWilb's opinion.

Seems reasonable for now. We'll have to keep an eye on it and make sure that it's the only place where we see the alignment issue. If we start seeing it crop up in more places, the better solution would just be to choose a different font that is aligned properly

Yeah you got it right, and I agree. I want to use this font, even with the vertical alignment issue. I posted a hacky fix in my comment above if you guys think it's worth.

Sorry, I missed the suggested fix in my first go-round. If you don't think that fix presents too much technical debt, then I'm ok with it. Need @JackWilb's opinion.

Seems reasonable for now. We'll have to keep an eye on it and make sure that it's the only place where we see the alignment issue. If we start seeing it crop up in more places, the better solution would just be to choose a different font that is aligned properly.

I think changing the font just to fix a small alignment problem is overkill but it could be a solution.

I'll make some issues in the other repo to track this.

Which repo?

I think changing the font just to fix a small alignment problem is overkill but it could be a solution.

Not if the alignment issue is in the font, which I think might be the issue. We should just keep an eye on it

I'll make some issues in the other repo to track this.

Which repo?

Sorry, I meant for tracking the font migration for each of the repos, so multinet-client and multi-link. We should have a consistent font across the whole ecosystem.
Do you think we need an issue here to track the potential alignment issue, too?

Met with Alex and he request that the menu be "more compact"

@waxlamp Should we add the icon for logins here?

I'm not sure what to do about a login icon. I am thinking it doesn't belong in this PR in any case.
How would it work? Can we redirect folks to the OAuth login sequence for Multinet client but have them redirected to this app? Sounds like a question @AlmightyYakob could weigh in on. @JackWilb, since this will take some thinking and discussion, I would file an issue and defer discussions and decisions to there.

We could also add the about dialog from the main client, too. Let me know what you think.

An about dialog sounds like a good idea. But again, I'd defer it to future work.

I also added a section for future widgets, should I remove that from the PR?

Yes I think so. Since this is going to production, an empty "widgets" section just looks incomplete. Is there some concrete plan for what we want to put in here?

@waxlamp Should we add the icon for logins here?

I'm not sure what to do about a login icon. I am thinking it doesn't belong in this PR in any case.
How would it work? Can we redirect folks to the OAuth login sequence for Multinet client but have them redirected to this app? Sounds like a question @AlmightyYakob could weigh in on. @JackWilb, since this will take some thinking and discussion, I would file an issue and defer discussions and decisions to there.

Opened #155

We could also add the about dialog from the main client, too. Let me know what you think.

An about dialog sounds like a good idea. But again, I'd defer it to future work.

Opened #156

I also added a section for future widgets, should I remove that from the PR?

Yes I think so. Since this is going to production, an empty "widgets" section just looks incomplete. Is there some concrete plan for what we want to put in here?

No explicit plans. I replaced it with "Color Scale Legend" and moved the legend out of the controls section. I think that makes more sense and if we have widgets, we can add it later.


Update UI, make it responsive on multiple devices (mobile/tablet)

Hey, would love to help. Kindly spill what you want to do

Hey @thisudoriguy, thanks for your interest. As you can see in app.component.css, width/height have fixed values.
On larger/smaller resolutions game container is bigger or smaller. It should be the same size on every device. If it can be made to look properly on mobile devices, will introduce controls for it also.
Here is how it currently looks like on mobile and tablet:

Hey @SamirHodzic I'm also interested and will have a pop at it for you

Hey @AntonyFenton, sure, feel free to do it.

Hey @SamirHodzic Trying to push my changes but having Permission errors

You can't push directly.
Make a fork of this repository, push changes to the new branch and make a pull request.


Abort on startup: CommandLine Error: Option ... registered more than once!

Bcc static links llvm by default, if we then link LLVM again we get llvm command line fatal errors during startup.
We should either:

find out whether we can fix this at link time. Maybe if we specify bcc before llvm it grabs most of the stuff it needs from the bcc archive, preventing issues. Although it seems tricky to get right
Note in the docs that bcc needs to be built with shared llvm, see:

Did you build bcc yourself, if you did make sure you dynamic link against llvm. See debc79ef9ad4784258705a92ae70f9c7689a9c24
Originally posted by @fbs in https://github.com/iovisor/bpftrace/issues/251#issuecomment-846424774

: CommandLine Error: Option 'openmp-ir-builder-optimistic-attributes' registered more than once!

This is only known to be an issue with LLVM 12.
The current suggested workaround is to build bcc with: -DENABLE_LLVM_SHARED=1 cmake flag

-DENABLE_LLVM_SHARED=1 solve my problem.

The bcc with -DENABLE_LLVM_SHARED=1 fix worked for me, thanks!

Me too.

The bcc with -DENABLE_LLVM_SHARED=1 fix worked for me, thanks!

The bcc with -DENABLE_LLVM_SHARED=1 fix worked for me, thanks!

Thanks very much! It saved me.

Thanks very much! It saved me.

Are you dnf up on centos stream9?

No, I use apt on Ubuntu 20

Thanks very much! It saved me.

Are you dnf up on centos stream9?

No, I use apt on Ubuntu 20

I also encountered this problem when compiling bpftrace after reinstalling bcc. My solution is to delete the build directory of bpftrace and recompile (cmake, make, make install)

The bcc with -DENABLE_LLVM_SHARED=1  doesn't solve my problem
: CommandLine Error: Option 'filter' registered more than once!
LLVM ERROR: inconsistency in registered CommandLine options

The bcc with -DENABLE_LLVM_SHARED=1 doesn't solve my problem
: CommandLine Error: Option 'help-list' registered more than once!

Using Arch with LLVM version:
$ llvm-config --version
14.0.6

Recompiling bcc with -DENABLE_LLVM_SHARED=1 fixed it for me, as well.

Ubuntu 22.04:
$ llvm-config --version
14.0.0

Recompiling with -DENABLE_LLVM_SHARED=1 has not fixed the problem for me 🤔

To be clear, -DENABLE_LLVM_SHARED=1 is intended for bcc, not bpftrace.

For me, the error was caused by the bcc installed from source. I fixed it with:
cd bcc/build
sudo make uninstall

sudo apt install bpftrace bpfcc-tools libbpfcc  --reinstall

This is only known to be an issue with LLVM 12.
The current suggested workaround is to build bcc with: -DENABLE_LLVM_SHARED=1 cmake flag

@danobi just to make it clear, when building bcc we will do
git clone https://github.com/iovisor/bcc.git
mkdir bcc/build; cd bcc/build
cmake ..
make
sudo make install
cmake -DPYTHON_CMD=python3 -DENABLE_LLVM_SHARED=1..
pushd src/python/
make
sudo make install
popd

I get a similar error on ubuntu 24.10 (oracular). I installed the bpftrace package and all dependencies using apt.
apt info bpftrace
Package: bpftrace
Version: 0.21.2-1ubuntu1

bpftrace --version
: CommandLine Error: Option 'debug-counter' registered more than once!
LLVM ERROR: inconsistency in registered CommandLine options
Aborted (core dumped)

running bpftrace in the focal container seems fine
docker run -it ubuntu:focal /bin/sh -c "apt update; apt install -y bpftrace; bpftrace --version"
...
bpftrace v0.9.4

with the oracular container i can reproduce the issue i have on my host:
docker run -it ubuntu:oracular /bin/sh -c "apt update; apt install -y bpftrace; bpftrace"
...
: CommandLine Error: Option 'debug-counter' registered more than once!
LLVM ERROR: inconsistency in registered CommandLine options
Aborted (core dumped)

I'm confused about the bcc dependency situation.  According to https://github.com/bpftrace/bpftrace/pull/2265 it should be removed, right?
it's replaced by libbcc in focal:
docker run -it ubuntu:focal /bin/sh -c "apt update; apt install -y bpftrace;ldd $(which bpftrace) | grep bcc"
...
	libbcc.so.0 => /lib/x86_64-linux-gnu/libbcc.so.0 (0x000078e1d4400000)

however it's back in oracular:
docker run -it ubuntu:oracular /bin/sh -c "apt update; apt install -y bpftrace; ldd $(which bpftrace) | grep bcc"
...
		libbcc_bpf.so.0 => /lib/x86_64-linux-gnu/libbcc_bpf.so.0 (0x00007cf23ad98000)

I'm confused about the bcc dependency situation. According to https://github.com/bpftrace/bpftrace/pull/2265 it should be removed, right?

Unfortunately, we still use bcc for some things e.g. symbolication and some probe lookup/attaching but we're hoping to get rid of it completely by end of next year.
@devoli170 As far as the crash goes, that looks like an LLVM issue. What version of LLVM are you on? Maybe try upgrading it past 12.

llvm-cov --version
Ubuntu LLVM version 19.1.1
  Optimized build.

however the bpftrace package depends on libllvm18, so this lib is installed alongside the llvm-19 packages. bpftrace also uses libLLVM-18.so.18.1 => /lib/x86_64-linux-gnu/libLLVM-18.so.18.1

Hello everyone! My Ubuntu release is 22.04. I installed bcc from the source (without the flag -DENABLE_LLVM_SHARED=1) and then bpftrace from apt. What I've noticed through the command, dpkg -l | grep llvm is that they both installing different releases of LLVM (11, and 13). Should I need to uninstall bcc and build it again using the DENABLE_LLVM_SHARED=1 flag and also installing bpftrace from the source?

On Ubuntu this is caused by having two copies of LLVM in the same process.

bpfcc is linked to LLVM dynamically as expected.
bpftrace pulls in the static libraries, despite Ubuntu's build process not asking for it.

This causes to copies of LLVM to be loaded into the resulting bpftrace process.
After some investigation, this is due to the following check failing in src/ast/CMakeLists.txt:
  find_library(found_LLVM LLVM HINTS ${LLVM_LIBRARY_DIRS})
  if(found_LLVM)

This is despite find_package correctly finding our LLVM. Removing this check and always dynamically linking LLVM if STATIC_LINKING=OFF` seems to fix the issue.
To users in this thread, the fix will be available in Ubuntu 24.10 as soon as feasible.


add ServerName to tls.Config when TLSAcceptAllCerts is false

this PR solves the issue "IMAP StartTLS - unclear error message #61"

Coverage increased (+0.3%) to 57.562% when pulling 9e6c8b2b71508152ec482b3c4526e03e3dcb76d5 on starttls into d338c4f49450b7ea62821b97e8424a043d2b971b on master.


Can't import SimplebarAngularModule on newly created Angular 15 project

Describe the bug
As soon as I add
import { SimplebarAngularModule } from 'simplebar-angular';
to my newly created Angular app the part 'simplebar-angular' is being red-underlined by Visual Studio Code complaining "Cannot find module 'simplebar-angular' or its corresponding type declarations.". My package.json looks like this:
"name": "example-app",
"version": "0.0.0",
"scripts": {
"ng": "ng",
"start": "ng serve",
"build": "ng build",
"watch": "ng build --watch --configuration development",
"test": "ng test"
},
"private": true,
"dependencies": {
"@angular/animations": "^15.1.0",
"@angular/common": "^15.1.0",
"@angular/compiler": "^15.1.0",
"@angular/core": "^15.1.0",
"@angular/forms": "^15.1.0",
"@angular/platform-browser": "^15.1.0",
"@angular/platform-browser-dynamic": "^15.1.0",
"@angular/router": "^15.1.0",
"rxjs": "~7.8.0",
"simplebar-angular": "^3.2.0",
"tslib": "^2.5.0",
"zone.js": "~0.12.0"
},
"devDependencies": {
"@angular-devkit/build-angular": "^15.1.3",
"@angular/cli": "~15.1.3",
"@angular/compiler-cli": "^15.1.0",
"@types/jasmine": "~4.3.0",
"jasmine-core": "~4.5.0",
"karma": "~6.4.0",
"karma-chrome-launcher": "~3.1.0",
"karma-coverage": "~2.2.0",
"karma-jasmine": "~5.1.0",
"karma-jasmine-html-reporter": "~2.0.0",
"typescript": "~4.9.4"
}
}
To Reproduce
Steps to reproduce the behavior:

Open Visual Studio Code with installed @angular/cli
Create new project with "ng new example-app"
Open newly created project
Install simplebar-angular by using "npm install simplebar-angular --save"
Add import to app.module.ts as described in the simplebar-angular documentation ( import { SimplebarAngularModule } from 'simplebar-angular'; )

Expected behavior
I can import the SimplebarAngularModule and use it in my Angular app
Additional context
I am on a Windows 10 machine with the latest updates and I am using Visual Studio Code for development
Your environment

Software
Version(s)

simplebar-angular
3.2.0

npm
8.19.3

Operating System
Windows 10

I just released a new version that should fix it.

Great! It seems to work! Thanks for the quick fix!


add random credit card

Description
Add random visa credit card
Screenshot
Optional

Checklist

[x] API used in this PR was not used in just-calling-random-api before.
[x] API used in this PR is Public or I have permission to use it.
[x] My contribution is working.
[x] My contribution doesn't expose any API keys.
[x] My changes don't generate any new warnings.
[x] My changes don't affect any work done by other developers.

To tick a box, replace [ ] with [x].

Hi @vedangj044 please kindly help review on this and thank your good project.

already fix some conflic

Yeah, I resolved it too. No issues. Thanks for the PR :handshake:


[2/x] Move veNft data to its own project type and change when it is fetched

What does this PR do and why?
Removes veNft context from V2ProjectContext and puts it into its own context type. It is fetched when the user opens the drawer and not at the project level.
Closes #1529
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #1586

PR #1587   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: 5peKCVtRG7hIeC6BSrBI

Graphite Merge Job
Current status: 🔄 Waiting for checks to pass
This pull request is currently waiting for CI and mergeability checks to pass before being merged.
This comment was auto-generated by Graphite.

Job Reference: 0agyc4HkFZeecGjH4TTU


DUPLO-14597: forces resource replacement on changes to is_cluster_autoscaled and can_scale_from_zero

User description
Change description

Changing the key from "tenant_id" to "id" as the field name differs between the resource schema and its datasource counterpart
Added "ForceNew: true" to the "is_cluster_autoscaled" and "can_scale_from_zero" attributes. This forces a new resource to be created when these attributes are changed.

Type of change

[x] Bug fix (fixes an issue)
[ ] New feature (adds functionality)

Related issues
No related issues
Checklists
Development

[ ] Pull requests may not be submitted to the master branch (use develop instead) - or they will be closed.
[ ] Lint rules pass locally
[ ] Application changes have been tested thoroughly
[ ] Automated tests covering modified code pass

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Code review

[ ] Pull request has a descriptive title and context useful to a reviewer.
[ ] "Ready for review" label attached and reviewers assigned
[ ] Changes have been reviewed by at least one other contributor
[ ] Pull request linked to task tracker where applicable

Type
Bug fix, Enhancement

Description

This PR primarily addresses two issues.
First, it fixes a bug in the dataSourceTenantRead function in duplocloud/data_source_duplo_tenant.go where the key "tenant_id" was incorrectly used instead of "id".
Second, it enhances the autoscalingGroupSchema function in duplocloud/resource_duplo_asg_profile.go by forcing a new resource to be created when the "is_cluster_autoscaled" and "can_scale_from_zero" attributes are changed.
The changes have been documented in the CHANGELOG.

Changes walkthrough
Relevant filesBug fix

data_source_duplo_tenant.goRefactor tenant data source read function

duplocloud/data_source_duplo_tenant.go

Changed the key from "tenant_id" to "id" in the dataSourceTenantRead   function.

+1/-1

Enhancement

resource_duplo_asg_profile.goEnforce resource replacement for autoscaling attributes

duplocloud/resource_duplo_asg_profile.go

Added "ForceNew: true" to the "is_cluster_autoscaled" and   "can_scale_from_zero" attributes in the autoscalingGroupSchema   function.

+2/-0

Documentation

CHANGELOG.mdUpdate CHANGELOG with recent fixes and enhancements

CHANGELOG.md

Added entries for the changes made in  duplocloud/data_source_duplo_tenant.go and  duplocloud/resource_duplo_asg_profile.go.

+4/-0

 ✨ Usage guide:
Overview:
The describe tool scans the PR code changes, and generates a description for the PR - title, type, summary, walkthrough and labels. The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on a PR.
When commenting, to edit configurations related to the describe tool (pr_description section), use the following template:
/describe --pr_description.some_config1=... --pr_description.some_config2=...

With a configuration file, use the following template:
[pr_description]
some_config1=...
some_config2=...

  Enabling\disabling automation

When you first install the app, the default mode for the describe tool is:

pr_commands = ["/describe --pr_description.add_original_user_description=true"
                         "--pr_description.keep_original_user_title=true", ...]

meaning the describe tool will run automatically on every PR, will keep the original title, and will add the original user description above the generated description.

Markers are an alternative way to control the generated description, to give maximal control to the user. If you set:

pr_commands = ["/describe --pr_description.use_description_markers=true", ...]

the tool will replace every marker of the form pr_agent:marker_name in the PR description with the relevant content, where marker_name is one of the following:

type: the PR type.
summary: the PR summary.
walkthrough: the PR walkthrough.

Note that when markers are enabled, if the original PR description does not contain any markers, the tool will not alter the description at all.

  Custom labels
The default labels of the describe tool are quite generic: [Bug fix, Tests, Enhancement, Documentation, Other].
If you specify custom labels in the repo's labels page or via configuration file, you can get tailored labels for your use cases.
Examples for custom labels:

Main topic:performance - pr_agent:The main topic of this PR is performance
New endpoint - pr_agent:A new endpoint was added in this PR
SQL query - pr_agent:A new SQL query was added in this PR
Dockerfile changes - pr_agent:The PR contains changes in the Dockerfile
...

The list above is eclectic, and aims to give an idea of different possibilities. Define custom labels that are relevant for your repo and use cases.
Note that Labels are not mutually exclusive, so you can add multiple label categories.
Make sure to provide proper title, and a detailed and well-phrased description for each label, so the tool will know when to suggest it.

  Utilizing extra instructions
The describe tool can be configured with extra instructions, to guide the model to a feedback tailored to the needs of your project.
Be specific, clear, and concise in the instructions. With extra instructions, you are the prompter. Notice that the general structure of the description is fixed, and cannot be changed. Extra instructions can change the content or style of each sub-section of the PR description.
Examples for extra instructions:
[pr_description]
extra_instructions="""
- The PR title should be in the format: '<PR type>: <title>'
- The title should be short and concise (up to 10 words)
- ...
"""

Use triple quotes to write multi-line instructions. Use bullet points to make the instructions more readable.

  More PR-Agent commands

To invoke the PR-Agent, add a comment using one of the following commands:

/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs 💎: Generate docstring for new components introduced in the PR.
/generate_labels 💎: Generate labels for the PR based on the PR's contents.
/analyze 💎: Automatically analyzes the PR, and presents changes walkthrough for each component.

See the tools guide for more details.
To list the possible configuration parameters, add a /config comment.

See the describe usage page for a comprehensive guide on using this tool.

Updated title and changelog file


Update Composer dependencies (2024-08-24-00-19)

Visual regression test passed!
View the visual regression test report


Duplicate documents inserted when importing GTFS bundle with multiple agencies

The bundle imported had 2 agencies:
agency.txt
"agency_id","agency_name","agency_url","agency_timezone","agency_lang","agency_phone"
"NSWTrainLink","NSW TrainLink","http://www.nswtrainlink.info/","Australia/Sydney","en","131500"
"SydneyTrains","Sydney Trains","http://www.sydneytrains.info/","Australia/Sydney","en","131500"

config.js
{
  "mongo_url": "mongodb://<IP_ADDRESS>:27017/tfnsw",
  "agencies": [
    {
      "agency_key": "SydneyTrains",
      "path": "../gtfs-files/sydneytrains.zip"
    },
    {
      "agency_key": "NSWTrainLink",
      "path": "../gtfs-files/sydneytrains.zip"
    }
  ]
}

Entire bundle is imported twice:
$ npm run import

><EMAIL_ADDRESS>import M:\tfnsw\node-gtfs
> node lib/import.js

SydneyTrains: Importing GTFS from ../gtfs-files/sydneytrains.zip
SydneyTrains: Importing - agency.txt - 2 lines imported
SydneyTrains: Importing - calendar_dates.txt - No file found
SydneyTrains: Importing - calendar.txt - 117 lines imported
SydneyTrains: Importing - fare_attributes.txt - No file found
SydneyTrains: Importing - fare_rules.txt - No file found
SydneyTrains: Importing - feed_info.txt - No file found
SydneyTrains: Importing - frequencies.txt - No file found
SydneyTrains: Importing - routes.txt - 133 lines imported
SydneyTrains: Importing - shapes.txt - 82724 lines imported
SydneyTrains: Importing - stop_times.txt - 799299 lines imported
SydneyTrains: Importing - stops.txt - 1236 lines imported
SydneyTrains: Importing - transfers.txt - No file found
SydneyTrains: Importing - trips.txt - 42371 lines imported
SydneyTrains: Post Processing data
SydneyTrains: Completed
NSWTrainLink: Importing GTFS from ../gtfs-files/sydneytrains.zip
NSWTrainLink: Importing - agency.txt - 2 lines imported
NSWTrainLink: Importing - calendar_dates.txt - No file found
NSWTrainLink: Importing - calendar.txt - 117 lines imported
NSWTrainLink: Importing - fare_attributes.txt - No file found
NSWTrainLink: Importing - fare_rules.txt - No file found
NSWTrainLink: Importing - feed_info.txt - No file found
NSWTrainLink: Importing - frequencies.txt - No file found
NSWTrainLink: Importing - routes.txt - 133 lines imported
NSWTrainLink: Importing - shapes.txt - 82724 lines imported
NSWTrainLink: Importing - stop_times.txt - 799299 lines imported
NSWTrainLink: Importing - stops.txt - 1236 lines imported
NSWTrainLink: Importing - transfers.txt - No file found
NSWTrainLink: Importing - trips.txt - 42371 lines imported
NSWTrainLink: Post Processing data
NSWTrainLink: Completed
All agencies completed (2 total)

Rows are duplicated:
> db.agencies.count()
4
> db.calendars.count()
234
> db.routes.count()
266
> db.shapes.count()
165448
> db.stops.count()
2472
> db.stoptimes.count()
1598598
> db.trips.count()
84742

Thanks for reporting this.
Can you link me to the GTFS file with this issue so I can reproduce it?

I now see the issue - you should only include each GTFS file once in your agencies array.  I updated the README to try to make this more clear, but I agree that it is confusing that the configuration array is names agencies instead of something like files.
Your config.json should look like:
{
  "mongo_url": "mongodb://<IP_ADDRESS>:27017/tfnsw",
  "agencies": [
    {
      "agency_key": "SydneyTrains",
      "path": "../gtfs-files/sydneytrains.zip"
    }
  ]
}

Once the data is imported, you can query routes and agencies by either agency_key which refers to all agencies in your GTFS file or agency_id which is the specific agency within your file.
Take a look at the updated readme and let me know how I can document this to make it more clear.


Missing imports

Fixed more missing imports and added the newest version of the linter instead of the hacky Python one

Very mysterious why random_between can't be found. :cry:


fix-faces difference after 2.2.0

the example in readme:
imagine "a couple smiling" --steps 40 --seed 1 --fix-faces
now no longer yields results as shown unless changed to:
imagine "a couple smiling" --steps 40 --seed 1 --fix-faces --fix-faces-fidelity 0
so not sure if that's intended, or the default value for an unspecified --fix-face-fidelity is wrong, or not working?

This is probably due to the changes in 48117bbcb11352273cc94602d532fcca2c6bfad8. The default fidelity was also changed from 0.2 to 0.5, where 0.2 was hardcoded and 0.5 is not the default argument value for ImaginePrompt.

I did intend to change but I should have just left it alone.  I'll revert back the default to 0.2

I set it back to 0.2


Added two missing example settings for unattended installation

This is simply a suggestion for adding a bit more documentation for unattended installation options on Debian based Linux systems

Sorry, I missed your PR @black0range. The documentation content was moved: https://github.com/arangodb/docs/pull/260


Add other data like reports to the approval

I want to be able to add comment and other data necessary to approval

Go to config/approvals.php
enable_approval_comments => true

Can I also include attachments like images

No, you’d need to override the method and build your own for that.
Many thanks
Tony
On Wed, 29 May 2024 at 20:55, Komolafe Temiloluwa Ayomide <
@.***> wrote:

Can I also include attachments like images
—
Reply to this email directly, view it on GitHub
https://github.com/eighty9nine/filament-approvals/issues/20#issuecomment-2138160691,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKWBFWPI4SUJOAIWND6MJTZEYXETAVCNFSM6AAAAABICWHFD2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZYGE3DANRZGE
.
You are receiving this because you commented.Message ID:
@.***>

Can you please give me a guide about how to go about that

Hello @T3mmyKay  please continue this discussion in the discover server of the plugin


custom fields: use VocabularyRelationSchema

related to https://github.com/inveniosoftware/invenio-vocabularies/issues/188

[x] implement BaseVocabularyRelationSchema. With id, so others like generic and contrib can extend.

[x] implement VocabularyRelationSchema and others needed for contribs, inheriting from the first one.

[x] Remove the usage of VocabularySchema in invenio-communities and invenio-rdm-communities

[x] Use VocabularyRelationsSchema as returned field in the VocabularyCF.field function. Need to use fields.Nested.

might need also UNKONW=EXCLUDE

PRs were merged


Test changed to work on older kernels

Socket option used for test was only introduced in 2.6.32. For the few
unfortunate souls that still depend older systems and compilers like gcc
4.1 this is a no go.


Pytorch optimization through FBP

Hi:
Firstly, thank you very much for the amazing tool.
I have a question for the optimization through the FBP algorithm. For example, if I use the forward projection, the optimization is working (no error).
Set up the projector
proj = Projector(forward_project=True, use_static=True, use_gpu=True, gpu_device=0, batch_size=1)
proj.set_volume(512, 512, 4, 1, 1, 0, 0, 0)
proj.set_conebeam(360, 8, 512, 1, 1, 256, 256, proj.leapct.setAngleArray(360, 360.0), 1000, 1500, tau=0.0, helicalPitch=0.0)
Opitmize the g_truth through the forward projection
g_truth= torch.zeros([1,4,512,512])
g_truth.requires_grad = True
f_estimated = self.projector(g_truth)
loss = self.loss_func(f_estimated.cpu().float(), torch.ones([1,360, 8, 512]).cpu().float())
loss.backward()
Opitmize the f_truth through the FBP reconstruction
However, if I use the FBP, the pytorch optimize is unable to find any gradient:
f_truth= torch.zeros([1,360, 8, 512])
f_truth.requires_grad = True
g_estimated = self.projector.fbp(f_truth)
loss = self.loss_func(g_estimated.cpu().float(), torch.ones([1,4,512,512]).cpu().float())
loss.backward()
gives "RuntimeError: element 0 of tensors does not require grad and does not have a grad_fn".
Could you help me to see whether the FBP algorithm is not supporting the optimization?
Thank you so much.

Hello.
Before I get into your main questions, I'd like to discuss some other things.
First of all, it seems that in your set_conebeam function you are setting your center row to 256, but you only have 4 detector rows.  You can do this, but this would mean that your detector is shifted very far up and I assume this is not what you wanted.  A perfectly centered detector would have this value at (4-1)/2.
Next, we usually use the variable "f" for the reconstruction volume and "g" for the projection data.  It seems like you have these switched in your code above which is a little confusing to me.  You can use whatever variable names you want, but if you switch these names it might be hard to read our code and it may contribute to some of the problems you are experiencing.
Lastly, I am not exactly sure what you are trying to do, but I'll make some comments, but feel free to write back if I did not answer your question.  OK, we do provide an implementation of FBP, but it is not setup to be a differentiable operator in our package.  If you check our Projector class, only our forward and backprojections are differentiable.  If you are interested in the derivative of FBP, you will have to implement this yourself, but it is not trivial to do.  You could implement the adjoint of FBP as the following.
proj.leapct.project(...)
proj.leapct.filterProjections(...)

Hi:
I think your answer is perfect actually. I will try the adjoint of FBP (I see them in the instruction) so I can calculate the derivative of FBP. Thank you so much!
Also I notice your suggestions and they are very valuable. Thank you so much!


optimize：Cj subsequent optimization

English | 简体中文
PR
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our Commit Message Guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] Built its own designer, fully self-validated

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

Background and solution
后续优化

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

Summary by CodeRabbit

New Features

Enhanced plugin navigation panel with drag-and-drop functionality for top and bottom navigation items.
Introduced a toggle feature for an AI robot component to improve user interaction with AI features.

Improvements

Updated confirmation logic for switching plugins to enhance user experience.
Refined context menu logic for more dynamic interactions.
Added visual indicators for drag state in the navigation and canvas components.

Refactor

Improved overall structure and state management of the navigation lists for better organization and maintainability.

插件隐藏功能没有生效


String quotes break the line

Some string are declared as follows:
lableA:
    .string "abc"
lableB:
    .acsiiz "abc"

However after formatting
lableA:
    .string "abc
"
lableB:
    .acsiiz "abc
"

The quotes break the line.

After further testing, I think this will  only happened when there is a comment above the lable.


MISSING ADA / TOKENS - IS THIS WALLET A SCAM??

I transfered my ADA and token to the Nami wallet, but it never arrived and seems to be in another wallet!! is this wallet a scam??

@alihassan01 Talk to Cardano Support on the Live support page so that your issue can be looked into.


Adding listen address option so that IPv6 and loopback interfaces can be used.

reference: #2477

I signed the cla. Not exactly sure how this works. Is that all I have to do?

LGTM

@grifball could you squash commits in to one
Thanks

I started writing something that automatically tries to set hostname when listenAddress is set but hostname isn't, but it seems out of scope. Figuring out which IP to use for hostname based on the value of listenAddress is really difficult, and can even be impossible in more complex interface setups. Maybe this isn't what you had in mind when you said "influence?"
I think that a better solution would be to give a warning if listenAddress is set and hostname isn't.

Let me know what you think of this solution. I'm not sure how to change the label on this PR.
I'll squash the commits once the code looks good.

Thank you, yes I think it's fine to just warn the user about this. Left some comments about the implementation.

Alright, I implemented those changes. Let me know if anything else needs changing.

Do we want weak equality (== & !=) checks here? Seems like a better way would be to check specifically if it is a non-trivial string.

This should be a safe enough check. Unless they're setting these values to null and expecting the server to work correctly, which explicit string checks wouldn't solve anyway.
If they're setting these values to non-string types, I assume there's other checks in the config file for this, but it's out of scope in this PR anyway. The added code won't change how we handle config type checking.

I added a simple test to make sure that listen address was called with the configured listenAddress. That should cover all the functionality for this PR.

Thank you :octocat:


Apr 20 17:47:44 witherspoon kernel[903]: [ 1109.950000] systemd: 36 output lines suppressed due to ratelimiting
Apr 20 17:47:44 witherspoon systemd[1]: Freezing execution.
Apr 20 17:47:44 witherspoon kernel[903]: [ 1109.950000] systemd-journald[644]: Failed to send WATCHDOG=1 notification message: Connection refused
root@witherspoon:~# uptime
 18:12:14 up 43 min,  load average: 0.52, 0.39, 0.36
root@witherspoon:~#

After discussion,  looks like there's a config file the systemd watchdog needs to be enabled in, and it isn't.

Oops wrong repo.. closing it..


crashed after signing up a logining in

Describe the bug
crashed after signing up a logining in
was working yesterday
To Reproduce
updated lens 5
was forced to create an account cause the 'skip' button didn't show
logged in
tried accessing my gcp cluster
crashed
'App crash at /overview'
Component stack:
at Icon (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:582595:9)
at _a (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:594495:17)
at p
at a
at http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:454338:23
at http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:454384:31
at http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:454445:30
at div
at NonInjectedSidebarItem (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/chunks/888.js:571:9)
at div
at div
at NonInjectedSidebar (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/chunks/888.js:780:26)
at div
at div
at NonInjectedMainLayout (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:586079:9)
at ErrorBoundary (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:582243:9)
at NonInjectedClusterFrame (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/chunks/888.js:2924:9)
at ThemeProvider (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:127160:24)
at Router (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:454780:30)

Error stack:
Error: Cannot find module './

.svg'
    at webpackContextResolve (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:615307:11)
    at webpackContext (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:615302:11)
    at Icon.render (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:582651:100)
    at qi (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:453323:188)
    at pi (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:453322:173)
    at ck (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:453405:427)
    at bk (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:453386:347)
    at ak (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:453386:278)
    at Tj (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:453386:138)
    at Lj (http://3fd37ed679b6191bda5b3e90cb182af8.localhost:50602/build/Lens.js?700ce8d46c05fdd6c192:453379:163)

What platform are you on? We have not run be able to reproduce on windows or on macos

on my side with macos 12.4 and m1 chip after login can't see any pods (blank window)

Oh okay, thanks for the full screenshot. I think this is a regression from https://github.com/lensapp/lens/pull/4894, will be creating a fix ASAP


-DcutOffDepth=N isn't how cutOffDepth works from the command line

The wiki documentation talks of 'cutOffDepth' as a setting but not how to set it, unless I missed it. I tried -DcutOffDepth=1 on the command line, but got a ton of nodes I was not expecting

I guess you are referring to the documentation of the Target Include Filter. The cutOffDepth is not a parameter of the depgraph-maven-plugin. It is only an internal counter that is used to construct dependency graphs when the targetIncludes parameter is set (this will construct a graph that contains only the paths to the dependencies matching the parameter).
I don't think that a cutOffDepth parameter is very useful for the plugin, because it would omit dependencies more or less randomly. I suggest you try some of the various filters (includes/excludes, transitiveIncludes/transitiveExcludes, targetIncludes) to reduce the nodes in your graph. These filters are quite powerful and can reduce the nodes significantly. There is also an entry in the FAQ about that topic.

I'd have to know which groupID/artifactIDs to exclude, then. OK


After restart then boot usb flash drive then again error Thinkpad Laptop T460s

Halting on critical error                                                    OC: Grabbed zero system-id for SB, this is not allowed

I had the same issue. I was able to get past that by editing config.plist and setting the value for SecureBootModel to "Disabled" instead of "Default" (found at https://www.reddit.com/r/hackintosh/comments/pkrntp/opencore_booting_error/hc8yjla/)


Set up more intelligent URL path escaping

When a non-fully-qualified URL is passed, we're currently not doing any encoding on it (pending #8). This should probably be updated to at least be compliant with the "reserved" section of IETF RFC 3986. I'm going to dig into the best way to handle this. Hopefully URI.js has something helpful built-in, I just have to see if that's the case.

Alrighty. So, I did a bit more digging on this. IETF RFC 3986 includes the following regular expression in Appendix B, "Parsing a URI Reference with a Regular Expression":
^(([^:/?#]+):)?(//([^/?#]*))?([^?#]*)(\?([^#]*))?(#(.*))?

I tested it in Rubular with their test URL and our use case, and either %20 or + are acceptable in any part of the URL. From what I've been able to find, however, the + character is supposed to be interpreted as a literal plus sign when it comes before the query string, instead of a space.
Since we're using URI.js' URI.encodeReserved now anyway, I'm just going to close this issue.


What is the relation between CpuThreshold option and request/limit cpu resource k8s deployment config?

I am using go-zero in a service deployed on k8s, and I set request / limit cpu 4 cores (4000m). I found that the range of CpuThreshold is [0:1000), so I set 950. Under this config, I found my service cannot occupy all the cpu I request, and the rpc service will response service overloaded when cpu usage is near 1000m。
What can I do to make my service occupy all the resource I request？

950 means 95% of your cpu limit. And 95% is a moving average value, you can consider it as the average value of last 5 seconds.

that is strange.
I review my config, I set resource like this:

then i got an error before:

but at that time, cpu monitoring looked like this:

it seems not touching the threshold.

Hi, any update?


ng serve --proxy-config proxy.conf.json no longer works for me when going to beta 31 (from beta 24)

After upgrading from cli beta 24 to beta 31 the proxy functionality has changed and no longer generates a /dist folder when doing a ng serve.  This is causing a problem because I use the proxy functionality to serve the app from an express app in the same project.
OS?

Mac OSX Sierra

Versions.
node --version: 6.9.1
ng --version:
@angular/cli: 1.0.0-beta.31
node: 6.9.1
os: darwin x64
@angular/common: 2.4.7
@angular/compiler: 2.4.7
@angular/core: 2.4.7
@angular/forms: 2.4.7
@angular/http: 2.4.7
@angular/material: 2.0.0-alpha.10
@angular/platform-browser: 2.4.7
@angular/platform-browser-dynamic: 2.4.7
@angular/router: 3.4.7
@angular/cli: 1.0.0-beta.31
@angular/compiler-cli: 2.4.7
Repro steps.
I just upgraded from beta 24 to beta 31.
I am running:
ng serve --proxy-config proxy.conf.json
my proxy.conf.json is as follows:
{
  "/": {
    "target": "http://localhost:3000",
    "secure": false
  }
}

NOTE: my project contains an express app that is used to serve the angular app.  We also use the express app for our API. Also, we serve the index.html file from the express app because we are dynamically setting the base href.  I guess we could figure out a better way to deal with the base href if needed.
Previously, in beta 24 this worked worked great, but in beta 31 it does not create the /dist folder so this proxy no longer works because the express app (running on port 3000) cannot find any of the files in the /dist folder because it is not there.
I thought that maybe there was a way to only proxy the API calls and not the index.html or any of the css or js, so I tried the following proxy.conf.json:
{
  "/analytics/api-dev": {
    "target": "http://localhost:3000/analytics/api-dev",
    "secure": false
  }
}

When I run:
It will now start to load the app, it looks like it was able to find my indext.html but I get a 404 on all of the CSS and js files, for instance here are a couple it 404's on:
GET http://localhost:4200/fleet/assets/dsh_main.css
GET http://localhost:4200/fleet/inline.bundle.js
GET http://localhost:4200/fleet/scripts.bundle.js
Is there a way to only proxy the API calls and have the index.html, css and js files not go thru the proxy?
Or is there a way to still get it to build the /dist folder and keep it up to date?
I won't be able to upgrade until I can solve this problem.
Thanks so much for your help!!!
... Ed

ng serve builds and serves everything in memory (beta.24 did this as well).  A recent change was made that causes ng serve to delete the output directory upon execution to prevent confusion in this regard.
An option, if you are serving the app with express, is to use build --watch.  This will give you an up to date output that express can then serve.

@clydin THANKS!!! This works great!
Do you know if there is a way to get the live reload to work with a build --watch?

Unfortunately, there's no live reload support for build --watch.

@clydin ok, thanks so much for your help.


Show contributors across all fern repos

It would be nice if all contributors to the generators (go/python/java/typescript) also got acknowledged!

That'd be very nice. I've looked into some of the options we have available:

There's a specification/app called all-contributors which supports maintaining a list of contributors outside of code contributions as well, e.g. translations, design, etc. We could use this, but it would entail installing a bot and following its workflows in order to maintain the contributor list over time.
There is a GitHub Action available (or an npm command) to merge all the contributors across repositories under our org into one, which we could then display in our main README.

We're currently using contrib.rocks - we could use the image for multiple repositories, but that means you, Zack, Danny and others would appear multiple times.
First one is more thorough, would take up more space in the readme, but overall would be more complete. The latter is a quick solution.

Deprioritizing this for now.


include and shade MinigameCore API

depends on #11, dont merge til that one is resolved
closes #14

This doesn't seem to depend on #11, this PR also adds SpongeStart.

um
well
xD
i checked out a new branch from that branch :/

the reason is that i have specific targets with each PR and dont want to mix them up

um i realized what i did wrong fixing.. the spongestart line was not supposed to be in this PR at all i copypasta-ed from a private gist :/

Copying comment from https://github.com/MinigameCore/MinigameCore/commit/e1a697bdc25de6450acb6625bf55281bf1c60275#commitcomment-18185939
"Ehm, any reason why you're not just using submodules? Makes things a lot easier. :S"


Move changes to external patch

Fix license file name
Patch -ObjC

@uilianries please fix CI errors

This PR is working now. All errors are related of Clang 8, which we don't have all packages available now. I'll merge it


Error: Could not autoload puppet/parser/functions/ip_address: cannot load such file -- puppetx/ip

When I'm trying to compile my manifests which use your module I receive the following error:
# puppet master --compile puppetmaster.fritz.box --environment master
...
Error: Could not autoload puppet/parser/functions/ip_address: cannot load such file -- puppetx/ip

# puppet agent --test
...
Error: Could not retrieve catalog from remote server: Error 400 on SERVER: Evaluation Error: Error while evaluating a Function Call, Could not autoload puppet/parser/functions/ip_address: no such file to load -- puppetx/ip at /etc/puppetlabs/code/environments/master/modules/foo/templates/bar.epp:25:13 on node puppetmaster.fritz.box
...

Do you have any clues how to solve this?
I run on Debian 8.2 with puppet 4.3.2 from the puppetlabs repos.
Thank you in advance!

This is a known issue in recent versions of Puppet. Here is an example from puppetserver: https://tickets.puppetlabs.com/browse/SERVER-973
I'm not sure what to do about it other than wait to see if Puppet Labs fixes it.


git lfs clone does not pass through git clone options?

Hi,
it looks like as if git lfs clone does not pass through git clone options properly. Or maybe I am doing something wrong? Consider this:
Ξ ~tmp → git lfs version
git-lfs/1.2.0 (GitHub; darwin amd64; go 1.6.1)

Ξ ~tmp → git lfs clone --verbose https://repo.url
git lfs clone [git clone options] <repository> [<directory>]

Clone an LFS enabled Git repository more efficiently by disabling LFS during the
git clone, then performing a 'git lfs pull' directly afterwards.

This is faster than a regular 'git clone' because that will download LFS content
using the smudge filter, which is executed individually per file in the working
copy. This is relatively inefficient compared to the batch mode and parallel
downloads performed by 'git lfs pull'.

Options:

All options supported by 'git clone'

Error: unknown flag: --verbose

Ξ ~tmp → git lfs clone -v https://repo.url
git lfs clone [git clone options] <repository> [<directory>]

Clone an LFS enabled Git repository more efficiently by disabling LFS during the
git clone, then performing a 'git lfs pull' directly afterwards.

This is faster than a regular 'git clone' because that will download LFS content
using the smudge filter, which is executed individually per file in the working
copy. This is relatively inefficient compared to the batch mode and parallel
downloads performed by 'git lfs pull'.

Options:

All options supported by 'git clone'

Error: unknown shorthand flag: 'v' in -v

Thanks,
Lars
/cc @sinbad

Doh, I forgot cobra would sanitise away the flags and I only tested with extra arguments. Sorry - I'll have to figure out whether I can get cobra to match all flags or whether I'll have to painstakingly re-define and passthrough all of them individually.


chore: disable no empty file rule

@Qix-
removes rule for empty files

Trying not to get in the regular habit of encouraging linter ignores. I understood the test.ts as being required by the repository base and they do not have tests yet during initial development.

I will close it, because if test.ts is empty I would just create an ignore in that file. Disabling this has a risk that we could have empty files we are not aware of.


Add ssh management

So far the results have been varied and inconsistent due to the nature of SSH.
The use case I'm trying to cater for may be better off with a shell-based solution post-provision.
I'm still wanting to press forward here though.


Add missing headers to azure-attestation\README.md

Description
Added sections per #4554. This is a work in progress.
@scbedd before I get really invested with the content under the new headers, I just wanted to make sure I'm going in the right direction with this. It's a pretty old issue, and I noticed that a few of the links in the original issue are broken and it appears there has been some refactoring in the repo since. Thanks.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

@microsoft-github-policy-service agree

@scbedd I'm glad I checked before moving ahead with anything else! I addressed your comments in a new commit. Let me know if I missed anything, or if you'd like additional changes.
I'll go find something else to work on. Since I'm new around here, if there are any issues that no one else wants to volunteer for that you may want to suggest, please let me know.

/check-enforcer override
Overriding as the mypy error is most definitively not from this PR.


CSS-formatting: smaller details

Link look: underlined, coloured in #0000b4

Can you use the Medium-cut (currently Regular) of Source Code Pro in CAPITAL letters for the following three subheadlines:
"I THOPUGHT WEB3 WAS PRIVATE?", "SETUP", "METAMASK"
for. Ex. here:

Add the HOPR-favicon as icon, 100% available :) :
128px: https://drive.google.com/file/d/1mCVTlchilIHO_Oyx4-4TBruYItNQaZF3/view?usp=sharing
32px: https://drive.google.com/file/d/1UE1nSPQLJlxuEH90wFRbsBNjSOLHTuOv/view?usp=sharing
Or just grab the one used on hoprnet.org

Rename the button "EXAMPLE 1" into "EXAMPLE 1: METAMASK LINKABILITY"

Inside EXAMPLE 1, rename the Headline "METAMASK LINKABILITY"

And if you still got time, some smaller white space between the "accordeon" and your leaderboard. Preferably the same value as the one between intro and the example-buttons.

Everything implemented except for the spacing.


MozJPEG lossless transform

I don't know if this should be V1 or not…
Some JPEG encoders support lossless transforms, such as MozJPEG. We should enable this.
Currently our encoders take image data, so we need to figure out when it's acceptable to pass through the source file directly.
The options UI may need to be aware of when lossless transforms are possible.

Maybe I'm wrong, but doesn't a 100% quality indicate lossless transform?

It doesn't, it means "run the MozJPEG encoder with a quality level of 100".

(that's how it works in Photoshop etc too)

Yeah, even the setting of 100 will just use a higher resolution quantization table for the DCT transform. That part might be close to lossless, but it’s not guaranteed. Before we get to that stage, tho, the codec will:

Convert RGB to YUV
Will subsample U&V to half the original resolution

which are all lossy steps.

(I think mozjpeg disables chroma subsampling at qualities > 90)

(I think mozjpeg disables chroma subsampling at qualities > 90)

You're right, MozJPEG disables it at qualities >= 90. You can use this image to make sure, it'll get darker if chroma subsampling is enabled.


Dependabot: Ecosystem updates

Summary
Updating ecosystem support for Dependabot version updates, to newer versions of ecosystems.
Intended Outcome
Allow users to stay on the latest version(s) of ecosystems, and reduce error rates in updates.
How will it work?
Dependabot will support the latest version of these ecosystems:

npm v7
bundler v2
pipenv 2020.11.15
piptools 5.5
terraform 0.14

https://github.blog/changelog/2021-02-08-dependabot-npm-v7-support/

https://github.blog/changelog/2021-01-19-dependabot-pip-compile-5-5-0-support/
https://github.blog/changelog/2021-03-31-dependabot-bundler-v2-support/


feat(containers): propagate non-primary container IDs in URLs when navigating

I've noticed that when you're viewing the details of a container, the URL hash starts with /containers/<container ID>, but that it also works if you replace it with /containers/<container name>.
I find that this can be useful, because otherwise if I were recreating a container under the same name, I could refresh the page and see the new container, unlike if I were using the container ID which always changes if a container is re-created.
The problem is that the container name in the URL gets lost when navigating around within the container details, having the container name replaced back with the ID again.
Although currently there's no way to have the name be in the URL other than manually typing it in, I think it would be great if the web app respected the current type of identifier used, and propagated the name if the name was used, but continue to use ID if that was used.
The change of mine should affect the Stats, Logs, Console, Inspect links in the container details page, and also the breadcrumbs within the container details page and its subpages.

Hey @amcsi ! Thanks for the contribution.

I find that this can be useful, because otherwise if I were recreating a container under the same name, I could refresh the page and see the new container, unlike if I were using the container ID which always changes if a container is re-created.

Portainer always redirect you to the containers view after a re-creation process, so I'm not sure if this is actually useful.

The problem is that the container name in the URL gets lost when navigating around within the container details, having the container name replaced back with the ID again.

I'm not considering this as a problem, we've built an UI to make it easy to manage/navigate through your Docker resources/environments, I don't think we want the user to have to specifically alter the URLs of the apps.
I understand your problem but I think that it's not affecting a lot of users (IMO not a lot of people are playing with Portainer URLs to navigate to containers) and I'm thinking that this implementation impacts the consistency/clarity of the codebase (removing the {id: containerID} parameter from query URLs might make it harder to determine where the identifier is coming from).
I was thinking about replacing the container identifier with the container name but with the upcoming implementation of #461, we'll need to rely on identifiers to ensure uniqueness between nodes.

Their shouldn't be any functional difference for normal use of this project, since container IDs would continue to be used in the URLs for anyone not rewriting the URLs manually.
But I don't know Docker Swarm that well, so if you feel like this would somehow negatively impact functionality related to Docker Swarm in the future, I can only trust you, especially since this is your project.


Not able to install

Hi,
am facing below err in cmd
npm ERR! 407 Proxy Authorization Required<EMAIL_ADDRESS>pls help on this to sort out
Thank you

i hope this issue resolved, thanks


Any chance we can get a 1.2.3 release that builds on node 12?

https://github.com/phusion/node-sha3/pull/44#issuecomment-437689120 shows that the native implementation is about 45x faster than the latest pure-JS version. For those of us that don't mind the build complications keeping the 1.2.3 branch updated would be much appreciated!

In #44 I mentioned that the 2.x branch makes a tradeoff between readability and performance. During that research I did come across some techniques (from studying the js-sha3 pure JavaScript implementation) that led to significant performance gains. I ended up not keeping that work because I wasn't happy with the quality of my implementation, but it's something I've been meaning to come back to.
The benchmarks show that near-native speeds are possible, as evidenced by the metrics for the js-sha3 library. Loop unrolling, alone, resulted in some significant performance gains, so before considering reverting back to 1.x, I'd propose a round of performance improvements to 2.x. Maybe we can have our cake (:cake: performance) and eat it, too (:fork_and_knife: no build complications).

#53 mitigates, to some extent, performance concerns. A 75% reduction in computation time translates to this library only being ~11x slower than a native implementation (vs 45x), which is much closer to the ballpark of acceptable tradeoffs for having a pure JavaScript implementation.

@davidmurdoch v2.0.2 has now been released with the performance improvements and formal support for Node.js 12.x. If this addresses your concerns underlying the request to maintain v1.x, feel free to close this issue. There are further opportunities to optimize, but v2.0.2 narrows the gap in the short term. For applications that are highly performance-sensitive, I'd still recommend js-sha3 or a library with bindings to a native C implementation.

Resolved in #55, which addresses node-gyp compilation issues caused by Node.js native API drift+deprecation+removal.

v1.2.3 has been published, which should address Node.js 12.x compatibility issues. Please give it a shot and let me know if you run into any issues.


Hanning/ma 616

@aleffert  please review

localization for push notification
code change to for new configuration

thanks

:+1: though in general I think we should be angling for higher test coverage. Though obviously sometimes that requires more of a refactor than we want to do at that moment.


Can't find where to enter myvaillant login details

Before submitting a new issue

[x] I redownloaded the latest release and restarted Home Assistant
[x] I've checked known issues
[ ] I've enabled debug logs and attached the relevant logs to this ticket

Problem description
I have installed the MyVaillant in HA using HACS. I see the MyVaillant under the HACS. But i can't find anywhere the option to enter my login infomation of vaillant information to make the connection to my heating system. The only thing the HACS MyVaillant shows is the update status the app is up to date. Where should i click to perform step 5 of the installation instructions
Sign in with the email & password you used in the myVAILLANT app (or MiGo app for Saunier Duval)
Kind regards,
Paul
Logs
Put your debug logs here
Be careful not to include tokens, emails, and other personal information

https://www.home-assistant.io/getting-started/integration/


Understand why this fails on Higgs

Does this problem still persist after the recent fixes ?

No, closing it


UI reporting - adjust middle div size to rule height

if only a few rules are displayed for a rulebase, do not use the full height but min (rule height, normal middle div height)
especially helpful if more than one gateway/rulebase are displayed

fixed


entproto: make it possible to configure generated output so that it can work with Buf/Connect

Hi Team,
This PR adds some extra configuration to the generate Hook so that it can output generated code that works with BufBuild's Connect.
Specific changes:

Make all method names unique so that specific keys don't collide or generate the method delete() in JS, and follow the naming conventions specified by the linter.
Don't create the generate.go file since we're using buf generate instead of protoc.
Allow configuring the generation target so we can collocate generated .proto with other hand written files outside of ./ent

Hey @ivanvanderbyl
Thanks for working on this. I prefer to review / discuss each option in its own PR. This allows for more granular discussion on each change as it doesn't bundle them together.
In addition, please add tests 🙏

Hi @rotemtam — okay, I will find some time this week to break it apart.

I love to see this.  I'm working with Connect as well. It requires a bit of boilerplate, but I'm just wrapping ent proto's service implementation inside of Connect methods.  For example, my handlers look like this:
package handlers

import (
	"context"
	"github.com/bufbuild/connect-go"
	"github.com/pgdevelopers/xxxx/ent/generated"
	entpbv1 "github.com/pgdevelopers/xxxx/ent/proto/entpb"
	xxxxv1connect "github.com/xxxx/ent/proto/entpb/entpbconnect"
	"google.golang.org/protobuf/types/known/emptypb"
)

type DeviceStateServer struct{}

var deviceStateSvc *entpbv1.DeviceStateService

// NewDeviceStateServer returns a new DeviceStateServer.
func NewDeviceStateServer(client *generated.Client) xxxxv1connect.DeviceStateServiceHandler {
	deviceStateSvc = entpbv1.NewDeviceStateService(client)
	return &DeviceStateServer{}
}

func (s *DeviceStateServer) Get(ctx context.Context, c *connect.Request[entpbv1.GetDeviceStateRequest]) (*connect.Response[entpbv1.DeviceState], error) {
	resp, err := deviceStateSvc.Get(ctx, c.Msg)
	if err != nil {
		return nil, err
	}
	return connect.NewResponse(resp), nil
}

And I register them like this:
...
func setupHandlers(client *generated.Client) *http.ServeMux {
	mux := http.NewServeMux()

	mux.Handle(xxxxv1connect.NewDeviceStateServiceHandler(
		handlers.NewDeviceStateServer(client),
		connect.WithInterceptors(
			otelconnect.NewInterceptor(),
			interceptors.NewAuthInterceptor(tokenHeader, authToken),
		),
	))

	return mux
}

func setupServer(mux *http.ServeMux) *http.Server {
	srv := &http.Server{
		Addr: ":" + port,
		Handler: h2c.NewHandler(
			mux,
			&http2.Server{},
		),
		ReadHeaderTimeout: time.Second,
		ReadTimeout:       5 * time.Minute,
		WriteTimeout:      5 * time.Minute,
		MaxHeaderBytes:    8 * 1024, // 8KiB
	}
	return srv
}

func main() {
        srv := setupServer(setupHandlers(devicesClient))

	if err := srv.ListenAndServe(); err != nil && !errors.Is(err, http.ErrServerClosed) {
		log.Fatalf("HTTP listen and serve: %v", err)
	}
}

I'd love to talk more about this at some point!


Update tins dependency

Coverage decreased (-0.6%) to 90.823% when pulling 5f65c2720760677dd1ff20660d05b02a5b72ac92 on tagliala:update-tins-dependency into 681f640f07dd750935ca95eafceb67e7f14875f8 on lemurheavy:master.

Coverage decreased (-11.4%) to 80.0% when pulling ef89fac8a07c951f8f999bbb085f4727e8b5149d on tagliala:update-tins-dependency into 681f640f07dd750935ca95eafceb67e7f14875f8 on lemurheavy:master.

Thanks @tagliala


Bug: Background image is not found on T environment

Describe the bug
On local environment, background image is displayed, but on testing environment HTTP 404 is thrown on HTTP GET for this asset.
To Reproduce
Steps to reproduce the behavior:

Go to website,
Open Devtools terminal,

Expected behavior
Background image must be displayed on both local and T environments.
Screenshots
Device (please complete the following information):
Additional context
The issue is probably caused by mismatch of URLs between local and T environments - on local /assets/images is working, but on T there need to be /RockPaperScissors/assets/images used.

Tested, fix works on test env.


Gradle: compose.kotlinCompilerPluginArgs.add("suppressKotlinVersionCompatibilityCheck=1.8.10") only works for JVM targets

Describe the bug
I want to suppress the kotlin version check (to use compose-jb with 1.8.10).
According to the docs, I should use this code:
compose {
    kotlinCompilerPlugin.set("1.4.0")
    kotlinCompilerPluginArgs.add("suppressKotlinVersionCompatibilityCheck=1.8.10")
}

But this does only suppress the checks in a JVM target, but not in any JS or iOS target.
Affected platforms
Select one of the platforms below:

Gradle Plugin

Versions

Kotlin version*: 1.8.10
Compose Multiplatform version*: 1.4.0
OS version(s)* (required for Desktop and iOS issues): n/A
OS architecture (x86 or arm64): n/A

To Reproduce

Open attatched reproducer
Assemble
Uncomment the KotlinCompile task options and enable line 47 again to use the documentated configuration
clean and build again

Expected behavior
A working build suppressing the kotlin version check
ComposeTodo Kopie.zip

Fixed by #2716

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Edit readme - It would help new users, since the order of topics is a bit jumbled

I am not sure what having links to what is right below the links helps.

I was using delayed job for the first time, and was trying out steps as I read the README. But it didn't work out untill I reached the database table section without which nothing would work. So, I just wanted to quote the steps in order they should have been, at the top of README.

The database table is part of rails generate delayed_job:active_record you are not suppose to add that manually. In fact it doesn't really belong in the root DJ README as it is now an implementation details of the AR backend not DJ as a whole.


Android studio freezes after fresh install

Issue description
Freshly installed android studio fails to open. It starts with a window that asks import a config or not, then shows the splash screen, then a grey window which does anything. I have been waiting for 30 minutes at least for it to go away but it is stuck.
Steps to reproduce

Add android-studio to environment.systemPackages
Run nixos-rebuild switch
Run android-studio
Pass the import question and see frozen screen.

Technical details

system: "x86_64-linux"
host os: Linux 4.19.46, NixOS, 19.03.172764.50d5d73e22b (Koi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.2.2
channels(lem0n): "home-manager"
channels(root): "nixos-19.03.172764.50d5d73e22b, home-manager"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Hi lem0n4de. This has been an issue with a few tiling window managers an java programs for a long time.
Are you perhaps using a tiling window manager?
If so the fix is to export _JAVA_AWT_WM_NONREPARENTING=1 to your environment.

Closing this due to inactivity (feel free to reopen with more information).

Closing this due to inactivity (feel free to reopen with more information).


Handle UpdateCron requests

This puts the project's cron configuration in ZooKeeper and simplifies the controller's work for retrieving it.

https://ocd.appscale.com:8080/job/Daily Build/4233/


Fix bg and ground using wrong colors and more

The game was activating sprites loaded before a death after a level reset. Also removed an unecessary call to init_sprites() as it was already being called in unrle_first_screen();. Another fix i did was that the game killed the player at the first attempt when switching levels if the player was supposed to die with noclip enabled.

All of this was a 3 lines change lmao


Use Rails.root to remove relative resolution of path

Without using Rails.root this will based off of the working directory of
the application. The reason this hasn't presented any problems before is
because this is nearly always the value of the Rails root.
However if we utilise the rails server feature of running this as a
daemon the value of the working directory is "/" which causes this path
to be looked for in the wrong place.

I didn't know about the --daemon flag... do you know what it does, as not even looking at the source is clear?
Its reasonably normal to set the working directory of a daemon, so while I think this is a good change, I'm not sure it specifically relates to running it as a daemon.

Ah, just tried it out, and it seemed to fork to run in the background.
:+1: on this change, but you may want to run the process from the rails root anyway.

Yeah I came across this when running the Rails server in the background as part of the Puppet spike. If you're in say /var/govuk/specialist-publisher and run rails s Ruby sees your cwd as /var/govuk/specialist-publisher whereas if you run rails s -d it see's your cwd as /.
When the app is run through unicorn the cwd is correct - hence why this hasn't been a problem.
I'm working on the basis though that if a standard rails command causes this problem, then it could catch someone out.


investigating lb-fs

Being able to mount a user's directory as a file system would be really cool. It would allow someone inside a program like microsoft word to open files inside lockbook. It would also give us another strategy for opening foreign files from within lockbook.
As far as I can tell, there's 2 broad strategies for doing this:

watching a file system and a destination for changes and reconciling them
exposing a virtual filesystem which is a source of truth

To me, 2 seems more appealing. We won't need to duplicate data, which is appealing because we don't want to leave (potentially all) files unencrypted on disk. Especially in the case of failures where the sync is interrupted (laptop battery dying).

On Linux you can use fuse. There are rust libs that help. While it seems like there's macOS support I wonder if it's going to be too much to ask our users to install something like macFUSE.
There are some platform specific implementations that may be better suited for those platforms long term:

Apple
This discussion around some windows alternatives.

If we're able to create a rust library called vfsrs that abstracts across the platform specific implementations I think it would be a really compelling addition to the rust ecosystem.


Folders not recognized as already pinned

The bug
When a folder is already pinned, its context menu does not show that it can be unpinned, rather offers to pin (which though does nothing)
To Reproduce
Pin a folder to the sidebar, then navigate to its parent, select the already pinned folder, open the context menu, "Pin to Sidebar" will be shown always.
Expected behavior
The context menu for already pinned folders should show "Unpin from Sidebar"
Desktop (please complete the following information):
OS Version: Windows 10 Home, Version 20H2, OS Build 19042.746, Windows Feature Experience Pack 120.2212.551.0
App version: <IP_ADDRESS>
Hardware: Processor Intel(R) Core(TM) i7-8565U CPU @ 1.80GHz 1.99 GHz
Installed RAM 12.0 GB (11.9 GB usable)
System type 64-bit operating system, x64-based processor
Pen and touch No pen or touch input is available for this display

This issue was marked as resolved a long time ago -> #1160 Might be a regression

This issue was marked as resolved a long time ago -> #1160 Might be a regression

I can't reproduce it, is this how you're doing the procedure?

Does it seem to happen with an folder?

I can't reproduce it, is this how you're doing the procedure?

Does it seem to happen with an folder?

Exactly like that. My result:

Type of folder doesnt make a difference

Exactly like that. My result:

Type of folder doesnt make a difference

@Hefaistos68 This behavior was recently changed and is now available in the Microsoft Store with v1.1. Please let us know if you can still reproduce this issue after updating to v1.1.

@Hefaistos68 This behavior was recently changed and is now available in the Microsoft Store with v1.1. Please let us know if you can still reproduce this issue after updating to v1.1.

Confirmed to be working correctly in v1.1

Confirmed to be working correctly in v1.1


Editable DatePicker with Mask field inserting characters while editing insted of overwriting them

Bug type
Component
Component name
MudDatePicker, Mask
What happened?
When you try to edit date in editable datepicker with Mask you are inserting characters which mean that you are shifting characters to right and you broke whole date.
Expected behavior
Something like InputType.Date in text field. When you click into day section you will overwrite day section instead of inserting characters, or if you click inside section you start overwriting one by one characters. It does not break whole date value as shifting characters to the right do.
May be that this behaviour is better for all Masks, because if you take IP address and you want to edit first section of IP it means that you want to overwrite first IP number not that you want move first number in IP to the right to secoun IP number of address. I cannot imagine any use of masked field where it is better to inserting characters then overwrite them (cardnumber, email, ........) I can always imagine that you want to write whole new walue or correct some portion of it.
Reproduction link
https://try.mudblazor.com/snippet/GOGckIPoIdHNMeLD
Reproduction steps

Click inside Date field.
Move to start of the date valuie
Start typing and you will see, that values are shifting to right
When the values shifted to the right, does not make sense (say after shifted, the month value is not correct)
Then Date value is being broken, so if i only want to correct day part, I will broke whole date ane need to correct whole number

...
Relevant log output
No response
Version (bug)
6.0.9
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Yes, needs isolation editing date's sections aka native input[type=date]


99_pytorch_doc: create docments

Add docments to 99_pytorch_doc
See the style guide and contribution guidefor more details
Docments progress:

[ ] _mod2page
[ ] pytorch_doc_link

I will take a stab at this.


feat(FTL-8340): implement active project UI changes

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

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Compilation breaks on Python 3.11.0a7 in PyFrame_GetLasti

Using the latest version of pythoncapi_compat.h currently breaks on Python 3.11.0a7.
I have seen this on GHA for Ubuntu and MacOS and locally on MacOS:
src/zope/container/_compat.h:254:14: error: incomplete definition of type 'struct _frame'
    if (frame->f_lasti < 0) {
        ~~~~~^
/Users/runner/hostedtoolcache/Python/3.11.0-alpha.7/x64/include/python3.11/pytypedefs.h:22:16: note: forward declaration of 'struct _frame'
typedef struct _frame PyFrameObject;
               ^
In file included from src/zope/container/_zope_container_contained.c:34:
src/zope/container/_compat.h:257:17: error: incomplete definition of type 'struct _frame'
    return frame->f_lasti * 2;
           ~~~~~^

See https://github.com/zopefoundation/zope.container/runs/6188321544?check_suite_focus=true#step:7:61 where pythoncapi_compat.h is src/zope/container/_compat.h.

If you don't need the latest enhancements of pythoncapi_compat.h, just keep your old version.
Sadly, the new version is not compatible with Python 3.11 alpha versions. PyFrameObject got a lot of changes in alpha versions. Hopefully, new public getter functions were added to solve most issues: https://docs.python.org/dev/c-api/frame.html Problem: implementing these getter functions for early alpha versions would require a lot of code in pythoncapi_compat.h, and I chose to not do that.
Workaround if you need to update pythoncapi_compat.h: manually remove the PyFrame functions which cause you build errors. Does it sound like a reasonable workaround to you? That's the approach that I adopted in my coverage PR: https://github.com/nedbat/coveragepy/pull/1353

(Oops, I clicked on the wrong button by mistake, I didn't want to close the issue.)

Python 3.11 beta3 is now released: can you update to a newer Python 3.11?

I am trying to do so in https://github.com/zopefoundation/zope.container/pull/45, but there are several other problems which arose over night in Python 3.10 and need to be fixed beforehand.

It seems to be unrelated to pythoncapi-compat. I close the issue.
Supporting alpha versions of Python 3.11 is too complicated. To get Python 3.11, use beta1 or newer.


Not compatible with module unification (MU)

Assumes app/styles/app.scss while it should be src/ui/styles/app.scss.

Related; import statement fails to locate the ember-bootstrap/bootstrap file when included in app.scss
Sass Syntax Error (SassCompiler) in src/ui/styles/app.scss:2:1
>
> Error: File to import not found or unreadable: ember-bootstrap/bootstrap.
>         on line 2 of tmp/sass_compiler-input_base_path-TvLIefue.tmp/src/ui/styles/app.scss
> >> @import “ember-bootstrap/bootstrap”;

This applies to the default blueprint, that tries to write to app.scss, as well as the styles broccoli funnel, as shown above 👆