Consider improvements for passing values to queries

There have been some ideas and discussions about improving the way values are passed to queries.
The purpose of this issue is to gather all of them in one place and decide what to do.
Unsigned values
As described in #409, our driver doesn't support passing unsigned integer values like u64 in a query.
This could be solved by implementing the Value trait for them, but we aren't sure if that's the best way to go about it.
When given an u64 value the driver should probably cast it to i64 and send it to the database, but it isn't exactly clear what should happen if the u64 value doesn't fit in i64.

We could throw a runtime error, but that could be a hidden trap for users who happily passed an u64 thinking nothing can go wrong.
Or we could send it as a BigDecimal (as one of users on Slack suggested) but that's way less intuitive.

Another problem is that this makes our API even less strongly typed - someone might pass an u64 by mistake and think that the compiler wouldn't allow anything like that.
New interface for Value?
It might be a good idea to extend the interface of Value trait.
Currently it looks like this:
pub trait Value {
    fn serialize(&self, buf: &mut Vec<u8>) -> Result<(), ValueTooBig>;
}

We could add a field to specify the target column type:
pub trait Value {
    fn serialize(&self, buf: &mut Vec<u8>, target: &ColumnType) -> Result<(), ValueTooBig>;
}

This way serialize would know what kind of column it's serializing for.
Given a big u64 value it could decide to output either i64 or BigDecimal depending on the target column type.
Additionally this would help avoid mistakes with passing incorrectly sized values, e.g passing i32 instead of i64.
Should user pass column types along with values?
To make the new interface possible we would have to get the type of columns from somewhere.
One way  would be to just force the user to specify types along with the values. That would make the API even more strongly typed, but harder and more verbose to use.
Another way would be to use metadata returned from the database after preparing a statement. PreparedMetadata seems to contain the information we need. For unprepared queries we could just prepare them under the hood, they aren't performance sensitive anyway.
AFAIR I think python driver does some local type checking, maybe they have figured out how to do this already and we could replicate their solution.
### Tasks
- [ ] https://github.com/scylladb/scylla-rust-driver/issues/800
- [ ] https://github.com/scylladb/scylla-rust-driver/issues/801
- [ ] https://github.com/scylladb/scylla-rust-driver/issues/802

New interface for Value?
Or, a new trait.

I also definitely agree that we should use prepared statement metadata if available - additional validation is one of the reasons for its existence in the protocol. We should probably make the option opt-in or opt-out, to allow performance-sensitive workloads to skip additional runtime checks.

I sat and pondered on this task for a little while, and tried to specify it a bit more and split into subtasks. I see it like this:
Getting rid of the Session::query problem
For unprepared statements, we don't have a way to determine column names and types of the bind markers. Obtaining this info requires parsing the statement and knowing the current schema - which we could theoretically do on the driver, but it's a huge effort and I'm not sure cannot be done without introducing races. In the case of prepared statements, this information is available because it is computed by the DB and returned in the response to the prepare request.
I see two ways out:

