Remove PodSecurityPolicy from requiredPlugins and add a user-facing warning to consider migration

How to categorize this PR?

/area usability control-plane testing
/kind enhancement
What this PR does / why we need it:
This PR adds a user-faced warning for shoots >= v1.23 and < v1.25, to consider migrating from PodSecurityPolicy to PodSecurity admission controller.
This PR also removes the PodSecurityPolicy from the required plugins.
The e2e tests for create-update-delete shoot is also enhanced.
Which issue(s) this PR fixes:
Part of #5250
Special notes for your reviewer:
/hold until https://github.com/gardener/gardener-extension-provider-openstack/pull/485 is released.
Release note:

It is now possible to disable `PodSecurityPolicy` admission plugin, please make sure you have updated the extensions to a version which supports this change.

/test all

@shafeeqes: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name	Commit	Details	Required	Rerun command
pull-gardener-check-vulnerabilities	70c6179 link	false	/test pull-gardener-check-vulnerabilities
pull-gardener-e2e-kind	70c6179 link	true	/test pull-gardener-e2e-kind
Full PR test history. Your PR dashboard. Command help for this repository. Please help us cut down on flakes by linking this test failure to an open flake report or filing a new flake report if you can't find an existing one. Also see our testing guideline for how to avoid and hunt flakes.

 Expected success, but got an error:
      <*retry.Error | 0xc000512a40>: {
          ctxError: <context.deadlineExceededError>{},
          err: <*errors.StatusError | 0xc000827180>{
              ErrStatus: {
                  TypeMeta: {Kind: "", APIVersion: ""},
                  ListMeta: {
                      SelfLink: "",
                      ResourceVersion: "",
                      Continue: "",
                      RemainingItemCount: nil,
                  },
                  Status: "Failure",
                  Message: "Shoot.core.gardener.cloud \"e2e-default\" is invalid: [spec.kubernetes.version: Forbidden: admission plugin \"PodSecurityPolicy\" should be disabled for Kubernetes versions >=1.25, please check https://github.com/gardener/gardener/blob/master/docs/usage/pod-security.md#migrating-from-podsecuritypolicys-to-podsecurity-admission-controller, spec.kubernetes.allowPrivilegedContainers: Forbidden: for Kubernetes versions >= 1.25, allowPrivilegedContainers field should not be set, please see https://github.com/gardener/gardener/blob/master/docs/usage/pod-security.md#speckubernetesallowprivilegedcontainers-in-the-shoot-spec]",
                  Reason: "Invalid",
                  Details: {
                      Name: "e2e-default",
                      Group: "core.gardener.cloud",
                      Kind: "Shoot",
                      UID: "",
                      Causes: [
                          {
                              Type: "FieldValueForbidden",
                              Message: "Forbidden: admission plugin \"PodSecurityPolicy\" should be disabled for Kubernetes versions >=1.25, please check https://github.com/gardener/gardener/blob/master/docs/usage/pod-security.md#migrating-from-podsecuritypolicys-to-podsecurity-admission-controller",
                              Field: "spec.kubernetes.version",
                          },
                          {
                              Type: "FieldValueForbidden",
                              Message: "Forbidden: for Kubernetes versions >= 1.25, allowPrivilegedContainers field should not be set, please see https://github.com/gardener/gardener/blob/master/docs/usage/pod-security.md#speckubernetesallowprivilegedcontainers-in-the-shoot-spec",
                              Field: "spec.kubernetes.allowPrivilegedContainers",
                          },
                      ],
                      RetryAfterSeconds: 0,
                  },
                  Code: 422,
              },
          },
      }

This failure is expected. Will now fix this in another commit.

/test all

/test all

/assign

@shafeeqes: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name	Commit	Details	Required	Rerun command
pull-gardener-e2e-kind	10216f8 link	true	/test pull-gardener-e2e-kind
pull-gardener-check-vulnerabilities	c10e8fb link	false	/test pull-gardener-check-vulnerabilities
pull-gardener-integration	c10e8fb link	true	/test pull-gardener-integration
Full PR test history. Your PR dashboard. Command help for this repository. Please help us cut down on flakes by linking this test failure to an open flake report or filing a new flake report if you can't find an existing one. Also see our testing guideline for how to avoid and hunt flakes.

Related to https://github.com/gardener/gardener/issues/6671
/test pull-gardener-integration

/retest

/test pull-gardener-e2e-kind

Is it safe to remove the check if PodSecurityPolicy is required for k8s versions before 1.22.0? IIUC this could cause some problems for older versions of the extensions that were not adapted to handle missing PSPs (@shafeeqes)?
I can suggest to add an additional flag to https://github.com/gardener/gardener/blob/3f57fd0f415117075724d94b6aa051cee558200d/pkg/utils/validation/admissionplugins/admissionplugins.go#L88
called RequiredBeforeVersion and have this defined for the PodSecurityPolicy entry here: https://github.com/gardener/gardener/blob/3f57fd0f415117075724d94b6aa051cee558200d/pkg/utils/validation/admissionplugins/admissionplugins.go#L65
@rfranzke wdyt?

Is it safe to remove the check if PodSecurityPolicy is required for k8s versions before 1.22.0? IIUC this could cause some problems for older versions of the extensions that were not adapted to handle missing PSPs (@shafeeqes)?
I can suggest to add an additional flag to
https://github.com/gardener/gardener/blob/3f57fd0f415117075724d94b6aa051cee558200d/pkg/utils/validation/admissionplugins/admissionplugins.go#L88
called RequiredBeforeVersion and have this defined for the PodSecurityPolicy entry here:
https://github.com/gardener/gardener/blob/3f57fd0f415117075724d94b6aa051cee558200d/pkg/utils/validation/admissionplugins/admissionplugins.go#L65
@rfranzke wdyt?

This extension issue can still happen for versions > 1.22 also, right?

We can probably add a release note in this PR, saying you need this version of extension before updating to this Gardener version?

