Suggestion: Set Nickname Command

On my server there are colour roles that give everyone the ability to express themselves with the colour of their name, including mods. This has unfortunately hidden some privileges my mods should have, such as changing other member's nicknames if found inappropriate. Since listing previous nicknames is already an option, what about setting it? It would still require the "Manage Nickname" permission of course.

I agree the nicknames feature would be very useful to get around that for the mean while Harucho I would just make a role that has nicking perms and revoke it if a user misuses their username.

I'll work on trying to make it happen. Will post updates here.

Feel free to PR but this is something we're actively against. Making this change is fairly simple, but it's not been done because it makes abusing YAG easier. Discord makes it clear to only change nicknames for moderation purposes and not to harass people, which is something that people would immediately do if they could use a bot to mass change nicknames via yag.

I just remembered that this bot is in go so It will probably be a while.

Feel free to PR but this is something we're actively against. Making this change is fairly simple, but it's not been done because it makes abusing YAG easier. Discord makes it clear to only change nicknames for moderation purposes and not to harass people, which is something people would immediately do if they could use a bot to mass change nicknames via yag.

It would be configurable per server. By default would be Mods/Admin roles only, just like the moderation stuff.

Abuse does not refer to unrestricted usage of the command by those lacking permissions; rather, it refers to abuse of the bot's functionality itself by server managers.

A probably not so great solution is to put it behind the premium thing. Not sure anyone would pay for it just to abuse the nicknames.

Even if they don't just subscribe for nickname changing, the fact that you can do it with premium still doesn't negate the abuse.


Display name using v-pre in auth scaffolding

Issue
The default auth scaffolding displays the user's name HTML escaped, but if the user has used curly braces the default Vue setup will try to interpolate it. This could lead to the user executing javascript or breaking the page rendering entirely. I don't believe this is a security vulnerability since it will only happen to the currently authenticated user, but a good example should be set for users.
Steps to Reproduce

Create a new Laravel project
Run php artisan auth:make
Register a user named {{ breaksPage }} or {{ alert('Hello World') }}

Solution
Use the v-pre Vue directive on the element displaying the name.

I don't really understand what you're talking about. Why would we put a v-pre directive on a page that doesn't even use Vue at all?

Out of the box Laravel includes a compiled JS file that mounts Vue to #app, so even if it's not intended, the page does use Vue.
This video follows those three steps exactly: https://streamable.com/k2uc5


KubeCon Shanghai 2018 docs sprints - temporary approvers

This PR adds approvers to sig-docs-zh-owners in order to help approve PRs for the KubeCon Shanghai 2018 docs sprint.
A follow-up PR to remove these owners after the sprint concludes:
/assign @hanjiayao

/approve


Add editorconfig file

The most important setting is to trim trailing whitespace, and use spaces not
tabs on the file system.
The rest can be argued. Indent size is taken from a (tiny) sample of the
existing ghcide code base.

I am a very typical windows user - I haven't changed many defaults. I think crlf is a choice you make in the git installer. The "mangle all files on checkout" option is the only one I don't follow defaults.


feat: add support for VPCEndpointServiceConfiguration CRD

Issue #, if available: https://github.com/aws-controllers-k8s/community/issues/1711
Description of changes: Essentially just recreates https://github.com/aws-controllers-k8s/ec2-controller/pull/134 but with the benefit of hindsight; this PR does not touch RouteTable CRDs, as the former did. It would seem that the problems resulting from this issue may have been solved, thus rendering us ready to rock.
@jaypipes Would you mind telling me if I've made a poor assumption somewhere? I see in the upstream issue about nested lists that @RedbackThomson was originally slated to work on it. I didn't see any linked PRs to that issue, however, when rebuilding https://github.com/aws-controllers-k8s/ec2-controller/pull/134 from scratch, I did not need to modify the RouteTable CRD to work around this issue, so perhaps it has been fixed?
Either way, we would love to get this upstreamed. I'd separately love it if someone could comment on the general safety of using a build based off of this PR in production just to gain access to VPCEndpointServiceConfiguration ASAP.
Thank you all, and I look forward to receiving feedback on this PR!
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Full context

https://github.com/aws-controllers-k8s/ec2-controller/pull/134

The original PR that added support for VPC endpoint services to ec2-controller. It seemed to be blocked pending a code-gen issue.

https://github.com/aws-controllers-k8s/community/issues/1718

The issue allegedly blocking the aforementioned PR from being merged. AFAICT, it had something to do with nested list structures.

It is by no means my intention to remove any credit from @ib-ak for this work! GitHub's WYSIWYG editor won't seem to let me tag them, but I hope it is working :frowning_face:. As soon as I type the "dash" in that username, the automatic username suggestions disappear.

Thank you @a-hilaly for engaging with this PR and providing reviews. I regret to have to tell you that we ended up straying away from ACK for managing our VPCEndpointServices, and lack the bandwidth to give this PR the attention that it requires for landing.
I do hope that someone takes up the mantle of aws-controllers-k8s/community#1711 (which, as a side note, has been closed as stale--perhaps that issue should be /lifecycle frozen?). It is a much needed feature!


Execute oni directly from terminal

Hi, I just installed oni with THIS method,
but I couldn't find how to execute oni directly.
It seems kind of plugin with neovim, but I still have same design of vim.
so question. how can I run a oni directly from the terminal
I'm using OS X

Hi @Kcrong ,
Thanks for trying out Oni! Yes, we're actually just working on finalizing this. There has been a PR in #646 that adds a helper to 'Add to Path' on OSX.  #636 is also tracking the remaining work, basically a shell script we can use to start Oni.
Probably the simplest way to get Oni in your path is to build from source, as the npm link command will expose a global oni shell command. Hope that helps!
It will be really nice to have this streamlined on OSX with the fix for #646 and the fix for #636 in.
I'll close this as a duplicate of #636, but let me know if you have any further questions.

@extr0py Thanks you for response!
I have one more question.
If I use command-palette,
it just show me Loading ... Like:

I entered command palette with Ctrl + p.

Sure thing! That looks like a duplicate of #662 - we don't do a very good job at the moment of handling the case where you open the fuzzy finder from home or a root dir.
If you follow the workaround mentioned there, and select a folder, like Ctrl+Shift+P -> Open Folder. You can also just use the vim command like :cd /my/folder. At that point, once a folder is opened, the fuzzy finder should work better. Hope that helps!

Thanks! it works very well :)
Also, it is really cool

@extr0py FYI, you can close this issue :)

Sweet, thanks @Kcrong , glad that worked for now 👍  Thanks for the kind words!


[#2584] Add AsyncContext

#2584

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@390a388). Click here to learn what that means.
The diff coverage is 29.32%.

@@            Coverage Diff            @@
##             master    #3085   +/-   ##
=========================================
  Coverage          ?   38.94%
=========================================
  Files             ?     1996
  Lines             ?    66154
  Branches          ?     9244
=========================================
  Hits              ?    25764
  Misses            ?    37771
  Partials          ?     2619

Impacted Files
Coverage Δ

...interceptor/SpanAsyncEventEndPointInterceptor.java
0% <ø> (ø)

...pinpoint/profiler/context/id/DefaultTraceRoot.java
76.78% <0%> (ø)

...oint/profiler/context/scope/DefaultTraceScope.java
0% <0%> (ø)

...ercorp/pinpoint/profiler/context/DisableTrace.java
26.08% <0%> (ø)

...avercorp/pinpoint/profiler/context/AsyncTrace.java
0% <0%> (ø)

...pinpoint/profiler/context/DefaultAsyncContext.java
0% <0%> (ø)

.../context/provider/AsyncContextFactoryProvider.java
100% <100%> (ø)

...rofiler/context/provider/TraceFactoryProvider.java
87.87% <100%> (ø)

...p/pinpoint/profiler/context/id/DefaultTraceId.java
89.74% <100%> (ø)

...vercorp/pinpoint/test/MockTraceContextFactory.java
66.15% <100%> (ø)

... and 7 more

Continue to review full report at Codecov.

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


The recent medium article only displays alternative text

Hi,
I hope you are well.
The medium article plugin only displays alternative text, even after waiting for more than 3 hours. Though, the plugin is really nice. I think something is broken. It is broken on your page also.

Hi @chayansraj
Yes , the underlying medium API is not providing the thumbnail URL anymore causing the serverless function to crash
<EMAIL_ADDRESS>This is the API that is used here to get the RSS data from medium and parse to JSON, in the JSON response the thumbnail var is just an empty string which is the root cause of the issue.

For the time being I have deployed a fix by taking the image from the description instead of the thumbnail. You can use it if u like :


HidAPI Hangs When Controller is Connected Via Wire

Sometimes this line can hang. Once it does happen, it happens consistently.
// controller.rs
let res = device.read_timeout(&mut buf[..], 3000).unwrap();

High priority as it prevents the app from functioning as intended.

This happens when the controller is connected wired rather than via bluetooth.

Currently wontfix as I don't see a workaround


StickyAction wrapper can repeat the old action for more than 1 step

The StickyAction wrapper now takes an optional argument to allow the old action to be repeated for more than 1 steps (default is 1). The original behavior is unchanged.
Description
No fix, this is an extra feature. It increases the difficulty of sticky actions and the "non-Markovianity" of the environment (the more steps the action is repeated, the more in the past the agent should look to predict the next state). It can be useful for RL of non-Markov decision processes.
Type of change
Please delete options that are not relevant.

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

Checklist:

[x] I have run the pre-commit checks with pre-commit run --all-files (see CONTRIBUTING.md instructions to set it up)
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@sparisi If I understand correctly, this will limit the number of repeat actions that can be taken.

Ahh I understand better now thanks for the description
Originally, the implement was just a single sticky action step where, given a probability, there was a chance of taking the last action.
In Atari, this is paired with FrameSkip, i.e., the agent takes X steps with a 0.25% of taking the last action taken.
ALE implements two options for frameskip of a deterministic, v5 with 4 frames and v0 randomly between 2 and 5.
This PR is added frameskip for a random value up to X which is equivalent to the v0 style randomly between 1 and X.
Am I understanding correctly?
If I am, I would shift it to the ALE style implement with either a deterministic number of frames, int, or a stochastic number of frames with tuple[int, int]

