Remove deprecated extend

this pr removed use of the deprecated extend from styled-components and resolves all outstanding warnings thrown when using the design system. this should reduce the noise for users

Codecov Report

Merging #416 into master will not change coverage.
The diff coverage is 100%.

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

https://github.com/pricelinelabs/design-system/issues/413


update filter types to conform to grid types

filter numeric fields should have the same types used in the grid/chain, they're all currently set to uint64

related to https://github.com/threefoldtech/tfgrid-sdk-go/issues/49


Extend Global SCSS Stylesheet in Component

Question
There may be an obvious solution that I am overlooking, but I am not able to import my main.scss file from assets  into a component.
<style lang="scss" scoped>
@import '~assets/main.scss';
</style>

This generates the following error:
This dependency was not found:

* !!vue-style-loader!css-loader!../node_modules/vue-loader/lib/style-compiler/index?{"id":"data-v-3bb4be04","scoped":true,"hasInlineConfig":true}!sass-loader!../node_modules/vue-loader/lib/selector?type=styles&index=0!./NavBar.vue in ./components/NavBar.vue

To install it, you can run: npm install --save !!vue-style-loader!css-loader!../node_modules/vue-loader/lib/style-compiler/index?{"id":"data-v-3bb4be04","scoped":true,"hasInlineConfig":true}!sass-loader!../node_modules/vue-loader/lib/selector?type=styles&index=0!./NavBar.vue

I have installed vue-style-loader and added it to my config file:
vendor: ['vue-style-loader'],

TLDR: How can I import a SCSS stylesheet from assets into a component?
Thanks,
Christopher

@Splode I think that error is because ~ alias is not working in sass loader. Can you try to directly importing using relative paths ? ../../assets ...

@pi0 Just tried using a relative path, but failed to build with same error.

I am using sass, and ~ links works fine with nuxt. You have to make sure node-sass and sass-loader packages are installed.
PS: I don't understand what is vendor: ['vue-style-loader']. This property is vendor js extraction as far as I know.

Thanks. I did sass-loader installed as a dependency, but not as a devDependency. Nuxt now compiles as expected.


Problem when using reserved go keywords

If any of go's reserved keywords are used as a field name,  whenever the lowercase version of that is used when generating code, you'll get a syntax error.
There needs to be some sort of safeguard in place so that the names don't collide. I suggest just prepending the words with an additional character. Example: type -> ttype etc.

Can you provide an example where this breaks the code?

It happens when model fields are used as function parameters.
Here's a simple schema to demonstrate the issue:
generator db {
  provider = "go run github.com/prisma/prisma-client-go"
  output = "./db/db_gen.go"
  package = "db"
}

enum UserType {
  ADMIN
  USER
}

model User {
  id String @id @default(cuid())
  email String @unique
  name String?
  type UserType
}

When running the generate command, I get this error:
	// Creates a single user.
	func (r userActions) CreateOne(
		// { }
				email userEmailSetParams,// { }
				type userTypeSetParams,
		optional ...userSetParams,
	) userCreateOne {
		var v userCreateOne
		v.query.Client = r.client
		var fields []builder.Field

		fields = append(fields, email.data)

		fields = append(fields, type.data)

		for _, q := range optional {
			fields = append(fields, q.data)
		}
		v.query.Inputs = append(v.query.Inputs, builder.Input{
			Name:   "data",
			Fields: fields,
		})
		return v
	}
	type userCreateOne struct {
		query builder.Query
	}
	func (r userCreateOne) Exec(ctx context.Context) (UserModel, error) {
		r.query.Operation = "mutation"
		r.query.Method = "createOne"
		r.query.Model = "User"
		r.query.Outputs = userOutput
		var v createOneUserResponse
		err := r.query.Exec(ctx, &v)
		if v.Errors != nil {
			first := v.Errors[0]
			return UserModel{}, fmt.Errorf(first.Message)
		}
		return v.Data.CreateOneUser, err
	}
 from file create.gotpl: 238:5: expected 'IDENT', found 'type' (and 10 more errors)
exit status 1

Notice that the name of one of the CreateOne function parameters are named type. This issue happens for all the reserved keywords I linked to earlier.

Thanks for the additional info. Will look into it next week

I did make a quick-fix for this locally, should I open a PR?

I'm currently struggling between just renaming these reserver keywords, such as type to ttype, or if I want to avoid that and just prefix the parameters with an underscore (so type becomes _type). What's your opinion about this @Nohac?

@steebchen I thought tools like golint etc. would complain if you had _ in variable names, but I checked, and it seems like it's fine with using _ as a prefix. I have never really seen _ being used in any go code I've read before, I've only seen things like typ and ttype being used in those situations, but since this is a bit deferent, I'd say go with _ since it should probably be a close to the original field name as possible.

Yeah I agree. Plus, the change is super simple (literally one char) and we don't need to introduce another library. We may re-visit when there's need for these variables in more places.


Update Homebrew formula for OSX install

See https://github.com/Homebrew/homebrew-core/pull/51193
See also https://formulae.brew.sh/formula/istioctl
This isn't something we fix in https://github.com/istio/istio , we must fix it in Homebrew.

As v1.5.1 was released, I have created a new PR with v1.5.1 Homebrew/homebrew-core/pull/52150.
(Homebrew/homebrew-core#51193 has been closed for that reason.)
It seems there was some build process change introduced with v1.5+, and thus required some fix on the script on Homebrew side.
This would mean that v1.5.0 won't be available on Homebrew archive - I suppose as long as the latest minor version (v1.5.1 as of now) is available, it wouldn't be a huge concern?

@rytswd Thank you!
1.5.x should be compatible with 1.5.0.  We shouldn't need back-level CLIs on Homebrew.

This has been merged with https://github.com/Homebrew/homebrew-core/commit/429cae217243e964c3718a019027d3adf8afd6d2.
I'm not too familiar with the steps for closing issue tickets like this, but as I understand the scope of this ticket is complete, I'm closing this ticket. If I have missed any details, extra information, etc., please do let me know and reopen as necessary!


Prims in C

Feature :
Prim's Algorithm is a greedy algorithm that is used to find the minimum spanning tree from a graph.
This algorithm is present in C++, Java and Python but is missing in C.
Description of solution :

Will add Prim's algorithms with adequate comments and proper documentation for readers to get a clear understanding.
To be added in C language.

Alternatives Considered :

Online Resources and Tutorials
External Libraries or Frameworks

I would like to work on this issue. Can you assign it to me under SSoC '23 ?
Thank you :)

am capable of implementing the Prims's algorithm in C++ and python languages. Therefore, I kindly request @Kumar-laxmi to assign me this issue.

I am capable of implementing the Prims's algorithm in C .
Therefore, I kindly request @Kumar-laxmi to assign me this issue.

Hi, I have optimised code for your problem, and I know it in C, C++, Java. I think I can provide justification for your problem. It would be very helpful for you if you assigned me this issue for SSOC-23 . I kindly request @Kumar-laxmi to assign me this issue.

Assigned!
@harshita-kaushik2 : C
I think by mistake you have mentioned Bellman Ford algorithm in the description, kindly update that too

Sir, Once go through the comment again I haven't mention  about bellman ford algorithm

@harshita-kaushik2  What is your update on this?

Hey @Kumar-laxmi
Due to a pending previous PR, I am unable to create a new PR for any other issue. All the changes I have made are getting pushed in the 1st PR on issue number #771
I have made the contribution on issue #682 and am attaching the images for reference :

And here is a screenshot of a sample input and the obtained output :

Let me know if I have to make any changes for the same...

@harshita-kaushik2 What is your status on this issue?

Hey @Kumar-laxmi I've already showed you my progress on the issue.
Can you please look into PR #771 so that I can create new PRs now...


Record audio from headset camera audio listener

Is there a way to change the code to record the audio from the user camera listener? If the spectator camera is far away you can't hear any sound. Thank you!

no this is currently possible.
to be clear, are you asking that a mobile device records the user via the hololens microphone?

I meant that I would like the spectator's scene's audio listener to be located at the user's camera position so that any spatialized audioclips would be heard from the point of view of the user and not the spectator. I think all that is needed is to deactivate the spectator camera's AudioListener component and add an AudioListener to a game object at the position of the user camera. Unfortunately, there appears to be a bug in iOS builds on Unity 2018 that removes spatial blending when you remove the AudioListener from the main camera. I also am not completely sure what the best way to find the position of the user camera is (on the spectator iOS app).


Register Arm SME passes in mlir::registerMlirPasses()

This allows these passes to be used in CLI options (e.g. --mlir-print-ir-after=<pass>).

cc @c-rhodes, @banach-space

@dcaballe  Are you okay to land this now? :)


Update broker key provider to accept custom tag

Needed for iOS broker to save any broker keys other than app key.

Added unit tests for the changed initializer.


how can I define a concrete repository operation?