PSP is not really required for a functional cluster, is it? If it was disabled then there would simply be no effect (it's like "unprivileged" mode).

PSP is not really required for a functional cluster, is it? If it was disabled then there would simply be no effect (it's like "unprivileged" mode).

Did you mean to say "privileged"?  Without the PSP admission plugin, there won't be any "restrictions" for the pods.

New provider-openstack version is released, We can now unhold this PR now /cc @shafeeqes .

/milestone v1.58

/unhold
All the extensions are adapted with necessary change and released.


feat(image): Adding Notes on WebP Support for iOS

WebP support on iOS is only available starting from iOS 14. If your app targets a lower iOS version other than iOS 14 react-native can't render the image. This PR aims to add notes about this limitation because there is no mention of this anywhere on React Native docs.
Repro
https://github.com/putrautama007/image-issue-ios-rn

And probably we should also mention that animated WebP is not currently supported.
Reference PR
https://github.com/facebook/react-native/issues/31654


Duplicate log entry lmao

New ID query from <IP_ADDRESS>:56748
Queried locations for new id
Queried locations for new ID

Actually wait this is a common theme
Object add request from <IP_ADDRESS>:56756
Requested to add object 2 to locations
Succeeded to update database: locations.json
Successfully added object 2 to locations

The ones in the second one were all intentional actually but I could make them do that less 🙃


Bee stack - raw prompt is undefined (local, ollama)

Summary
When using the Bee stack locally (podman compose), together with ollama running locally, with the default Bee Assistant, the raw prompt is reported as undefined
[ note: see below for sw config ]
Details
For example:

Note: If the same operation is done on an internal deployment, the bee assistant is able to report raw prompt
Both the default assistant, and a new, custom, agent produce the same result.
Version
git HEAD is at 71101f9e3915b261f442f4f942238533772f4e37, with the two additional commits from https://github.com/i-am-bee/bee-stack/pull/10 applied. Further the bee-api is at e7ced3f1ce87e8cd621d218531ef8ee961149d39 (compose yml updated to use a locally build container with this) since this was needed to allow use  local llama
Additional Info
It's possible that the issue relates to use of ollama locally.
not noticed anything obvious in container logs - anything particular to check?

This is more for bee-agent-framework or bee-observe, I believe we had an internal discussion about it already and made a conclusion that it does not make sense to include raw prompt for all providers because for some it's an implementation detail hidden from the ollama user. The interface with the ollama models is on the level of "chat messages" @Tomas2D @GALLLASMILAN
Maybe we should update the UI for models with unsupported raw prompts.

The raw prompt is available only for BAM and WatsonX. Other providers do not expose it (the rendering to the appropriate template is done on their site).
However, you have a list of all messages, and that should be enough.


How to steps restyle

The current how-to-steps bullet point has some styling that looks off. I was asked to fix it; make things align better, primarily the bullet point number and the heading!
I was given the following guideline!

Fix
Test URLs:
BLOG:

Before: https://main--express-website--adobe.hlx.page/express/learn/blog/cool-pfp-for-instagram
After: https://how-to-steps-restyle--express-website--adobe.hlx.page/express/learn/blog/cool-pfp-for-instagram

MAIN:

Before: https://main--express-website--adobe.hlx.page/express/create/video/intro/wedding
After: https://fix-how-to-step--express-website--adobe.hlx.page/express/create/video/intro/wedding
Before: https://main--express-website--adobe.hlx.page/express/create/logo/fortnite
After: https://fix-how-to-step--express-website--adobe.hlx.page/express/create/logo/fortnite
Before: https://main--express-website--adobe.hlx.page/express/feature/design/add-icons
After: https://fix-how-to-step--express-website--adobe.hlx.page/express/feature/design/add-icons
Before: https://main--express-website--adobe.hlx.page/express/discover/ideas/halloween-party
After: https://fix-how-to-step--express-website--adobe.hlx.page/express/discover/ideas/halloween-party

@MarquiseRosier closing because there has been no more activity on this task and it's probably not relevant anymore


Build instructions in README produces panic'ing binary

Sponsors SeaweedFS via Patreon https://www.patreon.com/seaweedfs
Describe the bug
README#Installation-guide Step 3 contains command:
go get github.com/chrislusf/seaweedfs/weed
It builds weed here without compiler errors, but weed binary panicing same as in #1058 :
panic: /debug/requests is already registered. You may have two independent copies of golang.org/x/net/trace in your binary, trying to maintain separate state...
goroutine 1 [running]:
go.etcd.io/etcd/vendor/golang.org/x/net/trace.init.0()
    /mnt/src/user/projects/pkgbuild/seaweedfs-git/src/src/go.etcd.io/etcd/vendor/golang.org/x/net/trace/trace.go:123 +0x1cd

BUT, if weed is built via git clone ... + make (without go get ...), everything is ok.
System Setup

/usr/bin/weed server -dir=/home/user/weedfs -volume.max=10 -volume.port=8080 -volume.publicUrl=<IP_ADDRESS>:9000
OS version: ArchLinux, fresh
output of weed version:

version 30GB 1.44 linux amd64
seaweedfs-git r2620.7de121d9-1

Additional context
I'm maintainer of seaweedfs-git PKGBUILD in Archlinux AUR for last 4 years.
I've not checked build last ~4 months, but before this time everything in PKGBUILD was worked without issues and strongly according to README's build instructions. Today, it switched to make due to panic in weed.

It's a known problem due to go module. I do not know a good way to resolve this yet.

rm -rf $GOPATH/src/go.etcd.io/etcd/vendor/golang.org/x/net/trace helped me!

fixed in https://github.com/chrislusf/seaweedfs/issues/1548
changed readme


Preferences Sync doesn't work with keybindings from PC to Mac.

VSCode Version: 1.44.0-insider
OS Version: Win10 & Mac OS Catalina

Steps to Reproduce:

Add custom key bindings to keybindings.json
Use the new Preferences sync feature to sync
Note that the keybindings to no 'roam' between OS

It seems that the keybindings written to the roam directory on Windows are not synchronised with the keybindings.json on the Mac.

Does this issue occur when all extensions are disabled?: Yes

By default keybindings are synced per platform. Syncing keybindings across platforms is advanced and users have to opt in using setting


AMX Matmul Scheduling, but with C++ changes removed

Seems like you need to pull from master?

@yamaguchi1024 ack sorry about that. I merged master in now.

@skeqiqevian Also, could you remove the unrelated code as much as possible? (e.g, changes to tests/amx/amx_experiment.c  seems not related to C++->C change?)

@skeqiqevian Also, could you remove the unrelated code as much as possible? (e.g, changes to tests/amx/amx_experiment.c  seems not related to C++->C change?)

NVM

@alexreinking
This PR removes all the concerns you had about the other PR, except the change to LoopIR_compiler.py where we introduced _int thing ( https://github.com/ChezJrk/SYS_ATL/blob/master/SYS_ATL/LoopIR_compiler.py#L679 ).
Let me try to explain why we still need that change even though we are using clang now. Btw, we need to use clang-13 because that is the only compiler that supports the AMX instructions currently.
AMX instructions such as _tile_zero take only integer constants. So the following code works in C++, but not in C, because const is not a compile-time constant in C.
const int tile = 2;
_tile_zero(tile);

Thus, we decided to use C macro, so that it becomes
#define tile 2
_tile_zero(tile);
#undef tile

We want to treat AMX tile as a buffer like everything else, but we don't want to index into it (using .data or _data would give tile[0] which doesn't make any sense here and it's certainly not a constant). That's why we need to have a codegen path that compiles a buffer without indexing.


Unable to locate publishable resources.

I followed all the instructions in the setup guide
Environment details:

PHP version: 7.4.9
Laravel version: ^7.0

When try to install sentry version 1.8 i got the following exception:
Problem 1
    - Installation request for guzzlehttp/guzzle (locked at 7.0.1) -> satisfiable by guzzlehttp/guzzle[7.0.1].
    - sentry/sentry-laravel 1.8.0 requires sentry/sdk ^2.1 -> satisfiable by sentry/sdk[2.1.0].
    - sentry/sentry-laravel 1.x-dev requires sentry/sdk ^2.1 -> satisfiable by sentry/sdk[2.1.0].
    - Conclusion: don't install sentry/sdk 2.1.0
    - Installation request for sentry/sentry-laravel ^1.8 -> satisfiable by sentry/sentry-laravel[1.8.0, 1.x-dev].

I used version 1.7 and try to publish config file like the docs says. I got
php artisan vendor:publish --provider="Sentry\Laravel\ServiceProvider"

Unable to locate publishable resources.
Publishing complete.

Sentry (Laravel) is currently not compatible with Guzzle 7 out of the box.
See here for updates and or workarounds: getsentry/sentry-php#1059
If you couldn't install 1.8 you should also not be able to install 1.7 I believe, not a 100% on that but that might be the cause why you cant publish the config file.

We've just fixed the issue with Guzzle 7 too, so everything should work as expected.
Please re-open / open a new issue if the issue persists.


[FLINK-18250] Enrich OOM error messages with more details in ClusterEntrypoint

Similar to what we added for TM in #11408,
this PR adds more information and hints about out-of-memory failures in ClusterEntrypoint.
It also factors out TM/JM related exception handling out of general ExceptionUtils.

@flinkbot run azure

Thanks for the review @tillrohrmann
The test failure looks unrelated, filed another issue. My Azure CI passes. Therefore, merging this.

merged into master by 276e8c66e877fcf891b3f665d68433552aa66de0
merged into 1.11 by 95d4f1b5951fff05b70597a64893074a15aaea91


question: product types from rest api

Hello, quick question. How are product types and such imported from the Rest API?
I've tried to import them from the RestResources object, as that's the only types being exported, but they do not seem to be useable, unless I'm missing something.
example:
import { RestResources } from "@shopify/shopify-api/rest/admin/2024-01";
import { shopify } from './utils/shopify' // setup of shopify API obj

function test(prod: RestResources["Product"] | null): void {
  console.log("hello world");
}

(async () => {
  // act like session & productId is defined
  const prod = await shopify.rest.Product.find({
    session: session,
    id: productId
  })

  test(prod);
})

This would result in the following error, as the types do not match:
- error TS2345: Argument of type 'Product' is not assignable to parameter of type 'typeof Product'.
  Type 'Product' is missing the following properties from type 'typeof Product': prototype, apiVersion, hasOne, hasMany, and 16 more.

Seems like we could technically use this:
type ShopifyProduct = Awaited<ReturnType<typeof shopify["rest"]["Product"]["find"]>>

But that just seems dumb & overcomplicated in my opinion.


soc: nordic: Minor updates for nRF54H20

needs rebase

@57300 please remember to rebase

@57300 . To keep up with the tradition, this is your weekly "please remember to rebase this PR" reminder.

I'm setting this PR as dnm until it is rebased, so it stops being in the short merge list.

Sorry @aescolar , I wasn't aware this was making noise. Rebased now.


Fix activation queue rate and enforce only activate upon finality

(ethereum/eth2.0-specs#1514)

https://github.com/ChainSafe/lodestar/pull/633

@ColinSchwarz It's already resolved in #633


Version 4.0.0

Version 4.0.0

Upgrade to ViewComponent 3.0.0rc1

Upgrade to [govuk-frontend 4.5.0](https://github.com/alphagov/govuk-frontend/releases/tag/v4.5.0

Improve the table component

Add support for table header scopes #382
Table header scopes are used to identify cells that are covered by a header cell.
They can be manually added using the scope parameter but will default to col for
table headings in the <thead> and to row for table headings in the <tbody>.

Add support for table footers #387
Table footers allow us to summarise a table's contents. Often they are used to
show numeric totals.

Add support for <colgroup> and <col> elements #393
Column groups define groups of columns within a table. They are useful when single
categories of data need to span multiple columns

There are examples on the new functionality on the table page of the guide

Add Summary Cards #351
Summary cards provide visual separation when displaying more than one summary list
on a page. They can also contain links to actions that affect the card as a whole.
There are examples on the new functionality on the summary list page of the guide

Upgrade to the latest versions of Ruby and Rails.

The supported Ruby versions are: 3.0.5, 3.1.3 and 3.2.1
The supported Rails versions are: 7.0.4 and 6.1.7

Upgrade guide
The latest version of ViewComponent changed the syntax for calling slot setter methods.
In versions prior to version 3.0.0, you could call a slot by its name. So for a PostComponent that
renders_many :comments, we could add comments by calling comment:
<%= render(PostComment.new(**post_args)) do |post| %>
  <% post.comment(**comment_1_args) %>
  <% post.comment(**comment_2_args) %>
  <% post.comment(**comment_3_args) %>
<% end %>

Since version 3.0.0 we need to prefix the slot call with with_, so in the above example we'd need to
do this:
<%= render(PostComment.new(**post_args)) do |post| %>
  <% post.with_comment(**comment_1_args) %>
  <% post.with_comment(**comment_2_args) %>
  <% post.with_comment(**comment_3_args) %>
<% end %>

Final things to do

[ ] complete the migration of to ViewComponent 3.0.0. Currently it's at rc1 but the release should be immanent
[x] write some release notes and an upgrade guide
[x] bump to latest versions of ruby/rails #400
[x] upgrade to govuk-frontend 4.5.0

There are some things that didn't make the cut for v4. They may reappear later:

Drop the classes: keyword. I wanted to get rid of it so we can treat all HTML attributes equally and consistently but so many people use classes it would be a chore to go through and change everything #398
Add new_tab: param to govuk_link_to. This would add the required attributes to make a tab open in a new window and added the '(opens in new tab)' text. It was pulled (#399) because adding via a keyword arg interferes with Rails' argument structure in a way that means other params (options, html_options) need to be wrapped in curly braces so Ruby treats them as a hash rather than a kwarg #363
I still want to introduce this functionality and plan on creating a separate module of link helpers that take a modern approach of govuk_link_to("link text", "link_href", new_tab: true, no_underline: true). This will probably land in 4.1 and be default but the old behaviour will be available via a config option.

Waiting on View Component 3.0.0 to be released before shipping v4 of GOV.UK Components. Looks like it's nearly there 🚀


feat: Fixes StatefulSet

Fixes StatefulSet so that the arguments (explicitly defined on extraArgs - values.yaml) can be correctly passed to manager by specifying the log level, etc.

Hi @wolf31o2, PTAL

Hi @verbotenj, if you have any permissions, then please merge this.


Specs

I've started adding specs. How's this looking so far?

@jgarber623 How's this spec setup work for you? I'm writing specs and force-pushing as I go. I'd like to get these specs done then get FrancisCMS to support Rails 5.

Awesome! Thanks for the PR, @shanecav84! 💪
Apologies for the delay in posting here. Taking a look now commit-by-commit and will post comments/questions as a review.

Also, it may be worthwhile adding spec/dummy/**/* to .rubocop.yml:
AllCops:
  Exclude:
    - 'db/**/*'
    - 'spec/dummy/**/*'
    - 'vendor/**/*'

Also also: Adding some details on how to run the test suite to README.md (in the "Testing" section) would be super helpful, too.


Update to Chefdk 0.5.0

This PR is here to update the ChefDK to version 0.5.0.
Currently 0.5.0-rc2 is out. Once the final version is out and everything works ok we can merge it.

ChefDK 0.5.0-rc2 build is succesful and all tests passing. I'm basically just waiting for ChefDK 0.5.0 final to come out.

Let's merge it to push out a 3.0-rc version of bills kitchen...


Need README!

Make sure to suggest installation with require-dev
Should cover:

[x] Installation
[x] Intro, including folder paths
[x] Basic usage of each command
[x] Make command, and excludes
[x] Config file

First version up, needs to be re-read/checked

Missing InterfaceParents from config section


follow symlinks when finding openstack creds

A user might symlink dirs (from keybasefs or dropbox) into
~/.password-store. Make find follow symlinks so that these dirs will
still work.

Love this pull request!


Remove return from example addAction

Documentation for editor.AddAction gives a return type of void | Promise<void>. Having the example return null is confusing given it doesn't really matter what it returns.

Thank you!


Use K.backend() instead of K._BACKEND

According to the Keras Documentation, K.backend() is the 'publicly accessible method'.

Do you know since when this is available? We might have to change the minimum required version of Keras if we merge this.

K.backend was added by
https://github.com/fchollet/keras/commit/99f564e9724d3456fc5b48b9dcd6254c83611242
and is available since Keras 1.0.7 so this PR won't affect the minimum required version of Keras (>=1.0.7) at this time.

Great, thanks for checking!


stringi data packet moved to new url?

          Thank you for the reply. I now managed to generate a .zip file directly (from the dplyr package), on my non-work PC (not restricted like the UiO PC) where I have RTools 4.3 (and R 4.3) installed. Unfortunately I did not check if it worked before installing RTools.

But it's not dplyr I'm trying to install. For the docstring package, I now get the following error message:
trying URL 'http://www.ibspan.waw.pl/~gagolews/stringi/icudt61l.zip' Error in utils::download.file(icudt61l, file.path(folder, "icudt61l.zip"),  :    cannot open URL 'http://www.ibspan.waw.pl/~gagolews/stringi/icudt61l.zip' In addition: Warning messages: 1: In utils::download.file(icudt61l, file.path(folder, "icudt61l.zip"),  :   downloaded length 0 != reported length 0 2: In utils::download.file(icudt61l, file.path(folder, "icudt61l.zip"),  :   cannot open URL 'https://www.ibspan.waw.pl//~gagolews/stringi/icudt61l.zip': HTTP status was '403 Forbidden'
No zip file gets generated.
It appears to be an issue with the stringi package, which returns some hits on Google as well. But this is way beyond my skill level to deal with!
Originally posted by @pedbra in https://github.com/LCBC-UiO/tsdrtools/issues/3#issuecomment-1591878191

@pedbra I am closing in a solution for this fairly soon, I think.
But in case you didnt know, TSD also have provided a little workaround for this package:
https://www.uio.no/english/services/it/research/sensitive-data/help/software/r/using-R.html#toc6


Added a Framework

I added the F3 Framework to the list. There should not be any conflicts.

Duplicate of https://github.com/ziadoz/awesome-php/pull/705.

https://github.com/ziadoz/awesome-php/issues/393


User wants to have string type

What?
User wants to have basic string support.

yes.

Good question. Never thought about it.
If int is a pointer to utf-8 bytes it make sense.


[Feature Request] Allow users to connect pr_agent to an existing SageMaker inference endpoint

Context: assume a user has e.g. a pre-configured LLM inference endpoint in SageMaker (for example, a self-hosted Llama model as described here). It would be nice to be able to allow the user to configure pr-agent to leverage that endpoint e.g. by means of a dedicated AI handler.
Discord chat: https://discord.com/channels/1057273017547378788/1057273018084237344/1197261978591309884
cc: @krrishdholakia

@krrishdholakia do you think this request is feasible ?

Hey @mattiaciollaro @mrT23 we already support sagemaker - https://docs.litellm.ai/docs/providers/aws_sagemaker
What am i missing?

@mattiaciollaro is this PR still relevant?

Sorry for the delay guys.

Hey @mattiaciollaro @mrT23 we already support sagemaker - https://docs.litellm.ai/docs/providers/aws_sagemaker

https://docs.litellm.ai/docs/providers/aws_sagemaker seems to support SageMaker JumpStart models specifically.
I am thinking of a different situation where a model is already deployed via SageMaker and a reference to the inference endpoint name is available (as in here). In that case, how can we instruct pr-agent to leverage the LLM behind that pre-existing endpoint? I am not sure I see a way of doing this via https://docs.litellm.ai/docs/providers/aws_sagemaker
In the context of a POC with my team, the way we accomplished this was to hack the pr-agent's default AI handler (which is the LiteLLM AI handler) and use the sagemaker SDK (specifically, the HF predictor to make requests to the pre-existing SageMaker endpoint.
I imagine a cleaner solution would be to implement a dedicated AI handler for this usecase?

@mattiaciollaro is this PR still relevant?

I don't have a PR out for this, but yes: I think the feature request is still relevant :) My apologies again for the delay!


Merge to live...

Something wrong with this merge. Master & live seems to be diverging even though they both have same changes.

The PR was likely rebased instead of merged.

Reset live to master and force-push it

I tried to salvage it with a merge at the time, but yeah... that will just cause merge commits forevermore

I merged live to master and created PR to merge master to live to settle the matter.


Use @wdio/logger. Update jest dependencies

Implements #49
This uses @wdio/logger for the already commented out console statements.
While doing this, I updated the jest dependencies, due a problem with a dependency of @wdio/logger that jest had.

LGTM 👍


docker-machine create with amazonec2 driver doesn't properly clean up on failure

When trying to launch a Docker Swarm on Amazon, some of the instances will fail to come up. Docker properly detects this and gives me the full error output (usually rate limiting or issues with retrieving a gpg key, but that doesn't matter). However, it doesn't try to clean up after itself, and thus prevents me from retrying my failed command without first going onto AWS and cleaning up the keypair that docker-machine create left behind.
 If the instance comes up properly, it will be possible to remove it by running docker-machine rm swarm-agent-01, which will terminate the instance and remove the keypair. However, if the instance for some reason fails to deploy, it will show up in docker-machine ls with a status indicating failure, and terminating it with docker-machine rm swarm-agent-01 will not remove the keypair.
When the keypair isn't removed, trying to run the command above will fail with an error message stating "There is already a keypair with the name swarm-agent-01.  Please either remove that keypair or use a different machine name.".
It would be nice if the driver would clean up after itself on failures, or at least offer to do so with a command-line argument? If the maintainers of the amazonec2 driver would accept a PR for this, I would gladly do it.

For what it's worth, this is the GPG error I'm seeing on certain launches
Executing: gpg --ignore-time-conflict --no-options --no-default-keyring --homedir /tmp/tmp.xGMTCSmiIA --no-auto-check-trustdb --trust-model always --keyring /etc/apt/trusted.gpg --primary-keyring /etc/apt/trusted.gpg --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys 58118E89F3A912897C070ADBF76221572C52609D
+ sudo -E sh -c apt-key adv --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys 58118E89F3A912897C070ADBF76221572C52609D
gpg: requesting key 2C52609D from hkp server p80.pool.sks-keyservers.net
gpg: keyserver timed out
gpg: keyserver receive failed: keyserver error

Maybe this is a separate bug?

@Kuzmin
It looks to me like docker/docker#13555.

@christianbundy
Right, yeah it looks like my comment is docker/docker#13555
I still think the things in the opening post is a separate issue

Totally, sorry -- I should've been more clear. The original issue is absolutely a problem (that I've dealt with again and again).

Yeah, we should clean up after ourselves, the fundamental Machine model doesn't lend itself to it particularly well unfortunately.  I guess it's basically a huge bug if docker-machine rm exits 0 without having properly cleaned up or using -f.

@nathanleclaire Is docker-machine rm really running after a failed docker-machine create? I only read amazonec2.go and couldn't find that behaviour there, but it makes sense if it's higher up.
If so, yeah, it doesn't seem to do the right thing when there is no instance but a leftover KeyPair. It seems like it short circuits on failure to terminate instance: https://github.com/docker/machine/blob/8141874120abdc5c9056b6dbeb09bfb76a3eb290/drivers/amazonec2/amazonec2.go#L750
Maybe a more proper behavior would be to add an existence check to Remove and if it doesn't exist, just remove the KeyPair? Could this have some other unforeseen effects that I haven't thought about? I'm willing to submit a PR for this.

Is docker-machine rm really running after a failed docker-machine create? I only read amazonec2.go and couldn't find that behaviour there, but it makes sense if it's higher up.

If create fails we don't try rm automatically (that causes all sorts of issues).  I think this issue is related to some kind of condition where maybe terminate call fails, but the instance is still removed, thereby causing future terminate calls to also return an error and ensuring that the SSH key block never gets hit.  Most likely we should do both and then return a bundled error.

I'm working on a fix to at least try the removal of both and then bundle up the errors into one instead of bailing if we hit an error for terminate


Jupyter notebooks' plots persist when their cells are deleted.

The following behaviour have been exhibited in VS Code Insiders version 1.57.0 running in both Linux (Ubuntu 20.04, Lubuntu 20.04) and Windows 10.
When I create a plot and delete the cell, then the output does not get deleted as it should but persists in the background permanently until I close the notebook.

@Adam-Antios please update to the latest Insiders, which should have fixed this issue.


On MacOS Sierra - binaries hard-terminated on launch

Note that this is happens building on go <1.8.1 (verified working on 1.8.1), and is actually an xcode, macOS bug. But there is a workaround as described here: https://github.com/golang/go/issues/19734 and https://github.com/shirou/gopsutil/issues/339
Since our tooling currently building on 1.7.3 we should probably bear this in mind.
Output Error Sample
rake test
0 linting issues found in ./pkg
0 linting issues found in ./cmd
vet found 0 issues in ./pkg
vet found 0 issues in ./cmd
signal: killed
FAIL    github.com/DataDog/datadog-agent/pkg/aggregator 0.005s

Additional environment details (Operating System, Cloud provider, etc):
MacOS Sierra 10.12.4
Steps to reproduce the issue:

Build with 1.7.3 on OSX
Try to run the agent, or tests, benchmarking, or whatever (really).
Watch binary crash with: signal: killed

Describe the results you received:
See step 3 and output above.
Describe the results you expected:
Normal execution.
Additional information you deem important (e.g. issue happens only occasionally):
Most likely this is the following: https://github.com/golang/go/issues/19734

Closing as we should just use go 1.8.1 or later, the README was updated accordingly


Rework flushing

Current flushing behaviour is unresponsive, which can be seen for example when subscribing, which requires some time to start getting messages.

This has been implemented and will be part of v0.3 release


Module parse failed: Unexpected token (473:30)

Bug Report
Clean install of last version of docz. When I run docz dev, I got:
./node_modules/docz/dist/index.m.js 473:30
Module parse failed: Unexpected token (473:30)
You may need an appropriate loader to handle this file type.
| async function loadFromImports(path) {
|     // tslint:disable-next-line
>     const { imports } = await import('~imports');
|     const { default: Component$$1, getInitialData } = await imports[path]();
|     const ExportedComponent = props => (createElement(AsyncComponent, Object.assign({}, props, { as: Component$$1 || 'div', getInitialData: getInitialData })));
 @ ./node_modules/docz-theme-default/dist/index.m.js 39:0-72 243:12-16 374:35-39 466:34-45 467:8-12 511:25-29 552:50-54 640:16-27 643:41-45 942:20-24 1024:28-39 1098:16-27 1490:33-44 1601:16-27 1872:16-27 1935:29-40 2029:45-56 2040:25-36 2041:8-18 2073:16-21
 @ ./.docz/app/root.jsx
 @ ./.docz/app/index.jsx
 @ multi ./node_modules/react-dev-utils/webpackHotDevClient.js<EMAIL_ADDRESS>./.docz/app/index.jsx

I can't share a project link.
I check webpack aliases with:
modifyBundlerConfig : (config) => {
    console.log(config.resolve.alias);
    return config;
}),

and ~imports is present.
Thanks in advance

If someone is stuck on this, and following @dfee's hints, this seems to fix it:
npm i -D<EMAIL_ADDRESS>
Thanks

+1 here.
mentionned trick works but as it's on our CI that this fails it really hurts to put this hack as a step in the build :(

when will v0.14 be released?

hi @dfee and everyone here with webpack issues
turns out webpack 4.29 has had some wacky npm dependency issues i dont fully understand. just saw this in an unrelated project i'm helping out on as well: https://github.com/nozzle/react-static/issues/1027#issuecomment-465695830
its a mess. use yarn as much as possible.

We are facing this issue :(

Same here

For the record. Same here.
For the record: I actually don't have Webpack installed in my project.

@bichotll I think they mean you need to update the version inside node_modules/docz-core. I've been using this cd node_modules/docz-core && npm i -D<EMAIL_ADDRESS>&& cd .. && cd ..

@mattietea oh! thanks for the heads up! 👍

npm i -D<EMAIL_ADDRESS>is ok. But it can be confusing for the novice. Look forward to releasing 0.14 soon.

Thanks a lot for your help. It looks like everything works with Yarn, and the problem appears only with NPM and the WebPack issue. I have an empty project for testing with only a package.json and using the instructions from @mattietea, I have managed to make docz work, but now I have an error in the browser. I think I will use Yarn until the<EMAIL_ADDRESS>update.

@timbergus I had the same issue, it gets solve if you lock<EMAIL_ADDRESS>I opened an issue around this:
https://github.com/bublejs/buble/issues/189

@pedronauck seems that the 1.0.0-rc.3 still have this problem :(

The<EMAIL_ADDRESS>trick doesn't work for me.
Just minutes ago I had the same error trying to compile with webpack and babel - I was simply missing the css configuration in webpack.config.js:
{
    test: /\.css$/,
    use: [{ loader: 'style-loader' }, { loader: 'css-loader' }]
}

Could this be related? Can we adjust the webpack config used internally?

Docz is using<EMAIL_ADDRESS>this bug keeps showing with this version?

Just got the same problem on a fresh installation. For me, the issue got solved by creating the project again with Yarn. npm installation failed with the error similar to what @ohardy posted.
Reproduction of an error:
npm init -y
npm install -D docz@next docz-theme-default@next
# Add scripts to the package.json from the documentation.
npm run docz:dev

Reproduction of a success:
npm init -y
yarn add docz@next docz-theme-default@next --dev
# Add scripts to the package.json from the documentation.
yarn run docz:dev

Full error:
 ERROR  Failed to compile with 2 errors                                                                       1:39:50 PM

 error  in ./node_modules/docz/dist/index.esm.js

Module parse failed: Unexpected token (15:38)
You may need an appropriate loader to handle this file type.
| import { MDXProvider } from '@mdx-js/tag';
|
> const BasePlayground = loadable(() => import('./chunk.esm2.js'));
| const Playground = props => typeof window !== 'undefined' ? (createElement(Suspense, { fallback: null },
|     createElement(BasePlayground, Object.assign({}, props)))) : null;

 @ ./.docz/app/root.jsx 9:0-59 16:2-15 18:19-23 25:25-31
 @ ./.docz/app/index.jsx
 @ multi ./node_modules/react-dev-utils/webpackHotDevClient.js ./.docz/app/index.jsx

 error  in ./node_modules/docz-theme-default/dist/index.esm.js

Module parse failed: Unexpected token (245:30)
You may need an appropriate loader to handle this file type.
| `;
|
> const CodeMirror = lazy(() => import('./chunk.esm2.js'));
| const getLanguage = (children) => {
|     const defaultLanguage = 'jsx';

 @ ./.docz/app/root.jsx 11:0-39 17:29-34
 @ ./.docz/app/index.jsx
 @ multi ./node_modules/react-dev-utils/webpackHotDevClient.js ./.docz/app/index.jsx

Versions:

node 11.6.0
npm 6.5.0
yarn 1.12.1

Hope this might help in some way.

@pedronauck to answer your question, I'm still having this issue on rc7 with webpack 4.29.6. Npm support is sadly a must for my organization, otherwise the workaround above does work.

Closed in favor of #704, the answer is there.


Addition of "Heart" parameter similar to existing "Pulse"

Some manufacturer use "heart" for sending pulse instead of "PULSE". This is exactly similar to issue - https://github.com/tananaev/traccar/issues/2291#issuecomment-301909949. If I understand correctly, the only addition that needs to be done is to accept the parameter as "heart" - rest of the things should work as is.
I have attached the official documentation from manufacturer.
---- from the attached document -----
32. Terminal heart uploaded
Terminal upload:
[3G *<PHONE_NUMBER> * len * heart, x]
X is the heart rate data per minute.
Example: [3G *<PHONE_NUMBER> * len * heart, 100]
The heart rate of the representative terminal is 100 times / minute
heart_ communication protocol.docx

Please provide a HEX message sample from Traccar logs.

Couple of sample HEX message from Traccar logs:
5b33472a363030353431323930322a303030372a68656172742c305d
5b33472a363030353431323930322a303030382a68656172742c37315d

I have added support for it.

That was too fast ! - Thank you! - when is the next official release? I am assuming this will be part of next official release.

Yes, it will be included in the next official release. It will probably be in a month or two.

Thank you!


not working on IE

First, Nice work done here.
I'm using it successfully with chrome and ff, but with IE it returns the file with wrong filename, not the file name setted in the class with [ExcelDocument("File name")], also the file type/extension is not recognized when prompted to save the file.
The filename returned comes with something as the name like,
=utf-8_B_W1BvcnRhbCBDSVJKXS1bNS8xMS8yMDE0XS1bRXhwb3J0XS1bTGlzdGEgZGUgQcOnw7Vlc10ueGxzeA===
Thanks in advance.
By the way, my controller method content:
var header = new MediaTypeWithQualityHeaderValue("application/vnd.ms-excel");
Request.Headers.Accept.Clear();
Request.Headers.Accept.Add(header);

var result = GetListOfObjects();//something similiar returning List<Object>
return Ok(result);

Hey @notvitor! Sorry I took so long to get back. I know this is old now, but I wanted to at least give you something.
I think the reason the file name won't be coming back is because you're clearing the HTTP request headers. Here's how I'm setting the file name for the returned Excel document:
headers.ContentDisposition = new ContentDispositionHeaderValue("inline")
                                             { FileName = fileName };

…where headers is the HttpContentHeaders value passed in to the formatter's SetDefaultContentHeaders method. My suggestion is to either let the package set the headers for you, or to take a look at that method and at the default headers used to make sure nothing's missing. :)
If you're still having problems—or I've missed something obvious, which is absolutely possible—please get back to me! Could you also grab a copy of the headers that are being sent with the file, and what version of Office (or equivalent) you have installed?

Haven't heard back on this for a while, so I'm going to close it for now. Feel free to reopen it if you have any more input, though!


PP-5061 Migration SQL to backfill English-language service names

Database migration to insert a row with the English-language service name into the service_names table if there is not already one present for the service.
This should ensure that every English-language service name is duplicated in the service_names table.
This won’t lock the table so it’s possible that other service name changes made while this is running may result in inconsistencies and there’s a tiny chance a service trying to update their service
name while this is running may encounter an error. However, the chances are very small (service names are not changed very often) and we can easily check for inconsistencies afterwards and fix any
that are found (which at the very worst may just mean old service names continue to be used for longer than anticipated).

We already have code that updates the English-language service name in both places when one is updated. It seems to be working (there’s one discrepancy in production but it’s clearly a test service).
Just to be certain, we’ll check that there aren’t any discrepancies before making major changes like only reading from the service_names table and stopping updating the name in the services table.


Document passing an array to exec creates

This is probably not what most people would expect...

Can one of the admins verify this patch?

Hopefully #9168 can be changed instead and the documentation updated to match that.

Thank you @yakatz I think it makes sense to merge this as-is, since it documents the current behavior. And we can investigate your proposal for changing the behavior separately. I'm going to close and reopen this PR so that the GH checks run (we've updated them since this PR was opened)


need some guidance

present we selected this as our final year project if you provide the document it will be more helpful to us....will you please provide the document with uml diagrams...

bro there is no animation png file could you please provide it's really helpful to me if you provide it

Do u have the uml diagrams of this project??if u have means please forward
it to me I will send u the animations file
On Mon, Mar 25, 2024, 2:09 AM MaramPavaniStudent @.***>
wrote:

bro there is no animation png file could you please provide it's really
helpful to me if you provide it
—
Reply to this email directly, view it on GitHub
https://github.com/jigargajjar55/Audio-Speech-To-Sign-Language-Converter/issues/5#issuecomment-2016936417,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCLY3FRLKISQE2FYJQPD7UTYZ42ZRAVCNFSM6AAAAABEO5OI22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJWHEZTMNBRG4
.
You are receiving this because you authored the thread.Message ID:
<jigargajjar55/Audio-Speech-To-Sign-Language-Converter/issues/5/2016936417
@github.com>

@vinoothnav are you able to run the  project , if yes please help us.

it is not indian sign language


Error: Cannot build player while editor is importing assets or compiling scripts

Error pops when I tried to click on 'Build' button. Googled online and led me to URL below, and it points me to something related to 'Using UnityEditor'.....
https://forum.unity.com/threads/i-cant-build-my-game.803667/
Context to the error....
Besides Porcupine I'm dealing with, I'm integrating SCORM. Integrating SCORM as a standalone, separate from Porcupine doesn't pop the error. But when done together with Porcupine the error below pops.
https://github.com/rstals/Unity-SCORM-Integration-Kit

are you trying to build for web?

are you trying to build for web?

To provide some context. It is SuccessFactor LMS that the developed e-contents will be loaded and the e-contents viewed using mobile (Chrome / Safari) - build for web?

What I am trying to say is that I don't think Porcupine's Unity binding supports browsers. Check the doc.

What I am trying to say is that I don't think Porcupine's Unity binding supports browsers. Check the doc.

Sad...shall look at alternatives! TQVM!


Force username prompt

When sharing a board with a lot of collaborators, you end up with a lot of "unknown user".
It might be nice to have an option to force prompt username. For example, if username is not set already and you open the link, display a popup that prompt name and greyd out the board.
Even if the popup is not mandatory and you can discard it with a click, most user will blindly fill it and it would give a smoothier experience for everyone.
I'm thinking of two implementations :

add a button next to the share link, that enable prompt username for the board.
make the prompt a default behaviour when you create a collaborative board. (could be toggle on/off server side for selfhosted instances)

I agree the unknown user is not a good experience and most of the time it happens unknowingly for fresh users because the name is not filled.
Showing a prompt might affect the smooth experience which we have right now (just open the link and start collaborating) so instead, if we can focus/highlight the collab button and show some kind of indicator, maybe a tooltip that informs that "username is not filled and will be shown as the unknown user to collaborators" so it can catch the attention of the user as well and doesn't interfere with the flow. Thoughts?

Another idea, generate username on the fly like GoogleDoc do with their "anonymous_animal" names. Make it visible to the user that their name is "anonymous_panda" so they can change it by clicking on it.

I like the idea of generating random usernames on the fly. That way flow would also remain smoother.

Can I take this up? :)

@h7y I have a module for generating this. We may wanna create an npm package for it.
Do we want to put this in core? My implem for this has ~4KB (just the generator part) and I need to use this in the host app as well. So if we do it in core, we must export the API so host apps don't need to bundle it 2x.
But ultimately, the collaborator list shouldn't really live in core either. /cc @ad1992

@dwelle yes let's have the npm package for name generator and use it here.


Do you have a translation file for traditional Chinese characters?

Hi.
Great gem, thanks.
I see you have a zh.yml for Chinese.  But as you may be aware, there're 2 versions of Chinese: traditional Chinese and simplified Chinese.
Simplified Chinese is used in China.
Traditional Chinese is used in Hong Kong and Taiwain.
Do you have a traditional Chinese file?
Thanks
Zack

Dear @zackxu1 ,
my apologize for contacting you this way but I wasn't able to find a email address or something else.
I assume you are a native "speaker" of Traditional Chinese, right?
I'm looking for a translation of a 100 words text into Traditional Chinese in the context of a backup software named Back In Time. The text is addressed to the users and tries to attract them contributing to the translation of the software they use them self.
Can you help us out here?


feat: block non github domain and drawer enhancement

Maybe we choose optional permission for enterprise GitHub instead of matching all route.

Sure, now we're using chrome extension's context menu to control enabling/disabling a-tree in specific domain, but we still need to set matches to all_urls


Add new Ctrl+F shortcut

Corresponds to https://github.com/FreeTubeApp/FreeTube/pull/4970. Would like to wait until just before the release to merge this. Or we can add a development branch for this repo.

See comment:

Would like to wait until just before the release to merge this. Or we can add a development branch for this repo.


FIXED THE SOCIAL HANDLE ISSUE

Related Issue
None
Fixes:  #2984
Description
just was a dark mode switching issue
ISSUE #2984
Type of PR

[x] Bug fix

Screenshots / videos (if applicable)
before

after

Checklist:

[x] I have made this change from my own.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

Since it's minor issue I can't assign level 2 work on other issue I will assign level based on difficult @HS202022

Try working on responsiveness of pages when viewed on mobile screen @HS202022


ERROR (ThrottledFolderBind): Exception calling "Bind" with "3" argument(s): "The request failed. The remote server returned an error: (401) Unauthorized.

Trying to use Merge-MailboxFolder.ps1. OAUTH has been configured as per the guide. I'm logging into the browser, obtaining a token and receiving the error as per title:
ERROR (ThrottledFolderBind): Exception calling "Bind" with "3" argument(s): "The request failed. The remote server returned an error: (401) Unauthorized."
Failed to open source message store<EMAIL_ADDRESS>During the token obtain process does the admin account used need to have a license, or delegated permission to the source mailbox? My impression was that the oauth app permission was used, so delegation is not needed.

If you are logging in as an account other than the mailbox owner, then you'd need FullAccess permissions on the mailbox to be able to access it.  Alternatively, set up application permissions and impersonation.  Note that administrators do not have rights to users' mailboxes by default.


Postcommit support for timeseries

Add support for postcommit hooks. This is a streamlined version of the KV postcommit hooks.
Also provides a new batch put interface designed for batches of data that are already msgpack-encoded.
Required for basho/riak_repl#738 (RIAK-1546) (RIAK-1823)

Single-value puts are currently broken in these PRs, am working on a fix.

Bugs corrected in most recent commit. Ready for review.

:+1: 8f6449e

👍 0751235

@borshop merge

@borshop merge


Ordered and unordered lists

I expected to being able to write
+ buy milk
+ read book

Like on markdown, and have them transform to a beautiful list, but it doesn't work (I tried with +, * and -). I don't know if it's not possible, or I haven't found how :P.
Thanks for the great app, I'm loving it

Raven doesn't support markdown right now (it's on the roadmap, though). I'm not sure lists make much sense in a text editor aimed at writing novels though.

Yeah, I know... I understand.... but they could be useful when creating dialogues, don't you think? :)


Svg images are not suport for old browsers

Svg images are not shown in UC browser for Android

UC has a market share of 1.8% and also has many privacy and security issues. You can convert the svg images to png for your usecase. Otherwise just don't use such a browser and I personally wouldn't worry about it at all.


Vertical & horizontal centering options in grid?

Hello, I was playing with carbon components, found there's no centering options in this package.
Like creating login page, there's a need for centering options in grid.
From carbon design guidelines, we can see some images like below:

Are there ways to achieve this? Simply adding parent div and adding flexbox doesn't work at all.

Generally, I would recommend writing custom CSS for vertical/horizontal centering.
If you're using the Grid from Carbon, you can use the offset prop to "center" a column.
Carbon Components Svelte forces a 16 column grid. In the example below, we have a column that spans 4 columns and has a left offset of 6 columns.
6 + 4 + 6 = 16
<Grid>
  <Row>
    <Column xlg={{ span: 4, offset: 6 }}>Column</Column>
  </Row>
</Grid>


Add CLI reference page

We should use Slate and create a good CLI reference page for each and every command (with examples that 100% work).

This is done. CLI reference is currently in the Markdown format - https://github.com/redhat-developer/odo/blob/master/docs/cli-reference.md


Stream type host_forwarding

@jsingle For SSH servers and fun stuff. Still WIP, needs tests and things. Fixes #589.

Example using this https://github.com/gamechanger/dusty-example-specs/pull/8/files

Aww no new integration tests :(

I think it looks good, reminder to squash commits before merging


Fix missing beam compile

This addresses issue #22.

Just a heads up that there's a maintained fork of this repo at : https://github.com/wk8/erlang_iso8601 :)

The original project has a new maintainer and a new home:

https://github.com/erlsci/iso8601

Old links to the original repo will redirect to the new home -- both in the browser and via the cli.
Note, however, that this work has already been merged.
Closing ...


'SWRConfig' is not exported from '@wundergraph/swr'

Description
I followed the tutorial below to get a basic Wundergraph app working with NextJS:
https://docs.wundergraph.com/docs/getting-started/nextjs-quickstart
It works fine and I'm able to run the server locally too. However when trying to build the application, I'm getting the below error:
 ✓ Linting and checking validity of types
Failed to compile.

<EMAIL_ADDRESS>Attempted import error: 'SWRConfig' is not exported from '@wundergraph/swr' (imported as 'SWRConfig').

Import trace for requested module:
<EMAIL_ADDRESS>./components/generated/nextjs.ts

I searched on Git and Google but wasn't able to find anything that could help me. I'm on the latest version of the Wundergraph SDK and Wundergraph Next package.
Bram
Reproduction
Follow this tutorial:
https://docs.wundergraph.com/docs/getting-started/nextjs-quickstart

Hello @brampurnot, thanks for your report.
We're not able to reproduce this, the following sequence of commands produces no error on our end:
npx create-wundergraph-app my-project --example nextjs
cd my-project
npm run build:wundergraph
npm run check
npm run build:next

Additionally, we can manually see that @wundergraph/swr does export SWRConfig:
$ grep SWRConfig<EMAIL_ADDRESS>
import { SWRConfiguration, SWRResponse, Key } from 'swr';
export { SWRConfig, useSWRConfig } from 'swr';
type UseQueryOptions<Data, Error, Input extends object | undefined, OperationName extends string, LiveQuery> = Omit<SWRConfiguration<Data, Error>, 'fetcher'> & WithInput<Input, {
interface UseUserOptions<User> extends FetchUserRequestOptions, SWRConfiguration<User> {

Could you please share the package.json that's being generated on your end? That might give us more information.

Thanks! I actually updated the wundergraph packages and now it seems to work! Not sure what happened.


Merging from main as we move to fix the webui

Pulling in changes from main
Lots of linting fixes - nothing was ever run
Adding storyboard

This PR is HUGE – please can we limit PRs to just 1 thing at a time?

the problem is that linting, compilation, etc was never run on the repo - so there are tons and tons of basic changes from that perspective.
the functionality of this repo is just to focus on those things - getting the project into a moderately decent shape moving forward. That said, I did also add storyboard - i'll split that into its own PR if you'd like.

the problem is that linting, compilation, etc was never run on the repo - so there are tons and tons of basic changes from that perspective.
the functionality of this repo is just to focus on those things - getting the project into a moderately decent shape moving forward. That said, I did also add storyboard - i'll split that into its own PR if you'd like.

the problem is that linting, compilation, etc was never run on the repo - so there are tons and tons of basic changes from that perspective.
the functionality of this repo is just to focus on those things - getting the project into a moderately decent shape moving forward. That said, I did also add storyboard - i'll split that into its own PR if you'd like.


[HELP] How can I add an external CA to the OpenSearch truststore?

Describe the bug
Hi!
I'm currently working with min.io for s3 storage and that is done using a non-trusted CA.
What I want to do is push snapshots to the min.io using repository-s3.
I've read through the documentation and can't figure out how to do this.
I've tried multiple ways:

adding it to the keystore
adding it as a property: plugins.security.ssl.http.pemtrustedcas_filepath
adding it using keytool

Nothing gives me the desired result.
Is there anyone here who has done this before or can point me in the right direction?
I'm using the Kubernetes Operator and am on version 1.3.16.
Related component
Plugins
To Reproduce

Setup OpenSearch to register snapshots
Use min.io with a self-signed cert
Try to trust the CA backing that certificate

Expected behavior
To have a clear and easy way to trust self-signed CAs
Additional Details
Plugins
repository-s3
Host/Environment (please complete the following information):

Kubernetes, Kubernetes Operator
Version 1.3.16

[Triage - attendees 1 2 3]
@NandoTheessen Thanks for filing, moving this into the security plugin repo


Investigate Wagtail 1.12 RichText Features

Wagtail 1.12 will introduce Rich Text Features (see wagtail/wagtail#3736), a way to configure the features (format, heading, etc) of a rich-text editor per field. The underlying goal is to the editor config (which is kinda global, i.e. config for the whole Draftail editor) and the features of a given field (i.e. allowing different formatting options for the intro and body blocks). Eventually, features will also be hooked with whitelisting and text conversion logic (the ultimate goal of having all rich text editor saving content in a uniform way to avoid editor lock-in (I can't find the discussion around that anymore sorry)).
I'm not sure whether it's too early to start implementing such thing, but we should at least think of it as it might shuffle quite a few things.

Looking at the Draftail docs, most of the editor configuration could be handled with this "enabled / disabled" features list. The only field that's not on/off would be the list nesting threshold, but I think WagtailDraftail can have a "safe default" of 9 as we suggested in the past.
To leverage this feature, we will need an additional abstraction layer that is missing at the moment. I think that's very welcome anyway – most people shouldn't have to know about draftjs_exporter, or draftail. This abstraction layer may not hold up for people who want to create their own rich text features, but that seems fine.

Work started in #63.


Pypi in documentation

Between using the Docker image, and working with the source, is there a story for the README about pulling this from pypi? Pushing to pypi is mentioned at the end, but it's not clear to me what depends on pull this.

Docs now say "pip install clodius" ... hopefully that is what I meant?


[FR] Conda channels support

That would be great to be able to indicate specific Conda channels when using convert. I can imagine that as both config and CLI option (which takes list as an argument and keep the order).

I have just written a tool that does this, but not with dephell: https://github.com/dojeda/poetry2conda


Reroll with Hero Point covers damage buttons

When rerolling an attack roll by right clicking the chat entry, the reroll covers up the damage and critical buttons. This makes it inconvenient to make those related rolls.

This is fixed in 4.4.0.

I wonder if the guy hosting just didn't update. Sorry for the unnecessary report. Couldn't find it when I searched the issues.


Register typescript definitions in package.json

Fixes #37. Fixes #63

That was fast. Thanks @kenjdavidson!

Github makes it easy.  I can't publish to NPM until I finish work though.

That's OK, I will use direct link to github for today in my package.json

If you've been using this actively, have you run into issue #59 ?
I'm curious how many people it's affecting, and whether I need to rush to fix it?

Hm, I'm not facing the issue, at least for the moment. I managed to subscribe both to bluetooth enabled & disabled events successfully


246 feature use symlinks instead of switching globallocal install directory

PR details
What is the purpose of this pull request? (put an "X" next to an item)
closes #246

pls have a look & deploy afterwards


Swift 2.0 support

I had to refactor a bunch of the code, to use with my project in Swift 2.0. You should make this compliant with that standard.

+1

@penne12 @nishkrishnan Planning to release an updated version tomorrow with Swift 2 and Swift Package Manager support. Sorry for the delay!

Yay! Thanks! Cocoapod support, too, please?

Version 1.0.0 has been tagged with Swift 2 support. No Swift Package Manager support because it doesn't work with iOS yet.
@penne12 There is already a podspec in the repo.

@indragiek awesomeness! Fastest I've ever seen a bug be fixed :grinning:. Didn't notice the pod spec, sorry.


Implement partial blocks

This allows for failover for missing partials as well as limited templating ability through the {{> @partial-block }} partial special case.
Partial fix for #1018

@mmun @wycats could I get your input on this? Trying to carve off pieces from #1018 that can go out in a more standalone fashion vs. a big bang meta feature.
My biggest questions here are:

Does the feature generally make sense in isolation?
Is the naming of @partial-block sensible?
Do we need a separate AST node for this or should we merge the two and use the existence of program as the differentiator in behavior? This is effectively the way that it's implemented now but it has all of the extra overhead of handling the distinct AST type.

Documentation PR here: https://github.com/wycats/handlebars-site/pull/120

Ember doesn't support {{> }} constructs so I don't have a lot of experience.

Roughly, yes.
It's ok, but I think a better solution is to introduce a {{yield}} + an isolated "partial" AKA components.
In this case, having a single AST node seems better if you can get away with it. (With mustache/block you can't because mustaches have an escaped field).

I'm not sure what you are suggesting. Could you provide a code example?
I was hoping that this could be done, but strip behavior was sufficiently different and there is also something to be said for introducing new types so non-supporting code doesn't have surprises, i.e. the program could be completely ignored, silently, vs. a thrown error.

Since the whitespace constraints are different I don't have anything useful to add. Is @partial-block implemented via "data" or is it effectively a keyword?
FWIW, I am still interested in the original named blocks proposal.

In ember you can write something like
{{#each-with-ul people as |p|}}
  Hello {{p.name}}
{{/each-with-ul}}

where the each-with-ul component is defined in a separate file as

  {{#each items as |item|}}

{{yield item}}

  {{/each}}

This generates the output

Hello Kevin

Hello Yehuda

I assume Thorax has something similar. The win here is that the scope of the second template above is completely encapsulated so its easier to reason about and easier to reuse. @partial-block is similar to yield, but with the partial context "leaking" behaviour.
Aligning Handlebars with {{yield}} would also give named blocks a better chance of landing, as they don't make much sense if you can't yield to them.

I'm having trouble working through your comments about encapsulation and leaking contexts or how this is different from the helper block-based ember example above. Can you help me understand that?
Regarding named blocks, the decorators branch implements a generic way to apply metadata to blocks and wrap in behaviors as well as implements the inline partials behavior that is the end goal. This PR needs to land before can do too much discussion on that, unfortunately.

Forget it, the docs give an example that removes my concerns.
{{#each children as |child|}}
  {{#> childEntry}}
    {{child.value}}
  {{/childEntry}}
{{/each}}

This works as expected, even if your your childEntry partial was something like
{{#with whatever as |child|}}
  {{> %block}}
{{/with}}

Sounds like we're mostly in agreement then. Leaving the token name as @partial-block and merging as is to unblock the decorators PR and the release in general.

Released in 4.0.0


FAI-10607 - Add run_mode config to support WebhookGaps mode

Description
Check for  WebhookGaps mode to select streams before records read.
TODO:

[ ] Add test case

Type of change

[ ] Bug fix
[ ] New feature
[ ] Breaking change

Related issues

Fix #1

Migration notes

Describe migration notes if any

Extra info

Add any additional information

Looking good! Let's add the test before merging...


Using @ on comments fails script execution on AE (Windows)

I'm using this library along with typescript. Thanks for this library by the way (:
But I can't use // @ts-ignore on the code that goes to AE. It looks like the @ makes the script fail on AE javascript engine.
// @ts-ignore

A workaround is to use minify = true. Doing this the error goes away.

please checkout for typescript  :

Nice! Thank you.


Add /system/info endpoint for faas-nomad

The purpose of this change is to make faas-nomad compatible with the
latest version of the API Gateway in OpenFaaS. Each provider should
be able to advertise the underlying provider and the version of the
code. A new version of faas-provider (0.6) is vendored into the repo
which brings in an optional health endpoint (turned off by default).
The version is being passed to the provider from main.go.
A subsequent change should implement the health endpoint.
A unit test was added for the info endpoint using the format found
for the other endpoints.
The advertised information in /system/info will be consumed by the
API gateway and logged for help debugging issues. It can also be
used to turn on/off UI features such as the "network" field which
does not make sense for all providers.
Signed-off-by: Alex Ellis (VMware<EMAIL_ADDRESS>Other comments
Nic this was tested via unit tests which I think prove the behavior works as expected, but not through bringing up a nomad environment. Please could you give it a run-through before or after merging?

go build and go test ./handlers works fine but it looks like there is an issue in the CI pipeline.
goreleaser: error: unknown short flag '-s', try --help
(^ That is likely unrelated to the PR.)

Bump

Sorry will check this tomorrow

Awesome, thank you Nic!


Consider integrating first-timers-bot for easy first-timers issues

Instead of fixing simple issues that are not high priority ourselves we could open them up the community, even for people who are new to Github etc.
https://github.com/hoodiehq/first-timers-bot

@36degrees is this something realistic we can do? If not I'd like to close this issue.

This has been successful, but I'm glad for the suggestion to try this manually, since I think we know now that one template cannot fit all issues.
So rather I think we should continue doing this but build up a library of templates specific for certain situations.
https://github.com/alphagov/govuk-frontend/pull/965


Build and Deploy Your SAP Fiori App to SAP Cloud Platform

Tutorial: https://developers.sap.com/tutorials/appstudio-fioriapps-mta-build-deploy.html
Step 4: Access the application on SAP Cloud Platform, Cloud Foundry
Sub step 4: cf html5-list -di FioriDemo-destination-service -u
We run the above command to get the details like url but the command is not working. Throwing error as "Too much arguments". Its only happening in AP region. EU and US worked fine.

Labels :
Landscape Labels: trail3
SCPTest-eboard: Develop an SAP Fiori App Using SAP Business Application Studio
Priority label :‘High-prio’
Best Regards,
Shivakumar P

Hi @shivakumar-p-sap ,
Where the issue happens (AP trial) - can you run in a BAS terminal the 'cf plugins' command and share the version for the HTML5 plugins?

Regards - Raz

Hi @shivakumar-p-sap ,
Where the issue happens (AP trial) - can you run in a BAS terminal the 'cf plugins' command and share the version for the HTML5 plugins?

Regards - Raz

Hi Raz,
The version is 1.4.3 in AP region. Please find the below screenshot.

Regards
Shivakumar P

Hi Raz,
The version is 1.4.3 in AP region. Please find the below screenshot.

Regards
Shivakumar P

Hi @shivakumar-p-sap ,
This seems to be the root-cause.
I'm working with Engineering on solving this.
Regards - Raz

Hi @shivakumar-p-sap ,
This seems to be the root-cause.
I'm working with Engineering on solving this.
Regards - Raz

Hi @shivakumar-p-sap ,
I was notified that the issue was solved.
Can you check / test?
Regards - Raz

Hi @shivakumar-p-sap ,
I was notified that the issue was solved.
Can you check / test?
Regards - Raz

Hi Raz,
It's working, so closing the issue. Thanks
Regards - Shivakumar P

Hi Raz,
It's working, so closing the issue. Thanks
Regards - Shivakumar P


Design image/color selection for wizards

tricky one!

Great, we should do a session tomorrow for unify this with the actual controls.
this looks great!

I don't like the arrows being centered. It breaks the vertical rhythm when scanning it vertically.

Yep, I worried about it to, could work better aligned to right ; )

Fixed

also added new symbols to document .../map-editor-1.2

yay!


NSView entry point

Is it possible to create NSView entry point which can do both Metal or GL on appropriate platform?
In examples, only NSWindow drawing can be found. I wish it would be possible to embed bgfx-backed engines into AppKit storyboards/xibs etc.

Needed this today too, this is the first result on Google for NSView bgfx.


[stable/joomla] Fix wrongly referenced ingress annotations

Is this a new chart
no
What this PR does / why we need it:
This chart contains a bug wrongly referencing the ingress annotations. Therefore the annotations on an ingress cannot be configured.
Which issue this PR fixes
none
Special notes for your reviewer:
This bug was introduced in a previous commit where a range block was moved.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test
/assign
/lgtm


unable to activate the extension.

您好，我的vscode版本为1.32.3，nodejs版本为10.15.3，使用插件时仍然有command 'leetcode.xxx' not found的错误提示。

Is there any logs in the console if you open Help > Toggle Developer Tools

What we be shown if you click the leetcode icon on the left activity bar?
Besides, is there any output message if you press Ctrl+Shift+U and toggle the channel from Tasks or 任务 to LeetCode?

@LucaPractice Just feel free to comment on the root cause and how you solve the problem. This information could help other users who also encounter the same problem.
Thanks.

I meet the same problem. The day before yesterday I could use the extension normally, today it fails. There is a "There is no data provider registered that can provide view data." on the sidebar and there is "leetcode.xxx not found"
. My environment is ubuntu 18.04lts + vscode 1.32.3 + node 10.15.3 .
Here is my log:

@jdneo Thanks!

@deadoggy I guess the root cause is that the installed files are corrupted. Try to uninstall the extension and remove it. You should be able to find it under $HOME/.vscode/extensions. Remove the folder shengchen.vscode-leetcode-* and reinstall it.
Please let me know if this could help solve your problem.

I uninstalled the extension and removed the folder under ~/.vscode/extensions/, but the problem still exists. Here is the console screenshot:

It's almost the same as the prior one.

Ah, Have no ideas so far.
But I noticed that there is some other strange output in the console:

Not sure if it is something wrong with your environment.
@Vigilans Do you have any ideas?

Found the problem.

@deadoggy The line 6 is require("./LeetCodeStatusBarItem"), the first letter to be upper case L.
However, in the published extension, there is

the first letter to be lower case l.
Ubuntu file name is case-sensitive, resulting in the NotFound problem.

@jdneo btw, I checked the folder structure in the extension folder and found another thing strange:

So there are two problems with the published extension:

wrong letter case of out/src/statusbar/leetcodeStatusBarItem.js
redundant file out/src/leetcodeStatusBarItem.js

I made a low-level mistake. I installed the extension first and then installed nodejs. I found that I can't use it and I reinstalled the extension. I did not restart the vscode after uninstalling the extension, so I can't use it after reinstalling the extension.

@Vigilans
It can be work when I use upper case L in out/src/statusbar/leetcodeStatusBarItem.js:6:33
Thanks!

@jdneo  @Vigilans It works after changing 'l' to 'L'. Thanks a lot!

@All
@Vigilans and I are working on release a hotfix 0.13.3 for this issue.

0.13.3 released. Please feel free to let us know if you still meet the problem.
Thanks.


Like should ignore casing

Currently, casing is taken into account for the where = Like filter.
{
  Get {
    Things {
     	Article(
      where: {
        valueString: "*Money*"
        operator: Like
        path: ["title"]
      }

Ideally, casing should not play are role.

I just verified this with two different cases. This depends on the property type (which ultimately boils down to how elastic treats them).
Property type string -> Casing is kept in the index, you also need specify the correct casing on your wildcard query
Property type text -> Everything is lowercased at index time, you can specify all lowercase in your query
This makes sense to me, as string is only meant for really exact matches like an ID or hash (where a lowercase letter could be a completely different result from an uppercase). Whereas text is meant for  human-readable text where Apple and apple is most likely the same word.

Check, thanks.


hub pull-request has trouble understanding editor variable

My ~/.gitconfig has the following in it (among many other things):
[core]
    editor = vim -u ~/.vimrc-mutt +startinsert

However, this makes hub pull-request fail with the following message:
error using text editor for pull request message

It works just fine when I change it to:
[core]
    editor = vim

Thanks for letting us know! We'll look into it. In the meantime you could try setting GIT_EDITOR=vim hub pull-request whenever you're making a PR, so you don't have to constantly change your original ~/.gitconfig.

Hi, I can't reproduce this. Can you confirm that you're using latest release hub 2.2.1? If not, please update.
Also, which platform are you on?

I was able to confirm this is no longer a problem with 2.2.1, must have been a problem with 1.12.4.

Thank you!

I just ran into this issue, this is my .gitconfig:
[core]
  editor = $(which vim)

And I'm getting the same error message when running git pull-request: error using text editor for pull request message
git commit opens vim just fine.
$ hub version
git version 2.6.4
hub version 2.2.2

I got this error on Windows 10.
> hub version
git version 1.9.5.msysgit.0
hub version 2.2.3

But I found I was missing the core.editor setting. So I ran git config --global core.editor vim and was good to go.

Thanks @dougludlow

@scottbasgaard :thumbsup:

I'm getting problems with hub.protocol=https core.editor=gvim --remote-wait
hub pull-request doesn't seem happy about that core.editor setting.


null values are not valid as string parameters

This was reported by @seif.
In short, if a component has a string parameter, and we make a POST request specifying null as value (which is a valid json type) the validation fails as the registry considers that not valid as string (typeof(null)!=='string').
Let's remember that in JS JSON.stringify({a: undefined}) => {} and JSON.stringify({a: null}) => {a:null}.
How to solve it?

We can add a new type of parameter which is stringOrNull - this means parameter is null and then is forwarded to the component as null - current behaviour for string wouldn't change.
We can be less strict and consider null a valid value for string by leaving it as null
We can be less strict and more opinionated and auto-sanitise a null parameter to '' in case of string and 0 in case of number
We can be hacky and leave everything as is on the registry and do a sanitisation on the node.js client in order to avoid this to happen on the registry level
Other ideas?

I personally like 2 but I like 1 too.

@matteofigus I prefer 2, providing null/undefined is the same as not providing any value at all from my perspective (component author saying that that param is not mandatory).

Sorry but I couldn't resist:
https://twitter.com/iamelgringo/status/845027370864001024
// cc @matteofigus @seif

+1 for 2 @matteofigus @seif

Option 2 sounds good to me.

I think we got a winner. I'll try to do this later 👍


How to read the sync camera labels?

Hello, can you please upload a new pip package supporting the latest dataset with sync camera labels?

I also tried to compile by myself, but meet the error:

you can use  std::min(1.0, std::max(  gt_dot_pd / gt_range / pd_range , 0.0))    to replace std::clamp()

you can use std::min(1.0, std::max( gt_dot_pd / gt_range / pd_range , 0.0))  to replace std::clamp()

Thank you, but I meet other errors after I fix this one.
I think it is not the way to fix the problem one by one. It should be caused by the different compilation environment.

Please install the latest pip package (pip version 1.4.5), and see if that fixes this issue.

please change c++ version to 17


Value of cornerRadius changed

Buttons now flatter, more aesthetically pleasing

Can you show two pictures showing the difference?

Before and after pictures as requested


Repeat view caching implementation

@EisenbergEffect Please take a look if this implementation of view caching in the repeater looks alright.

@martingust It looks pretty good to me. Seems cleaner than before. I took a very quick look...and you are the expert on the repeater now :) How do you feel. Is it ready to merge in?

@EisenbergEffect Great! Yes, go ahead and merge this in.

Ok, I will take care of it soon. Thanks for working on this.


Flywheel

Make flywheel in CAD
and calculate the inertia tensor

need to decide on

[ ] total weight
[ ] diameter
[ ] inner diameter (to fit the motor directly)


Update Azure Deploy Button

Summary
The focus of these changes is to update to the new Azure Deploy button image.

/check-enforcer override


Devices connected to same room URL don't 'see each other'.

This happens frequently for me. I'm in the USA testing multiple devices (Mac, Windows, iPhone, Android) all via Chrome on my wifi.
Essentially it appears to each device connected that they are the only one in the room.
Suggestions for how to troubleshoot are welcome.

Hi @jonathanhinterberger , thanks for the report, it's similar to the one reported by earlier.   Been doing inter contnentals tests ,  Sweden , England,  Turkey and USA ( Arizona ) recently with a successfull result.  But the error you reported is urgen to solve, so i'm on that right now.
I have a few more questions in this matter that i would likte you to answer.

Does the error / bug occur when randomly, lets say you start a room and then connect  more devices?  Does it drop connect and drop after a while?

2.As you are on wifi, its really strange if it drops..  Do you the poster ( no camera image ) , that poster should be visible if a proper peerconnection is established.

Its hard for us to arrange tests, the error reported cannot be reprocuced, but i guess thats can be related to my setup of devices.  So can we arrange a time/date for us fo do a test.

When you run into this error , you can press ctrl+uto generate a log that is very useful for me, please do so, if possible.
Again, thanks for your valuable feedback Jonathan.

Hi @MagnusThor. Here is what I've found so far:

I haven't had issues with dropped connections, but I have yet not tested for longevity.
The issue I'm talking about here is what appears to be no peer connection.
Happy to arrange a time if I can help. I am in US EST (right now it's 11:07am my time.)

I've run some tests today and come across a few small bugs which I'll open other issues for, but haven't reproduced the issue we've been discussing here again yet.
When I press ctrl+u does that generate a log on your server with no alert on my device? That's fine I suppose, I simply don't know if if is being created.

Great! @jonathanhinterberger  thanks for giving kollokvium some time..
I have been doing some changes, to the RTCPeerConnections  (thor-io.client)  that i do belive will improve the connectivity as well as the media streams the is lost sometimes.  this is now deployed for test , both at dev and https://kollokvium.se
The problem with "Devices connected to same room URL don't 'see each other' "  is new,  do you mean that they dont seem so appear in the same room ?  Interesting thou..
Regarding ctrl+u the application should fire  back 1-n files for download ( to your client ) , nothing is sent back to the server (or me)  , make sure you have the window focues when pressing ctrl-u ,  just click somewhere in browser .
I'm this week a bit off and busy, but maybe we can try to talk on kollkvium some day?
regards
Magnus

@jonathanhinterberger can we elaborate on this?

Closing this issue now,  can reopnen if needed.

Closing this issue now,  can reopnen if needed.


Fix aqua stale deps test

The aqua stale deps test is failing and needs fixed

Closed by https://github.com/CliMA/ClimaAtmos.jl/pull/1758


RFC: Unsealed table assignment creates an optional property

Add an RFC for assigning to an unsigned table.

For earlier discussion, see #67.


Remove 'to' from already_invited/already_in_team error message

https://github.com/outsideris/slack-invite-automation/blob/master/routes/index.js#L38

@jlegrone Did you mean it should be "visit XXX", not "visit to XXX"?

Yes, exactly!
On Mon, Mar 28, 2016, 4:41 AM JeongHoon Byun (aka Outsider) <
<EMAIL_ADDRESS>wrote:

@jlegrone https://github.com/jlegrone Did you mean it should be "visit
XXX", not "visit to XXX"?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/outsideris/slack-invite-automation/issues/33#issuecomment-202351226


asset: give README a proper doc link

when one becomes available.

@pongad, is this still relevant?

I think this one was fixed in #3985


space.map_bond doesn't like kwargs

I'm trying to run an NPT simulation for which I need to pass kwargs to the energy_fn, but it seems like space.map_bond cannot vmap over the kwargs. Here is the error I get
Traceback (most recent call last):
  File "/ocean/projects/cts180021p/anneveli/.conda/envs/jaxMD/lib/python3.8/site-packages/jax/_src/api.py", line 1501, in _mapped_axis_size
    size, = mapped_axis_sizes
ValueError: too many values to unpack (expected 1)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "tests/pressure/minimum_pressure.py", line 55, in <module>
    E.append(energy_fn(pos, nbrs, box=box))
  File "/ocean/projects/cts180021p/anneveli/.conda/envs/jaxMD/lib/python3.8/site-packages/jax_md/energy.py", line 1444, in energy_fn
    dr = space.map_bond(metric)(R[neighbor.idx[0]], R[neighbor.idx[1]],**kwargs) #might need to revise kwargs
  File "/ocean/projects/cts180021p/anneveli/.conda/envs/jaxMD/lib/python3.8/site-packages/jax/_src/traceback_util.py", line 162, in reraise_with_filtered_traceback
    return fun(*args, **kwargs)
  File "/ocean/projects/cts180021p/anneveli/.conda/envs/jaxMD/lib/python3.8/site-packages/jax/_src/api.py", line 1470, in vmap_f
    _mapped_axis_size(in_tree, args_flat, in_axes_flat, "vmap",
  File "/ocean/projects/cts180021p/anneveli/.conda/envs/jaxMD/lib/python3.8/site-packages/jax/_src/api.py", line 1514, in _mapped_axis_size
    assert treedef_is_leaf(leaf)
jax._src.traceback_util.UnfilteredStackTrace: AssertionError

The stack trace below excludes JAX-internal frames.
The preceding is the original exception that occurred, unmodified.

--------------------

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "tests/pressure/minimum_pressure.py", line 55, in <module>
    E.append(energy_fn(pos, nbrs, box=box))
  File "/ocean/projects/cts180021p/anneveli/.conda/envs/jaxMD/lib/python3.8/site-packages/jax_md/energy.py", line 1444, in energy_fn
    dr = space.map_bond(metric)(R[neighbor.idx[0]], R[neighbor.idx[1]],**kwargs) #might need to revise kwargs
AssertionError

The simulation I am trying to run uses the eam_neighbor_list potential and the periodic_general space. I think it may be something more general in the periodic_general space that is preventing map_bond from vectorizing over the inputs.
My solution was to use a partial operator to bake in the kwargs before the map_bond function.
metricp = partial(metric, **kwargs)
dr = space.map_bond(metricp)(R[neighbor.idx[0]], R[neighbor.idx[1]])#,**kwargs) #might need to revise kwargs

I tried to make a minimum working example, but the potential I'm using is in a format that the standard jaxmd couldn't read so I had to make some changes to load_lammps_eam_parameters
I added the following lines to accept potentials with 6 columns
if len(re.split(' +',raw_text[6].strip()))>1:
    data = [maybe_downcast([float(i) for i in re.split(' +',rt.strip())]) for rt in raw_text[6:]]
    data = jnp.concatenate(data)
  else:
    data = maybe_downcast([float(i)  for i in raw_text[6:-1]])

Here is the minimum working example
minimum_pressure.py.zip
and here is the potential
2016--Nichol-A-Ackland-G-J--Li-eam.txt

Thanks for raising this! Sorry for the delay in reply, I was traveling and lost track of issues. I think your analysis was spot on and the way we usually solve this is, exactly as you suggested, to partial evaluate the metric before sending into map_bond. e13f48fcf6a18a957ff7d8d46c5664f180cbfcf3 includes this fix for the EAM potentials.
It would be possible to have the map_bond, map_product, and map_neighbor functions automatically not vmap over keyword arguments (essentially folding the partial evaluation into those functions). I haven't been inclined to do that up to this point because it wasn't clear that this behavior was always what one would want, but maybe it would be better. What do you think?
Finally, is that 6 column format common for EAM parameters? I added your fix to our EAM loader for convenience.
Let me know if you come across anything else or if you have any suggestions and thanks for the report!

Thanks, Sam! This is great.
For the keyword arguments, to me it doesn't make a difference whether it's a separate partial evaluation or wrapped into the functions. I don't know enough to say if there is a performance boost one way or the other. I may look a little cleaner if the partial evaluation would be wrapped into the map functions. I can't see a reason why it wouldn't be the expected behavior to have the kwargs of the metric function evaluated with the metric function. Is it possible that one of the kwargs changes over the vmap?


Import error.

Reinstalled several times, but Xcode constantly gives import error. It is impossible to import "TensorFlow", but for some reason sometimes "CTensorFlow" is suggested by Xcode suggestions.

Also, in some cases playground just goes in infinite compilation, or works very very slow. What this could be caused by and how may I solve this problems?
Thanks in advance.
Xcode 9.4
macOS 10.13.5

Code completion not suggesting "TensorFlow" is a known bug. However, import should still work.
In your case, could you try restarting Xcode?

I have downloaded latest toolchain yesterday:

I've already restarted Xcode and even Mac several times, but nothing happened.
I've tried importing TF in REPL and just command line and got same error, but I haven't done any setup before.

No, that doesn't fails, but in Xcode still doesn't work..

@eldardamirov Can you give me the latest download link? I can not find .

https://github.com/tensorflow/swift/blob/master/Installation.md

@zhuiyizhiqiu Please select a package from the "pre-built packages" section.

In the future, please create a new issue if you have a specific question instead of commenting on an existing issue.

@eldardamirov
Has Xcode 9.4 ever worked for you, or not even once?
I'll download Xcode 9.4 tonight and see if I can replicate your errors.
It's known that the latest 06-01 toolchain doesn't work with the new Xcode 10 beta.
Playgrounds fail with the error: "LLDB RPC server has crashed". We're looking into this also: it's possible this will be fixed when we merge changes from upstream.

@dan-zheng  Thanks.

I verified that the 06-01 toolchain is problematic on Xcode 9.4.
The reason appears to be codesigning-related, from the Console log.
(It could also be fix by merging upstream changes for lldb.)
I'll try to fix this within a week.

Sorry, I've been busy this week, so couldn't take part in discussion. Yes, Xcode 9.4 has never worked for me, same thing with Xcode 10, but it is only first beta, so it has many issues beside this one. Looking forward to fixes)

Tried on 9.4 and 10.0 Can't get tensorflow to work

I apologize, I didn't get around to fixing codesigning this week.
Fixing the build for newer Xcode versions is my next priority after finishing my current tasks (automatic differentiation infrastructure). Currently, I believe our toolchains work only with Xcode 9.3 (and under).
A realistic ETA for working toolchains for Xcode 9.4/10 is 1.5-2 weeks from today. Thanks for your patience!

Closing, since issue is not reproducible.

The latest 6/29 prebuilt toolchain supports Xcode 10:


Using  JdbcSagaStore or JpaSagaStore in a non-spring application

We want to save our Sagas in persistent store for recovery purposes,
So obviously we need to use a SagaStore.
All the examples we've seen of creations of SagaStores were deeply relying on Spring's auto configuration and dependency injection.
Is the any documentation available for manually creating a saga store and connecting it to a database?

I'm unclear about the environment you're running in, but the Saga Infrastructure section does contain information on overriding the Saga Store in a Spring Boot Autoconfiguration scenario:
https://docs.axoniq.io/reference-guide/configuring-infrastructure-components/event-processing/saga-infrastructure
@Saga(sagaStore = "mySagaStore")
public class MySaga {...}
...
// somewhere in configuration
@Bean
public SagaStore mySagaStore() {
    return new MongoSagaStore(...); // default is JpaSagaStore
}

If you're not using that environment, it also contains a standalone implementation. Axon Framework also has a CDI module (https://github.com/AxonFramework/cdi) if that's what you're looking for.

The thing is that we're not running in a Spring application, and all the documentation we see relies on Axos running in a Spring application.

The code examples in the reference guide has a tab that allows you to switch between spring boot auto configuration and manual configuration. Here's the equivalent manual configuration:
Configurer configurer = DefaultConfigurer.defaultConfiguration();
        configurer.eventProcessing(eventProcessingConfigurer -> eventProcessingConfigurer
                .registerSaga(MySaga.class,
                              // Axon defaults to an in-memory SagaStore,
                              // defining another is recommended
                              sagaConfigurer -> sagaConfigurer.configureSagaStore(c -> new JpaSagaStore(...))));

// alternatively, it is possible to register a single SagaStore for all Saga types:
configurer.registerComponent(SagaStore.class, c -> new JpaSagaStore(...));

If that's insufficient, I've found great community support on the Axon Framework forums: https://groups.google.com/forum/#!forum/axonframework

@wleroux thanks for sharing that bit, as that's indeed the way to go.
@gioragutt sorry for the late reply, somehow this issue feel between the cracks I guess.
Wayne's suggestion is a fair thing to look at - the reference guide should be clear on how to achieve what your asking. If it is not, then we would much value a issue on the guide's issue tracker.
A part from that though, what you should be looking at is configuring your saga through the SagaConfigurer, which contains the configureSagaStore(Function<Configuration, SagaStore<? super T>>) method. This would require you to provide a lambda where the input is the overall Axon configuration and the output should be the SagaStore you want to use.
If this is still somewhat unclear to you, feel free to ask follow up questions regarding this.
Lastly, I will be closing this issue under the type Question. Like Wayne pointed out, the Axon Framework user group, or Stack Overflow with the tag axon, are the usual spots to place questions regarding the framework and server.


Annoying ladder rotation lock

Let us disable the ladder locking animation. I can see my whole body if I look backwards

When you climb ladders using this mod, your body is forced to face the ladder, please allow us to disable this. It's similar to not enough animations


Rbac with keycloak provider.provider keycloak still not working

Issue submitter TODO list

[X] I've searched for an already existing issues here
[X] I'm running a supported version of the application which is listed here and the feature is not present there

Is your proposal related to a problem?
Rbac with keycloak provider.
Describe the feature you're interested in
Rbac with keycloak provider
Describe alternatives you've considered
No response
Version you're running
0.7.1
Additional context
No response

I found some useful for auth with keycloak. we can authticate with keycloak and use rbac with oauth .

auth:
type: OAUTH2
oauth2:
client:
keycloak:
clientId: xxx
clientSecret: yyy
scope: openid
issuer-uri: https://<keycloak_instance>/auth/realms/
user-name-attribute: preferred_username
client-name: keycloak
provider: keycloak
custom-params:
type: keycloak
this custom parms to change
custom-params:
type: oauth
roles-field: roles
for rbac
rbac:
roles:
- name: "kafka-admin"
clusters:
- local-kafka
subjects:
- provider: oauth
type: user
value: "user1"
- provider: oauth
type: user
value: "user2"
permissions:
- resource: topic
value: ".*"
actions:
- VIEW
- MESSAGES_READ
    - resource: consumer
      value: ".*"
      actions: [ view ]

    - resource: schema
      value: ".*"
      actions: [ view ]

    - resource: connect
      value: ".*"
      actions: [ view ]

    - resource: acl
      value: ".*"
      actions: [ view ]

This can work only for specific user not for group with kafka. but this is useful before group rbac with keycloak.


implement templates for reference file system

Implements the use of templates in reference file system as in fsspec, but does not support the full jinja2 templating.
For example,
{
    "templates": {"u1": "https://some/url", "u2": "https://some/other/url"},
    "refs": {
        ... "/some/key/0": [
            "{{u1}}" 0, 100
        ],
        ...
    }
}

In this case, the "{{u1}}" will be replaced with the value of the "u1" template string.

Codecov Report
Attention: Patch coverage is 91.66667% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 82.55%. Comparing base (bc71a9d) to head (d47fd19).

Files
Patch %
Lines

lindi/LindiH5pyFile/LindiH5pyFile.py
0.00%
2 Missing :warning:

...ndi/LindiH5pyFile/LindiReferenceFileSystemStore.py
96.87%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #51      +/-   ##
==========================================
+ Coverage   82.33%   82.55%   +0.21%
==========================================
  Files          25       25
  Lines        1715     1748      +33
==========================================
+ Hits         1412     1443      +31
- Misses        303      305       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Use Hamcrest assertions instead of JUnit in  tests/integration/native-image/se-1 (#1749)

Part of #1749
Backport 2.x

/trigger