@pseudo-rnd-thoughts
I see.
I think the difference with FrameSkip is that the agent, when actions are repeated, does still try to execute an action. With FrameSkip, instead, the agent is not aware of the repeats.
The main reason I made this wrapper is to study how non-Markovianity affects classic RL.
"This PR is added frameskip for a random value up to X"
Actually, the number of steps the action is repeated is deterministic. What's random is when the action starts being repeated (the randomness in StickyAction).
Now I have added the possibility to have stochastic repeats within a range. When the agent starts a series of repeats, the duration is randomly determined within the range passed as argument to the wrapper.

@sparisi I came back the PR after a couple of days and hopefully better understand what you are talking about with the difference with this vs frameskip.
I've updated the code to change the variable names and added a test that checks the expected actions match the executed actions.
Could you check the code, then I would be happy to merge

@pseudo-rnd-thoughts
Checked and tried on an environment, all good. Thanks!


extend designator client with add_seen method

Depends on https://github.com/zooniverse/designator/pull/86
This will allow a user's seen subjects for a workflow to sync between the panoptes and designator APIs. This works by extending the designator client add_seen API to match the public designator API added in https://github.com/zooniverse/designator/pull/86
This will be called each time a classification is received by worker https://github.com/zooniverse/Panoptes/blob/f039669292a9709ca83a9e7e382b6ce3c8da2c6b/app/workers/notify_subject_selector_of_seen_worker.rb which in turn calls the designator selector https://github.com/zooniverse/Panoptes/blob/f039669292a9709ca83a9e7e382b6ce3c8da2c6b/lib/subjects/designator_selector.rb#L15 to call this client method to call the remote API.
Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

This looks like the first designator API call that's going to be happening every single classification, so before this gets deployed maybe we should figure out how best to measure any performance changes to our total per-classification time

As the designator call happens in a background worker I don't think this will impact on the panoptes API at all. It may have an impact on designator but as it's built for high concurrency and the in memory ETS cache (https://github.com/sasa1977/con_cache) updates are not blocking i can't see this being a problem. That said it would be nice to have some metrics we can compare.

Shouldn't be much at all but it seems like this was included and then short circuited at some point (the no-op notify worker) so good to keep an eye on.

FWIW The client adapter public API was originally modeled on the cellect client that does rely on signalling seens subject ids and hence this method was a no-op for designator as it didn't care about what you'd seen.


Breaking Change in v2.68.0 w/ Browserify

Looks like v2.68.0 introduced a breaking change by requiring a separate aws4 dependency.  Apparently with browserify, that dependency isn't optional, causing build failures.

Submitted #2041 to fix this.

@rmg, thanks for the quick action!


How to pass url in WebsiteAgent to Notifier

Hello
I am using a Website agent to monitor a website for when the price updates.
{
  "expected_update_period_in_days": "2",
  "url": "https://bleep.com/release/167916-drexciya-journey-of-the-deep-sea-dweller-i-iv-bundle",
  "type": "html",
  "mode": "all",
  "extract": {
    "price": {
      "xpath": "//*[contains(concat( \" \", @class, \" \" ), concat( \" \", \"price-wrapper\", \" \" ))]",
      "value": "normalize-space(.)"
    }
  }
}

This notifies an EmailAgent of the price. How can I get the Website Agent to also pass down the URL which I've already defined in the Agent json? Ultimately what I want is an event that goes to the EmailAgent with this body:
{
  "price": "EXTRACTED PRICE",
  "url": "https://bleep.com/release/167916-drexciya-journey-of-the-deep-sea-dweller-i-iv-bundle"
}

Thanks!

You can use the template option for this.
Either hard code it:
{
  "expected_update_period_in_days": "2",
  "url": "https://bleep.com/release/167916-drexciya-journey-of-the-deep-sea-dweller-i-iv-bundle",
  "type": "html",
  "mode": "all",
  "extract": {
    "price": {
      "xpath": "//*[contains(concat( \" \", @class, \" \" ), concat( \" \", \"price-wrapper\", \" \" ))]",
      "value": "normalize-space(.)"
    }
  },
  "template": {
    "url": "https://bleep.com/release/167916-drexciya-journey-of-the-deep-sea-dweller-i-iv-bundle"
  }
}

Or if you want it more dynamic (e.g. url comes from source agent) you can use the liquid templating variable {{ _url_ }}.

Amazing - thanks very much for the help!


Update mammon to latest

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] sha256 changed but version stayed the same (what is this?).
I’m providing public confirmation below.

Thank you for your contribution!

Apparently it's giving an error when attempting to unpack the .zip! 😱
==> Installing Cask mammon Error: Uh oh, could not figure out how to unpack '~/Library/Caches/Homebrew/Cask/mammon--latest.zip' Error: Install incomplete.
Any suggestion for what should be done? All I attempted to do was fix the broken download link.

Apparently a ?dl=1 is missing at the end of the url... 😅
Should I create a new pull request with the fix?

https://github.com/caskroom/homebrew-cask/pull/38648


test pull

Hi @Ryan3Lima ! I hope the Quarto Website Tutorial has been helpful to you. If you would like to make your own version of it, rather than making a pull request that would change the source, you have two options. 1) "fork" the source repository or 2) use the source repository as a template to create a new repo under your own account. That way you can make changes and even turn it into your own content.
Screenshot from https://github.com/Openscapes/quarto-website-tutorial

Hi Stefanie,
I am rather new to github, i did not mean to pull to the man fork, I was trying to get my gh-pages branch to match my main branch and I am still a bit confused. Not sure if folks at open scapes are aware but the current repo does not really match the tutorial, for example it has a .mdq instead of a .md and there are some differences that I was confused about following the tutorial.
I am  part of at major three university recharge project and I am trying to find tutorials like that to get everyone on the same page but even I was pretty confused trying to go through that. Apologies for the pull request, I didn see an option to cancel it when I realized what I had done.
Ryan Lima Ph.D.
Post-Doctoral Scholar
Northern Arizona University
@.***