I might have missed something obvious (or non-obvious), but there does not appear to be an example in the spec of how to define a concrete operation of a repository. It's extremely important that this be properly documented.
Naively, it looks a bit problematic: all the examples show repositories implemented as Java interfaces, and I don't see how one would inject an EntityManager, a DataSource, a Connection, or whatever, into an interface. Section 3.1 talks about injecting repositories into other things, but it doesn't seem to address injection into a repository.
Nor does there appear to be any operation of DataRepository or CrudRepository which would give me access to a connection.
(I've probably just missed something here.)

since I have only been involved in this spec from an advisory perspective, I can say how this works in Micronaut Data and whether the spec wants to follow suite is up to the folks actively working on it. With Micronaut Data you can use an abstract class and inject the EntityManager (or whatever persistence technology specific interface) as constructor arguments, See:
https://micronaut-projects.github.io/micronaut-data/latest/guide/#repositories

@graemerocher Right, that's the most obvious solution.

This one is currently being discussed in other issues, but agreement hasn't been reached yet and so it hasn't made it into the spec.  A couple of ideas that I've thought of are:

Let users write default methods with an EntityManager (or DataSource or Connection) parameter where the invoking code sends in null and the repository provider substitutes in the actual entity manager for the method to use,

    default void myMethod(int arg1, String arg2, EntityManager em) {
        ... use entity manager
    }

An advantage here is that it gives the repository provider the ability to ensure the EntityManager gets closed upon completion of the method.  I'm not sure how important that is for EntityManager, but if it were a JDBC Connection, I know that's important.

Add a method like getResource(Class<T> type) either one of the built-in repository interfaces or a mix-in interface for that purpose, that can be used to obtain the EntityManager,

    default void myMethod(int arg1, String arg2) {
        EntityManager em = getResource(EntityManager.class);
        ... use entity manager
    }

The repository provider could close resources here after the method completes, although it might be a little less straightforward to do it.
There are probably a variety of other options as well. It would be good to standardize one.
We don't have abstract classes as repositories defined in Jakarta Data yet, but that could get added.

The way this works in Hibernate 6 is as follows. I write:
interface Queries {
    EntityManager entityManager();

    // generated
    @Find
    List<Book> findBooksByTitle(String title);

    // handwritten
    default List<Book> findBooksByTitleAndType(String title, String type) {
        entityManager()
                .createQuery("from Book where title like :title and type = :type", Book.class)
                        .setParameter("title", title)
                        .setParameter("type", type)
                        .setFlushMode(COMMIT)
                        .setMaxResults(100)
                        .getResultList();
    }
}

The annotation processor sees that we have an operation with no parameters which returns something "session-like" (EntityManager, or something similar) and fills it in.


BAH-1734 | Added configuration for vitals form.

Co-authored-by: Kavitha S<EMAIL_ADDRESS>

@N0-man , @gsluthra , @angshu
can we please recheck the changes I have resolved the conflicts and added a new  custom concept Temperature based on reviews from desk check,as the CIEL concept temperature have units as degree centigrade but not Farenheit.


pages show up in the wrong orde

in the main pane, but not for thumbnails
the code is almost exactly the same, so I'm confused.

page rendering appears not to be idempotent while pages are being added to the collection.
not the end of the world. Fixed. 9078485


Avoid to serialize values into JSON strings

The set method of the Redis module always serializes values into JSON strings, regardless the original type of the value. It works for object values but, especially for string values, produces an unexpected behavior which changes original value (i.e. foo --> "foo").
Since Redis only accepts string values, keep JSON serialization for object value type and cast to string the others, returning responsibility of the type-check to the consumers of the client.

returning responsibility of the type-check to the consumers of the client

Can you elaborate on this?

Yes but we need to change get signature to this:
export interface Cache {
  readonly get: <A>(
    key: string,
    codec: t.Type<A, J.Json>, // from t.Type<A, unknown>
  ) => TE.TaskEither<Error, A>
  // ...
}

If you approve that, I'm going to apply it.

Why shall we?
declare const codec: t.Type<{ foo: string }, unknown, unknown>
declare const u: string
const x = JsonFromString.pipe(codec).decode(u)

It does not compile, because of the signature of the encode function:
Type 'Encode<{ foo: string; }, unknown>' is not assignable to type 'Encode<{ foo: string; }, Json>'.

What if we change the signature to something like Pick<C, 'decode'> for get and Pick<C, 'encode'> for set?

Do you mean something like that?

export interface Cache {
  readonly get: <A, C extends t.Type<A, unknown>>(
    key: string,
    codec: Pick<C, 'name' | 'decode'>,
  ) => TE.TaskEither<Error, A>
  // ...
}

Correct, but maybe it's not even needed if you avoid piping codecs and just run them one after the other.

Unfortunately this causes other type errors, because while using it other codec's properties are requested, decode/encode too, so we can't "pick" them – or if so we need to handle the incompatibilities.
Another quick solution – only for the get function – could be the following:
export interface Cache {
  readonly get: <A>(
    key: string,
    codec: t.Type<A, any>,
  ) => TE.TaskEither<Error, A>
  // ...
}

It's not too clean but type inference is preserved.

It works! Check it out :smile:

Great! Pet ESLint and don't forget the changelog entry! ;)


”Fundu Văii”, ”Oprișița”, ”Poienești-Deal” are listed in the drop-down menu

Description:
When user writes ”Fundu Văii”, ”Oprișița”, ”Poienești-Deal” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Fundu Văii"

Expected results
The location appears in the drop-down menu as "Sat Fundu Văii, VASLUI (Poienești)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Oprișița"

Expected results
The location appears in the drop-down menu as "Sat Oprișița, VASLUI (Poienești)".

Step 5
Write in the search bar "Poienești-Deal"

Expected results
The location appears in the drop-down menu as "Sat Poienești-Deal, VASLUI (Poienești)".


fix(rbac): update rbac-backend plugin to version 1.6.2

Description
Update rbac-backend plugin to version 1.6.2 to make plugin working with ssl.
Which issue(s) does this PR fix

Fixes janus-idp/backstage-plugins#913

PR acceptance criteria

Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

Looks like I need collect on more fix from this pr:  https://github.com/janus-idp/backstage-plugins/pull/929


ajax instead of anchors?

Can I use ajax instead of anchored content in hidden layer?

No, ajax is not supported. My advice would be to load the content using ajax manually.


Added Security cheatsheets

Added Security tools cheatsheets link (https://github.com/andrewjkerr/security-cheatsheets)

@Chan9390 LGTM, merged
The test do not pass, we will remove the 404 url:)


TODO: Create module for interacting with Supabase

This module should have the following features:

Setting up any parameters for requests (such as formatting headers) sent to Supabase
Sending out requests to the database hosted on Supabase
Processing the responses to these requests and returning relevant information

[x] Formatting headers
[ ] Constructing the reqwest client to handle making requests to Supabase
[ ] Building the base URL

Going to close this issue -- will address the third bullet point more precisely by creating separate issues/todos for different tables in the database


chore(gzip) Retry without gzip when response seems broken

Hello,
Sometimes servers return broken gzipped data.
We could try to fetch the webpage again as non-gzipped to get the content.

Hey guy, I suggest you to switch gzip off if gzip is not supported by the target website, rather than retry, because it introduces complexity. But thank you all the same.

Hey, thanks for your response.
I should have open an issue first to discuss about that need.
To reply to your suggestion, as a crawler you don't always know by advance all links that will be visited so if the page has a gzip issue or not. For your information in my case, the target website supports gzip but has encoding issues only for some of its pages and you can't know before visiting them.
In my case, a regular browser arrives to display the pages even if a part of the content can't be decoded, so the pages are not considered as "buggy" by its owner.
If you don't want this "retry" behavior to be enabled by default, what do you think about a configuration field to enable it?

Thank you guy, let's imagine that if the 'retry' behavior is not enabled by default, when do you use it? I think I'll turn to switch on when find a problem with unsupported website, which is totally same with you find it and switch off gzip.


Graph generation broken in 0.2.0+

Firstly, thanks for your hard work developing this tool. It is a great resource for laying the groundwork of our refactoring project.
I've noticed that the graphs generated in the latest version 0.3.1 don't provide any useful information. I've noted a couple issues:

Graphs do not show any class dependencies, but rather showcase function dependencies for each class (see representative sample below, I don't want to disclose too much but all my classes look something like this)

The generated analyzed.js file seems to be missing a whole bunch of classes (namely Activities)

I really want to use this tool, and was wondering why the generated outputs differ widely from your screenshots, so I thought it might have broken in an update. Fortunately, I was able to generate the expected graphs using an older version 0.1.5. This seems to resolve the above 2 issues. All versions after 0.1.5 somehow broke the output.
That said, the filtering options available in 0.3.1 are super helpful, and I wish I was able to integrate those back into 0.1.5 since 0.1.5 is the latest version that somehow works properly. but for the time being, what I have generated using 0.1.5 is a great start.
Let me know if you need me to provide more info to help debug this issue. Thanks again!

It looks like the fix outlined here solves this problem. I am running this on Windows: https://github.com/alexzaitsev/apk-dependency-graph/issues/60#issuecomment-565487508


HTTP 413 Payload Too Large when sending image inline

I'm using MockAI for a call that includes image inline. The call works toward OpenAI.
I'm not sure what the limit should be, but hopefully this project can simply discard and ignore the input?
 HTTP Request: POST http://mock_third_party:8000/v1/chat/completions "HTTP/1.1 413 Payload Too Large
Let me know if you need more info about it. By my napkin math, my request should be around 70MB

Thank you for using this repo. I don't have a large dataset to reproduce the bug. However, I found a very simple solution that might help you. If this is a major requirement, I can modify the code accordingly.
Error: request entity too large
app.use(express.json({limit: '50mb'}));
Hope it helps.


NameError: name '_C' is not defined

May I ask if this repo requires the installation of VC++buildtools?

May I use the torch-2.0.0+cu118 as the CUDA_HOME?It is include cuda toolkit 118?

Having the same issue. Did anyone resolve it?

May I use the torch-2.0.0+cu118 as the CUDA_HOME?It is include cuda toolkit 118?

When you build custom cuda extension that is _C file , it uses local cuda toolkit, hence you need to install cuda_toolkit 11.8 in your system , and then set CUDA_HOME to the location where it's installed , along with that also set up the the LD_LIBRARY  https://forums.developer.nvidia.com/t/path-ld-library-path/48080

Can I install the latest cudatoolkit 12.1 directly, even if I'm using cu118?

No, when building custom cuda extension local cuda toolkit should match to the pytorch's cuda support version , hence you need to install cuda 11.8 version
https://developer.nvidia.com/cuda-11-8-0-download-archive

By the way,I have other repo using cu117 etc, do I need to install more than one cudatoolkit 11.7 in addition?

Not sure if you can several cuda versions or not, but yess the cuda version in local must match your pytorch version only for the cases where you build custom cuda extension , else no need.


fix: turn off logging stderr+out to file

Removing the log file so we can just log to the container
stdout and stderr to be picked up by Loki or other means.

I'll create a tracking issue for making logs easily accessible via Loki, or if you're running in docker-compose, the logs would be visible via docker compose logs
issue: #455

@allan-bailey I've left a comment on the ticket as well, but to make sure it's not missed, let's make sure that README is updated and tell the user where they should expect to find the logs.

Once this new Dockerfile + entry.sh script is on main, I'll be able to fix our workflow to build and push the container image to docker hub.

Fixes #455


[Feat]: Add Issue Templates & Pull Request Templates

I would like to add several issue templates and a pull request template to improve the contribution process.
I propose adding the following issue templates:

Documentation Template: This template should include fields for the documentation topic, issue description, and any relevant context or screenshots.

Feature Request Template: This template should include fields for the feature description, use case, and any relevant screenshots or mockups.

Bug Report Template: This template should include fields for the issue description, expected behavior, actual behavior, steps to reproduce, and any relevant screenshots or error messages.

Other Issue Template: This template should include fields for the issue description, any relevant context, and any necessary instructions for reproduction.

I also propose adding a Pull Request Template. This template should include fields for the pull request description, changes made, any related issues or pull requests, and any necessary instructions for testing.

I had already created a similar issue ( #4 ) previously and @priyankarpal is working on it

We need one more template, Suggestion Template for any general suggestions.

@priyankarpal Are you working on it?

I had already created a similar issue ( #4 ) previously and @priyankarpal is working on it

We need one more template, Suggestion Template for any general suggestions.

@priyankarpal Are you working on it?

yes I'm working on it I will create a PR today

This issue is resolved.


Restart/Kill Server on Host...

Is there an equivalent to Remote-SSH: Kill VS Code Server on Host...  from the Command Palette?

Came about, as the official server has a similar option to remote.SSH.remoteServerListenOnSocket
It is listed at https://github.com/jeanp413/open-remote-ssh/blob/master/package.json#L90, but the official appends to the description:

When true, the remote VS Code server will listen on a socket path instead of opening a port. Only valid for Linux and macOS remotes. After toggling this setting, run the command "Kill VS Code Server on Host..." for it to take effect. Requires OpenSSH 6.7. Disables dynamic port forwarding and "local server" mode. Requires AllowStreamLocalForwarding to be enabled for the SSH server.

Not yet, need to implement that, but I think that command is only needed if the server is already running in the remote machine, if you set remote.SSH.remoteServerListenOnSocket to true before connecting it should work as expected

https://github.com/microsoft/vscode/issues/109475 has the most thorough answer, but tl;dr it provides security for multi-user server access, as generates user-specific UNIX sockets, whereas the default TCP port can be accessible to anyone.

Tbh, never considered that is could already be using websockets; I have attempted to take a glance at it, but was not sure what to look for;
I did find the remoteServerListenOnSocket option declared, and used in authResolver.ts, which leads to useSocketPath in ServerSetup.ts.
It could be just that the option is already there, just not listed as a user specified option - i.e. setting just missing from front-end.

Now thinking about it, when I opened the question, I had a different issue, but at the time I was hoping this could fix it - but eventually it was discovered to be an unrelated server issue, not a Remote SSH issue, so sort of solved itself.
So from my end these are sort of low-priority, as rarely had issues with Remote SSH.
I guess for robustness sake they should still be considered but definitely not critical.


Compiler error in Android/iOS(DartVM)

I want to use both image_picker and image_picker_web.
It work in flutter web.
But native platform error.
Compiler message:
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-1.0.8/lib/image_picker_web.dart:6:8: Error: Not found: 'dart:html'
import 'dart:html' as html;
       ^

../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-1.0.8/lib/src/web_image_picker.dart:2:8: Error: Not found: 'dart:html'

import 'dart:html' as html;

       ^

../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.12.1/lib/src/browser_client.dart:6:8: Error: Not found: 'dart:html'

import 'dart:html';

       ^

../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-1.0.8/lib/image_picker_web.dart:32:10: Error: Type 'html.File' not found.

  Future<html.File> _pickFile(String type) async {

         ^^^^^^^^^

../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.12.1/lib/src/browser_client.dart:34:18: Error: 'HttpRequest' isn't a type.

  final _xhrs = <HttpRequest>{};

                 ^^^^^^^^^^^
Running pod install...                                              2.8s
Building macOS application...
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-1.0.8/lib/image_picker_web.dart:33:16: Error: 'FileUploadInputElement' isn't a type.
    final html.FileUploadInputElement input = html.FileUploadInputElement();
               ^^^^^^^^^^^^^^^^^^^^^^
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-1.0.8/lib/image_picker_web.dart:33:47: Error: Method not found: 'FileUploadInputElement'.
    final html.FileUploadInputElement input = html.FileUploadInputElement();
                                              ^^^^^^^^^^^^^^^^^^^^^^
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-1.0.8/lib/src/web_image_picker.dart:7:16: Error: 'FileUploadInputElement' isn't a type.
    final html.FileUploadInputElement input = html.FileUploadInputElement();
               ^^^^^^^^^^^^^^^^^^^^^^
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-1.0.8/lib/src/web_image_picker.dart:7:47: Error: Method not found: 'FileUploadInputElement'.
    final html.FileUploadInputElement input = html.FileUploadInputElement();
                                              ^^^^^^^^^^^^^^^^^^^^^^
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-1.0.8/lib/src/web_image_picker.dart:12:20: Error: Method not found: 'FileReader'.
    final reader = html.FileReader();
                   ^^^^^^^^^^
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-1.0.8/lib/src/web_image_picker.dart:28:16: Error: 'FileUploadInputElement' isn't a type.
    final html.FileUploadInputElement input = html.FileUploadInputElement();
               ^^^^^^^^^^^^^^^^^^^^^^
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-1.0.8/lib/src/web_image_picker.dart:28:47: Error: Method not found: 'FileUploadInputElement'.
    final html.FileUploadInputElement input = html.FileUploadInputElement();
                                              ^^^^^^^^^^^^^^^^^^^^^^
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-1.0.8/lib/src/web_image_picker.dart:33:20: Error: Method not found: 'FileReader'.
    final reader = html.FileReader();
                   ^^^^^^^^^^
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.12.1/lib/src/browser_client.dart:58:34: Error: 'Blob' isn't a type.
      var blob = xhr.response as Blob ?? Blob([]);
                                 ^^^^
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.12.1/lib/src/browser_client.dart:46:15: Error: The method 'HttpRequest' isn't defined for the class 'BrowserClient'.
 - 'BrowserClient' is from 'package:http/src/browser_client.dart' ('../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.12.1/lib/src/browser_client.dart').
Try correcting the name to the name of an existing method, or defining a method named 'HttpRequest'.
    var xhr = HttpRequest();
              ^^^^^^^^^^^
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.12.1/lib/src/browser_client.dart:58:42: Error: The method 'Blob' isn't defined for the class 'BrowserClient'.
 - 'BrowserClient' is from 'package:http/src/browser_client.dart' ('../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.12.1/lib/src/browser_client.dart').
Try correcting the name to the name of an existing method, or defining a method named 'Blob'.
      var blob = xhr.response as Blob ?? Blob([]);
                                         ^^^^
../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.12.1/lib/src/browser_client.dart:59:20: Error: The method 'FileReader' isn't defined for the class 'BrowserClient'.
 - 'BrowserClient' is from 'package:http/src/browser_client.dart' ('../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.12.1/lib/src/browser_client.dart').
Try correcting the name to the name of an existing method, or defining a method named 'FileReader'.
      var reader = FileReader();
                   ^^^^^^^^^^
../../../development/flutter/packages/flutter_web_plugins/lib/src/plugin_registry.dart:29:5: Error: Method not found: 'webOnlySetPluginHandler'.
    ui.webOnlySetPluginHandler(_binaryMessenger.handlePlatformMessage);
    ^^^^^^^^^^^^^^^^^^^^^^^

Hi thanks for raising your issue.
This is not an error, it's a Flutter thing.
But there's a solution like Rody Davis is also using for handling cross-platform functionalities in one project: Just use this:
https://pub.dev/packages/universal_html
instead of Flutter's html package and your errors should be gone.
Closing this, because it's not related to this package. For further questions raise an issue on Flutter Framework.
Best regards

@abc873693 @Ahmadre can you please help me out by providing any reference or link in which both plugins have been used with UNIVERSAL HTML package.
Or maybe you have used it ....

We have good news for you: https://github.com/flutter/plugins/pull/2767
Official image_picker for web will come soon. You can link to that pr if you want to use it in early stage.

Thanks a lot @Ahmadre


Consider implementing infix commands like \over and \choose

It would basically work like ^ and _ work now: a node is parsed and then we check whether the next token is  \over or \choose, in which case we don't return the node yet and continue parsing.
Though, you actually get a warning if you use these in LaTeX today, so there's probably a good reason why you shouldn't use them.

I don't think I will do this after all. It certainly would be possible to support, but I don't really see the point.


refactor(panel): migrate Panel widget stack to TypeScript

This migrates the Panel widget stack to TypeScript. This is a non-conventional widget that benefits from being typed.
This anticipates new APIs that we might want to introduce in v5 regarding simplifying its usage (perhaps passing UI state on top of the whole render options).
There is a small bundle size increase because we now import preact/hooks. This will get "absorbed" once we remove classes from the codebase.

I would have put the refactoring to hooks in a separate PR, it would make easier for us to see what changed during the migration.

@yannickcr I forgot to mention, class components do not work with our TypeScript + Preact setup, so that was not possible.

Oh, I'm interested in more details :slightly_smiling_face:
But that's a good reason to make both at the same time then, worth mentioning it in the PR description tho :wink:

@Haroenv Nothing relevant in the tests, I just migrated to @testing-library/preact. You can try the stories.


How to get the set connection string to the static options?

There are probably lots of ways to do this but how do you recommend doing this?
Like the code below?
     public class Startup
    {
        private static readonly FeatureFlagOptions options
            = new FeatureFlagOptions().UseCachedSqlFeatureProvider(Configuration.GetConnectionString("connectionString"));

	public Startup(IConfiguration configuration)
        {
            Configuration = configuration;
        }

        public IConfiguration Configuration { get; }

Looks good to me!

It looks good but the things is like this I get the following build error

A field initializer cannot reference the non-static field, method, or property 'Startup.Configuration'

when trying to use Configuration.GetConnectionString("connectionString") like I do in the example above.
But after some trial and error I managed to get this to work with with both sql and in-memory.
UseCachedSqlFeatureProvider
private readonly FeatureFlagOptions options;

public Startup(IConfiguration configuration, IHostingEnvironment env)
{
    Configuration = configuration;
    options = new FeatureFlagOptions()
    {
       FeatureFlagAssemblies = new[] {
        typeof(MyServiceLayerThatUsesthis).Assembly
       ,typeof(MyFeatureClassInMyCoreLib).Assembly
    }}.UseCachedSqlFeatureProvider(Configuration.GetConnectionString("DefaultConnection"));
}

And staticly with UseInMemoryFeatureProvider
like this (with no need for a connection string)
public static readonly FeatureFlagOptions options = new FeatureFlagOptions
{
    FeatureFlagAssemblies = new[] {
        typeof(MyServiceLayerThatUsesthis).Assembly
       ,typeof(MyFeatureClassInMyCoreLib).Assembly
    }}.UseInMemoryFeatureProvider();

And then using options like before.

Explanation of code
MyServiceLayerThatUsesthis == I use this in my servicelayer togling not in a Controller.
MyFeatureClassInMyCoreLib == Since I need this both in my Web and servicelayer and my servicelayer should not know  about web I keep this feature class in my core lib.
UseCachedSqlFeatureProvider
I would add some doc that lets us know that UseCachedSqlFeatureProvider  creates a table RimDevAspNetCoreFeatureFlags where it stores the flags.
Question
But could you quickly tell me the difference between the two?
Thanks
But all in all I really like this. Thanks for creating it!

UseInMemoryFeatureProvider shouldn't be used except during local development since the settings don't survive when the application is shutdown. That said, UseCachedSqlFeatureProvider is the only viable option currently for actual use.
Updating sample code to use IConfiguration is a good idea.


Scheduler: an overweight task removed silently

Is there an existing issue?

[X] I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

[X] This is not a support question.

Description of bug
A task which does not passes this weight check will be removed silently.
https://github.com/paritytech/substrate/blob/a7bc9c2a8c0a14fd6033de1f972a11a2b3dbb73a/frame/scheduler/src/lib.rs#L1014
Steps to reproduce
Schedule a task with a big len, to not pass that check.
The task wont be postponed even if its not a first task and removed silently with no errors

Yeah, we should only drop the task when the weight is > 9/10 of max block weight.


But I'm just wondering if anyone could give me some advice on which source, and what settings to use in OBS to use the UDP stream via the command that's provided when I run the script? I'm guessing that'd be more efficient than just getting OBS to capture the VLC window preview directly, which is what I've been doing so far, especially whilst I'm streaming some more resource intensive stuff at the same time.

Glad you like it, I've just seen this related comment -> https://github.com/jschmid1/gopro_as_webcam_on_linux/issues/48#issuecomment-1401214318


Migrate integration tests to Vite

Some notes about the migration:

Vite tests don't use our PlaywrightFixture abstraction and instead use Playwright directly or via smaller helper utils.
Request/response tests now use Playwright's API testing feature rather than importing the built request handler. This is to avoid issues with hanging tests in Node 20.5.2+: https://github.com/remix-run/remix/pull/9000

Closing in favour of https://github.com/remix-run/remix/pull/9182.


Fluent bit docker images are not compatible with Apple M2 chips.

Hello @ThomasVitale
I was trying to run the observability stack of Chapter13-end, the issues is that the stack uses fluent-bit as agent and the docker image grafana/fluent-bit-plugin-loki:2.6.1-amd64  does not gets compiled on  Apple M2 chips.
This I conclude from below error.
 fluent-bit The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested

If my conclusion is correct, can you suggest what changes we have to make in order to use pormtail instead of fluent-bit. Or you can point me to a suitable alternative.

duplicate https://github.com/ThomasVitale/cloud-native-spring-in-action/issues/31


Add animation to Modal component?

Right now, the yew_bootstrap::component::Modal will appear and disappear without animation, while the JS Bootstrap component has an optional fade-and-slide effect. That effect will happen if the div.modal also has a .fade, and if this class is added to the HTML produced by component::Modal, it also works. Maybe there should be a property that allows using this behavior?

I agree with your proposal; it sounds like it would be pretty easy to implement. A boolean property which would trigger adding the "fade" class to the set of classes added to the modal. If we aren't using the appendable classes pattern for this component, we certainly are elsewhere in the project.
If you make a PR for this I'll be happy to merge it!


Check for the correct toggle

Who is this for?
Users/Toggles
What is it doing for them?

I went too fast earlier and wasn't checking for the correct toggle name. Now changed it to match.

Closing this as I'm removing that toggle completely (@gestchild good shout on the helper function though! Unsure why I didn't use it to start with.)


Stopped working in Slack Version 3.2.0?

Hi,
I used your script for quite some time now and it always worked fine. But now, since a week or so, it's just not rendering anymore.
I'm not sure if it's the new Slack version (I have v3.2.0 installed)?
I already tried everything like downloading the script again from the Github, re-running the script and checking with some basic math that should normally work.
Thanks in advance for any help!

Hi Kilian,
Me and another user both have this issue with Slack v3.2.0 (and now 3.2.1) on Ubuntu 18.04. Do you also use Ubuntu 18.04 or do you use another OS?

It doesn't work on Windows 10 with Slack v3.2.0 either

Hi Laurent,
I'm using MacOS High Sierra (latest version).

Hi @Michielskilian and thanks for the bug report. Please try this script: https://github.com/fsavje/math-with-slack/blob/d385db532d6698b16a79c6d06fe1bdcb52732888/math-with-slack.sh Please let me know how it works.
(Working on Windows version at the moment, @KeAWang )

@KeAWang Could you please try whether the updated script works, and report back to here?

Hi @fsavje, the script you attached works!
Thank you for the quick fix!

Unfortunately the newest version is not working for me in Win10, v3.2.0

A few of the paths aren't working even though the .js files are in the directory. The previous release, v0.2.3 runs without a hitch but none of the math is rendering either way...

@fsavje It still doesn't seem to work on Windows 10. The new script gives the following errors
Using Slack installation at: ...\slack\app-3.2.0\resources\app.asar.unpacked\src\static
The system cannot find the path specified.
FINDSTR: Cannot open ...\slack\app-3.2.0\resources\app.asar.unpacked\src\ssb-interop.js
The system cannot find the path specified.
FINDSTR: Cannot open ...\slack\app-3.2.0\resources\app.asar.unpacked\src\ssb-interop.js
Backup already exists: ...\slack\app-3.2.0\resources\app.asar.unpacked\src\static/ssb-interop.js.mwsbak
Press any key to continue . . .

@Servinjesus1 and @KeAWang Thanks for trying it out. I don't have access to a Windows machine atm, so I can't debug this properly. Could you try this script: https://raw.githubusercontent.com/fsavje/math-with-slack/7d7bc39d2723e8c299bff47d4d80d465d044a1af/math-with-slack.bat Thanks!

Works great now, thanks!

Thanks for checking!


Ddwrt dev

This component provides device tracker support for routers running DD-WRT firmware.  DD-WRT has been largely replaced by OpenWRT but is still used on many older routers.  THe code is heavily based on the Luci device tracker.

I won't actually merge it yet in case you have some comments, if I don't hear anything in the next few hours I'll just go ahead and merge it.

Sorry for all the commits, I'm getting different flake8 results locally to travis CI for some reason

Coverage decreased (-1.58%) to 64.82% when pulling a959c48708caaf7f22a8cd6a4e112aa37dd5fc82 on jamespcole:ddwrt-dev into 5a0251c3cd99e7aa937ef55de64bc4d4c186cfe0 on balloob:dev.

Flake8 might be reporting different errors if an update has been issued that makes things stricter. Try upgrading it: pip3 install --upgrade flake8.
Code LGTM :dolphin:

Coverage decreased (-1.51%) to 64.89% when pulling 242c143c85041c6f7253165c670e073ff6383ebf on jamespcole:ddwrt-dev into 5a0251c3cd99e7aa937ef55de64bc4d4c186cfe0 on balloob:dev.

Coverage remained the same at 66.39% when pulling 0b6d260fa6190271bcbc51dd27a19b8b6b61bde2 on jamespcole:ddwrt-dev into 5a0251c3cd99e7aa937ef55de64bc4d4c186cfe0 on balloob:dev.


Not all Dependabot languages support private repos

Ran into this (a quite frustrating one, given that Ruby isn't supported, which is what GitHub is built in.)
Didn't follow the review guidelines, sorry.

Thanks for opening a PR 👍 Please fill out the pull request template in the top comment in this pull request. Let me know when that's done and I'll get this up for review!

@janiceilene I'm not highly motivated to follow the particulars of GitHub's internal documentation guidelines, as it's not my job to work on GitHub's docs. I've highlighted an important deficiency in the docs about a frustrating limitation and tried to help out, but it's up to someone at GitHub to take it from here.

Sounds good, thanks everyone!


Add ability to limit the displayed decimal points for properties in the viewer footer

🚀 Feature
I propose adding the ability to control the displayed decimal points for properties in the viewer footer. The most straightforward way would be to inherit from padas.set_option("display.precision").
Motivation
I'm working with a Points layer, with an associated properties DataFrame. When I hover over a given point in the napari viewer, these properties are displayed on the bottom, which is a great feature!
What's not so great is that some of my properties have many decimal points, resulting in unnecessarily long numbers displayed in the viewer (see the circled "confidence" values in the screenshot below).

Controlling the precision via pandas.set_option("display.precision",  3) is ineffective. I could round the properties, but for various reasons I want to avoid that loss in precision, if possible.
Pitch
Since properties are stored as a pandas DataFrame, I'd expect pandas display options to effect how these properties appear in the viewer.
I propose altering the code here such that the printed property values inherit the display precision from pandas.
Alternatives
Instead of inheriting the display precision from pandas, this could also be handled with a napari setting. But if such a setting doesn't already exist, creating one just for this use-case seems a bit of an overkill.
Additional context
This matter was initially discussed on Zulip and I've opened this issue as prompted by @jni.
I'm happy to have a go at opening a draft PR, if people agree with the pitched solution.
One thing I currently don't know is how this is handled for other layer types, and whether the proposed solution should work across layer types (as opposed to only the Points layer).

With a bit more thought, I agree with @brisvag's comment on Zulip that we don't actually want to be bound to pandas specifically (as opposed to "some columnar data analysis library, could be pandas or polars or narwhals or dask...), so we probably instead want our own setting.
I also think display should be based on significant digits, not digits after the decimal. Python's float formatting has this facility with the 'g' flag:
In [1]: x = 1.0654820104354

In [2]: f'{x:.4g}'
Out[2]: '1.065'

In [3]: y = 22.7821

In [4]: f'{y:.4g}'
Out[4]: '22.78'

I think 4 is a reasonable default number (though I was taught 3 at school 😂).

Thanks for opening the issue!
I think a setting for precision globally in napari could be worth it -- we already have a hard-code 3 decimals utility function used for intensity values in the status bar:
https://github.com/napari/napari/blob/a9e7375e339a2240f6cc6affd05eceefa2e843e9/napari/utils/status_messages.py#L8-L10
So one option would be to use that function and then also make the value a setting?

Could we somehow just expose the formatting specifier itself? So one could just pass .4g or .f2 or whatever they prefer, and it would Just Work™?

Could we somehow just expose the formatting specifier itself?

yes but: I think that's more flexibility at the cost of readability — many people will have no idea how to type a valid formatter. We could do a two stage dialog: "floating point formatting: n significant digits OR specify formatter:". And we should have a preview box that changes as you choose the option.
I'm vaguely inclined to only do the simple setting first though, because the double setting just adds complexity...

yes but: I think that's more flexibility at the cost of readability — many people will have no idea how to type a valid formatter. We could do a two stage dialog: "floating point formatting: n significant digits OR specify formatter:". And we should have a preview box that changes as you choose the option.

Maybe just a text field with validation logic, that will treat entering number as a number of digits, otherwise accept use it as format string?

Agreed with a bunch of comments above.
I think the first step is to use napari.utils.status_messages.status_format in Labels._get_properties instead of the plain feature value. Then consider adding one setting to control the floating point precision with the standard Python f-string format specifiers.


[$1000] Web - Avatar displayed when creating a task on a thread without selecting an assignee

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the chat
Send a message
Click on "Reply in thread"
Go to the "Assign task" section within the thread
Enter a title for the task and click on the "Next" button
Click on the "Confirm task" button.

Expected Result:
The avatar should not be displayed since no assignee has been selected for the task
Actual Result:
When creating a task on a thread without selecting an assignee, a different avatar is being displayed instead
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.29-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/169a0b1d-a406-459b-8c0c-48efff491eda

https://github.com/Expensify/App/assets/93399543/ec723dbb-7015-4870-9336-1e4761586cd2
Expensify/Expensify Issue URL:
Issue reported by: @ayazhussain79
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1687162712035319
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c3a2b73ab04acc58
Upwork Job ID:<PHONE_NUMBER>931828736
Last Price Increase: 2023-06-26

Hmm, I don't know what should be the expected behaviour here. Is it an empty view?

cc @anmurali and @shawnborton


Process authorized request to account servlet through Oauth Proxy

For now we only have the use case of request to account.json, but let's make a foundation between modules so that we can add more cases in the future.
Based on what a request is authorized:

user session's oauth_token
user session's oauth_token_secret
(I choose these because these are the fields that, currently, saved for each user, clien_id doesn't seem to be there)

Implementation:
From the web client : a new search service will be created, where request to oauth proxy are defined. Included in request GET params are

oauth_token
oauth_token_secret
screen_name
servlet
& other related args, depends on the servlet option

E.g. http://localhost:3002/authorized?oauth_token=12345&oauth_token_secret=54321&servlet=account.json&screen_name=foo
From the oauth proxy : api will listen to the according path, i.e. /authorized?. When request from the client are made:

the oauthproxy then will request to loklak account servlet, with screen_name from the client, to retrieve the tokens
if matches, make request to the given servlet, along with the rest of the args (this is not needed in this case, since we're already got the result from account servlet from step 1, pass it right away)

From the loklak server : nothing changes, since oauth proxy will be making request from localhost to localhost.

This is similar to what I was thinking, but instead of having a dedicated /authorized endpoint, we can have middleware called isAuthorized which can be used on other routes as well.

The loklak_server should (if authentication is made right) only return account information for users which are authentified. Since 'user is authentified' is check in the loklak_webclient, this information must be passed to the server as additional call parameter. To be able to access the access.json servlet, the request must come from oauth-proxy, but the oauth-proxy must send along also the information which user has initiated the communication.
This will enable the loklak_server to strengthen the security of the account.json servlet: access should not only be granted to localhost access but also only to one specific account which the oauth-proxy names as the authentified user.

This is implemented.

The client can do authorized request to the oauth proxy with 4 params: servlet, screen_name, oauth_token, oauth_token_secret. Service code  https://github.com/loklak/loklak_webclient/blob/master/app/js/services/authorized-search.js
The proxy will pass a request along to the according servlet, which is only account.json for now. Line 42-63 in https://github.com/loklak/loklak_webclient/blob/master/oauth-proxy/api.js

Currently account servlet is not processing the oauth_token nor secret and return the result anyway to the oauth proxy. @Orbiter you can implement the tokens checking when you can. If some parameter needs to be changed, please comment here.

Suggestion, if we make this into a middleware, it can be used for other routes as well. For example, we define a function:
var isAuthorized = function(req, res, next) {
        if (/*condition to check if req has valid token and secret*/) {
            res.next();
        }
        else { res.end('Unauthorized'); }
    }

After that we can use this in any route.
For example, this is the handler for reading a wall object
//CREATE
router.post('/:user/:app', isAuthorized, function(req, res) {
    var newWall = req.body;
    getData(req.params.user, function(error, response, body) {
        var appData = JSON.parse(response.body).accounts[0].apps;
        if(!appData){
            appData = {};
        }
        if (!appData[req.params.app]) {
            appData[req.params.app] = [];
        }
        newWall.id = shortid.generate();
        appData[req.params.app].push(newWall);
        console.log(newWall.id);
        updateData(req.params.user, appData, function(error, response, body) {
            console.log(response.body);
            return res.json({
                id: newWall.id
            });
        });
    });
});````
All the wall routes require that the user be authorized. Currently, anyone can make direct requests to oauth-proxy which is a HUGE security hole. This patch is step in the right direction, but it doesn't look to be scalable. It will become useful once we start integrating payment systems. What do you think?

@aneeshd16, good point. I can wrap the authorized process in a reusable method with a callback.

Implemented, please checkout lines 45-85 for how to/ an example
https://github.com/loklak/loklak_webclient/blob/master/oauth-proxy/api.js


topologySpreadConstraints: handle nodeTaintsPolicy and nodeAffinityPolicy constraints

Rebased on top of:
https://github.com/kubernetes-sigs/descheduler/pull/1205
https://github.com/kubernetes-sigs/descheduler/pull/1208
to include e2e tests

/label tide/merge-method-squash

/retest

@a7i , Thank you so much for adding e2e test cases. If PR has been reveived can we merge it?

@nitindagar0 I appreciate your dedication here but every PR must go throw a review and get a lgtm and approve commands (from 2 different reviewers). https://kubernetes.io/docs/contribute/review/for-approvers/
We are also so late in the v0.28 that we probably don't want to include this in the release but rather wait to catch the next minor or patch release.

/lgtm

We can close the other one and
/hold this one until next minor/patch release

Sure @knelasevero , I have closed the other PR.
@a7i makes sense. Please do follow the required process and let me know if further changes required. Also any ETA like when the next patch will be release?

Sure @knelasevero , I have closed the other PR. @a7i makes sense. Please do follow the required process and let me know if further changes required. Also any ETA like when the next patch will be release?

We should have the Descheduler 0.28 release wrapped up this week so this PR should be merged prior to the end of this week. Worst case it will be released next quarter for Kubernetes 1.29, best case we need to cut out a release for 0.28.1

/approve
/remove-hold
Thank you @MarcPow and @nitindagar0 for your contribution :trophy:

Thanks a lot @a7i for all the help 👍

The image is available if you want to run it now: gcr.io/k8s-staging-descheduler/descheduler@sha256:373450514057e05a2d96533506ce0cf1f0350ca75d4459c84f081b548da39b0b
https://console.cloud.google.com/gcr/images/k8s-staging-descheduler/global/descheduler
but completely understand if you want to wait on an official release

Hi @a7i, Does this image also includes latest changes of master along with the fix?. Right now, we are using the image shared by your team earlier.

@nitindagar0 up to this commit: https://github.com/kubernetes-sigs/descheduler/commit/99246cd254643aa7a8ddbe41e5d51cfa1ecf5f16

Thanks @a7i  for sharing the image. We will update the image once official release is available.


Actions to edit state file are not using service principal but personal account logged in the system (az cli)

Hello,
Terraform version: 1.2.1
Terragrunt version: v0.38.4
I have 5 subscriptions:

shared
nonprod
prod

I am storing all state files of each nonprod/prod subscription in the shared subscription in a specific storage account.
Everything works fine when I try to do terragrunt apply/destroy/plan
Yesterday I tried to import a configuration on the state file and I encountered a warning and an error:
Warning: "use_microsoft_graph": [DEPRECATED] This field now defaults to `true` and will be removed in v1.3 of Terraform Core due to the deprecation of ADAL by Microsoft.
Error: Error building ARM Config: obtain subscription(subscription-id-i-don't-want-to-show) from Azure CLI: parsing json result from the Azure CLI: waiting for the Azure CLI: exit status 1: ERROR: Please run 'az login' to setup account.

After seeing this I checked with TF_LOG=DEBUG and I could see that the issue is right there: It's failing something in the "Testing if Service Principal / Client Secret is applicable for Authentication.."
Here is the log
terragrunt state list
2022-07-08T13:35:18.984+0200 [INFO]  Terraform version: 1.2.1
2022-07-08T13:35:18.984+0200 [DEBUG] using github.com/hashicorp/go-tfe v1.0.0
2022-07-08T13:35:18.984+0200 [DEBUG] using github.com/hashicorp/hcl/v2 v2.12.0
2022-07-08T13:35:18.984+0200 [DEBUG] using github.com/hashicorp/terraform-config-inspect v0.0.0-20210209133302-4fd17a0faac2
2022-07-08T13:35:18.984+0200 [DEBUG] using github.com/hashicorp/terraform-svchost v0.0.0-20200729002733-f050f53b9734
2022-07-08T13:35:18.984+0200 [DEBUG] using github.com/zclconf/go-cty v1.10.0
2022-07-08T13:35:18.984+0200 [INFO]  Go runtime version: go1.18.1
2022-07-08T13:35:18.984+0200 [INFO]  CLI args: []string{"terraform", "state", "list"}
2022-07-08T13:35:18.984+0200 [DEBUG] Attempting to open CLI config file: /Users/username/.terraformrc
2022-07-08T13:35:18.984+0200 [DEBUG] File doesn't exist, but doesn't need to. Ignoring.
2022-07-08T13:35:18.984+0200 [DEBUG] ignoring non-existing provider search directory terraform.d/plugins
2022-07-08T13:35:18.984+0200 [DEBUG] ignoring non-existing provider search directory /Users/username/.terraform.d/plugins
2022-07-08T13:35:18.984+0200 [DEBUG] ignoring non-existing provider search directory /Users/username/Library/Application Support/io.terraform/plugins
2022-07-08T13:35:18.984+0200 [DEBUG] ignoring non-existing provider search directory /Library/Application Support/io.terraform/plugins
2022-07-08T13:35:18.985+0200 [INFO]  CLI command args: []string{"state", "list"}
2022-07-08T13:35:18.986+0200 [INFO]  Testing if Service Principal / Client Certificate is applicable for Authentication..
2022-07-08T13:35:18.986+0200 [INFO]  Testing if Multi Tenant Service Principal / Client Secret is applicable for Authentication..
2022-07-08T13:35:18.986+0200 [INFO]  Testing if Service Principal / Client Secret is applicable for Authentication..
2022-07-08T13:35:18.986+0200 [INFO]  Testing if OIDC is applicable for Authentication..
2022-07-08T13:35:18.986+0200 [INFO]  Testing if Managed Service Identity is applicable for Authentication..
2022-07-08T13:35:18.986+0200 [INFO]  Testing if Obtaining a Multi-tenant token from the Azure CLI is applicable for Authentication..
2022-07-08T13:35:18.986+0200 [INFO]  Testing if Obtaining a token from the Azure CLI is applicable for Authentication..
2022-07-08T13:35:18.986+0200 [INFO]  Using Obtaining a token from the Azure CLI for Authentication
... fails

The above doesn't happen on a normal terragrunt output, which goes straight to the command output. Here's the log:
terragrunt output
2022-07-08T13:46:20.371+0200 [INFO]  Terraform version: 1.2.1
2022-07-08T13:46:20.371+0200 [DEBUG] using github.com/hashicorp/go-tfe v1.0.0
2022-07-08T13:46:20.371+0200 [DEBUG] using github.com/hashicorp/hcl/v2 v2.12.0
2022-07-08T13:46:20.371+0200 [DEBUG] using github.com/hashicorp/terraform-config-inspect v0.0.0-20210209133302-4fd17a0faac2
2022-07-08T13:46:20.371+0200 [DEBUG] using github.com/hashicorp/terraform-svchost v0.0.0-20200729002733-f050f53b9734
2022-07-08T13:46:20.371+0200 [DEBUG] using github.com/zclconf/go-cty v1.10.0
2022-07-08T13:46:20.371+0200 [INFO]  Go runtime version: go1.18.1
2022-07-08T13:46:20.371+0200 [INFO]  CLI args: []string{"terraform", "output"}
2022-07-08T13:46:20.371+0200 [DEBUG] Attempting to open CLI config file: /Users/username/.terraformrc
2022-07-08T13:46:20.371+0200 [DEBUG] File doesn't exist, but doesn't need to. Ignoring.
2022-07-08T13:46:20.371+0200 [DEBUG] ignoring non-existing provider search directory terraform.d/plugins
2022-07-08T13:46:20.371+0200 [DEBUG] ignoring non-existing provider search directory /Users/username/.terraform.d/plugins
2022-07-08T13:46:20.371+0200 [DEBUG] ignoring non-existing provider search directory /Users/username/Library/Application Support/io.terraform/plugins
2022-07-08T13:46:20.371+0200 [DEBUG] ignoring non-existing provider search directory /Library/Application Support/io.terraform/plugins
2022-07-08T13:46:20.371+0200 [INFO]  CLI command args: []string{"output"}
2022-07-08T13:46:20.378+0200 [INFO]  Testing if Service Principal / Client Certificate is applicable for Authentication..
2022-07-08T13:46:20.378+0200 [INFO]  Testing if Multi Tenant Service Principal / Client Secret is applicable for Authentication..
2022-07-08T13:46:20.378+0200 [INFO]  Testing if Service Principal / Client Secret is applicable for Authentication..
2022-07-08T13:46:20.378+0200 [INFO]  Using Service Principal / Client Secret for Authentication
2022-07-08T13:46:20.378+0200 [INFO]  Getting OAuth config for endpoint https://login.microsoftonline.com/ with  tenant some-tenant-id-which-you-shouldn't-know
2022-07-08T13:46:20.378+0200 [DEBUG] Obtaining an MSAL / Microsoft Graph token for Resource Manager..
2022-07-08T13:46:20.539+0200 [DEBUG] checking for provisioner in "."
2022-07-08T13:46:20.539+0200 [DEBUG] checking for provisioner in "/Users/username/bin"
2022-07-08T13:46:20.539+0200 [DEBUG] Building the Blob Client from an Access Token (using user credentials)
2022-07-08T13:46:21.413+0200 [DEBUG] Azure Backend Request:
... continues to output

Probably this is causing many other endpoints.
My configuration is this one:
./terragrunt.hcl
skip = true

terraform {
  extra_arguments "retry_lock" {
    commands  = get_terraform_commands_that_need_locking()
    arguments = ["-lock-timeout=20m"]
  }
  extra_arguments "azure_variables" {
    commands = concat(get_terraform_commands_that_need_vars(), ["output", "show"])

    env_vars = {
      ARM_SUBSCRIPTION_ID = local.azure_vars.subscription_id
      ARM_TENANT_ID       = local.azure_vars.tenant_id
      ARM_CLIENT_ID       = local.azure_vars.client_id
      ARM_CLIENT_SECRET   = local.azure_vars.client_secret
    }
  }
}
...
...
...
remote_state {
  backend                         = "azurerm"
  disable_dependency_optimization = true
  generate = {
    path      = "backend.tf"
    if_exists = "overwrite"
  }
  config = {
    subscription_id      = local.shared_vars.storage_account_subscription_id
    resource_group_name  = replace(local.shared_vars.resource_group_name, "prod", "nonprod")
    storage_account_name = replace(local.shared_vars.storage_account_name, "prod", "nonprod")
    container_name       = local.shared_vars.container_name
    key                  = "state/${get_path_from_repo_root()}/terraform.tfstate"
  }
}

generate "provider" {
  path      = "provider.tf"
  if_exists = "overwrite"
  contents  = <<EOF
provider "azurerm" {
  features {}
}
EOF
}

./resource-group/terragrunt.hcl
include {
  path = find_in_parent_folders()
}

locals {
  secrets_git = yamldecode(try(file("secrets_git.yaml"), file(find_in_parent_folders("secrets_git.yaml"))))
}

terraform {
  source = "/somerwhere/modules//rg"
}

This is a blocker if someone does not have direct access to the resources. This will not allow the user to import state using a service principal.

I don't have any experience with Azure so I don't think I can offer too much help as to whether you are configuring Terragrunt correctly for Azure, but my guess as to the problem here is that get_terraform_commands_that_need_vars doesn't include the state command, and thus the env vars you have configured are not being set when terraform is called.

This solved the issue.
Thanks for the quick reply!


Persistence Issue

Hi, awesome job with this library! I'm running into an issue when persisting the user between app loads. If I start the app for the first time and log in with a user, everything is good. But when I try to reload the app or restart the simulator, its calling the onLoginNotFound callback.

These are my dependencies
  "dependencies": {
    "react": "15.4.1",
    "react-native": "^0.39.2",
    "react-native-cli": "^2.0.1",
    "react-native-facebook-login": "^1.4.0",
    "react-native-maps": "^0.12.2",
    "react-redux": "^5.0.1",
    "redux": "^3.6.0",
    "redux-thunk": "^2.1.0"
  },
  "devDependencies": {
    "babel-jest": "18.0.0",
    "babel-preset-react-native": "1.9.1",
    "jest": "18.0.0",
    "react-test-renderer": "15.4.1"
  },

ios or android?

ios

is the keychain capability toggled on for the project?

Yes, but I'm not sure about that red flag

someone had a similar issue and it had to do with the keychain. i mostly deal with the android side so not very sure how to resolve it swiftly.
but seems like the app id in the apple developer console is where you should also toggle it on

hmm okay, I'll try to fix that issue thanks!

let me know if it resolves your issue, ill leave this open, in the event someone else can help as well

I've added the keychain sharing, but the issue is still there.

yea not entirely sure what's causing it. issue #166 seemed to have a similar issue and was resolved

Okay, I'll try to keep looking for a better solution.

Any update?


chore(jest-types): expose type CacheKeyOptions for getCacheKey

Summary
Expose  type CacheKeyOptions for getCacheKey
Test plan
Green CI

@SimenB this should be the only type left that ts-jest needs.


Don't delete local temp backup after restore

My current database takes about 10 minutes to fetch from Heroku.
If the pg_restore fails for any reason, I don't want to re-fetch the dump.
curl -o will already overwrite tmp/latest.backup
when it begins and tmp is ignored in Git by default in Rails apps.
It might be nice if Parity had a command to "restore from local backup"
but in the meantime, this worked well for me:
#!/bin/bash
set -euo pipefail

db="MYDB_development"

dropdb --if-exists "$db"
createdb "$db"
time pg_restore tmp/latest.backup --verbose --no-acl --no-owner --dbname "$db"
psql "$db" -c "UPDATE ar_internal_metadata SET value = 'development' WHERE key = 'environment'"

That's a good point about helping users of the gem clean up.
I think I'm personally going to switch from Parity to individual shell scripts that live in the repo and have no arguments, as documented below. If any of these ideas are useful for the maintainers, perhaps other PRs could be opened.
Create backup of production follower:
./bin/db_backup_prod_follower

Restore staging from production backup. The script puts staging in maintenance mode and post-processes some data via heroku pg:psql:
./bin/db_restore_staging_from_prod_backup

Create backup of staging:
./bin/db_backup_staging

Download staging backup:
./bin/db_download_staging_backup

Restore development from downloaded backup:
./bin/db_restore_dev_from_downloaded_backup

Optionally, clean up:
rm tmp/latest.backup

I haven't been able to use Heroku PG for almost a year now, so I've still found Parity very useful for general app management, but the PG cases have gotten quite complicated.


fix: temporarily block user input on start

:tada: This PR is included in version 3.1.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Incentive taxable options do not have labels

This makes it tricky to use inputs browser to identify SSC variable names for LK or SDK: https://sam.nrel.gov/forum/forum-general/3209

Solution is to add variable labels in the IDE.

Fixed in 96e34cc81


luaPackages.image-nvim: correct magick dependency

Description of changes
This is an attempt at fixing luaPackages.image-nvim (and thus vimPlugins.image-nvim).
The script has detected that this lua module was depending on magick.
However, luaPackages.magick is broken as it only works with JIT.
Thus we should use luajitPackages.magick.
Also, I got rid of the magick.patch because I think that it is not relevant anymore.
cc @donovanglover
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The plugin is not able to find magick. Isn't adding luajitPackages.magick to propagatedBuildInputs equivalent to the former patch ?

In the end, I left the patch and now the plugin works fine.

Seems useful however I'm not sure why the plugin "works on my machine" without this change

Nevermind I see it's broken in nixos-unstable-small. Tested with your branch and it fixed it for me, thanks :+1:

LGTM however I'm not sure what caused it to break in the first place

The vim plugin broke because image.nvim has been added to the repo of luarocks plugin.
Hence, the lua packages update script has automatically added it to the luaPackages set as luaPackages.image-nvim.
However, this automatically created packages has never worked because it depends on luaPackages.magick which is broken (only luajitPackages.magick works).
Then, because luaPackages.image-nvim now exists, the vim plugin update script has automatically shifted from buildVimPlugins to buildNeovimPlugins for vimPlugins.image-nvim and made it depend on luaPackages.image-nvim.
This patch fix/unbreak luaPackages.image-nvim which lets the vim plugin build fine too.

this is the part I dont get. It should depend on whatever lua neovim is using, by default luajit.
Also if we dont test this, it's bound to happen again. The current fix makes no sense, we pull luajitPackages even when using lua5.3 ?

I agree with you. I have no idea whether this fix actually makes the lua package works.
The only known benefit that it has, is "unbreak" the lua package so that the vim one is allowed to be instanciated along with its proper patch.
What is the context in which a user would be using one of the luaPackages ? Can they be used as independent applications ?

I am still facing the issue of image-nvim trying to get lua5.1-magick. Maybe it's my home-manager that is interfering but I'm not sure how to resolve this issue.


Improve KimCNN results

One and a half years later, I'm finally getting better results on KimCNN using the original hyperparameters in the paper. There are a few discrepancies with the PyTorch and Castor implementation:

Kim used an Adadelta Rho of 0.95 instead of 0.9. The paper did not mention this.
Kim used Xavier uniform initialization for the convolution layers. The paper did not mention this.
Kim did not use the equivalent of torchtext's BucketIterator. This is a difference in Castor.
Kim used the dev loss as the criterion for model selection. This is a difference in Castor.

After these changes, the original hyperparameters in the paper work quite well. I'm getting 87.8 for SST-2 multichannel now, which is an improvement over the current 87.4. It's still a bit off from the paper result of 88.1, though.
Reference: https://github.com/yoonkim/CNN_sentence/blob/master/conv_net_sentence.py

Seems like I spoke too early. Results fluctuate from high 85s to 87s.


java.lang.NoSuchMethodError: com.auth0.jwt.JWT.decode(Ljava/lang/String;)Lcom/auth0/jwt/JWT;

I'm trying to decode my JWT using this code:
  private static DecodedJWT decodeJWT(String token) {
    return JWT.decode(token);
  }

and getting these error:
...
Caused by: com.google.common.util.concurrent.ExecutionError: java.lang.NoSuchMethodError: com.auth0.jwt.JWT.decode(Ljava/lang/String;)Lcom/auth0/jwt/JWT;
...

I'm using Java JWT 3.2.0. Please Advice

I can't reproduce it. Any other info you can provide?

Hi @lbalmaceda,
I'll provide the context of what actually happened. I provide a library that depends to java-jwt version 3.1.0. Let's call this auth library.
@acaaad in his build.gradle, declared a dependency to my auth library. Somewhere in his repo, there's another module that depends to java.jwt version 3.2.0. Thus, the java.jwt library loaded at runtime is 3.2.0.
In my library I call this method:
com.auth0.jwt.JWT.decode(token)
In version 3.1.0, this method returns a JWT class that implements DecodedJWT interface. But, in version 3.2.0, JWT class doesn't implement DecodedJWT. The above method returns DecodedJWT.
Thus, at runtime we got the above error because JWT.decode doesn't return JWT anymore.
Question is @lbalmaceda, is there any reason why JWT doesn't implement DecodedJWT interface anymore? In this case it's not backward compatible. If you guys are using semantic versioning, minor increment should preserve backward compatibility.

Reading the commit: https://github.com/auth0/java-jwt/commit/72bceb977586a1736035ccc74220d14bc466f0c4#diff-8ad89ade07b642e1a02866acf5562b86
the intent was probably to make the JWT class the central utility object for JWT library.

@oktaviandi-nugraha Do you have any example of a library dependend on java-jwt:3.1.0?

@jspetrak My library is an internal library at work.

@oktaviandi-nugraha @acaaad Ok that makes more sense. We changed it here to avoid confusion. As @jspetrak said we wanted to have a single entry point (the JWT class) with a different output class. Although it's true this is breaking, it's not as breaking as removing methods or changing behavior. It's fixed by just by changing the returned class from JWT to DecodedJWT like the PR says, the rest remains the same. So your options here are:

Make both libraries use the same version of java-jwt. (honestly, it shouldn't take more than 2 mins to update your "auth library")
Request java-jwt 3.2.0 in the library and exclude the java-jwt 3.1.0 required by your "auth library" dependency, using some of the techniques explained in gradle docs.
Apologies for the confusion.

Hi @jspetrak @lbalmaceda thank you for your answer.
We finally made it work by making this authentication library depends on java-jwt version 3.2.0
Will close this issue now.


Typo in PDO::purgeOldCodesFromDatabase() in master

There is a namespace problem in purgeOldCodesFromDatabase(), in that it refers to PDO::PARAM_INT on line 448. This should be \PDO::PARAM_INT instead, so that the built-in PHP PDO class gets used.

Fixed in #96

Maescool fixed in #96 - thanks!


Expose ignore taints

How to categorize this PR?

/area auto-scaling
/kind api-change
What this PR does / why we need it:
This PR exposes the --ignore-taint flag of Cluster Autoscaler on Shoot API with which certain taint values can be let known to CA to be ignored.
Which issue(s) this PR fixes:
Fixes #5166
Special notes for your reviewer:

I want to know if there should be any validation to be added for this new field that is introduced.
Somehow make verify does not seem to work on my local system. It may fail in the pipeline as well.

Release note:

Cluster Autoscaler flag `--ignore-taint` is exposed at Shoot API level with which user can specify a taint to ignore in node templates when considering to scale a node group.

/invite @rfranzke @ialidzhikov

I need help to fix the build error. I am unable to comprehend the error
+ make check-generate verify
14:32:50
> Generate / Vendor Check
14:32:51
>> make generate
14:37:21
make generate needs to be run:
14:37:21
 M pkg/openapi/openapi_generated.go
14:37:22
make: *** [Makefile:182: check-generate] Error

My Mac is crashing everytime I run make verify on it.

Working on resolving conflicts.

/retest


Express with SSL example

This is an example how to use Express with SSL enabled.

Now this PR contains a simple example demonstrating only SSL.


Add GKE Autopilot support

Docs and configuration changes to support GKE Autopilot operation mode. The change depends on a kubelet read-only endpoint support which was added to OpenTelemetry Collector in 0.41.0

in autopilot, is running it as user root possible?

in autopilot, is running it as user root possible?
yes, it is


Add release note

Add release note for PR#701.

Codecov Report

Merging #705 (cd76248) into main (9f7bb5a) will decrease coverage by 0.10%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #705      +/-   ##
==========================================
- Coverage   72.33%   72.24%   -0.10%
==========================================
  Files          39       39
  Lines        5436     5436
  Branches     1019     1014       -5
==========================================
- Hits         3932     3927       -5
- Misses       1221     1224       +3
- Partials      283      285       +2

see 1 file with indirect coverage changes


Extending size of persistent storage in AKS

Hi
In AKS it is possible to extend the size of the persistent volumes just by changing the PVC value of “resources.requests.storage”.
After a little while persistent volumes are extended with the new size.
When I try so use the helm chart to update the PVC through volumeClaimTemplates:, using the override:
persistence:
qmPVC:
  enable: true
    size: 12Gi
I get the following error:
"Error: UPGRADE FAILED: cannot patch "nativeha-s1-ibm-mq" with kind StatefulSet: StatefulSet.apps "nativeha-s1-ibm-mq" is invalid: spec: Forbidden: updates to statefulset spec for fields other than 'replicas', 'template', 'updateStrategy', 'persistentVolumeClaimRetentionPolicy' and 'minReadySeconds' are forbidden"
It seems to be a restriction with helm that volumeClaimTemplates can’t be changed?
If I do not use the helm chart to maintain the PVC’s, the helm chart will be out of sync with the live values. Will that be a problem?
Can I circumvent the problem somehow ?
Rgds
John Barbesgaard

As you mention this is a generic Kubernetes StatefulSet issue. There is an interesting discussion in the community across several GitHub issues. I’ve included links for reference:
Enhancement Issue 661
Issue 68737
There is an active proposal for an enhancement being work here.
Which will hopefully improve the situation in the future, but until this is provided by Kubernetes we will need to fall back to a workaround similar to what you mention, and discussed in the issues above.
Updating the PVC under the covers shouldn’t cause any issues but you will need to assure the associated pods are disconnected from the storage for an adequate time for the resize to occur.
Closing the issue at this time, unless any additional information is required. Feel free to reopen as needed.


framework's strength and sys requirement

Hello, I just want to know about this framework's system requirement and its scalability along with how many users does this support, and how many transactions per second are supported?
Thanks in advace

Hello, we only provide Chinese version at the moment, so maybe not that suitable if you do not trade in China.


ImportError: No module named crab

from scikits.crab import datasets
Traceback (most recent call last):
File "<pyshell#0>", line 1, in
from scikits.crab import datasets
ImportError: No module named crab
what's wrong with this commend
I have installed crab by easy_install successfully

Is crab still alive?

Thank you @Flowerowl  , it's fixed :+1:

Thank you @Flowerowl . it really helped. I have documented my steps here to:
http://datapandas.com/index.php/2016/08/21/importerror-no-module-named-scikits-python-fix-jupyter-notebook/
but i am unable to get it work on Spyder IDE. Any help? Thanks


Issue523 idle sleep mode calibration

Fixes #523
Now the rows in which the idle sleep mode is activated are removed from the data before calculating the calibration coefficients. The test file for the g.imputeTimegaps function is extended.

Checklist before merging:

[x] Existing tests still work (check by running the test suite, e.g. from RStudio).
[x] Added tests (if you added functionality) or fixed existing test (if you fixed a bug).
[x] Updated or expanded the documentation.
[x] Updated release notes in inst/NEWS.Rd with a user-readable summary. Please, include references to relevant issues or PR discussions.
[x] Added your name to the contributors lists in the DESCRIPTION and CITATION.cff files.

Codecov Report

Merging #524 (63f936e) into master (1c5adff) will increase coverage by 0.02%.
The diff coverage is 100.00%.

:exclamation: Current head 63f936e differs from pull request most recent head 04ef95c. Consider uploading reports for the commit 04ef95c to get more accurate results

@@            Coverage Diff             @@
##           master     #524      +/-   ##
==========================================
+ Coverage   73.23%   73.26%   +0.02%
==========================================
  Files          93       93
  Lines       11219    11223       +4
==========================================
+ Hits         8216     8222       +6
+ Misses       3003     3001       -2

Impacted Files
Coverage Δ

R/g.calibrate.R
67.37% <100.00%> (+0.26%)
:arrow_up:

R/g.imputeTimegaps.R
100.00% <100.00%> (+5.88%)
:arrow_up:

Continue to review full report at Codecov.

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


Transfer slides individually and in low/high res variations

Once #8 is done, slide images should be sent one by one to the phone, starting with low-res versions, followed by high-res. This will make things more responsive without hindering times when the connectivity is strong.

Closed by #18.


Memories Off 6 Complete [PCSG00176]

App summary

App name: Memories Off 6 Complete
App serial: PCSG00176
App version: 1.00

Vita3K summary

Version: v0.1.6
Build number: 2841
Commit hash: https://github.com/vita3k/vita3k/commit/0c75bbb5
CPU backend: Dynarmic

Test environment summary

Tested by: nishinji
OS: Windows 10
CPU: Ryzen 9 3950x
GPU: RTX 3060ti
RAM: 32 GB

Issues
Crash
Screenshots

Log
vita3k.log
Recommended labels
Menu
Crash

as of https://github.com/Vita3K/Vita3K/commit/8d345f002187fb9e619fc386bd7a73ea64a393dc, the game is mostly ingame. In Vulkan, only the 2nd part is playable, while the 1st part of it simply crashes at the FMV, giving it a exception violation. On OpenGL though, the 1st part and 2nd part is working, although if you skip the FMV early it will give you a exception violation.
Screenshots

Logs
vita3k (1;2 OpenGL).log
vita3k (2;2 OpenGL).log
vita3k (1;2 Vulkan).log
vita3k (2;2 Vulkan).log
Recommended Labels

Playable
graphics bug
vulkan issue


Add ep14

Ongen tuika yoro.
:pray:

[ ] Set audio_file_size
[ ] Set duration

:eyes:

@miyaoka Thanks. I uploaded 14.mp3


RFC: eslint-config-standard-with-typescript to depart from standard

Background
eslint-config-standard-with-typescript (-with-typescript) was first published September 2017. Two years later, August 2019, the repository was transferred into the standard organization. We have enjoyed and appreciate the contributions by various members of the community, yet, I and later @rostislav-simonik and @jay-bulk have been the regular contributors since.
Popularity of -with-typescript has been on the rise since the beginning of 2020, with a significant increase in adoption rate since July 2022, which is about the time the mob was formed.
Policy discrepancy
Standard always had a certain approach with regards to the scope and impact of rules and the frequency of releases. And -with-typescript always had quite a different approach. For example, -with-typescript wasn't shy in making frequent major releases that introduced new rules. The reasoning has been that of continuous delivery, semantic commits/releases and the idea that users would be able to bump the version at their own pace.
Suggested solution
-with-typescript will depart from its dependency on eslint-config-standard and therefore depart from its relationship with Standard. This would permit -with-typescript the freedom to introduce such changes that would currently be incompatible with standard. For example, new core ESLint (non-TypeScript) rule.
Of course, We will continue to welcome asynchronous contributions from outside the regular mob sessions.
Co-authored-by: @rostislav-simonik

This makes sense to me. As I have been following things lately I have noticed this is the central tension in the project so I am glad to see there is a way everyone can get what they need/want out of it.

I'm 👍 on this, thanks for suggesting 🙏 There are some practicalities to it I guess, so lets sync up on those so that we are all on the same page as to what this would entail.
This is what I'm personally thinking of, from the top of my mind:

eslint-config-standard-with-typescript, both npm module and repository, will be transferred to the new organization and use existing names as the new org see fit
ts-standard, both npm module and repository, will be given the same treatment (else deprecated as per https://github.com/standard/ts-standard/issues/291)
Currently the VSCode support is given through (standard/vscode-standard)[https://github.com/standard/vscode-standard], if the new organization takes on ts-standard, then maybe it wants to have its own copy of standard/vscode-standard as well? To remove dependency on standard/vscode-standard. If so, then such a transition should be assisted with by standard/vscode-standard I think (Thoughts @LinusU, @theoludwig?)

Any other practicalities I'm missing? Any other modules or npm-packages that would need to be moved?
Also: Would the new org still rely on eslint-config-standard or become a self-contained module which copies the current rules over but after that keeps its own maintenance of those? (I think personally this would give the most freedom to the new org)

Also: Would the new org still rely on eslint-config-standard or become a self-contained module which copies the current rules over but after that keeps its own maintenance of those? (I think personally this would give the most freedom to the new org)

The latter.
Otherwise, sounds good.

@rostislav-simonik and I would like to implement this RFC (depart eslint-config-standard-with-typescript from @standard). We find it reasonable at this point to announce our intention to implement this RFC after a certain future date: 2024-01-12. DMs will be sent to @feross and @mcollina.

I'm ok with it, +1.

The new home is https://github.com/mightyiam/eslint-config-standard-with-typescript .

Could someone please make me owner of the eslint-config-standard-with-typescript npm package? @voxpelli perhaps?

My npm username is mightyiam.

I'm not an admin / owner on npm it seems, ping @LinusU, @bcomnes, @feross

Sorry for the delay here. Ok I removed the package eslint-config-standard-with-typescript from the standardjs developers team, and mightyiam and rostislav-simonik remain as maintainers.
Does that look right to everyone?

@bcomnes thank you. If the 'maintainer' role means full control, then that seems complete. Could you, perhaps do the same for ts-standard?

@mightyiam I tried adding you as a maintainer/collaborator on that package on npm but get a forbidden error. As an admin I am unable to invite you back to the org.

@feross perhaps you could? @bcomnes could you perhaps ping some who do have permission to do this?

@bcomnes Essentially we should do the same thing with ts-standard as we did with eslint-config-standard-with-typescript and simply give it over to @mightyiam and remove it form the standard organization – on both npm and on GitHub

The difference here's that he's no longer on the package as a collaborator, and I'm unable to add outside collaborators as an admin. J think only owners can do this.

@bcomnes a problem on both npm and GitHub or just npm?

Npm

ts-standard probably belongs together with eslint-config-standard-with-typescript.
I've been making progress there.
So, I would say, yes.


make mice not eat uranium bananium and pills

About the PR
fixes #16149 by adding BadFood marker component and gives it to uranium, bananium ore and pill items
ai will ignore food with BadFood
afaik no ore is edible so ore wasnt an issue, material sheets (uranium) were
Media
mouse not eat pill or uranium

mouse still eat breab

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl:

fix: CentCom's Ministry Of Truth has trained station mice to stop eating uranium.

Mice squad pulling up to salvage to eat the entire shift's supply of bananium:

Mice eating pills is funny leave that in

downside of that is you can do mouse genocide by putting radium 20u pills at every mouse spawn so they instantly suicide

Just remove food from bananium and uranium tbh.
Sometimes mice are used to deter chemistry from entityspam so might need to talk about that one.

downside of that is you can do mouse genocide by putting radium 20u pills at every mouse spawn so they instantly suicide

I’m 70% sure that spawncamping mice like that breaks powergaming/metagaming rules

Just remove food from bananium

Its called bananium for a reason

literally 1984

Sometimes mice are used to deter chemistry from entityspam so might need to talk about that one.

would suffix: Admeme mouse that ignores BadFood be ok for this fun activity

Sometimes mice are used to deter chemistry from entityspam so might need to talk about that one.

would suffix: Admeme mouse that ignores BadFood be ok for this fun activity

uhhhh sure though the pills one I'm still suss on.

@metalgearsloth admeme mouse that ignores bad food added, also despawns after a minute so you dont need to clean them up manually


Unexpected Node Error on node-red-contrib-play-soundfile

Current Behavior

Unexpected Node Error

Info:
Node object is not a node-red Node
Stack:
Error: test
at log_helper (/usr/src/node-red/node_modules/@node-red/runtime/lib/nodes/Node.js:512:20)
at Play_audio_file.Node.log (/usr/src/node-red/node_modules/@node-red/runtime/lib/nodes/Node.js:531:5)
at new Play_audio_file (/data/node_modules/node-red-contrib-play-audio-file/play_audio_file.js:4:14)
at Object.createNode (/usr/src/node-red/node_modules/@node-red/runtime/lib/flows/util.js:90:27)
at Flow.start (/usr/src/node-red/node_modules/@node-red/runtime/lib/flows/Flow.js:206:48)
at start (/usr/src/node-red/node_modules/@node-red/runtime/lib/flows/index.js:394:33)
Please report this issue, including the information logged above:
https://github.com/node-red/node-red/issues/

Expected Behavior
No response
Steps To Reproduce

install node-red-contrib-audio
drag mp3-node into board
fill in a mp3
compile

you get an error
Example flow
paste your flow here

Environment

Node-RED version:
Node.js version:
npm version:
Platform/OS:
Browser:

Thanks for raising.
This will eventually need to be reported against node-red-contrib-play-soundfile itself to get fixed. The error asks for it to be raised here first so we can take a look to see what that node is doing to cause this error.
Will update when we have a chance to look properly.

Actual node is node-red-contrib-play-audio-file which doesn't actually have a project on github at the URL in the package.json

This node was:

published 6 years ago
never updated
the github URL is broken so no way to report errors (after a bit of npm digging into the publisher I found a project [here]((It may have been moved to https://github.com/LeanBI/node-red-contrib-play-audiofile) but still it's not been touched since originally published)
will always fail as it's first line of useful code is trying to call this.log before the node has been initialised with RED.nodes.createNode(this,config);

I suggest we remove it from the catalog and the OP should uninstall the node as it is unlikely to ever be fixed.

PR raised against upstream project. Suggest if not merged/published in 2 weeks(?) remove from catalogue.


Fix typo

Not sure if this is what you meant, but I'm pretty sure that the current revision isn't what you meant.

+1, I had to read that a couple of times to get it.


Is the product amount updated?

I want to use this project for potentially one-off products. I understand the whole concept is serverless but is there a way to decrement the product amount after a purchase?
edit: Maybe using firebase.io for product details instead of static hosting?

No, that's the big limitation of this approach. If you have high-demand low-inventory products you'll probably be better off with a traditional e-commerce backend.
Depending on your use case, you might consider using Flatmarket with auth and capture enabled (see capture in the spec). This splits a sale into two steps where the seller must manually accept the payment from the Stripe admin. That way if you receive orders you can't fulfill, you can resolve them with the customer without charging them.


gulp work

cleanups and simplifications to the gulp file that I cam across while working on the file.

looks good

Awesome work... as I mentioned let's just make sure to test with runembed task first .

I think this is the PR in question @mforsyth https://github.com/twosigma/beaker-notebook/pull/1723

Ahh, you are right @samccone


feat: add spec port

What type of PR is this?

[ ] bug
[ ] feature
[ ] enhancement

What problem(s) does this PR solve?
Issue(s) number:

fix #275

Description:
How do you solve it?
Special notes for your reviewer, ex. impact of this fix, design document, etc:

@wenhaocs The ports exposed in spec looks good,  if the default value updated then add a flag to cmd line to overwrite local conf file.


2D Array - hourglass Java from hackerRank

Please take reference to this:
https://www.hackerrank.com/challenges/2d-array/problem
I would like to work on this issue, and would be glad if you could please assign it to me.
Thanks,
Zaahra.

Create a new folder 2DArray inside Array folder add ReadMe.md not 2DArray.md as well.


Logout Issue

Hi Deborah, really appreciate your effort ... but i think you missed something very important to close this track ---->  "Logout" it's really very important to me... should you add this part please

This issue is over 2 years old. I assume that it has been resolved or is no longer relevant. I'm closing this issue. Please reopen if needed.


updated latest icons by running composer update

Updated packages to include all the latest icon sets.

Thanks!