Before sending an unprepared statement, quietly prepare it first to obtain the information about the bind markers (it's only necessary to prepare on one node). This increases latency of unprepared queries twofold, but using unprepared queries is a performance antipattern anyway. We could provide an optimization: if the list of values to be bound is empty, we just send it as an unprepared query.
Remove the values argument from the Session::query and only allow for queries without bind markers. This is simpler to implement than 1. If the user wanted to send an unprepared query with some bind markers, now they will have to prepare it beforehand - latency-wise it will be the same as for 1., though 1. does not have to prepare the statement on every node which would be a win if somebody wants to execute the query only once.

Add new serialization traits
Let's use the convention introduced in the not-yet-done deserialization refactor and let's name the new traits SerializeCql and SerializeRow:
pub trait SerializeRow {
    fn serialize(&self, ctx: &RowSerializationContext<'_>, out: &mut impl std::io::Write) -> Result<(), std::io::Error>;
}

pub trait SerializeCql {
    fn serialize(&self, typ: &CqlType, out: &mut impl std::io::Write) -> Result<(), std::io::Error>;
}

The RowSerializationContext should contain information about the bind markers: column names and their types. Later, we may add more stuff there.
As this change will affect all Session::query and Session::execute calls which should be ubiquitous in the user codebase, we should make it as easy as possible to migrate to the new API, preferably step by step.
Current serialization API is heavily based on SerializedValues type which is an untyped container for serialized values. Session::{execute,query} receive the argument list as an impl ValueList type - which is just a trait that allows to convert the type to SerializedValues. In general, we should implement SerializeRow for each type that implements ValueList, which also includes SerializedValues.

Calls like session.execute(&prepared, (1, 2, 3)) should work automatically out of the box as we will implement SerializeRow for types that implement SerializedValues, and SerializeCql for types that implement Value. This should be the most common case.
In a generic context like session.execute(&prepared, generic_list) where generic_list: impl ValueList, the piece of code should be adjusted so that generic_list.serialized()? is called instead - which will convert to SerializedValues which does implement the new SerializedRow.
For the case of user impls of ValueList and Value, we can provide macros (not necessarily procedural) that generate a SerializedRow/SerializedCql implementation based on the existing ValueList/Value.

It will be necessary to update some examples and documentation, but I don't expect it to be much work compared to the deserialization refactor.
Macros for the new traits
The new SerializeRow and SerializeCql traits will need to have their corresponding procedural macros implemented. Those macros should match struct fields to columns/UDT fields by name. This should be done by default, but there should be an attribute that allows disabling it.

@cvybhu @havaker @wprzytula @Lorak-mmk thoughts? If there are no objections I will create sub-tasks.
If we go with way 2. in "Getting rid of the Session::query problem" (which I prefer) then we can probably work on it in parallel with the others. "Macros for the new traits" should be done after "Add new serialization traits".

We could provide an optimization: if the list of values to be bound is empty, we just send it as an unprepared query.

This is a great idea, if only we settle on implementing 1st solution. I am also for the 2nd, though.

for types that implement SerializedValues

You most probably meant ValueList.
My thoughts are mostly positive. Let's do it!

One possible problem with 2nd solution: it may encourage people to perform textual parameter substitution as they can't use query parameters - leding to security vulnerabilities (noSQL injection)

One possible problem with 2nd solution: it may encourage people to perform textual parameter substitution as they can't use query parameters - leding to security vulnerabilities (noSQL injection)

Hmm, right. Maybe the 1st option is better after all. Another quite convincing argument in its favor that I can see right now is that it will just break less of the existing code.

I've created the sub-tasks. If there are some subtask-specific issues to discuss, then let's do it on the subtasks.

@piodul Can we close this now, or is there more to implement?

To be fair, I think we've reduced the scope of the issue a little while we were working on 0.11 release. We did all that was necessary to ensure type safety, but the issue actually starts by describing a different problem and improving type-safety was a side effect: namely, it suggests making it possible to do things like pass integers of one size into the database column of other size and let the driver handle down/upcasting in a safe way.
We already have https://github.com/scylladb/scylla-rust-driver/issues/409, so I think we can reuse that issue for the sake of the discussion whether we should implement support for such casts at all (personally I'm not a big fan of the idea).
I'll close this issue.


Add extended metrics ( from logs )

We cannot have metrics by OriginIP on the default metrics, we need
to fetch logs ( last 5 minutes ) and create metrics from those.
We have now metric for each Cloudflare-DC, to each Origin-IP
We have the OriginResponseTime, and a count.

Got it with your explanations!
Still the numpy thing for which we should find a better solution IMO.

I add a TODO, is it ok for you ?


Synchronize the remote database to the repo

We want to be able to open up the Program database to contributions that take advantage of GitHubs security and contribution features. This will synchronize periodically with the database.
Assign after we have the database structured settled.

Database structure is in a good place and some needs to synchronize the database to the repo have arisen!
We discussed the need to:

Pull all scripts down to make global changes (e.g., updates to repo name (looking at you #39)) and push back to the database.
Have a more reliable way to back up the database outside of the admin panel.

Currently, ElephantSQL compresses the backup of the database nightly for at least a months worth. It is compressed as an .lzo file (https://www.lzop.org). To date, I've downloaded the occasional file and uploaded to our Google Drive which is helpful, but not ideal. ElephantSQL does have a built in "restore" function from any of the listed backup, or you can run a terminal command to restore from an older backup?

Assigning to @jbphet to investigate or ask for additional details.

I believe this is complete in its first iteration. There is a way to backup to GitHub and a script for devs to push changes to programs from the IDE back up to the database.
Anything else for the initial goals of this issue @jbphet ?

I think we're good.  Closing.


Saving?

Is it possible yet to save, if so tell me how.

@KosmicJ not currently, but this is definitely an important feature I'd like to have.
There are some other existing voxel saving modules out there, e.g. https://github.com/maxogden/voxel-level saves using level.js, but I haven't looked into integrating them yet. Voxel world data could hypothetically be stored in localStorage/indexdb/webfs on the local browser instance. But what I'd really want to do is have the world stored on and accessed from a remote server: GH-26 Server

Any news on this?  This is certainly a great repo to play around with and hack on, but it really needs saving.  Perhaps you could implement local saves for now, and then once that's done work on a global save server?
Thanks!  (I really like this btw :+1: :100: :1st_place_medal: :smile:)

@StickyFingies No news on this feature (saving), I never did complete it... https://github.com/voxel/voxelmetaverse/issues/26 covers my investigations into connecting voxelmetaverse to a server (almost the same feature) and there was some progress but nothing solid. This was one of the biggest flaws of this project, in my opinion. I focused a lot on the frontend, the UI, neglecting the data model, the backend, needed for persisting the world.

@StickyFingies No news on this feature (saving), I never did complete it... https://github.com/voxel/voxelmetaverse/issues/26 covers my investigations into connecting voxelmetaverse to a server (almost the same feature) and there was some progress but nothing solid. This was one of the biggest flaws of this project, in my opinion. I focused a lot on the frontend, the UI, neglecting the data model, the backend, needed for persisting the world.


Running npm start after new bot creation with npx fails

Bug Report
Current Behavior
I am following this guide https://probot.github.io/docs/development/#running-the-app-locally
I have successfully run: npx create-probot-app todo-bot, with the following values
? App name: todo-bot
? Description of app: get stuff done!
? Author's full name: Simon
? Author's email address: <sanitized>
? GitHub user or org name: SimonTheLeg
? Repository name: todo-bot
? Which template would you like to use? basic-js => Comment on new issues

I have not touched any other files so far.
Now when I try to run npm start it returns the following error:
❯ npm start

><EMAIL_ADDRESS>start /Users/simonbuilds/streams/02_gh_bot/todo-bot
> probot run ./index.js

(node:60034) UnhandledPromiseRejectionWarning: Error: [@octokit/auth-app] appId option is required
    at Object.createAppAuth (/Users/simonbuilds/streams/02_gh_bot/todo-bot/node_modules/@octokit/auth-app/dist-node/index.js:463:11)
    at getState (/Users/simonbuilds/streams/02_gh_bot/todo-bot/node_modules/octokit-auth-probot/dist-node/index.js:181:21)
    at createProbotAuth (/Users/simonbuilds/streams/02_gh_bot/todo-bot/node_modules/octokit-auth-probot/dist-node/index.js:196:17)
    at new Octokit (/Users/simonbuilds/streams/02_gh_bot/todo-bot/node_modules/@octokit/core/dist-node/index.js:112:20)
    at new _a (/Users/simonbuilds/streams/02_gh_bot/todo-bot/node_modules/@octokit/core/dist-node/index.js:166:30)
    at new OctokitWithDefaults (/Users/simonbuilds/streams/02_gh_bot/todo-bot/node_modules/@octokit/core/dist-node/index.js:142:11)
    at new OctokitWithDefaults (/Users/simonbuilds/streams/02_gh_bot/todo-bot/node_modules/@octokit/core/dist-node/index.js:142:11)
    at new Probot (/Users/simonbuilds/streams/02_gh_bot/todo-bot/node_modules/probot/lib/probot.js:38:25)
    at new ProbotWithDefaults (/Users/simonbuilds/streams/02_gh_bot/todo-bot/node_modules/probot/lib/probot.js:71:17)
    at new Server (/Users/simonbuilds/streams/02_gh_bot/todo-bot/node_modules/probot/lib/server/server.js:34:26)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:60034) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:60034) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Expected behavior/code
As described in https://probot.github.io/docs/development/#configuring-a-github-app, I would assume Probot to start in set-up mode, to help me create the GH App
Environment

Probot version(s): ^11.0.1
Node/npm version: >= 10.13.0
OS: MacOS 11.2.3

Confirmed, I'm looking into it

This is caused by https://github.com/octokit/auth-app.js/releases/tag/v3.4.0. @octokit/auth-app is not a breaking change, it just fails earlier when you use it incorrectly, which is what we probably do in Probot. But if I cannot fix it swiftly I'll pin the version until I figure this out


Implement AdaBound?

This is a newly proposed optimizer, seems to be promising. Official code in Pytorch here https://github.com/Luolc/AdaBound

@cmpute see #6388.

Sorry, didn't notice that


Is there a way not to load Desktop, just command line

I am using an ARM device and using qemu to simulate AMD_64. Loading Desktop will make the operation much slower.

Hello there!
I'm currently working on another flake that packages all the cybersecurity and productivity tools so that you can easily configure your system to your liking. It'll keep all the tools and terminal configured for pentesting, blue teaming, red teaming, and son on.
Until I get the structure set up and publish it, you can clone this repository and disable in the hosts/pwnix/user.nix file the hyprland, waybar, kitty, dunst, fuzzel and gtk modules. This way, only the packages, zsh, neovim configuration, and direnv will be installed.
In addition, you could comment out the following lines in the modules/system/configuration.nix file:

74-83 : XDG Desktop portals for hyprland.
21 : Wireshark (GUI app).
36-44 : Login manager, if no desktop environment is going to be used it can be omitted.
93-96 : GVFS, if no desktop environment is to be used, this can also be omitted.
180-182, 187 : Bluetooth stuff, if not needed.
51 : If thunderbolt is not to be used.

Also, you can try changing the system architecture in flake.nix, replacing x86_64-linux with aarch64-linux. It is possible that some packages or configurations may fail to compile in ARM environments, but it is worth a try.
Hope you find it helpful!

Hello there!
I'm currently working on another flake that packages all the cybersecurity and productivity tools so that you can easily configure your system to your liking. It'll keep all the tools and terminal configured for pentesting, blue teaming, red teaming, and son on.
Until I get the structure set up and publish it, you can clone this repository and disable in the hosts/pwnix/user.nix file the hyprland, waybar, kitty, dunst, fuzzel and gtk modules. This way, only the packages, zsh, neovim configuration, and direnv will be installed.
In addition, you could comment out the following lines in the modules/system/configuration.nix file:

74-83 : XDG Desktop portals for hyprland.
21 : Wireshark (GUI app).
36-44 : Login manager, if no desktop environment is going to be used it can be omitted.
93-96 : GVFS, if no desktop environment is to be used, this can also be omitted.
180-182, 187 : Bluetooth stuff, if not needed.
51 : If thunderbolt is not to be used.

Also, you can try changing the system architecture in flake.nix, replacing x86_64-linux with aarch64-linux. It is possible that some packages or configurations may fail to compile in ARM environments, but it is worth a try.
Hope you find it helpful!

Thanks, I just need the environment for pwn and am currently practicing your reply

This repository doesn't seem to be for PWN(https://ctf-wiki.org/pwn/linux/user-mode/environment/)?


Receive: Make out-of-order max cap parameter hidden

Signed-off-by: Matej Gera<EMAIL_ADDRESS>

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
Leftover from https://github.com/thanos-io/thanos/pull/5839, this parameter should be hidden (it's experimental).
Verification

Let's update the doc.

Sigh, looks like play.min.io is having issue with certificate, this causes a lot of tests to fail...

Tests are passing, merging since this is more of a cosmetic change.


xlsx is not supported by xlrd anymore

oemof-solph: v0.4.x - excel_reader
OS: Windows 10
Python: 3.9.1
If engine is not specified for xls = pd.ExcelFile(filename) the following error occurs:
raise XLRDError(FILE_FORMAT_DESCRIPTIONS[file_format]+'; not supported')
xlrd.biffh.XLRDError: Excel xlsx file; not supported

This happens as excel file was created for the extension xlsx which is not supported anymore by xlrd. The problem will be solved by installing openpyxl and specifying it as engine for parsing xlsx using pandas xls = pd.ExcelFile(filename,engine='openpyxl').
Another error occurs when running dispatch.py is that when parsed data are NaN (i.e. np.nan), if de["active"] return true. For avoiding that np.nan should be removed or check to be sure de["active"] is not NaN: if de["active"] and (not pd.isnull(de["active"]))

Thank you for reporting. I will check it.


GraphQL CRUD APIs

Problem
At present, Prisma does not have a great advantage in replacing ORM, and it does not conform to the readme description that can easily build GraphQL APIs. I still use Prisma to build GraphQl APIs and still need to define various Resovers myself. The Typing exported by the Prisma client api component itself can be used in GraphQL, but I am now all writing various Resolvers, which does not reduce the development workload compared to other ORMs.
Suggested solution
Alternatives
In addition to generating Prisma client js, you can add a binder to generate GraphQl DSL
Additional context
I initially used TypeORM for the development of GraphQL APIs, but I found that it was very complicated for me to query related data or filter data based on relationships. Then I was attracted by Prisma. After construction, Prisma is indeed much more convenient than TypeORM relational queries.
But I found that Typing of Prisma client js is relatively GraphQL-friendly. With NestJS, compared to TypeORM, I can use code first to build GraphQL AST, because TypeORM models and NestJS code first GraphQL AST will be easy to build together. After I switched to Prisma, I needed to write a lot of GraphQL AST by hand, and I could only use NestJS's schame first. In fact, it is not easy for Prisma to build GraphQL APIs. On the contrary, it is very simple to build REST full. Hope to increase the GraphQL AST binder, as simple as generating TypeScript typing. In this way, even if we need to write a large number of Resolvers, the mechanical duplication of definition work such as translating Typing into GraphQL AST is reduced.

Checkout nexus and nexus-plugin-prisma. I think that is basically what you are asking for.
Nexus plugin prisma can expose crud operations from the Prisma schema with a single line: https://nexusjs.org/docs/pluginss/prisma/overview#example
Nexus is a graphql schema building implementation maintained by Prisma and it interops with graphql.js based servers. It is a code first approach which encourages type safety as nexus can generate types for the resolvers automatically.

@pantharshit00 There is a discussion about binding Nexus in the NestJS Issues. The developers of NestJS think that type-graphql uses decorators to be more concise and friendly. https://github.com/nestjs/graphql/issues/139#issuecomment-473653770
I have used various GraphQL generators, so I agree with the NestJS developers.
In fact, I personally hope that the prisma document can explain how the client generator is developed and embedded in the Prisma CLI. At least this way I can write the Client generator myself to meet the definition of type-graphql.
Forgive me for my weak development ability. After reading the Prisma source code for a night, I didn't find the trick to develop a Client generator. After reading the generated source code of prisma-client-js, I know how to deal with it, but I don't know how to embed it in Prisma CLI.

@medz check out this project, it implements a custom generator.
In this case it generates JSON schema, but I played around with it and managed to get a bunch of NestJS modules/inputs/models/resolvers generated on each prisma generate.

Here is another example of a generator: https://github.com/pantharshit00/prisma-docs-generator
Basic docs for generator authoring: https://github.com/prisma/specs/tree/master/sdk/generators
This API is not documented right now in the official docs as it still needs some polishing but you can use it to create any client/other output from the prisma schema.
There is already a typegraphql prisma generator which is maintained by the typegraphql author(It is our collaboration project with them): https://typegraphql.com/docs/1.0.0/prisma.html
I think you should use that if you are typegraphql user.

@pantharshit00 Thanks，I hope that the client generator function can be released as soon as possible, so that Prisma users can build more binding with other frameworks or tools. 🎉

The current implementation of the generator is ready to use while using the existing community generators as inspiration and documentation. We will announce when we open that up as a stable API our community can rely on, until then there might be breaking changes from time to time that need minor adaptations.

@medz do you have any advise on this, I am looking to integrate prisma with nestjs, with some kind of auto code generation.


Make an ajax request after sleep mode

I don't see anything in javascript side handling this case.
Should we add an observer on page visibility or something else to ensure that we have to make an ajax call?
For me, "setInterval" is not sufficient in this case because if I am not wrong, in sleep mode, javascript just pause and get alive after. So it could spend hours between 2 ajax calls.

Hi @MoogyG, do you have a solution in mind?

Hi,
I made a quick test with this : https://github.com/serkanyersen/ifvisible.js
I wrote a setinterval every 500 ms and start it, then i enter in sleep mode, i wait 2 or 3 minutes and when i power on, it fires the event "idle".
Maybe you could extract the code for this event, and when it fires, check the timestamp of the last ajax request and call it if necessary.

Thanks @MoogyG, will look into this. :) Give me a few days, since I'll probably be busy with other things over the holiday week (Christmas and New Year).
Happy Holidays!

I am not sure that's a proper way and even if HTML5 could cover our case by another way.
Happy Holidays Mike.

This is covered by #72.
There is no way to tell the browser to NOT throttle its Javascript Engine when the window goes inactive. Even so, It shouldn't be a problem because setTimeout can be delayed by just mere seconds. The throttling (or halting in some cases) is handed differently by each browser.
Anyway, forcing the browser to dip when the window is not active should be avoided, because:

There is no reliable way to force a dip on time, or even force one.
Makes the browser consume more battery.
The JavaScript Engine resumes anyway when the window becomes active, and so the script's "Check if we need to reload the whole page" part.

That script part could be handy for fire instantly when the page becomes active, like window.on('from-idle-to-active', () => { ... }) but I don't see any critical scenario to include more lines of code if its checked anyway in 2 seconds.


Tab does not work well with org-mode in terminal

Tab in org-mode work as better-jumper-jump-forward, but not org-cycle

This is because C-i is bound to better-jumper-jump-forward, and in TTY emacs, the tab key triggers C-i.
Try adding this to ~/.doom.d/init.el:
(setq evil-want-C-i-jump nil)


[CANNOT CONNECT]

What client do you use?
Erelajs
Are you sure using secure options?
Yes
Are you sure using port 443 when connecting?
Yes
This is the error
getaddrinfo EAI_AGAIN lavalink-repl.username.repl.co

try to connect again, replit having issue before


Add wannapik.com into illustrations/vectors

Checklist

[x] I made something good today 💐.
[x] I put links in the correct format: [Tool](link) — description.
[x] My description starts with the lower-case letter and ends with the full stop.
[x] I put the tool in the alphabetical order.

Optional

[x] I added the appropriate labels: free, open-source or mac.
[x] I put the tool only to one category.
[x] I followed Contribution Guidelines.

@gudata Wannapik looks really nice! Thanks for adding it to our list :)


Sample is not working

WebWhatsAPI/sample/echo.py
import time
from webwhatsapi import WhatsAPIDriver
from webwhatsapi.objects.message import Message

driver = WhatsAPIDriver()
print("Waiting for QR")
driver.wait_for_login()

print("Bot started")

while True:
    time.sleep(3)
    print('Checking for more messages')
    for contact in driver.get_unread():
        for message in contact.messages:
            if isinstance(message, Message): # Currently works for text messages only.
                contact.chat.send_message(message.safe_content)`
```
Is not working(only repeats print(Checking more messages)),Using Chrome

It's working perfectly for me. Which chrome version are you using? Please notice, that the message isn't unread anymore, when you open it with your mobile phone.

Not working for me too.
I do not has any samples to consult.  Please make more examples.
I am using in windows 10 64x, it's giving several errors like 'WhatsAPIDriver' object has no attribute 'get_contacts'.

Note that the pip installed version might be behind the official github source and so might be causing trouble. Also youse the newest chromedriver / geckodriver

Hi
Send me this error:
'WhatsAPIDriver' object has no attribute 'send_media'
Help me, please

Update file init.py link functions required send_media(), convert_to_base64(), send_message_with_thumbnail().


Bing empty results

I am getting empty results for Bing search, is its scraping engine actual?

I don't have any issues at all with Bing. Perhaps your query doesn't produce results, do you get any results with any queries at all?

It happens with any request, empty html without results (see pic below, second is the normal desktop answer). I checked with a foreign proxy, and results appeared. Looks like a geoban as for Qwant (#35).
Thanks for checking, I think nothing can be solved with this at the tool level.


Webgl error when browser tab is minimized

Hi,
When browser tab is minimized or switched to other tab for 40 seconds..
and when I go back to the tag - PIXI.extra.Movieclip objects gets hidden automatically and I see some warnings in browser console.
[.Offscreen-For-WebGL-0000022870512F00]GL ERROR :GL_INVALID_OPERATION : glDrawElements: range out of bounds for buffer

WebGL: too many errors, no more errors will be reported to the console for this context.

Pixi version: 4.3.0
Can someone please help me with this?
Video demo: https://www.youtube.com/watch?v=ZKcDYLr0Yuk

Sorry.. issue was with my game loop.
Closing it now


Do not generate new() functions for actors

I think this is no longer relevant. It relates to generating the env.c and env.h files, which is no longer necessary. We generate most of it from builtin.act and only the manual code is still in env.c (which should perhaps move to an .ext.c file).
Anyway, closing this.


Add short names for Issuer and ClusterIssuer

Pull Request Motivation
As suggested in #7268 this adds short names for these resources.
Kind
/kind feature
Release Note
Added the `iss` short name for the cert-manager `Issuer` resource
Added the `ciss` short name for the cert-manager `ClusterIssuer` resource

I am ok with the suggested short names. In "my" operators the CRDs are generated from Go code (and markers), so I am bit surprised to see no changes in the Go code. 🤔 LGTM


index.html

try to clear unwanted comments it looks more clear your codes

comment is not always a bad thing ;)


Does dart allow nested mixin?

The code as follows:
class A {
  void abcd() {
    print('abcd');
  }
}

class B with A  {}

class C with B {
  // @override
  // void abcd() {
  // }
}

The error as follows:
Error: The non-abstract class 'C' is missing implementations for these members:
 - A.abcd
Try to either
 - provide an implementation,
 - inherit an implementation from a superclass or mixin,
 - mark the class as abstract, or
 - provide a 'noSuchMethod' implementation.

class C with B {
      ^
Context: 'A.abcd' is defined here.
  void abcd() {
       ^^^^

But when I override the function 'abcd' or make 'C' mixin A, it works well.
Why must I override the function 'abcd' ? How do i avoid overriding it?

It looks like you are using a very old version of Dart; in current Dart it is a compile-time error to use a class D as a mixin unless the superclass of D is Object. In the example, the superclass of B is Object with A, so B can't be used as a mixin.

It looks like you are using a very old version of Dart; in current Dart it is a compile-time error to use a class D as a mixin unless the superclass of D is Object. In the example, the superclass of B is Object with A, so B can't be used as a mixin.

@LeoFavor1562 does the comment above answer your question?

@LeoFavor1562 does the comment above answer your question?

@eernstg
Thank you for your answer. But I'm still confused.
My dart version is 2.10.0, and I added file analysis_options.yaml in my project with code:
analyzer:
  strong-mode: true
  errors:
    mixin_inherits_from_not_object: ignore

to solve the compile-time error.

if B can't be used as a mixin, why when I override the method abcd in class C, the error is gone?
if I modify the example code:

class A {
  void abcd() {
    print('abcd');
  }
}

class B with A  {}

class C with A, B {//<--
  // @override
  // void abcd() {
  // }
}

the error is gone too, why does this happen?

@eernstg
Thank you for your answer. But I'm still confused.
My dart version is 2.10.0, and I added file analysis_options.yaml in my project with code:
analyzer:
  strong-mode: true
  errors:
    mixin_inherits_from_not_object: ignore

to solve the compile-time error.

if B can't be used as a mixin, why when I override the method abcd in class C, the error is gone?
if I modify the example code:

class A {
  void abcd() {
    print('abcd');
  }
}

class B with A  {}

class C with A, B {//<--
  // @override
  // void abcd() {
  // }
}

the error is gone too, why does this happen?

If you ignore a compile-time error there are no guarantees. This mechanism is rather controversial in the first place, and it's only supposed to be used to avoid seeing errors in a situation where a library does not exist because it's provided by code generation, and similar situations where the errors are temporary. There is no general mechanism which could make a program work if it has a compile-time error, but it may be justified to use this mechanism to avoid some temporary noise.
However, in this particular case you'll probably end up using an undocumented enhancement of the mixin mechanism where it was allowed to use a class B as a mixin even though the superclass S of B wasn't Object. In that case the superclass S isn't actually included, you still just get the declarations from the class B that you're using as a mixin, nothing that B inherits from S. Stop using that loophole, it will go away. ;-)
Anyway, in the example this means that you don't get the implementation of abcd from A, you only get the promise (as in implements A) that there will be such an implementation, and then you need to write it yourself.
In the second example you are mixing in A and then mixing in B, and this means that you do get the implementation of abcd (when A is mixed in).
The meaning of the class B when used as a mixin is expressed as follows in current Dart:
mixin B implements A  {}

This is not a particularly useful mixin, but it does show clearly what it delivers: It introduces the requirement that any class mixing in this mixin must have a method void abcd() implementation; but you won't get it from this mixin, so you may need to write that implementation yourself (or mix in some other mixin that does deliver the implementation).
I'll close this issue because the underlying feature request is covered by https://github.com/dart-lang/language/issues/540.

If you ignore a compile-time error there are no guarantees. This mechanism is rather controversial in the first place, and it's only supposed to be used to avoid seeing errors in a situation where a library does not exist because it's provided by code generation, and similar situations where the errors are temporary. There is no general mechanism which could make a program work if it has a compile-time error, but it may be justified to use this mechanism to avoid some temporary noise.
However, in this particular case you'll probably end up using an undocumented enhancement of the mixin mechanism where it was allowed to use a class B as a mixin even though the superclass S of B wasn't Object. In that case the superclass S isn't actually included, you still just get the declarations from the class B that you're using as a mixin, nothing that B inherits from S. Stop using that loophole, it will go away. ;-)
Anyway, in the example this means that you don't get the implementation of abcd from A, you only get the promise (as in implements A) that there will be such an implementation, and then you need to write it yourself.
In the second example you are mixing in A and then mixing in B, and this means that you do get the implementation of abcd (when A is mixed in).
The meaning of the class B when used as a mixin is expressed as follows in current Dart:
mixin B implements A  {}

This is not a particularly useful mixin, but it does show clearly what it delivers: It introduces the requirement that any class mixing in this mixin must have a method void abcd() implementation; but you won't get it from this mixin, so you may need to write that implementation yourself (or mix in some other mixin that does deliver the implementation).
I'll close this issue because the underlying feature request is covered by https://github.com/dart-lang/language/issues/540.

@eernstg
Thanks a lot!

@eernstg
Thanks a lot!


readInputs in cli, returns the ScanInputs's returned Inputs and not a…

Resolves #65

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.083%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
794

Relevant Lines:
853

💛  - Coveralls


Describe config

Adds DESCRIBE SYSTEM CONFIG. I'm not sure how to test that. Given that some changes (listen_adresses) interfere with all other tests, some with few tests (Auth only blocks auth test, ports blocks more).
Also I'm not sure:

Do we need to emit CONFIG SYSTEM RESET Auth, as there is a single auth record on empty database, at least in dev mode
How to handle "set" parameters like listen_addresses. Currently, we can't assign an empty set. But RESET may mean a different thing for purposes of dump.

Fixes #1490

Do we need to emit CONFIG SYSTEM RESET Auth

We don't.  The Trust method is only added in dev mode for convenience.

How to handle "set" parameters like listen_addresses. Currently, we can't assign an empty set.

You cannot assign an empty set, because listen_addresses is declared as required.  What is the scenario in which you need to write an empty set there?

You cannot assign an empty set, because listen_addresses is declared as required. What is the scenario in which you need to write an empty set there?

This is weird because the default is an empty set.
The scenario is obviously "DESCRIBE SYSTEM CONFIG" used for dump/restore. I don't think omitting property is good idea here. Especially because this is generic code that works for all "multi" properties. And some future multi-property can have a non-empty default but having valid behavior with an empty value set manually.

Looks like this should be good enough at this point. We can do more generic handling of ports and auth later on.


Move startup props into a single configuration

Feature Request
Create a new prop where all current props that are only used during initialisation and cannot be changed during runtime into a single configuration prop.
Why it is needed
This will make it more clear to developer as to which props can be changed during runtime and which props can only be provided when creating the player. This will also simplify the initialisation logic to get rid of postDelayed calls etc.
Tasks

[ ] textTracks (external subtitle tracks)
[ ] bufferConfig
[ ] contentStartTime
[X] drm
[ ] header
[ ] Ads props

@freeboub and @armadilio3. Could you guys please come a list of props which is only for the player initialization? Any change to these props will be ignored after the initialization
fullscreen
controls

I prefer breaking the people build in alpha after fixing this. Do you guys have any concerns?

@wood1986 I already flag following props as 'startProps':

textTracks (external subtitle tracks)
startPosition (To be implemented)
bufferConfig
contentStartTime
drm
header
useTextureView
useSecureView
useTunnelingPlayback (a change from my working branch I need to backport)

The good news is that we should be able to implement this change without changing external API.
in Video.js, we can get public input paramters and reformat then to provide it to native.
That's my plan, but I have a lack of time :)

localSourceEncryptionKeyScheme for ios shall be moved to source.drm.localSourceEncryptionKeyScheme

localSourceEncryptionKeyScheme for ios shall be moved to source.drm.localSourceEncryptionKeyScheme

Done in: https://github.com/TheWidlarzGroup/react-native-video/pull/4136 thank you @KrzysztofMoch

reviewing the desription of the ticket, and it seems that headers property implementation is missing: https://thewidlarzgroup.github.io/react-native-video/component/props#headers
I cannot find implementation :/


Generate DSC from .NET not working, missing assembly?

Steps to reproduce
Generate a DSC file from a .NET Core application.
Expected behavior
The DSC code executes normally.
Actual behavior
It ends with the following error:
Cannot find the type for custom attribute 'Microsoft.PowerShell.DesiredStateConfiguration.ArgumentToConfigurationDataTransformation'. Make sure that the assembly that contains this type is loaded.
Environment data
PSVersion: {6.0.0-alpha}
PSCompatibleVersions: System.Version[7]
PSEdition: Core
BuildVersion: 3.0.0
CLRVersion: null
SerializationVersion: <IP_ADDRESS>
WSManStackVersion: 3.0
PSRemotingProtocolVersion: 2.3
GitCommitId: "Could not find file 'C:\\Users\\xxx\\.nuget\\packages\\system.management.automation\\6.0.0-alpha17\\lib\\netstandard1.6\\powershell.version'."

This is working with beta1.
Closing this.


openssl required by crate openssl-sys was not found

@rultor release, tag is 0.0.2

@rultor release, tag is 0.0.2

@rultor release, tag is 0.0.2

@h1alexbel OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.2

@h1alexbel Done! FYI, the full log is here (took me 29min)


[stable/concourse] Allow AWS role based authentication to Secrets Manager

What this PR does / why we need it:
This minor change that is localised to AWS Secrets Manager based deployments allows configurations that utilise role based authentication (on the EC2 instance or pod level using annotations) without the requirement of providing a key and secret.
Prior to this change, role based auth will fail as not setting these values (commented out in values.yaml) will result in the chart setting these environment variables on the web container instance with empty values resulting in failure to authenticate at run-time.
This change only sets these values at the container level to be used by Concourse when they are specified in the configuration.
Which issue this PR fixes

fixes #8489

Special notes for your reviewer:
Note that it is likely that other features such as AWS Systems Manager role based auth will likely have the same issue, but I've deferred a similar change to that section of the code for now to minimise any potential issues to SecretsManager. A seperate PR can be used for Systems Manager (or others) later once this is confirmed to work without issues.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

/assign william-tran

@william-tran @frodenas Did someone get a chance to look at this yet please?

We should not place conditionals on .Values.secrets.* because users should be able to manage secrets outside helm.

We should not place conditionals on .Values.secrets.* because users should be able to manage secrets outside helm.

@william-tran Thanks for taking the time to look at this.
I'm not sure what you mean by managing the secrets outside of helm. Is that being passed through the helm CLI or passed on somehow through though a parent shell? Can you please elaborate?
The problem being addressed in this PR is that when these values are not set, then helm will create empty environment variables and the application in turn will consume these values resulting in unexpected behaviour.
Also, the same file currently uses conditionals for a number of .Values.secrets.* variables (for example, awsSecretsManagerSessionToken in the same section being modified). Are those conditionals meant to be removed at some stage?

@william-tran @frodenas Have you had a chance to review my response?

Will create a new PR to include AWS Systems Manager role auth changes that are pretty much along the same lines.


Fix import for unused animation slice information.

Tentative fix for #68936
Problem brief: The gltf importer would import 256 slice information for each animation if a single animation option was changed. Making the .import file unnecessarily big.
This makes the gltf importer only save slice data default values for slices created by the user.
This won't fix imported assets that have already been bloated. But it won't create bloat on new imported assets.

This is my first time with Godot source so I chose this simple approach first to see if it works. It did.
Maybe a better way to solve this is to avoid creating the 256 ImportOptions in the first place here
https://github.com/godotengine/godot/blob/33c30b9e63a58b860cb2f36957c5e25cee34a627/editor/import/3d/resource_importer_scene.cpp#L2028-L2036
This is created when the import settings window is loaded, and it will always create those 256 slice default data.
But as they are all the same anyways, maybe its better to just load something like "slice_data" and use a similar strategy, where I checked for the slice_amount to decide if the slice default values needs to be filled or not.
Inside both the post_fix_node and post_fix_animation where we have that value.

Hmm I see what you're doing and it could work... basically it stops checking for slices (sets has_slices to false) at 1 + slice count.
It does feel a little hacky, and I would prefer if the underlying cause could somehow be addressed directly without having to make this loop kore complicated.
but honestly if it comes time for 4.4 or a 4.3 bugfix release and there isn't a better fix, I could go for this. I don't think it should cause any issues.

It does feel a little hacky, and I would prefer if the underlying cause could somehow be addressed directly without having to make this loop kore complicated.
but honestly if it comes time for 4.4 or a 4.3 bugfix release and there isn't a better fix, I could go for this. I don't think it should cause any issues.

Glad to hear that. I'm still getting familiarized with Godot's code so I'm a bit unsure about bigger changes.
PS: I just got my clang-formater working and forced push the formatting changes.


Fix hanging streaming client reader

Fix originally submitted by @xiam, thanks.

Tested against latest webrpc + SSE example & tests.


basic_file_body buffer size and ::sendfile optimization

Hi,
I use Beast in an application which is to be mainly run on SoCs such as the Raspberry Pi.
I have noticed that boost::beast::file_body suffers from VERY bad performance when sending large files. When I checked the code (Boost 1.66.0) I was surprised to see a very small buffer (4096 bytes) being used for reading the file, without the possibility of enlarging it easily. That's not nice at all.
I googled and googled and noticed that sendfile support was mentioned a few times, but I didn't find it in either Boost 1.66.0 or the develop version of Beast.
Is it hidden somewhere or what happened to it?

Use of ::SendFile or ::TransmitFile on Windows is automatic when file_body is an alias for basic_file_body<file_win32>. The code for it is here:
https://github.com/boostorg/beast/blob/develop/include/boost/beast/http/impl/file_body_win32.ipp
No efforts were made to optimize file bodies, so there is probably a lot of room for improvement especially on platforms which I have not personally developed on. If you would like to experiment with changing the buffer size or other improvements I would be more than happy to hear about the results and figure out how we can improve the interface or implementation to take advantage of it.

It would be great to have this feature.

diff --git a/file_body_win32.hpp.orig b/file_body_win32.hpp
index cfeb5aa..6d9fe5f 100644
--- a/file_body_win32.hpp.orig
+++ b/file_body_win32.hpp
@@ -340,6 +340,7 @@ class write_some_win32_op
         basic_file_body<file_win32>, Fields>& sr_;
     std::size_t bytes_transferred_ = 0;
     bool header_ = false;
+    bool only_header_;

 public:
     template<class Handler_>
@@ -355,6 +356,7 @@ public:
                 s.get_executor())
         , sock_(s)
         , sr_(sr)