From: Stefanie Butland @.>
Sent: Friday, August 23, 2024 8:56 PM
To: Openscapes/quarto-website-tutorial @.>
Cc: Ryan Lima @.>; Mention @.>
Subject: Re: [Openscapes/quarto-website-tutorial] test pull (PR #36)
Hi @Ryan3Limahttps://github.com/Ryan3Lima ! I hope the Quarto Website Tutorial has been helpful to you. If you would like to make your own version of it, rather than making a pull request that would change the source, you have two options. 1) "fork" the source repository or 2) use the source repository as a template to create a new repo under your own account. That way you can make changes and even turn it into your own content.
Screenshot from https://github.com/Openscapes/quarto-website-tutorial
Screenshot.2024-08-23.at.8.55.09.PM.png (view on web)https://github.com/user-attachments/assets/0454e803-9f16-4cdc-bc70-9cc43d290ddb
—
Reply to this email directly, view it on GitHubhttps://github.com/Openscapes/quarto-website-tutorial/pull/36#issuecomment-2308039207, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BBWRYR5Q3OOHZEA7PKD3X3DZTAAAJAVCNFSM6AAAAABNBAYZJGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBYGAZTSMRQG4.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks for practicing with this tutorial! We'll close this PR now :)


scala 2.12.9 and 2.12.10

Filling in the missing versions.

Pretty good. Thanks!

@all-contributors please add @pthariensflame  for code


Guestion: how to preprocess (modify) submitted code before it gets executed by C# kernel?

I'm working with Jupyter notebooks and my code is not pure C#. So, before it gets executed I need to modify it to make it valid C# code. Is it possible to do using middleware.
Or maybe there is a way of introducing my own kernel that would subclass the CSharpKernel? Is it possible at all?
Please, help.

Hi, can you share an example of "non" correct c# you want  to preprocess?


perconaserver55: fix libexec install

The percona-server55 formula is failing due to a missing libexec directory. This fixes it.

Merged in https://github.com/Homebrew/homebrew-versions/commit/ce4559f40f6b15fe4278d0668f75df124fa4af44. Thank you for your contribution to Homebrew @anfleene! :smiley_cat:


photon: change base image to debian:bullseye-slim

In order to use common base image. Note that tzdata is already installed in the new base image.

upgrade openjdk from 8 to 11
Add build args PHOTON_VERSION and PHOTON_HASH
Strip version suffix from jar filename

Ideally the image would be reworked/split into two to decouple the data processing from the build step and not include the data in the actual image as brought up in #4 but figured this can be done first regardless

Ideally the image would be reworked/split into two to decouple the data processing from the build step and not include the data in the actual image as brought up in https://github.com/ellenhp/headway/discussions/4 but figured this can be done first regardless

A few friends have convinced me that this is the way to go, so 100% agreed. What do you think of the graphhopper build config I wrote this morning? It's more complex for sure, but it does keep the data out of the image. Also just want to make sure you see #9 so you can avoid putting too effort into prettying it up, but I'll probably use a similar workflow for OTP. I'm considering cutting out one of the intermediate copies though?
It just seems overly complicated right now:
osm.pbf -> build volume -> preprocess step -> copy data out of build volume onto disk -> copy data into the runtime volume
But I like having the on-disk artifact because it makes it really easy to have the Makefile work incrementally.

What do you think of the graphhopper build config I wrote this morning?

Overall it's a step in the right direction I think!

But I like having the on-disk artifact because it makes it really easy to have the Makefile work incrementally.

You mean on the host filesystem outside of volumes? Agreed it makes things easier to work with.
What do you think about mounting the docker volumes directly to paths on the local filesystem, rather than using native docker volumes?
Compared to now I imagine something like this:

Add new env var HEADWAY_BASE_DIR (picked up both in docker-compose.yaml and Makefile) with some sane default
Instead of e.g. docker run --rm  -v headway_graphhopper_build:/headway_graphhopper_build: docker run --rm  -v ${HEADWAY_BASE_DIR}/graphs:/headway_graphhopper_build
There should be no need for cp/mv of data artifacts during the make stages; containers should be able to mount the same volumes and pick up directly from each other (so if any preprocessing is needed, that should be done inside containers via commands or arguments)

What do you think about mounting the docker volumes directly to paths on the local filesystem, rather than using native docker volumes?

I love this idea in theory (it's actually what I started with!) but I think it would require us to parameterize the docker images with the users UID and GID to prevent files being created in your home directory that you don't own. Is there a clean way to do that?

Also this PR looks good to me! Just tested it. We can continue this discussion here or in the discussions tab?

Yes, and that's something that is desirable either way (operators may want to use separate per-container uids with minimal permissions for security reasons, for example)
It may be that the only thing necessary is to tell docker to use the appropriate user (docker run --user 1234:1234, compose user(https://docs.docker.com/compose/compose-file/#user) (there's also group_add but that seems a bit overkill here)
In some cases one may need to add the user inside the container and maybe set some permissions in an init script  - but from a quick look I don't think it should be the case here.


What is the impact of the Ethereum 2.0 upgrade on gas fees?

It aims to reduce gas fees by improving scalability, though the full reduction in fees will depend on future updates.


Issuer is not set as JWT for service providers when the config is enabled

Description
Issuer is not set as JWT for service providers when the config is enabled.  If the token persistence configs are enabled with this feature, users cannot login to portals if a tenant user tries to login to the devportal of super tenant.

Publisher portal

Steps to Reproduce
Prerequisites

Add below config to deployment.toml file of the fresh pack

[apim.oauth_config]
enable_jwt_for_portals = true

Start APIM
Login to Management console
Navigate to -> Service providers list -> Edit the devportal sp -> Inbound Authentication Configuration -> OAuth/OpenId Connect Configuration ->Edit
Check the token issuer

Affected Component
APIM
Version
4.3.0
Environment Details (with versions)
No response
Relevant Log Output
ERROR - [introspect] Servlet.service() for servlet [introspect] in context with path [/devportal] threw exception [java.lang.NullPointerException] with root cause
java.lang.NullPointerException: null

Related Issues
No response
Suggested Labels
No response

@vishmi49 did you add all the configurations mentioned in https://apim.docs.wso2.com/en/4.3.0/install-and-setup/setup/security/securing-api-m-web-portals/#enable-jwt-for-web-portals? From the reproducing steps i could see only one config

Hi All,
The reproducing steps are as follows.

Start a fresh default pack without any configuration changes. In this case - APIM 4.3.0-beta.
Access the Management Console (/carbon) and create a new tenant, say wso2.com.
Access the developer portal and click on "SIGN-IN". (Logging in is not required) This will create a Service Provider for the Devportal App.
Access the publisher portal and similarly click on "SIGN-IN". This will create a Service Provider for the Publisher App.
If you access both the SP apps corresponding to the 2 portals above, you will see that no Token Issuer is selected. This means by default the token issuer type is "Default".
Now shut down the server and configure it for disabling token persistence by adding the below configuration to the deployment.toml.

[oauth]
add_tenant_domain_to_access_token = true

[oauth.token_persistence]
enable=false

[oauth.revoked_token_headers_in_response]
enable=false

[[oauth.extensions.token_types]]
name = "JWT"
issuer = "org.wso2.is.key.manager.tokenpersistence.issuer.ExtendedJWTTokenIssuer"

[transport.https.properties]
maxHttpHeaderSize = "12288"

[apim.oauth_config]
enable_jwt_for_portals = true

Restart the server.
Access the Devportal and click on carbon.super from the Tenant selection.
Login to the developer portal using the wso2.com tenant's credentials.
You will observer the errors mentioned in the description above.

However this is the expected behaviour because the SPs were created before enabling the token persistence configuration. To fix it, we need to follow the steps in the documentation [1] for an existing deployment.
As this is not a bug, I will close the issue.
[1] https://apim.docs.wso2.com/en/4.3.0/install-and-setup/setup/security/securing-api-m-web-portals/#enable-jwt-for-web-portals
Thanks,
Tharika.


Revert "fpga: add support for Arria10 DCP 1.0"

Reverting as this was not a good idea for two reasons:

Supporting old DCP version is additional mainentance effort
it breaks webhook deployment with this kind of errors:
Invalid value: "arria10_dcp1.0": a DNS-1123 subdomain must consist of
lower case alphanumeric characters, '-' or '.', and must start and end
with an alphanumeric character (e.g. 'example.com', regex used for
validation is
'a-z0-9?(.a-z0-9?)*'

Closing as compress.aocx still doesn't work with dcp1.1. I'll fix naming instead.

Codecov Report

Merging #131 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #131   +/-   ##
======================================
  Coverage    79.7%   79.7%
======================================
  Files          10      10
  Lines        1212    1212
======================================
  Hits          966     966
  Misses        223     223
  Partials       23      23

Continue to review full report at Codecov.

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


docs improvements

What does this PR do?

[x] general styling updates to existing components like cards, icons, and sidebar navigation
[x] add custom icons for sidebar and cards
[x] add custom svg and color styling for bitcoin section
[x] add footer to homepage
[x] update main footer section
[x] add a new RollButton component to scroll to top of page
[x] create a custom Callout component to reflect designs
[x] update styling for api reference and code snippets
[x] update styling for text tabs and tag tabs components
[x] update directory styling component
[x] update toggle content components
[x] add a fix for dynamic metadata (for analytics)

Chores included

[x] https://github.com/hirosystems/docs/issues/694
[x] https://github.com/hirosystems/docs/issues/693
[x] https://github.com/hirosystems/docs/issues/692
[x] https://github.com/hirosystems/docs/issues/689
[x] https://github.com/hirosystems/docs/issues/687

Thanks Ryan - I just made my review on Vercel and left some comments. Feel free to push this update and work on those refinements whenever it suits you!


Running on GPU.

Hi, guys,
I found out I can not use GPU when running merlin example.
And I seems find out why :
First, there always lock gpu when running demo but only when successfully running down, there will be free lock. So I add free lock in each step.
Second, seems that github code miss a file named "run_on_me_or_pid_quit", I download it from
"http://homepages.inf.ed.ac.uk/imurray2/code/gpu_monitoring/gpu_lock/run_on_me_or_pid_quit"
And now I can use gpu normally.

I had a similar GPU lock issue after killing processes during the demo. I had to free the GPUs (even though no processes were running) with this script:
https://github.com/CSTR-Edinburgh/merlin/blob/master/src/gpu_lock.py
python gpu_lock.py --free <gpu_id>


Image doesn't show when the picture is too big

Steps to Reproduce

Image widget to display a local picture file doesn't show , this picture is about 13m
the device info:  model is Che1-CL20, android version 4.4.4, RAM 2.0G
flutter version: Flutter (Channel beta, v1.7.8+hotfix.3)

the pic file load success but doesn't draw on the screen,  maybe the pic is too large?  flutter limit the pic size ?

Target Platform:
Target OS version/browser:
Devices:
Logs
no any error log

@qylshy I think it's a duplication of https://github.com/flutter/flutter/issues/41749. If you disagree, please comment and I will re-open the issue.


Update minor bug in mimetype

Just fixed a minor bug in mymetipe.

Thank you very much for your help

This change will be available only in the master branch, once other problems and improvements are solved, it will be included in the next stable version of Merly. Once again, thank you for your participation in this project.

No problem :).


PatreonDownloader.App.exe

i don't understan where i can download the PatreonDownloader.App.exe

https://github.com/AlexCSDev/PatreonDownloader/releases/tag/release_24


Fix: add extra_hosts to docker-compose.yaml to allow connection to ollama

This pull request enhances the Docker Compose configuration by adding extra_hosts entries to enable connections to ollama. This change is necessary to facilitate seamless communication between containers and the host system, particularly for the perplexica-backend and perplexica-frontend services.
Fixes #117

Hi, thanks for the PR! I'll look after the changes and review them before merging, please give me some time.

@ItzCrazyKns This PR resolves my connection issues with Ollama on Linux such as in #117

Thanks!


User-defined instructions not shown on some tasks

At least one task type ("Take a photo") does not show instructions, others may not as well. @JSunde could you PTAL?

Some other tasks still don't show user-defined instructions:

[ ] Drop a pin
[ ] Draw an area
[ ] Capture location

@JSunde @shobhitagarwal1612 PTAL?


added cognito-idp initiate_auth and PASSWORD_VERIFIER challenge to respond_to_auth_challenge

Coverage decreased (-0.1%) to 94.079% when pulling 7e6fcd2c1d666b90cefaa8ac293d732850010917 on xsphrx:master into 47a227921d26845f57766cb73e610ddbc30737d5 on spulec:master.

This is now part of moto >= 1.3.15.dev1037


Update regex.ts

Updated attrSplitRegex to make a coverage if non-object type props with whitespace passed

It has been decided that people who belong to the company should request PR by consolidating them into a company account, preferably not a personal account, and close. (SpookyJelly is my personal ID)


MSID: 86628 Version: 2 DOI: 2<IP_ADDRESS>8498

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[ ] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=86628)

or (only one should be ticked. remove other from description.)

[ ] Confirmed with Ted at bioRxiv that MECA archive is available

Send the following email to Ted and wait for his reply.
Hi Ted,

Please can you prepare the preprint MECA for 10.1101/2<IP_ADDRESS>8498

Thanks

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[x] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/86628/v2)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews
Who can help: Editorial team

[ ] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>8498)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/86628)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/86628

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/86628

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/86628/v2/86628-v2.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/86628)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Wrong version pulled in - the specific MECA package needs specifying in manuscripts.txt, which is a step that I missed on upload.

Reimport in https://github.com/elifesciences/enhanced-preprints-data/pull/88

Reimport in https://github.com/elifesciences/enhanced-preprints-data/pull/88

@elifesciences/epp-tech-team I can see the revised reviews are available in the DocMap (step _:b4).
Is EPP configured now to pull these in? Or do we need the reviews (and anything else?) to be hardcoded (as with 85111) in order to publish this revised RP?

@elifesciences/epp-tech-team I can see the revised reviews are available in the DocMap (step _:b4).
Is EPP configured now to pull these in? Or do we need the reviews (and anything else?) to be hardcoded (as with 85111) in order to publish this revised RP?

Hi Fred - @nlisgo has been working to enable this in the backend this week. I can see from https://prod--epp.elifesciences.org/api/reviewed-preprints/86628/v2/reviews (vs https://prod--epp.elifesciences.org/api/reviewed-preprints/86628/v1/reviews) that it is working for this manuscript, and the work to enable this for client is merged too. So in theory it is all in place, but because of the workaround we had in place earlier I also know @nlisgo had some more work to undo changes put in place for the earlier enablement.
So the good news is we don't need to hardcode reviews, but @nlisgo will be better able to answer if it's ready to publish a new one via this route.

So the good news is we don't need to hardcode reviews, but @nlisgo will be better able to answer if it's ready to publish a new one via this route.

We can check if this is working using the server api endpoint: https://prod--epp.elifesciences.org/api/reviewed-preprints/86628/v2/reviews
Seems to be working.


Plugin likes count

commit 5a426672ff8b0ae00f89d74da613cc4c3607e5f6
Merge: 07ca669 8ab8734
Date:   Wed Dec 16 10:43:18 2015 -0800

    Merge branch 'fix/public-92'


Error: invalid number of endpoint URLs provided

I was originally trying to get my own Magento installation up and running but hit this same error:
Error: invalid number of endpoint URLs provided
So I tried a vanilla start (windows 11 Empty Magento 2 Project with bootstrap command) and hit this same error after running the reward bootstrap --with-sampledata command.
I'm assuming I've missed something pretty obvious, I'm migrating from WinNMP, which is very hand holding compared to this haha. Willing to learn, just stuck, any ideas?

PS F:\Dev Work\Reward\test> reward env up your-awesome-m2-project_redis_1 is up-to-date your-awesome-m2-project_db_1 is up-to-date your-awesome-m2-project_rabbitmq_1 is up-to-date your-awesome-m2-project_elasticsearch_1 is up-to-date your-awesome-m2-project_php-fpm_1 is up-to-date your-awesome-m2-project_php-debug_1 is up-to-date your-awesome-m2-project_nginx_1 is up-to-date your-awesome-m2-project_varnish_1 is up-to-date INFO[2022-03-14T19:31:39Z] Syncing environment with mutagen... Error: invalid number of endpoint URLs provided Error: exit status 1
This is what I see when running env up following the manual initialization on the docs, if that is any more help

Hi @Chris-Kiddier !
I was able to reproduce your issue, it's related to the whitespace character in your working directory path (F:\Dev Work\). Under the hood Reward calls Mutagen to set up a sync session. Running Reward in debug mode it shows it's trying to run mutagen with these arguments / flags:
DEBUG[2022-03-15T11:21:10+01:00]common.go:842 github.com/rewardenv/reward/internal/core.RunOsCommand() Running command: cmd /c mutagen sync create -c C:\Users\mixe3y\Workspace\test 2\m22\.reward\mutagen.yml --label reward-sync=m22  C:\Users\mixe3y\Workspace\test 2\m22/ docker://06d198f8ba2b7cf2ab61f25c7cfaf01bb8fa4406fc616896653ac87a27664e04/var/www/html
Error: invalid number of endpoint URLs provided

According to this, mutagen thinks that we are calling it with three endpoint URLs:

C:\Users\mixe3y\Workspace\test
2\m22/
docker://06d198f8ba2b7cf2ab61f25c7cfaf01bb8fa4406fc616896653ac87a27664e04/var/www/html

I assume I'll have to escape that whitespace in the path. I think I'll be able to fix this tomorrow by creating a new release.

Hi @janosmiko !
Thanks for the quick reply and quick diagnosis. Appreciate your hard work and look forward to playing around with this awesome bit of kit.
Cheers

Something to add on to this, whilst waiting for the update I thought I'd try a test folder structure with no spaces, it still failed.
However, after some searching on the Mutagen issues GH I found that for Windows I needed the mutagen.exe (as described in documents) and also the mutagen-agents.tar.gz in the same folder.
This can be found in the mutagen_windows_arm64_v0.13.1.tar.gz file (on the Mutagen downloads section). After 7zipping I was able to extract the agents.tar.gz and place it in the same directory as the mutagen.exe path file. Now I have a working environment.
This might be common knowledge to many, but it's good to know if any one else gets stuck on steps after the fix. Every day is a school day, after all.
Chris

Yes that's a requirement for mutagen to work. You can find an FAQ/Frequent Errors section where you can find answers to some error messages:
https://rewardenv.readthedocs.io/en/latest/faq.html#frequent-errors

Hi @Chris-Kiddier !
I released a new version which should fix this issue.
Please test it and let me know if it works as expected.

Hi @janosmiko
I used the reward self-update, restarted and tried a fresh install (based on the Empty Mage tut) on the same drive. I got to bootstrapping again but this time I hit a different error.
INFO[2022-03-16T21:24:42Z] Syncing environment with mutagen... Error: unable to load configuration file: unable to load file: open "F:\Dev Work\Reward\your-awesome-m2-project\.reward\mutagen.yml": The filename, directory name, or volume label syntax is incorrect. Error: exit status 1
Knowing I can change the folder naming was very useful and have been experimenting with it on a different folder path. It will make a great dev environment!

Interestingly, I rolled back to the previous version as the latest update stopped my functioning reward setup from working.
This was just under  F:\Dev\your-awesome-m2-project - no spaces this time. I got the same message above with the latest update

Super weirdo working mechanism difference between windows cmd and mac/linux shell..
It should be fixed in https://github.com/rewardenv/reward/commit/d485ba093ee049f44bee4a11208581f135dec7b6.
@Chris-Kiddier could you run reward self-update --force please and test if it's working as expected?

Looks good! Session connected and I'm up and running.
Really appreciate it, coffee on the way @janosmiko

I'm really happy to hear it works!
Also I really appreciate the coffee! ☕


add more macro orientation options

This PR adds more macro orientation options.
Especially the placer needed to be made aware of the macro orientation, which for more extensive memory macros, was a problem. We tested this with the IHP13 srams: https://github.com/IHP-GmbH/IHP-Open-PDK/tree/main/ihp-sg13g2/libs.ref/sg13g2_sram
Thank you to the OpenROAD team for this amazing project 💯

I'm a bit confused as to how this problem arises.  Macros should be placed by the macro placer rather than dpo.
Are these macros intended to be instantiated in these extra orientations?  Often there are poly direction rules that forbid the 90/270 turns.

I'm also curious. Even if the poly/fin rules allowed rotation, usually the macro works best when aligned with the rest of the metal stack.

@maliberty @rovinski I think the question will be clearer once we do our project's fully open-source release (hopefully this week).
I will try to give a little overview but will pinpoint the repo and location once we have a release:
We wanted to manually place the macro as, for example, two memory macros need to face each other for the best QoR.
Our macros have the same config as the nangate45 or sky130 ones in the lef: (unfortunately, our lefs are not released yet. We are working on it: https://github.com/IHP-GmbH/IHP-Open-PDK/tree/main/ihp-sg13g2/libs.ref/sg13g2_sram/doc)
https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/blob/6ec980e1d49a1a8dcdd1e25ed81255b4bb8285c8/flow/platforms/nangate45/lef/fakeram45_128x116.lef#L5
SYMMETRY X Y R90 ;

which allows rotation in any direction for the macro.
Our code snippet. Again as soon as we have a release, I can give you a better overview :-)

proc placeInstance { name x y orient } {
  set block [ord::get_db_block]
  set inst [odb::dbBlock_findInst $block $name]
  odb::dbInst_setPlacementStatus $inst "none"
  odb::dbInst_setOrient $inst $orient
  odb::dbInst_setLocation $inst [ord::microns_to_dbu $y] [ord::microns_to_dbu $x]
  odb::dbInst_setPlacementStatus $inst "firm"
}

proc addHaloToBlock {halo name} {
  set block [ord::get_db_block]
  set inst [odb::dbBlock_findInst $block $name]

  set bbox [odb::dbInst_getBBox $inst]
  set minx [odb::dbBox_xMin $bbox]
  set miny [odb::dbBox_yMin $bbox]
  set maxx [odb::dbBox_xMax $bbox]
  set maxy [odb::dbBox_yMax $bbox]

  set minx [expr $minx - [ord::microns_to_dbu $halo]]
  set miny [expr $miny - [ord::microns_to_dbu $halo]]
  set maxx [expr $maxx + [ord::microns_to_dbu $halo]]
  set maxy [expr $maxy + [ord::microns_to_dbu $halo]]
}

# axi_data_0
set X [ expr $floorW - $sram_initX_L - $RamSize1024_H ]
set Y $sram_initY
placeInstance  $axi_data_0_low $X $Y MXR90
addHaloToBlock $haloBlock $axi_data_0_low
set X [ expr $X - $RamSize1024_H - 2*$haloBlock - $channel_no_cell ]
set Y $Y
placeInstance  $axi_data_0_high $X $Y R90
addHaloToBlock $haloBlock $axi_data_0_high

The fakeram's are not that realistic and I think that is a mistake.  You really don't want to turn a ram so the pins all require wrong way routing to access, even if that might be legal drc wise.  That is a recipe for horrible routing congestion.

It also means that the power rails will be harder to connect due to the wrong way stripes.

Stale


New auth handlers to support SSO features

connects #459

Currently the auth data structure is
{
    "facebook:<fb_id>": {
        "access_token": "....",
        .....
    }
}

After discussion, it is make more sense to put the id inside the dict. And we will also put the provider profile to the auth data.
{
    "facebook": {
        "principal_id": "<fb_id>",
        "access_token": "....",
        .....,
        "profile": <fb profile payload>
    }
}

Will reopen PR after updating


fix 3.9 jenkins job definitions

Fixes: 55c177cca3c740e7dae231b3fbcb7f51025f1e53

yeah, I have no idea how this happened.


WIP: Fragments for new reconciler

Closes #7 (for the new reconciler).
I started from the new-reconciler-fragments branch that @LPGhatguy made and finished repairing/adding tests and patching inconsistencies.
One thing that fell out of this change was the need to distinguish between these two scenarios:

Updating a virtual node's children from a list of new children