+        , only_header_(sr.split() && !sr.is_header_done())
     {
         (*this)();
     }
@@ -415,7 +417,7 @@ public:
         std::size_t bytes_transferred = 0)
     {
         bytes_transferred_ += bytes_transferred;
-        if(! ec)
+        if(!ec && (!sr_.is_header_done() || !only_header_))
         {
             if(header_)
             {

This works for me.
Do you want me to create a pull request (would have to wait until Monday)?

Hi,
Is there a timeline of when this feature would be made available?

Tried to implement file_body_linux body on top of current development branch but it fails.
can somebody more familiar with beast/asio explain me what is wrong.
Here is my first try. I just copied file_body_win32.hpp and replaced file_win32 with file_linux and TransmitFile with sendfile   But it doesn't work as expected...
As I thought beast must call function below for async operations, but for some reason it calls just get function of body writer so its still userspace<->kernel spacecopy way..
template<
    class Protocol, class Executor,
    bool isRequest, class Fields>
std::size_t
write_some(
    net::basic_stream_socket<
        Protocol, Executor>& sock,
    serializer<isRequest,
        basic_file_body<file_linux>, Fields>& sr,
    error_code& ec)

Can somebody help me what am I doing wrong?

ping


Obtain SoC part number from the device tree and not Kconfig

Virtually all of the SoCs require part number information to compile the code. Originally this information was provided by a Kconfig option. After introduction of the DT some part number dependent configuration settings such as RAM and flash size were moved to DT. Consequently we need to maintain and set part number information twice, once in Kconfig and once in DT.
This PR adds DT bindings to allow to obtain SoC part number from the device tree. The corresponding Kconfig options can be removed.
Zephyr DT is supposed to be compatible with Linux DT, however Linux DT does not seem to contain part number information. Following proposal is Zephyr specific.
Since the DT root node is supposed to represent the board I propose to attach the part-number property to the soc node. The proposal needs to be carefully reviewed.

Is it possible to do:
soc {
	compatible = "silabs,EFR32FG1P133F256GM48";
...

Is it possible to do:
soc {
	compatible = "silabs,EFR32FG1P133F256GM48";
...

We could do
	compatible = "silabs,EFR32FG1P133F256GM48", "soc,silabs";

maybe also
	compatible = "silabs,EFR32FG1P133F256GM48", "soc,silabs", "soc";

I'm not really sure that we need soc,silabs.yaml binding (in case of SiLabs chips) since we do not differentiate it with the general soc and most likely we will not in the future. It's also unlikely that other SoC families will need to use vendor specific properties. Nevertheless, I kept the design consistent with what we do for other DT nodes.

Codecov Report

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

@@           Coverage Diff           @@
##           master   #12073   +/-   ##
=======================================
  Coverage   48.05%   48.05%
=======================================
  Files         281      281
  Lines       43416    43416
  Branches    10406    10406
=======================================
  Hits        20864    20864
  Misses      18403    18403
  Partials     4149     4149

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 1870d99...c0d4d7f. Read the comment docs.

@mnkp,

however Linux DT does not seem to contain part number information.

This exists on some parts:
https://elixir.bootlin.com/linux/latest/source/arch/arm/boot/dts/stm32f429-disco.dts#L55
I definitely agree this is where we need to go (and potentially get rid of most soc/ code).
Having the info inside SoC dtsi files also makes sense. Let's see how we can make best use of it.

Is it possible to do:
soc {
	compatible = "silabs,EFR32FG1P133F256GM48";
...

Looking at some old discussions on Slack I think I may have misunderstood @galak's proposal. And @erwango may have had the same in mind.
The alternative approach would be not to add 'part-number' property but rather generate a define for 'compatible' property. That has some advantages. An obvious one is that 'compatible' is a standard property.
In the nearby future we will want to get rid of CONFIG_SOC_SERIES and CONFIG_SOC_FAMILY Kconfig options. The 'compatible' property approach could be expanded to support it. We would need to enforce the second and third string of the property to be the name of the series and the name of the family.
	compatible = "silabs,EFR32FG1P133F256GM48", "silabs,efr32fg1p", "silabs,exx32";

There is one disadvantage. The 'compatible' property is build around notion of selecting the best matching model from the list provided. In our case we would overload that meaning. The list wouldn't be optional but strictly predetermined. Skipping one of the elements would break compilation. That's not how compatible is supposed to work.
One other minor issue to keep in mind, the part number is mostly required to select a vendor provided include header, directory or pass as a preprocessor define. It must have a form of a string containing the part name with no decorations. e.g. "EFR32FG1P133F256GM48" and not "silabs,EFR32FG1P133F256GM48". Since cmake supports string processing operations the part number could be extracted from the property. Without cmake however that property wouldn't be of much value.
So, is using a 'compatible' property a better way to go?

@mnkp can you link to the Slack discussions?
Absent seeing that, I like putting it in compatible rather than creating a new property name.
I don't see an obvious motivation for compatible with "soc,silabs" or "soc".  Something like this might make sense:
soc {
	compatible = "silabs,EFR32FG1P133F256GM48", "silabs,EFR32FG", "silabs,EFR32";
}

If the tooling generated this as:
#define DT_SOC_COMPATIBLE_SILABS_EFR32FG1P133F256GM48 1
#define DT_SOC_COMPATIBLE_SILABS_EFR32FG 1
#define DT_SOC_COMPATIBLE_SILABS_EFR32 1

would that handle things?  That'd probably be enough for a generic header to produce a define with the correct string (selecting from a set of known values using #ifdef).
Given the (proposed) need for multiple strings in the compatible value I'm not clear how to automatically generate a define with a string value.  Maybe:
compatible = "soc,EFR32FG1P133F256GM48"

and special-case the processing of any compatible value starting with "soc," to generate the remainder:
#define DT_SOC_COMPATIBLE "EFR32FG1P133F256GM48"

though that's a little ugly.

tl;dr: There's a lot of variation in how vendors express the hardware configuration hierarchy. I think the overall goal of this PR is important, but that a successful solution requires a thorough review of existing practice and may require some vendors to change the way they do things today.
Previous slack discussion involving @galak and @mbolivar is here.
Stepping back, my understanding is that before device-tree Kconfig would generate defines like:
CONFIG_SOC_FAMILY_EXX32=y
CONFIG_SOC_SERIES_EFR32MG12P=y
CONFIG_SOC_PART_NUMBER_EFR32MG12P332F1024GL125=y

where the first two come from board_defconfig and the last from a select clause in Kconfig.board.  Silicon Labs and NXP follow this model; Nordic and Intel follow a model that produces:
CONFIG_SOC_SERIES_QUARK_SE=y
CONFIG_SOC_FAMILY_QUARK=y
CONFIG_SOC_QUARK_SE_C1000=y

where all three come from board_defconfig and there is no _PART_NUMBER component in the identifier.
With device tree you want a way to instead generate:
DT_SOC_FAMILY_EXX32=y
DT_SOC_SERIES_EFR32MG12P=y
DT_SOC_PART_NUMBER_EFR32MG12P332F1024GL125=y

from properties located in a device tree node, probably at /soc.
Is this an accurate summary of the problem?
You (@mnkp) also said:

The name of the SoC series and SoC family has to correspond strictly to the names of folders we use to keep relevant SoC files.

This doesn't seem quite correct as the path for above is soc/arm/silabs_exx32/efr32mg12p which requires a prefix silabs_ that isn't part of any existing symbol.  It's not obvious where that's getting injected.
You've also said this could be zephyr-specific, but @erwango pointed out existing practice in Linux, specifically for the compatible property.
So assuming that's all accurate: I don't like either soc,silabs or silabs,soc as compatible values: they both go against the standard of manufacturer,model for that property (the Linux example for st,stm32f429 conforms).
Looking at the current contents of the soc directory I would propose something like the following:
soc {
    compatible = "silabs,exx32";
    series = "efr32mg12p";
    part-number = "efr32mg12p332f1024gl125";
};

soc {
    compatible = "nordic,nrf"; // currently nordic,nrf52840{,-qiaa}
    series = "nrf52";           // currently nrf52x
    part-number = "nrf52840-qiaa";
};

soc {
    compatible = "intel,quark";  // currently intel,quark_se_c1000; atom and other families under x86 are anomalous
    series = "quark_se";
    part-number = "quark_se_c1000";
};

The compatible value should identify the soc/cpu/manufacturer_family file path root (some like atom would have to be renamed).  The last component of compatible could be used by the extractdts infrastructure to synthesize DT_SOC_FAMILY_model.  Alternatively a family property could be introduced and compatible left alone.  The series and part-number properties would translate directly with a synthesized prefix of DT_SOC_SERIES_ and DT_SOC_PART_NUMBER_ respectively.
Unifying the way that family, series, and part are expressed across vendors may require changes for some vendors in preprocessor identifier, file system hierarchy, or both.  Also the existing compatible properties at the soc node have wide variation including values like simple-bus which would have to be reviewed.

@mnkp Is this PR still relevant?

Is this PR still relevant?

It is unlikely that the solution proposed in this PR will be adopted as the one that got more traction was the proposal from @galak

So I think compatiable at the soc node makes sense, we can use the work I've got going PR #12108

There was some initial work going on in #12428 but the PR got closed. Using compatible seems a more proper approach, inline with the idea of DTS, but it's not quite as easy as having a dedicated part-number property.
@galak could you quickly revisit this PR and close it if it's no more relevant. The discussion around generating part number from DTS was never finalized.

@galak  ping

@galak, @mnkp, Can we close this PR?

stale for a long time already..


Some extra dependecies neeed in some evironments

I do not know how reproducible this is, but in typical data science jupyter environments  I get to run a couple of extra pip install lines:
!pip install git+https://github.com/njszym/XRD-AutoAnalyzer.git
!pip install asteval
!pip install numexpr --upgrade

Do you think it is needed to add asteval to the dependencies or pin a minimum version of numexpr?

Yes to asteval (I'll add this!) but I'm not sure about numexpr -- can you let me know where in the code you receive an error if that is not installed and up-to-date?

It is just a UserWarning that comes from the installed version of pandas:
UserWarning: Pandas requires version '2.8.4' or newer of 'numexpr' (version '2.8.3' currently installed).
  from pandas.core.computation.check import NUMEXPR_INSTALLED

Got it, added the version requirement, thanks for the info!


[BEAM-10764] Make is_in_ipython robust

To handle potentially corrupted IPython dep used by the user running a
environment that is not managed by setuptools, added an any-exception
statement to treat the current code execution as not in IPython kernel
on non-import errors.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

retest this please

Run Python PreCommit

Run PythonDocker PreCommit


Percentile filter

Enhanced code to filter quantiles based on user choice
Included --cql-ssl option to crated encrypted native connection
Added latest version of jars as the older version are having some
vulnerabilities

Check out TextFormatMetricFormatWriter:
We could filter all of them here:
                summary.quantiles.forEach(interval -> {
                    writeMetric(buffer, metricFamily, null, interval.value, summary.labels, interval.quantile.asSummaryLabel());
                });

by

I aggree

@zegelin thanks for your review and suggestion. I have made the changes, please review.

This PR fixes below two issues.
https://github.com/instaclustr/cassandra-exporter/issues/69
https://github.com/instaclustr/cassandra-exporter/issues/72

@zegelin , I thanks for your review. I have fixed formatting issues.

@zegelin, can you please approve and merge my pull request if you don't see any issues?


What does the 'arduino' avrdude programmer do?

PlatformIO (at least using CLion) executes
avrdude ... -c arduino ...

to load program the my Arduino Uno, but it's unclear to me what it's doing. All the documentation I've found refers to the value of the -c setting as the "programmer"; and all the documentation I can find about that term describe something that's a hardware component connected to the "ISCP header". But I'm connected over a serial port with no special hardware.
Is PlatformIO using the bootloader to program my Arduino? In that case what role does the -c arduino play (is it specifying the bootloader, or something that depends on how I'm connected)?

avrdude -v -p atmega328p -C "~/.platformio/packages/tool-avrdude/avrdude.conf" -c arduino -b 115200 -P /dev/cu.usbmodem1421 -D -U flash:w:.pioenvs/uno/firmware.hex:i

This is what I see from arduino IDE
C:\Program Files (x86)\Arduino\hardware\tools\avr/bin/avrdude -CC:\Program Files (x86)\Arduino\hardware\tools\avr/etc/avrdude.conf -v -patmega328p -carduino -PCOM3 -b115200 -D -Uflash:w:C:\Users\VENKAT~1.VEN\AppData\Local\Temp\build727352015918313731.tmp/sketch_dec09a.cpp.hex:i
Here is the documentation for avrdude : http://www.nongnu.org/avrdude/user-manual/avrdude_4.html#Option-Descriptions

@itsjustvenky thanks for the answer.
@orome you can control programmer-id/protocol using upload_protocol option.

@itsjustvenky — So? That's the documentation I quoted in the question. For arduino its explanation is 	"Arduino". I was looking for a bit more than that.
@itsjustvenky — The question is simply: What does choosing -c arduino do? Specifically, is it using the bootloader, when I'm connected to the Uno using the serial port?

@orome Best place to post this question is "Arduino Forums".
anyway
"arduino" is a configuration name.
Go through this file
https://raw.githubusercontent.com/adafruit/Trinket_Arduino_Linux/1.0.5/arduino-1.0.5/hardware/tools/avrdude.conf

@itsjustvenky — I'm a beginner. That file doesn't help much. What  "Arduino Forums" do you recommend?

@orome https://forum.arduino.cc/
if you find a clear answer, please do post the answer here. thanks


Coerce MultiPolygon to Regular polygon to enable leaflet.draw editing

Coerce MultiPolygon to Regular polygon to enable leaflet.draw editing

Leaflet.draw does not currently support multipolygon features.
This coerces a MultiPolygon to be a collection of individual Polygon
features so that they can be edited successfully.

@pawl can you review this pull request?

@kespindler thanks for this fix. But I'm getting leaflet TypeError: Cannot read property 'lat' of undefined in my browser console when trying this out locally, editing a regular POINT record.
Could you merge master into your branch and check what's broken?
Also, could you use the /examples/geo_alchemy/app.py and check expand the fix to include MULTIPOINT and MULTILINE geometry types in addition to MULTIPOLYGON?

closing this, this shouldn't be an issue with the latest leaflet update


no error event is emitted/logged in case of ibc err ack

In case an error ack is returned, the error should be emitted as event for debug purposes.
currently there is no error emitted which makes it hard to understand the root cause.

there is already an event right
https://github.com/cosmos/ibc-go/blob/893b7afd6e924a347866336918e37d880d67b0fc/modules/core/04-channel/keeper/events.go#L179-L205

https://github.com/cosmos/ibc-go/blob/6fbc8c926706969ee3dd47950014a0d65348fb3f/modules/core/04-channel/keeper/packet.go#L356

@danwt yea, but where the err at

but it's written right here https://github.com/cosmos/ibc-go/blob/893b7afd6e924a347866336918e37d880d67b0fc/modules/core/04-channel/keeper/events.go#L193
in the bytes

I know that, I'm talking about the actual ’err‘ that is the root cause, not the obfuscated generic message with abci code


[Hexagon] Add hexagon user DMA intrins for tensorization

Added some intrins for user dma on hexagon. Currently these seem to perform worse than all other options used in the test.
@adstraw

Nice work.  Just wanted to note that we will need to (and have plans to) put our heads together on a unified approach to DMA lowering.  Right now, the async DMA flow stems from TIR annotations added by the InjectSoftwarePipe pass.  This PR suggests a tensorization approach for synchronous DMA.  I feel we could benefit from a unified approach.  I recently deleted an old (TE based) tensorization approach for sync DMA scheduling.  Glad to have this new TIR based version and it may in fact be the path forward for a unified approach to DMA lowering.  Let's put our heads together and get the right design.


Unpin frontend_server_client and roll packages

https://github.com/dart-lang/sdk/commit/69b801f73f7b26adb661257ecab862eca6dcf7dd has made it so that AOT snapshots now get closed as soon as they're done being read. This means that bumping package:frontend_server_client no longer breaks batch_entrypoint_test. Is it ok to unpin package:frontend_server_client now, @christopherfujino?

dart-lang/sdk@69b801f has made it so that AOT snapshots now get closed as soon as they're done being read. This means that bumping package:frontend_server_client no longer breaks batch_entrypoint_test. Is it ok to unpin package:frontend_server_client now, @christopherfujino?

Yes, I was just waiting for that commit to roll, thanks!


texture rewrite

This refactors TextureId creation in Yakui, so that Yakui entirely relies upon the host renderer for texture id creation.
Here is the current workflow:
First, when users create yakui::State, they must provide a texture reservation function. This isn't terribly elegant, and i have some thoughts to allow users to skip this stage, but that's how we start things right now. This doesn't mean users need to uplaod textures (though they can, and return a variant of an enum to signify that they have), but it does mean that users need to reserve a texture id. You can see in yakui_wgpu how this may be trivially implemented in that scheme. In my own game engine, I just use slabs id reservation system with an Arc'd texture manager.
Inside yakui, when a widget needs a new texture (for example, very quickly the text renderer needs to allocate its atlas), it asks for an ID. yakui_wgpu doesn't upload textures when its asked to reserve the ID, so it only reserves the ID and then later will get a TextureDelta::Add event, which is when it actually uploads the texture.
Finally, calls outputs textures which the renderer at this point knows it always is aware of.
--
The big shortcoming rn is that this loaded function can be difficult for ~some~ users to write, but at least for me, this is not hard. I experimented with yakui keeping an internal id space (which it can allocate to freely), and an external id space, but (unlike egui which does this) we only use one pass to paint and generate our IDs, so we actually have already generated our PaintCall before we have an ID in user space. The only solution if we maintain that design is to later take our PaintCall (which has "InternalId" in it) and transmogrify it into ExternalPaintCall (with "external id" in it). Either we do that, or the user does that, but either way, there's a big lookup stage involved for every paint call.
This also encouraged me to rewrite the GlyphCache. I renamed it LateBindingGlyphCache and made it the default -- moreover, I made it into a trait object, which allows me to write a glyph cache as a user which already has all the glyphs etched onto it, and where every location is known. That's convenient to me, since I do my font atlas creation offline, and don't want to waste the effort of making it several times! I didn't make that yet, since more font work is needed before then.
However, doing this work also helped the default cause tremendously -- we're not quite there yet, but LateBindingGlyphCache is very well set up to have more than one texture in its cache.
--
How do we get rid of requiring a loader function?
I suspect the simplest option is for us to provide a default function, and allow users to override that function as they wish (and they will wish! which is why I haven't done it here), probably being one where we assign to numbers from u64 backwards. This also leads us to wonder about Internal vs. External ids, but see the sections above where I ran into pain points there.
--
Okay, that's mostly what I did! Let me know if you have any questions or concerns

ahh there's some docs here which will need to be updated, but it's making its way there

gonna go ahead and close this guy!


sw-6430 task types will show count #minor

Codecov Report
Patch coverage: 80.95% and project coverage change: -2.14 :warning:

Comparison is base (d2ddd92) 59.53% compared to head (2554288) 57.40%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #318      +/-   ##
==========================================
- Coverage   59.53%   57.40%   -2.14%
==========================================
  Files          13       13
  Lines         912      932      +20
==========================================
- Hits          543      535       -8
- Misses        330      358      +28
  Partials       39       39

Impacted Files
Coverage Δ

internal/sirius/get_task_list.go
62.26% <ø> (-22.65%)
:arrow_down:

internal/sirius/get_task_types.go
69.23% <ø> (ø)

internal/server/workflow.go
65.00% <80.95%> (+1.87%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Is a vulnerable picard still used ? log4j vulnerability.

Dear devs,
using this software on an older, conda based installation of nf-core rna-seq, a vulnerable picard.jar was found.
https://github.com/mergebase/log4j-detector
Do you know when it will be possible to replace the used picard.jar  with a patched version ?
There is already a patched release on
https://github.com/broadinstitute/picard/releases/tag/2.26.7
Thanks

Great timing @colindaven as I am prepping a release! Thanks for reporting.

Looks like a version based on 2.16 might be a better option, with the lookups completely removed. Broad picard is on 2.15.
https://lists.apache.org/thread/d6v4r6nosxysyq9rvnr779336yf0woz4
Might pay to wait a bit?

Best to merge to 2.15.x-based releases as soon as possible and update to 2.16 when it is ready.  Things take a while to make their way through bioconda/biocontainers etc.

Other tools in this pipeline possibly affected by this:

[ ] BBSplit
[ ] Qualimap
[ ] Picard (bumped in https://github.com/nf-core/modules/pull/1173)

While greatly preferable to have an official statement from the BBMap and Qualimap developer teams, my analysis shows that BBMap BBMap_38.94.tar.gz has no dependency on any version of log4j and Qualimap qualimap-build-31-08-20.tar.gz, while having a dependency on a picard-1.70.jar, has only a dependency on a 1.2.x version of log4j and no dependency on a 2.x version of log4j.

Thanks @heuermh. Ok I have bumped Picard to 2.26.7 in https://github.com/nf-core/rnaseq/pull/740 because we know that has been affected for definite. I will leave this issue open for now so we can track any updates before the next release too.


typecheck annotations

Sometimes annotations contain expressions. We should typecheck them so there aren't Types expressions in the Prog AST.

This may conflict with kaleidoscope branch and so we might want to check annotation contents only when evaluating them, and not put that back into the annotation.

We're not going to do this.


Look at and implement content-security-policy

https://www.youtube.com/watch?feature=player_detailpage&v=WljJ5guzcLs#t=731

old repo - clearing issues


Unzip/ Untar tar.gz

Can we extract a tar.gz file? I've tried to use the addCustomFileExtension(".tar.gz") method but returns error. Thanks

did u find any solution?


Fixing compilation errors

Changelog

Fixed compilation errors on classes that used HunkView

Thank you, I just added the same changes. As well as a flag for TS to be stricter about these
`"compile": "tsc --forceConsistentCasingInFileNames -p ./",```
A new version is on its way with these fixes


How to completely uninstall Yi-Hack the right way?

Hi,
I'd like to remove YiHack completely and revert the firmware of the camera to stock. What steps do I have to take and which could be possible pitfalls in regard to bricking the camera accidentially?
Thank you.
Kind regards
Catfriend1

To remove the hack you have to delete the /home/yi-hack directory and restore 2 files of the original firmware:

/home/app/init.sh
/home/base/tools/extpkg.sh

If you make a mistake editing the last two files the cams could brick.
I could create a "sd-unhack" but I need time.

Ok, reading through the yi hack scripts, I expect this could work:

/home/app/init.sh

Replace
sleep 2
./rmm &
sleep 2
#./mp4record &
#./cloud &
#./p2p_tnp &
#./oss &
#./watch_process &
/usr/sbin/telnetd &
/home/yi-hack/script/system.sh

by
sleep 2
./rmm &
sleep 2
./mp4record &
./cloud &
./p2p_tnp &
./oss &
./watch_process &

/home/base/tools/extpkg.sh

Should be okay to leave as it is (I don't have an unhacked original).

Here it is:
original.tar.gz
Remove /home/yi-hack and the symbolic link /home/yi-hack-v4

Cool, thanks :)

@Catfriend1
I forgot a detail.
There is another file "modified".
You have to rename /home/app/cloudAPI_real ---> /home/app/cloudAPI
overwriting the hacked /home/app/cloudAPI.

Ah yes , I noticed it beside reading the scripts. It's the cloud on/off :).

Probably it's a good idea to create a new "sd-unhack".

https://github.com/roleoroleo/yi-hack-Allwinner/wiki/Remove-the-hack

Great work, thanks :-)


Cannot load assets outside of webpack root with dynamic paths

There was a previous unresolved issue similar to this one: https://github.com/webpack/webpack/issues/580, but I don't think it has the same symptoms / cause.
In my code I have this setup of assets vs application code:
project/
    app.js
    webpack.config.js
    node_modules/
assets/
    img.png

In my app.js code I have a require statement:
var img = require("../assets/" + val + ".gif")

And I have this loader:
  {test: /\.gif$/, loader: 'url'},

According to http://webpack.github.io/docs/context.html#dynamic-requires (it would be nice if this had its own page, "context" is not an obvious place to look for require problems) this should work fine, and it should pull in the image. However, on build, it warns with this:
WARNING in ../common ^\.\/.*\.gif$
Module not found: Error: Cannot resolve module 'url' in /assets
@ ../assets ^\.\/.*\.gif$

And if you try it with the file loader:
WARNING in ../common ^\.\/.*\.gif$
Module not found: Error: Cannot resolve module 'file' in /assets
@ ../assets ^\.\/.*\.gif$

They aren't just warnings, when running the app the dynamic images do not show up in the browser, with the error:
Uncaught Error: Cannot find module './img.png'.

What gives? It looks like it's 404ing on the file based on the message, but the file exists...
Wait, maybe it can't find those loaders? But they exist, they're installed...
It works for paths inside the project...
It works for non-dynamic paths outside of the project...
I do not know the cause of this issue, however after much head-desk-banging I found out that if you do this in the parent folder…
npm install --save-dev url-loader -file-loader

…then it works! This is very strange, it looks like webpack is trying to read node_modules from the wrong folder. Or something. I'm really not sure. But this doesn't look like desired behavior either way?

Workaround is using the resolveLoader.root option.


copr: fix get_valid_int_prefix() to be compatible with TiDB(#13045)

Signed-off-by: guo-shaoge<EMAIL_ADDRESS>
What is changed and how it works?

Issue Number: Close #13045
What's Changed: Make get_valid_int_prefix() to be compatible with TiDB, i.e. cast function needs to give error/warning when input string is like float('0.3').

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
copr: fix get_valid_int_prefix() to be compatible with TiDB

/cc @wshwsh12

/run-all-tests

/run-check-issue-triage-complete

/merge

/test


Adding distance-map support

Reusing TimeMapResponse for this as the responses for time-map and distance-map appear to have the same structure.
Note: no_holes and snap_penalty are both valid params (according to the docs) that I didn't include here, because they're not included in other requests either. Let me know if we should add support for these params

Don't forget to add the test

I added support for snap_penalty for all endpoints in another PR. You can merge first and I'll add snap_penalty to discance-map.
As for no_holes, I guess we should add it to all endpoints, but in a separate PR.


CLAS: Add to deletion transport

Add transport entry when deleting object

hmm, has this never worked for CLAS?

judging by the number of issues in CI related to transports, very few projects have used AG to delete objects with transportable packages.

yea, in SAP systems we add stuff 😁


support certgen web service and client

Goal would be to allow issueing of certs to be off loaded to an external web service,
as to not require the root (CA) certs to be coupled to the proxy
A client could then also be made to talk to it. Ideally we would also allow rama-tls ServerAuth to make use of this client as an issuer, but it will require some thinking on how to do this without coupling it to a specific approach (e.g. http). One way could be by just having it work with a channel, where it sends to and receives the result back via a one shot 🤷

Blocked for now.
Cfr: https://github.com/plabayo/rama/issues/335#issuecomment-2564706840


Ensure that the index domain is last in the list of domains in T_{clean, noisy}_relation

#119 ensured that for a noisy relation with n domains that has a base relation with k <= n domains, the first k domains of the noisy relation are the same as those of the base relation.
For each relation (both clean and noisy) we also need to know which domain is the index/primary key, so I propose putting it last in the list of domains. (Putting it first is more intuitive, though that would require more of a refactor, maybe to put the base domains last -- I think that would be fine too, but I wouldn't necessarily prioritize it.)
This came up as I was working on #125 and I think we'll need it for Model 7 too.

Can you tell me more about this notion of the index or primary domain?  I don't see it mentioned anywhere in the GenDB document, and I'm also not seeing where in the code the last domain is currently treated any differently.
Or is this something that we will only need in the future?
The only thing I can say for sure is that currently, when PCleanSchemaHelper is computing the domains for the relations in a class, that class's name is used for the first domain in the list.

Sure -- I mean the id field of the tables in Figure 2, so the domain of each class that uniquely identifies entities (and is the only domain that has to take unique values). This should be the same as the domain that your last sentence refers to, whose name is the same as the class name.
This is necessary for sampling from the HIRM, since the Record class will have its primary key enumerated (and foreign keys, referring to other class, sampled from CRPs). More generally, it ensures that if School.city == 2, 2 uniquely identifies a row in the City table.

Unassigning myself for now since I've fixed this for models 5 & 6.  Up to you whether you want to keep it open for model 7.


refactor(config): allow to override readTsConfig

Summary

Allow to override readTsConfig
Expose _transformCfgStr, _configSet, _overriddenCompilerOptions

Test plan

Green CI
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
N.A.

Pull Request Test Coverage Report for Build 6286

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

Totals

Change from base Build 6281:
0.003%

Covered Lines:
1019

Relevant Lines:
1048

💛  - Coveralls


Delete modal and Rename modal does not work inside filemanager field.

Inside the filemanager field, the delete modal and the rename modal, does not work.
I suspect is has todo with two modal being opened at the same time. It might also has something to do with a portal being open inside another portal. I cannot quite tell yet.

Fixed on v2.5. Please check the last release and the new documentation.


Feature/168 family authorizations

This is my first issue in the frontend! I believe I have fulfilled waht was asked in the Issue description, but since I am new here, I may be susceptible to mistakes that this team are already taking into account. I have taken as reference the mock of figma and followed the expected functionality of the center_Exit_auth.
I am sure that some sections will not comply with the expected work, so please, if you find these problems, tell me ASAP so I can fix them!
As it is right now, when a user enters as a family and goes to the "Autoricaciones" section, two windows will be availible, already signed authorizations and lesson_events of their children that are avalible. From this second view, by specifying the allowed children, the file and the boolean value "are they allowed to go?", a new center_exit_auth will be created.
I hope it works as the general consensus expects it. If not, once again, tell me ASAP so I can correct it!

I created a user a with family role in api/auth/users and activated it but when i log in, in the frontend app I get a 403 error

Thank you for your feedback! You are right, I tried to fix codacy issues without checking functionality and messed up. Now it works as expected, plus I ahve significantly improved the styles of the section using Artificial Inteligence! Please check it again.
Also, this works with the "develop" version of the backend. Still, there is a fix that im looking into. For now, when you enter into any family (I have used the<EMAIL_ADDRESS>for testing) with the password "contraseña" you will recieve an error. This can be mended by either modifying the file NexONG_Backend/nexong/api/Authentication/views.py, to be precise the function UserApiViewSet with this code snippet:
class UserApiViewSet(ModelViewSet):
    http_method_names = ["get", "put", "delete", "patch"]
    serializer_class = UserSerializer
    queryset = User.objects.all()
    def get_permissions(self):
        if self.request.method == 'GET':
            return [allowAnyGet()]
        return [isAdmin()]

Or by using modheader, as claugp06 has told me.
Anyways, tell me if something is not correct!

I have also added right now the dates for all the sections, I did not realize they were missing in the first pass. Also I have checked that it works with all families!

Also, the expected use case is:
Login as family (any family!)
Enter in "Autorizaciones"
Check that it works as expected

Thank you for your input! I have changed the date format so that it is more user friendly. About your other requests:
Everything is set to the left in case the name or the description is too long. The current populate has very short descriptions, but the current system will be able to comfortably hold these longer texts.
I, too, have noticed the language change in that case. The error message is the default that comes from the backend. Though it is not official, I am certain that next week there will be an issue in the backend to mend this incongruencies (I will probably take care of it personally) so that the errors outputted are in Spanish. Though not currently, expect the app to output spanish error messages in the near future!

Hi! Thank you for your feedback. Adressing your concerns:
I cannot redirect to "Niños" as this page currently does not exist, but im sure that in the future this can be easily corrected.
I have justfied and made more responsive the text.
You cannot download that file because the populate does not include the file itself. It does, however, redirect to the correct download link, so that is expected functionality.
Thank you once again.


[Launcher] Shell Plugin commands should go through Run

When running ">" using shell plugin, it should go through Run, not command prompt. This will allow launcher to behave like run prompt.

@crutkas @jyuwono.   I'm a bit confused by what we actually want here.  When we created the issue I thought the fix was to force the code path to go through **  else if (_settings.Shell == Shell.RunCommand)  inside the shell plugin.  Which is simply to specify the default setting to be RunCommand.   However both of these will execute Process.Start.  So for example:
Before:
> ping www.bing.com would be executed as Process.Start("cmd.exe", {working_directory}, /k \"ping www.bing.com\");
If I change it:
> ping www.bing.com would be executed as Process.Start("ping", {working_directory}, /k \"ping www.bing.com\");  which is nonesense.

Is there an explicit use case that we are looking to solve?

One thing that does make sense to me would be to turn of 'Run as Admin' by default.  Especially since the users could just select the run as admin, or ctrl-shift-e on the command.

it wouldn't be.  it splits and does args so it would be the args, not the exe.  Look at lines 196-203
var parts = command.Split(new[] { ' ' }, 2);

I agree with turning off run as admin by default as we we can do that via shift-ctlr


added support to get headers directly from thrown exception if available

That would make Life so much easier. We need to add CORS headers to the exception to display them in an angularJs Frontend.

Someone already suggested this in #450. I think that patching data into exceptions isn't really the right way to go about this. I'd rather see a better custom errorhandler instead (#401 ?).
In your case, I think you should use Flask-CORS. You are probably running into the problem described in #229.

see #523


cleanup master - refactor charts

Open questions

The chart source folder (where Chart.yaml/values.yml is) can not be in the root of the repository (code link)
In case somebody has a helm chart in the root of the repo, this line will throw exception
Updating the version in Chart.yml and values.yml will work only for Chart.yaml as values.yaml doesn't have version as a pointer in json (code link)
Create chart.destionion folder if doesn't exist (code link)
Only new charts should be published, the code doesn't allow re-publishing with the same version (code link1, link2)
For armada, do we need to manage also appVersion ?

NOTE: gh-pages cleanup PR: https://github.com/ljubon/charts/pull/111
Demo workflows and runs

Storm (ljubon/charts repo) run and workflow
Armada charts example

ljubon/armada workflow  (invoking push workflow)
ljubon/armada run (invoking push workflow)
ljubon/charts publishing charts

Looks good. I guess that the TODOs that are visible in PR will be removed once public PR is opened right?

Correct, and thanks for exposing one - I'll do a cleanup of comments before public PR 🤖


Changing an actor's parent immediately after creation results in unpredictable behavior

If you create an Actor and change its parent before the created promise completes, things seem fine.  The update will be sent and the change will take effect in the Unity runtime as expected.  However, moments later when the object-spawned reply to the actor creation is received, it may reset the parentId property on the Actor to the original value.
Whether the object-spawned message contains the original or modified parentId appears to be dependent on timing.
If the parentId is overwritten with an incorrect value in this way, the defect may not be immediately obvious.  The Actor is parented on the Unity side as expected and the SDK just has the wrong parent value.  It becomes apparent when subsequent attempts to re-parent the Actor to its original parent are be ignored, because the incorrect parent value causes them to be treated as a noop.

Here is a quick example of an MRE that creates 2 parent nodes ("A" and "B") and 100 child nodes, reparented from A to B in this fashion, and labels each of those children with its parentId value.   Though all of the children are parented to B in Unity as expected, the text reveals the defect as a mix of "My parent is A" and "My parent is B".
app.ts:
import * as MRE from '@microsoft/mixed-reality-extension-sdk';

export default class HelloWorld {

	constructor(private context: MRE.Context, private baseUrl: string) {
		context.onStarted(async () => {

			const a = MRE.Actor.Create(context, {
				actor: {
					name: 'A',
					transform: {
						app: { position: { x: -0.5, y: 0, z: 0 } }
					},
					text: {
						contents: "A",
						height: 0.3
					}
				}
			});
			await a.created();

			const b = MRE.Actor.Create(context, {
				actor: {
					name: 'B',
					transform: {
						app: { position: { x: 0.5, y: 0, z: 0 } }
					},
					text: {
						contents: "B",
						height: 0.3
					}
				}
			});
			await b.created();

			const nodes: MRE.Actor[] = [];

			setInterval(() => {
				nodes.forEach(node => {
					node.text.contents = "My parent is " + node.parent.name
				});
			}, 0);

			for (let i=0; i<100; i++) {
				const node = MRE.Actor.Create(context, {
					actor: {
						name: 'Test Node ' + i,
						parentId: a.id,
						transform: {
							local: { position: { x: 0, y: -0.5 * (i + 1), z: 0 } }
						},
						text: {
							contents: "Initial text",
							height: 0.3
						}
					}
				});
				node.parent = b;

				nodes.push(node);
				await node.created();
			}

		});
	}
}

If we were to subsequently try to reparent these nodes back to A, only the ones labelled "My parent is B" would correctly move.
This bug can be reproduced with only a single child node, but is intermittent due to timing.  Using 100 nodes helps illustrates the random behavior.


Implement SpotPass data dumping and upload

This PR is looking to implement the following features:

[x] Dump SpotPass data to SD/USB
[x] Upload SpotPass tasks to a public archival API (https://bossarchive.raregamingdump.ca/api/upload/wup)

This was request by a user (@mistersheeple on Discord), it is deemed as important considering the Nintendo Network shutdown coming soon
EDIT: Most of the changes are formatting, even though I am using your provided editorconfig

I will now proceed to revert some of the formatting changes and implement actual menus for the upload procedure (as well as removing UDP logger)

There isn't only one file. There can be multiple, my fresh reset WiiU has 3 task.db files

Thanks for the pull request! I do have some suggestions, hope they all make sense. Just wanna keep the code clean and maintainable if this is a permanent feature to stay.

This probably won't be a permanent feature btw. The services shut down in two weeks so this is very temporary.

There isn't only one file. There can be multiple, my fresh reset WiiU has already 3 task.db files, it's probably around 1 global, and 1 per account

Ah, I didn't know. Then I guess the current approach is fine as long as its mostly temporary. I'll probably move it to the online files option once this is done. Can you still address some of the other changes?

IMO it's ready to be merged for a quick spotpass upload mechanism


Remove motd script from ceph monitor nodes

This script is doing more harm than good. When the cluster is in a bad
state, it takes forever to get the status and causes issues with ssh.

:+1: Good to merge once tests pass.

Please rebase on master to pick up required fixes to pass tests.


Do you plan to support freebsd?

FreeBSD does not have systemd, can airup be used as init system?

Yes, FreeBSD is supported by Airup.

Portability is one of Airup's design goals, and I have the personal plan to set up FreeBSD test machines. If any features necessary for FreeBSD are missing, I'm willing to implement them.


QIDO-RS menu on a study and series should query DICOM instances

When clicking on the QIDO-RS drop-down on a study or series the query is doing a study or series info query. This should be updated so it returns the Instances for the selected study or series.

Fixed in d9d3176cfaa1609e9b866999b31e168c18880365


Re-enable firefox tests

In #236 we disabled firefox tests, which failed spuriously in CI, in order to unblock the release of js-libp2p.
We should track down the source of test flakiness and re-enable the firefox tests.

has this been fixed? If so please link to PR

https://github.com/ChainSafe/js-libp2p-gossipsub/pull/259


Ferdium-Server Not Support arms server (ubuntu) - Docker issue

hi,
i got this error when deploy using Docker
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested
9bb2c072c942c03062f8136f33ef0abfc22c7526382

hope can support arm64/linux (ubuntu) server for ferdium
Thanks

Duplicate of #45


[JANSA] Add translated catalogs for Jansa branch

@miq-bot add_label internationalization

LGTM, I think. There are no new translations, just new strings.

LGTM, I think. There are no new translations, just new strings.

Yep, translations should come for the GA version, this is just for the alpha build.


new lesson: Quick Sort

⭐ Request Type
Add in-depth documentation for the lesson Quick Sort.
you can start your contribution from this file
Lesson Content
The lesson must contain at least these are the topics:

Introduction
Understanding the quick sort
implementation + illustration
code for Quick Sort
complexity analysis+stability
applications
Partition algorithm
implementation of partition algorithms
3-way quicksort
Quick sort vs Merge Sort

💻 Location
location of file is at:
section: 8
lesson: F
order: 8F
filename: quicksort.md
❌ Additional Info
You can take the help of other markdown files in the /lesson directory so that you get the idea of in which fashion you should write.
If you don't know how to do markdown, don't worry it is quite easy, just follow this docs-https://guides.github.com/features/mastering-markdown/

please assign this issue to me.

Yes sure!


request is missing required parameter: grant_type

Following instructions of this page, I keep getting this response when I try to acquire a token.
{
"error": "invalid_request",
"error_description": "AADB2C90083: The request is missing required parameter: grant_type.\r\nCorrelation ID: a83acc94-6e24-4a6a-8089-d32fad452716\r\nTimestamp: 2019-11-14 15:41:55Z\r\n"
}
I use REST Client extension in the VS Code to send this request (private data is masked with 'xxxxx'):
POST /xxxxx.onmicrosoft.com/oauth2/v2.0/token?p=b2c_1_xxx HTTP/1.1
Host: xxxxx.b2clogin.com
Content-Type: application/x-www-form-urlencoded
username=xxxxx&password=xxxxx&grant_type=password&scope=openid+xxxxx-xxxxx-xxxxx-xxxxx-xxxxx+offline_access&client_id=xxxxx-xxxxx-xxxxx-xxxxx-xxxxx&response_type=token+id_token
As you can see, the 'grant_type=password' is present in the form data. Why am I getting this error and how can I fix this?

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

ID: c54680bb-ff67-a9e2-4272-1710fe6eb863
Version Independent ID: 6d15d340-f556-6bb0-8b4c-f4535537189b
Content: Configure the resource owner password credentials flow in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/configure-ropc.md
Service: active-directory
Sub-service: b2c
GitHub Login: @mmacy
Microsoft Alias: marsma

@mghafoori Thanks for your feedback! We will investigate and update as appropriate.

Hi Saurabh. Did you get the chance to look into this issue?

@mghafoori I do not see any issue with your request but have you tried "Content-type": "application/x-www-form-urlencoded; charset=UTF-8" to check if it works.  Also, try to check if this works using Postman or fiddler.

I tried all above with no luck.

We abandoned this method completely.

I am also having the same issue, any update on this.

Please send GET request instead of POST. Docs say POST but it is working with GET request.