This is done by the RobloxRenderer to process the children of host nodes, and also the reconciler itself to process children specified via Roact.Children.
The children are expected to be in the form of a table.

Updating a virtual node's children with the result of rendering a component.

This is done by Component with its lifecycle methods, and also by the reconciler itself to mount/update function components.
The expected format is either an element or a fragment (or a boolean value or nil).

Right now, I'm enforcing the above rules with two different reconciler functions for updating a node's children.
Checklist before submitting:

[ ] Added entry to CHANGELOG.md
[ ] Added/updated relevant tests
[ ] Added/updated documentation

Fragments are a go!


Use separate subnets for different components [EKS]

This commit increases security by ensuring that cluster nodes, Nomad clients
and external VMs do not share subnets in EKS.
The subnets are distributed so that Nomad clients get half of the subnets
(rounded up if necessary), VMs all go into the same subnet and the cluster
nodes get the remaining subnets. If the Nomad clients are deployed to public
subnets, the all the private subnets can be used for cluster nodes except one
private subnet that is reserved for the VMs.
The Nomad clients now block all access from jobs to any VPC IP, except the CIDR
block for the subnet of the VMs and the VPC DNS server. This blocks all access
to any internal services and IPs from within standard jobs. ~It does not work
for jobs on external VMs, though.~ UPDATE: Using NACL rules, I was able to block
most traffic to pods. Directly cURL-ing the APIs of the services no longer works in
jobs running on external VMs
The following cURLs do not succeed from jobs with the changes applied, neither in Nomad jobs nor on external VMs with machine executor (they did previously):

Any cURLs to the Nomad server on port 4646, regardless if via AWS DNS name of the loadbalancer, the internal DNS record, the service's IP or the Pod IP - using a release that still exposed that port on the LB
Any cURLs to other services via internal DNS, e.g. VM service

Realitycheck still succeeds

After testing against master on staging-eks:

Port 4646 is no longer exposed via any external LB
I was also not able to get a response hitting 4647 via the exposed LB from a job container, the request was rejected bases on TLS

Unless you have a way of hitting either of those ports as master stand right now, I would suggest removing nomad from the scope of the PR

After testing against master on staging-eks:

Port 4646 is no longer exposed via any external LB
I was also not able to get a response hitting 4647 via the exposed LB from a job container, the request was rejected bases on TLS

Unless you have a way of hitting either of those ports as master stand right now, I would suggest removing nomad from the scope of the PR

Wouldn't that mean investing time for removing an additional layer of security? Why do you think this is necessary?
It's not that I block this communication to Nomad specifically, I rather ensure that only necessary communication to the cluster is permitted - As I think is best-practice. What exactly are you suggesting when you say "removing Nomad from the scope of the PR"?

I am suggesting a couple things.  Firstly removing this from the description.
"
Any cURLs to the Nomad server on port 4646, regardless if via AWS DNS name of the loadbalancer, the internal DNS record, the service's IP or the Pod IP - using a release that still exposed that port on the LB
"
also regarding this "The Nomad clients now block all access from jobs to any VPC IP" .  I have not been able to curl any pod IPs from jobs while testing master in staging, so as far as I can tell this does not need to be fix.  But I could be missing something, so send me a curl for staging-eks if you can reproduce

I am suggesting a couple things. Firstly removing this from the description.
"
Any cURLs to the Nomad server on port 4646, regardless if via AWS DNS name of the loadbalancer, the internal DNS record, the service's IP or the Pod IP - using a release that still exposed that port on the LB
"

I don't mean to be obnoxious, but why? This is still true and points out the additional layer of security added

also regarding this "The Nomad clients now block all access from jobs to any VPC IP" . I have not been able to curl any pod IPs from jobs while testing master in staging, so as far as I can tell this does not need to be fix. But I could be missing something, so send me a curl for staging-eks if you can reproduce

Sure, you shouldn't be able to curl pod IPs and you never were - but it was definitely possible to curl exposed services like vm-service from inside jobs, either using the load balancer DNS name or IP or the internal DNS record.


[GOBBLIN-807] TimingEvent is now closeable, extends GobblinEventBuilder

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-807

Description

[ ] Here are some details about my PR, including screenshots (if applicable):
Removed the new TimingEventBuilder
TimingEvent is now closeable and extends GobblinEventBuilder
EventSubmitter is no longer deprecated but older ways to submit events have been deprecated
Added CountEventBuilder that reports counts as GobblinTrackingEvent
GobblinEventBuilder should be used to build GobblinTrackingEvent and EventSubmitters are solely responsible for submitting the events.

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@ibuenros @zxcware Can you please take a look


189 display roads

Add layer for rendering roads and railroads.
Add a few roads at top left corner of c7-static-map for demonstration.
Movement costs weren't touched, roads just displayed.
Closes #189

Oh, one thing.  I noticed this is linked to #190, but it appears that while it adds support for BIQs/JSON saves, it doesn't for SAVs yet.  Which is okay, that was always intended as the next step.  Just wanted to confirm that we should leave #190 open but note that it's just SAV support that remains.

it appears that while it adds support for BIQs/JSON saves, it doesn't for SAVs yet.

I added roads to QueryCiv3.Sav, but didn't tested. Or you mean something else?

it appears that while it adds support for BIQs/JSON saves, it doesn't for SAVs yet.

I added roads to QueryCiv3.Sav, but didn't tested. Or you mean something else?

So you did... so I dug into it more why it wasn't showing up.  TILE in the SAV structure is more complex than in the BIQ, and it looks like the place you added it would work for Vanilla and PTW, but not for Conquests.
Below the area where you added it, there is this section:
        // if civ 3 conquests file (for now we'll assume this is always true)
        private fixed byte HeaderText2[4];
        public int Length2;

        // TODO: Most flags need to be added
        private fixed byte Flags2[12];

Based on the notes I have from a private Civ forum, Flags2 contains:
   OverlayBits      4            12         Overrides TILE#1 OverlayBits.
   Unknown          1            13
   TerrainBits      1            14         Split in two nibbles for real and base. Overrides TILE#1 TerrainBits.
   Unknown          2            16
   BonusBits        4            20         Overrides TILE#1 BonusBits.

Whereas Flags1 contains:
Unknown          2            24
OverlayBits      1            25           See TILE#2 for c3c.
TerrainBits      1            26           Split in two nibbles for real and base. See TILE#2 for c3c.
BonusBits        1            27           See TILE#2 for c3c.
RiverData        1            28

What is present right now is Flags1[2], index 0 and 1, which is correct for PTW/Vanilla.  I believe this means that Flags2[0], index 0 and 1, are what is required for Conquests.  It definitely should be tested with a SAV file that has some roads built after the scenario starts, since I am not sure that any open-source project has implemented this before.
(As a side note, while I've shared the SAV structure info I have via e-mail with some of our collaborators, we haven't yet found a place to store it for project members)

I loaded civ3, build city with the road near it, saved as .sav and opened it from C7. Map hasn't city, but road exits)


Add a tag Programmatically

Hi
I want to add a tag to a created and intialized input tags programmatically. I know that I can use addTag() with instance of AmsifySuggestags, but I do not know how I can retrieve the corresponding  instance of AmsifySuggestags of the created input.
I tried:
$('input[name="country"]').amsifySuggestags().addTag('xxx')
but no result

Actually the only way to add tag programmatically is by creating each instance of input like this
amsifySuggestags = new AmsifySuggestags($('input[name="country"]'));
amsifySuggestags._init();
amsifySuggestags.addTag('xxx');

You cannot use this initialization to add tag programmatically
$('input[name="country"]').amsifySuggestags();

thank you,
I have another question
Can I attach other hidden informations to the texts of tags. For example, using a select list,
we assign to each option a text that appears in the list and a real value in the value attribute.
Can I do the same thing with the tags?

Currently it does not have any such option. I will try to add this option and update here in this thread.

thank you

I have added a way to set tags with their respective values. you can set suggestions like this
$('input[name="color"]').amsifySuggestags({
	suggestions: [
					{'tag': 'Black', 'value': 1},
					{'tag': 'White', 'value': 2},
					{'tag': 'Red', 'value': 3},
					{'tag': 'Blue', 'value': 4},
					{'tag': 'Green', 'value': 5},
					{'tag': 'Orange', 'value': 6},
				]
});

Here these will appear as tags
Black
White
Red
Blue
Green
Orange

and the value will be submitted to form.
Lets say, if you select White and Blue tags then input will have their values separated by comma
2,4
Please let me know if its working fine.


Rare: A frontend for legendary, the open source Epic Games Launcher alternative.

Project description
Rare is a GUI for Legendary, a command line alternative to Epic Games launcher.
Metadata

homepage URL: https://github.com/Dummerle/Rare
source URL: https://github.com/Dummerle/Rare
license: gpl3
platforms: linux

Merged


Fallback for submodules

@casey Thanks for adding submodule support, it's working great.
Currently, when you invoke a submodule recipe within a directory containing a justfile with set fallback but no submodules defined (the requested submodule is defined in a parent justfile), the following error is encountered:
❯ just myapp::start
error: Justfile does not contain submodule `myapp`

It would be nice if the fallback setting (or another one, e.g. fallback-submodules) would allow just to look for a requested submodule in parent justfiles.
Sometimes you are in a nested folder structure, and you quickly want to call a recipe defined in the root justfile. Since fallback does not work for submodules, the workaround is to add the submodule to the justfile at your level in the hierarchy (even though the recipe you want to run is unrelated to that justfile), or to cd back to the root and run the recipe from there, or to add a recipe in the root justfile that calls into its own submodule definition.
Example
This is what happens currently:
❯ cd proj/foo/bar
❯ just --version
just 1.34.0
❯ just hello
hello
❯ just myapp::start
error: Justfile does not contain submodule `myapp`

This is what should happen:
❯ cd proj/foo/bar
❯ just hello
hello
❯ just myapp::start
starting

proj/justfile
mod myapp 'justfile-myapp.just'
default:
  just --list

proj/justfile-myapp.just
start:
  @echo "starting"

proj/foo/bar/justfile
set fallback
# set fallback-submodules

hello:
  @echo "hello"

Thanks for the report! Fortunately, this was extremely easy to implement. Fixed in #2329.

Available in 1.35.0, just released: https://github.com/casey/just/releases/tag/1.35.0

Thanks @casey!


ConsumerGroup connecting to partitions

My ConsumerGroup only picks one to three random partitions to connect to instead of all seven partitions I have created. I have no idea why. I am not doing anything special.
const topic = 'chatbot_notification';
const consumer = new kafka.ConsumerGroup({
  kafkaHost: ''slcq102hkf001.slcq102.com:9092'
}, topic);

Logs:
  kafka-node:KafkaClient Sending versions request to slcq102hkf001.slcq102.com:9092 +182ms
  kafka-node:KafkaClient broker socket connected {"host":"slcq102hkf001.slcq102.com","port":"9092"} +2ms
  kafka-node:KafkaClient updating metadatas +70ms
  kafka-node:ConsumerGroup Connecting kafka-node-client +0ms
  kafka-node:ConsumerGroup GroupCoordinator Response: +27ms { coordinatorHost: 'slcq102hkf001.slcq102.com',
  coordinatorPort: 9092,
  coordinatorId: 0 }
  kafka-node:KafkaClient getApiVersions request timedout probably less than 0.10 using base support +405ms
  kafka-node:KafkaClient setting api support to {"produce":{"min":0,"max":1,"usable":1},"fetch":{"min":0,"max":0,"usable":0},"offset":{"min":0,"max":0,"usable":0},"metadata":{"min":0,"max":0,"usable":0},"leader":null,"stopReplica":null,"updateMetadata":null,"controlledShutdown":null,"offsetCommit":{"min":0,"max":2,"usable":2},"offsetFetch":{"min":0,"max":1,"usable":1},"groupCoordinator":{"min":0,"max":0,"usable":0},"joinGroup":{"min":0,"max":0,"usable":0},"heartbeat":{"min":0,"max":0,"usable":0},"leaveGroup":{"min":0,"max":0,"usable":0},"syncGroup":{"min":0,"max":0,"usable":0},"describeGroups":null,"listGroups":null,"saslHandshake":null,"apiVersions":{"min":0,"max":0,"usable":0},"createTopics":null,"deleteTopics":null} +0ms
  kafka-node:ConsumerGroup joinGroupResponse {"members":[],"generationId":1562,"groupProtocol":"roundrobin","leaderId":"kafka-node-client-4919207f-e16f-4058-b66e-06ed13f058b3","memberId":"kafka-node-client-b06ac6c4-aead-4fc2-87e6-5554a45699e2"} from kafka-node-client +3s
  kafka-node:ConsumerGroup SyncGroup Request from kafka-node-client-b06ac6c4-aead-4fc2-87e6-5554a45699e2 +0ms
  kafka-node:ConsumerGroup SyncGroup Response +26ms
  kafka-node:ConsumerGroup kafka-node-client owns topics:  +0ms { chatbot_notification: [ 1, 4, 7 ] }
  kafka-node:ConsumerGroup kafka-node-client fetchOffset Response: {"chatbot_notification":{"1":63,"4":55,"7":53}} +31ms
  kafka-node:ConsumerGroup Has saved offsets +0ms
  kafka-node:ConsumerGroup generationId +0ms 1562
  kafka-node:ConsumerGroup kafka-node-client started heartbeats at every 10000 ms +4ms

As you can see, it only picks partitions 1, 4 and 7. Now if I produce a message to partition 5, my consumer never sees it (unless I restart my process and somehow it manages to connect to partition 5). This is annoying. Thanks for helping out.

According to the provided log you have another consumer with the same group id.  Looks like there are two members in your group and roundrobin will try to distribute the partitions evenly. That's why this one gets 3 partitions.
If you want a single consumer to own all the partitions give it a unique group id. By default the group is kafka-node-group.

Thanks!


Add Active (Clicked Down) UI Mode?

To have a better understand of I clicked down.
Something like:

If you click, the calendar will appear so you know you clicked. Perhaps I don't understand what you mean?

I mean something like this, the blue background show its in active mode.


fix: Update spring-security-config for CVE-2023-34034

Fixes https://github.com/appsmithorg/appsmith-ee/issues/1968

/ok-to-test

@sharat87 why are we updating just spring-security and not the all of spring-boot?

@nidhi-nair, I tried updating Sprint boot's version, but that didn't make the CVE go away. Spring security's version wasn't changing at all. That's when I went ahead and changed this version, and to only make minimal changes in this PR, didn't change Spring Boot's version.

Working on the build failure now.

/build-deploy-preview

Just verified the DP image and the CVE isn't there.

/ok-to-test

Aw why are props not allowed there ☹

@nidhi-nair, I wish we can figure this out, perhaps later. When using a property there, the reactiveCaching module is failing to build. Didn't get a chance to look into why. 🙂


Rewrite response Har content

Hello! My issue is to change json response from https site. I see it by getText(), then use setText(), and change its size by setSize and setBodySize. But It seems there is no effect and chrome receives the same old json... Is there a way?
I think maybe the point is that "response.getRawResponse().getEntity().getContentLength()" somehow has old value, but i can't rewrite it :( Or the cause is different at all?

What are you using to change the contents? Can you paste some code? Also need the version of BMP, etc.

BMP 2.1.0-beta-4 and Selenium 2.50.1
`    public static class ResponseModifier implements ResponseInterceptor {
@Override
public void process(BrowserMobHttpResponse response, Har har) {

    String contentType = response.getHeader("Content-Type");

    if (contentType!=null && contentType.startsWith("application/json")){
        response.getEntry().getResponse().getContent().setText(respond);
        response.getEntry().getResponse().getContent().setSize(respond.length());
        response.getEntry().getResponse().setBodySize(respond.length());
        response.getRawResponse().removeHeaders("Content-Length");
        response.getRawResponse().addHeader("Content-Length", ""+respond.length());
    }
}`

Maybe point is not in Har at all :) All I need is to change the json string in the server response.

I see what you're trying to achieve. A couple things will help:
First, don't use the legacy ProxyServer class. Use BrowserMobProxy and create an instance of BrowserMobProxyServer.
Second, use a filter to modify requests/responses. See the readme for details: https://github.com/lightbody/browsermob-proxy#http-request-manipulation

Thanks! It turn out to be far easier than I thought :)

@Zap4ick - Glad to hear. Were you able to successfully rewrite the body contents?

Yes, I just added a filter and rewrited text content.


lang-update-pt-br

Translated strings from English to Brazilian Portuguese, fixed typos and improved previous messages.

Thanks!

Thanks!


migrate to php7 Swift_IdGenerator

Changed

refactor Swift_Mime_IdGenerator, Swift_IdGenerator with return types
replaced redundant documentation

I'm closing this PR as we won't migrate to type-hints. Swiftmailer will be replace by Symfony Mailer soon.


Remove np.array

Proposed changes:

Remove the explicit np.array conversion. It is not necessary and creates a deprecation warning.
fixes https://github.com/RasaHQ/rasa/issues/6282

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

I just tested it. I still see
2020-08-10 08:49:11 INFO     rasa.nlu.model  - Starting to train component DIETClassifier
/Users/tabergma/Repositories/rasa/rasa/nlu/classifiers/diet_classifier.py:683: VisibleDeprecationWarning: Creating an ndarray from ragged nested sequences (which is a list-or-tuple of lists-or-tuples-or ndarrays with different lengths or shapes) is deprecated. If you meant to do this, you must specify 'dtype=object' when creating the ndarray

when executing rasa train nlu. Can we please also fix this?

This is odd. It vanished for me once I removed the array. Perhaps this also depends on the Python version... I'll check later this week.

@tabergma What are your numpy and python versions?

python3.7 and numpy==1.19.1


As a user I need to access reports for different kinds of stats in a MoneyLog

Feature/Fix
This is a feature proposal. The main reason we decided to work on this application is due to the fact that we need to pull a certain number and types of reports from recording all these items. These reports should aid in making adjustments to the money operations enabling us to meet criteria set for fiscal periods
Current Situation
At the moment we are struggling with this section as we cant really generate reports with the current system we are using. A number of formulas are required all dependent on other formulas this is seen as a simpler solution.
The following is a list of reports that would be needed among others

[ ]  How much is currently spent on a category vs the amount budgeted
[ ] Reports for total Earnings
[ ] Reports for total spending
[ ] Due items (things close to being paid for)

Acceptance Requirements
A user should be able to explore reports for fiscal periods/budgets

Closing this issue as it belongs to the Front End


Updating dependencies to latest Aurelia build

"aurelia-dependency-injection"<EMAIL_ADDRESS>

why not making a PR instead? Is really easy and will help more...

I'm going through the process of updating at the moment hope to have something out soon.


Label file sample

Hi,
It would be good to see sample of label file - is it the same format as X++ label file and stored in AOT or?

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

ID: ac87ed28-11fd-54d2-e20b-2a46b7bef037
Version Independent ID: 1eb0d33a-1f03-21ff-d098-4fc4a5824c6a
Content: Create localizable labels - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/user-interface/create-localizable-labels-client.md
Service: dynamics-ax-platform
GitHub Login: @RobinARH
Microsoft Alias: shshabazz

@OksanaKovaliova The following link might be helpful to you until we can update this topic with more information: https://kaya-consulting.com/how-to-create-a-new-language-or-country-to-dynamics-365-for-operations/
The link is not to a Microsoft post, but it shows some screenshots of working with label files.
We're tracking the work on this topic internally, so I will close this issue for now. If you have further suggestions for improving our help content, please don't hesitate to log more issues.


Add python bindings

broken coverage is expected until #273 is merged because of the docker image update.


Since Helm 3.1.0 we got a lot of warnings from awscli that too much clients are created.

Output of helm version:
version.BuildInfo{Version:"v3.1.0", GitCommit:"b29d20baf09943e134c2fa5e1e1cab3bf93315fa", GitTreeState:"clean", GoVersion:"go1.13.7"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.3", GitCommit:"06ad960bfd03b39c8310aaf92d1e7c12ce618213", GitTreeState:"clean", BuildDate:"2020-02-12T13:43:46Z", GoVersion:"go1.13.7", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.9-eks-c0eccc", GitCommit:"c0eccca51d7500bb03b2f163dd8d534ffeb2f7a2", GitTreeState:"clean", BuildDate:"2019-12-22T23:14:11Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
AWS
When we run install or upgrade then a lot of
W0303 09:43:12.111352    8329 exec.go:201] constructing many client instances from the same exec auth config can cause performance problems during cert rotation and can exhaust available network connections; 1961 clients constructed calling "aws"

occur. We use 1 prehook and 1 posthook. It was not happening with Helm 3.0.x. Does anybody know if this is ok or not? :-)

This appears to be some issue in using client-go to auth against AWS. This is the source of error:
https://github.com/kubernetes/client-go/blob/master/plugin/pkg/client/auth/exec/exec.go#L202
I'm wondering if there was some environmental issue with the "aws" plugin (which I'm assuming means the aws-iam-autheticator). I'm wondering if that needs to be updated.

I'll check out if we are outdated. I suppose that my PC is updated, but will check it out tomorrow. :-)

aws-iam-authenticator version
{"Version":"v0.4.0","Commit":"c141eda34ad1b6b4d71056810951801348f8c367"}

It's installed from https://docs.aws.amazon.com/eks/latest/userguide/install-aws-iam-authenticator.html

@jdolitsky would you know what PR could cause this? Older version of Helm 3 was doing fine and only Helm was updated to 3.1.x

@TomasKohout do you have a previous version of helm (specific 3.x version) that works?

3.0.3 was without thouse warnings. I've tried all rc versions of 3.1.0. Those warnings occur on all of them.

I think that this is actually a regression so it should be labeled accordingly. Also creating 3000 clients is not a very good practice. Why does Helm needs so much clients?

Looks like for every kubernetes object new client is created. @jdolitsky could you verify this? :)

That should not be the case. The kubernetes client is created once per invocation and is shared for the entire install operation.
See https://github.com/helm/helm/blob/59447f850c88251afcca9a67b8ce44131a73c46a/pkg/action/action.go#L225

Well, here are logs from upgrade action.
helm3-0-3.log
helm3-1-2.log

I do not know guys. Maybe new version of kubernetes package? :-)

Thank you for the logs.
I found the source of the log message. That is definitely code which the EKS authenticator would traverse.
Based on the information provided here, I can only see this as an issue coming from the EKS authenticator, which is outside of Helm's control.
I noticed you are running an older version of Kubernetes: 1.14. Have you considered upgrading?
I would highly suggest looking at the developer guide to see if you can try to nail the issue down yourself. I have not seen a similar report from another user, and I have not observed this behaviour before. This is why I'm asking to please let us know if you find anything.

I do not know guys. Maybe new version of kubernetes package? :-)

Helm 3.1.2 uses Kubernetes 1.17 core libraries, and HEAD uses 1.18. As far as Helm's concerned, we are keeping up to date with Kubernetes here.
Hope this helps.

I didn't want to add a "me too" reply, but because you said " I have not seen a similar report from another user"...
I'm on EKS 1.15 with Helm 2.11 upgrading to 3.2.1. I needed to update some of my helm packages to address deprecations in Kubernetes 1.16, and in order to do that I needed to upgrade Helm and figured I'd go all the way to 3.2.1. I'm migrating my packages using https://helm.sh/blog/migrate-from-helm-v2-to-helm-v3/ . During the final step of the conversion process (helm3 2to3 cleanup) I got the same warnings, with the final one being W0515 16:20:25.464868     541 exec.go:201] constructing many client instances from the same exec auth config can cause performance problems during cert rotation and can exhaust available network connections; 3034 clients constructed calling "aws". That does seem like a lot of clients :-) .
I haven't seen any adverse effects from this, just wanted to chime in to say that it isn't just that one guy running into it.

Found the upstream issue: https://github.com/kubernetes-sigs/aws-iam-authenticator/issues/308. Please follow up there. There's no action to take here on our end.

I've just had a similar scenario while using helm in a go app
Getting more details
kube.(*Client).IsReachable always asks for a new client and then eventually it reaches 1000 and starts to warn
https://github.com/kubernetes/client-go/blob/master/plugin/pkg/client/auth/exec/exec.go#L202
Stacktrace:
k8s.io/client-go/plugin/pkg/client/auth/exec.(*Authenticator).UpdateTransportConfig at exec.go:203
k8s.io/client-go/rest.(*Config).TransportConfig at transport.go:100
k8s.io/client-go/rest.TransportFor at transport.go:42
k8s.io/client-go/rest.RESTClientFor at config.go:295
k8s.io/client-go/kubernetes/typed/certificates/v1beta1.NewForConfig at certificates_client.go:47
k8s.io/client-go/kubernetes.NewForConfig at clientset.go:444

k8s.io/kubectl/pkg/cmd/util.(*factoryImpl).KubernetesClientSet at factory_client_access.go:83
helm.sh/helm/v3/pkg/kube.(*Client).IsReachable at client.go:91

In my scenario, I have one go app that interacts with multiple namespaces and helm releases
Would it be possible to cache and reuse the kube client?


How to train your own dataset?

How to train your own dataset?

Check out https://github.com/justinpinkney/stable-diffusion/blob/main/ldm/data/simple.py and either use or modify FolderData for your purposes. Make sure to edit the yaml file as well to use FolderData instead of hf_dataset.


python311Packages.google-cloud-compute: 1.12.1 -> 1.13.0

Changelog: https://github.com/googleapis/python-compute/blob/v1.13.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

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

2 packages marked as broken and skipped:

python310Packages.dalle-mini
python310Packages.dalle-mini.dist

30 packages built:

openai-full
openai-full.dist
pentestgpt
pentestgpt.dist
python310Packages.fastai
python310Packages.fastai.dist
python310Packages.google-cloud-compute
python310Packages.google-cloud-compute.dist
python310Packages.k-diffusion
python310Packages.k-diffusion.dist
python310Packages.langchain
python310Packages.langchain.dist
python310Packages.skrl
python310Packages.skrl.dist
python310Packages.spacy
python310Packages.spacy-loggers
python310Packages.spacy-loggers.dist
python310Packages.spacy-transformers
python310Packages.spacy-transformers.dist
python310Packages.spacy.dist
python310Packages.textacy
python310Packages.textacy.dist
python310Packages.textnets
python310Packages.textnets.dist
python310Packages.wandb
python310Packages.wandb.dist
python311Packages.google-cloud-compute
python311Packages.google-cloud-compute.dist
tts
tts.dist


Strange chars in dist/ExifTool.d.ts

Hi, I found some strange chars in dist/ExifTool.d.ts:
/**
 * Use this singleton rather than instantiating new ExifTool instances in order
 * to leverage a single running ExifTool process. As of v3.0, its `maxProcs` is
 * set to the number of CPUs on the current system; no more than `maxProcs`
 * instances of `exiftool` will be spawned. You may want to experiment with
 * smaller or larger values for `maxProcs`, depending on CPU and disk speed of
 * your system and performance tradeoffs.
 *
 * Note that each child process consumes between 10 and 50 MB of RAM. If you
 * have limited system resources you may want to use a smaller `maxProcs` value.
 */
export declare const exiftool: ExifTool;
ol;  // <=This

I tried yarn cache clean and reinstall it, but it still exist.
The ol; seems like... I don't know, maybe your cat on keyboard 😆 ?

Naughty :cat:!
I'll republish. Thanks for the catch.

Published v4.17.0 🎉


Inside the windows service this package not working

Library version
<IP_ADDRESS>
OS & OS version
Windows 10 Pro & 21H1
Describe the bug
Inside the windows service it does not trigger any events with this package.
When we tested this package, it worked in the console application. But this does not work within the Windows services. Is there are any restrictions for this to work inside the Windows services?
To Reproduce

Create a new Windows service project
Add the NuGet package USB. Events then initialize the functionality inside the OnStart event
But the events were not triggered, to track the function we used the event logger but there are no logs in the event viewer.

Events then initialize the functionality inside the OnStart event

These are not exact steps to reproduce the problem.
Please provide exact code to reproduce the problem.

To Reproduce

In Visual Studio 2019, we created a project for Windows services

We added the following code blocks to the Windows service's OnStart method

`try

{
EventLog.WriteEvent("TestService", new EventInstance(0, 0, EventLogEntryType.Information), new string[] { "Service start successfully." });
using (IUsbEventWatcher usbEventWatcher = new UsbEventWatcher())
{
EventLog.WriteEvent("TestService", new EventInstance(0, 0, EventLogEntryType.Information), new string[] { usbEventWatcher.UsbDeviceList.FirstOrDefault().DeviceName.ToString() });
Console.WriteLine(usbEventWatcher.UsbDeviceList.FirstOrDefault());
usbEventWatcher.UsbDeviceRemoved += (_, device) => EventLog.WriteEvent("TestService", new EventInstance(0, 0, EventLogEntryType.Information), new string[] { "Removed:" + Environment.NewLine + device + Environment.NewLine });
	usbEventWatcher.UsbDeviceAdded += (_, device) => EventLog.WriteEvent("TestService", new EventInstance(0, 0, EventLogEntryType.Information), new string[] { "Added:" + Environment.NewLine + device + Environment.NewLine });

	usbEventWatcher.UsbDriveEjected += (_, path) => EventLog.WriteEvent("TestService", new EventInstance(0, 0, EventLogEntryType.Information), new string[] { "Ejected:" + Environment.NewLine + path + Environment.NewLine });

	usbEventWatcher.UsbDriveMounted += (_, path) =>
	{
		Console.WriteLine("Mounted:" + Environment.NewLine + path + Environment.NewLine);
		EventLog.WriteEvent("Mounted", new EventInstance(0, 0, EventLogEntryType.Information), new string[] { "Ejected:" + Environment.NewLine + path + Environment.NewLine });
		foreach (string entry in Directory.GetFileSystemEntries(path))
			EventLog.WriteEvent("TestService", new EventInstance(0, 0, EventLogEntryType.Information), new string[] { "Entries:" + Environment.NewLine + entry + Environment.NewLine });

	};
}

}
catch (Exception ex)
{
EventLog.WriteEvent("TestService", new EventInstance(0, 0, EventLogEntryType.Information), new string[] { ex.Message });
}`

However, the events were not triggered. The function was tracked using the event logger, but there are no logs in the event viewer.

We missed anything here?

IUsbEventWatcher usbEventWatcher starts listening for USB events from the moment you call usbEventWatcher = new UsbEventWatcher() and it stops listening for USB events when you call usbEventWatcher.Dispose()
using (IUsbEventWatcher usbEventWatcher = new UsbEventWatcher()) calls usbEventWatcher.Dispose() when the execution reaches } because that is how using works:
https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/using-statement
solution:

make IUsbEventWatcher usbEventWatcher a  private member of the Windows service class
call usbEventWatcher = new UsbEventWatcher() inside OnStart - without using
call usbEventWatcher.Dispose() inside OnStop

@Jinjinov: Your efforts are greatly appreciated and thank you for providing some valuable solutions on time. It worked great and your quick response was really appreciated.


Use Electron v1.7.6

Electron deleted v1.8.0 due to stability / security issues.  Use v1.7.6 specifically as a dependency until they release 1.8.1.

A clean npm install works, since they've removed 1.8.0 from npm entirely.  I don't think we need to do anything here.


ARROW-18380: [Dev] Update dev_pr GitHub workflows to accept both GitHub issues and JIRA

This might require to be done once the merge_arrow_pr.sh script is also updated. Updating the merge PR script is tracked on GitHub for the migration here: #14720
I have tested this new workflow on my fork with the following PRs and issues on my fork.
Example of valid issue: https://github.com/raulcd/arrow/pull/41
Example of issues without labels or assignees: https://github.com/raulcd/arrow/pull/42 or https://github.com/raulcd/arrow/pull/43
Example of issue with non existent GH issue: https://github.com/raulcd/arrow/pull/45

The PR expects the title to be either as it was before or:
GH-XXXX: [Component] Description where XXXX is the GitHub issue id. It also checks that the GitHub issue is assigned and that it has any label on it. We can improve it in the future to validate it has specific labels for components.

We could add a Closes #123 to the PR body if an existing gh issue was detected from the title (and no keyword is in the body) to make sure they are linked. That way the issue is closed automatically and we don't have to add that to the merge script.

At the moment we can do this on the merge script as we do with JIRA. I don't want to automatically edit the PR body at the moment. We can tackle this later on.

Apart from some refactoring based on the review comments, the main change from the examples on the description is that the issue will be automatically assigned to the PR creator if there are no assignees. See this comment on my fork as an example: https://github.com/raulcd/arrow/pull/50#issuecomment-1327640826

Looks good to me!
I wanted to give the same comment as @assignUser that we could also update the top comment to include a link to the issue instead of adding it as a new comment (this could also be done for JIRAs, except that here it doesn't matter for auto-closing). But also agree this is fine to consider later, as that is a bigger change.
The cpython bot also seems to do that, see for example https://github.com/python/cpython/pull/99825, where the top-post was edited by the bot to include the issue from the title (and in addition, the issue top post was also updated by the bot to explicitly link to the linked PR: https://github.com/python/cpython/issues/99824)


Enable/Disable tool

What:
A toggle to enable and disable the whole tool.
Why:
For example, we have an app with its own debug menu. It would be great to have a toggle to enable and disable the whole tool on a debug build within this debug menu.
I have a branch ready to go

Hi @Mr-Anderson-01, thanks for getting involved!
Is there a reason you need a specific toggle, could you not do something like this?
class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      builder: (context, child) => isDebugFlagEnabled ? AccessibilityTools(child: child) : child,
      home: HomePage(),
    );
  }
}

In the readme I see you guys mentioned that it's only available in debug mode and not compiled into release mode. We need some way to allow this in release mode for different environments e.g QA/Stg which we can distribute to designers for reviewing all screens which have accessibility issues

Hi @SAGARSURI! Have you found a solution?
I've got an idea that availability of the tools could be made customizable. There could be a flag that changes behaviour: defaultBehavior would make it enabled in debug builds and disabled in release builds. alwaysOn could force enable it in any build, alwaysOff could force disable it in any build. Developers would be able to choose when tools are available based on app flavor, app state etc.
Is it something you're looking for?

I believe this comment answers the original question. For enabling tools in release mode please see #36


sigh fails to find existing certificate

Hi,
on our jenkins slave, we encounter an issue with sigh.
As you can see below, it thinks that there's no existing profile, but when it tries to create the profile, it realizes that the profile already exists and uses a different name.
This problem only occurs on our jenkins, which also still has problems executing xcodebuild, so this might be a problem on our side.
[17:52:11]: ------------------
[17:52:11]: --- Step: sigh ---
[17:52:11]: ------------------
[17:52:11]: Starting login
[17:52:12]: Successfully logged in
[17:52:12]: Fetching profiles...
[17:52:22]: No existing profiles, creating a new one for you
[17:52:24]: The name 'XXX AdHoc' is already taken, using another one.
[17:52:24]: Creating new provisioning profile for 'XXX' with name 'XXX AdHoc<PHONE_NUMBER>'
[17:52:27]: Downloading provisioning profile...
[17:52:28]: Successfully downloaded provisioning profile...
[17:52:28]: Installing provisioning profile...
[17:52:28]: Provisioning profile of app 'XXX' with the name 'XXX AdHoc<PHONE_NUMBER>' successfully analysed.

I seem to have the same problem with cert, cert fails to find an existing signing entity on that exact same computer.

But it's working when you start it directly and only causes issues when being run from a Jenkins slave?

I thought Jenkins might be the cause, but I also logged in to the User Interface of  that exact machine with a different user, and the problem persists.
On a different machine it works.

Can it be that you haven't installed the required code signing identity on that machine? sigh automatically verifies the code signing identity to only download provisioning profiles that can actually be used.

That fixed it, you're right! Thanks a lot! :+1:

I should probably make the error message more clear, will push a message soon.

Awesome!


Parallel requests

We need a rebase!

Rebas'd

This will sadly conflict with #126, but I'll do the rebase once that's merged!

I'll do the rebase later

Good idea


Hardware dependencies?

From the perspective of testing the TAGE + CNN's misprediction rates, are there any specific hardware dependencies -- like a custom processor, required to run this codebase?
I have a Linux system and have installed the following dependencies mentioned in the readme:

Linux (I mainly use CentOS)
Python packages, with the versions that I've used:
Package         Version

h5py            2.10.0
matplotlib      3.1.1
numpy           1.17.2
PyYAML          5.1.2
torch           1.3.0
torchvision     0.4.1
Intel Pin Tool for generating branch traces (I've tested with 3.5 and 3.11)
cmake3 and make

You would need a GPU to run CNN training and evaluation steps. You can technically run it on a CPU for small-scale experiments but it would be too slow.
Any GPU would be fine as long as Pytorch can recognize and use the GPU.

Thanks!


Logging discussion

The current approach to logging in Proto.Actor comes with a lot of limitations.
If we were to improve on this, what would be the best options?
Go for a standard logging library? improve what we already have to allow new ways to subscribe to the log stream?
Going forward, OpenTelemetry is the defacto standard for observability nowadays. Otel has support for Logging, which sadly is not implemented in the Otel Go API yet.
It would be great if we in the future could point our logs to such integration and get logs and traces unified.
Suggestions here?

Until the OpenTelemetry go sdk implementation is stable, we can consider using a mature logging tool like zap. zap itself is a lightweight interface, and the underlying zapcore provides the ability to configure the encoding (e.g., BSON), add log sink (e.g., Kafka), and more. And because zap is so popular, there are many tutorials for accessing Sentry or Rollbar (it's even mentioned in the official documentation). Developers can easily adapt the log output to the apm tool they are using.
See the Configuring Zap section on zap doc at https://pkg.go.dev/go.uber.org/zap#hdr-Configuring_Zap for details.


Allow parsing of an empty string

An empty string should result in returning the existing relation.

Interestingly, GitHub autocloses a PR if the branch it's being merged into gets merged. And I can't seem to open the PR again.

Okay I had to restore the branch that was auto-deleted when I merged, and now I can re-open it.

Once I re-opened it I could change the base branch back to master.


Update create_octree.ipynb

minor edits

Same here, I had to do the edits manually, but big thanks for reporting!


(IAC-91) Postgres flexible server support

[In DRAFT] added changes to support Azure Postgresql Flexible Server for #79
This PR replaces https://github.com/sassoftware/viya4-iac-azure/pull/171 given the extent of changes to Postgresql since PR 171 was opened.
Changes here follows this https://docs.microsoft.com/en-us/azure/postgresql/flexible-server/how-to-manage-firewall-portal uses 'Public access (allowed IP addresses)' option to configure Postgresql firewall rules for external access and to allow connections from Viya4 pods on Azure AKS and enables https://docs.microsoft.com/en-us/azure/postgresql/flexible-server/how-to-manage-firewall-portal#connecting-from-azure. This follows the currently supported pattern for Azure Postgres Single server.

Superseded by https://github.com/sassoftware/viya4-iac-azure/pull/261. Closing this old PR.


L515 "Not Functioning" Error "Backend in rs2_open_multiple...."

Have you seen this. It seems  one of our customers has a L515 that  is faulty. They have tried with several computers, several cables, several USB ports, under windows and linux with the same result. Screen shots show the error message both in Windows and Linux
The RGB works fine though and our other RealSense cameras (D435i and T265...) also work fine.

Thanks for submitting this.  Can you submit a ticket here:  https://support.intelrealsense.com/hc/en-us/requests/new


Added Marathon format

Coverage remained the same at 100.0% when pulling b1cc3cf73c8899bc6a3b7c507c95cbc9e2c373ca on marathon into e908795d0b34784502de3feed131fad8b601c184 on master.


Linking humans.txt in

Coverage decreased (-0.03%) to 58.852% when pulling 7279102ab328fcdadbdbfd89137ddc464b7e0506 on vishnayak:add-humansTXT-reference into 99d827d014018e4531e63c1c9cf13da2f7c3891c on pump-io:master.